meteor don't works with latest datasets

datasets dropped python 3.7 support, so this line won't work anymore:
https://github.com/huggingface/evaluate/blob/af3c30561d840b83e54fc5f7150ea58046d6af69/metrics/meteor/meteor.py#L18

Faced the same issue and solved with pip install datasets==2.13.1.
Or probably, the evaluate library should replace the import of importlib_metadata with import importlib.metadata as importlib_metadata

I am still getting the same error. I can use other metrics like bleu and rouge. But when i tried to use Meteor. This was the error : ImportError: cannot import name 'importlib_metadata' from 'datasets.config'. I am not sure what is the issue?

Need to do a new release - can you install evaluate from source in the meantime?

Also getting the below error while trying evaluate.load('meteor') with python 3.8 and 3.9:
ImportError: cannot import name 'importlib_metadata' from 'datasets.config' (C:\Users\agemawat\Anaconda3\envs\raitext\lib\site-packages\datasets\config.py)

Me too. This crashed my training code and I couldn't use meteor anymore. Here's a Colab reproduction: https://colab.research.google.com/drive/1JXVodEx5VRWA2GYzjk__FuyOIPauz6Ug?usp=sharing
And a traceback:
ImportError                               Traceback (most recent call last)
[<ipython-input-2-7fac65b59c62>](https://localhost:8080/#) in <cell line: 2>()
      1 import evaluate
----> 2 evaluate.load("meteor")

9 frames
[~/.cache/huggingface/modules/evaluate_modules/metrics/evaluate-metric--meteor/90426c9b4a2cf0bf10c6718aa8de221c7a50f46a694f4cc37bb3ee12fc9aa8c9/meteor.py](https://localhost:8080/#) in <module>
     16 import datasets
     17 import numpy as np
---> 18 from datasets.config import importlib_metadata, version
     19 from nltk.translate import meteor_score
     20

ImportError: cannot import name 'importlib_metadata' from 'datasets.config' (/usr/local/lib/python3.10/dist-packages/datasets/config.py)

I can replicate this issue too

Also get this error too

Can you try to install evaluate from main?

Faced the same issue and solved with pip install datasets==2.13.1. Or probably, the evaluate library should replace the import of importlib_metadata with import importlib.metadata as importlib_metadata

Fixed my issue with import importlib.metadata as importlib_metadata. Thanks!

Faced the same issue and solved with pip install datasets==2.13.1. Or probably, the evaluate library should replace the import of importlib_metadata with import importlib.metadata as importlib_metadata

Fixed my issue with import importlib.metadata as importlib_metadata. Thanks!

Could  you please explain how fixed the issue this way? I installed importlib-metadata in my conda but still I cannot use the evaluate library with METEOR. Thanks

We just released a new version of evaluate, you can update evaluate it will work with recent datasets versions
pip install -U evaluate

@lhoestq , I still get the error with datasets==2.14.5 and evaluate==0.4.1. However, downgrading to datasets==2.13.1 resolves it.

pip install -U evaluate
resolved it for me!
Thanks :)


Further improvements of input handlings

This PR has improved input handling, as additional works which are untouched in #296.
The following changes have been made:

input_handler::test_next_char() function has been replaced with input_handler::peek_next() function to more generally access the next input character without moving the current position.
Preparation of buffers to be filled with input characters have been improved by reducing the number of calls during the initialization process of the lexer for optimization purposes.
String manipulation in the lexer has been optimized.

Note that the public APIs are not affected by the above changes.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3049

Relevant Lines:
3049

💛  - Coveralls


Please add a license file

I'd like to add this to awesome-zsh-plugins, but it doesn't have a license file.
Please add a license file - if you don't have one in mind,  I recommend using choosealicense.com to help choose an appropriate one.
Thanks!

Thanks!  The MIT license should work just fine right?


Lambda to Firehose: Part 2

to: @chunyong-lin  @ryandeivert
cc: @airbnb/streamalert-maintainers
size: medium
Background
The Part 1 changes (#280) introduced the CLI and Terraform code to enable creation of a Kinesis Firehose Delivery Stream per StreamAlert log type.
Part 2 focuses on the application logic from the Lambda functions to PutRecords into the delivery stream.
The desired flow is demonstrated below:

Changes
Infrastructure

Attach a new policy to the rule_processor allowing write only privileges to all streamalert_data_* Firehose Delivery Streams.

Application

Load Firehose settings from conf/global.json
Send batches of classified payloads to their respective Firehose Delivery Stream.  For example, if a record is classified as the type cloudtrail:events, it will be sent to the streamalert_data_cloudtrail_events Delivery Stream.
Log metrics to understand Firehose send failures and throughput.
Add unit tests to verify the functionality above.

Other

@chunyong-lin PTAL at https://github.com/airbnb/streamalert/commit/533bfbecfac556610a117496211897e0386abb7c
Bump the library version to 1.5.0.
Fix two regressions: Removal of the Kinesis user/terraform outputs, and duplicate tf generate.

Coverage decreased (-1.4%) to 93.784% when pulling b8581bbe51efad521e96c56a7fae84226df2481f on jacknaglieri-lambda-to-firehose-pt-2 into 06f72f7620ed1cb9e63102d0ce19eab93a59ad9e on master.

Coverage decreased (-1.04%) to 94.194% when pulling 3a70ead2c5a511d6786af79e3ff80d441915d52e on jacknaglieri-lambda-to-firehose-pt-2 into 06f72f7620ed1cb9e63102d0ce19eab93a59ad9e on master.

note: the decreased coverage is due to a flakey unit test that has been disabled

@chunyong-lin just added some clarifying commits, PTAL

Coverage decreased (-1.04%) to 94.194% when pulling 175eecf54b0ff84a2ec6a1b164c4452d9731383d on jacknaglieri-lambda-to-firehose-pt-2 into 06f72f7620ed1cb9e63102d0ce19eab93a59ad9e on master.

I have additional changes to be made, but I'll leave it at a Part 3 instead of appending more commits here:

[ ] Allow only specific logs to be sent to Firehose (either in the form of inclusions/exclusions)
[ ]  Sanitize special characters in key names
[ ] Size check the entire payload before sending

🚢

Coverage decreased (-1.03%) to 94.198% when pulling c22d48c3534e432d264f2c9be52b6479422cbcac on jacknaglieri-lambda-to-firehose-pt-2 into 06f72f7620ed1cb9e63102d0ce19eab93a59ad9e on master.

Coverage decreased (-1.03%) to 94.198% when pulling b4a4b49fda36124f44de87507ad19b0340e94771 on jacknaglieri-lambda-to-firehose-pt-2 into 06f72f7620ed1cb9e63102d0ce19eab93a59ad9e on master.


Feature: allow all optional props for the NodejsFunction CDK construct to be passed via defineFunction in @aws-amplify/backend-function

Problem
defineFunction does not allow all props available on NodejsFunction
Issue number, if available: #1968
Changes

Allow passing optional props to NodejsFunction via defineFunction
Add test to allow passing retryAttempts prop to NodejsFunction via defineFunction

Validation
I created a unit test that validates the presence of AWS::Lambda::EventInvokeConfig with MaximumRetryAttempts in the generated CloudFormation when retryAttempts is passed as a prop to defineFunction
Checklist

[x] If this PR includes a functional change to the runtime behavior of the code, I have added or updated automated test coverage for this change.
[ ] If this PR requires a change to the Project Architecture README, I have included that update in this PR.
[ ] If this PR requires a docs update, I have linked to that docs PR above.
[ ] If this PR modifies E2E tests, makes changes to resource provisioning, or makes SDK calls, I have run the PR checks with the run-e2e label set.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hey @ataylorme :wave: thanks for taking the time to file this and your interest in contributing! A few of these properties will require values that rely on CDK scope like VPC configuration, or enums that must be imported from CDK. Are there any particular properties you're interested in?

@josefaidt the ones I am personally interested in are the log related options (systemLogLevelV2, logRetention, logGroup, etc.), maxEventAge and retryAttempts.
Other users will have other needs, which is why I made the PR to support all options.

Hey @ataylorme thanks for the clarification! Coincidentally enough we have a PR filed for exposing function logging props https://github.com/aws-amplify/amplify-backend/pull/2245
We'll work to evaluate exposing other NodejsFunctionProps in the meantime 🙂

Oh nice! I have been manually provisioning a log group in CDK with retention settings to match the Lambda names. That PR will make it much simpler.


Upgrade to Swift 3.0

This pull request upgrades the whole project, including unit tests, to Swift 3.0.
The majority of the conversion was done by the Swift migrator in Xcode.
In my quick test runs, everything seems to work and behave as before.
I've also added some @discardableResult annotations where I felt confident that they were appropriate.
Please note that currently there are 6 failures when running the unit tests. These are not new in this Swift 3.0 upgrade, these are present in the current, Swift 2.2, version.

@lrtitze Can you take a look please?

Looking now. Sorry about the delay, I was mired in other work.

@LouisDhauwe It all looks good! Thanks so much for your hard work!

Merged and credited. Good work.  :-)


Restore command doesn't recognize when restore job is completed

Describe the bug
ZITADEL operator is stuck on waiting for restore job to complete
To Reproduce
zitadelctl restore...


Calico/node: auto-detect hostname if not provided

It would be useful if one didn't need to pass an explicit hostname to calico/node, and it instead defaulted to the value of socket.gethostname().
Currently, there is no default: https://github.com/projectcalico/calico-containers/blob/master/calico_node/filesystem/startup.py#L306
My use case is running in a DaemonSet under Kubernetes, where the hostname is not readily available through the Downward API.

Fixed by #1085


Problem with setting values in store when using Composition API setup.

Reproduction
https://stackblitz.com/edit/nuxt-starter-efm7cs?file=app.vue
Steps to reproduce the bug

Use

It's increment on the server and then on the clientt. That's why you see 2.


updating to v<IP_ADDRESS>1 give ssl protocol error on OWA

Hello,
I have an issue on my exchange with the version latest version of win-acme v<IP_ADDRESS>1
I ran this command:
import the new certificate on IIS
It produced this output:
ssl protocol error in the OWA when I connect to my exchange on any browser
My web server is (include version):
Exchange 2016
The operating system my web server runs on is (include version):
Windows 2016
My hosting provider, if applicable, is:
free
I can login to a root shell on my machine (yes or no, or I don't know):
yes
The version of my client is (e.g. output of certbot --version or certbot-auto --version if you're using Certbot):
mycertificate produced by the version of v<IP_ADDRESS>1 is providing this error when I connect , if I do a roll back on v<IP_ADDRESS>1 it is working perfectly
any suggestions please ?
am I the only one ?
thanks in advance

Could you share the log and the exact error that you're seeing?

thanks for your question,, I had :
ERR_SSL_PROTOCOL_ERROR
prompting on chrome / firefox

That sounds like the private key might be missing. I have no idea how that could happen though. Could you share the log output generated by your renewal with version 2.2.9?


Separate functions for downloading pre-processed and datasets and downloading & processing

I think it would be a good idea, where possible, to have separate functions in the pile for downloading a pre-processed version of the dataset from a hosted version (i.e a single wget) and running the entire replication step.
The Stackexchange dataset, for example, requires a large amount of storage for processing, and is pretty slow. It's good that people are able to recreate the pipeline if possible (and the stackexchange data on archive.org is being updated fairly regularly, so will grow over time), but in general, it would be better to host the data somewhere.
There are also some functions that only provide download, and no replication steps. I think these two things should be separated where possible.
So, we should first provide a hosted dataset, then fall back to full downloading and pre-processing, if, say, we can no longer host.

I think that providing the replication code and a way to directly download the processed data sets separately is a great idea. I would go as far as to say that it's necessary for usability.


FIX Tokyo Olympic

"海の日", "体育の日" and "山の日" will move for the opening ceremony and closing ceremony of Tokyo Olympic. This bill was passed.
see: https://www3.nhk.or.jp/news/html/20180613/k10011476131000.html

Sorry, I'll fix test

I noticed it now, should I wait for this update?
https://github.com/holiday-jp/holiday_jp

@Peranikov Thank you for your commit !!
https://github.com/holiday-jp/holiday_jp updated !!

@k1LoW thank you for your information! by the way, Are you planning to fix this?

具体的には、「海の日」を7月23日（開会式前日）、「体育の日」を7月24日（開会式）、「山の日」を8月10日（閉会式翌日）に持ってくるとしている。
http://www.itmedia.co.jp/business/articles/1806/13/news116.html

https://github.com/holiday-jp/holiday_jp-ruby/blob/master/holidays.yml#L785

Oh sorry, I checked fixed it.
https://github.com/holiday-jp/holiday_jp/blob/master/holidays.yml#L787

@Peranikov
Could use https://github.com/holiday-jp/holiday_jp/releases/tag/v0.10.0 ?
Because holiday_jp-ruby does not yet support v1.x.
https://github.com/holiday-jp/holiday_jp#implemantations

I tried bundle exec rake with master, it seems to have failed.
Failure: test_yaml(TestHolidayJpYaml)
/projects/holiday_jp-ruby/test/test_holiday_jp_yaml.rb:10:in `block in test_yaml'
      7:     yaml = YAML.load_file(File.expand_path('../../holidays.yml', __FILE__))
      8:     holiday_jp_yaml = YAML.load_file(File.expand_path('../holiday_jp/holidays.yml', __FILE__))
      9:     yaml.map do |date, name|
  => 10:       assert_equal holiday_jp_yaml[date], name
     11:     end
     12:     holiday_jp_yaml.map do |date, name|
     13:       assert_equal yaml[date], name
/projects/holiday_jp-ruby/test/test_holiday_jp_yaml.rb:9:in `each'
/projects/holiday_jp-ruby/test/test_holiday_jp_yaml.rb:9:in `map'
/projects/holiday_jp-ruby/test/test_holiday_jp_yaml.rb:9:in `test_yaml'
<"天皇誕生日 振替休日"> expected but was
<"振替休日">

diff:
? 天皇誕生日 振替休日

I tried bundle exec rake with master, it seems to have failed.

Yes. Because holiday-jp/holiday_jp master branch holidays.yml is 1.x. holiday_jp-ruby does not yet support v1.x.
Could you use submodule holiday_jp v0.x ?

@k1LoW Thank you, I understood! and I've fixed it.
Please review this PR.

@k1LoW Thank you for your support!

🎉


Feature/request params

Adding support for RequestParams.  For specific details see the Ex Libris Brief Search documentation.
The main driver for my projects was perform a search against the Primo API that would simulate a user doing a search with the the "Expand My Results" checked in the Primo Web UI.  This can be achieved by adding the following XML to a search request:
<RequestParams>
  <RequestParam key="pc_availability_ind">
     true
  </RequestParam>
</RequestParams>

Coverage increased (+1.9%) to 99.693% when pulling 4208a45731c260b6147c6fe3e535e62ab62f925c on michael-harrison:feature/request_params into 1824acc7c1e630a2ab8b8a7d787d3c15c0491db4 on NYULibraries:master.

Hi @barnabyalter just a quick FYI.  We've been using this in production for the last couple of months.


Merge SplitView_themeresources.xaml files

Description

Previous duplicates of SplitView_themeresources.xaml files were causing weird NavView template issues. Files are now merged and duplicate resources are removed from deprecated_themeresources_any.xaml.
Motivation and Context

SplitView visual setters are set properly now and fixes #5342

/azp run


dicts/de/data.go:1:1: expected 'package', found version

Hello
I am using golem for a while now but after my last update i can not make it work anymore.
i get "dicts/de/data.go:1:1: expected 'package', found version" for every call i want to make
is there a preparation step i am missing, also the test give me the same output

Same issue. Replacing dicts/de/data.go file from this repository manually helped me.

First off: Apologies for my slow response.
I think the problem might be due to that we started using git lfs, but forgot to document it anywhere. Do you have LFS installed?

No worries, thx for the hint. i will check that in the next days

Sorry for bumping an old issue, but I can't get this working either.
I installed git-lfs as per the above instructions, to no avail. I have tried with the following installation commands:
$ git lfs install
$ git lfs install --skip-repo
$ sudo git lfs install --system

Removing the go mod downloaded file each time, prior to go get -u (the overkill to cover off any weird go get cases).
I also re-ran the above steps with git-lfs from pacman -Sy git-lfs with the same result as OP.

Thanks for the comment @jspc
I'll look into what the problem might be, if it's not git-lfs related.

Running git lfs pull in the repo solved the issue for me. There should be triggers for this in the config.
Could you give it a try?

Should be fixed in v3.0.2

Hello @aaaton , Still getting same error, any help will be appreciated

Does this link help?

Hi, @aaaton
Can't make it work:
$ git lfs install --skip-repo
Git LFS initialized.
$ go run main.go
go: downloading github.com/aaaton/golem v0.0.0-20191129093449-a9f1a1b6b185
main.go:12:2:
../../../../pkg/mod/github.com/aaaton/golem@v0.0.0-20191129093449-a9f1a1b6b185/dicts/en/data.go:1:1: expected 'package', found version

Are there any additional steps to do?

Are you using go-modules or the classic dep system? What version of go are you running?
Does a 'git lfs pull' in the repo do the trick?
A lot of probing questions, but I'm a little bit at a loss why it's not working

I'm using modules, so not entirely sure what to do in that case (other than cloning the repo and replacing it in ./mod directory, which is not an option). What do you think about not serving the dictionaries with the library, but allowing users to provide their own dictionaries location trough path or ENV variable? In that case, large files won't be in git + adding new languages won't require any changes and releasing a new version of the library.

Thanks for the suggestion.
I am also using this project with modules and building it using docker, so it is possible to get it to work, but we have to figure out what and why it's not doing so for everyone.
I will have a look at this over the weekend and hopefully figure it out.
Did the git lfs install have no effect?

Thanks for the detailed report. @smileart
I'll have a peek at it today. Maybe LFS is a bad fit for the dicts, like you said. The size of them is not really large enough to warrant LFS, but they are also too big for regular git (10-50mb). I'll see if I can reduce the size of the binary blobs by using something like a state machine or a btree. And just commit it with git instead.

@smileart
I've built this without git lfs now, and compressed the sizes significantly.  I've also made the dicts into submodules so you will only need to fetch what is necessary.
The big difference for users is that you need to get github.com/aaaton/golem/v4 with the version appended at the end.
You will also need to get the languages/dictionaries you are interested in:
go get github.com/aaaton/golem/v4/dicts/en for english, etc.
It seems to work fine when I'm testing, but please give it a go and let me know.

works fine with modules :+1:
but "go get" from readme gives me this:
$ go get github.com/aaaton/golem/v4/dicts/en
/home/slirx/dev/go/src/github.com/aaaton/golem/v4/dicts/en/pack.go:10:2: no Go files in /home/slirx/dev/go/src/github.com/aaaton/golem/v4

Is that outside of a module? I'm guessing the readme is only adapted for module use.
I haven't really done multi-module repos before and it was trickier than expected.

@aaaton thank you so much for the fix. I tested it in my project, and now it works like a charm! Frankly, meanwhile I've started working on my own lemmatizer to address some custom needs and I just wonder if there where any particular reasons why you went on with those binaries and code generation in the first place? Just so I won't make a mistake of not doing that in my own project. Thanks once again. I believe now README is correct and the project is usable for anyone again.

The binaries and code generation was mainly to be able to package the data with the code, so a "go get" would suffice for any project, without needing additional dict files, paths etc.
VFS or something similar would have done the same trick, but they tend to package the data encoded as a list of bytes, which is not expressed in a compact way in source.
If there is any chance your custom needs would be interesting for some else, please fork and PR.


Delete segment

I use setDigit() to write a character on a 7 seg. How to delete that character, so nothing is displayed? Not to write 0, but to print nothing?

use the void setDigit(int addr, int digit, byte value, boolean dp); function with value=22 (aka ' ')
see the LedControl.h source code file for more explications.


build(deps): remove direct dependency on types

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           56        56
=========================================
  Hits            56        56

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3452251...43172ae. Read the comment docs.


@key on not-required fields

Describe the bug
Following is my annotated GraphQL schema
type Todo @model
@key(name: "byName", fields: ["name", "description"], queryField: "byNameDescription") {
  id: ID!
  name: String!
  description: String
}

Step 1: I create a todo item with just the name as input in the mutation.
Step 2: Run the following query:
query GetByName {
  byNameDescription(name: "<same-name-used-in-mutation-above>") {
    items {
      id
      name
    }
  }
}

The above query returns an empty list
The reason for this is that we index "name" with "description" as the sort key and in my create mutation I'm  able to create the record just with a name since description is not required but the record is not indexed in the byName GSI since the description field is not required - resulting in the query returning empty results.
Expected behavior
There could be two things which could potentially happen:

Validation check pre-push to see if there's a not-required field used with @key to either warn or disallow the user to push
Logic in resolvers to add null values to not-required fields if those fields are not passed as inputs in the mutation

Amplify CLI Version
4.21.3

Hi @kaustavghosh06
I am able to reproduce this bug.Thanks for reporting this.


MDB/ACCDB export ole images

Is it possible export ole images from .mdb or .accdb format with this tool?
Thanks

Hi, @paulopires16
Never tried it before. This tool is a wrapper for mdb-export, anything extractable with mdb-export should be extractable with this tool. Do you have any ole images to try it on?

Yes I have
3 images: https://ufile.io/6nf8v
Paulo Pires

phone  964592113
e-mail<EMAIL_ADDRESS>crypt   myPGP public key for encryption emails
http://kerckhoffs.surfnet.nl/pks/lookup?op=get&search=0x878711213981EEF6
links    facebook http://facebook.com/paulopires16 · twitter
http://twitter.com/#!/paulopires16 · linkedin
http://www.linkedin.com/pub/paulo-pires/36/4a7/607 · netlog
http://netlog.com/paulopires16 · javase
http://download.oracle.com/javase/6/docs/api/ · oracle
http://www.oracle.com/us/syndication/feeds/index.html
Aviso Legal:
Esta mensagem de Paulo Pires é destinada exclusivamente ao destinatário.
Pode conter informação confidencial ou legalmente protegida. A incorrecta
transmissão desta mensagem não significa a perda de confidencialidade. Se
esta mensagem for recebida por engano, por favor envie-a de volta para o
remetente e apague-a do seu sistema de imediato. É proibido a qualquer
pessoa que não o destinatário de usar, revelar ou distribuir qualquer parte
desta mensagem.
Disclaimer:
This message from Paulo Pires is destined exclusively to the intended
receiver. It may contain confidential or legally protected information. The
incorrect transmission of this message does not mean the loss of its
confidentiality. If this message is received by mistake, please send it
back to the sender and delete it from your system immediately. It is
forbidden to any person who is not the intended receiver to use, distribute
or copy any part of this message.
2018-08-09 15:06 GMT+01:00 Kirill Pavlov<EMAIL_ADDRESS>
Hi, @paulopires16 https://github.com/paulopires16
Never tried it before. This tool is a wrapper for mdb-export
https://github.com/brianb/mdbtools/blob/master/doc/mdb-export.txt,
anything extractable with mdb-export should be extractable with this tool.
Do you have any ole images to try it on?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pavlov99/mdb-export-all/issues/3#issuecomment-411769908,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHosHLN6UNacgUu7duumYmCn2Xdt1a4uks5uPEHegaJpZM4V1uHY
.


Ref compile-time error

When I try to create a Ref<Material> the compiler shows the following error:
In file included from godot-cpp/include/core/Godot.hpp:12,
                 from src/init.cpp:1:
godot-cpp/include/core/Ref.hpp: In instantiation of 'void godot::Ref<T>::instance() [with T = godot::Material]':
src/TestClass.hpp:22:21:   required from here
godot-cpp/include/core/Ref.hpp:190:3: error: no matching function for call to 'godot::Ref<godot::Material>::ref(godot::Resource*)'
   ref(T::_new());
   ^~~
godot-cpp/include/core/Ref.hpp:17:7: note: candidate: 'void godot::Ref<T>::ref(const godot::Ref<T>&) [with T = godot::Material]'
  void ref(const Ref &p_from) {
       ^~~
godot-cpp/include/core/Ref.hpp:17:7: note:   no known conversion for argument 1 from 'godot::Resource*' to 'const godot::Ref<godot::Material>&'

I found it because of an error in set_material_override of MeshInstance, I instanced a Ref<ShaderMaterial> (that class doesn't get a compilation error) but when I tried to set the material to a MeshInstance the project would crash and show a backtrace.
I supposed it was related with Ref<Material> because that's the type set_material_override receives. I may be wrong and they could be unrelated errors.
I'm providing a reproduction project, just drop godot-cpp with the bindings compiled in bin/ and call ùse the makefile.
The important code is in src/TestClass.hpp
report-material.zip

MeshInstance *meshInst = new MeshInstance;

new won't work with gdnative objects, use _new static method instead:
MeshInstance *meshInst = MeshInstance::_new();

and meshInst->free(); to delete it.

Also Material is pure virtual class and can't be instanced (doesn't have _new() method).

Interesting, I didn't know about these changes in the API. I guess this can be closed.

The underscore is because new is a keyword in C++ and you can't have a method named like that

There was a time where we used an overload for operator new but that turned out to be quite problematic :/


Log on config get

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 26.5%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server/external.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
636

Relevant Lines:
2400

💛  - Coveralls


{} as fmt::Show during --verbose

@alexcrichton regarding RFC.
What are your thoughts on changing the default {} to use fmt::Show during rustc --verbose?
Why:
Say I have a library, and it returns custom errors. Lets pretend its a parser/lexer.
In the most basic case I want to print fmt::String. Just something simple letting the users know there was an "invalid character and at line 32". Meanwhile, if there was a bug, what I want is a lot more information. Maybe I need the full sate of my lexer? i.e. Basically, I want it to display fmt::Show.
In this case I don't want to have to go to my user-facing code {} and change it to {:?} before I can start debugging. I just want to tell the compiler, I want verbosity, and have it fulfill my request.
Small implementation detail: I get that this means, {} would need both fmt::Show and fmt::String. However, I think that should be easy to reconcile. Since Show is supposed to always be a superset of String's information, if there is no explicit impl for Show (including no #[derive(Show)]), then it should be safe to use String as a default impl for Show.
Let me know what you think.

Could logging handle this? Log at error level the user message, and log at debug more details. Then, setting log level to debug will show more.

I've rewritten this a few times. I've tried to be as succinct as possible. I've left quite a bit out, but still I wasn't very successful, sorry.
Loggers are definitely part of my inspiration here. However, I think this is a subset of their functionality that can easily be done without them.
First:
Maybe a silly reaction but I felt the need to counter your overall point:
Why have println! at all? Couldn't logging handle this?
I think the answer here is, familiarity and simplicity.

You start learning the language with println!("Hello World") not log!("Hello World")
Most times you really don't need a full logger in your application. We aren't all always building web browsers/servers/applications. Think about the grep/sort/uniq/<simple script you wrote to make your life easy> use cases.
Note: Maybe part of this is caused by my dislike of printing directly to the stdout. If the default(only?) way to output was a logger I don't think we would be having this conversation.

Second:
[Relatively minor, but worth noting] Ergonomics: Do you really want to see a useless debug! every time you output to a user?
// Tell the user something bad happened
debug!("There was an error: {:?}", error);
println!("There was an error: {}", error); // Maybe replace println! with info!

// Tell the user something really bad happened
debug!("There was an error: {:?}", error);
panic!("There was an error: {}", error);

Wouldn't you personally just end up making a helper macro?
Third:

As Rust grows outside the current pool of very diligent Rust builders to Rust users: Not everyone is going to listen to the guidelines. If the difference is between getting frustrated while debugging and always using {:?} I think people will just grow accustomed to always use {:?}.
There is a huge difference between:

"Hey follow our rules! And if you do, we can do cool stuff so you don't have to!"

Exactly the philosophy behind the borrow checker (strict but beneficial).

"Hey follow our rules! Its more semantically correct that fmt::Show is not displayed to users"

Don't get me wrong, I like this change it standardizes something I would have had to do myself

Previously maybe I would have created a .debug() or .pretty() depending on how I felt like using Show

However, this particular implementation now reduces my flexibility without substantial benefit. (Am I missing some of the better benefits?)

Not saying my suggestion should necessarily be the "cool stuff" that comes form this change, but I hope while coming up with guidelines like these that philosophy (strict but beneficial) stays intact.

Isn't {} a compile time thing? How would you get it to do something
different at run time?
On Sat, Jan 10, 2015, 11:10 PM Yazad Daruvala<EMAIL_ADDRESS>wrote:

I've rewritten this a few times. I've tried to be as succinct as possible.
I've left quite a bit out, but still I wasn't very successful, sorry.
Loggers are definitely part of my inspiration here. However, I think this
is a subset of their functionality that can easily be done without them.
First:
Maybe a silly reaction but I felt the need to counter your overall point:
Why have println! at all? Couldn't logging handle this?
I think the answer here is, familiarity and simplicity.

You start learning the language with println!("Hello World") not log!("Hello
World")
Most times you really don't need a full logger in your application. We
aren't all always building web browsers/servers/applications. Think
about the grep/sort/uniq/
use cases.
Note: Maybe part of this is caused by my dislike of printing directly to
the stdout. If the default(only?) way to output was a logger I don't
think we would be having this conversation.

Second:
[Relatively minor, but worth noting] Ergonomics: Do you really want to see
a useless debug! every time you output to a user?
// Tell the user something bad happened
debug!("There was an error: {:?}", error);
println!("There was an error: {}", error); // Maybe replace println! with info!
// Tell the user something really bad happened
debug!("There was an error: {:?}", error);
panic!("There was an error: {}", error);
Wouldn't you personally just end up making a helper macro?
Third:

As Rust grows outside the current pool of very diligent Rust
builders to Rust users: Not everyone is going to listen to the guidelines.
If the difference is between getting frustrated while debugging and always
using {:?} I think people will just grow accustomed to always use {:?}.
There is a huge difference between:

"Hey follow our rules! And if you do, we can do cool stuff so you
don't have to!"

Exactly the philosophy behind the borrow checker (strict but
beneficial).

"Hey follow our rules! Its more semantically correct that
fmt::Show is not displayed to users"

Don't get me wrong, I like this change it standardizes
something I would have had to do myself

Previously maybe I would have created a .debug() or
.pretty() depending on how I felt like using Show

However, this particular implementation now reduces my
flexibility without substantial benefit. (Am I missing some of the better
benefits?)

Not saying my suggestion should necessarily be the "cool stuff" that comes
form this change, but I hope while coming up with guidelines like these
that philosophy (strict but beneficial) stays intact.
—
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/rfcs/issues/568#issuecomment-69482984.

Oh you want a flag on the compiler, I thought you wanted something a flag
on your program. Ignore my comment.
On Sun, Jan 11, 2015, 3:17 AM Gulshan Singh<EMAIL_ADDRESS>wrote:

Isn't {} a compile time thing? How would you get it to do something
different at run time?
On Sat, Jan 10, 2015, 11:10 PM Yazad Daruvala<EMAIL_ADDRESS>wrote:

I've rewritten this a few times. I've tried to be as succinct as
possible. I've left quite a bit out, but still I wasn't very successful,
sorry.
Loggers are definitely part of my inspiration here. However, I think this
is a subset of their functionality that can easily be done without them.
First:
Maybe a silly reaction but I felt the need to counter your overall point:
Why have println! at all? Couldn't logging handle this?
I think the answer here is, familiarity and simplicity.

You start learning the language with println!("Hello World") not log!("Hello
World")
Most times you really don't need a full logger in your application. We
aren't all always building web browsers/servers/applications. Think
about the grep/sort/uniq/
use cases.
Note: Maybe part of this is caused by my dislike of printing directly to
the stdout. If the default(only?) way to output was a logger I don't
think we would be having this conversation.

Second:
[Relatively minor, but worth noting] Ergonomics: Do you really want to
see a useless debug! every time you output to a user?
// Tell the user something bad happened
debug!("There was an error: {:?}", error);
println!("There was an error: {}", error); // Maybe replace println! with info!
// Tell the user something really bad happened
debug!("There was an error: {:?}", error);
panic!("There was an error: {}", error);
Wouldn't you personally just end up making a helper macro?
Third:

As Rust grows outside the current pool of very diligent Rust
builders to Rust users: Not everyone is going to listen to the guidelines.
If the difference is between getting frustrated while debugging and always
using {:?} I think people will just grow accustomed to always use {:?}
.
There is a huge difference between:

"Hey follow our rules! And if you do, we can do cool stuff so
you don't have to!"

Exactly the philosophy behind the borrow checker (strict but
beneficial).

"Hey follow our rules! Its more semantically correct that
fmt::Show is not displayed to users"

Don't get me wrong, I like this change it standardizes
something I would have had to do myself

Previously maybe I would have created a .debug() or
.pretty() depending on how I felt like using Show

However, this particular implementation now reduces my
flexibility without substantial benefit. (Am I missing some of the better
benefits?)

Not saying my suggestion should necessarily be the "cool stuff" that
comes form this change, but I hope while coming up with guidelines like
these that philosophy (strict but beneficial) stays intact.
—
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/rfcs/issues/568#issuecomment-69482984.

We would have to grow a compiler flag for something such as this separate from --verbose as it's only intended for the compiler's output itself, but this is certainly an interesting idea! I'm not sure I'd be in favor of swapping out the meaning of {} at compile time, however. This might be a great idea for a plugin, but not necessarily for the core compiler itself.

One concern I'd have with this: String and Show have different purposes. I use String to write the proper string for headers to a tcp stream. If a flag were to change those uses of {} into {:?}, the headers would not be correctly formatted.

Thats interesting. I wouldn't have guessed that is the way you implemented it, but I suppose why not..
I get the convenience/correctness to use Header.to_string(). However, since tcp headers aren't usually meant to be "human readable", I just would have just imagined you using ToString, not String.
Maybe I'm just confused. I get the difference between String and Show, and I get that ToString is automatically derived if String is implemented, but would you clarify the difference between ToString and String please? I mean, they must be different or ToString would have been deprecated, right?

The ToString trait has a blanket implementation over T: fmt::String, so it's not possible to implement it individually. I also don't usually use the to_string() method myself. And I'm talking about HTTP headers, where a ContentLength(125) must be written as Content-Length: 125\r\n.
Some stripped code I use:
for header in self.headers.iter() {
    try!(write!(&mut stream, "{}", header));
}

You can imagine more complex headers, such as Cookie or CacheControl or what-have-you. They have a fmt::String implementation that writes according the spec, and they #[derive(Show)] to help when I'm debugging the Headers struct.

I think one thing the RFC doesn't talk about, but should:
When should you be using {} vs to_string()?
Example:
// impl String for Header
write!(&mut stream, "{}", header)

// impl ToString for Header (or String; if a user is meant to read it)
// write!(&mut stream, header.to_string())
stream.write_str(header.to_string())

Which is better Rust style?
I would have expected header.to_string(), for a couple reasons:

If you only need "{}" you don't need the format! macro. format! is only useful if you'd otherwise need to do a couple verbose string concatenations.
Because the RFC specifically says String is for a "user readable" [1] version of the struct, and the connotations associated with a header are that they are not meant to be "user readable".
I would have expected ToString to have been implemented instead of String [2].

P.S.
[1] What does it mean to be "user readable"? Clearly that could mean different things to different people. The way I see it is "user readable" implies loggers or printing to stdout.
[2] You're right though.. I tried and I can explicitly implement ToString for a local struct but I can't explicitly implement both String and ToString on the same struct. Which is again not what I would have expected. For some reason I just thought the compiler would override the more generic version of ToString for my revised version (basically giving precedence to the implementation in the same scope as the struct definition).

In most cases, using {} is better. Calling to_string() will allocate a new String on the heap, and if you're just going to write that String into another buffer, there's no need to waste the allocation; just write straight to the existing buffer (a stream, a Vec, another String, etc).


Changes for CheckRebuildPlugin to satisfy osbs-client#181

The CheckRebuildPlugin should:

be renamed to CheckAndSetRebuildPlugin
check if this is autorebuild (received label value is "false" (string, since Openshift can't use booleans as label values); if so, it should
if not, then set the received label to received value (which is always going to be "true" (again, string), when invoked through osbs-client).

Note: it's necessary to check build_json["metadata"]["labels"], not just build_json["metadata"] for the labels!

Now we need OSBS.set_label_on_buildconfig() :-)


fix(assets): add links in comments to M 317

add links to sigla in comments to M 317.

Codecov Report

Merging #835 (28ae99a) into develop (44c86de) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff            @@
##           develop     #835   +/-   ##
========================================
  Coverage    61.87%   61.87%
========================================
  Files          236      236
  Lines         3785     3785
  Branches       578      578
========================================
  Hits          2342     2342
  Misses        1412     1412
  Partials        31       31

Flag
Coverage Δ

unittests
61.87% <ø> (ø)

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

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 44c86de...28ae99a. Read the comment docs.


Publish: Managing and communicating rollover 2023

Added post about the emails we sent for rollover 2023.

 Design history: communicating rollover 2023


rename Azure to Mircosoft

What kind of change does this PR introduce?
Renames Azure to Mircosoft
What is the current behavior?
Everything named Azure
What is the new behavior?
Everything renamed Azure
Additional context
notion task

@Isaiah-Hamilton what was the motivation behind this change? the provider name used in gotrue-js is called azure and changing that would be breaking

@Isaiah-Hamilton what was the motivation behind this change? the provider name used in gotrue-js is called azure and changing that would be breaking

The motivation behind this is the general public might relate better to sign in with microsoft as opposed to sign in with azure, Notion task. But now seeing how much we would have to change I think it would be better to maybe only update the docs.

yeah that's true, maybe we can rename the stuff in the docs to Azure (Microsoft) ? would that be clearer?

Hey @Isaiah-Hamilton,
Apologies on this, I should have thought through this further.
Perhaps let's close this PR and rename the  docs to Azure (Microsoft)  as @kangmingtay has suggested?


Proxy is not working

Still not working:
# gvm version
Go Version Manager v1.0.22 installed at /root/.gvm

# echo $http_proxy
http://localhost:7777

# echo $https_proxy
http://localhost:7777

# echo $socks_proxy
socks5://localhost:1080

# cat ~/.curlrc
proxy=socks5://localhost:1080

# netstat -lnpt | grep -P '1080|7777'
tcp        0      0 <IP_ADDRESS>:1080          <IP_ADDRESS>:*               LISTEN      26126/ssh
tcp        0      0 <IP_ADDRESS>:7777          <IP_ADDRESS>:*               LISTEN      27209/cow
tcp6       0      0 ::1:1080                :::*                    LISTEN      26126/ssh

But when i issue this command:
# gvm listall

gvm gos (available)

fatal: remote error: Access Denied (not available from your location)

In order, when i issue the curl command directly, it's working fine:
# curl --proxy socks5://localhost:1080 https://storage.googleapis.com/golang/go1.6.src.tar.gz > tmp
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                             Dload  Upload   Total   Spent    Left  Speed
 66 12.0M   66 8144k    0     0  1562k      0  0:00:07  0:00:05  0:00:02 1659k

I'm mystified. What are you referring to? Can you please explain what you are trying to do?


Remoting Player

Hi there,
is it possible to replicate/extend the functionality of the remoting player? e.g. any plans to open source the remoting player app or provide frameworks wo write an own?
Motivation is that it useful to perform more computations (e.g. surface mesh/raycasting ..) on the hololens while still offloading the main rendering the to the remote player host pc.
Microsoft.Holographic.Remoting seems to contain server functionality - is this supposed to be used non-internally? e.g. CreateHolographicStreamerServer(..)? Are there any samples?
Thanks alot.

Maybe Not. https://github.com/Microsoft/HoloLensCompanionKit/issues/4#issuecomment-253691520
Hopefully remoting will be better supported in the future.


Updated Star Count of the Repo

What does this PR do?
Improve repo
For resources
Description
Updated star count to 200,000
Why is this valuable (or not)?
Readme is updated with Repository, not left behind.
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x]  Read our contributing guidelines

[x] Search for duplicates.

[ ] Include author(s) and platform where appropriate.

[ ] Put lists in alphabetical order, correct spacing.

[ ] Add needed indications (PDF, access notes, under construction)

Followup

Check the output of Travis-CI for linter errors!

I hadn't noticed!
Thanks, and welcome to free-programming-books!


Updated content to fit original repo

Updated table of content
Swapped Classes & SOLID
Updated example functions in Objects and Data Structures

@quangphuc789
The anchor link to 6.SOLID in table of content seems doesn't work when I click on it.
Please take a look.

@hienvd @tiennv91 I have made the changes to add anchor link to 6. SOLID
I have made another pull request. Can you review?

It works now, thanks @quangphuc789
Btw, no need to create another PR.
@tiennv91: Looks good to me, waiting for your review :)

@hienvd @tiennv91 I have made a new commit, which updated the Chinese language versions. Cheers :)


Add attributes as props

Hello,
is it possible to access the attributes added to markdown as props when adding a custom component like in your Callout.vue example?
Attributes are set as html paratmeters right now which makes them inaccessable:

Thanks

Never mind, i didnt spend enough testing it properly, it is working as expected.

how did you get props to work?
I'm trying this library and just keep getting Uncaught (in promise) TypeError: txtProp is undefined when trying to use a component that expects a prop


List keyboard shortcut

Others
Where I can see keyboard shortcuts?

@winardiaris right now only these few shortcuts, so if you are using media keys like how you use normally, you are all good !
https://github.com/EragonJ/Kaku/blob/d9dfab6bcafe8e2434e269ff9dac505fdf300c3f/bootup.js#L70

This reminds that it would be cool to have a ? shortcut or help menuItem to trigger a popup dialog to list down these things. Can keep this open first and waiting for any other ideas for now.

I this #431 complement this issue.

close old issue and will open a new one if needed.


[Datasource][Transport settings]No confirmation message when deleting parameters

Steps to reproduce:

Create a Data Service project and add a new Data Service.
In Transport Settings, in Authorization Provider Configuration, add Authorization Provider class and parameters.
Click on the delete button in front of the parameters.

Expected: A confirmation message to delete the parameters.
Actual: Without confirmation message, the parameter gets deleted.

Fixed in - https://github.com/wso2/devstudio-tooling-dss/pull/170


Check action design consistency (Chess, Shogi, AnimalShogi)

Is it easy to recover the square info with same operation?

Animal Shogi: x, sq = jnp.int8(a // 12), jnp.int8(a % 12)
Chess: from_, plane = label // 73, label % 73 (consistent to OpenSpiel)
Shogi: direction, to = jnp.int8(action // 81), jnp.int8(action % 81) (consistent to dlshogi)

Also, write these info to docs.

Seems sq = label % boardsize is natural? (considering Go and Hex)


Any possibility on making this free and open source?

I see that libdisorder is under GPL(v2) but that the rest of the source is not under a free license.  Is there any possibility that fastahack could be put under a free license?

I'm interested in this, too, as I'd like it to be packaged for GNU Guix.  Fasta.cpp and Fasta.h contain "All rights reserved" notices.  Is there any way to release them under a free license?

They should be under an open license. Let me make that change.
On Wed, Apr 20, 2016, 14:07 rekado<EMAIL_ADDRESS>wrote:

I'm interested in this, too, as I'd like it to be packaged for GNU Guix.
Fasta.cpp and Fasta.h contain "All rights reserved" notices. Is there any
way to release them under a free license?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-212416320

Any progress on this?

I'd also be very appreciative if the licencing was made clear for fastahack (as well as smithwaterman, intervaltree, multichoose, and tabixpp).
P.S. Congratulations on the PBWT manuscript

Same here as for libsmithwaterman:  To package for Debian a free license is needed.  I'd like to package freebayes for Debian and all its predependencies need a free license.

They should all be under a free license... Is there anything we can't
simply add an MIT license to?
On Fri, Jul 1, 2016, 09:27 Andreas Tille<EMAIL_ADDRESS>wrote:

Same here as for libsmithwaterman: To package for Debian a free license is
needed. I'd like to package freebayes for Debian and all its
predependencies need a free license.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-229875874, or mute
the thread
https://github.com/notifications/unsubscribe/AAI4ERfgt3wQo7CbohXBeM8KZirc69qhks5qRMFMgaJpZM4GbXRl
.

Hi Erik,
thanks for your quick response.  Could you please at least drop a
LICENSE file into the git repository that specifies this?  Otherwise per
definition a code with no explicite license is non-free.
Kind regards
  Andreas.

On Fri, Jul 01, 2016 at 12:35:10AM -0700, Erik Garrison wrote:

They should all be under a free license... Is there anything we can't
simply add an MIT license to?
On Fri, Jul 1, 2016, 09:27 Andreas Tille<EMAIL_ADDRESS>wrote:

Same here as for libsmithwaterman: To package for Debian a free license is
needed. I'd like to package freebayes for Debian and all its
predependencies need a free license.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-229875874, or mute
the thread
https://github.com/notifications/unsubscribe/AAI4ERfgt3wQo7CbohXBeM8KZirc69qhks5qRMFMgaJpZM4GbXRl
.

You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/ekg/fastahack/issues/8#issuecomment-229877634

--
http://fam-tille.de

Yes, I will do so.
For packaging, we are in the process of setting as default commonly used
options. It would be good if the packaged version were made after these
changes.
On Fri, Jul 1, 2016, 09:56 Andreas Tille<EMAIL_ADDRESS>wrote:

Hi Erik,
thanks for your quick response. Could you please at least drop a
LICENSE file into the git repository that specifies this? Otherwise per
definition a code with no explicite license is non-free.
Kind regards
Andreas.
On Fri, Jul 01, 2016 at 12:35:10AM -0700, Erik Garrison wrote:

They should all be under a free license... Is there anything we can't
simply add an MIT license to?
On Fri, Jul 1, 2016, 09:27 Andreas Tille<EMAIL_ADDRESS>wrote:

Same here as for libsmithwaterman: To package for Debian a free
license is
needed. I'd like to package freebayes for Debian and all its
predependencies need a free license.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-229875874,
or mute
the thread
<
https://github.com/notifications/unsubscribe/AAI4ERfgt3wQo7CbohXBeM8KZirc69qhks5qRMFMgaJpZM4GbXRl

.

You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/ekg/fastahack/issues/8#issuecomment-229877634

--
http://fam-tille.de
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-229881464, or mute
the thread
https://github.com/notifications/unsubscribe/AAI4ET3xT5oS_RBvYRrRNE3LYmKV8KJSks5qRMgcgaJpZM4GbXRl
.

Hi Erik,
On Fri, Jul 01, 2016 at 01:01:30AM -0700, Erik Garrison wrote:

Yes, I will do so.

Thanks a lot.

For packaging, we are in the process of setting as default commonly used
options. It would be good if the packaged version were made after these
changes.

Thanks for this helpful information.  However, I think I'll go with the
current version for a start since for new software it takes some time to
"pass the entrance" into Debian.  Once some software is in we can apply
changes quite quickly.
If you speak about new structure:  For Debian I have separately packaged
libdisorder to avoid a code duplication.  You can download the package
source from
https://packages.debian.org/source/unstable/libdisorder
or from packaging git that is linked from there.
May be this helps.
Kind regards
  Andreas.

--
http://fam-tille.de

Right, that makes sense. Getting it in first is a harder step than updating
the default parameters and bumping the version. Thank you so much for doing
this. Debian is awesome and it will be amazing to have freebayes and
related tools available there. I will apply the licenses within the next
few hours (need to get back to a connected computer).
On Fri, Jul 1, 2016, 14:39 Andreas Tille<EMAIL_ADDRESS>wrote:

Hi Erik,
On Fri, Jul 01, 2016 at 01:01:30AM -0700, Erik Garrison wrote:

Yes, I will do so.

Thanks a lot.

For packaging, we are in the process of setting as default commonly used
options. It would be good if the packaged version were made after these
changes.

Thanks for this helpful information. However, I think I'll go with the
current version for a start since for new software it takes some time to
"pass the entrance" into Debian. Once some software is in we can apply
changes quite quickly.
If you speak about new structure: For Debian I have separately packaged
libdisorder to avoid a code duplication. You can download the package
source from
https://packages.debian.org/source/unstable/libdisorder
or from packaging git that is linked from there.
May be this helps.
Kind regards
Andreas.
--
http://fam-tille.de
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-229949048, or mute
the thread
https://github.com/notifications/unsubscribe/AAI4EWfw05TgP-nXabzNYTS-RqTGIB00ks5qRRisgaJpZM4GbXRl
.

Any luck yet?

@ekg Thank you for adding the LICENSE file!
It would be very nice if you could also update the file headers to mention the license, because right now Fasta.cpp still says "All rights reserved", which makes it impossible for us to assume that the LICENSE file also applies to Fasta.cpp.
I'm looking forward to seeing this packaged for GNU Guix and other GNU/Linux distributions!

This is a friendly ping :)
We'd still like to be able to package it for GNU Guix.

All the file headers need to have the license?
On Mon, Jan 16, 2017 at 11:43 PM rekado<EMAIL_ADDRESS>wrote:

This is a friendly ping :)
We'd still like to be able to package it for GNU Guix.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ekg/fastahack/issues/8#issuecomment-272978876, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAI4EZj60VHRuGaT5qc-hJ4zh0PlAisHks5rS_KrgaJpZM4GbXRl
.

FWIW here's what the FSF says about it:

Is it enough just to put a copy of the GNU GPL in my repository?
Why should I put a license notice in each source file?

Since the LICENSE file is there the needed conditions are fullfilled.  Package is accepted by Debian ftpmaster so the license was also verified.  I think the issue can be closed.

I can confirm that Debian ftpmasters have accepted the package inside Debian which is a good sign that there are people who are quite picky about licenses agree that this bug can be closed.  Doing a real release (see issue 14) would be nice.


questions for future release ?

Hi,
Thank you for your library, it is very good, and is much more flexible than my homemade solution.
I was wondering things
I have several ESP modules that do (again with my update code) automatic updates when I release a new version.
There are identical modules, and variants, which may have slightly different hardware, in which case the bin is of course compiled accordingly.
But they share the same source, it's just #define that changes the code.
Is it possible to have a single json file that would contain multiple configs? instead of having a json/bin pair for each variant.
Steph

@stef-ladefense Thank you for your comment.  Yes, I need to make it clearer in the README, but the library is indeed designed to support multiple Configurations in the same .json.
Let's say you had one device that ran using the "ESP32 Device Module" board profile.  You might choose to create a Configuration block with
{
  "Configurations": [
    {
      "Board": "ESP32_DEV",
      ...
    }
  ]
}

If your other device is, say, ESP32 S3 Dev Module, you could make a second block with
{
  "Configurations": [
    {
      "Board": "ESP32S3_DEV",
      ...
    }
  ]
}

If you have two variants using the same Board, then perhaps that's when you'd define "Config" strings, e.g
      "Config": "16MB with PIR sensor",

or
      "Config": "4MB with no sensor",

The library simply uses the first Configuration block it comes to that fully matches.
Clear?

@stef-ladefense, May I ask... what tools do you use for your automated builds?  I, too, would like to be able to make a small change to a sketc, then automatically recompile (and post?) the 10 different binaries for all the various devices it runs on.
Thanks!
M

hi @mikalhart
i tested 2 config in one json
{
  "Configurations": [
    {
      "Board": "LOLIN_S2_MINI",
      "Version": "1.0.1",
      "Config": "CO2_S8",
      "URL": "http://menucd.free.fr/testweb.ino.lolin_s2_mini.bin"
    }
  ]
}

{
  "Configurations": [
    {
      "Board": "LOLIN_S2_MINI",
      "Version": "1.0.1",
      "Config": "CO2_S4",
      "URL": "http://menucd.free.fr/testweb.ino.lolin_s2_mini.bin"
    }
  ]
}

but only first work,
second : CheckForOTAUpdate returned -2 (No profile matches)

@stef-ladefense, May I ask... what tools do you use for your automated builds? I, too, would like to be able to make a small change to a sketc, then automatically recompile (and post?) the 10 different binaries for all the various devices it runs on.
Thanks!
M

unfortunately it is not automatic. I use the preprocessor. for example, I have several CO2 detectors, the card is the same, an esp32s2, but the sensor is different, senseair S8, Sensirion SCD40 or 41, etc.
so i have a party where i start a #define
and I make blocks between #ifdef and #endif where it will be different.
it must be possible to automate, but I now only make a few modifications, it is often adaptation following an update of the core or a library of a routine that no longer works.
the next will be the integration of your update routine.

@stef-ladefense Try this syntax:
{
  "Configurations": [
    {
      "Board": "LOLIN_S2_MINI",
      "Version": "1.0.1",
      "Config": "CO2_S8",
      "URL": "http://menucd.free.fr/testweb.ino.lolin_s2_mini.bin"
    },
    {
      "Board": "LOLIN_S2_MINI",
      "Version": "1.0.1",
      "Config": "CO2_S4",
      "URL": "http://menucd.free.fr/testweb.ino.lolin_s2_mini.bin"
    }
  ]
}

very good, work fine
y a the best

Great!

May I ask... what tools do you use for your automated builds? I, too, would like to be able to make a small change to a sketc, then automatically recompile (and post?) the 10 different binaries for all the various devices it runs on.

@mikalhart - Funny you should ask. We are using your lib inside our github action to compile our production firmware. While not exactly what you are looking for, we are compiling and posting different binaries (production vs beta release) to a binaries repo. To address the 4MB vs 16MB, we leverage a python based Firmware Uploader to ping the ESP32 for its flash size during prelim step, then use the appropriate partition binary when loading the new firmware. We don't do automatic build-on-push but it's trivial to implement that in the yml script.
Now, for those nuggets of (hopefully) helpful info, may I please request you do a new release so that I can stop maintaining a fork because GetVersion() is not in your release and is breaking our firmware builds? :)

Now, for those nuggets of (hopefully) helpful info, may I please request you do a new release so that I can stop maintaining a fork because GetVersion() is not in your release and is breaking our firmware builds? :)

@nseidle -

Never mind! I must be sleep deprived. It's working fine now.

hi @mikalhart ,
i test version for use with esp32 and esp8266.
look this : https://github.com/stef-ladefense/ESP32-OTA-Pull/commit/de5912382b0bbcbc7e554a932e91edd729aaba19


Fix metadata and pickup updated vectormath headers.

Incorrect metadata on plugin binaries was causing load failures on Android
Picked up new VectorMath headers with a Neon fix that addresses buzzing on Android

isPreloaded: 0

I think that we want the isPreloaded flag set to 1 for these plugins

Refers to: Source/Plugins/CrossPlatformPluginGenerator/Assets/Microsoft.SpatialAudio.Spatializer.Unity/Plugins/Android/libs/arm64-v8a/libHrtfDsp.so.meta:9 in 508dc91. [](commit_id = 508dc915caba6eea2644fe175cef832ec22bb236, deletion_comment = False)

isPreloaded: 0

I think that we want the isPreloaded flag set to 1 for these plugins
Refers to: Source/Plugins/CrossPlatformPluginGenerator/Assets/Microsoft.SpatialAudio.Spatializer.Unity/Plugins/Android/libs/arm64-v8a/libHrtfDsp.so.meta:9 in 508dc91. [](commit_id = 508dc91, deletion_comment = False)

I'm not sure we do, the plugin library will load it on-demand. At least in my testing I didn't need to have that set.


MPI Error

Thanks so much for the code, it helped me a lot.
But I'm having some problems running it.
 File "...../wdm/wdm/dist_util.py", line 10, in <module>     from mpi4py import MPI ImportError: libmpi.so.12: cannot open shared object file: No such file or directory
I have installed the related package and openmapi-4.1.5, by the way, I run it on CentOS

Do you have this problem? Thx for your help

Thank you. We conducted all experiments on Ubuntu 22.04 and we didn't have this problem... I think it's an environment issue related to the mpi4py package on CentOS.

It might be a version error of openmpi, the openmpi-4.1.5 corresponds to libmpi.so.40, not libmpi.so.12.
But before I successfully installed the lower version of openmpi, I reinstalled miniforge in a normal user env, and this time the installed mpi4py and mpich were not the same as the previously installed version, and ran successfully without installing openmpi. I don't understand what's going on...but it doesn't matter.

Thx very much for your help!


UPX compresses the binary but increases the RAM usage

The UPX section should be updated to note that, while UPX reduces the binary's disk size, it increases memory usage. This is because UPX disables demand paging by fully loading the decompressed binary into memory. In RAM-constrained environments, this can have a significant impact and may worsen the situation when using UPX.
That said, UPX might be a suitable solution for certain environments, but it comes with a trade-off. If the primary issue is the binary's size, using a compressed file system might be a more effective approach.

I think UPX also have caching problem?
If you have more than one processes running the same binary, they might end up duplicating the decompressed binary.
Not sure when that will happen as I believe upx does have some ways to cache it, but I think whenever Linux namespace is used, or a different uid, it cannot reuse the decompressed binary.
And the binary could also be under attack as it is stored on tmpfs, initially writable.
IMHO transparent compression support from filesystem seems better, with no duplication, no way to tamper with, and decompress by block on demand.

@janflyborg Do you have a reference that mentions this increased memory usage, such as a link into the UPX docs? I'm not sure how in the weeds we want min-sized-rust to be here, but since we already mention the anti-virus concerns perhaps this could be okay to mention as well.

I'm going to close this as out of scope for min-sized-rust. Thanks for sharing though!


python3Packages.scipy: schedule as big-parallel on Hydra

After 9503adb666001 the tests can take many hours or even time out: https://hydra.nixos.org/job/nixpkgs/staging-next/python310Packages.scipy.x86_64-linux#tabs-charts That's because without big-parallel we use --cores 2; I just hope that with big-parallel it won't be exploding like described around the 9503adb666001 commit.
(The usual checklist doesn't apply.)

Maybe it's a big rebuild; I'll let OfBorg estimate that.


Many functions do not include thread safety information.

Many class definitions do not include thread safety information. For example for https://github.com/Azure/azure-docs-sdk-dotnet/blob/b39513f/xml/Microsoft.Azure.DataLake.Store/AdlsClient.xml does not mention anywhere if the class's methods are safe to use from multiple threads (like WebClient is) or if we need a instance per thread.

Hi there!  I realize this is an generalized comment, but the change to the specific example requires a change to the underlying /// comments.
Created new issue https://github.com/Azure/azure-service-bus-dotnet/issues/431

@leftler FYI, product team has added thread safety information to the next release.


Fix broken link in README (#846)

I chose this link because it contains the same information as the previous page (as archived by the Wayback Machine from 2011 through 2021), in a clean and easy-to-read format.
If you'd like, I could also put together some screenshots of the various layouts and include it in a subsequent PR - but that seemed like overkill for this trivial change, and I wasn't sure you'd want to include the image files in this project's repository.

Thanks for this, @lyndsysimon. I'm not familiar with manpages.org. Do you know if it's used by many other projects? Is it safe to assume that it's stable? On that note, I wouldn't be opposed to linking to the archived version of either page.

Closed by #854.


Emulator Bug

[12:10:22] [err-client] Uncaught Error: Could not call remote function ''. Check that the function signature is correct. Underlying error: Error processing argument at index 0, conversion failure from null Error: Could not call remote function ''. Check that the function signature is correct. Underlying error: Error processing argument at index 0, conversion failure from null at callFunction (C:\Users\laliden\AppData\Local\botframework\app-3.5.25\resources\electron.asar\browser\rpc-server.js:228:11) at EventEmitter. (C:\Users\laliden\AppData\Local\botframework\app-3.5.25\resources\electron.asar\browser\rpc-server.js:316:5) at emitMany (events.js:127:13) at EventEmitter.emit (events.js:201:7) at WebContents. (C:\Users\laliden\AppData\Local\botframework\app-3.5.25\resources\electron.asar\browser\api\web-contents.js:219:13) at emitTwo (events.js:106:13) at WebContents.emit (events.js:191:7) C:\Users\laliden\AppData\Local\botframework\app-3.5.25\resources\electron.asar\renderer\api\remote.js 217 7 {}

Thanks @LarsLiden. What were you doing when you got this error? Was it at startup?

No, I'd been using the emulator for a while.
On Wed, Apr 5, 2017 at 8:30 AM, Eric Anderson<EMAIL_ADDRESS>wrote:

Thanks @LarsLiden https://github.com/LarsLiden. What were you doing
when you got this error? Was it at startup?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotFramework-Emulator/issues/147#issuecomment-291899239,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGOd5rdo2ZFI36lH3rqMxcx9-LcP0Lviks5rs7ObgaJpZM4Mx97p
.

Closing this issue due to lack of actionable info and no repro.


VirtualBox GUI has Changed - No Host-Only-Network tab

The "Host Only" tab does not exist in VirtualBox Preferences as directed in Part-1. I'm running the latest stable release: Version 5.2.2 r119230 (Qt5.6.2) which changed since this guide was created.
Screenshot of Preferences window without Host Only tab.

There are likely other changes to GUI that no longer match this guide.

Articles have been updated. Thanks for the heads up!


Doc Tech Edits

Reviewed the documentation, corrected grammar, spelling, and focused on readability. This addresses Issue #706 and Issue #709. This also fixes index hierarchy problems with the Ingest section and the ingeststats table formatting.

@rfecher Made fixes according to your comments.


list.focusForeground should override gitDecoration.ignoredResourceForeground

I have a contrast issue when using gitDecoration.ignoredResourceForeground, I have node_modules focused in the following screenshot:

To give a more obvious example of what's going on, with these theme keys:
"workbench.colorCustomizations": {
    "list.focusForeground": "#ff0000",
    "gitDecoration.ignoredResourceForeground": "#0000ff"
}

I see this (node_modules is focus, .bin is active):

I think node_modules should be #ff0000 in this case in order to fix my issue.
Theme used: https://marketplace.visualstudio.com/items?itemName=Tyriar.theme-sapphire

@Tyriar I'm not too sure... gitDecoration.ignoredResourceForeground seems to be way more specific than list.focusForeground, so I'm not sure I would alter their priority.

At first I wasn't convinced about it, but since it is about when the element is focused, I believe that makes sense.

Letting decoration colours win over selection/focus colours was a conscious decision. cc @bpasero

Letting decoration colours win over selection/focus colours was a conscious decision.

Yeah I cannot remember why we went one or the other direction. Anyway it looks like we are not consistent though:

The selection color wins of the tree but the focus color does not?

It is weird that active works but not focus, also if this is/was as designed then it basically prevents using a subtle color for the ignored state if I want to see it when it's focused.
Material theme (no 2 in marketplace) also sees this problem:

/duplicate of #41643


allow override invoice shop billing data

This PR allows extends the InvoiceShopBillingData entity.

Thank you, @jbcr! :tada:


Display stuck at various points??

I've just got a Cardputer and have installed M5 Launcher (v2.2.4) on it, which is working well.
After downloaded a binary to the SD card the screen appears to be stuck at "Downloaded". I randomly press buttons (ESC, Enter etc) hoping to clear it, but nothing does - eventually the display does return to normal operation.
I see similar behaviour after listing paritioins... the list hangs around but can't be cleared although it does eventually go away.
is this intentional? If so, the delay does seem a bit long - I'd personally prefer that a suitable keypress cleared the displayed info.

After downloaded a binary to the SD card the screen appears to be stuck at "Downloaded". I randomly press buttons (ESC, Enter etc) hoping to clear it, but nothing does - eventually the display does return to normal operation.

It is wainting for you to press "Ok" to go away

I see similar behaviour after listing paritioins... the list hangs around but can't be cleared although it does eventually go away.
is this intentional? If so, the delay does seem a bit long - I'd personally prefer that a suitable keypress cleared the displayed info.

It is waiting you press "Ok" to confirm the action, and go away

When I said I pressed 'Enter' as one of the keys I tried, I should have said 'OK'. Portraying it doesn't remove the message.

it is waiting for 2s after press the btn https://github.com/bmorcelli/M5Stick-Launcher/blob/0ba9a86b23edb1822c293c4295d23d6bf7f2f54c/src/onlineLauncher.cpp#L276
I will remove it from the code as well, because it is now waitinh user´s input

Ah, that explains it. Thanks.


stackingBehavior NEVER hide all buttons instead of ellipsize

Include the following:

Material Dialogs version: <IP_ADDRESS>
Device OS version: 6.0.1, 8.1
Device Manufacturer: Sony, Google
Device Name: Z3 Compact, AVD

Also, please wrap Java with correct syntax highlighting.
return new MaterialDialog.Builder(context)
                .customView(mCalculatorView, false)
                .positiveText(R.string.insert)
                .negativeText(R.string.cancel)
                .neutralText(R.string.clear)
                .autoDismiss(false)
                .backgroundColorRes(R.color.uni_form_cards)
                .onPositive((dialog, which) -> {
                   ...
                })
                .stackingBehavior(StackingBehavior.NEVER)
                .onNeutral((dialog, which) -> mCalculatorView.clear())
                .onNegative((dialog, which) -> dialog.dismiss())
                .show();

Reproduction Steps

text in resources are too long to show on one line in this language, so just use long text as placeholders
buttons will be completely missing, not on single line as ellipsized

Expected Result
ellipsized 3 buttons, not hiddne buttons
Actual Result
all buttons are missing
When we use adaptive, buttons are stacking on 3 lines...
Screenshots with NEVER and ADAPTIVE settings.

@afollestad  could you please look on this issue, or merge partial fix from me? ^

Will no longer apply with upcoming version 2.0


1.20.1NeoForgeERROR!

NeoForge version : 47.1.106
Error message: requested by 'persencefootsteps',Expected range[47.2.19,) Actual version 47.1.106
can u remake a version on 1.20.1 neoforge?

I made a pull request and got this working on neoforge 47.1.99  and also old forge 47.1.47 . only for MC 1.20.1  .    Links and more info here https://github.com/PaintNinja/Presence-Footsteps-Forge/pull/49


Update and document the http api to better match the core api

I think that using similar, or the same, routes that list the wiki in a header should work.

I am running (check any that apply, put an x inside the [ ] to check a box, like this: [x]):

[ ] Windows
[ ] OSX
[ ] Linux
[ ] Other

and using

[ ] The nodejs version
[ ] The single file executable

Before posting I read issue guidelines and:

[ ] I am using the newest version
[ ] The answer to my question isn't listed in the documentation or this isn't
a question
[ ] This is not a duplicate issue
[ ] I have not done anything that required me to set acceptance to
I Will Not Get Tech Support For This

The http api may change a bit in the future, but it is implemented now.


PhraseMate: Auto-capture phrase on write

From this forum discussion
Adds a new setting to the PhraseMate write panel: "auto-capture phrase". When it's enabled, PhraseMate will look for the first line in the selection to find the instrument and phrase settings commands, and write that phrase information to the pattern.
@bjorn-nesby I just took my best stab at it, I definitely would appreciate any feedback from you

[x] auto-capture phrase from pattern
[x] tranpose phrase data to pattern data
[ ] auto-capture multiple phrases in selection
[ ] auto-capture multiple phrases in track

Okay I added transposition! That was pretty simple. auto-capture and transpose phrase are a big chunk of the functionality I want, so I'm going to spend time playing with it and see if I run into any issues before trying to tackle the multiple phrases.

why has this been closed, @emuell ?


Properly disable uppercase literal suffix check

:notebook_with_decorative_cover: Description
Apparently clang-tidy 12 requires the trailing comma here.
:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
@lava check locally if this fixes it on your end as well.

so it seems like this is maybe a bug in clangd?

Might be, I only tested via clangd. Regardless, this fix is simple enough.


npm run prod - Unexpected token <

Hi,
when I try to run "npm run prod", the app partly renders and shows the next error:
"Uncaught SyntaxError: Unexpected token <"
Im using node v6.9.1 and local mongo is running.
Everything seems to work ok when using the dev script.
One more question is how should i host this on heroku? I just upload the whole project without changing anything?
Thanks in advance.

@JoleMile Thanks for reporting! I haven't used Heroku before, so no idea there.

@JoleMile Please pull and try again. If the problem persists, could you give me more details. Thanks!

Same issue here. /build/bundle.js(css) return a 404 page but build/bundle.js is a correct js file in fact.
Use koa-static instead of koa-static2 then it works fine.

And we need move
// Serve static files
config.http.static.forEach(staticRoute => {
    logger('app:static')(staticRoute.path)
    app.use(mount(staticRoute.url, serve(staticRoute.path)))
})

to the front
app.use(render)

Send a PR here: https://github.com/nightwolfz/mobx-starter/pull/16


How to run ./scripts/localization_online_visualizer.sh successfully?

System information

OS Platform and Distribution (e.g., Linux Ubuntu 14.04):Ubuntu16.04
Apollo installed from (source or binary):source
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0):3.5

Steps to reproduce the issue:
1.missing param.txt file while run './scripts/localization_online_visualizer.sh'';
2.Show nothing after run './scripts/localization_online_visualizer.sh'
Could you provide a "how to" document about how to run './scripts/localization_online_visualizer.sh' ? Thx a lot.

You can refer this document
How to Run MSF Localization Module On Your Local Computer

Closing this issue as it appears to be resolved. Feel free to reopen if you have additional questions. Thanks!


Fix code scanning alert - Potentially unsafe external link

Tracking issue for:

[ ] https://github.com/KalifyInc/website-findyourpet/security/code-scanning/1

https://github.com/KalifyInc/website-findyourpet/commit/0518767aa07eb32f41459d27d19d3d133b645f9b


Add clarification on folder name

Fixes need identified in issue #174.

not totally sure this helps newcomers to the kit. Think it might be adding information they don't really need and could confuse?

Yeah, I'm a little unsure of the need too. Is there a better way we could phrase it to make sure it doesn't confuse anyone?

I would find this confusing

OK - whilst it's good advice, on the basis of the above I'm going to close this request.


Permission denied when policy restricts to current version of secret

In the spirit of "least privileges", it's arguably a good practice restrict access to only the current version of a secret. Unfortunately the secrets-store-csi-driver-provider-aws runs into permission issues when doing so.
Steps to reproduce:

Create a role with a policy attached that only allows reading the current version of a secret:

{
    "Version": "2012-10-17",
    "Statement": {
        "Effect": "Allow",
        "Action": ["secretsmanager:GetSecretValue", "secretsmanager:DescribeSecret"],
        "Resource": "arn:aws:secretsmanager:eu-west-1:388662706032:secret:k8s-secret_foo_bar",
        "Condition": {
            "ForAnyValue:StringEquals": {
                "secretsmanager:VersionStage": "AWSCURRENT"
            }
        }
    }
}

See pod is stuck in ContainerCreating and events show PermissionDenied on reading the secret.
Remove the Condition from the policy, recreate the pod, watch it successfully read the secret.

The practice of restricting GetSecretValue to only the current version is not considered a best practice. The secret provides access and the privileges are provided by whatever the secret is connected to, such as a database. Either the old passwords are no longer valid or they provide access to the same set privileges (unless someone is repurposing secrets; which is not a good idea). There are also good reasons to let users access previous passwords (e.g. when a database is restored from an old backup).
The use of "secretsmanager:VersionStage" was shown in some of the API documents as an example of how to place restrictions on parameters and was not intended to become a common practice. We are working on replacing these examples in the docs.


gogoanime link

Test time!

It didnot work for me, but the updated url is the new url for gogo

Test time!

does it work?

It did not work for me, idk why, it should quite obviously work

It did not work for me, idk why, it should quite obviously work

you have to delete that appdata file to reset

v3 codebase be like 😭

It did not work for me, idk why, it should quite obviously work

you have to delete that appdata file to reset

I see, i'll try it out

Works!!


AudioCueEditor Feature Suggestion: More Robust Loop Editing

something like literally changing the numbers of the loop start and ends like Eternity Audio Tools
While dragging the points is cool and all, it also tends to be incredibly unprecise
Example: something like this + dragging the loop start and end points

Yeah that could be added. Never really done any loop stuff myself so this feature is a little lacking right now.
I've also noticed in the past that the loop points themselves aren't 100% accurate, at least with respect to the playback within ACE.

I've also noticed in the past that the loop points themselves aren't 100% accurate, at least with respect to the playback within ACE.

Yeah that's also an issue unfortunately

Better looping features was implemented in 0.9.
The inaccurate playback remains however.


Args now follow '--', to allow for multiple recipes on the command line

@aoeu: What do you think about this?
I've been finding that I want to run multiple recipes in one go far more often than I need to pass arguments to a recipe.
This changes it so that multiple recipes are allowed, and arguments only start after a '--'.
As far as I can tell, there is no way to do this sanely in the bourne shell, so it introduces a dependency on bash.

BTW, since discussion offline yesterday, I'm using this quite nicely in my workflow. just compile install run  xor just compile install resetCache run and many other variants typed on the fly, as needed. Good change!

Awesome ^__^


Why is number of points dependent on min distance?

Is this a real unbiased poisson disk algorithm? The gist seems to be to initialize a grid and set the grid cell size based on the specified min distance.
However one can easily imagine the need for a poisson disk where the specified min distance is reduced (to increase randomness) but the overall number of points stays the same. How does that figure into this? It seems like this can only generate one special type of poisson disk, not a real poisson disk with fair random distribution, isn't it?

Actually I think I just forgot conceptually what Poisson disc sampling even is. As I now understand, it inherently is not defined as "unbiased random sampling with specified min distance" and already implies a specific algorithm.

Actually I think I just forgot conceptually what Poisson disc sampling even is. As I now understand, it inherently is not defined as "unbiased random sampling with specified min distance" and already implies a specific algorithm.


vim.opt.ambiwidth = 'double' causes indent display issues.

Describe the bug
When setting the 'ambiwidth' option in NeoVim to 'double', the indent display becomes incorrect, and the text is no longer visible as the indentation gets deeper. This issue can be seen in the image below.

Normal.

If vim.opt.ambiwidth = 'double' is enabled.

To Reproduce
Add the following option to the lua file where the settings are written: vim.opt.ambiwidth = 'double'
Environment
OS: ArchLinux (on WSL)
NeoVim:
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3

(I'm a beginner Japanese user, so I'm sorry if the format is wrong.)

Hm this would be pretty difficult to support I think.
In your example, how should 2 space indentation look like, if the space character is already 2 cells wide, and adding the indent char would be 3? There is not enough room.
Could you use setcellwidths() to set the width of the utf8 char you use as space to 1? That would probably be the easiest solution.

I had never heard of the setcellwidths() option before. I actually tried it and found that the cause of this display error was "│"(0x2502), which I was able to solve by setting it as follows.
return{
  "lukas-reineke/indent-blankline.nvim",
  event = { "BufReadPost", "BufNewFile" },
  opts = function()
++    vim.fn.setcellwidths {
++      {0x2502, 0x2502, 1},
++    }

    vim.opt.list = true,
    vim.opt.listchars:append "space:⋅",
    --vim.opt.listchars:append "eol:↴"

    require("indent_blankline").setup {
      space_char_blankline = " ",
      show_current_context = true,
      show_current_context_start = true,
    }
  end
}

Awesome
I'll add this to the documentation


RAC Menu - put onClick on items

Provide a general summary of the feature here
I want to wrap the Menu component in an API design where onClick is passed to the MenuItems.
🤔 Expected Behavior?
onClick on MenuItem is called.
😯 Current Behavior
onClick on MenuItem is not called.
💁 Possible Solution
No response
🔦 Context
We are implementing a menu component and want to use react-aria-components Menu in its implementation.
💻 Examples
How our component is to be used:
<Menu>
  <MenuItem onClick={...}>Foo</MenuItem>
  <MenuItem onClick={...}>Bar</MenuItem>
</Menu>

🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Menu has the onAction prop:

onAction
(key: Key) => void
Handler that is called when an item is selected.

You can find this under Menu -> Events: https://react-spectrum.adobe.com/react-aria/Menu.html#menu-1
All of our collection components follow this API for things like action or selection. You can probably find some more discussion in issue or discussions, including this comment: https://github.com/adobe/react-spectrum/discussions/5058#discussioncomment-6973863

None of the RAC components support an onClick handler, so there isn't a clean way to do this, but if this is more of a case where you need to retrofit existing legacy components, you could do something like you have there.
Two cases the snippet you provided wouldn't handle:

supporting user-provided keys instead of just the auto-generated ones
supporting dynamic collections. You could get access to the collections (see Accessing state in the advanced customization guide).

Needing to include the click event itself would make this even tougher.
I would recommend creating a discussion to see if anyone else can chime in with recommendations.


Version 8.2.0 vs 8.2

The currently tagged version 8.2.0 resolves to ^8.2, that is, you may get greater versions (for instance 8.2.9). This can be misleading.
I'd create the following tags:

tag 8 => "concrete5/core": "^8" in composer (so that we install any 8.x version)
tag 8.2 => "concrete5/core": "^8.2" in composer (so that we install any 8.2 version)
... and so on

If you really have time, we could also add more specific tags (8.2.0, 8.2.1, ...), but that's not so necessary IMHO

The tags have been updated, they should track versions of concrete5/composer independent of the core.


Website doesn't support Firefox Android

I have the same issue. I don't get this error screen but when sending it gives me It was not possible to establish direct connection with the other peer. message


GPIO pin not mutablem bur´t no error

in example-embassy-rp23/src/bin/multicore.rs  we have
let led = Output::new(p.PIN_25, Level::Low);

hence led is immutable
but
#[embassy_executor::task]
async fn core1_task(mut led: Output<'static>) {
    info!("Hello from core 1");
    loop {
        match CHANNEL.receive().await {
            LedState::On => led.set_high(),

and in gpios.rs we see:
    pub fn set_high(&mut self) {
        self.pin.set_high()
    }

hence led in fact needs to be mutable
Why does this not result in a compile time error ?
Obviously this is due to #[embassy_executor::task]
But seems a bit confusing.
I unsuccessfully tried to make led mutable and unsuccessfully tried to use a "in-between" function that calls the Spawner to entangle the code in executer1.run()  (as I do spawn some additional task there ) .
Is this intended behaviour ?

it's working as intended, you're making it mutable here:
async fn core1_task(mut led: Output<'static>) {
                    ^^^
                    HERE

Yep. Thanks for answering !
This is obvious.
It feels odd that it's intended to convert an immutable variable to a mutable (in the other context).
I suppose I need to understand it something like "led is immutable with respect to core 0 but mutable with respect top core 1" which indeed is what the example is all about.
But given this, the core1 context does not even need to know about the led variable.
Hence I tried to "move" it into a function called by "run", but did not yet succeed.

in Rust it's the variables that are mutable or not, not the values. There's no such thing as a "not mutable LED".
You create a LED and you put it in an immutable variable, then you can't mutate it. However, you can move it to another variable that is mutable, and then you can mutate it.

I see.  (silly me)
I also was able to do the "in-between" function.
silly again, I forgot to claim the static lifetime for led.


TextView: Terminal Paste (Ctrl-V), pastes one char at a time, and has weird visual effects.

Describe the bug
Paste, pastes one char at a time, and has weird visual effects.
To Reproduce
Steps to reproduce the behavior:

Run Text Input Controls Scenario (or Editor)
Click on multi-line TextView
Go to https://baconipsum.com/?paras=5&type=all-meat&start-with-lorem=1
Copy contents to clipboard
Return to UI Catalog
Paste

Expected behavior
Text is pasted quickly, with no weird visual effects.
Screenshots

This was on Windows Terminal.
Note using the Terminal.Gui Paste (Ctrl-Y or context menu) works as expected.

I think that console paste will send each character one after the other to the terminal input stream.  Presumably either with a delay or flushing each character?  So each keystroke is read and processed one after the other with redraw after every one.

The Terminal.Gui context menu Paste will read the contents of the OS clipboard directly.  Terminal.Gui Copy/Paste has multiple implementations and some have dependencies (e.g. on linux the dependency is xclip).
Not sure what the most elegant way of dealing with this 'autotyping' is.

Does the TextView delay its updates to see if more keys are comming?
Does the driver delay releasing keys and use a flag to indicate when an event is part of a large collection of inbound keys?

I'm not sure either of these are great solutions.  But thought they were worth making to get the ball rolling discussing how we might address this issue.
FWIW I have seen this behaviour in other domains where 'auto typing' happens (e.g. keyboard macros or password managers).

For sure the Windows Terminal is sending each key from the clipboard to the running application. Terminal.Gui processes this keys in the same way if it's the user were typing. The only way to avoid this is delete the shortcut key Ctrl+V from the Windows Terminal. Unfortunately there are no option to disable it, so it have to be deleted. It is really a good idea the Windows Terminal provide an option do disable all the shortcuts when we are running the application, like the gnome for Linux has. For this reason when I'm debugging and testing something I prefer using the Host Windows Console for Windows and gnome for Linux. At least these terminal respects the setting in each Terminal.Gui drivers to disable the terminal defaults and thus only the Terminal.Gui application keystrokes are used.

Ok, so this is what it is. "As designed". Closing.


Refactor to separate TVM-specific bits from paged cached model

This is for enabling alternative model implementations like PT or TRT-LLM to run under mlc-serve.
No functional change.

or TRT-LLM to run under mlc-serve

Do you know if TRT-LLM has models having paged attention?

Yes, under https://github.com/NVIDIA/TensorRT-LLM/tree/main/examples there are many references to paged kv cache.


Initial support for cgroups v2

This is the initial support for cgroups v2. Currently either cgroups v1 or cgroups v2 are supported (no hybrid mode of v1 and v2).  If you want to test this you need cgroups v2 which you can enable by adding systemd.unified_cgroup_hierarchy=1 to your kernel cmdline if you do not already have it enabled.
Cpu and CpuSet controller are already implemented, rest can be implemented by someone else if there is interest. This is not yet completely finished, I have created some common functions that are used in the new stuff but the v1 cgroups need still be migrated to it. I am also investigating some weird behavior where child processes escape the cgroup.
Still I would like to merge this now and the rest of the work will be done in other PRs.

@Furisto
This is a cool PR. When I looked at the CI, it seemed that the integration test was failing.
Is this dependent on the CI environment? Is there any way to fix it?

Error: cannot get available controllers. "/sys/fs/cgroup/cgroup.controllers" does not exist

Could you apply the format?
https://github.com/rust-lang/rustfmt

I think what is happening is that cgroup v1 and cgroup v2 are mounted on the CI machine and that it returns CgroupV2 as filesystem type when it stats /sys/fs/cgroup. Thats why it tries to use /sys/fs/cgroup with v2 but it is actually located in /sys/fs/cgroup/unified.
Anyway the runtime-tools tests support only v1 as test target so I changed it to use v1 for the moment.

@Furisto
I think that in the future Youki should have our own integration test for this. For now, there aren't  integration tests for v2, so there isn't need to add integration tests.

Anyway the runtime-tools tests support only v1 as test target so I changed it to use v1 for the moment.

How about having markers (environment variables, etc.) to use only v1?

@utam0k
Sure, I have introduced an environment variable to control the behavior when both v1 and v2 are present. It will default to v1 as long as the v2 support is experimental. Later I will invert it to prefer v2.

@Furisto 👍

Later I will invert it to prefer v2.
Do I look okay to start a review?

@utam0k
Yes, it should be fine now to review.


[W6.6c][F11-B1] Law Keng Ying

Add in more User Stories

Accepting this PR after minimal review because another of your PRs for this tutorial was reviewed in detail.


Add aggregations?

Hello, Thank you for your hard work.
Maybe it would be a good idea to add some kind of aggregation api?
https://orm.drizzle.team/docs/select#aggregations
As an example: There are times when we need to count the number of rows without actually requesting the data as well (pagination, etc)
https://orm.drizzle.team/learn/guides/count-rows
Could count rows etc, be implemented with the use of the existing filters using the above? That way we can execute a query and get number of items that match it without doing a more costly operation.

Row count could be done by just adding the extra field to each type and then modifying query with count(*) if it is requested, expect that feature very soon. As for aggregation though, I am once again powerless because of current RQB limitations, however I'm all for it the moment RQB rolls out the required functionality.


⚠ ClamAV has degraded performance

In 8a21946, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 105 ms

Resolved: ClamAV performance has improved in 9ae6e33 after 534 days, 6 hours, 0 minutes.