Pylance client: couldn't create connection to server.

Pylance client: couldn't create connection to server.It doesn't detect errors in my code there is error showing in output.

Thanks for the report, unfortunately, this is not enough data for us to debug.
If this continues to be a problem, would you please reopen the issue and add the information in the report as described in the troubleshooting guide?

I also have encountered the  same issue

I have the same problem

Me too

But the worst part is I have been trying to find a solution for a dang long time but I can't

@jonleeacc, @Boltynight, if you are still seeing this please open a new issue and provide your "Python Language Server" log.

Just go to your .vscode folder in your computer. Then in .vscode/extensions. There you will see one (or several) folders related to pylance. Just delete them. Then reopen VSCode, reinstall Pylance, it shall work. Maybe at the end, you'll need to restart VSCode to apply the changes. ;)

I get a lot of errors with this.
and
this was in the report in output
2023-03-19 13:44:59.381 [info] [Error - 1:44:59 PM] Server initialization failed.
2023-03-19 13:44:59.381 [info]   Message: Pending response rejected since connection got disposed
Code: -32097
2023-03-19 13:44:59.381 [info] [Info  - 1:44:59 PM] Connection to server got closed. Server will restart.
2023-03-19 13:44:59.381 [info] true
2023-03-19 13:44:59.381 [info] [Error - 1:44:59 PM] Pylint client: couldn't create connection to server.
2023-03-19 13:44:59.381 [info]   Message: Pending response rejected since connection got disposed
Code: -32097
2023-03-19 13:44:59.381 [info] [Error - 1:44:59 PM] Restarting server failed
2023-03-19 13:44:59.381 [info]   Message: Pending response rejected since connection got disposed
Code: -32097
then
2023-03-19 13:44:59.885 [info] Traceback (most recent call last):
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/tool/lsp_server.py", line 48, in
name="pylint-server", version="v0.1.0", max_workers=MAX_WORKERS
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/pygls/server.py", line 358, in init
super().init(protocol_cls, converter_factory, loop, max_workers)
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/pygls/server.py", line 200, in init
self.lsp = protocol_cls(self, converter_factory())
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/pygls/protocol.py", line 166, in default_converter
converter = converters.get_converter()
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/lsprotocol/converters.py", line 17, in get_converter
return _hooks.register_hooks(converter)
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/lsprotocol/_hooks.py", line 35, in register_hooks
_resolve_forward_references()
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/lsprotocol/_hooks.py", line 30, in _resolve_forward_references
attrs.resolve_types(value, lsp_types.ALL_TYPES_MAP, {})
File "/home/gbowne1/.vscode/extensions/ms-python.pylint-2023.4.0/bundled/libs/attr/_funcs.py", line 408, in resolve_types
hints = typing.get_type_hints(cls, globalns=globalns, localns=localns)
File "/usr/lib/python3.7/typing.py", line 973, in get_type_hints
value = _eval_type(value, base_globals, localns)
File "/usr/lib/python3.7/typing.py", line 262, in _eval_type
ev_args = tuple(_eval_type(a, globalns, localns) for a in t.args)
File "/usr/lib/python3.7/typing.py", line 262, in
ev_args = tuple(_eval_type(a, globalns, localns) for a in t.args)
File "/usr/lib/python3.7/typing.py", line 263, in _eval_type
if ev_args == t.args:
File "/usr/lib/python3.7/typing.py", line 658, in eq
return frozenset(self.args) == frozenset(other.args)
After that this keeps repeating:
File "/usr/lib/python3.7/typing.py", line 664, in hash
return hash((self.origin, self.args))
File "/usr/lib/python3.7/typing.py", line 477, in hash
return hash((self.forward_arg, self.forward_value))
File "/usr/lib/python3.7/typing.py", line 663, in hash
return hash((Union, frozenset(self.args)))
File "/usr/lib/python3.7/typing.py", line 664, in hash
return hash((self.origin, self.args))
File "/usr/lib/python3.7/typing.py", line 477, in hash
return hash((self.forward_arg, self.forward_value))

I tried paulpacaud's solution of deleting the pylance-related folder and reinstalling the extension. It initially seemed to work, but after restarting the system and opening VS Code, the error showed up again. Please fix it. I've opened a new issue here.

My issue (on a remote server) was resolved by add the following to my workspace settings.json. Don't forget to adjust the correct python path.
{
    "python.languageServer": "Pylance",
    "pylint.path": [
        ">>>path/to/python<<<",
        "-m",
        "pylint"
    ]
}

I'm facing the same issue with my local server on Ubuntu. Can anyone help me?

I am facing this issue as well

I had the same issue. I will try the @Miladiouss solution. Hope it works.

Run VS Code as admin or globally disable trust.

Uninstalling pylance and restarting vscode solved the issue in my case.

FYI - Setting up a new VS Code profile worked for me which is much easier than reinstalling.

I'm facing the same issue with my local server on Ubuntu. Can anyone help me?

same issue

my error log said that my Jedi server was unable to boot. After changing the "python.languageServer" in workspace settings to "Jedi", the issue was fixed.

The same problem occurred with the version of the pylance vs code v2024.4.1 extension.
The rollback to version 2024.3.2 helped

It looks like it was some kind of problem with vs code, because now, after restarting it and updating pylance to the latest version, the connection failure no longer appears. But back than even a complete reinstallation of Python and pylance packages had no effect.

I had to install the newest Pylance to fix this


Fixed some bugs about etcd db object update authentication.

What this PR does / why we need it:
Fixed some bugs about etcd db object update authentication.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage decreased (-0.5%) to 56.13% when pulling 5af5440be949356b7b5597630aa0af39dcc38afa on xxwjj:etcd_bug into 7f2140dbfc5bbfcb73e32d7d8032653c6c5671d8 on opensds:development.

Coverage decreased (-0.5%) to 56.13% when pulling 5af5440be949356b7b5597630aa0af39dcc38afa on xxwjj:etcd_bug into 7f2140dbfc5bbfcb73e32d7d8032653c6c5671d8 on opensds:development.

Coverage decreased (-0.5%) to 56.13% when pulling 5af5440be949356b7b5597630aa0af39dcc38afa on xxwjj:etcd_bug into 7f2140dbfc5bbfcb73e32d7d8032653c6c5671d8 on opensds:development.

lgtm


Update _accessibility.scss

Focusable visually hidden element. Best for the "skip to content."

