Breaks the image into parts 32x16 1/8 scan

Hello.
I'm trying to make one matrix (not in a chain) 1/8 scan work.
I used (One_Eight_1_8_ScanPane)

1. /*************************************************************************
2.  * Description:
3.  *
4.  * The underlying implementation of the ESP32-HUB75-MatrixPanel-I2S-DMA only
5.  * supports output to 1/16 or 1/32 scan panels - which means outputting
6.  * two lines at the same time, 16 or 32 rows apart. This cannot be changed
7.  * at the DMA layer as it would require a messy and complex rebuild of the
8.  * library's DMA internals.
9.  *
10.  * However, it is possible to connect 1/8 scan panels to this same library and
11.  * 'trick' the output to work correctly on these panels by way of adjusting the
12.  * pixel co-ordinates that are 'sent' to the ESP32-HUB75-MatrixPanel-I2S-DMA
13.  * library.
14.  *
15.  **************************************************************************/
16. #include "ESP32-HUB75-MatrixPanel-I2S-DMA.h"
17.
18. /* Use the Virtual Display class to re-map co-ordinates such that they draw
19.  * correctly on a 32x16 1/8 Scan panel (or chain of such panels).
20.  */
21. #include "ESP32-VirtualMatrixPanel-I2S-DMA.h"
22.
23. #define R1_PIN 25
24. #define G1_PIN 26
25. #define B1_PIN 27
26. #define R2_PIN 14
27. #define G2_PIN 12
28. #define B2_PIN 13
29. #define A_PIN 23
30. #define B_PIN 19
31. #define C_PIN 5
32. #define LAT_PIN 4
33. #define OE_PIN 15
34. #define CLK_PIN 16 //RX2
35.
36.   // Panel configuration
37.   #define PANEL_RES_X 32 // Number of pixels wide of each INDIVIDUAL panel module.
38.   #define PANEL_RES_Y 16// Number of pixels tall of each INDIVIDUAL panel module.
39.
40.
41.   #define NUM_ROWS 1 // Number of rows of chained INDIVIDUAL PANELS
42.   #define NUM_COLS 1 // Number of INDIVIDUAL PANELS per ROW
43.
44.   // ^^^ NOTE: DEFAULT EXAMPLE SETUP IS FOR A CHAIN OF TWO x 1/8 SCAN PANELS
45.
46.   // Change this to your needs, for details on VirtualPanel pls read the PDF!
47.   #define SERPENT true
48.   #define TOPDOWN false
49.
50.   // placeholder for the matrix object
51.   MatrixPanel_I2S_DMA *dma_display = nullptr;
52.
53.   // placeholder for the virtual display object
54.   VirtualMatrixPanel  *OneEightMatrixDisplay = nullptr;
55.
56.   /******************************************************************************
57.    * Setup!
58.    ******************************************************************************/
59.   void setup()
60.   {
61.     delay(250);
62.
63.     Serial.begin(115200);
64.     Serial.println(""); Serial.println(""); Serial.println("");
65.     Serial.println("*****************************************************");
66.     Serial.println("*         1/8 Scan Panel Demonstration              *");
67.     Serial.println("*****************************************************");
68.
69.
70.      // 62x32 1/8 Scan Panels don't have a D and E pin!
71.
72.      HUB75_I2S_CFG::i2s_pins _pins = {
73.       R1_PIN, G1_PIN, B1_PIN, R2_PIN, G2_PIN, B2_PIN,
74.       A_PIN, B_PIN, C_PIN,
75.       LAT_PIN, OE_PIN, CLK_PIN
76.      };
77.
78.     HUB75_I2S_CFG mxconfig(
79.                 PANEL_RES_X*2,              // DO NOT CHANGE THIS
80.                 PANEL_RES_Y/2,              // DO NOT CHANGE THIS
81.                 NUM_ROWS*NUM_COLS           // DO NOT CHANGE THIS
82.                 //,_pins            // Uncomment to enable custom pins
83.     );
84.
85.     mxconfig.clkphase = false; // Change this if you see pixels showing up shifted wrongly by one column the left or right.
86.
87.     //mxconfig.driver   = HUB75_I2S_CFG::FM6126A;     // in case that we use panels based on FM6126A chip, we can set it here before creating MatrixPanel_I2S_DMA object
88.
89.     // OK, now we can create our matrix object
90.     dma_display = new MatrixPanel_I2S_DMA(mxconfig);
91.
92.     // let's adjust default brightness to about 75%
93.     //dma_display->setBrightness8(96);    // range is 0-255, 0 - 0%, 255 - 100%
94.
95.     // Allocate memory and start DMA display
96.     if( not dma_display->begin() )
97.       Serial.println("****** !KABOOM! I2S memory allocation failed ***********");
98.
99.
100.     dma_display->clearScreen();
101.     delay(500);
102.
103.     // create OneEightMatrixDisplaylay object based on our newly created dma_display object
104.     OneEightMatrixDisplay = new VirtualMatrixPanel((*dma_display), NUM_ROWS, NUM_COLS, PANEL_RES_X, PANEL_RES_Y, SERPENT, TOPDOWN);
105.
106.   // THE IMPORTANT BIT BELOW!
107.     OneEightMatrixDisplay->setPhysicalPanelScanRate(ONE_EIGHT_32);
108.   }
109.
110.
111.   void loop() {
112. /*
113.       // What the panel sees from the DMA engine!
114.       for (int i=PANEL_RES_X*2+10; i< PANEL_RES_X*(NUM_ROWS*NUM_COLS)*2; i++)
115.       {
116.         dma_display->setBrightness8(10);
117.         dma_display->drawLine(i, 0, i, 7, dma_display->color565(255, 0, 0)); // red
118.         delay(10);
119.       }
120.
121.       dma_display->clearScreen();
122.       delay(1000);
123. */
124. /*
125.       // Try again using the pixel / dma memory remapper
126.       for (int i=PANEL_RES_X+5; i< (PANEL_RES_X*2)-1; i++)
127.       {
128.         OneEightMatrixDisplay->drawLine(i, 0, i, 7, dma_display->color565(0, 0, 255)); // blue
129.         delay(10);
130.       }
131. */
132.
133.       // Try again using the pixel / dma memory remapper
134.       int offset = PANEL_RES_X*((NUM_ROWS*NUM_COLS)-1);
135.       for (int i=0; i< PANEL_RES_X; i++)
136.       {
137.         dma_display->setBrightness8(30);
138.         OneEightMatrixDisplay->drawLine(i+offset, 0, i+offset, 7, dma_display->color565(0, 0, 255)); // blue
139.         OneEightMatrixDisplay->drawLine(i+offset, 8, i+offset, 15, dma_display->color565(0, 128,0)); // g
140.         OneEightMatrixDisplay->drawLine(i+offset, 16, i+offset, 23, dma_display->color565(128, 0,0)); // red
141.         OneEightMatrixDisplay->drawLine(i+offset, 24, i+offset, 31, dma_display->color565(0, 128, 128)); // blue
142.         delay(10);
143.       }
144.
145.       delay(1000);
146.
147.
148.       // Print on each chained panel 1/8 module!
149.       // This only really works for a single horizontal chain
150.       for (int i = 0; i < NUM_ROWS*NUM_COLS; i++)
151.       {
152.         OneEightMatrixDisplay->setTextColor(OneEightMatrixDisplay->color565(255, 255, 255));
153.         OneEightMatrixDisplay->setCursor(i*PANEL_RES_X+7, OneEightMatrixDisplay->height()/3);
154.
155.         // Red text inside red rect (2 pix in from edge)
156.         OneEightMatrixDisplay->print("Panel " + String(i+1));
157.         OneEightMatrixDisplay->drawRect(1,1, OneEightMatrixDisplay->width()-2, OneEightMatrixDisplay->height()-2, OneEightMatrixDisplay->color565(255,0,0));
158.
159.         // White line from top left to bottom right
160.         OneEightMatrixDisplay->drawLine(0,0, OneEightMatrixDisplay->width()-1, OneEightMatrixDisplay->height()-1, OneEightMatrixDisplay->color565(255,255,255));
161.       }
162.
163.       delay(2000);
164.       dma_display->clearScreen();
165.
166.   } // end loop

Here's what it looks like.
https://user-images.githubusercontent.com/120684690/215460383-1fab701e-27f2-4e3d-b058-3f5de8ba5cb0.mp4
How can this be fixed?. Newbie in programming. Help me please.

As I see, the matrix is plain 32x16 1/8 matrix...
Hmmm @mrfaptastic, what is a problem with such matrices? Working principle for any indoor panels where scan is half of height (32x16 1/8 ... 64x32 1/16 ... 80x40 1/20....128x64 1/32 ... etc) is exactly the same. Why the 64x32 is supported, but 32x16 is not?

Why the 64x32 is supported, but 32x16 is not?

Not to sound rude, but if I knew why these panels didn't work I'd have coded a solution probably.
I think they are different at the electrical level. The fact @SergyUA's panel isn't showing sometihng on all rows (even if duplicated or wrong), makes me think there's some pin mapping issue here.
@SergyUA - Looks like an electrical matter, not software, triple check your wring, or start testing individual pins to see if you can at least get something to be displayed on rows 5-8 and 28-32

Why the 64x32 is supported, but 32x16 is not?

Not to sound rude, but if I knew why these panels didn't work I'd have coded a solution probably.
I think they are different at the electrical level. The fact @SergyUA's panel isn't showing sometihng on all rows (even if duplicated or wrong), makes me think there's some pin mapping issue here.
@SergyUA - Looks like an electrical matter, not software, triple check your wring, or start testing individual pins to see if you can at least get something to be displayed on rows 5-8 and 28-32
Can you please test using SimpleTestShapes with default configuration and NOT using the One_Eight_1_8_ScanPanel example. Please provide video.
@SergyUA - Are you using these panels to support the war effort at all? If so I am happy to try and provide enhanced help with your needs.

Can you please test using SimpleTestShapes with default configuration and NOT using the One_Eight_1_8_ScanPanel example. Please provide video.

@mrfaptastic. Thank you.
I took your advice and uploaded (SimpleTestShapes) and here is the result.
https://user-images.githubusercontent.com/120684690/215828978-b4b0d2ef-7072-42df-ae2a-d44c1f1ccf9e.mp4

@SergyUA - Are you using these panels to support the war effort at all? If so I am happy to try and provide enhanced help with your needs.

No, not to support the military, but for the residents of the city where I live.

I took your advice and uploaded (SimpleTestShapes) and here is the result.

