[BUG] Cython failed to build with new distutils

Describe the bug
This is filed to https://github.com/pypa/distutils/issues/13 originally.
Re-filing in cython to bridge the conversation and look for help.
It's likely to be their bug, but seems they don't have enough expertise to fix, so maybe cython side can help or work around.
With distutils 50.0.3, we hit an LTO issue when building cython.

    /usr/bin/gcc-8 -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -flto -fuse-linker-plugin -ffat-lto-objects -fPIC -I/usr/include/python3.6m -c /tmp/pip-req-build-ybinyobt/Cython/Plex/Scanners.c -o build/temp.linux-x86_64-3.6/tmp/pip-req-build-ybinyobt/Cython/Plex/Scanners.o
    x86_64-linux-gnu-gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-Bsymbolic-functions -Wl,-z,relro build/temp.linux-x86_64-3.6/tmp/pip-req-build-ybinyobt/Cython/Plex/Scanners.o -L/usr/lib -o build/lib.linux-x86_64-3.6/Cython/Plex/Scanners.cpython-36m-x86_64-linux-gnu.so
    lto1: fatal error: bytecode stream in file 'build/temp.linux-x86_64-3.6/tmp/pip-req-build-ybinyobt/Cython/Plex/Scanners.o' generated with LTO version 7.3 instead of the expected 6.2
    compilation terminated.
    lto-wrapper: fatal error: /usr/bin/x86_64-linux-gnu-gcc returned 1 exit status
    compilation terminated.
    /usr/bin/ld: error: lto-wrapper failed
    collect2: error: ld returned 1 exit status
    error: command '/usr/bin/x86_64-linux-gnu-gcc' failed with exit code 1

To Reproduce
This is a dockerfile showing the behavior change and breakage of distutils.
FROM ubuntu:18.04
SHELL ["/bin/bash", "-c"]
RUN apt-get update -y
RUN DEBIAN_FRONTEND=noninteractive apt-get upgrade -y
RUN DEBIAN_FRONTEND=noninteractive apt-get install -y gcc{,-{7,8}} python3{,-pip} git
RUN gcc --version && gcc-7 --version && gcc-8 --version
RUN python3 --version
RUN python3 -m pip --version
RUN python3 -m pip install -Uv pip setuptools==50.0.3
RUN python3 -m pip --version
RUN set -e; \
    export CC=gcc-8 CXX=g++-8; \
    SETUPTOOLS_USE_DISTUTILS=stdlib python3 -m pip install -Uv<EMAIL_ADDRESS>RUN set -e; \
    export CC=gcc-8 CXX=g++-8; \
    SETUPTOOLS_USE_DISTUTILS=local python3 -m pip install -Uv<EMAIL_ADDRESS>
Note the difference in SETUPTOOLS_USE_DISTUTILS. distutils is moving to local as default, and the second build is broken.
Environment (please complete the following information):

OS: Ubuntu 18.04 (default gcc is 7.5, gcc-8 installed as well)
Python version: stock version (3.6)
Cython version: 0.29.21 (probably not version-specific)

At the risk of stating the obvious, it looks like it's using the GCC 7 linker, but the GCC 8 compiler.
Unless you have evidence otherwise, I doubt this is specifically a Cython bug - Cython only really deals with generating the C files and leave the compilation up to setuptools or distutils. I'd suspect you see this error when compiling any C extension module.
If you just want to get Cython installed then you could try the --no-cython-compile command line argument - it'll run marginally slower but otherwise be exactly the same. You'll probably hit the same issue if you try to compile any cython-generated modules though.

That's my impression as well: x86_64-linux-gnu-gcc is wrongly used as linker instead of gcc-8.

Probably outdated. Please comment if this is still an issue.

It was a distutil bug for using the wrong linker:

https://github.com/pypa/distutils/pull/53

Thanks for reporting back.


Deployment commit for stage-stable

Merged main branch to stage-stable.
Use latest commit to update namespace ref in app-interface repo. Don't use merge commit, SHAs must be unique when images are created for each branch.

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

:exclamation: No coverage uploaded for pull request base (stage-stable@33c10e1). Click here to learn what that means.

Additional details and impacted files
@@               Coverage Diff               @@
##             stage-stable    #3811   +/-   ##
===============================================
  Coverage                ?   65.82%
===============================================
  Files                   ?      514
  Lines                   ?     9935
  Branches                ?     2330
===============================================
  Hits                    ?     6540
  Misses                  ?     3153
  Partials                ?      242

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


GIF generation doesn't work  -i

I was used to create GIFs on 3b1b MANIM by using code like this:
python -m manim script.py GIFGenerate -i

Although I am using the same code/commands, I can't generate GIFs on ManimCE.

Thanks for the report :D

@FilipeMarch can you please follow the steps outlined in the issue template?
Include at least:
1. Steps to reproduce the issue (e.g. the command you ran)
2. The unexpected behavior that occurred (e.g. error messages or screenshots)
3. The environment (e.g. operating system and version of manim)

For step one, can you please post the scene code you are using? Or find a minimal scene that reproduces the bug?
What version of manim CE are you using? Is there any error message? Are there any files generated? etc etc etc etc

TL;DR I started answering what you asked step by step and writing the results and suddenly  and on the end no bug occurred. Now, my old manim-master folder is not generating GIFs correctly, but I don't care anymore at this point, as I'm looking forward to only use ManimCE.

@FilipeMarch can you please follow the steps outlined in the issue template?

Sure

Steps to reproduce the issue (e.g. the command you ran)

On Linux Mint 19.3 Cinnamon, create virtualenvironment with python 3.8.1 for ManimCE:

pyenv virtualenv 3.8.1 ManimCE

Activate the environment and install ManimCE

pyenv activate ManimCE
python -m pip install manimce

Create a script.py file in any folder and paste the following code:
class GIFGenerate(Scene):
    def construct(self):
        text = TextMobject('ABC')
        self.play(Write(text))
        self.wait()

Open that folder on terminal, with ManimCE environment activated, and run this:
manim script.py GIFGenerate -i

File ready at
                    /home/user/desktop/
                    manimCE/media/videos/script/1440p6
                    0/GIFGenerate.mp4

Open the folder  /home/user/desktop/manimCE/media/videos/script/1440p60 and you will see a .mp4 file there.
Also, there is a GIF there. It is the GIF I needed. LOL
I could swear I followed the exact same steps yesterday but didn't find the .GIF image.
I can see on the folder /home/user/manim-master/media/videos/script/1440p60 the GIF I generated yesterday to post this issue: GIFGenerate.gif. I generated it with manim-master.
But on the folder:
/home/user/desktop/manimCE/media/videos/script/1440p60
I only had a GIFGenerate.mp4 with all frames black. So I posted the issue.
Now manim-master is not generating GIFs correctly and manimCE is doing it correctly, this is the most strange thing that happened to me on the last 48h. Anyway, I think you can close the issue if you can generates .gif files correctly as I am now with ManimCE. I have no idea what really happened here.

Thanks for the detailed answer. It sounds like you might have mixed up 3b1b/manim with manim-ce. That happens! Please feel free to open a new issue if you find the bug again.


qe/geo-replication.md: Clarify replicated pull paragraph

Solo merge as it is a follow-up edit to previous PR


Is it possible to add a feature for batch processing audio files?

Due to work reasons, I often need to process multiple audio files.

Due to work reasons, I often need to process multiple audio files.

You can use command line tools like demucs and iterate over them with
for f in *; do python3 -m demucs "$f"; done

The GUI allows you to drop as many audio files to process as you'd like

GUI 允许您放置任意数量的音频文件进行处理

MDX-Net: 292, 496, 406, 427 models not available in download options


just fyi: chestnut — encrypted storage built on nutsdb

https://github.com/jrapoport/chestnut

厉害了

谢谢


HEAD requests do not work on gateways

https://gist.github.com/Kubuxu/7e06b18c1d948197a90fdf3f2da4585f
Get requests work normally.

Okay I think this is simply multireq, which shouldn't try to buffer responses to HEAD requests.

Seems like this issue is back.
Try this:
curl -sv --max-time 30 -X HEAD http://gateway.ipfs.io/ipfs/QmXoypizjW3WknFiJnKLwHCnL72vedxjQkDDP1mXWo6uco/wiki/CacheFS.html

