pipenv check - Ruamel vendoring isn't working

Issue description
2022.11.23 included the new Safety version, which requires ruamel, the wheel for this pipenv version is missing the ruamel package therefore, pipenv check is failing with the following error:

Checking PEP 508 requirements...
Passed!
Checking installed packages for vulnerabilities...
Traceback (most recent call last):
  File "/Users/foo-user/.pyenv/versions/3.11.0/bin/pipenv", line 8, in <module>
    sys.exit(cli())
             ^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/cli/options.py", line 57, in main
    return super().main(*args, **kwargs, windows_expand_args=False)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/decorators.py", line 84, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/cli/command.py", line 510, in check
    do_check(
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/core.py", line 2985, in do_check
    from pipenv.patched.safety.cli import cli
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/patched/safety/cli.py", line 12, in <module>
    from pipenv.patched.safety import safety
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/patched/safety/safety.py", line 21, in <module>
    from .util import RequirementFile, read_requirements, Package, build_telemetry_data, sync_safety_context, SafetyContext, \
  File "/Users/foo-user/.pyenv/versions/3.11.0/lib/python3.11/site-packages/pipenv/patched/safety/util.py", line 17, in <module>
    from pipenv.vendor.ruamel.yaml import YAML
ModuleNotFoundError: No module named 'pipenv.vendor.ruamel'

Inspecting the wheel, we can see there isn't the ruamel package.

Expected result
pipenv check works as expected.

It is checked in though, I wonder if this is another case where the wheel built is excluding these files for some reason.

I merged a small change to setup.py to main that worked locally to include ramel in the wheel.   Feel free to try it out @yeisonvargasf  -- I plan to do a follow-up release 2022.11.24 either later tonight or tomorrow.

It is checked in though, I wonder if this is another case where the wheel built is excluding these files for some reason.

@matteius I reviewed this more in detail. I think the issue is that the ruamel directory under /pipenv/vendor/ruamel/ doesn't have a __init__.py, so in the build wheel process,ruamel is ignored.

I merged a small change to setup.py to main that worked locally to include ruamel in the wheel.

This change also worked for me.
I think adding a __init__.py makes more sense.

@yeisonvargasf adding an init file would get blown away during revendoring unless it was created via the patch, so I like this way for now.

I got it, it makes sense, and I agree. Thank you!

Ah bummer, the published wheel is still missing ruamel.

It is weird -- when I build with the same command that the github action locally uses, it does put ruamel in wheel file, but github actions does not.

Ok so will need a more permanent solution to keeping the init.py in place in the pipenv/vendor/rumael directory -- I'll keep this ticket open for that task.  However I just verified that new release 2022.11.25 does in fact include the required ruamel.


Using len(qubit_support) instead of qubit_support[0] to get number of qubits.

https://github.com/pasqal-io/qadence/blob/bfd4c4c4048f2bb76829667adceb3798d8d78017/qadence/blocks/primitive.py#L514

Thanks for this @caspergyurik could you please also post what caused the bug ?

n_qubits = 4
all_zero = "0"*n_qubits
projector = ProjectorBlock(all_zero, all_zero, n_qubits)

@caspergyurik This error is actually sensible: what you pass as qubit_support is the number of qubits. qubit_support is the set of qubit indices you want your operation to work on. It works if you do:
n_qubits = 4
all_zero = "0"*n_qubits
projector = ProjectorBlock(all_zero, all_zero, (0, 1, 2, 4)) <-- passing the qubit support


get  login QR-code as image?

is there a way to get the login QR-code as image or String? I want to send it as email from terminal application on server.

Use the qrHandler in WhatsAppOptions and pass those to the Whatsapp constructor


Out of Memory when running eval_knn.py

When I using pre-trained weights to evaluate knn
Traceback (most recent call last):
  File "../main/eval_knn.py", line 234, in <module>
    train_features, test_features, train_labels, test_labels = extract_feature_pipeline(args)
  File "../main/eval_knn.py", line 82, in extract_feature_pipeline
    train_features = nn.functional.normalize(train_features, dim=1, p=2)
  File "/root/anaconda3/envs/DINO/lib/python3.7/site-packages/torch/nn/functional.py", line 3591, in normalize
    return input / denom
RuntimeError: NPU out of memory. Tried to allocate 9.78 GiB (NPU 0; 32.00 GiB total capacity; 29.80 GiB already allocated; 1.59 GiB free; 50.34 MiB cached)

And the NPU(GPU) is 32GB, how can I solve it?
appreciate a lot!

btw, I set batch size to 16

Hey, where is the batch size ? I couldnt find it

Okay found it :D

Hey, how did u solve that ? I was tryin it on colab gpu but got same error


AAD / Sign in flow change

see Sign-In Flow change in OneNote
Work includes:

create new page for rejecting some logins

will be tested as part of https://github.com/NuGet/NuGetGallery/issues/5399


collect usage stats in a privacy respecting manner

Probably depends on #113 because the privacy respecting analytics services usually cost.


Customization features: fav routes, segments, etc

Convinced that it's valuable to have the ability to customize the UI to show the data I want to see—especially for commute
Way down the line: sign in with Google and it pulls in your work location etc

This will be useful when we implement this feature: https://github.com/gsklee/ngStorage

Station Cards: Star outline should be added to top-right corner of each and filled in when clicked
Favorites link in sidebar shows cards for all starred stations (sorted by alphabetical order)
List of favorited station ids will be stored in local storage

#35


Add 6Workstation, 7Workstation and 7.1 $releasever symlinks

Documentation for RHEL systems says to use the following URL for the salt repo https://repo.saltstack.com/yum/redhat/$releasever/$basearch/latest but there is no link for 6 and 7 Workstation machines. It would be nice if a symlink can be added to point to https://repo.saltstack.com/yum/redhat/(6|7)/$basearch/latest so workstation installs will work without needing to manually specify the version in the .repo file.
The default variables are 6Workstation and 7Workstation. It may also be useful to point out that some users (including me) use the point release for the $releasever variable. This means our 7.1 servers have $releasever set to 7.1 and not 7Server.
Copied from https://github.com/saltstack/salt/issues/30315.

Is there a need for 5Workstation ?

Added to staging area for repo, should be available with the next point release

Also added [7|6|5]Client and [7|6|ComputeNode similar to  http://ftp.redhat.com/pub/redhat/linux/enterprise/

@dmurphy18 good thinking. I also use computenode so that would be helpful too.


pkcs7_verify get actual error

Hi,
when i trying to verify pkcs7 signauture in case of error i want get the actual problem, such as self-signed certificate, the cert if outdated and so on. But https://github.com/openssl/openssl/blob/master/crypto/pkcs7/pk7_smime.c#L323 hides the actual error with PKCS7_R_CERTIFICATE_VERIFY_ERROR, is it possible to do smth with it?

you can get the error data via ERR_error_string_n


Setup a pipeline job for publishing Docs to daily branch.

The PR is to add a pipeline job of adding docs metadata json file to daily branch in nightly build.

This is the template build for testing purpose (I have remove some of the csv metadata filter logic to make it happen):
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1109217&view=logs&j=dc056dfc-c0cf-5958-c8c4-8da4f91a3739&t=fd8e7423-8fbc-52ed-a0aa-20018c9cc442
This the testing branch in python docs:
https://github.com/MicrosoftDocs/azure-docs-sdk-python/blob/daily/2021-09-23/metadata/preview/azure-template.json
For template release:
pipeline build:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1116250&view=logs&j=19b87903-f8bc-5cbb-22d9-4f6eb3469d8e&t=490c44f5-c811-50df-bb2c-044d037917a0
docs repo: https://github.com/MicrosoftDocs/azure-docs-sdk-python/blob/main/metadata/preview/azure-template.json

These runs and changes are exactly what we need. Please also verify against python - template so we can be 100% sure that there are no surprises in a package publish scenario.

@danieljurek I have triggered the python - template
Here is the python - template pipeline:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1130767&view=results
Here is the python - docindex pipeline:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1130786&view=logs&j=dc056dfc-c0cf-5958-c8c4-8da4f91a3739
Here is the py docs repo branch showing the changes:
https://github.com/MicrosoftDocs/azure-docs-sdk-python/tree/daily/2021-10-07
Here is the CI pipeline:
https://dev.azure.com/apidrop/Content CI/_build/results?buildId=257339&view=results
The review site did not show the template package by some reason. I guess we filter out the template in some of the steps.


fix(@angular-devkit/build-angular): add debugging and timing information in JavaScript and CSS optimization plugins

This can be useful to debug slow builds.
Example of output
LOG from build-angular.JavaScriptOptimizerPlugin
<t> optimize asset: runtime.ad5c30339e926c89.js: 221.959564 ms
<t> optimize asset: polyfills.ec3ffae5bac27204.js: 1071.080092 ms
<t> optimize asset: main.aa8a15155ca2133f.js: 3391.588635 ms
<t> optimize js assets: 3483.799739 ms

LOG from build-angular.CssOptimizerPlugin
<t> optimize asset: styles.d251c5bf54715558.css: 26.569907 ms
<t> optimize css assets: 34.441737 ms

LOG from build-angular.JavaScriptOptimizerPlugin
<i> polyfills.ec3ffae5bac27204.js restored from cache.
<i> runtime.ad5c30339e926c89.js restored from cache.
<t> optimize asset: main.69fb55a243b46bfa.js: 2618.5191210000003 ms
<t> optimize js assets: 2721.226144 ms

LOG from build-angular.CssOptimizerPlugin
<i> styles.d251c5bf54715558.css restored from cache.
<t> optimize css assets: 12.149169 ms

Should this use target: rc? Otherwise it will be present on 13.3.x and 14.1.x but not on 14.0.x, which doesn't seem desirable here?

During the RC phase target: patch will merge into all three branches.

Ah oh, thanks @clydin for the clarification.


docs(v2): Wrap section about custom domains in :::info

Motivation
Yesterday I reported #3889 because I believed that a bug in Docusaurus was erasing my custom domain from the repository settings on each deploy. Turns out the issue was as simple as the fact that I forgot to add a CNAME file, I for some reason completely missed the section telling to add CNAME at the root of the static/ directory. That being said, I believe the issue could have been avoided if this section was put more in evidence, hence the idea of wrapping the section in an :::info block.
Have you read the Contributing Guidelines on pull requests?
Yes
Related issues
#3889

thanks 👍


Check the RR bib

Action
Go over the .bib and make sure the newer data sources are included, because I forgot about this part of the process entirely for a while.
Context
References in the report are managed in a bib file, but it needs to be kept up to date.
Effort:
3

I did check this a while ago, but I only checked to see if any reference in the text existed in the bib (they all did).  There are likely sections  that should have both a bib entry and reference in the text, these are a little trickier to find though.


Automatic shutdown is not working

In the cetiTagSensors.c file (at or about line 483) the attempt to gracefully turn off the Pi when the battery goes below critical is not working as intended.  A return value of -1 (error) comes back.  The design idea is to just shutdown the Pi if the program arrives at this point, but something is blocking the attempt.
    CETI_LOG("updateState(): Battery critical, halting");
    system("halt");
    break;

The device just keeps going.  Eventually the battery reaches 5.2 V at which point the hardware undervoltage protection engages.  There is no evidence that the Pi tries to reboot on any residual charge or oscillation of the UV condition, however running the battery all the way down is not desirable in general.

This bug manifested on V2.1 Tag 3 - the tag discharged all the way down to the hardware UV safety.  The protection would not allow charging initially, but was able to recover by applying a voltage to VMID and got it to start taking charge.  One of the cells had gone all the way to 1.9V.   This was fortunate...

2.1-4 addresses this by using the FPGA and a new monitor program that samples the battery voltage once per minute.  If the voltage drops below the critical threshold (currently 3V), the monitor sends a command to cetiTagApp via named pipe.  This command arms the FPGA to shutoff if it detects a GPIO flag from the Pi going low.  Then the script powers the Pi down with shutdown -P +1.


Cannot install brackets without "dependencies

Prerequisites

[ ] Can you reproduce the problem with Debug -> Reload Without Extensions?
[ ] Did you perform a cursory search to see if your bug or enhancement is already reported?
[ ] Did you read the Troubleshooting guide?

For more information on how to write a good bug report read here
For more information on how to contribute read here
Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What you expected to happen]
I wanted to install brackets on my computer.
Actual behavior: [What actually happened]
Error message that says that my computer needs some sort of dependencies (I am using ubuntu studio 20.04)
Versions
1.14.1-17754
Please include the OS and what version of the OS you're running.
Please include the version of Brackets. You can find it under Help -> About Brackets (Windows and Linux) or Brackets -> About Brackets (macOS)

FOund out that you can install it using snap install brackets.


Update for 0.47, removed createJsModule()

This is the breaking change from 0.47.0 onwards.

I have merged another request for 0.47.


PartiQL

PartiQL statements have to be passed as strings, and all parameters have to be passed as attribute values.
It provides a low-level implementation that returns a response containing Documents.
There is also a way to map the response directly into classes to skip the attribute allocations.
If anything needs to be added or modified, please give me feedback, and I'll make the necessary changes.

Thanks for your contribution @ChrixApp! I will try to review and test it over the weekend.

The PartiQL API has been removed from the high-level API.
The BatchStatementError class has been removed.
List has been replaced by IReadOnlyList.
NextToken is now included in the ExecuteStatementRequestHttpContent.
NextToken, Limit, and ClientRequestToken are now set to nullable.

I've made the necessary changes based on your feedback, @firenero

Hi @firenero , are there any additional change needed?

Thanks, @firenero! It was a pleasure working with you as well.  We still have the high-level PartiQL implementation pending and I was wondering when you plan to make the next release so I can plan my projects and use low-level PartiQL.

@ChrixApp can you use pre-release version? If yes, PartiQL is available in 0.9.17-alpha.0.1. I was thinking about completing my work for #251 to bundle both features into single release. It might take a week or two because I have limited availability now. But let me know if you absolutely can't use pre-release version and I'll release stable with only PartiQL change early next week.

I can use the pre-release version 0.9.17-alpha.0.1, thanks @firenero


[docs] Fix 'Customize Compiler Options' example in the 'TypeScript Support' topic

https://devexpress.github.io/testcafe/documentation/test-api/typescript-support.html#customize-compiler-options
Update example as follows
{
    "compilerOptions": {
        "jsx": "react",
        "jsxFactory": "myFactory",
        "alwaysStrict": true
    }
}

because, at present, TestCafe doesn't support the paths and baseUrl options.

Changelog too - https://github.com/DevExpress/testcafe/blob/master/CHANGELOG.md#v130-2019-7-2


[QUESTION] Use Cache when starting Kodi

Every time I open Kodi it says the Elementum daemon is starting and the catalog begins loading and takes some time to finish.
Is there a way to use the cached data and not update the catalog until I do it manually?
I wasn't sure if it was Elementum or TheMovieDBHelper which updates everythin at the start of Kodi but I tried the option in Elementum to delay the start, and when I did then no Catalog is loaded whatsoever so I think it has to do with it.
Checking the log when starting Kodi these lines appear:
`
2024-05-11 17:28:14.099 T:27275   DEBUG : CDirectoryProvider[plugin://plugin.video.elementum/movies/popular]: refreshing..
2024-05-11 17:28:14.099 T:27275   DEBUG : CDirectoryProvider[plugin://plugin.video.elementum/movies/top]: refreshing..`
Thanks.

@leverageinventions elementum does use cache :-) otherwise it would be super slow to get all your data every time, like content of all your lists, all watched movies/episodes and etc.
but every launch and by timer it also reads data from trakt (if enabled) to see if anything changed (like you added something to a list or watched something) and updates internal elementum's database and (if enabled) it also writes .strm files and then launch update of kodi's library.
so before making any changes to code, can you please do some tests?
please do them 1 by 1, not simultaneously.

