Error when running Cognito auth app created from tutorial

Describe the bug
When clicking 'Sign In' on an app I created from the AWS tutorial at the following location: https://aws.amazon.com/getting-started/hands-on/build-ios-app-amplify/module-three/
On the third step, I complete as per the tutorial and launch my app. The sign in button appears as expected. When I click on it, I get an error which states the following:
Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value
The line of code it is marked against is as follows:
let hostURL = "https://\(webDomain!)"
Webdomain appears to be a variable that is assigned a value (or not in this case) as follows:
let webDomain = infoDictionary?["WebDomain"] as? String
The code seems to be in the file AWSMobileClient.swift.
To Reproduce
Steps to reproduce the behavior:
Create the app using Xcode 12.4 in the tutorial here:
https://aws.amazon.com/getting-started/hands-on/build-ios-app-amplify/
When creating the app, select IOS, APP (Single view app is not available), Interface:SwiftUI, LifeCycle: UIKit App Delegate, Language Swift.
Expected behavior
A login interface to appear
Screenshots

Hi, @Fuerteventura2022
Thanks for reaching out.
There are few things I wonder:

I wonder why single view app selection is not available.
Can you check if the amplifyconfiguration.json has the necessary information to configure Amplify? I assume you are using signInWithWebUI since you were following the tutorial. The json file should look like this:

{
    "UserAgent": "aws-amplify-cli/2.0",
    "Version": "1.0",
    "auth": {
        "plugins": {
            "awsCognitoAuthPlugin": {
                "UserAgent": "aws-amplify/cli",
                "Version": "0.1.0",
                "IdentityManager": {
                    "Default": {}
                },
                "CredentialsProvider": {
                    "CognitoIdentity": {
                        "Default": {
                            "PoolId": "us-west-2:****",
                            "Region": "us-west-2"
                        }
                    }
                },
                "CognitoUserPool": {
                    "Default": {
                        "PoolId": "us-west-2_****",
                        "AppClientId": "****",
                        "AppClientSecret": "****",
                        "Region": "us-west-2"
                    }
                },
                "Auth": {
                    "Default": {
                        "OAuth": {
                            "WebDomain": "****.auth.us-west-2.amazoncognito.com",
                            "AppClientId": "****",
                            "AppClientSecret": "****",
                            "SignInRedirectURI": "myapp://",
                            "SignOutRedirectURI": "myapp://",
                            "Scopes": [
                                "phone",
                                "email",
                                "openid",
                                "profile",
                                "aws.cognito.signin.user.admin"
                            ]
                        },
                        "authenticationFlowType": "USER_SRP_AUTH"
                    }
                }
            }
        }
    }
}

My suggestion
The tutorial you followed is a bit out-dated and there are few steps not accurate. You can follow this doc which we keep it up-to-date regularly. Feel free to reach out if it doesn't work

@ruiguoamz thank you so much for your help. I've got it to say YourApp wants to use Amazon Cognito to sign in and then it says page couldn't be found. I'm pretty sure that's because I don't know the correct details for the following:
"Auth": {                             "WebDomain": "****.auth.us-west-2.amazoncognito.com",                             "AppClientId": "****",                             "AppClientSecret": "****",                             "SignInRedirectURI": "myapp://",                             "SignOutRedirectURI": "myapp://",                             "Scopes": [                                 "phone",                                 "email",                                 "openid",                                 "profile",                                 "aws.cognito.signin.user.admin"                             ]                         },
I added that into amplifyconfiguration.json manually - it wasn't;'t self generated when the json files were created. Are you able to direct me to where I can find the missing info for:
WebDomain
AppClientSecret
SignInRedirectURI
SignOutRedirectURI
I'm guessing it's within AWS somewhere in Amplify? Many thanks again for your help.

As part of a different issue, @sebsto is looking at the tutorial content to see if we can clarify this flow--some of the instructions are outdated. It may be that after we clarify the flow, you'll note a discrepancy between the correction and the flow you followed. In the meantime, you can take a look at our Social sign in documentation for more up-to-date information on how to provision your Amplify backend and configure your client.
Leaving this open pending response from @sebsto.

This error usually happens when there is no callback entered during Cognito configuration.
This are lines 7 and 9 in the section "Create an authentication backend".
and step d. "Trigger Authentication at Runtime" on the same page
I will take action to make these two steps more clear in the tutorial.

When using Amplify Studio to configure authentication (instead of using the CLI like the the mentioned tutorial), I get the same error as the original poster.
So when using
Amplify.Auth.signInWithWebUI
I get:
WSMobileClient/AWSMobileClient.swift:508: Fatal error: Unexpectedly found nil while unwrapping an Optional value
But when using the new Amplify Studio, I shouldn't need to do "amplify add auth" and "amplify push", correct? Since everything is configured in Amplify Studio.


sqlite3.OperationalError: no such table: tbl

When I execute the following
termsql -i ../DEREG.txt -t Deregistered -d',' -1 -o FAA.db --max-rows=10 -D

It gets to the part where it creates the database file (if needed) and the table & columns from the first row..but before any data is inserted, it returns the following:
Traceback (most recent call last):
  File "/usr/local/bin/termsql", line 357, in <module>
    c.executemany('insert into tbl values ('+placeholder_str+')', inserts)
sqlite3.OperationalError: no such table: tbl

Just wanted to come back & mention that I resolved this issue by making the following update to line 357
c.executemany('insert into '+table_name+' values ('+placeholder_str+')', inserts)

Thanks for reporting this. Will commit the fix soon.

No problem. I submitted a pull request too, I hope I did it right!

The fix is commited. Closing this issue.


Release v0.3.0

Checklist

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

@conda-forge-admin, please rerender

Happy days 🥳

Cheers @ocefpaf - I've lost count of the number of beers that I owe you btw 🤣

@ocefpaf Is there something in particular that needs to be done on the feedstock to enable python_abi 3.12 support?
Or will that happen in the fullness of time?

The bot will do it. I'll check if there are any problems with it...


Consider add random lenght benchmark

I was looking at the implementation of size_hint. If it's always the same, that's fine.
https://github.com/Kogia-sima/sailfish/blob/119e282945ee1c0b2d9083a3d615dee09125ac89/sailfish-compiler/src/procmacro.rs#L278-L284
Better an ONCE-like implementation, sure to give better results.
Seriously take the above measurement and update all threads by a single AtomicUsize.
Well I'm looking forward to seeing it up and running with all those stars of people with so much experience.

Here is the benchmark result of malloc + free operation (glibc 2.31 for amd64 arch).

As you can see, 100kB allocation is as fast as 5kB allocation. Also 0kB allocation takes 75ns on average. That means, 100kB allocation is faster than two small allocations.

But common situation, the first measures n the second n / 2, the third n * 3, etc. I insist on trying it in some situation where the size is not constant.

How would it be a bottleneck? Even if the rendered size is randomized, the size hint will converge on its maximum size. After convergence, the memory is allocated only once per rendering, and never reallocate the memory.
Could you explain the situation where the current implementation becomes a bottleneck more concretely?

https://llvm.org/docs/Atomics.html#acquire
Acquire provides a barrier of the sort necessary to acquire a lock to access other memory with normal loads and stores.
https://llvm.org/docs/Atomics.html#release
Release is similar to Acquire, but with a barrier of the sort necessary to release a lock.

That "lock" implies the atomic fence, which is not related to the lock system provided by OS (such as mutex).
https://www.codeproject.com/Articles/1183423/We-Make-a-std-shared-mutex-10-Times-Faster
For a simple read/write operation, atomic fence prevent compilers to:

optimize some arithmetic calculations at compile time
reorder the instructions specified in IR
place its variable at specific memory location

Program does not wait until the write operation. Hence this would not become a bottleneck,  even in a multithreaded application.

Note that on the x86_64 system the atomic operations will be transformed into simple 'mov' instruction. (https://github.com/Kogia-sima/sailfish/issues/24#issue-654698709)

Well if I don't convince you with this you can always try https://github.com/botika/bytes/tree/alloc buffer implementation. I'm thinking about the text of the sponsor tier.

I've removed the benchmark in favor of https://github.com/djc/template-benchmarks-rs. If you have any complaint about benchmarks, please re-create an issue on the djc's repository.

But djc has blocked me, I can't make issues or PR to their repositories. How do I do it?


Add Kubeflow tensorboard viz and fix tensorflow file IO for cloud back-ends

Describe changes
Enable the tensorboard visualization for Kubeflow and fixes the tensorflow file IO support when shared cloud storage like AWS S3 is used.

fix the kubeflow container entrypoint to consider ZenML ModelArtifacts in addition to TFX ModelRun types when enabling the Tensorboard view
when tensorflow is installed, TFX activates the tensorflow file IO plugin with a priority higher than the ZenML IO plugin. Use a higher priority for the ZenML file IO plugin
the 2.6 tensorflow release moves the IO support into a separate python package tensorflow_io that must be installed and imported explicitly for the S3/GCP/Azure FS filesystem schemes to be supported
fix the Kubeflow Metadata Store implementation
suppress printing locals in rich trace logs if not in debug mode to reduce verbosity
implement a TensorboardService to track and manage locally running Tensorboard daemons
implement a step TensorboardVisualizer that starts a local Tensorboard daemon to visualize the entire history of a model logged by a step
change the kubeflow example to use Tensorflow instead of scikit-learn and make use of Tensorboard visualizations

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table.
[ ] I have added tests to cover my changes.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Thanks @htahir1 , I hope I've addressed all your suggestions with the last commit (see inline).

Thank you for taking this over Stefan! Really like it!
I would have some suggestions:

Potentially adding this example as a file in the kubeflow example to showcase the tensorboard integration. I dont believe this deserves an extra example unless we have a native tensorboard integration that works in a magical way for different configurations

I merged the Tensorboard and Kubeflow examples. This basically meant switching the existing Kubeflow example from sklearn to tensorflow and adding some post-execution visualizations.

Adding a picture of the 'Launch Tensorboard' button on the Kubeflow UI for users to understand the true value is when this is done on Kubeflow. This can be put on the examples/kubeflow/README.md in a subheading where we explain the kubeflow-tensorboard integration we just enabled

Done. There are new pics and new information regarding Tensorboard in that README.md file now.

Adding in the limitations of Kubeflow (e.g. with S3) as you decribed in the JIRA ticket

No need. I fixed all the limitations :)

