"Machine Learning Model (ML.NET)" does not appear in the new item list or is not created by Visual Studio for the Console App (C#)

System Information (please complete the following information):

Model Builder or CLI Version: 10.0.22621.2506 (product version of cmd.exe/Command Prompt)
Visual Studio Version (if applicable): Microsoft Visual Studio Community 2022 (64-bit) - Current Version 17.9.3

Describe the bug

On which page of the tutorial did you run into an issue (URL): https://dotnet.microsoft.com/en-us/learn/ml-dotnet/get-started-tutorial/create
Clear description of the problem: "Machine Learning Model (ML.NET)" does not appear in the new item list or is not created by Visual Studio for the Console App, when I follow the path:
right click on Console App > Add > Machine Learning Model.

To Reproduce
Steps to reproduce the behavior: previous explanation ↑
Expected behavior
To see "Machine Learning Model (ML.NET)" item on the list.
Screenshots

Additional context
When I try to re-install the ML.NET Model Builder 2022, it says this.

But when I go to extension list it does not show up there.

Maybe go check in VS installer to see if you can find model builder there?

It was installed on the machine and still is.
SS 1 (VS is up-to-date)

SS 2 (This looks suspicious: "Total space required 485 MB")

And it still does not appear in the new item list or is not created by VS 22 IDE for the Console App (C#)

I am also experiencing this issue.

@dansimon363
Can you try the following steps and let us know the results

create an empty file with extension “.mbconfig” and see if you can double click and open it
install model builder from gallery and see if it works

Thanks

Hi @LittleLittleCloud, thanks for your response. I created an empty file with the extension ".mbconfig" and this is what appeared:

I clicked OK and then this page appeared.

I was able to successfully train some data so it looks like this is a valid workaround for me. Thank you! 😃

In Extensions -> Manage Extensions, ML.NET Model Builder 2022 is listed under installed.

I can see it under Browse.

I get this message after clicking Install:

I originally installed the extension from here: https://marketplace.visualstudio.com/items?itemName=MLNET.ModelBuilder2022&WT.mc_id=dotnet-35129-website.

I tried using a console application that targets .NET Framework 4.7.2 and I don't see "Machine Learning Model (ML.NET)" item in the list. I also tried a different console application targeting .NET 8.0 as well and this produced the same behavior.

@dansimon363 Thanks for verifying the workaround. It means that the model builder is installed on your VS, it's just the new item dialog doesn't show .mbconfig item correctly.
We will try reproduce this error on our side and working on a fix for this bug.

You are welcome, sounds good and thank you.

In Visual Studio version 17.10.4, I can no longer reproduce this issue.


Update theme Doks from h-enk/doks to gethyas/doks

Dunno why the build fails with only this edit?

Related? https://github.com/gohugoio/hugoThemesSiteBuilder/issues/286 + https://github.com/gohugoio/hugoThemesSiteBuilder/issues/287?

Dunno why the build fails

From the Netlify log:
2:57:37 PM: Get Hugo Modules, config "config.json"
2:57:38 PM: verifying<EMAIL_ADDRESS>checksum mismatch
2:57:38 PM: 	downloaded: h1:kWbaK5JXp1sIqzOsrcS6cWw+OlzMjaFOIH1ZLsIu99g=
2:57:38 PM: 	go.sum:     h1:8Vy+1nQCgKgr9aDFZ4pAUjp0sVE/ZzXgsRrqCYehuaA=
2:57:38 PM: SECURITY ERROR
2:57:38 PM: This download does NOT match an earlier download recorded in go.sum.
2:57:38 PM: The bits may have been replaced on the origin server, or an attacker may
2:57:38 PM: have intercepted the download attempt.
2:57:38 PM: For more information, see "go help module-auth".


Package Initialization Error

Diagnostics
Request: GET https://api.teletype.atom.io/protocol-version

Versions
Teletype version: v0.13.4
Atom version: 1.49.0
Platform: win32

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way 😀


Multiple alerts

How to send multiple alerts using a single rule in elastalert ?

alert:
 - email
 - slack
 - whatever

Umm...that's fine but what if I need to apply frequency checks on two different fields in the same rule and send different alerts for the different frequency checks. Is this possible in a single rule?

You can use query_key on a specific field to have the frequency checks run separately (num_events will be counted independently for each value of the query_key field). If you want two separate num_events for two separate fields, you need two different rules.

I was able to do multiple alerts just fine. I did hipchat + pagerduty. I just had to make sure I had all the fields required for each alert mechanism. @nickedes can we close this?

@Qmando is there no way I can have separate num_events for different fields in the same rule :disappointed: ?
Because i have to create alerts for 50 different values of a field, and fire alert for each of them when threshold value is crossed.

Anyways I have fulfilled by requirement by hitting on the elastic index using curl in my script.
Thanks.


Remove initContainers in Dev in Helm

We migrated from AWS DocumentDB to MongoDB Atlas
The initContainers are no longer needed

This will resolve the failed CDs


Sending cause error on certificate (unknown_ca)

when calling deliver(swoosh_email) :
{:error, {:tls_alert, {:unknown_ca, 'TLS client: In state certify at ssl_handshake.erl:1895 generated CLIENT ALERT: Fatal - Unknown CA\n'}}}

It appends right after a new deploy, was working fine before.
Is it a problem from hackney certificate became obsolete ? How should I fix this right ?
Thanks a lot !

I "fixed" it with this patch:
      email
      [....]
      |> Swoosh.Email.new()
      |> Map.put(:private, %{hackney_options: @hackney_options})

And in my Dockerfile I added:
RUN update-ca-certificates
RUN ln -fs /etc/ssl/certs/ca-certificates.crt /etc/ssl/cert.pem

The elixir Swoosh config part I did was quite dirty, how to make it sweet ? :)

Hi, this happened to me once. I simply redeployed and it seemed to be "fixed". I'm not exactly sure what happened.
I guess maybe I should switch from hackney to Finch.

So short answer is you may not need to do what you did.
Longer answer is I don't know what really happened. Long term speaking it might be safer to base the http client on a mint based library, which finch is one, that would use CA information from the castore package. I might make a switch at some point.

Are you on Mailjet? Someone on Slack pinged me saying they have this issue.


ci(circleci): add node13 for tests

Which problem is this PR solving?

closes #458

Short description of the changes

Tests with node 13

Codecov Report

Merging #805 into master will increase coverage by 3.16%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #805      +/-   ##
=========================================
+ Coverage   92.53%   95.7%   +3.16%
=========================================
  Files         242     166      -76
  Lines       10630    5751    -4879
  Branches     1049     469     -580
=========================================
- Hits         9837    5504    -4333
+ Misses        793     247     -546

Impacted Files
Coverage Δ

...telemetry-scope-base/test/NoopScopeManager.test.ts
100% <0%> (ø)
:arrow_up:

...metry-core/test/context/BinaryTraceContext.test.ts
100% <0%> (ø)
:arrow_up:

...s/opentelemetry-core/test/trace/tracestate.test.ts
100% <0%> (+34.93%)
:arrow_up:

...emetry-exporter-stackdriver-trace/src/transform.ts
100% <0%> (ø)
:arrow_up:

...kages/opentelemetry-plugin-dns/test/utils/utils.ts
33.33% <0%> (-26.67%)
:arrow_down:

...elemetry-tracing/src/export/SimpleSpanProcessor.ts
100% <0%> (ø)
:arrow_up:

...ackages/opentelemetry-core/test/utils/wrap.test.ts
80.64% <0%> (ø)
:arrow_up:

packages/opentelemetry-api/src/api/metrics.ts
53.33% <0%> (ø)
:arrow_up:

...metry-core/src/trace/instrumentation/BasePlugin.ts
81.57% <0%> (-5.27%)
:arrow_down:

...ges/opentelemetry-exporter-zipkin/test/e2e.test.ts
100% <0%> (ø)
:arrow_up:

... and 198 more

This looks great 💯. Just want to make sure we aren't utilizing too much CircleCI resources otherwise it might result in extensive queuing for the entire otel organization.

I think in general we should do LTS releases and latest. So when 14 comes out we should drop 13 in favor of that LTS release. Eventually we should drop 8 too because it is out of official node support.

I think in general we should do LTS releases and latest. So when 14 comes out we should drop 13 in favor of that LTS release. Eventually we should drop 8 too because it is out of official node support.

I'm agree


Dreamview页面中画出的Point Cloud与carla中激光雷达点云整体存在90度差异

Describe the bug
辛苦，辛苦！
我在使用中发现，Dreamview中打开Layer Menu/Perception/Point Cloud选项后，页面中画出的点云和carla实际送入Apollo算法的点云似乎存在90度的整体差异，如下图所示：

绿色圆圈中显然是前方车辆的点云，但是通过绿色3d bbox可知算法已经给出了车辆在前方的感知结果。
如果我在objects.json文件中，将ego_vehicle的lidar的yaw角设成-90，则点云整体旋转了90，dreamview页面中画出的点云能够和实际情况对应，但是感知结果的3d bbox又向左旋转了90度，如下图所示：

In what area(s)?

/area runtime

/area operator

/area placement

/area docs

/area test-and-release

Try to modify the following content  in src/carla_cyber_bridge/lidar.py, you can see if it meets your expectations.
        lidar_data[:, 1] *= -1

#        idar_data = lidar_data[:, :3]
#        lidar_data[:, 0:2] *= -1
#        # we also need to permute x and y
#        lidar_data = lidar_data[..., [1, 0, 2]]

Try to modify the following content in src/carla_cyber_bridge/lidar.py, you can see if it meets your expectations.
        lidar_data[:, 1] *= -1

#        idar_data = lidar_data[:, :3]
#        lidar_data[:, 0:2] *= -1
#        # we also need to permute x and y
#        lidar_data = lidar_data[..., [1, 0, 2]]

Thanks. This code works for me.

I verified it, and I did solve my problem.

What was the transformation that worked for you?

What was the transformation that worked for you?

Actually the solution in the comments in this issue didn't work for me. But I didn't work on it after all.


Installing packges from requirements.txt complains about require-hashes and some packages not having their versions pinned

pip install -r requirements.txt using the file generated by pipenv lock -r complains about some packages not having their version pinned with ==. Among them, setuptools and six.
It may be related to #1417
Describe your environment

OS Type: MacOS 10.12.6 (with MacPorts)
Python version: 3.6.4
Pipenv version: 9.1.0 (installed from master, on Feb, 18th)

Expected result
I expected the all packages being installed in the target directory with versions coherent with Pipfile.lock
Actual result
In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    setuptools>=0.6b1 from https://pypi.python.org/packages/43/41/033a273f9a25cb63050a390ee8397acbc7eae2159195d85f06f17e7be45a/setuptools-38.5.1-py2.py3-none-any.whl#md5=908b8b5e50bf429e520b2b5fa1b350e5 (from injector==0.13.4->-r target/requirements.txt (line 5))
    six from https://pypi.python.org/packages/67/4b/141a581104b1f6397bfa78ac9d43d8ad29a7ca43ea90a2d863fe3056e86a/six-1.11.0-py2.py3-none-any.whl#md5=866ab722be6bdfed6830f3179af65468 (from purl==1.3.1->-r target/requirements.txt (line 6))

Steps to replicate
With this Pipfile
[[source]]

url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[packages]

purl = "*"
requests = "*"
python-dotenv = "*"
injector = "*"
voluptuous = "*"

[dev-packages]

"boto3" = "*"
pytest = "*"
pytest-mock = "*"
selenium = "*"
bpython = "*"

Commands
mkdir target
pipenv lock -r > target/requirements.txt
# with pip executed from a shell created with pipenv shell or from a new one
pip install -r target/requirements.txt -t target

Same here. In my case, there is a line in the requirements.txt that starts with #, quite suspicious.
bugsnag==3.4.0 --hash=sha256:07e4cce9aad5a41b9c21410102cfd4bc635e93facf2162746a6cda3d61e35ba3  --hash=sha256:5f7d20fe37183030b42570c2e5206949f3bae9fbe5f4fdbbe709d963d9666c9d  --hash=sha256:7fc68389bc1fb02ab77e1a65a1b4e21fd3cce752d44ed6c418a6bdc1dc1ab89f
certifi==2018.1.18 --hash=sha256:14131608ad2fd56836d33a71ee60fa1c82bc9d2c8d98b7bdbc631fe1b3cd1296  --hash=sha256:edbc3f203427eef571f79a7692bb160a2b0f7ccaa31953e99bd17e307cf63f7d
chardet==3.0.4 --hash=sha256:fc323ffcaeaed0e0a02bf4d117757b98aed530d9ed4531e3e15460124c106691  --hash=sha256:84ab92ed1c4d4f16916e05906b6b75a6c0fb5db821cc65e70cbd64a3e2a5eaae
idna==2.6 --hash=sha256:8c7309c718f94b3a625cb648ace320157ad16ff131ae0af362c9f21b80ef6ec4  --hash=sha256:2c6a5de3089009e3da7c5dde64a141dbc8551d5b7f6cf4ed7c2568d0cc520a8f
pymysql==0.8.0 --hash=sha256:04fa19fad017fdb21394fad2878c1d6bd346959d4fbfd1b66050a09fc636a321  --hash=sha256:32da4a66397077d42908e449688f2ec71c2b18892a6cd04f03ab2aa828a70f40
requests==2.18.4 --hash=sha256:6a1b267aa90cac58ac3a765d067950e7dbbf75b1da07e895d1f594193a40a38b  --hash=sha256:9c443e7324ba5b85070c4a818ade28bfabedf16ea10206da1132edaa6dda237e
# six==1.11.0 --hash=sha256:832dc0e10feb1aa2c68dcc57dbb658f1c7e65b9b61af69048abc87a2db00a0eb  --hash=sha256:70e8a77beed4562e7f14fe23a786b54f6296e34344c23bc42f07b15018ff98e9
urllib3==1.22 --hash=sha256:06330f386d6e4b195fbfc736b297f58c5a892e4440e54d294d7004e3a9bbea1b  --hash=sha256:cc44da8e1145637334317feebd728bd869a35285b93cbb4cca2577da7e62db4f
webob==1.7.4 --hash=sha256:63f4220492476c5c716b615baed7bf3d27040b3105014375787160dee0943115  --hash=sha256:8d10af182fda4b92193113ee1edeb687ab9dc44336b37d6804e413f0240d40d9

Btw, if there was an option for having pipenv install installing to a specific folder I would not be using pipenv lock -r.

Happening to me as well. We do builds in containers and ship in a requirements file as the builder doesn't respect pipenv yet. This change broke our build workflows pretty much across the board since just about everything in the world seems to have six as a dependency somewhere

Aside - it’s a little hacky, but you can construct your own virtualenv,
source that and then use pipenv to install there (thereby getting custom
output folder) or alternatively use --system or --python
On Tue, Feb 20, 2018 at 3:00 PM Bruce Stringer<EMAIL_ADDRESS>wrote:

Happening to me as well. We do builds in containers and ship in a
requirements file as the builder doesn't respect pipenv yet. This change
broke our build workflows pretty much across the board since just about
everything in the world seems to have six as a dependency somewhere
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pipenv/issues/1449#issuecomment-367151380, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABhjq0nYGOO2FNfguwDxCVflgLN00uBBks5tW06QgaJpZM4SJ4Fq
.

(In response to “Btw, if there was an option for having pipenv install
installing to a specific folder I would not be using pipenv lock -r.”)
On Tue, Feb 20, 2018 at 3:03 PM Jeffrey Tratner<EMAIL_ADDRESS>wrote:

Aside - it’s a little hacky, but you can construct your own virtualenv,
source that and then use pipenv to install there (thereby getting custom
output folder) or alternatively use --system or --python
On Tue, Feb 20, 2018 at 3:00 PM Bruce Stringer<EMAIL_ADDRESS>wrote:

Happening to me as well. We do builds in containers and ship in a
requirements file as the builder doesn't respect pipenv yet. This change
broke our build workflows pretty much across the board since just about
everything in the world seems to have six as a dependency somewhere
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pipenv/issues/1449#issuecomment-367151380, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABhjq0nYGOO2FNfguwDxCVflgLN00uBBks5tW06QgaJpZM4SJ4Fq
.

This seems like it needs to be addressed with some expedience. I haven't looked that carefully at the change but I assume if anyone can pinpoint and isolate the root that would help

Current workaround is to run Pipenv 9.0.3.

@techalchemy It looks like it was introduced in this commit https://github.com/pypa/pipenv/commit/9f41427a678e816538cec891f2e655e05a7eaad3
I'm not familiar with the issue that kenneth was trying to work around here but I assume there as a good reason but it is causing issues as it changed expected behavior.

this should now be fixed


different account for notebooks

currently, your repo needs a PAT of @ckolbPTB to publish the notebooks.
maybe we can create a github account PTBMRBot for these commits?

and / or set the commiter and author of these commits to github-actions https://github.com/actions4git/add-commit-push

sounds reasonable to me 👍

@schuenke @ckolbPTB


Add ci

I didn't want to fight with doing the actual release. If someone wants to remember how to do the Makefile stuff I'll happily take it :)
But this should add some transparency

