max_duration

Hello guys,
I am working with K2-Tedlium3 experiment.
In the following command what is the meaning of max-duration? I assumed that it is maximal number of frames in training utterances (i.e. 30 seconds). Is that correct?
./pruned_transducer_stateless/train.py
--world-size 4
--num-epochs 30
--start-epoch 0
--exp-dir pruned_transducer_stateless/exp
--max-duration 300
In any case, I could not work with max-duration=300 because I got out of memory GPU error. So I reduced this value to max-duration=200. In this case the training work completed successfully.
For my experiment I used 4 GPU's. And it seems that I got problematic results. For example for the greedy search on the test set in RESULTS.md file I see that the expected WER on the test set is 6.69%. But, in my experiment I got following:
2022-11-18 09:31:30,923 INFO [decode.py:378] batch 0/?, cuts processed until now is 8
2022-11-18 09:31:55,554 INFO [decode.py:378] batch 100/?, cuts processed until now is 1016
2022-11-18 09:31:58,018 INFO [decode.py:395] The transcripts are stored in pruned_transducer_stateless/exp/greedy_search/recogs-test-greedy_search-epoch-29-avg-13-context-2-max-sym-per-frame-1.txt
2022-11-18 09:31:58,072 INFO [utils.py:537] [test-greedy_search] %WER 8.23% [2339 / 28430, 622 ins, 755 del, 962 sub ]
2022-11-18 09:31:58,194 INFO [decode.py:408] Wrote detailed error stats to pruned_transducer_stateless/exp/greedy_search/errs-test-greedy_search-epoch-29-avg-13-context-2-max-sym-per-frame-1.txt
2022-11-18 09:31:58,198 INFO [decode.py:425]
For test, WER of different settings are:
greedy_search	8.23	best for test
It seems that I got meaningful degradation in the system performance. Maybe you used single GPU for your training? Is it possible to check? Do you have another explanations of the degradation (do you think that my changing of max-duration from 300 to 200 can have so meaningful impact on the results?).
Your help is more than appreciated.
Thanks a lot,
AlexG.

In the following command what is the meaning of max-duration? I assumed that it is maximal number of frames in training utterances (i.e. 30 seconds)

It is the number of seconds in a batch before padding.

pruned_transducer_stateless/exp/greedy_search/errs-test-greedy_search-epoch-29-avg-13-context-2-max-sym-per-frame-1.txt

Have you tried other values of --epoch --avg?

Hello,
I am not sure I understand you question. Sorry for that. I used following command for the decoding
./pruned_transducer_stateless/decode.py
--epoch $epoch
--avg $avg
--exp-dir pruned_transducer_stateless/exp
--bpe-model ./data/lang_bpe_500/bpe.model
--max-duration 100

I am not sure I understand you question. Sorry for that. I used following command for the decoding

I am meaning that have you tried other combinations of --epoch --avg instead of --epoch 29 --avg 13?

Oh, no. What values do you recommend to evaluate?

Oh, no. What values do you recommend to evaluate?

I suggest that you search for a combination that produces the best WER.

Following are 9 different results (the best test WER is 9.23)
Epoch AVG WER_test
29 13 8.23
29 16 8.34
29 10 8.14
26 13 8.31
26 16 8.4
26 10 8.27
23 13 8.41
23 16 9.07
23 10 8.25

Following are 9 different results (the best test WER is 8.14)
Epoch AVG WER_test
29 13 8.23
29 16 8.34
29 10 8.14
26 13 8.31
26 16 8.4
26 10 8.27
23 13 8.41
23 16 9.07
23 10 8.25

How many GPU's were used to get the reference result?

@AlexandderGorodetski
Please have a look at https://github.com/k2-fsa/icefall/blob/master/egs/tedlium3/ASR/RESULTS.md#2022-03-21
It says the training command is
./pruned_transducer_stateless/train.py \
  --world-size 4 \
  --num-epochs 30 \
  --start-epoch 0 \
  --exp-dir pruned_transducer_stateless/exp \
  --max-duration 300

which means 4 GPUs are used.

In my case I used 4 GPU's too. But I have meaningful degradation in WER. It seems that my change of max-duration from 300 to 200 caused this degradation? Do you agree?

Maybe for max-duration=200 I should use smaller value of learning rate? Is it possible to change the learning rate. Maybe I should run more epochs? What do you think?

Yes, you can change
https://github.com/k2-fsa/icefall/blob/e5d942696a813ea2266579a1ba16ee99e0aba34d/egs/tedlium3/ASR/pruned_transducer_stateless/train.py#L125-L129
to adjust the learning rate.
Also, you can train it for more epochs.

@csukuangfj
I will take more expensive GPU's from AWS and I will rerun the experiment with max_duration=300. In this case my experiment will be exactly same to yours. And I believe that in this case I will get results similar to yours.
I will keep you updated.
Thanks so much.

Maybe you can train the tedlium recipe with more epochs (>29 epochs) when max-duration is 200. And then test it.

I will try to add 15 epochs.
However the graph of valid_pruned_loss is starting to rise even with only 30 epochs, so it seems that there is some overfitting. Do you think we should use a smaller learning rate?

If anything a smaller learning rate will make overfitting worse, not better, by reducing gradient noise.
30 epochs is the number we normally train for.  It is really 90 epochs if you are using speed perturbation.

Are there any guidelines for running on 1 GPU ?  Because using the exact recipe with worldsize = 1, I get only on test WER = 8.4% instead of 6.3% ? And I don't have the compute power to run many experiments ...

More recent experiments (I think, pruned_transducer_stateless5 or so and later) use a scheduler "Eden" that doesn't need much adjustment as you change the world size and the max-duration.
For old ones that use the standard Transformer scheduler "Noam", it may be helpful to reduce the base learning rate if you reduce the num jobs, e.g. by sqrt(change in num jobs), like halve it if you reduce the num jobs by a factor of 4.  That's what I would try first anyway. It may be learning too fast, so the gradient noise would be too much.  This would lead to worse loss function on the training set.


Add option "buildtest tutorial-examples" to auto-generate documentation examples for Buildtest Tutorial

buildtest tutorial-examples

Hi @shahzebsiddiqui  I still need to add tab completion for "buildtest tutorial-examples". I might need help with that.

while you at it, please update https://buildtest.readthedocs.io/en/devel/contributing/build_documentation.html#generating-documentation-examples-for-buildtest-tutorial section of the docs to reflect the new command. I did a preliminary test and it seems to be working good job!
(buildtest) spack@b3da7219a082:~/buildtest$ buildtest tutorial-examples
======> Remove Test Directory
======> Removing Report File
======> Removing History Directory
======> Removing buildspec cache
Executing Command: buildtest build -b /home/spack/buildtest/examples/spack/install_specs.yml

while you at it, please update https://buildtest.readthedocs.io/en/devel/contributing/build_documentation.html#generating-documentation-examples-for-buildtest-tutorial section of the docs to reflect the new command. I did a preliminary test and it seems to be working good job!
(buildtest) spack@b3da7219a082:~/buildtest$ buildtest tutorial-examples
======> Remove Test Directory
======> Removing Report File
======> Removing History Directory
======> Removing buildspec cache
Executing Command: buildtest build -b /home/spack/buildtest/examples/spack/install_specs.yml

@shahzebsiddiqui  I have added in the documentation. Please let me know if anything else needs to be added.

while you at it, please update https://buildtest.readthedocs.io/en/devel/contributing/build_documentation.html#generating-documentation-examples-for-buildtest-tutorial section of the docs to reflect the new command. I did a preliminary test and it seems to be working good job!
(buildtest) spack@b3da7219a082:~/buildtest$ buildtest tutorial-examples
======> Remove Test Directory
======> Removing Report File
======> Removing History Directory
======> Removing buildspec cache
Executing Command: buildtest build -b /home/spack/buildtest/examples/spack/install_specs.yml

@shahzebsiddiqui I have added in the documentation. Please let me know if anything else needs to be added.

I think the documentation needs to be rewritten to emphasize the buildtest tutorial-examples command as pose to running via python. Looking at https://buildtest--1307.org.readthedocs.build/en/1307/contributing/build_documentation.html#generating-documentation-examples-for-buildtest-tutorial  as reader i see the text in code-format python scripts/spack_container/doc-examples.py and i would be running this. But at the bottom you have text saying you can run buildtest tutorial-examples.

Could you rewrite the section and see if it makes sense to you. Pretend you are new contributor trying to understand this process.


Closing a Maximized window doesn't update the taskbar.

See if there's a message to catch in this case.

Fixed in 1.0.8.

@Elestriel what happened @Elestriel did you stop developing it anymore?

I'm in Japan :p
On Sun, Feb 12, 2017, 8:18 AM Raiden<EMAIL_ADDRESS>wrote:

@Elestriel https://github.com/Elestriel what happened @Elestriel
https://github.com/Elestriel did you stop developing it anymore?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Elestriel/TaskbarTools/issues/13#issuecomment-279183089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALE5-VKCFi2s6Ik7GK9yEwwr0UCpR40qks5rbkG5gaJpZM4LjNMC
.

Hey that's really good, hope you are having a really good time there and you should totally enjoy your time.
I don't want to ask this when you are having a good time but will you gonna work on the feature users suggested or will it take time or are you going to work on them soon.
Don't worry it's completely okay if you don't want to work on this project anymore.

I do want to work on it but I'm going to be very busy. When I get back I'm
going to a few different states for work.
People are, of course, invited to help out. If you have any programming
knowledge, feel free to contribute!
On Sun, Feb 12, 2017, 8:49 AM Raiden<EMAIL_ADDRESS>wrote:

Hey that's really good, hope you are having a really good time there and
you should totally enjoy your time.
I don't want to ask this when you are having a good time but will you
gonna work on the feature users suggested or will it take time or are you
going to work on them soon.
Don't worry it's completely okay if you don't want to work on this project
anymore.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Elestriel/TaskbarTools/issues/13#issuecomment-279184728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALE5-blDozopS5eJl4QfVLZ-IQfAkjipks5rbkkLgaJpZM4LjNMC
.

okay.


Getting Stats of multiple Streams for NOALBS

