add examples of byoc, private link and multi az

@arendej @bardielle  Please review.

@datianshi can we close that PR?

/retest


fix: added the junit plugin and added the codecoverage reporting capa…

Added back the two fixes asked, regarding unit tests and the codecoverage.

Adding it here so that this is not forgotten.
The next steps of this work would be to prepare the ground for:
1 - Have a code coverage report (mvn jacoco:report)
2 - Evaluate of this action will somehow help us: https://github.com/marketplace/actions/jacoco-report
3 - Evaluate this one also: https://github.com/marketplace/actions/jacoco-badge-generator


k_busy_wait function hangs on when CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE is set with CONFIG_PM.

Describe the bug
k_busy_wait() function doesn't return when  CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE and CONFIG_PM are enabled.
For blinky example when above configuration options are enabled the LED is turned on once and doesn't change it's state.
Adding of few logs revealed that program hangs on the k_busy_wait() function and no code is executed after call to this function.
Hardware is: STM32WB55 NUCLEO board.
To Reproduce

Open samples/basic/blinky example main.cpp and use k_busy_wait() instead of k_sleep().
Add CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE=y and CONFIG_PM=y to prj.conf.
Build and run application.

Expected behavior
When enabled CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE and CONFIG_PM  have no impact on k_busy_wait() functionality.
Impact
Unable to use k_busy_wait() when CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE  enabled.
Logs and console output
Environment (please complete the following information):

OS: Linux
Toolchain 0.15.0
[27ec69eca18]

Additional context
It may be that the problem is caused by that the timer which is used to count ticks is not re-enabled after application returns from idle. Then ticks are not incremented and k_busy_wait() never returns from this for loop:
for (;;) {
		uint32_t current_cycles = k_cycle_get_32();

		/* this handles the rollover on an unsigned 32-bit value */
		if ((current_cycles - start_cycles) >= cycles_to_wait) {
			break;
		}
	}

In the stm32_lptim_timer.c, when setting a timeout of MAX_WAIT, the lptim is stopped, but depending on the CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE, it should either be stopped (true) or should count a very long period.

@FRASTM IMO your solution will make no difference for k_busy_wait behavior because described case is for enabled CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE.
For me it looks that problem is more generic and it is not strictly related to STM32. This is because there is not defined any logical connection (in generic code) between code using k_cycle_get_32 (like k_busy_wait) and the timer used in the implementation of k_cycle_get_32.
Either k_busy_wait should be not available when CONFIG_SYSTEM_CLOCK_SLOPPY_IDLE is enabled or there should be added some code to automatically start timer used in k_cycle_get_32.

Hi,
sorry for late answer.
@FRASTM i've tried your proposal but this doesn't help.

Would it be Ok to just add small disclaimer for k_busy_wait documentation?

Tiny PR with documentation fix added.


[BUG] Ecto dashboard is using incorrect value field for pool_size and timeout panels

Describe the bug
In the Ecto dashboard JSON, it's using /^Value$/ for the value fields. (here and here). This will render No Data in the generated dashboard the there's no such field with the Value string.
To Reproduce
Steps to reproduce the behavior:

Enable the Ecto plugin.
Start my sample application
Check the Ecto Grafana dashboard

Expected behavior
It should use the correct field name, in this case the name of the metric.

Your fix has been merged and is available in 1.10.0! Thanks :)


Return type inconsistency between jQuery 1.7.2.b1 and 1.7.1

Please take notice of this jQuery bug ticket concerning return types while using the Actual plugin:
http://bugs.jquery.com/ticket/11271
Maybe you know an answer right away.
Best of Luck! And thanks for the nice plugin.

Hey maxpower9000 I think I just missed this comment. I'll take a look at what's going on today :)

From the 1.7.2b1 [change log] #10782.
It fixed an incorrect calculating width bug but also change the .width() behavior.
Thats what makes this plugin broken.
I tested my demo with i1.7.2b1 and it returns the right value.
Therefore I think this plugin is not necessary anymore.

Fixed in commet 0530ce5c6447d4aa9ea0712e32f1f02f660ac61c. Try the lastest version v1.0.6. I also added some info on readme says that it is only needed if you have $ under 1.7.2.


4.3.0a_b23

Changes:

Correct proj pinning to use cbc value.

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:18: http_url: http://archive.ubuntu.com/ubuntu/pool/universe/s/spatialite/spatialite_4.3.0a.orig.tar.gz is not https
ERROR: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:39: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:40: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:41: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:42: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:75: missing_description: The recipe is missing a description
WARNING: /tmp/abs_9d1v70lhdx/clone/recipe/meta.yaml:76: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:18: http_url: http://archive.ubuntu.com/ubuntu/pool/universe/s/spatialite/spatialite_4.3.0a.orig.tar.gz is not https
ERROR: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:39: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:40: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:41: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:42: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:75: missing_description: The recipe is missing a description
WARNING: /tmp/abs_92y0n_9ipm/clone/recipe/meta.yaml:76: missing_description: The recipe is missing a description
Errors were found


add rustfmt support

implement #1901

It looks like cargo fmt errors are getting captured somewhere without warning the user.
With this code:
fn main() {

Running dx fmt --rustfmt gives no output and does not format the code
Running cargo fmt gives this warning:
 --> /../testing/src/main.rs:1:12
  |
1 | fn main() {
  |           -^
  |           |
  |           unclosed delimiter

@ealmloff
I've got some problems about dx fmt

In the gif above , dx was built from crates.io, dx-git was built from github
also, I try to print out the files that dx fmt found, I found it empty.

replace gitignore with ignore crate
rewrite autoformat_project with rayon
use prettyplease as formatter

fixed #1906


⚠ Bofrost.de has degraded performance

In 399fc89, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1312 ms

Resolved: Bofrost.de performance has improved in 89a6909 after 16 minutes.


[microTVM]Fix TVMC test on hardware

This PR adds stack size to run test on hardware for Zephyr. It is not added for Arduino since it is not required.
4096 is derived from experiment on nucleo_l4r5zi.

@tvm-bot rerun

@tvm-bot rerun

Hi all. I think it's important to echo @gromero's comment regarding the Commit Message Guideline. For next PRs, at merging time, please take a moment to adjust the commit message to be more self-explanatory and leave a good track of the changes in our git history.
As I'm involved in the ongoing release process, it complicates a lot the process to be able to track changes that have generic names or no commit message body.


Custom Data Annotation String Localizer / Extend asp-validation-for

Hello,
I hope this is the right place, I followed samples and comments posted here and I was able to quickly write a proof of concept on a custom FileStringLocalizerFactory and FileStringLocalizer and register them in services at startup and DI working great.
So far so good... The problem I am facing is that tag-helpers like asp-validation-for use data annotations info on the model which kind of works if you use resource based localization. I would like to use my own.
I see how regular localization providers (controllers and views) try to be modular and plugable, and it works, but I cannot see a way to plug my own Data Annotation Localization provider.
I other words, how can I make tag-helpers like asp-validation-for take data from a custom annotation provider I would write (file based).
Here is the code in the model
    public class RegisterViewModel
    {
        [Required(ErrorMessageResourceType = typeof(My.Resources.RegisterViewModel),
            ErrorMessageResourceName = "EMAIL_REQUIRED")]
        [EmailAddress(ErrorMessageResourceType = typeof(My.Resources.RegisterViewModel),
            ErrorMessageResourceName = "EMAIL_INVALID")]
        [Display(Name = "Email")]
        public string Email { get; set; }

        [Required(ErrorMessageResourceType = typeof(My.Resources.RegisterViewModel),
            ErrorMessageResourceName = "PASSWORD_REQUIRED")]
        [StringLength(maximumLength: 100, MinimumLength = 6,
            ErrorMessageResourceType = typeof(My.Resources.RegisterViewModel),
            ErrorMessageResourceName = "PASSWORD_MIN_LENGTH")]
        [DataType(DataType.Password)]
        [Display(Name = "Password")]
        public string Password { get; set; }

        [DataType(DataType.Password)]
        [Display(Name = "Confirm Password")]
        [Compare("Password",
            ErrorMessageResourceType = typeof(My.Resources.RegisterViewModel),
            ErrorMessageResourceName = "PASSWORD_NO_MATCH")]
        public string ConfirmPassword { get; set; }
    }

So when validation fails, I would like to pull the string from my provider/file instead of a resource file.
Either that or extend asp-validation-for so it can use another provider. That would work for me too.
TIA,
David

I think this a duplicate to an old issue #31

I am sorry for dup... to tell you the truth trying to follow the thread is rather diffcult since info is somewhat broken down... ;)
Either way, I hope it can be resolved.
After reading issue #31, adding another parameter to the tag help passing the provider instance injected into the view, maybe is not a bad idea.

If the you have such suggestion in the tag helper, i think you should post it in the MVC repo

I don't know enough about the insides of the architecture to make a call on what is the right way... I mentioned here a couple a couple of ideas since we are talking about localization. If indeed people think it is an ok thing to do, (which I dont know), then I can try to write something better in the MVC repo.
I do apologize if I dont do things completely right, I am very new to all this.
I appreciate your patience.

This is being covered in the aspnet/MVC#3518


Support the Namespace as Service

Is this a BUG REPORT or FEATURE REQUEST?:
FEATURE REQUEST
What happened:
It seems that the ClusterRoleBinding is mandatory? SystemAccount is not able to get CRDs if the RoleBinding is only in NameSpace Scope.
Is there anything that is blocking it to be a Namespace as Service ?
ERROR: logging before flag.Parse: E0904 03:33:21.348628       1 reflector.go:205] github.com/kubeless/kubeless/pkg/controller/function_controller.go:139:
Failed to list *v1beta1.Function: functions.kubeless.io is forbidden:
User "system:serviceaccount:solver-dev:kubelss-admin-v0" cannot list resource "functions" in API group "kubeless.io" at the cluster scope

What you expected to happen:
SystemAccount is able to get CRDs when only the RoleBinding with ClusterRole is defined.
How to reproduce it (as minimally and precisely as possible):
---
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: kubelss-admin-v0
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: custom-role
subjects:
- kind: ServiceAccount
  name: kubelss-admin-v0
  namespace: solver-dev