Hi! That's very useful.
Can you please change the code to (because it's a standalone utility class rather than a class modifier):
.sr-only-focusable { // focusable, visually hidden element
  &:not(:focus):not(:focus-within){
    @extend .sr-only
  }
}

and I will merge the pull request. Thanks

ok. I'll do just that.

Updated to utility.

Thanks for contributing! 😊


I would like to get the complete list of potential values that could appear in a log field for the “category”.

I would like to get the complete list of potential values that could appear in a log field for the “category”.
The documentation doesn’t mention the possible values for this field.

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

ID: e4222788-80fe-4080-df9d-ea4d147abd2f
Version Independent ID: 07c6561e-6d05-bcf8-c524-5d146ca4f4ae
Content: alert resource type - Microsoft Graph v1.0
Content Source: api-reference/v1.0/resources/security-alert.md
Product: security
Technology: microsoft-graph
GitHub Login: @BenAlfasi
Microsoft Alias: MSGraphDocsVteam

Hi @sreemolakildas,
We ran into the same issue of missing documentation when we were trying to develop some SIEM rules for the Microsoft Graph API security resource, but ultimately managed to track down the documentation here. For reference & posterity, I'll list out the values:

"General"
"CommandAndControl"
"Collection"
"CredentialAccess"
"DefenseEvasion"
"Discovery"
"Exfiltration"
"Exploit"
"Execution"
"InitialAccess"
"LateralMovement"
"Malware"
"Persistence"
"PrivilegeEscalation"
"Ransomware"
"SuspiciousActivity"

Please note, however, that these category values are only valid for the "alerts_v2" relationship: if you need the values for just "alerts" (i.e., alerts v1) let me know and I can provide those as well.


Update HOSTS.txt

Add some websites and <IP_ADDRESS> www.google.com #forcesafesearch.

Hi 11201010, in my case the force safe search doesn't work. Could you please remove it from your PR? Thanks

Okay, now #forcesafesearch has been removed.


Default subnet mask of 24 instead of 32 if value unset

Improvement
Default value for IPv4 subnet mask on interfaces should be 24 not 32
Currently the default value for a subnet mask is 32 if the value was not set before. This for example is the default if you are changing an interface from DHCP to Static IPv4. The downside is that someone who oversees the value might lock himself out from the machine.
Example:
Someone (indeed this someone was me 😜) is connected to the firewall through VPN and changes the WAN interface from DHCP to Static IPv4. After entering the right IPv4 address the person forgets to set the corresponding subnet mask from 32 to 24 (where the latter is mostly standard on local networks). After applying the changes the person gets locked out from the firewall and has to fix it locally because no traffic can get in and out.
A default setting of 24 instead of 32 would have prevented this situation. A subnet mask of 32 also doesn't make any sense on a WAN interfaces IP address from my perspective.

A mismatch of IP mask always has consequences one way or another. The fix might be nicer to handle with JS also but I've never heard this complaint in all the years before...

Another idea would be to leave the subnet mask blank after switching to IPv4 for the first time and let the browser handle the field which could be marked as required. So someone has to interact with the field just once before the setting is stored.

Unfortunately I'm not convinced that /24 is the right preselection. select doesn't have an empty value, adding and validating it adds even more code and makes the workflow longer than necessary. People can still lock themselves out choosing a number of values, not only /32.

Unfortunately I'm not convinced that /24 is the right preselection.

What would your recommendation be? I think we are aligned that 32 is not a good preselection, are we?

People can still lock themselves out choosing a number of values, not only /32.

You are right. Nevertheless the procedure to switch from Static IPv4 to DHCP or vice versa is a common task and should prevent situations like that by having proper default values. We have lots of ways to provide the user a experience where he has to overthing given values twice or he has to set wrong values actively – not passively.

select doesn't have an empty value, adding and validating it adds even more code and makes the workflow longer than necessary.

Theres another idea we could use to prevent this without setting a default value beforehand:
https://stackoverflow.com/a/46710230
The downside of those techniques would be that you would need to test them on most browsers – I don't know if all of them would respect it.

@AdSchellevis May I know why you just closed this PR? There is neither a proper solution for the issue I'm refering too, nor are there good arguments to just close the issue.
I understand if there might be more "important" issues to solve. But please keep in mind that just closing a PR without any comment or explanation for the closure does not help keeping the motivation high to contribute to a project.

We discussed this at length yesterday. The problem you state is that you haven’t set the proper netmask. Your patch changes the situation to a more favourable way, but it’s not a fix of the problem class. Secondly the problem here was never raised in the bug tracker before and the real world value is in doubt. Thirdly interfaces.php is a legacy page and we would like to avoid unnecessary or lengthy additions. So lastly a proper fix would require more logic and code, possibly in JS conflicting with other points here and it’s doubtful the complexity is worth doing something in this regard.
Cheers,
Franco

Hi Franco,
thanks for the answer.
I don't get the argument that you won't implement it just because nobody raised the issue ever before. That doesn't implies that the problem does not exist. I asked you if we are consent that the default value for the subnet mask of /32 doesn't make sense for most of the WAN cases but I unfortunately did not get an answer on this. I still think it is not a good default value.

So lastly a proper fix would require more logic and code, possibly in JS conflicting with other points here and it’s doubtful the complexity is worth doing something in this regard.

Another idea would be to simply replace the subnet dropdown with a text input in general which gets checked by a regex before sending the form. This way the field could be left empty by default (like the IP address which is empty by default too) and people have to interact with it at least once. This would not need anything in addition which is not already there and solves the problem.

but it’s not a fix of the problem class.

A final fix could also be to combine the fields for IP address and subnet mask into one field which would reduce overall complexity at the end because only one field has to be validated / propagated instead of two.
Another personal thing:
This is my first PR for OPNsense and it doesn't feel well how things went. It would have been nice to get an answer like yours before your collegue just closes the PR without any comment. I don't think that's a good way to threat people who are contributing and looking for a solution in a constructive way. Your arguments sound to me a bit like "we don't want to change anything here" and I'm missing the will of finding a solution together for the issue I raised.
If you personally don't want to implement a fix or don't see the necessity of a fix I'm fine with that.  But please don't mask your personal decision with arguments which you are not willing to discuss on.
I got from you that OPNsense is a system where you should think twice and check values again before you apply them. That is true and I fully back that. Thats what's a sysadmin's job. But I think all of us know that reality sometimes is a bit different and we are happy if technical systems "assist" us by avoiding potential lockout situations, isn't it?
Exactly this kind of avoidance is what I tried to implement with my patch.

Text input is probably the most elegant way here, drop-down breaks flow and an empty drop down value as an alternative seems silly since the point of drop-down is to offer a preselection. Merging fields is out of the question for complexity.
but now consider changing UX for one single component and leaving out the fact that drop downs are everywhere for subnet size.
Try to explain to others why any single change here is better than before. Others are far likely to complain about inconsistency as a whole or changes in particular that come out of nowhere without a technical reason underneath.
if you feel you have been treated unjustly that fair since everyone is entitled to their unique experience. For me I’m happy to stop responding because too much time went into this already.
Happy holidays,
Franco

It would have been nice to get an answer like yours before your collegue just closes the PR without any comment.

While preparing for 23.1 we (as a team) assumed there had been enough discussion on this PR, I should have probably mentioned that while closing the issue.
Long story short, at the moment we don't feel like changing this area as whatever fix we implement it will likely always raise similar issues. Enforcing a mask (don't split the address and mask) would likely be the best option, but given the complexity already in the page and the burden of maintenance for legacy components we simply don't think we should aim for that.
I can certainly understand that this wasn't the best first experience in contributing code, a way to prevent these things from happening is starting with a ticket first and asking for feedback.  If an issue is more wide spread, more people feel alike, it is more likely changes will be incorporated at some point. Unfortunately we still have the burden of some legacy code to maintain which in the long run we would rather replace, but we do have to prioritise.
Pull requests on core are weighted heavily, as in our experience the number of regressions we have to fix after merging code from others often out weights the amount of time needed to make the initial PR.
Looking at this PR for example we normally wouldn't pin a default in the select but move it on top of the page as an explicit default, in which case the select remains simple in terms of logic. For MVC pages these type of issues don't exist as the model and form functions handle this automatically.
When it comes to closing tickets and pull-requests, we automated the ticket part as mentioned in the contribution guidelines, pull requests we inspect manually before release preparation.
A lot of projects just keep them open endlessly which avoids these discussions, but also provides a false view on the reality in my humble opinion, priorities change and people do need to spend time.
I hope this explains things a bit.

@AdSchellevis I really thank you for your detailed answer and the explanations. This helped me to understand the background.

Pull requests on core are weighted heavily, as in our experience the number of regressions we have to fix after merging code from others often out weights the amount of time needed to make the initial PR.

A lot of projects just keep them open endlessly which avoids these discussions, but also provides a false view on the reality in my humble opinion, priorities change and people do need to spend time.

I totally understand what you are pointing out and I think we can just leave the PR closed according to this explanation. I may find a different way to fix this for me and others in the future.
I dislike the way @fichtner was communicating here – especially according to what I wrote about my personal experience. So we leave this as my personal "unique experience". No further discussion needed from my side.
Happy holidays too
Christian


Add yaml to query

Add yaml to query when uploading reports
Should fix issues when the codecov.yml file is not found or using .codecov.yml

In https://github.com/codecov/codecov-bash/pull/180 the failing tests in this regard were "fixed" - have not investigated, just noticed this.

The server gets told where the codecov.yml file is by way of the network section of the uploaded report here https://github.com/codecov/codecov-bash/blob/master/codecov#L1239
Since we don't parse the yaml= querystring server-side, I removed it from the tests, rather then merging this PR.


Update README.md

Fixed formatting issue and added software, blog, telegram, youtube links.

Not sure what problem the system experienced. Can you please check.

@AntaiIphone the problem is due to a lint process I have to fix in order to have awesome desci in the awesome lists.


Installed visual studio code using dsc but couldn't uninstalled it.

