Apply check in Contact form

In the contact us form there is no check. Users can only contact when they are signed In and fill all the fields in the form.
Till then the button should be disabled.
I guess you can do it @Kingshuk3006

If this is still available and open-source I'd also like to be assigned.

Sure go ahead. You are assigned. @Akidane47

What are the setup instructions? I clicked on the link in the README file but it caused a 404 error

Try now I have changed it now. And also you don't need to set up an env for this. So you can skip that part too. @Akidane47

Hey are you still working on this @Akidane47 ?

Yes sorry was having some trouble with setting up credentials but it's
fixed now, will be sending my version later today
On Fri, Aug 19, 2022 at 7:53 AM Rajdeep Sengupta @.***>
wrote:

Hey are you still working on this @Akidane47
https://github.com/Akidane47 ?
—
Reply to this email directly, view it on GitHub
https://github.com/Clueless-Community/clueless-official-website/issues/164#issuecomment-1220773797,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG4OHP7DY6MHMUVZ2KC4S3TVZ6NXFANCNFSM56UKOC6A
.
You are receiving this because you were mentioned.Message ID:
@.***
.com>

No worries! Take your time. @Akidane47 .

Is this completed? @Akidane47

If this is still available and open-source I'd also like to be assigned and can u tell me more about this issue

Yes sure I will assign you.
what you need to do is just make the 'submit' button disable until all the fields are filled. @ridhig16

This issue seems to not be assigned to anyone, can I have it assigned to me if that is the case?

Yes sure. Go ahead! @akebu6

Hey @Rajdip019, I am having issues with VSCode enabling TypeScript so it will take a while for me to work on this issue. Latest I should have it done this weekend after my College project proposal. Hope that is still okay

Yes it's fine. Take your time @Akidane47

I m a beginner, can u please describe the isssue in detaila nd assign it to me?

Hey @jyotika6221 I'm actually working on this so, can you kindly find another open issue go work on

Hey @Rajdip019, I think users can also contact when they are not signed In (Ref: https://ineuron.ai/contact-us), also disabling submit button would not be the best option. I guess form should be validated. I can get it done if assigned to me.

@Rajdip019 I think you can also assign it to @RajKanani1206 to make the form validation because I've also noticed that the firm inputs aren't validated.
He can work on the form validation part and I'll finish up the disable button until all fields and validation are done.
How does that sound @RajKanani1206 and @Rajdip019 ??

Okay work on it. @RajKanani1206

Done. I have created PR.


Disambiguate titles of dependency cards

Without this the EntityDependsOnComponentsCard and the EntityDependencyOfComponentsCard cards have the same title and you can't tell which is which.
sample-service-1 depends on sample-service-2.

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Argh. I added the wrong changeset. Fixing now.


FindById leaves open cursors on transaction

It seems like the 'FindById' operation on the database (and potentially other operations as well), leave an open cursor on the transaction.
This can easily be reproduced. Just begin a transaction on the database and run a 'FindById' operation on a collection. You will see that you can not commit the transaction as it still contains open cursors.
I think this has something to do with the combination of 'FirstOrDefault' and IEnumerable functions. My guess is that the cursor will only be returned after the enumerator completed and 'FirstOrDefault' will stop the enumerator after the first element.

Will leave an open cursor on the transaction (as the enumeration never completes):
collection.FindById(id);
Will NOT leave an open cursor on the transaction (as the enumeration completes):
collection.Find(x => x.Id == id, skip: 0, limit: 1).ToArray().FirstOrDefault();

https://github.com/mbdavid/LiteDB/blob/055f47d074d0a6f5da756a30cee817436c2f27eb/LiteDB/Engine/Engine/Transaction.cs#L38
This line currently stops you from running an 'FindById' operation inside of a transaction.

@clooudch Could you test it with v5.0.1?

Due to my tests, I can confirm that v5.0.1 fixed the problem. Thanks.


Newbie question

Is this for clone discord server and copy and update all the new messages in all channels in realtime?
Do I need to keep my computer open for the clone process?

same qst

How the program is supposed to run with the computer shutted down?


Expose build status before building

It would nice if there was a build_needed() method or something on the Builder struct so I could check this value before actually building and do other things if necessary in the build script. Right now at least I want to print something to the console saying that it's building the PTX kernel, but it could be useful for other things as well.

You can now use Builder::is_build_needed() method to check if we should build or not (we should not build from RLS or recursively).
New version 0.3.3 is released.


Volume and image sizes are not populated from rbd

Need to query the size from rbd and make sure that the datastore is populated with the appropriate sizes and then send that to the client.

(Updated to only refer to images, volume size mostly works.)


Doesn't work with postcss-import

postcss([
    postcssImport,
    use({
        modules: ['postcss-inline-svg']
    })
])

Input
// index.css
@import 'module.css';

.a {
    background: #fff svg-load('logo.svg', fill=red) 50% 50%;
    background-size: contain;
}

// module.css
@use postcss-inline-svg;

.c {
    background: svg-load('logo.svg', fill=blue) 50% 50%;
    background-size: contain;
}

output
.c {
    background: url("data:image/svg+xml;charset=utf-8,...") 50% 50%;
    background-size: contain;
}

.a {
    background: #fff url("data:image/svg+xml;charset=utf-8,...") 50% 50%;
    background-size: contain;
}

Has it turned to solve?  I have the same problem…

Are you making sure to load postcss-import before postcss-usein your build configs?

I’m seeing if I can help out here. I’m not sure this is a bug;
PostCSS Use couldn’t be applied until after PostCSS Import imported the file. At that point, the files have been merged, which means @use is being applied to the entire merged file.
I think this can be closed as a limitation in either PostCSS Use or PostCSS itself.
There might be a workaround in PostCSS Import using the plugins option, but at this time that would be out of the scope of this plugin.

Closing due to inactivity.


Toolset integrations

@itisallgood
Moving the prerequisite check for the toolsets when the server is started is a great improvement.
In the current code, both the Internet and Findings toolset do run the prerequisites when they are instantiated. This means that each of these toolset will still run the prerequisites for each request to Holmes server.


Adjust the sdk pages to support selecting the FIdP for CLI login

Purpose

Adjust the sdk pages to support selecting the federated IdP from the UI.

Goals

Adjust the sdk pages to support selecting the federated IdP from the UI.

Approach

Adjust the sdk pages to support selecting the federated IdP from the UI.

User stories

N/A

Release note

N/A

Documentation

N/A

Training

N/A

Certification

N/A

Marketing

N/A

Automation tests

Unit tests

N/A

Integration tests

N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples

N/A

Related PRs

wso2-cellery/sdk#384

Migrations (if applicable)

N/A

Test environment

N/A

Learning

N/A

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@66e0c11). Click here to learn what that means.
The diff coverage is 23.52%.

@@            Coverage Diff            @@
##             master      #22   +/-   ##
=========================================
  Coverage          ?   36.18%
=========================================
  Files             ?       39
  Lines             ?      713
  Branches          ?      113
=========================================
  Hits              ?      258
  Misses            ?      425
  Partials          ?       30

Impacted Files
Coverage Δ

components/portal/src/components/App.js
33.33% <ø> (ø)

components/portal/src/components/SignIn.js
14.28% <0%> (ø)

...ts/portal/src/components/sdk/FederatedIdpSelect.js
20% <0%> (ø)

...mponents/portal/src/components/sdk/SDKOrgCreate.js
8.33% <0%> (ø)

...s/portal/src/components/sdk/sdkAppLayout/Header.js
11.11% <0%> (ø)

components/portal/src/utils/api/httpUtils.js
72% <100%> (ø)

components/portal/src/components/sdk/index.js
33.33% <33.33%> (ø)

components/portal/src/utils/api/authUtils.js
61.22% <57.14%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66e0c11...59922b7. Read the comment docs.


Reorganize docs index a bit

Demote presentations from the top TOC,
it performs terribly on google analytics
Remove stale and missing content from presentations
Promote deployment doc to top section (performs well, related to installation)

Merging in 24 hours if there are no further comments.

These changes make sense to me! Thanks for removing stale/missing things.

Cool.  Merging now.  Happy to revert if anyone has concerns shortly.


Change name of "test" folder

Normally in software projects, the "test(s)" folder is where all your Unit Tests are, not experimental code. "Unit Tests" are functions that assert various states about your code. If you have code that activates a motor when you press a button, it's good to have a separate piece of code that verifies that functionality. If the behavior of that function is modified in the future, say by changing what button needs to be pressed to activate the motor, the test will catch that and will fail as a result. This can be used in combination with git hooks to prevent code from being committed to a branch unless all tests pass.
I would suggest "experimental" or "sandbox" for the folder names. More information here: https://docs.wpilib.org/en/stable/docs/software/wpilib-tools/robot-simulation/unit-testing.html

I suggest that we split it into "experiments" and "archive". "Experiments" would be for stuff like the led test code. "Archive" would be for stuff that should have never been in tests, like DEATH STARE(++) and nt-drive.


java-openliberty fails RH certification

The kabanerobeta/java-openliberty container fails RH certification on the following points....

Requires a summary label
Requires a vendor label
Requires a name label
Requires a description label

The labels need adding to the Dockerfile-stack (see nodejs for an example).

Fixed (master): https://github.com/kabanero-io/collections/pull/290
Fixed (release-0.6): https://github.com/kabanero-io/collections/pull/292

The latest stack updates resolve the failure.


optimizing count queries and indexes for project falco

Count queries in postgresql are slow... so this sidesteps the issue by either constraining to a specific set of days or just not counting at all.


Remove version: GitHub.GitHubDesktop version 3.3.18

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Publisher has changed their URL structure;


Carthage support

I'd like to see carthage support alongside cocoa pods support so that I can use my dependency manager of choice

Can't you just put github "Raizlabs/BonMot" in your Cartfile? Would anything in this repo need to change?

I could add a section to the ReadMe.

You need an xCodeProj that build bonMot as a framework with a public target to support carthage

This was closed when #65 was closed.


Can't input \supset

My snippet is default.
I can't input \supset. How can I modify the snippets to enable inputing it.
By the way, considering input subset bring about \subset, it seems there should be a snippet of subset in place of \\subset but I can't find it. It is confusing.

