Mounting a host folder is broken?

Hi, I have a mysql image that defines 2 volumes.
When running it as a container, I want to mount two host folders into the image volumes using the remote API. Before v1.4.0, I was doing it only at starting the container (which looks deprecated). In v1.4.1 I need to define HostConfig.Binds both at container creation and also when starting the container.
I would expect to do it only at creation time or when starting the container.
@cpuguy83 Is this by design? Thanks

@pchico83 It should only be needed either/or, if you have to do it at both that seems off.
1.4.0 had some issues with binds and volumes-from passed in at start.
At start is not deprecated.

@cpuguy83 These are my experiments in 1.3.3 and 1.4.1
1.3.3

Client version
Mounting At
Result

v1.14
START
OK

v1.15
START
OK

v1.15
CREATE
KO

v1.15
BOTH
OK

1.4.1

Client version
Mounting At
Result

v1.14
START
KO

v1.15
START
KO

v1.15
CREATE
OK

v1.15
BOTH
OK

v1.16
START
KO

v1.16
CREATE
OK

v1.16
BOTH
OK

Looks like 1.4.1 has fixed mounting at creation time, but has broken mounting at starting time. I can send you the exact API requests I am doing (I didn't test it for volumes_from), but we should fix these inconsistencies.

I'd need to see an example as these are all tested for.

@cpuguy83 I found the case where this happens. I had a mysql container mounting 2 volumes and I removed the container without removing its volumes. Now, I want to deploy a new mysql container reusing those volumes, which live at:
    "Volumes": {
        "/etc/mysql": "/var/lib/docker/vfs/dir/fc53b53ea3720d5c7b5276601f80cc172049793cb25287c5dcbe247844b10b0a",
        "/var/lib/mysql": "/var/lib/docker/vfs/dir/47ecf269f5ffc15ff560c9d95639b250506b53b73d6f58fce2fb78154c736ef8"
    }```

Then, I do:

https://<IP_ADDRESS>:2375/v1.15/containers/4758a0d88a5b93a268bdd47aa353d0a82317bb56eebc1385d181678bf3307920/start
{'Binds': [u'/var/lib/docker/vfs/dir/387f6dddfd4b2b82442c567a8729cc64b353220874f8e1bb2f5052145a96d0b0:/etc/mysql:rw', u'/var/lib/docker/vfs/dir/202869818d9571a693d39ec97cad1b616762434095769921d61d440f4e3c514b:/var/lib/mysql:rw', u'/var/lib/docker/vfs/dir/47ecf269f5ffc15ff560c9d95639b250506b53b73d6f58fce2fb78154c736ef8:/var/lib/mysql:rw', u'/var/lib/docker/vfs/dir/fc53b53ea3720d5c7b5276601f80cc172049793cb25287c5dcbe247844b10b0a:/etc/mysql:rw']}

This does not work in 1.4.1 and works in 1.3.3. The other option is to pass these binds at creation and not at starting, which would work in 1.4.1 but not in 1.3.3. I know this might be better by using data containers, but I think we should fix this inconsistency while we don't have #8484

Note that the issue is only when mounting host folders created by docker in `/var/lib/docker/vfs/dir/`, other host folders look fine.

@cpuguy83 I found the case where this happens. I had a mysql container mounting 2 volumes and I removed the container without removing its volumes. Now, I want to deploy a new mysql container reusing those volumes, which live at:
"Volumes": {
   "/etc/mysql":
      "/var/lib/docker/vfs/dir/fc...0a",
   "/var/lib/mysql":
      "/var/lib/docker/vfs/dir/47...f8"
}

Then, I do:
https://<IP_ADDRESS>:2375/v1.15/containers/47...20/start
{
    'Binds': [
        u'/var/lib/docker/vfs/dir/fc...0a:/etc/mysql:rw',
	u'/var/lib/docker/vfs/dir/47...f8:/var/lib/mysql:rw'
     ]
}

This does not work in 1.4.1 and works in 1.3.3. The other option is to pass these binds at creation and not at starting, which would work in 1.4.1 but not in 1.3.3. I know this might be better by using data containers, but I think we should fix this inconsistency while we don't have #8484
Note that the issue is only when mounting host folders created by docker in /var/lib/docker/vfs/dir/, other host folders look fine.

You can pass bind mounts on create or start in 1.3.3, they just won't actually be created until the container is started.
That said, I cannot reproduce this on 1.4.1.
$ curl -H 'Content-Type: application/json' -XPOST localhost:2375/containers/foo/start -d '{ "Binds": ["/var/lib/docker/vfs/dir/f6cd891e831f181650c69911e65b58e1d639898059d06e3be9253549248da928:/foo"]}'

$ docker inspect -f '{{ json .Volumes }}' foo
{"/foo":"/var/lib/docker/vfs/dir/f6cd891e831f181650c69911e65b58e1d639898059d06e3be9253549248da928"}

These are the steps to repro it:
docker run -d --name aaa tutum/mysql
docker inspect -f '{{ json .Volumes }}' aaa
   {
      "/etc/mysql":
         "/var/lib/docker/vfs/dir/d4aebcc984652d51bd01fe5874560fe1fce8b8779b4596d440c290290d0a4e73",
      "/var/lib/mysql":
         "/var/lib/docker/vfs/dir/eee566ec7b51f1b8c7f0f1b92a0d61e1cfc445bc00e7e0012686da865b2d82ba"
   }
docker create --name bbb tutum/mysql
curl -H 'Content-Type: application/json' -XPOST localhost:2375/containers/bbb/start -d '{ "Binds": ["/var/lib/docker/vfs/dir/d4aebcc984652d51bd01fe5874560fe1fce8b8779b4596d440c290290d0a4e73:/etc/mysql"]}'
docker inspect -f '{{ json .Volumes }}' bbb
   {
      "/etc/mysql":
         "/var/lib/docker/vfs/dir/d9229725caf411ef79127b62af7157815fb27e0017a2a0835971f4726414c264",
      "/var/lib/mysql":
         "/var/lib/docker/vfs/dir/e5db26a469fb9ebb2ceae0ebf42a271986786e7b2f6ef39a81fc96a25f14752d"
}

Did you try this on master?  You can grab a pre-compiled binary here: https://master.dockerproject.com

It is still happening in docker 1.5.0

@pchico83 see referenced pr

@cpuguy83 Thanks!


ISPN-9386 Remove WebSocket server module

https://issues.jboss.org/browse/ISPN-9386

RIP websocket. you were always misunderstood, never truly appreciated


No need to load json. (follow up on PR #200)

On Sun, Mar 01, 2015 at 08:13:58AM -0800, David Rio Deiros wrote:

No need to load json.

We use this to support:
DEBUG=true
and:
DEBUG=false
Without the ‘json.loads’ wrapper, both of those will enable DEBUG.

What do you mean? The output from os.environ.get() is not json so json.loads() fails.
$ python
Python 3.4.2 (default, Feb 12 2015, 14:15:15)
[GCC 4.4.7 20120313 (Red Hat 4.4.7-11)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import json
>>> import os
>>> os.environ.get('DEBUG', 'true')
'FALSE'
>>> json.loads('FALSE')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/swc/drd_python/Python34/lib/python3.4/json/__init__.py", line 318, in loads
    return _default_decoder.decode(s)
  File "/home/swc/drd_python/Python34/lib/python3.4/json/decoder.py", line 343, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/swc/drd_python/Python34/lib/python3.4/json/decoder.py", line 361, in raw_decode
    raise ValueError(errmsg("Expecting value", s, err.value)) from None
ValueError: Expecting value: line 1 column 1 (char 0)

More discussion here 1, as part of #206.  In response to @gvwilson's
comments, the standard library is a useful hammer, but it doesn't cost
any more than the Python install we already need for amy ;).
I've looked online and through bash(1), python(1), and gcc(1) in
search of conventions for boolean environment variables.  Bash is all
over the place (EMACS is ‘t’ for true, OPTERR is ‘1’ for true, and
POSIXLY_CORRECT is set for true, possibly including being set to the
empty string).  Python is consistently “set to a non-empty string for
true”.  Personally, I think accepting ‘true’ and ‘false’ as values
makes for a more accessible interface than “non-empty” vs “empty or
unset”, but this is a Python application, so I'd be ok with:
DEBUG = os.environ.get('DEBUG')
I don't think having a default-true value makes sense with the “empty
for false” interpretation, because:
DEBUG='' ./manage.py …
isn't clearly turning off debugging (to me).  On the other hand, I
think:
DEBUG=false ./manage.py …
is clearly turning off debugging.

On Sun, Mar 01, 2015 at 01:38:30PM -0800, David Rio Deiros wrote:

What do you mean? The output from os.environ.get() is not json so
json.loads() fails.

You can set the value of the environment variable however you like, so
obviously you can pick non-JSON values.  I'm fine crashing in those
cases, but I'm happy to set things up to give a more personal error
message if we want.

os.environ.get('DEBUG', 'true')
'FALSE'

That's not JSON, but ‘false’ works as expected:
$ DEBUG=false python -c "import json, os; print(json.loads(os.environ.get('DEBUG', 'true')))"
False

Ok. That makes sense.


Adds support for --quiet option

The command line interface of ESLint supports --quiet to only report errors and ignore warnings. The ESLint CLIEngine do not expose this option because quiet only has to do with output and CLIEngine is designed specifically to not output anything.
This is is useful when you lint something where warnings don't matter e.g. a git pre-commit hook. It can be confusing to see a lot of warnings and still see the commit run.
Let me know if there is a better place or a better way to do this =)

Coverage decreased (-0.96%) to 75.56% when pulling b003247157056683efed1dec5e45f9c6c4dd1bcc on anddoutoi:master into 5a617304f5a5988863ea3090e92a2d48d133b719 on adametry:master.

I can see your point and appreciate the PR; though, I believe this may be better implemented via format or formatEach since the aim is to change reporting.
Here's a quick example of what can work now:
gulp.task('warnless', function() {
	return gulp.src(['**/*.js', '!node_modules/**', '!coverage/**'])
		.pipe(eslint())
		// format/formatEach can be used to transform reports as well
		.pipe(eslint.format(function (reports) {
			// count & remove warning messages from each eslint file report
			reports.forEach(function (report) {
				report.messages = report.messages.filter(function isError(message) {
					return message.fatal || message.severity > 1;
				});
			});
			// The returned value goes to the console (by default)
			// If nothing is returned, nothing is written.
			return '(Eslint Report Silenced)';
		}))
		// Now, we'll print what remains of the eslint reports
		.pipe(eslint.format());
});

Does this work for you or what else might be done?

That would do the work and produce the result I want. I can definitely live with that solution.
One could however argue that the formats job should be to only format the data for different consumers and respect data integrity. The fact that grunt-eslint supports it maybe also is a carrot? ;)
Anyway, thanks for getting back on this.

@adametry Thanks for the snippit. I have to agree with @anddoutoi though - this would be nice to have as a baked-in feature rather than a special formatter (my guess is that most people would not know to find this GitHub Issue, and just assume it is impossible if it isn't in the docs).

What do you think about having an option severity parameter for format and formatEach? This could allow reporting warnings and errors independently/optionally. It might looks something like:
var fileOut = fs.createWriteStream('lint-errors.log');
gulp.src('**/*.js')
    .pipe(eslint())
    .pipe(eslint.format()); // print errors and warnings to stdout
    .pipe(eslint.format(2, 'compact', fileOut)); // print errors (severity = 2) to file
```
*Note:* the severity numbers are the same as the ones used when configuring eslint rules.

This would be more flexible in how the data is consumed and maintain data integrity (with no reduction).

Has there been any progress on this?

A release candidate has been published that includes a "quiet" option. If set to true, it will remove warning messages from Eslint results. If set to a function(message) results will only include messages that return a truthy value. For example:
gulp.task('lint-quiet', function () {
    return gulp.src('src/**/*.js')
        .pipe(eslint({quiet: true}))
        .pipe(eslint.format());
});

...or only warnings:
function isWarning(message) {
    return message.severity === 1;
}
gulp.task('lint-warn', function () {
    return gulp.src('src/**/*.js')
        .pipe(eslint({quiet: isWarning}))
        .pipe(eslint.format());
});

Try the release candidate out and let me know how well it works for you.
npm install gulp-eslint@rc

The option is now in the latest version of gulp-eslint (1.1.0).


Setup docker and litestream

Aims to close

#7
#16

The docker setup is not tested yet, I'll quickly do that tomorrow and also not integrated in CI yet due to

#8

not resolved yet.
On the other hand I want to keep PRs small and I think there is enough material to review here.
Tomorrow if I find time I'd like to test this locally to make sure before the merge.
If you'd like to check out what litestream is: https://litestream.io/guides/
This PR will likely receive a few commits more and some values are place holders and guesses.

LGTM


Add NR1-FLEET type

Relevant information
Add a new entity type for Fleets.
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

 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.Anthony Trupiano seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Scatter plot with in-viewer filtering set does not show data after adding a calculated column

Steps to reproduce:

Open a table, e.g. SPGI
Add a scatter plot
Set up some filtering in scatter plot, e.g. ${Series} in "Pyrrolidines"
Add new calculated column

Expected results: Adding a calculated column should not affect any viewers.
Actual results: No data on scatter plot after adding a calculated column:

Multiple errors appear in the console:
error: Invalid argument(s): Lengths differ (3624 != 0)
js_primitives.dart:30 stackTrace: packages/$sdk/lib/_internal/js_runtime/lib/js_helper.dart 1772:17      Object.t
packages/$sdk/lib/_internal/js_runtime/lib/js_helper.dart 1807:25      Object.R
packages/ddt/src/collections/bit_set.dart 48:9                         cZ.axg
packages/$sdk/lib/async/stream_controller.dart 169:49                  Object.aIS
packages/d4/src/viewer_base/data_frame_viewer.dart 316:27              lL.Fn
packages/d4/src/scatterplot/scatterplot_renderer.dart 94:7             dcI.nD
packages/d4/src/scatterplot/scatterplot_core.dart 1076:7               lL.f5
packages/d4/src/viewer_base/canvas_viewer_mixin.dart 169:7             lL.a6D
packages/d4/src/viewer_base/canvas_viewer_mixin.dart 106:9             bdj.
packages/$sdk/lib/async/zone.dart 1132:12                              aXf
packages/$sdk/lib/async/zone.dart 1012:12                              d5C.zT
packages/$sdk/lib/async/zone.dart 909:14                               d5C.GZ
packages/$sdk/lib/async/zone.dart 936:23                               d5F.
packages/$sdk/lib/_internal/js_runtime/lib/js_helper.dart 2456:48      fxX.
packages/$sdk/lib/_internal/js_runtime/lib/isolate_helper.dart 474:16  NS.KZ
packages/$sdk/lib/_internal/js_runtime/lib/isolate_helper.dart 59:16   Object.FY
packages/$sdk/lib/_internal/js_runtime/lib/js_helper.dart 2456:14      fxV
packages/$sdk/lib/_internal/js_runtime/lib/js_helper.dart 2476:14
js_primitives.dart:30
Similar errors are seen for other viewers with in-viewer filtering available: Line chart, Bar chart, Box plot. But for these viewers data is still shown after adding a calculated column.

Fixed in v1.8.3


Trigger backport workflow on comment

Applies the latest example from the backport-action README, to enable triggering the backport action by writing a comment.

@korthout Do you want me to review this? Or are you still preparing the PR?

@pihme yes please :)