Hey there,
first of all I need to say that I'm totally new to this and don't have any coding experience but always like to learn new things.
I've set up the nginx server following the instructions and succeeded in adding more rtmp streams (via application name in the nginx config).
Though they need quite a while to show up in OBS when starting to stream to one of them it works somehow (still have to figure out why it might take up to 2 minutes until the stream is visible within the OBS source).
My problem is that I can't figure out how to set up the config.json file to get the stats of at least 2 rtmp Streams at the same time so that the automatic scene switcher would only change scenes, when BOTH streams have issues (or one is offline and the other one has low bitrate problems or a disconnect).
The main goal would be that 2 Streamers can stream to the same Twitch channel and hand off the stream from one to another without having to end the stream and start a new one (wich would already be possible if there is no automatic scene switching involved) BUT aditionally with an automatic scene switcher that changes to a LOW BITRATE or BRB Scene when not at least one of those streams is stable.
Example:
Streamer A is streaming and ends his stream for Streamer B to take over.
NOALBS would only react to Stream A (at default json config) and would switch to BRB when Stream A goes offline even though Stream B is online at that moment (because NOALBS is only set to "listen" to Stream A).
The default json Code is:
    "rtmp": {         "server": "nginx",         "stats": "http://localhost/stat",         "application": "publish",         "key": "live"     },
I tried to add more applications so it looked something like this:
    "rtmp": {         "server": "nginx",         "stats": "http://localhost/stat",         "application": "publish",         "key": "live"         "application": "StreamA",         "key": "live"         "application": "StreamB",         "key": "live"         "application": "StreamCMobile",         "key": "live"     },
I might be totally wrong here but (without any coding experience) this listing seemed to make sense to me... ;-)
Well, as you can guess, it might have seemed to make sense to me but not for NOALBS! :-D
At least under http://localhost/stat it does indeed show me all of the Streams' info and I can connect using the different application names.
How do I need to change the config.json to achieve the described goal?
Ist it even possible with multiple streams?
Or do I have to find a different solution?
Do you need more or other info?
Thanks in advance!

NOALBS doesn't currently have multi-stream monitoring, it has been brought up before and 715209 plans to add it in a future release, just no ETA on it though.

NOALBS doesn't currently have multi-stream monitoring, it has been brought up before and 715209 plans to add it in a future release, just no ETA on it though.

Hi! This has been possible since NOALBS v2.


fix number_chars/2

#196
fix number_chars/2 so that it satisfies the test cases in http://www.complang.tuwien.ac.at/ulrich/iso-prolog/number_chars Section 0 and 2.

Codecov Report

Merging #199 (14f34d0) into main (f89c74e) will increase coverage by 0.03%.
The diff coverage is 94.91%.

@@            Coverage Diff             @@
##             main     #199      +/-   ##
==========================================
+ Coverage   96.68%   96.71%   +0.03%
==========================================
  Files          20       20
  Lines        6244     6246       +2
==========================================
+ Hits         6037     6041       +4
+ Misses        170      168       -2
  Partials       37       37

Impacted Files
Coverage Δ

engine/parser.go
93.52% <57.14%> (-1.85%)
:arrow_down:

engine/builtin.go
96.26% <100.00%> (+0.43%)
:arrow_up:

engine/float.go
90.90% <100.00%> (ø)

engine/integer.go
95.83% <100.00%> (ø)

engine/lexer.go
100.00% <100.00%> (ø)

engine/exception.go
100.00% <0.00%> (+0.99%)
:arrow_up:

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 f89c74e...14f34d0. Read the comment docs.


Add past events on /

📝 Changed routes:

/

(view full size)

Commit e730d45fffb1779afdfd97c4e0c12814c1e96292 (https://hackathons-gmsu2tdbs.hackclub.dev).


Align with IntelliJ

groupedImports = Merge to prevent conflict with IntelliJ
expandRelative = true to make converging with IntelliJ's Optimize imports faster

Maybe there are other changes necessary to match IntelliJ 100%, but from my experience over many months, using
OrganizeImports {
  expandRelative = true
  groupedImports = Merge
  groups = [
    "*"
    "re:(javax?|scala)\\."
  ]
}

doesn't cause any conflicts between IntelliJ and scalafix-organize-imports.

Thanks, @sideeffffect. I left a comment about the changes in the tests, otherwise LGTM.
I'm afraid we won't be able to be 100% IntelliJ compatible because OrganizeImports values correctness more than conciseness because it is supposed to be used in a batch manner. The default IntelliJ configuration values conciseness more even though sometimes it may produce wrong results. For example:

Class count to use import with '_': 5

The above configuration is equivalent to OrganizeImports.coalesceToWildcardImportThreshold = 5, which is not recommended and may result in compilation errors in corner cases (see here).
Changes proposed in this PR are safe, though.

Oh, actually, another issue: defaulting expandRelative to true may produce unused imports that cannot be removed even if you set removeUnused to true. See here. This means that if you set both expandRelative and removeUnused to true, then the output may not be idempotent unless you run OrganizeImports at least twice. It was the biggest reason why I didn't enable it by default.
I'm not quite sure whether it's a good idea to enable it by default.

I'm thinking maybe we can add a basedOnStyle option to provide different sets of default configurations. Say:
OrganizeImports.basedOnStyle = IntelliJ_2020_3

implies:
OrganizeImports {
  coalesceToWildcardImportThreshold = 5
  groups = [
    "*",
    "re:(javax?|scala)\\."
  ]
  groupedImports = Merge
}

Users can still override individual options.

I've changed default expandRelative = false. I think this is good to go into master and then release. Then we can wait for feedback how it works in practice :wink:
While in the future it may turn out handy to have a basedOnStyle setting, I don't think we're there yet. So far I would bet that we can get away without using it, so let's not add it just yet. We'll see how far we can get with just IntelliJ-aligned defaults after this PR is merged and a new scalafix-organize-imports version released.

I've changed default expandRelative = false. I think this is good to go into master and then release. Then we can wait for feedback how it works in practice :wink:
While in the future it may turn out handy to have a basedOnStyle setting, I don't think we're there yet. So far I would bet that we can get away without using it, so let's not add it just yet. We'll see how far we can get with just IntelliJ-aligned defaults after this PR is merged and a new scalafix-organize-imports version released.

Thanks and merged!

Thanks and merged!


Segmentation Fault in combination with Distributed

Affects: PythonCall
Describe the bug
When trying to combine PythonCall with Pluto, I get Segfaults which seem to be due to a combination with Distributed. But not really sure yet.
It already took a while to extract the Segmentation fault error message - want to share t here
329] signal (11.1): Segmentation fault
in expression starting at none:1
_PyEval_EvalFrameDefault at /home/jolin_user/.julia/environments/v1.10/.CondaPkg/env/lib/libpython3.11.so.1.0 (unknown line)
_PyEval_Vector at /home/jolin_user/.julia/environments/v1.10/.CondaPkg/env/lib/libpython3.11.so.1.0 (unknown line)
PyEval_EvalCode at /home/jolin_user/.julia/environments/v1.10/.CondaPkg/env/lib/libpython3.11.so.1.0 (unknown line)
builtin_exec at /home/jolin_user/.julia/environments/v1.10/.CondaPkg/env/lib/libpython3.11.so.1.0 (unknown line)
cfunction_vectorcall_FASTCALL_KEYWORDS at /home/jolin_user/.julia/environments/v1.10/.CondaPkg/env/lib/libpython3.11.so.1.0 (unknown line)
PyObject_CallObject at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/C/pointers.jl:303 [inlined]
macro expansion at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/Py.jl:132 [inlined]
pycallargs at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/builtins.jl:220
unknown function (ip: 0xffff691d316f)
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
pycall#21 at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/builtins.jl:243
pycall at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/builtins.jl:233 [inlined]
_#11 at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/Py.jl:357 [inlined]
Py at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/Py.jl:357 [inlined]
pyexec at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/builtins.jl:1326
pyexec at /home/jolin_user/.julia/packages/PythonCall/Nr75f/src/Core/builtins.jl:1325
unknown function (ip: 0xffff69be5067)
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
do_call at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/interpreter.c:126
eval_value at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/interpreter.c:223
eval_stmt_value at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/interpreter.c:174 [inlined]
eval_body at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/interpreter.c:635
jl_interpret_toplevel_thunk at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/interpreter.c:775
jl_toplevel_eval_flex at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/toplevel.c:934
ijl_toplevel_eval_in at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/toplevel.c:985
eval at ./boot.jl:385
jfptr_eval_25809.3 at /usr/local/julia/lib/julia/sys.so (unknown line)
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
jl_f__call_latest at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/builtins.c:812
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
do_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/builtins.c:768
invokelatest#2 at ./essentials.jl:892
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
do_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/builtins.c:768
invokelatest at ./essentials.jl:889
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
do_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/builtins.c:768
110 at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/usr/share/julia/stdlib/v1.10/Distributed/src/process_messages.jl:287
run_work_thunk at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/usr/share/julia/stdlib/v1.10/Distributed/src/process_messages.jl:70
109 at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/usr/share/julia/stdlib/v1.10/Distributed/src/process_messages.jl:287
unknown function (ip: 0xffff6c0f92b7)
_jl_invoke at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
start_task at /cache/build/default-aws-aarch64-ci-0-2/julialang/julia-release-1-dot-10/src/task.c:1238
Allocations: 20773219 (Pool: 20743047; Big: 30172); GC: 24

using plain julia 1.10.5

So I can reproduce this and it happens when I send a python code object from the one julia process to the other. (supported by setting JULIA_PYTHONCALL_PICKLE=dill).
This was not failing on previous versions of PythonCall... not precisely sure which change made the difference.

Can you reproduce the issue with the latest version of PythonCall installed everywhere?

I updated everything and indeed the error is gone. Closing this issue respectively. Hope it is going to help others in case they find it by searching on github.


bump version in package.json to v2.15.4

release version v2.15.4

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Make Visual Studio CMake integration work

Currently the flow for native debugging of a Windows project is to run flutter build windows at least once, then open the sln file that's created in build/windows/ and run from there. This is not at all intuitive, and has the downside that people may not realize that changes they make there aren't permanent.
Visual Studio also has the ability to directly open a directory with a CMake project, but currently it doesn't work (largely because when I was doing the build conversion to CMake, the VS/CMake integration was completely broken on my machine, and it was never enough of a priority to come back to.) It default to the ninja generator, which our Windows CMake files don't currently support (they are written exclusively for multi-config generators), and it looks like it also defaults to clang-cl. It looks like we should be able to add a configuration file to the template that would set:

the generator (we wouldn't be able to do this dynamically, but we could just set it to VS 2022 and people could change it if they needed to)
the compiler
ideally the build output folder so it's using the same destination we are
anything else that needs adjusting from defaults

We should see if just doing that makes it possible for someone to get a good VS experience without having to ever deal with the generated solution file. (One thing we'll need to look at is how it handles the install step, since we've co-opted that to make the self-contained bundle that's needed to run.) If so, we should add that to the template.

That would be helpful, not only for VS 2022 but also for VS Code.

That would be helpful, not only for VS 2022 but also for VS Code.

CLion too...


3 more commits for the backport

the 3 remaining commits as noted in https://github.com/cloudevents/sdk-javascript/pull/336#issue-478882247

@lholmquist I didn't see this until I had already pushed those changes to origin/v3.2.x-backport. Thanks for keeping me on it though!

But oops! I shouldn't have pushed to origin. My bad.

 I am going to fix this now.

lol   :facepalm:


Updating ghostunnel builder & base images to be consistent with ART

Updating ghostunnel builder & base images to be consistent with ART
Reconciling with https://github.com/openshift/ocp-build-data/tree/f66c03011773dc3755ad874fc691be612914d65f/images/ghostunnel.yml
If you have any questions about this pull request, please reach out to @art-team in the #aos-art coreos slack channel.
Depends on https://github.com/openshift/images/pull/44 . Allow it to merge and then run /test all on this PR.

/close


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d32zux4agi6o78
Region: us-east-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

This has been resolved.
Thank you for all your help.
Melinda


Caret missing in bootstrap dropdown

I switched versions from 3.3.4 to 3.3.5 and suddenly the carets on dropdown buttons disappeared. Removing the IE hack in https://github.com/twbs/bootstrap-sass/blob/next/assets/stylesheets/bootstrap/_dropdowns.scss#L14 fixed it but I'm sure there must be a better option in

@pevers What version of IE are you using? Does the problem still occur if you use the CSS from http://www.bootstrapcdn.com/ ?

Closing due to lack of response.


GH-1379 (Add docs directory to .gitignore)

Summary

The docs directory gets created when make run is executed. It is only used for viewing the development version of the app in the local machine.
Since there is no need of pushing the development version artifacts to Github, the said directory has been added to .gitignore.
Ticket Link

Fixes #1379.

Hello @chessmadridista,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!


[no ticket] Update CI.yaml

Insert JIRA Issue number and update link
This PR aims to disable the CI check for those PR that don't need it
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

@r00ta As per retro, can you please send a brief update chat/email to the team so that everyone is aware of this change


ECMA5 example upgrades.

[ ] Scanner-Less parsing
[ ] Error Recovery example.
[ ] Output an AST.
[ ] Apply precedence of binary operators.
[ ] Grammar inheritance to implement some ESNext features.


PP-4882 Refactor GatewayAccountDao to not return GatwayAccountResourceDTO

GatewayAccountDao is a JPA dao paramaterized to type GatewayAccountEntity. This
changes the list and listAll methods to return GatewayAccountEntity and
remove the concern of converting GatewayAccountEntity to a GatewayAccountResponseDTO.
This is in the interest of maintaining a sensible separation of concerns, to
permit future refactoring of the numerous Gateway Account return type objects and move
towards a common means of fetching Gateway Accounts from the database and converting
to the required return type object (at the time of this change there are numerous ways
which causes confusion to new team members, especially when adding a new attribute
to Gateway Account.

Change list and listAll methods to return a list of GatewayAccountEntity
Add .fromEntity static method to GatewayAccountResourceDTO.
Move responsibility for converting between the entity and DTO into the
GatewayAccountService.
Update test and add new test for .fromEntity method.

WHAT
Whilst helping Gideon with a story to introduce a new attribute to the Gateway Account model it was evident that the current numerous ways of fetching a gateway account and converting to the various return types is confusing. Broadly speaking at present we either (cannot guarantee this is exhaustive):

Use the dao to query and directly convert to GatewayAccountResponseDTO - which this PR addresses.
Use the dao to fetch a GatewayAccountEntity and return it.
As above but remove the password from the credentials.
Use the dao to fetch a GatewayAccountEntity then call withoutCredtentials() to convert to a Map before returning it (seems this should be a sub-type of its own).
Upon patching a gateway account it returns a GatewayAccount type.
The gateway account can also be retrieved on a FrontendChargeResponse as a GatewayAccountEntity.

Most of the confusion was centred on not knowing why the different types are returned (GatewayAccount, GatwayAccountResourceDTO, GatewayAccountEntity, GatewayAccountEntity.withoutCredentials)  exist and which ones to add a new attribute to, along with the process of fetching, transforming and returning them not being clear.
HOW
This PR removes the concern from the GatewayAccountDao of converting between GatewayAccountEntity to a GatewayAccountResourceDTO as a stepping stone towards creating a more common and understandable process surrounding gateway accounts. Following this PR if approved I would like to look at the various return types for a gateway account and understand how it can be simplified and in the case of public Map<String, Object> withoutCredentials() look to make this a proper sub-type, e.g. GatewayAccountWithoutCredentials.
Please let me know your thoughts both on this specific PR and whether you're in agreement that process/types around GatewayAccount should be made clearer. If you feel its ok as-is and this is just a result of us lacking experience with this area of the code-base then that's ok, please let me know 👍

@alexbishop1 I think you understand our Gateway Account processes as well as anyone, would you mind taking a look if only for your opinion and words of wisdom rather than a detailed review please?

Please hold-off on a detailed review, since @gidsg  work will conflict with this and I'll have to rebase with his changes.

I've rebased with Gideon's changes and addressed Alex's comments so its ready to review whenever someone has time please?


Add Repo Maintainers to CODEOWNERS file

Update CODEOWNERS to provide maintainers ownership of all files in their repos
Description:
Update this repositories .github/CODEOWNER file to ensure that the repository maintainer
group(s) have full control of the files in this repository.

No CODEOWNERS file

No updates needed


By Bolrik: Tile overlay restructure

~ Tile Overlay Restructured
~ Tile Clone Shortcut now also copies the Config


(Major?) Bug with mcr.microsoft.com/mlops/python:latest and Azure Machine Learning extension in DevOps

TLDR; I believe this will prevent anyone from using MLOps inside of Azure DevOps.
We're running an MLOps workshops (in partnership with Microsoft) with customers, and things started failing at the Azure ML Model Deploy step.
/usr/local/envs/mlopspython_ci/bin/az ml model deploy -n mlops-aci --model oilwells_model.pkl:4 --ic /__w/1/s/oilwells/scoring/inference_config.yml --dc /__w/1/s/oilwells/scoring/deployment_config_aci.yml -g MLOps-2020-09-22-team-02-prod -w mlops-AML-WS --overwrite
The command failed with an unexpected error. Here is the traceback:

cannot import name 'PROFILE_METADATA_CPU_KEY' from 'azureml._model_management._constants' (/usr/local/envs/mlopspython_ci/lib/python3.7/site-packages/azureml/_model_management/_constants.py)
    op = import_module(mod_to_import)
  File "/usr/local/envs/mlopspython_ci/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/AzDevOps_azpcontainer/.azure/cliextensions/azure-cli-ml/azext_ml/model.py", line 17, in <module>
    from azureml._model_management._constants import ACI_WEBSERVICE_TYPE, AKS_ENDPOINT_TYPE, AKS_WEBSERVICE_TYPE, \
ImportError: cannot import name 'PROFILE_METADATA_CPU_KEY' from 'azureml._model_management._constants' (/usr/local/envs/mlopspython_ci/lib/python3.7/site-packages/azureml/_model_management/_constants.py)

We've traced the issue somewhat to mcr.microsoft.com/mlops/python:latest - there are no versions on docker hub, so we had to go with latest (which we pulled about 30 minutes before this message), and if you look at /usr/local/envs/mlopspython_ci/lib/python3.7/site-packages/azureml/_model_management/_constants.py there is no PROFILE_METADATA_CPU_KEY (though there is PROFILE_RECOMMENDED_CPU_KEY which seems suspicious). I can't find /home/AzDevOps_azpcontainer/.azure/cliextensions/azure-cli-ml/azext_ml/model.py, but I found in the DevOps logs that this was created eventually via /usr/local/envs/mlopspython_ci/bin/az extension add -n azure-cli-ml  which installs version 1.14.0 of the extension. If I check out that file, then (removing cruft) ...
from azureml._model_management._constants import ... PROFILE_METADATA_CPU_KEY ...

So it definitely seems to be an error. My guess is that it's a version mismatch somewhere, and if we have time tomorrow, that's what we'll be digging into - and, if there's no hotfix, we'll have to build our own docker container on top of mcr.microsoft.com/mlops/python:latest where we fix any issues ... which would be painful. (Hint hint to anyone else who comes along this and wants to help out!)
The second day of the workshop is tomorrow, so needless to say, this is somewhat urgent.
Other info:

Latest version of the Azure Machine Learning extension in DevOps. We uninstalled and reinstalled it with the same issues.
Linux build agent on a VM scale set.
Was working maybe 10 hours ago, so I'm guessing the docker container or azure-cli-ml were updated in that time. UPDATE: yes, azure-cli-ml was bumped to v 1.14.0 7 hours ago ... https://github.com/Azure/azure-cli-extensions/commit/39d5ca49000ef75dd47d4a0f8b2900e30d95f894

OK, probable fix is upgrading azureml-sdk to 1.14.0 in the docker container. (Untested, but I suspect pip install --upgrade --upgrade-strategy eager azureml-sdk will do all that's needed.) I don't know why this isn't done automatically as part of /usr/local/envs/mlopspython_ci/bin/az extension add -n azure-cli-ml, but that seems to be the cause.

I just encountered this issue today.

We've hit this issue a few times now. I am making a PR to use the latest version of the azureml-sdk. During version updates, we can run the docker image creation to update the version in the docker image.
Mitigation - run pip install --upgrade azureml-sdk as a step at the beginning of your pipeline.

I have updated the public docker image and updated the ci_dependencies. Please run again and reopen if you're still seeing the issue!

I removed the pip install commands from my pipeline and re-ran the pipeline a couple of times and the previous error went away.  Thanks


Making Hansen-Law transform work for non-square images

Following the discussion in https://github.com/PyAbel/PyAbel/pull/49, this PR fixes the second-to-last line in the iabel_hansenlaw_transform function in order to allow the transform to work with non-square images.
I am actually not sure if this line is even necessary. The center few pixels are noisy in either case, and I cannot really tell if there is a difference by eye. @stggh, does this line serve a specific purpose?

Also, I added a np.atleast_2d() statement to make iabel_hansenlaw_transform work if it's passed an array that is 1D (also changed the np.gradient to work for 1D).
Does it make sense for iabel_hansenlaw to also work with 1D images? It will require a little bit of extra hacking, since the up/down quadrants concept doesn't apply. Right now, iabel_hansenlaw does not produce the right result when passed a 1D array due to the quadrants, so I made it throw an error if it gets a 1D array.

(1)I am actually not sure if this line is even necessary. The center few pixels are noisy in either case, and I cannot really tell if there is a difference by eye. @stggh, does this line serve a specific purpose?
 AImg[:,ncols-1] = AImg[:,ncols-2] is to provide a similar (sensible) value for the innermost pixel, otherwise there would be a vertical zero pixel line visible through the centre of the image.
(2) Parameter comment ~line 66, may still refer to `one row' of the image
(3) Agreed, 1-d images may go straight to iabel_hansenlaw_transform
(4) I will reset my fork once the updates are merged and look at implementing vertical/horizontal_symmetry parameters

Oh, yes, you are completely correct. I somehow fooled myself about this earlier. Were you mentioning earlier that there was a better long-term solution for this?

Fixed mention of "one row".

Otherwise, are we ready to merge? @rth, should we merge this first, and then your other PR?

Better, only in relation to a possible use of a non-uniform grid Δ for evaluating the last pixel of an odd size image. However, the HL algorithm stops at n=N-2 in any case, so nothing needs to be done.

Oh, it just stops at n=N-2? Interesting. Seems like cheating! :)


[Feature]: Report Cypress code coverage to CodeCov

Description
Just as we have done for unit tests, we should also report Cypress Code coverage to CodeCov.
Suggested Solution

Consider watching https://about.codecov.io/resource/ship-safer-code-with-cypress-and-codecov/
Adapt to using flags (https://docs.codecov.com/docs/flags) to split up coverage reports

Alternatives
No response
Additional Context
No response

Not really going to be feasible, would require I move to babel over the preferred (faster) default of swc. For now, not going to compute code coverage for E2E tests or use them in CodeCov.


Control logging level and collect p2p logs from Subutai

We want to be able to swtich p2p log level by issuing the following command:
p2p set -log <LEVEL>

 can be:

TRACE
DEBUG
INFO
WARNING

Also, we need to be able to collect and see logs from Subutai UI

Also we need to log p2p logs properly, currently they logged as INFO
which can be checked by journalctl -p INFO ...

Snappy Ubuntu does not allow to configure service unit to set log level. Stdout logs getting single level.
We can use grep for filtering logs with required levels.


Refer to correct ID when joining orders

@da-kami I've trimmed down the test a bit, let me know if it still conveys the right idea from your perspective.

I verified that the test fails without the fix.


Switch type checker

Description
Changes
Tests
Known Issues
Notes
Checklist

[ ] My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
[ ] I have updated the documentation to reflect my changes.

I keep getting
    eval(self.__forward_code__, globalns, localns),
E   NameError: name 'Computation' is not defined

and not sure how to fix it

@InnocentBug found this link from Pydantic that could be the lead to fix the issue:
https://docs.pydantic.dev/latest/usage/postponed_annotations/


Invalid HTML attributes rendered using Astro + Preact

Describe the bug
Hey!
I encountered a problem when rendering an icon in a Preact component using Astro.
In the StackBlitz project you can see that the rendered SVG contains attributes in camel-case (strokeWidth instead of stroke-width), which causes a visual bug since the HTML is not valid.
Any idea on how to fix this?
Thanks for your help!
Moritz
Reproduction
https://stackblitz.com/edit/github-wpq4zq?file=astro.config.mjs
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

It seems a problem with Astro: https://stackblitz.com/edit/github-wpq4zq-wuz3k4?file=src%2Fpages%2Findex.astro


[FEATURE] Optimizing Hover Effects for Image.

Description
When a user hovers over the images, the blurring of the border can potentially detract from the overall user experience. Furthermore, the transition lacks smoothness, which may compromise the user's satisfaction with the interface
Expected Behavior
When a user hovers over an image, a smooth transition should be implemented, accompanied by a sutable drop shadow effect to make the image stand out. Additionally, it is important to ensure that the border remains unaffected and does not become blurred
Actual Behavior
When a user hovers over an image, applying a blur effect to the entire image may cause it to blend in with the surrounding images instead of standing out.

@Vijaykv5 I would like to work on this issue .  please assign it to me under hacktoberfest.

I think its not needed @Abhishek2nayak as per the Description you mentioned.
Please feel free to add a screenshot or more detailed (video)


Make get_data optional in AbstractDataStream.

I've noticed that it's often a lot more expedient to stub out get_data if all you care is epoch iterators, so we might as well make it an optional implement.

I understand you right, your point is that very often the only method of a data stream that is actually called is get_epoch_iterator. Thus, it makes sense to make all other methods optional.
If my understanding is correct, I approve this change. I guess you still have to rebase.

@rizar I made everything but get_epoch_iterator optional. Waiting on a Theano fix to Theano/Theano#5088 to un-break our tests.

THeano PR is merged.


Trigger creation in DBFlow.

DBFlow Version: 3.1.1
Issue Kind (Bug, Question, Feature): Triggers
After inserting a record on Table A, I would like to insert another record on Table B via a trigger.
Table A schema is (id INT PRIMARY KEY, name TEXT, rootId INT)
Table B schema is (id INT PRIMARY KEY, name TEXT)
Now I would like to add the following trigger:
CREATE TRIGGER trigger_name AFTER INSERT ON A BEGIN INSERT INTO B (id, name) VALUES(NEW.rootId, dummyname); END;
How can I add the id value of B with rootId value of A via the DBFlow library?
Thank you.

DBFlow team, any update on this?

@agrosner , request help on this matter. Thank you.

Trigger.create("trigger_name").after()
  .insert(A.class)
  .begin(new Insert(B.class)
    .columns(id, name).values(...))
  .enable();

This should help. Sorry for late reply.


add DigitalMinerIntegrations

Adds some integrations for the digital miner (´cause i'm too impatient to wait for mechanism 10.1)

Sweet, thank you for the contribution.


Installer stalls after Github App creation

I'm able to successfully start the installer, get the web interface, create the Github app, and get redirected back to coolify at the domain I have it running on.  But upon getting redirected, it just hangs at an empty console.
I looked at this PHP script in the installer repo: https://github.com/coollabsio/coollabs-installers/blob/main/index.php
Line 197 in that file is the last one to be rendered in the HTML source which leads me to believe there is some error I'm hitting, but I can't find the PHP logs anywhere.  There is nothing out of the ordinary output to STD_OUT:
####################################
Installer started on http://:8080
####################################

[Tue Feb  8 04:59:01 2022] PHP 8.0.7 Development Server (http://<IP_ADDRESS>:8080) started
[Tue Feb  8 04:59:40 2022] <IP_ADDRESS>:36274 Accepted
[Tue Feb  8 04:59:40 2022] PHP Warning:  Undefined array key "QUERY_STRING" in /usr/src/app/coollabs/index.php on line 3
[Tue Feb  8 04:59:40 2022] <IP_ADDRESS>:36274 [200]: GET /
[Tue Feb  8 04:59:40 2022] <IP_ADDRESS>:36274 Closing
[Tue Feb  8 05:08:44 2022] <IP_ADDRESS>:36528 Accepted
200[Tue Feb  8 05:08:49 2022] <IP_ADDRESS>:36528 [200]: GET /?code=[github_code]&state=[email]
[Tue Feb  8 05:08:49 2022] <IP_ADDRESS>:36528 Closing

And then it just hangs here forever.  Looked at some of the other installation issues created in Github and it seems its worth noting that no other docker containers have been created at this point.
The webpage I'm redirected back to from github looks like this:

Hey @funkybunch !
You tried out the first version of Coolify. I released the next one, which is completely rewritten.
Could you please try it with the new one as well?
I'm closing this issue, but please reopen it if you encounter the same error.

@andrasbacsai thanks for the update!  I saw you were working on one but wasn't sure when it was coming out.
Installation went smoothly for the most part - would be helpful to have the option to configure the web port manually as part of installation.

See #120


Fix glow renderer not working for minor version = 0

Fix for #598
This PR just changes imgui-glow-renderer::Shaders::get_shader_sources to use major * 100 + minor * 10 to calculate the #version number.
This makes 4.0 work correctly for me.

Thanks! Change makes sense - I've started the CI checks, shall merge once they are all happy


Let refmeterr be built with 4.9.0 ocaml

This unblocks rely and refmterr on 4.9.0 of ocaml since the work to fix Re was already done this makes sense.

Duplicate of https://github.com/facebookexperimental/reason-native/pull/227. I forgot to bring my work machine home this weekend and don't have the ability to merge that PR until Monday when I will merge it and cut a release.

Great, I didn't look close enough and this solved my immediate issue so I opened a PR for it. Sorry about the noise

No worries. I just released refmterr 3.3.0 which has this update


Level 4 project

hey guys, how did you implement your check out counters? i'm thinking of changing my server class to abstract so i can inherit a self check out counter class from it? but I'm wondering if there are other ways too! Thanks!!

I made a abstract staff class then both server and SCO counter inherits from it

I created a self-checkout class that extends Server and tweak some of the methods accordingly. But as what ChrisHo has mentioned, I think it would be more ideal to create an abstract staff class and have both server and SCO inherit it from there!

extend from server will be easier i think. coz u don't need to remove everything from level3 and just need to change a few method according to your project structure

I too made my self-checkout class extend from server and overrided the methods from the Server class and changed it accordingly


JSON-API + feathers-rest

Hey,
I'm currently trying to implement that if the Accept header is "application/vnd.api+json" a json-api (http://jsonapi.org/) conform response is send back. First I was thinking about cloning feathers-rest and modifying it but it won't work as feathers-rest would still catch all GET, POST, PUT, etc. calls in my understanding. So there is no way to do it that clean (or is it?).
So now I'm think about rewriting the response in the formatter to be json-api conform but it feels a little bit hacky after all.
What do you think? What is the best way in you opinion?
Greets,
Nico

Does this package solve your problem?
https://yarnpkg.com/en/package/feathers-hooks-jsonapify

@zusamann Well I saw this but wasn't sure if it always returns json-api then or only if Accept header is "application/vnd.api+json". But I'll have a look - thanks!

@nicoknoll it will always return json-api, you can add the conditionality of going through the header yourself

Yup, I think that'll do it. Thank you @zusamann !


Flickering Sprites on Samsung Galaxy S7

Hi,
I'm experiencing some strange flickering in my game on an S7.
I went through the PIXI demos to see if it occurred  on any of those and I found the Slots Demo exhibited the same behaviour.
I tried the legacy, antialias, preserveDrawingBuffer & transparent render options as per the Wiki - but with no luck. I also tried a gl.flush() after the render call.
I've been trying to isolate the cause by removing filters, blendmodes & masks but whatever i remove the flickering persists.
Anyone else having trouble with this device? I'm kind of out of ideas of what to try next.

If I remove the Blur filter from the Slots Demo by commenting out the line: rc.filters = [reel.blur]
then the flickering stops.
Hopefully that might be a clue!

Flickering problem in my android 4.2.1 phone, model BLU VIVO 4.8 HD.
I tried to open the examples from the pixijs website and they flicker too. Some times when scroll the page down until the scene disappear and back up again, they stop the flickering.

@Patrick-Hammond Any luck in investigating this? I'm wondering if browser updates will have meant Chrome has blacklisted certain commands for certain hardware to get WebGL working on them without issues

@themoonrat I'll take another look at it when I'm back at work Monday. I've since tested another PIXI based project on the S8 without problems, so it remains a bit of a mystery!

Indeed, we have no issues on that phone with the games at our company.

Just tested again and the flickering still persists on the S7 (not S8 as I mentioned previously) after a factory reset and then a Chrome update. We haven't seen this issue on any other Samsung device. Our solution was to sweep the problem under the rug and hope that noone trips over it anytime soon. If anyone can offer clues, perhaps regarding hardware or drivers on the S7, would be most appreciated!

I can confirm this is an issue, we're having the same problem. Below are the phone specs I tested on;
Device: Samsung Galaxy S7
Model: SM-G930F
Android Version: 7.0
Kernel Version: 3.18.14-12365438
Build #: NRD90M.G930FXXU1DQJ8

So I have perhaps more information on this @bigtimebuddy / @GoodBoyDigital
I too, am now having issues only on the Galaxy S7. One of the games has flickering, the other game just crashed the WebGL context. The former we solved by just not using a blur filter on the S7, but the later crash occurs when using a sprite mask.
I've attached an image of where the exception is found.

The problem occurs when the compileShader  function fails, and returns null. It appears valid for compileShader to return null if something goes wrong, but nothing handles it. So then you're trying to pass through null to attachShader, and then Armageddon occurs.

I've done some more investigation on this and I can confirm the issue is related to android 7 because it doesn't occur on android 6 or 8. Also it's not limited to the BlurFilter I've managed to reproduce this with other filters also, including using only the AlphaFilter.

Does modifying the default precision settings change this at all? For instance:
PIXI.settings.PRECISION_FRAGMENT = PIXI.PRECISION.HIGH;
PIXI.settings.PRECISION_VERTEX = PIXI.PRECISION.HIGH;

@bigtimebuddy Unfortunately no it doesn't. I tried LOW and MEDIUM also with no effect.

@st3v0 - my crash happens on our s7 that's on Android 6, happened on our s7 edge on Android 7. We updated the s7 edge to Android 8... happens there as well! Not tested the flickering since the upgrade tho.

@themoonrat unfortunately I haven't been able to reproduce the crash more then a couple of times. The only time it occurred for me was on the<EMAIL_ADDRESS>I did test the flicker issue on<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>none of them crashed however the flicker occurred on all Nougat devices.

I realize this is closed however I'd just like to update everyone interested where I've gotten with this flicker issue, since I've become stuck, maybe my findings will help spark a thought.
I setup a codepen to help replicate the problem. Firstly to easily highlight the problem I set legacy: true, preserveDrawingBuffer: true, clearBeforeRender: false. Also a BlurXFilter is being applied to each column. And the sprites are being tinted based on the column their in. Please look at the codepen if you want to know more.
Image 1 in the attached screenshot demonstrates what the pen looks like rendered on a desktop or android 8+ device. The other 2 images are showing the flicker being captured due to the above settings. These are screenshots taken from an<EMAIL_ADDRESS>
Findings
The first interesting thing I found was that setting the quality of the blur filter to 1 resulted in the flicker not occurring. This suggested to me that it was a problem with the flip/flop in the filters apply method, since a quality of 1 results in 1 pass and skips the flip/flop altogether.
I then thought it was a problem with the missing clear call in filterManager.getRenderTarget since the filter passes true for the clear arg however that arg isn't used in that method. So I added a clear check and cleared the render target before returning however this didn't solve the problem.
I then discovered that the renderTargets are cached and reused based on powers of 2. In my example each column was the same size which results in the same renderTarget being reused for all of them. Based on image 2 above this lead me to believe that the render target isn't being cleaned properly before be reused. However executing clear on the target did nothing.
To prove further that the render targets are not be cleaned properly I first adjusted the height of each column so there was 1 in column 1, 2 in column 2, 3 in 3, etc... This results in each column getting their own targets because the size of each column is unique for each the target for column 1 was 128x128, column 2 was 128x256, 3 was 128x512 and column 4 was 128x1024.
This test resulted in no flicker occurring YAY!. I then changed column 3 to have 4 images instead of 3, which meant it would reuse the render targets from column 4 and BOOM! image 3 shows the flicker again but only between the last 2 columns.
Conclusion
Unfortunately this is where I'm stuck I can't figure out why calling renderTarget.clear() doesn't actually clean it up? Why when it reuses a render target does it draw what was last drawn on that target. Could it be a problem with the gl.clear or is it something in pixi, I don't know. Is there another way to clear the render targets?

@bigtimebuddy @GoodBoyDigital I created a new example using pixijs 5.0.0-rc2 and this issue still occurs.


Not getting enough views

Hii, I uploaded over 50 videos using this TikTokUploader, but I got maximum 50 views per video.

This just a tool to upload videos 发自我的 iPhone在 2023年3月8日，下午6:30，mtuub @.***> 写道：
Hii, I uploaded over 50 videos using this TikTokUploader, but I got maximum 50 views per video.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

发自我的 iPhone在 2023年3月8日，下午6:30，mtuub @.***> 写道：
Hii, I uploaded over 50 videos using this TikTokUploader, but I got maximum 50 views per video.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

发自我的 iPhone在 2023年3月8日，下午6:30，mtuub @.***> 写道：
Hii, I uploaded over 50 videos using this TikTokUploader, but I got maximum 50 views per video.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


delve: cannot fork/exec but can exec

Please answer the following before submitting your issue:
Note: Please include any substantial examples (debug session output,
stacktraces, etc) as linked gists.

What version of Delve are you using (dlv version)?
Delve Debugger
Version: 0.11.0-alpha
Build: f454207a340cc6b16b69f5924c647938b3c1a771
What version of Go are you using? (go version)?
go version devel +b57ac33 Wed Apr 20 15:31:42 2016 +0000 darwin/amd64
What operating system and processor architecture are you using?
darwin/amd64
What did you do?
cd go/src/fmt; dlv debug
What did you expect to see?
A working debugger.
What did you see instead?

Here is the trace. Note that I can do 'dlv exec' if I build the binary, and it works great, but 'dlv debug' gives me the dreaded fork/exec error. This makes me think it might be an actual bug rather than a codesigning issue.
dunnart=% dlv debug
could not launch process: could not fork/exec
dunnart=% dlv exec fmt.test
Type 'help' for list of commands.
(dlv) r
Process restarted with PID 85560
(dlv) break main.main
Breakpoint 1 set at 0x21a3 for main.main() fmt/_test/_testmain.go:192
(dlv) continue

main.main() fmt/_test/_testmain.go:192 (hits goroutine(1):1 total:1) (PC: 0x21a3)
(dlv) stack
0  0x00000000000021a3 in main.main
at fmt/_test/_testmain.go:192
1  0x0000000000027855 in runtime.main
at /Users/r/go/src/runtime/proc.go:188
2  0x00000000000504b0 in runtime.goexit
at /Users/r/go/src/runtime/asm_amd64.s:2013
(dlv) quit
dunnart=%

Just tested this with go1.6.1 and I think the problem is just that there is no main in go/src/fmt. dlv test should be used instead of dlv debug (the error message could use improvement, though).

In case this is a code signing issue, I made a homebrew formula that solves it once and for all at https://github.com/derekparker/delve/issues/508

@aarzilli sounds right, we need to do a better job of detecting this and reporting the error.
@steeve although this isn't a codesign issue, I appreciate your work in packaging Delve for HomeBrew! Should hopefully make it easier for folks to get started with Delve on OSX. Also, for the next release we plan on including binaries along with the release, and hope to include a binary for OSX signed with an Apple Developer cert, which should remove all burden on the user for any codesigning tasks.


Fixed issues with download of NUnit 3.11.1

Changed so logic gets applied for all version of NUnit greater than 3.5.
Added new test to test the changed logic.

The following test fails for me when I run it directly from master as well:
com.ullink.functional.NUnitPluginFunctionalTest > nunit for two parallel namespaces successfully creates the merged TestResult.xml FAILED


Tax export: Feature request to include additional fields in tax-export feature

The tax-export page mentions the output is BitcoinTax-compatible data export format, but doesn't make use of all the fields currently available in the format.
The following fields are supported and could be populated:

TxHash – The transaction hash or ID.
Sender – The sending account address.
Recipient – The receiving account address.

There isn't great documentation for the BitcoinTax CSV format, not all fields are documented in the one place, but these fields work and are visible in exports that use the same format.

I implemented this code about two years ago, and I don't remember stumbling upon these fields in their export/import files. Definitely makes sense to add them.
Thanks for the heads up :+1:

Another field that could be populated is Account with a friendly name, something like Stellar GXXX or Stellar GXXX...XXXX maybe. This is obviously more subjective. Right now because Account is not provided it falls back to the TxSource value and so the account is specified as StellarNetwork.

Implemented in v0.15.19

Right now because Account is not provided it falls back to the TxSource value and so the account is specified as StellarNetwork.

BitcoinTax recommends using wallet or exchange name here:

Source (optional, source or wallet name, e.g. Coinbase or Blockchain)

So we specify "Stellar Network" here to differentiate from CEX trades. I think that after adding Sender and Recipient fields it will be easy enough for users to differentiate operations.

Thanks @orbitlens!
Regarding Source and Account fields. Where is that snippet from you quoted? I know in 2018 the CSV format had the Source field, but now the CSV fields are listed in the UI as the following. They don't list all fields, but they have Account instead of Source now:
Trading:
Date (date and time as YYYY-MM-DD HH:mm:ss Z)
Action (BUY, SELL or SWAP)
Symbol (BTC, LTC, DASH, etc)
Volume (number of coins traded)
Currency (specify currency such as USD, GBP, EUR or coins, BTC or LTC)
Account (override the exchange or wallet name, e.g. Coinbase)
Total (you can use the total Currency amount or price per coin)
Price (price per coin in Currency or blank for lookup)
Fee (any additional costs of the trade)
FeeCurrency (currency of fee if different than Currency)
For example,
Date,Action,Account,Symbol,Volume,Price,Currency,Fee
2020-01-01 13:00:00 -0800,BUY,Coinbase,BTC,1,500,USD,5.50

Income:
Date (date and time as YYYY-MM-DD HH:mm:ss Z)
Action (INCOME, GIFTIN or MINING)
Account (account or wallet name, e.g. Coinbase or Blockchain)
Symbol (BTC, ETH, LTC, etc)
Volume (number of coins received)
Total (Fair price or value in Currency or your home currency, or blank for market value
Currency (optional, specify alternative to your default currency, such as USD, GBP or EUR)
Memo (optional, name of sender or item sold)
For example,
Date,Action,Account,Symbol,Volume
2020-01-01 13:00:00 -0800,INCOME,"Blockchain Wallet",BTC,

Spend:
Date (date and time as YYYY-MM-DD HH:mm:ss Z)
Action (SPEND, DONATION or GIFT)
Account (name of account or wallet, e.g. Coinbase or Blockchain)
Symbol (BTC, LTC, ETH, etc)
Volume (number of coins spent)
Total (Fair price or cost in Currency or your home currency, or blank for market value
Currency (optional, specify alternative to your default currency, such as USD, GBP or EUR)
Memo (optional, name of recipient or item purchased)
For example,
Date,Action,Account,Symbol,Volume,Total,Currency
2020-01-01 13:00:00 -0800,SPEND,"Blockchain Wallet",BTC,1,

Two benefits of specifying more than StellarNetwork in the Account:

For opening/closing reports it shows where the purchase/income was initiated. Without that the reports say only StellarNetwork and don't break it down.
The UI shows the Account and Memo side-by-side. In the CSVs stellar.expert supports the Memo is usually the "other account", and so by setting Account to the main account of hte report, the UI displays the sender/receiver, receiver/sender side-by-side which is nice.

One more benefit:
3. In the gain reports it has columns for 'account acquired' and 'account sold', but when all Stellar accounts have the account name StellarNetwork those columns in the gain report is not useful.

You are right, I was referencing an outdated article from their knowledgebase which didn't contain the Account field description.
Updated the code with Account=current account and Source="StellarNetwork" (for compatibility).
Thanks for the heads up!


Run a container which is used by another container

Being familiar with Dobi for about a month I haven't yet used it anywhere (despite I have lots of different setups), because whenever I try it - I either can't find the appropriate example or I can't really get the concepts behind it, and so I have to fallback to docker-compose every time. At that, I feel like Dobi is much more interesting for me then docker-compose. :) Ok, so here is a new attempt to start with Dobi.
I have three services:

1st service-provider - is a service process which is assumed to run forever
2nd service-configurator - is a task process which configures the 1st one and then exits
3rd project  is a composer process which runs several containers which can use the 1st process.

So this is my dobi.yaml (which doesn't work):
mount=dir:
  bind: .
  path: /dir

image=service-provider-image:
  image: service-provider

image=service-configurator-image:
  image: service-configurator

job=run-service-provider:
  use: service-provider-image:
  mounts: [dir]

job=run-service-configurator:
  use: service-configurator-image
  mounts: [dir]
  depends: [run-service-provider]
  interactive: true

composer=project
  depends: [run-service-configurator]
  files: [docker-compose.yml, docker-compose.override.yml]

What I expect from it is when I run e.g. dobi project:detach it would:

run run-service-provider task which starts up the service and goes into background
run run-service-configurator task which configures the previous service and exists
run project composer task which startups the project

Currently it just stops on the first service - run-service-provider.
Any clue how can I use it?

Yup, this is something that is unfortunately still missing. job=run-service-provider would need to be something like service=run-service-provider using #45.
The only workaround is to use compose=run-service-provider and have a small compose file that runs the image.
You'll also need to add net-mode: projectname_default to job=run-service-configurator so that it can communicate with the service started by compose=run-service-provider. This is also something that i'd like to make more automatic, but I haven't found a good way to express it yet.

Thanks for the advice. Let me try, I'll write back.

Let me switch now to some real examples:

service-provider starts ssh-agent
service-configurator adds ssh keys via ssh-add
project is the same as above but it also a client of ssh-agent

They don't need some network for communication as they simply share the same SSH_AUTH_SOCK path.
This is my dobi.yaml file (slightly edited):
mount=home_ssh:
  bind: ~/.ssh
  path: /home/user/.ssh

mount=ssh_sock:
  name: ssh_sock
  path: /ssh

image=ssh:
  image: image
  context: images/ssh

compose=ssh-agent:
  files: [docker-compose.ssh.yml]
  project: '{env.PROJECT}'

job=add-keys:
  use: ssh
  mounts: [ssh_sock, home_ssh]
  depends: ['ssh-agent:detach']
  interactive: true
  env:
    - APP_UID={env.UID}
    - RUN_AS_USER=1
    - KEYS=id_dsa
  command: /ssh-add-keys.sh

docker-compose.ssh.yml:
version: "3.2"
services:
  ssh:
    build:
      context: images/ssh
    volumes:
      - ssh:/ssh
    environment:
      APP_UID: ${UID}
      RUN_AS_USER: 1
    command: /ssh-agent.sh
volumes:
  ssh:

Now dobi ssh-agent:detach - starts up a container.
But dobi add-keys - doesn't work: it can't connect to ssh-agent.
However, it works fine when I start it like this: docker-compose -p project -f docker-compose.add-keys.yml run add-keys where docker-compose.add-keys.yml is a file similar to docker-compose.ssh.yml - it only redefines the command and adds environment variable KEYS:
version: "3.2"
services:
  add-keys:
    build:
      context: images/ssh
    volumes:
      - ssh:/ssh
      - ~/.ssh:/home/user/.ssh
    environment:
      APP_UID: ${UID}
      RUN_AS_USER: 1
      KEYS: id_dsa id_rsa_4096
    command: /ssh-add-keys.sh
volumes:
  ssh:

It's hard for to tell what happens exactly, but it could be that volume 'ssh' is not share between compose'd container (ssh-agent) and dobi's container (add-keys).

Just to bring the matter to the end, I decided to eliminate possible volume naming problem.
So this is my dobi.yaml:
meta:
  project: project

mount=home_ssh:
  bind: ~/.ssh
  path: /home/user/.ssh

mount=ssh_sock:
  name: '{env.PROJECT}_ssh'
  path: /ssh

image=ssh:
  image: image
  context: images/ssh

compose=ssh-agent:
  files: [docker-compose.ssh.yml]
  project: '{env.PROJECT}'

job=add-keys:
  use: ssh
  mounts: [ssh_sock, home_ssh]
  depends: ['ssh-agent:detach']
  interactive: true
  # net-mode: projectname_default
  env:
    - APP_UID={env.UID}
    - RUN_AS_USER=1
    - KEYS=id_dsa id_rsa_4096
  command: /ssh-add-keys.sh

Note that I had to prefix the name of the mount=ssh_sock with {env.PROJECT} name. Now when I run: PROJECT=bb8 dobi add-keys it outputs like:
[local]$ dobi add-keys
[image:build ssh] image is fresh
[compose:detach ssh-agent] docker-compose.ssh.yml project up
Creating network "bb8_default" with the default driver
Creating volume "bb8_ssh" with default driver
Creating bb8_ssh_1 ...
Creating bb8_ssh_1 ... done
[compose:detach ssh-agent] docker-compose.ssh.yml Done
[mount: ssh_sock] :/ssh Created
[job: add-keys] /ssh-add-keys.sh Start
Enter passphrase for /home/user/.ssh/id_dsa:
Identity added: /home/user/.ssh/id_dsa (/home/user/.ssh/id_dsa)
Identity added: /home/user/.ssh/id_rsa_4096 (/home/user/.ssh/id_rsa_4096)
[job: add-keys] /ssh-add-keys.sh Done

and `bb8_ssh_1` container seems to be running still, so I think it would work when I start up my main project.

I must say that the same result I can achieve with one docker-compose.yml, joining all the stuff in the same file:

version: "3.2"
services:
ssh:
build:
context: images/ssh
volumes:
- ssh:/ssh
environment:
APP_UID: ${UID}
RUN_AS_USER: 1
command: /ssh-agent.sh
add-keys:
build:
context: images/ssh
volumes:
- ssh:/ssh
- ~/.ssh:/home/user/.ssh
stdin_open: true
tty: true
environment:
KEYS: id_dsa id_rsa_4096
APP_UID: ${UID}
RUN_AS_USER: 1
depends_on:
- ssh
command: /ssh-add-keys.sh
web:
...
db:
...
mail:
...
volumes:
ssh:
db:

running one command to initialize ssh-agent and add the keys:

docker-compose -p bb8 run add-keys
and another one to start the project:

docker-compose -p bb8 up web

I'm happy to hear that it sounds like you got it working.
It is definitely possible that docker-compose is a better fit for  some use cases. I'm hoping that with #45 and some other additions dobi will be able to cover more use cases.
I think the tools are still focused on different workflows, so there will always be some cases that work better with docker-compose and some that work better with dobi.
Since the primary feature request here is covered by #45, I'm going to close this issue.


Use $CARGO_BUILD_TARGET_DIR to determine where output should go

What does this PR do?
This PR adjusts the file-name creation routines (which make all the target/machine paths) to take CARGO_BUILD_TARGET_DIR into account, which should enable workspace builds as in #19.
It also adjusts the way filename handling is done to use PathBuf, and switches to create_dir_all (which should ensure subdirectories, if they ever get added, get created correctly).
Testing /:
I couldn't formulate a reasonable test for this - all the tools I tried (trybuild mainly) reset the cargo env variables, which makes it impossible to adjust that. But I tried the build in a workspace crate by pulling in the patched version of machine by path... and that worked.)

lgtm


LibreSSL version 3.5.0 implements the OpenSSL 1.1.0 API

Compile with LibreSSL version 3.5.0
Description of changes:
LibreSSL version 3.5.0 and up implements the OpenSSL 1.1 API.
Check all that applies:

[x ] Did a review by yourself.
[ ] Added proper tests to cover this PR. (If tests are not applicable, explain.)
[ ] Checked if this PR is a breaking (APIs have been changed) change.
[ ] Checked if this PR will not introduce cross-platform inconsistent behavior.
[ ] Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

[ x] Linux
[ ] Windows
[ ] Android
[ ] MacOS
[ ] IOS
[x] Other Platforms

I have created a pullrequest for s2n:
https://github.com/aws/s2n-tls/pull/3589
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The patch for s2n has now been merged

Do I need to do something more for this patch to be reviewed?

Hi @jorgen ,
Thank you a lot for your change. We definitely would prefer to merge this change.
We are performing more deep internal review and approval process, mostly because it touches the "security" code path. Personally, I don't see anything wrong as of now.
I hope it clarifies a bit the status of this PR, please sorry for a long review time.
Best regards,
Sergey

@SergeyRyabinin I have updated the patch. This is a simple build fix for newer versions of LibreSSL. Surly forcing developers to use older versions of LibreSSL is a bigger security risk. The equivalent patch has been merged into s2n (https://github.com/aws/s2n-tls/pull/3589), so your library is using similar code from before.


Tentative async approve/reject

This is an attempt to solve #4291 . The observation of SD behaviour is that it randomly loses connection to CHQ after an approve/reject command is issued. I suspect therefore that this is due to the git operation hanging up the entire command listener thread, as SD is still able to post 'Goodbye, cruel world' message to CHQ but not accept any command. This is an attempt to make approve/reject asynchronous, in order to verify the cause of the bug.
Note: I am not sure what magic is happening for thread raising exceptions. Hence, I am not sure whether the code is exception-safe.

No longer needed as the underlying issues have been solved.


Update quickstart and actor guide with component examples

Feature or Problem
This PR updates our quickstart with Rust and TinyGo component examples. It also formalizes the use of <Tabs> groups/query strings around the documentation so that users who select one language/os/environment will continue to see that everywhere possible.
I also included a WASI reference which includes all imports/exports that we support in the host as generated by wit-bindgen. Unfortunately some are not shown here, like wasi:cli, but that's handled by wasmtime which isn't in our wit world. We may consider adding those explicitly to our worlds in the future.
Related Issues
Requires https://github.com/wasmCloud/wasmCloud/pull/1266 to fully let the examples be easy to add keyvalue/logging
Manual Verification
I ran through these examples through "adding additional functionality". This is a draft as I still need to:

[x] Add query string parsing to the tinygo component
[x] Add the keyvalue example
[x] Update the actor developer guide to refer to components

@brooksmtownsend I can resolve the conflict and merge this if you'd like. If I don't hear from you by EOD I'll assume you're having a nice time off and will take care of it :)

@connorsmith256 that would be wonderful 💗 thank you. From my perspective this is good to go once the other linked PR merges!


[Event Request] Codeunit 6620 OnSplitPstdPurchLinesPerILETransferFields

Please add a new Event trigger OnSplitPstdPurchLinesPerILETransferFields in codeunit 6620 in procedure SplitPstdPurchLinesPerILETransferFields. This similar Event is already implemented in OnSplitPstdSalesLinesPerILETransferFields:
LOCAL SplitPstdPurchLinesPerILE(ToPurchHeader : Record "Purchase Header";FromPurchHeader : Record "Purchase Header";VAR ItemLedgEntry : Record "Item Ledger Entry";VAR FromPurchLineBuf : Record "Purchase Line";FromPurchLine : Record "Purchase Line";VAR
…
…
IF NOT FromShptOrRcpt THEN
UpdateRevPurchLineAmount(
FromPurchLineBuf,OrgQtyBase,
FromPurchHeader."Prices Including VAT",ToPurchHeader."Prices Including VAT");
IF FromPurchLineBuf.Quantity <> 0 THEN BEGIN
//++++++++++++++++++++++++++++++++++++++++++
OnSplitPstdPurchLinesPerILETransferFields(VAR FromPurchHeader,VAR FromPurchLine,VAR FromPurchLineBuf,VAR ToPurchHeader);
//------------------------------------------
FromPurchLineBuf.INSERT;
AddPurchDocLine(TempDocPurchaseLine,FromPurchLineBuf."Line No.","Document No.",FromPurchLineBuf."Line No.");
END ELSE
SkippedLine := TRUE;
END ELSE
IF "Remaining Quantity" = 0 THEN
SkippedLine := TRUE;
UNTIL (NEXT = 0) OR (FromPurchLine."Quantity (Base)" = 0);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Fix start tutorial button url for french version

fix the href attribute of the start tutorial button in the french documentation

I signed it

/LGTM


eq does not work when your field name is "apikey"

If your table has a field called "apikey" queries will always be truthy regardless of if they really are
data = supabase.table('users').select('email, api_key').eq('api_key', 'fakekey').execute().dict()
data will always return data... Found the javascript issue after 2.5 hours of questioning the nature of reality.
This bug existed and was fixed in the javascript version of supabase
https://github.com/supabase/supabase/issues/5465

Closing, please prefix your key as user_apikey instead..


Implementation Profile Page: Linking from Chat page

activity-helper.jsp has some methods that you could use. Like formatUserName, that returns an HTML link to the user profile.
It's also recommended to do so, for instance, if we want to change the UI we would like to change just a file.


[docker] Update the tilck-ci-build to v2

@michelececcacci

LGTM. Personally, i would keep the comment about how switching to debian slim actually increases image size(since it's unexpected). It's probably too verbose, but i think it's worth explaining. I actually find the comment on the debian frontend more unnecessary than the previously mentioned one.

As discussed offline, we cannot at the moment fully trust that comment we'll need to re-run a clean re-build of the image with debian-slim to check that. Therefore, in case of doubt, better remove the comment than having a misleading comment.


For what need directory other_pages?

For what need directory other_pages? @kgretzky

**Hi 👽 ! If you are interested in ⚙Advanced Phishing⚙ and the ✨Evilginx3✨ tool then I offer you Google Gmail and Custom Phishlets with a ✅Free Tutorial✅ which arrives if the 📈Telegram reaches 50 Subscribers📈
⏳ GOOGLE GMAIL PHISHLET 2024 WITH PROOF ⏳
->
⛔https://www.youtube.com/watch?v=nA47UknIE6A⛔
🔓 Ask your questions and a free tutorial from 50 subs on telegram !!!
->
📩https://t.me/Noremorse1011📩**


Add cgmanifest.json for generating correct third party notice file

PR Summary
Add the cgmanifest file for the modules that are shipped with PowerShell.
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Does this file have to be at the root directory? More and more configuration files are put at the root directory, and I wonder if any of them can be put in sub-directories.


Added Entity::IsTouching, Entity::HasBeenDamagedBy

I really like this. I could really use this in my mod.
Good job!


[support]: Visualizing Tau Trace Results with Jumpshot

Name of Software
tau
Contact Details
<EMAIL_ADDRESS>HPC System
Cori
Request Description
User is trying to use tau to profile MPI code to debug which processor is causing idle performance.  This is a user ticket at NERSC INC0175660
At first the user tried using the tau instance we installed in extra_modules which is manual install and I dont who performed this action. Then i recommended user to use tau from e4s/21.05 stack.
(tmp) siddiq90@cori04> module av tau

------------- /global/common/software/nersc/cle7/extra_modulefiles -------------
tau/2.28.2(default)

Here is the initial response from user in the ticket.

Dear NERSC Admin,
We are trying to use tau for profiling the processor idling our MPI program. We had success compiling our program with tau (thanks to your previous help!) However, the converted slog2 failed to be opened by the jumpshot program (in the tau module) with the following error message (last line):
[~/CSCR/take/owen-2021-08-17/03_tau_analysis.tracing] 10:13:03 (jobs:0)
hsinyu@cori05 $ jumpshot tau.slog2
GUI_LIBDIR is set. GUI_LIBDIR = /usr/common/software/tau/2.28.2/hsw/intel/craycnl/lib
Java is version 11.0.10.
Starting the SLOG-2 Display Program .....
GLib-GIO-Message: Using the 'memory' GSettings backend. Your settings will not be saved or shared with other applications.
Jumpshot-4 setup file : /global/homes/h/hsinyu/.jumpshot4.conf
Initialize Parameters:
Y_AXIS_ROOT_LABEL = SLOG-2
INIT_SLOG2_LEVEL_READ = 4
AUTO_WINDOWS_LOCATION = true
SCREEN_HEIGHT_RATIO = 0.5
TIME_SCROLL_UNIT_RATIO = 0.01
Y_AXIS_ROOT_VISIBLE = true
ACTIVE_REFRESH = false
ROW_RESIZE_MODE = Row
BACKGROUND_COLOR = black
STATE_HEIGHT_FACTOR = 0.9
NESTING_HEIGHT_FACTOR = 0.8
ARROW_ANTIALIASING = default
MIN_WIDTH_TO_DRAG = 4
CLICK_RADIUS_TO_LINE = 3
LEFTCLICK_INSTANT_ZOOM = true
STATE_BORDER = ColorRaised
ARROW_HEAD_LENGTH = 10
ARROW_HEAD_WIDTH = 6
EVENT_BASE_WIDTH = 8
PREVIEW_STATE_DISPLAY = CumulativeExclusionRatio
PREVIEW_STATE_BORDER = ColorXOR
PREVIEW_STATE_BORDER_W = 3
PREVIEW_STATE_BORDER_H = 0
PREVIEW_STATE_LEGEND_H = 2
PREVIEW_ARROW_LOG_BASE = 5
SEARCH_ARROW_LENGTH = 20
SEARCH_FRAME_THICKNESS = 3
SEARCHED_OBJECT_ON_TOP = false
HISTOGRAM_ZERO_ORIGIN = true
SUMMARY_STATE_BORDER = ColorRaised
SUMMARY_ARROW_LOG_BASE = 5
LEGEND_PREVIEW_ORDER = true
LEGEND_TOPOLOGY_ORDER = true

The file block pointer to the Tree Directory is NOT initialized!, can't read it.
Could you please help us with this issue?
Some additional detail of what we did:
The profiling run (job id: 47966556) is located on Cori at:
/global/cscratch1/sd/hsinyu/take/owen-2021-08-17/03_tau_analysis.tracing
The tau even tracing were enabled using our tau compiled program with a line the the submission script:
export TAU_TRACE=1

We converted the tau trace files to jumpshot format (slog2) with:
module load tau
tau_treemerge.pl
tau2slog2 tau.trc tau.edf -o tau.slog2

Please let us know if you need any further information. Thank you so much!
Best,
Hsin-Yu
Relevant log output
hsinyu@cori11 $ tau_treemerge.pl ; echo $?; tau2slog2 tau.trc tau.edf -o tau.slog2 ; echo $?
...
31604000 Records read. 99% converted
1521 enters: 0 exits: 0
Reached end of trace file.
java.lang.IllegalArgumentException: Comparison method violates its general contract!
at java.base/java.util.TimSort.mergeLo(TimSort.java:781)
at java.base/java.util.TimSort.mergeAt(TimSort.java:518)
at java.base/java.util.TimSort.mergeForceCollapse(TimSort.java:461)
at java.base/java.util.TimSort.sort(TimSort.java:254)
at java.base/java.util.Arrays.sort(Arrays.java:1441)
at base.drawable.Shadow.writeObject(Shadow.java:477)
at logformat.slog2.BufForShadows.writeObject(BufForShadows.java:291)
at logformat.slog2.output.TreeNode.writeObject(TreeNode.java:259)
at logformat.slog2.output.OutputLog.writeTreeNode(OutputLog.java:126)
at logformat.slog2.output.TreeTrunk.flushToFile(TreeTrunk.java:160)
at edu.uoregon.tau.Tau2Slog2.main(Tau2Slog2.java:202)
Exception in thread "main" java.lang.NullPointerException
at logformat.slog2.LineIDMapList.writeObject(LineIDMapList.java:33)
at logformat.slog2.output.OutputLog.writeFilePart(OutputLog.java:163)
at logformat.slog2.output.OutputLog.writeLineIDMapList(OutputLog.java:192)
at edu.uoregon.tau.Tau2Slog2.main(Tau2Slog2.java:213)
1

Reproduce Bug
1. module load e4s/21.05
2. spack load tau

After significant digging I haven't found the actual root of the problem, which seems to lie in the slog2 library rather than the TAU-side code. However the trouble is related to an issue described here:  https://stackoverflow.com/questions/11441666/java-error-comparison-method-violates-its-general-contract
Basically a more recent implementation of java's sorting algorithm is more strict and can fault on input that worked with the prior implementation. The fix I will be applying to TAU, described in that stackoverflow link, is to revert to the legacy sort implementation, which can be done without touching the slog2 library code via System.setProperty("java.util.Arrays.useLegacyMergeSort", "true"); Another workaround is to use jdk6 or earlier.
I can't apply this change to the TAU installs on cori but it would be a matter of replacing the jar file with my rebuilt one.
This is the result of the merge on my local system, with only the 0 threads included: https://yu.nic.uoregon.edu/~wspear/cori-full-tau.slog2
Note, however, that the merged trace from this example would be extremely large and unwieldy to view over X. If you isolate the first 16 ranks and and merge and convert those it may be easier to look at, if that provides sufficient coverage of the application's behavior. The first 16 ranks can be merged with the cori-installed TAU without encountering this bug. I have provided such a trace here: http://yu.nic.uoregon.edu/~wspear/cori-16-tau.slog2

@shahzebsiddiqui  This fix will be in place in the upcoming tau-2.31 release. In the meantime you can update existing installs of tau by doing:
wget http://yu.nic.uoregon.edu/~wspear/tau2slog2.jar
cp ./tau2slog2.jar </path/to/tau2>//lib

Hi @wspear,
Thank you so much for your help! Sorry that I was not able to get back to you earlier. I was asked by @shahzebsiddiqui to provide our email discussion to provide a bit more context (please find them in the PDF files below).
TAU_slog2_error.pdf
TAU_slog2_error_2.pdf
Thanks very much again for your help,
Hsin-Yu

We have updated the jarfile for the installation root for tau. just to confirm this is the checksum we have that we downloaded
siddiq90@dtn01> sha256sum /global/common/software/spackecp/e4s-21.05/software/cray-cnl7-haswell/intel-<IP_ADDRESS>4/tau-2.30.1-otn7sw7po5it6n3koe7cvorxzvrykluj/lib/tau2slog2.jar
ced189f5c54f272b557c9b7523bb950295208fc445649cf4015775bad909e8e8  /global/common/software/spackecp/e4s-21.05/software/cray-cnl7-haswell/intel-<IP_ADDRESS>4/tau-2.30.1-otn7sw7po5it6n3koe7cvorxzvrykluj/lib/tau2slog2.jar


Add MTU to connection status

Fixes #2389
I carry the data through Mesh in a map to make it generic; AWSVPC declines to supply it in this implementation.
Sample output:
$ weave status connections
<- <IP_ADDRESS>:59366    established sleeve 6a:41:53:e1:d7:29(vagrant-ubuntu-wily-64) mtu=1438
<- <IP_ADDRESS>:42128   established fastdp 5e:7e:09:ac:95:94(host1) mtu=1410
$ weave report
...
        "Connections": [
            {
                "Address": "<IP_ADDRESS>:59366",
                "Outbound": false,
                "State": "established",
                "Info": "sleeve 6a:41:53:e1:d7:29(vagrant-ubuntu-wily-64)",
                "Data": {
                    "mtu": 1438
                }
            },
            {
                "Address": "<IP_ADDRESS>:42128",
                "Outbound": false,
                "State": "established",
                "Info": "fastdp 5e:7e:09:ac:95:94(host1)",
                "Data": {
                    "mtu": 1410
                }
            }
        ],
...

Left some comments at https://github.com/weaveworks/mesh/issues/64

@bboreham would you mind adding a test case to 700_status_and_report_test.sh?

GetDisplayNameAndAttrs() is an odd function name and has an odd signature. Wouldn't it be better to either include the display name as an attribute (so we would only have GetAttrs(), or rather Attrs()), or have two functions?

@rade two functions were rejected as a source of races.  Name as an attribute is plausible; wdyt @brb ?

I fine with name as an attribute.

I changed the method to Attrs() returning just a map, and adjusted all uses to assume the name is under "name".
I also added a test case; note the test is renamed because it requires two hosts now.

I've merged the PR in mesh. You can update the vendored dep.
I think we should update docs. In particular: https://www.weave.works/docs/net/latest/troubleshooting/#weave-status-connections

I updated the test again; PTAL

Documentation changes:
modified troubleshooting


internal: Replace mocha with a custom test runner

Closes #10723
r? @Aloso

The output looks like this:

I'm not sure about those empty newlines, they seem to be produced by console.log.

For errors can you include a stacktrace?

LGTM

I'm not sure about those empty newlines, they seem to be produced by console.log.

console.log only adds 1 line break. Something else is going on here, but since this is only a minor inconvenience, it isn't necessary to investigate.

bors r=Aloso
Fun fact: this yields a -1030 line diffstat on package-lock.json.


Requested enhancement to "essentialCarefulBreak" --> "essentialDropToInventory"

Describe the feature
Rule Name: essentialDropToInventory
Description: Mining blocks or loot dropped from mobs/entities will be put straight into your inventory.
Options:

activated: false, true
requires crouch/sneak: false, true
Categories: ESSENTIAL, COMMAND, SURVIVAL, FEATURE

Expected behavior
This will mirror the functionality of "CarefulBreak" and expand it to include loot drops from mobs/entities. It will be off by default. Players should have the option to toggle each option for themselves.
Additional info
There is a Spigot plugin which does this called "Drop2Inventory Plus" (https://www.spigotmc.org/resources/⭐-drop2inventory-plus-⭐.87784/), along with some other features. I have listed the core functionality only for this request. Another developer has implemented this partly as "carefulBreakNoSneak", but without the mob loot drops included. (https://github.com/enjarai/carpet-addons)

This will be included in the next release

https://github.com/Super-Santa/EssentialAddons/releases/tag/1.1.0


drop support for Kubernetes 1.13

It's an old version which is difficult to support due to its bugs
and/or limitations. Without clear demand for it, we better should
focus on more recent versions.

@avalluri let's merge this first, then rebase PR #380 and merge that, then PR #383.


Find way to call contract methods by block hash, not block number

The eth_call in web3 and in the Ethereum JSON-RPC spec has a block number as parameter. Block numbers do not reliably identify recent blocks due to the possibility of chain reorganizations. This will lead to incorrect data being returned when the Graph node and the Ethereum node have a different interpretation of what blocks are on the "main chain".

This EIP should allow us to do this https://eips.ethereum.org/EIPS/eip-1898.

Done in #1905.


[patch] add pprof endpoint

Codecov Report

Merging #29 into master will increase coverage by 1.65%.
The diff coverage is 95.52%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage    81.7%   83.36%   +1.65%
==========================================
  Files          14       14
  Lines         492      559      +67
==========================================
+ Hits          402      466      +64
- Misses         79       81       +2
- Partials       11       12       +1

Impacted Files
Coverage Δ

router/debug_routes.go
88.09% <95.52%> (+29.27%)
:arrow_up:

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 d0d0c8f...79bd739. Read the comment docs.

need a flag to enable profiling separate?

LGTM
need a flag to enable profiling separate?

thinking we shouldn't expose profiling interface in production environment so i just put it in debug server. what do you think?

@WindzCUHK please review latest commits and comment :)

LGTM for the add comments commit.
For merge commit I think we do not need it?

LGTM for the add comments commit.
For merge commit I think we do not need it?

git problems... Please ignore that commit.
add test for coverage: https://github.com/yahoojapan/authorization-proxy/pull/29/commits/2dd061e96ec1a8bb5673062ce049a3583595d958
plx review

Maybe you use git merge instead of rebase?

I cannot trace if you changed any logic or not....

For latest test is LGTM other than the merge commit.

I cannot trace if you changed any logic or not....

git UI has bug...
select commit by URL directly
https://github.com/yahoojapan/authorization-proxy/pull/29/files/227f75c484a72d5b106f609f6e781c47a8c1ee5f..2dd061e96ec1a8bb5673062ce049a3583595d958

I think it is not UI bug... If I only select the merge commit then all changes in this PR will included into 1 commit... I will try to force push it.
https://github.com/yahoojapan/authorization-proxy/pull/29/commits/3ba9c5829cb3327c7e83a3f9e61ac408e563ad08

LGTM


when the first state is load, fire a function for once

Hi,
I'm new at ui-router
I want to have a function to be run, when the first state of the application loaded,
sort of this is the function for managing some DOM controlling using jQuery

I have founded this $viewContentLoaded,
but i want to run it only for the first load! not be repeated!

Add a listener for that then remove the listener when you run it.


Deploy script byk-deploy.sh to deploy Bürokratt Chat + Backoffice

AS AN Architect
I WANT TO to have a script to cover all deployment steps necessary to get Bürokratt Chat + Backoffice up and running
SO THAT I could be confident that there are no manual installation and/or configuration steps needed to set up the service
Acceptance Criteria

[ ] NoOps/Docker/Chat+Backoffice/byk-deploy.sh is used to deploy Bürokratt Chat + Backoffice on a server
[x] Use Chat+Backoffice Docker Compose file
[ ] Replace environment variables for every component in Docker Compose based on the values of {component}/env.conf
[x] Environment variables are defined in separate files for every component under {component}/env.conf (see example)
[x] Service-specific DSLs are deployed to appropriate components (Ruuter, Resql, etc.)
[x] As the last step, Docker Compose is launched to set up Bürokratt
[ ] Fully functional services up and running after testing

Scripts testing in a test environment were successful.