feat(logger) - adding logger support to the chart

So here's my plan on this:
Merge this change into the main Deis chart for now.
When we break down the Deis chart into components, we need to discuss whether or not we are going to mark this as required. IIRC, because it uses DaemonSet, it won't be supported on all of the 1.1 clusters (since, IIRC, that feature is turned off by default). If that's not the case, then we should definitely make this part of the full install.
Any objections to that?
Also, if this is ready for review, can you mark it so, @jchauncey

Starting to work on fixing this PR. Updated to use new deisci image. We also need to push the fluentd image to deisci as well.

updated to add the namespace label. working on building a fluentd image using alpine. so that should be the last change we need

This PR now moves logger into its own chart deis-logger

I'm testing this, but not totally sure what the desired outcome should be.

Deis logs won't work yet. You'll need to curl the URL for the service at
port 8088/fluentd or whatever the container is called
On Dec 17, 2015 4:58 PM, "Matt Butcher"<EMAIL_ADDRESS>wrote:

I'm testing this, but not totally sure what the desired outcome should be.
—
Reply to this email directly or view it on GitHub
https://github.com/deis/charts/pull/27#issuecomment-165619470.

@jchauncey since this isn't connected to workflow yet and we query the logger's RESTful interface directly to get logs... do you want to consider making logger routable?
If so, consider modifying the service as follows:
apiVersion: v1
kind: Service
metadata:
  name: deis-logger
  namespace: deis
  labels:
    routable: "true"
  # ...
  annotations:
    deis.io/routerConfig: |
      {
        "domains": [ "logger" ]
      }
# ...

I have the change to get this hooked up to workflow but was busy yesterday getting the etcd crashloop bug fixed so I could test it too. I will submit that in just a minute

Ok... then strike my last suggestion.  Connecting it to workflow is obviously better.

So the problem we will have by making this a separate chart is that workflow wont know where to find the logger if its installed first.

@jchauncey is the solution to make that more configurable on workflow, or just document this? (That is, until we get all this chart stuff sorted out IRT daemon sets)

im not sure what to do. it bother me that logging wouldnt work out of the box with workflow but it also sucks to depend on daemonsets

I think we have to include logger in the main workflow chart. If we dont then when someone wants to enable logging the have to first install the logger chart and then restart the workflow rc.
One other possible option is to include the logger service definition in the workflow chart so that the ip and port is set when the workflow rc starts.

I think we have to include logger in the main workflow chart. If we dont then when someone wants to enable logging the have to first install the logger chart and then restart the workflow rc.

I think that's perfectly acceptable for testing on alpha, IMO. Until DaemonSets are a part of the stable API, if we threw this in workflow everyone testing would have bootstrap k8s with the v1beta api extensions. It'll drag down the entire user experience with the rest of the platform.

One other possible option is to include the logger service definition in the workflow chart so that the ip and port is set when the workflow rc starts.

That could be a happy medium... or people could just install deis-logger before installing workflow. That way the service is installed before workflow boots.

yeah but in either case they have to enable DaemonSets.

Because of the daemonset thing, I think we will have to accept that the user will need to restart workflow to get logger installed. It's definitely less than optimal, but given that logger's not in the official requirements for alpha, it's acceptable.
This issue will almost certainly be resolved in the Beta timeframe, since k8s 1.2 will have daemonsets enabled without requiring a special flag.

This issue will almost certainly be resolved in the Beta timeframe, since k8s 1.2 will have daemonsets enabled without requiring a special flag.

Does that mean that starting with beta we're targeting only k8s 1.2+?

There is no way you'll get me to commit to that without ever seeing 1.2. ;-)

Ok so I have heard that logger is both required and not required for alpha. We should get some clarification on that. @slack @gabrtv

Not required.  This is the current behavior is logger is not running:
$ deis logs
Error:
500 INTERNAL SERVER ERROR
"Error accessing logs for node"

I think that is fine for alpha.

Alright sounds good. Just wanted to make sure =)

That said, we're at the place where logger is working, right? So the chart is totally usable with Alpha.

I just need to test and submit the PR for the workflow piece and then yeah its good (i hope)

What components depend on the workflow-service?

@jchauncey AFAIK only builder and the router (indirectly) relies on the workflow service.

can we get this merged? Would make it easier to test https://github.com/deis/workflow/pull/263

@deis/core-maintainers can we get this merged?

this works for me for the time being until 1.2 is out and GKE is using it in prod.

I think we better document the huge caveat that this eliminates most zero to k8s options from consideration by users unless they are willing to make an investment in customizing their cluster or willing are willing to wait for k8s 1.2.

There is language in the readme of the chart and the in the Chart.yaml that explains that it needs daemonsets to work and links to the documentation.
For coreos based installations all you need to do is change the fleet file that starts the api server and restart it.
Oh it should also be noted that if you install this chart after deis, you will need to bounce the workflow pod.

Ah... I just remembered this is a separate chart for now.  So this doesn't impact the installability of the Deis PaaS atop most zero to k8s options.  Strike my complaint.


[edn] Investigate surprising pattern matching behavior for unordered and unique value collections

The current pattern matching rules are tricky and surprising in certain cases, because the pattern matching "tokens" we've chosen are, in fact, symbols that obey the rules of the container collections they're a part of. This results in a rather leaky abstraction when it comes to pattern rules.
For example: #{1 _ _} cannot match #{1 2 3} because sets guarantee uniqueness of values, and since _ is just a symbol, the first set will have a length of 2, while the second one will have a length of 3. Maps follow a similar story, where: {1 2, _ 4, _ 6} cannot match {1 2, 3 4, 5 6} for the same reasons.
One can get around the above limitations by naming the "any operator" _:  #{1 _a _b} will match #{1 2 3} and {1 2, _a 4, _b 6} will match {1 2, 3 4, 5 6}. Support for this has been added in #338, however this approach might not be ideal. An alternative would be syntax support for patterns in the edn parser itself, where something like #p _ would parse into a unique "any" pattern.
Another issue stems from matching named ?symbols in unordered collections. For example, when matching #{?a ?b} against #{1 2}, there's no clear definition for what ?a and ?b should end up being. Currently, assignment follows iteration order, which results in surprising (but consistent, not undefined) results where #{?a ?b} =~ #{2 1} has ?a = 1 and ?b = 2. This needs to be investigated, and if no solution found, consider outlawing such cases.

Careful: the solution to this turns into building a Prolog interpreter. This is a constraint satisfaction problem: you're trying to define a set of constraints (in undecorated EDN, for some reason), and check whether an input document meets them.

I've tried to use the existing pattern matching in initial work in #271.  The issue under discussion here immediately raises it's head, and shows that greedily binding placeholder values can never work.  However, I'd like to avoid doing anything fancy (i.e., backtracking or other SAT solving strategies) entirely.  For the tests I have currently written, it's possible to:

Order the values being matched out of the store, by sorting by [e a v tx added] in some way.  The details of the order are less important than being deterministic.
Never pattern match Value::Set instances: that is, only match Value::Set instances literally.
Never pattern match the keys of Value::Map instances: that is, only match Value::Map key instances literally.  Pattern matching the values of Value::Map instances is pretty much required.

This simplifies the matcher significantly.

Order the values being matched out of the store, by sorting by [e a v tx added] in some way. The details of the order are less important than being deterministic.

If I understand correctly, you're saying that [3 ... 1] ~= [1 2 3]? I get that we could see #{3 ... 1} ~= {1 2 3}, but that seems less obviously true for vectors. I suspect I'm missing something here.

If I understand correctly, you're saying that [3 ... 1] ~= [1 2 3]? I get that we could see #{3 ... 1} ~= {1 2 3}, but that seems less obviously true for vectors. I suspect I'm missing something here.

No, I'm saying that the thing that produces the Value to be pattern matched will ensure the collections are ordered (Value::{Vector,List}) and will order them deterministically, rather than having the pattern matching be clever.  That is, I'll produce [[e a v tx added]] rather than #{[e a v tx added]}, even though conceptually the set of datoms introduced in a transaction is unordered; and I'll impose an (arbitrary but consistent) ordering on the produced vector.
Does that make more sense?  What I want is to get us matching fixed prefixes and suffixes in ordered collections, so we can eagerly bind placeholder values (no backtracking!); and not to handle unordered collections in much generality.

Early morning thought → [1 2 ... 2 3] ~= [1 2 3]. Match? I think it shouldn't, but getting that right will take thought.

@joewalker That definitely shouldn't match. There's an extra 2 in the pattern vet that doesn't exist in the input vec.

Early morning thought → [1 2 ... 2 3] ~= [1 2 3]. Match? I think it shouldn't, but getting that right will take thought.

It shouldn't, in three ways:

Precondition failed: concrete match element count larger than input vec. (4 > 3)
Consume two, split: remaining elements fewer than remaining match elements (1 < 2)
Reverse iterator over [3] can't match the sequence [3 2].

You might be justified in worrying if one were to support multiple splits, in effect creating a sliding window constraint satisfaction, but even so I think this whole thing is possible and safe by sequential calls to match over slices, chaining bounds between them.


New: Add binder image to run tutorials

Try them here (based on this branch): https://mybinder.org/v2/gh/rigetti/qiskit-rigetti/binder?filepath=examples

Note that the duplicate plots is a bug in qiskit's visualization extra. It even shows up in their own docs: https://qiskit.org/documentation/tutorials/circuits/2_plotting_data_in_qiskit.html


Docs for plugin support

Hi again!
I've worked out how to add the Geocoder plugin to my map, by way of the useMap function. I think you should add a comment to the README about how to do this, so that others know it's compatible with plugins.
Here's my code, for what it's worth:
import { useMap } from "solid-map-gl";
import { Component, onMount } from "solid-js";
import mapboxgl from "mapbox-gl";
import MapboxGeocoder from "@mapbox/mapbox-gl-geocoder";
import<EMAIL_ADDRESS>
const Geocoder: Component = () => {
    const map = useMap();

    onMount(async () => {
        let accessToken = map()._requestManager._customAccessToken;
        map().addControl(
            new MapboxGeocoder({
                accessToken: accessToken,
                mapboxgl: mapboxgl,
            })
        );
    });

    return <></>;
};

export default Geocoder;

Cheers,
Cameron

Hi Cameron,
Thanks for that. Yes I really have to document the useMap context provider.
I will update my documentation with you example.


Annotations.py fails- dependency on old sklearn modules?

