Cogwheel popup menu location inconsistent with "File/Edit/Selection/etc..." menus

Type: Bug
Summary
The cogwheel menu[1] is different from other menus, leading to mistakes.
For other menus, you can click them to open them and see the options, then select a desired option.
For the cogwheel menu, clicking it is a selection. You have to press-and-not-release your mouse button in order to look at the menu.
Environment
I have tested on distros Pop!_OS and Kubuntu.
On Pop!_OS, this issue replicates for all 14 combinations of activity bar position (left, right) and VSCode location (fullscreen, half screen left, half screen right, NE, NW, SE and SW quarter screen location)
On Kubuntu, this issue only replicates when the activity bar is on the right-hand side of the screen.
Workflow descriptions
The results for the "File" workflows replicate for other top-bar menus e.g. "Edit."
Workflow "FILE View menu options":

Move cursor over "File" menu
Without moving the cursor, press left mouse button
Without moving the cursor, release left mouse button

Results
Actual behavior at step

--
Menu opens adjacent to "File" button and not underneath cursor.
Menu remains open

Expected behavior: Same as actual.
Workflow "FILE Select 'New text file' with 2 clicks":

Move cursor over "File" menu
Without moving the cursor, press left mouse button
Without moving the cursor, release left mouse button
Move the cursor to "New text file"
Without moving the cursor, press left mouse button
Without moving the cursor, release left mouse button

Results
Actual behavior at step:

--
Menu opens adjacent to "File" button and not underneath cursor.
Menu remains open
The "New Text File" option highlights
No visible change ("New text file" remains highlighted)
The menu closes AND new file editor opens

Expected behavior: Same as actual
Workflow "FILE- Select 'New text file' with single click"

Move cursor over "File" menu
Without moving the cursor, press left mouse button
Keeping the button pressed, move the mouse to "New text file"
Without moving the cursor, release left mouse button

Results
Actual behavior at step

--
Menu opens adjacent to "File" button and not underneath cursor.
The "New Text File" option highlights
The menu closes AND a new file editor opens

Expected behavior: Same as actual.
Workflow "COGWHEEL - View menu options"

Move cursor over cogwheel
Without moving the cursor, press left mouse button
Without moving the cursor, release left mouse button

Results
Actual behavior at step

--
Menu opens directly beneath mouse cursor
The menu closes and the option that covered the cogwheel has been selected

Expected behavior: NOT this. "Press+Release," aka "click," should simply open the menu for viewing.
Workflow "COGWHEEL - Select 'User tasks' with 2 clicks":

Move cursor over cogwheel
Without moving the cursor, press left mouse button
Without moving the cursor, release left mouse button
Move the cursor to "User tasks"
Without moving the cursor, press left mouse button
Without moving the cursor, release left mouse button

Results
Actual behavior at step:

--
Menu opens directly beneath cursor
The menu closes and the option that covered the cogwheel has been selected

Expected behavior: NOT this. After clicking the menu, it should remain open so an option can be selected with a second click.
Workflow "COGWHEEL - Select 'User tasks' with single click"

Move cursor over cogwheel
Without moving the cursor, press left mouse button
Keeping the button pressed, move the cursor to "User tasks"
Without moving the cursor, release left mouse button

Results
Actual behavior at step

--
Menu opens underneath cursor
The "User tasks" option highlights
"User tasks" open

Expected behavior: Approximately this. The menu should have opened adjacent to the cogwheel icon rather than directly underneath the cursor.
[1]:
The actual name is not described in the interface guide but This issue is about the cogwheel icon at the bottom of the Activity Bar.
Searching "is:issue is:open cogwheel" reveals 4 issues, none appear related.
 It looks like you may be using an old version of VS Code, the latest stable release is 1.86.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Does the setting given in https://github.com/microsoft/vscode/issues/167941#issuecomment-1650991424 make any difference for you?

Leaving the issue open because it seems the problem occurs for more than just right-aligned sidebars.
@SRNissen did the setting help on either OS?

Hah, I quit that job after opening this issue and completely forgot about it.
I no longer have access to the machine I experimented on, but

on my current machine
without that setting manually set

it currently works as desired - the menu does not spawn directly under the mouse when the wheel is clicked.


fix: tokens page url

:tada: This PR is included in version 1.118.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


not charging more than 65%

# python3 Full-featured-demo-code.py
Raspberry Pi power supply voltage: 4.908 V
Current current consumption of Raspberry Pi: 1143.482 mA
Current power consumption of Raspberry Pi: 5618.167 mW
Batteries Voltage: 4.200 V
Battery current (charging), rate: 21.951 mA
Current battery power supplement: 97.561 mW
Current processor voltage: 3293 mV
Current Raspberry Pi report voltage: 4947 mV
Current battery port report voltage: 4226 mV
Current charging interface report voltage (Type C): 5056 mV
Current charging interface report voltage (Micro USB): 16 mV
Currently charging through Type C.
Current battery temperature (estimated): 45 degC
Full battery voltage: 4616 mV
Battery empty voltage: 3700 mV
Battery protection voltage: 3700 mV
Battery remaining capacity: 65 %
Sampling period: 3 Min
Current power state: normal
No shutdown countdown!
Does not automatically turn on when there is an external power supply!
No restart countdown!
Accumulated running time: 166276 sec
Accumulated charged time: 166278 sec
This running time: 162639 sec
Version number: 9

more than 18hours cable connected  and Battery remaining capacity: 65 % not  charge more of this 65%,.
Some reason ? :|  Just charge only 3% since yesterday.
Batteries: 2x  3.7Volt 2600mAh 5C

How come the full battery voltage is so high?
The charging current of 22 mA looks quite normal, so something is
protecting your batteries despite the  far too high full battery voltage.
Op 07-08-2021 om 14:46 schreef hellresistor:

|Full battery voltage: 4616 mV Battery empty voltage: 3700 mV Battery
protection voltage: 3700 mV Battery remaining capacity: 65 % |

Hi.
This battery remaining capacity its bad.
It can be seen that the voltage of the batteries is 4.2v. They are fully charged.
Batteries Voltage: 4.200 V
But according to the code, the batteries are considered fully charged at 4.6V:
Full battery voltage: 4616 mV
The 'Full battery voltage' should be: 4200mV not 4616 mV

How come the full battery voltage is so high? The charging current of 22 mA looks quite normal, so something is protecting your batteries despite the  far too high full battery voltage. Op 07-08-2021 om 14:46 schreef hellresistor:
…
|Full battery voltage: 4616 mV Battery empty voltage: 3700 mV Battery protection voltage: 3700 mV Battery remaining capacity: 65 % |

i dont know :|

Hi.
This battery remaining capacity its bad.
It can be seen that the voltage of the batteries is 4.2v. They are fully charged.
Batteries Voltage: 4.200 V
But according to the code, the batteries are considered fully charged at 4.6V:
Full battery voltage: 4616 mV
The 'Full battery voltage' should be: 4200mV not 4616 mV

what is the correct address register to change the value to 4200v ??
i2cget -y 1 0x17 25

Some days ago I tried to manipulate the battery full voltage, and even
though the write operation seemed succesful, the value stored didn't change.
The only way to set full and empty voltages seems to be 'learning'.
First do a factory defaults reset.
I'd recommend starting with partly charged batteries. Set the protection
voltage as you like, 3700 mV or 3200 mV ... Do not run any UPS control
script.
Prepare a script to report on the essential register content (I should
upload my latest version to GitHub ...) . You can run this script now
and then in a terminal during the learning process to check what is
going on. Let the UPS charge the batteries. The full voltage will be
pushed up little by little staying ahead of the actual voltage by some
10's of mV's.
When you're satisfied the batteries are fully charged, disconnect the
charger. Now the Pi will drain the batteries and the UPS will push the
empty voltage down little by little staying below the actual voltage by
some 10's of mV's. When the UPS f/w decides the batteries are empty
(protection voltage/empty voltage), it will suddenly cut power to the
Pi. Mostly the OS doesn't suffer damage from the sudden power cut.
You can repeat that charging and discharging process (without the
factory reset, naturally) a few times and see how the full and empty
voltage values settle to their proper levels when things are left to the
UPS f/w alone.
Later you can of course run your control script and set the protection
voltage to whatever you think suitable.
To avoid uncontrolled power cuts, control scripts need to implement a
margin of 200 mV to stay away from the empty voltage even during sudden
load upswings in case the protection voltage is set equal to the empty
voltage.
Seconds ago I uploaded the latest version of my reporting script
UPS_report_for_UPSPlus_mqtt.py to my GitHub (I apologize, it is super
messy; I am not into GitHub; e.g. I applied changes made by Frtz13 to my
own versions of his scripts using Kompare on my laptop ;-)
It's easy python, if you wonder what's in it ;-) I managed to make it
use comma's as decimal separators. You learn something every day week ;-)
Good luck!
Op 10-08-2021 om 12:17 schreef hellresistor:

Hi.

This battery remaining capacity its bad.

It can be seen that the voltage of the batteries is 4.2v. They are
fully charged.
|Batteries Voltage: 4.200 V|

But according to the code, the batteries are considered fully
charged at 4.6V:
|Full battery voltage: 4616 mV|

The 'Full battery voltage' should be: 4200mV not 4616 mV

what is the correct address register to change the value to 4200v ??
|i2cget -y 1 0x17 25|
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/geeekpi/upsplus/issues/69#issuecomment-895908754,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANCBKNR5J5PBL3NHMSGDVZDT4D4FFANCNFSM5BXMFESQ.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

The battery full voltage occupies two bytes 0x0D-0x0E.
I wrote the script below based on another script, but the full voltage "doesn't take":
#!/usr/bin/python3
-- coding: utf-8 --
adapted from scripts provided at GitHub: Geeekpi/upsplus by nickfox-taterli
modified from script by @frtz13: ar - 08-06-2021, 03-08-2021, 04-08-2021
This script behaves as if it writes the new value, but in reality nothing changes ...
import sys
import smbus2
Define I2C bus
DEVICE_BUS = 1
Define device i2c slave address.
DEVICE_ADDR = 0x17
print("-"*60)
print("Modify batteries full voltage for UPS Plus")
print("-"*60)
FV_Mini_mV = 4000
FV_Maxi_mV = 4325
Raspberry Pi communicates with MCU via i2c protocol.
bus = smbus2.SMBus(DEVICE_BUS)
currentFullVoltage_mV = bus.read_byte_data(DEVICE_ADDR, 0x0E) << 0o10 | bus.read_byte_data(DEVICE_ADDR, 0x0D)
print("Current value:  %d mV" % currentFullVoltage_mV)
if len(sys.argv) > 1:
try:
givenFV_mV = int(sys.argv[1])
if givenFV_mV >= FV_Mini_mV and givenFV_mV <= FV_Maxi_mV :
bus.write_byte_data(DEVICE_ADDR, 0x0D, givenFV_mV & 0xFF)
bus.write_byte_data(DEVICE_ADDR, 0x0E, (givenFV_mV >> 0o10) & 0xFF)
print("Successfully set the batteries full voltage to: %d mV" % givenFV_mV)
else:
errMsg = "Valid batteries full voltage values range from {:.0f} to {:.0f} mV".format(FV_Mini_mV, FV_Maxi_mV)
print(errMsg)
except Exception as exc:
print("Incorrect parameter: {}. ({})".format(sys.argv[1], str(exc)))
else:
print("Usage: {} <batteries_full_voltage_in_mV>".format(sys.argv[0]))
print("<batteries_full_voltage_in_mV> between {:.0f} and {:.0f}".format(FV_Mini_mV,FV_Maxi_mV))


