How to send emoji in the comments on windows?

Before submitting an issue, make sure you have:

[x] Updated to the latest version
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

How to send emoji in the comments on windows?
I want to use the ultimate schedule.
When I worked on Linux, everything was fine.
But now I would like to transfer everything to Windows.
I suffered for a long time with encodings and I still didn’t work ...
All I got was an error:
UnicodeDecodeError: 'charmap' codec can't decode byte 0x98 in position 2: character maps to
Or instead of comments, there was something like this:
РєРЅРє
яю;C;
Р»СѓР»
п»їР»СѓР»
I'm already tortured ... nothing happens.
If this is not possible, then you will have to return to Linux.

are you using python 3?

@ohld Yes. Need to use python 2?

I am not on Windows, I use macOS Mojave ...I just copied the Emoji from https://getemoji.com/ and pasted it straight into the ultimate schedule comments I wanted the bot to comment with. Have you tried this?

Not. I have not tried. But I have no doubt that it will work, as well as on Linux. I'm just wondering if this is possible on windows.
The fact is that when I send Emoji via Instabot to Direct (on Windows), Emoji looks fine. But when I leave comments, they vilify like strange signs.
To send in direct, made this:
`#!/usr/bin/env python3
-- coding: utf-8 --
from glob import glob
import os
import sys
import threading
import time
import datetime
from tqdm import tqdm
sys.path.append(os.path.join(sys.path[0], '../'))
import schedule
from instabot import Bot, utils
username = "super_mega_username"
password = "super_mega_password"
USERSLIST = "USERSLIST.txt"
MESSAGE = open("MESSAGE.txt", "r", encoding="utf8").read()
SENDED_USERSLIST = "SENDED_USERSLIST.txt"
bot = Bot()
bot.login(username=username, password=password)
wait = 1500  #Delay
for user in utils.file(USERSLIST).list:
if user not in utils.file(SENDED_USERSLIST).list:
try:
current_time = datetime.datetime.now().strftime('%H:%M:%S')
if bot.send_message(MESSAGE, user):
print (current_time, " >>> Sent a message to user: " + user)
open(SENDED_USERSLIST, "a").write(user + "\n")
open("MESSAGE.txt", "r", encoding="utf8").close()
time.sleep(wait)
else:
print (current_time, " >>> Can't send message to user: " + user)
except:
pass
`