Looks good to me? I take it the only thing you changed in the example with the MATRIX_WIDTH and MATRIX_HEIGHT values?
I can't see why problem with this output? Seems like it is a normal 1/2 scan panel (scans two halves/parallel at the same time).... Just a 32x16 pixel one.

Looks good to me? I take it the only thing you changed in the example was the PANEL_RES_X and PANEL_RES_Y values?

Exactly.

I can't see a problem with this output now. Seems like it is a normal 1/2 scan panel (scans two halves/parallel at the same time).... Just a 32x16 pixel one

It looks like it. Although on the panel (on the sticker) 1/8. And first I tried the PxMatrix library. The panel worked at 1/8 scan.

It looks like it. Although on the panel (on the sticker) 1/8. And first I tried the PxMatrix library. The panel worked at 1/8 scan.

That's because the use of fractions to describe these panels is ambiguous. Nobody is consistent, especially the Chinese sellers on AliExpress.
What it SHOULD mean is that how many rows are updated in parallel at the same time as the display updates from top to bottom.
What most people seem to think it means is how many rows between the rows getting updated.
I even get this wrong in my panel descriptions of panels in this library!
For example, the OneEightMatrixDisplay example should have been called OneQuarterMatrixDisplay. Because it's about trying to make this library work with panels that update four lines at once.
What your panel actually is, and what this library supports natively is 1/2  (half) scan. That means, the panelist split in half and two rows are updated in parallel.
Now depending on the actual physical dimensions of a panel (for example let us use a 32 px high panel).... It is also described as a...
TL/dr:
1/2 * 32 = 16 or '1/16' scan panel, which is technically right but is also WRONG and confusing especially when used interchangeably. I need to write a ReadMe  on this, update all my examples to be consistent.

It looks like it. Although on the panel (on the sticker) 1/8. And first I tried the PxMatrix library. The panel worked at 1/8 scan.

That's because the use of fractions to describe these panels is ambiguous. Nobody is consistent, especially the Chinese sellers on AliExpress.
What it SHOULD mean is that how many rows are updated in parallel at the same time as the display updates from top to bottom.
What most people seem to think it means is how many rows between the rows getting updated.
I even get this wrong in my panel descriptions of panels in this library!
For example, the OneEightMatrixDisplay example should have been called OneQuarterMatrixDisplay. Because it's about trying to make this library work with panels that update four lines at once.
What your panel actually is, and what this library supports natively is 1/2  (half) scan. That means, the panel is split in half and two rows are updated in parallel.
Now depending on the actual physical dimensions of a panel (for example let us use a 16 px high panel).... It is also described as a...
1/2 * 16 = 8 or 1/8, scan panel...
... which is technically right but is also WRONG and confusing especially when used interchangeably. Your panel is still a half or '1/2' scan panel.
I need to write a ReadMe  on this, update all my examples to be consistent.

OneEightMatrixDisplay is now dead and has been replaced with 'Four_Scan_Panel' example. Library's README has been updated as well.
This is to avoid confusion.

because the use of fractions to describe these panels is ambiguous. ...
it SHOULD mean is that how many rows are updated in parallel at the same time
closely, but not correct

most people seem to think it means is how many rows between the rows getting updated.
nothing similar :)

@mrfaptastic
Descriptions 1/2 1/4 1/8 mean what part of the whole matrix glows at each moment. There is nothing ambiguous in it. This is the common terminology
OP matrix is exactly the matrix 1/8. Trying to call such matrices 1/2 only confuses people.

No, not to support the military, but for the residents of the city where I live.

Let me know if there's a local fundraiser I can contribute to.


playsound whenever user interaction is required

This PR adds a sound notifications to every function that requires user interaction.
This was overlooked in the 0.4.5 update.
This PR also removes the on_lookaway function and instead uses the on_idle function for that state, as they should do the same thing currently. This might change with new needs of fullauto (e.g. open doors).
Also, this PR moves the semifisher engine start beep from hotkey code to engine code. This comes with the drawback that it will beep both when pressing f9 or the UI-Button.
It also adds a second beep for when the engine stops.

Both requests are reasonable. Will adapt them asap


Line 200 '||' token is invalid

Tried to run the script on my machine but was unable to. Kept running into the error shown below

I was able to fix it by changing || to -or.

Another issue I am facing is that after using this script, Samsung continuity service is just broken. It won't start and won't open no matter what. Tried re-installing it in every possible way but nothing worked. In MS store the app does not even show the open button. Multi control works fine tho, i can use my mouse and keyboard on the other device but i cannot copy on one device and have it paste on another.

Tried to run the script on my machine but was unable to. Kept running into the error shown below
I was able to fix it by changing || to -or.

This may be because of a difference in the PowerShell version being used. I was using PowerShell v7 for this, and it appears that the logical operators && and || among others were introduced in v7 and are not present in v5. I will check for other compatibility issues.

Yeah, i think it was cause of that, but the thing is I used the command you posted to grab the latest powershell. Very weird, but could be the install was bonked somehow. Another (and now bigger) issue I have is that Samsung's continuity service just refuses to work. It worked for a day once and now has just fully stopped. I can use multi control but i can use the copy and paste feature between laptop and phone.

I also cannot launch the continuity service on my laptop, I think it needs another app to launch it, possibly Samsung Settings that I cannot get to work.


Add a definition for grpc_global_wakeup_fd in ev_posix.c

This fixes the linker error mentioned in #8766. Running make on MacOS will still not build, but now the problem lies in recent changes to Cronet. This PR will unblock @gcasto from finishing his Cronet gRPC fix discussed in #8684.
This fix does allow clients and servers that don't use cronet (like make interop_server and make interop_client) to be built (this was not the case before, since the linker error broke everything).
This change does not affect the status of building on Linux at all.

A little more context. This was broken in #8680. MacOS's extern grpc_global_wakeup_fd previously linked to this line

Cherry picked some changes from #8684, and now MacOS builds with no errors

Thank you Noah. We are in need of this fix here at mobile :)

Merged in #8862 since those changes clashed with mine. This fix is still needed to get the build working on MacOS again

Moved the relevant changes to #8869. This can now be closed


Why is there a azurecli login method, but no azureps method?

I just feel like I have to ask this question, since I don't see it answered elsewhere!
Assuming that I use the az CLI, the azurecli option is by far the least intrusive authentication option, but although I'm working with Azure all day, I'm usually using Azure PowerShell, and I'm basically always logged in that way, but not always logged in with the az CLI.

because there isn't any SDK providing such option. The one we use is https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/azidentity/azure_cli_credential.go


listener: in place filter chain update

Commit Message:
Provide new listener update path which could update the filter chains without draining all the connections of the old listener.
The in place filter chain update flow is an optimization of listener update. If the supportUpdateFilterChain() passes and runtime
"envoy.reloadable_features.listener_in_place_filterchain_update" is not explicitly disabled,
the existing connections may not be drained if the owning filter chains are not updated in the new listener config.
Additional Description:
patch 3/N of #9773
Follow up of #10528
This PR is providing new listener update path guarded by supportUpdateFilterChain()
The in place filter chain update flow is an optimization of listener update. If the above check pass and runtime
"envoy.reloadable_features.listener_in_place_filterchain_update" is not explicitly disabled,
the existing connections may not be drained if the owning filter chains are not updated in the new listener config.
When the feature is disabled by the runtime, the following listener update won't use the optimized path.
The new update flow starts with the construction of ListenerImpl.
ListenerImpl will share PerListenerFactoryContext, and the warmup callback utilize the below worker interface

addListener() with overridden listener
removeFilterChains()
Those 2 methods are landed in #10528

Risk Level: Medium
Connection life time changes by default.  Can be disabled by setting "envoy.reloadable_features.listener_in_place_filterchain_update" to false.
Testing: integration test,unit test, runtime flag test.
Docs Changes:
Release Notes: see current.rst

@mattklein123 Ping. Could you leave some comment for the first round?

I had an experimental branch under the philosophy of reusing the ListenerImpl. The statement is way more complicated than the current one. Let me see if I can recover it and bring it to discussion.

Yeah maybe spend a bit of time and time box it and then we can look at it together? I'm OK to ship and iterate if we need to, but per my other comment I do think that re-using an existing ListenerImpl will be a cleaner long term solution.

@mattklein123 The integration test is probably fragile on mac since it depends on libevent behavior. Can we disable it on macos?

@mattklein123 The integration test is probably fragile on mac since it depends on libevent behavior. Can we disable it on macos?

No. For this type of core feature it needs to work on OSX. I don't know why the test would depend on libevent behavior. I will look when I review now but please fix.

@lambdai please clean up the PR description per the new template. We won't merge without that done.

No. For this type of core feature it needs to work on OSX. I don't know why the test would depend on libevent behavior. I will look when I review now but please fix.

The prod code doesn't rely on OSX nor libevent feature but the integration test does. Let me see if I can improve the test

Update the description by splitting
Commit Message and Additional Description
The runtime key is mentioned as well

Still I don't understand why we need the test on both v4 and v6 for this high level feature.

Because some people do not run v4 tests (including Google). This is non-negotiable so let's please move on.

Spent 4 hours on learning ssl connections... almost there

Because some people do not run v4 tests (including Google). This is non-negotiable so let's please move on.

The analogy is that I don't have to run a string test on both v4 and v6. This PR is a string test IMHO
Anyway I revert the integration tests and now the newly added LDS tests locates in single xds_integration_tests.cc.
The new integration test helps me locate a bug :)

@mattklein123 Integration test is now on both v4 and v6. What is the next step?

@mattklein123

is_stopped flag is added. I give up in place update in this situation. People will see full listener update. Not sure if it is the best solution but it will be the less surprising.
filterChainOnlyUpdate: My intention is to add the notes here if there are further accounting fields in the Listener message (e.g, a time, a version number), we should update this function, to ignore those fields beside filter chain. I know our general principle is not to worry about the unlikely change but this one need somewhat attention.
The newly added integration test has many duplicated codes, among my newly added cases and among neighbor test cases. I plan to refactor either after this PR. If you feel it is too ugly and some refactor must be done in this PR, please let me know, thanks!

I may have some sense of uniting RawConnectionDriver and SslClient.
Leave TODO and will do in next round

@mattklein123 Is RawConnectionDriver in https://github.com/envoyproxy/envoy/pull/10998 matching your expectation?

Yeah!
@mattklein123 Thank you for the guidance these days!
I will clean up some TODOs in a slower pace :)


vit_base_resnet50_384 not found

inside results folder all the recent updated csv file contains this model name "vit_base_resnet50_384" but i can't use it,i get model not found error.