shortcut could contain skipped real edges with same nodes

An edge can have the same start and end node (a loop) but having this as skipped edge for a shortcut makes only sense if the loop is a shortcut itself (e.g. for CH with turn costs) and not a real edge. When this is a real edge this caused some problems with path extraction in a downstream dependency (NullPointerException) but I'm relative sure that this could happen for CH routing too although I could not yet reproduce it.
Assume the following graph with all bidirectional edges:
1<->1<->2
ie. 1<->1 is just one node connected to itself and the shortcut is 2<->1. (I wonder why this shortcut was possible at all as the direct edge should be always better. Probably weight(1<->2) instead of weight=weight(1<->1)+weight(1<->2) was used.)
Now if this call is done with real edge 1<->2 and to==1 it will succeed (wrong!) and then empty==false leads to a null edgeState in this later call because the graph was asked for the edge 1<->1 with from==2.

Due to the recent refactoring #1450 this does not happen in the downstream dependency. (Thx @ammagamma)


Cannot be compiled with MSVC 2013

This can be reproduced by trying to compile the VS2013 solution of RetroArch (found here) -
https://github.com/libretro/RetroArch/tree/master/pkg/msvc

1>  griffin_cpp.cpp
1>c:\libretro-super\retroarch\deps\spirv-cross\spirv_common.hpp(726): fatal error C1001: An internal error has occurred in the compiler.
1>  (compiler file 'f:\dd\vctools\compiler\cxxfe\sl\p1\c\convert.cpp', line 9608)
1>   To work around this problem, try simplifying or changing the program near the locations listed above.
1>  Please choose the Technical Support command on the Visual C++
1>   Help menu, or open the Technical Support help file for more information
1>cl : Command line warning D9028: minimal rebuild failure, reverting to normal build

Candidate patch:
diff --git a/deps/SPIRV-Cross/spirv_common.hpp b/deps/SPIRV-Cross/spirv_common.hpp
index c4829f4643..28eb7b972e 100644
--- a/deps/SPIRV-Cross/spirv_common.hpp
+++ b/deps/SPIRV-Cross/spirv_common.hpp
@@ -723,16 +723,30 @@ struct SPIRConstant : IVariant
 	{
 		Constant r[4];
 		// If != 0, this element is a specialization constant, and we should keep track of it as such.
-		uint32_t id[4] = {};
+		uint32_t id[4];
 		uint32_t vecsize = 1;
+
+		ConstantVector::ConstantVector()
+		{
+			unsigned i;
+			for (i = 0; i < 4; i++)
+				id[i] = 0;
+		}
 	};

 	struct ConstantMatrix
 	{
 		ConstantVector c[4];
 		// If != 0, this column is a specialization constant, and we should keep track of it as such.
-		uint32_t id[4] = {};
+		uint32_t id[4];
 		uint32_t columns = 1;
+
+		ConstantMatrix::ConstantMatrix()
+		{
+			unsigned i;
+			for (i = 0; i < 4; i++)
+				id[i] = 0;
+		}
 	};

 	inline uint32_t specialization_constant_id(uint32_t col, uint32_t row) const
diff --git a/deps/SPIRV-Cross/spirv_cross.cpp b/deps/SPIRV-Cross/spirv_cross.cpp
index 996c0205c3..c5db07d60a 100644
--- a/deps/SPIRV-Cross/spirv_cross.cpp
+++ b/deps/SPIRV-Cross/spirv_cross.cpp
@@ -3368,7 +3368,7 @@ void Compiler::analyze_variable_scope(SPIRFunction &entry)
 	CFG cfg(*this, entry);

 	// Analyze if there are parameters which need to be implicitly preserved with an "in" qualifier.
-	analyze_parameter_preservation(entry, cfg, handler.accessed_variables_to_block,
+	this->analyze_parameter_preservation(entry, cfg, handler.accessed_variables_to_block,
 	                               handler.complete_write_variables_to_block);

 	unordered_map<uint32_t, uint32_t> potential_loop_variables;
@@ -3393,7 +3393,7 @@ void Compiler::analyze_variable_scope(SPIRFunction &entry)
 				// The continue block is dominated by the inner part of the loop, which does not make sense in high-level
 				// language output because it will be declared before the body,
 				// so we will have to lift the dominator up to the relevant loop header instead.
-				builder.add_block(continue_block_to_loop_header[block]);
+				builder.add_block(this->continue_block_to_loop_header[block]);

 				if (type.vecsize == 1 && type.columns == 1)
 				{
@@ -3447,7 +3447,7 @@ void Compiler::analyze_variable_scope(SPIRFunction &entry)
 			// If a temporary is used in more than one block, we might have to lift continue block
 			// access up to loop header like we did for variables.
 			if (blocks.size() != 1 && this->is_continue(block))
-				builder.add_block(continue_block_to_loop_header[block]);
+				builder.add_block(this->continue_block_to_loop_header[block]);
 		}

 		uint32_t dominating_block = builder.get_dominator();
@@ -3462,10 +3462,10 @@ void Compiler::analyze_variable_scope(SPIRFunction &entry)
 				// This should be very rare, but if we try to declare a temporary inside a loop,
 				// and that temporary is used outside the loop as well (spirv-opt inliner likes this)
 				// we should actually emit the temporary outside the loop.
-				hoisted_temporaries.insert(var.first);
-				forced_temporaries.insert(var.first);
+				this->hoisted_temporaries.insert(var.first);
+				this->forced_temporaries.insert(var.first);

-				auto &block_temporaries = get<SPIRBlock>(dominating_block).declare_temporary;
+				auto &block_temporaries = this->get<SPIRBlock>(dominating_block).declare_temporary;
 				block_temporaries.emplace_back(handler.result_id_to_type[var.first], var.first);
 			}
 		}

Merged.


Default scope to "create"

Apparently, "scope" is optional and an endpoint should default to "create" https://www.w3.org/TR/indieauth/#authorization-request-li-6
I've been testing the new iA Writer, and it auths beautifully, but won't accept posts since IndieKit complains that there is no scope.
Here are some logs:
https://gist.github.com/bmann/21c01a324e6d796bfed4031639e164f2
I'm going to look at adding this in myself and will PR it if I get it solved.

OK, I'm still not sure that I got this right, but I did get iA Writer working.
Not sure if this is correct as per spec, but basically

if there is no requiredScope, it should default to create required

if there is no tokenScope, it should default to create as the default / minimal grant

  if (!requiredScope) {
    requiredScope = new String('create');
  }

  if (!tokenScope) {
    tokenScope = new String('create');
  }

Ah, I too had seen that iAWriter is not working with IndieKit, but didn’t investigate further. However, having made the changes in 8cc9899bcc1e1994a908b20df30753f2801dbb42 (and updated respective tests) but I’m still getting Invalid request error. Furthermore, I’m not seeing any logs. Can you confirm if you are seeing the same?

I made the the change as posted in check-scope and it’s working for me, and I can see logs in Heroku that look as expected.
I’m pretty sure iA isn’t requesting anything and your stricter interpretation is “better”, but you know, Postel’s Law.
Maybe try and get a new token? I didn’t get a new token and the above code made it work for me.
Let me know how I can assist!

Ahh, I was testing this change on the wrong test app… pointed to the correct one, and it works as expected! Thanks for reporting – great to have this working in one of my favourite apps.


Added managed cluster label logic to rbac-query-proxy

Signed-off-by: dislbenn<EMAIL_ADDRESS>

/hold

Closing this PR.


FIRECERT-1567: GetMethodResponseHandler in IntentReader.js

Description
Fix for RPC Only test cases failure in I register glue code
JIRA Ticket
Include a link to the JIRA Ticket linked to this PR.
FIRECERT-1567
Additional Info (Test steps / Steps to reproduce etc.)
Details on how to test the feature

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ann Kshema Mathew K seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


right-sidebar: Remove border on the top of user list

Remove the border on the top of user list, especially on the top
of "USERS" word. The border is moved to bottom of feedback section.
Zulip conversation: https://chat.zulip.org/#narrow/stream/frontend/subject/border.20at.20top.20of.20users/near/126756

Before:

After:

Automated message from Dropbox CLA bot
@rafidaslam, it looks like you've already signed the Dropbox CLA. Thanks!

merged, thanks for the quick fix, @rafidaslam !


line absent added to update last image drawn at getImageData()

To solve this issue: https://github.com/chafey/cornerstoneWebImageLoader/issues/1

onluiz - thank you for this PR!  I somehow missed this and unfortunately spent my own time debugging and fixing it, wish I had seen this.  I'll give you credit for it :)

:)


Mat/Pat - Spacing issues

Fix the following spacing issues in the form -
Private Zenhub Image
Private Zenhub Image
Private Zenhub Image

need to book collaborative UX/Tech workshop to crunch through this

@Andrew,  assigned this to you. Thanks.

TAB = Parent leave - leave type and Length TAB

TAB = Parent Leave Start Date

Nice this looks beautiful Andrew = ready to deploy

White spaces removed in this one:

this one looks good to me - my eyes say this is a pass

@Andrew-Vargas-bcgov I'm still seeing no space here - but I swear it was fixed in TEST

Tested in PROD, but looks to still need a space between the info and warning bubbles at the bottom

FYI, I also checked the TEST, there is space.

FYI Andrew, I can see the space in "Edit Mode", but when in user mode, the space disappeared.
Edit Mode

User Mode

This is what it looks like for me:

Sweet Looks absolutely beautiful!! - closing tix


After replace tooltip from a previously 'Functional Groups' or 'Salts and Solvents' is displayed on hover

Steps to Reproduce

Open Ketcher
Add to canvas 'Functional Groups' or 'Salts and Solvents'
Select any 'Functional Groups'
Click on 'Functional Groups' or 'Salts and Solvents' on canvas
Hover cursor over abbreviation

Actual behavior
After replace tooltip from a previously 'Functional Groups' or 'Salts and Solvents' is displayed on hover
Expected behavior
Tooltip should display an image corresponding to its abbreviation
Screenshots
https://user-images.githubusercontent.com/62464241/222482166-d1e397f3-94e9-4794-bde9-aa7fd531b614.mp4
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 109.0.5414.120 (Official Build) (64-bit)

Ketcher version
Version 2.8.0-rc.5
Test case
[EPMLSOPKET-10106]
[EPMLSOPKET-10107]
Additional context
Add any other context about the problem here.

