Docker in Mac - Cannot find rclone remote

Unifi Protect Backup version: 0.7.3
Unifi Protect version: 2.0.1
Python version: 3.9.12
Operating System: MacOS 12.4
Are you using a docker container or native?: Docker

Description
I am trying this out in Docker on a Mac but having trouble with the app finding the rclone remote. Error message:
raise ValueError(f"rclone does not have a remote called `{remote}`")

ValueError: rclone does not have a remote called `unifirclone`

What I Did
Docker script
docker run \
  -e UFP_USERNAME='unifibackup' \
  -e UFP_PASSWORD='REDACTED' \
  -e UFP_ADDRESS='<IP_ADDRESS>' \
  -e UFP_SSL_VERIFY='false' \
  -e RCLONE_DESTINATION='unifirclone:/12t-unifi-rclone' \
  -e RCLONE_RETENTION='180d' \
  -v /Users/porterranch/.config/rclone:/data \
  --mount type=bind,source="/Users/porterranch/.config/rclone/rclone.conf",target=/config/rclone.conf \
  -v /Users/porterranch/.config/rclone/:/config \
  ghcr.io/ep1cman/unifi-protect-backup

Rclone Config
[unifirclone]
type = dropbox
client_id = REDACTED
client_secret = REDACTED
token = {"access_token":"REDACTED","token_type":"bearer","refresh_token":"REDACTED","expiry":"2022-08-15T11:05:50.859498-07:00"}

Screenshot of docker inspect window

Traceback
Traceback (most recent call last):

  File "/usr/bin/unifi-protect-backup", line 8, in <module>

    sys.exit(main())

  File "/usr/lib/python3.10/site-packages/click/core.py", line 1137, in __call__

    return self.main(*args, **kwargs)

  File "/usr/lib/python3.10/site-packages/click/core.py", line 1062, in main

    rv = self.invoke(ctx)

  File "/usr/lib/python3.10/site-packages/click/core.py", line 1404, in invoke

    return ctx.invoke(self.callback, **ctx.params)

  File "/usr/lib/python3.10/site-packages/click/core.py", line 763, in invoke

    return __callback(*args, **kwargs)

  File "/usr/lib/python3.10/site-packages/unifi_protect_backup/cli.py", line 109, in main

    loop.run_until_complete(event_listener.start())

  File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete

    return future.result()

  File "/usr/lib/python3.10/site-packages/unifi_protect_backup/unifi_protect_backup.py", line 278, in start

    await self._check_rclone()

  File "/usr/lib/python3.10/site-packages/unifi_protect_backup/unifi_protect_backup.py", line 414, in _check_rclone

    raise ValueError(f"rclone does not have a remote called `{remote}`")

ValueError: rclone does not have a remote called `unifirclone`

Thanks for your patience, could you try running the following to see if your config file is getting properly loaded:
docker run \
  -e UFP_USERNAME='unifibackup' \
  -e UFP_PASSWORD='REDACTED' \
  -e UFP_ADDRESS='<IP_ADDRESS>' \
  -e UFP_SSL_VERIFY='false' \
  -e RCLONE_DESTINATION='unifirclone:/12t-unifi-rclone' \
  -e RCLONE_RETENTION='180d' \
  -v /Users/porterranch/.config/rclone:/data \
  --mount type=bind,source="/Users/porterranch/.config/rclone/rclone.conf",target=/config/rclone.conf \
  -v /Users/porterranch/.config/rclone/:/config \
  ghcr.io/ep1cman/unifi-protect-backup
  rclone listremotes

if you ignore the s6-rc log entries, there should somewhere be a list of the remotes it has configured, for example I see:
[ls.io-init] done.
s6-rc: info: service 99-ci-service-check successfully started
local:
onedrive:
s6-rc: info: service 99-ci-service-check: stopping

Hi,
I just updated the README as there were some issues reported by another user. Please check that out to see if it helps

@ep1cman yes that did it, thank you.... for anyone who comes back here using Docker, just make sure that the rclone config file target is "/config/rclone/rclone.conf" instead of "/config/rclone.conf."


RFC: avoid recommending github pages

💥 Proposal
Github pages is a free hosting solution that works for Docusaurus.
It's the first thing that users see here: https://v2.docusaurus.io/docs/next/deployment
But it's not the most easy to setup, requires commiting to a gh-branch of the repo, give write access...
There are issues like #2846 where people struggle to set it up, while it's not directly related to Docusaurus

I think we should stop recommending Github pages as the primary deployment option, and actually document that it's the most complicated option to setup.
Alternatives like Netlify, Vercel, Surge, Render etc... :