Dear,
I wanted to perform annotations locally or with our cluster. So I made a new conda environment and went with the script available (for colab session) at your website
upon running any cell where I have to import 'annotations', it throws an error as below.
from annotation import get_pretrained_model_genes, check_genes_is_subset
pretrained_scanvi_path = os.path.join(tissue, tissue + "_scanvi_model")
pretrained_scvi_path = os.path.join(tissue, tissue + "_scvi_model")

training_mode='online'
is_subset = False
if training_mode == 'online':
    pretrained_genes = get_pretrained_model_genes(pretrained_scvi_path)
    query_genes = query_adata.var_names.to_numpy().astype("str")
    is_subset = check_genes_is_subset(pretrained_genes, query_genes)

if is_subset and training_mode=='online':
    ref_adata = ref_adata[:, pretrained_genes]
else:
    training_mode = 'offline'

---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Input In [11], in <cell line: 16>()
      1 # !conda install -c conda-forge sortedcontainers --yes
      2 # !conda uninstall tensorflow --yes
      3 # !conda uninstall tensorboard --yes
   (...)
     14 # !conda install tensorboard --yes
     15 #!conda install -c conda-forge tensorflow --yes
---> 16 from annotation import get_pretrained_model_genes, check_genes_is_subset
     17 pretrained_scanvi_path = os.path.join(tissue, tissue + "_scanvi_model")
     18 pretrained_scvi_path = os.path.join(tissue, tissue + "_scvi_model")

File /mnt/DATA1/Fibrosis/Fibrosis_2022/Annotation/scripts/annotation.py:14, in <module>
     11 import scvi
     12 import seaborn as sns
---> 14 from OnClass.OnClassModel import OnClassModel
     16 from sklearn import svm
     17 from sklearn.ensemble import RandomForestClassifier

File ~/anaconda3/envs/tsapiens/lib/python3.9/site-packages/OnClass/OnClassModel.py:6, in <module>
      3 import sys
      4 from sklearn.preprocessing import normalize
----> 6 from OnClass.OnClass_utils import *
      7 from OnClass.BilinearNN import BilinearNN
      9 class OnClassModel:

File ~/anaconda3/envs/tsapiens/lib/python3.9/site-packages/OnClass/OnClass_utils.py:19, in <module>
     17 from sklearn.metrics.pairwise import cosine_similarity
     18 #from libs import *
---> 19 from sklearn.utils.graph_shortest_path import graph_shortest_path
     20 from scipy.sparse.linalg import svds, eigs
     22 def get_ontology_parents(GO_net, g, dfs_depth):

ModuleNotFoundError: No module named 'sklearn.utils.graph_shortest_path'

It will be nice if a yaml file for the conda environment or list of versions of each package is provided.
Thanks again and Kind regards,
Saeed

Hi,
Just saw a yml file on this github reposiory. I assume it's the one that I can try with.
Thanks and Kind regards,
Saeed

Update: The yml file do not include scanorama and bbknn.
Still not able to manage.

I am having the same issue here. I wonder if this is the right function?
https://scikit-learn.org/stable/modules/generated/sklearn.utils.graph.single_source_shortest_path_length.html?highlight=utils

Hello,
I have the same issue. Do anyone had solved this problem ?


Compatibility with NIPs

A compatibility table in README would be helpful with current NIPs. Would help decide immediate TODOs for new contributors.

Agreed, I will work on that.

Added NIP compatibility list with c7eadb1154afc.


IC-1312 get eosr by referral

What does this pull request do?
Adds an endpoint to get an end of service report by referral id
What is the intent behind these changes?
To be able to retrieve an end of service report for a referral.

I'm a bit confused, does this overlap with #143?

I'm a bit confused, does this overlap with #143?

This story is to retrieve an end of service report by a referral id. The other story is to retrieve a eosr by the id of an eosr.
Each of these stories has been branched off the previous one. As they haven't been merged yet, the work from each is included in each pr if that's what you mean?

As they haven't been merged yet, the work from each is included in each pr if that's what you mean?

Yes, thanks for clarifying. Using the other PR's branch as the base branch for this PR might be useful in these situations, if that's helpful for explicit ordering?

wants to merge 1 commit into IC-1309-get-eosr from IC-1312-get-eosr-by-referral-id


Autosave by observing model.scratch in editor

No issue

removes keypress handling in the editor component
automated value changes via shortcuts still autosave

Whops, I've spotted an unintended side effect here:
In the case that you start a new post, the autosave fires after 3 seconds, regardless of whether you have focused into the editor and typed something. So if you're in the middle of typing a really long title, the screen suddenly redraws and some of your text lands in the editor.
Also seeing this error, but not sure if it's due to this PR or may also occur on master, will check

Ah, sounds like something on this PR. I'll get you more info in a bit
On Wed, Mar 18, 2015 at 9:08 AM, Hannah Wolfe<EMAIL_ADDRESS>wrote:

Whops, I've spotted an unintended side effect here:
In the case that you start a new post, the autosave fires after 3 seconds,
regardless of whether you have focused into the editor and typed something.
So if you're in the middle of typing a really long title, the screen
suddenly redraws and some of your text lands in the editor.
Also seeing this error, but not sure if it's due to this PR or may also
occur on master, will check
https://camo.githubusercontent.com/426d928a8880824c2e3c2ae3d8098ee0da1bb10a/687474703a2f2f7075752e73682f67467156652e706e67
—
Reply to this email directly or view it on GitHub
https://github.com/TryGhost/Ghost/pull/5046#issuecomment-83015717.

Updated with a new commit - will squash if the new code's worth it

This is all shiny and hopefully GTG

Taken this for a spin and it seems like it's working and quite a lot simpler :+1:


Use spread operators instead of Object.assign

Currently babel doesn't work with it.

Fixed by #26


PSX multitap emulation only support DualShock 2 device

Init issue in wired/ps_spi.c that prevent using flightstick/mouse/kb device type on the emulated multitap.

Fixed in v1.4