Support Unsigned 64-bit integer

As I know number field in JSON is not limited by the number of digits,
but parser is limited by their own language specification.
In terms of the datatype, uint64 is frequently used for identifier, hash value...
and it is natural that many of Cpp based project uses unsigned long.
My opinion is that it would be great if the parser supports both uint64 and int64.
I am really impressed with this artwork. thanks a lot =)

That's reasonable and could be added without too much work.

Close via https://github.com/lemire/simdjson/pull/306
Credit: @saka1


Remove writeLockLastEdge

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3842/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3844/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3847/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.0 please.

alluxio-bot, cherry-pick this to branch-2.0 please.


New subscription page and also responsive design

Fixes Issue
Fix issue : #2187
Closes : #2187

Changes proposed

Screenshots

Note to reviewers

Summary by CodeRabbit

New Features

Revamped subscription management interface for improved user interaction.
Introduced dynamic selection of subscription plans, payment methods, and discount codes.
Added total price calculation based on user selections and applied discounts.

Enhancements

Modernized layout with a focus on user experience, including updated design elements.
Included buttons for choosing between quarterly and yearly payment options.

Refactor

Renamed component from SubscriptionManagementPage to SubscriptionPage.

kindly attack an screenshot so that i can review your PR and also take a look of code rabbit review but fix in locally


streaming: improve through-put of ingesting data via insert

Is your feature request related to a problem? Please describe.
The INSERT statement is useful for data ingestion from user's customized data pipeline, so the performance matters.
Describe the solution you'd like
We may do some benchmarking and profiling first to see where could be improved.

FYI, in release mode

Around 15,000 row/sec in single connection.

This issue is just a reminder that we didn't do any testing/benchmarking on this case yet. May not be very critical now.


Change the name description of install

The original does not describe what it is for.
I think alias make more sense. 🌷

Good point. What about "Preferred name of the definition"?

The issue with alias is that implies that the original name might be used still.

What about "Preferred name of the definition"?

IMO "Preferred" is too soft. When user specifies it, that's the only way to reference that module.
"Module referenced name"? "Mapped name of the module"?

Sure, just "name of the installed definition" or something like that sounds good.

How about:
"Mapped name of the installed definition"
"Alternative name of the installed definition"
Just "name of the installed definition" is the same as the original "Module name of the installed definition"

Haha, fair enough. I guess that is just how I think then. "Alternate name of the definition" sounds good 👍

Updated


proposal: debug/pe: add Machine type with String method

When using the debug/pe package to look up information about binaries, I noticed that there's a subtle API difference compared to debug/macho and debug/elf.
For instance, in the debug/elf package, Machine is defined as being able to call:
f.Machine.String()

Whereas when using the debug/pe package, Machine has no such definition in the FileHeader.
It would be good to align the API of this package alongside how we do it in the other packages:

Extract a type for Machine
Add a String() method
Add a GoString() method

Hey folks, just wondering if there's anything I can do to help this along? Appreciate it's only a small proposal and there are a lot of the bigger ones to go through at the moment!

That's fair - I'm sure there's a way we can make this non-breaking, while still allowing for this new metadata to be usable?

