Import-Module IISAdministration Fails With "Could not load file or assembly 'Microsoft.Web.Administration..."

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
On Windows, run the following script in PowerShell 7.2.1:
Install-Module -Name IISAdministration
Import-Module IISAdministration
Get-Module -ListAvailable

I have confirmed on a VM running 19042 that the Windows 10 build is not a factor as far as I can tell, because on a 20H2 VM I still see the problem behavior. I have also enabled IIS Management Console and IIS Management Scripts and Tools. The bulk of the documentation I could find referred to earlier versions of PowerShell 7 not supporting IISAdministration, but this doesn't seem to apply to 7.2.1 - if it did globally, I wouldn't have myself and multiple other team members running the scripts with no issues. I appreciate any help!
I did see in issue #13818 the reporter was told to look in the https://github.com/PowerShell/PowerShellModuleCoverage repo and I reported the issue there as well, but then I saw the repo appears to be defunct. Its last commit was literally 4 years ago. ALl the other issues I can find reported for IISAdministration and PowerShell Core / 7 are at least two years old and refer specifically to earlier versions of PowerShell being incompatible, but I can see from the fact that the module works on several developer machines in my department that is no longer true.
Expected behavior
The import should succeed and the module should be listed:
Script     <IP_ADDRESS>               IISAdministration                   Core,Desk {Get-IISServerManager, Set-IISConfigAtt…

Actual behavior
The import fails with the following error:
Import-Module: Could not load file or assembly 'Microsoft.Web.Administration, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.

The system cannot find the file specified.

The Get-Module list shows no cmdlets associated with IISAdministration:
Script     <IP_ADDRESS>               IISAdministration                   Core,Desk

Error details
No response
Environment data
On the machine exhibiting the problem behavior:
PS C:\Users\thoma> $PSVersiontable

Name                           Value
----                           -----
PSVersion                      7.2.1
PSEdition                      Core
GitCommitId                    7.2.1
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

On the machine working successfully:
PS C:\Users\thomas.parikkaADM> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.1
PSEdition                      Core
GitCommitId                    7.2.1
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Another year later and still no response...
I found an issue with at least one installation of IIS and this module. For some reason, in the C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules folder the IISAdministration module was installed possibly by the IIS feature installation, and the files are set to read-only and execute for permissions outside of the TrustedInstaller owner permissions. If it is installed with Install-Module, the folder C:\Program Files\PowerShell\Modules is used and installed into the versioned folder there.
The workaround I figured out is a multi-step process.

Install the module: Install-Module -Name IISAdministration -Scope AllUsers -AllowClobber
Copy the version folder (current version as of this comment: <IP_ADDRESS>) to C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\IISAdministration
Uninstall the module: Uninstall-Module -Name IISAdministration -AllVersions
Import the module with the -RequiredVersion parameter: Import-Module -Name IISAdministration -RequiredVersion <IP_ADDRESS>

This works perfectly, although very annoying. Note that if you do not use the -RequiredVersion parameter, you will get the previous version. This version and the previous version do not show on Get-Module -ListAvailable because, for some reason, PS7+ ignores the PSModulePath value of C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules (bug or expected behavior? I'm leaning towards the latter as you might get some very unexpected results from very old modules in PS7+.)

As a side note, when importing the module you get a message to use the -SkipEditionCheck parameter. This results in the aforementioned Could not load assembly error.

Using this parameter fails to load the module as expected. Ignoring the initial warning message appears to still load the module and allow cmdlets to work.

I was issued a new laptop by work last week, Windows 10 22H2, and PS 7.3.3:
PowerShell 7.3.3
PS C:\Users\thomas.parikka> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.3.3
PSEdition                      Core
GitCommitId                    7.3.3
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Same issue occurs, was forced again to delete the IisAdministration module from C:\Program Files\PowerShell\Modules before anything would function.

At the end of the day, this is an issue for the IIS Team. They need to update the module to work properly with .NET Core. Until then, the best you will probably get is using the module via the Windows PowerShell compatibility mechanism. The fix here is probably external (and frankly, given the Windows Division;'s apparent lack of enthusiasm) will probably never happen.  I'd love to be wrong, but the IIS and other teams have known about PowerShell for half a decade and have done little to make the various modules directly useable in PowerShell 7.
In my latest PowerShell book, I gave up on the notion that MS would change things like this, so the last chapter is how to use WSUS with PowerShell 7. Not pretty, but here are the scripts: https://github.com/doctordns/PacktPS72/tree/main/Scripts/Ch 14 - WSUS

@doctordns Does the IIS team have a GitHub where an issue like this would better belong? It seems wild to me that this would be left by the wayside given Microsoft as a whole is moving past .NET Framework in every other respect.

I have retested this today and here is what I';ve found. If I load the Web-Server in Windows Server 2022 and  include the management tools: as expected, the IISAdministration module is present, can be loaded, and works as designed. The IISAdministration module on ther PS Gallery appears to be the very same module - same Guid in the manifest, for example. I downloaded this and can install the module and use it.
tl:dr - I can not currently repo the issue
I also note an issue at your end: namely IISAdministration module is being loaded using the compatibility mechanism. On all my servers, using PowerShell 7.2.6, the module loads natively.

The cmdlet working group couldn't repro the issue on a Windows 11 test system. If issues exist, they are related to this particular module and it requirements that go beyond PowerShells'  built-in backwards compatibility machanism. The module was last updated in 2017. To solve the issue, you'd have to approach the module author.

Another year later and still no response...
I found an issue with at least one installation of IIS and this module. For some reason, in the C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules folder the IISAdministration module was installed possibly by the IIS feature installation, and the files are set to read-only and execute for permissions outside of the TrustedInstaller owner permissions. If it is installed with Install-Module, the folder C:\Program Files\PowerShell\Modules is used and installed into the versioned folder there.
The workaround I figured out is a multi-step process.

Install the module: Install-Module -Name IISAdministration -Scope AllUsers -AllowClobber
Copy the version folder (current version as of this comment: <IP_ADDRESS>) to C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\IISAdministration
Uninstall the module: Uninstall-Module -Name IISAdministration -AllVersions
Import the module with the -RequiredVersion parameter: Import-Module -Name IISAdministration -RequiredVersion <IP_ADDRESS>

This works perfectly, although very annoying. Note that if you do not use the -RequiredVersion parameter, you will get the previous version. This version and the previous version do not show on Get-Module -ListAvailable because, for some reason, PS7+ ignores the PSModulePath value of C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules (bug or expected behavior? I'm leaning towards the latter as you might get some very unexpected results from very old modules in PS7+). Import-Module does not ignore that path.

@doctordns Unfortunately, I did not had any luck with this method. I cannot replace the existing files in C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\IISAdministration
All my tests are done in Windows Server 2019 v1809 build 17763.4499
This version of the OS come with version 1.0 of IISAdministration and for multiple reasons I need the version <IP_ADDRESS>
The command Import-Module IISAdministration -RequiredVersion <IP_ADDRESS> (With or without -SkipEditionCheck) fail. So I need to install the latest version present in the PowerShell Gallery.
As mentioned above, install the module in PS7 with the following command:
Install-Module -Name IISAdministration -Scope AllUsers -AllowClobber

After the installation:
Import-Module IISAdministration -RequiredVersion <IP_ADDRESS>
Import-Module: Could not load file or assembly 'Microsoft.Web.Administration, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified.

To solve this, copy C:\Windows\System32\inetsrv\Microsoft.Web.Administration.dll in the directory where IISAdministration <IP_ADDRESS> is installed: C:\Program Files\PowerShell\Modules\IISAdministration\<IP_ADDRESS>
Now import the module again and it will work:
PS C:\Windows\System32> Import-Module IISAdministration -RequiredVersion <IP_ADDRESS>
PS C:\Windows\System32> Get-Module -List IISAdministration

    Directory: C:\Program Files\PowerShell\Modules

ModuleType Version    PreRelease Name                                PSEdition ExportedCommands
---------- -------    ---------- ----                                --------- ----------------
Script     <IP_ADDRESS>               IISAdministration                   Desk

I hope this can help.

I was also getting Assembly with same name is already loaded errors, but trying to get commands from the module also failed:
Import-Module: C:\Users\<redacted>.ps1:47
Line |
  47 |      Import-Module -Name IISAdministration
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Assembly with same name is already loaded

Get-Command: C:\Users\<redacted>.ps1:48
Line |
  48 |      Get-Command -Module IISAdministration -Name Get-IISAppPool
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'Get-IISAppPool' is not recognized as a name of a cmdlet, function, script file, or executable program.
     | Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
WARNING: Failed to stop the Revenue Cycle Dashboard Pool The 'Get-IISAppPool' command was found in the module 'IISAdministration', but the module could not be loaded due to the following error: [Assembly with same name is already loaded]
For more information, run 'Import-Module IISAdministration'.

Flipping the import to use the -UseWindowsPowerShell flag resolved the issue for me:
Import-Module -Name IISAdministration -UseWindowsPowerShell


[Error] Vcpkg install darknet  When building FFmpeg package cause failed

Host Environment

OS: Windows10
Compiler: Visual Studio 2019

To Reproduce
Steps to reproduce the behavior:
./vcpkg install darknet[opencv-base,cuda,cudnn]:x64-windows
Since msys2 download server has been down for few days, I have add string(REGEX REPLACE "^https://repo\.msys2\.org" "http://mirrors.ustc.edu.cn/msys2" url ${url}) at scripts\cmake\vcpkg_download_distfile.cmake:161 to redirect the download server to another mirror.
Failure logs
-- Building Release Options:  --extra-cflags=-MD --extra-cxxflags=-MD                                                -- Building ffmpeg for Release                                                                                CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:85 (message):                                          Command failed: C:/Users/459194/vcpkg/downloads/tools/msys2/8f86d90f10246c56/usr/bin/bash.exe ./build.sh             Working Directory: C:/Users/459194/vcpkg/buildtrees/ffmpeg/x64-windows-rel                                           Error code: 2                                                                                See logs for more information:                                                                                C:\Users\459194\vcpkg\buildtrees\ffmpeg\build-x64-windows-rel-out.log                                                C:\Users\459194\vcpkg\buildtrees\ffmpeg\build-x64-windows-rel-err.log                                                                                Call Stack (most recent call first):                                                                                ports/ffmpeg/portfile.cmake:411 (vcpkg_execute_required_process)                                                     scripts/ports.cmake:79 (include)                                                                                Error: Building package ffmpeg:x64-windows failed with: BUILD_FAILED                                                 Please ensure you're using the latest portfiles with .\vcpkg update, then                                          submit an issue at https://github.com/Microsoft/vcpkg/issues including:                                                Package: ffmpeg:x64-windows                                                                                Vcpkg version: 2020.06.15-nohash
Additional context

build-x64-windows-rel-err.log

make: *** [/cygdrive/c/Users/459194/vcpkg/buildtrees/ffmpeg/src/n4.2-2f6d2343f6.clean/ffbuild/common.mak:59: libavutil/buffer.o] Error 2
make: *** Waiting for unfinished jobs....
Attachments are the error log:
build-x64-windows-rel-err.log
build-x64-windows-rel-out.log

Seems I saw this issue before.
@Neumann-A @cenit what do you think?

C:/Users/459194/vcpkg/buildtrees/ffmpeg/src/n4.2-2f6d2343f6.clean/libavutil/buffer.c(19): fatal error C1083: Cannot open include file: 'stdatomic.h': No such file or directory

I can't repro this issue now.

I currently have the same issue. If anyone can help, please do!

Can you use command ./vcpkg install ffmpeg --editable and modify VCPKG_ROOT/buildtrees/ffmpeg/src/FOLDER_WITHOUT_CLEAN/libavuti;/buffer.c line 19,
from #include <stdatomic.h> to #include <atomic>
and try again?

The msys2 upstream supported TLS 1.2, so I think this issue is resolved.


Allow DOI to be used to download dataset

Now that kaggle datasets have associated DOIs, allow that DOI to be used as an alias to download the dataset, e.g.
instead of
kaggle datasets download fairumn/cdms-dataset

one could do
kaggle datasets download 10.34740/kaggle/dsv/2660709

Thank you so much for information


Has anyone implemented "GetExpressCheckoutDetails" to retrieve Shipping details?

I need to retrieve the shipping information that was entered into PayPal, however the Response object only returns the following data:
string 'array (
  'TOKEN' => 'EC-XXXXX,
  'TIMESTAMP' => '2012-05-22T03:19:53Z',
  'CORRELATIONID' => 'fd3aba347de6',
  'ACK' => 'Success',
  'VERSION' => '65.1',
  'BUILD' => '2860716',
)' (length=193)

Has anyone implemented GetExpressCheckoutDetails? http://www.paypalobjects.com/en_US/ebook/PP_NVPAPI_DeveloperGuide/expresscheckout.html#2063856
Thanks

This issue is PayPal-specific so I'm closing it in favour of schmittjoh/JMSPaymentPaypalBundle#25


Add jump to folder sample solution

Q
A

New sample?
yes

What's in this Pull Request?
Adding new sample: Jump to folder list view command set extension

There seems to be an issue related to git and WSL2...files seemed to be messed up so will close and submit again


[Docs] Add "How to Authenticate" to the REST API docs

Current State
Developers using Meshery's REST API are not provided authentication instruction and example.
Desired State
The start of the REST API docs should have a section on how to retrieve and use Meshery API tokens to interact with the REST API.

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Put me on this one @leecalcote , I need a list of things to do in my API doc/updates framework I am working on.

@leecalcote For both this issue and the GraphQL docs token issues, would a link, with description of link to "authenticate-with-meshery" page be sufficient or do you want a complete reprint (with pictures?) of that information on the API reference page?

@Revolyssup this is what I was mentioning in CI today

@leecalcote this issue can be closed now, right ? // @Chadha93


Make use of padding space when possible

Normally when adding or removing tags, the data is just directly inserted or cut out.
With these changes the padding space in the metadata is used instead (overwritten when tags are added, or inserted when tags are removed), to preserve the original file size if the tag changes are small enough.
This works for my use case but I realize the usefulness of this functionality is subjective, so feel free to close this PR if you don't think they are worth including.

Thank you for your proposal.
Here’s what https://datatracker.ietf.org/doc/html/rfc7845.html has to say about the extra_data block:

Immediately following the user comment list, the comment header MAY
contain zero-padding or other binary data that is not specified here.
If the least-significant bit of the first byte of this data is 1,
then editors SHOULD preserve the contents of this data when updating
the tags, but if this bit is 0, all such data MAY be treated as
padding, and truncated or discarded as desired.  This allows informal
experimentation with the format of this binary data until it can be
specified later.

To be honest, I have no idea what people do with this block, so opustags currently always keeps it intact. Your code seems to assume the extra_data is always padding, but according to the specs that’s not always the case.
Regarding the usefulness of the feature, I don’t undestand why you would like to keep the original file size instead of shrinking it when removing data from the tags. Do you have a use case where preserving the file size matters?

Well, since most of the bytes of the file are left unchanged, I think any slightly smart binary comparison tool will align data properly. Regarding conservativeness, it’s unclear which is better between keeping extra_data intact or the file size unchanged.
I’m gonna keep the current state because it’s simpler and safe, unless someone comes with a concrete use case.


[rust] Add cargo raze to the project

I've included a number of patches, all corresponding to open PRs with
the cargo_raze project:
google/cargo-raze#481 - Link the project-local libssh2 with libssh2-sys.
google/cargo-raze#488 - Add compiler flags to build correctly in a C11 environment.
google/cargo-raze#491 - Facilitate local crate development by allowing local_repository
overrides.
google/cargo-raze#492 - Allow skipping submodule initialization on git repositories.
Cargo raze can be invoked from bazel and will update dependencies in known Cargo.toml files:
bazel run //:cargo_raze

I confirmed, this will require upgrading the bazel custom image on our nightly regression infrastructure (otherwise it will completely break all nightly regressions). If you could hold off on merging this change until we can upgrade the bazel custom image (and announce it to the DV folks) that would be great. I have already prepped the change and will make the announcement and submit the change tomorrow AM. Sorry for any inconvenience. (Note: #13064 is already tracking the need for a way to detect / automatically update said infrastructure when changes like this are made.)

This can now be merged, as the airgapped caches have been updated. Thanks for holding!


Exception raised while executing Node _UploadMetric

What happened?
MRIQC crashed while attempting to upload metrics. Internet connectivity should not be an issue on this computer, and it looks like a variable intended to be a str was coded as something else?
What command did you use?

Singularity
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
Node: _UploadMetrics1
Working directory: /scratch/mriqc_wf/funcMRIQC/5f1d50ce95794c9e867ecec5e22ed18a7961e99c/UploadMetrics/mapflow/_UploadMetrics1

Node inputs:

auth_token = 8sSYVI0XjFqacEMZ8wF4
email = <undefined>
endpoint = https://mriqc.nimh.nih.gov:443/api/v1
in_iqms = /base/derivatives/mriqc/sub-10803sp/func/sub-10803sp_task-sharedreward_acq-mb3me3ip0_echo-2_part-mag_bold.json
modality = undefined
strict = False

Traceback (most recent call last):
  File "/opt/conda/lib/python3.11/site-packages/mriqc/engine/plugin.py", line 64, in run_node
    result['result'] = node.run(updatehash=updatehash)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/lib/python3.11/site-packages/nipype/pipeline/engine/nodes.py", line 527, in run
    result = self._run_interface(execute=True)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/lib/python3.11/site-packages/nipype/pipeline/engine/nodes.py", line 645, in _run_interface
    return self._run_command(execute)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/lib/python3.11/site-packages/nipype/pipeline/engine/nodes.py", line 771, in _run_command
    raise NodeExecutionError(msg)
nipype.pipeline.engine.nodes.NodeExecutionError: Exception raised while executing Node _UploadMetrics1.

Traceback:
	Traceback (most recent call last):
	  File "/opt/conda/lib/python3.11/site-packages/nipype/interfaces/base/core.py", line 397, in run
	    runtime = self._run_interface(runtime)
	              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
	  File "/opt/conda/lib/python3.11/site-packages/mriqc/interfaces/webapi.py", line 186, in _run_interface
	    errmsg = '\n'.join(
	             ^^^^^^^^^^
	TypeError: sequence item 6: expected str instance, bytes found

Additional information / screenshots
Data are available on OpenNeuro: https://openneuro.org/datasets/ds005085/versions/1.0.0

I am getting the same error, were you able to figure out a solution?

I ended up adding --notrack --no-sub to prevent any uploads. With those flags in place, it will run.

thank you!

I encountered the same problem with version 24.0.2 and solved it the same way ( thanks @DVSneuro ! )
A suggestion for the developers: The whole processing should not crash if the upload/submission fails as that can be quite frustrating, both in terms or repeating the computationally expensive processing as well as in terms of the time consuming debugging.
A warning/notification could be printed instead (possibly requesting a manual upload if those logs are super important for you ; )


possible_enhancements

@DKirwan - I saw your calendar on HN today and like what you've done with it.
I've done a little work with this visualization as well, which I've shared in this file.
Most notably chained transitions and data generation.

Thanks, Matt! I'll have a look at integrating your calendar animation (http://bl.ocks.org/mph006/b8c8161390001dbf4e99).
P.s.: The sparkline animations here are nice. Definitely something for me to look into for a separate project I'm working


Update sites.yml

Add pvac.xyz

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[ ] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: pvac.xyz
  url: https://pvac.xyz/
  size: 10.4
  last_checked: 2023-05-04

GTMetrix Report: https://gtmetrix.com/reports/pvac.xyz/F8pzV0qr/

I am not sure my HTML only approach fits or not the "ultra lightweight site", so I didn't check. Take care.

Yes, this is a ultra minimal site. Thanks for taking the time to stop by though!

Thank you for time!


Finalise store

Codecov Report

Merging #38 (6243b51) into master (c088f7d) will increase coverage by 0.14%.
The diff coverage is 82.97%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
+ Coverage   71.04%   71.18%   +0.14%
==========================================
  Files          23       26       +3
  Lines         442      472      +30
  Branches       60       56       -4
==========================================
+ Hits          314      336      +22
- Misses        119      125       +6
- Partials        9       11       +2

Impacted Files
Coverage Δ

projects/store/src/lib/store/store.ts
58.13% <55.00%> (-16.33%)
:arrow_down:

...rojects/test/src/lib/expect-observable/matchers.ts
66.66% <66.66%> (ø)

...s/test/src/lib/expect-observable/test-scheduler.ts
83.33% <83.33%> (ø)

projects/store/src/lib/utilities/scan-instance.ts
90.90% <90.90%> (ø)

projects/store/src/lib/effect/effect.ts
100.00% <100.00%> (+30.00%)
:arrow_up:

projects/store/src/lib/reduce/reduce.ts
100.00% <100.00%> (ø)

...est/src/lib/expect-observable/expect-observable.ts
100.00% <100.00%> (+6.06%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Enable Hot-reload for CLI

Why do we need this ?

The current dev experience for the main app is made super convenient using ./dev.sh which triggers the CLI to manage the Docker containers
The dev experience for the CLI itself is not convenient as simple changes to the CLI require a reload.

Acceptance Criteria

[ ] Enable Hot reloading on the CLI
[ ] Make sure the hot reload to the CLI itself does not take too long and does not build the containers again from scratch

Further Comments / References

Hey @samad-yar-khan  Can you explain this issue in more detail
Currently we build and start the CLI its like starting point of the development server that handles everything and  we have dev script in the CLI if someone want to run that in watch mode with hot reload.
so Can you please explain are we trying to achieve here ?

@VipinDevelops this issue basically meant that when you're working on a change in the CLI itself, instead of shutting down and restarting all changes, it should have a live-reload functionality similar to how you'd have experienced in the usual UI dev.

Hey @jayantbh Thanks for explaining, but we already have a dev scripting in the CLI that can run it with hot reload.

@VipinDevelops does it actually work like live-reload for you? Because it seems to basically trigger a whole-app-reload for the CLI for me on changes.

So @jayantbh correct me if I am wrong we want to make a feature that lets use inject updated source code into a running app without a restart

Correct. The running app here is the CLI app. Not the frontend or backend code.

Got it and that code with run other services thanks for explaining I will try to explore potential solution for this.


sonnar-runner is now called sonar-scanner

Changed ocurrences of sonar-runner to sonar-scanner.

Coverage remained the same at 72.344% when pulling 8ba5c3667956f535327a558b35f9a1648ee45d1f on alessandrojcm:master into 5c71ecaa744861e8a1d17601d5db5452ddbb44e6 on pybuilder:master.

Coverage remained the same at 72.344% when pulling 8ba5c3667956f535327a558b35f9a1648ee45d1f on alessandrojcm:master into 5c71ecaa744861e8a1d17601d5db5452ddbb44e6 on pybuilder:master.

So, the build fails because Sphinx does not support Python 2.6 anymore. What else should I do?

Nothing for now. :smile: We'll merge your changes after 0.12.0, which deprecates 2.6 and 3.3.

Please rebase on latest master.

Done.

Coverage increased (+2.7%) to 75.006% when pulling 88f2fdc6dbec530da5fb2ec58a55394b605dec11 on alessandrojcm:master into 5c71ecaa744861e8a1d17601d5db5452ddbb44e6 on pybuilder:master.

That doesn't look like a rebase to me ;)
git rebase -i master and then fixup or squash all commits but the first one. Push-force into a branch.

Sorry, still new to git. Could you check now?

Coverage remained the same at 75.006% when pulling dc7b31505a7ab0fcaeee9dec7694554691346066 on alessandrojcm:master into d0f8f809a7babe365927492bce641bdce9bad71d on pybuilder:master.

Hi, does anyone know when we can expected this will be merged? Thank you


Rename Python::with_gil -> with_python ?

An idea with the upcoming free threaded Python around the corner.
Maybe instead of Python::with_gil, a future PyO3 API can just be pyo3::with_python. No functional difference except dropping the GIL name as that won't always be relevant.
e.g.
Python::with_gil(|py| /* do stuff */)

becomes just
with_python(|py| /* do stuff */)

Maybe we include with_python in the prelude, maybe we invite users to import it.

Just throwing some alternatives in the room: Python::in_scope or Python::with_token could also be something if we want to keep it as an (associated) method, but without references to the gil.

I also have recently been thinking about Python::attach (and .allow_threads could become `.detach()) - this is the terminology introduced in freethreaded Python - you're either attached to a Python thread state, or the thread state is detached from the interpreter.

I also have recently been thinking about Python::attach (and .allow_threads could become .detach()) - this is the terminology introduced in freethreaded Python

After reading through #4577 I think I quite like these.


Fix UPDATED_NEW return values differences

This should fix #2798
The general idea was to play all the UpdateItem queries on a real DynamoDb and dump the value in the tests.
I then added a method that recursively checks for changes in the new item. It works but as stated in the issue I may miss some corner cases.
I don't do a lot of python so this may not be the cleanest code you've reviewed, i'll be glad to improve it following advices 😉

Coverage decreased (-0.2%) to 93.952% when pulling ba1bf09474933976f422179e3e902d3866b0892c on ImFlog:fix_dynamodb_updated_new into 55a150082743ac11f50a4675a33d07e613fd392b on spulec:master.

Hi @bblommers, I just added some new tests on the existing update_item ones. Also I saw an existing test that does exactly what you asked for :
def test_update_return_attributes():
    dynamodb = boto3.client("dynamodb", region_name="us-east-1")

    dynamodb.create_table(
        TableName="moto-test",
        KeySchema=[{"AttributeName": "id", "KeyType": "HASH"}],
        AttributeDefinitions=[{"AttributeName": "id", "AttributeType": "S"}],
        ProvisionedThroughput={"ReadCapacityUnits": 1, "WriteCapacityUnits": 1},
    )

    def update(col, to, rv):
        return dynamodb.update_item(
            TableName="moto-test",
            Key={"id": {"S": "foo"}},
            AttributeUpdates={col: {"Value": {"S": to}, "Action": "PUT"}},
            ReturnValues=rv,
        )

    r = update("col1", "val1", "ALL_NEW")
    assert r["Attributes"] == {"id": {"S": "foo"}, "col1": {"S": "val1"}}

    r = update("col1", "val2", "ALL_OLD")
    assert r["Attributes"] == {"id": {"S": "foo"}, "col1": {"S": "val1"}}

    r = update("col2", "val3", "UPDATED_NEW")
    assert r["Attributes"] == {"col2": {"S": "val3"}}

    r = update("col2", "val4", "UPDATED_OLD")
    assert r["Attributes"] == {"col2": {"S": "val3"}}

    r = update("col1", "val5", "NONE")
    assert r["Attributes"] == {}

    with assert_raises(ClientError) as ex:
        r = update("col1", "val6", "WRONG")

So the upsert is covered and overall I think that the majority of the cases are.
I'll be glad to improve it if someone spot a difference during the tool usage :)

The tests don't pass on python 2.7, I found a really strange behavior while debugging the issue, with two DynamoType I end up with
original == changed # True
original != changed # True

And that's why it doesn't pass...
Any idea about what could cause this ?

List ordering in Python2 and 3 is usually different - make sure both lists are sorted before comparing them

I finally understand what happened, this is because of the fact that ne need to be redefined when eq is redefined (https://docs.python.org/2.7/reference/datamodel.html#object.ne) in Python2.

Good find. Thank you!


For key "Exec" in group contains an invalid field code "%a"

After solving https://github.com/pjones/plasma-manager/issues/103 - I've come across a new, different issue, which has to do with the command string I'm using for a hotkey:
    hotkeys.commands."alacritty-dropdown" = {
      name = "Launch Alacritty";
      key = "Alt+Space";
      command = "tdrop -a -h 90% alacritty";
    };

Error is:
error: builder for '/nix/store/dn5f2j2dbcfga84xwm9mzlc3cb2z3623-plasma-manager-commands.desktop.drv' failed with exit code 1;
       last 1 log lines:
       > /nix/store/yli5k9b9vrnf3cg6jjjckzshdbh0y8mi-plasma-manager-commands.desktop/share/applications/plasma-manager-commands.desktop: error: value "/nix/store/dzp7d4k1d94s1x49p9171mvcsfyxr7bj-systemd-254.6/bin/systemd-cat --identifier=plasma-manager-commands-alacritty-dropdown  tdrop -a -h 90% alacritty" for key "Exec" in group "Desktop Action alacritty-dropdown" contains an invalid field code "%a"
       For full logs, run 'nix log /nix/store/dn5f2j2dbcfga84xwm9mzlc3cb2z3623-plasma-manager-commands.desktop.drv'.
error: 1 dependencies of derivation '/nix/store/k0v7b7zs5vakpws67rcxq6q6ksd3q1rn-home-manager-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/qswlsl2a5dm9iwgf4bl3skrq0s75f9bz-home-manager-generation.drv' failed to build

Trying to escape the % with \ doesn't seem to work.
Full nix config is here (linked to offending line in question: https://github.com/TimekillerTK/nix-test/blob/dev/home/tk/beltanimal.nix#L73

It appears that escaping normally in nix doesn't work as this is using systemd syntax, and the % must be escaped in systemd syntax. You should be able to escape % with %%. Try this and let me know if it works. I got it working on my end anyway. We could consider adding code to escape it automatically though.

It manages to build successfully with %%, but that somehow results in two % being passed to the tdrop command on hotkey press:

You can replicate this behavior by running the command in a terminal - it runs with a syntax error, but causes the position of the window to be reset:
> tdrop -a -h 90%% alacritty
2024-03-31 08:44:54 Debug command: tdrop -a -h 90%% alacritty
gawk: cmd. line:1: BEGIN {printf("%.0f", 0.01*90%*1600)}
gawk: cmd. line:1:                               ^ syntax error
/nix/store/ik4mnsdny18wczp5iij57z16n52jfwjb-tdrop-0.5.0/bin/.tdrop-wrapped: line 892: /tmp/tdrop_tk_:0/auto_hidden/wid: No such file or directory

We could consider adding code to escape it automatically though.

I think that's a really good idea.

To clarify, while doing %% solves the build issue, it doesn't resolve the problem because the command receives both % somehow.

I can't really reproduce. Are you sure the error is caused by there being two % in the command. I can't really get tdrop to set the height anyway. When runnning the following for example:
  hotkeys.commands."demo-percent-in-command" = {
    name = "Demo";
    key = "Meta+O";
    command = ''konsole -e bash -c "echo 95%% && sleep 2"'';
  };

I get out a window where "95%" is displayed indicating there isn't any problem with two % being sent in. I probably will add code to automatically escape % though, I think that would be a better solution than what we have now anyway.

I tired your exact example: https://github.com/TimekillerTK/nix-test/commit/a653b189b57137867689dcd65be50170d51b1748
The issue still occurs on my end; hitting Meta+O causes a konsole window to flash on my screen for a fraction of the second, which probably means that the %% are preventing the sleep from executing.
Any idea what more I can do to troubleshoot this?

If you are on the latest release I have added escaping by default from #106. If you are on that release you can use only one % and see if that works. If not I don't really know what's going on.

Can confirm that with latest version including https://github.com/pjones/plasma-manager/pull/106, works with only one %. Thank you very much for the quick patch!
I'm still having the same problem with one (or two) %, so it's probably something else causing the problem.
I'll investigate a bit more on my end by deploying to a VM, I'll report back here if I find a solution.


CI: Conformance Ginkgo: K8sDatapathServicesTest Checks N/S loadbalancing Supports IPv4 fragments [It]

CI failure
K8sDatapathServicesTest
/home/runner/work/cilium/cilium/test/ginkgo-ext/scopes.go:461
  Checks N/S loadbalancing
  /home/runner/work/cilium/cilium/test/ginkgo-ext/scopes.go:461
    Supports IPv4 fragments [It]
    /home/runner/work/cilium/cilium/test/ginkgo-ext/scopes.go:515

    Failed to account for IPv4 fragments to <IP_ADDRESS> (in)
    Expected
        <[]int | len:2, cap:2>: [0, 5]
    To satisfy at least one of these matchers: [%!s(*matchers.EqualMatcher=&{[0 4]}) %!s(*matchers.EqualMatcher=&{[4 0]})]

    /home/runner/work/cilium/cilium/test/k8s/services.go:636

https://github.com/cilium/cilium/actions/runs/10708948982/job/29737595048

Also observed in https://github.com/cilium/cilium/pull/34618


Only set featured link descriptions for relative links

Trello: https://trello.com/c/a8aT8ExJ/1051-allow-featured-links-in-accordions-to-link-to-external-links
We want to allow content designers to feature links that are to external
website (such as NHS). This was previously blocked due to needing to
pull a description from the Publishing API which required a relative
path. This adds a guard clause that doesn't pull in description for an
absolute link
It also does some refactoring to introduce the extra test so that they
are more succinct.
Co-authored-by: Alex Newton<EMAIL_ADDRESS>Co-authored-by: Rosa Fox<EMAIL_ADDRESS>⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
This is intended to supersede: https://github.com/alphagov/collections-publisher/pull/1254

@kevindew
Looks good! As discussed on Trello and a chat this morning, we are assuming that absolute paths won’t be used for internal links by content designers. We may add validation for this at a later stage, but it will be better to do this after a larger task to address validation from happening in both the model and presenter.
I would approve, but Rubocop is failing!
Featured link (internal)

Featured link (external)

@kevindew
Looks good! As discussed on Trello and a chat this morning, we are assuming that absolute paths won’t be used for internal links by content designers. We may add validation for this at a later stage, but it will be better to do this after a larger task to address validation from happening in both the model and presenter.
I would approve, but Rubocop is failing!
Featured link (internal)

Featured link (external)

Thanks @Rosa-Fox, and oops, I've sorted the Rubocop issue.

Thanks @Rosa-Fox, and oops, I've sorted the Rubocop issue.


As a Marist Student, I want to register my vehicle so that I can bring others with me.

When on the launch page,  it will prompt you to terms and services that you will be ok to trust other people to drive and yourself to drive. Registering vehicle is not neccessary.

High priority to create term of conditions display


Fix label-sync jobs

/assign @xrstf

/ok-to-test

/test pull-infra-prow-labelsync

/override pull-infra-prow-labelsync
Will be fixed (:crossed_fingers:) once this is merged.
/approve
/lgtm


Updated README to more clearly state that linguist runs in a Git repo

Attempt to help others avoid the same mistake I made https://github.com/github/linguist/issues/4854 .

CC @lildude

Updated PR, I used your wording verbatim (since this is your repo not mine ; )


Automatic column width determination for command line tool

Determine the number of output columns for the list and help commands using the command names rather than hard coding it (conditionally).
Loosely an extension of #4144 (kind of).

The fp != functions in list_type probably wasn't doing what was intended (when the type being filtered on didn't start the list).  This has also been changed to something that seems more appropriate.

Merged, thanks.


Should refactor syncK8sStatus method in service controller

In GitLab by @wknet123 on Jul 1, 2020, 01:35

In GitLab by @luotian-github on Sep 8, 2020, 01:24
service status, pod status


Does the check for unknown positional parameters work?

Hi,
thank you for this useful gem! I have one question about the design choice of how to generate the constructor of the object.
https://github.com/dry-rb/dry-initializer/blob/3167b5aee1d093886fe02448901db5a851f74d7b/lib/dry/initializer/builders/signature.rb#L8-L10
Why is the '*' included in the parameter list? Wouldn't that just match all positional arguments regardless of whether they are defined or not?

@kfischer-okarin Hi
You are right, the gem does not check the number of params, but just ignores the unknown ones.
This was made by design, because at the moment I got no good solution for adding stars (I mean both * and ** into the DSL). Now the migration to stricter checks is possible in theory, but this decision would cause a huge backward-incompatibility, so I would prefer not to make the change.

I see... I was just asking since it seems to contradict an intuitive reading of the documentation ( https://dry-rb.org/gems/dry-initializer/options-tolerance/ ). So it would maybe be good to add a section about this behaviour....
About the strict parameters..
How about parametrizing the module (builder) and adding parameters strict checking parameters for this?
like... For the Container version this would work out of the box as in
include Dry::Initializer.define(allow_unknown_params: false) -> do
  ...
end

Though I'm not sure what would be an backcompatible way for the normal extend version. Maybe a something like.
extend Dry::Initializer.with_options(allow_unknown_params: false)

just bumped in this myself as well. Should the documentation be updated to correctly reflect the fact that positional arguments are not strict? Right now it says the following:

By default the initializer is strict for params (positional arguments), expecting them to be defined explicitly.

which doesn't seem to be case. As the current real behaviour is different from Ruby's own behaviour, I think it's worth documenting this correctly.

Does the much stricter Ruby behavior around handling args and keyword args make it simpler for us to enforce the same in the gem?


Binding to native isotope events

I'm trying to bind to the arrangeComplete event listed here - https://isotope.metafizzy.co/events.html#isotope-events
It looks like this plugin is manually emitting events for things like filter, sort, layout, etc. But there doesn't appear to be a way to bind to the native isotope events.
@David-Desmaisons is that possible?

Well, this event could be added. I will provide this enhacemet in some point in the future. And PR are welcome...

I'd like this too
or an event to reload items? 👍

found out you do have access to the isotope instance
       Vue.nextTick(() => {           this.$refs.yourref.iso.reloadItems();           this.$refs.yourref.sort('title');         });
that way you can use any of the isotope native events

@mwleinad What is your scenario where you need an explicit cal to reloadItems?

@puglyfe and @mwleinad why would you need the arrangeComplete  event?

@mwleinad You save my day dude!


First Crack at Vercel Integration

Going ahead a merging this even though it is not working in Vercel yet as some of the changes made in #6 broke stuff. :)


title

No description provided.

I'm so sorry for my mistake in testing this plugin, please delete this issue.


Obligatory mod not supported in newest BG version Patch #7

No rush... but also like this game is unplayable without this mod. 😅  Thank you so much!

Agree, without this mod, it is very difficult to control the movement of the character in certain situations, which is very inconvenient.

Really enjoyed this mod pre-patch 7. After seeing the release notes on nexusmods for 1.9.5 patch I gave this amazing mod another shot on my win10 PC.
But WASD mvmt isn't working on patch 7. After cleanly installing -- tried both manually and via Vortex. I even nuked game folder and AppData and reinstalled BG3 cleanly just for this to work but no avail.
I see that you fixed WASD for patch 7 a day after this issue was reported so I assume others are using it successfully. Any idea what's wrong?

What's the error pattern? There is a FAQ on Nexusmods. You can also post the question on Nexusmods, then the community can help.

Fixed by reinstalling via Vortex. (The issue was that the mod seemed to have but the wasd keys weren't moving the character.) All's well in the world again!
Details: I removed all mods from Vortex, updated it to latest version (1.12.6) and reinstalled Native Mod Loader and WASD. The issue also may have been due to a silent failure from a mod called "LSLib/Divine Tool" missing. Once Vortex was updated, it prompted to install it for BG3 specifically.


Opengl full

What does the pull request do?
Full GL api as from https://github.com/KhronosGroup/OpenGL-Registry/blob/master/xml/gl.xml with documentation from https://github.com/KhronosGroup/OpenGL-Refpages/tree/master/gl4.
What is the current behavior?
in GlInterface currently there are only some of gl api available without intellisense documentation neither for parameters due to the property get instead of method declaration.
What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?
Autogenerated using tool from https://github.com/devel0/netcore-sci/tree/d23959d0fba177382cd23241a6cc51a3ca3356c3/tools/gl-csharp-bindings-generator.
Checklist

[ ] Added unit tests (if possible)?
[X ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation

Breaking changes

following gl.xml spec some int var should declared as uint
most methods have enum group

var oneArr = new int[1];
gl.GenFramebuffers(1, oneArr);
_fb = oneArr[0];
gl.BindFramebuffer(GL_FRAMEBUFFER, _fb);

becomes
var oneArr = new uint[1];
gl.GenFramebuffers(1, oneArr);
_fb = oneArr[0];
gl.BindFramebuffer(FramebufferTarget.GL_FRAMEBUFFER, _fb);

Fixed issues
none

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: devel0 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Another possible better solution would to use OpenGL.Net but I found the way proposed in this PR more immediate in terms of integration with current code and I haven't all the knowledge required to consider that integration; this PR effectively only extends available api methods, document and type enum for them.
Eventually if useful I can consider to add Attributes to methods describing extension that can retrieved from gl.xml spec file.

You are changing int to uint all over the place. I don't think that is really needed and just makes it harder to review this.

I'm sorry, but we can't accept that to be included in the framework itself.

We have to support rendering through both OpenGL ES and desktop GL, so the idea is to have a minimum set of APIs that are present everywhere and absolutely required to get things going.

Our built-in OpenGL bindings are not intended to be used outside of the framework. The only APIs that should be used by end-user apps are context management (creation, MakeCurrent) and glGetProcAddress.
The rest should be done via some exiting OpenGL bindings such as Silk.NET OpenGL.Net and OpenTK. They do support working on top of pre-existing OpenGL context with custom glGetProcAddress delegate.
The general plan was to provide integration shims for those bindings, so there would be OpenTKControl, SilkNETControl, OpenGLNetControl working on top of OpenGLControlBase and shipped via separate NuGet packages.

@kekekeks Thanks for clarification, I totally agree with you.
I would be useful to specify in the roadmap what you said here about OpenTKControl, SilkNETControl, OpenGLNetControl just to allow anyone to know directions.
Consider that I wrote extension proposed in PR just for my need because I'm new to 3d and I got some trouble without having intellisense for example for a function like glPolygonMode etc, but when I did these for my need I thought that opening a PR  could helpful to clarify the question about the infrastructure you described which I eager to see in operation.
Thanks for the work you are forwarding about 3d control availability for the AvaloniaUI.

I wrote a PR #4310 that should fit with needs and its very essential,
Hopefully could fit with your requirements to avoid incompatibilities.


Move listener clients to this repo

I was trying to update clients for the new server version, and I found it a bit difficult for a beginner (find link to GitHub device page in documentation, fork the repo, update it and open a PR in a repo that can be forgotten by maintainers).
So, I did what hrueger asked in #214 and I started moving every device firmware code to this repo, in the directory listener_clients.
Doing this, I think that opening issues regarding clients, contributing to firmwares and updating them when we update TallyArbiter server code can be easier, even for a beginner.
I tried to migrate original commits, to keep authors and history (I replaced some... useless test values in some of you commits @josephdadams), using git format-patch, replacing some values and using git am for "importing" them into this repo.
Currently, there are some TODOs:

[x] Check if there are open PR in devices repos
[x] Update firmwares for v3.0.0 server
[x] Add support for MDNS discovery and add captive portal for WiFi connection
[x] Detect server version
[x] Import clients from other repos
[x] Configure CI to compile firmwares at every release

Tested clients:

[x] M5Atom
[x] TTGO_T
[x] blink1
[x] ESP32 with Adafruit NeoPixel
[x] gpo
[ ] M5Stick(c)
[x] Pimoroni Blinkt
[x] relay

I tried to update the logic.
@mg-1999 can you test the last commit?
BTW now I have a TTGO-T to test the firmware, but Windows won't let me delete an arduino library from the folder, reporting "access denied" even though I'm using powershell as administrator and if "Administrator" should have write rights to the file... so i can't compile Arduino firmwares on my setup.
I have to re-install the Arduino IDE and everything...

Last commit works as expected! Thanks for fixing it.
Now onto the next one 🚀  M5StickC?

Last commit works as expected! Thanks for fixing it.
Now onto the next one 🚀 M5StickC?

Yes! Thanks!
I'm working on TTGO, and now it's working quite well.

Tested everything except M5Stick(c)

m5Stick C has the same problem as the m5atom. The tally state is stuck after there are no tally updates for the device.

Fixed.

Done. CI is working and every listener client supports v3.0.0.
@mg-1999 can you test M5Stickc?

@MatteoGheza let's be sure to update all the readme.md files for each client as well as the official documentation on how to load the clients, if we are shipping the .hex files

https://github.com/josephdadams/TallyArbiter/blob/fb3085ca156c72eacd58eb4b9560b3477765a0b7/listener_clients/gpo-listener/gpo-listener.py#L73
Should this be port 4455?

Yes the m5stickC was indeed fixed, but don't have the Cplus to test. But don't think it will be a major problem

@josephdadams @mg-1999 what do you think about this README?
Please note that the "1. Using a pre-builded version" section is going to change ASAP.

Merging this so I can start working on #270
I'll open a new issue for the updating the documentation

Sounds great.
I just saw https://github.com/electron/electron/issues/14545 . But as a workaround until that issue is fixed, we can detect if we're in the electron window and then open a Chrome window.


Dashboard has deleted after update plugin 3.10.0

Describe the bug
One dashboard has deleted without any action and error React loading after update plugin.
Screenshots

"
t=2019-02-16T22:32:59+0700 lvl=info msg="Syncing plugin dashboards to DB" logger=plugins pluginId=alexanderzobnin-zabbix-app
t=2019-02-16T22:32:59+0700 lvl=info msg="Deleting plugin dashboard" logger=plugins pluginId=alexanderzobnin-zabbix-app dashboard=zabbix-server-dashboard
"
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

5.1.4
4.0.4
3.10.0

How did you install plugin? Try to clear browser cache as well.


[cache warm_up] warm_up slice with dashboard default_filters

CATEGORY
Choose one

[x] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Superset offers a couple of ways to warm_up cache:

cache warm_up endpoint:
https://github.com/apache/incubator-superset/blob/8764ae385206c8bdccba1e7aa42def1929f6fba7/superset/views/core.py#L1645
Celery task:
https://github.com/apache/incubator-superset/blob/000a038af193f2770dfe23a053bd1ff5e9e72bd4/superset/tasks/cache.py#L257

Superset will generate a query like form_data={slice_id: _slice_id_} so that we can warm_up query saved with a given slice_id.
Now a lot dashboard are having default filters (and filters can have scopes), warm_up single slice without dashboard context make our cache hit rate declining. Currently in airbnb we have 10% dashboard with default filters, but 20% of dashboards are landed with default_filters.
This PR is to add dashboard context into the cache warm_up call. You can pass extra dashboard_id parameter to wam_up API, indicating the slice is called with a given dashboard. It should generate a query like
form_data = {"slice_id":926, "extra_filters": [
    {
      "col": "region",
      "op": "in",
      "val": [
        "East Asia & Pacific",
        "Latin America & Caribbean"
      ]
    }
  ]}

TEST PLAN
added new unit tests
REVIEWERS
@john-bodley @serenajiang @etr2460 @dpgaspar  @mistercrunch

Codecov Report

Merging #9311 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9311   +/-   ##
=======================================
  Coverage   59.08%   59.08%
=======================================
  Files         374      374
  Lines       12202    12205    +3
  Branches     2986     2989    +3
=======================================
+ Hits         7209     7211    +2
- Misses       4814     4815    +1
  Partials      179      179

Impacted Files
Coverage Δ

...plore/components/controls/FixedOrMetricControl.jsx
51.78% <0.00%> (+0.84%)
: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 b1916a1...06b1d96. Read the comment docs.

looking good so far!
after looking through this and not knowing potential values for the filter object, i'm wondering if most of the .get("value", {}) and .get("value", []) statements should be .get("value") or {} and .get("value") or []

I follow documentation here:
https://docs.python.org/2/library/stdtypes.html#dict.get
and
https://stackoverflow.com/questions/33263623/dict-getkey-default-vs-dict-getkey-or-default
I would say most of my usage is for the dict that doesn't have key. Why do you think they should be .get("value") or {} and .get("value") or []?

The main difference here applies if it's possible for a key to be set to a falsey value. If you have the dict:
d = {
  "key": None,
  "foo": False,
}

then:
d.get("key", []) # equals None
d.get("key", []) # equals None

The main difference here applies if it's possible for a key to be set to a falsey value. If you have the dict:
d = {
  "key": None,
  "foo": False,
}

then:
d.get("key", []) # equals None
d.get("foo", {}) # equals False
d.get("key") or [] # equals []
d.get("foo") or {} # equals {}

This is really important to get right if you try to iterate through an array after "getting" it like this, as iterating over False or None will throw

Correct. If the key exists,  I should use false value, instead of assign a default value. See my example above.

What i'm saying is if you default it to [] with or then you don't even need to check if it's falsey. The empty array will simply bypass the for loop

What i'm saying is if you default it to [] with or then you don't even need to check if it's falsey. The empty array will simply bypass the for loop

i didn't check the value is falsey or not. My question is this comment:
after looking through this and not knowing potential values for the filter object, i'm wondering if most of the .get("value", {}) and .get("value", []) statements should be .get("value") or {} and .get("value") or []
Why should change most of dict.get(key, default_value) to dict.get(key) or default_value?

Here's a specific example:
You have filter_scopes = json_metadata.get("filter_scopes", {}) in the code, and then you later assume filter_scopes is a Dict. However, because the json_metadata can be edited by the user, it could be None or False. This would break with your current code, but not with json_metadata.get("filter_scopes") or {}
In general, I think we need to be a lot more defensive with parsing the json_metadata blob here, because an invalid metadata field isn't validated on save and it would break the code here


search Category section card  update

search Category wise job card section get updated. adding images and effects and button to make it more attractive . just check once if any changes is there regarding image or styling let me know . so i can improve designs and  modified according to this.

Great !! @AakankshaSarode

thank you ! @PatilHarshh


Adjust SecurityManager permissions methods to take a SecurableResource instead of a SecurityPolicy

Rationale
Passing in a SecurableResource gives us more flexibility downstream, e.g., for linked schemas. https://www.labkey.org/home/Developer/issues/Secure/issues-details.view?issueId=45251

@labkey-jeckels See my chat (to you & Matt) about this and related PRs... I'm inclined to ditch the whole lot and not pursue this direction. We can discuss if you'd like.

This was handled in https://github.com/LabKey/platform/pull/5577


Error occured when trying to run the script mnist_1.0_softmax.py

$ python3 mnist_1.0_softmax.py
Tensorflow version 0.12.1
Extracting data/train-images-idx3-ubyte.gz
Extracting data/train-labels-idx1-ubyte.gz
Extracting data/t10k-images-idx3-ubyte.gz
Extracting data/t10k-labels-idx1-ubyte.gz
Traceback (most recent call last):
File "mnist_1.0_softmax.py", line 79, in
I = tensorflowvisu.tf_format_mnist_images(X, Y, Y_)  # assembles 10x10 images by default
File "/home/raveen/tensorflow-mnist-tutorial/tensorflowvisu.py", line 42, in tf_format_mnist_images
everything_incorrect_first = tf.concat([incorrectly_recognised_indices, correctly_recognised_indices], 0) # images reordered with indeces of unrecognised images first
File "/home/raveen/.local/lib/python3.5/site-packages/tensorflow/python/ops/array_ops.py", line 1075, in concat
dtype=dtypes.int32).get_shape(
File "/home/raveen/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 669, in convert_to_tensor
ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
File "/home/raveen/.local/lib/python3.5/site-packages/tensorflow/python/framework/constant_op.py", line 176, in _constant_tensor_conversion_function
return constant(v, dtype=dtype, name=name)
File "/home/raveen/.local/lib/python3.5/site-packages/tensorflow/python/framework/constant_op.py", line 165, in constant
tensor_util.make_tensor_proto(value, dtype=dtype, shape=shape, verify_shape=verify_shape))
File "/home/raveen/.local/lib/python3.5/site-packages/tensorflow/python/framework/tensor_util.py", line 367, in make_tensor_proto
_AssertCompatible(values, dtype)
File "/home/raveen/.local/lib/python3.5/site-packages/tensorflow/python/framework/tensor_util.py", line 302, in _AssertCompatible
(dtype.name, repr(mismatch), type(mismatch).name))
TypeError: Expected int32, got list containing Tensors of type '_Message' instead.

the second line says it all: "Tensorflow version 0.12.1"
The current version of Tensorflow is 1.4


SIMD for SIA miner

Hello. Accept to another branch (simd). I don't think that it help for main branch, because increase MH/s only in CPU. Because Intel does not support AVX2 with long4, uint8 etc., increasing too few.
I'm think crash market, by 100 CPU.... Need only rewrite OpenCL!

Now I got 16 MH/S with Spir-V.


Large images stretched out of border in PDF

Hi, thanks for your great work! But it seems that large images referenced in Markdown files will be stretched out of border in rendered PDF. How to solve this issue please?

There does not seem to be a standard way to specify an image's width in Markdown so maybe you could scale down the image before referring to it.
Or after you convert the Markdown to DITA using the quick fix, you could try to add a @scale or a @width attribute on the DITA image element.

Alternatively, the quick fix that converts the Markdown fragment to a DITA image can also add the scale-fit="yes" attribute.  For example after this line
https://github.com/oxygenxml/ditaMark/blob/553c06ce0be5e0ad8c47f712373ed83826d33885/frameworks/ditaMark/rules/md2dita.sch#L53
add
<xsl:attribute name="scale-fit" select="'yes'"/>

Regards,
George


Using Marker bindTo method

I'd like to bind a Circle to a Marker so it follows the marker around.
How can I do this in react-google-maps?
See https://stackoverflow.com/questions/825794/draw-radius-around-a-point-in-google-map

@krabbypattified hey, we currently don't have an API for this. Would you propose an API for this?

How about  ?
Seems pretty intuitive to me :)

Unless it's not for position. Then again position seems to be the primary use case.

Alternatively:
<Marker ref = "myMarker"/> <Circle postion={this.refs.myMarker}>

Currently working on a proof of concept for this API:
<Marker position={{lat:10,lng:10}}>
    <Circle center={bindTo("position")} />
</Marker>

It works! Here's the gist:
https://gist.github.com/krabbypattified/63ce708b414cfc5ea7aced3833b5c33e

@krabbypattified Interesting!
In react-google-maps/{Marker,Circle}, since we can render InfoWindow as a child of Marker, we can safely render Circle as a child of Marker as well (Marker exposed ANCHOR from its context). Then in the Circle, if we find an ANCHOR from context, we just bind to it's location?

I think that would only work for marker's initial position, if at all, and it wouldn't make use of bindTo.
If you want to properly bind a circle/infowindow to a marker (i.e. it follows the marker if you drag it), then bindTo must have a google.maps.MVCObject (i.e. google.maps.Marker) as its second parameter.
So:
circle.bindTo('center', marker, 'position')
or more generally, MVCObject1.bindTo('propOnMVCObject1', MVCObject2, 'propOnMVCObject2')
...assuming I understood your question.

@krabbypattified check the InfoWindow source code, in there we can access the instance of google.maps.Marker from the context (since we put it as a child of Marker). Applying the same trick for Circle, we can call circle.bindTo('center', marker, 'position') in any lifecycle method that's applicable.


CDN's is not working (404 error)

The following CDN's give 404 errors:

http://cdnjs.cloudflare.com/ajax/libs/Sortable/1.6.1/Sortable.min.js
http://cdn.jsdelivr.net/sortable/1.6.1/Sortable.min.js

The only CDN that seems to work is:

http://cdn.jsdelivr.net/sortable/latest/Sortable.min.js

jsDelivr switched to a fully automated system, that can serve files from npm and GitHub. This means all versions published later than in May are available automatically, but use a new link structure.
The correct link for new versions is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>for latest. You can always find the correct links at https://www.jsdelivr.com/package/npm/sortablejs

Can we please update the README file then

Also what about CloudFlare?


New packages: PRASCore, PRASFiles, PRASCapacityCredits, PRAS v0.7.0

This is a manual pull request, prepared with LocalRegistry.jl, to simultaneously register v0.7.0 of four packages from NREL's Probabilistic Resource Adequacy Suite (PRAS). The four packages are PRASCore.jl, PRASFiles.jl, PRASCapacityCredits.jl, and PRAS.jl, a convenience metapackage that loads all the others. All four packages reside in https://github.com/NREL/PRAS
For convenience I'll try to proactively talk through some of the automerge checks:

About the package name - yes, we realize this probably isn't going to be looked on too favorably, particularly PRAS.jl which fails multiple automerge rules. PRAS as a brand exists outside of the Julia ecosystem (e.g. https://www.nrel.gov/analysis/pras.html) and we're hoping we can get an exception on this. We received feedback from @goerz and @gdalle in https://github.com/JuliaRegistries/General/pull/120172 that this may be possible, and are hoping folks here agree?

About the repository name - while today PRAS is exclusively a Julia project, we're actually in the middle of a rewrite and our monorepo will be hosting Rust, Julia, and Python code in the near future. So we don't think having a repo name that ends in .jl would be appropriate.

About the version number - while 0.7.0 is the first version of PRAS submitted to General, it is not the first version of PRAS. v0.6 and earlier were registered in NREL's public registry (https://github.com/NREL/JuliaRegistry).

All four packages are MIT licensed and a copy of the license is included in each of the respective subdirectories of the monorepo.

Happy to answer any further questions or take feedback if there are things here we should be doing differently!

Given the maturity of the project, I don’t see any problem with the name PRAS and would be happy to see this merged.
You will probably have to go on Slack and post a message in the #pgk-registration channel to find someone with merge permissions.

Per guidance on Slack, we're submitting these individually via Registrator with the understanding that this PR will count as the three days waiting period.


Pasting text with 10k lines is slow

I'm testing the scalability of Draft.js and one of the scenarios I'm exploring is pasting a large amount of text. A plausible use-case would be if someone's pasting the content of a large source code file.
I was testing with https://facebook.github.io/draft-js/ and generated text content using:
printf '.\n%.0s' {1..10000} | pbcopy

This copies 10,000 lines (each with a single .) to the clipboard. Pasting this into Draft.js (the demo at https://facebook.github.io/draft-js/ will do) demonstrates the performance issue.

It looks like the time is tied up in React. What approach can we take to fix this?
CPU profile:
CPU-20160603T133957.cpuprofile.zip

One thing worth nothing is that the Draft home page is using the development version of React. If you're going to analyze performance make sure you use the production minifed version. Not to say there won't be some issues with large amounts of text but that's a first step.

It would also be useful to see comparisons with other web editors to know whether this is something that is unique to Draft, and to know whether edit operations are sluggish.

At the moment I've shortlisted ProseMirror and Draft.js as candidates to build an editor on, so I only compared performance of those two. Here's the testing environment:

iOS (iPhone 6S) Safari with profiling enabled (via OSX Safari) — so these times are worse than a user would experience, but are indicative of computational complexity
Draft.js was the example on https://facebook.github.io/draft-js/ (which is
ProseMirror was the example on http://prosemirror.net/#demos
Times are in seconds, the average of a single sample of size two, and measuring script execution only

Scenario
Draft.js
ProseMirror

Pasting 1,000 lines, 100 char per line
8.77s
1.97s

Pasting 1,000 lines, 1 char per line
4.02s
0.15s

Pasting 100 lines, 1000 chars per line
5.32s
2.37s

Pasting 100 lines, 100 char per line
1.10s
0.10s

Pasting 100 lines, 1 char per line
0.68s
0.07s

Press return 5 times after 1,000 lines, 1 char per line
6.94s
0.15s

Press return 5 times in the middle of 1,000 lines, 1 char per line
5.62s
0.27s

Press return 5 times before 1,000 lines, 1 char per line
4.04s
0.45s

Initialise 100 editors, each with 100 char
2.74s
0.31s

Initialise 10 editors, each with 100 char
0.34s
0.08s

You can see ProseMirror is often an order of magnitude faster.

Thanks for the details!
As @zpao notes, it would be appropriate to test this with the production version of React rather than the development version. (That's not to say PM isn't faster than Draft -- it may well be! But the comparison should be with production React.)

Just to chime in I'm also experiencing this issue in automattic/simplenote-electron#501
The editor performance is fine when updating the content once it's already rendered but the load time is slow. Some basic investigation revealed that most of the time when initially rendering the editor was spent in Firefox in the garbage collector.
We're using ContentState.createFromText( someString, '\n' ) but I've timed that and it doesn't appear to take much time. As much as I can figure out tonight, most of the time is spent allocating and freeing objects used in building draft blocks.
@hellendag if there's anything I can do to help out I would be glad to offer my assistance. There are some pictures of dev tools snapshots in the linked issue with Simplenote. Just to repeat as well, this doesn't not appear to be problematic when editing contents already in the editor, just when loading the content into the editor at once.
Even corresponding to the chart above I think this is more proportional to the number of actual lines and not to the number of characters in the editor. For example, and maybe @bradleyayers can help clarify, but I'm guessing that 100/1000 ends up wrapping around to make far more than 100 lines; 100/100 wraps to expand out to a reasonable size, maybe 200 actual lines, and 100/1. Those counts would more closely correlate rendered lines with render time. I did try to test this theory out by timing the render on a wide window and also on a narrow window but did not find a noticeable difference in delay though.

I've experienced slowness with as few as 3,000+ words even in plaintext mode. (See my application here: https://thoth-reader.herokuapp.com).

We're seeing a sudden and quite severe slowdown in the editor when working with about 1500 lines of text. On a lower end machine we are dealign with a ~6 second delay once typing stops, before visual feedback that typing has occurred

@davidawad, I was curious so I ran a quick profile into your app. I pasted a big chunk of text and pressed enter a few times:

generateTextScores and other functions you run take a long time. Didn't dig much deeper but it seems to be your app's code. I'd suggest looking into debouncing and optimizing it.


[3.x] Make WebXR touch events proportional to render target size, not viewport

This is a fix to #56819 which is only in 3.x.
The position of touch events was previously proportional to the viewport. This is problematic when the render target from WebXR is a different size than the viewport - the touches will be in the wrong place.
(Note: This isn't the ideal fix - it would be better if the viewport resized to match the render target, but for some reason that isn't "sticking". I suspect this may require some deeper changes in the HTML5 platform. But that's for a future PR!)

Interesting that this should be a problem because there shouldn't be a reason for handheld AR devices to change the render target size away from the viewport. That said, seeing when a viewport is marked as the XR viewport the render target size overrides the viewport size and the viewport size becomes defunct, this solution seems sound.

Thanks!

That said, seeing when a viewport is marked as the XR viewport the render target size overrides the viewport size and the viewport size becomes defunct

Yeah, the thing is the viewport size isn't "sticking" for some reason, it keeps resetting to the size of the browser window. I think this an HTML5 platform thing, I just haven't had a chance to dig that deep yet. Something for my next PR! :-)


Fix invalid ordering of mouse effect enum

The SDK has changed the ordering of the mouse effect enum members yet again and they'll need to be fixed (SDK ver 1.1.2).
References #125.

A reference note here that the SDK version is 1.1.5 yet the documentation is noted 1.1.2 as it didn't change in those 3 patch versions.  This doesn't affect this issue, but is useful to know.

Was this fixed or was it found out that it wasn't needed?

The enum in the 1.1.5 docs match what we currently have in Colore, so I'll close this. It must've been fixed at one point and we forgot to update this issue.


Volunteer introductions

Hello! Thank you so much for expressing your interest in volunteering for hackseq. It's gonna be busy and bustling.
Things we'll mostly likely need assistance with:

Setting up, for example, carrying food and moving furniture
Assist people with technical support. There will be some people who are unfamiliar with Git, command line, and other tools.
Administration and logistics, for example, signing in, making sure things are communicated, people know where to go, answering questions.
Helping out with specific projects, especially when people unfamiliar with the technology need to ramp up.

This list is not conclusive. We'll be running into many issues and tasks as the hackseq event unfolds. Since we'll all be working closely together to make this event successful, please introduce a bit about yourself, tell us what you'll like to help out with, and other technical knowledge that you want to share with the participants, and include some questions if you have any.
We already got your availabilities if you responded to the volunteer form, but feel free to include it if your schedule has changed.
Thank you and we look forward to meeting you!

To start, I'm Santina, a bioinformatics master student. I do all my work on the computer, so I always look for opportunities to look away from screens!  I enjoy hiking, eating, playing games and learning new things.  Ask me questions about things to do in Vancouver! ☀
I can help out with general logistics and ramp up any people who need a refresher or a crash course on things like command line, Git, Python, R, etc.

I'm Erin, a research associate in the Microbiology and Immunology department at UBC. I analyse RNA-Seq data and clinical metadata to identify potential drug targets and study factors that are indicative of disease severity.
I can assist with any and all of the items listed above. Specifically, my computational expertise lies in the use of command line, R and python.

Hi, I'm Dan. I work at Fusion Genomics, where I work on infectious disease assay development. I split my time between the wet lab and computation. I'm currently developing a web app to track the production of our sequencing libraries.
I'll be participating in HackSeq on Team 9. Outside of my team responsibilities, I'll be happy to help out with setup and admin stuff. I've volunteered at Software Carpentry Bootcamps on git, python and R, and would be happy to help out with quick beginner-level troubleshooting.

Hey there. My name is Matthew Emery and I'm a graduate student in Data Science, former Python software developer with a background in biochemistry.
I can help with set up and tech support.

Hi everyone, I'm Minh. I'm an undergraduate student major in Management Information System. I look forward to working with all of you!

Hi guys. My name is Kai He and I'm currently a Master of Data Science student. My undergraduate study was Biochemistry with a Minor in Math.
I can help with setting up, tech support, and signing in.

Hi guys. I am XiaoFei. Please don't be upset if you will mispronounce my name :D I am working at Fusion Genomics as a computational biologist. For my work, I am developing computational tools for the typing of infectious diseases, assisting in the development of web portal for the management of data and projects, and analyzing NGS data.
I can help with tech support, set up of coding environment, any computer-related stuff, etc. I especially have a lot of coding experience, so feel free to ask me any question related to coding.

I'm Ka-Kyung, a postdoctoral fellow in Translational Genomics Informatics Lab at Severance Biomedical Science Institute. I'm doing translational disease-mutation analysis, visualization, and integration for multiple investigators and research projects with NGS data. I'm currently focusing an integrative analysis of whole genome and whole transcriptome sequencing in inherited brain disease.
I can assist with any of the items listed above. Especially, I hope my experience on integrative WGS and RNA-seq study would be helpful in project#4 (Pseudo-WGS variant calling for common cell types aggregating ChIP-seq, RNA-seq and DHS from ENCODE and Roadmap Epigenomics data).
And, I have a question. Can I book accommodation of Suite UBC or Triumf which is posted at 'http://www.hackseq.com/participant-resources' (or I'm looking for room mate during hackseq 2016).

Hello everyone!
My name is Laura. I am currently a Co Op student working at BCCRC IT. I have taken workshops on Git, Command line and Python thanks to Software Carpentry and have experience setting up and providing people with tech support. I would be happy to provide both skills during this weekend :).
-Laura

@irobii : You can stay wherever you want. The recommendations we gave on the website are places that are on UBC campus, so you can travel easily between where you're staying and hackseq's location. You can definitely arrange to room with someone as well :).  I'll be adding you to the hackseq channel so you can ask if anyone is still looking for accommodation!

Hi! I'm Daisie Huang and I'm a fairly frequent instructor for Software Carpentry. I usually teach Git and sometimes Python. I used to be a postdoc at UBC, working on phylogenomics in Populus. Currently I am a freelance scientific software developer and I primarily work for Dryad Digital Repository.
I would be happy to help with anything that needs doing!

Awesome! I've invited you guys to join the hackseq slack channel. Please also download the slack app on your phone (it'll probably prompt you if you open the invite link in the email on your phone). We'll be using that to communicate during the event. I'll be assigning the time slots in a bit.

Hi
I am Rad, senior bioinformatician at bc cancer agency / UBC
Although I would like to help with all programming issues, which I would do with great pleasure, I think I will bring my camera and take pictures of this event which will take me a bit away from computers for the weekend :) if you need a photographer, am in :)

Yes!! We do need a photographer. Thanks for offering that :D

Hi everyone, as discussed in #55 , we are having the last planning meeting tomorrow at 5pm at UBC, at the site of hackseq to go over the last todo logistics items as well as going through some run-throughs. The address is:
2350 Health Sciences Mall, Vancouver, BC V6T
If you're in Vancouver, please come join us ! :)


Different result for @ and xr.dot with multiple matrices

What happened?
I compute both products
da0 @ da1 @ da2 @ da3,
xr.dot(da0, da1, da2, da3, optimize=True)

which should have the same result. This is not the case, and I don't know why.
I was unable to create a MWE reproducing the issue with randomly generate matrices, so I attached the matrices that I used.
What did you expect to happen?
No response
Minimal Complete Verifiable Example
Zip file containing the matrices:
product.zip
import xarray as xr

ds = xr.open_dataset('product.h5')

da0 = ds['da0']
da1 = ds['da1']
da2 = ds['da2']
da3 = ds['da3']

# 2 factors: no error
xr.testing.assert_allclose(
    da0 @ da1,
    xr.dot(da0, da1, optimize=True)
)

# 3 factors: no error
xr.testing.assert_allclose(
    da0 @ da1 @ da2,
    xr.dot(da0, da1, da2, optimize=True)
)

# 4 factors: error
xr.testing.assert_allclose(
    da0 @ da1 @ da2 @ da3,
    xr.dot(da0, da1, da2, da3, optimize=True)
)

MVCE confirmation

[x] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[x] Complete example — the example is self-contained, including all data and the text of any traceback.
[x] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[x] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
No response
Anything else we need to know?
 There will be floating-point differences that result from the order of execution. Again, we just dispatch to np.einsum so we can't do much here.


Error on run:sh wp-init.sh

$ sh wp-init.sh
[+] Running 3/3
 - Network zld-headless_wordpress-network  Created                                                                                0.7s
 - Container zld-headless-mysql-1          Created                                                                                0.2s
 - Container zld-headless-wordpress-1      Created                                                                                0.2s
[+] Running 2/2
 - Container zld-headless-mysql-1      Started                                                                                1.4s
 - Container zld-headless-wordpress-1  Started                                                                                1.7s
: not foundbin/install-wp: line 2:
Error: This does not seem to be a WordPress installation.
Pass --path=`path/to/wordpress` or run `wp core download`.
/usr/local/bin/install-wp: line 5: --path=/var/www/html: not found
/usr/local/bin/install-wp: line 6: --url=http://localhost:8000: not found
/usr/local/bin/install-wp: line 7: --title=zenleafdispensaries: not found
/usr/local/bin/install-wp: line 8: --admin_user=admin: not found
/usr/local/bin/install-wp: line 9: --admin_password=F)'L%Sk@rV3-: not found
: not foundbin/install-wp: line 10<EMAIL_ADDRESS>: not foundbin/install-wp: line 11:
Error: This does not seem to be a WordPress installation.
Pass --path=`path/to/wordpress` or run `wp core download`.
/usr/local/bin/install-wp: line 14: permalink_structure: not found
/usr/local/bin/install-wp: line 15: --skip-themes: not found
: not foundbin/install-wp: line 16: --skip-plugins
: not foundbin/install-wp: line 17:
Error: This does not seem to be a WordPress installation.
Pass --path=`path/to/wordpress` or run `wp core download`.
Error: This does not seem to be a WordPress installation.
Pass --path=`path/to/wordpress` or run `wp core download`.
: not foundbin/install-wp: line 21:
Error: This does not seem to be a WordPress installation.
Pass --path=`path/to/wordpress` or run `wp core download`.
Error: This does not seem to be a WordPress installation.
Pass --path=`path/to/wordpress` or run `wp core download`.
: not foundbin/install-wp: line 25:

REPORT

: not foundbin/install-wp: line 27:
== User List ==
' is in a plugin or theme, pass --path=`path/to/wordpress`.ser list
' is not a registered subcommand of 'user'. See 'wp help user' for available subcommands.
Did you mean 'list'?

: not foundbin/install-wp: line 32:
== Theme List ==
' is in a plugin or theme, pass --path=`path/to/wordpress`.heme list
' is not a registered subcommand of 'theme'. See 'wp help theme' for available subcommands.
Did you mean 'list'?

: not foundbin/install-wp: line 37:
== Plugin List ==
' is in a plugin or theme, pass --path=`path/to/wordpress`.lugin list
' is not a registered subcommand of 'plugin'. See 'wp help plugin' for available subcommands.
Did you mean 'list'?

I'm trying to use this project. I get this error msg when run sh wp-init.sh this in bash.

Hi @feroz39,
I tried to reproduce the issue, unfortunately without success. I never face this issue before now.
I suppose you changed something in the installation files beyond the .env one. Seams a not standard installation.
Could you please list all steps to reproduce the issue?
Thanks

@cosimoscarcella I also have this error, can you help me?


updating manuscript/code/hello-world example(s) with recent c* code

no more warnings
char* instead of uint64_t*

@OxiAction Your input is appreciated but your PR still won't make a difference here, and the warning from selfie is still there. The sole purpose of supporting the char keyword is to make gcc/clang stay silent during bootstrapping selfie. However, the hello world programs do not require bootstrapping.


Refactor typing_utils.narrow

This pr introduces some small changes in typing_utils.narrow:

Write a complete documentation.
Use __name__ in the error string rather than the str representation.
Use typing.cast for complete type propagation.
Change the argument order to narrow(type, obj) to keep consistency with typing.cast

Some tricky error occurred around cast, investigating the cause.

cast does not work with TypeVar.
https://github.com/python/mypy/issues/6440
As the type inference can be performed correctly without cast (because of the isinstance narrowing), simply avoiding cast may be the right choice for now.


Test doesn't pass because Strapi timeout

Hi,
After pulling your repo, I try to run test and got these errors.
[2020-06-03T10:04:03.051Z] warn The bootstrap function is taking unusually long to execute (3500 miliseconds).
[2020-06-03T10:04:03.055Z] warn Make sure you call it?
[2020-06-03T10:04:05.791Z] debug POST /auth/local (1316 ms) 200
FAIL  tests/app.test.js (11.972 s)
✕ strapi is defined (6 ms)
✕ should return hello world (5 ms)
✕ should login user and return jwt token (4221 ms)
✕ should return users data for authenticated user (220 ms)
● strapi is defined
Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.Error: Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.

  at mapper (node_modules/jest-jasmine2/build/queueRunner.js:29:45)

● should return hello world
Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.Error: Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.

  at mapper (node_modules/jest-jasmine2/build/queueRunner.js:29:45)

● should return hello world
TypeError: Cannot read property 'address' of undefined

  3 | it("should return hello world", async (done) => {
  4 |   await request(strapi.server) // app server is and instance of Class: http.Server
> 5 |     .get("/hello")
    |      ^
  6 |     .expect(200) // Expect response http code 200
  7 |     .then((data) => {
  8 |       expect(data.text).toBe("Hello World!"); // expect the response text

  at Test.serverAddress (node_modules/supertest/lib/test.js:55:18)
  at new Test (node_modules/supertest/lib/test.js:36:12)
  at Object.get (node_modules/supertest/index.js:25:14)
  at Object.<anonymous> (tests/hello/index.js:5:6)

● should login user and return jwt token
Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.Error: Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.

  at mapper (node_modules/jest-jasmine2/build/queueRunner.js:29:45)

● should return users data for authenticated user
Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.Error: Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.

  at mapper (node_modules/jest-jasmine2/build/queueRunner.js:29:45)

Test Suites: 1 failed, 1 total
Tests:       4 failed, 4 total
Snapshots:   0 total
Time:        12.078 s, estimated 14 s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Can you answer if it's my fail or a problem with your repo. Thank you !

Had just run test on my machine MacOs, node v12.16.1
git clone https://github.com/qunabu/strapi-unit-test-example.git
Cloning into 'strapi-unit-test-example'...
remote: Enumerating objects: 97, done.
remote: Counting objects: 100% (97/97), done.
remote: Compressing objects: 100% (54/54), done.
remote: Total 97 (delta 20), reused 96 (delta 19), pack-reused 0
Unpacking objects: 100% (97/97), done.
➜  localhost cd strapi-unit-test-example
➜  strapi-unit-test-example git:(master) yarn
yarn install v1.13.0
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
[3/5] 🚚  Fetching packages...
[4/5] 🔗  Linking dependencies...
warning "strapi-admin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>- 3".
warning "strapi-admin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "strapi-admin > @buffetjs/core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "strapi-plugin-users-permissions ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[5/5] 🔨  Building fresh packages...
✨  Done in 23.44s.
➜  strapi-unit-test-example git:(master) yarn test
yarn run v1.13.0
$ jest --forceExit --detectOpenHandles
[2020-06-03T10:12:08.745Z] debug GET /hello (10 ms) 200
[2020-06-03T10:12:09.798Z] debug POST /auth/local (902 ms) 200
 PASS  tests/app.test.js (7.692 s)
  ✓ strapi is defined (2 ms)
  ✓ should return hello world (53 ms)
  ✓ should login user and return jwt token (1050 ms)
  ✓ should return users data for authenticated user (157 ms)

Test Suites: 1 passed, 1 total
Tests:       4 passed, 4 total
Snapshots:   0 total
Time:        7.764 s
Ran all test suites.

Called yarn test many times, without getting this error
Is there any other environment settings i can test this against ?

same timeout error for me. what are all the changes required when using npm with mysql database

maybe isnt working because its running on windows?

Can you provide an environment I can replicate this error?

Windows 10 64 bit
Node: 13.8.0
await Strapi().load(); strapi can't be loaded using await keyword
so
it("strapi is defined", async () => {
expect(strapi).toBeDefined();
done();
});
above test case failing. when i am running  *Strapi().load(); *instead
of await
Strapi().load(); the test case got passed.
but that is not the actual issue.
Could you please help me
On Wed, Jul 15, 2020 at 9:53 PM Mateusz Qunabu<EMAIL_ADDRESS>wrote:

Can you provide an environment I can replicate this error?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qunabu/strapi-unit-test-example/issues/1#issuecomment-658862300,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEY44XMM266W33ZJGH4L23DR3XJW3ANCNFSM4NRQUOSQ
.

Windows 10 64 bit
Node: 13.8.0

it("strapi is defined", async () => {
expect(strapi).toBeDefined();
done();
});

above test case failing.

When i am running  Strapi().load(); instead of await Strapi().load(); the test case got passed.

but that is not the actual issue.

@mithuns14  same here, it seems isnt acessing the db

@qunabu: Could you please look into this, and suggest a solution

Can you try with latest pulls and let me know if anything has changed.
Also I've just add a CircleCI for reports on those test.

I will check and let you know. I am using MySQL as a database, so what's the main thing I need to change

It does work in CI/CD environment

how the test *"filename": ".tmp/test.db" file can be configured for mysql
*database. Is it using real database ?
On Thu, Jul 16, 2020 at 1:32 PM Mateusz Qunabu<EMAIL_ADDRESS>wrote:

It does work in CI/CD environment
https://circleci.com/gh/qunabu/strapi-unit-test-example
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/qunabu/strapi-unit-test-example/issues/1#issuecomment-659232164,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEY44XPFAIPBRO2A2JGEGALR32X25ANCNFSM4NRQUOSQ
.

This example is as simple as possible - so it is just for sqlite.
In order to have tests for mysql, postgres or mongo you would need to have a separate database just for tests. Each test would require to clear it from any data.
Therefore @mithuns14 - this question is unrelated, please create another issue or follow the tutorial - everything you asking is there, yet I'm not planning to put mysql example in this repo.

@Kazdan1994 @rubenzup can you check again with updated code.

Thank you so much for the attention! We are also using docker-compose with env files, to a PostgreSQL database in Amazon RDS, and thats the problem too, we cant figure out if the problem is in docker-compose file, or any configuration

its funny cause when i change a test for something like
it('should return Header', async (done) => {
  await request('http://localhost:1337') // app server is an instance of Class: http.Server
    .get('/header')
    .expect(200) // Expect response http code 200
    .then((data) => {
      const json = JSON.parse(data.text);
      console.log('json', json)
      expect(json.link[0].Title).toBe('Header'); // expect the response text
    });
  done();
});

getting from localhost, instead of strapi.server it works.

@Kazdan1994 can you create MR with fixed code ?

ok @Kazdan1994  but, is im using a db like Postrgres, and i have a test.db at tmp folder, does, Jest/Strapi understands it have to get this file instead of my PostgresSQL?

Hi. Sorry I close this issue since I fork the repository and add some fix. You can download fixed version at git@github.com:Kazdan1994/strapi-unit-test-example.git.
Your problem is due to jest, who timeout at 5000ms. So strapi isn't load and the instance is undefined. And your tests failed because you call method like services or query on undefined.
So the solution is just to increase the jest timeout.


Light Map Blocks are not editable

the Light Map block's light level is not able to be edited, while the delay is, meaning it is practically useless

they work without any problems for me-
the light level is the level attribute and it'll only actually emit light when the flag bound to it is active

for me, it didnt change the level number, it just stayed at 0, I do have an older world, so that could be causing it

https://medal.tv/games/minecraft/clips/iq6jbP710BzVBMYic/d1337D2bTs7Z?invite=cr-MSxOcE4sMjA1MTc4Mjg0LA
A clip i made of it not working, you can see that delay can be changed, but level cannot for some reason

that's odd.. it doesn't even give command feedback
I'll try to look into it


Updating a record in FixIT+

Details
An older, untrimmed version of a file currently appears on FixIT+ (cpb-aacip-45c3f8a195f), would it be possible to put up the updated, trimmed version currently on the AAPB into the FixIT+?
Submitted by: Miranda Villesvik
CC in communications:
Priority: Medium (within this month)
URL: https://fixitplus.americanarchive.org/transcripts/cpb-aacip-45c3f8a195f
Slack message thread:

Switched out the longer form version with current trimmed version


work on allow removing canvas

breaking change
should be supported in codux,
allows toggling canvas on and off

done by peter