WebApp Cert Bind - can't find cert

Describe the bug
Command Name
az webapp config ssl bind
Errors:
Certificate for thumbprint 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX' not found.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

create an app service plan (premiumV2) and a web app for containers (linux), in different resource groups but same sub
add a verified custom domain name (cname & txt records)
az webapp config ssl create --resource-group {} --name {} --hostname {} # works fine
get the {certificate_thumbprint} from the json object output and set in next step
az webapp config ssl bind --resource-group {} --name {} --ssl-type SNI --certificate-thumbprint {}

Expected Behavior
Bind the managed certificate to the custom hostname in the web app.
Environment Summary
macOS-11.0-x86_64-i386-64bit
Python 3.8.5
Installer: HOMEBREW

azure-cli 2.10.0

Additional Context
I ran in debug mode and it clearly switches contexts from searching for the certificate under the webapp to the underlying server farm. Instead of running:
GET /subscriptions/{sub}/resourceGroups/{rg}/providers/Microsoft.Web/certificates

with {rg} of the webapp, it runs:
GET /subscriptions/{sub}/resourceGroups/{other-rg}/providers/Microsoft.Web/certificates

with {other-rg} of the webapp's serverfarm when searching for the certificate by its thumbprint.
Thumbprint is also not the best ID to use here, cant we use the certificate ARM ID? e.g.
/subscriptions/{}/resourceGroups/{}/providers/Microsoft.Web/certificates/{custom-domain-name}
as when the certificate gets rotated, it will change the thumbprint?

I think this is a duplicate of #13929


Please do not use uppercase header. use dbghelp.h instead. Cross compiling always needs lowercase header.

/home/cqwrteur/toolchains_build/ninja/src/minidump-win32.cc:18:10: fatal error: 'DbgHelp.h' file not found
   18 | #include <DbgHelp.h>
      |          ^~~~~~~~~~~
1 error generated.

https://github.com/trcrsired/windows-msvc-sysroot/tree/main/include

PR welcome :)
Regarding mt.exe: I'm not sure how to fix that.

DbgHelp.h is the Windows SDK header name, lower case dbghelp.h is the mingw-w64 name.

minidump-win32.cc is only enabled for MSVC target, maybe you can try GNU target (aarch64-w64-windows-gnu) instead?
https://github.com/ScintillaOrg/ninja-build/ninja/9d31ebba91515097e64184ca5bd460f6a7d6dfe5/src/minidump-win32.cc#L15-L20

DbgHelp.h is the Windows SDK header name, lower case dbghelp.h is the mingw-w64 name.
minidump-win32.cc is only enabled for MSVC target, maybe you can try GNU target (aarch64-w64-windows-gnu) instead?
https://github.com/ninja-build/ninja/blob/9d31ebba91515097e64184ca5bd460f6a7d6dfe5/src/minidump-win32.cc#L15-L20

That is the problem. Cross-compiling Windows MSVC target is impossible unless you make the header lowercase. For, Windows case does not matter but it matters for linux.
I am using msvc's target for the same behavior as gcc or it just fails to compile

https://github.com/trcrsired/windows-msvc-sysroot/blob/main/include/dbghelp.h
@zufuliu
Everything must be lowercase since even Microsoft is not consistent with their own shit

Regarding the mt.exe issue, it was caused by llvm-mt requiring libxml2 as a dependency, which I hadn't built. This oversight was entirely my own, as confirmed by the LLVM team.

@zufuliu Also ninja uses windows.h, based on your logic, you should use Windows.h instead.
Just use lowercase header and it should all be okay. Plus I think $ARCH-windows-gnu targets can also use this minidump. I will submit a PR

I have submitted a PR #2545

See:
https://github.com/trcrsired/windows-msvc-sysroot
https://github.com/trcrsired/llvm-releases/releases

based on your logic, it should use Windows.h instead.

tested clang with -Wnonportable-include-path, it does not warn on change file case, so PR #2545 is OK to me.


Is there a limitation on APK size?

I try to decompile the Facebook apk file and want to take a list of classes with getComponent_infos in tutorial codes.
When I run it, OutOfMemoryError occurs with Java heap space or GC overhead limit exceeded. Is there a limitation on APK size or Do I have to set any other options?

What's the java heap size you set when you run argus-saf?
On Tue, Dec 13, 2016 at 7:53 PM wonupbi<EMAIL_ADDRESS>wrote:

I try to decompile the Facebook apk file and want to take a list of
classes with getComponent_infos in tutorial codes.
When I run it, OutOfMemoryError occurs with Java heap space or GC overhead
limit exceeded. Is there a limitation on APK size or Do I have to set any
other options?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/arguslab/Argus-SAF/issues/9, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACYaAxJDQH3E-n9etdJQcVwm1GUjJwNLks5rHodMgaJpZM4LLpPC
.

I set the javaOptions as Xmx4G and -XX:-UseGCOverheadLimit

Hi, Im trying to run the tool for taint analysis. I get a Java Heap space error while scanning applications like ACR, Facebook etc. I've tried increasing the heap space for Java but it doesnt work. Is there some other way of going about it?

I test on facebook using:
java -Xms512m -Xmx8g -jar argus-saf-3.0.1-assembly.jar t com.facebook.katana_v<IP_ADDRESS>.70-54966242_Android-4.0.3.apk
It finished.

Hey,
I tried it with 8 GB heap space. It gives overhead error after hours of
scanning. Anyway, is there any reason for using apks for Android 4.0.3?
Just curious as to where do you get your apks from. Thanks for your help
Clinton
On Sun, Apr 23, 2017 at 2:12 PM, Fengguo Wei<EMAIL_ADDRESS>wrote:

I test on facebook using:
java -Xms512m -Xmx8g -jar argus-saf-3.0.1-assembly.jar t
com.facebook.katana_v<IP_ADDRESS>.70-54966242_Android-4.0.3.apk
It finished.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/arguslab/Argus-SAF/issues/9#issuecomment-296470485,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AX6Y0TOiwMSg301ubBsPV2lcVp3eCPiTks5ry5R1gaJpZM4LLpPC
.

The 4.0.3 is just something I randomly downloaded from internet. In general if you want to get large volume of apks for research purpose you can visit:
https://androzoo.uni.lu/
http://amd.arguslab.org/


I could not find function class_weights() in the utils.py

I am trying to train a model with images and sketches with my data set.
So I used train_image.py and sketch_image.py in the utils folder.
However, I get an error that there is no class_weights function used to find weights in line 116.

The class_weights() function could not be found among various functions defined in utils.py.
1. Where is the class_weight function?
If not, how do I define it?
or
Is class_weights() correct for sklearn.utils.class_weight.compute_class_weight()?
And there is one more issue.
2. Where is the utils.accuracy() function?
Without this too, how can we define it?

I checked that the train epoch was running, but I couldn't check the accuracy.
please answer about my question.........
Thank you

The class_weights and accuracy functions look like as below:
def class_weights(clss):
    uniq_clss, clss_cnts = np.unique(clss, return_counts=True)
    wts = np.zeros(len(clss))
    for i in range(len(clss)):
        wts[i] = clss_cnts[np.where(uniq_clss == clss[i])]
    wts = 1 / wts
    return wts

def accuracy(output, target, topk=(1,)):
    """Computes the precision@k for the specified values of k"""
    maxk = max(topk)
    batch_size = target.size(0)
    _, pred = output.topk(maxk, 1, True, True)
    pred = pred.t()
    correct = pred.eq(target.view(1, -1).expand_as(pred))
    res = []
    for k in topk:
        correct_k = correct[:k].view(-1).float().sum(0)
        res.append(correct_k.mul_(100.0 / batch_size))
    return res


Add new attribute for T0 Container rules

Give proper priority for T0 container rules
Fixes #10737
Status
Tested with a T0 Replay
Description
Small change to redefine the priority for T0 container rules.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
None, but additional information related to this PR is in the following JIRA Ticket: https://its.cern.ch/jira/browse/CMSTZ-897
External dependencies / deployment changes
No

If we ever need to change this again, then I'd suggest to make it configurable. For now, it looks good to me. Thanks for providing these changes, Jhonatan.

Hi @amaltaro , thanks for your suggestions. While I was working on #10747 I noticed that unsubscribedDatasets gives information about priority. For example:
{'custodial': 'n', 'request_only': 'n', 'move': 'n', 'site': 'T0_CH_CERN_Disk', 'priority': 'high', 'phedex_group': 'express', 'path': '/TestEnablesEcalHcal/Tier0_REPLAY_2021-Express-v2108100621/RAW', 'id': 5}

but is given as a string (for sure from PhEDEx era). I could modify #10747 to use that information and will follow your suggestion of making this new attribute configurable. Just let me know if you would like to see that development.


dsn_levels v.0.3.2

Fixed jump immediately occurring when jump (or start) is pressed
Fixed self.respawn spawning not exactly being in the middle of the respawn block
Added a held option for selecting levels
Holding "A" or "D" for selecting levels will speed up after 3 seconds (be warned that the animation is very choppy and may cause nausea - it will be fixed later)

#183 is fixed?

#183 is fixed?

Fixed temporarily, might be good enough for now but I'll reopen if the issue still persists (not a range yet)


Content Creator: Cannot add movie

User role: Content Creator
Action: Adding movie
Issue: Cannot add movie due to Validation Error: Movie Creation Fails Due to Missing 'Feedback' Field

Use embed link

That was a different issue, this is when the content creator/editor is adding a movie. For some reason the server does not like having a "null"/nothing feedback field as default. I think the fix would be to explicitly make the feedback field a blank quote instead of "null"/nothing

Ok hold on let me check that

Ah ok, I think it might be a feedback page not working yet issue

We fixed this by making the feedback field not required.

Adding movies now works without crashing


Legacy (Xorg, I think) applications are not themed properly by global theming on Gnome

