Social icon bottom page cropped badly on landing page

Description
Clearly and concisely explain the problem.
You can mention unclear or missing information, typos, grammatical mistakes, etc.
Any contributions and suggestions are welcome!
The orange icon is cropped on the top part on small smartphone viewport. I'm looking at it from a MacBook Pro.
Website Section URL
Website live preview, paste the page URL or section name (eg. Docs, if it happens on all docs page) below.

Closed into #108


Feat/battle

Objective and Motivation

Pokemons from Trainer1 will battle against Pokemons from Trainer2.

run_turns() definitely can be improved 😼

I noticed now that there is a problem. The pokemons are not in the correct order. I'll try to fix it tomorrow morning.


Unexpected character error does not show correct column

Here's a reproduction example:
import { createToken, Lexer } from 'chevrotain'

const WHITESPACE = createToken({ name: 'WHITESPACE', pattern: /\s+/, group: Lexer.SKIPPED })
const NUMBER = createToken({ name: 'NUMBER', pattern: /(0|[1-9]\d*)(\.\d+)?/ })

const lexer = new Lexer([WHITESPACE, NUMBER], { ensureOptimizations: true })

const { tokens, errors } = lexer.tokenize('-"5"')

console.log('tokens', tokens)
console.log('errors', errors)

which logs

As the lexer can only understand whitespace and number literal, only 5 is captured as tokens here. There are 2 'unexpected character' errors:

The - with offset 0 and column 1 (correct)
The (second) " with offset 3 but column 2 (incorrect as 2 is the column of the first ") instead of 4

Thanks for reporting this @baodart
There is another issue here, the valid token 5 column is reported as 1 instead of 3
I suspect that the logic to keep track of column numbers in case of lexer error recovery.

https://github.com/Chevrotain/chevrotain/blob/master/packages/chevrotain/src/scan/lexer_public.ts#L693-L694

fix is released in 11.0.3 version available on npm


arachidonate 12(R)-lipoxygenase activity

Hi,
PMID=10100631 and 11256953 describes arachidonate 12(R)-lipoxygenase activity.
Could you please create a go term for this function.
Many thanks.
Guilaine

Hi @Argoudpuy
Can you please write out the reaction is this is not correct?
Arachidonate + O(2) <=> (5Z,8Z,10E,14Z)-(12S)-12-hydroperoxyicosa-5,8,10,14-tetraenoate
(EC <IP_ADDRESS>)
https://en.wikipedia.org/wiki/ALOX12
RHEA:10428 (5Z,8Z,11Z,14Z)-eicosatetraenoate + O2 = (12S)-hydroperoxy-(5Z,8Z,10E,14Z)-eicosatetraenoate Cross reference: EC <IP_ADDRESS>
suggested is_A parent?

I see we already have
GO:0004052 arachidonate 12-lipoxygenase activity
talysis of the reaction: arachidonate + O(2) = (5Z,8Z,10E,12S,14Z)-12-hydroperoxyicosa-5,8,10,14-tetraenoate.
XREF: EC:<IP_ADDRESS>
RHEA:10428;
Both RHEA and EC specify the product as the 12(S), and the reaction defined in GO mentions 12S although the GO term name does  not.
So basically the only difference here is that the product is 12(R) correct?
I'll need to request a RHEA fo your proposed term so I need to be sure about the product name
arachidonate + O(2)=(5Z,8Z,10E,12R,14Z)-12-hydroperoxyicosa-5,8,10,14-tetraenoate

Term created; rhea pending
+id: GO:0106237
+name: arachidonate 12(R)-lipoxygenase activity

Also renamed GO:0004052 to reflect the RHEA and EC assigned
arachidonate 12-lipoxygenase activity - > arachidonate 12(S)-lipoxygenase activity

I see this is already done so closing

Rhea has not yet made id for
+arachidonate 12(R)-lipoxygenase

should have come through; checking

rhea added removing tag


Attempt to delete saved query seems to multiply it instead

I tried deleting my recent query for AbART osoba ID but it instead seems to multiply in the database after each attempt ...
Can you please delete all of them and I will create one fresh?

Done

Thank you. Is it now save to remove my saved queries if the need arises?


[backport] Require SciPy 1.2 for sparse comparison

Backport of #4033

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 238ea4282376fa943163c04bcbcb95539c11e1f8, target branch v8) failed with status FAILURE.


Virtual Learning of America

@aguisoft Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@aguisoft I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


nix derivation for the sgx sdk

We'd like to have a nix derivation for the sgx sdk that is good enough to submit as a pull request on the nixpkgs repository.
See https://nixos.org/manual/nixpkgs/stable/#chap-quick-start to get started.
Some things to figure out:

[x] How to handle the various build options (e.g. with or without mitigation, with ipp crypto or sgxssl, etc)
[x] How to handle the ipp-crypto dependency

At first glance it looks like the ipp-crypto lib should have its own derivation, but the current way to build it for the sgx sdk is coupled to the sgx sdk codebase.

Draft PR is up at https://github.com/NixOS/nixpkgs/pull/126990. Will close this.


Update GLFW to 3.4

