Response time

Hello.
Recently, the Spellbook's response time has dramatically lengthened. The tavern waits only 60 seconds, after which it issues an error. However, the Spellbook can sometimes give an answer after 2, 3, 5 minutes.
Is it possible to change tavern's waiting time?

This could be made adjusted. Currently, the default timeout is used (which is a minute I think). I will see if I can implement a default in configuration that could be changed.
If you want to test it out, you can add timeout attribute to all ajax calls, set to the desired maximum in milliseconds. See: https://www.w3schools.com/jquery/ajax_ajax.asp

@NikolayWannaCode The timeout should now be adjustable via config.conf constant connectionTimeoutMS. I've increased the default to 2 minutes. Please let me know if there are any issues.
commit ce709869
commit 22e41ca3


Union type guards are not conferred via the ternary operator

TypeScript Version:  2.0.3
Code
// A *self-contained* demonstration of the problem follows...
class A {
  constructor(public a:string) {
  }
}
class B {
  constructor(public b:string) {
  }
}
function test(p:A | B):string {
  return (p instanceof A) ? p.a) : p.b;
}

let q = new A('hello');
console.log(test(q));

Expected behavior:
The type inference conferred by the instanceof type guard should also work within the functions that use the same scope of the ternary operator.
Actual behavior:
The type is not remembered any more, and p must be re-cast as A or B.

The provided code compiles fine on 2.0.3, after fixing the typo in the ternary.

Ah, I see, I had the code formatted differently:
class A...
class B...
function test(p:A | B):string {
  return
    (p instanceof A) ? p.a
    p.b
    ;
}

I didn't figure that the return being on a separate line made a difference, but I see now that the compiler interprets the return statement as a completed statement and then does something very weird with the ternary.  Fix (workaround?  I guess this is valid javascript) by putting a parenthesis on the same line as the return, and enclosing the ternary.


How Do I Download Tracks / Albums

Hi,
As I have installed applemusic-core using npm i applemusic-core command from readme.
But after that how to use the script..? As, I am not much experienced in these.
Thanks

Hello, thank you for downloading our module. At this time, we have not implemented code to download albums/tracks, we will notify you when we add it.

You can use the one in readme.md. sorry, at this time there is no code for downloading the album, you can wait for the next version, thank you.

Hello, thank you for downloading our module. At this time, we have not implemented code to download albums/tracks, we will notify you when we add it.

Ohhhh okay

You can use the one in readme.md. sorry, at this time there is no code for downloading the album, you can wait for the next version, thank you.

I had gone through readme. But as I am not much familiar with programming env. So, was just asking when downloading tracks/albums are available.
Then what is the command to dowload the same..?

You can use the one in readme.md. sorry, at this time there is no code for downloading the album, you can wait for the next version, thank you.

I had gone through readme. But as I am not much familiar with programming env. So, was just asking when downloading tracks/albums are available.
Then what is the command to dowload the same..?

I don't know when it will support albums for sure, but we will update the album downloader code as soon as possible: D
The album support download function seems to be different, but just wait until we update. Thank You.

You can use the one in readme.md. sorry, at this time there is no code for downloading the album, you can wait for the next version, thank you.

I had gone through readme. But as I am not much familiar with programming env. So, was just asking when downloading tracks/albums are available.
Then what is the command to dowload the same..?

I don't know when it will support albums for sure, but we will update the album downloader code as soon as possible: D
The album support download function seems to be different, but just wait until we update. Thank You.

Okay thanks

You can use the one in readme.md. sorry, at this time there is no code for downloading the album, you can wait for the next version, thank you.

I had gone through readme. But as I am not much familiar with programming env. So, was just asking when downloading tracks/albums are available.
Then what is the command to dowload the same..?

I don't know when it will support albums for sure, but we will update the album downloader code as soon as possible: D
The album support download function seems to be different, but just wait until we update. Thank You.

Okay thanks

Hellow, You can now download the album/playlist
u can PR if find a bug and want to fix it.
https://github.com/rizxyu/applemusic-core/blob/main/lib/index.js

Has added album/playlist

You can use the one in readme.md. sorry, at this time there is no code for downloading the album, you can wait for the next version, thank you.

I had gone through readme. But as I am not much familiar with programming env. So, was just asking when downloading tracks/albums are available.
Then what is the command to dowload the same..?

I don't know when it will support albums for sure, but we will update the album downloader code as soon as possible: D
The album support download function seems to be different, but just wait until we update. Thank You.

Okay thanks

Hellow, You can now download the album/playlist u can PR if find a bug and want to fix it. https://github.com/rizxyu/applemusic-core/blob/main/lib/index.js

Here I have lissted my doubts :-
1st doubt :-
As I am not much experienced in coding. How to use the program..? Do, I need to write program myself to use this program..?
2nd doubt :-
What is the quality of tracks / albums / playlist which gets downloaded...?

I will answer your second question, the audio quality has been set to normal audio so you can't mess with it to get better quality.

I will answer your second question, the audio quality has been set to normal audio so you can't mess with it to get better quality.

Okay thanks for info

The feature has been added in version v1.1.0, thanks for waiting!
@Rajeshsahoo12


Suppoer LibEv

Please add LibEvLoop for react

How long it take to support this feature ?

Im not sure ,maybe a week or longer.

OK , I want waiting and hope you implement this feature
Thanks you

Any update?

Dec 11, 2016  from now has not changed?

I have no time to do that.
This is an open source project, you can send PR if you need.

tnx

walkor
please do it i need this feature

Any updates?

I tried to add LibEvLoop for react but unfortunately there is coredump for libev extension and which I  can not solve.
So I decide not to support LibEvLoop until libev is stable enough.


Change data link arrow head to correct default size

The data link arrow head should be "M -6 6 L 0 0 -6 -6" instead of the current much larger size.

Done


Unable to build cmdstan if bash is not at /bin/bash

Describe the bug
install_cmdstan() fails with the following
* Latest CmdStan release is v2.24.0
* Installing CmdStan v2.24.0 in /home/fergie/.cmdstanr/cmdstan-2.24.0
* Downloading cmdstan-2.24.0.tar.gz from GitHub...
* Download complete
* Unpacking archive...
* Building CmdStan binaries...
cp bin/linux-stanc bin/stanc
make: /bin/bash: Command not found
make: *** [make/stanc:61: bin/stanc] Error 127

Warning message:
There was a problem during installation. See the error message(s) above.

To Reproduce
Run install_cmdstan on a system where bash is not at /bin/bash
Expected behavior
Should use /usr/bin/env bash to correctly load bash
**Other information
Changing line 128 in stan/lib/stan_math/make/libraries in the to read
SHELL = /usr/bin/env bash

makes it work.
I can't figure out where that file is in this repository otherwise I would submit a patch along with this issue

Great! Thank you
https://github.com/richardfergie/math/blob/develop/make/libraries#L128 is what the change looks like when it works for me.
I don't really know how to setup the necessary tests for this so I won't submit a pull request for now

For such a simple change you don't need to verify anything locally. Just open a pull request and our CI will run the tests.
But if you do not wish to bother with that, I can apply your patch to an open PR already dealing with some minor compile flag stuff. Will make sure to give the proper credit in that case. Let me know. And thanks again!


"Highlights from past summits" needs keyboard focus styles

https://deploy-preview-388--cds2019.netlify.app/devsummit/

Confirmed, nice!


Poor performance of incremental selection in strings

When using incremental selection, in addition to string content, double or single quotes are also selected.
However, in other languages, only the string content was selected; Therefore, in practical applications, it is not possible to quickly change the content of a string, and it is necessary to re quote double or single quotes.

seems like a nvim-treesitter issue


Global setting to disable icons

Hello. Is there a global setting to disable icons like file_icons and git_icons? Right now the user has to set it manually for every single command which is quite a lot and not very elegant. Also it seems the setting preview_opts is not documented in the readme. I stumbled by accident on it in some other issue. I set it to hidden. Is the preview-window still generated in the background but just not visible?

No global option for the icons, someone asked to add it in the past but I opted against it as not all providers support icons and I thought it was confusing, it's basically only files/git/grep.
The reason preview_opts has no documentation is because it was removed and is now supported only through backward comparability code, the correct option to use would be under winopts.preview.hidden, when set to hidden preview window isn't generated at all, it will only be visible if you press the toggle-preview keybind (default <F4>).

What may seem not necessary for you may seem necessary for someone else, for example, what if someone wants to disable git icons for files and enable them only for git_files?

I understand that it would be useful if somebody for some reason would want to have context-dependent icons but anybody could want anything and than every plugin would have to be turing-complete. I am just wondering why somebody would want that. All other plugins and programs I ever encountered usually only have one option. Icons on or icons off. And I never saw anybody complain. But more than one people complained about not having a global setting. Just wanted to explain my reasoning. But I respect your decision. Thanks for the great plugin.

I understand your reasoning too and it might as well be the case but since it's already been available I can't take this functionality away, especially since it's not really an issue but rather just a minor inconvenience on the first setup (in case you do not want icons).
If you can think of a way to add global option that would not be confusing or inconsistent with overriding "finer tuned options" feel free to open this again and we can discuss it.

If you can think of a way to add global option that would not be confusing or inconsistent with overriding "finer tuned options" feel free to open this again and we can discuss it.

i think we could go with something like disable_all_file_icons that defaults to false, with a note that setting it to true disables fine grained control.
I just wanted to disable icons on all sources, but there are 17 places (In https://github.com/ibhagwan/fzf-lua/blob/main/lua/fzf-lua/config.lua) where I would need to explicitly set this to false. Having this option would help a lot.

Awesome, thanks!


Update protobuf-ts to v2.9.1

Now that v2.9.1 has been released and includes https://github.com/timostamm/protobuf-ts/pull/567 this PR updates the version of protobuf-ts used to v2.9.1 and updates the failing_tests.txt. The conformance tests were then re-run which automatically updated the generated files and README.md.
> cd impl/protobuf-ts
> ./test.sh

WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
I0000 00:00:1690906633.002076       1 conformance_test_runner.cc:315] runner.ts

CONFORMANCE TEST BEGIN ====================================

CONFORMANCE SUITE PASSED: 2028 successes, 0 skipped, 1 expected failures, 0 unexpected failures.

WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
I0000 00:00:1690906636.075112       1 conformance_test_runner.cc:315] runner.ts

CONFORMANCE TEST BEGIN ====================================

CONFORMANCE SUITE PASSED: 0 successes, 8 skipped, 112 expected failures, 0 unexpected failures.

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


Fixing responsive design of sign in button

Closes #341

As discussed, updated version to v4.0.45


Implement List API

Resolves [ch9190]
For a given set of actors, an object_type, and action we can return a relation of object that they can access

@rreinhardt9 ok! this should be good-to-go as of our conversation earlier this afternoon
I broke out all the other 'namespace' changes into here: https://app.clubhouse.io/lessonly/story/9537/rename-all-references-of-object-type-namespace

@rreinhardt9 - Is this good to go?

Thanks @ehourigan ! I'll be getting back to taking a look at this... probably mostly after lunch time at this point :+1:


Split Dex & DexApp into separate charts

Currently Dex and DexApp are deployed with the same chart.  Dex does not require DexApp and in fact we should be using something different less onerous then DexApp.
This ticket should focus on splitting these two applications into their own charts.

This will be addressed with the work that @venezia is doing moving these into their own repo.

@joejulian can I get some issues for the work he's doing or is this not being tracked in github?


hoverfly returns ←[36mresponse←[0m="&map[error:nil response]"

Description of the bug
I'm using hoverfly as a proxy (windows 64 bit)  on my corporate network

when I make a soapui request (to an api on our corporate network) I use hoverflly as the proxy
when I make a request (RESTful webservice which enforces basic auth) I get a valid/expected resposne back in soapui
when I view the hoverfly logs I see a valid request going through but I see ←[36mresponse←[0m="&map[error:nil response]" where I would expect the response
when I export the simulation I see the follwoing

"response": {
"status": 401,
"body": "",
"encodedBody": false,
"headers": {
"Content-Type": [
"text;charset=UTF-8"
],
"Date": [
"Tue, 18 Jun 2019 03:26:12 GMT"
],
"Hoverfly": [
"Was-Here"
],
"Server": [
"none"
],
"Set-Cookie": [
"JSESSIONID=xxx; Path=/xx; Secure"
],
"Transfer-Encoding": [
"chunked"
],
"Www-Authenticate": [
"Basic realm="xxx at<PHONE_NUMBER>567""
]
},
"templated": false
}
Steps to reproduce the issue
Question: how can I get more verbose logs out of hoverfly. its like its acting as a proxy correctly just not capturing the response somehow
Observed result
Hoverfly error messages seen (If none, say none)
(paste here)

If possible, add screenshots to help explain your problem
Expected result
Additional relevant information

Hoverfly version:  master-3266 windows_amd64
Anything that might help us to diagnose the problem

My issue above is when I run hoverfly on my corporate network. Ive tried just now (same win64 install) on my home pc with the time.jsontest.com tutorial and hoverfly seems to work fine.

so Hoverfly can proxy the request fine, but the response being capture is different?  Can you provide some steps to reproduce the problem? Providing the commands you use and the hoverfly logs you got will be helpful.
You can get debug logs from hoverfly:
$ hoverfly --log-level debug

I think I found my issue (soapui related)
I was using soapui initially (soapui -> hoverfly as proxy -> some endpoint)
I had soapui 'pre-emptive auth' as 'use global preference': the consequence of this si the http request is made with no authorization header and if that fails then soapui makes a second request with the authorixation details (i.e. because my endpiont enforces auth then soapui must always make two requests)
hoverfly was capturing the response to the first request (401 unauth) and ignoring theresposne to the second request (might be a hoverfly matcher feature?)
when I replay with soapui setting to 'authenticate pre-emptively' then I see hoverfly capture correctly
when I replay with curl I see no issues

Glad you've figured it out. If the second request is exactly the same as the first one, then it won't be captured.


Support for new portal myuplink.com

Nibe published new firmware versions that allow older VVM devices to connect to the new myuplink portal, nibeuplink.com is deprecrated.
New api endpoint is now api.myuplink.com.
Great about this change is that many more devices are accessible now through this new api.
Could you have a look into this to see what effort it would mean to update the plugin to this new api?
Kind regards!
Ristridin

+1

+1

My F1145 has finally been migrated to myuplink, so I can begin implementing it 😃🎉 Stay tuned! 😉

Hi, my new VMM S320 (running on 3.0.10) with a S2125-8 has been installed 2 weeks ago. It is connected to myuplink and both premium add-ons are activated, since they were included after registration. I am not a developer, but an IT systems integrator, so if you need any testing or feedback feel free to contact me.

API replacement and refactoring of the plugin is done. I will update the unit tests and release the "beta" version soon. The "beta" will contain only temperature sensors.

Thanks @hpruszyn for working on an update to support the new myuplink portal.
Any news on when a beta version will be available?

The first release with temperature sensors is done. The next one will be related to hot water accessory #231.


Joss poetry

Update build with poetry
Update documentation
addressing #19

For https://github.com/openjournals/joss-reviews/issues/6147


cardinal  and concatenate

Hello, author, as shown in the figure, your paper concatenates multiple cardinal, but I can't find the concatenate code in your code, and I find cardinal = 1. Is it because cardinal = 1 that concatenate splicing is not required? Why do you set cardinal = 1?Thanks ！

We use Efficient Radix-major Implementation as described at Section 3.2. It is identical to the architecture described at Section 3.1


nrf/pdm: Add continuous sampling API

I modeled this API off the continuous sampling API provided by the saadc module.  The existing one-shot sampling API does not make it possible to stream continuous samples because it throws away at least one sample between each invocation.
It also fleshes out the Config struct to allow configuring more of the PDM block's settings.

unfortunately pdm.rs had CRLF endings, and your editor auto-changed it to all LF endings so the diff is not useful (it shows all lines as edited). I've fixed the CRLF stuff in #1669, could you rebase? You might get ugly conflicts, the easy way to fix them is to copy pdm.rs out before rebasing and pasting it back in.

Done, I also ran cargo fmt since it looked like I was failing a CI check for that. And I fixed some compatibility problems the CI found with other PACs.


MP > DP experiment

DP [10-60], 32, 8 layer, 400 200

#GPU
Sec per epoch

1
300.412

2
336.081

3
377.795

4
340.281

MP [10-60], 32, 8 layer, 400 200

#GPU
Sec per epoch

1
336.121

2
219.809

3
377.795

4
340.281

Triplet
AlexNet+Euclidean
AlexNet + Cosine

Coarse
0.75814
0.27342
0.28571

Fine-grained
0.80465
0.43965
0.44453

FG: 8599 query sets;
Coarse: 3009 query sets

Speedup of Model Parallel and Data Parallel
DP [10-60], 32, 8 layer, 400 200

I realized that in the DP experiment, we didn't scale batch_size with the number of GPUs :(


Fix type exports in package.json.

I'm not an expert on node packages but this fixed my issue (output below) locally.
src/App.tsx:5:34 - error TS7016: Could not find a declaration file for module 'solid-codemirror'. '/repodir/node_modules/solid-codemirror/dist/esm/index.js' implicitly has an 'any' type.
There are types at '/repodir/node_modules/solid-codemirror/dist/types/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'solid-codemirror' library may need to update its package.json or typings.
5 import { createCodeMirror } from "solid-codemirror";
~~~~~~~~~~~~~~~~~~
src/App.tsx:14:11 - error TS6133: 'editorView' is declared but its value is never read.
14   const { editorView, editorRef } = createCodeMirror();
~~~~~~~~~~
Found 2 errors in the same file, starting at: src/App.tsx:5

Hey, this error is strange since the type is already defined in the root level. What bundler are you using?
anyway I’ll merge this soon

Hey, this error is strange since the type is already defined in the root level. What bundler are you using?
anyway I’ll merge this soon

I'm using Vite v5
Cheers 👍

Ok so probably it’s something that was not breaking in previous version 😄

Available in 2.3.1 😊 thanks
https://github.com/riccardoperra/solid-codemirror/releases/tag/solid-codemirror%402.3.1


Code for visualization

Could you please provide the visualization code of the paper? Thank you very much!

Have you solved this problem?After I change the weight path in the config file to the path of the trained model, it can run.
python demo/demo.py --config-file configs/PASCAL_VOC/base-training/R101_FPN_base_training_split1.yml \--input input1.jpg
But I just tested it here in base-training. In the fine-tune stage，I can't do the same thing.The content of the error report is
AssertionError: get_event_storage() has to be called inside a 'with EventStorage(...)' context!
Can we talk about this issue？

Put storage = get_ event_storage () in roi_ heads.py Change  to:
with EventStorage():
storage = get_event_storage()


AmboVent

I do not know if this fits in the list, but you may be interested in https://github.com/AmboVent/AmboVent .

It is #2.


Fixing validation of libraries

Added validation of  libraryId per provider
Consolidated all validation of libraries (including conflicts of libraries and files within a library) in one place : LibrariesValidator.

Notes:

I also added validation for duplication of libraries for a given provider (basically same library name and same provider). Is this what we want? Do we want it to be even more generic and fail on any duplication of libraries based on name only?

@prafullbhosale , some of your tests are now failing, Specially the update ones, Can you please take a look and see if you need to call some of the new validation API's or if there are bugs with the new implementation? Looks like if fails when tries to get the latest version.

    public void Execute_PartiallyValidManifest()

If the manifest is partially invalid, do we do a partial restore?

Refers to: test/Microsoft.Web.LibraryManager.Build.Test/RestoreTaskTest.cs:52 in 2922363. [](commit_id = 2922363bee60080ae6ee400a0c4623d5d58300bd, deletion_comment = False)

@prafullbhosale , some of your tests are now failing, Specially the update ones, Can you please take a look and see if you need to call some of the new validation API's or if there are bugs with the new implementation? Looks like if fails when tries to get the latest version.

I looked at the failing tests. Most of them are failing due to the fact that now we don't auto-populate defaults in the LibraryInstallationState instances when reading from JSON.
These can be fixed easily.
The tests failing for Install however are due to the fact that CLI will sometimes set a DefaultProvider after the manifest is instantiated. The LibrariesValidator isn't aware of the change made to the manifest and then fails to validate because we are trying to install with a default provider.

For the first ones, you can call LibrariesValidator.GetExpandedLibraryAsync. In general we should always call operations (Install, Update, Restore, etc) on the expanded version of hte library and persist the original one to the manifest.
For the second one, can we only validate after the Manifest has all the default settings?

In reply to: 397390739 [](ancestors = 397390739)

            await writer.WriteAsync(buffer, 0, buffer.Length, cancellationToken).ConfigureAwait(false);

Are we only writing the first 4KB of the manifest?  What if the buffer is > 4096B long?

Refers to: src/LibraryManager/Manifest.cs:445 in 2922363. [](commit_id = 2922363bee60080ae6ee400a0c4623d5d58300bd, deletion_comment = False)

Might be unrelated to this pr. I was testing things in this branch and I noticed we log error message twice.

Caused too many regressions. We can leave like this for now and implement proper validation later on.


New Theme: aerc

aerc email client
https://github.com/staal0/aerc

It's 🧟 Alive: https://draculatheme.com/aerc


NullReferenceException in MyBlueprintIdTracker.OnAdded

This exception is logged when the server (Torch or DS) loads the world (it is handled by MGP's own error handling).
16:44:58.5704 [ERROR]  Multigrid Projector: : System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.NullReferenceException: Object reference not set to an instance of an object.
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Sandbox.Game.Entities.MyBlueprintIdTracker.OnAdded(MyProjectorBase projector, HashSet`1 reservedIds)
   at Sandbox.Game.Entities.Blocks.MyProjectorBase.SetNewBlueprint(List`1 gridBuilders)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at MultigridProjectorServer.Patches.MyProjectorBase_Remap.Prefix(MyProjectorBase __instance)

This seems to be harmless, but still not nice to happen.
Root cause: MyBlueprintIdTracker.OnTrackedIdAdded is null in the MyBlueprintIdTracker.OnAdd method.
Crashing code line:
MyBlueprintIdTracker.OnTrackedIdAdded.InvokeIfNotNull(reservedIds, projector);

Workaround: Add listeners to all three event handlers defined in MyBlueprintIdTracker, so they are never null.

Workaround implemented in MGP 0.7.6. This may be obsolete once #92 is solved.


Resolve bre.config.paths.deployments to default when not provided.

bre.config.paths.deployments is ./deployments by default. But it does not seem to be reflected in the buidler runetime enviroment.
Is it possible to assign this default to the buidler runtime enviroment?

fixed in hardhat-deploy


Adds compose v2 and v1 support

Please reject the changes to the main.yaml file. Those changes do not affect the functionality. the change was made for testing purposes.
the rest is OK and tested.

can you reset the main.yaml file


Fix space used by empty issue body in IssueActivity

This space is not because of empty bodies. It is because of the overflow button and comment count.


Enables replacement of all ${env:VARIABLE} replacements

Now handles all env variable replacements for launch.json configs.
Even if you use them in your bsconfig.json file they will replace - which is a nice side effect for some of the projects I'm part of right now actually.
I left the existing piece that was setting default values based on config keys that you put in the env file but I'm not sure how applicable that is so you can let me know if that needs to just get scrapped now.
Let me know if anything sounds like it might break something else but it appears to be working as intended as of right now.

Looks great. Concerning the "existing piece that was setting default values based on config keys", it seems like that does the same thing that your code does, so if you're pretty confident yours will catch all of the env substitutions, then we can scrap that piece. So yeah, go ahead and remove that piece and then I'll merge the PR.
Also an unrelated side-note: if you put env settings in your bsconfig, you then can't run the bsc tool to build the code from the command prompt using that bsconfig.json. Not a big deal if you only use bsconfig.json for the extension, but just wanted to mention it just in case.

Looks great. Concerning the "existing piece that was setting default values based on config keys", it seems like that does the same thing that your code does, so if you're pretty confident yours will catch all of the env substitutions, then we can scrap that piece. So yeah, go ahead and remove that piece and then I'll merge the PR.
Also an unrelated side-note: if you put env settings in your bsconfig, you then can't run the bsc tool to build the code from the command prompt using that bsconfig.json. Not a big deal if you only use bsconfig.json for the extension, but just wanted to mention it just in case.

It's not that I think they do the same thing per say... just not sure it does anything at all...
Are there "default" env variables you would put in place that woul dneed to be replaced even if the .env file doesn't contain them? Possibly ROKU_USER or something?

@TwitchBronBron Yeah, we need to have the defaults for your tests to pass.

So I ended up leaving it in... feel free to merge whenever you're ready.

@TwitchBronBron Any last concerns to get this merged?


Change cookbook url

URL(s)
Example
https://developer.mozilla.org/en-US/docs/Web/CSS/Layout_cookbook/Pagination
Request type

[ ] New documentation
[X] Correction or update

Details
I really like the term recipe book which conceptually conveys the idea. Could you change the url to make it more concise?
from
/docs/Web/CSS/Layout_cookbook/Pagination
to
/docs/web/CSS/cookbook/pagination

@rachelandrew is currently doing some work on thinking about how to represent more cookbook type material on MDN. Where there are multiple cookbooks, we'll have to think about how to represent the URLs.

@rachelandrew is currently doing some work on thinking about how to represent more cookbook type material on MDN. Where there are multiple cookbooks, we'll have to think about how to represent the URLs.

But I'm closing this for now. We'll open a more general issue to deal with this when the time comes.

But I'm closing this for now. We'll open a more general issue to deal with this when the time comes.

Some design in the concept of cooking should be intresting 👍👨🍳👩🍳🔪🍗🍙🍲🍳🌮🌯🍚

Some design in the concept of cooking should be intresting 👍👨🍳👩🍳🔪🍗🍙🍲🍳🌮🌯🍚


feature request: API to create new snippet

It would be great if we could create a go-playground example by sending a HTTP POST request of some code, and get a snippet ID in return, instead of having to create a playground share link manually.
Thanks for your consideration!

@golightlyb The project uses the original Go Playground API to create snippets. See goplay.Share method.


Change static methods, move stuff around and add some more problems

Need to update static class after going from netcore3.1 to net5.0

lgtm


Worker stuck at running with current job None

When raised error from a test sometimes (if not all the time) the app hangs with state "running" and current_job is None,
Reproduce:

raise an error from a test and run.

it was hanging with status "running" and current_job "None" at the second time that you run the test sequence


No semantic changes are intended.
Replaces: #508

Hello @oesteban, thank you for submitting the Pull Request!

In the file niworkflows/interfaces/__init__.py:

Line 5:1: F401 '.segmentation.FASTRPT as FAST' imported but unused
Line 6:1: F401 '.registration.FLIRTRPT as FLIRT' imported but unused
Line 6:1: F401 '.registration.ApplyXFMRPT as ApplyXFM' imported but unused
Line 6:1: F401 '.registration.RobustMNINormalizationRPT as RobustMNINormalization' imported but unused
Line 6:1: F401 '.registration.ANTSRegistrationRPT as Registration' imported but unused
Line 6:1: F401 '.registration.ANTSApplyTransformsRPT as ApplyTransforms' imported but unused
Line 6:1: F401 '.registration.SimpleBeforeAfterRPT as SimpleBeforeAfter' imported but unused
Line 14:1: F401 '.utils.CopyXForm' imported but unused
Line 14:1: F401 '.utils.CopyHeader' imported but unused
Line 14:1: F401 '.utils.NormalizeMotionParams' imported but unused
Line 14:1: F401 '.utils.SanitizeImage' imported but unused
Line 15:1: F401 '.plotting.FMRISummary' imported but unused
Line 15:1: F401 '.plotting.CompCorVariancePlot' imported but unused
Line 15:1: F401 '.plotting.ConfoundsCorrelationPlot' imported but unused

To test for issues locally, pip install flake8 and then run flake8 niworkflows.


API - List Projects

Allow clients to request a list of repositories/projects that this augur instance is tracking.

This endpoint will be at http://giants-project.sociallycompute.io:5112/api/unstable/giants-project/repos

The endpoint at http://giants-project.sociallycompute.io:5113/api/unstable/giants-project/repos successfully lists all repo_ids as of commit f8ab01777c5df2a9abaf5371affeb1728eb622f8 on branch giants-api.


Open none text files

It would be neat to edit other file types like configs etc.

I can set it so that it is possible to choose any type of file, but I would like to make it a choice just so its easy to choose .txt and get only .txt files. I will most likely add an additional option to be able to open any type of file.

donzo!

I reverted the commit. I am instead going to go with an approach where I only add specific mime types that can be viewed and edited the same as text files to avoid the user opening images, videos, zip's, and other files that are not compatible with editing as text, since that was not the intention of this feature.

In the future I will add an option or button with a warning to open any file type, even ones not compatible with text, such as images or videos. Not high priority though, since it seems a bit niche.


Atom.Object.defineProperty.get is deprecated.

(autogenerated) atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace), which returns an HTMLElement.
Referenced on:
Object.activate - .../the-closer/lib/the-closer.coffee:12:9
Object.activate - .../the-closer/lib/the-closer.coffee:20:9
Object.activate - .../the-closer/lib/the-closer.coffee:23:9

Thanks for reporting this -- should be fixed in v0.3.0 which I just published.


feat: add calendar app

Basic calendar: show current month and day, browse previous and next months

Looks great - thanks!


No description on HACS page

Home Assistant version

SmartIR version

SmartIR configuration
N/A

Add SmartIR relevant part of HA configuration.yaml  formatted as a code here

Describe the bug
No INFO/README text when opening HACS SmartIR page

Expected behavior
To see the full readme from github: (after editing hacs.json and renaming INFO.md to README.md)

To Reproduce
Steps to reproduce the behavior:

Go to 'HACS'
Click on 'SmartIR'
See error

Debug log
N/A

I'll put in pull request?

Hello, thank you for catching the issue and solving it as well. Your PR was merged into master.


Edit Posts and Comments

I saw an option on steemit.com where user can edit there posts and comments.
Can anyone tell me how can we apply this and insert edited posts again into blockchain.
Are there any methods available on steem for the same.

Hi, @PJ1423.
Editing a comment/post is similar to creating it - make sure to keep the parent permlink, author and permlink the same and change whatever you want in the body, tags, etc.
Hope that helps :)


fix: remove verbose mode from normal runs

Verbose mode could be turned on using extra parameters.
At the moment, you get excessive logs which makes it hard to read the output.

@tembleking @manuelbcd would you mind reviewing?

good point @pawnu, it makes sense. Let me double-check it from a functional point of view and I will get back to you.
Best


Images and Fonts by organization

As discussed today, we will most likely need to manage images, logos, fonts on organization level.
The files should not be directly visible as mediafiles in meetings but follow  "default_logo", "default_font" settings.

duplicate, see openslides-client/issues/1752


Subclassed BitField for special keyspaces

Waiting on project-rig/rig#91.

If project-rig/rig#94 goes ahead then get_with_indices will need some minor changes and pretty much everything else can go.


allow the possibility to add an input max width

add with title feature
allow the possibility to add an input max width

Codecov Report

Merging #400 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #400      +/-   ##
=========================================
+ Coverage    99.8%   99.8%   +<.01%
=========================================
  Files         155     155
  Lines        3677    3679       +2
  Branches      878     879       +1
=========================================
+ Hits         3670    3672       +2
  Misses          7       7

Impacted Files
Coverage Δ

src/components/filterBox/FilterBox.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79b41d5...1e1421f. Read the comment docs.

Should the input have ellipsis ( ... ) to show that it's cropped ?

@GermainBergeron I don't think so, you can navigate in it, that's an input


Sysmon service fails after a while

Describe the bug
When the Sysmon service is started (systemctl start sysmon), the service fails after some time. The duration is not the same for every run. One time the service fails within a couple of seconds and the other time it fails after two minutes. Also, the Sysmon service only fails for about half of our servers. The other half is working fine.
To Reproduce
Unknown
Sysmon version
v1.3.1
Distro/kernel version
Operating System: Red Hat Enterprise Linux 8.8 (Ootpa)
CPE OS Name: cpe:/o:redhat:enterprise_linux:8::baseos
Kernel: Linux 4.18.0-477.10.1.el8_8.x86_64
Architecture: x86-64
Sysmon configuration
config.txt
Logs
status.txt
journalctl.txt (reverse chronological order)
Expected behavior
The Sysmon service runs without failing after a short amount of time
Additional context
We also notice a memory leak, maybe this has something to do with the problem.

I have the same / similar issue on Ubutnu server:
VM on HyperV
Distributor ID:  Ubuntu
Description:      Ubuntu 22.04.3 LTS
Release:            22.04
Codename:       jammy
Kernel:              5.15.0-88-generic
Sysmon version:
v1.3.1
After install it runs for a bit then just stops, no specifc time interval.
If a system is under load and I do a fresh install of sysmon it will not start at all.
In my case it does not start again after stopping.


Add a version argument to tern CLI

This is a preliminary commit to add a --version option to Tern's CLI.
Currently there is no way to identify the installed version and this
commit will fix that. This will be useful for individuals opening
bugs who need to provide the version of Tern that they are running.
Super #209
Signed-off-by: Rose Judge<EMAIL_ADDRESS>

It's also often useful to put environment information in the version output.  That way when a bug is report you can simply ask the originator to copy the output from the --version command.
For example, see some of the comments on a similar PR for bandit:
https://github.com/PyCQA/bandit/pull/284

It's also often useful to put environment information in the version output. That way when a bug is reported you can simply ask the originator to copy the output from the --version command.
For example, see some of the comments on a similar PR for bandit:
PyCQA/bandit#284

Thanks for the pointer, Eric! I updated my PR to print the python version for now. I'll also work on adding OS information and amend this PR tomorrow or Monday.


refactor: improvements

:white_check_mark: Build TheIdServer 4.4.1-ws-fed-metadata1-0056 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/9a87eb944c by @)


HDFS-16585.Add @VisibleForTesting in Dispatcher.java

Description of PR
Add @VisibleForTesting in Dispatcher.java

JIRA: HDFS-16585

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
37m 19s

trunk passed

+1 :green_heart:
compile
1m 40s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
1m 34s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 26s

trunk passed

+1 :green_heart:
mvnsite
1m 45s

trunk passed

+1 :green_heart:
javadoc
1m 24s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 48s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 42s

trunk passed

+1 :green_heart:
shadedclient
22m 58s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 24s

the patch passed

+1 :green_heart:
compile
1m 26s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
1m 26s

the patch passed

+1 :green_heart:
compile
1m 20s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 20s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  4s

the patch passed

+1 :green_heart:
mvnsite
1m 26s

the patch passed

+1 :green_heart:
javadoc
0m 57s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 32s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 18s

the patch passed

+1 :green_heart:
shadedclient
22m 23s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
247m  2s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m 15s

The patch does not generate ASF License warnings.

355m 50s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4337/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4337

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 7719c07d5ce8 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 153756847d8b013d00d15acddcb9b7af15d4e7df

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4337/1/testReport/

Max. process+thread count
3460 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4337/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Twitter to Signal

This PR adds some new functionality to the scanner bot in the form of a Twitter-to-Signal transport.
When the bot is launched, a secondary task will "track" (Twitter API term) the desired hashtags. If the "auto-scanner" is enabled, tweets that meet certain criteria (namely from trusted tweeters) will be sent out to the Signal channel. The "auto-scanner" can be enabled or disabled by anyone in the Signal group by messaging "Auto scan on" or "Auto scan off" (casing and whitespace don't matter).
A couple of technical changes had to be made in order to accomplish this:

I renamed a bunch of functions/constants and moved things around so that dependencies could be managed a bit better and things could be clearer
It turns out that the signal-cli client cannot run more than one instance at a time. Since both tasks are running concurrently, this caused a deadlock since the signal listener was always running. In order to manage this, I've changed the signal listening loop to stop periodically (every 10 seconds) to allow other signal-cli calls to run. Additionally, I've added a python lock to make sure that no deadlocks occur.
Tweepy has an "async" option for their streaming API, but it runs the stream in a separate thread, not python's async interface. So I had to create a task by using asyncio's interaction with ThreadPoolExecutor.

All that to say, I've got this out in production now and it appears to be working well!

I am also surprised you couldn't just run two processes and maybe write a temp file / files to a temp dir that are picked up by the secondary process, processed, then the picked up file removed. That seems like it would solve the locking issue maybe? Not sure just thinking out loud.

I think it would just be good to have that as like "hey this loop will run until requested interrupt which then triggers this other script to run for 10 seconds which then exits and the entire system restarts w/ the original loop triggered" -- if that is what happens then I understood you comment a bit more than I think I did which is cool.

You're totally right, this could use a more accurate description of what's happening! I'll add that, but TL;DR is that only 1 instance of signal-cli can be running at once, and it can either be listening or sending but not both. So these loops go back and forth both listening to their streams, and then sending messages as needed (while balancing which one is using the signal-cli lock)


Add tenant group filter to sites list

Environment

Python version: 3.6
NetBox version: 2.5.2

Proposed Functionality
On the acutally Views (list see below) you could only show "Tenant".
The "Tenant-Groups" doesn´t show up.
Extend the following Views with "Tenant-Groups" and include also the "Tenant-Group" Search on the right Side of every selected View-Template.
In the Table View you should see the Tenant-Group Member by show a Hover-Popup if you move the mouse on the Name of the Tenant-Group inside the "Table-View".
dcim/sites/
dcim/racks/
dcim/rack-elevations/
dcim/rack-reservations/
dcim/devices/
ipam/ip-addresses/
ipam/prefixes/
ipam/vrfs/
ipam/vlans/
virtualization/virtual-machines/
circuits/circuits/

Use Case
If you use different Tenant-Groups because actually you could only select ONE Tenant you wouldn´t see them in the actually views.
For example on Sites it should also show up like this:

and on the Search-Bar (right Side):

Database Changes
Dont know.

External Dependencies
No.

@jeremystretch Should tenant-group value be visible in the API call to these endpoint? Basically update the class of NestedTenantSerializer to output somethiing like
"tenant": {
    "id": 314,
    "url": "http://localhost:32770/api/tenancy/tenants/314/",
    "name": "Tenant A",
    "slug": "tenant-a",
    "group": "Customers"
},

Another question i have is that adding more columns always takes up a lot of screen width in the table. Would it be better to mangle both the name and the group into the already existing table Tenant by formatting it something like Tenant A -- Customers or Tenant A (Customers) or any other variant that would look good.


Tooltip - Option to remove spacer

Can there be an option to remove the spacer between the 2 specs?

I thought about this, and there are a lot of cases where this wouldn't be desirable. I thought about making a compact mode, where only the score would be displayed (not gems, procs etc), but even then there are Offhand scores and there may be other stuff in the future that makes sense for people to see. Overall it would be kind of confusing. Closing this.


Getting Error msg immediate after starting watcher

Hi, I am getting Error msg immediate after starting watcher
" in 1:45-2 hours" has sent too many requests to MTurk and has been automatically stopped."
Hope I am facing issues after updating the mturk engine. Please do the needful.
Specifications

Mturk Engine version: 1.7.0
Browser: Firefox 59.0.3 (64-bit)

Hope old version is better for me 1.6.2.. So please share me How to degrade the version from 1.7.0 to 1.6.2?
Unable to catch the Hit through desktop notifications but earlier It got easily max hit which showing desktop notification ...
Please do the needful

This is actually intended behavior if a watcher runs into a 429 response from Mturk, which is caused when you've made too many requests (usually because too many watchers are running or because a watcher has too low of a delay).
If you think this is being caused because of something else, feel free to reopen this issue.


Evaluation Script For The Other Benchmarks

Hello! First of all, this is really fascinating work. Thanks for the contribution.
I wanted to reach out and ask if you could share the evaluation scripts for mPLUG-OWL2 you used for benchmarks shown in the main figure (e.g. SEEDBench, QBench, etc.). It would also be great if you could provide (or include in the script) any specific prompting you might have done for your zero-shot evaluation on those datasets.

Hi, for the SEEDBench and Q-Bench, we treat this as the Multiple Choice Question for open-generation. You can refer to the MMBench (which is also a MC benchmark) for reference, especially for the prompt reference.


Error on DemoProject

Hi,
When i try to build the demoProject with Xcode 8.2.1 i get the attachd error,

1.5.3 was released. https://github.com/metasmile/NSGIF2/releases/tag/1.5.3
Thanks for your report.


Timeline for testing code and full training code

Impressive and innovative work! Could you please share any information on the expected release timeline for testing code and full training code? Having access to these would be immensely beneficial. Thank you for your contributions and looking forward to the update.

Thank you very much for your interest in our work. We are currently busy with a recent significant DDL. We plan to publish our code by the end of next month.

您的邮件已经收到！


Add support for per-template Handlebars partials

To support recursive rendering of types

Done 0.2.1


remove myself from a bunch of maintainer roles

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks for all your work on Electron , firefox, chromium, various libraries, .... !!!! There are a lot of commits in 2024 alone 😂 you will be missed. Godspeed!

^ this is not a maintainer removal, just abandoning some packages


Update swift-argument-parser

@swift-ci Please test

Can you also make sure to update the Package.swift for Swift-DocC? This will cause issues over there if not.

Can you also make sure to update the Package.swift for Swift-DocC? This will cause issues over there if not.

Yep, see also apple/swift#40103 which is trying to do a coordinate update across all affected repos.


RSpec with Rails 4.1.0.beta, warning: you should require 'minitest/autorun' instead

In trying to run RSpec 2.14.0.rc1 on Rails 4.1.0.beta, I immediately get a warning:
you should require 'minitest/autorun' instead

that comes up in the backtrace assigned to the line
require 'rspec/rails'

in spec_helper.rb If I add this before it
require 'minitest/autorun'

the warning goes away. Alternatively, the problem goes away without changing the spec_helper.rb file if I add
gem 'minitest'

to the gemfile
It will do this with a single spec file that only contains
require 'spec_helper'

Note for people ending up on this thread as I did: I required 'minitest/autorun' as advised, which fixed the warning, but generated new errors, as it was now overriding Object to add a stub method, and I was already using the stub method that was already being imported by rspec.
After some fiddling, it turns out that I can require 'minitest/spec' instead, which will fix the warning, without messing with rspec's Object#stub.


Update action-groups.md

The following fact hasn't been mentioned anywhere in the doc:
"Action groups are created for each unique set of users that will be notified. For example, if User1, User2 and User3 have to be notified for two alert rules, you need to create only one action group."

Hi @azarboon ,
Thanks for opening this PR. I am trying to understand the change you are trying to make. What happens if the same users have to be notified, but the action is different? I am concerned that your change implies that as long as the intended recipients are the same, even if the actions are different, you can still create only one action group.
Happy to discuss over email or Teams, and to include @jacegummersall for his input.
Thanks,
Abby

@AbbyMSFT how about this? Please feel free to edit it:
"So long as actions are the same, Action Groups are created for each unique set of users that will be notified. For example, if User1, User2 and User3 have to be notified e.g. by email for two alert rules, you need to create only one action group."

@azarboon , much better! I've edited a little bit:
"Action Groups are defined by the unique set of actions and the users to be notified.  For example, if you want to notify User1, User2 and User3 by email for two different alert rules, you only need to create one action group which you can apply to both alert rules."
How does that sound?
Abby

@AbbyMSFT its great. Please edit  and merge it.

Hi @azarboon
It is your branch, so you need to make the change, and then I can approve.
Thanks,
Abby

@AbbyMSFT modified it and committed changes. Kindly please approve it.

#sign-off


[Bug] unable to load multiple yaml files with dicts

Test case:
1.yaml:
a:
  b: b

2.yaml
somevar: value

layeredconfigbug.py:
from layeredconfig import LayeredConfig, YAMLFile
yamls = [YAMLFile('1.yaml'), YAMLFile('2.yaml')]
cfg = LayeredConfig(*yamls)

executing layeredconfigbug.py crashes on last line with:
Traceback (most recent call last):
  File "layeredconfigbug.py", line 7, in <module>
    cfg = LayeredConfig(*yamls)
  File "/usr/local/lib/python2.7/dist-packages/layeredconfig/layeredconfig.py", line 87, in __init__
    cascade=self._cascade))
  File "/usr/local/lib/python2.7/dist-packages/layeredconfig/yamlfile.py", line 29, in __init__
    with codecs.open(yamlfilename, encoding="utf-8") as fp:
  File "/usr/lib/python2.7/codecs.py", line 881, in open
    file = __builtin__.open(filename, mode, buffering)
TypeError: coercing to Unicode: need string or buffer, NoneType found

Tested with layeredconfig 0.2.1, 0.2.2.dev1

Excellent bug report -- I can confirm that I see the same issue. Will try to fix it and release version 0.2.2 ASAP.

Great to hear that, can't wait for the fix ;)

Fixed by the PR from @badkapitan

Version 0.2.2 is now up on pypi. Again, thanks for the help!

Not a problem, glad I could help :) Good luck with the rest! :)


