Include custom form media in templates

Problem
When custom import/export forms contain complex widgets with linked media it is not added to output.
Solution
Added standard form.media to templates.
Acceptance Criteria
This fix does not require tests and documentation changes.

Coverage decreased (-0.2%) to 95.404% when pulling fc7fe0ab85f0c2a213c89d6a47b976457eecaa88 on andreynovikov:form-media-fix into 3cf5e3f9796a5caf7c5ea3928119af7ce4706c0d on django-import-export:master.

Hey @andreynovikov . Thanks for the PR. When you say

custom import/export forms contain complex widgets

Can you give an example of this and some screeshots? Also how does this effect current/standard behavior?

I've also ran into this problem. If you want to recreate in the test app, you can do so by updating tests.core.forms.AuthorFormMixin like so:
from django.contrib import admin
from django.contrib.admin.widgets import AutocompleteSelect

class AuthorFormMixin(forms.Form):
    author = forms.ModelChoiceField(
        queryset=Author.objects.all(),
        widget=AutocompleteSelect(Author.objects.all(), admin_site=admin.site, using=Author.objects.all().db)
    )

If the form media is loaded correctly, the field should use the nice autocomplete widget you get in an edit form when you use autocomplete_fields on a ModelAdmin instance. When it isn't, you see a standard select field without any options (because the JS to load them isn't included).

Hey! Is this safe to be merged to master?

I would be OK with including this but I would like to understand exactly how it works.  Can anyone supply steps to reproduce / screenshots / or a more detailed explanation of what's going on.  I read Andy's comment but I'm still not clear on what the nature of this change is.

@matthewhegarty In a nutshell, Django has a mechanism that allows form widgets to register additional js and css so that they can function properly. However, if you do not render form.media (a collation of all the JS and CSS required for all fields/widgets used for that form), that JS and CSS is unavailable in the front-end, and widgets do not look or function correctly.

@ababic Andy - thanks for the explanation, I understand what's going on now.  I was able to reproduce the issue in the v3 codebase as follows (adapted from Andy's post):

forms.py

class AuthorFormMixin(forms.Form):
    author = forms.ModelChoiceField(
        queryset=Author.objects.all(),
        widget=AutocompleteSelect(Book._meta.get_field('author'), admin.site)
    )

admin.py

class AuthorAdmin(ImportMixin, admin.ModelAdmin):
    search_fields = ['name']

class CustomBookAdmin(BookAdmin):
    """BookAdmin with custom import forms"""
    autocomplete_fields = ['author']

(then add the PR changes to import.html)
This results in the correct autocomplete widget being show, with 'author' values being dynamically populated.

I've updated this PR with merges from v3.
I have tested this change manually in the admin site.

Thank you @andreynovikov and @ababic

Thanks @matthewhegarty! Happy Easter to you :)


Add HTTP status codes to docs

Feature request
We should add a description of the HTTP status codes to our docs
Motivation
Users can be confused when using lorax if they see random status codes like 424.

Thank you noah!


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Currently there's a failure on windows: https://gitlab.com/MatrixAI/open-source/js-logger/-/jobs/2748280466.
[Error: ENOTEMPTY: directory not empty, rmdir 'C:\Users\GITLAB~1\AppData\Local\Temp\logger_handlers-GtJEbz'] {
  errno: -4051,
  code: 'ENOTEMPTY',
  syscall: 'rmdir',
  path: 'C:\\Users\\GITLAB~1\\AppData\\Local\\Temp\\logger_handlers-GtJEbz'
}

It's possible that fs.rm with recursive doesn't work on windows.

Pipeline Attempt on 593021978 for e2985dd6d851b1d83d8c8c98cbdf260b7abc8ac5
https://gitlab.com/MatrixAI/open-source/js-logger/-/pipelines/593021978

Pipeline Attempt on 593035381 for 42829046c9ab2a4a8933a7d0a2868ca135627608
https://gitlab.com/MatrixAI/open-source/js-logger/-/pipelines/593035381

Pipeline Attempt on 593035969 for 8a1713a052c240d9277d6f881406587e4578a4b5
https://gitlab.com/MatrixAI/open-source/js-logger/-/pipelines/593035969

Pipeline Succeeded on 593035969 for 8a1713a052c240d9277d6f881406587e4578a4b5
https://gitlab.com/MatrixAI/open-source/js-logger/-/pipelines/593035969

The windows problem is strange, but it fixed after I both forced the deletion and also ensured that I closed the standard stream. Windows does require standard streams to be closed prior to deleting the directory, but on matrix-win-1, it was a EPERM error not ENOTEMPTY. At any case it's passing now...


Update bundler deps

Changes

Consolidate the uglify config into a common constant.
Update babel-preset-env, gulp-clean-css, gulp-header, gulp-imagemin, gulp-replace, gulp-sourcemaps, uglifyjs-webpack-plugin to latest.

Testing

./frontend.sh should pass.
Site should function the same.

@anselmbradford, 👍. This should allow us to enable proper elimination of unused imports. https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/125


Fix 'Back to top'.

Fix 'Back to Top' link which isn't working because of smooth scrolling for anchor links.

Fixes #66

fixed here https://github.com/olamy/reflow-maven-skin/pull/10


pass qos_tags to create_request

 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.


tunnel flags doesn't support flags as sauce-connect-launcher

i put my username and key in the tunnel.json file like it's possible there: sauce-connect
{
  "username": "{{ sauce_username }}",
  "accessKey": "{{ sauce_access_key }}",
  "connectVersion": "4.4.7",
  "noSslBumpDomains": "*.toucantoco.com,*.toucantoco.guru"
}

but i get theses errors
[ERROR] [Saucelabs Executor] Error! Sauce requires username to be set. Check if the environment variable $SAUCE_USERNAME is defined.
[ERROR] [Saucelabs Executor] Error! Sauce requires accessKey to be set. Check if the environment variable $SAUCE_ACCESS_KEY is defined.
[WARN] [Saucelabs Executor] Warning! No connectVersion is set. This is set via the environment variable $SAUCE_CONNECT_VERSION . This isn't required, but can cause problems with Sauce if not set
[ERROR] [Magellan] There are errors in loading executors
[ERROR] [Magellan] Error: Missing configuration for Saucelabs connection.

i forget to put --sauce_tunnel_config tunnel.json sorry


ImportError: No module named usb.core

I have:

[X ] searched open and closed issues for duplicates

Bug description
Hi, I'm using Flask/Docker.  I add
from from escpos.printer import Usb (as my line 2)
the following error occurs:
web_1  |   File "/web/web/blueprints/webhook/views.py", line 2, in <module>
web_1  |     from escpos.printer import Usb
web_1  |   File "/usr/lib/python2.7/site-packages/escpos/printer.py", line 3, in <module>
web_1  |     import usb.core
web_1  | ImportError: No module named usb.core

Device info
I have a printer called a POS-8250.  It's a cheapy from china and is epson compatible but I don't think this is a printer issue.

