Set up S3 replication to another account

Summary
Per spike #53, create a system for backing up the S3 data bucket(s) to another account using S3 replication.
Requirements
The more specific the better.

[ ] Set up an S3 bucket in another account (using CDK)
[ ] Update CDK in both the source bucket and destination bucket to automatically replicate
[ ] Determine if we would like to move or remove the data after a certain time period

*Related tickets

#53

I believe this is finished, right? Feel free to close if it is @maxinelasp

Still needs some tests - I was having some trouble and was hoping to pair program it. I basically have the tests written already, just getting some odd cloudformation errors


device address format validation error

The set device address validation only accepts a hostname:port, and rejects a TCP URL.
Syncthing expects a url style,  tcp://hostname:port.
Adding new device from the cli that needs an address as below results in a device in syncthing that does not work, until address is manually corrected in syncthing GUI.
syncthing-cli devices add DEVICE
syncthing-cli devices set DEVICE name whatever
syncthing-cli devices set DEVICE address HOST:PORT

Yeah, this is know, as the cli works against a very old API.
It hasn't had much traction, hence any external effort to help fix these is welcome.

Tralalala closing as part of general closing spree, now lives in main repo, will get fixed at some point I promise(ish)


Restrict phone length

After selecting or typing the country code, it is possible to input an arbitrary number of symbols. It would be great if the component had a prop to restrict the number of symbols in the telephone number according to the selected country's standards.

Duplicated of https://github.com/viclafouch/mui-tel-input/issues/56


decouple TLF metadata out of PathItems 4: component changes - other

Merged from https://github.com/keybase/client/pull/13144


Order clause causes Cypher syntax error when used with with_associations

As discussed in Slack, neo4j 6.0.1 introduced a bug in which order clauses are output in the wrong place in the cypher query, resulting in a syntax error. Here is a code sample:
class Conversation
  # ...
  has_many :out, :comments, type: :comment

  def all_comments
    comments.with_associations(:author).order(created_at: :asc)
  end
end

# usage:
conversation.all_comments.to_a

Error:
     Neo4j::Session::CypherError:
       Invalid input 'D' (line 1, column 228 (offset: 227))
       "MATCH conversation36010 WHERE (ID(conversation36010) = {ID_conversation36010}) MATCH conversation36010-[rel1:`CONV_COMMENT`]->(result_comments:`Comment`) OPTIONAL MATCH result_comments<-[:`author`]-author WHERE (author:User) ORDER BY result_comments.created_at ASC RETURN result_comments, [collect(author)]"
                                                                                ^

Could you check to see if 7.0.0 fixes this?

Stumbled across this while looking for a with_associations issue. This has probably been fixed given the lack of activity.

Closing for now.  Comments / re-opening welcome


Ie8 support

I needed ie8 support and added a png version of every flag to the mix.
This modification in the current form will only work if the class "no-svg" is present which is automatically set if modernizr is used.

#33

Yes.. guys sorry.. it's already hard to maintain all those flags in two different formats and adding support for something that was last updated 3 years ago is really not worth it :)

If someone is force to supported IE8 then they will have to do it on their own..

@lipis: question: could this "png" images situation perhaps be handled with a script that produces the png:s from the source svg:s using a cli tool? then this repo could distribute the script, and users in need could run it to produce the png:s, and we dont need to have the generated images in the repository. Just an idea

The PNGs used to be generated in the original project, that unfortunately is no longer exist.. and from what I remember it required bunch of planets to be aligned in order for it to work properly.. plus bunch of options for sizes and ratio... so to tell you the truth I'm not willing to maintain that thingy :) having said that if there is a simple script I could possibly include it to generate stuff.. or at least point to where you can find a clone.. for now I will close this issue and if anyone wants to maintain a IE8 fork.. please do so and let me know so I can link to it in the readme..

@lipis ah, well was just my 2 cents. i have no personal need for this, so won't put any effort into it

Me neither... thanks @martinlindhe


chore: bump rock version

Description
Use the latest version of upstream project.
Checklist

[ ] I have performed a self-review of my own code.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that validate the behaviour of the software.
[ ] I validated that new and existing tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

AMF and SMF do not need version update since NRF response caching was already implemented

AMF and SMF do not need version update since NRF response caching was already implemented

We started to use nrfcache library from openapi instead of nrf module. Hence, bumping version is also required for AMF and SMF. This will not affect NRF response caching directly however, dependencies are reduced.


devdocs: readme: fix grammar

Summary of the Pull Request
Fix a grammatical error in readme.md under doc/devdocs.
PR Checklist

[ ] Applies to #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
Grammatical error fix to readme.md under doc/devdocs.

@CaelestisZ
thanks.


Tag a v1.1.0 release

Hi @jlewi, we haven't been tagging a v1.1.0 release tag and instructions on https://www.kubeflow.org/docs/gke/deploy/management-setup/ are still pointing to master.

Thanks @Bobgy I sent you #122 with some mixing fixes to the blueprint cluster that was preventing us from switching to the management cluster to the 1.1 branch.
I'm on the fence about whether our docs should refer to a branch or a specific/commit tag. WDYT?

I think we should always be referring to a commit tag, that's the only way to make user errors reproducible.
What concerns do you have? Do you worry management cluster might not change much, but we may need to update its version in doc each time?

Comparing to tag vs branch, I was actually more confused by release branch process in gcp blueprints repo.
I don't think we are going to work on any PRs not meant to be fixing issues for KF 1.1 right now, but we have to cherry pick every PR (a lot of manual effort).
I'd want to propose a lazy procedure for cutting release branch:
we cut a release branch when we find the first merged PR we don't want to release.

Was the reason for 1.1 branch as a way to release? e.g. we document to let users to use the branch.
Then if we switch to tag for release, we can save the cherrypick process of maintaining two branches for now.

@Bobgy The lazy approach you mention sounds like a great idea. I only cut a branch because I was replicating what we already do for the kubeflow repos.

Great, I'll see how we can switch the process, so far it looks like master and v1.1 is almost the same, with really minor diffs.
I guess one thing we need to handle is, we need to change autodeploy process to use the same branch, but parameterize where manifests should be get, one should use master manifests and one should use v1.1.0 branch.
Another question is: how do we tag a release for a repo? Sounds like I need more permission to the repo.
Would https://github.com/kubeflow/internal-acls/pull/311 address this?

kubeflow/internal-acls#311 should address this; although you could also create a specific team with permissions on the repo.

I guess one thing we need to handle is, we need to change autodeploy process to use the same branch, but parameterize where manifests should be get, one should use master manifests and one should use v1.1.0 branch.

Yeah I didn't think of that. Would it make sense to still cut branches for the v1.X.Y releases so that we can change the manifests on that release branch to point to the correct manifests release?
We could just cut a new release branch off the latest master for each v1.X.Y release. So we don't need to cherry-picks.
Alternatively, you could just merge master into the v1.X-branch before cutting adding the tag v1.X.Y.
I guess its just a matter of what you prefer

Merging in all changes on master
Cutting from master and updating the manifests version

We could just cut a new release branch off the latest master for each v1.X.Y release. So we don't need to cherry-picks.
Alternatively, you could just merge master into the v1.X-branch before cutting adding the tag v1.X.Y.

Great idea, sounds like both works, let me try merging master first because it's less intrusive (no need to delete the existing v1.1 branch).

I'll cut a 1.1.0 release tag just to archive current status.

Tagged https://github.com/kubeflow/gcp-blueprints/releases/tag/v1.1.0

@jlewi I just realized management cluster in gcp-blueprints is pointing to manifests master. When do you think we can pin a release tag on manifest repo for v1.1.1 or v1.1.1-rc.1? How do we usually coordinate work across kubeflow?
I'll pin management cluster to v1.1-branch in manifest first.

@Bobgy SGTM thanks

Pinned 1.1.0 and updated documentation


[GEN-1478] Update config.yaml

Updating the MELANOMA seq_max date to Apr-2022 and site's target case counts per this file https://aacr.ent.box.com/s/h4wkij15geg7ekls583b7wdydrbu7z2w

Member


get "msg: Unable to send msg: HTTP Error 401: Unauthorized" when running role in first time

Hi,
I'm running the role as a park of a playbook on gce instance.

{ role: stackdriver-ansible-role, stackdriver_api_key: '{{api_key}}', stackdriver_mongodb_enabled: yes }

I have mongo already installed and running.
When I run the role in first time, it throws me this exception:
NOTIFIED: [stackdriver | restart collectd] ************************************
changed: [yg-test] => {"changed": true, "name": "stackdriver-agent", "state": "started"}
NOTIFIED: [stackdriver | restart extractor] ***********************************
changed: [yg-test] => {"changed": true, "name": "stackdriver-extractor", "state": "started"}
NOTIFIED: [stackdriver | stackdriver updated] *********************************
failed: [yg-test -> <IP_ADDRESS>] => {"changed": false, "failed": true}
msg: Unable to send msg: HTTP Error 401: Unauthorized
FATAL: all hosts have already failed -- aborting
In second run, I don't see any errors probably there is no update to run.
Thanks.
Keep up the good work!

The error is about a failure to contact the Stackdriver gateway, which will also prevent your agent from sending data. This could, for instance, be caused by an invalid API key. You can test directly by SSHing into the host and executing sudo /opt/stackdriver/stack-config info. The first few lines of output should look like this:
Stackdriver Host Info Dump
Resource Id: [...]
API Key: [...]
Customer ID: [...]

I get :
Error talking to Stackdriver gateway No JSON object could be decoded
But within the gce project, in the monitor section I can view the stackdriver info and looks ok.

You may see the instance and its CPU usage, which the GCE infrastructure reports, but you won't see memory usage if data is not being received from the agent.

I can see all processes details includes mongo memory usage,connections an operations.
BTW, in the handlers/main.yml I changed the "stackdriver update" to:

name: stackdriver updated
service: name=stackdriver-agent state=restarted

Thank you for reporting this. It's exactly as you said. I failed to reproduce it at first because the issue only affects Google Cloud Monitoring, and not generic Stackdriver. I will be following up.