Connection Pooling results in unpredictable isolation levels

Description
A connections default isolation level is "read commited". However, once a transaction has been committed or rolled back, the isolation level remains as it was set by the transaction, until the connection is closed or disposed. (see note under https://docs.microsoft.com/en-us/dotnet/api/microsoft.data.sqlclient.sqlconnection.begintransaction?view=sqlclient-dotnet-core-2.1)
However because SqlClient reuses connections in a pool, connections are not actually closed between uses of different SqlConnection instances. Because of this, after using a SqlConnection and setting the isolation level via a transaction, another SqlConnection created elsewhere can inherit this isolation level.
This can cause unexpected increases in isolation level (especially when using Serializable and Repeatable Read) that can have knock on consequences such as deadlocks. In the other direction, you can get dirty reads if a connection inherits a "read uncommitted" isolation level from a previous use of the connection.
This strikes me as potentially very dangerous behaviour, as unless you explicitly set the isolation level for every use of the connection, you can get essentially a random level.
To reproduce
Code tested with Microsoft.Data.SqlClient 2.1.3 (and System.Data.SqlClient 4.4.0), and uses Dapper (2.0.90) to execute the SQL (though Dapper is not related to the problem).
using System;
using System.Data;
using System.Linq;
using System.Threading.Tasks;
using Dapper;
using Microsoft.Data.SqlClient;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            MainAsync(args).GetAwaiter().GetResult();
        }

        static async Task<string> GetIsolationLevel(IDbConnection connection, IDbTransaction transaction = null)
        {
            string result = null;
            await connection.QueryAsync<string, string, string>("DBCC USEROPTIONS",(option, value) =>
            {
                if (option == "isolation level")
                {
                    result = value;
                }
                return option;
            }, transaction: transaction, splitOn: "Value");
            return result;
        }

        static async Task MainAsync(string[] args)
        {
            var connStr = "Server=[redacted];Database=[redacted];User Id=[redacted];Password=[redacted];";

            Console.WriteLine("Using no transaction");

            (await Task.WhenAll(Enumerable.Range(1, 10).Select(async i =>
            {
                await using var conn = new SqlConnection(connStr);
                await conn.OpenAsync();
                return (Test: i, IsolationLevel: await GetIsolationLevel(conn));
            }))).ToList().ForEach((t) => Console.WriteLine($"Test {t.Test}: {t.IsolationLevel}"));

            Console.WriteLine();
            Console.WriteLine("Using serializable iso level transaction");

            (await Task.WhenAll(Enumerable.Range(11, 10).Select(async i =>
            {
                await using var conn = new SqlConnection(connStr);
                await conn.OpenAsync();
                await using var tran =
                    conn.BeginTransaction(i % 2 == 0 ? IsolationLevel.Serializable : IsolationLevel.ReadUncommitted);
                var result = (Test: i, IsolationLevel: await GetIsolationLevel(conn, tran));
                tran.Commit();
                return result;
            }))).ToList().ForEach((t) => Console.WriteLine($"Test {t.Test}: {t.IsolationLevel}"));

            Console.WriteLine();
            Console.WriteLine("Using no transaction");

            (await Task.WhenAll(Enumerable.Range(21, 10).Select(async i =>
            {
                await using var conn = new SqlConnection(connStr);
                await conn.OpenAsync();
                return (Test: i, IsolationLevel: await GetIsolationLevel(conn));
            }))).ToList().ForEach((t) => Console.WriteLine($"Test {t.Test}: {t.IsolationLevel}"));
        }
    }
}

