 within

In
https://github.com/erc-dharma/tfa-pallava-epigraphy/blob/master/texts/xml/DHARMA_INSPallava00020.xml
line 166, I have been compelled to use <hi> within <choice> against Epidoc.
Shall we allow, against Epidoc, <hi> within <choice>?

Tough one. One thing I am quite sure of is that the <hi> should be inside the <unclear> element in this case, not the other way round. But that of course does not solve the problem. I see two ways to go ahead:

allow <hi>  inside <unclear> , preferably not against EpiDoc, but proposing this rule change to the EpiDoc community; or
change our encoding method and instead of <hi rend="grantha"> , henceforth use <g type="grantha"> to wrap each grantha character separately. For longer stretches of grantha text, we may need to continue permitting <hi rend="grantha"> , since one <g> tag should not, in my opinion, contain more than the transliteration of a single akṣara. So transition to this system may not be possible to automate on the already encoded files.

@ajaniak : do you have any other suggestions? What is your opinion of the above alternatives?
@manufrancis : is this encoding really necessary? Would you consider it an inferior solution to put your vote down on one reading or the other, and mention in an apparatus note that what you read in the edition as a grantha śa could also be read as a Tamil ca?

Tough one. One thing I am quite sure of is that the <hi> should be inside the <unclear> element in this case, not the other way round. But that of course does not solve the problem. I see two ways to go ahead:

allow <hi>  inside <unclear> , preferably not against EpiDoc, but proposing this rule change to the EpiDoc community; or
change our encoding method and instead of <hi rend="grantha"> , henceforth use <g type="grantha"> to wrap each grantha character separately. For longer stretches of grantha text, we may need to continue permitting <hi rend="grantha"> , since one <g> tag should not, in my opinion, contain more than the transliteration of a single akṣara. So transition to this system may not be possible to automate on the already encoded files.

@ajaniak : do you have any other suggestions? What is your opinion of the above alternatives?
@manufrancis : is this encoding really necessary? Would you consider it an inferior solution to put your vote down on one reading or the other, and mention in an apparatus note that what you read in the edition as a grantha śa could also be read as a Tamil ca?

@danbalogh, you could ask for the Epidoc to change their guidelines to allow <hi> inside <unclear>. Such a solution would be the easier to implement, even though it will be a nightmare to add in DHARMA tree model and codes.
I am not sure regarding <g> element. It will be bothersome to encode each character, but I am not sure it really matches the use if this element as describe by Epidoc which uses it for symbols and non-standard characters regarding unicode. Grantha is rather as a second script (at least to my understanding) so I wouldn't say it is non-standard.
As shortcut, we could use the attribute @rend="grantha" on <unclear>, allowing it only if the grantha and unclear are a strict match.

@danbalogh, you could ask for the Epidoc to change their guidelines to allow <hi> inside <unclear>. Such a solution would be the easier to implement, even though it will be a nightmare to add in DHARMA tree model and codes.
I am not sure regarding <g> element. It will be bothersome to encode each character, but I am not sure it really matches the use if this element as describe by Epidoc which uses it for symbols and non-standard characters regarding unicode. Grantha is rather as a second script (at least to my understanding) so I wouldn't say it is non-standard.
As shortcut, we could use the attribute @rend="grantha" on <unclear>, allowing it only if the grantha and unclear are a strict match.

Adding @rend="grantha" to <unclear> sounds good to me because it involves a minimum of fuss, but I guess it's not very good practice. I'll ask about <hi> in <unclear> on MARKUP, just in case.

Adding @rend="grantha" to <unclear> sounds good to me because it involves a minimum of fuss, but I guess it's not very good practice. I'll ask about <hi> in <unclear> on MARKUP, just in case.

Thanks, Dan, for asking MARKUP!
As for your suggestion:

"@manufrancis : is this encoding really necessary? Would you consider it an inferior solution to put your vote down on one reading or the other, and mention in an apparatus note that what you read in the edition as a grantha śa could also be read as a Tamil ca?"

Yes, indeed, I was thinking about such a solution here. But I foresee that in other instances, having the encoding option could be useful.

Thanks, Dan, for asking MARKUP!
As for your suggestion:

"@manufrancis : is this encoding really necessary? Would you consider it an inferior solution to put your vote down on one reading or the other, and mention in an apparatus note that what you read in the edition as a grantha śa could also be read as a Tamil ca?"

Yes, indeed, I was thinking about such a solution here. But I foresee that in other instances, having the encoding option could be useful.

As both of you may have seen on Markup, I've received some input, mainly from Gabby Bodard, on this issue. Here are the main points from my perspective.

It seems to me that the EpiDoc community is dead set on prohibiting anything but <g> inside <unclear>. Their reasoning is that by the EpiDoc definition of <unclear> (which is narrower than the TEI definition), this element may only contain character data, and anything beyond characters and <g> is not "character data" but "editorial interpretation". I don't really agree with that and remain convinced that EpiDoc's own (not fully TEI-compliant) use of <hi> in fact concerns character data, not editorial interpretation, in at least some cases. However, even if I could convince the EpiDoc folks about that, there remains the matter that for anyone who may try to utilise our files five or fifteen years into the future may not be as familiar with our EGD as we all are, and find it hard to figure out what some of our idiosyncratic encoding solutions stand for. So, in a nutshell, I propose that we go with the mainstream and forget about permitting <hi> in unclear.
For dealing with the immediate problem of this thread, Gabby has suggested using <app type="alternative">, an encoding method that does not seem to be documented at all, but that nonetheless seems to have gained some acceptance in the EpiDoc community, since it has been used by at least two different projects. Both examples of its use that I know of are inline, i.e. not in a separate apparatus division but right there in the text. I'm not sure we should complicate our already complex encoding with adding an option for inline apparatus entries, but if we agree to use this only for ambiguities pertaining to the script, then it may be possible to implement its display without increasing the complexity of the transformation too much. The display I am thinking of would be exactly identical to that of choice with unclear, so that e.g. an inline ABCDEFwould appear just likeABCDEF, with the only difference that ABC and DEF can contain tags such as in the former even though they cannot in the latter. If that is too complex to implement, then I think it should also be possible for us to introduce a new type ofinto our external apparatuses, such as@type="alternative"` - which would then be displayed in the external apparatus, not as a reading by a different editor but as an alternative reading. @manufrancis and @ajaniak , can you think about, and perhaps discuss, which of these would be more desirable and more practicable to implement in order to create a solution for Manu's problem?
Finally, regardless of what decision we make about number 2 above, perhaps we should reconsider our use of <hi> for script. EpiDoc folks seem to agree we ought to use @xml:lang for script changes even when a language change is not involved. Thus, according to Gabby, <seg xml:lang="Gran">śa</seg> would be "more conventional EpiDoc" than <hi rend="Grantha">śa</hi>. (Or @xml:lang="tam-Gran", but according to his line of thought the language part is inherited from the rest of the text, so encoding the script suffix here would be sufficient.) I'm willing to accept this line of thinking, mainly for the same reason as above, namely that our code may be more intuitively intelligible to others this way. However, it raises again a basic problem that we discussed once in the early days, but I don't have clear memories of the discussion. That is, since all our editions are transliterated, we either use no script suffix or use "-Latn" to indicate that our texts are transliterated. In other words, our suffixes specify the script of the edition, not the script of the original. A second associated problem is that we use ISO-compliant language and script codes, whereas I suppose "-Gran" is not such. If we want to change our method of encoding script changes to xml:lang on seg (or a pre-existing container where available), then we would have to stipulate that any script suffixes would also indicate the script of the original. That is to say, we would then not use -Latn anywhere, and simply declare that a language code without a script code means "a language written in the default script appropriate to the location of the inscription in space and time and presented here in Romanised transliteration" - where only the bit in bold is new, and the rest we already have. We should discuss at some time, not urgently, this issue number 3, also involving @arlogriffiths . If we don't want to reconsider our usage of @xml:lang, then it may still be possible for us to use @rend on <seg> (instead of on <hi>) to make our code more EpiDoc compliant.

Thanks, Dan! Will discuss these issues with Axelle when she we will back from vacation in early March.

Prompted to rethink script encoding by this thread, @ajaniak and @arlogriffiths have proposed a new method for encoding the original script. I agree with the proposal in most details. The method is outlined in EGD 7.5.5, with some ongoing discussion in comments. (The section number may change in the future.)
But this does not yet solve the original problem. @manufrancis and @ajaniak , have you come to any decision about how you would like to handle cases like this?
If not, I have a proposal. I'm not very eager to introduce it, since I am not aware of anything similar being in use, but it seems to be correct TEI. It would involve two <seg> elements within a <choice>, e.g. like this
<choice><seg corresp="class:36768 maturity:83213">ś</seg><seg corresp="class:38776 maturity:83213">c</seg></choice>
Where the funny numbers refer to our OpenTheso vocabulary items as outlined in the revised EGD section 7.5.5 (the first one meaning mediaeval Grantha, the second meaning mediaeval Tamil), and where I am proposing to use @rendition instead of @corresp (which is a detail irrelevant to the subject of this thread).
This method is not sanctioned by EpiDoc, but nor is it prevented by the EpiDoc or DHARMA schema (except that the DHARMA schema permits only “pun” as a value of @rend on <seg>, but that can be changed). TEI explicitly permits <seg> in <choice>, but probably not exactly for this purpose. Nonetheless, we could perhaps co-opt this, only for the cases where there is a script uncertainty regarding a character or string of characters
This would also avoid the use of unclear for something that is clearly readable (Gabby on MARKUP: [unclear], in EpiDoc, uniquely records incomplete, damaged, or illegibly executed characters, not per se the editor's uncertainty in interpretation of them).
Everyone is welcome to give their opinions on this suggestion. Most especially, I need opinions from @manufrancis and @ajaniak .

This too has never been brought to a close, although I'm happy to forget about it if @manufrancis and others can live with the lack of an encoding solution.

Thanks Dan, yes, let us forget about about this.


Refactor BaseConnectionPool empty method

Description
ConnectionPool.empty() never calls callback when there is no connection. Refactor empty method so that callback will get always called.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] Linter check was successfull - yarn run lint doesn't show any errors
[ ] Commits are signed per the DCO using --signoff
[ ] Changelog was updated.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks good, needs a test please.

Looks good, needs a test please.

Added UT with no connection.


artisan view:cache can't guarantee that all view files are compiled