Potentially adding a 'TensorboardVisualizer' and showcasing launching tensorboard in a post-execution workflow. This should be trivial but exemplrary

You're going to love this: I implemented a TensorboardService to wrap around the Tensorboard local server, and a visualizer that makes it easy to start/stop Tensorboard services. This goes along with the idea that Services are for more than just model serving. The visualizer also works with Jupyter notebooks, which reminds me: I also updated the notebook in the kubeflow example to reflect this.

I have made the following changes to this branch.

Added docs
Added the following error to make sure kubeflow orchestrator does not fail non gracefully

@stefannica and @schustmi  please do take a look

@htahir1 The only thing i'd change is to not use an AssertionError. Maybe a RuntimeError instead?


Crash on demo scanner app when trying to use the camera

The resolution of the device could be lower than the minimum resolution configured for the view (720px).

Update the resolution selection algorithm to bypass the configured minimal resolution if no suitable resolution exists (in this case simply take the next highest resolution available)


[Task]: Enable/Disable certain integrations based on Enterprise/Business Plan

SubTasks
This task will need a task breakdown of its own. For now, I have added this as a placeholder to account for the complexity.
The breakdown will be further done going forward with the implementation.

Hey team! Please add your planning poker estimate with Zenhub @AmanAgarwal041 @sneha122

Adding the final estimate to 3 as per planning poker estimate


Improved the selectable prefixlengths in dropdown

Added logic that the minimal prefix length as defined by business rules is applied to the values of the prefix length dropdown

Shouldn't these restrictions/validations be configured in the back-end?

@florisie it would be better if the component got it from the backend, but that is beyond my skills/time to implement it. The Pydantic validation will still kick in if it falls outside the allowed ranges.


ROS2 Update

Changelog
[2.0.0]
Added

Unity package supports UPM and requires Unity 2022.3.
ROS2 support is available for both the Unity Package (com.siemens.ros-sharp) and the complete ROS# .NET solution, including RosBridgeClient, MessageGeneration, and Urdf (Libraries).
New ROS packages: File Server, Unity Simuatlion Scene, and Gazebo Simulation Scene, with ROS2 support.
Unity Simulation Scene and Gazebo Simulation Scene, included in the Unity Package for ROS2 support.
RawImageSubscriber script is now part of the Unity package.
Post-build events for Visual Studio streamline development between Unity and .NET.

Fixed

UrdfTransfer files use serializer-specific methods instead of Newtonsoft JSON.

Changed

RosBridgeClient, MessageGeneration, and Urdf support .NET 8.0.
RosBridgeClient, MessageGeneration, and Urdf source code is included in the Unity Package; and no longer dynamically linked to the Unity package.
Switched from websocket-sharp to websocket-sharp.netstandard.
The JoyAxisReader script in the Unity package inherits from IAxisReader interface for increased applicability.
The ImageSubscriber script in the Unity package is renamed to CompressedImageSubscriber.
URDF export and import windows in the Unity package utilize the existing RosSocket component in the scene.

Removed

Newtonsoft BSON is no longer supported.

Surprise, there's no need to use other forked versions anymore.


[Event Parameter Request] Page 21 Customer Card - procedure ActivateFields - event OnAfterActivateFields

We would like the parameter/-s IncludeSender added to the following event.
Page 21 Customer Card - procedure ActivateFields - event OnAfterActivateFields
Original:
Page 21 Customer Card - procedure ActivateFields - event OnAfterActivateFields

[IntegrationEvent(false, false)]
local procedure OnAfterActivateFields(var Customer: Record Customer)
begin
end;

Change:
[IntegrationEvent(true, false)] // <-- IncludeSender true
local procedure OnAfterActivateFields(var Customer: Record Customer)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52027.


Test Error Expects Wrong Average Value for Ratings

Functional Programming: Use the reduce Method to Analyze Data
The test requires the average to calculate to 8.675, but the average of the 5 IMDB ratings is actually 8.52 based on hand calculations and re-running the script outside of freeCodeCamp. I don't believe this is a rounding error of any sort.
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/functional-programming/use-the-reduce-method-to-analyze-data
Tell us about your browser and operating system

Browser Name: Firefox
Browser Version: 60.0.2 (up-to-date)
Operating System: macOS High Sierra, v10.13.5

If possible, add a screenshot here

While (8.8 + 8.6 + 9.0 + 8.3 + 7.9) / 5 = 8.52, the instructions say

Use reduce to find the average IMDB rating of the movies directed by Christopher Nolan.

As Christopher Nolan didn't direct all of the movies listed, why do we need to find the average of all 5?
In other words, don't we need to find the average of the movies that only have "Director": "Christopher Nolan" listed, which does comes to 8.675?

Good catch. I missed the “Nolan” qualification. I’ll see that it stays as/is. Thank you.

@thatnini, thank you for pointing that out! I also missed that. I'll revert the PR now.

Reopened with https://github.com/freeCodeCamp/freeCodeCamp/pull/17568

Can someone confirm that no change is required? If yes please comment and close.

@raisedadead, no change is required for this challenge. Here's a working solution:
let nolanFilms = 0;

const averageRating = watchList.reduce((acc, obj) => {
  if (obj.Director === "Christopher Nolan") {
    nolanFilms++;
    return Number(obj.imdbRating) + acc;
  }

  return acc;
}, 0) / nolanFilms;

With that the original test, 8.675, passes.


Callback error, when .cancelRequest

Why when I call .cancelRequest (pass request ID), fires error callback in .continuousLocation ?

Got it; 0.1.4: cancel an active request no longer report an error of type "operation cancelled"
commit cc4f1f2f384702d66dd7df3a9d44d110cf146eab


groovy escaping was wrong for aar

The printout was wrong since single quotes don't interpolate in groovy

Merging. I couldn't get frob to run again but gradle files shouldn't affect google.

Merging. I couldn't get frob to run again but gradle files shouldn't affect google.


Improve GitHub templates

🔎 Overview
This PR improves the GitHub issue and PR templates.
Its does:

move PULL_REQUEST_TEMPLATE.md into .github directory
remove the redundant ISSUE_TEMPLATE.md
use all lowercase filenames for issue templates
provides some minor text improvements

Thanks for tidying up the place :+1:


Stop command line processing once a script has been found

With this simple script:
p ARGV

Calling this in Ruby:
ruby -rset args.rb -c foo
["-c", "foo"]

But Natalie keeps consuming the arguments, so instead of running the script with "-c foo" as arguments, it compiles an executable named foo. We have to manually add -- to mark a boundary:
bin/natalie -rset -- args.rb -c foo
["-c", "foo"]

We should probably stop consuming the arguments once we've found an argument we shouldn't consume

The relevant code is in bin/natalie and lib/optparse.rb and both are pure Ruby


Fixed file association to work with new command line arguments

@Atria1234 Thank you for the quick fix! 👏


add method apply - allow call external methods

It's possible in the actual implementation call methods/functions that exist outside MyModule?

result = FooModule.chain_from(initial_state).
apply {|x| do_stuffs(x) }.
unwrap

where x.upcase can call any method passing x and returning a new ChainableMethods
by the way, I don't usually write ruby, so may have a better implementation,
Thanks!

@mateuspv hey, sorry I didn't pick up your PR. I later revised the code and added a #call method that does pretty much what you suggested.


1689 dev update fhsashandoccupancyagent to get the latest sash opening

Add /latestsash endpoint to retrieve the latest sash opening of the provided fumehood iri.

No test case added for this endpoint, because it is built with existing functions and the functions are tested with previous test cases.
Pass manual test with local stack.

Unit tests ran successfully and tested agent in stack. Original and new functions works properly. README looks good as well.


completion: fix zsh completion

Description
Update zsh completion per https://github.com/spf13/cobra/pull/1690
Also relates to https://github.com/Homebrew/homebrew-core/pull/107394
Motivation and Context

[ ] I have raised an issue to propose this change, which has been given a label of design/approved by a maintainer (required)

How Has This Been Tested?

Are you a GitHub Sponsor yet (Yes/No?)

[ ] Yes
[ ] No

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Documentation

[ ] I have updated the list of tools in README.md if (required) with ./arkade get -o markdown
[ ] I have updated the list of apps in README.md if (required) with ./arkade install --help

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

[ ] I have tested this on arm, or have added code to prevent deployment

Actually this is not needed, my bad.


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/example.justlists.ts
Checklist

[x] I understand that this is not the place to request a token listing.
[x] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[x] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL:
JustList Name:
Link to the official homepage of the JustList manager:

Please close this issue

Sorry, your issue will be closed as you did not submit your information in the correct format.


MTC Feed Merge does not remove trips when calendar_dates removed

Observed behavior
During the MTC service period feed merge, if a calendar_dates.txt entry is removed from the current/active feed (and there is no calendar.txt entry with the same service_id), the trips that reference this now removed service_id are not correctly removed in the resulting merged feed.
Expected behavior
Any trips that reference the removed service_id should also be removed during the merge process.
Steps to reproduce the problem

Upload current GTFS.zip (note that this file has service_id 7 entries in calendar_dates.txt, which are to be removed in the merge, but no service_id 7 in calendar.txt).
Upload future GTFS.
Merge the two feeds.
Download and inspect the resulting GTFS trips.txt file and observe that trip with trip_id 666177 (and many others) are not removed from the feed and reference a non-existent service_id (e.g., a feedId-scoped value Livermore1:7).

Any special notes on configuration used
The MTC extension must be enabled in order to expose the service period merge feature.
Version of datatools-server and datatools-ui if applicable (exact commit hash or branch name)
latest dev branches.

:tada: This issue has been resolved in version 3.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Trinity should relay transactions

What is wrong?
Currently we do nothing when people send us transactions.
How can it be fixed
Basic transaction relaying should be a good minimal first step towards a transaction pool.

