Error when convert tfrecords data to npz data

Hi, thanks for the good work. When I downloaded tfrecords data from PlaneNet,  and then ran the commands to convert it to npz data and generate json files required by detectron2,  I met the following error. How can I solve it? Thanks!

...
...
18899
18999
19099
19199
19299
Traceback (most recent call last):
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1365, in _do_call
return fn(*args)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1350, in _run_fn
target_list, run_metadata)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1443, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.OutOfRangeError: FIFOQueue '_2_batch/fifo_queue' is closed and has insufficient elements (requested 1, current size 0)
[[{{node batch}}]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "data_tools/convert_tfrecords.py", line 59, in
img, gt_dict = sess.run([img_inp_train, global_gt_dict_train])
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 956, in run
run_metadata_ptr)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1180, in _run
feed_dict_tensor, options, run_metadata)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1359, in _do_run
run_metadata)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1384, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.OutOfRangeError: FIFOQueue '_2_batch/fifo_queue' is closed and has insufficient elements (requested 1, current size 0)
[[node batch (defined at /home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py:1748) ]]
Original stack trace for 'batch':
File "data_tools/convert_tfrecords.py", line 46, in
img_inp_train, global_gt_dict_train, local_gt_dict_train = reader_train.getBatch(filename_queue_train, batchSize=1, getLocal=True)
File "/home/zlq/code/PlaneRecTR/PlanarReconstruction/data_tools/RecordReaderAll.py", line 62, in getBatch
tf.train.batch([image, planes, depth, normal, semantics, segmentation, boundary, numPlanes, features['image_path'], features['info']], batch_size=batchSize, capacity=(NUM_THREADS + 2) * batchSize, num_threads=1)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/util/deprecation.py", line 324, in new_func
return func(*args, **kwargs)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/training/input.py", line 1020, in batch
name=name)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/training/input.py", line 789, in _batch
dequeued = queue.dequeue_many(batch_size, name=name)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/ops/data_flow_ops.py", line 489, in dequeue_many
self._queue_ref, n=n, component_types=self._dtypes, name=name)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/ops/gen_data_flow_ops.py", line 3862, in queue_dequeue_many_v2
timeout_ms=timeout_ms, name=name)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/framework/op_def_library.py", line 794, in _apply_op_helper
op_def=op_def)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/util/deprecation.py", line 507, in new_func
return func(*args, **kwargs)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py", line 3357, in create_op
attrs, op_def, compute_device)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py", line 3426, in _create_op_internal
op_def=op_def)
File "/home/zlq/anaconda3/envs/planerectr/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py", line 1748, in init
self._traceback = tf_stack.extract_stack()

Hi, would you mind to directly share the file 'scannetv1_plane_len47693_train.json'? Thanks!

Hi, scannetv1 json files were uploaded to https://drive.google.com/drive/folders/1ldpL1vs1YPCi-OvyRmSA4D-nFAwCJV6E?usp=sharing

Hi, scannetv1 json files were uploaded to https://drive.google.com/drive/folders/1ldpL1vs1YPCi-OvyRmSA4D-nFAwCJV6E?usp=sharing

Thanks. I only generated 18,467 training samples and 760 validation samples. I believe the validation set matches your configuration. Could you please share the entire 'scannetv1_plane' folder (as shown below)? Because I am still unable to run the commands successfully.

Sorry for the late reply. Due to the large size of the file, it took some time and effort to upload it. You can download it from here.

Thank you very much!


2218 BugFix: Queued Transactions not showing up

Fixes #2218
This PR replaces the getAllTransaction API request with getMultisigTransactions request. Most of the properties are the same except for the transfers property.
There are two spots where this is changes somethings.
Dashboard:
Will no longer show Treasury activities. These would be either a Card itself or added text to the bottom of the description that would display any transfer information.
Treasury:
Was able to use a combination of getIncomingTransactions and getToken from Safe's api-kit to retrieve the same data and put Treasury back together.
Testing

Ensure that Multisig queued Proposals show up
Ensure Transactions (Transfers) show up on Treasury Page (any safe)

@Da-Colon - see test results below
Ensure that Multisig queued Proposals show up ✅
Ensure Transactions (Transfers) show up on Treasury Page (any safe) - ❌
Detail --> Ensure that Multisig queued Proposals show up ✅

Proposals seems to be appearing that are pending. I tested x2 types
Link here -

Detail --> Ensure Transactions (Transfers) show up on Treasury Page (any safe) - ❌

@Da-Colon here's the issue as seem to be missing some txs

link - here

For app.dev I get this

For this PR I get this

@Da-Colon - see test results below
Ensure that Multisig queued Proposals show up ✅ Ensure Transactions (Transfers) show up on Treasury Page (any safe) - ❌
Detail --> Ensure that Multisig queued Proposals show up ✅

Proposals seems to be appearing that are pending. I tested x2 types
Link here -

**Detail** --> Ensure Transactions (Transfers) show up on Treasury Page (any safe) - ❌

@Da-Colon here's the issue as seem to be missing some txs
link - here
For app.dev I get this

* For this PR I get this

Yeah works on both the Prod and dev preview links for me?

@Da-Colon look closer at Tom's screenshot for "on app.dev i get this"
it includes a transaction at the top for incoming 425 USDC from 4 days ago. that TX is not on this PR's deploy preview (according to Tom)

@Da-Colon look closer at Tom's screenshot for "on app.dev i get this"
it includes a transaction at the top for incoming 425 USDC from 4 days ago. that TX is not on this PR's deploy preview (according to Tom)

I see it now. Sorted and Fixed

@Da-Colon just gave my approval, however i would like to see all open conversations in this PR resolved before you merge. thank you!


Walk through for creating a project

Walk through for creating and submitting a j unit tests should exist

@amrdraz for the api or the web interface?

