Create ashleywilliams

Hi @karasowles - it looks like we're missing the images for Ashley Williams. Documentation to do this can be found on our Readme file:
https://github.com/github/globalmaintainersummit.github.com#step-2---profile-image


MbppPlus has problems in Reference Solutions

Mbpp/459
Problem Description
Write a function to remove uppercase substrings from a given string.
assert remove_uppercase('cAstyoUrFavoRitETVshoWs') == 'cstyoravoitshos'

Reference solution
# wrong version (Now)
def remove_uppercase(str1):
  return ''.join(c for c in str1 if c.islower())
# remove_uppercase("Hello World") = "elloorld"

Correct reference solution
# correct version
def remove_uppercase(str1):
    return ''.join(c for c in str1 if not c.isupper()) # just need to remove uppercase characters
# remove_uppercase("Hello World") = "ello orld"

cc @ganler

Yes this GT is wrong and needs to be fixed.
Also, let's acknowledge https://github.com/crux-eval/eval-arena for the finding.

Mbpp/581
Problem Description
Write a python function to find the surface area of a square pyramid with a given base edge and height.
assert surface_Area(3,4) == 33

The Height should indicate the perpendicular distance from the base to the apex of the pyramid, instead of the slant height.

But our Reference solution is
# wrong version (Now)
def surface_Area(b,s):
    return 2 * b * s + pow(b,2)

Correct reference solution
def surface_Area(base_edge, height):
    slant_height = math.sqrt(pow(base_edge / 2, 2) + pow(height, 2))
    return 2 * base_edge * slant_height + pow(base_edge, 2)

Mbpp/581
Problem Description
Write a function to convert a snake case string to camel case string.
assert snake_to_camel('python_program')=='PythonProgram'

The example in the description is incorrect. For "Camel Case", the first word does not have its initial letter capitalized.
New Description should be:
Write a function to convert a snake case string to camel case string.
assert snake_to_camel('python_program')=='pythonProgram'

Also the reference solution
# wrong version (Now)
def snake_to_camel(word):
    return ''.join(x.capitalize() or '_' for x in word.split('_'))

Should be correct as:
def snake_to_camel(snake_str):
    components = snake_str.split('_')
    return components[0] + ''.join(x.title() for x in components[1:])

Description:
Write a python function to find the sum of the per-digit difference between two integers.
assert digit_distance_nums(1,2) == 1

This description does not specify the program's behavior where the two integers have varying lengths. But the "plus_inputs"  includes some cases like this.
Regarding the various model responses to such scenarios:

GPT-4 opts to trigger an assertion error when faced with these situations.
GT disregards the additional digits present in the longer integer.
A number of other models handle this by padding the shorter integer with leading zeros.

We could add special logic in evalplus or just remove this problem

Mbpp/87
Problem Description
Write a function to merge three dictionaries into a single dictionary.
assert merge_dictionaries_three({ "R": "Red", "B": "Black", "P": "Pink" }, { "G": "Green", "W": "White" },{ "O": "Orange", "W": "White", "B": "Black" })=={'B': 'Black', 'R': 'Red', 'P': 'Pink', 'G': 'Green', 'W': 'White', 'O': 'Orange'}

This description does not specify the order in which values are updated during a key conflict (dict1 -> dict2 -> dict3 or ...).
Reference solution
# Update order: dict3 -> dict2 -> dict1
import collections as ct
def merge_dictionaries_three(dict1,dict2, dict3):
    merged_dict = dict(ct.ChainMap({},dict1,dict2,dict3))
    return merged_dict

GPT-4's solution:
# Update order: dict1 -> dict2 -> dict3
def merge_dictionaries_three_4(dict1, dict2, dict3):
    merged_dict = {**dict1, **dict2, **dict3}
    return merged_dict

We could add special logic in evalplus.

Mbpp/559
Problem Description
Write a function to find the sum of the largest contiguous sublist in the given list.
assert max_sub_array_sum([-2, -3, 4, -1, -2, 1, 5, -3], 8) == 7

Reference solution
def max_sub_array_sum(a, size):
  max_so_far = 0
  max_ending_here = 0
  for i in range(0, size):
    max_ending_here = max_ending_here + a[i]
    if max_ending_here < 0:
      max_ending_here = 0
    elif (max_so_far < max_ending_here):
      max_so_far = max_ending_here
  return max_so_far

I think initializing max_so_far to 0 is incorrect, even if description doesn't mention about this.
New version is:
def max_sub_array_sum(a, size):
  max_so_far = a[0] if len(a) else 0 # initialize with the first element
  max_ending_here = 0
  for i in range(0, size):
    max_ending_here = max_ending_here + a[i]
    if max_ending_here < 0:
      max_ending_here = 0
    elif (max_so_far < max_ending_here):
      max_so_far = max_ending_here
  return max_so_far

We could add special logic in evalplus.

Looks good! Note that to align with original MBPP we don't want to modify description as much as possible.


Updates to prevent Swift 5.0 deprecation warnings

I'm using SwiftDigest in my own project; thank you! This PR is to remove some deprecation warnings that cropped up as a part of the Swift 5.0 update.
This does seem to be working properly for me, but I strongly advise some independent testing to be sure I didn't mess something up. 😆

Hi @cliss,
We tried importing your changes in our own PR on SPM support. This branch uses Xcode 11.2.1 with swift 5.1.2.
We did not included it because the following show up:

Still has deprecation warnings in MD5DigestPerformanceTests.swift
Two tests are failing:

SwiftDigest/Tests/SwiftDigestTests/MD5DigestTests.swift:54: error: -[SwiftDigestTests.MD5DigestTests test4KBytes] : XCTAssertEqual failed: ("Optional(819bdd24c8fa9f88b855cb593603d0dc)") is not equal to ("Optional(7052292b1c02ae4b0b35fabca4fbd487)")
SwiftDigest/Tests/SwiftDigestTests/MD5DigestTests.swift:62: error: -[SwiftDigestTests.MD5DigestTests test4MBytes] : XCTAssertEqual failed: ("Optional(17b2895b94697b127abf91be59998e9f)") is not equal to ("Optional(f8a4ffa8b1c902f072338caa1e4482ce)")

It seems your changes are affecting the way the hash is computed, at least when using swift 5.1.2. We don't have the time to look into this so we are staying with the warnings, but we thought you should know since your implementation might be broken if you upgrade to swift 5.1.2/Xcode 11.2.1.
Best,
Stefan


Refactoring: Use boolc instead of IF

Following,
https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#use-xsdbool-to-set-boolean-variables
but note that abapGit targets 702, which does not have xsdbool, so use boolc instead,
/src/apack/zcl_abapgit_apack_migration.clas.abap[210, 7]     - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_object_ddls.clas.abap[264, 11]      - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_object_oa2p.clas.abap[240, 7]       - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_object_sprx.clas.abap[300, 7]       - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_object_vcls.clas.abap[177, 7]       - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_object_wdcc.clas.abap[305, 7]       - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_object_xslt.clas.abap[159, 7]       - Use boolc instead of IF (use_bool_expression) [E]
/src/objects/zcl_abapgit_objects_program.clas.abap[434, 7]   - Use boolc instead of IF (use_bool_expression) [E]
/src/syntax/zcl_abapgit_syntax_highlighter.clas.abap[212, 7] - Use boolc instead of IF (use_bool_expression) [E]
/src/ui/zcl_abapgit_popups.clas.abap[940, 7]                 - Use boolc instead of IF (use_bool_expression) [E]
/src/utils/zcl_abapgit_adt_link.clas.abap[194, 11]           - Use boolc instead of IF (use_bool_expression) [E]
/src/zcl_abapgit_merge.clas.abap[379, 7]                     - Use boolc instead of IF (use_bool_expression) [E]

I'm not convinced this is such a great idea. IMHO boolc is an abomination and should NEVER be used. SAP made a big mistake that boolc returns a string when abap_bool is a char.
Thus mistakes can easily slip in, especially with the false sense of security that comes with such a seemingly simple refactor. According to the following, pigs can fly:
    DATA(foo) = 'Blah'.
    IF boolc( foo IS INITIAL ) = abap_false.
      WRITE / foo.
    ELSE.
      WRITE / 'Pigs can fly'.
    ENDIF.

With xsdbool, pigs stay firmly on the ground. So I'd rather use the traditional IF for versions < 7.4 (I think ?) and only use the functional style if xsdbool is available. SAP also recommend xsdbool in the doc.

in all the above examples, the result of boolc is assigned directly to a variable, which will make the conversion happen

Agree, and if the abapLint catches bad type comparisons that would be good. I also think thatboolc will be OK most of the time as best practice is to formulate conditions as positive. But when refactoring, someone may just refactor it to the problematic = abap_false to fit in with an existing code block.
Also, not everything gets linted, and I still think it's way too easy to get into a habit of using boolc and would rather discourage anyone from using it for conditions. It's just my opinion of course.

yea, boolc has some problems, but so far there is 96 usages of it, https://github.com/larshp/abapGit/search?q=boolc&unscoped_q=boolc, and I don't remember we have hit the conversion problem
In the ideal world we could use xsdbool, but we wont get there until the automatic downport is running. And I think its safe to automagically upport boolc to xsdbool, assuming there are no bad usages of boolc
So for now I think its more important to reduce the amount(lines) of code.