in trakt tab, disable the setting https://github.com/elgatito/plugin.video.elementum/blob/d06801d6f2682ef8a8d31e9ba77a3149b4b18aaa/resources/language/messages.pot#L1845
relaunch kodi, see if elementum launched faster in your opinion. enable the setting.
in library tab, disable 2 settings
https://github.com/elgatito/plugin.video.elementum/blob/d06801d6f2682ef8a8d31e9ba77a3149b4b18aaa/resources/language/messages.pot#L1789
https://github.com/elgatito/plugin.video.elementum/blob/d06801d6f2682ef8a8d31e9ba77a3149b4b18aaa/resources/language/messages.pot#L1841
relaunch kodi, see if elementum launched faster in your opinion. enable the settings.
disable all 3 settings, relaunch kodi, see if elementum launched faster in your opinion. enable the settings.

and then write results here.

@antonsoroko thanks for the insights. Here are my findings:

For the trakt synchronization setting it didn't seem to have improved the loading times. It was on but I don't use trakt for my own lists, i don't have it even configured or have an account, I just use it to get the top/popular movies/shows lists, so maybe that's why this setting didn't do anything.
For the library settings I already had the library integration disabled. The library synchronization setting was enabled because I thought that disabling library integration would disable the other library synchronization processes. Disabling library synchronization seems to have helped a little bit with the start up times. However, it could be just placebo since I don't have anything in my kodi library, I just use kodi for streaming with Elementum, I don't have local media library.
I left all 3 setting disabled because I dont even use a trakt account or kodi library.

Now my question is the following, is there a way that Elementum uses the caches BEFORE it looks for changes in the lists? So that when I start Kodi the movies and shows are already populated instead of being an empty canvas?
And another question, does Elementum does the triggering for the lists updates or the triggering of the updates is done by kodi or the skin or skinsettings?
Thanks for your help.

@leverageinventions

I don't have it even configured or have an account, I

then it will not do any sync in background or during startup! i really mean ANY.
from your initial message i thought that you heavily use trakt, like many of us: huge watchlist, many own lists, collection.
only those 3 things are synced!
looks like you thought that we get info about public list during kodi start:

I just use it to get the top/popular movies/shows lists|

but these lists are not synced/cached in background/startup. their content is gathered only when you open them.

Now my question is the following, is there a way that Elementum uses the caches BEFORE it looks for changes in the lists?

as i said, public list content is gathered only when you open them.
then we cache it for
So that when I start Kodi the movies and shows are already populated instead of being an empty canvas?

JFYI: on my quite old home pc ( Intel NUC Kit BOXNUC8i5BEH2 ) elementum is readly immediately if w/o tratk+library and it is still immediately ready with trakt+lib (i mean i do not see difference).
btw

I tried the option in Elementum to delay the start,

it simply means "wait X seconds and only then start elementum". thus obviously you can't open elementum immediately.
to give you background info:
elementum consists of 2 parts (so called server-client architecture):

python part. it is a kodi plugin. used for UI and for communication with kodi. client in terms of server-client architecture.
golang part. a binary. used for communication with trakt/tmdb/other web sites and for download torrents. server.

so kodi launch python plugin, and then plugin launch binary.
when binary is ready it shows you notification. you can disable that notification if you want.
if device is slow - then you can see notification that elementum is still starting.

Thanks for the explanation. I have been seeing a little of the code since I have been debugging this issue, mainly the navigation.py file which seems to be the one that gets and processes all the calls.
You got it exactly right when you said the following:

but if you added some list from elementum to your kodi's main page - then maybe your skin will open that list, consequently elementum will open it (and maybe read data from scratch if list was changed).

Basically I have 3 pages with 2 to 3 lists of top/popular movies/shows on Kodi's main page which are part of the interface. So, when Kodi starts the skin loads the lists as you said (automatically since they are in the main page) and shows all the posters when the data is ready, then I can just enjoy and browse and use the torrenting part.
The problem is that it takes some time to show everything, from the kodi launch to elementum checking/updating the lists. In the meanwhile I only see an empty canvas with loading indicators until Elementum gets the lists to be displayed. This process happens every time I open Kodi since it is an android tv box and not a dedicated kodi system.
I have been debugging and reducing the overhead by disabling/uninstallng things I don't use in Kodi but still there is some wait for the canvas to be populated with the posters. That's why I wanted a way to show the cached data instantly and later (when elementum was ready or manually) to update that data.

@leverageinventions
i see. but i do not see a good way to do what you want.
if we will not update list when user opens list - then how/when we should update list content?

by open settings and click new button called "invalidate lists cache"? this is counter-intuitive and and extremely inconvenient.
or by adding menu item for list like "update list content"? this is again counter-intuitive.

i mean, waiting 1 week (or any other amount of time) before update cache is not ok. this is not what i as a user want to have, i want to have accurate info, not outdated info.
it is possible to implement an option like "never update list content cache until cache is expired or user asked for update", but personally I do not know how valuable this would be. and again, to update cache you need to implement either button in settings or menu item.
anyway, let's wait for @elgatito opinion about this.
(btw, if you know some coding - feel free to add such option  :-D )

