Knshq reformatting 2

Running your branch, I immediately got a "Render Error". Please review comments made in render view, ensuring they are correctly formatted comments utilising React Native's proper methodology.
You can read more here:
https://github.com/airbnb/javascript/tree/master/react
A conflict also needs to be corrected before I can merge this into the main branch. If you fix what I have mentioned, I will be happy to review it again and see if everything works as expected.

I have corrected the errors.

Fixed all known issues and tested the app for any potential problems.


Run tooling in module mode by removing GO111MODULE=off

Best reviewed per commit, GH hides diffs to autogenerated files, and they're all one-liners.
Running make generate-k8s-api locally mysteriously fails:
import "/home/timo/github/cilium/cilium/vendor/k8s.io/code-generator/cmd/go-to-protobuf":
cannot import absolute path

Since go modules, using go run from within a Go module will automatically select and build the version of the tool based on the contents of go.mod. This makes scripts that call 'go run' on a local package redundant in most cases. This also enables module mode for tools that support it. ./generate-groups.sh currently does not and is therefore run with Go modules disabled.
Move the GOPATH check into the Makefile to prevent littering GOPATH with generated files if the repo is checked out outside GOPATH and make generate-k8s-api is run. Make setting GOPATH and GOBIN explicit and only for the duration of the task.
Run tooling in module mode by removing GO111MODULE=off

A similar change to go vet and go list has been proposed here: https://github.com/cilium/cilium/pull/21560.

Thanks for the reviews, marking this ready.


">" characters don't translate well to presets

PPT>SCR renders as

Thanks for this, I've pushed a fix but unable to test, could you test? I'll update in a few minutes

Did you test?

close due no response


Simple Options workshop documentation

First Merge https://github.com/CosmWasm/cosmwasm-examples/pull/92
Resolves https://github.com/CosmWasm/docs/issues/100

First iteration. I will go over it once more.

There are some code references to example/simple-option master branch which is under development.

I will merge https://github.com/CosmWasm/cosmwasm-examples/pull/95 and cut a release
Change simple-option master branch references to the tag
And then merge this branch


Pathfinder-necros: Fixes skills

Added armor check penalty to all skills that required it
Removed armor check penalty from the Religion consolidated skill
Added damage type to the default attack rolls
Exposed the skills macro for client modification and special use cases. This should make customizing the sheet for special class skills much, much easier.

There was a merge conflict and vince and I have been merging our stuff together behind the scenes so he had my changes in his pull request anyways.


Add new filtering modes

Git has gone crazy, squash merge this and i'll reset my local repository

I've implemented this myself ;)


Segmented image disappears in a split second

Whenever i run segement anything, it does work exactly like it should but the image on right side only comes up for some milliseconds and then it's gone. I did reinstall the extension, disabled all other extensions and restarted webui but still same problem. Any solution?

In the past, there have been instances where the sketch function did not function properly when privacy protection extensions were enabled in browsers like Google Chrome and Microsoft Edge. While it's uncertain if this is related to your current issue, you could try disabling these extensions as a potential solution.

The gradio package in the venv might be outdated. Consider deleting the venv folder located within stable-diffusion-webui, then launch webui.bat or webui.sh and reinstall the gradio package in the new venv.

I have fixed for the section that appeared to be causing the issue in the latest version of the code.

Thanks for the fix mate. Everything's working perfect again. Great work

Since the problem where the segmentation image disappears momentarily had been fixed, this issue will be closed.


Incorrect regex

During discovering project scripts, i noticed that this part is likely to be incorrect. What we want is to exclude files that end with index, matrix or skeleton.
But regex excludes files that end with a character that appears in previously mentioned strings instead. For example, it excludes filenames like wwwi.xml, because it ends with i, which appears in index. Or yyyx.xml, because it ends with the same character matrix ends.
I believe it is not the expected behavior.
I'll open PR in advance with the corrected regex, which will exclude files that end with  index, matrix or skeleton. names.

@yegor256 what do you think?

pr has merged, closing issue


[Feature request]  Add Prefix on Feature Flags, namespacing, and import/export

Hello,
Could you add the prefix on feature flags ? so it's consistent with the way it's working on normal configuration.
Also it's not possible to have this kind of feature flag  MyApp:featureFlag since the : is not allowed in the name of the feature flag
Thank you

Hello @riaadh. A few questions to make sure I know what you're looking for. Are you using feature flags with one of our client libraries? Is this the type of prefix you are using "MyApp:"?

Hello @riaadh. A few questions to make sure I know what you're looking for. Are you using feature flags with one of our client libraries? Is this the type of prefix you are using "MyApp:"?

Hello,
I am using azure app configuration ( key values and feature flags) with c# asp.net core
My problem is there is no consistency in the azure portal and in the SDK between keys and feature flags
I explain: when using key I structure my keys like this (the way the documentation recommend)
AppName:Service1:ApiEndpoint
AppName:Service2:ApiEndpoint
In the SDK for my application I select all keys with the prefix AppName I trim the keys so the application is aware only of the last part of the key example  Service1:ApiEndpoint
But in the feature flags first we don't have prefixes  and also the : is not allowed so I have to create feature flags like this
AppName_Service1 enabled
I hope it's clear.
Thank you

That helps a lot, thanks!

Hello,
Now i am facing also another issue when i export my configuration. Since there is no Prefix (namespacing) in the Feature Flags they are not exprted.
there is a lot of incoherance in the feature flag functionality, in .net Core it's already namespaced as "FeatureManagement": { } in the app setting file, in the azure portal it's on .appconfig.featureflag/
Please harmonize the behaviour, make feature flag on the same namespace in both appsetting.json and in the azure portal, allow  Prefix, and allow export of all config + feature flags on the azure portal
thank you

To summarize :
Allow Prefix  to Feature Flags
Allow SDK to select feature flags by Prefix
Consistent namespace for Feature flags between .net core and azure portal (under FeatureManagement like .net core)
Allow import/export for the feature flags in the azure portal  the same way for config (the namespace will define if it is a config or a feature flag)
Thanks

Hello @riaadh, we released the ability to add prefix on feature flags using Azure CLI in version 2.27.0.
With this change, the feature flag prefix can be directly added to the key-value key in AppConfig. The feature name can be different from the key and doesn't need to be namespaced.
For example, if your feature name is "Beta" and you want to namespace feature flags like "AppName:Beta", you can run the following command to create this feature flag with your custom key:
az appconfig feature set -n <AppConfigName> --feature Beta --key .appconfig.featureflag/AppName:Beta
Then in your code, you can select all feature flags starting with "AppName:" and you don't need to trim anything from the feature name. Please see FeatureFlagOptions.Select API to learn how to select feature flags beginning with a specific prefix.

@avanigupta The Azure dashboard UIs don't seem to know about that yet?
I created your example feature with a path string separated by :
When I use the FeatureManager UI to add labels to it. It'll create "new" features without the path (only named Beta) 😕

@avanigupta thank you for this change, but I confirm what @MichaCo  said the azure dashboard doest not allow yet

This is a first step for harmonizing feature flags and config.
Could you have a look at this 2 points

Consistent namespace for Feature flags between .net core and azure portal (under FeatureManagement like .net core)
=> in .net core app setttings file we add flags under the section FeatureManagement in the JSON file, but in azure we see flags under the section .appconfig.featureflag, if it could be the same between .net core and azure app config i think it will be easier

Allow import/export for the feature flags in the azure portal the same way for config (the namespace will define if it is a config or a feature flag), son one export can export everything (config, keyvault reference and feature flags)

Thank you

@riaadh, @MichaCo, your understanding is correct. This is not available in Azure portal yet but we are actively working on providing the same functionality in the portal too. In the meantime, you can use Azure CLI to achieve namespacing for feature flags as I mentioned earlier. You will see some errors in Azure portal, but your feature flags can still be accessed via any ConfigProvider/SDK/CLI/REST APIs. Once we release this functionality in portal, the errors will automatically disappear.
About consistent namespacing and exporting all data in AppConfig to a single file, we are working on providing import/export of feature flags using Azure portal. In the meantime, you can try using Azure CLI to import/export all AppConfig data (including feature flags) to a single file. Our import/export functionality understands the feature flag file schema and appropriately imports/exports all feature flags under the FeatureManagement section in file. Please see az appconfig kv export or az appconfig kv import for more details on command usage.

The support for feature flag namespace/prefix has been added to the Azure portal for a while, but we forgot to update this thread. You may see it already.

Regarding exporting all data in App Configuration to a single file, the support is added in Azure CLI. Please check out the details from https://github.com/Azure/AppConfiguration/issues/552#issuecomment-964584196. We are working on official documentation.


DNSCrypt Service sometimes isn't starting

Greetings! Sometimes, like 1/10 startups, the DNSCrypt Service just won't start on its own. Don't know why, don't know how. All I know is that sometimes my connection just won't work until I manually start the service.
Write what you need and I'll supply you with whatever info you need.

Confirm. In my case 2/4 at this time. There is no error logs.

I confirm. There is such a problem. The service does not always start.

I can't start it as well.

I keep trying the hinting the starting button but nothing happened

start WindowsPower Shell as administrator:
cd "C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy"
.\dnscrypt-proxy.exe

Are any errors displayed there?
Which operating system is used?

Windows 10 Pro lastest release