Output
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to gateway.ipfs.io (<IP_ADDRESS>) port 80 (#0)
> HEAD /ipfs/QmXoypizjW3WknFiJnKLwHCnL72vedxjQkDDP1mXWo6uco/wiki/CacheFS.html HTTP/1.1
> Host: gateway.ipfs.io
> User-Agent: curl/7.61.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Mon, 05 Nov 2018 11:33:42 GMT
< Content-Type: text/html; charset=utf-8
< Content-Length: 10697
< Connection: keep-alive
< Vary: Accept-Encoding
< Accept-Ranges: bytes
< Cache-Control: public, max-age=29030400, immutable
< Etag: "zb2rhgCYMgtmZntxNFCwMZDaud3uj33Zq3KF6WPR2TizKeUXR"
< Last-Modified: Thu, 01 Jan 1970 00:00:01 GMT
< Suborigin: ipfs000bciqizol2actmscqu5tyyfwbwgwb5iaxwtem6hdyetdw5twjklqbkpna
< X-Ipfs-Path: /ipfs/QmXoypizjW3WknFiJnKLwHCnL72vedxjQkDDP1mXWo6uco/wiki/CacheFS.html
< Access-Control-Allow-Origin: *
< Access-Control-Allow-Methods: GET, POST, OPTIONS
< Access-Control-Allow-Headers: X-Requested-With, Range, Content-Range, X-Chunked-Output, X-Stream-Output
< Access-Control-Expose-Headers: Content-Range, X-Chunked-Output, X-Stream-Output
<
* Operation timed out after 30001 milliseconds with 0 out of 10697 bytes received
* Closing connection 0


Update middleware.rst

Update link to middleware sample code.

Hi @romansp, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Thanks for the PR! Merged in a19bf41f3aac6ef4c3c520a54697744e7e63f469


Question about Regression Model csv

I am trying to build a regression model to indentify if the audio data that I have is of an angry/irritated/upset person or a happy/neutral person.
What I am trying to figure out from your explanation on https://github.com/tyiannak/pyAudioAnalysis/wiki/4.-Classification-and-Regression is:
How did you come up with the  (b) the value of the variable to be trained.
I did see a similar question on https://github.com/tyiannak/pyAudioAnalysis/issues/4
But, I dint understand what x would be in  (x - (Max-Min)/2) / (Max - Min)
As in what feature/parameter is that? And how would I go on about identifying it in my case?

Hi @viveksj did you figure this out?

No I couldnt.
My best guess would be that these were subjective values evaluated by human listerers.
That is what I did in the end, i.e. Ask human reviewers to give feedback and average it.

Ah alright! Thank you!!

What @viveksj suggested is right, u need human annotation, exactly as in classification tasks. The only difference is that, since regression can be a little vague, more than one human annotators may be needed, and maybe a computation of inter-annotator agreement is also required to exclude non-confident samples (i.e. samples for which the annotators disagreed a lot)


Update LG template content range

Originally, the Body of LGTemplate presents the first line except for comments, to the last line except for comments.
for example:
> comment1
# t1
> comment2
- hi

> comment3
# t2
- hi

originally Body of template  t1 is
- hi

After this pr, the body of the template presents from the first of template content to the last line before the next template (or end of file).
So, the body of tempale t1 is
> comment2
- hi

> comment3

Additionally, allow empty comments.

Pull Request Test Coverage Report for Build 110813

29 of 29 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 83.303%

Totals

Change from base Build 110694:
0.05%

Covered Lines:
9273

Relevant Lines:
10714

💛  - Coveralls


Constructor args

Use explicit arguments in the constructor, for better validation and affordance.

Pull Request Test Coverage Report for Build 196

11 of 11 (100.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.7%) to 99.617%

Files with Coverage Reduction
New Missed Lines
%

nexmo/init.py
2
55.17%

Totals

Change from base Build 193:
0.7%

Covered Lines:
260

Relevant Lines:
261

💛  - Coveralls


Fixes #2953 - Url resolved with special characters

This PR Fixes #2953. I have added the + character to the username set. Rest all characters letters , numbers, - and _ were already doing fine with the URL.

Any updates on this PR ?

This could use a test confirming that it does indeed work. Similar to this: https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/rtd_tests/tests/test_views.py#L115-L124

Thanks for the review @ericholscher! I will add a test for this!

@ericholscher can you have a look here ?

I linked the tests file above: https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/rtd_tests/tests/test_views.py#L115-L124
You could also test the regex itself, outside of a view, and just confirm that it's matching what we expect.

@ericholscher I have added the tests!

Merging, Thanks!


Make ROSS objects pickable

Currently we are not able to pickle ROSS objects:
import ross as rs
import pickle

rotor = rs.rotor_example()
pickle.loads(pickle.dumps(rotor))

---------------------------------------------------------------------------
PicklingError                             Traceback (most recent call last)
<ipython-input-7-09fc4ffa5e19> in <module>
      3
      4 rotor = rs.rotor_example()
----> 5 pickle.loads(pickle.dumps(rotor))

PicklingError: Can't pickle <class 'ross.rotor_assembly.GlobalIndex'>: attribute lookup GlobalIndex on ross.rotor_assembly failed

pickle is used by the multiprocessing module, so it would be good to have the possibilite to use it with ROSS if we want to be able to have some code running in parallel.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.

Closed with #844.


Flaky test fix. Query only 1 broker to test quota split

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

Project coverage is 61.81%. Comparing base (59551e4) to head (bb49686).
Report is 845 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #13771      +/-   ##
============================================
+ Coverage     61.75%   61.81%   +0.06%
- Complexity      207     1315    +1108
============================================
  Files          2436     2557     +121
  Lines        133233   141058    +7825
  Branches      20636    21936    +1300
============================================
+ Hits          82274    87194    +4920
- Misses        44911    47208    +2297
- Partials       6048     6656     +608

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
27.78% <ø> (-33.93%)
:arrow_down:

java-21
61.81% <ø> (+0.18%)
:arrow_up:

skip-bytebuffers-false
61.79% <ø> (+0.04%)
:arrow_up:

skip-bytebuffers-true
61.78% <ø> (+34.05%)
:arrow_up:

temurin
61.81% <ø> (+0.06%)
:arrow_up:

unittests
61.81% <ø> (+0.06%)
:arrow_up:

unittests1
46.21% <ø> (-0.68%)
:arrow_down:

unittests2
27.79% <ø> (+0.06%)
:arrow_up:

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

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

Any progress on this fix?

Hey @Jackie-Jiang  I could not find any direct way of getting the signal that query quota rate limiter change is propagated so right now I am checking for the delay between 2 successful requests and ensure it lies within the expected changed rate (with 20% margin).

@klsince Can you help verify the test logic?


progress bar overlaps on timer and full screen icon

I tried implementing chewie like the examples out there but i'm getting  the output differently. in my output, the fullscreen button is not touchable and timer is hidden by the progressbar

@Sarathlal13 is there a reproducible code sample that you can provide so that we may further diagnose this issue? Thanks in advance.


Cannot update the masterVariant when the variantId is not 1

Description
When the masterVariant has a variantId apart from 1, the importer is throwing an error.
[ '[row 2] We need a product before starting with a variant!' ]
Expected Behavior
Should update the masterVariant with the appropriate variantId
Current Behavior

When the masterVariant has variantId is not 1, the importer is throwing an error.
When the same masterVariant is updated with variantId 1, the update is succeed. But actually the variantId is not 1. The module seems to assume that the masterVariant will always have an ID of 1, which is not always the case (specifically after running the changeMasterVariant update action)

Context

Makes it impossible to update the master variant of some products in a project

This issue has been reported and is still valid

We've discussed with @wizzy25 and @VasumathiJayakumar three options. As we'd like to go forward with option 1, we'd appreciate your input whether it's manageable for the projects you are working on @mitsch2000 @MelNiermann @ValeSauer @butenkor . Thanks!
Option 1
Differentiation between products is done by evaluating the product key (key is optional);
CON: Not all current projects use the product key, therefore these projects depending on the Product Import would have to update the key after this fix.
Option 2
Add a new field (boolean) to the import which states whether the respective variant is the master variant or not.
CON: breaking change
Option 3
Differentiation between products is done by evaluating the product name; whenever a new name is detected, the importer concludes that a new product is given and the first variant is the master variant.
CON: name is not unique, therefore it could be error prone

I am not sure i exactly understand the issue. Was the issue already addressed here or it was different one: https://github.com/sphereio/sphere-node-product-csv-sync/pull/192?
If i understand it right in Option 1 every variant row would need to provide mandatory product key value which will allow to find all variants of same product? How masterVariant will be determined in this case - by variantId = 1? How masterVariant is identified if variants are matched by SKU and not variantId (which would not be supplied)?
In general we do not use sphere-node-product-csv-sync for import. I know one case for data feed export.
Are there any plans for usage of sphere-product-import as dependency here? sphere-node-product-csv-sync would take care of mapping from CSV/XLS to JSON only and sphere-product-import would do the matching and execute update/create?
In sphere-product-import we use SKU for matching only. SKU is unfortunately optional in API but all customers have unique variant ID/SKU as it is required for inventory (and eventually later for price endpoint) linking. Usage of product/variant keys for matching might introduce more issue than expected as both key and SKU are unique and most probably will be the same (data duplication). If they will not be the same then there might be conflicts. Example:
Existing product -> product_key1, variant_sku1
New product -> product_key2, variant_sku1
This would lead to an attempt of failed product creation with product_key2, variant_sku1. Usage of keys for product/variants might make sense when we have dedicated variants endpoint.
We try to address/solve most known conflicts within commercetools-node-variant-reassignment module, which soon will be integrated into sphere-product-import.

I am not sure i exactly understand the issue. Was the issue already addressed here or it was different one: #192?

It's the same issue as was tried to fix in #192, but that didn't resolve it. This function checks that a variantId is 1, then assigns it as the masterVariant (and the beginning of a new product). However, with the introduction of the changeMasterVariant update action, the masterVariantId is not guaranteed to always be 1, and so the module does not know what the masterVariant is at parse time. The idea of the key was to implement something on the product level that will be unique which can serve to differentiate products, with the first row (regardless of the variantId) serving as the masterVariant

If i understand it right in Option 1 every variant row would need to provide mandatory product key value which will allow to find all variants of same product? How masterVariant will be determined in this case - by variantId = 1? How masterVariant is identified if variants are matched by SKU and not variantId (which would not be supplied)?

No. The supplied key is the productKey and not the variantKey, so it's supplied on the same level as the name or slug. MasterVariant will be determined by being the first row in the products

Are there any plans for usage of sphere-product-import as dependency here? sphere-node-product-csv-sync would take care of mapping from CSV/XLS to JSON only and sphere-product-import would do the matching and execute update/create?

Currently there are no plans. This approach could be given more thought, especially with the variant-reassignment coming soon.

In sphere-product-import we use SKU for matching only. SKU is unfortunately optional in API but all customers have unique variant ID/SKU as it is required for inventory (and eventually later for price endpoint) linking. Usage of product/variant keys for matching might introduce more issue than expected as both key and SKU are unique and most probably will be the same (data duplication). If they will not be the same then there might be conflicts. Example:
Existing product -> product_key1, variant_sku1
New product -> product_key2, variant_sku1
This would lead to an attempt of failed product creation with product_key2, variant_sku1. Usage of keys for product/variants might make sense when we have dedicated variants endpoint.

This should not be a problem (I think). The SKU is on the variant level, and the variantKey is still not required (could be left out or duplicated, doesn't matter). Again, it's the productKey that will be required

We try to address/solve most known conflicts within commercetools-node-variant-reassignment module, which soon will be integrated into sphere-product-import.

The actual problem is from the way this module parses the CSV to JSON, and not the importer itself

I would prefer option 1. From my point of view utilization of keys becomes more and more mandatory if you use syncs, imports and so on anyway - so it fits into our overall strategy.

@mitsch2000 and @MelNiermann, could you please also give us your feedback on the options in order for us to take next steps? Especially in regards to option 1, which would use the product key and would need to be updated if not yet used in your projects.
Thanks!
FYI @wizzy25

Hey @Oehmi
does this mean, that the product key will become mandatory for the CSV import?
Because if this is the case, we can't change this so easily, since the marketplace has a few partners which are using the CSV Connector and import all data via CSV files.
We have to inform all partners first, so they have to change the implementation on there side, this means either we need a bit of time to inform partners and five them the chance to chance the imports  exports or the import can handle both (imports with keys and without).
Best
Melanie

@MelNiermann would it be possible to get a timeframe for informing the partners?

FYI @MelNiermann : If you are using CLI, then you won't be affected unless you update to the newest version.
However, on Impex the changes will break. Most likely, the next Impex update with these changes will be in approx. 3 weeks.

@stmeissner @VasumathiJayakumar @MelNiermann
We will put this issue on hold for now and not deploy any changes. If you are aware of any other customer that this causes problems for, please do let us know.
@wizzy25 Regarding the partners, they had not been informed so far.

The "fix" for this issue still exists on this branch if it ever becomes necessary again to be implemented


[Feature request] Make CSS responsive for mobile view

Great idea and implementation, design looks very good on desktop and PDF.
Just on mobile, it could really be better. It's not really usable there, only with much pinching and zooming in out, even then, not the whole line is readable.
Since 2012, when this seems to have been started, mobile has become much more important. Maybe some of the forks contain tweaks to make it responsive, if not, would be great if someone could touch it up.
This is how it looks on my OnePlus 6T with Opera:

valid point. Would you care to contribute a new .media view or a responsive css ?

Ok will try to make it mobile responsive but it will take time 😅


add Faculty of Sciences of Gabes

Website URL: http://www.fsg.rnu.tn/

@dhouibi Pull request merged. Thank you!


Alternative approach to sending to multiple devices

@MihaelIsaev it appears that the approach you took for sending messages to multiple devices is to create a topic for each group of 20 recipients, then for each topic send the message and delete the topic afterwards. Although it works, it seems a bit fragile and suboptimal. Firebase accepts sending messages to up to 500 recipients at a time using a "mixed/multipart" request to https://fcm.googleapis.com/batch:
https://firebase.google.com/docs/cloud-messaging/send-message#send-messages-to-multiple-devices
Have you experimented with that?

is to create a topic for each group of 20 recipients

it looks like there are no limitation for topic devices amount in docs, but I choose 20 just in case. It was better than send request per each token.

Firebase accepts sending messages to up to 500 recipients at a time using a "mixed/multipart" request to https://fcm.googleapis.com/batch:
https://firebase.google.com/docs/cloud-messaging/send-message#send-messages-to-multiple-devices

Wow, thank you @siemensikkema for the information!
I could bet that I read all the docs 100 times by myself and also some people tried to help me with that and no one seen this endpoint before. But webarchive says that it was there since April 1st 2019! That's unbelievable! 🤪
Sure we should use that awesome endpoint for sending multiple messages then.
I could implement it this or maybe next week. Or let me know if you would like to send pull request 🙂

@MihaelIsaev great! Glad my googling can be of help to you 😊 I don't think I'll have time to implement it myself soon but I can help out reviewing. So feel free to tag me when you get around to implementing it!

@MihaelIsaev did you manage to take a look at this? Is it a large undertaking you think? It seems to require a slightly different approach to contacting Firebase.


OneRowRelation will cause fallback

Describe the bug
Some queries like select length("gluten"); or select md5("gluten") will fallback because of the child whose output is empty. This code logic is in FallbackEmptySchemaRelation. If we make this situation bypass the detection of FallbackEmptySchemaRelation through some matching, an exception will be thrown in GlutenColumnarBatches and ArrowColumnarBatches because the column is 0. gluten need to support this kind of queries.
https://github.com/oap-project/gluten/blob/d9c38e6897f0b94687934cbf5ac68c12cda2bc96/gluten-data/src/main/java/io/glutenproject/columnarbatch/GlutenColumnarBatches.java#L56
https://github.com/oap-project/gluten/blob/d9c38e6897f0b94687934cbf5ac68c12cda2bc96/gluten-data/src/main/java/io/glutenproject/columnarbatch/ArrowColumnarBatches.java#L56
To Reproduce
select length("gluten");
Expected behavior
not fallback

@rui-mo Is this by design?

@rui-mo Is this by design?

@liujiayi771 I think this rule was added to enhance the robustness when the child's output is empty. Because in that case, we cannot know the if the child's output is from Velox, Arrow or vanilla Spark.
@zhztheplayer Do you have anything to add?

@liujiayi771 I think this rule was added to enhance the robustness when the child's output is empty. Because in that case, we cannot know if the child's output is from Velox, Arrow or vanilla Spark.

Yes this explained the reason accurately.
An intermediate columnar batch (aka native columnar batch) in Java should at least have one column: Because the first column should maintain the metadata of the whole batch. Because Spark (until 3.2) doesn't allow us to store metadata in the ColumnarBatch instance.
See also an unresolved issue https://github.com/oap-project/gluten/issues/540

@rui-mo Is this by design?

@liujiayi771 I think this rule was added to enhance the robustness when the child's output is empty. Because in that case, we cannot know if the child's output is from Velox, Arrow or vanilla Spark.
@zhztheplayer Do you have anything to add?

Ok, although the two examples I mentioned above can be calculated natively, since the returned data has no column information, we should fallback to enhance the robustness.


New version: Ubisoft.Connect version 148.1.10975

Pull request has been created with Komac v1.11.0 :rocket:
Resolves #128282
Microsoft Reviewers: Open in CodeFlow

Duplicate of https://github.com/microsoft/winget-pkgs/pull/128314

@stephengillie
That commit shows the version as <IP_ADDRESS>75.
Actual installer is 148.1.10975 (screenshot taken on 2023-Nov-30)


build blockly  fail

Build the first setp fail（build blockly fail）,the log :
SUCCESS: msg\js\tr.js
SUCCESS: msg\js\uk.js
SUCCESS: msg\js\vi.js
SUCCESS: msg\js\zh-hans.js
SUCCESS: msg\js\zh-hant.js
Processed Ardublockly translation: msg\json\en_ardublockly.json
Processed Ardublockly translation: msg\json\es_ardublockly.json
Processed Ardublockly translation: msg\json\fr_ardublockly.json
Processed Ardublockly translation: msg\json\it_ardublockly.json
Processed Ardublockly translation: msg\json\nl_ardublockly.json
Processed Ardublockly translation: msg\json\ru_ardublockly.json
SUCCESS: blockly_uncompressed.js
Exception in thread Thread-2:
Traceback (most recent call last):
File "D:\python\lib\threading.py", line 530, in __bootstrap_inner
self.run()
File "build.py", line 177, in run
self.gen_core()
File "build.py", line 210, in gen_core
self.do_compile(params, target_filename, filenames, [])
File "build.py", line 274, in do_compile
conn.request("POST", "/compile", urllib.urlencode(params), headers)
File "D:\python\lib\httplib.py", line 946, in request
self._send_request(method, url, body, headers)
File "D:\python\lib\httplib.py", line 987, in _send_request
self.endheaders(body)
File "D:\python\lib\httplib.py", line 940, in endheaders
self._send_output(message_body)
File "D:\python\lib\httplib.py", line 803, in _send_output
self.send(msg)
File "D:\python\lib\httplib.py", line 755, in send
self.connect()
File "D:\python\lib\httplib.py", line 1152, in connect
self.timeout, self.source_address)
File "D:\python\lib\socket.py", line 567, in create_connection
raise error, msg
error: [Errno 10060]

the same problem with you!!

Have you guys pulled the latest commit from master? In particular, make sure you have https://github.com/carlosperate/ardublockly/commit/df16522025dafa03d750d88fd24c67979ec3af93


UI missing changes

Description
This PR fixes one missing label and one edit icon
Edit Template

Edit Backup offering icon

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan ui

@harikrishna-patnala a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/7231 (QA-JID-47)

@harikrishna-patnala @nvazquez can we put any missing labels in a single PR, please? (context: #7230 )

Yes @DaanHoogland tried to look for others, but for now I found only these.

shall I push this commit into #7230 PR ?

shall I push this commit into #7230 PR ?

well @harikrishna-patnala , this has a better title, but I am inpartial to which we use in the end.


Move hydra-explorer out of the mono-repo

https://github.com/cardano-scaling/hydra-explorer/pull/5

Is this the same thing as https://github.com/cardano-scaling/hydra-explorer/issues/1?


Add "@types/chai" devDependency to package.json

Chai doesn't have types associated with it. This diff just adds a devDependency on @types/chai
#1511

thanks!


Spread operator support

https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Operators/Spread_operator

Fixed with #56


Astronomer-on-AWS proof-of-concept

As part of cagov/data-orchestration#4, I'd like to stand up AWS infastructure for Astronomer.
I'm copying some ideas from https://github.com/cagov/data-infrastructure/issues/138 on things to look out for when evaluating how easy-to-use this approach is. Along each of these dimensions, I would score them from "makes me happy" to "makes me sad".
Custom software environments
The GIS stack often requires custom software environments. That is to say, whatever default image the tool uses will not do the job, so we'll need to provide our own image. We'll want to evaluate how easy it is to build our own image and provide it to the orchestration tool.
Compute resources
Do they provide a compute cluster or other batch-like service for running custom jobs? Or will we need to bring our own K8s/ECS/Batch cluster? If the latter, how easy is it to set up?
Integration with AWS/GCP services
If we do have services running in our own cloud account, how easy is it to interact with them? Are there nice user interfaces for securely providing service account credentials?
API access and CI integration
How painful is it to deploy new versions of a pipeline? Are there CI tools or custom GitHub actions for doing this? Ideally it is simple to deploy-on-merge.
dbt Integration
This is one of the most important: all of the major orchestrators have been implementing some level of integration with dbt (which provides its own DAG abstraction). How pleasant are these integrations to use?

Is it easy to trigger a dbt run from the API? In the test, we'll want to kick off dbt after the initial load.
Is it easy to have a task triggered by the completion of a dbt run? In the test, we'll want to send off an email/ad-hoc-report after the dbt run is complete.
Is there visibility into a dbt run while it is going on? Can we see:

The structure of a dbt DAG
The current status of the dbt DAG as it runs
Any error states or warnings from dbt
Logs from the dbt cl

Closing this as not planned. The POC story will be taken up later if needed. Please refer to this ticket for additional information - https://github.com/cagov/data-infrastructure/issues/378

Closing this as not planned. The POC story will be taken up later if needed. Please refer to this ticket for additional information - https://github.com/cagov/data-infrastructure/issues/378


Newsletters: add 73 (2019-11-20)

[ ] Final links for Bitcoin Core <IP_ADDRESS>
[ ] Services & clients section @bitschmidty

Read it over and things LGTM. Thanks @harding.

ACK 0534346. I agree with @jonatack's comment https://github.com/bitcoinops/bitcoinops.github.io/pull/271#discussion_r347229433 that we should clarify that there is no 0.19 release and the bitcoind release is <IP_ADDRESS>.

Added a commit for the 'Changes to services and client software' section

ACK 647769e23d693d12b6ea2d105cb38e7cea5ab0c8 (modulo my previous comment: having 0.19 in the newsletter summary, <IP_ADDRESS> in the action items and 0.19.0 in the news section is a little confusing for readers without context)

ACK 4033dc4e1c2b9a5084994722d81bc369d8e26db7 once the links are finalized. Thanks as always, @harding !

Pushed a commit dropping the Bitcoin Core release announcement as it looks like they're still waiting for osx codesigned gitian stuff, so the release probably won't be today.  This can be squashed, as I have a backup of the content for next week.

tACK 9ef62ab
Squashed


overriding my date configuration

I just noticed that when I have am using your package to benchmark, my timezone is overwritten
By default I have it set to Singapore
in my config/app.php
I have
'timezone' => 'Asia/Singapore',

I was running benchmarking on aggregate queries that use the dates and I was noticing that my graphs were off and different from my production data even though I am using the same database data
I think it's because of this line:
    private function setDefaultTimeZone()
    {
        // Check date function
        if( ! ini_get('date.timezone'))
            date_default_timezone_set('UTC');
    }

In your ConfigHandler.php
Is this really necessary ? Shouldn't your benchmarking work regardless of what timezone is set?

Hi Vesper8,
I checked whether it is necessary to configure the default timezone. I also think that the performance tool should not set default timezone.
The code for setting the default timezone has been removed in version v2.3.4.
Thanks for your feedback
If you have any suggestions please let know


feat(list-ops): add exercise list-ops to vlang track (resolves #144)

General
Found this initially difficult. Though once I understood that an array could be a generic parameter, everything dropped into place. I've given this a difficulty rating of 5. What do you think?
This exercise requires an understanding of generics and touches on some aspects of functional programming techniques (mapping, filtering, folding, ..., etc). Do these topics need to be mentioned anywhere as a prerequisite in the prerequisites field of list-ops/config.json?
Implementation (list-ops.v)
As per instructions the implementation uses no inbuilt functions/methods. I do call the .len property on an array though, but the length method could be used in its place.
All code has been tested with 2D arrays as input.
The map function has been changed to map_of to avoid a conflict with Vs built in map datatype.
Tests (run_test.v)
All tests explicitly state the types of the generic parameters even if the compiler could infer them. The main reason for this is that the testcases will give the student a hint as to how to properly use generics to solve the
problem, especially when it comes to multi-dimensional (e.g. []int)
test_concat_empty_lists(line 25). According to the problem specification expects an input of [] & an output of []. Because V is statically typed the input has been changed to [[]].
test_foldl_with_direction_dependent_function (line 113) - uses the suggested test input from UUID d2cf5664-... due the integer division issue.
This is the test scenario I want you to examine test_foldr_with_non_empty_list_directional_dependent_function (line 133). From the problem specification:
{
      "uuid": "be396a53-c074-4db3-8dd6-f7ed003cce7c",
      "description": "direction dependent function applied to non-empty list",
      "comments": [
        "Expects integer division (expects / to discard fractions).       "
      ],
      "property": "foldr",
      "input": {
        "list": [2, 5],
        "initial": 5,
        "function": "(x, y) -> x / y"
      },
      "expected": 2
}

I would expect 0 as the fractional part is discarded and I'm expecting 'x' to be the accumulator and 'y' the element. If the roles are transposed (i.e. x -> y, y -> x) then, we will get 2. There's a discussion here.
I follow the V convention of specifying the accumulator as the first argument and the element as the second. See arrays.reduce & arrays.fold.
test_foldr_direction_dependent_function_applied_to_non_empty_list (line 151). Uses data from be396a53- due to discarding fractional part after division.

Honestly, I would probably bump the difficulty up to 6. This one requires a decent amount of code and understanding lots of different functions.
The prerequisites field of list-ops/config.json is specifically if there are concept exercises (as opposed to practice exercises) that are prereqs for this. Once we have some of those functional concepts as concept exercises, we should start updating existing prerequisite fields, but leaving that blank is good for now.
For the foldr test on line 133, your solution seems reasonable to me. I'm not even sure how the canonical data is supposed to be equal to 2 (based on the comment showing what x and y correspond to (accumulator and element)).
Any reason this is still marked as draft?

I'm not even sure how the canonical data is supposed to be equal to 2 (based on the comment showing what x and y correspond to (accumulator and element)).

Same. Curiosity led me to petertseng's explanation:

note for posterity: The possible ambiguous interpretation for this function were:

acc, el -> acc / el

acc: 5, el: 5. new acc 1
acc: 1, el: 2. new acc 0

acc, el -> el / acc

acc: 5, el: 5. new acc 1
acc: 1, el: 2. new acc 2

Would never have occurred to me to use the second method, but that explains it.

@hraftery ahhhh okay so in order to get that interpretation, you have to have (element, accumulator) -> element / accumulator AND traverse the array right-to-left, yeah?

Thanks for the extra pair of eyes. Now marked as ready for review. I'll push the difficulty up to 6 and update this PR.


Add hungarian language (replaces #211)

Current coverage is 94.37% (diff: 100%)

Merging #231 into master will not change coverage

@@             master       #231   diff @@
==========================================
  Files            32         32
  Lines          2736       2736
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2582       2582
  Misses          154        154
  Partials          0          0

Powered by Codecov. Last update f75cc77...adaaf11


build runs kj/compat/http-test which appears to reach out to the internet

I was building behind a firewall, so this eventually timed out and failed, and it looks like the build succeeded anyway, but it took a long time.  The IP address it was trying to reach was the one shared by capnproto.org and sandstorm.io.

Dang, I carefully wrote the test such that if the DNS lookup or connection failed, it would skip the test, but I didn't account for a network that succeeds at DNS lookup but then silently drops packets. I suppose I could add a timeout.

(All it is doing is an HTTP GET to test the HTTP client code against a real server. In theory I should, like, bundle a real HTTP server into the test in order to do this hermetically, but...)


invalid sql type StringArray (slice) for postgres

I want to use the postgres array and I'm using the pq.StringArray as a type for may array which I know and expect that it has implemented the Scanner interface but when I use AutoMigrate(&Post) for example I still get this error:
invalid sql type StringArray (slice) for postgres
import "github.com/lib/pq"
...
type Post struct {
    ...
    Tags        pq.StringArray
}

I solved it by using the gorm tag like this:
import "github.com/lib/pq"
...
type Post struct {
    ...
    Tags        pq.StringArray `gorm:"type:varchar(64)[]`
}

I tried it the first time too but there was an accidental space after the gorm: and it wasn't working without any errors like this:
Tags        pq.StringArray `gorm: "type:varchar(64)[]`

anyway it's working now

package main

import (
	"github.com/jinzhu/gorm"
	"github.com/lib/pq"
	"log"
)

func main() {
	db, err := gorm.Open("postgres", "user=lw dbname=kk sslmode=disable")
	if err != nil {
		log.Panic(err)
	}
	db.AutoMigrate(&Test{})
}

type Test struct {
	ID   uint64         `gorm:"primary_key"`
	Tags pq.StringArray `gorm:"type:varchar(100)[]`
}

Output:
panic: invalid sql type StringArray (slice) for postgres

goroutine 1 [running]:
github.com/jinzhu/gorm.(*postgres).DataTypeOf(0xc420139e40, 0xc420070ea0, 0x0, 0x0)
	/go/src/github.com/jinzhu/gorm/dialect_postgres.go:77 +0x425
github.com/jinzhu/gorm.(*Scope).createTable(0xc420141500, 0xc420143180)
	/go/src/github.com/jinzhu/gorm/scope.go:1097 +0x2e2
github.com/jinzhu/gorm.(*Scope).autoMigrate(0xc420141500, 0x695ee0)
	/go/src/github.com/jinzhu/gorm/scope.go:1176 +0x3e4
github.com/jinzhu/gorm.(*DB).AutoMigrate(0xc42016a000, 0xc42014bf48, 0x1, 0x1, 0x1)
	/go/src/github.com/jinzhu/gorm/main.go:557 +0x77
main.main()
	/main.go:14 +0x198

Hi,
It seems to me that it should work.
What am I doing wrong?

@mehdy Thanks for your solution. I found a small typo in your snippet
import "github.com/lib/pq" ... type Post struct {     ...     Tags        pq.StringArray gorm:"type:varchar(64)[]" }
(Missing " at the end)

@mehdy no space work for me

I used gorm: "type:text[]"
anything wrong with that?

It would be great if there was an example in the docs.
This was very helpful.


RD2 Custom Form: Donor Type Selection

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-040321
Also W-034369

Tracking W-040321

cci flow enable_rd2 fails:
2020-06-12 15:58:07: [Failed]: Update of CustomObject npe03__Recurring_Donation__c: Error: RD2_EntryForm does not exist or is not a valid override for action New. 2020-06-12 15:58:07: Exception in task deploy_rd2_config

@npsp-reedestockton There's actually no code in the component to do anything with the Account or Contact selected, so I'm not sure why selecting an Org Contact would anything different on this page than it would elsewhere. What did you select for "Donor Type" when you got this error?

@npsp-reedestockton I am able to reproduce your error though. Looking into it.

@andrewyu-salesforce - Instead of "Complete this field" for the validation error message, can we use "Contact is a required field."?

@sam-knox The 'Complete this field' validation can be customize to anything. However, this is the standard Salesforce validation message for required field. If we want to change the contact's validation message, I think it will be better to also customize all other required fields' messages on the entry form. @force2b

@sam-knox and @andrewyu-salesforce It doesn't appear that the built-in validation messages (ex: "Complete this Field") can be customized in LWC. I tried a number of variations. Even the documented setCustomValidity() javascript function isn't available in LWC. For now I think we're stuck with the message that Salesforce generates for these.

@force2b Everything looks great

@force2b @andrewyu-salesforce Good morning.  Wanted to let you know that I am all finished testing the WIs associated with this PR.  Testing passed.  I restarted the Feature Test - Robot last night to see if we could get a pass on that.  The Namespaced one (I think it was that) was still running.  It looks like Mike restarted that this morning due to it failing.

QE testing is now complete for this PR and associated WIs.  Based on the WI's Descriptions, ACs, UI design documents, test charters, and additional feedback, tests have passed.
For further WI and testing details, see:
W-034369
https://salesforce.quip.com/zYKMAcbD0snD
W-040185
https://salesforce.quip.com/xLJDA8moOYZw
W-040321
https://salesforce.quip.com/7VXFA6P5pQu3
W-041485
WI statuses will be set to 'Closed' once post merge builds have successfully passed.


[Feat] Add Bluesky as social platform option

Problem Statement
Bluesky seems to be raising in popularity very fast and there is no option here to add it as social platform
Suggested Solution
Add Bluesky as an option just like Twitter
Additional Considerations
Nope, all good! Thanks for all these SDKs they are truly a breeze to use!  =)

Created a PR for it here:
https://github.com/FlineDev/LinksKit/pull/5

@ChristianSko As you said that #5 was your first open-source contribution, here's a tip:
If you write something like "Fixes #4" or "Implements #4" in the PR description, the related issue will auto-close when the PR is merged. Just a convenience for your future contributions. 😊
Closing now, thanks again for fixing this! 🙏

Duly noted, @Jeehut, that is really cool; I didn't know! Thanks for sharing!
I'm looking forward to doing more.
Actually, thank you for all these SDKs; they are helping me a lot lately. I'm just happy I can give back somehow.😄


Incompatible in GNOME Shell 40.4

https://extensions.gnome.org/extension/1193/maximized-by-default/
At official Gnome extensions site it "INCOMPATIBLE" for my Gnome installation on Fedora 34 :(

I would also love to see this compatible with current Gnome versions -- seems like an ideal plugin for small screens like my XPS 13.


Build a map SHA256 checksum ~> bag ID

For the purposes of the Preservica cleanup in #4187.

Actually, nah.


Will get you banned from Valorant!!!

Just got banned for a week, opened it on a different computer on a different network with a new account, and got instantly banned on it too. THIS WILL NOW GET YOUR ACCOUNT BANNED.

See #73


Feature/Dev-543-Performance-tests-for-Challenges-service

feat: add performance tests to challenges service

Why does the branch name contain the Jira issue ID..?

Why does the branch name contain the Jira issue ID..?
Plus: this should be a test branch, not feature.

Plus: this should be a test branch, not feature.

Plus: the module name is missing. (test(challenges)/...)

Plus: this should be a test branch, not feature.

Plus: the module name is missing. (test(challenges)/...)

Should I make another branch/commit/PR ?

For now it's okay if you just change the PR title. But please keep in mind to carefully think about the commit messages and branch names in the future.

The title of this PR is still wrong


rpython main.py throw

Hi, I was excited to find this project. I can't wait to give a try. And I meet the following error
emcc: warning: linker setting ignored during compilation: 'WASM' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'ALLOW_MEMORY_GROWTH' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXPORTED_FUNCTIONS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXTRA_EXPORTED_RUNTIME_METHODS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'WASM' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'ALLOW_MEMORY_GROWTH' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXPORTED_FUNCTIONS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXTRA_EXPORTED_RUNTIME_METHODS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'WASM' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'ALLOW_MEMORY_GROWTH' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXPORTED_FUNCTIONS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXTRA_EXPORTED_RUNTIME_METHODS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'WASM' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'ALLOW_MEMORY_GROWTH' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXPORTED_FUNCTIONS' [-Wunused-command-line-argument]
emcc: warning: linker setting ignored during compilation: 'EXTRA_EXPORTED_RUNTIME_METHODS' [-Wunused-command-line-argument]
emcc -s WASM=0 -s ALLOW_MEMORY_GROWTH=1 -s 'EXPORTED_FUNCTIONS=["_main", "_malloc", "_onresolve", "_onfunctioncall"]' -s 'EXTRA_EXPORTED_RUNTIME_METHODS=["ccall"]'   -o main.js testing_1.o data_rpython_flowspace.o data_rpython_memory_gctransform.o data_rpython_rlib.o data_rpython_rtyper_lltypesystem.o data_rpython_translator_c.o nonfuncnodes.o implement.o rpython_flowspace.o rpython_memory_gctransform.o rpython_rlib.o rpython_rtyper.o rpython_rtyper_lltypesystem.o rpython_translator.o rpython_translator_c.o thread.o entrypoint.o mem.o exception.o rtyper.o support.o profiling.o debug_print.o debug_traceback.o asm.o instrument.o int.o stack.o threadlocal.o ../module_cache/module_0.o  -lrt  -Wl,--export-dynamic
emcc: warning: EXTRA_EXPORTED_RUNTIME_METHODS is deprecated, please use EXPORTED_RUNTIME_METHODS instead [-Wdeprecated]
emcc: error: undefined exported symbol: "_onfunctioncall" [-Wundefined] [-Werror]
make: *** [Makefile:104: main.js] Error 1
child_process.js:655
    throw err;
    ^

Error: Command failed: make -j 4
    at checkExecSyncError (child_process.js:616:11)
    at Object.execSync (child_process.js:652:15)
    at Object.<anonymous> (/home/ubuntu/rpython/bin/compile.js:108:17)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
    at internal/main/run_main_module.js:17:47 {
  status: 2,
  signal: null,
  output: [ null, null, null ],
  pid: 1042169,
  stdout: null,
  stderr: null
}

Any idea?

Hmm I know what this is.. This usually happens when your main.py doesn't call to any javascript APIs (import javascript). Try to import javascript and call javascript.Object.get('console', 'log').call('hello world') on your main function, and please attach or paste your main.py file so I can reproduce the error that occurs

And I think, I need to update my example main.py from the README. Either that or I need to fix it to enable programs that don't interface to javascript (which doesn't serve a purpose but at least it will clear the confusion)

I solved this by removing "_onresolve", "_onfunctioncall" from
EXPORTED_FUNCTIONS=["_main", "_malloc", "_onresolve", "_onfunctioncall"]

I solved this by removing "_onresolve", "_onfunctioncall" from
EXPORTED_FUNCTIONS=["_main", "_malloc", "_onresolve", "_onfunctioncall"]

That was a pretty bad idea... onfunctioncall is for @function decorator and onresolve is for @asynchronous decorator, without it you can't interface to javascript API except with writing your own exported functions...

Sure, just wait for the correct solution here.

Is it possible to generate a standalone wasm file that can be loaded by something like pywasm?

Is it possible to generate a standalone wasm file that can be loaded by something like pywasm?

It is not possible, currently what this fork does is replacing gcc with emcc on RPython's program C source compiling phase. So it generally depends on all the emscripten glue code, hence I name the implementation emscripten.py on the javascript directory. Maybe in the future when there's another alternative that replace cc like emcc does, that uses WASI (so it will run on any webassembly runtime, not just browsers and node.js) I will add another implementation.


get blank subject

Following the README and I receive an email but missing subject. I'm on Ubuntu 14.04 & python 3.4.
Also using google smtp and 2 factor pwd for authentication.

Hi, could you make sure that it happens like you said and provide me with the code you used (minus the password and auth) to send the email? In theory this shouldn't happen because there are tests in place to ensure that the Subject header is set, but we may have overlooked something.


Generated java models workflow

What does this PR do?:
This PR adds a new workflow to generate, build, and publish java models. The maven publication targets the GitHub packages at the moment, the artifact name is io.devfile:api-model:2.2.0-alpha. I have altered the generate.sh so that it supports both typescript and java now. That potentially can break the typescript-models workflow, though I have run the workflow in my fork and it looked good for all steps before the actual yarn publication (the NPM_TOKEN is missing in my fork).

[ ] Documentation

Updated the README.md to reflect the fact that the generate.sh script can be used for both typescript and java.

[ ] Client Impact

Shouldn't impact existing clients.
How to test changes / Special notes to the reviewer:
I've updated the workflows, so they should be run to verify their correctness.

Thx, FYI you need to add SignedOff on the commits

Thanks, I will rebase the commits.
Also, can we change
    "attributes": {
      "description": "Map of implementation-dependant free-form YAML attributes.",
      "type": "object",
      "additionalProperties": true
    }

to
    "attributes": {
      "description": "Map of implementation-dependant free-form YAML attributes.",
      "type": "object",
      "additionalProperties": {
        "type": "object"
      }
    }

in the devfile.json for better types generation in java ?


CHEBI:92181 has a human-friendly name

CHEBI:92181 is actually lasalocid, an antibiotic compound produced by several Streptomyces species.

Thanks for letting us know. This entry has now been fully annotated.


Cant find ruby after updating version

After updating my gemfile from ruby 2.4.1 to ruby 2.6.3 when I run the installed package I get the error /usr/bin/env: ruby: No such file or directory.
Is 2.6.3 not supported?

All ruby version are supported. Closing since it is an old bug, but feel free to reopen with more details.


Entity mirroring isn't working properly

Since @naor2013 @Khroki took one look at this issue and ran away, I'll post it here for anyone else that may know how to resolve this. @codewarrior0 maybe you'll know something about this too.
Currently:

What it should do:

That's because it's treating it like a block and rotating it twice by adding 180 degrees to the rotation tag, when it's more of a circle that goes from -360 to 0 back up to 360, mirroring doesn't work on these diagonal angles. @Khroki did a working method for banners/signs/player heads that have 16 rotations by just matching the mirrored values here: https://github.com/Khroki/MCEdit-Unified/blob/master/pymclevel/blockrotation.py#L377 but here we need a formula for decimals. And I found it:
east <-> west = Rot * -1
north <-> south = Rot * -1 + 180
If anyone knows how to get that formula working, feel free to take a crack at it.

Which direction does Rot=0 face?

Whichever it is, one of them is going to be -Rot and the other 90-Rot

Ahh nevermind, found it! Fixed with https://github.com/Khroki/MCEdit-Unified/commit/cfb9b278d3e611c86f9a4488505f3625a602bc4d


How to instantiated WebTiledLayer

WebTiledLayer is sealed class

@winstone-su WebTiledLayer is not part of the ArcGIS Maps SDK for Kotlin Beta release. The functionality will be enabled during the final release of ArcGIS Maps SDK for Kotlin.  Check out the beta release notes


Add ability to know how many records were inserted and how many were updated on a BulkInsertOrUpdate call

BulkInsertOrUpdate should have an option to return how many records were inserted and how many were updated. It seem like the code is already keeping track of this internally, but I believe there should be a way to extract that information after the call has been completed.
Thanks!

I assume you are doing insert/update by some unique column(s).
At the moment you can set SetOutputIdentity to True in BulkConfig which will load back inserted data. This will give you Id created by Db (int type, autoincrement).
Now there is no explicit info which ones were inserted and which were updated, and I am not sure how to check for this in the library itself.
But I believe you can split that list into 2 these 2 group after BulkInsertOrUpdate().
One way would be to use Id(s). If it was int type then before inserting get last Id and all those that have Id > LastId are inserted, others were update.
On the other hand If Id is of type Guid, SeqGuid created not in Db but in application, which you would have to set for all items in list, and then after function is executed those that have Id value, would be ones that were updated, because Id was loaded from existing record.
However I have another procedure for situation like this.
I first load records in memory into a dictionary where key is joined string of unique columns.
Then make separation of Insert group and Update group.
It is done so that Audit columns (CreatedTime, UpdatedTime) could be properly set.
This approach requires loading existing records into memory, options to do it:

Load all - feasible only if table is not very big.
Load only ones for update
2.a) Use Join or Where with Contains (more info in ReadMe) - translated to Sql IN
2.b) Use BulkRead - I am currently adding this method, and it will be published in a few days.

Hey @borisdj thanks for your reply.
I was thinking more along the lines of this solution.
Loading all records in memory is not a good solution for this case as the number of records can be very large. I think the most efficient way to get this info is to get the info as the operation is done as in the url above.
Thanks for your help!

I'll take a look.

I cloned your repository and hacked the code a bit without any design in mind. I got it to work and performance was descent. I tested a few different scenarios using tempdb and not.
Take a look at this link especially the solution by Rahul Sherawat. I think this could be become a new option where you can get the stats on the action performed if you'd like since it does increase overhead. That way people can choose it depend on their scenario.
Let me know if you'd like me to share my hack and I'll be glad to commit it to github...
Thanks!

I dont think its a good Idea to bring dapper as a dependency into a package that should work with EF ....

@JanEggers for sure. It was the easiest way to map the data of the temp table to a class object we can return.
As I said, it was just a hack and could use a proper design, but I think this is a good proof of concept and I think the project can benefit from an option like that...

I just merged your latest changes so everything is updated and compiles.
If you guys can figure out a way to get rid of Dapper and make everything work with ExecuteSqlCommand, I think everything we would be in good shape.

Hey @borisdj,
I made some changes to my commit to remove the dependency on Dapper. I'll create a pull request to help out with the merge. Hope this helps!

Ability added.
In BulkConfig thete is now CalculateStats which when is set to True result is return in object 'BulkConfig.StatsInfo', params: (StatsNumberInserted and StatsNumberUpdated).

Hey @borisdj,
I like your implementation of it. It's simpler than what I came up with.
One thing that you could improve in the future is to also calculate stats on deletes instead of only inserts and updates since the $action would also take care of that.
Thanks for your good work!

I'll see to add it with next version.
You are welcome.

It should work even if only one is set.
If there is some problem then could be a bug, but I have just tested it with only CalculateStats = True, and it in StatsInfo I got the result.

It's a bug then.
I thought it works because I tested it with only Import list of 1000, and got results 1000/0 but that was only because bit IsUpdate is false by defualt.
Instead of
if (tableInfo.BulkConfig.SetOutputIdentity)
it shoud be
if (tableInfo.CreatedOutputTable)


Sketch 52.2 not compatible plugin

I was update to latest sketch and compatible with version 4.2.1

What version of Sketch are you running right now? The latest 55.2?

Same issue here. Yes Sketch 55.2.

Just to let you guys know, I'm currently looking at this issue. For now, you can manually enable it on the plugin manager and it work just fine. I will post an update as soon as I find the issue.
Thanks for understanding it.

Should be fixed now, please update the plugin

JFYI, RenameIt does work when manually reenabled, but I do not see any updates available for the plugin via Sketch.

@chsWeb have you tried restarting Sketch?


Internet Archive Podcasts

There are many podcasts published on Internet Archive under permissive licenses that can be transcribed with an ASR system like whisperX.
Below are the Internet Archive search queries for English podcasts under different licenses (see IA Search Guide for how to filter by license):

Public Domain - 8,131 results
CC-BY-SA - 10,061 results
CC-BY - 4,530 results

I have not tested and cannot verify whisperX transcription quality on non-English audio, but here are the search queries for permissively licensed podcasts in all languages:

Public Domain - 33,826 results
CC-BY-SA - 18,270 results
CC-BY - 17,720 results

With these search queries we can follow these instructions to bulk download the podcasts and then pass them through an ASR system.

Napkin math: 20k podcasts, ~30 minutes/podcast, 150 tokens/minute = 90M tokens? Not sure we should put a ton of effort into this but if it's easy to reuse the YouTube pipeline then it wouldn't hurt.


[Spec Impl] Add checksum to all writes

Background
Every write from/to Wallet has a possibility to have corruption. Hence all writes should include a checksum at the last.
What to do

Modify all the structures chunk write, transfer report request, transfer summary report to include checksum at the end.
Finalise on the CRC to be used. The suggestion is to use CRC16-CCITT-True which is used in Bluetooth 5.3
Update Transfer report structure to match with the specification. Also investigate if the mentioned structure is okay for a success and failure case. Since it will have CRC, on a success case, it will still have CRC value to differentiate between a success report Vs a spurious empty characteristic read.

References for finalising the CRC
https://reveng.sourceforge.io/crc-catalogue/16.htm

This requirement has been moved to Jira.
Link to Jira ticket: https://mosip.atlassian.net/browse/MOSIP-26906


[python] Introduce GetClusterInfoById

Problem
What is being fixed?  Examples:

Add mapping or lookup method of Cluster IDs for verification on Test Harness.
Fixes #8575

Change overview

Update python/chip/clusters/ChipClusters.py to support looking up method.

Testing

Added test.TestPythonAPI(self) to test this API.

Could  you separate out the 'generate file' from the rest of the commits to make code review easier?

@erjiaqing - merge conflicts


1.13.0 - Not all artifacts in maven central

   > Could not find io.gitlab.arturbosch.detekt:detekt-tooling:1.13.0.
     Required by:
         project :web-app > io.gitlab.arturbosch.detekt:detekt-cli:1.13.0
   > Could not find io.gitlab.arturbosch.detekt:detekt-parser:1.13.0.
     Required by:
         project :web-app > io.gitlab.arturbosch.detekt:detekt-cli:1.13.0
   > Could not find io.gitlab.arturbosch.detekt:detekt-core:1.13.0.
     Required by:
         project :web-app > io.gitlab.arturbosch.detekt:detekt-cli:1.13.0
   > Could not find io.gitlab.arturbosch.detekt:detekt-rules:1.13.0.
     Required by:
         project :web-app > io.gitlab.arturbosch.detekt:detekt-cli:1.13.0

See https://github.com/detekt/detekt/pull/3060 hopefully jcenter will resync everything.

I will spend some time in setting up direct publishing to maven central.
Bintray is highly unreliable for us since we upload checksums and signatures.
Could take some days.

also:
Could not find io.gitlab.arturbosch.detekt:detekt-rules:1.13.0

Also detekt-formatting

I will spend some time in setting up direct publishing to maven central.

Have you worked on this @arturbosch? Otherwise I can pick it up

I will spend some time in setting up direct publishing to maven central.

Have you worked on this @arturbosch? Otherwise I can pick it up

Yes, I had already set it up. Thanks for the offer!
I just published 1.13.1 to maven central directly.
It should be up soon.


the endpoint-operator renamed to public repo klusterlet-addon-controller

Signed-off-by: Xiangjing Li<EMAIL_ADDRESS>

@mikeshng  FYI. As discussed, the placementrule repo has been updated to pick up the public klusterlet-addon-controller repo


Add scalac_jvm_flags option to toolchain

This is useful for globally setting the max heap usage of Scalac workers. At Stripe, we've seen Scalac workers get killed by the kernel OOM killer because the default JVM heap size is too large.

happy to merge when you fix the comment pt found.

Thanks all! Comment fixed.
I also made a slight behavior change. It occurred to me that flags on the target should override flags passed on the toolchain (rather than combining them). WDYT?

@beala-stripe +1 to making the target override. The toolchain should be the default, but if you write it on a target, we get exactly what is there.
That's a good approach.


tsdoc-reference-unquoted-identifier is incorrectly reported for certain identifiers

For example, {@link type} gives this error:
tsdoc-reference-unquoted-identifier: The identifier "type" must be quoted because it is a TSDoc system selector name  tsdoc/syntax

Originally posted by @diegosogari in https://github.com/microsoft/tsdoc/issues/148#issuecomment-1867669304

Any updates on this? I'm facing the same issue for {@link Something.type}. When I replace it with {@link Something["type"]}, it throws the following error:
tsdoc-reference-missing-dot: Expecting a period before the next component of a declaration reference

Any updates on this? I'm facing the same issue for {@link Something.type}. When I replace it with {@link Something["type"]}, it throws the following error:
tsdoc-reference-missing-dot: Expecting a period before the next component of a declaration reference

@sagargurtu, you just need to use {@link Something."type"}. See @link documentation.


API: Make reset() private and call it from join() and personalize()

All examples call ttn.reset() before they call ttn.join() or ttn.personalize(). If this is a required step, why not make it private and call it from these two methods.
Current:
ttn.reset();
ttn.personalize(devAddr, nwkSKey, appSKey);

Result:
ttn.personalize(devAddr, nwkSKey, appSKey);

To @johanstokking's note, if the device comes pre-flashed with keys you'd call the methods with no arguments:
while (ttn.join()) {
}

The methods should then not call reset() and skip the mac set commands.

Yes, I think we can do the reset() in personalize() and join(). The reason why it's separate, was to be able not to reset and join with the values that are stored in the RN module, instead of providing the values. But we can account for that in join() and/or with an overload that doesn't take keys and therefore doesn't reset.

Yes, that seems like a good simplification since doing a reset() or personalize() without reset() will be the common scenario. I'll leave this assigned to you as I'm finding my way around in C/Arduino still ;)

Of course if we call reset() from join() this will mean that if it takes a while to do a successful join, every new attempt will first reset() the device. This is a bit of an overhead.

Also, I think we should make it optional not to reset the device at all. I can imagine the AppEUI and AppKey being pre-flashed, especially in The Things Node, that we don't want to reset.

Closed by #58


ListView with "Pull to Refresh"

I want to implement a ListView with Pull to Refresh control like Android.
But I feel confused about how to trigger the event and how to notify the "ListView" to update. I'm not using Flux. Just react. Is there any "ListView with Pull to Refresh" demo? What is the best for controlling the data and event flow if not using any Flux.
It's very urgent, if there is no available code, you can tell me a idea or solution to it?
Thx.

+1

I'd suggest starting here:
https://github.com/apeatling/web-pull-to-refresh
Based on the way its written, I believe you could place the refresh indicator inside the loading area (ID ptr in the demo). Then just set the status prop to ready and tie the size/percentage to the size of the loading area container.
At some point if I actually implement this I'll post it back here.

I can attempt this if this is still open?

I have added the waiting for users upvotes tag. I'm closing the issue as we are not sure people are looking for such abstraction. So please upvote this issue if you are. We will prioritize our effort based on the number of upvotes.

@oliviertassinari this is interesting when you implement, for example, a PWA and you need to override the default "pull" behavior of the browser to perform, for example, updating local data

@UseMuse Thanks for sharing, we used to have this component but we dropped it. it seems that few people needed it. However, we have added the "waiting for upvotes" label to gain a better understanding of the need.
For benchmark purposes, I could find:

https://ionicframework.com/docs/api/refresher


add MUON token

[ x] I will not ping the Discord about this pull request.

Please provide the following information for your token.
Please include change to the src/lib/.json and in the PR
At minimum each entry should have

Token Address: muons1SXVihhnwCJAC7T7o97jHHVcT8e2o7NkC6VFmo
Token Name: Muon
Token Symbol: MUON
Logo: https://raw.githubusercontent.com/muon-labs/muon-assets/main/muon-final.png
Link to the official homepage of token: https://muon.fi
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Domain is currently migrating from https://projectmuon.com -> https://muon.fi there may be some delay.

Incorrect josn doesn't build


[5.1] added queue.finished event for tracking queue throughput

Allows you to throw a call to StatsD in there via a closure or any number of things for specific use cases.

Thanks! I renamed it to Queue::after to be more consistent with a few other parts of the framework.

Awesome!


Requires PULID to keep face consistent

I guess if we one incorporates PULID into workflow face won't change so much anymore.

@Priestru any idea how to implement PULID because I had implemented it but it is taking a lot of time to process for pulid pipeline inference


Display custom fields with bootstrap form-control css class

Add a widget with the CSS class form-control for every type of custom fields in order to have a unified form in the ticket creation page.

Codecov Report

Merging #892 into master will decrease coverage by 0.08%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #892      +/-   ##
==========================================
- Coverage   73.47%   73.38%   -0.09%
==========================================
  Files          51       51
  Lines        4863     4870       +7
  Branches      562      564       +2
==========================================
+ Hits         3573     3574       +1
- Misses       1134     1140       +6
  Partials      156      156

Impacted Files
Coverage Δ

helpdesk/forms.py
58.11% <14.28%> (-1.10%)
:arrow_down:

helpdesk/lib.py
67.60% <0.00%> (-0.97%)
:arrow_down:

helpdesk/email.py
77.77% <0.00%> (ø)

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 54c7f0b...ea558d7. Read the comment docs.


Videos not always perfectly in sync

I'm seeing videos being sometimes slightly out-of-sync between the different "tiles".
I can likely work on this.
cc @nerik

I can second that, for me when loading they are usually out of sync. Here is an example
Screencast from 28.06.2023 10:21:11.webm

@yellowcap so you should always be able to pause and then play again which "re-syncs" the videos. But will work on running some sort of "sync" operation in a loop to keep managing the sync.


Groups are not handled correctly

Group Hue is a group with one device in it namely FRITZ!DECT 200 #1
Device ID:           DB:55:D2-900
Device Name:         Hue
Device State:        ON
Device Power:        HTTP/1.0 500 Internal Server ErrorContent-Length: 0Content-Type: text/plain; charset=utf-8W
Device Energy:       HTTP/1.0 500 Internal Server ErrorContent-Length: 0Content-Type: text/plain; charset=utf-8Wh
Device Temperature:  inval degree Celsius

Device ID:<PHONE_NUMBER>56
Device Name:         FRITZ!DECT 200 #1
Device State:        ON
Device Power:        0.0W
Device Energy:       775Wh
Device Temperature:  inval degree Celsius

After updating tot 6.52 groups are still not handled correctly.
Device ID:           DB:55:D2-900
Device Name:         Hue
Device State:        ON
Device Power:        0.0W
Device Energy:       0Wh
Device Temperature:  inval degree Celsius

Device ID:<PHONE_NUMBER>56
Device Name:         FRITZ!DECT 200 #1
Device State:        ON
Device Power:        0.35W
Device Energy:       1203Wh
Device Temperature:  8.5 degree Celsius


📚 Documentation: Update InputFile documentation about what to use for Flutter Web

💭 Description
On Flutter Web, you must use InputFile.fromBytes(). This was fine in the past because Flutter file/image picker packages didn't provide a path. They've added support for path now, but the path is an Object url and not an actual image path. As such, this path can't be used by the Appwrite SDK.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@stnguyen90 I could take this issue if it hasn't been worked on?


Warn on total scores not divisible by 5

This should produce an error if the format doesn't allow for non-multiple-of-5 scores, see #24.

I believe the general case for this would be to warn/output error if score % greatest_common_divisor(all_scores) != 0?

Where does all_scores come from? But yes, assuming it is properly defined.

Sorry, I meant the tossup values. So if you had something like [10, 20, -5] as your tossup values, the gcd would be 5 and all scores should be divisible by 5.

That's not correct; bonus scores could contribute too.

So the gcd of the tossup values and the value of a bonus part?

That only works if all bonus parts have the same value, which is not necessarily true. I'd just do gcd(all tossup values, bonus GCD) where bonus GCD is entered manually.

Currently, the application assumes each bonus part is worth the same, so the above would work. If at a later time I go back and add in support for bonus parts potentially being worth variable amounts, I'd have to change it to your solution.

That's inconsistent with ACF rules, so I suggest changing it sooner rather than later :-)

Would it really be that high of a priority, though? According to the ACF rules I found, "in virtually all cases, a bonus will consist of three parts worth 10 points each".

I can't tell you how to prioritize your project, but it is still possible for bonus parts to have different values in ACF-rules quizbowl. The most common case would be a part that asks for two answers worth 5 points each.

I don't think it would be hard to add in, but I'm wondering if the other issues should take precedence.

Warning added in commit 9b6f4a2dbd29fc26367b5c06e33547ff41beeb03


Fix sed induced typo

s/pollent/pops

@sreecha , if you catch this before @ctiller

Lgtm... merge on green (comment only change)

Failures are known issues (tsan/msan timing out).


EOD/eod/models/heads/utils/bbox_helper.py", line 341, in clip_bbox     dw, dh = img_size[6], img_size[7] IndexError: list index out of range

I found in Inferece.py the prepare are:
def fetch_single(self, filename):
        img = self.image_reader.read(filename)
        data = EasyDict(
            {"filename": filename, "origin_image": img, "image": img, "flipped": False}
        )
        data = self.transformer(data)
        scale_factor = data.get("scale_factor", 1)

        image_h, image_w = get_image_size(img)
        new_image_h, new_image_w = get_image_size(data.image)
        data.image_info = [
            new_image_h,
            new_image_w,
            scale_factor,
            image_h,
            image_w,
            data.flipped,
            filename,
        ]
        data.image = data.image.cuda()
        return data

which image_info max size is 7, so that above index [7] is out of indices. How to resolve?

I found in Inferece.py the prepare are:
def fetch_single(self, filename):
        img = self.image_reader.read(filename)
        data = EasyDict(
            {"filename": filename, "origin_image": img, "image": img, "flipped": False}
        )
        data = self.transformer(data)
        scale_factor = data.get("scale_factor", 1)

        image_h, image_w = get_image_size(img)
        new_image_h, new_image_w = get_image_size(data.image)
        data.image_info = [
            new_image_h,
            new_image_w,
            scale_factor,
            image_h,
            image_w,
            data.flipped,
            filename,
        ]
        data.image = data.image.cuda()
        return data

which image_info max size is 7, so that above index [7] is out of indices. How to resolve?

https://github.com/ModelTC/EOD/blob/a45b74430070d82d9248a10fb5e1116bb7ababe1/eod/data/datasets/coco_dataset.py#L297.  This is a bug for yolov5 (center pad) inference, we will fix it later

@jinfagang The bug has been fixed.

@Joker-co can we have a demo.py sinply read a image as input and show result? Don't involve with any dataloader so that it can easily use and quickly vis final result.

We think the demo now seems to be able to meet the demand. Config is necessary in order to parse the test configuration, and we don't build dataloader. @jinfagang

@yqyao something like this could be better, whole code easy to modification rather than deeply wrapped inside EOD package.
And this demo give wrong result:
https://github.com/ModelTC/EOD/issues/3#issuecomment-956043047


[TA] Autorest Customization, Rename class

Refactor Autorest README to rename models.

API change check
API changes are not detected in this pull request.


Improve fetching and matching

First of all I want to indicate that I know there is a pull request  modifying the behavior of result filtering. However I'm not sure if this goes into the right direction, from the description of the pull request:
Title matching changes:

    - Title matching is always run
    - If no results have a matching title, fall back to all results
    - Adds warning when title matching fails

I think the title matching is way too restrictive, I'm using prowlarr and zilean so I cannot disable it (I'm not sure why, documentation says it needs to be enabled for zilean).  Here are some logs I fetched from my setup:
2024-11-14 11:48:27 | 📰 INFO | stream.stream - No cache found for Daredevil S03E01 with user configuration
2024-11-14 11:48:27 | 📰 INFO | stream.stream - Start of prowlarr search for Daredevil S03E01 with indexers ['veryLegalIndex1', 'veryLegalIndex2', 'veryLegalIndex3', 'veryLegalIndex4', 'veryLegalIndex5']
2024-11-14 11:48:27 | 📰 INFO | general.get_zilean - 1 torrents found for Daredevil S03E01 with Zilean
2024-11-14 11:48:38 | 📰 INFO | stream.stream - 387 torrents found for Daredevil S03E01 with prowlarr, Zilean
2024-11-14 11:48:38 | 📰 INFO | stream.stream - 11 torrents passed title match check for Daredevil S03E01
2024-11-14 11:48:46 | 📰 INFO | stream.stream - 11 info hashes found for Daredevil S03E01
2024-11-14 11:48:46 | 📰 INFO | stream.stream - 0 cached files found on realdebrid for Daredevil S03E01