Could we expose a new method in debug/* to get at the Machine? (What would it be called?)

We can add the type, but if we can't use it anywhere in the API, is there much value?

Folks could type assert to the larger interface?

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

There's not really a larger interface, just a different integer type. So you'd have to say pe.Machine(fh.Machine).String(). I guess that's fine but it's a little weird.

Interesting. Any suggestions for how we can move forwards?

I believe the proposal is:
In package pe, add:
type Machine uint16
func (m Machine) String() string
func (m Machine) GoString() string

similar to elf.Machine. The existing Machine uint16 in type FileHeader would not change, for compatibility. Users who want to print a file header’s machine need to use
fmt.Println(pe.Machine(fh.Machine))

Gotcha, that makes sense, thanks!
Would we also be able to make the following change:
-	IMAGE_FILE_MACHINE_UNKNOWN     = 0x0
+	IMAGE_FILE_MACHINE_UNKNOWN     Machine = 0x0

Or would this introduce a breaking change?

@jamietanna Yes, that would be a breaking change, and we can't do that.

Gotcha, thanks!
I've amended the proposed code to fit within this, which should now no longer be a breaking change. Thanks for the feedback and suggestions folks.
I've also verified it with the following test:

package pe

import (
	"fmt"
	"testing"
)

func TestMachineToString(t *testing.T) {
	c := IMAGE_FILE_MACHINE_WCEMIPSV2
	fmt.Printf("c: %v\n", c)

	fmt.Printf("Machine(c): %v\n", Machine(c))
	fmt.Printf("Machine(c).GoString(): %v\n", Machine(c).GoString())

	c = 0
	fmt.Printf("c: %v\n", c)

	fmt.Printf("Machine(c): %v\n", Machine(c))
	fmt.Printf("Machine(c).GoString(): %v\n", Machine(c).GoString())

	c = 0x5128
	fmt.Printf("c: %v\n", c)

	fmt.Printf("Machine(c): %v\n", Machine(c))
	fmt.Printf("Machine(c).GoString(): %v\n", Machine(c).GoString())
}

Test output:
=== RUN   TestMachineToString
c: 361
Machine(c): IMAGE_FILE_MACHINE_WCEMIPSV2
Machine(c).GoString(): pe.IMAGE_FILE_MACHINE_WCEMIPSV2
c: 0
Machine(c): IMAGE_FILE_MACHINE_UNKNOWN
Machine(c).GoString(): pe.IMAGE_FILE_MACHINE_UNKNOWN
c: 20776
Machine(c): IMAGE_FILE_MACHINE_RISCV128
Machine(c).GoString(): pe.IMAGE_FILE_MACHINE_RISCV128

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group
In package pe, add:
type Machine uint16
func (m Machine) String() string
func (m Machine) GoString() string

similar to elf.Machine. The existing Machine uint16 in type FileHeader would not change, for compatibility. Users who want to print a file header’s machine need to use
fmt.Println(pe.Machine(fh.Machine))

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group
In package pe, add:
type Machine uint16
func (m Machine) String() string
func (m Machine) GoString() string

similar to elf.Machine. The existing Machine uint16 in type FileHeader would not change, for compatibility. Users who want to print a file header’s machine need to use
fmt.Println(pe.Machine(fh.Machine))

Awesome 👏🏽 for next steps, is it a case of the linked changeset getting reviewed when the core team have a chance?


[Vulkan] Fix ternary operator shader compiler expression

Same as https://github.com/godotengine/godot/pull/31473 but for Vulkan

You need to rebase as I force pushed the vulkan branch.


Debug cloud code locally

More of a question than a bug. I've set up Parse Server on Heroku and wondered how I could run and debug cloud code locally ? Is there a guide somewhere explaining how to do this ?

Run the project locally, instead of on heroku.  It's very easy.  Clone your repo from Heroku if you used the 1-click button, and then npm install and run it local.  I use mostly console.log debugging, but this looks like it will work as well: https://nodejs.org/api/debugger.html
I feel like we've failed somewhere in guidance, because I feel like running it local should always be the first step.

If i setup my parse server in ibm bluemix? can i write my cloud code in bluemix (same server) or i need to have heroku account?


Upgrade Required

When i install the multiogar on my windows machine it give the Upgrade Required

How to fix this issue

You need to connect to your server using a client:
https://github.com/Barbosik/MultiOgar#ogar-clients

omfg.

go to agar.io/?ip=<IP_ADDRESS>:443

:)


Update flow-approval-cds-provisioning-errors.md

changes "Microsoft Flow" to Power Automate
changes "Common Data  service (CDS)" to Dataverse
as these are the "new" names.

Harysh Menon's comment received via email, "From the PR it looks like they’re essentially moving from Flow to Power Automate and Common Data Service to Dataverse. Yes I think that’s fine to merge."

Hi @LuiseFreese, thanks for your draft. I also made some minor changes. Feel free to let me know if the changes are OK to merge. Also loop @hamenon-ms here for any further suggestions.


XmlPort Error (The Element is expected by Min Occurs value XXX: Once. Element received: YYY) .

Describe the bug
Im Continually getting this error in my xmlport The Element  is expected by Min Occurs value: Once. Element received: . Ive tried various MinOcurrs settings and MaxOccurs settings nothing seems to resolve my issue..
To Reproduce
Debugged the XmlPort Code but unable to ascertain what is causing this issue there is no clear reason why. I have looked
at examples of similar code and can't really see a difference

Go to '...'

AL Code to reproduce the issue
xmlport 80002 "PC Purchase Order Import"
{
Direction = Import;
DefaultFieldsValidation = TRUE;
Format = VariableText;
UseRequestPage = false;
TextEncoding = MSDOS;
FieldDelimiter = '"';
FieldSeparator = ',';
RecordSeparator = '';
TableSeparator = '';
TransactionType = UpdateNoLocks;
schema
{
    textelement(ROOT)
    {
        TableElement(PH;"Purchase Header")
        {
            MinOccurs = Once;
            MaxOccurs = Once;
            SourceTableView = WHERE("Document Type"=Const(Order));
            Autosave = True;
            FieldElement(FLD1;PH."Buy-from Vendor No.")
            {
            }

            FieldElement(FLD2;PH."Location Code")
            {
            }
            FieldElement(FLD3;PH."Currency Code")
            {
                trigger OnAfterAssignField()
                begin
                    If PH."Currency Code" = 'AUD' then clear(PH."Currency Code");
                end;
            }
            textelement(FLD4)
            {
                trigger OnAfterAssignVariable()
                begin
                    If not Evaluate(LineDisc,FLD4) then
                        Error('Invalid Line Discount')
                    else If LineDisc > 100 then
                        Error('Line Discount > 100');
                end;
            }
            textelement(FLD5)
            {
                trigger OnAfterAssignVariable()
                begin
                    If not Evaluate(invDisc,FLD5) then
                        Error('Invalid Line Discount')
                    else If InvDisc > 100 then
                        Error('Inv Discount > 100');
                end;
            }
            TableElement(PL;"Purchase Line")
            {
                AutoSave = true;
                SourceTableView = Where(Type=Const(item));
                LinkTable = PH;
                LinkFields = "Document No." = field("No."), "Document Type" = Field("Document Type");
                FieldElement(FLD1A;PL."No.")
                {
                    trigger OnAfterAssignField()
                    var
                        myInt: Integer;
                    begin
                        lineNo += 10000;
                        PL.Validate("Line No.",lineno);
                    end;
                }
                FieldElement(FLD2A;PL."Quantity")
                {
                }
                FieldElement(FLD3A;PL."Unit Of Measure Code")
                {
                }
                FieldElement(FLD4A;PL."Direct Unit Cost")
                {
                    trigger OnAfterAssignField()
                    begin
                        PL.Validate("Line Discount %",LineDisc);
                    end;
                }
                trigger OnAfterInitRecord()
                begin
                    Cnt2 += 1;
                    If Cnt2 < 4 then currXMLport.Skip();
                END;
            }
            trigger OnAfterInitRecord()
            begin
                Cnt1 += 1;
                If Cnt1 = 1 then currXMLport.Skip();
            END;
        }

    }
}
trigger OnPreXmlPort()
begin
    Clear(cnt1);
    Clear(cnt2);
    Clear(LineNo);
    Win.Open('Importing SKU #1##############');
end;
trigger OnPostXmlPort()
var
begin
    Win.Close;
end;

var
    LineDisc:Decimal;
    InvDisc:Decimal;
    LineNo:Integer;
    Cnt1:integer;
    Cnt2:integer;
    win:Dialog;

}
The CurrXmlPort Skip functionality is being used to skip over Excel header lines
The Excel file layout is as follows It consists of two header rows One for purchase header information, Then Other for purchline
information
Supplier   No | Location | Currency | Line Discount % | Invoice   Discount %
SUP-00001 | NSW | AUD | 10 | 0 |
SKU | Qty | UOM | Direct Unit Cost |   |
SKU-000001 | 2 | PCS | 45 |   |
SKU-000001 | 2 | PCS | 45 |   |
SKU-000001 | 2 | PCS | 45 |   |
SKU-000001 | 2 | PCS | 45 |   |
SKU-000001 | 2 | PCS | 45 |   |
  |   |   |   |   |
  |   |   |   |   |
  |   |   |   |   |
  |   |   |   |   |
Expected behavior
'I'm Trying to Open a CSV excel file and read various fields to construct a purchase Order with attached purchase lines.
I expect that I could produce the Purchase header and via the lnkfields, link table build the purchase lines.
But I continually get the error described and I can't understand what the issue is and why it occurs??
Screenshots

5. Versions:

AL Language:

Business Central:

I figured out the issue by going back to the old CSide XmlPort Designer. It appears that when you use Variable Text format you are not allowed to use Indented Tables the only format it allows is XML Format to do this. Unfortunately AL happily compiles it all and doesn't notify you of a problem.  I spent hours and hours trying to understand what it was telling me, Just goes to show the old CSide XmlPort Designer tells you that what you are trying to do is invalid upfront before going thru the aggravations of trying to run the code. AL unfortunately is not so co-operative in that department. Live and learn I guess.

@vpacker we will look into adding more validation for this in VSCode. Could you provide the TXT file for C/SIDE and a screenshot of where the diagnostic is raised so that we can pick this up in the future? Sample code helps us reproduce the issue way faster than a textual description.

Hi Alexandru,
The issue is that AL code does not recognise a problem exists before hand when you design an xmlPort that uses variable text format but have designed the XMlport to use multiple linked table structures.
In CSide Xmlport designer informs you of the problem before you attempt to run it. Al on the other hand has no concept of the problem before hand but gives you an obscure reference message after executing the code that I raised as the issue.
I’m guessing the way AL works it makes assumptions that you know before hand the possible pitfalls you can create without actually knowing it. The problem I experiences was very frustrating as I was 100% sure that the XMLport code I wrote was structurally sound, had no real clue as to what I was doing wrong.
Out of necessity I went back to the CSide XmlPort dsigner to do exactly the same thing I did in AL and after coding it all up I got no errors then I remember I needed to change the format from XML to variable Text and presto the problem revealed itself straight away as Cside immediately complained of an error. It was at this point I realised just what the AL issue was all about!!! It seems obvious now but with out Cside Xmlport designer I would still be struggling trying to figure out just what I was doing was wrong.
Thanks
Vic
From: Alexandru Toader<EMAIL_ADDRESS>Sent: Monday, 16 November 2020 9:43 PM
To: microsoft/AL<EMAIL_ADDRESS>Cc: vpacker<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/AL] XmlPort Error (The Element is expected by Min Occurs value XXX: Once. Element received: YYY) . (#6300)
@vpacker https://github.com/vpacker  we will look into adding more validation for this in VSCode. Could you provide the TXT file for C/SIDE and a screenshot of where the diagnostic is raised so that we can pick this up in the future? Sample code helps us reproduce the issue way faster than a textual description.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/AL/issues/6300#issuecomment-727896006 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKZ6VZWYASAECY4I2TZF3ALSQD62DANCNFSM4TU2JPJA .


[8.x] camelCased tests names

Stay consistent and always use camelCased tests names.

We actually want to change to the other casing

Hello @driesvints , eventually will use snake_case in the future?


open to a PR for for a container query plugin?

it would be really nice to have some way to be able to do container-queries (being able to set classes on a DOM element based on some criteria like element width.)
Happy to try to write one if one doesn't already exist, just don't want to re-invent the wheel here.

I've put together a proof-of-concept, here's a demo of it in action:
https://test-snabbdom-container-query.nekoflux.vercel.app/
If you inspect the <main> element and resize the window, you'll see different classes (BP1, BP2, BP3) applied to the element. This looks like a @media query but it's actually specified at the component level!
h('main', {
    attrs: {
        'data-breakpoints': '{"BP1":400,"BP2":800,"BP3":1200}'
    },
  }, `Just some test string`)

Is this something we'd be interested in adding as another optional module to snabbdom? I'm happy to put this into a PR if so.
The only production issue with this module is it assumes presence of the ResizeObserver API.  Support is overall quite good but I think we'd want to ponyfill that capability as needed, which would add a little complexity to the implementation, but nothing crazy I think.
my current implementation:
snabbdom-containerquery.js:
// inspired by https://philipwalton.com/articles/responsive-components-a-solution-to-the-container-queries-problem/

// currently, it's more performant to have 1 resizeObserver observe multiple elements rather than 1 per element.
// see https://groups.google.com/a/chromium.org/g/blink-dev/c/z6ienONUb5A/m/F5-VcUZtBAAJ
const observer = new ResizeObserver(function (entries) {
	for (const entry of entries) {
		const breakpoints = JSON.parse(entry.target.dataset.breakpoints);
		let prev;

		for (const breakpoint of Object.keys(breakpoints)) {
			const minWidth = breakpoints[breakpoint];
	        if (entry.contentRect.width >= minWidth) {
	        	entry.target.classList.add(breakpoint);

	        	// exclusively match only a single breakpoint, so remove any other breakpoint
	        	// classes that might have previously matched.
	        	if (prev)
	        		entry.target.classList.remove(prev);

	        } else {
	        	entry.target.classList.remove(breakpoint);
	        }

	        prev = breakpoint;
		}
	}
});

function update (oldVnode, vnode) {
	if (vnode.elm.dataset.breakpoints) {
		try {
			// ensure that the data attribute parses as valid json before we start observing it
			JSON.parse(vnode.elm.dataset.breakpoints);
			observer.observe(vnode.elm);
		} catch (er) {
			throw new Error('Invalid breakpoints declared. must be valid JSON', er);
		}
	} else {
		observer.unobserve(vnode.elm);
	}
}

function destroy (vnode/*, removeCallback*/) {
	observer.unobserve(vnode.elm);
}