---
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: kubelss-admin-v0-crd
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: admin-role
subjects:
- kind: ServiceAccount
  name: kubelss-admin-v0
  namespace: solver-dev

Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
v1.17.0
Kubeless version (use kubeless version):
v1.0.7
Cloud provider or physical cluster:
Internal

The default behavior for Kubeless is to be able to deploy functions in the whole cluster. If you want to scope it to a specific namespace, you can do so changing the configuration: https://kubeless.io/docs/function-controller-configuration/#install-kubeless-in-different-namespace

Thanks~ It turns out that the functions-namespace should also be specified in the ConfigMap.
It works now.


Remove .eot font files

Woff and Woff2 are well supported by the browsers we care about now.

Addressed in: https://github.com/icelab/berg-assets/commit/d760785310179e16314204309423bba3832fbd67

✔


Visage Version Updated to 4.0.0

Updated

Version of Visage in package.json is updated to v4.0.0

@rk132 feel free to take it over and release it.


feat(gripper): update new gripper eeprom chip

Adds the new DR variant of the chip and use that for the rev c version of the gripper board

Codecov Report

Merging #603 (2953578) into main (18ce0ee) will decrease coverage by 0.12%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##             main     #603      +/-   ##
==========================================
- Coverage   83.43%   83.32%   -0.12%
==========================================
  Files          84       84
  Lines        3013     3017       +4
==========================================
  Hits         2514     2514
- Misses        499      503       +4

Impacted Files
Coverage Δ

include/eeprom/core/hardware_iface.hpp
80.95% <20.00%> (-8.53%)
:arrow_down:


Issue with flushing with bloomfilter v1.5.0b

It seems from version 1.4.0-1.5.0 there is a bug with the bloomfilter and writing and reading from pager I believe.  It's hard to say currently.
For the test below
func TestMemtableFlush(t *testing.T) {
	dir := setup(t)
	defer teardown(dir)

	k4, err := Open(dir, 2764/2, 60, false, false)
	if err != nil {
		t.Fatalf("Failed to open K4: %v", err)
	}

	for i := 0; i < 100; i++ {
		key := []byte("key" + fmt.Sprintf("%d", i))
		value := []byte("value" + fmt.Sprintf("%d", i))

		err = k4.Put(key, value, nil)
		if err != nil {
			k4.Close()
			t.Fatalf("Failed to put key-value: %v", err)
		}
	}

	k4.Close()

	k4, err = Open(dir, 1024*1024, 2, false, false)
	if err != nil {
		t.Fatalf("Failed to reopen K4: %v", err)
	}
	defer k4.Close()

	// get all keys
	for i := 0; i < 100; i++ {
		key := []byte("key" + fmt.Sprintf("%d", i))
		value := []byte("value" + fmt.Sprintf("%d", i))

		got, err := k4.Get(key)
		if err != nil {
			t.Fatalf("Failed to get key: %v", err)
		}

		if !bytes.Equal(got, value) {
			t.Fatalf("Expected value %s, got %s", value, got)
		}
	}
}

=== RUN   TestMemtableFlush
    k4_test.go:172: Expected value value0, got
--- FAIL: TestMemtableFlush (0.06s)

We are failing on specifically
if !bf.Check(key) {
    return nil, nil
}

Within the function func (sstable *SSTable) get(key []byte) ([]byte, error)
This was working fine over here:
https://github.com/guycipher/k4/tree/a852e946d6c32ab5911f6dc96bf15fa6945aba93
Doing code review to find this bug, it's an odd one.  Feel free to assist, an extra pair of eyes and brain would be awesome!

I've reviewed majority of the flushing logic, we are creating the bloom filter, writing keys to the bloom filter, serializing and writing it to sstable, nothing changed there.  The compression logic was introduced not to long ago but I did not add anything that would truly change the original logic just adding to it.

Still investigating.  Narrowed down to this commit
https://github.com/guycipher/k4/commit/32ff5db6ab8b3fd81ef13dce6810e13a145c2813#diff-ca5e6b75d800b93cb0f5df5bbd62d9de4695638e22aa3114df827c372b080ae7

Found it.  It's the
bytes.Compare(value, []byte(TOMBSTONE_VALUE)) == 0  Woops, I must be a bit tired.  Fixing up.


feat(resource): add import to resource_subaccount_role

Purpose

Add import functionality for resource subaccount_role
Closes #89