This is not a bug, but go to your snippet file (for example through settings).
Here you can remove the line: {trigger: "set", replacement: "\\{ $0 \\}$1", options: "mA"},, which will prevent the string set from triggering anything.

By the way, considering input subset bring about \subset, it seems there should be a snippet of subset in place of \subset but I can't find it. It is confusing.

I'm not entirely sure about what you mean, and I'm not sure my tip will work, but I think you can do this: Go to Settings>LatexSuite>AdvancedSnippetSettings>SnippetVariables. Do you see |subset| somewhere in there? If you replace that |subset| with |subset|supset|, then I think it should work. The strings subset and supset behave differently because subset is part of a snippet variable, but supset is not. This change ensures they're both part of that snippet variable.
I hope it still helps.

Thanks

This is not a bug, but go to your snippet file (for example through settings).
Here you can remove the line: {trigger: "set", replacement: "\\{ $0 \\}$1", options: "mA"},, which will prevent the string set from triggering anything.

By the way, considering input subset bring about \subset, it seems there should be a snippet of subset in place of \subset but I can't find it. It is confusing.

I'm not entirely sure about what you mean, and I'm not sure my tip will work, but I think you can do this: Go to Settings>LatexSuite>AdvancedSnippetSettings>SnippetVariables. Do you see |subset| somewhere in there? If you replace that |subset| with |subset|supset|, then I think it should work. The strings subset and supset behave differently because subset is part of a snippet variable, but supset is not. This change ensures they're both part of that snippet variable.
I hope it still helps.

Thanks, your advice works. Just replace that |subset| with |subset|supset|.

The same problem is with \times, interfering with "im" snippet. Solution to this problem is detecting whether user has typed backslash and characters without space or not and then applying the snippet.


Ready for Review - [Mouse Jump] Customisable appearance - borders, margins, colours, etc - final part

Summary of the Pull Request
This is the last part of the Mouse Jump styles work that finally enables the feature in Settings UI and in the Mouse Jump preview image.
See https://github.com/microsoft/PowerToys/issues/27511 for the original feature suggestion.
Changes in this PR

Moved the Mouse Jump "settings ui" xaml and code into a separate "MouseJumpPanel.xaml" control to make it easier to make changes to the settings controls
Added a new "Appearance" expander in Mouse Jump settings with all the new border, colour, 3d effect, spacing, etc settings controls to allow the user to set custom properties
Also added some predefined style options:

"Compact" is the original Mouse Jump style with just a borders around the whole image and no other chrome
"Bezelled" is the new default with some simple spacing between screens and 3d bezels drawn around them

Reviewer Notes

Added a reference to CommunityToolkit.WinUI.Controls.Segmented in PowerToys.Settings to allow the "Compact | Bezelled | Custom" options for Preview Style

Added a reference to MouseJump.Common in PowerToys.Settings to allow rendering the preview image when settings are changed

I've made "Bezelled" the new default style as a precursor to the proposed Mouse Without Borders integration in #34126 - it'll be easier to make the bezel colours vary by host if they're already visible.

Code touches the internals of the Segmented control to force the preview image to be centered - see PreviewImage_Loaded method - might be brittle if the implementation changes?

Config upgrade seems a bit odd - the in-memory settings get upgraded fine with "version": "1.`" but it seems to get reverted and saved again as "version": "1.0" and then overwritten again as "version"": "1.1". I've worked around this by additionally checking if the new settings have values before upgrading, but it seems like that's what the version flag is supposed to be for. It works for now anyway, but not sure if this is the right way to do it...

Does the "Copy to Custom preview style" need an "Are you sure? Yes / No" before overwriting and / or a "Copied!" message after copying? I'm happy to leave it as is, but feedback welcome...

PR Checklist

[x] Closes #27511
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[x] Tests: Added/updated and all pass
[x] Localization: All end user facing strings can be localized

all ui strings come from Resources.resw

[x] Dev docs: Added/updated

no changes required

[x] New binaries: Added on the required places

no changes required
[x] JSON for signing for new binaries
[x] WXS for installer for new binaries and localization folder
[x] YML for CI pipeline for new test projects
[x] YML for signed pipeline

[x] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

no changes required

Validation Steps Performed

Workflow tests

[x] Automated tests passing locally
[x] Minimal actions workflow (spelling check) passing for PR
[x] Full actions workflow (msbuild) passing for PR

UI tests

[x] Happy path

[x] preview image appears when activated
[x] clicking the preview image moves the mouse cursor to the correct location
[x] right-click dismisses the preview image without moving the mouse
[x] pressing escape dismisses the preview image without moving the mouse

[x] Works on multiple monitors with different dpi scaling settings (e.g. 100% vs 150%)

https://github.com/microsoft/PowerToys/pull/23566#issuecomment-1411869418
https://github.com/microsoft/PowerToys/pull/23566#issuecomment-1412834413

[x] Handling negative coordinates on non-primary monitors if higher or "lefter" than primary monitor

https://github.com/microsoft/PowerToys/pull/23566#issuecomment-1404931694

[x] Mouse crosshair moves when Mouse Jump moves the cursor (mouse clicks and keyboard shortcuts)

#24523
#24527

[x] Activating when the preview window is already visible moves the form to the new mouse position
[x] Number and key shortcuts (1-9, Home / End, Left / Right Arrow) jump to the appropriate monitor
[x] Number-pad shortcuts (1-9 jump to the appropriate monitor

Settings tests

[x] Changing thumbnail size settings updates the size of the thumbnail
[x] Changing preview type between Compact, Bezelled and Custom shows the correct preview type
[x] Changing custom preview settings shows the correct settings
[x] Launching with settings version 1.0 upgrades settings to version 1.1, with "Bezelled" as the default style and the "Custom" settings preconfigured to match "Bezelled"

Lifecycle tests

[x] Starting PowerToys Runner launches MouseJumpUI.exe when enabled, and not when disabled
[x] Enabling / disabling Mouse Jump in settings starts / stops MouseJumpUI.exe
[x] Exiting PowerToys Runner stops MouseJumpUI.exe
[x] Killing runner exe via Task Manager stops MouseJumpUI.exe
[x] Stopping Visual Studio local debug run stops MouseJumpUI.exe

note - runner needs to be in non-admin mode otherwise Visual Studio debugger disconnects at launch

[x] Hotkey and size settings are automatically reloaded when config file is modified from Settings UI
[ ] Hotkey and size settings are automatically reloaded when config file is modified manually (e.g. in notepad) while runner and MouseJumpUI.exe are running

Internal Test Suite

[x] Enable Mouse Jump. Then:
[x] Press the activation shortcut and verify the screens preview appears.
[x] Change activation shortcut and verify that new shortcut triggers Mouse Jump.
[x] Click around the screen preview and ensure that mouse cursor jumped to clicked location.
[x] Reorder screens in Display settings and confirm that Mouse Jump reflects the change and still works correctly.
[x] Change scaling of screens and confirm that Mouse Jump still works correctly.
[ ] Unplug additional monitors and confirm that Mouse Jump still works correctly.
[x] Disable Mouse Jump and verify that the module is not activated when you press the activation shortcut.

I think this PR is ready for review now - I still need to do the full set of regression tests but the happy path works fine.
There's some notes for reviewers in the original post at the top.

Hi, just merged main to check with .NET 9 :) I'll review this PR soon.

I guess there's an overhead in that any future changes to MouseJump.Common will require at least superficial testing if the Settings UI now, but it was the only way I could come up with to get the graphical preview, and that felt like it was a useful addition to the settings controls (although I guess you could change a setting, activate mouse jump, rinse and repeat, and maybe that would have been enough...)


speedup: ifdef snprintf

implementation of
https://github.com/SciTools/conda-recipes-scitools/blob/master/libmo_unpack/speedup.patch
from
https://github.com/SciTools/conda-recipes-scitools/tree/master/libmo_unpack
ifdef debug round the snprintf statements gives a massive straight line speed improvement

Please could you rebase now that #10 is merged.

Thanks for bringing that patch over @marqh :+1:


Failed to parse source map

Checklist

[x] This is not covered in the repository examples.
[x] I am using the package latest version.
[x] This is a bug report or an enhancement proposal.

Description
When building these warnings popout "Failed to parse source map"
Failed to parse source map from '/redacted/node_modules/mui-rte/src/MUIRichTextEditor.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/MUIRichTextEditor.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/Autocomplete.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/Autocomplete.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/Blockquote.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/Blockquote.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/CodeBlock.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/CodeBlock.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/Link.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/Link.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/Media.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/Media.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/Toolbar.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/Toolbar.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/ToolbarButton.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/ToolbarButton.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/components/UrlPopover.tsx' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/components/UrlPopover.tsx'

Failed to parse source map from '/redacted/node_modules/mui-rte/src/utils.ts' file: Error: ENOENT: no such file or directory, open '/redacted/node_modules/mui-rte/src/utils.ts'

I am having the same issue

This bug seems to be happening when react-scripts version used is 5.0.0. You can disable the source map by adding the proper variable in your .env. Check the example here https://stackoverflow.com/questions/51984146/how-to-disable-source-maps-for-react-js-application. I'm going to make a pull request for this repo, to configure the source map properly for future react-scripts versions.

Having the same issue and actively using maps, so disabling it is not an option

Same issue; however there's a pending pull request and tasks are done ? How to make it approve ?

This is still going on.
I think the simple solution is to remove source maps in the deployment package. this way there will not be a reference to non-existent source files.
This is the standard in production packages isn't it? @niuware any chance one of the maintainers can address this issue? it seems simple to solve and as a user of the repo i find it intrusive to my workflow (makes it harder to see if my build has errors in dev mode since there are always errors..)


[Alpha Test] Runic Claymore in left hand looks different

It's looks cool....

For the record, left hand is my main hand!

Whoops. I'll see if I can fix that. I inverted my logic somewhere along the lines.

Oh the Great Axe does this in the left hand as well.

Ah, this was fixed.


godeps: bump diskv to 2.0.0 tag

https://github.com/peterbourgon/diskv/releases/tag/v2.0.0
https://www.youtube.com/watch?v=0WYKmzcEKjg