Reproduced in 2.7.2; moving to 2.9.0-rc.1


Adds 'Number to Nepali Words'

Please Read
Link Submission
Please submit your link(s) to the official Vue.js Awesome List repository (read first their contributing guide).
If they merge your pull request there, it will also be updated in this repository.
But if you really want to add it here, I would still review it.
Bug Fix or New Feature
For bug fixes or new features to this repository, delete this template text and proceed to provide information about the changes introduced in this pull request.
Contribution Agreement
By submitting a pull request to this repository, you are accepting this contribution agreement:

I dedicate any and all copyright interest in this software to the public domain. I make this dedication for the benefit of the public at large and to the detriment of my heirs and successors. I intend this dedication to be an overt act of relinquishment in perpetuity of all present and future rights to this software under copyright law.

Hello @bibhuticoder, thanks for opening this pull request, I just have a couple of issues tho:

You added this entry to the "Open Source" section, but there is no LICENSE in the repository. And;
I couldn't find any usage Vue.js in the source.

Thanks! :pray:

@rmjordas Sorry for the trouble.
I have updated the pull-request with another repository. Please verify and proceed with it.
Thanks for maintaining awesome-vue 🙂


Add a way to "unmatch" a package

Any idea when this might be added? I unintentionally added a match for a package that definitely was not a correct match, and now it wants to update. Barring this enhancement, is there any way to manually unmatch that package, or do I need to completely remove this app and reinstall it?

Any idea when this might be added? I unintentionally added a match for a package that definitely was not a correct match, and now it wants to update. Barring this enhancement, is there any way to manually unmatch that package, or do I need to completely remove this app and reinstall it?


View Assignments in Work List

Reference: 1608E068
Description
As a user, I want to see the assignment of a work item in the work item list view.
Functional Acceptance Criteria

The work item list view shows the current assignee in the list entries.
The assignee is not editable in the list view.

Non-Functional Acceptance Criteria
None.
Dependencies

Userstory #341

@michaelkleinhenz the current designs have the ability to assign a user from the list view. Is this a dev limitation? If so let us know @mindreeper2420 so we know how to move forward with you.

@Mgranfie This is not a dev limitation.

The assignee is not editable in the list view.

Considering this comment is old, I am just confirming that it is still valid @michaelkleinhenz . We are not updating assignee from list view.
@Mgranfie : Can you please give me the link to wireframe of list view with assignee column ?

@pranavgore09 in general, UX works ahead of implementation and defines features that are not necessarily in the current "iteration step" of the the implementation (aka this story). You, as the assignee, should decide if it is in the story or not. Some points to be considered:

is the story getting too large? So is it still a "reasonable" chunk of focussed work that can be done in a sprint by the team without compromising other implementation work?
is the story "exceeding context", does the story implement several different usecases or contexts?
is the task to be added to the story difficult to implement or will add considerably more work? (this is basically a condensed consideration of the above two things).

My 0.02€ on this: I think allowing change of assignee from the list does not add considerably more work and would not exceed the scope, so I would add it to the story.

@pranavgore09 @michaelkleinhenz
InVision Links to the Wireframes:
(I don't have permissions to edit this story, adding links here...)
Desktop Prototype:
https://redhat.invisionapp.com/share/2D92EG6BH
Mobile Prototype:
https://redhat.invisionapp.com/share/AH8X9RLU3
Mobile Workflow:
https://redhat.invisionapp.com/share/PW95K913C

thanks @michaelkleinhenz , :laughing: Got better clarity.
As per the designs by @Mgranfie , assignee is visible on detail page only.
Story: View Assignments in the work item list.
UX:

Wireframe - where to show assignee on work item list.

UI:

Read work item fields and show assignee on list view.

Core:

No additional tasks (Assignee is already present in the fields.)

Hey @mindreeper2420
Can you please help UI team to show the place where to display assignee on list view ?
https://redhat.invisionapp.com/share/2D92EG6BH this List view is without assignee column. We will need the prototype which has list view with assginee column.

@pranavgore09 here is the correct link: https://redhat.invisionapp.com/share/T89G321KQ#/195933683_A-0100.
The Assigned User's profile picture appears to the left of the kabob menu. If there is no picture, then the generic user icon (pficon-user) is shown, with a box around it.

@mindreeper2420 thanks, got it.
Currently it is only assignee but in future there will be more like created_by in the same fashion(show profile pic).
And we do not have column headers like - title, id, assigned to etc etc.
So, when we show created_by in future, it will not be predictable that which column means what.
I can think of - when we hover on the icon it should say Assigned to USER_FULL_NAME in this story.
Is it already there ? I could not interact with the link you given so I thought I just write you this.

Completed in Sprint#124.


amd-feature-targetid merged in

Would it be ok if we pushed this branch to the Tensile repo instead of your fork, then opened a PR using this new branch? I've disabled fork PR CI runs temporarily


perf: SonarCloud Metrics Fix

ignored lines in backend/src/server/httpUtils.test.ts that sonarcloud mistakes for mongodb credentials ( they were mocks)
ignored all *.json files from being included in sonarcloud coverage metrics.

Sonarcloud check is failing because the new configuration I've added will not be complete unless sonarcloud get's the current PR number for the PR analysis. Aparently it's a required field. I've modified the sonarcloud workflow file to run a bash script that gets the PR number from the ${GITHUB_REF} environment variable and modifies the properties file.

This may not be the best way to accomplish the desired outcome...


compare immutable with mutable values

fluent-asserts/core/fluentasserts/core/array.d(20,26): Error: cannot implicitly convert expression reference of type string[] to immutable(string)[]

Same issue with const and the other way around:
fluent-asserts/core/fluentasserts/core/array.d(20): Error: cannot implicitly convert expression reference of type const(string)[] to string[]


Package does not exist. Attempting to creating it...

Upload 97%, then hint "Package test does not exist.Attempting to creating it...", and finally failed. What should I do?

The log is  Could not create package HTTP/1.1 404 Not Found. If you have any idea, please let me know, thanks very much.


intermittent Jenkins Linux amd64 CI failures

seems to be failing fairly frequently lately and typically succeeds when retried, e.g.:
https://www.onnxmlir.xyz/jenkinx/job/ONNX-MLIR-Pipeline-Docker-Build/11009/
https://www.onnxmlir.xyz/jenkinx/job/ONNX-MLIR-Pipeline-Docker-Build/11006/
cc @tungld
I can't see from the logs why it failed:
{
  "url": "https://api.github.com/repos/onnx/onnx-mlir/statuses/a92406f8367c7c01345b1d7eb311439984d70139",
  "state": "failure",
  "description": "Build #11006 [synchronize] Merge branch main into s... failed after 28 min",
  "context": "Jenkins Linux amd64",
  "message": null
}
[...]
ERROR: script returned exit code -1
Finished: FAILURE

My PR #2232 also suffered this failure even after I tried to build it again.

I continue to see quite a lot of Linux amd64 CI failures which require a retry, e.g.:
https://www.onnxmlir.xyz/jenkinx/job/ONNX-MLIR-Pipeline-Docker-Build/11207/
How can I tell if this was because of OOM?
I'm not sure but I don't think much else was building at that time.
Is there any way we can reduce the frequency of these failures? (add memory or limit to fewer concurrent jobs?)

I think the failures are less frequent now, yay, so closing this issue


Questions on the training mode

Dear author,
Could you tell me the difference between the training mode 'Recons' and the training mode 'Fusion'? Which mode should be used first? Thank you.

By the way, why do you change the brightness of test image by simply * 5 and divide the mean value of pixels of the green?
https://github.com/megvii-research/DVN/blob/58735ac57472cf53d8af291456867f134e53db33/dataset.py#L138

Dear ronjonxu,
Did you have run this code? I have some problem on env setting...
When I install the requirement package, the refile latest version is 0.4.1, so I can't run this code, please give me some help, thank you!


check bodyUsed property for post request

R: @philipwalton @jeffposnick @tropicadri
Fixes #2552
It seems analytics.js used to send GET request so it didn't have any problem. However, it has been changed to POST request only. As a result, it sends an empty payload.
Solution: Check the bodyUsed property and if it is false, make it to use url.searchParams.

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


Training Curves

Hi!
I'm wondering if anyone can provide some training statistics (e.g. the tensorboard data during 1st and 2nd stage training)?
I want to make sure if I'm on the right track.
Thanks!

I can provide tensorboard data of stage 1 training + the first 34 epochs of stage 2 + train log. The model I'm training already sounds okay but I expect much improvement once joint training kicks in.
Also, please keep in mind that I was resuming training multiple times, so I guess some of those tensorboard data might reflect that (if data is incremental - I've not used tensorboard before, so I don't know whether it rewrites old data with new training or appends to them... however, the first 150 epochs of stage 1 seem to be there).
https://file.io/C6TufCPLGjmm

Hi, the link of your file seems dead, is it possible for you to reupload them ?
Thanks.


[Feature Request] Alter Express Object Attributes

Not yet finding if this is possible, so sorry if I missed it.
The idea is having it so when someone buys something, it alters one or multiple attributes to (an) existing Express Object(s). Such as, showing that someone bought a ticket for an event, and putting their username in a list, or altering it so they now have that ticket bought.
Is this currently feasible? If not, could we get this added please?

You could use community store events to react when somebody buys something and do whatever you want with the data.
The event you probably want to use is on_community_store_order which is dispatched when an order has been completed. From that event, you get the order object and from that object, you can grab the customer ID which in turns allows you to get the customer details and do whatever you want with them like updating an Express object.

@mnakalay well, I'm more of a system arch/implementer myself. How would I connect those dots? I think I understand what you mean, but I'm not sure how to execute on that.

@BloodyIron first I'd suggest you have a look at the event example package @Mesuva has. You can find it here: https://github.com/concrete5-community-store/community_store_event_example
You can just use that as a base for what you want to do and make sure it hooks to the event you need. In my previous message, I suggested on_community_store_order
I also suggest you change the package handle so it doesn't interfere in the future if you need to use this event package example again.
Now that's just the first step. Your second more difficult step is to do the Express stuff.
Luckily, Express does have some good doc available and it works in a way that tries to be intuitive. I suggest you read this page from the doc: https://documentation.concrete5.org/developers/express/creating-reading-searching-updating-and-deleting-express-entries.
As a quick-and-dirty first step helper, what you can do is:
In the event package, edit the file controller.php and, inside the on_start() function, keep only the event you need.
If you keep the on_community_store_order as I suggested you see that it calls the function 'orderPlaced' which is inside the file community_store_event_example/src/Event/Order.php
You will put your Express updating code inside that 'orderPlaced' function.
If you don't need any of the product related events, just delete the file Product.php
you see you can get the order object from the even in that function by doing:
$order = $event->getOrder();
Once you have the order you can get the user ID doing
$customerID = $order->getCustomerID();
that customer ID is actually the same as the Concrete5 user ID so once you have it you could use C5 user helpers but I suggest you use Community STore own customer class.
So you would so something like:
$customer = new \Concrete\Package\CommunityStore\Src\CommunityStore\Customer\Customer($customerID);
and then you could do:
$email = $customer->getEmail();
For the Express side of things, I'll let you read the doc.
If you want to know more about what you can do with the order and the customer, look in Community Store for the files community_store\src\CommunityStore\Order\Order.php and community_store\src\CommunityStore\Customer\Customer.php
Any function in them that starts with 'get' and is public you can use the same way we did above to get the data you need. So, for instance, you can do:
paymentMethodeName = $order->getPaymentMethodName();


non-issue: Thanks, this is an awesome library

I just wanted to say  thank you, I made this in about 10 minutes and it's awesome!

Thanks!


Morphdom Exception: component.js:257 TypeError: Cannot read properties of undefined (reading 'nodeType')

Component only works until parent component removes it and adds a different instance with a different model... only happens when used as child component. just get this any time I change the <select> model value via the select:
component.js:257 TypeError: Cannot read properties of undefined (reading 'nodeType')
    at morphdom (morphdom.js:727:29)
    at MorphdomMorpher.morph (morphdom.js:9:12)
    at messageSender.js:229:27

Currently reproducible via sidewaysdata user dashboard. Just change "type" dropdown to any value. Happens any time dropdown value changes. Cards get added/removed from DOM still fine with this error. However, once you re-render the whole component, it stops working.
Again - only happens when a child component. When used directly from Django view I do not see this exception.

@adamghill Here is a video:
https://youtu.be/pDZQa87mU4Y
I tried to fix this in sideways data commit cb98d2c. So basically all I have is:
{% unicorn 'find_events' user=user filter_event_ids=selected_event_ids %}

Then I just change selected_event_ids via:
    def switch_tab(self, new_tab: str):
        self.current_tab = new_tab
        if self.current_tab == 'competing':
            self.selected_event_ids = self.competitor_upcoming_event_ids
        elif self.current_tab == 'organizing':
            self.selected_event_ids = self.my_organized_event_ids
        elif self.current_tab == 'volunteering':
            self.selected_event_ids = self.upcoming_volunteering_event_ids
        elif self.current_tab == 'volunteer-captain':
            self.selected_event_ids = self.upcoming_volunteering_captain_event_ids
        elif self.current_tab == 'completed':
            self.selected_event_ids = self.competitor_entries_completed_event_ids
        else:
            self.selected_event_ids = []

But this doesn't work.
So I have no way to have multiple children in the same component? I am very confused. It's pretty frustrating.
The exception in the ticket and the child component might working may or may not be related...

@adamghill
so my workaround for this was to abstract out the 2nd level child (parent -> dashboard -> filter/list component) into a separate class. Then we just have one component... and use inheritance...
class DashboardEventsBrowserView(FindEventsViewCommon, UnicornView):

this sucks
I do not understand why my component works fine when used directly from main view, but not from another component? I am not doing any thing fancy IMO - I pass a list of ids to the component and show some dropdowns.

I think this should be fixed by 0.58.0 of Unicorn which drastically improves child components. Feel free to re-open this if it still is a problem, though.


Indent arguments/parameters based on line of first argument/parameter

The julia-ts-align-argument-list-to-first-sibling and julia-ts-align-parameter-list-to-first-sibling options apply regardless of the context. If set to t, you can end up with something like:
some_function(
              a,
              b)

It seems to me that if the first argument is on its own line, the user would always want that case to indent as:
some_function(
    a,
    b)

That is, the two configuration variables are really only useful for distinguishing between the following two indentation styles, where the first argument (or parameter) is on the same line as its parent node:
# julia-ts-align-argument-list-to-first-sibling is nil
some_function(a, b, c
    d, e, f)

# julia-ts-align-argument-list-to-first-sibling is t
some_function(a, b, c,
              d, e, f)

This PR modifies the indentation rules to check if the first argument or parameter is on the same line as its parent node, in which case the configuration variables are honored. If the first argument or parameter is on its own line, then it is indented from the parent node's beginning of line regardless.
This is of course subjective... Do others agree that this is the intuitive behavior?
I suppose another option is to expand the configuration variables to make this behavior opt-in. I'd be happy to do that.
Also, my elisp is pretty rusty, and I'm very new to tree sitter. I'm happy to make any updates that are necessary to improve my PR!

I'm noticing lots of places where I could make the indentation rules more contextual like in this PR. For example, assignment expressions, tuple expressions, curly brace expressions, and vector expressions. I'm happy to keep plugging away at those changes, but I'll wait to see what folks think of this, first.

Hi @danielmatz !
This is awesome! It really makes sense to me and it will make my workflow easier, since sometimes I need to switch between those two methods :) Thank you very much.