Idea: Since a comparison to 'abap_false' is the most problematic one, might it be worth the linter checking for this?
IF(.+)boolc\((.+)= abap_false\.

I was thinking more of when it would make sense to use boolc or xsdbool in an IF condition?
IF boolc( foo IS INITIAL ) = abap_false. should be IF foo IS NOT INITIAL.

I also agree that a comparison to abap_true/false is kindof redundant, perhaps my example isn't the best.
If I generate the following pig_can_fly method definition via quick fix then ADT defines result as a string and if I'm a hasty dev who doesn't check their code or I get interrupted by someone bringing cake into the office then we end up with flying pigs:
  METHOD pigs_can_fly.
    result = boolc( 1 = 0 ).
  ENDMETHOD.

  METHOD run.
    IF pigs_can_fly( ) = abap_false.
      WRITE / 'Correct result'.
    ELSE.
      WRITE / 'Pigs can fly'.
    ENDIF.
  ENDMETHOD.

Of course this is a simple example, and abapGit is generally made up of pretty decent code. But it's just the type of thing that can creep in when refactoring.

PS. All of this is a good case for strong typing :-)

I am not using the quick fixes for magic variable declarations again after getting burned by too short PACKED types and date constants being generated as decfloat16.
In my opinion just disallow boolc completely in conditions should be enough.

Yes, always check those quick fixes. They are right 80% of the time but I find they still save a bit of time when they're not. Ctrl-F1, F3 to jump to the result.


A question about your method

Hello,
I have a question after reading your paper. Does your defense require knowing the attack type in advance? PointCRT needs to train a classifier to distinguish clean and backdoor samples, making it attack-dependent. However, I think such a defense assumption is unreasonable.
Could you help me solve this issue?  Looking forward to your reply.

@Wilmido @ycz11 any updates?

Thanks for your interest！
Your point is very valid, but PointCRT does not require the knowledge of the backdoor attack types in advance.
The purpose of training a classifier is to let the classifier know what a clean sample is, and all other samples are considered as backdoor samples. So we don't need to known backdoor samples as positive samples. As shown in Figure 6, we have actually conducted transferability experiments, and concluded that classifiers trained by transformation-based backdoor triggers as known backdoor attacks exhibits excellent transferability.
I hope the above response is helpful to you.

Thanks for your response! Since I just had a quick look at the paper, I may have missed some details.
I have one more question: In your main experiments instead of the ablation study, do you train a series of classifiers on each to-be-detected attack or train a classifier on a default attack type and then detect all attack types?

We rigorously conducted the main experiments on TeCo and SCALE-UP following their original repositories. They are conducted by the known backdoor attack. To be honest, we do not notice this problem. However, this setting ensures the fairest comparison with these methods. As you can see, directly applying their predefined thresholds from 2D images domain would be much more inappropriate.  But, I personally, also agree that this seems tricky.

@Wilmido
Thanks for your response and sorry for the late reply.
If I understand correctly, both TeCo and SCALE-UP claim that they do not have any prior information about the backdoor attack. They should just need the clean samples to determine the detector's threshold. I haven't looked at their code yet,  do you mean they also need known poisoned samples to determine the threshold?
Please correct me if I'm wrong. Thank you.

Indeed. The ROC evaluation code, including both TeCo and SCALE-UP, compares the predicted labels with the ground-truth labels  (whether it is a backdoor sample) by employing corresponding functions sklearn.metrics.roc_curve to obtain thresholds.
from sklearn import metrics
fpr, tpr, thresholds = metrics.roc_curve(y, pred, pos_label=1)
print(metrics.roc_auc_score(y, pred))

So, you won't find the definition of "thresholds" in their repository codes because they are not manually selected! Our approach, in contrast, involves training a classifier to detect backdoor samples, eliminating the need for hyperparameter selection. However, it must be acknowledged that the requirement for a backdoor attack to train it is unavoidable, as you pointed out earlier.

Ok, I see. Thanks for your kind reply.


ManyToMany don't dipslay (all tags) in list datatable

CF down

        ->add('tags', TextColumn::class, ['label' => 'Tags','field' => 'tag.name','render' => function($value, $context) {
            $str = '';
            foreach ($context->getTags() as $key => $tag) {
                $str .= ''.$tag->getName().'';
            }
            return $str;
        }])
        ->createAdapter(FetchJoinORMAdapter::class, [
            'entity' => Client::class,
            'simple_total_query' => false,
            'query' => function (QueryBuilder $builder) {
                $builder
                    ->select('e')
                    ->addSelect('tag')
                    ->from(Client::class, 'e')
                    ->join('e.tags', 'tag')

                ;
            },
        ])

I try this code, it's better.
Example: First item have 2 tags (red, blue) and second item have 2 tags too (red, green)
If no search, both items are display with 2 tags.
If i search "red", both items are display but only red tag is display.
It's possible to display all tags when i search "red"?

Nobody? =P

Up?

Hi ArmandArthur,
did you succeed ?


Broken 404 page for docs.docker.com

Earlier a missing page(404) redirected to the homepage, now it shows the following error :

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

We have CI for docs now to prevent this to happen. Closing as fixed. Thanks for the report.


Cannot create a folder or a file in any of KnownFolders' locations.

Describe the bug
Trying to create a file or folder in any location accessible via KnownFolders throws exceptions like,
"could not be found. (0x80030002 (STG_E_FILENOTFOUND))" and
"The system cannot find the file specified. (0x80070002)".
This can be easily reproduced by creating a simple new blank WinUI3 project and adding the following line of code.
await KnownFolders.SavedPictures.CreateFolderAsync("MyFolder", CreationCollisionOption.ReplaceExisting);
await KnownFolders.SavedPictures.CreateFileAsync("MyFile.txt", CreationCollisionOption.ReplaceExisting);
But the same lines of code works with ApplicationData.Current.LocalFolder instead of any KnownFolders' locations.
Steps to reproduce the bug

Create a new "Blank App, Packaged (WinUI 3 in Desktop)" project.
Add the following lines of code in the myButton_Click event in MainWindow.xaml.cs.
await KnownFolders.SavedPictures.CreateFolderAsync("MyFolder", CreationCollisionOption.ReplaceExisting);
await KnownFolders.SavedPictures.CreateFileAsync("MyFile.txt", CreationCollisionOption.ReplaceExisting);
The first line above would throw "The system cannot find the file specified. (0x80070002)".
The second line above would throw "could not be found. (0x80030002 (STG_E_FILENOTFOUND))".

Expected behavior
Should be able to create a folder or file without any exceptions when the values are valid in any location accessible via the KnownFolders.
Screenshots

NuGet package version
Windows App SDK 1.4.1: 1.4.230913002
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
The same APIs work fine in a UWP project.

The difficulty here is what are you expecting with the behaviour of Application.LocalFolder.
The documentation for ApplicationData.Current states:
"Provides access to the app data store associated with the app's app package."
So, what is this data store and how does it relate to the various directories in the user profile. Well, let's use a simple application to look at these paths.
#include <Windows.h>

#include <winrt/Windows.Storage.h>

int WINAPI wWinMain(_In_ HINSTANCE, _In_opt_ HINSTANCE, _In_ LPWSTR, _In_ int)
{
	auto saved_pictures = winrt::Windows::Storage::KnownFolders::SavedPictures();
	auto sp_path = saved_pictures.Path();

	auto ad = winrt::Windows::Storage::ApplicationData::Current();
	auto lf = ad.LocalFolder();
	auto lf_path = lf.Path();

	return 0;
}

This is a C++ desktop application which is nice and simple. It queries the path of the saved pictures library and then queries the path of the current application directory. First, and most interestingly, if this is run from an unpackaged application, ApplicationData.Current fails with the error 0x80073D54, "The process has no package identity." Already, this would hint at ApplicationData.Current pointing to a different directory to the saved pictures library. The saved pictures library ends up being:

Okay, so lets package the application and see how things change. Saved pictures has the same path as above. The application has a package identity so ApplicationData.Current works. But wait:

That path is a little different from the saved pictures path. So for this package, the full path that this is pointing to is:

Unfortunately, your success case here seems to be a misunderstanding because you are creating directories and files in a completely different location than you think. While this is a non WinUI 3 C++ project, this should be irrelevant in this case. This is how packaged applications work. The pictures library doesn't have permission for packaged applications to access anything under it by default, where the application package data directory does.
Finally, there is a lack of information on how your project is configured. You state that it is packaged, but as the KnownFolders.SavedPictures documentation states, the application manifest must specify the pictures library capability if you want to access it. However, you do not show your Package.appxmanifest file (or AppxManifest.xml file if you are manually using makeappx).
Do you have:

selected? If you are manually packaging the application, then you should look for:
  <Capabilities>
    <!--Other capabilities here-->
    <uap:Capability Name="picturesLibrary"/>
    <!--Other capabilities here-->
  </Capabilities>

If you don't have the pictures library capability enabled, do things change if you enable it?

I wanted to create folders and files within the PicturesLibrary which I'm unable to do with the KnownFolders API. I've added the capability to access PicturesLibrary already but I missed out to post that here. It didn't change anything though.

