Missing port binding for mongodb emulator

Hello! I've been attempting to run the Linux emulator on MacOS using the MongoDB API. After following the steps and visiting https://localhost:8081/_explorer/index.html, I'm able to retrieve the following Mongo connection string so that I can connect using a MongoDB client (mongoid):
mongodb://localhost:C2y6yDjf5%2FR%2Bob0N8A7Cgv30VRDJIWEHLM%2B4QDU5DE2nQ9nDuVTqobD4b8mGGyPMbIZnqyMsEcaGQy67XIw%2FJw%3D%3D@localhost:10255/admin?ssl=true
However, I receive the this connection error: connect ECONNREFUSED <IP_ADDRESS>:10255.
I was able to successfully connect after updating the docker run command to include port 10255: --publish 10251-10255:10251-10255. Should the documentation be updated or am I connecting incorrectly?

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

ID: 0a59d2ab-cacd-5545-bddb-b9304f204605
Version Independent ID: bbcf9128-f9c3-fa84-b085-ba566f45d56f
Content: Run the Azure Cosmos DB Emulator on Docker for Linux
Content Source: articles/cosmos-db/linux-emulator.md
Service: cosmos-db
GitHub Login: @StefArroyo
Microsoft Alias: esarroyo

@joeltaylor Thanks for reaching out, We will investigate and update you shortly.

Hi!
Yes, documentation should be updated accordingly!
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: GeethaThatipatri-MSFT @.>
Sent: Thursday, August 18, 2022 11:42:55 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Estefani Arroyo @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Missing port binding for mongodb emulator (Issue #97291)
@joeltaylorhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjoeltaylor&data=05|01|Estefani.Arroyo%40microsoft.com|f049c5db06f443b6a70808da81497719|72f988bf86f141af91ab2d7cd011db47|1|0|637964449776494814|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=zEIDSACzEf5fgxKu4o3gIRYwT%2F3iuf9Y5iZ84m8s0Cw%3D&reserved=0 Thanks for reaching out, We will investigate and update you shortly.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F97291%23issuecomment-1219816307&data=05|01|Estefani.Arroyo%40microsoft.com|f049c5db06f443b6a70808da81497719|72f988bf86f141af91ab2d7cd011db47|1|0|637964449776494814|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=rp7OBUtnISq4H3fkE24AyMArlj8ixa7WZKWJ4z3Q1Fg%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAG65UDBZFZ4AJICE4NP4CMTVZZ727ANCNFSM5656OLJQ&data=05|01|Estefani.Arroyo%40microsoft.com|f049c5db06f443b6a70808da81497719|72f988bf86f141af91ab2d7cd011db47|1|0|637964449776494814|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MRUwaHnC%2FjTM1%2F%2BhZ%2Fr6L5sq1xGTmOO6rZraR2XUg20%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@joeltaylor Document is now updated and you will be seeing the changes by tomorrow.


"az pipelines build definition list" not working from DevOps

When running az pipelines build definition list from the Cloud Shell, it seems to return the same information as az pipelines list. I assume it should be like this, but if there's a difference between the two commands, it should be elaborated in the documentation.
However, when running the two command from a DevOps pipeline powershell script on ubuntu-latest, az pipelines build definition list return only a few results, while az pipelines list seems to result all the results.
example.yaml.txt
[Enter feedback here]

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

ID: b7077ac1-bb22-976f-77f3-9a40cfcbce81
Version Independent ID: b3841ca2-43a1-1438-7907-0240d06b7502
Content: az pipelines build
Content Source: latest/docs-ref-autogen/ext/azure-devops/pipelines/build.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

pipelines


#1 BACK END PHASE 1: Do preliminary research on the API.

Do preliminary research on the API.

Found documentation on URL and identified necessary parameters


setTimeSeriesMaxRecords description doesn't contain info about received dates.

From documentation
https://github.com/devcartel/pyrfa/blob/master/API.md#timeseries
it is unclear which dates are used to get time series by getTimeSeries method. In fact, last dates are received. But I want to be sure in it. It should be added to docs.

By calling setTimeSeriesMaxRecords(N) it will make getTimeSeries() retrieve only the last N records from TS1. Will put that in the docs. Thanks.


Failed to verify Kube ApiServer Access

Problem
Output of kubectl logs
Starting kube-monkey...
--
open /var/run/secrets/kubernetes.io/serviceaccount/token: no such file or directory
panic: open /var/run/secrets/kubernetes.io/serviceaccount/token: no such file or directory

goroutine 1 [running]:
main.main()
/home/user/go/src/github.com/asobti/kube-monkey/main.go:24 +0x108

Environment
Trying to run kube-monkey in a kube cluster that is behind a firewall hence no auth configured.
This error could be because I'm running v1.5.2 of kubernetes.
Is there any reason not to upgrade the kubernetes/client-go version?
[user@box] $ curl http://localhost:8080/api/
{
  "kind": "APIVersions",
  "versions": [
    "v1"
  ],
  "serverAddressByClientCIDRs": [
    {
      "clientCIDR": "<IP_ADDRESS>/0",
      "serverAddress": "xxx.xxx.xxx.xxx:x443"
    }
  ]
}
[user@box] $ kubectl version
Client Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.2", GitCommit:"c55cf2b7d8bfeb947f77453415d775d7f71c89c2", GitTreeState:"clean", BuildDate:"2017-02-06T23:54:31Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.2", GitCommit:"c55cf2b7d8bfeb947f77453415d775d7f71c89c2", GitTreeState:"clean", BuildDate:"2017-02-06T23:54:31Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}

Debugging
Tracing the error goes to
https://github.com/asobti/kube-monkey/blob/master/kubernetes/kubernetes.go#L11
which calls Google's Kubernetes APi verification at
https://github.com/asobti/kube-monkey/blob/master/vendor/k8s.io/client-go/1.5/rest/config.go#L262
In the kubernetes docs, "/var/run/secrets/kubernetes.io/serviceaccount/token" should always be automatically generated. It's possible that this problem occurs if the service account admission controller is not enabled. Additionally "spec.serviceAccountName field has been automatically set" is not true for any deployed pods.
Our current configuration does not generate anything in the service account. i.e.
[user@box ~]$ kubectl describe sa
Name:           default
Namespace:      kube-system
Labels:         <none>

Image pull secrets:     <none>

Mountable secrets:      <none>

Tokens:                 <none>

Reason for Issue
I'm primarily opening this issue

if I find a fix, other people can resolve this quickly.
ask why not upgrade the kubernetes/client-go version

Hmm, interesting. I haven't seen an environment where serviceaccount/token is not generated. It might be possible to make unauth access a config.
Regarding client-go upgrade - no reason not to. I just haven't had a chance to test a newer version yet.

So this isn't an issue with kube-monkey per se, but rather with kubernete's client-go. AFAIK they're only supporting authenticated clusters. I've opened up an issue  with them, but this may be avoidable depending on however client-go actually connects and talks to the kube apiserver.
For anyone curious who might also be running an unauthenticated server.

You must have the serviceaccount admission controller enabled. There is a recommended order found here
Use openssl genrsa (because kubernetes expects a specific format for keypairs)

sudo openssl genrsa -out /var/run/kubernetes/serviceaccount.key 1024
sudo rsa -in  /var/run/kubernetes/serviceaccount.key -pubout > /var/run/kubernetes/serviceaccount.pub

Insert the required flags which are referenced here

sudo sed -i 's/KUBE_CONTROLLER_MANAGER_ARGS=""/KUBE_CONTROLLER_MANAGER_ARGS="--service-account-private-key-file=\/var\/run\/kubernetes\/serviceaccount.key"/gi' /etc/kubernetes/controller-manager
sudo sed -i 's/KUBE_ADMISSION_CONTROL="--admission_control=NamespaceLifecycle,NamespaceExists,LimitRanger,SecurityContextDeny,ResourceQuota"/KUBE_ADMISSION_CONTROL="--admission_control=NamespaceLifecycle,NamespaceExists,LimitRanger,SecurityContextDeny,ServiceAccount,ResourceQuota"\nKUBE_API_ARGS="--service-account-key-file=\/var\/run\/kubernetes\/serviceaccount.pub"/gi' /etc/kubernetes/apiserver

Restart your apiserver service and controller manager at minimum