@ohld
@arlidge, No. I have not tried. But I have no doubt that it will work, as well as on Linux. I'm just wondering if this is possible on windows.
The fact is that when I send Emoji via Instabot to Direct (on Windows), Emoji looks fine. But when I leave comments, they vilify like strange signs.
To send in direct, made this:
`
#!/usr/bin/env python3
from glob import glob
import os
import sys
import threading
import time
import datetime
from tqdm import tqdm
sys.path.append(os.path.join(sys.path[0], '../'))
import schedule
from instabot import Bot, utils
username = "super_mega_username"
password = "super_mega_password"
USERSLIST = "USERSLIST.txt"
MESSAGE = open("MESSAGE.txt", "r", encoding="utf8").read()
SENDED_USERSLIST = "SENDED_USERSLIST.txt"
bot = Bot()
bot.login(username=username, password=password)
wait = 1500 #Delay
for user in utils.file(USERSLIST).list:
if user not in utils.file(SENDED_USERSLIST).list:
try:
current_time = datetime.datetime.now().strftime('%H:%M:%S')
if bot.send_message(MESSAGE, user):
print (current_time, " >>> Sent a message to user: " + user)
open(SENDED_USERSLIST, "a").write(user + "\n")
open("MESSAGE.txt", "r", encoding="utf8").close()
time.sleep(wait)
else:
print (current_time, " >>> Can't send message to user: " + user)
except:
pass

@EightShift - I think because you are encoding with utf8. I think emojis have two utf8 encodings...  I have not seen that in any of the scripts that comment. In the .txt file for my comments I just paste the emoji directly and it works, all in python3.

@arlidge i know. my script for sending emojies in direct - works correctly. But, when i try to send a comment with emojies using ultimate method, i dont get wanted result. Maybe instabot have another method to send comments?
When i using linux, i using emojies in txt like you. And i want to get this on windows, but cant :(
@ohld

Im not too sure then mate, I thought it would be the same because it's python. Keep hunting, hopefully someone can help you soon. Sorry I am of no help.

@ohld, @arlidge I GET IT!
in utils.py i added encoding="utf8":
    @property
    def list(self):
        with open(self.fname, 'r', encoding="utf8") as f:
            lines = [x.strip('\n') for x in f.readlines()]
            return [x for x in lines if x]

and emojis looks fine!!!
but i have an error in console, and i added .encode("utf-8") in bot_comment.py:
def comment_medias(self, medias):
    broken_items = []
    self.logger.info("Going to comment %d medias." % (len(medias)))
    for media in tqdm(medias):
        if not self.is_commented(media):
            text = self.get_comment()
            self.logger.info("Commented with text: %s" % text.encode("utf-8"))
            if not self.comment(media, text):
                self.delay('comment')
                broken_items = medias[medias.index(media):]
                break
    self.logger.info("DONE: Total commented on %d medias. " %
                     self.total['comments'])
    return broken_items

...and goodbye error :)

Woohoo :) stoked you got it figured out! Now onto automation. 💯


Fix ADS stream hanging on errors

Currently, if we have an error in Recv before initialization completes,
we may hang forever, since we have <-con.initialized that blocks. This
fixes this by closing the initialized channel when recieve exits.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

I saw the label - +1 for 1.9

Yes

In response to a cherrypick label: new pull request created: #30876

This is not the best fix, imagine when the con.initialized is closed, it might enter the case
https://github.com/istio/istio/blob/fd6ecf7e4edbb080c31a0146c856c30ae66efb87/pilot/pkg/xds/ads.go#L330-L335

Hmm great point! looks like select order is not defined either:
https://play.golang.org/p/sIGqmdwfcgs
On Thu, Feb 18, 2021 at 6:52 PM Zhonghu Xu<EMAIL_ADDRESS>wrote:

This is not the best fix, imagine when the con.initialized is closed, it
might enter the case
https://github.com/istio/istio/blob/fd6ecf7e4edbb080c31a0146c856c30ae66efb87/pilot/pkg/xds/ads.go#L330-L335
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/30875#issuecomment-781777526, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXKMVL3ZXJDZN2TYGZDS7XG7DANCNFSM4XXF6AVQ
.

This is not the best fix, imagine when the con.initialized is closed, it might enter the case

Actually, I think its not an issue:

If proxy is initialized succesfully, behavior is same as before
If proxy is never initialized, then we do not add it to client list, and therefor will never get a push on pushChannel
I will send a PR for regression test

If proxy is never initialized, then we do not add it to client list, and therefor will never get a push on pushChannel
I will send a PR for regression test

This is not right. From the initConnection function, the client is added first, and then initialized channel is closed.

If the proxy is successfully initialized this PR has no impact. The only change here is when the proxy is not initialized (due to bad node id for example)

Not a big deal, just one unnecessary push. In this case, if it is initiated successfully, and the stream recv fails.


Adds repository cohorts content.

Repository cohorts content
This coming in via fork as needed to make into fork to test setup for github pages page worked without needing PR approval for each small change during debugging.
Adds a directory called repository_cohorts that has a README and another directory called framework which holds an Observable Framework generated static site that visualizes repository cohorts for a one time fixed export of public metadata for Microsoft governed repositories.
Doing PR into cohorts branch and will turn on github pages for that branch during development. Have working draft. Want to get feedback from team and partners before further work, so github pages page needs to be live.

Thanks James. Will take the minified source out later.


bash completion for docker search --limit

Ref. #23107
Thanks for the ping @thaJeztah

Thanks @albers for covering the completion.

LGTM 🐼
/cc @thaJeztah @justincormack @runcom

LGTM

Thank you!!


Incorrect JSX Line Comment of Attribute (Re-open issue #36175)

VSCode Version: 1.29.1
OS Version: macOS 10.14.1 (18B75)
I apologise for opening a new issue for an issue that has previously already been closed+locked (and apparently, solved): #36175. I am still experiencing this and suspect that it is due to some flawed extension as mentioned here. Unfortunately, the extension(s) at fault are not mentioned and I have no idea which ones of mine are causing the problem.
(Also, I don't think this is relevant but I noticed that this issue seems to only happen in js/jsx and not ts/tsx.)
Thank you.
List of extensions in use:

Tyriar.terminal-tabs
aaron-bond.better-comments
AlanWalk.markdown-toc
alefragnani.Bookmarks
christian-kohler.path-intellisense
CoenraadS.bracket-pair-colorizer
dbaeumer.vscode-eslint
dinhani.divider
eamodio.gitlens
eg2.vscode-npm-script
esbenp.prettier-vscode
formulahendry.auto-close-tag
formulahendry.auto-rename-tag
gayanhewa.referenceshelper
Gruntfuggly.activitusbar
joshpeng.sublime-babel-vscode
joshpeng.theme-charcoal-oceanicnext
mikestead.dotenv
monokai.theme-monokai-pro-vscode
ms-vscode.sublime-keybindings
ms-vscode.typescript-javascript-grammar
ms-vscode.wordcount
NuclleaR.vscode-extension-auto-import
PKief.material-icon-theme
rbbit.typescript-hero
robertohuertasm.vscode-icons
Shan.code-settings-sync
shd101wyy.markdown-preview-enhanced
timreilly.typewriter-sounds
wayou.vscode-todo-highlight
xabikos.JavaScriptSnippets
xabikos.ReactSnippets

This is likely caused by the sublime-babel-vscode grammar extension
I just tried #36175 with all extensions disabled and confirmed we still get the correct comment style

Thank you Matt; you're right, I've been able to verify that disabling the sublime-babel-vscode extension gets the comment style to work correctly.


#75 ticket save logic

model logic updated + 1 test

Current coverage is 94.65% (diff: 100%)

Merging #104 into master will increase coverage by 0.26%

@@             master       #104   diff @@
==========================================
  Files            23         23
  Lines           499        524    +25
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            471        496    +25
  Misses           28         28
  Partials          0          0

Powered by Codecov. Last update f9236dc...60fb1db

Thanks for feedback :) should be fixed now...


Adding RouteBuddy

RouteBuddy commercial mapping tool - can be updated to full product with key

Thank you for the contribution. It needed some fixes, so they were made in the referenced pull request. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

Thanks!


metering

Is there any way to reliably retrieve metering information from this player. I know this is relatively straight-forward with an AVAudioPlayer class but I can't seem to find a way to do it using MPMusicPlayerController.
Thanks

GVMusicPlayerController is using AVPlayer, so you'll have to look into that. Can't really help you, sorry.


not able to access and run my java application

Hi all,
what i did --
downloaded tomcat:8.0-jre8 from the docker hub
docker file written
created/build my gol image
ubuntu@ip-172-31-92-196:~/images/game-of-life$ docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
gol                 1.0                 8f957fa0b467        12 hours ago        466MB
gol                 1.2                 8f957fa0b467        12 hours ago        466MB
tomcat              8.0-jre8            5a2f2ba17238        2 weeks ago         463MB
ubuntu@ip-172-31-92-196:~/images/game-of-life$
now i m trying to run this image but getting below error,

Thanks khaja sir ,issue resolved actually there was issue with port conflict .


How to utilize WAMR with SGX in my own .wasm app？

For example, I have a rust app and compile to .wasm.
I want run this .wasm in the SGX enclave.
I find the mini-product linux-sgx and samples in it.
But how can I run my app in SGX enclave with WAMR runtime?
Is there any guide or examples? Thanks.

And in warmc, I found -sgx feature, it describe "generate SGX code", what doest it mean? Is that means, when i exec iwasm xxx.aot, it will run in SGX enclave?

@leyao-daily currently the wasm file is statically linked to runtime by byte array, you can replace file product-mini/platforms/linux-sgx/enclave-sample/Enclave/test_wasm.h with your own file by using binarydump to generate it:
cd test-tools/binarydump-tool
mkdir build && cd build
cmake .. && make
./binarydump -o test_wasm.h -n wasm_test_file test.wasm
(or ./binarydump -o test_wasm.h -n wasm_test_file test.aot)
remove "__aligned(4)" in test_wasm.h
cp -a test_wasm.h ${WAMR_ROOT}/product-mini/platforms/linux-sgx/enclave-sample/Enclave/
cd ${WAMR_ROOT}/product-mini/platforms/linux-sgx/enclave-sample
make clean && make

We have basically implemented running wasm file in host by sending it from host to enclave to execute with sgx ecall, we will submit the patch soon.
For the AoT, yes, wamr supports running aot file in SGX, by default it is disabled, you can use cmake -DWAMR_ENABLE_AOT=1 to enable it when building linux-sgx:
cd ${WAMR_ROOT}/product-mini/platforms/linux-sgx/
mkdir build && cd build
cmake .. -DWAMR_BUILD_AOT=1

@wenyongh It is very helpful, Thanks a lot. 👍

Welcome @leyao-daily , we have submitted a new patch, now the enclave sample is built as binary iwasm, which can run host wasm file or aot file (send to enclave to run), similar to iwasm built in Linux platform, please pull and try again.

Excellent! I have tried right now and it works well.
Another question: I complie .wasm to aot file with warmc, and one with no -sgx and one with warnc -sgx -o xx.aot xx.wasm
Both of them can be run by linux-sgx iwasm.
So what is the usage of the -sgx in warmc command. It describe "generate SGX code". What is that mean, what code it generates? Thanks again.

Please had better use -sgx option to generate aot code for SGX platform, or llvm codegen might generate machine code with unsupported relocation type for SGX, and we cannot run that code in SGX.


Easier way to access the site

Prints IP in the console along with success print out

What about multi-homed systems? I fear that if this spits out an RFC 1918 IP when, in fact, another interface is serving on a publicly-facing IP, that could be misleading.

What do you think of the QR code addition?


Implement 'backwards tab'

This escape sequence isn't officially documented anywhere, but is
supported by xterm and gnome-vte.

Looks good; thanks for the PR!


This is my approach to add the feature I requested on #18

I requested a feature on #18 and after seeing the source I think this should add what I wanted however I am not really experienced with mustache and (since I am a bit of a noob with js) I couldn't find how to test tis solution. However I think the code is quite simple.
I did 3 things:

on index.js I added a flag to allow for the -0 in the name of the output to be dropped (making the output look like atlas.json, atlas-1.json, atlas-2.json...) This allows for seamless fallback in case you are building with a pipeline like webpack. (By default this flag is false to avoid breaking any existing behaviour)

on FilesProcessor.js I added two lambdas, one to create a list of all the altas definition files that will be created and another to exclude the current file from the array (an atlas definition must not reference itself as part of the multipack). I also added the corresponding checks for the new flag introduced on index.js

on JsonHash.mst I added what I think is the correct syntax for an array of string in the meta object under the name relatedMultiPacks. I am not familiar with the other export templates so I didn't dare to modify them.

Please test this before merging because I couldn't find a way to test it due to my inexperience with javascript library programming.

I managed to splice the lib into my webpack dependencies and tested it. It now works flawlessly.
Also added some sanity checks so that the json is not polluted with an empty array when not using the multipack feature.

using it into our pipeline I found that an array gets reused so I needed to clean the flag

I don't think this is a necessary feature. If you need to change the naming of files, you can do so with your build system (webpack or gulp for example) after packing.


Error occurs when using union in query

Describe the bug
in pg
postgres=# select 1 union select 2;
 ?column?
----------
        1
        2
(2 rows)

in df
❯ select 1;
+----------+
| Int64(1) |
+----------+
| 1        |
+----------+
1 row in set. Query took 0.002 seconds.
❯ select 2;
+----------+
| Int64(2) |
+----------+
| 2        |
+----------+
1 row in set. Query took 0.001 seconds.
❯ select 1 union select 2;
Plan("UNION ALL schemas are expected to be the same")

I think there two problems:

can not get result.
use union but get UNION ALL: i think they are totaly different

related to #2032
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

I will try to fix this😊

@Ted-Jiang I think there is a duplicate issue: https://github.com/apache/arrow-datafusion/issues/1818

@Ted-Jiang I think there is a duplicate issue: #1818

Thanks for your info❤

It doesn't work in the current version.
select 1 union select 2;
ArrowError(ExternalError(Execution("Arrow error: External error: Internal error: Unsupported data type in hasher: Null. This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker")))
SELECT 1 a UNION ALL SELECT 2;
+-------------+
| placeholder |
+-------------+
|             |
| 1           |
+-------------+

I fixed it. Now result is:
select 1 union select 2;
+----------+
| Int64(1) |
+----------+
| 2        |
| 1        |
+----------+

SELECT 1 a UNION ALL SELECT 2;
+---+
| a |
+---+
| 1 |
| 2 |
+---+


Create an Included Image with Node 16.10 For Angular 13 Support

The latest version of Angular 13 does not support node versions lower than 16.10. Currently the highest base version we have is 16.5.

Changed the title as we currently have multiple base images above 16.10. We need an included image that is based off on of those.


python310Packages.localstack: 3.0.0 -> 3.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.localstack is: A fully functional local Cloud stack
meta.homepage for python310Packages.localstack is: https://github.com/localstack/localstack
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.2 with grep in /nix/store/43vbqfjknkk3dqsshcjr74s5fpi3cwra-python3.10-localstack-3.0.2
found 3.0.2 in filename of file in /nix/store/43vbqfjknkk3dqsshcjr74s5fpi3cwra-python3.10-localstack-3.0.2

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
localstack
python310Packages.localstack
python311Packages.localstack

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/43vbqfjknkk3dqsshcjr74s5fpi3cwra-python3.10-localstack-3.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.localstack https://github.com/r-ryantm/nixpkgs/archive/40733bbe457736612a40107cadb6dae01adb0db4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/40733bbe457736612a40107cadb6dae01adb0db4#python310Packages.localstack

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/43vbqfjknkk3dqsshcjr74s5fpi3cwra-python3.10-localstack-3.0.2
ls -la /nix/store/43vbqfjknkk3dqsshcjr74s5fpi3cwra-python3.10-localstack-3.0.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

localstack (python311Packages.localstack)
python310Packages.localstack

Maintainer pings
cc @jonringer for testing.

Merge conflict, please open a new PR.


[Bug][Connector-Iceberg]fix create iceberg v2 table with pks

Purpose of this pull request

This close #6889
Does this PR introduce any user-facing change?

No
How was this patch tested?

Added additional UT cases
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

@ruanwenjun @hailin0 please take a look

@sunxiaojian  please take a look


Documentation: clarify the time unit for 'step'

The step property accepts minutes but this is not explicitly mentioned. Although the default value of 30 suggests that minutes should be given, this is not immediately obvious.

thanks!


Two-way marshaling between NSData and ArrayBuffer

Introduce TNSDataAdapter to facilitate wrapping ArrayBuffers and ArrayBufferViews in an NSData interface without copying.
Introduce the interop.bufferFromData function which created an ArrayBuffer backed by an NSData instance.

run ci

:+1:

I have a question about the wrapping and unwrapping of buffers:
In the following snippet:
var intRef = new interop.Reference(interop.types.int8, interop.alloc(4));
intRef[0] = 49;
intRef[1] = 50;
intRef[2] = 51;
intRef[3] = 52;
var data = NSData.dataWithBytesLength(intRef, 4);

var buffer = interop.bufferFromData(data);
var wrappedArrayBufferViewData = TNSObjCTypes.alloc().init().methodWithNSData(buffer);

console.log(wrappedArrayBufferViewData === data);

Should the result be true or false?

When you send an ArrayBuffer to something on the native side and we wrap it in TNSDataAdapter and then you get the adapter back, we'll unwrap the ArrayBuffer and hand it back to you, instead of an NSData object that is backed by the buffer. That's what we do for TNSArrayAdapter and TNSDictionaryAdapter as well.
However, when you have a native NSData object and wrap it in an ArrayBuffer with bufferFromData, passing the NSData object back and forth will always return the same object, not the wrapped buffer. In the sample case, wrappedArrayBufferViewData === data.bytes and wrappedArrayBufferViewData === buffer should be true.
Do you think we can improve this behavior in any way?

@fealebenpae do you know if this going to make it in 1.3? of the IOS runtimes?

This should make it in 1.3, yes. I haven't merged it yet because I'm waiting for some outside testing to see whether this actually satisfies a good number of use-cases. Can you share some details about your particular use-case?

@fealebenpae I am finishing up a WebSocket library for Android & IOS, and the IOS side uses a NSData decendant for the Binary side of things.     At the moment I created a "hack" in place that allows me to work around getting a NSData object.       At least on the IOS side once your marshalling makes it in; the hack will auto depreciate as the object will then be marshalled in as a ArrayBuffer rather than NSData.     On the Android side; I'm going to actually have to do a second release once the bug fix is committed as using ArrayBuffer's actually crash the Android runtime...


[feature request] format shell script block using shfmt?

How about wrap shfmt (https://github.com/mvdan/sh) to format shell script blocks. It can avoid trouble that an extra space causes the snakemake to re-run the code.
Snakemake now re-run pipeline by default when code changed. But it does not detect the content of the code. Thus, even an empty space or an indent is added, the pipeline will be re-run. But if shfmt can standardize the code. This problem can be avoided.

Thanks for the request. I don't think formatting people's shell scripts is within the scope of this project. Additionally, this would add an external dependency to snakefmt, which I dont think is necessary.


code XK for Kosovo

Hello.
You forgot XK code for Kosovo.
Bye ;)

This package is based on ISO 3166-1 standard. Kosovo has no ISO 3166-1 assigned code, and XK you're refering to is in the list Since this code is in the list of user-assigned elements


JSXText is missing {extra: {raw}} field

<

only gives
value: '<'

http://astexplorer.net/#/uslr7onHiq/1
but not {extra: {raw}} like we have for "StringLiteral" which is very handy.
I wonder if we could have it added? :)

Actually just looked into it and found https://github.com/babel/babylon/blob/b918554d73ed20c847068243edf99d19ca146dbe/src/plugins/jsx/index.js#L407-L408
https://github.com/babel/babel/issues/2078
node.extra = null;

It is currently there for babel to be able to distinguish if the tokens value is escaped as js-value or xhtml-value.
We also have another issue #271 that is related, so maybe we can find a better way to handle this.

Related, I believe, is that

 translates to a breaking space, which @vjeux points out in this comment on the prettier repo, referencing this flow ast compared with this babylon ast.
In any case, I'm hopeful that the removal of extra that @hzoo mentions above is the real culprit, and worth fixing, as @danez mentioned above. Comparing the original fix to the current code, I'm under the impression that the node.extra = null hack is already unnecessary.
I'll try simply removing said hack and testing locally.

Presumably JSXText should also be changed to read from raw, correct?

K, I think I have this addressed between https://github.com/babel/babylon/pull/344 and https://github.com/babel/babel/pull/5256 , though I might be missing something.


mbuffer: Use objdump from environment when cross-building

Motivation for this change
The mbuffer configure scripts fails to recognize the correct
objdump binary during cross-building for foreign platforms.
The correct objdump is exposed via the environment variable
$OBJDUMP, which should be used in such cases.
Things done

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

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

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

@GrahamcOfBorg build mbuffer
@GrahamcOfBorg build pkgsCross.raspberryPi.mbuffer

I suppose in this case applying this conditionally would be fine in which case that would be preferred. Maybe do include a comment this is especially needed for cross otherwise the next person may remove it again.

Maybe do include a comment this is especially needed for cross otherwise the next person may remove it again.

Done


140/best n sampling

Attempts to deliver work for https://github.com/lvwerra/trl/issues/140
Fixes #140
Gist
Showcase usage of best-of-n sampling

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@younesbelkada  thank you for the review. I took the liberty of shifting the notebook to a directory within examples and have included a README.md within said directory. Whenver you can, please let me know what you think of it

Sorry was a little swamped with travel and stuff last week, but I'm stoked about this @metric-space !

Hey folks, I just realized that the description was edited to say this fixes #140 (implies finality to me) but I think the notebook is a precursor to actual framework code for best-of-n sampling as put here
https://github.com/lvwerra/trl/issues/140#issuecomment-1425431127

One idea would be to wrap that logic in a helper class like BestOfNSampler, but starting with a notebook sounds like a good way to go!

Happy to put in the work to get said additions in. Just waiting on more PR reviews to see whether I'm in the clear

Hey folks @natolambert and @younesbelkada, hate to poke/ping people but also don't want this development to remain dormant and go cold. Any word on this?

Looks good to me. Let's merge the notebook and think about BestOfNSampler in a separate PR?

Thank you folks for this and I appreciate the response. I'll start up a draft PR and conversations on the thread (if needed) around the BestOfNSampler shortly


refactor(parser): name byte handler functions

This PR solves the problem of lexer byte handlers all being called core::ops::function::FnOnce::call_once in the flame graphs on CodSpeed, by defining them as named functions instead of closures.
Pure refactor, no substantive changes.

It works! Lexer flame graphs are now much more revealing. Identifiers take the majority of the time, with strings (QOT handler) the runner-up.


[BUG] inner classes in @Utility classes are static in eclipse, not static in javac.

Describe the bug
inner methods and fields are auto-marked as 'static' in a @Utility class. However, for inner types, evidently in eclipse they are marked as static, but in javac they are not.
To Reproduce
@lombok.Utility public class Test {
    public void example() {
        return new Foo();
     }

    public class Foo {}
}

The above compiles in eclipse, but in javac will fail, erroring out on the new Foo call, saying that this cannot be referenced in a static context.
lombok v1.18.4
Expected behavior
All inner types in @Utility classes should be static, there's no point having them be instance-based.

Cannot reproduce.


metadata.labels lost its flexibility

Andrew's (successful) run with Jacob's code had
  machineInventoryLabels:
    clusterName: test
  machineName: m-${System Information/Manufacturer}-${System Information/Product Name}-${SystemInformation/UUID}

in its machineRegistration
However, with the current elemental-operator, the line above gets rejected with
│ time="2022-07-14T12:55:56Z" level=error msg="error creating machine inventory: MachineInventory.elemental.cattle.io \"m-qemu-standard-pc-q35-ich9-2009-not-specified\" is invalid: [metadata.labels: Invalid value: \"m-${SystemInformation/Manufacturer}-${System Information/Product Name}-${SystemInformation/UUID}\": must be no more than 63 characters

in the elemental-operator log in the management cluster.
Even shortening it to
 machineName: m-${SystemInformation/UUID}

still doesn't work
time="2022-07-14T15:25:10Z" level=error msg="error creating machine inventory: MachineInventory.elemental.cattle.io \"m-qemu-standard-pc-q35-ich9-2009-not-specified\" is invalid: metadata.labels: Invalid value: \"m-${SystemInformation/UUID}\": a valid label must be an empty string or consist of alphanumeric characters, '-', '_' or '.', and must start and end with an alphanumeric character (e.g. 'MyValue',  or 'my_value',  or '12345', regex used for validation is '(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])?')"

63 characters limit is an upstream limitation:
https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#syntax-and-character-set
Labels are key/value pairs. Valid label keys have two segments: an optional prefix and name, separated by a slash (/). The name segment is required and must be 63 characters or less, beginning and ending with an alphanumeric character ([a-z0-9A-Z]) with dashes (-), underscores (_), dots (.), and alphanumerics between. The prefix is optional. If specified, the prefix must be a DNS subdomain: a series of DNS labels separated by dots (.), not longer than 253 characters in total, followed by a slash (/).```

