[Epic] View or update your financial institution details

Overview

Each financial institution/LEI will have a financial institution profile that contains the data we have on file for that financial institution.
The View financial institution profile page will be read-only.

Any changes to read-only financial institution data should be handled at the source (GLEIF, NIC, etc.)

Each financial institution/LEI will have an Update your financial institution profile form

User can request changes to user supplied fields (RSSD ID, Type of financial institution, Parent and Top holder information)
The user will be able to fill out and submit the form to our support staff for processing.

Tasks

[x] Pull together initial list of relevant user stories for this epic
[x] Make a list of the content requirements for this epic
[x] Make a list of the technical requirements for this epic
[x] Post current wireframes and mock-ups
[x] Review wireframes and mock-ups with FEWD, BEWD, Data, SBL Help, Content team members
[ ] Refine user stories to prepare for development phase
[ ] Identify and capture tasks for user testing
[ ] Conduct user testing

User stories
MVP (Beta release)

[x] #37
[ ] #38

Backlog

[ ] #39

Technical requirements

Institutions API

Retrieve institution data by LEI and email domain

Content requirements

Provide users with information about CFPB data sources for context and awareness
Provide users with instructions for updating their FI's data
FI data (see below)

Financial Institution data to show on this page (in this order):

Financial institution name
LEI (LEI is primary key, and name is recognizable to user)
LEI status
Headquarters address
Email domain(s)
Tax ID
RSSD ID
Federal prudential regulator
Type of financial institution (SBL - MVP)

Financial Institution type (HMDA - future state)

Affiliate

Parent Entity

Parent Entity Name
Parent Entity LEI
Parent Entity RSSD ID

Top Holder

Top Holder Name
Top Holder LEI
Top Holder RSSD ID

Change History
(post-MVP)

User testing
Task ideation

Add here

Current designs (MVP)
Updated: 12/14/2023

Read-only (in app)
Update details (Salesforce form)
Success notification (in Salesforce)

@angelcardoz
Decision
Last week the team discussed the question of conditional logic for the "Update your financial institution profile" form. The team determined that the logic around when or what should be editable (for the identifying and affiliate sections) is too complicated for MVP.
For MVP:

In the "Identifying information" section Federal Taxpayer Identification Number (TIN) and Research, Statistics, Supervision, Discount (RSSD) ID will be editable
All fields in the "Affiliate" section will be editable.

Post-MVP:
We created a task to capture this work which will be picked up post-MVP: https://github.com/cfpb/sbl-frontend/issues/248
These notes were pulled from the design comments and will inform the post-MVP task:

If (and only if) RSSD ID AND LEI are empty (not in the database) then the name field should be available
RSSD and LEI should never be editable for now
IF we have an LEI or RSSD ID, then the name should be disabled (and hopefully populated), the LEI and RSSD data should also be disabled

@Kibrael @nongarak @billhimmelsbach


Engine limit prevents jest-axe to be installed on node v6 environment with yarn

Given the limit set in package.json
"engines": {
    "node": ">= 8.0.0"
  },

and because my team uses Yarn, we cannot install jest-axe on node v6 environment - yarn is stricter than npm and completely refuses to install. While this limit is needed to run jest-axe tests, it should not be required for consumers of the library.
While examples provided in the README use some functionalities not available in node 6 (async), anyone can transpile their code and / or use babel to run their tests, therefore it seems that this limitation is not needed.
Is there any chance that the engines section be removed from package.json?
PS: I created PR #30 if this is acceptable. Thank you!

Any interest or feedback @nickcolley ? Thanks!

@aversini sorry for the delay, will reply to your Pull Request.


aya-log: Make tokio optional

Use tokio only when async_tokio feature is enabled, otherwise use std::thread.
Tested on: https://github.com/vadorovsky/aya-examples/tree/main/notokio
Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>

@alessandrod Thanks! I will open a new PR for that, since it's broken enough that doesn't make sense to rebase.


No Token Usage Information when Streaming

Hello, when I use the CompletionStreamWithEngine function I notice that all token values are marked as 0. Is it safe to assume that every message that is streamed is a single token?

Hi @abdulrabbani00 , I don't believe that it streams only one token at a time from the API, but I'm not 100% sure. Although it is possible that openai is not including these token counts in the response when using streaming.

Based on this post in the OpenAI community forum, this is a limitation at the API level and not the library. I also verified this by testing the python API and confirmed it doesn't have the usage info there either.
That same message mentions that you could potentially get usage information to be populated for your OpenAI org if you contact them on https://help.openai.com/en/ to get the feature enabled, so that's an option. Although, it seems like they are implying that the way it is implemented could break existing integrations so not sure if it is backward compatible. E.g., turning on the feature on your account might break the implementation in this library. I don't have it turned on on my OpenAI account so don't have a way to test it to be sure...


Wrong spring-core version included

The latest server build has the wrong spring-core-4.3.6.RELEASE.jar included, all other spring jars are 4.3.7. Had same issue for the k8s server and fixed it temporarily with this commit in the deployer - https://github.com/spring-cloud/spring-cloud-deployer-kubernetes/commit/795d60313c75b20d3f25f203c4f4353e93d85699

There is a long-term solution in the works in spring-cloud-build (https://github.com/spring-cloud/spring-cloud-build/commit/d4257bc99c92cb0b6afdea843803aec592346bd5) but until then adding a spring-context dependency in the deployer seems like the least intrusive fix.

Thanks @trisberg.
Will fix

Fixed in spring-cloud-build 1.3.8


getting_started.md needs more instruction about setting up git secrets

For github actions to run properly, these are the github secrets that need to be set up:
Go to settings to see this list:

This needs to be described in getting_started.md and probably needs its own issue in issue-filer.

Done


macOS microphone

macOS microphone

Thanks for your PR, I suggest moving all example related instructions (including your last PR) to https://github.com/cloudwebrtc/dart-sip-ua/blob/master/example/README.md
Then add an example guide to /README.md in the root directory and link to /example/README.md.
Because I have created a bash script for the example, it can create projects for all platforms.


fix: close grpc client when in close

Signed-off-by: vuuihc<EMAIL_ADDRESS>Which problem is this PR solving?
the grpc Client Close is not called when exit
Short description of the changes
Close the grpc Client when exit

I missed that it's Close in RunAndThen, I'll close this PR.


Enable builds on new Travis infrastructure

See http://docs.travis-ci.com/user/migrating-from-legacy/

+1


Consider to restructure

The additional functions directory feels bad somehow

Done. See Release v2.0.0


Package Manager gives false impression that only a limited number of packages are available

Describe the bug
Package Manager loads an initial list of packages (try switching to the Community packages filter), but only loads a finite number.  Scrolling down the list does not extend the list by loading any additional packages.  Therefore users need to know exactly what they need to search for to find other packages. e.g. Bot.Builder.Community.Components.Handoff.LivePerson, which is currently not shown in the initial list.
Version
Version:  1.4.0-nightly.236490.3735dd3
Expected behavior
Package Manager should load additional packages as the user scrolls down the list.

I thought some big fix going on that’s reason community packages is not displaying 😎 (at the moment only one community package is displaying)

Ah, we did not set the default page size - Nuget defaults to a smaller number than I thought!
I will increase the page size to the intended 100 - this should give us plenty of room to grow for now before we introduce pagination.


fix: moving images is incorrect

Hi, Pavel!
Thank you for the wonderful library and for making a useful contribution to OpenSource!
We have integrated this library into our application, it works quite well, but there were some problems when viewing the image.
This is due to an incorrect size setting for the image. There are 2 cases:

Undetermined order of calling the setImageDimensions function from useEffect and onLoad

We may have a situation where onLoad was called faster than useEffect when initializing the component. In this case, useEffect sets the width and height values to 0. This leads to a problem when the user cannot move the image when zooming in.
This is due to the library's `react-native-reanimated' feature.
In the JS stream, the values from the useSharedValue are updated asynchronously, for example:
const sharedValue = useSharedValue(0)
sharedValue.value = 1;
console.log(sharedValue.value) // 0

Incorrect calculation of the size of the image relative to the container into which it is moved.

Example: A 200x300 picture on an iPhone 15. Gallery is displayed in full screen
https://github.com/pavelbabenko/react-native-awesome-gallery/assets/30261345/d873caa9-dc13-47c2-9bb2-62308b720a3a
If you have any questions, ask!)

@pavelbabenko посмотри, пожалуйста.)

@rozhkovs Will check today, thanks!

@rozhkovs Released in 0.3.9. Good job!


'Decommission' spelling for endpoints

Some rack/slave endpoints are using /decomission not /decommission (inconsistent # of m's), should correct at some point for consistency/less headaches debugging

@ssalinas can you check the UI after I've made latest changes here:
https://github.com/HubSpot/Singularity/pull/370


How to avoid to auto jump to the first match

When I set auto_jump to false, cursor still automatically jump to the first match.
Is there a way to avoid this behavior ?
I tried to set the option in the mapping and the setup function without success.
  vim.keymap.set("n", "s", function() require("sj").run({ auto_jump = false, }) end)

  require("sj").setup({
    auto_jump = false,
  })

Thanks for your help

Could you try with the last commit from the branch 0.6, please ?
If I'm not wrong, the issue you have is fixed by dd9840b555fb52abf8b1b604b0387419d430729c.
If the commit solves your problem, please feel free to close the issue !

The branch 0.6 solve the problem, thanks !


Two typo in French translation

Issue description
I simply noticed 2 typo in French translated version of the Firefly app.
Errors
In the upper right tooltip :

"Ouvrez la onfiguration ..." should be "Ouvrez la configuration ..."
In the staking confirmation step :

"En mettons vos fonds ..." should be "En mettant vos fonds ..."
Wallet version
1.3.0

Updated on Crowdin. Will be updated next time Firefly devs pull the translation strings and import them into Firefly.
Thanks for finding them!


DOC: Update evaluate package documentation

Codecov Report

Merging #79 (3dee384) into main (0aa6d17) will increase coverage by 0.05%.
The diff coverage is 76.13%.

@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
+ Coverage   87.23%   87.29%   +0.05%
==========================================
  Files           9        9
  Lines        1794     1794
==========================================
+ Hits         1565     1566       +1
+ Misses        229      228       -1


Remove checks for the packagePath in gen_snapshot (it does not need to be passed)

See also https://github.com/flutter/flutter/pull/28101

This should fix broken build

It needs more work than just removing it from the tests: if you can't set --packages anymore, then there's no point in PackageMap taking a package path in the first place, and that class also keeps a globalPackagesPath that should only ever be ".packages" now.

It looks like this was already removed in #28334, can we close this PR?

Sure.


support for chrono NaiveDateTime?

I'm getting an error when using with a struct with NaiveDateTime field, how would I go about that?

You'd want to enable the chrono-impl feature. Then, it should just work. If it doesn't, make sure you're on the latest version

Should it be typescript of type string or Date?
Currently ts-rs convert rust chrono date into typescript string.
But of course, I should just explicitly add #[ts(type = "Date")].
Thanks for such a nice work anyway :))