Does this introduce a breaking change?
[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?
[ ] Bugfix
[X] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code

Execute the documented import of the resource

What to Check
Verify that the following are valid

The state of the remote resource is imported into a tfstate file

Other Information

Sonar Cloud check fails, currently we neglected import functionality for composed keys.


test: increase timeout

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Payment - Adding payment method for first time shows a console warning

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open App
Go to settings
Click on payment methods.
Click on Add Payment Method.

Expected Result:
No Console Warning Should Be Displayed
Actual Result:
Console Warning Is Displayed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.28-0
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/93399543/1aa888be-b6e0-4e2d-be4f-99c9643b64bd
Expensify/Expensify Issue URL:
Issue reported by: @syedsaroshfarrukhdot
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686778963750789
View all open jobs on GitHub

Duplicate https://github.com/Expensify/App/issues/20688, but it closed there.
Proposal
Please re-state the problem that we are trying to solve in this issue.
Console error on add payment method popup in payments
Warning: Failed prop type: The prop `anchorPosition.horizontal` is marked as required in `PopoverWithMeasuredContent`, but its value is `undefined`.
    at PopoverWithMeasuredContent

What is the root cause of that problem?
After click on add payment method, we show menu list to select the new payment method here.
This menu is based on PopoverWithMeasuredContent which require props to specific the position that the menu need it.
https://github.com/Expensify/App/blob/cb522f3a60b5aec645d2824297e0c38168895504/src/components/PopoverWithMeasuredContent.js#L19-L22
We use lib/getClickedTargetLocation to get the click position on web only and return default values in native because the popover will be bottom mounted.
https://github.com/Expensify/App/blob/1af8b3bda2fe75ab1bc939071e5dd80d7b548ec5/src/libs/getClickedTargetLocation/index.native.js#L1-L11
the issue is the values above doesn't have the x and y values, so it returns as undefined on native mobile.
https://github.com/Expensify/App/blob/cb522f3a60b5aec645d2824297e0c38168895504/src/pages/settings/Payments/PaymentsPage/BasePaymentsPage.js#L159-L160
What changes do you think we should make in order to solve the problem?
edit getClickedTargetLocation to return the same object in the web.
{
  "x": 0,
  "y": 0,
  "width": 0,
  "height": 0,
  "top": 0,
  "right": 0,
  "bottom": 0,
  "left": 0
}

or only add x and y.
What alternative solutions did you explore? (Optional)

Agree that this is only a bug on dev and not user facing so we don't need to fix. Closing


Intercepting XMLHttpRequests is broken

Intercept Version : 4.4.0
WDIO Version: 8.32.3
I do not fully understand why it is failing, but when intercepting XMLHttpRequests, they will be duplicated inside the session storage.
Each intercepted request will create additional ones inside the array in session storage. Means, the 3rd request will create 3 entries, the 4th will create 4 entries ... and so on ... All have same URL, method etc ...
Looking at the stacktrace inside XMLHttpRequest.prototype.send shows that it somehow got replaced multiple times. Each send will call an additional send.
The only thing I found to fix this is to check at the beginning of replaceXHR (https://github.com/webdriverio-community/wdio-intercept-service/blob/b2df722aa9a138cab4408513582470089e9550ba/lib/interceptor.js#L118) if XMLHttpRequest.prototype.send is already overwritten:
if (XMLHttpRequest.prototype.send.toString().includes("[native code]")==false) {
    return;
}

But still, I am a bit clueless here if that would be the fix here. @tehhowch  Would you mind to share some wisdom here?
I will try to prepare something reproducable ...

A better check would be to have setupInterceptor only do the replaces if its own flag has not been set - it's possible for page code to do its own polyfills / middleware.
The issue you are experiencing is not due to the library itself - otherwise our unit tests with multiple requests would be failing.

Thanks for answering. I pushed an example here: https://github.com/muhserks/wdio-intercept-service/tree/reproduce-776
In my WDIO framework, I am only waiting  that a request was completed after a click. I call always setupInterceptor and waitUntil browser.hasPendingRequests() is false. But when 2nd request is done, browser.hasPendingRequests() is always true.
The weird thing is, this works with wdio intercept V4.2.0.
Anyway, Let me see if I can figure out a way to avoid the replacement in setupInterceptor.

Why do you repeatedly call setupInterceptor, when the interceptor is already loaded?

Well, simplest example is clearing all the requests. Referring to https://github.com/webdriverio-community/wdio-intercept-service/issues/249#issuecomment-1066201741 it was the suggested way and it makes sense.
But my case is that in the application, same page, when you select certain things, ajax requests are made for each click/select/whatever. If I don't wait for it to be finished, subsequent actions will fail. There is no DOM selection possibility to do it in any other way.
Bottom line is, I have this in my framework (setupInterceptor and waitUntil browser.hasPendingRequests() is false) for a couple of methods which are getting called throughout the tests. I do not set up the interceptor for each test seperately, it is just setup when it is used (and hopefully disabled afterwards when I can finally update ....).
Putting the interceptor stuff directly into the tests is no option. Keeping track on my side if it is already setup or not is also not ideal imo.

If you navigate through your application and change pages, you need to setup the interceptor again and I do not want to do that for each link I may click. I only execute browser.url once to get to the login screen. It is also not the case that I always need the interceptor on that page with ajax requests.
Even though I implemented disable interceptor (shame on me), enabling it is missing. Calling setupInterceptor to enable it again will not work as we already figured out (at least if you stay on the same page).
Clearing the requests is not only window.__webdriverajax.requests = [];. The session store needs also to be cleared. Otherwise you will get weird results and you could also hit the limit again.
I did an implementation now on my side via browser execute to take care of all 3 cases.
For me, these changes do make sense and it could also avoid surprises for other people using the interceptor. But that is just my opinion.
If you are cool with these changes, I can create a PR and we could continue from there. It is up to you.
With that being said, I am still a bit confused about window.__webdriverajax.requests VS the requests in sessionstorage and how they are used. getRequests() will use sessionstorage (if available), but hasPendingRequests will not rely on session storage. window.__webdriverajax.requests can differ from sessionstorage after a page change, but gets cleared while calling the needed setup again. It will also differ if URLs are excluded, because URLs are only excluded while pushing into session storage
So why is the sessionstorage even needed? I am probably missing here something, but wouldn't it be sufficient to just use the namespace?
Oh, and thanks for the feedback.


Create a contact info block

Add links to social networks (Linkedin, Facebook, Instagram), phone number, email, Telegram username, etc.

Added contact info block


Control Bar Out Of Bounds On Small Screen

PR Fixes: #754

1 Decreased the width of controls on small screen
2 Removed Divider and Duration From the Control Bar

Resolves #754
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

Screenshots or GIFs
Issue : -

After Fix : -

thank you sir


Discoverability in binary

The category name shows up in the binary

Which can tip people off to look for something along the lines of: ("My password")

Yeah, the constant strings are stored in some place of the __cstring section and there is no way to get the generated string from there (because we assemble it at runtime). But we still compile the code that combines them toghether, and the selectors (which are actually called as the strings) are stored in the __text section. Which is simple to disassemble and as we see from the picture to understand the unobfuscated string.

"Simple" is a relative term. The average cracker is not going to be looking at assembly code. If you need stronger encryption for a stronger type of hacker, then a stronger library is probably more fitting for your app.
The only way I see around this, using the same API we currently use, would be to use some sort of cipher to generate and map the public api calls to private method names, the ones that show up in the binary. But I think thats overkill for this tiny extension.

+1 @sferrini do you think it's like that in release builds as well?

Ok, apparently it would; I see people suggesting to use #define to substitute method names to some scramble, e.g. something like this
#define A Y191789e
#define B f16852da
...etc

I may be wrong, but reviewing this again, when you strip debug symbols and linked products from your production build it should remove these completely.

Ok, to play around with that I've created empty xcode app called ObfuscationTestApp and added a sample class:
@interface MyTestClass : NSObject
- (void) myTestMethod1;
- (void) myTestMethod2;
@end

After archiving for app store deployment, exporting .ipa on disk and disassembling with hopper you can clearly see this:

Function names are perfectly discoverable and the password is almost plaintext as aforementioned by @tparry
After prepending this class with #defines like this the situation is improved - you can no more find any references to MyTestClass.
#define MyTestClass obj_ffxxoodd
#define myTestMethod1 sub_fbdd83
#define myTestMethod2 sub_cdeb22

@interface MyTestClass : NSObject
- (void) myTestMethod1;
- (void) myTestMethod2;
@end

Another interesting thing is method implementation; instead of using @"I" as literal string I've did the following implementation using char code:
- (void) myTestMethod2
{
    unichar ch = 0x0049;
    NSString *myStr = [NSString stringWithCharacters:&ch length:1];
    NSLog(@"%@", myStr);
}

...which translates to the following code in hopper. String literals visible no more!

I believe combining these 2 technics would significantly obfuscate discoverability of passwords in binary.

I don't think this is necessary. Did you enable "Strip Debug Symbols During Copy" and "Strip Linked Products" on your release build?

I've used the default settings. I'd vote for something that works with no manual build configurations.

That didn't really answer my question.  Unless you can confirm that debug symbols are not stripped out using the above settings, I have to close this issue. Stripping debug symbols (the method names) from your production builds is anti-cracking 101

@coneybeare "Strip Debug Symbols During Copy" will compromise your crash logs. You will no longer be able to read the stack trace, nor other useful informations.

@sferrini @coneybeare agree on stack trace readability; something that concerns me as well regarding stripping debug symbols.
Interestingly enough with this build settings method names still perfectly discoverable by hopper:

Ok, another interesting thing I've just realised: the app binary is encrypted (when installed from AppStore).
So after pulling one of my apps from iTunes backup folder on the osx, and looking at them with otool -ov and class-dump here's what I see:

Though to read plaintext password entered in source codes would require to decrypt app binary. Not as simple as using strings or hopper, that's good news.
If we had a way to decrypt own app downloaded from App Store (maybe with provision or something) we could easily see how the binary looks there. Anybody knows a way of doing that without a jailbreak?

Keeping a copy of your DSYM is trivial, and a copy of it is even uploaded to Apple when you submit to the app store for them to be able to symbolicate your crashes. That is not a valid argument for not enabling the setting. What's unsettling is that hopper is still able to figure out some of the method names from the stripped product.  Perhaps filing a bug with the Xcode/LLVM team is the best course of action here.

@coneybeare you'll have to keep dsyms for each build of each app you submit; and apply correct dsym when you get crashlogs based on app and build# (if you can symbolicate? don't know anything about that). That looks like an argument for me.
class-dump is able to get names as well:

The best way to see how method names look in the app binary IMO would be to decrypt and look own app downloaded from App Store; then you can know for sure what apple distributes to users' devices.
Anybody has any ideas on this point?

It's trivial. Setup a build script to stash the DSYM on every production build. The use any number of the symbolization helpers out there to use them. Google around for symbolicate and dsym and you will find it is quite easy and effective.

The more interesting question still viewing the "end-result" so to say, how the binary looks on user's iphone.

Not for me. The strip debug symbols is supposed to be doing this to prevent apps from discovering method names. If that is not working as advertised when enabled, then a bug need to be filed with Apple and the LLVM team so they can fix the tool to do what it says it is. Without doing a deeper dive, my best guess would be that hopper is locating the DSYM file to do it, or they have fully cracked that feature making it obsolete until Apple fixes.
The other possibility is that you still scanning a debug version of your app by mistake, one that includes the debug symbols.
Either way, UAOfuscatedString was never meant to be a full encryption library or bullet-proof tool. There are much stronger ways to encrypt your strings, and if your app warrants a type of protection that UAObfuscatedString can't provide, then I would suggest something like RNCryptor or rolling your own. This library was only meant to prevent the string from showing up as a whole.

Ok, I've laid out things that was on my mind for this topic here: https://github.com/truemetal/iOS-apps-security-notes
Anybody who'd like to proceed the investigation / discussion is welcome on that repo!
Thanks

So just to clarify, no changes have been made to this library to address the vulnerability that was found by @tparry right?

No, @tparrys issues were caused by not enabling the standard "Strip Debug Symbols During Copy" and "Strip Linked Products" compiler flags on release products. When used properly, all symbols are obfuscated as intended by UAObfuscatedString.


feat: enable scaffolding for host builds

:gift: creates the scaffolding package
:gift: OCI builder updated to use scaffolding package and thus function source
:bug: Fixes a bug where local jobs were not receiving context cancellation
:bug: Fixes a bug where signatures of instanced HTTP was throwing errors
:broom: adds a concurrency test which ensures only one build executes per
process

Connects the scaffolder to function source code by factoring out scaffolding into a package which is used both by the client and by other packages, allowing for the removal of the mocks.
/kind enhancement
- Fixes a bug where local jobs were sometimes not canceling immediately
- Startup timeout for local run tasks now configurable

Thanks for the review @lance.  The update to the error message you noticed has been made, as well as several other updates to improve error handling and test coverage (PR description updated)
- configurable platforms for build tasks
- additional typed errors and reorganization
- expanded test cases to check typed errors are used
- configurable global and per-run startup timeout
- tests run readiness endpoint is considered and respects timeouts
- adds a concurrency test which ensures only one build executes per
  process
- passes the --platform flag through as a build option

I have it build locally and tested OK. The code looks like good starting point with tests.
I have also tested that existing functionality is still working oK
 ../func-scaffolding-enable/func deploy --builder=pack
   🙌 Function built: docker.io/aslom/wt1:latest
   ⬆  Deploying function to the cluster
W0606 12:36:00.466532   68896 warnings.go:70] Kubernetes default value is insecure, Knative may default this to secure in a future release: spec.template.spec.containers[0].securityContext.allowPrivilegeEscalation, spec.template.spec.containers[0].securityContext.capabi
   ✅ Function updated in namespace "default" and exposed at URL:
   http://wt1.default.<IP_ADDRESS>.sslip.io

Looking forward to see more functionality.
LGTM

/lgtm 🥂

/lgtm

/unhold


[1.10.2] Crash when pulling energy from turbine

I've had no problems with Mekanism before now.
I have created an industrial turbine, then I placed down a basic universal cable attached to an energy storage block (can't recall the name). When I tried to set the cable to "pull", my game crashed. I now can not load my world.
Not sure what reports you need, hopefully these will suffice.
crashreport.txt
gameoutput.txt

It looks like the turbine doesn't implement a certain interface. What mod is an "industrial turbine" from?

Mekanism v9?

Heh, sorry, I'm used to older versions. From looking through the source, I can't find a problem directly besides that maybe the turbine doesn't implement a few interfaces, but that seems unlikely. Good luck :)

Fixed in development.


Java intelli Sense not working after last Update

When I click ctrl + space then suggation not coming. After restart its working few times.

/extJava


trait DeserializeJson

If there was
trait FromJSON {
   fn from_json_slice(s: &[u8]) -> Result<Self> {
      serde_json::from_slice(s)
   }
   fn from_str(s: &str) -> Result<Self> { .... }
   ...
}

impl<T> FromJSON for T where T : serde::Deserialize { }

Then we could
fn foo(data: &[u8]) -> Result {

   let t = MyType::from_json_slice(data);
   ...

}

instead of let t : MyType = serde_json::from_slice(data)?;.

I think I would prefer not to build this into serde_json; but it would be reasonable to make a separate crate providing that.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_cbk21spvpm/clone/recipe/meta.yaml:24: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_cbk21spvpm/clone/recipe/meta.yaml:30: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_cbk21spvpm/clone/recipe/meta.yaml:34: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, closest match: BSD-3-Clause
WARNING: /tmp/abs_cbk21spvpm/clone/recipe/meta.yaml:38: missing_description: The recipe is missing a description
ERROR: /tmp/abs_cbk21spvpm/clone/recipe/meta.yaml:38: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Draw figure's icon using an icon font instead of code

See #522
As of now, figure's icon is drawn using custom code. Another option would be to use an icon font. This would have several advantages:

There are several free/open icon fonts available, making it easy to extend the set of icons to allow people to use non default notation. This opens the door for language customization and stereotyping
There are good and free tools online (like [icomoon)[https://icomoon.io/app/#/select]) that provide an easy way to import SVG and convert it to a font. This is an easy path to add new icons when needed.
There is now some fonts that use two colors or two shades. This might be helpful (if supported) to have even beautiful icons ;-)

Hi JB,
This would simplify a lot my specialization plugin if Archi could natively replace figure's icons by custom ones :-)
Best regards
Hervé

This would simplify a lot my specialization plugin if Archi could natively replace figure's icons by custom ones :-)

Yes. In fact my long term goal is to add native support for language customization in Archi. So this is a first step.

Did you have a chance to have a look to my alpha version of my specialization plugin v2 ?
Unluckily, I lack time for this plugin but the direction I took is to define specializations at the model level, a specialization being composed of a an image, graphical configuration (color, line size, ...), and a set of properties.
Then Archimate concept by Archimate concept, add the ability to assign a specialization which automatically sets the image, the graphical configuration and the properties.
If this is close to what you've got in mind and if Archi implements this natively, then my specialization plugins will not be required anymore :-)

Yes, that's what I have in mind.

Great :-)

The thing with setting icons and images is making sure they are bundled with the *.archimate file.

The thing with setting icons and images is making sure they are bundled with the *.archimate file.

Not fully true. If we provide an icon font (containing standard ArchiMate icons and a good set of other icons) with Archi, then we don't have to bundle icons with the .archimate file. Of course, if we want to support real images, then they have to be bundles. But in this case, we can already do it (we have images in canvas and Hervé's plugin uses this trick to store custom images).

