WIP: Force account removal after uninstall

Issue link

https://github.com/tasitlabs/tasit/pull/298#discussion_r276225655
#295
Auto-close the issue?

Closes #295
Types of changes
New feature (non-breaking change that adds functionality)
Notes
The way that it is, all accounts will be removed because none of them have the param isFirstUse stored yet.
Maybe we can introduce storeIsFirstUse(false); only for now and check this var (and remove account) later? Any better idea?

Maybe we can introduce storeIsFirstUse(false); only for now and check this var (and remove account) later? Any better idea?

This works for me, with a TODO to remove it soon.

Maybe we can introduce storeIsFirstUse(false); only for now and check this var (and remove account) later? Any better idea?

This works for me, with a TODO to remove it soon.

We could get fancy with checking app version from the manifest to determine behavior, but that's too much overhead for now

I think we can remove WIP from the name of this PR

I renamed the PR to say "prep for" because only once we un-comment that line in a future PR will the feature be done. For now it's preparation for that.


Support for RSA-OAEP-256

👋 Is there any plan to support RSA-OAEP-256?

I have no plan.

Is there a reason why not? Very few seems to support it.


PHP 8.0 compatibility

PHP 8.0 was released 3 days ago.
I would be great to add the compatibility.

I forked the library, made it compatible for PHP 8 and fixed some tests:
https://github.com/enso-media/ReverseRegex
Released it also as v0.2 on Packagist:
composer require enso-media/reverse-regex
You can use it as a drop-in replacement for the original library.
All credits stays of course for @icomefromthenet for his great work with this lib.

I forked the library, made it compatible for PHP 8 and fixed some tests:
https://github.com/enso-media/ReverseRegex
Released it also as v0.2 on Packagist:
composer require enso-media/reverse-regex
You can use it as a drop-in replacement for the original library.
All credits stays of course for @icomefromthenet for his great work with this lib.

I close this PR, the library is abandoned.
#12


Some kind of failure during Github automated attempt to build AsciiDoc spec?

See here for some logs: https://github.com/p4lang/p4-spec/actions/runs/11830509745

Working on sorting this out here:

https://github.com/p4lang/p4-spec/pull/1335


Configuration Server Run-Mode functionality

This PR accomplishes two things:
1.) Setting up init folder structure to a more domain-driven design - this will help with segregating/sharing code across future run modes. (coming in other PRs). I've looked through both new mongodb/db structure, and init/pre-init structure to make sure nothing was missed, and it seems to be a mostly organizational change. The exception seems to be s2i - which I was not sure of the intent of this, so I didn't attempt to incorporate it. I might need someone with the backstory to add the commit adding that functionality. p.s why extend the image with s2i as opposed to just a plain docker build? I guess I would expect the tool to be used more exclusively in quick turn-around type uses; as opposed to extension of a middleware image - where I would expect iterations to be much more well defined, and steady.
2.) Added run-configsvr
Just a note, https://github.com/jbornemann/mongodb-on-os I plan on submitting to container-quickstarts as an example of how all of these run modes fit together on OpenShift in a sharded Mongo environment.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@omron93 this is probably a good first PR to start on, in terms of splitting up https://github.com/sclorg/mongodb-container/pull/255

There are some new features which are good. But also there is a lot of changes (naming, structuring,...) which was done probably by writing code independently. It would be good to have reason for these changes - useless rewriting of code can bring only hidden bugs ;-)

The intent was to structure the code in a way such that functions are shard, or segregated based on run-mode - to easily find functions, and reduce duplication. With the introduction of new run-modes, code organization problems are uncovered that weren't previously there.

s2i
No strong feelings about having vs not having it. Personally, if I were a user wanting to extend the image; I would just extend it via Dockerfile, add scripts, and change the entrypoint - but that is just personal taste. To me, s2i is more of a tool for deploying applications on an existing image within a mostly pre-defined lifecycle (assemble, save-artifacts, run) - although, yes, you can extend it.

Also I have idea about config server

Sure, sounds good to me. Those two things seem to be likely to change together, so probably does make sense to consolidate.

rs_remove()

I delegate to @msurbey again for this one. No strong feelings.

[test]

@omron93 just FYI, if it suites your fancy - feel free to help move these PRs forward. I likely won't have a whole lot of dedicated time to this.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

mongodb container is not maintained any more in this org. closing.


Fix Colab third party packages install

This fixes #344 (Likely due to Colab updating and breaking something)
It uses condacolab to install mamba which speeds things up dramatically.
It also downgrades PyYAML from 5.4.1 to 5.3.1 because of this issue: https://github.com/yaml/pyyaml/issues/724

fixed already by #372


Create uk_ua.json

Hello! Sorry to bother you, but I recently revamped the mod and was wondering if you could do the uk_ua translations for the newly implemented stuff? I understand if not, but thank you if you do!

yeah sure


Export in AVIF

This file is produced by XChart/OpenHAB.  It is 45kb.  If I convert it to AVIF format, then the new size is 10kb.
Please consider extending the software to support export in AVIF.

This library doesn't do any of the export stuff. openHAB uses BitmapEncoder.saveBitmap for that, so your request would have to be implemented in the JDK somehow (or use a library that can turn a Bitmap in an AVIF file).

AFAIK the AVIF format isn't even supported yet when calling getImageWritersByFormatName(), so support for that needs to be added first in Java.

So plugins have to be installed, like https://github.com/haraldk/TwelveMonkeys/, to generate SVG or WebP by XChart?


💡 [FEAT] The clipboard functionality seems to be missing in the new version

The ability to cut/copy and paste todos and workspaces seem to be missing. Makes it very hard to move tasks across different lists
Describe the solution you'd like   🤔
clipboard functionality (cut, copy + paste). + keyboard shortcuts for the same.
Describe alternatives you've considered  🧐
havent considered any alternatives.
Additional context 📝
...

HI @noelzubin,
Ah yes. Now that you mention it, My plan was to remove it from the core package and introduce it back as a plugin. Totally forgot. My bad. I'll start working on it now.
Thanks. Apart from that, lemme know your thoughts on the newer version :)

Looking at the Api exposed. I dont see an api that lets me add a Todo

