text:
.babelrc files in src/

Problem
Placing a .babelrc file in the same directory as source files does not respect the nested configuration.
References

Babel will look for a .babelrc in the current directory of the file being transpiled. If one does not exist, it will travel up the directory tree until it finds either a .babelrc, or a package.json with a "babel": {} hash within.

https://babeljs.io/docs/usage/babelrc/#lookup-behavior

Before Babel will actually compile your code, it needs to be configured. Create a new file, src/.babelrc:
{
  "presets": ["es2015-rollup"]
}

http://rollupjs.org/guide/#using-rollup-with-babel
Thoughts
I suspect that the source files are being copied to a temporary location, before babel is invoked, without copying the .babelrc file(s).

The readme for this repo and the "Using Rollup with Babel" section of the rollup guide might need to be updated. They seem to reference old versions of the libraries first with notes and caveats for the latest versions. It would be nice if the documentation for the current version was the default, and support for older versions was secondary.

Can you provide a repro? It's using Babel's builtin lookup behaviour, so should work correctly (and it's worked correctly for me in the past).

They seem to reference old versions of the libraries first with notes and caveats for the latest versions.

Can you show us where? Thanks

After building a minimal repro I realized the issue was that /src had a .babelrc file, but /lib did not, so nothing in /lib was being transpiled, just rolled up. The issue seems to be caused by my lack of understanding of the order of operations. Sorry for the confusion.

I think my documentation gripe was rooted in thinking the "Configuring Babel" preamble implied the es2015-rollup preset was no longer supported. It sounds like I can go back to using es2015-rollup instead of manually configuring .babelrc.
Again, sorry for the confusion.

repo: rollup/rollup-plugin-babel


text:
Cannot read property 'override' of undefined

I have error for some modules in node_modules.
.babelrc
{
  "presets": [
    [
      "es2015",
      {
        "modules": false
      }
    ],
    "stage-1"
  ],
  "plugins": ["external-helpers"]
}

rollup.config.js
import babel from 'rollup-plugin-babel';
import commonjs from 'rollup-plugin-commonjs';
import resolve from 'rollup-plugin-node-resolve';

export default {
  input: 'app.js',
  output: 'app.dist.js',
  plugins: [
    resolve(),
    babel({
      exclude: 'node_modules/**'
    }),
    commonjs()
  ]
};

error
app.js → stdout...
[!] (commonjs plugin) TypeError: Cannot read property 'override' of undefined in /Users/maksim/Documents/rotator/node_modules/express/node_modules/body-parser/index.js
node_modules/express/node_modules/body-parser/index.js
TypeError: Cannot read property 'override' of undefined in /Users/maksim/Documents/rotator/node_modules/express/node_modules/body-parser/index.js
    at error (/Users/maksim/Documents/rotator/node_modules/rollup/dist/rollup.js:185:14)
    at Promise.resolve.then.catch.err (/Users/maksim/Documents/rotator/node_modules/rollup/dist/rollup.js:9448:6)
    at <anonymous>

please update acorn
https://github.com/ternjs/acorn/issues/608

Thanks @Maxim-Chugaev for the hint, I have raised the minimal acorn version to 5.2.1 in rollup-plugin-commonjs@8.2.6. Please re-open this issue if you still experience this problem after updating.

source: gharchive/issue

authors: Maxim-Chugaev lukastaegert

repo: rollup/rollup-plugin-commonjs

license: mit

license_type: permissive

license_source: bigquery


text:
Remove rom-support from dependencies

fixes #76

depends on https://github.com/rom-rb/rom-model/pull/5

@cflipse ready to go

done in master

source: gharchive/pull-request

repo: rom-rb/rom-rails


text:
Creating with a changeset fails with Oracle

From @mrship on January 30, 2017 12:39
Using a changeset to create a record is failing for me when using Oracle, because (I think) you don't get the result of the INSERT statement back from Oracle using Sequel.
This is my code:
changeset_for_create = changeset(participant.to_hash)
  .map(:add_timestamps)
  .associate(participant.user, :users)
create(changeset_for_create(participant))

Which fails with:
2017-01-30 12:28:42 - Dry::Struct::Error - [ROM::Struct[Participant].new] :id is missing in Hash input:
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/dry-struct-0.1.1/lib/dry/struct/class_interface.rb:80:in `rescue in new'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/dry-struct-0.1.1/lib/dry/struct/class_interface.rb:74:in `new'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/class.rb:30:in `constructor_inject'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/function.rb:47:in `call'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/function.rb:47:in `call'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/array.rb:41:in `block in map_array'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/array.rb:41:in `map'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/array.rb:41:in `map_array'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/function.rb:47:in `call'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/transproc-1.0.0/lib/transproc/function.rb:47:in `call'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/rom-mapper-0.5.0/lib/rom/mapper.rb:95:in `block in call'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/rom-mapper-0.5.0/lib/rom/mapper.rb:95:in `each'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/rom-mapper-0.5.0/lib/rom/mapper.rb:95:in `reduce'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/rom-mapper-0.5.0/lib/rom/mapper.rb:95:in `call'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/rom-repository-1.0.0.rc1/lib/rom/repository.rb:326:in `map_tuple'
	/Users/shipmana/.rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/gems/rom-repository-1.0.0.rc1/lib/rom/repository/class_interface.rb:117:in `block in define_command_method'

This works fine with sqlite (in a test) but not with our production database of Oracle. I have had a similar issue in the past when expecting create to return a ROM::Struct but it does not with Oracle because of the issue with Sequel not using RETURNING as part of the INSERT query. As I understand it Sequel returns false for supports_returning? for Oracle.
My workaround is not to use changesets at all...
attributes = participant.to_hash.merge(user_id: participant.user.id)
create(attributes)

which is a real shame as I like the interface they provide and the convenience of the associate method and adding timestamps etc.
Can changesets be made to work without expecting a result from the underlying INSERT?
Copied from original issue: rom-rb/rom-repository#55

From @flash-gordon on January 30, 2017 12:54
I bet that's because rom-sql does not use returning statement for Oracle. Thanks for reporting this, I'll have a look. Actually, I'm going to add more full-featured Oracle support later this month. It seems Oracle's licensing policy does allow us to have XE on Travis and it can be downloaded & installed in acceptable time. I found other OSS projects that do so.

There was 0 work done for Oracle so far, so it doesn't surprise me :/

From @mrship on January 30, 2017 20:12
In reading the rom 3.0 blog post I realised that I can use the commit method on changeset instead of using create.
changeset_for_create = changeset(participant.to_hash)
  .map(:add_timestamps)
  .associate(participant.user, :users)
changeset_for_create.commit

That's a practical workaround for me for now.

From @mrship on January 31, 2017 11:50
It turns out that using commit is not a practical workaround as it doesn't actually work.
Per the code below if you call #commit on an updated changeset, it actually updates every record, rather than the one specified by id.
I feel like I'm doing something wrong though because I can't credit that this is the expected behaviour?
#!/usr/bin/env ruby
require 'bundler/inline'

gemfile do
  source 'https://rubygems.org'
  gem 'rom-sql', '~> 1.0'
  gem 'rom-repository', '~> 1.0'
  gem 'dry-types'
  gem 'sqlite3'
end

require 'rom-repository'
require 'rom-sql'
require 'dry-types'

class Participants < ROM::Relation[:sql]
  schema(:participants, infer: true) do
    associations do
      belongs_to :user
    end
  end
end

config = ROM::Configuration.new(:sql, 'sqlite::memory') do |container|
  default = container.default

  default.create_table :participants do
    primary_key :id
    string :name, null: true
  end

  default.connection.execute "INSERT INTO participants (name) VALUES ('Fred')"
  default.connection.execute "INSERT INTO participants (name) VALUES ('Bob')"

  container.register_relation(Participants)
end

module Repositories
  class Participant < ROM::Repository[:participants]

    def by_id(id)
      participants.by_pk(id).one
    end

    def update_name(participant, name)
      changeset(participant.id, participant.to_hash.merge(name: name)).commit
    end
  end
end

container = ROM.container(config)
repo = Repositories::Participant.new(container)
puts repo.by_id(1).inspect
puts repo.by_id(2).inspect

participant = repo.by_id(1)
repo.update_name(participant, "Jane")

puts repo.by_id(1).inspect
puts repo.by_id(2).inspect

#<ROM::Struct[Participant] id=1 name="Fred">
#<ROM::Struct[Participant] id=2 name="Bob">
#<ROM::Struct[Participant] id=1 name="Jane">
#<ROM::Struct[Participant] id=2 name="Jane">

@mrship seems like a weird and nasty bug, looking into it

From @mrship on January 31, 2017 12:29
Thanks for the quick response.

@mrship OK I fixed it in 1.0.1 (released already)

From @mrship on February 13, 2017 17:37
@flash-gordon Thanks for looking at this. I had a similar issue when trying to use changesets on a table that has no primary key; again it fails because there is, er, no primary key.
I can insert records directly into one of these tables, and tbh I'm not sure if changesets can/should be made to work without a primary key in the table definition, but this was not an Oracle-specific bug as it fails on sqlite too.
Let me know if you think this is worth reporting as a separate issue. I saw your comments in gitter today that changesets expect a primary_key of id, so perhaps this is beyond the current implementation?

From @flash-gordon on February 12, 2017 22:22
I have some news about this. Sequel currently does not support RETURNING for Oracle. Instead it queries the sequence associated with the table for the current value after running the insert statement. It works so far, but if two conditions are met:

You passed autosequence: true with connection options.
Your sequences are called "seq_#{table}_#{primary_key}". And otherwise you're out of luck.

Actually, Sequel's connection object can be hacked by setting a proper value for @primary_key_sequences instance variable which maps tables and sequence names. I've no idea why Sequel does not offer a public API for this provided that adding it is obviously a piece of cake.
It's still not perfect, i.e. won't work for composite PKs and requires a separate query (though a very fast one), but should do the trick for you @mrship
I'll be adding support for RETURNING to Sequel, these manipulations with @primary_key_sequences is a temporary workaround.

@flash-gordon you do what you gotta do :) I don't use Oracle so I don't have much to say. Feel free to push a release once this is sorted out

From @flash-gordon on February 26, 2017 18:36
heh, it took me a while https://github.com/jeremyevans/sequel/pull/1312/files

From @flash-gordon on February 14, 2017 13:58
@mrship I recently added a check for PK presence https://github.com/rom-rb/rom-sql/blob/40885890ef33f4c1cd23e71019a4a616985f7f0b/lib/rom/sql/relation.rb#L94-L97

repo: rom-rb/rom-sql


text:
Links are broken in documentation footers.

These links have %%7Bversion%7D instead of version number.
Clicking on it leads to 404.
Example:
https://rom-rb.org/5.0/learn/core/schemas/
and all other pages too.

Thanks for reporting this, I've fixed it via https://github.com/rom-rb/rom-rb.org/commit/890948a1aa72c55e8dda52ff96eeb2cd6ea8d26f
ps. bugs related to the website should be reported in rom-rb/rom-rb.org repo 😄

authors: alex-oleshkevich solnic

repo: rom-rb/rom


text:
Do we want to include actual_gain_arr in some data objects?

In both K-Gain and Non-linearity calibration functions, there's an input array with the actual EM gain values (measured vs. commanded) that is used in the functions. Do we want to include this information in the inout Dataset? Currently, the only keyword related to EM gain is CMDGAIN, which is associated with the commanded gain.
See also the comment on PR#116: https://github.com/roman-corgi/corgidrp/pull/116#discussion_r1669132879

Just to understand that right: I see that this is an input array, which is stored in a txt file? Where and how was this generated? I think this should be a calibration file, so that we can convert the commanded EM gain to the actual one, store it in the header and use it?

@hsergi I think this is a duplicate issue. Please see #108. If you agree I'd like to close this issue.

authors: JuergenSchreiber hsergi maxwellmb

repo: roman-corgi/corgidrp

license_source: github-api


text:
[improvement] Chaining keeps original query

Consider the following test:
async def test_find_changed(preset_documents):
    query1 = Sample.find_many(Sample.integer > 1)
    query2 = query1.find_many(Sample.nested.optional == None)
    assert await query1.to_list() != await query2.to_list()

This fails as the query2 = ... line also changes the first query. Should this instead return a new object with the parameters set differently?

The thing is, query1 is a FindQuery object, which supports chaining operations. Like
Sample.find(...).find(...).find(...).sort(..))

But your point is very good. As FindQuery is in fact a mutable object, I think for this case it would be good to implement __copy__ (and even __deepcopy__) method to follow the pythonic way here. To be able to do things like:
async def test_find_changed(preset_documents):
    query1 = Sample.find_many(Sample.integer > 1)
    query2 = copy(query1).find_many(Sample.nested.optional == None)
    assert await query1.to_list() != await query2.to_list()

What do you think about this?

I'm also working on rewriting the Query classes to merge the Many and One versions as i mentioned on the discord. For that I was trying to change this as well by copying by default when the chaining functions are called. I think something can be sad for both ways. Maybe wait a bit till I get my rewrites working as it would be trivial to change those to your suggestion and then see what works better?

Yes. I saw the commit, you accidentally pushed to another PR. Some things are really good like __await__ def with to_list inside for the group search. And I had some questions also. Looking forward to the final version of this. If you would like to discuss it - I'm open for talks :-)
Also, sorry for the off-topic, could you pls take a look at your discord? I texted you there with my new project idea. Probably it would be interesting for you to join.

authors: jorants roman-right

repo: roman-right/beanie


text:
Run server??

How do you run this on windows?

You can run it with $ node --harmony bin/www, or with $ babel-node bin/www (if you comment out line 5 in bin/www). And also have all dependencies installed. But how to do that on windows, I have no idea.

Same procedure. Thanks.

authors: dfsdfsdfs romanmatiasko

repo: romanmatiasko/reti-chess


text:
web_console: Make running debugger instance URL more explicit

Rewrite the running debugger instance URL in log message to include http:// prefix, to make it easier to one-click open debugger in web browser from command line or editor/IDE terminal window.
For example:
CRITICAL:root:Web-PDB: starting web-server, running on http://myhostname:5555

Closes: #51

Thank you for your contribution. But the protocol can also be https. It may be a rare case but still.

Thank you for your contribution. But the protocol can also be https. It may be a rare case but still.

@romanvm Can you explain this? I can't see anything in the documentation on how to enable https support. The obvious workaround would be a proxy server but I didn't see mention of that either. If there is a supported way to enable https then the code could be updated to put http or https depending on how it was enabled.
I know this is a really minor thing but a change like the one proposed just makes debugging more smooth, so I can focus on the problem I'm debugging without distractions (how ever minor ;-)).
This debug interface is really great, so easy to deploy and use anywhere.

@mmtj  and @romanvm recommend closing as this was resolved in https://github.com/romanvm/python-web-pdb/commit/b7b3a412ca988ad655f56db80fdea96674c88da2

Oh, sorry for the late reply. Sadly this PR slipped my mind...

But the protocol can also be https. It may be a rare case but still.

Fair point. And I can see the case when proxies are involved.
However my guess is that it won't be much of an issue, because in case https is enforced somehow, it is quite common to have automatic redirect http -> https in place. Having the http as default would definitely cause less trouble than trying to do it the other way around :)
Anyway, I am glad that the original issue is resolved now, thanks @romanvm
Therefore I am closing this PR as redundant.

authors: clach04 mmtj romanvm

repo: romanvm/python-web-pdb


text:
Quickstart not working with Neon

Hi Romeerez,
I tried going through the docs to build a demo application for your ORM which I found interesting, but when trying to create the database and initialize the schema it keeps throwing errors since Neon (the platform where I'm hosting the Postgres instance) requires ssl. When enabling SSL in die config or even on the db string itself I get infinitely prompted with the following image:

Any idea why this might be?
If I remove SSL then I get the following:

db create database is trying to create a database and requires admin permissions for that.
When using hosting, simply skip this step, database creation is done by hosting. Only run db migrate to create tables.

I've improved the script to handle this, and the quickstart docs, thank you for pointing!
Now when trying to create a db and it requires ssl, it will output:
SSL is required: append ?ssl=true to the database url string

And after setting it to true and trying to create the database on a cloud hosting, it will warn and confirm:
Permission denied to create database.
Don't use this command for database service providers, only for a local db.
? Would you like to share admin credentials to create a database › (Y/n)

authors: Stoffberg romeerez

repo: romeerez/orchid-orm

license: MIT


text:
高清城市搜索hyouka vcb报错，疑似单条结果会出问题

PT 助手版本:

PT 助手安装方式：（市场安装，或zip包安装）

浏览器名称及版本：

浏览器是否安装了其他插件：

停用其他插件后是否正常工作：

问题描述：
似乎搜frds没问题，不过第一条丢了

相关截图：

重现步骤：

两个都没号，有号的同学帮忙测试一下。

没事...我有空会自己看下的，现在是鸽制状态

应该和站点主题啥的有关系，这个站不同主题返回的页面格式很不相同

原始主题未能复现

原始主题未能复现

hdcity.city么？我才发现消息都没获取到

原始主题未能复现

hdcity.city么？我才发现消息都没获取到

是的，如图

不过确实有个问题，label没有颜色，这个种子前面有个50%的label

原始主题未能复现

hdcity.city么？我才发现消息都没获取到

是的，如图

我进站到现在一直是procity这个主题，你也是么？

或者种子列表风格是？

或者种子列表风格是？

ProCITY+Default Modern

或者种子列表风格是？

ProCITY+Default Modern

那我有空看看是不是因为我在保种...，frds那第一个我也在保种

cmct,第一页50个结果，我搜的只显示42个，而且目测上传下载全是错的。先这样吧，有空研究下

不过确实有个问题，label没有颜色，这个种子前面有个50%的label

我这一步就啥都没有，我看页面是class 是 tr_normal trblock

哭了...除了tr_review还有tr_inpro，我说咋又少了一个...

authors: Rhilip ronggang ted423 tongyifan

repo: ronggang/PT-Plugin-Plus


text:
line 35 in local.ts prevent sending request with the user data in the body should be updated like line 21

Hi, thanks for the fix in #19, I think the register also should be fixed the same as #19 in the line 35 of local.ts.
I'll make a pull request for it.

included in version 3.0.4

authors: ronzeidman yahyaKacem


text:
fix example config spelling in readme

fix "seperator" spelling in README.md

Thanks :)

authors: luukvbaal roosta


text:
🛑 TROJAN 🇺🇸 United States USO 1 is down

In 838256f, TROJAN 🇺🇸 United States USO 1 (https://ust-2.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇺🇸 United States USO 1 is back up in 67117f8.

repo: roosterkid/opentunnel-status-server


text:
🛑 TROJAN 🇬🇧 United Kingdom UKO 1 is down

In 9997e62, TROJAN 🇬🇧 United Kingdom UKO 1 (https://ukt-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇬🇧 United Kingdom UKO 1 is back up in e661a1c.


text:
🛑 TROJAN 🇸🇬 Singapore SGO 1 is down

In 1dd8033, TROJAN 🇸🇬 Singapore SGO 1 (https://sgt-2.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇸🇬 Singapore SGO 1 is back up in 0934b81.


text:
🛑 V2RAY 🇺🇸 United States USO 1 is down

In 140942b, V2RAY 🇺🇸 United States USO 1 (https://usv-4.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: V2RAY 🇺🇸 United States USO 1 is back up in 87210bb.


text:
🛑 XRAY 🇩🇪 Germany DEH 1 is down

In a8998eb, XRAY 🇩🇪 Germany DEH 1 (https://dex-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇩🇪 Germany DEH 1 is back up in 9e2750a.


text:
🛑 TROJAN 🇩🇪 Germany DEH 1 is down

In c1a8677, TROJAN 🇩🇪 Germany DEH 1 (https://det-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇩🇪 Germany DEH 1 is back up in 57ecea6.


text:
🛑 SSH 🇮🇩 Indonesia IDA 1 is down

In a00f42a, SSH 🇮🇩 Indonesia IDA 1 (http://ids-2.optnl.com:8080/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSH 🇮🇩 Indonesia IDA 1 is back up in 13f5dc1.


text:
🛑 TROJAN 🇬🇧 United Kingdom UKH 2 is down

In 21d5b4b, TROJAN 🇬🇧 United Kingdom UKH 2 (https://ukt-2.optnl.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇬🇧 United Kingdom UKH 2 is back up in 1e36016.


text:
🛑 V2RAY 🇺🇸 United States USF 1 is down

In 2527595, V2RAY 🇺🇸 United States USF 1 (https://usv-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: V2RAY 🇺🇸 United States USF 1 is back up in 535e861.


text:
🛑 TROJAN 🇮🇩 Indonesia IDJ 1 is down

In c58d4e9, TROJAN 🇮🇩 Indonesia IDJ 1 (https://idt-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇮🇩 Indonesia IDJ 1 is back up in 504cce5.


text:
🛑 TROJAN 🇮🇩 Indonesia IDJ 1 is down

In a9a35e4, TROJAN 🇮🇩 Indonesia IDJ 1 (https://idt-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇮🇩 Indonesia IDJ 1 is back up in 137bbe9.


text:
🛑 TROJAN 🇸🇬 Singapore SGP 1 is down

In 2e7bde3, TROJAN 🇸🇬 Singapore SGP 1 (https://sgt-2.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇸🇬 Singapore SGP 1 is back up in 7aee571.


text:
JavaScript Source maps "not used"

Hey,
any hints how to debug why JavaScript source maps are not "working" in my actual setup?
The source maps get generated by gulp, in Chrome they are enabled (scss source maps are working…), the comment is present in the uglified dist js file, but Chrome does not apply the source map…
I am on Max OS X 10.10.4 and Chrome 43.0.2357.132.
Thanks

Please can you first take this kind of discussion to the Roots Discourse forum.
The issues tracker is for PRs, confirmed bugs and the occasional feature request as per our Contributing Guidelines.
Thanks.

authors: Foxaii pablopaul

repo: roots/sage


text:
Change default Controller path to app/controllers

Move the Controller default path to app/controllers.
Controller PR to change which I will merge if/when this is merged—https://github.com/soberwp/controller/pull/40

can you pls rebase since i just merged #1912? then we could prolly coordinate the merges so we don't screw up any new builds

Cool, will rebase this weekend.

authors: darrenjacoby retlehs


text:
Consolidate on GitHub actions

[x] I've read the guidelines for Contributing to Roots Projects
[x] This request isn't a duplicate of an existing request
[x] This is not a personal support request that should be posted on the Roots Discourse community

Summary
We should consolidate the build process on GitHub actions instead of a confusing mix of AWS Lambda + Travis CI
Motivation
Simplicity so everything is in one place

Done as of https://github.com/roots/wordpress/pull/28
If we need to, we can manually trigger workflow runs or re-enable the old method (AWS Lambda).

repo: roots/wordpress


text:
Loop of LND Unlock, Filtering Blockchain

Hey guys,
I'm struggling for some months about one problem:
Whenever I want to do some stuff in the main menu (like transferring my onchain funds), I'm prompeted to insert my LND password. After that, some strange block filtering starts with 570323/570323 and wait and inserting LND password again and again and again.
SOMETIMES after  this, I may get back to the main menu and I can select an option. But then everything starts again.
What I already did:

upgraded from 0.9 to 1.0 and waited about 5 weeks to verify
upgraded from 1.0 to 1.0
replaced power supply with the recommended one from the main post about raspiblitz
tried some commands from #292

I'm totally lost. I only want my device running and maybe have some fun with Joule, Zap and connect to my Casa node.
Thank you so much guys for any kind of help

A Unlock-Loop is happening, when LND is crashing and the RaspiBlitz tries to start it up again. On every re-startup after a crash, you are asked to unlock your wallet again.
The question is why LND is crashing. If we look at the logs you posted (under LND Error Logs), there is "LND panic invalid page type 13434: 10" and last normal log around that time was "syncing channel graph" that seems to be the reason. But I have no idea what that is. That seems to be a LND problem. So it would be best to ask at the LND Dev Slack or GitHub for more details: https://dev.lightning.community
Possibly related LND GitHub issues could be those: https://github.com/LightningNetwork/lnd/issues?utf8=✓&q=is%3Aissue+panic+sync+channel+graph
But we are preparing the new RaspiBlitz v1.2 release which will have the new LND 0.60 on board (once the final release is out). There is a change that this is fixed than, but thats not 100%.

Thanks, I'm patient to the next cool versions of raspiblitz.
Is there at least a way to load my funds from the Wallet? (they are onchain, not LND)

A Unlock-Loop is happening, when LND is crashing and the RaspiBlitz tries to start it up again. On every re-startup after a crash, you are asked to unlock your wallet again.
The question is why LND is crashing. If we look at the logs you posted (under LND Error Logs), there is "LND panic invalid page type 13434: 10" and last normal log around that time was "syncing channel graph" that seems to be the reason. But I have no idea what that is. That seems to be a LND problem. So it would be best to ask at the LND Dev Slack or GitHub for more details: https://dev.lightning.community
Possibly related LND GitHub issues could be those: https://github.com/LightningNetwork/lnd/issues?utf8=✓&q=is%3Aissue+panic+sync+channel+graph
But we are preparing the new RaspiBlitz v1.2 release which will have the new LND 0.60 on board (once the final release is out). There is a change that this is fixed than, but thats not 100%.

A similar LND panic was reported here: https://github.com/lightningnetwork/lnd/issues/2626
And @Roasbeef said

"Looks like rasp pi disk corruption."

So you might want to try testing your power supply to make sure it's adequate, and perhaps buy a better new SD card and burn it and start anew.

Hey,
I'm in the loop again after upgrading to v1.2

Any ideas how I can fix this? Is there any way to release my 0.05 BTC (onchain)?

Please provide links to these full logs.
syslog: sudo cat /var/log/syslog | nc termbin 9999
bitcoind: sudo cat /mnt/hdd/bitcoin/debug.log | nc termbin.com 9999
lnd: sudo cat /mnt/hdd/lnd/logs/bitcoin/mainnet/lnd.log | nc termbin.com 9999

In the meantime there is a recent LND release in the past 24 hours that has some bug fixes. So perhaps we can apply it and see if it fixes things for you?
https://github.com/lightningnetwork/lnd/releases/tag/v0.6.1-beta-rc1

@fluidvoice thanks!
raspiblitz: https://termbin.com/l68f
syslog: https://termbin.com/ec8e
bitcoind: https://termbin.com/6xt4
lnd: https://termbin.com/pwwu

@wuestenigel I would try burning the SD card and trying again to see if the problem is reproducible or perhaps something bad on disk.

@fluidvoice Thanks for the idea. I'm struggling with this for months since v0.9.
Completely changed my power supply and SD Card and flashed it like 10x times.
I just want to release my funds so I'm safe and may start from scratch again. :/

@fluidvoice Thanks for the idea. I'm struggling with this for months since v0.9.
Completely changed my power supply and SD Card and flashed it like 10x times.
I just want to release my funds so I'm safe and may start from scratch again. :/

Oh, I didn't realize this was the same problem (lnd panic) as before.  Very strange as nobody else is seeing this.

@fluidvoice Thanks for the idea. I'm struggling with this for months since v0.9.
Completely changed my power supply and SD Card and flashed it like 10x times.
I just want to release my funds so I'm safe and may start from scratch again. :/

@wuestenigel A LND dev said about your issue "it seems like you've encountered database corruption."

@fluidvoice Thanks for your help. Problem is this may take another weeks or months(!!!) to download the data again. I had this experience already.
Do you know if it may be ok to delete everything without the blockchain data?

@fluidvoice Thanks for your help. Problem is this may take another weeks or months(!!!) to download the data again. I had this experience already.
Do you know if it may be ok to delete everything without the blockchain data?

You could try it.  Delete only the LND data (/mnt/hdd/lnd) and burn a new SD card, while keeping the blockchain data (/mnt/hdd/bitcoin).  It might work but might not. I'd prob try it...
sudo systemctl stop lnd.service
sudo systemctl stop bitcoind.service
sudo rm -rf /mnt/hdd/lnd
sudo shutdown now
now burn new SD card and reinstall.
btw, I too got frustrated with delays in downloading/syncing the blockchain so now I made a 2nd copy as a backup on an old spare 3.5" HDD.  No more weeks of downloading for me!

wait...

it doesn't work by default that way. I tried it.  You cannot remove the entire directory.  Let me see how to remove only the data portion of that directory.

Hey @fluidvoice,
great. Should I rename now or just wait until you experimented a little bit?
Very much appreciated!
Marco

Hey @fluidvoice,
great. Should I rename now or just wait until you experimented a little bit?
Very much appreciated!
Marco

you can restore via the seed phrase when you first created your LND wallet. For me no. I cannot find my seed phrase but no big deal as only a tiny amount of $ was in it.

@fluidvoice
I tried to rename the folder, but after creating a new SD card it says:
./00raspiblitz.sh: line 336: [: -eq: unary operator expected
WARN: LND needs re-setup
*** Check Basic Config ***
*** Check bitcoin Running ***
*** Check bitcoin Responding *** (can take a while)
OK - chainnetwork is working
*** Check LND Config ***
cp: cannot create regular file '/mnt/hdd/lnd/lnd.conf': No such file or directory
sed: can't read /mnt/hdd/lnd/lnd.conf: No such file or directory
chown: cannot access '/mnt/hdd/lnd/lnd.conf': No such file or directory
FAIL - Was not able to setup LND
admin@wuestenigelRaspi:~ $

Working again!!! There is nothing as good as a clean install. Thanks!!

authors: fluidvoice rootzoll wuestenigel

repo: rootzoll/raspiblitz


text:
Add doxyfile

Needed for #278 on all branches for multiversion.

https://github.com/Mergifyio backport foxy galactic humble iron


text:
Enable Travis

As per subject.
This would enable CI for this repository.

Done.


text:
Fix plotting of values containers in an array

Before this commit, attempting to plot a value in an array would trigger the following exception:
Traceback (most recent call last):
  File "/opt/ros/eloquent/lib/python3.6/site-packages/rqt_plot/plot_widget.py", line 259, in on_topic_edit_textChanged
    plottable, message = is_plottable(self._node, topic_name)
  File "/opt/ros/eloquent/lib/python3.6/site-packages/rqt_plot/plot_widget.py", line 154, in is_plottable
    fields, message = get_plot_fields(node, topic_name)
  File "/opt/ros/eloquent/lib/python3.6/site-packages/rqt_plot/plot_widget.py", line 137, in get_plot_fields
    for slot, slot_type in field_class.get_fields_and_field_types().items():
AttributeError: 'NoneType' object has no attribute 'get_fields_and_field_types'

Line 129 in plot_widget.py:
    field_class = message_helpers.get_message_class(slot_type)

...would be executed with slot_type set to sequence<foo_msgs/Foo> which
returns a None value. field_class is set to None and the call to
get_fields_and_field_types is then failing.
This pull request adds an extra layer of logic before calling get_message_class
to ensure that, if the type is an array, we extract the inner array type
instead of attempting to run get_message_class on the DDS sequence type
directly.

What is the timeline of this issue? Will the eloquent or foxy debian package be patched with this fix? I'd like to plot sensor_msgs/JointState position[i] etc in ROS2.

@dirk-thomas Is this ready to be merged? It looks like @thomas-moulard made the necessary fix in his most recent commit.

@Michael-Equi I am not maintaining this repository anymore. Maybe the current maintainer can step up?

Closing in favor of https://github.com/ros-visualization/rqt_plot/pull/71.

authors: Michael-Equi briansoe66 dirk-thomas ivanpauno thomas-moulard


text:
keep build files

I am not sure if this fixes the builds on the old farm (http://jenkins.ros.org/job/devel-jade-rospack/ARCH_PARAM=amd64,UBUNTU_PARAM=trusty,label=devel/7/console) and/or if it is a problem for the new farm.

A partial revert is not an option.

repo: ros/rosdistro


text:
cost_map: 0.3.1-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository cost_map to 0.3.1-0:

upstream repository: https://github.com/stonier/cost_map.git
release repository: https://github.com/stonier/cost_map-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.23
previous version for package: 0.3.0-0

cost_map_ros
* minor api name fixes for Costmap2D references

This one has me by the short and curlies...I am unable to draw any conclusions from travis.
Build cache for "lunar"
- trying to use local cache
- trying to fetch cache
- update cache
E.
======================================================================
ERROR: test_build_caches.test_build_caches
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/travis/build/ros/rosdistro/test/test_build_caches.py", line 20, in test_build_caches
    generate_distribution_caches(INDEX_YAML, dist_names=dist_names)
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/rosdistro/distribution_cache_generator.py", line 62, in generate_distribution_caches
    ignore_local=ignore_local, include_source=include_source, debug=debug)
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/rosdistro/distribution_cache_generator.py", line 75, in generate_distribution_cache
    include_source=include_source)
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/rosdistro/distribution_cache_generator.py", line 206, in _get_cached_distribution
    cache.update_distribution(rel_file_data)
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/rosdistro/distribution_cache.py", line 92, in update_distribution
    for repo_name in sorted(new_data['repositories'].keys()):
AttributeError: 'NoneType' object has no attribute 'keys'

Sorry for the inconveniences. This bug was exposed with the addition of ROS Lunar. This will be addressed soon: see ros-infrastructure/rosdistro#94

Holding to wait for sync...

authors: dirk-thomas stonier


text:
Adding rosgpt to documentation index for distro

i'd like my repo "rosgpt" to be index and documented on ros.rog.

No response in over a month, so I'm going to close this out.  If you'd like to resubmit, please fix the issues identified above.  Thanks.


text:
mapviz: 2.4.3-1 in 'jazzy/distribution.yaml' [bloom]

Increasing version of package(s) in repository mapviz to 2.4.3-1:

upstream repository: https://github.com/swri-robotics/mapviz.git
release repository: https://github.com/ros2-gbp/mapviz-release.git
distro file: jazzy/distribution.yaml
bloom version: 0.12.0
previous version for package: null

mapviz
* update launch params (#830 <https://github.com/swri-robotics/mapviz/issues/830>)
* Contributors: DangitBen

mapviz_interfaces

No changes

mapviz_plugins
* update plugin subscribers to use all of qos (#827 <https://github.com/swri-robotics/mapviz/issues/827>)
  Co-authored-by: Ben <mailto:benjamin.andrew@swri.org>
  Co-authored-by: David Anthony <mailto:djanthony@gmail.com>
* Added kludgy install for autogened TopicSelect header that is required if external plugin packages include topic_select.h (#825 <https://github.com/swri-robotics/mapviz/issues/825>)
* Contributors: DangitBen, Robert Brothers

multires_image

No changes

tile_map

No changes

Holding for Jazzy sync


text:
hri_face_body_matcher: 2.1.0-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository hri_face_body_matcher to 2.1.0-1:

upstream repository: https://github.com/ros4hri/hri_face_body_matcher.git
release repository: https://github.com/ros4hri/hri_face_body_matcher-release.git
distro file: humble/distribution.yaml
bloom version: 0.12.0
previous version for package: null

hri_face_body_matcher
* re-license as Apache 2.0
* Contributors: Séverin Lemaignan

Resubmission of #42871 after re-licensing as Apache 2.0


text:
Failing test in CI: test_rclcpp gtest_subscription__rmw_connextdds

System Info

OS

Ubuntu 22.04

ROS version and installation type

rolling/source

RTI Connext DDS version and installation type

Connext Pro 6.0.1

RMW version or commit hash

rolling branch (a6053be96c4cddf4cece95424a68fbb54373e705 as of this writing)

Bug Description
If you run the gtest_subscription__rmw_connextdds test from the test_rclcpp package with no load, it "always" passes.  It also "always" seems to pass in CI.  However, if you put load on the machine, it fails very often, maybe like 75% of the time.
Expected Behavior
The test always passes, even with a lot of load on the machine.
How to Reproduce
In terminal 1, put a lot of stress on the machine.  In my case:
$ stress --cpu 16

In terminal 2, run the test:
$ colcon test --event-handlers console_direct+ --packages-select test_rclcpp --ctest-args -R gtest_subscription__rmw_connextdds

You may have to adjust the amount of stress on the machine, and you may have to run the test a few times, but it should fail fairly quickly.
Workarounds
Mark this test as xfail.
Additional context
This same test works fine on Fast-DDS (gtest_subscription__rmw_fastrtps_cpp) and Cyclone DDS (gtest_subscription__rmw_cyclonedds_cpp), even with load on the machine.
This has come up now because we are about to merge in https://github.com/ros2/rclcpp/pull/2142, which seems to exacerbate this problem.  However, I can produce this completely with rolling packages as of today, so it is not the fault of that PR.
I did some additional debugging to try to track this down.  When it fails, the executor is waiting for new data to come in via rmw_wait via a condition variable: https://github.com/ros2/rmw_connextdds/blob/a6053be96c4cddf4cece95424a68fbb54373e705/rmw_connextdds_common/src/common/rmw_impl_waitset_std.cpp#L579 .  The failure occurs because the condition variable never wakes up and times out.  In turn, this suggests that the condition variable is never triggered.
That condition variable should be triggered when new data comes in via the on_data_available callback in the subscriber: https://github.com/ros2/rmw_connextdds/blob/a6053be96c4cddf4cece95424a68fbb54373e705/rmw_connextdds_common/src/common/rmw_impl_waitset_std.cpp#L131 .  I've verified that it is installed to the listener at https://github.com/ros2/rmw_connextdds/blob/a6053be96c4cddf4cece95424a68fbb54373e705/rmw_connextdds_common/src/common/rmw_impl_waitset_std.cpp#L630 .
Finally, I've verified that the publisher side is indeed writing the data out via DDS_DataWriter_write_untypedI in message (https://github.com/ros2/rmw_connextdds/blob/a6053be96c4cddf4cece95424a68fbb54373e705/rmw_connextdds_common/src/ndds/dds_api_ndds.cpp#L771) .
So as far as I can tell, those are all of the pieces necessary to get this working, and it does work sometimes.  But under load, it seems to fail.  I could use some advice on how to debug this further.
In the meantime, I'm going to propose a PR to mark that particular test as xfail so we can make progress on the other PRs.

i did not get to the root cause, https://github.com/ros2/rosbag2/blob/4882d30fc2d1e5b9305e5d46b8460466f9280d27/rosbag2_transport/test/rosbag2_transport/test_burst.cpp#L404-L408 looks like the same issue. when it comes to transient workload, it sometimes miss the subscription data, that leads to the test failure of that test here.

I think this issue is happening consistently on Windows repeated, and it's also flaky on windows debug and rhel repeated.
It's a timeout of this test.
Reference builds:

https://ci.ros2.org/job/nightly_win_rep/3453/ (consistent)
https://ci.ros2.org/job/nightly_linux-rhel_repeated/1999/ (flaky)

Test regression:

projectroot.gtest_multithreaded__rmw_connextdds

Log output:
  Start 12: gtest_multithreaded__rmw_connextdds

12: Test command: C:\Python38\python.exe "-u" "C:/ci/ws/install/share/ament_cmake_test/cmake/run_test.py" "C:/ci/ws/build/test_rclcpp/test_results/test_rclcpp/gtest_multithreaded__rmw_connextdds.gtest.xml" "--package-name" "test_rclcpp" "--output-file" "C:/ci/ws/build/test_rclcpp/ament_cmake_gtest/gtest_multithreaded__rmw_connextdds.txt" "--env" "RMW_IMPLEMENTATION=rmw_connextdds" "--append-env" "PATH=C:/ci/ws/build/test_rclcpp/Release" "--command" "C:/ci/ws/build/test_rclcpp/Release/gtest_multithreaded.exe" "--gtest_output=xml:C:/ci/ws/build/test_rclcpp/test_results/test_rclcpp/gtest_multithreaded__rmw_connextdds.gtest.xml"
12: Working Directory: C:/ci/ws/build/test_rclcpp
12: Test timeout computed to be: 90
12: -- run_test.py: extra environment variables:
12:  - RMW_IMPLEMENTATION=rmw_connextdds
12: -- run_test.py: extra environment variables to append:
12:  - PATH+=C:/ci/ws/build/test_rclcpp/Release
12: -- run_test.py: invoking following command in 'C:\ci\ws\build\test_rclcpp':
12:  - C:/ci/ws/build/test_rclcpp/Release/gtest_multithreaded.exe --gtest_output=xml:C:/ci/ws/build/test_rclcpp/test_results/test_rclcpp/gtest_multithreaded__rmw_connextdds.gtest.xml
12: Running main() from C:\ci\ws\install\src\gtest_vendor\src\gtest_main.cc
12: [==========] Running 5 tests from 1 test suite.
12: [----------] Global test environment set-up.
12: [----------] 5 tests from test_multithreaded
12: [ RUN      ] test_multithreaded.multi_consumer_single_producer
    Test #12: gtest_multithreaded__rmw_connextdds .............................***Timeout  90.01 sec

Flaky ratio:

job_name
last_fail
first_fail
build_count
failure_count
failure_percentage

nightly_win_rep
2024-08-21
2024-08-06
13
13
100.0

nightly_win_deb
2024-08-20
2024-08-20
14
1
7.14

nightly_linux-rhel_repeated
2024-08-18
2024-08-14
16
2
12.5

nightly_linux_coverage
2024-08-07
2024-08-07
15
1
6.67

WDYT @clalancette?

Reference build: https://ci.ros2.org/job/nightly_win_rep/3504/
Failing test: test_multithreaded.multi_access_publisher with ConnextDDS
Log output
[ RUN      ] test_multithreaded.multi_access_publisher
[WARN] [1728461640.564957000] [rclcpp]: logging was initialized more than once
Waited 0 microseconds for the subscriber to connect to topic 'multi_access_publisher'
Publishing message 0
Publishing message 1
Publishing message 2
Publishing message 3
Publishing message 4
Publishing message 5
Publishing message 6
Publishing message 7
Publishing message 8
Publishing message 9
Publishing message 10
Publishing message 11
Subscription callback 0
callback() 0 with message data 7
Publishing message 12
Subscription callback 1
callback() 1 with message data 8
Subscription callback 2
callback() 2 with message data 9
Publishing message 13
Subscription callback 3
callback() 3 with message data 10
Publishing message 14
Subscription callback 4
callback() 4 with message data 11
Subscription callback 5
callback() 5 with message data 12
Subscription callback 6
callback() 6 with message data 13
Publishing message 15
Subscription callback 7
callback() 7 with message data 14
Publishing message 16
Subscription callback 8
callback() 8 with message data 15
Publishing message 17
Subscription callback 9
callback() 9 with message data 16
Publishing message 18
Subscription callback 10
callback() 10 with message data 17
Subscription callback 11
callback() 11 with message data 18
Publishing message 19
Subscription callback 12
callback() 12 with message data 19
C:\ci\ws\src\ros2\system_tests\test_rclcpp\test\test_multithreaded.cpp(421): error: Expected equality of these values:
  num_messages
    Which is: 20
  subscription_counter
    Which is: 13

[  FAILED  ] test_multithreaded.multi_access_publisher (5521 ms)

I see this error is happening mostly on windows. These started failing a lot more since testing parallelization in Feb 25th. Check:

First windows debug failure this year: https://ci.ros2.org/view/nightly/job/nightly_win_deb/3014/
First windows repeated failure this year: https://ci.ros2.org/view/nightly/job/nightly_win_rep/3275/

Flaky ratio in the last 15 days:

job_name
last_fail
first_fail
build_count
failure_count
failure_percentage

nightly_win_rep
2024-10-09
2024-09-25
10
10
100.0

nightly_win_deb
2024-10-09
2024-09-27
14
7
50.0

nightly_linux_repeated
2024-10-05
2024-10-05
13
1
7.69

nightly_win_rel
2024-09-28
2024-09-28
13
1
7.69

Flaky ratio since paralelize testing (277 days in oct 09):

job_name
last_fail
first_fail
build_count
failure_count
failure_percentage

nightly_win_rep
2024-10-09
2024-02-25
229
167
72.93

nightly_win_deb
2024-10-09
2024-02-25
209
86
41.15

nightly_linux_repeated
2024-10-05
2024-04-09
246
7
2.85

nightly_win_rel
2024-09-28
2024-03-04
262
11
4.2

nightly_linux-rhel_repeated
2024-08-18
2024-03-15
265
7
2.64

nightly_linux_coverage
2024-08-07
2024-03-29
224
12
5.36

nightly_linux_debug
2024-04-07
2024-04-01
247
5
2.02

authors: Crola1702 clalancette fujitatomoya


text:
Add DDS_TIME_INVALID and DDS_Time_is_invalid for connext micro

For whatever reasons ConnextDDS micro exports DDS_TIME_INVALID but it doesn't define it. Therefore this results in a runtime error - because during runtime as the shared library is loaded, the symbol to DDS_TIME_INVALID can not be resolved.
Also DDS_Time_is_invalid is not defined. So both need to be added in order to make rmw_connextmicro work.

This PR is basically a direct follow up to #120. At the time I only checked whether rmw_connextmicro build successfully. Later I discuvered this issue with Connext Micro exporting symbols which it doesn't define.
@asorbini it would be great if you could have a look at this.

@asorbini I found neither for ConnextDDS Micro 3.0.3.28 nor for 2.4.13.4 any definition for DDS_TIME_INVALID.


text:
Fix pose selection when disabling display

When disabling a pose display, the selection remained active (wire box present without arrow or axes).
In addition, selection wireboxes acted weirdly for axes.
Both issues are fixed with this PR.

CI:

Linux
Linux-aarch64
macOS
Windows

CI after rebase:

Linux
Linux-aarch64
macOS
Windows

We'll close this PR as everything is contained in #315

authors: Martin-Idel-SI greimela-si


text:
Dehydration: Count field in reagent concentration should be removed

The count field in the Reagent Concentration dropdown should be removed.
For most protocols the count will be 1 with only a few exceptions, so it seems silly to add extra work for the user to set this to 1 most of the time and to something else infrequently when just having the entry in the first place functions to indicate a single entry.  For those times when the same thing is done 2x, the user can just input the second time as a separate entry.

I had a second thought about this - should the count section be updating to indicate the order? I noticed that you could add as many instances of this info as needed and that they could be re-ordered, but the count didn't update... Not sure if this is a bug or if it's an unnecessary field.
Worth a convo with Sriram I think before we do anything.

@MicheleCD Could you please explain what it means to update the count to indicate order? Don't understand whats the purpose of this field

@MicheleCD confirmed we can delerte this field

ready for review and merge on pull request #48

repo: rosalindfranklininstitute/empiar-spw


text:
Missing frame_id for the published attitude/euler message

No frame_id field has been set for the euler_msg (of type geometry_msgs::Vector3Stamped) that is published at line 439 in rosflight_io.cpp. I believe that coordinate frame information of the published data would be useful to other users of the node, makes code more readable.

Could be!  Feel free to submit a PR! :wink:

Could be!  Feel free to submit a PR! :wink:

authors: sayali-purdue superjax

repo: rosflight/rosflight


text:
Pretrained Discriminator

Thanks for your repo!
Can you add functions to convert the pretrained TF Discriminators?
E.g.
In convert_weight.py, you have
_, _, g_ema = pickle.load(f)
Can you update the conversion scripts to allow:
_, D, g_ema = pickle.load(f)

Added discriminator porting options in fd039ed.

It works -- thank you!
You may want to update your README from
python convert_weight.py --repo ~/stylegan2 stylegan2-ffhq-config-f.pkl
to
python convert_weight.py --repo ~/stylegan2 --gen [--disc] stylegan2-ffhq-config-f.pkl

authors: mbbrodie rosinality


text:
Move "sparse" argument to run_modal()

Following @raphaeltimbo idea of moving n_eigen from Rotor.__init__() to  Rotor.run_modal() (Issue #573), I'd like to propose the same to sparse argument. It's used only when running the modal analysis, solving the eigenvalues / eigenvectors. And as this argument is called only to define what numpy/scipy method to call when running the modal analysis, I think it makes sense to move sparse to run_modal().
What do you think?

Totally agree with that.

PR #588 closes this issue.

repo: ross-rotordynamics/ross


text:
🛑 Roundup-users mailing list is down

In 4d2ae00, Roundup-users mailing list (https://sourceforge.net/p/roundup/mailman/roundup-users/) was down:

HTTP code: 403
Response time: 72 ms

Resolved: Roundup-users mailing list is back up in 348fb4e.

repo: rouilj/RoundupAssets


text:
🛑 Fossil based IRC Log is down

In eafff99, Fossil based IRC Log (https://rouilj.dynamic-dns.net/fossil/roundup_irc_logs/doc/trunk/log/roundup/) was down:

HTTP code: 503
Response time: 600 ms

Resolved: Fossil based IRC Log is back up in a54b55a.


text:
Blank page when loading prod

The quick setup with the command :
make setup PROJECT_MODE=prod
Display a blank page, both on Windows WSL and Debian. I have fixed the makefile error for assets (see my other issue) and nothing is shown.
Could not find the entrypoints file from Webpack

This issue is caused by the following issue: https://github.com/royalphp/frankenphp-sylius/issues/2

authors: royalphp vasilvestre

repo: royalphp/frankenphp-sylius


text:
Incompatibility with pyside2 (napari bundle), due to outdated black import

Failed to import command at ‘napari_chatgpt._widget:OmegaQWidget’: cannot import name ‘FileMode’ from ‘black’
Originally posted:
forum.image.sc/t/failed-to-import-command-at-napari-chatgpt/99260
The napari conda bundle uses pyside2 as Qt back end.
The issue looks like an incompatibility between the specific black import and pyside2
With black, from black import FileMode fails in the bundle or napari env with pyside2
but works with pyqt5.
Here's the error:
from black import FileMode
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[1], line 1
----> 1 from black import FileMode

File ~/Dev/miniforge3/envs/napari-050ps2/lib/python3.9/site-packages/shiboken2/files.dir/shibokensupport/feature.py:139, in _import(name, *args, **kwargs)
    137     default = 0 if name.split(".")[0] == "PySide2" else -1
    138     pyside_feature_dict[importing_module] = default
--> 139 return original_import(name, *args, **kwargs)

File src/black/__init__.py:67

File src/black/linegen.py:73

File src/black/trans.py:251

TypeError: mypyc classes can't have a metaclass

That import looks outdated? see:
https://github.com/psf/black/commit/06f2790b5ca3fea45515e33c9660ad6265120a5a

Ok, working on a workaround, making the import of black local, should help. Will post an update here is the bug is confirmed gone.

authors: psobolewskiPhD royerloic

repo: royerlab/napari-chatgpt


text:
tracking parameters related to the weighting of the raw intensity values

Hello,
Very cool work!
When running the tracking (after we already have segmentations), I see that we are passing in the raw images themselves to the track function in ultrack/core/main.py. I understand that that is because the florescence intensity is used for the linking steps. But for cases where we are tracking nuclei from brightfield, where the raw images don't have as meaningful intensity gradients, Is there a way to change the weight of the importance of the raw intensity values that are extracted for the ILP? Some specific parameter in the tracking_config schema perhaps?
Thanks!

Hey @gouthamr321,
We only use fluorescence information for tracking if you provide the images parameter of the track function.
Otherwise, we use the cells' segments mask to compute the association between segments.
def track(
    config: MainConfig,
    *,
    labels,
    sigma,
    detection,
    edges,
    images,   # <- don't use this in your case.
    scale,
    vector_field,
    overwrite,
):

Feel free to drop other questions :)

Glad to hear @gouthamr321! I'll close the issue.

repo: royerlab/ultrack


text:
Implement RngCore for RingOscillator

Implement rand_core::RngCore for RingOscillator from the get_random_bit function. This is not suitable for security purposes so rand_core::CryptoRng has not been implemented.

Thanks for taking a look. I'd be happy to go either way. I don't have enough experience to really have an opinion on which is better. If you think it would be an improvement I can make the change when I get home later today.

repo: rp-rs/rp-hal


text:
Add french template

Add french language template.

I made some comments based on the community translation from aidedd.org. If you have an official translated book I think it trumps this source though.

I used the french Monster Manual as a source.
Here is an example:

I am going to close this since we already have a French translation on the develop branch. If you find errors with it or find that we need new functionality to truly handle a French translation, please add your findings to #308.

I am going to close this since we already have a French translation on the develop branch. If you find errors with it or find that we need new functionality to truly handle a French translation, please add your findings to #308.

authors: BrianCriswell notsag


text:
Text to speech added successfully.

Issue that this pull request solves
#271
Closes: # (issue number)
#271
Proposed changes
Brief description of what is fixed or changed
Using gTTs library I have implemented text to speech. The user can enter the text an audio file will be saved in wplay/audioFiles. The user can use this feature in chatting, message-schedule, and message blast. The user has to type command #_TTS then the user has to enter the text it wants to convert to audio file then a list of languages supported will be printed it can enter the code. There also many more language option available but I have added some common and all Indian languages. Then the audio file will be saved and automatically the file attachment option will open the user can select the file and it will be sent.
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Screenshots
Please attach the screenshots of the changes made in case of change in user interface

Other information
Any other information that is important to this pull request

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity increasing per file
==============================
- wplay/utils/io.py  1
- wplay/terminal_chat.py  1
- wplay/text_to_speech.py  2

Clones added
============
- wplay/save_chat.py  3
- wplay/terminal_chat.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- wplay/text_to_speech.py  2
- wplay/terminal_chat.py  1
- wplay/utils/io.py  1

See the complete overview on Codacy

@rpotter12 please review my PR.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- wplay/text_to_speech.py  2
- wplay/terminal_chat.py  1
- wplay/utils/io.py  1

See the complete overview on Codacy

@rpotter12 I have resolvded the conflicts please review my PR.


text:
SyntaxError: Only one default export allowed per module.

I had this error when run npm run watch.
`
ERROR in ./src/lib/interactive/makeInteractive.jsx
Module build failed: SyntaxError: Only one default export allowed per module.
204 | export default makeInteractive;
`

I created a PR with single line.

Thank you

authors: ZachLiuGIS rrag

repo: rrag/react-stockcharts


text:
🛑 Tu Empleo Ideal is down

In 5873b3d, Tu Empleo Ideal (http://www.vacantestei.com/home) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Tu Empleo Ideal is back up in 95d380f.


text:
added python homepage example

added python homepage example

LGTM thanks!

authors: jell-o-fishi viglucci

repo: rsocket/rsocket-website


text:
Support cargo install

It would be nice if I could use cargo install cargo-check.

This ability already exists. Maybe the README.md should be updated?

authors: Wilfred rsolomo

repo: rsolomo/cargo-check


text:
Update docs for raising errors with required params

I would like to update the documentation for raising errors with required parameters. I have run into an issue multiple times where I've tried to raise an error class, but the class requires args for instantiation, like
expect(double).to receive(:msg).and_raise(ErrorWhichRequiresArgs)

without realizing the issue. Ruby gives an error message about "wrong number of arguments", which has repeatedly led me (personally) down the wrong path of troubleshooting other issues.
If we added this example, I think it would help us avoid this issue (or fix it faster). For justification that I'm not the only one with the issue, this question on StackOverflow has an accepted answer with 15 points: https://stackoverflow.com/questions/40775978/rspec-wrong-number-of-arguments-when-raising-error
Thanks!

Thanks!

And thanks for reminding me @benoittgt

authors: JonRowe locofocos

repo: rspec/rspec-mocks


text:
Anonymous controller breaks ActionController::UrlGenerator on delete route

What Ruby, Rails and RSpec versions are you using?
Ruby version: 2.5.1
Rails version: 5.2
Rspec version: 3.9.0
Observed behaviour
When adding an anonymous controller to a controller test suite, it breaks the delete route. Removing the anonymous controller leaves everything else working fine.
Expected behaviour
I expect:
delete :destroy
To not throw:
ActionController::UrlGenerationError:
       No route matches {:action=>"destroy", :controller=>"sessions"}

When an anonymous controller is present.
 # Test layout rendering
  controller do
    def test_index
      render 'anonymous/index'
    end
  end

Can you provide an example app?

Running this test, the anonymous controller test passes.
The delete session tests fail with the error ActionController::UrlGenerationError: No route matches {:action=>"destroy", :controller=>"sessions"}

require 'rails_helper'

RSpec.describe SessionsController, type: :controller do
  # Resources
  let(:manager) { create(:manager) }

  # Test layout rendering
  controller do
    def test_index
      render 'anonymous/index'
    end
  end

  describe 'defaults to unauthenticated layout' do
    before do
      routes.draw { get 'test_index' => 'sessions#test_index' }
      get :test_index
    end

    it 'renders the unauthenticated layout' do
      expect(subject).to render_template('layouts/unauthenticated/base')
    end
  end

  describe "DELETE #destroy" do
    before do
      login(manager)

      delete :destroy
    end

    it 'redirects to the login path' do
      expect(response).to redirect_to(login_path)
    end

    it "returns http 302" do
      expect(response).to have_http_status(302)
    end
  end
end

If the anonymous controller is removed, the delete paths run as expected.
require 'rails_helper'

RSpec.describe SessionsController, type: :controller do
  # Resources
  let(:manager) { create(:manager) }

  describe "DELETE #destroy" do
    before do
      login(manager)

      delete :destroy
    end

    it 'redirects to the login path' do
      expect(response).to redirect_to(login_path)
    end

    it "returns http 302" do
      expect(response).to have_http_status(302)
    end
  end
end

What does your routing look like for SessionsController? If you read the documentation for the currently released version (which is 3.8 not 3.9) you'll see that routes are only created for standard resource routes, so what I think what is happening here is you are missing the correct route for delete :destroy as the "standard resource route" expects an id. You can fix your spec by supplying an id or drawing the route manually as you have for your test_index.

@JonRowe is this because we don't reset the routes after the routes.draw call?

@samphippen No I think it's because we use the default "multiple resources" REST routes e.g. DELETE /resource/:id and theres no ID.

ahh gotcha. I think this is probably a valid bug then. I'll see if I can work up a fix.

Having an anonymous controller breaks my routes also.
require 'rails_helper'

RSpec.describe Users::SessionsController, type: :controller do
  fixtures :users

  controller do
  end

  describe "GET #ask_email" do

    it "returns a success response" do
      get :ask_email, params: {}
      expect(response).to be_successful
    end

  end

end

Results in
  1) Users::SessionsController GET #ask_email returns a success response
     Failure/Error: get :ask_email, params: {}

     ActionController::UrlGenerationError:
       No route matches {:action=>"ask_email", :controller=>"users/sessions"}
     # /Users/cjbottaro/.gem/ruby/2.5.3/gems/devise-4.5.0/lib/devise/test/controller_helpers.rb:35:in `block in process'
     # /Users/cjbottaro/.gem/ruby/2.5.3/gems/devise-4.5.0/lib/devise/test/controller_helpers.rb:102:in `catch'
     # /Users/cjbottaro/.gem/ruby/2.5.3/gems/devise-4.5.0/lib/devise/test/controller_helpers.rb:102:in `_catch_warden'
     # /Users/cjbottaro/.gem/ruby/2.5.3/gems/devise-4.5.0/lib/devise/test/controller_helpers.rb:35:in `process'
     # ./spec/controllers/users/sessions_controller_spec.rb:14:in `block (3 levels) in <top (required)>'

Simply removing the controller block fixes it.

@cjbottaro in your case this is expected, controller creates a new controller which has the standard resource routes, thus as you haven't defined a route called ask_email for it you get the error, you need to draw the routes yourself.

authors: JonRowe cjbottaro samphippen sebscholl

repo: rspec/rspec-rails


text:
Schema Migrations Maven Plugin

A plugin that allows schema migrations with just one command

fixed in 0.2.0

repo: rssvihla/cassandra-commons


text:
Add tags to metadata

Related to rstudio/pins-r#570
Let's add the new tags option to the metadata in Python:
library(pins)
b <- board_rsconnect()
#> Connecting to RSC 2022.11.0 at <https://colorado.rstudio.com/rsc>
b %>% pin_meta("julia.silge/great-numbers")
#> List of 12
#>  $ file       : chr "great-numbers.rds"
#>  $ file_size  : 'fs_bytes' int 61
#>  $ pin_hash   : chr "4d1eb93dc2188114"
#>  $ type       : chr "rds"
#>  $ title      : chr "julia.silge/great-numbers: a pinned integer vector"
#>  $ description: NULL
#>  $ tags       : chr [1:2] "i-love-integers" "numbers-are-fun"
#>  $ created    : POSIXct[1:1], format: "2022-11-21 14:09:27"
#>  $ api_version: num 1
#>  $ user       : list()
#>  $ name       : chr "julia.silge/great-numbers"
#>  $ local      :List of 4
#>   ..$ dir       : 'fs_path' chr "~/Library/Caches/pins/rsc-e62371cfd77db754024f9c5ed3556a73/cc8ef31c-d6b3-4e0a-be8e-7992efffaee0/66084"
#>   ..$ url       : chr "https://colorado.posit.co/rsc/content/cc8ef31c-d6b3-4e0a-be8e-7992efffaee0/_rev66084/"
#>   ..$ version   : chr "66084"
#>   ..$ content_id: chr "cc8ef31c-d6b3-4e0a-be8e-7992efffaee0"

Created on 2022-12-05 with reprex v2.0.2
Once we have this implemented in R and Python, we can set tags = "vetiver" when we pin a vetiver model.

Any pin saved with tags in the metadata is currently unreadable in pins-python 😭. I'll add tags to the metadata today... (sorry, not sure how I missed this in the PR)

pins with a tags field should at least be readable now in v0.7.3

@juliasilge WDYT of us having R pins add tags metadata as either null or a list of strings? Currently, if you pin_write(..., tags="some_tag") it saves it as a string, rather than a list of strings.
AFAIK the result will be the same when reading meta into R (NULL or a vector of strings), so this is purely for convenience on the python side :) (which would be None, str, or list[str] right now)