LGTM


Fail CI when codecov upload failed

The codecov upload step will still show success when coverage report uploading is actually failed, so set the fail_ci_if_error to true to make sure developers be aware that the report upload job need to be re-run when it's failed.
Signed-off-by: Lan Luo<EMAIL_ADDRESS>

@tnqn @wenqiq could you take a look at this PR? thanks.

@tnqn @antoninbas could you help to check and move forward? thanks.


Show an incorrect icon for one ADLD Gen2 folder

Storage Explorer Version: 1.12.0
Build: 20200226.4
Branch: master
Language: Czech/ German/ Spanish/ French/ Hungarian/ Italian/ Japanese/ Korean/ Dutch/ Portuguese (pt-BR)/ Portuguese (pt-PT)/ Russian/ Swedish/ Turkish/ Chinese (zh-CN) / Chinese (zh-TW)
Platform/OS: Windows 10/ Linux Ubuntu 18.04
Architecture: ia32/x64
Regression From:  Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select '中文（简体）' -> Restart Storage Explorer.
Open one ADLS Gen2 blob container -> Upload one folder to it.
Check the icon of the folder.

Expect Experience:
The folder icon is correct.

Actual Experience:
The folder icon is incorrect.

More Info:

This issue doesn't reproduce for one regular blob.
This issue doesn't reproduce for folders under one file share.
This issue doesn't reproduce in Polish.

This issue is fixed oon build 20200227.1. So we close it.


Document browser support and supported JS features

There's a lot of abstractions in place while developing apps in the IO platform. Without having to mess with bundler, transpiler and/or processor configurations, it's easy to forget that one must be careful about not using some syntax or feature of the language that is not supported throughout our environment.
Things to discover/discuss:

What is our official browser support? Is it different for storefront and admin areas? This is extremely important, not only for JS but also for CSS. i.e I've seen people using display: grid and, without an official documentation about browser support, it's possible that we're serving a capped experience for a small portion of our client's users.
What kind of polyfills are automatically added to our sites, if any? i.e Can we use IntersectionObserver without thinking about polyfilling it?
If a developer wants to use a relative new feature, how she/he is supposed to polyfill it? Should it be encouraged?
How do we get our (VTEX's) browser usage stats?

There are probably more points related to this, but I couldn't think of all of them right now.

Reference: https://github.com/vtex/render-server/blob/master/node/utils/assetLoader.ts#L27

For storefront apps IE11 is the oldest browser we do support.
For admin apps maybe we can expect only ever green browsers.

When documenting this, we can also link to some useful resources such as Can I Use and MDN and teach people how to check if what they want to use is available in their targeted environment.

@guilhermebruzzi Hmmmm, I like those class definitions. We can definitely improve the https://github.com/vtex/javascript/blob/master/docs/Browser Support.md with something akin to this at some point.

Partially done in the above link.


Demo install of two blogs

What are you changing in this pull request and why?

This is a demonstration of how two installs of the blog plugin would work.
Preview Links
https://docs-getdbt-com-git-feature-release-notes-blog-dbt-labs.vercel.app/release-notes

hey @john-rock , just wanted to check if this pr was still ok to be in draft or ok to close?

hey @john-rock , just wanted to check if this pr was still ok to be in draft or ok to close?

Hey @mirnawong1 thanks for surfacing this. All good to close this out!
More context here.


nixos/nscd: be more specific in the nscd.enable description on what breaks

Motivation for this change
#43607
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Maybe also mention that turning this off/on requires a reboot as processes may have the old state in their caches?

The new description is definitely better, but I'd guess it still doesn't mean much to most people. Tbh, before I was forced to look into it, I barely knew anything about the whole mechanism (not that I really understand a lot about it now…).
So maybe we should give a "real world" example. Something along the lines of "For example, this means you will not be able to see which dynamic user owns a file or directory."

I see the description mostly as a way to further discourage people to disable this. The default is to enable this, people disabling it will eventually read the description and don't disable it ;-)
@dasJ if I understood the NSS NixOS module properly, there also should assertions failing when any NSS module except the systemd one is enabled, but nscd disabled, right?

Yes, you are correct.


main: add dumphistoricalchannel command.

This command dumps relevant data of already closed channels. Tries to bring more insights to https://github.com/lightningnetwork/lnd/issues/8130
How to use:
chantools dumphistoricalchannel   --channeldb ./channel.db --chan_point XXXXXX:Y

Thanks for the PR. Is this ready to take a look at or should I wait?


Change github pages to use /docs directory

@lholmquist If I'm not wrong it seems basically to change an option

yea,  i think that is correct.   When we did it for nodeshift,  i created this issue,  then realized there was no docs :man_shrugging:

https://nodeshift.dev/openshift-rest-client/


Make a wish for a docker image to run mamba.

We're busy with ICML deadline, but hopefully Santa will stop by in the next few weeks!

Thank you so much for considering my request amidst your busy schedule with the ICML deadline. Wishing you all the best for the conference.


Binomial Distribution undefined

Hi,
When doing a binomial estimate using a low chance and large number of trials, such as 3600 trials and a 0.08 probability, the result is undefined.
Any ideas as to if this can be patched?

Looks like "nfactorial" returns Infinity, which could be the issue.

Factorial of 3600 is 2.54... × 10^11241, too big to represent in javascript integer. We could use BigInteger, but a better solution would be a different solution for the binomial probability


Updating gracefull-fs.

To be able to run node-s3-client on node v12 we need to update  gracefull-fs package.

This needs to be fixed

We need this - please accept this PR.
@andrewrk @kacole2 @faceleg @maki10 @dvalentiate

@andrewrk @kacole2 @faceleg @maki10 @dvalentiate It appears this simple update would significantly reduce problems we're having using the library. We've been on Node 12 for a while. Merging this update would be fantastic.

^ bump this affects my company as well.

Hi,
Any update on this? We all urgently need it, as the node 12 is a minimum requirement for our projects.

While @andrewrk is busy creating zig try this if you are using yarn...
Add following to your package.json
"resolutions": {
    "node-s3-client/graceful-fs": "4.2.4"
  }


Toolik sample inputs all yrs

Several mostly related things:

Adds new input data set that was generated to cover all 109 years of historic period and 100 years of future (took a few hours running in serial on my laptop).
refactors create_region_input.py to use library function for path handling
changes default averaging period of equilibrium climate

I did a run using these modification, and am not seeing the marked drop in PAR (and related variables) that I had noted in an email to @rarutter ...
Ran the model with this:
    $ ./dvmdostem -l note  --pre-run 100 --max-eq 2000 --sp-yrs 250 --tr-yrs 109 --sc-yrs 100 --cal-mode

Manually archived the json files between each stage, and then generated the plots like this:
    $ ./calibration/calibration-viewer.py \
     --from-archive aug22-2016-devel-4c6ff289-all-years-all-outputs-default-params/sp-data.tar.gz \
     --archive-series aug22-2016-devel-4c6ff289-all-years-all-outputs-default-params/tr-data.tar.gz aug22-2016-devel-4c6ff289-all-years-all-outputs-default-params/sc-data.tar.gz

Not showing the equilibrium stage here, as the timescale drowns out the data in the other stages.

Looks good to me.


Extend assert.async to accept a count of asynchronous operations

Currently, assert.async returns an invoke-once callback, which can lead to overly-cumbersome patterns (e.g., manual counter tracking or collect and pop). It would be more convenient to specify the expected count of invocations explicitly by argument:
var done = assert.async( 4 );

asyncOperation1( function( result ) {
	assert.equal(…);
	done();
} );
asyncOperation2( function( result ) {
	assert.equal(…);
	done();
} );
…

This would be great and seems a common enough pattern.

+1 for this feature.
@jzaefferer Could I work on this issue?


docs: update showcase

What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

It's so slow, it's almost crashing my macbook M1


Allow configurable body factory response max size

New config: proxy.config.body_factory.response_max_size

[approve ci]

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/1317/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1424/ for details.

[approve ci]

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/1318/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1425/ for details.

Please run clang-format on the code.  It is not passing CI.

[approve ci]

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/1326/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1433/ for details.

Still needs a clang-format.  I will merge it and clang-format it once it is committed.


Unable to discover Master that's on wifi

I have 3 computers, A, B and Ground Station
Ground Station and A are on Ubuntu 16.04, Kinetic
B is on Ubuntu 14.04, Indigo
My Current setup:

A and B are connected to each other via Ethernet, both of them is connected to wifi. Ground Station is connected to the router over Ethernet.
All A,B, and Ground Station's ROS_MASTER_URI is set to their eth0 ip address.

A and B could discovery each other
Ground station could discover both A and B
neither A or B could see discover ground station.

I want at least B to be able to discover ground station so I could sync topics over it
I previously followed the the technical report to set things up, but after setting up with that, the only thing that changed was my ground station wasn't able to discover A or B anymore, so i reverted my settings back.
My current settings on all 3 computers:
cat /proc/sys/net/ipv4/ip_forward outputs 0 i.e no ip_forwad
cat /proc/sys/net/ipv4/icmp_echo_ignore_broadcasts outputs 1 i.e. no multicast
Hope to someone could help soon.

THank you ! This worked!


add support for subdomains on localhost

It would be great if subdomains on localhost could be supported. Is there a particular reason it isn't included or simple oversight as it's probably not very commonly needed?
exmaple
tld.getSubdomain("vhost.localhost"); // should return vhost

Nah it is supposed to work. Will investigate this. Thanks!

Okay gotcha. This is related to #25: getDomain and getSubdomain indeed look for valid TLD (publicly known and listed in the Public Suffix list). And localhost is considered as a TLD, not a domain.
Ideally, we'd like a way to enable specific hosts to work on.


Update TOC.yml

Ordered with newest on top and added "Best practices for building collaborative UX with Human-AI partnership" as well as "Hidden tags in Microsoft Azure."

#sign-off


Update nacos-sdk-go for meeting compliance requirements

Problem
https://github.com/nacos-group/nacos-sdk-go currently uses https://github.com/toolkits/concurrent which is unlicensed. Per this active issue, https://github.com/nacos-group/nacos-sdk-go/issues/394, they are moving to golang's sync/semaphore which will make the SDK compliant.
To do

track the concurrent library update in nacos-sdk-go
update the nacos-sdk-go version in this repository

Ref https://github.com/dapr/dapr/issues/4158

/assign

Changes are checked-in, waiting for next release to take up the changes.

https://github.com/nacos-group/nacos-sdk-go/releases/tag/v2.0.1 to be taken up.


Update react router alpha to beta

React router has released beta version of it, I think we should update our deps too.

Thanks for this @diondirza!  It is a helpful reference. :)
I have been trying an upgrade myself, however, there are still some issues to be resolved in regards to SSR.

What kind of issue is it? I also had some issue with latest react router version, and have posted the issue on their github, do you face this kind of issue?

@diondirza That's the one he is referring to. Waiting for it to make it into a release.

@carsonperrotti ah i see, i also looking forward to it.

I get annoying forceUpdateon a unmounted component error every time I try to navigate back/forward (everything else than history.push i think).

Big big big update to this branch coming soon. With tons of fixes.

@birkir I don't see any error you've mentioned either in browser or console on my branch. Is this related to some lib?

@diondirza False alarm. I traced the source of the problem to react-async-component (or react-tree-walker to be exact), it is badly broken.

I think this PR not relevant anymore, will make another PR for solving 404 status code


Better Italian translation

We planned to work on it with Makina-Corpus. We'll use the translation of the V1 to the V2 and PNM will complete the words missing.

This one is in Geotrek-mobile, just a few changes as there was already a IT version.
I think you are talking about adding IT translation in Geotrek-rando V2 that wasn't translated in IT yet.
You can do it anytime with creating the IT version of this file : https://github.com/makinacorpus/Geotrek-rando/tree/master/src/app/translation/po/fr
When you want. Quickly would be good. :-)
Geotrek-rando V1 IT translation are in a few files in each module :

https://github.com/makinacorpus/Geotrek-rando/blob/v1.X/rando/trekking/locale/it/LC_MESSAGES/django.po
https://github.com/makinacorpus/Geotrek-rando/blob/v1.X/rando/core/locale/it/LC_MESSAGES/django.po
https://github.com/makinacorpus/Geotrek-rando/blob/v1.X/rando/feedback/locale/it/LC_MESSAGES/django.po
https://github.com/makinacorpus/Geotrek-rando/blob/v1.X/rando/flatpages/locale/it/LC_MESSAGES/django.po
https://github.com/makinacorpus/Geotrek-rando/blob/v1.X/rando/view3d/locale/it/LC_MESSAGES/django.po


Specifying max steps as 10 in build options was inadvertently removed

This should solve some of the E2E testing timeout issues we've encountered. My mistake.

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


Updating deploymentreadme and docs/readme

Deployment Readme changes:

Changed odim to odimra in:
$ docker exec -it odim_1 /bin/bash
Edit the parameters in the odim_config.json file located in this path: /etc/odim_config/odim_config.json and save.

Added Docker proxy content.

Semantic and formatting changes.

Docs/readme changes:

Semantic and formatting changes.
Fixed broken internal reference links.
Added "Required privileges" column for the list of APIs under every section.
Moved content under "User roles and privileges" to a separate section called  "role-based authorization"  under "Authentication and authorization".

Added TOC for docs/Readme and Deployment Readme.


Reverse usage of String Templates in preparation for JDK23

Description
Reverse the usage of String templates as they are going to be removed from JDK23.
Problem description
If the patch provides a fix for a bug, please describe what was the issue and how to reproduce the issue.
Backend/s tested
Mark the backends affected by this PR.

[x] OpenCL
[ ] PTX
[ ] SPIRV

OS tested
Mark the OS where this PR is tested.