is this still being worked on?
i just noticed this myself when trying to use "y" to copy and "p" to paste (assuming that's what you guys where talking about)

Hi @El3ctricFX!

is this still being worked on?

I think I should be able to do this today. Its just that the week has been so fkin hectic that I couldnt find the time to do this stuff

i just noticed this myself when trying to use "y" to copy and "p" to paste (assuming that's what you guys where talking about)

There are 2 kinds of copy/yank, One if to copy the description to the system clipboard which is currently supported,
The other one is to actually copy paste the todos completely which will be available via plugin


Different text under Original Quote for Source and Target

In Ruth, found two instances where the Source and Target Original Quote has variations.

Not seeing this issue in Beta-13. This has been fixed with the Branch workflow implementation.


1.x unable to finish w3af dependency 3/3

openssl/evp.h missing, later python.h missing, later libxml/xmlversion.h missing, need to add provisioning for libssl-dev and python-dev and libxml2-dev and libxslt1-dev.
recommend removal or review of provisioning/requirements, I had to remove role 'stouts.python' as during test run trying to get everything working, this got in the way.

also need to add a call to:
pip install --upgrade pip


Identify pickle files

Is your feature request related to a problem? Please describe.
Recently, I came across Python pickle files used in a suspicious package, and I'd like to eventually use some analysis for pickles. To implement any in an AL service, AssemblyLine would need to correctly identify pickle files. Currently, the identification is left as unknown.
Describe the solution you'd like
I looked at the definition of the protocol, and when it looks impossible to detect every pickle version, newer versions (since Python 2.3) have specific start bytes indicating protocol version, and later (since Python 3.4, default in 3.8) also uses frames what seems to introduce another static byte.
See following Python source:

https://github.com/python/cpython/blob/dcfe198867e4fbfeb7a1661adce45ae345fbe5bc/Lib/pickletools.py#L2123-L2133 - PROTO opcode
https://peps.python.org/pep-3154/#framing - PEP 3154 explaining structure of pickle in protocol version 4
https://github.com/python/cpython/blob/dcfe198867e4fbfeb7a1661adce45ae345fbe5bc/Lib/pickletools.py#L2365-L2369 - pickletools enforces frames on optimization in version >= 4
https://github.com/python/cpython/blob/dcfe198867e4fbfeb7a1661adce45ae345fbe5bc/Lib/pickletools.py#L2150-L2160 - FRAME opcode

Thus, every object pickled in protocol v4 starts with 80 04 95, and in v5 80 05 95. So far, I haven't found any other file format using those bytes as header. Given that AssemblyLine doesn't assign any type to pickle files, it looks safe to introduce those bytes as identifiers.
I'd suggest adding them to the file identification.
Describe alternatives you've considered

It looks like 80 02 and 80 03 may also be unique, but I feel unsure about using just them. In addition, v4 has been the default for years, so I don't think it's necessary.
Every pickle should end with STOP opcode; however, I don't think it makes sense to check this.

Additional context
For some reason, identifying pickles isn't popular yet. The python module does not perform any special verification - it just breaks once an unknown opcode is found.

This issue fell between the cracks. Sorry for such a delay.
This PR should allow you to identify the different versions of pickle files, as resource/pickle/*. I decided to add the version to the filetype as it was easy. It may not change much, but I was thinking that it could be interesting to have the precision in the filetree on the submission details page, or if someone wants to search for specific versions.

Thank you!


Fix charting waveform

This fixes the audio waveform not showing up in the chart editor, which was broken after the new caching method was added (returned a sound file instead of a text path).
There is probably a way to get an audio buffer straight from a Sound file instead of having to use a path, but I'm not smart enough for that.

hey i am gonna merge this to my psych engine but better github page that ok with you?

hey i am gonna merge this to my psych engine but better github page that ok with you?

You can go ahead


Consider adding MedicalZoo Pytorch

Hello, would you please consider adding MedicalZoo Pytorch  in the medical section?
It is a deep learning framework for multi-modal 2D/3D medical image segmentation.
Link: https://github.com/black0017/MedicalZooPytorch
Great work by the way!
N.A.

Done! :) Very useful library.
https://github.com/ritchieng/the-incredible-pytorch/commit/97de767e13c718e921ff69041df8ae8b6ca245c7


Hyper 2.1.0 doesn't open Ubuntu 18.04

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Ubuntu 18.04

Hyper.app version: 2.1.0

Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere): NA

The issue is reproducible in vanilla Hyper.app: True

Issue

I have a fresh Ubuntu 18.04 install. I installed hyper v2.1.0 from the releases page.
On running hyper -v I get,
A JavaScript error occurred in the main process
Uncaught Exception:
Error: EACCES: permission denied, open '/home/raghavkukreti/.config/Hyper/config.json.3982131248'
    at Object.fs.openSync (fs.js:652:18)
    at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:172:20)
    at Function.writeFileSync [as sync] (/opt/Hyper/resources/app.asar/node_modules/write-file-atomic/index.js:180:17)
    at ElectronConfig.set store [as store] (/opt/Hyper/resources/app.asar/node_modules/conf/index.js:94:19)
    at ElectronConfig.Conf (/opt/Hyper/resources/app.asar/node_modules/conf/index.js:38:14)
    at ElectronConfig (/opt/Hyper/resources/app.asar/node_modules/electron-config/index.js:11:3)
    at Object.<anonymous> (/opt/Hyper/resources/app.asar/config/windows.js:9:13)
    at Object.<anonymous> (/opt/Hyper/resources/app.asar/config/windows.js:24:3)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)

[3558:0923/060556.952017:ERROR:cache_util.cc(134)] Unable to move cache folder /home/raghavkukreti/.config/Hyper/GPUCache to /home/raghavkukreti/.config/Hyper/old_GPUCache_000
[3558:0923/060556.952054:ERROR:disk_cache.cc(132)] Unable to create cache
[3558:0923/060556.952058:ERROR:shader_disk_cache.cc(604)] Shader Cache Creation failed: -2

A small update on this. I chown'd the .config folder and Hyper started opening.
Is this a problem during installation?
Some NPM permissions?


Explicitly stop timer if context finishes early

Previously, if the context finishes early, timeSleeper returns the
context's error and ignore the timer.
This has no correctness problem, but the GC will only be able to collect
the timer after the timer expires.
This commit explicitly stop the timer, so that the GC can claim it
immediately.

Current coverage is 85.32% (diff: 100%)

Merging #37 into master will increase coverage by 0.13%

@@             master        #37   diff @@
==========================================
  Files             4          4
  Lines           216        218     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            184        186     +2
  Misses           24         24
  Partials          8          8

Powered by Codecov. Last update ed6ab75...f307889


Update bbedit to 11.6.7

After making all changes to the cask:

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

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Thank you for the contribution. It was merged directly as d5bd9ee2b447ef0c50b4a037b3d4518d0203ff44. Your contribution is still credited to you.


VM status is not updated after resetting the password

Steps to reproduce

Disable "Save VM passwords by default" in settings
Reset password of VM
Press Save password
Should appears dialog which asks about autosave passwords. Press any button.
Exit from dialog

Actual result

VM has a stopped state
Expected result

VM has a running state
Environment

Browser:
- [ ] Chrome version XX
- [ ] Firefox version XX
- [ ] Safari version XX
- [ ] Opera version XX
- [ ] IE version XX
- [ ] Edge version XX

Others:

Connected feature ID

settings_password_saveall
vm_password_reset
vm_password_save
vm_card
Screenshots

PR #1150 fixes this issue

Please, test this issue on master

Tested on Master


Improving Storybook's Design

Since we use our storybook documentation as a landing page of our project, we want to improve the design and sync design language with our design system.

Here you can reach the storybook design. We are still working on the images that are on the welcome page and component list page.


“Keystore was tampered with, or password was incorrect” when attempting to generate signed apk

Version 0.29.1
Android Only
I start off by running keytool -genkey -v -keystore my-release-key.keystore -alias my-key-alias -keyalg RSA -keysize 2048 -validity 10000 in my project directory.
Then I put my-release-key.keystore in my project's android/app directory.
Now, there isn't a ~/.gradle/gradle.properties directory like facebook says there is, but there is a gradle.properties in the android folder, so I set up my gradle variables and my gradle.properties ends up looking like this:
android.useDeprecatedNdk=true
MYAPP_RELEASE_STORE_FILE=my-release-key.keystore
MYAPP_RELEASE_KEY_ALIAS=my-key-alias
MYAPP_RELEASE_STORE_PASSWORD=password123
MYAPP_RELEASE_KEY_PASSWORD=password123

Next, I open up android/app/build.gradle and add my gradle variables and set it for release like this:
...
android {
    ...
    defaultConfig { ... }
    signingConfigs {
        release {
            storeFile file(MYAPP_RELEASE_STORE_FILE)
            storePassword MYAPP_RELEASE_STORE_PASSWORD
            keyAlias MYAPP_RELEASE_KEY_ALIAS
            keyPassword MYAPP_RELEASE_KEY_PASSWORD
        }
    }
    buildTypes {
        release {
            ...
            signingConfig signingConfigs.release
        }
    }
}
...

And then I finally run cd android && ./gradlew assembleRelease and get the error in the title saying that my Keystore was tampered with or my password is correct.
Because I've done this cycle so many times, I'm inclined to believe that I am somehow tampering with the keystore.

i have same problem。When i use AS build release apk，AS is ok。i try many time。

I am facing the same issue.  Using Build -> Generate Signed APK in android studio works, but putting the config in build.gradle doesn't.

~/gradle/gradle.properties means
c:/Users/UserName/.gradle/gradle.properties (Windows)
That file isnt local to Ur project. Its available to all Ur projects which use gradle.
Works for me if I edit the above file instead of files in project.
Same steps as U. Only change being location of gradle.properties.

@IgorGanapolsky How?  What directory do you have to be in to generate the APK with Android Studio?
@ashwin0003 Could you elaborate?

I'm going to close this issue since this is an issue specific to React Native. Feel free to continue discussion here or move to StackOverflow.

~/ means Ur user home directory.
For windows 7/10 its C:\Users\UserName
For Ubuntu its /home/UserName
Setting up gradle variables
Place the my-release-key.keystore file under the android/app directory in your project folder.
Edit the file ~/.gradle/gradle.properties and add the following (replace ***** with the correct keystore password, alias and key password),
MYAPP_RELEASE_STORE_FILE=my-release-key.keystore
MYAPP_RELEASE_KEY_ALIAS=my-key-alias
MYAPP_RELEASE_STORE_PASSWORD=*****
MYAPP_RELEASE_KEY_PASSWORD=*****
These are going to be global gradle variables, which we can later use in our gradle config to sign our app.

The first step is at Ur react native project folder.
The 2nd step is at the user home folder.
Note his explanation about global gradle variables. 1st step project folder. 2nd step global gradle installed folder.
Main point being ~/ means C:\Users\UserName\ for windows and /home/UserName/ for Ubuntu.
Example.
My folders
Windows 10.
React project e:\React\AwesomeProject
Windows drive c:
Setting up gradle properties.

Placed keystore file at e:\React\AwesomeProject\android\app
Edited c:\Users\Ashwin.gradle\gradle.properties

U messed up the 2nd step. U edited the wrong file. U eidted e:\React\AwesomeProject\android\gradle.properties. Ur supposed to edit c:\Users\Ashwin.gradle\gradle.properties
No probs with react-native. U didnt understand the meaning of ~/. That is a pretty basic thing to know.

I was trying to build based on a project-specific gradle.properties (i.e. at projectDir/android/gradle.properties), but a global gradle.properties I had configured during a previous project was being used instead. The fix for me was to delete my global gradle.properties (i.e Users/YourName/.gradle/gradle.properties)

I have the same problem, did create multiple keystores over and over again but still the message.
'Keystore was tampered with, or password was incorrect'. The gradle.properties is located when building and is finding the correct keystore. I can list the entries of the keystore.
Version React Native: 0.46.1
OS: Win 10
keytool -genkey -v -keystore test.keystore -alias test -keyalg RSA -keysize 2048 -validity 10000
gradle.properties
MYAPP_RELEASE_STORE_FILE=test.keystore
MYAPP_RELEASE_KEY_ALIAS=test
MYAPP_RELEASE_STORE_PASSWORD=mypassword
MYAPP_RELEASE_KEY_PASSWORD=mypassword

##\android\app>
Enter keystore password:

Keystore type: JKS
Keystore provider: SUN

Your keystore contains 1 entry

test, Jul 10, 2017, PrivateKeyEntry,
Certificate fingerprint (SHA1): ##:##

After two years, I solved this by removing ~/gradle.properties
Completely forgot about posting this lol


docs: modernize patterns in subset of samples

This doesn't capture everything, but it does get the quickstart and a few other samples.

@JustinBeckwith, thank you for doing this!


mismatch owing to pretrain model_final.pkl.

After i download the model_final.pkl,  runing  the model shows the issue, like "expected 4-dimensional weight (64,3,7,7) but got input size of (4,10,3,320,320)".

It seems that you set more than one query image in one GPU. The bs should be equal to the GPU number.

thanks for your reply, when i evaluate via tools/test_net.py, there is keyerror: devkit_directory. after checking, i found lossing the index of fsod dataset. so could you advise me the information about the index?


Consider enforcing no leading underscore in tag keys

I would like to consider ensuring that the storage engine does not accept any tag keys with leading underscores, essentially leaving the _ prefix as a private key space for internal series key information.
With that as an invariant, we could make certain performance improvements around reading and parsing keys.

This was resolved via https://github.com/influxdata/influxdb/pull/12391

measurement overwrite _measurement tag in flux table.
// write
any,_field=ff,_measurement=mm,_tag=tt foo="bar"

// query: "from(bucket:"purple")|> range(start:-10h)"
,result,table,_start,_stop,_time,_value,_field,_measurement,_tag
,_result,1,2019-05-31T08:09:08.867394Z,2019-05-31T08:19:08.867394Z,2019-05-31T08:19:00Z,bar,foo,any,tt

at least, _start,_stop,_time,_value,_field,_measurement these tag keys should not be accepted.

https://github.com/influxdata/influxdb/blob/530ceb9cbdae077302f37201bf3ef6cb8192e4fc/storage/readservice/cursor.go#L143-L153

@zhulongcheng thanks for the issue! Hmm I think this is an issue in the read side. @stuartcarnie can you take a look?

@e-dard the issue I see here is that _measurement was allowed via a write, and therefore stored as a real tag. Specifically, the series key is stored as:
<org><bucket>,\x00=any,_field=ff,_measurement=mm,_tag=tt,\xff=foo

which should never be allowed.
When a read occurs, the \x00 and \xff bytes are remapped to _measurement and _field respectively:
https://github.com/influxdata/influxdb/blob/530ceb9cbdae077302f37201bf3ef6cb8192e4fc/storage/readservice/cursor.go#L143-L153

@stuartcarnie yeah seems about right. I think then we’ll have to check and block these. I’ll put this on the back log.

don't close.


replicate entries in pr2 4.12.0

I'm getting replicate entries in v 4.12.0. Taxa entry is identical, but sequence entry is different. I'm trying to use the mothur fasta and tax files in a pipeline to train PR2 for use in QIIME2 ASV determination and tax assignment - these duplicate entries throw an error.
Replicate entries: KC486130.1.1651_U, AB275104.1.1590_U, EU087243.1.813_U, EU087247.1.847_U, FJ459738.1.1634_U
I know this was an issue from the last release and it was updated quickly. Is there an additional script you can include with each release that performs this de-duplication?
https://github.com/shu251/db-build-microeuks

Hi Sarah
Thanks for pointing out this issue (this was due to repeated sequences in the pr2_sequence table).  I have added one more check to ensure that this will not occur in future releases.
I have fixed the files from the 4.12.0 release as well as the pr2database R package.  Cheers.  Daniel


Update xbox-360.md

Fixing the word "Kink" being used instead of "Kong" in one spot and the misspelling of "suspicious" as "suspicius".

Well spotted!


Certify the Kabanero Operator

Build upon #2, but formally certify.

OperatorHub certification is accounted for as part of #2 and #4


CESM2 ceda-cc report

The following errors were raised from sampling the experiment ssp370-126aer for CESM2 data.
Incorrect mip_era
For all files, the Global attribute mip_era is 'CMIP6' when it should be 'RAMIP' according to the tables.
FAILED:: Global attributes do not match constraints: [('mip_era', 'CMIP6', "['RAMIP']")]
Variables missing from table RAMIPday
The following variables not in table RAMIPday:
hus850
ua250
ua500
ua850
va250
va500
va850

Also, the experiments have capitalised regions in filenames (and within files) which do not match the CV files:
e.g. ssp370-SAS126aer vs ssp370-sas126aer in MIP tables.


NIFI-4559: Add non-zero status relationship to ExecuteStreamCommand

Sorry about the whitespace changes, I must have slightly different (but still compliant) formatting settings in my IDE.
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@mattyb149 As mentioned, I tried this out and got it to work. So I see no reason it cannot be merged.

@MikeThomsen Thank you for your review! I think I tested it fairly well also, so with your +1 I can merge this in.


Should we minify our library packages?

I wonder whether we should add or not minify: true to our tsup config?
Most libraries we used ship unminified. It does help because it's easy to open the source code and understand how it works and even add debugging instructions.
What do you think?
The downside is that the applications build scripts will have to minify the code which might increase the build time. Still, I am not sure if application build scripts minification process ignore "already minified" libraries.

No it shouldn't! Our tsup config has already been updated accordingly.


Investigate npm/node.js-free possibility

I see that the project is depending heavily on npm packages and dependencies as well as sbt. It would be easier to use if only sbt was required to build and use the library.
Do you think that is possible? All dependencies from npm/bower is possible to find/import to webjars.org and get from sbt.

for mobile examples we must use npm/node.js
for web module examples we can replace with sbt but from my previous experience all js components/frameworks don't publish to webjars and npm.webjars doesn't work out of the box in few cases. i personally prefer npm/node.js(easy to learn and we must need this when developing mobile apps)  for js deps. That being said i am happy to take PR's for template projects(sri-web-sbt ... )  based on sbt,

Yes, the infrastructure and scripts from react tools are probably useful. But we might get them in jars and connect them using sbt targets.
Yes, imports sometimes fails, but they fix the issues quickly: https://github.com/webjars/webjars/issues/1246 and https://github.com/webjars/webjars/issues/1247

And upstream FB has fixed this issue for next version: https://github.com/facebook/react/issues/5461

Yes, the infrastructure and scripts from react tools are probably useful. But we might get them in jars and connect them using sbt targets.

I must admit that i'am sbt noob , not sure whether its an easy task or not especially for react-native tool chain and relay-babel-plugin!. But its good to have templates with sbt only, looking forward to u r findings ;)

@mkotsbak I've update the web-template to remove npm need :). Take a look and tell me what you think:
https://github.com/onilton/sri-web-template


BlinkAPI feature request – enable per account/token rate limiting

During stress testing of BTCPay / Bitcoinize / Blink payment setup, the API rate limits were hit, prompting an idea / request to allow for rate limiting based on account or token.

hey @nicolasburtey I would like to work on it. could u assign it to me?

@vr-varad its a hard feature to work on. I would recommend starting with something easier.


GPD Win Max 2 (2023) BIOS menu typecopy -- inclusion in this wiki?

Ahoj,
I made a typecopy of the BIOS menu structure of the GPD Win Max 2 (2023) (AMD7840U), BIOS version 0.35.
I made an announcement ↗ here on reddit, and I asked where I can put it as a wiki. It was suggested ↗ here that I contact you, @SabrinaRDC, because you made this GPD Win Max 2 Wiki (github.com/SabrinaRDC/WM2-Help).
So hereby I contact you -- lacking of other means to reach you (I cannot find an email of yours, and I don't use Telegram or Discourse) I try it this way.
What do you think of including the GPD Win Max 2 (2023) menu typecopy into this wiki? Currently, I have uploaded it ↗ here.
Regards!

This looks awesome! I'll add it to the guide, do you want me to credit you in any other way other than your github?

Ahoj,

This looks awesome! I'll add it to the guide,

do you want the raw Markdown?
And: Can it be added so that it is easily community-editable (in the web browser like other wiki pages, utilising the github wiki function)?

do you want me to credit you in any other way other than your github?*

You could instead refer to reddit.com/u/dreieckli.
Regards!

OK, I now made it a wiki at codeberg.org:
https://codeberg.org/dreieck/GPD_Win_Max_2_2023_BIOS_menu_documentation/wiki/01_-_Overview
.. just link there.
Regards!


chore(travis-coveralls): updated travis file for coveralls support

Changes Unknown when pulling 07a2e46c7b3c9f6a486208fdd5183316fa5439af on doc-update into ** on master**.

Changes Unknown when pulling 8fd0500185ec01f5b2be94218eb9e912c50a61d6 on doc-update into ** on master**.


Bump swift-argument-parser dependency version

swift-argument-parser 0.4.0 and up didn't seem to introduce any changes that break XCLogParser.
Though I'm not sure how extensive and reliable the tests are, both swift build and swift test seem fine.

This is already bumped to 1.2.0. See https://github.com/MobileNativeFoundation/XCLogParser/pull/177

#177 Came a year and a half later. What's the preferred method of bringing attention to patches in the future?


Cannot create a database which name starts with a number

When I run:
"create database 1test"
I get:
Syntax error: Wrong number
(version <IP_ADDRESS> (official build))

You have to quote database name like this:
CREATE DATABASE `1test`

or this:
CREATE DATABASE "1test"

I tried single quotes like '1test' and it did not work, did not try double quotes.
Using double quotes solves it, but it is still sort of inconvenient because tables always need to be referenced like "1test".table1 instead of 1test.table1, but that's minor.
Thanks!


The password of root user in container 'apollo_dev_'

Descriptions
I met the same problem as #9219 mentioned. For some reason, I need to install the network proxy tools inside the container, which required the password of root user in container.
Does anyone know this password?
Steps to reproduce the issue:

In host:

cd <APOLLO_ROOT_DIR>
sudo service docker start
bash docker/scripts/dev_start.sh
bash docker/scripts/dev_into.sh

In container apollo_dev_<username>:

sudo apt update, with outputs:sudo: effective uid is not 0, is /usr/bin/sudo on a file system with the 'nosuid' option set or an NFS file system without root privileges?

apt update, with outputs:Reading package lists... Done
E: Could not open lock file /var/lib/apt/lists/lock - open (13: Permission denied)
E: Unable to lock directory /var/lib/apt/lists/

su root, need password.

System information

OS Platform and Distribution: Ubuntu 20.04.3 LTs with kernel 5.4.0-91-generic
Apollo installed from (source or binary): binary
Apollo version (3.5, 5.0, 5.5, 6.0): 5.5
Output of apollo.sh config if on master branch: crashed, with following info:

guoqing@in_dev_docker:/apollo$ bash apollo.sh config
System check passed. Build continue ...
[WARNING] ESD CAN library supplied by ESD Electronics does not exist. If you need ESD CAN, please refer to third_party/can_card_library/esd_can/README.md.
Traceback (most recent call last):
  File "modules/tools/configurator/configurator.py", line 162, in <module>
    curses.wrapper(main)
  File "/usr/lib/python2.7/curses/wrapper.py", line 43, in wrapper
    return func(stdscr, *args, **kwds)
  File "modules/tools/configurator/configurator.py", line 151, in main
    conf = Configurator(stdscr)
  File "modules/tools/configurator/configurator.py", line 63, in __init__
    moduleconf.parse_from_file()
  File "/apollo/modules/tools/configurator/ModuleConf.py", line 45, in parse_from_file
    mod = importlib.import_module(self.proto_file)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
ImportError: No module named modules.canbus.proto.canbus_conf_pb2

Supporting materials (screenshots, command lines, code/script snippets):
The screenshot:

Thanks for your reply! I tried the password but it didn't work.
However, I found a way to login container as root and no password required.
Just change docker/scripts/dev_into.sh:
docker exec \
    -u $USER \
    -e HISTFILE=/apollo/.dev_bash_hist \
    -it apollo_dev_$USER \
    /bin/bash
xhost -local:root 1>/dev/null 2>&1

to:
docker exec \
    -u root \
    -e HISTFILE=/apollo/.dev_bash_hist \
    -it apollo_dev_$USER \
    /bin/bash
xhost -local:root 1>/dev/null 2>&1

The basic usage of Docker :joy:  I forgot it


feat: update aws sdk go to latest

feat: update aws sdk go to latest

@angelo-chan can you help merge this? I didn't have the permisison. Thanks a lot


Migrating to input/output facade

This is not complete migration as we are missing operations like

removeSync is from fs-extra, fs.rmdir has recursive:true option, but node 12 onwards.
unlinkSync needs to be added to broccoli-output-wrapper https://github.com/SparshithNR/broccoli-output-wrapper#apis
symlinkOrCopy is one more method that is used extensively. Do we want to support that as well?

@SparshithNR good points!
for 1) let's implement/provide a fs.rmdir + recursive: true polyfil in FSMerger
for 2) let's add it
for 3) let's add it

Depends on https://github.com/SparshithNR/broccoli-output-wrapper/pull/4

@SparshithNR is this pending update of broccoli-output-wrapper?

broccoli-output-wrapper has now all the operations mentioned above

This is not complete migration as we are missing operations like

removeSync is from fs-extra, fs.rmdir has recursive:true option, but node 12 onwards.
unlinkSync needs to be added to broccoli-output-wrapper https://github.com/SparshithNR/broccoli-output-wrapper#apis
symlinkOrCopy is one more method that is used extensively. Do we want to support that as well?

Windows failures are due to fixed, but not yet released, node version. Context: https://github.com/broccolijs/broccoli-funnel/issues/119

released as v3.0.1 🎉


Dark Sky platform crashes HASS: EOF occurred in violation of protocol

Home Assistant release (hass --version): 0.38.3
Python release (python3 --version): Python 3.5.3
Component/platform: Dark Sky
Description of problem:
I have no automation based on the Dark Sky platform.
Update interval: 30 minutes
No issues with my internet connection. Other platforms perform just fine.
It already happened a couple of times since I updated to 0.38.3.
Additional info:
17-02-18 23:22:06 ERROR (Thread-11) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)
17-02-19 00:55:01 ERROR (Thread-10) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)
17-02-19 01:23:55 ERROR (Thread-3) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)
17-02-19 01:36:05 ERROR (MainThread) [homeassistant.core] Error doing job: Fatal read error on socket transport
Traceback (most recent call last):
  File "/usr/lib/python3.4/asyncio/selector_events.py", line 582, in _read_ready
    data = self._sock.recv(self.max_size)
TimeoutError: [Errno 110] Connection timed out
17-02-19 02:23:41 ERROR (Thread-6) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)

An update:
For an update interval of 30 min why am I getting 700 requests per day instead of 50 or so?*
Is it a request per parameter/monitored condition (e.g. - apparent_temperature)? Even that doesn't make any sense since I include all parameters.

  - platform: darksky
    api_key: xxx
    update_inverval:
      minutes: 30
    monitored_conditions:
      - apparent_temperature
      - cloud_cover
      - daily_summary
      - ...

I seem to be getting tons of these lately myself.
17-02-19 04:19:52 ERROR (Thread-7) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)
17-02-19 04:34:19 ERROR (Thread-3) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)
17-02-19 07:13:25 ERROR (Thread-6) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)
17-02-19 08:41:45 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/home/hass/deps/aiohttp/server.py", line 282, in start
    self.transport.write_eof()
  File "/usr/lib/python3.4/asyncio/transports.py", line 111, in write_eof
    raise NotImplementedError
NotImplementedError
17-02-19 08:50:30 ERROR (Thread-5) [homeassistant.components.sensor.darksky] Unable to connect to Dark Sky. EOF occurred in violation of protocol (_ssl.c:600)

Home Assistant release (hass --version): 0.38.2
Python release (python3 --version): Python 3.4.2

I got this error today,
I decided to just have my program catch the exception and try again in a few seconds.

Same errors in my logs and high usage in console.

An employee of Dark Sky replied about this in another issue, going to close this in favor of that one: #5110


V2 : Add center parameter to cylinderElliptic / cylinder primitives

While converting some examples to V2, it became apparent that centering of cylinders would simplify the creation of designs.  I like the idea of removing the center parameter from the primitives. This will simplify the parameters/definitions for each primitive, which will help later in future plans. In addition, the center function as well as the translate functions are easily available.

@z3dev as said in hangouts, I do think that is the way forward, BUT is there any case where we need to store the center in the primitives data for any reason (I cannot think of any valid reason to do so)

I don't think there's any reason to store the center information as part of the primitives definition. We just need to be consistent that all primitives are 'centered' at the axis, always. This is required for symetric rotations anyway.
I think we should consider a few more measurements just to help the users. See Issue #23

measureCenter()  ; current method of finding midpoint between min and max points
measureMidpoint() ; average of all points
measureCenteroid ;  "centre of gravity" or the "center of mass" see http://paulbourke.net/geometry/polygonmesh/

@z3dev the default positioning for cubes etc is not centered sadly (thanks to openscad) , but 100% agreed otherwise.
In addition to the measurements (I agree with all of them) we also need (I have pre-existing code too)

align (top , bottom, on  a plane etc)
center (the operation of centering)
distribute

Cool. I’ll open new issues next week for all of the new functionality, one for each. And mark them for V2.
I suggest we just leave center parameters in for now, and write code and examples that use the new functionality. And, if all goes well, the center parameters are removed.
How do you want to proceed?

So you mean adding an issue for 'removal of center' , closing this pr and remove the centers in a later pass ?

Correct.
Let’s remove center in the cleanup pass. Coming soon?

Hmm... here’s one reason for keeping center in the primitives. SVG defines circle / ellipses / etc. with a center.
<rect x="80" y="60" width="250" height="250" rx="20"/>

AUtocad DXF also has the concept of center point for some ‘entities’.
20, 30 DXF: Y and Z values of center point

For svg that is just a position/ translation in the end right ?
Dxf i dunno
The thing is we do not have concepts of centers for 90% of the primitives (because it does not make sense for most of them).
Here it is also only a parameter that is making a shorthand for primitive + translate.
So i think our previous discussion about not keeping them is stil valid.
(unless we get back into geometry vs transforms, haha :-)

agreed. now that all primitives are created 'centered on the axis'. we can translate easily.

@z3dev cool, glad we agree on simplying :D, so what shall we do with this PR ?

Closing this PR based on discussions. I will save the code until V2 is done and dusted.

Sorry. I re-opened this pull request as I think there may be a good reason to include 'center' in primitives.
As we move forward from V2, the vision is moving to an 'expressive' set of primitives, which can have several implementations underneath. However, the expressive primitives need to be 'transformable'. In other words, the basic transforms can apply to the expressive primitives, if the position of the expressive primitive is known.
let geometry = circle({radius: 10, center: [-5, 10]})
geometry = geometry.translate([5, 5]) // translate can be applied to the expressive primitive
geometry = geometry.scale([0.5, 2.0]) // scale can be applied to the expressive primitive!

I just wanted to bring this up for discussion before putting this to sleep.
@kaosat-dev Thoughts?

@kaosat-dev sorry, but I think this should go into V2. From the conceptual standpoint, translation should be separate, but...
From the user point of view, centering a shape is easy, and easy to calculate as well
From the performance point, only one function to create and place the shape
From the stack point, only one data model, not two
I’d really like this to be part of V2.

@z3dev Sorry, I should have answered here as well last time and not just on hangouts.
I have to dissagree on this, sorry: imho it is not just about adding an extra parameter into ever single primitive, this is really the same discussion as with transforms:

you need to store the center (which center btw ? the one for the geometry? the one for the transform, more confusion)
what happens when you apply a transform ? what happens with the center ?
is it local space, model space, world space ?

From the performance point, only one function to create and place the shape

No, under the hood, it is still multiple operations at some point or another :)

From the stack point, only one data model, not two
not sure I understand ?

Closing as this was PR is part of V2.

Closing as this was PR is part of V2.


fix(preset-mini): selector match variantSpaceAndDivide

fix 1246
I'm not sure if the current solution is correct because I don't understand why the ~ selector was used before

fix 1246
I'm not sure if the current solution is correct because I don't understand why the ~ selector was used before

I'm guessing it's because of the order in which the variants are resolved when multiple variants are used. Changes to variant order in previous PRs.
Although there are resolved,  but space is special, it has both rule and variant.
Maybe this isn't a better way to resolved it.

The ~ is so that the selector wont target the first element in the list.

@chris-zhu why was this PR closed?

@chris-zhu why was this PR closed?

This isn't a right way to resolve it. And the PR haven't active for a long time, If you need, i will reopen it.

@chris-zhu why was this PR closed?

This isn't a right way to resolve it. And the PR haven't active for a long time, If you need, i will reopen it.

This PR was made for my ticket #1246. I kept that ticket open by bumping it (to prevent stale bot), because this PR was still open at the time. I'll close my issue now so I don't have to keep bumping it anymore. If anyone needs a proper solution to #1246, feel free to request re-opening of that issue. You can work around this bug using custom rules.


Entries in the journal log.
What did you see instead?
No entries in the log.
$ ls -l /var/log/journal/43bd22923f6945c18f3bbc73498bed92/system.journal
-rw-r-----    1 root     root       8388608 Aug 22 16:08 /var/log/journal/43bd22923f6945c18f3bbc73498bed92/system.journal
$ sudo journalctl -M rkt-43bd2292-3f69-45c1-8f3b-bc73498bed92
Journal file /var/log/journal/43bd22923f6945c18f3bbc73498bed92/system.journal uses an unsupported feature, ignoring file.
-- No entries --

By default stage1 journal creates a xz compressed log. If the host has no or only another compression method enabled we will get no journal entries on the host.
We could:

make an option to disable journal compression in stage1
leave as is and document the fact

Given we chose 2. we can update the documentation for the next release.

/cc @lucab

For reference, a matching host should have:
$ systemctl --version
[...] +XZ -LZ4


mnist model for predicting gray-scale picture problems

cat /etc/mendel_version : 4.0
python: 3.7.3
cv2: 3.2.0
I follow a web page to train an mnist model ：「https://www.tensorflow.org/lite/performance/post_training_integer_quant」
And successfully compiled ：mnist_edgetpu.tflite
After I google a digital picture：'7.jpg'
After that I started to make predictions on the dev board
cmd run:python3 mnist.py
This error appears：AttributeError: 'numpy.ndarray' object has no attribute 'read'
1.txt
mnist.py.txt

Hello, to clarify, I don't think that this is a problem with the dev board at all, but rather how the library is being used. What is the input tensor shape of the model?
The classification engine requires the input shape [1, width, height, channel] I suggest trying the basic engine instead?
Also, it seems that the error you paste here are inconsistent, because you read the image with opencv while this is a numpy error.
Could you paste the full error message?

I use the following code to view tensor shape
import numpy as np
import tensorflow as tf
Load TFLite model and allocate tensors.
interpreter = tf.lite.Interpreter(model_path="/home/must/Download/mnist_tflite_models/mnist_model_quant_io.tflite")
interpreter.allocate_tensors()
Get input and output tensors.
input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()
print(input_details)
[{'name': 'input_2', 'index': 11, 'shape': array([ 1, 28, 28], dtype=int32), 'dtype': <class 'numpy.float32'>, 'quantization': (0.0, 0)}]
After i execute：python3 newmnist.py   Below is the complete error message
Traceback (most recent call last):
File "mnist_download.py", line 16, in
for result in engine.classify_with_image(img, top_k=3):
File "/usr/lib/python3/dist-packages/edgetpu/classification/engine.py", line 86, in classify_with_image
'Invalid input tensor shape! Expected: [1, height, width, channel]')
RuntimeError: Invalid input tensor shape! Expected: [1, height, width, channel]
newmnist.py.txt

Hi as I suggested, our classification engine expects a channel argument to input tensor shape. I suggest you use the tflite API instead of the classifying engine as result won't be correct even if it execute. You can review our doc for the tflite api.
import numpy as np
import sys
from tflite_runtime.interpreter import Interpreter
from tflite_runtime.interpreter import load_delegate

if len(sys.argv) < 2:
    print('Usage:', sys.argv[0], 'model_path')
    exit()

def main():
    """Runs inference with an input tflite model."""
    model_path = str(sys.argv[1])
    delegates = [load_delegate('libedgetpu.so.1.0')]
    interpreter = Interpreter(model_path, experimental_delegates=delegates)

    interpreter.allocate_tensors()
    input_details = interpreter.get_input_details()
    images = np.zeros(input_details[0]['shape'], input_details[0]['dtype'])

    print('starting inference')
    interpreter.set_tensor(input_details[0]['index'], images)
    interpreter.invoke()
    output_details = interpreter.get_output_details()
    outputs = interpreter.get_tensor(output_details[0]['index'])
    print(outputs)
    print('Success.')

if __name__== '__main__':
    main()

Thank you, I can predict successfully.
Does it work on TPU or CPU?
There is another question. Do you not need .txt for prediction?

@LINYOUWEI0804 tflite APIs are going to execute the model on TPU. With .txt do you mean a label file here?

Yes,
Like this example: python3 classify_image.py
--model models / mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite
--labels models / inat_bird_labels.txt
--input images / parrot.jpg
.Txt file is also required for prediction
Manoj<EMAIL_ADDRESS>於 2020年5月12日 週二 下午12:20寫道：

@LINYOUWEI0804 https://github.com/LINYOUWEI0804 tflite APIs are going
to execute the model on TPU. With .txt do you mean a label file here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google-coral/edgetpu/issues/108#issuecomment-627098806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6LHWNYQPKN3UAWTEJI3ATRRDE7PANCNFSM4M35GTWA
.

@LINYOUWEI0804 when you train the model, you should know your label id and class, that info is what you use to create the label file.

I mean do you need the --label parameter？
import numpy as np
import sys
from tflite_runtime.interpreter import Interpreter
from tflite_runtime.interpreter import load_delegate
if len(sys.argv) < 2:
print('Usage:', sys.argv[0], 'model_path')
exit()
def main():
"""Runs inference with an input tflite model."""
model_path = str(sys.argv[1])
delegates = [load_delegate('libedgetpu.so.1.0')]
interpreter = Interpreter(model_path, experimental_delegates=delegates)
interpreter.allocate_tensors()
input_details = interpreter.get_input_details()
images = np.zeros(input_details[0]['shape'], input_details[0]['dtype'])

print('starting inference')
interpreter.set_tensor(input_details[0]['index'], images)
interpreter.invoke()
output_details = interpreter.get_output_details()
outputs = interpreter.get_tensor(output_details[0]['index'])
print(outputs)
print('Success.')

if name== 'main':
main()

You don't need it, it is use for you to map the outputs id to a class. The code I sent you is just a POC to show that there is nothing wrong with the library/model. You can write code to load the model and run inference on it just fine, I've attempted to make use of the outputs since this is not my model, you may understand the outputs better.

Thank you so much


Illegal character in URI using BSP

Mill version: 0.9.7
BSP version: 2.0.0
OS: Windows 10
When syncing with BSP (mill.bsp.BSP/start), the following error occurs on Intellij IDEA:

java.net.URISyntaxException: Illegal character in opaque part at index 2: C:\Users\rapha\AppData\Local\Coursier\cache\v1\https\repo1.maven.org\maven2\com\lihaoyi\mill-scalalib_2.13\0.9.7\mill-scalalib_2.13-0.9.7-sources.jar

Mill returns invalid URI from scalacOptions:

URI must not contain backslashes.
The String before the first : should be the file protocol.

The valid form of the above example should be file:C:/Users/rapha/AppData/Local/Coursier/cache/v1/https/repo1.maven.org/maven2/com/lihaoyi/mill-scalalib_2.13/0.9.7/mill-scalalib_2.13-0.9.7-sources.jar

Same for me:

java.net.URISyntaxException: Illegal character in opaque part at index 2: C:\Users{my user name}\AppData\Local\Coursier\cache\v1\https\repo1.maven.org\maven2\com\lihaoyi\mill-contrib-buildinfo_2.13\0.9.7\mill-contrib-buildinfo_2.13-0.9.7-sources.jar


updating the list of beta features

Changes
This PR includes two changes to the table with beta features:

Alpha Release: an additional column to keep history of when the feature was first available as alpha
TEP-0076: Array results was promoted to beta in PR

https://github.com/tektoncd/pipeline/pull/6103

/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

Thanks @pritidesai LTMG

/lgtm


Remove spurious define of TARGET_IPHONE_SIMULATOR.

This was originally added according to comments for cocos2d compat.  Looking at the sources, this isnt required anymore.  Worse, it interferes with suspend/resume handling on apps where due to a workaround in the TARGET_IPHONE_SIMULATOR codepath, we are trying to resume a track while suspending, resulting in the track not playing on resume.
We no longer pretend to be TARGET_IPHONE, so there is no need to pretend to be the simulator either.

@DHowett-MSFT is added to the review. #Closed

@bbowman is added to the review. #Closed

I can provide the details of the test app offline.

:shipit:

:shipit:

:shipit:

:shipit:


FLUME-3339:Fix the taildir source read the data duplication.

If the file a.log reaches a certain size, it will be renamed a.log.1 and a new file a.log, then a.log and a.log.1 inode are the same, but the file name is different, the default flume taildir source  will think that this is a new file, it will be re-read, that causing data duplication;

close


[FEAT EXTRACTION] Difference between voxel-based and segment-based feature extraction

Hi, I noticed that when executing the feature extractor, there is a parameter "voxelBased" to select whether to use voxel-based or segment-based feature extraction.
Could you please help me clarify the difference between this two methods? I think the segmentation mask is required when executing feature extractor. Thanks!

@LiJiaqi96 it controls how the features are calculated. Segment-based means the feature values are based on the entire segment (aka ROI, Mask, Labelmap, ...), i.e. yielding 1 scalar value per feature and is the most standard application of radiomics feature extraction. The second, voxel-based, extraction calculates a feature value for each voxel in the segment. It does so by computing a kernel region around each voxel and calculates radiomics features based on voxels inside that kernel. The process is repeated for each voxel, i.e. yielding a scalar value for each feature for each voxel  in the segment, better know as parameter maps for each feature. Results in this latter case are also stored as image objects for each feature.
Kernel size, en whether or not to take the intersection between the main segmentation and each kernel when defining the region around a voxel to use for feature calculation are controlled by the voxel-based specific settings.

Can we understand how the voxel-based feature extraction is happening in the background? I want to understand how you use the kernel over the image to get the radiomic features. I have a 2D image and I want to compute a radiomic feature for every pixel and then I want to visualize that radiomic feature. Can it be possible to use the voxel-based method? Please give some feedback on this.
Thank you!
Ankush


ci: switch to clang-12/13

Signed-off-by: Dmitry Rogozhkin<EMAIL_ADDRESS>

Done within #2741.


Mark this package as abandoned

Marking the package as abandoned and setting jimtools/jwt-auth as the replacement

I’d just like to say thank you for everything you’ve done. I do truly appreciate your efforts over the years. ❤


chore: first phase of team-provider refactor

Description of changes

This is the first of many steps in getting rid of team-provider-info.json. In this PR, the file is not removed but most interactions with the file happen through a new EnvironmentParameterManager. There are a few cases where cloudformation metadata needs to be read from environments besides the current environment. In these cases, this information is still read directly from team-provider-info. In a future PR, these interactions will be refactored to use service calls to fetch this metadata.
Still working through some test failures, so not ready to merge yet but I don't expect major changes at this point
Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)
[x] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

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

Codecov Report

Merging #10679 (77ec5d9) into dev (5ea0b9a) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 77ec5d9 differs from pull request most recent head 92cd246. Consider uploading reports for the commit 92cd246 to get more accurate results

@@           Coverage Diff           @@
##              dev   #10679   +/-   ##
=======================================
  Coverage   47.37%   47.37%
=======================================
  Files         669      669
  Lines       33066    33066
  Branches     6673     6673
=======================================
  Hits        15664    15664
  Misses      15724    15724
  Partials     1678     1678

Impacted Files
Coverage Δ

...amplify-cli-core/src/feature-flags/featureFlags.ts
82.40% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Genomes Tab: Sorting by Antimicrobial Resistance column gives an empty table

Dustin, sorting is not working because antimicrobial_resistance is a multi value fields.
We need to either disable sorting or handle it gracefully, instead of table not loading.

Yes, that column just needs to set sortable to false.
https://github.com/PATRIC3/p3_web/blob/master/public/js/p3/widget/GenomeGrid.js#L269-L278 https://github.com/PATRIC3/p3_web/blob/master/public/js/p3/widget/GenomeGrid.js#L269-L278
need to have “sortable: false” added as is done at:
https://github.com/PATRIC3/p3_web/blob/master/public/js/p3/widget/GenomeGrid.js#L88 https://github.com/PATRIC3/p3_web/blob/master/public/js/p3/widget/GenomeGrid.js#L88
If there are other multivalve fields, the same treatment should be done on them.

On Aug 19, 2016, at 12:57 PM, mshukla1<EMAIL_ADDRESS>wrote:
Dustin, sorting is not working because antimicrobial_resistance is a multi value fields.
We need to either disable sorting or handle it gracefully, instead of table not loading.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/861#issuecomment-241073245, or mute the thread https://github.com/notifications/unsubscribe-auth/AALobnLaJ2iID4cmrTY2JxLjg5IxQxiCks5qheBigaJpZM4Jn64V.

Fixed all multivalue fields in the GenomeGrid


jsonlab_selftest.m does not work in R2017a

The script fails on the second example.

Same here, on 2017b (Win 10 Pro).
Undefined function or variable 'st'.
Error in savejson>matlabobject2json (line 448)
txt=struct2json(name,st,level,varargin{:});

@rotton, this issue was previously reported as Issue #39 and #34, and was fixed by this commit: https://github.com/fangq/jsonlab/commit/86ef12a57bbe54a8532c0e0015f52c7850519833
can you download the latest version from github and test it again?


Rethink GCToolKit API flow: GCAnalyzer returns GCAnalysis

For discussion:
Bruno and I had a discussion about this API in general. It was very refreshing to have his un-initiated perspective here. One of the ideas he tossed into the ring was having an 'analyzer' API, to which my response was that is what GCToolkit is.
Later, I got to thinking about it and I see where he's coming from. I think, perhaps, we should rename the GCToolKit class to GCAnalyzer, and have the analyze method return a GCAnalysis which would be what JavaVirtualMachine is now. I think this would make it more clear to the user what to do and what to use.
Thoughts?
Originally posted by @dsgrieve in https://github.com/microsoft/gctoolkit/pull/70#issuecomment-910364145

This is also related to #6.
// @kcpeppe @karianna

I agree that the name of JavaVirtualMachine could be improved to more accurately represent what it is (which is really the GC information for that JVM as pulled out of the log).  I'm not 100% sure it's a full analysis though, more of a model?

@kcpeppe you mentioned the following:

It sounds good but we’re not offering a GC analysis.

And that is correct. Perhaps the method analyze(dataSource) shouldn't be called that. It should be called aggregate then?

completed with refactoring


fix(all): regenerated Error SDK(s)

This pull request was auto-generated
This PR adds the latest generated ERROR SDKs .
Actions:

Review changes
Approve and merge.
Create a release if needed

@dimakis on those we ping UI engineers so they are aware that there are new errors to handle


Document Third Party Dependencies

Need to document the additional dependencies like etcd and influxdb similar to comms. Perhaps document them all together?

The structure has been implemented in #94  but currently says "TBD". Closing this issue since the structure is there - there are plenty of other areas that say "TBD". The structure that's there will serve as a good reminder to get this documentation in place.


Save BOLD derived brain mask in surface space

Use nearest neighbours interpolation and gifti file format. This should be available for the fsnative and fsaverage.

So looked at this again. To be clear, you're suggesting a surface where a vertex is 1 if it's in the bold_space-t1w_brainmask.nii.gz, and 0 if not? I guess these ought to be label.giis?

Yes 2) possibly I'm not sure what is the standard way of doing this in
gifti land.

On Apr 18, 2017 7:18 AM, "Chris Markiewicz"<EMAIL_ADDRESS>wrote:

So looked at this again. To be clear, you're suggesting a surface where a
vertex is 1 if it's in the bold_space-t1w_brainmask.nii.gz, and 0 if not?
I guess these ought to be label.giis?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/issues/395#issuecomment-294859326,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp9JxTfCZR50scrjbatTsHtca043Pks5rxMZGgaJpZM4MhB-h
.

In the lights of the derivatives survey and after more than a year open, is this still something to be implemented?

I'm closing this one for now. @chrisfilo please reopen if this is still needed.

By closing this issue you might give an impression that you do not want such functionality in the package. Why not having such feature requests open, but tagged with low priority or help needed?

Yes, no problem, I was just trying to clean up a bit.


github link for dlib is broken

Getting following error while cloning submodules. Please update this link
Submodule 'dlib/src/main/dlib' (https://github.com/tzutalin/dlib-android) regist                                                                                ered for path 'dlib/src/main/dlib'
Cloning into 'C:/Users/IBM_ADMIN/gaze-detection-android-app/dlib/src/main/dlib'.                                                                                ..
Submodule path 'dlib/src/main/dlib': checked out '32840232a6c37f03f4e31e599d544b                                                                                b114bbe9cd'
Submodule 'jni/dlib' (https://github.com/tzutalin/dlib.git) registered for path                                                                                'dlib/src/main/dlib/jni/dlib'
Cloning into 'C:/Users/IBM_ADMIN/gaze-detection-android-app/dlib/src/main/dlib/j                                                                                ni/dlib'...
remote: Repository not found.
fatal: repository 'https://github.com/tzutalin/dlib.git/' not found
fatal: clone of 'https://github.com/tzutalin/dlib.git' into submodule path 'C:/U                                                                                sers/IBM_ADMIN/gaze-detection-android-app/dlib/src/main/dlib/jni/dlib' failed
Failed to recurse into submodule path 'dlib/src/main/dlib

When I browsed the code from github, I get this link for dlib-android https://github.com/tzutalin/dlib-android/tree/32840232a6c37f03f4e31e599d544bb114bbe9cd. Code structure on this link is seems to be different from here https://github.com/tzutalin/dlib-android/.
I think there are some changes in dlib-android which are not reflected in your repo.
Would you please look into it.

I think this is the culprit. If you open this https://github.com/tzutalin/dlib-android/blob/32840232a6c37f03f4e31e599d544bb114bbe9cd/.gitmodules,  content of this file is
submodule "jni/dlib"]
path = jni/dlib
url = https://github.com/tzutalin/dlib.git

I tried cloning android-dlib separately, directly from tzutalin. But even after that I get this error, dlib/src/main/dlib/jni/dlib/Android.mk not found. I checked at this page https://github.com/tzutalin/dlib-android/tree/master/jni, but it seems there is not any android.mk there as well, hence my copy also does not have it. Looks like tzutalin have changed directory structure or something. I have invested significant amount of time to debug it but without any results. Please look into it whenever you get time.
I think there is multiple

Hi @seemanagar.
I'm having the same problem. Unfortunately there is not much I can do since the problem is coming from an external repository. I'll try to get the sources from my computer when I'm back at the office and publish them in this repo.

Thanks for the confirmation. Please update once resolved.

Hi, sorry to bother you again but wanted to know whether you have resolved the issue ?

I've updated the submodules. Please try again and let me know if it works fine now.

Thanks for the update. I got a copy again from github.
I am getting this issue now:
src/main/clm/jni/FaceAnalyser/../../../../../../dlib/src/main/dlib/jni/dlib/dlib/image_processing/../image_processing/../geometry/../matrix/../memory_manager/../binary_search_tree/binary_search_tree_kernel_2.h:6:48: fatal error: binary_search_tree_kernel_abstract.h: No such file or directory
Would you please look into it.

This is complete log of build process
Information:Gradle tasks [:app:generateDebugSources, :app:generateDebugAndroidTestSources, :clm:generateDebugSources, :clm:generateDebugAndroidTestSources, :dlib:generateDebugSources, :dlib:generateDebugAndroidTestSources, :openCVLibrary310:generateDebugSources, :openCVLibrary310:generateDebugAndroidTestSources]
:app:preBuild UP-TO-DATE
:app:preDebugBuild UP-TO-DATE
:app:checkDebugManifest
:app:preReleaseBuild UP-TO-DATE
:clm:preBuild UP-TO-DATE
:clm:preReleaseBuild UP-TO-DATE
:clm:compileReleaseNdk UP-TO-DATE
:clm:compileLint
:clm:copyReleaseLint UP-TO-DATE
:clm:checkReleaseManifest
:clm:preDebugAndroidTestBuild UP-TO-DATE
:clm:preDebugBuild UP-TO-DATE
:clm:preDebugUnitTestBuild UP-TO-DATE
:clm:preReleaseUnitTestBuild UP-TO-DATE
:openCVLibrary310:preBuild UP-TO-DATE
:openCVLibrary310:preReleaseBuild UP-TO-DATE
:openCVLibrary310:compileReleaseNdk UP-TO-DATE
:openCVLibrary310:compileLint
:openCVLibrary310:copyReleaseLint UP-TO-DATE
:openCVLibrary310:mergeReleaseProguardFiles UP-TO-DATE
:openCVLibrary310:packageReleaseRenderscript UP-TO-DATE
:openCVLibrary310:checkReleaseManifest
:openCVLibrary310:prepareReleaseDependencies
:openCVLibrary310:compileReleaseRenderscript UP-TO-DATE
:openCVLibrary310:generateReleaseResValues UP-TO-DATE
:openCVLibrary310:generateReleaseResources UP-TO-DATE
:openCVLibrary310:packageReleaseResources UP-TO-DATE
:openCVLibrary310:compileReleaseAidl UP-TO-DATE
:openCVLibrary310:generateReleaseBuildConfig UP-TO-DATE
:openCVLibrary310:generateReleaseAssets UP-TO-DATE
:openCVLibrary310:mergeReleaseAssets UP-TO-DATE
:openCVLibrary310:processReleaseManifest UP-TO-DATE
:openCVLibrary310:processReleaseResources UP-TO-DATE
:openCVLibrary310:generateReleaseSources UP-TO-DATE
:openCVLibrary310:compileReleaseJavaWithJavac UP-TO-DATE
:openCVLibrary310:processReleaseJavaRes UP-TO-DATE
:openCVLibrary310:transformResourcesWithMergeJavaResForRelease UP-TO-DATE
:openCVLibrary310:transformClassesAndResourcesWithSyncLibJarsForRelease UP-TO-DATE
:openCVLibrary310:mergeReleaseJniLibFolders UP-TO-DATE
:openCVLibrary310:transformNative_libsWithMergeJniLibsForRelease UP-TO-DATE
:openCVLibrary310:transformNative_libsWithSyncJniLibsForRelease UP-TO-DATE
:openCVLibrary310:bundleRelease UP-TO-DATE
:clm:prepareComAndroidSupportAppcompatV72311Library UP-TO-DATE
:clm:prepareComAndroidSupportSupportV42311Library UP-TO-DATE
:clm:prepareGazeDetectionAndroidAppOpenCVLibrary310UnspecifiedLibrary UP-TO-DATE
:clm:prepareReleaseDependencies
:clm:compileReleaseAidl UP-TO-DATE
:clm:compileReleaseRenderscript UP-TO-DATE
:clm:generateReleaseBuildConfig UP-TO-DATE
:clm:generateReleaseAssets UP-TO-DATE
:clm:mergeReleaseAssets UP-TO-DATE
:clm:generateReleaseResValues UP-TO-DATE
:clm:generateReleaseResources UP-TO-DATE
:clm:mergeReleaseResources UP-TO-DATE
:clm:processReleaseManifest UP-TO-DATE
:clm:processReleaseResources UP-TO-DATE
:clm:generateReleaseSources UP-TO-DATE
:clm:ndkBuild
Android NDK: WARNING:src/main/clm/jni/CLM/../../../../../../dlib/src/main/dlib/jni/dlib/Android.mk:CLM: LOCAL_LDLIBS is always ignored for static libraries
Android NDK: WARNING:src/main/clm/jni/FaceAnalyser/Android.mk:FaceAnalyser: LOCAL_LDLIBS is always ignored for static libraries
Android NDK: WARNING:src/main/clm/jni/Android.mk:FeatureExtraction: LOCAL_LDFLAGS is always ignored for static libraries
Android NDK: WARNING:src/main/clm/jni/Android.mk:FeatureExtraction: LOCAL_LDLIBS is always ignored for static libraries
[armeabi-v7a] Compile++ arm  : FaceAnalyser <= FaceAnalyser.cpp
Information:(Unknown) In file included
src/main/clm/jni/FaceAnalyser/../../../../../../dlib/src/main/dlib/jni/dlib/dlib/image_processing/../image_processing/../geometry/../matrix/../memory_manager/../binary_search_tree/binary_search_tree_kernel_2.h:6:48: fatal error: binary_search_tree_kernel_abstract.h: No such file or directory
#include "binary_search_tree_kernel_abstract.h"
^
compilation terminated.
make.exe: *** [build/intermediates/ndk/obj/local/armeabi-v7a/objs/FaceAnalyser/src/FaceAnalyser.o] Error 1
:clm:ndkBuild FAILED
Error:Execution failed for task ':clm:ndkBuild'.

Process 'command 'C:\work\android-ndk-r10d\ndk-build.cmd'' finished with non-zero exit value 2
Information:BUILD FAILED
Information:Total time: 6.217 secs
Information:1 error
Information:0 warnings
Information:See complete output in console

Have you checked if the file actually exists in that folder? It's in the repo at least https://github.com/wearebase/dlib/blob/e0aa3c1fa02ba918ad24cb5dd0e35d613af5f277/dlib/binary_search_tree/binary_search_tree_kernel_abstract.h

Yes, I checked and the file do exists, but somehow ndk is not finding it. May be ndk, needs to have current directory in the path or something. But I don't know how to do that.

I've updated the version of gradle. Please try to get the sources again and let me know if that helps. I've also have the following in the ~/.bashrc file:
export ANDROID_NDK_HOME="$HOME/apps/android-ndk-r10e"
export PATH="$PATH:$HOME/apps/android-ndk-r10e"

I have fixed the problem by putting the path of the .h files in the android.mk file directly.
Thanks a lot for your help. I am working on windows, so .bashrc would not apply.
Bytheway I get an error in initializing opencv manager with my android 6 device. Earlier with some other application I got the same error, that time I fixed by including opencv in the application directly, not depending on the opencv manager.   For your app, should I do the same ?

What the best way to include opencv in the app only instead of using opencv manager

It should work fine with the version for OpenCV Manager. I don't know exactly what's the issue and what should be done to integrate OpenCV directly inside the app.
I'll close the ticket so I consider that the initial issue has been solved. Please, do not hesitate in share your contribution if you make some progress, since the app is not being developed further from our side.


New checker formatter

Closes https://github.com/Antonboom/testifylint/issues/33

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

160 of 178 (89.89%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 93.791%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/checkers/call_meta.go
7
9
77.78%

internal/checkers/helpers_format.go
9
11
81.82%

internal/checkers/formatter.go
102
116
87.93%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
2024

Relevant Lines:
2158

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

160 of 178 (89.89%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 93.791%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/checkers/call_meta.go
7
9
77.78%

internal/checkers/helpers_format.go
9
11
81.82%

internal/checkers/formatter.go
102
116
87.93%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
2024

Relevant Lines:
2158

💛  - Coveralls


Remove RunHistory from Poll and CheckUnknown

i'll do this once LRU cache is merged

will do this after LRU gets merged. on further inspection, it looks like  (s *Schedule) CheckUnknown() does use its RH though (but no caching). :?

fixed in LRU PR.


Asset management feature

Currently I use Snipe-it for mange my assets, intended as tools, software licenses etc.
This is overly complicated for the average user, as this software is really well designed it would be possible to add just some new features to enable management of assets.
One of the missing features is the ability to assign a stock/asset to a specific person instead of a location, this is useful when you give something to someone and you want to track where your assets are.
Next can be useful also to add reminders for maintenance checks, as well keep track of repair events and costs.

@lettore I do not mean to disregard you but as far as I see it InvenTree is meant for the SME EEs and hobbyists. Tracking stuff down to a specific person does not really make sense for that. Inventory and asset-management are two different kinds of things. The first is for manufacturing the later for logs and life-cycle management for assets.
There is a lot of complex business logic needed for states, maintenance checks, events etc. I know of a software that is well maintained, very agile with its states, checks etc. and meant for exactly what you are describing: snipe-it. I keep several deployments running for different orgs and I honestly do not know how we should make something simpler than that while also enabling all the features you listed.
I think this proposal is a prime example of feature creep but @SchrodingersGat is the maintainer and if he is willing to accept a PR for this stuff why not. It certainly will add a lot of complexity and maintenance-cost for a fringe use-case.

@lettore I do not mean to disregard you but as far as I see it InvenTree is meant for the SME EEs and hobbyists. Tracking stuff down to a specific person does not really make sense for that. Inventory and asset-management are two different kinds of things. The first is for manufacturing the later for logs and life-cycle management for assets.
There is a lot of complex business logic needed for states, maintenance checks, events etc. I know of a software that is well maintained, very agile with its states, checks etc. and meant for exactly what you are describing: snipe-it. I keep several deployments running for different orgs and I honestly do not know how we should make something simpler than that while also enabling all the features you listed.
I think this proposal is a prime example of feature creep but @SchrodingersGat is the maintainer and if he is willing to accept a PR for this stuff why not. It certainly will add a lot of complexity and maintenance-cost for a fringe use-case.

As you said if your target audience is the hobbyist market it's difficult to imagine to manage two different system (Inventree and Snipe-it) just to take track of some assets.
Obviously, this means that the functionality should be reduced to the essential, because if you need more than just basic asset management you should use a dedicated solution.

@lettore would you say this request is a duplicate of https://github.com/inventree/InvenTree/issues/656 ?
There have been a few other requests for similar "lending" features too. I think it would be well received if such functionality could be added in a "simple" manner.
However "simple" integrations are never simple, as @matmair rightfully pointed out above ;)
I think the furthest I'd like to see the scope creep is to allow "lending" to "customers" (but a "customer" could be a client, workmate, parent, whoever)...
This actually is somewhat related to the concept of an "RMA" or "product return" which is a feature I want to add in the future.

@SchrodingersGat should we close this as a duplicate of #656 ? Has a big potential to be a feature creeper.

Duplicate of #656


Declare type for Array.fromAsync

this is already in chrome, but not known to typescript.
it fully replaces streamToPromise.

it seems that the tests you have here aren't really needed, since they're retesting a core API.

it was useful to demonstrate

the actual behavior of that api, since i always get questions every time i do something like this
the behavior of the api matches our use-case, by applying the old streamToPromise tests to the browser api

not making changes


Initial cut of vgo-aware godef

DO NOT MERGE
Pushing up an initial cut to make godef vgo-aware
To try this out locally:
cd $(go list -f "{{.Dir}}" github.com/rogpeppe/godef)
git fetch origin refs/pull/80/head:pr_80
git checkout pr_80
go install

Closing this until there is more movement in https://github.com/golang/go/issues/14120#issuecomment-383994980

@krasi-georgiev there has been movement... I'll investigate over the weekend.

appreciated, thanks!


Issue with CollapsingToolbarLayout, Runtime Exception: cannot inflate CollapsingTollbarLayout

[✓] I have verified there are no duplicate active or recent bugs, questions, or requests
[✓] I have verified that I am using the latest version of Material Dialogs.
[✓] I have given my issue a non-generic title.
[✓] I have read over the documentation (before asking questions on how to do something).

Include the following:

Material Dialogs version: <IP_ADDRESS>
Device OS version: 4.4.2
Device Manufacturer: Huawei
Device Name: Mediapad T1

java.lang.RuntimeException: Unable to start activity ComponentInfo{}: android.view.InflateException: Binary XML file line #21: Error inflating class android.support.design.widget.CollapsingToolbarLayout
                                                                           at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2347)
Caused by: android.view.InflateException: Binary XML file line #21: Error inflating class android.support.design.widget.CollapsingToolbarLayout

Caused by: java.lang.reflect.InvocationTargetException

Caused by: java.lang.NoSuchMethodError: android.support.v4.graphics.drawable.DrawableCompat.setLayoutDirection

None of that is related to code in this library.

it is infact a library issue. Collapsing toolbar works when i remove the library dependency from app.gradle.

Well, it's not; the log does reference anything in my library.
But, I'm guessing it's because you're using a version of AppCompat or the Support Library that doesn't match the version which this library uses.

Ok, i'll check it out. Thank you for the quick response


Cumulative PR - omega branch (wRPC v2, integrations, typescript-v2, resolver v2)

This is a cumulative PR that merges the following PRs:
On kaspanet/rusty-kaspa:

#480 - wRPC v2 (refactored Borsh serialization)
#470 - CLI wallet watch-only account implementation by @1bananagirl
#489 - CLI updates
#486 - Updated typescript bindings (a.k.a. typescript-v2)

On aspectron/rusty-kaspa:

https://github.com/aspectron/rusty-kaspa/pull/73 - Fix Wallet API guards (related to Wallet API producing "insufficient-funds" error in async clients like KNG)
https://github.com/aspectron/rusty-kaspa/pull/69 - Custom input signature script API enhancements by @KaffinPX (extends kaspa-wallet-core/PendingTransaction APIs allowing for non-standard transactions such as KRC-20).
https://github.com/aspectron/rusty-kaspa/pull/68 - Enhancements to WASM API for ScriptBuilder by @surinder83singh and @KaffinPX
https://github.com/aspectron/rusty-kaspa/pull/67 - HexView interface for ScriptBuilder (formatted hex view for scripts)
https://github.com/aspectron/rusty-kaspa/pull/66 - client-side framework improvements to Mass Calculator (Kip9::Beta variant)
https://github.com/aspectron/rusty-kaspa/pull/59 - rebased #470

This branch also implements the following RPC methods or updates (all of them are related to kaspa-resolver refactoring):

get_sync_status() - provides node sync status (can be used for low-bandwidth node sync status polling)
get_connection_count() - provides the total number of TCP sockets in use by the node (sum of gRPC clients, wRPC clients, and connected peers)
get_sysem_info() - provides a unique ID of the system (a.k.a. machine-id in the Linux universe, with fallback on the mac address on other platforms), git commit hash of the rusty-kaspa repository (if available), total system RAM, number of CPU cores, and the internal FD limit of the kaspad process. These metrics are used by kaspa-resolver to profile the node for its socket (RPC client) connection capacity.
get_metrics() RPC call introduces a provisional dictionary-based map for custom metrics.

IMPORTANT: get_server_info() used by the client-side framework is broken in this update due to the change in the data structure (this includes wRPC and gRPC protocols).
wRPC v2 Borsh encoding introduces breaking protocol changes. wRPC resolver has been reconfigured to operate against the new public node infrastructure that uses wRPC v2 protocol. (ecosystem contributors are currently deploying this infrastructure). Rust and WASM SDK users will not experience any issues as when they update to the new version of the Rust or WASM framework based on this branch, they will switch over to already operational resolvers supporting the new protocol.
wRPC v2 now introduces custom serializers (located beside their respective associated primitives) where each struct (as well as structs serialized with serialize!() macro are serialized into a Vec<u8>, which is then serialized into the wrapping structure.  This allows each data structure serialization to become "elastic" and introduce new data into payloads, while older clients will ignore it.

Other updates:

kaspa-resolver crate has been removed from the workspace and relocated to aspectron/kaspa-resolver. This was done as a part of the v2 refactoring that uses an improved methodology for public node tracking.
Implement paginated access to the Transaction store backed by the browser IndexDb WASM interface inside of the kaspa-wallet-core crate (by @surinder83singh).
Fix conditions in kaspa-wallet-core/UtxoProcessor causing balance fluctuations during batch transaction processing (by @surinder83singh).
Introduced kaspa-utils/git now contains Git commit hash (long and short), which is now appended to kaspad version (a similar functionality was previously PRed in - #338, but in this update, @biryukovmaxim and I have created a custom solution that has a better security profile as it does not require any external build.rs dependencies).

@coderofstuff @surinder83singh fixed this yesterday. It's the API guard. An "async gift that keeps giving". Should be back normal now (albeit i haven't yet had a chance to test myself).

I found some inconsistencies in TS types and actual data. Using latest node build and wasm sdk from this branch.
In ITransactionOutpoint field name transactionId, in data transaction_id
In ITransactionOutput field scriptPublicKey has type IScriptPublicKey { version : number; script: HexString; } , in data - hex string

I found some inconsistencies in TS types and actual data. Using latest node build and wasm sdk from this branch.
In ITransactionOutpoint field name transactionId, in data transaction_id
In ITransactionOutput field scriptPublicKey has type IScriptPublicKey { version : number; script: HexString; } , in data - hex string

I unfortunately can not find what you are referring to.  If you can give an actual description of what "data" means, I would appreciate.
The issue that we have is that data comes from different directions - there is data structures that come from consensus made available via RPC, which are serialized using serde (so property names are auto-translated using serde "camelCase" attribute), there is a WASM implementation of all primitives (that mostly present data as getters)..   I've updated ITransactionOutput::scriptPublicKey to declare as IScriptPublicKey | HexString, however I can't see where transaction_id could be coming from.
If you could give an example in a form of "I am calling this and in response I get this data" would be helpful.

I unfortunately can not find what you are referring to. If you can give an actual description of what "data" means, I would appreciate.

Data is what actual data I get from RPC call using wRPC borsh

If you could give an example in a form of "I am calling this and in response I get this data" would be helpful.

I get blocks like this
let blocks = await rpc.getBlocks({
  lowHash,
  includeBlocks: true,
  includeTransactions: true,
});

and for example
blocks[0].transactions[0].inputs[1].previousOutpoint.transaction_id // tx id is here
blocks[0].transactions[0].inputs[1].previousOutpoint.transactionId // no such field

but there is a type error "Property 'transaction_id' does not exist on type 'ITransactionOutpoint'. Did you mean 'transactionId'?"
Same with blocks[0].transactions[0].outputs[1].scriptPublicKey
Probably it's already fixed somewhere, I will wait for the release and check it again.

@ZenVoich my bad.  Thanks for pointing this out.  One of the type aliases was converted to an actual struct, but I didn't realize that and I kept looking in the wrong place.  Fixed in https://github.com/aspectron/rusty-kaspa/commit/9cec07787edf473a68f5a300842fab81747ba67e


fix(bchjs): Removed BITBOX stub for native bchjs

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Andrew's test branch

to update what I have on my computer so I don't screw up the .exe file

I didn't know what I was doing


Cleanup CNI documentation ("how-to-implement-the-kubernetes-networking-model")

This is a Feature Request

discussed at the 8/4/2020 SIG Docs meeting

The page how-to-implement-the-kubernetes-networking-model is an eyesore and not very helpful to a Kubernetes newcomer.
In addition, some of the listed networking plugins blur the line of acceptable 3rd party content. Some, but not all, networking providers fall under the criteria of "third-party OSS content [...] that [is required] to function."
The resolution consists of three parts:

Modify the page to be more accessible for folks new to Kubernetes with clear "next step" guidance.

Outline the specific criteria of what 3rd party content is accepted, and why, for specific page(s) in the content-guide. Mainly the criteria for accepting or rejecting content.

Remove any providers that do not meet specific outlined criteria

Why is this needed
A cluster provisioning tool kubeadm has stopped providing examples of CNI providers (dual sourced content) due to the fact that there has been an increase in "tag along" PRs. kubeadm does not want to be in the business of choosing "winners" for CNI's and directs users to how-to-implement-the-kubernetes-networking-model as a result. (reference PR: https://github.com/kubernetes/website/pull/22600)
how-to-implement-the-kubernetes-networking-model needs to be cleaned up if we are sending more traffic there.
The content-guide update is needed for reference if SIG Docs closes a PR that does not meet the "third-party OSS content [...] that [is required] to function" criteria. In addition it could assist with decision making. (reference PR: https://github.com/kubernetes/website/pull/22799)
/cc @sftim @kbarnard10 @onlydole @kbhawkey @zacharysarah @celestehorgan

At least some details about cluster networking will find a better home inside https://kubernetes.io/docs/concepts/services-networking/

/sig network
/kind cleanup
/priority important-longterm

This is also more distantly relevant to the other SIG Network.

Agreed on the issue and challenges here. I made my first PR (22973) to try address the issue of the simple install instructions being removed from the kubeadm page. In doing so I reviewed all the projects to try see if they had simple install instructions so I could add them, but ended up very confused myself! Additionally, a few of the projects are only relevant if you're using the vendors specific cloud / infrastructure / platform, but that's not always clear from the description.

/triage accepted

/remove-lifecycle stale
Happy New Year, everyone!

/remove-lifecycle stale
Happy New Year, everyone!

/remove-lifecycle rotten

I'd be happy to move the list of network plugins into the reference section.

/lifecycle frozen

/close
I believe the section mentioned in this issue report has been revised long ago.
No 3rd party contents is still there.


Update to tailwind-merge v2

Fixes #139

@jrgarciadev This is a breaking change and will need to be marked as such in release notes.

Oh sweet, looking forward to get this PR merged. Bundle size will thank me for not including 2 versions of tailwind-merge.


Opencast endpoints to change event/series metadata/ACL fast

Basically: When changing metadata/ACLs in Opencast we don't want to wait for a workflow to finish for the changes to end up in Tobira.

Is this about making this faster or making it possible?

Both, kind of. In theory this is just about making it faster, yes. But the status quo of it being very slow would have multiple unfortunate effects, all of which would make the implementation in Tobira a lot harder. Since we can expect at least like 20seconds or so (often longer) to pass for a metadata update, we can't just show a spinner next to the "save" button in Tobira. Instead we have to show a message explaining to the user that this might take a while. During that time we have to "lock" that event or series so that no user can edit its metadata again. We also then need to adjust the data synchronization between Opencast and Tobira such that Tobira knows when a metadata update has successfully propagated. For all of that the database in Tobira needs to be adjusted quite a bit, adding complexity at multiple places. And I'm probably still missing spots that this would make more complicated.
In the end, I'm very convinced that "properly fixing it", i.e. solving this issue as described, actually saves us time compared to not touching Opencast and just trying to deal with it on the Tobira side.
I hope that answered your question?

Looking at your explanation, I would say it is about "making it possible". No one wants to wait 20sec and longer for metadata to be changed...


ChartJS Version 3: This NPM Package Appears to Work

I, like many others here, ran into an issue using ChartJS version 3 and @nagix's colorschemes plugin.  I use NPM + Webpack to manage my Javascript dependencies, so if you're looking for a working NPM package, I suggest checking out:
https://www.npmjs.com/package/chartjs-plugin-colorschemes-v3
The package is based on @m08pvv's fork of @zsu's fork of this repo.  Kudos to both of them!
What about ChartJS Version 4?
I tested chartjs-plugin-colorschemes-v3, and unfortunately, it does not work with version 4.  I spent a few hours trying to figure out the problem, but I hit a brick wall.  Perhaps someone smarter than me can figure out why colorschemes won't register itself as a global plugin in version 4.
I hope that @zhu and @m08pvv update their repos to enable Issues so we can report bugs and such.

I, like many others here, ran into an issue using ChartJS version 3 and @nagix's colorschemes plugin. I use NPM + Webpack to manage my Javascript dependencies, so if you're looking for a working NPM package, I suggest checking out: https://www.npmjs.com/package/chartjs-plugin-colorschemes-v3
The package is based on @m08pvv's fork of @zsu's fork of this repo. Kudos to both of them!
What about ChartJS Version 4?
I tested chartjs-plugin-colorschemes-v3, and unfortunately, it does not work with version 4. I spent a few hours trying to figure out the problem, but I hit a brick wall. Perhaps someone smarter than me can figure out why colorschemes won't register itself as a global plugin in version 4.
I hope that @zhu and @m08pvv update their repos to enable Issues so we can report bugs and such.

I think it should be @zsu in the last line.

Oops! You are correct.  Sorry about that. I have updated the first post.

I, like many others here, ran into an issue using ChartJS version 3 and @nagix's colorschemes plugin. I use NPM + Webpack to manage my Javascript dependencies, so if you're looking for a working NPM package, I suggest checking out: https://www.npmjs.com/package/chartjs-plugin-colorschemes-v3
The package is based on @m08pvv's fork of @zsu's fork of this repo. Kudos to both of them!
What about ChartJS Version 4?
I tested chartjs-plugin-colorschemes-v3, and unfortunately, it does not work with version 4. I spent a few hours trying to figure out the problem, but I hit a brick wall. Perhaps someone smarter than me can figure out why colorschemes won't register itself as a global plugin in version 4.
I hope that @zsu and @m08pvv update their repos to enable Issues so we can report bugs and such.

I believe I've figured out how to make this plugin (the v3 version) work in v4 (at least, it seems to work in my brief test of it).
Summary of the changes:
On line 10, removed ", global.helpers"
On line 11, remove ", helpers"
On lines 1124, 1132 and 1155, replace "helpers.color" with "Chart.helpers.color"
That was all I needed to do.

https://github.com/nagix/chartjs-plugin-colorschemes/issues/43