This feature is now controlled by the value of stackdriver_events_enabled, which defaults to no.


Close button does not minimize to tray

Greetings! So when the minimize to tray option is enabled clicking the close button closes the program outright. This is counter-intuitive as the need for this program is high and there is no purpose to accidentally closing the program outright. If one desired to actually close it (while it's set to minimize to tray) then a couple extra clicks would be fully justified.

This is counter-intuitive
True.

I'll have a look at it this weekend.

The problem should be fixed. If it is, please close this issue. Download the latest release here.

ROCK! Done! Closed! Thank you!


Use github releases instead of bintray

Recently I found that it's pretty easy to upload the artifacts to tagged github release as part of the maven release plugin.  Should do that for sstable-tools to automate the process of making a release.

Done


Update docs for feat: impl admin command

A document change request is generated from https://github.com/GreptimeTeam/greptimedb/pull/4600

Not completed.


Update api::v3::users [PEOPLE-65]

JIRA: https://netguru.atlassian.net/browse/PEOPLE-65
Adds:

membership serializer for Api::V3
user_with_memberships serializer for Api::V3

:+1:


Initial upload of -TeamsAppInstallation cmdlets

@kenwith

Docs Build status updates of commit 05584ce:
:clock10: Full build: the average full build time is 12 min(s) 53 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 05584ce:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Add-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Update-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit f5cd0ad:
:clock10: Full build: the average full build time is 12 min(s) 53 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit f5cd0ad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Add-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Update-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 31b8797:
:clock10: Full build: the average full build time is 12 min(s) 58 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 31b8797:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Add-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Update-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 92fa9ac:
:clock10: Full build: the average full build time is 12 min(s) 59 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 92fa9ac:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Add-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Update-TeamsAppInstallation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Layout type: "sfdp" not recognized

When I try to use the sfdp layout in with viz.js (@see http://www.graphviz.org/Documentation.php) I get an error message
Error: Layout type: "sfdp" not recognized. Use one of: circo dot fdp neato nop nop1 nop2 osage patchwork twopi
I did not compile the viz.js I just used the version http://mdaines.github.io/viz.js/viz.js and adapted the form.html to support sfdp.
You may easily reproduce it if you change the graph in http://mdaines.github.io/viz.js/form.html and add the layout attribute with layout="sfdp". It will display the error in the console.
digraph G {
layout="sfdp";
	subgraph cluster_0 {
		style=filled;
		color=lightgrey;
		node [style=filled,color=white];
		a0 -> a1 -> a2 -> a3;
		label = "process #1";
	}

	subgraph cluster_1 {
		node [style=filled];
		b0 -> b1 -> b2 -> b3;
		label = "process #2";
		color=blue
	}
	start -> a0;
	start -> b0;
	a1 -> b3;
	b2 -> a3;
	a3 -> a0;
	a3 -> end;
	b3 -> end;

	start [shape=Mdiamond];
	end [shape=Msquare];
}

This would require building and packaging GTS. I may add this option in the future.


Support connection pool configurations in database

Hi,
Can you provide more context on this issue?
Thanks

Right now setting such as number of idle connections, max connection and other db connections are not configurable we want to support that but not from configs.

I see many approaches you can take to implement this, such as Providing an API endpoint that allows administrators to update the connection pool settings at runtime, employing a database table to store these settings, which the application reads and applies at startup or during runtime, etc.
How do you suggest to approach this issue? (please do reply if you had something specific in mind)

Done in #739


OS chef upgrades over omnibus chef, breaking things

When I install omnibus chef onto debian/ubuntu, I end up with a opscode distributed packaged called 'chef'. In this case, version 11.8.0-1.ubuntu.13.04 This runs into an interesting bug.
The OS itself, distributes a chef package. Sometimes the OS's chef version is newer. When that happens, the OS upgrades the chef package. (For example, right now ubuntu distributes 11.8.2-2). When this happens, things start to break, one of the immediate ones, is the missing ruby-shadow.
At least for ubuntu/debian, I think there are a several of ways to fix this.
On the tack of preventing the OS from upgrading it. You could name the package something else; or set an epoch in the version. You could also install an apt source for the omnibus chef, and ensure you're always newer.
I think I probably favor the epoch route.

FWIW, I ended up with chef 11.8.0-1.ubuntu.13.04 by way of knife bootstrap, on a  https://manage.chef.io  server

@directionless  you can tell dpkg to not install a specific package. using apt-mark [1]
[1]http://askubuntu.com/questions/18654/how-to-prevent-updating-of-a-specific-package

Yes, I can pin the package any number of ways. But I think this is a bug in the underlying chef omnibus packaging. Why make everyone work around it

We support apt repos via packagecloud.io now and if you install and apt_repository pointing to that then that repo will always be more up-to-date than the distro repo.


Implement a new ListBuckets command and its consumer API.

Summary

Feature: The serde_types now store chrono::DateTime<chrono::Utc> instead of String, wherever there was any datetime-like data to be stored.
Feature: Two new APIs to list existing buckets (i.e. Bucket::list_buckets), and to check if a given bucket instance exists (i.e. Bucket::exists).
Bug fix: CI was failing because of an unused tls_connector variable, so its been renamed to _tls_connector. Not too sure if this is the ideal long-term fix but its a band-aid for now.

Bucket::list_buckets(region, credentials)
Given a region, and credentials, list all the buckets that are visible.
Usage
use s3::{Bucket, BucketConfiguration};
use s3::creds::Credentials;
use s3::region::Region;
use anyhow::Result;

#[tokio::main]
async fn main() -> Result<()> {
    let region = Region::Custom {
      region: "eu-central-1".to_owned(),
      endpoint: "http://localhost:9000".to_owned()
    };
    let credentials = Credentials::default()?;

    // Async variant with `tokio` or `async-std` features
    let response = Bucket::list_buckets(region, credentials).await?;

    // `sync` feature will produce an identical method
    #[cfg(feature = "sync")]
    let response = Bucket::list_buckets(region, credentials)?;

    // Blocking variant, generated with `blocking` feature in combination
    // with `tokio` or `async-std` features.
    #[cfg(feature = "blocking")]
    let response = Bucket::list_buckets_blocking(region, credentials)?;

    let found_buckets = response.bucket_names().collect::<Vec<String>>();
    println!("found buckets: {:#?}", found_buckets);
    Ok(())
}

Bucket::exists
Determine whether the instantitated bucket exists.
Usage
use s3::{Bucket, BucketConfiguration};
use s3::creds::Credentials;
use s3::region::Region;
use anyhow::Result;

#[tokio::main]
async fn main() -> Result<()> {
    let bucket_name = "some-bucket-that-is-known-to-exist";
    let region = Region::Custom {
      region: "eu-central-1".to_owned(),
      endpoint: "http://localhost:9000".to_owned()
    };
    let credentials = Credentials::default()?;

    let bucket = Bucket::new(bucket_name, region, credentials)?;

    // Async variant with `tokio` or `async-std` features
    let exists = bucket.exists().await?;

    // `sync` feature will produce an identical method
    #[cfg(feature = "sync")]
    let exists = bucket.exists()?;

    // Blocking variant, generated with `blocking` feature in combination
    // with `tokio` or `async-std` features.
    #[cfg(feature = "blocking")]
    let exists = bucket.exists_blocking()?;

    assert_eq!(exists, true);
    Ok(())
}

haha. I didn't know when I issued this PR but this also closes #251.

@aalekhpatel07 Nice work, thank you!

Thank you for merging!


forms.MultipleChoiceField do no materializing

forms.MultipleChoiceField has no materialized implementation. It's standard widget.
Django 1.9.2
django-material 0.5.1

That's not true

It's good with widget=forms.CheckboxSelectMultiple, but no changes without widget specification.

Have you missed to include jquery before includes/material_js.html?

No.
I use Python 3

I tried your example code and got the same.
My code render result of {% form %}{% ednform %}:

        Язык <select class="material-ignore" name="languages" id="id_languages" multiple="">
            <option value="**">Не зависит </option>

        <option value="EN">Английский </option>

        <option value="RU">Русский </option>

        <option value="DE">Немецкий </option>
        </select>

        Язык <select class="material-ignore" name="languages" id="id_languages" multiple="">
            <option value="**">Не зависит </option>

        <option value="EN">Английский </option>

        <option value="RU">Русский </option>

        <option value="DE">Немецкий </option>
        </select>

You should check javascript errors in the browser console

** class="material-ignore"** lol, are you about admin part? This is not MultipleChoiceField

No. It's my FormView with your example form or my form. Result is the same.
Default multiple select widget if field = forms.MultipleChoiceField(..., widget=forms.SelectMultiple)
2016-02-19 13:54 GMT+03:00 Mikhail Podgurskiy<EMAIL_ADDRESS>
** class="material-ignore"** lol, are you about admin part? This is not
MultipleChoiceField
—
Reply to this email directly or view it on GitHub
https://github.com/viewflow/django-material/issues/80#issuecomment-186170196
.

I have no idea, what's wrong with you code.
You copy-pasted the code that could be rendered only from django_filteredselectmultiple.html That's the admin part.

I don't include material.admin to INSTALLED_APPS even.
templates/material/fields/django_selectmultiple.html content:
{% load material_form material_form_internal %}

{% render bound_field template='fields/django_select.html' %}
    {% part field add_control_attr %}multiple{% endpart %}
    {% part field add_group_class %}multiselect{% endpart %}
    {% part field add_control_class %}material-ignore{% endpart %}
    {% part field options %}
        {% for value, choice in field.widget.choices %}
        <option {% if value in bound_field.value %}selected="selected"{% endif %}
                {% if value == None or value == '' %}style="display:none"{% endif %}
                value="{{ value|default_if_none:"" }}" >{% if value == None or value == '' %}{{ form_select_empty_label|default:choice }}{% else %}{{ choice }}{% endif %} </option>
        {% endfor %}
    {% endpart %}
{% endrender %}

That's not true - https://github.com/viewflow/django-material/blob/master/material/templates/material/fields/django_selectmultiple.html

I use last release 0.5.1
2016-02-19 15:02 GMT+03:00 Mikhail Podgurskiy<EMAIL_ADDRESS>
That's not true -
https://github.com/viewflow/django-material/blob/master/material/templates/material/fields/django_selectmultiple.html
—
Reply to this email directly or view it on GitHub
https://github.com/viewflow/django-material/issues/80#issuecomment-186189087
.

You should use git, before posting issue requests

I tried. Master is broken.
Invalid filter: 'select_options'
In template /home/dima/.virtualenvs/mathtrade/lib/python3.4/site-packages/material/templates/material/fields/django_select.html, error at line 18

So, why you spend my time on a fake error, instead of report the real one?

And how did you get the error? Did you grab the whole code or just html template?

It's DEBUG=True output.
I installed master version.
2016-02-19 15:17 GMT+03:00 Mikhail Podgurskiy<EMAIL_ADDRESS>
And how did you get the error? Did you grab the whole code or just html
template?
—
Reply to this email directly or view it on GitHub
https://github.com/viewflow/django-material/issues/80#issuecomment-186194079
.

I have no such error.

It's OK now
2016-02-19 15:24 GMT+03:00 Mikhail Podgurskiy<EMAIL_ADDRESS>
I have no such error.
—
Reply to this email directly or view it on GitHub
https://github.com/viewflow/django-material/issues/80#issuecomment-186196731
.


Dependabot not closing "old" PRs after proposing updates for newer versions of the same dependency

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
pip and others
Package manager version
N/A
Language version
N/A
Manifest location and content before the Dependabot update
dependencies/python/checkov.txt
dependabot.yml content
https://github.com/super-linter/super-linter/blob/main/.github/dependabot.yml
Updated dependency
Happens with several dependencies.
Example: checkov
From:
What you expected to see, versus what you actually saw
Dependabot should close PRs that propose old versions of the same dependencies.
The example is about pip and python, but happens with other ecosystems as well.
This looks like a regression.
Thanks for your support!
Native package manager behavior
Not needed because the dependency updates are proposed as expected. The issue is that Dependabot doesn't close old PRs about the same dependency.
Images of the diff or a link to the PR, issue, or logs
https://github.com/super-linter/super-linter/pull/5848
https://github.com/super-linter/super-linter/pull/5831
https://github.com/super-linter/super-linter/pull/5803
https://github.com/super-linter/super-linter/pull/5782
Smallest manifest that reproduces the issue
version: 2
updates:
  - package-ecosystem: "pip"
    commit-message:
      prefix: "deps(python)"
    directory: "/dependencies/python/"
    schedule:
      interval: "weekly"
    open-pull-requests-limit: 100

/cc @zkoppert
/cc @Hanse00

It doesn't appear in my case with maven. Maybe it's only related to python or with specific options
My config:
version: 2
updates:
  - package-ecosystem: "maven"
    directory: "/"
    schedule:
      interval: "weekly"

In our case, it also happens for npm, for example:
https://github.com/super-linter/super-linter/pull/5854
https://github.com/super-linter/super-linter/pull/5833
https://github.com/super-linter/super-linter/pull/5795
And for ruby:
https://github.com/super-linter/super-linter/pull/5827
https://github.com/super-linter/super-linter/pull/5778

Happening for me as well with pip and terraform

I think this issue is duplicate of #10103

I think this issue is duplicate of #10103

It's not. Here, the case is about not closing old PR, but well creating new one. The issue you mentionned is only about not creating new PR

I'm seeing the same with docker.

Facing the same issue on dependabot. Old PR's are not getting closed after proposing updates for newer versions of the same dependency.

Seeing the same issue for gradle, but only for one of two repositories:

Repo1 has build.gradle in the repository root. This repo is not affected by this bug.
Repo2 has build.gradle in a subdirectory. This repo is affected by this bug.

Maybe there's a connection?

This is fixed, but closing as a duplicate of https://github.com/dependabot/dependabot-core/issues/10103

This should be fixed now, I was able to reproduce the problem and verify the fix.
The Dependabot service had a bug preventing superseding but it only affected non-root directories when wildcards weren't in use (e.g. **/*).
Going forward, PRs should be superseded correctly. You're welcome to leave them open to verify yourself that it's solved.
Thanks for you patience!

@soeren-glimm-insurfox yep, you were right!


[CI]Pin shell to use as bash

Minor change:As the shell bash is currently supported in GitHub CI windows platform, we can now to not write spaecial code for Windows.
Test for this PR:
See the build workflow run in my repo

Thank you for noticing this, but the purpose of test is to cover as many scenarios as possible, especially use cases in reality. On Windows, very few people are actually using bash, so we should not move to bash for testing just because it saves us a couple of lines of code in CI file. Especially considering that objprint is a library that "prints" stuff which might be affected by the shell itself. I remember some powershell/CMD specific issues, which would not be caught if bash was used to test.

But a color issue will only find out by user even without this change, color is disabled in the tests
Is there any other hidden issue?

Color is disabled by default in the tests, but some test will enable it explicitly.
Color is also just an example to explain why we should not use bash on Windows. The problem of hidden issues is that they are hidden - we don't know them. The principle of testing, is to cover as many use cases as possible, and to mimic the environment the user is using as much as possible.
The majority of users on Windows are using Powershell and it should be covered by the tests.
The benefit of testing with bash on Windows is minimal - we made the yml file slightly cleaner - that's it. Okay maybe we also get a better coverage for bash on Windows - which only represents a very small group of users. The cost is significantly larger than the benefit - we lost coverage on the most popular shell on Windows.


🐞 Bottom Drawer Defects

@StefKors : Some problems.
https://user-images.githubusercontent.com/20476002/164035159-43290e1e-2bc7-4173-8aca-89f136f89c14.mov
https://user-images.githubusercontent.com/20476002/164035034-c40effba-7212-454c-9a12-b9b945fec490.mov
https://user-images.githubusercontent.com/20476002/164035048-10568f91-7b3f-473f-8aa1-d29d4686abd1.mov
Originally posted by @Angelk90 in https://github.com/CodeEditApp/CodeEdit/issues/423#issuecomment-1102767213

Thanks for creating the issue @austincondiff 👍

Note that the bottom bar (where the filter textfield is) has an opaque background in Xcode.

@StefKors : The behavior of codeEdit:

The behavior of xcode:
https://user-images.githubusercontent.com/20476002/164064491-860b0b0c-2173-459c-b120-1698eb731a2a.mov
Open tabs and everything else also disappear.

Unrelated issue but I've noticed that the terminal text goes right up to the edge. Do we want to add a little padding here - 5 pixels maybe?

@StefKors when you address these issues, would you mind fading out the tabs when closing the drawer and fading them in when opening?

When the program is not active it should be faded like xcode does.

@lukepistrol :  When the terminal is open to the maximum height, the sidebar and central part must be at the same height, as can be seen from the images.

Closing as most of these things mentioned are resolved except that last one mentioned by @Angelk90. Closing this issue and creating a new one based on that comment.


The library seems not to work, fails to function

Even after cloning on android studio and compiling it still doesn't work, I created a new project with the dependencies needed but still no avail. Please what's wrong

Change to
implementation 'com.github.lincollincol:AudioTool:1.2.1' in settings.gradle file.


How to get learning rate during training ?

I'm new using keras, I want to get the learning rate during training LSTM with sgd optimizer, I have set the decay parameter, it seems it works, but when I use model.optimizer.lr.get_value() to read the learning rate, it didn't change at all, my setting is as follows:
lr_init=0.12; decay_init=1e-2; batch_size_init=30
momentum_init=0.9; np_epoch_init=1
sgd = SGD(lr=lr_init, decay=decay_init, momentum=momentum_init, nesterov=True)
model.compile(loss='categorical_crossentropy', optimizer=sgd)
early_stopping = EarlyStopping(patience=2,verbose=1)
lr_record=[]
for iteration in range(1, 60):
    print('Iteration', iteration)
    hist=model.fit(X, y, callbacks=[early_stopping],validation_data=(X_val, y_val),batch_size=batch_size_init, nb_epoch=np_epoch_init)
    lr_temp=model.optimizer.lr.get_value()
    lr_record.append(lr_temp)

after ran it, the learning rate recorded in lr_recorddid not change a bit at all, I'm wondering if someone could help me, thanks in advance!

Base learning rate (lr) is unchanged in SGD and the learning rate is calculated according to iteration. Ref: code.

Thank you, but with decay parameter(I think keras use the 1/t decay), how could I monitor the learning rate by each epoach?

Use callback to access model.optimizer.
class SGDLearningRateTracker(Callback):
    def on_epoch_end(self, epoch, logs={}):
        optimizer = self.model.optimizer
        lr = K.eval(optimizer.lr * (1. / (1. + optimizer.decay * optimizer.iterations)))
        print('\nLR: {:.6f}\n'.format(lr))

# define your model

sgd = SGD(lr=0.01, decay=0.9)
model.compile(loss='mse', optimizer=sgd)
model.fit(x, y, callbacks=[SGDLearningRateTracker()])

Thanks very much!

@joelthchao  I tried your Callback class and it runs for SGD. However, when i set an RMSprop optimizer, i get the error:
AttributeError: 'RMSprop' object has no attribute 'iterations'
keras.__version__
'0.3.2'

I can see on https://github.com/fchollet/keras/blob/master/keras/optimizers.py that the RMSprop class does have an iterations attribute. Is my version too old?

@joelthchao, thank you for you suggested solution. However, you didn't include your imports. What is the "K" from "K.eval"? Putting in keras does not work.

import keras.backend as K

And how to deal with less trivial schedulers such as ReduceLROnPlateau? Is there a way to access the actual learning rate instead of recalculating it?

Judging from the code of ReduceLROnPlateau the lr = float(K.get_value(self.model.optimizer.lr)) should return it, so no arithmetics with optimizer.decay * optimizer.iterations etc are needed. What is the rationale behind using those, @joelthchao?

@DSLituiev
To clarify, there are two different kinds of "lr":

base_lr: optimizer.lr.
real lr that apply on gradient: optimizer use its algorithm and base_lr to calculate it. (e.g. SGD: optimizer.lr * (1. / (1. + optimizer.decay * optimizer.iterations))

In your case, ReduceLROnPlateaudirectly changes base_lr, therefore, you still need to calculate real lr depends on which optimizer do you use.

@joelthchao When I use this code, I get an error: Incompatible type conversion requested to type 'float32' for variable of type 'int64_ref'. I am using Keras with Tensorflow backend. Then I converted values from optimizer using tensorflow.to_float(x ,name='ToFloat'), it seems to work.
_lr = tf.to_float(optimizer.lr, name='ToFloat')
_decay = tf.to_float(optimizer.decay, name='ToFloat')
_iter = tf.to_float(optimizer.iterations, name='ToFloat')

lr = K.eval(_lr * (1. / (1. + _decay * _iter)))
print(' - LR: {:.6f}\n'.format(lr))

Could we have official support to display the learning rate rather than some hack?

How does one feed current lr into the tensorboard? Custom Callback?

Closing this issue since its resolved. Thanks!

LambdaCallback and on_epoch_begin
train_lr_callback = LambdaCallback( on_epoch_begin= lambda epoch,logs: print("LearningRate of %e" % (K.eval(model.optimizer.lr)) )
model.fit_generator(train_loader,
steps_per_epoch = totalTrainImgsNum // batch_size, #2019年2月12日 前值 400万 #2019年2月9日 前值 5000000
epochs = total_epoches,
initial_epoch = 22,
validation_data = test_loader,
validation_steps = 35080 // batch_size,
callbacks = [checkpoint, changelr, tensorboard,train_lr_callback])


Updated Remote Desktop Gateway Solution

This PR updates and adds to the Remote Desktop Gateway sample.
Based on learnings working with customers implementing this solution (or a similar), this PR aims to:

Add security
Simplify deployment
Update or use recent Azure offerings

Updates include (not exhaustive):

Function app

Code cleanup and simplify
net461 -> .netcoreapp3.1
Update function app runtime to v3
Use KeyVault references

Gateway

New deploy script
Simplified the certificate stuff (no need to base64 encode and figure out thumbprint, just pass in a file path)
One arm template (+ script) handles full deployment
Function app now exposed as a private endpoint (now not on public internet)
Now uses VM scale sets
Added bastion host to administer gateway VMs (now not on public internet)
Deployment script uses KeyVault to generate signing cert

Note: The how-to guide on Microsoft docs associated with this sample will need to be updated.  That said, there are broken links in that doc and broken download links in the current version of the sample that prevent it from deploying and successfully.

@markusheiliger

Closing.  Will reopen after discussing with DTL engineering team.


Possible problem with name

Hi Billy, I'm Brian. I'm currently working on touching up your README and I'll shoot you a PR as soon as I'm done. In the mean time, I wanted to get your input about some things.
First, how attached are you to the name: pryx?
I know that you're not a native English speaker, so you might not be familiar with our slang.  So your gem kinda sounds like the plural of that. Namely, pricks.
You can see how, even if the correct pronunciation of pryx is supposed to be like pry and then the 'x' sound, I think most everybody is going to read it as the other way, like the slang term above.
So, how do you feel about adding a dash? Like, pry-x? Then it seems like it's pronounced pry and then the 'x' sound. I'm guessing that that's how you meant for it to be pronounced.
But, if you don't care, then that's cool too :)
I might have some questions for you about the README, here in a minute.

That syntax:
RUBYOPT+=' -rpryx'

is that for zsh?
So, would it be different for bash? Like:
RUBYOPT=' -rpryx'

Let me know (if they are both valid) if I should add both examples

pryx sounds like pricks.

Wow, this is a bit bad.

RUBYOPT+=' -rpryx'

This is a syntax for BASH, probably need a more recently version, BASH 4? check https://www.baeldung.com/linux/bash-append-variable-to-itself#the--operator-in-bash

Wired, I don't know why, but someone create an issue for my project,   I didn't receive any github notification or email.
Anyway, i create a new commit 25ea435 for explain how to set RUBYOPT in others shell.


Allow chaging displayed direction of an Edge

To change the arrow markers on an Edge
see https://github.com/txlabs/vnodes/pull/14

#14 cancelled


Is this repository still being updated and maintained?

Due to an internal issue, I lost access to the repository but it's being worked on right now. Once resolved I should be able to update the repository again.

Very much looking forward to it

any update?

I should have access now. Only missing time to push latest updates since we haven't fully decided yet the actuel process of doing so.

We are eagerly awaiting the update of your library)

any update?

I just synchronized this repository with our internal version.


Fix ESLint Errors in tests/perf-test/ReportActionsList.perf-test.tsx

Encountered ESLint errors in this PR but couldn't fix them there since it was a straight revert. Creating this issue to track the separate fix.
It's 3x replacing measurePerformance with measureRenders in https://github.com/c3024/App/blob/4a6d6d2bc3050157b6e295ff48478fc985dc0604/tests/perf-test/ReportActionsList.perf-test.tsx

@bfitzexpensify this is followup from https://github.com/Expensify/App/pull/49770 - I don't think we need to handle payment etc. on here

Deployed to prod - @dangrous, any further action needed here?

nope I think we can close - nothing actually needed here now.

oh wait I lied, we might need to keep this open for completion, but we won't need payment either way. Checking in Slack.

@dangrous is there a PR linked to this I can keep an eye on? Is it just https://github.com/Expensify/App/pull/49770/files?

Yeah I was originally thinking we needed to handle this as a follow up, but since it was just a revert and not original code, I think it's okay to close this out until it's fixed by the next update to that file. Thanks all!


local server not reponding

Hi,
I'm not able to view the sample application after running "gulp run".  When loading localhost:9000, I see the status as "pending".  Any idea why the server is not serving content from "dist" folder?

Ugh. I wasn't getting notifications about new issues. Sorry!
Are you still having problems? Can you tell me anything more about your platform?

closing inactive.


[Feature Request] Early Hints 103

It seems Golang released a new version which now supports Early Hints via the 103 Status code.
https://twitter.com/dunglas/status/1526635022148685825 / https://github.com/golang/go/pull/42597
I think it would be a great feature for roadrunner bringing support for this Early Hints to PHP. Maybe a special method could be provided which make it possible to send early hints headers via 103.

Nice think we can close this one :)


TiKV's log is too large

Tikv logs will not be deleted circularly. One of our tikv instance logs reaches 1.4T (4T hard disk).How to set up a fixed number of days, or a fixed size, for tikv logs?

Please set this field in your tikv.toml:
log-rotation-timespan = "24h"

If it's not supported yet, please upgrade your TiKV to latest versions.

Hey, any idea why the log file size although seen as 1036 bytes by ls -la and stat, is reported as 140Meg by du -h and compsize, as seen here: https://github.com/kilobyte/compsize/issues/27#issue-502197861
I mean, why the wrong file size? maybe's something I did?

@howaboutsynergy Please note we don't recommend running TiKV on COW filesystems like btrfs or zfs.
But you may wish try disabling COW on the data dir of TiKV? https://wiki.archlinux.org/index.php/Btrfs#Disabling_CoW


Feature/review field api

Brought GribField methods in sync with the decisions of the API review discussions and added doctrings to them.
Fixes #64

There is a read the docs page showing the GribField documentation:
https://earthkit-data.readthedocs.io/en/feature-review-field-api/_api/data/readers/grib/codes/index.html
Please carefully check it, especially the arguments used in metadata() and data().

Is it possible to exclude the more 'internal' classes from the docs?

I dont mind that the internal classes are also in the docs, as long as we make very clear that they are internal details and should no be considered as part of the API.

With sphinx-autoapi we can fine-tune what classes end up in the documentation. We just need to agree on what an internal class means and make sure non of the documented methods return internal classes. E.g. GribField has the public @handle property of the type CodesHandle. Now I think CodesHandle is an internal class, so the users should not have access to it. But if it is true @handle should be private too e.g. @_handle.

The API docs has been greatly reduced, see: https://earthkit-data.readthedocs.io/en/feature-review-field-api/_api/data/readers/grib/codes/index.html


Popover and Modal Position / Anchor Sometimes Wrong (Potential Race Condition?)

🐛 Bug Report
We have been using UI Kitten for a while now with Expo Go, and it has been great! However, when we recently made a development build, we encountered a weird bug on (as far as we know) iPhone 13's running IOS 16.3.1 with the popover component where it wasn't using the proper anchor or Popover Placements.
We found this extremely odd, so first we checked and upgraded our package.json to use the latest versions "@ui-kitten/components": "^5.3.0", "@ui-kitten/eva-icons": "^5.3.0" and tried running it again on Expo Go, and we encountered it again. What we noticed that was also odd was the fact that it happens randomly where sometimes it is correct and other times where it is not correct, suggesting almost a race condition? (See attached video). When we then switched to using a modal component as a workaround as this didn't have the placement position, we noticed this still occurs (See second attached video), suggesting it may be something else and not just the Popover Placements.
Videos
https://user-images.githubusercontent.com/74498986/233358287-07f19ce0-8a8e-4195-9df9-4f15be1b2146.MP4
https://user-images.githubusercontent.com/74498986/233358294-4555ef61-da2c-47d5-9460-b04082417a0d.MP4
To Reproduce
A snapshot of our code for this section is as followed, and we think we are using this component correctly; happy to try and create a snack as well if it helps.
const renderScheduleCallOptions = () => (
    <TouchableOpacity
      onPress={() => {
        setScheduleMenuVisible(true);
      }}
      style={styles.scheduleCallButton}
    >
      <Icon
        style={styles.scheduleButtonIcon}
        name={props.isCallScheduled ? "calendar-times-o" : "calendar-plus-o"}
        color="white"
        size={32}
      />
    </TouchableOpacity>
  );

  return (
    <Popover
      anchor={renderScheduleCallOptions}
      visible={scheduleMenuVisible}
      placement="left end"
      backdropStyle={styles.backdrop}
      onBackdropPress={() => setScheduleMenuVisible(false)}
    >
            <Text category="s1" style={{ paddingHorizontal: 40 }}>
              Be called in...
            </Text>
            <Select
              style={{ marginVertical: 16 }}
              selectedIndex={scheduleOptions.timeSelectIndex}
              value={scheduleOptions.displayValue + " mins time"}
              onSelect={(index) => onTimeSelectChanged(index as IndexPath)}
            >
              <SelectItem title={"2 mins time"} />
              <SelectItem title={"5 mins time"} />
              <SelectItem title={"10 mins time"} />

https://user-images.githubusercontent.com/74498986/233358154-cfd7a7da-69d3-4314-93c5-f0e1597d09f7.MP4

https://user-images.githubusercontent.com/74498986/233358176-d146c188-73f9-473f-8d30-53d272f97fda.MP4

              <SelectItem title={"15 mins time"} />
              <SelectItem title={"20 mins time"} />
              <SelectItem title={"25 mins time"} />
              <SelectItem title={"30 mins time"} />
            </Select>
            <Button
              onPress={() => {
                setScheduleMenuVisible(false);
              }}
            >
              Schedule
            </Button>
    </Popover>
);

Expected behaviour
Should be consistent and at the right placement
UI Kitten and Eva version

Package
Version

@eva-design/eva

@ui-kitten/components
5.3.0

Environment information
Using UI Kitten with Expo Go but also affects development builds

My Modal sometimes losses it's anchor position and renders 1/2 off screen.


Allow sending a message to a given producer

The question is: what is the best API? Return a name? Return all names? Have an API for sending messages directly or broadcasting?

We will return a list with all names. :)