Laravel Version: 5.7.12
PHP Version: 7.2.12
Database Driver & Version: Mysql 5.7.22

Description:
Hello.
It is a usual case when application has not permissions to write to disk directly. For example you may have a few nodes with your application and you can not correct share created and changed files for all nodes after deploy your application.
In this case we can not write at runtime to storage/framework/views too.
Of course, you can run artisan view:cache before deploy and compile all paths specified in config view.paths section. But external packages may to call their views different than compiler:
/var/www/src/../resources/file.blade.php instead /var/www/resources/file.blade.php for example. sha1() hashes of this is different, compiled file for first option is not exists, application try to compile and store it at runtime but failed because he has not permissions.
Steps To Reproduce:

Install any external package who has view files: laravel/telescope for example (I have it problem exactly with him)
Add realpath(base_path('vendor/laravel/telescope/resources/views')) to paths section in your config/view.php
Run artisan view:cache to pre-compile all specified paths
Make sure that vendor/laravel/telescope/resources/views/layout.blade.php is actually compiled
Set reduced permissions for your application: chmod 555 . -R
Run your application in browser. It works because view files are compiled now.
Run /telescope in browser. It crashed because compiled view file is not exists. Telescope try to get
telescope/src/../resources/views/layout.blade.php instead telescope/resources/views/layout.blade.php

I hope you understand me. Thank you.

#26358

Will be fixed in a future version.


Obsolete dosage compensation complex assembly involved in dosage compensation by hyperactivation of X chromosome and hypoactivation

Obsolete
GO:0016457 dosage compensation complex assembly involved in dosage compensation by hyperactivation of X chromosome
and GO:0042715 dosage compensation complex assembly involved in dosage compensation by hypoactivation of X chromosome
with replaced by  GO:0042714 dosage compensation complex assembly
1 EXP FlyBase
1 EXP WB
Thanks, Pascale

The reason for obsoletion is that this represents a GO-CAM model.

@hattrill

@edwong57 We can go ahead with the obsoletion. Annotations to obsoleted terms will appear in reports and can be fixed later.
Thanks, Pascale


Update index-add.md

Updated this article to reflect the new Index Lookup Tool

@eric-urban

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

These changes have been made in the live article. Closing this PR. Thanks for contributing!


Content Migration - Transparency Testing Tools Pages

Minor pull request that adds the content in markdown format for the following transparency testing tools pages:

/ooni/
/glasnost/

These pages can also be previewed here.

@shredtechular LGTM. Merging.


Duffel Bag - Design reasons

The idea here is to add another relic based on Ryu, in his case, his duffel bag that he carries around.
Following the idea that

As a fighter, Ryu surely would take with him things to take care of damages received during training/fighting;
As a thing, Duffel Bags are used to carry stuff, and people in the Spire probably used it for a while...

...the function of the Duffel Bag in game is to give to the player an increase in his capability of survive fights, not to end them fast; also some small general help with extra relics.
So, this is how the relic will work.

When the player gets hit, he receives:

1 Bandage Up card (possible +4 HP per fight);
1 Panacea (less problems with debuffs);
2 random common relics.

Given that no base game relic gives cards to the deck, this brings something new to the game, and also is a good step for the future T.Hawk's relic.


Added sort functionality to IMDb rating

Closes #68
https://user-images.githubusercontent.com/71957423/154852927-e4c12d34-7c34-456d-8347-8041a6fc33c1.mp4

@SaraswatGit Please have a look at this!

I have merged your PR gr8 Work !!


Avoid spurious noexcept warnings

This re-adds #6343, removes one line which seems to be the source of the spurious warnings, and add a bunch of tests to make sure we don't get the spurious warnings.
Goes into 3.0.x
Fixes https://github.com/cython/cython/issues/6428

I see now this causes some issues when on legacy_implicit_noexcept mode. I'll try to figure out another way.
This all has to do with what happens when there are two declarations (one in .py/.pyx , another in .pxd).

This seems a better fix:
--- a/Cython/Compiler/Nodes.py
+++ b/Cython/Compiler/Nodes.py
@@ -3126,6 +3126,8 @@ class DefNode(FuncDefNode):
             if scope is None:
                 scope = cfunc.scope
             cfunc_type = cfunc.type
+            if cfunc_type.exception_check:
+                has_explicit_exc_clause=True
             if len(self.args) != len(cfunc_type.args) or cfunc_type.has_varargs:
                 error(self.pos, "wrong number of arguments")
                 error(cfunc.pos, "previous declaration here")

Do you have a quick example of some code that you addition fixes?
I'm just trying to make this as tested as possible

Do you have a quick example of some code that you addition fixes?
I'm just trying to make this as tested as possible

It's already tested in legacy_implicit_noexcept_build.


btree prefixnode type bug

command line: bin/test_mem_btree --gtest_filter=*/0.RandomRemove
Error:
HomeStore/src/tests/btree_helpers/btree_test_helper.hpp:161: Failure
Expected equality of these values:
removed
Which is: false
m_shadow_map.exists(*pk)
Which is: true
Removal of key 43770 status doesn't match with shadow

@shosseinimotlagh pls verify if this ticket still valid?


Box component refactor from Styled Components to Emotion

This pull request is an experiment to see how much we can improve performance, specifically during hydration, by using Emotion for styles instead of Styled Components.

I marked this PR as draft because the useDeviceMode hook always returns 'desktop'.

I marked this PR as draft because the useDeviceMode hook always returns 'desktop'.

It seems that we only use it for dynamic control options, though, so it might no be a huge issue...


feat: add topic operator based on Kafka admin client

This change allows to deploy an alternate version of the topic operator,
which creates topics using the Kafka admin client, instead of relying
on Strimzi Kubernetes resources.

/test


[Feature]: Tabs: Prop to not unmount once the Tab is viewed.

Describe the feature
Currently, we have a prop to forceMount a tab content. But, if a child component which fetches data is placed inside a background Tab Content, then the child component will still compulsorily fetch the data (albeit only once), even if not the user doesn't view/open the tab.
However, it would be optimal if we have a prop called "unmountOnExit" (which can be set to false). This way, Tabs work normally (i.e., no need to force mount background tabs if a tab is never viewed), but if a specific tab is viewed (which has this prop), then the child component would fetch the data only once, and then would not re-render again when the tab is switched back again.
Additional information

[ ] I intend to submit a PR for this feature.
[ ] I have already implemented and/or tested this feature.

can you use KeepAlive vue component

Can you kindly share the anatomy of such usage? I mean, which element should we wrap with KeepAlive... the TabRoot or ...?

i didn't try it out ..but i was meaning the tabContent that you want to keep it unmounted ..wrap it with keepAlive.

As @peter-emad99  suggested, you should leverage KeepAlive component from Vue. Or you can use v-show.
Check this example out

Will close this out as the component already offers the capability to achieve the desired output.
Check https://github.com/radix-vue/radix-vue/issues/756#issuecomment-2003338097


flake/nix-command based eval?

would it be possible to use nix eval, so that the evaluation is cached? I feel it would speed up usage a fair bit

I don't think that would work. nixpkgs currently does not expose all packages in the flake and legacyPackages is not cached afaik.

This is the first ive heard of not all packages being exposed

If you do $ nix flake show --legacy you see that it won't include any nested package sets such as python packages. It's probably possible to have an eval script that iterates over all packages in the same way nix-env does, but it likely also slower.

What would be the problem with writing a small expression that gives you all of the packages?

Than you won't benefit from any cache and just reimplement nix-env in a slower way. Also there are some exceptions thrown in nixpkgs every once in a while that you cannot catch from nix.
This is the script that I used before to go over all of nixpkgs. https://gist.github.com/Mic92/0a30fa6babd3eb9c8445fd877ae8781f

Why not put that script in a flake and modify the nixpkgs input?

I'd say it would be easier to just cache the nixpkgs package list that nix-env generates: https://github.com/Mic92/nixpkgs-review/blob/30a6655d29da7b380d504b9fdbe06a60ae9ea911/nixpkgs_review/review.py#L344
and less error prone than those terrible nix expressions.


Option to ignore inf

Pollutes r(max) in SEResNets, and r(min) isn't sufficiently informative
Meant to open this as feature request