FunctionApp:az functionapp deployment failed

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp deployment source config-zip
Errors:
HTTPSConnectionPool(host='xxx.scm.azurewebsites.net', port=443): Max retries exceeded with url: /api/zipdeploy?isAsync=true (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x07BFB7B0>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed',))
Traceback (most recent call last):
pip-install-t7say61i\urllib3\urllib3\connection.py, ln 159, in _new_conn
urllib3\urllib3\util\connection.py, ln 57, in create_connection
...

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

Put any pre-requisite steps here...
az functionapp deployment source config-zip --src {} -n {} --resource-group {}

Expected Behavior
Environment Summary
Windows-10-10.0.17763-SP0
Python 3.6.6
Shell: cmd.exe

azure-cli 2.0.77

Additional Context

route to service team

Can you just please re-try the error indicates there was an issue for the CLI client to reach the Scm endpoint for deployment.

@athapong Hope you have re-tried the command again.

closing due to lack of response. Please file a new issue if this is still a problem.

Sorry for my late response..
Once I fixed some issue due to organization policy it's working now.
Appreciate for your help

Sorry for my late response.. Once I fixed some issue due to organization policy it's working now.
Appreciate for your help

which policy was it?


ex.rkt does not exist

Hello, the file ex.rkt seems to be missing:
$ scribble +m tutorial.scrbl
tutorial.scrbl:3:10: cannot open module file
  module path: #<path:/home/orchid/Downloads/redex-aam-tutorial/ex.rkt>
  path: /home/orchid/Downloads/redex-aam-tutorial/ex.rkt
  system error: No such file or directory; errno=2
  context...:
   standard-module-name-resolver

Fixed.  Thanks.


Increase timeouts

Increase default timeout for requests in a wait() to go out from 5s to 60s
Increase default timeout for responses in a wait() (and some other calls) to go out from 30s to 60s
Increase timeout for Tiny MCE to initialise from 2s to 60s
Remove 2s timeout on waiting for firstname field in Edit Client test

Configuration docs here https://docs.cypress.io/guides/references/configuration


Fix concurrency issues

Running cpu and gpu build in parallel, there was a race condition when saving example output
Throttling was broken, fixed with semaphore

Signed-off-by: andreasjansson<EMAIL_ADDRESS>

✨


Applying font color for past events

Somehow, I cannot see the fontWeight: 'bold' manifesting in the rendered titles. Any idea why?

fontWeight: bold is now displayed on iOS. Could not test on Android as build failed (I think @ferrannp is already aware of the issue - https://github.com/barteksc/AndroidPdfViewer/issues/583#issuecomment-397294434 )


Whether release training code or not?

Thanks for your wonderful work!!!
The balance of effectiveness and efficiency of OSEDiff is very shocked!!!
I trained our network with vsd loss, but maybe my training code have some difference with you, the results are terrible, so would you release training code in the future?
Hope your reply!

same terrible results:(

Thank you for your interest. However, we're not currently planning to open-source our training codes in the short term.

Have you tried not using vsd loss, only using lpips loss. First, make sure this step is correct
I'll try, thanks~

Thank you for your interest. However, we're not currently planning to open-source our training codes in the short term.

ok, Thanks for your reply~


1054 bug select contribution type buttons display

Description
Overlapping of divs on dynamic change of contribute create page fields is fixed.
The breadcrumb divs are changed to container classes. The MoveSelector function now places the "select contribution type" buttons above the breadcrumbs at the bottom (div id #cd-multi-steps-text-bottom).
Tried multiple field changes like adding and removing environment variables, open source code radio button on and off , adding and deleting capability/talent.
Fixes #1054
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Merging to develop


Change standard ctx documentation

From: Standard context object containing the current user and the current tenant
To: Current execution context

On hold until inviting guests goes through

Done already.


⚠ ClamAV has degraded performance

In 891127b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 210 ms

Resolved: ClamAV performance has improved in d7620e3 after 1 minute.


Add vimr-snapshot SNAPSHOT-232

After making all changes to the cask:

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[X] Named the cask according to the token reference.
[X] brew cask install {{cask_file}} worked successfully.
[X] brew cask uninstall {{cask_file}} worked successfully.
[X] Checked there are no open pull requests for the same cask.
[X] Checked the cask was not already refused in closed issues.
[X] Checked the cask is submitted to the correct repo.

From the acceptable casks policy (emphasis added):

Refuse legacy versions of gratis or open-source software, unless there is a clear demonstrable need for them.

What’s the need for this snapshot?

The snapshots provide the latest version of VimR and will help to test Swift 4 support.
If you feel this isn't enough justification, feel free to close this PR.

That rule is more for cases like “the latest version has a terrible bug”.
Thank you for the PR, but we’ll refuse it.


support logging build output to JSON in order for Domino pipeline ingestion

WIP obviously, but this was my first pass and along with https://github.com/cerebrotech/charts/pull/552 it actually seems to work. Not sure I love the way it's formulated so it's worth discussion.

@steved I know this is a WIP but just let me know when you want to put :eyes: on it.

@sonnysideup please review. Ignore all Dockerfile, e2e, cmd/, config/, and skaffold changes.

replaced by #22


Amplify console api throwing an error

Describe the bug
amplify console api is returning error for an amplify project configured with api category.
Amplify CLI Version
amplify -v
4.41.2

To Reproduce
$ amplify init
Note: It is recommended to run this command from the root of your app directory
? Enter a name for the project TODOTest
? Enter a name for the environment dev
? Choose your default editor: Visual Studio Code
? Choose the type of app that you're building ios

$ amplify add api
? Please select from one of the below mentioned services: GraphQL
? Provide API name: todotest
? Choose the default authorization type for the API API key
? Enter a description for the API key: testingTODO
? After how many days from now the API key should expire (1-365): 7
? Do you want to configure advanced settings for the GraphQL API No, I am done.
? Do you have an annotated GraphQL schema? No
? Choose a schema template: Single object with fields (e.g., “Todo” with ID, name, description)
? Do you want to edit the schema now? No

$amplify push
✔ Successfully pulled backend environment dev from the cloud.

Current Environment: dev

| Category | Resource name | Operation | Provider plugin   |
| -------- | ------------- | --------- | ----------------- |
| Api      | todotest      | Create    | awscloudformation |
? Are you sure you want to continue? Yes

The following types do not have '@auth' enabled. Consider using @auth with @model
	 - Todo
Learn more about @auth here: https://docs.amplify.aws/cli/graphql-transformer/auth

App not deployed yet.
App not deployed yet.

GraphQL schema compiled successfully.

? Do you want to generate code for your newly created GraphQL API No

✔ All resources are updated in the cloud

GraphQL endpoint: https://xxx/graphql
GraphQL API KEY: xx2-xxx

$ amplify console api
? Please select from one of the below mentioned services: GraphQL
Error opening console.
Missing AmplifyAppId in amplify-meta.json

Expected behavior
amplify console api should open the aws console.
Screenshots
NA
Desktop (please complete the following information):

OS: Mac
Node Version. v12.18.3

Log output
Include any relevant log output under ~/.amplify/logs/amplify-cli-<issue-date>.log
Found this error "LimitExceededException: Failed to create app. You have reached the maximum number of apps in this account."
Additional context
Content from /amplify/backend/amplify-meta.json
{
  "providers": {
    "awscloudformation": {
      "AuthRoleName": "xxxx-authRole",
      "UnauthRoleArn": "xxx-unauthRole",
      "AuthRoleArn": "xxx-authRole",
      "Region": "xxx",
      "DeploymentBucketName": "xx",
      "UnauthRoleName": "xxx-unauthRole",
      "StackName": "xxxx",
      "StackId": "xxxx"
    }
  },
  "api": {
    "todotest": {
      "service": "AppSync",
      "providerPlugin": "awscloudformation",
      "output": {
        "authConfig": {
          "defaultAuthentication": {
            "authenticationType": "API_KEY",
            "apiKeyConfig": {
              "apiKeyExpirationDays": 7,
              "description": "xxx"
            }
          },
          "additionalAuthenticationProviders": []
        },
        "GraphQLAPIIdOutput": "xxx",
        "GraphQLAPIEndpointOutput": "xxxx",
        "GraphQLAPIKeyOutput": "xxx"
      },
      "providerMetadata": {
        "s3TemplateURL": "xxxx",
        "logicalId": "xxx"
      },
      "lastPushTimeStamp": "2021-02-11T18:00:41.433Z",
      "lastPushDirHash": "xxx="
    }
  }
}

@royjit did you hit the 25 app limit in your account? Otherwise, there should be an AmplifyAppId after a successful init

@jhockett ah, looks like I have. I will free up some apps and test again.

Working as expected after removing some stale apps. Thanks!


Move Working Groups off of Amazon Chime

Description
Working groups are hosted on Amazon Chime, where the meeting place for working groups should be somewhere more commonly used across all cloud providers, like zoom.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Seems like we need to be a CNCF project listed in the project-maintainers file which only includes a few SIGs from Kubernetes. @dtzar do you know if there's a way to do this or if there's another way to request?

Discussed out-of-band with @gjtempleton. Basically, the work that is required here is to define a new mailing list for the sub-projects and then create a Zoom account for that mailing list email. Examples can be found in the links below:

Create a new mailing list: https://github.com/kubernetes/k8s.io/pull/5519/files
Create a new Zoom account for the mailing list: https://github.com/kubernetes/community/issues/7238#issuecomment-1628545293


docker 镜像 下载斗鱼直播出错

ykdl报错

Traceback (most recent call last):
File "/usr/local/bin/ykdl", line 10, in
sys.exit(main())
File "/usr/local/lib/python3.7/site-packages/cykdl/main.py", line 209, in main
info = parser(u)
File "/usr/local/lib/python3.7/site-packages/ykdl/extractor.py", line 23, in parser
info = self.prepare()
File "/usr/local/lib/python3.7/site-packages/ykdl/extractors/douyu/live.py", line 62, in prepare
ub98484234(js_enc, self, params)
File "/usr/local/lib/python3.7/site-packages/ykdl/extractors/douyu/util.py", line 98, in ub98484234
ub98484234 = js_ctx.call('ub98484234', extractor.vid, did, tt)
File "/usr/local/lib/python3.7/site-packages/ykdl/util/jsengine.py", line 369, in call
return self._eval(code)
File "/usr/local/lib/python3.7/site-packages/ykdl/util/jsengine.py", line 527, in _eval
raise ProgramError(result)
ykdl.util.jsengine.ProgramError: SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode

经检查更新Node.js可以解决该问题,原docker镜像的Node.js版本为v4.2.6

https://github.com/ForgQi/biliup/pull/66/commits/0f0ce7f27c1a3e8f403412da3584eb5aa9d4904b

感谢PR,已经合并


UF Docks - Resizing dock panel issues if no title is set

@paulovmr  @romartin could you please take a look on this?

Hi @ederign , tested on the new designer and it works fine :)
+1

Hi @ederign, looks good to me! +1


Ruby 2.2+ compatibility

References issue https://github.com/onelogin/aead/issues/8
Gist of the problem:

Caused by conversion to the typed data system ( https://github.com/ruby/ruby/blob/trunk/doc/extension.rdoc#encapsulate-c-data-into-a-ruby-object ) for the openssl extension in https://github.com/ruby/ruby/commit/163cb5b43de68cef7ae4fa053d710098399d1359 too
Related issue was that the modules and classes for OpenSSL etc. was setup in the extension which can lead to namespace clobbering depending on usage context.

Gist of the fixes:

Explicitly require openssl on extension init and let OpenSSL::Cipher and OpenSSL::OpenSSLError be references to the constants defined in ext/openssl instead.
Fixed misuse of the Ruby convention of modules being prefixed with m and classes prefixed with c ( VALUE mOSSLCipher -> VALUE cOSSLCipher )
Support the TypedData system with the secret sauce being the RTYPEDDATA_TYPE(obj) macro for parity for the typed data definition that the aead extension reads with the one that OpenSSL::Cipher allocates objects with.
That way we're not setting ourselves up for potential fails on another upgrade with the proposed hacky fix in https://github.com/FredericHeihoff/aead/commit/70a74051054edfae1dd73b18389d05ba608e37c0

@pallan @csfrancis @jnormore

My C is rusty but this looks okay to me

LGTM 👍


nrf52_bsim: Enable real flash peripheral

nrf52_bsim: Enable real flash(nvmc) peripheral

The HW models include now the real flash peripheral.
Enable it.

manifest: nrf hw models: Use latest w flash

Update to the latest NRF HW models which include
models of the NVMC and UICR peripherals,
as well as some other (very) minor fixes.

drivers: nrf5 flash: Fix for simulation

Use new nrfx hal function to memcpy from flash
instead of accessing the array directly.
That function is inlining a memcpy for real targets,
so there is no practical difference for those.

manifest: nordic hal: Update to latest

Which includes some new APIs and minor changes needed
to enable the simulated flash.

tests/net: For nrf52_bsim use flash model
Now that the nrf52 HW models support it,
let's use their flash model instead of falling back
to the openthread RAM settings backend.

Note that the flash model defaults to just keeping the
flash content in the Linux process heap.

tests/flash: Fix max test size check

The condition just needs to be <=, as it is ok
to write say 128KB in a 128KB storage partition.

CC @ilhanates @canisLupus1313

The CI failure was due to https://github.com/zephyrproject-rtos/zephyr/pull/57640
Hopefully the rerun fixes it


[FLINK-11447][table] Deprecate new Table(TableEnvironment, String)

What is the purpose of the change
This pull request deprecates the constructor new Table(TableEnvironment, String)
Brief change log

Deprecates new Table(TableEnvironment, String)
Introduces Table.lateralJoin, Table.leftOuterLateralJoin
Move the implicit of 'TableFunction to Table' to ExpressionDsl as the otherwise this implicit always has the highest priority

Verifying this change
This change added tests and can be verified as follows:
(example:)

Added tests CorrelateStringExpressionTest.testCorrelateJoinsWithJoinLateral

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (yes)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs & JavaDocs)

@twalthr @dawidwys Could you help to take a look at this PR when it's convenient for you? Thanks in advance.

@flinkbot approve description
@flinkbot approve consensus

@twalthr  Thanks a lot for the remind. I have updated the documentation.


RoadMap

Follow up on https://github.com/dmlc/xgboost/issues/574 This will serve as roadmap issue for Year 2016
This issue will be the centralized place for links to on going proposals and plans on improving xgboost. Please reply to the issue for discussion. The major goals and specifications will be marked as Roadmap Label.
Distributed Version

[x] Distributed python version of xgboost, enable features such as custom objective function
[ ] XGBoost JVM Package.

Data Frame Integration

[ ] Have a clear specification of dataframe to DMatrix conversion.

External Memory

[ ] Being able to use external memory version in language packages, R/python/Julia from native data structure

distributed python is included in this PR https://github.com/dmlc/xgboost/pull/897

not tried this yet, but this iterated bagging seems to me very similar to gradient boosting and paper shows better performance
http://www.cs.utexas.edu/~ml/papers/bv-ecml-05.pdf
But I understand this is actually little bit different algorithm... something between boosting and randomforest...
I have no insight at deep code of xgboost (out of my skill), and not sure how difficult this can be...
But another view: bagging can be paralelize at higher level = building several trees at same time.. so I guess this can be even faster...
But sure, it can be a lot of work for maybe small difference...??

Proposal 1

I believe that this was the Roadmap for last year, is there any update on it ?


refactor: possibly refactor the many launching functions

We currently have like 4 functions to launch providers and get wallets, that might be confusing

There is 3 functions that setup a provider and wallets:
launch_provider_and_get_single_wallet
launch_provider_and_get_wallets
launch_custom_provider_and_get_single_wallet
Since we opted for making the names extra verbose, I think its fine.
However, IMO launch_custom_provider_and_get_single_wallet should also accept WalletConfig and be able to setup multiple wallets. Since its a niche use-case I think its ok if we don't provide a shorthand for just getting a single wallet.

There is also setup_test_provider, setup_test_client.

I think that's too much. We should have something like:
launch_provider_and_get_wallet (default case, people want out of the box provider with a single wallet)
and launch_custom_provider_and_get_wallets  (can customize wallets and launcher).
setup_* functions I'm not sure what to do with them.

The change to the launch_* methods sounds good.
As for setup_test_client, I think we can make it private. setup_test_provider just introduces a wrapper while not taking away any of functionality.

Agree with this I think, updated the issue description.


F-Droid reproducible build failed

app-fdroid-release.apk.html.zip
Is your apk built from the tag? Can you try cleaning the cache and rebuilding it?

Is your apk built from the tag?

Yes.

Can you try cleaning the cache and rebuilding it?

I've updated it: https://github.com/sakethpathike/Linkora/releases/tag/release-v0.7.2

Thanks!

can we reopen? ref: https://gitlab.com/fdroid/fdroiddata/-/jobs/7925026174#L1825

Should I rebuild it using the same tag that was used to build 0.8.1 (29th version)?

Not sure if rebuild can fix it but we can have a try. :)

I've updated it: https://github.com/sakethpathike/Linkora/releases/tag/release-v0.8.1

https://gitlab.com/fdroid/fdroiddata/-/jobs/7950794004 In face 0.8.2 has been built reproducibly... 0.8.1 is also fixed but we'll just skip it. Thanks!


Import normalization

Currently, this repo does not use file extensions when importing modules. It also uses index imports in various locations.
As outlined in differences between ESM and CJS in the Node.js documentation, Node will no longer allow extensionless imports nor use of a directory index for ESM modules.
Webpack – used in this repo – is a bit of a different beast and may support these features for some time to come. For the sake of consistency across repos we could opt to force webpack to also require extensions and disallow use of a directory index:

resolve.enforceExtension
resolve.mainFiles

This would require restructuring of nested modules. For example, refactoring gfx:
src/gfx
├── Color.mjs
├── Device
│   └── index.mjs
├── Screen
│   ├── Layer.mjs
│   └── index.mjs
├── Shader
│   ├── Registry.mjs
│   └── index.mjs
├── Texture
│   ├── Registry.mjs
│   └── index.mjs
...

Into:
src/gfx
├── Color.mjs
├── Device.mjs
├── Screen.mjs
├── ScreenLayer.mjs
├── ShaderRegistry.mjs
├── Shader.mjs
├── TextureRegistry.mjs
├── Texture.mjs
...

You can still keep the directories around with the new style of imports:
src/gfx
├── Screen.mjs
│   ├── Layer.mjs
...

You'd just need to re-export ScreenLayer from Screen.
On a related note: personally, I much prefer naming modules explicitly: Screen/Layer.mjs requires reading the path for context. ScreenLayer.mjs doesn't.

We can still keep the directories around with the new style of imports

Ah, interesting, hadn't considered that variant. Agree on explicitly naming modules.
How would you feel about:
src/gfx/screen/
├── Screen.mjs
├── ScreenLayer.mjs
...

I think getting rid of index where appropriate may be worth it regardless of whichever structure we land on.


How to reference different version dependencies for different frameworks

I am using the Microsoft.Owin NuGet package in my solution.
My solution needs to be compiled for .Net 4.0 and .Net 4.5. I am doing this by having two .csproj files in the same folder. Both of the .csproj files reference all of the same source files but target different .Net framework versions and specify different output folders.
This works great with NuGet, but I can not see how to make this work with Paket because projects compiled for .Net 4.0 must reference Microsoft.Owin version 2.1.0 and projects compiled for .Net 4.5 must reference Microsoft.Owin version 3.0.1 and I can only have one paket.references file in my project folder.
You can see my source code here
Is this possible with Paket?

You will need to use paket groups to do this.

Why not have 2 paket reference files...

Yes, split out your framework-dependent deps into two groups in your paket.dependencies file. Each group should have the appropriate framework constraint in it.
source ...

nuget ...
nuget ...

group Net40Deps
  source normal_nuget
  framework: net40

  nuget Microsoft.Owin 2.1.0

group Net45Deps
  source normal_nuget
  framework: net45

  nuget Microsoft.Owin 3.0.1

Then, in the single, unified paket.references, you can conditionally include some deps or the other based on framework like so:
# deps from main group
Blah
Foo
Bar

#deps for net40
group Net40Deps
  Microsoft.Owin framework: net40

# deps for net45
group Net45Deps
  Microsoft.Owin framework: net45

Two files with the same name in the same folder?

On Oct 1, 2018, at 12:27 PM, Chris Blyth<EMAIL_ADDRESS>wrote:
Why not have 2 paket reference files...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I can not find any documentation on conditional groups, but some of the error messages referred to them, so they are supported in some way.
I tried making files exactly as you suggested, and many other variations of placement for the various parts of the syntax, but I could not find any combination that worked.
With this paket.dependencies file
framework: net40, net45
storage: packages

source https://nuget.org/api/v2

// These dependencies are fixed to a specific version because
// they are passed on in Nuspec files

nuget Ioc.Modules = 1.3.2
nuget Newtonsoft.Json = 7.0.1
nuget Svg = 2.3.0
nuget Urchin.Client = 1.2.1
nuget Moq = 4.2.1507.0118
nuget Moq.Modules = 1.0.2

nuget Owin = 1.0
nuget Owin.Framework = 2.6.8
nuget Owin.Framework.Mocks = 2.6.8

// These dependencies are not passed on in Nuspec files

nuget NUnit ~> 3.5

// These dependencies are only for the sample apps

nuget Ioc.Modules.Ninject ~> 1.3
nuget Microsoft.Owin.Host.SystemWeb = 2.1.0
nuget Microsoft.Web.Infrastructure = <IP_ADDRESS>
nuget Owin.Framework.DefaultDocument ~> 2.6
nuget Owin.Framework.Documenter ~> 2.6
nuget Owin.Framework.ExceptionReporter ~> 2.6
nuget Owin.Framework.NotFound ~> 2.6
nuget Owin.Framework.Urchin ~> 2.6
nuget Ninject = <IP_ADDRESS>

// These Microsoft packages need specific versions for specific .Net frameworks

group net40
  source https://nuget.org/api/v2
  framework: net40
  nuget Microsoft.Owin = 2.1.0

group net45
  source https://nuget.org/api/v2
  framework: net45
  nuget Microsoft.Owin = 3.0.1

and this paket.references file
Ioc.Modules
Newtonsoft.Json
Owin.Framework

group net40
  Microsoft.Owin framework: net40

group net45
  Microsoft.Owin framework: net45

The output from paket install is
Paket failed with -> Installation Errors :    Package Microsoft.Owin is referenced in different versions in {path}.csproj (4.0 vs 2.1), (inspect the lockfile for details) to resolve this either add all dependencies to a single group (to get a unified resolution) or use a condition on both groups and control compilation yourself.

 The framework: net40 qualifiers are not necessary in the paket.references file. The difficult and fragile part is that you need to get the whole dependency tree into the same group, but some of the modules I am referencing are not version specific and Paket complains if I put these into groups so I can't just duplicate everything in both groups. This means that there are definitely dependency graphs that are impossible to configure in Paket.

My team and I hit this issue while trying to share common libraries across projects within the solution who may target framework, netstandard, or netcore.
Paket will not properly restore packages, yielding a significant amount of MSB3245, MSB warnings (> 300) when building, meaning it flat-out couldn't find the package and won't do anything about it.
To compound the issue, Paket seemingly doesn't respect when a framework was upgraded when in this confused state. Despite performing an 'add' on every single dependency in every single project effectively re-installing every possible dependency, paket leaves the package hint paths pointing to an older framework version than what the projects were updated to.
@Bikeman868 : did you have much success getting around this or did you abandon it / stick with an alternate pattern?
This is also similar to what we were experiencing: https://github.com/dotnet/standard/issues/481
I have not tried the forceful binding redirect option yet however--I'm hesitant to do that when there is still an underlying issue with the graph that might not be solvable. I should point out that the issue mentioned above in dotnet/standard ultimately resulted in this announcement: https://github.com/dotnet/announcements/issues/31

What do you mean it does not restore packages properly?

@forki Transitives, in particular, will not be copied to the output directory. They may “restore” in the sense that they make it to the package cache, but they will not be copied to the output folder which causes runtime errors.
Some of these transitives, when converted/specified as a direct dependency in the paket.references file, will still not be copied to the output directory. That one in particular is especially frustrating.
I am using the latest non-preview Visual Studio 2019 at the time of writing, but I’m trying to get a CI build working. I’m therefore using MSBuild from the command line for the majority of builds. There are also dotnet build commands included, but those all work without issue—its only the .NET framework projects that are encountering this issue.

do you have a small repro?

@forki This repository is somewhat large and is the result of attempting to merge together a few different repositories into a single solution.
I may be able to come up with a small repro, but it may be difficult to do so. I'm mostly looking for information about workarounds and whatnot, as this may be the result of any number of issues that were encountered while attempting to merge said repositories.
I'm not logging a bug against paket, I'm mostly looking to see if the issue was worked around in the past or if there is perhaps some what to accomplish what the original author was trying to do. I don't want to place the burden on anyone other than myself at this point basically.

When I opened this ticket I included a link to an open source project that demonstrates this problem.
Whilst waiting for a bug fix I am manually maintaining project references but it’s a real pain.
I recently started using Visual Studio 2019 and it’s Nuget Package Manager is considerably better than in earlier versions so I am considering abandoning Paket.

On Aug 12, 2019, at 8:30 AM, Jim Hume<EMAIL_ADDRESS>wrote:
@forki This repository is somewhat large and is the result of attempting to merge together a few different repositories into a single solution.
I may be able to come up with a small repro, but it may be difficult to do so. I'm mostly looking for information about workarounds and whatnot, as this may be the result of any number of issues that were encountered while attempting to merge said repositories.
I'm not logging a bug against paket, I'm mostly looking to see if the issue was worked around in the past or if there is perhaps some what to accomplish what the original author was trying to do. I don't want to place the burden on anyone other than myself at this point basically.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Android react-native sometimes optimizes and removes View as single child element of custom ScrollView, which causes an exception

In Android, ScrollViews are only allowed to have a single child element.  I created a custom component that extends ScrollView, and I wanted to setup my layout like this:
<CustomScrollView>
    <View>
        <Text>text</Text>
        <TextInput />
    </View>
</CustomScrollView>

I found that if I didn't add any styling or anything to that View, that it would get optimized out and removed from the view hierarchy, which would cause an exception ("ScrollView can host only one direct child").  I now know that adding a backgroundColor style will prevent it from being optimized out, but I haven't investigated what will prevent the optimization.
So basically, the optimization code needs to check whether the parent is a ScrollView, and if it is, don't optimize out any views.
I'm using the latest version in master as of a couple days prior to this posting (https://github.com/facebook/react-native/commit/736e90053243a03c48c00f072a2f3bbbdcb07567), which is after released version 0.19.

try setting collapsable to false.
<CustomScrollView>
    <View collapsable={false}>
        <Text>text</Text>
        <TextInput />
    </View>
</CustomScrollView>

Reference from https://facebook.github.io/react-native/docs/view.html#collapsable

Views that are only used to layout their children or otherwise don't draw anything may be automatically removed from the native hierarchy as an optimization.

This worked, thanks!


plot_species fails

The plot_species function fails with "Error in plot.window(...) : need finite 'ylim' values" on index 5. Would also be good if this output depth plots.

Fixed with 5b6d34802c204b445e429f3f70546a754bc6edcf


Causing build error: "Module not found: Default condition should be last one"

When trying to use this library in a next.js project I get a build error with the message "Module not found: Default condition should be last one"
According to https://stackoverflow.com/questions/76127288/module-not-found-error-default-condition-should-be-last-one this is cause by the "default" key in an export map not being the literally last key, which kinda sounds absurd but I'm not too familiar with how export maps work.
https://nodejs.org/api/packages.html indicates that the "default" key "should always come last", but maybe it actually must?
Confirmed that swapping the order of the keys in the package.json file within my node_modules fixes the issue

@spalger good catch! Can you show me your fix/edit from package.json? Or feel free to send PR - I will merge it.

Opened https://github.com/sentienhq/ultralight-s3/pull/13


sparse lanczos solver

This PR implements a sparse lanczos solver for real symmetric or complex hermitian
(sparse) matrices.
This PR depends on stuff in #253, so  #253 should probably be merged first.
@Thenerdstation could you have a look at this and #253? I need this stuff for the DMRG
that I'm coding, would like to reduce the number of dependent branches on my fork a bit.
thanks!

Hey @Thenerdstation can you have a look at this and #234? removed the TensorNetwork dependencies and merged in the latest changes. If this is in by monday I could start working on the DMRG during the code sprint

The only case I can make this work is by specializing eigsh_lanczos to DMRG (this means it will ONLY work for a DMRG-style matrix vector products). This is really ugly, but I can do it ...

actually, even if specializing this will still be really messy if we want jitted functions (either tensorflow or jax). I think we need to code this seperately for each individual backend, the annoying thing is that we can't have DMRG if we don't have this, and there is no alternative.
I will raise individual issues for all these cases.

Additionally, this also means that we REALLY need our custom eigensolver (there's no alternative)

You have not convinced me this is the case. Especially since any eigenvalue solver we make is going to be worse than any one developed on the libraries.
Why can't we just use jax's eigen solver?

neither of jax, tensorflow or pytorch have a suitable eigensolver. Only numpy is offering sparse solver, but wrapping those in jit is essentially impossible/useless (they can only run n CPU).

From my experience, my lanczos eigensolvers usually work equally well for DMRG applications but are considerably faster than e.g. scipy arpack. Moreover, to implement real time solvers like
e.g. TDVP (or even a generic time dependent lanczos) we need additional control over return
values and control-flow, which are not accessible from e.g. the standard scipy API (or  can only be implemented with some amount of hacking). You have to take my word for it (or Guifre's or Ash's, if you need more convincing): if we want a backend independent DMRG, we need this solver.

I was hacking around with jax this morning to see if I can get this to work, but it turns out to be quite a headache (same for tensorflow). Bringing all of this together into a unified API is going to be a lot of work. I don't have so much time right now, so I'll close this PR and put the DMRG on ice.

Ok, first off don't try to implemented your own eigensolver. Numerical stability is very hard, and I don't want to put resources into testing/maintaining it when there are full time researchers on that problem that will do it better than we ever can.
Second, JAX has several eigen solvers builtin, and I don't understand why you can't use them.
jax.scipy.linalg.eigh
jax.numpy.linalg.eigh
jax.numpy.linalg.eig

Problem is we don't have the sparse ones (scipy.sparse.linalg.eigsh), however I'd prefer to fix this at the source if possible, seems Jax is tracking it
https://github.com/google/jax/issues/765

thanks @MichaelMarien, that's what I've been trying to say! While I would totally prefer a jax-native implementation, I doubt that this will come soon. google/jax#765 suggesting implementation of sparse arrays, but if this suffices for our case depends very much on the final implementation. There are many different definitions of "sparse" and the one we need is rather uncommon, namely that our matrix H is composed of a small sum of direct products. The "standard' definition of having almost only zeros in the matrix is useless in our case. Furthermore, this will only fix one case: what about pytorch and tensorflow?

Ok, first off don't try to implemented your own eigensolver. Numerical stability is very hard, and I don't want to put resources into testing/maintaining it when there are full time researchers on that problem that will do it better than we ever can.
Second, JAX has several eigen solvers builtin, and I don't understand why you can't use them.
jax.scipy.linalg.eigh
jax.numpy.linalg.eigh
jax.numpy.linalg.eig

While I would prefer having someone else build this for me, it has unfortunately in the past turned out that the burden usually falls to use to do this. I have meany years of excperience building these things, and controlling stability is usually not too complicated. We can also keep this hidden from the public API.

Aw, I see now. Yes I was going by the "zeros nearly everywhere" definition of sparse, which tensornetwork doesn't even support anyway. (at least we've never tested it).
So the issue right now is that we don't have an way of finding the eigen values of a tensor network when the network is treated as a linear operator. I see... ok I now take back that we shouldn't implement it ourselves, as I don't think we have much of a choice.
I'm still placing a hard requirement that any implementation we do has to run under a jax.jit and tf.function calls, as failing to support these will effectively remove any computation advantage we had before. The main blocker in doing that is the while converged and passing the function as an argument. It might be best to have totally different implementation for each backend and throw a NotImplementedError for backends we haven't supported yet.
The function passing is less of an issue in my mind (Just don't jit that function directly). The while converge is going to be more difficult, but we can likely "fake" it by taking a predetermined number of  steps or using the jax.while_loop.

I agree that the best course of action is to have this implemented in each backend independently. My hacking attempts today have confirmed that jax.jit is OK with "partialized" functions, so this indeed won't be an issue. I think it will be possible to get an equivalent implementation using  jax.lax.while_loop and jax.lax.cond control-flow ops, it's just a bit tricky to do.
I think a first step is to support it for numpy, and then take it from there. Not sure if there are any complications to expect from pytorch

I'll do the JAX implementation of you can do a numpy version.

No problem. How do you suggest to implement in the backend? Should this be a staticmethod of the backend?

Yeah that sounds ok

Allright, I'll open a new PR for this!

I'll try to come back as quickly as possible. I have some hard deadlines coming up, will probably take me a few days

No worries!


Fix: Allow running a SignalK Server with a simple include

#548 Changed the _dirname path to app.config.configPath so that users would be able to set their own configurations. We are now able to use a .env file to set up our own configurations and just npm install signalk and use it as a dependency. This allows for a more robust and flexible use of signalk.

The code above your change does the exact same thing. See line #31.
This change would make so that we don't load plugins that get installed by default.
Will look more into it tomorrow, but app.config.appPath might be the right choice here.

Thank you for pointing that out @sbender9. We realized that we just needed a check for that folder. We reconstructed the function with keyword to create the check and so that it would be able to check the app.config.configPath without throwing an error message. Before we did this change we would get a package.json not found. With this fix no errors were thrown.

@tkurki when we tried changing the configuration path to our own folder in our application and using SignalK as a node module we would get an error stating that our configuration path was not in the specified location, even though it was, we reconstructed the function so that this would not happen. So now on our application can use SignalK as a node module. Since we were only using certain functionality from SignalK before this fix we had to manually change the bit of code so that we wouldn't get this error.

I believe @sbender9 is correct. app.config.appPath is what we're looking for here. I'm setting appPath with the value returned from the app-root-path module and making the change in the first commit with appPath fixes this for me.
I'll submit a pull request on the pull request.

Okay @taggie05 I made a pull request https://github.com/taggie05/signalk-server-node/pull/1 If you merge that I believe it will update the code here. I also merged the master branch.

Did #601 take care of everything here?

I have to check a couple things that I was working on with #595 to see if anything is left. I'm working on this today.

I am closing this, things proceed in #695, #689, #691. Let me know if we missed something.


admin auth fixes

Codecov Report

Merging #188 (18bb72f) into master (050f0e7) will decrease coverage by 3.81%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
- Coverage   31.81%   28.00%   -3.82%
==========================================
  Files          92       92
  Lines        3860     3875      +15
==========================================
- Hits         1228     1085     -143
- Misses       2523     2702     +179
+ Partials      109       88      -21

Flag
Coverage Δ

integration
?

unittests
28.00% <11.11%> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/interceptors/auth.go
0.00% <0.00%> (ø)

internal/topic/validation.go
76.92% <0.00%> (-3.08%)
:arrow_down:

service/web/adminserver.go
26.92% <0.00%> (-0.71%)
:arrow_down:

internal/credentials/model.go
68.00% <12.50%> (-20.24%)
:arrow_down:

internal/credentials/validation.go
65.38% <33.33%> (-5.45%)
:arrow_down:

pkg/messagebroker/kafka.go
0.00% <0.00%> (-35.51%)
:arrow_down:

pkg/messagebroker/factory.go
33.33% <0.00%> (-33.34%)
:arrow_down:

pkg/messagebroker/validations.go
35.71% <0.00%> (-7.15%)
:arrow_down:

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 050f0e7...18bb72f. Read the comment docs.

No longer needed.


absurd!

that you've removed the precious

See: https://github.com/pablo-cabrera/bilbo/blob/bcd654c490a30db4d07aef7978acbabcd7d001c9/lib/bilbo.js#L156-L181


Enhancement - Switches

-v should still compile (used in a script you would get version number when you compile)
They should also combine.
e.g.
-lv

Same as enhancement #32


Add styles for modal button icons

Centering the text and image

Reviewer: @aldipower

we got an integration test failing. looks like it might be legit

these are the integration tests that failed, looks like they timed out ):

testes failing

Tests good!

i'll defer to @jfurrow for this one

Chris and I tackled this together earlier and I should have given my :ship:s at that point in time!

Will :ship: when tests pass!

Please rerun these tests!


GPP information in openRTB 2.5 adapters.

The GPP information is not made available into the openRTB 2.5 adapters when the auction request is openRTB 2.6.
The BidRequestOrtb26To25Converter class provides in the modifyRegs method the gdpr and the usPrivacy data but not gpp.
final ExtRegs extRegs = ExtRegs.of(gdpr, usPrivacy);
copyProperties(regs.getExt(), extRegs);

The gpp should be provided under the extension object.

@radubarbos - As far as we know, regs.ext.gpp was never formalized. It was in a June 2022 draft, but ended up going directly into regs.gpp. So we would prefer that you upgrade your adapter to be able to handle ORTB 2.6.
What it takes to support 2.6 is:

don't crash on unknown fields
support regs.gdpr, regs.us_privacy, source.schain, user.consent, and user.eids.

However, as an interim, we would accept a community pull request that changes the downgrade behavior for regs.gpp and regs.gpp_sid.

Are you saying that until our backend are updated we can not send gpp info?

The point of an "adapter" has always been to prepare the Prebid Server request for the backend. If your backend doesn't understand 2.6, there's always an option to do the "down-converting" yourself -- i.e. move regs.gdpr to regs.ext.gdpr, etc.
So those are your choices:

change your adapter to accept 2.6 fields and do whatever conversion/moving needed
submit a PR against PBS-core's downgrade code change the regs.gpp downgrade behavior. We're very busy right now and could use the help. If you absolutely cannot do this, will try to squeeze this in the next few weeks.

Thank you for the information.
We'll change our adapter to do the conversion ourselves, we feel it's the fastest approach and we can control better what it's sent to our back-ends until they support openRTB 2.6.
This issue can be closed.

Awesome thanks. Yes, we agree that the purpose of an adapter is to adapt to whatever the backend needs!


Showcase an idea to convert base Transfomer model for Masked input training.

Hi @geek101 ! Thank you for the contribution. Unfortunately, I cannot merge the pull request, because our goal is to keep the repository more or less the same (modulo fixing bugs) as it was when we were working on the paper. As an alternative, you can share a link to your implementation in discussions or issues.


Feat/cache repository methods

This seems less testable than annotating the service methods but cleaner in terms of where methods are used.

For what it's worth, compared to the  current 'master' branch, this one runs a fair bit faster (in limited test scenarios, anyway). Sync'ing posts on master takes 560ms, this branch takes 190ms. The main time saving is in the SiteSyncService.findById() which takes up 314ms on master (multiple calls), and in this branch only 5ms.

For what it's worth, compared to the current 'master' branch, this one runs a fair bit faster (in limited test scenarios, anyway). Sync'ing posts on master takes 560ms, this branch takes 190ms. The main time saving is in the SiteSyncService.findById() which takes up 314ms on master (multiple calls), and in this branch only 5ms. Caching at work! :)

This sounds like a good improvement but were you thinking we should do better?  I imagine the other lookups e.g. placementSyncService.findByEmployingBodyId() are a large part of the remainder.  I think we may need to put those in additional cache.

For what it's worth, compared to the current 'master' branch, this one runs a fair bit faster (in limited test scenarios, anyway). Sync'ing posts on master takes 560ms, this branch takes 190ms. The main time saving is in the SiteSyncService.findById() which takes up 314ms on master (multiple calls), and in this branch only 5ms. Caching at work! :)