As the title suggests, this PR updates GLFW to version 3.4.
It seems to work on Linux with both X11 and Wayland (tested on Weston), but we cannot use GLFW's new runtime platform detection at the moment when building raylib with the Makefile instead of CMake, as explained in https://github.com/raysan5/raylib/pull/3827#issuecomment-1963041768. Maybe we can merge the PR as is and check this issue at a different moment.
As for testing on other platforms, I would like to ask for help from other users, although all checks from GitHub Actions are successful as of 6a58cbb.
Update: ~It looks like gamepads have stopped working~ (actually, only one of the gamepads I tested has stopped working, which is probably a rare case and we don't need to care).

Thanks for starting this update! It was in my TODO list! But note it requires testing on Windows, Linux and macOS.

Might need to revert https://github.com/raysan5/raylib/pull/2883 since https://github.com/glfw/glfw/commit/2c3eb75748584c1c9283787e8c40f944fa055d6f removed the dependency on wayland-protocols.

@planetis-m Reverting #2883 is not necessary, as 260a039 contains the necessary changes.

The new GLFW version supports platform detection at runtime, which enables a single executable to run on both X11 and Wayland. However, it looks like raylib cannot offer the same platform detection functionality at the moment because some of GLFW's local (static) functions in different platforms have the same name. As a results, we cannot combine source files from multiple platforms into one, as done in rglfw.c, unless GLFW's developers agree to change the names of these functions.

@M374LX I just updated GLFW to 3.4 in raylib Go bindings. I did rename some functions in https://github.com/gen2brain/raylib-go/commit/8596a5db729eea07b31804d0421fd5536cbefcd3 though. I know this will affect a few more projects, like https://github.com/go-gl/glfw/issues/393, so I hope it will be solved, but for now, it is not a big deal, at least for me. There are also changes for Wayland generated headers, they are renamed, and there are some additions, I use a script for that https://github.com/gen2brain/raylib-go/commit/5abcc8b79ce4184ebac57f111dfbf40391fa7d64.
Now the single binary runs on both X11 and Wayland, you can check the output:
WARNING: GLFW: Error: 65548 Description: Wayland: The platform does not provide the window position
WARNING: GLFW: Error: 65548 Description: Wayland: The platform does not provide the window position
WARNING: GLFW: Error: 65548 Description: Wayland: The platform does not support setting the window position

It would be nice to check glfwGetPlatform and add that to i.e. INFO: Platform backend: DESKTOP (GLFW).

This is a bit change, it needs some careful review... but it will be merged.

05ba358 implements @gen2brain's suggestion to output the platform selected by GLFW to TRACELOG.

@M374LX @gen2brain  Changes look good to me for now, please, let me know when you think this PR is ready for merge.
Some point to consider:

Are gamepad mappings updated? It has been manually updated in the past.
Is this PR tested on all GLFW supported platforms? It should be tested on: Windows, Linux, macOS and Web.

The gamepad mappings are supposed to be the ones provided by GLFW.
I am able to test this PR on Linux and can confirm that it works on both X11 and Wayland (despite https://github.com/raysan5/raylib/pull/3827#issuecomment-1963041768), but I prefer to leave the tests on Windows, macOS and Web for someone with access to these systems and Emscripten set up.
Otherwise, I think it is ready to be merged.

@M374LX Thank you very much for this improvement!


Api definition guidelines

Description
This PR introduces a framework for managing OpenAPI definitions for InstructLab APIs and Platform APIs.

This looks pretty good to me, thanks!
My main request is that we merge #69 first (should be able to next week) and then we treat this as one of the first artifacts coming out of that working group.
The only change I'd suggest then is to move the doc/api-definition-guidelines.md file under docs/backend/, but that requires #69 to be in first.
There is a bit of additional context I'd like to capture, but I'm OK not blocking this PR on it. For example, this is written with an architecture in mind about an InstrutLab API and supporting platform APIs.  I'd like to write a high level backend direction doc that would introduce these concepts. I have existing content for this. If feedback on that doc changes anything substantially, we can always adjust these docs too.  It's all a moving target.

The hold label is just for waiting on #69.

That sounds good. I have an internal platform ADR about the decision to make REST APIs as the common service layer that we can adapt for the general backend architecture perspective.

#69 merged, so I'll drop the hold

This looks good to me!

Directionally, this looks fine. The initial guidelines are quite abstract and perhaps will be clarified / filled with essence once actual APIs are proposed and actual bindings are generated. The main point is - the guidelines clarified that bindings will be auto-generated and hosted elsewhere; and there will be a separate repo for schema itself. This should be enough to merge it IMO.

@markstur @russellb The guideline looks good to go. Do you have any further comments or obejctions to merging and moving forward?


.dbg files not deleted from OUT directory when using DeploymentItem on Class

Description

It seems when adding the [DeploymentItem("my_file_name")] attribute on the test class, the *.pdb files from the project are not deleted after test run.
The TestRun directory will start to bloat quickly with content, as I have over 20Mb of pdb files in each OUT directory now.

Steps to reproduce

Add the DeploymentItem attribute to test class.  In my case I am copying a .bak file from my build directory (copy always).  This .bak file is used in a [ClassInitialize] method to create a zero-pop database used by all tests.
Run any test method

Expected behavior

When finished the OUT directory is empty

Actual behavior

The OUT directory has all .pdb files

Environment

MS Test v2
.NET 4.7.2 - Visual Studio 2019

Hi, I'm not able to reproduce this issue, can you please share a sample repro project?

Yes I have attached a simple test project...instructions to reproduce behavior

Open solution and restore nuget packages
Rebuild solution
Debug the test
Look in the TestResults...out directory and see .dbg files

DemoTestProject.zip

Thank you for the repro. I was able to repro this and can see there are pdb files left behind during the debug scenarios. (Please note that this does not happen in a run test scenario however. The out directory gets cleaned up completely).
Thank you for this feedback but this is not high on our priority list right now to pick up.


falco: allow multiple socket paths for containerd

Motivation
We already support multiple containerd socket paths in Falco :point_down:
https://github.com/falcosecurity/libs/blob/34371e18cafa429107cf5d600bad5fd0c44718ab/userspace/libsinsp/container_engine/cri.cpp#L64-L68
This is not currently supported by the Falco chart.
This would particularly enable k3s by default but also allow users to add multiple paths in case their use case requires that.
Feature
Make the containerd.socket accept an array of paths. For example:
https://github.com/falcosecurity/charts/blob/86536781753f7e7d0a342b42af1feb5fa158202f/charts/falco/values.yaml#L266-L270
:point_down:
 containerd:
   # -- Enable ContainerD support.
   enabled: true
   # -- The paths to look for a ContainerD socket. The first match will be used.
   socket: [/run/containerd/containerd.sock, /run/k3s/containerd/containerd.sock]

Alternatives
The alternative would be to add a config node for each specific case (e.g., k3s). However, this is overkill and will not allow flexibility.
Additional context

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Related https://github.com/falcosecurity/falco/pull/3266

This likely needs to be addressed in 0.39, since https://github.com/falcosecurity/falco/pull/3266 has been merged.
/assign @alacuku

/remove-lifecycle stale


Add character images

Hi @gwendolyn954 , thanks for creating this project, this is super cool.
I was trying to use this API for one of my fun side projects, I noticed that you currently don't support the character images, are you planning to add it?

Hi @MadhavBahl , thank you so much!  Yes, I certainly plan to add character images.  I'm behind schedule but I hope to start adding them this month.


Update plugin.yml

It's default, not deafult.

:hushed:

Version 3.0.3 is pending, I will get this out in the next release.


Update readme with annotationProcessor instead of apt

Current coverage is 96.05% (diff: 100%)

Merging #755 into master will not change coverage

@@             master       #755   diff @@
==========================================
  Files            87         87
  Lines          2459       2459
  Methods           0          0
  Messages          0          0
  Branches        329        329
==========================================
  Hits           2362       2362
  Misses           68         68
  Partials         29         29

Powered by Codecov. Last update a5d3fdb...552b1da


Annotate typing

add type annotations  & add MyPy to pre-commit hooks
This is quite a large change, but the functional change is kept to a minimum.

@lkstrp could I kindly ask you to take over fixing the pre-commit bot. It looks like the pre-commit CI is sort of stuck with another set of (cached ?) arguments. Locally it works fine. What I want in the end is to run mypy checks on linopy/ and test/

@lkstrp awesome thanks


Make Site Responsive

I want to contribute to this issue. Please assign it to me. I am a full stack (MERN) developer.


Add PDF canvas element

Features

Add PDF canvas elements.

PDF files are imported via a document picker.
PDF files are displayed using PDFKit's PDFView.

Additional Details

Update info.plist with NSDocumentsFolderUsageDescription so that the application can access these resources.
Add canvasCenter to CGPoint to address the change in origin in #12.

Design Trade-offs
Handling different behaviours upon dismissing the sheet
Previously, we decided to use an enum to represent the currently displayed sheet. See https://github.com/Dynavity/dynavity/pull/9#discussion_r595957187 for more information. However, we run into the issue where each sheet can only have a single onDismiss action. With the addition of PDF canvas elements, we now need to call either addImageCanvasElement or addPdfCanvasElement in CanvasViewModel, depending on which sheet was last displayed.
The first solution I tried was to create another function in CanvasViewModel that called either one of the above functions by checking the value of activeSheet. Unfortunately, this is not possible as activeSheet would have already been set to nil by the time onDismiss is called.
Subsequently, I moved the activeSheet variable out of ToolbarView and into CanvasViewModel. There, I added setter functions for activeSheet (i.e., displayCamera, displayPhotoGallery, displayPdfPicker). In addition, I also separately kept a private variable lastActiveSheet. This variable was set along with activeSheet in each of the setter functions. By doing so, we are able to know what the value of activeSheet was before the sheet was dismissed. This solution was not ideal for the following reasons:

Unable to enforce private(set) on the activeSheet variable in CanvasViewModel. This is because the sheet itself accesses activeSheet through a two-way binding and thus requires write access. Unfortunately, this means that we cannot enforce changing the value of activeSheet only through out setter functions.
The use of the activeSheet and lastActiveSheet variables is not very elegant as the states of both variables have to be constantly kept synchronised. Future extensions to the code can very easily break the invariants we have set out, making the code brittle.
I discovered a separate issue that renders this solution unfeasible anyway. See below for a detailed explanation of the issue.

Delay in document picker setting the URL of the picked file
Unlike the image picker, the document picker has a slight delay in updating the selectedFile binding. With the previous solution discussed above in place, it was found that addPdfCanvasElement would trigger before the selectedFile binding was updated by the document picker. This might be due to the file picker having to create a copy of the file. However, this also raises the possibility that the image picker might suffer from this behaviour too (if the selected image is huge).
Clearly, some form of synchronisation is needed to solve this issue. Potential solutions include busy waiting, or setting a fixed delay before the functions to add the canvas elements are called. The former is undesirable as we will needlessly use up CPU cycles (as well as fail CS3217 since Prof Wai Kay will get an aneurysm if he sees us do that), while the latter not only makes the application feel less responsive, it also does not guarantee that the bindings will be updated before the functions are called.
Instead, we have a very elegant solution, which is to use the didSet property observer on our selectedImage and selectedFile bindings.
Update: @sebastiantoh correctly identified that didSet will trigger multiple times which is not ideal. After further consideration, I realised that I can just pass addImageCanvasElement and addPdfCanvasElement as callback functions to the image picker and document picker respectively. Not only does this solve the issue of the order of execution being unsynchronised, it also reduces coupling between CanvasViewModel and the respective pickers as we no longer need to maintain the selectedImage and selectedFile variables.
Issues
When adding PDFs, they follow the dimensions of the last added canvas element. This should be resolved once the logic to place items on the canvas based off the current view of the canvas is added.

Regarding some of the decision choices made, I was wondering if it's possible for the ActiveSheet enum to also contain an onDismiss handler?
I'm not sure if such an implementation would be susceptible to the "Delay in document picker setting the URL of the picked file" issue.

As long as we call the functions to add the canvas elements upon dismissing the sheet, the issue with the delay should be present. Also, I don't think it's possible to do anything with the ActiveSheet enum upon dismissing the sheet because the activeSheet var would have been set to nil already by then.

The reason why I think this could be better (if it works) is that we are guaranteed that onDismiss will only be called once (unless there's a bug with how SwiftUI implements it), whereas didSet will probably be triggered more than once (though this isn't really that big a deal)

Currently, the functions to add the canvas elements already have checks in place such that they should only add canvas elements once. However, let me explore the possibility of passing a callback function into the image picker and document picker directly.

Tested on my iPad, aside from the mentioned issue of the PDF following the dimensions of the last added canvas element, it was also found that whilst zooming, the pdf would "stutter"?
RPReplay_Final1616216172.MP4

Yea, this is another issue which surfaced after merging #12. Haven't narrowed down what's causing it but I suspect any changes we make to the PKCanvasView might also affect this behaviour. As such, I would like to get the repositioning, resizing, etc of canvas elements up first.

Tested on my iPad, aside from the mentioned issue of the PDF following the dimensions of the last added canvas element, it was also found that whilst zooming, the pdf would "stutter"?
RPReplay_Final1616216172.MP4

Yea, this is another issue which surfaced after merging #12. Haven't narrowed down what's causing it but I suspect any changes we make to the PKCanvasView might also affect this behaviour. As such, I would like to get the repositioning, resizing, etc of canvas elements up first.

Alright sounds good! Will merge this in first then.


Bad the Table's text-weight

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Material design table example
Table's text-weight should be 500.

Current Behavior
Your table example
text-weigth is 400

Your Environment

Tech
Version

Material-UI
beta.32

React
16.2.0

browser
Chrome 64.0.3282.140

Table's text-weight should be 500.

It's already 500:
https://github.com/mui-org/material-ui/blob/1f7930fa470342038e833eab69c1fb58c044d2dd/src/Table/TableCell.js#L27
https://github.com/mui-org/material-ui/blob/1f7930fa470342038e833eab69c1fb58c044d2dd/src/styles/createTypography.js#L15

@oliviertassinari see my example

@MaximKudriavtsev We follow the specification.


Failing to compile for iOS with simd enabled

By reading the source, I've discovered I can disable simd with -d:zippyNoSimd so this isn't a blocker for me anymore, but in case it's fixable, I've filed this anyway. Feel free to close if it's not worth the time.
# samp.nim
import zippy
let gz = compress("something")
doAssert uncompress(gz) == "something"

Compiling samp.nim with this command:
nim c --cpu:arm64 --threads:on --parallelBuild:1 --tlsEmulation:off "--passC:-arch arm64" "--passC:-isysroot" "--passC:/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS13.2.sdk"   /tmp/samp.nim

Results in the error Cannot select: intrinsic %llvm.aarch64.crc32b:
Hint: clang -c  -w -ferror-limit=3 -pthread -arch arm64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS13.2.sdk   -I/Users/matt/.choosenim/toolchains/nim-1.6.6/lib -I/private/tmp -o /Users/matt/.cache/nim/samp_d/@m..@s..@sUsers@smatt@s.nimble@spkgs@szippy-0.10.3<EMAIL_ADDRESS>/Users/matt/.cache/nim/samp_d/@m..@s..@sUsers@smatt@s.nimble@spkgs@szippy-0.10.3<EMAIL_ADDRESS>[Exec]
fatal error: error in backend: Cannot select: intrinsic %llvm.aarch64.crc32b
clang: error: clang frontend command failed with exit code 70 (use -v to see invocation)
Apple clang version 11.0.0 (clang-1<IP_ADDRESS>)
Target: aarch64-apple-darwin18.7.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
clang: note: diagnostic msg: PLEASE submit a bug report to http://developer.apple.com/bugreporter/ and include the crash backtrace, preprocessed source, and associated run script.
clang: note: diagnostic msg:
********************

PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT:
Preprocessed source(s) and associated run script(s) are located at:
clang: note: diagnostic msg<EMAIL_ADDRESS>clang: note: diagnostic msg<EMAIL_ADDRESS>clang: note: diagnostic msg: Crash backtrace is located in
clang: note: diagnostic msg: /Users/matt/Library/Logs/DiagnosticReports/clang_<YYYY-MM-DD-HHMMSS>_<hostname>.crash
clang: note: diagnostic msg: (choose the .crash file that corresponds to your crash)
clang: note: diagnostic msg:

********************
Error: execution of an external program failed: 'clang -c  -w -ferror-limit=3 -pthread -arch arm64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS13.2.sdk   -I/Users/matt/.choosenim/toolchains/nim-1.6.6/lib -I/private/tmp -o /Users/matt/.cache/nim/samp_d/@m..@s..@sUsers@smatt@s.nimble@spkgs@szippy-0.10.3<EMAIL_ADDRESS>/Users/matt/.cache/nim/samp_d/@m..@s..@sUsers@smatt@s.nimble@spkgs@szippy-0.10.3<EMAIL_ADDRESS>

Unfortunately ARM has no way of runtime detecting features / instructions so I must chose a default. In this case I chose to use the instruction since it is important for performance on M1 / M2 and future stuff. You'll want to use the -d:zippyNoSimd for cases where any instructions are not available, I am not aware of any way to avoid this without slowing down the default ARM executable a lot.


"Build Tools for Visual Studio" workloads do not install localdb

The documentation states:

Alternatively, you can install LocalDB through the Visual Studio Installer, as part of the Data Storage and Processing workload, the ASP.NET and web development workload, or as an individual component.

While this works for me for the standard Visual Studio Installer, it does not work when choosing the corresponding workloads on "Build Tools for Visual Studio 2022".  It might be helpful to note this on the documentation.  When installing "Build Tools for Visual Studio" it appears that there are no workloads which will install it, so for build servers, you currently have to install localdb separately if it is needed.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: ea66ca27-e0b1-cd1e-a8ef-55911169c71a
Version Independent ID: f2340ce9-9771-851e-60a7-237c6e55f085
Content: SQL Server Express LocalDB - SQL Server
Content Source: docs/database-engine/configure-windows/sql-server-express-localdb.md
Product: sql
Technology: configuration
GitHub Login: @rwestMSFT
Microsoft Alias: randolphwest

@davisnw Thank you. We are discussing the LocalDB experience in a separate internal conversation, so this is valuable feedback.

@rwestMSFT - this looks like a bug in the SSDT/VS installer for that SKU, rather than a documentation issue. A VSFeedback ticket would be more appropriate, I think, so the SSDT Team can look into it and see what can be done about it. There might be a slim chance that this is "by design", in which case it would make sense to update the documentation.

Thank you @Matteo-T. In that case I will close this issue.
@davisnw: please open a feedback item so the team can investigate further. You can do so from within Visual Studio.

#please-close


Random numbers with None value in shape

There are several use cases where the shape of random numbers is to be determined at runtime. E.g. for variational auto encoder or injection of noise into an MLPs units. As of now, this seems not possible in an elegant way with tensorflow:
import tensorflow as tf
x = tf.placeholder('float32', [None, 784])
noised = x + tf.random_normal(x.get_shape())

The above throws this traceback. I have to replace None above with a number known before runtime. But then I have to fix my batch size beforehand (to which the None corresponds) and need different expressions for each batch size.

TF.shape(x) should work instead.

Thank you!

like

Uh! that's actually really helpful :) thanks. Why is that?

Sorry but this doesn't seem to be working with TF2, as placeholders don't exist anymore.
Using None to specify the shape doesn't work either, i.e,
tf.random.normal((None, 784))
gives
ValueError: Attempt to convert a value (None) with an unsupported type (<class 'NoneType'>) to a Tensor.
Any idea, apart from using tf.shape() on another tensor of the model to extract the batchsize ?

Also having the same problem,
tf.shape() returns some wierd number for placeholder's shape.
    def call(self, inputs, training=True, **kwargs):
        print(inputs)
        print(tf.shape(inputs), inputs.shape)

gives me:
Tensor("Placeholder:0", shape=(None, 128, 128, 3), dtype=float32)
Tensor("RandomHSVPreprocessor/Shape:0", shape=(4,), dtype=int32) (None, 128, 128, 3)


Cover Size

how can i change the cover Art size when showCovertArt is true

In modules/MMM-NowPlayingOnSpotify/css/styles.css you can change how the module looks.  I recommend staying with percentages rather than in pixels to maintain your ratios during scaling.
In my example you can see that I changed the album cover to 80% width.  Adjust this to your liking.  If you need help with how CSS works, I highly recommend looking through the W3Schools site.


Specify input build for deployment

🚀 Feature
We would like to be able to specify an input for deployment - that is a folder with the whole documentation site built. This is because we would like to be able to:

Build the documentation
Then run some validation on that build
Supply this build for deployment instead of the deploy script building the documentaion again

Have you read the Contributing Guidelines on issues?
Yes.
Motivation
Speed up our deployment steps.
Pitch
Would open up the deployment script more. Could be used like yarn deploy --input "path/to/build".

The docusaurus deploy command already has an option for this purpose, isn't that what you need? https://v2.docusaurus.io/docs/cli#docusaurus-deploy

I might be misunderstanding something, but isn't that option just what the folder for the output should be?
What I would like is for there to be an input option, so you're effectively skipping the build step of the deploy script because you're supplying a build yourself.

Fine, then we need a new option something like yarn deploy --skip-build, right?

Yeah that would work as well.


Change toLiteralKey to use .path for Link

The toLiteralKey query.ts was returning "link: [[Object]]" for every link in the array so they all sorted into the same key. Fixed that by dereferencing to the path in the "link" case.

This fixes #161

Thanks!


slow with react

If i add a watch with react it takes around 29 sec to build.

I need more information. Can you provide your gulpfile?

Sure this is what my gulpfile looks like: https://gist.github.com/Mech7/2fb3be27652dde7f5b39
when i comment out line 30: .pipe(to5()) it goes fast again.
Am not really sure if this is a issue with gulp-babel or babel though.

How big is /home/chris/www/react-mediabank/react-mediabank/bundle.js?

it is 361 kb pretty much all of it is react

@Mech7 Why are you running babel over your builds (which will also transpile unnecessary files such as react) instead of individual files?

I'm getting the same message with Angular. I'm using Babelify and Browserify Shim as transforms and running in a Gulp task so everything is going through Babel.

@john-cheesman Ignore that folder/file. You should not under any circumstance be transpiling vendor files through Babel as it's unnecessary overhead and there are differences between ES5 and ES6 such as implicit strict mode that will outright break vendor code.

Sorted, thanks. I was using it as a global transform, bad idea :smile:

Ah ok i didn't know i wasn't supposed to do that either :)

Can you guys tell me how exactly you omitted babel-compilation of things like require('react')? Is it done by blacklisting in babel options?

@bf There's an example in the docs


Script is not reading the Examples table i am giving for assertions into my request body.

My script to assert value to a particular field in my requets body is:
Scenario: Register API and validate the swagger
Given url ApiAdminURL
And path AdminPath
And header apigateway-apikey = 'REGTEST'
And header apigateway-basepath = 'lambdaTest'

json myReq = read('users.json')
*** print 'subscription before', myReq.apiConf.subscriptionTiers**  // result for this line is  ["Unlimited","Gold", "Silver"]
*** set myReq.apiConf.subscriptionTiers = ''**
*** print 'subscription after', myReq.apiConf.subscriptionTiers**  // result for this line is comins=g as the tag itself  and not ["Unlimited", "Gold"] as set in the table
And request read('users.json')
When method post
Then status 200
Examples:
|subscriptionTiers    |
|Unlimited,Gold			  |

This script is setting the whole tag "" as the new value instead of taking the values from the table i am providing.
Can you please help ?

look, if you spend some time you will be able to create stand-alone examples, where I don't need to scratch my head as to what your URL is etc. see this example: https://stackoverflow.com/a/51731275/143475
for the last time, I request you to follow the instructions here: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue

myproject2.zip

I Have attached my project. Please help me with this.

And i think you didn't get my question initially. I just wanted to know the right syntax for "Examples" table i passed for subscription because the error i am getting is "not able to parse /Admin.feature file".
I am learning your framework, please don't be rude an help wherever possible.

I am learning your framework, please don't be rude an help wherever possible.

I'm trying to help, but it is clear you haven't bothered to read the documentation or look at the examples properly.
When you have Examples: you should use Scenario Outline: instead of Scenario:. Please read the documentation: https://github.com/intuit/karate#data-driven-tests

Oh my god!! I am so so sorry.
My bad completely.
Thanks a LOT!!


Installation of CoreDNS in the hub

This issue is stale because it has been open for 30 days with no activity.

This issue was closed because it has been inactive for 30 days since being marked as stale.


Expired profile public key. Check that your system time is synchronized and try restarting your game

Expected behavior
when re validating the session everything needed to jaoin a serv er is updated and works
Observed/actual behavior
after getting an invalid session I used this mod to make my session valid again however when I then tried to join a server I got the message 'Expired profile public key. Check that your system time is synchronized and try restarting your game' and was prevented from joining the server
Steps/models to reproduce

get an invalid session
use the mod to make your cession valid
try to join a server with the default chat validation settings

Version
1.19-4.1.0
minecraft: 1.19.2
Agreements

[X] I am running the latest version of the mod.
[X] My version of Minecraft is supported.
[X] I have searched for and ensured there isn't already an open issue regarding this.

Other
No response

2022-10-02-1.log
log file if that will help

Thanks for this, I'll try reproduce it soon by joining a server with chat verification turned on. I suspect the new chat message signing needs a new key generating.

I've found this related error message aswell:


externals set amd not work

config:

but bundle:

expected:

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template, or open a new issue.


Forms not working in deno with dev server

What version of HonoX are you using?
0.1.25
What steps can reproduce the bug?
Requires Deno 2
deno upgrade
deno upgrade rc

Repro: https://github.com/cotyhamilton/deno-honox
deno install
deno task dev

Vist http://localhost:5173/form and submit form
Additionally you can follow the instructions in the README and create form route to reproduce manually
What is the expected behavior?
The form is submitted successfully
What do you see instead?
The server errors with the message Missing content type in console and Internal Server Error 500 response returned to the client
Additional information
This only happens when running the dev server. If you build and serve the output the form submits successfully
deno task build # deno run -A npm:vite build
deno task preview # (cd dist && deno serve -R index.js)

Hi @cotyhamilton
Is this issue still occurring?

Yes it does. I just updated the deps in the linked repo and tested.
Or an alternate way is just scaffold a honox project with the cli and replace the adapter with @hono/vite-dev-server/node and create a form route
This is with latest deno version 2.1.4. (it could be an issue in deno instead of honox/hono vite dev server).
Thanks for looking!


Format (-F) command is not honouring the baseline file

I created a baseline file from the below command,
ktlint --baseline=ktlint-baseline.xml
and I could see no errors are reported if I run it again since there are no changes against the baseline file. But when I add the format  flag, the baseline file is getting ignored and the terminal spit out the fixed errors,
ktlint --baseline=ktlint-baseline.xml -F does not work as expected!
Can anyone help?

Current baseline implementation does not prevent linting, but rather it just suppresses found errors.
So in case of -F flag - format is still happening and, if non-formattable error present in the baseline still occurs, it will be printed. Another issue with format and baseline - it relies on error position in text file, so when format changes Kotlin file, errors positions may be updated.

I also ran into this inconvenience. I am looking for a workaround.

same! ^^^ would love if we could filter formatting with the baseline file

same! ^^^ would love if we could filter formatting with the baseline file

Please feel free to contribute a PR ;-)