Using no transaction
Test 1: read committed snapshot
Test 2: read committed snapshot
Test 3: read committed snapshot
Test 4: read committed snapshot
Test 5: read committed snapshot
Test 6: read committed snapshot
Test 7: read committed snapshot
Test 8: read committed snapshot
Test 9: read committed snapshot
Test 10: read committed snapshot

Using serializable iso level transaction
Test 11: read uncommitted
Test 12: serializable
Test 13: read uncommitted
Test 14: serializable
Test 15: read uncommitted
Test 16: serializable
Test 17: read uncommitted
Test 18: serializable
Test 19: read uncommitted
Test 20: serializable

Using no transaction
Test 21: serializable
Test 22: read uncommitted
Test 23: serializable
Test 24: read uncommitted
Test 25: read committed snapshot
Test 26: read committed snapshot
Test 27: read committed snapshot
Test 28: read uncommitted
Test 29: serializable
Test 30: read uncommitted

Expected behavior
First batch of 10 tests show "read committed snapshot" isolation level
Second batch of 10 oscillate between "serializable" and "read uncommitted"
Third batch of 10 tests all show "read committed snapshot" again
Actual behavior
Third batch of 10 is a mismatch of "read committed snapshot", "read uncommitted" and "serializable"
Further technical details
Microsoft.Data.SqlClient version: 2.1.3
.NET target: Core 3.1
SQL Server version: SQL Server 2017
Operating system: Windows, Linux

