upscayl: 2.9.8 -> 2.9.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/upscayl/versions.
meta.description for upscayl is: Free and Open Source AI Image Upscaler
meta.homepage for upscayl is: https://upscayl.github.io/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.9.9 in filename of file in /nix/store/4671rgnc90ihk4lcvmpikh8scqxan3bz-upscayl-2.9.9

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
upscayl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4671rgnc90ihk4lcvmpikh8scqxan3bz-upscayl-2.9.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A upscayl https://github.com/r-ryantm/nixpkgs/archive/a5f9ace1e1f6819d78d0ae09062c7c58cc67a6b0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a5f9ace1e1f6819d78d0ae09062c7c58cc67a6b0#upscayl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4671rgnc90ihk4lcvmpikh8scqxan3bz-upscayl-2.9.9
ls -la /nix/store/4671rgnc90ihk4lcvmpikh8scqxan3bz-upscayl-2.9.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

upscayl

Maintainer pings
cc @Icy-Thought for testing.

Question, will this pull the released version or the pre-release? Because the release was 2 hours ago and the pre-release was since yesterday.


Tomcat user gets invalid login shell on Redhat Enterprise Linux 6

Cookbook version
2.3.1
Chef-client version
12.11.18
Platform Details
RedHat Enterprise Linux 6
Scenario:
In order to bind the tomcat_user to a specific UID (for internal IT compliance reasons), the user is created before the tomcat_install. It is expected that the user retains it's properties during the tomcat_install run, which isn't the case.
Steps to Reproduce:
#
# Create users, groups, directories
#
group bsi_group do
  gid bsi_gid
  action :create
end

user bsi_user do
  supports:manage_home => true
  comment 'BSI User'
  uid bsi_uid
  group bsi_group
  action :create
end

user template_user do
  supports :manage_home => true
  comment 'BSI Template User'
  uid template_uid
  group bsi_group
  action :create
end

#
# Install and configure Tomcat
#
tomcat_install 'bsicrm' do
  install_path tomcat_path
  version tomcat_version
  tomcat_user bsi_user
  tomcat_group bsi_group
end

Expected Result:
User bsi_user, which is created before tomcat_install, retains it's properties after tomcat_install.
Actual Result:
User bsi_user has an invalid login shell after tomcat_install (/bin/nologin), whereas under RedHat Enterprise Linux 6 it should be /sbin/nologin. The error is in resource install.rb:
114   user new_resource.tomcat_user do
115     gid new_resource.tomcat_group
116     shell '/bin/nologin'
117     system true
118     action :create
119   end

@jmaass Thanks for filing this issue! Do you happen to know if this is invalid on other Enterprise RHEL versions or just 6?

It is invalid on RedHat Enterprise Linux 7 as well, I just checked. On this OS, nologin is in /usr/sbin/nologin, so you need to set the shell depending on the platform and the version.

Thanks for verifying that it affects more than the one OS. Checking other versions of OSes as well now.
Looks like RHEL 5 also impacted and should be
➜  tomcat git:(master) kitchen login multi-instance-centos-5
[root@9c88b5f116e7 /]# cat /etc/shells
/bin/sh
/bin/bash
/sbin/nologin
SUSE looks to also be /sbin/nologin.
Debian 7&8, Ubuntu 12-16, Centos 7 looks to be
/usr/sbin/nologin
At this point, not sure which OS is /bin/nologin. Thanks for filing this issue!

I'll create a helper for the shell paths. Easy enough

This is fixed by bf62b026c63c18fde8ba1a77304dec4c87dded03 with the use of /bin/false instead


clojurebridge swallos exceptions

at least when using like this:
python -c 'from clojurebridge import cljbridge;cljbridge.load_clojure_file(clj_file="train.clj",mvn_local_repo="/home/carsten/.m2/repository")'

This is quire sever, I would say.
Any type of exception in a clj file run via:
python -c 'from clojurebridge import cljbridge;cljbridge.load_clojure_file(clj_file="file.clj")'

is not visible to a user. Additionaly the process hangs, it seems to me.