@paul-dingemans Could you validate this issue? I believe fixed in 0.49.0-SNAPSHOT.

This issue should not be marked as bug. It can be seen as an enhancement. A possbile way to solve would be as follows:

pass baseline  to KtlintRuleEngine

parse code to AST as usual

Dynamically add a suppress annotation of a new type (for example @BaselineSuppress("ktlint:rule-id")) to the AST. Add this new type of annotation to the SuppressionLocatorBuilder.

lint/format the code as usual.

Errors for code suppressed in the baseline will no longer be reported due to the dynamically added suppression.
In case some other line of code in the files is autocorrected, the error which was suppressed in the baseline will not be affected anymore because of the dynamically added suppression.

In format function:

Update baseline. Due to format of lines of code which were not suppressed, the line/column of baseline errors might have changed.
when writing the output in the format function, filter out the dynamically added suppression (e.g. remove all @BaselineSuppress annotations.

Introduction of the annotation above also have other implecations that need to be mitigated:

Dynamically adding the annotation affects the line numbers of errors on lines below that annotation
Special care has to be given to the AnnotationRule. Dynamically inserting an annotation with an argument  @BaselineSuppress("ktlint:rule-id") on top of a construct that was already annotated might cause all annotations to be wrapped to separate lines. This is unwanted because the annotation will be not be persisted.

This change will require a considerable refactoring. I will not implement it myself as I am not convinced that it is worth to maintain the baseline functionality in the long term. I am a strong believer that a project that implements ktlint also should take the effort to resolve existing problems.


Code Review : ArmbreakerScraper.php

This review is according to the https://github.com/sylae/armbreaker/commit/3b46958d17b49580287bddd5dbc01ac067778944 commit.

Edit: Sorry could not miss this.

see #11 :p


Update Plugin Artifacts Links for v0.152.1

Automated changes by create-pull-request GitHub action

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


no image selected after captured image

In Android 13 there was an error - 20 then I updated the PR of https://github.com/ochakov/cordova-plugin-camera.git
now the camera is opening but once the captured image and clicked on the ok icon then "no image selected" error comes.
Attached video
https://github.com/apache/cordova-plugin-camera/assets/111554315/e1026472-708a-4c57-8768-40af35b12cc2

Can you try the formal v7 release? It would have contained a rebased version of the Android 13 support PR which I believe had additional fixes applied.

While building in production getting this error
ERROR: \app\build\intermediates\bundle_manifest\release\AndroidManifest.xml:203: AAPT: error: resource xml/camera_provider_paths (aka com.xxxx.XXXX:xml/camera_provider_paths) not found.

platforms/android/app/src/main/xml/camera_provider_paths.xml should exist. If it doesn't can you try again but with --verbose ?
cordova plugin remove cordova-plugin-camera
cordova plugin add cordova-plugin-camera@7 --verbose

attach file
verbose.txt

After the update to cordova-android@12 ,  same error "no image selected"
{
"name": "myAPPLite",
"version": "0.0.1",
"author": "Ionic Framework",
"homepage": "https://ionicframework.com/",
"scripts": {
"ng": "ng",
"start": "ng serve",
"build": "ng build",
"test": "ng test",
"lint": "ng lint",
"e2e": "ng e2e"
},
"private": true,
"dependencies": {
"@angular-material-components/moment-adapter": "^7.0.0",
"@angular/animations": "^14.3.0",
"@angular/cdk": "^13.3.9",
"@angular/common": "^14.0.3",
"@angular/core": "^14.0.3",
"@angular/forms": "^14.0.3",
"@angular/google-maps": "^14.0.4",
"@angular/material": "^13.1.1",
"@angular/platform-browser": "^14.0.3",
"@angular/platform-browser-dynamic": "^14.0.3",
"@angular/router": "^14.0.3",
"@awesome-cordova-plugins/android-permissions": "^5.44.0",
"@awesome-cordova-plugins/camera": "^6.4.0",
"@awesome-cordova-plugins/camera-preview": "^5.44.0",
"@awesome-cordova-plugins/core": "^5.43.0",
"@awesome-cordova-plugins/deeplinks": "^6.3.0",
"@awesome-cordova-plugins/file": "^5.44.0",
"@awesome-cordova-plugins/file-opener": "^5.44.0",
"@awesome-cordova-plugins/file-transfer": "^5.44.0",
"@awesome-cordova-plugins/geolocation": "^5.43.0",
"@awesome-cordova-plugins/launch-navigator": "^5.44.0",
"@awesome-cordova-plugins/onesignal": "^5.44.0",
"@capacitor/android": "^4.2.0",
"@capacitor/app": "1.1.1",
"@capacitor/core": "^3.5.1",
"@capacitor/haptics": "1.1.4",
"@capacitor/keyboard": "1.2.2",
"@capacitor/status-bar": "1.0.8",
"@ionic-native/app-availability": "^5.36.0",
"@ionic-native/core": "^5.36.0",
"@ionic-native/date-picker": "^4.20.0",
"@ionic-native/network": "^5.36.0",
"@ionic-native/zbar": "^5.36.0",
"@ionic/angular": "^6.1.11",
"@ionic/cordova-builders": "^6.1.0",
"@ionic/core": "^6.1.11",
"@ionic/storage-angular": "^3.0.6",
"@mediapipe/tasks-vision": "^0.10.2",
"@ngneat/tailwind": "^7.0.3",
"async-function-delay": "^2.0.0",
"bootstrap": "^5.2.0-beta1",
"bootstrap-icons": "^1.8.3",
"chart.js": "^3.9.1",
"cordova-browser": "6.0.0",
"cordova-plugin-androidx": "^3.0.0",
"cordova-plugin-geolocation": "^4.1.0",
"date-fns": "^2.28.0",
"echarts": "^5.3.2",
"hammerjs": "^2.0.8",
"js-datepicker": "^5.18.0",
"moment": "^2.24.0",
"ng-circle-progress": "^1.6.0",
"ng-lottie": "^0.3.2",
"ng-multiselect-dropdown": "^1.0.0-beta.19",
"ng2-pdfjs-viewer": "^13.2.2",
"ng2-search-filter": "^0.5.1",
"ngx-bootstrap": "^0.0.1",
"ngx-echarts": "^8.0.1",
"ngx-material-timepicker": "^5.5.5",
"ngx-star-rating": "^2.1.0",
"ngx-webcam": "^0.4.1",
"node-sass": "^7.0.1",
"rxjs": "~6.6.0",
"tslib": "^2.2.0",
"zone.js": "~0.11.4"
},
"devDependencies": {
"@angular-devkit/build-angular": "^14.0.3",
"@angular-eslint/builder": "~13.0.1",
"@angular-eslint/eslint-plugin": "~13.0.1",
"@angular-eslint/eslint-plugin-template": "~13.0.1",
"@angular-eslint/template-parser": "~13.0.1",
"@angular/cli": "^14.0.3",
"@angular/compiler": "^14.0.3",
"@angular/compiler-cli": "^14.0.3",
"@angular/language-service": "^14.0.3",
"@angular/localize": "^14.0.3",
"@capacitor/cli": "^3.5.1",
"@ionic/angular-toolkit": "^6.0.0",
"@tailwindcss/aspect-ratio": "latest",
"@tailwindcss/forms": "latest",
"@tailwindcss/line-clamp": "latest",
"@tailwindcss/typography": "latest",
"@types/echarts": "^4.9.15",
"@types/jasmine": "~3.6.0",
"@types/jasminewd2": "~2.0.3",
"@types/node": "^12.11.1",
"@types/offscreencanvas": "^2019.7.0",
"@typescript-eslint/eslint-plugin": "5.3.0",
"@typescript-eslint/parser": "5.3.0",
"autoprefixer": "^10.4.1",
"cordova-android": "^12.0.0",
"cordova-background-geolocation-lt": "^4.8.1",
"cordova-ourcodeworld-appinstalled": "github:ourcodeworld/cordova-ourcodeworld-appinstalled",
"cordova-plugin-actionsheet": "^2.3.3",
"cordova-plugin-android-permissions": "^1.1.3",
"cordova-plugin-androidx-adapter": "^1.1.3",
"cordova-plugin-appavailability": "^0.4.2",
"cordova-plugin-background-fetch": "~7.1.1",
"cordova-plugin-camera": "^7.0.0",
"cordova-plugin-camera-preview": "^0.12.3",
"cordova-plugin-cszbar": "^1.3.4",
"cordova-plugin-device": "2.0.2",
"cordova-plugin-dialogs": "^2.0.2",
"cordova-plugin-file": "^7.0.0",
"cordova-plugin-file-opener2": "^3.0.5",
"cordova-plugin-file-transfer": "github:apache/cordova-plugin-file-transfer",
"cordova-plugin-googlemaps": "github:mapsplugin/cordova-plugin-googlemaps#",
"cordova-plugin-ionic-keyboard": "^2.0.5",
"cordova-plugin-ionic-webview": "^5.0.0",
"cordova-plugin-network-information": "^3.0.0",
"cordova-plugin-splashscreen": "5.0.2",
"cordova-plugin-statusbar": "^2.4.2",
"eslint": "^7.6.0",
"eslint-plugin-import": "2.22.1",
"eslint-plugin-jsdoc": "30.7.6",
"eslint-plugin-prefer-arrow": "1.2.2",
"ionic-plugin-deeplinks": "^1.0.24",
"jasmine-core": "~3.8.0",
"jasmine-spec-reporter": "~5.0.0",
"karma": "~6.3.2",
"karma-chrome-launcher": "~3.1.0",
"karma-coverage": "~2.0.3",
"karma-coverage-istanbul-reporter": "~3.0.2",
"karma-jasmine": "~4.0.0",
"karma-jasmine-html-reporter": "^1.5.0",
"ngx-tailwind": "^3.0.0",
"onesignal-cordova-plugin": "^3.0.4",
"postcss": "^8.4.5",
"protractor": "~7.0.0",
"tailwindcss": "^3.0.8",
"ts-node": "~8.3.0",
"typescript": "~4.7.4",
"uk.co.workingedge.phonegap.plugin.launchnavigator": "^5.0.6"
},
"description": "An Ionic project",
"cordova": {
"plugins": {
"cordova-plugin-statusbar": {},
"cordova-plugin-device": {},
"cordova-plugin-splashscreen": {},
"cordova-plugin-ionic-webview": {},
"cordova-plugin-ionic-keyboard": {},
"cordova-plugin-androidx-adapter": {},
"onesignal-cordova-plugin": {},
"cordova-plugin-camera-preview": {},
"cordova-plugin-android-permissions": {},
"cordova-plugin-googlemaps": {},
"cordova-plugin-file": {
"ANDROIDX_WEBKIT_VERSION": "1.4.0"
},
"cordova-plugin-file-opener2": {
"ANDROID_SUPPORT_V4_VERSION": "27.+"
},
"cordova-plugin-cszbar": {},
"cordova-plugin-file-transfer": {},
"uk.co.workingedge.phonegap.plugin.launchnavigator": {
"GOOGLE_API_KEY_FOR_ANDROID": "",
"OKHTTP_VERSION": "3.12.0"
},
"cordova-plugin-background-fetch": {},
"cordova-background-geolocation-lt": {
"GOOGLE_API_VERSION": "19.+",
"APPCOMPAT_VERSION": "28.+",
"OKHTTP_VERSION": "3.12.+",
"EVENTBUS_VERSION": "3.3.1"
},
"cordova-plugin-network-information": {},
"com.ourcodeworld.appinstalled": {},
"cordova-plugin-appavailability": {},
"ionic-plugin-deeplinks": {
"URL_SCHEME": "soslite",
"DEEPLINK_SCHEME": "https",
"DEEPLINK_HOST": "soslite.page.link",
"ANDROID_PATH_PREFIX": "/",
"ANDROID_2_PATH_PREFIX": "/",
"ANDROID_3_PATH_PREFIX": "/",
"ANDROID_4_PATH_PREFIX": "/",
"ANDROID_5_PATH_PREFIX": "/",
"DEEPLINK_2_SCHEME": " ",
"DEEPLINK_2_HOST": " ",
"DEEPLINK_3_SCHEME": " ",
"DEEPLINK_3_HOST": " ",
"DEEPLINK_4_SCHEME": " ",
"DEEPLINK_4_HOST": " ",
"DEEPLINK_5_SCHEME": " ",
"DEEPLINK_5_HOST": " "
},
"cordova-plugin-camera": {
"ANDROIDX_CORE_VERSION": "1.6.+"
}
},
"platforms": [
"browser",
"android"
]
}
}

Can you reproduce in a minimal sample app? It should contain vanilla Cordova project (e.g. no frameworks) with just the minimum plugins to reproduce the issue (e.g. the camera plugin). The goal here is to isolate the issue and remove all other variables from play.
If the issue can be reproduced in the sample app, please share it and someone can take a deeper look.
If the issue cannot be reproduced, then it suggests the issue is something external of the plugin, or something else might be interfering with the plugin in some way.

Hello, this situation had solution ? I'm with the same problem.

My problem was resolved with a fork on git hub
ionic cordova platform remove android
ionic cordova plugin remove cordova-plugin-camera
ionic cordova plugin add https://github.com/jalios/cordova-plugin-camera.git
ionic cordova platform add<EMAIL_ADDRESS>

Closing as stale, no sample app was produced.


Android: Editing existing entry with history and saving causes invalid chars in protected fields

I'm currently playing around with your lib.
Opening, listing etc. works great.
But when i try to edit an entry and it contains protected fields (e.g. password), upon save, the password is "unreadable" for other keepass viewers. The testapp itself is capable of decoding the protected fields.
File extStore = Environment.getExternalStorageDirectory();
        mKeePassDatabaseFile = extStore.getAbsolutePath() + "/test_simple.kdbx";
        mDatabase = KeePassDatabase.getInstance(mKeePassDatabaseFile).openDatabase("test");
        Group parentGroup = mDatabase.getRoot().getGroups().get(0);
        Entry entry = mDatabase.getEntries().get(0);
        Entry newEntry = new EntryBuilder(entry).username("hello1").buildWithHistory();
        new GroupBuilder(parentGroup).removeEntry(entry).addEntry(newEntry).build();

        KeePassDatabase.write(mDatabase, "test", mKeePassDatabaseFile);

Before editing the entry was: username: "user1222", password: "pwd1"
The result entry is: username "hello1", but password is "c:w�"
These results i get when:

Entry has no history but i use buildWithHistory()
Entry has history and I use buildWithHistory()
Entry has history and I user build()

in short: only no history and with "build()" works...
Can you please tell me what I'm doing wrong? 😢
Thanks in advance

I'll take a look at it.

If I test with the code you provided it works perfectly on my system when I open the database with KeePass.
KeePassFile mDatabase = KeePassDatabase.getInstance(ResourceUtils.getResource("test1.kdbx")).openDatabase("test");
    Group parentGroup = mDatabase.getRoot().getGroups().get(0);
    Entry entry = mDatabase.getEntries().get(0);
    Entry newEntry = new EntryBuilder(entry).username("hello1").buildWithHistory();
    new GroupBuilder(parentGroup).removeEntry(entry).addEntry(newEntry).build();

    KeePassDatabase.write(mDatabase, "test", ResourceUtils.getResource("test1.kdbx"));

Can you provide a bit more details?

Is it working if you're trying it on a desktop system and not Android?
You mentioned that

The password is "unreadable" for other keepass viewers

what are these other keepass viewers? Does it work with the original KeePass program?
3. Can you provide a testcase that is not working? As mentioned, the code above works for me.

Well it seems to be a problem with Android only.
Tried it on the Desktop and it works without problems.
On Android the same code results in a "destroyed" password field.
to answer your questions:

It works on Desktop system
I tried the "original" KeePass Password Safe v2.35 on Desktop. Keepass2Android and KeepassDroid
I tried the same Code, on Android and Desktop. Works on Desktop, doesn't work on Android...

Below the file in it's different stages...
Android after edit: https://dropfile.to/O4UQ6wV
Desktop after edit: https://dropfile.to/k7bxX2T
Befoer edit: https://dropfile.to/ZycWp2c

Just a short update:
I might have found a new "hint".
It seems like the gzip compression breaks the file on Android.
I just created a new kdbx file with the windows client and set compression to "none".
Now editing and saving files with you library seems to work without problems.

Thanks for the hint, I can reproduce the issue and trying to fix it.

I finally found the issue. The problem is not the gzip compression, it's the behavior of Simple XML on Android. It does not preserve the order of XML tags on Android.
However, I managed to fix the issue.


helm-release not created but only resources are

Type of question
General question:
I'm reusing the github.com/operator-framework/operator-sdk/pkg/helm/release.NewManagerFactory and the github.com/operator-framework/operator-sdk/pkg/helm/release.NewManager() methods from the operator-sdk versionv0.9.1-0.20190724001845-d6e1aba9fa51`
Question
I created a new manager by providing
	f := helmrelease.NewManagerFactory(mgr, chartDir)
	helmManager, err := f.NewManager(o)

then I call the helmManager.Sync() and a helmManager.InstallRelease()
What did you expect to see?
I expect to see an helm release being created. helm list
What did you see instead? Under which circumstances?
All resources get created including the deployment and the application is running but no helm release is created.
Environment

operator-sdk version: v0.9.1-0.20190724001845-d6e1aba9fa51

Kubernetes version information:
insert output of kubectl version here

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:55:54Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5+icp-ee", GitCommit:"eb4df6c6fb47f5b4fd1ed8bfbfe2d0ed5ea636e1", GitTreeState:"clean", BuildDate:"2019-05-08T02:18:32Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:

Additional context
Add any other context about the question here.
Any idea? is the helmManager.InstallRelease() not enough to create the release?
When I look at the code, it seems that the InstallRelease loop on all resources and do a kind of kubectl create instead of an helm install with the whole chart. That's maybe the reason why the release doesn't appear.
Thx

@itdove The helm-operator library makes use of Helm and is compatible with Helm charts, but you shouldn't expect to use the operator and the Helm CLI interchangeably.
Specifically, this isn't working for because the helm operator runs its own internal tiller, scoped to just the namespace and permissions it needs to manage its built-in chart(s). The Helm CLI commands that query the cluster require Helm's actual tiller to be installed and configured, which the helm operator doesn't use. In addition, the helm operator uses a Secret backend (Helm 2's default is ConfigMap) and stores the release secret in the same namespace as the CR (Helm 2 stores it in the namespace of the global tiller, IIRC).