I have to admit, that is very surprising.
https://github.com/microsoft/WindowsAppSDK/assets/52577874/c8862db3-d9c4-4b60-9112-cada778999d4
With that capability, I have no issues.

I wonder why it doesn't work for me. I have the latest version (1.4.231115000) of Microsoft.WindowsAppSDK installed and the issue still exists. Also, this works fine for me in a sample UWP app. The issue is only with WinUI3 app. I've noticed that though the app has capabilities added, it doesn't show up in the list of apps that can access the pictures Library in the privacy settings of the PC. I added picturesLibrary capability to a sample UWP app, and it shows up in the list but the WinUI3 app didn't.


Avoid compiler and dialyzer warnings

The function spec used for Console.halt/1 is missing the :ok result that is returned when 0 is passed as the error code.
The deprecated System.stacktrace/0 function was being used instead of __STACKTRACE__/0.

Ping

@bitwalker I've also added a modification to replace System.stacktrace/0 with __STACKTRACE__/0, as the former as been deprecated for a long time and raises a lot of warnings with Elixir 1.11.

Thanks!


Search appends same GET Params over and over

What steps will reproduce the problem?
Search form without manually placing the action url via Url::to()
What is the expected result?
replacing the get param with the new param
What do you get instead?
Appends the same get param over and over
Additional info
If you do not place the action url like
 ActiveForm::begin(['method' => 'get', 'options' => ['data-pjax' => true]]);
it will append the get over and over however, if you do
 ActiveForm::begin(['action' => Url::to(['/catalog/index']), 'method' => 'get', 'options' => ['data-pjax' => true]]);
it will not.  If you inspect the form both output the same url.  My form is also wrapped in a standard pjax widget like Pjax::begin(['id' => 'catalog']);.

Q
A

Yii version
2.0.10

PHP version
7.0.10

Operating system
Windows 10

can you provide some arguments why the ActiveForm should filter these by default? And what are the potential risks of doing so?

If you do not place the action url like
ActiveForm::begin(['method' => 'get', 'options' => ['data-pjax' => true]]);
it will append the get over and over however, if you do
ActiveForm::begin(['action' => Url::to(['/catalog/index']), 'method' => 'get', 'options' => ['data-pjax' => true]]);
it will not.

This is expected because the form will send to the current URL by default, which includes some get params. As it is a 'get' form, it will append to the current url. If you specify the action of the form, the params are not included.

I would think it's not intended.  If you search "name" it will append "name=test" if you clear and search again (not refershing the page.  i.e. pjax)  it will do "name=test&name=test1" and so forth.  If you manually put the action URL  it will leave all other get params in tact while not appending the same params over and over.
Depending on what your server setting are you can get errors for too long of url.  I actually only noticed this because I have a catalog page and i filtered based on  about 10 mfgs (checkbox list).  I unchecked the 10 mfg. then checked about 10 more and hit the url error.
my url was something like.  http://localhost/yii2/catalog?CatalogSearch[company_id]=
each check of a mfg filter adds "CatalogSearch%5Bmfg%5D=" multiple that by 20 and it's pretty nasty looking as well as really long (potentially throwing the apache error).
If you unchecked then recheck a new filter it just appends leaving the old filters.  There is no behavioral differences and the old params are not even passed to the action they are just left in the url.

I know, there is just no way to change this from yii code. The url is specified explicitly in the code generated by gii as this is the way it should be done: https://github.com/yiisoft/yii2-gii/blob/ffefc12a47f9bdabdd53076767b92dbceb6283e9/generators/crud/default/views/_search.php#L23


Implement the whole HashMap interface

First of all, thank you very much for the quick responses and gentle guidance. I also stumbled over the missing len() method, which I implemented for myself in my fork. A lot of these methods are very easy to pass through from the underlying HashMap. Here is a list of all methods missing compared with HashMap.
Methods

[ ] capacity
[ ] clear
[ ] drain
[ ] entry
[ ] get_key_value and then also get_key_value_alt
[ ] hasher
[ ] is_empty
[ ] iter_mut
[ ] keys and then also get_keys_alt
[x] len
[ ] raw_entry maybe not
[ ] raw_entry_mut maybe not
[ ] remove_entry
[ ] reserve
[ ] retain
[ ] shrink_to
[ ] shrink_to_fit
[ ] try_reserve
[ ] values
[ ] values_mut
[ ] with_capacity_and_hasher
[ ] with_hasher

Trait Implementations

[ ] Clone
[ ] Extend
[ ] FromIterator
[ ] Index

Blanket Implementations

[ ] ToOwned

If you have no objections, I would try to implement some of the these, in the coming weeks and create PRs as soon as I accumulated a meaningful contribution. If someone else is working on it, we could check the boxes, to avoid duplicate work

I just bumped into not having clear() which was a little disappointing (though I was able to work around it with mem::replace).
Rather than tackling all of these, maybe it's best to attempt the low-hanging fruit first? I'd probably avoid naively implementing with_hasher because you may want to use different hashers for each underlying HashMap, but things like values_mut should be relatively straightforward. I'd like to PR some of these in, if this project is still maintained.

I just bumped into not having clear() which was a little disappointing (though I was able to work around it with mem::replace).
Rather than tackling all of these, maybe it's best to attempt the low-hanging fruit first? I'd probably avoid naively implementing with_hasher because you may want to use different hashers for each underlying HashMap, but things like values_mut should be relatively straightforward. I'd like to PR some of these in, if this project is still maintained.

That sounds good to me @jeff-hiner :)

That sounds good to me @jeff-hiner :)

Can we change the existing API?
For example changing the version of crate to 2.0


NPC chat System (need help using Tellraw Command)

Hi, there I need some help,
I am trying to make a chat system.
now I am not very experienced just as a note, but everyone has to start somewhere
here is my script
_BaysRiver-Preacher:
    type: assignment
    actions:
        on assignment:
        on enter proximity:
            - random 4
            - chat "&lHello"
            - chat "&lWelcome"
            - chat "&lHi there"
            - chat "&lsorry do I know you?"
    interact scripts:
    - 1 PreacherBaysRiver

PreacherBaysRiver:
    type: interact
    steps:
        1:
            click trigger:
              script:
              - random 3
              - execute as_server "/tellraw @p ["",{"text":"[Preacher] -> You: ","color":"green"},{"text":"Are you looking for work?","bold":true,"color":"green"},{"text":"\n"},{"text":"[","bold":true,"clickEvent":{"action":"run_command","value":"Yes, I am looking for work."}},{"text":"Select","color":"gold","clickEvent":{"action":"run_command","value":"Yes, I am looking for work."}},{"text":"]","bold":true,"color":"gray","clickEvent":{"action":"run_command","value":"Yes, I am looking for work."}},{"text":" ","bold":true,"color":"gray"},{"text":"Yes, I am looking for work.\n"},{"text":"[","bold":true,"clickEvent":{"action":"run_command","value":"No, I'm not looking for right now."}},{"text":"Select","color":"gold","clickEvent":{"action":"run_command","value":"No, I'm not looking for right now."}},{"text":"]","bold":true,"clickEvent":{"action":"run_command","value":"No, I'm not looking for right now."}},{"text":" ","bold":true},{"text":"No, I'm not looking for right now.\n "}]"
              - chat "&7Do you need help?"
              - chat "&7Would you like to learn more about Navi"_

I am trying to use the new tellraw command, and I have tried a few things I think I know what went wrong, but can some one help me out a bit maybe you have a batter idea?

You have double quotes all over that.
You have a few options.