Not true for all exception. But this for example:
Execution error at libpython-clj2.python.ffi/check-error-throw (ffi.clj:708).
Traceback (most recent call last):
  File "/home/carsten/.conda/envs/libpython-clj--194/lib/python3.9/site-packages/simpletransformers/classification/classification_model.py", line 361, in __init__
    raise ValueError(
ValueError: 'use_cuda' set to True when cuda is unavailable. Make sure CUDA is available or set use_cuda=False.

To reproduce :
crash.clj
(ns train
  (:require [libpython-clj2.python :refer [py.- py.] :as py]
            [libpython-clj2.python.ffi :as ffi]))
            ;; [libpython-clj2.require :refer [require-python]]
            ;; [libpython-clj2.python.gc]
(println "-------- manual-gil: " ffi/manual-gil)

(def locked (ffi/lock-gil))
(println :gil-locked)

(println :before-crash)
(py/import-module "not existing")
(println "crash fininished")

Run as embedded:
python -c 'from clojurebridge import cljbridge;cljbridge.load_clojure_file(clj_file="src/crash.clj",mvn_local_repo="/home/carsten/.m2/repository")'

I think we might be speaking past each other.  I did add an api - https://github.com/clj-python/libpython-clj/issues/221#issuecomment-1264682923
(py/with-manual-gil
(println :gil-locked)
(println :before-crash)
(py/import-module "not existing")
(println "crash fininished"))
- try the try-with-resources pattern.

yes, but this does not prevent the hanging, if somebody does not use teh API.

For me the "hanging" is a very frustrating experience for a beginner with libpython-clj, so I think we should prevent it, if we can. (or document it more prominently at least)

The libpython-clj API has now three different with-*** type of calls.
Maybe we could summarize in one place when they should be used, and what they could protect against.
Because on "first sight" none of it is needed, and the code seems to work anyway.

Totally agreed that documentation could be updated, changed, or added.  I honestly think usage.md is a bit out of date - when I get tmd updated to latest dtype (nontrivial full version change) I will circle back and spend some time on libpython-clj.

In this case the crashing is just something is trying to lock the gil when something else has it so in this case the crash is easy but the cause for for manual gil management in the first place, aside from perf, is that it reduces the crashiness of the system by quite a lot.  It is that, the original crashiness, that I would still like to know more about.


What is class SurfConext_Identity and is it still needed?

Is this still necessary?
https://github.com/OpenConext/OpenConext-engineblock/tree/master/library/SurfConext

This class was (is?) used for authentication purposes in Surfnet_Zend_Helper_Authenticate's direct method, but I cannot find evidence of it being called anywhere.

Would be nice if it could be dropped then

Fixed in #397


I need a way to add/control the number attemps when downloading a source.

Hello, first of all, thank you for this awesome tool! I've been using it for a couple of years now and it works amazingly!
But there's a problem that over time has become unbearable, and is that over the years I have built a massive list of host sources (187 sources so far) and sometimes one... or a few of those sources becomes temporarily unavailable or more commonly, their connection becomes slow or intermittent; which constantly makes hblock close with an error and I have to run it again, and again... and again until either succeeds or I give up after an hour wanting to throw the computer out the window!
Is there a way of making hblock automatically repeat several times (with a parameter controlled by the user) before giving up and instead of just exit with an error, just move to the next source in the list?
I REALLY hope this is possible to implement, 'cause this is becoming unbearable as I make my sources list more extensive.
Thanks in advance!

With the -c (--continue) option you can ignore download errors, but currently there is no retry or timeout option implemented.
I think it's a good idea to offer more control over downloads, as long as curl, wget and fetch support the feature it's not hard to add one more option.
In the meantime you can manually modify the fetchUrl function to suit your needs.


Docs for PckPacker flush verbose param are misleading

Godot version
3.4 stable, but probably All?
System information
N/A
Issue description
For this method
Error flush ( bool verbose=false )

the docs say

...If verbose is true, a list of files added will be printed to the console for easier debugging.

which is misleading given the implementation seemingly only prints out messages every 100 files:
https://github.com/godotengine/godot/blob/c55aa03c436fb40f9a42128397b69872d85fb237/core/io/pck_packer.cpp#L257-L264
So for small pck packing with < 100 added files, this will print nothing...
Steps to reproduce
mostly following the docs in PckPacker to produce a minimal example:
extends Node

func _ready() -> void:
	var small_packer = PCKPacker.new()
	assert(OK == small_packer.pck_start("small.pck"))
	assert(OK == small_packer.add_file("res://test.png", "icon.png"))
	assert(OK == small_packer.flush(true))
	# check output and see that nothing is printed

Minimal reproduction project
No response

I'd say the "every 100 files" check should be removed, and a line should be printed for every exported file instead.


Source generate a manifest of types in a project

Describe the feature
It's a common scenario to want to iterate over types, for instance, to register in EFCore.
Having a manifest generated will make this easier. Something like:
public static class MyProjectManifest : IEnumerable<IVogen>
{
   public static IVogen[] Types { get; }
   ... GetEnumerator() ... => Types.GetEnumerat();
   ...
}

The example above won't work as-is, but something similar should make registering everything in an assembly easier.

Just looked at this again and there isn't a common IVogen type that would be suitable as a base interface for all value objects. So there's no way to represent a collection of value objects.
I think the best that we can do is generate particular implementations for each scenario, which is what we do for things like System.Text.Json mapping, OpenAPI schema maping, etc.


[TODO] Remove specialist topics from database

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
This application is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs.
Follow these steps if you are doing a Rails upgrade.

Sorry @unoduetre am only getting to this now...
I don't think we need/should do this:

When a document is unpublished/redirected it remains in the database. We should follow that pattern.
Publishing API is the source of truth, it stores all of the documents, along with their current state (via the Edition) so deleting these documents means we are losing access that that history.

@brucebolt do you agree?

@hannako I agree, not sure we need to do this. I assume all the specialist topics are already marked as withdrawn or unpublished so I don't think there's a need to do this

Closing the PR.


Support lsp v31.0.6960

Summary
prisma-language-server has changed the bin.js output path from dist/src/bin.js to dist/bin.js since v31.0.3644
https://github.com/prisma/language-tools/commit/c45b1a5bf3e9206329ea0cb37576a61ce337d880#diff-1238398146196c4e652a4d775b225e4740198f2e59a0d4e1db9be9df69a05af0
Because of that change, this extension does not work, so I have corrected the serverModule path.

@pantharshit00 can this be merged?

@dickfickling Sure, thanks for the reminder


Copy placeholders and articles from fluidframework.com

Docs Build status updates of commit 569e99f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.emptyschema.json
:white_check_mark:Succeeded

.vscode/settings.json
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/architecture.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/compatibility.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/container-management.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/containers-runtime.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/dataobject-aqueduct.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/dds.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/debugging.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/concepts/service.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/howtos/auth.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/howtos/azure-frs.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/howtos/custom-logging.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/howtos/enable-logs.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/howtos/service-clients.md
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/articles/toc.yml
:white_check_mark:Succeeded

azure-fluid-relay-preview-pr/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update class diagrams to reflect current state

Codecov Report
Base: 61.42% // Head: 61.42% // No change to project coverage :thumbsup:

Coverage data is based on head (0209f16) compared to base (5b3bc8f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #314   +/-   ##
=========================================
  Coverage     61.42%   61.42%
  Complexity      627      627
=========================================
  Files           112      112
  Lines          2263     2263
  Branches        257      257
=========================================
  Hits           1390     1390
  Misses          779      779
  Partials         94       94

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


DELETE_INSTANCE_ID is set false by default

Description
DELETE_INSTANCE_ID requires Athenz permission to do so, and should not be set true by default as it is now.
This also handles the issue in https://github.com/AthenZ/k8s-athenz-sia/issues/44

generated issue from this pr: https://github.com/AthenZ/k8s-athenz-sia/issues/86


Using other input/output size than the 320x320 used in the multi class segmentation example

How do I extend the image input/output to anything greater than 320x320 which is currently used in the multi class segmentation tutorial in this package?
When I simply changed the input size to 512 the run crashed, so I figured that the network should be configured to a different size before. How do I do that?
And how is that related to the pretrained model? If I'm using the pretrained model in the multi class segmentation tutorial then do I have to use 320x320 sized images? What other pretrained models are available here, especially for pathology applications?
Thanks

The readme lists all of the architectures and all of the encoders. None of them are ready for use on histopathology images right out of the box because the pre-trained weights are for ImageNet. You'll need to either fine-tune a model with those weights, or train from scratch.
Changing the input/output to anything greater than 320x320 is shown in the docs:

model = sm.Unet(BACKBONE,
                             encoder_weights='imagenet'
                             input_shape = (512, 512, 3)

If your computer can't run anything larger than 320x320 because it crashes, that's a memory issue and means you need to get a different GPU that can allocate more memory, or just use what it can handle. If its not a memory issue, then you should show the error here otherwise no one will know what you mean by it 'crashing'.

So the model definition in the example is not using any pretrained network (model = sm.Unet(BACKBONE, classes=n_classes, activation=activation)) and basically means I'mm training from scratch? And I just need to add encoder_weights='imagenet' to have it train starting with the imagenet-based weights which will further be optimized to my problem during training? I consulted with fellow pathology image analysis groups and they confirmed they also use imagenet because apparently there isn't a pathology-specific pretrained network.

"So the model definition in the example is not using any pretrained network (model = sm.Unet(BACKBONE, classes=n_classes, activation=activation)) and basically means I'mm training from scratch? And I just need to add encoder_weights='imagenet' to have it train starting with the imagenet-based weights which will further be optimized to my problem during training?"
Yes, and if you use one of the efficientnet encoders, look at using noisy-student weights, which come from the paper Self-training with Noisy Student improves ImageNet classification


Showing 0 review the 'xExtents' prop of ChartCanvas

I'm having this error when loading my data. The problem is that my array is empty before the data is loaded from the server and I get this error. The graph works properly after the data is there. The graph can be loaded empty.
https://github.com/rrag/react-stockcharts/blob/49f2e4e49f7d0445552085a1128ef3278c030315/src/lib/ChartCanvas.js#L240

Cannot render graph with empty data because the chart calculates the x and y domain based on the data you send in. Hence the error. This is per design. The solution is to show a loading icon/message prior to data being available

@rrag thanks for the reply. Does this mean the whole ChartCanvas should be mounted after the data is available? and in case the data goes away completely removed?

yes mount after data is available.


Inquiry About Future Support for Controller Inputs in Flet

Hello Flet Team,
I am currently working on a project using the Flet framework and am interested in incorporating controller(joystick) input, specifically for devices like the Nintendo Switch controller. I have searched through the documentation and existing issues but couldn't find relevant information about this feature.
I wanted to inquire if there are any plans to support controller inputs in the Flet framework in the near future. This feature would be highly beneficial for developing more interactive and diverse applications, especially those requiring non-traditional input methods.
If this is something that's already being considered or in development, could you please share any relevant details or progress updates? Additionally, if it's not currently on the roadmap, I would like to express my interest in this feature and hope to see it in future updates.
Thank you for your time and the excellent work on Flet. Looking forward to your response.
Best regards,
carimatics

What do you think of this pypackage : https://pypi.org/project/pyjoystick/ ?

Thank you for your prompt response.
I was not aware of that package until now.
Having briefly looked through the documentation, I feel it might be possible to integrate it into the Flet app I am developing.
I would definitely like to give it a try.
Thanks.

Nice!
Will be great we move this thread to a discussion, wdyt?

I think moving this thread to a discussion is a very good idea.
In my opinion, whether the code using pyjoystick works correctly or not, it would be very welcome if Flet could natively support controller input handling(like Keyboard shortcuts).
However, I cannot gauge whether this request would be useful for many others as well.
To ascertain that, I think moving this thread to a discussion is a very good idea.

Following your advice, I tried combining Flet with pyjoystick in my app.
As a result, I successfully achieved the desired behavior.
However, a new problem has arisen: the app does not launch when created with PyInstaller (it closes without any error message), but I have made progress from the initial issue.
Therefore, I would like to close this issue for now.
I am very grateful for your precise advice.
Additionally, I will create a new discussion regarding support for controller inputs.
Thank you as well for your advice on this matter.
Thanks!

Awesome!
Feel free to open discussions to discuss on the issue that will pop up with time. 😉
Eager to see how your app will look like at the end. (I find the idea with the joystick very interesting and amazing.)


Add data validation for rules engine

Input to the rules engine should be validated:

Move the data models from the test_examples into the rules engine (will require restructuring of test input data)
Move validation from test_examples into rules engine
Add new validation as appropriate

Separate out the data objects into separate classes (e.g., outdoor temperature, billing period usage)
Evaluate operations performed on data and restructure classes if necessary (for instance, take a look if the functions called in the initialization of BillingPeriod should be moved elsewhere)
Investigate how fuel data will be input into the rules engine (i.e., will it be parsed by the Javascript or does it need to be validated by the Python)

Data models broken out into #106.

Validation is provided by the Pydantic models.


InvalidRequestError: This model's maximum context length is 4097 tokens. However, your messages resulted in 5822 tokens. Please reduce the length of the messages.

System Info
Hey! Below is the code i'm using
llm_name = "gpt-3.5-turbo"
# llm_name = "gpt-4"
os.environ["OPENAI_API_KEY"] = ""

st.set_page_config(layout="wide")

def load_db(file_path, chain_type, k):
    loader = PyPDFLoader(file_path)
    documents = loader.load()
    text_splitter = RecursiveCharacterTextSplitter(chunk_size=2000, chunk_overlap=300)
    docs = text_splitter.split_documents(documents)
    embeddings = OpenAIEmbeddings()
    db = DocArrayInMemorySearch.from_documents(docs, embeddings)
    retriever = db.as_retriever(search_type="similarity", search_kwargs={"k": k})
    qa = ConversationalRetrievalChain.from_llm(
        llm=ChatOpenAI(model_name=llm_name, temperature=1),
        chain_type=chain_type,
        retriever=retriever,
        return_source_documents=False,
        return_generated_question=False
    )
    return qa

Even though i'm using RecursiveCharacterTextSplitter function, it is returning below error.
InvalidRequestError: This model's maximum context length is 4097 tokens. However, your messages resulted in 5822 tokens. Please reduce the length of the messages.
Is there anything which will fix this issue?
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction

Expected behavior
I'm using RecursiveCharacterTextSplitter function where it'll make use of the function and will exceed the context length. It should work right?

Seems related to https://github.com/langchain-ai/langchain/issues/1349.

I also encountered this problem, but I haven't found a solution yet. I only asked 1+1, and this error occurred. I want to know how to see the content of prompt and completion when he said that the length is exceeded.


Execute validate entry point before accepting a transaction

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

Feature

What is the current behavior?
Currently all transactions are not being validated prior actual execution.
Resolves: (https://github.com/keep-starknet-strange/madara/issues/69)
What is the new behavior?
Transactions (Invoke, Declare, Deploy Account) are being validated using validate_tx method that invokes:
validate, validate_declare, validate_deploy on the Cairo contract
Does this introduce a breaking change?
No

thanks a lot for this contribution, do you think you could add integration tests (in typescript) as well ?
And also would be awesome if you could run benchmarks and share the overhead on performance of adding this validation.

@marioiordanov are you on the telegram group ?
u need to register to Only Dust to receive payments for your contributions.
https://app.onlydust.xyz/

@marioiordanov are you on the telegram group ?
u need to register to Only Dust to receive payments for your contributions.
https://app.onlydust.xyz/

Sorry for my delayed answer I am involved in another project. Actually I am in the telegram group.
About the typescript integration tests, I will make a PR when I find some free time.

@marioiordanov are you on the telegram group ?
u need to register to Only Dust to receive payments for your contributions.
https://app.onlydust.xyz/

@abdelhamidbakhta Sorry for my delayed answer I am involved in another project. Actually I am in the telegram group. About the typescript integration tests, I will make a PR when I find some free time.
What is your handle on telegram ?

@mrdninv


boards: Fix arduino_nano_33_ble PCB initialization problem

The board_internal_sensors_init function was called erlier than the GPIO
driver was initialized. This commit fixes the issue.
Signed-off-by: Przemyslaw Bida<EMAIL_ADDRESS>

@galak @MaureenHelm Can I count on our review?

@galak @MaureenHelm Can I count on our review?


Add Erik von Reis as maintainer

Checklist

[x ] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Well that's special.  The conda build files for python3.10 use imports not available until python3.11.


Stopped working on iOS 10

The playground no longer shows up for me when I visit it in Safari or Firefox on iOS 10. All I see is the light brown background. iOS 10 is a fairly old version and it's reasonable to not support it, but I though I'd let you know in case it affects other iOS versions or is an easy fix.

Hmm. This certainly wasn't super intentional, but it is following the Rust recommended guidelines for browser compatibility. Is there something preventing you from upgrading to iOS 11?


'+' not supported

If you subscribe to aaa/+/bbb the messages sent to 'aaa/ccc/bbb' are not displayed.
You might want to use something like MQEmitter to route the messages to the right subscriber UI.

Fixed in https://github.com/sandro-k/MQTTLensChromeApp


Sqlite provider added + fixes in sql_server_user provider

Add SQLite provider (drop and query support)
Allow to pass options to the connection string for all databases for setting initial settings
fix in sql_server_user provider where a unused variable was used (sql_role -> sql_sys_role attribute mismatch)

Hi @tas50, I've corrected the requested items, and added some tests too. please let me know if there are any additional things that need taken care of. Thanks!

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@enzor

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

i'll recreate this pull request, as the rebase made it a mess


fix: register service worker in dev

This PR also includes:

fix linting,  excluded a few folders for dev and build
included prev/next links ready for next VitePress alpha release: https://vitepress.vuejs.org/guide/theme-prev-next-link.html
update deps with taze -r -w: beware, taze is resolving VitePress versions wrong, resolving to draft instead to alpha
split plugins

EDIT: there is no race condition, useImportRegister never updated on dev with dev options enabled:
https://github.com/antfu/vite-plugin-pwa/blob/main/src/index.ts#L133
supersedes #319 (sw in dev part)

@antfu I'll merge into main, you can release it or wait until I finish the new PR about adding support also for unregister the sw/pwa via selfDestroying option


Pull the route reflector version indicated in versions.yml

Description
Now that master's route reflector is only compatible with v3.0+, pull the route reflector version indicated in versions.yml.
Let's see how the STs do with this change.
Release Note

None required

@neiljerram - can you include these changes in your RR work on calico:master?

Thanks for handling this @bcreane ; yes, I will add the same change to #1419


[Select] Demo flickering on open in v4 docs using iOS v15.4

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

material-ui v4.12.4 Select  component flickers when opening on IOS v15.4 on both chrome and safari. i have also checked the new mui Select v5 and it's kind of better but not completely smooth and some flickering still happening.
https://v4.mui.com/components/selects/
Expected behavior 🤔
Selects should open and close smoothly on IOS devices.
Steps to reproduce 🕹
Steps:

visit https://v4.mui.com/components/selects/ with an IOS v15.4 device
tap on one of the select components to open

Context 🔦
i'm currently using the @material-ui/core v4.12.4 in my web application and this behavior is affecting my user experience.
i'm not able to migrate to v5 right now, although this issue also applies to the new version as i mentioned earlier.
Your environment 🌎
No response

https://stackoverflow.com/a/65801101/13802486
The solution here worked for me, if anyone is looking for a temporary answer. It disables the animations though.

https://stackoverflow.com/a/65801101/13802486
The solution here worked for me, if anyone is looking for a temporary answer. It disables the animations though.

Just in case, if anyone uses select with TextField, you can pass SelectProps: {MenuProps: {transitionDuration: 0}} to apply this temporary answer.
<TextField
  select
  SelectProps={{
    MenuProps: {
      transitionDuration: 0,
    },
  }}
  {...otherTextFieldProps}
>
  // some MenuItems
</TextField>

This appears to be a duplicate of #32138 and #31380. It's the result of a bug in Safari 15.4. See #31380 for more information and a workaround (which has already been implemented in MUI 5.x).


1.5.1 patch release with regex fix

We must do a 1.5.1 bugfix release with this commit: (already included in 1.6.0)
https://github.com/railslove/rack-tracker/commit/b3d11e65dfc4ff60d7909c449a0e336f4da2ffb5

I'm not sure if this is really necessary 🤔
What might be a possible reason to not be able to update to 1.6?

because semantic versioning. it is a rather important bug fix and project that specify ~>1.5.0 as a version should get that fix.
the patch version says it is a backwards-compatible bug fix.
the minor version introduced new features that are backwards-compatible.

Alright then, do it! :)

thanks!

bump, @DonSchado  can you still to do a 1.5.1 release?


Disallow flags union as generic type argument

It is possible to use enumerators as generic type arguments, in which case Crystal resolves the enumerators to their underlying constants:
enum Foo
  X
  Y
end

class Bar(N); end

Bar(Foo::X) # => Bar(0)
Bar(Foo::Y) # => Bar(1)

However it doesn't work so well when a flags union is formed:
@[Flags]
enum Foo
  X
  Y
end

class Bar(N); end

Bar(Foo::X)          # => Bar(1)
Bar(Foo::Y)          # => Bar(2)
Bar(Foo::X | Foo::Y) # => Bar(Foo)
Bar(Foo.flags(X, Y)) # Error: expecting identifier 'class', not 'flags'

The third line is rather misleading, as one may expect the union of two flags constants to be another enumerator, not to interpret both sides of | as types. I'm in favor of making this specific case a compiler error until a more general solution like #5427 is implemented. Something along the lines of:
Error: can't use union of flags enum as generic type argument yet

Code that relies on such unions would have to explicitly define the corresponding enumerator:
enum Foo
  X_or_Y = X | Y
end

Bar(Foo::X_or_Y) # => Bar(3)

I think that the first snippet compiles is actually the bug. None of these should compile. Generic type arguments can only be types or number literals.
The problem is that enum members, and constants, are represented as types and so in many places they can be passed as such, but that shouldn't be allowed.

I think that the first snippet compiles is actually the bug. None of these should compile. Generic type arguments can only be types or number literals.
The problem is that enum members, and constants, are represented as types and so in many places they can be passed as such, but that shouldn't be allowed.

Support for enumerators is due to 2bb844c437f301d93e2fc54e861815065f07a9fc. The actual code (as it is right now) doesn't distinguish enumerators from other kinds of constants.
The original intent is to support more constant expressions in the extent argument of StaticArray. Putting the extents in an enum is probably a code smell so I wouldn't miss it if enumerators are disallowed altogether. Disallowing other numeric constants at this point is too much.

Support for enumerators is due to 2bb844c437f301d93e2fc54e861815065f07a9fc. The actual code (as it is right now) doesn't distinguish enumerators from other kinds of constants.
The original intent is to support more constant expressions in the extent argument of StaticArray. Putting the extents in an enum is probably a code smell so I wouldn't miss it if enumerators are disallowed altogether. Disallowing other numeric constants at this point is too much.


Null-safe equals docu and Cypher link

Add docu of null-safe equals operator to Filtering and Appendix
Add mini tutorial on Cypher to Cypher Section

Thanks again!!!


What data is nvstore supposed to be used to store?

This might be a silly question, but there's no indication in the README.

Fair enough. NVStore is an infrastructural module, which can be used to store any item on the flash. However, it is mainly intended for the storage of security related items, like crypto keys. I'll try to add a few words there to clarify.

so what is recommended to store configuration type stuff on nrf52 devices?

This feature will be used as an infrastructure for other features. For instance, we develop a feature called Device Key, which holds a per device unique seed for random generation (for devices that don't have true random generators). Later on, we will develop a secure storage, which will store the file signatures there.

so do you have a recommendation for configuration storage for all mbed devices (especially nrf52 devices). I really dig the interface of this vs wrangling with FlashIAP.

NVStore can be good for this purpose if you want to use a relatively small number of different keys (lets say 50), whose data size is also small (lets say <100), which don't get updated too often. Reason is that larger numbers would result in a lot of garbage collection, leading to sector erases (an operation that takes long time). On the good side, NVStore is probably the only mechanism that can work on just about any board, as all boards have internal Flash. All other boards have different storage mechanisms (SD card, external SPIF flash etc.), meaning that you will have hard time to find a uniform mechanism working for all of them. I'm not really familiar with the nrf52 devices. Will try looking at them when I'm back at work after the weekend.

most folks doing stuff with nrf52 end up using the sdk's fstorage code as far as i can tell, since there really wasn't an alternative. That's what the mbed Eddystone implementation  uses as well. It's not exactly a friendly API though.
The other mbed native way is setting up your own via FlashIAP, but that assumes you want to know a lot about the underlying storage topology. I was just hoping to find a method that all folks who use mbed and the BLE APIs could use to to store configuration data.

Does fstorage use the internal flash? If it does, then all issues I raised apply there as well. In this case, NVStore should definitely be a good replacement. Not to mention the FlashIAP driver (on top of which NVStore is implemented), which isn't supposed to be used for this purpose.
NVStore has one drawback though - its key management. While there are currently no features using NVStore yet, release 5.9 should introduce a few, each of which using its own NVStore key or key range. NVStore should have a way of reserving keys according to feature by then, but it is not implemented yet.

ARM Internal Ref: MBOTRIAGE-588


How about to register docbox on npm?

Docbox is awesome! How about to register it on npm so that we can integrate docbox on API services

👍 https://www.npmjs.com/package/docbox


Manual installation instructions not being clear enough

README.md:
Manual Installation

Install the addon loader and d3d wrapper.

Well, how do install those? Neither of those repos have instructions and there isn't any instructions here either.

addon loader:

put its zip's content (bin64, addonLoader.dll, d3d11.dll, dxgi.dll) in the gw2 installation directory

d3d wrapper

put its zip's content (folder named "d3d9_wrapper") under addons

As much as I understand your request, I really prefer referring to their respective Github pages for installation instructions so that if anything were to change, my own instructions wouldn't become outdated.
I'll see if I can get updated instructions posted on both addon repos.

Pull requests:

loader-core: https://github.com/gw2-addon-loader/loader-core/pull/5
d3d9_wrapper: https://github.com/gw2-addon-loader/d3d9_wrapper/pull/2

Thank you so much, all merged!

Thank you for your time on gw2radial!


[SPARK-40142][PYTHON][SQL][FOLLOW-UP] Make pyspark.sql.functions examples self-contained (part 5, ~28 functions)

What changes were proposed in this pull request?
It's part of the Pyspark docstrings improvement series (https://github.com/apache/spark/pull/37592, https://github.com/apache/spark/pull/37662, https://github.com/apache/spark/pull/37686)
In this PR I mainly covered missing parts in the docstrings adding some more examples where it needed.
Why are the changes needed?
To improve PySpark documentation
Does this PR introduce any user-facing change?
Yes, documentation
How was this patch tested?
PYTHON_EXECUTABLE=python3.9 ./dev/lint-python
./python/run-tests --testnames pyspark.sql.functions

Can one of the admins verify this patch?

@HyukjinKwon @srowen @itholic   please review

Merged to master.


[Question] How can I overrideWithProvider a StateProvider?

Hi guys, hope you are doing well :)
I've been working on an ID verification flow and I achieved it, but now I want to clean the code with the help of riverpod and I'm facing the following issue:
What I want
final fileProvider = StateProvider<XFile>((_) {
  throw UnimplementedError('XFile not provided.');
});

final frontFileProvider = StateProvider<XFile>((_) => null);

final backFileProvider = StateProvider<XFile>((_) => null);

final selfieFileProvider = StateProvider<XFile>((_) => null);

// ...

ProviderScope(
  overrides: [
    fileProvider.overrideWithProvider(frontFileProvider),
  ],
  child: const FrontPage(),
);

Why I want this approach
I want this approach because I have a front/back/videoSelfie page and a CameraShutter widget that reads the fileProvider and works with that information.
MAYBE I'm not understanding or covering the problem correctly, I am open to new ideas of course.
Please and thank you so much :)
The exception
Unsupported operation: Cannot override providers on a non-root ProviderContainer/ProviderScope

Use a family https://riverpod.dev/docs/concepts/modifiers/family
The argument to family will be an enum probably for (Front/Back/Selfie)

@TimWhiting thank you for the response. Sadly, I read the documentation and tried implementing it but I don't understand what the logic would be like. How does (for example) the CameraShutter know what enum is on top of it?

If you want to not have it auto dispose then just don't add the auto dispose modifier. (I don't think keeping up to 3 XFiles around for the duration of the app will take up much memory).  So for people who are facing the same problem I will leave the code that I'm using so that they understand the solution.
Thank you so much for the help, @TimWhiting 💙
Parent page
final kycMediaTypeProvider = ScopedProvider<KycMediaType>((_) {
  throw UnimplementedError('KycMediaType not provided.');
});

final fileProvider = StateProvider.family.autoDispose<XFile, KycMediaType>(
  (_, type) {
    switch (type) {
      case KycMediaType.front:
        return null;
      case KycMediaType.back:
        return null;
      case KycMediaType.videoSelfie:
        return null;
      default:
        throw UnsupportedError('$type not supported.');
    }
  },
);

// ...

Consumer(
  builder: (_, ScopedReader watch, __) {
    // Watching every KycMediaType to avoid unwanted early disposing because
    // there was no watch(fileProvider) in this file.
    for (final KycMediaType type in KycMediaType.values) {
      watch(fileProvider(type));
    }

    return PageView(
      controller: watch(pageControllerProvider),
      physics: const NeverScrollableScrollPhysics(),
      children: const [
        PreVerificationPage(),
        FrontPage(),
        BackPage(),
        SelfiePage(),
      ],
    );
  },
),

Each "children" page
ProviderScope(
  overrides: [
    // You can do this override dynamically if you need.
    kycMediaTypeProvider.overrideWithValue(KycMediaType.front),
  ],

// ...

Sorry but this doesn't really make sense to me? how's the solution even using the state provider? It just listens to it but there's no use of the value


Testing emitter from Channel

I'm trying to find a way to proper test this, I've been reading the docs and a lot of people seems to have a similar issue.

function emitterFn (instance) {
  return eventChannel(emit => {
    const listener = instance.changes()
     .on('change', info => {
      emit(info)
    })
    return () => listener.cancel()
  })
}

function* something () {
 const emitter = yield call(emitterFn)
   try {
    while (true) {
       let payload = yield take(emitter)
       /* code omitted */
      }
   }
}

What is the proper way to get the pattern/channel for that emitter?
   let output = saga.next({}).value
   let expected = take({})

I doesn't read the proper channel, I've also tried to instanciate the channel and pass it as an argument and it didn't work
     let channel = myChannelFunction()
     let output = saga.next(channel).value
     let expected = take(channel)

Any ideas?

this should pretty much work
const saga = something()
let actual = saga.next()
let expected = call(emitterFn)

const chan = channel()

actual = saga.next(chan).value
expected = take(chan)

@Andarist almost, the expectation is working good, however, for some reason the emitter here is undefined
function* something () {
 const emitter = yield call(emitterFn)
   try {
    while (true) {
       let payload = yield take(emitter) // this emitter is undefined on tests
       /* code omitted */
      }
   }
}

The code is fine and working perfectly, when I run it, it has a value.
Any idea?

@Andarist Almost, It still throws the take channelOrPattern is not valid, I checked a little and notice the emitter is undefined when running the tests, however, it works fine one the code:
function* something () {
 const emitter = yield call(emitterFn)
   try {
    while (true) {
       let payload = yield take(emitter) // <--- this is undefined
       /* code omitted */
      }
   }
}

That is causing the error when testing I guess, so I tried adding the emitter to the previous next() to make it available, but I guess the same result.
If it helps, my event channel is the same as #631

You're trying to test the emitterFn function in your example, correct? If that is the case, you should do that outside of a saga. You'll need to fake the instance that you pass into emitterFn, something like this FakeEmitter class that allows you to control when events are emitted and the emit function from the eventChannel is called. Looks like you're mock instance might be slightly more involved with properly mocking the behavior of the changes method, but pretty similar.
When you go to test the channel's closing behavior, create a spy on the cancel method of the instance you passed it, and assert that its called. i.e.
const cancel = jest.spyOn(emitter, 'cancel')
channel.close()
expect(cancel).toHaveBeenCalledWith(/*whatever arguments it should have been called with*/)

When you go to write a test for a saga that uses the channel creator function, you don't need to mess with channels, you just call next on the iterator with whatever you need. When you need to pass in a fake channel, you can do it with a couple of simple no-op functions.
Here's an example from one of my project's tests (using redux-saga-test-plan , which I heartily recommend)


Migrate to CircleCI 2.0 before August 31st

CircleCI 1.0 will be decommissioned on August 31st so we should make sure to migrate the configuration to 2.0 before that happens.

More dtails: https://circleci.com/blog/sunsetting-1-0/

Duplicate of #3288

ops, my bad


Preserve filename when resizing

Hi,
I've been using your app for years, and I've only one word: thank you!
I've a very simple suggestion: when creating a resized image, could the app name it using the original image (plus a suffix, if you need uniqueness)?
Currently, when an image is resized, the resized image displayed is always named "VT-Compress (tm) Xing Technology [...]", which is quite a pitty. I so loose the possibility to double-tap and open the image in order to get its name (name of the painting, date of the photo, etc.).
Would it be possible?

@OhSoGood the fact is that metadata exists for one reason. Resizer apps shouldn't touch the name metadata (they have other attributes like description or even custom ones iirc).


warning removal by initializing animationDuration

tiny thing, but the warning is bugging me

This is already fixed on the master branch.


Update site.gems references to site.plugins

Fixes checks to determine if jekyll-seo-tag and jekyll-feed are active.
Related to #24

Could these conditionals be updated to check for both gems and plugins to cover those who might haven't updated their _config.yml file?

Good thinking. All checks have been updated.

Looks good, thanks!


Add Dom.Storage.setItem and Dom.Storage2.setItem

Fixes #93

duplicate of #85, but happy to merge this is you add the same changes to the v8.0.0 docs with proper Reason syntax.

duplicate of #85, but happy to merge this is you add the same changes to the v8.0.0 docs with proper Reason syntax.

Done.

Sorry Dan, #85 by @bodhish was replying faster than expected, so I went ahead and merged his' first 😅

Okay will close this since the feature is already in master!


Create abstract contract for OnlySource authentication

Background
In the @connext/interfaces package we want to have an abstract contract that can be inherited by integrators to easily kick-start the implementation of an authenticated receiver.
Currently, examples assume there is only 1 origin sender and origin domain while it would be most useful to support any number of origins. There should probably be a mapping of origins and the registration of those origin values would be set in the constructor and/or through setter functions.
We need to think through this pattern and determine if prescribing a specific way to register origins is too limiting for integrators.
Non-authed template is being worked on in #3885 (https://github.com/connext/connext-integration/tree/main/contracts/.Template). See if the template pattern here can be extended for auth'd.

Impact: 3
See the discussion in Backlog Prio 1
Powered by Parabol

Urgency: 5
See the discussion in Backlog Prio 1
Powered by Parabol

Complexity: 13
See the discussion in Sprint Poker #22
Powered by Parabol

update: needs errors to be moved to named errors + other review comments

Updated, ready for re-review.

https://github.com/connext/connext-integration/pull/3

abstract-auth ready for rereview


resize only if original is bigger than requested style

Hello,
I would like to know if there is an easy way to resize an image only if it is bigger than the size defined in the requested style. If the image is smaller, we just do not touch it.
Thank you for your help
Regards,

any idea ? We're still stuck on this...
Thanks

I can't think of a way to do this (off the top of my head) using a plain style. I think that you could probably achieve something very similar to this by using a closure for your style and then inside the closure comparing it against the dimensions your using so that you can resize accordingly. What's your use case for this?

We use userside uploaded pictures for display in a quiz. The image need to be smaller than 600x400 for example.
If it is bigger, we resize and crop. If it is smaller, we would like to leave it as it is. (as of now, it is resized to 600x400 making small images unreadable)

@tabennett Do you have by any chance any example of such a closure ?
Thanks in advance for your help

Bottom of the page. You could alter the watermark example to get just grab the size of the image and then resize it to whatever you want if the dimensions are larger than 600x400.
The UploadedFile object and the ImagineImage instance will be injected in for you automatically.

@jbajou did you ever find a solution for this? I have the same requirement. I'm creating a "zoom" style for user uploaded images. The zoom style takes the original image and resizes it proportionally to a width of 1280px. However, if the original image is less than 1280px wide, I'd like to just leave the image as is, essentially making the zoom style size the same as the original.
Currently, it looks like Stapler will stretch the original image to the width of the zoom style, which is not desirable. It would be great if there was some way to build a flag into a style such as enlarge: false, or something like that, which tells the image processing that the original file should never be enlarged. It should either be left as is, or shrunk to a style.

@flyingL123 didn't get the time to work on that unfortunately.
It's in the pipe for this month though. I'll post the snippet here as soon as it's done.
In the meantime, if you are more quick than me on this, well, do the same! :)
It actually does feel kind of strange that Stapler doesn't offer such an option.

It looks like this logic is supported by ImageMagick, so I wonder if there is a way to use that.  From the docs, I it would be as simple as:
'300>'

I've adjusted app/config/packages/codesleeve/laravel-stapler/stapler.php to use Imagine\Imagick\Imagine as the image_processing_library. I then changed my style dimensions on the model to 'dimensions' => '1280x1280>', but it doesn't seem to be working. The image is still being enlarged.

@jbajou - Here's what's working for me. In my case I am only concerned with the width of the original image. If the original is greater than 1280px wide, I resize it it proportionally to have a width of 1280. If not, I leave the size unchanged.
'zoom' => function($file, $imagine) {
	$image = $imagine->open($file->getRealPath());

	// Auto rotate the image
	$filter = new \Imagine\Filter\Basic\Autorotate;
	$filter->apply($image);

	// Get the current size
	$size = $image->getSize();

	// Scale down to zoom size only if
	// image is wide enough.
	if ($size->getWidth() > 1280) {
		$newSize = $size->widen(1280);
		$image->resize($newSize);
	}

	return $image;
}

Hi @flyingL123,
I adapted your code to my needs (changed size, etc) and it works like a charm! Thanks a lot, and sorry for my late feedback.


WorkspaceConfigurationService - Folder remove setting from all targets

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=189401&view=logs&j=55ac390b-ffb1-50ea-3650-525dd9a3fd80&t=a0dd9636-50d2-5f8a-f17f-6bcaa254d802
The followings tests are failing:
 - WorkspaceConfigurationService - Folder remove setting from all targets (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)

I do not find any reason for this test to timeout unless there is a system issue.


NAS-125923 / 24.04 / Add root password disabled override info to debug

In some cases users may disable password-based authentication for root account on NAS without providing alternative administrative account. In this situation we need to perform password validation internally in middleware (and not go through PAM). Since this impacts authentication and authorization we should flag it in the debug.

Jira URL: https://ixsystems.atlassian.net/browse/NAS-125923

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.


rename to pacta.interactive

or should it be pacta.interactive.plot? @AlexAxthelm @jdhoffa @MonikaFu

I think I like pacta.interactive.plot


NUM_TRACKS fails under some circumstances

For some sets, soundscrape will download all tracks available rather than downloading the number of tracks passed via -n on the command line.
It seems to manifest when tracks need to be fetched via get_soundcloud_api_playlist_data() rather than via resolved.tracks. Resizing the array and then setting aggressive to True so it won't try downloading the files further down seems to have fixed the issue.

Looks like Travis failed, but I'm not sure if it's related. Can you see if the tests pass locally for you? Could be DDoS related..

It looks like 2 test have failed, even locally:
FAIL: test_mixcloud (test.TestSoundscrape)
Traceback (most recent call last):
File "/Users/atros/Documents/Projects/SoundScrape/tests/test.py", line 114, in test_mixcloud
self.assertTrue((new_mp3_count > mp3_count) or (new_m4a_count > m4a_count))
AssertionError: False is not true
-------------------- >> begin captured stdout << ---------------------
HTTPSConnectionPool(host='stream14.mixcloud.com', port=443): Max retries exceeded with url: /c/originals/3/9/f/4/f513-a72c-4741-9e4f-c021bb01058f.mp3?v=0.1 (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x10e7e1150>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))
--------------------- >> end captured stdout << ----------------------
FAIL: test_soundcloud_hard (test.TestSoundscrape)
Traceback (most recent call last):
File "/Users/atros/Documents/Projects/SoundScrape/tests/test.py", line 54, in test_soundcloud_hard
self.assertTrue(new_mp3_count == 1) # This used to be 3, but is now 'Not available in United States.'
AssertionError: False is not true
The first one seems to be related to a DNS issue, it looks like stream14.mixcloud.com doesn't resolve any longer. I tried it on different DNS servers and the outcome was the same.
As for the second, it looks like it's the same sort of issue I'm trying to fix but it's getting hit for other reasons. I've committed new changes to my fork that should address the second issue. As for the first one, I believe it the test needs to be updated.