Thank you for the clarification!


How can I get back the old history search behavior in insert mode?

General information

Basic examination

[x] I have read through the README page
[x] I have the latest version of zsh-vi-mode
[x] I have tested with another terminal program

Problem description
When using the plugin, normal search behavior is gone. I mean that you can type sudo and then search all commands starting with sudo by hitting the up arrow key.

Reproduction steps

Install oh-my-zsh
Install ZVM
See the different history searching behaviour in insert mode

Expected behavior
I would like the same behavior in insert mode as without zvm.

Same here.
Great plugin, however this history thing is very annoying. Any plan to fix it or is there any workaround for the time being?

I fixed it by ditching this plugin completely. Just add bindkey -v to your .zshrc and you will have almost the same functionality.

Thanks @jclsn. I had the same issue. I added bindkey -v to my .zshrc and it works like a charm.

@jeffreytse No worries. Just merge this when you find the time :)


Add Karen Chu as recognized contributor

I am nominating or self-nominating a Recognized Contributor.
Name: Karen Chu
GitHub Username: @karenchu
Nomination
Karen has been part of building the Bytecode Alliance community by helping organize events like the BA Componentize The World event.
Optional: Endorsements

Bailey Hayes (@ricochet)

[ ] I have read and understood the qualifications for a Recognized Contributor