@Rictusempra
First Command: cd "C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy"
Press ENTER
Second Command: .\dnscrypt-proxy.exe
Press ENTER

No error (all [NOTICE] messages) this time but I still failed to start the service

@Rictusempra
cd "C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy"
.\dnscrypt-proxy -service install
.\dnscrypt-proxy -service start

3 Commands :)

working this time, thx.
But why not working on Simple DNSCrypt itself?

@Rictusempra i can`t say. Maybe the service was not installed correctly.
Is this the first version you have installed?

Is this the first version you have installed?

Sure

After auto update to 2.0.6, the service can't start.

.\dnscrypt-proxy -service install

.\dnscrypt-proxy -service start

Any comment ?
W10 Pro x64 with Bitdefender Antivirus.

Try going into regedit and manually delete HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\EventLog\Application\dnscrypt-proxy

@lltechguyll

The system cannot find the file ....

is there a filename?

My problem caused by the Antivirus. It detected below suspicious file and automatically put it into quarantine zone which make the installation corrupted.
Gen:Variant.Graftor.471159
If I deactivated the AV and do installation, adding DNScrypt into whitelist, then the program work like a charm. Definitely can't install when AV is active.

@lltechguyll https://github.com/jedisct1/dnscrypt-proxy/issues/205

@bitbeans I don't have an AV and this still happens for me.

I also confirm this problem. Tested on Windows 10 1709 x64 and Windwos 7 SP1 x64.
Occurs at system startup or after waking up from sleep with an approximate frequency 1/10.
Used AV: Comodo Internet Security <IP_ADDRESS>14.
System Log:
Error	Service Control Manager	7009
A timeout was reached (30000 milliseconds) while waiting for the DNSCrypt client proxy service to connect.

Error	Service Control Manager	7000
The DNSCrypt client proxy service failed to start due to the following error:
The service did not respond to the start or control request in a timely fashion.

I don't know much about Comodo Internet Security, or about AVs in general, but they include a copy of an older dnscrypt-proxy version in some of their products. Maybe both are fighting to acquire the same port number after a wakeup.

Hi, I'm also experiencing this problem.
It's been about 2 weeks since I updated to the new interface and I encountered this about 3-4 times.
I never encountered this problem with the old interface.
Should I downgrade?
Windows 10 64 bit 1709

I have the same problem.

https://github.com/jedisct1/dnscrypt-proxy/issues/234
Same issue here and a solution

Why was this closed?
It's for sure an issue.
On 64bit Windows 10 1709:

C:\Program Files (x86)\bitbeans\Simple DNSCrypt\dnscrypt-proxy>dnscrypt-proxy -service start
[2018-04-02 19:23:43] [NOTICE] Source [https://download.dnscrypt.info/resolvers-list/v2/public-resolvers.md] loaded
[2018-04-02 19:23:43] [NOTICE] dnscrypt-proxy 2.0.4
[2018-04-02 19:23:43] [NOTICE] Service started

Yet service is NOT running and Event Viewer > System shows:

The DNSCrypt client proxy service terminated unexpectedly.

After my first reboot after installation, my service did not start either. The event log reveals the following error:

The DNSCrypt client proxy service failed to start due to the following error:
The service did not respond to the start or control request in a timely fashion.

I just want to report about what happened since my last post here.
I downgraded to 0.4.3 and I never get this service not starting problem since then.
So I'm sure something is wrong with the new version not my computer.
One thing I noticed when using new version, the server list was empty when the service failed to start. I think maybe the old version worked because it stored the server list somewhere before downloading new list?

Hi, I also experienced problems when my client upgraded to a newer version.
In my case the problems were caused by EMET (on Windows 10 and on Windows 7).
I had configured a rule for "dnscrypt-proxy" in EMET and the newer client doesn't like the EAF protection.
You don't have to delete the complete rule in EMET, just disable EAF.
If you have the latest version of Windows 10,then EMET has been replaced with Exploit Guard, interestingly I had no issues with Exploit Guard, although it also uses EAF.
Finally, my antivirus-software (emsisoft) flagged (the behavior of) dnscrypt-proxy as malicious. Not sure what the new client does so differently, but it seems to trigger responses from anti-virus software and you have to keep an eye on your anti-virus software too.

Correction: I have an issue on Windows 10 now, but my best bet is that it is caused by my configuration in Exploit Guard... will create a rule for dnscrypt-proxy (without EAF at least) to see if that solves my problem.

Two days with disabled service
2018-04-13 20_11_53_Error.log
2018-04-16 22_40_39_Error.log

[FIX?]
Adding references fix DNScript service startup problems on my system. Hope, it is not only coincidence.
You can simply do it in CMD:
sc config dnscrypt-proxy depend= Winmgmt

Is there any update on this?  Its quite annoying having to manually start the service everytime I reboot.

On my system, after adding dependencies as above, everything works fine since month.
BTW: You can check new version service: https://github.com/jedisct1/dnscrypt-proxy
BTW2: SimpleDnsCrypt project looks bad - almost 2 months without any update, with such critical bugs like non starting service, errors in saving configuration? Abandonware?

I am also facing the same problem. Every once in a while, the service does not start automatically after reboot and I have to start it manually. I would expect this could be because the service tries to start before the network connection is up and hence, fails. I came across this commit for dnscrypt-proxy:
https://github.com/jedisct1/dnscrypt-proxy/commit/ce62981c44913f59a0d638397edf17ebd7da27bb
I am going to test the new version and see if it solves the problem.

@p0k33m0n The last commit for Simple DNS Crypt was 2 days ago
The last commit for DNSCrypt-Proxy was 4 days ago
I don't think either of them are abandoned. I agree that this should be a higher priority, but we have to remember that the developers (most likely) have lives outside of developing two free programs.

Since updating to a newer version of dnscrypt-proxy manually with the fix I mentioned above, I have only encountered once in over 10 reboots that the DNSCrypt service failed to auto-start. I recommend people suffering from this issue to update dnscrypt-proxy manually.

You can use SERVICES.MSC to edit the proxy service and ask Windows to restart it (up to 3 times) should it fail to start the first time. What I did is I added this line to the proxy-install script: sc failure dnscrypt-proxy reset= 3600 actions= restart/60000/restart/60000// which configures it to restart up to 2 times...

I've encountered this issue too where DNSCrypt doesn't start on boot. I also experienced the same issue with WSL and the suggested solution was to turn off fast boot; this temporary fix works for DNSCrypt as well. However, I still experience the same issue whenever I wake up my laptop that has automatically gone into sleep mode. It's kindda frustrating because Windows Hello doesn't work without Internet.

is there a problem with the pre-release version? eg: https://simplednscrypt.blob.core.windows.net/deploy/SimpleDnsCrypt/bin/x64/SimpleDNSCrypt_x64.zip (dnscrypt-proxy 2.0.14)

@bitbeans I have a problem with the pre-release version on Windows 10.
The blacklist.txt seems to be not effective anymore. In other words the blacklist feature doesn't work anymore, but is was working before.
Otherwise the new version is working:
Service starts even when I have done a complete shutdown.
Domains appear in my OpenDNS stats.

I can confirm that my issue has been resolved with the latest update. The service now starts properly even if I turn fast boot on. I'm currently using Simple DNSCrypt 0.5.5 (x64) [dnscrypt-proxy 2.0.14]. Thanks!

I'm having difficulties using Simple DNS Crypt Proxy for Windows 10. I keep clicking on the service and it won't work. I went on Windows Powershell and typed "C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy"
.\dnscrypt-proxy.exe and that didn't help.

After update to DNSCrypt 0.5.5 (x64) the problem still occurs. The Fast Boot is off.
I noticed that the dnscrypt-proxy service does not start in the scenario when the system boots without any active network connection and even a manual attempt to start the service ends in failure:
PS C:\Windows\system32> ipconfig

Windows IP Configuration

Wireless LAN adapter Wi-Fi:

   Media State . . . . . . . . . . .  : Media disconnected
   Connection-specific DNS Suffix  .  : Media disconnected
   Connection-specific DNS Suffix  .  : Media disconnected
   Connection-specific DNS Suffix  . :

PS C:\Windows\system32> net start dnscrypt-proxy
The service is not responding to the control function.

but after establishing a connection (in this case Wi-Fi), the service can be started:
PS C:\Windows\system32> ipconfig

Windows IP Configuration

Wireless LAN adapter Local Area Connection* 1:

   Media State . . . . . . . . . . . : Media disconnected
   Connection-specific DNS Suffix  .  : Media disconnected
   Connection-specific DNS Suffix  .  : Media disconnected
   Connection-specific DNS Suffix  .
The DNSCrypt client proxy service was started successfully.

Tested on Windows 10 Pro x64 1803

The problem still occurs from time to time. I can't narrow down the exact
cause though.
On Tue, Jun 5, 2018 at 1:15 AM d3bil0x<EMAIL_ADDRESS>wrote:

After update to DNSCrypt 0.5.5 (x64) the problem still occurs. The Fast
Boot is off.
I noticed that the dnscrypt-proxy service does not start in the scenario
when the system boots without any active network connection and even a
manual attempt to start the service ends in failure:
PS C:\Windows\system32> ipconfig
Windows IP Configuration
Wireless LAN adapter Wi-Fi:
Media State . . . . . . . . . . .  : Media disconnected
Connection-specific DNS Suffix  .  : Media disconnected
Connection-specific DNS Suffix  .  : Media disconnected
Connection-specific DNS Suffix  . :
PS C:\Windows\system32> net start dnscrypt-proxy
The service is not responding to the control function.
but after establishing a connection (in this case Wi-Fi), the service can
be started:
PS C:\Windows\system32> ipconfig
Windows IP Configuration
Wireless LAN adapter Local Area Connection* 1:
Media State . . . . . . . . . . . : Media disconnected
Connection-specific DNS Suffix  .  : Media disconnected
Connection-specific DNS Suffix  .  : Media disconnected
Connection-specific DNS Suffix  .
The DNSCrypt client proxy service was started successfully.
Tested on Windows 10 Pro x64 1803
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bitbeans/SimpleDnsCrypt/issues/251#issuecomment-394430709,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAH2UWJJs-aOHlhrK6QFlywKSDhVjkz6ks5t5Ws5gaJpZM4SZJCe
.

@bitbeans
Would it be worth creating a scheduled task, as this will automatically start the service at start up or at login, all without having to do much extra work or debugging.
Also, it has built in options for what to do if the service fails to start.

@zero77 Yes, the working temporary solution - the task triggered by the event as follow in the Task Scheduler:

@d3bil0x
Yes exactly although, I meant for it to be created automatically when Simple DNS Crypt is installed and not for users to create a task themselves.

has anyone tried 0.5.6 x64 on Windows 10 1803? This release resolved the issue for me. It might not have fixed it for others though.

@gashtal I tested 0.5.6 x64 on Win10 1803 and the problem still occours.

Same here on Windows 10 Pro N 1803. The service didn't start this morning :(

Seeing this too,  latest version on W8:

start Windows w/ cable unplugged (so no internet)
connect cable after a while
can't resolve anything
open the app...see that the service is stopped
start the service
everything is ok

Starting with cable plugged works ok as expected

It was happened to me too. Start with @licaon-kter  actions, I found that the service will start automatically and running smooth in condition; "the computer was connected either to wired network or WiFi before I logged in to Windows."

I have the same issue and I see the following logs in event viewer
A timeout was reached (30000 milliseconds) while waiting for the dnscrypt-proxy service to connect.
Or
The dnscrypt-proxy service failed to start due to the following error:
The service did not respond to the start or control request in a timely fashion.
What I did was add DHCP Client service as dependency for the DNSCrypt client proxy service using sc config:
sc config dnscrypt-proxy depend= Dhcp
If this doesn't work I can just remove it by issuing:
sc config dnscrypt-proxy depend= /
So far I've rebooted twice and everything seems to be working just fine. This was tested on Windows 10 x64 build 1803. Maybe others can test this and report back if it does indeed work for them or not?

I am having this exact issue, on Win 7 Pro 64-bit.
I did a workaround that seems to be working - see my post at https://github.com/bitbeans/SimpleDnsCrypt/issues/356
But I really hope this gets fixed because I'd like to recommend this to friends.

I'm having the same issue. Using Windows 10 64bit Enterprise Edition.
Upon restart probably 70% of the time I have to manually go in to simple dnscrypt and start dnscrypt service. I have tried a different fallback resolver other than the default, as the error log is stating issues with resolving raw.githubusercontent.com a copy of this is below:
2019-01-03 14:22:37.2119 ApplicationUpdater System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.WebException: The remote name could not be resolved: 'raw.githubusercontent.com'
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.Http.HttpClientHandler.GetResponseCallback(IAsyncResult ar)
   --- End of inner exception stack trace ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Net.Http.HttpClient.<FinishSendAsyncUnbuffered>d__59.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Net.Http.HttpClient.<GetByteArrayAsyncCore>d__30.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at SimpleDnsCrypt.Helper.ApplicationUpdater.<DownloadRemoteUpdateFileAsync>d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at SimpleDnsCrypt.Helper.ApplicationUpdater.<CheckForRemoteUpdateAsync>d__1.MoveNext()

I have also tried netprobe timeout = 0 as has been suggested and it doesn't help. I have the following error that shows up in event viewer:

I just noticed that my router is setup to: "Use internal DNS - Allow dnsmasq to be your DNS server on LAN" so I disabled that, maybe that was causing an issue upon startup. I'll see if this makes any difference.

@bitbeans is there any work being made to fix this? I think Simple DNSCrypt is an amazing program and it used to be a must-have-install on my computer. Now it's pretty much useless because of this, since it's completely ruining an otherwise great program.

Are you on the latest version?  I have not had this problem since upgrading to the latest version.

Dyras and cs96and -- I agree with Dyras - this is a great program but it needs to be 99.9% foolproof before I can recommend it to friends and family.
I am on the latest version (Simple DNSCrypt 0.6.2 with dnscrypt-proxy 2.0.19) and I think it's better, but occasionally - maybe one out of ten reboots - there is an issue and my PC does not connect to the internet.  There is sometimes an odd Stop-Start-Stop with the dnscrypt service.
I am still running my workaround that seems to help - see my post at #356 #356.
But not yet at 99.9%.
Thanks,

I can say that after performing the above described strategy- this is finally solved for me. By the way I set the DNSCrypt client proxy service in windows to manual. Dnscrypt is now working freaking awesome and not once has it failed to start over at least 50 restarts.

I believe this problem is caused by DNSCrypt proxy attempting to start before a network interface is available.  I made the service dependant on network services being available and all problems seem resolved.  The easiest way of doing this is sc.exe config dnscrypt-proxy depend=nsi from a command line.  I've not had any problems since.
@PhAlAnXcOiN can you try this instead of your workaround?  What you did with task scheduler definitely shouldn't be the main way of fixing this.

@Xalaxis fails with
[SC] OpenSCManager FAILED 1722:
RPC server unavailable.

What am I missing?

@licaon-kter Did you perhaps accidentally put a slash in the command? It looks like SC is attempting to connect to a remote machine.  Did you run the command as administrator?

@Xalaxis well, I've used \\localhostsname as the help says.
Else, without it, it would just show the help, as if the command is missing some param or I've made a typo.
As Admin, Win7 64bit.

@licaon-kter That's strange, this works fine for me with no changes in both Powershell and command prompt.
PS C:\Windows\system32> sc.exe config dnscrypt-proxy depend=nsi
[SC] ChangeServiceConfig SUCCESS
PS C:\Windows\system32> cmd
Microsoft Windows [Version 10.0.17763.379]
(c) 2018 Microsoft Corporation. All rights reserved.

C:\Windows\system32>sc.exe config dnscrypt-proxy depend=nsi
[SC] ChangeServiceConfig SUCCESS

Have you made sure you aren't using quotes around the commands?  Have you modified the service control manager in any manner?

Additional thought: Perhaps Windows modified the behavior of sc.exe in Windows 8/10+?

I'm trying the following now (using the same dependencies as the official NSI service):
[SC] ChangeServiceConfig SUCCESS
PS C:\Windows\system32> sc.exe qc dnscrypt-proxy
[SC] QueryServiceConfig SUCCESS

SERVICE_NAME: dnscrypt-proxy
        TYPE               : 10  WIN32_OWN_PROCESS
        START_TYPE         : 2   AUTO_START
        ERROR_CONTROL      : 1   NORMAL
        BINARY_PATH_NAME   : "C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy\dnscrypt-proxy.exe"
        LOAD_ORDER_GROUP   :
        TAG                : 0
        DISPLAY_NAME       : DNSCrypt client proxy
        DEPENDENCIES       : rpcss
                           : nsiproxy
        SERVICE_START_NAME : LocalSystem```

