Fix android crash on app launch after app upgrades

Describe the bug
In most cases the app works as normal, but after a series of use-and-appUpgrades, the app fails to start and always crashes as soon as the app is launched by tapping on it's icon, no splash screen is visible, just
To Reproduce

Install an older version of Inji app
Use the app to transfer VCs
Upgrade the app
Repeat steps 1-3 until the app crashes on launch

I installed release apps from these release dates to reproduce this bug; 20th Feb --> 28th Feb --> 20th Feb --> 28th Feb --> 6th March.
Expected behavior
On app upgrades the app should not crash.
Actual behavior
After usage & upgrades of the app, the app fails to start and this is a known issue for a certain range of React native versions as documented here.
Add the screenshot of the profile page with commit id
The bug applies to all versions of Android app on Android 9, 10, 11 as documented by the issue on SoLoader repository
Smartphone:
This has been successfully reproduced on Nokia 6.1 Plus(Android 10, BLE 5.0). Since this bug is a react native issue, wallet & verifier details are not relevant here as the app fails to start. This is a React Native issue, so it applies to more devices.
Further relevant details:
React Native version: 0.64.4
Original issue: https://github.com/facebook/SoLoader/issues/77
Logs:
android-onlaunch-crash.txt

@ravikp please attach relevant labels

verified this issue on the new build, I tried to install 3 different old builds back again and again, just like @vharsh mentioned in above, the issue is not observed anymore, the task is able to close
Tested on :
device : pixel 6
ble : 5.2
android : 13
device : redmi k20 pro
ble : 5.0
android : 11
device : vivo y73
ble : 5.0
android : 12
MOSIP Version : <IP_ADDRESS>
MOSIP server : qa-1201-b2.net
APP link : https://github.com/mosip/inji/actions/runs/4447486079

@Anushree09-N @santhoshsunder Can this issue be closed now?

@vharsh the issue can be closed, issue is not replicating, hence closing this issue


Requirements.txt

I think that it's pretty obvious.

The pull request #20 has been opened so closing this!


Can you add number of code lines that metagpt can maintain now , to readme

Could you add the medium lines that new version's MetaGPT can maintained ?
its better to show a change figure in readme.md
from 2023.07 to now , the number of code lines that metagpt can maintain from 2000 to ....(maybe 10k? 20k?)

Our metrics are not yet available to the public, and some metrics may be updated in version 1.0.


support different dns providers

A shower thought:
What if you could simply set a variable like dns_provider = "cloudflare" and this module "just worked"? If a new variable, dns_provider continued to default to "aws" then this change could even be backwards compatible perhaps.
That would be something. I'm going to give it a shot.

I tried https://github.com/samstav/terraform-mailgun-aws/pull/12
and things got weird. Going to shelf this until a proper 0.12 upgrade is complete


[ChatGPT] An error occurred after using the API Endpoint.

Extension
https://www.raycast.com/abielzulio/chatgpt
Description
Error:
Error: Request failed with status code 401

gt.exports:createError.js:16:14

---
13:  * @returns {Error} The created error.
14:  */
15: module.exports = function createError(message, config, code, request, response) {
16:   var error = new Error(message)

### Steps To Reproduce

1. Set API KEY
2. Enabled API Endpoint and filled in
3. Enter "ask" and press Enter.
4. An error message appears: "Something went wrong You can try reloading, or report the error to the extension developer, by copying the log from the Action Panel #f K”

I suspect that the API Endpoint supplier is unable to provide the models list API, or other interfaces are not supported.

### Current Behaviour

_No response_

### Expected Behaviour

_No response_

Can anyone tell me how to stop being pinged on these issues?

Can anyone tell me how to stop being pinged on these issues?

Maybe you need to open a new issue to ask the official maintainers😂
btw I have the same question😅

It has been solved, because after adding API Endpoint, some API Endpoint addresses need to add v1 after the link, e.g. https://xxx.xxx.com/api replaced by https://xxx.xxx.com/api/v1

It has been solved, because after adding API Endpoint, some API Endpoint addresses need to add v1 after the link, e.g. https://xxx.xxx.com/api replaced by https://xxx.xxx.com/api/v1.
这个问题已经解决了，因为在添加 API 端点后，一些 API 端点地址需要在链接后添加 v1，例如 https://xxx.xxx.com/api 替换为 https://xxx.xxx.com/api/v1。

It has been solved, because after adding API Endpoint, some API Endpoint addresses need to add v1 after the link, e.g. https://xxx.xxx.com/api replaced by https://xxx.xxx.com/api/v1.


python3Packages.aws-sam-translator: enable tests

Motivation for this change
Enable tests
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 135399 run on x86_64-linux 1

8 packages marked as broken and skipped:

python38Packages.pathy
python38Packages.spacy
python38Packages.spacy-transformers
python38Packages.textacy
python39Packages.pathy
python39Packages.spacy
python39Packages.spacy-transformers
python39Packages.textacy

42 packages built:

aws-sam-cli
buildbot (python39Packages.buildbot)
buildbot-full (python39Packages.buildbot-full)
buildbot-ui (python39Packages.buildbot-ui)
python38Packages.aws-sam-translator
python38Packages.buildbot
python38Packages.buildbot-full
python38Packages.buildbot-ui
python38Packages.celery
python38Packages.cfn-lint
python38Packages.django-raster
python38Packages.djmail
python38Packages.flower
python38Packages.gensim
python38Packages.moto
python38Packages.slack-sdk
python38Packages.smart-open
python38Packages.tensorboardx
python39Packages.aws-sam-translator
python39Packages.celery
python39Packages.cfn-lint
python39Packages.django-raster
python39Packages.djmail
python39Packages.flower
python39Packages.gensim
python39Packages.moto
python39Packages.slack-sdk
python39Packages.smart-open
python39Packages.tensorboardx
snakemake
sourcehut.buildsrht
sourcehut.coresrht
sourcehut.dispatchsrht
sourcehut.gitsrht
sourcehut.hgsrht
sourcehut.hubsrht
sourcehut.listssrht
sourcehut.mansrht
sourcehut.metasrht
sourcehut.pastesrht
sourcehut.todosrht
tts

Result of nixpkgs-review pr 135399 run on x86_64-linux 1

8 packages marked as broken and skipped:

python38Packages.pathy
python38Packages.spacy
python38Packages.spacy-transformers
python38Packages.textacy
python39Packages.pathy
python39Packages.spacy
python39Packages.spacy-transformers
python39Packages.textacy

42 packages built:

aws-sam-cli
buildbot (python39Packages.buildbot)
buildbot-full (python39Packages.buildbot-full)
buildbot-ui (python39Packages.buildbot-ui)
python38Packages.aws-sam-translator
python38Packages.buildbot
python38Packages.buildbot-full
python38Packages.buildbot-ui
python38Packages.celery
python38Packages.cfn-lint
python38Packages.django-raster
python38Packages.djmail
python38Packages.flower
python38Packages.gensim
python38Packages.moto
python38Packages.slack-sdk
python38Packages.smart-open
python38Packages.tensorboardx
python39Packages.aws-sam-translator
python39Packages.celery
python39Packages.cfn-lint
python39Packages.django-raster
python39Packages.djmail
python39Packages.flower
python39Packages.gensim
python39Packages.moto
python39Packages.slack-sdk
python39Packages.smart-open
python39Packages.tensorboardx
snakemake
sourcehut.buildsrht
sourcehut.coresrht
sourcehut.dispatchsrht
sourcehut.gitsrht
sourcehut.hgsrht
sourcehut.hubsrht
sourcehut.listssrht
sourcehut.mansrht
sourcehut.metasrht
sourcehut.pastesrht
sourcehut.todosrht
tts


Autoscroll

How about autoscroll? Standart autoscroll is awful inhibits.

And any scroll is awful inhibits on material form


Turn Spectro into a library

At the moment Spectro just builds to a demo. It should be turned into a library, with the existing demo being moved into a demo or examples folder.
I imagine the API looking something like this:
// Creates a spectrogram attached to the given canvas, provides automatic handling
// of when the canvas is resized
const canvas = document.getElementById('spectrogram-canvas');
const spectrogram = new Spectrogram(canvas);

// Change display parameters of the spectrogram
spectrogram.updateRenderSettings({
    zoom: 1.5,
    minFrequencyHz: 0,
    maxFrequencyHz: 12000,
    scale: 'mel',
    color: /* object representing a color gradient */
});

// Start recording from mic, will prompt for permissions and return a promise once
// recording has started
await spectrogram.recordFromMic();

// Play audio file, can take URL or File object with option to mute output
await spectrogram.playAudioFile(file, mute = true);

// Manual API where audio data is streamed to the spectrogram
const renderAudioBuffer = await spectrogram.streamAudio();
// Call renderAudioBuffer() each time there is an audio buffer to render, returns a
// promise
// Calling renderAudioBuffer() after a call to stop() is an error

// Stop recording from mic or playing audio file
await spectrogram.stop();

// Add event listener to handle state changes
spectrogram.addStateChangeEventListener((state) => {
    // state can be one of 'stopped', 'playing-mic', 'playing-file', or
    // 'playing-custom'
});

Really looking forward to this !

Hi @calebj0seph
I was looking at the possible libraries around, and I saw your demo, it looks quite good.
I have two questions about making the library issue?

do you have a plan to release on npm?
do you plan to maintain the repository and accept pull requests?

@calebj0seph
Please have a look at these websites:
https://stackoverflow.com/questions/6663222/doing-fft-in-realtime
https://www.dsprelated.com/showarticle/776.php
They refer to what is called Sliding Window Fourier Transform which is a much faster way to compute the spectrogram.  The FFT cost is nLogn where as the SWFT is n.
Using the SWFT should reduce the lag between the sound and the display of the spectrogram.
I'm looking for a Javascript version I had and will send it up if I can find it.
RONC

@rechmbrs Thanks for the links!
From what I understand a sliding DFT is used when you want to compute the DFT for windows which are only 1 sample apart. In this case, yes it makes much more sense to use an O(N) algorithm each window rather than an O(N*log(N)) algorithm since you're computing it every sample.
For a spectrogram though, we don't need to compute the DFT every sample. In Spectro, we only do this every 1024 samples with a window size of 4096 samples. If we do some quick calculations, we end up with (samples / 1024) * (4096 * log(4096)) operations. If we were to replace this with a sliding DFT, instead calculating the DFT for a window every sample, we end up with samples * 4096 operations. This might seem simpler at first, but if you do the math, the sliding DFT ends up being orders of magnitude slower since we have to compute it every sample instead of every 1024 samples.
Unless I'm misunderstanding something, it doesn't look like this would help performance at all.

@gokgozf Sorry for the late reply, haven't really touched any of my Github projects since COVID all started. Now that I'm getting back into things, I'm hoping to make Spectro into a reusable library with an npm package while also keeping this repository maintained. I've started by upgrading all of the dependencies, next I'm going to be refactoring the codebase into a library while putting the current demo in a separate directory that uses the new library. No ETA yet as I haven't worked with the code in a year, but hopefully not too long!

@calebj0seph
the sliding DFT can be used to drop m samples and pickup p samples.  Most often m = p = 1.
Please contact me at rechmbrs [at] gmail [dot] com  to follow on with this.  I have a number of other parts to discuss.  We can then bring results back here for others.
RONC


a better tuple_cat implementation

tuple_cat can be made far more efficient by decreasing the function instantiations and parameters passed. This has a significant effect for both compile times and for the resultant assembly generated.
A solution I came up with for my own tuplet-like library was to create a tuple of tuples, to build an index LUT for that 'big' tuple, and then to do the entire reconstruction process in a single pass.
A working version of my code is available here: https://godbolt.org/z/5WoWx8Wv6
adapted from my earlier works,
here: https://gist.github.com/mechacrash/b0b941b1bff29beb76811b69186284e6
and here: https://gist.github.com/mechacrash/39c8af5187e5f54ea6747416c1fa95c4
I reimplemented a version that works with tuplet, and supports the earliest versions of the clang, GCC and MSVC compilers that tuplet currently supports (11.X, 10.X, and 19.23 respectfully - this is limited by support for <compare>) and threw together a quick demonstration of the issue,
See a comparison here:

before, https://godbolt.org/z/dxsjeKxsh
after, https://godbolt.org/z/5n1MWeo4c

unoptimised
gcc : 123'700 lines of asm -> 7'860 lines of asm
clang : 116'903 lines of asm -> 8'501 lines of asm
msvc : 121'603 lines of asm -> 12'988 lines of asm
optimised (-O3, /O2)
gcc : fails to build (compilation time exceeded) -> 3 lines, fully optimised out
clang : 31'056 lines of asm -> 3 lines, fully optimised out
msvc : 63'944 lines of asm -> 4'158 lines of asm
Apologies for not supplying this as a pull request, just wanted to quickly throw this across and found this was the easiest path. You're more than welcome to use any part of this code however you like though - adapt or discard as you wish!

thanks for the information, I've updated the implementation to use the base_list instead of get: https://godbolt.org/z/venzKnKdG
There's some further improvement to binary size too!
unoptimised
gcc : 123'700 lines -> 7'860 lines -> 4'433 lines
clang : 116'903 lines -> 8'501 lines -> 3'339 lines
msvc : 121'603 lines -> 12'988 lines -> 8'307 lines
optimised (-O3, /O2)
gcc : fails to build (compilation time exceeded) -> fully optimised out -> EQ
clang : 31'056 lines -> fully optimised out -> EQ
msvc : 63'944 lines -> 4'158 lines -> 2'680 lines
I can't test compilation speeds right now (very limited on time and don't have a proper working environment available to me right now), but anecdotal evidence suggest there's no significant change.
The original limitation applies here too (variadics of variadics get messy), so I still think there's the potential to optimise away the need for the type_list operator+ instantiations, but I still don't have a good approach for it 😞