Yes, that's what I have in mind.

And BTW, don't expect it to be implemented anytime soon. There is a lot of discussions and design decisions involved prior any code.

FYI: Both Font Awsome and Nerd Fonts (https://www.nerdfonts.com/) may be useful references / starting points for custom fonts.


Feature request: Expanded support for using resource throttling

We use resources to throttle tasks that interact with an API that's easily overwhelmed, but we'd like to set up targets for our tasks that check in with this API as well. Unfortunately, it doesn't look like there's a way to throttle requests to an API made by targets (or in the complete() method) using resources. We could use a semaphore, but it doesn't look like there's a way to share a semaphore across multiple workers.
Would it be possible to provide a way to use resources within a task like a semaphore coordinated by the scheduler, so we could do something like this?
def complete(self):
    with self.resources(slow_api=1):
        return super().complete()

Or, alternatively, Luigi could provide a way to configure the resources a task should require to schedule, separately from configuring the resources required to run.
class ApiScheduledTask(luigi.Task):
    complete_check_resources = {'slow_api': 1}

I'm going to pursue handling this with a home-brewed solution using the posix_ipc library, but it would be much more valuable to be able to do this in a way that uses the same pool of resources as workers.


bugfix: modify volume usage

1.Describe what this PR did
Modify volume usage.
2.Does this pull request fix one issue?

fixes #128
fixes #129
3.Describe how you did it
4.Describe how to verify it
5.Special notes for reviews
Signed-off-by: Rudy Zhang<EMAIL_ADDRESS>

LGTM


Enhancement: for usability I think the arrow from the menu should animate

Hi,
Please open this -> http://quemfazsite.com.br/temp/aaa/modelo12/
Hover your mouse in the menu links at the top right. You can see that some links have an arrow right after them. The arrow is pointing down, meaning that if the user hovers this link it will display a menu below it.
After the menu is showing up you can see that the arrow changes it's orientation and now points upwards. So the user can know that if he gets out of the menu it will contract/slideUp.
If you check the MENU element and DROPDOWN element from SEMANTIC UI, they both show a nice arrow right after the parent element of the menu. HOWEVER after the user hovers the mouse in those parents the arrow keeps pointing down, there is no move.
I see many many websites that change the arrow orientation for usability reasons. Some websites even animate the arrow rotating it 180 degree so it points down/up with a nice animation.
I am not asking for animation, but if at least semantic ui could change the arrow orientation it would great, superb! It is a good step torwards usability and I think this is a good think cause, from my beginner perspective, semantic ui has a huge focus on usability! That's why I droped jquery ui, bootstrap... and started using exclusively semantic ui!

hope someone apply this enhancement :)

@batata004 Can you provide a jsfiddle because your link is broken


Fixing arrays

I originally opened issue #62 based on the work I was doing to try and port the LegacyEffects shader in Indigo. I've tested this branch against it and the problem has been resolved.


[ci] Drop check for deprecated tags

This PR cleans up after #18150. Now that we've removed the deprecated "vivado" tag, there's no need to keep this check around.

Merging without complete CI approval because this literally just deletes a CI step :)


Cypress install script updates

Updated to the 2.6.0 version of Cypress as the default installed version.
Also updated some of the text output to the terminal, to format it better/hide text that shouldn't be displayed.

Rebased and merged


Jenkins Pipeline Development

This project will be used to run my jobs in CI/CD pipeline.

This project will be used to run my jobs in CI/CD pipeline.


CUDA 9.1 and Tensorflow GPU 1.5.0 Visual Studio 2017 and Python 3.6 error

I installed CUDA 9.1, cuDNN 7.31 for CUDA 9.1 and I have installed Visual Studio 2017. However when I run the CUDA samples I get these following outputs:

But when I try to fit my model using tensorflow-gpu it show "python.exe" stopped working and then when I click Debug, I get this output inside my Visual Studio 2017:

Any idea what might fix this issue?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

TensorFlow expects CUDA 9.0. Please install version 9.0 and change the [path var].(https://www.tensorflow.org/install/gpu#windows_setup).

Anyway I fixed this issue through this process of installation: https://www.pugetsystems.com/labs/hpc/Install-TensorFlow-with-GPU-Support-on-Windows-10-without-a-full-CUDA-install-1172/


Error: Missing service linked role for ECS

Default configuration install.
Error occurred because missing service linked role for ECS.
Problem was resolved by running command:
aws iam create-service-linked-role --aws-service-name ecs.amazonaws.com --profile
Maybe this should be included in the pre-requisites?

yep. it's at the troubleshooting section of the readme.md.
but you're right. it might be good to mention it earlier in the docs.


stat can't work in osx

Executing uutils stat -f tmpfs and it should crash then since there's no /etc/mtab.

original stat -f on mac does something different to linux:

mac:
-f format
Display information using the specified format.  See the FORMATS section for a
description of valid formats.
linux:
-f, --file-system
display file system status instead of file status

so i think this is a case where we can't replicate behaviour on mac.
Having said that I ran: uutils stat -f / and uutils stat -f /dev  on my mac and it provided me with block information that looked vaguely sensible.


Implement WidgetExt on custom widgets

Is your feature request related to a problem? Please describe.
I cannot use my custom widget as a template parameter to a generic struct that requires a WidgetExt.
use fltk::button::Button;
use fltk::prelude::*;
use fltk::widget_extends;

#[derive(Default)]
struct MyCustomButton {
    button: Button,
}

widget_extends!(MyCustomButton, Button, button);

struct WidgetWrapper<W: WidgetExt> {
    wrapped_widget: W,
}

fn wrap<W: WidgetExt>(widget: W) -> WidgetWrapper<W> {
    WidgetWrapper {
        wrapped_widget: widget,
    }
}

fn main() {
    wrap(Button::default());
    wrap(MyCustomButton::default()); // this does not work
}

Describe the solution you'd like
widget_extends could implement WidgetExt and (optionally GroupExt) on the custom widget, delegating method calls to the base.

I plan to expose more trait macros to the user. The currently exposed macros handle widgets which have an underlying widget pointer. I plan to add macros similar to wodget_extends which basically implement the trait macro via the passed widget.
Adding that functionality to widget_extends is difficult, since finding out what traits a widget implements is difficult.

I'm running into issues creating such a macro for WidgetExt, since it requires changing the closure signature in set_callback to accomodate for the custom widget, and this breaks the trait. Another problem with macros is that they don't accommodate generic types.
For the time being, you can still benefit from the Deref impl:
use fltk::button::Button;
use fltk::prelude::*;
use fltk::widget_extends;

#[derive(Default)]
struct MyCustomButton {
    button: Button,
}

widget_extends!(MyCustomButton, Button, button);

struct WidgetWrapper<'a, W: WidgetExt> {
    wrapped_widget: &'a W,
}

fn wrap<'a, W: WidgetExt>(widget: &'a W) -> WidgetWrapper<'a, W> {
    WidgetWrapper {
        wrapped_widget: widget,
    }
}

fn main() {
    wrap(&Button::default());
    wrap(&*MyCustomButton::default());
}

Thanks, but I intend to implement additionnal methods on WidgetWrapper<MyCustomButton>...
We could require that the custom widget implements the Clone trait, like all the other widgets. In that case set_callback could be implemented:
impl WidgetExt for $widget
where $widget: Clone
{
    // ...
    fn set_callback<F: FnMut(&mut Self) + 'static>(&mut self, cb: F)
         where Self: Sized
    {
        let mut widget = self.clone();
        self.$member.set_callback(move |_| {
            cb(&mut widget);
        });
    }
    // ...
}

I've used your suggestion, and have added several trait macros (for WidgetExt, GroupExt, and WidgetBase). The widget base macro probably doesn't work as expected. You can try things out:
use fltk::button::Button;
use fltk::prelude::*;
use fltk::enums::{self, *};
use fltk::app;
use fltk::group;
use fltk::macros::widget::*;

#[derive(Clone, Default)]
struct MyCustomButton {
    button: Button,
}

impl_widget_ext_via!(MyCustomButton, button);
impl_widget_base_via!(MyCustomButton, Button, button);

struct WidgetWrapper<W: WidgetExt> {
    wrapped_widget: W,
}

fn wrap<W: WidgetExt>(widget: W) -> WidgetWrapper<W> {
    WidgetWrapper {
        wrapped_widget: widget,
    }
}

fn main() {
    wrap(Button::default());
    wrap(MyCustomButton::default());
}

