Make analysis sets immutable by default

Make analysis sets immutable by default
Make AnalysisHashSet.Clone() thread safe
Add peak memory usage to memory tester output
In my testing so far, this is looking around 2x improvement in analysis time (for e.g. bokeh and conda sources) with minor improvement in memory usage (as opposed to the huge regression I was expecting).
Please test and see if you notice values disappearing from your own code (imported packages are not affected). I'm keen to take this, provided we can't see regressions in results. The tests still pass.

I'm not noticing any obvious regression.

I've run some benchmarks versus standard ImmutableHashSet.
Here is the code:
        [GlobalSetup]
        public void SetupTestData() {
            var rand = new Random();
            _input = new AnalysisValue[1001];
            for (var i = 0; i < _input.Length; i++) {
                var offset = rand.Next();
                _input[i] = new TestAnalysisValue {_name = "Q" + offset, Value = "q" + offset };
            }

            _immutableHashSetAdd = ImmutableHashSet<AnalysisValue>.Empty.WithComparer(UnionComparer.Instances[0])
                .Add(new TestAnalysisValue { _name = "A", Value = "a" })
                .Add(new TestAnalysisValue { _name = "B", Value = "b" })
                .Add(new TestAnalysisValue { _name = "C", Value = "c" });

            _analysisHashSetAdd = (AnalysisHashSet)AnalysisSet.CreateUnion(
                new[] {
                    new TestAnalysisValue { _name = "A", Value = "a" },
                    new TestAnalysisValue { _name = "B", Value = "b" },
                    new TestAnalysisValue { _name = "C", Value = "c" }
                },
                UnionComparer.Instances[0]);

            _immutableHashSetContains = _immutableHashSetAdd.Union(_input);
            _analysisHashSetContains = (AnalysisHashSet)_analysisHashSetAdd.Clone().Union(_input);
        }

        [Benchmark]
        public void ImmutableHashSet_Add() {
            _immutableHashSet = _immutableHashSetAdd;
            foreach (var t in _input) {
                _immutableHashSet = _immutableHashSet.Add(t);
            }
        }

        [Benchmark]
        public void AnalysisHashSet_Add() {
            _analysisHashSet = (AnalysisHashSet)_analysisHashSetAdd.Clone();
            foreach (var t in _input) {
                _analysisHashSet = (AnalysisHashSet)_analysisHashSet.Add(t);
            }
        }

        [Benchmark]
        public void ImmutableHashSet_Contains() {
            foreach (var t in _input) {
                _b = _immutableHashSetContains.Contains(t);
            }
        }

        [Benchmark]
        public void AnalysisHashSet_Contains() {
            foreach (var t in _input) {
                _b = _analysisHashSetContains.Contains(t);
            }
        }

Here are the results:

BenchmarkDotNet=v0.10.13, OS=Windows 10 Redstone 3 [1709, Fall Creators Update] (10.0.16299.309)
Intel Core i7-4810MQ CPU 2.80GHz (Haswell), 1 CPU, 8 logical cores and 4 physical cores
Frequency=2728057 Hz, Resolution=366.5613 ns, Timer=TSC
  [Host]     : .NET Framework 4.7.1 (CLR 4.0.30319.42000), 32bit LegacyJIT-v4.7.2633.0
  DefaultJob : .NET Framework 4.7.1 (CLR 4.0.30319.42000), 32bit LegacyJIT-v4.7.2633.0

Method
Mean
Error
Gen 0
Allocated

ImmutableHashSet_Add
742.03 us
5.5747 us
104.4922
331709 B

AnalysisHashSet_Add
70,181.17 us
277.1101 us
3750.0000
11937407 B

ImmutableHashSet_Contains
100.63 us
0.5204 us
-
0 B

AnalysisHashSet_Contains
63.41 us
0.6005 us
-
0 B

As you can see, AnalysisHashSet is faster than ImmutableHashSet by 60% in searching, but is significantly worse in adding, especially in terms of memory.

I've added a test for enumeration:
        [Benchmark]
        public void ImmutableHashSet_Iterate() {
            foreach (var t in _immutableHashSetContains) {
                _value = t;
            }
        }

        [Benchmark]
        public void AnalysisHashSet_Iterate() {
            foreach (var t in _analysisHashSetContains) {
                _value = t;
            }
        }

Method
Mean
Error
Gen 0
Allocated

ImmutableHashSet_Iterate
204.85 us
0.5041 us
-
0 B

AnalysisHashSet_Iterate
16.26 us
0.0537 us
-
56 B

So iterating ImmutableHashSet is 12 times slower.

Here is the full result, compiled against x64 and including mutable option:

BenchmarkDotNet=v0.10.13, OS=Windows 10 Redstone 3 [1709, Fall Creators Update] (10.0.16299.309)
Intel Core i7-4810MQ CPU 2.80GHz (Haswell), 1 CPU, 8 logical cores and 4 physical cores
Frequency=2728057 Hz, Resolution=366.5613 ns, Timer=TSC
 [Host]     : .NET Framework 4.7.1 (CLR 4.0.30319.42000), 64bit RyuJIT-v4.7.2633.0
 DefaultJob : .NET Framework 4.7.1 (CLR 4.0.30319.42000), 64bit RyuJIT-v4.7.2633.0

Method
Mean
Error
Gen 0
Allocated

ImmutableHashSet_Add
937.85 us
18.4960 us
187.5000
590087 B

AnalysisHashSet_Add
72,876.41 us
1,375.8347 us
7562.5000
23862718 B

AnalysisHashSet_Add_Mutate
167.15 us
0.3410 us
11.2305
35464 B

ImmutableHashSet_Contains
100.54 us
1.1985 us
-
0 B

AnalysisHashSet_Contains
67.58 us
0.2566 us
-
0 B

ImmutableHashSet_Iterate
205.10 us
1.8505 us
-
0 B

AnalysisHashSet_Iterate
16.46 us
0.1114 us
-
56 B

@zooba , I'm not sure how to interpret these results considering your measurements. One possibility is that most of the memory taken by AnalysisHashSet_Add becomes garbage in real life scenario, but that will automatically mean more GC calls. Another possibility is that AnalysisHashSet.Add isn't called that often.

@AlexanderSher I wasn't comparing AnalysisHashSet to ImmutableHashSet, so you've started doing brand new work here.
I was comparing the default .Union(canMutate: true) which allows adds to occur in-place when supported (which is only supported by AnalysisHashSet, and not the special cases for 0, 1 and 2 element sets), and the alternative .Union(canMutate: false) which always creates a new set when adding new elements.
My guess is the difference is at the macro level. By mutating shared instances of the set, values were showing up in other locations and causing analysis to take much longer than it should have. Without that, we complete much more quickly. I would hope that at the individual operation level it is slower with the change, but overall it is much faster.
If I don't hear any concerns with this PR in the next hour or so, I'm going to merge it.

I was comparing the default .Union(canMutate: true) which allows adds to occur in-place when supported (which is only supported by AnalysisHashSet, and not the special cases for 0, 1 and 2 element sets), and the alternative .Union(canMutate: false) which always creates a new set when adding new elements.

How many elements do you add to the Union?


jdk21: java/util/Properties/StoreReproducibilityTest failed on mac1014 passed on mac12

Tests are available on 20+. History shows consistently failed on all mac1014 ci.role.test&&hw.arch.x86&&(sw.os.osx||sw.os.mac)&&xcode10 and passed on mac12
https://trss.adoptium.net/deepHistory?testId=64f6926017052c6715143752
19:03:54   stdout: [Current default timezone is sun.util.calendar.ZoneInfo[id="Europe/Dublin",offset=0,dstSavings=3600000,useDaylight=true,transitions=227,lastRule=java.util.SimpleTimeZone[id=Europe/Dublin,offset=0,dstSavings=3600000,useDaylight=true,startYear=0,startMode=2,startMonth=2,startDay=-1,startDayOfWeek=1,startTime=3600000,startTimeMode=2,endMode=2,endMonth=9,endDay=-1,endDayOfWeek=1,endTime=3600000,endTimeMode=2]]
19:03:54  ];
19:03:54   stderr: []
19:03:54   exitValue = 0
19:03:54
19:03:54  java.lang.RuntimeException: Expected date comment Tue Sep 05 00:03:43 IST 2023 to be after Tue Sep 05 00:03:42 IST 2023 but was Mon Sep 04 19:33:43 IST 2023
19:03:54  	at StoreReproducibilityTest.assertCurrentDate(StoreReproducibilityTest.java:434)
19:03:54  	at StoreReproducibilityTest.testEmptySysPropValue(StoreReproducibilityTest.java:256)
19:03:54  	at StoreReproducibilityTest.main(StoreReproducibilityTest.java:71)
19:03:54  	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
19:03:54  	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
19:03:54  	at com.sun.javatest.regtest.agent.MainActionHelper$AgentVMRunnable.run(MainActionHelper.java:333)
19:03:54  	at java.base/java.lang.Thread.run(Thread.java:1583)
19:03:54
19:03:54  JavaTest Message: Test threw exception: java.lang.RuntimeException
19:03:54  JavaTest Message: shutting down test

https://ci.adoptium.net/view/Test_openjdk/job/Test_openjdk21_hs_sanity.openjdk_x86-64_mac/124/console

The test  test if it can parse the time by the format "EEE MMM dd HH:mm:ss zzz uuuu" . Test failed if the default timezone is "Europe/Dublin". The reason is the Abbr of both  Irish Standard Time and  India Standard Time is same as IST. If the format "zzz" is "IST" it is parsed as  India Standard Time. In our case it is actually is  Irish Standard Time, which fail the test.
@zzambers thoughts? Could you please look at this?

@sophia-guo sorry for late response, I was on PTO last week. I'll take a look.

Looking into JDK sources, some abbreviations of zone names are indeed ambiguous (root locale). In particular IST can be:
Irish Summer Time
Israel Standard Time
India Standard Time

I have found mention of this issue in DateTimeFormatterBuilder  class:
... Many textual zone names are not unique, such as CST can be for both "Central Standard Time" and "China Standard Time". In this situation, the zone id will be determined by the region information from formatter's locale and the standard zone id for that area, ...
Particular locale can give precedence to specific interpretation of abbreviation, to be used, when parsing local date. For example IST in en_IE locale.
However FORMATER in this test has to use ROOT (US) locale (see JDK-8282023; this is because Properties.store makes use of Date.toString) It does not seem like there is an easy solution using FORMATER, which would work for all zones out of box (given time format used by Properties.store).
Possible workarounds would be ether to avoid ambiguous time zone abbreviations by explicitly setting timezone in test. Or skipping test which compares dates.