A new long running service, TransactionPool
The pool should maintain something like a Queue where batches of transactions may be placed.
The pool should maintain individual bloom filters for each peer to track what transactions we've sent to each peer.

When we receive transactions, push them onto the TransactionPool queue.  Ensure that we update the bloom filter for the peer who sent us the transactions with the newly received transactions.
In the main loop of the TransactionPool service:

pull a batch of transactions from the queue.
for each peer, filter the transactions to only ones that have not been transmitted to that peer and send them the transactions.

Partially wanting to prioritize this because it'll clean up the DEBUG logs a lot, but it also helps make us a better network citizen.


Conflict

Tested with clean install of Laravel
$ laravel new laravel-5.2.8
$ cd laravel-5.2.8 && php artisan --version
# Laravel Framework version 5.2.8

$ composer require barryvdh/laravel-cors 0.7.x

#   Problem 1
#     - Conclusion: don't install barryvdh/laravel-cors v0.7.3
#     - Installation request for symfony/http-foundation == <IP_ADDRESS> -> satisfiable by symfony/http-# foundation[v3.0.1].
#     - Conclusion: remove laravel/framework v5.2.8
#     - Conclusion: don't install laravel/framework v5.2.8
#     - barryvdh/laravel-cors v0.7.0 requires illuminate/support ~5.0.14|5.1.x -> satisfiable by # illuminate/support[v5.0.22, v5.0.25, v5.0.26, v5.0.28, v5.0.33, v5.1.1, v5.1.13, v5.1.16, v5.1.2, # v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
#     - barryvdh/laravel-cors v0.7.1 requires illuminate/support ~5.0.14|5.1.x -> satisfiable by # illuminate/support[v5.0.22, v5.0.25, v5.0.26, v5.0.28, v5.0.33, v5.1.1, v5.1.13, v5.1.16, v5.1.2, # v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
#     - barryvdh/laravel-cors v0.7.2 requires illuminate/support ~5.0.14|5.1.x -> satisfiable by # illuminate/support[v5.0.22, v5.0.25, v5.0.26, v5.0.28, v5.0.33, v5.1.1, v5.1.13, v5.1.16, v5.1.2, # v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
# ...
#     - Installation request for laravel/framework == <IP_ADDRESS> -> satisfiable by laravel/framework[v5.2.8].
#     - Installation request for barryvdh/laravel-cors 0.7.x -> satisfiable by barryvdh/laravel-cors[v0.7.# 0, v0.7.1, v0.7.2, v0.7.3].

a~h duplicate issue.  Sorry to interrupt you.


Exclude Internal API from documentation

As a user, I should be able to exclude internal code from the generated documentation.
Internal code is the code that it is not exported in the files. If the user didn't export specific code snippet, like variables, or functions, probably means that it is not part of the public API so I would like to exclude those from the documentation.
Currently, I have to be putting @hidden everywhere in my source code, which makes the source code a little bit ugly.
Maybe introduce a flag that will allow me to do this.

Thanks for the input. This would be covered by #639 so I'll close this as a duplicate.


corrected tense crib cheat git

not complete at all but enough for the task :)

makes sense to me.  Good update.


Revert "wallabag: Drop swiftmailer patch"

The patch is still needed in 2.5.3. It was removed because it did not apply to previous tarball, which was accidentally generated from wrong branch. It applies again after 2692bb8570e09a255a2e81a6f2f911df9d6b9ef4.
This reverts commit 91391118c7a19c11f9cf2b95f6d0b84dfe04794e.
Follow up to https://github.com/NixOS/nixpkgs/pull/214706.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Wow, that clearly wasn't my day. Sorry for breaking that.
@ofborg build wallabag

It was more upstream’s fault.


FromRow is imported, but it is only a trait without a derive macro

Bug Description
When trying to derive the FromRow trait using the macro provided by SQLx, the following error appears:
FromRow is imported here, but it is only a trait, without a derive macro
Minimal Reproduction
Attempting to derive the sqlx::FromRow trait on any struct similar to that provided in the documentation. For example:
use sqlx::FromRow;

#[derive(FromRow)]
struct User {
    user_id: String,
    username: String,
    user_email: String,
    user_password_hash: String,
}

Info

SQLx version: 0.6.2
SQLx features enabled: ["runtime-tokio-rustls", "postgres", "chrono"]
Database server and version: psql (PostgreSQL) 14.3
Operating system: Fedora Linux 37 (Workstation edition), Kernel: Linux 6.1.14-200.fc37.x86_64
rustc --version: rustc 1.67.1 (d5a82bbd2 2023-02-07) (Fedora 1.67.1-1.fc37)

Update: THIS IS NOT A BUG but a result of my own blindness. To anyone else with this issue, you need to add the "macros" feature to SQLx.


stopped working with react-native 0.43.3

I'm calling it like so: import { SplashContainer, FooterTabsContainer, SettingsContainer } from '~/containers'  it should be reading from directory /Users/jasan/Desktop/ReactModoro/app/containers
It seems to add app/containers/Navigator in the path before
my babel rc file:
{  "presets": [
    "react-native-stage-0"
    ],
  "plugins": [
    ["babel-root-import", {
    "rootPathSuffix": "app"
    }]
  ]}

react-native start --reset-cache solved it

Ok cool, thanks for commenting the solution :)

Also, if you are having similar path issues with your Jest tests, see Jest cache incorrectly resolving modules on how to clear the Jest cache.

@jasan-s I'm getting this error every time, when I don't use --reset-cache postfix. Did you face this problem too?


Update library to remove compiler warnings on upcoming 0.14.0 PS release

This commit was pushed to the ps-0.14 branch.


Add support for numerous links

Currently, only 3 columns of up to 4 links are supported.
It would be ideal to have up to N columns of M links, where N and M can vary along some scale. Possibly limited by bootstrap widths of columns on a page.

Would like to have a save pattern similar to the following:
[
  {
    "Social": [
      {"Facebook": "http://www.facebook.com/"},
      {"Twitter": "http://www.twitter.com"}
    ]
  },
  {
    "Some other header": [
      {"link title": "link address"},
      {...}, ...
    ]
  }
]