Hi Sir
I have installed visual studio code using dsc resource cchoco (Enable = "Present") and was successful, but I'm not able to uninstall it by replacing (Enable = "Absent"). Eventhough, it says package uninstalled, still I can use visual studio code.
Configuration Ran First:
Configuration basicinstallconfig {
Import-DSCresource -Modulename "cchoco"
Node "Localhost" {
cChocoInstaller "installchoco" {
InstallDir = "C:\Choco_Package_Manager"
}
cChocoPackageInstaller "installGit" {
Name = "Git"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
cChocoPackageInstaller "installvsc" {
Name = "vscode"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
cChocoPackageInstaller "installchrome" {
Name = "google-chrome-x64"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
cChocoPackageInstaller "installnotepadplusplus" {
Name = "notepadplusplus"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
}
}
basicinstallconfig
Configuration Ran Next:
Configuration basicinstallconfig {
Import-DSCresource -Modulename "cchoco"
Node "Localhost" {
cChocoInstaller "installchoco" {
InstallDir = "C:\Choco_Package_Manager"
}
cChocoPackageInstaller "installGit" {
Name = "Git"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
cChocoPackageInstaller "installvsc" {
Name = "visualstudiocode"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "absent"
}
cChocoPackageInstaller "installchrome" {
Name = "google-chrome-x64"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
cChocoPackageInstaller "installnotepadplusplus" {
Name = "notepadplusplus"
DependsOn = "[cChocoInstaller]installChoco"
Ensure = "present"
}
}
}
basicinstallconfig
Error in the Console: Chocolatey v0.10.11 Uninstalling the following packages: visualstudiocode  visualstudiocode v<IP_ADDRESS>80730  Skipping
auto uninstaller - No registry snapshot.  visualstudiocode has been successfully uninstalled. visualstudiocode not
uninstalled. An error occurred during uninstall:  Unable to find package 'visualstudiocode'.  Chocolatey uninstalled
0/1 packages. 1 packages failed.  See the log for details (C:\Choco_Package_Manager\logs\chocolatey.log).  Failures  -
visualstudiocode (exited 1) - visualstudiocode not uninstalled. An error occurred during uninstall:  Unable to find
package 'visualstudiocode'.  If a package uninstall is failing and/or you've already uninstalled the  software outside
of Chocolatey, you can attempt to run the command  with -n to skip running a chocolateyUninstall script, additionally
adding --skip-autouninstaller to skip an attempt to automatically  remove system-installed software. This will only
remove the packaging  files and not things like software installed to Programs and Features.  If a package is failing
because it is a dependency of another package  or packages, then you may first need to consider if it needs removed  as
it is typically installed as a dependency for a reason. If you  decide that you still want to remove it, head into
$env:ChocolateyInstall\lib and find the package folder you want  removed. Then delete the folder for the package.

Hi kishore,
The package name should be "vscode"
The second script you have attached package name you have given "visualstdiocode" please correct it try to run once
thanks
Ravi

@KishoreKrishna
Pls respond is it done or not ?

Hi Sir & Ravi
sorry for the delayed reply
Yes it is, thank you.
Things I noticed:
On running commands,
choco install visualstudiocode -y OR choco install vscode -y
package manager is installing the package VSCODE (visualstudiocode v1.28)
And when I try to uninstall using the command:
choco uninstall visualstudiocode -y
It throws an error package not found
But with the following command:
choco uninstall vscode -y
it works fine
MayBe Reason: deprecated visualstudiocode at v1.23
With command: choco install visualstudiocode -y we get v1.28 VSC, but when running the uninstallation command:
choco uninstall visualstudiocode -y it may be looking for v1.23 which is not installed, so the error package not installed.

Once again, thank you for the quick responses and your time


[SECURESIGN-1281] | Moving Sensitive Variables out of the Defaults main.yml

One question: why is this PR replacing the image urls/shas?

These are just the sha's from the main branch, my pr was from before they were updated so my sha's were older ones

LGTM now, thanks! Feel free to merge when the CI passes.

Awesome will do!


Request, Add this theme as a built in dark mode.

I was thinking that you could add this theme as a built in dark mode for people to use. The problem with just taking the CSS it that it's missing a few icons and I'm sure this would save people from having to manually create a dark mode theme.

I'm not a big fan of the theme myself, but there's not that many free Qt CSS around, so I could give it a try, as well as allowing to add resources to themes (which seem to be the issue here).

Sounds great. <3

@Bionus How is this coming along?

Cant wait

@Bionus Will this possibly be in the v5.3.0 release?

Depends on the level of complexity required to add the icons without changing Grabber's build steps.


libarm_compute-static.a export OpenCL API which is unnecessary

libarm_compute-static.a expose OpenCL API which is unnecessary, when I linked it to application which is also linked to library contains OpenCL API, it got conflicted, cannot build successfully
strings libarm_compute-static.a | grep arm_compute_version
(base) mtk31770@NB22120271:~/ComputeLibrary/build$ strings libarm_compute-static.a | grep arm_compute_version
arm_compute_version=v24.06 Build options: {'Werror': '1', 'debug': '0', 'asserts': '1', 'neon': '0', 'opencl': '1', 'embed_kernels': '1', 'os': 'android', 'arch': 'armv8a', 'benchmark_tests': '1', 'mali': '1', 'benchmark_examples': '1'}
Platform:
Operating System:
Android platform

Problem description:
the issue happens when I tried to link libarm_compute-static.a into Geekbench benchmark code
Here is part of the error log
ld.lld:error: duplicate symbol: clCreateBuffer

defined at OpenCL.cpp
OpenCL.o (clCreateBuffer) in archive third_part/acl/libarm_compute-static.a
defined at opencl_library.cpp
opencl_library.o (.text + oxCo) in archive build.android.aarch64/libhalogen.a

Could we can make OpenCL API call hidden?, I thought ACL should only export Op related class and API, not low level API

Thanks @YiGe-MediaTek for reporting it, looking into it and coming back soon.

Any update on this? Thanks!

@morgolock  Thanks for tips. I cannot move all these functions from global namespace into arm_compute, I understand where you come from. These OpenCL API is like wrapper of real implementation, if I put them into arm_compute, the linker won't find corresponding symbol, it is going to fail. BTW, libhalogen.a is module from Geekbench benchmark.

@morgolock for reproducing the issue, in the Geekbench SConstruct, add following

then just build the Geekbench, it will give u the error. Geekbench is licensed software, I cannot share with you, but ARM has the license (GB version 6.3). I also tried dynamic loading, During running, it will give bad cast error.

Hi @YiGe-MediaTek
I'd try making the opencl api symbols in ACL private moving them into the arm_compute namespace as show in the patch below and then when building libarm_compute-static.a make sure you link against libOpenCL.so or libhalogen.a.  You can do it with the following scons option  extra_link_flags="-L./libopencl/ -lOpenCL -lmali" or extra_link_flags="-L./path/to/libHalogen -libhalogen.a"
user@dev:~/work/acl/ComputeLibrary$ git diff
diff --git a/src/core/CL/OpenCL.cpp b/src/core/CL/OpenCL.cpp
index 2ebc3274a..15d94753f 100644
--- a/src/core/CL/OpenCL.cpp
+++ b/src/core/CL/OpenCL.cpp
@@ -255,7 +255,6 @@ bool opencl_is_available()

     return CLSymbols::get().clBuildProgram_ptr != nullptr;
 }
-} // namespace arm_compute

 cl_int clEnqueueMarker(cl_command_queue command_queue, cl_event *event)
 {
@@ -1257,3 +1256,7 @@ cl_mem clImportMemoryARM(cl_context                      context,
         return nullptr;
     }
 }
+
+} // namespace arm_compute

The patch above will make the symbols in ACL private.
This approach should work and all the OpenCL api call within ACL will get resolved with the symbols in libhalogen.a
Hope this helps

@morgolock Thanks a lot for helping out. Yes, either put stub function into arm_compute namespace or just remove it, it will compile as long as you build directly against the openCL library, but cl_sgemm test case will fail, somehow, it is related to tunner.
BTW, I will close this one, and work with you guys through ARM internal support. Thanks!


Search with fields separated by spaces as keywords without using regex search mode.

I want vscode to search with fields separated by spaces as keywords  without using regex search mode. For example, when I type foo bar baz in the search box, any lines that include all of these keywords will be matched.
Regards,
Zhao

How would the editor know that you don't want to search for foo bar baz literally then?

To restate or clarify my intentions, I mean, match the lines that include foo.*bar.*baz by just typing foo bar baz.

Yes, but some people are going to want to search for a sentence like asdasdad foo bar baz 23424234 and find that only with that strict sequence with nothing between the words.  Which is the current result.  So how to know which type of search is desired?

You can add fuzzy/unordered matching options to what is discussed here.

This is basically a concept for a "fuzzy text search" which has come up before, but I don't think it fits into vscode at the moment


V2

Codecov Report

Merging #11 (109974a) into main (b65d4d2) will increase coverage by 3.89%.
The diff coverage is 97.58%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   92.55%   96.45%   +3.89%
==========================================
  Files           6        5       -1
  Lines         215      141      -74
==========================================
- Hits          199      136      -63
+ Misses         10        3       -7
+ Partials        6        2       -4

Impacted Files
Coverage Δ

routine_store.go
96.07% <96.07%> (ø)

routine_g.go
96.66% <96.66%> (ø)

internal/g/g.go
100.00% <100.00%> (ø)

routine_api.go
91.66% <100.00%> (+3.78%)
:arrow_up:

routine_storage.go
100.00% <100.00%> (ø)

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 b65d4d2...109974a. Read the comment docs.


Help with running online editor with server compilation

I would like to create an online computer graphics course with Python, where users can write code in the browser, have it  compiled to Javascript by a server and see the results on a canvas (very similar to the Transcrypt Turtle demo: http://www.transcrypt.org/live/turtle_site/turtle_site.html).
I'm struggling with setting up a local server to run the Transcrypt compilation and serve the results back to the browser. I could not find anything on this topic in the docs. Is there a minimal example somewhere on how to achieve this?

Thanks a lot @JennaSys. That really helped my understanding of the client-server process. I will try to replicate this behaviour on a local machine.

Closing since no further comments.


Too aggressive with keywords

Hi... this plugin interferes with the wiki skill; I can't request an article on "pink floyd" because this intercepts the "pink". Can something be done to make this not listen when the request starts with "wiki" or "tell me about" or one of the other wiki skill triggers?

@chowbok I believe the now merged PR #10 addresses the issues you and penrods discussed. If you'd like to give it another try, I think this issue may be ready for closure.


recime-cli run gives a panic error

https://github.com/Recime/recime-template/archive/1.0.1.zip does not exist
Downloading https://github.com/Recime/recime-template/archive/1.0.1.zip
Error while creating /Users/benhalverson/recime-app/recime-1.0.1.zip - open /Users/benhalverson/recime-app/recime-1.0.1.zip: no such file or directory
panic: open /Users/benhalverson/recime-app/recime-1.0.1.zip: no such file or directory

goroutine 1 [running]:
panic(0x31c2e0, 0xc420013890)
	/usr/local/Cellar/go/1.7.3/libexec/src/runtime/panic.go:500 +0x1a1
github.com/Recime/recime-cli/cmd.check(0x4ef800, 0xc420013890)
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/src/github.com/Recime/recime-cli/cmd/init.go:122 +0x5c
github.com/Recime/recime-cli/cmd.Run(0xc4200137a0)
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/src/github.com/Recime/recime-cli/cmd/run.go:65 +0x44c
main.main.func7(0xc42009fb00, 0x533f98, 0x0, 0x0)
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/main.go:108 +0x185
github.com/spf13/cobra.(*Command).execute(0xc42009fb00, 0x533f98, 0x0, 0x0, 0xc42009fb00, 0x533f98)
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/src/github.com/spf13/cobra/command.go:636 +0x443
github.com/spf13/cobra.(*Command).ExecuteC(0xc42009fd40, 0xc420053e70, 0xc420074501, 0xc42009fd40)
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/src/github.com/spf13/cobra/command.go:722 +0x367
github.com/spf13/cobra.(*Command).Execute(0xc42009fd40, 0xc420053e68, 0x1)
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/src/github.com/spf13/cobra/command.go:681 +0x2b
main.main()
	/private/tmp/recime-cli-20161201-27896-93cjms/recime-cli-1.0.6/main.go:130 +0x76a

Hi Ben,
Thanks for Reporting it. Right now you can make it work by manually creating "recime-app" directory under /Users/benhalverson/ . We are working on a version later this evening that will resolve the issue.
Thanks
M


[Fix] Update draw_points_on_image for nuScenes

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

@Xiangxu-0103 Please reorganize the commit log

Why do the diff files have configs/pvrcnn/pvrcnn_8xb2-80e_kitti-3d-3class.py](https://github.com/open-mmlab/mmdetection3d/pull/2081/files? Pull the latest code?

This PR will be closed and new PR https://github.com/open-mmlab/mmdetection3d/pull/2160 has been created to avoid relatively complicated rebase operation.

Closed since continued in #2160


Module paths generated after downloading with jspm are corrupted

Symptoms
As an example from observable/ArrayLikeObservable.ts:
import {Scheduler} from '../Scheduler';
import {Observable} from '../Observable';
import {ScalarObservable} from './ScalarObservable';
import {EmptyObservable} from './EmptyObservable';
import {Subscriber} from '../Subscriber';
import {Subscription} from '../Subscription';

Generated JS:
/* */
"use strict";
var __extends = (this && this.__extends) || function(d, b) {
  for (var p in b)
    if (b.hasOwnProperty(p))
      d[p] = b[p];
  function __() {
    this.constructor = d;
  }
  d.prototype = b === null ? Object.create(b) : (__.prototype = b.prototype, new __());
};
var Observable_1 = require('.js');
var ScalarObservable_1 = require('./ScalarObservable');
var EmptyObservable_1 = require('./EmptyObservable');

var Observable_1 = require('.js'); is what concerns me, because those modules cann't be later properly requested for in browser:
<EMAIL_ADDRESS>Failed to load resource: the server responded with a status of 404
Versions
rxjs<EMAIL_ADDRESS>(from jspm)
TypeScript: 1.8.2 (C:\Program Files (x86)\Microsoft SDKs\TypeScript\1.8)
npm: 3.6.0
jspm: 0.16.30
Sample
TypescriptSystemRxTemplate.zip
Archive contains small template I made to reproduce the issue:

npm install
jspm install -y
typings install
gulp watch

It's weird but on some npm environments everything is generated OK and on some not, so far I've tried it on 4 npm environments. 2 OK, 2 have this issue and I wasn't able to get what was the difference between them because they all had the same versions of packages installed.

(*I'm not expert to JSPM, might be incorrect)
I've tried couple of examples and came to think this is somewhat related in JSPM or its loader. Original distribution via jspm preserves right path context such as
var Observable_1 = require('../Observable');
(npm package rxjs)
and while jspm serves it to browser, served files are augmented and losing those context, while original file still stays same and untouched.
What I found was those occur only on import name is same as path, such as
var Observable_1 = require('../Observable'); //there's path 'observable' and module 'observable'
var Subject_1 = require('../Subject'); //there's path 'subject' and module 'subject'
maybe some of loader, or jspm was not able to distinguish if module and path have same name and reside in same hierarchy.
Would you be also able to confirm my finding's same to your cases? If so, I'd like to suggest to ask JSPM (or ts loader) also to get some ideas.

Yeah, I've noticed the same pattern. I've tried to install rxjs via npm install rxjs and imports in the very same files had correct paths.
I'll probably add the same issue to jspm repo and ask why jspm install npm:rxjs works in an unexpected way and npm install rxjs works just fine.
Thank you for your time.


120s timer becomes 0ms timer

When the server is up for more than 24 days, repeating timers get bugged.

It's related to tickcount that will start giving wrong values after 24 days of server uptime, you gotta update frequently as i do or plan auto restart every 24s


yarn approve should be a top-level script

Currently, approve only exists in the @itwin/itwinui-css package.
https://github.com/iTwin/iTwinUI/blob/3b9b327ed8cf84db87dc1d61d356445cd5b051ba/packages/itwinui-css/package.json#L57
This means that we need to either cd into that workspace or use yarn workspace @itwin/itwinui-css approve, which is not very nice. We should create an alias in the root package.json to approve test images.

Closed with #778


Adding new Feature to SensorTile

Hello,
I am trying to add new features so that I can use in the BlueSTSDK iOS on the SensorTile.  I am using ALLMEMS1_ST firmware.
I have defined and added a new Service and corresponding Characteristics. I took care to follow the patterns in the ALLMEMS1_ST uuid definitions.  I didn't choose 0x10 as uuid[15] as in the example because I noticed COPY_MIC_ANGLE_UUID ( is 100000000-0000-11e1-ac36-0002a5d5c51b) in case that mattered.
I have defined as such:

Service:   00000000-000D-11e1-9ab4-0002a5d5c51b
char1:       0a000000-000D-11e1-ac36-0002a5d5c51b
-char2:       0b000000-000D-11e1-ac36-0002a5d5c51b
-char3:       0c000000-000D-11e1-ac36-0002a5d5c51b

The API is able to discover the service!  I am trying to follow "How to add a new Feature" section in the Readme.  Would you please clarify my confusions on step 1 and 2?

Steps i and ii are hard for me to understand and implement correctly. An explicit example as done in Step 2 would be very helpful for how to define MyNewFeature.
I don't see an "addFeatureForBoard" method  in the BlueSTSDK or in the documentation. The closest I see is addFeatureForNode - I tried this method following the example in this step, but it isn't working. It's likely that MyNewFeature is declared incorrectly? Or how do I properly use addFeatureForNode? Perhaps after a node is selected?

Thank you!

Hi Stave,
When you create a custom feature you have to:

extend the class BlueSTSDKFeature and implement the abstract methods
register your feature inside the Manager/Node to tell to the sdk what it has to do when it found that UUID

Create the Feature
To crate a valid extension of the class BlueSTSDKFeature you have to:

create a constructor that has a node as a single parameter. the class BlueSTSDKFeature has a constructor that accept a node and a name. so you have to write something like:

-(instancetype) initWhitNode:(BlueSTSDKNode *)node{
    self = [super initWhitNode:node name:FEATURE_NAME];
    return self;
}

or in swift
public override init(whitNode node: BlueSTSDKNode) {
    super.init(whitNode: node, name: MyFeature.FEATURE_NAME)
}

Implement the getFieldDesc method: this method is not used inside the SDK, is present because  in our idea a Feature should be self describing, so if you have a generic Feature you can query it to know witch fields it is exporting.
and implementation can be:

-(NSArray<BlueSTSDKFeatureField*>*) getFieldsDesc{
    return @[[BlueSTSDKFeatureField createWithName:FEATURE_DATA_NAME
                                                        unit:FEATURE_UNIT
                                                        type:FEATURE_TYPE
                                                         min:@FEATURE_MIN
                                                         max:@FEATURE_MAX]];
}

or in Swift:
    public override func getFieldsDesc() -> [BlueSTSDKFeatureField] {
        return [BlueSTSDKFeatureField(name: MyFeature.DATA_NAME,
                                         unit: MyFeature.DATA_MIN,
                                         type: .uInt8,
                                         min: NSNumber(value: MyFeature.DATA_MIN),
                                         max: NSNumber(value: MyFeature.DATA_MAX))]
    }

Implement the extractData method: this method is called each time the SDK receives a notification (or a read) from your feature and its responsibility is to parse the bytes and extract the actual values that the node sent, and pack it into a BlueSTSDKSample object that will be notified to the user thought the BlueSTSDKFeatureDeletage.
in theory a single characteristics can export multiple Feature, so this method has an offset parameter that tell you where the unparsed data starts.
a simple implementation can be:

-(BlueSTSDKExtractResult*) extractData:(uint64_t)timestamp data:(NSData*)rawData dataOffset:(uint32_t)offset{
if(rawData.length-offset < 2){
        @throw [NSException
                exceptionWithName:BLUESTSDK_LOCALIZE(@"Invalid data",nil)
                reason:BLUESTSDK_LOCALIZE(@"The feature need almost 2 byte for extract the data",nil)
                userInfo:nil];
    }//if

    float angle = [rawData extractLeUInt16FromOffset:offset]/100.0f;

    NSArray *data = @[@(angle)];
    BlueSTSDKFeatureSample *sample = [BlueSTSDKFeatureSample sampleWithTimestamp:timestamp data:data ];
    return [BlueSTSDKExtractResult resutlWithSample:sample nReadData:2];
}

or
    public override func extractData(_ timestamp: UInt64, data: Data, dataOffset offset: UInt32) -> BlueSTSDKExtractResult {
        if ((data.count - Int(offset)) < 2) {
            NSException(name: NSExceptionName(rawValue: "Invalid Data"),
                        reason: "No Bytes",
                        userInfo: nil).raise()
            return BlueSTSDKExtractResult(whitSample: nil, nReadData: 0)
        }
        let RSSIValue = NSNumber(value: (data as NSData).extractLeUInt16(fromOffset: UInt(offset)/100.0))
        let sample = BlueSTSDKFeatureSample(timestamp: timestamp, data: [RSSIValue])
        return BlueSTSDKExtractResult(whitSample: sample, nReadData: 2)
    }

note: the first 2 bytes of the notification will be used to extract a timestamp.. this field should be an increasing number that can be use to know if a sample is more recent than another or when the sample was acquired, if you don't care you can extend BlueSTSDKDeviceTimestampFeature instead of BlueSTSDKFeature, in this case the timestamp value will be the current time and you can parse also the first 2 bytes of the notification (offset will start from 0 instead of  from 2)

Optionally you can create some static methods that help the user to get the needed value from a sample, without knowing how the sample is organised.

for example:
+ (int32_t)getHeartRate:(BlueSTSDKFeatureSample *)sample {
    if(sample.data.count>=HEART_RATE_INDEX)
        return [sample.data[HEART_RATE_INDEX] intValue];
    return -1;
}

Register the feature
now that you have a your custom Feature you have to register it.
there are 2 way of doing that. If you are using an "ST" characteristics, so something like:
XXXXX-0001-11e1-ac36-0002a5d5c51b you have to use the addFeatureForNode before start the discovery like:
let charMap = [
            0x00080000 : MyFeature1.self,
            0x00040000 : MyFeature2.self,
            0x00008000 : MyFeature3..self
        ]
        let nodeId = 0x02
        BlueSTSDKManager.sharedInstance().addFeature(forNode: nodeId, features: charMap)

note that each key associated to a feature must have only 1 bit to 1..
By the way this is not your case and I'm thinking to deprecate this way in the next update..
In you case you have to register directly the characteristics/feature pair to the node before connecting, using the method:
(NSDictionary<CBUUID *, NSArray<Class> * > *)getManagedCharacteristics {
    NSMutableDictionary *dict = [NSMutableDictionary dictionary];
    CBUUID *uuid = [CBUUID UUIDWithString:@"0a000000-000D-11e1-ac36-0002a5d5c51b"];
    [dict add:uuid feature:[MyFeature1 class]];
 // add all your Features
    return dict;
}

....
    [node addExternalCharacteristics:[ self getManagedCharacteristics]];
...

or in Swift:
public func getManagedCharacteristics() -> [CBUUID:[AnyClass]]{
        var temp:[CBUUID:[AnyClass]]=[:]
        temp.updateValue([MyFeature1.self],
                         forKey: CBUUID(string: "0a000000-000D-11e1-ac36-0002a5d5c51b"))
 // add all your Features
        return temp;
    }
...

node.addExternalCharacteristics(getManagedCharacteristics());
...

I hope that this is more clear.
Best Regards
Giovanni

Hi Giovanni,
Thank you for responding so quickly and the the detailed description.
I am confused by Step 1 - my problem is importing initWhitNode to create the constructor properly.  I have a single view application right now for simplicity.
When I place the function from Step 1, my error is:

No visible @interface for 'UIViewController' declares the selector 'initWhitNode:name:'
I am placing this at:
`
#import "BlueSTSDKFeature.h"
...

@interface ViewController() <BlueSTSDKFeatureDelegate,......>
@end
@implementation ViewController {
BlueSTSDKManager *mManager;
NSArray *mNodes;
.....
}
.....
-(void)viewDidLoad { .....}
.
.

(void) node:(BlueSTSDKNode *)node didChangeState:(BlueSTSDKNodeState)newState
prevState:(BlueSTSDKNodeState)prevState{.....}
.
.
.
-(instancetype) initWhitNode:(BlueSTSDKNode *)node{
self = [super initWhitNode:node name:FEATURE_NAME];
return self;
}
.
...

`
Xcode doesn't seem to recognize that method in this context. I tried adding to declarations to headers and @interface and within @implementation... but that didn't make error go away.
I did notice that in viewDidLoad if I write:
mNodes[0] initWhitNode: (.....)
This method appears! But not the one with name as a function.
I am not very familiar with how to do add a method to a class, so please excuse me if this is very obvious.
Best Regards,
Steve

Hi Steve,
You don't have to create a Feature instance, the sdk will allocate it for you, when you connect to the node.
You have to create a new class different from your view controller, better if in its .h/.m file.
Take a look to a simple one like the Compass to create your one: header, source
Best Regards
Giovanni

Hi Giovanni,
I apologize for such a delayed response... Thank you for pointing me to the Compass as reference, this makes sense to me now!


Clarify half-open range definition

Rookie mistake! Thank you, posted to https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3472

I'm having a hard time with these


[TEVA-1683] Landing page descriptions

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1683
Changes in this PR

Don't assign nil parameters when stripping checkboxes of parameters that aren't there
Add page descriptions for landing pages
Fix bug where facet links used uppercase subjects and route constraints checked downcase subjects

Next steps

Map pretty params in pretty URLs to their respective subjects and job roles

Do I take it from the downcase commit that you're planning to make the urls have Title Case Subjects?

Do I take it from the downcase commit that you're planning to make the urls have Title Case Subjects?

Do I take it from the downcase commit that you're planning to make the urls have Title Case Subjects?

They already do (I just mistakenly broke that by adding the downcase bit in my previous PR)

Do I take it from the downcase commit that you're planning to make the urls have Title Case Subjects?

They already do (I just mistakenly broke that by adding the downcase bit in my previous PR)


CLI Page Generator

Running
ionic g page myPage
We get in myPage.js
import {Page} from 'ionic/ionic';
Instead of
import {Page} from 'ionic-framework/ionic';
My ionic info is
Cordova CLI: 5.3.3
Ionic Version: 2.0.0-beta.1
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
ios-deploy version: 1.8.5
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v0.12.7
Xcode version: Xcode 7.2.1 Build version 7C1002

Should be fixed in the latest release! Just a note generators are using ionic-angular as we're switching package names.


Reactify confirmation step correctly

Short Description

@nfelger  made some very good comments on my PR  #319 after I merged it: https://github.com/digitalservice4germany/steuerlotse/pull/319#pullrequestreview-797838781
This PR addresses these requested changes

Changes

The field prop types were used in multiple files. I took the freedom to also move fieldPropType into the specified file
Removed todo comment for test and instead created a ticket
Removed unnecessary messages
Moved the finish-button-text setting into the react component, away from flask
I also noticed that the messages in translations.js were still wired to the registration page, instead of the confirmation page. Fixed that too.

Feedback

Was that the place you had in mind when moving the form.finish text? Or would you have set it as a default for the props?

@JulianRoesner I think this is fine to merge but since you're assigned, do you want to take care of that?


Implementing Illuminate\Contracts\Queue\ShouldQueue on Notification class cause strange behaviour with database channel

Laravel Version: 5.4.24
PHP Version: 7.0
Pgsql & 9.5.6:

Description:
I tried to impement the contract "ShouldQueue" with the trait "Queueable" on a fresh Notification class with the channels "mail" and "database", the mail notification works great. But the column "data" of the row inserted in "notifications" table is empty. If I remove the interface "ShouldQueue" the "data" columns contains the proper json. Of course I did provide a toArray() method and even toDatabase() method.
Beside calling the notify method on a notifiable object like App\User inside the handle method of a listener implementing "ShouldQueue" cause the channel "database" not to be called.

Ok I managed to isolate the issue a little bit. It comes from the queue driver which is redis. If I switch to database it's ok. I guess the serialization/deserialization process in redis is not done right.


Work Space config.yaml

Hi,
I was trying the package and appeared to set up the config.yaml in the Work Space wrong. My Slicer reported the errors after running the embedding:
Traceback (most recent call last):
  File "C:\Users\czhan2\Desktop\SAM\samm\samm\SammBase\SammBaseLib\WidgetSammBase.py", line 112, in onPushStartMaskSync
    self.logic.processStartMaskSync()
  File "C:\Users\czhan2\Desktop\SAM\samm\samm\SammBase\SammBaseLib\LogicSamm.py", line 137, in processStartMaskSync
    memmap = numpy.memmap(self._parameterNode._workspace + '/mask.memmap', \
  File "C:\Users\czhan2\AppData\Local\NA-MIC\Slicer 5.2.2\lib\Python\Lib\site-packages\numpy\core\memmap.py", line 228, in __new__
    f_ctx = open(os_fspath(filename), ('r' if mode == 'c' else mode)+'b')
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\czhan2\\.continuum\\anaconda-client/mask.memmap'
Traceback (most recent call last):
  File "C:\Users\czhan2\Desktop\SAM\samm\samm\SammBase\SammBaseLib\WidgetSammBase.py", line 112, in onPushStartMaskSync
    self.logic.processStartMaskSync()
  File "C:\Users\czhan2\Desktop\SAM\samm\samm\SammBase\SammBaseLib\LogicSamm.py", line 137, in processStartMaskSync
    memmap = numpy.memmap(self._parameterNode._workspace + '/mask.memmap', \
  File "C:\Users\czhan2\AppData\Local\NA-MIC\Slicer 5.2.2\lib\Python\Lib\site-packages\numpy\core\memmap.py", line 228, in __new__
    f_ctx = open(os_fspath(filename), ('r' if mode == 'c' else mode)+'b')
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\czhan2\\.continuum\\anaconda-client/mask.memmap'
Traceback (most recent call last):
  File "C:\Users\czhan2\Desktop\SAM\samm\samm\SammBase\SammBaseLib\WidgetSammBase.py", line 112, in onPushStartMaskSync
    self.logic.processStartMaskSync()
  File "C:\Users\czhan2\Desktop\SAM\samm\samm\SammBase\SammBaseLib\LogicSamm.py", line 137, in processStartMaskSync
    memmap = numpy.memmap(self._parameterNode._workspace + '/mask.memmap', \
  File "C:\Users\czhan2\AppData\Local\NA-MIC\Slicer 5.2.2\lib\Python\Lib\site-packages\numpy\core\memmap.py", line 228, in __new__
    f_ctx = open(os_fspath(filename), ('r' if mode == 'c' else mode)+'b')
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\czhan2\\.continuum\\anaconda-client/mask.memmap'

Can you give me some suggestions about how to located the proper config.yaml?
Thank you very much!

Hi, thanks for trying it out. The readme was just updated (2 minutes ago). On my test environment, the TLDR instruction worked well.

Thank you very much for the update. I did get the error missing "sam_vit_h_4b8939.pth". Can you let me know where I can find "segment-anything/notebooks/sam_vit_h_4b8939.pth"?

https://dl.fbaipublicfiles.com/segment_anything/sam_vit_h_4b8939.pth


Feature: If I could control the event framenavigated of  mainFrame?

Steps to reproduce
Tell us about your environment:

Puppeteer version: 0.13.0
Platform / OS version: MAC OS 11.0
URLs (if applicable):

What steps will reproduce the problem?
Well, I use framenavigated event to inject custom js code to the browser before other js code excuted:
    page.on('framenavigated', async () => {
         await page.addScriptTag('xxxxx')
    })

Well. If this page has no other iframes. The custom js code will be injected to the browser only on time. But if the page loaded other iframes, this event would emit several times.So the custom js code will be injected the browser serveral times.
If I could control the mainFrame like:
   page.mainFrame().on('framenavigated', async () => {
        await page.addScriptTag('xxx')
   })

Well, in this way, the other iframes or frames will not affect the mainFrame. And the custom js code will be injected to the browser only one time.
What is the expected result?
the framenavigate event will be emitted only one time if this page has other iframes or frames.
What happens instead?
the framenavigate event will be emitted serveral times if this page has other iframes or frames.And the code I want to inject will be injected serveral times.

@CommanderXL The framenavigated event is passed a frame. So you can filter out those you don't like:
page.on('framenavigated', async frame => {
  if (frame !== page.mainFrame())
    return;
  await page.addScriptTag('xxxxx')
})

The framenavigated solution works but with a lag unfortunately. So if you're trying to see how the page changes from first mainframe request, say with a Mutation Observer, lots of changes are missed before the script loads.


change "park_name" to "Park_Name" like in plot01

https://github.com/benda18/Rollercoaster_Evolution/blob/26a24dda56e5497aa29fbafe9f738c8518f85b43/shiny/rollercoasters/app.R#L178

going to need to do a join at line 160-161


Using clipboard.js

Hi, I am using middleman 4, kramdown and middleman-syntax latest.
I am looking for a way to use https://clipboardjs.com/ in my middleman pages. But this requires that
the non-highlighted code syntax is present somewhere.
While when using middleman-syntax
In my documentation.md I have:

Some code example:

```js
  code code code
```

Which is then turned to:

Some code example:

  code code

So I cannot bind clipboard.js to the code example because the content is split for good, for syntax highlighting.
I would like to use the clipboard.js system to provide copy/paste on the code sample.
I am very new to middleman/ruby and I am sure this is not exactly the right place to ask this. But do you have some pointers for me?
Thanks.

With jQuery you could just do $(selector).text() which removes HTML tags.

Else, you'd need to patch the highlighter method, which returns highlighted HTML from the code argument.
You could simply append the original code as another HTML tag for copying, which would be hidden e.g. #{code}.

Do you have an example on how to monkey patch the highlighter method (and where to put it?)

With jQuery you could just do $(selector).text() which removes HTML tags.

I thougt so but I would then miss all indentation and linebreaks right?

Hey actually this works really well, silly me!
document.querySelector('.code').innerText

Will get back the indented code raw. Thanks a lot!


error

I keep getting:
node_modules/rxjs-signalr/dist/src/signalr-state.d.ts (12,47): Cannot find
namespace 'SignalR'.
I'm using it in a service in angular 4.
added
"scripts": [
"../node_modules/jquery/dist/jquery.min.js",
"../node_modules/bootstrap/dist/js/bootstrap.min.js",
"../node_modules/signalr/jquery.signalR.js"
]
to the angular.cli.json
and the other instructions you wrote in your readme
what am i missing?

@DiegoResnik do you have a repro of this I could take a look at? I know its been a while but I've got some time coming up I can take a look at this?

@DiegoResnik Do you have some repro steps for this that I could take a look at? I've got some time coming up I can take a look.


hclfmt to process also other *.hcl files

Can hclfmt also format other config files with .hcl extension?
Currently I use a script that renames all *.hcl to *.tf and uses terraform fmt to fix them.

for me, hclfmt correctly processes a file with the name region.hcl.
@ChristophShyper Can you describe WHY you need to rename your files and use terraform fmt?

But in documentation it says it only applies to terragrunt.hcl files. So maybe information should be corrected?
terraform fmt works only on .tf, .tfvars, etc. but not on .hcl, or does it now and I missed it in release notes?

I just tested, terraform fmt did NOT work on my file which is region.hcl
But terragrunt hclfmt DID work. So yes, if the docs say that it only works on terragrunt.hcl then the docs are wrong.

Ah, you're right, the docs do only mention terragrunt.hcl files: https://terragrunt.gruntwork.io/docs/reference/cli-options/#hclfmt. Is anyone up for a quick PR to fix that?

Would it be possible to add functionality to reformat all .hcl files in all subdirectories?
I have a massive tree, hundreds of terragrunt.hcl modules with many additional configuration files.
Currently I'm using my action-format-hcl for it. But it's slow when dealing with amount of files I have.

@ChristophShyper running terragrunt hclfmt formats all .hcl files in all subdirectories recursively:
terragrunt hclfmt
[INFO] Getting version from tgenv-version-name
[INFO] TGENV_VERSION is 0.29.7
INFO[0000] Formatting terragrunt.hcl files from the directory tree /projects/test.
INFO[0000] Formatting /projects/test/infrastructure/tf/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/common/root.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/prod/sns/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/common/monitoring/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/dev/monitoring/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/dev/iam/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/int/monitoring/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/int/iam/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/int/sns/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/dev/sns/terragrunt.hcl
INFO[0000] Formatting /projects/test/infrastructure/tf/common/sns/terragrunt.hcl

@kromol Thanks. Confirming it works on 0.29.10 and is very quick.
But still documentation is stating something else https://terragrunt.gruntwork.io/docs/reference/cli-options/#hclfmt

All fixed in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.30.1
Thanks a lot.


Fix more unicode errors

This diff corrects:

Incorrect parsing of Postgres' COPY FROM output for strings containing unicode, and
Incorrect processing of Python escape sequences like \n in unicode strings

Let's just port this to python3.

Closing in favor of #551


Updated some missing params and return sections in the documentation.

Added some missing parameters and return sections to the documentation.
In std.conv i've moved part of the doc block and the unittest to better document the to function within the castFrom template, a'la std.algorithm.aggregate.
All changes can be previewed here: http://nomad.so/misc/d/dlang.org/fix-ddoc-parameter-warnings/phobos-prerelease/index.html

Doc LGTM minus capitalize

Updated documentation to remove unnecessary symbol highlights in descriptions. I've updated the preview to, see the link in the original pull description.

Auto-merge toggled on


Initialize subscriptions during server startup in clustered mode

Consider following situation: two nats servers in clustered node, we also have a subscriber S connected to server A, periodically clients send PUBs to server B, server B knows about subscriber S and successfully redirects all messages to server A. But after some time server B falls and boots again now we have situation when server B can accept PUBs from new clients, but it doesn't know about subscriber S, therefore it doesn't send any message until server A doesn't send info about subscribers.  May be in clustered mode server should pull info about subscribers before it starts accept new clients?

I don't think this is feasible, or at least the use would be very limited: it may work but only for servers that have explicit routes. Even then, if server A and B have explicit route to each other, there is a race that can cause one of the server to drop the route (after it was established) to be re-established later. If a server starts accepting clients (and PUBs) after establishing a route, some PUBs could still be lost in the event I just described. And again, that would not work at all for servers just accepting routes, but not initiating them (that is, say that B does not have a route defined to connect to A, when it starts, it will accept clients right away and so PUBs could be "lost").
I would even say, how is your application going to make the difference between publishing to B while A is down (or route connection lost, and messages are necessarily not getting to S), and the situation where messages were published just as B was restarted.
I think that when you use NATS, you need to accept the fact that a PUBs can be lost. If you want added certainty of not losing the message, you should send it with a request/reply pattern.

Thanks for explanation. Now I understand that it's a bad idea.


[RT Test] Check that we can call INVOKE_DPE::DeriveChild > PL0_DPE_ACTIVE_CONTEXT_THRESHOLD times if retain_parent is not set

Closing out since this test is not needed. Even if retain_parent isn't set, we would just make the parent context retired, but retired contexts still count towards PL active context thresholds.


Labels / Axis? for matrices

Right now the matrices don't have any labels for the columns:

This is problematic - I can't read off the chart what the years actually are.
There are two approaches to solving this:

labels for each column, if possible
if we have something like years which is continuous, we could show axis or labels only for the first and last one

However, this is even more problematic for larger matrices that don't have any order semantics, such as patients. How do we want to show identifiers there?

cheap solution for now: add labels to description and show in the summary the first & last label, similar to #342

part of lineup app since matrix specific:


fix typo assitive -> assistive

Deploy preview for react-native processing.
Building with commit dee73821a47919ec88aedb960f9181c6762f9814
https://app.netlify.com/sites/react-native/deploys/5e92f15f61129d0007a6fbf8


[ExpansionPanel] adds parameter to customize expansion indicator icon

Description
Adds new parameter expandIconBuilder to customize the expansion indicator icon for ExpansionPanel. The PR continues the work of @shihaohong from #35416.
WIP: Add interactive samples to document and explain the usage of expandIconBuilder.
Related Issues
#32802
Tests
I added the following tests:

added test to check ExpansionPanel and ExpansionPanelRadio respects expandIconBuilder.
added test to check the semantics when a user provides a custom indicator icon.
added test to check hit-testing for custom indicator icon is ignored for ExpansionPanel and ExpansionPanelRadio when canTapOnHeader is true.

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

It seems like the migration to NNBD has created a ton of merge conflicts for this PR.

It seems like the migration to NNBD has created a ton of merge conflicts for this PR.

Sure, I'll solve it tomorrow morning :)

@shihaohong I have updated the PR, PTLA when you have time :)

Hi @AyushBherwani1998, this PR has not been updated in a while, and it is not in a state for code review, so I am going to close it for now. Please feel free to re-open this to continue working on it if you'd like. Thank you!

@Piinks, @AyushBherwani1998  any chances of this being implemented


Add multiple high-resolution experiments with orography in long-run buildkite pipeline.

Current Build#870 will track results for high-resolution test cases with both DCMIP and Earth-like topography, with adjusted timesteps, and with increased divergence damping.

Superseded by #1715


tvOS support

@matteocrippa I've updated CLKParametricAnimations to CalmParametricAnimations which does support tvOS, and I've updated KenBurns to support tvOS as well 👍


Feat 20 unmark student attendance

Added unmark function and improved OOP use in mark command by using Messages.java's constants

Turning into draft to add docs first

Now improving code coverage as per https://nus-cs2103-ay2324s2.github.io/website/admin/tp-w12.html#3-polish-the-deliverables

Resolves #20

Thanks for the PR! I have some nits for consideration.
Additionally, avoid adding an blank lines into files such as in AddressBook.java, Model.java ModelManager.java, that doesn't do anything style wise (according to this). While not really an coding style issue, you may want to remove these empty lines for greater consistency and readability within our codebase. You may also want to consider structuring your vertical whitespace better, as too much vertical whitespace can make code harder to read and makes our codebase less consistent overall.

Wow. Appreciate the detailed feedback Titus, I'll look through them and make the changes tomorrow


Showas option for sonoff D1 Dimmer

Please explain your feature request in a one or two sentences.

I would like to have my sonoff D1 to show as a fan in Home app (used to control the fan speed of my fireplace)

Is your feature request related to a problem? Please describe.

Not a problem as such, your plugin is great!

Any particular eWeLink devices that this relates to?

Sonoff D1

Anything else?

Never known this use case for a D1 but definitely something I can look into!

Yes, the fan is max 70W and to be used with a dimmer, so D1 can easily be used for that purpose and sonoff is by far best price for it.
Thank you in advance, looking forward for it !

Please install the beta version of the plugin and use the config 'Light Devices' section to expose the D1 as a fan type. I have not tested this as I do not have a D1, but it should work!

Great! Perfectly working and I see it now under the climate part of Home App 👌🏼
Thanks you very much

Great! Perfectly working and I see it now under the climate part of Home App 👌🏼
Thanks you very much

Would you mind providing a model number of your fan for reference. I'm just curious about what fans might work with this. Thanks.

Got it! Great! Thanks!

Hi,
Closing this just as been a little overwhelmed with issues recently.
If this is still an issue or a feature you would like implemented then please do re-open this ticket!
Thanks for your understanding!


Change position to a (next / previous) foreign key?

I feel that would be a bit more stable long term

it's not a very small change though, as it would involve changing the page iterators

closing in favor of https://github.com/SexualHealthInnovations/django-wizard-builder/issues/97


[flutter_tools] Simplify flutter test internals

This is a refactoring to simplify how flutter test runs tests with FlutterTestPlatform.
Currently, the tester can crash before and after the test harness connects to the websocket, with the use of InitialResult. The control flow is nested, resulting in the rough state diagram below:
       Start tester
            |
   Observatory connected
    /                \
Websocket connected   Tester Process Crash (1)
    |
Connect to websocket,
run test
    |       \
All ok     Tester Process Crash (2)

Instead, in this PR, we share the same crash state, which simplifies the logic and makes the workflow easier to reason about.
       Start tester
            |
   Observatory connected
    /                \
Websocket connected---\
    |                  \
Connect to websocket,---\
run test                 \
    |                     \
All ok                 Tester Process Crash

Other more minor changes:

Skip waiting for the observatory, and just wait for the remote webSocket connection because this is what is really needed
Currently, we don't call Watcher?.handleTestCrashed when the tester crashes after the harness connects to it, but we do call it when the tester crashes before the harness connects to it. This is fixed here.
We also don't call watcher?.handleStartedProcess(ProcessEvent(ourTestCount, process, processObservatoryUri)) when enableObservatory is false. The code path is only active when that is enabled, which is fixed here with the refactoring to use a helper method to deal with connecting to the observatory. Because of this, we need to fix the package:integration_test tests
Remove when from _getExitCodeMessage for simplicity, so that we don't have to track the state of the test purely for logging purposes. This doesn't seem useful because when is mainly an implementation detail. When troubleshooting (in verbose mode), the trace messages added would be much more useful in determining the state of the test, and some additional trace messages have been added.
Remove InitialResult, TestResult, introduce a private _TestHarnessStatus

Existing tests should cover this refactoring.
Related: #66264. This PR will make it easier to decouple FlutterPlatform from Process (#74236)

That sounds good. Does that mean that we should not merge this to master now? I'm not too familiar with the release process, do you know when will that be?

That sounds good. Does that mean that we should not merge this to master now? I'm not too familiar with the release process, do you know when will that be?

Yes, please hold off on merging this. We're going to cut the next release in the next few days, after which point I will let you know that this is safe to land.

Yes, please hold off on merging this. We're going to cut the next release in the next few days, after which point I will let you know that this is safe to land.

LGTM, this is good to land

LGTM, this is good to land


Taking screenshots and saving PostScript files instead of parsing messages

I’d like to try the script but I get the weirdest bug.
Instead of turning my messages into CSV files, it turns my cursor into a cross. I can then select areas of my screen. It allows me to do this five times, and will create five screenshots, each saved into PostScript files inside the running folder, respectively named sys, re, pickle, csv, and pd.
I’m guessing there is some kind of overlap between the name of a dependency and the name of a screenshot tool installed on my machine. Anyway, I never had a piece software working so remotely of what I expected it to do :)

Wow that's strange. I think you tried the parser.py script, right?
Can you check where in the script this weird behavior occurs? So try to comment something out to find the wrong lines. Thanks in advance!


Don't run RevApi on non-client SDK pipeline runs

Description
Disables running RevApi on non-client SDKs as they don't use azure-client-sdk-parent which defines the RevApi plugin.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - storage - data

/azp run java - storage - data

/azp run java - storage - data

Starting to look a little complex but looks correct. Did you also want to start passing SDKType in for some pipelines to opt them out or is that already passed through.

SDKType is being newly passed into run-and-validate-linting.yml but was already available in the other YMLs. And I agree this is a bit complex but the SDKType differentiates which parent POM is being used (azure-client-sdk-parent or azure-data-sdk-parent). Using this helps cut out different complexity in requiring a bunch of plugins to be skipped in the linting pipeline, which is easier to maintain.


[turborepo] defining passThroughEnv breaks next.js internal revalidate api

What version of Turborepo are you using?
1.10.12
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Mac
Describe the Bug
I'm not sure whether this issue belongs to the next.js, or the turbopack repository.
I recently ran into a weird bug where the nextjs on-demand revalidate api suddenly was broken. Calling the response.revalidate() would throw an error which, at least to me, was not helpful at all.
error:
Error revalidating: Failed to revalidate /test: Cannot find module '/var/task/apps/web/.next/server/pages/test.js'\nRequire stack:\n- /var/task/node_modules/.pnpm/next@13.4.19_react-dom<EMAIL_ADDRESS>/var/task/node_modules/.pnpm/next@13.4.19_react-dom<EMAIL_ADDRESS>/var/task/apps/web/___next_launcher.cjs

After some debugging I was able to figure out that setting the passThroughEnv array was the culprit.
Expected Behavior
I expect the next.js on-demand revalidate api to keep working when I configure passThroughEnv variables in turbo.
To Reproduce
See bug in action: https://turborepo-next-revalidate-bug-git-main-klaasman.vercel.app/api/revalidate?path=/test
(here is a working preview deployment w/o the "passThroughEnv" array, introduced in this commit)
Reproduce:

Clone repo https://github.com/klaasman/turborepo-next-revalidate-bug
deploy to vercel
try to revalidate the test page by visiting /api/revalidate?path=/test

Reproduction Repo
https://github.com/klaasman/turborepo-next-revalidate-bug

Ooh this is a good one. Thanks for the detailed repro / project! I'm chatting with the Next.js team to see exactly what might be missing here and will follow up.

Alright @klaasman, quick unblock is to make sure the $NOW_BUILDER env var can make it to the build. In your case: "passThroughEnv": ["NOW_BUILDER"] (sent a PR to your reproduction repro)
We're going to need a better way to support this in the future with some type of automatic framework config

Great, can confirm that helps works 👍

We're going to need a better way to support this in the future with some type of automatic framework config

Doesn't turbo already have "Automatic environment variable inclusion"?

Yea good question - it does, and this is all you need in most cases. However the automatic variable inclusion relies on the environment variable using a framework prefix (we detect your framework, and then use that to allow all correct the env vars in).
This var is internal to the Vercel build process and is used to communicate features that are supported to Next. So we're looking at some other solutions to make this one work. This is likely going to be something that changes on our end and will "just work" for you when it's out.

Great, good luck fixing this bug and glad I could help to surface it 👍

😄 thanks! I'm going to close this one since the specific issue has a resolution, but we're tracking this internally to determine a fix.


No error highlighting

Error highlighting is complete gone. Intellisense, tooltips, LineLens and ect. works fine.
To Reproduce
Steps to reproduce the behaviour:

Create a .fsx file
Error highlighting does not work

Expected behaviour
Red line under errors in code.
Screenshots

Environment (please complete the following information):

OS: Mac OS 10.14.6
Ionide version: 4.1.0
VSCode version: 1.38.1
dotnet SDK version: 3.0.100
mono: <IP_ADDRESS>

Duplicate of #1220


API Deprecate unused / undesirable create_new_password implementation

This method isn't used by any core components any more, and has been since superseded by more secure behaviour.

Could we merge this?


[FR]: Show Caddy Certificates Expiry Dates

Describe the new feature or enhancement idea
I have the integration setup and working (everything up to date). I'm using Caddy for reverse proxy Let's Encrypt certificates. On the OPNsense Lobby Dashboard there's a widget which states when the certificate expires. Is it possible to have this data communicated using the OPNsense plugin > Home Assistant?
Will you be building this?

[ ] I intend to submit a PR to implement this

Additional Details
No response

Certificate Sensor added in v0.3.8

I think OP refers to Caddy certificates, not OPNsense. For Caddy, there actually is an API: https://docs.opnsense.org/development/api/plugins/caddy.html
@n13ldo can you please check with Postman the API /caddy/diagnostics/certificate and post the result?

I think OP refers to Caddy certificates, not OPNsense. For Caddy, there actually is an API: https://docs.opnsense.org/development/api/plugins/caddy.html
@n13ldo can you please check with Postman the API /caddy/diagnostics/certificate and post the result?

POSTMAN POST: https:///api/caddy/diagnostics/certificate
Response:
{
"status": "success",
"content": [
{
"hostname": "wildcard_.",
"expiration_date": "Dec 12 10:14:59 2024 GMT",
"remaining_days": 39
},
{
"hostname": "wildcard_.",
"expiration_date": "Dec 12 10:14:58 2024 GMT",
"remaining_days": 39
},
{
"hostname": "wildcard_.",
"expiration_date": "Dec 12 10:14:58 2024 GMT",
"remaining_days": 39
}
]
}

Thanks. I edited your post with edit tags of json type for better readability.
Now it's up to @Snuffy2, if he decides to implement this.

Sorry missed that you wanted Caddy certs and not OPNsense certs.
Out of curiosity, don't your certs auto-renew?  I use Traefik instead of Caddy (on a separate machine) and they auto renew so I don't need to do anything about them. I understood that Caddy did the same.

Sorry missed that you wanted Caddy certs and not OPNsense certs.
Out of curiosity, don't your certs auto-renew? I use Traefik instead of Caddy (on a separate machine) and they auto renew so I don't need to do anything about them. I understood that Caddy did the same.

No problem. Apparently Caddy does auto-renew but I'm within their first 90 day block and hoping no intervention needed.

No problem. Apparently Caddy does auto-renew but I'm within their first 90 day block and hoping no intervention needed.

I'd say you put a reminder in HA (todo list) to track the first 90 days. Or, you can also create a REST sensor in HA, you know how to get the JSON data needed.
It's not something that is generally needed by user, so I'll close the FR.

No problem. Apparently Caddy does auto-renew but I'm within their first 90 day block and hoping no intervention needed.

I'd say you put a reminder in HA (todo list) to track the first 90 days. Or, you can also create a REST sensor in HA, you know how to get the JSON data needed.
It's not something that is generally needed by user, so I'll close the FR.

Excellent suggestion. I use Node-Red for most stuff inside HA so will go that route. Thanks for all the help people.

I use Node-Red for most stuff inside HA

With NR it will be very easy. I use it too for some automations.


Replace web framework

I expect
The web framework to be updated
Note
The current framework is vulnerable

fixed in #5


Simplify ROCm install guide

Jenkins, test this please

Jenkins CI test (for commit 7f4f08a967e56e9db7bea643fdca1034eabf85c6, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 7f4f08a967e56e9db7bea643fdca1034eabf85c6, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 7f4f08a967e56e9db7bea643fdca1034eabf85c6, target branch master) succeeded!


Enhanced C++ implementation

The NMS-Resampling method is very useful and easy to understand.
However, I have found that applying the official implementation in Python (see here) really slows the training process.
I have managed to implement it in C++ and integrate it in mmcv, which in my case gives between 3x and 4x speed-ups in the whole training process.
If you are interested, I can create a pull request with the code and the instructions to make use of this faster implementation. I think that this make help pepople using this useful tool.
Best regards!

Thanks for your interests! Sure, it is great to integrate your better implementation into this repo if it has no compatibility issues. Thanks for your contribution! Did you integrate NMS sampling into official mmcv or mmdet repo?

I have not integrated in the official repositories, but it should be something easy to do.
Indeed, it could be interesting to try a pull request to those repositories in the future.
Regarding this repo, I would prepare a PR (inlcuding the code and instructions) in the following weeks and let you know once it's ready.

Pull request merged and thus close the issue


Jasmine running despite no GPS data

Commit hash (the first line of output from git show): 7fceb601a5083848190c25acb041f977e8e0b6ca
Describe the bug
Jasmine is attempting to run forest on an individual despite recognizing that the user does not have sufficient data
To reproduce
import sys
beiwe_ids = ["g4wkenld"]
study_id = "B6GXRdjYVcnObzvs2eVh3Czc"
dest_dir = "test_data"
server = "studies"
time_start = "2008-01-01"
time_end = None
data_streams = ["gps"]
import .py file located in another directory if needed
import mano
sys.path.insert(0, '/Volumes/SanDisk/onnela_lab/keyring_files')
import keyring_studies
kr = mano.keyring(None)
import os
from helper_functions import download_data
download_data(kr, study_id, dest_dir, users = beiwe_ids, time_start = "2008-01-01",
time_end = None, data_streams = None)
import forest.jasmine.traj2stats
from forest.jasmine.traj2stats import Frequency
output_dir = "gps_output"
tz_str = "America/New_York"
option = Frequency.BOTH
save_traj = True
places_of_interest = None
place_threshold = None
save_log = False
forest.jasmine.traj2stats.gps_stats_main(
dest_dir, output_dir, tz_str, option, save_traj,
save_log = save_log,places_of_interest = places_of_interest,
threshold = place_threshold, participant_ids = beiwe_ids
)
Observed behavior
Downloading data for g4wkenld
User g4wkenld: There are no gps data in range.
User: g4wkenld
Read in the csv files ...
AttributeError                            Traceback (most recent call last)
Input In [4], in
33 place_threshold = None
34 save_log = False
---> 37 forest.jasmine.traj2stats.gps_stats_main(
38     dest_dir, output_dir, tz_str, option, save_traj,
39     save_log = save_log,places_of_interest = places_of_interest,
40     threshold = place_threshold, participant_ids = beiwe_ids
41 )
File /Volumes/SanDisk/opt/anaconda3/envs/python_3_8/lib/python3.8/site-packages/forest/jasmine/traj2stats.py:1000, in gps_stats_main(study_folder, output_folder, tz_str, frequency, save_traj, parameters, places_of_interest, save_log, threshold, split_day_night, person_point_radius, place_point_radius, time_start, time_end, participant_ids, all_memory_dict, all_bv_set, quality_threshold)
998     parameters.h = parameters.r
999 if orig_w is None:
-> 1000     parameters.w = np.mean(data.accuracy)
1001 # process data
1002 mobmat1 = GPS2MobMat(
1003     data, parameters.itrvl, parameters.accuracylim,
1004     parameters.r, parameters.w, parameters.h
1005 )
File /Volumes/SanDisk/opt/anaconda3/envs/python_3_8/lib/python3.8/site-packages/pandas/core/generic.py:5583, in NDFrame.getattr(self, name)
5576 if (
5577     name not in self._internal_names_set
5578     and name not in self._metadata
5579     and name not in self._accessors
5580     and self._info_axis._can_hold_identifiers_and_holds_name(name)
5581 ):
5582     return self[name]
-> 5583 return object.getattribute(self, name)
AttributeError: 'DataFrame' object has no attribute 'accuracy'
Expected behavior
Jasmine should stop running after it returns the warning User g4wkenld: There are no gps data in range.
Additional context
After we fix this, we should add a test that verifies this behavior so it is not inadvertently reintroduced.

Hi @clementzach, I am unable to replicate the data using your code above. I wanted to ask when you download the data for the user how does his directory structure look like? Is there an empty gps folder or there is no such folder at all?


Implement vendor prefixes

NOT HIGH PRIORITY
http://microformats.org/wiki/microformats-2#VENDOR_EXTENSIONS
would be cool if something special happened

As of today, this is complete and released


Fix DAGCircuit.collect_runs()

Summary
Fixes #1871 Fixes #1820
dag.collect_runs() should only collect runs of gates that are non-conditional, otherwise the optimizations that are based on them won't work.

Do you mind writing a test that failed master but passes with your PR?

@1ucian0 test added.


Microsoft.AspNetCore.NodeServices.HostingModels.NodeInvocationException: No NgModule metadata found for 'AppModule'

hi guys,
when i building the project: npm run build:prod or npm run build:vendor
and then try to run the project: dotnet run
im getting this error (in run time):
Microsoft.AspNetCore.NodeServices.HostingModels.NodeInvocationException: No NgModule metadata found for 'AppModule'.
Error: No NgModule metadata found for 'AppModule'.
any ideas?

Its still not working since there are other issues, but, you shouldnt use npm run build:vendor after npm run build:prod as you are rebuilding vendors again without prod mode
this is what you should use
npm run build:prod
dotnet run


Haven't got permissions yet

Hello, last week you replied to me:

Please use this link, the same as the link in the readme, to visit: https://drive.google.com/drive/folders/1SH8HWu8DQtwUgSFoIJOL8vJEvgFptBCD?usp=sharing

I've been applying on GoogleDrive for a week but still don't get permission, hoping to hear from you, thank you.

Sorry for the waiting. We have updated the access rule of that folder. You should now be able to visit and download the files.


p-s to react on pushes to an upstream branch

This request is coming from libguestf family of projects:
https://github.com/libguestfs
Their workflow is to review patches on ML and have a tooling to push changes to the github git repos. This project is not using issues nor pull requests. Hence the project requires to have a trigger on push to a branch instead of reacting to PRs
CC @rwmjones

AI: implement this - test pushes to a branch (configurable), set commit status once the build & tests are done
AI: create an issues if something fails (configurable) - stretch goal

@lachmanfrantisek isn't this done?

@lachmanfrantisek isn't this done?

Not yet.

[ ] Turn on the events in the production github app
[ ] Finish the database update. Otherwise, only the baby-sit task will return the responses.

good points, thanks for the update!

Hi guys, how does this look like?

@bocekm Sorry, we were very busy with the centos-stream in the last two weeks. I would like to finish that in the next week.

Finally, we are there!
The workflow and configuration are the same as for pull-requests. Just use trigger: commit in the job configuration. (More details can be found in the documentation.)
- job: copr_build
  trigger: commit
  metadata:
    branch: build-branch

The results are shown as commit status. (You can access it with selecting the branch in the commits tab on GitHub.)

An example build for new commit to the branch can be found here:
https://github.com/packit-service/hello-world/commits/build-branch

Build triggered. Building SRPM.

SRPM successful. Building RPM in Copr.

Build successful. Running tests in Testing Farm.

Tests passed.

(For screenshots I used the staging instance of packit-as-a-service but it's now live on prod as well.)
You can use custom project name as well:

https://github.com/packit-service/hello-world/commit/9d15a951d64754a5ebbc777031b70de2d15a21fe
https://copr.fedorainfracloud.org/coprs/packit/the-build-branch-test/build/1419637/

@lachmanfrantisek I'm so happy we finally have this capability \o/
Franta, that write-up is so fancy! Would you be interested in adding it to docs?

I need to calm your happiness, we have a small problem with the job-config matching when you combine more triggers and build configs with test ones (e.g. current state in packit).
I am working on the fix now.

@lachmanfrantisek, I'm not sure if your previous comment relates to this, but the status of the copr build is not updated for the commit. It says "Pending — Starting" event though the build has succeeded.
To be seen for example here: https://github.com/oamg/convert2rhel/commit/2b0d22e90a7de2849301ebfb1373b728aa9d014d

@lachmanfrantisek, I'm not sure if your previous comment relates to this, but the status of the copr build is not updated for the commit. It says "Pending — Starting" event though the build has succeeded.
To be seen for example here: oamg/convert2rhel@2b0d22e

This should not be that problem. (If you hit it, it will not do anything.) The fix (#648) is on stage now.
I don't see anything related in Sentry. Maybe some fedmsg issue. I'll take a look to logs.

@bocekm I've reproduced that and created an issue for that:
https://github.com/packit-service/packit-service/issues/650
I'll take a look tomorrow.


feat(module/virtual_network_gateway): New module for Site-to-Site VPN connections

Description
PR delivers new module for Site-to-Site VPN connections, which internally:

creates Azure Virtual Network Gateway
creates Azure Local Network Gateway (which in fact is customer gateway)
creates VNG connections

Motivation and Context
#303
How Has This Been Tested?
Code was tested by deploying VM-Series in common design.
In GCP cloud it was created another environment with Panorama .
Then there were established tunnels between Azure (using that module) and GCP and VM-Series from Azure were bootstrapped by Panorama in GCP.
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

@FoSix according to our offline discussion, I've updated PR about:

adding azure_bgp_peers_addresses, which is a map of addresses used on Azure side (then in other parts of configuration we are referencing key)
introducing map for peering_addresses in local_bgp_settings (instead of list), in which also we are referencing to keys for IP address
merging changes from main branch e.g. removing Makefile and adding Terratest

/help

/sca