[14.0.x] ISPN-15208 IRAC state transfer may skip data

https://issues.redhat.com/browse/ISPN-15208

Merged, thanks


Add python utilities

Hello @danielsuo ,
what kind of python utilities do you expect, may you please specified?


Fix CPU/GPU restart order bug

Not affecting normal system operation, however if the CPU is started before the GPU the data transfer is desynced.

Satisfied by abdd624ae49cc2165393f0d1be180fcfdd8d38c8


garbled audio

Hi
Not sure why, but when I run either of the example code - I get garbled audio - when I use the Mini Model I get the "correct" audio
I have the following setup
Ubuntu 22.04
Python 3.11.4
nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Thu_Nov_18_09:45:30_PST_2021
Cuda compilation tools, release 11.5, V11.5.119
Build cuda_11.5.r11.5/compiler.30672275_0
python3 -m pip show torch
Name: torch
Version: 2.5.1
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3-Clause
Location: /data/parler-tts/.venv/lib/python3.11/site-packages
Requires: filelock, fsspec, jinja2, networkx, nvidia-cublas-cu12, nvidia-cuda-cupti-cu12, nvidia-cuda-nvrtc-cu12, nvidia-cuda-runtime-cu12, nvidia-cudnn-cu12, nvidia-cufft-cu12, nvidia-curand-cu12, nvidia-cusolver-cu12, nvidia-cusparse-cu12, nvidia-nccl-cu12, nvidia-nvjitlink-cu12, nvidia-nvtx-cu12, sympy, triton, typing-extensions
Required-by: descript-audio-codec, descript-audiotools, julius, parler_tts, torch-stoi, torchaudio
Thanks