If I make a manual search on prowlarr with same query string, I can see that most results are relevant. One thing I noticed is that for movies the matching works much better while for series I almost get no results on stremio. Example of movie:
2024-11-14 12:33:24 | 📰 INFO | stream.stream - Start of prowlarr search for The Wild Robot with indexers ['veryLegalIndex1', 'veryLegalIndex2', 'veryLegalIndex3', 'veryLegalIndex4', 'veryLegalIndex5']
2024-11-14 12:33:24 | 📰 INFO | general.get_zilean - 48 torrents found for The Wild Robot with Zilean
2024-11-14 12:33:27 | 📰 INFO | stream.stream - 243 torrents found for The Wild Robot with prowlarr, Zilean
2024-11-14 12:33:27 | 📰 INFO | stream.stream - 193 torrents passed title match check for The Wild Robot

I think the approach of the pull request is a bit too simple because if we do not get any result from title matching then it returns everything. I'm not sure exactly why we need a title matching in comet, I think it needs to be clarified. An issue I also observed is that sometimes you only get 1 result but all other relevant results are filtered by title matching which wouldn't be solved with the new approach.
I did not read the changes of the pull request in details but I can help if required, I have some knowledge in python and async.

Hello, the time you got 0 was absolutely not due to the title match check. It's the RealDebrid API that's acting a bit crazy at the moment and sometimes returns 0 cached results even though there are some. If you try again in a few minutes, you'll see that you'll get plenty of results.
Also, you can disable the title match check if you feel like experimenting. The warning that says you shouldn't do it when Zilean is activated was put in for an older version of Zilean that returned a lot of crap.