# Stop on all master servers
sudo -i
ls -Al /usr/lib/systemd/system/*kube* | sed -r 's/^.+\///' | xargs -L 1 -I f service "f" stop
# Restart on all three servers
ls -Al /usr/lib/systemd/system/*kube* | sed -r 's/^.+\///' | xargs -L 1 -I f service "f" start

Unfortunately after enabling service account tokens, it also expects the certificates as well...
[user@box ~]$ kubectl logs kube-monkey
Starting kube-monkey...
E1023 17:09:28.169399       1 config.go:269] Expected to load root CA config from /var/run/secrets/kubernetes.io/serviceaccount/ca.crt, but got err: open /var/run/secrets/kubernetes.io/serviceaccount/ca.crt: no such file or directory
<nil>
panic: Unable to verify client connectivity to Kubernetes server

goroutine 1 [running]:
main.main()
        /home/user/go/src/github.com/asobti/kube-monkey/main.go:24 +0x108

While I was setting up km, I ran into errors that were annoying since assumptions of the configuration wasn't explicitly stated. Maybe a more experienced kubernetes admin wouldn't have problems, but I think it'd be helpful.

The config map must be created before even attempting to deploy km.
The config map, if deployed from a file MUST be named config.toml, otherwise the file expected at /etc/kube-monkey/config.toml won't be named config.toml

For example: if you had saved the config as km-config.toml
 kubectl create configmap km-config --from-file=config.toml=km-config.toml

The config map must be deployed in the same namespace.
The sample config in the README hasn't included any of the new features such as the override apiserver from #6

Aha! I think I've figure out how to make this work.
If you're running an unauthenticated server and using cluster dns, then by disabling the InClusterConfig and pointing to the insecure API server url to the dns ip/port.
To support this very special case... it's require a bit of editing of the InClusterConfig part or handle the error. One approach was to "check" if the server was not using TLS https://github.com/kubernetes/kubernetes/pull/15230/files but this was quite contentious for the kubernetes people.

So the custom configuration basically used the same example code from https://github.com/kubernetes/client-go/blob/master/tools/clientcmd/doc.go
The custom configuration was supported in client-go 1.5 as well so I imported the corresponding lib.
https://github.com/asobti/kube-monkey/pull/19


chore: bdd-spring-1572610447 to 0.0.1

chore: Promote bdd-spring-1572610447 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Community contribution to #119: Ascii to binary converter

Based on the binary visualization from Challenge 119, I build an ASCII to binary visualization: https://doxanthropos.github.io/sketchbook/2018102301/

Nice job!!


upgraded neutron api to v1beta2

Removed everything related to v1beta1

API client (python/swagger_*)
Docs (python/docs)
Examples (examples/0.3)

Removed docker related stuff
(Do we need use docker to dev? I believe virtual env should be good enough)
Publish
@gangtao I'm not sure how to publish it. I assume I'll need to do the following:

Merge this PR
Tag the develop to the version we'd like to use e.g. 1.2.0
make build && make upload && make upload_test

https://github.com/timeplus-io/neutron/issues/1238

(Jove Github Bot) moved this ticket out of the GitHub project(up to 1200 tickets for one project).


Made kedge config parsing more resilient on user errors.

Also some cleanup in doc.

Codecov Report

Merging #16 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   43.77%   43.77%
=======================================
  Files          12       12
  Lines         514      514
=======================================
  Hits          225      225
  Misses        265      265
  Partials       24       24

Continue to review full report at Codecov.

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


Use the full buffer width for the VT viewport

That's a weird one I found while working on #17445...
The _viewport in conhost is basically the section of the buffer that
gets render on the screen. The _viewport.Left() is effectively the
horizontal scrolling position. I don't think we should use that for
the VT viewport. This PR changes it so that the VT viewport always
spans the full width of the buffer. This makes it possible to address
the full width of the buffer with CUP sequences.
Part of #14000

This can't be right. The VT code already uses the full buffer width for its page width. See here:
https://github.com/microsoft/terminal/blob/a80539c2f74e5b4299e4a303f7ddc89d9978a0c3/src/terminal/adapter/PageManager.cpp#L69-L74
And it uses the viewport when it genuinely needs to know the visible range, or the scrolling offset. For example see here:
https://github.com/microsoft/terminal/blob/a80539c2f74e5b4299e4a303f7ddc89d9978a0c3/src/terminal/adapter/PageManager.cpp#L86-L89
Changing the GetVirtualViewport implementation is just going to break that functionality. And I strongly suspect that's not the only thing that will break.

Ah I see. The XPanOffset is used for DECRPDE. But more importantly, reading through the other places that GetVirtualViewport is used it's immediately obvious how this PR breaks its usage. I'll reword the comment on the function though:

For VT operations, this is essentially the mutable section of the buffer.

That's not quite correct after all, isn't it? The mutable section spans the entire width (unless margins are set).
That aside, is this code actually correct?
https://github.com/microsoft/terminal/blob/a80539c2f74e5b4299e4a303f7ddc89d9978a0c3/src/terminal/adapter/InteractDispatch.cpp#L135-L150
SetConsoleCursorPositionImpl expects an absolute coordinate but the viewport that GetBufferAndViewport returns has a .left that equals the horizontal scroll position. That's wrong, no?

For VT operations, this is essentially the mutable section of the buffer.

That's not quite correct after all, isn't it?

Yeah, that statement would have been correct when the VT support was first implemented, but that behavior was fixed years ago, and the comment obviously wasn't updated.

That aside, is this code actually correct?

No. That looks wrong. But I think it possibly doesn't matter in practice (or at least didn't matter), because as far as I know it's only used by conpty as a way to "inherit" the cursor position, and in a conpty session the viewport width should always match the buffer width. But that may not be true forever.
And in an ideal world that cursor position should really be going through the AdaptDispatch class rather than being a partial reimplementation of that code, but I'm not sure if that's feasible. I don't know this part of the code base very well.

// Returns the section of the text buffer that would be visible on the screen
// if the user didn't scroll away vertically. It's essentially the same as
// GetVirtualBufferViewport() but includes the horizontal scroll offset.
Viewport SCREEN_INFORMATION::GetVirtualViewport() const noexcept {
    // the existing code
}

// Returns the section of the text buffer that's addressable by VT sequences.
Viewport SCREEN_INFORMATION::GetVirtualBufferViewport() const noexcept {
    // the code from this PR
}

Does this make sense? 🤔 I'm not sure what to call it but "viewport". I also considered calling it GetVirtualAddressableRange().

Does this make sense? 🤔 I'm not sure what to call it but "viewport". I also considered calling it GetVirtualAddressableRange().

I'd personally avoid "viewport" in the name, since that suggests to me the viewable/visible area, which this is not (it's unfortunate that it's returning a Viewport object). In VT terms it's really the "page", so something like GetVtPageArea might be clearer, but I don't think GetVirtualAddressableRange is terrible either.
That said, I'm somewhat concerned that we even need this in the SCREEN_INFORMATION class, because that shouldn't really have knowledge of VT state, and in the long term the actual page boundaries may not be something we can calculate here.

I'm using this information in COOKED_READ_DATA to map the TextBuffer's cursor position back to VT coordinates so that I can emit appropriate CUP sequences that work not just under ConPTY (back buffer size == coordinate space), but also in conhost.

[...] and in the long term the actual page boundaries may not be something we can calculate here.

Ah, that makes sense... I guess the alternative is to use SetConsoleCursorPositionImpl from inside COOKED_READ_DATA in place of using CUP sequences. This gets rid of its dependency to map between the coordinate spaces, and also allows us to write VT text outside of the valid VT-coordinate range (thanks to the call to UpdateBottom()). The only reason I'm not doing that already, is because it somehow felt even whackier than the coordinate space mapping I'm doing now. I could try doing that.

I don't think this is a big deal, because if you're using console APIs (and I would include cooked read in this category), then you really shouldn't have any expectation for how things will work when mixed with VT operations. But the edge cases I would be worried about would be things like the VT margins, and line renditions, which can affect the addressable range of the page.
The one idea I had considered for dealing with this was that we could maybe reset the VT state to some base condition whenever someone tried to use a legacy console API. This would include clearing the margins, and maybe setting/resetting certain modes. I'm not sure it's worth the effort though, and certainly not something you should worry about now.

The one idea I had considered for dealing with this was that we could maybe reset the VT state to some base condition whenever someone tried to use a legacy console API. This would include clearing the margins, and maybe setting/resetting certain modes. I'm not sure it's worth the effort though, and certainly not something you should worry about now.

(FWIW I think this is a great idea!)


Defer system updates until substantial battery power is available?

Here are some user reports indicating that a device running out of power during a system update can result in file-system corruption, sometimes seemingly severe:

https://discuss.grapheneos.org/d/9324
https://discuss.grapheneos.org/d/8958
https://discuss.grapheneos.org/d/8474
https://discuss.grapheneos.org/d/8214
https://discuss.grapheneos.org/d/7081
https://discuss.grapheneos.org/d/4057
https://discuss.grapheneos.org/d/4039
https://discuss.grapheneos.org/d/1594

This situation is presumably rare, but it seems to be troublesome for affected users.  Might it be helpful to either require a 50% charge before applying an update, or add a user-settable threshold?

Each of these threads is about power running out during the initial boot of a new version. Most of these issues appear to be a kernel bug and the situation is resolved by automatic rollback. It also appears f2fs does not always work as intended if power is cut during the late boot process. That's not particularly surprising since we know that f2fs is quite buggy. There's not really anything that can be done about this. We can enable the non-low battery requirement for the update client by default, but the issue is not caused by power loss during installation of the update but rather during first boot of the new version. If someone has an update fully installed, then runs out of power later, they'll potentially trigger these firmware or f2fs bugs when they try to boot it without hooking it up to fast charging. What can be possibly do about that? We know about these firmware bugs and they're reported upstream. We know f2fs is buggy but there's little hope of a short term solution to that.

We had the same "Your device is corrupt. It can’t be trusted and may not work properly" issue after a power loss. It seemed that a "merging" process had been interrupted. We managed to continue it with fastboot snapshot-update merge while in fastbootd (not fastboot, mind the "d") mode.
See our blog post or cross-post on the GrapheneOS forum: https://discuss.grapheneos.org/d/12814

It also appears f2fs does not always work as intended if power is cut during the late boot process. That's not particularly surprising since we know that f2fs is quite buggy. There's not really anything that can be done about this.

I have been thinking about this off and on, and I think wonder if maybe there is something that could be done.
Would it be possible for the end of the kernel's boot sequence to refuse to hand off to user space, or for the beginning of user space to refuse to proceed, if the battery is low?  For example, could something near the beginning of user space check the battery level and, if it's too low, just display a low-battery icon and stall in some low-power state?  Or display a low-battery icon for some seconds and power the device off?
My trigger for commenting on this closed issue is a recent report on the discussion forum:  https://discuss.grapheneos.org/d/14823-stuck-in-reboot-loop-flashing-the-lock-screen-after-app-optimizations
It may be rare, but I think some users are being nailed by a power outage somewhere during the update process.  I can easily imagine it might be f2fs's fault, but it would be nice if there were a way to protect some users.


Adding support for RPC client with password.

I was trying to use your repo together with an bison trails node which contains API login and I haven't found the way to do so. This would be a very interesting addition the same way other projects like go-ethereum already have this option

interesting. could you provide some resource to me?


Kite errors out when quoted csv header contains space

I tried kite sdk to create avro schema out of csv file using below command:
kite-dataset csv-schema test.CSV --class Sample
It fails (when column header has spaces):
"field 1","field 2","field 3"
"Agwam","Agwam, MA","25007"
with following error:

Unknown error: Bad header for field, should start with a character or _ and can contain only alphanumerics and _ 0: "field 1"

Success example:
"field1","field2","field3"
"Agwam","Agwam, MA","25007"
Not sure if there is a way to handle it.

So, it looks like this is because Avro requires field names to obey the above specs. See https://github.com/kite-sdk/kite/blob/release-1.1.0/kite-data/kite-data-core/src/main/java/org/kitesdk/data/spi/filesystem/CSVUtil.java#L179 for the exact code where this is happening.
If the field name doesn't match the specs, it's impossible to generate an Avro schema using that name. So this is a case of that more general issue. We can call this something like 'Kite CSV schema inference errors when headers don't obey Avro specs'.
One solution would be to slightly change the inferSchema and inferNullableSchema methods that currently take the Set<String> requiredFields parameter. We could make this Map<String, String> requiredFieldMappings to let the user specify Avro-safe names for all the required fields. Then inferSchemaInternal uses the mappings to generate the schema.


Update LICENSE.txt

I don't understand this. Are you implying we didn't copy the proper text for Apache2?


Resolved screen requiring render library to load.

Resolves #366; Client libraries are now loaded after the player has been authed.

Clientside libraries are being loaded twice. Is L741 supposed to be there?

No it is not, resolved now.


bug/Tables are not being parsed from markdown files

Describe the bug
Tables are being returned as UncategorizedText .
To Reproduce
Example Markdown file:

# A table

Item | Price | # In stock
---|---|---
Juicy Apples | 1.99 | 739
Bananas | 1.89 | 6

Expected behavior
It should return a  Table element.
Additional context
The reason for this is because for python-markdown to support tables, it needs to have an extension enabled: https://python-markdown.github.io/extensions/tables/
Changing this:
html = markdown.markdown(text)
to this:
html = markdown.markdown(text, extensions=['tables'])
in partition_md, fixes the issue.
I can submit a pull request if that's an acceptable solution.

Hi @phowat , I've verified the solution you propose works. If you're willing to put together a pull request, please do!

CC @amanda103


Add functionality to take server MOTD from a file motd.md.

Contains
Add feature for server MOTD input from file as mentioned in #2855
How to test

Create a motd.md file in the root directory(temporary placement of the file for development)
Spin up a server using ./gradlew server
Stop the server and edit the config.cfg file in the server directory as below
"upstreamBandwidth": 1024,
"serverPort": 25777,
"serverMOTD": "[Text would appear as MOTD in absence of the motd.md file]",
"masterServer": "meta.terasology.org"
Now start the server again and start a client.
Join the server (localhost if on the same machine).

Can one of the admins please verify this patch?

@GooeyHub: add to whitelist
Hopefully we can start merging more of these PRs :-)

Hooray Jenkins reported success with all tests good!

On second thought / review of this - is this adding the check in the right place? The message of the day option is already fully defined, the idea is just to load it either from config.cfg or a new file. Shouldn't it be possible to load it from an alternative location but then use the same property to store it so we don't need to regenerate the NetData class?

@Cervator I reverted the changes made in the Network layer and tested the feature.
One bug (not sure if it can be considered a bug) which I detected was that the MOTD was displayed from the motd.md file even if the config.cfg has no serverMOTD property defined. Is this a desired behaviour or should this be handled as no MOTD defined?

Hooray Jenkins reported success with all tests good!

I would be inclined to treat that as correct behaviour. I would say that the fallback should be something like
motd.md file. Then try the serverMOTD property. Then display a default MOTD

Summarizing some chat just now on Slack:

May close this PR as it is fairly minor and include the functionality in a redo that's module based (should submit an issue with updated details)
Can simply use a new MOTD module that has a UI showing the MOTD with a default message. If the module is enabled, pop the message via showScreen in the postBegin of a MOTD System class. The default message could simply contain instructions for how to configure the MOTD, including setting it empty somewhere to disable the MOTD
Later on can draw in an actual message to display via config. This is where it gets interesting

Could we still read it from a module specific part of config.cfg just to have a first working version? Key bindings can work that way but are somewhat special
A FlexibleConfig could probably provide it (module specific FC I'm sure that's a thing. Where would it store to?)
A file with the message stored in a new module sandbox file storage location could be read if present (think sandbox/modules/MOTD/motd.txt with new engine functionality to allow reading stuff from there). This is an alternative to leaving a file in the data root (shove more stuff into modulespace)
A console command used in-game could update the MOTD if the user has the appropriate permissions

Later on more advanced alternatives could be implemented like one supporting graphical elements in the MOTD. These could be either a new variant in the MOTD module or a whole new module (server admins would just enable one)
Other functionality like score boards or tutorials could also display on initial join, but aren't they just similar from using the same postBegin() to show the message? They're more like gameplay while a MOTD is a system type thing

I might not have caught everything so anybody feel free to chime in with extras or questions :-)

Alright, MOTD in a module now exists! https://github.com/Terasology/ServerMOTD - thanks @Naman-sopho :-)
I created a few follow-up issues in that repo for further improvements, like the file based approach listed here, and am linking back here for reference. With that done I'll close this PR and let work continue in the module


MultiInput with multilinemode enabled is collapsing on clicking of scroll anywhere inside the input box

OpenUI5 version:1.44.8
Browser/version (+device/version):Chrome
Any other tested browsers/devices(OK/FAIL):Passed in IE
URL (minimal example if possible): https://jsfiddle.net/ru4ht4sL/
User/password (if required and possible - do not post any confidential information here): NA
Steps to reproduce the problem:

Click on Button
Click on More in MultiInput.
Click on Scroll or anywhere else in input.

What is the expected result?
The control should allow me to scroll by clicking on scroll without collapsing. Also when expanded if I am clicking anywhere inside the control it should not collapse
What happens instead?
The control is collapsing whenever I am clicking on scroll or whenever I am clicking inside the input.
Any other information? (attach screenshot if possible)
The collapsing of control on click of scroll is only happening when we have the control inside popup. But the collapsing of control when clicking on control is happening everywhere.
If you check in IE, the control is working as expected.

Hi,
Thank you for your findings. I have created an internal incident:<PHONE_NUMBER>.
Best Regards,
Irina

Hello @asw-sanjay,
the issue has been fixed. The fix will be available with the next version 1.48 and with the next released patches for versions 1.38, 1.44 and 1.46.
Regards,
Marin


SU7 / GOY: Ignition and starter switches inop

With v<IP_ADDRESS> releasethe ignition and starter switches (UI only) are stuck at Start and Auto and can't be clicked any more.
Workaround
Use key bindings, ect.

have same problem

Fixed in v0.6.2


Docs Design

[x] Linkable headers (https://github.com/influxdata/docs.influxdata.com/pull/435)
[ ] Navbar covers anchor links
[ ] Code blocks appear blank until user interacts with them
[ ] Version colors
[ ] Chronograf version is missing:

The search bar is also not working.

@gunnaraasen, looks like search is working again - is it working for you?

Yup, it's working again. There was an issue with the indexing script not parsing the content correctly in the new layout.

We may want to investigate some of these approaches to fix the anchor links. http://stackoverflow.com/questions/10732690/offsetting-an-html-anchor-to-adjust-for-fixed-header

@willpiers @wfro @mark-rushakoff @121watts any ideas?
@toddboom @pauldix possible hackday project post 1.0 release.

Just pushed some updates that color code each product

Fixed Navbar covers anchor links with help from @toddboom

Just pushed a commit that fixes #357 I tested it on a few different things and it looks okay. I ended up z-indexing the code blocks higher and that fixed the issue

I wondered if it was a z-index thing. Thanks, @alexpaxton!

🎉 thanks, @alexpaxton!!


DOMElement::$attributes is not nullable

I got an inconsistency here between PHPStan and Psalm, and at first I though Psalm was wrong: https://github.com/vimeo/psalm/issues/9242
But, after reading the doc description, they are right and this PR fixes it on this side.

@Jean85 this property inherited from DOMNode, probably it should be updated there instead.

No that would be incorrect, because in the parent it IS nullable. It's not nullable only in the child class.

because in the parent it IS nullable.
Sorry, but are you sure? see the link I've provided https://github.com/php/php-src/blob/master/ext/dom/php_dom.stub.php#L324 as well as https://www.php.net/manual/en/class.domnode.php#domnode.props.attributes or null otherwise. Do I miss something?

Ah, sorry, my bad. Yeah, that's interesting, despite the property is inherited it can't contain nulls..ok, let's merge it.


Show the source for Class Properties

For Class Properties only the docblock is shown, and unlike the methods no source.
Showing the source will also display any default configuration.

This now works http://li3.me/docs/api/lithium/1.1.x/lithium/core/Object::$_autoConfig


Install problems macOS Ventura

Hello 5T33Z, I have big problems installing macOS Ventura on my T430.
The previous EFI's worked idly what could it be I hope you can help me
because you are a professional in the field and have great respect for your work and effort to create EFI's and make us all very happy with it so that we can also enjoy macOS
many thanks

Why do you expect support from me for a machine I don't even own and this EFI isn't even for?
Read and follow the deployment notes: - especially this part:  https://github.com/5T33Z0/Lenovo-T530-Hackintosh-OpenCore#re-enabling-acpi-power-management-in-macos-ventura
Unless you have a custom BIOS with CFG Lock disabled you cannot use the EFI as is as. Add  the ControlMsrE2.efi to EFI/OC/Tools and run it from bootpicker, and check if it the MSR 0xE2 register locked or unlocked. If it is "locked" you have to disable the 2 kexts for AppleIntelCPUPowerManagement and force-enable XCPM instead to have decent CPU Power Management.
For pacthing your BIOS you can follow these instructions; https://github.com/5T33Z0/Lenovo-T530-Hackintosh-OpenCore/issues/31#issuecomment-1368409836

I no longer get error messages, but I see the Apple logo, the screen is gray and absolutely nothing is happening do you have an idea.

I no longer get error messages, but I see the Apple logo, the screen is gray and absolutely nothing is happening do you have an idea.

Did you run ControlMsrE2.efi as I asked you to? What was the result?

This firmware has LOCKED MSR 0xE2 Register

Enable verbose mode (-v)

I have one more question I've now managed to get into the installation program but when I try to install ventura I get the message that ventura is not compatible with the mac I've already tried no compat check but it doesn't work and on the other disk I have can install it with no problems and runs idly and when i copy the efi from the working hard drive over to the new one it texts me when trying macOS ventura is not compatible so i find that very odd

Read the inszall instructions. Especially the part about SMBIOS
https://github.com/5T33Z0/Lenovo-T530-Hackintosh-OpenCore/blob/main/macOS_Install/OpenCore/OC_macOS13.md

This firmware has LOCKED MSR 0xE2 Register

You may have better luck using the 5T33Z0's previous release without the AppleIntel CPU power management. I'm up and running on that with minimal changes.

An error occurred while preparing the software update. What is the procedure to get rid of this error?

NONE. And it's explained in the install instructions for macOS 13 right before the "Notes" section… and it's marked by a ⚠
What more do you expect me to do?
So, since you have obviously managed to install macOS Ventura. I am closing this issue.


Cannot Compile Clipper on MacOS

Hi Everyone,
My MacOS is Catalina 10.15 and I am trying to install gdspy on it. However, neither pip install gdspy nor python setup.py install works and they all got stuck with the following error.

building 'gdspy.clipper' extension
gcc -Wno-unused-result -Wsign-compare -Wunreachable-code -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -I/Users/Roseldon/anaconda3/include -arch x86_64 -I/Users/Roseldon/anaconda3/include -arch x86_64 :/usr/local/include -I/Users/Roseldon/anaconda3/include/python3.7m -c gdspy/clipper.cpp -o build/temp.macosx-10.7-x86_64-3.7/gdspy/clipper.o -std=c++11 -mmacosx-version-min=10.9
gcc: error: :/usr/local/include: No such file or directory
error: command 'gcc' failed with exit status 1

I have been really straggling to figure out a solution for this. The folder /usr/local/include does exist and I don't know why it doesn't see it.
Best

@Lambdauv, I dont have access to OS X, but the colon (:) before the path looks odd to me... do you have both XCode and Command Line Tools properly updated?


cache/stash long_descriptions rendered by readme_render

RE https://github.com/pypa/readme_renderer/issues/95
A handful of long descriptions have long render times, such as https://gitlab.com/hansroh/skitai/blob/master/README.rst.
A proposed solution:

Store long_description as usual at upload time.
Call a task to attempt to render the long description.
Task renders long_description and stores resulting html with the version number of readme_renderer
When rendering a project page, check for rendered version and fallback to attempting render
On readme_renderer upgrade, have some mechanism to rerender all readmes via the same celery task as above.

cc @jonparrott

Research notes (for my own benefit/for the benefit of whoever takes this on):

Current un-rendered description field is here.
The description is only rendered for HTML views and is done in the template here using the readme filter.
The filter readme is defined here.

For the benefit of accomplishing this task and testability, we should likely pull the rendering logic out of the context filter and into its own thing (not sure where they would live yet) and just have the filter be a thin wrapper. We could then use the common function from both the task and the templates.

Might not even be necessary to keep it as a filter, it could be passed into the template from the view instead, which would make it a little easier to determine a cache key.

That could definitely work.


Refactor code

refactor code for better reusability

Coverage increased (+0.008%) to 92.147% when pulling d6ef088b4c2f8bb705c53353f30a1e2f4da31220 on jelneo:add_login into b96ed652f538f9097a7c4a6ee145d293df96ba09 on CS2103AUG2017-T17-B1:master.

Great refactoring done jel. Code indeed looks neater.


Update the support information for structuredClone (PR 14392 follow-up)

When the structuredClone polyfill was added, the support information in Safari was unclear. Given that an actual version number is now available, see below, it seems like a good idea to update the comment accordingly.
https://developer.mozilla.org/en-US/docs/Web/API/structuredClone#browser_compatibility

Thanks!


Alternative custom empty state approach

This is a branch off of #3317 to try out a different approach to adding custom empty states.
The code needs to be cleaned up but the main things to have a look at in this PR:
d2l-filter-dimension-set-empty-state:
- Removed the change events to clean up code. We could add this back to listen to changes to the component properties in the future if a consumer needed it
d2l-filter-dimension-set:
- Removed the d2l-filter-empty-state-change and d2l-filter-empty-state-slot-change events. When the empty state slots change, it regrabs the EmptyState for that slot and dispatches the existing d2l-filter-dimension-change event.
d2l-filter:
- Since we removed the empty state change events we can remove the handlers here and the d2l-filter-dimension-data-change handler handles the empty state changes without any updates to it.
It worked pretty seamlessly. I really like this approach and it cleans it up a lot. I think it would be a good idea to pivot to this.

Yeah, this cleans up a lot of the complicated bits. I like it too!

Removed the change events to clean up code. We could add this back to listen to changes to the component properties in the future if a consumer needed it

And if we did need to add this back, we could just run _handleSearchEmptyStateSlotChange whenever a component property changes.  Something like:
<slot name="search-empty-state" @d2l-filter-dimension-set-empty-state-change="${this._handleSearchEmptyStateSlotChange}" @slotchange="${this._handleSearchEmptyStateSlotChange}"></slot>

(Could just rename it to _handleSearchEmptyStateChange at that point.)
Arguably slightly less efficient to grab all three properties again, but much simpler logic. Not nearly the same perf hit as having to grab and recalculate all d2l-filter-dimension-set-values each time one changes.


from_pretrained has correct return type (i.e. HookedSAETransformer.from_pretrained returns HookedSAETransformer)

Description
This changes the type signature of from_pretrained, so its return type is the generic type of the base class. For example, now when we use HookedSAETransformer.from_pretrained, the expected return type is HookedSAETransformer not HookedTransformer.
I changed this because in my SAE intro material I was having to explicitly type-annotate as sae: HookedSAETransformer = HookedSAETransformer.from_pretrained(...), and so just changing the type signature seemed like the best option.

Makes sense!


Work with other React renderers like react-three-fiber

Hi :wave:
I love this project and would like to know if it works with other react renderers like react-pixi and react-three-fiber?
At the moment I'm getting the following error:
SyntaxError: The requested module '/node_modules/react-reconciler/constants.js' does not provide an export named 'ConcurrentRoot'
<EMAIL_ADDRESS>
Stackblitz link: https://stackblitz.com/edit/github-yisbzq?file=src/Button/Button.story.tsx

Hey @RodrigoHamuy!
Thanks for kind the words and we don't support those yet. This product is in alpha and will be changing a fair amount before the next major release. We can look at supporting them when we have a more stable foundation.


Allow moving to no iteration

In our sprint workflow we'd like to move all "In progress" issues to the next iteration, but want to move all "Todo" issues to no iteration to show that we overplanned and need to actively put it in our sprint again.
Would it be possible to add none to the allowed fields in new-iteration as a way to unset the iteration field for a selection of issues?

I might have time to dive into this, but don't think so.
One question upfront: from experience with the Github API, do you know if this is even possible? Do they allow POST/PUT with null?


Switch to brute force for hnsw when topk is super large

Fix #218, Set threshold to 0.5 for safety.

/kind improvement

/lgtm


tutorials for the new way of doing data ingestion of the new, combined SAP IoT product

we have tested this tutorial successfully with 80 partners (html exported version), 6 new employees and 18 participants of a hackathon in africa (html exported version) and have incorporated the feedback.

@marcuscbehrens Please address my review comments and sign CLA.

I signed the CLA yesterday and adjusted the text as requested. Thanks for the feedback!

I committed the 2 changes you requested. Thanks for your advice. Can this be pulled now? We have some partners waiting for this to be published.


min_width needs better docs

Hi!,
Docs says:

min_width - minimal width. If window width is less, grid will be shown in one-column mode (default: 768)

However, thats not true. Without overriding CSS media query, lesser values won't be taken into account (yeah, grid will be locked, but it still goes one column mode earlier than expeced)
# responsible line
@media (max-width: 768px) {}

I'm not really sure, but something like might help:
document.querySelector('style').textContent +=
    "@media (max-width: 768px) { .grid-stack-item { /*undo styles?*/ }}"