Well, for a regular user I think the way it works right now should be the default. I was not thinking on changing that behavior but in adding it as an option, like the ones you gave me at the beginning (don't synchronize kodi library, don't synchronize trakt).
I was thinking it could work like this: an option in settings to use caches on the start of kodi and update while on playback. That's it.
About the coding stuff, I know basic coding, I am also looking to see if I could just add that but I am missing some stuff, for example I don't know how to extract the lists from the caches. Currently what I was thinking was in adding a condition in the navigation.py code so that if the url is for example "plugin://plugin.video.elementum/movies/popular" (which is one of the lists I am using) and kodi or elementum have just started then immediately send back the list in the cache.

@leverageinventions

I was thinking it could work like this: an option in settings to use caches on the start of kodi and update while on playback. That's it.

please elaborate. i do not see a way to do that, apart from ways i described above.
what do you mean by playback? we open list, not play it.
and there is not difference between if skin opens list and if you open list. same action.
cache is in golang part, so unfortunately to make any real fix you need to change golang part and compile it.

Well I just meant to add an option in settings to delay (or turn off) the update of the lists like there are options already to turn off tract or library synchronization. So the default behavior would continue to be as is but this option will allow other users to delay it.
By playback I just meant what this setting does:
https://github.com/elgatito/plugin.video.elementum/blob/d06801d6f2682ef8a8d31e9ba77a3149b4b18aaa/resources/language/messages.pot#L1849
Got it, I don't know golang just some basic python so I cant go there, unfortunately.
Thanks for the insights, now at least I know what does what and what can and cannot be done.

"Enable synchronization while playback"

It means that if you do not disable it then we do background synchronization even if you play video which means that on very slow device video can stutter.

like there are options already to turn off tract or library synchronization

Those options are about the background synchronization, but for public lists we don't do background synchronization, as a I already mentioned.

allow other users to delay it.