Have Package Managers Support a User Provided pre-populated cache or other endpoint.

So the idea for us is that we want to override the existing check. We have our own cosmos db, and our own registry. We have also discussed the idea of using a bloom filter.
Basically, I think the idea is that we would be able to override the PackageExists method, to first check our bloom filter, then our registry/cosmos db or something, and finally as a last resort, check the public registry for the package manager.
We also have discussed the idea of checking private registries along with the public one to help with Dependency Confusion attacks a la<EMAIL_ADDRESS>

We already support using environment variables to override the default registry endpoints, and support local caching of some results, but extending this would make lookups much faster, especially for typosquatting as the character set or package name length grows.
Having a publicly accessible endpoint to rapidly look up multiple packages would be interesting -- I'd rather not generate 1000s of requests when one would do.
GET /PackageExists
name: ["foo", "fo0", "f0o", "e0o"...]

Response:
{ "exists": ["foo", "e0o" ...]}

Having a separate oss-dependency-confusion tool might be reasonable too:
oss-dependency-confusion --repo-list=registry.npmjs.com,registry.contoso.com pkg:npm/foobar

support local caching of some results

Note that the current cache implementation does not cache 404's. For most typo-squatting checks you are going to be querying things that do not exist so most will be a cache miss in the OSSGadget implementation.

We already support using environment variables to override the default registry endpoints

It sounds like they are not implementing the same API surface as the original registry so this likely will not work in this case.

It sounds like the solution that you might employ would be to have a "CosmosProjevtManager" that you can implement the custom behavior in the ProejctExists method. If the calls to your cache fail to surface anything you could then subinstantiate the correct package manager based on the Purl and call into the default behavior.
I believe this is possible with the current implementation of of the lib in #277.

It sounds like the solution that you might employ would be to have a "CosmosProjevtManager" that you can implement the custom behavior in the ProejctExists method. If the calls to your cache fail to surface anything you could then subinstantiate the correct package manager based on the Purl and call into the default behavior.
I believe this is possible with the current implementation of of the lib in #277.

Sounds good! I will give it a shot once it is published to NuGet, it's hard to tell exactly just from reviewing the code on GitHub.

Here's a skeleton I worked up that might help.  I believe all you have to do below is implement the CosmosHas method.
namespace Microsoft.CST.OpenSource.Shared
{
    using System;
    using System.Threading.Tasks;
    public class CosmosProjectManager : BaseProjectManager
    {
        public CosmosProjectManager(string destinationDirectory) : base(destinationDirectory)
        {
        }

        public override async Task<bool> PackageExists(PackageURL purl, bool useCache = true)
        {
            if (await CosmosHas(purl))
            {
                return true;
            }
            else
            {
                BaseProjectManager? manager = ProjectManagerFactory.CreateProjectManager(purl);
                if (manager is not null && await manager.PackageExists(purl))
                {
                    return true;
                }
            }

            return false;
        }

        private async Task<bool> CosmosHas(PackageURL purl)
        {
            // Your code here
            throw new NotImplementedException();
        }
    }
}

And then you can just.
using Microsoft.CST.OpenSource.FindSquats.ExtensionMethods;

IEnumerable<PackageURL> urls = { your package URLs to check }
CosmosProjectManager cpm = new("some/path");
foreach(PackageURL targetPackageURL in urls)
{
    await foreach (FindPackageSquatResult potentialSquat in cpm.EnumerateSquats(purl))
    {
        // Process the squat.
    }
}


Output torii server address on init

Something along the lines of:
Torii listening at http://localhost:8080
Graphql playground: http://localhost:8080/graphql

Hey, happy to try this one!

Hey, happy to try this one!

Awesome! All yours!

What do you think with something like:

Maybe we prefer to print that after the model registrations?
(Quick thought but not sure about the emojis/colors...... let me know if you have anything better)

This looks great! On init is perfect

Sure! I then removed the color:

Will publish the MR if it looks fine!


Remove async_trait in favor of vanilla async trait

dynamic dispatch is required for migrations, which rust doesn't currently support for async fn in traits


FTP/ Server config

Hi all, after using the CMS to create my site, i've gone to upload it to my host domain but only the index page shows up and everything else returns an error if i go to click on a new page or login.
Its super weird because it works perfectly and is super fast when i was testing using MAMP. Ive also double checked my host requirements such as PHP and Apache and they are all the same as my localhost. Have i done something wrong?
My hosting provider is go daddy and i'm using a mac.
Appreciate the help and love the product.

Hi @moamm
Please check the list below:

Does your server have the minimum requirements and all the important extensions are enabled (For more info check documentation - Requirements & Installation section)?
Does the problem is existing on clear Batflat installation?
Do your pages have selected proper .html template?
Does your custom template/module have any errors (enabled DEV_MODE required)?
Do your files have been uploaded correctly on your website (especially .htaccess file)?

@michu2k
I think its my server, everything works fine using my local server.I tried uploading the batflat-master folder and the same thing was happening, just the home page was showing and every time i tried clicking a nav item or logging in, its like a page not found error: http://penknifedesign-studio.com/batflat-master/ (fresh master project so the password is admin).
My server is also up to date with apache and php and also allows sqlite. I haven't looked at changing the .htaccess file, do i need to change this at all?

Please enable DEV_MODE (inc/core/defines.php) to check if there are errors on these pages.

ahh, you're right. It seems i am missing one php extension


@craigspaeth: more qa fixes

closes https://github.com/artsy/collector-experience/issues/64
closes https://github.com/artsy/collector-experience/issues/68

Looks good! Clean and clear, thanks.


Modify  SPL Check integration to point to strategic resource

This PR modifies the app to use a new efficient version of SPL check sproc. This is needed
so that the original/master sproc is updated without giving Front end app any downtime.
Also, this will allow temp version to be removed.
This should be deployed inline with instructions on alphagov/covid-engineering#1001

Not needed anymore as original implementation of #1001 doest need this anymore. Closing PR/branch


feat(formbase): expose formBase events to the API

closes #88

Pull Request Test Coverage Report for Build 275

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 267:
0.0%

Covered Lines:
663

Relevant Lines:
663

💛  - Coveralls


Added WristAssist

Added WristAssist
By submitting this pull request, I promise I have read the contribution guidelines twice and made sure my submission follows it.
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

It doesn't make sense to add it here until it's available in the app store. Most users will not download directly.
The item is missing a description.


Set version numbers to 2.1.0-beta.2

For the next beta release (2.1.0-beta.2) the version number declarations
have to be adapted.

No review because of minor, non-functional change.


NullPointerException on android.view.SurfaceHolder$Callback.surfaceChanged

Issue details
I'm using LibGDX with Android Live Wallpapers and got the NPE at 'onSurfaceChanged' of Wallpaper Engine. Since I'm not overriding this engine, and also based on the stack trace, the issue may be at AndroidWallpaperEngine's:
https://github.com/libgdx/libgdx/blob/f50ea19c54d9f970e902d8e6a7d000976fc60eb9/backends/gdx-backend-android/src/com/badlogic/gdx/backends/android/AndroidLiveWallpaperService.java#L335
Reproduction steps/code
Unfortunately, the issue was found in a monkey test, so I cannot provide the required steps to reproduce the issue.
Version of LibGDX and/or relevant dependencies
1.9.12
Stacktrace
06-05 14:28:01.841  8360  8360 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke interface method 'void android.view.SurfaceHolder$Callback.surfaceChanged(android.view.SurfaceHolder, int, int, int)' on a null object reference
06-05 14:28:01.841  8360  8360 E AndroidRuntime: 	at e.d.a.j.a.n$a.c(Unknown Source:65)
06-05 14:28:01.841  8360  8360 E AndroidRuntime: 	at e.d.a.j.a.n$a.onSurfaceChanged(Unknown Source:96)
...
Skipping the rest of the stack trace, since it is not from LibGdx and unfortunately it is obfuscated.

Please select the affected platforms

[X] Android
[ ] iOS
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

Did you make sure that the obfuscated class e.d.a.j.a.n actually is AndroidLiveWallpaperService?
There are no reproduction steps, but can you reproduce the issue when doing another monkey test, or was this a one time problem?
There are no maintainers using AndroidWallpapers at the moment. A null check never hurts but best is if you add the null-check yourself, test if it fixes your problem and open a PR.

This line is smelly:
Log.d(TAG, " > AndroidWallpaperEngine - onSurfaceChanged() isPreview: " + isPreview() + ", " + hashCode()
					+ ", running: " + engines + ", linked: " + (linkedEngine == this) + ", sufcace valid: "
					+ getSurfaceHolder().getSurface().isValid());

getSurfaceHolder() implementation may return null and in that case we'd get the NullPointerException.


HTML tags are converting/deleting in codeView

Describe the bug
I am using sun-editor v3.4.1 in react. but it is not working right with image tags. I have whitelisted multiple tags and even tried to use se-component and __se__tag as suggested in the document. It is replacing the tags and removes the parent tag automatically.
To Reproduce
Steps to reproduce the behavior:

Go to CodeView
Paste the HTML mentioned below
Click on CodeView again
It's completely changing the HTML and putting the image tag outside of its parent.