Saw this issue as well. Was able to fix it to generate the media query based on the value of min_width with:
/* Comment this portion out of the CSS */
/*
@media (max-width: 768px) {
  .grid-stack-item {
    position: relative !important;
    width: auto !important;
    left: 0 !important;
    top: auto !important;
    margin-bottom: 20px;
  }
  .grid-stack-item .ui-resizable-handle {
    display: none;
  }

  .grid-stack {
    height: auto !important;
  }
}
*/

GridStack.prototype._update_styles = function (max_height) {
...
    // Add this line below to the bottom of GridStack.prototype._update_styles to update the media query to the value of min_width
    Utils.insert_css_rule(this._styles, '@media (max-width: ' + this.opts.min_width + 'px)', '.grid-stack-item { position: relative !important; width: auto !important; left: 0 !important; top: auto !important; margin-bottom: $vertical_padding; } .grid-stack-item .ui-resizable-handle { display: none; } .grid-stack { height: auto !important; }', 0);

};

This is a bug. I was just about to open the same PR. The media query that is putting it into one column mode:
@media (max-width: 768px) {
  .grid-stack-item {
    position: relative !important;
    width: auto !important;
    left: 0 !important;
    top: auto !important;
    margin-bottom: 20px;
  }
  .grid-stack-item .ui-resizable-handle {
    display: none;
  }

  .grid-stack {
    height: auto !important;
  }
}