Note - when using this (rather crude) wrapper around the SqlConnection that sets the iso level to READ COMMITTED when the Open method is called, the results change to be as expected.
using System.Data;

namespace ConsoleApp1
{
    public sealed class SafeConnection : IDbConnection
    {
        private readonly IDbConnection _connection;

        public SafeConnection(IDbConnection connection)
        {
            _connection = connection;
        }

        public void Dispose()
        {
            _connection.Dispose();
        }

        public IDbTransaction BeginTransaction()
        {
            return _connection.BeginTransaction();
        }

        public IDbTransaction BeginTransaction(IsolationLevel il)
        {
            return _connection.BeginTransaction(il);
        }

        public void ChangeDatabase(string databaseName)
        {
            _connection.ChangeDatabase(databaseName);
        }

        public void Close()
        {
            _connection.Close();
        }

        public IDbCommand CreateCommand()
        {
            return _connection.CreateCommand();
        }

        public void Open()
        {
            _connection.Open();
            var command = _connection.CreateCommand();
            command.CommandText = "SET TRANSACTION ISOLATION LEVEL READ COMMITTED;";
            command.ExecuteNonQuery();
        }

        public string ConnectionString
        {
            get => _connection.ConnectionString;
            set => _connection.ConnectionString = value;
        }

        public int ConnectionTimeout => _connection.ConnectionTimeout;