I'm noticing lots of places where I could make the indentation rules more contextual like in this PR. For example, assignment expressions, tuple expressions, curly brace expressions, and vector expressions. I'm happy to keep plugging away at those changes, but I'll wait to see what folks think of this, first.

Perfect! Thank you again!


Fixing errors introduced by recent JAX release

Summary
Needed to add additional Array wrapping in one spot in rotating_wave_approximation in which a raw JAX array was controlling a binary operation.
Also, in final_state_converter, JAX arrays were being fed into quantum info types, e.g. DensityMatrix, raising an error. I've changed this function to always convert the array to a numpy.array before wrapping in cls, as cls will always be a quantum info type, which can only accept numpy.arrays in any case. The doc string for this function has been updated to explain this.

Updates:

Added release note, as suggested.
Discovered that new JAX release advanced sparse support enough that LindbladModel.evaluate_rhs can now be reverse mode differentiated when in sparse mode. Removed this caveat from the documentation, and changed the test case for this function to test reverse-mode autodiff instead of forward mode (reverse is more restrictive so it's enough to just test this one). This test change was done by deleting the forward mode test case, which overrode an inherited reverse-mode test case.
Due to the above, bumped the minimum JAX version, and explained this in the release note.


Help migrating yaml build and classic release pipelines to multi-stage yaml.

I'm looking at migrating my old CI/ CD pipelines which are a mixture of the new YAML pipelines for builds and the 'classic' UI-based Releases pipelines into a multi-stage, environment-based YAML file, but I'm struggling to find the docs/ examples that follow along with what we are currently doing.
There are usually three environments: Test, UAT and Production.
They're all AKS based and at the moment the build that creates the docker container(s) will, as part of it's output, store a DeployService.yml file that's used by the release pipeline to update each environment (by modifying the image tag, and some environment variables) prior to running kubectl apply -f DeployService.yml.
Currently I've got several projects that follow this pattern:

Name
Purpose
Type
Description

{project-name}.ci.yml
CI build
YAML-based
Triggered on pull-requests. Builds, run tests, etc. The only output is test results and success/ failure in Azure DevOps.

{project-name}.cd.yml
CD build
YAML-based
Builds the output upon successful PR into master. This is more often than not a docker container.

{project-name} - Release
Release Pipeline
'Classic' UI
Release pipeline, triggered by {project-name}.cd.yml success and deploys into the Test environment. From there it's promotion to UAT and Production.

I'd like to move them all to the newer mult-stage YAML build and release pipeline using a single file.
As it stands, I've created a new yaml file by using the new pipeline template and then copying and pasting bits of my existing CD builds.
At at the moment the new build can:

Do a build that creates and pushes the docker container.
Trigger the release that pushes into the Test environment.

As part of this, the template created a manifests folder in the repo that contains a deployment.yml and a service.yml. It also created an environment which connected to our Test AKS cluster.
So my questions are these:

Given the deployment.yml needs to contain environment specific variables (think ASPNETCORE_ENVIRONMENT), how do I provide those additional environment specific variables per-release environment?
Can I re-create the same promotion-style release pipeline that I had using the Classic UI releases?
Will the CI build fit into this process at all? Or should I leave it in a separate file?
Given there are lots of projects that follow this pattern, and for the most part they all do the same thing but build project specific containers, is there any reusability available between repos?

Also very interested in answers to these questions, facing the same issues.

Would also like to express an interest in this question.

Closing this stale issue. Feel free to reopen if this still needs to be addressed

Closing this stale issue. Feel free to reopen if this still needs to be addressed


XDM Management & Monitoring: add JMX layer

all major management inrefaces must have corresponding MBeans:

XDMSchemaDictionary
XDMSchemaManager
XDMUserManagement, etc..

done.


win32: Restore main screen buffer

vim.exe --cmd quit quitted without switching back to the main screen buffer and the alternate screen buffer was kept shown.
Restore the main screen buffer even when termcap mode has not been started.

I've confirmed that both vim.exe --cmd q and vim.exe -c q restore the main screen buffer correctly.

thanks. I had the feeling, this was a somewhat recent regression, but I couldn't find a commit that touched this part of the code recently. Oh well 🤷

v9.0.1235 or v9.0.0868 might be related. This only happens when quitting in the initialization stage. So, no one might not notice this...
I noticed this when I tried to run the tests in runtime/indent.


Add GitHub score on the dashboard

Make GitHub score part of the dashboard and make it stand out since we don't have a lot of other content

code refactor


ci: bypass team check if it is dependabot

Description
With the new ansys/actions v8 (see #3470) I realised that dependabot cannot trigger the matrices because it is not part of any team, so that part of the action fails.
Issue linked
Related to #3470
Checklist

[x] I have visited and read Develop PyMAPDL.
[x] I have tested my changes locally
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the PyMAPDL coding style guidelines.
[x] I have added appropriate unit tests that also ensure the minimal coverage criteria.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have marked this PR as draft if it is not ready to be reviewed yet.
[x] I have made sure that the title of my PR follows Commit naming conventions (e.g. feat: adding new MAPDL command)

@pyansys-ci-bot LGTM.

Merging without waiting...


[WIP] Bump default mysql jar version to v8.3.0

[OB3] Bump default mysql jar version to v8.3.0
Issue link: https://github.com/wso2/financial-open-banking/issues/42
Doc Issue:
Applicable Labels: Spec, product, version, type (specify requested labels)

Development Checklist

[x] Built complete solution with pull request in place.
[x] Ran checkstyle plugin with pull request in place.
[x] Ran Findbugs plugin with pull request in place.
[x] Ran FindSecurityBugs plugin and verified report.
[x] Formatted code according to WSO2 code style.
[ ] Have you verify the PR does't commit any keys, passwords, tokens, usernames, or other secrets?
[ ] Migration scripts written (if applicable).
[ ] Have you followed secure coding standards in WSO2 Secure Engineering Guidelines?

Testing Checklist

[ ] Written unit tests.
[ ] Documented test scenarios(link available in guides).
[ ] Written automation tests (link available in guides).
[ ] Verified tests in multiple database environments (if applicable).
[ ] Verified tests in multiple deployed specifications (if applicable).
[ ] Tested with OBBI enabled  (if applicable).
[ ] Tested with specification regulatory conformance suites  (if applicable).

Automation Test Details

Test Suite
Test Script IDs

Integration Suite
TCXXXXX, TCXXXX

Conformance Tests Details

Test Suite Name
Test Suite Version
Scenarios
Result

Security Suite
VX.X
Foo, Bar
Passed

Resources
Knowledge Base: https://sites.google.com/wso2.com/open-banking/
Guides: https://sites.google.com/wso2.com/open-banking/developer-guides

We are checking the possibility of removing this jar as well. Waiting until further confirmation.


Raster source interpolation

Fixes #14415
Adds a interpolate option to ol/source/Raster for use when scaling output to view resolution.  Document that this does not apply if scaling input sources.

Many RasterSources have only one source, and I cannot think of any reason for the interpolate settings of that and the parent to be different.  If there are multiple sources with conflicting interpolation requirements the only way to handle that to to set resolutions: null which avoids scaling when rendering to the map and makes interpolation at that step irrelevant.  It would be possible to check all sources for conflicting requirements and set this.useResolutions_ false to keep that transparent to the user.

@mike-000. I can think of cases where users want to set the interpolation option of input and output separately. For example, if you want to classify input rgb values into a reduced color palette, but you don't want a blurry output. In this case you'd want your input source with interpolate: false, and the output with interpolate: true.

That would be needed only if the operation was being done at a resolution which was neither the native resolution of the source nor the view resolution.  If there is no scaling there will be no interpolation regardless of the setting.  Can that happen if there is a single source?  Multiple sources with conflicting native resolutions perhaps, but then the first source could be interpolate: true.

That can happen also with a single input source, when the view resolution is higher than the highest input source resolution.

OK, a user option would be needed for that (and also where retina tiles are used on a low res display) but I think using the source setting as default would better than defaulting to true.  And if the sources have conflicting settings and no user option is provided to only use view resolutions.

I think having to specify interpolate: false once for input and again for output is confusing.  If we have a user option for output which defaults to true regardless of what was used by the input then the docs should make it clear that specifying resolutions: null allows the input source interpolation settings to be used throughout.

Whichever approach is taken one example which may need attention is canvas Sea Level.  The interpolate: false option in the tile source now has little purpose as there would be no interpolation at tile resolution.  It was needed to prevent interpolation of red and green pixels which caused thin lines of sea close to 25.6 meter intervals and a more noticeable one at 3107 meters.  However as the maxZoom of the DEM is much smaller than the base map there is now a very blurry appearance when zooming in.

I think we need to review interpolate before spreading it around any more.  Ideally we would have a list of example uses cases that were clearly described and as simple as possible.

First of all, many thanks for the brilliant work of all contributors of OL.
Probably I can give you an use case for different interpolate settings:
For meterological maps, eg temperature, wind speed or what so ever, the rgb values of the input source are representing the value. This source is modified with raster-source manipulation to set an individual colouring sheme. For this case it would be a benefit to have the option of using the source with interpolation: false and setting output interpolation: true, to avoid artefacts in the output due to interpolation of the source.

@hweri69 The input source interpolation setting is always respected.  Currently the output is always interpolated, and the only way to avoid that is to specify resolutions: null so the output is not scaled.

@mike-000 Thank you for the quick reply. Anyway I have to admit, I can't follow you. Probably I'm doing something wrong.
var mapextent = [-20037508.3428,-20037508.3428,20037508.3428,20037508.3428];
var mapext31287 = [19500,219500,720500,620500];

var map = new ol.Map({
  maxTilesLoading: 256,
  pixelRatio: 1,
  layers: [
    new ol.layer.Tile({
        className: 'grey',
        source: new ol.source.OSM()
      })
  ],
  target: 'map',
  view: new ol.View({
    projection: availproj.EPSG31287,
    center: ol.extent.getCenter(mapext31287),
    extent: mapextent,
    resolutions:[2048,1024,512,256,128,64,32,16,8,4,2,1],
    zoom: 3
  })
});

var xlsource = new ol.source.ImageStatic({
        url: './raster.png',
        interpolate: true,
        projection: availproj.EPSG31287,
        imageExtent: mapext31287
      });

var dingsda1 = new ol.source.Raster({
    resolutions: [1],
    sources: [xlsource],
    operation: function(pixels) {
      var opa = pixels[0][3];
      var r = pixels[0][0];
      var g = pixels[0][1];
      var b = pixels[0][2];
      var pixel;
      pixel = [r,g,b,opa];
      return pixel;
    },
    threads: 0
  });

var xl = new ol.layer.Image({
      zIndex: 2,
      opacity: 0.9,
      source: dingsda1
    });

map.addLayer(xl);

The output depends only from interpolate option of the ImageStatic Source, any manipulation of the raster resolution option makes no difference, so if input interpolation is set to false, the output is without any interpolation.


⚠ ClamAV has degraded performance

In ed42078, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 123 ms

Resolved: ClamAV performance has improved in 9363c4f after 534 days, 15 hours, 40 minutes.


How do you make one object (Transformable Node) face another object?

As the question says how do you make one of the object face the other object? Their locations are randomly placed so you don't know how much to rotate it by, you only know their positions using node.getWorldLocation() (idk if that's even useful for the rotation purpose).

try getting the get world loocation of one object and according to that apply setlocalrotation to the other object

setlocalrotation takes in a quaternion while a getworldLocation returns a Vector3 so I don't really understand what you're trying to say. Instead I'm just working with the objects looking at the camera itself as workaround.

For advice and general questions, please use the Sceneform tag on
Stack Overflow: https://stackoverflow.com/questions/tagged/Sceneform
In addition, please see the Sceneform SDK developer documentation and API reference.


Rename james.data.xml.clss to james.text.xml.clss

Rename james.data.xml.clss to james.text.xml.clss.
Everything that is a text should be inside james.text.* directory.
For example

james.text.xml
james.text.json
james.text.strings

@mdbs99 You're trying change the unit name, aren't you? So, before change their names, we need analyze what is the common concept round all these things, because if these concepts share the same unit, imply they must have something in common. Thus, let's see what Wikipedia says about them:
XML:

In computing, Extensible Markup Language (XML) is a markup language that defines a set of rules for encoding documents in a format that is both human-readable and machine-readable. (...) The design goals of XML emphasize simplicity, generality, and usability across the Internet.[5] It is a textual data format with strong support via Unicode for different human languages.

JSON

In computing, JavaScript Object Notation or JSON (/ˈdʒeɪsən/ JAY-sən),[1] is an open-standard file format that uses human-readable text to transmit data objects consisting of attribute–value pairs and array data types (or any other serializable value). (...) JSON is a language-independent data format.

INI File

The INI file format is an informal standard for configuration files for some platforms or software. INI files are simple text files with a basic structure composed of sections, properties, and values.

We can see some common terms among them: standard, text and format. But, to an INI file, it's a informal standard; text is too much vague. So, the best term is format. Therefore, the new names should be:

james.format.xml.*
james.format.json.*

What do you think?

I think you're right, thanks. I will change as you're proposing.


Multi-region Deploy Notifications

Thought (Non Blocking)
I dont really have exceedingly strong opinions for this specific repo since it's very internally facing and likely not prone to much updating past a certain point but just a few thoughts:

main.go itself was a bit hard to follow and the biggest opportunity for cognitive overhead reduction might be breaking out all of the cloud storage operations into it's own separately testable interface
On a similar note for main_test.go having table-driven tests are sometimes a bit painful in the setup but make additional test cases a breeze once they're established, as opposed to a test-function-per-case approach.

Generally this is stuff that we'll find elsewhere, but figured now was a good time to at least mention a few bits. Otherwise it's lookin good!

Thought (Non Blocking) I dont really have exceedingly strong opinions for this specific repo since it's very internally facing and likely not prone to much updating past a certain point but just a few thoughts:

main.go itself was a bit hard to follow and the biggest opportunity for cognitive overhead reduction might be breaking out all of the cloud storage operations into it's own separately testable interface
On a similar note for main_test.go having table-driven tests are sometimes a bit painful in the setup but make additional test cases a breeze once they're established, as opposed to a test-function-per-case approach.

Generally this is stuff that we'll find elsewhere, but figured now was a good time to at least mention a few bits. Otherwise it's lookin good!

This makes sense, and something I considered doing. I hesitated with the cloud operations because I looked to see if they already had an interface, but then realized it would be pretty big to mock out, but I could just make my own wrapping interface instead for the bucket operations.
I think my only hesitancy is to put a bunch of work into something we might rarely touch, like you mentioned. That's why I kept the tests kind of ugly. But if you think it would be worth investing more time in, I'm happy to make those changes.
This is all stuff I agree with for our main projects, I just felt like it might not be worth investing the time in for this one-off 🤷🏻♂


Can i run grok in gcolab?

I tried to load this model in Google colab but it fails to load, is there any way that it will get optimized or something to load in Google colab or any other notebook like paperspace?

What was your error message?

What is the overall size of the model file?

What is the overall size of the model file?

Well the magnet file contains a 300GB checkpoint 🙃

Lol... Run grok in gcolab? Lool. Firstly, you simply don’t have enough space, even with a PRO subscription, and secondly, you don’t have enough resources to download such a model! 🤭

It may not be feasible, even with quantized variants. Wait a bit and we might be able to see those tuned quantized variants.

What is the overall size of the model file?

Well the magnet file contains a 300GB checkpoint 🙃

Let's wait for llama3🙃

Impossible to run in Colab. First, Colab doesn't have enough disk space to even load the model. Secondly, there isn't enough memory. Maybe llama.cpp will add support and you can run quants

Yeah colab is unfortunately not enough to run this model currently.


Exclude test providers from autoloading

[ ] Added or updated tests
[ ] Documented user facing changes
[ ] Updated CHANGELOG.md

Resolves #683
Changes
Previously an instance of all detected service providers was created during analysis. This is changed. Now  classes with namespaces defined in psr-4 paths in the autoload-dev configuration of composer.json are skipped.
Breaking changes
Since a number of classes previously autoloaded are skipped runtime behavior is changed. But I don't think it changes the outcome of the analysis.

Thank you very much @kwebble !


Removes the need to specify the package

This commit removes the need to specify the package by catching the
ast.File node in the astStream. The File node is then passed in a
wrapper instance to the interfaces. The astStream now processes nodes by
File rather than by order. The README was adapted to these changes.
This fixes #3

@haleykell Thank you for your contribution


DSE Service Implementations

Right now, we don't have a process in-place for bootstrapping DSE resources (i.e. Solr Config, Spark Jobs, etc.). Once we have a process in place for that, we'll need to fill the in the DSE implementations for services like Search and Suggested Videos.

@LukeTillman @SonicDMG Is it still relevant? I'd suggest to solve/close the issue.

Closed as inactive


Dashboard: Page Sub-navigation with Popover Panels

Adds page sub-navigation to the first two pages of the dashboard app
Adds the 3 different drop down styles
Adds Popover panels (the larger popover for color/style/layout filtering)

as an aside, looks like there's some linting on includes/Dashboard.php that's causing travis CI to fail:
FILE: includes/Dashboard.php
--------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------
 125 | ERROR | [x] Opening parenthesis of a multi-line function call must be the last
     |       |     content on the line
     |       |     (PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket)
 130 | ERROR | [x] Closing parenthesis of a multi-line function call must be on a line by
     |       |     itself (PEAR.Functions.FunctionCallSignature.CloseBracketLine)

as an aside, looks like there's some linting on includes/Dashboard.php that's causing travis CI to fail:

Very odd. I opened #752 to fix this. Once that's merged this branch can be rebased.

@mariano-formidable I think separating out those changes make this a lot cleaner. Nice suggestions!


What did you think about the process of writing a command?

What went well, and what could be improved?

tl;dr: WP-CLI's well documented/organized code made it pretty easy, but I struggled a bit with the test suite because I wasn't familiar with Behat and haven't worked much with Cucumber style tests.

Writing the actual command was pretty straightforward. It is mostly a wrapper around the existing wp plugin subcommands. Implementing those was just just a matter of grokking WP-CLI's php/commands/plugin.php to see how things were implemented. Where I needed custom logic beyond spawning a wp plugin command with WP_CLI::run_command(), I was glad to see that most of WP-CLI is pretty well documented. That was a huge help. I really like how the PHPdoc stuff in commands works to validate arguments. It forces you to write proper documentation and I think that is a great feature.
Implementing the test suite was where I struggled a bit. That was mostly due to my unfamiliarity with Behat and Cucumber style tests. I found that WP-CLI's test suite is pretty comprehensive, and it was easy to poke through your features to figure out what I needed to do. Bonus points for including tests in the scaffold -- I think its great that you're advocating testing!

👍  Thanks! If any documentation improvements cross your mind, I'd be happy to hear about them.


Create Second largest element

it clearly tackle out with all the edge cases and is the correct optimized solution for this
so if their is any case that I didn't mentioned or is incorrect I would definitely change it
thankyou

⭐ If you find this repository helpful, please give it a star! ⭐


Fix i18n fallback in country translations

This fixes an issue inadvertently introduced in version 5.0.0 by https://github.com/stefanpenner/country_select/pull/173.
In locales like en-US or de-CH, with version 5.0.0 we started seeing the ISO names instead of the common names in our country dropdowns. This PR fixes the default format so that fallbacks work correctly, while preserving the handling of nil/missing translations.

looks good. great tests! I'll release a new version asap.

@lavaturtle https://rubygems.org/gems/country_select/versions/5.0.1


Updated tutorial.md

- What I did
Followed the tutorial. Had some issues. Modified the doc adding the steps I've taken to progress.
"crictl create" was not working with the image path in container_redis.json so I changed it to the path that made it work for me.
- How I did it
Followed the tutorial, googled.
- How to verify it
Follow the tutorial on a fresh VM.
- Description for the changelog

Added clearer instructions to the tutorial.

LGTM

No need for additional testing since this is only a documentation change.
@TomSweeneyRedHat @umohnani8 @mrunalp PTAL

I might have squashed a commit which is not mine into this PR...

I think I made it... this was painful.

LGTM

@TomSweeneyRedHat @umohnani8 PTAL

LGTM

@ktzee Thanks
Merging since this is a docs only change.


Fix Error Handling in ChatBot and ImageBot

Properly pass through the HTTP Status code from the Chat/Image Bot proxy
Change-Id: I35c2852dec19ad7bdd4a61dab47e771783209614

Note: Changed ImageException to subclass Exception instead of IOException. Otherwise the exception thrown on line 567 is caught on line 569!

@ewpatton Yes it was. Check line 545 in ImageBot.java

I guess I am confused why it was changed for ImageBot but not ChatBot. Wouldn't we want ChatBot to also report -1 if a connection couldn't be established?

@ewpatton Oops, just fixed ChatBot


sails-new upgrade--joshua

Hello, sails community,
was tinkering around and felt if it will be good to maybe create a tool for sails developers to easily integrate with their database as for me MongoDB without  using a cli as for  now I am writing the CLI
and would later submit a PR, this would bring the same functionality from laravel.
Hope you guys will be willing to see how it works

@Jopchumba Sure thing, that sounds like a great idea.  Feel free to share the PR when you are ready. Appreciate the help!


provide youtube live video support also and video quality options , speedo meter(playback rate)

provide youtube live video support also and video quality options , speedo meter(playback rate).
i want this feature plz tell me  how to do this from my side or wait for update from your side.

Second this. How do you pick the video and audio streams? Would like to have at least 'broad' selection of quality from YouTube ie. 480p, 720p, 2160p etc. if they are available.


Fix typescript version

See comment in README. We should be able to pin the version in the package.json

Pinned ng-cli docker image version and added reinstall task for typescript to build script.


Update to latest plugins as of 2016-04-29

None of the plugins were bumped by a major version, so there should be
no breaking changes.
Having said that, it would probably be best if someone more acquainted
with testing the developer app had a run through it to be sure.
// @timkim @macdonst

@devgeeks if we are using something like ~1.1.0 the next time the app is built it will pick up the latest patch level. For the push plugin I need 1.6.3 or better which is why I specify ~1.6.3. So unless we need a specific new patch version some of the changes are not strictly necessary other than for clarification purposes.

Yeah, the patch bumps were mostly a case of clarification.

Ok, I've merged this into dev. When we're ready to cut a release, I'll merge into master then.


There are invalid links in the document

1.REST API menu is invalid

2.The page opened through the REST link of the Admin Api page is blank.

Yeah, I found that as well. The button in pic 1 only works in English site.
As for the link in pic 2, I reckon it points to the same endpoint as the one in pic 1, which is : https://pulsar.apache.org/admin-rest-api/?version=2.5.1# (got it from English ver.)


minDate for DateTimePicker does not honor time provided

Environment

Tech
Version

material-ui-pickers
2.1.1

material-ui
3.8.3

React
16.4.2

Browser
Any

Peer library
moment 2.22.2

Steps to reproduce

Create two DateTimePickers (start and end date); start date is provided as the "minDate" for the end date (live example below)
Click Start Date, select any date/time in the future
Click End Date, select the same date as the start date, but select a time earlier than the start time

Expected behavior
As a DateTimePicker, both the date and time should be considered when providing a minimum DateTime (if I didn't care about the time then I would use a DatePicker).
i.e. If I select 2/5/19 4:00:00pm as the start date and 2/5/19 1:00:00pm as the end date, the picker should enter into an error state.
Actual behavior
The DateTimePicker takes into account previous dates, but not a time that is previous to the selected time.
If I select 2/5/19 4:00:00pm as the start date and 2/5/19 1:00:00pm as the end date, the picker does not enter into an error state.
Live example
https://codesandbox.io/s/73v5xzxj5x

I'm sure someone somewhere has a very compelling argument why minDate should not account for the time. If there's a fear of backwards compatibility or loss of ease of use (expecting moment().add(1, 'd') to allow all times for tomorrow), then I would propose creating a minDateTime property that does allow for users to pass a Date (or moment, etc) and expect it to honor the time value that's been passed.

Duplicate of #519


AttributeError: 'module' object has no attribute 'MCP4725'

Hello,
I am using Adafruit_MCP4725 in Beagle-Bone Black Wireless device.
I am getting error:

AttributeError: 'module' object has no attribute 'MCP4725'

When I tried to run the example file:
sudo python simpletest.py

hiya we've written a new library and guide with easier instructions please visit
https://learn.adafruit.com/mcp4725-12-bit-dac-tutorial/python-circuitpython
thanks!


semantic problem

without uploading to portal- just save- will not work

@SychevIgor : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @SychevIgor, thanks for this edit. This particular article has been retired and the content was moved to https://docs.microsoft.com/azure/active-directory-b2c/session-behavior, where the same issue exists. Would you like to create a new PR against that article? Otherwise I'll go ahead and make the change - either way is fine. Thanks again!

Hi @SychevIgor, thanks for this edit. This particular article has been retired and the content was moved to https://docs.microsoft.com/azure/active-directory-b2c/session-behavior, where the same issue exists. Would you like to create a new PR against that article? Otherwise I'll go ahead and make the change - either way is fine. Thanks again!


Yandex browser problem

On Yandex browser not show "Delete element" in bottom of screen when try to delete geofence, but work in Chromium. Another messages not show too.
OS: Windows 11
Browser: Yandex
Version: <IP_ADDRESS>70

Have you checked console for errors?
Have you tried stable version?

No errors on console. Nothing happens.
I use last version 5.12

In "Incognito mode" all work perfectly. Sorry. Try to find reason.

AdGuard is reason. Sorry again.


Add fail_fast argument

Hello!
Thank you so much for developing this action. We are using it with great results in our org.
This PR adds a "fail_fast" option to the inputs and implements it. In our codebase, we have some checks that run for 1 minute and some checks that run for 45 minutes. If the check that takes 1 minute has already failed, we want to go ahead and fail the all-checks-passed action so that we do not continue to burn Github Actions minutes.
Tested in our repo using jacobcrell/allcheckspassed@v3
Screenshot below showing all-checks-passed check with fail_fast = true failed immediately with other failing check even though other check is still running.

Screenshot below showing all-checks-passed check with fail_fast = false still running with other failing check and other in progress check

Ah, yes, that is quite useful, thank you for the contribution!


chore(deps-dev): bump electron-builder from 22.6.0 to 22.7.0

Bumps electron-builder from 22.6.0 to 22.7.0.

Release notes
Changelog
Commits

Signed-off-by: dependabot-preview[bot]<EMAIL_ADDRESS>

Thank you for filling this, but we are already at electron-builder 22.8.0


language/python: create venvs with access to system site packages

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

A user may wish to use two use two brew-installed Python packages
together. For example, one might want to import numpy when using
jupyterlab or ptpython.
Currently, the only ways to do this I'm aware of is with some hacking of
PYTHONPATH or the creation of .pth files in a formula's prefix.
A better solution is to allow the virtualenvs that brew creates to
have access to system site-packages by default, so that import numpy
inside ptpython or jupyterlab just works.
Partially resolves Homebrew/homebrew-core#76950.

Review period will end on 2021-06-29 at 02:28:06 UTC.

Just to clarify what --system-site-packages does: it gives the virtualenv read-only access to the system site-packages directory (for us, that's $(brew --prefix)/lib/python3.x/site-packages). However, the virtualenv pip cannot alter any system site-packages. Attempting to do so returns an error:
❯ python3 -m venv --system-site-packages venv
❯ source venv/bin/activate
❯ pip uninstall six
Found existing installation: six 1.16.0
Not uninstalling six at /usr/local/lib/python3.9/site-packages, outside environment /private/var/folders/kb/pf4wbfc512z47l56g5072l900000gn/T/tmp.RoIdfZfoyv/venv
Can't uninstall 'six'. No files were found to uninstall.

Review period ended.


Cannot detect JS injection of reading document.cookie

If the injected JS code is like the following:
alert('cookie: '+document.cookie);

This site is not able to detect it, and it will still shows green message.
Cookie stealing is concerning, in case for example you tried to log-in to a site (within in-app browser), the host-app can inject malicious JS code to steal cookies, which essentially will steal your login session.
Currently have no idea on what to do, haven't check the repo code, just informing this behavior for now.
Optional: More background on this concern.

Maybe you could try this: https://github.com/iwestlin/InAppBrowser.com/commit/29fe7513547393c815d3b5a40e0666bc178a64a2


C++ ClientReader timeout can not be set on individual Read calls

In the C++ codebase ClientReader class does not allow setting timeout on an individual Read() call. The only way to set the reader timeout is at the time of the ClientReader construction. Which is problematic because the lifetime of the ClientReader which is a stream reader is determined by the amount of data we read from the server, which is a variable that is difficult to guess correctly in advance. The right way to set timeouts is per each Read() individually. With Read() call timeouts feature missing it is it very difficult to deal with server failures like these:  #6278. The only way to deal with server stream call failures right now is to set an overall stream lifetime deadline, which is problematic from very many prospective. If that timeout is set too high, the application suffers from long recovery time when we run into the server issues. If that timeout is set too low, legitimately long running streams are getting killed for no good reasons.

Is their any way to mimic this feature with he current C++ version?

Hello, is there any update about this request?

We do not plan to support per message deadlines for streaming RPCs.
What would it mean for one message to timeout? Would we guarantee that the server did no processing? Would the RPC still be alive? What if the server thinks everything was fine with the message?
Also, using our async API, this behavior can be approximated. Using AsyncNext with a deadline on a tag for a message send will accomplish the same idea.


Use Hostname rather than Ephemeral IPs

OpsCenter and Cassandra are currently binding to ephemeral IPs.  We know this could result in issues when the IPs change.  Ideally, we'd like to bind to hostnames.  This is going to require changes in both OpsCenter (which uses a twisted library that fails to resolve hostnames) and Cassandra (which resolves the hostnames to IPs and caches those for gossip).
In the best case, a fix is probably 6 month out, so we'll work around this for now.  Hostname resolution is probably also going to be necessary for Docker deployments.

We had a really interesting conversation on this today.
It seems that machines on GCP will retain their IP mapped to their hostname under most conditions.  This includes reboot and kernel panic.  The only case I'm aware of where you will be assigned a new IP is if the instance is stopped and then started again or deleted entirely and then recreated.
As such, it doesn't seems this is the issue we thought it was.  Great news!


Updated info.json

Added missing mandatory keys into info.json

@cs-suyog-jinturkar  Please review this


Security Issues from Dependencies

Target device: (Note: simulators are not supported for running the UIKit)
All
App Info (Note: Expo CLI managed workflow is not supported by the UIKit)
N/A
Describe the issue
There are security flags on this package. Specifically:
high      Lodash4.17.19->4.17.21
high      ajv6.11.0->v8.5.0
high      y18n4.0.0->y18n-v4.0.2
medium     hosted-git-infov2.8.5->v4.0.2
medium    mikaelbr/node-notifierv5.4.3->v9.0.1
medium    yargs-parserv13.1.1->20.2.7

To Reproduce
Some of the security issues are shown here
Others flagged by Black Duck Projects as seen below.
Screenshots

@EkaanshArora could you take a look at this this week?

I've updated the UI Kit to v3.1.3 with updated dependencies that have the vulnerabilities fixed.

@EkaanshArora looks like Black Duck is still seeing the following vulnerabilities:

dependabot has opened a PR for one of them #25


✨ Add function to map value from a ditionary

Example docker container output:
[INFO] Running normal execution

Got mapping {
  "Informational": 1,
  "Low": 2,
  "Medium": 3,
  "High": 4
}
Mapping High to 4

[INFO] Returned from execution!
[INFO] POST NEWRES RESULT!

oooh, nice. This kind of does the same as the "translate value", but multiple of them. Will definitely add a "multi-value" thing for this in the future so that you can add and remove values in the GUI instead of JSON

I think the translate function is more like a "replace", or even "sanitise" function. It finds multiple different values and replaces them with a single one.

Yep, you're right. Btw, as you're so good at using Emoji's with your plugins and such: How about we add a way to have emoji's/icons next to actions?
We'll probably end up adding something like Mitre Att&ck or Re&ct tags to actions as well in the future.

Yes excellent idea, it can reduce cognitive load and scanning time. I haven't tried but I assume emojis can already be used by adding them to the action text?
If you wanted a more structured definition, maybe a new YAML key for icon and do something to HomeAssistant and use a pre-loaded icon pack like Material Design Icons.
So for SMS - mdi:message-processing
Email mdi:at or mdi:email-send

An icon pack also has the benefit of consistency and availability cross platform.


Saving Modified PDFs with Acrobat Reader DC

When I save the pdf on the dokany mirror, I get a valid CreateFile request:
CreateFile status =<PHONE_NUMBER> - lastError = 2
###Create 0135
CreateFile : C:\Temp\B\10.pdf
  AccountName: , DomainName: -PC
		ShareMode = 0x0
		AccessMode = 0x12019f
		FILE_READ_DATA
		FILE_READ_ATTRIBUTES
		FILE_READ_EA
		READ_CONTROL
		FILE_WRITE_DATA
		FILE_WRITE_ATTRIBUTES
		FILE_WRITE_EA
		FILE_APPEND_DATA
		SYNCHRONIZE
		STANDARD_RIGHTS_READ
		STANDARD_RIGHTS_WRITE
		STANDARD_RIGHTS_EXECUTE
		FlagsAndAttributes = 0x80
		FILE_ATTRIBUTE_NORMAL
		CREATE_ALWAYS

Whereas, when I try to save it on the dokannet mirror, I get:
CreateFile('\10.pdf', {<null>, False, False, False, False, #9876, False, False}, [ReadAttributes, Synchronize], [ReadWrite, Delete], [Open], [None], [Normal]) -> ObjectNameNotFound

Maybe CreateAlways is missing on DokanNet?

We make a simple mapping from the CreateFile CreateDisposition to FileMode C# enum.
https://github.com/dokan-dev/dokan-dotnet/blob/master/DokanNet/DokanOperationProxy.cs#L181
Did you try a sample that make a CREATE_ALWAYS to see if it work ?
If it work, the issue have to come to somewhere else.

You're right, it's something else...
I ran:
int main()
{
	WORD wResId = 101;
	LPCWSTR lpszOutputPath = L"M:\\test.bmp";
	HRSRC hrsrc = FindResource(NULL, MAKEINTRESOURCE(wResId), RT_BITMAP);
	HGLOBAL hLoaded = LoadResource(NULL, hrsrc);
	LPVOID lpLock = LockResource(hLoaded);
	DWORD dwSize = SizeofResource(NULL, hrsrc);
	HANDLE hFile = CreateFile(lpszOutputPath, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
	DWORD dwByteWritten;

	//Write BITMAPFILEHEADER
	BITMAPFILEHEADER bfh;
	bfh.bfType = 0x4d42;
	bfh.bfSize = dwSize + sizeof(BITMAPFILEHEADER);
	bfh.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);
	bfh.bfReserved1 = bfh.bfReserved2 = 0;
	WriteFile(hFile, &bfh, sizeof(bfh), &dwByteWritten, NULL);

	WriteFile(hFile, lpLock, dwSize, &dwByteWritten, NULL);
	CloseHandle(hFile);
	FreeResource(hLoaded);
	return 0;
}

and the mirror received:
CreateFile('\test.bmp', {<FileStream>, False, False, False, False, #32188, False, False}, [WriteData, AppendData, WriteExtendedAttributes, ReadAttributes, WriteAttributes, ReadPermissions, Synchronize], [None], [Create], [None], [Normal]) -> Success

Did you find a difference in the calls before CreateFile ?
For me if he request to Open, it mean that he got somewhere the information that the file already exist.

If it is corrupted, try to run the mirror with one thread.

In mirror.c it works perfectly.
In mirror.cs it doesn't. I also tried with 1 thread and it didn't work.

Ok
From what I see in C# and C, there is a lots of call to file system behavior overview.pdf with Open.
After a time the C get a CREATE_ALWAYS but don't know what can be the difference.
You really have only this logs for C# ?

Here is the unfiltered log from C#:
http://pastebin.com/ddL1nVbV

@shlatchz could you make a test with the new mirror ?

Feel free to comment if you have any update.


Fixed false DDoc warnings (check parent template decl)

This is tricky. Some params in ddoc comments refer to a nested template function. This was not handled before and would cause some of the Ddoc: function declaration has no parameter warnings.
Fixed by checking the parameters of all overloads of the nested template function of the same name.
Note that it only looks for nested template functions, not regular functions. For regular functions the ddoc comment should be on the function, not the parent template declaration.
I created an overload for isFunctionParameter taking a single Dsymbol*.

Please file the issue in bugzilla, in order to list the bugfix in the changelog of the next release.
And you have to add a test case.

https://issues.dlang.org/show_bug.cgi?id=14633

Thanks. I'll review this tomorrow.

@9rnsr Did you ever get a chance to check this PR? Please let me know.
There's one missing case in utf.d, but it might not be worth complicating this DDoc logic and can likely be fixed by moving comments around.

This has been stale for half a year now. I've updated the code. @9rnsr Any comments?
This fix is one step closer to being able to enforce DDoc checks in our build system.

@WalterBright @andralex I need help getting this reviewed. Sorry for the summon, but not sure how I get this to move along.

Auto-merge toggled on


.Net: Google connector API version selection

Motivation and Context

Closes #5659
Description

The update introduces support for different versions of the Google API in various services and clients. A new enum, 'GoogleApiVersion', has been added to represent stable and beta versions of the Google API. Affected classes have been updated to accept this new parameter, and use its value when constructing API endpoints.
GoogleAI endpoints currently support only BETA.
cc: @RogerBarreto
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Done. I think this way is ok:


Add web test container agnostic client target

Depending on the web test container passed to
ResourceTestRule#setTestContainerFactory, one would either invoke
client() or getJerseyTest(), which is inconvenient. The fix is to
standardize on a new method that can be used regardless of web test
container that also doesn't infringe on the law of Demeter (as the other
solution would be to rewrite all usage as using getJerseyTest())

Coverage increased (+0.02%) to 82.296% when pulling 0141b2007f2b41383a7b1da79938a31dab8e947c on nickbabcock:webtest-client-conform into 029ef03046724f6002f0084cf702da6ed4cce2d4 on dropwizard:master.

Updated release notes in 9a41974


Bad Handlebars namespace naming

Hi, thanks for the library: it's neat, fast, and small! Kudos!
Maybe I'm missing something, but: am I the only one having issues with the Handlebars namespace?
The guidelines about namespace naming state: "DO NOT use the same name for a namespace and a type in that namespace".
The "why" is that currently, if I try to use Handlebars from my code, I can't set a "using Handlebars" directive, because Handlebars.Compile(...) is going to give a bad reference, because the compiler is going to search the Handlebars NAMESPACE, and not the class.
IMO, this means that Handlebars namespace should be renamed something else (HandlebarsDotNet, maybe?), to keep the usage of the static Handlebars class methods.
In the test classes the problem is not present, because the class is going to be resovled first, given the tests live in a Handlebars.Test namespace. Out of there, you can't.
I can Resharper out the namespace for you and make a pull request. Would you be interested?
Either this, or I'm a complete noob and I'm completely missing the point. ;)

This question has come up a few times. It was a goof to let it go out without fixing the namespace, but once it went into the wild I decided not to change it since it would be a breaking change for everyone already using it.
It seems like we may be on the cusp of a larger swell in adoption though, so it may be best to eat it now and go ahead and change.

IMO, if it stays as it is, the library is not going to get the traction it deserves: the provided examples can't be used as they are, because if you're not in the Handlebars namespace, there is no way you can reference Handlebars just by name.
It's not good marketing for an otherwise neat library.
Just my 2c.

Pushed 1.3.0 to NuGet


Companion / HTTP Protocol how to

With integrated Companion HTTP Protocol it is possible to work with NOVASTAR H Series Out of the Box.
Setup Novastar:
Username: admin (must be)
Passwort: admin (must be)

Add HTTP Protocol and Add IP Adress: http://192.168.x.10

(load preset 1 in screen 1 example)
2. add Button
3. choose HTTP
4. URL: http://192.168.x.10/api/preset/play
5. body part: {"presetId":0, "deviceId":0, "screenId":0}
6. header part: {"token": "gAN9cQAoWAIAAABpZHEBSwFYAwAAAGtleXECWCAAAAAyMTIzMmYyOTdhNTdhNWE3NDM4OTRhMGU0YTgwMWZjM3EDWAQAAAB0aW1lcQRHQdjhzg9xkRJYCgAAAHBlcm1pc3Npb25xBShLAV1xBl1xB11xCF1xCV1xCnRxC3Uu"}
7. Content Type: Json
(load preset 2 in screen 1 example)
2. add Button
3. choose HTTP
4. URL: http://192.168.x.10/api/preset/play
5. body part: {"presetId":1, "deviceId":0, "screenId":0}
6. header part: {"token": "gAN9cQAoWAIAAABpZHEBSwFYAwAAAGtleXECWCAAAAAyMTIzMmYyOTdhNTdhNWE3NDM4OTRhMGU0YTgwMWZjM3EDWAQAAAB0aW1lcQRHQdjhzg9xkRJYCgAAAHBlcm1pc3Npb25xBShLAV1xBl1xB11xCF1xCV1xCnRxC3Uu"}
7. Content Type: Json
(Take Screen 1 example)
2. add Button
3. choose HTTP
4. URL: http://192.168.x.10/api/layer/screenLayerTake
5. body part:{"screenId": 0, "deviceId": 0, "enNonTime": 1, "manualPlay": 1}
6. header part: {"token": "gAN9cQAoWAIAAABpZHEBSwFYAwAAAGtleXECWCAAAAAyMTIzMmYyOTdhNTdhNWE3NDM4OTRhMGU0YTgwMWZjM3EDWAQAAAB0aW1lcQRHQdjhzg9xkRJYCgAAAHBlcm1pc3Npb25xBShLAV1xBl1xB11xCF1xCV1xCnRxC3Uu"}
7. Content Type: Json

for who need CURL Function
run in Terminal or script:
(load preset 1 in screen 1 example)
curl -v
-X POST
-H "token:  gAN9cQAoWAIAAABpZHEBSwFYAwAAAGtleXECWCAAAAAyMTIzMmYyOTdhNTdhNWE3NDM4OTRhMGU0YTgwMWZjM3EDWAQAAAB0aW1lcQRHQdjhzg9xkRJYCgAAAHBlcm1pc3Npb25xBShLAV1xBl1xB11xCF1xCV1xCnRxC3Uu "
-H "Content-Type: application/json"
-H "User-Agent: HTTPBot-iOS/2021.1"
-d "{"presetId":0,
"deviceId":0,
"screenId":0}"
"http://<IP_ADDRESS>/api/preset/play"
(load preset 2 in screen 1 example)
curl -v
-X POST
-H "token:  gAN9cQAoWAIAAABpZHEBSwFYAwAAAGtleXECWCAAAAAyMTIzMmYyOTdhNTdhNWE3NDM4OTRhMGU0YTgwMWZjM3EDWAQAAAB0aW1lcQRHQdjhzg9xkRJYCgAAAHBlcm1pc3Npb25xBShLAV1xBl1xB11xCF1xCV1xCnRxC3Uu "
-H "Content-Type: application/json"
-H "User-Agent: HTTPBot-iOS/2021.1"
-d "{"presetId":1,
"deviceId":0,
"screenId":0}"
"http://<IP_ADDRESS>/api/preset/play"

i tried and the result is:
{
"data": {},
"msg": "User_Not_Login",
"status": 106
}
how can i fixed, pls
i have admin, and pass but i don't know how to put them in use

please help me or do you have APIs document for this

I can’t help you with this as we have to sign NDA with Novastar to get this document for API calls.
If you need it you must reach out to Novastar sign an NDA which prohibits me from sharing it or posting on public forum. Sorry.

On Jun 5, 2023, at 4:28 AM, huudan_jr @.***> wrote:
please help me or do you have APIs document for this
—
Reply to this email directly, view it on GitHub https://github.com/bitfocus/companion-module-novastar-h-series/issues/4#issuecomment-1576345789, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC5RXGGWL6ACR2MPMVH65HTXJWKBLANCNFSM6AAAAAASPRRAYM.
You are receiving this because you are subscribed to this thread.


Document the flag heirarchy

This all makes sense to me, and it seems like we have some consensus in related discussions. The open question about how to implement variant identifiers in various vendors that don't explicitly carry such a concept seems like an implementation concern that can be relatively easily dealt with by vendors in their providers and associated doc (hashing or mapping of some other attribute associated with a flag value).


feat: add iframes and viewport previewing

👉 Demo 👈
Description
This PR:

adds a fully drag-and-drop compatible iframe to Puck.Preview, enabling style isolation and media queries
adds viewport switching for default small, medium and large
adds zooming to the viewport switching
introduces a new API, viewports, for customising the default viewports
exposes viewport control via the UI state and dispatcher (with exception of zoom)

To support this, we needed to implement two additional libraries:

measured/dnd: A fork of hello-pangea/dnd (an actively maintained fork of react-beautiful-dnd) that supports both iframes and CSS transforms.
measured/auto-frame-component: A new library to sync styles from the host to the iframe

https://github.com/measuredco/puck/assets/985961/93a76dd5-2a48-4200-872d-e4eb57ad269a
Testing
Play around with the viewports, ensuring drag-and-drop still works as expected.

Demo
Docs

Notes

Whilst viewports and drag-and-drop work on mobile, managing a large viewport (such as desktop) can cause iOS Safari to crash due to high GPU usage.
We could consider removing animations, or introducing an API to disable animations.
We should disable transition when prefers-reduced-motion is set.

Future tasks

Add API for iframe style isolation
Add API and UI for adding a select menu of viewports (rather than just icon buttons)

Testing

Unlike the current implementation, the preview does not scroll when dragging a component up or down from the current preview scroll position. So can only drop on an area of the preview that is already scrolled into view. Tested in FF & Chrome. If the place you want to drop is not visible, you need to put the dragged component down, then the scroll the window, then drag the component again. Assume this is a limitation of iFrame approach? Noting as does impact usability. Potentially I just need to get used to it - you can get around it with a repeated drag>drop>scroll>drag>drop>scroll manoeuvre. Could also zoom in to show more of the preview I guess. But other than that, I can't discern a difference in dnd bevaiour from the current version 🎉
Stats component default state breaks layout in narrow viewport. Could potentially add some media query CSS to fix this now.


BUG: fix u8 and i8 dtype names

I encountered the following error when trying to call the create_array method to generate a uint8 array.
Traceback (most recent call last):
  File "../zarr_implementations/generate_data/generate_zr3.py", line 19, in <module>
    a = h.create_array('/raw', shape=im.shape, chunk_shape=CHUNKS, dtype=im.dtype,
  File "/media/lee8rx/data/Dropbox/Dropbox/Quansight/public/zarr/zarrita/zarrita.py", line 273, in create_array
    dtype = _check_dtype(dtype)
  File "/media/lee8rx/data/Dropbox/Dropbox/Quansight/public/zarr/zarrita/zarrita.py", line 147, in _check_dtype
    assert dtype.str in {
AssertionError

It seems u8 and i8 need to have | prepended to match NumPy. Based on the zarr v3 spec, the value stored in data_type should indeed have the | stripped away, though.

Thanks @grlee77. Looks like the CI is failing for an unrelated reason, I'll merge here and follow that up separately.


Use different UE filter for sections on calculator pages

added  back the emi and the eligiblity calculator blocks to component-model.json and component-definition.json
defined a separate filter defintion for sections that only allows to add calculator types
extended editor-support.js to swap the section filter on pages that have template set to calculator.

Fix #none
Test URLs:

Before:

Home page  see filter when adding adding blocks to sections:
<EMAIL_ADDRESS>Calculator page is using the same filter setting:
<EMAIL_ADDRESS>

After:

Home page  still has the same filter for adding blocks to sections:
https://author-p133703-e1305981.adobeaemcloud.com/ui#/@piramal/aem/universal-editor/canvas/author-p133703-e1305981.adobeaemcloud.com/content/piramalfinance-eds2/index.html?ref=add-template
Calculator page only allows to add calculators to sections:
https://author-p133703-e1305981.adobeaemcloud.com/ui#/@piramal/aem/universal-editor/canvas/author-p133703-e1305981.adobeaemcloud.com/content/piramalfinance-eds2/fragments/calculators/home-loan-emi-calculator.html?ref=add-template

closing will create a new one based on latest discussions


where can I download 'electricity.csv'

Hi Zhihan,
Thanks for the very useful repository. Could you point me to the place where I can download the 'electricity.csv'. From the link in the README, Electricity dataset , I can only get a file named LD2011_2014.txt.zip. Not sure how to convert it to the 'electricity.csv'.

We resample the data into the total consumption (sum) of each hour. The data can be preprocessed like this:
import pandas as pd
data_ecl = pd.read_csv('LD2011_2014.txt', parse_dates=True, sep=';', decimal=',', index_col=0)
data_ecl = data_ecl.resample('1h', closed='right').sum()
data_ecl = data_ecl.loc[:, data_ecl.cumsum(axis=0).iloc[8920] != 0]  # filter out instances with missing values
data_ecl.index = data_ecl.index.rename('date')
data_ecl = data_ecl['2012':]
data_ecl.to_csv('electricity.csv')

awesome! thank you.


BigFontName option for Notes (probably other dialogs too) doesn't work?

Can't seem to make it work. Try replacing Arial with Courier New to see it easiest.

Works here, tried on macOS and Windows. The BigFontName settings are read and written at REAPER launch/exit though, so it can only be modified while REAPER is closed.