same here, any solutions?

same here, mini model works, large model generates just noise.
Debian 11 / python 3.9.2, Cuda 12.6, torch 2.5.1

same here too, but with the mini model, my audio_arr vector get always a length of 5632. When the sample rate is then applied, the audio duration drops to 0. Could someone help me solve this issue please !?

Looking at this thread #157 it seems like maybe the change of the transformers version might have broken the larger model's inferencing, using 5d0aca9753ab74ded179732f5bd797f7a8c6f8ee I was able to generate audio using the larger model.

This issue seems important, the large model just does not work. I suggest to use a test suite, and run the output through whisper or something.

Hey everyone, thanks for opening the issue and helping on identifying where it blocks.
It actually comes from the audio encoder. While I still have to figure out how to actually fix it while being backward compatible for person using it with the previous repo version, you can find fixed weights for the current version by using:
model = ParlerTTSForConditionalGeneration.from_pretrained("parler-tts/parler-tts-large-v1", revision= "refs/pr/9")

Amazing @ylacombe  - this fixed things perfectly!


appelettv.com

Addon Manifest URL
https://iptv-tester close
Addon Description
spanish english vob tv
Language of Content
Multilingual (Default)
Choose Labels

[ ] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[ ] live tv
[ ] metadata
[ ] misc
[X] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[X] tv shows
[ ] video games