[X] Linux
[ ] OSx
[ ] Windows

Did you check on FPGAs?
If it is applicable, check your changes on FPGAs.

[ ] Yes
[ ] No

How to test the new patch?
make jdk21

Is this PR only for the OpenCL backend? We use the String templates in PTX and SPIR-V, right?

This now covers PTX and SPIRV @jjfumero @stratika


error TS4053: Return type of public method from exported class has or is using name 'Response' from external module "/node_modules/@angular/http/src/static_response" but cannot be named.

Hi everyone,
I tried to update my project with Ionic2 RC0. I choose the way creating a new Ionic 2 project and copying my project files to it. What can I do to solve this ? The first line with error..
 login(user:User) {
     const data = JSON.stringify({u: user.username, p: user.password});
     return this.http.post(this.actions.login, data)
 }

Maybe by:
 login(user:User): Response {
     const data = JSON.stringify({u: user.username, p: user.password});
     return this.http.post(this.actions.login, data)
 }

Idk, I can't reproduce it.

Hello @klochko7 ! Thanks for using Ionic! This is actually a typescript error and not something caused by the framework. Because of this I am going to close this issue and ask that you ask this question on the forum.

Hi @klochko7 and @jgw96 - ran into same ts error on http.post code when upgrading to ionic-angular 2.0.0-rc.0. Same exact code is used in 2.0.0-beta.11 version w/ no ts error and in a separate Angular 2 app using @angular 2.0.0 and build w/ angular-cli .0.0-beta.16. After digging around for awhile, I noticed diffs in tsconfig.json files pertaining to declarations under compilerOptions:

upgraded Ionic2 app w/ ts errors had declaration set to true
previous version of same Ionic2 app had no declaration parameter (default false?)
separate NG2 app had declaration set to false
When I changed the declaration compilerOption to false in my upgraded Ionic2 app, then the ts errors were gone.

Not disputing @jgw96 pointing out this is Typescript issue, but I wanted to share my findings. My understanding of the declarations parameter under compilerOptions is related to generating ts definition files, which I certainly do not need in my app. Further comments/thoughts welcome.
Thanks Ionic! Cheers.


[v6] not able to have two same components with different initial values

Say we have a table contains some rows, each row contains some data that can be edited. One example is the react-redux-universal-hot-example

In this example, we can edit more than one row.

However, in the V6 version, editing one row is fine, but when clicking the edit button to edit the second one, the value of the second row will override the first row, like the following:

I found this is due to that there is only one EditForm object created under state form, and the initial value is shared between these two forms, just wondering is there any way to make it work? Or it supposed to be like this. Thanks

You need unique form names. You can pass the form name in as a prop.
<WidgetRowForm form={`widgetForm_${widget.id}`} initialValues={widget}/>

Thanks, @erikras , will close this issue. :+1:


Trace back the original document and return it as the answer

If answer is not found, then just chain it to lmgtfy. ?

Done via #219 but different approach


chore(release): v0.10.7 🎉

:robot: I have created a release beep boop
0.10.7 (2023-02-10)
Bug Fixes

bin: ensure bin uses oclif core (9db7b1b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/declapract/releases/tag/v0.10.7 :sunflower:


BUG

Hello, thank you for sharing your code generously,
I want to know what are the keywords_gpt_total.json and keywords_gpt_total.json .
And I always get accept the following error when I run the code ' python cli.py train with model=no_gt_sos fix_gpt_epoch=5 feature_names="['video', 'images']" '
ERROR - lsmdc - Failed after 0:04:15!
Traceback (most recent calls WITHOUT Sacred internals):
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/cli.py", line 48, in train
all_args = prepare()
File "/root/TAPM-0/tapm-master/code/cli.py", line 31, in prepare
model, tokenizer, ckpt, datasets, epoch = prepare_model()
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/cli.py", line 25, in prepare_model
return get_model_ckpt(model_name)
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/ckpt.py", line 63, in get_model_ckpt
datasets = get_datasets(data_path, pretrain_path)
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/data/dataloader.py", line 71, in get_datasets
datasets[k] = Dataset(p)
File "/root/TAPM-0/tapm-master/code/data/dataloader.py", line 17, in init
self.task, self.path = load_tasks(data_path)
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/data/task_loaders.py", line 30, in load_tasks
data, global_data = f(path)
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/data/task_loaders.py", line 39, in load_task1
return load_task1_group_with_features_and_keyword(path)
File "/root/TAPM-0/tapm-master/code/data/task_loaders.py", line 158, in load_task1_group_with_features_and_keyword
total = load_keyword_only(path, name='keywords_gpt_total.json')
File "/root/TAPM-0/tapm-master/code/exp.py", line 48, in captured_function
return lazy_capture(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/exp.py", line 40, in call
return self.lazy_fn(*args, **kwargs)
File "/root/TAPM-0/tapm-master/code/data/task_loaders.py", line 378, in load_keyword_only
assert len(paths) > 0, f"no keyword candidate for {name}"
AssertionError: no keyword candidate for keywords_gpt_total.json

First of all, thank you for notifying the issue.
Keyword is now a deprecated feature, so in theory the code should run fine without it.
I suspect a bug in the code, and currently inspecting the cause.
Thanks.


cypress/included:5.0.0 does not support TypeScript.

Tests written in TypeScript throw an error.
Error: You are attempting to run a TypeScript file, but do not have TypeScript installed. Ensure you have 'typescript' installed to enable TypeScript support.

Maybe this is caused by Cypress now requiring TypeScript 3.4 or higher?

You'll need to ensure typescript is installed as a dependency on your project. And yes, it will require a typescript version above 3.4. Our images do not pre-install TypeScript since not every project would require this.


[ui] Add demo mode which shows single key presses as an overlay on the screen

intended for use during presentations or training sessions
add --demo for demo mode
Demo mode shows single key presses on the screen
Add post process to detect pyfiglet
Pass args.demonstration to UI

I feel like this might be better left to an external thing (the screen capture tool in the case of recordings, or another tool running (for presentations) that shows current keypresses). But maybe others have other opinions.

RE pyfiglet, good call. Although it wasn't really a runtime dep, there's no reason to pull it in...

The more I think about this, I would like to have this included for a while. Not a permanent thing, but during early adoption, sharing, demoing and showing others, I think this will be handy.  I would have loved it for some early demos I gave.
My only concern about leaving this to a 3rd party recording tools is that it won't work live, and will not be able to differentiate between colon commands and single key presses.

My only concern about leaving this to a 3rd party recording tools is that it won't work live

There are external tools that can show keypresses without recording.
In the interest of avoiding scope creep, I'm still -1 on this, especially for 1.0. But even beyond that, I still think this is better left to a dedicated tool.

+100 for the embedded links :)  I had tried a couple of those and realized they would never work with asciinema.
Nor does it appear asciinema will support an overlay.
I will mark this as dnm, and we can leave it out of 1.0.0.  I would like it left around until we get a better asciinema in the README.

This can either be removed or merged after 1.0.0


improved the search functionality of the search component

PR #33
I have modified the search functionality  by changing it to onclick event

Uploading TrendTrove - Google Chrome 2024-06-14 11-10-57.mp4…
this is the demo video of search functionality._

https://github.com/Niharika0104/TrendTrove/assets/127289297/87a3be17-a969-4c8c-a0e3-19c9a82d548c
here is the demo of search functionality

@Rajneesh2223 hey..instead of performing search functionality on the cards which are there are on the home page,just redirect them to that page.Like when user types sarees and clicks on search take them /sarees page this is already set on the category cards(like when u click on saree card you will be redirected to sarees pages).You also do the same,becoz in e commerce applications like Myntra and Amazon whenever you search for something you will be redirected to a page with the products that you searched for.And remove that clear button on navbar,it's not needed.

thanks for the review i will work on it .

https://github.com/Niharika0104/TrendTrove/assets/127289297/2f188646-eb4e-41c2-a14c-0de02042a3bb
@Niharika0104
I have changed the implementation of search bar , please review so that i can make pull request . If any changes required I will do it

@Rajneesh2223  I merged the code your video shows something what's been deployed it shows something else.Also,when users types on sarees and clicks on search redirect them sarees page(https://trend-trove-client-side.vercel.app/Sarees) right now you are
only filtering on the home page category cards.Did you push the changes to the pr?

Ma'am i just send you the video  I havent made the pull request , that why It is showing this


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit cc3829f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-11-13-35-20 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-11-13-35-20

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Sensor status can be displayed on the plan?

Hi, not sure if it is posible already or not but I'm interesting on displaying couple sensors data on the plan. In the past I use SVG version were it was possible to add watter or room temp on the plan as in the below picture.
Do you think it is posible also on floor3d-card?

Thanks

I found the solution (RFM) Read the f.... maual was...section Text :)


Updates to Parkinson to Synchronise With Open Source with App Store Version

This Pull Request contains four primary commits:
o    Changes in Tasks and Schedules JSON file for New Scheduler
o    Changes in Survey JSON file keywords
o    Changes to Parkinson to cater to new Pre-Survey capability, where the most recent 'Did you take your medications' result is stashed away so that the result can be uploaded even when the question is not asked.
o    Plus a Gallimaufry of minor typographic changes . . .

'xxxxx' should have read 'four'

On Jun 30, 2015, at 5:37 PM, Yuan Zhu<EMAIL_ADDRESS>wrote:
Merged #6 https://github.com/ResearchKit/mPower/pull/6.
—
Reply to this email directly or view it on GitHub https://github.com/ResearchKit/mPower/pull/6#event-344830239.

    Henry


Automatic docsgen is not working

The documentation for this provider has not been updated since its initial addition to the package registry.
We have moved on from v0.4.0 to v1.4.0, with a few outstanding upgrades, so it appears that the automation that is running the Publish Package Metadata(example) job is not set up for this provider.

This appears to be fixed. The registry currently displays:

Databricks v1.36.0 published on Friday, Apr 19, 2024 by Pulumi

That correctly reflects this provider's last release.


Change the name.

istanbul-lib-hook implies something istanbul specific.
pirates and wrap-require have been suggested.