(also for #1307 via https://github.com/peterbourgon/diskv/issues/28)


Nextra for docs

Source

Reopen on later date, non priority, may not be the choice for the project


handling keyboard interupt and exiting main thread safely

also, re arranged the main method a bit so that it makes more sense

Okay, it's twicefold. It does not do the thing, when you press ctrl+c on the splash process.
However it does work after the splash process is closed.

done required changed in #153


Adding fluent builder withLink for addLink method

It would be very convenient to add links to another containers using fluent builder. Currently addLink method has no fluent alternative. This PR introduces one: withLink.

AFAIK, linking is discouraged and using Docker networking should be preferred after all.

Indeed, Docker recommends to avoid usage of legacy link functionality unless absolutely necessary.
Then I believe there are two logical alternatives: either addLink stays to be used with caution and withLink is introduced for fluent symmetry, or addLink method is deprecated. Does it make sense?

Networking in Testcontainers should be stable enough to deprecate addLink. What do you think @bsideup ?

@kiview sounds good :) Networks are stable, less error-prone and easier to use :)

@rnorth are you fine with it as well? Is addLink the only method using Docker links, or do we have to deprecate multiple methods?

I'm fine with deprecating use of links; this predates the existence of network support in Docker, and I'd agree that we've waited long enough.
There are a few things I'd like to do in the process of deprecating - I'll raise an issue for further tracking/discussion.
@orange-buffalo thank you for raising the PR, but would you be OK with closing based on the above?

@rnorth, sure, I will close the PR.


CORE-2366   shouldRun defaulted false

I found out that system does not work with Derby Client Driver in my project.  3.3.3 seems to work but it was false positive, because maven plugin gave me shouldRun=false and warning ....
I think shouldRun should be true by default.

Thanks!


[5.1] Extract variable usage to getter

This provides an easy way to set dynamic error bag names (without hijacking another method) when you have multiple forms using the same form request on the same page (like create and edit).
Example usage with bag name included in input
public function getErrorBagName()
{
    $this->dontFlash[] = '_error_bag';
    return $this->input('_error_bag', $this->errorBag);
}

Going to hold off on this.


Resolve Open-Redirect / iFrame Injection

This PR aims to resolve any/all open-redirects and iframe-injections within the javadoc for this project

[!NOTE]
While this is a security concern, due to its low-impact nature, a simple PR can suffice.

Astyanax has been retired and is no longer under active development but may receive dependency updates to ease migration away from Astyanax.


Commenting code and then beautifying removes line breaks

Description
I use beautify via the VSCode extension and, at least in scss files (did not try for other types yet), it removes line breaks before commented rules. Sometimes, I just want to comment a few lines, save it (I have format on save turned on), see the result on the web page and then go on. But this behaviour is quite annoying for this scenario
Input
The code looked like this before beautification. Notice that the height property is commented, because I quickly wanted to see the result without it.
        #search-text {
            width: 43%;
            max-width: 270px;
            // height: 100%;
            border: none;
            border-radius: 5px;
            padding-left: 0.5rem;
            font-size: 1.1rem;
            color: $main-color;
        }

Expected Output
Should leave it as it is
Actual Output
The  code actually looked like this after beautification. Notice that the line break before the comment has been removed. This might be wished in some cases, but I think in most cases, this is not desired, because when you start a comment in a newline, you normally do so by intention. You don't want to move that comment up on the previous line, next to the code.
        #search-text {
            width: 43%;
            max-width: 270px; // height: 100%;
            border: none;
            border-radius: 5px;
            padding-left: 0.5rem;
            font-size: 1.1rem;
            color: $main-color;
        }

Steps to Reproduce
Just beautify the code (scss file) provided above. Perhaps it also occurs for other file types, not sure, did not try.
Environment
OS: Kubuntu
IDE: Visual Studio Code
Settings
{
    "indent_size": 4,
    "end_with_newline": true,
    "wrap_line_length": 120,
    "preserve_newlines": true,
    "max_preserve_newlines": 3
}

Fixed in 1.8.0


Update robomongo to 1.0.0,89f24ea

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask's name and version.

Thank you for the contribution. It was merged directly as 03c113dd8b442ee68d8dddd0a08424b847897056. Your contribution is still credited to you.


Update Composer dependencies (2020-05-23-00-09)

Visual regression test passed!
View the visual regression test report


Test teardown sequencing issue with the setupEngine helper.

Since incorporating the fix in https://github.com/ember-engines/ember-engines/issues/875 to properly clean up engine instances, I've run into another issue with some of our engines. The symptoms are tests that fail with one of these two failures:

Error: Assertion Failed: calling set on destroyed object

Error: Can not call .lookup after the owner has been destroyed

These happen when the component under test has a willDestroy or similar hook that attempts to use the engine (e.g. service lookup) or set an attribute on another engine-owned object.
It appears that this is because of how the rendering context is set up using the setupEngine helper, which renders the component under test as an outlet in a view that is owned by the dummy app:
https://github.com/emberjs/ember-test-helpers/blob/master/addon/addon-test-support/%40ember/test-helpers/setup-rendering-context.ts#L260
As a result, with the patch from https://github.com/ember-engines/ember-engines/pull/876, the order in which the objects are destroyed at the end of the test from the destroy(context) callback in teardownContext is:

Destroy engine & it's container.
Destroy application & it's container (which is rendering the test case).

As a result, even though the component being tested isn't destroyed (since it's rendering is managed by the parent application), the engine has already been destroyed, so the component can't do anything requiring the engine container, e.g. looking up services.
I think the right sequence of teardown when an engine is in use would be:

Destroy the testing view (i.e. the component under test).
Destroy the engine.
Destroy the application.

I am able to simulate this by invoking clearRender() in an after hook in my tests, and this does in fact resolve the issue by first un-rendering the component before the engine is destroyed via teardownContext.

Thank you for the detailed explanation @brettburley!
Before reading last sentence, was thinking about using clearRender(), so appears we both think in the same direction.

Ha! I was considering introducing the clearRender() call to the setupRenderingTest helper in ember-qunit as an afterEach hook. However, it seems to trigger failures in many of our test suites that have blur handlers on inputs and end the test with a focused input... Chrome has a long-time bug where the blur event is incorrectly fired on DOM removal: https://issues.chromium.org/issues/41484175, and that triggers the tracked property assertion failure as reported in https://github.com/emberjs/ember.js/issues/19010.
It seems like clearRender() removes the DOM before destroying the component instance. When the view is destroyed through the teardownContext destroy chain, destroy is invoked first on the component which removes the event handlers before the DOM is removed.


Reduce test times

The tests are way faster in release mode - they are more likely to fail flaky tests too I have noticed, which could be considered a boon or a burden.

I think it makes sense to add --release to cargo build and cargo clippy too. We're building everything in release mode anyway, so there's no point building in debug mode for those two commands earlier.

I think that my LOGAF level is low, but the changes by building in release are: less space used on disk, takes significantly longer to compile.
I would say that longer compilation times/longer CI is less optimal.

I think --release everywhere should be objectively faster.
Running cargo test --release requires you to build everything in release mode anyway. Running cargo build (without --release) before this means you will also spend time building everything in debug mode.
In other words (< means is quicker than):

cargo build < cargo build --release
cargo test < cargo test --release
cargo build + cargo test --release > cargo build --release + cargo test --release

(at least as far as I understand)


Feature sign up  page

Finish the Sign-up Page. Add features to sign up with Google and Facebook.

add changes of the sign up UI to the main branch.


[Crash, reproducible]: Resize type Ratio

Version: 2.1.3 on Android 12
Feature: Batch resize
Use case:

Import 2 or more Images
set Resize type to Ratio
Enter new value at data field Height

Observed result (1):
The display for Height shows other values then typed.
Observed result (2):
Trying to correct the values for Height will crash the app, see log below.
java.lang.OutOfMemoryError
java.lang.OutOfMemoryError
at android.graphics.Bitmap.nativeCreate(Native Method)
at android.graphics.Bitmap.createBitmap(Bitmap.java:1104)
at android.graphics.Bitmap.createBitmap(Bitmap.java:934)
at android.graphics.Bitmap.createScaledBitmap(Bitmap.java:789)
at q9.r.t(SourceFile:135)
at q9.j.j(SourceFile:89)
at q9.j.a0(SourceFile:13)
at j5.e.F1(SourceFile:5)
at j5.e.S1(SourceFile:74)
at q9.r.s(SourceFile:27)
at g8.g.j(SourceFile:68)
at k6.a.n(Unknown Source:8)
at c7.c0.run(Unknown Source:109)
at k.j.run(SourceFile:253)
at i7.i.run(Unknown Source:2)
at i7.a.run(SourceFile:94)
Suppressed: h7.e: [m1{Cancelling}@dc43d24, Dispatchers.Main.immediate]

You're trying to make it too large to be displayed on your device

Oh yes, that's due to the root cause described as "Observed results (1)" - the Height input value can't be edited correctly in this case (as it is for the Width value). To be more precise: the original value at the input field can't be deleted or overwritten. For example: if one or more digits are deleted with the backspace key, the original value pops up again. Only if a Preset is choosen, the Width and Height fields are updated. Adding additional digits to this entry will crash the app.
On the other hand side: if there are limits for input values, its better to let the app care for it. Or the input fields has to be locked for user input if it is the intended use case that valid inputs are only allowed from the Presets.

Fixed

Wait for the 2.2


Send mails with custom headers with firestore-send-email extension

Extension name
firestore-send-email
What feature would you like to see?
the ability to add custom header in a mail
How would you use it?
We use Mandrill API to send emails and we can add some custom metadata  to every message.
We can also add metadata sending email with Mandrill smtp but we need to pass them in the X-MC-Metadata header.
It would be great if we can specify in each message and at template level some headers to include in the message

Thanks for the feature request. I'll pick this up internally & figure out how/if this can be integrated.

Hi
Would love to have this feature. Am using SES but can't even enable basic SES open and click tracking without custom header.

This looks like it's covered by #108


[$500] Desktop - Chat - Infinite loading displayed on selected chat when navigate to RHP with deeplink

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.13-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:
Precondition: user should be Signed Out on Desktop

Open http://staging.new.expensify.com/
in Web and select any conversation in LHN
Navigate on http://staging.new.expensify.com/enable-payments
Click on "Allow" to open the Desktop app
Click outside of the RHP to close it