i don't know what this is, but it's not an addon, closing it


Softmax tutorial crashes (invalid arith.select) when n_cols is a multiple of 16 but <= 128

triton.__version__ is 3.0.0 for me
The tutorial code 02-fused-softmax.py given in https://triton-lang.org/main/getting-started/tutorials/02-fused-softmax.html fails to compile a kernel during the warmup when n_col is a multiple of 16 that is less than or equal to 128 (i.e., <= 16 * num_warps). Error looks like:
(triton) mantissa@MantissaAmpere:~/junk$ python3 ../Downloads/02-fused-softmax.py
loc("/home/mantissa/junk/../Downloads/02-fused-softmax.py":97:22): error: 'arith.select' op expected condition type to have the same shape as the result type, expected 'tensor<128xi1, #triton_gpu.blocked<{sizePerThread = [1], threadsPerWarp = [32], warpsPerCTA = [8], order = [0]}>>', but got 'tensor<128xi1, #triton_gpu.blocked<{sizePerThread = [4], threadsPerWarp = [32], warpsPerCTA = [8], order = [0]}>>'
Traceback (most recent call last):
  File "/home/mantissa/junk/../Downloads/02-fused-softmax.py", line 196, in <module>
    y_triton = softmax(x)
  File "/home/mantissa/junk/../Downloads/02-fused-softmax.py", line 144, in softmax
    kernel = softmax_kernel.warmup(y, x, x.stride(0), y.stride(0), n_rows, n_cols, BLOCK_SIZE=BLOCK_SIZE,
  File "/home/mantissa/junk/triton/lib/python3.10/site-packages/triton/runtime/jit.py", line 764, in warmup
    return self.run(grid=grid, warmup=True, *map(MockTensor.wrap_dtype, args), **kwargs)
  File "/home/mantissa/junk/triton/lib/python3.10/site-packages/triton/runtime/jit.py", line 662, in run
    kernel = self.compile(
  File "/home/mantissa/junk/triton/lib/python3.10/site-packages/triton/compiler/compiler.py", line 282, in compile
    next_module = compile_ir(module, metadata)
  File "/home/mantissa/junk/triton/lib/python3.10/site-packages/triton/backends/nvidia/compiler.py", line 317, in <lambda>
    stages["ttgir"] = lambda src, metadata: self.make_ttgir(src, metadata, options, self.capability)
  File "/home/mantissa/junk/triton/lib/python3.10/site-packages/triton/backends/nvidia/compiler.py", line 189, in make_ttgir
    pm.run(mod)
RuntimeError: PassManager::run failed

Repro: modify line 195 in 02-fused-softmax.py from
x = torch.randn(1823, 781, device='cuda')

to
x = torch.randn(1823, 80, device='cuda')

and run.

This seems broken ... the tutorial builds a simple kernel cache using softmax_kernel.warmup which depends only on BLOCK_SIZE. However it seems that warmup actually specializes based on the kernel arguments themselves (not just meta-parameters) and this is where the compiler gets in trouble ... it seems to give a different kernel entirely when y.stride(0) is divisible by 16. So the kernel that's cached depends on the parameters to the first run of the softmax() function (since that determines how the kernel is compiled). So if I write a function
def do_it(n_cols):
    torch.manual_seed(0)
    x = torch.randn(10000, n_cols, device='cuda')
    y_triton = softmax(x)
    y_torch = torch.softmax(x, axis=1)
    assert torch.allclose(y_triton, y_torch), (y_triton, y_torch)

then run
do_it(n_cols = 80)

then this crashes like before, but if I do
do_it(n_cols = 79)
do_it(n_cols = 80)

then it works OK, as the kernel correctly compiled for the n_cols = 79 case is successfully re-run for the n_cols = 80 case.
By the way, this also means I can crash the example by running
do_it(n_cols = 512)
do_it(n_cols = 511)

because the n_cols = 512 call specializes the cached kernel for aligned data, which doesn't work correctly for n_cols = 511. I'm seeing
Traceback (most recent call last):
  File "/home/mantissa/junk/tl_softmax.py", line 157, in <module>
    do_it(n_cols = 511)
  File "/home/mantissa/junk/tl_softmax.py", line 154, in do_it
    assert torch.allclose(y_triton, y_torch), (y_triton, y_torch)
RuntimeError: CUDA error: misaligned address
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1
Compile with `TORCH_USE_CUDA_DSA` to enable device-side assertions.

I don't know what is going on with the original bug, but I did some investigation on the second issue I found, and it looks like there already is a cache implemented for JITFunction inside runtime/jit.py that specializes based on whether input tensors are aligned or not. This seems to be implemented with compute_spec_key (checks 16-byte alignment) and the sig_and_spec portion of the key used to index self.cache. Since there's a built-in cache, I'm not sure why the tutorial implements its own (broken) caching system.

Hey, thanks for identifying and narrowing down this problem! I have a fix for the first problem identified in https://github.com/triton-lang/triton/pull/5161 (the compilation failure). I'll dig into trying to fix the tutorial code in a bit -- I also found it confusing when going through the tutorial!


chart does not include   sync clear buttons

chart does not include   sync clear buttons

Whoops , now it does ??

Most likely this particular sensor is connected to your iOS device.
Other options:

Firmware of this sensor does not support syncing
this sensor is out of range and you getting measurements via Ruuvi Cloud

Also syncing sensor with another phone will make sensor look as non connectable to other devices.

No other phone was syncing that time.  Notice a few minutes later, less than the screen shot indicates.

IOS station supports sync to this tag. But see recent related
iOS  issue

On Sep 5, 2022, at 11:26 AM, Denis @.***> wrote:

Most likely this particular sensor is connected to your iOS device.
Other options:
Firmware of this sensor does not support syncing
this sensor is out of range and you getting measurements via Ruuvi Cloud
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

This is added in 2.0


chore(schema): Avoid circular ref

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

218 of 218 (100.0%)  changed or added relevant lines in 24 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 96.193%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
8347

Relevant Lines:
8355

💛  - Coveralls

:tada: This PR is included in version 6.63.1 :tada:
The release is available on:

v6.63.1
GitHub release

Your semantic-release bot :package::rocket:


"Manage connections and gateways"

Select the Datasets tab, and then select the AdventureWorksProducts dataset from the list of datasets.
the msit.powerbi.com settings page now says "Manage connections and gateways"
instead of datasets

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

ID: 3f589c5a-4ee4-be0e-85d1-af9953969631
Version Independent ID: 0ccf859d-8b81-9d01-62ac-a9c0576887fa
Content: Tutorial: Connect to on-premises data in SQL Server - Power BI
Content Source: powerbi-docs/connect-data/service-gateway-sql-tutorial.md
Service: powerbi
Sub-service: powerbi-gateways
GitHub Login: @arthiriyer
Microsoft Alias: arthii

Thanks for your comment, @avongonzales2021. This issue has been assigned to the doc author for review.

Hi @avongonzales2021,
Can you please point to the place in Refresh data from an on-premises SQL Server database that needs to be fixed?
Thanks,
Kesem

No response. Closed.


Typo in an example of Binary crossentropy

The Binarycrossentropy estimated using TF2.0.0-alpha0 is 11.522857 (listed on website as 12.0007) where as Binarycrossentropy estimated using TF1.13.1 is 12.0007 which is correct. Gists are provided below. I suspect there was a change in tf.math.eps between TF1.13.1 and TF2.0. Thanks!
Gist using TF2.0.0-alpha0
Gist using TF1,13,1

@yifeif can you please merge this PR

Closing this PR as this not against master, please open a new PR against master
CC @mihaimaruseac


Flags for unconfigured swagger doc in http nodes

The following scenario has been brought up.
1) add HTTP In (say /path2) to canvas but don't Add/Edit the Doc settings (aka don't define any response codes)
2) wire in a function node and add an HTTP Out node
3) in swagger ui, you can test the API, but there is no definition of the expected response codes or output