Whatever success I was getting before seems to have gone away now. The same problem has returned.

(Running Simple DNSCrypt 0.6.4 (x64) [dnscrypt-proxy 2.0.22)
Okay, I seem to have finally resolved all my problems.
The service not starting was caused by the netprobe setting.  By setting it to 0 as described by @PhAlAnXcOiN I seem to be getting 100% reliable starts now.
Windows thinking there is no network access was caused by a semi-rare quirk where some computers can't passively detect internet traffic properly.  I resolved this issue by following the instructions described under "A Fix from Microsoft (Windows 10 Fall 2017 Creators Update)".
I'll report back if I start having issues again, but this seems to fix things for me.
@licaon-kter Have you tried the netprobe setting tweak already?

On a reboot just now of Win 10 Pro 64-bit, dnscrypt-proxy did not start.  On the next reboot, it did.
Simple DNSCrypt 0.6.4 (x64) running dnscrypt-proxy 2.0.22.
This is rare, but it happens.
Unfortunately, when it does, my PC is cut off from the internet, and I can't get to it remotely to reboot it or manually start the service.
My workaround is not running on the Win 10, only on my Win 7 (where I actually shut it off - will also keep an eye there).
Oh, I wish this didn't happen - I really want to recommend it to friends and family.

Update to my comment immediately above - with Simple DNSCrypt 0.6.4 pushing dnscrypt-proxy 2.0.22, I haven't had a bad start in five days on both sides of my dual-booting 7 + 10 x64 machine.  I think my issue directly above may have been something else.
Fingers crossed.

Update to my comment immediately above - with Simple DNSCrypt 0.6.4 pushing dnscrypt-proxy 2.0.22 and Netprobe Timeout set to 0, I haven't had a bad start in five days on both sides of my dual-booting 7 + 10 x64 machine. I think my issue directly above may have been something else.
Fingers crossed.

Hello,
I am also affected by this problem and none of the suggestions here seem to help (except starting the service through scheduled tasks, which I have not tried yet). What I tried so far:

Added "-netprobe-timeout 0" to the service.
Made the service depend on rpcss and nsiproxy.

It most likely has to do with my internet connection being a bit dodgy and that my router sometimes only provides a connection after I logged in. However, the current behaviour is, in my opinion, far too dependent on a stable internet connection at startup.
Would it be possible to have the service not crash when there is no internet connection, but instead periodically poll for connections and resolving using the fallback DNS server until a connection can be made? As it is now, about 1/10 to 1/20 startups crashes.
Logfiles:
log_crash1.txt
log_crash2.txt

Just updating on my experience.  With the same two changes @psisquared2
made, I'm at about a 1/50 fail to start rate.  They're right though, that's
still too high to consider the issue fixed.
On Mon, 27 May 2019, 18:25 psisquared2<EMAIL_ADDRESS>wrote:

Hello,
I am also affected by this problem and none of the suggestions here seem
to help (except starting the service through scheduled tasks, which I have
not tried yet). What I tried so far:

Added "-netprobe-timeout 0" to the service.
Made the service depend on rpcss and nsiproxy.

It most likely has to do with my internet connection being a bit dodgy and
that my router sometimes only provides a connection after I logged in.
However, the current behaviour is, in my opinion, far too dependent on a
stable internet connection at startup.
Would it be possible to have the service not crash when there is no
internet connection, but instead periodically poll for connections and
resolving using the fallback DNS server until a connection can be made? As
it is now, about 1/10 to 1/20 startups crashes.
Logfiles:
log_crash1.txt
https://github.com/bitbeans/SimpleDnsCrypt/files/3224603/log_crash1.txt
log_crash2.txt
https://github.com/bitbeans/SimpleDnsCrypt/files/3224604/log_crash2.txt
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bitbeans/SimpleDnsCrypt/issues/251?email_source=notifications&email_token=ADCWV3AFE4OHVZM6BASGHW3PXQKQTA5CNFSM4ETESCPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWKH4HQ#issuecomment-496270878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADCWV3G7CVRPPJLIZWUANJDPXQKQTANCNFSM4ETESCPA
.

Not sure if it's the same or a different issue, but I can't get the DNSCrypt service to start at all. I get the following when I try to launch it in elevated PowerShell.

PS C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy> .\dnscrypt-proxy.exe
[2019-10-18 13:14:53] [NOTICE] Network connectivity detected
[2019-10-18 13:14:53] [NOTICE] Source [public-resolvers.md] loaded
[2019-10-18 13:14:53] [NOTICE] dnscrypt-proxy 2.0.25
[2019-10-18 13:14:53] [FATAL] listen udp <IP_ADDRESS>:53: bind: Only one usage of each socket address (protocol/network address/port) is normally permitted.

This is on Windows 10, with SimpleDNSCrypt 0.6.6.

If I change the listen address to <IP_ADDRESS>:5353 I can enable the service, but then it appears my system isn't using DNSCrypt at all. I must be missing something.

If I change the listen address to <IP_ADDRESS>:5353 I can enable the service, but then it appears my system isn't using DNSCrypt at all. I must be missing something.

did u installed/have active "internet-security" software running ?
some of those internet-security have secure connection, which then connecting/forwarding to their DNS for blocking malicious address
so first check for internet-security / firewall software

I'm pretty sure Windows will only use DNS running on the default port.

I also have random issues with dsncrypt service throug suspend/reactivate on laptop

If anyone is interested in trying an alternative, you can always check out YogaDNS.
Gonna try it out myself now.


welp the plugin is broken again

In the latest 0.29.1 update in arch , I cant compile the plugin.
these are the error logs
g++ -shared -g -fPIC --no-gnu-unique -std=c++23 `pkg-config --cflags pixman-1 libdrm hyprland` -Iinclude -DWLR_USE_UNSTABLE src/main.cpp -o dwindle-autogroup.so
In file included from /usr/local/include/hyprland/src/defines.hpp:1,
                 from /usr/local/include/hyprland/src/Window.hpp:3,
                 from src/main.cpp:3:
/usr/local/include/hyprland/src/includes.hpp:149:10: fatal error: ext-workspace-unstable-v1-protocol.h: No such file or directory
  149 | #include "ext-workspace-unstable-v1-protocol.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:26: all] Error 1

I can't reproduce this issue. How did you install hyprland, via AUR? I've compiled hyprland from commit 96d555e8, which corresponds to the git tag of v0.29.1. Building the plugin works without issues for me there. Maybe try compiling from source, and see if there's a difference in comparison to AUR.
Although from the error there, it seems like an issue originating in the Hyprland header files in /usr/local/include/hyprland. There is an include of a header file which apparently wasn't found. I can't do anything about those hyprland header files from the project's perspective though. Those are put there during hyprland installation, and should work properly. This seems like you're missing an entire header file in there.
If this issue really does persist even after installing hyprland via manual compilation, could you try doing a git bisect and finding the actual hyprland commit from which this issue appears? (i.e. check out the last commit at which things worked, so for you, probably the previous stable version, compile from that, and see if it works, if it does, run git bisect good, then check out v0.29.1, and do git bisect bad, git will now automatically check out versions to find the exact commit at which things stopped working (binary search), you can just keep using these 2 commands depending on whether the plugin compilation worked or not. After a while, git will tell you which commit was the faulty one.) From there, I should be able to at least see what change in hyprland caused this.

switching to hyprland-git fixes this issue
thanks for the response

Still it is pretty odd that you even encountered such an issue, I don't use the AUR version, so it is possible that it's something to do with how it's built, the include files are there purely for extensions afaik, so even if they're missing it shouldn't affect hyprland itself. This means that it's possible the AUR installation doesn't build them or something, maybe that's the reason. Hard to say.
Well either way, glad you were able to fix it, I'm still kinda curious why this happened though.

Re-closing the issue as can't repro, rather than completed/resolved.


Mocha does not run tests with ts-node if main entry of package.json does not exist

Prerequisites

[x] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
I have an npm package written in Typescript with a test suite using Mocha. I have set up Mocha with ts-node so that no compiled Javascript will be outputted. When running the tests, Mocha looks for the "main" entry in package.json, which points to a non-existent ./dist/index.js, and does not run tests.
Steps to Reproduce

Run npm test with the following:

File structure
| node_modules
| src
    - index.ts
| test
    - index.test.ts
- package.json
- tsconfig.json

package.json
{
    "name": "test",
    "version": "1.0.0",
    "description": "",
    "main": "./dist/index.js",
    "types": "./dist/index.d.ts",
    "files": [
        "/dist"
    ],
    "scripts": {
        "test": "mocha"
    },
    "keywords": [],
    "author": "",
    "license": "ISC",
    "devDependencies": {
        "@types/mocha": "^7.0.1",
        "mocha": "^7.0.1",
        "source-map-support": "^0.5.16",
        "ts-node": "^8.6.2",
        "typescript": "^3.7.5"
    },
    "mocha": {
        "package": "./package.json",
        "extension": [
            "ts"
        ],
        "spec": [
            "test/**/*.test.ts"
        ],
        "recursive": true,
        "file": [],
        "require": [
            "ts-node/register",
            "source-map-support/register"
        ],
        "ui": "bdd",
        "exit": true
    }
}

tsconfig.json
{
    "compilerOptions": {
        "target": "es6",
        "lib": [
            "es2017",
            "es7",
            "es6",
            "dom"
        ],
        "declaration": true,
        "sourceMap": true,
        "outDir": "./dist",
        "removeComments": true,
        "strict": true,
        "noImplicitAny": false,
        "strictNullChecks": true,
        "noImplicitThis": true,
        "alwaysStrict": true,
        "noUnusedLocals": true,
        "noUnusedParameters": true,
        "noImplicitReturns": true,
        "noFallthroughCasesInSwitch": true,
        "moduleResolution": "node",
        "baseUrl": "./",
        "esModuleInterop": true,
        "forceConsistentCasingInFileNames": true,
        "downlevelIteration": true
    },
    "typeRoots": [
        "node_modules/@types"
    ],
    "types": [
        "node",
        "mocha",
        "chai"
    ],
    "include": [
        "src/**/*.ts",
    ],
    "exclude": [
        "coverage",
        "node_modules",
        "dist",
        "test/**/*.ts"
    ]
}

src/index.ts
Empty file.
test/index.test.ts
describe('title', () => {
    it('case', () => {
        return;
    });
});

Expected behavior: Mocha runs tests without looking at the "main" entry in package.json.
Actual behavior: Mocha looks at the "main" entry in package.json, quits if the file doesn't exist and does not run tests.
Error: Cannot find module '$MY_PROJECT_ROOT/dist/index.js'. Please verify that the package.json has a valid "main" entry
    at tryPackage (internal/modules/cjs/loader.js:297:19)
    at Function.Module._findPath (internal/modules/cjs/loader.js:528:18)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:780:27)
    at Function.Module._load (internal/modules/cjs/loader.js:686:27)
    at Module.require (internal/modules/cjs/loader.js:848:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at $MY_PROJECT_ROOT/node_modules/mocha/lib/mocha.js:311:36
    at Array.forEach (<anonymous>)
    at Mocha.loadFiles ($MY_PROJECT_ROOT/node_modules/mocha/lib/mocha.js:308:14)
    at Mocha.run ($MY_PROJECT_ROOT/node_modules/mocha/lib/mocha.js:849:10)
    at Object.exports.singleRun ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/run-helpers.js:108:16)
    at exports.runMocha ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/run-helpers.js:143:13)
    at Object.exports.handler ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/run.js:305:3)
    at Object.runCommand ($MY_PROJECT_ROOT/node_modules/yargs/lib/command.js:242:26)
    at Object.parseArgs [as _parseArgs] ($MY_PROJECT_ROOT/node_modules/yargs/yargs.js:1087:28)
    at Object.parse ($MY_PROJECT_ROOT/node_modules/yargs/yargs.js:566:25)
    at Object.exports.main ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/cli.js:68:6)
    at Object.<anonymous> ($MY_PROJECT_ROOT/node_modules/mocha/bin/mocha:133:29)
    at Module._compile (internal/modules/cjs/loader.js:955:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
    at Module.load (internal/modules/cjs/loader.js:811:32)
    at Function.Module._load (internal/modules/cjs/loader.js:723:14)
    at Function.Module.runMain (internal/modules/cjs/loader.js:1043:10)
    at internal/main/run_main_module.js:17:11
npm ERR! Test failed.  See above for more details.

Reproduces how often: Every time.
Versions

The output of mocha --version and node node_modules/.bin/mocha --version: both 7.0.1
The output of node --version: v12.15.0
Your operating system

name and version: macOS High Sierra 10.13.6
architecture (32 or 64-bit): 64-bit

Your shell (e.g., bash, zsh, PowerShell, cmd): fish
~Your browser and version (if running browser tests):~
Any third-party Mocha-related modules (and their versions):

@types/mocha 7.0.1

Any code transpiler (e.g., TypeScript, CoffeeScript, Babel) being used (and its version):

tsc 3.7.5
ts-node v8.6.2
source-map-support 0.5.16

Additional Information

Compiling output with npx tsc and running npm test results in:

$MY_PROJECT_ROOT/dist/index.js:1
import { Thing } from './file';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Module._compile (internal/modules/cjs/loader.js:891:18)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
    at Module.load (internal/modules/cjs/loader.js:811:32)
    at Function.Module._load (internal/modules/cjs/loader.js:723:14)
    at Module.require (internal/modules/cjs/loader.js:848:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at $MY_PROJECT_ROOT/node_modules/mocha/lib/mocha.js:311:36
    at Array.forEach (<anonymous>)
    at Mocha.loadFiles ($MY_PROJECT_ROOT/node_modules/mocha/lib/mocha.js:308:14)
    at Mocha.run ($MY_PROJECT_ROOT/node_modules/mocha/lib/mocha.js:849:10)
    at Object.exports.singleRun ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/run-helpers.js:108:16)
    at exports.runMocha ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/run-helpers.js:143:13)
    at Object.exports.handler ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/run.js:305:3)
    at Object.runCommand ($MY_PROJECT_ROOT/node_modules/yargs/lib/command.js:242:26)
    at Object.parseArgs [as _parseArgs] ($MY_PROJECT_ROOT/node_modules/yargs/yargs.js:1087:28)
    at Object.parse ($MY_PROJECT_ROOT/node_modules/yargs/yargs.js:566:25)
    at Object.exports.main ($MY_PROJECT_ROOT/node_modules/mocha/lib/cli/cli.js:68:6)
    at Object.<anonymous> ($MY_PROJECT_ROOT/node_modules/mocha/bin/mocha:133:29)
    at Module._compile (internal/modules/cjs/loader.js:955:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
    at Module.load (internal/modules/cjs/loader.js:811:32)
    at Function.Module._load (internal/modules/cjs/loader.js:723:14)
    at Function.Module.runMain (internal/modules/cjs/loader.js:1043:10)
    at internal/main/run_main_module.js:17:11
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test: `mocha`
npm ERR! Exit status 1

Creating an empty index.js file in dist allows me to run tests with no problems.

Re "Additional Information": ts-node does not support ES modules, see mocha-examples.

Should this be closed since ts-node does support ESM modules?  Should mocha-examples be updated to reflect this as well?  How often is mocha-examples used as an example these days?

@cspotcode I opened an issue in our mocha-examples repo and added some links to your issue/sample in ts-node.


[Feature] Remove multiple frames

This PR
PR Guidelines

[X] I have read the contribution and PR guidelines

Related tickets

EDT-1187

Shouldn't we now redirect remove to the new removeFrames and completely remove the exposed removeFrame on engine-side?

agree, it could be an option. not sure what others think?

Shouldn't we now redirect remove to the new removeFrames and completely remove the exposed removeFrame on engine-side?

agree, it could be an option. not sure what others think?

Agree :+1:


Add project filter like flytectl --filter.xxx flags

Our team have many projects.
so I want to filter the project by label on the main screen like  "flytectl get project --filter.fieldSelector" command.

@kimjunil how many projects do you have - approximately? Also what should the filter do - filter by regex or prefix or maybe just run the filter on the client side?

In fact, we haven't opened Flyte across the company and we don't have a lot of projects yet.
But It's going to be open to many teams soon.
Then, each team will create multiple projects.
What we want to do the most is to label like team name on the project and filter it by the team name.

And if possible, it would be nice to use the url parameter so that we can deliver the filtered project list right away.
Like this:
https://flyte.company.com/console?filter.fieldSelector=labels.team%3Dinfra

we'll need to do some design work to figure out what the main flyte console page looks like, besides listing out all the projects


Adds new samplers and benchmarks

This introduces two options for trace sampling:

BoundaryTraceIdSampler - For high-volume sites, based on Finagle
CountingTraceIdSampler - Trades lower performance for high accuracy

These include benchmarks and both work well, although the boundary
one is an order of magnitude faster under contention.

CountingSampler is a better default, as support questions have come up with boundary tracing. A lot of people don't have much traffic and they can be surprised to find nothing being sampled at all!
This counting sampler is a thread-safe version of the old one, which gives an exact sample rate. It can do several samples per microsecond under contention, which is good enough for people getting started.
Basically, by switching to this, people can have a better experience, and save the tricky stuff for later.

@kristofa @eirslett if you don't mind.. ptal!

LGTM!


fix: add vulnerability label to dependabot PRs with alert state

Feature or Bugfix

Enhancement

Detail

add vulnerability label to dependabot PRs with alert state

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 26b39fe049f871bc7896d2a68abc93e9bb046b5d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 26b39fe049f871bc7896d2a68abc93e9bb046b5d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


cer证书申请失败

[Wed Jul 31 22:30:03 CST 2019] You didn't specify a Cloudflare api key and email yet.
[Wed Jul 31 22:30:03 CST 2019] You can get yours from here https://dash.cloudflare.com/profile.
[Wed Jul 31 22:30:03 CST 2019] Error add txt for domain:_acme-challenge.lt.hn.189.cn
楼主这个怎么解决

You didn't specify a Cloudflare api key and email yet.
https://github.com/Neilpang/acme.sh/wiki/dnsapi#1-use-cloudflare-domain-api-to-automatically-issue-cert


Cocopods integration seems broken, not fully implemented

After running the cocoPods install and opening my workspace, I see nothing but compile errors attempting to use authorize-me

var provider: Provider? ... Use of undeclared type 'Provider'

provider = TwitterSystemProvider() .. Use of unresolved identifier 'TwitterSystemProvider'

and on and on and on.  I am sure this can be fixed by me with adding to build path or something, but plenty of cocoPod projects do not require that.

Explain how to fix please
Implement a permanent fix in your code please

BTW...  Authorize me is in my search paths...

$PODS_CONFIGURATION_BUILD_DIR/AuthorizeMe

I think i just figured this out.  The name of my app was "AuthorizeMe".  Guess you should add to your wiki that people should NOT name their app that.. even a test app

This can be closed.  USER ERROR


Adding a new parameter to the database (even if not used) breaks code.

Adding a new parameter to the database, even if not used, breaks the code, see e.g., the unit tests here.
In this case we have added an arrray_layouts parameter which is nowhere used in the current main branch.
This should be fixed and made more robust.

Need to discuss if this is an issue or an feature: if new parameter is added to the DB and no schema file is available yet, simtools fails (e.g., the simulation software type is read from the schema files in names. get_simulation_software_name_from_parameter_name()).
That tests fail when adding an unused parameter is maybe annoying, but probably the best way to make sure that nothing unwanted is added.
I therefore suggest to define this as 'feature' and not as 'issue'. This means that we have to always add generate new databases when changing / adding parameters. This avoids conflicts with other branches (I usually do it; but thought it wouldn't be necessary in this case).

I am closing this, as no worries are raised about the assumption that all parameter require schemas. So this is a feature.


Add the ability to set windowrules based on the command it was ran from.

I am currently working on a overlay-type menu that runs firefox (ran within a profile with no extra ui elements that come with a web browser) so I can see my calendar (proton calendar) by pressing a keybind. However, I am unable to set a windowrule for a specific window (or web page) for firefox. When I open htop or anything similar it allows me to see the original command it was ran with, arguments and all. But when I run hyprctl clients, there isn't any unique information for that window (apart from maybe the title which is dynamic based on what day/week it is). A possible fix for this is allowing exec to run commands with arguments to tell hyprland where to put the resulting window (I'm not sure if this is too complex for hyprland though).
Summary: The problem is I can set a windowrule for firefox, but it is global and I want to set it for a specific window.

I would need something similar: the ability to run a command and open the resulting window on a specific monitor.
I know, a possible solution was mentioned here, but this feature would help me for the same reason as OP.
I would suggest to make this a optional argument for the exec dispatcher:
exec <command> [<workspace>]

I think something like
bind=SUPER,E,exec [ workspace 3;noblur;size 1280 720 ],kitty

would work nicely

I think something like
bind=SUPER,E,exec [ workspace 3;noblur;size 1280 720 ],kitty

would work nicely
edit: actually, due to the backend, it'd be easier to make it
bind=SUPER,E,exec,[workspace 3;noblur] kitty

since [...] blah blah isn't valid bash anyways, this shouldn't be an issue

yeah this sounds like a good solution, kinda like how i3/sway does it

sorry for the long wait
added in 851df11eb595c5863bb931c32cbaa446c7071813


Implement basic web bash console using tornado and xterm.js

This terminal implementation should be compatible with the xterm.js API, besides, the terminal execution shall be asynchronous and non-blocking. Also it shall support POSIX compatible Operating Systems and Windows command execution.

I came across this package https://ergonomica.github.io/ I think It could be a good option but maybe It's out of the scope of this plugin.

@rlaverde, we need a real shell to use here. This means, a shell that can run Vim or Emacs inside it.
And I doubt Ergonomica or Xonsh have those capabilities :-)