The definition of impl_widget_ext_via refers to fltk modules as crate::<mod> which causes the compiler to search for the definitions in my crate when the macro is expanded.
macros/widget.rs:1103
macros/widget.rs:1268
macros/widget.rs:1366
macros/widget.rs:1454
macros/group.rs:306
The macro also uses fltk_sys::widget::Fl_Widget which will require the user to add a dependency to fltk_sys.
The fltk crate would need to expose fltk_sys::widget::Fl_Widget.
macros/widget.rs:1219
macros/group.rs:1536

The changes are in master and haven’t been published yet.

Do you have a date planned for publishing the changes?

Around 5 days. It’s usually a week between releases.


Model Architecture

Where can I find the model architecture of this image captioning model. ?

Hi @navydhara79-zz ,
For checking the model architecture details one can use model.summary() and tf.keras.utils.plot_model(). It will give the information about each layers with input and output sizes and which layer they are connected to etc.
For example I have taken Retinanet model and using the above mentioned APIs to check the architecture details and it was plotted. You can refer to attached gist.
Thanks!


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4cmrjgevvp/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_4cmrjgevvp/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_4cmrjgevvp/clone/recipe/meta.yaml:54: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_4cmrjgevvp/clone/recipe/meta.yaml:54: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_4cmrjgevvp/clone/recipe/meta.yaml:54: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


fixed css in mobile view #2184

Related Issue
#2184
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #2184
Description
removed
after
https://github.com/anuragverma108/SwapReads/assets/142989448/1fc6987f-a5c5-4a9e-9af0-66cf137bbc5a
before
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@huamanraj please also added level badge

@huamanraj please also added level badge

Important Note:
This repository serves as a gateway for newcomers to explore open-source projects. To ensure fairness in learning, we're introducing a cap on the points awarded to individual contributors. Going forward, each contributor will be restricted to earning a maximum of 150 points. and there is no level for some too basic changes!!. This limitation specifically applies to the GSSOC project.
Thank you for your understanding and cooperation in maintaining an inclusive learning environment.


Update app-governance-whats-new.md

@batamig

Pulled into https://github.com/MicrosoftDocs/CloudAppSecurityDocs-pr/pull/3591

#label:"aq-pr-triaged"


Backport of regexp fixes and SHA improvements to 0.8.x

I've backported 00bb02978e33417d72ddf573a61ee7c10b35d9aa and 9f5a83a6b2861c256861375e81859d5cbfdfca70 to address security issue OSVDB-110439.
Should e88afeceb036fe4d44f7c7787c7e988e1350c2dc also be backported, or is this PR sufficient?
Also, I pinned capybara to a version that supports Ruby 1.9.2.

sorry for the delay (I've been away) - why is this necessary, rather than upgrading the application to a newer version of dragonfly?

closing as no activity for a while - in any case the recommended fix was to upgrade - this wouldn't change that as the user would still need to upgrade to get these fixes.


Add test coverage for transactions in EthereumRPC

Base those tests on https://github.com/ethers-io/ethers.js/blob/main/src.ts/_tests/test-transaction.ts

Once this is fixed https://github.com/chainsafe/ethereumrpcsharp/issues/13 I will jump into this


fix: makes default token url universe aware

Makes the default sts token url logic in external account credentials universe aware.

Coverage test is complaining about new getUniverseDomain, where the 2nd branch isn't exercisable by tests.  We can ignore for this PR.


[WIP] Activate admin password security feature check in core

Activate admin password security feature check in ODB core module.

Codecov Report

Merging #9488 (db384fd) into develop (c5888ee) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #9488      +/-   ##
=============================================
- Coverage      54.23%   54.19%   -0.04%
+ Complexity     35928    35898      -30
=============================================
  Files           2470     2468       -2
  Lines         194288   194239      -49
  Branches       35295    35287       -8
=============================================
- Hits          105367   105268      -99
- Misses         73814    73874      +60
+ Partials       15107    15097      -10

Impacted Files
Coverage Δ
Complexity Δ

...nologies/orient/core/db/OrientDBConfigBuilder.java
90.32% <ø> (ø)
12.00 <0.00> (ø)

.../collection/closabledictionary/OClosableEntry.java
64.40% <0.00%> (-11.87%)
19.00% <0.00%> (-3.00%)

...echnologies/orient/core/type/ODocumentWrapper.java
44.73% <0.00%> (-10.53%)
9.00% <0.00%> (-2.00%)

...e/cache/local/doublewritelog/DoubleWriteLogGL.java
37.91% <0.00%> (-7.50%)
14.00% <0.00%> (-5.00%)

...com/orientechnologies/orient/core/db/OrientDB.java
89.47% <0.00%> (-5.27%)
20.00% <0.00%> (-2.00%)

...ient/core/storage/disk/OLocalPaginatedStorage.java
53.74% <0.00%> (-5.17%)
47.00% <0.00%> (-3.00%)

...inated/atomicoperations/AtomicOperationsTable.java
53.60% <0.00%> (-4.80%)
20.00% <0.00%> (-4.00%)

...ientechnologies/orient/core/db/OSharedContext.java
80.64% <0.00%> (-4.36%)
18.00% <0.00%> (+3.00%)
:arrow_down:

...ogies/common/concur/collection/CASObjectArray.java
75.00% <0.00%> (-3.58%)
10.00% <0.00%> (-1.00%)

...e/impl/local/paginated/wal/OLogSequenceNumber.java
50.00% <0.00%> (-3.58%)
10.00% <0.00%> (-1.00%)

... and 66 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #9488 (db384fd) into develop (c5888ee) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #9488      +/-   ##
=============================================
- Coverage      54.23%   54.19%   -0.04%
+ Complexity     35928    35898      -30
=============================================
  Files           2470     2468       -2
  Lines         194288   194239      -49
  Branches       35295    35287       -8
=============================================
- Hits          105367   105268      -99
- Misses         73814    73874      +60
+ Partials       15107    15097      -10

Impacted Files
Coverage Δ
Complexity Δ

...nologies/orient/core/db/OrientDBConfigBuilder.java
90.32% <ø> (ø)
12.00 <0.00> (ø)

.../collection/closabledictionary/OClosableEntry.java
64.40% <0.00%> (-11.87%)
19.00% <0.00%> (-3.00%)

...echnologies/orient/core/type/ODocumentWrapper.java
44.73% <0.00%> (-10.53%)
9.00% <0.00%> (-2.00%)

...e/cache/local/doublewritelog/DoubleWriteLogGL.java
37.91% <0.00%> (-7.50%)
14.00% <0.00%> (-5.00%)

...com/orientechnologies/orient/core/db/OrientDB.java
89.47% <0.00%> (-5.27%)
20.00% <0.00%> (-2.00%)

...ient/core/storage/disk/OLocalPaginatedStorage.java
53.74% <0.00%> (-5.17%)
47.00% <0.00%> (-3.00%)

...inated/atomicoperations/AtomicOperationsTable.java
53.60% <0.00%> (-4.80%)
20.00% <0.00%> (-4.00%)

...ientechnologies/orient/core/db/OSharedContext.java
80.64% <0.00%> (-4.36%)
18.00% <0.00%> (+3.00%)
:arrow_down:

...ogies/common/concur/collection/CASObjectArray.java
75.00% <0.00%> (-3.58%)
10.00% <0.00%> (-1.00%)

...e/impl/local/paginated/wal/OLogSequenceNumber.java
50.00% <0.00%> (-3.58%)
10.00% <0.00%> (-1.00%)

... and 66 more

Continue to review full report at Codecov.

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


Strip trailing slashes to avoid 404 from Home Assistant's API endpoint

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Strip trailing slashes to avoid 404 from Home Assistant's API endpoint.
Type of change
Please delete any options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[ ] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)
Please do not use any external image service. Instead, just paste in or drag and drop the image here, and it will be uploaded automatically.

Hey @lassebm,
just out of interest. How did you define your Home Assistant check? Mine is configured this way and no fix via a PR was needed. Do you have a better check I might want to adapt?

@ThomDietrich If you had a trailing slash in the URL before the PR, HA would 404 due to double slashes in the constructed URL.

ah ^^ okay that makes sense. Thanks for your PR :)


Twitch chat websocket disconnects with status of WebSocketCloseStatus = InternalServerError

I have been noticing that after a period of time (seems to be around 30 minutes or so) the Twitch chat client tosses a disconnect event with a status of InternalServerError.   I haven't been able to identify a root cause for the issue yet.

I would need to know the details of the actual error, that's not enough information to go off of.


Update 2019-05-21T13:55:54.904Z

I am just a bit unsure but I am creating pull request anyways...

 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.:x: Scapenighter sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @Scapenighter, thanks for your contribution but I'm not sure what you're trying to achieve here.  This seems like it may have been forked from the gh-pages branch which is an autogenerated Web site, and it appears to delete most of the files that make up the extension.  Could you clarify what you're trying to fix or improve, and we can maybe help you rework your pull request?  Thanks!

I noticed that the PR as sent is actually merging the gh-pages branch in the main Azure repo into the master branch.  This definitely isn't going to do what you want - the gh-pages branch contains a generated Web site and is always derived from master, not vice versa; plus, the Azure repo won't contain any of your changes.  What you need to do is this:

fork the repo into your own GitHub account
take a branch off master
make changes on that branch
push the branch with changes to your fork
create a pull request from the pushed branch in your fork

Hope this makes sense - happy to provide more detail on any of these steps if that would help.

Sorry about that .I had few to many things going on on my computer and mixed a few things up in the process...Right now I am decluttering my computer before I do any more work... hopefully my next pull request make more sense, not just for me, but for you too...I am a little new at this and still have a lot to learn...Thank you for taking the time to point out the right direction, I really appreciate it...👍😁😎
Sent from my iPhone
On Feb 9, 2020, at 3:02 PM, itowlson<EMAIL_ADDRESS>wrote:

I noticed that the PR as sent is actually merging the gh-pages branch in the main Azure repo into the master branch. This definitely isn't going to do what you want - the gh-pages branch contains a generated Web site and is always derived from master, not vice versa; plus, the Azure repo won't contain any of your changes. What you need to do is this:

fork the repo into your own GitHub account
take a branch off master
make changes on that branch
push the branch with changes to your fork
create a pull request from the pushed branch in your fork