This sounds like a good improvement but were you thinking we should do better? I imagine the other lookups e.g. placementSyncService.findByEmployingBodyId() are a large part of the remainder. I think we may need to put those in additional cache.

Yes, agreed (sorry, the 'for what its worth' was not to pooh-pooh the improvement, but rather just to reflect my slight concern over the testing I've done - I'm not sure it is representative of running it 'live'). Funnily enough, the placementSyncService.findByPostId() calls take ~100ms in both this branch and master, which seemed odd to me. I was going to look into that a bit, but if you have any ideas why the caching doesn't seem to have improved matters there, it might be good to discuss. This is over a set of 100 calls to sync a Post, where the Post's trainingBodyId and employingBodyId differ from call to call but the Post lD is the same. Its a bit weird.

Of course, the placementRepository.findByPostId() is not cached at this point, that is why the performance has not changed. I need more coffee! :)
Results of Site-based syncing also show good improvement: ~3000ms for 'master' vs. 540ms for this branch (a lot of the efficiency is in the large number of calls to PostSyncService.findById(): these take 2450ms on master, only 28ms on this branch)


adding key length check

No tests, no docs, no nothing. :fearful:. :stuck_out_tongue:.

worst commit ever !


Improve directory listing

Feature Request
Is your feature request related to a problem? Please describe.
The directory of course documentation refreshes with the page and returns to the top after each click
Describe the solution you'd like
Can the directory on the left side of the document keep the position after clicking, instead of refreshing to the first one along with the page, which is not friendly to long directories