We discussed this in the last TSC meeting, and all agree that your contributions are worthy of recognition in the BA. Thank you for all the hard work!
@karenchu, to finalize the addition, please send an email to<EMAIL_ADDRESS>to let us know which address to use for communicating with you as a Recognized Contributor, and to confirm that you accept the nomination.
That address will be added to our RCs mailing list, which receives community and project updates every few weeks or so.
Once we've received that email, we'll add you to the mailing list and merge this PR. Thank you!


AntToast fix direction

If the prop is set to bottomRight/topRight the toast should slide from the right side in to the window and should slide out from the right side out of the window

The easyiest way is if the toast are sliding in from the top/bottom in to the window becaus if the toast are sliding from the right in the window the scrollbars are showing for a short moment


Add publisher examples

https://trello.com/c/R46p9TMH/366-docs-publisher-examples-for-publishing-api

Adds publisher v2 examples
Generates separate v2 content and links schemas in default task

/cc @jamiecobbett @danielroseman @tuzz

I wonder if https://github.com/alphagov/govuk-content-schema-test-helpers should default to checking against v2 publisher schemas?

@jamiecobbett Not sure about changing the default - it'll be more a question of changing the test configuration in publishing apps to use "publisher_v2" instead of "publisher" when calling GovukContentSchemaTestHelpers.configure.