should be driven by the passed in min_width param or the docs need to change. Also, this doesn't really make much sense to me:
GridStack.prototype._is_one_column_mode = function () {
        return (window.innerWidth || document.documentElement.clientWidth || document.body.clientWidth) <= this.opts.min_width;
};

especially given that the docs say:

minimal width. If window width is less, grid will be shown in one-column mode

I would expect a minimum width param to mean "If the container is this size or greater, normal mode. If less than this, one column mode". If my window is 768px, and I'm using the default min_width:768 param, I should still be able to move widgets around and I wouldn't yet be in single column mode. I am trying to support moving on iPhone tablet portrait, which is 768px. I shouldn't have to override a max-width media query and set my min-width to 767 to achieve this.

I am going through the same issue,
@josephndenton way looks better if we want to set the min width? can't this be include that in the application?

As mentioned in https://github.com/troolee/gridstack.js/issues/223 the readme has been updated to better explain this.


fix: SIGINT worker processes when process_manager starts

Without this change, I found that workers could get stuck on a long-running task. Worker processes are sent a SIGINT, and then that signal is handled appropriately to exit the prefetcher & runner.
Is this appropriate as general purpose behaviour, and if not, would it make sense as a command-line option?

I note that tasks cancelled in this way have their result returned as error = KeyboardInterrupt.
In my own application, I have a middleware that re-kicks these tasks when the worker next starts.


User is not being created by 10-postal