        public string Database => _connection.Database;

        public ConnectionState State => _connection.State;
    }
}

Similar to #96.
Root cause explained: https://github.com/dotnet/SqlClient/issues/96#issuecomment-298443763

Thank you for your quick response - I had not spotted that other ticket.
Given it is 4 years old, am I right in assuming a fix is not likely to be on the horizon? I understand that the isolation level persisting for a connection after transaction rollback or commit is likely an intended behaviour within SQL Server's implementation, but when combined with connection pooling in the SqlClient it produces behaviour that seems both unexpected and potentially dangerous.
Connection pooling is a useful abstraction to avoid having to manage actual connections manually, but I think this abstraction heavily implies that separate SqlConnections will be independent, regardless of whether they use the same underlying shared connection. After all, SqlConnection instances are opened, closed, and disposed, with the state of the actual connection being completely abstracted away behind these actions.
Resetting the isolation level (as well as any other state on a connection) when a connection is returned to the pool (or alternatively when it is selected from the pool) would mean new SqlConnections are homogenous, and all behave identically, regardless of whether the connection has been used before and in what way.
I have no idea how complicated implementing this would be, but I assume its non-trivial as the behaviour has been known about for several years. I would definitely add my +1 to any attempt to fix this though, as unexpected locking and dirty reads feels pretty dangerous.


Update modify.md with params for #6359

https://github.com/iterative/dvc/pull/6359 adds some additional parameters to the azure config. This PR is the documentation for those.

Thanks @meierale . I moved this information to a hidden note for Windows users above all of the custom auth params. That's what this affects right?
@isidentical PTAL when you can (please approve if correct).


Two questions

Hello. I have two questions:

How many connections including active can the server support? I tried to implement a datagram server using dotnetty and was faced with the fact that their implementation of the UDP server works only in single-threaded mode.
Are you considering the inclusion of support for the header from the new network unit? They use a 32 bit header.

https://github.com/Unity-Technologies/multiplayer/blob/b7c2798078ab2408caf232b8775aa6ff9ac71ccd/com.unity.transport/Runtime/NetworkProtocols.cs#L25
Library version: [commit id (9f2a1d1a8244510d9ec502cc002bfdddba666d09) or release version (<IP_ADDRESS>)]
Framework: Unity, dotnetcore

@iamkisly

depends on server hardware and game logic. Some games works with 1000 clients, some can only with 100 at maximum.
No


nButtons &= ~IN_ATTACK stuck

Hi, im trying to fix my aimbot cause pistols shots only 1 shoot.
When i do nButtons |= IN_ATTACK it shoot but i cant unset in_attack cause if i did it my game not shoot.
And if i dont clear in_attack, it shoot and still holding...
In unknowncheats told me "change CMD to 0x54" but i dont understand at all.... anyone can help me to fix it and fix in base too?
Thanks!

I don't think this is a base's problem


key repetition in macOS terminal

The keyboard repetition in the macOS terminal seems not to work correctly. The repetition actually works, but does not appear, e.g. when scrolling through the users roster, the repetition only shows, when the key is released again. The same happens when moving through the message input field.

I'm sorry to hear this.  it looks like this is an issue in the notty library -- and I don't have a system to test this with the macOS terminal.  maybe @pqwy has an idea what could be the issue.

I further noticed that sometimes using key repetition in a macOS terminal using jackline on a remote server, e.g. scrolling up and down through the user roster, there is "~5" and "~6" shown as an output.

@lix ah, interesting. It does indeed sound like notty doesn't handle repeat_char aka rep aka rp from your client.
Could you provide the output of these commands:
echo "$TERM"
infocmp -I
tput rep 65 8 | xxd