Skip row with certain value in field

Is it somehow possible to skip a row if it contains a certain value in a specific field? I have a huge CSV and it contains a field that is either 0 or 1. Most of the rows have a 0 and i dont need them. Looking to make a performance improvement here as I currently have to go over each row with a foreach and that takes a lot of time to check that value.

@tebeso thanks for using the package. Did you read the documentation ? the part about Statement it should resolve your issue.


1.13: GEODE-9486: Fix validate-serializable-objects (#6823)

Back port the fixes for GEODE-9486 to support/1.13:

GEODE-9486: Fix validate-serializable-objects (#6823)
Rename DistributedSystemService to SanctionedSerializablesService and
remove unused init(DistributedSystem).
Move SanctionedSerializablesService to geode-serialization.
Implement SanctionedSerializablesService in both geode-core and
geode-management to remove special case for each in
InternalDataSerializer.
Fix sanctioned serializables support in geode-management and
geode-apis-compatible-with-redis.
Add sanctioned serializables support to geode-serialization and
geode-pulse.
Add sanctioned serializables support to geode-junit and geode-dunit
to simplify writing tests for validate-serializable-objects and prevent
having to list out DUnit Rules and other test framework classes when
validate-serializable-objects is enabled.
Use ExecutorServiceRule and reformat json strings in
RestRegionAPIIntegrationTest.
Reorganize QueryCommandDUnitTestBase.
Use InvalidClassException instead of Exception in
ObjectInputStreamFilterWrapper fatal log message.
Improve assertion messages in ResourceUtils.
Move loadSanctionedSerializablesServices and loadClassNames to
new SanctionedSerializables in geode-serialization.
Exclude Pulse GemFireAuthentication from sanctioned serializables.
Add SerializationTest Category to all AnalyzeSerializables integration
tests.
Tidy up SANCTIONED_SERIALIZABLES_DEPENDENCIES_PATTERN.
Convert to AssertJ and use BeforeClass in
InternalDataSerializerSerializationAcceptlistTest.
Note: If Git or GitHub is showing invalid file renames in the diffs, you
may need to pull the branch locally and configure diff.renameLimit to
something lower than the default value of 50.
(cherry picked from commit acbd0ff3c37a5e1fe3018d3f7288df385159ac4c)
(cherry picked from commit 8e569666b52d0179530a5c43e94c90e8c45de68f)

GEODE-9486: Improve AnalyzeSerializables integration tests (#6878)
Allow geode modules to have an AnalyzeSerializables integration
test without implementing SanctionedSerializablesService.
Improve debugging information for AnalyzeSerializables integration
tests:

Provide new failure message when no SanctionedSerializablesService
exists.
Create ANALYZE_SERIALIZABLES.md to provide detailed instructions for
failures involving AnalyzeSerializables integration tests.
Reference ANALYZE_SERIALIZABLES.md in failure messages.

Remove geode-serialization and geode-common jars from geode-pulse
.war file:

Change getModuleClass() to return Optional.
Delete PulseSanctionedSerializablesService and its resources.
Change geode-pulse dependency on geode-serialization to be for
integrationTest only.

(cherry picked from commit 7bd1d73dcd02712a10e5c3f2ac5ac0522923bf9e)
(cherry picked from commit d13d6d8f51377290bc2cfc8ddf787b770e9fd4a9)

Flaky test LocatorUDPSecurityDUnitTest had an unrelated failure.


Tiles need RSI texture support.

At least loading of single-direction textures. Animated textures would be a massive pain to implement so I'm fine just using Frame0

I demanded animated honking clown tiles.


Aerial View of Bing Maps - Map Style

Any chance this could be added to Flowmap please?

I see, this should be easy, just never occurred to me before :)

hi, it is in the v1.2 now


Global Password Reset Emails

When a user forgets her/his password, s/he should receive a password reset email in the same language as the version of the site s/he is viewing. i.e. if I am located in Brazil, I should receive the Brazilian Portuguese reset password email.

[ ] Brazilian Portuguese password reset email
[ ] Mexican Spanish password reset email

Note: I don't think we have to add anything for the "Global English" users, because our default US English transactional email should trigger and should be good to use. Only potential pitfall is if any funny business needs to happen to the link we provide in the email.
@sergii-tkachenko @sheydari-ds if you have any thoughts please include here

[ ] Update dosomething_user_user_pass_submit() to include user language
[ ] Update dosomething_user_user_pass_submit() to include password reset template name
https://github.com/DoSomething/phoenix/blob/dev/lib/modules/dosomething/dosomething_user/dosomething_user.module#L415-L437

      $params = array(
        'email' => $account->mail,
        'uid' => $account->uid,
        'user_language' => dosomething_user_get_field('field_language', $account),
        'email_template' => $template,
        'first_name' => dosomething_user_get_field('field_first_name', $account),
        'reset_link' => user_pass_reset_url($account),
      );

[ ] Update dosomething_mbp_get_template_name() to define the template name based on the user language rather than the current dosomething_settings_get_affiliate_country_code(). Use the user_language value in the $param value passed in the dosomething_mbp_request() call.

Note: Country code is currently return with returned rather than language of user / campaign.

Related:

Trello card for Message Broker:

https://github.com/DoSomething/phoenix/issues/4955
https://trello.com/c/HT3AuNGA
https://trello.com/c/n9DN4bKF

BLOCKED by:

https://github.com/DoSomething/phoenix/issues/5168 There a need for the user language setting to define the transactional template name.

Related Trello card on Message Broker board: https://trello.com/c/HT3AuNGA

dosomething_user_user_pass_submit() $params:
Array
(
    [email] =><EMAIL_ADDRESS>    [uid] => 1705348
    [first_name] => Test Dee
    [reset_link] => http://dev.dosomething.org:8888/user/reset/1705348/1443018806/bnCyX8yrx66VCC7PCvE4g8QUvsL8gj-78HuCuHfSD4Q
    [user_language] => en-global
    [user_country] => US
)


Common platform used by many municipalities

While looking for gazettes for Paranavaí the official site redirects to a portal that seems to be used by lots of smaller municipalities.
http://www.diariomunicipal.com.br
This portal even allow to search for terms like "Dispensa"
http://www.diariomunicipal.com.br/amp/pesquisar?entidadeUsuaria=&titulo=Dispensa&nome_orgao=&dataInicio=01%2F05%2F2018&dataFim=31%2F05%2F2018&Enviar=&_token=aAfXBvwcSMnp4lRq1qoriAGxW9rav7iawfmpUeHGLMk
Only problem is that the result is not a single pdf, but a page for each result. I believe that another pipeline would be needed for scraping this website.
Although i haven't found any city on the top 100 using this portal, maybe it would be worth scraping it given the amount of cities that uses it.
What do you think?

Also it seems they have a rest API
http://www.diariomunicipal.com.br/amp/rest/materia/search-materias?entidadeUsuaria=&titulo=&nome_orgao=&dataInicio=2018-05-01&dataFim=2018-05-31&sEcho=1&iColumns=4&sColumns=entidade%2Ctitulo%2Corgao%2CdataCirculacao&iDisplayStart=0&iDisplayLength=10&mDataProp_0=0&sSearch_0=&bRegex_0=false&bSearchable_0=false&bSortable_0=false&mDataProp_1=1&sSearch_1=&bRegex_1=false&bSearchable_1=false&bSortable_1=false&mDataProp_2=2&sSearch_2=&bRegex_2=false&bSearchable_2=false&bSortable_2=false&mDataProp_3=3&sSearch_3=&bRegex_3=false&bSearchable_3=false&bSortable_3=false&sSearch=&bRegex=false&iSortingCols=0&_=1527800170218

Hey there, @beothorn
As stated in the contribution guidelines, we intend to collect from official sources.
For instance, smaller municipalities from the state of Espírito Santo decided to delegate the task of publishing their gazettes to Associação dos Municípios do Estado do Espírito Santo. When such thing happens, we look for two requirements. Let's take Laranja da Terra (ES) as an example:

[x] The official website links to the aggregator when mentioning its gazette.

[x] A municipal law staging the aggregator as an authorized publisher. As http://www3.laranjadaterra.es.gov.br/norma.aspx?id=817.

When we can find examples of both, we're more than happy to integrate with the aggregator, as we did with the previous website.

Hi,
in the case of Paranavaí, we found the law stating the aggregator as an authorized publisher (http://www.controlemunicipal.com.br/inga/sistema/arquivos/12101/040817154719_lei_municipal_46042017__institui_o_diario_oficial_dos_municipios_do_parana_como_veiculo_oficial_pdf.pdf).
Now the problem is with the format, that is a single HTML page and not a PDF.
Do you have some suggestion how we could work on this? We believe that the pipeline must be changed, but we are not sure how.

Just to add, the diário oficial from Rio de Janeiro has a similar "problem". Although the spider we created extracts the PDFs, it does have a version in HTML, which would be much easier to parse than OCRing a PDF file.

Even if the HTML content is available (and easier to parse its content). I believe it is worthy to also download the PDF, so it can be made available in this format as well (if the effort isn't too large).

@rennerocha, do you suggest download the HTML content and convert to PDF? Because there is not available a PDF version.

@antoniovendramin I understood that the PDF is also available for download! In this case, I don't see any gain generating a PDF for the content.


net/http/httputil: proxied streaming responses appear to lose some data on the way to the client

What version of Go are you using (go version)?

$ go version
go version go1.16.4 darwin/arm64

Does this issue reproduce with the latest release?
I'm about to test that today.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/rolandgroza/Library/Caches/go-build"
GOENV="/Users/rolandgroza/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/rolandgroza/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/rolandgroza/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/homebrew/Cellar/go/1.16.4/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/homebrew/Cellar/go/1.16.4/libexec/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.16.4"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/rolandgroza/Work/echo-streaming-bug/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/_9/htkb4h950nv8qctq6pfxc6mr0000gn/T/go-build365273782=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

I've setup a simple echo server and I'm using streaming responses to stream data to clients through another proxy server.
Everything seemed to work fine, but I later noticed that after some time (25 - 30 mins), browser clients would get a:
SyntaxError: Unexpected token { in JSON at position 119
    at streamData (<anonymous>:20:40)

I initially thought the issue was in https://github.com/labstack/echo/issues/2125. But then I also tried to use the net/http/httputil reverse proxy directly and I could still reproduce the error. So I think the issue could either be in the reverse proxy, the net/http server or could be in the browser.
I've tried my best to describe all the steps to reproduce the error at transcelestial/echo-streaming-bug.
I'll try to remove echo and use a simple server to check if the issue still persists.
What did you expect to see?
I don't expect to see any errors or data loss on the client end.
What did you see instead?
I get
SyntaxError: Unexpected token { in JSON at position 119
    at streamData (<anonymous>:20:40)

after ~ 25 mins.

I don't see this is a Go project's issue.
Could you provide a more concise example in https://go.dev/play?

I don't see this is a Go project's issue. Could you provide a more concise example in https://go.dev/play?

Hi. Well, the issue only happens in browser clients. So data sent by the Go server is somehow corrupted along the way. I wouldn't be able to reproduce in a play env.
I'm not certain if this is a browser bug or something in the reverse proxy or net lib. But I can reproduce in different browsers, which led me to believe that the issue must be on the server side.
Is there something wrong with the details I've shared in that repo? Are you not able to reproduce?

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions

https://github.com/transcelestial/echo-streaming-bug/blob/74b734755472389d5dac7d00aab1cc4da75f3c9f/cmd/client/main.go#L57-L61
	for {
		line, err := reader.ReadBytes('\n')
		if err != nil {
			log.Fatal(err)
		}

It appears that your client can't split messages correctly when your data/content contains \n.

@mengzhuo there may be an actual bug within one of the net libs. I would assume someone would help look into it. I'm not asking a question as to how to stream responses or read them as that's already done and working.
I'm experiencing an actual issue where the browser based clients do not receive the full data. I can look into it myself, I just need some guidance so that I know what to look for or where I should be looking.
I've already made some effort in trying to explain as best as I could how to reproduce. I would at least expect the same effort on your end to check it out and tell me with certainty that the issue is probably on the client side. I'm definitely aware this is an open source project and you or other contributors have no obligation to help, but if this is a bug, I'm sure you'd like to fix it at some point.
I would appreciate it if you didn't close this issue as I think it's relevant. For context, it's very similar to https://github.com/golang/go/issues/27816.

Sure, This triage is based on my understanding of your code.
I will reopen it if you think it's a bug.
cc @neild net/http owner

Sure, This triage is based on my understanding of your code. I will reopen it if you think it's a bug.
cc @neild net/http owner

I appreciate that! Thank you! As mentioned, I'm more than happy to look into it if someone points me in the right direction.

我也遇到类似的情况。使用 rust 反向代理测试正常，但是使用 httputil 就会出错〜，模拟发送http请求也不行。像是go的 http 实现有问题或有默认的参数设置问题，目前找不到原因。

I've reported the issue at https://bugs.chromium.org/p/chromium/issues/detail?id=1305928 as well. I've also ran more tests just using net/http and net/http/httputil to see if the issue could be in echo, but I could still get failures.
So that leaves either the browsers or net/http (since I can get failures from net/http w/o using a proxy).

@seankhliao ok, got it. I was under the impression that the browser would read line by line at every occurrence of a \n char.
But I guess you're right since I found a handling text line by line example which illustrates how to handle chunks not ending on line boundaries. It was just not very clear from the docs that the readable stream doesn't necessarily split chunks at line endings.
Thanks for taking the time to check the code I shared and giving more insight into the issue!


ActionView::Template::Error (File to import not found or unreadable: bourbon.

I'm installing rails_admin for the first time and seeing this error right away. Here is the diff of my Gemfile.lock, I don't know whether posting the entire Gemfile or .lock file would be helpful: https://gist.github.com/kbighorse/af4985d16ac665e42e652b5d82cc9ab3
If I git checkout -- . the error goes away. I've encountered the error immediately after modifying the Gemfile and again after running rails g rails_admin:install.
Bourbon is installed:
$ gem list bourbon

*** LOCAL GEMS ***

bourbon (5.0.0.beta.7)

Please use StackOverflow for the support request, thanks.


Add 'Contact Us' to the website footer

Context
As an internet user, if I want to contact the org behind the website, my instinct is to scroll to the footer and look for 'Contact Us'. We don't currently have that. We have the small envelope button which links to this page: https://opencollective.com/contact (which is great), but I almost never use this button on websites because it auto-opens the email client on my computer, and I don't use that either.
MVP

---> which links to https://opencollective.com/contact
(It could go under 'Community' as well)

Note that this icon already links to /contact:

But text links are always clearer, I'm supporting this issue.


Javiercn/js interop improvements part 1

Adds exception sanitization
Adds default timeouts for server-side blazor JS interop calls.
Adds test to validate that completions received for cancelled async JS interop calls don't cause the circuit to fail..

🆙 📅


Wire backups list to the API

Fixes #172

This also fixes some isues with @na3d's work from before he went on vacation.

Coverage decreased (-0.4%) to 93.69% when pulling d7530b86dacf5848f9d0c17877c86102a6bc8f5b on list-backups into 45321776f9027ea3cfbad2883b600b39611afc80 on master.

Nice and simple. Love it.

Should I let the -.4% go here?

I personally don't really care but I could write tests if you ask me nicely

Thanks


Update ci-kubernetes-build-farm.md

Run unit tests step breaking for golang:1.15.
Updated the image to golang:1.17 for which it is working.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

This is approved but there is a break in prod that needs to be fixed before we can merge it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


scripts/Kconfig: Add dt_node_has_prop Kconfig function

Add dt_node_has_prop function to query the presence of 'prop'
for given node label.
Cf use in https://github.com/zephyrproject-rtos/zephyr/pull/30864
Signed-off-by: Erwan Gouriou<EMAIL_ADDRESS>

Can you implement both this and dt_node_has_bool_prop using a single function instead of copy/pasting? Similar to how dt_node_reg_size_int and dt_node_reg_size_hex are both implemented using dt_node_reg.

@mbolivar dt_node_has_bool_prop takes a 'path', while this function takes a 'label'. Do you want me to change this as well ?

Can you implement both this and dt_node_has_bool_prop using a single function instead of copy/pasting? Similar to how dt_node_reg_size_int and dt_node_reg_size_hex are both implemented using dt_node_reg.

@mbolivar dt_node_has_bool_prop takes a 'path', while this function takes a 'label'. Do you want me to change this as well ?

@mbolivar-nordic This patch has been merged as is as part of https://github.com/zephyrproject-rtos/zephyr/pull/30864 (I probably didn't advertised enough this late dependency)
Though, it is not too late to fix as per your request, if you could answer previous comment (I guess with the correct @nickname will work better)

@mbolivar-nordic This patch has been merged as is as part of https://github.com/zephyrproject-rtos/zephyr/pull/30864 (I probably didn't advertised enough this late dependency)
Though, it is not too late to fix as per your request, if you could answer previous comment (I guess with the correct @nickname will work better)

dt_node_has_bool_prop takes a 'path', while this function takes a 'label'. Do you want me to change this as well ?

Heh, yes, I didn't notice that on my first review, but it's a good reason to do it in terms of existing code.
The name dt_node_has_prop is not a good one then, IMO. It should be dt_nodelabel_has_prop; notice other functions like dt_nodelabel_has_compat.
Can you please send a follow-up making dt_node_has_prop and if necessary dt_nodelabel_has_prop?

dt_node_has_bool_prop takes a 'path', while this function takes a 'label'. Do you want me to change this as well ?

Heh, yes, I didn't notice that on my first review, but it's a good reason to do it in terms of existing code.
The name dt_node_has_prop is not a good one then, IMO. It should be dt_nodelabel_has_prop; notice other functions like dt_nodelabel_has_compat.
Can you please send a follow-up making dt_node_has_prop and if necessary dt_nodelabel_has_prop?


Move the plugin from Bash to Golang

In recent times we had lots of discussions regarding whether should we move the plugin to Golang or keep using bash.
I'm opening this issue to get everyone's thoughts here and discuss the positive and negative sides of moving the plugin to Golang.

Currently,  the positive side on moving to Golang I see are

It will easily maintain with new releases and build
New contributors can contribute since not everyone like bash
With bash, it will be difficult to manage the verrrry long scripts which with time it will keep on increasing and it will headache to maintain and will be error prone.

Negative sides on moving to Golang

It will require big time for both the contributor and reviewer to get things going faster.
It will slow down the project
Test the new build process

@travisn @BlaineEXE @Madhu-1 @sp98 @parth-gr ^^

Yes actually as @travisn mentioned, the downside is the transition from bash to golang, once it's done everything will be easy for maintenance and contribution.

I like the idea of using bash helper/wrapper of sort for golang that @travisn  pointed out, we do a similar thing in teuthology(the testing framework for Ceph) and have python as the primary language.

In regards to extensibility using existing Golang libraries such as Cobra and Viper will make adding, e.g., more subcommands, easier as well.

We finally moved to golang.


Minimal context in docs

The docs contain a single paragraph explaining the context within which this package operates: https://scienxlab.org/redflag/index.html
I think it would make sense to provide a bit more information to position the software within the context of the ML field so that users can develop a more intuitive understanding of its benefits and use cases. Relatedly, the JOSS checklist states:

A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?

This is covered nicely in the JOSS paper itself, and perhaps some of this content can be worked into the docs. I actually like the current single paragraph intro in the docs index page before jumping into the quickstart, so perhaps a more detailed overview can be provided on a separate page and referenced from the index page.
Overall, the docs are great. I appreciate the attention given to this. Also, +1 on Furo :)

Ping https://github.com/openjournals/joss-reviews/issues/6065

Thanks for the kind words, appreciated.
Totally agree that more scene-setting was needed -- I often look for this myself in other packages. I've tried to address this with a new 'What is redflag?' page in the docs. I have self-plagiarized it from the draft paper -- I hope that's okay? Happy to re-word if it seems like too much of a shortcut.
Proposed fix in a525174aa6937c351b871a9053904fbfc279c57f


Custom UI: is it possible?

Hello, sorry for my dummy question,
I'm really a newby to pdf-lib.
It look awesome, but would it be possible to customize the UI (prev-next page buttons, etc..) ?
I can't see any reference to the UI on the docs.
Many thanks!

What do you mean with "UI"? A PDF file does not include a UI by any means. The UI you are referring to is probably the one provided by your own reader program (e.g. Adobe/Chrome). Hence you can't modify the UI from this library.

@issimissimo You're most likely referring to the viewer that pdf-lib ships with. In the docs you can see that the viewer has very few options https://pdf-lib.js.org/docs/api/classes/viewerpreferences. While you could monkey patch your adjustments, you'll likely find it easier to use a sperate pdf display library for the time being. I personally use https://mozilla.github.io/pdf.js/getting_started/

pdf-lib doesn't actually provide a viewer at all. The ViewerPreferences are only used to configure how a PDF document is treated when opened by a viewer (like pdf.js, Acrobat, etc...).


Verify scroll on range

Should be able to scroll fine w/ range in input-range and native scrolling

Works, also verified on edge and android


Documentation error for multiple word data types

CrateDB version:
3.2.5
Problem description:
In the docs:
https://crate.io/docs/crate/reference/en/latest/general/ddl/data-types.html
there is this example code for creating a table:
create table my_table13 (
    title text,
    author object as (
        name text,
        birthday timestamp with time zone
    )
)

which, when run in the Crate Console, results in this error:
Error!

SQLActionException[SQLParseException: line 5:28: no viable alternative at input 'author object as (\n name text,\n birthday timestamp with']

Changing the query to:
create table my_table13 (
  birthday timestamp with time zone
)

results in:
Error!

SQLActionException[SQLParseException: line 2:22: mismatched input 'with' expecting {',', ')', 'NOT', 'PRIMARY KEY', 'INDEX', 'STORAGE'}]

Hello @jmellicker,
In the statements that you have posted you use data type names that have not been released yet. It will be available in the future CrateDB releases ( >= 4.0.0). Please, see the documentation for CrateDB data types that corresponds to the version you use.
However, you can use those statements in one of the CrateDB [nightly (https://crate.io/download/#cratedb) builds.

Ah! Thanks!


#392: Fixed incorrect cardinality for Encounter.diagnosis element as …

…well as ProcedureNotDone generated without elements

Closing this PR to resubmit it against the v14 branch


Change the ballerina command to jballerina in vscode extension

Purpose

Change the ballerina command to jballerina in vscode extension

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #15958 into jballerina will increase coverage by <.01%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             jballerina   #15958      +/-   ##
================================================
+ Coverage         14.56%   14.56%   +<.01%
  Complexity         3689     3689
================================================
  Files              1491     1491
  Lines             75467    75433      -34
  Branches           9881     9874       -7
================================================
  Hits              10989    10989
+ Misses            63094    63060      -34
  Partials           1384     1384

Impacted Files
Coverage Δ
Complexity Δ

.../ballerinalang/stdlib/log/AbstractLogFunction.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...in/java/org/ballerinalang/stdlib/log/LogDebug.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ain/java/org/ballerinalang/stdlib/log/LogInfo.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...in/java/org/ballerinalang/stdlib/log/LogTrace.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ain/java/org/ballerinalang/stdlib/log/LogWarn.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...in/java/org/ballerinalang/stdlib/log/LogError.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Shift/Ctrl + Clicks

Was this feature ever considered? I first attempted to use a shift click and realized it conflicted with the middle screen key, but then attempted a control click and had no luck either. Is this something easily done or does it conflict with the mouse/keyboard aspect of this all?

I've never seen the need for it. What is your use case? If you want to open links in a new tab in your browser you can already achieve this using middle click (,).

It's actually more for when I'm doing things in the browser or when I'm using a program and doing selections on things such as files. If you have three files in order, clicking the first will highlight that, then holding shift+clicking the last will select all the files in between, in this example the second, as well as the file clicked. When using control+click, only each additional file clicked will be added to the selection group.
I use this for different things depending on what application I'm using. In my seedbox, I select multiple torrents and perform different actions on groups of them while using this method. Admittedly it's probably not the most useful thing to everyone out there, but shift and control clicking is something I actually did find myself doing frequently. Just figured I'd put it in an issue.
Again, I've been really loving this application! The hinting and then making small adjustments like I saw you mention elsewhere has been amazingly quick! Thanks again!

Thanks for the detailed use case. I've included support for this in the latest commit. Let me know if you encounter any bugs.

No problem at all. I really love this program, so thank you for writing it. It's become one of my favorite pieces of software haha. Just as I was becoming more and more frustrated with Vimium, I saw this!
Everything seems to be working fine as far as shift and control clicking now. If you feel comfortable closing this now, I am personally more than satisfied with the fix.
Thanks again!

Thanks :). I'm closing this for now

Sorry to bother about this issue, but I cannot find how to make a ctrl+click.
What's the method (keystrokes?) to accomplish making a left click while ctrl (or any other modifier key) is pressed?

I believe that once you've updated far enough, it should just work. After I mentioned it to the author, he fixed it pretty quick for me.
Are you having issues with control and shift clicking? What's the problem @fboumol ?

@fboumol What platform (X/wayland/macos) are you using?. This was implemented for macos and X.

I am working using X (under ubuntu 20.04 with xmonad). The problem is that, inside normal (cursor) mode launched with warpd --normal (not with the daemon) , I do not get any effect when pressing "m" (for my left click) together with "ctrl" modifier. The same problem applies to the shift modifier and to the other two buttons.

Sorry, it looks like this was indeed broken. The latest commit should fix it.


Dependency system

Changes introduced with this PR
This PR adds a dependency system to the graph.
It is an alternative way to use the graph. It does not break the existing way of using it. This was tested.
Unfortunately I didn't catch the fact that the white-space format was completely overwritten. Disregard lines that only have white-space changes.

By contributing to this repository, I agree to the contribution guidelines.

This update is excellent! Nice job, @jaredoconnell. (I'm sorry that you felt the test updates were so challenging -- in hindsight, they look very straightforward, like we might have been able to make most of the changes with a regex-find-and-replace, which would have been fun! 😀)
Below, I've called out a few items for you to consider: there's an assertion which you might want to restore, and I have a coding suggestion; and, there are a few nits.

It wasn't too bad overall. I did use regex!

assert.SliceContains(t, (\w+), readyNodes)

Replaced with

assert.MapContainsKey(t, $1.ID(), readyNodes)

For example.


Example of how to use with arguments?

I have an existing async function that takes four arguments:
pub async fn sync(
  dbpath: &Path,
  file_path: &Path,
  uid: i64,
  callbacks: &mut Callbacks,
) -> Result<PrivateReplyMessage, Box<dyn std::error::Error>> {

I'd like to run this in a job.  Currently async closures are nightly, so I'm looking at making a function that takes the above arguments and returns another function that takes Monitor, runs sync, and sets the Job Status to 'sync complete' or whatever.  The types and whatnot are a bit tricky though, would be nice to have an example.

Good point, I should probably add some examples.
It looks like you have a bunch of borrowed data that the job needs. Since the job runs "in the background", you need to ensure they are either 'static, or wrap them in Arc or similar. (Or you can give ownership of the values to the job.)
Perhaps you're looking for something like this:
let dbpath: PathBuf = todo!();
let file_path: PathBuf = todo!();
let uid: i64 = todo!();
let callbacks: Callbacks = todo!();

let job = Job::start(move |mon| async move {
    writeln!(mon, "starting!");
    sync(&dbpath, &file_path, uid, &mut callbacks).await.unwrap();
    writeln!(mon, "done!");
});

Let me know if you have any other questions!

Wow quick response!  I gave that a go:
    PrivateRequests::SyncRemote => {
      let dbpath: PathBuf = config.orgauth_config.db.to_path_buf();
      let file_path: PathBuf = config.file_path.to_path_buf();
      let uid: i64 = uid;
      let job = Job::start(move |mon| async move {
        writeln!(mon, "starting!");
        sync::sync(&dbpath, &file_path, uid).await.unwrap();
        writeln!(mon, "done!");
      });

      Ok(PrivateReplyMessage {
        what: PrivateReplies::SyncStarted,
        content: serde_json::to_value("meh".to_string())?,
      })
    }

Because Callbacks is not Send, it was causing errors.  For now I just moved that bit into the sync() function (didn't help 😄).
Unfortunately that reveals that the sync function is chock-full of less tractable lack-of-Send errors in addition to the Callbacks, notably these third party lib issues:

awc::Client::new()
tokio_util::io::StreamReader;

These I don't think I can do without, or change!   I don't suppose there's any cure for this lack of Send?
error.txt

Looks like this is due to tokio::spawn requiring Send, and the requirement is passed on up.  The nature of async thread pooling?  Maybe I'll just spawn regular threads for this.

Yeah, that's the nature of async runtimes -- there's no guarantee which thread will run the task, and tasks can even switch threads at any await point.
You could consider using tokio::task::spawn_blocking to run the code in a blocking thread:
let job = Job::start(move |mon| async move {
    writeln!(mon, "starting!");
    tokio::task::spawn_blocking(move || {
        // do your non-Send things here...
    })
    .await;
    writeln!(mon, "done!");
});

Yeah that compiles.  Even can pass in the monitor.  Thx for the help!!

Glad to help! :)

Well I got around to trying the spawn_blocking solution, and while it did compile, it was just doing the thing where you get a future back which is never executed, lol.  Adding an .await produces an error.  It seems spawn_blocking gives you a thread to execute non-async, non-Send code, but my function is async and non-Send.
For now I resorted to just starting a regular thread and creating a new runtime and localset there.  It runs!
      let job = Job::start(move |mon| async move {
          std::thread::spawn(move || {
            let rt = Runtime::new().unwrap();
            let local = LocalSet::new();
            let mut callbacks = &mut zknotes_callbacks();
            let r = local.block_on(&rt, sync::sync(&dbpath, &file_path, uid, &mut callbacks));
          });
        });

I did find this other approach in the tokio docs, where they suggest creating another thread with its own persistent localset and runtime.  Then you send tasks to it with mpsc.  I guess the rationale is to save creating and destroying the runtime and localset each time you want to run a task.
https://docs.rs/tokio/latest/tokio/task/struct.LocalSet.html#use-inside-tokiospawn
This seems very like the kind of thing one would find in a job manager!  ; )

Right, I suppose that makes sense. Looking into it a bit more, tokio requires Futures to be Send, and some of your libraries (awc) are incompatible with this assumption. I guess you should be using actix_rt instead, which allows !Send futures.
Right now, girlboss uses Tokio to spawn its tasks, which means it requires Send. Perhaps I could refactor my library to support multiple runtimes instead of just Tokio.


Use topologicalSort with Identifiable on ResolvedTarget

Motivation:
A serious performance regression was introduced for deep target dependency graphs in https://github.com/apple/swift-package-manager/pull/7160. After converting ResolvedTarget to a value type, its synthesized Hashable conformance traversed the whole dependency tree multiple times when using topologicalSort. This made package resolution seemingly hang for packages that had a lot of nested target graphs.
Modifications:
We already have an implementation of topologicalSort on Identifiable, so we're using that now instead. I've also added a performance test for this in PackageGraphPerfTests to prevent future regressions in this area.
Result:
Resolved rdar://119807385.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test windows


Create project's Github page

I think we should have a page to show off the screens and the features.

@cecilia-ombulabs What's the status of this?

@etagwerker sorry, I was not aware of this issue. I'll work on it.


helmfile sync --interactive does not ask for confirmation before applying

Operating system
macOS 12.15.1
Helmfile Version
v0145.4
Helm Version
version.BuildInfo{Version:"v3.9.4", GitCommit:"dbc6d8e20fe1d58d50e6ed30f09a04a77e4c68db", GitTreeState:"clean", GoVersion:"go1.19"}
Bug description
When I run sync with the --interactive flag, I do not get a confirmation before helmfile makes changes to my cluster.
Example helmfile.yaml
N/A
Error message you've seen (if any)
N/A
Steps to reproduce
N/A
Working Helmfile Version
N/A
Relevant discussion
No response

@WoodyWoodsta PR is welcome.

@WoodyWoodsta From the code. In sync subcommand, no logic for interactive.

@WoodyWoodsta I will try to add it in sync. wait my PR or your PR. haha

Thanks @yxxhero - if I get some time I can have a quick look but I work full-time, so we'll see 😄

@WoodyWoodsta OpenSource requires all of us.

Hey! Thanks for reporting and investigating.
Probably this was an unintended regression introduced when we migrated our CLI framework from urfave/cli to cobra?
Just to be clear, the point of --interactive was to let Helmfile ask AFTER showing the diff and BEFORE actually applying the changes.
So, from the author's perspective, I can say it's working as intended.
Please use helmfile apply if you need interactivity!

And we might be adding support for sync and delete as well via #328

@mumoshu Thanks - explained that way I now understand that apply is a "superset" command including diff and sync, and that --interactive is to ask for confirmation before those two steps. Coming from kubectl apply and helm upgrade, I assumed sync to be more of the latter (which does a diff internally).


chore: Update deps

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ba9456c) 86.28% compared to head (fc2a899) 79.05%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   86.28%   79.05%   -7.24%
==========================================
  Files         688      688
  Lines        9750     9750
  Branches      858      858
==========================================
- Hits         8413     7708     -705
- Misses        619     1295     +676
- Partials      718      747      +29

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


initial approved cellar cache feature

Closes #93

This one is still pending merge conflict resolution.

Another ping here, an additional merge conflict after the ABI update.

I see this has been moved into draft -- is it still a work in progress or ready for review?


[FEATURE] Add m1/aarch64-darwin support

Description of the requested feature
Add aarch64-darwin (m1) support for macOS, currently the flake fails with the following output:
~/nix-darwin-dotfiles (main ~~9 ...4) λ nix build ~/nix-darwin-dotfiles\#darwinConfigurations.shaunsingh-laptop.system
warning: Git tree '/Users/shauryasingh/nix-darwin-dotfiles' is dirty
error: attribute 'aarch64-darwin' missing

       at /nix/store/mgcpf2l79r3zwq937k51cv8d74376c3q-source/flake.nix:21:23:

           20|
           21|         naersk-lib = (naersk.lib."${system}".override {
             |                       ^
           22|           cargo = pkgs.cargo;
(use '--show-trace' to show detailed location information)

Additional context
https://github.com/nix-community/naersk/commit/ee7edec50b49ab6d69b06d62f1de554efccb1ccd naersk supports aarch64-darwin, not sure whats the issue. This fix was also implemented 4 months back, and the flake.lock was update 2 months back

For testing, I overrode the inputs and matched them with the latest from github: https://github.com/shaunsingh/nixpkgs-s2k/blob/2b5abc67bc3bec1fa5d514c1eb53e32559bb2fb8/flake.nix#L60, same error. @AusCyberman any idea, I'm not very experienced with nix flakes

this could be that naersk straight up doesn't support aarch64-darwin. ill do some research

This project https://github.com/yaxitech/ragenix supports aarch64-darwin and uses naersk, so I think it should be possible

maybe someone could actually start developing for it, anyone in up who uses mac?

From: ElKowar @.>
Sent: Thursday, November 18, 2021 11:24:00 PM
To: elkowar/eww @.>
Cc: AusCyberman @.>; Mention @.>
Subject: Re: [elkowar/eww] [FEATURE] Add m1/aarch64-darwin support (Issue #353)
Even if you did manage to compile this on MacOS, it'd not actually be all that useful, as we currently don't interact with the macos display server at all, and thus eww behaves like any normal window (again, assuming this works at all). Then, because MacOS and GTK are weird, from what I've heard eww doesn't even behave like a normal window, but also get's completely ignored by things like yabai, making it impossible to actually position the window where you want it.
This is sadly not really feasible for me to fix, as maintaining support for a platform I have absolutely no way of developing for or even just testing on is not something I'd be willing to do
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/elkowar/eww/issues/353#issuecomment-972818748, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC4FK5RXTMHTDZDPN5VHHHLUMTV6BANCNFSM5ICZTBJQ.

see, even in that case, the build would probably constantly break, and I'm not even sure how you'd do CI for a mac build and test ^^'


Custom path of babel.config (for babel-register) ?

mocha test.js --require=@babel/register

scripts/babel.config.js
package.json

How can I get this to work without adding a new file or moving my Babel config?

What version of mocha/babel etc?
How does your app point to that babel config? Does that work?
Any reason you did not want to place the config in root (or package.json), which i believe is the recommended approach?

@babel-core v7.5.0
mocha v6.1.3

"scripts": {
  "build": "babel lib/ --config-file=./scripts/babel.config.js --out-dir=lib-cjs/ --source-maps"
}

I prefer not to have too many files in the project root.

@stevenvachon you may need a new file to load babel and specify the config value, see below. AFAIK there is no way to pass a babel cli option from mocha cli directly.
mocha config
require: 'scripts/register-babel.js'

scripts/register-babel.js
require("@babel/register")({
  configFile: "./scripts/babel.config.js",
});

Does that help?

Does that help?

Yup! Thanks!
I didn't use a mocha config file, though; instead opting for the --require CLI option.


Description

#@Description:
Fixes

#

Describe your changes.
Related issue:
Add link to the related issue.
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

ProTip

It's a great one

I'm going to close the pull request, because it looks like a spam.


CSV validation issues [check]

So as discussed, we have a file from WB that is expanded and comes from test which looks pretty strange in CSV on PC Office 365, see:
https://docs.google.com/spreadsheets/d/1mF65-wa4f_HVc948Iz0Qvg0zEHJdM-3UnTdfdHkclvo/edit?usp=sharing
This is the CSV validation report: https://csvlint.io/validation/5f5a342408d9380004000060
Then we have a SIDA CSV file @ https://docs.google.com/spreadsheets/d/16tygCiAoB7a9v_fzInJKYYAr8cQjPRw6rzaLF0hDJ8o/edit?usp=sharing that has the following CSV validation report: https://csvlint.io/validation/5f5a350708d9380004000061
And one from live -non CSV Expanded- @ https://docs.google.com/spreadsheets/d/1oYap_nRFpx8dqtOsHDMBOvMMdRYzgUNgJIrPpSVsyKs/edit?usp=sharing  CSV validation report
https://csvlint.io/validation/5f5a34965f85c60004000094  (original request on test @ https://test-datastore.iatistandard.org/search/activity-sector?q=reporting_org_ref:(SE-0)&wt=xslt&tr=activity-sector-csv.xsl&rows=10)
That LIVE version just has 1 warning and 1 message. The WB Expanded file does open on PC Office 365, but in a very strange way. and is not usable on Excel for PC.
The SIDA file has similar white spaced structure that looks odd and the CSV does not even properly validate.
It would be good to solve all the messages and errors on the CSV Expansion exports to a bare minimum, I would also like to have UTF-8 as our standard export, not any ASCII type,

Hi @siemvaessen in several exports I've made from the Datastore, the solution to making the file usable for Microsoft Excel was adding a UTF8 Byte Order Mark (BOM) to the top of the file. I'm not sure if this would fix the spacing issue, but it makes the file usable across a greater variety of software.

Hi @akmiller01 as team has made changes today, please recheck. According to csv files I have downloaded in the last few hours and using file --mime filename.csv renders an expected text/plain; charset=utf-8 - how did you check encoding and what file did you check?

I was passing files (arbitrary CSV exports) to a Windows user, who said that special characters (accent marks) were not rendering correctly in Microsoft Excel. It's not easy for me to check on my laptop because I'm running Ubuntu.
Ultimately I'm using an R package to add the BOM, but it's hard to find the real source of this package because it's written in C++. https://github.com/tidyverse/readr/blob/296f30624c218293b19ecfe13739ad0b1d16709d/R/write.R#L98-L112
But as far as I can tell from their docs, all that function is doing is adding a 'U+FEFF', Byte Order Mark (BOM), and that worked well for the Windows user I was making exports for.

Petya is doing some testing on Windows, and is getting incorrect encoding opening the files natively in Excel.

Just for the record, CSV export is totally usable it is Excel for PC that does not interpret the encoding properly, the Datastore xslt file extracts the data from the Solr fields which contain the accent marks as you can see on Ubuntu, Mac, Google Drive w/o any problem. Happy to look into the matter of how to manipulate the CSV file header, but closing this ticket and raising a new one for that.


Remove microdata from product miniatures

Questions
Answers

Description?
Microdata is not useful on listing pages because it is already present on product pages in JSON-LD.

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/Prestashop#30543

How to test?
See issue

Ping @NeOMakinG for review.

PHPStan errors are fixed there:
https://github.com/PrestaShop/productcomments/pull/149
@Hlavtox you could approve :D

Hello @NeOMakinG
Could you please check the CI failures?

We think to do a quick check of the issue.
Thank You!

@HanaRebaiQA It complains, that we have a list of errors to ignore, which are no longer thrown. The tests are OK. Tests will stop complaining after https://github.com/PrestaShop/productcomments/pull/149 is merged. 👍

thank You @Hlavtox We are waiting for other PR then :)

Thanks everybody 🎉

@micka-fdz Bro the PR is merged, rebase it, reapprovals and we can QA.

@micka-fdz yes, I'm afraid you need to rebase your PR

@Hlavtox @kpodemski rebase done ✅


Simplify dictionary to schemas process (and make more conventional)

Currently, the yaml dictionary specifies whether it is intended to be a "json" or "csv" portion of the respective schema. However, this is not standard jsonschema. It is possible to make this standard with the following proposal
In future, instead of the current dictionary with specs:
 encodings:
    title: 'Variable Value Encodings (i.e., mappings; value labels)'
    description: |
      Encodings (and mappings) allow categorical values to be stored as
      numerical values. IMPORTANT: the ==key should be the value represented IN
      the data== and the ==value should be the to-be-mapped label==. Many
      analytic software programs use numerical encodings and some algorithms
      only support numerical values. Additionally, this field provides a way to
      store categoricals that are stored as  "short" labels (such as
      abbreviations)
    _jsonSpec:
      type: object
      examples:
        - '{0:"No",1:"Yes"}'
        - '{"HW":"Hello world","GBW":"Good bye world","HM":"Hi, Mike"}'
    _csvSpec:
      $ref: "#/definitions/csvObject"
      examples:
        - '0=No|1=Yes'
        - 'HW=Hello world|GBW=Good bye world|HM=Hi,Mike'
      ```

      It will be easier and more readable, less complex, amd potential more interoperable to simply specify a large json schema spec with the anyOf keyword instead _csvSpec and _jsonSpec:

      ```yaml
    title: 'Variable Value Encodings (i.e., mappings; value labels)'
    description: |
      Encodings (and mappings) allow categorical values to be stored as
      numerical values. IMPORTANT: the ==key should be the value represented IN
      the data== and the ==value should be the to-be-mapped label==. Many
      analytic software programs use numerical encodings and some algorithms
      only support numerical values. Additionally, this field provides a way to
      store categoricals that are stored as  "short" labels (such as
      abbreviations)
      multipleformats: True # need to indicate there are multiple formats
      oneOf:
      - type:object #json spec
      - $ref: #/definitions/csvObject #csv spec
      ```

Another option would be to call these two variables different things -- e.g. encodings must always be in the 1=value|2=value format, while encodingsJson must always be a valid JSON dictionary in the format {"1": "value", "2": "value"}.

@gaurav Good idea! How about instead of 'encodingJson,' we call it 'encodingStringified' or something? (Just to not confuse - as it's technically not a valid json string).

#29 was merged

Closing out


Feature: Quick Sniper

Add feature to remove Scope quickly.

This might be useful. @danielkrupinski

Added auto scope to aimbot if this is what you meant.


fix usage and quoting to fix hub

This is a:

[X] bug fix PR with no breaking changes — please ensure the base branch is master
[ ] new functionality — please ensure the base branch is the latest dev/ branch
[ ] a breaking change — please ensure the base branch is the latest dev/ branch

Description & motivation
hub.getdbt.com isn't looking so great.  I think the culprit is the use of a > for quoting in markdown.  I'm submitting this PR to hopefully then see this fixed over at hub.getdbt.com.  While I was here, I also ensured that all usage headings were bolder for easier reading.
Here is an screenshot of hub.getdbt.com that isn't looking so hot right now.

Checklist

[X] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[ ] Postgres
[ ] Redshift
[ ] Snowflake

[X] I have updated the README.md (if applicable)
[X] I have added tests & descriptions to my models (and macros if applicable)
[X] I have added an entry to CHANGELOG.md

(closing out all things that might be open on my list) feel free to merge, not merge, close / whatever seems appropriate here 😄

Changing the base to dev/0.7.0 to avoid making another 0.6.x release


Flask app + Cognito Authorizer not working

Context

I have a flask API deployed with zappa and I want to integrate a Cognito User Pool authorizer to control API access. The API is called from a react PWA using the aws-amplify library, which embeds the requisite auth headers (x-amz-security-token, Authorization) into the request automatically.
    async getData() {
        let apiName = 'media';
	    let path = '/';
	    let myInit = {
	    	response: true
	    };
	    return await API.get(apiName, path, myInit);
    }

    componentDidMount() {
        this.getData().then(response => {
            console.log(response.json)
            this.setState({
                data: response.json,
            });
		}).catch(error => {
		    console.log(error)
		});
    }

I am unsure about the Cognito Authorizer and what exactly it expects - the AWS Amplify library suggests that no additional headers are required. Has anyone had a similar experience?
CORS is enabled on the API.
The Cognito Authenticated IAM role:
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Effect": "Allow",
            "Action": [
                "mobileanalytics:PutEvents",
                "cognito-sync:*",
                "cognito-identity:*"
            ],
            "Resource": [
                "*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": [
                "execute-api:Invoke"
            ],
            "Resource": [
                "*"
            ]
        }
    ]
}

Expected Behavior

The API endpoint should return a 200 response.
Actual Behavior

The API endpoint returns a 401 response.
Error: Request failed with status code 401
    at createError (createError.js:16)
    at settle (settle.js:18)
    at XMLHttpRequest.handleLoad (xhr.js:77)

Possible Fix

Do I need to configure flask itself to handle the authorizer?
Steps to Reproduce

Your Environment

Zappa version used: 0.45.1
Operating System and Python version: python 2.7.12
The output of pip freeze:

argcomplete==1.9.2
base58==0.2.4
boto3==1.7.14
botocore==1.10.14
certifi==2018.4.16
cfn-flip==1.0.3
chardet==3.0.4
click==6.7
docutils==0.14
durationpy==0.5
Flask==1.0.2
Flask-Cors==3.0.4
future==0.16.0
futures==3.1.1
geographiclib==1.49
geopy==1.13.0
hjson==3.0.1
idna==2.6
itsdangerous==0.24
Jinja2==2.10
jmespath==0.9.3
kappa==0.6.0
lambda-packages==0.19.0
MarkupSafe==1.0
pkg-resources==0.0.0
placebo==0.8.1
python-dateutil==2.6.1
python-geohash==0.8.5
python-slugify==1.2.4
PyYAML==3.12
requests==2.18.4
s3transfer==0.1.13
six==1.11.0
toml==0.9.4
tqdm==4.19.1
troposphere==2.2.1
Unidecode==1.0.22
urllib3==1.22
Werkzeug==0.14.1
wsgi-request-logger==0.4.6
zappa==0.45.1

Link to your project (optional):
Your zappa_settings.py:

{
    "dev": {
        "app_function": "$APPNAME.api.api.API",
        "aws_region": "ap-southeast-2",
        "profile_name": "api_user",
        "project_name": "api",
        "runtime": "python2.7",
        "s3_bucket": "zappa-********",
        "environment_variables": {
            "BUCKET_NAME": "$APPNAME-dev-media",
            "REGION_NAME": "ap-southeast-2",
            "PYTHON_EGG_CACHE": "/tmp"
        },
        "domain": "************",
        "cors": true,
        "certificate_arn": "arn:aws:acm:us-east-1:****:certificate/*******",
        "context_header_mappings": {
            "user_id": "authorizer.user_id"
        },
        "authorizer": {
            "type": "COGNITO_USER_POOLS",
            "provider_arns": [
                "arn:aws:cognito-idp:ap-southeast-2:****:userpool/ap-southeast-2_************"
            ]
        }
    }
}

Ok, I found it.  I needed to set iam_authorizer: true in zappa_settings to utilise the AWS v4 signing used in AWS Amplify.


download_post won't take sub folder as param

Describe the bug
download_post(postID, path)  always generates the folder named in path even if it's a subpath like media/shortcode
To Reproduce
import instaloader
L = instaloader.Instaloader()
post = instaloader.Post.from_shortcode(L.context, shortcode)
L.download_post(post, 'media/'+shortcode)

Expected behavior
Download the image in the subfolder
Error messages and tracebacks
no error
Instaloader version
4.10.3
Additional context
Tried with
BASE_DIR = Path(__file__).resolve()
os.path.join(BASE_DIR, 'media', shortcode)

It still creates a single folder named with the full path

Ok it seems what I was looking for could be achieved with
L.dirname_pattern = os.path.join('media', '{target}')


feat: log CommitStats if requested by the client application

Adds support for requesting CommitStats for read/write transactions. CommitStats that are returned by the backend are logged using the default logger of the connection.
The PR currently also includes the necessary proto changes in the first commit , as the proto changes has not yet been published.
Replaces #5480

I'll have a look at this on Monday.

CC @skuruppu

This looks great, thanks.
Could you rebase so that it's just two commits, and force push?

Done.

One problem: the manually-regenerated files will keep being regenerated "back" automatically, so it might be best to get this PR into a state where we'll be able to merge it when the protos go public, but wait until then. Does that make sense?

I'm not sure I exactly understand what you suggest in this case (?) I guess one option might be to create a new branch once the proto changes are public and cherry pick the second commit to that branch.

I'm not sure I exactly understand what you suggest in this case (?) I guess one option might be to create a new branch once the proto changes are public and cherry pick the second commit to that branch.

I'm suggesting we leave this pull request open for now. Once the proto changes are public, you can sync, then rebase interactively again, removing the first commit. Wait for green, then merge.

Removed logging of the overload_delay field that will be removed from the proto.

I've merged https://github.com/googleapis/google-cloud-dotnet/pull/5875, which should mean we can rebase this without the first commit... are we then happy to merge this PR?

I've merged https://github.com/googleapis/google-cloud-dotnet/pull/5875, which should mean we can rebase this without the first commit... are we then happy to merge this PR?

@olavloite the proto change required for this was published and the generated code is now available (#5875). You should be able to rebase now.

@olavloite the proto change required for this was published and the generated code is now available (#5875). You should be able to rebase now.

I've merged #5875, which should mean we can rebase this without the first commit... are we then happy to merge this PR?

I don't think the backend support is there yet for this feature so we should hold off on merging for now.

I've merged #5875, which should mean we can rebase this without the first commit... are we then happy to merge this PR?

I don't think the backend support is there yet for this feature so we should hold off on merging for now.

The PR is rebased on master and the previous commit with the generated changes has been removed.

The PR is rebased on master and the previous commit with the generated changes has been removed.


Feature/5826 validation

What
Trello Ticket Here
Add validation for multivariate filters.
separate out flexible validation.
Add Checks for cantabular dimension existence when multivariate.
Ensure that component tests are added to show multivariate ability.
How to review
Check the component tests to ensure that the test cases are covered.
Who can review
Team B can review

Some comments r.e formatting.
A bit worrying a missing return wasn't picked up by tests or otherwise.
I suspect the existing code around getCantabularDimension could be refactored take the populationType directly rather than the filterID, removing the need for two separate functions. I may be wrong but would probably be preferable if possible

Yeah I actually put a test in before this comment about putting in missing dimensions, and caught it before I saw the comment apologies for that, will work on the refactor on the getCantabularDimension right now


[zh] Translate Windows pages，add windows-security.md chinese version

Work for #34033 win-3
add windows-security.md chinese version

@zhangxyjlu Please read the style guide and strictly follow it.

/approve

/lgtm


Remember selected panels and handle case when user hasn't logged in

This PR implements:

The selected panel for each connection is stored in the component state and new connections default to the Connection panel.

If the users hasn't logged into Plotly, the Plot.ly panel shows a link to the login page.

@tarzzz @chriddyp I've just tested master and username isn't working in the desktop or the web app (see the screen captures below). I'll update this PR once this issue with login is fixed.

@n-riesco For server/headless version, you need to add the following lines to your settings.yaml:
ALLOWED_USERS:
    - n-riesco

The issue in electron-app is a known issue, but I have been unable to replicate it on my machine so far. Adding those lines to settings.yaml would let you authenticate on server-app and should be good for your dev. for this PR..

@tarzzz I've added those lines to settings.yaml and fix the auth error, but I get stuck on the login page. And if I reload the localhost:9494, I only see the link to login again.
See the screen capture:

@tarzzz @chriddyp I've updated the PR:

I've fixed the loadURL issue I showed above, so that now oauth2 works in the desktop app.
I've changed the approach to determine whether the user is logged (now it uses username).
I've also tried to address some of @chriddyp 's feedback in https://github.com/plotly/streambed/issues/10160#issuecomment-337425410 (moved the login/logout link in the desktop app into the Plot.ly panel)

Oauth2 isn't working yet in the web app (even after setting ALLOWED_USERS). Do you want me to have a look at this issue?

@n-riesco I suspect the Oauth issue is related to SSL_ENABLED setting.
When SSL_ENABLED=true in settings.yaml, secure flag is set on cookies, and they are not sent over http connections.

Closing since all the commits in this PR have been cherry-picked into #206.


Command vmaf --reference-vfilter  scale only works with full actual values (not -1)

Reference: 4k
Encode: 540p
Attempting to use scaling argument as documented/given as example in the help for vmaf, results in error thrown by ffmpeg. Specifically, attempting to use scale=960:-1, results in [Parsed_libvmaf_9 @ 0000029ab55a6dc0] input height must match error.
If using scale=960:540, it works. This only happens when using vmaf command. An example with the -1 argument is also given in the help for vmaf.
Error thrown:
.\ab-av1.exe vmaf --reference-vfilter scale=960:-1:flags=lanczos --reference "reference.mp4" --distorted "distorted.mkv"

Error: ffmpeg vmaf exit code -22

---stderr---
[...]
[Parsed_libvmaf_9 @ 0000029ab55a6dc0] input height must match.
[Parsed_libvmaf_9 @ 0000029ab55a6dc0] Failed to configure input pad on Parsed_libvmaf_9
[fc#0 @ 0000029ab554f0c0] Error reinitializing filters!
[fc#0 @ 0000029ab554f0c0] Task finished with error code: -22 (Invalid argument)
[fc#0 @ 0000029ab554f0c0] Terminating thread with return code -22 (Invalid argument)

I just tried to perform another comparison (different source vs encode set) and this time it's not working at all, ffmpeg keeps complaining about height and width for unexplainable reason. Again, using --reference-vfilter scale=960:540:flags=lanczos.
ffprobe output of source and encode:
Stream #0:0[0x1](und): Video: h264 (High) (avc1 / 0x31637661), yuv420p(tv, bt709/bt709/unknown, progressive), 1920x1080 [SAR 1:1 DAR 16:9], 7612 kb/s, 59.94 fps, 59.94 tbr, 60k tbn (default)

Stream #0:0: Video: hevc (Main 10), yuv420p10le(tv, bt709), 960x540 [SAR 1:1 DAR 16:9], 59.94 fps, 59.94 tbr, 1k tbn (default)

So, perfect 16:9 square pixel resolutions, the scaling should not produce any mismatch.

I agree that I would expect -1 to work here. But all ab-av1 is doing in your usage example is passing the vfilter to ffmpeg process. I can't see any action I can take to improve this.
You could try debugging to see the ffmpeg call that is generated and run that yourself. With that you could ask upstream why it behaves as it does.

Thanks for the debugging suggestion.
For the original issue, I see now that actually the SAR is different for the encode because I chose an option in Handbrake that maximizes the storage resolution to which a non-square pixel SAR is applied. This SAR discrepancy is caused by the encode "storage" resolution (the actual resolution of the video stream) which is forced to 960x540 which IS 16:9 but the source file resolution is 4096x2160 so not 16:9.  Thus, the encode has a SAR of 16:15 to match the source aspect ratio.
For the second point I mentioned, the error was caused by me because I supplied the source file also for the --distorted arg.
Apologies for the confusion, it is a bit more difficult sometimes working from a CLI to prevent mistakes.
it helped me see that the error was on my part for the second scenario I mentioned, in error I was supplying the source to the --distorted arg instead of the encode file.


Bug with option button

Hi,
I have a bug with the options I set minimal and I can not go back into the options.
https://gyazo.com/03f137794e166aac4baa914b6a16d765
Thank you

press ctrl + , to open settings


Edit/Change related button popup

Is there a way to add a Edit/Change button next to a model when overriding the field, just as with the add button there is the "add_another_url_name" parameter?
Would it be too difficult to add this functionality similar to this block
{% block add_another %}
    {# A link to add a new choice using a popup #}
    {% if autocomplete.add_another_url_name %}

    {% endif %}
{% endblock %}

There would be several ways, did you read the documentation about styling autocompletes ?

Did you sort it out ? Can we close this ?

Hello,
The link @jpic you write doesn't exists anymore. What would you advice to do for adding an edition link near to the current select button with last version of DAL. It would be usefull for OneToOneField() as well as for GenericForeignKey().
Thanks

Did you try https://github.com/jonashaag/django-addanother ?

I hate to ask in this existing thread, but @jpic any chance you could provide some input on how to integrate django-addanother with dal v3 using their ModelSelect2EditSelected instead of DAL's ModelSelect2? Specifically, I'm trying to get an Edit button on my DAL select2 widgets. But, the docs of addanother seem to be based on the stock django-select2 widgets...

I think every wrapper class works the same:
https://github.com/jonashaag/django-addanother/blob/master/django_addanother/widgets.py#L93-L114
You can use any of them around your own widget instance, it just needs the url to use for the buttons:
https://github.com/jonashaag/django-addanother/blob/master/docs/usage.rst
https://github.com/jonashaag/django-addanother/blob/master/docs/edit-related.rst#id1


AMBARI-24199 : Fix Atlas HA support in Ambari via blueprint deployment.

What changes were proposed in this pull request?
Currently deploying Atlas in HA mode via blueprint is not possible due to property atlas.server.bind.address being set by Blueprint Processor, need to remove the property from being processed in Blueprint Processor and let the default value to be effective.
How was this patch tested?
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.ambari.server.controller.internal.BlueprintConfigurationProcessorTest
[INFO] Tests run: 185, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 5.656 s - in org.apache.ambari.server.controller.internal.BlueprintConfigurationProcessorTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 185, Failures: 0, Errors: 0, Skipped: 0
Please review Ambari Contributing Guide before opening a pull request.

Hey @vishalsuvagia, could you give an example where blueprint deployment fails with the original code but succeeds with yours?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2896/
Test PASSed.

Hi @benyoka, Thank-you for the concern.
When Atlas-Server is to be installed on two or more than hosts, the blueprint processor tries to replace the value of property atlas.server.bind.address with the f.q.d.n of the host where Atlas is to be installed, this leads to error as the blueprint-processor is unable to resolve which host should be used to populate the property atlas.server.bind.address.
Ideally the value for atlas.server.bind.address is resolved internally on the agent side, so the value should not be processed by the blueprint-processor.

@vishalsuvagia Isn't atlas.server.bind.address configurable by the user?  It seems to me that it is always overwritten at the agent side, so this shouldn't be a config property in the first place.

How about using OptionalSingleHostTopologyUpdater instead of SingleHostTopologyUpdater? This would probably give backwards compatibility with HDP 2.6
cc @vishalsuvagia @rnettleton @adoroszlai

Closing the issue as it seems there is a work-around for the issue, Adding property atlas.server.bind.address in the blueprint with value as <IP_ADDRESS> seems to fix the problem.

It looks like the SingleHostTopologyUpdater prevents ATLAS_SERVER from being placed in multiple host groups.  Other HA components (eg. NameNode, Hive Metastore, etc.) have hard-coded logic, but Atlas Server runs into the IllegalArgumentException:
https://github.com/apache/ambari/blob/cc5e30c6350446c323fe5595691289b1af8332d0/ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java#L1866-L1946

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2915/
Test PASSed.

@adoroszlai , thank-you for re-opening the issue, seems I was mistaken, as I only had single Atlas configured in the blueprint in place of two hosts, I re-verified by adding an additional Atlas thereby enabling Atlas-HA and was able to reproduce this issue getting IllegalArgumentException.
@rnettleton ,
The property was added in blueprint processor for Hive-Atlas integration via AMBARI-13129, to resolve the property atlas.server.bind.address for Hive Atlas configurations and Atlas did not have High-Availability support then which was later added see ATLAS-510, hence the property is better left unprocessed as it is already handled on the agent side.

@adoroszlai , It looks like Atlas HA support was never implemented for Blueprints.
This work would require a few changes to the Blueprint processor, and should be implemented in a later release.
Thanks.
@vishalsuvagia , Can you please close out this issue?
Thanks.

closing the issue for now.


Remove Card default shadow options

By default, it seems like <Card /> features a box shadow on top of it. However if we want to setup a transparent navbar, we end up with something like this:

The idea would be to give access to these style properties tp disable this box shadow.

Problem solved: thanks!


Roll Engine from 56884b7c8858 to c03f14413073 (4 revisions)

https://github.com/flutter/engine/compare/56884b7c8858...c03f14413073
2021-06-29<EMAIL_ADDRESS>[iOS TextInput] Disables system keyboard for TextInputType.none (flutter/engine#26979)
2021-06-29<EMAIL_ADDRESS>Android: add support for TextInputType.none (flutter/engine#26585)
2021-06-29<EMAIL_ADDRESS>[refactor] Migrate to View.focus.*. (flutter/engine#27005)
2021-06-29<EMAIL_ADDRESS>Roll Dart SDK from e6e47f919791 to 59f9594aed9a (1 revision) (flutter/engine#27032)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Android Login Redirects no Longer working for oAuth2

My application was working a few weeks ago and out of the blue no re-directs seem to work any more. I log in, I get sent to the redirect with token, but then it stops at that point as if the oAuth flow has stopped. On iOS the same code seams to work fine in my app.

@jamesmontemagno I saw you closed this issue. Did you figure it out?

I have no idea what was causing it in general to be honest with you. Not sure if Google changed something in the TLS stack or what but in the Auth flow essentially we got back the redirect URL with code and then when we would go to Post to get the token it would just die, so it became stuck. I am puzzled because I haven't updated the app in about 2 months and it stopped working randomly for people last week.

Ok, I'm going to reopen this issue for visibility because we are getting other TLS reports. Of note we just did a large scale migration to AWS and in the process we've also changed our ssl certificates to a newer flavor which require SNI support on behalf of clients.

Ahhhh, yeah so I switched my android stack over I think from using TLS 1.1 over to TLS 1.2+

Let me know if I can help any way.

Did switching to TLS 1.2+ resolve the issue of introduce the issue?

Yup, I believe that is what I needed to do. I don't think I had an issue with iOS because I had made that switch a while ago in the networking stack since Apple did more enforcement on it. That was the only difference in my app that I could really think was causing the issue.

Good to hear. Thank you for cross posting some of what you learned on the other gh issue


Some tests are repeated two times

While analysing the release test file for #113 , I noticed something weird:
These tests here are repeated again here
Is there any reason for that?

If I remember correctly the difference is given a release with deprecated structure within 'releases' folder.


Move interactive prompt to generate access token into client

Description:
This resolves https://github.com/jhunschejones/pcloud_api/issues/12 by moving the interactive prompt to generate an access token inside of the client module. This should allow users to require the gem into a Rails console or IRB session and run Pcloud::Client.generate_access_token to generate the prompts.
🤔 this script is currently untested in the repo, does this PR also need to include tests? I didn't change any logic but...

I tested this in a local project, configuring an access token and then using it to access my pCloud account 👍🏻

Rebasing this on the latest version of master + adding a unit test for the new code


Mashm/us 463205

Note that there are 2 lint errors in the Checks. Those will need to be resolved.

fixed the lint errors and updated the PR


Does VGG not support training with CPU?

Hi， I'm a student  study  deep learning for several month and your job help me a lot ,It's a wonderful job!
But now I meet a problem that if I run the vgg , the code will stop running without any error ,and the systerm told me that  the python didn't work ,I never met such a problem before.
Besides this , in my data the Alexnet is much better than resnet ,I do not know how to explain this situation ，I both use your default parameter ,and the acc of Alexnet is 99.5% and the  resnet is only 91%.
Many thanks!

Thank you @Ostnie :+1:
I've just checked, I can train VGGNet on my 2014 MBP without any error. Models should work on both CPU and GPU. No hardware is specified in the code, tensorflow detects automatically which hardware it should run on. What version of python and tensorflow do you use? Can you add console output?
According to my experiences, ResNet performs better than AlexNet. But I think it's perfectly normal to expect AlexNet outperforming ResNet in some cases. ResNet is much deeper network, so it requires more data to train. If your dataset is relatively small, AlexNet can learn better than ResNet.

Hi，How can I use your code to predict new data ,I did have some try but failed ,could you please give me some examples or advice ?

pred.zip
Hi, I'm glad to tell you I have success run my code for prediction , but I think the code is a little strange ,maybe it can works well but I think it is not the right way to use for prediction .I only change the code after  96 lines, I guess when you want to use a model you should define all the variables again ,so I copyed the 0-96 line in your code ,but it makes me uncomfortable and I hope you can tell me the right way to use a model for prediction.
Another problem is that why the prediction speed become more and more slowly with the program running ? At the begining of the program ,one picture takes 0.4 second ,five minutes late,it takes 0.8 second a picture! and at last it takes 1.5 second ,I don't know how to solve it ,could you please help me?

That's a really tough question. Perform a grid search for hyperparameters. You can try with a small subset of your dataset in order to make sure network is working.

How big is your dataset and what is the number of classes? You can try to augment your dataset.
Take this example code to do just prediction. I'm not sure its working, I haven't tested it. But you can easily edit.
import os, sys
import numpy as np
import tensorflow as tf
import datetime
from model import ResNetModel
sys.path.insert(0, '../utils')
from preprocessor import BatchPreprocessor

tf.app.flags.DEFINE_float('learning_rate', 0.0001, 'Learning rate for adam optimizer')
tf.app.flags.DEFINE_integer('resnet_depth', 50, 'ResNet architecture to be used: 50, 101 or 152')
tf.app.flags.DEFINE_integer('num_classes', 26, 'Number of classes')
tf.app.flags.DEFINE_integer('batch_size', 64, 'Batch size')
tf.app.flags.DEFINE_string('val_file', '../data/val.txt', 'Validation dataset file')

FLAGS = tf.app.flags.FLAGS

def main(_):
    # Placeholders
    x = tf.placeholder(tf.float32, [FLAGS.batch_size, 224, 224, 3])
    y = tf.placeholder(tf.float32, [None, FLAGS.num_classes])
    is_training = tf.placeholder('bool', [])

    # Model
    model = ResNetModel(is_training, depth=FLAGS.resnet_depth, num_classes=FLAGS.num_classes)
    prob = model.inference(x)

    # Accuracy of the model
    correct_pred = tf.equal(tf.argmax(model.prob, 1), tf.argmax(y, 1))
    accuracy = tf.reduce_mean(tf.cast(correct_pred, tf.float32))
    saver = tf.train.Saver()

    val_preprocessor = BatchPreprocessor(dataset_file_path=FLAGS.val_file, num_classes=FLAGS.num_classes, output_size=[224, 224])

    # Get the number of training/validation steps per epoch
    val_batches_per_epoch = np.floor(len(val_preprocessor.labels) / FLAGS.batch_size).astype(np.int16)

    with tf.Session() as sess:
        sess.run(tf.global_variables_initializer())
        train_writer.add_graph(sess.graph)

        # Load the pretrained weights
        # model.load_original_weights(sess, skip_layers=train_layers)

        # Directly restore (your model should be exactly the same with checkpoint)
        saver.restore(sess, "/some/path/to.ckpt")

        # Start validation
        test_acc = 0.
        test_count = 0

        for _ in range(val_batches_per_epoch):
            batch_tx, batch_ty = val_preprocessor.next_batch(FLAGS.batch_size)
            acc = sess.run(accuracy, feed_dict={x: batch_tx, y: batch_ty, is_training: False})
            test_acc += acc
            test_count += 1

        test_acc /= test_count
        print("{} Validation Accuracy = {:.4f}".format(datetime.datetime.now(), test_acc))

        # Reset the dataset pointers
        val_preprocessor.reset_pointer()

if __name__ == '__main__':
    tf.app.run()

I'm a little curious,Does nobody use trained models to solve problems? I barely saw anyone give a tutorial on how to use the trained model to work ，they just train their model and get a number called accurancy ,but there is no sense if the model can't solve the wild problem .Maybe they just want to write a paper ?

Yeah, transfer learning and finetuning are commonly used techniques in the literature. I recommend to read "How transferable are features in deep neural networks?" paper.


WorkflowManagerSpec "Should manage multithreading correctly" fails intermittently

This happens locally, and on CI.  Need to figure out if the test is flawed, or if legitimately have multithreading issues.

#97


Functions could be python scripts

Would be nice to have python scripts loaded as functions, could be another scripting language also.

Yeah, that's a great idea! I'll probably use Python, Ruby or Lua depending on what is the easiest to use with Vala.

I think Google Sheets uses Javascript right? I wonder if it's worth trying to stay in line with what people might be familiar with from another spreadsheet program


[Build] Improve operation to archive packages

This commit is to improve the operation to archive binary package files.
Changes proposed in this PR:

Updated annotation
Removed duplicated copy operations
Moved 'move' operation to each build module for consistency and maintenace
Improved the complicated folder structure to archive package files
Fixed typos
Added a common file to maintain global variable consistently

Signed-off-by: www-data<EMAIL_ADDRESS>

:memo: Version: 1.20181123. Thank you for submitting PR #430. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/TAOS-CI/ci/taos/.

After this PR:

The aging test is successfully completed to avoid unexpected issue.

The PR is ready to be merged.