Describe the bug
Legacy applications don't follow the theme and, from looking at Gnome tweaks, this is because their theme is set to HighContrastInverse as a result of the script.
To Reproduce
Steps to reproduce the behavior:

Apply global theming on Gnome system;
Open legacy applications, such as Signal Desktop
See error.

Expected behavior
The theming is also applied to these apps.
Screenshots

Additional context
This issue is edited all over because I figured out the problem later, so don't bother seeing the previous iteration.
Checklist

[x] I checked for duplicate issues.
[x] I checked already existing discussions.
[x] This issue is not included in the roadmap.
[x] This issue is present on both stable and development branches.

If I could just go back, it would be enough for now, but from reading through the script, I don't know how to.

I just noticed now that this is also happening to all Xorg applications' tool bars.

This is not what I reported on the issue, though.
-------- Mensagem original --------
Em 01/11/2024 13:47, nyx  escreveu:

/close not planned
global theming for gnome only applies to gtk applications and gnome itself, it does not apply to 3rd party applications that do not use gtk.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

This is not what I reported on the issue, though.
…
-------- Mensagem original -------- Em 01/11/2024 13:47, nyx escreveu:
/close not planned global theming for gnome only applies to gtk applications and gnome itself, it does not apply to 3rd party applications that do not use gtk. — Reply to this email directly, [view it on GitHub](#810 (comment)), or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

as i said in my message its due to the applications not being GTK-based

I don't get it. The problem is that the script sets legacy application theming to High Contrast instead of LegacyAdwaita and it is solved by simply manually changing it back to LegacyAdwaita. It's not about non-gtk apps being unthemed.
-------- Mensagem original --------
Em 01/11/2024 15:59, nyx  escreveu:

This is not what I reported on the issue, though.
…
-------- Mensagem original -------- Em 01/11/2024 13:47, nyx escreveu:
/close not planned global theming for gnome only applies to gtk applications and gnome itself, it does not apply to 3rd party applications that do not use gtk. — Reply to this email directly, [view it on GitHub](#810 (comment)), or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

as i said in my message its due to the applications not being GTK-based
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


A button to download the notebooks directly

A button similar to "open in colab" that downloads the notebook to your ocmputer.

You can do this via HTML yourself, I"m not sure I want to include this as the options will become quite cluttered, give that the notebook is already stored in GitHub, and you can download it from there.

You are right, just found weird that so many cloud options are easily presented without the run locally option.

Thanks


tests : failing tests - 1) non-standard character not parsed, 2) TODO in title triggers a syntax error, 3) a title with just a tag triggers an error

I ran into these bugs when I was parsing some of my own org files.

Non-standard characters. They mostly seemed to be triggered by text that I had copied from either a slack message or an email into my org mode files.

The second test is about TODO appearing in the title. This test passes for some reason, but it definitely triggered a syntax error when I tried to parse it. I am not sure why this is.

The third is a title with only a tag.

I'm following the pattern I set up before - testing that a string gets roundtripped properly.
This PR doesn't contain the fix, I'm afraid, but I figured you'd probably want to see the bugs.

Thanks. I will take a look. I can tell you that bug 2 started appearing because now the default todo keyword is "TODO" so it can't appear after the title because the parser expects the todo keywords to appear before the title. The easiest solution is to pass "fake" todo keywords when parsing the file:
Org('my/file.org', todos={'todo_states': {'todo': "TDO"}, 'done_states': {'done': "DNE"}})

Yes, I just realized that this morning as I was brushing my teeth. I think
if the default todo keyword is set to TODO it should start invoking the bug.
Thanks for taking a look.
On Thu, Nov 16, 2023 at 2:01 PM Joe Riad @.***> wrote:

Thanks. I will take a look. I can tell you that bug 2 started appearing
because now the default todo keyword is "TODO" so it can't appear after the
title because the parser expects the todo keywords to appear before the
title. The easiest solution is to pass "fake" todo keywords when parsing
the file:
Org('my/file.org', todos={'todo_states': {'todo': "TDO"}, 'done_states': {'done': "DNE"}})
—
Reply to this email directly, view it on GitHub
https://github.com/durableOne/orgmunge/pull/12#issuecomment-1814491606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOJKNLCKEYDV2Y4DCZAYF3YEYMDVAVCNFSM6AAAAAA7LKGR6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUGQ4TCNRQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

All the tests are now passing as of https://github.com/durableOne/orgmunge/commit/cc5fa46618653bc024d27722ff4102829f9abc52

Empty titles are now supported
Titles can now have todo keywords in them
Some of the tests were failing because the assertion was comparing the original input string to the stringified version of the parsed tree. The problem is that the parser doesn't preserve whitespace. I have modified the failing cases to remove extraneous whitespace (or add more spaces before the tags in the test case with the empty title) to get the tests to pass. In the future, a better way to write assertions to avoid this is to compare the parsed headings to functionally equivalent headings that you construct using the Heading class.

That's amazing. Thank you. I'm gonna try it out now.

Working great :partying_face: :fireworks:


display the color of guesses

Somehow displays the color of your guesses -- either after the game, or for each guess

Finally got around to this. Sorry it took so long, hope you enjoy! :)
https://user-images.githubusercontent.com/20880695/167987405-768cce71-8095-4234-bce6-debc1769942c.mov


providing link to download obj files for items and scenes