I do not see a way to delay update (because there's is nothing to delay, we do not do background update for public list), only to disable update at all and then you have to manually update every list by clicking some button/menu (like I described above).
Probably it is possible to create some hack, like if Elementum started less than 1 minute ago and user opened list then do not update this list even if list was changed on trakt side. But this dirty hack for a very specific situation.


Independent drop list on 1 page

Hi, is it possible to make independent drop lists on 1 page? I mean to make 2 separate groups and do not allow to move items from one to another.

I found. [dndList]="{ allowedTypes: ['TypeName'] }"


[Duplicate of #6277]  Creating Dashboards in React using React Dashboards

Formatting and structure checks

[X] Have you used the correct folder and file structure? - see Contribution Guidelines

[X] Is your article properly formatted in Markdown? - see Github Markdown guide

[X] Have you used our preferred heading structure? - H3 (###) and up

[X] Does your article follow any of the suggested structure formats? - see example formats

[X] Is your article over 750 words? - Tool: Wordcounter.net

[X] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[X] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[X] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[X] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[X] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[X] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[X] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[X] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[X] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[X] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes
This closes #6277

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@M0nicah the article should be good now🤞

hi @Pato-254, kindly add an author page. It should contain details like your full name, a professional photo of you and at least 3 lines describing who you are.
Ping me when you are done!

@M0nicah Hello, I added the authors page

🚀 Awesome article @Pato-254  , please review the changes I made in the commits to help improve the flow and format for your future articles.
Thanks!


add version check to ci

Codecov Report

Merging #62 (8ca54fb) into develop (a462c2c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #62   +/-   ##
========================================
  Coverage    92.98%   92.98%
========================================
  Files            9        9
  Lines          442      442
  Branches        65       65
========================================
  Hits           411      411
  Misses          14       14
  Partials        17       17

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 a462c2c...8ca54fb. Read the comment docs.


Rework _.template with ideas from doT

The print function is no longer declared as a function.
It's just a regex inside evaluated blocks of a template.

The with statement has been removed. Inside a template,
the data object can be referenced with o:
var hello = _.template('Hello <%= o.name %>!');
hello({name: 'Ponce de Leon'}) //=> 'Hello Ponce de Leon!'

Now uses string concatenation instead of Array.join.

As a result, it's now much faster. The hello template from the
example above now benches at ~20,000,000 ops/sec on my machine,
where the current _.template got ~600,000.
It's not quite as fast as doT (which gets about 30,000,000) on
the same template, but that seems to be mostly due to the
overhead of wrapping the template function. In any case, this
version should still be a lot faster than most anything other
than doT.
NOTE: I've modified all the template tests so that they pass,
and added one for the print function.

It looked interesting, so I took a stab at Issue #483. Curious to see what other think.


Problem with the new front end

I just found that LJNodes mess with the new search bar. When you double clic on the UI for use the search, it open both at the same time.

Yes, I was just about to come here and say the same. Would love for this to be remedied, love this pack otherwise! :)

I can confirm this as well. Just uninstalled and things work now.


util: waitUntilConnected

I've been going through the code and identifying patterns or ways to help clean up the code to make things easier to read and understand.
(Note: I've been working off of the in-progress PR #290)
There are a few tests that require multiple nodes to be connected (ping, pubsub, dht).
Running through this I found this setup code for ping, which uses a method waitUntilConnected, which doesn't actually connect any nodes, but instead just waits around till the nodes find each other (as far as I can tell).
https://github.com/ipfs/interface-ipfs-core/blob/f238e9f0ba8132dba51d3bac1eb491701f505ccd/js/src/ping.js#L38-L52
It appears that it heavily relies on peer discovery, which is dependent on how the node is configured, particularity for js-ipfs-core.
@alanshaw I believe you wrote this code. I believe it was added so that the tests itself did not have to attempt to find and connect to the peers, and instead could rely on the peers already being connected before conducting the pings.
Any reasons to use waitUntilConnected over simply calling ipfs.swarm.connect?

In hindsight I think this was a mistake and I'm going to send a PR to just connect the nodes that need to be connected, if the test is not explicitly testing discovery.
To phrase it another way, we don't want to have to skip tests that are testing features that aren't discovery related.
I've come to appreciate the state of discovery a whole lot more. In windows AFAIK there's no MDNS so waiting could take an indeterminate amount of time while the DHT bootstraps and our peers get registered on it and there's also this weirdness where in 0.4.14+ our spawned nodes never acquire any peers.


Cannot read properties of null (reading 'domNode')

TypeError: Cannot read properties of null (reading 'domNode')
at y.Y in src/vs/base/browser/ui/list/listView.ts:932:13
at y.V in src/vs/base/browser/ui/list/listView.ts:818:10
at y.V in src/vs/workbench/contrib/notebook/browser/view/notebookCellListView.ts:18:9
at y.updateElementHeight in src/vs/base/browser/ui/list/listView.ts:520:8
at l.updateElementHeight2 in src/vs/workbench/contrib/notebook/browser/view/notebookCellList.ts:1180:21
at Me in src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts:2153:15
at doLayout in src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts:2165:4
at <anonymous> in src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts:1464:10
at g.invoke in src/vs/base/common/event.ts:718:17
at n.deliver in src/vs/base/common/event.ts:926:22
at a.fire in src/vs/base/common/event.ts:887:24
at a.fire in src/vs/base/common/event.ts:1060:11
at h.ob in src/vs/workbench/contrib/notebook/browser/viewModel/codeCellViewModel.ts:321:26
at h.layoutChange in src/vs/workbench/contrib/notebook/browser/viewModel/codeCellViewModel.ts:313:8
at set editorHeight [as editorHeight] in src/vs/workbench/contrib/notebook/browser/viewModel/codeCellViewModel.ts:58:8
at new s in out/vs/workbench/workbench.desktop.main.js:1673:52922
at r.j in src/vs/platform/instantiation/common/instantiationService.ts:119:18
at r.createInstance in src/vs/platform/instantiation/common/instantiationService.ts:85:18
at Kl.renderElement in src/vs/workbench/contrib/notebook/browser/view/renderers/cellRenderer.ts:368:73
at U.renderElement in src/vs/base/browser/ui/list/listWidget.ts:1145:13
at y.W in src/vs/base/browser/ui/list/listView.ts:895:13
at y.Q in src/vs/base/browser/ui/list/listView.ts:636:10
at y.splice in src/vs/base/browser/ui/list/listView.ts:538:16
at <anonymous> in src/vs/base/browser/ui/list/splice.ts:17:35
at Array.forEach (<anonymous>)
at t.splice in src/vs/base/browser/ui/list/splice.ts:17:20
at <anonymous> in src/vs/base/browser/ui/list/listWidget.ts:1468:57
at fn in src/vs/base/common/event.ts:1224:13
at l.splice in src/vs/base/browser/ui/list/listWidget.ts:1468:22
at l.splice2 in src/vs/workbench/contrib/notebook/browser/view/notebookCellList.ts:505:9
at <anonymous> in src/vs/workbench/contrib/notebook/browser/view/notebookCellList.ts:401:9
at Array.forEach (<anonymous>)
at l.jb in src/vs/workbench/contrib/notebook/browser/view/notebookCellList.ts:383:23
at <anonymous> in src/vs/workbench/contrib/notebook/browser/view/notebookCellList.ts:343:10
at g.invoke in src/vs/base/common/event.ts:718:17
at n.deliver in src/vs/base/common/event.ts:926:22
at h.fire in src/vs/base/common/event.ts:887:24
at j in src/vs/workbench/contrib/notebook/browser/viewModel/notebookViewModelImpl.ts:231:31
at compute in src/vs/workbench/contrib/notebook/browser/viewModel/notebookViewModelImpl.ts:275:6
at g.invoke in src/vs/base/common/event.ts:718:17
at n.deliver in src/vs/base/common/event.ts:926:22
at h.fire in src/vs/base/common/event.ts:887:24
at <anonymous> in src/vs/workbench/contrib/notebook/common/model/notebookTextModel.ts:260:30
at g.invoke in src/vs/base/common/event.ts:718:17
at n.deliver in src/vs/base/common/event.ts:926:22
at u.fire in src/vs/base/common/event.ts:887:24
at u.resume in src/vs/base/common/event.ts:1042:12
at Ra.applyEdits in src/vs/workbench/contrib/notebook/common/model/notebookTextModel.ts:522:27
at c in src/vs/workbench/contrib/notebook/browser/controller/cellOperations.ts:136:13
at u.runWithContext in out/vs/workbench/workbench.desktop.main.js:1591:21665
at u.run in src/vs/workbench/contrib/notebook/browser/controller/coreActions.ts:262:16
at handler in src/vs/platform/actions/common/actions.ts:618:42
at fn in src/vs/platform/instantiation/common/instantiationService.ts:68:11
at v.k in src/vs/workbench/services/commands/common/commandService.ts:95:46
at v.executeCommand in src/vs/workbench/services/commands/common/commandService.ts:60:17
at lr.run in src/vs/platform/actions/common/actions.ts:472:31
at D.runAction in src/vs/base/common/actions.ts:192:16
at D.run in src/vs/base/common/actions.ts:183:15
at b.onClick in src/vs/platform/actions/browser/menuEntryActionViewItem.ts:148:28
at HTMLLIElement.<anonymous> in src/vs/base/browser/ui/actionbar/actionViewItems.ts:150:10

Go to Errors Site

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Changed stageSize to an object

Changed stageSize to an object with two integer properties: "width" and "height".

See also comment about whether any other fields are required...

Looks good, merging...


Tidying

Tox runs ruff format -diff.
Remove suggestion to run tox -e reformat from readme.
GitHub action automatically applies linting (ruff, trailing-whitespace and end-of-file-fixer) to PRs. Note that a .pre-commit-config.yaml file was added to the repo for running pre-commit. Also, bots must be given read and write permissions to make commits:

This seems to work nicely.
Commit 9556263 was made automatically by the bot which applied ruff to a few files. The branch now passes the tox env quality (specifically ruff format --diff).

Lint GitHub Action fails because it doesn't have read and write permission to make commits:
Error: Error: To https://github.com/poliastro/czml3
!	refs/heads/tidying:refs/heads/tidying	[remote rejected] (refusing to allow a GitHub App to create or update workflow `.github/workflows/workflow.yml` without `workflows` permission)

See top comment to apply the required change to allow the action to make commits.

Reading from the UI, these are the default permissions. "You can specify more granular permissions in the workflow using YAML."
Can't we just change the YAML?

Also while we're at it... is this essentially equivalent to https://pre-commit.ci/ ?


feat: generalize hover state gradients for social icons on homepage ✨

Close #1494
Screenshots
https://user-images.githubusercontent.com/92252895/257058577-4e5726f0-c6c2-42c2-b024-ecff39b2dc07.mp4
Checklist:

[x] I have linked the PR to the correct issue.
[x] I have read the Contribution Guidelines
[x] I have built and tested the changes, and they do not break or show any errors.

@Vaishnavi-Patil2211 Please merge this PR.


VR support - y offset of XROrigin goes to nan

Lyuma's issue was fixed by removing the %APPDATA%\Roaming\groups_4_000 folder


Updating the top level project to cloud-bigtable-client-parent.

I'd like to use cloud-bigtable-client for the new API, so renaming the top level to include the '-parent' extension which is more maven-y.

LGTM. I have a similar change on a throwaway branch.


More manual virtualisation data

Dug up some information around the instance types where Linux Virtualization currently shows "Unknown". Unfortunately I wasn't able to find anything specific on cr1. Sources below.
cc2
https://aws.amazon.com/blogs/aws/next-generation-cluster-computing-on-amazon-ec2-the-cc2-instance-type/

This instance type uses hardware-assisted virtualization (HVM), so you’ll need to choose an AMI accordingly.

cg1
http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using_cluster_computing.html

You can launch CG1 and G2 instances using any HVM AMI.

hi1
http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/storage_instances.html

Using Linux paravirtual (PV) AMIs, HI1 instances can deliver more than 120,000 4 KB random read IOPS and between 10,000 and 85,000 4 KB random write IOPS (depending on active logical block addressing span) to applications across two SSD data volumes. Using hardware virtual machine (HVM) AMIs, performance is approximately 90,000 4 KB random read IOPS and between 9,000 and 75,000 4 KB random write IOPS.

hs1
http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/high_storage_instances.html

HS1 instances support both paravirtual (PV) and hardware virtual machine (HVM) AMIs.

Thanks for your work here!


INTERNAL ASSERT FAILED: Invalid user info response

Operating System version: Fedora 29
Firebase SDK version<EMAIL_ADDRESS>Library version<EMAIL_ADDRESS>Firebase Product: auth

I replaced the user info by '...' for security reasons.
Calling the method admin.auth().getUser('userId'), I'm getting this response from authRequestHandler.getAccountInfoByUid:
{
  "localId": "...",
  "email": "...",
  "displayName": "...",
  "photoUrl": "...",
  "emailVerified": true,
  "providerUserInfo": [
    { "providerId": "phone", "federatedId": "" },
    {
      "providerId": "google.com",
      "displayName": "...",
      "photoUrl": "...",
      "federatedId": "...",
      "email": "...",
      "rawId": "..."
    },
    {
      "providerId": "facebook.com",
      "displayName": "...",
      "photoUrl": "...",
      "federatedId": "...",
      "email": "...",
      "rawId": "..."
    },
    { "providerId": "phone", "rawId": "...", "phoneNumber": "..." }
  ],
  "validSince": "...",
  "disabled": false,
  "lastLoginAt": "1552858679099",
  "createdAt": "1552858670000",
  "customAuth": true,
  "phoneNumber": "..."
}

I started getting this error, after I imported this user using the method admin.auth().importUsers([user]) with the following data:
const user = {
      uid: '...',
      displayName: '...',
      email: '...',
      emailVerified: true,
      metadata: {
        creationTime: 'Sun, 17 Mar 2019 21:37:50 GMT',
        lastSignInTime: 'Sun, 17 Mar 2019 21:37:50 GMT'
      },
      phoneNumber: '...',
      photoURL: '...',
      providerData: [
        {
          uid: '...',
          displayName: undefined,
          email: undefined,
          photoURL: undefined,
          providerId: 'phone',
          phoneNumber: '...'
        },
        {
          uid: '...',
          displayName: '...',
          email: '...',
          photoURL:  '...',
          providerId: 'google.com',
          phoneNumber: undefined
        },
        {
          uid: '...',
          displayName: '...',
          email: '...',
          photoURL: '...',
          providerId: 'facebook.com',
          phoneNumber: undefined
        }
      ]
    };

I don't know why the API is returning this empty 'phone' provider info:
providerUserInfo:  [{ "providerId": "phone", "federatedId": "" }, ...]

It is hard to tell. The user seems to have 2 phone providers which should not be possible. Can you open a ticket with Firebase support. We will need to ask you for some sensitive information related to your project and the specified user and this is not the appropriate forum for this.

Please follow up via Firebase Support as mentioned above.

@evertonrobertoauler @bojeil-google @hiranya911 I am facing the same issue - two phone providers are getting appended to the user when I run importUsers.  Was this ever resolved?  Should I open a ticket?


The documentation link on the README is broken

The link in the README file to the documentation gives a 404 error.
Reproduce: Click on the link named "documentation" in README.md

Thanks @brenmcnamara.  This will be resolved as soon as the Auk docs are merged to master.  We'll leave it as is, for now, but the correct link until then is https://auk.docs.feathersjs.com/api/authentication/hooks.html

Sounds good! Thanks for the quick response :)


Allow themes to change the appearance of the About window and the Options window.

A possible codename for a version of the app launcher that includes this change is "Codename Lightyear". This is because themes would be applied everywhere in UXL Launcher, except for the file browser dialogs and messageboxes.

About window themes can just impact the controls directly if there are few enough, and for Options window, this may be helped by going through the controls like in the main window.

This is mostly functional now in the dev branch. These changes won't be merged into master until after Version 3.2 is out.
All that themes need to have done now is have their Theme Engine version compatibility thing upgraded to "1.03", and (optionally) apply theming to the controls now supported in TE 1.03. See TenDark for an example including support for the additional controls used in the Options and About windows. Eyesore2 can be used as an example test theme and includes support for these controls.

This has been merged into Master. Closing issue.


MergeDelete was not found in MergeTablesToSync

INFO DoSync(). Table dbo.MergeDelete was not found in MergeTablesToSync.
Always have this message in sync.log
Not sure if it's normal

MergeDelete should not be synced. It is local table created automatically on mobile.
What client do you use (I mean Android, iOS or maybe HTML)?

Ionic in Android

Fixed fd79474


Revert "SCT-1147 Fix get namespaces bug"

Reverts jgi-kbase/IDMappingService#104

Pull Request Test Coverage Report for Build 224

11 of 11 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.3%) to 95.027%

Files with Coverage Reduction
New Missed Lines
%

src/jgikbase/idmapping/storage/mongo/id_mapping_mongo_storage.py
1
87.67%

src/jgikbase/idmapping/builder.py
1
62.5%

Totals

Change from base Build 222:
-0.3%

Covered Lines:
1051

Relevant Lines:
1106

💛  - Coveralls


add code coverage for retrying tests (buildtest build --retry)

Add code coverage for buildtest build --retry. See https://app.codecov.io/gh/buildtesters/buildtest/blob/devel/buildtest/builders/base.py

@Mendi03 here are few hints for you to get started
The test method run will return if exit code is a 0 or when its run on a local executor because non-zero exit code tests should stop executing as pose to keep running again.
https://github.com/buildtesters/buildtest/blob/6b6277ce0429acb1b0edd332e253233c24789745/buildtest/builders/base.py#L378-L379
For instance if you look at this example test https://github.com/buildtesters/buildtest/blob/devel/tutorials/test_status/pass_returncode.yml simply running a exit 1 will cause test to fail. That being said we do need the or self.is_local_executor() in the if condition otherwise it will keep retrying.
I propose we change this if condition such that its something like so that we return immediately if buildtest build --retry is not specified and the section below would be logic for retry.

if not self._retry or ret == 0:
    return command

We should also change this line from 1 to None as the default.
https://github.com/buildtesters/buildtest/blob/6b6277ce0429acb1b0edd332e253233c24789745/buildtest/builders/base.py#L111.
There is a retry method that is used to get the value from the command line in this method
https://github.com/buildtesters/buildtest/blob/6b6277ce0429acb1b0edd332e253233c24789745/buildtest/builders/base.py#L326-L327
In the main build.py for every builder we pass the value of the command line buildtest build --retry <NUM> to each builder if that makes sense.
https://github.com/buildtesters/buildtest/blob/6b6277ce0429acb1b0edd332e253233c24789745/buildtest/cli/build.py#L1123
What this means we should expect the following

If we invoke buildtest build without --retry the test should run once regardless if test pass or fails
If --retry is specified we should only retry test that fails (i.e non-zero returncode) X number of times.

To add the regression test i would suggest you do the following

[ ] Update the logic in https://github.com/buildtesters/buildtest/blob/devel/buildtest/builders/base.py to change the way retry works
[ ] Add test method in class TestBuildTest to https://github.com/buildtesters/buildtest/blob/devel/tests/cli/test_build.py


lapack: add Dtgsyl and its test

Seeking to solve https://github.com/gonum/gonum/issues/1651.
At the time of making this PR panics on Dlaset due to OOB access. I can't seem to decipher this one and could use a little help.

Closing in favor of https://github.com/gonum/gonum/pull/1891


Bundle size increases with over 16 KB gzipped when only including a single SvgIcon

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
I included a single svg icon like so:
import EditIcon from '@material-ui/icons/Edit'

In a standard (non ejected) create-react-app project. The gzipped size increased with 16.25 KB.
This is the @material-ui dependencies as reported by source-map-explorer

Expected Behavior 🤔
The @material-ui/icons weight is only 614B and while I could expect some extra overhead, this is definitely more than that. The bundle includes a bunch of color pallets, transitions and shadows that are definitely not used.
Steps to Reproduce 🕹

Steps:

Create a new create-react-app project
Run yarn build
Install @material-ui/core and @material-ui/icons
Add and use the import import EditIcon from '@material-ui/icons/Edit'
Run yarn build and see the size diff in the output

Context 🔦

While a single icon might be an extreme case, it does demonstrate that material ui does pull in a bit too much of it styling helpers even when they are not used. While not as extreme, this does affect single or just-a-few component usage from @material-ui/core as well.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.5

React
16.12.0

Browser
N/A

TypeScript
3.7.5

react-scripts
3.4.0

It is using SvgIcon from /core. Which has some props to e.g. color it in primary or secondary colors. This means it's including the styling solution which is quite heavy.
It seems like you make no use of that which is why you're probably interested in #6218.
Otherwise this is expected since the styling solution is quite heave. The increase should be smaller if you're already using jss.


HIVE-25277: fix slow partition deletion issue by removing duplicated isEmpty checks.

HIVE-25277

What changes were proposed in this pull request?

Deleting a Hive partition is slow when use a Cloud object store as the warehouse for which ListFiles is expensive. A root cause is that the recursive parent dir deletion is very inefficient: there are many duplicated calls to isEmpty (ListFiles is called at the end). This fix sorts the parents to delete according to the path size, and always processes the longest one (e.g., a/b/c is always before a/b). As a result, each parent path is only needed to be checked once.
Why are the changes needed?

It improves partition deletion time. The improvement is around 7X in our test.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Tested partition deletion end-to-end.

@kgyrtkirk may you take a look and merge this PR?

Zoltan Haindrich Haymant Mangla Naveen Gangam may you take a look and merge this PR?

Thank you @medb! @kgyrtkirk @hmangla98 @nrg4878 a friendly ping, could you please take a look? Thank you!

@sunchao help take a look?

Thanks Zhou Fang . LGTM with two nits.

Thank you Chao for the helpful comments.

Merged. Thanks!
BTW if you want this in other branches, please open backport PRs against them accordingly.


Route deletion logic incorrect?

When deleting a route with the current logic, it seems the route will be deleted from the index, but not from the route_layer.routes list? If so can we fix:

Rereading, does not seem to be an actual issue


Should imshow() recognise 0-255 images?

Code Sample, a copy-pastable example if possible
import os
import urllib.request
import xarray as xr
import matplotlib.pyplot as plt

# Download the file from rasterio's repository
url = 'https://github.com/mapbox/rasterio/raw/master/tests/data/RGB.byte.tif'
urllib.request.urlretrieve(url, 'RGB.byte.tif')

# Read the data
da = xr.open_rasterio('RGB.byte.tif')

f, (ax1, ax2) = plt.subplots(1, 2, figsize=(9, 4))
da.plot.imshow(ax=ax1)
(da / 255).plot.imshow(ax=ax2)
plt.tight_layout()
plt.show()

# Delete the file
os.remove('RGB.byte.tif')

Problem description
In https://github.com/pydata/xarray/pull/1796, @Zac-HD added support for RGBA images. If an alpha channel is not found, it is added (code)
The problem is that adding this alpha channel requires the images to be normalized to 0-1, while plotting an image in 0-255 range without alpha channel works fine in matplotlib. Removing https://github.com/pydata/xarray/blob/master/xarray/plot/plot.py#L708-L715 would solve the problem, but I guess it was added for a reason.
@Zac-HD , thoughts?

Ouch, this is definitely a bug in the code I wrote - that's terrible 😭
Probably related to matplotlib/matplotlib/pull/10220, where I'm looking at clipping instead of wrapping colours from the other side.
Fixing this issue may be as simple as setting alpha to 255 instead of 1 for integer dtypes, or it might be more involved.  Either way I'll try to open a pull within the next few days.


[ Feature request ] Get mod who banned

Add the mod (mod id) in for example "event_raw_data".
Are you using TwitchIO within a Discord Bot?
No
What commit of TwitchIO are you using?
pip show twitchio command somehow doesnt work
its version 1.1.0 installed through pip

This is on the project board for 2.0 ( #123 )

This is on the project board for 2.0 ( #123 )


Feature Suggestion: Multiple Lib Folders

Hi there,
I was wondering if it'd be not too much of a stretch to implement this, perhaps in the context of a filesystem datasource. Currently nanoc supports a layouts_dir and content_dir per filesystem datasource. Perhaps it can also support a lib_dir in the same way, so that lib files would be searched in every lib_dir specified?
If someone can point me into the right direction in the code, I could try to implement this myself and issue a PR.
Thoughts, anyone?
Thank you.

Nanoc has a lib_dirs setting already.
lib_dirs:
  - foo/lib
  - lib

Helpers in foo/lib? will be overridden by helpers in lib?

Thanks! Good to know. I actually almost expected it, but since there was no mention of it in the documentation...
I was looking for a way to allow a "filesystem" to no only provide layouts and items, but also bring along any helpers that may be used in them.
How does the overriding take place? Is that on a file name basis, or method basis? I'm guessing it's the latter, and if there is a file named default.rb in both lib/ and foo/lib/, they will both be loaded and methods will be monkey-patched, right?

I just answered my own question:

Code is loaded from each folder in lib_dirs in sequence
Monkey-patching rules apply

Wonderful!  👍

Whoops, lib_dirs being undocumented is not intentional. I’ve opened a ticket on nanoc.ws: https://github.com/nanoc/nanoc.ws/issues/221


gitbutler: use pnpm directly to build

title + some deps have been corrected based on official documentation

this works on darwin. proper desktop integration is missing (i.e., icons, not showing up in launchpad) but it does open :)

I will close this following the discussion on NixOS/nixpkgs#289664. Feel free to open a PR of your own in Nixpkgs, though.


GO terms mapped to EC:<IP_ADDRESS>

The following 2 GO terms both have an EC:<IP_ADDRESS> xref:
id: GO:0003853
name: 2-methylacyl-CoA dehydrogenase activity
def: "Catalysis of the reaction: 2-methylbutanoyl-CoA + H+ + oxidized [electron-transfer flavoprotein] = (2E)-2-methylbut-2-enoyl-CoA + reduced [electron-transfer flavoprotein]." [PMID:10989435, PMID:6401712, RHEA:43780]
xref: EC:<IP_ADDRESS>
xref: RHEA:43780
is_a: GO:0003995 ! acyl-CoA dehydrogenase activity
id: GO:0102035
name: isobutyryl-CoA:FAD oxidoreductase activity
def: "Catalysis of the reaction: H+ + isobutyryl-CoA + FAD = methacrylyl-CoA + FADH2." [GOC:pz, PMID:3988734]
xref: EC:<IP_ADDRESS>
is_a: GO:0003995 ! acyl-CoA dehydrogenase activity
===
https://enzyme.expasy.org/EC/<IP_ADDRESS> says:
Name: short-chain 2-methylacyl-CoA dehydrogenase
Reaction: 2-methylbutanoyl-CoA + H(+) + oxidized [electron-transfer flavoprotein] <=> (2E)-2-methylbut-2-enoyl-CoA + reduced [electron-transfer flavoprotein]
There seems to be a mismatch in specificity between the name (short-chain 2-methylacyl) and the reaction (2-methylbutanoyl-CoA) here?
RHEA does map EC:<IP_ADDRESS>  to both reactions:
RHEA:43780 | 2-methylbutanoyl-CoA + H+ + oxidized [electron-transfer flavoprotein] = (2E)-2-methylbut-2-enoyl-CoA+ reduced [electron-transfer flavoprotein]
RHEA:44180 | 2-methylpropanoyl-CoA + H+ + oxidized [electron-transfer flavoprotein] = (2E)-2-methylpropenoyl-CoA+ reduced [electron-transfer flavoprotein]
(Note: 'isobutyryl-CoA' has synonym '2-methylpropanoyl-CoA', and 'methacrylyl-CoA' has synonym  '2E)-2-methylpropenoyl-CoA' in CHEBI)
@kaxelsen Can you comment on this please?
===
Possible tasks (pending Kristian's input):

[ ] Remove EC:<IP_ADDRESS> from GO:0102035
[ ] add RHEA:44180 to GO:0102035
[ ] Update GO:0102035 name and def to match RHEA:44180
[ ] Update GO:0003853 name to '2-methylbutanoyl-CoA dehydrogenase activity' to match GO/EC/RHEA def
[ ] change parent of both GO terms to the more specific "short-chain-acyl-CoA dehydrogenase activity" (GO:0016937) since 2-methylbutanoyl-CoA and 2-methylpropanoyl-CoA both have a chain length of C4.

There seems to be a mismatch in specificity between the name (short-chain 2-methylacyl) and the reaction (2-methylbutanoyl-CoA) here?

No, it is fine. The chains can be branched as long as they are still aliphatic. Regarding the name, the EC <IP_ADDRESS> has the highest activity with 2-methylbutanoyl-CoA but also has activity with other short-chain 2-methylacyl-CoA cf pmid:11013134

Thanks @kaxelsen. Yes, I see in pmid:11013134 that the enzyme acts on both 2-methylbutyryl-CoA (aka 2-methylbutanoyl-CoA) and isobutyryl-CoA (aka 2-methylpropanoyl-CoA), so it should remain an xref on both GO terms mentioned above.

@pgaudet Looking at this again, with the wider aim of not having duplicate ECs on different GO terms, I think we should merge the following two EC:<IP_ADDRESS>-based GO terms:
id: GO:0003853
name: 2-methylbutanoyl-CoA dehydrogenase activity
def: "Catalysis of the reaction: 2-methylbutanoyl-CoA + H+ + oxidized [electron-transfer flavoprotein] = (2E)-2-methylbut-2-enoyl-CoA + reduced [electron-transfer flavoprotein]." [PMID:10989435, PMID:6401712, RHEA:43780]
xref: EC:<IP_ADDRESS>
xref: KEGG_REACTION:R03169
xref: MetaCyc:2-METHYLACYL-COA-DEHYDROGENASE-RXN
xref: RHEA:43780
is_a: GO:0016937 ! short-chain fatty acyl-CoA dehydrogenase activity
id: GO:0102035
name: 2-methylpropanoyl-CoA dehydrogenase activity
def: "Catalysis of the reaction: 2-methylpropanoyl-CoA + H+ + oxidized [electron-transfer flavoprotein] = (2E)-2-methylpropenoyl-CoA + reduced [electron-transfer flavoprotein]."  [MetaCyc:RXN-21774]
xref: EC:<IP_ADDRESS> [exactMatch]
xref: MetaCyc:RXN-21774 [exactMatch]
is_a: GO:0016937 ! short-chain fatty acyl-CoA dehydrogenase activity


When used for LazyList items app got crashed with - Failed to copy pixels of the given bitmap!

When used for LazyList items app got crashed with - Failed to copy pixels of the given bitmap!
Inside sample app you blurred the whole grid instead what I want is to blur individual list item (Basically Image loaded with Coil), How can we do that can you share the sample

You can use this BlurHash and this component to perform that using coil.

@orlandev Wow thanks for help looking awesome

Hi @KothamireNarendra, check out the Blur Effects Depending on States and you can see the sample codes on this project.
You can consider using the Landscapist-Transformation plugin to use the blur effect for your remote images.

I am closing this issue for now.


URI is Comparable

Summary
java.net.URI is a Comparable, so AbstractUriAssert should extend AbstractComparableAssert.
I assume this to be a minor, non-breaking change. I'd do a PR, but the overhead seems to be too big in my eyes.
Note: java.net.URL is not comparable.

Sounds very reasonable to me.
What kind of overhead do you see?
I'd expect test classes to be added for all the methods exposed by AbstractComparableAssert, but they should be trivial like:
https://github.com/assertj/assertj-core/blob/517c9c6fca738cb4d088f125ae5186a3a1e4efac/src/test/java/org/assertj/core/api/long_/LongAssert_isLessThan_long_Test.java#L26-L37

I'd have to clone the project, create a branch, install the Eclipse-Formatting-Plugin, etc. etc. pp. ... that kind of overhead ;-)

Got it and I am happy to help get you onboarded. What you mentioned is sort of prerequisite for any contributions 🙂

Thanks for the offer. I know how to do all of that. I'll be happy to do this when I have something more substantial to add.

I can work on this.

I'd expect test classes to be added for all the methods exposed by AbstractComparableAssert, but they should be trivial like

Hi, as for test cases, methods of AbstractComparableAssert are already tested in org.assertj.core.api.comparable.AbstractComparableAssert_*_Test, so I don't think there is a need to repeat those.
Your example of LongAssert_isLessThan_long_Test is quite different as it tests isLessThan(long) which is not defined in AbstractComparableAssert (it comes from AbstractLongAssert).

@pbacz you're right, I overlooked both points. Thanks for the PR!


Node process is not attached.

Node processes pane just empty with "No running nodes".
Steps to reproduce
Tell us about your environment:

ndb version: 1.0.19
Platform / OS version: Ubuntu 16.04
Node.js version:10.7.0

What steps will reproduce the problem?
Start ndb with the node js script
Please include code that reproduces the issue.

What is the expected result?
To attach to the node process
What happens instead?

Thanks for your report!
Could you provide more details about your script? Does ndb work for other scripts? Is it possible to evaluate any expression from ndb console?

I will reopen this one as soon as we will get more feedback. Unfortunately we do not have enough information to investigate this problem. 😞

I'm seeing this as well. I've tried launching ndb from a local install, as well as globally, with no difference. I can't get it to connect to anything.
It launches the devtools panel, and after a delay, I see the local directory appear in the Filesystem tab. But nothing appears to be running. The console output is entirely empty, and the dropdown that normally lets you pick which frame's output to view in the console pane is just empty:

The terminal panel just prints out a couple of node inspector URLs initially. If I Ctrl-C, it seems to kill whatever was running there, but then trying to launch a node process from there also does nothing (besides just print out those debug URLs and hang as well):

The NPM Scripts pane is correctly populating as well, but clicking on the "Start" button on any of the tasks does nothing.

macOS 10.14.1
ndb 1.0.35

I dunno if this is relevant, but I just noticed a couple dozen node processes sitting around, despite me not running anything currently. It's possible that my main app's nodemon setup is botched and is leaving around server processes on restarts, or perhaps it's from ndb. Not sure, but figured I'd let you know just in case.

That seems to do it. I can now start the node process and it attaches correctly.
FWIW though - I can't seem to use ndb in anything other than a direct invocation of node, i.e. ndb node .... I've tried ndb nodemon ... and ndb yarn ... and it fails - it seems like it captures any command line args and passes them straight to node, rather than preserving them with the original binary invocation.
Happy to file a separate issue for that if you'd like.


Refactor

Refactor NodePool controller
This PR is a first step to refactor the NodePool controller:

Encapsulate subsets of logic into cohesive funcs to be unit tested.
Reduce the lines of reconcile() for readibility.
Similar to the HostedCluster controller it splits reconcile() (and fail early) into 1 - Reconcile conditions with current state of the world, 2 - reconcile towards expected state of the world.
Fixed left over Failure conditions which are now cleaned up.
Fixed RemoveStatusCondition, it was broken so it's duplicated until we update API machinery dep.
Fixed logging to be consistent and not be redundant.
Fixed an issue with MachineSets were failing to scale down during Rolling upgrades triggered by version updates that requierd a new bootstrap AMI and generated a new awstemplate.
Pass MachineTemplate in a provider agnostic way to reconcileMachineDeployment.

Follow up:
tests, tests, tests...
/hold
needs
https://github.com/openshift/hypershift/pull/320
https://github.com/openshift/hypershift/pull/331

/test e2e-aws

/lgtm
/test e2e-aws

feel free to remove hold if ready to merge ... previous e2e run passed, just failed in gathering cluster data

/hold cancel


Model training won't start

Trying to train on latest release, nothing happened: got this info in console

Same. I'm stuck at the "all-feature-done" message and my system is completely idle. Sometimes it'll suddenly start training and sometimes it just won't. I'm not sure what it's waiting for.

@KCGD metoo, have you solved it?


Update the Schedules API implementation

The Schedules API implementation needs to be updated to support the changes in the underlying Bridge API.
Attention is required to times and how they re specified in the schedule as this area has changed a lot.

Resolved in 2cc295fe9ac1d513fff020bedb8c1239691219f2


Empty placeholder is returned when cover image presence validation fails

Occurs with home page and work page featured items

Given we're not enforcing the presence of a cover image for Posts and Projects, this is no longer an issue.


ParameterBindingArgumentTransformationException when running PSWriteExcel

At the part of the script when it begins to write to an Excel portion I get this error:
Save-WordDocument : Cannot process argument transformation on parameter 'WordDocument'. Cannot convert the
"System.Object[]" value of type "System.Object[]" to type "Xceed.Words.NET.Container".
At C:\Program Files\WindowsPowerShell\Modules\PSWinDocumentation\0.1.0\Public\PSAD.ps1:49 char:53

$FilePath = Save-WordDocument -WordDocument $WordDocument `

CategoryInfo : InvalidData: (:) [Save-WordDocument], ParameterBindingArgumentTransformationException

FullyQualifiedErrorId : ParameterArgumentTransformationError,Save-WordDocument

Issue is with PSWriteExcel version 0.0.10
The problem goes away when I run PSWinDocumentation script without Export to Excel enabled and also if I uninstall the PSWriteExcel module and reinstall version 0.0.3

You may want to give it a try with new version.

Sorry for the late reply!  New version works like a charm!  Thanks!

Thats great!


gpt-4o-mini support

Love this project, thank you. Is gpt-4o-mini support on the roadmap?

OK, gpt-4o-mini & gpt-4o-mini-2024-07-18 Added in v1.3.7


Add built-in support for time.Time

Hi,
Following on https://github.com/mohae/deepcopy/issues/2 we added support for time.Time in our fork.
We first tried and make it a bit more general, like querying a 'Copier' interface on values to see if the user wanted to hook the copy, however given the lack of overloading in go and the fact that a member function cannot be added outside of the package of a type, it wasn't very useful for time.Time.
Alternatively we could query for the encoding.BinaryMarshaler or gob.GobEncoder interfaces but this will likely change the semantics of deepcopy to copy way more than expected by the user.
Thanks!

Hi,
Thank you for your pull request, I appreciate it.
As you may have noticed in issue #2, I wasn't sure about adding support for time.Time because that would raise the issue of what other types should have custom logic when deep copying structs.
However, this is something I'd be willing to merge if it addressed the issue at hand: unexported time.Time data not being copied.  Unfortunately, it does not appear that this pull request accomplishes that.  https://gist.github.com/mohae/b3078498205b314daede9e98242a1dc6 is a link to the test I wrote to validate the change (my main concern was validating that the TZ info was being correctly copied).  I'm expecting to be able to get the same information out of the original time.Time and the copy.
In looking at the source for time.MarshalBinary and time.UnmarshalBinary, it appears that your patch should work as expected.
It may be that the tests I wrote are invalid for some reason and/or I'm misunderstanding something here.  I hope to be able to take a closer look this weekend.
If you have any additional insight, it'd be appreciated.

Hi again,
It looks like it has already been reported as a bug in golang e.g. https://github.com/golang/go/issues/15716

Thanks for the link.
I think I'm going to hold off on merging this until it's verified as fixed in 1.8.
Also, the addition of tests, like what I linked to, to verify that all of the info was correctly copied would be helpful, but it won't hold up the merging of this pull request as I can just copy my test in.

Actually, thinking about it, I'm going to merge and add a note to the README.
Thanks, I appreciate your contribution!


Support mongo+srv URLs

Hey folks, as far as I can tell this connector doesn't support using mongo+srv style hosts. This is a problem if using services - such as MongoDBs hosted Atlas service - which publish their addresses as SRV records.
One simple solution would probably be to allow specifying a connection string and checking if it's present in the config then using the MongoClientURI based constructor for MongoClient?

Thanks for the suggestion. We could probably change the settings to accept a URI instead of the current host/port. Conveniently, this URI also encodes the username/password if any.

Should be fixed in the dev branch.


[Text input] Add ability for text input to let platform can unfocus.

List which issues are fixed by this PR. You must list at least one issue.

https://github.com/flutter/flutter/issues/123523

Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

(triage fly-by): @justinmc looks like Google Testing is unhappy with this one.

Seems that connectionClosed can do this thing. I will have a check. If that functions works, I think I can close this PR and just use that function^_^

Tested that connectionClosed can do this thing.

Reopen.
Context: https://github.com/flutter/flutter/issues/123523#issuecomment-1489872260

@LongCatIsLooong would you mind share your thoughts on this ^_^ ？

Also not sure why Google testing failed.. I m not googler, @justinmc  would you mind help me on that ^_^ ?

Ah, this is a breaking change for anyone that implements TextInputClient.  They will need to provide an implementation of unfocus.
Can you take a look at the process in the breaking changes guide?  I think we'll need to write a migration guide, even though it should be simple.

Emm why it is a break change? the text input client default unfocus implementation is empty.🤔

Yeah exactly.  Probably people should be using with if they don't want to be broken when a method is added.

I m thinking to reuse the function connectionClosed without any break changes. Because the connectionClosed behavior maybe different on web and mobile.
I will have that try. ^_^

That sounds like it could work too.  Let me know how it goes.

I m going to land https://github.com/flutter/flutter/pull/123929 since we can use exist function to do things.

Mark as closed. ^_^


Dashboard charts have errors for LZ reviews

The dashboard charts have errors for the LZ reviews as the import pulls in a Design Area of 'Application Deployment' that isn't part of an LZ review.
That's creating oddities on the dashboard charts. Example attached.

Thanks @AndrewJNorris , good catch! I sent this PR to fix it: https://github.com/Azure/review-checklists/pull/291


Download dashdb-local

Expected behavior
Actual behavior
unauthorized: authentication required
Information about the Issue
Steps to reproduce the behavior

...MacBook Pro
...Kitematic
New Container, search for ibmdashdb/local:latest or ibmdashdb/local:v1.6.0-kitematic
Click on CREATE

Please make sure that you've validated your
Please make sure that you've validated your account prior to pulling an image; this can be done by doing the following:

Login to cloud.docker.com
Click on Account Settings
Check and verify that your primary email has been verified

Hi,
In my account settings, my primary email is already verified.
But still ...

Vann Lam
IBM Certified IT Specialist
IBM France | IBM Analytics Technical Sales
+33158752608 (phone) | +33673193286 (cell) | +33158752608 (fax) | 38802608
(tie-line)
From:   French Ben<EMAIL_ADDRESS>To:     docker/kitematic<EMAIL_ADDRESS>Cc:     Vann Lam/France/IBM@IBMFR, Author<EMAIL_ADDRESS>Date:   23/03/2017 01:41
Subject:        Re: [docker/kitematic] Download dashdb-local (#2489)
Please make sure that you've validated your
Please make sure that you've validated your account prior to pulling an
image; this can be done by doing the following:
Login to cloud.docker.com
Click on Account Settings
Check and verify that your primary email has been verified
?
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
Sauf indication contraire ci-dessus:/ Unless stated otherwise above:
Compagnie IBM France
Siège Social : 17 avenue de l'Europe, 92275 Bois-Colombes Cedex
RCS Nanterre 552 118 465
Forme Sociale : S.A.S.
Capital Social : 657.364.587 ?

Hi,
Sorry for the delay I was very busy the last past days.
the command : docker pull ibmdashdb/local:latest ended up successfully
What is the right docker command to 'run' to deploy the container from
that image ?
Thanks

Vann Lam
IBM Certified IT Specialist
IBM France | IBM Analytics Technical Sales
+33158752608 (phone) | +33673193286 (cell) | +33158752608 (fax) | 38802608
(tie-line)
From:   French Ben<EMAIL_ADDRESS>To:     docker/kitematic<EMAIL_ADDRESS>Cc:     Vann Lam/France/IBM@IBMFR, Author<EMAIL_ADDRESS>Date:   23/03/2017 20:32
Subject:        Re: [docker/kitematic] Download dashdb-local (#2489)
Are you using a local registry? what happens if you do: docker pull your
image in a terminal?
?
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
Sauf indication contraire ci-dessus:/ Unless stated otherwise above:
Compagnie IBM France
Siège Social : 17 avenue de l'Europe, 92275 Bois-Colombes Cedex
RCS Nanterre 552 118 465
Forme Sociale : S.A.S.
Capital Social : 657.364.587 ?
SIREN/SIRET : 552<PHONE_NUMBER>4 - Code NAF 6202A


Scripts ran outside venv do not use global interpreter

When running scripts with no venv, the interpreter packaged with the .app is used, not the global installation. This is problematic when a user installs packages in scripts without a venv. Module import error will occur.

Fixed

Re-opening, new issue is that global interpreter appears as MenuScript.app/Contents/Resources/lib/python3.11
sys.path does not include user's global interpreter. May requiring installing script in menu files, calling sys.path in that script, and then feeding that variable into a global within MenuScript. Working to implement.


[Bug]: Error: Pushing to : Permission denied (publickey).

Describe the bug
obsidian-git is making the commits successfully but unable to push them. I think it must be a config issue on my side but I'm unsure of what needs to change. My typical workflow for manually using git is to open cmder, and it prompts me for the password to my id_rsa file. Then after that I'm all set and can do git commands. I'm guessing that obsidian-git isn't aware of the password, is there a way to set it so that it can do these pushes automatically?
Relevant errors (if available)
No response
Steps to reproduce

Have an id_rsa file with a password
Try using obsidian git with ssh rather than https connection to remote repository

Expected Behavior
No response
Addition context
No response
Operating system
Windows

Closing this as duplicates of #42 and #167


Individual volumes of multi-vol works showing in search results

E.g.: http://lib-plum1.princeton.edu/catalog?f[generic_type_sim][]=Work&works=mine

I think those are all volumes that failed to ingest completely.  So the ScannedResource was created and some pages were added, but the error failed to associate it with the parent MultiVolumeWork (which is why they show up in the search results).
So I think making the ingest more reliable will fix this, though we could also update the volume ingest code to link the ScannedResource to the MVW before ingesting pages.


ocamlPackages.lambdasoup: 0.6.3 → 0.7.1

Motivation for this change
Fixes & improvements
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocamlPackages.lambdasoup

Builds, looks good.


nixos/acme: Docs, explain how to set permissions

Closes #100276
Motivation for this change
As of 20.09 the /var/lib/acme/.challenges permissions will
not automatically be correct. Add instructions on how to
set them correctly.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

I would like this merged too before #106857


Add percision parameter to fileSize

Sometimes 3 MB is more humane than 3.12 MB. This allows specifying the precision as a parameter of fileSize() instead of having it hardcoded as 2:
Humanize.fileSize(1024 * 2000, 0)
// "1 Mb"

thanks @nathancahill That seems legit to me. Mind adding a test for it?
@b-ash do we still support node 0.8?

0.8 is pretty old, so I'd be okay with bailing on it.

Bump, added a test for file precision.

@nathancahill Sorry for the delayed responses :disappointed: Mind rebasing/re-applying these changes to the new source? I'll be sure to merge it in right away :+1:

Thanks @geekjuice! Submitted #70.


Autocorrection bug in Style/RedundantConditional cop

If "bad" code has elsif, Style/RedundantConditional generates wrong code when autocorrect.
if cond
  foo
elsif x == y
  true
else
  false
end

Expected behavior
Autocorrection generates the following code.
if cond
  foo
else
  x == y
end

Actual behavior
Autocorrection generates the following code actually.
if cond
  foo
  x == y
end

Steps to reproduce the problem

Put the above code as test.rb
Run rubocop -a

RuboCop version
master(ce6483c8f83691837519ab0a4f996a74f9de11dc)

Good catch! I can see this happening with how parser nests the AST for elsifs.


Lets make milestones

@Mosaddeq107 if you are maintainer (or when you do become it) could you start making some milestones

Also, make some labels. It will be useful for our project. @Mosaddeq107

@Mosaddeq107 if you are maintainer (or when you do become it) could you start making some milestones

Yes. I am still not being appointed as a maintainer. Once Dr. Tim does that, I will make mile stones.

Also, make some labels. It will be useful for our project. @Mosaddeq107

Okay, I will make some new labels as well.

Ok. good.


Move Unity3D test runner code to the AWS SDK for Unity repo

This repository contains Unity3D code for running tests (presumably on the aws device farm).
This code is more relevant for the AWS SDK for Unity than this repo.

The AWS SDK for Unity was merged into the AWS SDK for .NET. The older AWS SDK for Unity repository is kept around for historical purposes but future Unity development is done in this repository.

Good to know. Do you know if this SDK allows running tests on the Amazon device farm ?

Can you clarify what you mean? The SDK can be used to make calls against AWS Device Farm, so you could use the SDK to run tests on the service.

What i meant, was that we use Unity3d to make games.
We have some tests written in NUnit and in Unity test tools. all tests are created using C# and should be executed as part of the game's execution.
Reading the documentation for the AWS device farm, it is possible to run JUnit tests or all sorts of other UI testing frameworks. I was wondering if this SDK provides any tools to facilitate running tests such as the ones we use in Unity, on devices in the AWS device farm.

Sorry, no, there are no tools that our teams produces for running tests on AWS Device Farm. You'll need to use the supported test harnesses or write your own.

Closing due to inactivity. Feel free to open if you have more questions related to the topic.


Problem with Quest Tracking

@dragonjet I know the Quest Tracking is still being developed. I just want to see whether you get the same error message like this or not. And the translation is not displayed properly as well.

ah did you just click on "Delete Everything" button on the Strategy room?

I didn't. This has happened since I pulled the code from master. Should I click on it and try again?

whoops, for a moment there I thought you were @Crazyteitoku coz of the pic XD
for now you can try going to all pages of the quests to fill all data. re-visiting the quest page will avoid these errors. I'll push a fix in a few mins.

I clicked on "Everything" button and the translation now is correct but the error message still appears the same.

oh yeah you are right. After doing that, it's not happen anymore xD


Move extern template instantiates to the end of file

Problem
There is a difference in behavior between gcc and clang when it comes to
explicit instantiating extern templates. With clang, instantiating
before all templated member definitions are defined will leave those
members uninstantiated in the translation unit, causing undefined symbol
errors.
Change overview
Move the instantiation to the end of file so these platforms can build
with clang.
Testing
Built NXP lighting-app with clang.

@rgoliver @andrei-menzopol FYI


[Bug]: The fields that cause errors in widgets do not get a red highlight without clicking on them

Is there an existing issue for this?

[X] I have searched the existing issues

Description
We do not get to see the red highlight over the fields in the property pane that cause any errors in any widgets.
https://www.loom.com/share/a7e8ba9dd42545f1beecff592cacb8c0
Steps To Reproduce

Go to any application.
Set up binding(s) in any widget in such a way that it throws an error.

Public Sample App
No response
Issue video log
No response
Version
Cloud

@shadabbuchh please always mention the expected behaviour to give better clarity!
Right now from what I can perceive it seems like this is what is expected, until you click on the property pane field you will not see the red highlighted border


How to avoid user to input the date?

Is there any configuraton to avoid user input the non-date format characters?

@NingerJohn I just had that problem and came here to see if there was a solution.
I ended up solving it by adding an eventListener for keypress on the field and preventing default. That way, they can still chose a date and click on the input field, but can't actually type in the box.
Probably a better solution out there, but it was a quick fix that kept me moving.

Just control it with native events as mousepress and input.


Wrong OG data showing when sharing in facebook!

In facebook:
https://developers.facebook.com/tools/debug/sharing/?q=https%3A%2F%2Fwww.codetriage.com
og:url	https://www.codetriage.com/
og:type	website
og:title	CodeTriage
og:description	Redis is an in-memory database that persists on disk. The data model is key-value, but many different kind of values are supported: Strings, Lists, Sets, Sorted Sets, Hashes

Lol. Whoops. Can you send me a PR?

I will create one tomorrow :)

Do we have any specific meta information for header? @schneems  can you provide me with the meta information. I can't see here any.

Added a pull request #514 as a part of this fix.

@schneems can we close this issue now ?

yep, thanks again!


Wrong merging of documents

While working on #160, I noticed that the package is potentially processing equal documents in invalid way and potentially send multiple same (same _id) documents to the client unnecessary instead of merging it on the server.
Here you are pushing collData which is an array of documents onto a collection. In this way you get array of arrays. Later on you add arrays. And you push this array of arrays to the client, at least it seems so. There is no code which would be merging multiple documents with same _id together. At least I do not see it. You just combine arrays. And have arrays of arrays even.
The second issue is that in the flow-router SSR you are deep-merging documents. This is wrong. Meteor does only top-level merging. This might introduce unnecessary incompatibilities.
The third issue is that you should probably validate inputs into the update call and check if there are any top-level keys which start with $. Because added/changed/removed calls do not check that you might get a document with those in, from bad code or malicious code.

Fast Render Issue
Here's the client-side merge box like functionality I used - https://github.com/kadirahq/fast-render/blob/master/lib/client/fast_render.js#L41
(Anyway, now I think it's better to do that in the Meteor's way if we can use it)
FlowRouter SSR issue
I think yes. That's right. We need to do shallow copy
I really some help on this.

Here's the client-side merge box like functionality I used

Yes, but why client-side? This means that the injected data can be unnecessary huge! You could have same documents multiple times send to the client.

(Anyway, now I think it's better to do that in the Meteor's way if we can use it)

Yes, I think we could do that. We would just create a our own session object and then reuse the rest of Meteor codebase.

We need to do shallow copy

You could just use _.extend. :-)

I really like to have some help on this.

I could look into using sessions, but then the API will change to one returning documents themselves for the whole session (which would be one per one HTTP request made). But integrating this new API with the rest of your packages is probably best done by you.
So if I understand correctly, you currently create Context for one HTTP request, yes? I could do it so that internally it has a session object and then all subscribes work against that session object. And then you have a method to inspect data inside that session object.
I would remove that subscribe returns data. This looks unnecessary complication. So API would be somehow like:

you crate Context
you run all rendering of a component while Context is active
we wait for all subscriptions to stop, or we timeout
you ask Context for all data

I do not know when I will have time for this though.


Connection refused

Successfully built 584500ec72e1
Successfully tagged flask-docker-container-boilerplate_web:latest
WARNING: Image for service web was built because it did not already exist. To rebuild this image you must use `docker-compose build` or `docker-compose up --build`.
Creating flask-docker-container-boilerplate_web_1 ... done

### Deleting dummy certificate for meem.no ...

### Requesting Let's Encrypt certificate for meem.no ...
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator webroot, Installer None
Obtaining a new certificate
Performing the following challenges:
http-01 challenge for meem.no
http-01 challenge for www.meem.no
Using the webroot path /var/www/certbot for all unmatched domains.
Waiting for verification...
Challenge failed for domain meem.no
Challenge failed for domain www.meem.no
http-01 challenge for meem.no
http-01 challenge for www.meem.no
Cleaning up challenges
Some challenges have failed.

IMPORTANT NOTES:
 - The following errors were reported by the server:

   Domain: meem.no
   Type:   connection
   Detail: Fetching
   http://meem.no/.well-known/acme-challenge/qPqfKmnb_0Khm89bPyQnywU7D1ufzMp0JUQ7nX6m69w:
   Connection refused

   Domain: www.meem.no
   Type:   connection
   Detail: Fetching
   http://www.meem.no/.well-known/acme-challenge/axhzfLr7JZtePGupyRW-zNPwuNYxb99txR7WKZXY8D8:
   Connection refused

   To fix these errors, please make sure that your domain name was
   entered correctly and the DNS A/AAAA record(s) for that domain
   contain(s) the right IP address. Additionally, please check that
   your computer has a publicly routable IP address and that no
   firewalls are preventing the server from communicating with the
   client. If you're using the webroot plugin, you should also verify
   that you are serving files from the webroot path you provided.

### Reloading web ...
nginx: [emerg] BIO_new_file("/etc/letsencrypt/live/meem.no/fullchain.pem") failed (SSL: error:02001002:system library:fopen:No such file or directory:fopen('/etc/letsencrypt/live/meem.no/fullchain.pem','r') error:2006D080:BIO routines:BIO_new_file:no such file)

Hi, I have the same issue, how did you managed to correct it?
My NGINX Docker runs but impossible to have it accessible even though ports 80 and 443 are open.

Having the same issue as above @melvinmajor

I'm also getting similar errors.
I have no more "example.org" in any of the files
I'm only testing for a single domain pointing to a static IP on a linux EC2 server where I run docker-compose
I've tried the exact steps in the medium tutorial, and I've taken the latest code from the github repo.
I've tried it with the proposed nginx server, and with my own set-up of my server which, if run by itself, works (but only over HTTP)

I have the feeling the server with docker-compose doesn't start correctly. But I have no idea how to check this or how to fix it

My setup: Running with upstream node apps.
The error I was getting when I ran ./init-letsencrypt.sh was always "Challenge failed for domain <domain-name>"
So I first mounted the nginx access and error logs to a volume and it always showed 1 of 2 things
1. nginx: [emerg] cannot load certificate "/etc/letsencrypt/live/<domain-name>/fullchain.pem": BIO_new_file() failed (SSL: error:02001002:system library:fopen:No such file or directory:fopen('/etc/letsencrypt/live/<domain-name>/fullchain.pem','r') error:2006D080:BIO routines:BIO_new_file:no such file)
2. nginx: [emerg] host not found in upstream "auth" in /etc/nginx/sites-enabled/default.conf:13

-> The certificate error made sense as it did not exist and hence this error occured
-> The upstream error was the one I kept ignoring thinking it was due to certbot or nginx. But this in reality was the real error.
SOLUTION
My nginx server had no clue about the "auth" upstream as it would only be available when my docker service - "auth" would be up. So after a lot of ignoring the error, I just ran docker-compose up before the script and then ran the script.
Voila solved my issue.
Hope my stupidness helps others hours of googling. Cheers!


a-decentralised-exchange-dex-on-solana

Guide Outline

A decentralised exchange is a network that allows anybody to swap cryptocurrency tokens pairs over the blockchain by performing a transaction between the two tokens.

Link to Proposal

https://dev.builderdao.io/vote/a-decentralised-exchange-dex-on-solana

Comments

Please add any comments or information for the reviewers that will assist them in completing the review process. Link to any supporting documentation, especially references you have used in the process of writing your guide.

@devilla FYI, this test will be automatically published at https://dev.builderdao.io/learn/a-decentralised-exchange-dex-on-solana once merged 🥳.


MRG, VIZ, FIX: plot_sensors title and interactivity

closes #7850

plot_sensors now puts title as axes title instead of fig.suptitle
the on_pick callback now won't act unless the click was in the correct axes

Ready for review/merge. Only CI failure is the XDAWN test on py3.6 (a test which I think @larsoner is about to remove anyway).

Thanks @drammock


Update update-license.sh to match format_spec_file again

Format-spec-file got updated fetcher and we need to match it:
https://github.com/openSUSE/obs-service-format_spec_file/blob/master/fetch-licenses.sh
Namely we need to check for the DUPES properly too and then also generate some LICENSES.md file for easy overview (they put it to readme.md but that is bit long for us.

I will work on it probably week after this one, but Tom if you could look on it sooner it would be lovely.

And when this is done, we need to update rpmlint to take licenses from our location

Now should be fixed.

Commit ba4b35f1afc9b45ef4954802c1e9ea8734fc41b2


Copyright update year

Ok, just updated to 2014 - 2016 as you suggested, and yes, it's better.

Thanks for the fix, but the first commit has been made on 21 Aug 2011, so it should be 2011-2016

@domnikl thanks to you! I learned with this repo. Ok, just corrected that.

Looks good now, merged!


Add API Gateway examples

The following examples should be added:

[ ] Spring Cloud Zuul
[x] Spring Cloud Gateway
[x] GraphQL

Spring Cloud Zuul is outdated, so, examples were not added. The rest of the examples are added.


Validate Task step name

In this PR, webhook validation is added for task step name. It checks whether step name adheres to qualified DNS label rules.
Fixes : https://github.com/knative/build-pipeline/issues/273

cc @bobcatfish
cc @pivotal-nader-ziada

whoa amazing turnaround time @shashwathi !!

/approve


Define the SDK via an option

Can you create an option in the IDE to define the JDK location on a computer?
I have several JDK on my computer, gradle can find them and in a previous version of the plugin, I started my test with Gradle (because Kotest is not able to find my JDK)
However, since the last update of the plugin (1.3.72-IC-2023.2) on my IDE 2023.2.5, Kotest is always running when I want to start test execution for a file. The problem is that Kotest cannot find any of my JDK. So, it's impossible to run tests with Gradle (because Kotest take the lead) or with Kotest (because he cannot find my JDK)
Intellij Ultimate: 2023.2.5
Kotest-plugin: 1.3.72-IC-2023.2
Kotest: 5.8.0
Previous plugin (with which I could launch my tests with Gradle): 1.3.71-IC-2023.2
We cannot download a previous version on the plugin site https://plugins.jetbrains.com/plugin/14080-kotest/versions/stable

Workaround:
To run a test file with Gradle, create a custom configuration with these values (for JVM test):
:cleanJvmTest :jvmTest --tests "package.classname"