export default {
	create: update,
	update,
	destroy
}

I think it would be appropriate to maintain this module as a separate repository/package. That could, if you're interested, be a repository under the Snabbdom GitHub organisation.

no longer necessary, now that css container queries are approaching widespread availability.

Thank you for taking the time to closing the issue now that it is no longer necessary and thank you for sharing the container query plugin 👍


City listed in wrong state

Challenge Name
City listed in wrong state
https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/LocalGroups-List
Issue Description
Indian city 'Noida' is listed in 'Haryana' State, whereas 'Noida' is in 'Uttar Pradesh'
Screenshot

Hi @crowchirp,
Thanks for reporting this.
Closing in favor of FreeCodeCamp/wiki#984


Unable to use Library CUs function in AL environment

Hi Team, I am using NAV 2018 CU2 NA version . While I am creating Test Code unit by using available Libraries getting below
message: 'The type or method 'GenerateRandomCode' cannot be used for 'Extension' development.'
Please find below the functions which I am using via libraries but unable to do so because of errors:
LibraryUtility.GenerateRandomCode
-LibraryInventory.CreateItem
-LibraryInventory.CreateNonStock
-Assert.AreEqual
Please suggest how to proceed further in test case in AL environment.
Before logging a new issue please make sure to:

Read the FAQ
Search if there is an existing issue for what you want to log. If it's closed you can reopen it and add your comments.
Use the latest Developer Preview release both the server part and the AL Language extension in VS Code (listed on the repository home page)
If you're not using the latest released version (or are using a private build drop that is newer) please provide the version number in the issue

To help us reproduce the issue and find the root please try to:
A) Attach a minimal set of AL code that causes the issue
B) If the error occurs on the server - copy/paste the server stack trace from the Event Viewer

if I understood you correctly then you want to use dotNet variable? The Dotnet data type is not allowed in extension development. See the FAQ

That's right, you cannot use DotNet in the AL language on NAV 2018.
If you're trying to use functions that do not contain DotNet as part of their parameter list then you can simply change them from Internal to External in C/SIDE, regenerate symbols according to these Docs and then you can use them from AL.


Preview image for save games (logo, template placeholder or player-taken screenshots)

This is a pretty low priority but pretty cool idea suggested over in #1480
Enhance the current load game screen (which in the future could be reused for "scenarios" - premade worlds) with an image component showing a logo / template placeholder / screenshot the player has taken in that world.
Since currently screenshots are "global" (placed in /screenshots regardless of what world it was taken in) there'd need to be an added association of some sort. Symlinks and shortcuts could work, but sound like a messy OS-specific approach. We could just put screenshots into world save dirs instead. If that's done maybe a shortcut to each world save dir with screenshots could perhaps go in the global dir.
Modules and in particular gameplay template modules could include a logo or general screenshot from the world you play in. If such an image is available in a module it could be presented for a saved world based on that module template, if the player hasn't taken any screenshots yet. You could possibly even have said image component on the create game world showing if the template is selected. Or for individual modules on the module selection screen.
Ultimately any such logo could be used on a module tracking site too.
Might even be enough potential for a GSOC item - that time of year is coming up soon.

@Cervator Hello and it is a pleasure speaking with you. My name is Kelso Griffin and if you wish to get to know me a little better, you can find my introduction in the forums under the same profile name kgriff12 here. Although I have already submitted my proposal for a different for standalone NUI and mobile server management utility issues for GSOC I just happened to stumble across this issue. While I feel that I might be of better use with the other more difficult issues, I also wish to work on this issue as well. This issue is still open, correct?

Hello @kgriff12 - the issue is still open. Let us know if you need assistance. You can jump right in!

@Cervator Hello again :) I would like to work on this project and am currently working on a proposal for GSoC. I have an idea of one way this could be implemented but I am not sure if it is the most effective or feasible. My idea is instead of trying to fit in an image on the Create Game or Modules Screen or even creating an entirely new tab or detail screen which the user would have to go through an additional step in order to see the preview image, I was wondering if we could preview the world or module on the background image that is persistent throughout the UI. I believe this would be more effective since that background image is always present and can provide the user with a real-time preview of the world or module without an additional step to access that screen. Of course, there should be a dedicated "preview mode" that works with #2074 with the option to show the image full screen as the UI gets in the way. However, if we want to keep the original background image the same throughout, then a dedicated tab or screen for the preview image would work fine. I would appreciate any feedback so I can center my proposal around the better idea. I'm excited to possibly work with all the great people of Terasology in order to help improve the user experience of this awesome game! :)
P.S. Sorry for being so last minute with this. I only recently started to really dive into GSoC and Terasology last week when I went on break, but I greatly appreciate the welcoming community!

Closing this as completed some time ago - not really sure if there was anything left. can't remember the GSOC project well enough (hi @ar0ne! Hope you've been well). But we do have preview screenshots on the load game page, and also have a module details screen nowdays.


How to share a personal view

I have a personal view i want to share with a colleague.

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

ID: 5c5357c1-386b-f1a3-6dd8-251c06b847e3
Version Independent ID: df0c671f-1237-4fa8-af67-8b5488e183fe
Content: Create, edit, or save an Advanced Find search (Dynamics 365 Customer Engagement (on-premises))
Content Source: ce/customerengagement/on-premises/basics/save-advanced-find-search.md
Service: crm-online
GitHub Login: @mkaur
Microsoft Alias: mduelae

@makdaddy8888 Here's the steps on how to share a personal view.
https://community.dynamics.com/crm/b/crmtipstricksnewstomaster/posts/how-to-share-a-personal-view-in-microsoft-dynamics-365-crm
I will add these step to our help topic as well.
Thanks,
Mints


Adding responsiveness to website

Assign this to me

sir,
may i work with this issue ,can you assigned to me


Build failed without standard library

Hi,
I am trying to use mbedtls in no_std environment with x86_64-unknown-none target without standard library built-in, but I met the following errors when compiling mbedtls:
error[E0277]: the trait bound `rust_alloc::boxed::Box<SavedRawCipher>: Deserialize<'_>` is not satisfied
    --> mbedtls/src/cipher/raw/serde.rs:52:9
     |
52   |         inner_cipher_ctx: Box<SavedRawCipher>
     |         ^^^^^^^^^^^^^^^^ the trait `Deserialize<'_>` is not implemented for `rust_alloc::boxed::Box<SavedRawCipher>`
     |
     = help: the following other types implement trait `Deserialize<'de>`:
               &'a [u8]
               &'a str
               ()
               (T0, T1)
               (T0, T1, T2)
               (T0, T1, T2, T3)
               (T0, T1, T2, T3, T4)
               (T0, T1, T2, T3, T4, T5)
             and 102 others
note: required by a bound in `next_value`
    --> /home/tdvf/.cargo/registry/src/github.com-1ecc6299db9ec823/serde-1.0.101/src/de/mod.rs:1845:12
     |
1845 |         V: Deserialize<'de>,
     |            ^^^^^^^^^^^^^^^^ required by this bound in `next_value`

Some errors have detailed explanations: E0277, E0407.
For more information about an error, try `rustc --explain E0277`.
error: could not compile `mbedtls` due to 4 previous errors

error[E0405]: cannot find trait `Default` in this scope
    --> /home/tdvf/.cargo/registry/src/github.com-1ecc6299db9ec823/byteorder-1.3.2/src/lib.rs:1831:6
     |