Please see #271 .. it's on a branch w/ a pending PR right now... I'm still testing and adding some better support for imagenet21k right now on that branch so not ready to merge to master. The models and weights are there on that branch though, I ran the results update from that branch. It takes a few days to go thorugh it now so didn't want to run again once I merge :)

Please see #271 .. it's on a branch w/ a pending PR right now... I'm still testing and adding some better support for imagenet21k right now on that branch so not ready to merge to master. The models and weights are there on that branch though, I ran the results update from that branch. It takes a few days to go thorugh it now so didn't want to run again once I merge :)


whats the recipe of crafts gow can I look them

I cant craft things because IDK recipes of crafts

You might want to get a mod like JEI or REI which allows you to view all the recipes in the game.


An unreachable panic statement should not be an error

Please refer to : https://github.com/ballerina-platform/ballerina-spec/issues/1016

This is not a bug, an improvement done to the next update


Initial commit for Pinot Minion

Added basic Minion framework including PinotTaskManager, PinotHelixTaskResourceManager and ClusterInfoProvider
Added PinotTaskGenerator and PinotTaskExecutor interface
Implement DummyTaskGenerator and DummyTaskExecutor as example
Added a MinionClusterIntegrationTest to test basic functionality

Codecov Report

Merging #1352 into master will increase coverage by 0.05%.
The diff coverage is 75.52%.

@@            Coverage Diff             @@
##           master    #1352      +/-   ##
==========================================
+ Coverage   66.91%   66.96%   +0.05%
==========================================
  Files         781      794      +13
  Lines       38183    38362     +179
  Branches     5014     5024      +10
==========================================
+ Hits        25549    25691     +142
- Misses      10963    10994      +31
- Partials     1671     1677       +6

Impacted Files
Coverage Δ

...inkedin/pinot/minion/exception/FatalException.java
0% <0%> (ø)

...elix/core/minion/generator/DummyTaskGenerator.java
100% <100%> (ø)

...m/linkedin/pinot/common/utils/CommonConstants.java
37.2% <100%> (-1.43%)
:arrow_down:

.../java/com/linkedin/pinot/minion/MinionStarter.java
100% <100%> (ø)

...nkedin/pinot/minion/executor/BaseTaskExecutor.java
100% <100%> (ø)

...pinot/minion/exception/TaskCancelledException.java
100% <100%> (ø)

...kedin/pinot/minion/executor/DummyTaskExecutor.java
100% <100%> (ø)

...troller/helix/core/minion/ClusterInfoProvider.java
50% <50%> (ø)

.../linkedin/pinot/common/config/PinotTaskConfig.java
50% <50%> (ø)

...ntroller/helix/core/PinotHelixResourceManager.java
57.25% <66.66%> (+0.04%)
:arrow_up:

... and 32 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 c25c560...c8b516d. Read the comment docs.


Adding ability to manage tag protections via settings

This should allow tag protection

Fixes #772

also worth considering: https://github.com/repository-settings/app/issues/772#issuecomment-1986455617


Where is training code?

Hello, thank you for you work! I could not find your training code to reproduce your results. Where can I find this?

Duplicate of #3.


Terminal don't work in VS code inside a WSL2G distro Ubuntu

I did install wsl --install to enable wsl in latest insider release. After enable I install vs code. VS code installed and start running.
Every time I try to open terminal it's open a new instance. The terminal feature is not working. Please look.

VS Code Version: Latest
OS Version: WSL2G Graphic enabled WS2 Ubuntu

Steps to Reproduce:

setup the install and install code from .deb file.
Type CTRL + ~ to open terminal, it's open a instance rather open terminal.

Does this issue occur when all extensions are disabled?: Yes/No

What are the errors in Help -> Open Dev Tools -> Console?

give me sometime, I will check over weekends. Thanks