tags implemented as of v0.7.3

authors: isabelizimm juliasilge machow

repo: rstudio/pins-python


text:
GCS: gcsf not included in Dockerfile

Describe the bug
When I prepare a Dockerfile with vetiver.prepare_docker for a Board in a google cloud storage bucket the package the vetiver_requirements.txt is missing the package gcsfs. Therefore the Run step is not successfull. Adding gcsfs manualy in the vetiver_requirements.txt resolves it.
To Reproduce
Prepare Script:
import vetiver
import pins
board = pins.board_gcs('deployed_model', allow_pickle_read=True)
vetiver.prepare_docker(board, "deployed_model_test", path="cloud_run/rest_api/app")
In console:
sudo docker build -t model.
sudo docker run model
Expected behavior
Include gcsfs in vetiver_requirements.txt after running vetiver.prepare_docker(board, "nk_churn_test", path="cloud_run/rest_api/app")
Screenshots

Desktop:

OS: Linux Ubuntu
Version 22

Hi there @MartinBaumga, thank you so much for this report!
It looks like this is happening since vetiver is unaware of the backend pins is connecting to, so it is not tracking the requirements necessary to authenticate.
Looking at the equivalent's of these packages in the R language, pins-r side has a required_pkgs method for boards, then vetiver-r aggregates those with the modeling requirements when people run vetiver_prepare_docker().
I'll open up an issue in pins-python, so we can coordinate how to fix this!