Expected Result:
Previously selected chat should be loaded and properly displayed
Actual Result:
Infinite loading displayed on selected conversation when navigate to RHP with deeplink
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/83c2c43e-2b6e-46a0-bb5f-a0ecfcf32538

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ae306e7095356be1
Upwork Job ID:<PHONE_NUMBER>656286720
Last Price Increase: 2023-12-15

Proposal
Please re-state the problem that we are trying to solve in this issue
Infinite loading displayed on selected conversation when navigate to RHP with deeplink to desktop app.
What is the root cause of that problem?
The reason why we get the infinite loading spinner on local development desktop is because the desktop app is calling navigate with transition route and this action is not handled by the navigator.
The way the flow was designed to work for the desktop app:

user can be authenticated or not
user attempts transition from web to desktop app
web app prompts user to open URL in desktop app
upon opening,  web sends transition route to desktop app with exitTo, email and shortLivedAuthToken to re-authenticate the user then redirect to exitTo route at the end of all this

Important: transition route is used to deeplink web to desktop in this case, it was not designed to actually be called by the navigator once we're in the desktop app.
What changes do you think we should make in order to solve the problem?
https://github.com/Expensify/App/blob/173582ff506202ca32671cc0a7acea1f440dddb0/src/libs/actions/Report.ts#L2031
We have to make sure that the transition route passed from the main electron process is NOT called by the navigator once we are transitioned into the desktop app, for this we check:

the platform is desktop
route is transition type

if these conditions are met then we return meaning we not call the react navigation with the transition route which causes the issue on staging / local desktop app.
...

                // If the platform is desktop and route is `transition` we don't want to navigate
                // to the route because it will display the infinite loading indicator.
                // See issue: https://github.com/Expensify/App/issues/33149
+               if (getPlatform() === CONST.PLATFORM.DESKTOP && route.includes(ROUTES.TRANSITION_BETWEEN_APPS) {
+                   return;
+               }

             Navigation.navigate(route, CONST.NAVIGATION.TYPE.PUSH);

Videos

MacOS: Desktop (staging / local)

staging

https://github.com/Expensify/App/assets/56457735/5428f539-2b24-4149-8893-7516173b95c6

local dev

https://github.com/Expensify/App/assets/56457735/8bc8be14-30b2-41de-bc61-bf3811926968

Proposal from @ikevin127 looks good to me https://github.com/Expensify/App/issues/33149#issuecomment-1857799980
🎀 👀 🎀 C+ reviewed

@narefyev91 PR ready for review https://github.com/Expensify/App/pull/33225 🚀

Payouts due:
Issue Reporter: NA
Contributor: NA- paid as external contributor through Callstack
Contributor+: $500 @ikevin127 (paid through Upwork)
Eligible for 50% #urgency bonus? N
Upwork job is here.
@narefyev91 can you let us know about a regression test?

The PR that introduced the bug has been identified. Link to the PR - it was not a regression - just missing functionality
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment - N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion - N/A
Determine if we should create a regression test for this bug - N/A

sounds good! I think we can close this!


add support for "simple" projection

Inspiration from https://leafletjs.com/examples/crs-simple/crs-simple.html

done with https://github.com/geotiff/geotiff-tile#image-pixel-coordinates-and-simple-srs


slot template with scope value not working

Version
2.6.11
Reproduction link
https://jsfiddle.net/L514wjxn/
Steps to reproduce
@see reproduction link
What is expected?
show slot
What is actually happening?
not working

even el-input prepend slot not provide scope value, it should working .
ex: if i create a middle component, i don't know if sub component provide scope value

Please provide a boiled down reproduction, which means without 3rd party components like Element. Or you can report the problem to Element itself.

@Justineo   it's so easy to reproduction
https://jsfiddle.net/7kfarwus/1/

$slots does not include scoped slots. Please use $scopedSlots instead.
BTW notice that since 2.6 we also expose all normal slots in $scopedSlots.


Oauth updates

oauth: pass OAuthConfig as parameter to Convert
Rename Generate to Convert

@arapov, ah names choice! ;)
I agree Generate was to be replaced, I already applied Translate, although Convert is a good candidate. Let's @alexander-demichev vote!
Otherwise LGTM

Please, rebase as I moved oauth and so into ocp4.

Rebased. I will look into your code from last evening during today.


Add new resources

Hi! I'd like to add new interesting resources about patterns, fonts and icons. Please give the chance 😄

Of course, the website is still in development and may change some things, but you can contribute new resources, see CONTRIBUTING.md to see how to do it, if you have any questions I will be happy to answer them 😁.


Add support for attaching to existing console applications

Description of the new feature/enhancement
I migrated to Windows Terminal from ConEmu and one feature I miss is its "Attach to" function that captures an existing console window and moves it into the terminal as a tab. It's useful for managing windows that weren't — or more importantly, can't be — opened in the terminal.
Example 1: The launcher for the LynxOS development environment opens a bash console in the appropriate working directory and runs some setup commands it. I'd like to be able to move that console into Windows Terminal. (Replicating this via a WT profile is only partially possible, and even that much required a fair bit of sleuthing around to figure out.)
Example 2: Eclipse-based GUI applications can be launched with the console visible. Being able to move that console into WT would allow me to declutter my workspace.
I don't know if setting Windows Terminal as the default console host would alleviate these use cases. It might, but that feature is only available on Windows 11. This feature should work on any OS.

Proposed technical implementation details (optional)
The context menu for the title bar should have an "attach to" (or similarly named) option that opens a dialog of existing windows that may be captured and moved into Windows Terminal.

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #11387 for more discussion.
/dup #11387


Can't clone Amazon EC2 machine/host

v0.21.1
Steps:

Create a Amazon EC2 machine/host
Click on Clone in Amazon EC2 machine/host submenu
Click on Next: Authenticate & select a network

Results:
There is no Next: Select a Security Group button so I can go to the next step
Expected:
Should be able to go to next step in process

v0.21.3
Verified fixed


Donation widget/stripe addition

What:
Add integration with stripe while refactor the widget in order to:

Change the payment form to be last step
Increase the responsibility of each form on the widget
Update sendDonation service to meet membership application contract

Why:
Relates to the work for Develop donation widget
How:

Integrate @stripe/react-stripe-js and @stripe/stripe-js to handle card handling
Distribute responsibility of form manipulation to each step on the wizard so the parent component only acts as orchestrator

References:
As the pull request is a bit long, I think could be useful to illustrate the Widget architecture and somehow share some context of the code you are about to see.

Media:
Walkthrough the donation current flow

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


Upgrade action to use node16

Github will deprecate node 12 and all actions will run on node 16
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

It's done @simonbaudart version: v1.1.0 thanks for your help!


unused variables warning coming on npm start

Line 60:20:  clauseId is assigned a value but never used    no-unused-vars
Line 60:29:  templateId is assigned a value but never used  no-unused-vars

I am working on this issue

Also these were not defined in state but were extracted later in the code


Adding configmaps

when I deploy the app (using fmp), I am see the config map being created.
But, it seems to not directly create the topic
oc rsh my-cluster-kafka-0

cd /var/lib/kafka/kafka-log0
ls -la

Gives me:
-rw-r--r--. 1<PHONE_NUMBER> root    0 May  8 04:07 cleaner-offset-checkpoint
-rw-r--r--. 1<PHONE_NUMBER> root    4 May  8 04:07 log-start-offset-checkpoint
-rw-r--r--. 1<PHONE_NUMBER> root   54 May  8 04:07 meta.properties
drwxr-xr-x. 2<PHONE_NUMBER> root 4096 May  8 04:07 my-topic-0
-rw-r--r--. 1<PHONE_NUMBER> root   17 May  8 04:07 recovery-point-offset-checkpoint
-rw-r--r--. 1<PHONE_NUMBER> root   17 May  8 04:08 replication-offset-checkpoint

