Add Redux Icons and Folders

It would be great to have a specific icon for Redux files and folders

I've created some examples of the standard icons and folders

@keidsondesigner Thank you for using the template file! Can you share your Figma file so I can add them to the main one?

yep! what's your email?

You can just share a link that's visible to the public and I'll copy/paste them over

@miguelsolorio
https://www.figma.com/file/UQQzTjcRvWsZW9udePXOSr/Symbols---File-Icons-(Copy)?type=design&mode=design&t=zIiTKZHz8wKOQz7H-0

Can you share what file names/patterns these map to?

@keidsondesigner I've been looking at other file icon themes and I can't find examples of file names that these would map to, do you mind providing some examples? I don't use redux/react so I wouldn't know.

Hi @miguelsolorio it looks like this

I've added these redux files:

I also added the related folder icons as well but did not assign them in the theme because the names are a little too generic to conflict with other folders names (actions, selector, etc.). I also noticed other file themes do a similar thing. Feel free to define them in your user settings if you'd like to use them:
  "symbols.folders.associations": {
    "actions": "folder-redux-actions",
    "facade": "folder-redux-facade",
    "reducer": "folder-redux-reducer",
    "selector": "folder-redux-selector",
    "effects": "folder-redux-effects"
  }


Model breaks with new update

System Info
So I have a model that receives a list of strings as part of its operation. Sometimes the list is empty, sometimes it has data, and so the model is responsible for handling which layers this information gets distributed to.
As of this most recent update, I'm now getting an error:
Traceback (most recent call last):
  File "/home/lily/Desktop/Project/finetune_dynamic.py", line 304, in <module>
    fire.Fire(train)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/fire/core.py", line 475, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/home/lily/Desktop/Emme (copy)/finetune_dynamic.py", line 295, in train
    trainer.train(resume_from_checkpoint=resume_from_checkpoint)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/transformers/trainer.py", line 1539, in train
    return inner_training_loop(
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/transformers/trainer.py", line 1779, in _inner_training_loop
    for step, inputs in enumerate(epoch_iterator):
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/data_loader.py", line 553, in __iter__
    next_batch, next_batch_info = self._fetch_batches(main_iterator)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/data_loader.py", line 521, in _fetch_batches
    batch = concatenate(batches, dim=0)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 413, in concatenate
    return type(data[0])({k: concatenate([d[k] for d in data], dim=dim) for k in data[0].keys()})
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 413, in <dictcomp>
    return type(data[0])({k: concatenate([d[k] for d in data], dim=dim) for k in data[0].keys()})
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 411, in concatenate
    return honor_type(data[0], (concatenate([d[i] for d in data], dim=dim) for i in range(len(data[0]))))
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 84, in honor_type
    return type(obj)(generator)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 411, in <genexpr>
    return honor_type(data[0], (concatenate([d[i] for d in data], dim=dim) for i in range(len(data[0]))))
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 411, in concatenate
    return honor_type(data[0], (concatenate([d[i] for d in data], dim=dim) for i in range(len(data[0]))))
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 84, in honor_type
    return type(obj)(generator)
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 411, in <genexpr>
    return honor_type(data[0], (concatenate([d[i] for d in data], dim=dim) for i in range(len(data[0]))))
  File "/home/lily/anaconda3/envs/emme/lib/python3.10/site-packages/accelerate/utils/operations.py", line 415, in concatenate
    raise TypeError(f"Can only concatenate tensors but got {type(data[0])}")
TypeError: Can only concatenate tensors but got <class 'str'>

It appears accelerate is attempting to concatenate every single key in my input dictionary, regardless of what data type it is, and is then throwing a fit when it discovers that the data type isn't a list of tensors.
My question is whether or not there is a way to turn this functionality off or not. Accelerate doesn't need to handle my preprocessing, but I'm not even importing accelerate in my code, so I think this is something Huggingface is forcing on me.
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
To reproduce:

Create a model that takes a list of non-tensor inputs
Run a basic training shell, passing the non-tensor inputs as a key in the input dictionary

Expected behavior
Accelerate should skip over the key that it can't process, rather than preventing the program from functioning.

Can you give us a code reproducer please?

Sure, out of town today, but I should be able to mock something up shortly.

@sgugger I am also facing similar issue.
below I have provided reproducer.
>>> from datasets import load_dataset
>>> from torch.utils.data import DataLoader
>>> train_dataset = load_dataset("transformersbook/codeparrot-train", streaming=True)
Repo card metadata block was not found. Setting CardData to empty.
Resolving data files: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 183/183 [00:00<00:00, 973.81it/s]
>>> eval_dataset = load_dataset("transformersbook/codeparrot-valid", streaming=True)
Repo card metadata block was not found. Setting CardData to empty.
>>>
>>> train_dataloader = DataLoader(train_dataset['train'], batch_size=64)
>>> eval_dataloader = DataLoader(eval_dataset['validation'], batch_size=64)
>>>
>>> from accelerate import Accelerator
>>> accelerator = Accelerator()
Detected kernel version 5.4.0, which is below the recommended minimum of 5.5.0; this can cause the process to hang. It is recommended to upgrade the kernel to the minimum version or higher.
>>> train_dataloader, eval_dataloader = accelerator.prepare(train_dataloader, eval_dataloader)
>>> ti = iter(train_dataloader)
>>> next(ti)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.8/dist-packages/accelerate/data_loader.py", line 642, in __iter__
    next_batch, next_batch_info = self._fetch_batches(main_iterator)
  File "/usr/local/lib/python3.8/dist-packages/accelerate/data_loader.py", line 605, in _fetch_batches
    batch = concatenate(batches, dim=0)
  File "/usr/local/lib/python3.8/dist-packages/accelerate/utils/operations.py", line 519, in concatenate
    return type(data[0])({k: concatenate([d[k] for d in data], dim=dim) for k in data[0].keys()})
  File "/usr/local/lib/python3.8/dist-packages/accelerate/utils/operations.py", line 519, in <dictcomp>
    return type(data[0])({k: concatenate([d[k] for d in data], dim=dim) for k in data[0].keys()})
  File "/usr/local/lib/python3.8/dist-packages/accelerate/utils/operations.py", line 517, in concatenate
    return honor_type(data[0], (concatenate([d[i] for d in data], dim=dim) for i in range(len(data[0]))))
  File "/usr/local/lib/python3.8/dist-packages/accelerate/utils/operations.py", line 83, in honor_type
    return type(obj)(generator)
  File "/usr/local/lib/python3.8/dist-packages/accelerate/utils/operations.py", line 517, in <genexpr>
    return honor_type(data[0], (concatenate([d[i] for d in data], dim=dim) for i in range(len(data[0]))))
  File "/usr/local/lib/python3.8/dist-packages/accelerate/utils/operations.py", line 521, in concatenate
    raise TypeError(f"Can only concatenate tensors but got {type(data[0])}")
TypeError: Can only concatenate tensors but got <class 'str'>

Also experiencing this issue

Again we can't do much without code here. Please provide us some details of how you are making your dataset and what your snippet of code looks like relative to making your dataloader. My best guess is are you tokenizing your dataset to turn it into tensors?

@muellerzr  mentioned code snippet won't work to reproduce? it's standalone code to load dataset and describing error.

@PurvangL you need to tokenize the dataset and convert it to tensors. This is the issue

I'm getting this error as well. I'm working with PyTorch Geometric Data objects, which are fed directly into a PyG Graph network. Is there anyway to bypass this behavior?


Update Constants.kt

Description
Jacoco 0.8.4 updated for KotlinCoroutineFilter
Type of change
Check all that apply

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Refactoring (a change which changes the current internal or external interface)
[ ] This change requires a documentation update

How Has This Been Tested?
In case you did not include tests describe why you and how you have verified the
changes, with instructions so we can reproduce. If you have added comprehensive
tests for your changes, you may omit this section.
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation, if necessary
[x] My changes generate no new compiler warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Inspect the bytecode viewer, including reasoning why

Looks good to me!
@iNoles Do you think we should also update some other things as well?
json = 20180813
forge = 1.0.0-alpha2
jackson = 2.9.9
kotlinx-serialization = 0.11.0

kotlinx-serialization is a breaking change API. I am not too sure to add alpha versioning over unstable and breaking changes.  I will update other.

ah, you are right. Looks good to me otherwise.


Add Config that got missed previously

Self-merging it as this is a small, inconsequential config change.


json tags node access keys

Codecov Report

Merging #442 (aae3f63) into main (0a282ac) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
- Coverage   35.66%   35.55%   -0.12%
==========================================
  Files           6        6
  Lines         314      315       +1
==========================================
  Hits          112      112
- Misses        193      194       +1
  Partials        9        9

Impacted Files
Coverage Δ

client/explorer.go
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add Navigation Buttons for Growth Meter, Planner, and Tracker

I’m aiming to improve navigation within the application, as users currently find it difficult to access the Growth Meter, Planner Details, and Tracker features.
I propose adding buttons on the homepage that link directly to these sections, allowing users to navigate to specific routes like /growth-meter easily.
Implementing this will help contributors navigate the site more easily, reduce confusion, and address the remaining issues effectively.

we don't need this


Hacking protections for airlocks

About the PR

This PR addresses issue #18078. Namely, that metal plates and other security features can now be installed into airlocks to deter would be hackers
Levels of hacking protections can include

Medium: a steel plate has been welded to the frame that must be removed before an airlocks wiring can be accessed
High: the welded steel plate has been switched for one made of plasteel, which takes longer to remove
Max: the welded plasteel plate has also been covered with an electrified metal grille
Super max: same as max, but with another plasteel plate welded over the top

To see how airlocks can be upgraded with these protections, see the Youtube links below in the Media section
Some airlock doors on the station will come with hacking protections pre-installed. These include

Security and Atmospherics (medium protections)
Command (max protections)

Media

Making a medium level security airlock
Making a high to super max level security airlock

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: chromiumboy

add: Obstacles to deter hackers can now be added to airlocks
add: Security and Atmospherics airlocks start with a moderate level of protection
add: Command airlocks start with an even higher level of security

I feel like just having medium and high would be good enough, we don't want hacking to become too time consuming

ok but how do you weld doors now

weld it with panel closed

i think more than 3 levels is overkill, make it no security/steel/plasteel. Also electrified grille is a bit redundant (just make all wires electrified (not only power ones) instead)

i think more than 3 levels is overkill, make it no security/steel/plasteel. Also electrified grille is a bit redundant (just make all wires electrified (not only power ones) instead)

I'll drop super max security, but I like the electric grille, I think it's quite thematic (booby traps!)

if you want to shock people just cut a power wire thats always been a thing

i think everything but the grille makes sense. if you're hacking a door without insuls the power wire will already make it impossible, an electric grille is just a short wirecut.

working budgets still let you hack doors and survive grilles

Exactly, and the damage will add up over the course of the break in. It's not about prevention, it's a inconvenience / deterrence

there is no damage for working budgets
nostuns simply dont let you hack at all

It could help deter random greytiders from pre-cutting the plating from a door, and budget insuls / bare handed hacking are a thing

Define bare-handed door hacking, because there are 2 main uses to hacking doors, namely bolting it via maint panel, this is doable without any equipment short from a multitool/wirecutter, and disabling power on the door to be able to crowbar it, this being the main use and it does require some form of shock protention to prevent getting stunned on the hack attempt.
So the electrified grille at best is going to deal some damage to a person with budget insuls that got a medium roll, meaning they prevent stuns from shocks but don't prevent damage, this is extremely niche.

It could help deter random greytiders from pre-cutting the plating from a door, and budget insuls / bare handed hacking are a thing

Define bare-handed door hacking, because there are 2 main uses to hacking doors, namely bolting it via maint panel, this is doable without any equipment short from a multitool/wirecutter, and disabling power on the door to be able to crowbar it, this being the main use and it does require some form of shock protention to prevent getting stunned on the hack attempt.
So the electrified grille at best is going to deal some damage to a person with budget insuls that got a medium roll, meaning they prevent stuns from shocks but don't prevent damage, this is extremely niche.

Bare-handed, as in without any protective gloves. Yes, I realise it is niche, but as I said above, I think it's thematically appropriate. And when upgrading a door, you don't need to add the grille, you can just add the plasteel plate and call it a day. Hackers will still need to remove the plate to access the wires
Maybe pulsing wires with a multitool shouldn't shock you, only if you cut an electrified wire. Then hacking without gloves would be a more legitimate threat

I never thought that the idea of the security grille would stir so much resistance :P

actually now that i think about, it does have a good use
say in security, you dont want tiders running up and bolting it without any protection.
if you shocked the doors, now seccies all need insuls which is bad. using a security grille thing would be the best of both worlds, only trolling if you try to hack it

interesting conceptually, might just have to see how it plays out in game to get a grasp on how people interact with it

If this was added. Could it also come with the removal of cyptic wire names?
atm they are named after greek letters for no reason other than to require memorization each round. I understand it is a remnant from SS13, but it really doesn't do much. (altho i guess this could change when proper engiHUDs are added)
If this was implemented, then this obscuring feature would be less warranted.

interesting conceptually, might just have to see how it plays out in game to get a grasp on how people interact with it

But do we want 7 levels right now? We can try it now and remove later

trying to run tests one more time

tests are failing randomly rn (even with no C# tag) + they passed before + I tested it in game so it should be good


Add configurable date and time to output of status command

Fixes #33. See also the discussion there.
In this version I uses Arrow.strftime() to format the date and time, so arbitrary additional fixed strings can be included, e.g. Am %d.%m.%Y um %H:%M Uhr.
Since SafeConfigParser (resp. ConfigParser in Python 3) uses the percent sign for interpolation too and I don't think this feature is needed, I included a change to the import of the modul so that the version without interpolation is used. With this change you don't have to use two percent signs for the date/time placeholders in the config file.

Hi! Thanks for your PR :)
Could you please:

update the doc to introduce your new feature?
squash your commits?

Thank you!

Done.

Looks good to me, :+1:

:+1: Nice! Thx!

thanks @SpotlightKid


Fix missing asset_home error

We are currently getting this error in the app:
NoMethodError: undefined method `assets' for #<Rails::Application::Configuration:0x000056021751d640>
Did you mean?  asset_host
  /app/vendor/bundle/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/railtie/configuration.rb:96:in `method_missing'
  /app/config/environments/production.rb:28:in `block in <top (required)>'
  /app/vendor/bundle/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:251:in `instance_eval'
  /app/vendor/bundle/ruby/2.7.0/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:251:in `configure'
  /app/config/environments/production.rb:3:in `<top (required)>'

This commit fixes that.
Ref:

pr removing asset initializer

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

Argh - thanks for this. I'm just having a quick look at why this didn't show up as more of a problem than it clearly is (surprised the deploy succeeded and smokey passed). I'll merge this in when I'm done.


Upload data via getBlobOutputStream cause to java.lang.OutOfMemoryError: Java heap space

Hi ,
I am using azure blob storage version 12.9.0
I am trying to upload data to azure using the code
byte []bytes= taken from input stream.
BlobOutputStream blobOutputStream =  blobClient.getBlockBlobClient().getBlobOutputStream(true);
I am getting byte[] in a loop to upload
I invoke blobOutputStream.write(bytes);
This call only save the bytes in buffer in memory.
Only close method will send all buffer to azure.
When I have big data to transfer  I am getting OutOfMemory
How can write the data to the server in the middle of the transfer.
flush doesn't do anything.
Thanks,
Itay

Hi, @ihudedi. Thanks for opening this issue. You are correct that flush at the moment doesn't do anything #6310 is tracking that, and there are some details of the internals that make it a bit tricky. We can use this issue as an impetus for getting to that fix, though.
That said, the output stream still should only be holding up to a couple blocks of memory at a time before it sends data to the service and reuses the buffer. Are you setting any heap memory constraints on your jvm? Or is there anything else going on in the jvm instance that might be consuming memory as well?
I'd recommend configuring the ParallelTransferOptions inside of the BlockBlobOutputStream options. The javadocs should explain this more fully, but some of the parameters available on these options should allow you to effectively limit how much memory we'll use at once. You might also look over this issue and solution to see if it might help.

Hi @rickle-msft
I am using -XX:+UseG1GC -Xmx512m
The issue that only close method send the all buffer,It doesn;t create file in the middle of the transfer.
If I trying to upload big file from the azure portal nothing is uploaded until action is completed,Don;t see any file in the transfer.

I see. Yea so settng the heap size to 512 was consistent with the configuration in #10823, so I think the solutions there might be helpful. Have you tried looking through that thread yet?
As for when the file is created, this actually possibly won't change if/when we add flush. The reason being is that the method works via a series of stageBlock calls followed by a commitBlockList. Even if we flush the data to the service by staging a block, the file won't show up in the portal until we commit the block list, which currently happens in close.
We can explore the route of flush also committing blocks as well, but (and this is sort of just dev notes for the discussion on my team at this point, feel free to ignore...) we would have to probably then lock on the etag for a future commit, and also either store or retrieve the block list. We'd also have to ensure that even if a small amount of data is flushed at the beginning that we'd do a stage block call rather than a put blob so we can reuse the block in a future commit. Further, this also makes the flush/close pattern really poor for small files as it'll convert it from a single request to multiple requests (possibly even four+ in this paradigm. flush would do at least a stage and commit. Then close would also probably have to flush to be non breaking which is another stage and commit). So we should probably not commit on flush and only stage. And then for people with small files recommend skipping flush. This might actually be breaking if we add it without guarding it behind a flag, too.

Hi @rickle-msft
I tried to add
ParallelTransferOptions parallelTransferOptions = new ParallelTransferOptions().setBlockSizeLong(Long.valueOf(BlobClient.BLOB_DEFAULT_UPLOAD_BLOCK_SIZE)).setMaxSingleUploadSizeLong(Long.valueOf(BlobClient.BLOB_DEFAULT_UPLOAD_BLOCK_SIZE)).setMaxConcurrency(1);
to the blockBlobOutputStreamOptions with no success.still it buffer all the file (800MB) and throw
java.lang.OutOfMemoryError: Java heap space
Maybe I need another params.I am using streaming to upload the file via getBlobOutputStream and then write byte array.
Thanks,
Itay

The relevant comment from the other thread I mentioned is

System.setProperty("reactor.bufferSize.small", "5")
System.setProperty("reactor.bufferSize.x", "5")
// The upload code here, blocksize set to 4MB
// ParallelTransferOptions(4 * 1024 * 1024, null, { println("Uploaded $it") }, null), null, null, null, null,null)

Have you tried setting those system properties at all?
Additionally, if you could try using BlobClient.upload instead, that would also be interesting information for this investigation

Hi @rickle-msft
When I try blobClient.upload(inputStream,length) - I am getting OOM when I defined xmx512
When I add the System.setProperty("reactor.bufferSize.small", "5")
System.setProperty("reactor.bufferSize.x", "5")
It's succeeded

In general, I will not be able to tell you what the correct values are. Configuration options that deal with concurrency and memory usage are highly application dependent, so it will be up to you to experiment with what performs optimally in your environment. I recommend setting up testing a fairly large variety of combinations of these configurations and concluding what is best from that.
As for ParallelTransferOptions, I'll refer you to the documentation. The description of each of these fields details how it affects the network/parallel behavior of the sdk. Using that information should help give a direction for your testing.
For the reactor system properties, again there is no "correct" value. If you are seeing that your application is succeeding, then I would say it is approaching "correct" for your environment. There is an asynchronous framework sitting underneath the sdk. Those system properties define how large the queus are for holding requests/response items that help determine how eagerly the framework should be creating/processing requests.

Hi @rickle-msft
I understand but what is the meaning of
reactor.bufferSize.small
reactor.bufferSize.x
Thanks,
Itay

Those system properties define how large the queus are for holding requests/response items that help determine how eagerly the framework should be creating/processing requests. Documentation for them is here

Hi @rickle-msft
The main issue is that I can transfer large file but when it uploaded completely the memory isn't released/decreased only
after a long time or when I run System.gc() manually after some time.
Is this a default behaviour?
If the memory isn't released in the next treansfer I am getting heap space error.
Thanks,
Itay

This question looks to be a duplicate of https://github.com/Azure/azure-sdk-for-java/issues/18655. @ihudedi Can we close this issue and continue the conversation on the other thread?

Hi @gapra-msft
It's the same issue.
Thanks,
Itay

Combining low blocksize and the reactor.bufferSize properties was the only way to keep my app from below 500MB heap.

According to the linked reactor Queues class, this is the possible minimum:
    System.setProperty("reactor.bufferSize.small", "16"); // limit concurrency
    System.setProperty("reactor.bufferSize.x", "8"); // limit prefetch


Convert tabs / spaces keyboard shortcuts

Convert Tabs to Spaces  ⌥-⌘-[ (-> editor.action.indentationToSpaces)
Convert Spaces to Tabs  ⌥-⌘-] (-> editor.action.indentationToTabs)
Source - http://sweetme.at/2014/03/10/atom-editor-cheat-sheet/

I'm not finding these in the core editor. The functionality seems to be in this package, which does not define default keybindings.

Closing until I have more info


Also see Intersect which does the opposite.

Also see Intersect which does the opposite.

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

ID: a5fb66fb-9a33-2550-2783-7ce2a4760219
Version Independent ID: 2e80b927-7888-1fc8-7db9-d052b33ac87a
Content: Enumerable.Except Method (System.Linq)
Content Source: xml/System.Linq/Enumerable.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for noting the missing cross-reference, @simonpapworth6742. We've noticed that cross-references are generally missing from the Enumerable documentation. Might you be interested in submitting a PR to address this issue?


Remove duplicates in array - POTD | 27-Oct-2024

Remove duplicates in array
Link
Difficulty: Easy
Given an array arr consisting of positive integer numbers, remove all duplicate numbers.
Example:
Input: arr[] = [2, 2, 3, 3, 7, 5]
Output: [2, 3, 7, 5]
Explanation: After removing the duplicates 2 and 3 we get 2 3 7 5.
Input: arr[] = [2, 2, 5, 5, 7, 7]
Output: [2, 5, 7]
Input: arr[] = [8, 7]
Output: [8, 7]

Constraints:

1<= arr.size() <=106
2<= arr[i] <=100

Solved
def removeDuplicates(self, arr):
        # code here
        li=[]
        for i in arr:
            if i not in li:
                li.append(i)
        return li


Update index.md

Fixed typographical error.

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


Matrix power (sort of)

I'd like to add an atom that implements the following function. Let x in R^n and A in R^{m x n}. Then f(A,x) outputs an m-vector, with entries given by
f(A,x)_i = x_1^A_{i1}*x_2^A_{i2}*...x_n^A_{in}.

So it's sort of like matrix multiplication, but where you take powers instead of multiplying by coefficients and instead of summing you multiply.
The idea comes from this paper I wrote on log-linear dynamical systems. I haven't needed an atom yet because I do the log-transform by hand, but @moehle suggested a posynomial formulation (a sum of such functions), and in general it would be cool to see what we can compose this with.
Anyway this function is not hard to add, but I'm not sure what name to call it and am looking for suggestions.

After some thought, my proposal is the “geometric matrix multiplication”, ‘’gmatmul’’ in cvxpy. Mathematically, denoted A♢x 😜
Sent with GitHawk

I like this idea! And the notation is nice. Since python doesn't allow the creation of new infix operators, we'd probably have to stick with a named function like cp.gmatmul as you said.

If x is elementwise positive, each entry in the expression you’ve written would be a monomial, ie log-log affine. So you could compose lots of things with it.
I agree, it’s a cool atom!


Update adam.lua

Avoid unnecessary memory overhead when the user doesn't explicitly asks to use the state table

Would this be useful for increasing the memory efficiency of Adam in a project like Neural-Style?


Command Substitution

Nice exporter!
Is it possible to have shell expansions (command substitution) in the pattern of the config file?
I have tried but not got it to work.
If it isn't possible today, could you implement it?
For example to catch a file with todays date:
/tmp/*_`date +%Y%m%d`.log

Any thoughts on this yet?

I guess it is possible to have some templating but it won't be an open shell execution. It would be a huge security risk.

Sounds great! Hope you can implement it, I'm afraid I don't have skills to contribute on this.

@jonli662  The expression /tmp/*_{{now.Local.Format "20060102"}}.log should do the job with latest version


Fix behaviour of block_page_enabled block

For now, if block_page_enabled block is set - value will be updated each time regardless it set to true or false.
Fix for https://github.com/cloudflare/terraform-provider-cloudflare/issues/2923

thanks for the PR here @Nmishin - looks like we're failing one of the acceptance tests though.
TF_ACC=1 go test ./internal/sdkv2provider -v -run "^TestAccCloudflareTeamsRule_" -count 1 -timeout 120m -parallel 1
=== RUN   TestAccCloudflareTeamsRule_Basic
--- PASS: TestAccCloudflareTeamsRule_Basic (8.11s)
=== RUN   TestAccCloudflareTeamsRule_NoSettings
    resource_cloudflare_teams_rules_test.go:98: Step 2/2 error: Check failed: Check 8/8 error: cloudflare_teams_rule.ezalaphmxv: Attribute 'rule_settings.#' expected "0", got "1"
--- FAIL: TestAccCloudflareTeamsRule_NoSettings (10.10s)
FAIL
FAIL	github.com/cloudflare/terraform-provider-cloudflare/internal/sdkv2provider	19.019s
FAIL
make: *** [testacc] Error 1

I fixed the bug and updated the tests a bit. I think it would be better to test for values other than the default (true in our case).

thanks, acceptance tests all good now
TF_ACC=1 go test ./internal/sdkv2provider -v -run "^TestAccCloudflareTeamsRule_" -count 1 -timeout 120m -parallel 1
=== RUN   TestAccCloudflareTeamsRule_Basic
--- PASS: TestAccCloudflareTeamsRule_Basic (8.07s)
=== RUN   TestAccCloudflareTeamsRule_NoSettings
--- PASS: TestAccCloudflareTeamsRule_NoSettings (11.27s)
PASS
ok  	github.com/cloudflare/terraform-provider-cloudflare/internal/sdkv2provider	19.849s


#1704, CommandExecutor.writeMoreResults deadlock

Fixes #1704.

Pull Request Test Coverage Report for Build 6224

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 72.985%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/frontend/mysql/CommandExecutor.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/frontend/mysql/CommandExecutor.java
1
44.0%

Totals

Change from base Build 6223:
-0.006%

Covered Lines:
9364

Relevant Lines:
12830

💛  - Coveralls


DA: Implement data reconstruction from original columns

1. What does this PR implement?
Implements basic reconstruction method for original 50% of columns.
2. Does the code have enough context to be clearly understood?
This is somewhat the counterpart of encoding. This PR doesn't implement fancy interpolation for missing pieces. It just assumes the original pieces are present and rebuilds it.
3. Who are the specification authors and who is accountable for this PR?
@megonen @bacv
4. Is the specification accurate and complete?
This is a property of the polynomial schema used for encoding.
5. Does the implementation introduce changes in the specification?
No
Checklist

[x] 1. Description added.
[x] 2. Context and links to Specification document(s) added.
[x] 3. Main contact(s) (developers and specification authors) added
[x] 4. Implementation and Specification are 100% in sync including changes. This is critical.
[ ] 5. Link PR to a specific milestone.

Hi @danielSanchezQ , that was quick. Thanks for solution. Much simpler than when I was trying to figure using
extended_data.
So far test_reconstruct recovers happy case from N (not N/2) blobs?
And one more question is about when some of N/2 blobs are corrupted. Probably would need to tap into extended_data and perhaps recompute proofs anyway ? There could be entire tree of combinations to recover from?


Error getting folders, expected status 200 but got 401

Describe the bug
I get many many Kubernetes Events  "Error getting folders, expected status 200 but got 401"  but the dashboards are well uploaded on Grafana  ; take a little more time than usual but I think it's because the admin account is blocked (see grafana logs below). The bug affect all dashboards (in the operator dashboard or not) ; apparently not datasource
Version
Tried with v4.1.0 and v4.1.1 with Grafana 3.3.3 and 3.3.4
To Reproduce

Install the operator using kustomize ( namespace=grafana / add --scan-all) with all CRD, RBAC-related manifests and cluster roles
Deploy a Grafana :

spec:
    baseImage: grafana/grafana:8.3.4
    client:
      preferService: true
    config:
      auth:
        disable_login_form: true # tried with false too, same error
        disable_signout_menu: false
      auth.anonymous:
        enabled: false
      auth.github:
        allow_sign_up: true
        allowed_organizations: ***
        api_url: ***
        auth_url: ***
        client_id: ***
        client_secret: ***
        enabled: true
        scopes: user:email,read:org
        token_url: https://github.com/login/oauth/access_token
      feature_toggles:
        enable: ngalert
      log:
        level: warn
        mode: console
      plugins:
        enable_alpha: true
      server:
        root_url: https://***
      users:
        auto_assign_org: true
        auto_assign_org_role: Editor
    dashboardLabelSelector:
    - matchExpressions:
      - key: app
        operator: In
        values:
        - grafana
    dataStorage:
      accessModes:
      - ReadWriteOnce
      class: gce-ssd
      size: 10Gi
    deployment:
      securityContext:
        fsGroup: 472
        runAsGroup: 472
        runAsUser: 472
      strategy:
        type: Recreate
    ingress:
      annotations:
        cert-manager.io/cluster-issuer: letsencrypt-prod
        kubernetes.io/ingress.class: nginx-external
        nginx.ingress.kubernetes.io/proxy-body-size: 100m
      enabled: true
      hostname: ***
      path: /
      pathType: Prefix
      tlsEnabled: true
      tlsSecretName: grafana-ingress-cert
    service:
      labels:
        app: grafana
        type: grafana-service
      name: grafana-service

Deploy a GrafanaDashboard
See error logs everywhere

Expected behavior
The dashboard are deployed on Grafana without these errors
Suspect component/Location where the bug might be occuring
unknown (the credentials in Kubernetes secret are good)
Screenshots

Runtime (please complete the following information):

OS: GKE cos_containerd
Grafana Operator Version:  v4.1.0 + v4.1.1
Environment: GKE 1.21.6
Deployment type: Deployed following the doc

Additional context
I have 2 others Kubernetes cluster ( GKE 1.19 / RKE 1.19) with operator 4.1.0 and Grafana 8.3.3  and I don't have these bugs

Ok I think I found the problem :
Context
This cluster is new and replace an old cluster. Before the migration, I created the new cluster and deployed all the manifests from the "old" to this cluster (so same Grafana Ingress address). At the migration time, I just change DNS zone to point to the new cluster and this bug occured.
Remediation
I just set ingress.enabled: false in the old cluster (not yet deleted) and I don't see errors anymore.
Possible explanation
Looks like sometimes the operator was calling the old cluster (DNS cache not updated ?) with the credentials of the new cluster.


quote slash (/) as a key separator when communicating with augeas

Pull Request (PR) description
Quote sysctl keys with a slash in them
This Pull Request (PR) fixes the following issues
Partially fixes #17 by quoting slashes in keys (such as VLANs) when communicating with augeas
This requires augeas 1.14.x with patch to sysctl lens, that supports more non-alphanumeric characters such as a slash. Have tested by directly copying ne wlens to /opt/puppetlabs/puppet/share/augeas/lenses/dist/sysctl.aug of puppet agent 7.25
A new puppet agent with augeas 1.14.x needs to be advocated
Note that the changes provided here are probably not optimal and possibly break with comments, but its at the limit of my current provider knowledge

I am not sure what to do about the archlinux test failing

@johnwarburton ignore the archlinux job for now, we need to fix that in our CI. Are you able to add tests for this change?

Hi @bastelfreak - sure
I am not sure what image you are using for CI, but the puppet-dev-tools image has augeas 1.11, and we need, at least 1.14 to get the new sysctl lens that accepts a slash in the key. Otherwise these tests fail trying to load the test data
How can we ensure we are testing with a 1.14 lens? I've force installed http://ftp.au.debian.org/debian/pool/main/a/augeas/augeas-lenses_1.14.0-1_all.deb to test. I am reluctant to submit the tests in this PR if they fail immediately due to the old lens being present

Hi @bastelfreak
I have dug in and tried to write tests for this particular PR, but have come to the conclusion, that like the PR, all the tests need to be rewritten to cater for a / in a key. My reasoning below that augeas is OK, but the test harness is not
I am really in the weeds here and really do not know what I am doing nor how to progress.
If we start with a vanilla copy of this PR and add a single change - a valid sysctl.conf line that is for configuring VLANs, just the basic tests cannot parse the file correctly
## NB latest version of augeasproviders_core
root@060760e3f1f9:/repo# grep version spec/fixtures/modules/augeasproviders_core/metadata.json
  "version": "4.0.2-rc0",

root@060760e3f1f9:/repo# tail -1 /repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full
net.ipv4.conf.eth0/123.rp_filter = 321

root@060760e3f1f9:/repo# rake spec

Failures:

  1) Puppet::Type::Sysctl::ProviderAugeas with full file should create new entry next to commented out entry
     Failure/Error: raise AugeasSpec::Error, "augparse failed:\n#{output}" unless $CHILD_STATUS == 0 && output.empty?

     AugeasSpec::Error:
       augparse failed:
       Syntax error in lens definition
       Failed to load /tmp/d20230718-3445-exhbz6/test_augeasproviders.aug
       Test failure:/tmp/d20230718-3445-exhbz6/test_augeasproviders.aug:2.2-7.55:
        Expected:
       {
         { "net.bridge.bridge-nf-call-ip6tables" = "0" }
         { "#comment" = "net.bridge.bridge-nf-call-iptables = 0" }
         { "net.bridge.bridge-nf-call-iptables" = "1" }
         { "net.bridge.bridge-nf-call-arptables" = "0" }
       }

        Actual:
       {
         { "net.bridge.bridge-nf-call-ip6tables" = "0" }
         { "#comment" = "net.bridge.bridge-nf-call-iptables = 0" }
         { "net.bridge.bridge-nf-call-iptables" = "1" }
         { "net.bridge.bridge-nf-call-arptables" = "0" }
         { "123.rp_filter" = "321" }
       }

And we can see in the "Actual" results that the tests have parsed and split the net.ipv4.conf.eth0/123.rp_filter key on the slash
Now, I know that augeas isn't the problem as I can use augtool to read in the file and find the VLAN key correctly in the internal augeas format with the slash escaped
root@060760e3f1f9:/repo# augtool --version
augtool 1.14.0 <http://augeas.net/>

root@060760e3f1f9:/repo# augtool --root /repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/ --transform 'Sysctl incl /full'
augtool> print /files/
/files
/files/full
/files/full/#comment[1] = "Kernel sysctl configuration file"
/files/full/#comment[2] = "For binary values, 0 is disabled, 1 is enabled.  See sysctl(8) and"
/files/full/#comment[3] = "sysctl.conf(5) for more details."
/files/full/#comment[4] = "Controls IP packet forwarding"
/files/full/net.ipv4.ip_forward = "0"
/files/full/#comment[5] = "Controls source route verification"
/files/full/net.ipv4.conf.default.rp_filter = "1"
/files/full/#comment[6] = "net.ipv4.conf.default.accept_source_route: Do not accept source routing"
/files/full/net.ipv4.conf.default.accept_source_route = "0"
/files/full/#comment[7] = "SysRq setting"
/files/full/#comment[8] = "kernel.sysrq: controls the System Request debugging functionality of the kernel"
/files/full/kernel.sysrq = "0"
/files/full/#comment[9] = "Controls whether core dumps will append the PID to the core filename."
/files/full/#comment[10] = "Useful for debugging multi-threaded applications."
/files/full/kernel.core_uses_pid = "1"
/files/full/#comment[11] = "Disable netfilter on bridges."
/files/full/net.bridge.bridge-nf-call-ip6tables = "0"
/files/full/#comment[12] = "net.bridge.bridge-nf-call-iptables = 0"
/files/full/net.bridge.bridge-nf-call-arptables = "0"
/files/full/net.ipv4.conf.eth0\/123.rp_filter = "321"

Also its not the augeas ruby bindings - I munged a file from augeas core to simply read in and write out internal representation of the file  spec/fixtures/modules/augeasproviders_core/spec/lib/augeas_spec/fixtures.rb
root@060760e3f1f9:/repo# cat fixtures.rb
#!/usr/local/bin/ruby

require 'augeas'

  # Open Augeas on a given file.  Used for testing the results of running
  # Puppet providers.
  def aug_open(file, lens)
    aug = Augeas.open(nil, nil, Augeas::NO_MODL_AUTOLOAD)
    begin
      aug.transform(
        lens: lens,
        name: lens.split('.')[0],
        incl: file,
        excl: []
      )
      aug.set('/augeas/context', "/files#{file}")
      aug.load!
      raise AugeasSpec::Error, "Augeas didn't load #{file}" if aug.match('.').empty?

      yield aug
    rescue Augeas::Error
      errors = []
      aug.match('/augeas//error').each do |errnode|
        aug.match("#{errnode}/*").each do |subnode|
          subvalue = aug.get(subnode)
          errors << "#{subnode} = #{subvalue}"
        end
      end
      raise AugeasSpec::Error, errors.join("\n")
    ensure
      aug.close
    end
  end

target = '/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full'

aug_open(target, "Sysctl.lns") do |aug|
  aug.match("/files/#{target}/*").each do |node|
    val = aug.get(node)
    puts node + ' = ' + val
  end
end

root@060760e3f1f9:/repo# ./fixtures.rb
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[1] = Kernel sysctl configuration file
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[2] = For binary values, 0 is disabled, 1 is enabled.  See sysctl(8) and
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[3] = sysctl.conf(5) for more details.
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[4] = Controls IP packet forwarding
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/net.ipv4.ip_forward = 0
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[5] = Controls source route verification
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/net.ipv4.conf.default.rp_filter = 1
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[6] = net.ipv4.conf.default.accept_source_route: Do not accept source routing
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/net.ipv4.conf.default.accept_source_route = 0
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[7] = SysRq setting
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[8] = kernel.sysrq: controls the System Request debugging functionality of the kernel
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/kernel.sysrq = 0
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[9] = Controls whether core dumps will append the PID to the core filename.
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[10] = Useful for debugging multi-threaded applications.
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/kernel.core_uses_pid = 1
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[11] = Disable netfilter on bridges.
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/net.bridge.bridge-nf-call-ip6tables = 0
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/#comment[12] = net.bridge.bridge-nf-call-iptables = 0
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/net.bridge.bridge-nf-call-arptables = 0
/files/repo/spec/fixtures/unit/puppet/provider/sysctl/augeas/full/net.ipv4.conf.eth0\/123.rp_filter = 321


setuptools 45 installed on Python 2 with anaconda

I am running Anaconda Python on OS X as well as Ubuntu.
My base Ananaconda environment  on OS X is is 2.7.
How can I upgrade it?
When I try to save my environments to .yaml file I get python 2.7 errors.
$ conda -V
conda 4.8.2

 $ pip -V
pip 20.0.2 from /Users/davidlaxer/anaconda/lib/python2.7/site-packages/pip (python 2.7)

$ conda list setuptools
# packages in environment at /Users/davidlaxer/anaconda:
#
# Name                    Version                   Build  Channel
setuptools                45.0.0                   py27_0    conda-forge

$ conda env export > environment.yaml
/Users/davidlaxer/anaconda/lib/python2.7/site-packages/pkg_resources/py2_warn.py:19: UserWarning: ************************************************************
You are running Setuptools on Python 2, which is no longer
supported and
>>> SETUPTOOLS WILL STOP WORKING <<<
in a subsequent release. Please ensure you are installing
Setuptools using pip 9.x or later or pin to `setuptools<45`
in your environment.
If you have done those things and are still encountering
this message, please comment in
https://github.com/pypa/setuptools/issues/1458
about the steps that led to this unsupported combination.
************************************************************
  sys.version_info < (3,) and warnings.warn("*" * 60 + msg + "*" * 60)

On Ubuntu 16.06 LTS:
(base) ubuntu@ip-10-0-1-71:~$ conda -V
conda 4.8.2
(base) ubuntu@ip-10-0-1-71:~$ pip -V
/home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/pkg_resources/py2_warn.py:22: UserWarning: Setuptools will stop working on Python 2
************************************************************
You are running Setuptools on Python 2, which is no longer
supported and
>>> SETUPTOOLS WILL STOP WORKING <<<
in a subsequent release (no sooner than 2020-04-20).
Please ensure you are installing
Setuptools using pip 9.x or later or pin to `setuptools<45`
in your environment.
If you have done those things and are still encountering
this message, please comment in
https://github.com/pypa/setuptools/issues/1458
about the steps that led to this unsupported combination.
************************************************************
  sys.version_info < (3,) and warnings.warn(pre + "*" * 60 + msg + "*" * 60)
pip 20.0.2 from /home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/pip (python 2.7)
(base) ubuntu@ip-10-0-1-71:~$ conda list setuptools
# packages in environment at /home/ubuntu/anaconda2:
#
# Name                    Version                   Build  Channel
setuptools                41.4.0                   py27_0    anaconda
(base) ubuntu@ip-10-0-1-71:~$ conda env export > environment.yaml
/home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/pkg_resources/py2_warn.py:22: UserWarning: Setuptools will stop working on Python 2
************************************************************
You are running Setuptools on Python 2, which is no longer
supported and
>>> SETUPTOOLS WILL STOP WORKING <<<
in a subsequent release (no sooner than 2020-04-20).
Please ensure you are installing
Setuptools using pip 9.x or later or pin to `setuptools<45`
in your environment.
If you have done those things and are still encountering
this message, please comment in
https://github.com/pypa/setuptools/issues/1458
about the steps that led to this unsupported combination.
************************************************************
  sys.version_info < (3,) and warnings.warn(pre + "*" * 60 + msg + "*" * 60)
NVIDIA: no NVIDIA devices found
Traceback (most recent call last):
  File "/home/ubuntu/anaconda2/bin/conda-env", line 6, in <module>
    from conda_env.cli.main import main
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/conda_env/cli/main.py", line 44, in <module>
    from . import main_create
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/conda_env/cli/main_create.py", line 17, in <module>
    from .. import exceptions, specs
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/conda_env/specs/__init__.py", line 7, in <module>
    from .binstar import BinstarSpec
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/conda_env/specs/binstar.py", line 11, in <module>
    from binstar_client import errors
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/binstar_client/__init__.py", line 17, in <module>
    from .utils import compute_hash, jencode, pv
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/binstar_client/utils/__init__.py", line 17, in <module>
    from .config import (get_server_api, dirs, load_token, store_token,
File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/binstar_client/utils/config.py", line 22, in <module>
    from .yaml import yaml_load, yaml_dump
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/binstar_client/utils/yaml.py", line 4, in <module>
    from yaml import safe_load, safe_dump, SafeLoader
  File "/home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/yaml/__init__.py", line 8, in <module>
    from .loader import *
  File "/home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/yaml/loader.py", line 4, in <module>
    from .reader import *
  File "/home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/yaml/reader.py", line 45, in <module>
    class Reader(object):
  File "/home/ubuntu/anaconda2/envs/ai/lib/python3.6/site-packages/yaml/reader.py", line 137, in Reader
    NON_PRINTABLE = re.compile('[^\x09\x0A\x0D\x20-\x7E\x85\xA0-\uD7FF\uE000-\uFFFD\U00010000-\U0010ffff]')
  File "/home/ubuntu/anaconda2/lib/python2.7/re.py", line 194, in compile
    return _compile(pattern, flags)
  File "/home/ubuntu/anaconda2/lib/python2.7/re.py", line 251, in _compile
    raise error, v # invalid expression
sre_constants.error: bad character range

I can't really tell what's going on here, and it's very hard to do so without a clean reproducer. It seems like in some cases you're getting setuptools 45 installed on Python 2 and in other cases you are getting the Python 2 warning on Python 3?
The first one might be a duplicate of #1458, and there's some anaconda-specific thing that's ignoring the python_requires metadata. The second one might be more interesting.

My Anaconda base environment is 2.7.
Is there a way to upgrade this to 3.7 without starting from scratch (e.g. - recreating all my virtual environments - which I can’t seem to save to a .yaml file)?
It’s curious that this is happening on OS X as well as on Ubuntu.

On Feb 19, 2020, at 11:44 AM, Paul Ganssle<EMAIL_ADDRESS>wrote:
I can't really tell what's going on here, and it's very hard to do so without a clean reproducer. It seems like in some cases you're getting setuptools 45 installed on Python 2 and in other cases you are getting the Python 2 warning on Python 3?
The first one might be a duplicate of #1458 https://github.com/pypa/setuptools/issues/1458, and there's some anaconda-specific thing that's ignoring the python_requires metadata. The second one might be more interesting.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/pypa/setuptools/issues/2004?email_source=notifications&email_token=AAXWFWZ44Q446F3WZTMBF23RDWDZTA5CNFSM4KWXXTYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMJHMYQ#issuecomment-588412514, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXWFW2UUNEWB2CJN4IDVODRDWDZTANCNFSM4KWXXTYA.

Is there a way to upgrade [my Anaconda base environment] to 3.7?

This team is not familiar with Anaconda, so I'd reach out to that team for guidance on that.
On your Ubuntu system, the fact that you're getting the py2 warning for a file in the python3.6 source tree suggests to me that something (Anaconda maybe) has caused Python 2 to link to your Python 3 site packages. You'll want to inspect sys.path and possible do some debugging with python -v or tracing of import site to determine where or how that's happening. Also check that the environment variable PYTHONPATH isn't somehow implicated.
For your OS X environment, where you have Python 2, it may be the case that all you need to do is use conda to downgrade to setuptools<45.


road also tagged with railway=abandoned is rendered as railway

Description
https://www.openstreetmap.org/way/165024953 is tagged as highway=secondary but also as railway=abandoned. However, the rendering prioritizes the railway tag over the highway tag:

It should be rendered as a highway instead.
Version
2.00-alpha.3.3
What browser are you seeing the problem on? What version are you running?
Firefox v109.0
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-alpha3#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&map=19.08/32.39764/-86.98759

see note on #535 - that will fix this as well.


Crash when selecting EntryId in restriction editor dialog

MFCMapi x86 Build
Repro:

Right click search folder
Edit Search Criteria...
In "Entry ID List", click Add
crash

I can repro that with MFCMAPI x64 17.0.17099.01 with MAPI from Outlook 2016 from Office Professional Plus 2016 64-bit 1708 (Build 8431.2079 C2R) Monthly Channel.


[Doc Feature][Enable Link unfurling for compose box extension on Mobile][2270588]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/how-to/link-unfurling.md
84
9
link
:white_check_mark:

msteams-platform/sbs-botbuilder-linkunfurling.yml
80
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit d50a0ca:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/messaging-extensions/how-to/link-unfurling.md
:warning:Warning
View
Details

msteams-platform/sbs-botbuilder-linkunfurling.yml
:warning:Warning
View
Details

msteams-platform/messaging-extensions/how-to/link-unfurling.md

Line 33, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/Teams-link-unfurling/mobile-link-unfurl.png'.
Line 42, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/Teams-link-unfurling/desktop-link-unfurl.png'.

msteams-platform/sbs-botbuilder-linkunfurling.yml

Line 72, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/bots/createazurebot.png'.
Line 85, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/meetings-side-panel/newresource.png'.
Line 123, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/Teams-link-unfurling/config1.png'.
Line 180, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/Teams-link-unfurling/ngrok.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/how-to/link-unfurling.md
84
9
link
:white_check_mark:

msteams-platform/sbs-botbuilder-linkunfurling.yml
80
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 75c6258:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/sbs-botbuilder-linkunfurling.yml
:warning:Warning
View
Details

msteams-platform/assets/images/compose-extensions/Capture 4.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 5.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 7.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 8.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/capture_1.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/capture_2.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/capture_3.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/share-to-teams/auth-preview-updated.JPG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/share-to-teams/install-preview-updated-new.JPG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/auth-preview-updated.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/card-preview.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/desktop.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/desktop-link-unfurl.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/desktop-newūnfurl.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/feature-mobile-new.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/install-preview-updated-new.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/mobile-link-unfurl.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/Output-desktop-new.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/Output-mobile.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/result-output-new.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/result-output-new1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/search-new-desktop-updated.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/unfurl-desktop-updated.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/Url-preview.png
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.

msteams-platform/sbs-botbuilder-linkunfurling.yml

Line 72, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/bots/createazurebot.png'.
Line 85, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/meetings-side-panel/newresource.png'.
Line 123, Column 4: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/Teams-link-unfurling/config1.png'.
Line 180, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '~/assets/images/Teams-link-unfurling/ngrok.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 1a5c03e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/bots/createazurebot.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 4.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 5.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 7.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/Capture 8.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/capture_1.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/capture_2.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/compose-extensions/capture_3.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/meetings-side-panel/newresource.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/share-to-teams/auth-preview-updated.JPG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/share-to-teams/install-preview-updated-new.JPG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/auth-preview-updated.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/card-preview.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/config1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/desktop.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/desktop-link-unfurl.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/desktop-newūnfurl.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/feature-mobile-new.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/install-preview-updated-new.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/mobile-link-unfurl.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/ngrok.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/Output-desktop-new.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/Output-mobile.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/result-output-new.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/result-output-new1.png
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


feat: make children usage work

From discussion: https://main.elk.zone/indieweb.social/@thomasreggi/111569486541264803
Reproduction: https://gist.github.com/reggi/a4bc79ec03b9cdb1ee9fd6c61ee8e081
Found children usage is possible by using onInit hook and storing the initial children there: https://codepen.io/ayoayco-the-styleful/pen/gOqJbzW?editors=1011

Maybe a readonly this.children to access initial children will do

Also support  by allowing cloneNode of template contents


Update Makefile to the current kubernetes_version and kubernetes_build_date values.

Description of changes: Update Makefile to reflect the correct kubernetes_version and kubernetes_build_date values used to generate the official AWS AMIs.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I believe this PR has been superseded by other changes, closing.


Do not set X-ARR-ClientCert header on HttpClient

The example code on this section seems to imply that you should configure a HttpClient to send along a client certificate for mutual TLS by including the client certificate as part of the X-ARR-ClientCert header:
request.Headers.Add("X-ARR-ClientCert", cert.GetRawCertDataString());

This seems wrong since the appropriate way to do so is setting the certificate on the HttpClient's HttpClientHandler:
httpClientHandler.ClientCertificates.Add(certificate);

The confusion appears to come from the fact that an Azure web frontend server will forward the supplied client certificate to the the web app/api backend server in the X-ARR-ClientCert request header. The frontend web server however only does this after it verified that the client is in possession of the private key belonging to the certificate as part of the Certificate Verify phase of the TLS negotiation.
It seems to me that by sending the X-ARR-ClientCert as part of the initial HttpClient request you run the risk that on an improperly configured system you can trick the backend web server from thinking a proper client certificate was presented by only sending the (public) client certificate, since no proof of possession check is performed. This might therefore cause authentication/authoriation bugs.

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

ID: 35b7960d-6de5-78ee-bc45-d7218239bbc8
Version Independent ID: d2db534b-198a-dc99-5b5a-0525582d39ac
Content: Configure certificate authentication in ASP.NET Core
Content Source: aspnetcore/security/authentication/certauth.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @blowdart
Microsoft Alias: bdorrans

Any ideas, @blowdart or @damienbod?

Hi @etortec  @scottaddie @blowdart
I don't see the problem here. You can send the certificate either way. You rely on specific Azure logic, implementation which does proof of possession, but this should be done in the code, if you rely on this. Maybe you don’t want this, for example without Azure, local development etc. The example can be used anywhere, not only Azure, so I think it important to have an example. Maybe the 2 other more standard ways of sending certificates should be documented as well and the specific Azure use case could then be mentioned which forwards the certificate in this way, but should not be relied upon as the certificates can be sent directly in the header anyway.
I did a blog about how certificates can be sent in the HTTP requests using HTTPClient, maybe some of this could be added to extend this.
https://damienbod.com/2019/09/07/using-certificate-authentication-with-ihttpclientfactory-and-httpclient/
Greetings Damien

Hi @damienbod
As far as i know cert.GetRawCertDataString() simply returns the encoded (public) certificate without the private key. This implies no proof of possession checks can be done, and we are essentially authenticating with a shared secret where the secret is a (public) certificate. We are essentially doing a variant of basic authentication with a publicly known secret.
To the best of my knowledge proper implementation of mutual TLS in AspNetCore consists out of two steps:

(Done by App Service Frontend or IIS): Mutual TLS connection is negotiated between client and server during which server validates client is in possession of the private key belonging to the by the client presented certifcate. The server does not care which certificate is presented. It only cares that the client has proven he also possess the private key belonging to the presented certificate. If proof of possession is established the server forwards the (public) client certificate to Kestrel as part of the an HTTP header for further validation (e.g. in the X-ARR-ClientCert header).
(Done by AspNetCore): AspNetCore code validates that by client presented certificate in X-ARR-ClientCert header is actually a trusted certificate, granting or denying access as appropriate.

The proof of possession check in step 1 is essential since otherwise a malicious client can trick AspNetCore in granting them access by sending the public(!) certificate along in the X-ARR-ClientCert header (assuming the server does not explicitly drop this header).

I strongly agree with the previous comment. There's only one reliable way (and not a more standard way) to authenticate with a client certificate, which is using
httpClientHandler.ClientCertificates.Add(certificate);

X-ARR-ClientCert merely carries the public certificate as a means to transport identity, but not the proof of possession of the private key.
Other proxies (i.e. Envoy) use a different header name called x-forwarded-client-cert which highlights that this is a header-value in the family of the x-forwarded-* values, usually only passed by a proxy or load balancer.
The only use-case to send these header-values directly through a HttpClient should be testing if your application is correctly set up to function behind a load-balancer.
I also think it would be important to stress that when using a load-balancer/proxy which does the SSL negotiation for you there's no need to setup. It is mentioned in the very beginning under "Proxy and load balancer scenarios", but should probably be repeated once the section "Azure and custom web proxies" is started.

@etortec @daniel-munch-cko
I added a note to the docs about this. If the server accepts certificates using a header parameter from a HTTP client, then the server needs to process the requests correctly as you mentioned.
This means for Azure, a HttpClient sending the cert using a header param needs to be rejected, or processed correctly.
Does this make sense?
Greetings Damien

@damienbod  note the above

The sample is still outstanding, at least in 3.x code. @damienbod I missed this, for azure no you don't need to encode, ARR does that work. httpClientHandler.ClientCertificates.Add(certificate); is the only way to write a client. No encoding client side at all.

@blowdart Yes I removed this in the PR

Ah ok, is it just lacking my approval? (Would make sense, I took December off!)

Tested on my end. Azure drops the user-supplied X-ARR-ClientCert value and adds the actual request cert to be forwarded to the Function code.


Remember user's action to break strings into different translations

This is achieved by trying to use the saved resource name whenever possible. No overriding by group options.
Fixes: #1677

I noticed that when I click on the break button, the toast that is shown on mouseover stays visible and never goes away. Can you reproduce that?


Interactive Tutorial - Creating a Cluster

The platform seems unresponsive and un-clickable..
I am trying to click onto commands but they don't execute as they always do.
Continue button does not respond.
Terminal add button does not respond.
Expand button does not respond.
Settings button does not respond.
It's like a single static page loading, but when you leave or refresh it does throw warning.
I tried different browsers.
Firefox, Brave, Chrome.
Same on all the browsers.

/team katacoda

/label team/katacoda

I confirmed this scenario is working fine


Think about adjusting the folder structure of the repo to resemble the forecast hubs more closely

This would allow us to use the covidHubUtils functionality and other shared helper functions more easily

proabbly not bother about this one anymore as everything is working?

👍


Add TLS support to InfluxDB 2.0

Currently the 2.0 alpha does not support serving connections over TLS. This is something InfluxDB 1.x can do, and something we need to support in 2.0.
At a minimum we should provide the following to the user:

[ ] the ability to provide a certificate and private key via cli flags;

We should ensure that when provided TLS is enabled on the HTTP server, and that all resources are served over an established TLS transport.
When using the UI, we need to verify how a browser will handle backend API communication with the server is self-signed certificates have been used...
Some features that are currently in InfluxDB 1.x, which would be nice to carry over to 2.x are described in #15437

I'll close #12976 in favour of this issue with more information :+1:

Closed by #15411


Adds check for docker-compose vs docker compose

Adds check for docker-compose vs docker compose
server.py now checks for docker compose and falls back to docker-compose (also prints explanatory message if both are missing_
Fixes #935
Who can review?
Community members can review the PR once tests pass. Tag maintainers/contributors who might be interested: @agola11

We're in the process of revamping this actually and will include some fixes like this. If this is blocking you, I can fix and  merge now though

Nice! That's not a problem, I just figured I'd clear some low-hanging fruit 😎
I will abandon this PR, if there are any other help-wanted issues please let me know

Thanks, @EvanOman ! Incorporated some of the better error messaging into a PR for the upcoming version

I have both docker and docker-compose installed.
This solution doesn't work in my case:
$ docker compose --version && echo $?
Docker version 20.10.21, build 20.10.21-0ubuntu1~22.04.3
0

but
$ docker compose
docker: 'compose' is not a docker command.
See 'docker --help'

docker CLI doesn't return non-zero exit status or fail on unknown command.
$ docker-compose --version
docker-compose version 1.29.2, build unknown

Maybe better to analyze docker --help output on python or bash side?
Smth like:
docker --help | grep -E '^\s+compose\s+'


is it possible to set width and height of popover at individual step level dynamically?

I would like to set dynamic width and height for popover at individual step level.
I was trying to set width and height at onPopoverRender method with in individual step but not able to modify current step popover width and height.
Also verified CSS on inspector I could see max-width already present from library default css but I could override it.
Kindly let me know how can we set width and height of popover on the fly.

Default driver.js css max-width: 300px is way to small for my purpose.
So i try to overwrite the css. Almost works fine, but the arrow position is sometimes wrong (ie. on right side instead of left).

I use auto position of the driver (do not pass any arguments to force the position).
Any idea how to increase the max-width and keep the arrow to render in proper position?


Synchronize cpp examples with python

This PR is WIP, will be updated once it's ready to review.

Thanks for making these.  These will be great to have!  Will be so nice to have full Python and C++ examples for folks.

Oh and once these are done @csaba-luxonis - we should do the switcher on our docs so folks can switch between the Python code sample and the C++ code sample on the same page.

Yeah, that would be amazing.

I think we should do this in other direction:
Remove example numbers from python examples and leave these here as is.
Documentation should mark examples (code samples) with numbers but just as an easier overview

Oh, yes.  Sorry I didn't meant to say that we should have the numbers on these.  I agree we should remove the numbers from the Python examples.  And just have the documentation designate the order of the code samples.

Also one thing I was thinking, if we allow passing the output file as parameter (as well as the default), to be able to set it to /dev/null / nul for testing case. (Same applies to Python side)


Update hardware-support-app--hsa--steps-for-driver-developers.md

explained the RS5 AllowAny option, and stated when multiple SCCD files started to be allowed (starting RS4)
also, titles are capitialized inconsistently, most aren't capitalized, but some are FYI

Thanks @smadasam sorry for the delay on this one.  Should be live in a couple minutes.  I did some minor editing.


Submission 20220112-revl147

Hi there, thanks for your submission!
Unfortunately, your submission currently does not conform to our submission guidelines. The team name "Anonymous" and model description "Anonymous" are not allowed, even for a model under embargo (which we will respect).
See additional details at: https://github.com/microsoft/MSMARCO-Passage-Ranking-Submissions/#metadata-updates
Can you please fix?

fixed

Already updated for all three files.

Unfortunately, your submission still does not conform to our submission guidelines. The team name is not sufficiently descriptive - see additional details at: https://github.com/microsoft/MSMARCO-Passage-Ranking-Submissions/#metadata-updates
Please include a clearly identifiable group/organization or the names of the individuals responsible for the submission. For example, if I query your name with a search engine, I should get clearly identifiable results.
Can you please fix?

Hi, I have changed team name, please check again, thanks!

Okay, please check again, thanks!


Path based routing using ALB ingress is not working

I'm trying to use path-based routing using ALB ingress  (internal scheme) as following:
[root@ip-10-0-60-180 test-lb]# cat ingresstestlb.yaml
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
name: test-ingress
namespace: "default"
annotations:
kubernetes.io/ingress.class: "alb"
alb.ingress.kubernetes.io/certificate-arn: xxxxx
alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80}, {"HTTPS":443}]'
alb.ingress.kubernetes.io/scheme: internal
alb.ingress.kubernetes.io/subnets: subnet-1,subnet-2
alb.ingress.kubernetes.io/security-groups: sg-xxxxx
alb.ingress.kubernetes.io/actions.test123: >
{"Type":"forward","TargetGroupArn": "xxxxx"}
spec:
rules:

host: abcdef.com
http:
paths:
path: /abcservices/*
backend:
serviceName: test123
servicePort: use-annotation

I'm not able to browse the service in any browser but I'm getting response from curl.
On the other hand, If I simply configure host without any path configured, I am able to browse the services.
My requirement is to use path based routing.
Please advise what I'm missing here.
Thanks

external access should be
alb.ingress.kubernetes.io/scheme: internet-facing

@a5a5a9 No, I have to access those services within the VPC only, so that should be internal only.
I am able to browse services without path but not able to fetch the service when path is involved.

@M00nF1sh kindly help me in this query. What's missing in my ingress that the service is not turning on the browser?
PLs help.

Hi,
If you can access with "CURL", then the Ingress itself is working fine. (You can turn on neywork debug from your browser to see which request is failing)
Also, i saw u are using actions.test123 with a targetGroupArn. Is the targetGroup created outside of k8s? otherwise, use serviceName&servicePort instead.
Another thing i want to note is ALB don't support path rewrites. So your services should accept traffic at HTTP path you specified.

Hi @M00nF1sh , thanks for checking this out.
Yes, I created target group manually and then forwarded the Ingress ALB to that target group.
It seems application is running on root not on the path specified and that's why it is not able to resolve the path.
I will check it out on this and will get back
Thanks

Hi @M00nF1sh ,
Earlier all my were present directly inside the pod.
Now I have created a folder 'abcservices' inside the pod and then have kept my artifacts inside it.
now I am routing my ingress to abcdef.com/abcservices/* , but still it is not turning up and browser is giving my 500 error.
What I'm missing. Please help in this.


S3C-1003 FX: COPY External backends encryption

We need to exclude certain scenarios from using AWS/Azure server-side copy (for copy object and copy part) due to encryption.
Server side copy should only be allowed:

if source object and destination object are both on aws or both on azure.
if azure to azure, must be the same storage acocunt
if the source bucket is not an encrypted bucket and the destination bucket is not an encrypted bucket (unless the copy is all within the same bucket).
Also, we should make sure we are sending the sse header to aws if the destination location has SSE set to true in the locationConfig.

PR has been updated. Reviewers, please be cautious.

CONFLICT (add/add): Merge conflict in tests/functional/aws-node-sdk/test/multipleBackend/objectCopy/objectCopy.js
CONFLICT (add/add): Merge conflict in lib/data/wrapper.js
CONFLICT (add/add): Merge conflict in lib/data/external/AwsClient.js
CONFLICT (add/add): Merge conflict in lib/api/objectCopy.js
CONFLICT (add/add): Merge conflict in lib/Config.js


Ability to pass LiveComponent as field

This commit enables user to pass in custom LiveComponent for more
complex form field.
User do it like so:
form do
    field :newsletter_id, :component, render: SuggestionComponent, id: :newsletter_suggestion
end

This creates a stateful LiveComponent that can update your form input
values.
This works with a few assumption:

Your component has to be stateful, so you need to pass in :id
Your component has to use phx-keyup, because phx-change doesn't
work outside of form.
Your component cannot have <form>, because HTML5 spec mandates that
you cannot have nested form tags.

In your Suggestion component, you can show user the available
suggestions, then when they click, you should handle the event and then
let the FormComponent know. You do this by:

send self(), {:pick, {:your_field_name, value: value}}

This will forward the request to the parent LiveView, which will then
broadcast it to the FormComponent. For example, say you are editing a
schema called Listing. It has a field called newsletter_id
(belongs_to), and you want to have suggestions when user is typing so
they can search in-line. You would then send it as:

send self(), {:pick, {:newsletter_id, value: 4}}

This would create a hidden_input on your <form>, which would update
:newsletter_id to be 4.

This is what an example Suggestion component will look like:
defmodule SlickWeb.NewsletterSuggestionComponent do
    use SlickWeb, :live_component

    def mount(socket) do
      socket =
        socket
        |> assign(:suggestions, [])
        |> assign(:picked, nil)

      {:ok, socket}
    end

    def render(assigns) do
      ~L"""
      <input type="text" value="<%= @picked || @value %>" phx-debounce="500" phx-target="<%= @myself %>" phx-keyup="suggest" class="mt-1 focus:ring-indigo-500 focus:border-indigo-500 block w-full shadow-sm sm:text-sm border-gray-300 rounded-md transition">
      <%= if @suggestions != [] do %>

            <%= for {id, name} <- @suggestions do %>
              <div
                phx-target="<%= @myself %>"
                phx-click="pick"
                phx-value-id="<%= id %>"
                phx-value-name ="<%= name %>"
                class="cursor-pointer p-2 hover:bg-gray-200 focus:bg-gray-200">
                <%= name %>

            <% end %>

      <% end %>

      """
    end

    def handle_event("suggest", %{"value" => ""}, socket) do
      socket = socket |> assign(:suggestions, [])

      send self(), {:pick, {:newsletter_id, value: nil}}

      {:noreply, socket}
    end

    def handle_event("suggest", %{"value" => value}, socket) do
      require Ecto.Query

      suggestions =
        Slick.Newsletters.list_newsletters(query: value)
        |> Ecto.Query.limit(4)
        |> Ecto.Query.select([:id, :name])
        |> Slick.Repo.all()
        |> Enum.map(&({&1.id, &1.name}))

      socket = socket |> assign(:suggestions, suggestions)

      {:noreply, socket}
    end

    def handle_event("pick", %{"id" => id, "name" => name}, socket) do
      socket =
        socket
        |> assign(:picked, name)
        |> assign(:suggestions, [])

      send self(), {:pick, {:newsletter_id, value: id}}

      {:noreply, socket}
    end
  end

This works right now, but I'm not sure if this is a good use case to allow users to add custom hidden_input to form. Thoughts?


[ISSUE #2193] Switch statements containing the same code in different branches

Fixed Issues #2193
What is the purpose of the change
switch statements containing the same code in different branches
Brief changelog
switch statements containing the same code in different branches
Verifying this change
switch statements containing the same code in different branches
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.1%) to 51.331% when pulling ee44f17303e7f72c396a08206d1c3114a01d0e88 on thenx-wei:develop into fac30c3596d5577cd80a52af9ad89a0839016137 on apache:develop.

Hi,  @thenx-wei , plz follow RocketMQ community operation conventions, create an issue before submitting a pull request.

@RongtongJin Can we combine this submission ？

Generated By:JavaCC: Do not edit this line.


Asynchronous DNS client

Currently, asyncio calls the standard getaddrinfo() function in a thread
because this function is blocking. There is no standard asynchronous functio to
resolve an host name. It would be interesting to try to plug one of the
following asynchronous DNS client into asyncio:

http://www.gnu.org/software/adns/ => http://code.google.com/p/adns-python/
http://c-ares.haxx.se/ => https://pypi.python.org/pypi/pycares
https://github.com/getdnsapi/getdns
http://adns.sourceforge.net/

Old project:

http://pydns.sourceforge.net/ (no update since 11 years? blocking?)

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 6 Mar 2014 at 4:47

I want to resurrect this discussion.  It would be really cool if we can have an API for setting custom DNS resolvers in asyncio in 3.5.
Writing a pure Python DNS client is a great idea, but it's not an easy feat.  This can be done separately from asyncio if it has the API to plug such implementation in.

Well, a DNS client is not what we need. We'd need a DNS client plus a cache plus a way to consult the system resolver plus who knows what else... Do you have an actual use case and a suitable implementation, or is this just a completeness dream?

I kind of liked Victor's patch: https://codereview.appspot.com/72270043/patch/20001/30001
Specifically, loop.set_resolver() method.  Isn't cache something that resolvers can implement themselves?

So the actual use case it to: 1) try to integrate c-ares, 2) try to implement a cache policy for it.  OTOH, this experimentation is something that is possible to do with subclassing.

hi,
blindly calling getaddrinfo(3) is a really bad design choice, and is considered a bad pattern in C where it originates from.  at the very least, a better approach would be to call getaddrinfo(3) when DNS is actually required (i.e. inet_pton(3) fails).
i think that adding an API to override this behaviour would be even better, consider that a +1000 from me, because it's really the only bad aspect of asyncio i've encountered.

@kaniini

blindly calling getaddrinfo(3) is a really bad design choice, and is considered a bad pattern in C where it originates from. at the very least, a better approach would be to call getaddrinfo(3) when DNS is actually required (i.e. inet_pton(3) fails).

That's what we already do, see https://github.com/python/asyncio/blob/master/asyncio/base_events.py#L90

it is inconsistent, https://github.com/python/asyncio/blob/master/asyncio/base_events.py#L578-L593 - these functions are called by other dependencies such as asyncio.open_connection().  it would be desirable to ensure that there is consistency in what is done, i think.  shipping off thousands of getaddrinfo(3) calls to a thread pool when the code is connecting to a well-defined IP address results in significant CPU overhead.

I'm sorry, what you're saying doesn't make any sense to me.
Current loop.getaddrinfo (in asyncio master branch) checks if the address is an IP address string, and if it is, it does not use threadpools or system getaddrinfo at all.

oh, so it does.  that's great!  because on 3.5 it doesn't. :(

That'll be fixed once 3.5.2 is out.

There's http://man7.org/linux/man-pages/man3/getaddrinfo_a.3.html but libuv doesn't like it: https://github.com/joyent/libuv/issues/617

getaddrinfo_a() just spawns a thread to do the DNS resolution.
the most ideal approach would be to allow installing a new set of DNS functions, allowing for a true async resolver to be used, i.e. loop.set_resolver() or similar, as discussed earlier in this issue.


Add Edge support

Add Wildcard support for Microsoft Edge.

Test basic functionality
Test persistence of sort order across pageloads (to test: sort a page, reload it, see if it's still sorted). This uses browser local storage.
Test a basic formula: for example, just write a simple formula =<column> that references another column in the table and see if it loads the data
Add readme instructions for installing on Edge

@TylerMillis could you go through the checklist above and close out this issue when everything is completed?


Repo organization and junk in repo

Good to see that you are committing work on your project.
Your repo needs some fixes:

remove .idea and __pycache__ (don't commit these files)
add a .gitignore and be sure it includes the directories listed above
put images in a subdirectory, typically named images or assets. This is good style but not as important as items 1 and 2.

To remove .idea from git without removing it from your working copy,
use:
git rm -r --cached .idea

likewise for __pycache__.  Then commit the changes.
To move a git-tracked file to another directory (or change the file's name) use:
git mv someimagefile.png images/someimagefile.png

done!


update changelog

Update for new eval usage.

@ehazlett we should make updating the changelog a part of PRs

@sthulb +1 -- I'm also going to try to keep it updated as we go instead of all at once :)


withRouter() is needed, it should have thrown error.

I have been dealing with an issue where I was getting a blank page, after spending so much of time, somehow I found this link: https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/guides/redux.md#blocked-updates and I was able to resolve the issue.
But I believe if it could throw me some errors, it would be great for me to think like that instead of feeling blank and spending so much of time that could have been spent on actual development.
So my code was: export default connect(mapStateToProps)(AdminLayout) and it has to be export default withRouter(connect(mapStateToProps)(AdminLayout)); that was it, but I had to spend the tremendous amount of time.

There isn't a way for us to detect this. The underlying issue (blocking unseen context properties) makes this fundamentally impossible. If we could detect it, we wouldn't throw an error, we would just fix the issue! :)


Change the - to * in the .template interface.sw file

Motivation
Lines 8 & 12 use a - for a bullet point.
Change them to *.
Reasoning: there is no visual reason to do this except to standardise the repository. In terms of work done, the # Revert section tends to have more points than the number of parameters for a function (in # Arguments) therefore it's less work to switch everything to use * in another PR.

may I work on this?

may I work on this?

Sure! I'll assign you.


The content of the email alarm does not have an application ID

public String getEmailMessage() {
        return String.format("%s value is %s%s during the past 5 mins.(Threshold : %s%s)<br>", rule.getCheckerName(), getDetectedValue(), unit, rule.getThreshold(), unit);
    }

I think it would be better to add rule.getNote() here

Hi @fddOs
This is good idea. Why don't you try PR for this?


Feature/revert resize and reload#47

Outline 📝

Fix URL https://twitter.com/intent/tweet to https://twitter.com/intent/post
Revert reload button

Related issue 🚥
#47
Test 💯
No added.

for 0.7.0


chore(release): v0.2.3

:robot: I have created a release beep boop
0.2.3 (2023-03-31)
Bug Fixes

search wrong params make react-query crash. (19afc90)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/riccox/meilisearch-ui/releases/tag/v0.2.3 :sunflower:


Cell Content should be stretched even on Mobile

Expected Behavior

I expect that the content in the cells should stretch even on mobile
Current Behavior

Currently the content are being squashed together
Steps to Reproduce (for bugs)

ScreenShot
https://drive.google.com/file/d/16FQvAKuKdjiNWqq4jbhlpRcJm8pmw8j0/view?usp=sharing
Your Environment

Tech
Version

Material-UI
4.11.0

MUI-datatables
3.4.0

React
16.8.4

browser
Chrome

etc

Doing this worked on desktop.. but the cell content was still squashed on mobile.
const columns = [{
  label: "Ssn", name: "Ssn",   options: {
    filter: true,
    sort: true,
    setCellHeaderProps: value => ({ style: { width:200 } }),
  }
},
{
  label: "Lastname", name: "Lastname",   options: {
    filter: true,
    sort: true,
   }
},
]

What do you mean stretched? Have you tried setting the responsive option to "vertical"?
Also, try these options and see if it gives you what you want:
const columns = [{
  label: "Ssn", name: "Ssn",   options: {
    filter: true,
    sort: true,
    setCellProps: value => ({ style: { whiteSpace: 'pre' } }),
  }
},
{
  label: "Lastname", name: "Lastname",   options: {
    filter: true,
    sort: true,
    setCellProps: value => ({ style: { whiteSpace: 'pre' } }),
   }
},
]

What do you mean stretched? Have you tried setting the responsive option to "vertical"?
Also, try these options and see if it gives you what you want:
const columns = [{
  label: "Ssn", name: "Ssn",   options: {
    filter: true,
    sort: true,
    setCellProps: value => ({ style: { whiteSpace: 'pre' } }),
  }
},
{
  label: "Lastname", name: "Lastname",   options: {
    filter: true,
    sort: true,
    setCellProps: value => ({ style: { whiteSpace: 'pre' } }),
   }
},
]

setCellProps: value => ({ style: { whiteSpace: 'pre' } })
This just resolved my issue, so that on mobil view, the cell content are squeezed or squashed
Thank you very much!!