@atr0s
soundscrape was updated, test_mixcloud should pass now.

I had to rerun the job a few times but works now

Thanks for your contributions!


Add a safeguard to exclude the controller node from targeted nodes wh…

…en enabled
What does this PR do?

[ ] Adds new functionality
[x] Alters existing functionality
[ ] Fixes a bug
[ ] Improves documentation or testing

Please briefly describe your changes as well as the motivation behind them:

The PR adds a safeguard that excludes the node running the controller from the list of targeted nodes with it is enabled so we do not apply a disruption to the controller node by mistake (which could lead to an expected behavior like not being able to delete the related disruption)
Safeguards are enabled by default, but disabled locally through make install

Code Quality Checklist

[x] The documentation is up to date.
[x] My code is sufficiently commented and passes continuous integration checks.
[x] I have signed my commit (see Contributing Docs).

Testing

[ ] I leveraged continuous integration testing

[ ] by depending on existing unit tests or end-to-end tests.
[ ] by adding new unit tests or end-to-end tests.

[x] I manually tested the following steps:

x
[x] locally.
[ ] as a canary deployment to a cluster.

Looks very suitable to me! This doesnt prevent targeting pods on the controller's node though, so we could still nodeFailure the controller, is that alright?

Good catch, we could handle that easily as well by ensuring that the targeted pod node name is not the controller node name (maybe only focused to node failures though?).