There is some filter functionally already implemented, but it is currently not easily accessible to the user.
I try to come up with some minimally invasive way to make it accessible.
My proposed solution to this would be a string key, similar to this:
def create_graph_from_model(model: Model, filter: Optional[Union[str, Callable]] = None) -> EnrichedNetworkNode:
By default, no filtering is done, which is the technically correct way of computing receptive field sizes.
However, this may be misleading for things such as r(max), when attention-mechanisms come into play (SE-Modules are typical example). These are low capacity, which means they will likely not contain much critical information that helps directly solving the problem, while having an infinite receptive field size, which (as you correctly observed) pollutes the computation.
A supported alternative would be to filter all infinite values if there are any non-infinite receptive field sizes present, which can be selected using the string key "inf". If, at some point, you decide that another filter is necessary, for example a filter that combines all receptive field sizes into an average receptive field size, you can alternatively provide a callable filter Callable[[Tuple[ReceptiveFieldInfo, ...], Tuple[ReceptiveFieldInfo, ...]` that conducts your custom filtering for you.
This change permeates through multiple parts of the library, since it messes with all encoders of the framework. So, the change will take a bit of time to implement and test properly. I hope I get it done some time next week.
Also, let me know if you have any other suggestions regarding the proposed handling.

Yeah, a mix of simple str and extended Callable sounds perfect.

The new release 1.4.2 contains the discussed changes.
Here is some example code:
if __name__ == '__main__':
    from rfa_toolbox import create_graph_from_pytorch_model, visualize_architecture
    from torchvision.models.efficientnet import efficientnet_b7

    graph = create_graph_from_pytorch_model(efficientnet_b7(), filter_rf="inf")
    visualize_architecture(graph, "DUMMY", input_res=32).view()

Results in squeeze and excitation modules being ignored correctly by the rest of the network.
Here one example from the model above:


psusecompatibleSyntax goes a bit too far.

Not sure if something has changed but I've noticed this recently, and it the hadn't registered before
Summary of the new feature
I'm seeing this
The constructor syntax '[type]::new()' is not available by default in PowerShell versions 3,4
I got the same with class  We're trying to get people to move on from Windows PowerShell 5, but that has been around for 8 yeas or so and it might be time to review the pre-PowerShell 5 features people are told to avoid
Proposed technical implementation details (optional)
Drop checks for PowerShell 3 compatibility.
What is the latest version of PSScriptAnalyzer at the point of writing
1.22.0

You might also consider to link this type of rules to the #Requires -Version x.x statement.
(In case it doesn't exist in the concerned script: simply suppress the warning.)
Originally posted by @iRon7 in https://github.com/PowerShell/PSScriptAnalyzer/issues/2046#issuecomment-2546082319

It might also be considered to link this type of rules to the #Requires -Version x.x statement.
Originally posted by @iRon7 in #2046 (comment)

YES . If the solution were "Put a requires statement in if your code uses feature only in XX" (which only appeared if there was no such statement) that would be ideal.


Set_membership only if a user sign in,

Here is a pull request for your crazy named branch.
Updated the set_membership method to run it only if a user sign-in, otherwize redirect to root.
Added set_membership to devices controller.

Did it. Interesting:

when I made set_membership a public method in the application controller a user was redirected to the root (if not signed in);
when I made the method private, an unassigned user is redirected to a devise users/sign_in page with a message "You need to sign in or sign up before continuing." And in this case, I don't even need to check "if user_signed_in? ... else redirect_to root_path ..."
I like the second one better.


test for accessing daycodes

fixes #208
add test for checking if only admin can get large group daycode
add test for checking if user can get dayCodes

looks good. Merge after both pass


Adding http-parser library

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

Thanks for the solid work!


Version 2.0 on Intel processor results in conda conflicts

I'm trying to install the new 2.0 version (main branch) on my MacBook Pro with Intel Core i9 processor. But this command:
PIP_EXISTS_ACTION=w CONDA_SUBDIR=osx-64 conda env create -f environment-mac.yml

Always ends with:
Found conflicts! Looking for incompatible packages.

When I try it with the release-1.14.1 version, it works correctly. Is it a bug or am I doing something wrong?

seems like the conda environment always causes issues like this, since there are so many deps and many of them are moving targets.
i resolved this by installing mamba, which provides more coherent conflict resolution information. at some point along the line the conflicts were resolved by installing protobuf 3.20.x, however, it appears that textual inversion breaks with protobuf > 3.19.4, so that's not a proper solution.
with my most recent conflict, opencv-python was the problem. the environment calls for 4.6.0 and <IP_ADDRESS> was available but for some reason did not satisfy the dependency. so i explicitly called out <IP_ADDRESS> and that worked.
YMMV; i am running on an M1 mac.

try this sequence to start a clean environment
conda deactivate

conda env remove -n invokeai

conda clean --yes --all

PIP_EXISTS_ACTION=w CONDA_SUBDIR=osx-64 conda env create -f environment-mac.yml

I have the same issue. Removing and cleaning the environment didn't work for me. For now I'm back to 1.14.

I had to do cleaning twice (M1 Max MacOS) and then the last time I recall, created condo environment with this command:
conda env create -f environment-mac.yml

FYI we're working on a streamlined "1-click" install script for all major platforms. Hopefully will be ready soon.

this recently started acting up for me too, tried to install mamba using conda but getting the same conflict issues there too :-/ oh, python dependency management.

It's working for me now after the latest changes. Thanks!


Can't install rye due to 404

Steps to Reproduce

curl https://rye-up.com/get | bash

Expected Result
Rye installs
Actual Result
404
Version Info
None
Stacktrace
No response

Could you provide some info about your system? I can't repro on macos.

The issue is that you run curl rye-up.com/get instead of curl https://rye-up.com/get. That means that you do a HTTP instead of HTTPS request, so the server issues a redirect which curl does not pick up on unless you pass -L. Is there a command mentioned somewhere without the https in front?

Closing this. I cannot reproduce this.


Split rules module into a module per rule set

This significantly improves build and test times.
I think we now have a perfectly parallel build xD

        Caused by:
        java.lang.IllegalArgumentException: Kotlin script engine not supported
            at io.github.detekt.test.utils.KotlinScriptEnginePool.createEngine(KotlinScriptEnginePool.kt:37)
            ... 10 more

sounds like a missing dependecy?

Awesome change @arturbosch 🎉


feature request: option to show all hidden files if directory only has hidden files

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
If I open a directory that only has "hidden" files in it, I'd like an option that defaults to showing those hidden files do I don't have to do g. to show them. Probably doesn't have to be default behavior
Provide background
I've got my various .zshxxx files inside a .zsh folder in my home dir. If I run nvim .zsh to open that directory, I have to always do g. afterwards to show the files.
What is the significance of this feature?
nice to have
Additional details
No response

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


Cannot detect new seeder

Winter CMS Build
dev-develop
PHP Version
8.1
Database engine
MySQL/MariaDB
Plugins installed
No response
Issue description
Created new seeder under backend > database > seeds. Cannot detect new seeder
Steps to replicate

Created new seeder under backend > database > seeds
Run cmd php artisan winter:up > Cannot detect new seeder.
Try running php artisan db:seed > shows  ERROR  There are no commands defined in the "db" namespace.

Workaround
No response

@chammeow is this the Builder plugin you created the seeder in?

@chammeow seeders and migrations in Winter plugins are run the same way, either with winter:up or migrate. There is no separate db:seed command in Winter.


In stream load, the default value of strictMode is false.

After we updated starrocks from<EMAIL_ADDRESS>to<EMAIL_ADDRESS> we found that stream load a row contains long column (longger than schema limit) success, and the value changed to NULL.
In<EMAIL_ADDRESS>the load task would be failed.
Also in online document, default value of strictMode is true. (https://docs.starrocks.com/zh-cn/main/loading/StreamLoad)
I think this should be a bug.

I checked the code and this configuration is always False, this is a documentation error.

I checked the code and this configuration is always False, this is a documentation error.

I just create a dorisdb cluster and perform a test.
sh show_fe_version.sh
Commit hash: 4bd9d39
Build type: RELEASE
Build time: 2021-05-11 14:18:30
Build user: jenkins@doris-sandbox07
Java compile version: java full version "1.8.0_202-b08"

MySQL [(none)]> select current_version();
+-------------------+
| current_version() |
+-------------------+
| 1.15.0 4bd9d39    |
+-------------------+

CREATE TABLE t (
  t varchar(1)
) ENGINE=OLAP
DUPLICATE KEY(t)
DISTRIBUTED BY HASH(t) BUCKETS 1
PROPERTIES (
"replication_num" = "1"
);

and get a failed result after stream load
Reason: the length of input is too long than schema. column_name: t; input_str: [1111111111] schema length: 1; actual length: 10; . src line: [];

@ikaruga4600
Thank you very much to report it and give us a reproduce way.
In 1.15, non-vectorized load was used, however vectorized load was used in 1.18.
The key point is not the default strict value, but the default operation for the 'too long' string.
In my opinion, changing the default value is a big thing, because it will influence all users.
And treating 'too long' string as 'NULL' seems OK when strict_mode is false.
The original approach could be considered inappropriate and the current implementation corrects this approach.

@imay
Thank you for your reply.
Strict mode have the same behavior as sql command line, and i think strict mode as default behavior may more fits user's anticipate.
I agree about changing the default value is a big thing, but in my position, i prefer default enable strict mode.

@ikaruga4600
Thanks for your advise. I think we should make it clear to users.


Low Light or bad lighting no face recognition

This is an awesome program and really appreciate the work put in. Using Dell XPS with Ubuntu and Windows. I notice that windows facial unlock is turning on a light with the webcam to assist in low light or bad lighting situations to assist the camera. Is that possible with this application or is the feature not possible. That is really holding this application back. It really only works in perfect lighting conditions.
Well and also it doesn't open the key ring on login so you still have to type in your password. I know that the gnome devs won't allow it due to security reasons.

That works for almost everyone yes, are you sure you are using your IR camera and not the normal visible light webcam?

Sorry I guess I was on the wrong camera. I changed it and it looks a lot more clear when I test it. It was on video2 but video0 is the correct camera.


Bug: Token is cleared before Logout

I want to logout from my application in Axios interceptor when i receive 401 code. It works with setTimeout set to 5seconds, but it dose not without it. On the screenshot token is empty in if statement.
Also why do you clear token before checking if it exist?
I use:
"axios": "^1.3.2",
"react-oauth2-code-pkce": "^1.20.0",
Screenshots

The expected behavior
Token is not cleared before checking if it exist to be able to logout

Hi, thanks for reporting @tomekxoxo
I understand why this looks strange, and should perhaps be written in a different way. But I think there is something else going on here.
Reacts 'useState' is asynchronous, so if I access the variable right after its set, in an synchronous code block, I will in fact be referring to the old value.
Could be that the reliance on this causes issues if you are doing something funky by intercepting requests.
If token was empty, you should never be redirected, and therefore never get a 401.
If you do get redirected, but no token in the request, that would be very strange.
Can you provide full authConfig, console log, and network requests for further debugging?

I deliberately cut out a token from a network request to receive 401. Could that be an issue?

I think I found an issue in my interceptor, sorry for false alarm

Yeah i didn't eject interceptors when data changes :)


unarchiving does not maintain the original user/group of a file

An example to come, just writing an issue, so I remember.

Fixed at #22


kubectl client and workload exec shell stuck in "Connecting" state for EKS cluster

Rancher versions:
rancher/server or rancher/rancher:
Steps to Reproduce:

Create an EKS cluster.
Create workload and exec into the pod. It is stuck in "Connecting" state

Launch the kubectl in rancher. It is stuck in "Connecting " state

I have the same issue even if the other features work like import, start a new deployment etc. Only the features opening a shell, the kubectl console or to view the log don't work.

@soumyalj @jmirc how many nodes were in your cluster?

@jmirc what version of rancher are you  uisng?

Tested with master.

Create cluster with default values.
Create workload and exec into the pod. It was successful

Also, launching kubectl shell successful


feat(userconfig): required fields

About this change—what it does

adds support for required fields to userconfig generator

Why this way

to have required fields in place

lgtm


App name ideas

For inspiration
https://en.wikipedia.org/wiki/List_of_bulletin_board_systems

Pyllama part

No Alpacas BBS

The llama Research Facility
Purple cyber llamas
Llamas underground lair

COTLQ - Confessions of a Teenage Llamma Queen

Drama Llama

Llama.py Bulletin Board System

COTLaQ looks a quite serious acronym. How are you pronouncing it? "Cot-lack"?

Yea, cot-lack. Like Kotlin, but with llamas


scene graph generation

Thanks for your amazing work, could you plz tell me where is the code for Section II.C spatial location clustering?

even the function load scene graph has not been called? https://github.com/nuoxu/AKGVP/blob/19c9fb751f5cf9154557365a990ba790159a539b/models/AKGVP.py#L20

You can refer to "https://github.com/sx-zhang/HOZ/tree/main/graph_generation".


Commit to use with zig 0.11

The compat branch for zig 0.11 is missing. Here is the commit to use for 0.11: commit

Apologies for forgetting to do it last time. Now that 0.12 is released, there will not be a 0.11 tag. See https://github.com/Snektron/vulkan-zig/issues/130.


feat: add support for snapshot sets of volumes that are not in the same VG

Reason: MVP requirements call for management of snapshots as sets
Result: Users can manage sets of snapshots with the role.
sets are described as:
vars:
snapshot_set:
name: snapset1
volumes:
- name: snapshot VG1 LV1
vg: test_vg1
lv: lv1
percent_space_required: 20
- name: snapshot VG3 LV7
vg: test_vg3
lv: lv7
percent_space_required: 15
Enhancement:
Issue Tracker Tickets (Jira or BZ if any):

There are some issues with the python code

we use black for formatting - you can check locally with tox -e black - you can fix locally by doing

. .tox/black/bin/activate
black tasks/files/snapshot.py
deactivate

there are some codeql warnings

[citest]

[citest]

[citest]

One more change to the README - for snapshot_lvm_all_vgs - I think we need to say something like "If false, then you must specify snapshot_lvm_vg".


Jacobian of the SE3 residual is expressed in the right frame - [merged]

In GitLab by @cmastalli on Nov 16, 2018, 14:05
Merges solver -> devel
This fixed an important bug in the computation of the SE3 cost. Additionally I added data for the SE3 error which is updated in updateSE3Error. This bug was reported in #28.
I noticed that the gradient reduction is much better (a beautiful exponential for the HyQ problem), even though it's not request to rotate.

In GitLab by @cmastalli on Nov 16, 2018, 14:07
changed the description

In GitLab by @cmastalli on Nov 19, 2018, 09:26
merged

In GitLab by @cmastalli on Nov 19, 2018, 09:26
mentioned in commit a8a6b6a67711a29d6b9365420ed0c864a95241da


[Bug]: Configmap feature flag is overriden by the dashboard

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When trying to disable one application in the odh-enabled--applications-config, it will be overridedn if the kfdef of that application exists. This is due to this logic: https://github.com/opendatahub-io/odh-dashboard/blob/main/backend/src/utils/resourceUtils.ts#L274
Right here it will check if the application's kfdef exists, and if so, just enable it and update the configmap, so If you had that application disabled it will revert back to enable.
Expected Behavior
When you change a value in the configmap, the state of the application chagens and it reflects the status that you've selected.
Steps To Reproduce

Open odh-enabled--applications-config.
Change jupyter to false (or straight up delete the configmap).
Reload the dashboard and wait for the watcher to be triggered it again (around 2 mins).
The value in the configmap will be reverted to true.

Workaround (if any)
No response
OpenShift Infrastructure Version
No response
Openshift Version
No response
What browsers are you seeing the problem on?
No response
Open Data Hub Version
No response
Relevant log output
No response

I have some questions about the opinion here. The file odh-enabled--applications-config is used to record whether an application is available or not and show the enabled or disabled status on Enabled page, it's not designed to be manually changed. It's always checked by odh-dashboard backend and updated by the logic. So this is the expected behavior. When the application has kfdef and the kfdef exists, the application should be REALLY enabled. So this is the right behavior to overwrite the  odh-enabled--applications-config to set it to true, and this configmap should never be manually changed, it's just a flag set by the dashboard via some logic.
cc @andrewballantyne @maroroman @dlabaj

This config map is used show what is on the enabled page.  I don't understand why we would prevent this from being updated manually.  Once you enable an app it never can unenabled.  I understand we don't do that through the UI yet, but I think it's a valuable feature to have.  @DaoDaoNoCode Is there a reason why we wouldn't want to allow someone to not remove an app from the enable page.

I do think that we should use features.json to disable an app all together since it will remove documenation, and quick starts where this won't.

Closing this as an outcome of the discussion we had earlier.

Closing this as an outcome of the discussion we had earlier.


Listeners not reconnecting properly inside docker container after node restart

I'm having an issue where, when there are a large number of subscribers to a topic, killing and restarting the node that publishes that topic results in several of the connections not re-establishing correctly.
I am able to reproduce this problem with a simple node that publishes continuously to a topic, and with 40 "rostopic echo"s that subscribe to that topic inside of the ROS Noetic docker that is provided as part of this repository (dustynv/ros:noetic-ros-base-l4t-r35.1.0). The ROS master is also running inside that same docker image.
After a spin up a ROS core, my publisher node, and 40 subscribers, if I kill and restart the publisher node, I see CPU usage for the publisher node spike for several seconds to 100%. Screenshot below

For 30-90 seconds after restarting the node, roswtf will indicate that several nodes that are supposed to be connected are not connected. The subscribers that aren't connected at the beginning will not recieve messages from the publisher, and these initial messages will be lost.

In addition to this, sometimes a few subscribers will remain unconnected to the publisher seemingly indefinitely (>30 minutes), though that is more challenging to reproduce.
When running the same test code outside of a docker container on the Jetson, the connections recover from the restart almost immediately (<5s) and there is no spike in CPU usage when the node first starts up. Running this test in an x86 docker on a regular laptop gives the same results as outside of the docker on the Jetson.
This leads me to believe that something about either the way this particular docker image is configured, or something about running docker images on the Jetsons in general. Any suggestions about how to dig into that further?
I am happy to upload my docker image with these tests scripts if they will be helpful for someone trying to debug this.

@johnwason Have you seen anything like this before?

@Levi-Armstrong I haven't seen this exact issue before. There may be a kernel limit in /proc on the number of sockets a process can have open concurrently. There may also be an issue with too many sockets in TIME_WAIT.
What does:
netstat -an

say during the problem?

@johnwason - thanks for the reply.
netstatOutput.txt
I've attached the output of netstat -an.
Here are some additional outputs that may be useful:
root@cosmo-spacely:/# ulimit -a core file size          (blocks, -c) unlimited data seg size           (kbytes, -d) unlimited scheduling priority             (-e) 0 file size               (blocks, -f) unlimited pending signals                 (-i) 13939 max locked memory       (kbytes, -l) 64 max memory size         (kbytes, -m) unlimited open files                      (-n) 1048576 pipe size            (512 bytes, -p) 8 POSIX message queues     (bytes, -q) 819200 real-time priority              (-r) 0 stack size              (kbytes, -s) 8192 cpu time               (seconds, -t) unlimited max user processes              (-u) unlimited virtual memory          (kbytes, -v) unlimited file locks                      (-x) unlimited
root@cosmo-spacely:/# cat /proc/sys/fs/file-max<PHONE_NUMBER>854775807

I see a lot of TIME_WAIT sockets. If you wait about 15 minutes after stopping, does it work normally when you restart? It takes about 10-15 minutes to clear all TIME_WAIT sockets.


redirected_stdio does not work within a Jupyter Notebook

The following code works as expected within an IPython interpreter session, but throws an exception when performed within a Jupyter Notebook. The example code is shown when executed in an interpreter, the subsequent block is the exception that results when executed in a new and otherwise empty Notebook.
As a back story, I'm doing some Artifact API calls within a Notebook and I'm attempting to suppress the stdout messages that come from methods like qiime2.plugins.alignment.methods.mafft.
In [1]: from qiime2.util import redirected_stdio

In [2]: import os

In [3]: with redirected_stdio(stdout=os.devnull, stderr=os.devnull):
   ...:     print("hi")
   ...:

In [4]:

Notebook:
---------------------------------------------------------------------------
UnsupportedOperation                      Traceback (most recent call last)
<ipython-input-1-9142123c7c9c> in <module>()
      1 from qiime2.util import redirected_stdio
      2 import os
----> 3 with redirected_stdio(stdout=os.devnull, stderr=os.devnull):
      4     print("hi")

/Users/mcdonadt/miniconda3/envs/q2deblur_mvp_test/lib/python3.5/contextlib.py in __enter__(self)
     57     def __enter__(self):
     58         try:
---> 59             return next(self.gen)
     60         except StopIteration:
     61             raise RuntimeError("generator didn't yield") from None

/Users/mcdonadt/miniconda3/envs/q2deblur_mvp_test/lib/python3.5/site-packages/qiime2-2017.2.0-py3.5.egg/qiime2/util.py in redirected_stdio(stdout, stderr)
     20     with _REDIRECTED_STDIO_LOCK:
     21         if stdout is not None:
---> 22             with _redirected_fd(to=stdout, stdio=sys.stdout):
     23                 if stderr is not None:
     24                     with _redirected_fd(to=stderr, stdio=sys.stderr):

/Users/mcdonadt/miniconda3/envs/q2deblur_mvp_test/lib/python3.5/contextlib.py in __enter__(self)
     57     def __enter__(self):
     58         try:
---> 59             return next(self.gen)
     60         except StopIteration:
     61             raise RuntimeError("generator didn't yield") from None

/Users/mcdonadt/miniconda3/envs/q2deblur_mvp_test/lib/python3.5/site-packages/qiime2-2017.2.0-py3.5.egg/qiime2/util.py in _redirected_fd(to, stdio)
     39         stdio = sys.stdout
     40
---> 41     stdio_fd = _get_fileno(stdio)
     42     # copy stdio_fd before it is overwritten
     43     # NOTE: `copied` is inheritable on Windows when duplicating a standard

/Users/mcdonadt/miniconda3/envs/q2deblur_mvp_test/lib/python3.5/site-packages/qiime2-2017.2.0-py3.5.egg/qiime2/util.py in _get_fileno(file_or_fd)
     60
     61 def _get_fileno(file_or_fd):
---> 62     fd = getattr(file_or_fd, 'fileno', lambda: file_or_fd)()
     63     if not isinstance(fd, int):
     64         raise ValueError("Expected a file (`.fileno()`) or a file descriptor")

/Users/mcdonadt/miniconda3/envs/q2deblur_mvp_test/lib/python3.5/site-packages/ipykernel/iostream.py in fileno(self)
    357
    358     def fileno(self):
--> 359         raise UnsupportedOperation("IOStream has no fileno.")
    360
    361     def write(self, string):

UnsupportedOperation: IOStream has no fileno.

Hi @wasade,
IPython/Jupyter Notebooks seem to handle their streams in a way that this utility function (as-is) won't be able to help with. As a workaround, the built-in magic %%capture command may give you what you are looking for.

Excellent, thanks! Given the prevalence of the Notebook, do you think it would be worth a more informative error message?

No problem! For sure, the ideal solution would definitely be to fix redirected_stdio to work within the Notebook environment, and if that turns out to not be possible, due to whatever it is they are doing to their streams internally and still keeping it compatible with the CLI/native python environments, we could try and catch that UnsupportedOperation then re-raise with a clearer message and a suggestion to use %%capture.


Update autoDismissAnimated:atTimeInterval: to reset timer

With this change the timer will be reset with each call to this method allowing for default set-up's overridden if needed.

Merge conflicts need to be resolved.


[BUG] The link to how to vote leads to a page not found.

Describe the bug
When a user clicks the "How to vote" link it leads him to a page not found.
To Reproduce

Go to the vote page
Click the "How to vote"

Expected behavior
A page with an explanation of how to vote is opened
Screenshots

@Yanivshechter1 what page? Please provide URL of the destination page

Added the link to the bug description


Error occurred when combined with CupertinoSliverRefreshControl

I get an error when combining NestedScrollViewPlus with CupertinoSliverRefreshControl with delayed, specifically as shown in the video
======== Exception caught by widgets library =======================================================
The following assertion was thrown building CupertinoSliverRefreshControl(state: _CupertinoSliverRefreshControlState#aa3a8):
'package:flutter/src/rendering/object.dart': Failed assertion: line 2310 pos 14: '_debugRelayoutBoundaryAlreadyMarkedNeedsLayout()': is not true.
Link: https://drive.google.com/file/d/17_O1VQ_hMxcEazUBXtzXVpBET610urLt/view?usp=sharing

In a NestedScrollView, the standard refresh indicator is not designed for this specific use case.
You can check how to implement a top pull-to-refresh indicator here: idootop/scroll_master.
Demo: https://flutter-scroll-master.vercel.app

I may add a built-in custom top refresh indicator for easier use in the future.
In the meantime, you can implement the refresh indicator as shown in the demo above—it doesn't seem too complex.


More detailed installation guide

Hello, I actually want to try your great abaplint project as plugin for eclipse, but I am stuck (as I am not anyhow familiar with plugin concepts nor TS nor Node...).
So I started with your instructions:

Import all three projects into Eclipse  - so imported as GIT project, now have abaplint.site, com.abaplint.eclipse and org.abaplint.eclipse  in Project Explorer - I assume this is the point here
build project abaplint
copy web/script/bundle.js from the abaplint project to org.abaplint.eclipse/src/bundle/

but I am not sure how to perform step 2?  I assumed that

npm test

should do the work, but I don't see web/script/bundle.js
Could you help?
Thank you.

actually I dont think abaplint-eclipse works anymore, plus back in 2016 it was running really slow(Javascript in Java), since then a lot more code has been added
will add a deprecation notice in the readme, sorry
plus the filenames does not follow the abapGit standard, so abaplint will not work on it anymore

currently abaplint works best with Visual Studio Code

also note https://github.com/larshp/abapOpenChecks contains a lot of the same checks as abaplint, plus more

aah, ok then :) Thank you very much for your time!
Do you by a chance know if there is any ABAP source code formatter (for eclipse) which could automatically enforce some custom rules (like where to put brackets etc.) not just simple alignment and changing case for keywords (so to be better than ABAP Pretty Print) ?

there is some work at https://github.com/flaiker/prettier-printer but its not done AFAIK
I think the power tools of the future will be outside of Eclipse, much like eslint and similar


Implementation Plan: Filtering and designating sensitive results in the API

Fixes

Fixes #937 by @sarayourfriend
Opened 30 March, due 13 April.
Description

I've requested reviews from Staci and Dhruv as the IP touches Airflow, API, and ingestion server changes. Staci and Dhruv are both knowledgeable in at least one or several of these areas. If either of y'all are unable to review this, please let me know and I'll ask someone else.
Testing Instructions

Read the IP. I do not have any outstanding questions, but please focus on the plan and make sure that it makes sense and is feasible. If you have suggestions for how to break any steps up into separate PRs, please share those especially.
Checklist

[x] My pull request has a descriptive title (not a vague title like
Update index.md).
[x] My pull request targets the default branch of the repository (main) or
a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[N/A] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[N/A] I tried running the project locally and verified that there are no visible
errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@AetherUnbound Thank you for the editorial suggestions. I've applied them in https://github.com/WordPress/openverse/pull/996/commits/830f9a747b595362880bcdf117ae8965268a991f. Rather than noting that in each suggestion, I've gone ahead and just resolved the conversations, but I wanted to make sure you knew I was grateful for your suggestions and did apply them. I've also added the clarifications we discussed.
@stacimc I've made the changes we talked about to the feature flags, conflicting parameter handling, and the Airflow DAG configurations. @AetherUnbound I would appreciate you careful review of the DAG situation as well :pray:
@dhruvkb I made the change you suggested to use a URL instead of putting the list in the environment variables. It's a much more flexible solution that just generally makes more sense. I also added instructions for how to handle this in staging and local environments by using a locally served file from Falcon itself. This seemed like the easiest approach without adding too much in the code for handling fallbacks of different types of settings for different environments.

Thanks @AetherUnbound. Address your comments in https://github.com/WordPress/openverse/pull/996/commits/a1fad8c575d91ba6fc58c7e6608c78804cbcf137


cmd/go: mod edit doesn't respect direct/indirect blocks in go.mod

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

$ go version
go version go1.18 darwin/arm64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/andig/Library/Caches/go-build"
GOENV="/Users/andig/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/andig/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/andig/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/homebrew/Cellar/go/1.18/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/homebrew/Cellar/go/1.18/libexec/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/andig/htdocs/evcc/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/sv/rs_453y57xj86xsbz3kw1mbc0000gn/T/go-build1274200129=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.18 darwin/arm64
GOROOT/bin/go tool compile -V: compile version go1.18
uname -v: Darwin Kernel Version 21.4.0: Mon Feb 21 20:36:53 PST 2022; root:xnu-8020.101.4~2/RELEASE_ARM64_T8101
ProductName:	macOS
ProductVersion:	12.3
BuildVersion:	21E230
lldb --version: lldb-13<IP_ADDRESS>
Apple Swift version 5.6 (swiftlang-<IP_ADDRESS>3.62 clang-13<IP_ADDRESS>)

What did you do?
Repo at https://github.com/evcc-io/evcc/commit/6f2abf4a639bb5a48a3f18782a7adf58c1ee77c0:
go get github.com/panta/machineid

go.mod requires 1.17 at this point.
What did you expect to see?
Single require section.
What did you see instead?
2nd require section is added in go.mod:
require github.com/panta/machineid v1.0.1 // indirect

Replacing
github.com/denisbrodbeck/machineid

with
github.com/panta/machineid

in the code and running
go mod tidy

does not remove the 2nd require section.
UPDATE I think the root cause of

I think it's due to the require sections being impure

is in https://github.com/golang/go/issues/51983#issuecomment-1098379622.

Interesting. When I
go mod tidy

before the go get, go-pkcs7 is not being moved. Wondering why it's in the indirect section?

Notice that https://github.com/golang/go/issues/52296 has a flow to produce a go.mod with two separate require blocks where the latter contains not only indirect files. Looks like an issue in go mod.

Notice that #52296 has a flow to produce a go.mod with two separate require blocks where the latter contains not only indirect files. Looks like an issue in go mod.

Ah, yes, I noticed that as well; looks like a bug

That seems like a different issue than the one originally reported here

@seankhliao I think that‘s actually the root cause of this issue as it creates the impure section. #52296 is about module graph evaluation but it does accidentally show how to create an impure section.

@andig, could you summarize the repro steps here? (But, to set expectations: if they involve go mod edit, that's probably not going to be worth fixing. go get and go mod tidy are the supported ways to edit dependencies with correct direct/indirect markings. 😅)

It is indeed caused by go mod edit:
mkdir foobar && cd foobar

cat > main.go <<EOF
package main

import (
    "github.com/sirupsen/logrus"
)

func main() {
    logrus.Info("Hello foobar")
}
EOF

go mod init foobar

go mod edit -require<EMAIL_ADDRESS>go mod tidy

cat go.mod

cat > main.go <<EOF
package main

import (
    "github.com/containerd/containerd/pkg/apparmor"
    "github.com/sirupsen/logrus"
)

func main() {
    if apparmor.HostSupports() {
        logrus.Infof("Running Foobar Deluxe, with AppArmor")
    } else {
        logrus.Infof("Running Foobar Basic")
    }
}
EOF

# >>> the next step causes the problem
go mod edit -require<EMAIL_ADDRESS>go mod tidy

cat go.mod
module foobar

go 1.18

require github.com/sirupsen/logrus v1.8.1

require (
	github.com/containerd/containerd v1.6.2
	golang.org/x/sys v0.0.0-20211216021012-1d35b9e2eb4e // indirect
)

@andig, we've rewritten the direct-organizing logic at least twice now to try to fix the heuristic. (It intentionally doesn't mess with existing block structure, in case projects have their own block structure they're trying to maintain.)
I think the only viable improvement would be to give up on maintaining existing structure entirely. That would need a full proposal (since it would be a visible change in behavior), but even then there are all kinds of thorny issues around comment handling that I really don't have the bandwidth to address. 😩

No problem. Then let‘s just close this issue?


Images inserted in the markdown editor should be wrapped in an anchor tag leading to the original file

When inserting an image in the WYSIWYG editor a link to the original image file is added, it would be useful if the markdown editor did the same.
I'm just starting to migrate my content to BookStack, this feature would save me a lot of manual editing and copying and pasting URLs.
I've looked at the source; mind if I have a crack at implementing it? Anything special? Should a link be added even if no larger version of the image exists?

Now merged, To be in next minor release.


deleting an account removes all content

ISSUE TYPE

Improvement Request

CLOUDSTACK VERSION

4.19.1

SUMMARY

If someone deletes an account (e.g., accidentally), this will delete all content, including backups (volume snapshots), without any confirmation. After that admin will not able to restore anything

sorry, @top-secrett I converted this to a q&a discussion too eagerly. This reads like a complaint, I think you want to do a propose, however. Can you reformulate as an improvement request. The behaviour you see now is there, but ...

are you using any workaround to deal with this
how is that inconvenient
how would you want functionality to be implemented
preferable formulate it in  a scrum fashion:
As a user I want/do ...
As an operator I need ...
etc.
thanks and sorry for the over hasty reaction.


Which slime chunks are true slime chunks?

Hello, we recently used Chunker to convert a world from Bedrock -> Java edition. I simply wanted to inquire about the "slime chunks" inside of the world. Do the slime chunks stay the same on the world before it was converted, do they become slime chunks on the version after it was converted, or do already generated chunks retain "slime chunks" from the old version before the conversion?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9f2fb27f-0ce1-9541-e617-7fe8148f8297
Version Independent ID: 6777fa19-93ad-1984-5fd8-a9adc3dc3a3a
Content: Chunker - Overview
Content Source: creator/Documents/ChunkerOverview.md
Service: minecraft-bedrock-edition
GitHub Login: @neonerz
Microsoft Alias: mikeam

Hi! Slime chunks are based on the seed and not saved to the world, so it would use slime chunks specific to Java edition, which I believe will be different to Bedrock.
Hopefully this helps :)

I can confirm that there are no slime chunks in Bedrock. Thanks, @FormallyMyles !


minio: add warning about pre-existing files

This is somehow an obscure thingy. MinIO is a file storage with an S3 compatible API, so we support it.
If you set up a bucket on a directory with information that was not created through the API, let's say:
mkdir /tmp/bucket
minio server /tmp/bucket
echo "foo" > /tmp/bucket/foo

It will use a default ETag:
https://github.com/minio/minio/blob/d7060c4c32b0fd0672d8a58abcfe54473b49fa39/cmd/fs-v1.go#L45-L46
This will not work with DVC, since two objects with different content could have the same ETag.

Interesting find @mroutis
Not sure it's worth noting in the docs. What DVC commands does it affect?
Maybe just leaving a workaround here (can you think of one?) and resolving the issue will be good enough for any user who comes across this problem to find this conversation.

@jorgeorpinel it affects almost every command that deals with cache :sweat_smile:
A workaround is to download everything and use mc (minio cli) or aws (cli) to sync the directory again. At least that's what I did.
Let's close it, then :)
Also, another point for #499 , we could add notes like this one (about the idiosyncrasies of each remote interacting with DVC)

The problem with #499 is that the command reference lists commands, not other kinds of things (e.g. remote types), so both refs for remote add and for remote modify have expandable examples for all the remote types. Would have to have a 3rd level in each of those with repetitive docs per remote type.
But if you can come up with a good way to reorganize all this that makes sense and addresses your concerns, please reopen either issue or open a new one. Thanks!

Idea: a new use case or user guide to explain these remote type-specific caveats.

two objects with different content could have the same ETag.

@mroutis @efiop it might be a bigger problem than just docs.

it affects almost every command that deals with cache

I think it affects only external dependencies/outputs, regular stuff like push/pull should work anyway.


Implement Java and Kotlin versions

I mentioned in the post that this could also be implemented in Java... while the implementation is the same it might be an interesting demo to do a Java and Kotlin version of both the client and tests.
The class names would need to be different just to keep it simple and all in one project, maybe: JavaUser and KotlinUser, etc.

Done


Problems with the config

Hello everyone, I can install the module and the authentication also works. As soon as I want to adjust the config for my interest, the MM goes into error. How did you customize your Conifig so that you can display it?
Thank you for your support

Can you share your MMM-OnSpotify config?

Remember to add the comma after the default config:
        {
            module: "MMM-OnSpotify",
            position: "bottom_right",
            config: {
                clientID: "x",
                clientSecret: "x",
                accessToken: "x",
                refreshToken: "x-x-x-x-x-x-x", // <--- Add a comma here
                /* Config example */
                experimentalCSSOverridesForMM2: [
			["--color-text-dimmed", "palette_vibrantlight",],
			["--ONSP-OVERRIDES-ICONS-COLOR", "palette_vibrantlight"],
		],
            }
        },

Yes, I was able to find the error, but I have no connection to Spotify...

Can you test removing completly the module and installing this version:
git clone --branch v4.0.0 https://github.com/Fabrizz/MMM-OnSpotify.git

Yes, I'd like to try it out, but which command can I use to delete the module?

I was able to delete the module through research, I have now used your version and used the existing client, clietID, acesstoken and refreshtoken. However, I still have no connection to Spotify...

Thats really strange, is your account premium right?

Yes, I have a Premium Family account

Maybe you are having conflict with another module, disable or remove everything but MMM-OnSpotify and check again


fixes AWS only upload and backwards compatability

Hi this should fix #316 and #321 it also goes a step further to ensure that vdropzone-s3-upload-success always emits the URL of the uploaded object so that if you bypass the server upload where it uploads twice, you can still send that URL to your server to save it to a database or attach it to your files.
This PR should be done as quickly as possibly as I think a lot of people are having issues with 3.1 with it basically breaking all the different upload types.

Thanks @gborcherds for tackling this. @vrajroham are you able to take a look at this?

Thanks for looking into it. I'll review tonight and merge it.


merge from arlac77/template-cli-app

package.json

chore(package): add ^1.32.0 remove ^1.31.1 (devDependencies.rollup)
chore(package): add arlac77/template-cli-app (template.inheritFrom)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: make nullable as default when alter table

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Fix bug https://github.com/GreptimeTeam/greptimedb/issues/364
Make nullable as default when alter table.
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Summarize your change (mandatory)
Make nullable as default when alter table
How does this PR work? Need a brief introduction for the changed logic (optional)
Describe clearly one logical change and avoid lazy messages (optional)
Describe any limitations of the current code (optional)

Checklist

[]  I have written the necessary rustdoc comments.
[]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

Thanks every one.
I will add tests to cover this.


created and linked a cep78 tutorial instead of the github. Github is …

created a tutorial for cep-78 using casper-client and javascript SDK

@KMCreatesWorlds to review and add comments

@jonas089 I resolved conflicts that appeared after structure migration :eight_spoked_asterisk:. Side comment: tutorial is not listed in sidebar.

Waiting for resolution of this issue:
https://github.com/casper-ecosystem/cep-78-enhanced-nft/issues/198


Add allowIDFA to react native GA

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have added all necessary documentation (if appropriate)

What was done
Added functionality to allow IDFA collection for the react-native Google Analytics package. This signals to GA whether the app is allowed to use the advertiser identifier for the device to deliver adds or collect ad click attribution.
Not sure how to go about adding a test for this, we've been using this change in production for a while now.

Nice work. I'm curious about something though...are there scenarios where we'd need to set allowIDFA multiple times throughout the lifecycle of the app? I'm thinking that perhaps this is more of a setting rather than an event:
function GoogleAnalytics(
  trackingId: string,
  GoogleAnalyticsTracker: any
  settings?: { allowIdfa?: boolean }
)

I agree with you - the only counter point would be to keep consistency with react-native-google-analytics-bridge?

Hmm...I see...
Honestly, I'd prefer if we remain consistent with our own abstractions. Otherwise all our targets might diverge in methodology/usage. I'd like the experience from target to target to be as similar as possible.

@ttmarek not a problem! I agree with you.


Stylis plugin not working as expected

So I've created a plugin called stylis-plugin-logical to convert logical CSS properties and values to values that have better browser support.
I've tried integrating it with styled-components but it doesn't behave correctly while when I use it with emotion v10 it works fine.
here is the emotion v10 codesandbox and here is the styled-components codesandbox example.
Is there something that I can do in the plugin to be more compatible with styled-components?

I'm having the same problem. It seems to strip out a whole lot of styles.
Did you ever find a solution?

We do have a reproduction here, but no information on the actual issue, expected behaviour, or what specifically is an issue, so due to a lack of time I can't unfortunately go after a vague description here 😅 Sorry!
I'm happy to reopen this, if we have an idea of what's going on, or if we get more details added here 🙌


Feature: sort-imports sort order by group order in line-length

What rule do you want to change?
sort-imports
Describe the problem
The possibility of sorting by group order.
I believe it would be something like: "sort-order-groups": true
sort-order-groups rule with first create after get:
"perfectionist/sort-imports": [
      "warn",
      {
        "type": "line-length",
        "order": "desc",
        "groups": [
          ["create", "get"],
          ["builtin", "external"]
        ],
        "custom-groups": {
          "value": {
            "get": "../utils/get/**/*",
            "create": "../utils/create/**/*"
          }
        },
        "newlines-between": "always",
        "sort-groups": true
      }
    ]

Code example
without sort-order-groups rule, sort this:
import { createEslintRule } from '../utils/create'
import { createEslint } from '../utils/create'
import { getLinesBetween } from '../utils/get'
import { getGroupNumber } from '../utils/get'
import { createRule } from '../utils/create'
import { getNodeRange } from '../utils/get'
import { getGroup } from '../utils/get'

with "sort-order-groups": true rule:
import { createEslintRule } from '../utils/create'
import { createEslint } from '../utils/create'
import { createRule } from '../utils/create'
import { getLinesBetween } from '../utils/get'
import { getGroupNumber } from '../utils/get'
import { getNodeRange } from '../utils/get'
import { getGroup } from '../utils/get'

Additional comments
I used Google Translate for this issue, sorry if there are some grammatical errors 😄
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Thank you for your issue!
Thank you for your idea. I don't think it will be in demand at this time. And at the moment we have no plans to implement this feature.


Can't override IconButton disabled color

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
I should be able to override disabled color.
Current Behavior 😯
It is always rgba(0, 0, 0, 0.26)

Steps to Reproduce 🕹
const styles = theme => ({
  button: {
    width: 8 * theme.spacing.unit,
    height: 8 * theme.spacing.unit,
    color: theme.palette.primary.main
  },
  buttonDisabled: {
    color: theme.palette.grey[900]
  },
  icon: {
    fontSize: theme.spacing.unit * 5
  }
});

<IconButton
  disabled={true}
  className={classes.button}
  classes={{ disabled: classes.buttonDisabled }}>
  <PauseIcon className={classes.icon} />
</IconButton>

Your Environment 🌎

Tech
Version

Material-UI
v3.6.1

React
16.5.2

Browser
Chrome Version 70.0.3538.110 (Official Build) (64-bit)

const styles = theme => ({
  button: {
    width: 8 * theme.spacing.unit,
    height: 8 * theme.spacing.unit,
    color: theme.palette.primary.main,
    "&$buttonDisabled": {
        color: theme.palette.grey[900]
    }
  },
  buttonDisabled: {},
  icon: {
    fontSize: theme.spacing.unit * 5
  }
});

<IconButton
    disabled
    classes={{ root: classes.button, disabled: classes.buttonDisabled }}>
    <PauseIcon className={classes.icon} />
</IconButton>

@joshwooding this hack works.

@Gentlee I believe this is intentional look at overriding internal states: https://material-ui.com/customization/overrides/ and the answer to #13672

@Gentlee Thank you for raising this concern. Why do people have to do:
A.
.icon-button-root.icon-button-disabled {
  color: 'grey',
}

/* or */

.icon-button-root:disabled {
  color: 'grey',
}

<IconButton
  disabled
  classes={{
    root: 'icon-button-root',
    disabled: 'icon-button-disabled',
  }}
/>

when they want to override the icon button disabled style and not just:
B.
.icon-button-disabled {
  color: 'grey',
}

<IconButton
  disabled
  classes={{
    disabled: 'icon-button-disabled',
  }}
/>

If you are using raw CSS and you don't want to repeat the classes, you can use dangerouslyUseGlobalCSS.
This is an explicit design decision, not a bug. It's a very important one. In order to implement B, we need to keep the CSS specificity of the disabled variant at the same level of the default style. Now, let's say you can to customize the non disabled background color. You inject some CSS after the Material-UI CSS. And just like that, you have overriden the diabled style too, you need to reapply all the styles 😱.
I would hande the problem like this:
import React from "react";
import PropTypes from "prop-types";
import { withStyles } from "@material-ui/core/styles";
import IconButton from "@material-ui/core/IconButton";
import DeleteIcon from "@material-ui/icons/Delete";

const styles = theme => ({
  button: {
    "&:disabled": {
      backgroundColor: "grey"
    }
  }
});

function IconButtons(props) {
  const { classes } = props;
  return (
    <IconButton disabled className={classes.button} aria-label="Delete">
      <DeleteIcon />
    </IconButton>
  );
}

IconButtons.propTypes = {
  classes: PropTypes.object.isRequired
};

export default withStyles(styles)(IconButtons);

The topic is partially covered in https://material-ui.com/guides/api/#css-classes and https://material-ui.com/customization/overrides/#overriding-with-classes.
I'm sorry for not making the documentation clear enough. I can understand your confusion. I think that we should update the documentation to better explain it.

@oliviertassinari ok i see that it is a design decision. Seems your decision forces me to know about implementation (CSS), that means it is a bad decision.
I thought it should be like: root default -> custom default [-> root disabled -> custom disabled]. And no side effects like this.
Anyways, i see that you are not going to change implementation, so thanks for the explanation.

@Gentlee Yes, you have to know about the underlying DOM and CSS structure.

I have a similar problem and I've tried to follow along here, but I still can't get the disabled or primary colors to change to my preferences. I'm using an Icon in my IconButton, and it doesn't seem to behave the same way. I have:
const colorDisabled = '#F6F6F6';
const colorPrimary = '#656668';

export const AudioIconStyles = createStyles({
    button: {
       fontSize: 15,
        color: colorPrimary,
        '&$buttonDisabled': {
            color: colorDisabled
        }
    },
    buttonDisabled: {},
    icon: {
        fontSize: 15,
    }
});
export interface AudioProps extends WithStyles<typeof AudioIconStyles> {
...
}
export const Audio = withStyles(AudioIconStyles)(
    class extends React.PureComponent<AudioProps, AudioState> {
        render() {
            return (
               <Tooltip title={stoppedTooltipText} placement='bottom-start'>
                   <IconButton
                       className={this.props.classes.button}
                       onClick={this.onStop}
                       id={this.props.id + '-stop'}
                       disabled={this.state.stopped}
                  >
                      <Icon className={classNames(this.props.classes.icon, 'fa fa-stop')}
                          classes={{colorPrimary, colorDisabled}}
                          color={this.state.stopped ? 'disabled' : 'primary'}
                       />
                  </IconButton>
              </Tooltip>
            );
        }
    }
);

But the primary color and disabled color are always the the ones which come from material-ui:

I have also tried straight inline CSS, so, like this:
<Icon className='fa fa-stop'
style={{fontSize: 15, color: colorPrimary}}
/>
But that doesn't work at all. I don't get the font size nor the color. Could you tell me what I'm doing wrong, please?

@kimwykoff try
button: {
       fontSize: 15,
        color: colorPrimary,
        '&:disabled': {
            color: colorDisabled
        }
    },

@joshwooding, I actually did, but it didn't help. The button stuff seemed ok, although it doesn't seem to do much. The real probably is in the Icon component. I couldn't get the color to set to 'disable'. It just doesn't seem like Icon reacts to any of the styling props you give it.
I finally just removed all the css injection and used simple class names that I defined in my .scss file. I still wasn't getting the disabled color, but I put a '!important' next to my color and it finally works.
The injection stuff is just too time-consuming to figure out. The doc needs a more complete example with explanations.

@kimwykoff Thank you for the feedback. I'm going to update the documentation to better explain the situation.

@kimwykoff I had a look at your example, it can't work. The buttonDisabled rule you are using is creating a corresponding class name, but you are not applying to the DOM.


Fix mismatch of return value of stub

Description
Makes it impossible to return the wrong type from stub.
Checklist

[x] Documentation has been added or updated.
[x] All applicable classes have unit tests.
[x] This PR contains no commented out code.

LGTM, just a few comments. Merge when you are ready.


How do I incrementally compile a port when using '--editable'.

I wanna change some codes in 'openmvg', and compile it many times.
Each time, I remove the port and reinstall it with '--editable'. Is there a more convenient way to do this?

@FishHe, you don't need to remove the port.
You only need to use '--editable' each time, it will use the source you modified.

@FishHe, please reopen this issue if it's not answer you want.

However, using '--editable' each time does not work for me. Vcpkg can't detect my changes on the source code.

That's strange, I double confirm again, it works for me, I installed the port with --editable before, uninstall the port, modify some changes in buildtree, reinstall it with --editable again, the changes still be there.
You can  see it use the source cached before.
PS F:\vcpkg\clean> ./vcpkg install openmvg:x64-windows --editable
Computing installation plan...
The following packages will be built and installed:
openmvg[core]:x64-windows
Detecting compiler hash for triplet x64-windows...
Starting package 1/1: openmvg:x64-windows
Building package openmvg[core]:x64-windows...
-- Note: openmvg only supports static library linkage. Building static library.
-- Using cached F:/vcpkg/clean/downloads/openMVG-openMVG-606d1c9e82123dd50de282128151a50bf42262fc.tar.gz
-- Using source at F:/vcpkg/clean/buildtrees/openmvg/src/0bf42262fc-5796248b75
-- Configuring x64-windows

It is true that making complete recompilation is working with '--editable'. However, recomputing the whole port is time consuming. To incrementally compile a port is what I really want.

@FishHe, you don't need to wait it start to build, end up the thread when it finished extracting source, it should be very quick, also you does't need to uninstall the port again, sorry for the confuse.


[Bug] different select express leading to dozes of times performance difference

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.0.8
What's Wrong?
create table sql:
CREATE TABLE  dws_mediago_bidder_dsp_multi_measures_hourly  (    campaign_id  int(11) NULL,    asset_id  int(11) NULL,    ssp  int(11) NULL,    ad_id  int(11) NULL,    crid  varchar(50) NULL,    domain  varchar(2000) NULL,    ip_country  varchar(200) NULL,    account_id  varchar(32) NULL,    account_name  varchar(500) NULL,    company_id  varchar(32) NULL,    company_name  varchar(500) NULL,    am_name  varchar(50) NULL,    platform_type  varchar(10) NULL,    account_category  varchar(100) NULL,    company_region  varchar(20) NULL,    charge_type  varchar(20) NULL,    target_cpa  DECIMAL(15, 9) NULL,    d_s  date NULL,    h_s  int(11) NULL,    ad_count  bigint(20) SUM NULL,    all_req_num  bigint(20) SUM NULL,    account_gross_click_cost  double SUM NULL,    click  bigint(20) SUM NULL,    click_cost  double SUM NULL,    conversion  bigint(20) SUM NULL,    cv  bigint(20) SUM NULL,    imp  bigint(20) SUM NULL,    imp_cost  double SUM NULL,    vimp  bigint(20) SUM NULL,    mcv  bigint(20) SUM NULL,    flr_sum_fix  double SUM NULL,    bid_price_sum  double SUM NULL,    req_num  double SUM NULL,    prctr  double SUM NULL,    pclick  double SUM NULL,    req_ad_num  bigint(20) SUM NULL ) ENGINE=OLAP AGGREGATE KEY( campaign_id ,  asset_id ,  ssp ,  ad_id ,  crid ,  domain ,  ip_country ,  account_id ,  account_name ,  company_id ,  company_name ,  am_name ,  platform_type ,  account_category ,  company_region ,  charge_type ,  target_cpa ,  d_s ,  h_s ) COMMENT 'OLAP' PARTITION BY RANGE( d_s ,  h_s )() DISTRIBUTED BY HASH( campaign_id ) BUCKETS 36 PROPERTIES ( "replication_allocation" = "tag.location.default: 3", "bloom_filter_columns" = "ssp, domain", "is_being_synced" = "false", "storage_format" = "V2", "light_schema_change" = "true", "disable_auto_compaction" = "false", "enable_single_replica_compaction" = "false" );
first sql:

second sql:

you can see different select express leading to dozens of times performance difference
first sql explain:

second sql explain:

difference:
VOlapScanNode PREAGGREGATION of first sql is off, second sql is on
What You Expected?
1.why VOlapScanNode PREAGGREGATION of first sql is off?
2.is VOlapScanNode PREAGGREGATION the key of performance diff?
3.is there any way to solve it?
How to Reproduce?
create table sql:
CREATE TABLE test2(  k1date NULL,  v2 int(11) SUM NULL DEFAULT "1" ) ENGINE=OLAP AGGREGATE KEY(k1) COMMENT 'OLAP' DISTRIBUTED BY HASH(k1) BUCKETS 32 PROPERTIES ( "replication_allocation" = "tag.location.default: 3", "is_being_synced" = "false", "storage_format" = "V2", "light_schema_change" = "true", "disable_auto_compaction" = "false", "enable_single_replica_compaction" = "false" );
explain select sum(if(k1='12',v2,0)) from test2;
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

table is aggregate model. when pre-aggregation mode is OFF, storage layer will merge tuples of same key into one tuple before return data for execution layer.
when pre-aggregation mode is ON, storage will do nothing.
for example, we have a aggreagate model table t1 with
c1 key,
c2 int sum

and then insert data into t1 with
insert into t1 values(1, 1), (1, 2)

when pre-aggregation mode is OFF, execution layer get
1, 3

but when pre-aggregation mode is ON, execution layer get
1, 2
1, 1

So, we set pre-aggregation to ON only when we can ensure not merging data will not leading to wrong result. Currently, we could only support the aggregation function exactly same with aggregation function in table. We will support more pattern in future.

you can upgrade your doris to 2.1 latest version. we have fixed this in 2.1 version. for 2.0 version, there is too much change, so we didn't merge this.
https://github.com/apache/doris/pull/34738

thank you for your answer


Multi-GPU inference problem.

File "/slurm-files/ock/fsar_llava/fsar_llava/videollava/model/llava_arch.py", line 207, in prepare_inputs_labels_for_multimodal
video_features_minibatch = self.encode_videos(videos_minibatch) # fake list [mini_b, t, l, c]
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:3 and cuda:0! (when checking argument for argument mat2 in method wrapper_CUDA_mm)

Same issue, have you solved it yet？

Modifying device_map and device gives the following result


Wishlist

Closes #334

@lauraluiz Using Wishlist when the type is a ShoppingList didn't feel right, so let's discuss if we both can come up with better names 😉

My quick thoughts on that (without checking the code):
Shopping list is the general term for all kind of lists: wishlist, wedding list or customized lists (e.g. "review later" list).
With that being said, it depends on the solution, whether it's too "wishlist" focused or not.
And now I check the code ;)

@lauraluiz I'm now done with the changes. I'm looking forward to your review 😄

@lauraluiz I applied all requested/discussed changes 😄 Looking forward to your next review 😉

@lauraluiz I applied the changes we discussed and added the improved pagination test. I'm looking forward to your code review 😉

I created a PR for sunrise-theme, when you approved the PR and the webjar is published I will then remove the templates from this PR :-)


Use GitHub's alert syntax

This is just a small thing, but it looks like GitHub has made their alert syntax official.

I love it, thanks for sharing this information @spenserblack I updated the Readme.md to make use of it 8a5cf38

Unfortunately, it seems like Markdown Alerts are a Github only feature. It doesn't display correctly on crates.io and vscode for example. I will revert 8a5cf38

Ah, yeah, I agree. This syntax can really only be used in GitHub-only files.


Search's Broken

Works on desktop now. Just change to filterstring on mobile and you're good.

Bloop! 😊


Opening .bat file in VSCode via context menu in windows explorer tries to execute it

VSCode Version: Code 1.19.3 (7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba, 2018-01-25T10:36:43.673Z)
OS Version: Windows_NT ia32 10.0.16299
Extensions:

Extension
Author (truncated)
Version

cmake-tools-helper
mad
0.1.1

cpptools
ms-
0.14.6

csharp
ms-
1.13.1

SystemVerilog
msh
0.0.17

cmake
twx
0.0.17

cmake-tools
vec
0.10.4

Steps to Reproduce:

right-click on .bat file in explorer (unpacked from downloaded .zip file)
select "Open with code"
get "Windows protected your PC" screen ("prevented an unrecognized app from starting")

So it seems that VSCode or the Explorer wants to run the .bat file instead of opening it for editing.  That's definitely not the behavior I expect when I want to inspect a batch file before executing it. (Opening the file via the File -> Open dialog in VSCode works fine. My system's security wasn't compromised because the .bat file was from a trusted source, but it sure looks like a security issue.)

I don't think it's a bug. It's just what SmartScreen does. It's being extra cautious and runs the check before any action - including the built-in Edit command. Then it starts the editor (even though the SmartScreen wording might suggest it runs the script).

Interesting didn't notice that, probably i just used less on a command line to view the batch file.
So it's a UI bug ("issue") in Windows. It's  kind of self-defeating if SmartScreen makes you believe that you're about to execute a file when you're trying to look at it in order to decide whether to execute it.
I'm leaving this open, as there may be a way to register "Open with Code" with the Explorer that doesn't trigger this warning, but please feel free to close if you consider it outside VS Code's scope.


bug in get_pandas_dataframe() function

For some runs with more than one config space, the get_pandas_dataframe() will carry out the hyperparameters of the last one for all three (in the screen shot for example)
In the screen shot, the run id (0,0,12) has 3 runs with budget 74, 222, 666

But when I get the data using get_pandas_dataframe(), it only gives the information of the last config space with budget 666 as you can see in the fourth column.

Hmmmm...that should only happen if you specify the budget argument of `get_pandas_dataframe'. Did you do that, by any chance? Otherwise, could you provide a minimal working example?

I didn't specify it. Also I am sorry I do not have much time to create a working example. However, I looked at the source code briefly(I am not 100 percent sure), is the problem on this line
all_runs = self.get_all_runs(only_largest_budget=False)
where only_largest_budget should be True?

If you set this flag to true, then it should only give you the largest budget, which is not what you want. I still don't really see where the problem could be...

If you set this flag to true, then it should only give you the largest budget, which is not what you want. I still don't really see where the problem could be...

X = res.get_pandas_dataframe()[0]
y = res.get_pandas_dataframe()[1]

these are the two lines I use after running the hyperparameter tunning. I don't  think I did something unexpected on my end tho

I have same error.
get_pandas_dataframe()[0] config is not exact ... some row copyed another config row


feature: add projects panel to footer

This is a problem with creating branches in yekaterinburg.design.
The changes are posted here:
https://edesign-feature-update-footer.vercel.app/

Strange :hover effect 👽


Use idiomatic code for Kotlin Gradle Plugin template

The current template of the functional test includes unused imports, syntactic redundancies such as semicolons, and straight up hacky code:
    @TempDir
    @JvmField
    val tempFolder: File? = null // <-- `val` with initializer that will get reassigned during runtime

This PR cleans up the code and makes it more idiomatic.

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

OK, I've already triggered a build for you.


Investigate Usage of Self-Hosted Renovate Runner for Registry

Which area/kind this issue is related to?
/area ci
/area registry
Issue Description
As part of https://github.com/devfile/api/issues/1133 there is an option to investigate the usage of a self hosted bot for renovate instead of using the mend.io runner. The renovate runner can be found here: https://github.com/platform-engineering-org/renovate-runner. As a result it would be good to try this renovate runner against a forked repo of devfile registry.
Target date: 30 July 2023

I'll try to test the self hosted runner against my registry fork in order to check the differences between mend.io and the self hosted format.

I've added a PR in the self-hosted runner repo in order to include my fork of the registry repo. If this is merged we can see in action the results for this automation

The PR was merged from the platform engineering team and we have the first run here: https://github.com/platform-engineering-org/renovate-runner/actions/runs/5691571384/job/15426998337
I've added to our renovate.json the shared config to check how renovate behaves with shared and custom config.
In general, I've noticed some things changing:

The self hosted config provides a dependency dashboard (an issue created like https://github.com/thepetk/registry/issues/53). There we can have an overview of all updates handled by renovate-bot. Ofc, this feature is available to the mend.io runner too.

In the shared config we merge all changes to one PR (so there will be one pr with all registry image updates). I'd say for all other repos is fine to have this behavior, but as registry updates demand a lot of testing, it might be good to have one PR per image update.

Thank you for testing it. For awareness, those settings can be overriden/customized.


Ensure we don't pre-allocate data segments and then not use them

We were pre-allocating data segments based on the DataCount section that we
parsed, but then only re-using those pre-allocated data segments for passive
data segments. For active segments, we created a different structure inside the
module's Memory. Then, when we emitted the module again, we would emit
duplicate data segments: both the unused pre-allocated segment and the segment
in the module's Memory. Add in the interaction with data.drop and which of
the duplicate data segments does it reference, and you get bugs!
Thanks wasm-opt -ttf for finding this bug for us :)
Anyways, this commit refactors the way we do data segments (in a breaking way!
we'll need to bump the major version on the next release) so that ModuleData
contains all of the data segments, not just the passive segments. And memories
just reference the active data segments that affect them, rather than owning
them. No more duplication and splitting where data segments live depending on
what kind of data segment they are!
Fixes #109

er oops spec-tests are failing; need to clean this up some more.

Green! \o/


Tree Problems and Algorithms in python

I want to work on most of Tree algorithm and its problems . Please give it a hacktoberfest tag and assign this to me

ok, just make a different folder inside the repo and raise the issue with the list of questions.


Multi-line completions

I've been testing this plugin on and off. I'm mostly interested in Cody. On the whole I'd say the plugin experience is more cohesive than what's offered by currently available Copilot plugins for neovim.
There's just one thing that stands out as lacking: multiline completions, i.e. suggested completions can span multiple lines. Today, each line needs to be completed indivudally. For a long block of code, that translates to a lot of completion steps.
Judging by this comment under #171, I'm guessing this is something the plugin author(s) are already aware of. Just seemed handy to have a clearly named issue where I and others can follow the status of this feature's development. Feel free to close it as a duplicate, however.

Also noticed this, could just be my config though, using cody with cmp.nvim
It would be good to see a more detailed example of how to get multiline completion working if they are supposed to be in neovim

With the latest updates, multi-line completions will trigger for certain filetypes. After #243 , we should see that expand as we'll be picking up more options from cody.
I'll close for now because multi-line completions are supported for many of the basic cases (you'll have to accept the completion to apply the multi-line completion)


Ensemble support

The intial model (#2) will have been implemented as a single simulation (CUDASimulation).
As a stochastic simulation, running the simualtions many times would be required for useful scientific outputs.
FALME GPU includes an interface to do this, with opportunity for performance improvements (and/or simplified running of the model).
This will likely require CLI change(s), and some refactoring.
The same total simulations would be possible without this, but the ensemble interface may offer improved performance and/or simplify running simulation batches.

Switched to agent initialisation via an init function in #24, which is a pre-requisite of ensemble support.
CLI changes still needed, and anonymous namespace members used to store data between simulations will need adjusting to do so in a thread-safe fashion.