1831 | impl Default for LittleEndian {
     |      ^^^^^^^ not found in this scope
     |
help: consider importing this trait
     |
83   | use core::default::Default;
     |

Some errors have detailed explanations: E0405, E0425, E0463.
For more information about an error, try `rustc --explain E0405`.
error: could not compile `byteorder` due to 173 previous errors

The error message is too long, so I only pasted part of it here.
The build command I'm using is:
cargo build -p mbedtls -Zbuild-std=core,alloc,compiler_builtins --target=x86_64-unknown-none --no-default-features --features="no_std_deps,rdrand"

The same result applies to:
cargo build -p mbedtls -Zbuild-std=core,alloc,compiler_builtins --target=x86_64-unknown-linux-gnu--no-default-features --features="no_std_deps,rdrand"

From the compiler output, it seems the default features of byteorder should not be enabled and the alloc feature of serde should be enabled under no_std.
I submitted a pull request https://github.com/fortanix/rust-mbedtls/pull/209 trying to fix this error. Please take a review, thanks🙂


Users list: sorting by last_name only

It is necessary to be able to sort the list of users only by the 'last_name' without regard to 'status'

This isn't possible in Zulip, since we don't have a "last name" field, just a full name field (which is desirable for internationalization -- not everyone's name has a "first last" structure).


chore: add example for parameter/value to Helm Values

Fixes #4009

I'm not sure why Docs preview failed 🤔 Re-running the other failed jobs too. Should be fine though!

@jsjoeio It's probably because I'm doing this from a fork, if I can't get it into a good state then I'll push it as a branch instead - I wanted to use the same workflow that our external contributors would use, as I think that is an important use case for us to support

@jsjoeio It's probably because I'm doing this from a fork, if I can't get it into a good state then I'll push it as a branch instead - I wanted to use the same workflow that our external contributors would use, as I think that is an important use case for us to support

Love that you're making sure we dogfood that workflow! I'm seeing this:

cc @BrunoQuaresma is there any reason this workflow doesn't work for forks? Is it because m is private? I'm not fully sure how this should work 🤔

@bpmct I think it is because of that since forks don't have access to our internal secrets.

Ah, I think we have to configure this for our secrets: https://docs.github.com/en/actions/reference/environments
Not worth doing right now though, I didn't change docs so I think this is safe to merge :crossed_fingers:


Pino v7+ transports

Pino v7+ allows you to create a transport, which is a module that runs in a worker thread to consume logs (docs).
Knip is currently giving a false positive for an unused file containing my transport.
Here is a minimal reproduction: https://stackblitz.com/edit/github-ggjbcu?file=src%2Findex.ts
import { pino } from 'pino';

const logger = pino({
  // ./transport.js, or in a TypeScript project ./transport.ts, should not be unused
  transport: { target: import.meta.resolve('./transport.js') },
});

logger.info('Hello, world!');

For this to work as expected, does Knip need to provide a Pino plugin?

Knip already adds entry files from require.resolve() specifiers. I think we just need to add the same for import.meta.resolve.

This is a core feature btw, no need for a plugin.


Document GameOfLifeState

Codecov Report

Merging #923 (adc8d5c) into main (64397c2) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #923   +/-   ##
=======================================
  Coverage   64.58%   64.58%
=======================================
  Files         153      153
  Lines        7751     7752    +1
  Branches     1441     1448    +7
=======================================
+ Hits         5006     5007    +1
+ Misses       1859     1854    -5
- Partials      886      891    +5

Impacted Files
Coverage Δ

...com/alexvanyo/composelife/model/GameOfLifeState.kt
100.00% <ø> (ø)

... and 8 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat: possibility to combine DI and middleware

Description
Currently when we want to declare a middleware we use this:
Handler middleware(Handler handler) {
  return (context) async {
    final response = await handler(context);
    return response;
  };
}

But adding a dependency injection inside the return doesn't seem to be possible like this:
Handler middleware(Handler handler) {
  return (context) async {
    handler.use(provider<String>((context) => 'sometext'));
    final response = await handler(context);
    return response;
  };
}

instead we have to declare the injection outside:
Handler middleware(Handler handler) {
  return (context) async {
    final response = await handler(context);
    return response;
  }.use(provider<String>((context) => 'sometext'));
}

Currently it doesn't seem like we can use the data in the request to inject anything.
The main issue I am having with this is, I want to implement some sort of an authentication and of course I don't want to check for the authentication inside the routes separately every time.
Since we don't have annotations like in flask, I thought the answer for that is inside the middleware, but I can't decode a JWT which would be asynchronous in this case and then pass it down to the connected routes. The same goes for the initialization of a db connection.
Would there be any way to pass down data inside the return in a middleware or is there perhaps a different way to do this?

Hi @dbilgin 👋
Thanks for opening an issue!
If I understood correctly, you should be able to define your own custom middleware and chain them via the use API like:
import 'package:dart_frog/dart_frog.dart';

Handler middleware(Handler handler) {
  return handler
      .use(_customMiddleware())
      .use(provider((context) => 'some text'));
}

Middleware _customMiddleware() {
  return (handler) {
    return (context) async {
      final response = await handler(context);
      print('custom middleware');
      return response;
    };
  };
}

Let me know if that helps 👍

@felangel thanks a lot for your answer!
I seem to be stuck in the same place unfortunately. Sorry if it wasn't clear before, really appreciate the help.
So let's say I receive a JWT in the header and I want to decode this JWT and check its validity asynchronously and if it is invalid, I would return a response of 401, but if it is valid, I would get some data from it and pass it onto my route. What I want to do is basically something like below:

Handler middleware(Handler handler) {
  return (RequestContext context) async {
    final authHeader = context.request.headers['Authorization'].toString();
    final String? jwtData = await resolveJWT(authHeader);
    if (jwtData == null) {
      return Response(statusCode: 401);
    }

    // ---> At this point I have checked the validity and returned 401 if necessary, after this
    // check I would want to inject the
    // `jwtData` into my route so that I can actually use the data for and I wouldn't have to decode the JWT
    // separately everywhere. But here I can't call something like below I think:
    handler.use(provider((context) => jwtData));

    final response = await handler(context);
    return response;
  }
}

So in this case I would have to get the jwtData one more time in my route separately or do you think there is a way to do this?

@dbilgin I put together an example of how this could look:
// routes/_middleware.dart
import 'dart:io';

import 'package:dart_frog/dart_frog.dart';

Handler middleware(Handler handler) {
  return handler.use(headerValidator());
}

/// Checks for presence of a `foo` header.
/// If `foo` header is not present, returns a 403.
/// If `foo` header is present, provides the value via context.
Middleware headerValidator() {
  return (handler) {
    return (context) {
      final request = context.request;
      if (!request.headers.containsKey('foo')) {
        return Response(statusCode: HttpStatus.forbidden);
      }
      final value = request.headers['foo'] as String;
      return handler(context.provide<String>(() => value));
    };
  };
}

// routes/index.dart
import 'package:dart_frog/dart_frog.dart';

Response onRequest(RequestContext context) {
  return Response(body: context.read<String>());
}

Let me know if that helps 👍


Fix voxel dim calc

square flat shapes cause the voxel dimensions to be miscalculated and allocate vastly more voxels than intended. see v-hacd latest

Thanks! CI failure is not caused by this PR.


3 Supporter Logos to add: Clairvolex, IGBA, EWH

@tsada

these were posted in the wrong place -- should be supporters not pledgors. removing from pledgors section now

moved to supporters

Thanks Jane, sorry for the mistake. I had asked Jing to work on this but I
think I could have given clearer instructions on adding links and having it
added in correct places
Best
Tricia
On Thu, Jun 4, 2020 at 10:07 AM Jane<EMAIL_ADDRESS>wrote:

moved to supporters
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/janetpkr/open-covid-pledge/issues/5#issuecomment-638985746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZRWIPB2T6IOXY3WTSQHPTRU7IERANCNFSM4NQ6ZOXQ
.

--
Tricia Sada
Industrial Designer
http://goog_1481471341
triciasada.com http://triciasada.com


Remove WITH_KIND

Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>Followup of #749

Proposed Changes

:broom: Removed WITH_KIND env when pushing data plane images. Now KO_DOCKER_REPO = "kind.local" is enough

cc @pierDipi

Hi Francesco 👋, thanks for doing this!
KinD actions are failing.

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests


Display Antennas on Map

Description
Use access_points.json to fetch the coordinates of each antenna. Use the coordinates to display the antennas on the map. You can use the <Circle />  component to render the antenna.
Definition of Done
Create a component that renders the antennas on the map.

Great progress so far, let's work on

Making the dots smaller
Making the dots not transparent


Trace panel not closed when trace is removed from server

If a trace is removed from the 'OPENED TRACES view using the remove button, then corresponding trace panel is not closed.

Picking this up


Solves UnboundLocalError and use temp dir context manager for cleanup

Hi Team,
I've got this error lately when using the --ml flag:
Saving segmentations...
  Saved in 0.00s
Traceback (most recent call last):
  File "/home/chris/.conda/envs/test/bin/annotator", line 8, in <module>
    sys.exit(main())
  File "/home/chris/.conda/envs/test/lib/python3.9/site-packages/annotator/script.py", line 131, in main
    seg = nnUNet_predict_image(tmp_input, args.output, task_id, model="3d_fullres", folds=folds,
  File "/home/chris/.conda/envs/test/lib/python3.9/site-packages/totalsegmentator/nnunet.py", line 230, in nnUNet_predict_image
    return img_data
UnboundLocalError: local variable 'img_data' referenced before assignment

I think I may have spotted the logic error in the function and get to add a proper way to cleanup temp files along the way, as suggested by the TODO comment.
Let me know how this works for you.
Great job guys anyway!

Thanks for this PR. I will have a look at it.

Looks great. Thanks for adding this. I merged it.


My First PR commit

PR comments

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: cjxd-bot-test
If they are not already assigned, you can assign the PR to them by writing /assign @cjxd-bot-test in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cjxd-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
7805f9ddebb8d9ba9ae23a92159e037061c11f1a
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Error at definition of Type

I try write this:
type T = {

but get an error:
SyntaxError: found "{", expected NamedType ~ Refinement.? | Refinement at index 9
type T = {
         ^

If i use scala REPL, all works fine:
scala> type T = {
     | type X = Int
     | def x: X
     | type Y
     | def y: Y
     | }
defined type alias T

Looks like a bug in ScalaParse where it's not properly inserting a ~/ cut after the open-curly after a type declaration:
https://github.com/lihaoyi/fastparse/blob/master/scalaparse/shared/src/main/scala/scalaparse/Types.scala#L32
The second ~ on that line should be a ~/ to make this behave correctly.

This is fixed in master


Fixing the testing scripts

@Kirkkt Please have a look, see if this seems correct

awesome! Thanks, John!
One thing real quick: make sure to run make under the web sdk root directory. this will update the dist/ folder according to your changes in the src/ folder. without running make, changed made in src folder won't be effective.

@Kirkkt new version up with the extra assert removed, keeping the one that was uncommented instead, and with the dist folder added

@Kirkkt Oops yeah I missed a few asserts...please have a look when you have time

Changes made to revive the queue, ptal

@Kirkkt Changed the queue continuation to handle errors better, PTAL when you can

Catch removed, finally on the new line...yeah, strange that the linter didn't catch it, it has in the past I thought...

great. looks good to me :+1:

:+1:


By default variables in XML templates aren't escaped

Doing some mocking with moto, and I have & in my test data. This kind of thing fails:
@mock_s3
def test_boto3_list_keys_xml_escaped():
    s3 = boto3.client('s3', region_name='us-east-1')
    s3.create_bucket(Bucket='mybucket')
    s3.put_object(Bucket='mybucket', Key='Q&A.txt', Body=b'is awesome')
    s3.list_objects_v2(Bucket='mybucket')

errors because the & isn't escaped <Key>Q&A.txt</Key>:
  File "/home/me/dev/moto/.tox/py34/lib/python3.4/site-packages/botocore/parsers.py", line 386, in _parse_xml_string_to_dom
    "invalid XML received:\n%s" % (e, xml_string))
nose.proxy.ResponseParserError: Unable to parse response (not well-formed (invalid token): line 9, column 18), invalid XML received:
b'<?xml version="1.0" encoding="UTF-8"?>\n<ListBucketResult xmlns="http://s3.amazonaws.com/doc/2006-03-01/">\n  <Name>mybucket</Name>\n  <Prefix>None</Prefix>\n  <MaxKeys>1000</MaxKeys>\n  <Delimiter>None</Delimiter>\n  <IsTruncated>false</IsTruncated>\n    <Contents>\n      <Key>Q&A.txt</Key>\n      <LastModified>2016-10-30T19:53:45.011Z</LastModified>\n      <ETag>"d32bda93738f7e03adb22e66c90fbc04"</ETag>\n      <Size>10</Size>\n      <StorageClass>STANDARD</StorageClass>\n      <Owner>\n        <ID>75aa57f09aa0c8caeab4f8c24e99d10f8e7faeebf76c078efc7c6caea54ba06a</ID>\n        <DisplayName>webfile</DisplayName>\n      </Owner>\n    </Contents>\n  </ListBucketResult>'

Similarly listing by prefix, regardless of objects in s3 fails:
s3.list_objects_v2(Bucket='mybucket', Prefix='<>')
# writes
<Prefix><></Prefix>

This is because in moto/core/responses.py _TemplateEnvironmentMixin.should_autoescape was set to False.
Is this a deliberate decision to not support characters that require escaping in XML?

Certainly not an intentional decision. A PR would be very welcome if anyone has time.

Fixed with #779


add a new prop to card in order to flex a class for a new style

Summary
New prop called variant used to flex to different card styles for new use cases. Falls back to default. Currently only adds the option for raised which adds a new css custom attribute for the box shadow.
Additional Details
Issue can be found here https://github.com/cerner/terra-core/issues/1432
Thanks for contributing to Terra.
@cerner/terra

@mjhenkes if @alex-bezek just wants to remove the enumeration from this PR and do it with strings, I can just pick it up in #1044 if we decide to go forward with it.

@dylan-sftwr Sounds good. Lets not block this PR.

Updated to use an object inside the component, but not export it out.
Also updated the default from none to the box shadow the @neilpfeiffer provided.
Deployment link here: https://terra-core-deployed-pr-1458.herokuapp.com/#/site/components/site/card/index
Let me know if there are any other issues

Tested: PR #1458
Resolves: Issue #1432
Reviewed:

Reviewed Enhancement to add an additional card variant style: raised
Follows pattern considerations documented in Cerner Design Standards: Cards
Follows UX defined design implementation for Terra-Card
Follows all defined theme designs: terra-theme

Verified by @neilpfeiffer and @scottwilmarth, Ready for Release after Functional Validation.

Pass

UX Review


build(deps): bump @testing-library/react-native from 12.1.3 to 12.3.0

This PR resolves []
Description
Bumps rntl

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

bump @testing-library/react-native from 12.1.3 to 12.3.0 - gkartalis

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (bump @testing-library/react-native from 12.1.3 to 12.3.0 - gkartalis)

  Generated by :no_entry_sign: dangerJS against 5dfc77be9551a0cc6f850e2775b92ef9208ebc53


[SPARK-38125][BUILD][SQL] Use static factory methods instead of the deprecated Byte/Short/Integer/Long constructors

What changes were proposed in this pull request?
This PR aims to use static factor methods instead of the deprecated Integer constructors and add Java/Scala linter rules to enforce new styles.
Why are the changes needed?
Byte/Short/Integer/Long constructors are deprecated in Java 9.

https://docs.oracle.com/javase/9/docs/api/java/lang/Byte.html#Byte-byte-
https://docs.oracle.com/javase/9/docs/api/java/lang/Byte.html#Byte-java.lang.String-
https://docs.oracle.com/javase/9/docs/api/java/lang/Short.html#Short-short-
https://docs.oracle.com/javase/9/docs/api/java/lang/Short.html#Short-java.lang.String-
https://docs.oracle.com/javase/9/docs/api/java/lang/Integer.html#Integer-int-
https://docs.oracle.com/javase/9/docs/api/java/lang/Integer.html#Integer-java.lang.String-
https://docs.oracle.com/javase/9/docs/api/java/lang/Long.html#Long-long-
https://docs.oracle.com/javase/9/docs/api/java/lang/Long.html#Long-java.lang.String-

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs with the newly added Scalastyle and Java Checkstyle.

Thank you, @srowen . Let me check.

Hi, @srowen . The PR is updated to handle Byte/Short/Integer/Long together.

Linter job passed and only one irrelevant flaky test failed.

Merged to master.

Thank you @srowen .


URL bar spellchecking  - red underline

The Issue
Disable spell check in URL . Some users are seeing:

System Configuration

Device:
Android version:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of the user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

Fixed.


Openshift 3.11 fails with exit status=2

Describe the bug
The template based installation fails with exit status=2
To Reproduce
Steps to reproduce the behavior:
Always when i run installation with template it fails.
az group deployment create --name OpenShiftAzure_Cluster3 -g OpenShiftAzure_Group2
--template-uri https://raw.githubusercontent.com/Microsoft/openshift-container-platform/master/azuredeploy.json
--parameters @./azuredeploy.parameters.json

Deployment failed. Correlation ID: 7626e2d1-a06b-4a59-84e4-a7d8e8e40b99. {
"status": "Failed",
"error": {
"code": "ResourceDeploymentFailure",
"message": "The resource operation completed with terminal provisioning state 'Failed'.",
"details": [
{
"code": "DeploymentFailed",
"message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.",
"details": [
{
"code": "Conflict",
"message": "{\r\n  "status": "Failed",\r\n  "error": {\r\n    "code": "ResourceDeploymentFailure",\r\n    "message": "The resource operation completed with terminal provisioning state 'Failed'.",\r\n    "details": [\r\n      {\r\n        "code": "VMExtensionProvisioningError",\r\n        "message": "VM has reported afailure when processing extension 'deployOpenShift'. Error message: \"Enable failed: failed to execute command: command terminated with exit status=2\n[stdout]\nWed Jul 3 04:34:31 UTC 2019  - Starting Script\nWed Jul 3 04:34:32 UTC 2019  - Configuring SSH ControlPath to use shorter path name\nWed Jul 3 04:34:32 UTC 2019  - Cloning Ansible playbook repository\nCloning into 'openshift-container-platform-playbooks'...\n - Retrieved playbooks successfully\nWed Jul 3 04:34:34 UTC 2019  - Create variable for routing certificate based on certificate type\nWed Jul 3 04:34:34 UTC 2019  - Create variable for master api certificate based on certificate type\nWed Jul 3 04:34:34 UTC 2019  - Create variable for master cluster address based on cluster type\nWed Jul 3 04:34:34 UTC 2019  - Creating Master nodes grouping\nWed Jul 3 04:34:34 UTC 2019  - Creating Infra nodes grouping\nWed Jul 3 04:34:34 UTC 2019  - Creating Nodes grouping\nWed Jul 3 04:34:34 UTC 2019  - Create Ansible Hosts file\nWed Jul 3 04:34:34 UTC 2019  - Running DNS Hostname resolution check\n\nPLAY [all] *********************************************************************\n\nTASK [Wait for DNS hostname resolution - will try for up to 33 minutes] ********\nfatal: [devopenshift2-master01]: FAILED! => {\"changed\": false, \"elapsed\": 2000, \"msg\": \"Timeout when waiting for devopenshift2-master01:22\"}\nfatal: [devopenshift2-infra01]: FAILED! => {\"changed\": false, \"elapsed\": 2000, \"msg\": \"Timeout when waiting for devopenshift2-infra01:22\"}\nfatal: [devopenshift2-node01]: FAILED! => {\"changed\": false, \"elapsed\": 2000, \"msg\": \"Timeout when waiting for devopenshift2-node01:22\"}\n\nPLAY RECAP *********************************************************************\ndevopenshift2-infra01      : ok=0    changed=0    unreachable=0    failed=1   \ndevopenshift2-master01     : ok=0   changed=0    unreachable=0    failed=1   \ndevopenshift2-node01       : ok=0    changed=0    unreachable=0    failed=1   \n\n\n[stderr]\n  % Total    % Received % Xferd Average Speed   Time    Time     Time  Current\n                                 Dload  Upload   Total   Spent    Left  Speed\n\r  0     0    0     0    0     0      00 --:--:-- --:--:-- --:--:--     0\r100    10  100    10    0     0   1522      0 --:--:-- --:--:-- --:--:--  1666\n [WARNING]: Could not create retry file '/home/infyadmin/openshift-container-\nplatform-playbooks/check-dns-host-name-resolution.retry'.         [Errno 13]\nPermission denied: u'/home/infyadmin/openshift-container-platform-playbooks\n/check-dns-host-name-resolution.retry'\n\"."\r\n      }\r\n    ]\r\n  }\r\n}"
}
]
}
]
}
}