I personally like shanghai :)
Note that this repo also provides a mechanism for hooking in vm.runInThisContext, vm.createScript etc. and this PR https://github.com/gotwarlost/istanbul/pull/465 in the istanbul repo adds more stuff to it. We should be careful about scope in a common library that is anticipated to be used in multiple places.
I also need to have a conversation with the open source gurus in my company @gyehuda and @davglass  so they can provide some input about the licensing stuff if the code were to be moved out of this org, renamed etc. Unless you guys are ok leaving it in this org and just renaming the repo.
link: https://github.com/bcoe/nyc/issues/70 as context for the new folks added to the thread

@gotwarlost: actually, in the linked thread, it was suggested that it was out of scope for this project, so I'm going to put it in a different one. You're welcome to do whatever you want, but I would rather that istanbul just used the new module, as the whole point is to make everything work together. (Also, I suggested the name pirates, and I'd kind of like to use it, if you don't mind).
Also, can I have permission to modify and include your code from this repo in the new project? (with attribution)

Yeah, I thought you might want to do something like this which is why I involved the open-source gurus who can figure out the details for us.

@gotwarlost: ok, thanks!
So, @gyehuda and @davglass: what steps should I take to get the ok to use and modify (Actually, I think in practice I would more just be taking ideas from) this repo? I plan to release it under MIT, if that helps. I can also just look at this implementation and then build my own, if that would be easier. Thanks!

I personally don't have a problem leaving it here. I just want something widely known and widely adopted.

We should be careful about scope in a common library that is anticipated to be used in multiple places.

Agreed. I take it you are suggesting vm.runInThisContext, and vm.createScript would be out of scope? I am not so sure. They all seem to serve the same goal (interjecting transforms). I think the reason most people only hook require is that vm is not widely known.

@jamestalmage: no, of you read the linked thread, I want to add some stuff which focuses on making various require hooks work well together, which is out of scope here

I have been monitoring (and participating) in the linked thread from the beginning.
I don't think "making various require hooks work well together", is at all out of scope for a library that installs require hooks.
I also do not have a problem moving it. The important part of my sentence was "I just want something widely known and widely adopted".

I guess we don't need this any more. We already have pirates and append-transform as the underlying layers on which hooks are built.


Exception at LWIP igmp.c Line 676

I have started receiving an exception.   It appears to be originating within LWIP logic.  I am raising this issue as a record of the story in the hopes that if others see it, it may spark something further:
Guru Meditation Error of type LoadProhibited occurred on core  0. Exception was unhandled.
Register dump:
PC      : 0x400ef56b  PS      : 0x00060930  A0      : 0x800ea5da  A1      : 0x3ffc6090
A2      : 0x8011884d  A3      : 0x3ffc36d4  A4      : 0x00000001  A5      : 0x3ffc58b8
A6      : 0x3ffc588c  A7      : 0x00000001  A8      : 0x00000000  A9      : 0x3ffc6070
A10     : 0x3ffc36d4  A11     : 0x00000000  A12     : 0x00000000  A13     : 0x00000000
A14     : 0x3ffc5994  A15     : 0x3ffc5954  SAR     : 0x00000013  EXCCAUSE: 0x0000001c
EXCVADDR: 0x8011885b  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0x00000000

Backtrace: 0x400ef56b:0x3ffc6090 0x400ea5da:0x3ffc60b0 0x400ea674:0x3ffc60d0 0x400ed06f:0x3ffc6100

And then attached GDB:
Remote debugging using /dev/ttyUSB0
igmp_tmr () at /home/kolban/esp32/esptest/esp-idf/components/lwip/core/ipv4/igmp.c:676
676	    if (group->timer > 0) {
(gdb) bt
#0  igmp_tmr () at /home/kolban/esp32/esptest/esp-idf/components/lwip/core/ipv4/igmp.c:676
#1  0x400ea5da in igmp_timer (arg=0x0) at /home/kolban/esp32/esptest/esp-idf/components/lwip/core/timers.c:210
#2  0x400ea674 in sys_timeouts_mbox_fetch (mbox=0x3ffbd750 <mbox>, msg=0x3ffc6100)
    at /home/kolban/esp32/esptest/esp-idf/components/lwip/core/timers.c:575
#3  0x400ed06f in tcpip_thread (arg=<optimized out>)
    at /home/kolban/esp32/esptest/esp-idf/components/lwip/api/tcpip.c:101
(gdb)

Do you have any steps to reproduce this exception?

Unfortunately I don't have any recreate steps.  As such, if we want to close then by all means.  The issue should still show up in searches at Github.  The thinking is that if one sees an exception in ESP-IDF supplied code, one could search in the issues and see if it has been seen before.  That way we may see patterns in reproduction or else learn that there is something "endemic" somewhere that might need attention.

Closed for just now.


ExerciseOption always results in 5 second timeout from Transactions.WaitForOrder

Expected Behavior

I'm unsure, but we shouldn't be timing out each time. There's two possibilities here:

We permit OptionExerciseOrder to fill on the same time step as its submitted
Remove Transactions.WaitForOrder (and asynchronous parameter) and always fill OptionExerciseOrder on next bar like all other orders (except market order)
More complicated behavior taking into actual settlement procedures which is T+2 for equities. [ preferred ]

My preference for number 3 is based on model fidelity. We should definitely test this and I even suspect different brokers have different rules, perhaps even based on the account and whether or not the options were purchased w/ margin. I checked my Series7 book and all it says is that they settle regular way, which is T+2. I could see certain brokers providing convenience by making shares/cash immediately available, but we would need to test to confirm the behavior. If desired, such behavior could be added to IBrokerageModel thereby allowing high fidelity modeling of how each broker behaves in these circumstances [ preferred ].
Actual Behavior

When calling QCAlgorithm.ExerciseOption a log message pops up indicating that we waiting for 5 seconds but the order didn't fill. This is because of a call to Transactions.WaitForOrder when the asynchronous parameter is set to false which is the default.
Potential Solution

It depends on what we it it to do:

Add OptionExerciseOrder to here
Simply remove asynchronous parameter and Transactions.WaitForOrder
Research IB behavior surrounding manual option exercise and determine when:
a. cash is credited and stock is debited for exercise of long put
b. cash is debited and stock is credited for exercise of long call

Reproducing the Problem

Run any algorithm that call QCAlgorithm.ExerciseOption and observe the 5 second timeout error in the syslogs.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Close by https://github.com/QuantConnect/Lean/pull/5169, OptionExerciseOrder is now process immediately see https://github.com/QuantConnect/Lean/blob/master/Brokerages/Backtesting/BacktestingBrokerage.cs#L272

Close by https://github.com/QuantConnect/Lean/pull/5169, OptionExerciseOrder is now process immediately see https://github.com/QuantConnect/Lean/blob/master/Brokerages/Backtesting/BacktestingBrokerage.cs#L272


Removing hyphen from in-active.

inactive is a word.  in-active is ... something else.

This is so sad.


Support more array-like data types for tabular data and list-like data types for text data

This pull requests is an attempt towards addressing #65. Note that this is my first ever public contribution, so don't be too hard on me 😉
For tabular-classification and tabular-regression tasks, we're currently supporting the following data types:

pandas.DataFrame
numpy.ndarray

Should this PR get merged, we will support any other data type that can be converted to a 2D numpy.ndarray using np.asarray.

For text-classification and text-regression tasks, we're currently supporting the following data types:

List of strings

Should this PR get merged, we will support any iterable of strings.
xref: https://github.com/skops-dev/skops/pull/45/files#r933221139

Thanks a lot @anferico for taking this.
I have only taken a quick look so far and noticed that you haven't added any unit tests for the changes. Could you please do that? Let us know if you need any help with that.
Also, please add an entry to the changes doc.

I haven't added any unit test for the 2 small helper functions I added, though I've modified existing ones in accordance to the changes I made. Not sure what's your policy here, but if needed I can add unit tests for those as well. I also added some docstrings and modified existing ones.
Everything should be fine since all unit tests were passing on my local machine, but let's wait until the pytest workflow runs successfully.

@BenjaminBossan I've tried addressing your comments, see if it's good now. One minor detail: I renamed the function _is_iterable_of_strings to _is_sequence_of_strings because now I'm not expecting generic iterables anymore as inputs to that function, which led me to remove the logic that prevents iterators and generators from being exhausted when iterated over. Without that logic, it just felt wrong to me calling it _is_iterable_of_strings, so I went for _is_sequence_of_strings.

@anferico are you still working on this?

@anferico are you still working on this?

I am, it's just that I haven't had much free time recently. I'm happy to continue working on this, but if it's really high priority please feel free to let someone else take over. Sorry about that

Hey, no worries, take the time you need. I just asked because sometimes people just forget or are waiting on some input from the maintainers.

@BenjaminBossan I think I've addressed all the points above, however now I'm facing another problem.
I noticed that when calling the test_get_column_names_pandas_not_installed function, I incur into a weird recursion error. Obviously, as a result, the test fails.
Here's the command I ran:
python -m pytest -s -v --cov-report=xml skops/hub_utils/tests/test_hf_hub.py::test_get_column_names_pandas_not_installed

and here is part of the output I get (see  log_test.txt for the full output):
  .
  .
  .
  File "/home/anfri/MyProjects/skops/skops/conftest.py", line 13, in mock_import
  File "/home/anfri/miniconda3/envs/skops/lib/python3.10/unittest/mock.py", line 1103, in __call__
  File "/home/anfri/miniconda3/envs/skops/lib/python3.10/unittest/mock.py", line 1111, in _increment_mock_call
  File "/home/anfri/MyProjects/skops/skops/conftest.py", line 13, in mock_import
  File "/home/anfri/miniconda3/envs/skops/lib/python3.10/unittest/mock.py", line 1103, in __call__
  File "/home/anfri/miniconda3/envs/skops/lib/python3.10/unittest/mock.py", line 1111, in _increment_mock_call
RecursionError: maximum recursion depth exceeded

I'm not sure what's happening here, because I haven't touched conftest.py. My suspicion is that the line from sklearn.utils import check_array that I added to _hf_hub.py could be the culprit because there are some pandas imports in there, so maybe us patching the import builtin in such a way that it throws an ImportError while trying to import pandas could be a problem, but I'm really not sure.

I'm not sure what's happening here, because I haven't touched conftest.py. My suspicion is that the line from sklearn.utils import check_array that I added to _hf_hub.py could be the culprit because there are some pandas imports in the source file containing that function, so maybe us patching the import builtin in such a way that it throws an ImportError while trying to import pandas could be a problem, but I'm really not sure.

It seems likely that this is the explanation, but I'm not sure why that would lead to an infinite recursion. I'd have to dig deeper. @adrinjalali any ideas?
Meanwhile, since some time has passed since this PR was opened, the changes.rst has been updated. You would need to move your entry to the current  version. Could you please do that?

No idea about the infinite recursion, but looking at this, two things come to my mind:

it might be a good idea for us to have a CI job where we have minimal dependencies installed, instead of patching load_module.
same issues as #211 would apply here: we should check if what we do would work on the inference API side.

It might be a good idea to reduce the scope of this PR to list of strings only, to make it easier to merge and tackle other things in a separate PR.

@anferico I figured out the recursion error. Honestly, I'm a bit surprised that it didn't occur previously, but patching can be tricky. Please change the pandas_not_installed fixture like so:
@pytest.fixture
def pandas_not_installed():
    # patch import so that it raises an ImportError when trying to import
    # pandas. This works because pandas is only imported lazily.
    orig_import = __import__

    def mock_import(name, *args, **kwargs):
        if name == "pandas":
            raise ImportError
        return orig_import(name, *args, **kwargs)

    with patch("builtins.__import__", side_effect=mock_import):
        yield

it might be a good idea for us to have a CI job where we have minimal dependencies installed, instead of patching load_module.

You mean a CI job where, for instance, pandas is not installed? Yes, that could be useful, but I still like to have a test that patches import, so that I can e.g. quickly test the functionality locally without the need to juggle multiple environments.
The extra CI environment would come in handy in case that the test itself is buggy or incomplete. However, having such a test environment would probably require adjusting a bunch of other tests too that currently assume all tests libraries to be installed.

same issues as Added support for Structured Arrays #211 would apply here: we should check if what we do would work on the inference API side.

Would it be possible to add this in a separate PR?

Would it be possible to add this in a separate PR?

what does "this" here refer to?

what does "this" here refer to?

I mean checking if what we do would work on the inference API side.

I mean checking if what we do would work on the inference API side.

Yes and no 😁 we should at least check if we train a model with these data types, scikit-learn understands them (in terms of feature names etc), and that we can predict using these types and don't get errors/warnings from sklearn.
Right now sklearn supports feature names mostly through pandas only.

@BenjaminBossan thanks, the recursion error just got fixed. I also fixed the changes.rst file.
A couple of comments regarding my latest commit:

I removed support for generators as valid containers for text-based tasks. The reason is that I couldn't find a way to peek at a generator without consuming it. Let me know if this is ok
The function get_example_input_from_text_data does not fail on empty containers. Is it desirable or not?

@BenjaminBossan  I've addressed the latest comments, let me know if this can be merged.


Range calculation omits the upper value

When asking for predictions for a range of values, the most upper value is not calculated.
Passibly due to the way we create the range of relevant distances with np.arange(0, 300, 10, dtype=int) / 100.
f.e for the example above, there will be no calculation for the 3m value. it stops at 2.9

adapted the np_arange parameters to add the upper value in a relevant _distance series


Flux end-to-end test failing with "field type conflict"

The Flux end-to-end test chronograf/aggregate_window_median_test.flux is failing post-split with Flux upgraded to v0.83.1, with this error:
   --- FAIL: TestFluxEndToEnd/testing/chronograf/aggregate_window_median (0.14s)
            end_to_end_test.go:226: error calling function "inspect" @0:0-0:0: error calling function "case" @testing.flux|20:10-20:16: error calling function "loadStorage" @aggregate_window_median_test.flux|105:11-105:43: error calling function "chain" @9:9-12:3: field type conflict

This error message is not present in the flux repo and seems to be coming from storage.
The test will be skipped in influxdb/query/stdlib/testing/testing.go.

Could this be a field type conflict of the input data to the test case? If so then we should update the input data to be valid, so that the field has a consistent type.


Groups collection without lock

As discussed with @davidfowl groups logic changed.

@Gurgen,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@Gurgen Could you sign the cla please?

@Gurgen, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

Signed

Don't understand, why check is not passed?

Known test failure, I'll restart it for you

Thanks

Wait, we need to remove connection on disconnect from groups where it present

@BrennanConroy Please pay attention to last commit

@Gurgen - disregard my comment about the race - I was wrong. If you fix the formatting, add a comment as requested by @BrennanConroy and rebase+squash on latest dev we should be able to merge your PR. Thanks for your patience!

@moozzyk, @BrennanConroy can you rerun AppVeyor build again? I think there same error

Done, please check

Looks good - I just need this to be squashed and rebased on dev to be able to merge.

While rebase, I get conflicts on BinaryMessageFormatter and BinaryMessageParser, but I didn't touch those files.

Did you pull the latest dev? Do you have any pending changes by any chance? I was able to rebase with just one conflict on this line: https://github.com/aspnet/SignalR/pull/997/files#diff-78e1f2ad8780817477e28d3ec3b11ff6L148.

Please check again, is rebase done?

Doesn't seem like it is - did you push? You will have to force push (i.e. use the --force option) otherwise git will reject your push because rebase changed commit history.

Please check one more time.
I'm working on fork. Maybe problem on that?

@Gurgen - I took your change rebased on latest dev, squashed and pushed as https://github.com/aspnet/SignalR/pull/1027. Take a look to confirm nothing was lost in the process and I can merge. Your are still the author of the commit.

Ok, thank you. After merge of new PR, close this one?
I confirm new PR.

I think the problem of rebase from fork. I work on fork repository, where updates doesn't come from main repository. How can I push commit to this repository?
I want to change User invocation process same way.

I will merge the other PR and will close this one.
You need to update your fork as described here: https://help.github.com/articles/syncing-a-fork/

It was merged as af286c8. Closing.

Thanks, @Gurgen!

@muratg nice to be a part of great work :)