If you have the swagger plugin enabled, and have http endpoints in your flows, but have not defined any additional documentation via the config, we use a placeholder in the swagger doc generation.
Should we present the user with a warning flag on the corresponding nodes, informing the user that they do not have any additional swagger doc attached?
This can be done in the swagger plugin itself, along with the check for http in nodes, that way we aren't changing the logic in the core nodes themselves.
Thoughts, @knolleary ?

Also, since this would be inside the swagger plugin, we wouldn't have to worry about the wrong people seeing it. If a user has the swagger plugin installed, a swagger related warning is pertinent to them.

+1


Fix the Callables Hierarchy

Summary
Currently we define things that we want to be accessible on all "callable" objects (functions, method references) on Any. This isn't really correct, as it means that things like <|, >> and << get suggested on any type, where they should really be suggested just on callables.
Value
Things work better.
Specification

[ ] Extend the Function atom with these operators.
[ ] Rename it to Callable or similar (think of a good user-facing name).
[ ] Add support for this in the hierarchy.
[ ] Ensure that all callable things look up within it for dispatch.
[ ] Ensure that this doesn't incur a performance penalty.

Acceptance Criteria & Test Cases

[ ] The thing works.

Probably worth doing #1828 at the same time as this, priorities and timing permitting.


Fix the format in the header of ECLab.

This should fix the problem that Frank had. The problem is with the header that are on the linux format instead of the dos format. I was able to fix it in vim but I can check that this patch will work because I can't run jupyter locally.

OK done

it definitely was it
here is a print out of headers before the change you suggested
['EC-Lab ASCII FILE\n', 'Nb header lines : 60                          \n', '\n', 'Potentio Electrochemical Impedance Spectroscopy\n', '\n', 'Run on channel : 1 (SN 2334)\n', 'User : \n', 'Electrode connection : standard\n', 'Ewe ctrl range : min = -1.00 V, max = 1.00 V\n', 'Ewe,I filtering : <None>\n', 'Channel : Floating\n', 'Acquisition started on : 10/06/2015 14:34:19\n', 'Saved on :\n', '\tFile : BC52-7-100C_C01.mpr\n', '\tDirectory : C:\\Users\\Administrator\\Documents\\EC-Lab\\Data\\Frank\\BC52\\\n', '\tHost : <IP_ADDRESS>\n', 'Device : SP-200 (SN 0337)\n', 'Address : USB\n', 'EC-Lab for windows v10.37 (software)\n', 'Internet server v10.37 (firmware)\n', 'Command interpretor v10.36 (firmware)\n', 'Electrode material : \n', 'Initial state : \n', 'Electrolyte : \n', 'Comments : \n', 'Cable : low current\n', 'Electrode surface area : 0.001 cm²\n', 'Characteristic mass : 0.001 g\n', 'Equivalent Weight : 0.000 g/eq.\n', 'Density : 0.000 g/cm3\n', 'Multi Sine          0                   \n', 'E (V)               0.0000              \n', 'vs.                 Eoc                 \n', 'tE (h:m:s)          0:00:0.0000         \n', 'record              0                   \n', 'dI                  0.000               \n', 'unit dI             mA                  \n', 'dt (s)              0.000               \n', 'fi                  1.000               \n', 'unit fi             MHz                 \n', 'ff                  10.000              \n', 'unit ff             mHz                 \n', 'Sel Nd              1                   \n', 'Nd                  6                   \n', 'Nt                  51                  \n', 'Log                 1                   \n', 'Va (mV)             10.0                \n', 'pw                  1.00                \n', 'Na                  3                   \n', 'E range min (V)     -1.000              \n', 'E range max (V)     1.000               \n', 'I Range             Auto                \n', 'Bandwidth           8                   \n', 'corr                0                   \n', 'nc cycles           0                   \n', "goto Ns'            0                   \n", 'nr cycles           0                   \n', 'inc. cycle          0                   \n', '\n', 'freq/Hz\tRe(Z)/Ohm\t-Im(Z)/Ohm\t|Z|/Ohm\tPhase(Z)/deg\ttime/s\t<Ewe>/V\t<I>/mA\tCs/µF\tCp/µF\tcycle number\tI Range\t|Ewe|/V\t|I|/A\tRe(Y)/Ohm-1\tIm(Y)/Ohm-1\t|Y|/Ohm-1\tPhase(Y)/deg\n']

OK great. Don't forget to restart the container for Frank and send him an
email or let me know and I will tell him.
2016-03-09 10:42 GMT-05:00 dalg24<EMAIL_ADDRESS>
Merged #57 https://github.com/ORNL-CEES/Cap/pull/57.
—
Reply to this email directly or view it on GitHub
https://github.com/ORNL-CEES/Cap/pull/57#event-583461067.

will do. waiting for docker hub to build the new image now.


Update FIPS 140-3 self tests (part 2)

Cleanup + remove a few tests that are not required.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Thanks for adding extended tests

It's almost tempting to make "severity: fips change" also enable "extended tests".

rebased

Hmm Are reviewers ok with this going to all branches?

I don't see a compelling requirement to back port this.
For 3.0, the requirements are quite different and 3.1 is missing a lot of the other recent activity.
I won't object to the changes being back ported.

@paulidale Do you see the failing tests? That is the reason for the backport

Yeah, backport then.  I'd forgotten this wasn't evp_test where we could condition out failing stuff.

24 hours has passed since 'approval: done' was set, but as this PR has been  updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

@slontis At least for 3.0 (but preferably also all other stable branches) I would prefer if just the test recipe change was backported. Could you please split the existing commit into 2 commits without any other changes so only the test recipe fix could be backported?

Ideally #25262 would go into older branches before this one merges.

Merged to the master branch. Thank you for your contribution.


Line is truncated when it is not modified