Please provide a screenshot or a gif illustrating the problem (attaching images won't work via email, you need to go to github.com). See this wiki page for more general info on how to report actionable terminal issues.
/label "needs more info" integrated-terminal

I make a small video demonstrate how it's react when I use terminal https://www.youtube.com/watch?v=SOShl3Vb4ew

I unregistered the distro in last few days. tried in new one. I install vs code in my pc so everytime I try to sync the sign in open back the vs code on windows.
the settings is showing {} but issue is still same.

@deepak1556 have you seen this where some processes in wslg will end up launching a new window because the electron executable is used?

Not aware of a similar issue, @anirugu can you launch the app like NODE_DEBUG='child_process' code --verbose from terminal and perform the above steps. Attach the log that is printed, thanks!


Make rmw_zenoh_cpp pass all core tests

This issue will track the tests failures in some ros core packages:

Package
Failing test
Flaky

rcl
test_events__rmw_zenoh_cpp

rcl_action
:heavy_check_mark:

rcl_lifecycle
:heavy_check_mark:

rclcpp
test_qos_event__rmw_zenoh_cpp

rclcpp_lifecycle
:heavy_check_mark:

rclcpp_action
:heavy_check_mark:

test_rclcpp

test_services_cpp__rmw_zenoh_cpp

test_events__rmw_zenoh_cpp
111: [INFO] [1724407583.785118284] [rmw_zenoh_cpp]: Successfully connected to a Zenoh router with id e3fd1217688e472236c9fabe6ce2acb.
111: /home/ahcorde/ros2_rolling/src/ros2/rcl/rcl/test/rcl/test_events.cpp:137: Failure
111: Expected equality of these values:
111:   ret
111:     Which is: 3
111:   0
111: provided event_type 7 is not supported by rmw_zenoh_cpp, at /home/ahcorde/ros2_rolling/src/ros2/rmw_zenoh/rmw_zenoh_cpp/src/rmw_event.cpp:54
111:
111:
111: >>> [rcutils|error_handling.c:108] rcutils_set_error_state()
111: This error state is being overwritten:
111:
111:   'provided event_type 7 is not supported by rmw_zenoh_cpp, at /home/ahcorde/ros2_rolling/src/ros2/rmw_zenoh/rmw_zenoh_cpp/src/rmw_event.cpp:54'
111:
111: with this new error message:
111:
111:   'event's implementation is invalid, at /home/ahcorde/ros2_rolling/src/ros2/rcl/rcl/src/rcl/event.c:223'
111:
111: rcutils_reset_error() should be called after error handling to avoid this.

test_qos_event__rmw_zenoh_cpp
130: [INFO] [1724407517.768452371] [rmw_zenoh_cpp]: Successfully connected to a Zenoh router with id e3fd1217688e472236c9fabe6ce2acb.
130: [ RUN      ] TestQosEvent.test_publisher_constructor
130: unknown file: Failure
130: C++ exception with description "Failed to initialize event: provided event_type 7 is not supported by rmw_zenoh_cpp, at /home/ahcorde/ros2_rolling/src/ros2/rmw_zenoh/rmw_zenoh_cpp/src/rmw_event.cpp:54" thrown in the test body.
130:
130: [  FAILED  ] TestQosEvent.test_publisher_constructor (6 ms)
130: [ RUN      ] TestQosEvent.test_subscription_constructor
130: unknown file: Failure
130: C++ exception with description "Failed to initialize event: provided event_type 1 is not supported by rmw_zenoh_cpp, at /home/ahcorde/ros2_rolling/src/ros2/rmw_zenoh/rmw_zenoh_cpp/src/rmw_event.cpp:109" thrown in the test body.
130:
130: [  FAILED  ] TestQosEvent.test_subscription_constructor (2 ms)
130: [ RUN      ] TestQosEvent.test_default_incompatible_qos_callbacks
130: /home/ahcorde/ros2_rolling/src/ros2/rclcpp/rclcpp/test/rclcpp/test_qos_event.cpp:207: Failure
130: Expected equality of these values:
130:   "New subscription discovered on topic '/ns/test_topic', requesting incompatible QoS. " "No messages will be sent to it. Last incompatible policy: DURABILITY_QOS_POLICY"
130:     Which is: "New subscription discovered on topic '/ns/test_topic', requesting incompatible QoS. No messages will be sent to it. Last incompatible policy: DURABILITY_QOS_POLICY"
130:   pub_log_msg
130:     Which is: ""

test_services_cpp__rmw_zenoh_cpp
# 1: [test_services_client_cpp-2] [ERROR] [1724407062.211503664] [rclcpp]: caught std::exception exception in GraphListener thread: rcl_wait unexpectedly timed out
101: [test_services_client_cpp-2] terminate called after throwing an instance of 'std::runtime_error'
101: [test_services_client_cpp-2]   what():  rcl_wait unexpectedly timed out

With https://github.com/ros2/rclcpp/pull/2626, I can get all rclcpp and rclcpp_lifecycle tests to run without any crashes!
However these tests from rclcpp still fail. Some should be fixed by https://github.com/ros2/rclcpp/pull/2626 while others require some fixes to rmw_zenoh_cpp.
The following tests FAILED:
	 33 - test_intra_process_manager (Failed)
	 41 - test_node_interfaces__get_node_interfaces (Failed)
	 44 - test_node_interfaces__node_graph (Failed)
	 62 - test_publisher (Failed)
	 67 - test_qos (Failed)
	 95 - test_executors (Failed)
	105 - test_events_executor (Timeout)
	108 - test_wait_set (Failed)
	132 - test_qos_event__rmw_zenoh_cpp (Failed)

Superseded by https://github.com/ros2/rmw_zenoh/issues/286


Feature/swift pm support

What did you change and why?

Added support for Swift Package Manager
Modified the folder structure to make it Swift PM compatible
Deleted Sources/TrueTime.h and Sources/TrueTime.m and replaced the declarations in Swift

Potential risks introduced?
What tests were performed (include steps)?

Tested within an app that previously used TrueTime via Carthage

Checklist

[ ] Unit/UI tests have been written (if necessary)
[x] Manually tested

I would love to see SPM support.

I have this which I will be keeping forever (or until the main repo adds SwiftPM support)


Run onTestSkipped for each value from data provider

TestNG Version

7.4.0

Expected behavior
Expected test case output:
Skipped test2(foo)
Skipped test2(bar)

Actual behavior
Actual test case output:
Skipped test2()

Is the issue reproducible on runner?

Gradle
IntelliJ

Test case sample
@Listeners(Example.MyListener.class)
public class Example {
    @Test
    void test1() { Assert.fail(); }

    @Test(dataProvider = "items", dependsOnMethods = {"test1"})
    void test2(String value) { }

    @DataProvider(name = "items")
    Object[][] items() { return new Object[][]{{"foo"}, {"bar"}}; }

    public static class MyListener implements IResultListener2 {
        @Override
        public void onTestSkipped(ITestResult result) { handle(result); }

        private void handle(ITestResult result) {
            Object param = result.getParameters().length != 0 ? result.getParameters()[0] : "";
            System.out.printf("Skipped %s(%s)\n", result.getName(), param);
        }
    }
}

I need handle method to be called for each value from data provider.
If I cannot configure it that way, I am ok to cover it in onTestSkipped but I would like to know what would be the easiest method to invoke data provider in a generic way (without a direct call to items method).

@karolrom-intel - Its not clear what you are asking for. Your title says something else, but the description says something else. Also we would like to confine questions to the testng-users google groups mailing list and would like to restrict bugs/new features alone to the github issues page.

I think the expected output is
Skipped test2(foo)
Skipped test2(bar)

where the actual output is
Skipped test2()

I think the asked behavior makes sense but I've no idea what can be the impacts.
As always, the new behavior should be opt-out by default.

@juherr - A data driven test that has an upstream dependency on a failed method, is going to fail for all iterations for sure.
We already provide the distinction of a method being a data driven test method which should suffice in the reports.
Dumping the parameters associated with every iteration (obtained from a data provider) and then marking it as a skipped for all those instances, just wondering what would that be serving as a purpose?

@karolrom-intel could you describe your usage and why you need it?

I use the listener to build custom reports but I think this problem is common for existing reporters.
Each report includes different statistics like the total number of tests or pass rate.
I count each iteration of the data-driven test as a separate test and they are of the same importance as other tests.
people who know nothing about TestNG but read our reports, see that number of tests fluctuates, and the pass rate is unreliable.

@karolrom-intel - TestNG evaluates a data driven test and reports it properly ONLY when there are passes and failures but never skips.
Are you asking that skipped data driven tests also be reported, like regular skipped tests ?

Yes, that is exactly what I mean.

@karolrom-intel - This is now part of 7.5-SNAPSHOT and will be available in the upcoming release as well.
To use this feature, you would to set the configuration includeAllDataDrivenTestsWhenSkipping.
For e.g., if you were to be enabling this on surefire plugin, your configuration would look like below (In the Maven world)
<configuration>
	<properties>
		<property>
		  <name>includeAllDataDrivenTestsWhenSkipping</name>
		  <value>true</value>
		</property>
	</properties>
</configuration>


RuntimeError: Missing the amt.tools configuration

Hi,
I have tried to install AMT, however when i tried it, i have the following error :
RuntimeError: Missing the amt.tools configuration
Here my gitconfig :
[merge]
    tool = amt
    conflictstyle = diff3

[mergetool "amt"]
    cmd = /Users/benjamin/Development/AutoMergeTool/amt.py -b "$BASE" -l "$LOCAL" -r "$REMOTE" -m "$MERGED"

[amt]
        tools = "gen_simplify;gen_woven;gen_additions;gen_deletions;meld"
        verbose = true

Here my tracelog :
git mergetool
Merging:
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java

Normal merge conflict for 'app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java':
  {local}: modified file
  {remote}: modified file
Traceback (most recent call last):
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 215, in <module>
    result = merge(config, args, launcher, analyser)
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 177, in merge
    raise RuntimeError('Missing the {0}.{1} configuration'.format(SECT_AMT, OPT_TOOLS))
RuntimeError: Missing the amt.tools configuration

Could you help me ?
Thanks in advance,
Benjamin
PS : i have python 3 installed :
python --version
Python 3.6.2

Hi Benjamin,
I'm going to take a look into this, just a quick question, it seems that in your git config you have different padding in the different sections. Can you try updating your git config so that all options use the same number of space at  the begining of the file ? I'm not sure it's the cause, but I want to check that first

Can you open the amt.py file and replace the last section with this, then run the tool again ?
if __name__ == '__main__':
    args = parse_arguments()

    local_config = find_local_config(args.merged)
    print(" [DBG] Local Config : " + local_config)
    config = read_config(local_config)
    print(repr(config.sections()))
    launcher = ToolsLauncher(config)
    analyser = ConflictedFileAnalyser()
    result = merge(config, args, launcher, analyser)

    if result == 0:
        clean_reports(args.merged)

    sys.exit(result)

Hi,
I have changed the padding and the last section, now i have the following tracelog :
git mergetool
Merging:
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java
Normal merge conflict for 'app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java':
{local}: modified file
{remote}: modified file
Traceback (most recent call last):
File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 212, in
print(" [DBG] Local Config : " + local_config)
TypeError: must be str, not NoneType

Ok so that narrows it down. can you remove the print(" [DBG] Local Config : " + local_config) line from your amt.py ?
Sorry for the slow debug process, I need to find where this is coming from, and without being able to reproduce it :/

No problem, i understand and i'm happy to help ;)
I have remove the line, i have to try again ? In doubt :
git mergetool
Merging:
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java

Normal merge conflict for 'app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java':
  {local}: modified file
  {remote}: modified file
['core', 'difftool "sourcetree"', 'mergetool "sourcetree"', 'user']
Traceback (most recent call last):
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 216, in <module>
    result = merge(config, args, launcher, analyser)
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 177, in merge
    raise RuntimeError('Missing the {0}.{1} configuration'.format(SECT_AMT, OPT_TOOLS))
RuntimeError: Missing the amt.tools configuration
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java seems unchanged.

Ok so apparently it's loading a git config file which might not be the one you edited.
Can you paste the path to the gitconfig file you edited ?
also in amt.py, can you add print(GLOBAL_CONFIG) on the line after print(repr(config.sections())) ?

Git config in the project Android : /Users/benjamin/Development/SPMAndroid/.git/gitconfig
Location of AutoMergeTool : /Users/benjamin/Development/AutoMergeTool
git mergetool
Merging:
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java

Normal merge conflict for 'app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java':
  {local}: modified file
  {remote}: modified file
['core', 'difftool "sourcetree"', 'mergetool "sourcetree"', 'user']
/Users/benjamin/.gitconfig
Traceback (most recent call last):
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 217, in <module>
    result = merge(config, args, launcher, analyser)
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 177, in merge
    raise RuntimeError('Missing the {0}.{1} configuration'.format(SECT_AMT, OPT_TOOLS))
RuntimeError: Missing the amt.tools configuration

Ok, so it's definitely not finding the project's gitconfig (which made me realize I made an error : the  local config file should be .git/config and not .git/gitconfig... I'm fixing that later).
Can you replace the find_local_config method (around line 50) by the following ?
def find_local_config(config_file):
    """
    Finds the nearest parent directory where there is a .git folder
    """
    print(" [DBG] Looking for local config in " + repr(config_file))
    parent = os.path.dirname(config_file)
    if parent == config_file:
        print(" [DBG] Found myself in a loop")
        return None

    git = os.path.join(parent, ".git")
    if os.path.exists(git):
        local_config_path = os.path.join(git, LOCAL_CONFIG_NAME)
        print(" [DBG] Found the .git folder, checking path " + repr(local_config_path))
        if os.path.exists(local_config_path):
            print(" [DBG] It exists !")
            return local_config_path
        else:
            print(" [DBG] It doesn't exist !")
            return None
    else:
        print(" [DBG] Moving up...")
        return find_local_config(parent)

 git mergetool
Merging:
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java

Normal merge conflict for 'app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java':
  {local}: modified file
  {remote}: modified file
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl/spm/core/activities/modules/notifications'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl/spm/core/activities/modules'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl/spm/core/activities'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl/spm/core'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl/spm'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu/bgl'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java/lu'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main/java'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src/main'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app/src'
 [DBG] Moving up...
 [DBG] Looking for local config in '/Users/benjamin/Development/SPMAndroid/app'
 [DBG] Found the .git folder, checking path '/Users/benjamin/Development/SPMAndroid/.git/gitconfig'
 [DBG] It doesn't exist !
['core', 'difftool "sourcetree"', 'mergetool "sourcetree"', 'user']
/Users/benjamin/.gitconfig
Traceback (most recent call last):
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 223, in <module>
    result = merge(config, args, launcher, analyser)
  File "/Users/benjamin/Development/AutoMergeTool/amt.py", line 183, in merge
    raise RuntimeError('Missing the {0}.{1} configuration'.format(SECT_AMT, OPT_TOOLS))
RuntimeError: Missing the amt.tools configuration
app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java seems unchanged.

can you run ls -l in your .git folder to check the permissions ? Apparently the OS is reporting the file as "non existent"

cd .git
ls -l
-rw-r--r--    1 benjamin  staff      41 Aug  4 10:07 COMMIT_EDITMSG
-rw-r--r--    1 benjamin  staff    1582 Aug  4 10:06 FETCH_HEAD
-rw-r--r--    1 benjamin  staff      37 Aug  4 10:11 HEAD
-rw-r--r--    1 benjamin  staff      41 Aug  4 10:11 MERGE_HEAD
-rw-r--r--    1 benjamin  staff       0 Aug  4 10:11 MERGE_MODE
-rw-r--r--    1 benjamin  staff    1548 Aug  4 10:11 MERGE_MSG
-rw-r--r--    1 benjamin  staff      41 Aug  4 10:11 ORIG_HEAD
-rw-r--r--    1 benjamin  staff     826 Aug  4 08:57 config
-rw-r--r--    1 benjamin  staff  330332 Aug  4 10:11 index
drwxr-xr-x    4 benjamin  staff     136 Jul 21 14:48 logs
drwxr-xr-x  249 benjamin  staff    8466 Aug  4 08:58 objects
-rw-r--r--    1 benjamin  staff    4806 Jul 21 14:48 packed-refs
drwxr-xr-x    6 benjamin  staff     204 Aug  4 08:58 refs
-rw-r--r--@   1 benjamin  staff     156 Aug  4 10:07 sourcetreeconfig

Hmm ok so indeed this is what I expected your local git config file is .git/config
I updated the repository, so you can download the latest version and replace it on your computer, or just change the value of LOCAL_CONFIG_NAME line 13 :
LOCAL_CONFIG_NAME = 'config'

Ok, it looks like it works, but apparently I doesn't have gen_simplify and meld, any idea how to resolve it ?
Normal merge conflict for 'app/src/main/java/lu/bgl/spm/core/activities/modules/notifications/NotificationsFragmentImpl.java':
  {local}: modified file
  {remote}: modified file
 [AMT] → Trying merge with "gen_simplify
 [AMT] — Ignoring tool "gen_simplify (unknown tool)
 [AMT] → Trying merge with gen_woven
 [AMT] ✗ gen_woven didn't solve all conflicts
 [AMT] → Trying merge with gen_additions
 [AMT] ✗ gen_additions didn't solve all conflicts
 [AMT] → Trying merge with gen_deletions
 [AMT] ? gen_deletions returned, but this should not be trusted
 [AMT] ✗ gen_deletions didn't solve all conflicts
 [AMT] → Trying merge with meld"
 [AMT] — Ignoring tool meld" (unknown tool)
 [AMT] ⚑ Sorry, it seems we can't solve it this time

In your config file, just remove the double quotes :
tools = gen_simplify;gen_woven;gen_additions;gen_deletions;meld

Merci infiniment ;) Now i can try the tool :)