Added watchify dev dep

When trying to run gulp unittestWatch I got an error for a missing watchify dep. This PR fixes it 👍

That's weird, watchify should not be a dependency anymore and I can't find where it's used in the project. gulp unittestWatch works for me, even after a fresh clone and npm install. What npm version do you use?

➜ node -v
v4.5.0
➜ npm -v
3.10.6
➜ gulp -v
[13:29:30] CLI version 3.9.0
[13:29:30] Local version 3.9.1

From the npm blog:

We will also be changing the behavior of peerDependencies in npm@3. We won’t be automatically downloading the peer dependency anymore. Instead, we’ll warn you if the peer dependency isn’t already installed. This requires you to resolve peerDependency conflicts yourself, manually, but in the long run this should make it less likely that you’ll end up in a tricky spot with your packages’ dependencies.

On a fresh install I'm getting:
<EMAIL_ADDRESS>requires a peer of watchify@>=3 <4 but none was installed.
So I'm guessing this is where the issue comes from

Nice catch, I'm on npm 2.15.5, that's why it works for me. Can you please amend your commit message to add a more meaningful description, like that we will not delete it again next time the project dependencies are cleaned up :)

@simonbrunel better commit message added. 👍


pull request for e2e test at 01/07/2017 20:26:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


cmdlet aliases not always supported in PSCore

I ran into an issue while training a colleague today. He's using a Mac running PSCore.  We couldn't get past the pester version check until we realized that the "sort" command error it was throwing was for the bash sort command.  Per https://github.com/PowerShell/PowerShell/issues/6684 the Sort alias for Sort-Object was removed due to the sort command already being present in bash.  We edited his dbachecks.psm1 to change "... | Sort Version ..." to "... | Sort-Object Version..." and were able to get past the error.
Aliases should probably be removed for cross-platform compatibility.

thank you :-) fixed in next release


FindInMap is allowed within Cidr

cfn-lint version: cfn-lint 0.21.5
Description of issue.
FindInMap is allowed within Cidr - the following code runs successfully in the CFN console, but it reports as an error in cfn-lint.
!Select [ 2, !Cidr [ !Ref VpcCidrBlock, 4, !FindInMap [CidrBitsMap, !Ref Size, AZCidrBits] ]]
Gives cfn-lint failure: E1024:Cidr sizeMask should be Int, Ref, or Select
See: https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/rules/functions/Cidr.py#L135

Yea I think that logic was originally built off the documentation.   Should be any fix and thanks for reporting this.
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/intrinsic-function-reference-cidr.html#intrinsic-function-reference-cidr-functions

Will be in the next release.


WFLY-4911 Allow expressions for the "alias"-attribute of undertow's h…

…ost configuration

Linux Build 7009 is now running using a merge of 2a8f3a7a24049ac48a19cb24ad0be4e36b482f54

Windows Build 2064 is now running using a merge of 2a8f3a7a24049ac48a19cb24ad0be4e36b482f54

Linux with security manager Build 1886 is now running using a merge of 2a8f3a7a24049ac48a19cb24ad0be4e36b482f54

Linux with security manager Build 1886 outcome was SUCCESS using a merge of 2a8f3a7a24049ac48a19cb24ad0be4e36b482f54
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:55

Windows Build 2064 outcome was FAILURE using a merge of 2a8f3a7a24049ac48a19cb24ad0be4e36b482f54
Summary: Tests failed: 1 (1 new), passed: 2914, ignored: 342 Build time: 0:48:34
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).failoverOnStop: <no details avaliable>

Linux Build 7009 outcome was SUCCESS using a merge of 2a8f3a7a24049ac48a19cb24ad0be4e36b482f54
Summary: Tests passed: 2914, ignored: 342, muted: 1 Build time: 0:51:13


concurrent.futures._base.CancelledError

Rasa version:1.7.4
Rasa SDK version (if used & relevant):1.7.0
Rasa X version (if used & relevant):
Python version:3.6
Operating system (windows, osx, ...):
Issue:
I use it occasionally concurrent.futures . base.CancelledError Problem, it appears randomly. I use restchannel and custom action. I can't locate the cause of the problem.
Error (including full traceback):
Traceback (most recent call last):
  File "/rasa/rasa/core/processor.py", line 834, in _run_action
    events = await action.run(output_channel, nlg, tracker, self.domain)
  File "/rasa/rasa/core/actions/action.py", line 549, in run
    json=json_body, method="post", timeout=DEFAULT_REQUEST_TIMEOUT
  File "/rasa/rasa/utils/endpoints.py", line 148, in request
    **kwargs,
  File "/build/lib/python3.6/site-packages/aiohttp/client.py", line 1012, in __aenter__
    self._resp = await self._coro
  File "/build/lib/python3.6/site-packages/aiohttp/client.py", line 483, in _request
    timeout=real_timeout
  File "/build/lib/python3.6/site-packages/aiohttp/connector.py", line 523, in connect
    proto = await self._create_connection(req, traces, timeout)
  File "/build/lib/python3.6/site-packages/aiohttp/connector.py", line 859, in _create_connection
    req, traces, timeout)
  File "/build/lib/python3.6/site-packages/aiohttp/connector.py", line 967, in _create_direct_connection
    traces=traces), loop=self._loop)