Describe the bug
Having an issue with a line of code getting truncated when saving. The setting to maintain source date is enabled.
To Reproduce
Here is the code I am saving:
               if TskOpnDs.MEMBER <> gMbrName;
                   wrtJobDbg(*ON);
                   wrtDebug(pgmname+': Opened member ' + TskOpnDs.MEMBER + '.+
                             Expected member ' + gMbrName + '.':*ON);
                   dump;
                   *inlr = *on;
                   return;
               endif;

When I save and I have NOT modified the wrtDebug line, it seems to truncate the .+ off the end of the line
When I save and I HAVE modified the wrtDebug line (to put them back), it does NOT truncate the line

Expected behavior
I would expect these two lines to left, regardless of whether the line was modified or not.
Environment (please complete the following information):

Extension version v0.8.30
IBM i OS version 7.4

@mattlavinder What is the record length of the source file you're working with?
Thanks!

I will be looking into this today.

@mattlavinder Think I've tracked it down. Fix will be coming out this evening.

@mattlavinder 0.8.31 just released. Will you give that release a whirl and try it out? Thanks!

@worksofliam Looks good to me.  Thanks!


feat(ntt): add zeta and zeta^2 values for bn256

Motivation
we need those values to match power distribution from sppark with halo2, in halo2-axiom domain.rs there is a function coeff_to_extended wich distributes powers on extended domain same as coset ntt,  in sppark coset ntt uses a bit different process, in halo2-axiom every three consecutive coefficient is pointwise multiplied to {1, zeta, zeta^2}.
in sppark every a_i is multiplied to root_of_unity^i.  so to match the we need those values, zeta and zeta_inverse,  to properly implement coeff_to_extended and extended_to_coeff.
Changes Overview
add zeta  and zeta^2 values for bn256

we need those values to match power distribution from sppark with halo2

Need more context about why those values are needed in halo2 and what are they used for, probably coset NTT calculation?

Will let @chaosma to approve this PR

I am confused why these changes are needed... can we not just send zeta to the distribute_zeta kernel you added to cusnark, then use the output of that as input to the standard NTT?

Closed as not necessary

zeta value and its square are used to calculate the "shifted" coefficients for coset_ntt.  They are needed, but the question is whether we want to add it here.  Like in John's reply: "can we not just send zeta to the distribute_zeta kernel you added to cusnark, then use the output of that as input to the standard NTT?".
This decision will up to the design of cusnark. But I  am not familiar this part


proxy template config changed for jupyter notebook. Fixes #338

Forwarding for RStudio instances stops working with this, so we probably need to make this configurable by blueprint


Error while fecthing extensions. XHR failed

Type: Bug
Error while fecthing extensions. XHR failed
VS Code version: Code 1.91.1 (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:08:12.169Z)
OS version: Linux x64 5.15.0-116-generic snap
Modes:

Hi @bagustris,
Thanks for reaching out. If you're using a VPN client or some local firewall, please turn these off and retry. If the error is still persisting, it will help us to investigate, if you can share the network log (command: Developer: Open Logs Folder).
Regards,
VS Marketplace Team

@Neelima10735584
I don't use vpn. I tried on two different networks, both failed. I tried to uninstall (vscode deb) and then install snap version, both failed. Then I tried several hours later; the problem was gone, and now I can install extensions again. :)
I have identic installation in another machine (v1.19.1 on Ubuntu 20.04), but doesn't face this problem.
Perhaps network problem, but I already tried on two different networks.

Hi @bagustris,
Thanks for your update. we are closing the issue. Please feel free to reopen or create a new issue if the problem still persists.
Regards,
VS Marketplace Team


Bump new version of Simla.com

Codecov Report

Merging #143 (c1b2e63) into master (d91ba6d) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head c1b2e63 differs from pull request most recent head 9fd39f8. Consider uploading reports for the commit 9fd39f8 to get more accurate results

@@            Coverage Diff            @@
##             master     #143   +/-   ##
=========================================
  Coverage     29.50%   29.50%
  Complexity     2328     2328
=========================================
  Files            80       80
  Lines          7154     7154
=========================================
  Hits           2111     2111
  Misses         5043     5043

Impacted Files
Coverage Δ

retailcrm/retailcrm.php
28.83% <100.00%> (ø)

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 d91ba6d...9fd39f8. Read the comment docs.


FILL plot misses data near 0, which shows up as white space

Reported by ansley b manke on 7 Jul 2009 17:27 UTC
Reported by Rolf Sander<EMAIL_ADDRESS>see also #363, possibly related.
This example data
yes? use fill_shade.nc
yes? show dat
     currently SET data sets:
    1> ./fill_shade.nc  (default)
 name     title                             I         J         K         L
 PRECL    Large-scale (stable) precipitat  1:23      1:23      ...       1:1

yes? list/wid=80 precl[            VARIABLE : Large-scale (stable) precipitation rate (m/s)
                        regrid: on Y
             FILENAME : fill_shade.nc
             SUBSET   : 23 by 12 points (LONGITUDE-LATITUDE)
             TIME     : 16-AUG-2002 11:30
      ... listing every   4th point
                 11.2W      0E(360)   11.2E      22.5E      33.8E      45E(405)
                  1          5          9         13         17         21
 71.2N / 12:  1.753E-08  1.603E-08  9.801E-09  1.389E-08  3.935E-09  6.106E-09
 65.6N / 11:  1.738E-08  8.787E-09  2.830E-09  2.852E-09  5.190E-09  1.378E-08
 60N   / 10:  1.613E-08  1.122E-08  3.675E-09  5.987E-10  5.738E-10  8.143E-09
 54.4N /  9:  1.443E-08  4.354E-09  4.573E-09  1.589E-09  1.554E-09  4.063E-09
 48.8N /  8:  1.825E-09  1.418E-08  8.197E-09  3.579E-09  1.168E-09  3.751E-10
 43.3N /  7:  3.768E-10  1.269E-08  5.993E-09  5.254E-09  1.038E-10  1.019E-08
 37.7N /  6:  3.307E-10  5.904E-10  1.571E-10  6.013E-10  7.560E-11  2.211E-11
 32.1N /  5:  3.511E-10  3.973E-11  4.444E-13  5.220E-13  6.902E-11  1.034E-16
 26.5N /  4:  8.384E-10  9.154E-15  4.926E-15  1.392E-14  1.167E-15  3.906E-15
 20.9N /  3:  1.752E-12  1.208E-12  3.549E-15  4.631E-15  1.452E-14  1.292E-14
 15.3N /  2:  2.778E-10  4.160E-11  1.364E-10  3.990E-10  6.091E-10  1.616E-09
 9.8N  /  1:  2.186E-08  5.653E-09  1.522E-09  5.002E-09  9.250E-09  6.185E-11
yes? stat precl

             Large-scale (stable) precipitation rate
             LONGITUDE: 12.7W(347.3) to 52E(412)
             LATITUDE: 8.4N to 72.6N
             Z:  N/A
             TIME: 16-AUG-2002 11:30
             DATA SET: ./fill_shade.nc

 Total # of data points: 529 (23*23*1*1)
 # flagged as bad  data: 0
 Minimum value: 9.92096E-17
 Maximum value: 3.442983E-08
 Mean    value: 4.072205E-09 (unweighted average)
 Standard deviation: 5.808465E-09

When plotted with most of the full region specified in square brackets, there is a gap between 20 and 25 deg N, but when no region or a smaller sub-region in x is given, or other combinations are tried, the fill plot is correct, as is a shade plot.
define view/x=0:0.5/y=.67:1 ul3
define view/x=0.5:1/y=.67:1 ur3
define view/x=0:0.5/y=.33:.67 ml3
define view/x=0.5:1/y=.33:.67 mr3
define view/x=0:0.5/y=0:0.33 ll3
define view/x=0.5:1/y=0:0.33 lr3
set win/asp=1.3
can mode logo

use fill_shade.nc

! this plot has a gap between 20 and 25 deg N:
set view ul3
fill/title="FILL precl[x=10W:50E,y=10S:70N](j=1:23:2]
)" precl[without specifying the region, it is ok:
set view ur3
fill/title="FILL precl" precl

! choosing a different region, it is ok:
set view ml3
fill/title="FILL precl[x=10W:40E,y=10S:70N](x=10W:50E,y=10S:70N]

!)" precl[after some mathmatics, it is ok:
set view mr3
fill/title="FILL (log(precl[x=10W:50E,y=10S:70N](x=10W:40E,y=10S:70N]

!)))" 10^(log(precl[using shade instead of fill, it is ok:
set view ll3
shade/title="SHADE precl[x=10W:50E,y=10S:70N](x=10W:50E,y=10S:70N]))

!)" precl[the same operations on a file created by saving 1.e8*precl yields the same behavior.  Using /hlim and /vlim instead of [x=, y=](x=10W:50E,y=10S:70N]

frame/file=fillbug.gif

Doing) also gives the same thing.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1675

Attachment from ansley.b.manke on 7 Jul 2009 17:28 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1675_GIT_947/fill_shade.nc

Attachment from ansley.b.manke on 7 Jul 2009 17:29 UTC
the image produced by the script
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1675_GIT_947/fillbug.gif

This behavior no longer happens


Definitions for gulp-plumber

Added definitions for gulp-plumber, using documentation and tests found in their repository: https://github.com/floatdrop/gulp-plumber

gulp-plumber/gulp-plumber.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/gulp-plumber
http://bower.io/search/?q=gulp-plumber
others?

[X] has a test file? (gulp-plumber/gulp-plumber-tests.ts or others)
[ ] pass the Travic-CI test?

@joeskeen thanks mate!


Fixes Collection filter and sort alignment if only one is enabled

Thanks Adrian! Absorbed into release v1.5.2. Closing pull request.


Video Slow Buffering - Would it be possible to allow watching videos at a lower resolution?

It may or may not entirely because of my internet, but the lecture videos buffers or becomes frozen a lot. But I have no problem viewing other videos such as youtube videos or have video calls from another continent. Would it be possible to improve on that? Maybe there can be an option for decreasing resolution?

Ok... I realized it might just be my browser's problem.

I'd still like to know where such issues might come from, did you experience this with live videos or recordings? Which browser did you use? Can you post the results of https://speedtest.in.tum.de? (Preferably at a time you usually experienced this)
Thanks for helping out :)

Not live videos. Just the recordings. I was using Chrome and it was very slow, but when I switched to Safari everything works smoothly. I think it's just Chrome; it has been slow for a while now. And I can't seem to open the speed test link.


fix(test): fix mocha compatible issue

fix https://github.com/angular/angular/issues/22150
in new version of mocha, the code like below
 beforeEach(async(() => {
    TestBed.configureTestingModule({
      declarations: [
        AppComponent
      ],
    }).compileComponents();
  }));

  it(`should have as title 'app'`, async(() => {
    console.log('title app test');
    const fixture = TestBed.createComponent(AppComponent);
    const app = fixture.debugElement.componentInstance;
    expect(app.title).to.equal('app');
  }));

will in the same Mocha.Runner.prototype.run.
so Mocha.Runner.prototype.run will run into ProxyZone instead of rootZone, we need to remove the root zone check, otherwise all mocha test will not begin.
also update mocha to 5.0.1

can you fix the conflicts?

@mhevery , I have rebased, please review.


Hackathon: Update RCLs for Server Side Next.js

Hackathon: Update RCLs for Server Side Next.js

@ancientTexts-net add thorough documentation on this with diffs

gitea-react-toolkit:

gitea-react-toolkit:

gitea-react-toolkit:

gitea-react-toolkit:

scripture-resources-rcl

scripture-resources-rcl

datatable-translatable

datatable-translatable

@ancientTexts-net have someone review this with you.

@ancientTexts-net have someone review this with you.


ci: Support using git for dependencies in test-installation

Fix the CI test that fails when dependencies in pyproject.toml contain git urls.
I didn't have time to update the tests, feel free to take over.