Now, when I execute the app, data is written to the topic (name resolved via ENV_VAR), I notice that the content of the ConfigMap gets overridden (or created by Strimzi, since it might think it's not there?)
oc get configmap fail.rate -o yaml
apiVersion: v1
data:
  config: '{"segment.bytes":"1073741824"}'
  name: fail.rate
  partitions: "1"
  replicas: "1"
kind: ConfigMap
metadata:
  creationTimestamp: 2018-05-08T04:09:28Z
  labels:
    strimzi.io/cluster: my-cluster
    strimzi.io/kind: topic
  name: fail.rate
  namespace: balh
  resourceVersion: "6266"
  selfLink: /api/v1/namespaces/balh/configmaps/fail.rate
  uid: 9a2b51b5-5275-11e8-b4a3-c85b765a6fb1

Now, when I edit this (e.g. 3 partitions), I immediately see the result

@scholzj any clue ?


Make the template option -v accept 4 templates

The new option -v will accept 4 template engines and error on the unknown input. #117 #92  @dougwilson

This has finally made it into the pending 4.14 changes, without the removal of the existing switches to make it easier to transition. Thanks again for all your hard work!


False positive on cronjob-backed pods

Found interesting thing:
I have cronjob (renovate) running inside k8s cluster. Since pod starts, performs job, and ends, it seems like coroot detects it memory usage grouping all pods as "renovate-github" in my case and reports memory leak. Screenshot attached.
Looks like it's aggregating memory usage on each cronjob pod run?

One thing that comes to mind is to look into metadata of pod, particulary this part:
metadata:
  ownerReferences:
    - apiVersion: batch/v1
      kind: Job

Which tells this is pod fired from job

It seems that the best solution here would be to skip this inspection for CronJobs:

There is a high probability of false positives due to the fact that the task launch process may appear as a leak.
Even a real leak in a short-lived process is not a problem.

This is fixed in v0.14.4


Command line parsing

Proper command line parsing that sets up the encoder chain.
For example: pencode b64 hex somethingelse which would read payload from stdin, base64 encode it, then hex encode the result and "somethingelse" encode  it before outputting the result to stdout.

This has been handled in the initial commit


Lazy loading

⚠ TODO: Wait for https://github.com/neo-technology/neo4j/pull/28382 to get merged and run tests to verify it works.

https://trello.com/c/4Xa94qr4

Kernel decided to not merge https://github.com/neo-technology/neo4j/pull/28382 and go for another solution instead, so closing this too.


CD workflow cancelled when merging a deployment PRs when another workflow job is pending start

We have specified a concurrency group of "deploy" for the depliyment workflow. It seems like that, we get it to run in a sequence if multiple workflows are scheduled to run, which is great!
The downside: this also comes with a behavior of cancelling pending jobs. So if we for example merge deploy PRs A, then B, then C, we likeley get A started, B pending, then B cancelled because C was merged while B was pending. This is trouble if C deploys anything less than B, but is otherwise fine.
So, the failure mode is quite rare, and will only happen if multiple conditions are met:

Merge 3 or more PRs
The cancelled pending PR(s) were deploying to more clusters than the last not cancelled PR

Here is how it can look when a workflow is pending and cancelled.

Where I also note that a push to another branch triggered the deploy workflow, which triggered cancellation of a pending PR without even being merged. And this happened even if...
push:
    branches:
      - master

There is a discussion about this topic in https://github.com/orgs/community/discussions/41518#discussioncomment-4833436, and it seems github doesn't want to provide a feature to opt-in to not cancel pending workflows.
Action points

Put a thumbs up emoji if you now have become aware of this possible way for our CI/CD system to fail to deploy something.
Figure out what if anything to do about this

I think this was the same as #3214.


Add Tower to Operator

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #328
Special notes for your reviewer:
Documentation:

Does this PR introduce a user-facing change?:

NONE

/hold

/lgtm

/unhold


permission to reference your work.

Gday Damo. Your work is amazing. I've got some work i want to share with the breezejs guys, and I'm using angular2 with code referenced from you . Just wanted to check if it's ok to share your identityserver4 angular2 setup? All credit will be given to yourself of couse.

of course, delighted you can use it, thanks
Greetings Damien

Thanks Damo appreciate that!
Just for my info, did you ever build in a 'token expiring' function to automatically renew tokens ? Just asking in case i need to program this .
cheers

No, need to implement this, I'm planning on implementing the OpenID session management next.
Greetings Damien

Oh cool that sounds really good.
Ill have a crack at tokenexpiring at my end and let u know how i go
Look forward to seeing how u go with openid session management
cheers


The example/warpctc/ocr_predict.py  has a bug,  it does not provide init_states values and run the ocr_predict.py several times and will get different results.

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System:ubuntu
Compiler: gcc
Package used (Python/R/Scala/Julia):
MXNet version:
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.
1.Run the predict.py several time and get different results.
2.It does not provide init_states values for the lstm .
3.
What have you tried to solve it?
1.The init_states values should be provided.
2.first,  init_states could be a var of class.
little changes in  def __init_ocr(self):
self.init_states = init_c + init_h
all_shapes = [('data', (batch_size, 41 * 64))] + self.init_states + [('label', (batch_size, num_label))]
#print (all_shapes)
3.  Provide the init_state values
#init_state_arrays = [mx.nd.zeros(x[1]) for x in self.init_states]
init_state_arrays = np.zeros((batch_size, num_hidden), dtype="float32")
init_state_dict={}
for x in self.init_states:
init_state_dict[x[0]] = init_state_arrays
self.predictor.forward(data= img1, **init_state_dict)

I think I have the similar problem with a cnn-lstm-ctc task when using predict API to infer the results after training. The accuracy does not match when using the same dataset and checkpoint in and training.

@xinghedyc To solve it, you just provide the init_states values which are zeros.

@lji72 Thank you so much, it works!

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Fix change filter on optional components

With the new optional components queries (TryRead and TryWrite) it will be useful to refer to this optional components while filtering by change. For now situation when optional component is missing on entity, but present in the filter results in panic. Not quite sure with the actual implementation though.

That build failure was something broken in master, which has now been fixed. Can you try merging master into this branch and pushing that up?

Done


Change MSGraph version beta to v1.0 for todo-list-set #2518

Enhancement: todo list set

Thank you! We'll review it asap

Merged manually. Thank you! 👏


Support DATE values for UNTIL parts of recurrence rule

Support DATE values for UNTIL parts of recurrence rule e.g. RRULE:FREQ=DAILY;UNTIL=20220904
Issue #70
Issue https://github.com/home-assistant/core/issues/81500
Issue https://github.com/home-assistant/core/issues/81568

Hi, I checked the log with the following error, hope you can help me (I see the error related to recurrence rule):
ValueError: Invalid recurrence rule: {"id": "xxx", "ical_uuid": null, "summary": "xxx", "start": {"date": null, "date_time": "2023-03-17T08:00:00+07:00", "timezone": "Asia/Bangkok"}, "end": {"date": null, "date_time": "2023-03-17T09:00:00+07:00", "timezone": "Asia/Bangkok"}, "description": null, "location": null, "transparency": "opaque", "status": "confirmed", "event_type": "default", "visibility": "default", "attendees": [], "attendees_omitted": false, "recurrence": ["RDATE;TZID=Asia/Bangkok:20230317T080000,20240404T080000,20250325T080000,20260413T080000,20270402T080000,20280321T080000,20290409T080000,20300329T080000,20310318T080000,20320405T080000,20330326T080000,20340414T080000,20350404T080000,20360323T080000,20370411T080000,20380331T080000,20390320T080000,20400407T080000,20410327T080000,20420317T080000,20430405T080000,20440325T080000,20450413T080000,20460402T080000,20470322T080000,20480408T080000,20490329T080000,20500318T080000,20510406T080000,20520326T080000,20530414T080000,20540403T080000,20550323T080000,20560410T080000,20570330T080000,20580320T080000,20590408T080000,20600327T080000,20610317T080000,20620405T080000,20630325T080000,20640412T080000,20650401T080000,20660321T080000,20670409T080000,20680329T080000,20690318T080000,20700406T080000,20710327T080000,20720414T080000,20730403T080000,20740323T080000,20750411T080000,20760330T080000,20770320T080000,20780408T080000,20790328T080000,20800317T080000,20810404T080000,20820324T080000,20830412T080000,20840401T080000,20850321T080000,20860409T080000,20870330T080000,20880318T080000,20890406T080000,20900326T080000,20910414T080000,20920402T080000,20930322T080000,20940410T080000,20950331T080000,20960320T080000,20970408T080000,20980328T080000,20990317T080000,21000405T080000"], "recurring_event_id": null, "original_start_time": null}: unsupported RDATE parm: TZID=ASIA/BANGKOK

Hi,
You're coming from home assistant? Then you should file a home assistant issue.. but from what you said, this PR should be fixing the issue in the home-assistant patch release. Commenting on the fix in another library isn't really the right way to discuss your issue, so keep your report within home assistant package please.


Add toolbar

The toolbar should contain profile and notifications

Closed as fixed, please if you notice that the issue still exists open it again.


Use std::uniform_int_distribution

https://github.com/nploi/Stack_game/blob/ccf49b74abd1443e78d83322e959b99f2fd5aef7/src/util.cpp#L48

Thank you !! I will improve it.

Fixed


Error for events on weave-gitops HelmRelease page

Describe the bug
When user navigates from Applications Page to weave-gitops object subpage and looks for events, the page displays error message instead of event list.
Screenshots

Is this a UI bug or a server bug?

[x] UI
[ ] Server

What is the severity of the bug

[x] severity/Critical: Weave GitOps is crashing or experiencing data loss, the UI is inaccessible or a key feature is unusable. There is no known workaround
[ ] severity/Major: Weave Gitops functionality is broken, there is a workaround, but the workaround requires significant effort
[ ] severity/Minor: Weave Gitops functionality is broken, but there is a fairly straightforward workaround
[ ] severity/Low: Doesn’t affect primary flow/functionality but would be good to fix

Environment

gitops: main
How you deployed the Weave GitOps server: Tilt
kubernetes:  [e.g. 1.20.4]

[x] KinD - version]
[ ] k3s - version
[ ] cloud [e.g., EKS, AKS]  version
[ ] other - name version

To Reproduce

Go to Applications page
Click on weave-gitops HelmRelease
Navigate to subpage and click on events
See error

Expected behavior
Event list should appear just like for any other Applications object.

I think it's fixed with https://github.com/weaveworks/weave-gitops/pull/2033. That PR fixed the EventTable that's used everywhere.
@rokshana-b : Can you verify that please?

I just tested this but it seems it is still a problem, I'll take a look

NVM it seem, this is working properly

Looks like we could not reproduce. Closing.

Duplicate of #2017


Validate alertmanager URL

Fixes #1455.
Why is URL validation so hard in golang ...
@discordianfish @fabxc

:+1: nice

I assumed we want to be strict and not nice about the the scheme.

Yay :rocket: :) That was quick. LGTM!


validateDOMNesting when using React + Mjml

Hi,
I think it's more a problem with React than Mjml itself. But caused indirectly by mjml. I think more people can facing with this same problem.
I have React components that generate pure mjml (using renderToStaticMarkup), so it is parsed with mjml and saved into a file.
Example:
EmailConfirmation.jsx
=>
email-confirmation.mjml
=>
email-confirmation.html

That's ok. Working. But I have some warning/errors generated by react, because it renders mj-table > tr:
console.error node_modules/fbjs/lib/warning.js:36
      Warning: validateDOMNesting(...):
 cannot appear as a child of <mj-table>. See AddressTable > mj-table > tr.

Is there any way to avoid (or disable) validateDOMNesting in this cases?

Hi @renatorib !
Can you share the content of this file ?
Beware that MJML components are not only simple React component, so you'll need to apply post render to the whole document

Hi @renatorib any news on that ?

Hi @renatorib , i'm closing this issue for now, feel free to reopen if you can provide more information about the content of" email-confirmation" file

Hello @iRyusa, sorry for delay.
It's about React, not mjml itself.
I use React to render to mjml with renderToString, and after, render mjml to html.
import React from 'react';
import ReactDOMServer from 'react-dom/server';
import { mjml2html } from 'mjml';

const Email = () => (
  <mj-table>

tr as a child of mj-table

  </mj-table>
);

const mjmlString = ReactDOMServer.renderToStaticMarkup(
  <Email />,
);
/*
console.error node_modules/fbjs/lib/warning.js:36
      Warning: validateDOMNesting(...):
 cannot appear as a child of <mj-table>. See Email > mj-table > tr.
*/

const html = mjml2html(mjmlString).html;

export default html;