Dev

No conflicts with the master branch. Added pieces to assist with adding a student. Ready to merge.


Error when dragging constrained object on top of another interactive object

Version

Phaser Version: 3.54.0

Operating system: Windows

Browser: Tested on both Chrome and FIrefox

Description

Error when dragging an object outside the game area, if the object is constrained and the object is on top of another interactive object.
In my testing, this did not occur when the game object was not constrained, and it did not occur when it was not on top of another interactive object. I discovered this when trying to create a volume slider for my game, for which the slider knob was constrained to the slider area.
Although it does not crash the game, the error shows up on console.
Steps to reproduce (using test code below):

Click the green box and begin dragging it
Keep dragging until the mouse pointer is outside the game area (e.g. in the white area of the page background)
Release the mouse button
Console will show:

Uncaught TypeError: Cannot read property 'renderList' of undefined
    at initialize.sortGameObjects (phaser.min.js:1)
    at initialize.processOverOutEvents (phaser.min.js:1)
    at initialize.update (phaser.min.js:1)
    at initialize.updateInputPlugins (phaser.min.js:1)
    at initialize.onMouseUp (phaser.min.js:1)
    at onMouseUpWindow (phaser.min.js:1)

This appears to be because the camera associated with the pointer is undefined when the event takes place outside the game area.
Example Test Code
https://codepen.io/lyger-zero/pen/PopNKbM
const game = new Phaser.Game({
  parent: "phaser-root",
  width: 800,
  height: 600,
  scene: {
    create: create,
  }
})