Please provide the steps to reproduce the behaviour on the Demo site.


Revert "Create wg_age_of_quic.md"

Reverts nprint/benchmarks#1

@jpcsmith I merged your page without commenting on accident. Looks great overall, thank you for the submission. A few very quick things:

It's likely ideal that we have either a single optimization metric (and single leaderboard) or multiple leaderboards if there are multiple optimization metrics. Whichever you prefer.
The r-precision link seems to be broken?
"There are additional samples which would need to be removed to have a dataset with exactly 200 samples per monitored class". Could you just take a sentence or two and expand on this further in the description? I'm not quite following.

Hopefully these are very quick fixes, thanks again for the submission!


Supporting v2 handlers and removing removed.

This would allow the wiring up of v2 handlers and preventing wiring of removed on V2 API.
Wanted to get feedback if this approach is suitable before I continue with adding tests to cover the validation aspect.
If you like the approach this is taking do you have any guidelines around testing against multiple versions of monolog? e.g. something like this https://github.com/g1a/composer-test-scenarios

Thanks @gmponos - added that message. What do you think?

Hello @inverse

any experience with multiple version testing? Is that covered by the definitions within travis?

To be honest no. I don't have that much.

WDYT?

What do you think?

Thank you for taking into account my feedback.. I truly wanted also the your opinion (and from the maintainers.. ) if it makes sense to have a deprecation on the configuration level.. It might be overhead since a deprecation will be triggered when the handlers will be initialized:
https://github.com/Seldaek/monolog/blob/1.x/src/Monolog/Handler/HipChatHandler.php#L100
I think that symfony configuration has a special option that you can set a config option as deprecated.. not sure if raises also a deprecation notice and if it would make sense to have it here as well..
Last sorry if I caused any trouble...