Hey @renatorib
You need to add a tbody to avoid JSX warning ! Note that we gonna totally drop React from MJML stack in a near future, see : https://medium.com/mjml-making-responsive-email-easy/hey-were-entirely-rebuilding-mjml-from-scratch-announcing-mjml-v4-ae92f83b725b#.xig1gdoc6

Haha, no problem!
Nice, I did not know it was possible to import mjml components directly.
Thanks!


Add direction for what to do after farmer has started

We should add an additional step at the end of the documentation to show where users can check their nodes and such.
Add Telemetry Link
Add Explorer Link
Add subscan link
just to prompt the user to know how to check their farmer.

I recommend closing this issue, as these resources are included in the Tips & Tricks section under Advanced CLI. If that is not suitable, I can add these resources as an admonition in a more appropriate location.


J

Hi! I will close this issue as I did not see any issue here, let me know if there is any additional issue.


[ Feature Request ] Windows support

The installation script works just fine in MSYS2, which is a great step forward, but pack itself fails with the message "[ fatal ] Failed to figure out package directory."
The problem seems to be the filepath library, which only supports Linux-style paths when parsing.
Even in MSYS2, environment variables are unfortunately separated by "\", so I believe there's currently no workaround to get pack working.
Is Windows support something we would like to pursue? If so, what would be a good first step? Modifying filepath, adding a new cmd argument to pack, or other solution?

Sorry for the late reply. Obviously, I'm not a Windows users, so I won't myself be able to test / maintain Windows support. This means that if Windows support breaks and nobody is around to fix it, I'll have to drop it. But I'll definitely accept contributions in this regard.
Currently, I think there are two issues that need to be addressed: First, as you correctly wrote, the idris2-filepath library currently only supports Unix-style file paths. Contributions there are very welcome. Second, many IO operations in Pack.Core.IO (and possibly in Pack.Core.Git) are implemented as syscalls to typical Unix utilities. These would need to be changed in such a way that the syscalls being made work correctly on Windows (if we are on a Windows system).
Finally, I'll only accept contributions together with an update to the docs and an inclusion in our CI tests.

For anyone else struggling on Windows: I solved it by using docker and base image  ghcr.io/stefan-hoeck/idris2-pack. I then created my own image where I installed neovim and let pack install idris2-lsp.


svg generation is non-determinstic

Under the hood, mon-artist represents SVG with xml elements from the treexml crate.
However, it seems like from run-to-run the rendering of these elements can change even when the input XML text has not changed. This is probably due to use of data structures with unspecified orderings (e.g. a hash table).
I should look into either making wrapper type around treexml where the rendering is stable from run-to-run, or switch to another xml library that has that guarantee.

See https://github.com/pnkfelix/presentations/commit/9227c0697e8ceb6f83659fc0f7f51aa6a6a6ef3b for an example of where this problem has bitten me.


Search by storage id lasts freacking long on my company account

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.31.1
Regression From
1.30
Architecture
x64
Storage Explorer Build Number
20230822.1
Platform
Windows
OS Version
10
Bug Description
Search by storage id lasts freacking long on my company account
it was much better several updates go but now it's veeeerrry long
Steps to Reproduce

Start app
In filter check only search by storage
input storage id and press enter to start searching
go to kitchen to drink coffee for about 5 minutes

Actual Experience
take too much time
Expected Experience
No response
Additional Context
No response

@Kotsiubynskyi You can right click the subscription and select "search from here" to scope the search in that subscription and reduce the unnecessary scanning. How many storage accounts do you have? Does it eventually give you the results or does it seem to hang forever?

I have 10+ subscriptions
Thanks, your advice helped but not much. The search under one subscription takes now 3+ minutes instead of 5 for all subscriptions. I still do not think it's acceptable assuming that on Azure Portal the storage can be found within 2 seconds. What's wrong with this desktop app? Why it differs so much in performance comparing to Portal?
I see 624 storage accounts under the one subscription. Possibly there are which are not visible for me.

@Kotsiubynskyi

on Azure Portal the storage can be found within 2 seconds. What's wrong with this desktop app?

The Azure portal search experience uses a different API than Storage Explorer, at the trade off of being able to do/not do certain things. We looked at swapping to it in the past, but given the cost we decided to first try making it easier for users to filter down what they want to search in an attempt to make Storage Explorer tree view search faster by making it so Storage Explorer doesn't waste time searching for things you don't care about

I see 624 storage accounts under the one subscription.

Just to clarify, you are saying that under the subscription you are searching you have only 624 storage accounts total, correct? If so, then the 3 minutes performance is bad. We need to fix that. That's so few storage accounts that we should be able to search all of them in effectively one API call. We will look into this further.

Yes, please. The is really a very bad experience that you know the storage account, but you cannot locate it in the Explorer.
In my cases, it takes about 5~ mins to search for an account in a subs with total 650 accounts.

We have identified a very possible cause of the perf regression and we have fixed it.


Add missing java api for StreamTestKit

Motivation:
Add more java native api to StreamTestKit.
refs:https://github.com/apache/incubator-pekko/issues/970
was: https://github.com/apache/incubator-pekko/pull/1049
continue the work of @naosense

@pjfanning @Roiocam WDYT, should it be included in m1 release？

Lgtm

Thanks


New Link for Your Awesome List!

Dear William S. Vincent,
Hello!
I just added a new link to my course The Complete Guide To Django REST Framework and Vue JS.
The course is on Udemy, but as you can see the reviews are all very high and it is also included in the Udemy For Business Library, being used by some companies to teach their employees how to use Django REST Framework and how to combine this with Vue.
I added the referral link so that everybody can enroll for the minimum price possible.
The code used in the course is available on my GitHub profile in two repos, here and here
It's over 13 hourse long, and as you can see from the course curriculum I tried to explain DRF to an audience as broad as possibile without sacrificing any topic or leaving anybody behind (and even though you should know some Django before enrolling, I also included a "pure" Django refresher!)
The course also includes an introduction to Vue JS and in the Final Section, I show how to build a SPA using DRF in the backend and Vue (w/ CLI) in the frontend.
Hope this can be useful... and worth being included in your awesome Django list!
Happy Coding!
Michele

Hi @pymike00 , Thanks for this PR and sorry for the delayed response on my part. This looks like a great course. I'm gonna to check it out personally. However, for now, we've removed links to video stuff on the repo. It's hard for me and @webology to curate video stuff. May change in the future but for now that's the decision to try to keep this list curated and small. I hope your course is doing well!


Get mentioned users of message

Description
Hey out there,
is there a way for me to get all the users mentioned (either by user group, directly, or with all f.e.) as a list from a message? Couldn't find anything in the new 2.0 docs.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

@khayalan-mathew 👋 You can check if the mentioning part like <!subteam^S12345678|@user-group-name> exists in text property in app.event listeners.
The following are the things you do for it.