are also free (have a good enough free plan, not worst than Github CI)
are much simpler to setup (no need to commit to the github repo, give write access...)
do not mess-up with the "doc last updated by X on X" (see #2952)
can add deploy previews for docs PR before merging

Most of the users are open source project developers and uses Docusaurus for their documentation website. Majority of them hosted on Github and their docs are hosted on Github Pages which integrates nicely. Even docusaurus.io itself is hosted on Github Pages?!

@shinebayar-g

Majority of them hosted on Github and their docs are hosted on Github Pages which integrates nicely.

What do you mean by "integrates nicely"? Have you tried alternatives and find they are not as good integration as Github pages? I have and really think Github Pages is not the most convenient tool to host a Github-based website, since a few years already.

Even docusaurus.io itself is hosted on Github Pages?!

Actually, we have 2 websites on this monorepo, v1 and v2. I don't even think it's possible to publish more than 1 website per repo on Github pages.
V1 seems to use Github Pages: https://github.com/facebook/docusaurus/tree/gh-pages
This requires v1 to provide publish-gh-pages.js, a 200 LOC cli to make it work correctly.
V2 is using Netlify.
It is very easy to setup and actually add a deploy preview to each PR.
Setting Github pages might be quite easy if you run locally, but in practice, people that want to automate the publish process (ie commit to master = deploy) struggle a bit, because their CI need write permissions to their repo.

people that want to automate the publish process (ie commit to master = deploy) struggle a bit, because their CI need write permissions to their repo.

Is there some reason why documentation requires SSH-key setup, instead of just using a provided GITHUB_TOKEN secret? With that actions workflow is pretty simple: example 1, example 2

@ark120202 didn't know it was actually possible.
Maybe we just need to rewrite the Github CI part of the doc?
https://v2.docusaurus.io/docs/deployment/#triggering-deployment-with-github-actions
Any PR would be appreciated, as I'm not familiar with Github CI, and users seems to follow our existing doc and some struggle a bit as the doc is unnecessarily complex.
After taking a look, it seems the Github workflow and Travis workflow could share almost the same code.
@Jason-2020 seems to be using a workflow similar to the documented Travis workflow while he's using Github CI: https://github.com/facebook/docusaurus/issues/2798#issuecomment-634375740

people that want to automate the publish process (ie commit to master = deploy) struggle a bit, because their CI need write permissions to their repo.

Is there some reason why documentation requires SSH-key setup, instead of just using a provided GITHUB_TOKEN secret? With that actions workflow is pretty simple: example 1, example 2

That's mentioned here too :)

So it seems @amimas agrees with me, feels less alone :D
https://github.com/facebook/docusaurus/issues/2736#issuecomment-634396631

In my opinion, this tool needs to focus in it's core functionality, which is to generate a static documentation site that is easy to maintain.
Publishing and deployment doesn't really fit in that core function although it can easily mistaken for thinking otherwise.
The generated site can be deployed in so many ways. For example:

github pages
gitlab pages
docker image
aws cloud
azure cloud
traditional web server with nginx or apache or tomcat

I'm sure I missed other options. Add other services like netlify that @slorber  mentioned and the list gets much bigger.
Basically this is all about how to deploy/publish a static site and it's applicable for any tools in this category. Adding deployment functionality means they also need to be maintained going forward. And it's not possible to satisfy every type of deployments nor is it the focus of this tool.

Revisiting this after a year: GitHub pages still remains as one of the most popular choices, if not gaining more popularity. For casual users (especially small open-source libraries) it's hard to say if GitHub pages is harder to set up than Netlify. The docusaurus deploy command has masked away all the details of the deploy workflow, and there are also out-of-the-box GitHub actions that takes the directory on master, runs build, and copies the output to gh-pages.
I think Docusaurus should be less opinionated in various ways, both in our code base and in our documentation. In the community, I'm constantly asked of how to do things that are actually pretty terrible—but instead of saying "it's terrible, you'd better do X", my responsibility is to just tell them that "do A, B, and C", and leave them to eventually figure out that there's another way to do things. It's the same for our documentation. GitHub pages is terrible in many ways, e.g. server-side redirects, but:
(a) Users know what they are doing when they go to the GitHub pages tutorial. We should not imagine our users asking "how should I deploy my site?" but rather "I want to host it on GitHub pages. How do I do it?" I think that's also the general positioning we are taking in our docs—provide solutions, not suggestions.
(b) At the end of the day Netlify is far less free than GH Pages—my own deployments are on the verge of crossing the limit every month.
(c) We already provide first-class support for GH Pages in our code base by allowing config options that only apply to GH Pages deployment, and also a docusaurus deploy command that's targeting nothing but GH Pages.
(d) GH Pages is non-proprietary (afaik) while the others are. While we document Crowdin for backward-compatibility and Algolia for partnership, I think recommending Netlify / Vercel makes our doc less neutral.

(a) Users know what they are doing when they go to the GitHub pages tutorial. We should not imagine our users asking "how should I deploy my site?" but rather "I want to host it on GitHub pages. How do I do it?" I think that's also the general positioning we are taking in our docs—provide solutions, not suggestions.

I don't really agree here. This overestimate the number of devs that have no idea that services like Netlify/Vercel exists and why they are way better than GitHub pages. When they discover such services they generally are very impressed by deploy previews etc and really willing to test this.

(b) At the end of the day Netlify is far less free than GH Pages—my own deployments are on the verge of crossing the limit every month.

I assume you talk about 300 Netlify build limits vs 2000 build minutes for GH? In both cases there's a limit.
Netlify can also be deployed with a CLI without any build minute.

(c) We already provide first-class support for GH Pages in our code base by allowing config options that only apply to GH Pages deployment, and also a docusaurus deploy command that's targeting nothing but GH Pages.

yes, not a huge fan of those

(d) GH Pages is non-proprietary (afaik) while the others are. While we document Crowdin for backward-compatibility and Algolia for partnership, I think recommending Netlify / Vercel makes our doc less neutral.

Being non-neutral also gives the opportunity to get some sponsors, and Netlify/Vercel are usually happy do to so.
Already suggested that, but afaik Facebook/Meta is not really willing to do that atm, or at least it requires some internal process that I'm not really aware of.

I don't really agree here. This overestimate the number of devs that have no idea that services like Netlify/Vercel exists and why they are way better than GitHub pages. When they discover such services they generally are very impressed by deploy previews etc and really willing to test this.

I think we can do a cost-benefit analysis here like we did for Crowdin vs. git

2000 build minutes for GH

2000 is quite a lot—6x compared to Netlify. My site takes 20 min for each build (yes, I still haven't figured out how caching works) and that means I can build 15 times for Netlify and 100 times for GitHub. Also I think we need to separate CI from hosting. In practice I also use CircleCI for two legacy websites, which offers 6000 min/mo.
All that said, deploy time has really been a bottleneck for me, despite Netlify being much more featured.
I also remember back in the days GH Pages was really hard to set up, but now it's quite intuitive. Already explained in our own docs: https://docusaurus.io/docs/next/deployment#github-pages-overview It's just my-org/my-project => https://my-org.github.io/my-project/ and my-org/my-org.github.io => https://my-org.github.io/. There's no magic. Even better: you can host your source on the master branch and deploy to gh-pages branch even if you are using my-org/my-org.github.io. I already do that myself: https://github.com/Josh-Cena/Josh-Cena.github.io

2000 build minutes for GH?

Github doesn't have any limits for Public repos btw.


Add currently_executing_a_context_hook? to ExampleGroup

This is useful because at the moment the only way to determine if an
example is currently executing a context hook is to look at it's inspect
string. Here, we add a simple boolean flag which is triggered in before
and after context hook execution.
This is needed by https://github.com/rspec/rspec-rails/pull/1501.

LGTM :+1:  some minor comments nothing I consider a merge blocker

Closes #2130


Add a "donate" button under settings

One thing that I've been asked before is how to donate, what about adding a donate option, mentioning it would go to the contributors their time (and likely to their OW funds). Is there an easy service that would do this? Even better if it actually showed contributions and total, an incentive for people to donate.

https://github.com/PrivacyApps/donations

Well damn, bitcoins will get us the toys... I'll look into adding this.
On Dec 29, 2017 1:12 PM, "Erik Babel"<EMAIL_ADDRESS>wrote:

https://github.com/PrivacyApps/donations
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ponewheel/android-ponewheel/issues/50#issuecomment-354501044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0LrR6RedyWBbKW_XYHZrqD3j3q9cVeks5tFVWtgaJpZM4RPRbJ
.

Closing, we got a PayPal link for now.


Two tests for the same thing?

The test case from 5.1: "half closed (remote): Sends a DATA frame" appears to be testing the same path as the test case from 6.1: "Sends a DATA frame on the stream that is not opened". Perhaps one should be deleted or the test cases disambiguated more clearly? The 6.1 test in particular is somewhat confusing since "not opened" could mean a few different states.

Thank you the feedback!
As you say, this test is duplicated. Since the test cases of h2spec have been created based on the RFC, if the description in RFC is duplicated, the test cases of h2spec is also duplicate.
However, the term "is not opend" will lead to confusion. So I'm going to change the description of the test case.

The description was fixed in 5f374d94ec81f4f7a59cd875c60cebe220ef6abc.


Wiki link not working

Wiki link redirects to blank page saying "We didn't receive a proper request from your browser. Please contact us if the problem persists."
Happens on macOS 1.15.4 with latest Safari, Chrome, Firefox

I'm just wondering where the docs are, I had to use pry and cd into OAuth2::AccessToken just to identify what method to use to know if a token if expired or not (expired? by the way).

I usually check rubydoc before open gem's source code. i.e. https://www.rubydoc.info/gems/oauth2/OAuth2/AccessToken
But agree that we need more documents and examples.

Fixed in https://github.com/oauth-xx/oauth2/pull/515. I'm going to update wiki per question issues.

Thanks @terberes  I updated wiki link in README.md


Password reset emails take 10min to arrive

We probably can not do anything in the short term. But this is sadly quite slow.

The last time I tried, it was pretty much instant for me 🤔

it came within a few seconds for me. Not sure if we have to keep it open @maxammann ?


Prevent corruption of UTF-8 multibyte codepoints at fragment boundary

Problem:

When a string payload is fragmented, it was possible for the boundary to fall within a multibyte code point. When the payload is converted to a string with each fragment, it's possible to have corrupted codepoints at the start/end of fragments, leading to data corruption.

Solution:

Concat buffers as before, but do not convert to a string and populate responseText until all fragments have been received.

Co-authored-by: Paul Ringseth<EMAIL_ADDRESS>

I've just released version 2.1.1 on npm which includes your fix.


[Docs]: Fix the inconsistencies in the cleanup proposal

Fixed the inconsistencies, and replaced the annotations with labels and similar fixes.
Closes #44

@VedRatan We need to make sure we can watch on two labels combined with an OR.

oh sure

If not possible we will have to use a single label (containing either a duration or a full date/time).

I need some time to figure out the right decision on both implementation strategies.

It’s always combined with AND https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#label-selectors

Oh, got it.

Any more changes required?

We’ll see during implementation.

What is the plan to make this feature stateless so that failure of one of the cleanup replicas does not cause disruption?

What is the plan to make this feature stateless so that failure of one of the cleanup replicas does not cause disruption?

What do you mean ? Can you elaborate ?

Assuming this reconciliation will be handled by the cleanup controller, it allows multiple replicas. If there are two replicas and the leader fails, will the other replica be able to handle all the cleanup duties for these labeled resources without losing some?

Yes

Ok. It's probably worth having a statement in this KDP to acknowledge this as a requirement: The solution must allow for proper tracking and accurate cleanup even upon controller replica failure or failover.

The desired state can be established at any point in time from the observed state. Reconciliation should always be possible and accurate.

Yes, but this needs to be explicitly tested to ensure such a failure can be tolerated. For example, if a replica failure occurs just before a resource is scheduled to be cleaned up and recovery or take-over by another replica should, within a minimal amount of time, see that resource has "missed" its cleanup interval and should be removed.

Such a test sounds difficult to implement, however the design will guarantee that if a resource failed to be deleted it will be picked up by the next leader.

Just saying this should be tested during development to ensure it actually happens as intended. We can handle automated tests at a later point.

@JimBugwadia
We don't emit events, there's no policy to attach events to, do you think we should/can emit events ?
For metrics, we will have standard controller metrics and we will create additional metrics next week.
We also have logs when a resource is deleted, nothing more for reporting.
Anything you want to add ?


Catch crypto error.

Unfortunately, the app doesn't seem to handle bad passwords well. For me it crashes.


CLI "put" fails due to path options handling inconsistency

Trying to execute e.g.:
bosch_cli put [...] --protocol XMPP --device EASYCONTROL --path /system/awayMode/enabled false
fails with 400 "Bad request" errors from the gateway. The log shows the following, note the tuple enclosing of the path string:
[bosch_thermostat_client.connectors.xmpp] Sending PUT request to ('/system/awayMode/enabled',) with value true
I believe the root cause is that the click options parsing in bosch_cli.py for the put command is configuring --path to be used multiple times (as in --query) but treating it like a non-multiple argument by virtue of the type used in the definition of the put() function:
_path_options = [
    click.option(
        "-p",
        "--path",
        type=str,
        required=True,
        multiple=True,
        help="Path to run against. Look at rawscan at possible paths. e.g. /gateway/uuid - Can be specified multiple times!",
    )
]

@cli.command()
@add_options(_cmd1_options)
@add_options(_path_options)
@click.argument("value", nargs=1)
@click.pass_context
@coro
async def put(
    ctx,
    host: str,
    token: str,
    password: str,
    protocol: str,
    device: str,
    path: str,    <====================
    debug: int,
    value: str,
):

Both --put and --query share the definition of _path_options. The latter defines path as list[str]:
@cli.command()
@add_options(_cmd1_options)
@add_options(_path_options)
@click.pass_context
@coro
async def query(
    ctx,
    host: str,
    token: str,
    password: str,
    protocol: str,
    device: str,
    path: list[str],    <====================
    debug: int,
):

My workaround is to change the type in the put() definition to list[str] and pass the first element from the list when calling _runpush (i.e. await _runpush(gateway, path[0], value)). This does lead to a successful put of the value to the thermostat.
A proper fix would probably be to have a separate non-multiple definition of _path_options for the put command, rather than have it share the definition with the query command.
bosch-thermostat-client v0.23.2 / Python 3.11.2 / click 8.1.7

Fixed in v0.25.0dev


panic: tocommit(25) is out of range [lastIndex(16)]. Was the raft log corrupted, truncated, or lost?

I have created different organizations in multiple servers. Each organization has multiple peer nodes and an orderer node. Everything on one server has been cleared. Later, I redeployed the organization with a configuration file. The orderer container has the following problems. Containers on other servers are running normally
Orderer logs:
[orderer.consensus.etcdraft] commitTo -> PANI bda tocommit(25) is out of range [lastIndex(16)]. Was the raft log corrupted, truncated, or lost? channel=fabric-channel node=16
panic: tocommit(25) is out of range [lastIndex(16)]. Was the raft log corrupted, truncated, or lost?
goroutine 60 [running]:
github.com/hyperledger/fabric/vendor/go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc00027e000, 0x0, 0x0, 0x0)
/go/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/zapcore/entry.go:229 +0x546
github.com/hyperledger/fabric/vendor/go.uber.org/zap.(*SugaredLogger).log(0xc0001302e8, 0x4, 0x1085297, 0x5d, 0xc0005a5aa0, 0x2, 0x2, 0x0, 0x0, 0x0)
/go/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/sugar.go:234 +0x100
github.com/hyperledger/fabric/vendor/go.uber.org/zap.(*SugaredLogger).Panicf(...)
/go/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/sugar.go:159
github.com/hyperledger/fabric/common/flogging.(*FabricLogger).Panicf(0xc0001302f0, 0x1085297, 0x5d, 0xc0005a5aa0, 0x2, 0x2)
/go/src/github.com/hyperledger/fabric/common/flogging/zap.go:74 +0x7c
github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft.(*raftLog).commitTo(0xc000012bd0, 0x19)
/go/src/github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft/log.go:203 +0x131
github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft.(*raft).handleHeartbeat(0xc00034f7c0, 0x8, 0x10, 0x7, 0x8, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
/go/src/github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft/raft.go:1324 +0x54
github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft.stepCandidate(0xc00034f7c0, 0x8, 0x10, 0x7, 0x8, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
/go/src/github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft/raft.go:1224 +0x7f0
github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft.(*raft).Step(0xc00034f7c0, 0x8, 0x10, 0x7, 0x8, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
/go/src/github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft/raft.go:971 +0x1398
github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft.(*node).run(0xc0007c9380, 0xc00034f7c0)
/go/src/github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft/node.go:357 +0x10d0
created by github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft.RestartNode
/go/src/github.com/hyperledger/fabric/vendor/go.etcd.io/etcd/raft/node.go:246 +0x31b

How did you redeploy? Did you backup the files of the orderer's file system or something?

How did you redeploy? Did you backup the files of the orderer's file system or something?

Thank you for your reply. I restarted all the containers in Docker-compose file and added the prepared channel file to the peer node. It is feasible, but there is a problem when installing the chain code. Moreover, the Orderer mirror is in a broken state to begin with, and the previous data should not be found. And I want this organization to resynchronize data from other organizations, what's a good way to do that?

I'm sorry but I don't understand what steps you followed.
Can you be more clear and accurately describe everything you did?

I'm sorry but I don't understand what steps you followed.
Can you be more clear and accurately describe everything you did?

In the first step, I deployed the Fabric network on multiple servers, each with an orderer node and multiple peer nodes. In the second step, I laid out the channel files and installed the chain code, approved the transaction, and initialized the chain code. Peer nodes on all machines can access the data synchronically and operate through the chain code. These two steps are normal deployment, and I have saved configuration files from deployment, including Docker-compose files, channel files, and so on. Step 3: Reinstall the operating system on one of my servers and all data is lost. I saved files to restore the organization of the machine, but found that the Orderer container and the peer container were restarted through the docker-compose file. I found that the orderer container was in exit state, so I went to see logs, and Panic happened, and nothing else happened. I just want to restore the organization, to bring the organization back into the network

I have created different organizations in multiple servers. Each organization has multiple peer nodes and an orderer node. Everything on one server has been cleared. Later, I redeployed the organization with a configuration file.

I suspect the problem is when the cleaned up/reset orderer node joined the orderering service network, the current leader(orderer) tried to send the blocks/msg based on the last known index of that orderering node.  But the node was restarted with the empty/initial state which did not match with the leader's match raft log index.
I able to create the problem with the fabric-samples test-network-nano-bash with the following steps:

Start the orderer 1, 2
Submit couple of transactions
Start the orderer 3 (which will join the quorum and replicate the blocks/raft log)
Stop the orderer 3 and delete all the ledger and raft logs
Start the orderer 3

Restart failed with the same panic:

2022-09-22 00:31:39.974 PDT 0325 INFO [orderer.consensus.etcdraft] becomeFollower -> 3 became follower at term 2 channel=test-system-channel-name node=3
2022-09-22 00:31:39.974 PDT 0326 PANI [orderer.consensus.etcdraft] commitTo -> tocommit(6) is out of range [lastIndex(3)]. Was the raft log corrupted, truncated, or lost? channel=test-system-channel-name node=3

unrecovered-panic] runtime.fatalpanic() /usr/local/go/src/runtime/panic.go:1065 (hits goroutine(43):1 total:1) (PC: 0x43d8a0)

To resolve the issue,

From the orderer log find out the current leader orderer node and restart it. This will help to pick a new leader orderer node which would initialize with the initial match index.  Then the other orderer node should be able to join the orderer network.
(or)
Restart all the orderer nodes

I have created different organizations in multiple servers. Each organization has multiple peer nodes and an orderer node. Everything on one server has been cleared. Later, I redeployed the organization with a configuration file.

I suspect the problem is when the cleaned up/reset orderer node joined the orderering service network, the current leader(orderer) tried to send the blocks/msg based on the last known index of that orderering node. But the node was restarted with the empty/initial state which did not match with the leader's match raft log index.
I am able to create the problem with the fabric-samples test-network-nano-bash with the following steps:

Start the orderer 1, 2
Submit couple of transactions
Start the orderer 3 (which will join the quorum and replicate the blocks/raft log)
Stop the orderer 3 and delete all the ledger and raft logs
Start the orderer 3

Restart failed with the same panic:

2022-09-22 00:31:39.974 PDT 0325 INFO [orderer.consensus.etcdraft] becomeFollower -> 3 became follower at term 2 channel=test-system-channel-name node=3
2022-09-22 00:31:39.974 PDT 0326 PANI [orderer.consensus.etcdraft] commitTo -> tocommit(6) is out of range [lastIndex(3)]. Was the raft log corrupted, truncated, or lost? channel=test-system-channel-name node=3

unrecovered-panic] runtime.fatalpanic() /usr/local/go/src/runtime/panic.go:1065 (hits goroutine(43):1 total:1) (PC: 0x43d8a0)

To resolve the issue,

From the orderer log find out the current leader orderer node and restart it. This will help to pick a new leader orderer node which would initialize with the initial match index.  Then the other orderer node should be able to join the orderer network.
(or)
Restart all the orderer nodes

Thank you very much. According your reply, this problem is solved.

I have created different organizations in multiple servers. Each organization has multiple peer nodes and an orderer node. Everything on one server has been cleared. Later, I redeployed the organization with a configuration file.

I suspect the problem is when the cleaned up/reset orderer node joined the orderering service network, the current leader(orderer) tried to send the blocks/msg based on the last known index of that orderering node. But the node was restarted with the empty/initial state which did not match with the leader's match raft log index.
I am able to create the problem with the fabric-samples test-network-nano-bash with the following steps:

Start the orderer 1, 2
Submit couple of transactions
Start the orderer 3 (which will join the quorum and replicate the blocks/raft log)
Stop the orderer 3 and delete all the ledger and raft logs
Start the orderer 3

Restart failed with the same panic:

2022-09-22 00:31:39.974 PDT 0325 INFO [orderer.consensus.etcdraft] becomeFollower -> 3 became follower at term 2 channel=test-system-channel-name node=3
2022-09-22 00:31:39.974 PDT 0326 PANI [orderer.consensus.etcdraft] commitTo -> tocommit(6) is out of range [lastIndex(3)]. Was the raft log corrupted, truncated, or lost? channel=test-system-channel-name node=3

unrecovered-panic] runtime.fatalpanic() /usr/local/go/src/runtime/panic.go:1065 (hits goroutine(43):1 total:1) (PC: 0x43d8a0)

To resolve the issue,

From the orderer log find out the current leader orderer node and restart it. This will help to pick a new leader orderer node which would initialize with the initial match index.  Then the other orderer node should be able to join the orderer network.
(or)
Restart all the orderer nodes

Thank you. This comment solved my problem.


Can you provide a screenshot "Then, go to Object > Node to Python, and type in the name of your node group"

Hey Brendan,
Cool project. I'm a coder just learning blender, I would love to explore some geo node's I found online as code.
If you mean Object in the outliner panel I don't see the option there after installing your script as an addon.

Sure thing!
You'll want to click on an object, and in the upper right there's the object panel. It'll register at the very bottom.

Once you click on it you'll be given a box in the lower left to type in the name of your node group.
If you prefer accessing it from the Python console, you can do so with
>>> bpy.ops.object.node_to_python(node_group_name = "YOUR_NODE_GROUP_NAME_HERE")

I don't like where I'm currently registering that button or the method of selecting a node group, so this will probably change in future releases.


Revert EXTERNAL_ISTIOD default true

Please provide a description of this PR:
A previous PR #38283 changed the default value of values.pilot.env.EXTERNAL_ISTIOD from false to true.
This has not been released yet, and after some offline discussions with @howardjohn about backward compatibility,
I think that we should probably be more conservative with this.
As it stands, when users upgrade from istiod 1.14 to 1.15, things will continue to work as before but, unless they add
additional annotations to their clusters, they will start seeing error messages in the istiod logs when using a multicluster configuration. Even though things will continue to work, this is not ideal.
So, rather than changing the default value, I think we should leave it off by default so there is no change in behavior unless the EXTERNAL_ISTIOD feature is turned on.
We will still make it "the default", by testing it with the flag turned on, and by changing the multicluster documentation to turn it on. I think this has the added advantage of giving us a clear place in the docs to explain what it means to turn on the EXTERNAL_ISTIOD flag when installing istiod.

@frankbu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
b75c15b3808a0ccc3e0362bd1a331dc461072ee9
link
true
/test release-notes_istio

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. I understand the commands that are listed here.

/test release-notes_istio

@frankbu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
b75c15b3808a0ccc3e0362bd1a331dc461072ee9
link
true
/test unit-tests_istio

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. I understand the commands that are listed here.

@frankbu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-distroless_istio

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. I understand the commands that are listed here.

@frankbu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-distroless_istio

integ-helm_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-helm_istio

integ-ipv6_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-ipv6_istio

integ-pilot_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-pilot_istio

integ-operator-controller_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-operator-controller_istio

integ-cni_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-cni_istio

integ-telemetry_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry_istio

integ-security_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-security_istio

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. I understand the commands that are listed here.

@frankbu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-distroless_istio

integ-helm_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-helm_istio

integ-ipv6_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-ipv6_istio

integ-pilot_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-pilot_istio

integ-operator-controller_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-operator-controller_istio

integ-cni_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-cni_istio

integ-telemetry_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry_istio

integ-security_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-security_istio

unit-tests_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test unit-tests_istio

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. I understand the commands that are listed here.

@frankbu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-distroless_istio

integ-helm_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-helm_istio

integ-ipv6_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-ipv6_istio

integ-pilot_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-pilot_istio

integ-operator-controller_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-operator-controller_istio

integ-cni_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-cni_istio

integ-telemetry_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry_istio

integ-security_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-security_istio

unit-tests_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test unit-tests_istio

integ-telemetry-mc_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry-mc_istio

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. I understand the commands that are listed here.

@frankbu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-distroless_istio

integ-helm_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-helm_istio

integ-ipv6_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-ipv6_istio

integ-pilot_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-pilot_istio

integ-operator-controller_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-operator-controller_istio

integ-cni_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-cni_istio

integ-telemetry_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry_istio

integ-security_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-security_istio

unit-tests_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test unit-tests_istio

integ-telemetry-mc_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry-mc_istio

release-test_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test release-test_istio

integ-telemetry-istiodremote_istio
7e24f9ffced19c35638623f7ef218bb294617222
link
true
/test integ-telemetry-istiodremote_istio

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. I understand the commands that are listed here.

/test all

/retest


Added $IKINGS

I hope we can merge :)