Expected behavior
It should deploy openshift
Screenshots
If applicable, add screenshots to help explain your problem.
stdout
Include the last 100 lines of stdout from Bastion host - see troubleshooting https://docs.microsoft.com/en-us/azure/virtual-machines/linux/openshift-troubleshooting
Template Information (please complete the following information):

OS: RedHat 7.6 latest

Additional context
Add any other context about the problem here.

template_content.txt
Attached the template file used

Any help  please

the value for domainName must be 'none' if you are not deploying a private cluster.

Hi @haroldwongms ,
Ya this is not a private cluster. Are you mentioning about this parameter from parameters file?

  "domainName": {
  	"value": "contoso.com"
  },

my parameter file has

  "masterClusterType": {
  	"value": "public"
  },
  "routerClusterType": {
  	"value": "public"
  }


Investigate SonarCloud and have a Proof of Concept on Besu repository.

We should investigate how to set up SonarCloud and Besu.
ideally the investigation will show the results on the analysis in an actual report.
Metrics to consider:

potential security vulnerabilities
technical debt size
code coverage

More context on this in issue [#2811](https://github.com/hyperledger/besu/issues/2811. SonarQube has since been deployed, see PR #2852


26/push library automatically

Codecov Report
Base: 94.56% // Head: 94.47% // Decreases project coverage by -0.08% :warning:

Coverage data is based on head (c2280d6) compared to base (594637b).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
- Coverage   94.56%   94.47%   -0.09%
==========================================
  Files          15       15
  Lines         478      489      +11
==========================================
+ Hits          452      462      +10
- Misses         26       27       +1

Impacted Files
Coverage Δ

src/aiotaskq/__init__.py
100.00% <100.00%> (ø)

src/aiotaskq/tests/test_cli.py
100.00% <100.00%> (ø)

src/aiotaskq/tests/test_worker.py
98.68% <100.00%> (-1.32%)
:arrow_down:

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

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


Update peribolos fork periodically

@tumido update this and discuss if it's really needed to maintain our own fork/patchset.


[Paradox.Console.PythonInterpreter] Automatically load types for method params

First iteration for param completion completed.


Function API support

So functions were recently announced, along with some other new API stuff. I'll see how this can be implemented into current API.

My OpenAI API key ran out of tokens and I can't really add balance, so I will be releasing functions experimentally. If any issues arise please create an issue so I can fix them as soon as possible.


Sharing is not allowed from an account on BUSINESS CRITICAL edition to an account on a lower edition.

When creating a share resource, for example:
resource snowflake_share share {
  name     = "share"
  accounts = "ABC12345"
}

This will end up issuing the following statement:
ALTER SHARE "SHARE" SET ACCOUNTS=ABC12345;
If the account running the command is on BUSINESS CRITICAL edition and ABC12345 is a lower edition you'll get the following error:
SQL execution error: Sharing is not allowed from an account on BUSINESS CRITICAL edition to an account on a lower edition. The following account(s) are on editions lower than BUSINESS CRITICAL: ABC12345. You can override this check for this share by using the OVERRIDE SHARE RESTRICTIONS privilege and the SHARE_RESTRICTIONS flag. Before overriding this check, confirm that the target account has appropriate security controls and is compliant for the data being shared. For example, editions lower than BUSINESS CRITICAL are not PCI-DSS certified.
Here's snowflakes documentation for reference.
The solution is to issue this statement as ACCOUNTADMIN (or with a role that has been granted the OVERRIDE SHARE RESTRICTIONS privilege):
ALTER SHARE "SHARE" SET ACCOUNTS=ABC12345 SHARE_RESTRICTIONS=FALSE;
Ideally we would be able to specify the resource like so:
resource snowflake_share share {
  name     = "share"
  accounts = "ABC12345"
  share_restrictions = false
}

or perhaps to support a mix of account types and still wanting to keep the restriction for some:
resource snowflake_share share {
  name     = "share"
  accounts = {
    locator = "ABC12345"
    share_restrictions = false
  }
}

It is rather irritating that this has to be bypassed via provisioner or managed outside of Terraform entirely. We run only Business Critical accounts in my company due to non-HIPAA related reasons, and sharing between them is a pain (as they are not HIPAA compliant and won't be, it does not matter that they are all Business Critical)


Added TBM.Trade feature

New TBM feature to retrieve received and requested Trades of a user

Looks good, is there a reason you didn't add "all" yet?

When adding all, i also want to add more information about a trade which is accepted/rejected.
Something like:

status of trade
private field information
Trade-rating information

Just hadn't enough time yet.
Will implement this in the next days.


docs(tofs): apply remaining comments from PR #37

Continues #37.

@myii merged it

Thanks, @aboe76.

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


Move validation of class enum trait to run time instead of import time

Please describe the use case that requires this feature.
Currently, we define class traitlets like this:
class Foo(Component):
    bar_type = create_class_enum_trait(Bar...).tag(config=True)

this runs at import time of Foo. Which makes it impossible to use subclasses of Bar that are only imported at a later stage.
This is problematic for subclasses defined in other submodules than the base class and the containing component.
THis was triggered by the machine learning module, where ShowerProcessor has a class enum trait of Reconstructor but because ctapipe.ml.sklearn cannot import Reconstructor without also importing ShowerProcessor (as ctapipe.reco.__init__ imports it), the ml reconstructors are not yet defined when the class enum trait is created.
This is even more problematic for the case of plugins.
Describe the solution you'd like
Move the check if a provided string is an available class to the runtime (i.e. the validate method).

So this is the reason for this:
https://github.com/cta-observatory/ctapipe/blob/05beb25fb31f27f243f25b4542b1dcea6201300e/ctapipe/reco/__init__.py#L3-L4

Exactly


Chord Diagram: Fix link logic

#97

@reb-dev Looks like we need to add an image to the Gallery example

@reb-dev Also there's a problem with labels alignment when radiusScaleExponent is not default:
https://user-images.githubusercontent.com/755708/218878729-d5cc7737-5b1b-4ca5-99bb-3c771bb3b456.mp4

@reb-dev One more thing, the fact that the links are being re-drawn every time the graph updates means that our component doesn't implement the enter-update-exit logic properly. We can fix that fairly easily by providing the second argument to the .data calls:

You can check how it's done for the Graph component and copy the code from there.


The lexer shouldn't need to allocate Strings

It should be possible to use &str somehow.

After #45 and #46 there is only one function left in lexer which allocates: read_text_group. The problem with it is that it may skip or transform whitespace, so it can't just return a string slice.

One idea is to do the text transformations later. For example in ast.rs.


Bug: White box below custom scrollbar

Describe the bug
There is a small white box below the custom scrollbar on the embeds
To Reproduce
Go to the homepage
Observe the white box below the scrollbar on the embed
Expected behavior
The box shouldn't be there

This can be fixed by adding box-sizing: border-box to the textarea to remove the horizontal scrollbar. This will be fixed in the next update.

It seems like this has been fixed despite no changes to the code. I'm leaving this open while I investigate why.

This reoccurred and was fixed by adding x-overflow: none to the textarea. This change will be released in the next update.


Check the args when init class and fix some typo.

throw the errors earlier
~common line_length = 79 in black config~

To be honest, I don't like the 79 limit, so I didn't configure this option before.

To be honest, I don't like the 79 limit, so I didn't configure this option before.

ok, I remove it.

BTW,  can support simple rate-limit in memory?

Yes, we can add a new memory backend. In fact, I wanted to add this function at first, but I've been busy with other work. If you are willing to submit PR, I will be happy to help you complete it together.

Yes, we can add a new memory backend. In fact, I wanted to add this function at first, but I've been busy with other work. If you are willing to submit PR, I will be happy to help you complete it together.

OK, I will add it later.


Unexpected behaviour with DD_PROCESS_AGENT_ENABLED=false

Describe what happened:
When DD_PROCESS_AGENT_ENABLED is precisely "false", neither container (unexpected) nor process-level metrics (expected) are collected.
Describe what you expected:
I would expect that the collection of container-level metrics would be unaffected by a value of "false" for DD_PROCESS_AGENT_ENABLED.
Additional environment details (Operating System, Cloud provider, etc):

AWS
ECS Fargate
docker/agent:6.13.0 (with local extensions for secrets injection only)
ECS tasks deployed by terraform (where conditionally not defining an environment variable is not easy).

Workaround
In my case, using the empty string in place of "false" had the desired effect of disabling process-level metrics while maintaining container-level metrics.

Hi @jonseymour
By container metrics, do you mean the metrics in the container live view?
These metrics are also handled by the process agent, which means that if it is turned off, these value won't be collected.
This behavior is listed in the bug fixes as it did not work like this before: https://github.com/DataDog/datadog-agent/blob/1e2200ba7b82009c1537310c83c3fa8b1cd2b018/CHANGELOG.rst#bug-fixes-2

Fix issue on process agent for DD_PROCESS_AGENT_ENABLED where 'false' did not turn off process/container collection.

Let me know if you have further questions,
Simon

I think the documentation improvements help a bit, but I have to say it isn't an intuitive design for that variable at all.
The name of the variable -- _ENABLED -- suggests that it takes a boolean value (true or false) and if neither is specified then one might assume that the default value would be false. However, one finds that if one follows this intuition, rather than having the prescience to look up the source code or, now, the documentation then the variable doesn't take boolean values at all. This is reinforced by the fact that the agent documentation for "Live Containers" makes no reference whatsoever to the concept of a process agent or configuration for same and there is a completely separate section for "Live Processes" which does explicitly talk about the process agent and its configuration and makes no reference to any of this configuration affecting the operation of the "Live Containers" feature described elsewhere.
It seems to me that DD_PROCESS_AGENT_ENABLED was designed it was, not for the benefit of the users of the product but rather for the convenience of the implementation.


android buildToolsVersion missing

when building android platform, the line
buildToolsVersion '2x.x.x'
is missing in file build.gradle generating a build error
AAPT: error: attribute android:appCategory not found.

I just built a debug APK, installed it on my phone (Samsung Galaxy S8), and it ran fine. I'm using build tools 28.0.3, and I don't think the current Gradle config needs the build tools specified. It's possible that there's a breaking change in later build tool versions. If I remember this right, liftoff used to put a build tools version in android/build.gradle, but an IDEA update gave a warning when that version was specified, saying it was no longer needed. I could easily be wrong about the file or other specifics.
What version of build tools are you using (in general, more Android info would be useful), and what Gradle command did you run that gave that error? I'm assuming you already updated to the current gdx-liftoff before posting an issue.
I don't know if I have enough disk space to install Android Studio in addition to IDEA; I am extremely low on available space, so it would be a challenge for me to test other build tool versions.

I just built an APK version with and without build tools specified, so I'm not sure what omitting it does...

OK, buildToolsVersion is present now in the latest code (not in a release yet); I don't know if that's sufficient for a fix, but I think I can close this.


update requirements

Codecov Report

Merging #8 (27901a5) into main (830ecdf) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   99.80%   99.80%
=======================================
  Files          12       12
  Lines         519      522    +3
=======================================
+ Hits          518      521    +3
  Misses          1        1

Impacted Files
Coverage Δ

pydeflate/__init__.py
100.00% <100.00%> (ø)

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


android getInitialNotification error

Hi! After update to react-native-fcm 9.6.0 i get this error on android device (android 5.1):

tmp workaround is to change FIRMessagingModule code:
if(activity == null || activity.getIntent().getAction().equals("android.intent.action.MAIN")){
     promise.resolve(null);
     return;
}

to
if(activity == null || activity.getIntent().getAction() == null || activity.getIntent().getAction().equals("android.intent.action.MAIN")){
    promise.resolve(null);
    return;
}

I am not a strong android developer and i don't know why activity.getIntent().getAction() could be null so i will not create PR cuz i think that the problem is deeper.
RN version is 0.44.0 if it matters

that's a bug, will fix it ASAP

fixed v9.6.1

Thank you for the quick response and great lib! Could you please look at my the comment to your commit

you are right..


npm publised version is not the latest tagged version

Could you please check, i see v1.0.4 on npm , while 1.0.6 is the latest stable in github.
Thank you.

Forgot to publish the releases, thanks for letting me know. It's now pushed.


Long tap delay

It feels sluggish, and looking at a trace it shows a ~300ms delay between tap and click.
The 300ms tap delay returneth? I wonder if it's because we have horizontal scrolling.
I thought touch-action would have sorted this, but it seems not.
Let's look at this once the GL renderer has landed.

This seems to have been fixed along the way.


WIP: Enable client tools and allow uyuni-master and uyuni-stable for all modules

Please check why Travis complains

Yes, this is still WIP, don't worry :-)
I will address it after Uyuni release.

@juliogonzalez What is left here that needs to be done? Fixes so Travis doesn't complain anymore or is there anything else?

Well, Travis doesn't complain aymore because the build is missing. I will rebase the README and try to relaunch the tests.

So... it's complaining. I will need to review what's happening :)

And finally PR is ready to be reviewed. I tested and the Uyuni environment is able to provision fine, with all the expected repositories.
Preparing now a PR for sumaform-test-runner about the new minionssh terraform module.

Examples changed, and added an example for Uyuni.

In fact, yes. We don't have CentOS6 client tools, or SLE15 client tools. Adding them now.


Create a Google CodeLab tutorial for OTNS2

See for current OTNS codelab: https://openthread.io/codelabs/openthread-network-simulator
Idea is to make an additional version for OTNS2.

Draft version available as PR: https://github.com/openthread/ot-ns/pull/530

Add to tutorial: minimum Ubuntu 22.04 is required (for the right Go >= 1.18 and Python >= 3.9 versions).


CC86571: Adding description of 'variant'

Hello, @mairaw, @isaac2004, @BillWagner, @mikeblome,
This proposed file change comes from https://github.com/dotnet/docs.ja-jp/pull/200 .
Could you review this contribution and help to merge if agreed?
Many thanks in advance.
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

I've made the suggested fix, @BillWagner and @v-makoud

Thanks @rpetrusha I've approved this PR.


https://www.scala-sbt.org/ Origin DNS error

Steps
https://www.scala-sbt.org/
Problem

@SethTisue @JustinPihony Do you know what could be up, or is this internet weather?

was the website not included in the handoff from Lightbend/Akka to the Center...? wondering if this is actually Fabien's wheelhouse

Not sure cc @adpi2, @sjrd?

":we don't own it... that was given to scala center iirc" says Justin

@fsalvi assuming Justin is right, can you check on it?

Yes, I double checked in my emails. This was turned on in early August 2023 and we turned it off from our end at the end of September 2023 after verification from Fabien.

https://www.scala-sbt.org/ healed itself after your comment. Friday the 13th spookiness.


Grid width not respected

The grid width isn't respected. It defaults to 12.
Example at: https://storage.googleapis.com/translucid-bucket/data/demo/two.html

well your problem is that you have not edited the css (sass) files and made the changes you want
in order for you to obtain the results your looking for you have to edit the sass varaible @gridstack-columns to be 20 ...
try that and tell us what happens ... i believe that spreading the columns structure over css and javascript is another problem that needs to be addressed ...

I've added:
<style type="text/css">
        #grid1 {
            background: lightgoldenrodyellow;
			width: 600px;
			height:400px;
        }

        .grid-stack-item-content {
            color: #2c3e50;
            text-align: center;
            background-color: #18bc9c;
        }

		.grid-stack-item {
			$gridstack-columns: 20;
			@for $i from 1 through $gridstack-columns {
				&[data-gs-width='#{$i}'] { width: (100% / $gridstack-columns) * $i; }
				&[data-gs-x='#{$i}'] { left: (100% / $gridstack-columns) * $i; }
				&.grid-stack-item[data-gs-min-width='#{$i}'] { min-width: (100% / $gridstack-columns) * $i; }
				&.grid-stack-item[data-gs-max-width='#{$i}'] { max-width: (100% / $gridstack-columns) * $i; }
			}
		}

    </style>