An user is not being created (nothing in the users table in the database) even though I can see the following in the console:
postal-app          | [WARN] ** [container] RabbitMQ Host 'postal-rabbitmq' is not accessible, retrying.. (20 seconds so far)
postal-app          | [NOTICE] ** [container] Configuring Postal
postal-app          | [WARN] ** [container] Detecting new Install. Initializing Database
postal-clamav       | Sun Oct 25 13:12:06 2020 -> Database test passed.
postal-clamav       | Sun Oct 25 13:12:06 2020 -> daily.cvd updated (version: 25967, sigs: 4335099, f-level: 63, builder: raynman)
postal-clamav       | Sun Oct 25 13:12:06 2020 -> main database available for download (remote version: 59)
postal-clamav       | Sun Oct 25 13:12:13 2020 -> Testing database: '/data/definitions//tmp.2e52b2ad9a/clamav-03931e9d0f42251e955663d0921e269e.tmp-main.cvd' ...
postal-app          | [NOTICE] ** [container] Creating Administrative User
postal-app          | [WARN] ** [container] No Admin Email entered, setting default email login to<EMAIL_ADDRESS>postal-app          | [WARN] ** [container] WARNING: No Admin Pass entered, setting default password to 'postal'
postal-app          | [WARN] ** [container] No Admin First Name entered, setting default to 'Postal'
postal-app          | [WARN] ** [container] No Admin Last Name entered, setting default to 'Admin'
postal-clamav       | Sun Oct 25 13:12:27 2020 -> Database test passed.
postal-clamav       | Sun Oct 25 13:12:27 2020 -> main.cvd updated (version: 59, sigs: 4564902, f-level: 60, builder: sigmgr)
postal-clamav       | Sun Oct 25 13:12:27 2020 -> bytecode database available for download (remote version: 331)
postal-clamav       | Sun Oct 25 13:12:27 2020 -> Testing database: '/data/definitions//tmp.2e52b2ad9a/clamav-6fe8d96b89f9b01c332090ecd21ada9f.tmp-bytecode.cvd' ...
postal-clamav       | Sun Oct 25 13:12:27 2020 -> Database test passed.
postal-clamav       | Sun Oct 25 13:12:27 2020 -> bytecode.cvd updated (version: 331, sigs: 94, f-level: 63, builder: anvilleg)
postal-clamav       | [INFO] ** [clamav] Container Initialization Complete
postal-clamav       | [cont-init.d] 10-clamd: exited 0.
postal-clamav       | [cont-init.d] 99-container: executing...
postal-clamav       | [cont-init.d] 99-container: exited 0.
postal-clamav       | [cont-init.d] done.
postal-clamav       | [services.d] starting services
postal-clamav       | [services.d] done.
postal-clamav       | [INFO] ** [zabbix] Starting Zabbix Agent
postal-clamav       | [INFO] ** [clamav] Starting ClamAV
postal-clamav       | [NOTICE] ** [freshclam] Definitions Updating Routines Initialized on Sun Oct 25 13:12:38 GMT 2020
postal-clamav       | [INFO] ** [freshclam] Downloading Latest Definitions
postal-app          | [cont-init.d] 10-postal: exited 0.
postal-app          | [cont-init.d] 99-container: executing...
postal-app          | [cont-init.d] 99-container: exited 0.
postal-app          | [cont-init.d] done.
postal-app          | [services.d] starting services
postal-app          | [services.d] done.
postal-app          | [INFO] ** [zabbix] Starting Zabbix Agent
postal-app          | [INFO] ** [container] Starting postal
postal-app          | Started Procodile supervisor with PID 1101
postal-app          | Procodile Version   1.0.26
postal-app          | Application Root    /app
postal-app          | Supervisor PID      1101
postal-app          | Started             2020-10-25 13:13:10 +0000

However, using docker exec -it bash and afterwards running /app/bin/postal make-user actually works fine.
If it makes a difference: it does create an user on my local Fedora 33 machine, but does not on a machine running Fedora CoreOS Stable which is remotely accessed with docker context.

Interesting. I will look into in a few days and see where the breakage is.

The expect script is fine, just checked by disabling delete of file
  # rm -rf /tmp/create-pass.sh

and after container is up
docker exec -ti postal_app_1 /usr/bin/with-contenv /tmp/create-pass.sh

spawn /usr/bin/with-contenv bash /app/bin/postal make-user
No secret key was specified in the Postal config file. Using one for just this session
Postal User Creator
Enter the information required to create a new Postal user.
This tool is usually only used to create your initial admin user.

E-Mail Address<EMAIL_ADDRESS>First Name          : Admin
Last Name           : Admin
Initial Password:   : ********************

User has been created with e-mail address<EMAIL_ADDRESS>
I tried adding some sleep time before script, to let postal start all its services before creating user, but no luck.
Maybe decoupling db initialisation and creating user as asynchronous task could help?
Of course it would be much better if guys from postalhq could implement non-interactive user creation https://github.com/postalhq/postal/issues/145

Looks like they won't, postalserver/postal#145 was just closed with no follow-up. :(


If onlySquare is enabled, execute fitImage on display.

#776
To fix above issue.
If onlySquare is enabled when displaying an Asset, call fitImage to crop it to a square.
You can see how it works in the sample apps in the feature/demo branch.
https://github.com/hanawat/YPImagePicker/tree/feature/demo

Before
After

With this fix, the selected image view can't be moved up when scrolling. Can you take a look?

@keremcesme Thank you for your review. Videos in the library also used fitImage.

@ramboli1986 Did you run that in the sample app? I checked and it works fine in the sample app.

@ramboli1986 Did you run that in the sample app? I checked and it works fine in the sample app.

No I was on latest master banch.

@ramboli1986 Did you run that in the sample app? I checked and it works fine in the sample app.

Nvm, I can't repo it any more.


error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

When I run make develop, using the instruction on the backend setup I am getting the below compilation error:
`reating build/temp.linux-x86_64-2.7/c
x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -DUSE__THREAD -I/usr/include/ffi -I/usr/include/libffi -I/usr/include/python2.7 -c c/_cffi_backend.c -o build/temp.linux-x86_64-2.7/c/_cffi_backend.o
c/_cffi_backend.c:15:17: fatal error: ffi.h: No such file or directory
#include <ffi.h>
             ^

compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Cleaning up...
Command /usr/bin/python -c "import setuptools, tokenize;file='/tmp/pip_build_vagrant/cffi/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-VxQZr4-record/install-record.txt --single-version-externally-managed --compile failed with error code 1 in /tmp/pip_build_vagrant/cffi
Storing debug log for failure in /tmp/tmp5Ael23
make: *** [install-python-requirements] Error 1`
Can I get a bit of assistance

Apologies for the late response. Looks like you're missing the headers forlibffi. These can be installed via the libffi-dev package on Debian-based systems.

I think I might be having a missing dependency:
`running build_ext
building '_cffi_backend' extension
creating build/temp.linux-x86_64-2.7
creating build/temp.linux-x86_64-2.7/c
x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -DUSE__THREAD -I/usr/include/ffi -I/usr/include/libffi -I/usr/include/python2.7 -c c/_cffi_backend.c -o build/temp.linux-x86_64-2.7/c/_cffi_backend.o
x86_64-linux-gnu-gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-Bsymbolic-functions -Wl,-z,relro -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -D_FORTIFY_SOURCE=2 -g -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security build/temp.linux-x86_64-2.7/c/_cffi_backend.o -lffi -o build/lib.linux-x86_64-2.7/_cffi_backend.so
running install_lib
creating /usr/local/lib/python2.7/dist-packages/cffi
error: could not create '/usr/local/lib/python2.7/dist-packages/cffi': Permission denied

Cleaning up...
Command /usr/bin/python -c "import setuptools, tokenize;file='/tmp/pip_build_vagrant/cffi/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-tv3iKB-record/install-record.txt --single-version-externally-managed --compile failed with error code 1 in /tmp/pip_build_vagrant/cffi
Storing debug log for failure in /tmp/tmpR1TXEE
make: *** [install-python-requirements] Error 1`

Did you do a sudo pip install at some point? That may have messed up the permissions on some of those directories. I'm not too familiar with Vagrant so I'm reaching the end of my ability to give useful suggestions here.

Thanks @andrewmwhite I am closing the ticket. I will find a way to resolve this

@hatityechindove I was able to get Stethoscope up and running on a debian/jessie64 image via Vagrant using the commands here:
https://gist.github.com/andrewmwhite/d495f498a751f3b1e7bf2cb0f9252514
Hope that helps.


Add load-pkg to the makefile and add the development guide link to README

Description of your changes
This PR adds load-pkg dev make target to the makefile.
Fixes: https://github.com/upbound/provider-gcp/issues/309
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Manually tested in turkenf/provider-gcp and my dockerhub org: turkenf
NAME                                 INSTALLED   HEALTHY   PACKAGE                                                             AGE
upbound-provider-gcp-cloudplatform   True        True      turkenf/provider-gcp-cloudplatform-amd64:v0.36.0-rc.0.9.g6940237d   14m
upbound-provider-gcp-config          True        True      turkenf/provider-gcp-config-amd64:v0.36.0-rc.0.9.g6940237d          14m

/test-examples="examples/cloudplatform/serviceaccount.yaml"


Using on property requires reload and loses animation

If the on value is set without calling reload, the entire checkbox disappears. It is actually still tappable, and would re-appear when it is tapped to on.
Then I tried calling reload after changing on value. This time the checkbox won't disappear, but the on/off animation will be gone forever in this checkbox instance.

Hi @danie11am,
Thank you for opening this issue!
What version of BEMCheckBox are you currently using?
This is potentially a duplicate of #6.

No problem! I'm using 2.1.0.

Thanks @danie11am, but the latest version of BEMCheckBox is 1.2.0!
Would you mind checking with this version that the bug still occurs?
In the meantime I going to close this bug because I strongly suspect it to be fixed with 1.2.0.
Thank again!

Argh! I'm sorry - I was using 1.1.0. Sure I'll try 1.2.0.

Yep it works. No need to call reload(). And the "loses animation" bit was probably my mistake, where my "didTapOnCheckbox" would cause a state change, and the didSet of my state change set the checkbox.on, therefore losing the animation. Fixed that too.

@danie11am,
Great to hear, thanks for your feedback!


feat(spantest): add support for washing DDL stmts from in-memory DB

spannertest does not yet support row deletion policies which causes the in-memory database to fail with unhandled DDL table alteration type spansql.AddRowDeletionPolicy.
This add supports for washing row deletion policy statements from the given DDL before applying it to the in-memory database so existing test do not fail

Codecov Report

Merging #170 (82a6ede) into master (827a37a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   44.64%   44.64%
=======================================
  Files          21       21
  Lines        4265     4265
=======================================
  Hits         1904     1904
  Misses       2249     2249
  Partials      112      112


Auto class pick-up

I suggest you add a auto class pick ( allows you to automatically select a class in config, so we for example do not have to manually select soldier, if we play with sondier config)

???

just pick any class, its not that hard???

You aren't owed features.

when crithack

this has been added and merged


predict() selects random actions