Added from a local branch. Your image name was wrong in the token information, be careful next time.


TW banner and page footer

Create the TW banner and page footer to finish MVP for Hawaii public page
Business Context:
I want to display the TW partnership banner and a footer on the Hawaii's public page.
In Scope

Include the TW banner on the bottom of the page
Below the banner, include the footer with links as specified on the mockups.

Links:
Banner (contributor partner): https://www.thoughtworks.com/about-us/social-change
About us: https://hcsoc.hawaii.gov/about-us/
Contact us: https://hcsoc.hawaii.gov/contact-us/
Terms of Use: https://portal.ehawaii.gov/page/terms-of-use/
Accessibility: https://portal.ehawaii.gov/page/accessibility/
2023 Privacy Policy:  https://portal.ehawaii.gov/page/privacy-policy/
Designed and developed in partnership with Thoughtworks:  https://www.thoughtworks.com/
Out of Scope
Acceptance Criteria

Scenario:
Given I go to /public-data using mobile, tablet or desktop
Then I see the TW banner on the page
And I can click on the banner and opens the TW website on a new tab  (https://www.thoughtworks.com/about-us/social-change)

Scenario:
Given I go to /public-data using mobile, tablet or desktop
Then I see a footer on the bottom of the page
And I can click on the links which opens the pages on a new tab.

Mockups
Link to the figma mockups
https://www.figma.com/file/SHMDMgzr7damRoT12ToEz5/Hawaii-CSOC-Advocacy--2?type=design&node-id=428%3A3237&mode=design&t=UZP5VFPI8pVjkP6G-1
You can find Desktop/Tablet/Mobile View in the Dev Handoff Section
Checklist (should be reviewed before grooming as the BA or Pair are writing the card—NOT reviewed for the first time at grooming)

[ ] Ensure to include all necessary mockups to this card.
[ ] Discuss what success/failure user feedback messages would be present, if any occur (front end/back end).
[ ] Discuss Feature Flag: Where could a development flag go to flag off incomplete work? Do we want to keep the flag long-term?
[ ] If this card is part of a sequence of cards, discuss if this card should be blocked until the testing of previous cards in the sequence have been finished.
[ ] Have we added edge cases as ACs that we're considered from a QA perspective
[ ] Should the new functionalities be offered only to users with certain permissions?
[ ] How should this functionality differ between different orgs, and if it does differ, what is the hinge point we should use for that functionality?

Notes:
Media-Query Breakpoints
mobile: 360px max-width
tablet: 834px max-width
desktop: 1920px max-width

To make a link open in a new tab, add target="_blank" to the  tag


func host start does not properly load azure.functions

Investigative information
Env: Windows 10 CMD, Python 3.6 w/venv installed via installer, Azure Functions Core Tools installed via chocolatey package
Function created with:
func init (Python)
func new (HTTP Trigger)
Following instructions on:
Repro steps
Provide the steps required to reproduce the problem:

func host start

Expected behavior
Application and python workers start as expected.
Actual behavior
Fails to start python worker:
(ENV) C:\Users\sttramer\src\azure-docs-cli-python\ci-merger>func host start

                  %%%%%%
                 %%%%%%
            @   %%%%%%    @
          @@   %%%%%%      @@
       @@@    %%%%%%%%%%%    @@@
     @@      %%%%%%%%%%        @@
       @@         %%%%       @@
         @@      %%%       @@
           @@    %%      @@
                %%
                %

Azure Functions Core Tools (2.3.199 Commit hash: fdf734b09806be822e7d946fe17928b419d8a289)
Function Runtime Version: 2.0.12246.0
[12/28/2018 11:24:29 PM] Starting Rpc Initialization Service.
[12/28/2018 11:24:29 PM] Initializaing RpcServer
[12/28/2018 11:24:29 PM] Building host: startup suppressed:False, configuration suppressed: False
[12/28/2018 11:24:30 PM] Reading host configuration file 'C:\Users\sttramer\src\azure-docs-cli-python\ci-merger\host.json'
[12/28/2018 11:24:30 PM] Host configuration file read:
[12/28/2018 11:24:30 PM] {
[12/28/2018 11:24:30 PM]   "version": "2.0"
[12/28/2018 11:24:30 PM] }
[12/28/2018 11:24:30 PM] Initializing Host.
[12/28/2018 11:24:30 PM] Host initialization: ConsecutiveErrors=0, StartupCount=1
[12/28/2018 11:24:30 PM] Starting JobHost
[12/28/2018 11:24:30 PM] Starting Host (HostId=sttramerspro4-639159208, InstanceId=dc3e9338-9106-488e-b539-8c5077f10100, Version=2.0.12246.0, ProcessId=18980, AppDomainId=1, InDebugMode=False, InDiagnosticMode=False, FunctionsExtensionVersion=)
[12/28/2018 11:24:30 PM] Loading functions metadata
[12/28/2018 11:24:30 PM] 1 functions loaded
[12/28/2018 11:24:30 PM] WorkerRuntime: python. Will shutdown other standby channels
[12/28/2018 11:24:30 PM] Starting language worker process:python  "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py" --host <IP_ADDRESS> --port 52822 --workerId d660bb33-ce01-4498-aede-e88eefe0d878 --requestId 19f23634-1710-43bb-9c26-d0b6468b802e --grpcMaxMessageLength 134217728
[12/28/2018 11:24:30 PM] python process with Id=19164 started
[12/28/2018 11:24:30 PM] Generating 1 job function(s)
[12/28/2018 11:24:30 PM] Found the following functions:
[12/28/2018 11:24:30 PM] Host.Functions.MergeExt
[12/28/2018 11:24:30 PM]
[12/28/2018 11:24:30 PM] Host initialized (209ms)
[12/28/2018 11:24:30 PM] Host started (222ms)
[12/28/2018 11:24:30 PM] Job host started
[12/28/2018 11:24:30 PM] Traceback (most recent call last):
[12/28/2018 11:24:30 PM]   File "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py", line 36, in <module>
[12/28/2018 11:24:30 PM]     main()
[12/28/2018 11:24:30 PM]   File "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py", line 23, in main
[12/28/2018 11:24:30 PM]     import azure.functions  # NoQA
[12/28/2018 11:24:30 PM] ModuleNotFoundError: No module named 'azure'
[12/28/2018 11:24:30 PM] Language Worker Process exited.
[12/28/2018 11:24:30 PM] python exited with code 1
 .
[12/28/2018 11:24:30 PM] Starting language worker process:python  "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py" --host <IP_ADDRESS> --port 52822 --workerId 9e890255-6327-42f3-abc1-d29a8733eb85 --requestId 590237f2-47d0-4336-8059-3159a465afca --grpcMaxMessageLength 134217728Hosting environment: Production
Content root path: C:\Users\sttramer\src\azure-docs-cli-python\ci-merger
Now listening on: http://<IP_ADDRESS>:7071
Application started. Press Ctrl+C to shut down.

[12/28/2018 11:24:30 PM] python process with Id=11484 started

Http Functions:

        MergeExt: [GET,POST] http://localhost:7071/api/MergeExt

[12/28/2018 11:24:30 PM] Traceback (most recent call last):
[12/28/2018 11:24:30 PM]   File "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py", line 36, in <module>
[12/28/2018 11:24:30 PM]     main()
[12/28/2018 11:24:30 PM]   File "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py", line 23, in main
[12/28/2018 11:24:30 PM] Language Worker Process exited.
[12/28/2018 11:24:30 PM]     import azure.functions  # NoQA
[12/28/2018 11:24:30 PM] ModuleNotFoundError: No module named 'azure'
[12/28/2018 11:24:30 PM] python exited with code 1
 .
[12/28/2018 11:24:30 PM] Starting language worker process:python  "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py" --host <IP_ADDRESS> --port 52822 --workerId 82171357-eed5-4f7b-a9a1-79166d4783fb --requestId d71361db-dae7-4bc9-84d1-29fb0c4bb41f --grpcMaxMessageLength 134217728
[12/28/2018 11:24:30 PM] python process with Id=13564 started
[12/28/2018 11:24:31 PM] Traceback (most recent call last):
[12/28/2018 11:24:31 PM]   File "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py", line 36, in <module>
[12/28/2018 11:24:31 PM]     main()
[12/28/2018 11:24:31 PM]   File "C:\ProgramData\chocolatey\lib\azure-functions-core-tools\tools\workers\python\worker.py", line 23, in main
[12/28/2018 11:24:31 PM] Language Worker Process exited.
[12/28/2018 11:24:31 PM]     import azure.functions  # NoQA
[12/28/2018 11:24:31 PM] python exited with code 1
 .
[12/28/2018 11:24:31 PM] ModuleNotFoundError: No module named 'azure'
[12/28/2018 11:24:35 PM] Host lock lease acquired by instance ID '000000000000000000000000792E1ADF'.
Application is shutting down...

Related information
When starting python3 in the venv, azure.functions can be loaded as a module:
(ENV) C:\Users\sttramer\src\azure-docs-cli-python\ci-merger>python
Python 3.6.7 (v3.6.7:6ec5cf24b7, Oct 20 2018, 13:35:33) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import azure.functions
>>> exit

This issue was moved to Azure/azure-functions-core-tools#954


Step 9

[Enter feedback here]
Maybe I'm misunderstanding the purpose of step 9. Should the second IF should include $user.LineUri.Replace("tel:+","") instead of just $user.LineUri to correctly match the lineuri in the file with the new output of Get-CSOnlineUser
if ($u.LineURI -ne $user.LineUri.Replace("tel:+","") -or $u.EnterpriseVoiceEnabled -ne $true)

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

ID: 5566de7d-1d1e-ba92-743b-dc772b49752a
Version Independent ID: c15342ff-eb3a-5f14-4bba-217c31aa1299
Content: Decide how to manage attributes after decommissioning - Skype for Business Hybrid
Content Source: Skype/SfbHybrid/hybrid/cloud-consolidation-managing-attributes.md
Product: skype-for-business-itpro
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


$('.selectpicker').selectpicker('mobile');

Some of my drop down boxes are blank after selecting an option when using $('.selectpicker').selectpicker('mobile');
If I add data-mobile="true" to my HTML everything works fine although doing it this way means that drop down boxes don't work on desktops.

See guidelines for contributing. Any chance you can link me to a demo or jsfiddle? Have you been able to reproduce this with a reduced test case?

Closing due to inactivity.


fix(sub)!: Abstract away substitution values

This will give us more implementation flexibility
Cherry pick 286ac5d (assert-rs#281)
Cherry pick be4ad91 (assert-rs#284)
Cherry pick 9e4ad59 (assert-rs#285)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

17 of 18 (94.44%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 50.998%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

crates/snapbox/src/filter/redactions.rs
17
18
94.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1303

Relevant Lines:
2555

💛  - Coveralls


Can't get the library to work on my site ??

I have been trying to get PHPML library to work ,
steps that I followed :

Downloaded the PHPML Library from github :
https://github.com/php-ai/php-ml
Composer install [ using PHP 7.1 ] {output} vendor folder size is
around 60MB
I wrote the following php page :

      <title>Test</title>

Testing

		 <?php

		 use Phpml\Classification\NaiveBayes;

require './vendor/autoload.php';

$samples = [[176, 70], [180, 80], [161, 45], [163, 47], [186, 86], [165, 49]];
$labels = ['a', 'b', 'a', 'b', 'a', 'b'];
$classifier = new NaiveBayes();
$classifier->train($samples, $labels);
print_r($classifier->predict([[172,40]]));

			?>

the issue that I face is the following when I try to browse the page :
Testing

Fatal error: Uncaught Error: Class 'Phpml\Classification\NaiveBayes' not found in /home4/ab12960/reviewsclassification.com/test.php:17 Stack trace: #0 {main} thrown in /home4/ab12960/reviewsclassification.com/test.php on line 17

I approciate the help
Regards,

Please check if path in require './vendor/autoload.php'; is correct.

yes , and I have tried require __DIR__.'/vendor/autoload.php';

@ahmadiesa Would you mind to post your composer.json file?

@ahmadiesa have you tried using composer create-project command and them composer require php-ai/php-ml instead of downloading PHP-ML from Github?


Allow buildsystem to rely entirely upon gradle

Fixes #4
Note: I had to downgrade the gradle version. If you attempt to step up to 5.3, everything just falls apart. So, yeah.

Holy cow, thank you! You have no idea how long I spent trying to get this right the first time.
I'll test and merge this in as soon as I finish the streaming.

aight, sure thing.
I've worked with gradle quite a lot, and sometimes in can really just do the stupidest things....

Hey, do you have a discord server by any chance?


[AutoPR consumption/resource-manager/billing/resource-manager] [DO NOT MERGE] Swagger for Tenant Properties API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5038

Doomed PR


Perform Make directory traversal in sort order

This patch is a revision to the implementation of CP-39, to guarantee
the visit order of subdirectories.
While undergoing an in-depth normalization review of each Turtle file
today, I and a person I was working with realized that the visit order
of subdirectories was non-deterministic between Make runs.  On the scale
of nuisances, this is minor, as it causes some confusion on when a
per-file review will be completed, but does not cause any true review
errors because UCO's subdirectories are all independently reviewable.
(The non-determinimism was not apparent in my reviews for the initial
patch because my testing tended to run with full parallelism from
'make -j'.)
Aside from directory and file visit order, no semantic or behavioral
changes are made in this patch.
References:

[OC-98] (CP-39) UCO should implement CI including at least rdf-toolkit
runs

Signed-off-by: Alex Nelson<EMAIL_ADDRESS>

@sbarnum , as this is a non-semantic update to CP-39, I think this doesn't require any committee action before it is merged.
If you don't merge this for 0.7.0, I'll work the change in as part of the proposal moving the Turtle files to /ontology.

@ajnelson-nist I agree with the value of making this change and with the rationale that this does not require committee action before merging.
I do not have the ability to independently test this change but recognize it is a very minor and localized change and trust that you have tested it.
As such, I will approve and merge the request.

@sbarnum Thank you!  And, there's no need to take my word for the test working - the CI runs any updates proposed in the branch that is the subject of the PR, so you can review the test yourself.
https://github.com/ucoProject/UCO/runs/2669381568
The grand hint this one worked is that the vocabulary namespace ran last...
https://github.com/ucoProject/UCO/runs/2669381568#step:4:1212
...and was immediately preceded by victim...
https://github.com/ucoProject/UCO/runs/2669381568#step:4:1192
and so forth.


Competing Consumers Support

Competing Consumer is a new type of subscription released in version 3.2.
Is it possible to add support for it?
http://docs.geteventstore.com/introduction/competing-consumers/

Yes, it will happen at some point. I am just swamped with multiple projects right now and can't get to it.
I will keep the issue open to act as a reminder for me to devote some time.

Hello Guys,
any news about this topic?
thank you so much!!

Hello,
I'm interested at this implementation too.
Is there any roadmap?
Btw...your client is very cool!!! Congrats!!

I am interested in the state as well.
I am guessing that the branch
https://github.com/bmavity/ges-client/tree/dev/tcp is the one worth
watching?
Greg
On Sat, May 14, 2016 at 11:42 AM, Roberto Sciarretta <
<EMAIL_ADDRESS>wrote:

Hello,
I'm interested at this implementation too.
Is there any roadmap?
Btw...your client is very cool!!! Congrats!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/bmavity/ges-client/issues/61#issuecomment-219211064

--
Studying for the Turing test

Hi All,
I have rid myself of my extra projects, so I should have some free time soon. The branch that is about to be merged is the https://github.com/bmavity/ges-client/tree/connection_events branch. It has more robust retries and fixes to some issues with internals.
Also @gregoryyoung , I emailed you in response to your inquiry a couple of weeks ago. Did you receive the email?

Yep I did but have been travelling the last few weeks so have not had a
chance to look through things yet.
On Sat, May 14, 2016 at 6:22 PM, Brian Mavity<EMAIL_ADDRESS>wrote:

Hi All,
I have rid myself of my extra projects, so I should have some free time
soon. The branch that is about to be merged is the
https://github.com/bmavity/ges-client/tree/connection_events branch. It
has more robust retries and fixes to some issues with internals.
Also @gregoryyoung https://github.com/gregoryyoung , I emailed you in
response to your inquiry a couple of weeks ago. Did you receive the email?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/bmavity/ges-client/issues/61#issuecomment-219229183

--
Studying for the Turing test


Documented service name, custom location and version feature in published REST service.

In 7.11 we will be releasing an new feature for Published REST service.
Users will now be able to choose custom paths when creating Published REST service.
Here is the reference documentation.

@grbinho Hi Ivan, there are conflicts now with this PR, as #939 has already been merged. Is the currently published doc correct?
https://docs.mendix.com/refguide/published-rest-service
Or does this PR also have to be merged? We need to work together to finish this pull request. Another option is to close this PR and open a new PR on the doc to make any more necessary changes. Please let me know. Thanks.

@Adam-Dupaski I fixed up this PR. should be good now.


Jessie and VK-172?

Stratux version: v0.9b3-31464f92ad.sh (actually v0.8r2 and above)

Stratux config:

SDR
[ ] single
[ X] dual
GPS
[ X] yes
[ ] no
Type: VK-172
AHRS
[ ] yes
[ X] no
power source: Ship Power via MyGoFlight USB power adaptor
usb cable: Anker Heavy duty (1 ft bench testing, 3 ft aircraft)
3.EFB app and version: (e.g., WingX Pro7 8.6.2) NA
EFB platform: (e.g., iOS 9.2) NA
EFB hardware: (e.g., iPad Mini 2) NA
4.Description of your issue: What is unique about Jessie that does not let it play nicely with the VK-172 at start-up?  After updating to v 0.8r2 and progressing through 0.9b3 I have noticed the VK-172 takes forever to start after an extended period of being powered down (i.e > 3 hours).    After a series of tests, the VK-172 will on average take approximately 43 minute to initialize with v0.8r2 and above. During initialization the Status page will show "0 in solution, 9+ seen; 10+ tracked."  Sometimes I can force a quicker initialization of the VK-172 by power cycling Stratux when running v0.8r2 or greater.
Using the same hardware put swapping the SD card image with v0.8r1-b2eb382c1a.sh the VK-172 takes on average 140 seconds to initialize.
My Stratux test hardware is using a Raspberry Pi 2, the Edimax WiFi dongle, two shielded Nono 2s, and a “bottom” shielded VK-172 GPS dongle.  The VK-172 is mounted above the 978 MHz assigned dongle while the WiFi dongle is mounted above the 1090 MHz assigned dongle.  Both image loads are on SanDisk Ultra 8 GB SD cards.
If possible, enable "Replay Logs", reproduce the problem, and provide a copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and http://<IP_ADDRESS>/logs/stratux.log.

Would you be able to measure the power consumption of v0.8r2+v0.9b3 vs. v0.8r1, please?

I'll need to rig up a USB current test cable.  I knew I should have got a USB volt / amp meter when I placed my last Amazon order.  Let me dig through my junk box an see what I can put together and I'll get the measurements.

I've got my line tester.  I can do a quick test tonight.
On Thu, Jun 30, 2016 at 11:26 AM, Axtel4<EMAIL_ADDRESS>wrote:

I'll need to rig up a USB current test cable. I knew I should have got a
USB volt / amp meter when I placed my last Amazon order. Let me dig through
my junk box an see what I can put together and I'll get the measurements.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/issues/451#issuecomment-229712202, or mute
the thread
https://github.com/notifications/unsubscribe/ACAzTEFbcZYaZoEN6fbK4iG3gW7oUDXuks5qQ-4jgaJpZM4JCRil
.

The USB volt / amp meters tend to cause even more trouble, I just use a multimeter now.

Do just want voltage measurement, or a voltage and current measurement?  I can get the voltage measurement in a few minutes.  For the current I'll need to rig up a cable.

Here are the voltage measurements.  I'll need to rig something up for the current measurement if you want those also.
I included the GPS stats after approximately 2 minutes after power-up.  I didn't see the GPS for 0.9b2 "jump" into WAAS mode like 0.8r1 did.  I went back and did some more testing and 0.9b2 might go into WAAS mode after 20 + minutes, whereas 0.8r1 took a mater of minutes.  The Stratux was setting on the same spot on my work bench in the house for all testing.
Measurements taken after boot and WiFi active. Anker 1ft HD cable. Driving a 5V fan on the 3.3V bus.
MyGoFlight USB converter
Port 1:
V0.9b2
R-Pi = 4.83V
VK-172 = 4.75V
3D GPS, 8 in solution,12 seen,13 tracked
V0.8r1:
R-Pi = 4.85
VK-172 = 4.77
DGPS, 7,10,12
Port 2:
V0.9b2
R-Pi = 4.81V
VK-172 = 4.73V
3D GPS, 8,8,13
V0.8r1:
R-Pi = 4.84 V
VK-172 = 4.76 V
DGPS, 7,10,14
Anker Power Drive 2 USB converter
Port 1:
V0.9b2
R-Pi = 4.84 V
VK-172 = 4.76 V
3D,8,11,13
V0.8r1:
R-Pi = 4.85 V
VK-172 = 4.77 V
DGPS, 7,8,13
Port 2:
V0.9b2
R-Pi = 4.85 V
VK-172 = 4.76 V
3D, 7,10,13
V0.8r1
R-Pi = 4.85 V
VK-172 = 4.76 V
DGPS, 7,9,13

Both images draw about 1.06 amps.  With both builds and using my lab supply I was able to drive my Stratux set-up as low as 3.89 V input to the Pi.  (That was the lower voltage limit of my lab supply.)  The Red LED was extinguished, both SDRs received traffic and the GPS remained locked and would go into WAAS mode, and the Stratux would communicate via WiFi with my lap top.

I am going to close this issue.  I have not been able to correlate the performance of the VK-172 with the Jessie version.  I plan to do some more testing and I will post those results in Issue 203 but it will be sometime after the Fourth of July holiday.
Here are my test observations to date:
The VK-172 lock issue appears to have more to do with the R-Pi build than with the VK-172.  I have two R-Pi 2 B's that I have been testing.  Both R-Pi’s are P.R.C. built units and came from MicroCenter with one being an Element 14 unit and the other was a “no name” R-Pi from the “Bargain Bin” that I don’t think they carry anymore. Unfortunately I don’t remember which R-Pi was which and the CCAs for both are marked the same.
Swapping hardware and Stratux OS between the R-Pi’s I noticed one is consistent with the VK-172 locking on in a matter of minutes, whereas the other P.R.C R-Pi will take 10’s of minutes to acquire a GPS lock.
Utilizing a stiff lab supply* and performing low voltage dropout tests, one R-Pi will operate consistently down to 3.84 V input, measured at Transorb D5.  When operating this R-Pi at 3.84V, the Red LED is not illuminated, but the Stratux retains WiFi connection, the GPS acquires and retains lock, and both SDRs receive traffic.  The other R-Pi has trouble maintaining a WiFi connection and GPS lock at 4.5 V. I have a UK built R-Pi in my RY-835 / NooElec Mini build.  My next plan is to extract the UK R-Pi from the Mini build and perform the same testing with it to see if it performs any different.
*Stiff lab supply is defined as a supply that sags less than 100 mV when the current is stepped from no load to 1 amp at the rated voltage.

@Axtel4, I  ran into the (trivial) same problem with ID'ing RPi's and Beaglebone Blacks.  My solution is to use a felt-tip permanent marker and write on the top of the ethernet connector.  My particular nomenclature is "U1-xx" for both RPi and BBB (because their physical differences are so obvious.  In one page of the engineering notebook I jot down that unit's name and whatever relevant data I wish to keep about it.  Mostly the numbering sequence simply corresponds to their order of purchase.
Trying to do the same with SD cards but they are more problematic as their contents can change so easily.  :)
With so many different types of SOC's at my house, I had to do  something.  :)


[Enchancement] Access scope for files in public storage

Is your feature request related to a problem? Please describe.
Files public storage provides an url for the file accessible for everyone
Describe the solution you'd like
Would be nice to have access scope added so only certain users can get access to the public url files, this is needed for example for audio files which are accessible only to certain users via network
Describe alternatives you've considered
Firebase has security rules to control access to public url files

If I remember correctly, this can be done through the private file storage. It doesn't have support for scopes, but you can retrieve a key to access the file. Adding scopes would be a nice addition.


build issue: error NU1100: Unable to resolve 'Raylib-cs (>= <IP_ADDRESS>)' for 'net6.0'

I followed the instructions exactly and i'm no stranger to software development, although i've never used C#.
Whatever i try, i get the error above which states that raylib is missing, so dependencies aren't being installed. Any idea how to resolve?
If i run "dotnet build" or "dotnet run", it gives me the error. Same if i use build commands through VSCode. I'm on W10. I'm surprised other people haven't run into the same issue.

Alright, it seems that there was nu NuGet source configured, so i had to run:
dotnet nuget add source "https://api.nuget.org/v3/index.json" --name "NuGet.org"
which fixed it!

I suspect installing Visual Studio configures this. I wanted to set it up in VSCode, so i just installed the SDK instead, which means NuGet wasn't configured.


No language added when entering common name if add lexicon is open

When you submit the add new common name form with a language selected but the add lexicon open (but empty), no language info is saved for the new common name. Since it is very simple to make a mis-click while selecting the language drop down, i think this behaviour should be changed to adding the selected language if the new lexicon field is open but empty.
BTW. It should be impossible to add a common name without a language! There are very many instances of this throughout the site.

Basically a duplicate of my own filed issue #1896


Add keycap designs

I found these originally on Reddit:
https://www.reddit.com/r/ErgoMechKeyboards/comments/13ypjor/cherry_ultra_low_profile_keycaps/

Awesome find, thanks @senekor!


code review

Hi, I have created a WordPress plugin for payment API using React JS. My branch name is PayPalUpdated-zayed please review the code and give me feedback on things to change.

Can you brief more about it @zayedutsho


Arrow navigation fix

See issue #9.

Hey @steviebaa, any chance this can be merged in? I am running into this issue.

@gkiely I didn't get around to completing this one sorry. It needs some work before it is ready. I won't be able to get around to it anytime soon.

No worries, thanks for the quick follow up. Remember what needed to be done to get it ready?


MethodDeprecatedError: Calling GitHubPrivateRepositoryReleaseDownloadStrategy is disabled!

My Formula stopped working, now install/upgrade presents:
 MethodDeprecatedError: Calling GitHubPrivateRepositoryReleaseDownloadStrategy is disabled! Use a vendored GitHubPrivateRepositoryReleaseDownloadStrategy in your own formula or tap (using require_relative) instead.
Any recent changes made by homebrew?
Formula below:
class FooRun < Formula
include Language::Python::Virtualenv
desc "CLI for foo containers"
homepage "https://acme.org/"
url "https://github.com/foo/eng-sysdev-foo-cli/releases/download/v1.4/foo_run.tgz", :using =>      GitHubPrivateRepositoryReleaseDownloadStrategy
version "1.4"
sha256 "2ebe14d1830c5c52710c7168d4858a01c4f51e4bf8cd20a096795cfb04eff905"
head "https://github.com/foo/eng-sysdev-foo-cli.git"
bottle :unneeded
depends_on "python@3"
def install
venv = virtualenv_create(libexec)
system libexec/"bin/pip3", "install", "-v", "--ignore-installed", "-r", "requirements.txt"
env = {
:PATH => "#{libexec}/bin:$PATH",
}
(bin/"foo_run").write_env_script(libexec/"foo_run", env)
libexec.install "foo_run"
libexec.install "libs"
end
test do
system "echo", "TEST"
end
def caveats; <<~EOS
Jumo run has been installed.
Please run 'foo_run' which should now be in your path.
EOS
end
end
I require the private strategy and I pass a HOMEBREW_GITHUB_API_TOKEN
Please advise

This is no longer supported. You need to copy this into your formula yourself (as the message says).

I tried, but japanese only.
https://qiita.com/eielh/items/812206e77696f947396f

Why this has been removed?

@victorcoder Because it wasn't used by any of Homebrew's supported formulae and we're a a volunteer run project who has to adjust our scope accordingly.


$fetch on localhost fails while <IP_ADDRESS> is fine

Environment

Operating System: Windows_NT
Node Version:     v18.12.1
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
With a REST API running on localhost:

Call it by using $fetch on <IP_ADDRESS>. It returns successfully, with the API showing a request in its logs:

<script>
export default defineComponent({
  async setup() {
    await $fetch("http://<IP_ADDRESS>:8000/api/");
  },
});
</script>

Call it by using $fetch on localhost. It returns 500 fetch failed (), with no request in the API's logs:

<script>
export default defineComponent({
  async setup() {
    await $fetch("http://localhost:8000/api/");
  },
});
</script>

Describe the bug
$fetch does not work with localhost, but is fine when substituted with <IP_ADDRESS>:
<script>
export default defineComponent({
  async setup() {
    // ✔ Good (200, request received):
    await $fetch("http://<IP_ADDRESS>:8000/api/");
    // ❌ Bad (500, local API does not receive a request):
    await $fetch("http://localhost:8000/api/");
  },
});
</script>

This bug appears to be prevalent in the Nuxt community:

"The problem was that fetch() didn't reconize localhost:9000 for some reason, but when I changed BASE_URL to <IP_ADDRESS>:9000 it started to work." - https://stackoverflow.com/questions/73098374/nuxt-3-fetch-method-doesnt-work-with-golang-servers
"Had a similar issue. I got it to work by changing localhost -> <IP_ADDRESS>." - https://www.reddit.com/r/Nuxt/comments/zqq0cg/nuxtjs_3_fetch_refuses_to_connect_to_local_server/

This behaviour was discovered at: https://github.com/johannschopplich/nuxt-api-party/issues/8
Examples
✔ <IP_ADDRESS> (200, API receives request)
async setup() {
  await $fetch("http://<IP_ADDRESS>:8000/api/users/lloydtao/");
}

❌ localhost (500, API does not receive a request)
async setup() {
  await $fetch("http://localhost:8000/api/users/lloydtao/");
}

✔ <IP_ADDRESS> (404, API receives request)
async setup() {
  await $fetch("http://<IP_ADDRESS>:8000/api/not-an-endpoint/");
}

❌ localhost (500, API does not receive a request)
async setup() {
  await $fetch("http://localhost:8000/api/not-an-endpoint/");
}

Logs
500
fetch failed ()

at async setup (...\usefetch-localhost\nuxt-app\app.js:9:5)

Node 18 LTS supports Happy Eyeballs which should resolve this issue. However, Happy Eyeballs needs to be activated by setting autoSelectFamily to true. From Node v18.13 this can be globally set by calling net.setDefaultAutoSelectFamily(true).


Syntax error in PHP53Patch.php

I'm running the new 1.24.0 release.
It looks like there's a trailing comma in PHP53Patch.php, which is causing a syntax error when I try to install PHP 7.3.
Output
[snip]
===> Checking patches...
PHP Parse error:  syntax error, unexpected ')' in phar:///usr/local/bin/phpbrew/src/PhpBrew/Patches/PHP53Patch.php on line 33

Expected Result
PHP 7.3 installs correctly
Command
phpbrew install 7.3 +default +iconv="$(brew --prefix libiconv)" +intl +pgsql +bz2="$(brew --prefix bzip2)" +gd -- --with-gd=shared --with-jpeg-dir=/usr --with-png-dir=/usr --with-zlib-dir="$(brew --prefix zlib)"

Platform
OS: macOS 10.15.1
Running PHP: PHP 5.6.36
Installing PHP: 7.3

Indeed. Having a trailing comma in function calls is valid as of PHP 7.3 and I didn't test this patch on an older PHP version.

Thanks for the quick fix!


Add: support for none git folders

In this case scm-id will be the folder name

I didn't think non-git folders were being supported? This will result in a large number of repositories being generated for an account

I didn't think non-git folders were being supported? This will result in a large number of repositories being generated for an account

Yes it should be supported and the upper level folder will be like "repository"


#184581357 A user should be able to mark one/all Notification as read

What does this PR do?
This PR adds the ability to mark notifications as read, or unread
Description of Task to be completed?
A user should be able to mark one/all Notification as read
How should this be manually tested?

run npm install && npm run migrate:all && npm start
Visit /api-docs and you should see notifications section in the swagger documentation -- use that for testing.

Any background context you want to provide?
n/a
What are the relevant pivotal tracker/Trello stories?
https://www.pivotaltracker.com/n/projects/2632830/stories/184581357
Screenshots (if appropriate)

Questions:
n/a

Resolve the unresolved conversation and update README otherwise LGTM

Actually, I'm unable to resolve that one conversation; since I can't seem to find it anywhere!


Mask window implementation

Another feature implementation if you want it. This should be close to correct modulo the supported graphics modes.
Caveats:

No special effort to support hires mode.
Uncertain whether colors should clip to black or to color 0. Not hard to change.
Some whitespace auto-formatting got through in rendering.rs. Whoops.

Done, also added & fixed some other comments. I do try not to leave a trail of mysteries behind me :)


Process fails on certain PDF files?

Hi,
Trying to run siteproc against a specific PDF file, causes a crash to my machine.
Can you please verify this file works for you? I notice it is a power point file converted to PDF
23847801770001241.pdf
The problematic process is the pdftoxml, when running against this file, the memory consumption rises until 100%, causing the machine to crash.

What do you mean by running citeproc-java against a PDF file? The tool does not support PDF files as input. Can you please send me the command line that you use or the API calls you're performing?

So sorry, wrong github source, please close and delete :)
thanks for the prompt response.


add voluptuous to ecobee, speedtest.net, fast.com, actiontec, forecast.io

Description:
For the speedtest.net and fast.com I'm not sure if I did it right.  I get this exception when values are out of range:
voluptuous.error.MultipleInvalid: value must be at most 23 @ data['hour'][0]
Also I ran a sed command to remove carriage returns from actiontec.py and now it sees the whole file as different.  I only added the PLATFORM_SCHEMA
Related issue (if applicable): fixes #2800

Nothing you can do with actiontec. You replaced the new lines on each line, so each line actually did change.

Sorry, was offline will working on this. I didn't get the comments by @balloob.


Timeout while getting a connection from pool when using isoneof in asp.net API

When calling a query with marten that uses IsOneOf in a web api call the connection never gets closed, and when the 21's call (with a pool size of 20)  happens an error gets thrown: Timeout while getting a connection from pool.

When i look in at the server status in pg admin i do see all the open but idle connections that are supposed to be reused.

It seems to only happen when using an API, possibly because every call is async?
Example here

@jefjef Are you disposing the sessions after your done with them? When we went to the "sticky connection" thing, you now have to dispose the IQuerySession or IDocumentSession.

we use sessions in a using statement, that should be enough right?

@jefjef From glancing at your code, yes. I'll take a look at this this afternoon and see if I can reproduce what you're seeing. I don't really have any explanation yet.

@jefjef Hey, if you're using Postgresql 9.5, can you try again w/ UpsertStyle = UpsertStyle.Standard to opt into the more efficient upsert mechanism? Another user is telling me today that he saw connection timeouts in stress testing that go away when you opt into the 9.5 style.

@jeremydmiller His issue is due to queries rather than upserts.

Yes, its only when i use a query that uses IsOneOf, session.Load and session.Store work fine

Yeah, but I didn't know if it was because connections were getting soaked up in the SaveChanges() with requests running in parallel

I can reproduce by running @jefjef code locally.

Does it really go away if you replace IsOneOf() with maybe a (prop == 1 || prop == 2) kind of query? Just trying to see if it's really isolated to that one Linq handler

Nope, it has nothing to do with IsOneOf.

trying to get a test for it

The following test fail (but passes when there's only one task running):
        [Fact]
        public void concurrent()
        {
            theSession.Store(new Target { Color = Colors.Blue, Number = 1 });
            theSession.Store(new Target { Color = Colors.Red, Number = 2 });
            theSession.Store(new Target { Color = Colors.Green, Number = 3 });
            theSession.Store(new Target { Color = Colors.Blue, Number = 4 });

            theSession.SaveChanges();
            var ts = new List<Task<List<Target>>>();
            Enumerable.Range(0, 2)
                .ToList()
                .ForEach(_ => ts.Add(theSession.Query<Target>().Where(x => x.Number < 3).ToListAsync()));
            Task.WhenAll(ts).GetAwaiter().GetResult();
        }

Are you hitting the exact same session every time?

Nice one!
I got a proper failing test:
        [Fact]
        public void concurrent()
        {
            theSession.Store(new Target { Color = Colors.Blue, Number = 1 });
            theSession.Store(new Target { Color = Colors.Red, Number = 2 });
            theSession.Store(new Target { Color = Colors.Green, Number = 3 });
            theSession.Store(new Target { Color = Colors.Blue, Number = 4 });

            theSession.SaveChanges();
            var ts = new List<Task<List<Target>>>();
            var sessions = new List<IDocumentSession>();
            try
            {
                Enumerable.Range(0, 20)
                    .ToList()
                    .ForEach(_ =>
                    {
                        var session = theStore.LightweightSession();
                        sessions.Add(session);
                        ts.Add(session.Query<Target>().Where(x => x.Number == 3).ToListAsync());
                    });
                Task.WhenAll(ts).GetAwaiter().GetResult();
            }
            finally
            {
                sessions.ForEach(s=>s.Dispose());
            }
        }

And it passes with 19 tasks!

20 is the default # of connections for Npgsql. Wanna jack that up to 50 and see if it breaks?
You aren't disposing the session, is that purposeful?

Disposing them in the finally clause. will give 50 a go.

Yup, 50 works like a charm and fails, as expected, when I up it to 50 sessions :smile:

Any difference w/ the Dispose()?

We could be more aggressive in shutting down the connection. Say if we were to immediately shut down the open connection right after SaveChanges() is called, then lazily open another connection if you need it later.

Ok, so this is passing (as you suggested)
        [Fact]
        public void concurrent()
        {
            theSession.Store(new Target { Color = Colors.Blue, Number = 1 });
            theSession.Store(new Target { Color = Colors.Red, Number = 2 });
            theSession.Store(new Target { Color = Colors.Green, Number = 3 });
            theSession.Store(new Target { Color = Colors.Blue, Number = 4 });

            theSession.SaveChanges();
            theSession.Dispose();

            var ts = new List<Task>();Enumerable.Range(0, 100)
                .ToList()
                .ForEach(_ =>
                {
                    var session = theStore.LightweightSession();
                    ts.Add(session.Query<Target>().Where(x => x.Number == 3).ToListAsync().ContinueWith(t=>session.Dispose()));
                });
            Task.WhenAll(ts).GetAwaiter().GetResult();
        }

Is this something we will have to roll inside marten or rather set as a best practice? Inside marten I suppose isn't really an option.

You mean disposing of IManagedConnection inside the session straight after executing any CRUD command/query, right?

Yes, or at least having IManagedConnection reset itself so it's not hanging on to the connection any longer.

I think it's a best practice, period. The docs to call that out.

I don't really understand this, we use a using statement with our sessions, so shouldn't the connection be released and added back to the pool when the using statement ends?

@jefjef And that makes two of us. The connection is disposed when the IQuerySession or IDocumentSession is disposed, but we're not explicitly closing it. I didn't think that mattered, but I've been wrong before.

Ok, I think I missed the whole thing. This is the real failing test:
        [Fact]
        public void concurrent()
        {
            theSession.Store(new Target { Color = Colors.Blue, Number = 1 });
            theSession.Store(new Target { Color = Colors.Red, Number = 2 });
            theSession.Store(new Target { Color = Colors.Green, Number = 3 });
            theSession.Store(new Target { Color = Colors.Blue, Number = 4 });

            theSession.SaveChanges();
            theSession.Dispose();

            var ts = new List<Task<IQueryable<Target>>>();
            Enumerable.Range(0, 142)
                .ToList()
                .ForEach(_ =>
                {
                    ts.Add(Task.Run(() =>
                    {
                        using (var session = theStore.LightweightSession())
                            return session.Query<Target>().Where(x => x.Number == 3);
                    }));
                });
            var targets = Task.WhenAll(ts).GetAwaiter().GetResult();
            targets.SelectMany(t=>t.ToArray()).Count().ShouldBe(142);
        }

since @jefjef was not calling ToList() after the Where() the sessions were all called roughly on the same time rather than query being executed and then session disposed.
This fixes it:
session.Query<Target>().Where(x => x.Number == 3).ToList();

Nice, that fixed it :smile:

I'm going to go ahead and close this one. Thanks @nieve for running through this one.

Yes, thanks guys :smiley:


Breaking changes in ratatui 0.27.0 + ansi-to-tui open to 0.27.0 in version constraints causing issues where cargo.lock ignored

I am getting an issue wherein with the new version of ratatui (unable to upgrade today due to other dependencies) when I cargo install and the cargo.lock is ignored, ansi-to-tui is pulling in 0.27.0, as well as the 0.26.3 I'm already using.
To abet this I propose #46 for the next patch, then following up with the bump to 0.27.0
This will allow lagging projects to pin to a version of ansi-to-tui (potentially only in the short term) and avoid sudden breakage

Yeah this is the same issue as the pinned issue.
For this I'll merge your pr and release a minor version of the 4.1. And from 0.27 onwards I'll make a major release of this crate corresponding to minor versions of ratatui.

Merged


NotImplementedError  when running python connect.py

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ X] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

As per the demo instructions (db setup in serverless mode)

Any log messages given by the failure

NotImplementedError

Expected/desired behavior

I'd expect such a simple bit of demo code to work without cryptic error messages.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)
Win10

Versions

Mention any other details that might be useful
It appears to be connection related. The code run until the cleanup_graph(client), then fails with the only error message "NotImplemented".  I've checked and re-checked the connection string details. They're definitely correct. Cosmos db setup db/collection name matches the names in the sample.
client = client.Client('wss://mygraphaccount.gremlin.cosmos.azure.com:443/', 'g',
username="/dbs/sample-database/colls/sample-graph/",
password="PRIVATEKEY",
message_serializer=serializer.GraphSONSerializersV2d0()
)

Thanks! We'll be in touch soon.

I'm also running into this error.

Fixed it. The requirements.txt contains a ref to the old versions of the library. Uninstall the first one. Run:
pip uninstall gremlinpython
pip install gremlinpython
The first will uninstall 'gremlinpython==3.4.8' which causes the issue. Will be looking at AWS Neptune much more closely now.


Tdunn/20171030 compass doc

@ldanzinger - Please review and merge. Thank you.

Thanks, @ldanzinger. Back to you.

Looks good


adding re-used dataset will break the backend/save feature when using certain projects

bug reconstructed, when importing datasets with descriptions that are too long.
error occurs in oracle database only (not postgres).
even though we do clip texts to 4000 characters, the entry in the DB is limited to 4000 bytes. since some characters require more than one byte to represent we have the issue that we cannot utilize the full 4000 limit for characters.
the best solution here is to utilize CLOB objects for long texts instead of VARCHAR inorder to not incur into this issue.


Update whitespace.txt

Thank you for your contribution! Unfortunately, we're following the original Skia repository, and this change cannot be merged.


LayoutLMV2 model fine tuning options

Hi,
I have been working on Invoice data extraction using the LayoutLMV2 model. I have been getting a good results with limited amount of data.
I am facing issue with model finetuning. Could u please help me, how can I finetune different hyperparameters of LayoutLMv2 model. It would be great if you could share any link of article for reference.
Thanks in advance.

@khadkechetan You may try this https://huggingface.co/nielsr/layoutlmv2-finetuned-funsd

Thanks for the prompt reply.
I can see only two hyperparameters learning rate and optimizer. Does it have more hyper parameters?

Hi @khadkechetan,
this is the notebook that I used to get that checkpoint: https://github.com/NielsRogge/Transformers-Tutorials/blob/master/LayoutLMv2/FUNSD/Fine_tuning_LayoutLMv2ForTokenClassification_on_FUNSD_using_HuggingFace_Trainer.ipynb

For more hyper params, please refer to https://huggingface.co/transformers/v4.5.1/main_classes/trainer.html#transformers.TrainingArguments.


BACKLOG-13981: Add-in the multi-select feature to grid view.

JIRA
https://jira.jahia.org/browse/BACKLOG-13981
Description
Relatively simple PR to use the existing selection redux state for gird view as well.
There is special request on initial if not 'checked' to not display the checkbox only when hovered; but once checked the will remain displayed unless 'unchecked'.
The shadowbox is slightly different from the existing used for preview as well (noted to @fpral )
Testing
Please see below video for the unit testing
https://user-images.githubusercontent.com/13266177/103023829-aa178100-451c-11eb-8733-ef08617a2f52.mp4

not technically a unit test but better than nothing.  at some point might need to setup a separate story to at least start putting in test framework for jcontent.


Remove binary module analysis caching in ModuleCmdletBase.cs

ModuleCmdletBase.cs goes to some effort currently to run and cache analysis on binary modules. However, in .NET Core, we don't actually do any analysis. And we no longer support FullCLR.
So we should consider removing binary module analysis for now, and reinstate it when we have a way to analyse a module without loading it. /cc @daxian-dbw.

We have the guideline that module manifest should not use the wildcard "*" for entries like FunctionsToExport, CmdletsToExport and etc. Currently, with PowerShell Core, we solely rely on the module manifest when it comes to module analysis for a binary module, and it seems working well enough. So maybe we can remove the remaining DLL inspection analysis code (not working) completely for now, and if we have a strong request at a later time, we can implement it in PowerShell Core.

@BrucePay The work for #6653 only requires reading the available public types from a given assembly, while binary module analysis would require more (attribute, base type, and possibly need to follow the base type chain to another assembly, which would require assembly resolution). So the work for #6653 would be a subset of the things we need for binary module analysis.
Given your comment, I think we should keep the existing binary module analysis code untouched. The code can be refactored when we get to the implementation.


Upgrade react-tether to 0.3.1

Needed a little bit of refactoring for a new API, but I think this actually sets us up better for when we make the datepicker components publicly consumable

Current coverage is 92.26%

Merging #345 into master will increase coverage by +0.73% as of 5ce40da

@@            master    #345   diff @@
======================================
  Files            9       8     -1
  Stmts          189     181     -8
  Branches        42      40     -2
  Methods         94      91     -3
======================================
- Hit            173     167     -6
+ Partial          4       2     -2
  Missed          12      12

Review entire Coverage Diff as of 5ce40da

Powered by Codecov. Updated on successful CI builds.


fix kafka version until Strimzi is released

Type of change

Bugfix

Description
After releasing Kafka 3.8.0 into Kroxylicious, system tests are broken because this version is still not supported by Strimzi. So we need this temporary fix to make them work again
Additional Context
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

@strimzi-ci run system tests

@franvila sorry, I forgot to trigger the system tests before merging #1356

:arrow_forward: Build started - check Jenkins for more info. :arrow_forward:

:heavy_check_mark: Test Summary :heavy_check_mark:
BUILD NUMBER: 95
KAFKA VERSION: 3.7.1
TOTAL: 21
PASS: 18
FAIL: 0
SKIP: 3


BaseRecordController

Created the BaseRecordController component to allow setting up a custom RecordController to record video in other formats that accept h264 and aac or in mp4 using something different from AndroidMuxer (Xuggler or JCodec for example)
By default the AndroidMuxerRecordController class is used.

I like it.
I will modify it a bit to fix a bug found and merge.

I fixed all and refactored the code.
Thank you for sharing :)

Thank you pedro for your library :)!
As soon as possibile i create a repo with 2-3 recordcontroller i wrote (i must refactor some code) so then you'll decide if create a new repo as RTSP-Server

I will add it to README for sure but I'm not sure if create my own repo or only fork it.
I think the fork is the best solution because the code is yours.

Hello pedro!
I have created a repo with two recordcontrollers ;).
https://github.com/FunnyDevs/rtmp-rtsp-stream-client-java-recordcontrollers