function create() {
  const rect1 = this.add.rectangle(400, 300, 800, 600, 0xff0000).setInteractive();
  const rect2 = this.add.rectangle(400, 300, 200, 100, 0x00ff00).setInteractive();

  this.input.setDraggable(rect2);

  rect2.on("drag", function(pointer, dragX, dragY) {
    rect2.x = Phaser.Math.Clamp(dragX, 200, 600);
    rect2.y = Phaser.Math.Clamp(dragY, 100, 400);
  });
}

Additional Information

Fixed this by manually setting camera for the mousePointer on 'dragend' event.
this.world.on('dragend', (): void => { this.input.mousePointer.camera = this.cameras.main })

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.

This issue has been mentioned on Phaser. There might be relevant details there:
https://phaser.discourse.group/t/uncaught-typeerror-on-dragend-event/9753/8


Extreme error message in GEMS bar plot visualization

In GEMS1, with Country subset to only include African countries, chose a box plot and set up the main variable as "Dry mouth", then chose "Case or control participant" as the overlay variable and got this error message. Plot worked fine if I chose another categorical variable from the participant entity.

overlayVariableDetails shouldnt be an array here. if its relevant it appears there only exist data for cases here. transferring to plot.data.


add gitattributes file

ensure that line-endings are handled consistently for all users, and that no lf <-> crlf conversion is performed on pkl and fpb files.

I'm a bit puzzled that 10 commits have been selected, I believe those changes have been already merged by some earlier PRs. Please let me know in case I should try produce a cleaner set of changes.


df.cache() question

PySpark allows for giving an argument for caching type. How can we pass this in Koalas?
Spark
>>> df.persist(pyspark.StorageLevel.MEMORY_ONLY)
DataFrame[id: bigint, name: string]

>>> df.persist(pyspark.StorageLevel.DISK_ONLY)
DataFrame[id: bigint, name: string]

Happy to see you @Harshitg , Thanks for reporting an issue for Koalas :D
Since Koalas is supporting cache as a cache() of PySpark under-the-hood, we can't specify the storage level for now. (It uses MEMORY_ONLY by default)
https://github.com/databricks/koalas/blob/fd60bfed206a1aef3e57a08356536fad9a3d7281/databricks/koalas/frame.py#L10025-L10027
For specifying the storage level (as you said, the 'caching type'), we should change the cache() to persist() for adding proper parameter which can accept the storage level.
@HyukjinKwon @ueshin What do you think about changing our caching mechanism from cache() to persist() for that users can specify the storage level by parameter??

As PySpark has both cache() and persist(), we can just add persist() method the same as PySpark.
Btw, the default value is MEMORY_AND_DISK instead of MEMORY_ONLY.
Also it might be good if _CachedDataFrame has storage_level property for users to check the current storage level.
@itholic Do you want to submit the PR?

@ueshin

Btw, the default value is MEMORY_AND_DISK instead of MEMORY_ONLY.

Thanks for the comment and fixing the mistake!
Sure, I'll add the DataFrame.persist() soon.
@Harshitg I'll try to make the DataFrame.persist() method until next release so that you can use the caching type after then. 😸

@itholic Thanks for the prompt response.

@Harshitg My pleasure :D


Updated R53 ID's to match what AWS do now

#1044 - I thought it too would confuse people for a second if it doesn't look like what they're used to.

Coverage increased (+0.001%) to 93.593% when pulling 17d9701d19a3713a5ac553543366ff1d85ef4146 on terrycain:route53_id_fix into d69221992758e8fcced467c7b1c7cbd5f5a2d671 on spulec:master.