never mind this I will do it,
it's for the front-end
Amr Draz
Teaching Assistant German University in Cairo (GUC)
Mobile: (+2)<PHONE_NUMBER>2)   |   *LinkedIn: *[Profile
http://eg.linkedin.com/in/amrdraz] | *Skype: * (bing91)
On Wed, Jan 28, 2015 at 12:58 PM, Ahmed Hisham Ismail <
<EMAIL_ADDRESS>wrote:

@amrdraz https://github.com/amrdraz for the api or the web interface?
—
Reply to this email directly or view it on GitHub
https://github.com/amrdraz/java-project-runner/issues/30#issuecomment-71815722
.


Functionality to add Node agent on new physical hosts in an existing HCI cluster

Title: Functionality to add  MOC Node/ cloud agent on new physical hosts in an existing AS HCI cluster
Description:
In situation where a physical host is added / re-installed to a AS HCI cluster where AKS is deployed, the MOC NodeAgent Service / WSSD Cloud Agent Service should be available on that host so that the ca generic service role can run on that host.
There is no clear documentation or process on this particular scenario. It appears that the only way is to reinstall AKS on the cluster, which for production clusters is not desirable.

The Add-AksHciNode command should do this, but it is not clear from the CmdLet help.  It would help to document the scenario.

Add-AKSHciNode should do this.


Extract the Generator from the WHEEL file as metadata

What's the problem this feature will solve?
Allow easy discovery of what built the artifacts and at what version
Describe the solution you'd like

Section on the side showing the first (multiple wheels possible) parsed build backend, or the full set
Add the backend to wheel JSON API response

Additional context
https://discuss.python.org/t/improving-discoverability-of-build-backends/20140/3

Thanks for the feature request. Could you add more detail about what the use case is here?
Would something like https://inspector.pypi.io/ suit this use case? e.g. https://inspector.pypi.io/project/hatch/1.6.3/packages/59/ab/44150b2177678139e5d6f5bf922adbd9b3f39c4cdbf7091fef1c63ebd47d/hatch-1.6.3-py3-none-any.whl/hatch-1.6.3.dist-info/WHEEL#line.2
Why does it need to be included in a machine-readable API?

It's possible I'm confused as to the relationship of this request to the build backend classifier discussion, but is it assumed that the wheel generator reflects the PEP-517 build backend used? According to the PEP, wheel generation is the domain of the build frontend, which in at least some cases will be an entirely separate program from the backend (and is the case for a backend I maintain, which relies on the wheel library to create wheels from the sdists that backend builds, so its wheels always have a generator of bdist_wheel (some version) in them which it effectively can't override as that's hard-coded into wheel itself).

Could you add more detail about what the use case is here? [...] Why does it need to be included in a machine-readable API?

Tracking usage and upgrade patterns
Ability to tell if a release was built by a version that had a bug or CVE that affects the contents

According to the PEP, wheel generation is the domain of the build frontend

Which PEP? That's 100% not true, pip, build, etc. do not modify the wheel

is it assumed that the wheel generator reflects the PEP-517 build backend used? [...] in at least some cases will be an entirely separate program from the backend

It should imo as that is what every backend but setuptools does (or users of wheel)

Apologies, you're right I misinterpreted the terminology section of PEP 517. Just to be clear, though, I maintain a build backend which creates sdists and then relies on the wheel library to create wheels from those, so the wheel generator metadata won't reflect the name of the the build backend for projects relying on it (and as you point out, the same goes for projects using SetupTools, which isn't exactly a negligible subset).

https://github.com/pypa/wheel/issues/479


spec, python: small issues fixed

Some fixes I didn't want to lose prior to moving to collections.

Fixing small typos in "ovirt-ansible-disaster-recovery.spec.in".
Minor code style improvements in "generate_mapping.py".

Signed-off-by: Pavel Bar<EMAIL_ADDRESS>

Verified on October 8th, 2020


Port to 1.21.1

this mod is the coolest thing i've ever seen, may u port it for 1.21.1? ty

Patiently waiting, also please make it compatible with the new sodium beta versions.

Done

Pls port it to neoforge too,or to make easier the job,mark it as compatible with neoforge and put as required dependency connector and ffapi


Relationships context in resolvers

The relationships context is available to interceptors via the environment. But the environment is not available to resolvers. Update the resolver interface so that the relationships-context is available along with the instance on which CRUD is being performed.

fixed here: https://github.com/fractl-io/fractl/pull/1230


Docs feedback: Add callout that user need to configure DD_APM_ADDITIONAL_ENDPOINTS to forward traces

URL
https://grafana.com/docs/alloy/latest/collect/datadog-traces-metrics/
Feedback
Issue
otelcol.receiver.datadog support receiving both metrics and traces.
However, Current Alloy doc describing details on receiving metrics and traces from Datadog agent only include configuration required for Datadog metrics.
Suggestion
Following Datadog documentation on dual shipping, users need to specify DD_APM_ADDITIONAL_ENDPOINTS environment variable in order to forward APM traces data.
Therefore, recommend to call this out under the current Alloy doc section

Heya @ptodev , yes the format is the same as DD_ADDITIONAL_ENDPOINTS examples.


⚠ Spryr has degraded performance

In 1fea161, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4406 ms

Resolved: Spryr performance has improved in d0686b6 after 15 minutes.


cli ng build --prod still not compiling when noUnusedParameters activated after upgrading to material 2.0.0-beta.7

Bug, feature request, or proposal:
After upgrading Material to 2.0.0-beta.7 ng build --prod still not compiling when noUnusedParameters: true in tsconfig.json.
What is the expected behavior?
It should compile without any errors in production mode with noUnusedParameters: true in tsconfig.json.
What is the current behavior?
Giving multiple errors like this:

ERROR in<EMAIL_ADDRESS>(2,1078): '$event' is declared but never used.

What are the steps to reproduce?
Just compile using Angular CLI with production mode using command ng build --prod.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"@angular/animations": "~4.2.3",
"@angular/common": "^4.2.3",
"@angular/compiler": "^4.2.3",
"@angular/core": "^4.2.3",
"@angular/forms": "^4.2.3",
"@angular/http": "^4.2.3",
"@angular/platform-browser": "^4.2.3",
"@angular/platform-browser-dynamic": "^4.2.3",
"@angular/router": "^4.2.3",
"@angular/material": "^2.0.0-beta.7",
...
"@angular/cli": "1.1.2",
"typescript": "~2.4.0"
Is there anything else we should know?
tsconfig.json file:
{
"compileOnSave": false,
"compilerOptions": {
"alwaysStrict": true,
"baseUrl": "src",
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"forceConsistentCasingInFileNames": true,
"lib": [
"es2016",
"dom"
],
"moduleResolution": "node",
"noUnusedParameters": true,
"outDir": "./dist/out-tsc",
"skipLibCheck": true,
"sourceMap": true,
"target": "es5",
"typeRoots": [
"node_modules/@types"
]
},
"exclude": [
"dist",
"node_modules",
"**/*.spec.ts"
]
}

When run
ng serve --environment=prod --target=production
error is the same

@AliShd It might related to typescript version, how about roll back to v2.3 and try?

No, not even working with typescript 2.3.4. Getting the same errors.

Yeah this is an issue of Angular itself at the moment. See https://github.com/angular/angular/issues/17131
Once that issue is solved I will update our AOT CI task to also check with noUnusedParameters (so we can see those issues as early as possible)


Exception in IntelliSense

This is from the ActivityLog.xml file that the error window pops up.
  <entry>
    <record>2910</record>
    <time>2017/10/20 21:29:53.987</time>
    <type>Error</type>
    <source>Editor or Editor Extension</source>
    <description>
    System.ArgumentException: Failed to get classifier from buffer

      at Microsoft.PythonTools.Intellisense.ReverseExpressionParser..ctor(ITextSnapshot snapshot, ITextBuffer buffer, ITrackingSpan span)

      at Microsoft.PythonTools.Language.TextViewFilter.GetDataTipText(TextSpan[] pSpan, String& pbstrText)

      at Microsoft.VisualStudio.Editor.Implementation.ShimQuickInfoSource.TryGetQuickInfoFromFilter(IQuickInfoSession session, TextSpan[] dataBufferTextSpan, String& tipText)

      at Microsoft.VisualStudio.Editor.Implementation.ShimQuickInfoSource.AugmentQuickInfoSession(IQuickInfoSession session, IList`1 qiContent, ITrackingSpan& applicableToSpan)

      at Microsoft.VisualStudio.Language.Intellisense.Implementation.QuickInfoSession.Recalculate()

      at Microsoft.VisualStudio.Language.Intellisense.Implementation.QuickInfoSession.Start()

      at Microsoft.VisualStudio.Language.Intellisense.Implementation.DefaultQuickInfoController.OnTextView_MouseHover(Object sender, MouseHoverEventArgs e)

      at Microsoft.VisualStudio.Text.Editor.Implementation.WpfTextView.RaiseHoverEvents()

      --- End of stack trace from previous location where exception was thrown ---

      at Microsoft.VisualStudio.Telemetry.WindowsErrorReporting.WatsonReport.GetClrWatsonExceptionInfo(Exception exceptionObject)</description>
  </entry>

This happened when I was hovering my mouse over the edit window while I was refreshing the intellisense database.  This is in Update 4, using a 3.6 x64 interpreter.  I can't get it to happen reliably, but if I can find a more consistent repro I will update this issue.  Also, if there's anything else I can do to get you something more useful than just this stack trace, happy to help.

This is plenty, but as it happens it's already fixed because I recently removed this part of the code entirely :)
You won't see the new version until we start releasing previews of 15.6.

When are there previews available? I am getting this error at everytime if I start to develop a python file.
After that nothing works. No autocomplete no intelisense.

@tjenssen The preview just went out today :) Get it from https://www.visualstudio.com/vs/preview/ or your existing install will prompt you to update shortly.


Add loss="auto" as the default loss

@stsievert  a parallel proposal to #208
This implements a default loss "auto" for KerasClassifier and KerasRegressor. An appropriate loss function is only selected if:

The user did not provide a loss function (i.e. the default "auto" was passed).
The user did not compile the model.

For KerasRegressor, it always defaults to "mse" and supports any number of outputs.
For KerasClassifier, it defaults to "binary_crossentropy" for binary targets, "sparse_categorical_crossentropy" for "multiclass" targets. Only single outputs are supported. An error is raised if there is >1 output or if a different task type not listed above is passed (eg: multilabel-indicator). An error is raised if a multiclass problem is paired with a single output neuron.
TODO:

[x] docs

Codecov Report

Merging #210 (885e2a5) into master (d1a9d12) will decrease coverage by 0.12%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage   99.71%   99.58%   -0.13%
==========================================
  Files           6        6
  Lines         693      727      +34
==========================================
+ Hits          691      724      +33
- Misses          2        3       +1

Impacted Files
Coverage Δ

scikeras/wrappers.py
99.24% <94.59%> (-0.22%)
:arrow_down:

scikeras/utils/__init__.py
100.00% <100.00%> (ø)

scikeras/utils/transformers.py
100.00% <100.00%> (ø)

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 d1a9d12...885e2a5. Read the comment docs.

For deployable things, like a containerized servicie, it makes builds
deterministic and faster.
For libraries, the use is more questionable.
I think the Poetry authors can probably offer better responses than I can:
https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
I think they've changed their wording: I think it used to say libraries MAY
not commit it, but now says SHOULD not commit it.
In any case, we should remove it.
On Thu, Mar 4, 2021, 8:35 PM Scott Sievert<EMAIL_ADDRESS>wrote:

@stsievert commented on this pull request.
In poetry.lock
https://github.com/adriangb/scikeras/pull/210#discussion_r587982693:

@@ -309,21 +309,22 @@ six = ">=1.9.0"

[package.extras]
aiohttp = ["aiohttp (>=3.6.2,<4.0.0dev)"]
+pyopenssl = ["pyopenssl (>=20.0.0)"]
What use does it provide?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adriangb/scikeras/pull/210#discussion_r587982693, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMPP5YBH5FSNR7TZZAJ7TTCA7PNANCNFSM4YSWOQ2Q
.

@stsievert do you think we should move forward with this PR?

Codecov Report

Merging #210 (dde0112) into master (1fa9341) will decrease coverage by 0.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage   99.71%   99.45%   -0.26%
==========================================
  Files           6        6
  Lines         693      734      +41
==========================================
+ Hits          691      730      +39
- Misses          2        4       +2

Impacted Files
Coverage Δ

scikeras/utils/__init__.py
100.00% <100.00%> (ø)

scikeras/utils/transformers.py
100.00% <100.00%> (ø)

scikeras/wrappers.py
99.75% <100.00%> (+0.29%)
:arrow_up:

scikeras/_saving_utils.py
96.55% <0.00%> (-3.45%)
:arrow_down:

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 1fa9341...dde0112. Read the comment docs.


Improve Lightify Component

I would like to improve the lightify component in the near future. However I can not guarantee success or enough time, to make it finally work. Here are my thoughts on what needs to be done.

Add device type awareness to python-lightify
Create a lightify bridge component like the hue component handling:

auto discovery
lights
plug/power sockets
motion sensors
groups
possibly fix this annoying error, where lightify picks up the color of a light wrongly.
switches (the tcp api doesn't seem to allow this)

Anything else that needs to be added?

So, I figured that the lightify bridge is just not a good bridge. It e.g. doesn't allow for switches to send events to HA, one cant configure lights to report there current state ... (well maybe on can but we don't know)
The protocol is only available due to passionate individuals reverse engineering the tcp connection https://github.com/noctarius/lightify-binary-protocol and other people building python libraries and stuff on top.
A good thing I can say about the bridge is that osram is currently not planing on removing the tcp connection, but who knows, we have seen things like this happen.
I therefore switched to zigbee2mqtt and started to contribute to the hassio addon. I will not pick up on this project any more as it is a waste of time for me.


How to run EasyRepro Test on remote server (Selenium Grid)

Hi,
I have created the project using Easyrepo for D365 and my project is ready I'm able to run it locally but I want to run it on a remote server like on selenium Grid
I have  given the have given url of my selenium hub where I have to run the test in app.config below is code:-

This is my testsettings class: -
namespace Microsoft.Dynamics365.UIAutomation.Sample
{
public static class TestSettings
{
public static string InvalidAccountLogicalName = "accounts";
    public static string LookupField = "primarycontactid";
    public static string LookupName = "Rene Valdes (sample)";
    private static readonly string Type = System.Configuration.ConfigurationManager.AppSettings["BrowserType"].ToString();
    private static readonly string RemoteType = System.Configuration.ConfigurationManager.AppSettings["RemoteBrowserType"].ToString();
    private static readonly string RemoteHubServerURL = System.Configuration.ConfigurationManager.AppSettings["RemoteHubServer"].ToString();

    private static BrowserOptions options = new BrowserOptions
    {
        BrowserType = (BrowserType)Enum.Parse(typeof(BrowserType), Type),
        PrivateMode = true,
        FireEvents = false,
        Headless = false,
        UserAgent = false,
        //DefaultThinkTime = 2000,
        RemoteBrowserType = (BrowserType)Enum.Parse(typeof(BrowserType), RemoteType),
        RemoteHubServer = new Uri(RemoteHubServerURL)

    };

    public static BrowserOptions Options { get => options; set => options = value; }

This is my base class : -
protected readonly Uri _xrmUri = new Uri(System.Configuration.ConfigurationManager.AppSettings["OnlineCrmUrl2"].ToString());
protected readonly Uri baseURL = new Uri(System.Configuration.ConfigurationManager.AppSettings["EmailAPiURL"].ToString());
protected readonly String _EmailAPIusername = System.Configuration.ConfigurationManager.AppSettings["EmailAPIUsername"].ToString();
protected readonly String _EmailAPIpassword = System.Configuration.ConfigurationManager.AppSettings["EmailAPIPassword"].ToString();
protected XrmApp xrmApp = new XrmApp(new WebClient(TestSettings.Options));
Can someone help me to run the test please ?

Hi @sanjotsaini
Is it possible to post your app config you use as well so people can see if you're not missing a property to run on the grid (like BrowserType / RemoteBrowserType / RemoteHubServer)
Thanks,

Change the value in app.conf file.:
BrowserType: in below config file is Chrome;

Change it like

And also change : your RemoteHubServer url.

<add key="BrowserType" value="Chrome" />
<add key="DriversPath" value="" />
<add key="UsePrivateMode" value="true" />
<!-- The following settings are only used if BrowserType = "Remote" above -->
<add key="RemoteBrowserType" value="Chrome" />
<add key="RemoteHubServer" value="http://<IP_ADDRESS>:4444/wd/hub" />


Expand edge case coverage with fuzz testing

Right now we do not have any fuzz testing. There are a lot of areas this could help us with. For example, we have had many bugs of pilot generating invalid config. We can probably catch many of these with fuzzing.
I have started some basic fuzzing and found issues, so it seems promising:

https://github.com/istio/istio/pull/14910
https://github.com/gogo/protobuf/issues/579

Ideally we would could have something that generates a bunch of random config (VS, SE, Gateways, etc) and feeds it to pilot to test generation of XDS responses. There are probably other areas where this is useful as well.
Most fuzzing is done by defining a function of bytes -> pass/fail, which makes this a bit hard. Envoy uses https://github.com/google/libprotobuf-mutator#integrating-with-libfuzzer which allows generating protobuf much easier, but we will need an alternative approach as I believe this only works for C.
https://github.com/dvyukov/go-fuzz seems to be the defacto standard for Go fuzzing and seems pretty solid. It looks like there is some work to integrate this with oss-fuzz at https://github.com/google/oss-fuzz/pull/2188 which would be ideal.
Areas to fuzz:

[ ] All CRD validation
[ ] All XDS response generation
[ ] ... TODO ...

Problems to solve:

[ ] How do we reliably translate raw bytes to many protos. https://github.com/dvyukov/go-fuzz/issues/251 has some tips on how not to do it at least.
[ ] Where do we run it? oss-fuzz is probably ideal if possible
[ ] Can we use libprotobuf-mutator if we use https://github.com/dvyukov/go-fuzz#libfuzzer-support?

Also, I explored https://github.com/google/gofuzz a bit. This is nowhere near as good as go-fuzz. go-fuzz is doing coverage guided fuzzing, so it will generate new inputs to try to gain more coverage. gofuzz is just generating random stuff and will never generate a valid config

Example of oss-fuzz (may have to login with google, but I think everyone has access): https://oss-fuzz.com/testcases?open=yes&project=envoy

Basically the scope of this task would be just getting the basics off fuzzing set up and running. Actually adding fuzz tests for various scenarios can be added once we have this

Another use-case for this: with fuzzed / generated resources, we can do performance testing at various scales. (e.g. https://github.com/istio/istio/issues/18008 is in the context of analysis).

could utilize https://github.com/munnerz/crd-schema-fuzz/blob/master/fuzz.go#L39

This would have caught the pruning bugs we say @jasonwzm

I am just chiming in here to let everyone in this thread know that 2 fuzzers have been added by way of go-fuzz which is supported by OSS-fuzz: https://github.com/istio/istio/pull/29295 and the integration with OSS-fuzz is set up here: https://github.com/google/oss-fuzz/pull/4762

Not stale

We have made a ton of progress here in tests/fuzz. We can continue to improve but I think we can consider this done.


Questions about the convergence

Hi, there~ Thanks to your kind guidance, I have successfully run ferminet program  and get some results about, for example, H4 circle. However the output energy for each iteration still has oscillating behavior roughly about 0.1 Eh even after 10^5 iterations.
So I just want to ask how can I know this network has been well trained?  I mean when can I stop updating the network parameters and use it to calculate energy accurately?
I note the default iteration in your program is 10^6, and I wonder whether this oscillating behavior will be cured at that time.

Is this in JAX or TF? Which optimizer - KFAC or ADAM? With KFAC (only available in TF) you'd expect it to be pretty well converged by 10^5. I'm less sure about the behavior with ADAM.

Please provide full details of your calculation configuration, including the batch size. It is impossible to know what variance to expect without this information. Have you used any Monte Carlo algorithms before? You will need to average the energy over multiple iterations (we typically use 10k-50k) to get a good energy estimate. During training, a rolling mean is also helpful to check convergence.
For H4 you can compare to FCI calculations (and extrapolate them to the complete basis set limit) to get a good ground truth estimate.

Myflag is "--system_type h4_circle --system_separation 3.2843 --system_angle 0.7853981633974483 --batch_size 1024 --iterations 100000 --multi_gpu" , and it's in Tf with Kfac optimizer.
I think my previous claim 0.1 Eh is wrong. The result for H4 is around about -2.013~-2.015Eh actually after 10^5 steps. But I think this accuracy is still not enough to determine the stable angle for H4 circle.

Ok, I think I know what should I do: using bigger batch_size, take the mean energy of the last several steps to judge the convergence. Then I can stop training and  average the energy over multiple iterations just like you adviced 10k-50k.

Fluctuations in the energy from a single batch of that range for H4 are entirely normal (c.f. Fig 3 in our paper). Averaging this over 10k-50k iterations (without parameter updates) will give an excellent estimate of the energy. Remember to use a method to remove the autocorrelation to give a good estimate of the standard error in the energy (e.g. blocking, as we did).
Even noisy estimates can give very good energies over a long period. Given a fixed function, the estimate from averaging the energy of a batch of size 1 over N steps is equivalent to averaging over a batch of size N over 1 step!


Update TGI to 1.4.0

Please also update the workflow file and README doc.

Please also update the workflow file and README doc.

modified in latest commit。

Don't forget the root README.md and the .github/workflows/text-generation-inference.yml file.

I'm very sorry, I force pushed a new commit, it mimicked commit 22542e7277312143802ac36384333e5793378a6c

Awesome, thanks! @zTaoplus


System.Runtime.InteropServices.ExternalException: A generic error occurred in GDI+

Actual Behaviour
Every once in a while, I get a popup message with the error from this issue's title. I have only installed Aurora yesterday, so it might be that I'm either doing something wrong or my Clevo laptop is not configured correctly, so that's why I'm opening this issue trying to seek help on what the problem might be.
2022-06-22 18.34.09.log
Reproduction steps and Frequency
Unfortunately, I have not yet figured out a way to reproduce the error. It seems random at the moment.
Aurora Version:
Latest stable (v54)
Previously an Issue?
Don't know, as only installed it yesterday evening and had the same error around 3 times since.

Looking at the log it seems this is related to Clevo controller code in Aurora. I will check it later today


unity update and code cleanup

update to unity 5.3.1.p3 assemblies
use new (unity 5.3.1.p1) API in Scene to get root game objects

Thanks - I'm just going to wait until this change to the Unity API is added to a non-patch release before merging this in.

5.3.2 here you go: http://unity3d.com/unity/whats-new/unity-5.3.2 ^^

This has been merged


LeaderNotAvailable disguised as UnknownTopicOrPartition

the problem
When running SimpleConsumer.subscribe in a LeaderNotAvailable scenario, a UnknownTopicOrPartition error is thrown:
KafkaError: This request is for a topic or partition that does not exist on this broker.

This can be reproduced (sometimes it works, sometimes it doesn't) running the code at https://github.com/Quadric/radiaction/tree/40d3433be9da803ab2c2207e51f4088bcb4ed069/examples/basic-example
It's important to have something done about it because such case is very hard to catch and debug. It took me days to find this error hidden inside SimpleConsumer.client.topicMetadata.  Keep in mind that it is never guaranteed that the error will be there next time you run your code, given the nature of a LeaderNotAvailable issue. That's how my topicMetadata looks sometimes (some other times it's just empty):
{
  "rick-morty__BUY_SAUCE": {
    "0": {
      "error": {
        "name": "KafkaError",
        "code": "LeaderNotAvailable",
        "message": "This error is thrown if we are in the middle of a leadership election and there is currently no leader for this partition and hence it is unavailable for writes."
      },
      "partitionId": 0,
      "leader": -1,
      "replicas": [],
      "isr": []
    }
  },
  ... // repeats for every topic

the solutions

a LeaderNotAvailable error should be thrown as result of a failing subscribe due to the lack of a leader
there needs to be a way to wait for a leader to be elected, and then be able to call subscribe again.

Topic:partition pairs that received LeaderNotAvailable error during subscribe will be retried to subscribe on each _fetch call. So thats exactly what you name as a second solution:

there needs to be a way to wait for a leader to be elected, and then be able to call subscribe again.


Don't warn when mixing border and borderRadius

border is not shorthand for borderRadius, don't warn about it.
"Radium: property "border" in style object Object {border: "none", borderRadius: "2px"} : do not mix longhand and shorthand properties in the same style object. See https://github.com/FormidableLabs/radium/issues/95 for more information."

We should replace the warning code with a more comprehensive version of React's shorthandPropertyExpansions: https://github.com/facebook/react/blob/2aeb8a2a6beb00617a4217f7f8284924fa2ad819/src/browser/ui/dom/CSSProperty.js#L70


Consider implementing Doppelgänger detection to help prevent slashing

For discussion.
Description
As a staker, I want my node to ensure that its validators are unique on the network when it starts so that I don't get slashed if another instance of the validator is running elsewhere.
One way this could be accomplished is to wait 2 (say) epochs before starting to attest and watch the network. If any attestations are seen from validators I believe I own, this should trigger the kill-switch.
It should be optional (default, but able to override?) as staking services are unlikely to want to miss epochs.
 Current folklore in the staking community is to wait at least two epochs after stopping a node before restarting; this mechanism would effectively automate that.

ref. #4453

Useful, Doppelganger Detection: Lessons Learned


ISSUE #1233: Feature refresh_database argument added to method find

Tickets
https://github.com/serengil/deepface/issues/1233
What has been done
With this PR, the method refresh_database (boolean) allows the user to choose between sync the pkl file with the database directory and keep it up to date with the files inside that directory or ignore any file changes and do the recognition only with the representations in the pkl file.
Also a logger.info saying that the user is possibly out of sync for better user experience;
How to test
make lint && make test

All resolved in the commit b4a35fe80b6e28e3d8b1880ef3822298af90b836;
Thanks for the feedback.

LGTM
PS: Pipeline failed because of linting but it mostly related to other files than this PR. I will handle it.

Great. Thanks for the work on this framework! It's been really useful to me.
Issue #1233 closed. :+1:


@zulip is treated as a UserMention

@zulip as a message is treated as a UserMention, which even capitalizes Zulip.

Are you seeing this on chat.zulip.org?  I think on that server there's some user who picked their full name as "Zulip".

Closed, as the user with the email<EMAIL_ADDRESS>registered under the name "Zulip".

Wait hold on, before closing. The key part is that I did @zulip without the ** around zulip. And it still treated it as a UserMention.


ng serve is broken after installing angular-switchery

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.2.0
node: 6.9.4
os: darwin x64
@angular/animations: error
@angular/common: error
@angular/compiler: error
@angular/core: error
@angular/forms: error
@angular/http: error
@angular/platform-browser: error
@angular/platform-browser-dynamic: error
@angular/router: error
@angular/cli: error
@angular/compiler-cli: error
@angular/language-service: error
@ngtools/webpack: 1.5.1
npm 5.2.0
macOS Sierra
Repro steps.
ng new myapp
cd myapp
ng serve                  ----   works
npm install angular-switchery
ng serve                   ----   does not work anymore

The log given by the failure.
The "@angular/compiler-cli" package was not properly installed.
Error: The "@angular/compiler-cli" package was not properly installed.
    at Object.<anonymous> (/Users/stevermeister/workspace/research/swttt/node_modules/@angular/cli/node_modules/@ngtools/webpack/src/index.js:14:11)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/stevermeister/workspace/research/swttt/node_modules/@angular/cli/tasks/eject.js:10:19)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)

Mention any other details that might be useful.
angular-switchery package was created with help of angular-librarian tool

Were there any other steps that you did? I'm trying to reproduce (to make sure angular-librarian isn't at fault) and following the outlined steps had no problem running ng serve

I realized I was on 1.1.1 and upgraded the CLI...have the same problem. Did the CLI change how it manages packages for a project when running ng new?

This may or may not be the issue but after quickly looking over the library, the Angular related dependencies should be setup as peer dependencies. (https://github.com/stevermeister/angular-switchery/blob/master/package.json#L80) Also, only the directly used packages should be mentioned.

Nice suggestion, @clydin, let me see what happens if i do that

This definitely is a problem with peer deps. By having them as dependencies instead, you are getting duplicates. You can check this with npm ls @angular/core (and others).


Integer overflow

Not sure if I have too big map, or something went wrong trying to fix another bugs, but:
RecastContour.cpp:483:26: runtime error: signed integer overflow: 466743685 * 466744550 cannot be represented in type 'int'
RecasttMesh.cpp:396:34: runtime error: signed integer overflow: 466743717 * 466743717 cannot be represented in type 'int'
Last one got silenced moving to init64_t 4 variables there.

Hi.  Sounds like your input geometry is quite large and/or you're not using tiled meshes.  If this is still a problem and you have a specific repro case, please open a new issue and include repro steps.  Thanks.


FileWriter provides the ContentOffset of the added file

Clients can keep track of this offset, if they want to efficiently
download the file without reading the central directory entry.

nice


RangeUpper issue with BallJoint in Unity Plugin

When importing an XML with a ball joint with limited range defined in degrees, the range value is converted to radians in the Unity component. The property hint itself is defined like that as well. However, when running the scene the actual limits are the degree "equivalent" of the radian range of motion (i.e., to get 90 degree range of motion, RangeUpper needs to be set to 90 "radians").
This is on Windows 10, Unity 2021.2.5, most recent version of the plugin.
Here's the XML I noticed this in:
<mujoco model="Ball">
    <compiler angle="degree"/>
    <option timestep="0.002" iterations="50" apirate="50" solver="Newton" tolerance="1e-10" cone="elliptic"/>
    <size njmax="600" nconmax="150" nstack="300000"/>

	<default>
	    <default class="free">
	        <joint type="free" damping="0" armature="0" limited="false"/>
	    </default>
		<joint type="hinge" damping=".2" stiffness="1" armature=".01" limited="true" solimplimit="0 .99 .01"/>
		<geom type="capsule" condim="1" friction=".7" solimp=".9 .99 .003" solref=".015 1"/>
	</default>

    <worldbody>
	<geom type="plane" pos="0 0 0" size="1.5 1.5 .025"/>

        <body name="mocap1" mocap="true" pos="-1 0 1" axisangle="0 0 1 3.141592">
            <geom type="box" group="1" size="0.02 0.02 0.02" contype="0" conaffinity="0" rgba=".9 .5 .5 1"/>
        </body>

		<body name="RightShoulder" pos="-1 0 1">
			<geom name="RightShoulderGeom" type="sphere" size="0.04"/>
			<joint class="free"/>
			<body name="RightArm" pos="0 0 0">
				<joint type="ball" name="right_shoulder" range="0 70" damping=".2" stiffness="1" armature=".01" limited="true" solimplimit="0 .99 .01"/>
				<geom name="RightArmGeom" fromto="0.04 0 0 .238 0 0" size="0.0398"/>
				<body name="RightForeArm" pos=".278 0 0">
					<joint name="right_elbow" axis="0 0 -1" range="-135 5" stiffness="0"/>
					<geom name="RightForeArmGeom" fromto=".04 0 0 .243 0 0" size=".04"/>
					<body name="RightHand" pos=".283 0 0">
						<geom name="RightHandGeom" type="box" pos="0.08 0 0" size=".08 .05 .0125"/>
					</body>
				</body>
			</body>
		</body>
    </worldbody>

	<actuator>
		<position name="right_shoulder1" group="1" gear="5 0 0"  joint="right_shoulder"/>
		<position name="right_shoulder2" group="1" gear="0 5 0"  joint="right_shoulder"/>
		<position name="right_shoulder3" group="1" gear="0 0 5"  joint="right_shoulder"/>
	</actuator>

	<actuator>
		<position name="right_elbow"     gear="2"  joint="right_elbow"/>
	</actuator>

	<equality>
        <weld body1="mocap1" body2="RightShoulder" solref="0.01 1" solimp=".9 .9 0.01"/>
	</equality>
</mujoco>

You're right.  In MjHingeJoint.cs you can see the various accommodations we go through to ensure radian/angle interplay (e.g., MjSceneImportSettings.AnglesInDegrees), which are currently not used for the ball joint class.  We will amend this in the next version, but if this is blocking you feel free to send a pull request.


Research on Indy

Starting the Indy node instructions: https://github.com/hyperledger/indy-node/blob/master/docs/source/start-nodes.md


Init Azure.ResourceManager.Arc.ScVmm

/azp run prepare-pipelines

/azp run net - arc-scvmm - ci

API change check for Azure.ResourceManager.Arc.ScVmm
API review is created for Azure.ResourceManager.Arc.ScVmm. You can review APIs here.

There are a lot of XXXId and Uuid properties with string type. Probably we need to check if their types should be ResourceIdentifier or UUID. I left some comments in APIView: https://apiview.dev/Assemblies/Review/f6ad73d603f44669b01480e6dd60409d


regproc and regclass shouldn't be KEYWORD

          `regproc` (and `regclass` above) does not need to be a [keyword](https://www.postgresql.org/docs/current/sql-keywords-appendix.html) and recognized by sqlparser. They can just be a `Custom(ObjectName)` like `int2` or `timestamptz`.

Originally posted by @xiangjinwu in https://github.com/risingwavelabs/risingwave/pull/12272#discussion_r1324195251

Resolved by #16933


Optimization for 'sign orders with delegate's address implementation'

Optizations for #297. Did not know where else to put this.
Saves about ~12500 gas on the 18000 gas increase.
Some thoughts about the optimizations (which all try to do the same thing: minimize delegate calls):

checkCutoffsBatch() was introduced to batch check all order cutoffs. The bad thing about this is that validateOrder() doesn't do this check anymore. If this is not wanted there's an alternate optimization in comments that just requests cutoffs and tradingPairCutoffs together.
addCancelledOrFilled is done in batchTransferToken(), which makes a lot of sense.
cancelledOrFilledAmounts fetches are done in batches of 3. Solidity doesn't support returning dynamic arrays so this is done as a workaround. The code for this is pretty clean and simple so this doesn't add much complexity.

(batchTransferToken changed interface but I have not updated those specific tests yet. I will wait for feedback and update those tests when needed.)

This PR will be merged to versoin1.5

about 7500 gas was saved as i tested.

Our master branch changed a lot so this PR was not merged to master. I reused some code in this PR and saved about 3300 gas.  I count this as your contribution，Thanks.

Alright, thanks for the clarification.


Artifactory provider plugin crashes with multiple module calls

Describe the bug
terraform-provider-artifactory plugin crash.
Requirements for and issue

I'm trying to create a module for repo creation in artifactory. There are 60+ repos. For creating each repo an individual module call will be created. When calling these repos using module calls, terraform plan succeeds but the terraform apply fails with "plugin crashed error". Whereas, if the resources are created directly then all the 60+ repos are created without any issue. Plugin crash occurs when there is a separate module call for each repo.

terraform code to reproduce the bug.
This is the module structure to create the project and repos
├── artifactory-projects
│   ├── main.tf
│   ├── outputs.tf
│   └── variables.tf
└── artifactory-repos
├── main.tf
├── outputs.tf
└── variables.tf
The file to call the modules
sample_code.txt
Module file for project creation
main.txt
Module file for repo creation
main.txt
I tired to apply the configuration with both version 6.19 and 7.0 with both the versions the plugin crashed. Attaching the log file too.
artifactory plugin crash log.txt
Expected behavior
Repos to be created seamlessly without plugin crash when there is multiple module calls.

@aisvarya2 Since the files you provided do not contain the full set, I have to recreate my own set of modules (based on yours). As the error in the logs shows a crash originates from the virtual repo resource, I defined a module with set of local, remote, and virtual docker repos.
main.tf in root directory:
module "local-docker" {
  source = "./modules/repos"
  local_docker = {
    key = "docker-local"
    description = "Test description"
    xray_index = true
    docker_tag_retention = 3
    docker_max_unique_tags = 1
  }
}

module "remote-docker" {
  source = "./modules/repos"
  remote_docker = {
    key                         = "docker-remote"
    description                 = "Test description"
    xray_index                  = true
    enable_token_authentication = true
    url                         = "http://tempurl.org"
  }
}

module "virtual-docker" {
  source = "./modules/repos"
  virtual_docker = {
    key                     = "docker-virtual"
    description             = "Test description"
    repositories            = [module.local-docker.local_docker_key, module.remote-docker.remote_docker_key]
    default_deployment_repo = module.local-docker.local_docker_key
  }
}

main.tf from ./modules/repos:
terraform {
  required_providers {
    artifactory = {
      source  = "jfrog/artifactory"
      version = "~> 7.0"
    }
  }
}

resource "artifactory_local_docker_v2_repository" "this" {
  count                = var.local_docker != null ? 1 : 0
  key                  = var.local_docker.key
  description          = var.local_docker.description
  xray_index           = var.local_docker.xray_index
  tag_retention        = var.local_docker.docker_tag_retention
  max_unique_tags      = var.local_docker.docker_max_unique_tags
}

resource "artifactory_remote_docker_repository" "this" {
  count                       = var.remote_docker != null ? 1 : 0
  key                         = var.remote_docker.key
  description                 = var.remote_docker.description
  xray_index                  = var.remote_docker.xray_index
  enable_token_authentication = var.remote_docker.enable_token_authentication
  url                         = var.remote_docker.url
}

# virtual docker
resource "artifactory_virtual_docker_repository" "this" {
  count                   = var.virtual_docker != null ? 1 : 0
  key                     = var.virtual_docker.key
  description             = var.virtual_docker.description
  repositories            = var.virtual_docker.repositories
  default_deployment_repo = var.virtual_docker.default_deployment_repo
}

variables.tf in ./modules/repos:
variable "local_docker" {
    type = object({
        key = string
        description = string
        xray_index = bool
        docker_tag_retention = number
        docker_max_unique_tags = number
    })
    default = null
}

variable "remote_docker" {
    type = object({
        key = string
        description = string
        xray_index = bool
        enable_token_authentication = bool
        url = string
    })
    default = null
}

variable "virtual_docker" {
    type = object({
        key = string
        description = string
        repositories = list(string)
        default_deployment_repo = string
    })
    default = null
}

outputs.tf in ./modules/repos:
output "local_docker_key" {
    value = length(artifactory_local_docker_v2_repository.this) > 0 ? artifactory_local_docker_v2_repository.this[0].key : null
}

output "remote_docker_key" {
    value = length(artifactory_remote_docker_repository.this) > 0 ? artifactory_remote_docker_repository.this[0].key : null
}

When I execute terraform apply, all the repos are created successfully. Tested with provider 7.2.0 and Artifactory 7.55.2.
Please share the full set of module files so I can reproduce this issue in my environment.

@alexhung I'll upload the entire set of files to reproduce the bug. Also, I would like to point out that the plugin crashes occur when multiple repos i.e say 15+ repos are created at once. As mentioned above, we have 60+ repos created directly using the resource block and it works seamlessly but when the same is modularized this way, we are facing plugin crash.
Files according to the module structure
├── artifactory-projects
│ ├── main.tf
│ ├── outputs.tf
│ └── variables.tf
└── artifactory-repos
├── main.tf
├── outputs.tf
└── variables.tf
Project creation
main.txt
outputs.txt
variables.txt
Repo creation
main.txt
outputs.txt
variables.txt
Module call
sample_code.txt
Attaching the crash log once again,
artifactory.plugin.crash.log.txt

@aisvarya2 Thanks for the files. I'll investigate further.

@aisvarya2 Thank you for sharing the module files. I am able to reproduce the issue.
Cause
Like a plan crash, it is sequence of many different events/errors that leads to the final one you reported.

Using modules means Terraform can parallelize the execution (vs single thread with one root module). By default the parallelism is set to 10. This means with enough modules, Terraform can saturate Artifactory ability to handle the incoming API requests. This is of course dependent on many factors (CPU, memory, etc.)
These parallel requests in your example result in Artifactory unable to create one of the repository.
This failure in creating a repository results in the module output being empty string (""):

output "local_docker_key" {
  value = try(artifactory_local_docker_v2_repository.this[0].key, "")
}

which in turn results in the repositories list containing an empty string.
4. Unfortunately, there's a bug in the Terraform SDKv2 framework that translate this empty string into nil value: https://github.com/hashicorp/terraform-plugin-sdk/issues/741 This is what the final error you see, as our provider tries to transform the repositories attribute value (list of repo keys) into list of strings for the Artifactory API request.
Workarounds and Fix
We'll update the code to filter out nil value in the provider. This will prevent the provider from panicing.
In the meantime, you can bypass this error by compacting the repository list:
repositories = compact([
  module.foo-pypi-dev-local.local_pypi_key,
  module.foo-pypi-dev-proxy.remote_pypi_key
])

But both will not prevent the root cause which is Artifactory being overwhelmed with requests. To workaround this, I recommend experimenting with decrementing the -parallelism=n argument with terraform apply.


Make DAP's conformance section overwritable.

This'll work, and I'll merge it, I'm just curious what your use-case is for having different conformance requirements per spec.

It's a special case for the Generic Sensor API. All the examples are made up. And so I needed to add a little text t to explain that.

Huh, okay.  I'd've just put that in the intro or something, but whatever floats your boat.
(Boilerplate is just there to CYA and to point to for complete newbs; it's read by virtually no one, and so anything put there might as well be invisible.)

Yeah. Maybe you're right and it belongs elsewhere. It was requested by the chair of one of the WG which might or might not decide to use the sensor api, so it is sort of CYA-type text.


Incorrect access log pattern in EI 6.1.1 documentation

Description:
As per the documentation [1]  %T can be used to capture "Time taken to process the request in seconds". However, after adding %T to the log pattern, it is logging Content-Type instead of the time taken. It should be rectified as shown in documentation [2].
Suggested Labels: Docs
Affected Product Version: EI 6.1.1
[1]. https://docs.wso2.com/display/EI611/Monitoring+ESB+Access+Logs
[2]. https://docs.wso2.com/display/ESB490/Access+Logs

Fixed from EI 6.1.1 to EI 6.6.0.
[1] https://docs.wso2.com/display/EI660/Monitoring+Access+Logs


Syntax error : "(" unexpected on "npm install"

I've got the following error :
Error: Get config /usr/local/lib/node_modules/prisma2/query-engine-linux-glibc-libssl1.0.2-ubuntu1604: 2: /usr/local/lib/node_modules/prisma2/query-engine-linux-glibc-libssl1.0.2-ubuntu1604: Syntax error: "(" unexpected
While trying to prisma2 generate, and
Error: Schema parsing /home/spliziws/webservices/splizi/node_modules/prisma2/query-engine-linux-glibc-libssl1.0.2-ubuntu1604: 2: /home/spliziws/webservices/splizi/node_modules/prisma2/query-engine-linux-glibc-libssl1.0.2-ubuntu1604: Syntax error: "(" unexpected
when trying to npm install
It works perfectly locally on my Windows 10 computer, but these errors happens on my linux server !

Hi,
I just testing this on a Ubuntu 16.04 Digitalocean instance and I was unable to reproduce the issue.
The output of uname -a of the machine that I used:
Linux ubuntu-s-1vcpu-1gb-fra1-01 4.4.0-166-generic #195-Ubuntu SMP Tue Oct 1 09:35:25 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

The version of Prisma 2 that I used<EMAIL_ADDRESS>binary version: 20b6dc13949cccccfef5be07c0be7a3d7c858abe
Can you please make sure that you are using the latest version of prisma2 and also please make sure that you install the prisma2 binary using the --unsafe-perm flag.
npm install -g --unsafe-perm prisma2

Hey
The uname -a :
Linux dev 4.4.127-mainline-rev1 #1 SMP Sun Apr 8 10:33:17 UTC 2018 armv7l armv7l armv7l GNU/Linux
The actual version of Prisma 2 on the server :
<EMAIL_ADDRESS>binary version: 20b6dc13949cccccfef5be07c0be7a3d7c858abe
I've got the same error after reinstalling it with the --unsafe-perm flag :
Error: Schema parsing /home/webservices/splizi/node_modules/prisma2/query-engine-linux-glibc-libssl1.0.2-ubuntu1604: 2: /home/webservices/splizi/node_modules/prisma2/query-engine-linux-glibc-libssl1.0.2-ubuntu1604: Syntax error: "(" unexpected

Looks like you are using Prisma on ARM. We currently do not compile Prisma for ARM.

If that is the case, please feel free to close this as a duplicate of : https://github.com/prisma/prisma2/issues/132

Oh ok, thanks then ;)

@pantharshit00 Please create an actual issue "Compile for ARM" if this is the actual wanted change here instead of keeping these bug issues around. Thanks.

#132 which I mentioned here is the issue tracking this and it has more context as well.
I will update the title of #132

A bug is not a feature request. Updating the title of a bug issue does not make it a feature request. Please take the time to create a new issue that is clear and summarizes the problem and ask.


Update icon pack

According to doc icon pack version is 4.5.95. Latest version is 7.0.96.
Consider updating the icon pack.
Also in the doc link to the jsdelivr is not helpful. It doesn't provide any list of icons (according to doc).

I looked into this with @mariobuikhuizen, but updating the icon pack will break people's existing codes since they have done quite some renaming.
However, the link in the docs was not very useful, but now it is:

https://ipyvuetify.readthedocs.io/en/latest/usage.html#icons
points to<EMAIL_ADDRESS>
I believe you were looking for the github icon, which is mdi-github-circle.
Does this help?


docs: 修改interaction中tooltip的crosshairs选项描述错别字

Checklist

[ ] npm test passes
[ ] benchmarks are included
[ ] commit message follows commit guidelines
[ ] documents are updated

Description of change

修改错别字 “暂时” => "展示"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10296

Relevant Lines:
11547

💛  - Coveralls


Console warning from header profile image dropdown

Description
Fixed the warning.
Type of change

[x] Bug fix
[ ] New feature
[ ] Refactoring
[ ] Documentation
[ ] Improvement

Solution description
Some proptypes is not correct. changed them.
Tests
All unit tests pass? NA
Builds successfully? Y
Do all SA checks pass? Y
Risk
Low. Might raise warning when changing prop type again.

Thanks for making the changes, noticed two other console errors that we may want to fix as well:

Thanks for making the changes, noticed two other console errors that we may want to fix as well

thanks for catching that, fixed the first one, the second will be fixed in this pending PR


DBZ-1224 SqlServer Snapshot Select Overrides

Based on PostgreSql connector existing settings, introduced below two settings into SqlServer Connector.

Introduced Snapshot Select Override setting.
Introduced rows.fetch.size configuration setting.

Hey @rsatishm, thanks a lot! We'll take a look into this asap; in the meantime, could you:

File a JIRA issue for this feature request and prefix the commit messages with its key
Use an email to author the commits that's linked to your GitHub account?

Thanks!

Hey @rsatishm, thanks a lot! We'll take a look into this asap; in the meantime, could you:

File a JIRA issue for this feature request and prefix the commit messages with its key
Use an email to author the commits that's linked to your GitHub account?

Thanks!

Thanks. I have filed the Jira issue - https://issues.jboss.org/browse/DBZ-1224
Also prefixed all my commit messages with the issue#

Hey @rsatishm, something seems to went wrong a bit here: there are way more commits in the PR than there should. Could you rebase the PR to the current master?
And could you also please use an e-mail associated to your GH account for authoring the commit; that way it will be properly attributed to you in the project stats.
Thanks!

Hey @rsatishm, something seems to went wrong a bit here: there are way more commits in the PR than there should. Could you rebase the PR to the current master?
And could you also please use an e-mail associated to your GH account for authoring the commit; that way it will be properly attributed to you in the project stats.
Thanks!

I see now snapshot row fetch setting is part of common config so would that be the reason? Also now my other part of change which configures the row fetch size is no more required. I will rebase the PR to the current master.

I see now snapshot row fetch setting is part of common config so would that be the reason?

I reckon at some point you did a merge instead of rebase. If you rebase it, the PR eventually should only contain your commit(s).

Hey @rsatishm, just a quick heads-up: we're planning to release Debezium 0.9.5 tomorrow, so if the things we discussed by then, we could include it. If not, it's no problem either, as we'll release every three weeks. So please don't feel pressured about it, just wanted to bring it up, just in case you had planned to wrap this up today anyways. Thanks!

Hey @rsatishm, howdy? How are things, are you still planning to wrap this one up?

Hey @rsatishm, howdy? How are things, are you still planning to wrap this one up?

Hi @gunnarmorling, my apologies, for some reason my stuck with the git login issues. I am able to login here in github but the same credentials some doesn't work in my IDE environment. I couldn't figure out yet why the login credentials doesn't work.

Doh, sorry to hear that. So what should we do then, should someone else take over this one? Or perhaps you try with a new IDE set-up to get unstuck?
Would be great to come to closure with this one in one way or another.

I'll try and rebase this and completing any outstanding task related to this.

Superseded by https://github.com/debezium/debezium/pull/888. I'll close this one here in favor of the other one (which contains the exact same diff) for the sake of a clean commit history. I'll rebase that other one next now. Thanks, @rsatishm!


Use serverDataFolderName in check-requirements-linux.sh

Instead of serverApplicationName it should be serverDataFolderName

serverDataFolderName matches with the data folder that is already there
serverDataFolderName starts with a . which is IMO what we want.

Any reason we want the file to be hidden by default ? I went with serverApplicationName to avoid the file from being hidden.

I think it should be hidden as it's an VS Code internal file. No point for a user to see or manipulate.
I was think that this is the file the extension will create after running the script the first time. Or maybe I misunderstood.
That said, the file won't work to remember that the OS passes the compatibility check. Maybe we need a file that has the error code as content?

I think it should be hidden as it's an VS Code internal file. No point for a user to see or manipulate.

No actually it is the opposite, this file is meant for users to force the legacy server. If the detection script is not returning the right result on a setup for whatever reason we need a way for the users to override the result.

Thinking more about it, the file won't work to remember that the OS passes the compatibility check. Maybe we need a file that has the error code as content?

I was thinking the extensions would setup that file depending on the exit code from this script.

If the detection script is not returning the right result on a setup for whatever reason we need a way for the users to override the result.

Thinking again, should this rather be a setting on the client side to force the legacy server and avoid introducing a separate file ? That way it works for both classic and exec server cases, also the extension controls the behavior.
remote.useLegacyServer: "true" | "false"

@deepak1556 Now that check-requirements is no longer part of the server start script, I suggest we remove the
if [ -f "$HOME/@@SERVER_DATA_FOLDER_NAME@@-use-legacy" ]; then

check and leave it to the extensions to have a marker file or a setting.


Cancel in-progress runs for open PRs

Resolves #18
Motivation and Context
If we push multiple updates to an open PR, GitHub Action runs will start to queue up. Action runs for all pushes other than the latest do not add any value and can be cancelled.
Description
We use concurrency to cancel any in-progress runs for an open PR.

Validation can be found with the cancelled action superseded by the latest action run


KYLIN-3737 refactor cache part for RDBMS

Pull Request Test Coverage Report for Build 3994

4 of 69 (5.8%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 26.804%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

datasource-sdk/src/main/java/org/apache/kylin/sdk/datasource/adaptor/AbstractJdbcAdaptor.java
3
5
60.0%

datasource-sdk/src/main/java/org/apache/kylin/sdk/datasource/adaptor/DefaultAdaptor.java
1
64
1.56%

Totals

Change from base Build 3992:
-0.02%

Covered Lines:
18522

Relevant Lines:
69102

💛  - Coveralls

Local CI has passed.


chore: merge stable to unstable

Motivation

#6702

Description

:tada: This PR is included in v1.19.0 :tada:


Convert []interface{} to specific slice

Is this library supposed to handle this case of a slice of interfaces? If so, I don't think I was able to convert a slice of []interface{} to []string

What code are you using to do the conversion? It should work okay.
from := []interface{}{"1.2", "34.5", "-678.9"}
var into []string
conv.Slice(&into, from) // type inferred from the element of `into`
fmt.Printf("Slice:\n  %#v\n    -> %#v\n\n", from, into)

My mistake, I must have been doing something wrong. Thanks!

No worries man glad ya got it working!


Train concept other than person?

Newcomer here, may sound like a dumb question --

How do I train something else other than a person? I notice in regularization images section there is only images for people. Say if I want to train an animal, or mascot of sort, what should I do? Should I first make a judgment on its category (toy, animal, etc) then create my own regularization image dataset?

How do I train something intangible like a style? How do I create a regularization dataset for that?

Thank you!

Got pretty good results with that basic approach.


Write relative paths with unnamed requirement syntax

Summary
This PR falls back to writing an unnamed requirement if it appears to be a relative URL. pip is way more flexible when providing an unnamed requirement than when providing a PEP 508 requirement. For example, only this works:
black @ file:///Users/crmarsh/workspace/uv/scripts/packages/black_editable

Any other form will fail.
Meanwhile, all of these work:
file:///Users/crmarsh/workspace/uv/scripts/packages/black_editable
scripts/packages/black_editable
./scripts/packages/black_editable
file:./scripts/packages/black_editable
file:scripts/packages/black_editable

Closes https://github.com/astral-sh/uv/issues/3180.

Me too.

Windows failures... no...


Udf example counter

simple reduce counter example

should we rename these example repos with the prefix map- or reduce-?

should we rename these example repos with the prefix map- or reduce-?

the docker image repo?

the docker image repo?
your call 😄

It would be good to put the example images into different groups, for example, quay.io/numaio/udf/go-counter-example - feel free to come up with better names.


Grouped updates for GHES?

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
I was looking at the public roadmap for grouped updates with Dependabot, but noticed that grouped updates aren't mentioned alongside GHES.
Is there plans to support update groups with GHES in the future, and if so is there a timeline?

Answering my own question here, found this line item. Looks like this is planned for GHES 3.11.


Show current status: mounted

In panel now 3 columns:
n│         Resource          │           User
More informatively add column Status
n│         Resource          │           User         │           Status
which will show mounted/disconnected and advise use Shift+F8 to unmount

first column now shows star sign for mounted resources


Helm Installation failed on EKS

Describe the bug
Installation failed via Helm on EKS
Expected behavior
Connaisseur install failed on EKS with the following error:
error: template: connaisseur/templates/deployment.yaml:9:24: executing "connaisseur/templates/deployment.yaml" at <include "getConfigFiles" .>: error calling include: template: connaisseur/templates/_helpers.tpl:146:3: executing "getConfigFiles" at <include (print $.Template.BasePath "/config.yaml") .>: error calling include: template: connaisseur/templates/config.yaml:12:24: executing "connaisseur/templates/config.yaml" at <deepCopy .>: error calling deepCopy: reflect: call of reflect.Value.Type on zero Value
Optional: To reproduce
Stepwise guide or instructions or minimal example to reproduce.
Optional: Versions (please complete the following information as relevant):

OS:
Kubernetes Cluster: <1.22>
Container registry:
Connaisseur: <connaisseur 1.4.4>
Other:

Optional: Additional context

This doesn't seem to be EKS related, but a general bug in the helm chart. Also I can't reproduce it, so i guess it is fixed on newer versions. Should that not be the case, feel free to repoen the issue.


Fix issue with static methods in class autoloader

Description of the Change
In #158 we added a way to auto-initialise and load classes in the MU plugin and theme.
Because the classfinder uses static methods, it only works on a single application root.
To get around this, we need to manually set the application root before we look for classes in a namespace, so that it can find the right composer.json file
This has been tested on a client project that's using this version of the scaffold.
How to test the Change

Add a class to the MU plugin and another to the theme.
Make sure that both are registered.

Changelog Entry

Fixed - Issue with class loading conflict.

Checklist:

[x] I agree to follow this project's Code of Conduct.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my change.
[x] All new and existing tests pass.

@joesnellpdx @tlovett1 wondering if one of you could review this since it fixes a bug that could cause engineers some pain in the scaffold.


DefaultAppEnvironmentsProvider improvement

It would be great if DefaultAppEnvironmentsProvider gets integrated with Microsoft.Extensions.Configuration.IConfiguration and read setting from appsettings.json
It would be nice if app gets reloaded after environments.json changes too.

This is a sample app which uses appSettings.json
Just call
AspNetCoreAppEnvironmentsProvider.Current.Init();
Sample


✨ Add ReDoc <noscript> warning when JS is disabled

Redoc shows a blank page if javascript is disabled.
Using a <noscript> element in the HTML, the user is informed that he/she should enable javascript in order to view the documentation.

Cool, thank you @evroon! :cake:


errors when attempting to use uTorrentPostProcess.py

Hi,
I have followed the installation guide and setup for uTorrent to launch the uTorrentPostProcess.py as described.  I have checked the uTorrent logs and the commands are being sent to the script, though nothing is happening - I took the command line from the logs and launched it manually to observe what is happening and it is failing with errors.  Also, no log is being produced in the automator directory.
OS LUbuntu 16.10 (fully patched and up-to-date)
Command line: /home/user/mp4_automator/uTorrentPostProcess.py tv http://open.nyaatorrents.info:6544/announce /home/user/torrents/complete/tv single [HorribleSubs] Akiba's Trip The Animation - 01 [720p].mkv 44FF40C8DC7C75FB36DE77858595C5B4ADB397DC [HorribleSubs] Akiba's Trip The Animation - 01 [720p].mkv
Error output in terminal:
import: unable to open image os': Permission denied @ error/blob.c/OpenBlob/2712. import: unable to open image re': Permission denied @ error/blob.c/OpenBlob/2712.
import: unable to open image sys': Permission denied @ error/blob.c/OpenBlob/2712. import: unable to open image shutil': Permission denied @ error/blob.c/OpenBlob/2712.
from: too many arguments
from: too many arguments
from: too many arguments
import: unable to open image logging': Permission denied @ error/blob.c/OpenBlob/2712. from: too many arguments /home/plex/mp4_automator/uTorrentPostProcess.py: line 11: syntax error near unexpected token os.path.join'
/home/plex/mp4_automator/uTorrentPostProcess.py: line 11: `fileConfig(os.path.join(os.path.dirname(sys.argv[0]), 'logging.ini'), defaults={'logfilename': os.path.join(os.path.dirname(sys.argv[0]), 'info.log').replace("\", "/")})'

Hi Michael,
I have given up on attempting to use any of the post process scripts now after 2 weeks of testing all of the integration - they are mostly hit and miss and will work with a custom script and utilise the manual processing script in conjunction with my custom script. Thanks for your time on this.


WIP: Oracle Security Delay

Still working on tests and comments. MarketOracle.sol open for comments.

@brandoniles Ready for another look

Tests ready fro review.


Hello

Summary
Provide an overview...
Description
Add more context to describe the changes
Details

[ ] some project details

References

Link #

Checks

[ ] Already tested changes in staging environment

Notes

Some awesome notes

Already tested in staging env, good to proceed to production

Already tested in staging env, good to proceed to production

Thanks for your hard work.


Improve discovery of Commonalties output documents by API Subprojects

Documentation update that fixes #70

Thanks for the updates @rartych and apologies for the slow responses (was on a short break the last week).


[v3.28] Update go to v1.22.7

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[x] Release note

Release Note

Don't run pprof on prometheus metrics port

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

Force-merging as only CI failures are related to GCP VM quota (and I've seen passes in previous runs)


please add a new icon short cut to toggle pannel for mouse user

when hide pannel ,for mouse user,thre are very tedious steps to show panel again.
please add an icon to state bar for mouse user to toggle panel
thanks

Please illustrate your feature request with screen shots.

I like this idea. I've previously added buttons for this in the Powershell extension: https://github.com/PowerShell/vscode-powershell/pull/2704
So if this doesn't get added you could always create your own tiny extension by just taking the icons and code I wrote in that pull request.

Please illustrate your feature request with screen shots.

thanks

@heartacker it is already possible to drag the panel out from the status bar using the mouse. hover over the editor-status bar border and drag upward.

@heartacker it is already possible to drag the panel out from the status bar using the mouse. hover over the editor-status bar border and drag upward.

thanks for your remind.but without your remind, Very few people konws this. so i think vscode team should re-think about that.


Update MANIFEST.in to enable a testable sdist

Resolves #264.
Whole bunch of stuff to make an sdist actually testable needs to be packaged.

Codecov Report

Merging #268 (ca95881) into main (a54f731) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #268   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           20        20
  Lines          743       743
=========================================
  Hits           743       743

Impacted Files
Coverage Δ

src/sphobjinv/version.py
100.00% <100.00%> (ø)

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


Issue labeling for new contributors

This is an updated version of the original page that I wrote for Kubernetes about good first issue and help wanted labels.
It includes some new advice and removes Kubernetes specific text.
My git attribution for that page was lost when it was copied elsewhere in the repo... but here's the first commit: https://github.com/kubernetes/community/commit/742a0445d62e8bd359b8d5d5025d94e8a6db4559 and this is the current page on the K8s site: https://www.kubernetes.dev/docs/guide/help-wanted/

the explanation of differences is perfect and its a good foundational document that folks need when recruiting and retaining contributors
LGTM!


Update year-in-review.html

added anchor tags to the twitter handles to link to twitter page, added the target='_blank' to allow for page to open in a new tab.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@juansnow01 You will likely need to sign the CLA for this PR to be merged : )


fix(DomEvent): change stopPropagation to fakeStop on disableClickPropagation

Hello!
The ev.stopPropagation() on the mousedown event was preventing the click event to be handled inside popup children elements. With this change we're able to handle click events inside the popup content.
Hopefully close #6044.

This approach looks in spirit of 5a7420dd1a43474cccaa8cdefa4f324452d18f36 (where  _fakeStop was introduced). Original reasoning: https://github.com/Leaflet/Leaflet/issues/301#issuecomment-3071362
I don't know if this could be useful in general, but to solve this particular issue (#6044) I'd prefer more local and precise fix: #7091.

Hi! Ok great I'll let you know if this eventually fixes my original problem, I hope so! Thanks!


docs(pull request template): removes the reference to the CLA

removes the refence to the CLA. Asks in the contributing file to use npm run commit for the commit
messages
fixes #42


Stop expanding content if beforeToggle returns false

This might sound silly, but I'm using readmore.js just because it cuts the text in a block for me. When the user clicks on the "read more" link I open a modal to display it. So I just need it to cut the text and avoid expanding it.
I thought that returning false from the beforeToggle callback might be a good way to implement this behavior.

This will be included in Readmore.js v3


How would one Refresh the user Token?

What would be the preferred way to refresh a token to create a sliding expiration using this approach?

What would be the preferred way to refresh a token to create a sliding expiration using this approach?

What would be the preferred way to refresh a token to create a sliding expiration using this approach?


Update test_postgres_metadata_extractor.py

Summary of Changes
see #108
Tests
Executed locally test in Python 2.7 and Python 3.
Documentation
This change is related to #108
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test
[ ] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

I'm a bit tired, I don't see why the test via the CI is now failing...

The CI test is failing on two test cases - here’s the assert for one of them: https://travis-ci.com/lyft/amundsendatabuilder/builds/119367368#L1276
It’s maybe failing because your change for the test is running against master which doesn’t include your PR #108 changes. Try committing and pushing https://github.com/lyft/amundsendatabuilder/pull/111/commits/61e6ed56adf50282da74933beb395884e002c2c9 to your https://github.com/thierryturpin/amundsendatabuilder/tree/thierryturpin-patch-1 branch and the test should run again.

With #117 now merged as an alternative solution to PR #108 I believe this PR is not needed and should just be closed
Cc @feng-tao

correct. thanks.


feat: Update to support AWS provider v4

From the top of my busy head... 23rd of February, I have done the required changes to the main/root module and implemented all existing features as separate resources there, verified parts of examples (commenting and uncommenting parts of existing examples).
Ideally, examples should be very similar to what was before. Also, add an upgrade guide to show commands to do state mv and import for resources, update versions to use the latest AWS provider (there were important bugfixes), submodules.
The wrapper is not very popular AFAIK, so we can skip it for now. I have a magic script for this somewhere in the gist.
I don't have time and the possibility to finish this PR myself at the moment, so if @bryantbiggs or anyone else can/want to finish this - you are welcome.
I will be able to review with a small delay.
Glory to Ukraine! 🇺🇦

Fixes #137
Closes #138

Is there a way for me to use this now? Rather than waiting for it to be merged? How is that done in the Terraform world. Also 👍  for making the changes.

Is there a way for me to use this now? Rather than waiting for it to be merged? How is that done in the Terraform world. Also 👍 for making the changes.

It's definitely not recommended to use in anything other than testing, and is probably very highly discouraged, but you can do this:
module "s3" {
  source = "git::https://github.com/terraform-aws-modules/terraform-aws-s3-bucket.git?ref=fix-tf-provider-v4
}

Thanks Anton,
I think you missed some var declaration, but overall it's work for me as expected.

HI All,
What needs to happen to get this across the finish line?  I'd be happy to contribute where I can.
This module has been a great help for me and I'd like for this get completed and merged.

Hello everyone!
I have been recovering and resumed working on this PR earlier today after some thinking about potential resolutions mentioned here (https://github.com/hashicorp/terraform-provider-aws/issues/23106). One of the things I have been thinking about was how to make this module compatible with v3 AND v4 to minimize state migration for resources.
Anyway, I plan to finish this PR and have support for AWS provider v4 as soon as possible - Tuesday, 29th of March.

in regards to state migration using moved blocks would be an easy win? no need for manual state moves by anyone? https://www.terraform.io/language/modules/develop/refactoring

This PR is included in version 3.0.0 :tada:


Fix Travis build on OSX

The key changes are:

ensure we use the pip corresponding to our Python

Don't assume nosetests is on the PATH, explicitly run python -m nose

This was merged, and then forced back out because macOS was segfaulting on not-4.0.
On my own machine, I got this:
$ nosetests tests.test_python_io
objc[74641]: Class AVFFrameReceiver is implemented in both /Users/mikeboers/Documents/PyAV/vendor/build/ffmpeg-4.0/lib/libavdevice.58.dylib (0x120da1220) and /Users/mikeboers/Documents/PyAV/vendor/build/ffmpeg-3.2/lib/libavdevice.57.dylib (0x12187a1d0). One of the two will be used. Which one is undefined.
objc[74641]: Class AVFAudioReceiver is implemented in both /Users/mikeboers/Documents/PyAV/vendor/build/ffmpeg-4.0/lib/libavdevice.58.dylib (0x120da1270) and /Users/mikeboers/Documents/PyAV/vendor/build/ffmpeg-3.2/lib/libavdevice.57.dylib (0x12187a220). One of the two will be used. Which one is undefined.
test_basic_errors (tests.test_python_io.TestPythonIO) ... ok
test_buffer_read_write (tests.test_python_io.TestPythonIO) ... Segmentation fault: 11

This was because I did not make clean before testing. Once I did so, I've not been able to segfault it.
Is it at all possible that the caches are somehow getting in the way? That is ludicrous, right?

I was testing a few of them at the same time. 😎
My feeling is that it was somehow colliding with the Homebrew FFmpeg. The last one built at least.
I'm making a few inconsequential commits and rebuilding just to give it a kick.


Use pointer pose for one handed manipulation in manipulation handler

Fixes #3863. See issue for background details on the change.

Make sure to check whether HandInteractionExampleScene needs to be updated.


click_lat_lng not working in latest version

It looks like the click_lat_lng property is simply not firing when clicking on the map.
Also when debugging, I simply to not get into the callback.
This did use to work with an older version (I think an older version of Dash).
But I did a clean pip install of dash-leaflet and dash and the problem remains.
I've made a really simple app to replicate the issue:
import dash_leaflet as dl
from dash import Dash, html, Input, Output

app = Dash(__name__)
app.layout = html.Div(
    [
        dl.Map(
            id="map",
            children=[
                dl.TileLayer(id="tile-layer"),
                dl.LayerGroup(id="layer-group"),
            ],
            style={"height": "90vh"},
            center=[50, 0],
            zoom=3,
        )
    ]
)

@app.callback(
    Output("layer-group", "children"),
    Input("map", "click_lat_lng"),
    prevent_initial_call=True,
)
def map_click(click_lat_lng):
    lat, lon = click_lat_lng
    marker = [
        dl.Marker(
            position=(lon, lat),
            children=dl.Tooltip("({:.3f}, {:.3f})".format(*click_lat_lng)),
        )
    ]
    return marker

Furthermore, I found this post on stackexchange that seems to suggest this is an issue with  as well:
https://gis.stackexchange.com/questions/465640/click-feature-doesnt-work-dash-leaflet
Lastly, I have found that with this latest version of that the map does not render unless I provide a center and a zoom.
If I leave these values to default, then I do not see the map. This is however a different issue.

I just found out that the function has been deprecated. An error would have been useful here.
Map: The click_lat_lng/dbl_click_lat_lng prop have been dropped in favor of clickData/dblclickData
https://www.dash-leaflet.com/docs/migration


Menu dropdown positioned incorrectly

When dropdown-content-expand is used with a dropdown that is not full width, the dropdown content is positioned incorrectly in relation to it's dropdown container.

Found not to be a bug -
This issue arises when using dropdown-content-expand and dropdown-right combination.
Solution -

Ensure the dropdown class is accompanied by the width-full and move-right classes.
Ensure the top most menu-activator button has the desired width set. (It can be full width, but most likely would be square).
Ensure the dropdown-right-expand class is accompanied by the width-full class.


e.preventDefault doesn't skip route change when called in .onunload

Relevant tests.
It initializes the controller when it shouldn't.

Fixed now? Looks like #1117 enabled those tests.

@2is10 Correct. This is a relatively old bug, anyways.


Avoid divisions calculation for one partition df in sort_values

This came up in tpch, there is no need to trigger a divisions computation if we are reduced to one partition anyway

I am merging this to include it in the release


Incorrect array size check generated

From @hator on July 5, 2016 15:14
For input:
response {
    status 200
    body( content: [
        value(
            id: 'article_accurest::1',
            tags: ['tag1', 'tag2', 'tag3']
        ), value (
            id: 'article_accurest::2',
            tags: ['tag1', 'tag2', 'tag3']
        )
])}

Generated:
DocumentContext parsedJson = JsonPath.parse(response.body.asString())
assertThatJson(parsedJson).array("content").array("tags").hasSize(3)

The problem is that using Json path <$.content[*].tags[*]> all elements of all tags arrays are counted giving total of 6 instead of 3. This of course causes tests to fail.
Copied from original issue: Codearte/accurest#293

From @hator on July 5, 2016 16:22
How would I model maps, then?
For clarity JSON that I would like to model and groovy contract: https://gist.github.com/hator/60d3104d12c23a5ffa5eea0c9e4b469d

hmm I think that your contract is a little bit wrong. We weren't expecting to insert maps between value(...). Try putting a concrete value there

value() is a helper method for Verifier to provide client side stubbing. It's a synonim of the metdho $ that you used.
In Groovy to use a map you use the [:] notation. I've rearanged your contract - now it should work better.
org.springframework.cloud.contract.spec.Contract.make {
    request {
        method 'GET'
        url '/api/articles?title=test'
        headers {
            header 'Content-Type': 'application/json;charset=UTF-8'
        }
    }
    response {
        status 200
        body( content: [
                [
                    id: 'article_accurest::1',
                    title: 'test title',
                    lead: 'test lead',
                    leadPhoto: [
                        url: "http://photo.png",
                        alt: "photo",
                        title: "photo"
                    ],
                    content: 'test content',
                    tags: ['tag1', 'tag2', 'tag3'],
                    bioId: 'test-bio-id',
                    publishDate: $(test(execute('isZonedDateTime($it)')), stub('2018-05-17T12:32:28Z')),
                    modifiedDate: $(test(execute('isZonedDateTime($it)')), stub('2018-05-18T14:32:20Z')),
                    modifiedBy: $(test(regex(".+")), stub('author1'))
                ],
               [
                    id: 'article_accurest::3',
                    title: 'test game',
                    lead: 'test lead',
                    leadPhoto: [
                        url: "http://photo.png",
                        alt: "photo",
                        title: "photo"
                    ],
                    content: 'test content',
                    tags: ['tag1', 'tag2', 'tag3'],
                    bioId: 'test-bio-id',
                    publishDate: $(test(execute('isZonedDateTime($it)')), stub('2018-05-17T12:32:28Z')),
                    modifiedDate: $(test(execute('isZonedDateTime($it)')), stub('2018-05-18T14:32:20Z')),
                    modifiedBy: $(test(regex(".+")), stub('author2'))
                )
              ]
            ]
        )
    }
}

From @jakubnabrdalik on July 6, 2016 9:1
It doesn't seem to be a problem with value() but rather with how size is verified by com.toomuchcoding.jsonassert:
<$.content[*].tags[*]>

The outcomes from a test fixed by your suggestions look like this:
java.lang.IllegalStateException: Parsed JSON <{"content":[{"id":"article_accurest::1","title":"test title","lead":"test lead","leadPhoto":{"url":"http://grey.png","alt":"Sasha photo","title":"Sasha Grey"},"content":"test content","tags":["tag1","tag2","tag3"],"createdDate":null,"createdBy":null,"modifiedDate":"2016-07-06T08:10:22.433Z","modifiedBy":"accurest_editor","publishDate":"2018-05-17T12:32:28Z","status":"DRAFT","bioId":"test-bio-id"},{"id":"article_accurest::3","title":"test game","lead":"test lead","leadPhoto":{"url":"http://grey.png","alt":"Sasha photo","title":"Sasha Grey"},"content":"test content","tags":["tag1","tag2","tag3"],"createdDate":null,"createdBy":null,"modifiedDate":"2016-07-06T08:10:22.451Z","modifiedBy":"accurest_editor","publishDate":"2018-05-17T12:32:28Z","status":"DRAFT","bioId":"test-bio-id"}],"totalPages":1,"totalElements":2,"last":true,"size":20,"number":0,"sort":null,"numberOfElements":2,"first":true}> doesn't have the size <3> for JSON path <$.content[*].tags[*]>. The size is <6>
	at com.toomuchcoding.jsonassert.JsonAsserter.hasSize(JsonAsserter.java:244)
	at

In other words, the count counts all tags in the json, not tags for a given element.
Does it even have to check the size?

Ah true - I get it.
It's close to impossible to properly check the size sometimes (I'd have to check every single element in the list). So I'd suggest to pass the system prop spring.cloud.contract.verifier.assert.size=false and disable the size check.
I'll try to fix it one day but any help would be more than welcome.

From @jakubnabrdalik on July 6, 2016 9:9
Any documentation on spring.cloud.contract.verifier.assert.size? Is it boolean? Should we just set it to false? Couldn't find it in official docs.

Yeah it's a boolean. Set it to false. I'll add it to the docs.

From @jakubnabrdalik on July 6, 2016 9:22
Setting
spring.cloud.contract.verifier.assert.size: false

in both application.yml and/or bootstrap.yml doesn't seem to have any effect.
Which makes sense, considering this is a gradle task, that has little to do with the rest of the spring app.
Should it be part of contractVerifier properties in gradle? Like:
contractVerifier {
   spring.cloud.contract.verifier.assert.size = false
   ...
}

From @jakubnabrdalik on July 6, 2016 9:10
Thanks.

From @jakubnabrdalik on July 6, 2016 9:24
Ah, ok.

Let's move here - https://gitter.im/Codearte/accurest

It's a system prop - you'd have to pass it as -D.

The docs got updated - http://codearte.github.io/accurest/#limitations

Ok workaround is here. Check the latest snapshots

This is can be fixed via - https://github.com/spring-cloud/spring-cloud-contract/issues/185 . You can provide a manual check of stuff via the matchers


Create FFD/Lattice Deformer

Multiple variations of the deformer with different resolutions is fine (FFD2x2, FFD4x4 etc), but ideally there will be a single deformer that lets you change the number of control points on each axis.
Ideally you can also change the interpolation type.

Got a basic lattice deformer working, resolution is configurable and each control point can be individually moved.
https://user-images.githubusercontent.com/17784523/106396608-e3d38580-6400-11eb-8068-596822339202.mov

Got a basic lattice deformer working, resolution is configurable and each control point can be individually moved.
https://user-images.githubusercontent.com/17784523/106396608-e3d38580-6400-11eb-8068-596822339202.mov

#44 merged


Add .editorconfig for uniform code style

[ ] Base code style discuss @sumory
[ ] Generate .editorconfig file
[ ] Wiki or document for introduce how to apply our code style.

done with this commit #3fa2ee

Yeah, a cool job. But need more document on configuration.


[3/3][AOT][DeviceAPI] Wire up cpacked Device API context

Adding the same functionality for the Device API to the cpacked calling convention. The MakePackedAPI pass now implicitly uses any variable named kDeviceContextVar as the resource_handle and this is then used in the cpacked calling convention which always expects some form of resource_handle to be passed.
This is stacked on top of #9500 and #9395

Thanks @mbs-octoml @Mousius . This is merged now!


hidden wallet logo icon

hi nopara73, I see there is a project to create logo hidden wallet icon, may i contribute? thanks
kind regards

Hi @ivannewgate89
Thanks for your interest.
About the logo: https://github.com/nopara73/HiddenWallet/issues/142
Today we are working on a different temporary repository, since we are rewriting the whole wallet in light of some new research: https://github.com/nopara73/MagicalCryptoWallet
You can see a roadmap there.
How did you think about contributing?


Adding smooth collpased.

Adding smooth collpased.
smooth collapsed can be added over the tabs that mentioned states name.

Closing. This does not add any value to the cause. Please see if there are functional issues that can help the site.