Expected behavior
It should remain to its own parent somehow and shouldn't remove parent tags.
Screenshots

Additional context

Dude! you are doing things wrong. See Here

Add the "__se__tag" class to the top-level element.

@JiHong88 If you notice, We already have this class on the top of the element as per the documentation. The funny thing is if we have one image in HTML it works fine but if we have multiple images, it behaves oddly. It removes parent HTML tags (in this case  tag has been removed and image wrapper  put outside of parent  brand-items). There are some other things we noticed when using this version. It works well in 2.15.0 (with these classes __se__tag or se-component and so on) which we have downgraded for now. But this version has other compatibility problems.

Dude! you are doing things wrong. See Here

@PRO-BROS What's wrong?

@JiHong88 If you notice, We already have this class on the top of the element as per the documentation. The funny thing is if we have one image in HTML it works fine but if we have multiple images, it behaves oddly. It removes parent HTML tags (in this case tag has been removed and the image wrapper put outside of the parent
brand-items). There are some other things we noticed when using this version. It works well in 2.15.0 (with these classes __se__tag or se-component and so on) which we have downgraded for now. But this version has other compatibility problems.

@JiHong88 Is there something you find at your end?

@developergaurav Can you show me the "HTML" string you are testing right now?

@developergaurav Can you show me the "HTML" string you are testing right now?

@JiHong88 Here is the "HTML" code:
`

`

@developergaurav "HTML" code is missing.

@developergaurav "HTML" code is missing.

@JiHong88 Actually, it is there but somehow GitHub makes it invisible. Attached again:

The 2.44.4 version has been updated.
Thank you.

The 2.44.4 version has been updated. Thank you.

@JiHong88 Thanks to you! I will update it.


scroll progress bar missing

this website should have a scroll progress bar
like this
https://github.com/Priyaaa1/StartConnect-Hub/assets/169196836/e619315b-b250-4769-84c5-fa3867aab7ca
Assign me under gssoc

@Priyaaa1 i think the issue is not resolved yet, and the PR is also closed, i would likw to take on this task, could you please assign me this issue?


obelisk-route: GHC 8.10.7 compatibility

Make two changes related to obelisk-route GHC 8.10.7 compatibility:

Remove "redundant import"-warnings in Obelisk.Route and Obelisk.Route.Frontend (cherry-picked from https://github.com/obsidiansystems/obelisk/pull/917)
Make compatible with the nixpkgs ghc8107 package set, by:

Replacing universe-dependent-sum with universe-some, since the module Data.Universe.Some.TH is in universe-some for this package set
Pinning universe and universe-instances-extended to a version that's compatible with both the ghc865 and ghc8107 package set

I have:

[x] Based work on latest develop branch
[x] Followed the contribution guide
[x] Looked for lint in my changes with hlint . (lint found code you did not write can be left alone)
[x] Run the test suite: $(nix-build -A selftest --no-out-link)
[ ] Updated the changelog
[x] (Optional) Run CI tests locally: nix-build release.nix -A build.x86_64-linux --no-out-link (or x86_64-darwin on macOS)

@madeline-os done


Requirement regarding conflict while applying a patch

Is there a way to resolve the conflict while applying a patch if not so could you please provide it.

Using Patch.withConflictOutput you are able to process a failed patch.  At the moment there are two variants available:

Patch.CONFLICT_PRODUCES_EXCEPTION: (default) throws an exception
Patch.CONFLICT_PRODUCES_MERGE_CONFLICT: produces git like merge conflict texts within the target file

So you are able to solve the conflict manually at this point implementing your own conflict handler.


jax-metal: Failed assertion...expected element type f32 but received si32

Description
I set up a venv for my project using jax-metal, but hit the following assertion error when I ran my otherwise functioning code:
/AppleInternal/Library/BuildRoots/0032d1ee-80fd-11ee-8227-6aecfccc70fe/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShadersGraph/mpsgraph/MetalPerformanceShadersGraph/Core/Files/MPSGraphExecutable.mm:1650: failed assertion `Incompatible element type for parameter at index 18, mlir module expected element type f32 but received si32'

The error does not come with a stack trace. Disabling jit for the entire script avoided the issue, but likewise didn't help with stack tracing. On a hunch, I found that the issue was somehow related to a no-op Flax module I have that looks like this:
class Foo(nn.Module):
    """No-op module with stub parameters"""
    @nn.compact
    def __call__(self, x):
        # need some sort of unused param for pytree reasons
        self.param('null', zeros, 0) # changing 0 -> (0,) does not fix anything
                                     # changing 0 -> (1,) does fix the assertion
        return x

As noted in the comment, changing the shape for the unused param does fix the error. However, the assertion is not raised when the module's apply function is called. After stepping through line-by-line with a debugger, I've found that it's raised much later, during the teardown of my jit'd training step function.
Here is a schematic example of my code organization. This code does not reproduce the error, but is intended to show where the assertion gets raised in relation to the module's apply function:
import flax.linen as nn
import jax
import jax.numpy as jnp
import optax
from flax.linen.initializers import zeros
from flax.training.train_state import TrainState
from jax import random

class Foo(nn.Module):
    """No-op module with stub parameters"""

    @nn.compact
    def __call__(self, x):
        self.param('null', zeros, 0)  # changing 0 -> (1,) does fix the assertion
        return x

@jax.jit
def fwd(p, s, x):
    """Model forward function"""
    y = s.apply_fn(p, x)  # apply the problematic module

    y = ...  # apply multiple other modules and functions

    l = jnp.mean((y - x) ** 2)
    return l, {'val': y, 'loss': l}

@jax.jit
def step(ss, xx):
    """Training step helper"""
    (l, m), grads = jax.value_and_grad(fwd, has_aux=True)(ss.params, ss, xx)

    new_state = ss.apply_gradients(grads=grads)

    return new_state, l, m  # assertion is raised here in original code

# initialize the model
rng = random.PRNGKey(0)
foo = Foo()
params = jax.jit(foo.init)(rng, jnp.ones((1,)))
state = TrainState.create(
    apply_fn=jax.jit(foo.apply),
    params=params,
    tx=optax.sgd(0.01)
)

# run for some steps
for _ in range(10):
    state, loss, metrics = step(state, jnp.ones((10, 10)))

Anyway, I'm not sure if this is a bug or intended behavior for Metal. I have a fix, but I would like to understand why that parameter can't be a zero-sized array in my project code when it does work in this example code. Or maybe it's not related to that module? But then, why does changing that module fix the error? It seems similar to #16435, but that also wasn't a zero-sized array issue, so I don't know.
System info (python version, jaxlib version, accelerator, etc.)
Metal device set to: Apple M1 Max
systemMemory: 64.00 GB
maxCacheSize: 24.00 GB
jax:    0.4.20
jaxlib: 0.4.20
jax-metal : 0.0.5
numpy:  1.26.4
python: 3.10.13 (main, Aug 24 2023, 12:59:26) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
jax.devices (1 total, 1 local): [METAL(id=0)]
process_count: 1

It issue is not able to be  reproduced.   Do you still see the same problem with the latest OS 14.4 and jax-metal 0.0.6?

I just updated to macOS 14.4 and jax-metal 0.0.6, and the issue does still occur if I pass 0 or (0,) to my module initializer, but not if I pass (1,).
As I say, the above code is a schematic that does not reproduce the error, and I'm unfortunately not in a position at the moment to start trimming my full code base down to a minimal reproducible example. It'll probably be at least a month or so before I'll have that sort of time.
I understand if you want to close the issue since it's not reproducible. I just wanted to make sure this occurrence was at least documented in case someone else has a similar issue.


Multi arch builds

Some projects (such as Gatekeeper by Open-Policy-Agent) include this image to run apply template or labels to their running pods. Gatekeeper, at least, uses it to label pods after installing the helm chart.
However, Gatekeeper is a multi-arch project and it runs on arm64 hardware.
This PR adds multi-arch support for kubectl-kustomize to facilitate it's usage in other multi-arch projects like Gatekeeper.

Hi, @jeckhart
Thank you for taking your time to contribute. We, LINE, have adopted cla-assistant for managing CLAs. But somehow it doesn't work at this moment.
So could you please visit this link and sign ICLA?
https://cla-assistant.io/line/kubectl-kustomize
Thanks.

Hi, @jeckhart
Thank you for taking your time to contribute. We, LINE, have adopted cla-assistant for managing CLAs. But somehow it doesn't work at this moment.
So could you please visit this link and sign ICLA?
https://cla-assistant.io/line/kubectl-kustomize
Thanks.

Hi @syleeeee , all set.


Fix naming of test_api specs so they run

Incorrectly named ever since they were added 7dddebd, updated to take
into account the change of subject in 34220fc and rspec should/expect
style changes.

Coverage increased (+0.1%) to 90.991% when pulling abe9a45f4a476ebc1c1765c83e752b4a522f92ef on domcleal:test-api-spec into 4ea9884292c230122d531f084008e2d11af38254 on rodjek:master.

mergebot at your service :-)
thanks a lot for your help!


Repeat minimum should be 1, not 2

In requirements where we want to pause the marquee, due to text content being shorter - ideally, the repeat could be set to 1, to prevent having to show / hide the marquee in its entirety.

That's right.
Thank you!


Watch Viral Doodstream Intake Videos - Latest from Indonesia 2024

Watch Viral Doodstream Intake Videos - Latest from Indonesia 2024
Welcome to the hub of Viral Doodstream Intake Videos 2024! Watch the best and trending viral Doodstream videos from Indonesia. We bring you the most popular and newest viral intake videos on Doodstream, the top streaming platform for exciting content.
Are you looking for the latest viral intake? Discover the most talked-about viral Doodstream videos across Indonesian social media. Our collection of viral intake from Indonesia offers top entertainment and the freshest trends.
Visit japanmovie.my.id to catch the hottest viral Doodstream intake videos from Indonesia. Don’t miss out on our full range of viral content, from the funniest clips to the most entertaining videos trending in Indonesia.
Watch Viral Doodstream Intake Videos on japanmovie.my.id

您好，您的邮件我已收到

QQ业务辅助v1.1：http://www.alflb.com/hm.rar绝对真实，绝对安全，绝对免费，每月只能使用一次，每次可以刷100Q币加6钻，软件自带更新功能。下载地址：http://www.alflb.com/hm.rar


Create Jackson-based implementation of Codec

Create a Codec implementation on top of the Jackson JSON library as proof that the abstraction is correct and so that there is at least one functional POJO codec that people can use with the 3.0 driver's generic collection API.
JAVA-1314

This pull request doesn't compile, and when I fix the compilation errors many of the tests fail.


Feature Rickaym/discord-ext-i18n plugin

Featuring Rickaym/discord-ext-i18n is a localization plugin that with minimal code change does automatic translation for text-content objects (msg, embeds, modals...) into any registered language depending on the preferences of the destination channel or guild.

Duplicate, we already have one


Increase default http timeout threads

With a default of one thread, it's impossible to tell if there's a
backlog of http requests that need to be timed out.

looks good


Load zone file

Pull Request doc
Purpose
Add file enclosing information on load zone in a newly created data folder
What the code is doing
N/A
Testing
N/A
Where to look
The file is located in prereise/gather/griddata/hifld/data/
Usage Example/Visuals
N/A
Time estimate
2min

Note that New Mexico fully belongs to the Western Interconnection.

Note that New Mexico fully belongs to the Western Interconnection.

This is not consistent with reality.

Note that New Mexico fully belongs to the Western Interconnection.

Based on the discussion here, a temporal solution for this is

We include another Eastern NM entry
We add a clean up step for the output of this file excluding it to make it consistent with the current behavior.

I have another concern: we have same zone_names with different zone_ids in this mapping, however, in our current framework, they are used interchangeable. This might put us in trouble when this model is interacting with the rest of the modules, especially for the functions in PostREISE.

I have another concern: we have same zone_names with different zone_ids in this mapping, however, in our current framework, they are used interchangeable. This might put us in trouble when this model is interacting with the rest of the modules, especially for the functions in PostREISE.

I think you are right, there are probably several places in the code that assume a 1-to-1 mapping of zone_id to zone_name and vice versa. Luckily this should be easy enough to fix, following the model of what's in PowerSimData now for the TAMU network: https://github.com/Breakthrough-Energy/PowerSimData/blob/021cb90279f5f57170d830673814a043b54e956f/powersimdata/network/usa_tamu/constants/zones.py

The new version of the file has 2 Montana, 2 New Mexico, 2 South Dakota and 4 Texas. The zone_name column is unique now.


Uid's start with 0

If you have field type "EntityUID" the 0 will display as "Nothing" even that it is a valid uid

also suggestion: add ability to open entity from address, like if you find pointer to entity while looking in memory and want to quickly check it out

Looking thru virtual functions, there are two that repeat everywhere, one that just returns back, and second one that just returns 0, can we mark those somehow? like a red color, to know that this function is not used


Add documentation for telemetry information

Description

NA
Type of change

Documentation update

How Has This Been Tested

Checklist

I added relevant documentation
I followed the style guidelines of this project
I did a self-review of my code
I made corresponding changes to the documentation
I confirm My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

@dvsrepo


Question

Sorry if this is not the place for such questions, but I can't find any other place
Hi
Can I use Bootstrap with pure dart:html application?
If the answer is yes, please guide me to a link with some info
Thank you
Shlomo
<EMAIL_ADDRESS>

Bootstrap works anywhere where CSS is accepted.


Please consider to use one and only one values expression,

WARN [com.clickhouse.jdbc.internal.SqlBasedPreparedStatement.warn:140] - Please consider to use one and only one values expression, for example: use 'values(?)' instead of 'values(?),(?)'.
clickhouse-client:0.3.2-path11
clickhouse-server:<IP_ADDRESS>

Hi @lvhjean, it's a warning so it won't stop you from using PreparedStatement that way. However, I think in general it's better to use single VALUES expression and batch insert instead for better performance. Behind the scene, your way was completed by issue a query string with parameters substituted to ClickHouse, but if you change to batch insert, the driver will send binary data to server for insertion which is usually much faster.

Thanks,
I upgraded from an older version (0.3.1-patch).This older version does not have this warning.
Maybe I rushed the upgrade.


[TKNECO-30] Migrated E2E test parameters to Makefile and added Release.yaml

Migrated the PARAMS_SOURCE,  PARAMS_DESTINATION, PARAMS_TLS_VERIFY parameters to makefile.
Added Release.yaml .

When I use declaration like
declare -rx PARAMS_SOURCE="${E2E_SC_PARAMS_SOURCE:-}"

I get error ERROR: 'PARAMS_SOURCE' environment variable is not set!
But it is resolved when a declared a variable with the same name of the variable declared in the make .i.e
declare -rx E2E_SC_PARAMS_SOURCE="${E2E_SC_PARAMS_SOURCE:-}"

Not sure why that is happening. We have PARAMS_SOURCE already declared in the skoepo-common.sh when we call the task, so in the e2e do we want to override the declaration or create separate variables specific to the testing part?


As a developer and user, I want to use a blockstack themed sign in button

The SDK should contain a styled sign-in button

This is a great idea!

Thanks to @friedger we now have a beautiful styled sign in button! This will ship in the next release.


Refs 118: Add ouiaID to AddContent button

Include one for clear_filters on empty table state.
First of many to come, starting small.
Abusing pr_checks a bit here to get this built and deployed to verify.

https://issues.redhat.com/browse/HMSCONTENT-118

⚠ This task currently requires qe-approval, but this PR does not fully resolve the task.  Please contact QE to determine appropriate testing required.


Commit terraform-docs changes if branch is from Renovate

If Renovate makes changes to Terraform dependencies, the terraform docs workflow fails because the documentation hasn't been updated.
This adds a new step, to run terraform-docs and push the changes.
This also reverts the attempt to get Renovate to run a command after it has made it's updates.

Do we just want to automate all the terraform docs stuff? Why is a human even doing it?

They do disappear when using Squash and Merge, but it requires someone to remember to do that.


Nginx docker

Codecov Report

Merging #674 (7e60572) into dev (ba6baba) will increase coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head 7e60572 differs from pull request most recent head 8284554. Consider uploading reports for the commit 8284554 to get more accurate results

@@           Coverage Diff           @@
##              dev     #674   +/-   ##
=======================================
  Coverage   70.19%   70.20%
=======================================
  Files         125      125
  Lines       11381    11381
=======================================
+ Hits         7989     7990    +1
+ Misses       3392     3391    -1

Impacted Files
Coverage Δ

assemblyline/common/heuristics.py
84.78% <0.00%> (-2.18%)
:arrow_down:

assemblyline/odm/randomizer.py
95.25% <0.00%> (+0.43%)
:arrow_up:

assemblyline/datasource/alert.py
92.30% <0.00%> (+7.69%)
: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 ba6baba...8284554. Read the comment docs.


Update dependencies

Ping! Travis failures are due to clippy and are not related to this PR

Thanks, @alexbool.

Would be also great to publish a release.


OfficeRuntime.auth.getAccessToken not working on production

I'm working on outlook addon using SSO. I've generated addon with yeoman. Locally everything works good, but once deployed to production and served built files from /dist SSO is not working anymore.
I found that locally this addon is running express.js server which is getting access token.
How I should handle this on production?

Perhaps this issue is similar to https://github.com/OfficeDev/office-js-docs-pr/issues/2279.  @davidchesnut is going to post documentation today that may help with this issue.
-Courtney

Hi @arozwalak,
This issue is also related to https://github.com/OfficeDev/office-js-docs-pr/issues/2279.
Since the code Courtney refers to is in a package, you can't update it to handle the Azure scenario. However you can get the same code from the Office-Add-in-NodeJS-SSO sample. And you find the same code at https://github.com/OfficeDev/Office-Add-in-NodeJS-SSO/blob/master/SSOAutoSetup/src/configure/ssoAppDataSetttings.js (which you can modify.)
So it might make more sense to start with this code sample, and modify it to handle the secret for Azure. I'm still working with the engineering team to figure out the specific steps to modify this code for Azure deployment. Once I have more info I'll follow up here, and also update the docs.
Thanks,
David

Hi @arozwalak , did you manage to deploy your add-in?

Hi @tobudim,
Unfortunately I can't help here. My addon is 'partially' developed, meaning that I just need accessToken, to authorize it with my platform. Looks like addon generated with office yeoman generator is not yet production ready. I was able to get around it and I'm able to authenticate user through our platform rather than through addon itself.

@arozwalak thanks for your feedback, actually it very well guided me. 🙏

Glad I could help @tobudim  :)

Hi @davidchesnut,
Update since my last message:

Azure web app:

As we've seen getAccessToken doesn't work as it does not get the secret when built (/dist).

Azure Windows Server VM:

As npm start works fine (I mirrored and updated the code to get the secret without asking for it locally) and not the build itself (/dist), I believed a reverse proxy would do fine.
As only Windows and Mac are supported I choose a Windows Server VM.
I could not go further than a 502 error, maybe or maybe not related to this repo but I could run other Node apps.

Bottom line: to me, I just can't use getAccessToken and I spend a lot of time building my app around this functionnality.
Any ideas would be appreciated, my goal is only to authenticate the user without asking him anything.

Hi @dimitri-bourreau,
Just a quick update to let you know that I did get the NodeJS SSO sample to deploy on Azure. In case this helps, here are my notes
Cheers,
David

Hopefully closing the loop on this now. We recently published documentation on how to deploy an SSO add-in from yo office. There were also updates made to yo office to better support deployment (be sure to use the latest office generator version.) For more details, see Deploy a single sign-on (SSO) Office Add-in to Microsoft Azure App Service


How does go module decide the version if the 3rd party repo doesn't have tag or releases?

Hi guys,
We are migrating to go modules.
One of our dependency is golang.org/x/crypto, however, there is no tags or releases maintained for this repo.
After does go module decide the version of the crypto?
golang.org/x/crypto v0.0.0-20191106202628-ed6320f186d4
Best Regards,

Please see - https://golang.org/cmd/go/#hdr-Add_dependencies_to_current_module_and_install_them

For each named package or package pattern, get must decide which version of the corresponding module to use. By default, get looks up the latest tagged release version, such as v0.4.5 or v1.2.3. If there are no tagged release versions, get looks up the latest tagged pre-release version, such as v0.0.1-pre1. If there are no tagged versions at all, get looks up the latest known commit. If the module is not already required at a later version (for example, a pre-release newer than the latest release), get will use the version it looked up. Otherwise, get will use the currently required version.

I will close this issue, but if you have further questions, please feel free to ask it in any of these forums below:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode

Thanks


Fix #9 Missing eslint from npm isntall

https://github.com/hyperoslo/eslint-config/pull/11 ;)

Haha! :grinning:

Also, it's a small thing, and I don't know if we have a policy on this, but I'd prefer if we didn't reference GitHub issue numbers in commit messages, due to the fact that they are meaningless in any other context. It's nice to reference the GitHub issue in the PR, but that can be done in the issue description (like in #11), which unties it from the commit itself. :smiley:

Will close this as #11 is merged. Thanks, @ohenrik. :smiley:

Haha, should check if there already is a PR :p


Manual Task Support / No Task Type support

When building processes, a common process is to mock out the entire process and leave certain steps as "manual task" steps which are skipped over by the engine, until that task is ready for implementation.
This allows the business to understand the process they want to build, and the developer to build/code in parallel.
It seems that there is no Manual Task supported in the modeler, and you cannot leave the task as a non task, as Kogito will return:
a Validation error of: "...Task has no task type."
Other than add everything as a user task, or a script task, etc, is there a viable alternative?
Was manual task omitted by design?
Thanks!

Created ticket: https://issues.redhat.com/browse/KOGITO-6594

For better tracking, we are now following this only at https://issues.redhat.com/browse/KOGITO-6594


Question

Through this tool, can I know the target ip.... Please explain in detail

Yes, you can find the user's IP using the getIPs() function.


[Docs][LoadBalancing] Add/Improve Javadocs for LoadSheddingStrategy impls

Motivation
I recently researched the implementations of the LoadSheddingStrategy interface. I thought the Javadocs could be improved to make it easier for users/developers to better understand the classes.
Modifications

Add Javadoc for ThresholdShedder. I got some of my understanding for the code and some from https://github.com/apache/pulsar/pull/6772.
Update Javadoc for OverloadShedder

Verifying this change
I read through the code to write this documentation. It is possible that I misunderstood it, so I will request reviews from contributors that have written these classes.
Does this pull request potentially affect one of the following parts:
No public/breaking changes.
Documentation
This is an update to the docs.

@hangc0276 - PTAL, since you were the original author of the ThresholdShedder.

/pulsarbot run-failure-checks

@Anonymitaet - PTAL. The tests are passing now. Thanks!


Add warning about NULL indicators not reset

Adds warnings in relevant sections in the documentation about the NULL indicators not being automatically reset between OCI_Execute calls. As this is extremely important to get right in multiple chunk calls, I think a warning is appropriate.

will be addressed in v4.7.6


Spike (doable?): Store hours display improvements e.g. "closes 9pm"

Description:  We need a discussion with developers and architects about how to display the store opening status in a single line. We (UX) would like:
Daily cycle:

If Closed: “Opens <9am>” (Red)
If Open: “Open until <9pm>” (Green)
If the opening or closing hours spans more than 24 hours:

If Closed: “Opens  <9am>” (Red)
If Open: “Open until  <9pm>” (Green)
If open for more than the next week:
“Open 24 hours”
If major holiday is approaching in less than one week, but otherwise 24 hours:
Open until  <1pm>

We should have #4166 done before.


Update day08.md

Sorry for f* up the initial PR

Hey, thanks again for writing this up, but please could you observe the format for the entry points as described in the schedule, also we should be getting a solution from @iusildra shortly

A, sorry. Wasn't aware off that

Hey, thanks again for writing this up, but please could you observe the format for the entry points as described in the schedule, also we should be getting a solution from @iusildra shortly

A, sorry. Wasn't aware off that

its fine! I should probably change the text in the website template

@prinsniels I have just applied a few small refactorings to enhance readability with the narrow width of the code blocks


Correct the name of the One True Editor

The title basically says it all. It's Emacs, not emacs. :-)

I guess so ¯_(ツ)_/¯


Remove double sentences from science blog 4

This PR removes some sentences which were in the science blog 4 two times.
The sentences were one time above the pictures and one time below the pictures. I removed the occurrences where the sentence was before the picture.
Example:

Before this change
After this change

Good morning @Ein-Tim ! Alrighty, thanks again! Enjoy your Sunday!


Key file dir broken....

Hi,
if we set our own key file directory then the file is not stored anymore and every restart need to pair TV again.

Hi,
this is a permission issue, if you change the directory of the keyfile then you need to make sure that homebridge has the correct permissions to access and write to that directory.

can confirm this error

@Strike2001 also here, this is most probably a permission issue.

No,
it’s not permission issue, in same directory are stored TV info and inputs correct.

Just tried it with mine TV, tested various different locations and all work fine for me (also same directory as TV info and inputs).
I suspect this is a permission issue on your side. If you have an error or log message can you provide that so we can have a look?

Just tried again and same effect for all my 3TV, nothing in the log with debug mode, I use this directory "/var/lib/homebridge/lgwebosTv" as prefsDir and keyFile but only inputs and tv info data is stored here correctly, no any keyFile there. If use default directory the key is stored in "/home/user/.lgtv2/".

maybe a clean install of the npm can fix ?
Von meinem iPhone gesendet

Am 21.09.2020 um 10:07 schrieb Grzegorz<EMAIL_ADDRESS>
Just tried again and same effect for all my 3TV, nothing in the log with debug mode, I use this directory "/var/lib/homebridge/lgwebosTv" as prefsDir and keyFile but only inputs and tv info data is stored here correctly, no any keyFile there. If use default directory the key is stored in "/home/user/.lgtv2/".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I hope you know that the keyFile property is should be a file not a directory...
"/var/lib/homebridge/lgwebosTv" this of course will not work since there is already a directory with that name so a file named lgwebosTv cannot be created. Can it be that this is the issue?

OK,
working....
the description and title in settings need to change from "Key file location", to "Key file name and location",
"The location where to store the tv key file. Default is: ~/.lgtv2/keyfile" to "The location and key file name where to store. Default is: ~/.lgtv2/keyfile"

I see the key file also do not use the prefsDir, if we set only name then is stored direct in Homebridge path. It is little misunderstanding. It will be better if "keyFile" use "prefsDir" as its path and we set only the "key name" in settings.

It is stated in the README that keyFile is a file and a directory should not be specified.
Maybe it would be better to put the keyFile indeed into the prefs directory. Actually the keyFile property could be entirely removed. But it will break the tv permissions for all users after the update so i am not sure if this is a good idea to change it now. Will think about it.

OK


Update tur.po

Credit to my Turkish friend for these translations: thank you!!
They weren't sure how to translate the "directed by.." string, and translated it roughly as "made by.."

Thanks a lot for the contribution! Anything helps.
I'll approve these changes once they are verified.


Awaiting futures from runZonedGuarded harmful

For example:
import 'dart:async';

Future<void> main() async {
  print('start');
  final f = runZonedGuarded<Future<void>>(
    () {
      print('in body');
      return Future<void>.error(Exception('error'));
    },
    (_, __) => print('in onError'),
  );
  await f;
  print('end');
}

Will produce the output:
start
in body
in onError

The reason we never see the final print, is because f never completes.
This dartdoc CL would make the documentation for runZonedGuarded more explicit that handling an error would lead to this future not being observed to complete.
See discussion on https://github.com/flutter/flutter/pull/141821 for more context.

Actually, the asyncGuard() function implemented in flutter_tools (https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/base/async_guard.dart#L81) correctly handles this, so we should just avoid runZonedGuarded for asyncGuard and we should be safe.

I can reproducible this on the latest master and stable channels.
stable, master flutter doctor -v

[!] Flutter (Channel stable, 3.16.9, on macOS 14.2.1 23C71 darwin-arm64, locale
    en-NG)
    • Flutter version 3.16.9 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 41456452f2 (4 days ago), 2024-01-25 10:06:23 -0800
    • Engine revision f40e976bed
    • Dart version 3.2.6
    • DevTools version 2.28.5
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C65
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (3 available)
    • Damilola’s iPhone (mobile) • 00008110-001964480AE1801E • ios            •
      iOS 17.1.1 21B91
    • macOS (desktop)            • macos                     • darwin-arm64   •
      macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)               • chrome                    • web-javascript •
      Google Chrome 120.0.6099.234

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.19.0-9.0.pre.206, on macOS 14.2.1 23C71 darwin-arm64, locale en-NG)
    • Flutter version 3.19.0-9.0.pre.206 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7e46394178 (51 minutes ago), 2024-01-29 01:46:24 -0500
    • Engine revision 210ed1dfb8
    • Dart version 3.4.0 (build 3.4.0-82.0.dev)
    • DevTools version 2.31.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C65
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (3 available)
    • Damilola’s iPhone (mobile) • 00008110-001964480AE1801E • ios            • iOS 17.1.1 21B91
    • macOS (desktop)            • macos                     • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)               • chrome                    • web-javascript • Google Chrome 120.0.6099.234

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

I think Chris had finished doing all of this (I can't find any more in flutter_tools).


Demo Site is down

your demo page is not available.
If the URL has changed, please open a pull request at
https://github.com/Kickball/awesome-selfhosted or create an issue. Thank you!
https://github.com/Kickball/awesome-selfhosted/issues/1767

https://github.com/pirate/ArchiveBox/issues/231


Docker setup suggestions

Here's an easier way to run it inside a container
$ docker pull azuresdk/azure-cli-python
$ alias az='docker run --rm -v ~/.azure2:/root/.azure azuresdk/azure-cli-python az'
$ az account login
...

this way

you don't have to create a new container everytime
you don't have to stay in the container's shell
you don't have to login over and over again

I suggest documenting it like this.

Adding @derekbekoe to take a look

I wouldn't remove the old documentation as that's a perfectly valid scenario.
With the current method (docker run -it azuresdk/azure-cli-python:latest), you can still achieve everything described:

you don't have to create a new container everytime as the one initially created will still be running
you don't have to stay in the container's shell as you can do docker exec <container> az vm list for example
you don't have to login over and over as the container holds the credentials

You also get isolation of the persisted login info as it's in the container rather than mapped to a host directory.

I think this suggestion is neat though so we can still add documentation to use the container in this way as an alternative method. I'll work on a PR for this.

you don't have to create a new container everytime as the one initially created will still be running

not if you reboot your computer, or accidentally hit ctrl+c in container's shell. The old container will still be around and will be stopped docker run creates a new container every time

you don't have to stay in the container's shell as you can do docker exec  az vm list for example

you can't docker exec once you exit the container (when it's stopped), you have to do docker start --a.

you don't have to login over and over as the container holds the credentials

Nope, as I said docker run creates a brand new container every time. Try doing docker run and    then az login then exit the container by typing exit, then run the same docker run, you won't have your ~/.azure == it's a new container.


refactor(frontend): rewrite 3 system functions using context

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR rewrites 3 frontend functions using #[function] with context:

pg_relation_size
pg_indexes_size
pg_get_userbyid

Currently these functions are implemented by rewritten to subqueries on system table, which is verbose and hard to maintain.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

thread 'rw-main' has overflowed its stack
fatal runtime error: stack overflow

The stack overflow (on debug mode) is caused by the following code:
https://github.com/risingwavelabs/risingwave/blob/144286099fc189f70eb716159fc2bd7edc10c1d5/src/frontend/src/scheduler/local.rs#L180-L185
If you print the size of each exec:
2848
5728
11520
23200
46432
92928

It seems the size will double each time it is nested. 🤯
cc @TennyZhuang @shanicky @yezizp2012 @st1page

QQ: I remember we force queries scanning system tables running in local mode. Do we do so for functions with context? (Do we need to?)

Yes. I guess we have not forbidden system functions in distributed mode or in mviews. But doing so will result in an context not found error or null output.

It seems the size will double each time it is nested. 🤯

We've encountered this issue before! 😄
https://github.com/risingwavelabs/risingwave/blob/1c52ee772c97218257e50709f8fdf3ccd34d9962/src/utils/pgwire/src/pg_protocol.rs#L244-L246
This blog could be a good reference: https://swatinem.de/blog/future-size/


(nodejs) add Kokoro configs for nodejs repos

🎉 @theacodes FYI.  It would be really nice to add #13 to help roll this out.

I know, but please keep in mind it's the most non-trivial feature request
we have right now.
On Wed, Aug 22, 2018, 8:27 PM Justin Beckwith<EMAIL_ADDRESS>wrote:

🎉 @theacodes https://github.com/theacodes FYI. It would be really nice
to add #13 https://github.com/GoogleCloudPlatform/synthtool/issues/13
to help roll this out.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/synthtool/pull/38#issuecomment-415271935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc9Cx6WES3PvbI0y48qMUrdgYlTHCks5uTiEfgaJpZM4WIxPO
.


Genomes missing a number of sequences

The following genomes lack a number in the "Sequences" field
{"id_list":{"genome_id":["1496.2589","1496.2590","1496.2592","1496.2593","1496.2595","1496.2594","1496.2591","1496.2598","1496.2597","1496.2596","1496.2599","1496.2600","1496.2601","1496.2603","1496.2602","1496.2604","1496.2605","1496.2606","1496.2608","1496.2607","1496.2657","1496.2658","1496.2635","1496.2636","1496.2637","1496.2639","1496.2640","1496.2638","1496.2641","1496.2642","1496.2645","1496.2643","1496.2644","1496.2646","1496.2647","1496.2649","1496.2651","1496.2650","1496.2648","1496.2652","1496.2653","1496.2654","1496.2656","1496.2655","699035.42","1496.2609","1496.2610","1496.2611","1496.2613","1496.2614","1496.2616","1496.2615","1496.2612","1496.2617","1496.2618","1496.2619","1496.2620","1496.2622","1496.2621","1496.2623","1496.2624","1496.2625","1496.2626","1496.2627","1496.2628","1496.2629","1496.2630","1496.2633","1496.2631","1496.2632","1496.2634","1496.2659","1496.2660","1496.2661","1496.2662","1496.2530","1496.2531","1496.2533","1496.2532","1496.2534","1496.2535","1496.2536","1496.2554","1496.2556","1496.2542","1496.2544","1496.2543","1496.2545","1496.2555","1496.2557","1496.2560","1496.2559","1496.2562","1496.2564","1496.2563","1496.2566","1496.2561","1496.2565","1496.2567","1496.2568","1496.2537","1496.2538","1496.2539","1496.2540","1496.2546","1496.2547","1496.2548","1496.2550","1496.2549","1496.2551","1496.2552","1496.2553","1496.2541","1496.2571","1496.2581","1496.2578"]},"name":"weird_cdiffs"}

Because of the earlier complains about the confusion between contigs and sequences, we have decided to deprecate the sequence column and rather use contigs to show total number of contigs (including complete chromosomes and plasmids)  in a genome.
We need to make sure that the tables on the website are showing contigs as the default column.
Also, need to remove sequences from tables as well as overview pages.


Azure Data Studio startup error on starting SQL Tools Service after latest ADS update

Issue Type: Bug
Applied latest ADS update. On restart, the following messages show up:
The AzureResourceProvider server crashed 5 times in the last 3 minutes. The server will not be restarted.
Source: mssql (Extension)
The SerializationProvider server crashed 5 times in the last 3 minutes. The server will not be restarted.
Source: mssql (Extension)
Azure Data Studio version: azuredatastudio 1.23.0 (d296b6397e0acfddc57e9085e736e084969cdaeb, 2020-10-13T23:03:09.799Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz (4 x 2295)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
16.00GB (10.50GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (16)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

admin-tool-ext-win
Mic
0.0.2

agent
Mic
0.47.0

azuredatastudio-postgresql
Mic
0.2.6

cms
Mic
0.8.0

dacpac
Mic
1.6.0

datavirtualization
Mic
1.10.0

import
Mic
1.1.0

managed-instance-dashboard
Mic
0.4.2

powershell
ms-
2020.4.0

profiler
Mic
0.11.0

query-history
Mic
0.1.0

schema-compare
Mic
1.6.0

server-report
Mic
0.2.2

whoisactive
Mic
0.1.4

sql-prompt
Red
0.2.2

We need more info to debug your particular issue. If you could attach your logs to the issue (ensure no private data is in them), it would help us fix the issue much faster.
To find your logs:

Open command palette (Click View -> Command Palette)
Run the command: Developer: Open Logs Folder

This will open the log file locally. Please include renderer.log

renderer.log just says
[2020-10-16 07:25:59.155] [renderer1] [error] An unknown error occurred. Please consult the log for more details.

This appeared in a later log of the renderer log:
[2020-10-16 13:42:53.744] [renderer1] [warning] UNRESPONSIVE extension host, 'Redgate.sql-prompt' took 50% of 7664ms, saved PROFILE here: 'C:\Users\xxxxxx\AppData\Local\Temp\7\exthost-7be509.cpuprofile' [{"id":"gc","total":11000,"percentage":0},{"id":"Microsoft.datavirtualization","total":1179000,"percentage":15},{"id":"Microsoft.notebook","total":4000,"percentage":0},{"id":"Microsoft.server-report","total":1392000,"percentage":18},{"id":"Microsoft.whoisactive","total":1129000,"percentage":15},{"id":"Redgate.sql-prompt","total":3835000,"percentage":50},{"id":"self","total":106000,"percentage":1},{"id":"vscode.git","total":4000,"percentage":0},{"id":"vscode.merge-conflict","total":2000,"percentage":0},{"id":"vscode.search-result","total":2000,"percentage":0}]
[2020-10-16 13:42:58.317] [renderer1] [error] An unknown error occurred. Please consult the log for more details.

I ended up uninstalling it. It was originally installed as a system install since we have a few users running it on the same terminal server but I've reinstalled as a user install and it seems fine.


election_date not always present

Missing from:
Kentucky
Louisiana
Maine
Maryland
Massachusetts
Montana
Nebraska
New_Jersey
Oklahoma
Rhode_Island
South_Carolina
South_Dakota
Vermont
Virginia

I am considering removing this column altogether since there could have been emergency changes that aren't reflected in our dataset...I'm worried about giving a false sense of precision..

Might make sense to do year instead so the whole thing is still importable without having to pull that from the file name.

Keeping the election_date column but also adding a source_date column which will indicate the data when the data was acquired.

Addressed in the new schema, export function has been updated to ensure test election date is included for each available file and that the election date matches the one specified in manifest.yaml.


Newsletters: add #42 (2019-04-16)

Of the newsletters I've written so far, I think I'm least happy about this one.  I'm sorry it's not better.
Note: the math on the multisig transactions is a bit lazy.  There's at least two places where 1-3 extra bytes are introduced when a field exceeds a certain length; I just used a constant value.  Since the max differences per tx is 6 extra bytes, I think this would be invisible in the chart, but I can fix it if need be.

Added a few separate small commits that I think address @jnewbery's feedback (thanks!).  They're separate commits to make them easy to revert if desired.  Please feel free to make any last-minute edits you think are necessary.

tACK 584f1b1740c81038dc59c26b767acf87a20d1ae6
left a couple comments but approve as is

Pushed a change for one of @moneyball's suggestions (thanks!).  Should be good to go.

ACK 32a8c9c41a4e206455851181d7c6ba66fc799e20.
Time to squash, merge and :ship:

Force-pushed a squashed branch. No changes to content: https://github.com/bitcoinops/bitcoinops.github.io/compare/32a8c9c41a4e206455851181d7c6ba66fc799e20..7164b52fb46b9ac5004686376ed9496f3294d983


Connector Shapes in Firefox

The connector ends in Firefox are cropped to a half of the image.
This happens on Windows and Linux versions of Firefox.  Works fine in Chrome.

CSS Fix should be in #88.


IDE should detect wrong properties on native component creation

The IDE should be able to detect inexistent properties introduced during a native component creation, and it should also be able to suggest the correct ones.
For custom components this is already working:

Tried every property-related compiler flags for the tsconfig file with no success.
    "alwaysStrict": true,
    "strict": true,
    "strictBindCallApply": true,
    "strictPropertyInitialization": true,
    "suppressExcessPropertyErrors": false,
    "suppressImplicitAnyIndexErrors": false,
    "exactOptionalPropertyTypes": true

The correct funcionality works not only for custom components but also for the Transform component, and no other native component.


add some tests for num instructions

about issue #6 : I started writing some tests , let me know what you think.
I have a question about testing "signed" operations; how do you manage it when the stack can contain only uint64 values ?

@ilmanzo thank you for this 🎖

I have a question about testing "signed" operations; how do you manage it when the stack can contain only uint64 values ?

Maybe you would better take a look at this link here: https://stackoverflow.com/a/50815961
basically all type of numbers are stored in uint64 with its corresponding bit representation

thanks for the suggestion; I've refactored the tests with a table-driven style.

ok I splitted each test in own function; (now using testify/suite) but each test have its own table . What do you think of this approach ?


Upload processing getting blocked by one video stuck in approved state

The video available at the link below, when uploaded to Nunaliit using a schema with a file attribute, will work its way through the process up until the approved state. At this point, nunaliit should be attempting to upload the converted version into couchdb as an attachment to the document. This never seems to complete. In the meantime, other file uploads are not processed. Restarting nunaliit will cause the upload servlet to find the document in the approved state and restart the next step, blocking the processing of other uploads again Deleting the document with this video removes the blockage.
https://rs-atlascine-dv.concordia.ca/rwanda/media/upl2742994457409505921.mp4

Logs show:
2020-11-16 14:48:38,001[INFO ]: Upload worker processing: Document Work(state:approved id:3714a5c74e36fdc306914f710d0ecbad att:VenantSeminari_web.mp4)
2020-11-16 14:48:38,003[INFO ]: Loading document 3714a5c74e36fdc306914f710d0ecbad
2020-11-16 14:48:38,027[INFO ]: Upload VenantSeminari_web.mp4 to document 3714a5c74e36fdc306914f710d0ecbad

(Full output of a nunaliit restart after upgrade with test patches here: https://pastebin.com/Ug2aJPa0)

I believe this is resolved or worked around in the Atlascine atlases but I do not recall what the resolution was. I will have to check with Emory.

Saw this behaviour again in a recent build. Still an issue.


"Add New System to Rack" link doesn't work

When I click on it, nothing happens.  There's no message in the console either.

I'm not sure if this ever worked. The link is pretty much coded back at itself. I'll remove the link to remove confusion.


⚠ PrivateHD has degraded performance

In 9f26a90, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3837 ms

Resolved: PrivateHD performance has improved in 609fd5b.


akka-persistence-query, ... 2.6.15 (was 2.6.14)

Updates

com.typesafe.akka:akka-persistence-query
com.typesafe.akka:akka-persistence-tck
com.typesafe.akka:akka-stream-testkit

from 2.6.14 to 2.6.15.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.akka" } ]

labels: library-update, semver-patch

Superseded by #140.


Fix thresholds helper, extract set_master_signer_weight

This PR modifies the set_options recipe step to be compatible with the latest form of the SetOptionsOp xdr struct.  In addition, it extracts a recipe step called set_master_signer_weight for more ease of use.  Finally, we add an example recipe that exercises. all the set_options related helpers.

:+1:


Please Review my PYPI Python Package

Items we can review on a live stream:

Open source project

It is a Python Package specially make for minecrafteers, and Bot developer to extract the stats, and display to the people.
It's not something really unique, But I keep it different by Using really good docstrings, and suggested in pep along with modularity in code, allowing better control over the objects, and accessing the data, and also showing them, It also supports various paradigms in Code.
Status: Alpha
Latest release : 0.0.7
New release within a week, Major fixes are being worked on in a single or 2 or 3 commits.
Link: https://github.com/janaSunrise/HypixelIO
You can also join the discord community here
Feel free to check out other cool repositories of Eddie Jaoude Community here

Thank you for sharing 👍 . It is the queue to get reviewed 🤓

Oh looks good 👍  (I was looking into fetching data from hypixel using requests, though this might make my work easier 🤔 )
It would be nice if we have issues, a more detailed README(maybe just add a link to the docs folder), a full fledged-release(all the current tags seem to be pre-release), display package on the repo page

Yeah, I've been trying a lot to make things, I need some time, as I am the lone contributor 😅 So, Pardon me if there are no external docs, or tests, or something is missing. I'll add them to the project board. And Also, I've been doing a lot of Fixes, to pull version 0.1.0 Which is a production ready build, and I'll be using dev branch for pre-releases, and also testing purposes, with PRs, so Main isn't cluttered with code, which doesn't work.

😄  You can open another issue on this support repo for "Contributors needed: <project_name>" with details 🤓

Oh Cool! I'll do that for sure :D

welp, stalebot is getting itchy fingers

welp, stalebot is getting itchy fingers

Yeah 😂 😂 I have stopped working on HypixelIO for a while, got other projects, and no contributors either, Might make a PR to get some help :)

Yeah 😂 😂 I have stopped working on HypixelIO for a while, got other projects, and no contributors either, Might make a PR to get some help :)

We will ignore stalebot for this 🤓 . I removed the label.
Maybe we should do a live stream today reviewing some open source projects? This issue will be 2nd 🎉

We will ignore stalebot for this 🤓 . I removed the label.
Maybe we should do a live stream today reviewing some open source projects? This issue will be 2nd 🎉

Yeah 😂 😂 I have stopped working on HypixelIO for a while, got other projects, and no contributors either, Might make a PR to get some help :)

I don't see any issues open on your repo...
Maybe you could open some, maybe we could contribute a little?

Yeah 😂 😂 I have stopped working on HypixelIO for a while, got other projects, and no contributors either, Might make a PR to get some help :)

I don't see any issues open on your repo...
Maybe you could open some, maybe we could contribute a little?

Reviewed in live stream https://youtu.be/_JjV9n-G-p0
Looks great, some of my thoughts:

Use Issues to track work and make it easier for the community to get involved
Project board is great, but suggest converting to issues for discoverability and so people can comment (join the conversation)
Suggest adding a Code of Conduct https://github.com/janaSunrise/HypixelIO/community

Reviewed in live stream https://youtu.be/_JjV9n-G-p0
Looks great, some of my thoughts:

Use Issues to track work and make it easier for the community to get involved
Project board is great, but suggest converting to issues for discoverability and so people can comment (join the conversation)
Suggest adding a Code of Conduct https://github.com/janaSunrise/HypixelIO/community


feat: Make array members non-optional by default

closes #227
Background
The more I looked into this, the more I became convinced that:

Array elements should be non-optional by default, full stop.
The reason they are not is a side-effect of the way types are built by papr, not an expected outcome of the papr design

For point 1. --  an array member being "required" or not has zero impact on the generated JSON Schema. And a collection with a JSON schema validator will not validate a document that has null or undefined array members. (Try it yourself!)
For point 2. --   types methods do double duty in papr: they output the generated JSON schema for the schema and, through type-casting, generate typescript types to use in code.
Before this change, the array method's return value was GetType<Item[], Options> In order to generate the appropriate JSON schema for array properties, we must pass a types method to the array method (e.g. types.array(types.number())). Thus, the generic Item type for the array method will be a papr type, and thus  subject to the required (in schema) = optional (in TS) expectation. However, a required type passed to an array does not generate any additional JSON Schema, because required only does so for object properties. So in this case, required simply functions to say whether Item is, for example, number or number | undefined.
I consider this an unexpected side-effect, because though it makes sense given that we are using a generic Item type, it is not actually useful behavior for consumers of the library.
Solution
Given that we do not want Item to ever be undefined, we can use the extant Typescript NonNullable utility type to make sure that array members are always required.
Admittedly, this means that required, when passed to a type that is then passed to array, no longer does anything (for TS, it already did nothing for the JSON schema), but I think this is probably an acceptable quirk of the API. Ideally, required would not be possible to pass to the type that is the first argument to array, but I didn't see a straightforward way to make that happen without a much larger change.

Let's add some documentation about this change in the types.array documentation in types.ts @ line 218.

Took a pass at it, I'm not crazy about the wording 🤷


go get fails on go1.11.2 linux/amd64

When running go get -u github.com/wellington/sass/sass (the command listed in the readme), Go outputs the following error:
../../../go/src/github.com/wellington/sass/sass/cmd/compile.go:31:11: undefined: compiler.File

I know this lib has not been updated in a long time, but it is not archived so I'm not sure if there are any plans to maintain this. Since it has not been updated in 2 years the build fail is probably due to some Go update, although I'm not altogether sure.

Looks like a dependency has changed. There should be a make target that Syncs dependencies better than go get

From: Cheese Cake Aficionado<EMAIL_ADDRESS>Sent: Saturday, December 22, 2018 7:09 PM
To: wellington/sass
Cc: Subscribed
Subject: [wellington/sass] go get fails on go1.11.2 linux/amd64 (#46)
When running go get -u github.com/wellington/sass/sass (the command listed in the readme), Go outputs the following error:
../../../go/src/github.com/wellington/sass/sass/cmd/compile.go:31:11: undefined: compiler.File
I know this lib has not been updated in a long time, but it is not archived so I'm not sure if there are any plans to maintain this. Since it has not been updated in 2 years the build fail is probably due to some Go update, although I'm not altogether sure.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/wellington/sass/issues/46, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAOmE1ZCwGOR8rbfm4vyCtpIrpkBQfkzks5u7sm5gaJpZM4Zfp_2.


422 response not rejected

We have a project that uses<EMAIL_ADDRESS>(upgrading to 0.15.3 did not resolve the issue) in many parts of the application. In all of the other parts of the app, we're able to .catch() 422 response codes. Recently we noticed that 1 request is resolving a promise that gets a 422 response. In all other parts of our application, a 422 causes axios promises to be rejected.
export function addTool(name: string): Thunk {
  return (dispatch, getState) => {
    Axios
      .post<Tool>(API.current.toolsPath, { name })
      .then(resp => {
        if (resp instanceof Error) { throw resp; }
        success("Tool has been saved.", "Success");
        dispatch(addToolOk(resp.data));
      })
      .catch((e: Error) => {
        dispatch(addToolNo(e));
        error(prettyPrintApiErrors(e));
      });
  };
}

Source code here
I was able to work around the issue by adding the following line to my .then():
if (resp instanceof Error) { throw resp; }

Which works fine, but I'm still confused as to why this particular 422 is resolved (rather than rejected). We also have some interceptors in use, but they haven't cause any issues for the other requests we make.
Are there any circumstances in that would cause axios to resolve a 422 rather than reject it? Please let me know if you need any other information.
Thanks for the help!

Confirmed. I'm also seeing something similar when using axios with redux-saga where some 422's aren't being handled as a rejected promise.

Never mind, found my problem. Interceptors weren't properly propagating errors down to the app layer, d'oh! 😳

After investigating this issue, I found that it was actually a local type error hidden deep in the call stack somewhere, leading me to believe that it is a local issue rather than a problem with Axios.


too many helm release secrets created on child cluster

What happened:
too many helm release secrets created on child cluster

What you expected to happen:
keep limited count of helm release secrets
Environment:

Clusternet version:

Clusternet-agent version (user clusternet-agent --version=json): 0.8
Clusternet-hub version (user clusternet-hub --version=json): 0.8

Kubernetes version (use kubectl version): k3s

The maximum number of historical releases is limited to 5 since v0.4.0.
https://github.com/clusternet/clusternet/blob/ae735577f4d637baa6482c0eb220f14a4cd59029/pkg/utils/deployer.go#L127-L133
But the MaxHistory here  is not inherited by Helm Upgrade action. That's why there are so many release history in child clusters.
Nice catch @jasine . Got fixed in #485.


Incorrect help on install

After installing with brew I got:
If you already have a project on Gemnasium, you'll need to run
  gemnasium projects configure

But this command does not seem to exist:
$ gemnasium projects configure
No help topic for 'configure'

Fixed in https://github.com/gemnasium/homebrew-gemnasium/commit/9484b350997adde5053777e010e5f33c2868d9d1
thanks for reporting.


*: peer tls

Sending out PR for early review

Another round of change. Notably:

The tls spec has been changed to:

type StaticTLS struct {
	// Member contains secrets containing TLS certs used by each etcd member pod.
	Member MemberSecret `json:"member"`
	// Operator contains secrets containing TLS certs used by operator and this cluster's backup sidecar.
	Operator OperatorSecret `json:"operator"`
}

type MemberSecret struct {
	// PeerSecret is the secret containing TLS certs for etcd peer communication used by each etcd member pod.
	PeerSecret string `json:"peerSecret"`
	// PeerSecret is the secret containing TLS certs for etcd client communication on server side
	// used by each etcd member pod.
	ServerSecret string `json:"serverSecret"`
}

type OperatorSecret struct {
	// ClientSecret is the secret containing TLS certs for etcd client communication on client side
	// used by operator and this cluster's backup sidecar.
	ClientSecret string `json:"clientSecret"`
}

user docs has been updated to mention SAN wildcard.
internally, we separate the path for "SecurePeer()".

@colhom can you take a look (ideally, pull down this pr and give it a try). this looks good to me in general as an intermediate step.

fixed.

LGTM. I would love @colhom to give this a try tough.

@hongchaodeng can you add a simple e2e test to this?

@xiang90
There is no simple way to do that. I would like to add e2e test later.
The difficulty comes from generating a test cert. We don't know the namespace and cluster name before the test. So we don't know what hostnames to allow in the test.

For more info, I think we can use something like:
https://golang.org/src/crypto/tls/generate_cert.go
to generate some certs dynamically.
That's a lot of code though. I would like to make it into another PR.

Test is submitted in another PR: https://github.com/coreos/etcd-operator/pull/945.
We want to make both PR smaller.

@hongchaodeng

The difficulty comes from generating a test cert. We don't know the namespace and cluster name before the test.

Pods will wait to start until the required secrets are there. So we can deploy etcd-operator and have it deploy the cluster before generating the TLS secret and loading it into the API?

@hongchaodeng great work, LGTM!
I've verified functionality on this PR using #960 and #961, which deal with implementing static/self-signed TLS generation in a cohesive way using the operator codebase.

@hongchaodeng let's squash and get this merged! thank you!


add Rustlang.Rust.GNU version 1.82.0

This PR is auto-generated. If there's something wrong, please file an issue at https://github.com/Rust-Winget-Bot/my-source-code/issues
Microsoft Reviewers: Open in CodeFlow

Hi @Rust-Winget-Bot,
This manifest removes the DisplayVersion from the AppsAndFeaturesEntries, which is present in previous manifest versions. This entry should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 897.)

@stephengillie this directive is in conflict with prior directions received in https://github.com/Rust-Winget-Bot/my-source-code/issues/1 please advise.

HI @Xaeroxe,
Sorry for my delayed reply. I've updated my automation to omit this message.

@stephengillie, okay, it's probably still valuable to re-open and merge this PR

I'm willing. The branch would need to be reverted before the PR could be reopened.

@stephengillie the branch has been restored. My bot will auto delete the branch typically so I've temporarily disabled it. I'll reenable it once this PR is merged.


fix college of faculty fellow program label

READY FOR REVIEW
Summary

Update College Faculty Fellows Program label

Review By 4/30
Criticality - 2
Review Tasks
Setup tasks and/or behavior to test

Check out this branch
Navigate to...
Verify...

Front End Validation

[ ] Is the markup using the appropriate semantic tags and passes HTML validation?
[ ] Cross-browser testing has been performed?
[ ] Automated accessibility scans performed?
[ ] Manual accessibility tests performed?
[ ] Design is approved by @ user?

Backend / Functional Validation
Code

[ ] Are the naming conventions following our standards?
[ ] Does the code have sufficient inline comments?
[ ] Is there anything in this code that would be hidden or hard to discover through the UI?
[ ] Are there any code smells?
[ ] Are tests provided? eg (unit, behat, or codeception)

Code security

[ ] Are all forms properly sanitized?
[ ] Any obvious security flaws or new areas for attack?

General

[ ] Is there anything included in this PR that is not related to the problem it is trying to solve?
[ ] Is the approach to the problem appropriate?

Affected Projects or Products

Does this PR impact any particular projects, products, or modules?

Associated Issues and/or People

JIRA ticket(s)
Other PRs
Any other contextual information that might be helpful (e.g., description of a bug that this PR fixes, new functionality that it adds, etc.)
Anyone who should be notified? (@mention them here)

Resources

AMP Tool
Accessibility Manual Test Script
HTML Validator
Browserstack and link to Browserstack Credentials

@forrestglick

GTG - merging into dev @forrestglick

Thanks @yvonnetangsu

@yvonnetangsu This looks great. Please push to Prod. Thank you!

@yvonnetangsu @forrestglick I don't think this PR has made it into main or the release from earlier today.
https://github.com/SU-SWS/ood_giving_site/compare/main...dev

wmui51

@yvonnetangsu @forrestglick I don't think this PR has made it into main or the release from earlier today.
main...dev

You're right - I must have somehow made a mistake 🤔 I'll push another release.


[UIAD] Add new object type UIAD

Check fails due to https://github.com/SchemaStore/schemastore/issues/3620

Feel free to merge, otherwise we will merge by the end of the week.

I just re-requested the review  of @schneidermic0


Standardizes Makefile usage to match newer approach in other projects

Target-specific make-variables, only evaluated when that target is called. Fewer unnecessary shell calls this way.
Make generally works on file-systems, so using slash-based targets allows utilizing some built-in make features (not that we're doing so here)... https://www.gnu.org/software/make/manual/html_node/Automatic-Variables.html
The only change that is functionally all that different is TERRAGRUNT_URL... when split across two lines as before, the releases string returned by the API was getting hung up on a single quote in the body of one of the releases. That caused the call the fail, which you can see here: https://travis-ci.com/plus3it/wrangler-watchmaker/jobs/139135244.


Enhancement: graph subscription add allow subscription for Event Hubs and Event Grid

The command graph subscription add has validation of notificationUrl that doesn't allow to create a subscription for notifications through Azure Event Hubs and Azure Event Grid.
To allow notifications through Azure Event Hubs and Azure Event Grid, modify the validator.
notificationUrl can start with

https://
EventHub:https://
EventGrid:?azuresubscriptionid

seems solid to me 👍💪
@pnp/cli-for-microsoft-365-maintainers anyone any other feed or should we open it up?

I will take it


FastRTPS failure while making agent.

Hello, so I am completely lost on how to solve this issue.
Here are the steps to replicate it.
Install px4 using the ubuntu.sh and ubuntu_sim_ros_melodic.sh on Ubuntu 18.04 LTS, next install FastRTPS 1.7.1 as done in the guide.
Next use git clone to pull down the px4 Firmware repository.
I followed the Ros2/Fast RTPS instructions after.
I did make px4_sitl_rtps
Then I did this
cd build//src/modules/micrortps_bridge/micrortps_client/micrortps_agent
mkdir build && cd build
cmake ..
make
It fails at make.
Next install FastRTPS from source 1.7.0 and 1.8.0 and then the master branch. In each of those cases I got the same exact error.
I tried to figure out what was going on by looking at this, but I couldn't get it to work.  https://github.com/PX4/Firmware/pull/12046

Okay. I'm sorry. I fixed it by doing this
make clean
make px4_sitl_rtps
Now this was after two days of constant struggle, and I think using the master branch of FastRTPS finally worked.


Add name, picture, twitter, email to profile

Add basic profile info to your connected account, and then show it on the impact page

Do you wanna fetch profile from connected acc with Sequence?

or Add profile editor? (that we store)
or Add connect profile button? :P

@Riglanto profile editor please :-)

Paused for now, but progress is in: https://github.com/Glo-Foundation/glo-wallet/tree/180-add-name-picture-twitter-email-to-profile