concurrent.futures._base.CancelledError

Command or request that led to error:

Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for the issue, @koaning will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Thanks for the issue, @koaning will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

@Prostar12 does this happen when your custom action takes more than 60 seconds to respond?

@Prostar12 does this happen when your custom action takes more than 60 seconds to respond?

@erohmensing No, my custom action just searches value by giving key from Redis, and it won't takes more than 60 seconds.

@erohmensing No, my custom action just searches value by giving key from Redis, and it won't takes more than 60 seconds.


[EL] Add HassNevermind

Add Greek support for a future HA intent: HassNevermind (see https://github.com/home-assistant/intents/pull/1549).

Please apply this minor fix: "ξέχνα το" to the commented lines.

I tried it with google TTS and it does a strange pause when I split the two.
I do not know how the STT would prefer to have it, but we also tend to say it almost like one word.
Should we put both in for better accuracy?
(I can make a new pull after this , with out the split.
This will help reverting it if necessary later.)

Hi @apo-mak, sure you can add it as one word as well.


Added the option of validating JWT on user selected http handlers.

Please review.

I don't seem to have the option to create a Merge Request.

Updated review comments.


death_fig-59

The choice of HIV/Non HIV feels like categorisation for the Cause of Death (COD), not the actual cause. If the current value is rather a categorisation, should we not rather reserve valueCodeableConcept for when the Cause of Death coded is added to the MDS?  There is a category field also... No further comment.


License?

What license is this project licensed under?
I would want to use snippets of the code in my library.

The repo doesn't have a license, meaning it is all rights reserved to the owner (@briantanner)
If you wish to use any part of this code, you must have explicit permission from him.

This now has a license.


Remove clock from ledger

If we merge https://github.com/bosagora/agora/pull/2670 then we can remove the clock in the Ledger as it was only used when the current time was used for calculating the time offset in the next block.

Closing this as the commit was included in PR https://github.com/bosagora/agora/pull/2676


RabbitMessageChannelBinder fails to initialize LocalizedQueueConnectionFactory if ssl.enabled is not specified

https://github.com/spring-cloud/spring-cloud-stream-binder-rabbit/blob/09c8b9bdc5a172b4034b32ff4d70b094ccb616b7/spring-cloud-stream-binder-rabbit/src/main/java/org/springframework/cloud/stream/binder/rabbit/RabbitMessageChannelBinder.java#L262
Shouldn't it be this.rabbitProperties.getSsl().determineEnabled() instead? The problem is if spring.rabbitmq.ssl.enabled is not specified in application.yml this line fails with NPE.
Also if virtualHost, username and/or password are specified on addresses instead of separate fields current implementation will ignore them, use of determine* alternatives might help here also (It looks like I should file these separately, but I'm not sure it makes sense).
More comprehensive solution might be discussed in https://github.com/spring-projects/spring-amqp/issues/1384

Issue moved to spring-cloud/spring-cloud-stream #2331 via ZenHub


Feature proposal: circleToCells

This SO question, and others where people asked for a radius lookup, made me think that there might be value in a circle-fill function that produced more accurate "cells in radius" output than kRing. One option here, which could make this fairly fast, is to calculate:

the minimum grid radius r1 required to contain the circle
the maximum distance r2 guaranteed to be in the circle

Then you could use kRingDistances(cell, r1) and check only cells with a grid distance greater than r2 using a true geo distance calculation to determine whether they should be in the output set.

The same containment configuration that we want to add to polyfill also applies here. Fully contained, centerpoint containment, and any part of the hexagon intersecting the circle are all things that different people will want for different reasons.
It may also make sense to make this reuse a lot of the upcoming polyfill changes for the above but just swapping out the point-in-polygon function for a point-in-circle function?

+1
I've had plenty of situations where this function would have been super useful. :)


Fixed typo

Fixed typo error on question number 58

@nobill1 Thanks for PR

You're welcome @sudheerj


Slovakia numbers - short numbers and proper area code detection

Country/region affected (e.g., "US"): Slovakia
Example number(s) affected ("+1<PHONE_NUMBER>"): +421216877
The phone number range(s) to which the issue applies ("+1 555 555-XXXX"): see below
The type of the number(s) ("fixed-line", "mobile", "short code", etc.): see below
Supporting evidence (for example, national numbering plan, announcement from mobile carrier, news article):
There are certain valid Slovak numbers that aren't recognized correctly. There are two main issues:

There are short numbers that don't have an area code (1911 and 18333 in the table below). These numbers can be dialed from everywhere in the country and don't have an area code at all, so there's no area code we can put in to make them work. The Google Library doesn't recognize these.
There's also another issue with how the lib is interpreting certain numbers. When you validate +421216877, then 42 is interpreted as the area code (which maps to a city called "Považská Bystrica") - however, this is not the intention, the +421 is the country code and the '2' is the area code (for Bratislava).

Valid SK phone numbers (regex that we use):
/international/
(ALLTRIM(ncountrycd) == '421');
AND ;
(;
/*short nomad phone numbers */
(EMPTY(ncitycd) AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '019[0-9]{2}'));
/long landline phone numbers for Bratislava region/
OR (RegExp(ALLTRIM(ncitycd),'02') AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '[2-9][0-9]{7}'));
/long landline phone numbers for other regions of Slovakia/
OR (RegExp(ALLTRIM(ncitycd),'0[3-5][1-8]') AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '[2-9][0-9]{6}'));
/long nomad other phone numbers/
OR (EMPTY(ncitycd) AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '06(0[1-2][0-9]{6}|5[0-4][0-9]{6}|9[0-6][0-9]{6})'));
OR (EMPTY(ncitycd) AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '08((00|0[6-9]|[5-9][0-9])[0-9]{6}|190000[0-9]{2})'));
/*long nomad mobile phone numbers */
OR (EMPTY(ncitycd) AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '09(00|0[1-9]|[1,4-5][0-9]|[6-8][0-9])[0-9]{6}'));
/short nomad phone numbers/
OR (EMPTY(ncitycd) AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '1(0[0-9]{2}|1[0-9]{1,4}|[2-4,8][0-9]{3}|5[0,5,8,9])'));
/short phone numbers/
OR (RegExp(ALLTRIM(ncitycd),'0(2|[3-5][1-8])') AND RegExp(ALLTRIM(CHRTRAN(ncvalue, ' ', '')), '1[6-7][0-9]{3}'));
)

@lfrailey
Thank you for taking the time to improve libphonenumber!
With respect to supporting the short numbers 1911 and 18333, we can't find official evidence that the ranges are valid. Please share any official evidence you may have. See our
guidelines.
With regards to your other query on the number “+421216877” being parsed and returning an unexpected geocode,  this is working as intended. It would not happen if you didn't have  the “+” symbol at the beginning of the number.
When the number “+421216877” is parsed with the “+” symbol it correctly identifies the country code as 421(slovakia). Please see demo.
When the “+” symbol is omitted (421216877 is provided as input with country code SK) , it is parsed as a national number and interprets 421 as the area code. This is the same number as when providing  +421421216877 as input. So the parsing of the number is working as intended . demo.
How are you using the library? Are the input numbers (like +421216877) provided by users and  do you strip the leading “+” symbol before validating them (shouldn't do this to user-provided input by the way, then you'd get unexpected results)?
You can find how the parse method works here.

This is a government record explaining phone formats in Slovakia. §7 talks about these shortened numbers:
http://www.teleoff.gov.sk/data/files/25211.pdf
We'll look into the parsing and get back to you if there are still issues.
Thanks!

@lfrailey
We will be adding 18XXX range short numbers.
With respect to the short numbers 19XX , these numbers are used for accessing data services from PSTN networks which we do not support at the moment.
Thanks

Fixed in release 7.4.4
http://repo1.maven.org/maven2/com/googlecode/libphonenumber/libphonenumber/7.4.4

Hi there,
We're having trouble validating the change. For example, using the same number provided above (18 333) we're still receiving an invalid number: https://libphonenumber.appspot.com/phonenumberparser?number=%2B421+18+333&country=SK.
We do see that the isValidShortNumber is TRUE, but isPossibleNumber is FALSE. What's the difference between these? There aren't valid formatting results either.
If you could explain this a bit more, that would be very helpful.
Thanks!
Liz

To validate short numbers, you need to use ShortNumberInfo.
PhoneNumberUtil, and its method isPossibleNumber, will return false.
Formatting is tricky: we don't prioritise formatting short numbers because
there is no standard, and we also don't want the as-you-type-formatter to
perform worse for cases where national numbers that are incomplete clash
with short numbers, so usually we just do them as a block unless we have a
good reason to do something else. Here you'll see 18333 is formatted as a
block.
2016-07-06 21:49 GMT+02:00 lfrailey<EMAIL_ADDRESS>
Hi there,
We're having trouble validating the change. For example, using the same
number provided above (18 333) we're still receiving an invalid number:
https://libphonenumber.appspot.com/phonenumberparser?number=%2B421+18+333&country=SK.
We do see that the isValidShortNumber is TRUE, but isPossibleNumber is
FALSE. What's the difference between these? There aren't valid formatting
results either.
If you could explain this a bit more, that would be very helpful.
Thanks!
Liz
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/libphonenumber/issues/1147#issuecomment-230886006,
or mute the thread
https://github.com/notifications/unsubscribe/AIjisw7Sym7_3AdvuUTE5POkXnk6hTdCks5qTAbSgaJpZM4IvilI
.


Help request with networking

