Optimized docker image sizes and run them as non-root user

Changes look good
@YeagerAI-Bulat Do you think adding users with less scope is worth the complexity? Could you explain the benefits of reducing the permission scope of the users? I understand that it's better in terms of having the minimum scope possible, but given that the containers are isolated I don't see much benefit
Adding users with less scope can indeed be complex, but it enhances security by minimizing potential attack surfaces. Even though containers are isolated, a compromised container could still be exploited to affect the host system. Reducing permission scope limits the damage an attacker can do within the container, adding an extra layer of defense. Check out this link for more details on Docker security and potential breakout scenarios: https://github.com/ivanversluis/pentest-hacktricks/blob/master/linux-unix/privilege-escalation/docker-breakout.md

@YeagerAI-Bulat @AgustinRamiroDiaz could you please resolve the conversations that has been addressed?

@YeagerAI-Bulat please update from staging and solve conflicts. We can merge it after that


Unable to get tabbed-admin working

Hi.
I'm unable to get tabbed-admin working.
No python errors, no javascript errors, jquery ui correctly loaded ... but simply I can't see any tab.
I've added tabbed_admin to INSTALLED_APPS, set TABBED_ADMIN_USE_JQUERY_UI to True, my model admin class is inheriting from TabbedModelAdmin and has a 'tabs' attribute.
Am I missing something? I'm using Django 1.9.7
Thanks

Have you tried to manually configure the tabs?
https://github.com/omji/django-tabbed-admin#configure-admin-tabs


Unable to publish/deploy the azure function solutions

We have a Azure Function which is built in .Net Framework 4.6 and recently we have updated the Azure Functions with .Net Framework 4.8 which works good in locally. So we tried to publish this to Live by going to select the existing Azure Function

But we are unable to publish due to below error (Publish has encountered an error).

Can you please suggest me If I missed any settings/configuration.

This repo is no longer actively monitored. Closing up old issues that have not had activity in while. If this is still an issue, please open a new issue in an appropriate repo listed in microsoft/dotnet#1275

This repo is no longer actively monitored. Closing up old issues that have not had activity in while. If this is still an issue, please open a new issue in an appropriate repo listed in microsoft/dotnet#1275


Error saving page as template in SitePages

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
The tenant has the new-ish "Save as template" capability available, so I should be able to save a page as a template.
Observed Behavior
When I try to do so, I get this error:
REST error Error with code -2147024809, System.ArgumentException due to Inner-Message: Value does not fall within the expected range. Exception-Message: [HTTP]:400 - [CorrelationId]:6cf8e39e-5056-8000-c5db-b14c253483f0 [Version]:<IP_ADDRESS>22 at Error: Inner-Message: Value does not fall within the expected range. Exception-Message: [HTTP]:400 - [CorrelationId]:6cf8e39e-5056-8000-c5db-b14c253483f0 [Version]:<IP_ADDRESS>22 at new t (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.007/sp-pages-assembly_en-us_f87e3760b9aa6c2684c21e30278a6b60.js:91:260926) at Function.e._getOdataRestApiError (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.007/18.sp-pages-statemanager_1b675d4c96764e72ad3a.js:1:11448) at https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.007/18.sp-pages-statemanager_1b675d4c96764e72ad3a.js:1:25623}
I've tried deleting the Templates folder (which was there from earlier attempts), but still no joy. It looks like it may be an issue moving the aspx page from the root of the SitePages library to the Templates folder after it is saved, as I do end up with new pages with names like 5cc7j2po.aspx.
The SitePages library has custom metadata columns of several types. Other SitePages libraries with the same metadata and structure work fine, so it may be some sort of damage.
This may be an isolated instance, but I wanted to log it in case it occurs elsewhere.
Steps to Reproduce
Create a new page, and save it as a draft (or publish). Open for editing again, and choose "Save as template". Add new name for the template and click the "Save page template" button. Error above.

As this is not a developer issue, we cannot unfortunately process this issue through this channel. All non-dev topics should be reported through the Tenant Admin support channels or through Premier support. Sorry for the inconvenience.


Feat: link to redirect officials careers portal

Describe the feature
Hey @PatilHarshh I want to redirect/link these cards to the official careers portal of that particular companies.so please assign me to this feature. ASAP :)
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

What's Your Thought ? means you want to add simply links to each image or Card ? or what @Mubashir2611

Actually I want to add links on these section that can redirect official career portal by help of the links. User can re directly reach the official career portal and they can searching for the jobs or applying the jobs.

@PatilHarshh Hey can you assign me or not?


Consider replacing the hamburger menu with the realm icon for the menu

This is a discussion that came up with @borisyankov while writing the iOS app store description.  It might make sense to replace the hamburger with something cooler, and a good choice might be the icon for the realm (might want to improve the defaults for those, but we want to do that anyway...).
There's precedent for that UI approach in Slack's Android app, and it'd be a nice way to simplify.

Hi, could you give more description about "the icon for the realm"? Are you talking about changing the vertical layout into something like a grid?  Maybe I misunderstand it, but I see no major difference between the current menu and the menu in Slack's Android app.

Hello @wilsonandusa to the extent I perceive, @timabbott is referring to replacing the hamburger menu icon with the icon of the organization to which the user is currently logged in.

We no longer have a hamburger menu, so closing this as no longer relevant.


AVRO-3925: [Rust] Fix decimal serialization format

What is the purpose of the change
(For example: This pull request improves file read performance by buffering data, fixing AVRO-XXXX.)
Verifying this change
(Please pick one of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Extended interop tests to verify consistent valid schema names between SDKs
Added test that validates that Java throws an AvroRuntimeException on invalid binary data
Manually verified the change by building the website and checking the new redirect

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

The pr in apache/avro has been merge: https://github.com/apache/avro/pull/2673, so I think it's ok to merge it. Other parts are for CI, which is not related to this issue.

Seems this commit is merge into avro. Can we just sync icelake-dev to avro/main. 🤔

Seems this commit is merge into avro/main. Can we revert this commit and pull avro/main directly to maintain the same history. 🤔

It's possible, ideally we should split this pr into two: one for ci, and one for this bug fix.


GUI - Text 'No configured devices' is not vertically centralized

In the device list screen, the text 'No configured devices' appears at the top of the page, in the other screens (templates, flow) the text is vertically centralized.

Text 'No devices to be shown' is not vertically centralized.

Text "No configured devices" is not vertically centralized.
Affected version: 0.3.0-nightly20180604


cwltool is not quite working with direct launch on workflows

Bug Report
Expected behaviour
Launch instructions with cwltool should just work
https://dockstore.org/workflows/github.com/dockstore-testing/md5sum-checker:develop?tab=launch
Actual behaviour
With workflows using multiple files (as opposed to tools or workflows in one file)
$ cwl-runner https://dockstore.org:443/api/api/ga4gh/v2/tools/%23workflow%2Fgithub.com%2Fdockstore-testing%2Fmd5sum-checker/versions/develop/plain-CWL/descriptor/%2Fmd5sum%2Fmd5sum-workflow.cwl Dockstore.json

works
$ cwltool github.com/dockstore-testing/md5sum-checker:develop Dockstore.json
$ cwltool --make-template github.com/dockstore-testing/md5sum-checker:develop > input.yaml

These two do not work. looks like they don't resolve the #workflow and md5sum/md5sum-workflow.cwl fragments. We can do part of this by generating:
cwltool \#workflow/github.com/dockstore-testing/md5sum-checker:develop Dockstore.json

But the resolver still needs the initial file path
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-627

Will need to add cwltool version for direct launch after PR to cwltool
https://github.com/common-workflow-language/cwltool/pull/1093

I went to https://staging.dockstore.org/workflows/github.com/dockstore-testing/md5sum-checker:develop?tab=launch and successfully ran the first 3 launch examples (dockstore convert, dockstore launch, and cwl-runner) on Ubuntu. Skipped the wget one.
The cwl-runner one fails on a Mac, but I think that's just a Mac/Docker issue.
The checker commands seem out of order -- the second one describes how to complete a template JSON, the first one describes how to run it. Since you need JSON to run it, seems like it should be the other way around.
I was able to get both checker commands to run.
@denis-yuen: Is this sufficient verification? Not entirely clear from the description.
Also is the order of the checker commands wrong, or am I misunderstanding? I will log a seprarate ticket if so.
Not sure if I'm misunderstanding stuff, so putting comment here for further discussion.

@coverbeck this ticket is for running directly with cwltool without using the dockstore CLI
so cwl-runner https://staging.dockstore.org/api/api/ga4gh/v2/tools/%23workflow%2Fgithub.com%2Fdockstore-testing%2Fmd5sum-checker/versions/develop/plain-CWL/descriptor/%2Fmd5sum%2Fmd5sum-workflow.cwl Dockstore.json would count (but seemed to be broken for you on a Mac)
https://github.com/common-workflow-language/cwltool#use-with-ga4gh-tool-registry-api has more info (same syntax).
Depending on when it broke on your Mac, it should be ok if cwltool launched, downloaded descriptors, and started Docker properly. Alternatively, try on a VM?

cwl-runner fails on a Mac because The paths /private/tmp/docker_tmpb1v2fkvt and /private/tmp/docker_tmplvfnp9i1 are not shared from OS X and are not known to Docker, which is a Mac limitation. The command does work on Ubuntu.
FYI, when running with Dockstore CLI on a Mac it works because we explicitly set the --tmpdir-prefix when invoking cwl-runnner. Or that's my guess.
Will log the order of launch commands as a separate ticket.
Verified with 1.7.0-rc.0 on staging.


Remove Info messages from error list

I'd like to be able to tell the error list to forget the info messages - they are just annoying. Can't find a way to specify that.

In VS Code, there is a way to filter the error types.
You can disable information messages with the filter:

Or, if you never want to see a specific error code again you can add add that code to the settings:


Version-aware analyzer

One thing I would love to see is the analyzer being aware of the project as a whole and the language constraints. In particular what I'm thinking is the analyzer being able to detect the SDK constraint (eg: sdk: ">=1.9.0") from the pubspec. Then provide warnings (probably informational?) based on that, such as if I had the 1.9 sdk constraint but tried to use null-aware operators (??=, etc). It would be nice if it triggered a warning saying the language usage doesn't match the sdk version.

This is now implemented; the analyzer will recognize new syntax and will report if the version constraints do not match. :)


Improve API for "snaking" grid_scan