On Slack app configuration page (https://api.slack.com/apps/{your app id})

Enable the Event Subscriptions feature
Subscribe to message events

Add app.event or app.message listeners to your Bolt apps

@khayalan-mathew Hello, was the above comment helpful? Except for https://api.slack.com/events/app_mention,  There is no specific feature to instantly get a list of mentions in a message so that your app needs to search <...> parts in a text. If you have something further to ask here, let us know.

I believe I've already answered the question here. Allow me to close this issue now. Please feel free to reopen if you have something further.


doc add agg_state in data-model

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall


MeshInstance3D AABB doesn't return merged AABB dimensions, nor is it affected by any transformations

Tested versions
Godot 4.2 .Net, Godot 4.3 RC1 .Net
System information
Win10 x64 22H2
Issue description
Merging multiple MeshInstance3D's AABBs does not return the dimensions of the combined AABB.
Also, AABB does NOT seem to be affected by any transformations.
Steps to reproduce

Place a standard MeshInstance3D Cube in the hierarchy. Leave cube size as default (1 x 1 x 1).

Place another standard MeshInstance3D Cube in the hierarchy. Leave cube size as default (1 x 1 x 1).

Move the second cube along the Global X-axis by 5 units. It can be any axis and any amount, but this is for testing.

Realize that the AABB which encloses both cubes SHOULD have an X-axis length of AT LEAST 5 units. Exact value should be 6 units.

Perform AABB mergedAABB = Cube1.GetAABB().Merge( Cube2.GetAABB() )

Perform GD.Print( mergedAABB .GetLongestAxisSize() )

See that the output is always 1. You can output AABB origin and dimensions as well with GD.Print( mergedAABB ). Observe that the values don't make sense.

Transform any cube by scaling/rotating it, then return its AABB. Observe that the AABB does not take into account any transformations.

Minimal reproduction project (MRP)
aabb-issue.zip

The AABB returned by MeshInstance3D::get_aabb returns the AABB of the mesh resource which doesn't have information about the transform of the Mesh. In other words, the AABB is in the local space of the mesh and needs to be transformed by the MeshIntance3D's transform.
This fact should be documented in VisualInstance:get_aabb (after checking that the same is true for all other classes that inherit that method)

Maybe I'm confused, but if the AABB is defined in mesh local-space, then it sounds like it's actually OOBB?
https://en.wikipedia.org/wiki/Bounding_volume#Common_types
https://www.youtube.com/watch?v=HYO5Pthe3TE

Maybe I'm confused, but if the AABB is defined in mesh local-space, then it sounds like it's actually OOBB?
https://en.wikipedia.org/wiki/Bounding_volume#Common_types https://www.youtube.com/watch?v=HYO5Pthe3TE

You could call it that. It depends on your frame of reference.

Ok thanks, I have opened a Godot suggestion!


Add Corvus -1.25m Two Kerbal Pod from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of OrionKermin, to add Corvus -1.25m Two Kerbal Pod to CKAN.
Please direct questions about this pull request to OrionKermin.

Also, the homepage is not listed directly on KS in the metadata, but in the description instead. http://forum.kerbalspaceprogram.com/threads/121062

I'm not sure how to handle either the licence or the install for this one, so I'll leave it for more experienced NetKANers.

License should indeed be "restricted" and I'm not sure how we should resolve the HGR problem. It has been requested that we add it here so I guess the correct course of action would be to add HGR before this mod is added and then set it as a dependency for this.
Adding a blocked-on-other for now until the depedency can be fulfilled.

It doesn't really need the whole of HGR, though. The reason HGR is held up doesn't apply to the limited parts this mod wants to use.

Could we make a "provides/conflicts" pack that can be also provided by full HGR when it is sorted out?

We could have but I much prefer https://github.com/KSP-CKAN/CKAN-meta/pull/540 :)


Legend labels pointStyle for pie partially not working?

Expected Behavior
I should be able to define all the point styles defined here https://www.chartjs.org/docs/latest/configuration/elements.html#point-styles
Current Behavior
cross, crossRot, dash, line, star point styles are not working. Nothing is displayed. Am I doing something wrong?

circle, rect, rectRounded, rectRot, triangle point styles are working correctly instead.

Steps to Reproduce

https://codesandbox.io/s/red-lake-16cx6?file=/index.html
Context

Related to #9337 / #9341
Environment

Chart.js version: 3.4.1
Browser name and version: 91.0.4472.124

It seems the cross is getting it styling from the borderColor and borderWidth propertys in the dataset, so when you set the borderColor the crosses will show and when you set the borderWidth it defines how big they are, downside is that you will get borders in your pie chart.
https://codesandbox.io/s/upbeat-smoke-f1221?file=/index.html

To work around the borders in the chart problem, one could override the generateLabels config option with something like this
const labelOpts = {
  generateLabels: (chart) => {
    const items = Chart.overrides.pie.plugins.legend.labels.generateLabels(chart);

    for (const item of items) {
      item.strokeStyle = item.fillStyle;
      item.lineWidth = 1;
    }

    return items;
  }
}

Thanks for the tips 👍🏻 , so as far as I've understood those point styles are actually being drawn but they have a size / color which doesn't make them show, correct?.

That's correct

Perfect, thank you, closing the issue then.


Can I use a custom font in version 3.2?

I saw in the official document that the current version 3.2 supports several fonts. Can custom fonts be supported currently? Or is there a way that I can replace the font? Because I want to show math formulas to kids, and wish the font cute.
Looking forward to your reply.

No, MathJax 3.2 does not support multiple fonts.  That has been added for v4.0, which is currently in beta release, and includes support for 11 fonts.  None of them would likely be termed "cute".  in order to use a font, MathJax needs extensive data about the font metrics, character bounding boxes, and so on, and this data needs to be made available in data files for MathJax.  The tools for making these file will be made available after the v4.0 release, once they are cleaned up and documented.  So after that, you would be able to make the data needed for your own font, but that will not be available for a while.

Any update on how the tool is coming along to generate font data files? We'd love to get access to it!

Unfortunately, v4 is still in beta release.  We are in the code cleanup phase (no new features), and updating the documentation.  Once we get the v4 release officially out, work can begin on making the font tools ready for public use.


boot2docker 19.03 image does not automount Users directory w/ Virtualbox 5.x

The symptom is the same as #1395 , where user directory is not mounted after booting the image. However the underlying cause is not the same.
#1395 indicates a change of VBoxControl automount option, but for this issue, no automount option is involved, and I'm using Docker Toolbox + Virtualbox 5.2.26 under Windows 7.
Reproduction steps

docker-machine start default
ssh into the boot2docker VM
sudo and obtain root privilege
VBoxControl --nologo sharedfolder list

Error output
root@default:~# VBoxControl --nologo sharedfolder list
Shared Folder mappings (2):

VBoxControl: error: Error while getting the shared folder name for root node = 0, rc = VERR_NOT_IMPLEMENTED
VBoxControl: error: Error while getting the shared folder name for root node = 1, rc = VERR_NOT_IMPLEMENTED

I'm aware of recent change in #1396, so removed the --automount option to emphasize it's unrelated.
https://github.com/boot2docker/boot2docker/commit/b5def189c5a5b577387090671651f5dee9576fde#diff-143faf222471f00b186177a1a20ebe0b
Besides, there are 2 combinations where such error would be resolved:

Downgrade boot2docker.iso to 18.09, and use it with Virtualbox 5.2.26
Keep boot2docker.iso 19.03, and upgrade Virtualbox to 6.0.10 (haven't tested earlier 6.0.x versions)

Is this desired behavior? If yes, is there any need to really stress (and document the situation) that boot2docker.iso and Virtualbox versions must match?

BTW, I have added extra shared folder to this default VM using Virtualbox GUI, so there are 2 shared folder entries. With default setting there should only be one.

I suspect I am experiencing a symptom of this problem: I'm no longer able to mount Docker volumes on Windows when running Docker via docker-machine, unless I explicitly specify --virtualbox-boot2docker-url https://github.com/boot2docker/boot2docker/releases/download/v18.09.8/boot2docker.iso when creating the VM.
I too am running VirtualBox 5.2.

Gah, we've previously had really good success with older versions of VirtualBox still being compatible with the newer version of the guest additions. :disappointed:

I confirmed that the guest additions version was the issue by rebuilding boot2docker version 19.03.1 with guest additions version 5.2.32 and that fixed the problem. The file is too big to attach to this comment or I would. If someone has a place to host the file, I will upload it, but it would be better if the owners of this repo did the alternate build and included it as a release artifact.
If you want to build it yourself, it's a simple change to the Dockerfile, but be warned that the build took over 2 hours on my not-too-shabby workstation.
diff --git a/Dockerfile b/Dockerfile
index 2513694..b20160a 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -355,9 +355,11 @@ RUN ( cd /usr/src/haveged && ./configure LDFLAGS='-static --static' ); \

 # http://download.virtualbox.org/virtualbox/
 # updated via "update.sh"
-ENV VBOX_VERSION 6.0.10
+# ENV VBOX_VERSION 6.0.10
+ENV VBOX_VERSION 5.2.32
 # https://www.virtualbox.org/download/hashes/$VBOX_VERSION/SHA256SUMS
-ENV VBOX_SHA256 c8a686f8c7ad9ca8375961ab19815cec6b1f0d2496900a356a38ce86fe8a1325
+#ENV VBOX_SHA256 c8a686f8c7ad9ca8375961ab19815cec6b1f0d2496900a356a38ce86fe8a1325
+ENV VBOX_SHA256 4311c7408a3410e6a33264a9062347d9eec04f58339a49f0a60488c0cabc8996
 # (VBoxGuestAdditions_X.Y.Z.iso SHA256, for verification)

 RUN wget -O /vbox.iso "https://download.virtualbox.org/virtualbox/$VBOX_VERSION/VBoxGuestAdditions_$VBOX_VERSION.iso"; \

So if the guest additions version is the problem, what's the best way to fix this? Should there be two ISOs in the next release, one with VirtualBox 5.2 guest additions, and one with VirtualBox 6.0 guest additions?

So if the guest additions version is the problem, what's the best way to fix this? Should there be two ISOs in the next release, one with VirtualBox 5.2 guest additions, and one with VirtualBox 6.0 guest additions?

I think so. It would also be nice to have a version with open-vm-tools version 10.1.5, so that I can use VMware Fusion 8.5 (the last version that runs on El Capitan).

If we can verify that the VirtualBox 5.x guest additions still work fine on 6.x, I think we should just revert to that and stick with 5.x until there's a compelling reason not to (like them not working on a newer version or a newer version committing to still support older versions).
Regarding open-vm-tools, I guess the same applies -- it seems weird to me that they'd remove support for older hypervisors in newer tools, but I suppose there's a maintenance burden either way no matter what.
So consider this an official call for help to verify that 5.x guest additions work on VirtualBox 6.x (easiest way to test likely being running https://github.com/boot2docker/boot2docker/releases/tag/v18.09.8 on newer VirtualBox and verifying the automount works).
@Nuru does our 18.09 release work on Fusion 8.5?  (It's on open-vm-tools 10.2.x, so it'd be more reasonable to not have to go all the way back to 10.1 IMO for that to be reasonable to consider.)

@tianon No, 18.09.9 does not work on Fusion 8.5. Fusion 8.5 stopped at VMware Tools 10.1.6. I tried to build a version of 19.03 that used open-vm-tools 10.1.5 but I could not get that to work either. I am not confident that I built it correctly, though, so don't take my failure as definitive.
I tried using docker-machine version 0.16.1 to create a machine with boot2docker.iso from v18.06.0-ce-rc1 (which has open-vm-tools v10.0.0-3000743) and that didn't work either, so I guess don't bother with the support for Fusion 8.5.
For the record, by "didn't work" I mean the docker-machine create fails at the point where it tries to enable file sharing (vmrun enableSharedFolders), even if you set --vmwarefusion-no-share. I was able to use v18.06.0-ce-rc1 anyway, but I could not mount any directories from the host file system, which is critical for my work, so it's of no use to me.

So consider this an official call for help to verify that 5.x guest additions work on VirtualBox 6.x (easiest way to test likely being running https://github.com/boot2docker/boot2docker/releases/tag/v18.09.8 on newer VirtualBox and verifying the automount works).

One :+1: from me, using VirtualBox 6.0.10. mount shows shared folder correctly, and there's no problem with read / write access. New VM created with:
docker-machine.exe create --virtualbox-boot2docker-url https://github.com/boot2docker/boot2docker/releases/download/v18.09.8/boot2docker.iso

Awesome, thank you!  Some testing on https://github.com/boot2docker/boot2docker/pull/1401 would be appreciated! :smile: :+1:

Test ISO: https://gist.github.com/tianon/829be6a16ade16f9b39be491bc13d0e3
Direct link: https://gist.github.com/tianon/829be6a16ade16f9b39be491bc13d0e3/raw/cbd58731532c04f6683733442a0eae5cc91f957d/boot2docker.iso

(Especially looking for both VBox 5.x and 6.x verification, if we can get it!)

@tianon  #1401 passes the smoke test running under VirtualBox 5.2.32. Host directories mount as expected.
I have been running my custom build (which has the same Dockerfile changes as #1401 but nothing else) for 2 days now without problem, using VirtualBox 5.2.32 on OS X 10.11.6

For the record, I backported #1401 to both 18.09.x and 19.03.x, so the following pre-releases both include VirtualBox Guest Additions v5.2.32:

https://github.com/boot2docker/boot2docker/releases/tag/v18.09.9-beta1
https://github.com/boot2docker/boot2docker/releases/tag/v19.03.2-beta1

Thanks @tianon !
For the record, all of the 18.09 releases had VirtualBox Guest Additions v5.2.32 or earlier. it was only the 19.03.0 and 19.03.1 releases that had VirtualBox Guest Additions v6. (v6.0.10 to be exact).


WIP | Linx Commerce working integration

What is this contribution about?
Ignore this Pull Request for now. This is mainly for tracking. Once im confident this can be used for everyone then i'm writing a detailed description.
A lot of things also need changes.
Loom

Record a quick screencast describing your changes to show the team and help reviewers.

Link

Please provide a link to a branch that demonstrates this pull request in action.

Still ongoing.


json-loader deprecated

Hi
https://github.com/webpack-contrib/json-loader seems to be deprecated and as noticed in the reamde, webpack 2+ does support json imports by default.
Think this loader can't be removed?

Good catch, I will need to revisit this.

already have fixed it, PR incoming;)

https://github.com/rmariuzzo/laravel-localization-loader/pull/10
what do you think?


Numpy 1.17 version is too high resulting in an error

Hello, install numpy 1.17 according to the environment version you provided, but I encountered the following problems when executing the create_model mode:
/home/zh/sda3/Anaconda3/envs/reinvent_2.0/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:541: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/zh/sda3/Anaconda3/envs/reinvent_2.0/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:542: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_quint8 = np.dtype([("quint8", np.uint8, 1)])
/home/zh/sda3/Anaconda3/envs/reinvent_2.0/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:543: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint16 = np.dtype([("qint16", np.int16, 1)])
/home/zh/sda3/Anaconda3/envs/reinvent_2.0/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:544: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_quint16 = np.dtype([("quint16", np.uint16, 1)])
/home/zh/sda3/Anaconda3/envs/reinvent_2.0/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:545: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint32 = np.dtype([("qint32", np.int32, 1)])
/home/zh/sda3/Anaconda3/envs/reinvent_2.0/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  np_resource = np.dtype([("resource", np.ubyte, 1)])

When I reinstalled numpy==1.16.4, I executed the command again without any errors,
So do you need to modify the provided environmental dependencies?

Hi, this isnt an error. It is just a warning about potential future issues between tensorboard and numpy. You can still use numpy 1.17 and that will not impact the results.
While on the versioning topic probably worth mentioning that the predictive models we supply as an example are pickled with older version of sckit-learn and from version 0.22 onwards they might break. So if you want to use newer versions of scikit-learn you will have to make your own predictive models.

@patronov
I think the new prediction model you can provide is the best result

Upgrading to scikit-learn 0.22 (or higher) will happen with the next major feature release. Until then however we are not planning an update on the provided  example models since this drags along environment changes and the need to properly test the code (and possibly adapt certain parts).


Client/Configuration Cache

Hello,
I have UI for client configuration in my app... However it seems that IdentityServer4 has some kind of cache, which is used and changes in client does not work immediately.
During configuration I add this: .AddConfigurationStoreCache()
However I do not see any way to clear it when needed. Do you have any suggestions?

We don't have an API to clear our cache. If you need this, then implement your own (look for the Caching section): http://docs.identityserver.io/en/release/topics/startup.html


synchronization of transforms in the scene tree and b2world.

I may play around a bit with this
void Box2DPhysicsBody::state_changed() {
	set_block_transform_notify(true);
	set_global_transform(b2_to_gd(body->GetTransform()));
	//if (get_script_instance())
	//	get_script_instance()->call("_integrate_forces");
	set_block_transform_notify(false);

	// TODO something? check rigidbody2d impl
	//if (contact_monitoring) {
	//	world_node->world.contac
	//}
}

Will definitely want to be able 'zoom' into, and rotate the view of the box2d world?   (Maybe putting the world under a node that is scale by 2, or even rotated or whatever.  If we set global transforms as the sync to tree from b2d, we're going to lose that.
How to handle in-tree transformations as well?  What happens if I added a Node2D under Box2DWorld, scale and rotate it, then add Box2dBodies under that node?   What if I animate that node running as well?
And maybe the best first solution is to just punt on any in-tree transformations between the Box2dWorld and Box2dBody nodes, but stuff above the Box2dWorld node should definitely work.

So after discussion, it seems scale can't be supported anywhere in-between the node holding the b2world (Box2DWorld), and a node holding a b2body (Box2dPhysicsBody).  And likewise between a body and a fixture.

I'm anyways, I'm unsure if that's inclusive of Box2DWorld.    I still wanna try and a do a relative to world transform, as I think this would be better and even more flexible than whats currently in godot

@jordo Is this topic closed by #20 ?

You can close this as well.  but will create and keep another issue open to refactor this once https://github.com/godotengine/godot/pull/44200 is merged upstream
The current implementation will break whenever a non-Node2D is in-between the Box2dWorld and a Box2dPhysicsBody.  With the upstream pull, I can make it cleaner, and support any CanvasItem between the World and Bodies.

As far as I know this is solved by #20, closing


Algo income estimation

Is that even possible?

Why not just use NiceHash?
https://www.nicehash.com/profitability-calculator


eosio.system got fatal error!

the master branch got a fatal error ,it can not be deployed on chain ,the error message is following:
root@ubuntu:/home/time/eosio.contracts/build/contracts# cleos set contract eosio eosio.system/
Reading WASM from /home/time/eosio.contracts/build/contracts/eosio.system/eosio.system.wasm...
Publishing contract...
Error 3070000: WASM Exception
Error Details:
env.is_feature_activated unresolveable
my nodeos  version is 1.8.1 and eosio.cdt is 1.6.1, may you tell me what happend about this contract?

the contract which got is eosio.system, may you fix it?

do you have any body to solve this issue ?

I also met this problem. How did you solve it  ，eosio v1.8.1 eosio.contracts v1.7.0-rc1

@dawnFromDark

@lsf1001 Running the step described here in the Preactivate feature
section solved the issue for me.


AirSim - PX4 - SITL issue

Hi.
I want to use PX4 in SITL mode in my AirSim simulator.
I use windows 10 so I first install Cygwin Toolchain.
Then I found a tutorial and so I just run the following commands :
mkdir -p PX4
cd PX4
git clone https://github.com/PX4/Firmware.git
cd Firmware
After I run :  git checkout v1.10.1 I have :

Then when I run make px4_sitl_default none_iris, after a minute or 2 a have the following error :

Thanks for yout help.

https://github.com/PX4/windows-toolchain/issues/18


Add infrastructure for consent banner and link

Closes https://github.com/ethyca/fides/issues/3043 and https://github.com/ethyca/fides/issues/3051 and https://github.com/ethyca/fides/issues/3160
Code Changes

[ ] Add consent infrastructure that renders dummy banner and link
[ ] Uses Preact + TSX
[ ] Link generation requires an HTML element with the id = 'fides-consent-link' somewhere on the page. We should either surface this info within the experience / notice flow OR make this configurable in privacy experiences

Steps to Confirm
Pre-requisites to test:

[ ] If you haven't already, install Turborepo globally so that you can use it on the command line npm install turbo --global
[ ] In clients folder, run npm install.

To test:

[ ] In clients/privacy-center folder, run turbo run dev, then navigate to http://localhost:3000/fides-js-components-demo.html
[ ] To test other configuration, you can edit the fidesConfig object passed into Fides.init() in privacy-center/public/fides-js-components-demo.html

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

Follow-up Tickets

Move consent-banner.cy.ts to fides-js instead of privacy-center- https://github.com/ethyca/fides/issues/3274

oh exciting, the E2E smoke test failed! though we can't tell why from CI because we are over our free limit 😅

I've restructured quite a bit such that we can now inject all dependencies into Fides.init().
A couple things to note:

cypress is broken, and I'm unsure why. I'll try to debug more tomorrow. This is the error message I get when I try to cd clients/privacy-center, turbo run dev, then in another terminal turbo run cy:run

Error: Webpack Compilation Error
../fides-js/src/lib/banner.module.css 4:0
Module parse failed: Unexpected token (4:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
| * Default CSS styles for the banner
| */
> :root {
|   --fides-consent-overlay-font-family: inherit;
|   --fides-consent-overlay-font-size: 16px;
 @ ../fides-js/src/components/ConsentBanner.tsx 20:0-34
 @ ../fides-js/src/fides.ts
 @ ./cypress/e2e/consent-banner.cy.ts

Since we're dependency-injecting into Fides.init(), the easiest way to test/dev this branch is to navigate to http://localhost:3000/fides-js-components-demo.html. This page calls Fides.init() with a default config object already packaged and ready to go. But if you wish to test with other config options, simply modify the config object in fides-js-components-demo.html.

@NevilleS / @allisonking let me know your thoughts, possibly how to improve dev workflow, too, with this new config injection pattern.

I've restructured quite a bit such that we can now inject all dependencies into Fides.init().
A couple things to note:
cypress is broken, and I'm unsure why. I'll try to debug more tomorrow. This is the error message I get when I try to cd clients/privacy-center, turbo run dev, then in another terminal turbo run cy:run

hmm that's a little mysterious, I guess it's because cypress doesn't know how bundle css modules, but it's also odd that it's trying to bundle the module. I'll take a look and see if there's a workaround.


GDB performs step instead of next

There is a code like this:
8:  void user_init()
9:   {
10:    f2();
11: }
both are located in flash.
Scenario:

hbr user_init
c
It stops at the beginnign of user_init, line 9
next
It shows
10 f2()
next
Expectation is that it goes to line 11
Actual result is that it goes into f2 and produces the message like this:
Warning:
Cannot insert breakpoint 0.
Error accessing memory address 0x4028f3ab: Input/output error.

Unfortunately, that's a hardware limitation combined with the way gdb handles single-stepping. With step 1, you already use the single available hardware breakpoint, forcing gdb to use software breakpoints from then on. With step 4, you implicitely ask gdb to set a breakpoint on the next line and run to there. Gdb tries to put a software breakpoint to do this, but because the routine is in flash, gdbstub is unable to and fails with the given message.

Hm, should it do it a proper way if i remove hbr between steps 1 and 2? I tried and the behavior is still the same, it does stepping into instead of stepping next line.

It should, but I don't know how gdb react if it can't place a software breakpoint on a certain location. Theoretically, gdbstub could intercept that and use a hardware breakpoint itself, but that's a pretty non-trivial thing to implement.