trained_model.predict() selects random actions for the same input. It is probably the expected behavior, but I need deterministic actions in some cases(I am using this library for algorithmic trading). For now I have implemented it with action_probability(), but it would be nice to have a method to return deterministic actions for test run.

Hi,
Yes, most algorithms have stochastic policies. Some (like DDPG) are stochastic during training but fully deterministic during testing. A workaround is what you have done with the action_probability.
That may be a good idea to add a deterministic keyword. What do you think @hill-a  ?

That's a good point, I’ll add it to the TODO list of the current branch.

@nagadomi #16 was just merged with master. It adds the "deterministic" argument to the predict function ;)
We plan to release it soon on pypi, but you can install from source (see documentation "Bleeding-Edge version" https://stable-baselines.readthedocs.io/en/master/guide/install.html#bleeding-edge-version)

Thank you for your prompt attention to this issue. :+1:


Small bug: model_name instead of deployment_name in AzureOpenAIEmbeddingSkill object creation

When creating the AzureOpenAIEmbeddingSkill instance in azure-search-integrated-vectorization-sample.ipynb, the model variable containing "text-embedding-ada-002" model name is used as input for deployment_id field, instead of the actual name of the model deployment.
https://github.com/Azure/azure-search-vector-samples/blob/692e2e38d5c29af8790b8280298a1e1f433730d9/demo-python/code/azure-search-integrated-vectorization-sample.ipynb#L301
I think it should be changed to deployment_id=os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYED_MODEL").

Also, in Search Index creation, when assigning an AzureOpenAIVectorizer, there must be the same fix (deployment_id=os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYED_MODEL")). Otherwise, the vector search would return the error 'vectorization endpoint returned status code '404' (NotFound)'.

@matteomarjanovic

As I can see there is not the same error as in the Jupyter Notebook I mentioned previously

Thanks for the feedback, we'll investigate this

Hi @matteomarjanovic ,
We have refactored this notebook to resolve the issue you pointed out. Please let us know if it works for you now.

"Could not complete vectorization action. The service failed to authenticate to the vectorization endpoint."
i am getting this error when using azure ai studio with azure search. is there any way that i can fix the issue?


:ArduinoChoose* commands do not work in Vim9

Describe the bug
Commands such as :ArduinoChooseBoard, :ArduinoChoosePort and :ArduinoChooseProgrammer, don't seem to work in Vim9, see screenshots below.
System information

OS: Mac Os X 13.4
Vim: 9.0.1276
CLI version: arduino-cli  Version: 0.33.0 Commit: ca60d4b4 Date: 2023-05-29T15:18:18Z
ArduinoInfo:

Board         : arduino:avr:uno
Programmer    :
Port          : /dev/tty.usbmodem101
Baud rate     : 9600
Hardware dirs : /Users/ubaldot/Library/Arduino15/packages/arduino/hardware/avr/1.8.6
Verify command: arduino-cli compile -b arduino:avr:uno -p /dev/tty.usbmodem101 --build-path "/Users/ubaldot/Documents/arduino/build" -v "/Users/ubaldot/Documents/arduino/pippo.ino"

To Reproduce
Install MacVim 9.0.1276, open a .ino file and run any vim-arduino command.
Expected behavior
I can operate with my arduino uno board.
Screenshots

Additional context
Congrats for the plugin! Starred! :)

I believe it should be fixed. Can you give it another try?

Yes, now it does. :) Thanks!


Updated config file path resolving

Updated config file path resolving to support both relative and absolute paths.
Also fixed the doc.

Merged
Thank you!


Blazor server crashes when setting base64 image

I want to use this library within a blazor server application. However, I don't have the images as image file. I only have them as base64 string.
When I try to display such string, my application crashes and gives me the following error message:
[2021-03-18T16:39:06.287Z] Error: Microsoft.JSInterop.JSException: Could not find 'WheelZoomJsFunctions.boundingRect' ('WheelZoomJsFunctions' was undefined).
r</e.prototype.findFunction/<@https://localhost:5001/_framework/blazor.server.js:1:67390
r</e.prototype.findFunction@https://localhost:5001/_framework/blazor.server.js:1:67350
v@https://localhost:5001/_framework/blazor.server.js:1:69092
beginInvokeJSFromDotNet/i<@https://localhost:5001/_framework/blazor.server.js:1:70038
beginInvokeJSFromDotNet@https://localhost:5001/_framework/blazor.server.js:1:70011
O</e.prototype.invokeClientMethod/<@https://localhost:5001/_framework/blazor.server.js:1:26293
O</e.prototype.invokeClientMethod@https://localhost:5001/_framework/blazor.server.js:1:26263
O</e.prototype.processIncomingData@https://localhost:5001/_framework/blazor.server.js:1:24201
e/this.connection.onreceive@https://localhost:5001/_framework/blazor.server.js:1:17286
J</e.prototype.connect/</</</i.onmessage@https://localhost:5001/_framework/blazor.server.js:1:46503
at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
at BlazorWheelZoomLibrary.Shared.WheelZoom.InitAsync()
at BlazorWheelZoomLibrary.Shared.WheelZoom.OnAfterRenderAsync(Boolean firstRender)
at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle)
I implemented your component as:
<WheelZoom Image="@($"data:image/png;base64, {image}")" width="583" height="820" />

Hi,
This error is not related to base64 image.
You forgot to add

in _Host.cshtml
Anyway without this missing line it was not working.
I fixed it, please upate and look at this example
https://github.com/iso8859/BlazorWheelZoom/blob/master/BlazorWheelZoomServer/Pages/Index.razor

@iso8859 I had this code in my _Host.cshtml. I did everything as you described in your READ.ME
I will check out the update later today and let you know if it works or not.

@iso8859  with the updated library it works fine for me. However, you forgot to publish the update to nuget.

@MarvinKlein1508 I forget. Update published.


Accessing public methods for multiple sliders

Hi,
I'm trying to build a page with multiple lightSliders. Each slider works individually, but I can't access any public methods. E.g. this code throws an error if jQuery selects more than one object:
var sliders = $('ul.lightslider').lightSlider({ item: 5 });
sliders[0].refresh(); // Uncaught TypeError: sliders[0].refresh is not a function

I really want to do something like this:
// event happens, find parent lightslider, call refresh()
$('#content').on('slid.bs.carousel', function (e) {
    $(e.target).closest('ul.lightslider').refresh();
});

Is the second scenario supported? Can I make a different selection or call lightSlider() again?

I'm doing the same thing by making a suffix to each slider and save it in the window variable.
Like
..
window.autoplaySlider_example1 = jQuery("#light-slider-example1").lightSlider();
Regards,
Casper

Sorry, should've opened this question on SO. I rewrote my local copy of lightSlider.js to support this.

@Turgar anything you can share with us?

Needed exactly this.
This how I did it.
var slides = [];
$('.card-image').each(function( index) {
    slides[index] = $(this).find('ul').lightSlider({item: 1});
});

$('body').on('click','.zoom-toggle', function() {
    var itemIndex = $('.zoom-toggle').index(this);
    slides[itemIndex].refresh();
});

I needed to have Slider controls outside of the HTML of the lightSlider, and having multiple of them per page. This is how I did it (thanks @Makkafella for the inspiration)

           var areaCarousels = [];
          $body.find(".area-carousel").each(function(index){
            areaCarousels[index] = $(this).lightSlider({
              item: 4
            });
          });

         // A bit complex because multiple sliders on
        //  the page. We need an external array to save
        // lightSlider object (that exposes the API)
          $.each(areaCarousels, function(index){
            var lS_Object = areaCarousels[index];
            var $that = $(lS_Object);
            // .carousel-block is a wrapper outside of .area-carousel
            var $sliderControls = $that.closest('.carousel-block').find('.lSAction.js-custom');
            if ($that.find('.views-row').length > 4) {
              $sliderControls.addClass('enabled')
            }
            $sliderControls.find('.lSNext').on('click',function() {
              lS_Object.goToNextSlide()
            });
            $sliderControls.find('.lSPrev').on('click',function() {
              lS_Object.goToPrevSlide()
            });
          });

I needed to have Slider controls outside of the HTML of the lightSlider, and having multiple of them per page. This is how I did it (thanks @Makkafella for the inspiration)

           var areaCarousels = [];
          $body.find(".area-carousel").each(function(index){
            areaCarousels[index] = $(this).lightSlider({
              item: 4
            });
          });

         // A bit complex because multiple sliders on
        //  the page. We need an external array to save
        // lightSlider object (that exposes the API)
          $.each(areaCarousels, function(index){
            var lS_Object = areaCarousels[index];
            var $that = $(lS_Object);
            // .carousel-block is a wrapper outside of .area-carousel
            var $sliderControls = $that.closest('.carousel-block').find('.lSAction.js-custom');
            if ($that.find('.views-row').length > 4) {
              $sliderControls.addClass('enabled')
            }
            $sliderControls.find('.lSNext').on('click',function() {
              lS_Object.goToNextSlide()
            });
            $sliderControls.find('.lSPrev').on('click',function() {
              lS_Object.goToPrevSlide()
            });
          });

Works like charm! Thanks!


Add readerInit to Code 128, Data Matrix, MaxiCode, PDF417

Adds new readerInit flag indicating a Reader Initialisation/Programming symbol to Result.h, and handling of it to ODCode128Reader, DMDecoder, MCDecoder and PDFDecodedBitStreamParser.
Aztec Code also has a Reader Initialisation option, but will do in a subsequent PR as it's a bit messy (done in the detector) compared to the simple ones here.
Unit tests for each. Test samples (generated by Zint) added for Data Matrix and PDF417, but not for Code 128 and MaxiCode, as they're pretty trivial, but can add them if thought best.

Thanks! I'm rather busy at the moment. therefore the delay...

Thanks for reviewing (compared to most projects the delay was nothing).


Provide dotenv examples

Please make sure to check the following tasks before opening and submitting a PR

[✅] I understand and have followed the contribution guide
[✅] I have tested my changes locally and they are working as intended
[✅ ] These changes do not have any notable side effects on other Revolt projects

From me on the Revolt.py server:

so, I've been looking at the examples for revolt.py bots - going off my discord knowledge, it's best practice for bot tokens to be secure and not be stored in source code, which typically means using the dotenv module to load a .env file containing the bot's token.

After 20 minutes of messing with Pyright and thinking it was a completely different tool, the check issue is fixed! :)

Hi thanks for the PR, however i dont believe env variables are a good solution for this, and furthermore the dotenv package is not required as python can handle environment variables just fine without it. storing tokens is pretty easy and i dont think it requires an example.