Fixed in #166, to be released in v0.2.1!

authors: MartinBaumga isabelizimm

repo: rstudio/vetiver-python


text:
Best free cloning app in french ? XTTS V2 ?

Hello, i'm french. i'm looking for a free cloning app ton integrate in  your webui with french voice.

Do you have an idea ?
is it possible to integrate XTTS coqui V2 ?
thanks

Hi, Bark has some benefits, but it's likely that you are looking for
tortoise cloning (XTTS is a variation of tortoise).
As for XTTS, coqui has decided to use a "strong" license that makes it
unusable for this project. Unless that has or will change, it will never be
in this project.
On Sun, Nov 19, 2023, 12:42 AM gandolfi974 @.***> wrote:

Hello, i'm french. i'm looking for a free cloning app ton integrate in
your webui with french voice.

Do you have an idea ?
is it possible to integrate XTTS coqui V2 ?
thanks

—
Reply to this email directly, view it on GitHub
https://github.com/rsxdalv/tts-generation-webui/issues/217, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTRXI4HQ6RJUCSOD6B2ICLYFDQPDAVCNFSM6AAAAAA7RCUBSGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDANBZGI4DSNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

thanks.

tortoise is the only free and good voice french cloning  for the moment ?
I use Pinokio on windows and there is a webui to use XTTS V2. I want the same for ubuntu in order to modify settings easily.
https://pinokio.computer/