The API for optionally "snaking" certain axes of a grid_scan as explained here is a bit confusing.
Current API:
grid_scan(detectors,
    motor1, start1, stop1, num_steps1,
    motor2, start2, stop2, num_steps2, snaked_or_not2,
    motor3, start3, stop3, num_steps3, snaked_or_not3,
    ...)

The fact that the first axis does not take a boolean but all subsequent axes must have a boolean trips people up, and it forces users who don't care about the snaking feature to pause and understand it.
A new API has been proposed:
grid_scan(detectors,
    motor1, start1, stop1, num_steps1,
    motor2, start2, stop2, num_steps2,
    motor3, start3, stop3, num_steps3,
    ...,
    snaked=SNAKED)
# where snaked is an optional, keyword-only argument, and SNAKED can be:
# False (default) -- Do not snake any axes.
# True -- Snake all axes
# tuple of booleans -- Snake some specific axes

I think this new usage is better in every way; I wish we had thought of it in the first place! Fortunately, it is possible to change grid_scan to support both the old and new usages by duck-typing the args and figuring out where the motors are. I propose to do that, advertise the new usage, and demote the old usage to footnote in the docstring.
Alternatively, we could write a separate function, but then we lose the name grid_scan and need something like new_grid_scan going forward. This is why I favor updating grid_scan in a backward-compatible way.

This makes a lot of sense to change the documentation and how the function arguments work.  I would recommend that the doc string contain an example on how to use the tuple of booleans.  I naively assume if I want to snake motor1 and motor3 for a 4 motor scan, then snaked=(True,False,True, False) respective to list of motor argument . If this is correct, then I should get python points. I am  just trying to say that I am not that all of our users know what a tuple is.

Python points for you! :-D
We will definitely document this with examples.

Any concerns from @prjemian, @teddyrendahl, @ZLLentz?

One finer point to work out:
If there are N motors, the first motor can't be snaked. Do we accept N-1 settings in the tuple, as in:
grid_scan(detectors,
    motor1, start1, stop1, num_steps1,
    motor2, start2, stop2, num_steps2,
    motor3, start3, stop3, num_steps3,
    snaked=(True, False))  # refers motor2, motor3 respectively

or
grid_scan(detectors,
    motor1, start1, stop1, num_steps1,
    motor2, start2, stop2, num_steps2,
    motor3, start3, stop3, num_steps3,
    snaked=(False, True, False))  # refers motor1, motor2, motor3 respectively

and raise if the first item in the tuple is anything but False. I submit that the latter will likely to be less confusing to use. The "off by one-ness" of this throws people off.

I guess we could support both and figure out which one is the right one by inspection of the length.

I think I would not like that because it makes wrapping your brain around the complete "rules" for how to use this and how it behaves even more complex.