Hide next scheduled start part of status

My status line is always showing Docked - Next scheduled start 01/01/1970, 01:00, because I don't use the schedule from Worx. I'm only interested in the first part of the status and show_status is showing the entire line, or nothing.
Could an option be added to hide the next scheduled start part of the status?

Fixed in https://github.com/Barma-lej/landroid-card/releases/tag/0.2.5


DOC Add crates.io badge to the readme

Makes it a bit easier to know which is currently the latest version.
Edit: the rendered version is here.

The lint issue is unrelated to this PR and would be fixed e.g. in https://github.com/vbarrielle/sprs/pull/175

Good idea, thanks!


run "./build/new_college ./scripts/data/surf64_k10L6.voc.gz ./scripts/data  "

Hi~, When I ran this command"./build/new_college ./scripts/data/surf64_k10L6.voc.gz ./scripts/data ", I encountered some errors：
Loading vocabulary from data/surf64_k10L6.voc.gz
terminate called after throwing an instance of 'std::__cxx11::basic_string<char, std::char_traits, std::allocator >'
Aborted (core dumped)
Looking forward to your reply soon!

Are you using the provided docker and sure you're not mistyping anything? That command is tested in CI so should be good.

Please refer to the CircleCI scripts in case you run into any issue with the readme: https://github.com/nicolov/simple_slam_loop_closure/blob/master/.circleci/config.yml


set the default service to ClusterIP

ES cluster are in general only accessible internally.
Use it as a more default secure configuration.

I definitely don't disagree with you, but given that this repo is intended primarily to be a Helm port of https://github.com/pires/kubernetes-elasticsearch-cluster, I'm thinking that it may be best to keep the default here as the same as the default there.


Swagger gh pages

Pull Request Test Coverage Report for Build a33088080c24136efbd884300bc2fe75b4a1356e-PR-3

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.339%

Totals

Change from base Build 0524ff7b5cbbfcd5792b42afedd5aaf641acb1da:
0.0%

Covered Lines:
225

Relevant Lines:
236

💛  - Coveralls


YouTube play button behind Plyr play button.

I've noticed recently that the YouTube play button is now sitting behind the Plyr play button. Seems to be that the query parameter 'controls' passed you YouTube isn't working as expected. Any plans/ideas on how to remove it within Plyr? You can see an example below (which occurs on the Plyr demo page):

Take it due to the iframe same-domain policy this will currently be impossible to remove (YouTube will have to fix/allow it to be hidden). For the time being I've just made Plyr's play button slightly bigger to cover it up.

This has been raised before (worth checking closed issues). Nothing can be done to hide that button. Changing the size of the plyr large play button with CSS is your only option.

On 23 Jun 2016, at 5:07 AM, Tristan Strathearn<EMAIL_ADDRESS>wrote:
I've noticed recently that the YouTube play button is now sitting behind the Plyr play button. Seems to be that the query parameter 'controls' passed you YouTube isn't working as expected. Any plans/ideas on how to remove it within Plyr? You can see an example below (which occurs on the Plyr demo page):
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Gotcha, I did mention the outcome in my second follow up comment. I'll close the issue.


Cannot access my internal tables

Unable to retrieve child resources.
Details:
Network timeout error: connect ECONNREFUSED <IP_ADDRESS>:10002
Tried to access an internal storage table. I've tried creating a new table but for some reason it wont allow me to acess any of them

@Danlough96 do you have proxy settings enabled in the app? Alternatively, do you have either of these environment variables set?

HTTP_PROXY
HTTPS_PROXY

Also, what OS are you on and what version of Storage Explorer are you running?

Hi sorry wasted your time. Didnt set up my environment properly be a it usually does it automatically. Human error! My bad


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.31.8

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.31.7 -> v1.31.8

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.31.8
Compare Source
Bug Fixes