Switch the wrapping Denizen-syntax quotes to single quotes ('),
Change all the internal double quotes to tags (<&dq>)
Use an alternate method like MessageConstructorLibrary ( http://one.denizenscript.com/denizen/repo/entry/90 ) to handle the tellraw more neatly! (I recommend this route)

okay, thank you very much!
your response time was great! and
yeah IK about the double quotes. :P

Actually, I have a quick question how do I use the library?
for example
`%script%      - The prefix used to find your authors
#  %title%       - The title to display in the Header
#  %subTitle%    - The subtitle to display in the Header
#  %command%     - The command to run when the page buttons are clicked
#  %page%        - The current page to display
#  %pageWidth%   - How many characters (roughly) to display on one line
#  %pageHeight%  - How many lines in the body of the message should be on one page
#  %entries%     - The complete contents of the document your are working with.
#                  Every entry in the list is displayed on its own line. Long
#                  entries are automatically linewrapped to fit the box message.
#
#  Usage:
#    - run s@msgBoxed 'def:%script%|%title%|%subTitle%|%command%|%page%|%pageWidth%|%pageHeight%|%entries%' `

I hate to take up to much of your time, but
could you please give me a quick example of usage. in a scrip?

They all have examples and syntax sets. The one you copied is a syntax detail that shows how to use it right there... others are even more clear, EG this set:
#  Example Usage:
#    - narrate "click <&e><proc[msgUrl].context[<&e>this|google.com/search|<&9>click]> link"
#    - narrate "read the <&e><proc[msgcommand].context[<&e>help|help|<&a>click here fool!]> docs"
#    - narrate "hover your mouse on the <&e><proc[msghover].context[<&e>secret|<&6>gold star!]> message"

@TheSodaPopPanda if you would like more assistance using my library, i would suggest opening an issue on my github, or coming to irc for real time help from me.

Oh! wow thank you guys so much! :smile:


Feature/doctor add delete reservation

Following is my PR

List Doctor
Doctor Details
Doctor Add
Doctor Delete
Added redux/reservation/reservationSlice.js
Added redux/reservation/thunk.js

Hi @siddghosh108 ,
Your project is complete!  There is nothing else to say other than it's time to merge it :shipit:
Congratulations! 🎉

Highlights
✅ The API endpoint returning a doctors List
✅One can add a new users from the frontend
✅ The code is top-notch! 💯


fix: make bottom padding consistent for dashboard

when disabling the sections  on dashboard , the bottom padding is also gone

this makes the padding consistent across the disable operations of sections in dashboard

We shouldn't add static padding this way because if the element is visible, it will double the padding. Ex

I will handle this in a different ticket. Basically we have to count the children in css and do:

If first child then add larger padding to top
If last child then add larger padding to bottom
Otherwise add normal bottom padding

true. i will close this in favor


regressioncasetpcds 10t plan shape for external table

Proposed changes
tracking tpcds 10t shape for exteranl table
simulating exteranl table by olaptable,

inject row count
tables are distributed by random

Issue Number: close #xxx

run buildall

run buildall

run buildall

run buildall

run buildall

run  buildall


Add Sentry monitoring of certificate expiry

Signed-off-by: Bernd Verst<EMAIL_ADDRESS>Description
Production users of Dapr need a way to set up alerting for expiring certificates. This PR checks the certificate expiration on an hourly basis and beginning 30 days from expiry on an hourly basis emits a log entry to the Sentry warning logs indicating that the certificate will expire in X hours. Additionally an expired certificate is also logged.
Production users can setup alerting for the specific warning strings from the Sentry service logs.

This is the same code for determining the certificate expiration as is present in the CLI:
See https://github.com/dapr/cli/blob/master/pkg/kubernetes/mtls.go#L168
	secret, err := getTrustChainSecret()
	if err != nil {
		return nil, err
	}

	caCrt := secret.Data["ca.crt"]
	block, _ := pem.Decode(caCrt)
	cert, err := x509.ParseCertificate(block.Bytes)
	if err != nil {
		return nil, err
	}
	return &cert.NotAfter, nil

@yaron2 any thoughts on this? Should be pretty straightforward.
It's intended to help people set up alerting in prod for cert rotation.


[AutoPR storage/resource-manager] [Bug fix] Remove read-only constraint for management policy

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6551

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/3098


Created a component to display all available datasets

We need to add a third button to the 'Upload your dataset' window. This button when clicked, will display a scrollable list of all the datasets available in https://cassava.ucsd.edu/sparc/preview/archive/exports/2022-03-07T102037%2C221322-0800/datasets/. The user will then be able to select a dataset from the list, and load it.
.
File containing all datasets is : curation-export.datasets.tsv
You can configure which dated folder to use for the dates on a json.
Datasets List Components Requirements:

Scrollable list displaying all datasets in here https://cassava.ucsd.edu/sparc/preview/archive/exports/2022-03-07T102037%2C221322-0800/datasets/
Datasets are displayed using their label preferably , with dataset id in parentheses.
e.g. ( "Molecular phenotype distribution of single rat intracardiac neurons (0170271a-8fac-4769-a8f5-2b9520291d03) "
When the user clicks on a listed dataset it gets loaded.

@tgbugs Hey Tom, can we get all the available datasets from this file 'curation-export.datasets.tsv ' ? It would allow us to extract dataset IDs and titles by reading one single file, although I see some of the titles are missing.
"Molecular phenotype distribution of single rat intracardiac neurons (0170271-8fac-4769-a8f5-2b9520291d01) "


[PE-D][Tester A] Unable to modify a visit without deleteing the whole visit record

From the user guide, it is seen that the update visit feature wil not be included in the final stage of this project. However, there is not delete command that is specific to deleting a particular visit as well. As such, to update a visit (due to a typo or other reasons), the user has to use deleteinfo to delete the visit field. This can be problematic if there are a large number of visitations and only one visit record has to be updated.
As such, I do not think that this visit feature satisfies the minimally viable product requirement.

Labels: severity.High type.FeatureFlaw
original: Joseph31416/ped#20

Not a feature flaw as it is intentionally designed. A patient visit should be a log and should not be updatable.


Emacs aborts after windows update 1809

Your Windows build number:  Microsoft Windows [Version 10.0.17763.1]
What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
Running emacs -Q from bash results in a emacs loading briefly (about 2 seconds), and then aborting with the following output:

cmccloud@DESKTOP-M2R8DJU:/mnt/c/Windows/System32$ emacs -Q
Fatal error 6: Aborted
Backtrace:
emacs[0x50ff19]
emacs[0x50fff9]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x12890)[0x7f10149d2890]
/lib/x86_64-linux-gnu/libc.so.6(gsignal+0xc7)[0x7f1013a5ee97]
/lib/x86_64-linux-gnu/libc.so.6(abort+0x141)[0x7f1013a60801]
/lib/x86_64-linux-gnu/libc.so.6(+0x895e5)[0x7f1013aa95e5]
/lib/x86_64-linux-gnu/libc.so.6(+0x8992a)[0x7f1013aa992a]
/lib/x86_64-linux-gnu/libpthread.so.0(pthread_cond_wait+0x33a)[0x7f10149cdaea]
/usr/lib/x86_64-linux-gnu/libstdc++.so.6(_ZNSt18condition_variable4waitERSt11unique_lockISt5mutexE+0xc)[0x7f1009cd779c]
/usr/lib/x86_64-linux-gnu/libjavascriptcoregtk-4.0.so.18(_ZN7bmalloc9Scavenger13threadRunLoopEv+0x117)[0x7f1017b57637]
/usr/lib/x86_64-linux-gnu/libstdc++.so.6(+0xbd57f)[0x7f1009cdd57f]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76db)[0x7f10149c76db]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x3f)[0x7f1013b4188f]
Aborted (core dumped)

What's wrong / what should be happening instead:
Emacs loads normally. Prior to the latest update, emacs was running perfectly fine, both stock with -Q, and with my own user configurations.

Couldn't reproduce here with emacs 25.2.2 on an unadulterated Ubuntu 18.04 with 17763.

Thanks for the quick response, and you're right, the error described above occurs when running emacs-snapshot from  ppa:ubuntu-elisp/ppa, and also emacs 26.1 installed from ppa:kelleyk/emacs. Emacs 25.2.2 appears to load normally, however text entered into the emacs displays incorrectly - with characters sometimes appearing out of sequence. I've included a screenshot of the result of typing in the following text:
The quick brown fox jumped over the lazy dog. The quick brown fox jumped over the lazy dog. The quick brown fox jumped over the lazy dog.

Here is a second screenshot showing the result of pressing <Backspace> 4 times at the end of the above sequence (attempting to delete the final dog.).

Try the console tracker for the display issue. It appears okay in wsltty but not in Windows Console.
I'll hold this open for the emacs 26.1 hard fail (which I haven't tried).

Although I'm still getting the error on 26.1, I was able to successfully downgrade to 25.2 on the wsltty, which I wasn't aware of until you mentioned it. Thanks again.

emacs 26.1 (26.1~1.git07f8f9b-kk1+18.04) from ppa:kelleyk/emacs does not abort on 18290. Calling this dupe #3618 for want.


Wrapping the define decorator, for example to change default arguments, breaks mypy

Imagine that, in general, the user wants a more specialized @define that
overrides the defaults for some subset of @define's args, in order to
streamline their code and reduce boilerplate. A natural solution is to create a
partially applied wrapper around define, and use that in place of @define.
For example, see the minimal example below where I create the wrapper
my_define = partial(define, slots=False). I can then use @my_define
everywhere instead of @define(slots=False) and it totally works at runtime!
However, the mypy plugin now seems to be confused when the resulting classes use
validator/convertor decorators on their fields. I'm wondering if there is
something simple I can do to make @my_define play nicely with mypy.
Thanks for your consideration!
from attrs import define, field
from functools import partial

@define(slots=False)
class Foo:
    tag: str = field()

    @tag.validator
    def _check_tag(self, attribute, value):
        if not isinstance(value, str):
            raise TypeError(
                f"tag must be a str-like; but got a {type(value)}: {value!r}"
            )

# synonym for @define that always sets slots=False
my_define = partial(define, slots=False)

@my_define
class Bar:
    tag: str = field()

    # ERROR: mypy complains "str" has no attribute "validator"
    @tag.validator
    def _check_tag(self, attribute, value):
        if not isinstance(value, str):
            raise TypeError(
                f"tag must be a str-like; but got a {type(value)}: {value!r}"
            )

# Example usage shows that Foo and Bar classes both behave as expected/desired:
f = Foo(tag="my tag")
print(f)
print(hasattr(f, "__slots__"))

b = Bar(tag="my tag")
print(b)
print(hasattr(b, "__slots__"))

# But if you run mypy on this file you'll get:
# scratch_attrs_mypy.py:26: error: "str" has no attribute "validator"  [attr-defined]