@sophia-guo Does it already have upstream openjdk bug, or should I create one?

@zzambers I don't think there is upstream openjdk bug for this.

I have created this issue:
https://bugs.openjdk.org/browse/JDK-8316540
I'll later prepare PR with fix.

The bug is was fixed for jdk/jdk by reseting the test to GMT (if i read it correctly).
In case of need it being  backported,  ProcessTools.createTestJvm is not in jdk8 (but is in 11 and up)
If you do nto wish to pull for some reason, I'm sure the failure is fasle negativum annd the machine canbe safely moved to GMT (not physically :) )

The tests are available on 20+. No worries for jdk8.

@judovana will it be backported to jdk21?

@sophia-guo eventually I have only created openjdk bug, someone was faster with proposing fix (creating PR). Not sure whether backporting that fix for jdk21 is planned by it's author, but we can backport it ourselfs, if necessary.

@zzambers the reason I'm asking is we might need to exclude it if it won't be backported.

@sophia-guo I have opened backport for jdk21u:
https://github.com/openjdk/jdk21u/pull/221

https://github.com/openjdk/jdk21u/commit/667e639964dcdfcdad52ea14cb8ac7e3576b9616 , close this one.


Latest angular translate plugins aren't published to npm

Subject of the issue
The main repo is published as 2.10.0, but all child plugins are still on 2.9.2 e.g.
https://www.npmjs.com/package/angular-translate-handler-log
https://www.npmjs.com/package/angular-translate-loader-url
This causes angular translate to be duplicated when installed with npm like so:
├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>
Thanks! :smile:

Sorry again. The steps are still not scripted enough.
Done.

Thanks so much for the quick response! :smile:


WIP: Improve gitlab integration tests logging on failure

Fixes #793

Codecov Report

Merging #798 (5816e6a) into develop (5554814) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #798   +/-   ##
========================================
  Coverage    94.91%   94.91%
========================================
  Files          142      142
  Lines         9172     9172
========================================
  Hits          8706     8706
  Misses         466      466

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Ready for review. You can only see the results on quartz-clang_10_0_0-src if it fails. Once everything is ready to be merged I'll revert the tests submodule back to normal so everything will pass.


Add polartoolkit

Insert URLs to the project here: https://github.com/mdtanker/polartoolkit

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All projects listed will be posted on our Mastodon channel

@mdtanker

Thanks for adding this!


PG: Collection replication role tag by default

What does this PR do?
Enable tag replication role by default.
Motivation
The tag information (master or standby) is extremely useful to have to identify who is the primary in a cluster. It is also cheap to get (only a call to pg_is_in_recovery) and a db host will mostly be tagged with a single value except when a promotion happens. By enabling it by default, we can rely on its presence for other part of the product like watchdog alerts.
Additional Notes
The unit test test_replication_stats was removed as it was redundant with integration test done on a replica.
test_replication_tag was moved from test_unit since it relies on an existing pg_instance.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Would it make sense to get rid of the function _get_service_check_tags()?
Yeah, it's not used anymore, removing it.

/merge


[feature]: random delay in typing

What feature would you like to see in sendkeys?
Thanks for the great tool. It has saved me a lot of work!  I use sendkeys when creating screen casts and would love for the delay between each keystroke to be in a random range. This would make it look a lot more like a human typing.
I'm going to see if I can write something hacky that inserts some random <p:> directives, but it'd be great for the tool to support it natively.
Other information:
No response

Might be able to introduce another input argument like --random-delay. Where this value is the number of seconds randomised between 0 and random amount to add to the default delay value.
What do you think?

Yep, that is what I was thinking. That's how I've seen it done in other similar libraries. Would make an attempt at it myself, but don't do Swift yet... that never ending "yet".


Code presenting broken for YAML

I have a presentation with some YAML and Java code. While Java markers scroll to correct place and highlight code, YAML does not scroll so highlight is not visible. Even when it is visible (at the beginning of the file), it is very faint.
Expected behavior:

correct portion of the YAML file should be shown
highlight should be more visible

It is visible in first code example in this presentation

Hi @zeljkot I suspect the issue is that  highlight.js (the underlying code highlighting library that is used by GitPitch) does not support YAML syntax in the core library. Given YAML is a superset of JSON you might be able to temporarily workaround this by using a JSON language hint on your code delimiter.
For example:
---?code=swagger/example/complete.yaml&lang=JSON

You can learn more about using code-delimiter language hints on the GitPitch Wiki here. For optimal rendering I recommend you use these on all code delimiters. Anyway, if you try this out and let me know how you get on.

FYI @zeljkot I will look into adding first class support for YAML syntax over the next few days. In the meantime, try the workaround described above.

It worked, thank you! And thank you for the quick reply! I am giving a presentation on Saturday and this really helped me!

Hi @zeljkot just to let you know that I've just pushed a patch that gives GitPitch native syntax highlighting support for YAML. This patch is now live on gitpitch.com. You can therefore use a YAML language hint for any future code blocks that contain YAML.
Note, the original workaround should continue to work as before although you may get improved syntax highlighting if you update your PITCHME.md to use a YAML hint.
Best of luck with your upcoming presentation.


Certificate used for mocks is expired

Hey there, it looks like the certificate for the "mockoapp.net" domain expired on May 1st 2024 (one day ago).