python-escpos version: 0.0.0
python version: 0.0
Python 2.7.16
AND
Python 3.6.8
(not sure why both are there in the docker image, but I'm targetting 3.x)
operating system:
ubuntu 18.04

Just an update, this can be close.  I was not using the correct pip install.  In the end using the PROPER install via python-escpos in my requirements.txt solved this.

No problem, thank you for the update!


math.div changes applied to non-division variables

assets/stylesheets/bootstrap/_variables.scss and templates/project/_bootstrap-variables.sass have had some lines converted to math.div, but the original variables were not division (or were in comments.) Examples:
$container-desktop: (940px + $grid-gutter-width) !default;
is now
$container-desktop: math.div(940px + $grid-gutter-width) !default;
// Small screen / tablet
is now
// Small screen, tablet
This causes a sass error<EMAIL_ADDRESS>Message:
    node_modules/bootstrap-sass/assets/stylesheets/bootstrap/_variables.scss
Error: Missing argument $number2.
  ┌──> node_modules/bootstrap-sass/assets/stylesheets/bootstrap/_variables.scss
338│ $grid-float-breakpoint-max: math.div($grid-float-breakpoint - 1) !default;
  │                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ invocation

Pushed v3.4.3 with the fix (https://github.com/twbs/bootstrap-sass/pull/1225)

We're still seeing something like this on v3.4.3:
SassC::SyntaxError: Error: Invalid CSS after "  margin: (math": expected expression (e.g. 1px, bold), was ".div($line-height-c"
        on line 8:16 of node_modules/bootstrap-sass/assets/stylesheets/bootstrap/mixins/_nav-divider.scss
        from line 25:1 of node_modules/bootstrap-sass/assets/stylesheets/bootstrap/_mixins.scss
        from line 7:1 of app/assets/stylesheets/legacy/variables/_manifest.scss
        from line 1:1 of app/assets/stylesheets/admin.scss
>>   margin: (math.div($line-height-computed, 2) - 1) 0;

/cc @AprilArcus

Looking

@mockdeep This error appears to be due to the use of SassC instead of dart-sass. SassC wraps libsass, which does not support the math.div syntax. SassC and libsass are deprecated.
If compatibility with SassC and libsass are necessary, I recommend you remain on 3.4.1. Otherwise, I recommend you upgrade your sass compiler to an appropriate version of dart-sass (i.e. "sass": "^1.33") for future-proofing.

Okay, thanks, will do.


Duplicate errors in RDS service definition

The RDS service definition contains a couple errors lists with duplicate entries.  Specifically:

RestoreDBClusterFromSnapshot has StorageQuotaExceededFault and DBSubnetGroupNotFoundFault twice
RestoreDBClusterToPointInTime has StorageQuotaExceededFault and DBSubnetGroupNotFoundFault twice

Not a big deal to work around, but figured I'd log the issue.

Thanks for reporting! I've informed the service team about the issue. I'm closing out the issue here, but feel free to report if you find any other such cases.


Changed synchronous refresh to async

Hi, thanks for the proposal. I'll take a look, sorry project has been a bit neglected lately

LGTM. I had actually started implementing this a bit differently, but this implementation addresses the current problems well enough.
Thanks for the contribution.


SQL-161 add unix socket support

SQL-161 Per https://github.com/yetanalytics/lrsql/issues/235 we'd like to be able to support the use of unix sockets with junixsocket.
This PR adds junixsocket at version 2.6.1 and provides some documentation on usage.

Is it possible to add a test for this?

Is it possible to add a test for this?

It would be possible to test using testcontainers + docker compose I suppose, but it's a lot of work to just test the underlying lib which is already tested.

It would be possible to test using testcontainers + docker compose I suppose, but it's a lot of work to just test the underlying lib which is already tested.

Okay, even then it should be simple enough to test manually by passing in that JDBC URL, which I'm assuming you already did on your end.

Yep, in the course of troubleshooting https://github.com/yetanalytics/lrsql/issues/235


crypto/ed25519: support signature verification

Add support for ed25519 signature verification of message based on
a public key.
fixes #331
Signed-off-by: Dominik Roos<EMAIL_ADDRESS>

I was not sure how to do the testdata file.
If you have a better pattern for generating the golden file, let me know.

re: golden file, there is a way to run the testsuite in update mode (https://github.com/cue-lang/cue/blob/a9b13e823e76986cf8722edf04174f8ee10c525c/internal/cuetxtar/txtar.go#L323) not sure how to invoke that myself though...
Seems like a good topic to add to https://github.com/cue-lang/cue/wiki/Contribution-tips-and-tricks, and generally guidance on writing such tests

Oh, I see. It is done via env var here: https://github.com/cue-lang/cue/blob/a9b13e823e76986cf8722edf04174f8ee10c525c/internal/cuetest/cuetest.go#L56
Hm, but that update does not really fit, because we need to generate valid signatures. But we could get rid of the manual output generation.
Thanks for the hint 👍
I will play around with.it tomorrow.

I updated the test to use cuetest.UpdateGoldenFiles
To update the golden files, you can simply run the following:
CUE_UPDATE=true go test ./pkg/crypto/ed25519/

Thanks @oncilla - @mpvl and I are, independently, taking some vacation at the moment so we will get back to this on our return.

:tada:


SecDefaultAction with 'nolog' doesn't work on IIS.

Manually specifying nolog as part of SecRuleUpdateById and as part of the rule seem to work, but when specified as SecDefaultAction has no effect.

I was able to get this working, not sure what the issue was previously, perhaps order... I used SecDefaultAction "phase:2,nolog,auditlog,pass"


fix(MetadataCardContent): expose subtitle property

Description
Exposes the subtitle property that exists in MetadataBase in MetadataCardContent.
References
LUI-1574
Testing
Ensure that adding a subtitle in the MetadataCardContent story adds the text to the component and shifts the description and descriptionDetails accordingly.
Automation
N/A
Checklist

[x] all commented code has been removed
[x] any new console issues have been resolved
[x] code linter and formatter has been run
[x] test coverage meets repo requirements
[x] PR name matches the expected semantic-commit syntax

Test Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.22.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Condition Check Improvement: Origininating Actor data

in an active auras conditional check, is there syntax I can use to check the originating actor instead of other actors within the aura?

Describe the solution you'd like
Ability to access aura's originating actor data for condition evaluation
Describe alternatives you've considered
From discord:

I was creating a shield wall automation for a user where they wanted tokens next to each other to get some bonuses if they had a shield. I was able to do most of it in the aa check with

token.actor.items.filter(i => i.name.toLowerCase().includes('shield') && i.system.equipped === true).length > 0 && !token.actor.effects.filter(e => e.name.toLowerCase().includes('blinded') || e.name.toLowerCase().includes('surprised')).length > 0

That works to check the tokens next to the actor, but can't check the aura token itself to make sure that it has a shield. So it won't receive the bonus but it can give the bonus to others when it shouldn't. My goal was to be able to do checks against self for validation essentially. Now that I'm thinking about it I could probably use the midi item use activation condition if it's too much trouble to be able to do that in the aa condition
Midi's activation conditions don't work on effects that are transfer to actor on item equip, so not a great option there

Done.


Fix Permission denied & Export issue

Related issue: https://github.com/earnestt1234/wubwub/issues/1

Sorry for the delay!  Thanks for your help on this : )
I changed the method for removing the leading period, otherwise kept your changes.  I also added a fmt parameter to export, so you manually specify the filetype if desired.


Hide code in Czech legislation

I would like to ask if it is possible to allow showing code for Czech legislation extensions in NEXT versions. We can't suggest events and other changes for next version before release.
If we download snadbox for BC18 we can't see code, but if we download onprem container we have there zip files with codes. Problem is that NextMajor and NextMinor version can be download as sandbox only and another problem is that Czech legislation extesions have hide code. It means that "Go to definition" does not work. We have to see the code otherwise it is not possible to develop our products that are dependent on Czech legislation.

This is already done for 19.0


detect possible sql injections

closes https://github.com/FriendsOfREDAXO/rexstan/issues/109
see testcases in
https://github.com/FriendsOfREDAXO/rexstan/blob/sqlinjection/tests/data/sql-injection.php

good enough for early feedback


Add support for parsing v8 >=8.6 IC format

Fixes #24 .

You may need to merge from master as I've done some updates there. Let me know if you don't have time. Happy to do it for you.


[Bug]: Error in uploadWebResource

What happened?
There is no possibility to upload and publish my JS Code.
What version of the tool are you using?
v2.2.1
What version of Visual Studio Code are you running?
No response
Any relevant session id from VSCode?
acd4a751-23ed-4028-946d-a8819cb859241733908322778
Relevant log output
VSCode version: 1.95.3.
Extension version: 2.2.1.
Error occured in uploadWebResource command.
Error code: undefined.
Error message: Cannot set properties of undefined (setting '@_version').
Error stack: TypeError: Cannot set properties of undefined (setting '@_version')
	at t.xmlToJSON (c:\Users\YasinGörgülü\.vscode\extensions\danish-naglekar.dataverse-devtools-2.2.1\dist\extension.js:2:801897)
	at t.WebResourceHelper.<anonymous> (c:\Users\YasinGörgülü\.vscode\extensions\danish-naglekar.dataverse-devtools-2.2.1\dist\extension.js:2:762308)
	at Generator.next (<anonymous>)
	at o (c:\Users\YasinGörgülü\.vscode\extensions\danish-naglekar.dataverse-devtools-2.2.1\dist\extension.js:2:751513).

I'm experiencing the same issue. Relevant log informaiton below:
VSCode version: 1.96.0.
Extension version: 2.2.1.
Error occured in linkExistingWebResource command.
Error code: undefined.
Error message: Cannot set properties of undefined (setting '@_version').
Error stack: TypeError: Cannot set properties of undefined (setting '@_version')
at t.xmlToJSON (c:\Users\msan.vscode\extensions\danish-naglekar.dataverse-devtools-2.2.1\dist\extension.js:2:801897)
at t.WebResourceHelper. (c:\Users\msan.vscode\extensions\danish-naglekar.dataverse-devtools-2.2.1\dist\extension.js:2:762971)
at Generator.next ()
at o (c:\Users\msan.vscode\extensions\danish-naglekar.dataverse-devtools-2.2.1\dist\extension.js:2:751513).

I'll look into this today. Thanks for raising this to my attention

I was not able to replicate the issue on my end based on what I understood the problem was. Can someone share the dvdt.linker.xml file so I can use that to replicate the issue?

The new version is released with the fix. Make sure to upgrade the version


Basic question about the use of my_policy

Hi,
Thanks for creating this amazing functionality!
I have a basic question about the use of policy functions to set certain precision levels.
As stated in the example of your README.md:
def layer(params, x):
  params, x = my_policy.cast_to_compute((params, x))
  w, b = params
  y = x @ w + b
  return my_policy.cast_to_output(y)

I am new to JAX, but the first thing I learned is that JAX likes pure functions.
Does the use of my_policy violate the pure functions paradigm?
Should it become:
def layer(params, x, my_policy):
  params, x = my_policy.cast_to_compute((params, x))
  w, b = params
  y = x @ w + b
  return my_policy.cast_to_output(y)

The function can be jitted by using partial.
from functools import partial
layer_compiled = jit(partial(layer, my_policy=my_policy))

Fundamental question
A more fundamental question (which I maybe need to ask at JAX ) is:
How should functions as input to functions be handled in JAX?
func_a(x,w,func):
  ...

func_a_compiled = jit(partial(func_a, func=func_b))

In order to jit this, I came up with the partial solution above. I assume your use of my_policy is valid, as you probably have more experience with JAX. But that creates some magic, which is undesirable for my use case. Is the jit(partial()) solution valid or is there a better way to handle functions as input to functions?
Have a nice day! J

Hi @JSchuurmans , jmp.Policy is an immutable dataclass, so as long as you don't re-assign it (e.g. my_policy = something_else) in your program it is fine to close over it as a constant (and the behaviour under JAX transforms is well defined, the value is a constant).
The JAX solution for passing non-JAX types (e.g. functions) to jitted functions is to either use partial (as you have done) or to use static_argnums=(0, 1, ...) to specify which arguments JAX should treat as static values.
Another common pattern for this is to use objects to hold configuration, and to use jit on bound methods (in the constructor) of those objects:
class Experiment:
  def __init__(self):
    # Fine for JAX since this is a constant.
    self.policy = jmp.Policy(..)
    # This is equivalent to `jax.jit(partial(Experiment.forward, self))`.
    self.forward = jax.jit(self.forward)

  def forward(self, params, x):
    my_policy = self.policy
    params, x = my_policy.cast_to_compute((params, x))
    w, b = params
    y = x @ w + b
    return my_policy.cast_to_output(y)

e = Experiment()
e.forward(params, x)

@tomhennigan, thank you for answering both questions. The suggested object gave me a good idea for my implementation!


Moved the va_copy hack into the StandardCLibrary.h

Cleaned up C dependent code.

Coverage remained the same at 99.848% when pulling 6e0ee2d05a032e8b5bb3f4b3fbe74df100974c6a on basvodde:master into 3a6241263196e6a16c7e651e17916cadfac895bb on cpputest:master.


improve error message for mismatched pipelines

improve error message for mismatched pipelines
fix issue: #18040

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] #18040
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tvalentyn
could I ask a question that why there are two checks were failed?

Apologies for the long response, I was OOO for a bit. Thanks for the fix!
Precommit errors are most likely a flake, we can re-run the suite. Lint errors is visible if you follow the "Details" -> Console Output link. The error is:
apache_beam  conftest.py  gen_protos.py  setup.py
Running pylint...
************* Module apache_beam.transforms.util
apache_beam/transforms/util.py:238:0: C0301: Line too long (88/80) (line-too-long)

--------------------------------------------------------------------
Your code has been rated at 10.00/10 (previous run: 10.00/10, -0.00)

see also: s.apache.org/beam-python-dev-wiki which talks about how to run linter locally during development.

Linter errors should be fixed, we can merge once tests run.


chore Add detailed msg of checking endpoint failed

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Set a reasonable initial max heap size

Some users report problems with initializing JVM under
certain circumstances:
Could not reserve enough space for object heap

(See travis-ci/travis-ci#3805)
We use JAVA_TOOL_OPTIONS to set the maximum heap size in the hopes
that this will eradicate the problem.
Notice that the command line options (e.g., java -Xmx1g …) will override JAVA_TOOL_OPTIONS.

Will this also be applied for android builds?

@meatballhat Yes. But for Android we would need additional tweaks, since android-update-sdk script would need to create JVM to install components, and this can also fail.


Banner Checkout Extension Display None on Mobile

Please list the related package(s)
checkout-ui-extensions
Is your feature request related to a problem? Please describe.
I am trying to have checkout disclaimers in my Checkout and I am using Banners to do it. So far everything works fine, but I am trying to make an option on the client side where if selected, the Banner doe not show up in mobile. I cannot seem to add styling through css or capture screen size.
Describe the changes you are looking for
I would like to be able to add classNames to the banner so I can style and use mobile queries.
Describe alternatives you’ve considered
I have tried returning Null if clicked which works, but not for specific screens. I want to only hide on mobile. I have also tried to use  but cannot make the banner disappear and cannot target based on screen size.
Additional context
`import React from "react";
import {
reactExtension,
Banner,
Style,
View,
useSettings,
} from "@shopify/ui-extensions-react/checkout";
// Set the entry points for the extension
const checkoutBlock = reactExtension("purchase.checkout.block.render", () => );
export { checkoutBlock };
function App() {
// Use the merchant-defined settings to retrieve the extension's content
const { title: merchantTitle, description, collapsible, status: merchantStatus, hide_banner, hide_banner_mobile } = useSettings();
// Set a default status for the banner if a merchant didn't configure the banner in the checkout editor
const status = merchantStatus ?? 'info';
const title = merchantTitle ?? 'Custom Banner';
if (hide_banner_mobile) {
return null;
}
if (hide_banner) {
return null; // Do not render the banner if hide_banner is true
}
return (
<View
maxInlineSize={Style.default(200)
.when({viewportInlineSize: {min: 'small'}}, 300)
.when({viewportInlineSize: {min: 'medium'}}, 400)
.when({viewportInlineSize: {min: 'large'}}, 800)}

      {description}

  );
}
`

Duplicate of https://github.com/Shopify/ui-extensions/issues/886. Feedback and usecases welcome there.


[SPARK-37452][SQL] Char and Varchar breaks backward compatibility between v3.1 and v2

What changes were proposed in this pull request?

We will store table schema in table properties for the read-side to restore. In Spark 3.1, we add char/varchar support natively. In some commands like create table, alter table with these types,  the char(x) or varchar(x) will be stored directly to those properties. If a user uses Spark 2 to read such a table it will fail to parse the schema.
FYI, https://github.com/apache/spark/blob/branch-2.4/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala#L136
A table can be a newly created one by Spark 3.1 and later or an existing one modified by Spark 3.1 and on.
Why are the changes needed?

backward compatibility
Does this PR introduce any user-facing change?

That's not necessarily user-facing as a bugfix and only related to internal table properties.
How was this patch tested?

manully

Test build #145576 has started for PR 34697 at commit 1934285.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50049/

Test build #145580 has started for PR 34697 at commit 799a761.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50049/

Test build #145576 has finished for PR 34697 at commit 1934285.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

cc @cloud-fan thanks

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50049/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145576/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50052/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50052/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50052/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145580/

Test build #145603 has started for PR 34697 at commit 0334309.

Test build #145603 has finished for PR 34697 at commit 0334309.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50075/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145603/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50075/

Test build #145605 has started for PR 34697 at commit d3a284f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50079/

Test build #145605 has finished for PR 34697 at commit d3a284f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145605/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50079/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50079/

FYI @xkrogen, @shardulm94

any more concerns from the CCers？


UiDemo not working

Hey,
first of all, thank you very much for this library!
Unfortunately I can't get the UiDemo to run. The display shows nothing. However, the other OLED demos work.
Heltec Wifi LoRa 32 V3 (HTIT-WB32LAF)
Arduino IDE 2.3.3
Library version 0.9.2
Ubuntu 22.04

it works if you add the command at the beginninig heltec_setup().
void setup() {
  heltec_setup();
  Serial.begin(115200);
  Serial.println();
  Serial.println();
......

Oh, whoops... Fixed...

(Also don't need the `Serial.begin then)
Thanks @Zep0 !


How to test the pre-trained models in my own images?

How to test the pre-trained models in my own images?

@ecetang see eval*.py, you should set PRED_DEBUG first

Hi @HiKapok  is it possible to get a pre-trained model for network, i would like to fine-tune it for my use case.
Thanks


Cheapest workaround for fluent UI build bug

bug
when build production, fluent UI will generate a unique css class name for each component in HTML files and JS files. they use different namespace, but share the same counter globaly (e.g <namespace>-<counter>). The generated class name doesn't match (coutner cannot match) with each other in JS and HTML files, causes bad CSS in website.
workaround
Add used components in all pages in the same order, which make the counter consists.

what is the build bug?

what is the build bug?

updated the description with the bug


Best way to approach luacheck support for nginx.conf files

Hi
We use nginx openresty with lua support in our project.
Basically this allows me to use lua code in specific blocks defined inside of nginx.conf file.
For example:
log_by_lua_block { require("logger").log_stuff() }
content_by_lua_block { ... }
init_by_lua_block { ..
I would like to run luachecker on all of them to detect possible errors.
I could write a script that would extract all the lua blocks to a file and then run luachecker on each one of them, but personally I would prefer to interface with luachecker and perhaps write a custom parser for it to look only at the *lua_block stuff.
Cheers

Hi,
I'm not sure that luacheck itself should be able to parse and check nginx.conf files, it's a bit of feature bloating.
You could write a Lua script to extract the Lua blocks and check them using luacheck.check_strings() function from the public luacheck module, and then format warning messages using luacheck.get_message(). See luacheck module docs.

Thanks, I will do that!


Automatic bug reports

Create a new setting like logExceptionsMax =1000. <=0 means no logging
We can use the OSystem database on the server for them
The OrientDB server should write the exception that wraps at HTTP and Binary protocol level
Since we don't have capped clusters, the insert should check if the size() > logExceptionsMax, then stop to add new log reports.
Studio should have a form where you can browse inside this class and send the one or more exceptions in some way.
The file could be a zipped export of the records (we can use database export with the includeClass=ThatClass)
The best would be attaching the file to an open issue

I think, there also should be a really annoying notification or indication about new exceptions, so it will be easier to send them to us than just ignoring them :D


Studio docs v4

Closes https://github.com/iterative/dvc.org/issues/2515

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

@tapadipti there are some broken links https://github.com/iterative/dvc.org/pull/2571/checks?check_run_id=2943878811 - we need to fix them I think before we merge

Nice work @tapadipti 😎  Let's create a separate ticket to review and adjust the content.


Arp ndp control plane

Compilation is broken when stopping at: 5dfa958 - (HEAD) arp: reply to all local iface addresses (81 minutes ago) .
``
$ git rbi origin/main -x 'make'
...
../modules/ip/datapath/arp_input.c: In function ‘arp_input_process’:
../modules/ip/datapath/arp_input.c:134:28: error: implicit declaration of function ‘arp_mbuf_data’; did you mean ‘mbuf_data’? [-Werror=implicit-function-declaration]
134 |                 arp_data = arp_mbuf_data(mbuf);
|                            ^~~~~~~~~~~~~
|                            mbuf_data
../modules/ip/datapath/arp_input.c:134:26: error: assignment to ‘struct arp_mbuf_data *’ from ‘int’ makes pointer from integer without a cast [-Werror=int-conversion]
134 |                 arp_data = arp_mbuf_data(mbuf);
|                          ^
../modules/ip/datapath/arp_input.c:135:25: error: invalid use of undefined type ‘struct arp_mbuf_data’
135 |                 arp_data->local = local;
|                         ^~

With this last issue fixed, the series lgtm.

Should be OK now. I have tested:
git rebase --exec 'make all lint unit-tests && sudo smoke/run.sh build' upstream/main

Seems ipv6 is broken: there is no neighbor solicitation response (packets are "hold" in ip6_output), and no next hop is created from the control plane.


Cache-Control: no-store only while upload is incomplete

Once upload is complete caching should be allowed.

Please be aware that the specification does only required the Cache-Control: no-store header for HEAD responses as the GET request/response is not defined by the protocol.
Even after an upload is finished, the HEAD response should not be cached as certain properties of the upload still may change, e.g. its expiration date or it may become totally unavailable. Therefore, I think, the HEAD response should not be cached.

as certain properties of the upload still may change

That is true. However, I think this should be a SHOULD and not a MUST once the upload has completed. The server might be able guarantee that it will not change if e.g. there is no expiration date.

Agreed, closing in favor of #121.


Unable to delete VPC with active VPC endpoints

When attempting to delete a subnet with an active VPC endpoint, I'm seeing failures due to a DependencyViolation error, causing the whole VPC to fail to delete.
Sample output from me trying to delete a single VPC with a single subnet which contains 10 VPC endpoints:
[cloud-nuke] INFO[2022-03-03T22:00:35Z] Terminating 1 resources in batches
[cloud-nuke] INFO[2022-03-03T22:00:35Z] Deleting all VPCs
[cloud-nuke] INFO[2022-03-03T22:00:35Z] Nuking VPC vpc-01d0f9365b69337c8 in region eu-west-2
[cloud-nuke] INFO[2022-03-03T22:00:36Z] ...no Internet Gateway found
[cloud-nuke] INFO[2022-03-03T22:00:36Z] ...deleting subnet subnet-0d062e6195500ab2c
[cloud-nuke] ERRO[2022-03-03T22:00:36Z] Error cleaning up Subnets for VPC vpc-01d0f9365b69337c8: DependencyViolation: The subnet 'subnet-0d062e6195500ab2c' has dependencies and cannot be deleted.
	status code: 400, request id: c00b3f50-3281-4d51-848e-accac2f54340
[cloud-nuke] ERRO[2022-03-03T22:00:36Z] [Failed] DependencyViolation: The subnet 'subnet-0d062e6195500ab2c' has dependencies and cannot be deleted.

Once all the endpoints were deleted, the subnet, and eventually the VPC, were successfully nuked.
Reordering the execution here to put nukeEndpoints before nukeSubnets should resolve this issue. Correct me if I'm wrong, but I don't think there should be anyDependencyViolations introduced by moving the endpoints nuke to be the call immediately before nuking the subnets.
It also looks like VPC endpoint deletion takes a while, and the subnet deletion won't succeed until these endpoints are  all cleared, so perhaps some polling will be needed to check the endpoints are actually gone before continuing (has something similar been done for other resources?).
If that fix sounds sensible, I'd be happy to put together a PR.

@sHesl you're welcome to take a stab at it!


[CI Environment] Do not delete tags that already exist on subscription level

Description
Currently, whenever the Microsoft.Resources/tags module is tested by the CI environment, it will delete the entire tags object, which in most cases, is not desirable.
This change gets all the previous tags from Azure and after deletion adds them back without the test tags.
Behaviour:

.\Invoke-ResourceRemoval -Type 'Microsoft.Resources/tags' -ResourceId '/subscriptions/6cc9c28f-2452-4c50-832d-187cad67447b/providers/Microsoft.Resources/tags'

Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline

Type of Change
Please delete options that are not relevant.

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (readme)
[x] I did format my code

@microsoft-github-policy-service agree company="Hogeschool Utrecht"

@microsoft-github-policy-service agree


[Bloc] MacOS horizontal scrolling is inverted now

Due to the upstream fix (https://github.com/rust-windowing/winit/commit/e2cf2a5754f6ee51a37fbe6e1f8b5ff8bd98da49) we should change the sign of the horizontal scrolling deltas

Moving two fingers on the touchpad:

right to left results in positive x component
left to right results in negative x component

The x coordinate of the scrolled content when moving two fingers on the touchpad:

right to left decreases x coordinate
left to right increases x coordinate


Highlighting over already highlighted text selects a lot of unwanted text

On chrome, when selecting text before a highlighted text, and hovering through the highlighted text, the selection suddenly includes all text from the initially selected text until the end of that page.
The issue happens with both TextHighlight and AreaHighlight, with and without MonitoredHighlightContainer.
Attached video of the behavior
https://github.com/user-attachments/assets/3d6155ce-3953-47f0-93cb-98da83d1ff81

facing this same prob. even w the latest versions of react-pdf-highlighter library.

This is caused by the the highlight parts having their own pointer events, so a fix would require disabling those events when a user is making a selection. This is annoying since it means highlights need to be aware of selection state which could cause a lot of re-renders. I've currently got it working by promoting isSelectionInProgress to state in PdfHighlighterContext.
There is also an aesthetic issue of how a selection should cover a highlight. Currently I've defaulted to reducing opacity when a selection is being made, but I'm open to changing this or making it configurable.


The window freezes when clicking on the title bar when "ExtendClientAreaToDecorationsHint = true" (Windows / 0.10.18)

Describe the bug
When ExtendClientAreaToDecorationsHint = true, the Window freezes (ie: the rendering seems to be temporary halted) when the title bar is pressed.
This is particularly visible if something is moving on the screen (like an indeterminate progress bar), or if the client-drawn title bar has a background that changes color when the window is focused.
To Reproduce
Steps to reproduce the behavior:

Create a Window with ExtendClientAreaToDecorationsHint = true
Add an indeterminate progress bar to it
Click on the title bar (without releasing the mouse button)
See the progress bar being temporarily frozen

You can also:

Apply a style to the title bar to make its background color change when the Window is focused
Unfocus the Window
Click on the title bar (without releasing the mouse button)
See the title bar's background color not changing instantly when the Window is focused

Expected behavior
The rendering should continue to be updated when the title bar is pressed (ie: indeterminate progress bars should continue moving, and the title bar's background color should change instantly when appropriate)
Desktop (please complete the following information):
OS: Windows 10 21H2
Additional context
In a development environment, the freeze is relatively minor: it only lasts ~1 second (or exactly until the user starts dragging the window), and then the window's rendering resumes as usual (even if the title bar is still pressed)
However, when publishing the application for the win-x64 platform, the behaviour changes: when clicking on the title bar, the window's rendering is halted until the bar is released, even if the user starts dragging it. In my tests, I published my project as a framework-independant win-x64 executable (it didn't matter if a single file was produced or not, or if native libraries were included for self-extraction or not)
I tracked down the issue to the Avalonia.Angle.Windows.Natives package (0.10.18 uses version <IP_ADDRESS>0091801). It contains a native library called av_libglesv2.dll, in the runtimes/win7-x64 directory. It seems to be loaded in the dev environment, but is not included when the application is published for the win-x64 platform.
I created a modified Avalonia.Angle.Windows.Natives package that contains the same library, but in the runtimes/win-x64 directory, and it did get included when I published the app for win-x64. As a result, I got the same, less severe behaviour as I got in my dev env.
An interesting side effect, though, was that the memory usage of the app increased. As stated in #5646 (by Oxygels), Avalonia uses ~100 Mo of RAM in its cross-platform state (ex: in the development environment), but has a significantly lower memory usage when published for win-x64 (In my case, I got it down to ~60 Mo). It seemed the lower memory usage was caused by the app not using the av_libglesv2.dll lib bundled with Avalonia. I'm guessing it either used a version of the lib included in the OS, or didn't use it at all (if it's even possible).

I was wondering if I have made a custom title bar extended to the decorations and an indeterminate progress bar. Neither in debug nor in release mode does it get stopped while dragging the window from my own title bar.
I may add I have implemented custom dragging and custom buttons on windows 11.

I was wondering if I have made a custom title bar extended to the decorations and an indeterminate progress bar. Neither in debug nor in release mode does it get stopped while dragging the window from my own title bar. I may add I have implemented custom dragging and custom buttons on windows 11.

Interesting. I guess the problem lies in the code handling the default behavior of the title bar.
On another note, the part about av_libglesv2.dll seems to be related to #6028.

I hit the same issue immediately after I published my app using dotnet publish -c Release -r win-x64.


Debugger 9000

Test it: https://maker.makecode.com/app/7cbac7abf891b2b2c6d68b6b9d04c296e48d2161-6082f7e3cd

[x] slow-mo replaced by debugger toggle button (needs better look)
[x] remove stop button (debugger toggle button)
[x] move slow-mo in debugger toolbar
[x] disable toolbar start/stop/restart while debugging
[x] auto step-into until a block/text is found
[ ] reconcile debugging with tracing: we should be able trace and pause (@riknoll @mmoskal  I'll need your help on this)
[x] typescript stepping
[x] in blocks, ensure that all steps land on a block?
[x] move category table under sliding menu
[x] add yellow halo around modified variable

Yes that sounds better actually, you have the “play mode” (first button) which is doing tracing. once you pause, you can step, and then restart.

From: Peli de Halleux<EMAIL_ADDRESS>Sent: Sunday, April 15, 2018 8:21:07 PM
To: Microsoft/pxt
Cc: Sam El-Husseini; Review requested
Subject: Re: [Microsoft/pxt] Debugger 9000 (#4134)
Actually in blocks we could simply get rid of the slow-mo button since it's always on unless you step
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F4134%23issuecomment-381469347&data=02|01|Sam.El-Husseini%40microsoft.com|01892b7933c64cb54c6408d5a3491871|72f988bf86f141af91ab2d7cd011db47|1|0|636594456698437605&sdata=V7rDN9TVOAYwHJwEgGmHi%2BNdVC0I%2F6kjukWUrZB0cPg%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAP6rzEl4hFuc80AgrX8itkmMFFzmUsOOks5tpA4jgaJpZM4TVzWU&data=02|01|Sam.El-Husseini%40microsoft.com|01892b7933c64cb54c6408d5a3491871|72f988bf86f141af91ab2d7cd011db47|1|0|636594456698437605&sdata=p398lnLAWpxfXTf6NIIs5q2iE7ZloA8OfXJJ8agQo64%3D&reserved=0.

Can we have a staged release for this feature? Lets enable in one target first (arcade\maker makes most sense for me) and iterate. Documentation and user testing are required before we enable this broadly.

Usage of tracing is quite low but sure we can keep the flag

From: Sam El-Husseini<EMAIL_ADDRESS>Sent: Monday, April 16, 2018 9:59:09 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Author
Subject: Re: [Microsoft/pxt] Debugger 9000 (#4134)
Yeah it's behind a target flag currently, but good point. @pelihttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpeli&data=02|01|jhalleux%40microsoft.com|e28868a8da3c47e353a108d5a3bb5feb|72f988bf86f141af91ab2d7cd011db47|1|0|636594947515467566&sdata=PyGjfDxwk0JSObscZKZdWsItQfYzi6ewYKwPRKbu4Fk%3D&reserved=0 If you plan on changing the behaviour of the slow mo icon or even the icon itself, you'll need to support both scenarios. If targets have the tracing flag on: do the old behavior, if they have the debugger flag: do the new behaviour.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F4134%23issuecomment-381674972&data=02|01|jhalleux%40microsoft.com|e28868a8da3c47e353a108d5a3bb5feb|72f988bf86f141af91ab2d7cd011db47|1|0|636594947515477575&sdata=JXwZNDWywAw7p1Xwa4IH0Gk%2FEhxEjzKhMwx316oZQ3A%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KUFXNHtrLAlCX40ra-tepgvZI20Jks5tpM3dgaJpZM4TVzWU&data=02|01|jhalleux%40microsoft.com|e28868a8da3c47e353a108d5a3bb5feb|72f988bf86f141af91ab2d7cd011db47|1|0|636594947515477575&sdata=sH3iJbFxsHyDCMMFRaH%2BB%2F%2BpnxTCXYGBez15PncV1vo%3D&reserved=0.

@abchatra tracing is back if "enableTrace" is true.


[chore] Easier instructions and setup

What changed? Why?
Easier integration and docs
Notes to reviewers
How has it been tested?

Niiice!


Added language translations of the answers form

Description:
Added English translations of the answers form and fix a grammar problem
Linked issues:
#143 and #141
Screenshots:

How to test:

Pk se ve ese titulo con Underscore _ en tu PR ?
@JoseFcoEstrella


Support keccak func for concrete data

related to #734, one more bug needs to be fixed in it.

Why didn't you replace the keccak function manager with BitVecFuncs, and solve the problem that way?

I happen to remove 2 if conditions and reindent, and thats the majority of it :smile:(which made it this big) . I am currently adding that change of replacing BitVecFuncs to handle keccak + replacing storage with hash of expr than a string.(which had some serious problems)

closing this as #1030 is more complete version of this.


fix: preserve default options when adding new translations

Description
Refactored language modules to merge with existing default translations in videojs.options.language. This prevents overriding default configurations when calling videojs.addLanguage. Changes have been applied across all packages in the monorepo and the templates for the generation of new packages.
Changes Made

Preserve existing translations when adding package specific keys.
Removed spaces from the skip button translation keys.

Checklist

[x] I have followed the project's style and contribution guidelines.
[x] I have performed a self-review of my own changes.
[x] I have made corresponding changes to the documentation.
[x] I have added tests that prove my fix is effective or that my feature works.

We have found that two javascript build tools produce two different results when json files are imported with the keyword import.
This demonstrates the unnecessary complexity of the javascript ecosystem. If we add minification (which can also produce unexpected results), TypeScript and so on, we're satellized on uranus.
That's it, I just needed to share a personal emotion.


performance: dua-cli full scan takes way longer than gdu

Directory scanning with dua i /some/folder takes orders of magnitude longer compared to gdu even when setting -t <some-number-bigger-than-number-of-cpu-cores>.
Didn't do any proper benchmarks, but just an example, while dua shows progress info with number of scanned files around 64k gdu in the same time on the same folder reaches around 300k+ files. dua-cli takes minutes longer to complete a full scan.
The huge speed difference has been observed with APFS (macos), HFS+ (macos), exFat (macos, linux), EXT4 (Linux with both armv7/arm64 and intel cpu's).
Note: dua-cli is still as fast or faster than ncdu, so apparently it's gdu that does some serious optimizations to speed up the scan. On macos APFS gdu full scan takes less time than calling ootb Apple's Finder Get Info on the same folder.

Thanks for reporting!
Can you try to use hyperfine and see the impact of the thread-count on performance? Note that I threw in pdu as well as it usually is the fastest way to iterate.
root=<path-to-measure>
hyperfine -N -w1 -M2 "gdu $root" "dua -t1 $root" "dua -t2 $root" "dua -t4 $root" "dua -t8 $root" "pdu $root"

The theory is that dua uses too many threads which can actually hurt performance on MacOS, and I noticed that 3 to 4 threads is usually giving the best performance. Maybe there is a number that is bringing it closer to gdu. Lastly, pdu is typically faster than dua and I'd expect it to be as fast as gdu or faster. Please note that it has flags for thread-counts as well, in case you want to dive deeper if the results are interesting.
Also note that this uses the non-interactive version of dua which uses the same traversal engine under the hood.

@Byron
hyperfine results
linux arm64 ext4 (772.98 GiB total)
Summary
  'gdu /media/t12/Music' ran
    1.07 ± 0.01 times faster than 'dua -t2 /media/t12/Music'
    1.13 ± 0.00 times faster than 'dua -t4 /media/t12/Music'
    1.31 ± 0.02 times faster than 'dua -t8 /media/t12/Music'
    1.49 ± 0.01 times faster than 'dua -t1 /media/t12/Music'

macos APFS (78,48 GiB total)
  dua -t8 ~/projects ran
    1.08 ± 0.00 times faster than pdu ~/projects
    1.30 ± 0.00 times faster than dua -t4 ~/projects
    1.50 ± 0.01 times faster than gdu ~/projects
    2.16 ± 0.00 times faster than dua -t2 ~/projects
    3.94 ± 0.02 times faster than dua -t1 ~/projects

The non-interactive dua mode is performing great, i.e. dua  -t8 ~/projects is very fast and finishes within . The slowness is observed with interactive mode with e.g. dua  -t8 i ~/projects which takes almost forever. Not sure what would be the hyperfine command for testing interactive mode as I suppose it probably cannot handle tty mode (?)

It's very interesting that gdu manages to be this much faster on Linux, and thread-scaling doesn't seem to do dua much good with -t2 being the best value on a 4-core machine.
On MacOS it scales much better, but the question remains why it's slow in interactive mode.
I have a hunch and implemented a fix in #225, which you are invited to try out. If you'd say that the ~/projects folder as a lot of top-level entries, then my hunch might be true.
Something you could also check is how many threads gdu uses by default - it's entirely unclear to me why it's so much faster on Linux except that maybe it's related to internal inefficiencies during traversal which weigh dua down (see #224).

compiling for apple silicon on macos m2 throws an error while running cargo install dua-cli
error[E0446]: crate-private type `FilesystemScan` in public interface
  --> ~/.cargo/registry/src/index.crates.io-6f17d22bba15001f/dua-cli-2.27.2/src/interactive/app/state.rs:42:5
   |
27 | pub(crate) struct FilesystemScan {
   | -------------------------------- `FilesystemScan` declared as crate-private
...
42 |     pub scan: Option<FilesystemScan>,
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't leak crate-private type

For more information about this error, try `rustc --explain E0446`.
error: could not compile `dua-cli` (bin "dua") due to previous error
error: failed to compile `dua-cli v2.27.2`, intermediate artifacts can be found at `/var/folders/py/73sb2fsj37xbmtkgw111l07w0000gp/T/cargo-installoMoXeN`.
To reuse those artifacts with a future compilation, set the environment variable `CARGO_TARGET_DIR` to that path.

same error when explicitly checking out the tag:
git clone https://github.com/Byron/dua-cli.git && cd dua-cli
git checkout tags/v2.27.2
cargo build --release

trie the Intel X86 binary from the releases:
/tmp/dua-v2.27.2-x86_64-apple-darwin/dua i ~/projects
Sort mode: size descending  Total disk usage: 149.07 GB  Processed 1743246 entries in 9.81s

the original m2-binary (v2.20.1 arm64) still shows scanning apparently even after scanning finished 🤔
Entries: 1 in 0s (472/s)  -> scanning <-                                                                  149.07 GB  Entries: 1743248 in 8.99s

compiling for apple silicon on macos m2 throws an error while running cargo install dua-cli

This is fixed now in main, see #226 .

the original m2-binary (v2.20.1 arm64) still shows scanning apparently even after scanning finished (although the number of entries is not identical) 🤔

This typically means that it is indeed still scanning, but all threads are stalled, presumably. I recommend to try again building the latest version. Let's see.

pulling, building and running latest main now makes dua -t8 i .. finish scanning in about the same time as gdu with just ~2.3 seconds difference on macos m2 (1744024 entries in 22.25s), on linux rpi 5 arm64 8GB RAM scanning a 765GB file system tree on a NVME m2 drive takes roughly equal time as gdu, hard to tell the difference
thank you so much for taking the time to look into this — much appreciated! 🙏

Thanks so much for letting me know, it's much appreciated, too :).
It's great to hear that the fix did indeed work, and that gdu isn't unconditionally faster anymore :).
Closing, as it sounds like this issue is no more.


Warnings in react-hotkeys when using React's strict mode

Hi!
React 16.3 added a new strict mode, which highlights potential issues in an app (such as those that will cause problems when using the upcoming async mode):
https://reactjs.org/docs/strict-mode.html
Enabling it for our app resulted in several errors coming from react-hotkeys:
Warning: Unsafe lifecycle methods were found within a strict-mode tree:
    in StrictMode (at job-view/index.jsx:28)

componentWillMount: Please update the following components to use componentDidMount instead:
HotKeys

-> see: https://reactjs.org/docs/strict-mode.html#identifying-unsafe-lifecycles
Warning: Legacy context API has been detected within a strict-mode tree:
    in StrictMode (at job-view/index.jsx:28)

Please update the following components: HotKeys

-> see: https://reactjs.org/docs/strict-mode.html#detecting-legacy-context-api
Warning: findDOMNode is deprecated in StrictMode.
findDOMNode was passed an instance of HotKeys which is inside StrictMode.
Instead, add a ref directly to the element you want to reference.

    in div (created by FocusTrap)
    in FocusTrap (created by HotKeys)
    ...

-> see: https://reactjs.org/docs/strict-mode.html#warning-about-deprecated-finddomnode-usage
Many thanks :-)

I'm using v2.0.0-pre4
I'm still getting this warning:
vendors~main.64d108ae760cbd63d1ea.bundle.js:138014 Warning: Legacy context API has been detected within a strict-mode tree:
    in StrictMode

Please update the following components: GlobalHotKeys

Learn more about this warning here:
https://fb.me/react-strict-mode-warnings

Hi @ndelangen, thanks for making me aware. I have created a separate issue to track this.


UMD style export instead of ES6 module

Could we get an UMD style export, so we can import the built file it into a build system or in the browser directly?
I am asking because If I let a translator translate stuff in my backend, I have to go in and pull all the translations, run artisan command to generate the .js file, then run my build commands so I can build my app.js again. If it was an umd module, I could just import it in the browser and call it from an Event for example.
I can try and do a PR if that would help?

Sounds good, PR welcome! Please implement it something like this:
keep existing behaviour if command is invoked as documented.
Add a --umd command line flag, to output in UMD style, so you can invoke:
php artisan vue-i18n:generate --umd
or similar

I forgot to document the new feature in the readme.md
Would you like to do it, or shall I do another PR ?

It would be helpful if you have time, please send a PR. If not, I'll get around to update README eventually

Either way, thanks so much for your contribution :)


Wrong Tenant ID is being used on Login

Describe the bug
When I login to Azure on Composer to publish or access resources, it tells me xxxx not found. When I go to my Azure account, neither of the 2 tenants I have have the ID that's being displayed on the error so I don't know where it's pulling this information from.
Version
v2.1.1
Browser

[ ] Electron distribution
[/] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[/] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Open Bot Framework Composer
Click on Publish
Select a profile
Select bots to publish
Click Publish selected bots
Prompt to login opens
Enter credentials
See error

Expected behavior
I should be logged in successfully as I have always been able to prior to this.
Screenshots

Additional context
I was having no issues when I was developing two bots using Composer. However, I ran into a problem implementing a feature on Composer and therefore tried to prototype a basic bot on SDK to understand the process. After I deployed this SDK bot using the Azure CLI following the documentation to the T, I started getting all of these tenant/login issues.

Yes! I also cleared all data for all time on my browser but I think likely the erasure of the keys was more the solution.


Upgrade sentry (redux^Wpart 2)

Rewrite all (untouched) Sentry code. There is no significatnt continuity between the old and new SDK code; neither, then, is any called for in our invocations thereof.
(Draft PR, as this is largely untested.)

I think you mentioned that it wasn't logging things right, but I don't recall the particulars. Does it log the exceptions at all? What's missing or wrong in them?

The code in logging.js was making calls using the wrong (i.e., old) function signatures. I don't recall whether it actually threw a new exception in mid-reporting attempt when it tried to access nonexistent fields, or whether it just sort of staggered through and managed to make a half-valid report anyway.
Since the broken version never made it out into an actual release (meaning that we won't have to wade through half-valid events in Sentry) I'm not too concerned about what the exact symptoms were.

The code in logging.js was making calls using the wrong (i.e., old) function signatures. I don't recall whether it actually threw a new exception in mid-reporting attempt when it tried to access nonexistent fields, or whether it just sort of staggered through and managed to make a half-valid report anyway.
Since the broken version never made it out into an actual release (meaning that we won't have to wade through half-valid events in Sentry) I'm not too concerned about what the exact symptoms were.

Cool, makes sense!
This would be good to add to the commit message, then. :slightly_smiling_face:

This would be good to add to the commit message, then.

... yes? It's (supposed to be) attached to the "sentry: complete rewrite..." commit. What's missing from it?

This would be good to add to the commit message, then.

... yes? It's (supposed to be) attached to the "sentry: complete rewrite..." commit. What's missing from it?

 The old `raven-js`
internals have gone away, replaced by a unified JavaScript SDK that
dispatches to native backends where possible.

Additionally Sentry has gone to some lengths to unify their API across
platforms: see https://docs.sentry.io/development/sdk-dev/ for
details. Unfortunately, the new API is in many ways only _subtly_
incompatible with the old -- and although TypeScript type definitions
are provided, Flow type definitions are not. Any old code in place
looked like it still worked, even though it should have generated
errors.

This effectively calls for a complete reanalysis and rewrite of every
part of our code that directly invokes Sentry. One should perhaps
think of this transition not as an upgrade so much as a movement to a
completely different SDK.

Which is the same version that was there when I asked the question in the first place. So from that you can at least infer that I didn't find the answer obvious from this description. :wink:
I think the key bits in your reply that I found helpful were:

The version before this change (but post-upgrade) probably doesn't really work, or at best half-works.

(I think maybe "it should have generated errors" was meant to convey something like that? But that "should have" sounds to me like a counterfactual -- like it's suggesting that it didn't generate errors. And I don't see anything else indicating what actually did go wrong.)

That version hasn't been in a release. (Which is why we don't mind being vague about the exact symptoms.)

Oh, relatedly: it would be good to mention commit ec101ddcb and the fact that we recently upgraded from an 0.x version to a 1.x version. That's information I did have, but is also important context for understanding this commit.

(I think maybe "it should have generated errors" was meant to convey something like that? But that "should have" sounds to me like a counterfactual -- like it's suggesting that it didn't generate errors. And I don't see anything else indicating what actually did go wrong.)

Less counterfactual than ambiguous: it's entirely possible that the failure was silent everywhere. I've attempted to clarify that.

That version hasn't been in a release. (Which is why we don't mind being vague about the exact symptoms.)

Oh, relatedly: it would be good to mention commit ec101ddcb and the fact that we recently upgraded from an 0.x version to a 1.x version. That's information I did have, but is also important context for understanding this commit.

Added (in both cases).

Withdrawn while I sort out (additional!) divergences in Sentry behavior from expectations.
Fortunately, it looks like the revision will have tests attached.


Release SQL-ODBC drivers

Is your feature request related to a problem? Please describe
SQL ODBC had a few bug fixes since previous release, we would like to initiate a new release for ODBC drivers to unblock some users. The artifacts are listed below:

Driver
Platform
Type

ODBC
Mac
zip

ODBC
Windows32
msi

ODBC
Windows64
msi

Describe the solution you'd like
Help from infra in signing and releasing the artifacts
Describe alternatives you've considered
No response
Additional context
No response

Hi @joshuali925 , can you start looking into automating this? From our offline conversation, I think the testing step can be done before forwarding the artifacts to our team for release.
I am not sure how these artifacts are generated today but the automated workflow would look like

Download the artifacts from given location (eg: GHA workflows artifacts, s3, etc)
Sign them
Upload them to production S3 bucket.

All of the above steps can be put into a JekinsFile. Example of one such component is data prepper: https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/data-prepper/release-data-prepper-all-artifacts.jenkinsfile
Thanks!

Yes, the signing will be same depending on the type of artifact. Looks like it is zip (maybe linux? not sure what we used before for zips) and windows.
Yes uploading to s3 will be same:
https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/data-prepper/release-data-prepper-all-artifacts.jenkinsfile#L68-L76
The path will be same what we used before in manual process.
Regarding testing I believe since this is directly affecting the production workflow, testing would be difficult. The underlying libraries are well tested. However, you can add tests for your jenkinsFile that shows the regression basically what calls are made. More details here: https://github.com/opensearch-project/opensearch-build/blob/main/DEVELOPER_GUIDE.md#jenkins-pipelines-and-shared-libraries
If you still want to play around with jenkins, please see https://github.com/opensearch-project/opensearch-ci which will allow you to set up your own instance by executing few commands.
Thanks!
cc: @bbarani

@mengweieric Can we close this issue here since we already released the ODBC?

@mengweieric Can we close this issue here since we already released the ODBC?
Hello Guys I am having same issue can you brief step how can solve this ..

@dharminfadia the new ODBC drivers are released on the bottom of https://opensearch.org/artifacts, could you try if they would work? If not feel free to create an issue under https://github.com/opensearch-project/sql/issues/
Closing this release issue as it's already done

Hello @joshuali925

latest Driver is install still having same issue

I am using Opensearch version is 1.2.4 but still facing issue .


[docs automation] Replace GitHub owner resolution with a DL

At some point codeowners will migrate away from individual owner aliases and resolving a primary owner will become more difficult.

Create DL with

Daniel
Jim
Lori
Justin

Remove codeowners discovery automation and use the alias instead

@danieljurek, if that alias is for content owners then you and I should not be on there.

ms.author: azsdkdocs


Clarification

Can you clarify this statement?
"/DefaultOutboundAction
This value is the action that the firewall does by default (and evaluates at the very end) on outbound connections. The merge law for this option is to let the value of the GroupPolicyRSoPStore win if it is configured; otherwise, the local store value is used. DefaultOutboundAction will block all outbound traffic unless it is explicitly specified not to block.
0x00000000 - allow
0x00000001 - block
Default value is 0 (allow)."
The statement "DefaultOutboundAction will block all outbound traffic unless it is explicitly specified not to block." seems to mean that the default state is Block. But the statement "Default value is 0 (allow)" seems to mean that the default state is to allow.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bfea4e59-ec8c-cbdf-a039-23612c2266a5
Version Independent ID: 7058544b-7ef2-a398-366a-cf71520aa9d8
Content: Firewall CSP - Windows Client Management
Content Source: windows/client-management/mdm/firewall-csp.md
Product: w10
Technology: windows
GitHub Login: @ManikaDhiman
Microsoft Alias: dansimp

This issue was fixed, so closing


Add retries to docker login to handle flakiness with docker container

Frequently we are seeing flakiness with our docker container host and it is unfortunately failing our pipelines.
We would really want to have this retry instead of failing the entire pipeline.
##[debug]Evaluating condition for step: 'Initialize containers'
##[debug]Evaluating: SucceededNode()
##[debug]Evaluating SucceededNode:
##[debug]=> True
##[debug]Result: True
Starting: Initialize containers
/usr/bin/docker version --format '{{.Server.APIVersion}}'
'1.41'
Docker daemon API version: '1.41'
/usr/bin/docker version --format '{{.Client.APIVersion}}'
'1.41'
Docker client API version: '1.41'
##[debug]Delete stale containers from previous jobs
/usr/bin/docker ps --all --quiet --no-trunc --filter "label=***"
##[debug]Delete stale container networks from previous jobs
/usr/bin/docker network prune --force --filter "label=***"
/usr/bin/docker login --username "***" --password-stdin ***
Error response from daemon: Get ***: dial tcp ***: connect: connection refused
##[error]Docker login fail with exit code 1
##[debug]System.InvalidOperationException: Docker login fail with exit code 1

Does my logic here seem sound? I copied this retry from DockerPull and would be happy to make it generic for both if the overall logic is agreeable here to add a retry.

/azp run

@anatolybolshakov  would you be the right person here for review?

Anyone?

There is another PR open that builds on this one #4100 and retries additional failures points.  @PavloAndriiesh @ivanduplenskikh @kirill-ivlev

There is another PR open that builds on this one https://github.com/microsoft/azure-pipelines-agent/pull/4100 and retries additional failures points.

To avoid confusion, closing this PR, let's focus on #4100.  Thank you for your contribution!


Add support for apollo-server-micro

I initially posted this in prisma-labs/graphql-yoga#614 but since yoga seems to be superseded by santa, I will post it here.

Although server support and lambda support is currently great, I'd been having troubles trying to expose a graphql-yoga endpoint in next.js.
Since I am using Next9, I want to leverage the API Routes feature, which although is a serverless function, has a slightly different signature than lambda functions.
The good news is that those API routes support any micro compatible middleware out-of-the-box. And even better, there is already an apollo-server-micro implementation, which is really similar to using the apollo-server-lambda exposed in GraphQLServerLambda.
Are there any plans on making the micro integration soon? If not, I can help with a PR.

Hey @iamkevinwolf to get up to speed on this stuff, could you point to a minimal example app and/or the relevant docs to read up on? Also wonder how much impact #231 would have on this issue (were we to go that way).


Not able to add the package to the app

It is giving the following error:
=> Errors while adding packages:
While downloading<EMAIL_ADDRESS>error: getaddrinfo ENOTFOUND
While downloading<EMAIL_ADDRESS>error: getaddrinfo ENOTFOUND
While downloading<EMAIL_ADDRESS>error: getaddrinfo ENOTFOUND
While downloading<EMAIL_ADDRESS>error: getaddrinfo ENOTFOUND

Could you check your internet? This seems like your machine can't reach Meteor's package server.


API Versioning example not work when use api.Use method

Hi
I try api-versioning example but it's not work because when request is coming the middleware does execute before extract path parameter.
Thanabodee

Hi @wingyplus it didn't work for me either, the reason turned out semver wasn't reading the #version number correctly from the URL, I ended up, manually extracting it from the PathParam:
var versionString string
if request.PathParam("version") != "" {
versionString = request.PathParam("version")
} else {
toBeExtractedFrom := request.URL.Path
parts := strings.Split(toBeExtractedFrom, "/")
versionString = parts[1]
}
	version, err := semver.NewVersion(versionString)

I'm sorry, I made a mistake when I converted this example to use the new API. I'm fixing it.

Should be fixed by https://github.com/ant0ine/go-json-rest-examples/commit/b19b09ad90be5c8697f3eb7b460a9421d5d9a3b3


convert .tekton/release-pipeline.yaml to use git

instead of upload-file-to-github.py which is not working anymore, use the git
command.
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

i am not totally sure it works tho! will probably need to make a release to be sure


Failure on not-that-large exports?

I'm getting timeouts trying to export our datasets. For now I'm getting around this by cloning the production DB locally and then running the django app locally to export. Here's the error I'm getting;

SystemExit: 1
(17 additional frame(s) were not displayed)
...
File "import_export/resources.py", line 424, in export
data.append(self.export_resource(obj))
File "import_export/resources.py", line 402, in export_resource
return [self.export_field(field, obj) for field in self.get_fields()]
File "import_export/resources.py", line 399, in export_field
return field.export(obj)
File "import_export/fields.py", line 101, in export
value = self.get_value(obj)
File "import_export/fields.py", line 75, in get_value
value = getattr(value, attr)

Some quick notes on our dataset:

17 columns
10 of them involve a foreign-key lookup
The deepest foreign key lookup is 2 hops deep
The fields are a mix of text and integers, the longest text field is maybe 200 chars.

There are about 1400 records that are being exported. Overall this isn't a lot, so it's a bit concerning. I see older / closed issue reports of timeouts, but they're happening at the level of 300,000 records, and this is much, much smaller. This dataset is < 200KB when the export worked using my 'local' hack.
Any thoughts?

What is exception thrown? Can you post full stack trace?

Would be happy to email the error along, the ST contains some semi-private info about my dev setup. Where should I send it?

@mallyvai bmihelac at mihelac.org


Select directory

Is it possible to select directory instead of a file?

hi @jycr
Yes this is possible. We have the file input component but we also have the directory input component.
The latter one will open a dialog where you can select a folder.


PPP Draft: Humphrey

Codecov Report

Merging #258 (b8c6235) into master (616b9c7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #258   +/-   ##
=========================================
  Coverage     70.19%   70.19%
  Complexity      900      900
=========================================
  Files           129      129
  Lines          2741     2741
  Branches        360      360
=========================================
  Hits           1924     1924
  Misses          613      613
  Partials        204      204

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


[BUG] ApiGateway metrics being skipped "metric":"IntegrationLatency","msg":"skipping metric unmatched by associator"

Is there an existing issue for this?

[X] I have searched the existing issues

YACE version
0.57.1
Config file
apiVersion: v1alpha1 discovery:   jobs:     - type: AWS/ApiGateway       regions:         - us-east-1       period: 300       length: 300       metrics:         - name: 4XXError           statistics: [Sum]         - name: 5XXError           statistics: [Sum]         - name: IntegrationLatency           statistics: [Maximum, Average, p99]         - name: Latency           statistics: [Maximum, Average, p99]
Current Behavior
ApiGateway v1 metrics appear to work correctly and show up in the metrics page and prometheus.
V2 have log messages as follows:
{"account":"redacted","arn":"","caller":"discovery.go:248","dimensions":"ApiName=my-api,Resource=/myenResource,Stage=mystage,Method=POST","job_type":"AWS/ApiGateway","level":"debug","metric":"IntegrationLatency","msg":"skipping metric unmatched by associator","region":"us-east-1","ts":"2024-07-16T21:08:36.186928023Z","version":"0.57.1"}
Expected Behavior
metrics are not skipped and show up in prometheus and are searchable by dimensions that show up on v1 resources
Steps To Reproduce

ensure metrics are visible with dimensions in cloudwatch for given resource
add above configuration to yace
restart yace to ensure config is loaded
read log messages from yace and observe no errors.
add debug flag to log and restart yace to observe skipping messages

Anything else?
We have tried both with and without the enable max dimension association and aws-sdk-v2 features, neither seems to have any impact. It is unclear why these are being skipped and adding required dimension names causes it to find no metrics at all and even the v1 metrics disappear
Additionally we have tried adding dimension requirements which causes ALL apigateway metrics to be skipped

Confirmed this issue also exists in latest release 0.61.2


fix browser alias for webpack compat

If you are trying to load this with webpack, the resolve algorithm will try to resolve the browser alias in the following way:
...
resolve result /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react/index.js
              aliased from directory description file /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react/package.json with mapping "index.js"
                resolve module browser.js in /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react
                  looking for modules in /Users/djpfahler/localhost/suggest-frontend/node_modules
                    /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js doesn't exist (module as directory)
                    resolve 'file' browser.js in /Users/djpfahler/localhost/suggest-frontend/node_modules
                      resolve file
                        /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js doesn't exist
                        /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js.js doesn't exist
                        /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js.jsx doesn't exist
                        /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js.es6 doesn't exist
                  looking for modules in /Users/djpfahler/localhost/node_modules
                    /Users/djpfahler/localhost/node_modules/browser.js doesn't exist (module as directory)
                    resolve 'file' browser.js in /Users/djpfahler/localhost/node_modules
                      resolve file
                        /Users/djpfahler/localhost/node_modules/browser.js doesn't exist
                        /Users/djpfahler/localhost/node_modules/browser.js.js doesn't exist
                        /Users/djpfahler/localhost/node_modules/browser.js.jsx doesn't exist
                        /Users/djpfahler/localhost/node_modules/browser.js.es6 doesn't exist
        use index.js from package.json
          resolve 'file' or 'directory' index.js in /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react
            resolve file
              resolve result /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react/index.js
                aliased from directory description file /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react/package.json with mapping "index.js"
                  resolve module browser.js in /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react
                    looking for modules in /Users/djpfahler/localhost/suggest-frontend/node_modules
                      /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js doesn't exist (module as directory)
                      resolve 'file' browser.js in /Users/djpfahler/localhost/suggest-frontend/node_modules
                        resolve file
                          /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js doesn't exist
                          /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js.js doesn't exist
                          /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js.jsx doesn't exist
                          /Users/djpfahler/localhost/suggest-frontend/node_modules/browser.js.es6 doesn't exist
                    looking for modules in /Users/djpfahler/localhost/node_modules
                      /Users/djpfahler/localhost/node_modules/browser.js doesn't exist (module as directory)
                      resolve 'file' browser.js in /Users/djpfahler/localhost/node_modules
                        resolve file
                          /Users/djpfahler/localhost/node_modules/browser.js doesn't exist
                          /Users/djpfahler/localhost/node_modules/browser.js.js doesn't exist
                          /Users/djpfahler/localhost/node_modules/browser.js.jsx doesn't exist
                          /Users/djpfahler/localhost/node_modules/browser.js.es6 doesn't exist
              Cannot resolve module 'browser.js' in /Users/djpfahler/localhost/suggest-frontend/node_modules/rx-react
...

This is fixed by providing a real local path.

:+1:

I thought this bug was fixed on webpack, thanks for the pull


Add certbot auto to Docker

@harrisreynolds I dont remember the context for adding this but I'm switching to installing it via APK due to changes in how certbot is distributed (only via snap, the wget is out of date).


error when trying to record video

Hi,
I am trying to setup video recording. I have the following code:
    let cameraManager = CameraManager()

    cameraManager.cameraDevice = .Back
    cameraManager.cameraOutputMode = .VideoWithMic
    cameraManager.cameraOutputQuality = .High
    cameraManager.writeFilesToPhoneLibrary = true

    cameraManager.addPreviewLayerToView(self.imageView)

    cameraManager.startRecordingVideo()

I get the following error when startRecordingVideo is called:
2015-12-23 22:52:25.643 RocketBash[1503:819683] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[AVCaptureMovieFileOutput startRecordingToOutputFileURL:recordingDelegate:] - no active/enabled connections.'
Anything else I need to do? or something I have done in error?

Hi Roberto,
thanks a lot for reaching out. The problem is that setting up the camera works asynchronously and that's why you're getting the error. I added two things to solve that:
CameraIsReady property to determine if it's already good to use and a competition block on add to preview function.
Let me know if you have any more troubles.
Natalia

Hi,
Thanks for the quick response.
I am not getting that error anymore however when I now start the recording in the callback the camera seems to freeze. Do you have a sample project with the camera that I can use as a reference?
I am now making use of the following code:
let cameraManager = CameraManager()
    cameraManager.cameraDevice = .Front
    cameraManager.cameraOutputMode = .VideoWithMic
    cameraManager.cameraOutputQuality = .High
    cameraManager.writeFilesToPhoneLibrary = true

    cameraManager.addPreviewLayerToView(self.imageView, newCameraOutputMode: .VideoWithMic, completition: {
        print("Camera ready")

        cameraManager.startRecordingVideo()
    })

Hi, you need to retain the cameraManager instance somewhere in order to retain all the configurations.
I tested your code and if I put it directly as is it freezes, but if I save the cameraManager as an instance constant if works fine.

Will give that a try shortly.

Ok, thanks for reporting. Do let us know if you need any more help! :-)


HA Clusters and Load balancing

This may not be the right channel for such questions so apologies in advance.

What is the right master/worker nodes ratio in a production type k8s cluster considering the following use cases.

 If not, how does client requests like (kubectl get pods) get "load balanced" or does it go just to one master every time ?

Is there a need to also place a load balancer between the masters and the workers ?
And what other factors should be considered when creating a production grade k8s cluster ?

Very classic Kubernetes architecture question. You should check out the official docs and recommendations.

The k8s doc does not say anything regarding master:worker ratio, all other details and recommendations are well explained and assumptions shouldn't be made that i did not take a look at the k8s official specifications. What i am trying to get here is personal experiences and question 2 is very specific to k3s.


Wrong length of message on RT interface

From time to time (if the robot is running for multiple hours) I see log messages like:
Wrong length of message on RT interface:<PHONE_NUMBER>
Wrong length of message on RT interface:<PHONE_NUMBER>
Wrong length of message on RT interface:<PHONE_NUMBER>
Wrong length of message on RT interface:<PHONE_NUMBER>
Wrong length of message on RT interface:<PHONE_NUMBER>
Wrong length of message on RT interface:<PHONE_NUMBER>
Wrong length of message on RT interface: 0

Is this something to worry about?
I had a look at ur_communication.cpp:
select(sec_sockfd_ + 1, &readfds, NULL, NULL, &timeout);
			bytes_read = read(sec_sockfd_, buf, 2048); // usually only up to 1295 bytes

Could it be sometimes more than one message is available, e.g. total data exceeding 2048 bytes and then with the next call a message is read in the middle? Maybe for unknown reasons the UR sends a burst or the thread context switching on the local machine was too slow?

I also see these messages sometimes.
It happens especially when I'm running the UR simulator at higher speeds, e.g. at frequency of 250 Hz rather than the standard 125 Hz (You can do this only in simulation of course) . That would confirm your suspicions.
I never had any issues with it though.

As noted in #42 this can safely be ignored. Apparently UR broadcast something undocumented on the realtime port from time to time. When I implement official support for firmware 3.2 ( See #17 ) this message will disappear.


Adding link color and increasing font size for nav. Fixes #204 & #242

What is the problem / feature ?

Link color was too low contrast
Text size of nav is too small.

How did it get fixed / implemented ?

Changed font color fo links
Increased text-size

How can someone test / see it ?
Navigate to the site and celebrate its increased readability!
Related Issues: #204  #242
Here is a cute animal picture for your troubles...

Looks good! Thanks, @Cara-Jo


add screen image on readme

I think that there is a screen image on the top page. Because it's so cool!

like this: https://github.com/SatoshiShimada/game_monitor/tree/add_screen_image


Release asset download URLs are broken (error 404)

I created a draft release via the action-gh-release.
As long as the release is still a draft, I can download the asset that were uploaded from the action-gh-release.
As soon as I publish the release, the download links are all broken, leading to 404 errors.
This is an example of such a broken link:
https://github.com/TheSlowGrowth/Wunderkiste/releases/download/refs%2Ftags%2Fv0.3/Wunderkiste.bin
This release was created with the script like this:
- name: Create Release Draft
        uses: softprops/action-gh-release@v1
        if: startsWith(github.ref, 'refs/tags/')
        with:
          tag_name: ${{ github.ref }}
          name: Release ${{ github.ref }}
          draft: true
          prerelease: false
          fail_on_unmatched_files: true
          files: |
            ./firmware/build/Wunderkiste/Wunderkiste.hex
            ./firmware/build/Wunderkiste/Wunderkiste.bin
            ./PCB_Files.zip
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

I noticed that the broken download link contains %2F characters which the draft-release didn't contain. They may be the reason for the 404. This is what a download link looks like BEFORE releasing the draft:
https://github.com/TheSlowGrowth/Wunderkiste/releases/download/untagged-[someCharacters]/Wunderkiste.bin
Making a release manually and uploading the files manually creates links that look different, like this:
https://github.com/TheSlowGrowth/Wunderkiste/releases/download/v0.3/Wunderkiste.bin
It seems to me as if the action-gh-release would internally use ${GITHUB_REF} to generate the various URLs/asset names. Maybe the "/refs/tags/" string needs to be sanitized?

Here's a workaround I found:
If I manually remove the /refs/tags/ from both the name and the tag_name argument, it creates links that work just fine.
- name: Get release name
        id: getReleaseName
        run: echo ::set-output name=RELEASE_NAME::${GITHUB_REF/refs\/tags\//}

[...]

 - name: Create Release Draft
        uses: softprops/action-gh-release@v1
        if: startsWith(github.ref, 'refs/tags/')
        with:
          tag_name: ${{ steps.getReleaseName.outputs.RELEASE_NAME }}
          name: Release ${{ steps.getReleaseName.outputs.RELEASE_NAME }}

believe this was fixed in https://github.com/softprops/action-gh-release/pull/88 . feel free to reopen the issue if it did not

       - name: Get release name
         id: getReleaseName
         run: echo ::set-output name=RELEASE_NAME::${GITHUB_REF/refs\/tags\//}
       - name: Create last unstable Release
         uses: softprops/action-gh-release@v1
         with:
           name: Latest unstable
           body: This is the latest unstable build version, feel free to test it out. Most of the time only the windows build got tested!
           tag_name: ${{ steps.getReleaseName.outputs.RELEASE_NAME }}-${{ GITHUB.RUN_NUMBER }}
           draft: false
           prerelease: true
           files: releases/**.zip
         env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

I do try to upload a build like that but I do get the same error as the inital creator. Now my problem this pipeline runs as soon as as someting is getting pushed to the develop branch.
This should provide some "latest builds" so the tag will be /refs/tags/develop-{Runner number}. But this leads to the 404 error if the build is not in draft :( Is there something wrong with my usage of the script?


feat: Add amount to Signifyd

Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

This PR solves issue #6119
Description

Additional Changes

[x] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[ ] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[ ] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

@pixincreate @ThisIsMani There seems to be many trait implementations absent in signifyd and also no tests implemented.
Should that be done in this PR?

@pixincreate @ThisIsMani There seems to be many trait implementations absent in signifyd and also no tests implemented. Should that be done in this PR?

no.
tagging relevant person for this for detailed clarification: @swangi-kumari

Hi @adi-kmt
Please run these
cargo +nightly fmt --all
cargo clippy

and resolve the CI Checks Error

Also you will have to make amount changes here too
juspay/hyperswitch/crates/router/src/connector/signifyd/transformers/api.rs


Update output in the examples to refer to the object and not the pointer

When using the examples as-is I would get a result like 0x56366332 instead of the value to which the pointer points. This fixes that.

Thank you for your PR. I believe a better approach would be to change it to resp.Content[0].GetText().

Done, updated


general protection faults on linux sytems

Openarc is generating general protection faults on a linux system:
kernel: [920575.093899] traps: openarc[9640] general protection ip:55ff9fed033f sp:7f533f9ad240 error:0 in openarc[55ff9fec5000+13000]
This happens a few times a week. Openarc itself is terminated and respawned by systemd.

Closing as I no longer use this software.


When moving file the old files are kept the same

When in watch mode when a file is moved a new converted file is created but the old one is not deleted.

Also seems like not all new files are found and converted in --watch mode

Merged into dev.
Just needs testing to merge into master.


Add missing "Provider Compliance/Control Policies" texts

Control -> Policies accordion -> Compliance/Control
explorer has correct child, but right side list had empty last row:

Links
Follow up to #11002.
@miq-bot add-label control, ui, bug, euwe/yes
cc @alongoldboim

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1411337

Euwe backport details:
$ git log -1
commit 5cf7817b057017681df31e4b0ffada638e20b91e
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Sun Dec 4 11:26:48 2016 +0100

    Merge pull request #12803 from cben/provider-policies

    Add missing "Provider Compliance/Control Policies" texts
    (cherry picked from commit 5e8b3b6695e9e7e3ed67055ee245560d39b3fda1)

    https://bugzilla.redhat.com/show_bug.cgi?id=1411364


Issue while building a Docker image

Hi guys ! I'm curious to try, but encountered an issue while trying to build a Docker image from the master branch
https://pastebin.com/fyTE9z1e
Thanks in advance!

UPD:
fixed by using rust:1.69


Support custom init containers and volumes in KServe

/kind feature
Describe the solution you'd like
The current KServe don't support init containers and uses webhook to inject storage initializer due to the limitation in KNative previously. Ref
KNative has support init containers and PVC in v1.2 natively.
Is it possible to support custom init containers and pvc in KServe as well so that users can use their own storage initializer and PVC, which provide more flexibility to the users. Thanks.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

/assign @rachitchauhan43