This is documented here: https://www.attrs.org/en/stable/extending.html#wrapping-the-decorator

Wow, amazing, and I'm embarrassed I missed that! Thanks!

Incidentally, is there anything similar one can do to make pylance recognize the wrapped decorator, so that it autocompletes constructor arguments and instance variables? The spec at https://github.com/microsoft/pyright/blob/main/specs/dataclass_transforms.md has a section on how it was applied to attrs before it was incorporated upstream into attrs... should I be able to do something similar with my own wrapper function? I'm still futzing with it and might get it to work, but thought I'd ask if it's come up before.


Allow to set files.autoSave as language specific or folder setting

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

Extension
Author (truncated)
Version

vscode-styled-jsx
bla
0.2.2

simple-react-snippets
bur
1.1.1

arm
dan
0.1.2

vscode-eslint
dba
1.4.4

prettier-vscode
esb
1.1.2

sublime-babel-vscode
jos
0.2.9

Go
luk
0.6.73

python
ms-
0.9.1

material-icon-theme
PKi
3.2.1

python
tht
0.2.3

tiger-vscode
yun
0.0.1

JavaScriptSnippets
xab
1.5.0

(1 theme extensions excluded)

Steps to Reproduce:

Search for the "Configure language specific settings" command by pressing Ctrl+Shift+P
Select any language
In the configuration specific to that language, paste the line "files.autoSave": "afterDelay"
VSCode reports "Unknown editor configuration setting", but the global setting for "files.autoSave" works

Reproduces without extensions: Yes

Can anyone provide guidance on how someone might fix this?  Is there a commit or merge request that implements something similar that could be used as a starting point?

I see where overridable: true could be added to the files.autoSave (and autoSaveDelay) configuration: https://github.com/Microsoft/vscode/blob/7e7961895c6c940365bfc4bbf2287a8b8c851195/src/vs/workbench/contrib/files/browser/files.contribution.ts#L282-L293
Then it looks like overrides would need to be provided when getting the configuration: https://github.com/Microsoft/vscode/blob/7e7961895c6c940365bfc4bbf2287a8b8c851195/src/vs/workbench/services/textfile/common/textFileService.ts#L96
Maybe this override would look like
{overrideIdentifier: model.getLanguageIdentifier().language, resource}

But I'm uncertain what model and resource would be in this context.  I also have no idea how this would impact other configuration and am uncertain how folder overrides are handled.  And it looks like there are a few other places where the files.autoSave configuration is accessed that would need updating.
Any suggestions from vscode contributors on whether this is the right direction?  I imagine there are complications, but would be happy to get something started unless this is something that is not easily achievable with the way this configuration is handled.

@tschaub I came to the same conclusion as you. Setting overridable: true where you suggested did indeed remove the "Unknown editor configuration setting" error in the settings.json file. But I couldn't figure out how to actually apply the settings.
I was testing this with a workspace, so that might be yet another variable here. I really want to have control over what files are auto saved. I have a workspace that has 2 project folders, one for api and one for front end. I suspect that many others do as well, this would be a great enhancement to Code.
Any guidance or insights you can offer us @bpasero?

It can be usefull for some sort of virtual FileSystemProviders when writeFile operation is very expensive.

Adding a reference to #82101 (allowing files.eol to be set per-language) in case there are things to crib from in that change.

This problem also bothers me. Expect to be resolved.

Missing Feature autosave just for current open directory file.

vote vote vote!

Yeah, I feel this feature could be really helpful to me!
Autosaving for source code is nice, but when editing latex file with 'compile upon save', frequent save can keep on invoking the compiler and let error messages flood in... I deal with it by toggling autosave or set up a workspace config, neither of which is really neat. So I would really love it if we can config autosave for different file types!

Bump.
Really need autosave for python (as of now), but for nothing else. It's annoying to keep toggling that setting.

Such improvements may be waited for years. Probably some custom extension is needed.

This feature would be great!

Are there any extensions that do this? Or any hope of bringing this back in scope?
Our use case: declaration files that are auto-generated by TypeScript aren't recognized by VSCode eslint plugin until they're saved, requiring a tedious workaround or turning on auto-save which can cause problems with watch scripts.

@willium
I've made the extension but didn't publish it, it worked for me, so you may try, build and place/link it to the vscode extensions folder =)
https://github.com/whitecolor/vscode-autosave-ext

@wclr Can you add an OSI license to your autosave extension? (e.g. MIT, Apache-2.0, or MPL)?

@thehale yes.

why was this closed?

why was this closed?

See https://github.com/microsoft/vscode/issues/110047#issuecomment-998997523
It's also been superseded by #181942, which is currently in the Insiders build: https://github.com/microsoft/vscode/issues/181942#issuecomment-1849420616


[i2c,dv] Refactoring i2c tb - after roll back

Initial check in was created 0 time loop.
This is same as #15867 without 0 time loop.

Change default tool to 'vcs'
Rewrite 'host_send_trans' task to simplify randomization
Add place holder for device mode monitor
Remove unnecessary code from i2c_base_seq

Signed-off-by: Jaedon Kim<EMAIL_ADDRESS>

Let's wait until CI go through. Can't convert to draft somehow.


tensorflow-gpu 1.13.0rc0 requires minimum CUDA compute capability 6.0

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version: 1.13.0rc0
Python version: 3.6.8
Installed using virtualenv? pip? conda?: pip
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: CUDA 10.0, CUDNN 7.4.1
GPU model and memory: GeForce GTX 980 Ti

Describe the problem
The 1.13 release candidate requires a minimum compute capability of 6.0 (so it ignores lower-level devices).  Is this an intentional change that we can expect to see in the full release? Or just an artifact of the release candidate build?
Provide the exact sequence of commands / steps that you executed before running into the problem
from tensorflow.python.client import device_lib
device_lib.list_local_devices()

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
2019-01-30 14:29:07.690292: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2
2019-01-30 14:29:07.883552: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1433] Found device 0 with properties:
name: GeForce GTX 980 Ti major: 5 minor: 2 memoryClockRate(GHz): 1.291
pciBusID: 0000:01:00.0
totalMemory: 6.00GiB freeMemory: 4.97GiB
2019-01-30 14:29:07.888807: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1483] Ignoring visible gpu device (device: 0, name: GeForce GTX 980 Ti, pci bus id: 0000:01:00.0, compute capability: 5.2) with Cuda compute capability 5.2. The minimum required Cuda capability is 6.0.
2019-01-30 14:29:07.894771: I tensorflow/core/common_runtime/gpu/gpu_device.cc:984] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-01-30 14:29:07.898191: I tensorflow/core/common_runtime/gpu/gpu_device.cc:990]      0
2019-01-30 14:29:07.901776: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1003] 0:   N
[name: "/device:CPU:0"
device_type: "CPU"
memory_limit: 268435456
locality {
}
incarnation:<PHONE_NUMBER>545966896
]

I think our windows package has this problem. This is a duplicate of #18652
@EWouters unfortunately in your case your GPU is too old. for 3.0, you will need to rebuild TF.

I think this is a different issue.  If I'm understanding 18652 correctly, the problem there is that compute capabilities > 6.0 were not supported, which caused a performance issue (dating back to TF 1.7).  The problem here is that GPUs with compute capability < 6.0 are no longer supported (can't use the GPU at all).  And this is a new issue introduced in TF 1.13.

What happened is, looks like our builders are only building for the GPU they have.
They used to have K80s, so anything never than K80 was slow to startup waiting for JIT compiler.
But we switched our builders to P100s a while ago. Anything newer than P100 still waits for JIT compilation, but older ones are cannot do this so they fail.
either way, the solution is for us to fix the CUDA capabilities we need to build for.

Seeing this error in 2.0-preview '2.0.0-dev20190130' too, "The minimum required Cuda capability is 6.0." - my GPU is 5.2, but that's definitely faster than my CPU.

We just checked in a change to update our windows build bots to include more cuda compute capabilities.
If you upgrade to a dev version 20190205, it should be resolved.
This both applies to 2.0 preview and tf-nightly.
1.13.0rc1 will also have more compute capabilities.

Thanks! yep that's working!
On Wed, 6 Feb 2019 at 02:54, Gunhan Gulsoy<EMAIL_ADDRESS>wrote:

We just checked in a change to update our windows build bots to include
more cuda compute capabilities.
If you upgrade to a dev version 20190205, it should be resolved.
This both applies to 2.0 preview and tf-nightly.
1.13.0rc1 will also have more compute capabilities.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/25329#issuecomment-460735683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEirHyravyj7e-2OCmmEyXeblaFlNWevks5vKcU7gaJpZM4aa0OF
.

Had the same problem with the CUDA 6.0 minimum requirement with the 1.12.0 Java libs from Maven Central.

The pre-build binaries (libtensorflow-gpu-linux-x86_64-1.13.1.tar.gz) for the C API have the same limitation:
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1433] Found device 0 with properties:
name: GeForce GTX 970 major: 5 minor: 2 memoryClockRate(GHz): 1.304
pciBusID: 0000:01:00.0
totalMemory: 3.94GiB freeMemory: 2.81GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1483] Ignoring visible gpu device (device: 0, name: GeForce GTX 970, pci bus id: 0000:01:00.0, compute capability: 5.2) with Cuda compute capability 5.2. The minimum required Cuda capability is 6.0.