Hi. I'm trying to send a few integers and strings from the LOVE app to some hand-written server via internet.
Preferred method would be an HTTP request, but at this point I'm in search of any method of sending the data at all, be it over socket, WebSocket or other. Only requirement is that I can ensure the data arrives unaltered, so theoretically, TCP would fit.
I've tried making HTTP requests via socket.http builtin module to a remote website. I've tried making socket tcp connection via socket module to both: a simple local tcp server and a simple local WebSocket server; both written in Python.
In the second case with socket connection, the servers notice some attempts of establishing a connection (inconsistently for some reason), but the test message is not transferred. For both servers, the client LOVE app just logs: WebSocket connection to 'ws://<IP_ADDRESS>:12345/' failed: WebSocket is closed before the connection is established.. In case with simple tcp server, it receives the HTTP GET request as a text that is similar to the request that is used to initialize a WebSocket connection, but not the test message. In case with simple WebSocket server, it throws an exception with a message: websockets.exceptions.ConnectionClosedError: no close frame received or sent.
I will appreciate any help, and thanks in advance for your time.

I've figured it out by using Love.js-Api-Player, it was pretty straightforward. I made a function in JS that used fetch to send http requests, I include the function to the generated by lua.js website and then call that function from Lua code and it works perfectly!
Hope it helps somebody out there.


Makefile Depends error

Package Scaner is missing dependencies for the following libraries
libpcap.so.1.3
libpthread.so.0
===> Makefile line 14 ：
DEPENDS:=+libpcap +libpthread
DEPENDS not Depends

For lib-missing problems, I solve them by building my own Openwrt firmware.
For libpthread.so.0, I simply build librt(or libpthread) into Openwrt which is in Base->Base system.
And libpcap.so.1.3 missing problem can be solved in this way too (forgot where it is).And of course you can directly install libpcap.ipk.
If you understand Chinese, please step to My Blog for more information.

Problems Solved. Thx.


addDecoratorsLegacy doesn't work for test

I'm using latest CRA 4.0.1 + latest customize-cra + rewired, the only option defined is addDecoratorsLegacy
yarn start and yarn build works perfectly.
yarn test does not, it terminates with an error of:
Support for the experimental syntax 'decorators-legacy' isn't currently enabled

I figured out, this is by design. Solution was to add babel.config.js with the following contents:
module.exports = {
  presets: ['react-app'],
  plugins: [['@babel/plugin-proposal-decorators', { legacy: true }]],
}

Nothing else was needed (no changes to config-overrides.js or to package.json).


Remove client requirement from list command

Description of PR
The base list command should not create a k8s client. Only the 'list packages' command needs the client. The upcoming 'list bundles' command does not. So, this patch moves the client into the child command instead of the base 'list' command.
Signed-off-by: Brad P. Crochet<EMAIL_ADDRESS>
Fixes #308

Changes (required)

Move the client creation into the subcommand of 'list packages'

Checklist (required)

[X] I have reviewed and followed the contribution guidelines
[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation if needed
[X] I have checked that my changes pass all tests

/lgtm

/approve


⚠ AGEG website (prod) has degraded performance

In a39fc64, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2019 ms

Resolved: AGEG website (prod) performance has improved in cca4a1b.


Updating a form with v-model breaks repeater under certain conditions

Reproduction
https://formkit.link/5da0c8fd5c4557ac946407fd1018a5d6
Describe the bug
When creating an uninitialized form with a repeater, the repeater shows at least one item, allowing us to directly input without clicking on the add button, but when initializing the form with some data using v-model or node.input, if the value of the repeater is an empty array, no items are shown, meaning that if add-button is set to false, we cannot interact with the repeater.

Using v-model: https://formkit.link/5da0c8fd5c4557ac946407fd1018a5d6
Using node.input: https://formkit.link/1f5f4df8d445af389c5ea261c23173a1

Environment
• OS: macOS 13.3.1
• Browser: Firefox
• Version: 114.0.2 (64-bit)

Gotcha, thanks for providing an example.
I wasn't aware of this behaviour, but it makes sense now that you mention it.
I think adding a line regarding this in the documentation would be a good idea. I'll see if I can send a PR.

@justin-schroeder
Just to make sure, when you say that this is the intended behavior, is there a reason why the repeater render an item when initializing the repeater with an empty array using :value ?
I would think this also falls under "you are explicitly requesting the form be set to an empty array".
Here's an example: https://formkit.link/0067615c7e3c87a4b61206875741f8af

At instantiation the repeater ensures that there are at least the min number of items in the array, if there aren’t it fills the array up till it reaches that number (https://formkit.link/6fbb1770c496725929119109690f6ea7). However after the repeater boots up if you explicitly change the value (via node.input or v-model) to something lower than the min it assumes that was an intentional decision on the developer’s part and allows it. The UI on the other hand does not allow a user to go over/under the min/max props. Again, that behavior can be modified via plugin.
Hope that clears up any confusion 👍

Thanks for the clarification, all clear now. Thanks for taking the time.


Norway

As noted in https://github.com/corona-warn-app/cwa-backlog/issues/9#issuecomment-652980998 , the Corona-Warn-App is now also available in App stores for Norway!

I was just about to complain that six days after this issue was closed the app is still not released, but apparently it has indeed been released in the last half hour.


Support away message

client sends the server the away message and we just need to store that in the db and send it to people when they send a message and the user is offline

Support for saving away messages to DB was added in commit 15d4e20 (a fair while back)

Removing good first issue since this is being implemented in the TypeScript rewrite


[Bug]: [currency input widget] Default value validation should throw error for boolean values

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Currency widget default value validation is coercing boolean values to numbers. We need to throw errors

in such cases instead of coercion.
Steps To Reproduce

Add a currency widget
add {{false}} as default value.
observe that evaluated value is coming as 0.

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
cloud

I have reproduced the issue

and I can see that it asks for a number yet I am giving a boolean and it is converting it to 0

Can I contribute to it?

I was able to reproduce the bug and fix it. This would be my first ever contribution. Can you assign it to me?

I can take this issue if no one is in line.

I am opting out on this one, If you want it then please go ahead

Started working on it

This bug was fixed. Not able to reproduce.

Hello, I would like to work on this, can it be assigned to me

Hi, This issue seems to be fixed. I am not able to reproduce this. Why is this still open ? Is there something else that needs to be done ?


Covert tool should create a lowercase schema

Mixed-case struct field names don't work in Hive.  There should be a way to convert a camel-cased JSON document into ORC without having to pre-process the JSON.
This pull request is a proof-of-concept which generates two schemas, one using the default case which is provided to the JsonReader as usual, and another schema which is lower cased and is provided to OrcFile.
TypeDescription is immutable and non-trivial to manually clone using public accessors, so to make the idea clear, I do the conversion at schema ingest rather than where it's provided to OrcFile.  The downside of this approach is that automatic schema detection doesn't benefit from these changes.  A more experienced implementer could certainly do better.

Actually, this is doesn't match my experience. Hive doesn't support uppercase letters in the top level column names, but it does support them in sub-structs. Furthermore, ORC from other systems can handle the uppercase letters.
You might also try setting "orc.schema.evolution.case.sensitive" to false to get name-based matching of the ORC types that isn't case sensitive.
In terms of this patch, can you add a parameter that allows the user to downcase the column names?

Thinking about this more, I'd like to suggest that if the user asks for a lowercase schema that when the fields of a struct are parsed, they are lowercased there. That will prevent "aa" and "AA" both being converted to "aa", but not being merged into a single field.


tests: enforce super in lifecycle hooks

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Made all existing setup and teardown methods call super, and added a LifecycleEnforcer test plugin that will enforce this for new tests.
This will allow us to have global setup and teardown for tests.
For example, we can automatically clear caches after each test, to avoid annoying intermittent failures like #1879 and #1886.

Great work. Feels like the best solution.


fix(libscap): add support for PT_ENUMFLAGS to scap_event_encode

Signed-off-by: Luca Guerra<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area driver-kmod

/area driver-ebpf

/area libscap

/area libsinsp

/area tests

/area proposals

What this PR does / why we need it:
Add the relevant PT_ENUMFLAGS 8, 16, 32 to the respective cases when encoding events.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
To complete the pull request process, please assign gnosek after the PR has been reviewed.
You can assign the PR to them by writing /assign @gnosek in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Hi @LucaGuerra. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/ok-to-test

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

LGTM label has been added.  Git tree hash: 4447be3c7d1fe5752a6363b1bf542e8d76af2281

Thank you @LucaGuerra!
/approve

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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


StoneBlock 3 Crashing after world loading

Please provide a brief summary of the bug
Hi,
Need some help with Stone Block 3 crashing a few seconds after world is loaded.
This is a new world first time playing.
Tried re-installing but got the same error.
Disabled MineTogether chat as well.
Working on Stone Block 3 version 1.5.0
Java installed: (build 1.8.0_351-b10)
FTB launcher sent out this error report file.
hs_err_pid3528.log
Please provide steps to reproduce where possible
Launch game normally and create new world.
Expected Results
World loaded
Actual Results
World loads but apps crashes after few seconds of world loading.
What Java Version are you using?
Version 8 build 351 (build 1.8.0_351-b10)
What is your operating system and platform?
Windows 10 , 64 bit Build 19041 (10.0.19041.2364)
How did you install Java?
an installer package
Did it work before?
No response
Did you test with other Java versions?
No response
Relevant log output
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x00007ffeaea622f0, pid=3528, tid=10924
#
# JRE version: OpenJDK Runtime Environment Temurin-17.0.2+8 (17.0.2+8) (build 17.0.2+8)
# Java VM: OpenJDK 64-Bit Server VM Temurin-17.0.2+8 (17.0.2+8, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, windows-amd64)
# Problematic frame:
# V  [jvm.dll+0x1222f0]
#
# No core dump will be written. Minidumps are not enabled by default on client versions of Windows
#
# If you would like to submit a bug report, please visit:
#   https://github.com/adoptium/adoptium-support/issues
#

Please try the recommended Microsoft Build of OpenJDK 17.0.3 (that comes built into the std Minecraft install) or Adoptium 17.0.5 and please post the error log here.

I let Overwolf take care of everything (recommended java setting) and got crash with the same problem

Log:
hs_err_pid38780.log

Check #666  this fixed my issue


Setup$

fixed typo: rcm -> rvm

Thanks for the PR!
rcm is not a typo. It is a library we use  for managing dotfiles: https://github.com/thoughtbot/rcm. We don't use rvm at thoughtbot. We use asdf.


Config to turn off categories

A config option to turn off item categories?
Like turn off weapons and bows, and only add armors.
Or turn off armors, but add all weapons and bows.

Yes exactly! Thank you!