I do see this as an API improvement. I am somewhat against having the N sized tuple where the first argument is alwaysFalse.
Spitballing here, any credence to passing in the motors you want snaked instead?
grid_scan(detectors,
    motor1, start1, stop1, num_steps1,
    motor2, start2, stop2, num_steps2,
    motor3, start3, stop3, num_steps3,
    snaked=(motor2, motor3)

+1 for @teddyrendahl suggestion. This is highly intuitive. Maybe “all” for all axes instead of true and None for None?

I would rather a bool be used for all or nothing decisions (with None being a proxy for False). String sentinels can get messy. I think a signature with snake=None|True|(motor2, motor3) could be interpreted as "do I want to snake?" True I would like to snake all the things.

IOM, I prefer the original API, not the proposed one.  Separating the snake kwargs into a tuple disconnects them from use.  The parameter list is of monumental length already.  Seems like this is a change that adds an additional way to describe the same thing.

IOM == IMO

What if a motor object knows about whether to "snake" or not? Just a thought...

How would it know if it needed to snake?

That sounds like a situational decision.  If there is a change, Teddy's proposal makes sense to me.  And it resolves one of the original problems where new users often have problems learning how to use this plan.

How would it know if it needed to snake?

False by default, configurable with setting an atrribute before running or within a plan.

I would rather a bool be used for all or nothing decisions (with None being a proxy for False). String sentinels can get messy. I think a signature with snake=None|True|(motor2, motor3) could be interpreted as "do I want to snake?" True I would like to snake all the things.

I think for the average user it is easier to think “which motors do I want to snake” None, all or a list of specific ones.
If the question is rephrased as “do I want to  the answer should be “yes”, “no” or “motors in this list”
And this is less intuitive, as is the use of True and False.

What if a motor object knows about whether to "snake" or not? Just a thought...

This is moving the logic to a place that is not so obvious for an average user. Keeping it in the plan API I think is a better option.

Separating the snake kwargs into a tuple disconnects them from use.

I had this thought too, @prjemian. Indeed, there is some advantage to keeping each snake parameter right next to the group that it belongs to. But overall maybe the pros of this new proposal outweigh the cons, particularly for new users.
It looks like we are all at least content with @teddyrendahl's refinement on this proposal, judging from the :+1:s. We can continue to support the original way indefinitely for situations/users that find more readability there.
Any objections to proceeding in that way?

Which of the True/False, “all”/None, “Yes”/“No” options are we suggesting using?

I vote for boolean for all or nothing behavior. Generally don't like having to remember special string names especially for binary options.

Yeah, I agree that anything but normal boolean would be very unusual. I can't think of an example of a scipy library that uses "Yes"/"No" or "all"/None when True/False would work.

What is going to be the default behavior since this was a mandatory argument before?

My original suggestion was False by default because that seems most likely to at least work (if not necessary the fastest). Some motors can't scan in reverse, I'm told.

Some motors do not scan well in both directions, usually due to gear backlash.  The EPICS motor record has code to account for this but with real hardware, can only do so much.  The correction involves target overshoot and then approach from the other direction.  Data are more reproducible with such axes when scanning in the direction where backlash correction is not needed.

I'm not clear with https://github.com/bluesky/bluesky/issues/1189#issuecomment-492867947.  Is that snaked can take any of these values?

None (default) - no axes are snaked
True - all axes but the first are snaked
list of axes names that are snaked

Yes. I had in mind False as the default value, but anything "false-y" would have the same outcome, so snaked=None would have the same result as snaked=False.

And thanks for the explanation @prjemian. This is why I think snaking should be an opt-in feature.

That's better.

Changed my table so it is clear.

Ok so given that I am in the minority by 4:1 I am ok with moving forward with this suggestion even though I disagree on the no axes and all axes values.

This approach has been applied to list_grid_scan by @awalter-bnl in #1190. It remains to apply it to grid_scan in a backward-compatible way. Per discussion above, it looks like we agreed that this change is worth making because (quoting @prjemian) "it resolves one of the original problems where new users often have problems learning how to use this plan." We can support the original usage as well, indefinitely.
I will assign the work of changing grid_scan to myself. Once that's done, this issue can be closed.

Some thought will need to be given to how to sniff out the types of the parameters. The start and stop parameters can in principle be anything (not necessary just a number) so I think the best bet is find where the motors are by using something like
In [21]: class Movable(metaclass=abc.ABCMeta):
    ...:     "Abstract base class for objects that satisfy the bluesky 'movable' interface."
    ...:     @classmethod
    ...:     def __subclasshook__(cls, C):
    ...:         return bool(hasattr(C, 'name')
    ...:                     and hasattr(C, 'set')
    ...:                     and ...)
    ...:
    ...:

In [22]: from ophyd.sim import motor

In [23]: isinstance(motor, Movable)
Out[23]: True

In [24]: isinstance(3, Movable)
Out[24]: False

where Movable is intended to be used not as a base class but as a duck-type checker like collections.abc.Iterable.

... where ... above is a placeholder for more methods that should be checked for. See https://blueskyproject.io/bluesky/hardware.html#readable-device.

I believe this has been resolved by #1334, or am I missing some additional changes desired?

Thanks @ksunden. You are correct, I believe. Closing.


Timeout for queries (error message & setting) #610

I feel the built-in YASGUI interface also prompts a Unable to get response from endpoint. Possible reasons: message if the endpoint just timed out.

Suggestion: add this as a reason to the list (maybe for YASGUI to fix)
Question: is there a native way to define the timeout through oxigraph settings directly or this this only for the nginx route to the API endpoint to be defined?

see also https://github.com/sparna-git/Sparnatural/issues/610

For YasGUI I don't know.
For Sparnatural the queries sent to populate dropdown lists or autocomplete fields are using the Javascript fetch method under the hood. My understanding is that this method will use the default timeout time of the browser (?)

For YasGUI I don't know.
For Sparnatural the queries sent to populate dropdown lists or autocomplete fields are using the Javascript fetch method under the hood. My understanding is that this method will use the default timeout time of the browser (?)

Yes, definitely not a sparnatural issue, sorry for cross posting

Hey! Thank you for using Oxigraph. The Oxigraph HTTP server has no time out system, if queries are timing out in YASGUI it's because the browser request timeout or a timeout in a proxy.


refactor(optionsmenu): refactor demos, per page text

fixes https://github.com/patternfly/patternfly-next/issues/1666
@jgiardino should we review the options-menu--label-text and toggle aria-label for this component, or wait for the react implementation? I believe someone is working on the react options menu in this upcoming sprint.

Deploy preview for pf-next ready!
Built with commit 5770f41703d3bd123ec5e33dc7a8d4acff18f64b
https://deploy-preview-1671--pf-next.netlify.com

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


App is blocked because it tried to gain sensitive information

Been using this extension for years, it now doesn't work.

Google suggests the security feature improvements of Google has broken it. Any chance we could get a fix??

Yes. Same thing.

Sorry, missed the initial comment here. I'll take a look and see what I can do!

Hi!

Yes there's three of us, we use it to development the json exports for our video game. One person is able to use it no problem so this tracks with what you are suggesting.
Personal Gmail accounts
Chrome in all three cases
USA for all three

We'll try having user 1 sign out and attempt again. It's a bit of a pain as sheets is tied to logging into everything with your Google account but it's probably more manageable than not having access to this tool as it's very unique.

Hello,
I am now owner of this sheet. I cannot install the extension. I'm assuming if you have it installed already there is no issues. Same error as before.
Maybe something like this would help?
https://developers.google.com/apps-script/guides/services/authorization#manual_authorization_scopes_forand

Hmm, do you already have ESD installed for your account? Or are you attempting to install it now?
If you already have it installed, are you able to make a new sheet and export JSON/XML from that?

Actually, I just tried doing so with my work account and I saw the blocked warning too...
I may need to update the Google Cloud Project for ESD to ensure its scopes are still correct. It could be that Google doesn't like granting access to Drive for file management purposes anymore, or at least has stricter scopes for it.

I think I have to do some OAuth verification steps on the project to fix this issue. That will require some web work on my end. This will take me a couple days due to my upcoming schedule unfortunately.
I will try to have the issue resolved ASAP. Sorry for the inconvenience!

Alright, I have ESD resubmitted for verification. Had to do a surprising amount of web dev and video production to verify an open source add-on, but hopefully it will be back up soon.

Quick update. I've been back and forth with Google's verification team this past week. I believe I've ironed out the last of their concerns with the latest round of revisions so hopefully the next time I hear from them it will be them approving everything.

Awesome keep us posted! Appreciate helping all of us out we love this addon 🥰

@Synthoid thanks for following up on this.  We are still facing this issue ; do you mean that your fix in https://github.com/Synthoid/ExportSheetData/issues/163#issuecomment-2227441128 is yet to be reviewed by Google, and once it is, we wont have this issue ?
Other than signing out of all other accounts, what is the known workaround until then?

To clarify, I have made requested revisions to various ESD resources and I am currently waiting to hear back from Google.
The turnaround on that is usually 24-72 hours so I should hopefully hear back within the next day.
Until I resolve the verification issues, there doesn't really seem to be a great workaround unfortunately. From my understanding, only around 100 people could actively choose to bypass the warning and that was probably a first come, first served situation.
If you can wait, I believe this will be resolved this week, ideally after this latest round of revisions. Barring some needling from Google's verification team...
I'm sorry I've let ESD languish and get into this state. I should have cleaned house months ago to prevent a lapse in service.

To clarify, I have made requested revisions to various ESD resources and I am currently waiting to hear back from Google.
The turnaround on that is usually 24-72 hours so I should hopefully hear back within the next day.
Until I resolve the verification issues, there doesn't really seem to be a great workaround unfortunately. From my understanding, only around 100 people could actively choose to bypass the warning and that was probably a first come, first served situation.
If you can wait, I believe this will be resolved this week, ideally after this latest round of revisions. Barring some needling from Google's verification team...
I'm sorry I've let ESD languish and get into this state. I should have cleaned house months ago to prevent a lapse in service.

Thanks for working through this to get it back up! Love your tool and am excited to pick it back up once Google approves it

So... some good news, and some bad news. The good news is that ESD has cleared Google's verification process!
The bad news is there is apparently another step. ESD has to clear a CASA assessment by a trusted third party. My initial look into Google's provided resources is not looking great. To even get a CASA assessment it's looking like I'd need to shell out $500+. That would become an annual expense as the evaluation has to happen every year... Even worse, the best evaluation process that gives maximum confidence for the application's security is a whopping $3000+ annually.
I'm going to look into my options here to see if I can get an exemption since it's just an Apps Script add-on that isn't storing user data and there were such exceptions in the past. This certainly wasn't part of the verification before but if it is indeed required now... I'm not sure that is an expense I can handle. The $3000+ evaluation absolutely isn't.
For the time being, that does mean ESD will be down for a little while longer. I'm sorry for the delay.

After looking into it some more, I may be able to avoid the whole CASA process as that is only required because ESD requests root Drive authorization for some processes.
Specifically the "Replace existing file(s)" option is the main thing using that scope to streamline workflows. I think I'll have to rework that so users have to select a file to replace instead of automatically clearing files in the export folder. It's a worse workflow, but that should mean ESD  will not cost me $3000+ a year...

Quick update. I've been banging my head against my keyboard this weekend trying to get things working. Unfortunately the scope issue goes deeper than I thought and all of the existing DriveApp code ESD uses to create, not just update/replace, JSON/XML files require the restricted auth/drive scope. This is not viable as it means I'll have to go through the CASA evaluation every year.
The most promising approach seems to be migrating over to the advanced Drive service so I can use the reduced auth/drive.file scope. I've made some minor progress on that, but it's looking like I'll be enduring some trial and error to get this fully working.

@Synthoid Thank you for looking into this issue. The issue is also happening on my end.
Looking forward to being able to use your amazing plugin again!

Alright, time for another quick good news/bad news update.
The good news, I have file exports working again! The migration over to the advanced Drive service to create files was painful, but with enough persistence I've bashed that brick wall down.
The bad news, ESD in general is going to be less user friendly going forward and I'm probably going to have to rip out file replacement and export folder customization. The more restricted auth/drive.file scope pretty much only lets me interact with files that you personally have created with ESD, which creates a minefield of potential errors when calling Google's services. I have to counteract this with lots of try-catch statements (made even better because now the export method has to be async).  I believe there is some potential way to get around this using the google file picker but, like the Drive API, documentation is somewhat lacking so I have to go forum spelunking. Posts like this make me a little worried, but I'm optimistic I can get things working.
My plan for now is to remove the Replace existing file(s) and Export Folder settings to get an updated build out the door so people can actually use ESD again. This does mean that exported files will be dumped into your root My Drive folder for the time being, but it's better than nothing. Fortunately I was able to keep the post-export modal popup functionality, so you will still see links to the exported file and a download button to get it. Just keep an eye on your root folder so you don't drown in .json and .xml files...
I'm going to take a couple more days to try and get the picker working, but I'm hoping ESD will be back up within a week after making all these changes.

Hello all! This will hopefully be the last dev update on this ticket before release. Thank you for being so patient!
I believe I have everything working now, including custom export locations and replacing existing files. To compensate for the new workflows, I've updated a lot of the sidebar's code to be more responsive and cleaner for long term maintenance. There will be some structural differences in settings, mostly due to how OAuth workflows are required to work, but I have taken steps to persist as many existing settings as possible.
Anyway, I'm going to take a few more days to finish cleaning up the new code and bug test to make sure everything is working properly. I'm aiming to submit ESD for review by the end of the week, so if everything goes smoothly it should be working for everyone sometime next week.
Again, I'm sorry that it took so long to get things up and running and I appreciate everyone's patience!

Thanks Chris!

Thank you!

Thanks Synthoid, hope it works soon!

One more quick update, since I'm a little behind on my promised delivery.
I've tested everything, and ESD seems to be bug free and 99% ready to submit for review. I'm currently updating screenshots for the add-ons browser listing since those are horribly out of date. Once that is done, I have a small adjustment to make on how custom export location/replace file target settings are stored, then ESD should be ready for review! If I don't get that submitted by tonight, it should be ready by EOD tomorrow.
Sorry for the short delay. I'll post any updates if Google approves or rejects the new submission, but I'm optimistic that you will all have access this week.

Thank you for your patience while we reviewed your submission. We've approved your OAuth App Verification request...

With that, I believe v66 is live! I was able to export a sheet, formatting and all. Being paranoid, I'll keep this ticket open for a little longer until I have some external confirmation that it's working...
Let me know if that isn't working for anyone!

Looking good here, thanks again Chris!

Groovy! Thanks for all the patience and support everyone!


User info endpoint

Closes #84
This PR actually includes changes in #101 since I head started with this.
Also in this PR is a change to not keep oAuthClient and oidcClient separate. The previous strategy of extending oAuthClient caused code complexities because the oidcClient needed to cloak behaviour in oAuthClient in some cases.
So now createOidcClient is now just an alias of createOauthClient but with some alternate validations and defaults.

@dennerkrans Yes let’s do that!


cmake: prevent C core from depending on libstdc++

When building with make, C core libraries intentionally don't depend on libstdc++ (for better portability)
This PR makes that cmake-built core libraries (and C# extension) do not depend on libstdc++ on Linux and Mac.
On windows and Android this is not necessary because the backward-compatibility is ensured by other means.
Changes:

where applicable, use C linker instead of C++ linker for C-core libraries and executables.
set -fno-exceptions and  -fno-rtti for .cc files that are part of C-core as they would add a requirement to link against libstdc++ (Make does the same thing).

Note: This PR is a prerequisite for building grpc_csharp_ext with assembly-optimized boringssl.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,792      Total (=)      1,964,792

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,887,060      Total (>)     10,887,058

 No significant differences in binary sizes


cleans up commented out code. modifies task to set all compliance fie…

…lds as none using iteratee first each function

Coverage remained the same at 23.002% when pulling 4f3d50b5fc2c74b9dc1d99dea2b5a2dc2f0e8db3 on theseyi:set-unspecified-fields into b0037ccbc22918de9fc9000bc4e1799345a36ff4 on linkedin:master.


PPC Campaign Landing Page Template Build

I need two templates built to create landing pages for our PPC campaigns (Long form and short form pages). I've built out the schema and content already in the manager, just need the designed template built out. Here is the link to the doc that outlines which content blocks I need from our template and notes about certain elements to take out (for example, some CTA buttons will need to be removed). Please assemble these and connect to the schema.
https://docs.google.com/document/d/1jLt6mOVrSrZk8aE7jNR3LPtW4UnyyRFGYt39QwQ6Ttg/edit?usp=sharing

header - is it possible to make the header Html

I believe it is in the schema - just a text field called "Who is Zesty H2"

Here are the edits for the long-form page. Let me know if you have any questions:

Hi @Katiemoser  can you draw , im thinking of the form will have white rectangle below and the form has border?

I want the whole form to be white like the screenshot above, just the background behind it to be blue.

Sorry, don't need the copy, just the form with the title and description that was there before.

I just like the styling of the white on blue.

Is this for the short form page? (/create-engaging-digital-experience-at-scale) If so, keep as is. I want to address the form on the long-form page (/digital-experience-platform)

Sorry, I should've separate these. For the long page, I want the form to be white with a background behind it of dark blue, similar to how it's style on the short form page.
SHORT FORM

 i revert it back the image , the form mostly covered the background image , we should remove the additional input fields?

The short form page is approved above. For logos, can we add a row below the bullet points? Also how should I update the background image? Should I add that into the schema?

Approved

shortpage image -yes just put a field for image

Let's just do 3 logos.

i remove the last 2 logo


Can we use this on Node v6.x or v7.x?

doesn't seem like it, but just thought I would ask! thanks

We recently did some modernisation work to bring support for Node 4.X.  I think there might be one or two more minor changes needed to provide support for 6.X.
Did you try to build the module and it failed?

I haven't tried it, the docs just explicitly state that it works for much older versions so wanted to see if it was a fool's errand to try to compile on newer versions, I will try it


Generate Kubebuilder project for ExecutionHook CRDs and controller

This PR generates a new Kubebuilder v2 project for the ExecutionHook CRDs and controller.

adds the ExecutionHook API
scaffolds out a controller for the ExecutionHook

The HookAction API needs to be added and the controller logic needs to be implemented as per the KEP (https://github.com/kubernetes/enhancements/blob/master/keps/sig-storage/20190120-execution-hook-design.md#proposal).
Signed-off-by: Adnan Abdulhussein<EMAIL_ADDRESS>cc @xing-yang @skriss @nrb @carlisia @ashish-amarnath

Awesome!  Now that @prydonius has submitted this PR, I'll close my POC PR https://github.com/kubernetes-sigs/execution-hook/pull/2.

@prydonius and I spoke offline and I'll be taking this PR forward.
Thanks for the clean hand-off, @prydonius!

I am also ok w/ getting this merged and working on top of that. It'll ensure author/contributor credits.

Closing this in favor of https://github.com/kubernetes-sigs/execution-hook/pull/4

/close


allow ocamlformat to work outside project

Allow ocamlformat to work outside project (by enabling --enable-outside-detected-project):

.ocamlformat found: read format options from .ocamlformat.
.ocamlformat not found: use --profile=ocamlformat as default option.

Looks good@


Find My Mouse Triggers Focus Mode

Microsoft PowerToys version
0.49.1
Running as admin

[X] Yes

Area(s) with issue?
Mouse Utilities
Steps to reproduce
Prequisite: Windows Focus Assist must be set to "activate focus assist when I'm using an app in full-screen mode".
Synopsis: With this Focus Assist rule turned on, Focus Assist will be active when the Find My Mouse overlay is active.
Repro:

Activate Find My Mouse (press Left Ctrl key twice)
Observe the 🌙 icon appear near the clock in the taskbar.

✔ Expected Behavior
Finding mouse does not activate focus mode.
❌ Actual Behavior
Finding mouse activates focus mode. Since using FMM is a very short-lived activity, it's a very low impact to the user.
Other Software
No response

Duplicate of #14122

Thanks for reporting this. This issue is a duplicate.
I'm linking this issue to #14122 to centralize the discussion thread for tracking.
Thanks for your help to make PowerToys a better piece of software!


Avoid creating a Docker layer containing the compressed model file.

Download and delete the compressed model file in the same layer to avoid an (often oversized) additional Docker layer.

@xuhdev Can you please rebase with master? I'll merge it once you're done. Thanks!

Done!


Hiding the .statusbar-overlay on landscape orientation

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: 1.5.3

Platform and Target: iOS

What you did
I added this code to css file.
@media screen and (orientation:landscape) { 	html.with-statusbar-overlay .framework7-root { padding-top: 0; } 	html.with-statusbar-overlay .statusbar-overlay { height: 0 ; } 	html.with-statusbar-overlay .framework7-root>.photo-browser, html.with-statusbar-overlay body>.photo-browser { top: 0; height:100%; } }
Expected Behavior / Actual Behavior
If you add the website to home screen and open the it as a web app,
iOS(iPhone) will hide status bar on landscape orientation. The .statusbar-overlay div will still be shown! The above code should hide it.

So, you already solved your problem?

@valnub Yes, I just share what I found. So, someone can add it to master.

Can you please post a pull-request instead? https://help.github.com/articles/about-pull-requests/
thanks


Add a spring-cloud-stream-binder for Amazon SQS

From @mstine on September 19, 2015 0:58
Seems like this should be relatively easy by leveraging the SQS support in Spring Cloud AWS. We'd like to use this to support https://github.com/spring-cloud/spring-cloud-netflix/issues/545, so that we can use SQS when running Spring Cloud Services on PWS.
Copied from original issue: spring-cloud/spring-cloud-stream#135

This would be extremely useful, especially if it could also support sending to a SNS topic and consuming from a SQS queue.

@tvrmsmith ,
Indeed, we can configure SQS queue to subscriber to the SNS topic.
But how about if we will just have a Kinesis Binder and get a gain of the whole bunch of possibilities?
https://aws.amazon.com/kinesis/streams/faqs/

I would for AWS SQS to be supported. SQS support would be a big help for our project.

I wonder why not Kinesis: https://github.com/spring-cloud/spring-cloud-stream-binder-aws-kinesis ?
The SQS doesn't fit fully the SCSt communication model. There is no publish-subscribe semantics, no partition and offset management etc.

I would say a combination of SNS and SQS can achieve a pub-sub semantics. Also, most of my use cases don't have a need for partitioning or offset management but find SCS without those to still be useful. This could be just because I don't understand the best scenarios for those features though.

I see your point, @tvrmsmith .
Thank you for sharing!
Unfortunately we don't have resources to jump to this area.
Contribution is welcome of course!

I've done some work on this but haven't found the time to get it into a working state.
Also, I just want to thank you guys for making Spring so awesome.

Ha-ha! No, that is big thanks to you, Community. Without your comprehensive feedback and valuable contributions the Spring wouldn't be so awesome! 😉

I've done some work putting together a project for this, but am running into an issue where the SQS API only accepts Strings for their message payloads. This would mean the default MessageConverters would have to be extended to send String representations (or keep the original String representations instead of converting to byte[]).  I think this would work, but that would mean the project wouldn't conform to https://docs.spring.io/spring-cloud-stream/docs/current/reference/htmlsingle/#mime-types-and-java-types which seems quite fundamental and thus maybe SQS just doesn't fit? @artembilan if you disagree I can continue down that route and finish up an initial version of this binder.

If you want to know my opinion, then I don't see reason in this binder since we have a Kinesis one.
But if you insists, I can come back to think about this after Spring One.
Anyway even byte[] can be represented as a String via Base64 encoding:
java.util.Base64
...
 /**
         * Encodes the specified byte array into a String using the {@link Base64}
         * encoding scheme.
         *
         * <p> This method first encodes all input bytes into a base64 encoded
         * byte array and then constructs a new String by using the encoded byte
         * array and the {@link java.nio.charset.StandardCharsets#ISO_8859_1
         * ISO-8859-1} charset.
         *
         * <p> In other words, an invocation of this method has exactly the same
         * effect as invoking
         * {@code new String(encode(src), StandardCharsets.ISO_8859_1)}.
         *
         * @param   src
         *          the byte array to encode
         * @return  A String containing the resulting Base64 encoded characters
         */
        @SuppressWarnings("deprecation")
        public String encodeToString(byte[] src) {

Right, but in the case of plain/text on the inbound channel, how do we know if the payload is the original string or a base 64 encoded string that we need to decode? It could be either correct?

Shouldn't the content type be indicating to the inbound channel how to decode/parse the message?

@artembilan As far as I understand, Kinesis doesn't support consumed-once semantics.

@chrylis ,
Well, you know we ensure that with the checkpoint for each consuming shard and store an offset for the records in the DymanoDB table.
So, I believe this should somehow meet consumed-once semantics requirements.
Anyway I don't mind against the AWS SQS+SNS Binder. Just we don't have resources right now to jump into this feature.
And my point is to use an existing solution as a fallback variant for time being.
Thank you for feedback!

Hi all,
any updates?

I've started working on SQS binder: https://github.com/maciejwalkowiak/spring-cloud-stream-binder-sqs.
Integration and most of the customisation options are missing, but the basics seem to work in simple scenarios. If someone would like to join the efforts - PRs are welcome! 😃

Looks great at a glance!
A couple remarks:

consider to rename all the kinesis variables to sqs 😉 . That's minor
I think you miss a big picture about consumer groups: http://cloud.spring.io/spring-cloud-static/spring-cloud-stream/2.1.1.RELEASE/multi/multi__main_concepts.html#consumer-groups. For that purpose I suggest to have an SnsMessageHandler on the producer side to publish to the SNS topic instead for possible distribution between subscribed SQS queues. The SQS queue is really might be based on the provided group and subscribed to the SNS via Topics.subscribeQueue(). Just an idea how to deal with consumer groups in Spring Cloud Stream.

Thanks so far for the effort!

Thank you @artembilan for quick feedback! Sure, there's a lot of things to be done there. Thanks for hints how to deal with customer groups.

After making it work well with SQS only I started implementing customer group support with SNS and I am having one issue perhaps @artembilan you can guide me.
I need to register two extra converters in the ConversionService (integrationConversionService) used by SnsMessageHandler. I thought putting @IntegrationConverter on the bean would do the job but seems it's not getting registered.
The code in on the branch here: https://github.com/maciejwalkowiak/spring-cloud-stream-binder-sqs/pull/8/

@maciejwalkowiak ,
first of all it is called consumer group. There is nothing to do with customers 😄
Your @IntegrationConverter is not visible as a bean for the IntegrationConverterInitializer because the @Configuration it is present is instantiated later via DefaultBinderFactory.
Anyway this is not what I would recommend to do. Such a converter in the global ConversionService is going to have an effect anywhere ConversionService is used with the byte[] and String.
To fix it properly I would suggest to have a postProcessOutputChannel() implementation with an extra ChannelInterceptor to convert incoming byte[] to String before the message reaches an SnsMessageHandler.

first of all it is called consumer group. There is nothing to do with customers 😄

Too much business podcasts 🤦♂
@artembilan, thank you for advice, using postProcessOutputChannel works like a charm!

@mKeRix I saw you had archived the repo. Do you aware if there are any plans to impl such binder officially SCS?

We were also planning on using Spring Cloud Stream with SNS (with a future switch to MSK)
Since the repository was archived (not sure on the reason though, we assume that there won't be any further support) we can't use Spring Cloud Stream anymore.
Would be great if these binders would be officially supported by Spring.

Missed the comment from a couple months ago, sorry about that. Unfortunately, I left that company about 2 years ago, so I wasn’t involved with the decision to archive. I believe that the team maintaining the repo chose to re-architect their workloads away from Spring Cloud Stream and therefore didn’t feel like they had the capacity to maintain these binders anymore.
I could create a fork on my private account, but I don‘t use Spring regularly anymore (in fact, haven’t touched Java since I switched jobs). So I don’t think that I would be a good steward for the project, since I couldn’t do any dogfooding and am also out of touch with what updates from all the related projects would change.
If there is anyone that feels like they could take up maintainership of these projects (maybe because you would like to use them?) let me know and I can try to broker whatever is needed to move the library ownership.


Include ChainerCV doc

Would you check the CI failure?

Makefiles are not used when publishing to RTD. Another hack may be needed.
https://readthedocs.org/projects/chainer/builds/7918379/

jenkins, test this please.

Jenkins CI test (for commit d6844d73972efe65e2b3cc8247cbbcc0e5930467, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit f1e9624fe2dc3810956334f067c9eaa019f7a6f7, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Can someone review https://github.com/chainer/chainer-test/pull/457?

jenkins, test this please.

Jenkins, test this please.

Jenkins CI test (for commit 650d0eb622faaed2ea57250a0c039d2862400354, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit 650d0eb622faaed2ea57250a0c039d2862400354, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Currently no doctest runs on chainercv repo?

No..
With the current implementation, Chainer would need to be able to import the latest released version of ChainerCV. This assumption can fail when functions imported by ChainerCV get deleted in Chainer's master branch.
I talked with beam2d-san offilne, and we decided not to integrate docs. Instead I will add links to ChainerCV's docs hosted here http://chainercv.readthedocs.io/.

Jenkins CI test (for commit 89c5d3a4ee9882df9951eb529b494d55baf7ac52, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit 83d673bfc686482d0b3b24d2e1b2224c29c64927, target branch master) succeeded!

LGTM!

@takagi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@takagi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@takagi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


SASS

Problem:
We need to provide SASS in this project.
Solution:
Download, Install and configure SASS.
http://sass-lang.com/

This issue was fix on a6f8ac7a75410d073849ce859246a6e49289d122


fix: bytes32 owner -> bytes20 outputGuard in eip712 TX_TYPE_HASH

@pgebal I found a bytes32 owner in the precomputed eip712 TX_TYPE_HASH

Good catch Kevin. Feel free to merge that after solving issues with Python tests.


[Draft] Manual edits in generated code to compiling against v0.4.0 gen output

As title states. Mostly for illustrative purposes.

We run cargo fmt as a post codegen step (i.e. the emitter does not do this) and should require the same for SDKs, verifying this in CI as well.


Feature Request: Add per ip or mac bandwith show & limiting feature

I feel absence of per user bandwidth monitoring and limiting feature at luci interface.
It's possible using external tools like https://github.com/wtc0der/wtBRMon (it's lightweight) but why luci interface does not have similar one? It's really good thing to find which device consumes the internet in real-time. I do not think if it's hard to add this or similar tool to luci.
Also many recent routers allows seting bandwidth limit per IP. I really like such a tool available at OpenWRT's luci.
Thanks.

For the monitoring part, the nlbwmon app (and luci-app-nlbwmon) may work for you.
But that is only monitoing, not limiting.


feat: DIA-1144: SDK demo for mensuration and polling

Hi @matt-bernstein!
This PR was created in a response to PRs in upstream repos:

https://github.com/HumanSignal/label-studio-sdk/pull/245

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.


better error handling if current directory isn't a repository

If the current directory is not a repository, most git commands fail with a StopIteration error. This PR tries to fix that and issue a better error message

Looks good to me.


Bump dcos-diagnostics

High-level description
What features does this change enable? What bugs does this change fix?
Add timeouts and retries in dcos-diagnostics
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-5097 Add timeouts and retries in dcos-diagnostics

Related tickets (optional)
Other tickets related to this change:

DCOS_OSS- Foo the Bar so it stops Bazzing.

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label Ready For Review


cannot customize ValidationException

ValidationException should be customizable....

solved in new release https://github.com/syamsoul/laravel-datatable-ssp/releases/tag/3.4.1


Support for stacked coordinate layout (64x64 with 2x64x32 panels) in addition to side by side (128x32)

I tried your library, it worked fine on my ESP32, thank you.
For 64x32 it just works.
That said, I have chained panels and would like to try 64x64 and ultimately 64x96 which does work with Louis' SmartMatrix already
I went in your library edited ESP32-RGB64x32MatrixPanel-I2S-DMA.h
#define MATRIX_HEIGHT			    64
Once I do, I get this
23:24:18.962 -> Setting up parallel I2S bus at I2S1
23:24:18.963 -> I2S setup done.
23:24:18.964 -> Flushing buffer 0Guru Meditation Error: Core  1 panic'ed (LoadProhibited). Exception was unhandled.
0x400d12af: RGB64x32MatrixPanel_I2S_DMA::updateMatrixDMABuffer(short, short, unsigned char, unsigned char, unsigned char) at /home/merlin/Arduino/libraries/ESP32-RGB64x32MatrixPanel-I2S-DMA/ESP32-RGB64x32MatrixPanel-I2S-DMA.cpp line 266
0x400d0c11: RGB64x32MatrixPanel_I2S_DMA::flushDMAbuffer() at /home/merlin/Arduino/libraries/ESP32-RGB64x32MatrixPanel-I2S-DMA/ESP32-RGB64x32MatrixPanel-I2S-DMA.h line 233
0x400d0c5c: setup() at /home/merlin/Arduino/libraries/ESP32-RGB64x32MatrixPanel-I2S-DMA/ESP32-RGB64x32MatrixPanel-I2S-DMA.h line 182
0x400e5a77: loopTask(void*) at /home/merlin/Arduino/hardware/espressif/esp32/cores/esp32/main.cpp line 15
Details:
23:24:18.859 -> *****************************************************
23:24:18.861 ->  HELLO !
23:24:18.862 -> *****************************************************
23:24:18.863 -> lsbMsbTransitionBit of 0 requires 198304 RAM, 113792 available, leaving -84512 free:
23:24:18.863 -> lsbMsbTransitionBit of 1 requires 149152 RAM, 113792 available, leaving -35360 free:
23:24:18.894 -> lsbMsbTransitionBit of 2 requires 124576 RAM, 113792 available, leaving -10784 free:
23:24:18.896 -> lsbMsbTransitionBit of 3 requires 112288 RAM, 113792 available, leaving 1504 free:
23:24:18.898 -> Raised lsbMsbTransitionBit to 3/7 to fit in RAM
23:24:18.898 -> ns per latch: 3200:
23:24:18.899 -> nsPerFrame: 3481600:
23:24:18.900 -> lsbMsbTransitionBit of 3 gives 287 Hz refresh:
23:24:18.925 -> Raised lsbMsbTransitionBit to 3/7 to meet minimum refresh rate
23:24:18.928 -> Descriptors for lsbMsbTransitionBit 3/7 with 32 rows require 12288 bytes of DMA RAM
23:24:18.930 -> SmartMatrix Mallocs Complete
23:24:18.930 -> Heap Memory Available: 367940 bytes total, 113792 bytes largest free block:
23:24:18.931 -> 8-bit Accessible Memory Available: 279436 bytes total, 113792 bytes largest free block:
23:24:18.933 -> 32-bit Memory Available: 367940 bytes total, 113792 bytes largest free block:
23:24:18.960 -> DMA Memory Available: 279436 bytes total, 113792 bytes largest free block:
23:24:18.961 -> Performing I2S setup.
23:24:18.962 -> Setting up parallel I2S bus at I2S1
23:24:18.963 -> I2S setup done.
23:24:18.964 -> Flushing buffer 0Guru Meditation Error: Core  1 panic'ed (LoadProhibited). Exception was unhandled.
23:24:18.964 -> Core 1 register dump:
23:24:18.965 -> PC      : 0x400d12af  PS      : 0x00060730  A0      : 0x800d0c14  A1      : 0x3ffb1f10
23:24:18.993 -> A2      : 0x3ffc2420  A3      : 0x00000000  A4      : 0x00000000  A5      : 0x00000002
23:24:18.995 -> A6      : 0x00000000  A7      : 0x00001f82  A8      : 0x00000002  A9      : 0x00000000
23:24:18.997 -> A10     : 0x00000000  A11     : 0x00000000  A12     : 0x00000001  A13     : 0x00000008
23:24:18.998 -> A14     : 0x00000002  A15     : 0x00000004  SAR     : 0x00000020  EXCCAUSE: 0x0000001c
23:24:19.025 -> EXCVADDR: 0x00000002  LBEG    : 0x400014fd  LEND    : 0x4000150d  LCOUNT  : 0xffffffff
23:24:19.026 ->
23:24:19.027 -> Backtrace: 0x400d12af:0x3ffb1f10 0x400d0c11:0x3ffb1f60 0x400d0c5c:0x3ffb1f80 0x400e5a77:0x3ffb1fa0
23:24:19.028 ->
23:24:19.028 -> Rebooting...

I tried 50, 100, 150, 200 for
    ramrequired += 200000; // HACK Hard Coded: Keep at least 100k free!

and they all crashed for 64 height.
You can easily reproduce this by changing the number if you'd like.
#define MATRIX_HEIGHT			    64
That said, I just wrote my own GFX library on top of SmartMatrix, so admittedly I don't need this anymore:
https://github.com/marcmerlin/SmartMatrix_GFX
Still, the DMA approach of your library is interesting, especially the full frame refresh approach as opposed to line refresh approach that SmartMatrix does (albeit with more bits per pixel), so it would be worth keeping IMO.

Ah.. I'm not sure if vertically stacked panels will work with this library. I did set the WIDTH to 128 (i.e simple horizontal chain of panels) and it seemed to run.
I'm thinking I might as well just port Louis's or your library and add DMA at some point.
What type of ESP device are you using?

@mrfaptastic  interesting, so you do support 4096 pixels as long as they are 128x32 and not 64x64, so at least it's not a memory problem.
Can you confirm that 64x64 crashes for you too?
For what you can do long term, I know Louis would be grateful to have your DMA support in his lib. I'm assuming it's less work to port your DMA code to his lib than his entire lib to your driver.
My lib does not contain drivers at all, unless you count a virtual framebuffer, a driver :)  I wrote it as a layer to put on top of Louis' library to add GFX and FastLED API support.
If you can, have a look at https://github.com/marcmerlin/FastLED_NeoMatrix_SmartMatrix_LEDMatrix_GFX_Demos/
it contains a lot of cool demos that will now run on top of SmartMatrix via SmartMatrix::GFX . Most won't work out of the box on your lib because they also use the FastLED array of CRGB API, but the fontzoom demo will work since I wrote that one and it only uses GFX.
See video: https://youtube.com/watch?v=bPLuH3Ln9gU
Code: https://github.com/marcmerlin/FastLED_NeoMatrix_SmartMatrix_LEDMatrix_GFX_Demos/tree/master/GFX/fontzoom
SmartMatrix::GFX could be made to work on top of your library with minimal effort, but given that it crashes for me in 64x64 .
Oh, I didn't realize your lib was 24bpp. The GFX library is 16bpp by the way.
I noticed you were using dma_display.color444 when GFX is 565 natively, but now I see that you have a 24bit bypass draw function just like I do.
I just updated https://github.com/marcmerlin/SmartMatrix_GFX/blob/master/README.md to correct the bit about your lib not being 24bpp.
ESP32, I'm using a bare chip with level shifters. Do you care which one exactly?
Picture/video here: https://www.youtube.com/watch?v=FjBL09Qceqs

Try again now. I don't have access to a 64x64 panel, or own two  64 (wide) x 32 (high) panels to able to test chaining.

I should note (you would already know this though), if you're using a single 64x64 pixel panel, you'll need to assign a E_PIN I think for this library to work with it.
If you're using 2 x 64*32 panels and sitting them on top of each other, this library isn't designed for that. You'll need to set the width to 128 and height to 32px in the library and in your animation code, do the necessary virtual co-ordinate mapping.

I have both 64x64 with E address that I know your library doesn't support, and 64x32x2 chained but as 64x64, not 32x128.  SmartMatrix supports 64x64 without changing my code to remap the pixels from 32x128, to 64x64, so I'll probably stick with that for now.
Is it not reasonably easy to add in the lib itself?  (yes I understand I can do it in code too, but it's a bit of a pain and something the lib should support :) )

Did my change not work? It's probably easier just use the smart matrix code. It would be easy to add support I think, but I can't as I don't have the physical panels to test.

I believe there is already a SmartMatrix experimental branch which uses the same DMA library.
https://github.com/pixelmatix/SmartMatrix/tree/teensylc

It adds support for ESP32 (not supported in the main branch), but I don't think it supports DMA. Louis is actually the person who told me to try your branch for DMA :)
(that said, I didn't inspect the driver code, I assumed it still did bit banging, but I didn't verify it)

you are correct, my bad (was re-reading the wrong README).
I think the difference he outlined then was that he was still doing line level PWM while you were doing full page PWM, which shows up on video where I get streaks if the refresh rate is too low with his lib, while with yours I get a rolling shutter effect which is not as bad.

Ok, I was just able to confirm that 32x128 works like you suggested, while 64x64 crashes, so it's "just" a layout problem in the coordinate mapping like you suggested.
I'll update the description accordingly.

Closed.

Are stacked panels supported now? (64x64, not 32x128)

They are now.
https://github.com/mrfaptastic/ESP32-RGB64x32MatrixPanel-I2S-DMA/tree/master/examples/ChainedPanels

woohoo, well done.

pardon me, I try to use stacked chain with VirtualMatrixPanel class. but can't call flipDMABuffer() and showDMABuffer().
is it unsupported?
I try to animate scroll text, but it so flickering. I'm using virtualDisp.fillscreen(0) each loop.

Can you post a video?
You don't call flipDMABuffer etc via the virtualDisplay class, you need to call the underlying DMA class. i.e. dma_display.flipDMABuffer();
virtualDisplay.()// do your painting
dma_display.showDMABuffer();


Readme.md Link error?

bit.ly/nodejs-min and bit.ly/nodejs-dev-install are links to the same file.
Intended or wrong link?

That's an oops.


✨ feat: supports regex operations in various cli commands

Description
This PR introduces batch or regex operations for the following commands

rio deployment delete
rio deployment update
rio package delete

Wrike: https://www.wrike.com/open.htm?id=1235227535

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


terraform012 #136 - Fix issue invalidParCombi between mon_role_arn and mon_interval

Description
This fix issue #136  for terraform 0.12
PS: This has not being tested =( since in my infra I still use 0.11. It would be nice if someone could test

No fix. I still get the error
module.db_instance.aws_db_instance.this[0]: Creating...

Error: Error creating DB Instance: InvalidParameterCombination: A MonitoringRoleARN value is required if you specify a MonitoringInterval value other than 0.
	status code: 400, request id: aeb9b125-e8fb-4290-a4e3-09d79502e5d7

ok, seems you didn't set the same as v0.11
This code works:
monitoring_role_arn = "${var.monitoring_interval > 0 ? coalesce(var.monitoring_role_arn, join("", aws_iam_role.enhanced_monitoring.*.arn)) : "" }"

I updated the code and added the join. @ozbillwang can you review again?

yes, it works with mysql.
I didn't test with mssql. should be fine.

Thanks, @flmmartins !
v2.12.0 has been released for Terraform 0.12 users.


Runnable discovery jar file in Linux

Added Main class: https://github.com/evercam/evercam-discovery-java/blob/master/src/io/evercam/network/Main.java

Well done @liutingdu  ... I'm sure @scandox can't wait to use it.

I can't. So.excited
On 22 Jul 2015 17:33, "Marco Herbst"<EMAIL_ADDRESS>wrote:

Well done @liutingdu https://github.com/liutingdu ... I'm sure @scandox
https://github.com/scandox can't wait to use it.
—
Reply to this email directly or view it on GitHub
https://github.com/evercam/evercam-discovery-java/issues/3#issuecomment-123782078
.

https://github.com/evercam/evercam-discovery-java/tree/master/release


extraQueryParameters does not work when using msal4j-brokers

Hi all!
extraQueryParameter does not work when using msal4j-brokers 1.0.2-beta and previous.
This instructions:
InteractiveRequestParameters parameters = InteractiveRequestParameters
    .builder(new URI("http://URI"))
    .scopes(scope)
    .windowHandle(1)
    .domainHint(domainHint)
    .loginHint(loginHint)
    .prompt(Prompt.LOGIN)
    .extraQueryParameters(MapOf("hsu","1))
    .build()

generate following exception:

Exception in thread "main" java.lang.NullPointerException: String initializer must be non-null
	at com.sun.jna.WString.<init>(WString.java:33)
	at com.microsoft.azure.javamsalruntime.AuthParameters$AuthParametersBuilder.additionalParameters(AuthParameters.java:104)

Function in which exception occours:
 public AuthParametersBuilder additionalParameters(Map<String, String> parameters) {
            if (parameters != null) {
                for (Map.Entry<String, String> param : parameters.entrySet()) {
                    MsalRuntimeInterop.ERROR_HELPER.checkMsalRuntimeError(
                            MsalRuntimeInterop.MSALRUNTIME_LIBRARY.MSALRUNTIME_SetAdditionalParameter(
                                    handle.value(), new WString(param.getKey()),
                                    new WString(parameters.get(param.getValue()))));
                }
            }
            return this;
        }

error does not occur using only MSAL4j

Hello @flowchart87 : Thanks for bringing this to our attention, it looks like the bug is from a mistake during some refactoring.
In that additionalParameters() method the line new WString(parameters.get(param.getValue())) is incorrectly trying to get a value from the original parameters map, and instead it should simply be new WString(param.getValue())
This will be fixed in the next beta release, and I'll update this thread once it's available.

Sorry for the lack of update on this, however this issue was fixed in an internal-focused beta release, and has just been officially released as part of version 1.14.0 of msal4j and 1.0.0 of msal4j-brokers.


fix: fix validation method in kubernetes provider

RBAC allows a user to define a wildcard * for a given field in the Resource Rule. Prefix/Suffix matching or globbing is not supported, just simple wildcards.
For example the cluster-admin role has a * on all apiVersion/resource/verbs and hence validation would fail.
Fixes #850

/approve


[WIP] Add a LuaMutex type to borrow the InsideCallback

Fix #10
Blocked by https://github.com/rust-lang/rust/issues/38673
Changes are:

The parameters of a Rust function passed to Lua must now implement LuaRead<&InsideCallback> (right now it's LuaRead<&mut InsideCallback> + 'static, which is a hack)
All basic types (String, integers, etc.) already implement LuaRead<&'a InsideCallback> so they will still work.
Types such as LuaFunction or LuaTable do not implement LuaRead<&'a InsideCallback>.
A new type LuaMutex<T> (where T can be LuaFunction or LuaTable) does implement LuaRead<&'a InsideCallback>.
The LuaMutex struct has a lock function that locks the InsideCallback and reads the T. As long as the T is alive, no other LuaMutex can be locked.

So for example if you want a Rust function that takes two tables as parameter, you can pass a closure like |(table1: LuaMutex<LuaTable<_>>, table2: LuaMutex<LuaTable<_>>)|. You can then read for example table1 by calling table1.lock(). The lock function will return a LuaTable.
As long as the LuaTable is alive, table2 can't be locked.
See #10 for an explanation about why this is necessary.
A future change is to allow LuaRead<&mut InsideCallback> if there is only one parameter, which would allow closures with only one parameter to bypass this LuaMutex system.

Also added the 'lua lifetime to InsideCallback to fix soundness.

I managed to make it work for InsideCallback<'static> only. Since this is the most common case, I'm going to push the change and fix this later.

The current code is still unsound because it allows reading and writing the same userdata simultaneously.


Animated images doesn't show

In UI I have weather card with animated images used for current and forecasted conditions.
Those images do not load on same connection (internal and external) on which direct shortcut on Android work flawlessly.
I'm using latest firebase version (26).

are you sure its firebase version 26? I only show 22 and 23.  I find it helpful to check the version under App Configuration when you load the app and hit the menu.

It was before the first coffee, it's 22.


Incorrect type inference

Consider the following Angular template.
<div *ngLet="flag$ | async; let flag">

Here, flag$ is an Observable<boolean>. Hence, I would expect flag to be boolean | null. It is nullable since the async pipe can return null when it doesn't have any values to return.
However, in VS Code when I hover over the flag variable then the type checker shows that it inferred the type as true, i.e. the only truthy value out of true, false, and null. This is incorrect. I can see from experimentation that falsy values are bound too.

Thanks! I've fixed in version 15.0.2 https://www.npmjs.com/package/ng-let


[Bug]: Should jsx-no-leaked-render cover unintended return values as well?

Is there an existing issue for this?

[X] I have searched the existing issues and my issue is unique
[X] My issue appears in the command-line and not only in the text editor

Description Overview
Conditionally rendering a string value with extra braces are hard to spot and can cause a runtime error.
Should jsx-no-leaked-render cover also return values?
type Props = {
  isInteractive: boolean
  label: string
}
const Test: React.FunctionComponent<Props> = ({ isInteractive, label }) => (

    {/* Will render `{label}` (object) instead of a string when `isInteractive` is false */}
    { isInteractive
      ? <button>{label}</button>
      : {label}
    }

    { isInteractive && <button>{label}</button> }
    {/* ...same issue as above */}
    { !isInteractive && {label} }

)

When isInteractive is false the code will return an object instead of a string, but it's easy to miss in the ternary case. The error is Uncaught Error: Objects are not valid as a React child (found: object with keys {label}). If you meant to render a collection of children, use an array instead.
Appears both in VS Code Eslint plugin (dbaeumer.vscode-eslint) and command line running npx eslint . with following config:
{
  ...
  "extends": [
    "eslint:recommended",
    "plugin:@typescript-eslint/recommended-type-checked",
    "plugin:react/recommended",
    "plugin:react-hooks/recommended"
  ],
  ...
  "parser": "@typescript-eslint/parser",
  "parserOptions": {
    "ecmaFeatures": {
      "jsx": true
    }
  },
  ...
  "plugins": ["react", "@typescript-eslint"],
  "rules": {
    "react/jsx-no-leaked-render": "error"
  },
  "settings": {
    "react": {
      "version": "detect"
    }
  }
}

Expected Behavior
Report extraneous braces around strings on conditional renders. Remove extra braces on fix.
const Test: React.FunctionComponent<Props> = ({ isInteractive, label }) => (

    { isInteractive
      ? <button>{label}</button>
      : label
    }

    { isInteractive && <button>{label}</button> }
    { !isInteractive && label }

)

eslint-plugin-react version
v7.33.2
eslint version
v8.47.0
node version
v20.2.0

I'm confused, label is typed as a string. Is the TS type wrong?

@ljharb I believe Typescript compiler can't guard against this issue because the React.FunctionComponent return type is ReactElement<any, any> | null and it can contain nested components, jsx elements and plaintext.
If we focus on the conditional, the issue is that one can accidentally return an object instead of evaluated string. So if the component is called with <Test label="Frank"  isInteractive={false} />
{ !isInteractive && label } – ts return value evaluates to string, Frank
{ !isInteractive && {label} } – ts return value evaluates to object, {label: "Frank"}, and it will cause this error.
This case is still quite easy to spot, but it took a long time for me to understand the issue when using a ternary. It would be great if Eslint could spot this issue when Typescript can't.

ah, i see what you mean.
I'm really unsure if 1) this behavior should be checked at all, or 2) checked in this rule, or 3) checked in its own rule. It's a very subtle case.


bump symfony/translation-contracts

https://github.com/symfony/translation-contracts/releases/tag/v3.0.0

Thank you :+1:

Thank you :+1:


"Had to run as admin" and "VCRUNTIME140D.dll not found" etc.

Multiple users seem to be having issues with the visual C++ installation. I am unaware of what the users have done. Some of them have reportedly tried installing all versions of C++ for reasons that are entirely beyond me. No Man's Sky installation process installs C++, meaning the user's environment can access C++ for NMS but not for any other program, and in one case can do so only when they use elevated privileges for other programs, but normal privileges for NMS.
I genuinely do not know what the users are doing, as they refuse to follow the basic instructions listed on the modpage itself, then do not elaborate on if they can reproduce the error, do not elaborate if a solution has fixed the issue, anything. These errors should, without hyperbole, not even be possible given that the game they are using this software for installs the prerequisites for them, otherwise the game would not run. Because I cannot know where this error comes from, I am filing this to be marked as completed when the switch back to AHK happens, as the executable will then ship with literally everything needed to ever execute it on any Windows machine at the cost of an entire rewrite and file size.

DLL issues should be fixed with version 2.3.0. I'l,l reopen this if the error crops up again.

The reason this build should work to fix this is for twofold: No Man's Sky used to install Visual C++ 2017, and now installs 2022. All versions past 2015 share the same redistributable. This means anyone, even people who are on older versions of the game, should have the necessary library already installed on their computer. For some reason, and I'd really have to check when, my VS decided 2013 was the version it was going to use to compile with, which was not what I have as default. This is likely when people started experiencing errors.
Not all people experienced issues. However, this may just mean that another game or another program etc. already installed VS 2013 on their machine already. As the developer (but mostly as someone with way too many game mods downloaded), I have quite a few versions of Visual C++ installed, so I didn't catch this change because I already had it installed, and so did the second machine I double-checked with. This caused me to suspect broken versions installed of C++ rather than my compiler which was supposed to be on defaults.
https://github.com/Kubia-Beta/Hold_to_Craft/releases/tag/v2.3.0 which was compiled with 2017, originally required by NMS, and the future versions will come installed with compatibility for by Microsoft's design.


Relations with an includeXxx method won't be eager loaded anymore

This opens up scenarios where there is a need to consider as a relation what in fact is just a method returning a Model or a Collection.
This resolves the original question of #84

It is possible to mock method_exists  using php-mock or with some trick using a different namespace for test suite, but I think that using a stub in this case is a more elegant solution

I see the stub has 'baz' as a auto-loaded relation in the $load property. I also see the tests pass, but shouldn't 'baz' also be eager loaded?

Nevermind, I see now that I missed the second includeXxx method in the Transformer

This only seems to handle root transformers though, what about nested relations that has transformers with includeXxx methods?

You're right I guess, I didn't noticed that aspect.
I don't really know where to put my hands to operate the filtering both on the top level relations and nested ones. Any hints?
Btw, while checking the code I found this which seems like an undocumented feature to define eager load query restriction on a per-Transformer basis
if (method_exists($this, $method = 'load' . ucfirst($relation))) {
    $eagerLoads[$relation] = function ($query) use ($method) {
        return $this->$method($query);
    };
}

I think we need to think along the lines of adding transformer mappings on the $relations property and then iterating over the transformers in the same way we extract default relations from $load. So basically, move the logic from the TransformBuilder class to the Transformer and make a recursive method in there.
About the loadXxx feature, it only worked for relations in $load which I believed would cause for confusion to the end user, so I let it out of the documentation until I could find a way to attach it to all relations. If we end up adding transformer mappings to the $relations property as well, I think that will also solve the problem for the loadXxx methods.

Do you have something already done about adding mappings also to $relations property?
Or shall I try to implement it myself?

Nope, haven't had time to dig into that yet. I'm approaching an important deadline at work and I've had to prioritize getting closer to that goal first. I will have more time at the end of the month though. With that being said, I would highly appreciate any efforts to solve it - I think most of the logic should already be in the HasRelationships trait, and it should just be generalized to work for all relations. But don't feel obliged in any way :)

I think that at this point the TODO list shall be:

add mappings on $relation property as well;
uniform the flow for $load and $relations where possible;
fix #72
fix definitively #84

In the meantime, I think a partial solution (this PR) is still better than no solution...

But don't feel obliged in any way :)
Don't worry, if I'm working on this is because I need it too :P
I'll spend this week trying to fully understand the library code and resolving as many issues as possible (the ones I opened at least)

Thanks! And sure, we can merge this for now. Feel free to hit me up on Larachat (Laravel's Slack channel, username: flugg) if you have any questions regarding the codebase, I'd be happy to discuss.


Documents can be saved before blocks are finished rendering resulting in page content vanishing.

Bug Report
When modifying a document it's possible to click save before blocks and other fields have finished loading in. When this happens, the POST request will be missing the block information and the document will lose all of its block content.
I believe this also happens with relationship and upload fields. I can't confirm but I've had cases where a background image goes missing mysteriously and I suspect this is the cause of it.
Expected Behavior
Either users shouldn't be able to save until all document content has finished loading or the admin interface shouldn't render until it has all the content in place.
Steps to Reproduce

Add a bunch of blocks to a document
Go back to the index
Click on the document and click save as soon as you see it
Reload the page and all the content is gone :(

Detailed Description
Version 0.13.6

Hey @richardvanbergen — good call. We will address this shortly with a patch version.
Thanks for pointing this out!

Hey @richardvanbergen — sorry it took so long to address this one but we just released a beta version with a fix:
<EMAIL_ADDRESS>
Give it a shot! It'll hit a minor release once we merge in Versions, Drafts, & Autosave coming up soon!


Simplify distribution, migrate maps to .tmx format, correct BMFont file extension, support use as a library

Thanks for the pull request!
I'd rather avoid interactive wizard-style questions, since it's meant to be run quickly/repeatedly/automateably. It doesn't necessarily need to be configurable either since it's not meant to be a general-purpose unpacker; if the .tmx support works well, I'd be fine just dropping .tbin support since I prefer TMX anyway.

I checked the tmx exports and the all looked like they should. I don't see a good reason to keep tbin around either. I removed the choice again.

@Platonymous When Tiled writes CSV tiles for a .tmx map, it breaks each row of tiles onto its own line. So if you have a 4x4 map, the data would be formatted like this:
<layer id="1" name="Back" width="80" height="75">
  <data encoding="csv">
0,0,0,0,
0,5,5,0,
0,5,5,0,
0,0,0,0
  </data>
</layer>

TMXTile puts them all on a single line like this instead:
<layer id="1" name="Back" width="80" height="75">
  <data encoding="csv">0,0,0,0,0,5,5,0,0,5,5,0,0,0,0,0</data>
</layer>

Could you change TMXTile to match the Tiled behavior?

I changed the extension for the BMFont files from xml to fnt, as that's what BMFont (https://www.angelcode.com/products/bmfont/) actually uses.

I changed the export format to csv, and TMXTile 1.5.1 has the layer-width dependend line breaks.

This PR has too many unrelated changes now, some of which could be merged now if they weren't part of a combined PR. Can you submit separate PRs so they can be reviewed/discussed/merged separately?
Suggested split:

Add assembly probe path to simplify distribution.
Add support for use as a library.
Correct BMFont file extension.
Migrate maps to TMX format.

All seperated. Closing this.


bump-version script: further tweaks

This PR add the following tweaks to the bump-version script:

When running npm run bump-version, it no longer increments the pre-release versions of the independent packages

Because it seemed like side-effect or accidental changes. For simplicity, let's remove this confusing behaviour.

Do list all the versions in more places

We were listing them while calling npm run bump-version. But now for more consistent DX, we're now doing it for the other variations too (e.g. npm run bump-version:retail-react-app)

How to test the changes
cd <monorepo-root>
npm run bump-version -- 3.1.0
# only the SDK versions are updated (and not commerce-sdk-react, retail-react-app)

npm run bump-version:retail-react-app -- 1.1.0
# you'll see in the output the list of versions for all packages in the monorepo

@vmarta Just making sure I understand.
By default, npm run bump-version will bump only the SDK versions and npm run bump-version:retail-react-app bumps just the retail-react-app but will afterwards print the versions of everything (SDK, commerce-sdk-react, retail-react-app) on the console?

@vmarta Just making sure I understand.
By default, npm run bump-version will bump only the SDK versions and npm run bump-version:retail-react-app bumps just the retail-react-app but will afterwards print the versions of everything (SDK, commerce-sdk-react, retail-react-app) on the console?

@vcua-mobify that's correct. The output log is like that because I want us to easily see the big picture. Yes, you're only incrementing the version of retail-react-app, but usually we'll need to think of it in the context of the entire monorepo.


PR try.v2 for issue #453

Allow HSET \ HSETNX to set value as interface{}.
Fix to #453

Thanks!

It's my pleasure


Add ChefSpec Unit Tests

Add ChefSpec and resources for unit test coverage of current cookbook resources.

@pwelch If you wanna rebase this I'd love to get it merged in

@tas50 Rebased and fixed some Rubocop warnings. Let me know if you need anything from me.
Thanks!


(WIP) Introducing new command Remove-ClusterGroupFromAffinityRule

PR Summary
Add new command to the command reference.
PR Checklist

[X] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[X] Summary: This PR's summary describes the scope and intent of the change.
[ ] Contributor's Guide: I have read the contributors guide.
[ ] Style: This PR adheres to the style guide.

Consolidated into #3570


Fine tune function checks

Fixes: #1591

@ttuegel the tests look like the correct behavior to me, but I think you should double-check them too to make sure the correct behavior was captured.


Dropdown not working if clicked on material-icons

Description
If we put a material-icons inside the trigger and we click on it, the dropdown is not working
Repro Steps
codepen
Click on the gear icon to test

Catch the click on the icon and deliver the click event to the dropdown.
Is this example / case described in our docs (dropdown with icon)?

Do you have an example ?
I tried this :
$(".dropdown-button .material-icons").bind('click', function() {
    $(this).closest(".dropdown-button").trigger('click');
})

But it's not working. I also tried with .click() instead of trigger.
The selector is working, if I do a .remove() instead of trigger, the anchor disappear.

This was fixed in 0.98.2
See: https://codepen.io/anon/pen/gWxQqY

Try this: http://codepen.io/anon/pen/wdqQVG

Thanks but upgrading to 0.98.2 worked :)


Add File tutorial.

Based on #1 this adds the second part of the series: Files.

Thanks for the review, waiting for editorial review at the moment. 👍


nerdctl stats --format={{json .}} outputs invalid CPUPerc / MemPerc

Description
When running nerdctl stats --format='{{ json . }}', the CPUPerc and MemPerc fields are invalid, causing JSON parsers to fail.
(This command appears to be used in some Docker Desktop extensions.)
Steps to reproduce the issue

nerdctl run -d --restart=always nginx/stable (or any other method of creating at least one container of your choice).
Run nerdctl stats --all --no-stream --format '{{json .}}'

Describe the results you received and expected
$ nerdctl stats --all --no-stream ; nerdctl stats --all --no-stream --format '{{json .}}'
CONTAINER ID   NAME          CPU %     MEM USAGE / LIMIT   MEM %     NET I/O         BLOCK I/O         PIDS
09fa776b3a1e   nginx-09fa7   0.00%     6.363MiB / 8EiB     0.00%     2.08kB / 822B   10.5MB / 8.19kB   3
{"Name":"nginx-09fa7","ID":"09fa776b3a1e","CPUPerc":"0.00%!"(MISSING),"MemUsage":"6.363MiB / 8EiB","MemPerc":"0.00%!"(MISSING),"NetIO":"2.08kB / 822B","BlockIO":"10.5MB / 8.19kB","PIDs":"3"}

Expected:
{"Name":"nginx-09fa7","ID":"09fa776b3a1e","CPUPerc":"0.00%","MemUsage":"6.363MiB / 8EiB","MemPerc":"0.00%","NetIO":"2.08kB / 822B","BlockIO":"10.5MB / 8.19kB","PIDs":"3"}

The difference is around the (MISSING) bits; looks like it might be fmt.Printf errors?
What version of nerdctl are you using?
Client:
 Version:	v1.3.1
 OS/Arch:	linux/arm64
 Git commit:	b224b280ff3086516763c7335fc0e0997aca617a
 buildctl:
  Version:	v0.11.5
  GitCommit:	252ae63bcf2a9b62777add4838df5a257b86e991

Server:
 containerd:
  Version:	v1.6.8
  GitCommit:	9cd3357b7fd7218e4aec3eae239db1f68a5a6ec6
 runc:
  Version:	1.1.4
  GitCommit:	5fd4c4d144137e991c4acebb2146ab1483a97925

That's the v1.3.1 tag.
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
Rancher Desktop for macOS
Host information
Client:
 Namespace:	default
 Debug Mode:	false

Server:
 Server Version: v1.6.8
 Storage Driver: overlayfs
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Log: fluentd journald json-file syslog
  Storage: native overlayfs
 Security Options:
  seccomp
   Profile: default
 Kernel Version: 5.15.107-0-virt
 Operating System: Alpine Linux v3.16
 OSType: linux
 Architecture: aarch64
 CPUs: 2
 Total Memory: 1.93GiB
 Name: lima-rancher-desktop
 ID: 167b5ee9-8314-4cce-b381-75ca5cd78c8a

WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

I'll take this


Admin portal Personal Info values should be comma separated

To easily wrangle and sort this data in a spreadsheet, we need standard comma separation of all personal info
For example, a current entry in the table is Ginny Fahs<EMAIL_ADDRESS>+14045507055
It should be Ginny Fahs<EMAIL_ADDRESS>+14045507055
Let me know if you have Qs about what we need here - thanks!

Good point! The fix will be in the next release 👍

Verified as fixed


[SPARK-40098][SQL] Format error messages in the Thrift Server

Context

Spark 3.4:

https://github.com/apache/spark/commit/abb4345073
[SPARK-40180][SQL] Format error messages by spark-sql https://github.com/apache/spark/commit/7046978949

Other related error formatting auditing issues: https://github.com/NVIDIA/spark-rapids/issues/6293

What changes were proposed in SPARK pull request?
This PR introduces new SQL config spark.sql.error.messageFormat to control the format of Spark SQL error messages.
The config can have at least the following string values:

PRETTY - textual form of error messages including error class/sub-class, message itself + query context in text form.
MINIMAL - JSON representation of SparkThrowable. Compact form w/ minimal set of fields. Aims to be parsed machinery.
STANDARD - JSON form contains the same set of fields as MINIMAL + message.

Why are the changes needed in Spark?

Spark SQL client tools need to parse out relevant information like parameters and positions in order
to render messages to their liking. This makes the API fickle, hard to change, and limits degrees of
freedom for UI.
Updates to error message texts result in a need to discover and update QA tests being affected.
This makes it expensive to improve error-messages and prevents e.g. tech writers from taking ownership
down the road.

Does this PR introduce any user-facing change?
Yes
Added new method getMessage() to SparkThrowableHelper which formats SparkThrowable + Throwable according to the requested format (see the SQL config). When the format is MINIMAL or STANDARD, the JSON record includes the following fields:

errorClass - human-readable, unique, and consistent representation of the error category. It is a string.
errorSubClass - optional sub-class of the error class as a string. It may be missing in the result string.
sqlState - optional string w/ error identifier across SQL engines. It may be missing in the result string.
message - the message template from error-classes.json. It is included to JSON when the format is STANDARD.
messageParameters - a map of message parameter names to its values.
queryContext - an array of query contexts of the error message which the following JSON fields. If it is empty, the array is not outputted:

objectType - the string represents the object type of the query which throws the exception. If the exception is directly from the main query, it should be an empty string.
objectName - the string represents the object name of the query which throws the exception. If the exception is directly from the main query, it should be an empty string.
startIndex - the integer represents the starting index in the query text which throws the exception. The index starts from 1. If the index is unknown, the field is not outputted.
stopIndex - the integer represents the stopping index in the query which throws the exception. The index starts from 1. If the index is unknown, the field is not outputted.
fragment - the string represents the corresponding fragment of the query which throws the exception. If the query fragment cannot be identified, the string is empty.

For example:

MINIMAL:
 {"errorClass":"DIVIDE_BY_ZERO","sqlState":"22012","messageParameters":{"config":"\"spark.sql.ansi.enabled\""}}

STANDARD:
  {
    "errorClass" : "DIVIDE_BY_ZERO",
    "message" : "Division by zero. Use `try_divide` to tolerate divisor being 0 and return NULL
                instead. If necessary set <config> to \"false\" to bypass this error.",
    "sqlState" : "22012",
    "messageParameters" : {
      "config" : "\"spark.sql.ansi.enabled\""
    }
  },

When errorClass is null, Spark outputs the error message in the legacy form where:

errorClass is LEGACY
messageParameters is an array w/ only one elements as a string. The element contains textual error message.

In the case if the specified format is PRETTY, Spark follows existing behavior and outputs error message as plain text, for example:
[DIVIDE_BY_ZERO] Division by zero. Use `try_divide` to tolerate divisor being 0 and return NULL instead. If necessary set "spark.sql.ansi.enabled" to "false" to bypass this error.
== SQL(line 1, position 8) ==
select 1 / 0
       ^^^^^

Why it might affect RAPIDS?

the new configuration spark.sql.error.messageFormat is set to by default to ErrorMessageFormat.PRETTY.
the error message of Spark depends on this configuration.

Impact on Testing?
Yes, comparing Errors may be affected.
Requires Doc update?
Yes

Does not affect the plugin code.


Extended Data Fig. 9 interpretation

Hi,
I'm hoping this is a good place to post, otherwise I'm happy to move this to the scverse Discourse.
I just want to make sure I'm interpreting the above figure from the expiMap paper correctly.
You build a reference atlas and map a query dataset onto it using 4 methods:

expiMap + scArches
scVI + scArches
(Harmony +) Symphony
Seurat V4

In figure 9b, you calculated several scib metrics for each of these methods.
For these scib metric calculations, what is the "unintegrated" AnnData object that you're comparing to?
My guess is that you:

concatenate the query dataset to the reference atlas
calculate a PCA embedding X_pca(i.e. this is not "integrated")
for scib metrics that are based on an embedding, you compare the "unintegrated" adata.obsm['X_pca'] to an "integrated" adata.obsm['X_scVI], or adata.obsm['X_pca_symphony'] etc.
for scib metrics that are based on a graph, you re-compute the neighborhood graph based on the integrated embedding and compare it to the "unintegrated" neighborhood graph.

Please let me know if I'm understanding this correctly.

Hello, @maarten-devries , correct, for those methods of scib that require comparison against unintegrated data, PCA is used.