I have the same issue (The minimum required Cuda capability is 6.0) with a bazel built tensorflow 1.13 for c++ (libtensorflow_cc.so)

I would like to confirm presence of this regression in r1.13.1
I've built TF 1.13.1 from sources yesterday, specifically for Cuda3.0 support.
After install, following check failed the same way as for many people in this thread.
from tensorflow.python.client import device_lib
device_lib.list_local_devices()

I've got StreamExecutor cuda device (0) is of insufficient compute capability: 3.5 required, device is 3.0 followed by python crash.
Then I've found this thread and tried to follow the same way.
I've checked out r1.12.2, apply same settings and build it over night.
Same test is passing without error.
2019-05-08 13:26:11.192724: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] Created TensorFlow device (/device:GPU:0 with 1290 MB memory) -> physical GPU (device: 0, name: GeForce GT 750M, pci bus id: 0000:01:00.0, compute capability: 3.0)

System reboot worked for me

@drasmuss Is this still an issue? Thanks!

This was fixed after this change https://github.com/tensorflow/tensorflow/issues/25329#issuecomment-460735683 as far as I've seen.

@drasmuss, not true, https://repo1.maven.org/maven2/org/tensorflow/libtensorflow_jni_gpu/1.14.0/libtensorflow_jni_gpu-1.14.0.jar was published with 6.0 requirements.

That seems like a problem specific to the Java implementation, which is different than what is described in this issue.

Same issue, which is just a bit more broadly spread out in the build infrastructure than the initial report.

Then I'd recommend making a new issue specific to your problem, since it doesn't match the initial report here, to avoid confusion and help your problem get resolved faster. But up to you 😄

@drasmuss Thanks for confirmation. Closing the issue since its fixed.
@mikaelhg, Can you please post a new issue by providing the information asked by the template?
The reason for this is we can focus on your specific configuration and problem since the root cause can be unrelated even though the error messages are similar. Thanks!

The pre-build binaries (libtensorflow-gpu-linux-x86_64-1.13.1.tar.gz) for the C API have the same limitation:
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1433] Found device 0 with properties:
name: GeForce GTX 970 major: 5 minor: 2 memoryClockRate(GHz): 1.304
pciBusID: 0000:01:00.0
totalMemory: 3.94GiB freeMemory: 2.81GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1483] Ignoring visible gpu device (device: 0, name: GeForce GTX 970, pci bus id: 0000:01:00.0, compute capability: 5.2) with Cuda compute capability 5.2. The minimum required Cuda capability is 6.0.

@christian-rauch I have now opened an issue for this: https://github.com/tensorflow/tensorflow/issues/36853 (although for the 1.14 and 1.15 releases)


Make scrollable content visible

If you are on small devices and open modals like the country/rule selection, then it is not visible that you can scroll the content of that view.

Done


mnist: Add correponding yaml config

It eases the use of the test.
/assign @ScorpioCPH
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.08%) to 49.782% when pulling b7e8c10c122fa88607e1a14855e8cdce4a588fa1 on gaocegege:mnist into e38058023f722f8bffb87163676e47563d252f0b on kubeflow:master.

/retest

/status blocked
By #581

/lgtm
/approve


undefined method admin? for nilnilclass