@kkaempf was smbios disabled for this? I dont know about it but seems like that content should be resolved to the actual data, but if smbios is disabled, maybe that data is not available and its not interpreted properly?

apiVersion: elemental.cattle.io/v1beta1
kind: MachineRegistration
metadata:
  name: test-nodes
  namespace: fleet-default
spec:
  install:
    reboot: true
    password: 'password'
    device: /dev/nvme0n1
  machineInventoryLabels:
    clusterName: test
  machineName: m-${System Information/Manufacturer}-${System Information/Product Name}-${System Information/UUID}

This is the file I try to kubectl apply (and fail)

Argh. Not sure where I failed. Tried it again and it works 🤦🏻

Im wondering if there was an error on the yaml and it took the machineName as a machineInventoryLabels instead of its own key. I mean, if you add two spaces to it, it turns into part of the machineInventoryLabels so...

You are not missing anything. In fact it's the other way around. You are passing the machineName as a machineInventoryLabel so it's getting parsed as a label and that doesn't interpolate anything, it stores what you pass literally.

Proper yaml:
spec:
  config:
    elemental:
      install:
        debug: true
        device: /dev/sda
        password: password
        reboot: true
      registration: {}
      system_agent: {}
  machine-name: m-${System Information/Manufacturer}-${System Information/Product
      Name}-${System Information/UUID}
  machineInventoryLabels:
    cluster-name: store-123

Thanks, confirmed working


/api-datapoint fixed

I believe it's fixed

Poke poke we need to test this on Wednesday poke poke

Pull Request Test Coverage Report for Build 33

89 of 93 (95.7%) changed or added relevant lines in 11 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.04%) to 33.221%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gis_csdt/models.py
13
15
86.67%

gis_csdt/serializers.py
20
22
90.91%

Totals

Change from base Build 30:
3.04%

Covered Lines:
591

Relevant Lines:
1779

💛  - Coveralls

Since it's not actually a change for us, I will allow the old versions to use allowany. But in the future, we should change the tests to log in, and then change the restriction to require authentication to reduce DDOS vulnerability.


Church in city

Describe the bug
We "The Gambettis" flew in today to find out the church that we use for ceremonies and business is no longer available. We are curious is to if this was a bug in the most recent update or if it was taken out in general. We use the church for our "Italian Family" promotions and also have hosted weddings in the past for people within the city. We have also used it as a place of refuge for people during purge nights, halloween murder nights etc. We would very much like it if this was back in the city if it was taken out. Thank you