Essentially, an array of dictionaries (collections), where each collection dictionary maps a string (that collection's header) to an array of dictionaries (links), where these link dictionaries map a string (title) to another string (address).
Or similar. The important points here are these:

Collections are ordered
Links are ordered
Link titles are directly associated with their respective link addresses

As long as these goals are met and the function convertFromLegacy(version) is completed for converting from version 1.0 to version 1.3, this issue should be completed.


read mail module

won't display buttons at the bottom of email, all other themes do
this has occurred after the last update

2.013 has only just become available for me but the issue is the same.
you say buttons can't be displayed top and bottom together but until 2.012 they were
thx, I'll use a different theme

2.013 has only just become available for me but the issue is the same.

I am surprised to hear this. Can you provide the screenshot of how it's broken for you?
If you want forcefully to display buttons at the bottom only you should go to Read User Mail module configuration and set Show buttons at top for to Never.


Updated ref attribute

Since inputRef support has been deprecated with version 8, I think it's time for the README to reflect it.
Regards

Thanks! I've missed this when updating the docs.


Exclude org.apache.beam.sdk.testing.UsesTriggeredSideInputs in StreamingRunnerV2 suite.

Follow up for: https://github.com/apache/beam/pull/26707

R: @reuvenlax @Abacn

Run Java Dataflow V2 ValidatesRunner Streaming


rake aborted! Is this gem should be used only with exisiting rails app?

I created empty Gemfile with
gem "motor-admin"
after bundle install I cant run
rails g motor:install && rake db:migrate
with errors
rake aborted!
No Rakefile found (looking for: rakefile, Rakefile, rakefile.rb, Rakefile.rb)

Wha am I doing wrong?

@niksfirefly you need to use motor admin gem in rails app (instead on creating empty Gemfile - run rails new and add gem into the rails generated Gemfile).
Also, for standalone non-rails motor admin you can try: https://www.getmotoradmin.com

Is there a simple way to recreate standalone config - users,authorization - same as standalone motor admin?

@niksfirefly it's in Motor Admin Rails Pro version:
https://www.getmotoradmin.com/ruby-on-rails - you can check the plans compression here.
Also in standalone version roles are available only in Pro too.

Can I implement it myself? devise,cancan etc.
according to guide
https://github.com/motor-admin/motor-admin-rails/blob/master/guides/defining_permissions.md
it is possible?

@niksfirefly yeah in rails open-source gem you can use devise/cancan just like in the guides link you shared.
Rails Pro has a UI so that Pro users can manage users/roles/permissions without code but with mouse clicks in the UI (good for non tech users).

Is there some guide howto modify or implement some logic in ruby to resources created from tables by motor-admin?

@niksfirefly you can use rails app for your back-end/api with standalone self-hosted motor-admin.
In this case motor-admin forms will trigger requests to your API implement in the rails app in ruby:
https://docs.getmotoradmin.com/guide/building_custom_forms.html


Mobil App shows deleted Teams

On the phone I can see deleted Teams as if they were never deleted.
Other colleagues have confirmed the same issue.
On the Desktop App, everything is as it should be.
MS Teams Mobile version: 1416/<IP_ADDRESS>0032405

It's very likely. The fix is included as part of the May release.


New yaml parser is too strict

Operating system
Fedora 37
Helmfile Version

▓▓▓ helmfile

  Version            0.150.0
  Git Commit         fc634c9
  Build Date         17 Jan 23 09:24 +07 (2 days ago)
  Commit Date        17 Jan 23 07:34 +07 (2 days ago)
  Dirty Build        no
  Go version         1.19.4
  Compiler           gc
  Platform           linux/amd64

Helm Version
{
  Version:"v3.10.3",
  GitCommit:"835b7334cfe2e5e27870ab3ed4135f136eecc704",
  GitTreeState:"clean",
  GoVersion:"go1.18.9"
}

Bug description
Before version 0.150.0 yaml has been merged without errors on duplicate keys, but in helmfile version 0.150.0 we get an error.
Previously, data with type map is merged and lists and strings are overwritten.
Example helmfile.yaml
templates:
  default: &default_chart_conf
    namespace: {{ .Namespace }}
    missingFileHandler: Error
    values:
      - values/{{`{{ .Release.Name }}`}}.yaml.gotmpl
  stateless-app: &stateless_app_chart_conf
    <<: *default_chart_conf
    chart: charts/stateless-app
    values:
      - values/{{`{{ .Release.Name }}`}}.yaml.gotmpl
      - imagePullSecrets:
          - name: regcred

releases:
  - name: app1
    <<: *stateless_app_chart_conf

Error message you've seen (if any)
err: failed to read helmfile.yaml: reading document at index 1: yaml: unmarshal errors:
  line 16: key "values" already set in map
  line 16: key "values" already set in map

Steps to reproduce
https://github.com/metajiji/helmfile-0.150-bug/tree/main
Working Helmfile Version
0.149.0
Relevant discussion
No response

@metajiji Thanks for your feedback. I will check out this issue.

I think this might be the problem:
https://github.com/goccy/go-yaml/blob/master/yaml.go#L164
Could maybe be fixed by using:
https://github.com/goccy/go-yaml/blob/master/yaml.go#L184
and pass in the following DecodeOption:
https://github.com/goccy/go-yaml/blob/6cdefc42e112ac71cbe316e1eed264ea62f58e25/decode.go#L39
But I'm lacking the time to try that out.

@metajiji you can use goccy/go-yaml by setting env var HELMFILE_GOCCY_GOYAML=true. the default yaml parser disables yaml overwrite feature. because it uses yaml.v2.
I will update the 0.150.0 release notes. Thanks for your feedback.

@pjotre86 Thanks for your anwser.

@metajiji I have updated the release note for 0.150.0. please read it. Thanks very much. t thanks for your feedback again.

@yxxhero Thanks for the answer! Please make the new parser's default behavior the same as the old one in the next release.

@metajiji yeah. v1.0 will be released in the future. and the goccy/go-yaml will be the default yaml parser.


Add CPU vendor HW requirement

Following the gpu.vendor and gpu.vendor-name.

Hi @happz, the subject of this issue includes CPU but the description includes gpu, is CPU in the subject a typo?

Hi @happz, the subject of this issue includes CPU but the description includes gpu, is CPU in the subject a typo?

Both are correct: gpu.vendor and gpu.vendor-name are already defined, we need their counterparts for cpu block.

Hi @happz, the subject of this issue includes CPU but the description includes gpu, is CPU in the subject a typo?

Both are correct: gpu.vendor and gpu.vendor-name are already defined, we need their counterparts for cpu block.

Hi @happz , for CPU, lscpu shows:
......
vendor_id       : GenuineIntel
......
So we just define cpu.vendor or also need cpu.vendor-name ?

Looks like vendor-name is what we need the most, and we can ignore vendor. See this paragraph on how we differentiate between foo and foo-name when it comes to properties that are like IDs.


fixed nuclei report parser

Nuclei seems to change names of variables in json reports, fixed nuclei report parser to match these names

Thanks. Can you add a unit tests with these new names/tags? And retain support for the old tags as well. We can't support all versions of all parsers, but in this case I think it's quite easy to retain backwards compatability.

Sure, I will tag you when it will be done

superseded by #6103


Fix NPE in Pulsar Functions CLI

As it stands, an NPE is thrown if you try to create or localrun a function with neither a class name nor a function name supplied, as the CLI will attempt to infer a function name from a class name that is null.

@jerrypeng @srkukarni ^^


Haskell-style Typeclasses

When I first started rtype, I was tempted to just steal Haskell's Hindley-Milner type annotations and just start using them, but I saw several problems with that:

Haskell types are all curry all the time. I wanted to be able to express: add2(a: Number, b: Number) -> Number.
How do you deal with this?
I wanted to be able to name parameters for clearer documentation. Haskell's types are very expressive and clean-looking, but it's sometimes hard to figure out what the type variables represent in real code.

So, we abandoned Hindley-Milner altogether, and forgot all about typeclasses. Ever since, we've been struggling with the question: How do we represent type polymorphism, and especially, higher order functions like map.
The Functor f => is called a type constraint. It basically says, "in this definition, f represents the Functor typeclass". Like generic type constructors in other languages, a typeclass takes a type parameter, so f a represents a type that is a member of the Functor typeclass, and f b represents a possibly different type that also a member of the Functor typeclass.
We currently have no way to express this in rtype, and because I do a lot of functional programming, I have been falling back on Haskell's Hindley-Milner types in a lot of my writing.
But I'm frustrated with that because as I mentioned already, switching from rtype to Hindley-Milner or vise verse, we lose a lot of valuable information. Hindley-Milner doesn't express enough about the names and purposes of variables, lacks ...rest, etc..., and also doesn't naturally allow for uncurried multi-arg functions.
If we try to express a lot of higher-order functions in rtype, it falls flat.  To complicate matters, I'm building lots of types that implement a bunch of stuff with methods, meaning I need a way to say that map is a method on a Functor type, and takes this as an argument.
We need the best of both worlds.
So, I have a proposal: Quit messing around trying to fit Java-style generics into this task, and adopt real typeclasses and type variables to manage this problem in a more natural way.
Note, in the following examples, I'm also adding this as an optional first argument in the signature.
Instead of specifying a type constraint with a variable name, we'll explicitly say Functor[a] or Functor[b] instead of f a or f b.
typeclass Foldable {
  reduce (this: Foldable[a], (accumulator: b, item: a) => b, initial?: a) => b
}

typeclass Semigroup {
  concat (this: Semigroup[a], ...args: [...Semigroup[Any]]) => Semigroup[Any]
}

typeclass Functor {
  map (this: Functor[a], a => b) => Functor[b]
}

typeclass Collection: Foldable & Semigroup & Functor;

// Arrays implement Foldable, Semigroup, and Functor, so they are
// automatically members of the Collection typeclass.

updateWhere(criteria: Object) => (updater: a => b) => Collection[c] => Collection[c]

updateWhere() is a curried function that takes a criteria object (e.g., { id: 2 }), an updater function from a to b, (where a and b represent any type, and may refer to the same type), and a Collection, and returns a new Collection of the same Collection type, with the matching elements replaced with updated versions.
Generic typeclass Instances
Because we have an Any type, our typeclass instances can also be generic, which is equivalent to saying that a member of that typeclass must satisfy the typeclass interface, but may contain any type:
typeclass Semigroup {
  concat (this: Semigroup[a], ...args: [...Semigroup[Any]) => Semigroup[Any]
}

Thoughts?

Perhaps Sanctuary can provide some guidance. Here's the definition of S.map:
S.map = def('map', {f: [Z.Functor]}, [Fn(a, b), f(a), f(b)], Z.map);

String representation:
S.map.toString();
// => 'map :: Functor f => (a -> b) -> f a -> f b'

Possible type errors:
S.map(x => x, 'XXX');
// ! TypeError: Type-class constraint violation
//
//   map :: Functor f => (a -> b) -> f a -> f b
//          ^^^^^^^^^                ^^^
//                                    1
//
//   1)  "XXX" :: String
//
//   ‘map’ requires ‘f’ to satisfy the Functor type-class constraint; the value at position 1 does not.

S.map(n => n < 0 ? null : Math.sqrt(n), [9, 4, 1, 0, -1]);
// ! TypeError: Type-variable constraint violation
//
//   map :: Functor f => (a -> b) -> f a -> f b
//                             ^
//                             1
//
//   1)  3 :: Number, FiniteNumber, NonZeroFiniteNumber, Integer, ValidNumber
//       2 :: Number, FiniteNumber, NonZeroFiniteNumber, Integer, ValidNumber
//       1 :: Number, FiniteNumber, NonZeroFiniteNumber, Integer, ValidNumber
//       0 :: Number, FiniteNumber, Integer, ValidNumber
//       null :: Null
//
//   Since there is no type of which all the above values are members, the type-variable constraint has been violated.

Sanctuary's checking happens at run time, though, so the ideas may not translate to this project.

Thanks for your input, @davidchambers -- very interesting.
I started working on rtype because I felt a need for a standard way to describe JavaScript function signatures and object interfaces that projects like TypeScript, Flow, and Haskell's signatures don't solve.
Haskell's signatures seem to come the closest to what I was looking for, but I also wanted to include parameter names in the signature, describe default values, etc... My thinking about how to express those concepts was heavily influenced by JavaScript itself. I wanted a signature notation that would be easy for JavaScript developers to read, even if they were not familiar with rtype signatures.
I use rtype signatures in documentation, blog posts, and books, the same way you'd use a Haskell signature to describe a function's type before expanding on it in a prose description.
We do want to build runtime type checking and static analysis tools that can benefit from Rtype, but those efforts haven't had much attention, yet.

Clarifying the Meaning of :
In Rtype, we already have the : symbol in our type annotations. At the root level, they mean "the thing on the left belongs to the type on the right".
Since Rtype is already a structural type system, that means that the left side symbol might belong to many types: 1:Many.
What if we take this thinking a bit farther? Currently, we expect the thing on the left to always represent the name of a parameter. What if the thing on the left is just a type variable, just like the thing on the right?
Then we could say things like this:
composeMixins(...mixins: [...Function]) => (
  instance = {}: (o: Object),
  mix: o => m: o & (e: Object)
) => m

In this example:

mixins is a type variable that is a member of a type class which represents any array of zero or more functions.
instance is a type variable which is a member of o,
o is a type variable which is a member of Object
mix is a function from o to m
m is a type that is a member of both o and e
e is a type that is a member of Object
o and e are probably different. While both are members of Object, o may not be a member of e and vice versa.

In other words, in all cases, the type variable on the left-hand side of the : is a member of the type specified to the right of the :.
We can make a few inferences, to reduce the amount of syntax noise in the example above:
A. Since instance defaults to {}, we can infer that o is a member of Object:
  instance = {}: o,

B. Since e extends the properties of o, we can also infer that it is a member of Object:
  mix: o => m: o & e

So, we can represent the composeMixins signature like this:
composeMixins(...mixins: [...Function]) => (
  instance = {}: o,
  mix: o => m: o & e
) => m

Here's the implementation:
const composeMixins = (...mixins) => (
  instance = {},
  mix = (...fns) => x => fns.reduce((acc, fn) => fn(acc), x)
) => mix(...mixins)(instance);

And here's how it's used:
/*
withFlying(instance: o) => m: o & {
  fly() => m, effects(change flight status to 'Flying'),
  land() => m, effects(change flight status to 'Not flying'),
  getFlightStatus() => 'Flying' | 'Not flying'
}
*/
const withFlying = instance => {
  let isFlying = false;

  return Object.assign(instance, {
    fly () {
      isFlying = true;
      return instance;
    },
    land () {
      isFlying = false;
      return instance;
    },

    getFlightStatus: () => isFlying ? 'Flying' : 'Not flying'
  });
};

const withQuacking = text => instance => Object.assign(instance, {
  quack: () => console.log(text)
});

// Compose mixins:
const createDuck = composeMixins(withFlying, withQuacking('Quack!'));
const malard = createDuck();
console.log(malard.fly().getFlightStatus()); // Flying
malard.quack(); // "Quack!"

Further, we can eliminate the clarifying names and only include necessary type information. The question is, does it make it more readable, or less?
mixin(a) => b: a & e

composeMixins([...mixin]) => (
  o,
  (...mixin) => o => m: o & e
) => m

Ping @Mouvedia @tomekwi @koresar @Raynos

mixin is a function from a to b where b is a member of both a and e

Does this mean that a and e are both types?
I'm confusing what's a type, what's a variable name, and what's a property name. Could you please capitalize all the types in that last (shortest) example?

a and e are both type variables, variables that represent membership in different type classes.
In the last (shortest) example, a, b, and e are type variables in scope within the definition of mixin. mixin itself is a variable that represents membership in the type, (a) => b: a & e.
Within the composeMixins() definition, mixins, o, m, and e are all type variables inside the scope of the composeMixins definition. composeMixins is a type variable, too.
Similar to function scope in JS, type variables defined inside the definition of a signature are only in scope for that signature, so e from mixin may be different from e from composeMixins.

I just wanna point out that mix: (...mixins) => o => (m: o & e) seems hard to read due to the chained =>; separators of some sort might come in handy. I am not a big fan of bi-directional inference: if you refactor the originator it will break in cascade. Apart from that, I don't think I'll be of a big help on this issue.

I wanted a signature notation that would be easy for JavaScript developers to read, even if they were not familiar with rtype signatures.

This is primordial.

@Mouvedia The "chained" arrow functions are just currying. Anybody familiar with arrow function currying in JavaScript, or Haskell type notation should recognize it immediately. Related: "Familiarity Bias is Holding You Back: It's Time to Embrace Arrow Functions"

Define:

bi-directional inference
Originator

In this comment, you are inferring numerous types; by bi-directional I meant that you will have to scan in both direction to find the reason. The "originator" either has a type explicitly set or a predefined type and does not depend on another "type variable".

If a type variable has not been defined, a new type variable is created automatically in the scope of the signature. You're correct in saying that we can infer things about the type based on how it's used, but generally speaking, that's a concern for static analysis tools. Developers can just use the types and not worry about how the inference works.

Note to self: Update the syntax, clarify any lingering questions, create a PR.
I'm already using higher kinded types in lots of documentation.

Updated original proposal description. I've been using this syntax for quite a while now and testing it in mentorship sessions. Mentees seem to pick up what's going on quickly, and people familiar with Haskell should be able to use what they know about Haskell to learn the ropes.


Allow user to configure CRA_HOST.

When working within a container, it's necessary to specify a host as an IP, or loopback (e.g. <IP_ADDRESS>).

For posterity, this update has been published to PyPi as v1.0.2: https://pypi.org/project/django-cra-helper/1.0.2/


The Null Object Thing

Describe your problem here.
i tried to load a chart from Vs Impostor V4 (pretty much has been happening a while when i tried to load other custom charts) and this just pops up:
Note.hx (line 158)
editors/ChartingState.hx (line 2683)
editors/ChartingState.hx (line 2642)
editors/ChartingState.hx (line 2156)
editors/ChartingState.hx (line 2460)
editors/ChartingState.hx (line 1561)
flixel/FlxState.hx (line 203)
flixel/FlxGame.hx (line 747)
flixel/FlxGame.hx (line 677)
flixel/FlxGame.hx (line 545)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)
Uncaught Error: Null Object Reference
im probably stupid but idk how to fix it and i need help lol
any kind of help would be great
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

something wrong with note types, maybe the chart has a note type that you haven't added yet

You don't use charts from there! The authors have put all sorts of different things there and it can't be transferred so easily! Search the Internet for alterations to the original engine. I recommend it: https://gamebanana.com/mods/422721   The man did a good job. Also used and everything is fine


added config to publish to maven repo in java-client pom.xml

Describe your changes :

added config to publish to maven repo in java-client pom.xml

Type of change :

[x] Improvement
[x] New feature

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
@mohitdeuex

@parthp2107 it will need the settings.xml as well ..right?

Yes, we can find it under the .m2 directory. Should I just copy and paste it into our OMD parent directory?

@mohityadav766


imp: delegate store operation of clientState and consensusState in initialize to contract

Description

This implements for now only the ibc-go part. Tests are expected to fail; we need new versions of contracts that handle setting the client state and consensus state.
closes: #3956
Commit Message / Changelog Entry
type: commit message

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

I've updated the contract according to the changes, but these two tests are failing because of an incorrect data, please check.
Can I push the new contracts to the repo so the CI can pass?

Thank you, @vmarkushin. I just gave you write permissions, so feel free to push to any of the 08-wasm branches. When you upload the contracts, I will check those tests.

Thank you, @vmarkushin. I just gave you write permissions, so feel free to push to any of the 08-wasm branches. When you upload the contracts, I will check those tests.

Deal, thanks!


cannot load such file -- celluloid/current (LoadError)

Getting this error after dead_end updated to 1.1.6
Traceback (most recent call last):
20: from bin/rails:9:in `<main>'
19: from bin/rails:9:in `require'
18: from /Users/waruna/.gem/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/commands.rb:18:in `<top (required)>'
17: from /Users/waruna/.gem/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:49:in `run_command!'
16: from /Users/waruna/.gem/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:85:in `server'
15: from /Users/waruna/.gem/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:85:in `tap'
14: from /Users/waruna/.gem/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:88:in `block in server'
13: from /Users/waruna/.gem/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:88:in `require'
12: from /Users/waruna/Project/example-server/config/application.rb:7:in `<top (required)>'
11: from /Users/waruna/.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/bundler.rb:174:in `require'
10: from /Users/waruna/.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/bundler/runtime.rb:58:in `require'
 9: from /Users/waruna/.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/bundler/runtime.rb:58:in `each'
 8: from /Users/waruna/.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/bundler/runtime.rb:69:in `block in require'
 7: from /Users/waruna/.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/bundler/runtime.rb:69:in `each'
 6: from /Users/waruna/.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/bundler/runtime.rb:74:in `block (2 levels) in require'
 5: from /Users/waruna/.gem/ruby/2.7.0/gems/dead_end-1.1.6/lib/dead_end/auto.rb:21:in `require'
 4: from /Users/waruna/.gem/ruby/2.7.0/gems/dead_end-1.1.6/lib/dead_end/auto.rb:21:in `require'
 3: from /Users/waruna/.gem/ruby/2.7.0/gems/sidetiq-0.7.2/lib/sidetiq.rb:10:in `<top (required)>'
 2: from /Users/waruna/.gem/ruby/2.7.0/gems/dead_end-1.1.6/lib/dead_end/auto.rb:50:in `require'
 1: from /Users/waruna/.gem/ruby/2.7.0/gems/dead_end-1.1.6/lib/dead_end/auto.rb:21:in `require'
/Users/waruna/.gem/ruby/2.7.0/gems/dead_end-1.1.6/lib/dead_end/auto.rb:21:in `require': cannot load such file -- celluloid/current (LoadError)

Further investigation into this issue, reveals that this was causing due deprecated library 'sidetiq'. After replacing that issue went away. Therefore I'm going to close this. Thanks for looking into it.


isl.gforge.inria.fr is down

https://github.com/BobBuildTool/basement/blob/ad124065ccf6adb5b3118f638b12e1a5cc3b2af4/recipes/libs/isl.yaml#L20
isl.gforge.inria.fr is down. maybe we should switch to sourceforge.

Yes, switching to sourceforge looks like a good alternative.


Differences

Hello,
I will like to test  this project and was wondering if you could explain the difference between your project vs the standard homebridge camera as copied below.
Currently, I’m having Live streaming issue using “homebridge-camera-ffmpeg“ I’m guessing is it’s due to the bug in ffmpeg.
https://www.npmjs.com/package/homebridge-camera-ffmpeg

@zllovesuki
adding to the previous post should both compiler be installed or either one  ffmpeg - v4l2rtspserver
and can this be used for IP cameras I noticed this plugin is built specifically for Logitec USB camera?


Fix code scanning alert - Bad HTML filtering regexp

Tracking issue for:

[ ] https://github.com/TNSQUAD/PhoenixMart/security/code-scanning/26

fix error


Fixing the missing spacing below full-width images

Classname was just incorrectly set.
Changes

Fixed the full-width-media_image classname

Testing

gulp build and create a blog post with images (or trust the screenshot)

Review

@KimberlyMunoz
@sebworks
@anselmbradford

Screenshots
Before

After

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

score

:+1:


WFLY-4869 Failed to start statistics service for datasources

retest this please

Linux with security manager Build 1787 is now running using a merge of a78aaab874e5fc9e2026766b33aaa2768e4cb802

Linux Build 6890 is now running using a merge of a78aaab874e5fc9e2026766b33aaa2768e4cb802

Windows Build 1965 is now running using a merge of a78aaab874e5fc9e2026766b33aaa2768e4cb802

Linux with security manager Build 1787 outcome was SUCCESS using a merge of a78aaab874e5fc9e2026766b33aaa2768e4cb802
Summary: Tests passed: 864, ignored: 311 Build time: 0:06:05

Windows Build 1965 outcome was SUCCESS using a merge of a78aaab874e5fc9e2026766b33aaa2768e4cb802
Summary: Tests passed: 2929, ignored: 367 Build time: 0:48:23

Linux Build 6890 outcome was FAILURE using a merge of a78aaab874e5fc9e2026766b33aaa2768e4cb802
Summary: Tests failed: 1 (1 new), passed: 2928, ignored: 367 Build time: 0:49:10
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.smoke.deployment.rar.tests.earmultirar.EarPackagedMultiRarDeploymentTestCase: <no details avaliable>

Windows Build 1978 is now running using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819

Linux with security manager Build 1800 is now running using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819

Linux Build 6903 is now running using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819

Linux with security manager Build 1800 outcome was SUCCESS using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests passed: 864, ignored: 311 Build time: 0:05:51

Windows Build 1978 outcome was SUCCESS using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests passed: 2930, ignored: 367 Build time: 0:47:58

Linux Build 6903 outcome was FAILURE using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests failed: 41 (41 new), passed: 2797, ignored: 367, muted: 1 Build time: 1:04:55
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.messaging.mgmt.ConnectionFactoryManagementTestCase.testWriteDiscoveryGroupAttributeWhenConnectorIsAlreadyDefined: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase.startAndSetupContainer: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase.testDatabaseCertLoginModule: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase.stopContainer: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.startAndSetupContainer: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testVerifyingConnector: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testNonVerifyingConnector: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testDefaultConnector: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testWantVerifyConnector: <no details avaliable>

org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.stopContainer: <no details avaliable>

org.jboss.as.test.manualmode.ws.ReloadWSDLPublisherTestCase.testHelloStringAfterReload: <no details avaliable>

##### there are 31 more failed tests, see build details

retest this please
On Tue, Jul 7, 2015 at 3:18 PM, WildFly CI<EMAIL_ADDRESS>wrote:

Linux Build 6903
http://brontes.lab.eng.brq.redhat.com/viewLog.html?buildId=59571&buildTypeId=WFPR
outcome was FAILURE using a merge of e4c0575
https://github.com/wildfly/wildfly/commit/e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests failed: 41 (41 new), passed: 2797, ignored: 367, muted: 1
Build time: 1:04:55
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.messaging.mgmt.ConnectionFactoryManagementTestCase.testWriteDiscoveryGroupAttributeWhenConnectorIsAlreadyDefined:
org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase.startAndSetupContainer:
org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase.testDatabaseCertLoginModule:
org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase.stopContainer:
org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.startAndSetupContainer:
org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testVerifyingConnector:
org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testNonVerifyingConnector:
org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testDefaultConnector:
org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.testWantVerifyConnector:
org.jboss.as.test.manualmode.web.ssl.HTTPSWebConnectorTestCase.stopContainer:
org.jboss.as.test.manualmode.ws.ReloadWSDLPublisherTestCase.testHelloStringAfterReload:
there are 31 more failed tests, see build details
—
Reply to this email directly or view it on GitHub
https://github.com/wildfly/wildfly/pull/7704#issuecomment-119200520.

Linux Build 6906 is now running using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819

Windows Build 1981 is now running using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819

Linux with security manager Build 1803 is now running using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819

Linux with security manager Build 1803 outcome was SUCCESS using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests passed: 864, ignored: 311 Build time: 0:06:12

Windows Build 1981 outcome was SUCCESS using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests passed: 2930, ignored: 367 Build time: 0:48:25

Linux Build 6906 outcome was FAILURE using a merge of e4c0575bcf0288e77b17efa6cfc47a8edc519819
Summary: Tests failed: 1 (1 new), passed: 2927, ignored: 367, muted: 2 Build time: 0:50:22
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>


Create env at Script/React build time

It's in the config file, so just plug that information directly into esbuild.


Implement Matrix of Deleting Things - recursive namespace deletion decision from F2F

During the F2F meeting we decided that object that owns other objects should recursively delete everything they own if they are deleted.
See the Matrix of Deleting Things

/lifecycle frozen
/remove-lifecycle stale

This project is being archived, closing open issues and PRs.
Please see this PR for more information: https://github.com/kubernetes/community/pull/6632


Confusing hosting for Google Go libraries

Google often serves important Go libraries like the /x/ libs, grpc, and Google Cloud API's, from repositories different than the import paths. And when you try to visit the repository path, you don't see a VCS system, but the Go docs. Finally, the implementation code is also hosted at GitHub mirrors, adding further to the confusion. These differences make it harder for newbies to make use of these important libraries, compared to just matching the import path with a valid VCS (would be nice if GitHub, but by no means necessary).
As a workaround, users can apply careful git-submodule paths, or manual vendoring, or Glide, or Go 1.11+ modules. But we shouldn't have to lean on complicated package resolvers whenever hosting directly from the same place as the import path is an option.

There are advantages and disadvantages to using vanity import paths like golang.org/x/tools. They are more future-proof: it allows the import paths not to change even when the underlying VCS changes. This has happened in the past, when the Go project moved from https://code.google.com/p/go to https://go.googlesource.com.

Finally, the implementation code is also hosted at GitHub mirrors, adding further to the confusion.

Most of the implementation code is hosted at https://go.googlesource.com. It is mirrored to GitHub, so that's merely a copy for code browsing convenience.
Changing import paths now is not something that can be done easily.

These differences make it harder for newbies to make use of these important libraries, compared to just matching the import path with a valid VCS

This is one of the disadvantages, that people who are new to Go have to read and become familiar with vanity import paths. They are documented at https://golang.org/cmd/go/#hdr-Remote_import_paths. If you have suggestions for how we can improve documentation to make this better, that'd be helpful.
If you'd like to discuss this further, there is a list of forums at https://golang.org/wiki/Questions that would be a better fit for this, since the Go project uses the issue tracker primarily for tracking bugs.


Dockerhub Build Fail

The new image is Alpine Docker 1.9 conform. Dockerhub is still on Docker 1.8 build slaves. This ticket is to remind me to check builds.
Imagelayers also seems to have problems when Dockerhub builds fail.

Building Image in my own CI System, solved.


Link not defined with FlowRouter example

I follow the FlowRouter example, however I get a blank page and in the console:
Uncaught ReferenceError: Link is not defined
    at Button.render (Button.jsx:17)

Same here

Stuck in the same spot... Did you guys figured out how to get trough? Thanks!

Closing due to inactivity @vephinx @jordaofranca @JaimeVelasco
Open new issue if it still persists.


[fix] Database change not sync

Codecov Report

Merging #11258 (847ff2b) into dev (d9516e2) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11258      +/-   ##
============================================
- Coverage     40.24%   40.21%   -0.03%
+ Complexity     4839     4835       -4
============================================
  Files           973      973
  Lines         37302    37295       -7
  Branches       4141     4134       -7
============================================
- Hits          15012    14999      -13
- Misses        20749    20753       +4
- Partials       1541     1543       +2

Impacted Files
Coverage Δ

...erver/master/processor/queue/TaskEventService.java
69.64% <0.00%> (-10.72%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.00% <0.00%> (-2.78%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
55.12% <0.00%> (-1.29%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.60% <0.00%> (-0.30%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
28.47% <0.00%> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

...phinscheduler/plugin/task/chunjun/ChunJunTask.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/api/ShellCommandExecutor.java
0.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <0.00%> (+0.05%)
:arrow_up:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

ref: #10700


Not supporting Redis ElasticCache cluster mode

thread 'main' panicked at /app/svix-server/src/queue/redis.rs:155:17:
backend-1  | error creating consumer group or stream: Moved

In general, Redis cluster isn't supported in the OSS version at the moment. I'll put this up for discussion.

Exactly what @svix-jplatte said, it's supported.


Every time I stream it gives an error.

Describe the bug
When I stream it says Stream error result: Failed
Details: [WNSError] Xccs : ErrorCallingWNS : Send command failed : State WaitingForServerToRegister
To Reproduce
No response
Expected behavior
No response
Environment
- OS: 15.7
- Device: iMac
- Greenlight version: 2.01

Anything else?
No response
New issue

[X] I confirm that this is a new issue and not a duplicate

Try to restart the console. This has been reported a lot lately and unfortunately there is no way to prevent this error as it happens on the normal xbox app too. Sometimes the xbox loses the connection and needs to be turned on manually before it connects to the xbox network. (Mostly happen after updates or when the user is signed off overnight)


Pyam, Statistics class

Dear, Pyam developers
Hi! I'm Jintae Kim from KAIST(South Korea).
In the Pyam manual, I saw 'the Statistics Class', but I can figure out how to use the Statistic Class...
It seems that there is no tutorial material for how to produces descriptive statistics (mean, median, ....)
Could you introduce me to how to use Statistics functions? It would be very helpful to me.
Best regards,
Jintae Kim.
Sustainable Development Center
Post Master Research Associate
Bldg. No.9,  471, 85 Hoegi-ro, Dongdaemun-gu, Seoul,Korea (02455)
T: +82-02-958-3583
E<EMAIL_ADDRESS>

Hi @kimjintae9011, thanks for raising this issue!
The Statistics class was developed and used as part of the work for the IPCC SR15, but we didn't spend much time on documenting it or writing tutorials (yet). You can look at how it was used in the SR15 in the open-source notebooks, see a rendered version or the GitHub repository.
You should look in particular at these notebooks:

sr15_2.3.3_global_emissions_statistics
sr15_<IP_ADDRESS>_primary_energy_statistics
sr15_2.SM.4_geophysical_characteristics

Hope this helps - and it would be great if you would be willing to start a little tutorial based on what you learn!

Dear, Daniel Huppmann,
Thanks, Daniel,
It is really good material and helpful to me.
Have a nice day!
Best,
Jintae Kim
-----Original Message-----
From: "Daniel Huppmann" @.>
To: "IAMconsortium/pyam" @.>;
Cc: "kimjintae9011" @.>; "Mention" @.>;
Sent: 2021-09-17 (Fr) 17:07:24 (UTC+09:00)
Subject: Re: [IAMconsortium/pyam] Pyam, Statistics class (#581)
Hi @kimjintae9011 https://github.com/kimjintae9011, thanks for raising this issue!
The Statistics class was developed and used as part of the work for the IPCC SR15, but we didn't spend much time on documenting it or writing tutorials (yet). You can look at how it was used in the SR15 in the open-source notebooks, see a rendered version https://data.ece.iiasa.ac.at/sr15_scenario_analysis/ or the GitHub repository https://github.com/iiasa/ipcc_sr15_scenario_analysis.
You should look in particular at these notebooks:
sr15_2.3.3_global_emissions_statisticssr15_<IP_ADDRESS>_primary_energy_statisticssr15_2.SM.4_geophysical_characteristicsHope this helps - and it would be great if you would be willing to start a little tutorial based on what you learn!
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub https://github.com/IAMconsortium/pyam/issues/581#issuecomment-921595487, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIYGKE6UZGLIEJ3YOCLOLDDUCLZLTANCNFSM5EGHSLSQ.Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign=notification-email&utm_medium=email&utm_source=github.

Hello @kimjintae9011, hope that this material has been helpful? Would you be interested in writing a simple tutorial for the statistics class, or do you have suggestions for improvements?


[feat] Generate executable job.sh files

Will fix https://github.com/eth-cscs/reframe/issues/1679

Hello @jgphpc, Thank you for submitting the Pull Request!

In the file reframe/core/pipeline.py:

Line 1400:33: E131 continuation line unaligned for hanging indent

Do see the ReFrame Coding Style Guide

Hello @jgphpc, Thank you for submitting the Pull Request!

In the file reframe/core/pipeline.py:

Line 1400:33: E131 continuation line unaligned for hanging indent

Do see the ReFrame Coding Style Guide

Codecov Report

Merging #1716 (724cad9) into master (b3654c2) will decrease coverage by 0.06%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master    #1716      +/-   ##
==========================================
- Coverage   87.31%   87.25%   -0.07%
==========================================
  Files          46       46
  Lines        7639     7656      +17
==========================================
+ Hits         6670     6680      +10
- Misses        969      976       +7

Impacted Files
Coverage Δ

reframe/core/pipeline.py
91.45% <28.57%> (-0.72%)
:arrow_down:

reframe/frontend/dependencies.py
96.50% <0.00%> (-1.99%)
:arrow_down:

reframe/core/launchers/__init__.py
100.00% <0.00%> (+5.00%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b3654c2...724cad9. Read the comment docs.

Codecov Report

Merging #1716 (724cad9) into master (b3654c2) will decrease coverage by 0.06%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master    #1716      +/-   ##
==========================================
- Coverage   87.31%   87.25%   -0.07%
==========================================
  Files          46       46
  Lines        7639     7656      +17
==========================================
+ Hits         6670     6680      +10
- Misses        969      976       +7

Impacted Files
Coverage Δ

reframe/core/pipeline.py
91.45% <28.57%> (-0.72%)
:arrow_down:

reframe/frontend/dependencies.py
96.50% <0.00%> (-1.99%)
:arrow_down:

reframe/core/launchers/__init__.py
100.00% <0.00%> (+5.00%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b3654c2...724cad9. Read the comment docs.

I can fix this PR.

I can fix this PR.

@jenkins-cscs retry dom

@jenkins-cscs retry dom


Get rid of the :format parameter from the captured params if the block arity isn't same with captured params size

ref #1868
This patch add more support for the captured params.
get "/foo/:id/", provides: :html do |id|
  id
end

# before
get "/foo/123/" #=> ArgumentError
# after
get "/foo/123/" #=> "123"

get "/foo/:id/", provides: :html do |id, format|
  "#{id}, #{format}"
end

# before
get "/foo/123/" #=> "123, html"
# after
get "/foo/123/" #=> "123, html"

Sorry, I made a mistake. Wait for confirming.


jvm args environment var

We need a way to set jvm args, most notably -XX:ExitOnOutOfMemoryError

Added 3 env vars to the example repo:
$XMX controls the JVM -Xmx argument.  Defaults to 512M
$XMS controls the JVM -Xms argument. Defaults to 64M
$JVM_ARGS additional arbitrary JVM args.
Additionally added the flag to die on OOM errors.

@Foxcapades @rbelnap  I wonder if we would benefit from adding this flag also, to help understand from the logs what we were dealing with: -XX:+PrintCommandLineFlags

I guess I'm wondering if we should just stick with this, and roll any memory config into the generalized JVM_ARGS param?
ENV JAVA_HOME=/opt/jdk \
    PATH=/opt/jdk/bin:$PATH \
    JVM_ARGS=""

COPY --from=prep /jlinked /opt/jdk
COPY --from=prep /workspace/build/libs/service.jar /service.jar

CMD java -jar -XX:+CrashOnOutOfMemoryError $JVM_ARGS /service.jar


feat(components): add reusable header and footer components

Icon imported from phosphor icon
Don't replace header and footer of landing page
If the component props type is too short, don't need to create a new file for this, let alone a folder

This is MR is solid to merge. However, there are still things to improve from comment


add note about provider language to readme

trying to unpack the term "provider"

cc @neolit123 how does this language look to you?


Issue on compiling 5.2.1

"monolithic headers should not be used by this module" , can not compile without specifically replacing this in the plugin code.

Everything works fine for me, I didn't have any errors or warnings. Besides, it's just a warning, it should still compile. Can you post the compilation log or the complete error lines?


async-coap-uri: Make AnyUriRef::write_to (and friends) unsafe.

From the rustdocs for AnyUriRef::write_to:

Safety
Calling this method is not unsafe, but implementing it is! The underlying guarantee is that the written URI reference SHALL be well-formed. If this method writes out something that is not a valid URI reference, the resulting behavior is undefined.

That this method is safe to use but "unsafe" to implement seems problematic in its current form.
This change creates a new automatically-assigned trait, AnyUriRefExt, and moves all of the methods from AnyUriRef that should not be overridden into it. The previous AnyUriRef::write_to method is made unsafe and renamed to AnyUriRef::write_to_unsafe. A new method, AnyUriRefExt::write_to is added that simply calls unsafe { self.write_to_unsafe(write) }.
This arrangement continues to allow the behavior of write_to to be changed, but instead of overriding it directly you must now override AnyUriRef::write_to_unsafe.
This change also deals with a similar problem with write_resolved, except in that case the entire method was simply moved to AnyUriRefExt to prevent it from being overridden at all.
Fixes #8

Unfortunately this diff is a tad longer than I would have wanted because I ended up doing a slight reorganization of some of the methods in AnyUriRef. Sorry about that.


[InputGroup] Automatic id binding if none provided

Describe the solution you'd like
It would be nice if we didn't have to put the same name attribute on both the InputGroup and Input components to get the label connected to the input.
The InputForm component could provide a unique id down to its child and be injected to use there.
// InputGroup.vue
provide('input-group', {
	id // eg. input-group-1k4hfe8sh
})

// Input.vue
const inputGroup = inject('input-group', { id: props.name })

This would take the usage of the InputGroup from this:
<form>
	<InputGroup name="name" label="Name">
		<Input name="name" />
	</InputGroup>
</form>

to this:
<form>
	<InputGroup label="Name">
		<Input />
	</InputGroup>
</form>

This is in no way an essential feature, but it would be a nice quality-of-life enhancement in my opinion.
Thank you for this awesome UI library! 🎉

Describe the solution you'd like
It would be nice if we didn't have to put the same name attribute on both the InputGroup and Input components to get the label connected to the input.
The InputForm component could provide a unique id down to its child and be injected to use there.
// InputGroup.vue
provide('input-group', {
	id // eg. input-group-1k4hfe8sh
})

// Input.vue
const inputGroup = inject('input-group', { id: props.name })

This would take the usage of the InputGroup from this:
<form>
	<InputGroup name="name" label="Name">
		<Input name="name" />
	</InputGroup>
</form>

to this:
<form>
	<InputGroup label="Name">
		<Input />
	</InputGroup>
</form>

This is in no way an essential feature, but it would be a nice quality-of-life enhancement in my opinion.
Thank you for this awesome UI library! 🎉


git submodules?

I noticed a long list of git clone steps in a recent commit.
Would it be better to use git submodules or gradle source sources for the various dependencies?

I went with cloning because this is the encapsulated project - so i was working on a gradle git checkout - so I could include the projects as gradle subprojects but i was hitting a few roadblocks.
I'm not tied to putting clone checkouts. I just haven't moved to anything else.
I'm assuming for gradle you're talking about https://blog.gradle.org/introducing-source-dependencies - which is preferable (to me).  I was using a different way before.  I just haven't gotten back to this.  Since I've now finished the build process I'm able to get back to cleaning up the build script.

Makes sense to me.
I haven't used the Gradle source repository feature yet.
What do you think of it's capabilities?
On Fri, May 21, 2021, 12:47 PM cdrchops @.***> wrote:

I went with cloning because this is the encapsulated project - so i was
working on a gradle git checkout - so I could include the projects as
gradle subprojects but i was hitting a few roadblocks.
I'm not tied to putting clone checkouts. I just haven't moved to anything
else.
I'm assuming for gradle you're talking about
https://blog.gradle.org/introducing-source-dependencies - which is
preferable (to me). I was using a different way before. I just haven't
gotten back to this. Since I've now finished the build process I'm able to
get back to cleaning up the build script.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CherokeeLanguage/cedproject/issues/1#issuecomment-846098138,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABH4XNCUDQTW4LAXWJMXWVTTO2FDZANCNFSM45FVVSHQ
.

I was using it before I decided to just clone.  it works nice.  it does essentially the same thing I'm doing except that all projects are now a part of the main project - so it does help compiling them all at one time instead of cd into a directory, running gradle, then moving on to the next directory.  it will check everything out into a sub-directory.  The reason the clones are in the build script is because I was having other issues and I just wanted to get the build script done so I could move on.  eventually, I'll go that route again now that the build script is complete.


[BUG] classNames are ignored in v5.13.0

Bug in v5.13.0: classNames like the Tailwind-Class bg-blue-900 are ignored somehow:
<Tooltip className="z-50 bg-blue-900" id="Home" delayShow={200}>
   Home
</Tooltip>

With v5.12.0 everything works just fine.

Hi, I tested here and the issue is:

CSS is being injected into the page

CSS works like objects, when we pass a custom class, this is the behavior:
Before v5.13.0

import CSS styles:
component receive a new class
browser process and render it

.tooltip-default-css {
  background: red;
}

.bg-green-500 {
  background: green;
}

--------- same as:

Object.assign({ background: 'red' }, { background: 'green' })

On version v5.13.0:

component receives a new class
CSS is injected into the page
browser process and render it

.bg-green-500 {
  background: green;
}

.tooltip-default-css {
  background: red;
}
--------- same as:

Object.assign({ background: 'green' }, { background: 'red' })

this is a bug, we will take a look as soon as possible. We are open to suggestions on fixing this because the CSS class from props and injected CSS have the same CSS specificity.
Solution 1 -> you need to increase the CSS specificity from your side - I don't like to force the community using our library to write extra code like this
Solution 2 -> we need to increase the specificity of the CSS received by props (how?) - I prefer this approach, but I don't have any ideas on how to do that at the momentn
reference: https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity

#1027 suggests a workaround

yeah, that workaround is just the dev increasing the specificity of his CSS query, it's the solution 1 that I mentioned.
another way to do that together with taiwlind is:
HTML - JSX
<div className="container">
  <Tooltip className="tooltip" />
</div>

CSS
.container .tooltip {
  @apply bg-green-500;
}

Hey guys, I don't think that's problem with the current version. Tailwind classes were ignored with<EMAIL_ADDRESS>This is why I had to create separate CSS class like:
.tooltipWrapper {
  @apply rounded-lgplus bg-tixy-1000 p-2 text-sm font-normal leading-4 text-tixy-10 shadow-sm;
}

or with the current version:
.tooltipWrapper[role="tooltip"] {
  @apply rounded-lgplus bg-tixy-1000 p-2 text-sm font-normal leading-4 text-tixy-10 shadow-sm;
}

Tomekmularczyk

There is definitely a problem with the current version 5.13.0. Everything works fine with version 5.12.0.

@danielbarion , how to completely remove default styling of tooltip? We use this lib hardly in our project, but every time I need to find some crazy way to override it's internal stylings :( Why just not add removeStyle prop for those who can deal with it low-level?

@NemeZZiZZ There's no equivalent to a removeStyles prop, as it would definitely be a rarely used feature. You can just use the styles prop to override whatever you need.
<Tooltip styles={myCustomStyles} />

Daniel did it on my request for v4 just before v5.
I think it's not so rare, you just have to create some crazy stuff to avoid style overlappings if styles are totally separated of code in your project and looks completely different.
style doesn't make you happy because you have no access to entire tooltip's markup and states this way.
I said this last year, will write it again:left and top calculations are more than enough to make tooltip work if you have completely different stylings

@NemeZZiZZ
That's totally understandable, but by our experience this hasn't been an issue to the package users.
You're welcome to open a PR with a rough idea of how you suggest we do this (doesn't have to be perfect, just enough to get the idea) so we can better discuss it.

hey @NemeZZiZZ @gabrieljablonski,
I need help here: https://github.com/ReactTooltip/react-tooltip/pull/1041
:)

@NemeZZiZZ please check: https://github.com/ReactTooltip/react-tooltip/pull/1041#issuecomment-1605726779

@NemeZZiZZ please check: #1041 (comment)

I'm sorry for being late with no support.
Well, I want to tell, that this just works. Maybe it's not so obvious in daily usage, but styleless and className-based re-styling fully can be achieved at last! That's great, thanks! Surely, it needs to be documented for others - but for now I just wait for next release with no patience :)
Thank you so much, Daniel!

Available at v5.16.0
I'll close this issue as resolved, @NemeZZiZZ and @ansgarsteinkamp  please feel free to open a new issue if needed,

In version 5.16.1, the issue remains unchanged. bg-blue-900 is still ignored.


Regenerate code for new version of Rust.

Please help keep the CHANGELOG up to date by providing a one sentence summary of your change:
Update generated code to apply rustfmt for rustc 1.34.0.

This needs to get in so Semaphore passes other PRs. 👍


ArgumentException with Example Code

Using the latest Nuget package (1.7.1) and the sample code in the Usage block of the Readme, I'm getting the following error on the Compile call:
"Additional information: Cannot bind to the target method because its signature or security transparency is not compatible with that of the delegate type."

I can't reproduce this... can you provide more info about your setup?

I am also having the same issue with 1.7.1 running Microsoft.NET 4.6 (not mono) and using the example from the front page.

When I run the test suite on Windows, it is using framework version 4.6.1586.0 and runtime 4.0.30319.42000 on Windows 10 without issue. Without more information about the specifics of your setup, I don't have enough information to help.

I had a similar issue and it turned out that my Visual Studio settings were the culprit. Make sure you have "Just My Code" setting checked in setting

I have the same issue.
To recreate you can create a console application and register an helper.
This line triggers the exception.

same issue here.


Option to only rebuild html ?

I was iterating on the css and most of the time I would be ok with sphinx not rebuilding the notebooks, iframe and jupyterlite environement, would there be a way to do so ? Would it be useful ?

Similar to #51, though #51 might be complicated to get the caching propoer. Here I'm thinking of an explicit option where you know you don't care and just want to rebuild the html.

Similar to #51, though #51 might be complicated to get the caching propoer. Here I'm thinking of an explicit option where you know you don't care and just want to rebuild the html.

I see, so the idea is that we wouldn't check if any examples sections with the directive have changed, and just keep the old, possibly out of sync, notebooks? I don't know enough to be sure how difficult it will be, but it sounds reasonable as long as it's documented that users need to ensure on their own that no examples sections have changed.


stop timer on view removal

This PR stops the NSTimer if the view is removed, to help prevent cpu being consumed in the background while not visible. It also cleans up some minor Xcode warnings.

Thanks! :bow:
Could you replace NSTimer with CADisplayLink if you could?

Done.  Not sure why the travis is saying it won't compile.  Works great for me, let me know your thoughts.

Awesome!! :clap: :clap:
Probably, travis build with Xcode6. I will fix it.


Use conversations.open in place of im.open

Resolves #1988
See https://api.slack.com/changelog/2020-01-deprecating-antecedents-to-the-conversations-api and  https://api.slack.com/methods/conversations.open

@benbrown is there anything I can do to help move this forward, please?

@ball-hayden I will include this in the next release.


Add Teachers' Digital Platform to optional apps and urls

This PR adds the Teachers' Digital Platform app to the optional apps in settings and to the urls with a TDP_RELEASE flag wrapped around it. The URL is currently temporary
Additions

TDP_RELEASE feature flag that enables the Teachers' Digital Platform url
Added teachers_digital_platform app to optional apps in settings
Added teachers_digital_platform to requirements/optional-public.txt

Testing

Go through the docker setup
Checkout https://github.com/cfpb/teachers-digital-platform into develop-apps/
Enable the TDP_RELEASE feature flag in the Wagtail Admin
Visit http://localhost:8000/tdp

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewer tool :arrow_right:

This PR should add wheel building: https://github.com/cfpb/teachers-digital-platform/pull/10

Wheels, now in two great flavors: https://github.com/cfpb/teachers-digital-platform/releases/tag/0.0.2

Wheels, now in two great flavors:

WOOHOOO! Two wheels are even better than one.

Is this waiting on anything?


timezone formats not supported correctly

The timezone parsing does not seem to follow the same standard that I can find on Wikipedia (https://en.wikipedia.org/w/index.php?title=ISO_8601&oldid=793312108#Time_zone_designators). Unfortunately, I cannot find any primary ISO source for the exact timezone format, so I have to rely solely on Wikipedia for the time being.
The page referenced in the code (https://github.com/Unidata/netcdftime/blob/1904de5c81fff7e40f3bbb23906dc2db83d7aaa1/netcdftime/_netcdftime.pyx#L33) is also not available any more. But judging from the archived version (https://web.archive.org/web/20161009150515/http://delete.me.uk/2005/03/iso8601.html), it suffers some - but not all of the format errors (see below).
What I think is wrong:

Colons are optional in the timezone specification (according to Wikipedia). Right now the timezone string "+01" cannot be recognized, even though this as well as "+0100" should be valid.
Both hour and minute specification should consist of exactly two numbers - not one OR two. This was actually done right in the JavaScript code quoted as reference. If this is fixed, then there won't be any ambiguities introduced by making the colon optional, either.

Both problems are relatively easy to fix by slightly modifying the regexes and maybe the parser code itself. If we can agree on the above points, I can work on a pull request to correct this.

some examples:
>>> import netcdftime
# correct: no timezone/UTC
>>> netcdftime._parse_date("2017-05-01 00:00:00Z")
(2017, 5, 1, 0, 0, 0, 0)
# works, but should not due to bad format because hour and minute fields are only of len 1
>>> netcdftime._parse_date("2017-05-01 00:00:00+1:0")
(2017, 5, 1, 0, 0, 0, 60.0)
# works as expected
>>> netcdftime._parse_date("2017-05-01 00:00:00+01:00")
(2017, 5, 1, 0, 0, 0, 60.0)
# does not work
>>> netcdftime._parse_date("2017-05-01 00:00:00+01")
(2017, 5, 1, 0, 0, 0, 0)
# also does not work
>>> netcdftime._parse_date("2017-05-01 00:00:00+0100")
(2017, 5, 1, 0, 0, 0, 0)

as not much seems to happen here and I need this to be fixed: any thoughts, @jswhit ? So far the fix for correct timezone handling has not been released, so changing the timezone format slightly should not introduce additional breakage for anyone.

pull request: https://github.com/Unidata/netcdftime/pull/18

closed via #18