@rlaverde, @ccordoba12 is right, We need a full featured system console to execute external commands that are not defined on Python neither on Spyder


add visibility annotation on component creation

Fixes

https://issues.redhat.com/browse/HAC-4586
Description

add image.redhat.com/generate annotation to components on creation
Type of change

[ ] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Verified the PR, it adds the right visibility annotation
/approved
/lgtm

/approve

/retest

/retest

/retest

/retest

/retest

/retest


Porting probot-config into Probot proper

Why do we need org wide configuration?
For some history on org wide configuration and why it's needed check out this dinosaur: https://github.com/probot/probot/issues/258
Current state of things
Right now org wide configuration require installing a separate probot extension, probot-config.
We do, however, have a built-in internal API for getting a config file for the repo an App is acting on. That method is context.config() and it takes a file name (located in the .github/ folder) and loads the contents of the yml file into an object.
Any time an app wants to add support for org-wide management, they have to require probot-config and using that to load their configuration files.
The future
It would be pretty amazing if org support was just built into probot, so that no apps need to update their code to support it. (aside from bumping their probot version?)
What this would look like: context.config() method would use probot-config under the hood.
Would love any 💭 or 👍 👎 🎉 ❤ 😆
cc/ @probot/maintainers
cc/ @jan-auer (original probot-config creator)