@gmponos Ahh you're right - it probably makes sense to allow the package not the bundle to flag such deprecation.
As for the Symfony one - looks like that's more for configuration keys which in this case is a string type which I don't think makes sense.
See: https://symfony.com/blog/new-in-symfony-3-4-deprecate-configuration-options

@lyrixx any input on this direction?

Thank you.


Remove unnecessary backslash in Gitea customization docs

Resolves #8247

Codecov Report

Merging #8249 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8249      +/-   ##
==========================================
+ Coverage   41.58%   41.58%   +<.01%
==========================================
  Files         488      488
  Lines       65211    65211
==========================================
+ Hits        27117    27120       +3
+ Misses      34608    34602       -6
- Partials     3486     3489       +3

Impacted Files
Coverage Δ

models/repo_indexer.go
68.21% <0%> (-1.94%)
:arrow_down:

models/repo_list.go
74.14% <0%> (+0.97%)
:arrow_up:

routers/repo/view.go
43.54% <0%> (+1.01%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
: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 2a2b46c...46c1ee2. Read the comment docs.


Outbound message hitting spam threshold cannot be redelivered

Hi,
I just had a message trigger an outbound spam threshold set to quite low and the spamassassin's classification not being something I want to worry about (for curious of you, it's LH_URI_DOM_IN_PATH - URI: Long-host URI having a domain name in the path part).
The message rightfully hit a Hard Fail:

And it's OK, but I then wanted to increase the threshold to let this one through, so I went to advanced settings and set it to 3.2. For my surprise, this happened:

Tried restarting Postal and spamassassin, but nothing budged.
So I thought to myself - hmm, ok, let me set it higher. Still, nothing helped:

I think that an outbound message once marked as spam, won't be re-evaluated, hence, creating this issue.

Definitely something awry here

Spam checks are run when the message arrives with Postal not when delivery is attempted to an external destination. In this case the message was detected as spam and won't be rechecked on subsequent delivery attempts.


Question: How is "uv" intended to be pronounced?

Is it "you-vee"? Or like "of"? Or something else entirely?

I'd guess "you-vee" considering what the name means :)

Nice, thanks!


Successive installation crashes inputstream.adaptive

When installing Widevine CDM twice in a single Kodi session, it triggers a crash in inputstream.adaptive.

I reported this at: peak3d/inputstream.adaptive#309

I can not upload input.streamadaptive from either repozitory or zip file for kodi leila running ubuntu focal fossa and/or change widewine setting


Remove node.js 8 from test.

Current tests fail because rollup does not support node 8.
https://github.com/rollup/rollup/issues/3594

Node 8 has been EOL for over a year now. Look here for actively maintained Node versions:
https://nodejs.org/en/about/releases/
We should test for 14, which is in active LTS. 10 and 12 are in maintenance LTS with 10 EOL soon 2021-04-30.
For some reason Node 10 test is failing. I tested this locally as well. I was able to replicate. Node 10 fails and Node 12 and 14 pass.

I figured out the cause.
File src/utils.js
Line 94: rimrafSync(tmpDir);
Behaves differently based on Node version.
The tmpDir variable is:
/home/nake89/.degit/github/mhkeller/degit-test-repo-nested-actions/tmp
I killed the script before executing rimrafSync(tmpDir) and looked at the folder contents:
~/projects/tiged >>> ls -la /home/nake89/.degit/github/mhkeller/degit-test-repo-nested-actions
total 20
drwxr-xr-x 2 nake89 nake89 4096 14. 2. 19:55 .
drwxr-xr-x 7 nake89 nake89 4096 11. 2. 23:32 ..
-rw-r--r-- 1 nake89 nake89  435 11. 2. 23:32 068d3374bcc5fed6fddf077f80826a74bf79720d.tar.gz
-rw-r--r-- 1 nake89 nake89   80 14. 2. 20:01 access.json
-rw-r--r-- 1 nake89 nake89  112 12. 2. 15:43 map.json
~/projects/tiged >>>