Hello pedro! I have created a repo with two recordcontrollers ;).
https://github.com/FunnyDevs/rtmp-rtsp-stream-client-java-recordcontrollers

I like it but only one hint.
Instead of set video configuration in constructor you can get it from MediaFormat like this:
    @Override
    public void setVideoFormat(MediaFormat videoFormat, boolean isOnlyVideo) {
        int width = videoFormat.getInteger(MediaFormat.KEY_WIDTH);
        int height = videoFormat.getInteger(MediaFormat.KEY_HEIGHT);
        int fps = videoFormat.getInteger(MediaFormat.KEY_FRAME_RATE);
        int videoBitrate = videoFormat.getInteger(MediaFormat.KEY_BIT_RATE);
    }

    @Override
    public void setAudioFormat(MediaFormat audioFormat, boolean isOnlyVideo) {
        int sampleRate = audioFormat.getInteger(MediaFormat.KEY_SAMPLE_RATE);
        int channels = audioFormat.getInteger(MediaFormat.KEY_CHANNEL_COUNT);
        int audioBitrate = audioFormat.getInteger(MediaFormat.KEY_BIT_RATE);
        //I should get a way to notify you this if necessary. It isn't a proper way to do it.
        int sampleSize = new MicrophoneManager(null).getMaxInputSize();
    }