+1 to having a single place to deal with config. I think built-into-core is the place for it given how often config is required (i.e. practically always).
Having probot core depend upon probot-config feels a little odd. I think the code for probot-config is small enough (2 files, ~100 lines for the feature + ~200 lines of tests) to be copied into the probot repo which allows the probot-config extension to be deprecated.
As a migration path there can be a final release of probot-config that calls context.config() once the migration is complete

just a note:
I've had some issues trying to figure out how to mock context.config() in tests. using getConfig allowed me to mock the the config response.
Either way, it'd be nice to document/provide a recommended way to mock the context, in tests.

Is there a reason no one's put up a PR? I'm willing to try.

Not that I know. Give it a go :)

Part of resolving this would probably be a maintenance plan for probot-config. I'd suggest

when the updated context.config is released, add a README header to probot-config that new development should happen in mainline probot (bug fixes might still be applied both places?)
archive probot-config. Seems reasonable to do that soon after the context.config release goes out since probot-config has no outstanding bugs or pull requests.

Thanks for taking this on Mikael! Given that it's hard right now to maintain Probot as is, I think I'd go with the low-maintenance choice, add a note to the README and then archive the repo

fixed via #975

Could you make a PR to the probot-config repo?

On the topic of mocking, the tests themselves use jest.spyOn(github.repos, 'getContents'), and that seems to work well.
Added a README PR: https://github.com/probot/probot-config/pull/36
Also, do you have any sense of the frequency that GitHub App hosted versions of Probot apps (I'm particularly interested in the Welcome app) get updated to new releases of Probot?

Also, do you have any sense of the frequency that GitHub App hosted versions of Probot apps (I'm particularly interested in the Welcome app) get updated to new releases of Probot?

I do not know that, I'm actually not sure how these are deployed. But find out. Maybe ideally they would be deployed with each new GitHub version? We could pin the probot version in the dependencies and use Greenkeeper to update to the latest probot with each release. As long as automated deployment is setup this should just work :tm:

The Welcome app is still running on Probot 7, the source code is here:
https://github.com/behaviorbot/welcome/
It'd be great to update it and the other apps it depends on to the latest Probot. Once we have that working, we can setup continuous deployment. Bex added me as admin to the @behaviorbot org so I merge PRs


Add quick need card to overview

Blocked by #1607, Blocked by #1608

We are not sure anymore that this is the way to go, closing


wkhtmltopdf : Add version 0.12.5-1

wkhtmltopdf and wkhtmltoimage are open source (LGPLv3) command line tools to render HTML into PDF and various image formats using the Qt WebKit rendering engine. These run entirely "headless" and do not require a display or display service.
There is already an App Manifest for wkhtmltopdf, See here. But, it use the wkhtmltopdf/packaging repository which only contains releases for 0.12.6. The older releases are in wkhtmltopdf/wkhtmltopdf repository & currently it is in archived state.
The requirement of wkhtmltopdf 0.12.5 is that : Odoo 16 needs 0.12.5 version of wkhtmltopdf as dependency in windows, See https://www.odoo.com/documentation/16.0/administration/install/install.html#setup-install-source.
Odoo is a suite of business management software tools including, for example, CRM, e-commerce, billing, accounting, manufacturing, warehouse, project management, and inventory management.

[x] I have read the Contributing Guide.

Thanks!


OSRS Touchbar for Macbook's

So, I play Runescape on my PC but also on my Macbook for when im laying in bed, the one annoying thing is for the touchbar not displaying any FKeys it's pretty frustrating as it rules out doing bossing/anything requiring switches for me anyways as OSRS has turned into a game where bosses require a lot of switching to happen which is great, just sucks when your on a Macbook with a touchbar!
I did some research and found someone did create a touch bar for OSRS but it doesn't function very well as I would have to load up a seperate application everytime I load up Runelite! I don't believe this feature would be incredibly hard to develop into Runelite especially if you was to work with the code already developed (I might be wrong, things always do seem a hell of a lot easier when you don't know how much work is really needed for a feature!)
The current application I use is from the following GIT Repo - https://github.com/pg8wood/OSRS-touch-bar
Basically giving functionality to use the touchbar and move certain keys around on the touchbar, would be awesome if there was a plugin like this on Runelite, and yes I know this is probably not the most sought after development from you guys, but I think a lot of people would benefit and appreciate an update like this, I could be wrong but I think it'd be awesome!
Please let me know what you guys think of this, and I completely understand if this isn't something that you guys will be developing on, just feel like it would be a proper sweet update to a lot of players!!

Duplicate of #11584

Just an FYI, you can set it to always display fkeys when you're focused on RuneLite:
https://www.imore.com/how-make-function-keys-default-touch-bar-display
Alternatively, enable the Key Remapping plugin and you can use number keys as fkeys.


audit: use API to drop managed fields

KEP: https://github.com/kubernetes/enhancements/pull/2982/files
PR: https://github.com/kubernetes/kubernetes/pull/94986

/lgtm
/approve

/retest


Cannot pass custom arguments to DockerCreateNetwork method

Agent Version: 2.165.0
OS: Ubuntu 18.04
Azure DevOps Type: dev.azure.com
I am trying to use Docker containers via the agent to run Azure YAML Pipelines.  The agent creates a new Docker Bridge Network to run the container in.
The code that creates this network does not seem to provide a mechanism to pass extra options to the underlying docker binary:
public async Task<int> DockerNetworkCreate(IExecutionContext context, string network)
{
    var usingWindowsContainers = context.Containers.Where(x => x.ExecutionOS != PlatformUtil.OS.Windows).Count() == 0;
    var networkDrivers = await ExecuteDockerCommandAsync(context, "info", "-f \"{{range .Plugins.Network}}{{println .}}{{end}}\"");
    if (usingWindowsContainers && networkDrivers.Contains("nat"))
    {
        return await ExecuteDockerCommandAsync(context, "network", $"create --label {DockerInstanceLabel} {network} --driver nat", context.CancellationToken);
    }
    return await ExecuteDockerCommandAsync(context, "network", $"create --label {DockerInstanceLabel} {network}", context.CancellationToken);
}

As a result, the Docker daemon (as per the documentation) does its best to not use an overlapping subnet for this network:
When you create a network, Engine creates a non-overlapping subnetwork for the network by default. This subnetwork is not a subdivision of an existing network. It is purely for ip-addressing purposes. You can override this default and specify subnetwork values directly using the --subnet option.

In my case this seems to create <IP_ADDRESS>/16 (just in case I decide to run 65,536 simultaneous builds, I guess).
The problem here is that 90% of my companies real network lives in that address space and is therefore unreachable from within the container.  So my builds cannot interact with any of the services in our on-prem infrastructure.
This would be a simple fix; I just need to be able to append the --subnet x.x.x.x argument to the docker network create command, eg:
docker network create --subnet <IP_ADDRESS>/24

However, I cannot see any way to do this via the Agent.  Is it possible to add this functionality or is there a workaround I have overlooked?

I have found a work-around for this issue, so I'll post it here in case anyone else stumbles across this before it's (hopefully) fixed!
It looks like docker is inspecting the hosts routing table to determine if a network is in use.  So I have explicitly added routes for the networks that we actually use.
Replace "<IP_ADDRESS>" in the following example with your hosts actual default gateway:
# ip route add <IP_ADDRESS>/12 via <IP_ADDRESS>
# ip route add <IP_ADDRESS>/8 via <IP_ADDRESS>
# systemctl restart docker

Now when the Agent creates a temporary network, it will not use any of that address space.
# docker network inspect vsts_network_9e31e7fe041d4b03bb8acbda9b218674
[
    {
        "Name": "vsts_network_9e31e7fe041d4b03bb8acbda9b218674",
        "IPAM": {
            "Driver": "default",
            "Options": {},
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/20",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        }
    }
]


[Feature] support load json into varchar column with $ path

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #9857
Problem Summary(Required) ：

Support load json row into varchar column through $ jsonpath. Previously only JSON type is supported.
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run all

@Mergifyio backport branch-2.4

run starrocks_admit_test


Migrating from 2.1.5 to 2.2.0?

I just upgraded from 2.1.5 to 2.2.0 and it broke quite a few things - one of which is that I needed to change the data-nodrag attribute to nodrag. Others I'm still debugging why they are broken and what I need to do to fix them. So two questions:

Is there a list of changes somewhere between 2.1.5 and 2.2.0 aside from the commit log?
Why wasn't this a full version bump (3.0.0)?

Ugg scratch that. Apparently 2.2.0 is being unpublished?

The v2.2.0 version has been reverted by @JimLiu. A new version is in the making. In the meantime, you can keep using v2.1.5.

Version 2.3.0 has been released. Does the issue still occur?

Seems to work fine with 2.4.0. Thanks!


fix: avoid exit handler nil pointer when missing outputs. Fixes #13445

Fixes #13445
Motivation
This PR addresses a critical issue where exit handlers fail when called on a failed step with a null or empty output parameter. Specifically, when a step crashes before creating the output file referenced in valueFrom, the exit handler encounters a runtime error: "invalid memory address or nil pointer dereference". The problem has been observed in Argo Workflow versions 3.5.5 through 3.5.10.
Modifications
To resolve this issue:

Added a nil check before calling params.Value.String() in the relevant code section.
This check ensures that we only attempt to access the parameter value when it exists, preventing the nil pointer dereference.

Verification
To ensure the effectiveness of this fix and prevent regression, I've added a new end-to-end (e2e) test case. This test case verifies:

The presence of an exit handler node in the workflow graph after a step failure.
Proper population of the exit handler's input based on the output from the failed step.

Previously, when the issue occurred, the exit node was never created, and the workflow terminated immediately after the first step. The new test confirms that:

The workflow correctly transitions to the exit handler node instead of prematurely completing.
The exit handler receives and can process the output parameter from the failed step, which during the next operate is populated and we can see the failed status.

/retest

Many thanks for the fix @miltalex & @agilgur5 !

But imo fixing that is beyond the scope of this PR/issue

True, it is. But I think empty string/zero value makes more sense than erroring or not resolving, since people want to detect non-existence in expressions.


Recursively delete a directory with readonly file results in inconsistent state

Describe the bug
Recursively deleting a directory with read only file results in an inconsistent state of the MockFileSystem:

The deletion results (as expected) in an UnauthorizedAccessException
The file is still present (File.Exists returns true)
The directory however is removed (Directory.Exists returns false)

To Reproduce
Consider the following unit test (using Xunit and FluentAssertions):
    [Fact]
    public void DeleteDirectoryRecursive_WithReadOnlyFile_ShouldThrowExceptionAndNotDeleteDirectory()
    {
      var sut = new MockFileSystem();
      sut.Directory.CreateDirectory("foo");
      sut.File.WriteAllText("foo/bar.txt", "xyz");
      sut.File.SetAttributes("foo/bar.txt", FileAttributes.ReadOnly);

      var exception = Record.Exception(() =>
      {
        sut.Directory.Delete("foo", true);
      });

      exception.Should().BeAssignableTo<UnauthorizedAccessException>();
      sut.File.Exists("foo/bar.txt").Should().BeTrue();
      sut.Directory.Exists("foo").Should().BeTrue();
    }

This test fails at the last line, as the directory is deleted, although the file still exists, resulting in an inconsistent state of the MockFileSystem
Expected behavior
The unit test should be successful.

Thanks for reporting! This would probably needs to be fixed in https://github.com/TestableIO/System.IO.Abstractions/blob/main/src/System.IO.Abstractions.TestingHelpers/MockDirectory.cs#L118.

Hello, @fgreinacher, I would like to work on this issue.

Hello, @fgreinacher, I would like to work on this issue.

Thanks!

You're welcome, @fgreinacher! I've already opened a PR (#863) for this issue.


Dynamic Partials

Hi,
I've just created a ticket on the Mustache project with regards dynamic partials, I thought I'd reference it here as it may be of interest: https://github.com/janl/mustache.js/issues/241
Kind regards,
Jamie

Don't see any movement on this idea in the spec, but would re-open if I'm wrong.


Updated styling in installation guide

Fixed minor styling issues per discussion in #1862

May I ask you please to add the following lines:
After pod install
Please make sure you open .xcworkspace, not .xcproject
And somewhere there:
Please make sure, that your XCode is 9.+, otherwise you'll get use of undeclared identifier 'MKMapTypeMutedStandard' error

@alvelig added. I am not a native english speaker, so feel free to point out if something is incorrect or hard to understand.
@rborn I see that although library is called GoogleMaps, they use "Google Maps" almost everywhere on their site. Let me know if I meant something different.

@satori-ytolstoguzov Neither am I lol. LGTM

@rborn #1914


.editorconfig: Format .bash files like .sh

Done under the assumption that disabling the indent size check is a temporary workaround (based on a quick skim of the original discussion). If maintainers disagree I could just remove the formatting commit.
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.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

This looks good to me, but I'm curious about the rebuild count.  If it's primarily dotnet stuff, it can probably go in master.  If it's more than that maybe staging.

This looks good to me, but I'm curious about the rebuild count. If it's primarily dotnet stuff, it can probably go in master. If it's more than that maybe staging.

Good catch! nixpkgs-review got to 1,000 packages before coming to a standstill. I've rebased onto staging instead.

See https://github.com/NixOS/nixpkgs/pull/353490.

See (the related) #353490.

Thanks! I've added a comment about the only part of it which I have an opinion about (other than saying that we absolutely should enforce things like indentation for sanity).


Finchley.RC1 not started for DataSource Circular Dependency

When I use spring.datasource.initialization-mode: always application cannot start and occur error message like below
The dependencies of some of the beans in the application context form a cycle:

   servletEndpointRegistrar defined in class path resource [org/springframework/boot/actuate/autoconfigure/endpoint/web/ServletEndpointManagementContextConfiguration.class]
      ↓
   healthEndpoint defined in class path resource [org/springframework/boot/actuate/autoconfigure/health/HealthEndpointConfiguration.class]
      ↓
   dbHealthIndicator defined in class path resource [org/springframework/boot/actuate/autoconfigure/jdbc/DataSourceHealthIndicatorAutoConfiguration.class]
┌─────┐
|  scopedTarget.dataSource defined in class path resource [org/springframework/boot/autoconfigure/jdbc/DataSourceConfiguration$Hikari.class]
↑     ↓
|  org.springframework.boot.autoconfigure.jdbc.DataSourceInitializerInvoker
└─────┘

I test previous version and I check initialization-mode working until Finchley.M7.
I use mariadb and HikariCP.

This is a duplicate, thanks

When I use mariadb and Tomcat DBCP initialization-mode is working.

When I use org.apache.tomcat.dbcp.dbcp2.BasicDataSource initialization-mode is working.


.storage folder valetudo_camera to store the data.

Description
As it is now the camera store data such logs, mqtt raw data, json and png on the .storage folder.
Solution
Will create a folder ".storage/valetudo_camera" where those data are stored in a more cleaner way.
Alternatives
No response
Context
No response

fixed, will be available with the next release.

version 2024.04.0 released.


[QoI] Don't walk into erroneous apply expressions while validating top level code

In checkTopLevelErrorHandling if apply expression did not type-check,
don't attempt walking inside of it. This accounts for the fact that we don't
erase types without type variables to enable better code complication,
so DeclRefExpr(s) or ApplyExpr with DeclRefExpr as function contained
inside would have their types preserved, which makes classification
incorrect.

/cc @rudkx @slavapestov

@swift-ci Please smoke test


replace our tach counters with PCNT unit

we probably ought use the ESP32's PCNT functionality rather than our handrolled tach counters. see https://docs.espressif.com/projects/esp-idf/en/v4.3/esp32/api-reference/peripherals/pcnt.html. it can use IRAM, so we don't lose anything there.

i don't think these really add much value. they have a glitch filter, but it's just "reject pulses shorter than this", and it puts a lock on dynamic frequency scaling. let's just filter noise properly with a low pass filter on the PCB.
however, i think we can remove our interrupt handler from IRAM.


Modify the OpenDRR/data website to link to B2/S3-hosted GeoPackage files

This is a follow-up to Issue OpenDRR/opendrr-data-store#42 and PR https://github.com/OpenDRR/opendrr-data-store/pull/43#pullrequestreview-633764055, especially Joost's (@jvanulde) recommendation:

to see a new issue created to track the requirement of S3 and/or a PR against the data repo with the new zip files.

This will need to be done for both:

the existing Jekyll site at https://opendrr.github.io/data/
the future Gatsby site at the same URL

@anthonyfok can we close this now?

Hi @anthonyfok since the data repo has been archived and we now like to release assets in it's replacement (downloads), can we close this?


[Doc Change]export GOPATH if not available.

Issue #, if available:
Description of changes:
GOPATH may not be available on an environment and in such case  go env GOPATH is helpful. Updated the file to export GOPATH to a default value.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/ok-to-test


Contributions for the PT-BR translation (guides)

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@JordyAraujo Thank you for  creating this PR.  However, since it was created, the underlying document has changed.  Can you please review and resolve the current merge conflicts?  It could be the changes to the underlying document are so significant that your changes are no longer relevant.  If the latter is the case, it might be easier to just close this PR and create a new PR with any changes you still feel are important to make.
If you need any assistance or have questions about resolving the merge conflicts, please reply back or contact someone in the Contributors Room on Gitter.
Thank you.

We are restructuring a lot of the code and learning resources referenced in your pull request, so for now I am going to go ahead and close this pull request. Thanks again for opening this pull request and for contributing to the community.


Update Delegated DCV dev docs with a recurring caveat

We're seeing multiple customer having issues with delegated DCV because any pre-existing DCV TXT values will inevitably conflict with the delegated DCV Cname and prevent Delegated DCV from actually working.

PCX-9588


Table height not accurate until manual save

When creating a new table, the value assigned for height is used even if the contents of the table increase the size of the table. This issue is resolved by manually opening powerpoint and saving using office, at which point the real height of the table (not the one assigned by the user) is given.
Edit: Using the pptx library to save the file and reopen it does not work

From looking through the code, it doesnt seem like table rows ever adjust their own height, it is dependent on the user to adjust the row height and then the table will adjust its stored height. @scanny Would you happen to know if there is a workaround to this, or a way I can calculate the expected cell height using the font, size, and text?

Resizing is a powerpoint runtime engine thing and is outside the scope of python-pptx.
It is straightforward (but a little tedious) to calculate height given a size.   Width is much trickier.

On Jun 26, 2017, at 11:59 AM, Robert C<EMAIL_ADDRESS>wrote:
From looking through the code, it doesnt seem like table rows ever adjust their own height, it is dependent on the user to adjust the row height and then the table will adjust its stored height. @scanny https://github.com/scanny Would you happen to know if there is a workaround to this, or a way I can calculate the expected cell height using the font, size, and text?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/scanny/python-pptx/issues/296#issuecomment-311103178, or mute the thread https://github.com/notifications/unsubscribe-auth/AD6CgdzaREEyWwlBLz3v45ow9aaW1locks5sH9V5gaJpZM4N4x0U.

@buzzm Could you point me in the right direction to figure out how to calculate height given a constant cell width?

The business of auto-sizing and/or auto-fitting (shrink to fit) is a job done by the renderer (PowerPoint) at runtime. python-pptx doesn't include a renderer, and is unlikely ever to.
There was a little work done a while back for a sponsor who absolutely had to have something on shrink-to-fit that would at least kind-of work, if you trace through from here in the code you can study that and maybe get some ideas:
https://github.com/scanny/python-pptx/blob/master/pptx/text/text.py#L64
In general it's a difficult thing to do in Python because rendering details like font metrics are platform specific and are not well supported by available libraries (at least at the time that bit was written).


error when clicked on markdown generated link

🐞 Bug report
Description
I have a link (/home/blog)  in my markdown  blog-post to blog-post-lust-view as shown below
When I click the link, site-map I am getting following error
libs-home-src-lib-home-module-es2015.js:902 problem during parsing static scully content TypeError: Cannot read property 'split' of undefined

Markdown blog
---
title: blog 1
description: This is the first demo blog in this sample.
---

# blog 1

## its a wild world after all

Related information [blog-2](/home/blog/blog-2)

[site-map](/home/blog)

🔬 Minimal Reproduction

💻Your Environment
Angular Version:

Scully Version:

🔥 Exception or Error
I think the error is caused at

https://github.com/scullyio/scully/blob/master/projects/scullyio/ng-lib/src/lib/scully-content/scully-content.component.ts#L75

Can you provide a repo that reproduces this? I seem unable to reproduce this scenario

https://github.com/xmlking/yeti

I will look at this tomorrow.

@xmlking

It works for me. I'm not sure what happens on your system?

Click site-map and see console logs

Ok, now I'm seeing the error.
It does not make sense, the error is on a page that doesn't (or at least should not) contain the scully-content component. yet the error is raised from there.
Can you provide more pointers on how this could happen?

@xmlking I found the root of the issue. Building a fix right now.

I still get error after going to blog page https://ngx-starter-kit.firebaseapp.com/home/blog/blog-4
then clicking on YETI  on the header

Are you sure you updated to the latest @scullyio/ng-lib ?

yes I am using 0.0.8.
how to reproduce.:
clone https://github.com/xmlking/yeti
yarn run scully:all
or
npx run scully:all

clicking links in blog posts works fine. even the site-map , but from block post, if you click YETI home link on the left, then you will see this error.

I also notices other problem, maybe related....
in the same repo run only angular ng serve
ng serve

then click blog post and then click , home. you will see scullyio-ng-lib.js:402 problem during loading static scully content  error
I dont have any scullyio in home(landing) page. why I do see this error when clicking /home linkk?

@xmlking That would be the same bug/reason. It looks like it regressed back somehow. going to check.

I looked into it. It isn't exactly the same bug, but closely related. Looking for a fix. Might take a day or 2.

I addressed this issue. I don't recall the exact PR, but it will be fixed when we drop the new version.
You can expect this to be later today. closing this now.
When you find any new regressions, please open a new issue, and refer this one.


Change the variable name to skip_cos_sm_auth_policy in instance DA

Change the variable name from skip_cos_kms_auth_policy to skip_cos_sm_auth_policy in instance DA
https://github.com/terraform-ibm-modules/terraform-ibm-cos/blob/1089405670f2814f8972f9a314b79d1fa212ffea/solutions/instance/variables.tf#L111
As source of auth policy is Secrets Manager not KMS
https://github.com/terraform-ibm-modules/terraform-ibm-cos/blob/1089405670f2814f8972f9a314b79d1fa212ffea/solutions/instance/main.tf#L22
Affected modules
By submitting this issue, you agree to follow our Code of Conduct

PR merged - closing the issue


The graphical installer can't to start (already on the right place)

Sorry,first I opened this issue on the Trident-core section:
https://github.com/project-trident/trident-core/issues/85

The situation is same with the Trident-19.04-U1-x64-20190515.iso, from DVD the graphical installer can't start, from USB key the installer can start.
If the Installation Complete blue screen came up and I click on the Live CD than the login prompt displayed.
I tried to read the /tmp/xorg.log, but I couldn't found it. The pc-sysinstall.log is empty yet.

The problem is missed in the new TOS-19.06 release.


fix(pages): / exclude rule in _routes.json overrides include routes for wrangler pages dev

Because of the way we perform custom route rule matching, and because we always apply exclude rules first, wrangler pages dev will ignore any include rules, if exclude contains a rule that references a root level index  (/).
This commit makes sure that root level exclude rules are properly handled and will not override any include rules we have in place.

closing this because we realised we need a larger scope than just handling / use cases.


4K vs 128K version

Hello there, a quick question here. I just read the technical report and I am not sure what is the beneifit of using the 4K version. Apparently the 128K has a similar performance across the board. My understanding is that the architecture is exactly the same, the only difference is the post-training stage and the data used for that stage.
Thanks!

Hi @ealmazanm  I noticed you also added the same question to hugging face discussions https://huggingface.co/microsoft/Phi-3-mini-4k-instruct/discussions/87#668cd9454912e2ac2155ef83
The Phi-3 model available in two context-length variants—4K and 128K tokens https://azure.microsoft.com/en-us/blog/introducing-phi-3-redefining-whats-possible-with-slms/
Here are some key differences and benefits of using the 4K version versus the 128K version:
Size and Efficiency
The 4K version of the Phi-3 model is smaller in size, which means it requires less computing power. This makes it more cost-effective, easier to fine-tune, and allows for faster response times. It's especially great for resource-constrained environments including on-device and offline inference scenarios, and latency-bound scenarios where fast response times are critical.
Context Awareness
The 4K version ensures efficient processing of input data while maintaining context awareness. On the other hand, the 128K is ideal for handling tasks requiring broader context comprehension.
The choice between the 4K and 128K versions depends on your specific needs. If you require a model that is more cost-effective and efficient, the 4K version would be a good choice. If your tasks require broader context comprehension, the 128K version would be more suitable. Both versions offer high performance for their respective sizes.