(It's worth noting that the certificate for the "mocko.dev" domain remains valid for a few more months, ie the above only affects making mock requests).
Perhaps there was some problem with automated certificate renewal?

Hey @mirekkukla ! Thanks for the report, I'm taking a look into it

@mirekkukla we're back online! 😄
I'm so sorry for the unavailability, the auto-renewal failed and no alerts were triggered.. I started the job manually but I'll take a look at what caused the issue.
Let me know if it worked!

Good to go, thanks for the quick fix!
On Thu, May 2, 2024, 2:18 PM Gabriel Pinheiro @.***>
wrote:

@mirekkukla https://github.com/mirekkukla we're back online! 😄
I'm so sorry for the unavailability, the auto-renewal failed and no alerts
were triggered.. I started the job manually but I'll take a look at what
caused the issue.
Let me know if it worked!
—
Reply to this email directly, view it on GitHub
https://github.com/mocko-app/mocko/issues/107#issuecomment-2091663346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG4RSVWRNZDK343TJEWFYTZAKURPAVCNFSM6AAAAABHEKXAQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJRGY3DGMZUGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Make MicroWin's New Desktop Shortcut have 'Run as administrator' property on

Commit Title
MicroWin - Make the New Shortcut have 'Run as administrator' property on
Commit Message Body
The added lines of PowerShell Code was taken from public function 'Invoke-WPFShortcut.ps1' under the 'functions/public' Folder.

Quick Note
I haven't tested these changes on my machine, so if anyone have the spare time to give this PR a a quick run for it's changes, I'll be happy to hear from you all if it works as expected or not.
Expected Behavior
The changes in this PR takes affect on the New Shortcut that made with MicroWin Utility, this Shortcut will appear after you've installed the new Windows Installation Disc ISO (The one made with MicroWin, and ends with .iso in its Filename) on a Physical Machine (A Laptop or Desktop), or on a VM (Virtual Machine), you'll see this shortcut on The Desktop, and it should have 'Run as administrator' property checked on by default, Check the Screenshot below.
The Screenshot showcasing how final result should look like

Looks good to me. By default, if the script is run without admin privileges, it will start another PowerShell process as admin; but it's good to explicitly start the script as an admin automatically


cast created with post_cast disappears after 24h

🐛 Bug Report
The method post_cast creates a cast, however it disappears after a given period of time (24h).
🔬 How To Reproduce
Steps to reproduce the behavior:
# farcaster = "^0.7.11"
c2 = Warpcast(private_key="MY-PRIVATE-KEY")
cast = c2.post_cast("test cast")
cast_hash = cast.cast.hash
print (f"cast_hash {cast_hash}")
# ... Wait 24h
c2 = Warpcast(private_key="MY-PRIVATE-KEY")
cast = c2.get_cast(cast_hash)
$ Error
    c2.get_cast(cast.cast.hash)
  File "/home/gabrielfior/miniconda3/envs/PMA/lib/python3.10/site-packages/farcaster/client.py", line 338, in get_cast
    response = self._get(
  File "/home/gabrielfior/miniconda3/envs/PMA/lib/python3.10/site-packages/farcaster/client.py", line 89, in _get
    raise Exception(response["errors"])  # pragma: no cover
Exception: [{'message': 'Cast 0xabdea1ff98521bef0c1f5e24ef0461ae21d91805 does not exist'}]

For example, the post with hash 0xabdea1ff98521bef0c1f5e24ef0461ae21d91805 was created 1 day ago and now it doesn't exist anymore.
Code sample
See above
Environment

OS - Linux (Ubuntu)
Python version - Python 3.10.14

Screenshots

📈 Expected behavior
I expected the cast to be stored permanently and be available for later retrieval.

get_cast don't work rn(probably they changed endpoint url).
you should try to access your cast by hubble https://docs.farcaster.xyz/reference/hubble/httpapi/casts


[Membership]: k8sgpt, k8sgpt-operator SubProject Approver - JuHyung-Son

GitHub Username
@JuHyung-Son
Membership Level
Subproject Approver
Subproject
k8sgpt
Sponsors
@AlexsJones @thschue
Requirements
Highly experienced active reviewer and contributor to a subproject
List of contributions to the project
testing functions.
fix bugs.
suggest and implement new features
Code of Conduct

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

Contributor Guide

[X] I have read and understood the Contribution Guide

Contributor Guide and Membership Requirements

[X] I have read and understood the Community Membership Document

I support this

I also support this, welcome on board

thanks!


Class mailes does not exist

Maybe the title has nothing to do with this library, however, I'm going to tell you about a problem I have.
I was using this library and everything was going very well, however today I started generating the following error: "Class mailer does not exist" when I used any command like composer update, after hours searching the internet and not finding any solution. It occurred to me to comment on the last library I had installed (FCM being the last one). By commenting on it in the config / app, any command works. However, I do not understand why.
Thankful that I can help me.

I have the same problem, i reverted this commit and it's started to work again: https://github.com/benwilkins/laravel-fcm-notification/commit/673adf7c5d8e2005b2a3efcd92a5ff9d50978da8
I'm using laravel 5.6

I'm using laravel 5.5.4 and before update this package i have this same problem. Class mailer does not exist

I have the same problem using laravel 5.6. The cause of the problem is this commit: 673adf7

I try revert this code in my vendor and solve the problem. This commit [673adf7] is the problem.

I'll look into it, but you can try using version 3.0.0 instead of 3.0.1 if you're on Laravel <5.8.

I'm on 5.8, and I'm not receiving this error.

I have the same problem in Laravel 5.5, to resolve the problem change to v3.0.0, no works with v3.0.1

i use Laravel 5.4, and v3.0.0. And I still have the same problem

Hello I was also having same issue. and it was not fixing by downgrading to 3.0.0.
So i did it by manually Placing old code in FCMNotificationServiceProvider class
Also you need to remove the parameter $notification from routeNotificationForFcm function placed in your model.

Reverting to 3.0.0 should fix this issue. Make sure you are actually using that version by checking your composer lock. If you are and it doesn't fix it, try reverting back to 2.0.2. Note that 2.0.2 will not work with Laravel 5.8.

I was having the same issue on Laravel 5.6. Setting the package version to 3.0.0 in composer.json and running composer install fixed the issue.


chore(dev-deps): upgrade lerna

Codecov Report

Merging #112 (934873b) into master (240c30f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #112   +/-   ##
=======================================
  Coverage   57.14%   57.14%
=======================================
  Files          59       59
  Lines        1449     1449
  Branches      291      291
=======================================
  Hits          828      828
  Misses        621      621

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 240c30f...934873b. Read the comment docs.


Fix include_missing for aliased parameters

include_missing currently includes aliased parameters even if they're not specified. The existing test for include_missing was also broken, it passed regardless of the value of include_missing.

Good work @jonasoberschweiber, thank you.


WIP- Updating Erlang deps reference PR #3368

Signed-off-by: jan shahid shaik<EMAIL_ADDRESS>Description
Reference PR - 3368
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/4694


Add PlasmaPy

Adds the implementation in PlasmaPy to our curated list of projects.
Thanks to @ritiek, @StanczakDominik, and @namurphy! :rocket: :sparkles:

Hi @ritiek, @StanczakDominik, and @namurphy,
I just invited you to our newly created slack space at https://openpmd.slack.com :)


feat: add IF EXISTS for DROP

For https://github.com/GreptimeTeam/greptimedb/issues/2794
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?

Summarize your change

Add the drop_if_exists to match the mysql statement for drop

How does this PR work? Need a brief introduction for the changed logic (optional)
Describe clearly one logical change and avoid lazy messages (optional)
Describe any limitations of the current code (optional)

Checklist

[ ]  I have written the necessary comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

PTAL @sunng87 @waynexia

Please run make to generate language-specific code

Please run make to generate language-specific code

Oh yes. I thought the generated code will be did by the bot.

LGTM.
Can you make a rebase to solve conflicts? Thank you!

Sorry, it has some conflict and can't rebase. BTW, I don't have complete compile chain like golang and cmake in my mac.

LGTM.
Can you make a rebase to solve conflicts? Thank you!

Sorry, it has some conflict and can't rebase. BTW, I don't have complete compile chain like golang and cmake in my mac.

Our makefile utilizes docker to generate code, only a docker env is required

LGTM.
Can you make a rebase to solve conflicts? Thank you!

Sorry, it has some conflict and can't rebase. BTW, I don't have complete compile chain like golang and cmake in my mac.

Our makefile utilizes docker to generate code, only a docker env is required

Yes, but my personal mac's docker client is connected to my company's internal docker server due to the limited disk space. And I found the golang dependency is still needed.

LGTM.
Can you make a rebase to solve conflicts? Thank you!

Sorry, it has some conflict and can't rebase. BTW, I don't have complete compile chain like golang and cmake in my mac.

Our makefile utilizes docker to generate code, only a docker env is required

Yes, but my personal mac's docker client is connected to my company's internal docker server due to the limited disk space. And I found the golang dependency is still needed.

Whoops...


Azure IMDS default endpoint does not use HTTPS

@microsoft-github-policy-service agree

@dpiche174 This PR updates generated code.  Unfortunately, directly updating the code will only correct this issue until the next time we regenerate the crate.
The underlying specification, which this crate is based upon, states it supports both HTTP and HTTPS.
https://github.com/Azure/azure-rest-api-specs/blob/4802c0f73151b90bd207df82cb0207d70db7b445/specification/imds/data-plane/Microsoft.InstanceMetadataService/stable/2021-12-13/imds.json#L8-L13
I've submitted an issue to track updating the rest API specification ( https://github.com/Azure/azure-rest-api-specs/issues/22581), but we can probably work on a work-around to address using the correct scheme until this is addressed upstream.

Note, in the meantime, you can use ClientBuilder.endpoint to specify an alternate endpoint of http://<IP_ADDRESS>/metadata.

This will be fixed in #1211 (or another PR like it).

Note, in the meantime, you can use ClientBuilder.endpoint to specify an alternate endpoint of http://<IP_ADDRESS>/metadata.

Works perfectly. Thank you!


Setting role based permissions on Form Submissions and Notification Recipients

Is there a way to control the permissions for the Strapi administration panel users such that users with a given role are able to perform CRUD operations on the Form Submissions and Notification Recipients collections? I am currently only able to use Super Admin roles to give access and control over these collections.

I overlooked the fact that the collections are named Submission and Recipient


finish dataview beta

[ ] ability to curate projects / tokens / channels
[ ] set a predefined initial view
[ ] set a predefined initial color
[ ] provide a thumbnail

https://github.com/openconnectome/neurodata.io/issues/94


Bugged enter key after applying xremap

I am running Nobara Linux and have a Lenovo IdeaPad 5 14ARE05 with model number 81YM.
After applying any xremap configuration I get a strange bug where the enter key is spammed in the terminal. What's stranger is that this only seems to affect certain applications. Pressing escape seems to stop the enter key from being spammed, however some applications continue to spam enter if i alt-tab out and back in to the application. So far some examples are that enabling any xmap configuration leads to the terminal being spammed but pressing esc will appear to permanently stop the enter spam.
On VSCode it appears that pressing esc stops the enter spam temporarily but continues it if I alt-tab out and back in.
Firefox seems to not be affected by this at all.

I have the same error

I have the same error on Gnome wayland running on arch. Enter key becomes unresponsive until I kill xremap. This only affects my external usb keyboard. The built-in keyboard on my laptop works fine. Laptop is an Asus Zephyrus G15

Did you end up finding a solution to the issue, or at least know what was causing it?

Did you end up finding a solution to the issue, or at least know what was causing it?

No luck :(

Did you end up finding a solution to the issue, or at least know what was causing it?

No luck :(

Been tearing my head out trying to figure out what is causing this. Literally any keymap is causing this weird enter key spam. What's even weirder is that I had xremap working before on my PC and not on my laptop but after reinstalling nobara and then downloading xremap again it also seems to be doing the enter key spam thing.

Been tearing my head out trying to figure out what is causing this. Literally any keymap is causing this weird enter key spam. What's even weirder is that I had xremap working before on my PC and not on my laptop but after reinstalling nobara and then downloading xremap again it also seems to be doing the enter key spam thing.

Are you also on gnome? have you tried on any other DE?

Been tearing my head out trying to figure out what is causing this. Literally any keymap is causing this weird enter key spam. What's even weirder is that I had xremap working before on my PC and not on my laptop but after reinstalling nobara and then downloading xremap again it also seems to be doing the enter key spam thing.

Are you also on gnome? have you tried on any other DE?

No I'm on KDE. Not willing to switch to a different DE for this unfortunately.

I also encountered this problem on Ubuntu Gnome Desktop OS

I've worked around the issue like this:
sleep 1 && xremap remap.yaml

Thanks! That worked for me too :)


Update bundled JDK 17 to latest

See the list of mitigated security vulnerabilities here: https://www.oracle.com/security-alerts/cpuapr2022.html#AppendixJAVA

Codecov Report

Merging #1016 (28e96c6) into 8.x (6c9e9b7) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                8.x    #1016   +/-   ##
=========================================
  Coverage     64.93%   64.93%
- Complexity    12226    12227    +1
=========================================
  Files          1697     1697
  Lines         56460    56460
  Branches       5216     5216
=========================================
+ Hits          36663    36665    +2
  Misses        17567    17567
+ Partials       2230     2228    -2

Impacted Files
Coverage Δ

...nternational/snowowl/core/events/util/Promise.java
73.21% <0.00%> (-1.79%)
:arrow_down:

...index/src/com/b2international/index/es/EsNode.java
81.25% <0.00%> (-1.05%)
:arrow_down:

.../b2international/index/es/client/EsClientBase.java
65.90% <0.00%> (+1.13%)
:arrow_up:

...national/index/revision/BaseRevisionBranching.java
90.64% <0.00%> (+1.16%)
:arrow_up:

...x/src/com/b2international/index/util/JsonDiff.java
77.64% <0.00%> (+2.35%)
:arrow_up:

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 6c9e9b7...28e96c6. Read the comment docs.


Re-enable system libunwind support

Recent discussion on https://github.com/dotnet/coreclr/pull/26082 suggests this should be working.

This includes a backport of work-in-progress https://github.com/dotnet/runtime/issues/42661

https://github.com/dotnet/runtime/pull/42853 was merged into 5.0, so all this PR needs to do now is set the cmake properties.

@crummel @dagood @dseefeld can someone please review/merge this? Thanks!


HTTP Error 403: request disallowed by robots.txt

The download.py script is failing on line 44, in login:
# First establish contact with Garmin and decipher the local host.
page = agent.open(BASE_URL)
pattern = "\"\S+sso\.garmin\.com\S+\""
script_url = re.search(pattern, page.get_data()).group()[1:-1]
agent.open(script_url)  ### FAILS HERE

The script_url being parsed here: https://sso.garmin.com/sso/js/gauth-widget.js?20130313, which points to a GAuth javascript widget. From what I can tell, this is not going to help with the login process.

add the agent line to the def login worked for me:
def login(agent, username, password):
global BASE_URL, GAUTH, REDIRECT, SSO, CSS
    # First establish contact with Garmin and decipher the local host.
    page = agent.open(BASE_URL)
    agent.set_handle_robots(False)   # no robots
    agent.set_handle_refresh(False)  # can sometimes hang without this

Fix works, thank you @mipapo!


lseek permission denied?

Steps to Reproduce
(1) The cfile is :
#include <stdio.h>
#include <fcntl.h>
#include <unistd.h>
#include <sys/stat.h>

int main() {
    const char* file_name = "Data/hello.txt";
    int open_style= O_RDONLY | O_CREAT;
    int fd = get_fd(file_name, open_style);
    fd_seek9qW7uWsf5B(fd);
    snapshot(fd);
    return 0;
}

int fd_seek9qW7uWsf5B(int fd) {
    int start_value = 32;
    int seek_parameter = SEEK_SET;
    return fd_seek(fd, start_value, seek_parameter);
}

int get_fd(const char* file_name, int open_style){
    // Open a file for reading
    int fd = open(file_name, open_style);
    if (fd == -1) {
        perror("Failed to open the file");
        return 1;
    }

    return fd;
}

int fd_seek(int fd, int start_value, int seek_parameter) {
    printf("Enter fd_seek.\n");
    off_t offset = lseek(fd, start_value, seek_parameter);
    if (offset == -1) {
        perror("lseek");
        close(fd);
        return 1;
    }

    printf("File size: %lld bytes\n", offset);
    printf("Leave fd_seek.\n");
    return fd;
}

int snapshot(int myfd){
    printf("Enter snapshot\n");

    struct stat file_info;
    if (fstat(myfd, &file_info) == -1) {
        perror("Error getting file attributes");
        close(myfd);
        return 1;
    }

    printf("File Size: %lld bytes \n", (long long)file_info.st_size);

    off_t cur_offset = lseek(myfd, 0, SEEK_CUR);
    if (cur_offset == -1) {
        perror("Error getting current offset");
    }
    printf("Current offset: %lld \n", (long long)cur_offset);

    if (close(myfd) == -1) {
        perror("Error closing file");
        return 1;
    }

    printf("Leave snapshot\n");
}

(2)compile the c file into wasm: ./wasi-sdk-16.0/bin/clang --target=wasm32-unkown-wasi --sysroot=./wasi-sdk-16.0/share/wasi-sysroot lseek.c -o lseek.wasm
(3)exeute open.wasm
wasmtime run  --dir=./Data lseek.wasm
The permission of Data/hello.txt is 0400 or 0500, user1 create the Data/hello.txt directory before and user1 execute the Wasm file.
Expected Results
Using gcc lseek.c -o lseek and ./lseek to execute get the following result:
Enter fd_seek.
File size: 32 bytes
Leave fd_seek.
Enter snapshot
File Size: 30 bytes
Current offset: 32
Leave snapshot

And wamr, wasmedge,wazero also print the above message.
Actual Results
wasmtime prints:
Enter fd_seek.
Failed to open the file: Permission denied
lseek: Invalid seek
Error getting file attributes: Bad file descriptor

I'm not sure whether wasmtime and native ubuntu and other runtimes get different could be a bug or only difference ?
Versions and Environment
wasmtime 13.0.0
Operating system: Ubuntu 20.04
Architecture: x86_64

As with prior bugs, I'll note again that the C source as-listed here doesn't compile due to requiring reordering functions. One thing I'd also recommend is improving error handling in the source code. For example the issue here is that the file can't be opened due to permissions, but the program continues thinking that the open file descriptor is "1" which means you're trying to seek stdout which doesn't work.
Otherwise though the main issue here is that with native permission bits of 0400 and 0500 they don't have the write bit. The file is being opened in Wasmtime for writing, though, which is why this is failing. The file is being opened here with O_RDONLY | O_CREAT. That eventually translates to requesting a write permission when opening.
I don't know enough about everything in play here to say definitively which is correct. I understand that Linux seems to allow opening a preexisting read-only file with O_CREAT. Wasmtime doesn't seem to allow this (maybe cap-std? unsure.). I'm also not sure what other platforms do. If you're interested assistance in investigating this would be appreciated.


Probably breaking change without updating semver major.

This error showed up after running npm i with package locked up on major version: "glamor": "^2.17.14",. It seems that you removed the fontFace method from 'glamor' exports and replaced the usage with css.fontFace without updating major semver version.

yeah that was my bad. I've since brought it back though, what version are you on?

hey, same issue here when installing the latest from npm which appears to be 2.20.8

2.20.8 installed in node_modules

I'll check and fix this today. sorry!

could you folks check 2.20.9?

closing this, feel free to reopen if it happens again.


[BACKLOG-30125] PDI Replace in string step not removing file extension

I've added two test classes with a total of 130 unit tests that cover a few scenarios for all possible combinations of "Use ReGex"/"Case sensitive"/"Whole word"/"Is Unicode" values.
@pentaho/tatooine @ppatricio

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/replacestring/ReplaceString.java
  engine/src/test/java/org/pentaho/di/trans/steps/replacestring/ReplaceStringProcessRowNoUnicodeTest.java
  engine/src/test/java/org/pentaho/di/trans/steps/replacestring/ReplaceStringProcessRowUnicodeTest.java
  engine/src/test/java/org/pentaho/di/trans/steps/replacestring/ReplaceStringTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.4% :small_red_triangle_down:
Line Change: -3.8% :small_red_triangle_down:

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UIEERepositoryUser

Branch Change: -8.3% :small_red_triangle_down:
Complexity Change: -6.7% :small_red_triangle_down:
Instruction Change: -0.9% :small_red_triangle_down:
Line Change: -3.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/replacestring/ReplaceString.java
  engine/src/test/java/org/pentaho/di/trans/steps/replacestring/ReplaceStringProcessRowNoUnicodeTest.java
  engine/src/test/java/org/pentaho/di/trans/steps/replacestring/ReplaceStringProcessRowUnicodeTest.java
  engine/src/test/java/org/pentaho/di/trans/steps/replacestring/ReplaceStringTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.step.StepMeta

Branch Change: -0.6% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -4.4% :small_red_triangle_down:
Complexity Change: -5.1% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Complexity Change: -1.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Branch Change: -2.6% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.Trans

Branch Change: -0.1% :small_red_triangle_down:
Complexity Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:


Is there support for "Evening Delivery", "Extra Insured"?

Just started using your package to implement DHL label printing in a project (labels are printed after packing, so number of labels and package type will be decided at the end). Shipping options are retrieved from a Magento2 site. The DHLParcel extension for Magento2 has options to choose for a evening delivery or insure the package. Package insurance is not selectable for the customer, but an evening delivery is. The shipment type will be "DOOR,EVE" instead of "DOOR". I could not see any support for this in the code I think. Could this be implemented?
It would be nice to have an option like $parcel->isEveningDelivery() and $parcel->isExtraEnsured()

I just made a PR for the insured shipping option @noromamai, hopefully this helps you as well!
https://github.com/mvdnbrk/dhlparcel-php-api/pull/63


[VL] vcpkg: Broken libelf mirror

The link we use for downloading libelf no longer works and is causing failures in static build:
https://ftp.osuosl.org/pub/blfs/conglomeration/libelf/libelf-0.8.13.tar.gz
Change to another mirror.

@PHILO-HE

perhaps should be ports back to 1.2


Framework version Bump

$sub

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6815917562

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6815917562
Status: success


add_to documentation

I noticed that the put_to function has been renamed add_to, but the docs and readme still refer to put_to. I was going to open a PR to update the docs, but was curious what the reasoning was for the change. What is the significance of add_* vsput_*?

My mistake on the docs.
The reasoning for changing it from put_to/2 to add_to/2 and add_to/3 is because SendGrid's latest API for sending transactional mail made it much easier to deal with multiple recipients. I wanted to make that feature available for people to use. Since the TO field can now have a list of recipients, I change the function to add since all other places in SendGrid.Email that modify lists also use the word add.
If you create a PR for the doc updates I missed, I'll gladly merge.

Will do. Is it possible to assign this issue to me?

Unfortunately, I'm unable to assign an issue to you.

That is true, starting to use today and all docs saying about put_to and not add_to. Has none references to this.

Latest version correctly uses add_to in the docs.


Is it possible to use this plugin with Phonegap BUILD?

Is it possible to use this plugin with Phonegap BUILD?
Thanks

Yes, it has been tested successfully on android via Phonegap build.

Thanks for the fast response. What about iOS ?

Since the ios part of this plugin relies on the swift-support plugin which uses hooks, it will not compile on phonegap build. Only when PGB allows running of cordova hooks, then this plugin will work. Adobe clearly mentioned it has no intention of allowing hooks to run on the build service.

Hi @DanLuczywo the latest version of this plugin has been re-written on objective C and should now work on Phonegap Build.


Sdc gallery app single choice component workflow

Is your feature request related to a problem? Please describe.

Design single choice questionnaire json with required fields. (error message will be shown for mandatory single choice)
Design single choice questionnaire json without required fields. (error message will not be shown as field is not mandatory)
Api to retrieve single choice questionnaire json from model view.
Handle single choice item click
Implement questionnaire fragment and model view to show questionnaire json
Show action bar, update title as per screen, align title text.
Update color palette hex codes.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Would you like to work on the issue?
Please state if this issue should be assigned to you or who you think could help to solve this issue.

Continuing implementation ...

@shelaghm
Sdc library has two views which support a single choice component (radio button).

QuestionnaireItemRadioGroupViewHolderFactory : type is choice as per fhir specification, which can have a number of radio button options, and the user can select one out of them.

QuestionnaireItemBooleanTypePickerViewHolderFactory : type is boolean as per fhir specification, which will have two radio button options yes and no.  And the user can select one out of them.

I am thinking of having 2 separate single choice components for choice and boolean type in the components list. Is it okay?
cc @jingtang10 @aditya-07

@santosh-pingle this makes sense to me to show both options as separate components. Please create a new githib issue and tag me.
Based on this I'll create a new icon for the Boolean choice and will use different example content for the two different components. I'll update the figma file to reflect this.

@shelaghm
Thanks for sharing color palette!
Can you please also share color PrimaryVariant and color SecondaryVariant color hex code.

@santosh-pingle Decided we will not use PrimaryVariant and color SecondaryVariant color. Just posting it here so we close the loop on the decision.


Categories edit issue without any node .

Hi Team,
the issue has been displaying on each category's edit section .

@adeshsuryan
Hi, thanks for reporting this issue. Would you please draw your category hierarchy and tell me which one do you wanna delete?
many thanks

@adeshsuryan
I could not generate your issue. Tell me what you did. Many thanks

Hi Team,
I'm sharing with you the error with full details.
OS and config :
Ubuntu 18.4, PHP 7.3
Laravel: 7.x
Issue: The issue has happened when the user edits blog categories the first time. That time we not updating any root categories and that time node looking left and right node etc.

Let me check them.
We have a big release on 1st Dec. Good news wait for it.


Automatic stepsize and batchsize determination for SVRG

Is your feature request related to a problem? Please describe.
For some model configurations (Poisson GLM, soft-plus, with Ridge regularization) and optimal stepsize and batch size can be calculated for SVRG according to the theory in Sebbouh et al. 2019.
However, for an un-regularized model, the strong-convexity hypothesis does not hold. What should we do in this case? How about for Lasso and GroupLasso?
Describe the solution you'd like
Define what to do for non-strongly convex problems and/or non-L-smooth problems.

This Ada-SVRG method seems to tackle the issue of non-strong convex problems keeping a linear convergence rate:
https://arxiv.org/pdf/2102.09645
This relaxes the strong convexity assumption and adjusts the stepsize. It has a projection of the parameters into a compact set, but in our case the parameter can live in $\mathbb{R}^k$.
I am not sure if removing the projection step will make the algorithm potentially unstable.


msg.sender and tx.origin is 00000...0

Have had a report earlier, and this has apparently been an issue. msg.sender and tx.origin are zero when calling from outside the vm, but not from within (i.e. from other contracts). Not sure whether it's tx only, or both with calls and txs.

This is not an eris-db specific issue, it's because the client does not set an address for call or "simulated calls", but it does so for transactions. Setting the address (so it works like it does with transactions) will be done temporarily but we need to go over Ethereums code and see that we do the same thing they do.
Keeping this issue open.

done


Update base docker image

Did you update the tests?

[ ] yes
[ ] no
[x] n/a

Did you update the docs?

[ ] yes
[ ] no
[x] n/a

The cooking grocery lists service is a GAE app that until recently ran on go v1.16. Yesterday I found out that GAE no longer supports go v1.16 at all, and requires restructuring our repo a little bit to accommodate other changes to upgrade to v1.18. Our other option is to downgrade to v1.15 👎
Using v1.18 is in preview mode and requires gcloud cli version 417.0.1 or later

@tonglil Sounds good. I won't be able to merge or tag anything in this though

Thanks, @mattgrunwald I have tagged v0.3.0 release.

Hi @mattgrunwald, unfortunately due to incompatible changes being introduced with the sdk upgrade, we had to revert this PR.
Before upgrading the sdk any further, the plugin should be migrated off of using appcfg. I believe this is possible without introducing a breaking change as the drone plugin parameter interface can abstract the changes from the user.


Always return PhysicalResourceId

Although the documentation does not list PhysicalResourceId as a parameter for a FAILED Create Response, if it is not included CloudFormation will error with Invalid PhysicalResourceId.  CloudFormation then tries to send a DELETE request back to the lambda function.
This module should ensure that a PhysicalResourceId is sent with any call back to CloudFormation so that the correct error is displayed in CloudFormation Events and no DELETE request is initiated.

CloudFormation is always sending a DELETE request on any FAILED status with rollback.  I don't think there is a way to stop CloudFormation from doing this, even though the create failed and there should be no PhysicalResourceId, CloudFormation is demanding one and also seems to be forcing a DELETE during rollback.


Improve layout for Parts of Speech dropdown

Making it look and behave more like the default dropdown on both desktop and mobile.

@hanatgit, The cause of the problem was the "parts of speech" were using "tax" in the query string, and so is the semantic domain when you click on it. I changed parts of speech to use "pos" in the query string and that seems to have fixed the problem.


fix(remix-node): fix NodeRequest's clone function

Closes: #5520

[ ] Docs
[x] Tests

Testing Strategy:
Add test case

@tomer-yechiel Seems like something went wrong with rebasing.
Please rebase onto latest dev & set base branch of this PR to dev as well

@tomer-yechiel Seems like something went wrong with rebasing. Please rebase onto latest dev & set base branch of this PR to dev as well

@MichaelDeBoey done

Superseded by #6512


Connecting payment gateway

Connect payment gateway to wordpress site, use Singapore bank

Account name ,Account number , Bank name , IFSC , IBAN, BIC / Swift.......Need these details....
|

Payment option is to be added while customer buys reports??


feature: getrawtransaction output should match zcashd

Motivation
In order for zebrad to be able to act as a drop in replacement for zcashd we should ensure that the Zebra output for the getrawtransaction RPC call matches the zcashd output.
This was reported by a user in Discord.
Specifications
No response
Complex Code or Requirements
No response
Testing
No response
Related Work
#8645

I didn't manage to figure out what the desired output should be from the issue description. Zebra returns only a subset of what zcashd returns, but the returned data matches zcashd in all checks. The reduced output is due to tight scope. Closing until we figure out what we need to add or change.


Analyze only one specific file against some deptrac configuration / IDE integration

It would be nice if there was a function to analyze only one specific file against some deptrac configuration. It would be a first step to integrate a validator in an IDE.
That would be awesome! Imagine you import some namespaced class and PhpStorm (or another favorite IDE) instantly marking this as an error in your file because it violates your defined architecture.

hi @playr1983g
it's not really possible to run deptrac just against one file.
deptrac is quite flexible and would allow you to write collectors that takes a look at other files.
for now i'll close this ticket as a duplicate of #71. from my point of view this ticket would be solved, if you were able to get instant feedback by retrunning deptrac. lets hope, that i'll be able to make it that fast.


prunes the JSON tree

I have putted my code in base/utils.js.

What problem is this solving and where is it going to be used? Just trying to clarify the context for a prune function. Is there an issue number related to it?
And take a look at https://github.com/logicerpsolution/vizabi/blob/develop/src/base/utils.js#L939
var allowed_IDs = ["time", "gdp_per_cap", "gdppc_pday"];

The columns should be configurable, not hardcoded, so they might be passed as an argument to the function you want to implement.
And not sure why, but the buid failed. Does gulp build work locally for you?

@arthurcamara1 @valorkin please don't look here until i approved this.
the related issue is #564

Cool @angieskazka


Change the Radio Adress?

How can i change the Radio adress?
When i tried the change it from .ini file radio doesnt play any solutions?

You did leave a wrong link. That's all.


What test.py is used for?

As the Table of Contents
We train the network by train.py
python train.py
We evaluate the trained network by eval.py
python eval.py
What puzzles me is that test.py is used for what?
When I should use it?
python test.py

I have the same question, do you have a answer now?

I have the same question, do you have a answer now?


refactor: set v8 flags before initialize v8 platform

What kind of change does this PR introduce?
Bug fix, Refactor
Description
After #227 the platform is initialized explicitly at the entry point. However, after initializing the platform, it was no longer able to set the v8 flags [1] [2] [3], so it is necessary to adjust the invocation order of the set v8 flag function to avoid v8's static assertion.

Thanks for the contribution @nyannyacha (sorry for the delay in merging, I was on holidays)


MODELIX-889 Make skipping of subtrees in bulk-sync production-ready

These changes are the continuation of work that has been started in #606.
Once #701 is merged, the first commit can be dropped.

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


AudioStreamPlayer3D is stuck with playing=true in the editor

Godot 3.0.5
Windows 10 64 bits
At some point when adding sounds to my game, I found one of my scenes started to play sound everytime in the editor. The playing property is true, and I cannot set it back to false. Eventually reopening the editor solved it.
The same happens with another AudioStreamPlayer3D, however restarting the editor doesn't reset it.
It is configured with no initial stream (because I dynamically choose a random sound with this one), but it is also stuck with playing = true, and I think this one also generates a warning when I play the game:
ERROR: set_stream: Condition ' !p_stream.is_valid() ' is true.
   At: scene/3d/audio_stream_player_3d.cpp:554

Sorry I don't know what is the repro steps for this, but I would suggest to investigate what can cause an editor user from being unable to stop a player.
https://youtu.be/1_nokkxePA0

@Zylann Can you (or anyone else) still reproduce this bug in Godot 3.2.1 or 3.2.2beta4?

Tried again in the project where it happened, that doesn't seem to reproduce. I was never able to find precise reproduction steps so I can't tell for sure if it was fixed.

@Zylann I'll close this for now then.
If anyone else can reproduce this bug in Godot 3.2.1 or 3.2.2beta4, please comment.


Add environment divergence subcommand

refactor(environment): cleanup consistent calls to same method
feat(environment): add divergence subcommmand
refactor(environment): remove unwanted methods from divergence
test(environment): add additional tests to divergence

Codecov Report
Base: 36.26% // Head: 37.47% // Increases project coverage by +1.20% :tada:

Coverage data is based on head (43e9de9) compared to base (6d077c8).
Patch coverage: 75.90% of modified lines in pull request are covered.

:exclamation: Current head 43e9de9 differs from pull request most recent head 0fc320c. Consider uploading reports for the commit 0fc320c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #347      +/-   ##
==========================================
+ Coverage   36.26%   37.47%   +1.20%
==========================================
  Files          46       47       +1
  Lines        2636     2719      +83
==========================================
+ Hits          956     1019      +63
- Misses       1568     1582      +14
- Partials      112      118       +6

Impacted Files
Coverage Δ

pkg/environment/divergence.go
75.90% <75.90%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Ability to blacklist peers

What is wrong?
When disconnecting from a peer, some kinds of disconnects suggest that we should not try to reconnect to it later (is peer myself? is it replying with protocol-breaking headers? etc). It would be best to not try to connect to it again later, but nothing is preventing us from retrying.
How can it be fixed

Add a peer blacklist
Check peer against blacklist before trying to connect to it
Add peer to blacklist for certain kinds of peer failures (like a protocol error)

The blacklist can be in-memory, resetting on every trinity reboot.

It'd be nice to use something with constant size storage, otherwise this will be a DOS vector.  Lets use a bloom filter and a salt to ensure that remotes cannot generate their own hash collisions.

Two wei:

some kinds of disconnects suggest that we should not try to reconnect to it later (is peer myself? is it replying with protocol-breaking headers? etc).

A common case on the ETH public mainnet is other-genesis peers with the same network ID that otherwise follow all [sub-]protocol rules correctly.
This can get annoying/overwhelming, especially since it consumes precious resources. geth/parity APIs aren't very comprehensive on this, so in a few cases I've resorted to firewall-level connection filtering (think iptables/shorewall/fail2ban).

Add a peer blacklist

May I suggest more generic/neutral language?.. Like: blocklist; but really: tier, filter, sieve.
The reason is that these things tend to get layered. A not entirely fictional scenario:

first I want to just block the irrelevant;
but then I want a limited set of "priority" peers (e.g. a VPN ring specified in a text file);
then there happens to be "customers" that maybe I'd drop on a rainy day, but they're paying, so I try not to;
eventually, there's the "too slow" category that gets dropped during low load periods, and when I myself get dropped for the same reason during high load, this is the pool I'd like to dip into, instead of thrashing around randomly.

I guess the gist is: all of these involve peer management, are application-specific and can't be guessed by developers in advance.
A blacklist/blocklist is fine as a start, but long-term this is best given away to the node operator's middleware.

A common case on the ETH public mainnet is other-genesis peers with the same network ID that otherwise follow all [sub-]protocol rules correctly.

We're already dropping other-genesis peers on connect. It seems reasonable to add them to the blacklist (name TBD), too.

May I suggest more generic/neutral language?.. Like: blocklist; but really: tier, filter, sieve.
The reason is that these things tend to get layered. A not entirely fictional scenario

Yeah, we'll have to think through the best API to let people plug in their own peer discovery logic without having to redefine a bunch of basic peer handling logic. Blacklist is a simple/temporary solution.

Relevant: https://github.com/ethereum/py-evm/issues/870

I'm running into this as part of #945 where the transaction pool wants to black list peers that send invalid transactions.
I wanted to throw out a rough implementation outline for this:

Implement a new standalone service e.g. BlockedPeerService
Expose an event API that other services can consume (Not strictly speaking events but using a generic version of our existing msg queue pattern)
Events would be: PeerAdded, PeerRemoved, ...
API methods would be:

add_peer(peer: BasePeer)
remove_peer(peer: BasePeer)
is_blocked(peer: BasePeer
...

Then I guess the PeerPool would simply listen to events from the BlockedPeerService to perform the actual disconnect. Services such as the TxPool would call blocked_peer_service.remove_peer(douchebag) and everyone else would receive a new event PeerAdded accordingly.
Thoughts?

What do you think about making this part of the PeerPoolSubscriber API and allow subscribers to voice their opinion within PeerPool.is_valid_connection_candidate?

and allow subscribers to voice their opinion within PeerPool.is_valid_connection_candidate

Sounds like you want to maintain the blocklist in the PeerPool then?
I'm not sure if we actually need that form of coupling. Maintaining this blocklist in a separate service that exposes an API to add / check peers and raises events when new peers are added seems more flexible to me.
We would essentially keep the PeerPool free from any of this logic and just interact with the BlockedPeerService by checking blocked_peer_service.is_blocked(peer) as well as subscribing to the events to get notified when new peers are added to the block list to perform a disconnect.

No, the idea is that the PeerPool.is_valid_connection_candidate would have a section in it which delegated checks to the subscribers.
class PeerPoolSubscriber:
    def is_valid_connection_candidate(...):
        ...

class PeerPool:
    def is_valid_connection_candidate(self, ...):
        ... # current logic
        for subscriber in self.subscribers:
            if susbscriber.is_valid_connection_candidate(...):
                continue
            else:
                return False

Something like that (but probably all with async and probably we can call all of the subscriber methods concurrently.

Ok, but each PeerPoolSubscriber doesn't want to maintain its own version of a blacklist to answer that is_valid_connection_candidate() call, does it?

Here are some from my book:
Tx Pool wants to block a peer

TxPool recognizes bad_peer has fed us junk too often (using TxPool internal accounting)
TxPool calls self.blocked_peer_service.block_peer(bad_peer)
BlockedPeerService propagates PeerBlocked event to everyone whos listening for it
PeerPool who is among the listeners, recognizes the new blocked peer and disconnects.

New Peer Joins the Pool

A new peer joins the pool but the PeerPool checks self.blocked_peer_service.is_blocked(new_peer) before it lets it in and propagates the new peer to PeerPoolSubscriber

My interest in this has renewed while working on syncing:
WARNING  08-31 14:05:18       chain  ETHPeer <Node(0x865a@<IP_ADDRESS>)> announced difficulty<PHONE_NUMBER>041421, but didn't return any headers after <BlockHeader #913134 52277e0e>@278543309
    INFO  08-31 14:05:18        peer  ETHPeer <Node(0x865a@<IP_ADDRESS>)> finished, removing from pool
    INFO  08-31 14:05:26        peer  Adding ETHPeer <Node(0x865a@<IP_ADDRESS>)> to pool
    INFO  08-31 14:05:26       chain  Starting sync with ETHPeer <Node(0x865a@<IP_ADDRESS>)>
 WARNING  08-31 14:05:27       chain  ETHPeer <Node(0x865a@<IP_ADDRESS>)> announced difficulty<PHONE_NUMBER>391173, but didn't return any headers after <BlockHeader #913134 52277e0e>@278543309

Very soon after booting a peer, they reconnect and gum up the sync (by announcing a high TD but then not returning any headers)


Use GitHub Action to automatically add "triage" label to new issues

See https://github.com/Senzing/template-repository/issues/23

Fixed via https://github.com/Senzing/github-util/issues/10


proposal: reflect: add generic variants of TypeOf() and ValueOf()

The current implementations of reflect.TypeOf() and reflect.ValueOf() each take an any for the value about which they are retrieving information. This works just fine in most cases, but can also cause annoyances when trying to get reflection info about an interface type itself. Due to the way interfaces work, the functions never even see the original interface type unless it's inside of a structured type of some kind, such as a poitner. For example,
var ex fmt.Stringer
t := reflect.TypeOf(ex) // Results in the underlying type, not fmt.Stringer itself.
p := reflect.TypeOf(&ex).Elem() // Results in fmt.Stringer itself.

This problem is exacerbated in the case where the value in question is from a non-addressable source, such as a function return, requiring an extra variable just to be able to address the interface type.
This is not true for generic functions, however. As such, I propose adding two new functions which I will call GenericTypeOf() and GenericValueOf() for lack of a better name. Feel free to bikeshed.
func GenericTypeOf[T any](v T) reflect.Type
func GenericValueOf[T any](v T) reflect.Value

These functions behave exactly as the existing two do, but with the difference that when passed a value of an interface type, they operate directly on the compile-time interface type, rather than the underlying runtime type.
Addendum
As an aside, a reflect.TypeOf() variant that only takes a type and not a value, i.e. func OnlyTypeOf[T any]() reflect.Type, might be useful as well for the case where you need to perform reflection on a type that you know for sure at compile-time, such as when you need to store the reflect.Type representing something to check other values against it later. One example that comes to mind is a reflection-based serialization scheme that has special handling for time.Time. The implementation could just do var timeType = reflect.OnlyTypeOf[time.Time]() somewhere in the global scope instead of having to do the somewhat less elegant var timeType = reflect.TypeOf(time.Time{}). This didn't seem useful enough to warrant mentioning in the main part of the proposal, however.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

The two that @rogpeppe points out seem plausible, but it seems like we should wait on adding generics still.
Especially in reflect.
Let's put this on hold for a cycle.

Placed on hold.
— rsc for the proposal review group

Both #51649 and #52310 would be alleviated by a generic version of reflect.ValueOf.
After all, the root of both issues is that reflect.ValueOf(nil) return a zero reflect.Value instead of reflect.ValueOf(new(any)).Elem() since type information about the original interface is lost.

@dsnet

Both https://github.com/golang/go/issues/51649 and https://github.com/golang/go/issues/52310 would be alleviated by a generic version of reflect.ValueOf.

I'm not entirely sure it would help much. In my experience, reflect.ValueOf is most often called on a value of interface type, and in that case, the value returned by a generic version of reflect.ValueOf would always have a Type.Kind of Interface, and so in practice you'd just end up calling IsNil to check if it's OK to call Elem (because you're almost always interested in what's going on inside the interface value that's passed to ValueOf), which I think amounts to much the same thing in the end as checking for nil on the original interface value.

I think this should come off hold and be merged with https://github.com/golang/go/issues/60088.

This proposal is a duplicate of a previously discussed proposal, as noted above,
and there is no significant new information to justify reopening the discussion.
The issue has therefore been declined as a duplicate.
— rsc for the proposal review group

This proposal is a duplicate of a previously discussed proposal, as noted above,
and there is no significant new information to justify reopening the discussion.
The issue has therefore been declined as a duplicate.
— rsc for the proposal review group

I think the auto-message is wrong. It's a duplicate of a newer issue, not a previously discussed proposal. Anyway, seems fine to move all discussion to https://github.com/golang/go/issues/60088.


Add support for more accessories

Thanks for the integration! Been working great w/ no hassle
The new Timberline / Timberline XL grills support a few extra accessories out of the box, namely an induction cooktop (hob) and a bluetooth probes. These show up under acc, but the type isn't "probe" and properties are slightly different.
Sample snippet pulled from the MQTT topic:
    "acc": [
      {
        "channel": "bt",
        "hob": {
          "fw": "01.00.55",
          "level": 0,
          "get_temp": 35,
          "set_temp": 0,
          "status": 0
        },
        "type": "hob",
        "uuid": "xxxxxxxxxxxx",
        "con": 1
      },
      {
        "channel": "BT0",
        "btprobe": {
          "batt": 8,
          "fw": "XX.YY.ZZ",
          "alarm_fired": 0,
          "get_temp": 129,
          "set_temp": 155,
          "ambient_temp": 1542
        },
        "type": "btprobe",
        "uuid": "xxxxxxxxxxxx",
        "con": 1
      }
    ],

This PR adds support for those additional known-good extra accessories

Can you create a PR against https://github.com/systemswizard/hass_traeger I am picking up this project until @sebirdman wants to pick it up again as they seem very inactive


Vending Machine Update Information

ITS-Pro time tracking: https://its-pro.ucsd.edu/projects/MA/issues/MA-42?filter=allissues
This is complete when:

[ ] Options are presented for displaying the following information: Campus ID Needed for COVID Vending Machines Students and campus employees will soon be required to swipe their campus ID card when picking up a self-administered test kit from a vending machine.

Notifications. A series of them spaced out over a few days or weeks.
Link on Covid-19 Info card - "Vending Machines" - links directly to the page/section about the Vending Machines on the RTL site.
Temporarily replace the promo banner from the current "Add CA Covid Notify" to a banner about the Campus ID requirement. Content to mimic whatever is created for RTL.

@woznaut Confirmed to go with #3 option.

#3 Done in QA (7.6 build 1270)
Link: https://go.ucsd.edu/2A9wdkI
Awaiting go-live direction

Live in prod


Biped/Quadruped Example Files

Hi Alex,
After a while, I ended up getting TOWR to work on 18.04 (and will go back to debug in 14.04 in a bit).
In regards to setting up a biped/any model with more than one leg, am I correct to push_back multiple times for all the end effector variables? I've modified the hopper file as follows, but I've run into issues with the following message:
terminate called after throwing an instance of 'std::out_of_range'
what():  vector::_M_range_check: __n (which is<PHONE_NUMBER>3709551615) >= this->size() (which is 21)
Aborted (core dumped)
It seems that this has to do with the number of end effectors.
In addition, is the meaning of:
"initial_EE_W" meant to refer to the initial angular velocity of the end effectors?
int main()
{
  NlpFormulation formulation;

  // terrain
  formulation.terrain_ = std::make_shared<FlatGround>(0.0);

  // Kinematic limits and dynamic parameters of the hopper
  formulation.model_ = RobotModel(RobotModel::Biped);

  // set the initial position of the hopper
  formulation.initial_base_.lin.at(kPos).z() = 0.5;
  formulation.initial_ee_W_.push_back(Eigen::Vector3d::Zero());
  formulation.initial_ee_W_.push_back(Eigen::Vector3d::Zero());

  // define the desired goal state of the hopper
  formulation.final_base_.lin.at(towr::kPos) << 1.0, 0, 0.5;

  // Parameters that define the motion. See c'tor for default values or
  // other values that can be modified.
  // First we define the initial phase durations, that can however be changed
  // by the optimizer. The number of swing and stance phases however is fixed.
  // alternating stance and swing:     ____-----_____-----_____-----_____
  formulation.params_.ee_phase_durations_.push_back({0.4, 0.2, 0.4, 0.2, 0.4, 0.2, 0.2, 0.4, 0.2, 0.4, 0.2, 0.4, 0.2});
  formulation.params_.ee_phase_durations_.push_back({0.4, 0.2, 0.4, 0.2, 0.4, 0.2, 0.2, 0.4, 0.2, 0.4, 0.2, 0.4, 0.2});
  formulation.params_.ee_in_contact_at_start_.push_back(false);
  formulation.params_.ee_in_contact_at_start_.push_back(false);
  formulation.params_.SetSwingConstraint();
  formulation.params_.OptimizePhaseDurations();

  // Initialize the nonlinear-programming problem with the variables,
  // constraints and costs.
  ifopt::Problem nlp;
  SplineHolder solution;
  for (auto c : formulation.GetVariableSets(solution))
    nlp.AddVariableSet(c);
  for (auto c : formulation.GetConstraints(solution))
    nlp.AddConstraintSet(c);
  for (auto c : formulation.GetCosts())
    nlp.AddCostSet(c);

  // You can add your own elements to the nlp as well, simply by calling:
  // nlp.AddVariablesSet(your_custom_variables);
  // nlp.AddConstraintSet(your_custom_constraints);

  // Choose ifopt solver (IPOPT or SNOPT), set some parameters and solve.
  // solver->SetOption("derivative_test", "first-order");
  auto solver = std::make_shared<ifopt::IpoptSolver>();
  solver->SetOption("jacobian_approximation", "exact"); // "finite difference-values"
  solver->SetOption("max_cpu_time", 20.0);
  solver->Solve(nlp);

  // Can directly view the optimization variables through:
  // Eigen::VectorXd x = nlp.GetVariableValues()
  // However, it's more convenient to access the splines constructed from these
  // variables and query their values at specific times:
  using namespace std;
  cout.precision(2);
  nlp.PrintCurrent(); // view variable-set, constraint violations, indices,...
  cout << fixed;
  cout << "\n====================\nMonoped trajectory:\n====================\n";

  double t = 0.0;
  while (t<=solution.base_linear_->GetTotalTime() + 1e-5) {
    cout << "t=" << t << "\n";
    cout << "Base linear position x,y,z:   \t";
    cout << solution.base_linear_->GetPoint(t).p().transpose() << "\t[m]" << endl;

    cout << "Base Euler roll, pitch, yaw:  \t";
    Eigen::Vector3d rad = solution.base_angular_->GetPoint(t).p();
    cout << (rad/M_PI*180).transpose() << "\t[deg]" << endl;

    cout << "Foot position x,y,z:          \t";
    cout << solution.ee_motion_.at(0)->GetPoint(t).p().transpose() << "\t[m]" << endl;

    cout << "Contact force x,y,z:          \t";
    cout << solution.ee_force_.at(0)->GetPoint(t).p().transpose() << "\t[N]" << endl;

    bool contact = solution.phase_durations_.at(0)->IsContactPhase(t);
    std::string foot_in_contact = contact? "yes" : "no";
    cout << "Foot in contact:              \t" + foot_in_contact << endl;

    cout << endl;

    t += 0.2;
  }
}

Ah, I see, so it seems I've set up the problem 'correctly' (in this case, a Biped). This is the output of the above file:
``******************************************************************************
This program contains Ipopt, a library for large-scale nonlinear optimization.
 Ipopt is released as open source code under the Eclipse Public License (EPL).
         For more information visit http://projects.coin-or.org/Ipopt
******************************************************************************

Exception of type: IpoptException in file "Unknown File" at line -1:
 Exception message: Unknown Exception caught in Ipopt
ERROR: Ipopt failed to find a solution. Return Code: -101

************************************************************
    IFOPT - Interface to Nonlinear Optimizers (v2.0)
                © Alexander W. Winkler
           https://github.com/ethz-adrl/ifopt
************************************************************
Legend:
c - number of variables, constraints or cost terms
i - indices of this set in overall problem
v - number of [violated variable- or constraint-bounds] or [cost term value]

                                   c             i             v
variable-sets:
   base-lin                       294        0....293           0
   base-ang                       294      294....587           0
   ee-motion_0                     69      588....656           0
   ee-motion_1                     69      657....725           0
   ee-force_0                     102      726....827           0
   ee-force_1                     102      828....929           0
   ee-schedule0                    15      930....944           0
   ee-schedule1                    15      945....959           0

constraint-sets:
   terrain-ee-motion_0             24        0.....23           0
   terrain-ee-motion_1             24       24.....47           0
   dynamic                        300       48....347          57
   splineacc-base-lin             141      348....488           4
   splineacc-base-ang             141      489....629           0
   rangeofmotion-0                186      630....815          69
   rangeofmotion-1                186      816...1001          69
   force-ee-force_0                85     1002...1086           0
   force-ee-force_1                85     1087...1171           0
terminate called after throwing an instance of 'std::out_of_range'
  what():  vector::_M_range_check: __n (which is<PHONE_NUMBER>3709551615) >= this->size() (which is 25)
Aborted (core dumped)

hi @Neotriple we are trying to implement same for quadruped. We are getting same VECRO::_M_range_check error. can you share if you are able to achieve anything on this.

hi @bsprime . To be honest, this was a long time ago, so I don't remember exactly what I did; but I think as @awinkler mentioned, this was due to indexing something that hadn't been initialized. Perhaps an initialization feature has been implemented since then (to be automatically initiated before running the optimization).

hi @bsprime . To be honest, this was a long time ago, so I don't remember exactly what I did; but I think as @awinkler mentioned, this was due to indexing something that hadn't been initialized. Perhaps an initialization feature has been implemented since then (to be automatically initiated before running the optimization).

Yeah, I understood that I have been trying to initialize and I had no success.
`std::vector<Eigen::Matrix<double,3,1> > vec;
for (int j = 0; j < 4; j++) {
vec.push_back(Eigen::Vector3d::Zero());
    }

formulation.initial_ee_W_.push_back(vec);
`
I have been trying this and it is giving some error like
no known conversion for argument 1 from ‘std::vector<Eigen::Matrix<double, 3, 1> > to ‘std::vector<Eigen::Matrix<double, 3, 1> >::value_type&& {aka Eigen::Matrix<double, 3, 1>&&}’
I feel there must be some data type error.


Create a gravatar for Docker Hub avatar

Motivation
If you open up kitematic all the other big images have an avatar. We have a nice logo, we should add it to Docker Hub too!
Scope of change
We need a wordpress account to make a gravatar. After the gravatar is made, open up https://hub.docker.com/u/vinyldns/dashboard/, go to settings, and add our gravatar there

Done!


x/tools/gopls: InvalidIfaceAssign wrong type for method, have H() H want H() H

gopls version

gopls -v version
Build info
----------
golang.org/x/tools/gopls v0.8.4
<EMAIL_ADDRESS>h1:zGZsAXAb0LLws/Z+2BCWR17dkPHhIO2GYwYSSkeXX5c=
<EMAIL_ADDRESS>h1:dtDWrepsVPfW9H/4y7dDgFc2MBUSeJhlaDtK13CxFlU=
<EMAIL_ADDRESS>h1:81/ik6ipDQS2aGcBfIN5dHDB36BwrStyeAQquSYCV4o=
<EMAIL_ADDRESS>h1:we8PVUC3FE2uYfodKH/nBHMSetSfHDR6scGdBi+erh0=
<EMAIL_ADDRESS>h1:qyrTQ++p1afMkO4DPEeLGq/3oTsdlvdH4vqZUBWzUKM=
<EMAIL_ADDRESS>h1:6zppjxzCulZykYSLyVDYbneBfbaBIQPYMevg0bEwv2s=
<EMAIL_ADDRESS>h1:5KslGYwFpkhGh+Q16bwMP3cOontH8FOep7tGV86Y7SQ=
<EMAIL_ADDRESS>h1:rm+CHSpPEEW2IsXUib1ThaHIjuBVZjxNgSKmBLFfD4c=
<EMAIL_ADDRESS>h1:olpwvP2KacW1ZWvsR7uQhoyTYvKAupfQrRGBFM352Gk=
<EMAIL_ADDRESS>h1:CJwk4qG1fov4WP7/DWhhb7OQVZlQKAl1rEMnDF+ceGU=
<EMAIL_ADDRESS>h1:jokAF1mfylAi1iTQx7C44B7vyXUcSEMw8eDv0PzNu8s=
<EMAIL_ADDRESS>h1:2LdYUZ7CIxnYgskbUZfY7FPggmqnh6shBqfWa8Tn3XU=
<EMAIL_ADDRESS>h1:kTojdZo9AcEYbQYhGuLf/zszYthRdhDNDUi2JKTxas4=
<EMAIL_ADDRESS>h1:tzxjVAj+wSBmDcF6zBB7/myTy3gX9xvi8Tyr28AuQgc=
go: go1.18.2

go env

 go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/tiger/Library/Caches/go-build"
GOENV="/Users/tiger/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/tiger/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/tiger/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/tiger/git/udistribution/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/62/wkysd_4n0w57ljl9ycfsd9cc0000gn/T/go-build690636367=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Open this file in VSCode
https://github.com/kaovilai/udistribution/blob/aa22efb91d74e7412c437eb618cc02f4ad46f28a/pkg/client/client_test.go after checking out the repo

What did you expect to see?
No problem reported, go test ./... run no issues.
What did you see instead?
go test ./... run no issues.
The rr variable from
gotClient.GetApp().ServeHTTP(rr, rq)

from https://github.com/kaovilai/udistribution/blob/aa22efb91d74e7412c437eb618cc02f4ad46f28a/pkg/client/client_test.go#L83
has an error coming from gopls. This is the message.
[{
	"resource": "/Users/tiger/git/udistribution/pkg/client/client_test.go",
	"owner": "_generated_diagnostic_collection_name_#5",
	"code": {
		"value": "InvalidIfaceAssign",
		"target": {
			"$mid": 1,
			"external": "https://pkg.go.dev/golang.org/x/tools/internal/typesinternal?utm_source%3Dgopls#InvalidIfaceAssign",
			"path": "/golang.org/x/tools/internal/typesinternal",
			"scheme": "https",
			"authority": "pkg.go.dev",
			"query": "utm_source=gopls",
			"fragment": "InvalidIfaceAssign"
		}
	},
	"severity": 8,
	"message": "cannot use rr (variable of type *httptest.ResponseRecorder) as http.ResponseWriter value in argument to gotClient.GetApp().ServeHTTP: *httptest.ResponseRecorder does not implement http.ResponseWriter (wrong type for method Header)\n\t\thave Header() http.Header\n\t\twant Header() http.Header",
	"source": "compiler",
	"startLineNumber": 83,
	"startColumn": 33,
	"endLineNumber": 83,
	"endColumn": 35
}]

Editor and settings

    "go.toolsManagement.autoUpdate": true,
    "go.executable.path": "/usr/local/go/bin/go",
    "go.delveConfig": {
        "dlvLoadConfig": {
            "followPointers": true,
            "maxVariableRecurse": 1,
            "maxStringLen": 64,
            "maxArrayValues": 64,
            "maxStructFields": -1
        },
        "apiVersion": 2,
        "showGlobalVariables": false,
        "debugAdapter": "legacy",
        "substitutePath": []
    },

Logs

Thank you for filing this issue. I tried and was not able to reproduce. A couple questions that may help me repro:

Are you using vscode's "open file" feature, or are you opening a directory? If the latter, which directory are you opening?
Does this repro every time for you?
Are you using a go.work file for this workspace?

Ok I made a new dir and tried my repro steps. Did not appear to work. The same dir also no longer have the issue after reopening.. Last time I reopened the issue did not go away.
Must be some lingering state from somewhere. But I have seen the pattern of InvalidIfaceAssign wrong type for method, have H() H want H() H more than once in VSCode in different projects.
Closing for now until I can find a consistent reproducible steps.

I'd love to understand how your gopls got into this state.
I could see this happening if you'd updated your go command (say to 1.18.2), and we somehow didn't propagate invalidations of standard library types. Is it possible that this was from an old session that was started before you updated Go?

Could be. Don't have any solid proof. :/

Ok got another one @findleyr
I think this is right around after running go mod tidy, adding some new imports to package, and typing a few lines.

oops nvm.. too quick didn't read..


Setting for advanced logging

@micheljung is that possible. Pretty sure it is, not sure how tho, but I wanna keep the idea. So we should make a setting to turn on advanced logging, so that everything is logged including debug, I would say so people can turn that on if they have an issue and aditionally information is needed.

This is possible. Spring Boot Actuator does this  and you can also do it directly with logback

Ok what we want is setting the log level on runtime in the settings.

How about setting this toggle under Settings > General > below the "Write ICE Log File"

Yes I guess would be good

I got the toggle working to switch from default (INFO) to DEBUG, then from DEBUG to INFO.
Should debug log stay enabled after closing the client, or should it revert to default log configuration on a client restart?

Very interesting are also the settings for the remote package. That is where server messages get logged. I am not sure if you only turn the debug log on for all packages where it is not set separately or set it for all packages there are. But maybe turning it on for all packages there are would be the correct thing to do. Cause otherwise all the interesting stuff is not logged

I've tested the logging for the remote package, also created the installer to verify behaviour.
Changing the log level of a package does not propagate the change in log level parent package to child packages as I expected it to. I had to set log level on each logger individually instead.


Fix user-data for ec2 metadata source

The commit change the endpoint url of the ec2 metadata service for the user-data. By default the endpoint is reachable at "http://<IP_ADDRESS>/latest/user-data" without the slash "/" at the end.
For this reason see the ec2 metadata service documentation: http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-instance-metadata.html
It is also fixed in the coreOS repository, the source of this file:
https://github.com/coreos/coreos-cloudinit/blob/master/datasource/metadata/ec2/metadata.go

There is no impact, both ways are no problem:
rancher@ip-172-31-26-84:~$ curl http://<IP_ADDRESS>/latest/user-data
test from zhiborancher@ip-172-31-26-84:~$

rancher@ip-172-31-26-84:~$ curl http://<IP_ADDRESS>/latest/user-data/
test from zhiborancher@ip-172-31-26-84:~$

Hello,
for sure it works on AWS but I tested it on Open Telekom Cloud (OTC) which use the ec2 metadata service too. And RancherOS can't retrieve user-data from the URL http://<IP_ADDRESS>/latest/user-data/ because it is not the right endpoint and deviates from the reference as I had described.
Second the change to this endpoint was not in the original repository of CoreOS where the source code come from.
It is possible to include the change to promote the the spread of RancherOS to other platforms like OTC?

Ok, I can merge this PR.

It is possible to include the change to promote the the spread of RancherOS to other platforms like OTC?

Sorry, there is not much time to focus on other cloud platforms.


only send the link and not the url , i already send the pic , but today , when i put the link in a chat that only put a link... and not a impage

only send the link and not the url , i already send the pic , but today , when i put the link in a chat that only put a link... and not a impage , i reply to my issue when i ofund the issue :)

what language are u speaking in 💀


Documentation: Update default filter_ex shown in documentation for derive_var_trtsdtm / derive_var_trtedtm

Please select a category the issue is focused on?
Function Documentation
Let us know where something needs a refresh or put your idea here!
https://pharmaverse.github.io/admiral/reference/derive_var_trtsdtm.html - at top says:
filter_ex = (EXDOSE > 0 | (EXDOSE == 0 & str_detect(EXTRT, "PLACEBO"))) &     nchar(EXSTDTC) >= 10,
But instruction in the Arguments section below says:
Default: EXDOSE > 0 | (EXDOSE == 0 & str_detect(EXTRT, 'PLACEBO')
And the code does include: & nchar(EXSTDTC) >= 10, so this should be included in the Arguments default shown in the documentation.
Same issue as above exists for https://pharmaverse.github.io/admiral/reference/derive_var_trtedtm.html

Have updated the documentation on derive_var_trtsdtm.R and pushed it to the 1017 Update branch. Will message somebody about it tomorrow morning and confirm whether everything looks okay and can go from there! If I have done everything alright happy to make similar change for derive_var_trtedtm.R!

Hi Jack.  You can just do a Pull Request and that will alert folks to any issues in the code.  You can assign me as a reviewer.

Great. I will do that tomorrow morning, thanks Ben.

From: Ben Straub @.>
Sent: Monday, March 21, 2022 6:09:40 PM
To: pharmaverse/admiral @.>
Cc: Jack McGavigan @.>; Assign @.>
Subject: Re: [pharmaverse/admiral] Documentation: Update default filter_ex shown in documentation for derive_var_trtsdtm / derive_var_trtedtm (Issue #1017)
EXTERNAL
make sure to PR into devel
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/pharmaverse/admiral/issues/1017*issuecomment-1074244813__;Iw!!AoaiBx6H!k70vAN1mTcO643norlXfFNz1pKlDaz_h9Iviq4eWEsSDmiHqM-SB8mT4fPX7cbDciUfM$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AYIYJOCZLR67FIFA3GH2NTDVBC3OJANCNFSM5QZFCT2Q__;!!AoaiBx6H!k70vAN1mTcO643norlXfFNz1pKlDaz_h9Iviq4eWEsSDmiHqM-SB8mT4fPX7cZjZ9HVr$.
You are receiving this because you were assigned.Message ID: @.***>
GSK monitors email communications sent to and from GSK in order to protect GSK, our employees, customers, suppliers and business partners, from cyber threats and loss of GSK Information. GSK monitoring is conducted with appropriate confidentiality controls and in accordance with local laws and after appropriate consultation.

This e-mail was sent by GlaxoSmithKline Services Unlimited
(registered in England and Wales No. 1047315), which is a
member of the GlaxoSmithKline group of companies. The
registered address of GlaxoSmithKline Services Unlimited
is 980 Great West Road, Brentford, Middlesex TW8 9GS.

Thanks @JMcGavigan32 for your contribution! Just a heads up that we usually have convention to keep the issues open until the update gets merged into main for a release - this way users are less likely to re-raise issues already fixed in devel. You can add "closes then # and issue number" in your PR and this gets done automatically when merged with main later.

Hi Jack,
The Issues will automatically close once we pull the dev branch into main - fingers crossed.  No need to close them yourself.  :)

great minds @bms63 ...

This process better work or I am going to riot!

:D "...and breathe"


Fix django cache key cleanup, closes #32

Actually no, don't merge that. It won't work well. Cache keys are prefixed, I'll reopen a new PR.

No this is fine, the regex matches things inside the key, not the whole key

Yeah but cache keys like django.contrib.sessions.cacheabcdefghijklmnopqrstuvwxyz01zz45 will be mangled to django.contrib.sessions.#zz#, which will keep a variable part...

Rewrote the fix, should be fine now.

Oh I see, I thought you mean the cache backend prefix, not the session backend's extra prefix. It is odd that there isn't even a separator of some kind between session backend key prefix, it's probably just an oversight.

Merged in 541f5a3cad04b7aa127e39c4f5053a70b3a0f26c, thanks! I added an improvement in 0ead0d068776f3ece2457007f6709310766395fc to use regex lookbehind to check for the prefixes, so we can leave the it in the 'fingerprinted' version so it's easier to understand from the .perf.yml file where the cache operations are coming from 😄