This is the real usage of this methods and by this way you can make it really easy to use because you don't need know anything about your configuration. It is configured with stream object parameters.

I don't recommend it.
Stream without audio or without video are supported so I recommend you change record status when the last of both is called.
One way is use 2 bolleans to know if that method was called and start record when both boolean are true
Also, If you can support record only video or only audio it is better to fully support all record modes.
You can know it using "boolean isOnlyVideo" (in setAudioFormat it should be named isOnlyAudio, it is a typo)

Ok, i should have modified the files but, before pushing them, i would like to finish testing onlyVideo and onlyAudio conditions.
To test onlyVideo i have called only prepareVideo(), but how could i test onlyAudio? If I call prepareAudio() without prepareVideo() i obtain a crash
 java.lang.NullPointerException: Attempt to invoke virtual method 'void android.media.MediaCodec.setCallback(android.media.MediaCodec$Callback, android.os.Handler)' on a null object reference
        at com.pedro.encoder.BaseEncoder.initCodec(BaseEncoder.java:70)
        at com.pedro.encoder.BaseEncoder.start(BaseEncoder.java:61)

You can use RtmpOnlyAudio class


Incorporate Fog's data for Android TV Live Channels integration

Would it be possible to add Fog's data as a data source and generate a m3u8 files passed on it?
I'm trying to integrate SmoothStreams TV into my Android TV box (Nvidia Shield) Live Channels app.  I have found an app, TVirl (https://play.google.com/store/apps/details?id=by.stari4ek.tvirl&hl=en), which takes an m3u8 file and an XMLTV file for EPG data and puts it into the Live Channels app.  Fog has released the URLs for his data (see below).  This would allow me to run the script locally and generate the necessary m3u8 and XMLTV files for the TVirl app and have that integrated into the Live Channels app.
Is this at all possible?
Thanks in advance.

Forgot Fog's URLs
http://sstv.fog.pt/utc/xmltv.xml
http://sstv.fog.pt/utc/SmoothStreams.m3u8

Hi Tim!
Very sorry for the delay in responding to your issue here, my GitHub notifier extension has been really unreliable.
I appreciate the feature suggestion, but as decided in the closing of issue #12, I'd like to keep this script as a self-contained module that can be built on by others.  I don't currently have the time (I'm a full time engineering student) or resources to adequately maintain enhancements like this.
Feel free to fork the project and add the features yourself.  I'll add your fork to the README if you do build on it.
I hope you can understand why I wouldn't want to half-ass an enhancement like this.  Thanks again for your effort and idea.
Cheers Tim 🍻


HardwareInterface error for single module errors

This adds a check to the read method of the CAN hardware interface that makes sure the entire hardware interface stops once a single module has an unrecoverable error. This is not tested besides making sure it compiles as of yet.

The change works great for the ReBeL, but not as good for the din rail modules. Either the startup sequence needs to be adapted somewhat or the modules take longer to go out of errorstate causing an unnecessary failure.

The fix for the stepper modules required some small changes to the reset and startup behaviour. Now both types of modules are working fine with regards to startup and stopping the hardware interface on errors.