I tried multiple times different days and I get the same behavior for different episodes. I will try to figure out manually what is causing this issue (could be real debrid api indeed), it also take a lot of time to fetch from real debrid which I'm not sure can only be explained by the load on their api ( Torrentio with rd was always instant)

User who does not understand that Torrentio results are instant because everything is already cached.


HTML5 and responsive layout

I've moved the catalog pages over to HTML5 and:-

Moved the grid system to unsemantic/adapt.js (http://unsemantic.com/) for a cross-browser responsive layout (both js and non-js)
Added head.js (http://headjs.com/) and moved the css and js around for optimum load times.

Thanks. A responsive layout will be introduced in  v2.3.6 onwards.


SignatureSha256 and PackageFamilyName

Resolves https://github.com/microsoft/winget-pkgs/issues/24066

@vedantmgoyal2009

Thanks, now I will add some more things. Actually, I don't know about YamlParsing which is why I could not proceed.

Once this will get merged, I will do some more chnages. Thanks for this. I do not know about YamlParsing, so I couldn't proceed.


DiffCover / Jacoco issue in Bitrise Ci

I succeed to run diff_cover on my macOS, but it does failed on the Bitrise Ci server:
Here is the issue:
Traceback (most recent call last):
  File "/usr/local/bin/diff-cover", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/diff_cover_tool.py", line 220, in main
    diff_range_notation=arg_dict['diff_range_notation']
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/diff_cover_tool.py", line 178, in generate_coverage_report
    reporter.generate_report(output_file)
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/report_generator.py", line 254, in generate_report
    report = template.render(self._context())
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/report_generator.py", line 303, in _context
    context = super(TemplateReportGenerator, self).report_dict()
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/report_generator.py", line 182, in report_dict
    src: self._src_path_stats(src) for src in self.src_paths()
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/report_generator.py", line 84, in src_paths
    return {src for src, summary in self._diff_violations().items()
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/report_generator.py", line 176, in _diff_violations
    for src_path in self._diff.src_paths_changed()
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/diff_reporter.py", line 147, in src_paths_changed
    diff_dict = self._git_diff()
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/diff_reporter.py", line 196, in _git_diff
    for diff_str in self._get_included_diff_results():
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/diff_reporter.py", line 169, in _get_included_diff_results
    included = [self._git_diff_tool.diff_committed(self._compare_branch)]
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/git_diff.py", line 54, in diff_committed
    '-U0'
  File "/usr/local/lib/python2.7/dist-packages/diff_cover/command_runner.py", line 43, in execute
    raise CommandError(stderr)
diff_cover.command_runner.CommandError: fatal: ambiguous argument 'origin/develop...HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

Any idea what could be wrong ?

I use this configuration:
  diff-cover "$NEWSFEED_JACOCO_REPORT" \
    --compare-branch=origin/develop \
    --html-report "$NEWSFEED_DIFF_COVER_REPORT" \
    --src-roots "$NEWSFEED_SRC"

Any chance you are working from a shallow clone of the repo? That would explain it.

I would also try to run the diff outside of diff cover. Just with a vanilla git command
git diff origin/develop...head

@Bachmann1234 That was the case, I did a git diff in the meantime, and "tada", it doesn't work too.
So for others in my case, you need to fetch the develop branch in your ci runtime, by:
git fetch origin develop then the branch is available so diff-cover can work.


Android 7.1.2 app crashes or "Unable to create bitmap!" when calling camera from work profile app

Dear team, first of all thanks for maintaining this fantastic plugin 👍
We experience an issue with the 4.0.3 plugin version in an app delivered with Microsoft Intune. The app is only available in Android work mode, when work profile is switched on.
When starting the camera, we get a message toast "you're using this app outside of your work profile". The camera opens and we can take a picture.
Then depending on destinationType the following happens:

DATA_URL: the app stops/crashes without any message
FILE_URI: we return to the app and the value we get in success callback is "Unable to create bitmap!"

We had some Microsoft people having a look at the FILE_URI version and this is their feedback:
The debugger informs in the beginning of the logs that the crash was due to “Attempt to invoke virtual method 'android.os.Bundle android.content.Intent.getExtras()' on a null object reference”
10-22 10:46:45.771 32379 32379 E AndroidRuntime: FATAL EXCEPTION: main
10-22 10:46:45.771 32379 32379 E AndroidRuntime: Process: com.aaa.bbb, PID: 32379
10-22 10:46:45.771 32379 32379 E AndroidRuntime: java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=33, result=-1, data=null} to activity {com.aaa.bbb/com.aaa.bbb.Test}: java.lang.NullPointerException: Attempt to invoke virtual method 'android.os.Bundle android.content.Intent.getExtras()' on a null object reference

 10-22 10:46:45.771 32379 32379 E AndroidRuntime: Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'android.os.Bundle android.content.Intent.getExtras()' on a null object reference

I read all the logs from the end and going up and found a few errors:
10-22 11:55:33.184  6106  6135 D StrictMode: StrictMode policy violation: android.os.StrictMode$StrictModeDiskReadViolation: policy=65599 violation=2

10-22 11:55:33.184  6106  6135 D StrictMode:  at android.os.StrictMode$AndroidBlockGuardPolicy.onReadFromDisk(StrictMode.java:1438)

10-22 11:55:34.139  3670  7369 E TaskPersister: File error accessing recents directory (directory doesn't exist?).

We appreciate your consideration in this matter
Best Regards

We are experiencing this issue too.

Sorry that this issue hasn't received any attention.
<EMAIL_ADDRESS>is severely out-dated now. If you're experiencing this issue, I would suggest updating to the latest version available on both the plugin and the platform, which at the time of writing, these are versions 4.1.0 and version 9.0.0 respectfully.
If you still experience the issue on the latest releases, then please do open a new issue with the issue form filled out.

same issue occured today in cordova android12


footer problem on page update

I'm creating an Angular version of AdminLTE https://github.com/csotomon/Angular2-AdminLTE
When I load a module that has little content, the footer is not located at the end of the screen, but if I open and close the left menu, the footer is located at the bottom of the screen.
You can se the problem here: https://youtu.be/g8Au_lOZKFg.
How can i solve this problem?

I fixed it calling
$(function(){         $.AdminLTE.layout.activate();         $.AdminLTE.layout.fix();         $.AdminLTE.layout.fixSidebar();       });


Add shebang to aws2tf.sh

As discussed in #10 adding a shebang to avoid syntax error on zsh and other shells.
Changes:

Added shebang
Minor indentation fix in the usage() function. I can remove this commit if needed.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you !


Integrate Maestro with Hive 0.12

CDH 5, Scheduled for end of July (within CBA), will bring Hive version 0.12

Done as part of the CDH5 upgrade. Currently on the CDH5 branch and will be merged next week.