however no result.

i tested it out and it is working ..
it seems that you're doing it wrong ...
where do you have that peace of code above .. you know that sass has to be compiled ?!

Thanks for the help, I think I'll just stick with the gridster then.

Please check this section https://github.com/troolee/gridstack.js#extra-css
You can include pre-made CSS with different columns setup.
Here is a sample http://troolee.github.io/gridstack.js/demo/two.html


Request: Fakesign all the binaries in the .ipa for jailbroken users to use with AppSync

This way jailbroken users don't have to sign the app (won't cause any trouble for non-jailbroken ones). For every macho just do: ldid -S /path/to

Does this work with the required entitlements? I.e. has this been tested? I don’t have a jailbroken device.
Is the preferred format IPA or DEB?

Yes, this should work. (I don't know if anyone has tested it with UTM specifically)
IPA, because most app sideloaders(iFunbox, impactor and co.) only accept IPA files.

I'm asking 1 specifically for someone to test it because UTM requires get-task-allow entitlement. I'm not sure if ldid will give it. My hunch is no.

Do you have a fakesigned IPA? I could try to install it and test if it works

No, I don't.

Sorry, but initially answered because I knew the answer for your question 2. about the preferred format of apps is.

I made a script that automatically fakesigns an .ipa file using ldid from Homebrew.
Just type "path/to/fakesigner.sh path/to/UTM.ipa" and it will produce a file called UTM.ipa-fakesigned.ipa
This build has been tested by installing it on a jailbroken iPhone 7 with AppSync Unified via iFunBox and booting a VM

#43 please help me!

I made a script that automatically fakesigns an .ipa file using ldid from Homebrew.
Just type "path/to/fakesigner.sh path/to/UTM.ipa" and it will produce a file called UTM.ipa-fakesigned.ipa
This build has been tested by installing it on a jailbroken iPhone 7 with AppSync Unified via iFunBox and booting a VM

Your script didn't work for me. It repackaged the ipa but I still couldn't install it. However, I was able to install the binary you provided.
For anyone coming across this with a jailbroken device and appsync. You don't need tools like iFunBox because the command line tool appinst should have been installed with appsync (It's from the same creator and can be found on her repository).
Use it like this (with a mobile terminal or ssh):
appinst <path to ipa file>