Coverage increased (+0.001%) to 93.593% when pulling 17d9701d19a3713a5ac553543366ff1d85ef4146 on terrycain:route53_id_fix into d69221992758e8fcced467c7b1c7cbd5f5a2d671 on spulec:master.

Coverage increased (+0.001%) to 93.593% when pulling 17d9701d19a3713a5ac553543366ff1d85ef4146 on terrycain:route53_id_fix into d69221992758e8fcced467c7b1c7cbd5f5a2d671 on spulec:master.

This is a really great idea

This is now released as Moto 1.1.14


Added fullscreen and unfullscreen button.

Added icons.
Updated activity.css and activity.js
Updated index.html

That's nice.
BTW I think language and font buttons should be hidden in fullscreen mode because it's override by the unfullscreen button.

Can I adjust the button to the left...
It will be very handy if we include the language and fonts button.. What are your thoughts.I will do as you say.

It don't work when doing fullscreen on the home page or on the exercise page
There is an error in the console
activity.js:51 Uncaught TypeError: Cannot read property 'style' of null
    at HTMLButtonElement.<anonymous> (activity.js:51)

Converted my methods into four and added them in util.js

Good job. It's perfect now.


Draw divider line down the middle

IIRC, the original pong had a dashed line running down the middle of the game screen.

For reference, the orignal version of Pong: https://upload.wikimedia.org/wikipedia/commons/f/f8/Pong.png

Closing as this has been resolved.


Organization of the Op-Design Repository(Designfree week)-2

-Organization of GitHub OctopOS Prime Design repository.
-Organizing the 2D and 3D visuals of our Octopos Prime project.
-Indicating what are the tasks performed.

The 2D visuals I made for our Octopos Prime project have been edited and uploaded to github with Illustrator files.


Add .editorconfig file

Editorconfig is a pretty nifty plugin. It exists for most editors and IDEs and synchronizes everybody's settings with regards to tabs vs spaces, the style of newline, and whether there's a newline at the end of the file among others.
The idea is that it'll make sure there aren't noisy whitespace changes in PRs, and settle the tabs vs spaces debate, at least in our little corner of the internet.

sounds good to me


Start and stop Appium server automaticlly

We need to add in MobileCapabilities.properties the ability to start and stop the Appium server programmer to move forward to be a standalone framework and just depend on the emulator app only as an external support app

That's perfectly doable... Would you like to own the implementation?

Let's try to to do it
Thanks
Muhammad L. Sultan
CTAL-TM®, PMP®, SFPC®
Head Of Software Quality
Mobile: +20<PHONE_NUMBER>
Email: @.***
Address: Building 7H, Street 263 New Maadi, Cairo, Egypt
Website: www.paymob.com