Hi @cfcs
user@host:~$ echo "$TERM"
xterm-256color
user@host:~$ infocmp -I
Reconstructed via infocmp from file: /lib/terminfo/x/xterm-256color
xterm-256color|xterm with 256 colors,
am, bce, ccc, xenl, km, mir, msgr, npc, mc5i,
cols#80, it#8, lines#24, colors#256, pairs#32767,
acsc=``aaffggiijjkkllmmnnooppqqrrssttuuvvwwxxyyzz{{||}}~~,
cbt=\E[Z, bel=^G, cr=^M, csr=\E[%i%p1%d;%p2%dr, tbc=\E[3g,
clear=\E[H\E[2J, el1=\E[1K, el=\E[K, ed=\E[J,
hpa=\E[%i%p1%dG, cup=\E[%i%p1%d;%p2%dH, cud1=^J,
home=\E[H, civis=\E[?25l, cub1=^H, cnorm=\E[?12l\E[?25h,
cuf1=\E[C, cuu1=\E[A, cvvis=\E[?12;25h, dch1=\E[P,
dl1=\E[M, smacs=\E(0, smam=\E[?7h, blink=\E[5m, bold=\E[1m,
smcup=\E[?1049h, dim=\E[2m, smir=\E[4h, sitm=\E[3m,
rev=\E[7m, invis=\E[8m, smso=\E[7m, smul=\E[4m,
ech=\E[%p1%dX, rmacs=\E(B, rmam=\E[?7l, sgr0=\E(B\E[m,
rmcup=\E[?1049l, rmir=\E[4l, ritm=\E[23m, rmso=\E[27m,
rmul=\E[24m, flash=\E[?5h$<100/>\E[?5l,
is2=\E[!p\E[?3;4l\E[4l\E>,
initc=\E]4;%p1%d;rgb:%p2%{255}%%{1000}%/%2.2X/%p3%{255}%%{1000}%/%2.2X/%p4%{255}%*%{1000}%/%2.2X\E\,
il1=\E[L, kb2=\EOE, kbs=\177, kcbt=\E[Z, kdch1=\E[3~,
kcud1=\EOB, kend=\EOF, kent=\EOM, kf1=\EOP, kf10=\E[21~,
kf11=\E[23~, kf12=\E[24~, kf13=\E[1;2P, kf14=\E[1;2Q,
kf15=\E[1;2R, kf16=\E[1;2S, kf17=\E[15;2~, kf18=\E[17;2~,
kf19=\E[18;2~, kf2=\EOQ, kf20=\E[19;2~, kf21=\E[20;2~,
kf22=\E[21;2~, kf23=\E[23;2~, kf24=\E[24;2~,
kf25=\E[1;5P, kf26=\E[1;5Q, kf27=\E[1;5R, kf28=\E[1;5S,
kf29=\E[15;5~, kf3=\EOR, kf30=\E[17;5~, kf31=\E[18;5~,
kf32=\E[19;5~, kf33=\E[20;5~, kf34=\E[21;5~,
kf35=\E[23;5~, kf36=\E[24;5~, kf37=\E[1;6P, kf38=\E[1;6Q,a
kf39=\E[1;6R, kf4=\EOS, kf40=\E[1;6S, kf41=\E[15;6~,
kf42=\E[17;6~, kf43=\E[18;6~, kf44=\E[19;6~,
kf45=\E[20;6~, kf46=\E[21;6~, kf47=\E[23;6~,
kf48=\E[24;6~, kf49=\E[1;3P, kf5=\E[15~, kf50=\E[1;3Q,
kf51=\E[1;3R, kf52=\E[1;3S, kf53=\E[15;3~, kf54=\E[17;3~,
kf55=\E[18;3~, kf56=\E[19;3~, kf57=\E[20;3~,
kf58=\E[21;3~, kf59=\E[23;3~, kf6=\E[17~, kf60=\E[24;3~,
kf61=\E[1;4P, kf62=\E[1;4Q, kf63=\E[1;4R, kf7=\E[18~,
kf8=\E[19~, kf9=\E[20~, khome=\EOH, kich1=\E[2~,
kcub1=\EOD, kmous=\E[M, knp=\E[6~, kpp=\E[5~, kcuf1=\EOC,
kDC=\E[3;2~, kEND=\E[1;2F, kind=\E[1;2B, kHOM=\E[1;2H,
kIC=\E[2;2~, kLFT=\E[1;2D, kNXT=\E[6;2~, kPRV=\E[5;2~,
kri=\E[1;2A, kRIT=\E[1;2C, kcuu1=\EOA, rmkx=\E[?1l\E>,
smkx=\E[?1h\E=, meml=\El, memu=\Em, oc=\E]104\007,
op=\E[39;49m, dch=\E[%p1%dP, dl=\E[%p1%dM, cud=\E[%p1%dB,
ich=\E[%p1%d@, indn=\E[%p1%dS, il=\E[%p1%dL,
cub=\E[%p1%dD, cuf=\E[%p1%dC, rin=\E[%p1%dT,
cuu=\E[%p1%dA, mc0=\E[i, mc4=\E[4i, mc5=\E[5i,
rs1=\Ec\E]104\007, rs2=\E[!p\E[?3;4l\E[4l\E>, rc=\E8,
vpa=\E[%i%p1%dd, sc=\E7, ind=^J, ri=\EM,
setab=\E[%?%p1%{8}%<%t4%p1%d%e%p1%{16}%<%t10%p1%{8}%-%d%e48;5;%p1%d%;m,
setaf=\E[%?%p1%{8}%<%t3%p1%d%e%p1%{16}%<%t9%p1%{8}%-%d%e38;5;%p1%d%;m,
sgr=%?%p9%t\E(0%e\E(B%;\E[0%?%p6%t;1%;%?%p5%t;2%;%?%p2%t;4%;%?%p1%p3%|%t;7%;%?%p4%t;5%;%?%p7%t;8%;m,
hts=\EH, ht=^I, u6=\E[%i%d;%dR, u7=\E[6n, u8=\E[?1;2c,
u9=\E[c,
user@host:~$ tput rep 65 8 | xxd
user@host:~$

Thanks for investigating on it, @cfcs
I have no strange output in xxd when repeating a key. Just within Jackline. Btw. it also occurs sometimes when I paste something into Jackline.

@Iix Pasting should be handled by notty, at least it works pretty well for me on Linux.
I'm sorry, it's tricky for me to investigate, but I'll try to look into it next time I see someone who's got a mac.


ci: script to delete old content model

:tada: This PR is included in version 1.32.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[FUSE] @model directive is not mapped

See screenshot. The circled area should be classified as C# ("ErrorModel" should be blue) and there should be C# transition markers around the fully qualified name

Non-FUSE:

This was found from integration test failures, being worked around in https://github.com/dotnet/razor/pull/10959

https://github.com/dotnet/razor/pull/10959 introduces a tooling unit test that should fail when FUSE is enabled

@inherits is going to have the same issue too. We need to actually track where the base / model types are coming from so we can emit them with mapping information.


Create Ramdisk on T2

How to create Ramdisk for MacBook T2?

for some reason this tool even tho it support T2 ramdisk and it can make it but it doesn’t work, I was able to make ramdisk for T2 in a manual way & thats only way i got ssh to work


Move #available_claim_types to service object

As discussed this would probably better as methods on the respective model classes.


Make BBR more robust to large stdout streams.

This PR depends on #431 and replaces #426 .
We have heard reports that if a backup or restore script prints out huge amounts of data to stdout, this can cause the BBR CLI to crash.
The reason this happens seems to be that a method in the CLI was effectively buffering all the data from the script's stdout. This PR changes that method so that it streams stdout to an io.Writer, instead of caching and returning it.
Changes to files in the fakes directory can be ignored, since they are auto-generated. Each commit keeps all the tests green, and is individually motivated in the commit message.

I find the separation of the commits between Refactor: inline remote_runner.runOnInstanceWithLabel and Refactor: do not cache stdout in RunScriptWithEnv artificial and confusing.
I recon this is a personal preference and I understand why you prefer the approach you took.
By using several commits you are documenting your train of thought, but for me it made it more difficult and taxing since I was reviewing some changes in a commit, just to find out that you completely change them in the following one.


Break the dependency of :detekt-api with :detekt-psi-utils

Expected Behavior
:detekt-api shouldn't depend on :detekt-psi-utils.
Current Behavior
It does
Context
This idea comes from this comment: https://github.com/detekt/detekt/pull/7105#issuecomment-2034125369
I did a fast check and the main reason to have this dependency is because :detekt-api uses FilePath and that class is declared on :detekt-psi-utils. That have little sense, that class should be defined on :detekt-api. But once you move that class to :detekt-api a lot of :detekt-psi-utils doesn't compile because they use FilePath. So maybe the solution is to invert the dependency, I'm not sure.
Also this could help to unblock #7123.

and that class is declared on :detekt-psi-utils. That have little sense, that class should be defined on :detekt-api. But once you move that class to :detekt-api a lot of :detekt-psi-utils doesn't compile because they use FilePath

Which class are we talking about?

FilePath

I'd love to get rid of that class. We have basePath, which should be required from tooling, or have a sensible default. If other paths are absolute, we can always generate the relative path from the absolute path and basePath. No need to store all versions of the path, nor do we need to convert anything to a FilePath instead of just using a Path.
Relative paths are only needed when outputting results. Absolute paths should always be used internally.

That's true... And it shouldn't be difficult to change.

My findings:
Right now on a KtFile we store as UserDataProperty 3 different paths:

relativePath
absolutePath
basePath

For sure we don't need 3 because with 2 of them we can calculate the other. So we could reduce it to:

relativePath
basePath

But we can do it even better. basePath is ALWAYS the same for ALL the KtFiles. We don't need to store this value on every single KtFile. So we can just store relativePath and then store basePath somewhere else.
Note: because we just have one path now on KtFile we could rename it to just path. But I'm going to keep calling it relativePath here to avoid confusions.
The problem with relativePath is that it is a Path and if you call relativePath.absolute() you expect to get the absolute path of it. But that's not true if basePath != System.getProperty("user.dir"). And it is way to easy to fall on that error.
My proposal to avoid this is to change System.getProperty("user.dir") to ensure that it is always the same as basePath. This way we fixed two problems:

relativePath behaves as expected. When you call absolute it really returns the correct path.
We don't need to store basePath anywhere.

What do you think? I'm going to implement it to see how it looks like (or at least try it).

We should actually go further, but I don't think it's possible to solve while we support autocorrect.
If we created each KtFile the proper way the absolute file path would be available at KtFile.virtualFile.path and we wouldn't need to store it again. The relative path can be generated from this & the base path. This is actually how we get the absolute path with the compiler plugin:
https://github.com/detekt/detekt/blob/68b949f3ed9965a03797d2951cce68fd7dfc87a2/detekt-psi-utils/src/main/kotlin/io/github/detekt/psi/KtFiles.kt#L33-L37
Why is it available in compiler plugin, but not when we generate them? Because we use a different method to create each KtFile. detekt uses createPhysicalFile here:
https://github.com/detekt/detekt/blob/68b949f3ed9965a03797d2951cce68fd7dfc87a2/detekt-parser/src/main/kotlin/io/github/detekt/parser/KtCompiler.kt#L34-L37
While the Kotlin compiler will add the source files to the KotlinCoreEnvironment then use KotlinCoreEnvironment.getSourceFiles() which returns List<KtFile>.
Why can't we do the same? Because KtFile created by the compiler are not writeable, but files created by the KtPsiFileFactory are. And files have to be writeable for autocorrect to work, at least how it's currently implemented, otherwise we would get errors like:
Caused by: org.jetbrains.kotlin.com.intellij.util.IncorrectOperationException: Cannot modify a read-only file 'D:\IdeaProjects\detekt\detekt-api\src\main\kotlin\io\gitlab\arturbosch\detekt\api\internal\PathFilters.kt'.
	at org.jetbrains.kotlin.com.intellij.psi.impl.CheckUtil.checkWritable(CheckUtil.java:33)

Since we now say autocorrect support is not available in detekt itself, but can be implemented in the rules or rule providers themselves, we could switch to using KotlinCoreEnvironment.getSourceFiles() and say that rule implementers have to create writeable versions of files themselves if they want to support autocorrect.
I personally think that's a reasonable tradeoff.
We can easily get rid of FilePath then :D

Once my PRs are merged, it's trivial to create a PR to remove the last couple of detekt-psi-utils usages from detekt-api.
However the question becomes, do we want to keep it as an api dependency on detekt-api? Removing it means adding it as a dependency to nearly every detekt rule set, but it seems more correct to remove it.

do we want to keep it as an api dependency on detekt-api?

No, we don't want to keep it as a dependency. It's better to declare it on every rule module.

do we want to keep it as an api dependency on detekt-api?

No, we don't want to keep it as a dependency. It's better to declare it on every rule module.

Agree on this 👍 I'd rather have explicit dependency on each ruleset rather than having api dependencies around


Panic with invalid memory address

I was running some load testing comparisons between the different golang oracle drivers (rana/ora, mattn/oci8, and goracle), and the rana/ora driver gave the following panic multiple times.
I have not had time to isolate which query/update/insert was causing this.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x56280f]
goroutine 8954 [running]:
github.com/xxx/vendor/gopkg.in/rana/ora%2ev4.(*Ses).Break(0xc4203e82c0, 0x0, 0x0)
	/go/src/github.com/xxx/vendor/gopkg.in/rana/ora.v4/ses.go:809 +0x1af
github.com/xxx/vendor/gopkg.in/rana/ora%2ev4.(*DrvStmt).ExecContext.func1(0xc420024a80, 0xc7eee0, 0xc421034a20, 0xc42000fa18)
	/go/src/github.com/xxx/vendor/gopkg.in/rana/ora.v4/drvStmt_go1_8.go:39 +0x1a7
created by github.com/xxx/vendor/gopkg.in/rana/ora%2ev4.(*DrvStmt).ExecContext
	/go/src/github.com/xxx/vendor/gopkg.in/rana/ora.v4/drvStmt_go1_8.go:30 +0x2b8

That's the Break after context cancelation.
Chris Duncan<EMAIL_ADDRESS>ezt írta (időpont: 2017. nov. 20.,
H, 18:18):

I was running some load testing comparisons between the different golang
oracle drivers (rana/ora, mattn/oci8, and goracle), and the rana/ora driver
gave the following panic multiple times.
I have not had time to isolate which query/update/insert was causing this.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x56280f]
goroutine 8954 [running]:
github.com/xxx/vendor/gopkg.in/rana/ora.v4.(*Ses).Break(0xc4203e82c0, 0x0, 0x0)
/go/src/github.com/xxx/vendor/gopkg.in/rana/ora.v4/ses.go:809 +0x1af
github.com/xxx/vendor/gopkg.in/rana/ora.v4.(*DrvStmt).ExecContext.func1(0xc420024a80, 0xc7eee0, 0xc421034a20, 0xc42000fa18)
/go/src/github.com/xxx/vendor/gopkg.in/rana/ora.v4/drvStmt_go1_8.go:39 +0x1a7
created by github.com/xxx/vendor/gopkg.in/rana/ora.v4.(*DrvStmt).ExecContext
/go/src/github.com/xxx/vendor/gopkg.in/rana/ora.v4/drvStmt_go1_8.go:30 +0x2b8
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rana/ora/issues/234, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPoSoFS98NVuL8p9fhFyuUxZZqfWjUMks5s4bRjgaJpZM4Qkmwt
.

Shouldn't a break just return an error, not panic?
Also, I'm not cancelling any contexts until after the queries return (with defer).  Most likely there is a race condition just like the other 2 drivers had.

PR above to solve the race condition: https://github.com/rana/ora/pull/235

Thanks!


typo fix several files

Fixed some typo and added a new line for more clear understanding. Review it and merge it.

Thank you, could you please sign your commits the next time 🙂 here is more info https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

Thank you for teaching me about signing commits. 😀 I will sign commits from the next time.


Google now accepts zips with a key in their manifest

http://www.rubydoc.info/gems/extensionator/

--inject-key. If you have an extension in the store, you know you can't have a "key" property in your manifest file

I know this used to be the case but it no longer appears to be true - I just uploaded a zip with a key in its manifest and no errors were thrown.

Oh cool. Good to know. I'll update the docs and deprecate the option.

Added a note to the Readme about this and other important-to-know changes about Chrome


rich text generate no param field

{
  "myText(rich)": "Welcome $name. Please click ${underline(here)}!"
}

flutter pub run slang

	InlineSpan get myText => TextSpan(children: [
		const TextSpan(text: 'Welcome \$name. Please click \${underline(here)}!'),
	]);

there is no parameters for myText.

Hi. What is your string_interpolation configuration? It seems that you should use {name} or {{name}} instead of $name

Hi. What is your string_interpolation configuration? It seems that you should use {name} or {{name}} instead of $name

thanks, i change to double braces string_interpolation and then fix it


Update npm registry

Thanks for the great project! Could you publish latest repository to the npm registry? I found that some recent changes (like https://github.com/PrismJS/prism-themes/pull/64) are not in npm.

@Golmote @zeitgeist87 I wonder if we can have automated releases in this one, since none of us is that focused on this repo? E.g. every N merged PRs…

It's a nice idea but I have no clue how to do that.

@LeaVerou We're currently hitting an issue which could be resolved by having an up to date version of prism-themes hosted on the npm registry
https://github.com/conorhastings/react-syntax-highlighter/issues/95
https://github.com/conorhastings/react-syntax-highlighter/pull/96
Mind if I start poking around with automated releases using something like Travis CI ?
https://travis-ci.com/

Thank you!
We do the same for the normal Prism repo too :)

Ah, I see, fantastic!
btw you have what looks to be an api secret key in your travis.yml file (main prism repo) maybe it's on purpose ? 😧

btw you have what looks to be an api secret key in your travis.yml file (main prism repo) maybe it's on purpose ? 😧

Wow, that's really weird. And git blame says I did it. 😕
I'm pretty sure I just followed some tutorial from somewhere.
@apfelbox do you remember anything related to this?

Although hold on... it looks like that might be ok after all
https://docs.travis-ci.com/user/environment-variables/#Defining-encrypted-variables-in-.travis.yml

@LeaVerou nope, sorry. We only installed the testing side of Travis back then, the deployment part came later and I wasn't involved with that.
But as @GeeWizWow already found out there is nothing wrong with your .travis.yml: the secure: part tells you, that you are indeed using Travis' encrypted variables – so you actually encrypted the variables with a secret only you and Travis know (and without this secret the key can't be decrypted).

Days since spooking people for nothing: 0
As far as i can tell, the steps for enabling automatic deployment in this
repo would be to commit the same .travis.yml file as in prism here and to
change the repo name in the deploy section.
The only item i'm unsure about is if the encrypted env variable is account
or project specific.
On Thu, 14 Dec 2017 at 17:37 Jannik Zschiesche<EMAIL_ADDRESS>wrote:

@LeaVerou https://github.com/leaverou nope, sorry. We only installed
the testing side of Travis back then, the deployment part came later and I
wasn't involved with that.
But as @GeeWizWow https://github.com/geewizwow already found out there
is nothing wrong with your .travis.yml: the secure: part tells you, that
you are indeed using Travis' encrypted variables – so you actually
encrypted the variables with a secret only you and Travis know (and without
this secret the key can't be decrypted).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PrismJS/prism-themes/issues/66#issuecomment-351782940,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIlRkrWAxKnZAi4KINwIHrkPdDjyHFfTks5tAVzxgaJpZM4KrYvs
.

Been a little while, but just published v1.1.0!


verify contract fail

Use hardhat to verify the contract. When the verified contract method has construction parameters, the verification will fail and return Unable to verify
code is very simple
`// SPDX-License-Identifier: MIT
pragma solidity >=0.6.0;
contract test  {
string public name;
constructor(string memory _name){
    name = _name;
}

function add(uint256 a, uint256 b) public pure returns (uint256) {
    return a + b + 1;
}

}`

why remove my issue? @vbaranov

It is not removed - it has been transferred to the repository, where the smart-contract verification service is developed. Stay tuned - someone will look into your issue soon.

hello @wzglgo, could you describe the way you are trying to verify the contract using hardhat in more detail?


Nested symbols in different libraries

If you have nested symbols that you want to send to different libraries (e.g. components that contain icons, and you want a separate Components and Icons library), it might be worth pointing out that the process outlined in the Readme wouldn't quite work. When you copy your components symbols out of your original file, you'll bring with them the icon symbols that they contain, which would create duplicates (the same icon would become two symbols, one in Components and one in Icons).
To remedy that you can either:

Run the replacer plugin within your Components library first, to replace all icon symbols by icons from the Icons library. You should make sure that Components doesn't contain any other symbols than the component ones. Then you can run the plugin within your original file.
Or: run the replacer plugin in your original file after transferring out the icons, but before transferring out the components themselves. That way the components will already be linked with the Icons Library's icons, instead of the local ones. If you have circular dependencies between your libraries, this approach might not be feasible.

I believe the overrides will be preserved with either method; but who can claim to know the mysteries of Sketch?

yeah hopefully this will be slightly improved with Sketch 48, where they fixed the issue with copy-paste symbols not staying pointing to their library.
I've added your comments to the README.md and credited you 👍


Bump minimum supported node version to v16.13.2 (Gallium)

The version of Node that we had been using (v16.7.0) is listed as "Current" in the Node v16 changelog. We should be using LTS releases, which as of now includes v16.13-15.

(To clarify, I don't think we should land this right now.)

@threepointone (Yeah that makes sense)

I found two places to add for this PR:

https://github.com/cloudflare/wrangler2/blob/5d6de58a1410bd958e9e3eb4a16c622b58c1a207/packages/wrangler/bin/wrangler.js#L8
https://github.com/cloudflare/wrangler2/blob/adbdbf5f209608a5ba94db9e0601f6630f0ec415/examples/remix-pages-app/.node-version


Issue with lowercase search conflicting with vernaculars

taxon("gorilla") - add a keyword to prefer scientific names

gorilla
elephant
mouse


Update http_breaker.go

Test results more intuitive

Coverage decreased (-3.4%) to 84.071% when pulling e9f0a0b2b1225af7fa0d480a361093e91abc9ed7 on xlplbo:patch-1 into 1501d51a5ce38beb1794857981b134a246ae200b on sony:master.

Coverage decreased (-3.4%) to 84.071% when pulling e9f0a0b2b1225af7fa0d480a361093e91abc9ed7 on xlplbo:patch-1 into 1501d51a5ce38beb1794857981b134a246ae200b on sony:master.


Ability to edit the local file while in a comparison view with a previous version

Ability to view the comparison between the local file version in editor and a previous commit. Then to be able to edit the local file inside of the comparison and then save the local file.

You should be able to do this today. You can use the Compare File with Working Tree command anytime you are in a revision and it will put the revision on the left and the working file on the right (which will be editable).
Let me know if this doesn't meet your needs.


Doumented PLaceandFaceValue, PythagorasTheorem, ProfitLoss

Issue Id you have worked upon -
#311
Briefly explain your program logic -
Kindly write your answer here
Screenshots(Attach 2 screenshots of your own input and output) -

By raising this PR I affirm that -

[x] My code follows the guidelines of this project.

[x] I have performed a self-review of my own code.

[x] I have commented my code.

[x] My code gives the correct output.

[x] I affirm that I strictly follow contributing guidelines and code of conduct.

Congratulations 🎉 @DanishGBU on your contribution.
Thanks for adding value to this repository. 😊
Let's celebrate this fest 😃 by providing more developers 👨💻👩💻 an opportunity to contribute. Help us make this repository reach far and more by sharing this to your friends and colleagues.
Your efforts can make difference.


Year in CHANGELOG is wrong

Cosmetic issue, for sure. But it seems that the dates in the CHANGELOG are being copied, and using the previous year.

Here's a super simple PR to resolve this issue:
https://github.com/johnpryan/flutter_map/pull/511


Remove unreachable code in CustomShellCommands.cs

PR Summary
Fix #13248
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@xtqqczze please resolve merge conflicts

@rjmholt resolved

@PoshChan please remind me in 1 hour

@rjmholt, this is the reminder you requested 1 hour ago


MVVM UDF DataBinding

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy


Bugfix/es6 import basic modules

This should resolve issue #91 where the imported file isn't the package base (ie is a import of a JS file).
Upgraded to latest esm-export as well.

Good catch! Feel free to fix it
On Thu., 15 Feb. 2018, 5:24 pm Nikita Gusakov<EMAIL_ADDRESS>wrote:

@nkt commented on this pull request.
In src/utils/export-module-completion.js
https://github.com/nkt/atom-autocomplete-modules/pull/95#discussion_r168395166
:

@@ -6,7 +6,8 @@ const fs = require('fs');
const PATH_SLASH = process.platform === 'win32' ? '\' : '/';

const escapeRegExp = require('lodash.escaperegexp');
-const { parseModule, parseFile } = require('esm-exports');
+const parseModule = require('esm-exports').module;
+const parseFile = require('esm-exports').file;
const { module: parseModule, file: parseFile } = require('esm-exports');
=)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/nkt/atom-autocomplete-modules/pull/95#pullrequestreview-96753231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKXAhiSVtLFU78FcI64p2nUc7k6aozpjks5tU9uOgaJpZM4SGLtD
.


Allow to configure image GC thresholds for kubelet configuration in Shoot API

How to categorize this PR?

/area storage scalability
/kind enhancement
What this PR does / why we need it:
Allow to configure image GC thresholds for kubelet configuration in Shoot API.
/squash
Which issue(s) this PR fixes:
Part of #4281
Release note:

It's now possible to configure the `imageGC{High,Low}ThresholdPercent` fields for the kubelet configuration (defaults: `50` for the high threshold, `40` for the low threshold) in the `Shoot` API via `.spec.{provider.workers[].}kubernetes.kubelet.imageGC{High,Low}ThresholdPercent`.

@rfranzke  Shall we get this in for v1.26?

@timebertt it can also wait for v1.27, no issues

BackupBucket envtest is failing. Can you rebase onto https://github.com/gardener/gardener/pull/4265?