thanks.

tortoise is the only free and good voice french cloning  for the moment ?
I use Pinokio on windows and there is a webui to use XTTS V2. I want the same for ubuntu in order to modify settings easily.
https://pinokio.computer/
https://www.youtube.com/watch?v=cASFP-p3sxI

Sorry for the late follow up, I'm still returning from vacation.
Tortoise is not the only one, XTTS is "free" but you are probably going to infringe on the non-commercial part (and that's part of the reason I will not include it in this project).
There's also bark and bark voice cloning. Bark is "good" in the sense that it makes a life-like voice. Bark is bad in that it often disregards the prompt.
For tortoise and other models (even fairseq), you can get an extra improvement by "upscaling" the output with an RVC model. Arguably that is the best reasonable TTS right now. It ought to be better than what I saw in the youtube video you linked.

thanks.

i'm french and XTTS is in french not bark.
do you have a link or tutorial for "upscaling" output with RVC model ?

thanks.

i'm french and XTTS is in french not bark.
do you have a link or tutorial for "upscaling" output with RVC model ?

bark purportedly supports French, but I understand that it's performance might not be good enough.
You can see tortoise + RVC search on youtube, there are multiple videos with different examples and depths.
https://www.youtube.com/results?search_query=tortoise+rvc

what do you think about this app ?  https://github.com/Mangio621/Mangio-RVC-Fork
Do you know a free app to synchronize cloning voice with face ? For the moment i use facefusion and i want to make the modified character speak with cloning voice

what do you think about this app ?  https://github.com/Mangio621/Mangio-RVC-Fork
Do you know a free app to synchronize cloning voice with face ? For the moment i use facefusion and i want to make the modified character speak with cloning voice

As for Mangio's fork - I think there were some good changes in it, but I do not follow what's been going on with it. They decided to avoid improving the main project due to language barrier; then the original RVC project got improvements, but Mangio was already too different and could not just "get" these improvements. Then they started asking other devs to help them. I can say that I've seen several RVC forks (I own a small fork to make it installable), and several groups that are working on their RVC. Myself, so far I am happy with the original RVC for this project, but you can install all of the RVCs, they certainly have their pros & cons.
The face sync, I have not done that, I'd recommend searching on youtube, and if you want to discuss it, several discord servers are quite advanced on this topic. Just as an example, camenduuru's server. There must be more but I don't know them. Some stable-diffusion themed AI servers have these discussions as well.

I think I'll close this issue. If there's more questions or discussion, please feel free to reopen.

thanks.

tortoise is the only free and good voice french cloning  for the moment ?
I use Pinokio on windows and there is a webui to use XTTS V2. I want the same for ubuntu in order to modify settings easily.
https://pinokio.computer/
https://www.youtube.com/watch?v=cASFP-p3sxI

Hey @gandolfi974
I'm currently working on a French voice cloning project and maybe I could use some help.
Kindly reply to this quote or just ping me an email directly. Thanks!

For reference, XTTS is now available as an extension; however the disclaimers about its license still remain. From what I know, within non-English tts space, xtts is one of the best.

thanks.

tortoise is the only free and good voice french cloning  for the moment ?
I use Pinokio on windows and there is a webui to use XTTS V2. I want the same for ubuntu in order to modify settings easily.
https://pinokio.computer/
https://www.youtube.com/watch?v=cASFP-p3sxI

Hey @gandolfi974 I'm currently working on a French voice cloning project and maybe I could use some help. Kindly reply to this quote or just ping me an email directly. Thanks!

hello, i have try https://github.com/daswer123/xtts-finetune-webui but the trained french voice with default argument is not enough similar

You can try applying RVC after that

thanks.

tortoise is the only free and good voice french cloning  for the moment ?
I use Pinokio on windows and there is a webui to use XTTS V2. I want the same for ubuntu in order to modify settings easily.
https://pinokio.computer/
https://www.youtube.com/watch?v=cASFP-p3sxI

Hey @gandolfi974 I'm currently working on a French voice cloning project and maybe I could use some help. Kindly reply to this quote or just ping me an email directly. Thanks!

hello, i have try https://github.com/daswer123/xtts-finetune-webui but the trained french voice with default argument is not enough similar

thanks,

how can i try RVC ? i can't load my XTTS finetune model in xtts-webui,  only RVC model. my personnal model  works only with "xtts-finetune-webui".
Is it possible to create RVC model with my TTS model ?

my personnal model works only with "xtts-finetune-webui".
that sounds like the result of "Slightly improved official version for finetune xtts" If the model is truly incompatible, the author should not be calling it xtts...
Either way, first you generate audio with whichever XTTS variation, then you use RVC conversion.
Is it possible to create RVC model with my TTS model ?
No, because it will sound just like the TTS model, you have to use the original source audio to get a better result than the XTTS.

You might want to join the discord.
If the model was compatible with other XTTS projects you could use it in my webui alongside RVC.

I have train new model.

how i can use my model with your XTTS projects ? do you have a tutorial ?
Same question to improve audio result with RVC
Do you recommand a project for training french TTS ? better than XTTS-FINETUNE-WEBUI ?

thanks

I have train new model.

how i can use my model with your XTTS projects ? do you have a tutorial ?
Same question to improve audio result with RVC
Do you recommand a project for training french TTS ? better than XTTS-FINETUNE-WEBUI ?

thanks

As far as I understand we have gone over these in Discord, but I will post them here too:

The XTTS plugins work normally without anything special (since this question was asked the plugins have been improved for easier UI)
RVC present in the WebUI is almost the same as other RVCs so any Youtube video should be an ok tutorial on the basics.
The problem with training a French TTS is the need for a base model, and I am not aware of good base models other than XTTS.

authors: fatima-falconit gandolfi974 rsxdalv

repo: rsxdalv/tts-generation-webui


text:
Adding a demo with Storage Access API

Description
One of the solutions to bypass the third-party cookie deprecation is using the Storage Access API; the demo can use the same flow as the analytics example when domain a and domain b are using the domain c to track the user. When the application where the application will check if the user can store/read a cookie before requesting the ID from the analytics application from domain C.
This demo should not interfere with the other demos and should have the option to revoke the permissions for storage access API.

This issue was closed on PR #48

repo: rtCamp/privacy-sandbox-demos


text:
Add client ID to Google Analytics requests

This almost reverts #4347 but is slightly different as it sends a client ID instead of a user ID.
Google's documentation says that the client ID should be a UUID but in my testing and in Google's own JavaScript implementation this works without that being the case.

Did we test this locally against GA, and see that it worked?

Yes I did.

Just to clarify, when I removed the user ID, I did so based solely on the documentation which doesn't appear to be correct.
With adding this client ID, I tested an event with a client ID, saw it in the real-time report, and waited a day and saw it in processed reports.


text:
Clarify

I made 3 suggested changes

.cargo/config -> .cargo/config.toml. Now extensions for vs code recognizes that it is a toml-file.
Moved a note about how to configure target in cargo.toml to by-example.md from app_init.md.
changed all occurrences of  .cargo/config  to .cargo/config.toml in the ru and eng version of the book.

bors retry

repo: rtic-rs/cortex-m-rtic


text:
NavLink activeClass doesn't work with hashIntegration

Active classes aren't applied when used like this in hashIntegration:
<NavLink activeClass="bg-primary text-light" class="btn btn-link ml-2" href="">
    Dashboard
</NavLink>
<NavLink activeClass="bg-primary text-light" class="btn btn-link" href="monitor">
    Monitor
</NavLink>
<NavLink activeClass="bg-primary text-light" class="btn btn-link" href="deployed">
    Deployed
</NavLink>
<NavLink activeClass="bg-primary text-light" class="btn btn-link" href="integrations">
    Integrations
</NavLink>

this is at the root of the application so clicking monitor goes to /monitor but doesn't highlight the NavLink. I think this is happening because I am hosting the app like localhost:8080/web-app#/monitor

Hmm, looks like the activeClass prop is always just being overwritten by the default class "is-active". It's just an oversight on the way I assign the default. I'll fix that.

Whoops, didn't mean to close with that PR merge. Anyways, should be fixed now with v0.2.2.

Also wanted to mention that in your example, you likely want to add the end prop to your dashboard NavLink so it only matches on the "/" path.

thanks, also thank you for publishing so fast

You're welcome, thanks for the PR. Keep the issues coming.

authors: ozgrakkurt rturnq

repo: rturnq/solid-router


text:
using fetchJSON instead of loadXML and upgrading release version

RMD-1972
Changes:
Change to use fetchJSON instead of loadXML in attempt to debug errors on next gen

Love it!

authors: VaneCGo ayoformayo

repo: rubiconmd/rubiconmd-js


text:
False positive in Naming/MemoizedInstanceVariableName due to conditionally assigned instance variable

When a method implicitly returns an instance variable that is conditionally assigned, RuboCop detects it as memoization, so it triggers a false positive for the cop.

Expected behavior
No offense should be raised.
Actual behavior
The implicitly returned instance variable is reported as a misnamed memoized variable.
Steps to reproduce the problem
Simple example:
class foo
  attr_accessor :problem_var

  def bar
    @problem_var ||= 'foobar'
  end
end

Where first observed: https://git.io/vp4Z7
RuboCop version
$ rubocop -V
0.55.0 (using Parser 2.5.1.0, running on ruby 2.5.1 x86_64-linux)

When a method implicitly returns an instance variable that is conditionally assigned, RuboCop detects it as memoization

Indeed. How could Rubocop differentiate your example from actual memoization?

Well, as for the production example I gave, it's been "fixed" because we realized that we didn't need the conditional assignment line at all, but of course, that doesn't solve the larger issue. Thanks for showing how the simple example could be solved, though. As for how RuboCop could differentiate, in all honesty, I'm not totally sure; I don't have a lot of experience with this sort of thing. I just wanted to make sure that a possible bug didn't go unreported. I'm certainly interested in and willing to learn more in order to help though!

I don't think this is a bug. When the code looks exactly the same as memoization from Rubocop's point of view, but the developer didn't intend it to be memoization, I think the only recourse is to disable the cop (either inline, or in the project's config file).
One way to make the code not look like memoization is to return some other value under the @ivar ||= something line.

