Create GitHub Action Code

What about this ?
Coded for GitHub Action about Admina App Syncs.
What can be done ?
We can register AzureAD SSO Apps with Admina on a CLI basis.
Docker files and other files to run on GitHub Action will be prepared after this.
refs:
https://moneyforward.slack.com/archives/C0421LR2E23/p1689815107744759
refs:
https://github.com/actions/typescript-action

> node dist/index.js

Starting sync...
Start Sync Data from AzureAd to Admina .
Getting Azure AD data...
Getting access token...
Detected 39 users in テストログインアプリ
Detected 55 users in Google Workspace
Registering custom service...
Sync finished.
Workspace already exists | ServiceName: Single Sign-On(xxxxx) ,WorkspaceName: テストログインアプリ(xxxxx)
Workspace already exists | ServiceName: Single Sign-On(xxxxx) ,WorkspaceName: Google Workspace(xxxxx)
Register data into テストログインアプリ: existingUsers 39 , newUsers 0 , deleteAccounts 0
Register data into Google Workspace: existingUsers 55 , newUsers 0 , deleteAccounts 0

The whole thing is AI generated?

@peter-asztalos-mf
The main code is mostly GPT. I gave a little more specific instructions than before (separate class structures, etc.)
Ecosystem code such as npm copied the sample repository!

the sample repo gave us making typescript work with GitHub actions.


[Dev-branch 8.0.0] Server-side crash from attempting to load tickable sound class

heres the crash log
http://paste.fedoraproject.org/199562/42669190

Have the same crash, here is my crashlog: http://pastebin.com/9eDvf5WR

Seems that some recipe stuff somehow is referencing client side stuff.

looking at the code in WasherRecipe its not immediately apparent to me HOW its referencing client stuff.

stacktrace references these lines:
public FluidInput waterInput = new FluidInput(new FluidStack(FluidRegistry.WATER, TileEntityChemicalWasher.WATER_USAGE));
this(new GasInput(input), new GasOutput(output));

Perhaps it interacts with the chemical washer class somehow causing the server to attempt loading the client side sound stuff?

New tile sound implementation is a bit rough with its client references, I'm trying a fix but it's acting weird.  I'll push a fix as soon as I figure it out.

src/main/java/mekanism/common/tile/TileEntityNoisyElectricBlock.java has the import for mekanism.client.sound.IResettableSound; which is likely involved in this.

Yes. The import won't necessarily matter though unless it is referenced...

Yeah, it seems to happen the INSTANT TileEntityChemicalWasher is loaded.

looks like line 22 might be the culprit.

In the entityNoisyelectric class i mean

the sound member variable has type IResettableSound which extends import net.minecraft.client.audio.ITickableSound;

Fixed

is the code in an uncommittable state atm? I notice there is no new commit, so as of yet , can't test for further bugs.

There we go, forgot to push ^^

odd... now i'm getting a buildcraft related crash that I wasn't getting before.

http://paste.fedoraproject.org/199748/26707473/

Does that happen without Mekanism installed? Also, @aidancbrady, I figured out a solution to the server-side sound issues that works and seems to avoid the issues you were encoutering, and doesn't require the usage of a wrapper class. Shall I revert it before I pull, or go with my solution in a merge?

yes, without mekanism it launched.

@unpairedbracket, just revert and add your change - my fix was definitely messy :P

buildcraft.api.transport
used in the following files:
src/api/java/buildcraft/api/transport/PipeWire.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IExtractionHandler.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IPipeConnection.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IPipeConnection.java:import buildcraft.api.transport.IPipeTile.PipeType;
src/api/java/buildcraft/api/transport/package-info.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IPipePluggable.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IPipe.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IPipeTile.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IStripesPipe.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/IStripesHandler.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/transport/PipeManager.java:package buildcraft.api.transport;
src/api/java/buildcraft/api/gates/IGate.java:import buildcraft.api.transport.IPipe;
not sure yet what the error might be.

I put the crash report and the other message into a new issue, since i'm not certain it has anything at all to do with the sound problem.


The sync view can be a little slow

Thanks for creating the sync’ed view feature! I was testing it on a model with <= 20k nodes, can it can be a little slow (a few seconds) before the other pane reacts to a click. When a node does not have a match the viewer will do nothing, so I am often usure if there is no corresponding nodes or if it’s simply loading when the view doesn’t change after a node selection.

Hi @justinchuby, thanks for your feedback. @jinjingforever, can you please take a look? Thanks.


Take service broker name from request query in find-outdated-deployment API

Pull Request Test Coverage Report for Build 3678

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0008%) to 89.094%

Totals

Change from base Build 3676:
0.0008%

Covered Lines:
9165

Relevant Lines:
9929

💛  - Coveralls


fuzzing: improve archive fuzzer

Modify archive_fuzzer.go so that it creates a manifest file if it is not created by TarBytes().
Signed-off-by: AdamKorcz<EMAIL_ADDRESS>

/ok-to-test


Support matrix multiplication for vector

Currently, there is a calculation bug for matrix-vector or vector-vector multiplication due to the small block size. For example, (1xD)x(DxD) and (1xD)x(Dx1) return wrong output. I know this is a reported bug. But I currently need triton support this very eagerly. Hence I am asking when can such a function be supported? Sorry for my urge, but I have been stuck by it for a long time.

@tdzdog can you please elaborate a bit more, what is your challenge here?   Are you using Docker image or building Triton?

@tdzdog can you please elaborate a bit more, what is your challenge here? Are you using Docker image or building Triton?

Thanks for the reply, @awaisbajwaml. I am trying to build a simple kernel in triton, it calculates the matrix multiplication of a D-dimensional vector (1xD) and a matrix (DxD). It also calculates the dot product between two D-dimensional vectors as (1XD)x(Dx1) = 1x1 (a scalar). However, when I am using tl.dot to achieve such two calculations, the results are both wrong. I notice some early issues mention the reason is because of the small block size of 1XD block. Hence I am asking when can such a bug be fixed.

Hey! Have you tried to emulate this with broadcast and tl.sum() ? Also, are you trying FP16 or FP32 ? I'd be surprised if FP32 doesn't work :sweat_smile:

Hey! Have you tried to emulate this with broadcast and tl.sum() ? Also, are you trying FP16 or FP32 ? I'd be surprised if FP32 doesn't work 😅

Hi, @ptillet , thanks for your reply. I have tested my codes as you asked. I found the reason is that I execute a loop in kernel. In such case, it causes many strange results. Currently, I have divided the loop into multiple threads, it works greatly for vector-matrix or vector-vector. However, I have met another problem. I execute two consecutive tl.dot and it returns wrong results. To be specific, for a (1xD) x, (DxD) w and (Dx1) y, I want to calculate their matrix multiplication results as xWy, a (1x1) scalar. I first use tl.dot to calculate xW and it works. But when I use tl.dot to calculate (xW)y, it returns wrong results. Do you have any suggestions?

This sounds like it might be the same bug as https://github.com/openai/triton/issues/328; not sure if it's worth merging. That one has a minimized test case of what I ran into.

Closing this. On the latest main, use tl.sum(matrix*val[None, :], 1) for matrix-vector mutliplication. Just double-checked that it works.


updated the link to the repository

Hi @cobrce and thanks for your contribution! We have actually fixed this up separately in 2cf15b1115981c82c5f03328d583083ae0bb98cd, so I am going to close this PR.


The automated release is failing 🚨

:rotating_light: The automated release from the master branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you could benefit from your bug fixes and new features.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can resolve this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
 You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here is some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

No npm token specified.
An npm token must be created and set in the NPM_TOKEN environment variable on your CI environment.
Please make sure to create an npm token and to set it in the NPM_TOKEN environment variable on your CI environment. The token must allow to publish to the registry https://registry.npmjs.org/.

Good luck with your project ✨
Your semantic-release bot :package::rocket:

token added, cli not working
StatusCodeError: 403 - {"message":"{\\"message\\":\\"API rate limit exceeded for <IP_ADDRESS>. (But here\'s the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)


xClusterPreferredOwner: Set-TargetResource does not handle Ensure correctly

Details of the scenario you tried and the problem that is occurring:
Set-TargetResource will set preferred owners regardless if they are already correct or not. Set-TargetResource should only set preferred owner if they are not in desired state.
The DSC configuration that is using the resource (as detailed as possible):
n/a
Version of the Operating System and PowerShell the DSC Target Node is running:
n/a
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Dev

Indeed, using this resource always returns a change with multiple consequent runs


Server-Side Data Filtering

Is there any way to get Server-Side Data Filtering working?
I tried to pass a callback to onFilter but this does not work.
this.columns = [
            {
                text: 'Name',
                dataField: 'name',
                filter: textFilter({
                    delay: 1000,
                    onFilter: myCallback
                })
            }
]

HI @anikakalbrun the remote filter is totally different from react-bootstrap-table, you can check this firstly, and check this to see a remote filter example.
Let me know if you still get any questions, thank you

I don't know how I could miss the remote filter example...
Thanks a lot Allen.


Close editor under file dropdown doesn't work

File > Close Editor
Doesn't do anything.

What OS/browser are you using? Are there any errors in the developer console? I can't seem to replicate this on Mac/Chrome

Tried on Windows/Chrome Mac/Chrome and Mac/Safari and none of them work. Maybe the functionality wasn't implemented for the browser?

Currently, it is set to close the currently focused tab. I suppose with no windows open it would appear to do nothing

Oh, I thought it was supposed to close the editor itself since the shortcut that It's linked to is Ctrl/Command + W which is the browser tab close shortcut on most browsers.

That is a good point, we could definitely add that

I found that Close Editor worked in closing tabs, but of course there's no UI to tell you what it does when no tabs are open. I also found that Exit did nothing as well.

Currently, it is set to close the currently focused tab. I suppose with no tabs open it would appear to do nothing

I would like to suggest to change it. There is no option to logout, to force the login window.
What about the idea that, the File -> Exit forces a logout ?
And after the logout it may close the tab as described earlier in this thread.
Anyone any thought about this idea ?

Currently, it is set to close the currently focused tab. I suppose with no tabs open it would appear to do nothing

I would like to suggest to change it. There is no option to logout, to force the login window.
What about the idea that, the File -> Exit forces a logout ?
And after the logout it may close the tab as described earlier in this thread.
Anyone any thought about this idea ?

We're talking about the File -> Close Editor button here, but putting that aside, I think that setting File -> Exit to force a logout is a great idea


[Bug Report] ttnn.conv2d has low PCC with large kernel

Describe the bug
When the kernel is large (e.g. [2048, 1024, 1, 1]) for conv2d, the PCC is degraded (< 0.99)
To Reproduce
Run the example below:
import torch
import ttnn

from tests.ttnn.utils_for_testing import assert_with_pcc

def main(device):
    input_shape, weight_shape, stride, padding, dilation, transposed, groups = (
        [1, 1024, 45, 80],
        [2048, 1024, 1, 1],
        [2, 2],
        [0, 0],
        [1, 1],
        False,
        1,
    )
    n, in_c, in_h, in_w = input_shape
    out_c, _, k_h, k_w = weight_shape

    n, in_c, in_h, in_w = input_shape
    out_c0, out_c1, k_h, k_w = weight_shape
    out_c = out_c1 if transposed else out_c0
    torch_input = torch.rand(input_shape, dtype=torch.bfloat16)
    torch_weight = torch.rand(weight_shape, dtype=torch.bfloat16)
    torch_bias = torch.rand((out_c,), dtype=torch.bfloat16)
    input_tensor = ttnn.from_torch(torch_input.permute((0, 2, 3, 1)), layout=ttnn.ROW_MAJOR_LAYOUT)
    weight_tensor = ttnn.from_torch(torch_weight, layout=ttnn.ROW_MAJOR_LAYOUT)
    bias_tensor = None

    expected_output = torch.convolution(
        torch_input, torch_weight, torch_bias, stride, padding, dilation, transposed, (0, 0), groups
    )
    expected_output = expected_output.permute((0, 2, 3, 1)).reshape(1, 1, -1, out_c)

    output_tensor = ttnn.conv2d(
        input_tensor=input_tensor,
        weight_tensor=weight_tensor,
        bias_tensor=bias_tensor,
        batch_size=n,
        in_channels=in_c,
        out_channels=out_c,
        input_height=in_h,
        input_width=in_w,
        kernel_size=(k_h, k_w),
        stride=stride,
        padding=padding,
        dilation=dilation,
        groups=groups,
        device=device,
    )

    assert_with_pcc(expected_output, ttnn.to_torch(output_tensor), 0.99)

if __name__ == "__main__":
    try:
        device = ttnn.open_device(device_id=0, l1_small_size=16384)
        main(device)
    finally:
        ttnn.close_device(device)

Please complete the following environment information:

OS: Ubuntu 20.04
Version of software 2dfe62c6cb2d0c3d2df49d39aa1af7b5a743caa2
Card: N300

using normal distribution for input tensor generation would resolve this one as well.


KON-624 KoParameterDeclaration should not have a fullyQualifiedName property

Removed KoFullyQualifiedNameProvider from KoParameterDeclaration, KoPackageDeclaration and KoVariableDeclaration

Changed implementation KoPackageDeclaration.name property, eg:
Declaration:
package com.samplepackage

val packageDeclaration = scope
	.packages
	.first()

Before:
packageDeclaration.name // returns "samplepackage"

After:
packageDeclaration.name // returns "com.samplepackage"

Merged in  https://github.com/LemonAppDev/konsist/pull/1238


Provide the right extension for tags when using uglyURLs

This PR fixes an edge case bug when setting the site parameter uglyURLs to true. This would make all the tag links point to blank pages.
You can find more information about this bug on https://github.com/gohugoio/hugo/issues/1989 .
The fix is shamelessly lifted from https://github.com/olOwOlo/hugo-theme-even/pull/129 . Thank you kind stranger.
I usually don't do whatever this language is. Please clobber me for style.

@matcornic Are there any reasons this can't be merged?

Sorry but this is not a good fix for the underlying issue.
The variable uglyURLs is set at the top level of config however that value does not appear to be accessible to the template. The fix you brought in is suggesting that the config contain duplicated settings for uglyURLs which would have to be kept manually in sync.
uglyURLs
and
Params.uglyURLs
Hugo should know the correct path to a taxonomy page, but annoyingly even their own documentation page "guesses" at the url of the tag rather than asking Hugo for it
https://github.com/gohugoio/gohugoioTheme/blob/master/layouts/partials/tags.html.


Alias name restriction (Firewall:Alias)

when trying to add a new device, i tried to do this according to the scheme i always did it so far.
unfortunately, this seems to be no longer accepted without obvious reason.
my scheme is:
<device type, 1 letter>_
as an example:
W_MyWifiDevice
this is not accepted and results in the message "The name must start with a letter or single underscore, be less than 32 characters and only consist of alphanumeric characters or underscores.". i think this is fulfilled.
but using
WW_MyWifiDevice
or
_W_MyWifiDevice
is accepted.
i'm running
OPNsense 23.1.1_2-amd64
Intel(R) Celeron(R) CPU J3455 @ 1.50GHz (4 cores, 4 threads)

https://github.com/opnsense/core/commit/158220ebd606a1a173f99b75c62f95737da6bd74


MacOS support

Added MacOS support. Please don't merge this PR yet, since it should be modified according last changes it haxm (cpuid feature setting addition).
Tested it on windows host with image from https://github.com/intel/haxm/issues/149. It runs but ends in a damaged disk error window. Same with qemu+tcg, so this looks like installation iso issue, at least on my host.
MacOS can read/write different MSRs in different runs, when some MSR is not emulated, the guest crashes. To debug this, I've added logging of accesses to these MSRs. When you encounter crashes, look to the log.
MacOS writes data to UEFI vars flash. When these are written wrong (due to unexpected halt etc.), you may encounter display issues. The first sign of these is that the apple in not in the center of the monitor. In such a case restore UEFI vars flash to it's original file.
This qemu patch is needed to support UEFI in qemu for hax:
0001-Add-support-for-ROMD-regions-for-hax.patch.txt

@nevilad, could you help to rebase the commit onto the latest release version so that it can be tested on macOS? Thanks.

This PR is not ready! I'm waiting for my set_cpuid feature, so the user can set Penryn cpu. Now it's hardcoded in this line:
state->_eax = 0x10673; //Penryn
Another thing is IA32_PERF_STATUS. I think it would be better to return as bus ratio part of the MSR the value from the underlying hardware, if it is new and contains this data in this MSR.
When you wan't check cr8 impact on booting empty multicore guests on MacOS, it is OK to test on this code, without latest changes from master.


Multiple minor issues: typographical error, regression in image quality, hierarchy issue.

Opening just one issue for all these minor problems as they really are minor.
Issue 1: Texts
Typo 1
Typo in main section's SVG artwork containing Name of Org and Sub-heading. The typo is in the sub-heading.
- A Organization which core goals is to promote open source in Nepal
+ An organization whose core goals are to promote open source in Nepal.

Typo 2
In FAQ's "What is GNOME Nepal"
- Gnome Nepal is the open source organization whose main motive is to spread collaboration and open source in Nepal.
+ GNOME Nepal is an open source organization whose main motive is to spread collaboration and open source in Nepal.

Typo 3
FAQ's second question
- How we can join this Community?
+ How can we join this community?
OR
+ How can I join this community?

Typo 4
FAQ's last question should be re-written entirely.
- Is there are exciting event?
+ Are there any exciting events coming up?

Enhancement 1
Use a concise version of OSI's open-source definition in FAQ's open-source definition.
Enchancement 2
text can be refined for empty state of calendar. The current text isn't as clear as it should be.
- Check back later for new events.
+ Please check back later for updates on new events.

Issue 2: Image quality
The latest PR has regressed the image quality of Group photo greatly. People's faces are pixelated. A bit more pixels are affordable.
Issue 3: Hierarchy
The text below the banner image is at 16px but text in FAQ section is 18px. That does not make sense. A better typographical hierarchy must be established at the very least for the "Who we are" and "Our mission" section.

Thanks for pointing this out, @sudanchapagain. Can you work on resolving it? I’ll assign it to you

I have fixed the typo and image regression and opened a PR https://github.com/GNOME-Nepal/website/pull/22 . However, i will not be dealing with the design issues.


Create a Pull request to merge the dev Branch to main

Created but there are some merge conflicts that should be resolved manually


Build of gcc 5.3.0 fails on Arch Linux

Dear all,
I am reporting this failing on Arch Linux, but I am almost sure that this issue is due to Arch Linux and not to your great installer. Anyhow, I know you may want to have any feedback.
I tried to build and install gcc 5.3.0 with the following command
stefano@zaghi(10:46 AM Fri May 20) on master
→ ./build.sh -p gcc -c -f -j 10 -I 5.3.0 -i /opt/arch/gcc/5.3.0

Presently, Arch Linux has gcc 6.1.0 as default and trying to install also the 5.x trunk is painful. As a matter of fact, the manually building from source of gcc 5.3.0 fails in the same way the building by means of your script, thus I am quite sure that this problem is due to Arch Linux. For the sake of completeness I attached the log file of failing build.
gcc-v5.3.0-building-fails.log.zip
I report this in case you like to advice other Archers that this is an issue of Arch Linux and not of the installer script.
Close this as soon you want.
My best regards.
Stefano

I haven't dug into the specifics, but I know multi-threaded GCC builds often fail. When installing OpenCoarrays directly, the script uses a single threaded GCC build. Could you try removing the -j 10 flag?

@zbeekman Yes, I can surely test this. Indeed, I am quite sure that the last time I have installed gcc (gcc 6 before it was a default for Arch) I not changed my default make settings, but my memory could be wrong, and you have always the right answers :smile: I do a check right now, stay tuned!

:cry: no way...
I tried with both your build script and manually disabling multi-thread compilation, but it is still failing
gcc-v5.3.0-building-fails.log.zip
Anyway, do not care about this: I am almost sure that this is not an issue related to your installer, but due to Arch Linux, as a matter of facts also the manual way fails similarly.
Thank you Zaak for your help.

I also haven't looked into this in detail and won't be able to for a few hours at least.
I don't understand the empty -c -f -j flags.  Each of those flags requires an argument, e.g.,
./build.sh -p gcc -c /opt/local/bin/gcc -j /opt/local/bin/gfortran -C /opt/local/bin/g++ -I 5.3.0 \
-i /opt/arch/gcc/5.3.0

Please try the above.

@rouson Right, sorry for my too fast read of build help, I am re-run without them (this should use the defaults, right?)
./build.sh -p gcc -I 5.3.0 -i /opt/arch/gcc/5.3.0

Yes. That should use the defaults.  How did it go?

Also, I just realized a couple of things that I should clean up in a patch release:

I'm thinking the documentation will now recommend using ./install.sh for all purposes. Although it should work to call ./build.sh directly, I don't think there are any specific reasons to do so at the moment. install.sh invokes build.sh but has additional capabilities so the recommended command is

./install.sh -p gcc -I 5.3.0 -i /opt/arch/gcc/5.3.0

The documentation and error-detection should make it clear that which flags require an argument (such as is the case for all flags we've discussed).  This is a very easy fix that we should be able to get post in a bug fix release (1.6.1) next week. In fact, the -f, -c, -C have no default values defined, but on quick review, it appears the build.sh uses the values of the FC, CC, and CXX environment variables if they are set.   If you have any preferences or opinions on whether FC should override -f or vice versa, please let us know.

@szaghi, I'm not sure if this is related, but I just created Issue #189 to demonstrate that we need better detection of missing arguments.  I'll ask @zbeekman to look into this.

@rouson @zbeekman I am sorry for my delay. I cannot say if the last tentative is worked: I had to left the office yesterday after the launch of it (it happens that sometimes my wife like to see me one hour per day, she has strange preferences :smile: ).
Later today I will try to ssh my workdtation and check the log, but with only my smartphone I cannot promise the success.
Thank very much for your help.

No rush.  Thanks for the feedback.

@rouson @zbeekman just checked, still failing, the exit seems the same of above, but I cannot diff the log with the older with my smartphone. Monday I can upload the log if you want.
Have a good weekend, cheers.

Yes seeing the log may be helpful.
On Sat, May 21, 2016 at 8:34 AM Stefano Zaghi<EMAIL_ADDRESS>wrote:

@rouson https://github.com/rouson @zbeekman
https://github.com/zbeekman just checked, still failing, the exit seems
the same of above, but I cannot diff the log with the older with my
smartphone. Monday I can upload the log if you want.
Have a good weekend, cheers.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/sourceryinstitute/opencoarrays/issues/188#issuecomment-220775463

@zbeekman
here is the log:
gcc-v5.3.0-building-fails.log.zip
The test was
./build.sh -p gcc -I 5.3.0 -i /opt/arch/gcc/5.3.0 > gcc-v5.3.0-building-fails.log 2>&1

Anyhow, I really think the problem is due to Arch Linux update: it being a rolling release it could happen that updating the system makes very complex the support of older programs that before the update work well. As a matter of facts, I am not able to install gcc 5.3 manually, whereas I was able to do it before the update.

Thanks for this additional feedback. Given that we don't have much idea about what to do and you suspect the problem is not on our side, I'll close this issue.  If you disagree, please feel free to reopen it.  We appreciate your continued interest in OpenCoarrays and value the issue reports.

Hi @szaghi,
It seems that this issue is caused when trying to build an older GCC with GCC 6. It appears they have patched gcc-5 branch to allow it to be built with GCC 6, however, if you want to build 5.3 it seems like you will have to use an older compiler. If you have an older GCC still installed on your system, install.sh has the -c, -C, and -f flags to let you specify which compiler to use to perform the build. This might let you build 5.3 with an older compiler (4.8 for example...).
Here are the details on the bug tracker: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69959

@zbeekman you are better than Sherlok Holmes. I guessed somenthing along this line (I was betting it was due to some newer version of dependencies, but not to gcc itself...), but I will never find the bugzilla issue, great Zaak!
Feel free to close this (tomorrow I will try to find solution, but I am pretty sure that is very difficult with my current Arch).

Yup, issue is closed already... as I mentioned if you have another, older version of GCC hanging around somewhere on your system, the install.sh script can be told to use a different compiler by specifying the full path to the -c, -C and -f flags....

ops... I have to improve my github-smartphone experience. Unfortunately I have only gcc 6.x (6.0, 6.1 manually installed when Arch was on 5.3 and 6.1 now in the system)...

ok, well in that case, I don't know if you'll be able to get 5.3 installed... you might be able to install the gcc-5 branch using the script--it sounded like they committed changes to enable building with 6.x ... I'm not sure it's currently possible to build the gcc-5 branch with the script, but it looks like Damian may be adding this capability.
I'm surprised theres no older GCC lying around somewhere like /usr/bin or /bin etc.

When Arch updates the system, all the system must be self-referenced... partial updates are not supported for rolling-release distro, so when gcc is updated pacman updates just /usr/bin exes... the only hope is that some Archer like me provides an aur package build for re-add gcc 5.x trunk.

well that sounds like a bit of a pain... but at least you don't have to wait too long for packages I would guess

@zbeekman
The happy end
Some Archers have had pity of me: to whom may concerns, now there is an AUR package build to compile gcc 5.3.0 alongside the official gcc 6.1.1
https://aur.archlinux.org/packages/gcc5/
For the sake of completeness, I report in the following the build script that successfully built gcc 5.3.0 on my Arch Linux.
The package build script is quite verbose (the configure invocation has many flags that I did not use)
# $Id$
# Maintainer: Sebastiaan Lokhorst<EMAIL_ADDRESS># Contributor: Ruben Van Boxem<EMAIL_ADDRESS># Contributor: Allan McRae<EMAIL_ADDRESS>
pkgname=('gcc5')
pkgver=5.3.0
_pkgver=5
_islver=0.15
pkgrel=6
_snapshot=5-20160503
pkgdesc="The GNU Compiler Collection"
arch=('i686' 'x86_64')
license=('GPL' 'LGPL' 'FDL' 'custom')
url="http://gcc.gnu.org"
makedepends=('binutils>=2.26' 'libmpc')
options=('!emptydirs')
source=(#ftp://gcc.gnu.org/pub/gcc/releases/gcc-${pkgver}/gcc-${pkgver}.tar.bz2
        ftp://gcc.gnu.org/pub/gcc/snapshots/${_snapshot}/gcc-${_snapshot}.tar.bz2
        http://isl.gforge.inria.fr/isl-${_islver}.tar.bz2)
md5sums=('907f3b860ebbaf33b3ccd3c15dddc9b1'
         '8428efbbc6f6e2810ce5c1ba73ecf98c')

#_basedir=gcc-${pkgver}
_basedir=gcc-${_snapshot}

prepare() {
  cd ${srcdir}/${_basedir}

  # link isl for in-tree build
  ln -s ../isl-${_islver} isl

  # Do not run fixincludes
  sed -i 's@\./fixinc\.sh@-c true@' gcc/Makefile.in

  # Arch Linux installs x86_64 libraries /lib
  [[ $CARCH == "x86_64" ]] && sed -i '/m64=/s/lib64/lib/' gcc/config/i386/t-linux64

  echo ${pkgver} > gcc/BASE-VER

  # hack! - some configure tests for header files using "$CPP $CPPFLAGS"
  sed -i "/ac_cpp=/s/\$CPPFLAGS/\$CPPFLAGS -O2/" {libiberty,gcc}/configure

  mkdir ${srcdir}/gcc-build
}

build() {
  cd ${srcdir}/gcc-build

  # using -pipe causes spurious test-suite failures
  # http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48565
  CFLAGS=${CFLAGS/-pipe/}
  CXXFLAGS=${CXXFLAGS/-pipe/}

  ${srcdir}/${_basedir}/configure --prefix=/usr \
      --build=${CHOST} \
      --libdir=/usr/lib --libexecdir=/usr/lib \
      --mandir=/usr/share/man --infodir=/usr/share/info \
      --with-bugurl=https://bugs.archlinux.org/ \
      --enable-languages=c,c++,fortran,go,lto,objc,obj-c++ \
      --enable-shared --enable-threads=posix --enable-libmpx \
      --with-system-zlib --with-isl --enable-__cxa_atexit \
      --disable-libunwind-exceptions --enable-clocale=gnu \
      --disable-libstdcxx-pch --disable-libssp \
      --enable-gnu-unique-object --enable-linker-build-id \
      --enable-lto --enable-plugin \
      --with-linker-hash-style=gnu --enable-gnu-indirect-function \
      --disable-multilib --disable-werror \
      --enable-checking=release \
      --program-suffix=-${_pkgver} \
      --enable-version-specific-runtime-libs
      #--enable-install-libiberty

  export LD_PRELOAD=/usr/lib/libstdc++.so
  make
}

package() {
  cd ${srcdir}/gcc-build

  make -j1 DESTDIR=${pkgdir} install

  # Lazy way of dealing with conflicting files...
  rm -rf ${pkgdir}/usr/share/{info,locale,man}

  # Move potentially conflicting stuff to version specific subdirectory
  mv ${pkgdir}/usr/lib/lib* ${pkgdir}/usr/lib/gcc/${CHOST}/${pkgver}/
  mv ${pkgdir}/usr/lib/gcc/${CHOST}/lib/lib* ${pkgdir}/usr/lib/gcc/${CHOST}/${pkgver}/

  # Install Runtime Library Exception
  install -Dm644 ${srcdir}/${_basedir}/COPYING.RUNTIME \
    ${pkgdir}/usr/share/licenses/$pkgname/RUNTIME.LIBRARY.EXCEPTION
}

I'm glad you got it working for arch linux... Another approach I would have recommended would be to install via http://linuxbrew.sh

Shall we incorporate this script into our installation scripts? Obviously we won't attempt anything close comprehensive coverage of all distros and all compiler combinations, but since @szaghi was nice enough to share this script with us, we might as well innoculate our build system against this scenario.  Thoughts?

My preference would be no. The installation machinery is already quite sophisticated and this is a corner case that seems to be well handled through the usual means.... also, it sounds like arch linux moves quite fast and has no notion of package versions. This script is only to build older versions of GCC with GCC 6.1 on arch linux a very specialized case.

@rouson unless I am misunderstanding you, which may be possible. If you mean, attempt to adapt our current GCC installation process to be able to use some of the configure options etc. so it might work on arch linux (without breaking on ubuntu and OS X) then I'd be willing to try, but that seems like a lower priority.

I agree with your first response.  We can skip it.
Regarding the second response, I was just thinking of a simple conditional to check for the specific situation that started this thread and then a single line invocation of something very close to the script that @szaghi pasted into this thread.
I'm making an analogy to an immune system in which the conditional and submitted script play the role of an antibody should our build system encounter this situation again.  The benefit might not be huge if this scenario is unlikely to occur again, but the cost is also small, given that the script has been written for us.
Then again, I'm no biologist. :)

@zbeekman @rouson
My dear, Arch Linux is a very corner distro, similar to Gentoo, I feel that you can forget about it. My intention was just to give you my feedback.
For the sake of completeness, I would like to give you some details of Arch philosopy. It is a rolling release, so it updates continuosly. It has the notion of packages version, but it is designed to work (indeed very well) with that version for that update, e.g. the current update is designed to work with gcc 6.1.1. In general, installing older version is very simple and do not affect the system: this is the first time I have had such a pain (as Zaak point out because of gcc 5.x has problems to be built with gcc 6.x), but at the end the Archers community is very fast to solve the problems. Indeed, this approach works very well, you have a very update distro while the system remains stable. There are cons too: partial updates are not supported. You can decide if update your system or not (I do almost every day), but if you update it you update all the system, not single packages.
Closed this Arch divagation, here are some of my poor thougths for what can be valued: your installer is great, but already sophisticated/complex enough, trying to add a so comprhensive support for very corner case like mine can make installer complicated without adding a real value. Very special case are better solved with special solution. I am aligned with Zaak. If Damian still would help donkey like me a lightweigth approach could be to add some descriptive informations of the possible issues to the script. For example, if another Archer fails into my case (use the installer to install gcc 5.3 in the current Arch) the installer (after the failing) could recognize that the OS is Arch and prompt to the user

dear archer your OS has known issues for installing older gcc, try to use the package build gcc5

In other words, let Archers to help Archers with a minimum impact on the installer complexity.
This approach has its own cons: you should take care to update the list of known issues. For Arch I can help you, but for Gentoo, Slackware, OpenSuse...?
My feel is that the installer is already great and you do not need to make it great again due to my very corner case.
My best regards.

@zbeekman and @szaghi, many thanks to both of you for your input. :clap:  It sounds like we can move on to solving other problems.


No Tester CancelWorkflow() Method

There seems to be a need for the tester package to provide a method for workflow cancelation.  I can't find any way of achieving it without a contribution, so I made one: #368
If there is already a good way to do this, please let me know and this issue will be resolved.

Thanks for reporting and the fix!


feat(specs): rewrite spec of syntax

The current spec has several formal ambiguities:

Point 1. does not (formally) allow scope.
Examples of scope and description include type.
feat is not a verb.
Unclear what ‘type/scope prefix’ is.

I tried to remove these and I hope this change achieves that.
Do you think it is OK? Is it readable?

@robinpokorny will give it a read and provide feedback as soon as possible.

@robinpokorny this is a bit more sweeping of a change than I'd like to land; perhaps open an issue around some of the specific points you'd like to address and we can move towards this?
tldr; I'd like to spread the rewrites you've done over a few more pull requests.

@robinpokorny I'm going to close this for now, since I haven't heard back from you; would happily open a discussion around some of these edits however -- please don't hesitate to open an issue, and we can have a discussion with some of the other folks who've contributed to this site.

@bcoe, go ahead. I'll open it when I find some time 😉


Characters count as both prefix and suffix of pattern literal

TypeScript Version:  v4.1.0-dev20200925

Search Terms: pattern literal prefix suffix
Code
const s: `:${string}:` = ":";

Expected behavior:
Assignability error because the pattern `:${string}:` has two literal : characters, while ":" has only one.
Actual behavior:
No error
Playground Link
Related Issues: #40598

Related to this issue,

It's great that in dev you can use string as a value placeholder! This is a marked improvement from beta.
The ability to define a prefix or suffix for a key is very valuable, but currently neither of these usecases work:

type CustomProps = Record<`x${string}`, any>;
const a: CustomProps = {};
a.x11 = ''; // ERRORS OUT

type CustomProps ={[key: `x${string}`]:  any}; // ERRORS, key must be string or number
const a: CustomProps = {};

Would be amazing if this could work!


Add John Ling

Add John Ling

Add John Ling

Hey, @JohnL77 I just merged a PR before yours. Can you please pull and push back so the conflict can be fixed? 😅

Sure solved!!

Add John Ling

Hey, @JohnL77 I just merged a PR before yours. Can you please pull and push back so the conflict can be fixed? 😅

Sure solved !!

Already merged in #94


Calling

can't make calls on live chat apps such as discord,  despite perfect connection. Please fix asap. @pyamsoft

Hi, this is because at the moment, things like voice calls use a UDP connection. The Android system does not let me proxy UDP connections, but an issue tracking this is over in #101


A 3 x 3 grid containing placeholder text displays when user opens new tab.

When a user opens a new tab, they should see a grid of 9 items with squares containing the following:

a favicon
text next to the favicon that indicates the name of the site
a placeholder image that represents the thumbnail image for the website.

Next step:
Issue #5

Resolved by #10 - adding favicon will be completed with #9


Call Attribute #5

I need to make a MS Graph call for just Extended Attribute 5. This will call all 15, but do you know how to specify only #5?
https://graph.microsoft.com/v1.0/me?$select=onPremisesExtensionAttributes
We think something like this should work, but it doesn't:
https://graph.microsoft.com/v1.0/me?$select=onPremisesExtensionAttributes/extensionAttribute5

Hi @kristymadimike,
I am not sure how to only get a specific Extension Attribute.
Please post your question on the Microsoft Graph Q&A forum where someone from the appropriate team will be able to assist you.


Varnish multi-arch builds

This PR adds ARM64 architecture builds for Varnish images
Status:

❌ Varnish 4.1 image isn't supporting arm64 architecture as it's based on centos 7 that don't have arm64 support
✅ Varnish 6.0
❌ Varnish 6.4 image for which we don't have arm64 packages
✅  Varnish 6.5 image
✅  Varnish 6.6 image (added in this PR)
✅  Varnish 7.0 image (added in this PR)

All images are currently available at  https://github.com/ihor-sviziev/warden/pkgs/container/varnish, they're built from the following commit: https://github.com/ihor-sviziev/warden/commit/727e086d3731f4cbda557ce3481d43a0d5b93f3d
Note: Marked with ❌ symbol images are old images and they won't support arm any time soon.

About the 4.1 image - it uses x86_64 base image, so it can't support arm64 architecture.
About the 6.4 image - just looked - the arm packages are simply not available, but available for 6.5, 6.6, 7.0
Maybe we should build the multi-arch images for 6.0, 6.5, and add 6.6, 7.0 release lines. What do you think?

Okay, so now I built successfully the arm builds for varnish 6.0, 6.5, 6.6 and 7.0.
All the images are available at https://github.com/ihor-sviziev/warden/pkgs/container/varnish. They were built with https://github.com/ihor-sviziev/warden/runs/5208800122?check_suite_focus=true
Will revert all not related changes

@davidalger, could you please review this PR?

@ihor-sviziev You can use a custom image to make Varnish 4.1 work?
# must be a multi arch image that supports amd64 and arm64
# https://hub.docker.com/_/debian?tab=tags&page=1&name=stretch-20211115-slim
FROM debian:stretch-20211115-slim

# https://varnish-cache.org/docs/4.1/installation/install.html#compiling-varnish-from-source
RUN apt-get update && \
    apt-get install -y pkg-config automake tree autotools-dev libtool build-essential curl git libedit-dev libjemalloc-dev libncurses-dev libpcre3-dev python3-sphinx python3-docutils

RUN git clone --single-branch --branch varnish-4.1.11 https://github.com/varnishcache/varnish-cache/
WORKDIR varnish-cache
RUN ./autogen.sh && \
    ./configure && \
    make && \
    make install && \
    ldconfig && \
    varnishd -V 2>&1 | grep "varnish-4.1.11"

EXPOSE 80

COPY default.vcl /etc/varnish/default.vcl

# The sleep is here because it takes a second for `varnishd` to create the
# `_.vsm` file that `varnishlog` reads (on a local VM at least)
CMD varnishd -p vcl_dir=/etc/varnish/ -a :80 -f /etc/varnish/default.vcl -p http_max_hdr=128 && \
    sleep 1 && \
    varnishlog

I think we shouldn't rebuild and add support for old and reached the end of life versions.
https://varnish-cache.org/releases/

Release
Date
EOL Date
Download

7.0.2
2022-01-25
2022-09-15
varnish-7.0.2.tgz

6.6.2
2022-01-25
2022-03-15
varnish-6.6.2.tgz

6.0.10
2022-01-25
Supported
varnish-6.0.10.tgz

All releases not mentioned above are End-Of-Life and unsupported.

Agreed its EOL was just showing the approach that would maintain current feature parity (assuming 4.1 current works?)

@davidalger cool! I also added few other commits in my own branch. Feel free to copy the changes from https://github.com/ihor-sviziev/warden to make it available for everyone


Fix memory leak in NetworkAvailabilityTransmissionPolicy.

:shipit:

@abaranch , can you please also take a look?

:shipit:


fix: try to fix unstable fuzz test

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
!!! DO NOT LEAVE THIS BLOCK EMPTY !!!

ignore PoolTimedOut error for now

Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

Set acquire_timeout to 60s

Can you explain why you do this?

Set acquire_timeout to 60s

Can you explain why you do this?

Good catch, it seems we don't need to change this value.


Update develop branch Spring Cloud Azure version to 4.5.0 beta.2

As title.

Hi, @Netyyyy

Please rebase this PR.
Please click Ready for review if the PR is ready for review.


mitmproxy 5.0.0 crashes on macOS 10.12 (ok on 10.13)

Downloaded https://snapshots.mitmproxy.org/branches/master/mitmproxy-master-osx.tar.gz
Crashes on macOS 10.12.6 with error

./mitmdump --version
[787] Error loading Python lib '/var/folders/sj/f0p38wnd2dz1483n0zlns51w0000gn/T/_MEIJdk2Qd/Python': dlopen: dlopen(/var/folders/sj/f0p38wnd2dz1483n0zlns51w0000gn/T/_MEIJdk2Qd/Python, 10): Symbol not found: _futimens
  Referenced from: /var/folders/sj/f0p38wnd2dz1483n0zlns51w0000gn/T/_MEIJdk2Qd/Python (which was built for Mac OS X 10.13)
  Expected in: /usr/lib/libSystem.B.dylib
 in /var/folders/sj/f0p38wnd2dz1483n0zlns51w0000gn/T/_MEIJdk2Qd/Python

-> It runs without problem on 10.13.6

Yeah, this is a bit annoying. GitHub Actions currently only provides runners for macos-latest. PyInstaller wants us to compile on the oldest supported macOS version. Two options:

We declare that our binaries only support the latest macOS release.
We fix compatibility with older versions.

For Option 2, I see the following possibilities:

Use self-hosted Github CI runners with an old OSX version. This is suuuuper painful, see e.g. https://help.github.com/en/actions/automating-your-workflow-with-github-actions/about-self-hosted-runners#self-hosted-runner-security-with-public-repositories
Bring back Travis CI to build binaries. Re-introducing another CI provider is kind of annoying, but should work. We definitely want to exclude that from Pull Requests to keep build times short.

I have no idea how bad Option 1 would be. @mitmproxy/devs, any opinions?

Option 1 doesn't sound too bad...

Ok. Let's try 1 and reopen here if we receive complaints.

We declare that our binaries only support the latest macOS release.

Where/how can we know which macOS versions are supported for a specific mitmproxy release ?
Current last version is 10.15, soon it'll be 11.0, does mitmproxy team simply specify "macos-latest", or you'll guarantee/test let say 10.14/10.15/11.0 and will explicitly mention it somewhere ?
(Also this link fails : https://pythonhosted.org/PyInstaller/usage.html#building-mac-os-x-app-bundles)

GitHub now exposes a "10.15" environment, which wasn't the case when we set this up. I've pinned our CI workers to that environment now. We're not going to make any commitments, but this is likely to remain that way until GitHub deprecates it.


[$250] Hybrid app - "Welcome to Expensify" modal when signing into the existing account on a new device.

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: 9.0.39-0
Reproducible in staging?: Y
Reproducible in production?:
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: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1726864963819969
Action Performed:

Go to staging.new.expensify.com and create an account (Any other platform)
Signin with the same credentials on Hybrid app
Tap Try New Expensify from menu

Expected Result:
No welcome modal as user already signed in another platform
Actual Result:
"Welcome to Expensify" modal when signing in
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence

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

Upwork Job URL: https://www.upwork.com/jobs/~021838490385865012073
Upwork Job ID:<PHONE_NUMBER>865012073
Last Price Increase: 2024-12-17

Issue OwnerCurrent Issue Owner: @alitoshmatov

Proposal
Please re-state the problem that we are trying to solve in this issue.
The user has already logged into the New Expensify on a web browser but when he logs into the hybrid app he is shown the welcome to new Expensify modal even though he has already been using it in the web.
What is the root cause of that problem?
We have not taken into account that the user might login first through the web and then in the app. The functions that check if the user should see the welcome modal use the onyx key nvp_tryNewDot.classicRedirect.completedHybridAppOnboarding which is only set when the user logs on the native app on the first time.
What changes do you think we should make in order to solve the problem?
We can use the nvp_onboarding.hasCompletedGuidedSetupFlow flag from onyx to detect that the user has already used the NewDot. This flag when an account is used in the web for the first time is set to to true but on the native app is set to true only once the guided welcome flow has completed. If we add this into the isFirstTimeHybridAppUser function in https://github.com/Expensify/App/blob/main/src/libs/actions/Welcome/index.ts we can make sure that the welcome message does not show.
The first check in that function should be the following:
   const hasCompletedGuidedSetupFlow = !(Array.isArray(onboarding) || onboarding?.hasCompletedGuidedSetupFlow === undefined) && onboarding?.hasCompletedGuidedSetupFlow
        if (hasCompletedGuidedSetupFlow) {
            return;
        }

We can also put this block before isFirstTimeHybridAppUser is called:
function handleHybridAppOnboarding() {
    if (!NativeModules.HybridAppModule) {
        return;
    }
       const hasCompletedGuidedSetupFlow = !(Array.isArray(onboarding) || onboarding?.hasCompletedGuidedSetupFlow === undefined) && onboarding?.hasCompletedGuidedSetupFlow
        if (hasCompletedGuidedSetupFlow) {
            return;
        }

What alternative solutions did you explore? (Optional)

Proposal ready for review in https://github.com/Expensify/App/issues/49611#issuecomment-2370707723 @alitoshmatov

I couldn't reproduce it in our existing app. As far as I understand this is only happening in hybrid app(which I don't have much info about). But based on this C+ should get access soon.
@bfitzexpensify I am not sure what is the next step. If it is not an urgent issue I think we can hold till C+ get access and we can continue then

If it is not an urgent issue I think we can hold till C+ get access and we can continue then

Agreed. Setting to weekly and starring myself to keep checking in on this. We can revisit once C+ have access

@alitoshmatov you should be able to download the HybridApp from the Apple App store if you go to the "Expensify" listing.
I'm not sure if anyone will run into this bug now in practice, because if they sign-up on web either on new.expensify.com or expensify.com and meet the conditions to get redirected into NewDot, the tryNewDot.classicRedirect.dismissed NVP will be set to false. Meaning, the next visit on mobile will launch NewDot not OldDot and require clicking the "Try New Expensify" button.
To illustrate that:

Sign-up on expensify.com with a new account
Choose the "1-9" SignUpQualifier, so you get redirected to NewDot and the tryNewDot classicRedirect.dismissed NVP set to false
Complete the onboarding modal steps
Go to the Apple App Store and download the latest version available of the "Expensify" app (not the "New Expensify" app).
Log-in with those credentials
The NewDot experience will load

CC: @Julesssss @AndrewGable

I'm not sure if anyone will run into this bug now in practice

I agree with this. I don't think this normal user will come across this issue

@puneetlath I see you reported this one. Any thoughts on what I said here before we potentially close this out?

Issue not reproducible during KI retests. (First week)

Issue not reproducible during KI retests. (Second week)


Ensures the contrast between foreground and background colors meets WCAG 2 AA contrast ratio thresholds.

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Ensures the contrast between foreground and background colors meets WCAG 2 AA contrast ratio thresholds
Repro steps:

Launch  Angular powered Bootstrap (ng-bootstrap.github.io)
Tab to Date picker
Tab to Example and press enter.
Run AI4W and check for the results.

Actual Results:
Ensures the contrast between foreground and background colors meets WCAG 2 AA contrast ratio thresholds.
Expected Results:
Color contrast of the text with respect to background ratio should be greater than or equal to 4.5:1
Reduced test cases
Fix any of the following:
Element has insufficient color contrast of 2.88 (foreground color: #17a2b8, background color: #f8f9fa, font size: 9.6pt (12.8px), font weight: normal). Expected contrast ratio of 4.5:1
Environment: Chrome version <IP_ADDRESS>
====
This accessibility issue was found using Accessibility Insights for Web 2.35.0 (axe-core 4.4.1), a tool that helps find and fix accessibility issues. Get more information & download this tool at http://aka.ms/AccessibilityInsights
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
15.1.1

Attachment:

Hey @msftedad
You're not on the right repository. Here's the good one for ng-bootstrap: https://github.com/ng-bootstrap/ng-bootstrap
Closing this issue here


Support actions for Windows Apps

Would it be possible to support actions for Windows Apps? So that one could open an elevated terminal, for example?
These are the actions available in the Windows menu/search for the Windows Terminal for example (in German):

I added actions to run as admin and open the store page of the app. Check out the new version (needs a restart of keypirinha to work properly)
I really wanted to add a action to open the app-settings, but I didn't found a way to do it :/


fix issue#240: POST invalid workflow API will always triggers Graph.BootLiveCD

resolves https://github.com/RackHD/RackHD/issues/240
Throw error if the graph name is missing or in wrong format.
@RackHD/corecommitters @iceiilin @WangWinson @cgx027 @pengz1

:+1:

👍


Redisson does not reconnect after failed DNS Monitor change

Editing and repurposing this issue after finding the underlying cause. Originally this issue was to improve the error messaging. The underlying problem is a failure to reconnect after a failed DNS change. After this scenario plays out, there are no connections in the pool, and the error message we end up seeing is an ArrayIndexOutOfBoundsException
Expected behavior
Reconnection to redis node
Actual behavior
after DNS monitor detects a change to an ip it cant reach, subsequent dns change to a good ip does not restore connections.
Steps to reproduce or test case

connect to a single redis master node
add a host file entry for some bogus ip address,
watch redis connection failures
remove the bogus host file entry, so that DNS monitor gets the good ip address again
re connection does not happen

Redis version
azure cache for redis 4.0.14
Redisson version
3.14.0

discovered some difficulty triggering the DNS change via hosts file

It's a bad practice, since these entries are cached by netty resolver. Try to use mara dns instead.


Reseller credits duplication

Only subtract up to the maximum number of credits from a reseller
See #9

Thanks for finding and sharing how to fix this bug :)


Ability to launch a notification in Sublime?

this is probably NC (not critical) for the MVP but should come quickly after
can layer this in to failures w/ version logic/ or PiecesOS not installed
also can be used to launch success notifcations "selection successfully saved to pieces", xyz

Yup there is a message dialog like the save prompt for example we can work on that


feat: created nullable object option without getter

This PR is for supporting nullable objects without a getter function (mentioned in issue 203)
I took some inspiration from @roertbb solution, but solved it a little bit differently. I haven't changed the ModelValueType definition but rather made a new NullableObject class that accepts a value of the type NestedModelDefinition, since this is in fact what it should receive. It also accepts a second arg that indicates if the NullableObject should default to null (I made it so that the complete object definition will always be given while creating an instance of the class)
In general, I think that @kettanaito request here to implement it without a getter function makes a lot of sense as it is more consistent with regular nested objects (because we don't define them with a getter function as well)

Thanks @jchris !!
I would also like to get an approval from someone that is more familiar with the code, but so far no luck with that :(
@kettanaito I saw you mentioned that you are rewriting the package to release a new major version. But is there a chance to insert this change to the current major version? Thanks !!

This seems dead so @kettanaito Any chance you can review this as requested by OP? Quite keen on this functionality. Thanks 🙏

Released: v0.14.0 🎉
This has been released in v0.14.0!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/data@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


added 0/1 knapsack in Java -> Dynamic-Programming. Issue #72

🛠 Issue #72 0/1 Knapsack - Dynamic Programming approach
👨💻 Changes proposed

Added code for 0/1 Knapsack (Dynamic Programming)
Updated README.md

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Added 0-1 Knapsack (Dynamic-Programming) in Java
recursion -> memoization -> tabulation -> space optimzation
Starting with the naive recursive solution and gradually improving time complexity with memoization, then addressing the space complexity with tabulation and finally with space optimization.
Reorganized README.md and renamed 0/1 knapsack java file
Resolved merge conflict
(mentioned time complexity and space complexity of each approach)

@harshraj9988 Please resolve the MERGE CONFLICT

Resolved Merge Conflict.
Kindly review again.

@harshraj9988 Please don't update the README.md as there has to be some changes in it, please remove updations of README.md from this PR

sure! will remove README.md from this PR


Improve skipped, incomplete, risky, error, unkown and warning output

@ventrec @ecrmnn what are you thinking about this? Could you give feedback if changes are needed?

Ping @ventrec @ecrmnn


Add an example for describing a list in a schema.

Add the following to README.md after the section on Recursion:
List
You can specify a list using the Include validator.
This schema:
person: list(min=1, include='human')
---
human:
    name: str()
    age: int()

Will validate this data:
person:
  - name: Bill
    age: 50
  - name: Jill
    age: 20
  - name: Will
    age: 10

I am willing to make this change.  I could not create a branch using the GUI (presumably due to permissions).  It would be my first contribution on Github!

Thanks for your interest in Yamale. Have you looked at this section: https://github.com/23andMe/Yamale#lists
Does that provide what you need?


Retrieve idTokenClaims for logged in accounts using msal-browser 2.x

Please follow the issue template below. Failure to do so will result in a delay in answering your question.
Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
I'm using msal-browser 2.0 together with B2C. I'm successfully authenticating a user and retrieves an idToken (together with idTokenClaims) in handleRedirectPromise(). The data received in the idTokenClaims.groups defines what groups the logged in user is associated with. This data is needed client side to customise the user experience (e.g. to give access only to the user authorised pages).
However, I can't find a way to programatically ask msal-browser 2.0 for e.g. the idTokenClaims. This becomes a problem on a full page reload where the handleRedirectPromise() will be called with a null response (as per the docs).
My questions:

How come msal-browser 2.x doesn't expose an api to get the id token data for the already logged in accounts (as in getAccounts())? This seem to have been changed from comparing with msal-browser 1.0 where id token data can be obtained by calling getAccount(). Can't seem to find any docs about best practise for this use case. Or am I missing something?
If the answer on the previous questions is that it's not possible to programmatically retrieve, then what's the best practice to store and retrieve idTokenClaims for logged in accounts using msal-browser 2.x? One idea I have is to save the token response from handleRedirectPromise() in e.g. local storage and then read it from there when needed, but I can't understand why the library don't just expose this data to the consumer via an api (since the lib have stored it in session/local storage).

@dangrahn Thanks for bringing this to our attention. This can be possible and I am marking this as an enhancement to our library. Please watch out this space for a PR.

@jasonnutter  Any update on this?

Thanks for doing the work @jasonnutter - any idea when that will be released?

When will this feature be released?

Any indicative timeline on when this will be released?

<EMAIL_ADDRESS>is now available on npm.


ios safari not reacting to gyroscope and fullscreen

on ios (10) safari does not go to fullscreen when clicking the fullscreen or vr button (it does switch to vr mode and changes fullscreen button, but only inside the iframe) and vrview does not react to the gyroscope: check here https://developers.google.com/vr/concepts/vrview-web
and also in current master branch

Dupe: https://github.com/googlevr/vrview/issues/31


LSP Config for F# using FsAutoComplete

Tested on Ubuntu and Windows.
Ran make lint

Ok, so tested and g2g?

Yep, looking good.


Fix: version from tag_name is incorrectly parsed

Codecov Report
Patch and project coverage have no change.

Comparison is base (0382c2a) 87.78% compared to head (f2eb9eb) 87.78%.
Report is 2 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   87.78%   87.78%
=======================================
  Files           1        1
  Lines         131      131
  Branches       25       25
=======================================
  Hits          115      115
  Misses         14       14
  Partials        2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[JS]: Closures in Javascript

Description 📜

What are closures?
How closures work?
What are the advantages of closures?
What are the Disadvantages of closures?

Domain of Contribution 📊

[x] JavaScript

@paramsiddharth Sir Can you please assign me this issue that I just created.

And please remove the java tag .It has been added automatically.

/assign

@daminimehra Assigned! Please note that a PR has already been merged for documentation on closures in JavaScript. If you wish to extend it or contribute a different format, please continue.


[iGemm] Workaround a tuning issue of non-xdlops gemm

When E == EPerBlock, the non-xdlops v4r1 fwd kernel failed on Vega20 (ROCm 3.5/3.7) with scratch memory
Added a restrict that E (i.e., GemmK) must be multiple of 2 x EPerBlock

Seen with the 3.9 as well.

@zjing14 Since this is a hot-fix, please open an issue to track the failing configuration, and refer to this PR.

What is the reason of the issue?

@atamazov this is a workaround about issue:https://github.com/ROCmSoftwarePlatform/MIOpen/issues/659


Sign in and account creation using email only

Medium offers sign in and account creation using only your email address. Is this an option (abstraction layer) you'd consider implementing?

Passwords are neither secure nor simple. They’re hard to remember or easy to guess, everyone re-uses them (even though they know they shouldn’t), and they’re a pain to type on mobile. They don’t even keep you that safe.

Btw, thank you. This has been my go-to authentication library for Scala since version one :)

Nice idea. I think this could be implemented with a token provider instead of a credential provider.


Ui Issues - Lightning Settings

https://xd.adobe.com/view/1b7355c9-e125-47f4-895c-1a6d1d5dc438-4e20/screen/176c8469-7f2e-4482-8e42-be67502a2730/
Heading is missing
Remove row dividers and add cards
check icon sizes

Verified this issue on dev app v2.0.71(451)


Dropping support for Ruby < 3.2 in 2.0

I’m considering whether we should drop support for Ruby versions below 3.2, since 3.2 is where the Fiber[] and const_added features we depend on for Phlex::Kit were introduced.
We could still maintain a Phlex 1.0 branch supporting Ruby >= 2.7 for a few more years. I just think it might be easier to say, “here’s the features in 2.0” without having to qualify which Ruby versions have access to them.

Ship it and solicit https://github.com/ruby-next/ruby-next implementations from anyone who finds themselves locked out and unable to upgrade Ruby.


Add --env-file param and improve overall env-var loading

Please take a look.

Codecov Report

Merging #202 (7a5f03f) into 2021.6.x (3edef9e) will decrease coverage by 0.00%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##           2021.6.x     #202      +/-   ##
============================================
- Coverage     39.63%   39.62%   -0.01%
============================================
  Files           623      624       +1
  Lines         61572    61588      +16
============================================
+ Hits          24402    24407       +5
- Misses        34368    34378      +10
- Partials       2802     2803       +1

Impacted Files
Coverage Δ

pkg/logger/logger.go
68.29% <ø> (ø)

pkg/cli/env.go
25.00% <25.00%> (ø)

pkg/handle/handle.go
85.71% <0.00%> (+7.14%)
:arrow_up:

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 3edef9e...7a5f03f. Read the comment docs.


I am looking for support in WebSocket samplers plugin. Do we have any workaround for plugins like this?

I am currently using external plugin 'WebSocket Samplers by Peter Doornbosch' in JMeter GUI to test WebSocket. I want to implement the same using JMeter DSL for continuous integration. Can you please help me with an option to work on this?

Hello, thank you for bringing this! Websockets is something we should definitely add to the DSL.
Have you checked this section in user guide.
Do you feel confident to contribute with an implementation? If not, is ok, just evaluating if we can get additional help from community to cover several pending features. Here are some guilde lines for contribution.
Additionally, can you share some examples on how you use the plugin? Mainly trying to understand what main features you need.

Thanks for your quick reply. I will share some examples of how we use the plugin soon.

We do have it in the backlog, but currently is not at the top of it. For the meantime you can use wrappers as commented previously. But we are eager for users to demonstrate interest in this and help prioritize this higher.
Any particular needs? can you detail how are you currently using the plugin in JMeter? So we can start evaluating it's  design (required vs optional fields, sane defaults, etc).


Managed database schemas

What problem are you facing?

It would be ideal to support managing database schemas declaratively using this provider.
How could Crossplane help solve your problem?

Add a Schema managed resource to this provider? I'd love to see a sketch of what that managed resource would look like as YAML. We could probably take inspiration from https://github.com/schemahero/schemahero here.

Any updates on adding support for creating schemas?

so i'm using this provider to create a database and a role in AWS RDS instances.
i'm able to do it, and also set the role as the owner of the database.
but, due to how AWS setups RDS (ie, master user is not a superuser and new roles are not owners of db schemas), i also need to change the schema owner (to the new role i'm creating).
the command i need to issue is:
ALTER SCHEMA public OWNER TO <role>;

note: need to connect to the database first with the master/root user
as far as i understand from looking at the provider code, neither Database or Grant supports this use case.
is anyone using this kind of stuff on AWS RDS? how are you configuring this stuff ?
i might be able to contribute to the provider, but would like to first talk with its maintainers to understand the best way to go about this:

create a Schema type like @negz said? would work like the Database one, just not sure how it could just change ownership of something that already exists and not only creating a new schema with an owner ...
increment Database scope somehow ( add additional fields to control changing schema ownership or not? ... )
just set the schema owner to the same owner of the database right after creating the database? (this fits my use case but is probably the less flexible one)
...

thank you.

Hi,
I'm also interested on this feature, being able to manage Postgre schemas on a RDS cluster.
I suppose adding a Schema object would be nice ? Not sure you need any reference to the Schema on the Database ? The Schema owner would be set on the Schema specs ?

@Bastichou, @dneves - see PR #181 for CREATE SCHEMA support (including owner).
I think @negz is talking about a full fledged schema hero replacement (Table etc)


when i share image with text length above 3 lines, text area seem doesn't expand

Finally, I found link:
https://github.com/twitter-archive/twitter-kit-ios/pull/9
And I fixed and rebuild your lib. Thank you!


--quiet should be true by default

Actually, no. Maybe later on create an option called verbose and deprecate quiet or something.


fix: pageserver_ondisk_layers metric

Start by adding a wrapper for LayerMap for these metrics concerns which allows using PersistentLayer methods.
Very much an early draft now, want to get a CI run for any surprises.
Eventually fixes #3705.
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Need to check if we have any test for this metric, if not, fix that and then last split the metric up.

deref-abuse
Did a poll internally, no one is horrified.

merge-allure-reports failed (debug)[https://github.com/neondatabase/neon/actions/runs/4374610146/jobs/7654510396]?
+ aws s3 cp --only-show-errors s3://neon-github-public-dev/reports/pr-3775/lock.txt ./lock.txt
fatal error: An error occurred (404) when calling the HeadObject operation: Key "reports/pr-3775/lock.txt" does not exist

It's non-trivial to move all affected metrics in (I wasn't expecting there to many other) so drafting while waiting for time to look at this.

Mentioned for generation/version number which would be handy in number of cases.

Sad to leave this out because of scope creep.


[FEATURE REQUESET] Captions and controls

Would be great to have the ability to add captions as well as controls for fullscreen and previous/next if used in a gallery.

Sorry, this library intends to do image zoom only.


Adding Changelog for v0.3.0

What type of PR is this?
/kind documentation
What this PR does / why we need it:
This adds a changelog for v0.3.0. Approving this PR indicates that you approve of a v0.3.0 release based on this commit.
Does this PR introduce a user-facing change?:
NONE

/cc @bowei @danehans @hbagdi @jpeach

Adding a hold on this until at least 2 maintainers LGTM this to ensure we have consensus on this release.
/hold

/lgtm

/lgtm
/approve

/hold cancel


Docker files become very large

Hi.
I set the tor to fork 50 processes so I can have 50 open ports.
The problem, the docker log file takes more than 80% of the server space! the total space is 917G, and the docker files are around 700 GB!
WAW. ps the docker has only once running container. which is rotating-proxy
even cleaning Docker dos not help. the only way to reclaim the wasting space is by uninstalling the docker and deleting the docker folder altogether.
Please advice

This is a docker issue.  See https://docs.docker.com/engine/logging/configure/ for possible fixes.
You can change the docker daemon - system wide, or configure it per container.
Configuring for one service in docker-compose.yaml:
services:
  proxy:
    ... existing lines ...
    logging:
    driver: "json-file"
      options:
        max-file: "5"
        max-size: "16m"

This would have a log for your container with max size 16MB.  Having huge log files fills up the drive, and makes docker-compose logs scroll for awhile.


MB-12620 Change PPM advance requested reads to use new columns

Jira ticket for this change
Summary
We're renaming a couple of PPMShipment advance fields (see thread for details). This is step 4/6 of the renaming process, switching all reads of the data to use the new columns.
Renamed fields (actual case varies based on the standards for the file, e.g. pascal case vs camel case):
advanceRequested -> hasRequestedAdvance
advance -> advanceAmountRequested

Change Log
Main Changes

[x] Update any file that reads the advance requested fields to read from the new advance requested fields.

There's a lot of files changed, but generally all the changes are to swap which fields are used for reads.

[x] Updated src/components/Office/ShipmentIncentiveAdvance/ShipmentIncentiveAdvance.jsx to re-use some of the same logic used on the customer side for calculating max advance and for the formatting of the advance and incentive values.

The formatting was updated to match the customer side more. I asked about the difference in this post and we discussed it in this thread

[x] Update pkg/handlers/ghcapi/internal/payloads/payload_to_model.go to write the advance and advance requested fields to both the new and old columns.

This function hadn't existed when I worked on #8604 and I didn't get to reviewing #8647 until after it merged so I'm making that update in this PR.

Minor Changes

[x] Fix prop error in src/pages/MyMove/Profile/EditServiceInfo.test.jsx

Testing the Changes
So the thing about these changes is that they really shouldn't be visible to the user. It's all about changing what database columns we're reading data from. So, the testing should be to make sure everything still looks and works like it should when it comes to requested advances.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Checking storybook

Access local storybook
Check the ShipmentIncentiveAdvance component and make sure it still looks good.

One thing to note is that the "Maximum Advance" formatting changed a bit in this PR, see thread for details

Check the AdvanceForm component to make sure it all looks good.

Checking customer app

Access the local customer app
Sign is as a PPM customer. Any of the @ppm.unsubmitted users should be good enough.
Go through the PPM flow and ensure the advance page and review page still work as expected wrt to advances (well and really everything, but I only messed with advances in this PR).

Verification Steps for Author
These are to be checked by the author.

[x] Request review from a member of a different team.
[x] Have the Jira acceptance criteria been met for this change?

Note that the AC are really for all steps to be completed, but I'm considering them done for this step.

Verification Steps for Reviewers
These are to be checked by a reviewer.
Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output

Screenshots

Messages

:book:
:link: MB-12620

  Generated by :no_entry_sign: dangerJS against 14fb988f21dbff1cc8eb38c07376e3d7de5ac141


Update gitlab_docker.md

Replace Python 3.7 with 3.12: Python3.7 is End-of-Life
Update .gitlab-ci.yml example file: global image and cache definitions are deprecated

Thanks for your contribution!


Boolean value default to false

Some of the boolean options defaulted to true. However due to browserify use minimist to parse the subargs, there's no way to overwrite a default true option to false in command line.
browserify -t [sassify --auto-inject --base64Encode] //base64Encode set to true, auto-inject set to true
browserify -t [sassify --auto-inject] //auto-inject set to true, but base64Encode still set to true

There's basically no way to set base64Encode to false in command line invocation as it defaults to true.
I think we should change the boolean options default to false. Please kindly comment. @davidguttman

Sorry for the delay on this -- slipped through.
I agree with that change.

In v1.0.0


All mails in queue

All emails are in queue even the test mails

Can anyone help me with this

This is not a bug. Use discussions.


Feature request : Share expiration, account management & usernames instead of e-mails

Hello,
Very good project I like it so far.
I've installed it on UnRAID and it works perfectly, if anyone needs help with that I'll be happy to help.
I've got a few requests if possible :
My domain is shared between my and my friends so we can share files with each others
I'd ideally want to manage things this way :

Manual account creation by the admin only (first account ever created for example? I don't know if the accounts have an ID or something)
Accounts can be created with usernames or email (so we don't need to share emails)
Share expiration/max size when not logged in is set and cannot be changed

I hope I'm not asking for too much
Thanks a lot for the work already done, I'll be following this project closely.

Hey thanks for your feedback:)
Currently I'm working on admin functionalities, like managing configuration from the frontend, managing users, and audit the shares that were created.
You're right, I'll add an username attribute!
I'm not really sure what you mean with "Share expiration/max size when not logged in is set and cannot be changed", could you explain this further?

And if you have time, could you create two separate issues for the admin and email feedback, this would help me to track the tasks better :)

Hello,
What I meant is I'd like it if it was possible to put different settings when you're logged in and for when you're not logged in.
For example :
Logged in :

5GB max size
Can change the expiration date

Logged off :

5mb max size
Cannot change expiration date

Thanks a lot for taking my suggestions in account, I will create separate issues for the other topics, sorry I didn't do that first I'm not really used to github!

Okay I see. There should be different quotas for anonymous and signed in users. This could get a bit complicated to implement and there are other features that I want to prioritize first.
But definitely looking forward to implement this in the future :)

Okay I see. There should be different quotas for anonymous and signed in users. This could get a bit complicated to implement and there are other features that I want to prioritize first.
But definitely looking forward to implement this in the future :)

Hello,
Is a 2 factor authentication for the admin planned ?
so that the admin account is better protected ? with an authentication app. like google authenticator or Authy
I would be happy if this could be implemented.

Currently I don't plan to implement this feature. In my opinion Pingvin Share is only used by people that the hoster knows so quotas are not necessary. If I miss something or you have a good use case feel free to reopen the issue!


Images don't show up when attached to email

RainLoop version, browser, OS:
1.10.5, Chrome/Firefox, Linux/Windows
Expected behavior and actual behavior:
Images should be displayed in email. It doesn't and images are replaced by some grey space.
Steps to reproduce the problem:
Try to display an email with images attached to it (images linked from a website will show up)
Logs or screenshots:
Grey space instead of image :

Don't have this problem. Using FF ESR 45.4.0 on Windows

Shows up in email I send to myself. During composing, in send items and in inbox.

Hmmm, weird. All sent messages (saved in Sent folder) or received messages don't show images. I forgot to mention that the problem appears after a Rainloop update : 1.10.3 to 1.10.4
Another thing : when I save a message in drafts and if I reopen it, my signature image is greyed out too.

Try updating to the latest: <IP_ADDRESS>.

I'm already up to date : RainLoop <IP_ADDRESS>

Then I'm out of ideas right now. Perhaps @RainLoop can quote on this.

Adblocker?

I tried with and without it with no noticeable effect.

Try right-clicking on the image, call the Web Inspector and see if the HTML wasn't changed by some browser plugin or something. Also try opening the image in a new tab/window (get the URL from right click or Web Inspector).
First, we need to see if it's an issue with Rainloop or your browser.
Also, running any Rainloop plugins? Try enabling/disabling the local proxy for external images option, somewhere in Rainloop (admin area maybe, can't recall).

Hello,
All these images are base64 encoded. When inspecting the form of URL I noticed that the base64 payload is truncated and way too short. For example : data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAAAXNSR0IArs4c6QAAAARnQU1BAACxjwv8YQUAAAAJcEhZcwAADsQAAA7EAZUrDhsAAAANSURBVBhXYzh8+PB/AAffA0nNPuCLAAAAAElFTkSuQmCC

Food for thought...
https://sendgrid.com/blog/embedding-images-emails-facts/
https://www.campaignmonitor.com/blog/email-marketing/2013/02/embedded-images-in-html-email/

Inline Embedding (Base64 Encoding)
Pros

Much simpler to achieve
Much faster to do
Requires much less deep dive into MIME and application code

Cons

Can really increase size of emails especially if you use more than one image
Most likely blocked by default in many webmail services
Blocked completely in Outlook

Upon further testing, I think this is related to the browser. For example, the first image (You Tube) on this post (which I get in my Inbox as part of my RSS feeds) always shows up as a gray placeholder. Sometimes, scrolling down the message shows other images as gray but as I keep scrolling they load up. Other times I have to refresh the browser for them to load.
This happens in Chrome and Firefox, Windows.
At home, on Safari, OS X, all images show up correctly.

I never see these blank images outside of my Rainloop webmail. Nor on Chromium, nor on Firefox.
I already encountered this problem before, after a Rainloop upgrade (fixed in 1.9.4 version if I remember well). And this time, this is the same scenario : after an upgrade (1.10.5), the problem appeared.

I explained it wrong. It's related to Rainloop and the browser. I never get this blank images outside of Rainloop either. But on Safari, I always get all images, inside or outside Rainloop.
And now that you mention it, I remember always having this problem of some images appearing gray on scolling (specially long messages) but scrolling up and down or refreshing the browser eventually loaded them up, so I just thought it to be a "natural flaw" in Rainloop fetching remote images.

I now also face this issue when sending mail with inline images pasted from the clipboard. Sometimes it shows no image in my sent item / receiver inbox and sometimes the gray box where the image should be. Difference in source code of the e-mail is one or two base64 encoded parts for the image. Why two? Don't know but they are different and that e-mail shows the gray box, the other with one part for the image shows nothing.
@RainLoop, do you want the .eml for the two emails?

This is actually a duplicate of #1252. @blakkheimgw, can you close this one?

In my tests the problem occurs with png images larger than 60KB when inserted into the body of the message.


[dev] TornadoVM collection types return backing array

This PR addresses  Github issue #25
This patch includes:

Get backing array for wrap-types (e.g., ImageByte3#getArray())
Clean-up TornadoVM Wrapper-APIs

All unittests are passing as well as kfusion-tornadovm.

Looks good @jjfumero
Couple points for discussion:

Depending on the class type we use either getStorage or getFlattenArray. Shall we use the same name convention for all?
I think we should start adding proper JavaDoc to our API so we can compile the HTML API

@kotselidis ,
getStorage is an old API. I will remove that.  I call getArray for types that are wraps-arrays, and getFlattenArray for types that wrap a matrix type. We can call all of them getArray.
What do you think?
Concerning the Javadoc, I will open a task to work on that on a separate issue.

I think the name convention you propose is good.
Regarding Javadoc, yes this shall be a different Issue

@stratika ,  I call getArray for types that are wraps-arrays, and getFlattenArray for types that wrap a matrix type. See previous comments

@stratika , I call getArray for types that are wraps-arrays, and getFlattenArray for types that wrap a matrix type. See previous comments

but the getFlattenedMatrix is used.

Sorry, I meant getFlattenMatrix Shall we go for FlattenedArray instead?

Sorry, I meant getFlattenMatrix. Shall we go for FlattenedArray instead?

I do not mind. I pointed out because earlier it is mentioned as Array. I think both are correct.
Perhaps FlattenedArray is more precise?

Changes done


Media-repo requests whoami even if the token is not provided in request

This probably affects performance:
When a client send request for a preview, it may not provide accessToken. But following code even with an empty token will try to send whoami request to matrix server
https://github.com/turt2live/matrix-media-repo/blob/539f25ee75ba6cdbb0410314b29978f4b8b1d7fe/src/github.com/turt2live/matrix-media-repo/api/auth.go#L36-L43
Same situation in AccessTokenRequiredRoute - you should probably immediately generate an error without sending a request to the server if the token is empty.
P.S. Warnings about token verification fail are not informative. Probably they should be supplemented with information about the error?

Ups, seems i missed this:
https://github.com/turt2live/matrix-media-repo/blob/539f25ee75ba6cdbb0410314b29978f4b8b1d7fe/src/github.com/turt2live/matrix-media-repo/matrix/auth.go#L13-L17
But still actual: warning about failed token verify when token was not present confusing.

The token may be required for all routes in the future, so it is a valid warning.


Random Flickering Blue Light

Hi,
I did everything as per the steps mentioned. Here is my user_config.h file. When I flash the firmware and reboot the esp, the ESP blue LED flickers rapidly. When I connect to Arduino IDE serial 115200 baud, I get gibberish dump. What am I doing wrong?
#ifndef USER_CONFIG
#define USER_CONFIG
typedef enum {SIG_DO_NOTHING=0, SIG_START_SERVER=1, SIG_SEND_DATA, SIG_UART0, SIG_CONSOLE_RX, SIG_CONSOLE_TX, SIG_PACKET } USER_SIGNALS;
#define WIFI_SSID            "MyWiFiSSID"
#define WIFI_PASSWORD        "MyWiFiPassword"
#define WIFI_AP_SSID         "RangeExtenderAP"
#define WIFI_AP_PASSWORD     "none"
#define MAX_CLIENTS	     8
#define MAX_DHCP	     8
//
// Size of the console buffers
//
#define MAX_CON_SEND_SIZE    1024
#define MAX_CON_CMD_SIZE     80
//
// Define this if you have a status LED connected to GPIO LED_NO
//
#define STATUS_LED      1
#define LED_NO		2
// Select the GPIO init function according to your selected GPIO
//#define SET_LED_GPIO	PIN_FUNC_SELECT(PERIPHS_IO_MUX_U0TXD_U, FUNC_GPIO1)
#define SET_LED_GPIO	PIN_FUNC_SELECT(PERIPHS_IO_MUX_GPIO2_U, FUNC_GPIO2)
//#define SET_LED_GPIO	PIN_FUNC_SELECT (PERIPHS_IO_MUX_MTDI_U, FUNC_GPIO4)
//
// Define this to support the "scan" command for AP search
//
#define ALLOW_SCANNING      1
//
// Define this to support the "sleep" command for power management and deep sleep
// Requires a connection of GPIO16 and RST (probably not availabe on ESP01 modules)
//
#define ALLOW_SLEEP         1
//
// Define this if you want to have access to the config console via TCP.
// Ohterwise only local access via serial is possible
//
#define REMOTE_CONFIG      1
#define CONSOLE_SERVER_PORT  7777
//
// Define this if you want to offer monitoring access to all transmitted data between the soft AP and all STAs.
// Packets are mirrored in pcap format to the given port.
// CAUTION: this might be a privacy issue!!!
//
#define REMOTE_MONITORING  1
#define MONITOR_BUFFER_SIZE 0x3c00
// Define this if you want to cut packets short in case of too high data rate
#define MONITOR_BUFFER_TIGHT 0x1000
// Define this if you want to silently drop any packet that cannot be send to the monitor
//#define DROP_PACKET_IF_NOT_RECORDED 1
#endif

Do you use an ESP-01 module? Try to compile it with SDK 1.5.4. this works for me.

Yes I tried it on ESP-01. How do I compile with SDK 1.5.4?

Switch in the Makefile of esp-open-sdk to:
VENDOR_SDK = 1.5.4
You can get the SDK from:
http://bbs.espressif.com/download/file.php?id=1469&sid=9c7919d26a8c0bacacd54c5168d0b698
First you can test the firmware by flashing my precompiled images from the firmware_1.5.4 directory.

Will do and update. Thanks

Tried your 1.5.4 bin files and it worked perfectly. Gonna recompile 1.5.4 and try with my settings and see if it works well. Will keep you posted.

Hey, I recompile to 1.5.4 and tried to flash again. When I Compile the firmware files, it always creates 0x00000 and 0x10000. However, 1.5.4 looks for 0x40000. I made the necessary changes I clouding my wifi details and flashed to esp successfully. When I open the serial window, I can now see the status of connectivity however, it always keeps saying trying to connect. What's going wrong?

That is the normal behavior when it cannot establish the connection. "show" gives you the right parameters? Can you connect to the esp's AP? Try to edit the params via the commandline (serial or wifi over the stationAP) and then "save" and "reset".

What can I say martin. Simply Awesome! It works perfectly with 1.5.4. I managed to telnet to the ESP nd changed the parameters too. Thank you so much. You have a great weekend man. Cheers!

I've my Raspberry Pi set as Access point with the following: inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>. I am trying to use the ESP to range extend for the Pi. By default, the firmware sets the ESP to <IP_ADDRESS> for router access and <IP_ADDRESS> for network access. I noticed that when I telnet to ESP and type set network ip-addr where I can define a IP range like <IP_ADDRESS> and ESP can then be booted with this network range. I can successfully connect my ESP to my RPi AP's SSID. However, both these devices seems to be in different subnet. How can I make these 2 devices to be on same subnet so that I can use the network access via ESP to access a webserver running on Raspberry Pi (for instance at <IP_ADDRESS>/index.html?

As the ESP works as NAT-router, the AP interface and the STA interface of the ESP must be in different subnets (I first tried to make it work as a real layer 2 repeater, however this fails, as the closed WiFi-routines do not allow for sending the required frames).
From the AP-side (network 192.168.4.x) of the ESP you can transparently connect to the outside networks (e.g. <IP_ADDRESS>.) and the Internet. So, a station connected to the range extending ESP network (e.g. a node <IP_ADDRESS>) should be able  to connect to <IP_ADDRESS> without any problems (given that it is configured via DHCP and has received the ESP as default gateway <IP_ADDRESS>).
To connect from the Internet to a host in the internal network (e.g. <IP_ADDRESS>) define a port forwarding on the esp, e.g.
portmap add TCP 80 <IP_ADDRESS> 80
and connect to the port 80 of the ESP.

Ok, I don't know what happened, but it is working now. Previously I was trying to access by typing the host name like raspberrypi.local and I couldn't connect. Now I type the IP address of the AP and it works perfectly. Thanks for the detailed instructions.

I guess this will be the last on. I'm using the monitor mode with wireshark. Previously, I used Promiscious mode to capture probe request. When I try nc 172.xxx.xxx.xxx 8888 | sudo wireshark -k -S -i - the wireshark app launches and I only see TCP traffic. When I try to filter using wlan.fc.type_subtype == 0x04, I see nothing. I did try to enable and disable my mobile phone wifi when I did this test to capture a probe. It didn't appear. I then tried http.user_agent == "iPhone" to see ig I can see my iPhone, it too didn't show any result, What am I doing wrongly?

Nothing wrong: you are looking for IEEE802.11 frames (wlan.fc.type_subtype "Probe Request").
These frames are only visible, when you are physically monitoring the wifi. Can be done with the ESP, but not while in stationAP mode. With the monitoring feature of the repeater you can observe the data frames that are going through the AP including ARP/IP/TCP/UDP, but not physical IEEE802.11 frames. Sorry about that - not possible at this level...
However, the advantage of this monitoring is, that you see decoded frames, not just WPA encoded stuff.

Hi,
I was fiddling around the repeater for a while and I realized that, each time a device connects/ disconnects from the repeater, ESP prints the device's MAC address join, device MAC address leave. Is there any way that I can send this info via MQTT?

Definitely possible. It would need to integrate e.g. this project https://github.com/tuanpmt/esp_mqtt . The repeater itself can run any TCP/IP application independantly of its repeating (routing) functionality.  Publishing should be done in the WiFi callbacks.
However, nothing I would like to add in the mainline development as it is pretty special feature.

Is there any .ino files which I can play around with and see if I can add it?

Sorry, no .ino as it is not a Arduino project but based on the native C SDK (esp-open_sdk).
Maybe I find some time to set up a fork - This functionality might be even interesting for smart home, as it indicates that a certain user has probably entered the covered area (e.g. a home)...
What about this MAC-scanner alone? Would it be of interest to have a scanner, that publishes MQTT events as soon as an new (or a certain well-known) MAC enteres the scene? (without router?) Could probably easily be done as a sketch.

That's precisely the use case I have. I've been trying out a few PCAP tools available for esp. but all of them have the same result as to show the raw output on screen. My programming skills are not that fantastic and I'm banging my head to try and find a suitable method where I can listen to probe request via ESP and send it over MQTT to a server for further processing. A typical tool like wireshark allows one to liste. To probe requests on networks. However, your script does it nicely. It prints the MAC of someone when connected and again when they leave. I'm not concerned about specific MAC address as my primary use case is to find occupancy in a big area like office or a lab. Any help could be much much appreciated.

The router option is not mandatory as I can use the router and the scanner in conjunction to achieve the result.


Cloud Run Jobs customization

Hi I'm trying to customize a few things about the cloud run jobs deployment, but can't find any documentation or examples on how to change:

cpu/memory resources
timeout
number of retries
VPC connector
total number of tasks

Would love some help! Thanks

Thanks for the issue and will add some more documentation to the example config. Most fields can be set in either the job spec or job container fields. https://cloud.google.com/run/docs/reference/rest/v1/TaskSpec for a reference.
The total number of tasks is calculated by goblet based on the number of task_id set for the specific job.
ie
app.job("test", task_id=0)
app.job("test", task_id=1)
app.job("test", task_id=2)
will result in number of tasks to be 3.

By looking at the code it was possible to see how one could change this values by directly controlling the YAML that goblet deploys the Job with: https://github.com/goblet/goblet/blob/0c0146a0a54eacff48fff4555e14c4d60f7e20f0/goblet/resources/jobs.py#L46-L88
For example, I now deploy with this in my .goblet/config.yaml:
{
	...,
    "job_spec": {
        "serviceAccountName"<EMAIL_ADDRESS>        "timeoutSeconds": "3600",
        "maxRetries": 3
    },
    "job_container": {
        "resources": {
            "limits": {
                "memory": "8Gi",
                "cpu": "2000m"
            }
        },
        "env": [
            {
                "name": "GOOGLE_PROJECT",
                "value": "<project-id>"
            },
            {
                "name": "GOOGLE_ENV",
                "value": "test"
            }
        ]
    }
}

I believe you can set the VPC connector in this YAML as well, but haven't checked.

@rasnes thanks for the example. i have updated the example config.json and will try to add more examples there going forward as it is a pain to read the code or go through the gcp docs to figure out where to place what.
also it looks like the vpc connector info is actually set in the metadata fields, so added a pr that allows those fields to be passed in as well (#262 )

Thank you both, this was very helpful!
job_annotations - who woulda guessed.