rimramSync() is just 	fs.rmdirSync(dir, { recursive: true, force: true });

More debugging:
/tmp/rmtest >>> nvm use 10
Now using node v10.23.3 (npm v6.14.11)
/tmp/rmtest >>> node index.js
Node version: v10.23.3
Running: fs.rmdirSync("/tmp/rmtest/this-dir-doesnt-exist", { recursive: true, force: true });
{ Error: ENOENT: no such file or directory, rmdir '/tmp/rmtest/this-dir-doesnt-exist'
    at Object.rmdirSync (fs.js:684:3)
    at Object.<anonymous> (/tmp/rmtest/index.js:9:6)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
  errno: -2,
  syscall: 'rmdir',
  code: 'ENOENT',
  path: '/tmp/rmtest/this-dir-doesnt-exist' }
/tmp/rmtest >>> nvm use 12
Now using node v12.20.2 (npm v6.14.11)
/tmp/rmtest >>> node index.js
Node version: v12.20.2
Running: fs.rmdirSync("/tmp/rmtest/this-dir-doesnt-exist", { recursive: true, force: true });
/tmp/rmtest >>> nvm use 14
Now using node v14.15.4 (npm v6.14.11)
/tmp/rmtest >>> node index.js
Node version: v14.15.4
Running: fs.rmdirSync("/tmp/rmtest/this-dir-doesnt-exist", { recursive: true, force: true });
/tmp/rmtest >>>

So indeed the behavior of fs.rmdirSync has changed between node versions. Post node 10 versions don't throw errors on dir not existing.
Is the dir supposed to be empty? If so, then I could fix this by ignoring the empty dir throw.
If there are no objections I'm gonna safely ignore that error.
"removes and adds nested files"  test at some point runs stashFiles. Not sure why. Which in turn runs rimrafSync on line 94 on src/utils.js
@ArtskydJ Do you wanna weigh in? If not I'm gonna ignore this error, because it appears to be a false positive. As this error occurs only pre 12 and ignoring this error does not seem to break things but test actually passes.

Even if I fix that like I said. I get another error caused by rimrafSync at line 102
https://github.com/tiged/tiged/blob/main/src/utils.js#L102
And I found out why. Because node 10 fs.rmdirSync does not take options.
https://nodejs.org/docs/latest-v10.x/api/fs.html
https://nodejs.org/docs/latest-v12.x/api/fs.html#fs_fs_rmdirsync_path_options
This project used to use
 import { rimrafSync } from 'sander';
before switching to native fs. Problem with native fs is that it has never worked with node 10 or previous versions.
I would however not go back to using sander because it is not really maintained and fs-extra is what is more commonly used to extend fs.
I will look more at this tomorrow.

I chose rimraf instead of fs-extra. Since we only need rimraf function and not all of fs-extra functionality. Also it has less deps. Tests work now. Updated badges as well.


BUG- When a table is filtered, on_action() does not pass the correct row number of the original DataFrame

Description
If a user filters a table, and an on_action() callback is set on the table, then the payload["index"] contains the row number of the filtered table, but there is no way to know what real row that is in the original DataFrame.
We are pretty sure that the same error occurs with on_edit() as well.
How to reproduce
from typing import Any

import pandas as pd

from taipy.gui import Gui, Markdown, State

table1 = pd.DataFrame({"x": [1, 2, 3], "y": [4, 5, 6]})

indexval = -1
xval = "-1"
yval = "-1"

page = Markdown(
    """

<|{table1}|table|filter=True|rebuild=True|width=fit-content|on_action=on_action|>

<|Indexval|text|> <|{indexval}|text|>

<|xval|text|> <|{xval}|text|>

<|yval|text|> <|{yval}|text|>

"""
)

def on_action(state: State, var_name: str, action: str, payload: dict[str, Any]):
    indexval = payload["index"]
    xval = state.table1.iloc[indexval]["x"]
    state.xval = str(xval)
    yval = state.table1.iloc[indexval]["y"]
    state.yval = str(yval)
    state.indexval = indexval

if __name__ == "__main__":
    gui = Gui(page=page)
    gui.run(title="Filter Action")

Sequence of events:

Filter the table on "x" "equals" "3"
That will leave one row.  Choose that row
The action updates the index values and the corresponding row of the DataFrame, which is not the filtered value

Expected behavior
The row number should be of the filtered DataFrame, or there should be some way to know what rows remain after filtering.
Runtime environment
Please specify relevant indications.

OS:  Windows 10 2H22
Browser: Chrome  114.0.5735.134
taipy 2.3.1

Good find! Using the index in the payload seems impossible as it corresponds to the filtered data.


[1tv] Add support for hls

Hello coders, as soon previous request was a bit postponed here, I want to rise up another 1tv.ru issue, please pay attention, people.
R:>youtube-dl -v -F http://www.1tv.ru/shows/vremya-pokazhet/vypuski/ukraina-bez-rossii-vremya-pokazhet-vypusk-ot-19-01-2017
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-F', 'http://www.1tv.ru/shows/vremya-pokazhet/vypuski/ukraina-bez-rossii-vremya-pokazhet-vypusk-ot-19-01-2017']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2017.01.18
[debug] Python version 3.4.4 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg 3.2.2
[debug] Proxy map: {}
[1tv] ukraina-bez-rossii-vremya-pokazhet-vypusk-ot-19-01-2017: Downloading webpage
[1tv] ukraina-bez-rossii-vremya-pokazhet-vypusk-ot-19-01-2017: Downloading JSON metadata
[download] Downloading playlist: Украина без России. Время покажет. Выпуск от 19.01.2017
[1tv] playlist Украина без России. Время покажет. Выпуск от 19.01.2017: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[info] Available formats for 401872:
format code extension resolution note
ld mp4 unknown 350k
sd mp4 unknown 950k
hd mp4 unknown 3800k (best)
[download] Finished downloading playlist: Украина без России. Время покажет. Выпуск от 19.01.2017
What do we see here ? Usual picture, ld is low quality 640x360 video, sd is standard quality in the same res, yet hd that was 1280x720, now when I try to get 'hd' version of video I still get SD one.
It would've been ok to understand if I did not use netvideohunter plugin for firefox's fork named Seamonkey, and this NVH is ABLE to get separated .TS chunks those ARE in 1280x720 res (when NVH is saving those .tses to disk they're renamed to .mp4, as it shown here):
C:>ffprobe.exe "Украина без России Время покажет Выпуск от 19012017.mp4"
ffprobe version 3.2.2 Copyright (c) 2007-2016 the FFmpeg developers
built with gcc 5.4.0 (GCC)
[skip]
Duration: 00:00:10.07, start: 10.106333, bitrate: 4068 kb/s
Program 1
Stream #0:0[0x100]: Video: h264 (High) ([27][0][0][0] / 0x001B), yuv420p(progressive), 1280
x720 [SAR 1:1 DAR 16:9], 25 fps, 25 tbr, 90k tbn, 50 tbc
Stream #0:1[0x101]: Audio: aac (LC) ([15][0][0][0] / 0x000F), 48000 Hz, stereo, fltp, 130 kb/s
Stream #0:2[0x102]: Data: timed_id3 (ID3 / 0x20334449)
I.e., we see here that YDL is being "cheated" by direct url pointing, yet 1tv.ru is able to give out (half)HD videos, so I am asking - could someone put his/her hands on this case to fix YDL to get (half)HD res videos from 1tv.ru along with others - ld and sd ? THANKS IN ADVANCE!!!
-t

Works fine, thanks !


Decode morse

Project name: Decode Morse code
Project description:

In this project, a Morse code message will be decoded and the equivalent string would be represented. Built using Ruby.

Project requirements:

Create a method to decode a Morse code character, takes a string parameter, and return the corresponding character in uppercase (e.g. decode_char(".-") returns "A").
Create a method to decode an entire word in Morse code, takes a string parameter, and return the string representation. Every character in a word will be separated by a single space (e.g. decode_word("-- -.--") returns "MY").
Create a method to decode the entire message in Morse code, takes a string parameter, and return the string representation. Every word will be separated by 3 spaces (e.g.  decode("-- -.--   -. .- -- .")

returns "MY NAME").
Reuse methods whenever possible. Make sure that your code is DRY.
Once your code is ready use it to decode the message from the old bottle:

      .-   -... --- -..-   ..-. ..- .-.. .-..   --- ..-.   .-. ..- -... .. . ...

You can use IRB for that :)
Project requirements link:

https://github.com/microverseinc/curriculum-ruby/blob/main/simple-ruby/morse_code.md

Hi @rloterh ,
Thanks for your feedback. I have removed the Live Demo link according to your suggestion.
Best Regards,
K. Inoulen Shanongba


Show totalGrossPrice on MaterialCard

Definition of Done

[ ] The code does at least what is defined in the ticket and does not affect any other functionality
[ ] The PR has a meaningful title
[ ] There is a link to the issue-id
[ ] There are no 'hacks' or shortcuts refactoring is preferred
[ ] The happy case of the app was tested at least once manually

This PR is not needed anymore.


ApiGroovyCompiler throws NoClassDefFoundError due to incorrect user classpath

I was in the middle of extracting some code from core to a new module in gradle/gradle, and got the following exception during a Groovy compile:
Caused by: java.lang.NoClassDefFoundError: org.gradle.internal.buildoption.BooleanBuildOption
	at groovyjarjarasm.asm.signature.SignatureReader.parseType(SignatureReader.java:206)
	at groovyjarjarasm.asm.signature.SignatureReader.accept(SignatureReader.java:122)
	at org.gradle.api.internal.tasks.compile.ApiGroovyCompiler.execute(ApiGroovyCompiler.java:277)

As I'm moving modules around, there are classpath problems. I think the error is legitimate, but the failure mode is poor. There is nothing to tell me about the problematic use site. There are no other diagnostics from “the compiler”.
This can be reproduced with https://github.com/gradle/gradle/tree/ldaley/groovy-api-compiler-bug, by running :enterprise:compileIntegTestGroovy.
Expected Behavior
Error diagnostics providing some insight into where the problem is at least.
Current Behavior
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':enterprise:compileIntegTestGroovy'.
> org.gradle.internal.buildoption.BooleanBuildOption