This PR still requires v1 metadata in v2 schemas which isn't right (content_id and update_type being inaccurate) so closing until we resolve this omission.

@danielroseman updated to omit content_id and update_type from v2 schemas.

@steventux is practical to make base_path required in v2 in this PR?

@jamiecobbett thanks for spotting this, will add it to this PR :+1:

@steventux I think the attributes of the links schema should be nested inside a links attribute:
https://github.com/alphagov/content-tagger/blob/91b7e03f10b3e1694c5546cb1efdddf9f6f962f3/app/forms/tagging_update_form.rb#L28-L34
If we want to validate the entire payload previous_version should probably be a valid attribute as well.

@tijmenb thank you for spotting this, will update.

Updated.
/cc @tijmenb @danielroseman @jamiecobbett

@tijmenb see https://github.com/alphagov/govuk-content-schemas/commit/01e7a6c2206992cb7e654f05b755cfb147cda571 for update

:+1: looks good to me now.


[WIP][multimodal]Upgrade object detection backend to mmdet 3.0

wip
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for updating the version. Can you provide some benchmark results and attach them in the description? Thanks

Thanks for updating the version. Can you provide some benchmark results and attach them in the description? Thanks

Just completed the basic changes. I'll run a small benchmark after working on a PR in autogluon-bench. And will do a complete benchmarking after upgrading presets to more recent models (RTMDet/DINO).


operator: introduce cec l7 envoy loadbalancing cell

This commit moves the L7 loadbalancing registration from the legacyOnLeader cell into its own cell.
In addition, the corresponding config properties have been moved into its own config struct where possible.

/test

Let's cellify the world. ✔ 🐝

:smile: it's the last one for sig-servicemesh (withinlegacyOnLeader of the operator)

All tests have passed except "netlify/docs-cilium-io/deploy-preview". I'm closing and opening the PR to trigger it and see if it passes since the current failure looks like it was a flake.

@mhofstetter it looks that it didn't work, can you push force again? Thanks

rebased to main to re-trigger check netlify/docs-cilium-io/deploy-preview

/test


make separate typeclasses for "Add" and "Multiply" and "Scale"

Perhaps exclude Num and define operators ourselves.

Look at Numeric prelude, algebra and subhask.


Can't create a role assignment for Storage account with the latest CLI version: No registered resource provider found for location 'northcentralus' and API version '2020-04-01-preview' for type 'storageAccounts'.

Can't create a role assignment for the Storage account with the latest Azure CLI version. The API version '2020-04-01-preview' requested by CLI is not supported in all Azure regions.
Command Name
az role assignment create
Errors:
No registered resource provider found for location 'northcentralus' and API version '2020-04-01-preview' for type 'storageAccounts'. The supported api-versions are '2022-09-01, 2022-05-01, 2021-09-01, 2021-08-01, 2021-06-01, 2021-05-01, 2021-04-01, 2021-02-01, 2021-01-01, 2020-08-01-preview, 2019-06-01, 2019-04-01, 2018-11-01, 2018-07-01, 2018-03-01-preview, 2018-02-01, 2017-10-01, 2017-06-01, 2016-12-01, 2016-05-01, 2016-01-01, 2015-06-15, 2015-05-01-preview'. The supported locations are 'eastus, eastus2, westus, westeurope, eastasia, southeastasia, japaneast, japanwest, northcentralus, southcentralus, centralus, northeurope, brazilsouth, australiaeast, australiasoutheast, southindia, centralindia, westindia, canadaeast, canadacentral, westus2, westcentralus, uksouth, ukwest, koreacentral, koreasouth, francecentral, australiacentral, southafricanorth, uaenorth, switzerlandnorth, germanywestcentral, norwayeast, westus3, jioindiawest, swedencentral, qatarcentral'.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az storage account create --name $NAME --resource-group $RESOURCE_GROUP --sku Standard_LRS --kind Storage
STORAGE_ACCOUNT_ID=$(az storage account show --name $NAME --query id -o tsv)
az role assignment create --name SomeName --scope $STORAGE_ACCOUNT_ID --role "Storage Table Data Contributor" --assignee $AD_OBJECT_ID

Expected Behavior
Role assignment created
Environment Summary
Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.31, Ubuntu 20.04.5 LTS
Python 3.10.8
Installer: DEB

azure-cli 2.43.0

Extensions:
account 0.2.5
front-door 1.0.17
aks-preview 0.5.120
k8s-extension 1.3.7
log-analytics 0.2.2

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

@jiasli for awareness

Could be similar to https://github.com/Azure/azure-cli/issues/24919

I am not able to repro:
> az group create -n jlrg1219 -l northcentralus
> az storage account create -g jlrg1219 -n jlst1219 -l northcentralus
...

@jiasli You've missed  --sku Standard_LRS --kind Storage options for the az storage account create command from my example (that options forces to create V1 storage account instead of default V2).

Btw, after small investigation I just found that this bug is even more weird than I thought initially: it's not related to the storage account type but to the value for the --name option of az role assignment create command :-/

This command:
az role assignment create --name "/a/providers/Microsoft.Storage/t" --scope $STORAGE_ACCOUNT_ID --role "$ROLE_NAME" --assignee $AD_OBJECT_ID
results in:
No registered resource provider found for location 'northcentralus' and API version '2020-04-01-preview' for type 'storageAccounts'. The supported api-versions are '2022-09-01, 2022-05-01, 2021-09-01, 2021-08-01, 2021-06-01, 2021-05-01, 2021-04-01, 2021-02-01, 2021-01-01, 2020-08-01-preview, 2019-06-01, 2019-04-01, 2018-11-01, 2018-07-01, 2018-03-01-preview, 2018-02-01, 2017-10-01, 2017-06-01, 2016-12-01, 2016-05-01, 2016-01-01, 2015-06-15, 2015-05-01-preview'. The supported locations are 'eastus, eastus2, westus, westeurope, eastasia, southeastasia, japaneast, japanwest, northcentralus, southcentralus, centralus, northeurope, brazilsouth, australiaeast, australiasoutheast, southindia, centralindia, westindia, canadaeast, canadacentral, westus2, westcentralus, uksouth, ukwest, koreacentral, koreasouth, francecentral, australiacentral, southafricanorth, uaenorth, switzerlandnorth, germanywestcentral, norwayeast, westus3, jioindiawest, swedencentral, qatarcentral'.

The same command but with another --name:
az role assignment create --name "abc" --scope $STORAGE_ACCOUNT_ID --role "$ROLE_NAME" --assignee $AD_OBJECT_ID
gives:
The role assignment ID 'abc' is not valid. The role assignment ID must be a GUID.

And without --name option or with some GUID value for it the command succeed:
az role assignment create --scope $STORAGE_ACCOUNT_ID --role "$ROLE_NAME" --assignee $AD_OBJECT_ID
az role assignment create --scope $STORAGE_ACCOUNT_ID --role "$ROLE_NAME" --assignee $AD_OBJECT_ID --name $AD_OBJECT_ID

You've missed  --sku Standard_LRS --kind Storage options for the az storage account create command from my example (that options forces to create V1 storage account instead of default V2).

This is not relevant as role assignment doesn't care which kind of resource it is.

Btw, after small investigation I just found that this problem is even more weird.

Not really. 🙂
az role assignment create supports --name since https://github.com/Azure/azure-cli/pull/24324.
If you provide it like /a/providers/Microsoft.Storage/t, the URL for calling Role Assignments - Create API becomes something like
https://management.azure.com/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/resourceGroups/jlrg1219/providers/Microsoft.Storage/storageAccounts/jlst1219/providers/Microsoft.Authorization/roleAssignments/a/providers/Microsoft.Storage/t?api-version=2020-04-02-preview

which makes ARM think you are calling 2020-04-02-preview for type storageAccounts.
I can now repro the original error with such URL:
> az rest -u https://management.azure.com/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/resourceGroups/jlrg1219/providers/Microsoft.Storage/storageAccounts/jlst1219/providers/Microsoft.Authorization/roleAssignments/a/providers/Microsoft.Storage/t?api-version=2020-04-02-preview
Bad Request({"error":{"code":"NoRegisteredProviderFound","message":"No registered resource provider found for location 'northcentralus' and API version '2020-04-02-preview' for type 'storageAccounts'. The supported api-versions are '2022-09-01, 2022-05-01, 2021-09-01, 2021-08-01, 2021-06-01, 2021-05-01, 2021-04-01, 2021-02-01, 2021-01-01, 2020-08-01-preview, 2019-06-01, 2019-04-01, 2018-11-01, 2018-07-01, 2018-03-01-preview, 2018-02-01, 2017-10-01, 2017-06-01, 2016-12-01, 2016-05-01, 2016-01-01, 2015-06-15, 2015-05-01-preview'. The supported locations are 'eastus, eastus2, westus, westeurope, eastasia, southeastasia, japaneast, japanwest, northcentralus, southcentralus, centralus, northeurope, brazilsouth, australiaeast, australiasoutheast, southindia, centralindia, westindia, canadaeast, canadacentral, westus2, westcentralus, uksouth, ukwest, koreacentral, koreasouth, francecentral, australiacentral, southafricanorth, uaenorth, switzerlandnorth, germanywestcentral, norwayeast, westus3, jioindiawest, swedencentral, qatarcentral, eastus2euap, centraluseuap'."}})

So in order to make it work, either omit --name or provide --name with your own GUID.

Would it be good to add checking for the  --name argument to be GUID?

This is a good idea! I am considering refining the help message for --name in https://github.com/Azure/azure-cli/pull/24926.
However, "add checking for the --name argument to be GUID" is a client-side behavior and is hard to maintain cross client tools - Azure PowerShell, Azure SDK, etc. I will discuss within the team.


Change favicon and title

You have to change the website title to "Forum" and also change the favicon of this website.
For favicon, you can use the logo from HERE
Note

Make sure you follow the given template before making the pull request.

claim

Go ahead @sushantk1274

updates @sushantk1274 ?


Tables: Monotonicity preserving C1 interpolation

Reported by beutlich on 11 May 2015 18:24 UTC
In #1039 Martin O. proposed monotonic interpolants for the ModelicaStandardTables. Well, this should not be too complicated, at least for the univariate case.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1717

Comment by beutlich on 11 May 2015 18:33 UTC
Resolved by 6c5de73996a56837ab3c4cbcb7d86100c5d45f68.
CombiTable2D gives a not-implemented error for smoothness=MonotoneContinuousDerivative.

Changelog modified by beutlich on 11 May 2015 18:33 UTC
Added univariate Fritsch-Butland-spline interpolation (for CombiTimeTable, CombiTable1D and CombiTable1Ds)