Hope this makes sense - happy to provide more detail on any of these steps if that would help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/vscode-kubernetes-tools/pull/694?email_source=notifications&email_token=ACXK2HYLGCEIOXQ2WP4JUFLRCCDO7A5CNFSM4KRW7ZW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELG24CA#issuecomment-583904776, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACXK2H2PKXRRNFN5OIDSAM3RCCDO7ANCNFSM4KRW7ZWQ.

@Scapenighter Since you alluded to sending stuff in your next PR rather than amending this one (which I'm not sure is even possible), I'm closing this one if that's okay. If you do want to keep this one going, please feel free to reopen; otherwise, looking forward to your contribution and thanks in advance!


Fix implicit animationg bug

Details
This fixes an issue that could cause the calendar to layout with an odd animation. Recently, we changed the way layoutSubviews works - we check if we're currently in an animation closure, and if we are, we layout a larger region than what's currently visible so that offscreen items animate in / away correctly.
This behavior change only makes sense if the reason we're in layoutSubviews is because of a call to setContent (which will do some animation preparation). If we end up in layoutSubviews in an animation closure without setContent getting called first, then we didn't do any animation preparation in setContent, and the new layoutSubviews behavior doesn't make sense anymore.
The solution is to set a flag in setContent to indicate that the next call to layoutSubviews should use the new behavior for animated updates, rather than checking if we're in an animation closure (which could happen if a random part of code calls layoutIfNeeded in an animation closure without also calling setContent first).
Related Issue
N/A
Motivation and Context
Found a bug in the Airbnb app
How Has This Been Tested
Example app, Airbnb app
Types of changes

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

Checklist

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

@brynbodayle I fixed another (related) issue with pinned days of the week. I was drawing them at weird positions due to the whole 3x visible bounds thing that I added. The 2 new commits fix that + add unit tests for it.


Reduce openzipkin links to just home page and allow a user-configurab…

…le support link.
Discussion in #2375
After (with supportUrl defined)

After (without supportUrl defined)

Before

maybe before and after image to make it more clear?

Should we use English "Help" or "Support" ? I tend to think of the word support, and whatever we put in here lives in config, so figured worth a short bikeshed.
As far as I can tell in the code, by default we don't add a default now. Maybe a screenshot of default (help/support defined) can ... help :)
@jcarres-mdsol does this make sense for configuring your slack url?
@Monkeyanator thoughts on the reduced UI clutter (for lack of better word)?

Was hoping to leave the other screenshot to imagination ;) But added it.
Have no opinion at all on help or support - if no one else votes on it I'll rename to support.

I like the without visually as it balances the white icons top and bottom. Nice!

+1

Renamed from help -> support

Good job team. thanks for driving this home @anuraaga!


release v2.5.0

updated with Pierre's 2.5.0 release. Will merge this afternoon unless there are any comments


What is this for?

Curiosity killed the cat xD

Hi Mr. Schrödinger,
This is a custom GitHub action to build and publish to a registry (e.g. https://hub.docker.com) Docker images.
It's still under development and will probably be ready by the end of week.
I'm currently testing it's integration in https://github.com/manusa/mnIMAPSync. More specifically the boundaries of the action's expression syntax to see if more features will be needed (you can check the progress here: https://github.com/manusa/mnIMAPSync/blob/feature/%2330-dockerize/.github/workflows/build.yml#L26).
Any inputs are welcome ;)

Hello Mr. Manusa,
Thank you for the introduction.
Nice, I will keep an eye on it.
In your repos, are you building more stuff besides the Docker image, or it's just a step in your build process?

It's usually one of the latest steps on the build pipeline, and should only be triggered when pushing a Git tag (release version) or push to master (a.k.a. latest) when every previous step completed successfully .
I usually include the following steps in most of my projects (build, test, quality + quality gate, release->Docker, finally depending on the project deploy).

So effectivelly you are replacing your current CI/CD pipeline with GitHub actions then?

Look into Azure Pipelines as well.

So effectivelly you are replacing your current CI/CD pipeline with GitHub actions then?

For all new projects I'm probably going to use GitHub actions/workflows, I find them much more modular than other solutions I've tried and I feel very comfortable creating new actions as they can be implemented using any technology and be easily implemented.
Only feature I miss so far is a global dashboard for all my projects, so I probably end up building one myself.

Look into Azure Pipelines as well.
I sure will

"so I probably end up building one myself"
I wouldn't expect any less from you :-)

The only problem would be vendor lock in, right?


[Feature] Also check for Polkadot Address in User Profile

People in the Polkadot ecosystem constantly contributing might also have their address included in their profile.
For example like me:

This, for example, is the pattern needed for all Fellowship members to get their github verified.
It would be good that if we trigger a tip, that we can also check this user profile for the address, in case the user does not include it directly into their github pull request body.
This would be a fallback to the PR body check.

@shawntabrizi we already have that, but parsing it a bit differently - similar to a PR body
https://github.com/paritytech/substrate-tip-bot/blob/master/src/util.ts#L65
polkadot address: <ADDRESS>
Do you suggest to use this address regardless of prefix or something else?

haha nice, sorry i didnt know.
thank you for thinking ahead for me :)


Update price not include decimals

Now price multiplier with its decimals is unnecessary. We'll update when have time.

price = price * quoteDecimals
amount = amount * baseDecimals

We update Base token's balance by amount.
We update Quote token's balance by the formula: (price * amount) / baseDecimals


Module/include cells

I would like to have a stacking module system in each window, where modules are added to the tab container as new tabs which can be edited individually. This will be post IPyCells 0.1.1 because I'll need a split ends function.
Anyway this will just make it easier to work with modules and included files in olive.it will be pretty easy I just make a new Project{:module} type, populate it when the cell is ran. The module cell contains cells of its own then I write a string function that turns it into a normal module.
The hardest part of this will be getting cells out of the reader. Will need to find all of the modules and then sort the cells that belong into those modules properly into the modules.

Include cells have a partial implementation, will soon be added along with module cells :) better project structure and organization, etc.

closed by #94 ! :')


Add feature-flags on config and create collector mode on insight collector

What this PR does / why we need it:

create CollectorMode for insight collector.( #1583 )
Add feature-flags on config and load CollectorMode from config

Which issue(s) this PR fixes:
Fixes #1582
Does this PR introduce a user-facing change?:

NONE

@nghialv
Thanks for reviews, I checked your reviews and fixed the points

@sanposhiho Thank you. Great work!
Let me merge this PR and fix some commented nits myself.
/approve


Fix link to documentation

Can one of the admins verify this patch?

Thanks!  Good catch, @nzachow


Creating a Export Review VM fails with "EntityDoesNotExist"

Describe the bug
When trying to create a export review VM it thoughs the error "Error creating review resource"
When looking in the Logs it shows this:
Failed to retrieve Airlock Review configuration for workspace <workspace ID for Airlock review workspace>
Traceback (most recent call last):
  File "/api/api/routes/airlock.py", line 136, in create_review_user_resource
    updated_resource, operation = await create_review_vm(airlock_request, user, workspace, user_resource_repo, workspace_service_repo, operation_repo, airlock_request_repo, resource_template_repo, resource_history_repo)
  File "/api/services/airlock.py", line 198, in create_review_vm
    user_resource, operation = await _deploy_vm(airlock_request, user, workspace, review_workspace_id, review_workspace_service_id, user_resource_template_name, user_resource_repo, workspace_service_repo, operation_repo, resource_template_repo, resource_history_repo)
  File "/api/services/airlock.py", line 220, in _deploy_vm
    workspace_service = await workspace_service_repo.get_workspace_service_by_id(workspace_id=review_workspace_id, service_id=review_workspace_service_id)
  File "/api/db/repositories/workspace_services.py", line 55, in get_workspace_service_by_id
    raise EntityDoesNotExist
db.errors.EntityDoesNotExist

Azure TRE release version (e.g. v0.14.0 or main):
v0.16.0
Deployed Azure TRE components - click the (i) in the UI:
UI: 0.5.17
API: 0.16.9
Cosmos DB: OK
Service Bus: OK
Resource Processor: OK

Hi @james-annages,
It sounds like the Export Workspace Service ID as described here: https://microsoft.github.io/AzureTRE/latest/tre-admins/configure-airlock-review/#configuring-airlock-vm-for-research-workspace
Cannot be found. Typically this is the ID of the Apache Guacamole Service in the workspace you are doing the export from.
Let me know if this helps.

Hello, @marrobi
thanks for the suggestion,
However, they are all the same.
Config from the Airlock settings in the workspace

The settings in the DB for that workspace

Settings from the Workspace services for the Apache Guacamole services that I want to use for exports (they are all in the same work space.

That is odd. The query that is failing is:
SELECT * FROM c WHERE c.resourceType = "{ResourceType.WorkspaceService}" AND c.workspaceId = "workspace_id}"' AND c.id = "{service_id}"

(from   File "/api/db/repositories/workspace_services.py", line 55, in get_workspace_service_by_id)
In the Resources collection. You could always try the query in CosmosDB wit the values substituted in.
Is the request definitely being created from the workspace starting 6b...? My only other guess is is there some hidden characters somehow introduced via copy/paste...

Sorry just seen there is some incorect information in the bug report,
the call in coming from the workspace that has submitted the airlock request.

Hello, sorry for the confution.
ID of the Airlock requeting enviroment: bc0c7799
ID of the workspace where the guacamole service is located: 6b5a6866
ID of the guacamole service for the export: 9596936c
when running the query from the ID of the where the Airlock request was submitted:

I get no results
when running the query from the ID of the where the Airlock review workspace is located:

i also go no results.

It's c.id not c.service_id, see my example above.

re tried the query with the requested information, when we try and use the ID for the workspace that submitted the airlock request we don't get any responses.
when I submit the query with the ID for the airlock review workspace

Ok, for EXPORT the Guacamole service needs to be in the workspace that you are exporting from. The data must not leave that workspace until reviewed, hence that's where the VM needs to be to review it.
The "airlock IMPORT review" workspace is only for imports.
As per: https://microsoft.github.io/AzureTRE/latest/tre-admins/configure-airlock-review/#configuring-airlock-vm-for-research-workspace
"For Export Review Workspace Service ID, use the GUID of the Workspace Service deployed into the Research Workspace from step 4."
Deploy the Guacamole Workspace Service into the workspace you are exporting from, update the ID, and you should be good.

@james-annages did you get this resolved? Thanks.

Hello it did. Sorry ment to update this.


correlated_with disease list

follow on from
https://github.com/pombase/pombase-chado/issues/853
Could you provide me with the "correlated-with" disease gene list?
It  could be useful to have a filter for this in the query tool

Do you mean "contributes_to"?

Yes!

There are 119 contributes_to annotations.  Is this all the information you need?:
disease_contributes_to.csv

uniquename
name
termid
term_name

SPBC2G2.08
ade9
MONDO:0011120
neural tube defects, folate-sensitive

SPAC56F8.06c
alg10
MONDO:0013367
long QT syndrome 2

SPAC664.14
amt2
MONDO:0859172
hemolytic disease of fetus and newborn, RH-induced

SPAC2E1P3.02c
amt3
MONDO:0859172
hemolytic disease of fetus and newborn, RH-induced

...
...
...
...

It could be useful to have a filter for this in the query tool

I'm working on that now.  It will probably be 3-4 hours work.

Yep!

I'm working on that now. It will probably be 3-4 hours work.

I didn't get very far before we had our call and decided not do implement that.  I changed one line of code.  :-)
But I did establish how much work it would be so if it becomes useful to constrain by qualifier it could be done in half a day.


File "D:\anaconda3\envs\gupiao\lib\site-packages\pandas\core\indexes\base.py", line 6251, in _raise_if_missing     raise KeyError(f"{not_found} not in index") KeyError: "['EPS', 'Revenue', 'ROE', 'P/E'] not in index" Error fetching financial metrics: HTTPSConnectionPool(host='fc.yahoo.com', port=443): Max retries exceeded with url: / (Caused by ProxyError('Unable to connect to proxy', FileNotFoundError(2, 'No such file or directory')))

This is the error that occurred when running fetch_data.py. How should it be resolved?

I kindly ask for some time to resolve these issues. I have a remote job that I work on daily, and I have also been attending classes since I was a college student. Additionally, I need to prepare for an upcoming hackathon, and my exams are approaching. On top of that, I must complete my college projects and assignments before the exams begin.
I am currently going through a difficult period in my life, and it's challenging for me to stay motivated. Some days, I find it hard to get out of bed; I often stare at the white ceiling and feel exhausted.
Apologies for the inconvenience caused. To be honest, this project was meant to be a group effort, but everyone abandoned me, and now I have to complete it on my own. I hope you understand.
Thanks for understanding.

Hey,
Thanks for your patience and understanding. I have updated the code. I hope your issues have been resolved.
Harshit Kushwaha


Consider hiding the merged interface

Right now the API for components looks like this:
@Component
@MergeComponent
@SingleInAppScope
abstract class AppComponent(
  @get:Provides val input: Input,
) : AppComponentMerged

We could turn it into the follow and generate the real component ourselves:
@MergeComponent
@SingleInAppScope
abstract class AppComponent(
  @get:Provides val input: Input,
)

That saves a step in the setup and avoids running an initial build to generate AppComponentMerged. The downside is that the IDE doesn't know that AppComponent will implement the contributed components and inheritance in the IDE doesn't work properly. The inheritance chain from:
ContributedComponent -> AppComponentMerged -> AppComponent

To this:
ContributedComponent -> GeneratedAppComponent
                      ^
AppComponent --------/

There is no connection between ContributedComponent and AppComponent.
I personally prefer giving the IDE more power and add the interface manually.
Note that the situation for Anvil itself is different, because it's able to modify source code.

Note that the situation for Anvil itself is different, because it's able to modify source code.

I'm curious: Would a compiler plugin be able to automatically add the Merged component as a base type from which to inherit? I.e., provide the best of both worlds?

That's how anvil's original plugin worked but would require running in FIR to work in K2. KSP will also not run FIR plugins.


feat: transaction list

Overview
This pull request introduces the new transaction list feature within our web3modal interface. Aimed at increasing user engagement and providing transparent financial tracking, this update allows users to easily access and review their recent transactions directly from their web3modal.
Designs
Technical Details

[x] New list item on the account view.
[x] Integration with blockchain APIs to fetch transaction data.
[x] Transaction list view and list all the transactions depending on the response
[x] Pagination to handle large transaction histories efficiently.

Screenshots
https://github.com/WalletConnect/web3modal/assets/19428358/9ca3a597-83e1-4dd1-9fb1-e97b94dfa2e2

I ran into an issue with this flow

Connected with my metamask extension and loaded up the transactions
DIsconnected
Connected using zerion through qrcode
Going to transactions now shows me my old list from metamask / is not loading new data

I ran into an issue with this flow

Connected with my metamask extension and loaded up the transactions
DIsconnected
Connected using zerion through qrcode
Going to transactions now shows me my old list from metamask / is not loading new data

This should be fixed now

@svenvoskamp @0xAsimetriq Seems like we still have the test error in CI. I cannot reproduce this locally. I'm still looking for it.
Error: Failed to resolve entry for package "@web3modal/common". The package may have incorrect main/module/exports specified in its package.json.

I'm planning to use act to debug the issue but since I don't have env variables, it might not be as easy as I think. Did you use it before?

https://github.com/nektos/act


How to create Event in Calendar to a subfolder or specific folder ?

Hi,
Im trying to create an event to a subfolder of my Exchange calendar.
Here is my code to retrieve subfolders' Ids :
 class calendarSearch {

private $socket;
private $obj;

public function __construct($ews) {

$this->socket = $ews;
$this->obj = new EWSType_FindFolderType();
$this->obj->Traversal = EWSType_FolderQueryTraversalType::DEEP;
    $this->obj->FolderShape = new EWSType_FolderResponseShapeType();
    // only retrieve default properties
    $this->obj->FolderShape->BaseShape = EWSType_DefaultShapeNamesType::DEFAULT_PROPERTIES;
    $this->obj->ParentFolderIds = new EWSType_NonEmptyArrayOfBaseFolderIdsType();
    $this->obj->ParentFolderIds->DistinguishedFolderId = new EWSType_DistinguishedFolderIdType();
    // specify that we want subfolders of the main calendar folder
    $this->obj->ParentFolderIds->DistinguishedFolderId->Id = EWSType_DistinguishedFolderIdNameType::CALENDAR;

  }

  public function getList() {

    $response = $this->socket->FindFolder($this->obj);
    if($response->ResponseMessages->FindFolderResponseMessage->RootFolder->TotalItemsInView > 0){
        $folders = $response->ResponseMessages->FindFolderResponseMessage->RootFolder->Folders->CalendarFolder;
    }
    return $folders;
}

}
And here is my code to add a new event to my calendar :
class calendarSearch {

private $socket;
private $obj;

  public function __construct($ews) {

    $this->socket = $ews;

    $this->obj = new EWSType_CreateItemType();

    $this->obj->Items = new EWSType_NonEmptyArrayOfAllItemsType();
    $this->obj->Items->CalendarItem = new EWSType_CalendarItemType();

    // Set the subject.
    $this->obj->Items->CalendarItem->Subject = 'Basic Calendar Item Insertion';

    // Set the start and end times. For Exchange 2007, you need to include the timezone offset.
    // For Exchange 2010, you should set the StartTimeZone and EndTimeZone properties. See below for
    // an example.
    $date = new DateTime('10:00 AM');
    $this->obj->Items->CalendarItem->Start = $date->format('c');
    $date->modify('+1 hour');
    $this->obj->Items->CalendarItem->End = $date->format('c');

    // Set no reminders
    $this->obj->Items->CalendarItem->ReminderIsSet = false;

    // Or use this to specify when reminder is displayed (if this is not set, the default is 15 minutes)
    $this->obj->Items->CalendarItem->ReminderMinutesBeforeStart = 15;

    // this is where the difference lies
    $this->obj->ParentFolderIds = new EWSType_NonEmptyArrayOfBaseFolderIdsType();
    $this->obj->ParentFolderIds->FolderId = new EWSType_FolderIdType();
    // you can just plug in your folderid and changkey here if you aren't looping though all calendars
    $this->obj->ParentFolderIds->FolderId->ChangeKey = "changekey of folder";
    $this->obj->ParentFolderIds->FolderId->Id = "Id of folder";

  }

  public function getList() {
        // Build the body.
        $this->obj->Items->CalendarItem->Body = new EWSType_BodyType();
        $this->obj->Items->CalendarItem->Body->BodyType = EWSType_BodyTypeType::HTML;
        $this->obj->Items->CalendarItem->Body->_ = 'This is <b>the</b> body';

        // Set the item class type (not required).
        $this->obj->Items->CalendarItem->ItemClass = new EWSType_ItemClassType();
        $this->obj->Items->CalendarItem->ItemClass->_ = EWSType_ItemClassType::APPOINTMENT;

        // Set the sensativity of the event (defaults to normal).
        $this->obj->Items->CalendarItem->Sensitivity = new EWSType_SensitivityChoicesType();
        $this->obj->Items->CalendarItem->Sensitivity->_ = EWSType_SensitivityChoicesType::NORMAL;

        // Add some categories to the event.

        //$this->obj->Items->CalendarItem->Categories = new EWSType_ArrayOfStringsType();
        //$this->obj->Items->CalendarItem->Categories->String = array('Testing', 'php-ews');

        // Set the importance of the event.
        $this->obj->Items->CalendarItem->Importance = new EWSType_ImportanceChoicesType();
        $this->obj->Items->CalendarItem->Importance->_ = EWSType_ImportanceChoicesType::NORMAL;

        // Don't send meeting invitations.
        $this->obj->SendMeetingInvitations = EWSType_CalendarItemCreateOrDeleteOperationType::SEND_TO_NONE;

        $this->obj->SavedItemFolderId = new EWSType_TargetFolderIdType();
        $this->obj->SavedItemFolderId->DistinguishedFolderId = new EWSType_DistinguishedFolderIdType();

        $this->obj->SavedItemFolderId->DistinguishedFolderId->Id = "calendar";

        $response = $this->socket->CreateItem($this->obj);

    return $response;
}

I can figure out to create an event into my main calendar, but impossible to set the current calendar to the specified one, in order to create the event in it.
Is there any solution to that ?
Thanks for yours advises !

Would you mind add a copy of the fully working solution? Thanks


Reset user session

ref https://github.com/nebulab/solidus_square/issues/26
When the user lands to the redirectURL specified on configuration, we should show a thank you message and reset his session, in order to avoid that he will make any other changes to the order.
I'm not sure if it would be better to remove the user from the order or if it will be enough to link a new order to the user.

A simple reset_session would do the trick I guess?

@garciajordy I don't think so, because the order can be accessed via API as well. We need to be sure that a user logged in with the API (with the Authorization Header) will see their current order empty after the order is completed on Square.

I spoke with @kennyadsl IRL and we agreed that the best way to handle it would be to create a new empty cart linked to the current user.
So the previous order will stay in the system waiting to be processed by the webhook.
The new order will be the current order of the user.


training speed very slow

The training speed is very slow, did anybody encounter this problem?

And the GPU is in use:

I tried it but it's much slower...

I tried it but it's much slower...

You may try to raise workers_per_gpu.

I tried it but it's much slower...

Have you solved this problem？ I encounter the same issue with the A100 GPU.

The processing of radar point cloud is too time-consuming. You can try the method in Bevstereo. Store the point cloud as an npy file, and then read it.

Thanks for your advice！
Can you give me a demo code?

The processing of radar point cloud is too time-consuming. You can try the method in Bevstereo. Store the point cloud as an npy file, and then read it.

Thanks for your advice！
Can you give me a demo code?

The training speed is very slow, did anybody encounter this problem?
And the GPU is in use:

Have you solved this problem?

Not solved yet, I used the implementation in BEVDet.


fix: use the native new URL for generating the link

Use the new URL for parsing the URL

This will fix #185

URL constructors already present in the latest release


[Feature Need?] FID evaluation Metric module for Super-Resolution

Related to https://github.com/open-mmlab/mmediting/issues/333.
Hello. In these days, some SR studies use FID score for evaluation.
Unlike PSNR/SSIM, this kinds of metric computes the distance between two distributions or image sets (SR outputs and GTs).
Examples

https://arxiv.org/abs/2201.09109
https://arxiv.org/abs/2002.11397

We have our own implementation for this feature, and this can benefit similar metric in the feature.
Is it acceptable to the plan of mmediting?

Yes that would be good if you can help! You can submit a pull request and we can include it into MMEditing.
Thanks for your contribution in advance!


Animations laggy in IOS Facebook/Messenger built-in browser

The issue persists only when our site is opened from IOS Facebook/Messenger app. It is opened in a built-in browser, and for some reason, the animations are sometimes laggy, sometimes way faster and have less resolution, sometimes won't work at all. No clue what the solution could be.
System is the latest IOS (16.2). AOS version is the latest release. Everything works on browsers like Chrome for IOS.

Someone?


Feature request: comments to and from localizers

Established internationalization and localization tools have the ability for authors to leave comments for localizers. For example, if a button says "File," authors can comment as to whether they mean a noun or a verb, and whether they are referring to a folder of papers or the digital equivalent or a tool or process for smoothing objects.
This feature could be hacked in under the existing angular-translate structure by using something like TRANSLATION-ID.COMMENT. However, it might make more sense to build it into angular-translate.
While the established internationalization tools have a place for comments from the original author, why not allow comments from localizers, too. These may give feedback to the author and to other localizers. They might be especially helpful to localizers who are localizing based on a localization one or more localizations removed from the original.
If this feature is already included, please change this to a request for direction to documentation. Thank you for your great work.

@innerpeace0  We have such a idea some time ago, but this would require a (meta) change of the translation tables (better: the loader).
Anyway: You can define such a layout on your own already today; you only have to build a small custom loader filtering the translation texts correctly.

Other than your statement that this would require a meta change, I don't understand much of your reply @knalli, but I do think it would be quite simple to implement it by my idea of using something like TRANSLATION-ID.COMMENT or TRANSLATION-ID.SUB-ID.COMMENT, etc. Authors can easily put them into their translation tables, but they won't put them into the actual app. For the most part, localizers won't need to translate them (except, perhaps, when the original language is something other than Engish), but will see them when they go to localize the strings.
Perhaps your team could agree on some kind of convention so that there will be a consistency. The master plan that I see is being able to automate things so much that localizers will need to know no code in order to localize apps.
I nominate ".COMMENT" (always in English) as the convention for this purpose.

But this is a change, too. Until now this would be valid translation key. After all, that's a breaking change.
We had also the idea of changing the meta structure of translations: For each translation there would be not a string, but an object containing a value (the translation), and comment and something else we don't know yet.
But again (my last sentence): This is logic being part of the loader. And everyone have the control to use a custom, selfmade loader which also means you can transform easily any existing structure into one which is required by angular-translate. In your case, it would skip all keys ending with ._COMMENT_.

As I understand it, angular-translate wouldn't even have to skip the keys in the translation table with ._COMMENT_ if the author didn't put anything with that into the markup itself. They would just sit there in the translation tables and never be seen by the users. When automating the localizations, they would show up right under their parent keys.
I like the ideas of going to an object and providing for comments within the object. I noticed that other users have a need for objects, not just strings. Is there a timetable for implementing this?

They would just sit there in the translation tables

Yep, and allocating space (ideally much more than the actual translation text) and will be never used. I don't like this, regarding memory consumption concerns.

I noticed that other users have a need for objects, not just strings.

No, that's not the same. I've only spoken regarding the format of translations which will be processed.. but not how they are stored internally nor be available.
I'm repeating myself: The loader can make such processes already today. You have to differentiate between translation texts loaded (i.e. by a server/JSON resource) and translation stored and being available internally in angular-translate.

I agree that having a way to filter out the comments so that they don't get loaded where they are not needed is a good idea. That's one reason why I was hoping that you'd come up with some kind of standard convention for them. It sounds like this is already on the drawing boards for future enhancements. Add my vote to it. Thanks.

As I said already.. that standard applies only for the loaders we are adding. But you can build your own loader (which is very tiny! and actually only one function) or your own http response transformer handling a JSON structure of your own choice.
We also would put that kind of logic into the loader or a post processor (i.e. http response transformer)
Regardless at which time and which result we decide doing something related to your question, you can use something you want. Don't wait for us. ;)


Streamfield rendering breaks for blocks without an ID

Wagtail doesn't require a StreamBlock to have an ID until the point that it is saved in the database. This is helpful for things like pre-populating inital content into a streamfield, where you can initialise it with JSON such as:
json.dumps([
    {'type': 'paragraph', 'value': '
This is a paragraph
'}
    {'type': 'paragraph', 'value': '
This is another paragraph
'}
])

Doing so breaks the front-end handling of this field by wagtail-react-streamfield however - it looks like the JS expects an ID, and doesn't handle the case where there isn't one. The following works:
json.dumps([
    {'type': 'paragraph', 'value': '
This is a paragraph
', 'id': '5670367b-d5fe-4d1f-8e20-2e3f585d0864'}
    {'type': 'paragraph', 'value': '
This is another paragraph
', 'id': 'ce18d9bc-c0d6-4862-a3c3-756aa622309c'}
])

It would be good if it could handle the case where there is no block ID (as Wagtail does) for initial content - e.g., by just auto-generating one.

Weird, I’m pretty sure I handled that scenario. I will try to reproduce it.

OK, I identified the issue, it is fixed by c285a111db3233de5aaad4f4e013bdbca087259e.


Remove helper functions when useing Trans

If a certain module uses Trans, two new functions will be added to that module:

with_translations/3
with_translation/5

Those functions are simply wrappers over the functions provided by Trans.QueryBuilder. The wrapper functions simply read the default translation configurations for the module and eliminate the necessity  for repeating over and over again with each Trans.QueryBuilder call.
We could use a different approach though. When using Trans from our module, we could set up some __special__ functions returning the translation configurations. Calls to Trans.QueryBuilder could check the existence of those functions and automatically call them if they exist.

This issue has been more detailed in issue #28 and tackled in PR #31, which is already merged and ready for the release 2.0.0 of Trans.


Replace jasmine with karma / jasmine test suite

This pull request is in reference to #841
I experimented with replacing a headless jasmine test suite with karma's chrome launcher. Now tests can actually load in a browser. This gives access to the web audio api technology and allows wavesurfer to actually load audio files. The wavesurfer api appears to be fully testable, but I was unable to actually render the graphical representation of the waveform. Also, I couldn't get sound working. I don't find these relatively important, though.

I ditched the jasmine istanbul coverage because I wasn't sure what to do with it for now. I need some advice on that front.

Looks like travis is going to fail until I can figure out this error.
https://travis-ci.org/katspaugh/wavesurfer.js/builds/170590640#L320
I'm actually not sure if karma chrome launcher can even work with a CI environment

Merged. Thank you very much, this is an amazing move!


HDDS-11986. Enable SCM Ratis in TestSCMNodeManager

What changes were proposed in this pull request?
As titled, please take a look, thanks!
What is the link to the Apache JIRA
HDDS-11986
How was this patch tested?
It can be tested by CI, if the unit tests are passed or not.

Thanks @chiacyu for the patch.


Add cython to list of devices

Our list of devices includes Python, Cpp and Genn. "Python" should refer to Numpy-Python (prefs.codegen.target = 'numpy'), which always works, and Cython would then refer to Cython-Python (prefs.codegen.target = 'cython'), which can be faster sometimes.

Fixed in 540daa6d05d559a7f85b094dffadade3c87137c3


Integrate the postgresql and mysql databases

The goal is to allow integration with postgresql and mysql through their respective postgresql_client and mysql_client interfaces.

I'm waiting for https://github.com/canonical/flask-k8s-operator/pull/20 before merging this PR (if everything is solved).


Deploying Redpanda to K8s for production

Redpanda configuration from https://docs.redpanda.com/docs/platform/deployment/production-deployment/
Tunning k8s workers - sudo rpk redpanda tune all
Minimum resources CPU, memory and storage

This is a good example of a consumer of this content: https://redpandacommunity.slack.com/archives/C01AJDUT88N/p1669046127254399