There are no limitations when fakesigning, you can have any kind of entitlement (you can even disable sandbox etc) with it. The preferred format is .ipa, debs are used if distributed with Cydia.

Please check https://github.com/utmapp/UTM/releases/download/v0.2-fakesign/UTM.ipa to see that the fake sign has worked.

It works 👌

We also need someone to test #67 basically compare v0.2 and v0.2 fakesign and see if the fakesign version supports larger memory sizes.
In my personal experience, I have Windows 7 on iPad 2018 with 2GB of RAM. On v0.1 this is fine but on v0.2 it crashes on boot. I don't have a JB device to test, so I don't know if the fakesign version can bypass the iOS memory limits.

I’ll test in similar conditions using the fakesigned on my jailbroken phone @osy86

@osy86 I’m having problems: https://github.com/utmapp/UTM/issues/32#issuecomment-590649167


CAMEL-20181: camel-openapi-java - refactor tests to support both Open…

…API 3.0 and 3.1
Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test camel-openapi-java


x86 encoding information

This PR is based on #520's code with the following changes and additions:

cs_x86_encode renamed to cs_x86_encoding
its redundant disp_valid and imm_valid members have been removed
moved modrm, sib, disp to cs_x86_encoding
x86_imm_size has been extended to include the encoded immediate size, immediate sizes for bit test and shift/rotate operations have been corrected
disp has been promoted to 64-bits to accomodate moffset64 operands
moffset operands will show up as disp in cs_x86_encoding instead of imm

This should address PRs #520, #444, #331 and issues #525 (partially), #521, #509, #443 (partially)

merged, Great!

@obs1dium


conformance regression with conformance/testcases/animation_spline_5/input.jxl ?

Describe the bug
with git of today, compilation of examples/jxlinfo. Output:
$ ./jxlinfo.exe ~/gitroot_64/conformance/testcases/animation_spline_5/input.jxl
./lib/jxl/decode.cc:2100: invalid signature
Decoder error
Couldn't print basic info

Expected behavior
basic info of this file
Environment

OS: Win10
Compiler version: gcc
CPU type: x86_64
Highway scalar

It works for me, but maybe symlinks don't properly work in Windows? Does it work with animation_spline/input.jxl? (which should be the exact same file)

there is no symlink (like unix symlink) on Windows.
animation_spline/input.jxl works, and is animated with my viewer

indeed, animation_spline_5/input.jxl is 29 byes long. So git clone did something wrong

ok, it's a sym link in git. Why having the same jxl file ?

We have defined two levels of jxl conformance: level 5 and level 10 (more levels might be added later).
Level 10 is more strict than level 5 in terms of precision.
The conformance criteria are specified in terms of how much error is tolerated between the reference decoded images and your decoder's decoded images. Some jxl files are only used in level 10 (e.g. because they are encoded with a higher bitdepth than what you have to support for level 5), but most are used in both levels, just with a higher tolerance threshold in level 5 than in level 10. To avoid making the repo unnecessarily large, we did the duplicates with symlinks instead of copies.
It's somewhat annoying that Windows still does not have symlinks - it's a pretty useful feature :)

symlink introduced in Vista can only be used in Administrator mode, so they are actually useless.
so, ok i close this invalid  issue,


hw/mcu/dialog: Reset UART before reconfiguration

If TX FIFO is not empty (and maybe other conditions) during programming
new baudare UART can endup in BUSY state that will never end.
This adds reset of UART prior to configuring it in similar way as is
already done in uart_da1469x.c

Changes look good. Question: were these changes made in case hal_uart_config was called when the fifos had data in them or something like that (to prevent garbage being transmitted or received, for example)? If that is the case I am wondering about the ordering of some things in this function. For example, maybe NVIC interrupts should be disabled and cleared before setting the clock registers. If not, all good.

It come out when I prepared changes for 6MHz UART; Main purpose was to have high speed UART debug traces that could be used for analyzing BLE related-timing (or at least I thing it was the purpose).
I did it to trace stuff going on USB and timing of data was supposed to be as fast as possible so basically debugging code did not use hal UART code at all just wrote single bytes to UART data register.
It turned out that USB (when cable was connected) started to generate events before main function had time to open UART, then when main started configuration, UART already had something in the TX FIFO and during configuration BUSY interrupt fired.
So in real usage it will not be a problem I guess.


scoop install mkcert, download url is not valid in window

when i run
scoop install mkcert

in window shell .
and i dot know what should i do after this

anyone else knows how to solve this

It works fine for me. Maybe just a connection issue?
Please try installing it again.
If it still fails, see if you can visit https://github.com/FiloSottile/mkcert/releases/download/v1.4.0/mkcert-v1.4.0-windows-amd64.exe correctly.


Duplicated mapping key passes validation

Problem statement
Spec with duplicated mapping key passes validation and fails to work in Swagger UI with this error message:
Parser error on line 171
duplicated mapping key

Swagger specification
swagger: '2.0'
info:
  version: "1.0.0"
paths:
  /a:
    get:
      responses:
        "200":
          schema:
            $ref: '#/definitions/A'
definitions:
  A:
    properties:
      label:
        type: string
      label:
        type: string

Steps to reproduce
Environment
swagger version: 0.13.0
go version: 1.9.4
OS: Linux ef5acc98e0b3 4.9.87-linuxkit-aufs #1 SMP Wed Mar 14 15:12:16 UTC 2018 x86_64 Linux
Docker image: quay.io/goswagger/swagger:0.13.0 and latest

There is a specific check for this in validate, but something might have been overlooked.
Can you reproduce this on master?

@aleksandr-vin have you tested the spec validation with a json spec?
In my opinion, the yaml parser silently filters the duplicate key (https://github.com/go-yaml/yaml/issues/154).

Duplicate keys:

are allowed in JSON (result unspecified)
forbidden in YAML

not enforced by go-yaml v2, unless we use StrictMode, which is not an option as it requires all attributes to be known in advance
enforced by go-yaml v3

I believe the only practical way would be to migrate to go-yaml v3.
I think it preferable to wait for proper stable tagging of this new branch: https://github.com/go-yaml/yaml/issues/487.
go-yaml v2 inner workings:
https://github.com/go-yaml/yaml/blob/f221b8435cfb71e54062f6c6e99e9ade30b124d5/decode.go#L682
go-yaml v3 inner workings:
https://github.com/go-yaml/yaml/blob/fc94e3f7165204dd261b8d39d39a1e1773b931c9/decode.go#L747

+1