Mistakenly identify 3.2 in "test version 3.2 by tomorrow" as a date using nlp

Using Australian locale and then to be sure defining separators:
c = pdt.Constants('en_AU')
c.dateSep = ['-', '/']
But insists that 3.2 is a date.  Is there any way to prevent this happening?

The piece of code is in init.py line 2581 where . and - are added as separators.  Have removed these for my purposes as I expect the separators I have defined to be respected.
   dateSeps = ''.join(re.escape(s)
                        for s in self.locale.dateSep + ['-', '.'])

I added in the - in all locales awhile back to support yyyy-mm-dd standard format which should be supported in all locales. However, that alternate separator should only be allowed for patterns that include all three components. I will work on fixing that pattern to avoid false positives.
Does anyone know why the . character is added here? It was there before I added the - so I preserved it but now cannot recall whether we discussed the origin of that separator. I suspect that it would have similar behavior matching only yyyy.mm.dd and not x.x as in OP's example but that does not look like a standard format.

There are actually test cases for the Australian locale suggesting that it should match with . as a date separator despite never explicitly defining that as a separator for the locale. There are countries that use that separator but I would be very hesitant to ever include it by default for the high likelihood of false positive matches on decimal numbers.
I am going to proceed by fixing the overzealous expression and removing those Australian test cases. The . is actually included in the default base locale dateSep = ['/', '.'] so even after this change most people will still have issues with decimals parsing as dates.


fix: coverage attribute rename

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

Current dependencies on/for this PR:

main

PR #1179   👈

This comment was auto-generated by Graphite.


Deprecate pipe-separated format

The pipe-separated format is very little used and for the reasons explained in #5199 we have decided to remove it. Removing it is a backwards incompatible change and this issue covers deprecating it.

An open design decision related to the deprecation is when to emit warnings and how to mark deprecation in the parsing model. The pipe-separated format is recognized line-by-line, but it certainly isn't a good idea to emit a warning for each line. A warning per file would probably be good, but a warning only once per execution can be considered as well.
A technical challenge related to this is separating deprecation warnings from errors in the parsing model. At the moment there's no such separation. Everything is reported as an error and then the code handling errors when the execution model is built just knows based on context are errors actually warnings or not. That may not work too well in this case and we probably should add some better mechanism.

I would report it for every file, but I'd say if it's possible to report block-wise, if the entire file is pipe-delimited, then it's one block, and it only reported once, but I've seen robot files where only a few lines of a file are pipe-separated to mimic the behavior of cucumber/gherkin example tables.

maybe add a custom parser that continues to support Pipes.
As i  #5199

Some users on Slack have told they really like the pipe-separated format and have lot of tests using it. To avoid breaking them, we could consider having the pipe format available optionally and in that case we shouldn't deprecate it. We want to get RF 7.2 out soon, so let's move this issue to RF 7.3 and decide about it later.

Would it be good to deactivate the pipe-separated format by default. This avoids that new projects / users use it.
A commandline / config parameter could allow to activate it for those users who have already existing test suites.
This procedure would support the transition.


tests: Fix pki with newer easyrsa

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

jenkins: test this please


Suggestions to v4 exomes release HT branch

Are the annotations in the order we want them in?
For instance:
'rsid': set
'filters': set
'was_split': bool
'a_index': int32
are randomly in the middle I think.

I havent thought about the order. I feel in the past when asking about order for VCF, people generally havent had a strong opinion but we can certainly change this if we want to move things up. Maybe @ch-kr could weigh in?

We need to decide on what interval info that we want to add to the release and where it should go? Under region_flags? We have calling intervals (both UKB and DSP) and we do also have interval QC that we can add if wanted.
Region flags or a new struct intervals sounds good to me. Probably also a @ch-kr question.

I understand why we would want these internally, but I'm not positive I like having them on a release HT:
datasets=Struct(dbsnp='gs://gcp-public-data--gnomad/resources/grch38/dbsnp/dbsnp_b156_grch38_all_20221116.ht', filters='gs://gnomad/v4.0/variant_qc/exomes/gnomad.exomes.v4.0.final_filter.ht', in_silico=['gs://gnomad/v4.0/annotations/in_silico_predictors/gnomad.v4.0.spliceai.grch38.ht', 'gs://gnomad/v4.0/annotations/in_silico_predictors/gnomad.v4.0.pangolin.grch38.ht', 'gs://gnomad/v4.0/annotations/in_silico_predictors/gnomad.v4.0.revel.grch38.ht'], info='gs://gnomad/v4.0/annotations/exomes/gnomad.exomes.v4.0.info.split.ht', freq='gs://gnomad/v4.0/annotations/exomes/gnomad.exomes.v4.0.frequencies.final.ht', vep='gs://gnomad/v4.0/annotations/exomes/gnomad.exomes.v4.0.vep.ht', region_flags='gs://gnomad/v4.0/annotations/exomes/gnomad.exomes.v4.0.frequencies.final.ht')
What about having a JSON output that includes this info so we have it internally, but don't need to have it on the release HT?

That works for me.

I'm not super concerned about the VCF ordering, though I agree that these fields feel like they should likely be earlier in the struct. One small question since I haven't looked at the code yet: are rsid and filters going to get unnested from the info struct so that they can exported into the ID and FILTER fields?
Yes they are unnested. At the moment, this produces the schema within the v4 release doc.

I'm going to merge this into my branch and make TODOs for the region flags, gencode and mane select vep globals, dropping the model ID, datsets to json, schema order, and validity checking the output.


WIP: Update gomobile

The commits cf270416ee26...8f9c870a5eee introduce visual changes on linux. :mag_right: 1 changed

Changed: Profile-Your_Profile_Broken_Proof_Menu (before) (after)

@keybase/react-hackers this should speed up our go build
please do a yarn rn-gobuild-ios and rn-gobuild-android and build the apps
once this merges we should kick off a bot build also


Add optional block_index arg to get_tx_info()

This params is needed for the kickstart.

Coverage increased (+0.01%) when pulling b431a8e7d8d039a66b3f9ebcc00c2de7a128932f on block_index into 499fff185dec2ccc4860d473b57b8fb903214ca9 on develop.

Coverage increased (+0.01%) when pulling b431a8e7d8d039a66b3f9ebcc00c2de7a128932f on block_index into 499fff185dec2ccc4860d473b57b8fb903214ca9 on develop.


Update dependency-injection-guidelines.md

Related to #39761, and was missing from #39743. See https://github.com/dotnet/docs/pull/39761#discussion_r1508944695

Internal previews

📄 File
🔗 Preview link

docs/core/extensions/dependency-injection-guidelines.md
Dependency injection guidelines

Something's not right yet. The image isn't displayed.

Something's not right yet. The image isn't displayed.

I see the inline image, but not the lightbox image - https://review.learn.microsoft.com/en-us/dotnet/core/extensions/dependency-injection-guidelines?branch=pr-en-us-39766#scoped-service-as-singleton.


Standardize "args" accross all commands, bug fixes, and other rafctors

Description

[ ] Refactors

[x] Standardize order of parameters for  all commands
[x] Standardize variable naming inside many commands (underscores instead of camelCase)
[x] Currently, command functions take a list of lower case list of arguments. The problem is that the command might actually care about the original capitalization of the args. The original solution was to pass the original command string to certain commands as well. However, this has resulted in difficult maintainability and bugs. For example, a mention prefix appears to have broken many things as many commands that do care about capitalization have switched to using the original command string (and not the passed args). Therefore, a list of args with their original capitalization should be passed to each command and commands should no longer use the original command string whatsoever. (That list of args should not include the prefix that was sent and should instead supply that prefix to the server_prefix argument for commands.) Commands are then free to use the original capitalization or use a lower case version.
[x] Refactor tabling commands

[x] Major overhaul for commands that accept a player name to use get_player_number_in_room
[x] General code refactor for tabling commands

[x] Some functions were renamed so they are more easily understood

[x] Bug fixes

[x] When a request to mkwx main page failed, functions erroneously returned a successful status (which resulted in AttributeError: 'NoneType' object has no attribute 'decompose' in the error logs)
[x] Slash command for setting flag failed because it was referring to a deleted variable. Additionally, fully removed flag exceptions code. (Some was leftover.)

[x] Documentation

[x] More documentation for SmartTypes module

[x] Removals

[x] Fully remove log_command and associated files and variables.

[ ] New Features/changes

[x] Invite message updated
[ ] Helper function UserDataProcessing.proccessed_lounge_add (see cdee0a7ab20508019e1ea124ca3663bfcf32c410) - refactored code where possible to use this function.
[x] Tabling commands that accept player names now support SmartTypes. Essentially, this means that all tabling commands can receive any player type (lounge name, FC, discord ID, mention) along with normal player number.

[x] Additionally, originally when you used a player name for tabling commands, you couldn't have a space in their name. Now you can.

Classification

[ ] Code changes have been tested
[x] This PR fixes an issue
[ ] This PR is not a code change (ie. documentation, typehinting, comments, etc.)

Still need to double check everything in TablingCommands and test everything.


Replace current lint:fix and format by single lint:fix

Actual
Currently keeping code formatted and linted requires to use two commands:

yarn lint:fix (or npm run lint:fix)
yarn format (or npm run format)

Both of that are proceed by another tool.
Expected

[ ] yarn lint:fix (or npm run lint:fix) should get same result like current solution.
[ ] Please consider about using eslint instead of currents commands

Further (do not resolve it):

add [QA] Auto lint:fix

As @Metastasis point in https://github.com/Siemienik/XToolset/pull/97#discussion_r531739830 , tslint being to be deprecated. I added info into description about migration into eslint


Cake displays raw ANSI output after running specific executables