[Bug]: (+) icon symbol is not displayed by default on selecting none for menu and input widget

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Icon property behavior is not consistent for icon widget and other widget  having icon as their property
Note : Issue exists for the widgets which has icon property
ex :

menu widget
Input widget

Attached video : https://www.loom.com/share/60e053095e424356867e067cea5426da?sid=5bae4489-5f2f-4fa9-b6ca-0c161b33835c
Steps To Reproduce

Drag and drop menu widget/input widget
Go to styling panel
Select any icon (lock icon) in icon property
now search for number in icon property such as 123, it will give none as search result
Select none and observe no icon displays
Repeat same steps for icon widget and observe  (+) icon will be displayed by default on selecting none

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Appsmith v1.9.35-SNAPSHOT

@dilippitchika - I don't think showing (+) icon when we search for something and click on (none) is a good idea. (none) should ideally mean no icon is displayed - as we do in the input and the menu button widget. But since we have set the default icon for the icon button as "plus", it automatically picks up the plus icon. We can change this default to none, but that would result in a weird empty icon. Not sure what our next steps should be here.

The reason we added this was that we have a default icon in this icon button, can't we just hide the none option for icon button altogether?


Pip not able to install climetlab (v0.11.31) in windows with Python 3.7 : HDF5 headers not found

The title is quite explicit :p . I get this error in the tests for the release of a new version of the EUMETNET benchmark climetlab plugin. Is climetlab still supported in Python 3.7?
See the error log here:
Collecting climetlab
  Downloading climetlab-0.11.31.tar.gz (128 kB)
     -------------------------------------- 128.1/128.1 kB 3.8 MB/s eta 0:00:00
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'done'
Collecting numpy
  Downloading numpy-1.21.6-cp37-cp37m-win_amd64.whl (14.0 MB)
     --------------------------------------- 14.0/14.0 MB 46.7 MB/s eta 0:00:00
Collecting pandas
  Downloading pandas-1.3.5-cp37-cp37m-win_amd64.whl (10.0 MB)
     --------------------------------------- 10.0/10.0 MB 57.9 MB/s eta 0:00:00
Collecting xarray>=0.19.0
  Downloading xarray-0.20.2-py3-none-any.whl (845 kB)
     ------------------------------------- 845.2/845.2 kB 26.9 MB/s eta 0:00:00
Collecting requests
  Downloading requests-2.28.1-py3-none-any.whl (62 kB)
     ---------------------------------------- 62.8/62.8 kB 3.3 MB/s eta 0:00:00
Collecting dask
  Downloading dask-2022.2.0-py3-none-any.whl (1.1 MB)
     ---------------------------------------- 1.1/1.1 MB 22.2 MB/s eta 0:00:00
Collecting netcdf4
  Downloading netCDF4-1.6.0.tar.gz (774 kB)
     ------------------------------------- 774.2/774.2 kB 24.6 MB/s eta 0:00:00
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'error'
  error: subprocess-exited-with-error

  python setup.py egg_info did not run successfully.
  exit code: 1

  [25 lines of output]
  reading from setup.cfg...

      HDF5_DIR environment variable not set, checking some standard locations ..
  checking C:\Users\runneradmin\include ...
  hdf5 headers not found in C:\Users\runneradmin\include
  checking /usr/local\include ...
  hdf5 headers not found in /usr/local\include
  checking /sw\include ...
  hdf5 headers not found in /sw\include
  checking /opt\include ...
  hdf5 headers not found in /opt\include
  checking /opt/local\include ...
  hdf5 headers not found in /opt/local\include
  checking /opt/homebrew\include ...
  hdf5 headers not found in /opt/homebrew\include
  checking /usr\include ...
  hdf5 headers not found in /usr\include
  Traceback (most recent call last):
    File "<string>", line 36, in <module>
    File "<pip-setuptools-caller>", line 34, in <module>
    File "C:\Users\RUNNER~1\AppData\Local\Temp\pip-install-wpzd3q9h\netcdf4_0e84f88a843e43dab815c3ef9955b375\setup.py", line 444, in <module>
      _populate_hdf5_info(dirstosearch, inc_dirs, libs, lib_dirs)
    File "C:\Users\RUNNER~1\AppData\Local\Temp\pip-install-wpzd3q9h\netcdf4_0e84f88a843e43dab815c3ef9955b375\setup.py", line 385, in _populate_hdf5_info
Error:       raise ValueError('did not find HDF5 headers')
  ValueError: did not find HDF5 headers
  [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

Encountered error while generating package metadata.

See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
Error: Process completed with exit code 1.

This seems to be a common problem with NetCDF4 after a new release on Python 3.7:
https://github.com/Unidata/netcdf4-python/issues/901#issue-425297888
https://github.com/Unidata/netcdf4-python/issues/1031#issue-665205433

Indeed, apparently, the issue comes from the netcdf4 package, there is not much that we can do from climetlab.

I want to comment on the fact that this is painful for plugin maintainers, because I cannot push to pip server due to this.
Plugin users start complaining that they cannot reach latest features because pip and github are not aligned.
If I have to wait 6 months that this is solved for this very particular version, then this is a problem.
The solution I see is dropping some Python versions in the tests to publish.
@floriankrb :
What would be the best according to you :

Drop all Windows tests (problems are usually there)
Drop only Python 3.7 Windows tests

?
Thank you in advance.

Yes, droping support for Windows python 3.7 would make sense IMO, depending on your users of course.
You can achieve this in the .github/workflow yaml file using exclude as in:
    strategy:
      fail-fast: true
      matrix:
        platform: ["ubuntu-latest", "macos-latest", "windows-latest"]
        python-version: ["3.7", "3.8", "3.9", "3.10"]
        exclude:
          - platform: "windows-latest"
            python-version: "3.7"

Ok, I will do that. I hope this is not a slippery slope though :-)


propagate catched exceptions messages in the flow

@lpuskas , please take a look!

propagate catched exceptions messages in the flow
fix cluster status update failure handlers
create proper status updates for rollbacking

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/997/

lgtm


Recursive Index in page tables?

Is it always index 0? Should this be passed in the BootInfo struct?

We used to hardcode the index to 511, but this changed with #71. Now we dynamically look for a free level 4 entry. The chosen index is indirectly reported through the recursive_page_table_addr field in the BootInfo:
https://github.com/rust-osdev/bootloader/blob/90f5b8910d146d6d489b70a6341d778253663cfa/src/bootinfo/mod.rs#L32-L34
It shouldn't be 0, however, because the bootloader maps itself to that slot…

Thanks! I eventually figured that out. Perhaps a "get recursive index" method could be added that just masks out bits 21:12 and returns a PageTableIndex?


Analytics UI: CSS padding / positioning / display styling on results table

A couple of CSS bugs:

When displaying a large number of results from an Analytic, the table takes up the full extent of the visible vertical scroll. Consequently the pagination buttons / dropdowns below the table are no longer visible.
Indentation looks broken. HTML elements on the edge of the view port.

I'll claim this one.

Closed due to redundancy


somehow ffmpeg permanently hangs when attempting to finalize a video that already exists but wasn't detected/skipped

Not sure what causes it to happen, but it will start to download a video and go through the entire process, then freeze when doing the ffmpeg info. It's somehow not detecting an already downloaded video and re-downloading it (guessing the size changed or something?), but because the name already exists, it can't finish the process so it just sits there forever.
https://cdn.discordapp.com/attachments/468977376794247199/841765883276820480/unknown.png One screenshot showing somebody else having the issue. Band-aid solution is to close the program, and delete both the original.mp4 and the new blank extension file and make it redownload the whole thing once more.

Okay. Sorry I've been slowly towards adding some functionality, including
manually removing tasks. That might be a solution, and I'll tag it on this
issue when it's out (at least in alpha). I'm not sure how you're using
ffmpeg to report the progress to the bars, but it seems kind of weird to me
that ffmpeg would hang because of this.
On Tue, May 11, 2021, 15:09 Hezkezl @.***> wrote:

Not sure what causes it to happen, but it will start to download a video
and go through the entire process, then freeze when doing the ffmpeg info.
It's somehow not detecting an already downloaded video and re-downloading
it (guessing the size changed or something?), but because the name already
exists, it can't finish the process so it just sits there forever.
https://cdn.discordapp.com/attachments/468977376794247199/841765883276820480/unknown.png
One screenshot showing somebody else having the issue. Band-aid solution is
to close the program, and delete both the original.mp4 and the new blank
extension file and make it redownload the whole thing once more.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kamiyo/multi-progress-bars/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWCYT4HMJTPWQO63BPHXA3TNGFHDANCNFSM44WYLRNA
.


Could YTXAnimations can manual install?

I don't want to use pod since my project is a static library.

Please try to add use_framework in podfile

I copied the source code and add some include like UIKit and CoreGraphics, then it works fine :）


Update akka-stream, akka-testkit to 2.6.13

Updates

com.typesafe.akka:akka-stream
com.typesafe.akka:akka-testkit

from 2.6.4 to 2.6.13.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.akka" } ]

labels: library-update, semver-patch

Superseded by #68.


2.5k 404's in the last 30 days

Need to research and determine where the majority are coming from. One culprit is definitely https://docs.microsoft.com/en-us/microsoftteams/add-bots (that page should just be removed entirely)

The page that Andrew is referencing has been removed and the link now redirects to this page: https://docs.microsoft.com/en-us/microsoftteams/platform/bots/what-are-bots. That page is up-to-date. The issue is resolved.


Console and notebook need larger UI/UX difference

Right now, there's little distinction that you're in a notebook or in a console cell -- the only real difference is the background of the cell.

The title of the notebook could be more clear and bigger. This would help emphasize you're in a notebook.

Right now in the notebook you just click on the title to change it, and this could be worked into the proposed changes. This is the first thing I tried to do in my user testing and could make the transition easier.

There could be more distinction between the notebook and console.

I would like to see using Enter instead of Shift-Enter to execute the code in the current console cell. That's the every other console works and that's what I find natural. This is perhaps the largest annoyance of mine.
I propose outlining the console cell in a different color to represent that hitting Enter will execute the current cell. This also provides another more apparent UI difference -- I find the background to be a small difference -- it doesn't pop out at you.

I have mocked up this interface I have in mind:

Hi @stsievert, thank you for the suggestions.

This is somethign that we have discussed. In particular, the console supports a very different interaction in that only the last cell can be edited. In my mind that is the main different. Because of that I am thinking about something like the following:

For the final console cell that is being edited, use the same exact visual style as the notebook cells in edit mode. That visual style is already optimized to communicate "this cell is being edited and can be typed in and then run". This would include the slightly darker grey back ground to the code mirror input area.
Create a new visual style for the previously run cells in the console that is different in more significant ways. Some ideas: keep white background, put a more significant visual divider between the last cell and the others, scroll the already-run cells separately, etc.

That visual style is already optimized to communicate "this cell is being edited and can be typed in and then run".

When I use the console, I'd like to use the Return key to run the typed code, not Shift-Enter. I proposed a red outline to resolve this difference in key mapping.

I think the above better represents edit + new key mapping better.

Here is a mockup of a console redesign. The main points are these:

Cell styles are the same as the notebook.
Active input cell is always at the bottom with a white background and green border.
Run cells get appended to a scrolling area above (but visually behind) the active input cell.
The run cells get a light grey background to indicate their inactive state and are each separated by a thin grey line.
Toolbar at the top with kernel related icon buttons.

Assigning to @faricacarroll to finalize the visual design (see me for the sketch file). Who wants to code this one up?

Here are mockups of the console redesign:
There are two versions: one version with a top toolbar and the other with a floating toolbar. The top toolbar takes up a lot of space for only having 4 icons. The abundance of white space on the toolbar also looks a little awkward to me.
The floating toolbar would be a new element to JupyterLab, but could be used in other widgets (like the ImageWidget) which also have too few toolbar items to fill up space on a toolbar appropriately.

I suggest having two separate issues: keeping this one for the UI design (since it's already going in that direction) and having a new one to focus on the Enter/Shift-Enter discussion.
@stsievert, it turns out that can be done but it's tricky to get right. In the Qt console we indeed manage to do a decent job of getting single-Enter execution in most cases while allowing for multi-line input easily.  But we'll need to discuss how much of that we can bring here as part of our attempt at feature-parity (so we can replace the qtconsole completely with users being happy about the transition).
Do you want to open another one just for the keyboard execution behavior?

@faricacarroll @ellisonbg the only other thing I'd like to see is have the output be right above the input. Past that, it looks good -- nice UI redesign.

Looking really great! Some ideas to explore:

Output should be aligned with the code of the input.
Let's use a small drop shadow on the past in/out scrolling area to indicate it is lower than the white stuff.
For the grey of the past in/out and the dividers, play around with different MD greys in the 50-300 range.
To keep things simple let's use the notebook style toolbar here, but with (run, stop, restart, clear all, kernel type, kernel status)

@stsievert as far as where new outputs go when it is empty (top or bottom) I am not sure. Let's build it and play to see which makes more sense. I am open to both!

This is looking pretty cool. I think we could even drop the green border for the input cell, since it clearly already stands out, and is the only cell with which interaction is possible.

is the only cell with which interaction is possible.

an input() call might take focus, so there might be other cell outputs that have interaction.

Fair enough. My intent was more "this is clearly the one-and-only cell which supports cell editing, i.e. you can't go back and modify history".

And I agree that setting off the cell padding color as white (as in the example), or having a thick line above the input cell, does a fine job of isolating the single input area that is active, so I agree that the green border is not needed.

Some iterations:

Used a notebook toolbar and included items for run, stop, restart, clear all, kernel type, kernel status

The closest fontawesome icon I could find to represent clear all was an x inside a circle

Removed the green border
Scrolling area grey color is MD Grey 100 with dividers of MD Grey 300. Bottom input cell grey is MD Grey 50.
Output cell is aligned with code of input cell (in the current console it is not aligned #bug)
Added slight drop shadows to grey scrolling area

Floating toolbar:

To keep things simple let's use the notebook style toolbar here, but with (run, stop, restart, clear all, kernel type, kernel status)

One related remarque with respect to toolbar vs floating toolbar which might be of interest for  long-time usage.
I'm seeing more and more (completly subjective measure) users who have touch screen; especially on windows (surface) and some chrome pixels (I think), and iPad + smart cover.
In particular they both hold screen by the side in tablet mode; and I believe having the floating toolbar (on the right) is extremely useful as actions are reachable by the thumb (for right-handed user).
Even on "Laptop" mode, many of these users still reach  for the screen to touch it, and the position of the toolbar on the upper left side on the screen is the least optimal for right handed people; While floating position seem relatively natural to reach.
Regardless of the above notions, it seem to me that the 2 area user will click on are:

the input field (bottom of th screen)
the toolbar (top of the screen)

So it seem that the 2 main target in interactive workflow with the console are almost as far from each other as we could. (They could be left/right, agreed)
So it seem like the current location is less than optimal in most cases; I understand that in the perspective of time and to get things out we should not bikeshed on the location on things; but I think keeping the above point in mind for a potential redesign or improvements; or even for a "touchscreen" mode would be great.

@Carreau while I don't disagree, it would be great to have some survey data on the number of users wanting to use JLab with touch interaction, to ensure we focus on the right things.

@Carreau @sccolbert :+1: to mobile use. I've also heard concerns about JLab + mobile and will likely use on mobile in the future.
Hm, how 'bout the design below? In this design, the input cell always stays right below the output and moves as the output expands. This is similar to how every other console works.

But initially all the white space is a tad distracting; maybe we could logo-stamp? As a user, I'd like to see the input console cells gradually cover the logo. I mocked that this idea up with a rough approximation to the style/colors above:

I do think the header text that print information about the console should remain.

@faricacarroll I think your last mock up is getting close. But I do think we want to keep the green border, as it communicates the same thing that it does in a notebook = cell is ready for you to type.
@stsievert I did think about not pinning the input area to the bottom of the panel, but I ran into the problem you noticed - way too much white space.
There are other benefits to having the input pinned to the bottom:

Always in the same place visually
Easy to shrink the vertical size of the console and still see the input area
Once a user has a screen full of run cells, the input area will be bottom pinned anyways.

I am in favor of building the version shown in @faricacarroll last mock up (but with the green border) and then having everyone play with it to see how it feels.

@ellisonbg I can agree with the input being pinned to the bottom.

Once a user has a screen full of run cells, the input area will be bottom pinned anyways.

But I dislike how far my eye has to travel to see the output for what was just typed, even if that only occurs for the first couple input cells. I still advocate for the output being pinned to the bottom as in https://github.com/jupyter/jupyterlab/issues/648#issuecomment-242923367
With the input + output being pinned to the bottom

the most current output is always in the same place
the eye doesn't have to travel too far at any time, similar to other consoles

and all the benefits you mentioned

@stsievert actually, keep in mind that there is a "banner" with the kernel and session information that will take up a bunch of vertical space ... so the gap might not be as large as you fear. This will be different for different kernels, but the IPython banner is quite big:

I understand the concern @stsievert has about the large gap between input/output initially. However, I agree with @ellisonbg that for now, we can build up the original version and then user test to see how users feel about it.
Here's a mockup with the green border:

Some points of data: the chrome console inputs are directly below outputs until you get to the bottom of the screen, and then of course they are against the output. On the other hand, the Firefox console input is at the bottom, but they echo the inputs above the outputs like in the mockup above.

Actually @afshin, the IPython banner just got quite a bit smaller :)
This is what it looks like now (at a terminal, but the same info is printed on the Qt console):

Fixed in master!


[Windows] Wrong directory generated for newly created content on Windows

An extra spurious directory is created when generating new content if hugo is invoked with a full path on Windows 7.
Precondition
Hugo 0.12 amd64 freshly installed in C:\Software as hugo_0.12 and not added to the path.
Steps to reproduce
C:\Users\Andrea\Documents\development> \Software\hugo_0.12 new site myblog
C:\Users\Andrea\Documents\development> cd myblog
C:\Users\Andrea\Documents\development\myblog> \Software\hugo_0.12 new about.md
ERROR: 2014/11/02 Config not found... using only defaults, stuff may not work
C:\Users\Andrea\Documents\development\myblog\Software\content\about.md created

As reported by the hugo new command output, a spurious Software directory is prepended to content.
This behaviour can be replicated always: the installation path of the hugo executable is always prepended to content.

OK, I interpret silence as a positive sign. Close this for now.


Scheduler page - Not showing correct data on filter using some type

Describe the bug

when we filter using monthly and bi-monthly it showing some other type also

https://www.berrycast.com/conversations/0217bfa9-15e7-54f6-9054-b6106cf466ee

https://www.berrycast.com/conversations/ab6c43c9-4c10-59a8-9c45-c7c8fd148464


An error occurred. Error message: "giscus is not installed on this repository".

Make sure giscus is installed for repo from repo settings > github apps and even if issue occurs.
You have to go to giscus repo
And follow the steps on their readme.
Give your repo's read and write access to giscus.
Here's the changes that we have done which resolved the issue "https://github.com/linux-club-dev/linux-club-web/commit/81c09a02735fb418a564ae25a40942dcc9695b93"


Deletion of one of the authorised users from LDAP kills the Access Control page

Rancher versions:
rancher/server: 1.6.7
Infrastructure Stack versions:
healthcheck: 0.2.3
ipsec: 0.11.2
network-services: 0.2.2
scheduler: 0.7.5
Docker version: (docker version,docker info preferred)
1.12.5
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Ubuntu 14.04, with 4.4 kernel
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
AWS
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node, external DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Setup LDAP access control
Tick "Allow members of Environments, plus Authorized Users and Organizations"
Add a user to the list
Save config
Delete user from LDAP source

Results:
Access Control page is blank, and Javascript console shows "Uncaught TypeError: Cannot read property 'externalIdType' of null"
Was able to manually delete the user using the API, but it's annoying.

Hello,
i have the same problem. Thanks for sharing.
One question:
When a list all users with v1/accounts/ the deleted ldap user are not shown.
How did you delete the deleted ldap user using the api?
Thanks in advance

Goto the API page, and click "Add Account API Key"
curl -u <access key>:<secret key> http://<rancher host>/v2-beta/openldapconfigs | jq .data[0] > ldap.json
Edit "ldap.json" and look at the "allowedIdentities" list. There should be a null entry. Delete it.
curl -H 'Content-Type: application/json' -u <access key>:<secret key> -XPOST http://<rancher host>/v2-beta/openldapconfigs --data @ldap.json

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it

Hi @loganhz, I just faced the exact same issue. Deleting the null entry resolved it for me, too, but it's still annoying. Almost the exact same setup as @palfrey.