This requires a change to the Groovy compiler, like identifying what it is trying to compile. There is nothing that can be done in Gradle alone for this.


Error subscribing theme: The current theme is not compatible with the selected master theme, the theme identifiers do not match.

I just installed APEX material theme using the empty app. Then I tried to restore theme subscription of an existing app which was using 42.Universal Theme and I got the error:
Error subscribing theme: The current theme is not compatible with the selected master theme, the theme identifiers do not match.
Is there something I can do... or should I create a new application based on the material design theme?

I found the solution... I exported the theme from the empty app... imported it into my app and then I changed the theme subscription...

Great :) Thanks for documenting your solution.


Disable topk optimisation on dispatch when content distribution is se…

…verly skewed.
When the skew is too large the assumption that docs are evenly and randomly distributed hold.
The impact and is larger on smaller systems. In large systems the where this optimisation is more important,
the probabilitity of large skew will be less.
@jonmv PR

Node.getActiveDocuments() only counts the primary copy which is the only one that is considered.

Node.getActiveDocuments() only counts the primary copy which is the only one that is considered.

Ouch, Boolean return value! You're right, that would be an NPE.
How do the primary copies change during redistribution from a retiring node?

Ouch, Boolean return value! You're right, that would be an NPE.
How do the primary copies change during redistribution from a retiring node?

Once a bucket has been copied to a new node, the copy which is now primary will be activated and the previous will be deactivated. Only active copies are searched. For a short period of time(milliseconds) there might be duplicates, but that is filtered higher up in the stack based on the 96bit GID.

Once a bucket has been copied to a new node, the copy which is now primary will be activated and the previous will be deactivated. Only active copies are searched. For a short period of time(milliseconds) there might be duplicates, but that is filtered higher up in the stack based on the 96bit GID.

So if all content nodes in a group are concurrently migrating from a set of old nodes to a set of new ones, and they're all halfway ±5%, then this will enable the optimisation anyway? Only in this scenario? If it is enabled in this scenario, we'll get wrong results, assuming the search considers not just the primaries on a node (which I believe it does?).

So if all content nodes in a group are concurrently migrating from a set of old nodes to a set of new ones, and they're all halfway ±5%, then this will enable the optimisation anyway? Only in this scenario? If it is enabled in this scenario, we'll get wrong results, assuming the search considers not just the primaries on a node (which I believe it does?).

Yes once they are halfway it will be enabled again, and results will be correct.
Only 1 copy of a bucket is searched, except for the small subset of buckets being activated/deactivated.

Yes once they are halfway it will be enabled again, and results will be correct.
Only 1 copy of a bucket is searched, except for the small subset of buckets being activated/deactivated.

Ok, perfect :)

Ok, perfect :)


Remove All Catena-X (NG) references across repositories

As part of the process of archiving the Catena-X (NG) organization, we need to remove all references to Catena-X (NG) in the code, configuration, and documentation of all repositories under the eclipse-tractusx organization.
Objective
The goal is to ensure that there are zero references to Catena-X (NG) across all repositories. This includes:

Code
Configuration files
Documentation
Comments

Each product team is responsible for searching, removing, and adapting any references found in their repositories.
Steps for Each Product Team

Search for references to Catena-X (NG):

Search your repository for all instances of the term catenax-ng.
You can use this search query as an example: GitHub Search Query. Maybe there are more words/ways to search for it

Fix the identified issues:

Update the code, configuration, or documentation to remove or adapt the references to Catena-X (NG).
Ensure that no references remain.

Track progress:

Use your own issue to track process, this issue is just the summery issue
Keep the issue updated as you make progress.
Close the issue once you confirm that all references have been removed and the repository is free of CatenaX-ng references.

Be aware: if something is already operational, please discuss it in the next office hour

DCM EG Issue to remove Catena-X (NG) Reference

Hi @stephanbcbauer I created this issue for the Committer Meeting https://github.com/orgs/eclipse-tractusx/projects/61/views/6?pane=issue&itemId=88268044


feat: Add getter for Azurite blob, queue and table endpoint

What does this PR do?
Exposes extra methods on azurite to get the individual endpoints for blob, queue & table.
This is useful if managed identity is used in the actual implementation.
Why is it important?
The storage connection string parser is internal in the azure sdk.
Related issues
n/a

What does this PR do?
[...] This is useful if managed identity is used in the actual implementation.

@pregress Curious to know how this helps with Managed Identity specifically.

What does this PR do?
[...] This is useful if managed identity is used in the actual implementation.

@pregress Curious to know how this helps with Managed Identity specifically.

If you use this ctor: https://learn.microsoft.com/en-us/dotnet/api/azure.data.tables.tableserviceclient.-ctor?view=azure-dotnet#azure-data-tables-tableserviceclient-ctor(system-uri-azure-core-tokencredential-azure-data-tables-tableclientoptions)
And your environment under test uses the URI in the environment variables. You need the URI and not the connection string.


[Bug] Cannot save cover on some sources

Device information

Tachiyomi version: r340
Android version: 8.1.0
Device: Samsung J7 Prime

Steps to reproduce

Select a manga on mangadex or mangapark (i know for sure it don't work on these sources maybe there is more).
Save the cover.

Expected behavior
The cover is saved and can be open.
Actual behavior
The cover is not saved and an error is given

Other details
It is not an extension bug as it work fine on J2K.
Not all sources are affected (i tested with mangahub, webtoon and mangasee)

Looks like it happens to manga that aren't in library or were just added to library. In the second case you can reopen source or open manga from library for it to work.

Yeah seem so. Hope it fix soon thx

Save/Share actions are now handled by main Tachi, if this still happens its a main Tachi issue


Add ward lookup (plus some extras)

Overview of changes
This was originally to add a specific lookup for Wards, though has spiralled into a few other improvements along the way.
Why are these changes being made?
Currently we were just treating wards as a lower level geography and did some basic tests like avoiding duplicate names. Now we have our first team publishing this level of data on EES we've realised this isn't really suitable and instead we can pull lookups from the Open Geography Portal to get an actual list to check against.
It's worth noting that many Wards do share names, so I've included the parent LADs in the lookup file as that is what distinguishes them (along with a unique code).
Detailed description of changes

Add a Ward lookup file

Generalise the existing update lookup file functions, and then use these for creating the Ward look up in a way we can keep updating it

Rebuild the pcon_la lookup

I've done this as when I tested the new functions on both the existing pcon and lsip lookups, the pcon_la lookup had changes (even though I was running it with the same file that it had been built from). Digging into it I think there's a number of LADs (e.g. Barnsley) that had snook into that as LAs (e.g. cont. should be South Yorkshire) in the past. So I've switched to the version the new functions create and rebuilt it with files going back as far as 2017.
It's worth double checking the logic where it splits off LADs from LAs, that was stolen from @rmbielby's code

Updated the passes_everything.csv test file

I did this as with Ward now having its own tests, we didn't have any 'lower level geographies' in the file so some tests were now being ignore, added in MAT data to compensate and make sure every test runs and passes against that file

Removed tidy_code() in favour of styler::style_code(), and updated README to match

Added a specific test for checking Ward combinations

Updated overcompleted cols test to include ward as a special case, and make the code more readable / clear on what it's doing (partly as the level of abstraction of it was tripping me up and I'm the one who originally wrote it!)

Changed to use the dfeR round_five_up function

EDIT:
9. Deleted tidyCode GHA, it was failing with unhelpful error messages so took the chance to move to a pre-commit hook instead as that feels like a cleaner approach anyway.

Also made some edits to the GitHub Actions scripts as I noticed they were using out of date actions, based this on examples from r-lib

 Add Ward level lookups to the data screener

Just noting here whilst I remember, I don't think we've got the new 2024 boundaries in yet. They're available here:
https://geoportal.statistics.gov.uk/datasets/12eff0dc5c4549a39003f73830ecbe49_0/explore
Could add as part of this PR or I'm happy to add them when I'm back after Easter.

Just noting here whilst I remember, I don't think we've got the new 2024 boundaries in yet. They're available here: https://geoportal.statistics.gov.uk/datasets/12eff0dc5c4549a39003f73830ecbe49_0/explore
Could add as part of this PR or I'm happy to add them when I'm back after Easter.

Added this to trello to pick up separately. I think it'll need to be a new lookup that's just for PCons as the 2025 boundaries don't have the LA lookup yet - https://trello.com/c/Fz9Hlyeu/1288-add-pcon-boundary-file-to-the-screener-specifically-with-2025-boundaries-in

Rebuild the pcon_la lookup

I've done this as when I tested the new functions on both the existing pcon and lsip lookups, the pcon_la lookup had changes (even though I was running it with the same file that it had been built from). Digging into it I think there's a number of LADs (e.g. Barnsley) that had snook into that as LAs (e.g. cont. should be South Yorkshire) in the past. So I've switched to the version the new functions create and rebuilt it with files going back as far as 2017.

Worked out what was happening here, was actually me not realising that Barnsley sits in a metropolitan county, so is both the LAD and LA, even when South Yorkshire is the UTLA. It's been an education!

@rmbielby - I have updated the tests to use shinytest2 now as we spoke about on this branch, done a few bits of tidy up around that too. Tagging @jen-machin for sight too as if we're happy with the changes here I can close #106.
The tests themselves should generally be much clearer / easier to see genuine failures now too as I've taken advantage of exporting values from the app in places where before we had masses of html output to sift through.
All of the changes are in a single commit so you can review the diff there. The only exception is some workflow faffing afterwards that ended up a bit trial and error eske. Happy to chat through if you have any questions!

@rmbielby - I have updated the tests to use shinytest2 now as we spoke about on this branch, done a few bits of tidy up around that too. Tagging @jen-machin for sight too as if we're happy with the changes here I can close #106.
The tests themselves should generally be much clearer / easier to see genuine failures now too as I've taken advantage of exporting values from the app in places where before we had masses of html output to sift through.
All of the changes are in a single commit so you can review the diff there. The only exception is some workflow faffing afterwards that ended up a bit trial and error eske. Happy to chat through if you have any questions!


No_tax_id NHS admin portal changes

NHS admin must be able to view employees which registered without tax_id.

[ ] According to this Employee Request list must be changed and additional filters should be added apiary
by edrpou
by legal entity name
by no_tax_id flag
by id
[ ] to Employee Request by id add field 'no_tax_id'
[ ] to Employee List add filters apiary:
no_tax_id
[ ] to Employee details by id add field 'no_tax_id'

[ ] reset page number while searching by parameters.
[ ] add filters to  Employee List

- party_id
- edrpou
- legal_enyity_id