Updating the tests should be just: UPDATE_GOLDEN=1 pytest tests/integration/test_cookiecutter_generation.py::test_golden.
Ideally this repo should get the fix.
I plan to dedicate some time to repo-config in a not so distant future, if you didn't take care of it until then I can take it over.

OK, I took over this one and updated with the missing stuff. But now it needs an approval from someone else :)

Yeah, I can't approve this, being the author and all.


Nodemon + Node20 => TypeError [ERR_INVALID_ARG_TYPE]: The "code" argument must be of type number. Received type string ('SIGUSR2')

Bug description
After upgrading Node and my deps; I started getting

TypeError [ERR_INVALID_ARG_TYPE]: The "code" argument must be of type number. Received type string ('SIGUSR2')

when nodemon is restarting my process.
Here is the full stacktrace with DEBUG="*" :
rs
  nodemon bus emit: restart +10s
  nodemon:run sending kill signal to 1261 +10s
  nodemon:run checking ps tree for pids of 1259 +2ms
  nodemon:run bus.emit(exit) via SIGUSR2 +3ms
  nodemon bus emit: exit +48ms
[nodemon] starting `dotenv -- node --loader @bleed-believer/path-alias src/serve.ts`
  prisma:client:libraryEngine:exitHooks  exit event received: SIGUSR2 +0ms
  prisma:client:libraryEngine:exitHooks  exit event received: exit +1ms
  nodemon:run spawn sh -c dotenv -- node --loader @bleed-believer/path-alias src/serve.ts +4ms
  nodemon bus new listener: exit (1) +4ms
  nodemon:run start watch on: [ '*.*', re: /.*\..*/ ] +0ms
  nodemon:watch early exit on watch, still watching (1) +10s
node:internal/errors:490
    ErrorCaptureStackTrace(err);
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "code" argument must be of type number. Received type string ('SIGUSR2')
    at process.set [as exitCode] (node:internal/bootstrap/node:124:9)
    at process.exit (node:internal/process/per_thread:188:24)
    at process.<anonymous> (/Users/cyrilchapon/Code/wib/mtn/mtn-api/node_modules/@prisma/client/runtime/library.js:99:2170)
    at Object.onceWrapper (node:events:626:26)
    at process.emit (node:events:511:28) {
  code: 'ERR_INVALID_ARG_TYPE'
}

How to reproduce

Run a script importing prisma from nodemon
Type rs in terminal

Expected behavior
I'd expect it not to change behavior from when using Node 18
Prisma information
Irrelevant
Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: v20.1.0

Additional info :

"nodemon": "^2.0.21"
"ts-node": "^10.9.1"

Prisma Version
prisma                  : 4.13.0
@prisma/client          : 4.13.0
Current platform        : darwin
Query Engine (Node-API) : libquery-engine 1e7af066ee9cb95cf3a403c78d9aab3e6b04f37a (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 1e7af066ee9cb95cf3a403c78d9aab3e6b04f37a (at ../../node_modules/@prisma/engines/migration-engine-darwin)
Format Wasm             : @prisma/prisma-fmt-wasm 4.13.0-50.1e7af066ee9cb95cf3a403c78d9aab3e6b04f37a
Default Engines Hash    : 1e7af066ee9cb95cf3a403c78d9aab3e6b04f37a
Studio                  : 0.484.0
Preview Features        : postgresqlExtensions, clientExtensions

Oh my bad. I thought v20 was actually current; misread.
It was working perfectly under Node 18

Nice! I'll keep this open, we will check this later with v20 once we start fully testing against it.

@SevInf told me this should already be fixed with Prisma 4.14.0

@SevInf told me this should already be fixed with Prisma 4.14.0

That's indeed the case 🤩 Confirmed by upgrading.
Closing !


Docs.MS Release Updates for azure-core-serializer-avro-apache

Update docs metadata and targeting for release of azure-core-serializer-avro-apache

Docs Build status updates of commit ffe99f4:
:clock10: Full build: the average full build time is 97 min(s) 54 sec(s), this is based on the last 825 build(s) for this repository.

Docs Build status updates of commit ffe99f4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/core-serializer-avro-apache-readme-pre.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

package.json
:white_check_mark:Succeeded

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


Example for usage of openai-kit

@dylanshine This seems to be nice library for integration of OpenAI. I m here just suggesting if you can include Example of how to use this.  e.g. usage with env variables and usage without env variables and demonstrating usage of basic classes.

I think the difficulty with giving examples of injecting env variables is that there are so many different iOS and SSS frameworks to support. I would encourage consumers of this framework to research idiomatic patterns to handle env variables.

This wasn't about just environment variables, actually it will be better to have some running example so users can download and run by just updating the API Key.

Literally what the test suite is used for.


download rendered job assets via Optimus job inspect command

Description
Building upon https://github.com/odpf/optimus/issues/492 , add support for downloading rendered job assets.
Acceptance Criteria

[ ] Download job assets with support for custom schedule time in specified local directory

Depends on : https://github.com/odpf/optimus/issues/492
Tech Details
Post analysis Capture the details of the work needed to finish the task.

[ ] Take inspiration from Optimus job render command

handled in https://github.com/odpf/optimus/issues/577


faraday should pop the stack properly

@jamescway
Faraday was not properly restoring the spanID after it created a new one in a faraday outbound connection.
This solves that bug

:+1:


animationListener set and (remove all) animation

thanks to this lib.

can a animationListener be added to ItemAnimator?
call notifyItemRangeRemoved and clear the list will not animation?

any help is appreciated.

any help is appreciated.


 Also why did you write all this in caps? not only that why is your message written into a file in your PR?

This isn't the way to get people to help you. Should've made an issue if you don't contribute anything.


[5.8] Remove formatNotifiables that used in the next two methods 'queueNot…

remove formatNotifiables that used in the next two methods 'queueNotification' and 'sendNow'

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


dockerTools: Fix chown in fakeRootCommands

Description of changes
proot's --root-id "allows" chown only in the sense that it makes it succeed vacuously, i.e. a no-op. This is undesired if the goal is to actually create a layer with some files owned by different users.
Fortunately, fakeroot does allow persistence of emulated file owners, and it is possible to combine fakeroot with proot, so replace proot --root-id with fakeroot to do so.
This fixes a regression. Originally broken by 0aa8bd2dd97422b91ecb177cc93de1466be3da77 (#257462); #268458 fixed the build failure, but did not restore the original behavior.
CC @roberth @Mic92 @r-k-b
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

#283300 was merged first, so I updated this to only add a test.
CC @mplanchard @Mic92

@ofborg build nixosTests.dockerTools

Thank you!


Custom endpoint URLs for specific filter/action

My endpoints are:
/articles # all articles
/users/:id/articles # all articles by user

I want to use the same defined resource for both endpoints, but with a different filter.
I see that endpoints can't be changed after definition, even when sending a new one as an option.
Article.findAll() // /articles
Article.findAll({}, { endpoint: "/users/" + userId + "/articles" }) // /users/:id/articles

How about adding to DSHttpAdapterProvider http methods overriding option, exactly like options.baseUrl || resourceConfig.baseUrl but for endpoint?
DSUtils.makePath(
  options.baseUrl || resourceConfig.baseUrl, // exists
  options.endpoint || resourceConfig.endpoint // currently only resourceConfig.endpoint
)

How is this done in v3?


Pinterest bridge no longer seems to be working

RSS bridge now returns empty results for Pinterest results.
Example url: https://www.pinterest.com/ashie235/photography/
Atom RSS feed output - https://ghostbin.com/paste/dqbya
I also noticed that Pinterest seems to have a rss feed if you amend .rss at the end of the URL - https://www.pinterest.com/ashie235/photography.rss/
The only issue is that it gives you a small picture as opposed to the actual one.
In RSS feed - https://s-media-cache-ak0.pinimg.com/236x/44/ce/2c/44ce2cfa6267fde44790205135a78051.jpg
actual picture - https://s-media-cache-ak0.pinimg.com/564x/44/ce/2c/44ce2cfa6267fde44790205135a78051.jpg
Changing 236x to 564x in URL provides the full picture.

Hello! Did someone find a fix this?
I believe html from Pinterest boards have changed and now the parsing is broken. Btw, I don't know if I'm missing something, but source code of Pinterest boards look horrible, like it was purposefully made to be hard to parse!?
I've tried to duck-tape something from the existing PinterestBridge, but it didn't work.

I believe html from Pinterest boards have changed and now the parsing is broken. Btw, I don't know if I'm missing something, but source code of Pinterest boards look horrible, like it was purposefully made to be hard to parse!?

At least it was not prepared to be parsed easily. They most probably use some code generator with minifier to remove any unnecessary whitespace in order to reduce bandwidth, though it seems to be implemented very poorly as only certain parts are affected. You can paste the HTML in a compatible beautifier to make it readable again (this one for example).

I've tried to duck-tape something from the existing PinterestBridge, but it didn't work.

Since they now return RSS feeds by the means described above, we can make use of the FeedExpander. I got a working version already, though only for a given username and board. It won't work for searches. However they provide a gigantic amount of JSON-Data in their HTML, so I'll play around with it a bit and see what I can make of it.

Thanks for the explanation.
Please share the implementation using FeedExpander. That would be good enough, at least for my needs.

I would like to see how it was implemented. FeedExpander seems like more simple and easy way to get started with bridges :)

Please share the implementation using FeedExpander. That would be good enough, at least for my needs.

Hmm okay, I just finished a full implementation using the JSON data (it provides a bit more data than the RSS). The bridge now provides both approaches (via provided RSS or optionally using the custom parser) though it is not an easy read.
For a better reference please take a look at the wiki and one of the bridges already using it, like Acrimed.

Hi! Thanks for the update.
There is one issue. When using by username and board, returned RSS contains links to low resolution images (236x instead of 736x), its the same as using RSS directly from Pinterest.
Checking "Use provided RSS" on/off doesn't make any difference, but using "From search" method works as expected – shows large images.

Should be fixed now.

It works! Thanks.
But this code is beyond my level of understanding, I will have to start with something simpler :)