In order to be able to learn through your examples and be able to reproduce them to some extent, would you please be able to share links to examples of scene and 3d objects to download so we could run your examples?
Here are a few:
salle_de_bain_separated.obj
I tried searching for some of them but couldn't find them on Web.
(nvisii) mona@ard-gpu-01:~/NVISII/examples$ python 08.obj_scene_loader.py
Found 1 GPUs available for rendering.
Importing content/salle_de_bain_separated/salle_de_bain_separated.obj...
Traceback (most recent call last):
  File "/home/mona/NVISII/examples/08.obj_scene_loader.py", line 35, in <module>
    sdb = nvisii.import_scene(
  File "/home/mona/anaconda3/envs/nvisii/lib/python3.9/site-packages/nvisii/__init__.py", line 17171, in import_scene
    return _nvisii.import_scene(*args, **kwargs)
RuntimeError: Error: "content/salle_de_bain_separated/salle_de_bain_separated.obj"Unable to open file "content/salle_de_bain_separated/salle_de_bain_separated.obj".
CUDA call (cudaFree((void*)d_pointer)) failed with code 4 (line 94): driver shutting down
terminate called after throwing an instance of 'std::runtime_error'
  what():  fatal cuda error
Aborted (core dumped)

(nvisii) mona@ard-gpu-01:~/NVISII/examples$ python 06.textures.py
Found 1 GPUs available for rendering.
Traceback (most recent call last):
  File "/home/mona/NVISII/examples/06.textures.py", line 36, in <module>
    dome = nvisii.texture.create_from_file("dome", "content/kiara_4_mid-morning_4k.hdr")
  File "/home/mona/anaconda3/envs/nvisii/lib/python3.9/site-packages/nvisii/__init__.py", line 18668, in create_from_file
    return _nvisii.texture_create_from_file(name, path, linear)
RuntimeError: Error: failed to load texture image "content/kiara_4_mid-morning_4k.hdr". Reason: can't fopen
CUDA call (cudaGetDevice(&savedActiveDeviceID)) failed with code 4 (line 145): driver shutting down
terminate called after throwing an instance of 'std::runtime_error'
  what():  fatal cuda error
Aborted (core dumped)

(nvisii) mona@ard-gpu-01:~/NVISII/examples$ python 09.meta_data_exporting.py
created folder 09_metadata/
Found 1 GPUs available for rendering.
Warning: Could not do CUDA graphics resource sharing for the display buffer texture (unknown error)... falling back to slower path
Traceback (most recent call last):
  File "/home/mona/NVISII/examples/09.meta_data_exporting.py", line 71, in <module>
    brick_base_color = nvisii.texture.create_from_file("bricks_base_color", "./content/Bricks051_2K_Color.jpg")
  File "/home/mona/anaconda3/envs/nvisii/lib/python3.9/site-packages/nvisii/__init__.py", line 18668, in create_from_file
    return _nvisii.texture_create_from_file(name, path, linear)
RuntimeError: Error: failed to load texture image "./content/Bricks051_2K_Color.jpg". Reason: can't fopen
Segmentation fault (core dumped)

(nvisii) mona@ard-gpu-01:~/NVISII/examples$ python 11.instance_motion_blur.py
Found 1 GPUs available for rendering.
Traceback (most recent call last):
  File "/home/mona/NVISII/examples/11.instance_motion_blur.py", line 51, in <module>
    mesh = nvisii.mesh.create_from_file("obj", opt.path_obj)
  File "/home/mona/anaconda3/envs/nvisii/lib/python3.9/site-packages/nvisii/__init__.py", line 21923, in create_from_file
    return _nvisii.mesh_create_from_file(name, path)
RuntimeError: Error: "obj"Unable to open file "content/dragon/dragon.obj".
CUDA call (cudaGetDevice(&savedActiveDeviceID)) failed with code 4 (line 145): driver shutting down
terminate called after throwing an instance of 'std::runtime_error'
  what():  fatal cuda error
Aborted (core dumped)

We provide a script to download the content used in the example, check https://github.com/owl-project/NVISII/blob/master/examples/download_content.sh


refactor access logs to use separate functions listener access logs

Similar to how buildListenerFileAccessLog - Implements the same for others for readability and consistency

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@zirain PTAL when you get chance.

I tried to follow the same pattern we used for buildListenerFileAccessLog here also to be consistent and avoid code like if listener do some thing. But if folks think, this is more complex I can drop this PR

@howardjohn @hzxuzhonghu WDYT about this?


Log version

 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.

Pull Request Test Coverage Report for Build 221

6 of 8 (75.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 89.349%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

main.go
0
2
0.0%

Totals

Change from base Build 219:
-0.3%

Covered Lines:
302

Relevant Lines:
338

💛  - Coveralls

Does this work? I know there were some issues with SM logging the version, so I was wondering if we would have the same problem here

stale


[Ribosome] wait for pdfReady or reportReady

Going forward, we want to add a div to both web and pdf reports when they are done rendering. We can use this div to check if the report is done rendering during both pdf generation and visual regression tests. Rather than have a different div for pdf and web reports, we are adding the same div (reportReady) to both.

pdfReady is for backwards compatibility. Going forward, both pdf reports and web reports will add the reportReady div when they are done rendering. The reason we want to add a div to web reports is for visual regression testing, so we don't take a div before the web report is done rendering.

This is so cool! 🚢


Examples responsive breakpoints in small widths

Maybe we could wrap title and icons to next lines when width is low?

Can you take a look at it @antfu?


Include MARC example records

I converted official examples from https://www.loc.gov/marc/classification/examples.html to MARCXML. They are from the 21st edition and English, so the conversion assigns wrong language tags. Once this is fixed we could use the examples for automated testing. I checked that they at least result in no error:
for file in examples/ddc21en*; do mc2skos $file out.skos http://example.org/ --scheme http://example.org/; done

Thx, merged in 38e992a7aab41d3e081656e5e1e54be2f7e417bb


Question: Is it possilbe for this to be ran from a USB

Hi,
Is there a way of getting this to run via A USB stick? (Like doku wiki). If so any pointers in setting it up? I have little to no experience with PHP hence the question.
Thanks for making this free!

Everything that @domainzero wrote is correct, but it should be usable with a portable solution (xampp, easyphp, ...)
But it is not as easy as a compiled version since you will have to setup everything in the webserver and database manually


[Bug]:README fix for market docs URL

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
As simple as pointing to an updated URL : https://docs.alpaca.markets/ for the market doc link.
Expected Behavior
The URL for the market doc changed to : https://docs.alpaca.markets/
SDK Version I encountered this issue in
LATEST
Steps To Reproduce
just open the README and the link for the market doc is a 404

Filled out the Steps to Reproduce section?

[X] I have entered valid steps to reproduce my issue or have attached a minimally reproducible case in code that shows my issue happening; and understand that without this my issue will be flagged as invalid and closed after 30 days.

Anything else?
Keep on ROCKING !!!

@desavera Thank you for the report! Fixed.


fix: Select Recommended Assets

What is the purpose of the change
The initial attempt to select a recommended asset like USDT was broken because the asset was not available in the assets array during the first fetch. To resolve this, we now also include a search for the asset within the recommended assets array.
Testing and Verifying

[ ] Can Select Tokens not available on first fetch like USDT and TIA

The recommended assets are not loading initially. At least on mobile


(possible regression bug) Tag list not shown in (<tag>), [<tag>], (<tag>:1.2) syntax

Tagcomplete version: https://github.com/DominikDoom/a1111-sd-webui-tagcomplete/commit/92a08205d05e98a54215d8b2ffeadd19a01cfd3d

Editing tag in weight parenthesis and no tag list shown, different from the demo here.

Editing tag in simple weight syntax ((<tag>), [<tag>]) and no tag list shown unless blanks before and after the tag.
When edit cat in (cat) (alias of (cat:1.1)), it searchs (cat) exactly, only tags like chen_(cat) matches but cat_ears do not.

discussion: tag can and many of them do have parenthesis, but that's not needed when speaking of searching at most of times, for example cat have hundreds of related tags while (cat) is a valid tag component too but have only 9 matches out of 10k tags.
So maybe it's safe (or to say tolerable) to ignore parenthesis of user input?

You are correct, seems to be a regression introduced by an oversight in e49862d4228ad0f1a3691a9afeb1b05ff119108d.
It's an easy fix luckily, will push it in a moment.

discussion: tag can and many of them do have parenthesis, but that's not needed when speaking of searching at most of times, for example cat have hundreds of related tags while (cat) is a valid tag component too but have only 9 matches out of 10k tags.
So maybe it's safe (or to say tolerable) to ignore parenthesis of user input?

Problem is that with the current way I replace/insert the completed text, it needs to match what the user typed 1:1, else it won't find what region to replace. So if we ignore parentheses while typing, replace will likely not work, and an "approximate match" is pretty hard to do properly.


KeyError: 'type'

Not sure if this is necessarily a bug, but I keep running into this issue and wanted to see if anyone on here had any idea on why it keeps occurring.
To Reproduce
Input:
ludwig experiment --data_csv C:\Users\name\Downloads\20140306_20190305_EOD\AAPL.csv --model_definition "{input_features: [{name: date, type:timeseries}], output_features: [{name:close, type:numerical}]}"
Output:
WARNING: The TensorFlow contrib module will not be included in TensorFlow 2.0.
For more information, please see:

https://github.com/tensorflow/community/blob/master/rfcs/20180907-contrib-sunset.md
https://github.com/tensorflow/addons
If you depend on functionality not listed there, please file an issue.

c:\users\name\appdata\local\programs\python\python37\lib\argparse.py:2402: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
result = type_func(arg_string)

| |_  _ | | __ ()_ _
| | || /  \ V  V / / _ |
||_,_,|_/_/|_, |
|__/
ludwig v0.1.1 - Experiment
Traceback (most recent call last):
File "c:\users\name\appdata\local\programs\python\python37\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\users\name\appdata\local\programs\python\python37\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\name\AppData\Local\Programs\Python\Python37\Scripts\ludwig.exe_main.py", line 9, in
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\cli.py", line 94, in main
CLI()
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\cli.py", line 60, in init
getattr(self, args.command)()
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\cli.py", line 65, in experiment
experiment.cli(sys.argv[2:])
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\experiment.py", line 466, in cli
experiment(**vars(args))
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\experiment.py", line 194, in experiment
**kwargs
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\train.py", line 181, in full_train
model_definition = merge_with_defaults(model_definition)
File "c:\users\name\appdata\local\programs\python\python37\lib\site-packages\ludwig\utils\defaults.py", line 217, in merge_with_defaults
'type']].default_validation_measure
KeyError: 'type'
File Used:
APPL file in zip
Environment (please complete the following information):

OS: Windows
Version: 10
Python: 3.7.3
Ludwig version: 0.1.1

The YAML syntax requires a space after the key, so your model definition is not syntactically correct. A correct version would be:
ludwig experiment --data_csv C:\Users\name\Downloads\20140306_20190305_EOD\AAPL.csv --model_definition "{input_features: [{name: date, type: timeseries}], output_features: [{name: close, type: numerical}]}"

This should solve the issue, so I'm closing it, please keep posting if that's not the case and I will reopen the issue.


Fix gridlabd-library subcommand

This PR fixes issue #979.
Current issues
None
Code changes

[x] Fixed curl usage and internal variable so they refer to correct github repo folders.
[x] Add -l|--library options

Documentation changes

[x] Add /Command/Library

Test and Validation Notes
None

Support for -l and --library can not find the files
**aivanovalocal@PC99458 docker % gridlabd library index
overhead_line_conductors.glm
wood_pole.glm
aivanovalocal@PC99458 docker % gridlabd -l wood_pole.glm
ERROR    [INIT] : unable to load '/usr/local/opt/gridlabd/4.2.26-210817-develop_fix_library/share/gridlabd/library/wood_pole.glm': No such file or directory
FATAL    [INIT] : shutdown after command line rejected**


Dynamic on JS side