Adding to this issue, I just ran into this case where we needed to look up the given value or fallback to a known safe default.
@thing   = Thing.find_by(name: params[:name])
@thing ||= Thing.find_by(name: "default")

In this case, if the call is overriding a variable is defined earlier in the method it should avoid triggering the cop.

Another example:
module AuditLog
  def self.setup
    @requests ||= {}
    @credentials ||= {}
  end
  ...
end

The presence of two @variable ||= ... patterns should be enough to exclude this cop, right?

@gaffneyc I can't reproduce an offense for that code. Perhaps it was resolved since you commented.
@thegorgon That example warrants its own issue, in my opinion.

Yeah, this probably isn't something that can be "fixed" on rubocop's end. I'm closing the issue.

@mikegee - opened https://github.com/rubocop-hq/rubocop/issues/6271

authors: ChallahuAkbar gaffneyc mikegee thegorgon

repo: rubocop-hq/rubocop


text:
Layout/SpaceAroundMethodCallOperator is slow

This cop inspects in O(n^2) instead of O(n) as it should (culprit is this loop) (where n is the order of send nodes / tokens)
As an example, about 10% of rake internal_investigation is currently spent in this cop alone. This would be way worse if the source files were not kept kind of small.
@saurabhmaurya15 would you like to look at this? Using tokens may have been a nice generic way of handling things but we can't afford it. We must use node.loc.