What You Are Seeing?
ASCI character are returned to console for any output after running specific executables.
It works fine in the vscode console but exabits the problem in windows cmd console even when using PowerShell.
The same code used to work fine when running the older powershell version of cake but started happening when I switch to the dotnet tool version.
So far I can consistently reproduce with SqlCmd.EXE but it does not happen with something like ping which seems to works fine.
I'm passing an invalid argument for a simple poc that does not require a sql connection.
When the exe is successful for example change -??? to just -? the cake output after the exe is run is still incorrect.
Sqlcmd: '??': Unexpected argument. Enter '-?' for help.
←[41m←[37;1mAn error occurred when executing task '←[0m←[41;1m←[37;1mDefault22←[0m←[41m←[37;1m'.←[0m
←[41m←[37;1mError: ←[0m←[41;1m←[37;1mOne or more errors occurred. (Restore database failed)←[0m
←[41m←[37;1m    ←[0m←[41;1m←[37;1mRestore database failed←[0m

What is Expected?
To not see raw ASCI characters
Sqlcmd: '??': Unexpected argument. Enter '-?' for help.
An error occurred when executing task 'Default22'.
Error: One or more errors occurred. (Restore database failed)
    Restore database failed

What version of Cake are you using?
1.0.0-rc0002
Are you running on a 32 or 64 bit system?
64
What environment are you running on?  Windows? Linux? Mac?
Window
Are you running on a CI Server?  If so, which one?
No
How Did You Get This To Happen? (Steps to Reproduce)
Task("Default22").Does(() => {
    var settings = new ProcessSettings{
      Arguments = @"-???"
    };
    var exitCode = -1;
    using(var process =<EMAIL_ADDRESS>settings))
    {
        process.WaitForExit();
        exitCode = process.GetExitCode();
    }
    if(exitCode == 1){
        throw new Exception("Restore database failed");
    }
    Information("BlaBla");
    Information("Exit code: {0}", 5);
    Information("Exit code: {0}", exitCode);
    Information("BlaBla");
});
RunTarget("Default22");

where SqlCmd.EXE
C:\Program Files\Microsoft SQL Server\Client SDK\ODBC\170\Tools\Binn\SQLCMD.EXE

dotnet cake .\build.cake

It works fine in the vscode console but exabits the problem in windows cmd console even when using PowerShell.
The same code used to work fine when running the older powershell version of cake but started happening when I switch to the dotnet tool version.

There's no PowerShell version of Cake, you probably mean Cake Runner for .NET Framework. PowerShell is only used for bootstrapper, like you can have a PowerShell bootstrapper for Cake .NET Tool. Beside the host the different runners share most of the codebase including handling of console output.
That being said, what you see was introduced with Cake 0.38.0 with support for ANSI escape codes (https://github.com/cake-build/cake/issues/2749).

It works fine in the vscode console but exabits the problem in windows cmd console even when using PowerShell.
The same code used to work fine when running the older powershell version of cake but started happening when I switch to the dotnet tool version.

There's no PowerShell version of Cake, you probably mean Cake Runner for .NET Framework. PowerShell is only used for bootstrapper, like you can have a PowerShell bootstrapper for Cake .NET Tool. Beside the host the different runners share most of the codebase including handling of console output.
That being said, what you see was introduced with Cake 0.38.0 with support for ANSI escape codes (https://github.com/cake-build/cake/issues/2749).

I can confirm it was the powershell bootstrapper before.
Downgrading to the 0.37.0 version has restored color and no longer dumps asci after launching the exe.
The only main reason we use windows console is because we double click a .bat file which launches a menu and is where how cake is actually launched.
I attempted setting NO_COLOR env variable but it looks like it didn't make it into version 1.0.0-rc0002 or at least it wasn't working for me.  The NO_COLOR looks like it should work for us when it's released.
So currently by default in windows console there is no color before running the exe and what looks like raw asci after running the command.

Edition	Windows 10 Pro
Version	20H2
Installed on	5/28/2020
OS build	19042.630
Experience	Windows Feature Experience Pack 120.2212.31.0
Thanks so much for the assistance

I can confirm it was the powershell bootstrapper before.
Downgrading to the 0.37.0 version has restored color and no longer dumps asci after launching the exe.
The only main reason we use windows console is because we double click a .bat file which launches a menu and is where how cake is actually launched.
I attempted setting NO_COLOR env variable but it looks like it didn't make it into version 1.0.0-rc0002 or at least it wasn't working for me.  The NO_COLOR looks like it should work for us when it's released.
So currently by default in windows console there is no color before running the exe and what looks like raw asci after running the command.

Edition	Windows 10 Pro
Version	20H2
Installed on	5/28/2020
OS build	19042.630
Experience	Windows Feature Experience Pack 120.2212.31.0
Thanks so much for the assistance

https://github.com/cake-build/cake/commit/e55cca4375ae3d530ab5d5e0d44237ee503cdec2

https://github.com/cake-build/cake/commit/e55cca4375ae3d530ab5d5e0d44237ee503cdec2

NO_COLOR env https://github.com/cake-build/cake/commit/e55cca4375ae3d530ab5d5e0d44237ee503cdec2

NO_COLOR env https://github.com/cake-build/cake/commit/e55cca4375ae3d530ab5d5e0d44237ee503cdec2

I attempted setting NO_COLOR env variable but it looks like it didn't make it into version 1.0.0-rc0002 or it isn't working for me. The NO_COLOR looks like it should work for us when it's released.

Support for NO_COLOR will come in 1.1.

So currently by default in windows console there is no color before running the exe and what looks like raw asci after running the command.

The ANSI codes oyu see seem to be related to the output of sqlcmd as it doesn't happen with other commands and can also be reproduced in PowerShell. Still investigating on this.

I attempted setting NO_COLOR env variable but it looks like it didn't make it into version 1.0.0-rc0002 or it isn't working for me. The NO_COLOR looks like it should work for us when it's released.

Support for NO_COLOR will come in 1.1.

So currently by default in windows console there is no color before running the exe and what looks like raw asci after running the command.

The ANSI codes oyu see seem to be related to the output of sqlcmd as it doesn't happen with other commands and can also be reproduced in PowerShell. Still investigating on this.

The issue is that SQLCMD blindly overrides the console mode at the start, effectively disabling ANSI support from that point on. IMO it's a bug of SQLCMD (I reported it on their UserVoice, and you might want to open a ticket with Microsoft if your company has a paid support plan).

From the Cake side, I believe the only thing that can be done is to take a snapshot of the current console mode before starting a process and, after the process ends, restore it back to what it was in case the process changed it.
I opened PR https://github.com/cake-build/cake/pull/3021 that implements this behavior (1) as a possible workaround.

(1) With a small caveat: If you decide to write anything to the Console before the process ends, you'll still get raw ANSI output, of course. e.g.
using(var process =<EMAIL_ADDRESS>settings))
{
    // Writing ANSI output here won't work if SqlCmd is still running
    process.WaitForExit();
    // Writing ANSI output from this point onwards is fine as the console mode has been restored
    exitCode = process.GetExitCode();
}

The issue is that SQLCMD blindly overrides the console mode at the start, effectively disabling ANSI support from that point on. IMO it's a bug of SQLCMD (I reported it on their UserVoice, and you might want to open a ticket with Microsoft if your company has a paid support plan).

From the Cake side, I believe the only thing that can be done is to take a snapshot of the current console mode before starting a process and, after the process ends, restore it back to what it was in case the process changed it.
I opened PR https://github.com/cake-build/cake/pull/3021 that implements this behavior (1) as a possible workaround.

(1) With a small caveat: If you decide to write anything to the Console before the process ends, you'll still get raw ANSI output, of course. e.g.
using(var process =<EMAIL_ADDRESS>settings))
{
    // Writing ANSI output here won't work if SqlCmd is still running
    process.WaitForExit();
    // Writing ANSI output from this point onwards is fine as the console mode has been restored
    exitCode = process.GetExitCode();
}

Fixed by #3021

Fixed by #3021


Moctokit is not working with Act runEvent

Describe the bug
There is an issue when trying to set up a GitHub API mock using Moctokit to generate the mock and send it when running an event with act-js.
To Reproduce

Set up a YAML file with a GitHub script that queries the GitHub API, for example, to retrieve a repository.
Create a test using act-js and Moctokit with a mock to run the workflow, modifying the original response.
Run tests e.g. npm test.

Expected behavior
The response from running the workflow should contain the mocked information.

From the act-js documentation (I will add steps on how you can use it with octokit here soon)

You can use Mockapi and Moctokit to mock any kind of HTTP and HTTPS requests during your workflow run provided that the client being used honours HTTP_PROXY and HTTPS_PROXY env variables. Depending on the client, for HTTPS they might issue a CONNECT request to open a secure TCP tunnel. In this case Act won't be able to mock the HTTPS request.

If the client you are using does not honor these env variables, then the only way to actually make this work would be to make the client use a proxy forcefully. For octokit you can do something like this:
https://github.com/kiegroup/github-action-build-chain/blob/main/src/service/git/octokit.ts#L39
The only way I could think of bypassing this problem of CONNECT request was to setup a MITM server and somehow configure all the containers created by act to use that server as a proxy for all outgoing and incoming requests. This however is out of scope for this library, at least for now. Maybe if there is enough interest we can potentially start discussing ideas to implement it.

@shubhbapna  I thought it would be simpler without requiring extra modifications. A container-level proxy server seems like a good solution to explore.

@david-all-win-software Actually you might be able to use it without extra modification in example you mentioned.
@actions/github-script uses getOctokit from the @actions/github library - https://github.com/actions/github-script/blob/main/src/main.ts#L49
In the @actions/github library it seems that have a proxy configured which points to https://api.github.com :

https://github.com/actions/toolkit/blob/main/packages/github/src/utils.ts#L16
https://github.com/octokit/octokit.js#proxy-servers-nodejs-only

It looks like we can control the base url of the proxy using the GITHUB_API_URL env variable:
https://github.com/actions/toolkit/blob/main/packages/github/src/internal/utils.ts#L23
So in your test file all you need to do is initialize moctokit to use http instead of https and set the env before executing act:
const moctokit = new Moctokit("http://api.github.com");
const result = await act.setEnv("GITHUB_API_URL", "http://api.github.com").runEvent("workflow_dispatch", {mockApi: [...]})

Hopefully this works. I haven't gotten the chance to try it out but let me know if it works. I would like to document it

@shubhbapna Thanks, very good suggestion, I just tried it, but it's generating a 500 error status. It seems that it's not respecting the HTTP protocol we are setting.

I believe you're not initialising the moctokit properly. You are not passing the http protocol as suggested by @shubhbapna
const moctokit = new Moctokit("http://api.github.com");

I had a similar issue and this helped solve it.

Hmm I finally was able to try this out using the http protocol and I am still getting the same error. It seems like the proxy setting in github-script behaves a bit weirdly. It seems like for some reason when using an http protocol, actions/github-script just adds port 80 to the host but doesn't change the protocol. I get the following error:
HttpError: maximum redirect reached at: https://api.github.com:80/repos/kiegroup/mock-github

I will investigate and see if I can fix it in act-js

Just to update y'all, I think the issue over here is the proxy configuration in actions/github which results in establishing a TLS connection even when the proxy and the target are using http protocol.
I have reported the bug, lets see if the maintainers agree

Hi @david-all-win-software sorry for the delay, after talking to various people it seems like the proxy configuration in @actions/github is valid but unnecessary so I am not entirely sure they will accept my PR.
In the mean time I was able to figure out how to get this to work in my library itself. See the following PR for detailed explanation - https://github.com/kiegroup/act-js/pull/50
I have released the new version of @kie/act-js which contains this feature and you should be able to test your workflow easily now (hopefully). I have also added an example of the same here, feel free to refer to it:
https://github.com/shubhbapna/mock-github-act-js-examples/blob/main/workflow/github-script/README.md