With this PR I moved processing of dynamic to JS side. We need this, as currently JSIL doesn't understand Roslyn dynamic patterns. Also, it will open path for full dynamic support (probably only with translated BCL).
This PR brakes support of Verbatium.Expression inside dynamic calls (VerbatimDynamic test moved to FailingTests). I will work on restoring it some day (if I'll here any complains - it will be sooner).
As using dynamic to work with JS objects provide not-very-readable and slow code, we need work on #532 ASAP as a primary way for JS interop.
This PR partially fix #913.
ILSpy version is updated to latest one with JSIL-specific patches. Would be good if somebody verify Mac build, as some patches was omitted.

Let's wait for integration build results. Hope it should be good.

One alternative to #532 would be to do something like how packed arrays work, where the compiler is given extra type information and can use it to generate the raw variation on dynamic it produces right now. Not sure how viable that would be, though.

I thought about additional type info for dynamics, but we will still need decompile dynamic calls back once again. We will need implement proper roslyn support for it and still have feature that works correctly only with C#.
As I already said somewhere, I see very small difference to:
SomeJSObject.Property1.Property2.SomeCall() and SomeJSObject["Property1"]["Property2"]["SomeCall"].Invoke() or even `SomeJSObject["Property1.Property2.SomeCall"].Invoke()

The main problem with the non-dynamic form is that now you need to write dramatically different code to interact with JS objects. For example, now if you swap an OpenGL wrapper out for raw WebGL contexts, you have to rewrite everything.

Interesting point. But it can be solved by using interfaces. You just need cover each used JS object with corresponding C# wrapper.
Even better, we may introduce some MetaAttribute, that will force dynamic-like behavior for some interfaces (you already created similar one for methods JSRuntimeDispatch, probably we just need extend it to interfaces).

I even thoughts about automatically generation C# wrappers (and probably corresponding proxies) based on *.d.ts files.

At the end, dynamic still work with raw JS objects, probably not so optimal and with less readable code.

Yeah, a combination of still-working-but-less-clean support for JS objects & dynamic, along with interfaces being able to point at raw JS objects will probably be good enough. Generating code from .d.ts would be cool.

Looks that test were passed. Do you have any review notes or need some more time?

I will try to review it this weekend. I may merge it to a branch.

Added new JsAPI. Found some problems, so we could discuss principal decisions, but PR is not yet ready to merge.

I dislike this approach to JS interop strongly enough that I think maybe something entirely different is necessary, like how Mono does inline assembly (which is literally impossible to find documentation for...)
For example, a more sophisticated take on Verbatim where you hand it a block of code and variables you want it to close over. We could even accept typescript in those blocks and have JSILc run those snippets through tsc with appropriate type information and shout at you if they fail to compile.
I think trying to enforce the .NET type system onto external JS objects is a bad idea and the extremely verbose code it produces demonstrates why. If we really want to produce raw JS, and we can't do it with dynamic, we should just write raw JS. It's not as if any JsAPI code will run on native .NET so the only advantage we get here is static type checking - but the static types aren't guaranteed to match runtime types anyway.

Looks like we keep in mind different problems. This JsAPI doesn't try to give you way to create raw JavaScript code in C#. It will be much better to just write it in .js file, or, if you really need it - use Verbatium.Expression and write any code that you want.
On other hand, I'd like some wrappers, that will allow me to interact with JsObjects that was passed in my code with some filling, that it is still an object. Most of this work will only read/write one property or call one method.
Probably we should discuss it more. I hoped, that Adopt<T> will help me, but I don't see how it should work if some JsMethod expect to get some JsClass (so, object with some prototype).

On other hand, I'd like some wrappers, that will allow me to interact with JsObjects that was passed in my code with some filling, that it is still an object. Most of this work will only read/write one property or call one method.

Yeah, that's an important use case. But I can't see JsAPI as a worthwhile solution to that, it's too complicated.

I should clarify though: I think the idea of creating an adapter between a raw JS object and a .NET interface is a good idea, and that should be pursued for sure. It addresses many of those use cases.
Attempting to fully pull arbitrary JS objects into a statically typed universe is risky. Other than typescript I don't think anyone has done it successfully. Dart semi-famously has attempted to do this and failed spectacularly.

When you say "complicated", do you mean that you should write many explicit code with casts?
As I tried preserve API as simple as possible. It contains only (Get/Set, Call, Invoke, New, Delete, In, AssumeType/As/Is/Cast). All other methods are just overloads for methods in list. Even in this list, As/Is/Cast are helpers that could be removed.

Really, I started working on JsAPI, as after removing special dynamic handling, I was not very happy with Verbatium.Expression return dynamic - dynamic will spread through our source a little bit uncontrollable.
I'll fully OK, if for now, our main method of JS-interop will be Verbatium.Expression (and remove all code for #532), if it will return object instead of dynamic (same as other JS-interop methods). In most cases, in our project, returned object will be just saved as field or passed into next Verbatium.Expression.

Maybe I'm incorrect, but I suppose that casting to dynamic should be optional - so user may cast Verbatium.Expression result to dynamic if he really needs it.

Yeah, I'm OK with stripping out dynamic. It breaks that 'dynamic DOM' example but not in an unfixable way. For things like WebGL I think we should just make interfaces work good.
TBH the codegen for dynamic has never been as good as I wanted anyway.
And yeah, they can cast to dynamic if they want and hit the (fully compatible, right?) binder-based slow path.

I hope that with cast to dynamic everything will work (except passing to dynamic call Verbatium.Expression call #548), at least test are passing.
Let's confirm our plan:

For now I will remove from this PR all work for #532 and just change all JS-iterop API to return object instead.
We may (and probably even need) continue discussion for #532, but original thread will be probably better place.

Yeah. I'm OK with JsAPI living on in a branch - maybe we can find a good home/use case for it in the codebase - but for now I think we just want to change all the dynamics back into object. Alongside that we can think about how to improve the UX. I can imagine a way to make Verbatim work a bit better based on closures so that values can round-trip through reliably and evaluation order issues go away.

Removed JsAPI - related commits into separate PR (#929).
This PR now could be treated as stable. If it would be merged, we need to reopen #548.
Should we use this or separate PR to change all the dynamics back into object?

Should we use this or separate PR to change all the dynamics back into object?

Separate PR, please. It'll probably require test changes and changes to the examples.


How do you handle POST -> REDIRECT -> GET sequence to pass task_id?

It is generally considered good practise to use POST for data changing requests, which in my case is done in the task. Also I always try to REDIRECT after a POST to a GET, so that if the user clicks refresh, the same task wouldn't be started.
How do you handle this, in combination with passing around the task_id?
Do you also put the task_id in the session?

We pass the task ID around. This makes the URL unique, and allows the user to return to the URL later, or even share it, if they want. You could put the task ID in session storage.

okay, thank you!


Release v1.1.0

#57  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @aschepis and the rest of your teammates on  Graphite


raftstore: Add slow log for peer and store msg

What is changed and how it works?

Issue Number: Ref #16600

What's Changed:
Add slow log for peer and store msg

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

Add slow log for peer and store msg

/merge

/merge

/cherry-pick release-7.5


Finish Chat screen when navigating back to Call screen

MOB-2055
Steps:

start audio engagement (Call screen is opened)
Tap "Chat" icon (Chat screen is opened)
Tap "back" icon from application (Call screen is opened)
Tap "back" icon from application

Expected result: Call screen is closed, Main screen is opened
Actual result: Chat screen is opened again (loops back several times)
PR fixes this.
Besides, there was (and still is) a difference between application "back" and device "back" behavior on the 3rd step.
Application "back" closes Chat screen and navigates to Call screen. I would say it's expected.
Device "back" closes Chat screen and navigates to Main screen.
I haven't found a way to align the behaviors, unfortunately.
If you have an idea - I would appreciate it.
Anyway, right now behavior is improved (primary goal of the ticket is achieved).

Correct me if I am wrong, but ChatActivity:
@Override
    public void onBackPressed() { if (chatView.backPressed()) super.onBackPressed(); }

And
private ChatView.OnBackClickedListener onBackClickedListener = () -> {
        if (chatView.backPressed()) finish();
    };

Shouldn't these be the same if we want same behaviour?

Back press logic on Chat screen is different from Call screen.
Chat screen has also different logic for device "back" and application "back" button, not sure if there were requirements for that.

Maybe I'm mistaken but can't we handle the system back(return false to block system back) and call the same method we call when the app back button is called?
Also as I remember public void onBackPressed() should be deprecated and replaced with onBackPressedDispatcher

No comments from me. But I would check what David wrote )

@gugalo I'm working on this, no worries. 😉

@DavDo @Pelkar @gugalo @AndriiHorishniiMOC My attempt to handle the system back the same way as AppBar back in ChatView ends up to the following changes: a7a811cc90c06d38daccb7ab03786e8a2188d64b
Changes overview:

rethought all "back handling" logic, came to conclusion that it can be simplified a lot
renamed some functions to understand the logic better (can restore some of the names now though)
moved all business logic to Controller
collected all "back handling" logic in a single place (controller)

!squash

!merge


material-moment-adapter doesn't work in<EMAIL_ADDRESS>

🐞 Bug report
Class with @angular/material-moment-adapter not opening in production build<EMAIL_ADDRESS>version.
It worked without problems until version<EMAIL_ADDRESS>Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Description
Class with @angular/material-moment-adapter not opening in production build<EMAIL_ADDRESS>version.
It worked without problems until version<EMAIL_ADDRESS>🔬 Minimal Reproduction
1) Update<EMAIL_ADDRESS>and material-moment-adapter
npm i<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>2) Import material-moment-adapter into the page.
import { MAT_MOMENT_DATE_FORMATS, MomentDateAdapter, MAT_MOMENT_DATE_ADAPTER_OPTIONS } from '@angular/material-moment-adapter';
import { DateAdapter, MAT_DATE_FORMATS, MAT_DATE_LOCALE } from '@angular/material/core';
3) Add providers to the page.
@Component({
providers: [
{ provide: MAT_DATE_LOCALE, useValue: 'en-US' },
{ provide: DateAdapter, useClass: MomentDateAdapter, deps: [MAT_DATE_LOCALE, MAT_MOMENT_DATE_ADAPTER_OPTIONS] },
{ provide: MAT_DATE_FORMATS, useValue: MAT_MOMENT_DATE_FORMATS }
]
})
4) Production build
ng build --prod
5) After running the app, Page does not open.
6) Downgrade<EMAIL_ADDRESS>npm i<EMAIL_ADDRESS>7) Production build
8) After running the app, It runs without problems.
🔥 Exception or Error

🌍 Your Environment

Angular CLI: 9.1.0
Node: 10.18.1
OS: darwin x64

Angular: 9.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, service-worker
Ivy Workspace: Yes

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.901.0
@angular-devkit/build-angular      0.900.7
@angular-devkit/build-optimizer    0.901.0
@angular-devkit/build-webpack      0.901.0
@angular-devkit/core               9.1.0
@angular-devkit/schematics         9.1.0
@angular/cdk                       9.2.0
@angular/fire                      5.4.2
@angular/material                  9.2.0
@angular/material-moment-adapter   9.2.0
@angular/pwa                       0.901.0
@ngtools/webpack                   9.0.7
@schematics/angular                9.1.0
@schematics/update                 0.901.0
rxjs                               6.5.4
typescript                         3.8.3
webpack                            4.41.2

Duplicate of #17320


Added Italian translation

There could be some minor problems with certain words and gender agreement, for example add / removed & directory.
Suppose a file is added: in English <file> add is shown while in Italian <file> aggiunto ("file" is a masculine noun in Italian). Now if a directory is added we get <directory> aggiunto cartella while it should be <directory> aggiunta cartella (since "cartella"=directory is a feminine noun in Italian).
add          => aggiunt[o]
add dir      => aggiunt[a] cartella

So for 100% correct Italian translation 2 different add words are required.
This holds also for removed.

Eccellente!
I added words.addDirectory and words.removedDirectory so those unique pairings of words can have a specific translation. Also removed words.directory since it was not needed anymore.
Here is what the words section of language/it-it.json looks like now.
"words": {
    "add"             : "aggiunto",
    "addDirectory"    : "aggiunta cartella",
    "build"           : "Assemblaggio",
    "clean"           : "Pulizia",
    "change"          : "modificato",
    "done"            : "Fatto",
    "removed"         : "rimosso",
    "removedDirectory": "rimossa cartella",
    "seconds"         : "secondi",
    "stats"           : "Statistiche",
    "watch"           : "Monitora",
    "watching"        : "Monitorando"
}

All language files have been updated in the latest version of Feri (0.2.7). It is available right now on GitHub and will be available tomorrow (Friday, January 8th) on NPM as soon as I wake up. ^_^
Thank you so much for the translation. You are awesome! :D

:thumbsup: Thak you for this project!


[QUESTION] Gradients are Exploding given Different Mesh

Hello,
I was able to successfully write a script that optimizes material parameters for a soft mesh using add_soft_grid. The environment starts with two identical models (model and target_model) and applying a force on the mesh watching it deform. Now, I tried to replace the mesh with the bear example. However, the gradients are exploding giving NaNs along the entire gradient path.
At first, I thought it was because there were more particles for the bear mesh (1986) than the soft grid (125), accumulating more loss per particle. However, when I measured the same number of particles the gradients still explode. I also removed the force on the mesh so they are fixed. Still the loss between the target and model is very high.
Is there something wrong with the initialization of the models? I can try clipping the model but is there some way to stabilize the gradients regardless of the mesh that is inputted? I guess this comes down to difficult gradient optimization architecture design. Would batching possibly work? The thing is, we can't reliably split the model into batches like traditional neural network architectures because we need the entire mesh model to be passed through.
Let me know what you think. Thank you again for this amazing project!

Upon closer inspection, Seems like add_soft_mesh does not compute mass and consideres all particles as mass=0. In example_walker it seems like modifying the tet_activations is valid, but my script is applying force on particles instead of tetrahedrons.
Could these deficiencies from the soft grid explain this erratic behavior? I'd be happy to submit a PR to extend the robustness of add_soft_mesh.


Cannot define a function pointer without initializing

Description:

Originnally the funciton pointer is defined as :
function () returns (error|null) onTriggerFunction;
the complier focess to initilize that
variable 'onTriggerFunction' is not initialized
This also does not work even if we make runnable
(function () returns (error|null)| null) onTriggerFunction = null;
throws error
incompatible types: expected 'function () returns (null|error)', found 'null'
The only possible option is assigning an empty function as below:
function () returns (error|null) onTriggerFunction = function () returns (error|null) { return null;};
I believe there an easier way of initializing this.
Affected Versions:
0.964.1-SNAPSHOT

(function () returns (error|null))? onTriggerFunction; works


raftstore-v2: pd worker

What is changed and how it works?
Issue Number: Ref https://github.com/tikv/tikv/issues/12842
What's Changed:

Related changes
Check List
Tests

Unit test

Release note
None

Could you give a description of how much is changed compared to v1 pd worker

Nothing changed (except for the file structure). Many non-critical parts are still missing and commented. @Connor1996

/merge

/run-test


lxqt.xdg-desktop-portal-lxqt: add libexif and menu-cache dependencies

Description of changes
xdg-desktop-portal-lxqt was failing because the libraries menu-cache and libexif were not being found.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2621


[new series] database

[ ] sql, nosql
[ ] how to speed up query
[ ] why is it a bad idea to store blob in database instead of pointer to file
[ ] full table scan
[ ] redis
[ ] elastic search

Is it safe to expose mongoDB ObjectID in URLs/APIs?


Fixed length donations during RD migration dry run

We fixed an issue with the handling of fixed length donations during Enhanced Recurring Donations migration dry run.
Critical Changes
Changes
Issues Closed
5409
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch:attach W-041168

Tracking W-041168

@force2b @RadSFDeveloper Ready for your review.
@sam-knox Please review PR description.
Thanks all!


Microsoft.RecoveryServices/vaults: sku can't be set to RS0

According to the spec https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/Microsoft.RecoveryServices/stable/2016-06-01/vaults.json, it should be possible to set "sku" to RS0, however when trying to set that I get:
Status=400 Code="InvalidRestApiParameter" Message="sku parameter is invalid.\r\nPlease provide a valid sku"

Another problem is that it isn't documented what the RS0 value actually does. I assume that it sets the storage redundancy type to LRS instead of GRS, but this is nowhere documented. If I change the redundancy portal in the portal, the generated ARM template changes:
from:
    "sku": {
        "name": "Standard"
    },

to:
    "sku": {
        "name": "RS0",
        "tier": "Standard"
    },

See also related issue: https://github.com/terraform-providers/terraform-provider-azurerm/issues/2274

Is this fixed ? This is blocking . We want to create LRS RSV(s) and not GRS , which we are unable to do now . Please let us know.

Same here!

Is this fixed ? This is blocking . We want to create LRS RSV(s) and not GRS , which we are unable to do now . Please let us know

I'd like to chime in on this too. I have a requirement to keep all data locally so need the RSV set to LRS. At the moment, I need to create the RSV, manually set LRS, then run VM enrollment separately.

Is there any reaction to this or does anyone have an idea?

As of now, you can only create vaults with GRS and then immediately update it with LRS. Once any backup items are protected, you CANNOT change this value.

#please-close

#please-close

why close! dont close! this is ridic, how you will code this? that destroys the meaning of automation and infrastructure as a code

Any chance to reconsider this ticket's closure?

#please-close

Yes, We cannot close it. you need to fix it, else it destroys the meaning of automation!

Has this issue been fixed ?

ABerTSC ,
Will terraform not try to change it to GRS on next run as per the input config ?

@turjachaudhuri, I created a vault, manually flipped to LRS, then re-ran apply. TF did not detect the change to LRS and did not try to flip it over.
However, you cannot set the vault to LRS after you onboard a client, essentially making onboarding a VM automatically (or any other service) into RSV completely worthless using terraform. So right now it's GRS or bust that I can see.

I'm also currently having an issue with this, and it's blocking a full IAC transition from ARM JSON to Terraform. As this is a business continuity/disaster recovery resource, please consider bumping this issue up in priority. Somebody initially reported this issue about two years ago in 2018, and it is now nearing the final quarter of 2020. We love this product; let's iron out this bug!

It has been almost 2 years since this was first reported. Has any progress been made on this?

There is a fundamental misunderstanding here. In order to update storage redundancy of the vault, SKU is NOT the right parameter. It is storageType and this API is required to be called. However, this API should be called immediately after vault creation and this value cannot be changed once any backup item is protected. So, the sequence of calls is 1) Create vault (PUT API) 2) Update storage config (PUT API) 3) Configure backups (PUT API).
Hope this helps in automation sequence.

The original reason why this issue was raised is because azurerm_recovery_services_vault resource used in Terraform cannot specify RS0 for local storage because the API returns that the SKU is invalid (see the first post).

But RS0 is not required for local storage. As I see, the comment was: "RS0 seems to be undocumented by Microsoft, but looking at the generated ARM template it seems to be what you need to set if you want LRS storage for the backups (instead of the default GRS)." This is not the case. You don't need to set the SKU to RS0 to set the value to local storage. By default, all vaults will be created by GRS (there is no choice here, as of now). So, you have to create the vault and then call the PUT API to update the storage redundancy configuration from GRS to LRS.
Hope this helps.
Hope this helps.

Deploy RSV for my firm and have same problem:
Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="InvalidRestApiParameter" Message="sku parameter is invalid.\r\nPlease provide a valid sku"
Refer to documentation:
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/recovery_services_vault
I can see that RS0 is supported. I DON'T WANT GRS, only LRS. I suppose Terraform can do it. Not normal since 2 years and no solution .... :-(
Please help !

@CedSadowski : As I mentioned earlier, SKU doesn't define LRS or GRS. You have to create a vault first and then update. Can you please file an issue/request with the terraform repo? I suppose they can help.

Hi,
To bypass this problem, I have defined my RSV and declare a script in AZ CLI to change immediatly the RSV SKU.
`resource "azurerm_recovery_services_vault" "vault" {
name                = "ced-recovery-vault"
location            = azurerm_resource_group.main.location
resource_group_name = azurerm_resource_group.main.name
sku                 = "Standard"
soft_delete_enabled = true
provisioner "local-exec" {
command = "az backup vault backup-properties set --backup-storage-redundancy LocallyRedundant --name ${azurerm_recovery_services_vault.vault.name} --resource-group ${azurerm_recovery_services_vault.vault.resource_group_name}"
}
}`

Hi,
To bypass this problem, I have defined my RSV and declare a script in AZ CLI to change immediatly the RSV SKU.
`resource "azurerm_recovery_services_vault" "vault" {
name                = "ced-recovery-vault"
location            = azurerm_resource_group.main.location
resource_group_name = azurerm_resource_group.main.name
sku                 = "Standard"
soft_delete_enabled = true
provisioner "local-exec" {
command = "az backup vault backup-properties set --backup-storage-redundancy LocallyRedundant --name ${azurerm_recovery_services_vault.vault.name} --resource-group ${azurerm_recovery_services_vault.vault.resource_group_name}"
}
}`

Any progress on this? I don't see an equivalent Azure backup policy for SQL databases as well.
I have been trying to do it via PowerShell modules, but that wouldn't work.
resource "azurerm_recovery_services_vault" "RSV"{
  name = "RSV"
  location = "eastus"
  resource_group_name = "RGTEST"
  sku = "Standard"

  provisioner "local-exec" {
    command = "Get-AzRecoveryServicesVault -Name ${azurerm_recovery_services_vault.RSV.name} | Set-AzRecoveryServicesBackupProperty -BackupStorageRedundancy LocallyRedundant"
    interpreter = ["powershell", "-Command"]
  }
}

For people looking at this, this is how we deploy the vault in bicep. Should be easy enough to build it into json.
`
resource RecoveryServicesVault 'Microsoft.RecoveryServices/vaults@2021-03-01' = {
name: VaultName
location: Location
tags: Tags
sku: {
name: 'RS0'
tier: 'Standard'
}
properties: {}
}
resource VaultConfig 'Microsoft.RecoveryServices/vaults/backupstorageconfig@2021-04-01' = {
name: '${RecoveryServicesVault.name}/VaultStorageConfig'
properties: {
crossRegionRestoreFlag: false
storageModelType: BackupType
}
dependsOn: [
RecoveryServicesVault
]
}
`

For those using Terraform, we had to use an ARM deployment right after creating the RSV to reconfigure it to use LRS (we did it in a Terraform module).
resource "azurerm_template_deployment" "rsv_lrs" {
  name                = join("", [azurerm_recovery_services_vault.rsv.name, "-lrs-config"])
  resource_group_name = azurerm_recovery_services_vault.rsv.resource_group_name
  template_body       = file("${path.module}/storage_type.json")
  deployment_mode     = "Incremental"

  parameters = {
    "vaultName"     = azurerm_recovery_services_vault.rsv.name
    "vaultLocation" = azurerm_recovery_services_vault.rsv.location
  }
}

{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "vaultName": {
            "type": "string",
            "metadata": {
                "description": "Name of the RSV"
            }
        },
        "vaultLocation": {
            "type": "string",
            "metadata": {
                "description": "Region that the RSV is in"
            }
        }
    },
    "variables": {},
    "resources": [
        {
            "name": "[concat(parameters('vaultName'), '/vaultstorageconfig')]",
            "type": "Microsoft.RecoveryServices/vaults/backupstorageconfig",
            "apiVersion": "2016-12-01",
            "location": "[parameters('vaultLocation')]",
            "properties": {
                "storageModelType": "LocallyRedundant",
                "storageType": "LocallyRedundant",
                "storageTypeState": "Unlocked"
            }
        }
    ]
}

For Terraform, LRS is now being supported with storage_mode_type which was introduced in v2.91.0 by https://github.com/hashicorp/terraform-provider-azurerm/pull/14659.
But I think there is still one question remaining, according to API Definition since sku property allows RS0 and Standard. However there is no official document explaining what this property is and how the value affects the recovery vault. @pvrk if this property is actually controlling something, I think it would be better to have a document for it, otherwise user won't be able to know how to use this property excepting trying and guessing it. If it's no longer used any more, maybe we can remove it in the new api version and put a deprecation notice in the document and in the issue


upload/download - return output

Sometimes it might be needed to get only final stats and not progress of rsync command.
Currently we can only do it by using verbose flag which is a bit noisy + changing upload configs flags to -az and adding --stats to options (or alternatively use rsync contrib recipe which also would need some code overwriting since it doesn't return run / runLocally output as well).
    upload(__DIR__ . "/bin/", '{{release_path}}', [
        'options' => ['--stats'],
        'flags' => '-az'
    ]);

If would be great if upload / download were returning final output so anyone can use it as desired if needed.

Nice idea! Or maybe as a configurable option? Can you create a PR for it?

Sure, will try to figure it out over the weekend.

May I assume this ticket is fixed @fliespl ? <3 If not please tell me, then I'll reopen.
Thanks for the PR!

@Schrank that's correct :) And you are welcome!


Update wagtail url paths for Django 2.0

url() is deprecated in Django 2.0 needs to be changed to re_path()
show the necessary django.url imports for path, re_path, and include just in case the project does not already have them

Thanks @marykatefain!


Unable to see my devices

I think you are French, so am I. But I'll keep it in english for the sake of everyone's understanding.
I am migrating from domoticz to Rfxcom2mqtt, and I am using the Home Assistant Add-on
I think I managed to configure everything correctly, knowing that there is not much to configure besides the MQTT server. This is perfect.
When I open my MQTT explorer, or when I look at the logs, I can see that a single device is getting added and updated. It's not even a device I own, it must be from someone in the neighborhood.
The device is getting updated very frequently, which make me think everything is working perfectly.
But I don't see any of my personal devices. I have tried many things, I have read the documentation ... But I can't find anything.
Here is what I see in MQTT explorer :
rfxcom2mqtt > bridge > info
{
  "coordinator": {
    "receiverTypeCode": 83,
    "receiverType": "433.92MHz transceiver",
    "hardwareVersion": "3.0",
    "firmwareVersion": 1038,
    "firmwareType": "ProXL 1",
    "enabledProtocols": [
      "BLYSS",
      "RUBICSON",
      "FINEOFFSET",
      "LIGHTING4",
      "RSL",
      "BYRONSX",
      "IMAGINTRONIX",
      "MERTIK",
      "LIGHTWAVERF",
      "HIDEKI",
      "LACROSSE",
      "LEGRAND",
      "BLINDST0",
      "BLINDST1",
      "X10",
      "ARC",
      "AC",
      "HOMEEASY",
      "MEIANTECH",
      "OREGON",
      "ATI",
      "VISONIC",
      "KEELOQ",
      "HOMECONFORT"
    ],
    "transmitterPower": 10,
    "noiseLevel": 107
  },
  "version": "1.1.1",
  "logLevel": "info"
}

And in my devices :
{
  "subtype": 10,
  "id": "0xA301",
  "seqnbr": 34,
  "temperature": 12.6,
  "humidity": 72,
  "humidityStatus": 3,
  "batteryLevel": 9,
  "rssi": 6,
  "type": "temperaturehumidity1",
  "deviceName": [
    "Rubicson",
    "IW008T",
    "TX95",
    "Xiron-EN6",
    "WH5"
  ],
  "group": false,
  "subTypeValue": ""
}

And here is what I had in domoticz :
ID        Name          Type        SubType
0x15553C $SCS 1/OFF      Lighting 4  PT2262
0x155533 $SCS 1/ON       Lighting 4  PT2262
0x155D03 $SCS 4/ON       Lighting 4  PT2262
0x84C271 [Ext] Sonette   Lighting 4  PT2262
0x15750C $SCS 5/OFF      Lighting 4  PT2262
0x157503 $SCS 5/ON       Lighting 4  PT2262
0x080001 Volets          RFY         RFY
0x155D0C $SCS 4/OFF      Lighting 4  PT2262
0x15570C $SCS 3/OFF      Lighting 4  PT2262
0x1555C3 $SCS 2/ON       Lighting 4  PT2262
0x1555CC $SCS 2/OFF      Lighting 4  PT2262
0x155703 $SCS 3/ON       Lighting 4  PT2262

I also noticed that in the RfxCom "information" tab, the protocol RFY was not listed.
Can you help me understand how I can add my devices to the Home Assistant Add-on

All right ... I went back to flashing my device to the latest version.
After flashing on my PC, I realized I still wasn't receiving my remote command, but only the old device from above.
I launched the RFX com windows configuration tool and changed the frequency to my region's frequency, and all of a sudden, I was able to see my remote commands.
All of the above is almost resolved. I still need to RTFM to get my blinds synchronized to my RFX com. I am unsure if the actual module will handle Somfy devices.

I do see commands from my remote, but I don't know how to integrate them in home assistant :
[2024-10-02 03:51:10][GATEWAY] INFO: receive from rfxcom : {"subtype":0,"seqnbr":60,"data":"0x155D03","commandNumber":0,"command":"Data","pulseWidth":175,"rssi":6,"type":"lighting4","deviceName":["PT2262"],"group":false,"subTypeValue":"PT2262"}
[2024-10-02 03:51:10][GATEWAY] INFO: publish to discovery : undefined
2024-10-02 15:51:10.565 [rfxcom] on /dev/serial/by-id/usb-RFXCOM_RFXtrx433XL_DO3RW1KO-if00-port0 - Packet type 13 handler threw exception TypeError: Cannot read properties of undefined (reading 'replace')

I guess I will stop using the module.
I do not understand how to integrate custom PT2262 devices. I have looked in many places, but couldn't find any documentation clearly explaining me how to convert an incoming message to a device.
I will have a try with the native rfx module


Migrate logs to OBS (S3 storage)

We use loki for gathering logs. Need to migrate storage of logs to OBS (S3 storage)

https://grafana.com/docs/loki/latest/fundamentals/architecture/deployment-modes/


Security Threats for NFTS and Possible Solutions

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

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

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

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

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

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

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

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

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

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

Grammar & plagiarism checks

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

Technical checks

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

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #7235

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

Plagiarism check done ✅

Hello @BrianaNzivu , It seems you have deleted some of the contents in the index.md with your commit.
Kindly check it out, so that I can work on the revision thanks.

Hello @BrianaNzivu, I have used Grammarly to correct grammatical mistakes 🚀 . Kindly review.
Thanks.

Jumping on this one

@Sreign254
Can you please reach out via a DM asap 👍
Thank you

Hello sir..

Hello @Sreign254
Are you in the EngEd community Slack?

@Sreign254
Need some help finding your account in the database


Update logback version in application logback toolkit

Update logback version in application logback toolkit, based on #813 , where I explained that the logback library is not really used, just for compiling the bridge. But anyway, no harm to fix the risk.
FYI @apache/skywalking-committers . This is not a real leak/risk.
CC @fredsjones

Coverage remained the same at 19.204% when pulling 4544d9967338f101df7e62f96af82e749b8a4bb4 on update/logbck-version into 8276eac7eea595512f1d7128173a69e7193e9c7a on master.


[MINOR] fix: fix log content of MultiReplicaClientReadHandler#readShuffleData

What changes were proposed in this pull request?
fix log content of MultiReplicaClientReadHandler#readShuffleData to different content for last server and others.
Why are the changes needed?
The existing log can make user stress, as we have multi replica or servers to handle shuffle data read, so we should warn to user until we cannot get all blocks from last server.
Does this PR introduce any user-facing change?
Yes, the read task log will be more friendly.
How was this patch tested?
Test locally.

@zuston Thanks for your review and suggestion, PTAL.


Support swift 5.9+ syntax.

This revision adds support for Swift Syntax 5.9 and above.

[!IMPORTANT]
It also points to the latest revision of @stackotter's fork of swift-argument-parser, which otherwise conflicts with the name of the GenerateManual plugin if not brought to that version. As well as updating the swift-log dependency to the latest v1.5.4.

@stackotter When I run:
./generate_schema.sh

I receive the following output:
Enum 'PlistValue' requires an explicit schema

And the entirety of Bundler.schema.json gets deleted, lol. I guess something got messed up...

@stackotter When I run:
./generate_schema.sh

I receive the following output:
Enum 'PlistValue' requires an explicit schema

And the entirety of Bundler.schema.json gets deleted, lol. I guess something got messed up...

lol whoops, yeah it works for me (on the commit before your changes), must've messed up something in the explicit schema parsing stuff (cause it's true that PlistValue requires an explicit schema, but it's also true that one is provided in its doc comment).

@stackotter actually the deprecations were so stupidly minimal I've included them as well to void stupid warnings, they were simple renamings.
~ also ew, I am unsure why my rebase committed your commits 😕

[!IMPORTANT]
Do not merge until ./generate-schema.sh runs successfully without file changes.

 Feel free to merge @stackotter 👍🏻

Also, since parsing the documentation alone was half the logic in the parse(from:) function, it was making the core parsing logic unclear, I opted to move documentation parsing into it's own function: parseDocumentation(from:), and the guard was removed from parse(from:) because we still want the property even if there is no doc comment.

Remove needless stack allocation for docs.

The ultimate premature optimisation 😅 (don't think that affects performance at all, swiftc rearranges things and the stack is constant sized within a function, see this godbolt experiment).
Lgtm 👍

To be honest, I wasn't aware swiftc was that smart, lol!

Hahah yeah it's pretty smart, today I found out some pretty cool tricks it uses to optimise converting enums to raw values (and vice versa)


Fixed inverted result of PAL_isManualFadeDone

Haha, definitely done that one too quickly !
Thanks for fixing the typo ;-)


Issue when log in embed facebook and youtube

Bug description:
Hi everyone, I am facing an error related to account login on platforms like facebook and youtube. When I login with youtube and facebook to embedded website from webview link these errors appear . The page loading login to foverer in google platform "One moment please..." , and facebook platform is "Please close this tab" Hope you guys can help me deal with it
My code is
<WebView           javaScriptEnabled={true}           originWhitelist={['*']}           onMessage={(e) => getMessage(e)}           userAgent="Mozilla/5.0 (Linux; Android 4.1.1; Galaxy Nexus Build/JRO03C) AppleWebKit/535.19 (KHTML, like Gecko) Chrome/18.0.1025.166  Safari/535.19"           useWebKit           bounces={true}           allowsInlineMediaPlayback={true}           mediaPlaybackRequiresUserAction={false}           source={{             uri :"https://drrabbittutor.com/",           }}           ref={webview}           style={styles.webView}           onNavigationStateChange={handleNavigationChange}           mixedContentMode="always"           allowsFullscreenVideo={true}           allowFileAccessFromFileURLs={true}           allowingReadAccessToURL={true}           allowsBackForwardNavigationGestures={true}           allowUniversalAccessFromFileURLs={true}           domStorageEnabled={true}           setSupportMultipleWindows={true}           javaScriptCanOpenWindowsAutomatically={true}           thirdPartyCookiesEnabled={true}           allowFileAccess={true}         />
To Reproduce:
Expected behavior:
Expected to user can login facebook and youtube from webview
Screenshots/Videos:

Environment:

OS:MacOs
OS version:11.2.1
react-native version:0.62.2
react-native-webview version:9.0.1

Facing same issue with Apple login

I think the problem from login facebook and apple need for each sdk for mobile version, I currently using webview userAgent for web version facebook will know it and block for login, you can change userAgent for mobile version or if not solve issue. I have 2 options for solve this problem, the first is using platform sdk , ex : facebook has lib for react-native sdk is fbsdk, the second if stituation i want in inheriant web version is using lib react-native-inapp-browers. Maybe it for help you


Why does a SQLModel class with table=True not validate data ?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from pydantic import BaseModel
from sqlmodel import SQLModel, Field

class BaseModelTest(BaseModel):
    id: int
    some_bool: bool
    desc: str

class SQLModelTest(SQLModel, table=True):
    id: int = Field(default=None, primary_key=True)
    some_bool: bool
    desc: str

# This will work as expected
sqlmodel_instance = SQLModelTest(id=3, some_bool=False, desc="This will work")
print(sqlmodel_instance)
# This will work as expected
pydantic_instance = BaseModelTest(id=3, some_bool=False, desc="This will work")
print(pydantic_instance)

# This will work but should not
sqlmodel_instance = SQLModelTest(id=3, some_bool="blob", desc=False)
print(sqlmodel_instance)
# This will not work as expected
pydantic_instance = BaseModelTest(id=3, some_bool="blob", desc=False)
print(pydantic_instance)

Description
When creating a SQLModel class instance, I thought that I'd have all the advantages of pydantic and SQLAlchemy classes toghether.
I was quite surprised when I realized that SQLModel classes created with table=True do not validate data as a pydantic class would do (see my example code).
Having read about all SQLModel documentation, I didn't find anywhere this is stated.
Especially in https://sqlmodel.tiangolo.com/tutorial/fastapi/multiple-models/ it would make sense to explain that a table=True won't do data validation as a pydantic class would do.
Maybe I did not understand the purpose of SQLModel enough, but this is quite disturbing for me.
I think it would be really nice to update the documentation regarding this particular point.
Operating System
Windows
Operating System Details
No response
SQLModel Version
0.0.8
Python Version
Python 3.10.7 x64
Additional Context
PS: Sorry, I'm comming here from Tortoise-ORM, and it's my first time using SQLModel, so I cannot guarantee that I will be quite helpful resolving issues here yet ;) I'll do my best.

To guess, when used as a table object, there is no verification process because it is used to map data that has already been verified in database.
(i think because SQLModel use DeclarativeMeta when table=True. link)
In particular, when calling many objects from database, there is no unnecessary verification process, so there is a performance advantage.
Of course, if the verification process is essential, SQLModel.validate method can be used.
I've asked the same question before, but I'm just using SQLModel.validate method for now. I want to know why, too.

I do understand that too, reads from database don't need validation. But writes do. Nevertheless it's not really clear in the documentation.
And since SQLModel is supposed to reduce code duplication, having a class for ORM and a class for Pydantic validation.... doesn't help deduplicate.

@deajan
I don't know the answer to why, but I found a simple way to solve it.
link

I think this is answered here: https://github.com/tiangolo/sqlmodel/issues/52#issuecomment-1311987732
For future reference, I found this while looking for why model_validate_json did not properly validate for a SQLModel with table=True.

I agree - this just totally threw me off. I expected it to be fully validating my table data.  why bother inheriting from BaseModel at all if not.

This feature/bug should be fully documented, as I (and many others) expect the SQL model to validate data. Only after doing a lot of digging to find out why one of my field_validator wasn't working did I come across this GitHub issue.

This was also very surprising to me. Looks like this PR may fix the issue, although it's unclear if it's the right approach.


MNT: Update pinned versions.

Hi! This is the friendly conda-forge-admin automated user.
I've bumped some of the conda-forge pinned versions, and noticed that it impacts this feedstock.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

hi @conda-forge/core, I get one of these MNT: Updated pinned versions PR every 3 or 4 days. Is there a way to find out how a given feedstock is affected?
cheers, piem

There's no point in merging them. They are caused by a buggy script. Can you update the dependency ffmpeg to ffmpeg 2.8.* in both the run and build section manually?

@croth1 oh, good to know! ffmpeg dep updated.


add support for Emojicode

Tweaks are welcomed.

Thanks for yet another language support @geeseven, very happy to see that you took advantage of the TrueColors feature.
With 7728237, I took the liberty to switch to brighter colors to make it more visible on dark terminals.

Emojicode is an open-source,
full-blown programming language
consisting of emojis.
...a m a z i n g :heart_eyes:

With 7728237, I took the liberty to switch to brighter colors to make it more visible on dark terminals

I debated doing the same, but went with color accuracy.
I also toyed with the idea of making the logo completely out of 🍇.


"Bosanci" is listed in the drop-down menu

"Bosanci" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Bosanci" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Bosanci, SUCEAVA".


https://github.com/tchasinga/space-travelers

in this project we've two members :
tchasinga
Log-benjamin

Hi @tchasinga and @Log-benjamin,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request
Kindly provide the kanban link in the issue
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Update envoyproxy go-control-plane

Update to the latest version of go-control-plane so we can make use of the new removeRequestHeaders field for https://github.com/istio/istio/pull/8984

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

Needs approval from an approver in each of these files:

OWNERS

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
10cb97716896b2360c0534f9a6f863e873da7f08
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rshriram]

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


Kube state metrics pods restarting continuosly after upgrade

Hello Team,
The kube-state-metrics container is continuosly restarting after upgrading to the latest version
Image : tag: v1.9.7
Trace[443632888]: [15.007406853s] [15.007406853s] Objects listed
Trace[443632888]: [19.347061718s] [4.339539637s] SyncWith done
E1109 13:18:24.465091       1 reflector.go:156] pkg/mod/k8s.io/client-go@v0.0.0-20191109102209-3c0d1af94be5/tools/cache/reflector.go:108: Failed to list *v1.MutatingWebhookConfiguration: the server could not find the requested resource
E1109 13:18:24.466749       1 reflector.go:156] pkg/mod/k8s.io/client-go@v0.0.0-20191109102209-3c0d1af94be5/tools/cache/reflector.go:108: Failed to list *v1.ValidatingWebhookConfiguration: the server could not find the requested resource
E1109 13:18:27.627646       1 reflector.go:156] pkg/mod/k8s.io/client-go@v0.0.0-20191109102209-3c0d1af94be5/tools/cache/reflector.go:108: Failed to list *v1.MutatingWebhookConfiguration: the server could not find the requested resource
E1109 13:18:28.123469       1 reflector.go:156] pkg/mod/k8s.io/client-go@v0.0.0-20191109102209-3c0d1af94be5/tools/cache/reflector.go:108: Failed to list *v1.ValidatingWebhookConfiguration: the server could not find the requested resource
E1109 13:18:29.335506       1 reflector.go:156] pkg/mod/k8s.io/client-go@v0.0.0-20191109102209-3c0d1af94be5/tools/cache/reflector.go:108: Failed to list *v1.MutatingWebhookConfiguration: the server could not find the requested resource
E1109 13:18:29.731342       1 reflector.go:156] pkg/mod/k8s.io/client-go@v0.0.0-20191109102209-3c0d1af94be5/tools/cache/reflector.go:108: Failed to list *v1.ValidatingWebhookConfiguration: the server could not find the requested resource
But we can see the clusterrole has permission to list and watch these resources.

apiGroups:

admissionregistration.k8s.io
resources:
mutatingwebhookconfigurations
verbs:
list
watch

apiGroups:

admissionregistration.k8s.io
resources:
validatingwebhookconfigurations
verbs:
list
watch

Any help on this is appreciated. I see this is a duplicate of the issue https://github.com/kubernetes/kube-state-metrics/issues/1093 but I dont think that issue is fixed as well. Any help on this is appreciated

What version is your kubernetes cluster?
cc @lilic

Hey! Like Frederic mentioned can you detail your cluster version and the manifest you are using to deploy kube-state-metrics and the RBAC files.

https://github.com/kubernetes/kube-state-metrics/issues/1090#issuecomment-728698307
Repeating myself:
I was having this issue with a cluster on v1.15.12-gke.20 version.
The most recent version of kube-state-metrics that worked for me without flooding the logs with these errors was: v1.9.4

@rpherrera yes you are using a too new version of kube-state-metrics, see compatibility matrix here https://github.com/prometheus-operator/kube-prometheus#kubernetes-compatibility-matrix or the one in kube-state-metrics https://github.com/kubernetes/kube-state-metrics/#compatibility-matrix neither claim to support 1.15 version of Kubernetes, so might be easier to roll back to 1.9.4 or earlier supported versions. We fixed a bug in that version of kubernetes to add support for newer versions, so that is what you are seeing.

Closing as this appears to be answered. Please reopen if you think this is still relevant.


PATRIC AntibioticResistance Data Landing Page has reverted again

https://www.patricbrc.org/portal/portal/patric/AntibioticResistance
Reachable from carousel Investigating Antibiotic Resistance link or from Data / Antibiotic Resistance dropdown
In the description at the top, it Is back to saying "In the near future, we will also incorporate antibiotic susceptibility testing results as additional genome metadata"   I know that Ron has fixed this numerous times.  We need to find a way to keep the website from reverting to old content

Harry,
It looks like the content on data landing page is not in sync with what is in wordpress.
Can you please  check?
https://www.patricbrc.org/portal/portal/patric/AntibioticResistance
This is what it says on in the corresponding wordpress page wright now:
Antibiotic Resistance refers to the ability of bacteria to develop resistance to antibiotics through gene mutation or acquisition of antibiotic resistance genes. We collect known antibiotic resistance genes from reputed external databases and publications and map them to homologous genes in PATRIC using BLASTP. For more information see Specialty Genes FAQs.
We also incorporate antibiotic susceptibility testing results as additional genome metadata. Summary filterable AMR metadata are available from PATRIC genome lists (see Antimicrobial Resistance and Other Clinical Metadata). Additional AMR data (including panel data when provided), are available for download.

This keeps happening.  I've pushed the update several, several times.
Harry figured it out at some point, but I don't know what the long term
fix is.
Ron Kenyon
PATRIC Project Manager, patricbrc.org
Project Management Director
Virginia Bioinformatics Institute, Virginia Tech
Ph:<PHONE_NUMBER>
On 8/4/2015 5:26 PM, mshukla1 wrote:

Harry,
It looks like the content on data landing page is not in sync with
what is in wordpress.
Can you please check?
https://www.patricbrc.org/portal/portal/patric/AntibioticResistance
This is what it says on in the corresponding wordpress page wright now:
Antibiotic Resistance refers to the ability of bacteria to develop
resistance to antibiotics through gene mutation or acquisition of
antibiotic resistance genes. We collect known antibiotic resistance
genes from reputed external databases and publications and map them to
homologous genes in PATRIC using BLASTP. For more information see
Specialty Genes FAQs.
We also incorporate antibiotic susceptibility testing results as
additional genome metadata. Summary filterable AMR metadata are
available from PATRIC genome lists (see Antimicrobial Resistance and
Other Clinical Metadata). Additional AMR data (including panel data
when provided), are available for download.
—
Reply to this email directly or view it on GitHub
https://github.com/PATRIC3/patric3_website/issues/372#issuecomment-127767236.Web
Bug from
https://github.com/notifications/beacon/ADCnWq8nilv7CS4w4CWpx9UaPYEpY7LIks5okSWNgaJpZM4Fk3_u.gif

DLP caches will be generated at deploy time.


TsLint => EsLint

Description
Tslint has been deprecated
see https://github.com/palantir/tslint/issues/4534
The idea of this PR is to have only eslint across all the code, but keep specific settings for different types of code:

typescript will use typescript-eslint https://github.com/typescript-eslint/typescript-eslint/blob/master/README.md

same rules for Ts should apply to the SDK and manager folders
Manager will have React, a11y, and hooks checks (equivalent to the previous config)
rules from the Tslint should be ported (either by default, or by adding the rules) to eslint

e2e tests of cypress: will use eslint with the eslint-cypress plugin

cypress/support should support for ES6 like code
cypress/plugin should support Node like code

storybook will use eslint with some node specific specific checks (var, require)
e2e/ : is ignored as it is mostly deprecated

Also:

The Prettier config will also be like for tslint be integrated to eslint, and should not change

It is not a goal of this PR to make the fixes for these eslint warnings
I made some changes to configs/scripts/cypress for minor tests, not to src/ or the sdk.
Choices of change made:

I use .eslintrc.js not the json format or yaml, because it felt like one of the most common, and .js supports comments
Multiple .eslintrc.js, in subdirectories to have different settings per folder
when the code styles are different in a same folder (For instance storybook mixes with the ts code) Se use overrides to override the rule of the folder on files matching a pattern
I decided to apply the Prettier config to js files too, and i updated the files

scripts

lint staged is used as before, but not will trigger an error on commit for Js files too

specific rules
Default are here:
https://github.com/typescript-eslint/typescript-eslint/tree/master/packages/eslint-plugin
array-type we had this disabled, but no error. We now use default (off)
ban-types: was disabled, but no error, we use default now (on)
interface-name => actual rule is interface-name-prefix, was off, but no error, is now default
member-* rules were off, that is the default value
no-namespace was off, but no error, we now use default value: on
react rules https://github.com/yannickcr/eslint-plugin-react
jsx-boolean-value was off, no error, check there is no implicit bool cast. Now default is always
jsx-no-lambda was false, we have no error with new setting, now default
Other:
no-console is WARN level, except in scripts/config/cypress where it is off
sort-keys is a WARN level
no-unused-vars and @typescript-eslint/no-unused-vars are Warn, prepend an argument with _ to indicate that you know it is unused
rules of hooks
I do not know hooks enough, this error is reported in ResizeCluster it seems like it makes sense that we return early if cluster is null, although this rule would enforce that you do not call hooks conditionally (may it mess up react state?)
@Jskobos you would probably know best
no-empty-interface
Interface definition (or extension) without anything in it is discouraged
=> Here we use this rule but skip a lot (>100 times) using inline comments, we could actually.fix this by using type aliasing:
/* tslint:disable-next-line:no-empty-interface */
// export interface AppBarProps extends _AppBarProps {}
export type AbbBarProps = _AppBarProps;

TODO:

[x] port each rule of previous config
[x] clean formatting of eslintjs files themselves
[x] check that the script work as planned during commit and push (review w/ team)
[x] remove all traces of tslint configs
[x] CANCELLED: clean inline tslint rule comments, this would add changes all over the code, it should be done incrementally
[x] Should we keep sort keys, as warn on all files or not?
[x] Could we use type aliasing instead of empty interfaces
[x] Update docs

Type of Change

Non breaking change ('update', 'change')

Note to Reviewers
Check if the rules seem to work
yarn lint runs without autofix in quiet mode (only errors) in root or packages
to run directly use --ext
eslint ./path/ --ext .js,.ts,.tsx
It will automatically load the right lint config.
In VSCode with Eslint plugin, remember to add  --ext .ts,.js,.tsx

A lot of rules are set at the Warn level to avoid alert fatigue, we can turn them back on.
We currently have 286 errors in manager and 1  in SDK
=> 147 have autofix

On the newest version the error rate is pretty high, which is both good and bad:

A lot of this is that the config (or maybe it's just ESLint) doesn't handle type imports very well:

If it's possible to tweak the config so it doesn't issue warnings for these, that would be great. Plenty of the errors/warnings seem to be legit things we should fix; I think followup PRs to resolve some of the warnings and errors is a good idea.

Thanks Jared for the comment, i ll look into these import warnings to see if this error is more noise, or a setup issue.

Agreed,
On the accessibility warnings, i think the difficulty is more that we do not understand some of these rules well, and i feel wary of disabling them
On the list of things to potentially disable:

cognitive complexity
display-name
Functions that return promises must be async

Other suggestions?

Something seems wrong:

Running yarn lint on my machine took ~10 minutes (I did this twice)
I can't commit anything because I get the following error:

  0:0  error  Parsing error: "parserOptions.project" has been set for @typescript-eslint/parser.
The file does not match your project config: .eslintrc.js.
The file must be included in at least one of the projects provided

Is this just me? I've been reading up on how to config this, haven't run into anything obvious yet.

Looking into it, the parserOptions.project = tsconfig, is set in both packages .eslintrc.js


Add recipe for fried chicken

I added a recipe for fried chicken

Add Tittle, Instructions and Ingredients in markdown format