Note: one could use token(node) as a quick improvement, but ideally this could be done without tokens I think

@marcandre Can you help me out with how you did profiling on this? I can give it a shot then.

I need to create a rake task to do this properly, but I want to finish #7868 beforehand. In the meantime there's a rake task for testing a particular cop you could use. If you manage not to use tokens at all it would be a big improvement. Or wait a few days and there will be a rake task to measure the cops...

Also ran into the same issue, enabling this cop pushed rubocop run (cache disabled) from ~1 or 2 minutes to over 20 minutes on my codebase.


text:
Extract #assertions_count

The method is duplicated in 2 places.

Nice!

authors: fatkodima koic

repo: rubocop/rubocop-minitest


text:
Header is not displaying in Swagger UI

This Code is not displaying header field in swagger UI
desc "Return super-secret information", {
  headers: {
    "XAuthToken" => {
      description: "Valdates your identity",
      required: true
    },
   "XOptionalHeader" => {
     description: "Not really needed",
     required: false
   }
 }
}

Earlier It used to be like

But now it is showing no header field with above Code

How do I fix this issue.

would be cool to write a spec (similar to this one: api_swagger_v2_headers_spec.rb
so it could be reproduced

I am using version 0.20.1.

ok, I see …
the so defined header should go into the response object,
to define header parameters, take a parameter and set the param_type to header

It would be better if you could provide an alternative solution in form of some patch. Sorry but I think modifying 100s of apis to add just one extra parameter does not look a good approach. Can I accept some sort of patch in near future from you

@brahamshakti What exactly are you proposing? What would you like to write in your API? We'll accept patches from you, thanks.

doing from the referenced test case
        params do
          optional :in_query, type: String, documentation: { param_type: 'query' }
          optional :in_header, type: String, documentation: { param_type: 'header' }
        end

does not really work.  It will make valid swagger spec, but the grape code is looking for the in_header parameter in the params list not in the header.  If it were a required field it would fail if you actually put the parameter in the header.

but the question is always, how to differ between stuff for request and stuff for response for documentation …
what are your thoughts?

@brahamshakti give it a try, and close the issue

Thanks to @wleeper, my issue is resolved with this pull request. I am closing this issue for now. Thanks to all.

authors: LeFnord brahamshakti dblock wleeper

repo: ruby-grape/grape-swagger


text:
What should rescue_from block return ?

The documentation says that:
The rescue_from block must return a Rack::Response object, call error! or re-raise an exception.
but if we use the rescue_from method only to log things and we want to keep the original HTTP answer what should we return ?
If I rethrow the catched exception, it does not keep the original behavior.
For example, some of my tests generate 400 responses but with:
  rescue_from :all do |e|
    Rails.logger.error(e)
    raise e
  end

They catch 500 responses.
The better solution I found to reproduce the original behavior is this code:
  rescue_from :all do |e|
    Rails.logger.error(e)
    if e.respond_to?(:status)
      error!(e.to_json, e.status, e.headers, e.backtrace)
    else
      error! "Internal Server Error"
    end
  end

I should check if the catched exception responds to status because it could be a RuntimeError for example, and RuntimeError does not answer to status.
Am I doing this right ? Or there is a better solution to do that ?

I also posted the question here : http://stackoverflow.com/questions/35718873/what-should-return-the-grape-rescue-from-all-block

Check out this conversation #939

Just for reference, the solution in #939 that @railsmith points is right, catch only things you want to catch, ie.Grape::Exceptions::Validation and Grape::Exceptions::ValidationErrors. But generally for a logger I suggest using a middleware instead.
I'm going to close this, lmk if it needs any other action.

authors: dblock guizmaii railsmith

repo: ruby-grape/grape


text:
HTTP GET request with parameters in body and Content-Type header

We're trying to replace our custom json-parser middleware in our project with grape formatter middleware recently,
and found that if we send a GET request with parameters in body and with application/json Content-Type, parameters will disappear.
It looks like grape doesn't accept this kind of request
https://github.com/ruby-grape/grape/blob/f5d9831bac2e2dd439d0f3901797995a91139690/lib/grape/middleware/formatter.rb#L78-L81
I'm not sure if setting Content-Type HTTP header in GET request is a good practice or not.
Could grape support this just like it do for DELETE in https://github.com/ruby-grape/grape/pull/448? Or add this kind of support into config, so that we can do our own configuration?

Do the RFCs say that you can/cannot have a body on a GET with a content-type? I imagine we'd want to stick to whatever that says.
I don't see a problem parsing a body on a GET, but this is a pretty major breaking change, so it would need to be done carefully, major version upgrade & al. Something configurable (list of methods allowing a body?) may be a good way to start.


text:
specs for declared(params) inside route_param bug.

I check some specs on my project and I find out that this functionality work little bit wrong with declared when we have several route_param in one namespace.
For example we have:
class Artists
  namespace :artists do
    route_param :id, type: Integer do
      get do
        declared(params)
      end

      params do
        requires :filter, type: String
      end
      get :some_route do
        declared(params)
      end
    end
    route_param :artist_id, type: Integer do
      mount Compositions
    end
  end
end

declared(params) should return us next hash {id: 1}, but it return hash {id: 1, artist_id: nil}.
Also in first route_param in second endpoint(get :some_route) when we call declared(params) we have only filter params in returned hash {filter: 'some_filter'}
P.S.: I find out where we have problem! Problem is in options for route_param. where I use route_param :id without type: Integer everything is ok.

@dblock @namusyaka please review this.

Well these specs pass... weren't they supposed to fail?

@dblock It's a regression specs. They represent a problem. Or you want to see a regular specs instead of regression specs?

@dblock change specs. not it failed and show problem correct.

Hi guys. Any news about this issue?

@dblock @namusyaka  Please review my code. I find out where was a problem and fixed it. I really need this fix in my project so if everything ok can you please merge it ASAP.

Looks good to me.
@dblock Any thoughts on this?

Looks good, needs a CHANGELOG entry please.

@dblock Done.

@dblock update.

Excellent, merging.

authors: Arkanain dblock namusyaka


text:
Correctly deserialize non-UTF-8 strings for JavaScript

The Problem
In Ruby, it is possible to create a literal string containing an invalid byte sequence:
str = "\xFF\xFF\xFF"
str.encoding  # => #<Encoding:UTF-8>
str.codepoints  # => invalid byte sequence in UTF-8 (ArgumentError)

This happens on my machine because the default encoding is set to UTF-8:
Encoding.default_external  # => #<Encoding:UTF-8>

When Prism parses this code, it uses a TextDecoder to convert the byte sequence to a JavaScript string. For the example above, the decoder produces the following result:
const buf = new Uint8Array([255, 255, 255]);
new TextDecoder().decode(buf).codePointAt(0);  // => 65533

65533 is the Unicode replacement character U+FFFD (instances of TextDecoder assume UTF-8 by default). Since the byte sequence above isn't valid UTF-8, the decoder replaces each of the bytes with the Unicode replacement character and returns as if nothing went wrong.
Potential Solution
Rather than produce a string containing Unicode replacement characters, I propose Prism fall back to ASCII decoding. This way, the string produced by Prism will be the same as the string produced by Ruby - a sequence of bytes represented in ASCII encoding.
const buf = new Uint8Array([255, 255, 255]);
new TextDecoder("ascii").decode(buf).codePointAt(0);  // => 255

Prism can fall back only in the case that UTF-8 decoding fails, eg. by catching the error thrown by TextDecoder.new("utf-8", {fatal: true}).

I suppose we could use a combination of both strategies. We can't use ASCII for any/all non-UTF-8 encodings because Ruby string literals can include both valid UTF-8 sequences and arbitrary bytes, eg. "\xFF鹿\xFF". We will still need the approach in this PR to handle decoding those kinds of strings embedded in UTF-8-encoded files.
The idea behind this PR was to essentially encode a series of bytes in UTF-16, but I think we're in a bit of a tough spot here. There's currently no way for the consumer of the parse tree to know when they're receiving a "normal" string containing a well-formed sequence of UTF-16 characters versus a string that only looks like UTF-16 but actually consists of a sequence of packed bytes. It's even more confusing if we fall back to packing when encountering an invalid byte sequence in a UTF-8 string (i.e. the proposal in this PR).
In light of that, I think we probably want the following:

Modify the parser to set a flag when encountering arbitrary byte sequences in string literals, eg. \xFF.
When deserializing, check the value of the flag. If the string contains arbitrary byte sequences, use the ASCII decoder. Otherwise use the UTF-8 decoder.

So these flags should already be there, depending on the combination of the byte sequences and the encoding of the file. For example:
# encoding: ascii
"\xFF"

is going to give you:
@ ProgramNode (location: (2,0)-(2,6))
├── locals: []
└── statements:
    @ StatementsNode (location: (2,0)-(2,6))
    └── body: (length: 1)
        └── @ StringNode (location: (2,0)-(2,6))
            ├── flags: forced_binary_encoding
            ├── opening_loc: (2,0)-(2,1) = "\""
            ├── content_loc: (2,1)-(2,5) = "\\xFF"
            ├── closing_loc: (2,5)-(2,6) = "\""
            └── unescaped: "\xFF"

and:
# encoding: ascii
"\u{80}"

will give you:
@ ProgramNode (location: (2,0)-(2,8))
├── locals: []
└── statements:
    @ StatementsNode (location: (2,0)-(2,8))
    └── body: (length: 1)
        └── @ StringNode (location: (2,0)-(2,8))
            ├── flags: forced_utf8_encoding
            ├── opening_loc: (2,0)-(2,1) = "\""
            ├── content_loc: (2,1)-(2,7) = "\\u{80}"
            ├── closing_loc: (2,7)-(2,8) = "\""
            └── unescaped: "\xC2\x80"

so in theory you could look at the flags on the string to know how to decode.

Ah yeah, I saw those flags while looking into this, but they didn't immediately make sense to me. I just wrote a little script that tries various combinations of strings and file encodings to help me visualize what they mean. Maybe the table will be helpful to others who run across this PR:

Encoding
String
Forced UTF-8
Forced binary

UTF-8
\xFF\xFF\xFF
true
false

UTF-8
\xFF鹿\xFF
true
false

UTF-8
鹿
false
false

UTF-8
\u{9e7f}
true
false

ascii
\xFF\xFF\xFF
false
true

ascii
\xFF鹿\xFF
false
true

ascii
鹿
false
false

ascii
\u{9e7f}
true
false

Things to note:

Strings in UTF-8-encoded files are never forced binary.
Strings in UTF-8-encoded files are forced UTF-8 if they contain bytes via eg. \x syntax or \u syntax.
Strings in ASCII-encoded files are only forced binary if the string contains bytes via eg. \x syntax.
Strings in ASCII-encoded files are only forced UTF-8 if the \u syntax is used.

I'm not sure there's enough information to correctly choose the right TextDecoder. We can use the ASCII decoder if either forced flag is set. Unfortunately "\u{9e7f}" is forced UTF-8 in both UTF-8- and ASCII-encoded files despite containing a valid UTF-8 byte sequence - we should use the UTF-8 decoder in that case.
Does that make sense or am I totally missing something?

That's almost correct.

Strings in UTF-8-encoded files are never forced binary.

This is correct. Regardless of bytes, UTF-8-encoded files always contain UTF-8-encoded strings. Files can contain syntax errors if they have invalid bytes in UTF-8 however, as in:
irb(main):001> eval(%Q{"\xFF"})
(irb):1:in `eval': (eval at (irb):1):1: invalid multibyte char (UTF-8) (SyntaxError)

Strings in UTF-8-encoded files are forced UTF-8 if they contain bytes via eg. \x syntax or \u syntax.

This is true. Strings in Ruby can be "broken" and contain invalid bytes. That means they'll respond false to String#valid_encoding?. In the context of prism, if a string is forced_utf8_encoding then it means there was an escape sequence that locked in the encoding. This can happen in other files as well, for example if you have:
# encoding: Shift_JIS
"\u{80}"

then the string will be UTF-8 encoded, not Shift_JIS encoded.

Strings in ASCII-encoded files are only forced binary if the string contains bytes via eg. \x syntax.

This is true if you're talking about valid files only. Invalid files can have an error attached but have strings with forced_binary_encoding if they have anything >= 0x80 in them.

Strings in ASCII-encoded files are only forced UTF-8 if the \u syntax is used.

This is true.
Given these constraints, maybe it's best to do something like you said, as in:
if flags & forced_binary_encoding
  just use raw bytes
else
  encoding = flags & forced_utf8_encoding ? utf-8 : file encoding
  try { decode with encoding } catch { just use raw bytes, capture what the encoding should be, set flag saying valid_encoding? is false }
end

also it appears that more encodings than I thought are supported by browsers according to https://encoding.spec.whatwg.org/#names-and-labels. You might be able to get support for most of the encodings that prism/Ruby supports through it.

We might even be able to support other encodings with x-user-defined if we're clever.

Ok awesome, that makes senes to me 👍 I'll try to update this PR with that algorithm.

also it appears that more encodings than I thought are supported by browsers

Oh yeah, I noticed that too. I think all these strings can be passed to TextDecoder's constructor.

Alright, I'm going to close this PR and open a new one with an updated description. I'd rather do that than edit this one, so our conversation can be preserved.

authors: camertron kddnewton

repo: ruby/prism


text:
Switches from facebookgo/clock to benbjohnson/clock

This PR switches from the facebookgo/clock dependency to benbjohnson/clock which is the more actively maintained drop-in replacement.
It also initializes the module using go mody init.

Changing to github.com/benbjohnson/clock causes tests to fail, it seems that function implementations are also different 🤔

authors: FilWisher arshamalh

repo: rubyist/circuitbreaker