Nvidia Game Ready Drivers

Package Requested

[ ] I would like someone else to build the manifest.
[x] I would like help so I can submit the manifest.
[x] I have performed a search and couldn't find this package.
[ ] I think there is a new version available and I have provided the URL.

Manifests:
Desktop:
PackageIdentifier:   # publisher.package format (example: "Microsoft.WindowsTerminal")
PackageVersion:      <IP_ADDRESS>
PackageLocale:       International
Publisher:           Nvidia Corporation
PackageName:         Nvidia Game Ready Driver
License:             Proprietary
ShortDescription:    Game-ready graphics driver for Nvidia GeForce GPUs
Installers:
 - Architecture:     x64
   InstallerType:    exe
   InstallerUrl:     https://us.download.nvidia.com/Windows/471.11/471.11-desktop-win10-64bit-international-dch-whql.exe
   InstallerSha256:  c81810bd968ba6efe0b1052301c242d081dfd843d91fc9e2e46cbce3fc527719
ManifestType: singleton
ManifestVersion: 1.0.0

Laptop:
PackageIdentifier:   # publisher.package format (example: "Microsoft.WindowsTerminal")
PackageVersion:      <IP_ADDRESS>
PackageLocale:       International
Publisher:           Nvidia Corporation
PackageName:         Nvidia Game Ready Driver
License:             Proprietary
ShortDescription:    Game-ready graphics driver for Nvidia GeForce GPUs
Installers:
 - Architecture:     x64
   InstallerType:    exe
   InstallerUrl:     https://us.download.nvidia.com/Windows/471.11/471.11-notebook-win10-64bit-international-dch-whql.exe
   InstallerSha256:  c81810bd968ba6efe0b1052301c242d081dfd843d91fc9e2e46cbce3fc527719
ManifestType: singleton
ManifestVersion: 1.0.0

Notes:

The installer hashes seem to be equal despite them having different installer names and URLs. Would it be necessary to differentiate between the two if the packages are exactly the same?
How should I determine the package identifier for this application?
I haven't tested for a silent install option, nor do I know how to. Could someone please help me with this?
If you're thinking about closing this issue since there is already a winget package available for Nvidia GeForce Experience, please don't! This are not the same package, and Nvidia GeForce Experience necessitates the installation of the entire Nvidia software stack and requires you to have an account with them in order to install driver updates.

Thanks in advance!

Does the package produce different results when installed in Add / Remove Programs?
I'm not sure why there are two URLS. It might just be for tracking purposes.
For "silent" switches, you can try using the Windows Package Manager Manifest Creator. I wouldn't necessarily "submit" the manifest because you might want more/better metadata, but you could use it to see if it can detect the installer type based on parsing the .exe.

Does the package produce different results when installed in Add / Remove Programs?
I'm not sure why there are two URLS. It might just be for tracking purposes.
For "silent" switches, you can try using the Windows Package Manager Manifest Creator. I wouldn't necessarily "submit" the manifest because you might want more/better metadata, but you could use it to see if it can detect the installer type based on parsing the .exe.

The two packages are for different devices, one for notebook and the other is for PC. So the problem is can winget detect device.

These should not be needed as in order for you to install Nvidia Drivers you NEED the Nvidia Experience installed any way for the drivers to be installed.

@mddtest That's not true, please see the article here that shows the installation process without GeForce Experience.

Does the package produce different results when installed in Add / Remove Programs?
I'm not sure why there are two URLS. It might just be for tracking purposes.
For "silent" switches, you can try using the Windows Package Manager Manifest Creator. I wouldn't necessarily "submit" the manifest because you might want more/better metadata, but you could use it to see if it can detect the installer type based on parsing the .exe.

The two packages are for different devices, one for notebook and the other is for PC. So the problem is can winget detect device.

@lychichem As I said initially, though, the file hashes are exactly the same, so unless there's an online component in the installer (which, for the record, I don't believe there is), this could just be for tracking purposes, as denelon suggested earlier. Regardless, I'll have to test the desktop drivers on my laptop at some point to prove this theory.

Does the package produce different results when installed in Add / Remove Programs?
I'm not sure why there are two URLS. It might just be for tracking purposes.
For "silent" switches, you can try using the Windows Package Manager Manifest Creator. I wouldn't necessarily "submit" the manifest because you might want more/better metadata, but you could use it to see if it can detect the installer type based on parsing the .exe.

The two packages are for different devices, one for notebook and the other is for PC. So the problem is can winget detect device.

@lychichem As I said initially, though, the file hashes are exactly the same, so unless there's an online component in the installer (which, for the record, I don't believe there is), this could just be for tracking purposes, as denelon suggested earlier. Regardless, I'll have to test the desktop drivers on my laptop at some point to prove this theory.

I see. As far as I know, some driver packages include all the driver file for all devices and select the files to install base on certain .inf files (The .exe is actually a detector and an inf file selector). Maybe it's same for the Nvidia driver.

Does the package produce different results when installed in Add / Remove Programs?
I'm not sure why there are two URLS. It might just be for tracking purposes.
For "silent" switches, you can try using the Windows Package Manager Manifest Creator. I wouldn't necessarily "submit" the manifest because you might want more/better metadata, but you could use it to see if it can detect the installer type based on parsing the .exe.

The two packages are for different devices, one for notebook and the other is for PC. So the problem is can winget detect device.

@lychichem As I said initially, though, the file hashes are exactly the same, so unless there's an online component in the installer (which, for the record, I don't believe there is), this could just be for tracking purposes, as denelon suggested earlier. Regardless, I'll have to test the desktop drivers on my laptop at some point to prove this theory.

I see. As far as I know, some driver packages include all the driver file for all devices and select the files to install base on certain .inf files (The .exe is actually a detector and an inf file selector). Maybe it's same for the Nvidia driver.

Yeah, this is definitely something I'll need to look into and test out some more. Thanks for your help, though!

I'm stuck here as it doesn't seem to run in unattended mode:

Also, the validation pipelines are running on virtual machines, and to have NVIDIA drivers installed (since there's a compatibility check in the installer), it would fail.

I'm stuck here as it doesn't seem to run in unattended mode - all manifests must run in unattended mode without requiring the user's attention:

Also, the validation pipelines are running on virtual machines, and to have NVIDIA drivers installed since there's a compatibility check in the installer, it would fail as there's no NVIDIA GPU attached in the VMs.
Related to #10113 (comment):
We've been discussing how we're going to deal with this class of scenarios. I don't think we anticipated needing custom hardware to validate packages. from @denelon

Thanks for your help! I find it weird that the self-extracting package won't run unattended, though. Is the installer using a 7-Zip-like self-executing executable, or does this appear to be a more custom solution? I'm a novice when it comes to unattended installations, but I'm pretty sure 7-Zip executables can be executed automatically. If it's a custom solution, though, that would definitely throw a wrench into this situation... How could the GeForce Experience application run unattended while the standalone driver package couldn't? Maybe this is unrelated, so please correct me if this is the case, but I don't see why one tool would differ from the other in such a way. Again, thanks for your help, and I'm interested to hear your thoughts on this :)

I'm stuck here as it doesn't seem to run in unattended mode - all manifests must run in unattended mode without requiring the user's attention:

Also, the validation pipelines are running on virtual machines, and to have NVIDIA drivers installed since there's a compatibility check in the installer, it would fail as there's no NVIDIA GPU attached in the VMs.

Related to #10113 (comment):

We've been discussing how we're going to deal with this class of scenarios. I don't think we anticipated needing custom hardware to validate packages. from @denelon

Thanks for your help! I find it weird that the self-extracting package won't run unattended, though. Is the installer using a 7-Zip-like self-executing executable, or does this appear to be a more custom solution? I'm a novice when it comes to unattended installations, but I'm pretty sure 7-Zip executables can be executed automatically. If it's a custom solution, though, that would definitely throw a wrench into this situation... How could the GeForce Experience application run unattended while the standalone driver package couldn't? Maybe this is unrelated, so please correct me if this is the case, but I don't see why one tool would differ from the other in such a way. Again, thanks for your help, and I'm interested to hear your thoughts on this :)

The installer of the driver is just a self-extract package. It extracts itself to a directory and run setup.exe. The GeForce experience installer is a program packaged using 7z format, it can automatically release to temp directory and run installer then remove temp directory. It can also accept parameters. So, the installers are different.

@lychichem Got it, thanks for the explanation. So, is the consensus right now that this can't be done?

The way I understand https://nvidia.custhelp.com/app/answers/detail/a_id/2985/~/how-can-i-perform-a-silent-install-of-the-gpu-driver is that the initial unpacking step cannot be performed silently, unfortunately. Maybe winget could support it in the future if it adds support for extracting installers from ZIP/self-extracting files.

@chausner Ok, that's understandable, and thanks for clarifying all of this for me.
@denelon What would you suggest doing from here? I can either close the issue outright, or you can tag it with Blocking-Issue. I'm fine with either, just let me know what you decide.
Thank you to everyone who participated in this thread for your help and clarification!

@mariobrostech I added "Blocking-Issue". I think keeping the issue open this way is a better way to avoid others having to go through all of the challenges with this one. Thank you for the effort and for your contributions!

👋 Hey everyone, I just discovered that the Nvidia installer now supports full silent installs. I made a draft pull request here: https://github.com/microsoft/winget-pkgs/pull/110618
Testing this is obviously challenging since it relies on certain hardware. I also provided a list of things that I am unsure about. I would be very grateful for any help in this matter.

Does the install require the hardware to be present?

Does the install require the hardware to be present?

Yes, unfortunately the installer will fail when it doesn't find a compatible GPU while installing. I guess there could be a command line argument to bypass that, but nothing stands out to me in that regard (there is a list in the PR comment).

Do you know what error would be thrown by the installer if the hardware isn't present?
We should add that to an expected return code.

Do you know what error would be thrown by the installer if the hardware isn't present?
We should add that to an expected return code.

  ExpectedReturnCodes:
    - InstallerReturnCode:<PHONE_NUMBER>
      ReturnResponse: systemNotSupported

By the way, the schema: https://github.com/microsoft/winget-pkgs/blob/master/doc/manifest/schema/1.4.0/installer.md says the key should be name ExpectedReturnCode, but that throws a validation error, it seems to want InstallerReturnCode.

Supporting custom hardware in validation is "Not Planned".

https://github.com/microsoft/winget-pkgs/issues/199504


DllNotFoundException on Android

Hi! I am using Firebase Crashlytics to track crashes and non-fatals and I am observing the following issue. On many Android devices (especially on Samsung devices), the following exception occurs:
Non-fatal Exception: java.lang.Exception: DllNotFoundException: Unable to load DLL 'lottie-player'. Tried to load the following dynamic libraries: Unable to load dynamic library 'lottie-player' because of 'Failed to open the requested dynamic library (0x06000000) dlerror() = dlopen failed: library "lottie-player" not found
at Gilzoide.LottiePlayer.RLottie.RLottieCApi.lottie_init(Gilzoide.LottiePlayer.RLottie.RLottieCApi)

With this stacktrace:
at Gilzoide.LottiePlayer.RLottie.RLottieCApi.lottie_init(Gilzoide.LottiePlayer.RLottie.RLottieCApi)
       at Gilzoide.LottiePlayer.RLottie.RLottieCApi..cctor(Gilzoide.LottiePlayer.RLottie.RLottieCApi.)
       at Gilzoide.LottiePlayer.NativeLottieAnimation..ctor(Gilzoide.LottiePlayer.NativeLottieAnimation.)
       at Gilzoide.LottiePlayer.LottieAnimationAsset.CreateNativeAnimation(Gilzoide.LottiePlayer.LottieAnimationAsset)
       at Gilzoide.LottiePlayer.ImageLottiePlayer.RecreateAnimationIfNeeded(Gilzoide.LottiePlayer.ImageLottiePlayer)
       at Gilzoide.LottiePlayer.ImageLottiePlayer.OnEnable(Gilzoide.LottiePlayer.ImageLottiePlayer)

What could be causing the issue with loading the library? And could the following link.xml file with the specified lines help in this case:
<linker>
    <assembly fullname="lottie-player" preserve="all"/>
</linker>

Also I've noticed that all the crashes are coming from the ARMv7 VFPv3 NEON processor. Maybe this will help to solve the problem

Hey @TonSharp, thanks for the report.
Hmm, weird.

Also I've noticed that all the crashes are coming from the ARMv7 VFPv3 NEON processor. Maybe this will help to solve the problem

Well, the rlottie library does use NEON instructions, and it has even some assembly code for ARMv7 (specifically, the pixman code). I haven't tried this library on any ARMv7 devices, only ARM64, likely the build is wrong for this architecture, sorry about that 😔 I don't think I added this assembly to my amalgamated C++ code.
I do have a spare Android device with an ARMv7 CPU, so I can try in it later.

This is not relates to the issue topic but also needs to be discussed

Then please create a new issue for it, so that we can discuss it properly.

Hi @gilzoide , is there any news about non fatals on android?

No, I haven't tryed it yet. It's quite possible that fixing the build for armv7 including the assembly code will fix it, I'll try looking into it this week.

Okay, thank you. I hope the fix will help and I can integrate the new version in the next release of my app. Good luck!

Hey @TonSharp, I've confirmed that the problem was building without including the pixman arm neon assembly code.
I had problems building it with the Android NDK 26, so instead I'm using a custom version of rlottie that just ignores the custom neon code when building with clang. I've tested the fix in a real Android device with arm32 CPU, and it works now!
I'll relase this fix as 1.0.1 soon, so you can update in your app.
Please let me know if you still have problems after this.
Cheers \o/


[Proposal] Allow users to force display order from the pageType defaults

Currently in concreteCMS when adding a block to the pageType defaults the block is always sent to the bottom of the area regardless of the position entered.
So for example on your blog entry page type you have decided to add a new page_attribute_display block to below the title in the main area.
You then setup on child pages and ask for the new block to be added, however when you view the page the block will be at the bottom.
Which means if you have a lot of pages, you have to manually change the position of these newly added blocks.
Example of this behaviour from 8.5/Its the same on 9.x as well
https://user-images.githubusercontent.com/23059797/146507669-3f480d5f-1ae4-4fad-9015-269313a68dde.mov
I propose that on the setup on child pages we add a new option to allow users to force the display order of the block to match the pageType default.

Dealing with rearranging or adding new blocks in legacy (Just poked a <IP_ADDRESS> Install to refresh) was tolerable.  We could easily and rather quickly un-setup all the blocks, move them around and re-set them up.  Ever since we lost the multiple checkboxes to choose which pages the block was/wasn't set up on it's been an absolute nightmare!
with v8+, at best you got one option.  Use an empty area to copy paste all the blocks in your first area to the new area.  add the new blocks in where you want them.  remove all the blocks in the first area.  And then move all the blocks in the second area to the first area and set them each up from top to bottom.  BORING!!
I'm not certain this is something that should be an option though?? (Forcing the Page Type Defaults Order)  A user places a new block between the second and third existing blocks in an area and saves page type defaults.  Wouldn't they then expect to see the block in that position on every page of that type automatically?
If we're not providing page specific default inclusion any more (like the checkboxes in the legacy setup dialogue) this may be better just the way things happen to begin with rather than it being an option...

Fixed with #10174

I've been thinking a lot more about this. I think we could probably simplify this entire process by moving this out from the block level – where it really doesn't make much sense – to the area level instead. For example, what if you could tinker with the blocks in the sidebar endlessly – adding new ones, moving them around, deleting old ones, etc... – but "Setup on Child Pages" was actually at the area level, so when you finally had that Sidebar looking nice, you went to the Sidebar area and basically chose "Update area on Child Pages" which would just make it look exactly the same as in the defaults? This bears more discussion. I think our various options here on the Setup on Child Pages dialog are a little confusing at this point, but I was able to make the sidebar look the same on all my blog pages throughout some adventurous edits, which is a better experience than what we have today, so i think this is a good improvement. The option to push blocks down on the child pages based on adding the new block really helps here.

I agree! Always I want to sync area on child pages when I update the defaults.

Added a new issue for discussion on this topic: https://github.com/concrete5/concrete5/issues/10235


Update terraform version to support newer ssh key algorithms.

Update terraform version to support newer ssh key algorithms.
For https://github.com/longhorn/longhorn/issues/4320
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

LGTM, but the one thing I am curious about is that the weak hashing function used in the SSH connection is rejected by the connected host, but why changing the terraform version can actually fix the problem? IIRC, the SSH keys are generated by us instead of any terraform plugins.

LGTM, but the one thing I am curious about is that the weak hashing function used in the SSH connection is rejected by the connected host, but why changing the terraform version can actually fix the problem? IIRC, the SSH keys are generated by us instead of any terraform plugins.

It's because the rejection is due to the host has disabled the weak algorithm and the client (terraform) still wants to use this weak algorithm to connect the host (So this error would only happen in terraform. It works fine when we try to ssh into the host through our laptops), and terraform already fix this issue in v1.2.0 (The last enhancement https://github.com/hashicorp/terraform/pull/30962). After this version, terraform wouldn't try to use weak algorithm so it won't be rejected.


offline installation fails: "Please install CMake first"

I want to install offline. However, the following problems have occurred.How can I solve it?
Windows PowerShell
版权所有（C） Microsoft Corporation。保留所有权利。
安装最新的 PowerShell，了解新功能和改进！https://aka.ms/PSWindows
PS G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package> python setup.py install
running install
INFO:LightGBM:Starting to compile the library.
INFO:LightGBM:Starting to compile with MSBuild from existing solution file.
WARNING:LightGBM:Compilation with MSBuild from existing solution file failed.
INFO:LightGBM:Starting to compile with Visual Studio 17 2022 (x64).
Traceback (most recent call last):
File "G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package\setup.py", line 97, in silent_call
subprocess.check_call(cmd, stderr=log, stdout=log)
File "G:\Anaconda3\lib\subprocess.py", line 373, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['cmake', '--build', 'G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package\build_cpp', '--target', '_lightgbm', '--config', 'Release']' returned non-zero exit status 1.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package\setup.py", line 354, in
setup(name='lightgbm',
File "G:\Anaconda3\lib\site-packages\setuptools_init_.py", line 153, in setup
return distutils.core.setup(**attrs)
File "G:\Anaconda3\lib\distutils\core.py", line 148, in setup
dist.run_commands()
File "G:\Anaconda3\lib\distutils\dist.py", line 966, in run_commands
self.run_command(cmd)
File "G:\Anaconda3\lib\distutils\dist.py", line 985, in run_command
cmd_obj.run()
File "G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package\setup.py", line 263, in run
compile_cpp(use_mingw=self.mingw, use_gpu=self.gpu, use_cuda=self.cuda, use_cuda_exp=self.cuda_exp, use_mpi=self.mpi,
File "G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package\setup.py", line 207, in compile_cpp
silent_call(["cmake", "--build", str(build_dir), "--target", "_lightgbm", "--config", "Release"], raise_error=True,
File "G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package\setup.py", line 101, in silent_call
raise Exception("\n".join((error_msg, LOG_NOTICE)))
Exception: Please install CMake first
The full version of error log was saved into C:\Users\86159\LightGBM_compilation.log
PS G:\★★★study\2022\3.投稿材料\小论文\LightGBM-master\python-package>

Thanks for using LightGBM!
As the error message in that log says:

Exception: Please install CMake first

You can follow the instructions at https://cmake.org/install/ to install it on your machine. Once CMake and LightGBM's dependencies have been installed, internet access should not be required to compile LightGBM.

Thanks for using LightGBM!
As the error message in that log says:

Exception: Please install CMake first

You can follow the instructions at https://cmake.org/install/ to install it on your machine. Once CMake and LightGBM's dependencies have been installed, internet access should not be required to compile LightGBM.

But I have installed successfully. I suspect it is a problem with the path, but I'm not sure.

How to add cmake to my PATH？

PATH is an environment variable containing a list of folders that your computer looks in when you tell it to run commands. Find cmake.exe on your machine, then add its location to that variable.
For help, see a tutorial like https://www.java.com/en/download/help/path.html. Follow the Windows instructions, ignore anything about Java.
After editing PATH, close and re-open whatever program you are using to compile LightGBM (cmd prompt, Visual Studio, etc.).

I have add the address of cmake.exe on the PATH variable. But it still doesn't work.

Please share a screenshot with the contents of your PATH environment variable, and we can try to help you.


ZMQ default port

Hi @xydesa
In the experiments of this week, we found some problems with the default ports used by ZMQ. They are the same used by Nav2, and it produces an exception. It makes me think that we should change them because it is probably a widely used configuration.
Do you think that the best fix is only to change our default ports?
Best

Yes, I think just changing the default ports used by plansys2 should be enough.


Get rid of GrammaticalMeaning interface

Fixes #57

@rultor merge

@rultor merge

@suseika OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@suseika Done! FYI, the full log is here (took me 1min)


Split prowjobs; move them to .prow directory

Signed-off-by: Waleed Malik<EMAIL_ADDRESS>What this PR does / why we need it:
🤷♂
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Optional Release Note:

NONE

/retest


Support analyzing specific repositories

While scanning an entire organization is nice, it can be quite painful when an organization has hundreds of repositories. My organization is migrating in chunks, so we would like to do analysis only on specific repositories when we're ready to migrate them.
Perhaps add a new parameter for a list of repositories and/or a new parameter for a file of listed repositories (or enhance the logic around the input parameter to recognize repositories)?
I could expect that if repositories were from different organizations or if both a list of repositories and an organization was provided, then the tool could error out.

This was solved in #29


buildEnvironment does not pass correct params to build process

Using the latest build 1.2.1, it seems that specifying buildEnvironment as a param to multiple RustFunction instances causes only the first passed env vars for the first hit RustFunction instance to be passed for all following creations of RustFunction.
Minimum example:
const foo_lambda = new RustFunction(this, 'foo_lambda', {
	directory: path.resolve(path.join(__dirname, '../lambda')),
	package: 'foo',
	buildEnvironment: {
		FOO_ENV: 'foo',
	},
})

const bar_lambda = new RustFunction(this, 'bar_lambda', {
	directory: path.resolve(path.join(__dirname, '../lambda')),
	package: 'bar',
	buildEnvironment: {
		BAR_ENV: 'bar'
	},
})

With the rust code doing e.g.:
const FOO_ENV: &str = env!("FOO_ENV");
const BAR_ENV: &str = env!("BAR_ENV");

I'm getting the error when compiling the second rust functionenvironment variable "BAR_ENV" not defined, however - the FOO_ENV is available for the second function. If reversing the order of the two RustFunctions, now BAR_ENV is available, and compilation will fail with FOO_ENV not defined.
This issue can be worked-around by building each package individually by doing:
import { Settings } from 'rust.aws-cdk-lambda'
Settings.BUILD_INDIVIDUALLY = true

Hi @timkaas, first off thanks for opening this issue. I had a look, and realized that you are totally right - this is likely because we only call cargo lambda once by default on all bins/workspaces in a project, so the first buildEnvironment that the RustFunction encounters is the one that will be used for a first-time (and project wide) compilation.
More info and context can be found in these lines:
https://github.com/rnag/rust.aws-cdk-lambda/blob/main/lib/build.ts#L116-L138
As can be seen, shouldCompile is a boolean flag and is only true for once per project, or else is always true when BUILD_INDIVIDUALLY is enabled.
Additionally, it can also be seen, that we default to BUILD_ENVIRONMENT is buildEnvironment is not passed in to RustFunction the first time:
options.buildEnvironment ||
Settings.BUILD_ENVIRONMENT;

Per these notes above, I would say that it might be slightly more performant to only call cargo lambda once per project, rather than once per RustFunction (which can easily add up if you have a lot of them).
I would suggest using Settings.BUILD_ENVIRONMENT in this case, and having merged environment defined under there:
Settings.BUILD_ENVIRONMENT = {
	FOO_ENV: 'foo',
	BAR_ENV: 'bar'
};

const foo_lambda = new RustFunction(this, 'foo_lambda', {
	directory: path.resolve(path.join(__dirname, '../lambda')),
	package: 'foo',
        # don't use `buildEnvironment` as we want to default to merged environment above
})

const bar_lambda = new RustFunction(this, 'bar_lambda', {
	directory: path.resolve(path.join(__dirname, '../lambda')),
	package: 'bar',
        # don't use `buildEnvironment` as it will be ignored here
})

Please do try that out, and let me know how it goes.

A few TODO items I see, and potential areas of improvement in terms of documentation and clarity purposes:

Documentation should be updated to indicate that BUILD_INDIVIDUALLY should be enabled (true) when multiple instances of buildEnvironment are passed in, since only the first value of buildEnvironment will be used for compilation step (since cargo lambda only runs once per project).
We should definitely log or print out a user-friendly warning when subsequent buildEnvironment is passed in, but shouldCompile is false, meaning compilation step was already run. I feel a friendly message would be to suggest merging those env values into Settings.BUILD_ENVIRONMENT instead, before declaring any RustFunctions.


Getting "Timed out while attempting to fetch data" & Runs fetched coming as empty Array[]

Hey @Codex-
Usecase:
I am trying to make a workflow dispatch call from our public repo to our private repo using codex-/return-dispatch action.
issue:
The call actually successfully triggers the workflow in the private repo but there are two issues.
Issue1: It is not waiting until 600 seconds which i passed in the workflow_timeout_seconds: 600
Issue2: It always fails after 1 minute stating - "Timed out while attempting to fetch data"
I have run the workflow in debug mode and I can see it is printing - Runs Fetched: [] which means it is not able to fetch the run IDs. however when I use octokit to locally fetch the run IDs it actually works.
This is my calling workflow code - this
My receiving workflow has distinct_id as input params and also the echo step as the first step in the job.
*note - My receiving workflow is again calling 5 child workflows using workflow call.
Please help.

Xero 👀 know a few people there
I'm having a look at the moment, will keep you posted :)

Found the issue, thanks for raising this, a shame on the timing because I'm doing a fairly large breakdown for v2 that resolves this already but will backport this fix to v1

@SangeetJoy fix merged, v1 tag moved to latest
Please try this and let me know if you still encounter this issue
Happy to help if you're still running into problems :)

Hey @Codex- Thank you for resolving the timeout code however it is still not able to fetch the run IDs as it is coming as empty Array.
I am reopening this issue, could you please check the screenshots once and help me understand why the run IDs are coming as empty array,
I have tested in local and it can fetch the run IDs.


Ingress not created

Is this a BUG REPORT or FEATURE REQUEST?:
BUG
What happened:
Sometimes, when I deploy an HTTP trigger, in my cluster the trigger itself is created but the corresponding ingress no, and clients receive an http error (404).
If I re-deploy the trigger it seems to work, but I can't understand the reason of this behaviour.
I deploy functions and triggers via yaml file like this:
apiVersion: kubeless.io/v1beta1
kind: HTTPTrigger
metadata:
  labels:
    created-by: kubeless
    app: ****
    chart: ****
    release: ****
    heritage: Tiller
    version: 0.1.0
  name: trigger-name
  namespace: kubeless
spec:
  function-name: function-to-be-triggered
  host-name: ****.****.***.azure.com
  gateway: nginx
  path: "get-positions"

@AlessiaPileri is the creation of the ingress rule what fails or is the ingress controller the one that doesn't properly handle the new ingress rule?
In case the ingress rule doesn't get generated (you cannot find it executing kubectl get ingress <func_name>) you can find the cause of the error from the Kubeless controller: kubectl logs -n kubeless -l kubeless=controller -c http-trigger-controller.
In case the ingress rule gets generated but it doesn't work you need to check the error outside Kubeless. Depending on the Ingress controller that you are using you need to check its logs to discover possible issues.

Hi @andresmgot, I check the logs in the Kubeless controller with the command you suggest me, but I obtain this error:
container http-trigger-controller is not valid for pod kubeless-controller-manager-3663698413-2xfxk
this can be a kubeless deploy issue? I check the 'ingress-enabled' flag in kubeless-config config map and it's set to true

that command only applies to the latest version. If you have a previous one you can check the logs with just kubectl logs -n kubeless -l kubeless=controller (in the last version we split the functionality of the controller in three different containers, that's why the -c http-trigger-controller is needed).

It’s been a few weeks perhaps we can close this issue for now?

Yes, let's close this issue for now. @AlessiaPileri if you find that the issue is related to kubeless we can open this again.


Multiple rst tutorial menu

With this pull request the help menu get a nuw submenu, just below the spyder introduction, that contains link to other tutorial, that are indexed by the name of the rst tutorial file.
This pull request is connected to SEP2: Interactive Tutorial Content.

Thanks a lot for your contribution. I want to say two things:

Could you post a screenshot of what you've achieved so far?

I think @goanpeca had a different idea for the SEP you're referencing. What do you say @goanpeca?

I'm sorry for the confusion.
I restored the old SEP, created a new one (number 6) and modified the commit accordingly (renaming the function call to the old show_tutorial)

This is a preview of the resulting menu and reproduction of the bare tutorial in rst

I have other idea to tackle this: I'd like to add a home page to the Object Inspector with links to several resources, instead of adding them to the Help menu.
What do you think @EnricoGiampieri?

@ccordoba12 that sounds great:-), but then we would need to add a Home button to the object inspector to go there right?

It sounds like a very good idea to me, @ccordoba12. Another great improvement would be a search function for classes and syntax. I teach python classes and an inline help is one of the most requested things that I encounter.

@goanpeca, yes, the code is already there (look at the Online help plugin). We just need to use it for the Object Inspector.
Obviously, if we do that, we should also add tabs to the Object Inspector, so people can read several things at the same time. I've been thinking to do that for a long time but haven't found time to make it a reality :-) Maybe for 2.4...
@EnricoGiampieri, adding search functionality is also planned. It's just tricky to do it right but there is a very good Python library on which we can rely, called Whoosh

Since we all agree that creating a home page for the Object Inspector is the better option here, I'd like to know if it's ok for @EnricoGiampieri to close this PR.
@EnricoGiampieri, what do you think?

I will gladly close it. There is any way I can help with the new design?

I'm all for improving the object inspector and adding a Home page, but for tutorials the Help menu seems mode discoverable ?
To limit the length of the menu all the tutorials could go in the same submenu. Having tutorials is great but they're not used that often.

Since everyone agree, I'm closing it.


Merge Filesystem::init with constructor

I'd like to suggest merging Filesystem::init with the constructor, and removing Filesystem::init as a (public) member function.
I believe this is more consistent with current C++ design principles. If the Filesystem object experiences an error during initialization, it should throw from the constructor, preventing object creation. This means you can never have a partially initialized zombie object. That removes the need to test for things like isInit in each member function before doing anything. If the object isn't initialized, then it never existed, and so there was never any object to call member functions on. I think this would make for a much cleaner design.

Changes:

Move the init parameters over to the constructor

Have the constructor call init
If an init error occurs, throw std::runtime_error (or derived)

Remove init from the public member section
Remove isInit style checks from member functions

Optionally, the init method can be removed entirely if the contents are moved into the constructor.
There should be no default constructor on Filesystem.

Will be interested to see what you come up with.
I know this is a bit late to mention, but as for my (very recent) https://github.com/lairworks/nas2d-core/pull/243#issuecomment-569457030, this change could eliminate the need for things like mIsInit. That could be a way of simplifying PR #243, and cutting down on its bulk. Doing so though would require merging this first as an independent change, and rebasing the other branch, which could potentially be a bit of work. Maybe not really fair for me to ask this late into changes. It would really help cut down on the size of a review chunk though.

In agreement here -- I wouldn't be entirely opposed to just moving the init code into the constructor in an effort to maintain a tidy interface even though that flies in the face of best practices. Will investigate and see how best to approach this.


Worked amazing up until today, now suddenly stuck on party 3 - the no briv party - and not going back to party 1 with briv. Manually changing he goes back out front, then switches to 3 again without going back to 1...any ideas? Cheers!

Have you been updating your imports when new versions of the game are released?
Either using the tools at https://github.com/antilectual/ScriptHub-AutomaticOffsets or from the Import Updates Thread in the #scripting channel in the Official IC Discord?
The game updates more frequently than the script and the script needs to be updated on where to read game data from. The discord community is very quick on creating/sharing new Imports for each game update, and are quite helpful in general.

Imports have been updated.

Nailed it - THANK YOU!


Usage of CloudEvents SDK GO

Describe the proposal
While looking at the code of the contrib repo, especially around the subject of "Cloud Events", I noticed on areas like here:
https://github.com/matzew/components-contrib/blob/master/pubsub/envelope.go#L105-L117
that the Golang SDK for CloudEvents is not used.
Would the community be interested in using that SDK?

@berndverst Do you have context here, why the SDK was not choosen? If interest in using that, I could take a look...

I do not know @matzew -- the foundations of that code were before my time. I assume it has to do with maximizing compatibility. @yaron2 will know more here. Perhaps it's simply that folks didn't know there was a Go SDK at the time.
That being said, I highly doubt the existing behavior can be exactly retained without breaking changes by switching to the cloud events Go SDK.
Also keep in mind that we need to maintain the JSON / struct representation anyway because we must not change the behavior of our API output (HTTP JSON, and gRPC protos), so there may be little value in using the official cloud event SDK.
Unless there is a functionality / behavior reason for swapping this, I do not suggest making this swap.
There is more code here - https://github.com/dapr/dapr/blob/master/pkg/runtime/pubsub/cloudevents.go and all over pkg/runtime in dapr/dapr.

One example:
For legacy reasons we must support both traceid and traceparent at this time (they have the same value). This of course wouldn't when using the Cloud Events 1.0.2 spec alone.


Global settings are shadowed by the addition of empty_label default value

Related to #6

Have to look into this, got introduced by accident fixing another problem.


Added community colleges and minor updates

Hello. I attempted to follow the contributions guidelines as close as possible. Apologies if I did not do something correctly. Also sorry for not following the provided template. Due to the number of colleges added, it is not practical in this situation.
All the community colleges in Alaska, Michigan, North Dakota, South Dakota, and Vermont were added, according to the list provided by Community College Review. The domain of ITT Technical Institute was corrected to include a hyphen. Removed the domain of "edu.tf" as it is nonexistent. Removed the domain "edu.vlerick.com" as "vlerick.com" is present. Lastly, corrected the domain of the International Turkmen-Turkish University.

Hi David,
Thanks for opening this.
Fine not to use the template in this case. However it would be much appreciated if you could do out a checklist like this:

[ ] Oxford University
[ ] IKing's College London
[ ] University of Cambridge
...

this will make it much easier to review the institutions as I can run down the list and check them off as I go

OK. It'll take me a couple of minutes.


Fix formatting and add required information

Completed all from #7, and also added some missing information:

Added section 2.6 about Qualifiers
Added information about & and < attributes
Added actual information to some sections, fixed examples

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@c71n93 @yegor256 Oops, I failed. You can see the full log here (spent 14min)
Unpacking python3-pygments (2.3.1+dfsg-1ubuntu2.2) ...
Setting up python3-pygments (2.3.1+dfsg-1ubuntu2.2) ...
+ sudo -i pip3 install pygments
Requirement already satisfied: pygments in /usr/lib/python3/dist-packages (2.3.1)
\u001b[33mWARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv\u001b[0m\u001b[33m
\u001b[0m--- Logging error ---
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/utils/logging.py", line 177, in emit
    self.console.print(renderable, overflow="ignore", crop=False, style=style)
  File "/usr/local/lib/python3.8/dist-packages/pip/_vendor/rich/console.py", line 1673, in print
    extend(render(renderable, render_options))
  File "/usr/local/lib/python3.8/dist-packages/pip/_vendor/rich/console.py", line 1305, in render
    for render_output in iter_render:
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/utils/logging.py", line 134, in __rich_console__
    for line in lines:
  File "/usr/local/lib/python3.8/dist-packages/pip/_vendor/rich/segment.py", line 249, in split_lines
    for segment in segments:
  File "/usr/local/lib/python3.8/dist-packages/pip/_vendor/rich/console.py", line 1283, in render
    renderable = rich_cast(renderable)
  File "/usr/local/lib/python3.8/dist-packages/pip/_vendor/rich/protocol.py", line 36, in rich_cast
    renderable = cast_method()
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/self_outdated_check.py", line 130, in __rich__
    pip_cmd = get_best_invocation_for_this_pip()
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/utils/entrypoints.py", line 58, in get_best_invocation_for_this_pip
    if found_executable and os.path.samefile(
  File "/usr/lib/python3.8/genericpath.py", line 101, in samefile
    s2 = os.stat(f2)
FileNotFoundError: [Errno 2] No such file or directory: '/usr/bin/pip3.8'
Call stack:
  File "/usr/local/bin/pip3", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/cli/main.py", line 70, in main
    return command.main(cmd_args)
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/cli/base_command.py", line 101, in main
    return self._main(args)
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/cli/base_command.py", line 223, in _main
    self.handle_pip_version_check(options)
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/cli/req_command.py", line 190, in handle_pip_version_check
    pip_self_version_check(session, options)
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/self_outdated_check.py", line 236, in pip_self_version_check
    logger.warning("[present-rich] %s", upgrade_prompt)
  File "/usr/lib/python3.8/logging/__init__.py", line 1458, in warning
    self._log(WARNING, msg, args, **kwargs)
  File "/usr/lib/python3.8/logging/__init__.py", line 1589, in _log
    self.handle(record)
  File "/usr/lib/python3.8/logging/__init__.py", line 1599, in handle
    self.callHandlers(record)
  File "/usr/lib/python3.8/logging/__init__.py", line 1661, in callHandlers
    hdlr.handle(record)
  File "/usr/lib/python3.8/logging/__init__.py", line 954, in handle
    self.emit(record)
  File "/usr/local/lib/python3.8/dist-packages/pip/_internal/utils/logging.py", line 179, in emit
    self.handleError(record)
Message: '[present-rich] %s'
Arguments: (UpgradePrompt(old='22.2.2', new='22.3.1'),)
+ pdd --file=/dev/null
ERROR: book.tex; PDD::Error at book.tex:494: TODO must have a leading space to become a puzzle, as this page explains: https://github.com/cqfn/pdd#how-to-format
If you can't understand the cause of this issue or you don't know how to fix it, please submit a GitHub issue, we will try to help you: https://github.com/cqfn/pdd/issues. This tool is still in its beta version and we will appreciate your feedback. Here is where you can find more documentation: https://github.com/cqfn/pdd/blob/master/README.md.
container 4a97b7356d3be9561098bd2303a38ca22e181ca2349171885de54b3cb10128d9 is dead
Sat 26 Nov 2022 03:52:43 AM CET

@c71n93 try to run pdd -f /dev/null locally and you will see what is the problem. Probably, you use TODO not how we expect it to be used.

@yegor256 I have fixed TODOs

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 1min)


Broken links found in the XMLData spec

Description:
There are the broken links below found on the XMlData spec (mostly, in the tables) and we need to fix them.

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Fixed via https://github.com/ballerina-platform/module-ballerina-xmldata/pull/451.


Stub files should use "as"

https://www.python.org/dev/peps/pep-0484/#stub-files

https://github.com/microsoft/pyright/issues/1128

https://youtrack.jetbrains.com/issue/PY-43203?clearDraft=true&description=

PC-201.7846.77

pyright now seems to support using libraries with inline type annotations (https://github.com/microsoft/pyright/commit/8cb9bfca41cb05ded74dde172472a2cfa6f42bd6).


Fix side nav logo in Firefox

Documentation layout was strange in Firefox. (tested on Mac Firefox 42)

Obviously I forgot to test on Firefox. Will test with a few more browsers tomorrow and then merge. Thanks!

Fix confirmed. Thank you!


Remove Makeshift Mobile Formatting

Most recent commit: f3b3cffb2c095a74f0ab6eef1bbeb4689410dc09
There is now a temporary solution for forcing mobile format. It limits the width of the app to 0.75 of the window height. This has been done because the app has been designed without consideration of the desktop format and looks a bit silly when it's in landscape mode.
When the app has been designed for the desktop format, all of the code introduced in f3b3cffb2c095a74f0ab6eef1bbeb4689410dc09 should be removed, and anything that depends on it should be refactored.

Relevant commit: 6ebba78af156abb5af96f1e198669b1c30ee8aab


Resolved Conflicts in #354

Related Issue

Closes: #354
Description of Changes

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots
|        Updated           | :
Please provide any necessary screenshots to illustrate the changes made.

Thank you @abhi03ruchi nice work

Just for mobile mode! improve it a little.... it does not look good. first paragraph text it not aligned.
It is good to merge!

That plus sign in not there in mobile view!


DCOS-40649: Health column header truncated in FF

Even at fullscreen, the Health column header was being truncated in Firefox
Closes: DCOS-40649
Testing
Use Firefox to test this change
Add a framework from the Catalog (I used Kafka)
Navigate to the "Services" page
Click on the new service
The health column header should not be truncated
Trade-offs
A better way to fix this would be to rendered this table with the components from ui-kit. However, that would take a little more time in order to support the checkbox selection of table rows (ticket created here)
Dependencies
None
Screenshots
Before:

After:

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


feat: add labels filter to proposals list

Summary

Closes: https://github.com/snapshot-labs/workflow/issues/254
Re-submission of https://github.com/snapshot-labs/sx-monorepo/pull/910 which was merged, but reverted due to issue with SX api.
This PR is pushing the same changes again, but with some minor fixes to support both onchain and offchain API.
Particularly, onchain API is filtering labels using metadata.labels_contains, whereas offchain uses labels_in.
How to test

Go to http://localhost:8080/#/sep:0xb58B05A1c8263441Bd74454544fD66CE25D300AD/proposals
You should be able to filter proposals by labels (filtered using AND)
http://localhost:8080/#/s:test.wa0x6e.eth/proposals
You should be able to filter proposals by labels (as of now, filtered using OR, will use AND once https://github.com/snapshot-labs/snapshot-hub/pull/957 is merged)

tACK
Might be worth waiting for hub update to finish before merging.

Hub PR has been merged, you can merge this one as soon as the sx-api instances have been switched

@wa0x6e We've switched the instances already


Introduce logging platform

Need to be capturing logs somewhere. Need to find a logging platform which is free for opensource/nonprofit.

Now using lambda, this automatically uses cloudwatch


Quickstart tutorials/examples

Submitting PR for #1507
It is work in progress - with some open questions and is incomplete since I ran into issues (eg. errors when using the concat function).
Submitting for discussion and as a starting point for feedback.

Coverage decreased (-0.9%) to 88.496% when pulling ce076c20304e3c049554c65177797d090a1bce01 on sandhujasmine:sandhu/quickstart-tutorials into 1aa004750250e9768983c3b01be68f76eca9c547 on blaze:master.

Closing to cleanup old PRs. Feel free to submit a new PR.


Error: Could not locate the binding file. Tried: node_modules/frida/build/frida_binding.node

Easy solution: use a server outside of China to get an unblocked network connection.
Hard solution: download frida_binding.node from https://github.com/frida/frida/releases/ manually, then put it into node_modules/frida/build folder.

Explanation:
frida
is using prebuild-install
to install binary files
from GitHub Release.
If your server has any trouble with visiting the Amazon S3 servers (which is all the GitHub release artifacts are hosted on), then your NPM install will fail to download the binary files (frida_binding.node in our case).

Update
We have created https://github.com/huan/sidecar/blob/main/scripts/post-install.js to check and re-install via CDN if necessary.


#63: Divide "matrix/matrix" to "matrix/provision" and "matrix/droplet"

https://github.com/BR0kEN-/cikit/compare/issues/63

Tested and ready for review/merge.

Ready again.


Bug 1393936: Use hardcoded color values for expanded icons

Issue: The expanded icon cannot display clearly in attach to process window after setting the theme to dark

Dark Theme
Before:

After:

Blue Theme
Before:

After:

Note: will remove extraneous comments, but I believe it is better to explain where the hex values come from. This can be polished up.

Can you also share pictures of the window when its on a High Contrast Theme?

Can you also share pictures of the window when its on a High Contrast Theme?

Expanded icon is problematic in High Contrast White.
Screenshots below:
Blue (Extra Contrast):

High Contrast Black:

High Contrast White:

High Contrast 1:

High Contrast 2:


Mapping in component folders

Add more mapping the different folders.
The component/message is very big and "messy". Add one subfolder for each functionality.
Verify that everything works.

Wait with merge until there are no other open branches.


Q: would you be interested in writing rust bindings for libmdbx?

libmdbx is revised and extended descendant of LMDB.

Probably my English is not very good:

Any cursor created by mdb_cursor_open(), when it becomes unnecessary, must be explicitly freed by mdbx_cursor_close().
Any cursor created by mdb_cursor_open(), may be reused by mdbx_cursor_renew(), untils it explicitly freed by mdbx_cursor_close().

Hi, I made some progress on building the mdbx bindings in Rust and found out that Rust doesn't like overflowing number like MDBX_RESULT_TRUE: c_uint =<PHONE_NUMBER>3709551615. And found strange that a c_uint is in fact an unsigned int and is attributed the max value of an unsigned long instead of the unsigned int one. Is it normal? I understand that it work but...

Ok so here is the mdbx-sys repository with most of all the bindings generated in Rust. I blacklisted (for now) the MDBX_page_type_t, MDBX_pgvisitor_func, mdbx_env_pgwalk types/functions, because of the upper cased problem and will probably #[allow(non_upper_case_globals)].

The MDBX_page_type_t, MDBX_pgvisitor_func and  mdbx_env_pgwalk are an internal/auxiliary  API, i.e.  shouldn't be used in a regular application.

The problems with error codes since libmdbx defines an aliases for some system error codes:

are system-depended, i.e. differ on Linux, *BSD, MacOS, Windows, etc.
may be defined as signed integers (Linux, Unix, MacOS) or unsigned (Windows), but signed int is preferable since mdbx functions returns int type.

I will blacklist those like I did before.
Do think that I will need to define those by hand? like this part of the lib?


Unable to add teams to a repo

Problem Description
Safe Settings is able to remove teams from a repo based on configuration, but not able to add teams to a repo.
What is actually happening
^^
What is the expected behavior
The GitHub teams should be added to the repo.
Error output, if available
No errors. There is a comment in the code refering to the GitHub API. Has the endpoint changed for github.com and this is not reflected in code?
Context
We are trying to run safe-settings with suborg's settings files.
Are you using the hosted instance of probot/settings or running your own?
We are running probot in our own container.
If running your own instance, are you using it with github.com or GitHub Enterprise?
Github.com
Version of probot/settings
Latest.
Version of GitHub Enterprise
NA

We are using safe-settings with teams and members, and we do not have any problems. But our teams do not have any space in its name.
But its usage is a bit confusing. You have to put the name of the team instead of the team_slug or the id. Example:
If you have a team: https://github.com/orgs/ORG/teams/dev and its name is "Devs", in safe-settings, you have to put "Devs" instead of the team_slug, "dev" or "@org/dev".
That is, you would have to put these names:
ORG=myOrganization
gh api \
   -H "Accept: application/vnd.github.v3+json" \
   /orgs/${ORG}/teams --jq ".[].name"

@svg153 thanks for helping out.


Fusion initial implementation

Closed with https://github.com/ardatan/graphql-mesh/commit/a7984e5ab214ddd7f75dca0f03b2e7e8ad768211


meta: Communicate plugin categories in releases and changelog

Feature scope
Other
Description
Something like this would be helpful for developers looking at the changelog:
1.0.0
Taps

fix: A change that impacts all taps

REST

feat: A new feature available to REST taps

Targets
SQL

feat: A new feature available to SQL targets

Still relevant


Flow 0.55.0 breaks react-redux for some cases

Hi all,
A quick note that when updating to flow 0.55.0 I started getting errors with some connect functions I have with react-redux. Here is an example on flow.org/try. Note that there is an error in 0.55.0 and not 0.54.1.
It seems that if you have mapStateToProps as a function that returns a function, flow can't figure out the correct intersection. It gets confused between cases:
declare function connect<S, A, OP, SP>(
  mapStateToProps: MapStateToProps<S, OP, SP>,
  mapDispatchToProps: Null,
  mergeProps: Null,
  options?: ConnectOptions
): Connector<OP, $Supertype<SP & { dispatch: Dispatch<A> } & OP>>;

and case:
declare function connect<S, A, OP, SP, DP>(
  mapStateToProps: MapStateToProps<S, OP, SP>,
  mapDispatchToProps: MapDispatchToProps<A, OP, DP>,
  mergeProps: Null,
  options?: ConnectOptions
): Connector<OP, $Supertype<SP & DP & OP>>;

Which to me seems like a bug and that case 1 is the only valid choice.

Related flow-typed issue: https://github.com/flowtype/flow-typed/issues/1269

I tried to make a reduced testcase, but after reducing a log, I got a case where it's breaking in both 0.54 and master ;)
Still I'd like to mention these 2 cases:
Case 1 works in both versions, while Case 2 fails in both versions. If I mention them here is that I don't really understand why they yield different results, and I feel like this can be related to this issue.
Now I'm going back at finding a reduced test case.

I think this is the most reduced test case I can find:
declare type MapStateToProps<SP: Object> =
  () => (() => SP) | SP;

declare type MapDispatchToProps<DP: Object> =
  (() => DP) | DP;

declare function connect<SP>(
  mapStateToProps: MapStateToProps<SP>,
): void;

declare function connect<SP, DP>(
  mapStateToProps: MapStateToProps<SP>,
  mapDispatchToProps: MapDispatchToProps<DP>,
): void;

connect(
  () => ({ foo: 42 })
);

There is something strange happening because changing MapStateToProps to
declare type MapStateToProps<SP: Object> = () => SP;

(which is the correct type here) removes the error. I don't see why changing the type for MapStateToProps makes it correctly decide that mapDispatchToProps can't be null.

Calling connect like this works:
connect(
  (): * => ({ foo: 42 })
);

Which makes things even weirder to me :)

Heh, yeah, it gets even weirder. Try defining your connectors in one file then applying them to a component in another. It magically makes many checks fail and others pass. I think it comes down to when the types are evaluated, in combination with the fact that subtyping rules for object function args are different than for normal object types. I'm hoping to pick it back up this evening - I took a crash course in Ocaml the other day just in case!

A couple of other relevant issues in flow-typed:
https://github.com/flowtype/flow-typed/issues/1432
https://github.com/flowtype/flow-typed/pull/1317
It seems that the react-redux libdef is no longer catching quite a few errors starting in 0.55. Doesn't seem to be totally clear if it's a flow bug or libdef issue though.

I've found the commit that broke things. I went through and did a git bisect on the flow source code, and found that both the OP issue AND the issue described in https://github.com/flowtype/flow-typed/issues/1432 start happening after Flow commit https://github.com/facebook/flow/commit/bd938abfb8970972d512847415d0f68563120480 .
I'm not at all familiar with the flow source code so I don't know whether this commit introduces a bug, or just changes the API. Hopefully identifying the breaking commit will make it easier for people who actually understand how flow works (ie, not me!) to work out what the issue is. @calebmer and @samwgoldman : you wrote/reviewed the commit in question, if you get a chance would you be able to shed any light on this?

As @TLadd said it seems most of the $ExpectErrors just emit warnings saying they weren't used, which doesn't actually fail the tests. I encountered this myself when I was trying to fix #1498, I had test code working in a project I'm working on but I couldn't get the flow-typed test to run and catch errors. I think there might be a bigger issue and until the test is fixed we can't fix the libdefs, @villesau do you know if this was ever fixed I remember seeing a comment you made about it somewhere.

@djb5143 the $ExpectError issue was fixed in this PR for flow-typed: https://github.com/flowtype/flow-typed/pull/1407 and tested in https://github.com/flowtype/flow-typed/pull/1411
So instead of relying on exit code of flow, now it checks the output instead.
Try out with up to date master branch of flow-typed?

React-redux libdefs have been rewritten twice after this so I assume this is no longer valid.


sa

export

Hi. I'm not sure what you're trying to merge here. This repo (rshipp/python-codacycov) is an old, outdated version of the project which is now maintained by Codacy (codacy/python-codacy-coverage). You should open any issues/PRs there instead.


WG: With 3+ players, the game ends immediately after first player reaches target article

This occurred on an iPad, Samsung Tablet, and two desktop windows.

closing until reproduced


Dependency upgrades for mitigating a vulnerability

The codecov package had a vulnerability.
The istanbul package is no longer maintained. The project recommends switching to the nyc package, which was done.
The vulnerabilty, as per npm audit:
Severity: high
Command injection in codecov (npm package) - https://github.com/advisories/GHSA-xp63-6vf5-xf3v
codecov NPM module allows remote attackers to execute arbitrary commands - https://github.com/advisories/GHSA-5q88-cjfq-g2mh
Improper Neutralization of Special Elements in Output Used by a Downstream Component in Codecov - https://github.com/advisories/GHSA-mh2h-6j8q-x246
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/codecov

Tests ran, all passed locally.

Thanks!


Add setting for cookies in request headers

This PR adds the ability for setting cookies in request headers. Cookies may sometimes be required to access pages behind a login or some kind of consent page.
Might be a partial solution for #53.
Note: This solution works for zeit.de by using something like config['DEFAULT']['COOKIES'] = 'zonconsent=' + datetime.now().isoformat() for the cookie header.

Codecov Report

Merging #108 (cfb9cb7) into master (6a03547) will decrease coverage by 0.22%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   94.36%   94.13%   -0.23%
==========================================
  Files          19       19
  Lines        2571     2575       +4
==========================================
- Hits         2426     2424       -2
- Misses        145      151       +6

Impacted Files
Coverage Δ

trafilatura/downloads.py
92.24% <90.00%> (+0.24%)
:arrow_up:

trafilatura/utils.py
95.32% <0.00%> (-1.17%)
:arrow_down:

trafilatura/cli_utils.py
90.29% <0.00%> (-0.98%)
:arrow_down:

trafilatura/spider.py
89.36% <0.00%> (-0.71%)
:arrow_down:

trafilatura/metadata.py
96.86% <0.00%> (-0.40%)
:arrow_down:

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 6a03547...cfb9cb7. Read the comment docs.

Hi, thanks for the changes! Why is the COOKIES option renamed to COOKIE? Is there a problem with multiple cookies? I saw that older versions of Python treated an inline semicolon as start of a comment, but in recent versions of Python this seems to be fixed, or is there another reason?

Hi @muellermartin, thanks!
We could also think about supporting a series of cookies if you're interested – probably as key-value pairs website/cookie.

Thanks for merging and your additions <3
As far as I understand it, multiple cookies can now be added by separating them with a semicolon in the Cookieheader (reference). A better interface for adding cookies per website like suggested would be nice though. I thought about using something like a CookieJar, but the currently used urllib3 seems not able to use them...

It seems feasible via a SessionManager but I cannot find a user guide for it in the docs:

https://github.com/urllib3/urllib3/pull/904
https://urllib3.readthedocs.io/en/latest/changelog.html?highlight=cookiejar#id24

It seems feasible via a SessionManager but I cannot find a user guide for it in the docs:
* [Approved SessionManager changes urllib3/urllib3#904](https://github.com/urllib3/urllib3/pull/904)

* https://urllib3.readthedocs.io/en/latest/changelog.html?highlight=cookiejar#id24

It seems like 904 was merged into "urllib3:sessionmanager", but this was then deleted, so it never got into master. It seems like the idea is to use CookieJar instead...
https://github.com/urllib3/urllib3/pull/1229/files#diff-8bb5645ed9da6ed97491165b18352e739650f1d4422e2134c6f5500d6384b6daR39
and
https://docs.python.org/3/library/http.cookiejar.html#http.cookiejar.CookieJar.add_cookie_header

@Foorack thanks!
Would any of you be interested in trying this in a PR? (cc @muellermartin)


Tabbable onFocusVisible prop

🚀 Feature request
Motivation
There are cases where we want to perform actions on focus, but not when focus is triggered by a pointer event.
For example: https://github.com/reakit/reakit/pull/633#discussion_r488117927
Example
const [show, setShow] = React.useState(false);
<Tabbable onFocusVisible={() => setShow(true)} onBlur={() => setShow(false)} />
{show && <div />}

Possible implementations
Maybe checking mouse and keyboard events before focus and only triggering onFocusVisible when it's preceded by a keyboard event? I don't know.

Can probably reference the work done on the focus-visible polyfill as it's pretty well documented.
I use it in my company's project and have no issues with it. Extracting out the javascript code into a hook might not be that bad.

I have a working prototype that I'll PR soon ✋


Add asset symbol and decimal for polkadex

In this PR, Assets Symbol and it's decimal added for Polkadex Network.

Hey @ap211unitech , thamk you for your contribution! This will be merged and released either today or tommorow latest. (Waiting for this PR to be merged first https://github.com/paraspell/xcm-tools/pull/189#pullrequestreview-1990791910)

Merging this. Will release soon and ping you afterward @ap211unitech .

Hey @ap211unitech SDK is now released containing updated asset data on for your chain.
https://github.com/paraspell/xcm-tools/pull/194#event-12440809379

Thanks @dudo50


2024-11-05_01-35-33_master-next_aws-cli-v2

Automatically created.

Backport failed for kirkstone-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin kirkstone-next
git worktree add -d .worktree/backport-10253-to-kirkstone-next origin/kirkstone-next
cd .worktree/backport-10253-to-kirkstone-next
git switch --create backport-10253-to-kirkstone-next
git cherry-pick -x 39cab8c3eef83774347e2069d21f4346b6532049

Successfully created backport PR for scarthgap-next:

#10256

Successfully created backport PR for styhead-next:

#10257


x-axis and y-axis problem

In this library how to set label for x axis and y axis in 'line char' and 'bar chart'.

this library can't set label for x axis and y.


Broken link to LaTeX on homepage schedule

Currently the schedule displays the link to our LaTeX content as
    [LaTeX][latex]

which seems to indicate we're missing a markdown link to
    https://www.cs.cmu.edu/~15131/f16/topics/latex/

Oh you're right; do you want to make a PR?

I don't have time tonight, but maybe later in the week? Just wanted to make an issue in case someone else was able to get to it before I could.

Closed by #11


Clean up vmmlist and running

If there is nothing in VMM or if there is nothing running, tell the user.

This also needs to be cleaned up in conlist as well.

VMM List is now cleaned. Conlist has it's own issue, and the rest don't error out, they just show empty. VMM list would error if empty.
iohyve v0.3.6


create medical program list endpoint

https://github.com/edenlabllc/ehealth.api/issues/1098

Changes Unknown when pulling c9398a8a8aec86c0b180904da3ef476d55736a74 on medical_program_list#1098 into ** on master**.

Changes Unknown when pulling 63db36106106e9f9953feb8833faff0db8cfb7dd on medical_program_list#1098 into ** on master**.


add and remove a couple questions

fixes #2, fixes #6, fixes #7, fixes #8, fixes #9, fixes #10

yeah the diff is not easy on the eye, but i'm basically just solving all the issues:

rm code coverage question
add question about academic position
merge questions on automated testing and usage of travis-ci/gitlab-ci ...
add general question testing (do you do unit tests, integration tests, ...)
add usage question on jupyter (since relevant to see if perhaps majority already using jupyter)

Thanks a lot!


Change favicon to new circle logo

Currently the favicon is an awkwardly chopped clam. Long live the clam, but let him be free to be a full clam away from the constraints of a square favicon.

Agreed. I suppose we should find another subtle place for the clam to live on.

clam must live on.

s/must/should/
We can find a less official place for it than the favicon.


recent review feedback for "search-howto-sql indexer" article

Hi @chaosrealm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


QueryVariable: Use correct option property for variable options sorting

sorting was carried out agains option.text instead of option.label

📦 Published PR as canary version: 5.36.1--canary.1015.12484333229.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>

also, for numeric sort you can just subtract the values from each other and skip the if/conditions.


Syntax highlighting second single quotes are black in backtrace

see screenshot: https://skitch.com/russcloak/8fiak/pm-task-reminder-job-spec.rb-rpp-web-app

Note to self, fixed this issue in my development branch.


Several people had trouble getting into the sandbox

See https://forum.mendixcloud.com/link/questions/107208 for one of them. Thought it would be nice to help them out in the documentation too.

@MarkvanMents please take a look from deployment POV.

Reviewed on Slack.


mustHaveVGPUs variable are reduced to single form.

Errors occur when interacting with PVE, this commit corrects this. This is due to the different name of the variable mustHaveVGPUS.

Thank you very much!!
Merged ;-)


Fireblocks Integration: V1

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
SDFSOF-15: Implemented controller for handling Fireblocks webhook notifications. Added signature validation.
SDFSOF-17: Define Anchor template config for FB integration
SDFSOF-21: Implement cron job scheduler for FB updates
SDFSOF-26: Update runner for Custody component
Why
Initial commit
Known limitations
[TODO or N/A]

Something went wrong with PR preview build please check


Git validation is disabled by default

Changes
Fix issue: https://github.com/shipwright-io/build/issues/659
Default, build controller won't check sourceURL unless build.shipwright.io/verify.repository annotation is true.

Switch to don't validate sourceURL by default.
Update related unit tests and integration tests.
Update related docs.

Submitter Checklist

[x] Includes tests if functionality changed/was added
[x] Includes docs if changes are user-facing
[ ] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes

Git validation behaviour changes

sourceURL validation is disabled by default.

thanks !


Webpack skeleton

An Aurelia skeleton app built with webpack. Only e2e tests don't work at this moment.
PR contents:

Aurelia es2016 skeleton app with webpack support;
Production bundling and minifying example config
Unit tests with Karma

Very cool! I'll merge it in shortly.

@martijnboland , thanks for the effort!
I wanted to try it out, so i ran
npm install && npm run dev
and everyting seemed OK so far (including message "webpack: bundle is now VALID."),
but when opening http://localhost:3000/ i get an error both on FF an Chrome:

Is there anything missing from the instructions?
Also it would be nice to mention skeleton-es2016-webpack in top-level README.md and to include some motivation for using webpack over SystemJS.

+1 on this. I get the same error as @atsu85 and haven't found a solution yet.
But still, this looks great! Really good work @martijnboland, thanks!

Hmmm, this might be a timing thing. The aurelia-loader-webpack module sets itself into a global PLATFORM.Loader variable. The createLoader function in aurelia-bootstrapper checks if this variable is set and if so, it uses this loader instead of the default one. I suspect that createLoader is executed before PLATFORM.Loader is set.
Could you try to switch the import statements in /src/main.js and see if that helps? First aurelia-loader-webpack and then aurelia-bootstrapper.
@EisenbergEffect is defining a custom loader by setting the global PLATFORM.Loader variable actually the right way to use a custom loader?

@martijnboland Switching the import statements does not help unfortunately. I'm not really sure on what to try next but I am happy to help if you have any other ideas.

I think there's probably going to need to be a custom bootstrapper of some sort. Basically, today's bootstrapper is pretty tailored to require.js/system.js type scenarios. So, for webpack, we might need to have a different bootstrapper to get things up and running.
Does a webpack bundle have the notion of a main or entry module? If so, we could use that to create a bootstrapper that does what today's bootstrapper does but that is hardcoded to use the webpack loader. It would be pretty simple, based on our existing bootstrapper but removing all the System stuff, which wouldn't be necessary.

@devcube What OS are you using? Which node.js version? Do you have webpack installed globally?

@EisenbergEffect a custom bootstrapper will probably be the best way. Webpack does have an entry point where I currently import aurelia-bootstrapper and aurelia-loader-webpack. I'll give that a try.

Ok, sounds good. Just ping me if you have any questions or want me to review anything along the way. Again, thank you for helping out with this!

@martijnboland I'm on Windows 10 with node v4.2.4 and yes - webpack is installed globally.

I've updated the skeleton to use a dedicated bootstrapper (aurelia-bootstrapper-webpack, which is hardcoded to use the webpack).
@devcube, @atsu85 and others: could you please check if it works properly now?
@EisenbergEffect could you create an empty bootstrapper-webpack repo? Then I can send the new bootstrapper as PR.

I'll take care of this a bit later tonight. No problem. Thanks!

Ok, you are ready to go: https://github.com/aurelia/bootstrapper-webpack

@atsu85 and others: could you please check if it works properly now?

@martijnboland, now i'm getting different error:

Uncaught (in promise) TypeError: _aureliaPal.DOM.injectStyles is not a function

I can reproduce the issue with npm v2. It probably has something to do with the fact that npm v3 flattens and dedupes dependencies by default and v2 does not.

run 'npm dedupe' and see if that works?

nice, that did the trick (y)

I can reproduce the issue with npm v2

I'm also using npm v2 that came with current node LTS version

@martijnboland I have merged the bootstrapper PR and made a release to NPM. If you want to update this PR with that and confirm it all works, I'll merge this into our skeletons repo. Let me know and thank you for this great work!

@EisenbergEffect final commits added to the PR.

Works at my machine! :-)

Awesome work. I think we'll do a blog post this week to let people know it's available and get them to try it out. @martijnboland is there anything in particular you would like us to say about you in the post or about the skeleton?

@EisenbergEffect
While investigating Aurelia, I had some troubles with building and bundling and I thought why not give webpack a try, how hard can it be?
I started a spike to see what's possible, pretty much gave up on it, got enthusiastic again and then couldn't stop before at least the skeleton was working. It was a real challenge and great fun and I gained a lot of respect for Aurelia and its community. The overall feeling of the framework is that 'everything just makes sense' (except for the building and bundling story which felt overly complicated and brittle).
But here's the thing: I'm not really an Aurelia user (yet). My job is mainly .NET development and frontend work with React, Angular 1 and jQuery. I just wanted to check out Aurelia and things got a bit out of hand :).
It would be great if others from the Aurelia community could take over the webpack integration.
Regarding the skeleton and webpack integration: I don't know if the current webpack plugin and the Aurelia bootstrapper and loader can cover all scenarios. That something that still has to be found out by the community.
Also a little disclaimer: the way Aurelia loads the modules, plugins and templates with only a string identifier is something that still doesn't sit well with webpack and that won't improve with webpack 2. Webpack needs at least some context to resolve modules dynamically, something a string identifier only can not provide. The aurelia-webpack-plugin essentially builds a static map (the context) where aurelia-loader-webpack loads its modules from.

@martijnboland, thanks also for the story :)

Webpack needs at least some context to resolve modules dynamically, something a string identifier only can not provide. Therefore I had to create the aurelia-webpack-plugin that essentially builds a static map (the context) where aurelia-loader-webpack dynamically loads its modules from.

I may have misunderstood, but it sound a lot like config.js file used by SystemJS that also provides the static map that resolves string module names to their location in file system. It looks like there is one difference: while config.js with dependencies map for systemjs is created using jspm cli (or manually), then webpack plugin builds the dependencies map on the fly (based on package.json, just like jspm cli). Have i misunderstood smth?
@EisenbergEffect, I'm really waiting for the aurelia-webpack blog post and hoping that in addition to usage and benefits of using webpack, the blog post also covers the disadvantages and how the aurelia-webpack works internally.

@atsu85 exactly, that's how it works. Btw, you could provide a static map yourself via the webpack plugin (the contextMap option).


The app doesn't get built for iOS

My app fails to build for iOS, yet works fine on Android. Flutter clean or pod install commands don't help.
Steps to Reproduce
flutter build ios

Logs

Encountered error while building for device.
Elliots-Mac-mini:club_lime elliot_thm$ flutter clean
Deleting 'build/'.
Deleting '/Users/elliot_thm/Documents/club_lime/.dart_tool/'.
Elliots-Mac-mini:club_lime elliot_thm$ flutter build ios
Building com.vivaleasure.clubLime for device (ios-release)...
Automatically signing iOS for device deployment using specified development team in Xcode project: 6NDB4SY6HS
Running pod install...                                              1.1s
Running Xcode build...
 ├─Building Dart code...                                    34.7s
 ├─Generating dSYM file...                                   0.3s
 ├─Stripping debug symbols...                                0.0s
 ├─Assembling Flutter resources...                           1.2s
 └─Compiling, linking and signing...                         1.7s
Xcode build done.                                           39.0s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Release ===
    2019-07-08 12:13:10.058 defaults[26804:842682]
    The domain/default pair of (/Users/elliot_thm/Documents/club_lime/build/ios/Release-iphoneos/Runner.app/Frameworks/App.framework/flutter_assets/ios/Flutter/App%202.framework/Info.plist, CFBundleExecutable)
    does not exist
    fatal error: lipo: can't map input file: /Users/elliot_thm/Documents/club_lime/build/ios/Release-iphoneos/Runner.app/Frameworks/App.framework/flutter_assets/ios/Flutter/App%202.framework/ (Invalid argument)
    fatal error: lipo: can't map input file: /Users/elliot_thm/Documents/club_lime/build/ios/Release-iphoneos/Runner.app/Frameworks/App.framework/flutter_assets/ios/Flutter/App%202.framework/ (Invalid argument)
    Failed to extract arm64 for /Users/elliot_thm/Documents/club_lime/build/ios/Release-iphoneos/Runner.app/Frameworks/App.framework/flutter_assets/ios/Flutter/App%202.framework/. Running lipo -info:
    fatal error: lipo: can't map input file: /Users/elliot_thm/Documents/club_lime/build/ios/Release-iphoneos/Runner.app/Frameworks/App.framework/flutter_assets/ios/Flutter/App%202.framework/ (Invalid argument)

flutter analyze
Analyzing club_lime...
No issues found! (ran in 2.5s)

flutter doctor -v
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.5 18F132, locale en-AU)
    • Flutter version 1.5.4-hotfix.2 at /Users/elliot_thm/flutter
    • Framework revision 7a4c33425d (10 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at /Users/elliot_thm/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • ios-deploy 1.9.4
    • CocoaPods version 1.7.2

[!] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.36.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.2.0

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)

! Doctor found issues in 1 category.

At the end I was able to resolve the issue by creating a new project using
flutter create -i swift -a kotlin project_name
and moving lib folder there without any changes to the dart code.

This doesnt help at all... Older projects cannot do this.


No layer.zip file

I cant find the layer.zip file in your repo.

You're right, it's missing on master branch. But the layer.zip is there on the mysql-5.6 branch.
I'll add the missing build_output/layer.zip for master branch in the next few mins.

@SunainaDG, the layer.zip file is now available in the build_output directory.
What version of MySQL do you need it for?

@SunainaDG : the layer.zip file is now available. If this meets your requirements, can this issue be closed now?

@nonbeing : I need it for 8

Hey you think there is a way to build a similar package for AWS glue?

@SunainaDG the layer.zip file is available for 8 in master branch in the build_output directory as shown here:

I'm not aware of how to use this for AWS glue, this is meant for AWS Lambda Layers.

Ah no yes I know its for Lambda, just thought of asking in case you might know.
Anyway thanks for the zip. :)

Closing this issue.


Add link to project page in simulator#1729

Fixes #
Describe the changes you have made in this PR -
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage increased (+2.5%) to 88.039% when pulling aa9c7ac8a8087979e867616b6c47d6aa676b8686 on GAUTAMSAHARAN:add-link-to-project-page-in-simulator#1729 into f2583fa656b3e0cb340920c515a0a34351855f78 on CircuitVerse:master.


Cast to dynamic shows an error at runtime and not at compile time.

Expected
Show a warning or error until reflection is supported.
Actual
Compile fine and show an error at runtime (Uncaught TypeError: value.CompareTo is not a function).
Steps To Reproduce
http://live.bridge.net/#8b601e588207bb89bf4c2e1fed66f41b
https://dotnetfiddle.net/BT1OpF
public class App
{
    public static void Main()
    {
        var value = (dynamic)0;
        Console.WriteLine((int)value.CompareTo(0));
    }
}

@vladsch, @geoffreymcgill, @LeonidVeriga, I think now reflection is implemented this could be fixed.


Enabling bayer

@syohex ,
Could you please check and merge this PR if there seems to be no problem?
Thank you very much for always helping us.

Thank you for merging!


User CreatesPR  Normal Mode – Significant  CLA Signed and  Approved_02ndApril2015

Hi @brilliomsinterop, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Issue with react router

Somehow I cant get this working with react router.
So following is my routing code:-
import { Router, Route, browserHistory, IndexRoute } from 'react-router';
import React from 'react';
import ReactDOM from 'react-dom';
import App from './App';
import About from './About';
import Contact from './Contact';
import Home from './Home';

ReactDOM.render((
     <Router history={browserHistory}>
      <Route path="/" component={App}>
       <IndexRoute component={Home}/>
       <Route path="/about" component={About}/>
       <Route path="/contact" component={Contact}/>
  </Route>
  </Router>
), document.getElementById('app'));

Following is the App component:-
class App extends React.Component{
  render(){
  var href = this.props.history.createHref('/about');
  return(

hello router

                <Link to='/about'>About</Link>
                 <Link to="/contact">Contact</Link>
                {this.props.children}

  )
}
}

So the issue I am facing is when i click on about the link changes but the content does not. However now if i press browser back I reach home and then i press browser forward I reach about again and this time both url and content change.
So to check this out i used the same generated bundled js file in a static html and tried to access to that file after npm start, it worked fine over there. However when I used the same file in django using web pack loader here I see the issue mentioned above

So after debugging a bit further and deep I see that when I use following in my html, the issue is seen:-
If i include javascripts.html in my index.html, the above issue is seen:-
`<!doctype html>

     {% load staticfiles %}
     {% load pipeline %}
     {% stylesheet 'wl-font-awesome' %}
     <link href="{% static 'lib/bootstrap/dist/css/bootstrap.min.css' %}" rel="stylesheet">
     {% stylesheet 'wlmain' %}
     <meta charset=utf-8/>
     <title>My First React Router App</title>

            {% load render_bundle from webpack_loader %}

{% render_bundle 'main' %}
{% load render_bundle from webpack_loader %}
{%include 'javascript.html' %}

`
Javascripts.html:-
`  {% load staticfiles %}

`
So basically when i call  {% load staticfiles %}, the issue is seen. Can you please share your comments on this?

So guys I did something really silly, I had a js file which was calling html.pushState :/ because of which URLS were changing :/ . I will go ahead and close the issue now. Sorry for the trouble.