From: Mohab Mohie @.>
Sent: Sunday, October 23, 2022 11:41:37 PM
To: ShaftHQ/SHAFT_ENGINE @.>
Cc: Muhammad Sultan @.>; Author @.>
Subject: Re: [ShaftHQ/SHAFT_ENGINE] Start and stop Appium server automaticlly (Issue #744)
That's perfectly doable... Would you like to own the implementation?
—
Reply to this email directly, view it on GitHubhttps://eur06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FShaftHQ%2FSHAFT_ENGINE%2Fissues%2F744%23issuecomment-1288207442&data=05|01|muhammad.sultan%40paymob.com|63fc8e5c80c9447dbaf408dab53f5d93|96bf202797ea4dfc8f5b7121b770f292|0|0|638021580993968096|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=vFWofcruPp0EURJXHEw82BrWJSHzQh0l1UTz5UTyF1s%3D&reserved=0, or unsubscribehttps://eur06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA2B3ZUAPH22XKZODOZHFFN3WEWWJDANCNFSM6AAAAAARL2II74&data=05|01|muhammad.sultan%40paymob.com|63fc8e5c80c9447dbaf408dab53f5d93|96bf202797ea4dfc8f5b7121b770f292|0|0|638021580993968096|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=evhGVaALj8y%2BQNfxufzcOGD1FGWTpQOy%2Bi%2F2dxupvag%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>
The contents of this e-mail message and any attachments are confidential and are intended solely for the addressee. It is strictly forbidden to share any part of the information included in this email or its attachments with any third party, without a written consent of the sender. The information may also be legally privileged. This transmission is sent in trust, for the sole purpose of delivery to the intended recipient. If you have received this transmission in error, any use, reproduction or dissemination of this transmission is strictly prohibited. If you are not the intended recipient, please immediately notify the sender by reply e-mail or phone and delete this message and its attachments, if any.

Ps. I encourage you to open a PR and add paymob to the list of SHAFT users

For sure, it's a pleasure to that
Thanks
Muhammad L. Sultan
CTAL-TM®, PMP®, SFPC®
Head Of Software Quality
Mobile: +20<PHONE_NUMBER>
Email: @.***
Address: Building 7H, Street 263 New Maadi, Cairo, Egypt
Website: www.paymob.com

From: Mohab Mohie @.>
Sent: Sunday, October 23, 2022 11:43:30 PM
To: ShaftHQ/SHAFT_ENGINE @.>
Cc: Muhammad Sultan @.>; Author @.>
Subject: Re: [ShaftHQ/SHAFT_ENGINE] Start and stop Appium server automaticlly (Issue #744)
Ps. I encourage you to open a PR and add paymob to the list of SHAFT users
—
Reply to this email directly, view it on GitHubhttps://eur06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FShaftHQ%2FSHAFT_ENGINE%2Fissues%2F744%23issuecomment-1288207723&data=05|01|muhammad.sultan%40paymob.com|c2030fd302914ed52e8c08dab53fa0ec|96bf202797ea4dfc8f5b7121b770f292|0|0|638021582121067297|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4IpHbBHtvbYQjXOquIFmbhFpyqUQ54uu23QU1W7wAnw%3D&reserved=0, or unsubscribehttps://eur06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA2B3ZUGSV5JKW55ATHSHME3WEWWQFANCNFSM6AAAAAARL2II74&data=05|01|muhammad.sultan%40paymob.com|c2030fd302914ed52e8c08dab53fa0ec|96bf202797ea4dfc8f5b7121b770f292|0|0|638021582121067297|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=k8TN%2BCxzsI%2BWkwxZYYc288xCOtTtVZ3rzyRJ%2BFT3udY%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>
The contents of this e-mail message and any attachments are confidential and are intended solely for the addressee. It is strictly forbidden to share any part of the information included in this email or its attachments with any third party, without a written consent of the sender. The information may also be legally privileged. This transmission is sent in trust, for the sole purpose of delivery to the intended recipient. If you have received this transmission in error, any use, reproduction or dissemination of this transmission is strictly prohibited. If you are not the intended recipient, please immediately notify the sender by reply e-mail or phone and delete this message and its attachments, if any.


Don't use explicit named setters/getters in BaseClass.

All tests pass.

Not really measurable, but it saves around ~20 function calls per layer creation, so hey!

🎉


Use unique IDs for every tab, and update the related actions to take …

…a tab ID rather than an index. r=bgrins
Closes #202

Coverage increased (+0.6%) to 39.717% when pulling 75cafa93f453b7b2613e42effd4f3e56106fa784 on jsantell:uuid-tabs into b7fda0d59a92c6dedcb69b13386be71385169e60 on mozilla:master.

Coverage increased (+0.6%) to 39.717% when pulling 0ba8af5afa511f19ed330c50ae2288c4b11a0ecb on jsantell:uuid-tabs into f8aa2397bde5be44c2b0ddbd542b8d1d21ebb92f on mozilla:master.


Test coverage, typescript and tsx

Hey, I'm trying the new code coverage feature, first of all, thank you for the work.
It doesn't work for me, I don't know if I'm missing some configuration. Currently using this configuration:
{
  "nodejs-testing.extensions": [
    {
      "extensions": ["mjs", "cjs", "js"],
      "parameters": []
    },
    {
      "extensions": ["mts", "cts", "ts"],
      "parameters": ["--import", "tsx"]
    }
  ]
}

Everything works well except test coverage, I only have this that appears:

And everthing shows up as 100%:

So it looks like something does not work. When I run the following command by hand:
node --test --enable-source-maps --experimental-test-coverage --import tsx $(find src -name '*.test.ts')

I obtain:

So the node test runner seems to recognise my files.
Are you aware of such a thing? Can I help you with identifying the root cause?
Thank you for your time and have a nice day

We just toss everything into the c8 module and let it work, but it won't have had access to scripts from the runtime.
I plan to rework coverage shortly with the addition of attributable coverage in VS Code (based on https://github.com/microsoft/vscode/pull/212325) which does not have this issue.

I admit I don't understand quite well all the mechanisms, but I guess I'll be waiting that this gets implemented! Thanks for taking the time to respond :) I'll try to take some time to explore how c8 works exactly and how it relates to this project

Hey there @coderhammer, it seems to work just fine for me.
package.json
  "engines": {
    "node": "20.x"
  },
  "main": "dist/src/functions/index.js",
  "scripts": {
    "build": "tsc --project tsconfig.json && tscpaths -p tsconfig.json -s ./ -o ./dist",
    "watch": "tsc -w",
    "clean": "rimraf dist",
    "prestart": "npm run clean && npm run build",
    "start": "func start",
    "pretest": "npm run clean && npm run build",
    "test": "node ./dist/__tests__",
    "test:debug": "npm run pretest && node --inspect-brk ./dist/__tests__",
    "test:html": "nyc --reporter=html node dist/__tests__ && start coverage/index.html",
    "posttest": "nyc --reporter=lcov --reporter=text node dist/__tests__"
  },

settings.json (user settings)
  "nodejs-testing.extensions": [
    {
      "extensions": ["mjs", "cjs", "js"],
      "parameters": []
    },
    {
      "extensions": ["mts", "cts", "ts"],
      "parameters": ["--import", "tsx"]
    }
  ],
  "nodejs-testing.nodejsPath": "C:\\Program Files\\nodejs\\node.exe",

tsconfig.json
{
  "$schema": "https://json.schemastore.org/tsconfig",
  "display": "Node 20",
  "include": ["src/**/*.ts", "__tests__/**/*.ts"],
  "exclude": ["dist", "node_modules/*"],

  "compilerOptions": {
    "lib": ["es2022"],
    "module": "commonjs",
    "target": "es6",
    "outDir": "dist",
    "rootDir": ".",
    "sourceMap": true,

    "strict": true,
    "esModuleInterop": true,
    "resolveJsonModule": true,
    "skipLibCheck": true,
    "forceConsistentCasingInFileNames": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "strictPropertyInitialization": false,

    "baseUrl": "./src",
    "paths": {
      "Models": ["models/index"],
      "Container": ["container/index"],
      "Availability": ["modules/availability/index"],
      "Agenda": ["modules/agenda/index"],
      "Shared": ["modules/shared/index"],
      "Utils": ["utils/index"],
      "Errors": ["errors/index"],
      "Authentication": ["authentication/index"],
      "Middlewares": ["middlewares/index"],
      "Functions": ["functions/index"]
    }
  }
}


Multi users

I think get_max_short is ok as an initial pass. We need to think of a fast approximation to iteratively solving it, and probably include an optional precise iterative solution.


Dialog box and text button components + Link guide dialog box

Changes

Implement a Dialog box component (custom made, tried material-ui's but it had a delta +22.5 kB)
Implemented a Text Button component (no background behind it)
Refactored the Modal component to be re-used with the Dialog component
Added subtle transitions when opening/closing the Modal/Dialog
Refactored part of the GridView to allow for tapping outside to close the Modal

Wassim - let's change the OK to  GOT IT and not use BACK. Two reasons: !1) back is generally a navigation action and those are usually shown on the top. "Got it"  or "OK, Got it" should supply enough clicking space. 2) I used material buttons. Isn't someone building a material button component? Can you make sure all the buttons behave the same way

Rebasing this one should also fix the Travis CI hiccups btw.


WIP: Trying to extract 2020 data from EPACEMS

Using pudl_datastore to download EIA923 2019 data, EIA860 2019 data, EIA860m 2020 data, and EPACEMS 2020 data.
This seems to work, but my current issue is that I put in a hack to "fix" any missing UTC offsets. There must be a better solution but I currently am not sure what it should be.
Running the following etl yaml config:
# This file controls the PUDL ETL process, and is used as input to pudl_etl

# You can create several different settings files if you wish, and specify
# which one the ETL script uses on the command line. e.g.:
#
# pudl_etl custom_pudl_etl.yml

# A "package bundle" is a collection of PUDL data packages that were created
# together in the same ETL process. They should be mutually compatible, and
# can be used together to create a database, or other products. Each package
# bundle needs a name, which will also be the name of the directory into which
# it is written. All the tabular data resources / data packages within a
# package bundle share a unique ID that we use to ensure only compatible data
# is combined.
datapkg_bundle_name: 2020-etl

# The package bundle settings are a list of individual data package
# specifications, each of which may contain one or more data sources.
datapkg_bundle_settings:
  ###########################################################################
  # EPA CEMS HOURLY SETTINGS
  ###########################################################################
  # The EPA CEMS data goes back as far as 1995, but before 2000 it is not as
  # complete.  Note that the EPA CEMS data set is much larger than any of the
  # other data sets here.  Pulling in all the years of data for all of the
  # states requires ~100GB of free disk space and takes around 8 hours on a
  # reasonably fast laptop.
  - name: epacems-eia-example
    title: EPA Continuous Emissions Monitoring System Hourly
    description: Hourly emissions, power output, heat rates, and other data for most US fossil fuel plants.
    datasets:
      - eia:
          # This is the full list of EIA 923 tables.  Many of them are
          # interdependent, and are used in the definition of the overall
          # database, so it is recommended that you import either all of them
          # or none of them. Additionally, there are many relationships between
          # the EIA 923 and EIA 860 tables, and in general they should be
          # imported together.
          eia923_tables:
            - generation_fuel_eia923
            - boiler_fuel_eia923
            - generation_eia923
            # NOTE: coalmine_eia923 REQUIRES fuel_receipts_costs_eia923
            - coalmine_eia923
            - fuel_receipts_costs_eia923
          #eia923_years: [2009, 2010, 2011, 2012, 2013, 2014, 2015, 2016, 2017,
          #               2018, 2019]
          eia923_years: [2019]

          # See notes above about the entanglement between EIA 923 and EIA 860.
          # It's best to load all the tables from both of them, or neither of
          # them.
          eia860_tables:
            - boiler_generator_assn_eia860
            - utilities_eia860
            - plants_eia860
            - generators_eia860
            - ownership_eia860
          #eia860_years: [2011, 2012, 2013, 2014, 2015, 2016, 2017, 2018, 2019]
          eia860_years: [2019]
          # If you want to add the year-to-date data from EIA 860M, eia860_ytd
          # must be True. This will only add generator tables! You cannot load
          # just the eia860_ytd and no other years of eia data.
          eia860_ytd: True
      - epacems:
          # Note that the CEMS data relies on EIA 860 data for plant locations,
          # so if you're loading CEMS data for a particular year, you should
          # also load the EIA 860 data for that year (2011-2017 only)
          #epacems_years: [1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002,
          #                2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010,
          #                2011, 2012, 2013, 2014, 2015, 2016, 2017, 2018, 2019]
          epacems_years: [2020]
          # Just Idaho, as it is has the smallest data of any state.
          epacems_states: [ALL]
          # This will load all 50 states -- it's a lot of data!
          #epacems_states: [ALL]

PR #1078 should bring all the 2020 CEMS data in so I'm going to close this. Do let us know if you run into any problems with the 2020 data though.


Failed connecting to proxy

Hello, i keep getting theses errors:
"failed connecting to proxy" or "proxy is ratelimited"
Do you guys know how i could get past this error, or do you guys know a working proxy ?
Is a proxy necessary ? since none of the one i'm trying seems to work do i really need a proxy ? or can't i remove the "ratelimited" error ?

A proxy is used to change your IP and bypass rate limit protections.
Your proxy getting rate limited means that IP has a rate limit on the discord api


after update to 122 some website loged out after browser restart

last 121 tested normal,
with [disable machine id] and [disable encryption] enabled,
some website can't stay login after browser restart,
for example reddit twitter youtube,
while 9gag seems working fine,
clear cookie on exit disabled,
tested third party cookie settings has no effect,

is this fixed?
just tested with 122.0.6261.128
same behavior

still not fixed in version123


Rename and cleanup.

Merges 3d_mapping.lua into backpack_3d.lua. Renames launch files to be
consistent. Fixes a typo.

LGTM


Fix $output so [class*="col-"]:nth-child(1n + 1) doesn't clear: both

$output needs more work than this. Will open new issue.


fix typo in Security/README.md

Issue #, if available:
Description of changes:
fixed typo "immediately"
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

this change was included in the other pull request


A11y improvements

Test Me!
Test Gist

Found out about HTMLSlotElement.assignedNodes, which returns the slotted or default content correctly

this fixes the bug of different myuw-helps sharing focusable elements

Flatter conditional structure
Consistent open and close logic, whether triggered through attribute, event, button click, or new open()/close()/toggle() methods

this fixes the bug where the open modal is positioned way off-screen to the right when setting the open attribute

Add/remove event listeners correctly, to prevent duplicates or memory leaks
:new: Added ArrowUp and ArrowDown navigation, because the default content looks like a menu to me… do you think this is a good idea?

If you like this change as-is and choose to merge it, I recommend that you squash again to keep the revision log clean and concise.


OVA: Execute OVA server provider check only when the provider is not being deleted

If the OVA provider server fails to be created for any reason, we keep hitting the timeout check in the reconcile and return from there, so we do not reach the cleanup. To solve this issue, a check was added for the provider query to perform only when the provider has not started the deletion process.

@bkhizgiy thanks for making the previous changes per our offline conversation, I don't care much about the order of the blocks, since they should be mutual exclusive - we should ensure they really are mutual exclusive and by doing that we can minimize the changes here

looks good, I'd suggest to separate out the refactoring to a different PR - we should minimize the changes for a backport

@bkhizgiy and also please update the title and the description of the PR (and commits)

backport to v2.6  https://github.com/kubev2v/forklift/pull/835

@bkhizgiy please squash the two commits and update the title and description


SSR and DSG not working

Steps to reproduce:

Create a project from official starter https://github.com/gatsbyjs/gatsby-starter-rendering-modes
npx gatsby new gatsby-starter-rendering-modes https://github.com/gatsbyjs/gatsby-starter-rendering-modes
add 'gatsby-plugin-netlify'
yarn add -D gatsby-plugin-netlify
add plugin to gatsby

// gatsby-config.js
module.exports = {
    plugins: ['gatsby-plugin-netlify'],
}

push code to github
deploy to netlify

package.json
{
  "name": "gatsby-starter-rendering-modes",
  "private": true,
  "version": "1.0.0",
  "description": "A Gatsby starter to demonstrate new rendering modes in Gatsby 4",
  "main": "index.js",
  "scripts": {
    "start": "gatsby develop",
    "build": "gatsby build",
    "serve": "gatsby serve",
    "clean": "gatsby clean"
  },
  "author": "Brent Jackson<EMAIL_ADDRESS>  "license": "0BSD",
  "dependencies": {
    "gatsby": "^4.0.2",
    "node-fetch": "^2.6.2",
    "react": "^17.0.2",
    "react-dom": "^17.0.2"
  },
  "devDependencies": {
    "gatsby-plugin-netlify": "^4.0.0"
  }
}

Result:
https://awesome-liskov-661da1.netlify.app/

https://awesome-liskov-661da1.netlify.app/dsg
https://awesome-liskov-661da1.netlify.app/ssr

I had the same problem.
Then I have added Gatsby essential plugin manually through the Netlify UI and I am getting the error:
A Netlify Function failed to require one of its dependencies.
11:06:57 AM:   Please make sure it is present in the site's top-level "package.json"

Cannot find module '../../../.cache/functions/manifest.json' from '/opt/build/repo/.netlify/functions-internal/__api

And my build is failing. not sure it was a step forward to the solution or just a dead-end though.

Closing as the solution is to upgrade to the latest version of the Essential Gatsby Plugin.


browser.setLocation() not working in browser, browser.get() not working on device with cordova and appium

Hi there!
Bug report

Node Version: 6.9.4
Protractor Version: 5.0.0
Angular Version: 1.5.0
Browser(s): chrome and cordova app on Android Emulator, real device or iOS simulator and real devices
Operating System and Version MacOS Sierra
Your protractor configuration file (for browser test)

'use strict';

// An example configuration file.
exports.config = {
  capabilities: {
    'browserName': 'chrome'
  },

  baseUrl: 'http://localhost:3000',
  specs: ['test/protractor/**/*.js'],
};

Your protractor configuration file (for hybrid app test)
(connected using appium and clear instructions here)

'use strict';

exports.config = {
  seleniumAddress: 'http://localhost:4723/wd/hub',
  capabilities: {
    app: '/full/path/to/app/file.app',
    autoWebview: true,
    browserName: '',
    platformName: 'iOS',
    xcodeConfigFile<EMAIL_ADDRESS>    automationName: 'XCUITest', // use with xcode8
    platformVersion: '', // include version of available simulator os
    udid: 'b4adddddda76e9d4e69be5f6b19c7dfc11544224',
    deviceName: 'iPad ABC', // to start on simulator: iPad 2; to get udid from real device, type `instruments -s devices`
  },

  baseUrl: 'http://localhost:3000',

  specs: ['./test/protractor/**/*.js'],

  onPrepare: function () {
    var wd = require('wd'),
      protractor = require('protractor'),
      wdBridge = require('wd-bridge')(protractor, wd);
    wdBridge.initFromProtractor(exports.config);
  }
};

A relevant example test

'use strict';

describe('Debug page', function () {

    // console.log('window', window);
    // window.location.hash('http://localhost:3000/#/main/list');

    // works on device and browser
    // browser.get('file:///var/containers/Bundle/Application/A4B07338-8600-48F0-ADEF-6D0983BD369A/My%20Project.app/www/index.html#/main/debug');

    // works only in browser
    // browser.get('/#/main/debug');
    // browser.get('http://localhost:8100/#/main/debug');

    // works only on device
    // browser.setLocation('/main/debug');
  });

  it('should grade passwords', function () {

    var passwordInput = element(by.model('ctrl.password.input'));
    var passwordStrength = element(by.binding('ctrl.password.strength'));

    // weak
    passwordInput.sendKeys('my');
    expect(passwordStrength.getInnerHtml()).toEqual('weak');
    expect(passwordStrength.getAttribute('class')).toContain('badge-assertive');

    // medium
    passwordInput.sendKeys('test');
    expect(passwordStrength.getInnerHtml()).toEqual('medium');
    expect(passwordStrength.getAttribute('class')).toContain('badge-energized');

    // strong
    passwordInput.sendKeys('tesyasdft');
    expect(passwordStrength.getInnerHtml()).toEqual('strong');
    expect(passwordStrength.getAttribute('class')).toContain('badge-balanced');

  });
});

Output from running the test

using browser.setLocation in browser, I get window.angular is not defined.
using browser.get in hybrid app doesn't do anything

The URL you are running your tests against (if relevant)

You can try to install app yourself like this:

clone https://github.com/mwaylabs/generator-m-ionic-demo
npm i && bower i
gulp --cordova 'prepare android'
gulp --cordova 'build android'
run appium
run protractor CONFIG_ABOVE

My question is... how can I achieve running browser.get() without adding full path on hybrid app or at least so, that my test would ALWAYS work, no matter if testing in chrome or through hybrid app.

To answer myself, and ask the community if there's any other non-hacky way:
adding protractor part in config:
  onPrepare: function () {
    var wd = require('wd'),
      protractor = require('protractor'),
      wdBridge = require('wd-bridge')(protractor, wd);
    wdBridge.initFromProtractor(exports.config);

    //To navigate using file:// rather than http://
    var defer = protractor.promise.defer();

    /*globals browser*/
    browser.ignoreSynchronization = true;

    browser.executeScript('return window.location;').then(function (location) {
      browser.resetUrl = 'file://';
      browser.baseUrl = location.origin + location.pathname;
      defer.fulfill();
    });

    return defer.promise;
  }

fixes that, so that browser.get('#/main/debug'); works. The workaround/fix was found on next website: http://flashquartermaster.com/2016/08/02/automated-cross-platform-testing-of-ios-and-android-hybrid-ionic-cordova-angular-apps-using-cucumber-protractor-chai-and-appium/

So to be clear, the problem here is that you've got a different domain origin on different browsers?  Personally, I'd set browser.baseUrl in config.onPrepare.  We don't answer support questions here, and this isn't really a bug, so I'll mark it as a feature request.  But again, I think the best workaround is to set browser.baseUrl in config.onPrepare.  I'll even document it 😄

Thank you for your reply @sjelin. I'd thought this was some kind of issue since I've had to constantly use different approach in test itself, to achieve the same result.
Overwriting baseUrl sounds like falsy way-to-go.
But then again, I'm probably wrong. However, there's 0 docs about this topic that's why I've posted this as issue.
Thank you for updating the docs and replying.
👍

@sjelin  What is the configuration then for  in windows OS, on Android Physical Actual Device connected via USB to Windows machine, appium and protractor combination.  The objective here is to Open a chrome browser on the Android Actual Device and go to Angular website.
I keep getting errors where in one time chrome browser on the android device opens if i use appium desktop, but the browser remains at data and hangs
If i kill appium desktop and use cmd and type this command of webdriver-manager start --avd none --appium
then it says error code of  E/launcher - Process exited with error code 199 and in trouble shoot mode it says baseurl was not given.  I am Not testing an App.  I am only testing angular website url on chrome browser on actual android device connected to windows machine via USB cable
My conf,js code is below
exports.config = {
// directConnect: true,
//seleniumAddress: 'http://localhost:4733/wd/hub', //appium
seleniumAddress: 'http://localhost:4723/wd/hub', //appium
specs: [
'lmsportalexampleTC3_spec.js'
],
 capabilities: {

  'browserName': 'chrome',
 //appium-version: '1.6',

 'deviceName': '0715f7905cae3234',
 //deviceName: 'Android Device',
 'platformName': 'android',
// udid: '0715f7905cae3234',
 //platformVersion: '4.4',
 'platformVersion': '7.0',

 //'browserName': 'Android',
 autoWebview: true,
 autoWebviewTimeout : 10000,
 newCommandTimeout: 300000

},
//directConnect: true,
  // baseUrl: (

// baseUrl: 'http://www.google.com/'
//),
onPrepare: function () {
var wd = require('wd'),
protractor = require('protractor'),
wdBridge = require('wd-bridge')(protractor, wd);
wdBridge.initFromProtractor(exports.config);
},
//ignoreUncaughtExceptions: true,
// configuring wd in onPrepare
// wdBridge helps to bridge wd driver with other selenium clients
// See https://github.com/sebv/wd-bridge/blob/master/README.md
// configuring wd in onPrepare
// wdBridge helps to bridge wd driver with other selenium clients
// See https://github.com/sebv/wd-bridge/blob/master/README.md
// onPrepare: function () {
//	require('webdriver-js-extender').patch(
//  require('selenium-webdriver/lib/command'),
//  require('selenium-webdriver/executors'),
//  require('selenium-webdriver/http'));
//browser.get("http://www.thetestroom.com/jswebapp/");
//  var extendedWebdriver = require('webdriver-js-extender').extend(webdriver);
//   extendedWebdriver.setNetworkConnection(5);
// },
//const { SpecReporter } = require('jasmine-spec-reporter');
//  specs: {

//  'lmsportalexampleTC3_spec.js'
//  },
		//cap.setCapability(MobileCapabilityType.DEVICE_NAME, "My Actual Android Samsung EJ Phone");
	//cap.setCapability(MobileCapabilityType.DEVICE_NAME, "My Phone"); // Actual Device So, We are using EJ Samsung Phone due to emulator error on EJ machine
	//cap.setCapability("udid", "0715f7905cae3234"); // These steps are required for EJ Samsung Phone
	//cap.setCapability("platformName", "Android");
	//cap.setCapability("platformVersion", "7.0");

// baseUrl: 'http://yourwebsite.com'),
useAllAngular2AppRoots: true,

framework: 'jasmine2',

jasmineNodeOpts: {

//showColors: true,

defaultTimeoutInterval: 30000
},
};
====================================================
TestCase1spec.js code is below
describe ("Text Site for input field on home page", function() {
it ("To Test input field and check text output", function() {

  //browser.get("http://www.thetestroom.com/jswebapp");

    browser.driver.get("http://www.thetestroom.com/jswebapp");

	        //browser.get("http://www.google.com");

      //extendedWebdriver.get("http://www.thetestroom.com/jswebapp/");

     //browser.get("http://www.thetestroom.com/jswebapp/");

	element(by.model("person.name")).sendKeys("Test LMS Portal Text");

	element(by.binding("person.name")).getText().then(function(text){
		console.log(text);
			});

});

});
cmd output is here
[10:53:27] D/launcher - Running with --troubleshoot
[10:53:27] D/launcher - Protractor version: 5.3.0
[10:53:27] D/launcher - Your base url for tests is undefined
(node:13376) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
[10:53:27] I/launcher - Running 1 instances of WebDriver
[10:53:27] I/hosted - Using the selenium server at http://localhost:4723/wd/hub
[10:53:28] E/launcher - An unknown server-side error occurred while processing the command. Original error: Cannot read property 'forceStop' of undefined
[10:53:28] E/launcher - WebDriverError: An unknown server-side error occurred while processing the command. Original error: Cannot read property 'forceStop' of undefined
at Object.checkLegacyResponse (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\lib\error.js:546:15)
at parseHttpResponse (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\lib\http.js:509:13)
at doSend.then.response (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\lib\http.js:441:30)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
From: Task: WebDriver.createSession()
at Function.createSession (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\lib\webdriver.js:769:24)
at Function.createSession (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\chrome.js:761:15)
at createDriver (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\index.js:170:33)
at Builder.build (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\selenium-webdriver\index.js:626:16)
at Hosted.getNewDriver (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\built\driverProviders\driverProvider.js:53:33)
at Runner.createBrowser (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\built\runner.js:195:43)
at q.then.then (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\built\runner.js:339:29)
at _fulfilled (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\q\q.js:834:54)
at self.promiseDispatch.done (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\q\q.js:863:30)
at Promise.promise.promiseDispatch (C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\node_modules\q\q.js:796:13)
[10:53:28] E/launcher - Process exited with error code 199
C:\Users\UserName\AppData\Roaming\npm\node_modules\protractor\example>
Please help me. It has been 4 days with this issue and it is really sad for QA.
Thanks,
EJ