This git is for the Admin Panel.
I think this would fall more under the Assets Issue Tracker.  (Here: https://github.com/InZidiuZ/legacy-assets-issue-tracker/issues)


Example on how to put together a Reader class

We need to show a quick and simple example on how to develop a Reader class to extract data and metadata from proprietary instrument data files.
We already have a tutorial for developing Translator classes in pyUSID documentation. I suggest using the same example, but stopping before the second component - writing to disk. Instead show users how to populate a Dataset object with raw data and metadata.

The tutorial for pyUSID.Translator has been ported to ScopeReaders here.
We need to complete it by showing:

How one would construct the sidpy.Dimension and sidpy.Dataset classes
How one would put all the code together into a sidpy.Reader class for ScopeReaders

This document should highlight important points:

that the self._input_file_path (or whatever I called it) be used for read() and can_read().
it is extremely important that can_read() be implemented (can show an example - it should take just 1 line) in order for this reader to be visible at the top-level ingest() function
by default can_read() only checks the file extension. Far more robust checks are definitely encouraged.
that init could potentially call can_read() since it is responsible for doing the validation of whether or not this file can be read.


Need a release

@sofroniewn @tlambert03
We need to do a release of napari-svg to resolve https://github.com/napari/napari/pull/5582
It seems like the two of you are the maintainers of napari-svg on pypi. Could you give access to someone else on pypi (my handle there is kisharrington)?
Thank you!

@kephale just invited you to be a maintainer - let me know if you need more

@sofroniewn shucks, I don't have proper access rights to this repo on Github. I need to be able to push to cut a new release. Could you please add me?

I have pushed v0.1.7 to not block work until someone solve permissions


BUG: Issue owner not recognized when creator is logged in

Test Platform:
The Issue "The software does not work on mondays" is shown as labelled "by Anonymous" when the user who created the issue is logged in. So he's also not able to edit the issue anymore.

Logged in user with "User role" cannot edit issues even if they created the issue.

I think users should be able to edit their own issues as long as their tags allows this.
So for example if I post an issue and its initial status is NEW then I should be able to edit the issue as long as NEW has a role access limit of USER and no readonly.
Then after moving the status of the issue to OPEN (which has a role access limit of MANAGER and a readonly of USER) I should not be able to edit the issue anymore.
I think that the tag system makes it perfectly possible to decide when a user can edit his/her own ticket and when not.

Please see #39. according to my own notes editing your own ticket must have been already possible then.

I close this issue report single problems in detail when I encounter them.


Replace banner logos for Operaton

Describe the bug
Starting Operaton Run shows this banner:

To Reproduce
Build distro/run
Unpack the resulting tar.gz
Start the application
Expected behavior
The banner is for Operaton
Additional Context
Affects 2 distros. Search for banner.txt:

spring-boot-starter/starter-client/spring-boot/src/main/resources/banner.txt
distro/run/core/src/main/resources/banner.txt

The banner
spring-boot-starter/starter/src/main/resources/banner.txt
already contains an Operaton banner:
     _/_/                                              _/
  _/    _/  _/_/_/      _/_/    _/  _/_/    _/_/_/  _/_/_/_/    _/_/    _/_/_/
 _/    _/  _/    _/  _/_/_/_/  _/_/      _/    _/    _/      _/    _/  _/    _/
_/    _/  _/    _/  _/        _/        _/    _/    _/      _/    _/  _/    _/
 _/_/    _/_/_/      _/_/_/  _/          _/_/_/      _/_/    _/_/    _/    _/
        _/
       _/

@javahippie Which Asciiart Generator & Font did you use for spring-boot-starter/starter/src/main/resources/banner.txt?

I used https://devops.datenkollektiv.de/banner.txt/index.html, but it seems to be down right now

Question: Do we need to add "Run" to the banner, or would we simply add the existing Operaton banner?

Matter of taste. I actually would like to have "Run" in there.
Site is up again. The font that was used lean. Also suitable for Operaton Run is speed or combination of lean with speed or block.
Examples:
     _/_/                                              _/                              _/_/_/
  _/    _/  _/_/_/      _/_/    _/  _/_/    _/_/_/  _/_/_/_/    _/_/    _/_/_/        _/    _/  _/    _/  _/_/_/
 _/    _/  _/    _/  _/_/_/_/  _/_/      _/    _/    _/      _/    _/  _/    _/      _/_/_/    _/    _/  _/    _/
_/    _/  _/    _/  _/        _/        _/    _/    _/      _/    _/  _/    _/      _/    _/  _/    _/  _/    _/
 _/_/    _/_/_/      _/_/_/  _/          _/_/_/      _/_/    _/_/    _/    _/      _/    _/    _/_/_/  _/    _/
        _/
       _/

_______                           _____                   ________
__  __ \_________________________ __  /_____________      ___  __ \___  ________
_  / / /__  __ \  _ \_  ___/  __ `/  __/  __ \_  __ \     __  /_/ /  / / /_  __ \
/ /_/ /__  /_/ /  __/  /   / /_/ // /_ / /_/ /  / / /     _  _, _// /_/ /_  / / /
\____/ _  .___/\___//_/    \__,_/ \__/ \____//_/ /_/      /_/ |_| \__,_/ /_/ /_/
       /_/

     _/_/                                              _/
  _/    _/  _/_/_/      _/_/    _/  _/_/    _/_/_/  _/_/_/_/    _/_/    _/_/_/
 _/    _/  _/    _/  _/_/_/_/  _/_/      _/    _/    _/      _/    _/  _/    _/
_/    _/  _/    _/  _/        _/        _/    _/    _/      _/    _/  _/    _/
 _/_/    _/_/_/      _/_/_/  _/          _/_/_/      _/_/    _/_/    _/    _/
        _/                ________
       _/                 ___  __ \___  ________
                          __  /_/ /  / / /_  __ \
                          _  _, _// /_/ /_  / / /
                          /_/ |_| \__,_/ /_/ /_/

     _/_/                                              _/                         _|_|_|
  _/    _/  _/_/_/      _/_/    _/  _/_/    _/_/_/  _/_/_/_/    _/_/    _/_/_/    _|    _|  _|    _|  _|_|_|
 _/    _/  _/    _/  _/_/_/_/  _/_/      _/    _/    _/      _/    _/  _/    _/   _|_|_|    _|    _|  _|    _|
_/    _/  _/    _/  _/        _/        _/    _/    _/      _/    _/  _/    _/    _|    _|  _|    _|  _|    _|
 _/_/    _/_/_/      _/_/_/  _/          _/_/_/      _/_/    _/_/    _/    _/     _|    _|    _|_|_|  _|    _|
        _/
       _/

Thanks for testing the site again!
My favorite would be to have the same font for both. If the whole "Operaton Run" is to big for the terminal, I would not mind chosing another one for all the distributions

Thanks for the ideas, @kthoms. But "Run" is for me also not so nice. Because it is a part of a former product name. Why not use "Engine". The original name was "Camunda BPMN" and they renamed it to "Camunda Platform Run".
And we have here an Engine ;). But "run" says me nothing in the context of the "process engine".
Is my point of view and for discussion 🙂.

This could be the base for a forum post, because we are now touching pretty fundamental naming discussions

@javahippie this is a good point.

https://forum.operaton.org/t/name-for-the-runnable-distribution/114


[JSON RPC] migrate CLI 'get_events' functionality from gRPC to JSON RPC

Summary
Migrate Libra CLI's get_events functionality to use JSON RPC instead of gRPC. This PR also:

removes proofs from the events and account states returned
removes ascending parameter in get_events
implicitly implement get_events and get_account_state for LibraClient

@bors-libra r=thefallentree

:pushpin: Commit 738ce85 has been approved by thefallentree

:hourglass: Testing commit 738ce85cf85052ea7e53770f66c392f7975af207 with merge bce8f4d54d9f59da42aa1eebdcf47eecc77f9e31...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit 738ce85cf85052ea7e53770f66c392f7975af207 with merge 1d242516657d399b6ce4117428ab6df6b8598279...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: thefallentree
Pushing 1d242516657d399b6ce4117428ab6df6b8598279 to master...


Rust 1.4 build appears broken

It seems that Rust 1.4 can no longer build this project.
src/comment.rs:545:42: 545:71 error: use of unstable library feature 'iter_order': needs review and revision (see issue #27737)
src/comment.rs:545     let res = code_comment_content(orig).ne(code_comment_content(new));

I think the minimum version needs to be bumped to Rust 1.5.0

Where do we specify a minimum version for Rustfmt? Current stable is 1.5, so Travis should only run 1.5, 1.6, and 1.7.

Oh it was in the readme I think. :-)
On Jan 14, 2016 6:31 AM, "Nick Cameron"<EMAIL_ADDRESS>wrote:

Where do we specify a minimum version for Rustfmt? Current stable is 1.5,
so Travis should only run 1.5, 1.6, and 1.7.
—
Reply to this email directly or view it on GitHub
https://github.com/rust-lang-nursery/rustfmt/issues/771#issuecomment-171527363
.

Oh right, found + fixed, thanks!


Generate static page artefacts via API

This PR allows generation of static pages using the API. However, it does not address the recent issue on the use of load_and_annotate_study (except in the static-sitegen/generate_image_page.py script).

Doesn't look like it's going to be a clean merge, should we rebase on the current main in cases like this?

@liviuba definitely rebase on main before PR, ideally every commit if main is moving fast

You can force push your rebased branch here that'll work


Failing spec for prerelease version with a fourth position

Needs an update to the processing in javascript/lib/updater.js.

@hmarr - my JavaScripts are no good, but we just saw a sentry for this case.

@greysteil thanks, all sorted.


bugfix: if used cache, when delete a repository, should clear the cache, otherwise, can not push this repository again.

ref: https://github.com/docker/distribution/issues/3322
Signed-off-by: 刘晓敏<EMAIL_ADDRESS>

@dk-lockdown thanks for the PR. Can you please add DCO signature as per contribution guidelines

@dk-lockdown thanks for the PR. Can you please add DCO signature as per contribution guidelines

@dk-lockdown thanks for the PR. Can you please add DCO signature as per contribution guidelines

signed

@dk-lockdown thanks for the PR. Can you please add DCO signature as per contribution guidelines

signed

@dmcgowan @stevvooe can you help me review this pr? thanks.

@wy65701436 can you help me review this pr? thanks.

Hi @dk-lockdown ,
I have tried with this commit changes, but the deleted repository still can't be pushed successful after GC, because of the blob cache.
This commit just clear the manifest cache after we sent a delete request, and then GC will remove the blobs without clear cache.
So that you can see you pushed the deleted repository successful after GC, but the registry will tell you "Layer already exists",
but it is just the cache, the registry didn't have these blobs.
Thanks.

Hi @dk-lockdown ,
I have tried with this commit changes, but the deleted repository still can't be pushed successful after GC, because of the blob cache.
This commit just clear the manifest cache after we sent a delete request, and then GC will remove the blobs without clear cache.
So that you can see you pushed the deleted repository successful after GC, but the registry will tell you "Layer already exists",
but it is just the cache, the registry didn't have these blobs.
Thanks.

this problem solved，clear the cache when gc.

Hi @dk-lockdown ,
I have tried with this commit changes, but the deleted repository still can't be pushed successful after GC, because of the blob cache.
This commit just clear the manifest cache after we sent a delete request, and then GC will remove the blobs without clear cache.
So that you can see you pushed the deleted repository successful after GC, but the registry will tell you "Layer already exists",
but it is just the cache, the registry didn't have these blobs.
Thanks.

this problem solved，clear the cache when gc.

The trouble is distribution have two caches, one is global cache, one is repository scope cache.
In this clear cache commit https://github.com/distribution/distribution/pull/3323/commits/47f696d7f03aa1b0eec800f5ccd954be6f845664, it only clear the global cache, and the repository scope cache still exists.

Do you want to rebase @dk-lockdown

Do you want to rebase @dk-lockdown

rebased!

@milosgajdos this pr need a review

Ping @dk-lockdown


Add metrics & events

Description
Currently logging is the only source of information made accessible about controller activity, we should also publish Kubernetes events and metrics.
Events should be used to surface significant points of interest for operators deploying and manage resources with the controllers.
Metrics should be used to monitor performance.

https://github.com/open-component-model/ocm-controller/pull/334 for ocm-controller is finished


Add base experiments for common tasks

i.e., create sample experiments for classification, segmentation, etc. (low priority and just a maybe)

Add this to 'experiment.py'

Also make 'datasets.py' which defines common datasets to use such as XYDataset, Episodic classification dataset, classificationdatasetfromfolder, episodic xy dataset, meta batch sampler, etc.
Also make 'algorithms.py' which implements new algorithms such as MAML, protoner, DDPM, etc.


child collider delegation of circular drive script is broken

the circular drive script has a child collider field, which supposedly allows the collider of the specified child to be used. it says this needs to be specified in case there two or more children, which makes sense. unfortunately this field does absolutely nothing. regardless of which child is selected here, all colliders of each of and every one of the children will be used by the circular drive.

I noticed this the other day too. Looking at CircularDrive.cs, the public property childCollider is indeed being assigned but never used anywhere as far as I could tell. What I found that helped my case was that the attached Interactable (which is required by CircularDrive), has a Hide Highlights property that is an array of gameobjects to NOT show highlights for.
In my case, I had a classic ship's wheel as a parent GameObject with child objects of: an axle; 6 separate arms; a center hub; and an outer ring. On my interactable on the parent wheel object, I set the axle, center hub, and outer ring to be hidden. That way, only the arms appear interactable and can be spun.

I noticed this the other day too. Looking at CircularDrive.cs, the public property childCollider is indeed being assigned but never used anywhere as far as I could tell. What I found that helped my case was that the attached Interactable (which is required by CircularDrive), has a Hide Highlights property that is an array of gameobjects to NOT show highlights for.
In my case, I had a classic ship's wheel as a parent GameObject with child objects of: an axle; 6 separate arms; a center hub; and an outer ring. On my interactable on the parent wheel object, I set the axle, center hub, and outer ring to be hidden. That way, only the arms appear interactable and can be spun.


update tictactoe game

stop importing angular2/debug
replace _BINDINGS with _PROVIDERS
replace deprecated bind() with provide()
remove unnecessary @View

thanks! I'll add you to the repo too


Floating player position-listener fixes

Issues

Closes #251
Resizing the browser on homepage, even when FP is not there, was laggy.
The floating player is also supposed to track the right edge when the browser is resized (albeit not very accurately).  That behavior was lost.

Recent changes and/or refactoring combined the effects or added dependancies into the effects, causing them to re-run excessively.
Changes

Restored effects to their original behavior.
Don't perform the boundary checks when dragging -- only do it when released.
Do proper debouncing for the 'resize' listener -- the previous method was incorrect as a new function is created on each render.

It's still pretty sluggish when vjs goes crazy.  Anthony will look at it.
Floating-player high CPU usage on Chrome #158


Adding support for projects using local plugins.

Using local plugins changes the structure of the plugin object.
Adding support for this "enhanced" plugin object.
Example below:
plugins:
  localPath: './.serverless-plugins'
  modules:
    - slow-down
    - serverless-prune-plugin
    - serverless-s3-local
    - serverless-sentry
    - serverless-plugin-split-stacks
    - serverless-plugin-existing-s3
    - serverless-offline
    - serverless-webpack

https://serverless.com/framework/docs/providers/aws/guide/plugins/

Hi @SeverinMahoney-Marsh
Thanks for your great PR !! The aim of your PR is good for this project. But I have a minor question. Please check my reviewing.
@n1ru4l
Thanks for your reviewing. This is the my first time other developers reviewed the PR to my project ! It's exciting !!

@ar90n sorry about that, you're absolutely right. That was a careless copy-paste on my behalf.
Thanks for keeping the project maintained, it's been very helpful.

Hi @SeverinMShift
I merged this PR and published new version. Please check it !!

Hi @ar90n,
Thanks, that's solved the issue.


bloodhound py puts "." and ".localdomain" at the end of domain controller ip instead of using the domain name i specified

./bloodhound.py -c all -d htb.local -dc <IP_ADDRESS> -u #### -p #####
WARNING: Could not find a global catalog server. Please specify one with -gc
INFO: Connecting to LDAP server: <IP_ADDRESS>
Traceback (most recent call last):
File "./bloodhound.py", line 5, in
bloodhound.main()
File "/root/htb/forest/BloodHound.py/bloodhound/init.py", line 265, in main
disable_pooling=args.disable_pooling)
File "/root/htb/forest/BloodHound.py/bloodhound/init.py", line 68, in run
self.pdc.prefetch_info('objectprops' in collect, 'acl' in collect)
File "/root/htb/forest/BloodHound.py/bloodhound/ad/domain.py", line 329, in prefetch_info
self.get_domains(acl=acls)
File "/root/htb/forest/BloodHound.py/bloodhound/ad/domain.py", line 203, in get_domains
for entry in entries:
File "/root/htb/forest/BloodHound.py/bloodhound/ad/domain.py", line 105, in search
self.ldap_connect()
File "/root/htb/forest/BloodHound.py/bloodhound/ad/domain.py", line 60, in ldap_connect
q = self.ad.dnsresolver.query(self.hostname, tcp=self.ad.dns_tcp)
File "/usr/lib/python2.7/dist-packages/dns/resolver.py", line 1002, in query
raise NXDOMAIN(qnames=qnames_to_try, responses=nxdomain_responses)
dns.resolver.NXDOMAIN: None of DNS query names exist: <IP_ADDRESS>., <IP_ADDRESS>.localdomain.
if i put a / at the end of the domain name it does consider but then it just times out after 3 seconds. all the prerequisites are fully updated and this persists on both python 2 and 3

hey, the DC argument needs a hostname, not an IP. If this hostname does not resolve with your current DNS then you need to specify the DNS server as well (the DNS server should be an IP)


Feature Request - Season Folders option when adding TV Shows

By default, when shows are added to sonarr, they're not in season folders

ok this shouldn't be too hard I'll get it in the next update

fix is in dev

you can add the season_folder attribute to sonarr which will default to true


Default mysql conf should be /etc/mysql/my.cnf

I think in all installation /etc/mysql/my.cnf is present while /etc/my.cnf is not

Where you found this path?
Is it hard coded in code? Could you please clarify a bit?

oops, it's in /etc/bck.conf I think (I copied it from the docs, I couldn't find it nowhere in the repo)
mycnf=/etc/my.cnf

This thing is related to MySQL itself.
So it needs to be edited after installation to reflect your path.

... yes but well i made a very small PR to illustrate what I mean, on some installation we don't have /etc/my.cnf so why using this one?? I bet every distribution has /etc/mysql/my.cnf


Improvement : receive sharing intents

receive sharing intents from other apps and populate the text field automatically.

I went to wikipedia highlighted a noun and try to share it to the app it both copies the text as well as the url. I also tried sharing a text message to the chat app and it works as intended. Unless it is just quirks as android it partially works as intended.

Ok this seems to work and the behavior is similar to other system apps. This is the Android part to #81


validate onboarding ticket signature.

This PR validates the signature of the onboarding ticket
used by the consumer cluster to onboard a storage provider cluster
Note: This PR assumes that the public key secret to be created by the deployer will be called onboarding-ticket-key
Signed-off-by: Santosh Pillai<EMAIL_ADDRESS>

@kesavanvt Can you please review?

/cherry-pick release-4.10


Allow for globs in probe.resolvers.intellij.repositories config

Currently, in git-machete-intellij-plugin -> ideprobe.conf we re-use the IntelliJs downloaded by runPluginVerifier Gradle task... but there's still no easy way to re-use IntelliJs downloaded by Gradle, which live under paths like:
$ ls -d ~/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/*/*/ideaIC-*/
/home/plipski/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/2022.2/42c296374014a649785bb84aa6d8dc2d18f2ca0e/ideaIC-2022.2/
/home/plipski/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/222.3345.16-EAP-SNAPSHOT/4a1d1d2eb2524a87cc7fd9bd7a2aed2c7efa6593/ideaIC-222.3345.16-EAP-SNAPSHOT/
/home/plipski/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/222.3345.47-EAP-SNAPSHOT/bf10a6ac5e737b257349ade298202da993b357ca/ideaIC-222.3345.47-EAP-SNAPSHOT/
/home/plipski/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/222.3345.90-EAP-SNAPSHOT/1edc98c7aa0e16fa9f95e2cd396a556b1a03aaf6/ideaIC-222.3345.90-EAP-SNAPSHOT/
/home/plipski/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/222.3739.24-EAP-SNAPSHOT/fd84e4e4a7796908228a8f72c6b156de1c5c916d/ideaIC-222.3739.24-EAP-SNAPSHOT/

Due to the hash in path, we can't easily define a single expression with [revision] to put into probe.resolvers.intellij.repositories.
Let's:

check if the Gradle-downloaded IntelliJ can indeed be used by ide-probe: just set probe.resolvers.intellij.repositories to a single element, a specific path from Gradle caches, for a specific version (the hash included)
if so, implement support for globs so that a path template like

"file:///"${HOME}"/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/[revision]/*/ideaIC-[revision]/"

works for any version (snapshot or release)

update the docs accordingly

Also, update git-machete-intellij-plugin -> ideprobe.conf accordingly once done

TBD what to do if multiple files match a glob: whether to abort&fail, or just pick any matching? or maybe make the behavior configurable?

TBD what to do if multiple paths match a glob: whether to abort&fail, or just pick any matching? or maybe make the behavior configurable?

Did you encounter such case for these IntelliJs downloaded by Gradle? From quick checks on my machine it looks like I don't have such case. 🤔 It would not make sense to me if this Gradle's tool that downloads IntelliJs would download the same thing twice and save it in two directories.

Did you encounter such case for these IntelliJs downloaded by Gradle?

Hmmm pls note that this globs can have any form theoretically, like: file:///${HOME}/*/*/*... it's not strictly related to Gradle in any way (ide-probe doesn't know anything about Gradle, after all). It's more of a question of what to do in case of multiple matches on IDE path when globs are used 🤔


feat(app): show offset deletion warning if previous offsets exist

close #8841

Overview

Show alert warning on LPC intro screen indicating previous offsets will be deleted only if previous offsets exist
Changelog
Put alert behind check for existing labware offsets

Review requests

Confirm alert is present when previous offsets exist
Confirm alert is not present when no offsets exist
Risk assessment

Low

I was able to get through the flow a few times and the modal works as expected when 1) offsets are added and 2) offsets are not added. However, if you go through the flow once and change 1 thing, say X = 0.1, then if you go through the flow again and make X = 0, all the offsets are technically 0 but the banner will still show up if you do LPC again. Not sure if this is the intended behavior or not?

This is because we just check whether the current run has offsets on it, so even if you explicitly set the offset to 0,  an offset will technically exist. @shlokamin do you think I should modify the getLatestLabwareOffsetCount function to only count offsets if they aren't 0?

I was able to get through the flow a few times and the modal works as expected when 1) offsets are added and 2) offsets are not added. However, if you go through the flow once and change 1 thing, say X = 0.1, then if you go through the flow again and make X = 0, all the offsets are technically 0 but the banner will still show up if you do LPC again. Not sure if this is the intended behavior or not?

This should be fixed with the most recent commit! cc @shlokamin @jerader


Replace window.open with tabs API to allow file:// scheme from croutonurlhandler

Implements the simpler part of #1860.
(CLA signed.)

Thanks!


feat(templates): support optional .hbs extensions in templates.

Thanks for this project!
I thought it'd be easier to work with the templates if they had the correct extensions (for syntax highlighting, making prettier not wrongly format them on save etc. etc.).
This PR:

adds .hbs extensions to template files in /templates (where appropriate)
rm's .hbs extension from names when writing generated files (if present)
updates register-partial to rm .hbs (if present) and any preceding extension (if present). So .partials/info.md.hbs is still used like {{> info }}

Thanks for getting back to me @fmvilas.
No worries if you don't want to merge this, I can see its maybe more of a personal preference thing than an improvement over what's already in place. That's why I tried to implement it in such a way that it was optional.
I've updated the PR with a fix to only replace hbs from the end of filenames, and updated the readme to reflect the changes in this PR.
To each of your points specifically:

What happens if you really want to have a Handlebars file in your template? Say, for instance, a template wants to provide views with Handlebars and it has files named like user-profile.hbs. As it is right now, they will also get affected while they shouldn't.

As is, this would remove the extension yes. You could name them .hbs.hbs if you needed to persist the extension (I've added this to the README) but I see that in this case it's not really as optional as I'd wanted.
Also I don't know how common a usecase this is? it's very meta. I was tempted to put a yo dawg in the readme 😆.
An alternative approach here might be to just not rm .hbs if it's the only extension. This would mean you can't generate files with no extension though - I don't know if that would ever be a problem. We could make sure it still worked for dotfiles. If you'd prefer that approach, I can update the PR. Let me know.

This is replacing any existence of .hbs in a file name. Should it use a regex like /.hbs$/ to make sure it only removes occurrences at the end of the file name?

Yup, thanks. Fixed.

If the extension is .hbs, your editor will highlight Handlebars syntax but will not highlight the "final" syntax of the file, e.g., "js", "markdown", "html", etc. Are you translating a problem of the editor to the code?

In other templating languages I've used, I'd expect the file to have the extension of the templating language, prefixed by the extension for the outputted file. This is as much for developer ergonomics as IDEs. In my opinion - I'd prefer to look at a filename and know what kind of file it was.
IDK that IDE's can correctly highlight js  in a '.js' file if its interspersed with handlebars syntax anyway? Vscode was certainly freaking out a bit for me (more around linting that syntax highlighting IIRC). I'd think IDE's have a better chance of perfectly highlighting a .js.hbs than a .js w/ hbs inside, even if today that means hbs higlighting and no js highlighting.
Appreciate this is all maybe a bit subjective, and happy for you to do whatever you think's best with this PR.
Thanks again for the project, and your time.

Awesome! Thanks for the explanation!
Honestly, I think this is a problem editors have to solve. Think, for instance, how every editor supports HTML+CSS+JS even when the extension is .html. That said, given the reality is different and you are the second person who suggest this change already, I think this PR does have value and it's worth merging.
Thanks for taking the time to contribute and explain it so well!

It's published on npm, version 0.11.0. Thanks, @peterlenagh!


Redundant upper-case 'Web'

Hi @weitzhandler, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;


最新版1.4.1版本出现Saving time is too long. (> 500 ms)

不知道是什么原因，刚更新最新版本运行项目就出现这个提示

+1

我顺便补充一下自己所使用的环境
Node.js v16.19.1
VScode v1.77.3
Vue Language Features (Volar) v1.4.1
TypeScript Vue Plugin (Volar) v1.4.1
项目所使用的TypeScript版本 v5.0.4
vite v4.2.1
vue v3.2.47

it's the new feat add in v1.4.0 for long time operation.
you can waiting for the action finish or disable it right now.

我把版本退回到1.2.0了，1.4版本全局报红

问题很大  大家忍一下

是的。我也出现了这样的错误。在代码保存时，出现该错误。

This is a feature to avoid codeActions causing long save times when used with other extensions. v1.4.2 increases the preset time to 1000ms, you can also change it to a larger value.

Still got this issue.

It happens when batch replacing text in multiple files.

It happens when batch replacing text in multiple files.

Change the value to something larger than 2665ms.Although I feel like this is not the way to go on.

Every time I save, the time increases, but the file is correctly saved immediately. It looks like an accumulator in the background is not cleaning the time of the files saved previously.
In this example, it has increased from 70000 to 400000, and now 500000.

@andreshg112 I have no idea why, but https://github.com/vuejs/language-tools/commit/3ace934bc61711e4e5fd5e095766d3bb2c4b69ad should avoid the problem.

+1，看来不是我的问题，突然报错，难受

@johnsoncodehk Hi Johnson, can you add this to the document or pin a new issue about the saving time? This problem is caused by other conflict extensions and it will be good to let users know.

@so1ve added in https://github.com/vuejs/language-tools/commit/3baacde7e7c093003fca3bc159bea631a1e3a857, thanks.

vscode Saving time is too long. (1734 ms > 1000 ms),


Swift 5 issue

As I am using Xcode 10.2, I want to upgrade this plugin to 3.0.0.
At time installing this plugin through cordova it is taking 2.6.0 version
When this plugin be available in 3.0.0

The same here
XCODE 10.1
Plugins/cordova-plugin-qrscanner/QRScanner.swift:38:42: Type 'AVLayerVideoGravity' (aka 'NSString') has no member 'resizeAspectFill'
`Ionic:
ionic (Ionic CLI)  : 4.12.0 (/usr/local/lib/node_modules/ionic)
Ionic Framework    : ionic-angular 3.9.2
@ionic/app-scripts : 3.2.3
Cordova:
cordova (Cordova CLI) : 6.5.0
Cordova Platforms     : android 7.1.4, ios 4.3.1
Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.2.0, (and 13 other plugins)
System:
ios-deploy : 1.9.4
NodeJS     : v10.15.3 (/usr/local/bin/node)
npm        : 6.4.1
OS         : macOS Mojave
Xcode      : Xcode 10.1 Build version 10B61`

Version 3.0.1 is available right now. https://www.npmjs.com/package/cordova-plugin-qrscanner
Thanks.

Well, no luck with PGB (PhoneGap Build) for iOS. I tried everything for hours - or days I should say. I don't even know if PGB is ready for Swift5 anyway. But thank you all for getting that plugin a boost!

tested v3.0.1 working with ionic v4.0.0 release.


Timeout for larger scan

Hi, I was trying to run a scan on a larger (combined) specification file but I constantly get into the following error.
Is there a way for me to configure the amount of retries for the report?
Thank you in advance. I redacted some parts in the log
Ignoring network errors
Reading config file '42c-conf.yaml' in '/github/workspace'
Matched branch name
GET https://platform.us.42crunch.cloud/api/v2/sqgs/audit 200
GET https://platform.us.42crunch.cloud/api/v1/organizations/me/settings/apiNamingConvention 200
GET https://platform.us.42crunch.cloud/api/v1/organizations/me/settings/collectionNamingConvention 200
Looking for OpenAPI files in:
Looking for OpenAPI contents in:
Discovered OpenAPI files:
Looking for the technical collection name:
POST https://platform.us.42crunch.cloud/api/v1/collections/technicalName 200
GET https://platform.us.42crunch.cloud/api/v1/collections/1dd4d329-b812-488f-86e9-3819088621ab/apis 200
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
PUT https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Reading assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
GET https://platform.us.42crunch.cloud/api/v1/apis/76c3e932-ac4f-430c-9be4-6a2c1164c074 200
Assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074 is not ready, retrying.
Error: Error: Unexpected exception "Timed out while waiting for the assessment report for API ID: 76c3e932-ac4f-430c-9be4-6a2c1164c074"

Hi,
Since the question is related to the use of our Enterprise platform, we have created a support ticket and sent you an email via our Support team.
Please check your inbox, and in case you haven't received that email, please let us know.
Thanks.

This issue has been fixed in v3.7.0


[stable/rocketchat] update to 1.3.1, upgrade mongo to latest helm chart

Signed-off-by: Bart Verwilst<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

What actual versions of mongo is this moving between?

Both are 4.0.9, so no changes there.

/lgtm


How to train the MO3TRTemp andBoxTransformer Module?

hello, I am interesting in your paper especially the MO3TRTemp andBoxTransformer Module. I read your code but I have some questions as follows:

How to train the MO3TRTemp andBoxTransformer Module? End to End or separate with the whole model?
How to set the loss function for these two module?
Hope to receive your reply. Thanks!

Thanks for your prompt reply! I do wonder which way to choose to train the BoxTransformer, now I see.
And could you tell me how long have you train the model? Because I rebuild these modules by my way and try to train them.


JsonPath filter is not evaluating variable

Hi all,
I am trying to verify that result of previous HTTP Methods is correct. I am doing that by performing new HTTP Method which result I have to filter.
Simplified code looks like this:
  * def myId = 'abcd1234'

	Given path 'pullmyobject'
	When method get
	Then status 200
		And def result =<EMAIL_ADDRESS>== '#(myId)')]
                * print 'RESULT IS: ' + result
		And assert result.size() == 1

Basically I am getting array of objects and i have to verify that only one object has this id. But JsonPath filter is not evaluating variable so I am getting following error:
15:19:21.972 [main] INFO  com.intuit.karate - [print] RESULT IS: []
15:19:21.981 [main] ERROR com.intuit.karate - assertion failed: assert evaluated to false: result.size() == 1

Can anyone please help?

@koprivica The issue looks to be related as this one https://github.com/intuit/karate/issues/217.
Please take a look.

yes closing as duplicate of #217

Thanks guys! This is what I was looking for.
Sorry for inconvenience.


fix: use hygieneInfoKind

This was not supposed to be committed, it was from a test.

bors merge


fix(explore): redandant force param

SUMMARY
The chartDataUrl contains an empty force parameter due to null value from getUrlParam(URL_PARAMS.force).
This commit cleans out the redundant force parameter for the falsy value case.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS
Go to any dashboard and check the url parameter for /api/v1/chart/data
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks for the cleanup @justinpark, removing the bare force parameter helps greatly with the readability as previously it was akin to saying force=false which was somewhat misleading.


bug: logical errors in the filtering algorithm

I've been getting some incorrect results with filter files. Upon studying the code, I found that there are two key parts of the code which are flawed. I have generated my filter files according the the documentation and the answer here: #420. I have created a fix and will be submitting the pull request (edit: submitted, #445), but here I will document the issues I've been seeing.
The first issue originates here: https://github.com/ckotzbauer/vulnerability-operator/blob/5e293175d9afc0461a068277f86587512390eb26/internal/vuln/filter/filter.go#L38
The error here is that there may be a filter in the array whose context array matches one container, and then another filter later in the array whose context array matches another container. However, the algorithm will shortcut out once it hits ANY filter which matches ANY container.
Secondly, there is a flaw here: https://github.com/ckotzbauer/vulnerability-operator/blob/5e293175d9afc0461a068277f86587512390eb26/internal/vuln/filter/filter.go#L51
The code here is basically saying "for each context, add to the applied list any containers which match. add to the not applied list any containers which don't match". For certain filter files, you end up adding containers to both the applied and not applied lists. The correct logic should be "for each container, if it matches any context, add it to the applied list. if it matches no context, add it to the not applied list.
I wrote test cases to illustrate these issues here: https://github.com/samcornwell/vulnerability-operator/commit/d13d251af43bfbe75245ea7d20cf3ac973fbe2d1. The current release of the vulnerability-operator fails these tests.


Searching DB coupon page doesn't reveal "duplicate" coupon uploads with different bar codes

Some coupons have a "duplicate" upload in the DB: same item, but a different bar code. On the http://www.hfqpdb.com/browse page, it will only list the first uploaded coupon version. If a DB coupon has a different bar code from the one on https://www.harborfreight.com/coupons, the script will say the coupon is missing from the DB (potentially correct behavior). However, the coupon could have already been uploaded, and by clicking the individual DB coupon page, it is revealed the two coupons with different bar codes are both present (false positive behavior).
The script needs to be altered to go into each active coupon page and compare the active coupon(s) to the HF coupon. This will prevent false positives that say the coupon is missing from the DB.

This issue was worked around with e2df55d24b77defa5535601a9af81b55bf4fdb59 and b961bb353292214244ccc47314d54e084c5c6e35.


[bazel] rollback #15119

Rolling back #15119, as this did not fix bazel airgapped builds.
Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>

Merging as this is blocking nightlies from running.


(FYI) Demo site is down while I migrate providers

In the middle of migrating providers and hit an unexpected snag halfway through everything - on it!

Any plans to resurrect this project?


Slow updates when showing directory or application

Updates are slow when using either the application or directory module.
So, if I change my directory or if I launch a new process, these changes will get reflected in tmux, however it can take up to 10 seconds for the changes to appear.
Is there a setting I can set to make this more instantaneous?

You could try to set the interval for tmux update to 1 second:
set -g status-interval 1

The default is 15 seconds.

Closing this one as there had been more than 2 weeks without reply from the author.


Update sqlserver.bicep to use the cheapest SKU by default

When deploying todo-csharp-sql-swa-func using azd up, SQL server instance is on general pricing at £200 a month. This should align with the other free/cheap SKUs used for other resources at £3.50 a month)

This is important to save costs during development.
In addition, is there a way to set the SKU variables by environment?
For example:
// dev
"sku": {
	"name": "Basic",
	"tier": "Basic",
	"capacity": 5
}

// prod
"sku": {
	"name": "GP_Gen5",
	"tier": "GeneralPurpose",
	"family": "Gen5",
	"capacity": 2
}

You should be able do using the template parameters for that but I am quite surprised the default is not set to the cheapest option... Especially since Static app is set to free.  That is quite a steep price and does not align with the rest. I hope this gets reviewed and approved soon so others don't get stung without noticing.

You should be able do using the template parameters for that but I am quite surprised the default is not set to the cheapest option... Especially since Static app is set to free. That is quite a steep price and does not align with the rest. I hope this gets reviewed and approved soon so others don't get stung without noticing.

I created PR #10 which allows that.


[doc] add background info and links for BrainVision file format

With BrainVision being one of the recommended formats in EEG-BIDS, I wanted to update the information on the file format here in the MNE-Python documentation.
It also links to an MNE-BIDS example, which is soon to be merged: https://github.com/mne-tools/mne-bids/pull/104/

thx @sappelhoff
we have issues with CIs now but we'll merge when fixed.

Now that the CIs are fixed if you rebase you should get the docs to render so you can check the remote output, too.
Also it would be good because things will turn green

thx a lot @sappelhoff


CADL API review - API review is not generated when emitter has warnings

API review file is not generated in output path when emitter throws warnings. This issue only happens on CI machine.  By default warnings are not treated as errors but CI considers warnings as errors.

@praveenkuttappan and I met and looked into this. The PR-based pipeline for generating token files works even if there are warnings (which is what happens if you run apiview locally too). It's only the manual, url-based pipeline that fails to output if cadl outputs warnings, which means the issue lies in that pipeline.
Notably, the manual pipeline uses Powershell while the PR-pipeline uses CMD, so that might be a source of the discrepancy.

Fix was included in following PR to find the token file recursively from output path. https://github.com/Azure/azure-sdk-tools/pull/5196


EXO-36 ⁃ [exo] Reformat List Page

┆Issue is synchronized with this Jira Task by Unito

Refactor to include Star color and size in star description


Project Review

portfolio project: a project built with HTML and CSS

it is a mobile version of the portfolio
it uses HTML semantics tags and CSS layouts.

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

Highlights

All linter checks passed ✔
Descriptive pull request ✔
Used GitHub flow ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Project Status: APPROVED :green_circle:
Hi @clin2on3mun,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

Inline comments added :100:

Optional suggestions

Great job adding inline comments as mentioned by the previous reviewer :+1:. I will suggest that you include emojis in your reviews so it sends a friendly tone to the recipient. Also, I will suggest that you make sure your sentences are well phrased so the student understands what is expected of them and is able to deliver :100:

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[✨] Allow Signal<string | undefined | null> for bind:value on inputs

Is your feature request related to a problem?
I'm working with Supabase and when you query for data on a string field that is allowed to be null, the generated types from Supabase set the field type to string | null.  I can't set the field directly to bind:value because it is always expecting Signal<string | undefined> so I have to keep converting them back and forth.
Describe the solution you'd like
Set the type for bind:value to Signal<string | undefined | null>
Describe alternatives you've considered
My only other alternative without having to have the conversion code is to not use bind:value and keep the signal up-to-date using an onInput$ callback.
Additional context
No response

  const opacity = useSignal(0.5);
  return (
    <div>
      Opacity:{" "}
      <input
        type="range"
        min="0"
        max="1"
        step="0.05"
        value={opacity.value}
        onInput$={(e) =>
          (opacity.value = (e.target as HTMLInputElement).valueAsNumber)
        }
      />{" "}
      <input type="range" min="0" max="1" step="0.05" bind:value={opacity} />{" "}
        //                                            ^^^^^^^^^^
       // Type 'Signal<number>' is not assignable to type 'Signal<string | undefined>'.
       //   Types of property 'value' are incompatible.
      //     Type 'number' is not assignable to type 'string'.ts(2322)
      {opacity.value}
      <Child opacity={opacity.value} />
    </div>
  );

This is because bind:value is a macro expands to
      <input
        value={opacity.value}
        onInput$={(e, t) => (opacity.value = t.value)}
      //                     ^^^^^^^^^^^^^
      // Type 'string' is not assignable to type 'number'.ts
      />

As such the macro has no understanding of types. It just assumes .value, which is a string.
I think this is a Won't Fix as there is no easy way for the bind:value macro to know what the right thing to do is.

I am going to close this as won't fix, to keep the repo clean, but feel free respond and or discuss this here..


Quick Start update for permission denied error

What would you like to be added:
Update Quick Start to account for any permission denied error message (need to change permission to 0774)
Why is this needed:
When manually installing a provider the end user can potentially run into this error message
Error: Error asking for user input: 1 error(s) occurred:

* provider.rubrik: fork/exec /Users/andrewdraper/.terraform.d/plugins/terraform-provider-rubrik: permission denied

Fixed in PR31 - https://github.com/rubrikinc/rubrik-provider-for-terraform/pull/31


Update DevHub with the Service Definition/Overview

Describe the issue
There are two documents, Service Overview from the Business Model Canvass and the Service Definition from the funding request effort.  We want to take the best info from both and place as a tile on DevHub.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context
Add any other context, attachments or screenshots
Definition of done

Take content from #943 and convert to markdown in our repo
Tile is visible in DevHub, following the same format as the Cloud Pathfinder one (or update that one if that's better - you decide).
Service Definition tab in login page has a filtered view on all CPF related tiles on DevHub.

This content will be developed as part of the new Public Cloud website.


Fix #4645, Add an extension to _mocha for people with loaders

Description of the Change
Alternate Designs

Moving mocha to mocha.js: Docs hard reference mocha without extension.
Making a copy of _mocha to mocha.cjs: This is uncommon. Though it's strongly recommended by Node and others when you transpile to CJS from a native ES Module. https://redfin.engineering/node-modules-at-war-why-commonjs-and-es-modules-cant-get-along-9617135eeca1
Loading mocha with a PassThrough stream in mocha.js: That a little exotic and depends on stdio
We could make a mjs wrapper, but that seems unnecessary right now... it is a workaround however

// mocha-4645.mjs
import cli from './node_modules/mocha/lib/cli/index.js'
cli.main()

Why should this be in core?
It cleanly allows for TS and ESM support, using loaders. A recurring long request.
Benefits
ts-node/esm and other node loaders will work as expected. As well as supporting custom loaders,  custom loaders for logging, performance, etc.
Possible Drawbacks
If anybody directly references _mocha, without the alias, they would have bad day.
Applicable issues
#4645

Coverage remained the same at 94.344% when pulling 62d870875faa0593ec8bd931f218e5f9b0747188 on rayfoss:node-loader-bugfix into a93d759c4c4cfd6da2656be5cbcb6fb9a125a4a1 on mochajs:master.


Scan for CSS classes in workplace freezes at 99.xx%

Reported this here: https://github.com/zignd/HTML-CSS-Class-Completion/issues/154
This is still happening

Idea: Maybe this is a caching problem because some are still available?

Idea 2: Could it help to ignore node_modules?
Log when it froze:
console.ts:134 [Extension Host] Looking for parseable documents...
console.ts:134 [Extension Host] Found all parseable documents.
console.ts:134 [Extension Host] Parsing documents and looking for CSS class definitions...

After window reload (solves it 50% of time):
console.ts:134 [Extension Host] Looking for parseable documents...
console.ts:134 [Extension Host] Found all parseable documents.
console.ts:134 [Extension Host] Parsing documents and looking for CSS class definitions...
console.ts:134 [Extension Host] Summary:
console.ts:134 [Extension Host]% 2641 parseable documents found
console.ts:134 [Extension Host]% 179736 CSS class definitions found
console.ts:134 [Extension Host]% 4538 unique CSS class definitions found
console.ts:134 [Extension Host]% 3 failed attempts to parse. List of the documents:
console.ts:134 [Extension Host]
project/app/node_modules/angular-ui-bootstrap/src/tooltip/docs/demo.html
project/app/bower_components/angulartics/samples/inspectlet.html
project/app/node_modules/angulartics/samples/inspectlet.html

Its more stable than the old plugin but not rocket super duper stable :smile:
More info:
HTML-CSS-Class-Completion still installed? Nope, but switched
Project has 100+ SCSS files

I reproduced this behavior on one of my workspaces, which is very large. I will try to implement a workaround. But i have to make sure this is not related to VSCode 1.31.1.

I'm currently working on vscode 1.33.1

Did this also happens on previous VSCode versions or only introduced in 1.33.1?

Yes this also happend on previous version.

Do you set "html-css-class-completion.excludeGlobPattern" to "" in your settings.json?
In my large project,  I set this option to blank. This causes stuck at 99%. This mean it process node_modules/. When I remove that key (which is default point to node modules) it worked.

I have added node settings to exclude note strictly. I will test and publish soon.

I've changed CSS/SCSS Pre-processor usage. Can you try with latest version? Also node_modules is in exclude path.

Great to hear! I've tested it a few hours (version 1.5.17) and I think it's gotten better, but it's still gets stuck a some times. I'd say it works 60% of the time.

I had another idea: How about ignoring all .gitignore(ed) files? When you build a project folders like public, dist or temp beeing generated with css files inside, but they are only a compiled version of the classes that exist in the source and this could save some time.
I don't know if you could make it work with .gitignore paths because of the !keep/folder/, example.file and node_modules/ symantic and also the relative paths. But how about an array of paths in the vscode/settings.json that you can define globally and per project?

We may support .gitignore as a secondary source of exclude setting. First we look for include and exclude via settings.json then add more ignores based on .gitignore.
BTW, have you ever tried excludeGlobPattern setting that mentioned in README?

I didn't know there where so many usefull settings for your plugin. It works by removing .css files from include glob pattern.
:+1: Thank you for your awesome maintenance of this plugin. :tada:

I'm very happy to hear that. 🎉 Thanks! If you encounter any issue, do not hesitate to open a new issue.


[ADAP-1042] Persist docs fails when a column name matches a Snowflake no-argument function

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I have a model with column name current_time and when I set persist docs to enabled for columns, my dbt run fails. The query that adds the comment to the table is failing with error message SQL execution internal error:  Processing aborted due to error 300002:3929359486.
After investigating this further it seems any columns that share the name of a Snowflake no-argument function (ones that do not require brackets like CURRENT_TIME, CURRENT_DATE) will fail with this error.
This is the query dbt attempts to run but fails:
alter table db.schema.sf_test alter

        "CURRENT_TIME" COMMENT $$test1$$

        "CURRENT_DATE" COMMENT $$test2$$

        "CURRENT_TIMESTAMP" COMMENT $$test3$$

        "LOCALTIME" COMMENT $$test4$$;

Expected Behavior
I would expect no error to occur for these column names when persist docs is enabled.
Steps To Reproduce

Create a new dbt project
Add the following to the dbt_project.yml:

models:
  +persist_docs:
    relation: true
    columns: true

Create a new model file (models/test_table.sql) with the following content:

{{
  config(
    materialized = 'table',
    )
}}

select
  1234 as current_time,
  1234 as current_date,
  1234 as current_timestamp,
  1234 as localtime

Create a new yaml file models/models.yml with the following content:

version: 2

models:
  - name: test_table
    columns:
      - name: current_time
        description: "test1"
      - name: current_date
        description: "test2"
      - name: current_timestamp
        description: "test3"
      - name: localtime
        description: "test4"

Execute dbt run
See error

Relevant log output
(dbt-env) ➜  demo_dbt_snowflake git:(main) ✗ dbt run
16:57:36  Running with dbt=1.7.1
16:57:36  Registered adapter: snowflake=1.7.0
16:57:36  Unable to do partial parsing because a project config has changed
16:57:37  Found 1 model, 0 sources, 0 exposures, 0 metrics, 430 macros, 0 groups, 0 semantic models
16:57:37
16:57:39  Concurrency: 1 threads (target='dev')
16:57:39
16:57:39  1 of 1 START sql table model DBT_GEORGE.test_table ............................. [RUN]
16:57:41  1 of 1 ERROR creating sql table model DBT_GEORGE.test_table .................... [ERROR in 1.99s]
16:57:41
16:57:41  Finished running 1 table model in 0 hours 0 minutes and 3.56 seconds (3.56s).
16:57:41
16:57:41  Completed with 1 error and 0 warnings:
16:57:41
16:57:41    Database Error in model test_table (models/test_table.sql)
  000603 (XX000): SQL execution internal error:
  Processing aborted due to error 300002:3929359486; incident 9969370.
  compiled Code at target/run/demo_dbt_snowflake/models/test_table.sql
16:57:41
16:57:41  Done. PASS=0 WARN=0 ERROR=1 SKIP=0 TOTAL=1
(dbt-env) ➜  demo_dbt_snowflake git:(main) ✗

Environment
- OS: MacOSX
- Python: 3.11.5
- dbt-core: 1.7.1
- dbt-snowflake: 1.7.0

Additional Context
Adding a comment using this alternative method works as expected for these column names:
COMMENT ON COLUMN db.schema.sf_test.CURRENT_TIME IS 'test';

Hi @georgewoodhead, these keywords are reserved in Snowflake and should not be used as column names:

CURRENT_DATE : Cannot be used as column name (reserved by ANSI).
CURRENT_TIME : Cannot be used as column name (reserved by ANSI).
CURRENT_TIMESTAMP : Cannot be used as column name (reserved by ANSI).
CURRENT_USER : Cannot be used as column name (reserved by ANSI).

So dbt should not support them either, we will not fix this.
We're actually surprised the model was built successfully to be honest.


What does app.locals.cache = 'memory'; actually do?

From MEAN.io standard express config:

// cache=memory or swig dies in NODE_ENV=production
app.locals.cache = 'memory';

From tj/consolodate.js Issue (emphasis mine):

A workaround
app.locals.cache = "memory"

which enable both express cache and swig cache

I'm trying to understand more about this line of code in the express config file.  I have a few questions:

Are there any other implications other than ensuring swig works properly?
Are there other options other than 'memory'?
Does this cause large JSON responses to remain in memory for any period of time?

I tried researching this on the expressjs API site, but nothing specifically talks about this cache variable, just locals in general.  I looked at this page, but it only refers to view HTML/CSS caching.

I'm not sure. Express.js does not recognize app.locals.cache value of memory. If third-party template engines do (since they can read that value), then it probably means something to them.

Basically, what you found in your research is correct: Express.js does not care what you put in your app.locals, since Express.js itself does not read anything from there; it simply passes the object to your rendering engine, as a messenger, basically. If some rendering engine wants you to set that for it, that's perfectly valid, but not something Express.js would be able to really answer.

Gotcha.  Thanks for the clarification!

No problem!