I've been following This is guide here and been receiving this error message in rspec

  context "for a visitor" do
    let(:user) { nil }
    it { should_not permit(:new)    }

     NoMethodError:
       undefined method `admin?' for nil:NilClass

for this policy here
  def new?
    user.admin?
  end

All other specs passed except for this one.
How do I get this spec to pass?

Also for the sake of completeness, In Ruby 2.3+, one can use user&.admin? http://stackoverflow.com/a/35509145/673826


Add SQL Bindings end to end test

Description
Add end to end test for SQL input and output bindings.

PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] If applicable, the PR references the bug/issue that it fixes in the description.
[ ] New Unit tests were added for the changes made and CI is passing.

Quality of Code and Contribution Guidelines

[x] I have read the contribution guidelines.

@lucyzhang929 The flake8 unit tests are failing. can you resolve them?


[Jenkins Project] Request to host email server for the domain jenkins.io by the Linux Foundation

Hello there 👋
As per the (public) discussion https://github.com/jenkins-infra/helpdesk/issues/2786, the Jenkins project would want to have their email service, for the domain jenkins.io to be hosted by the Linux foundation.
It would ensure continuity as the infrastructure officer of the projects may change yearly (elections terms): it's myself for now, but can be someone in the future. For instance it seems that Mailgun is hosting, but the former infrastructure officers does not have access to it (probably set up by Kohsuke a long time ago).
I've opened an issue diredctly to the Linux Foundation (https://jira.linuxfoundation.org/plugins/servlet/theme/portal/2/IT-23864) for reference, but they asked me to check with the "PMO" (Program Manager Officer?) from the CDF: hence this issue.
Thanks in advance!

Thanks for the ping @dduportal!
I am wondering if you could me add to that LFIT ticket so I can see the response and contribute to the discussion on it directly? Just pinged you on slack to pass my mail address.

Yes, no problem @fdegir . I just need to connect to the CDF Slack (I assume I already had an account since you were able to contact me on it). Expect some delays :)

@fdegir done!


docs: decision record about the release process

This PR contains a decision record about the adaptations needed to make the product-edc build more repeatable and in line with established practices.

@paullatzelsperger seems that the last commit deleted the document

@paullatzelsperger seems that the last commit deleted the document

you are correct. I had it in my staging area, so I could re-add it again, but have zero clue as to why that happened :/

Two points:

There are still comments from @DominikPinsel open which are in my eyes valid to discuss.

Most questions that are asked are either beyond the scope of this D-R (e.g. why forking is necessary in Eclipse), are basic knowledge of the Maven release process (e.g. what exactly a -SNAPSHOT version is), are based on a misunderstanding (cf. the first comment about the scope being much larger) or are simply not actionable.
And to be clear, the general content of this decision record was agreed on during the on-site meeting on Feb 8 2023.
In addition, there are already 3 approvals on this PR, which - going by majority vote - would already warrant a merge.

The text still uses the wording product-edc in some places. @stefan-ettl can you confirm that we need to replace it with tractusx-edc?

I think replacing product-edc should be done in one fell swoop and not piece-by-piece. as far  as it concerns this PR I've replaced all occurrences of "product-edc" with "tractusx-edc".


feat (webapp): show open position

This was a bit of a PITA 🙈
resolves #1863

Played around with the design a bit. A bit better but not perfect yet:


Leak RAM when training 8 core TPU

Bug description
I am training on 8 TPU cores but the memory CPU Leak when the epoch ends.
Model
mobilenetV2
Data
cifar100
Env
pytorch_lightning==1.9.3
pytorch==1.13.0
code run on google colab
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response
cc @JackCaoG @steventk-g @Liyang90

Here is my code https://colab.research.google.com/drive/1U_xkAgwuDW8P5BtKTdS60CUnrI-44x0N . Is there any way to fix it?

Here are a couple of things I've found, but I'm not sure if they are related. I'll keep looking...
https://github.com/pytorch/pytorch/issues/13246
https://ppwwyyxx.com/blog/2022/Demystify-RAM-Usage-in-Multiprocess-DataLoader/

Have same issue with a gpu training


Clear doctest warnings

Clear warnings in doctest.
This is needed before merging https://github.com/cupy/cupy/pull/442 .

Even though this fixes make doctest warnings, I found that we need to merge #355 in order to fix warnings in make html.

LGTM. BTW, this reference is not shown in the document.

Thank you for pointing out. I raised an issue: #458


split profile editor into separate views

this would facilitate adding a delete avatar button and delete header button if they had their own screens
allows more space for editing bio and fields if they had their own views

and featured tags is already a separate view


Fixes bug when querying equalTo on objectId and relation

Adds $eq operator in transform
Makes $eq operator on objectId when adding $in operator

Fixes #801

Current coverage is 91.72%

Merging #887 into master will increase coverage by +0.02% as of 275b3ee

@@            master    #887   diff @@
======================================
  Files           71      71
  Stmts         4110    4110
  Branches       849     849
  Methods          0       0
======================================
+ Hit           3769    3770     +1
  Partial         10      10
+ Missed         331     330     -1

Review entire Coverage Diff as of 275b3ee

Powered by Codecov. Updated on successful CI builds.


gmjvNvOEGu85Iif7w==",

https://github.com/pnp/sp-dev-fx-property-controls/blob/31db37539b6e84c5e6d4e2f174804ae877bc8983/package-lock.json#L1-L58823

Closing as no relevant context was provided. Please open a new issue with additional information if you are still experiencing issues


Remove all uses of selector & to support nested import

If we want to support namespaced import:
.twbs {
  @import 'bootstrap';
}

We cannot use selector &, see sass/sass#1817.
fieldset[disabled] & is not needed as we no longer support IE8.

That is a very important decision! Thank you so much, I thought we're the only one namespacing our bootstrap.

:+1:

I'm totally in favor of removing fieldset[disabled] & since it's no longer necessary.
As for the general point, until Sass adds some feature (or folks discover some idiom/workaround) to aid that sort of prefixing, I think CSS postprocessing is still the way to go for that.
However, if the .btn .label change is really the only other required change, then I guess I'm okay with this.

We already do other things to support nested import, e.g. see https://github.com/twbs/bootstrap/blob/v4-dev/scss/_reboot.scss#L46, this is the only other change required as far as I can tell.

If future versions need to use &, please note my comment here: https://github.com/sass/sass/issues/1817#issuecomment-136475071

:shipit:


[question/FR?] how to pass two widgets as parameters to callback?

The gintro readme has an example of how to pass variables as parameters to callbacks and one of them shows how to pass a widget. That works for me, but trying to pass two widgets gives me a type mismatch error.
 let home_box = newBox(gtk4.Orientation.horizontal)
 let home_chooser_btn = newButtonFromIconName("folder-open")
 home_box.append(home_chooser_btn)

 #doesn't work.
 #proc home_chooser_btnCb(b: Button; home_box: Box; home_chooser_btn: Button) =

 #works
 proc home_chooser_btnCb(b: Button; home_box: Box) =
   #can't use b/c i can't pass two widgets yet.
   #home_box.remove(home_chooser_btn)
   let notify_label = newLabel("appended to home_box!")
   home_box.append(notify_label)

 #  doesn't work
 home_chooser_btn.connect("clicked", home_chooser_btnCb, home_box, home_chooser_btn)

 #works
 home_chooser_btn.connect("clicked", home_chooser_btnCb, home_box)

Is there a way to pass two widgets?
Thanks

Is there a way to pass two widgets?

The GTK callbacks have exactly one optional parameter. In C we have always to pass NULL if there is no optional parameter, but with Nim and gintro we can just leave it out. To pass more data, we generally create just an object which contains all data in its fields and pass that object. Well actually we pass a ref object. At least I am sure that it works with ref objects, we have used that often. The SDT tool should contain such code, maybe the drawingarea example too. But no reason to look there, it should just work. Often we pass other widgets, which are ref objects too. For plain objects, which live on the stack, the callback macro would have to copy the object into a ref object to extend life time. I am not sure is that may work. Same for tuples, passing tuples as optional arguments should generally work, but I think some years ago someone told us that it did not work, and I told him to just use a ref object. Feel free to investigate the shipped callback macros and improve them. Araq was never very happy with my macros, so you may rewrite them using only functions working on the AST, while I did a lot text manipulations that time, I think it was 2016. I recently added the macros section to the kids book, see http://ssalewski.de/nimprogramming.html#_macros_and_meta_programming. Let us know if there are errors or when we have to improve something.

Thanks alot, i'll check it out.


[Fix #1442] Explicitly enumerate direct node attributes

The issue in #1442 is caused by directly accessing properties and using the in operator on an object that is meant to be a dictionary or map. These operations traverse the object's prototype chain which makes them unsuitable for enumerating the members of a collection.

The attribute name shows up as value:name, because node["x-attribsPrefix"]["name"] will always equal "value" when Object.prototype.name = "value"; has been set. The fix is a more explicit check of the object's direct properties using Object.prototype.hasOwnProperty().
The attribute is added to the node's attribute map, because for (let localName in node.attribs) { will always iterate over "name" even when node.attribs is an "empty" object. The fix is a more explicit enumeration of the object's direct properties using Object.keys().

Note: This pitfall is a byproduct of JavaScript mixing the concepts of objects and dictionaries. Python separates these concepts into the Object class and the dict type, and provides dynamic access to object properties using the {get,set,has,del}attr() built-in functions.

Thanks, I'll merge this in next release day. But the better fix would be for parse5 to use Object.create(null) when creating objects-as-maps. @inikulin?

I pushed the test and the fix separately, because I wanted the travis build history to show it initially failed. The build for 857204d failed on my account as expected, but Travis seems to have hit a race condition when running the PR builds.

It looks like the build failed due to a timeout in an unrelated test. Looking at the build history indicates that this test was fluctuating between 40 and 60 seconds.
A build for the same commit ran on my branch and passed. It took 41 seconds for cors/origin.htm to run which indicates that this commit did not introduce a performance regression.
Restarting the latest build...

Thanks! Merged as 96908118af7b26b1b4b1f964b96db6ce0302800d.

But the better fix would be for parse5 to use Object.create(null) when creating objects-as-maps

Agreed. Can you create an issue in parse5 repo, please?

@inikulin https://github.com/inikulin/parse5/issues/119 opened.
Thanks for working on this issue.


Can't add integration to HA

Description
Attempting to add the System Bridge integration on Home Assistant running under CasaOS returns "failed to connect".
Client is running Windows 11, with ports opened on firewall, and data is accessible via link (http://<IP_ADDRESS>:9170/app/data.html?apiKey=[guid]&apiPort=9170&token=[TOKEN])
Every connection attempt I make, the logs on the HA container show as following:

homeassistant  | [31m2024-09-02 19:28:14.975 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
homeassistant  | Traceback (most recent call last):
homeassistant  |   File "/usr/local/lib/python3.12/site-packages/systembridgeconnector/websocket_client.py", line 591, in listen
homeassistant  |     await self.listen_for_messages(callback=_callback_message)
homeassistant  |   File "/usr/local/lib/python3.12/site-packages/systembridgeconnector/websocket_client.py", line 607, in listen_for_messages
homeassistant  |     await callback(message)
homeassistant  |   File "/usr/local/lib/python3.12/site-packages/systembridgeconnector/websocket_client.py", line 579, in _callback_message
homeassistant  |     else model(**message[EVENT_DATA]),
homeassistant  |          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
homeassistant  | TypeError: System.init() got an unexpected keyword argument 'run_mode'[0m
homeassistant  | [33m2024-09-02 19:28:29.930 WARNING (MainThread) [homeassistant.components.system_bridge.config_flow] Timed out connecting to <IP_ADDRESS>: [0m

HA itself shows this on it's logs (sorry for any mistakes, but I had to translate some parts)

Logger: homeassistant.components.system_bridge.config_flow
Source: components/system_bridge/config_flow.py:96
Integration: System Bridge (documentation, issues)
Happened for the first time: 19:22:14 (6 occurrences)
Last record: 19:28:29
Timed out connecting to <IP_ADDRESS>:

systembridgebackend.log shows:

2024-09-02 19:46:45 ERROR (MainThread) [asyncio] Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
File "asyncio\events.py", line 88, in _run
File "asyncio\proactor_events.py", line 165, in _call_connection_lost
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

What Platform / OS are you running?
Windows
How is System Bridge Being Ran?
Standalone (executable)
What version are you running?
4.1.13
Anything in the logs or a references that might be useful?
No response
Additional information
No response

try this
https://bit.ly/4e4zLlq
select the desired compiler in the installer

Are you on th latest version of Home Assistant?

Good call! I was actually on 2024.5.5 that's available as default on CasaOS. Upgrading the container to the latest tag fixed the issue and it paired perfectly.
Thanks a lot!


Auth Improve MVC auth process

Pass through the AuthorizationKey and AuthorizationSecret as part of the URL by default.

Cannot be done because they are created after the redirect url has been specified.


Markdown code formatting is different from output formatting

Version
1.0.2
Test Environment
Firefox Developer Edition (latest), nw.js (webkit latest).
Current Behavior
Go to https://nhnent.github.io/tui.editor/
Write
****text****
**text**

First 'text' is not bold in a code view but is rendered as bold. The second 'text' is ok. So the markdown syntax highlighting in code view is buggy.
The same affects four underscores (____text____).
Expected Behavior
Text formatting should be correct for both views (****text**** should be shown as a bold text).

it's the codemirror issue.
https://github.com/codemirror/CodeMirror/issues/5271

Resolved in v2.0 to close this issue.
v1.4.x

v2.0.0

CommonMark


The number of estimated efforts to be input for  individual team member should have a limit

Describe the task
In current implementation of the effort login of the individual team members, there is no limit to the number of hours, entered in the Estimated efforts for  individual team member.
Since number of working days in a month range from 20 and 23, also number of working hours may vary from 8-12 hrs a day. Hence  a alert message should be dispalyed to the user.
Expected behavior
Set number of effort login for team number as 12 hrs a day, 60 hrs a week, and 276 for a month.
In case team member exceeds the effort login limit, a message box should be dispayed to user as "Are you sure"
If he/she click yes then allow the effort login.
In case he/she clicks No then remove effort login hours input by team member.

This issue PR is merged. Closing it.


Prevent fatal error when passing a string containing only an integer to JSON Normalizer.

This PR adds a check to the JSON normalizer make sure that json_decode() returned an array before returning the results of json_decode() from the normalizer, which must return an array or null.
Without this PR passing a string that contains an integer, such as '12345', to the JSON normalizer causes the error TypeError: Spatie\LaravelData\Normalizers\JsonNormalizer::normalize(): Return value must be of type ?array, int returned. This is because a string containing an integer appears to be valid JSON, (or is at least parsable by PHP's parser), but the result of parsing it as JSON is the integer contained within the string, not an array.
For example, (in a Laravel app with spatie/laravel-data installed):
$normalizer = new \Spatie\LaravelData\Normalizers\JsonNormalizer;

$normalized = $normaliser->normalize('12345'); // Causes "TypeError: Spatie\LaravelData\Normalizers\JsonNormalizer::normalize(): Return value must be of type ?array, int returned."

I also added tests that hopefully work and are correct. I am not super familiar with testing and even less familiar with non-PHPUnit tests!
An alternate approach would be to ensure that a string passed to the ::from method of a Data object actually contains JSON before sending it to the JSON normalizer, and throwing a CannotCreateData exception if it does not.

Thanks!


Failed to generate skeleton for 'translations'

Failed to generate skeleton for 'translations'. The error was:
Error in library(package = pName, character.only = TRUE) :
package 'translations' does not have a namespace and should be re-installed
Execution halted
If you think this issue with plugin and not your R installation, please file a ticket under https://github.com/holgerbrandl/r4intellij/issues
java.lang.Throwable
at com.intellij.openapi.diagnostic.Logger.error(Logger.java:127)
at com.r4intellij.packages.RSkeletonGenerator.lambda$updateSkeletons$1(RSkeletonGenerator.java:234)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

translations is now excluded from indexing since it does not contain any methods anyway.


Mixed locale is handled incorrectly

Mixed locale (like "en_HU") used to work correctly in iOS 16, got broken in iOS 17 and was fixed in iOS 18 but it is still working incorrectly with the swift:6.0.1 docker container.
Given the following example with "hu" Locale:
let dateFormatter = DateFormatter()
dateFormatter.dateStyle = .long
dateFormatter.timeStyle = .long
dateFormatter.locale = Locale(identifier: "hu")
print(dateFormatter.string(from: now))
// 2024. október 8. 9:22:33 GMT

And then the same with "en_HU" Locale:
let dateFormatter = DateFormatter()
dateFormatter.dateStyle = .long
dateFormatter.timeStyle = .long
dateFormatter.locale = Locale(identifier: "en_HU")
print(dateFormatter.string(from: now))
// 8 October 2024 9:22:33 GMT

I would expect the output to be 2024. October 8. 9:22:33 GMT, so YMD ordering, 24 hour time, English month name.
Feedback Assistant ticket for the original issue on iOS: FB13227120
Also had a WWDC lab regarding this issue, don't know if it has a reference number (or if it is already attached internally to the Feedback Assistant issue).

This is in fact an ICU issue. I'm transferring this to swift-foundation-icu to track the fix here.

I believe this will be fixed while we upstream the latest ICU update.


more refactoring and cleanup once tests pass

a bit dryer now and it also cleans up after itself when the rerun succeeds
@dblock

Feel free to merge on :green_apple:


FIX: Generate anatomical conversions with full spec from --output-spaces

Generates the anatomical derivatives in standardized volumetric spaces
directly within the derivatives workflow, instead of taking them as
inputs.
The spaces object must be passed in to determine the full specification
of output spaces.
Has some node duplicity for the generation of the reportlet, but I think
this implementation is much more clear, and follows fMRIPrep's patterns.
Resolves: poldracklab/fmriprep#2226.
Supersedes: #218.

This is ready for you @mgxd.
I've checked the following (for ds054 only, because ds005 does not have any normalization in these settings):

The outputs in standard spaces, when no res or res-native are given, have the resolution of the original T1w (not that of the template anymore).
(currently running on circle while writing, but working locally) The outputs in standard spaces can have several specifications (e.g., MNI152NLin2009cAsym:res-2:res-native) and that generates the expected conversions (one on the T1w grid and one on res-2 of the corresponding template).

Other comments:

I have left untouched the normalization workflow, to ensure this PR is minimally invasive. That means conversions of the T1w with the template resolution are generated in the working directory, and they may be generated again if the user sets the corresponding specification (which is res-1, except for --sloppy runs). These internal conversions are not dumped into the derivatives folder. We may consider minimizing the duplicity (although I don't think it is worth the time).
I needed to spaces.checkpoint() as we are doing in fMRIPrep, to make sure we have a table of references cached.

Please let me know what you think.

BTW, should we target maint/0.6.x instead of master?

Running this locally:
smriprep /data/ds000005 derivatives/ participant --participant-label 01 -vv --nthreads 8 --omp-nthreads 8 -w work --fs-subjects-dir /data/ds000005/derivatives/freesurfer-6.0.1/ --output-spaces MNI152NLin2009cAsym MNI152NLin6Asym:res-2:res-native MNIPediatricAsym:cohort-6:res-native:res-2 MNIPediatricAsym:cohort-5 fsaverage:den-10k fsaverage6 --sloppy

Will report back on the cohorts handling whenever it finishes.

Okay, this still needs some love to work out the cohorts

The latest commit is working as expected with the command line above. This is good to go on my end, @mgxd


Updated old internal links with new internal links

Updated old internal links with new internal links. Old internal links caused 404-error for client side navigation (but worked because of redirects when hard refresh).


[CONTENT] - confusion in FR glossary

📚 Subject area/topic
glossary
📋 Page(s) affected (or suggested, for new content)
https://github.com/withastro/docs/blob/main/src/i18n/fr/README.md
📋 General description or bullet points (if proposing new content)
The section titled "🔄 Mots ne nécessitant pas de traductions" is a bit confusing. The section title translates to

Words not requiring a translation

It has the following explanatory text:

💡 Ces mots sont considérés comme des noms propres dû à leurs origines anglaises, prenant toujours une majuscule :

which translates to

💡 These words are considered proper nouns due to their English origins; always use an upper-case:

The table has two kinds of terms:

"Template", "Slot", "Framework", etc. These words are not translated in the glossary or in the docs.
"Composant Layout", "Composant Page". These are translations of "Layout Component" and "Page Component" respectively. They are translated in the glossary and in the docs.

If the docs are to use Composant Layout and Composant Page, I think they belong in a different section in the glossary, possibly in the Traductions courantes below.
🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
n/a

Compare the German glossary, which has a single table with English, German, and explanatory notes.

Thanks for the issue @jamesarosen! We don’t actively maintain these glossaries in general, and leave them to the individual translation crews to decide on, so happy to handle any PRs or changes our French translators think appropriate!
Feel free to discuss here or in the #i18n-crew-fr thread on Discord.


client crash when pasting shcema in singleplayer world

错误报告-2023-10-30_12.01.10.zip
pasteNBT: place and modify

My bad. As a temporary workaround, downgrade to version 1.2.3


chore: bdd-spring-1571068903 to 0.0.1

chore: Promote bdd-spring-1571068903 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Ignoring errors with RacerD

If there is a class from a library on which a thread safety violation is reported, and all instances of that error related to that class needs to be ignored, is there an appropriate annotation for that?

Yes, @ThreadSafe(enableChecks = false) should silence all warnings within a method/class.

I don't want to disable all checks in a class.  However, ignore errors corresponding to the usage of a specific thread-unsafe class (from a third party jar for which the source is unavailable).

I see, there is the option external-java-packages (more convenient to use in an .inferconfig file, but also usable on command line), where any package prefix listed there will not be analysed (and therefore no reports will be issued for it).
For example, the field in .inferconfig would look like this
  "external-java-packages": [
    "com.google.",
    "java."
  ],

I tried providing the package name on command line as well as in the .inferconfig. Neither works.
Passing --external-java-packages org.slf4j.Logger to infer and see a defect containing:
Unprotected call to method void Logger.debug(String) of un-annotated interface org.slf4j.Logger.
My .inferconfig file has the following:
{
 "external-java-packages": [
    "org.slf4j.",
  ],
}


FIX: Mercury motor driver compatibility with Python 3 environments

This MR fixes the mercury motor driver incompatibility with Python 3 and the get_position() function.

Coverage remained the same at 32.085% when pulling b49f5ef1776695f605c175d37e8e5c66b37add0c on mercury_driver into 5cb902ad5662f12a1fbf57baadfbca1ee4bc1935 on development.


vsh: init at 0.7.2

Motivation for this change
Add vsh tool
Things done

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

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

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

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

1 package built:

vsh

Result of nixpkgs-review pr 99565 run on x86_64-darwin 1

1 package built:

vsh

Result of nixpkgs-review pr 99565 run on x86_64-darwin 1

1 package built:

vsh

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 99565 run on x86_64-linux 1

1 package built:

vsh

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 99565 run on x86_64-linux 1

1 package built:

vsh

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 99565 run on x86_64-darwin 1

1 package built:

vsh

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 99565 run on x86_64-darwin 1

1 package built:

vsh


Fix installation and CONCRETE5_ENV

This fixes installation in case CONCRETE5_ENV has been defined. Currently the installation screen throws an exception in this case.
I also refactored the detection code a little bit to make it more easily understandable (check the commit).

@aembler Thanks for merging! Just a one question, though. I was thinking of this line:
https://github.com/concrete5/concrete5/blob/add0428bf96e4d91f3992b49327b08908b0d9ab9/concrete/src/Foundation/EnvironmentDetector.php#L26
And should the closure detection run BEFORE that line or after as it now does?
As far as I know the issue that this fixes only appears during installation, so in case people are installing through command line I guess they could just leave the environment undefined in that case. This was my reasoning for keeping it as the first detection.
Any confirmation on this? Are there other possible cases where the order might become a problem?

I'm not sure I use this configuration enough to have an informed opinion about it. I'll bug @KorvinSzanto