deps: update gomod (#809) (5f06800)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 4.11.5 :tada:
The release is available on:

GitHub release
v4.11.5

Your semantic-release bot :package::rocket:


Displaying incorrect role name in account.

Framework Version: 0.1.2
Displaying role name as administrator always.

Fetched Role n

ame  from database , Now its showing Properly.

Framework Version: 0.1.3
Fixed


Enforce outdenting public/private/protected

After I realize how the parser really sees these, I think we should change how we do them
http://collectiveidea.com/blog/archives/2016/09/09/my-ruby-epiphany/
The funny thing is I don't actually like this. I've been a fan of indenting, but I've come to realize I'm doing it wrong. I don't expect this PR to be without controversy.
An alternative that I hadn't thought about was mentioned in the comments of my post:
class Foo

  private def some_method
    ...
  end
end

This is really using the syntax private :some_method because in Ruby 2.1+, defining a method returns a symbol. (more here: https://www.sitepoint.com/look-ruby-2-1/)
This is meant to start the discussion. What do we think?

I kinda like private def some_method.  Maybe it's just because I'm used to it from other languages, but it makes it much more clear that the method is private.  Otherwise, it's easy to miss the private declaration in a large file.

Not a fan of this one.

Why not? Can you give me detail?

Because I have never seen adding visibility on a per method basis. Also, this isn't Java with our private static void volatile <generic> emo accounting method() signatures. Out denting it to the level of the class just feels weird. Really weird.
I think the standard (from the code I've worked with) is this:
class Foo
  def pooblic
  end

  private

  def privat
  end
end

I don't like outdenting private to the class level, either. But "this isn't java" doesn't seem like a very good reason not to do something.
You're right, though, that the standard is the indented private section. It seems to me that @danielmorrison's suggestion of private def hurrdurr is empirically better than the standard. It's more explicit and readable than having a hidden private somewhere in the file.


Failed to resolve import "vite-plugin-node-polyfills/shims/buffer"

When using a version higher than 0.17.0, and polyfilling a bunch of stuff (see vite config snippet below), I get the following error:

The relevant section of my vite config is this:
"plugins": [
// ...
 nodePolyfills({
      globals: {
        Buffer: true,
        global: true,
        process: true,
      },
      include: [
        'fs',
        'path',
        'crypto',
        'stream',
        'util',
        'http',
        'https',
      ],
    }),
// ...
]

With this config everything works fine in 0.17.0 but in the testet versions 0.19.0 and 0.21.0 the error occurs
When trying to include "buffer" in the include array, it also fails because there is no polyfill for it (or it gets already polyfilled by the globals section above).
( ERROR: [plugin: node-stdlib-browser-alias] Cannot find module 'vite-plugin-node-polyfills/shims/buffer' from '.')

Just got bit by this one on v0.21.0 @davidmyersdev this is a gnarly regression, could we get a fix in por favor?

Similar issue here

Hello, also, I got same issue. I accepted my senior's comment. just change version to "0.17.0".
And it works. Thank you bros.

Um... Something happens.
I can build with latest version. (not 0.17.0) when I add "vite-plugin-node-polyfills" at the "top-level" of monorepo.
my projectstatus is.

Using yarn berry. (PnP zero install)
monorepo

apps/~
packages/~

apps/client using "vite"
using turborepo for managing 'monorepo'

before "vite", I was developing with "cra".
Because I'm amature yet, I can not pick the reason why this situation happens...

As Above comment, And scan that repo, pnpm user seems to have Same issue. (not only yarn berry)

Situation

I just "yarn add -D ~" at the "top level of monorepo" by mistake.

After, I  move to "apps/client" dir, And command "yarn add -D ~".

two package.json file has "vite-plugin-node-polyfills" dependency.

yarn turbo build. -> client build successfully.

"why success??" -> I just remove the top level dependency. and yarn install at the top level.

-> fail.

After that experience, I have some test.
I don't know why this solve the problem...

monorepo top level dependency, solve the problem.

Running into this issue as well -- it occurs from within a lerna repo and a common package is referencing Buffer (but not using vite).

Hello, also, I got same issue. I accepted my senior's comment. just change version to "0.17.0". And it works. Thank you bros.

This

Changing the version to 0.17.0 helps me as well. Thank you @raduciobanu22 🙏

I wasn't able to reproduce this error with just the config provided above. I'll need a StackBlitz or CodeSandbox example in order to diagnose the issue and get a fix together.

Happens for me when polyfilling crypto on v0.22.0, and the import comes from the ethers package:
Rollup failed to resolve import "vite-plugin-node-polyfills/shims/global" from "app/node_modules/ethers/lib.esm/crypto/crypto-browser.js".

As with others, downgrading to v0.17.0 resolves the issue.

Same here. Using Quasar/Vite.  Needed to go back to version ~0.17.0

Going back to v0.17.0 clears the error Buffer error, but gives me this:
TypeError: Cannot destructure property 'Uint8Array' of 'globalThis' as it is undefined.
    at<EMAIL_ADDRESS>(helpers.ts:226:31)
    at __require (helpers.ts:13:60)
    at shims/dist/index.js (helpers.ts:1919:46)
    at __init (helpers.ts:10:66)
    at shims/banner/index.cjs (helpers.ts:1927:13)
    at __require (helpers.ts:13:60)
    at helpers.ts:1933:5
    at helpers.ts:1934:3

Also, I would like to note that this error and the Buffer not foun only occur in vite dev. The built application works as expected.

For those of you coming here with a problem with Buffer or any other polyfill, maybe this will help you:

Import the plugin before your react or vue plugin in the Vite config file. For some reason, if I import it after, it won't work.

So this will work:
plugins: [nodePolyfills(), react(), TanStackRouterVite()],

but this won't:
plugins: [react(), TanStackRouterVite(), nodePolyfills()],

We've just updated to 0.22.0 and using nodePolyfills() as the first  vite plugin as @GiovaniGuizzoYUB suggested seems to work fine. (And makes a ton of sense if you think about it :D)
I'll close this now, as it is fixed for us, but for some, downgrading to 0.17.0 might still be a valid "fix".

With Quasar, including the plugin v 0.22 first still does not work.
quasar.config.ts
{
  vitePlugins: [
  [nodePolyfills, {include: ['buffer', 'path']}, {server: false}],
   // ...
}

[vite]: Rollup failed to resolve import "vite-plugin-node-polyfills/shims/buffer" from "/Users/steven/dev/projects/ct/ct-urm/src/v3/plugins/resource/platformResourceProvider.ts".
This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`

I was able to fix this by monkey-patching the Vite plugin
const nodePolyfillsFix = (options?: PolyfillOptions | undefined): Plugin => {
  const origPlugin = nodePolyfills(options)
  return {
    ...origPlugin,
    resolveId(this, source: string, importer: (string | undefined), opts: any) {
      const m = /^vite-plugin-node-polyfills\/shims\/(buffer|global|process)$/.exec(source)
      if (m) {
        return `node_modules/vite-plugin-node-polyfills/shims/${m[1]}/dist/index.cjs`
      } else {
        if (typeof origPlugin.resolveId === 'function') {
          return origPlugin.resolveId.call(this, source, importer, opts)
        }
      }
    }
  }
}

Or more simply because the plugin does not currently define  resolveId.
const nodePolyfillsFix = (options?: PolyfillOptions | undefined): Plugin => {
  return {
    ...nodePolyfills(options),
    resolveId(source: string) {
      const m = /^vite-plugin-node-polyfills\/shims\/(buffer|global|process)$/.exec(source)
      if (m) {
        return `node_modules/vite-plugin-node-polyfills/shims/${m[1]}/dist/index.cjs`
      }
    }
  }
}


Not appearing in Package Control

I have no idea what I'm missing here, but no plugin called Git shows up in Package Control for me. I searched Git and scrolled through every result until it was clear the searches were expanding too far due to no matches being found. I installed this successfully several weeks ago, but it's gone now. Is something up?
(I'm installing manually, no big deal, just wanted to report in case something is actually wrong and it's not just me.)

 I recently reformatted my computer, didn't realize I had already reinstalled the package. Sorry!


Test run: v0.3.6-alpha.1

for testing only - DON'T publish

Looks good on macOS (El Capitan 10.11.6) + Linux (Ubuntu 18.04.5)


implementation of mixture of experts by keras

Hi, I am trying to implement mixture of expert neural networks using keras.
For instance, I have 3 expert neural net, each have 2 outputs. A gated network assigns a probability to each expert. The gated network in my case has 3 output generated by softmax. Inputs to all experts and gated network are same.
My question is how I can customize the loss function in the following form:
loss=sum_i(p_i*(y_true-y_i)^2)
where p_i is the probability that gated network assigns to expert i, y_true is the desired output (in my case 2 dimensional), and y_i is the output of expert I. and sum is on the number of experts.
merged = Merge([gated, expert1, expert2,expert3], mode='concat')
model = Sequential()
model.add(merged)
model.compile(loss=loss function, optimizer=Rmsprop)
model.fit([in,in,in,in],[out,out,out])

Hi,
Could you impelemnt your code with loss function?
I faced with this problem. If you did, can you help me?
thank you

Hi,
Could you impelemnt your code with loss function?
I faced with this problem. If you did, can you help me?
thank you


download.py has very high memory usage

The main program itself is using nearly 4GB total, before downloading anything.

I can replicate this on my end, although I am not entirely sure of the cause. It seems that the annotation parsing is what requires this large memory. The annotations for classification training are 274 MB in total so it seems odd that the parsing would require such a large amount of memory.
What I can say is that the script doesn't leak memory, and so if you have a system with at least 8 GB RAM then you should be good to go.

Closing this issue due to inactivity. Feel free to reopen if the need arrises.


[REBUILD] Doom 2 Redux Requires Testing!!!

Doom 2 Redux was just re-added to the rebuild branch. This requires testing from the crack team of mapset testers.
Please add a note to the ticket if you are testing this and please report back on the following:

Any texture issues (missing or incorrect textures etc)
Thing issues (incorrect decorations. monsters etc
Show stopping bugs
Any other issues

This should be tested within the week before the weekend with the other mapsets added.

Redux is missing textures but they are also missing in the originial wad file. Closing for now as issues were raised.


Docs: Fix the text size issue for Map and Reduce section

Describe the bug
Fix the text size issue in Map and Reduce section of docs
Expected behavior
The text size should be similar to other command line text
Screenshots

Message from the maintainers:
Impacted by this bug? Give it a 👍. We often sort issues this way to know what to prioritize.

I would be happy to help with the issue. Could you assign this issue to me?

@boringlyBoring - thanks! Assigned.

Thanks! I will start working on it Friday night

I can pick this up


Error in OpenCL.dll

Hi!
I run
nheqminer.exe -t 8 -l europe.equihash-hub.miningpoolhub.com:20570 -u user.worker -p x

on windows 7 system from latest build. But this computer have videocard from IBM. I got an error

the entry point in the procedure could not be found, clReleaseDevice

I copy opencl.dll in miner folder, but this not work.

Copy OpenCL.dll from https://github.com/nicehash/NiceHashMiner/releases/download/<IP_ADDRESS>/NiceHashMiner_v<IP_ADDRESS>.zip and place it in the same directory as nheqminer.exe
The DLL needs to be included in the release zip

https://github.com/computeronix/nheqminer/blob/master/OpenCL.dll?raw=true

OpenCL solver are dropped from nheqminer. Latest release.
Closing issue...


Fix for No vaild linked folder path

I believe this commit introduced a bug, that prevents converting from Note to Folder Note

thanks! it's now included in v1.3.0


Middleware Datasource List: Add uniqueness

This is feature request to add more info in middleware datasources list page to make datasources unique there.
The Issue: It shows "DS Name" and "Server" fields in datasources list. So hard to see uniqueness when server name is default.
Suggestion: Add additional columns in list, for instance Driver Name.
Add "hostname/servername" for server value in list and in DS summary page, like requested in   https://github.com/ManageIQ/manageiq/issues/9039
Screenshot of DS list and details pages.

@abonas

@miq-bot add_label enhancement, providers/hawkular, ui

@hhovsepy, regarding adding more server details in the datasource summary page - the server in the summary page is a "relationship" link - meaning that the user can click on it and see full details of the server. this is the miq current standard. So duplicating information from server entity into datasource properties in the summary page would be the less advised approach.

@abonas ok thank you, then only having uniqueness in row level could be enough, like "hostname/servername" instead of "servername".


Add Dead Product component

Add component for when a product has already been killed.

Closed in latest PR


.yml suffix are not recoginzed

myfile.yaml is working, but
myfile.yml is not working
yml suffix should also be allowed

Hi @exocode
thanks for pointing it out, will add the support for .yml format too


test_text.py fails when running pytest

Running the tests from test_text.py directly with pytest tests/test_text.py works fine.
However when running only pytest it fails at test_mathtext_example[png] because some of the text is rendered as black boxes instead of normal text.
This could be caused by some OpenGL context from previous tests influence the others.

This seems to be related to the pyqt version. The latest pyqt version where all tests pass is v5.10.1. The first release tested where the test fails is v5.11.2.

This seems to be a Problem with GPUObjectCache. If caching of objects is disabled for text the tests pass correctly.
However just increasing the cache size does not seem to help.

Was fixed by PR #14.


exim: 4.96.2 -> 4.97

Description of changes
https://lists.exim.org/lurker/message/20231104.135832.37148bbd.en.html
does not build yet
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

4.97.1 has been released to fix CVE-2023-51766. https://www.openwall.com/lists/oss-security/2023/12/29/2 This will also need a backport to the stable channel.

I don't think this needs to be backported:

Version(s): all up to 4.97 inclusive

Quoting Arthur (2024-01-01 23:28:17)
I don't think this needs to be backported: > Version(s): all up to 4.97 inclusive
I believe this means all previous versions are impacted, including the one on the stable channel. The other advisory is clearer:
Exim before 4.97.1 allows SMTP smuggling in certain PIPELINING/CHUNKING configuration
https://nvd.nist.gov/vuln/detail/CVE-2023-51766

Oh, sorry!


join helper doesn't work in Handlebars 2.0+

I get output like this when using the joinAny function: [object Object],[object Object],[object Object]
http://jsfiddle.net/stewx/umj3Lbb0/

Would you mind trying out the join helper on the dev branch? see if that works for you?

Sounds like this was resolved, since the lib was refactored. See the new README docs on usage.
I'm going to close but please feel free to reopen or create a new issue if:

the bug is still happening
you're still having problems


Modularize hover and autocomplete code

Here I modularized the autocomplete and hover funcitonalities. This will make it much easier to add future language support

Should I make this 0.12.1 or wait until I add the next langauge(s) as part of 0.13.0

Clean 👌


Headers on loadMedia

I need to use headers in the url to load my video, I have not seen any options to pass header when load media. That are any way that I can do?

+1 I need send referer in the header.


[MSHARED-1330] Always overwrite files

Make sure the files are made writable before copying them.  The attributes are always restored after the copy.

why do you need to set the execute bit?

We don't, we just need it to be readable and writable, I'll remove that line.

why do you need to set the execute bit?

Fixed


Topic/refactor robothandler

Hello guys,
I did a substantial refactor on robot handler. It is now split into RobotModelHandler (for immutable data, such as robot model) and RobotDataHandler (for mutable data)
I updated the bindings, tests and benchmarks.
However, we do not have test in python, so I am not sure of what I broke in the python API...
Cheers

Looks good to me

This PR is ready for merge, all testes passes on my machine :)

I trust you on this


[Feature Request]: Implement addScenes() function

Origins of the Feature
Currently, to add multiple scenes, you need to call the SceneManager.addScene() function multiple times. If many scenes need to be added, this can get messy quickly.
The New Feature Idea(s)
Implement a addScenes() function which takes in a list of scenes, and adds all the scenes at once. This will make it easier to add many scenes with one function.
Alternatives
N/A
Code of Conduct

[X] I agree to follow this project's Code of Conduct

also, I think that It'd be great if it has both addScenes(Scene...) and addScenes(List<Scene>) (or addScenes(Scene[]))

Thats a good idea, ill remember to do that!


Fix windows boosted_trees test failure

In the Windows cmake build, the test "python/kernel_tests/boosted_trees/training_ops_test.py" is failing on a Proto comparison. This commit changes one of the expected values to use
higher decimal precision which should pass on all builds. (-24.0143 to -24.014299)
The windows error of topic:
13:01:23 Traceback (most recent call last): 13:01:23   File "C:/tf_jenkins/workspace/tensorflow-master-win-cmake-py/tensorflow/python/kernel_tests/boosted_trees/training_ops_test.py", line 1363, in testPostPruningOfAllNodes 13:01:23     """, res_ensemble) 13:01:23   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\framework\test_util.py", line 840, in assertProtoEquals 13:01:23     self._AssertProtoEquals(expected_message, message, msg=msg) 13:01:23   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\framework\test_util.py", line 817, in _AssertProtoEquals 13:01:23     compare.assertProtoEqual(self, a, b, normalize_numbers=True, msg=msg) 13:01:23   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\util\protobuf\compare.py", line 107, in assertProtoEqual 13:01:23     msg=msg) 13:01:23 AssertionError: 'tree[570 chars]4.0143\n  }\n}\ntree_metadata {\n}\ngrowing_me[62 chars]n}\n' != 'tree[570 chars]4.014299\n  }\n}\ntree_metadata {\n}\ngrowing_[64 chars]n}\n'
This stems from the 32 bit floating point imprecision. For example:

import numpy as np
p = np.float32(-24.0143)
'{:.12f}'.format(p)
'-24.014299392700'

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 36 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Similar change has been implemented in 5cef54072782a9a893eda69bec30fcf79cd0086b.


[QUEST] Update jquery to 3.0

https://nvd.nist.gov/vuln/detail/CVE-2015-9251
Currently just bumping the version and building breaks other things in the app.

Removing jQuery instead by removing angular


feat(gateway): send errors during connection setup to client

Currently, the gateway will only log errors during DNS resolution and / or accepting a connection without responding to the client. As a result, the client has rely on timeouts for these situations.
The portal already allows us to send opaque data as part of the payload field during the connection setup. Whilst the event connection_ready is a bit of a misnomer for errors, we can use this to send back an error response to the client in case DNS resolution or accepting a connection fails.
This should allow the client to react much quicker to problems on the gateway and at least clean-up its local state and trigger another connection intent. It also allows us to print a more specific warning into the clients logs.
Finally, the log levels for these problems is turned up to WARN on the gateway. Failure to accept a connection or resolve DNS is an issue that massively impacts firezone's functionality and is thus be something that an admin needs to be alerted about.
This is backwards compatible. Old clients will simply fail to deserialise error message.

This still needs manual testing.

@conectado Whilst rebasing this onto main I realised that we can probably do something better here now. We can always accept the connection and send back an answer and only have the allow_access call fail if we don't have DNS resolved yet, in fact, we can decouple DNS resolution entirely from accepting the connection and do it in the background.

@conectado Whilst rebasing this onto main I realised that we can probably do something better here now. We can always accept the connection and send back an answer and only have the allow_access call fail if we don't have DNS resolved yet, in fact, we can decouple DNS resolution entirely from accepting the connection and do it in the background.

You mean after the DNS refactor on the client?

See https://github.com/firezone/firezone/pull/5457.


Services section

We're only displaying 3 services but there should be 6 > https://xd.adobe.com/view/8a626811-2fea-419b-5abd-4177a925eb9e-fc53/
I mentioned in #22 this section can be 1040 wide
#37 explains the icons for this section
There are mistakes in the current text, replace with this...
Customer Development
Let me help you understand prospective customers and their needs that you may be able to satisfy. You cannot validate your assumptions on your own, I’ll take you through a challenging process where we talk with customers to discover if your assumptions are true.
Hyper-convenient Design Thinking
I like to remove things that create friction and distraction and this typically means a shorter route to a solution. Less is often more. Whilst I’m not a graphic designer I do think deeply about the lack of attention span people have when they’re confronted with new products or steps within those products.
Marketplace Research
Rarely is a marketplace as simple as supply and demand. And whilst the internet empowers us to remove middlemen there are always other stakeholders to consider. I can help you figure out the wider ecosystem that your product or marketplace operates in, and if, and how you could fuel adoption.
Product Architecture
The requirements are clear but there are many ways for the features and components of your product to fit into the overall system and business model. I take a user-centred approach to map the functions and features you require, to the form of your product.
Crowdfunding
There’s nowhere to hide if you launch a crowdfunding campaign unprepared. The eager and eagled-eyed investors will pick apart every word, picture and number in your materials. I’ve had previous success with equity crowdfunding and I can help you get your ducks in a row.
 I can distill your vision and convert your messaging & content into a compelling and flowing narrative.

services section is built dynamically , from this file
https://github.com/kelvinlockwood/kelvinlockw.xyz/blob/main/src/_data/services.js
its in JSON format
I updated the values but you can change them anytime by editing that file
I also responded/fixed the other issues


NDRS-568: Dial down verbosity of execution result logging.

This PR reduces the verbosity of execution result logging, to reduce log spam.
https://casperlabs.atlassian.net/browse/NDRS-568

bors r+


Update CONTRIBUTING.md with default black line length

Summary
Black uses a line length of 88 by default, so this updates CONTRIBUTING.md accordingly.  Alternatively, we could use a line length of 100; in this case, we would have to pass it as an argument to black in tox.ini.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
11

Relevant Lines:
11

💛  - Coveralls

I'm going to merge this, since our goal is to keep the template repository in a near-perfect state at all times.  I'll add the commit that reverts this change to #46, so that everything will be consistent then too if we decide to change the line length to 100 and merge it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
11

Relevant Lines:
11

💛  - Coveralls


Using multiple custom tools only works with the first one

I'm using the following configuration in order to customize the tools available for grepping:
  let g:grepper = {
      \ 'tools':     ['ag', 'csearch'],
      \ 'dispatch':  1,
      \ 'open':      1,
      \ 'switch':    0,
      \ 'jump':      0,
      \ 'ag': {
      \   'grepprg': 'ag --nogroup --nocolor --column',
      \   'grepformat': '%f:%l:%c:%m',
      \ },
      \ 'csearch': {
      \   'grepprg': '~/gocode/bin/csearch -n',
      \   'grepformat': '%f:%l:%m',
      \ }}

Then, using the command :Grepper only shows ag and hitting TAB doesn't change to csearch.
Also, when running the command :Grepper -tool csearch fails with an error No such tool: csearch. With :Grepper -tool ag works fine.
Grepper version d66c5fb

~/gocode/bin is probably not in $PATH. All non-executables from g:grepper.tools will be filtered out. That way it gets pruned of all the default tools that are not available on the system. (https://github.com/mhinz/vim-grepper/blob/master/autoload/grepper.vim#L49)
I suggest to either put a symlink to csearch into a path from $PATH (or just add ~/gocode/bin to it) or use the full path in tools, e.g.:
let g:grepper = {
    \ 'tools': ['foo', '/home/user/gocode/bin/csearch'],
    \ '/home/user/gocode/bin/csearch': .....

I'll commit 2 patches:

document that the tools in g:grepper.tools have to be executable
expand() the tools in g:grepper.tools, so you can use ~/gocode/bin/csearch instead of /home/user/gocode/bin/csearch

Does this sound good to you?

Thanks for your response, you were totally right.
Exposing the corresponding binaries in $PATH did the trick.
Documentation will be really good to have.
Go ahead with the second point if you want, not mandatory, but I think might be an easy tweak to avoid having this "issue" again.

I like the simple elegance of call filter(s:options.tools, 'executable(v:val)'), but I guess we could make it way more convenient for the user to use any arbitrary name in g:grepper.tools like this:
for tool in s:options.tools
  if has_key(s:options, tool)
        \ && has_key(s:options[tool], 'grepprg')
        \ && !executable(matchstr(s:options[tool].grepprg, '^[^ ]*'))
    call remove(s:options.tools, index(s:options.tools, tool))
  endif
endfor

As always in programming, it's a code size vs. user convienience tradeoff. I'll have to sleep a night over this.

Okay, g:grepper.tools allows arbitrary names now. This should resolve the original issue.

Thanks for your work.
It works but with the full path, ie: /Users/santiago/gocode/bin/csearch. With ~/gocode/bin/csearch it doesn't work, but I don't know if we want to allow that.

About overwriting ag, it's because in Windows I'm still using an older version that doesn't support --vimgrep parameter.  I didn't find a new binary.

Argh, I said it before but forgot to include it in the patch itself. I added the missing expand() now. :-)

Excellent, it works!

Great!
One last question: is the --nocolor really needed? I hacked up a special case for ag that uses ag --column --nogroup --noheading for versions < 0.25.

Well, now that you mention, it seems that it isn't strictly necessary. However I can't try it in Windows right now.
I would say go with it since it won't do any harm.

I skipped the --nocolor since none of the default grepprg values don't use those "disable color" options.
Thanks for all the feedback, very appreciated!

Thanks to you for this additional change!


List tenant cluster apps.

This PR proposes a way to fetch the apps that are installed on a given tenant cluster.
FYI @giantswarm/sig-ux : Operations can have multiple tags, so it will show up in the Cluster category as well as the App Katalog category in api docs.
That seems desirable to me since I would personally expect to see it in both places. But opinions are welcome here.

let's think about the linking and if it least to confusion

For me it's fine, it might not be exactly what you expect, but then you read it and it makes sense.
At least you'll find it in the places where you might expect to find it right?

ack, go with it

@marians: There are two resources that we currently want to expose or have access to.
Those resources are apps (on a tenant cluster) and appcatalogs (an installation wide concern).
Apps might have subresources that we want to configure (like secrets or configmaps) but it's not decided yet if that'll just be part of the app resource or if they will be separate sub resources.
I can imagine the following operations and endpoints then:
Get all catalogs configured for this installation:
  GET  /v4/appcatalogs/

Add a catalog configured for this installation:
  POST /v4/appcatalogs/

Get all apps installed on a tenant cluster:
  GET  /v4/clusters/:clusterid/apps/

Get more details about an installed app:
  GET  /v4/clusters/:clusterid/apps/:appname/

Install an app:
  PUT /v4/clusters/:clusterid/apps/:appname/

Delete an app:
  DELETE /v4/clusters/:clusterid/apps/:appname/

Make changes to an app:
  PATCH /v4/clusters/:clusterid/apps/:appname/

I'll be moving forward with this spec and start implementing GET /v4/clusters/:clusterid/apps/ now.
ping @marians , any comments since you requested more info before?

Fine with proceeding here! Thanks for providing more context. That helps!


Write Part - 3 Step - 11 & unit test

Implement Step11Controller and write all unit tests. Verify the display name of the test conforms to the pattern of "Part X Step Y". Follow the standardize naming, comments, etc. to be consistent.

Similar to Part 01 Step 20 (#25)


I am using Alamofire.download method. How I can detect if the url is broken

The case I am talking about is that the server is online. the base url does exists. but, the file itself doesn't. the alamofire download doesn't detect an error! it downloads the error page itself. I am expecting an image no error is there.
Alamofire.download(URL(string: fileUrl as String)!, to: destination).response { response in
                if response.error == nil {
                    Log.debug("Play image downloaded successfully.")

                    // hide progress
                    self.progressView.isHidden = true

                    // Save image url
                    Common.setValue(value: self.fileUrl, key: self.competition_id)

                    self.displayImage()
                } else {
                    // image failed to download
                    Log.debug("Image download failed with error: \(response.error)")
                }
            }
            .downloadProgress(closure: { (Progress) in
                    //Progress.fractionCompleted
                Log.ver("Progress: \(Progress.fractionCompleted)")
            })

There are many ways you could solve this. One of the simplest would be to use .validate, as described in our documentation to ensure the response type is an image rather than the type of the error page.


What is in this program?

I used this exe at work and it got flagged and my pc for locked out of the network. While yes you should not do anything personal on a work pc. I did after a scan get two ransomware messeges on the pc.
There is something with this program that promts this

Python executibles have a high false positive rate for Malware.. you can
look through the code.
On Thu, Dec 19, 2024, 4:24 AM Djwhitelion @.***> wrote:

I used this exe at work and it got flagged and my pc for locked out of the
network. While yes you should not do anything personal on a work pc. I did
after a scan get two ransomware messeges on the pc.
There is something with this program that promts this
—
Reply to this email directly, view it on GitHub
https://github.com/RavinMaddHatter/Structura/issues/239, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUC5BNWZGOLOKNGD4BS3T2GK3JRAVCNFSM6AAAAABT43444KVHI2DSMVQWIX3LMV43ASLTON2WKOZSG42TAMJVHA3DKNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Then i have something to say to it atleast. The fault is mine for trying it at work. Maby. Just maby add a warning about the probability of a high risk of false positives because of the format? Just a thought.
The fault is still all mine.

I am trying to get away from exe. Honestly, you shouldn't. Trust me unless
you have reason to. I could make malware and tell you that.. I do say in
the first video that you shouldn't trust random strangers on the internet
On Thu, Dec 19, 2024, 5:27 AM Djwhitelion @.***> wrote:

Then i have something to say to it atleast. The fault is mine for trying
it at work. Maby. Just maby add a warning about the probability of a high
risk of false positives because of the format? Just a thought.
The fault is still all mine.
—
Reply to this email directly, view it on GitHub
https://github.com/RavinMaddHatter/Structura/issues/239#issuecomment-2553969882,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUC5BCO5DHFV4Y3UIBHCD2GLCTXAVCNFSM6AAAAABT43444KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTHE3DSOBYGI
.
You are receiving this because you commented.Message ID:
@.***>

Haha. And this is why i am so up front on saying its my fault. 🤣👍


readFromSecureStore() return object type instead of SecureStorableResultType

I really like the new feature where you can create a struct, respond to a few protocols, and return the data variable and Locksmith will automatically be able to save the object. However, if it could also fetch the object, that would be even better. It is nice to be able to have Locksmith fetch the object, but I still have to manually pull my properties out of the data object and it seems like that process could be improved.
For example, if the structs were instead subclasses of NSObject, then they could work with the setValue forKey and valueForKey functions. Then the data property could return a list of strings of the key names for all properties that will be stored in the keychain. From there, the createInSecureStore could piece together the data to be stored by iterating over the strings in data and calling valueForKey on each. Then when reading from the keychain with readFromSecureStore the function would return an object of your custom type and your properties would be populated with setValue forKey on each property that is in data and was returned from the keychain.
This would however require a few changes, so there's probably a better and more elegant way to make this possible. What do you think?

Thanks for the feedback! I really like the idea.
My original intention was to be able to use generics to do this, i.e. the user could specify the type of the data to be saved and read back. Unfortunately, I haven't yet been able to come up with the right way to do this (always end up getting stuck by Protocol 'GenericProtocol' can only be used as a generic constraint because it has Self or associated type requirements. errors).
I'm also sort of hoping there will be some language support for protocol generics (rather than using type aliases) in the future, which would make this easier.
If/when I do the next big release, this will definitely be one of the goals, so feel free to make suggestions/submit PRs that use generics to accomplish this :)
Closing this issue because there aren't really any actions to make immediately, but feel free to reopen at any point 👍


Only the Flyout component of a ToggleSplitButton can be interacted with using Touch

To reproduce:
Add a Checkbox control to the Flyout element of a ToggleSplitButton.
Try pressing the left portion of the button with touch.
Expected:
The button should toggle states.
What happens:
Regardless of where the button is touched, only the flyout content will show/dismiss.
See attached video where I first interact with the control with a mouse and then with touch.
https://github.com/MicrosoftDocs/winui-api/assets/26647325/cc90c236-1466-4c96-a8f3-e805213ef090

Sorry just realized this is for docs.


feat: port Algebra.Algebra.Basic

Can you restart the port here? These files are out of date with mathlib3: https://leanprover-community.github.io/mathlib-port-status/file/algebra/algebra/basic

I can do it in ~1h.

Ideally, by using the script at #2249 so that we can check it still works :)

It seems that these changes are not in https://github.com/leanprover-community/mathlib3port/blob/master/Mathbin/Algebra/Algebra/Basic.lean yet.

It seems mathport is stuck, https://github.com/leanprover-community/mathport/issues/228

bors merge

bors merge


fix: add SystemEndpoint vmid into whitelist

Codecov Report

Merging #338 (75af4ae) into main (4c31f25) will decrease coverage by 0.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #338      +/-   ##
==========================================
- Coverage   63.84%   63.67%   -0.18%
==========================================
  Files          41       41
  Lines        7155     7160       +5
==========================================
- Hits         4568     4559       -9
- Misses       2205     2218      +13
- Partials      382      383       +1

Impacted Files
Coverage Δ

plugin/tower/pkg/controller/policy/controller.go
83.05% <0.00%> (-0.14%)
:arrow_down:

pkg/agent/controller/policy/policy_controller.go
78.03% <0.00%> (-1.52%)
:arrow_down:

pkg/controller/group/group_controller.go
69.85% <0.00%> (-1.09%)
:arrow_down:


OM 2.9 introduces "Recreate the BOSH Director" feature

Two things:

Ops Manager 2.9 has introduced a "Recreate the BOSH Director VM" parameter. This parameter is a boolean called bosh_director_recreate_on_next_deploy. It is used just like the existing bosh_recreate_on_next_deploy parameter. FWIW the story that introduced this change is here: https://www.pivotaltracker.com/story/show/170879932

The prose that describes the bosh_recreate_on_next_deploy option should be updated to emphasize the fact that this option typically only re-creates BOSH-deployed VMs and does not touch the Director.

cc: @ystros @mingxiao

Proposal:
Edit to the apply-changes --recreate-vms behavior to include the bosh director during the recreate (so that the command is honest). To keep the command consistent with other apply-changes flags:
om apply-changes --recreate-vms --product-name {product} should recreate the vms for that product
om apply-changes --recreate-vms --skip-deploy-products should recreate only the BOSH director vms
om apply-changes --recreate-vms should recreate all product vms, and the BOSH director vm.
Would recommend doing something similar for this as we did for --recreate-vms: update the properties in the bosh director for a single run. This will likely require some refactoring of logic to accommodate the differing use cases. Also, keep these new features fully tested in both apply-changes.go and director_service.go

At first pass this looks really good to me.  @aegershman and myself have been anticipating these capabilities for a while very exciting to see it come to pass.

this will be included in om 4.7.0


Running qsub with post-install

I want to start a workflow with qsub as soon as the cfncluster creation is successful. However when I put the command into the post install script specifically for the master node, it returns qsub command not found. What is going on?

post-install actions are called after cluster bootstrap is complete, as the last action before an instance is considered complete. Typical post-install actions may include changing scheduler settings, modifying storage or packages.
Ref: http://cfncluster.readthedocs.io/en/latest/pre_post_install.html?highlight=post_install#custom-bootstrap-actions
Workflow has to be initiated outside of cfncluster, cfncluster post_install as mentioned above is part of cluster bootstrapping. Poll for cluster creation completion & execute qsub command via Ec2 Run command or via ssh.

Is there a way to run command as ec2-user automatically when the cfncluster creation is successful, without ssh or ec2 run?

Yes we believe so, but it will take a but of work based on your environment.
Since you are running as root, you will need to su to your user (ec2-user it sounds like), then create your script, then add your needed PATH env variables, then you should be able to run qsub.
As a trivial example:
su - ubuntu; echo "#!/bin/bash echo Hello World" > script.sh; PATH=$PATH:/opt/sge/bin/lx-amd64/ (+ more variables); qsub script.sh
For an ubuntu and SGE example (NOTE this does not fully work, but it does find qsub).
You should be able to work out a post install script that allows you to run qsub.
Overall this sounds like it could be a useful feature, and will add it as an enhancement.

I'm wonder how true is this statement from @mohanasudhan

Typical post-install actions may include changing scheduler settings, modifying storage or packages.

I'm trying to add a complex variable when the compute node runs the post_install script and I'm getting the error below.
command:
su -c "SGE_ROOT=/opt/sge /opt/sge/bin/lx-amd64/qconf -mattr exechost complex_values ram_free=30G `hostname`" sgeadmin

Error:
denied: host "ip-172-31-31-210.us-west-2.compute.internal" is neither submit nor admin host

If I run the exact same command from the compute node after it gets visible in qhost, then the command works without problem.

Hi, I am trying to do the same thing as jprobichaud by calling qconf to modify settings for a compute node.  I am also getting the same denied message:
denied: host <hostname> is neither submit nor admin host

post-install script is meant to allow the configuration of a node before this gets added to the scheduler and that is why you are not able to run any scheduler specific config command on compute nodes at that stage.
What you could do is to execute scheduler configuration commands on the master node only, by adding a conditional statement to your post-install script as described in the official docs. Of course at that point in time the scheduler won't have the compute nodes available in its configuration.
An idea for a potential enhancement is to expose a hook to execute a custom script after a node gets added to the scheduler.

There is no plan for now to support job submission in post-install scripts because such scripts are meant to provide hooks for software installation and are executed when the scheduler daemons haven't been initialized.


Extend DP with a gRPC plugin for enodebd callbacks

To enable Domain proxy interaction with ocr8r/enodebd, a new DP plugin must be developed that will be responsibe for listening to calls coming from enodebd (calls which will be made by enodebd whenever a eNB state changes)

[x] create grpc server in RC for handling requests from enodebd (register CBSD, create or update config, fetch state)
[x] add relationship between grants and channels (update CC and prepare migration)
[x] write an intergration test with whole DP (using the same API as enodebd would)

POC implemented and initially tested on DP deployed on FreedomFi AWS.
Code works but in an E2E scenario and a proper PR might require changes.
Those further modifications and PR will be handled in a separate task.