Trying to use this with many different csprojs that reference each other

I have project A which defines a set of enums using your generator.  in project b i have yet another set of enums. while trying to reference project a enums in project b i cant seem to find the extensions methods.  Also say i have project c and i want it to have enums, while my project has referenced the nuget package i cant seem to see the [EnumGenerator] flags
I am referencing your generator like such

any ideas why i am seeing this, cool project if i can get it to work

Hmmm, if the enums are public, and the generated extensions are public, then they should be accessible?

Also say i have project c and i want it to have enums, while my project has referenced the nuget package i cant seem to see the [EnumGenerator] flags

I don't understand this - you're saying the NuGet works in A and B but not C? What's different with C? Can you provide a simple reproduction that demonstrates the problem?

Going to close this for now - feel free to reopen if you have more information


Fonts are very small on high dpi monitors.  Is there any way to adjust.

Doesn't seem to conform to ubuntu settings

I have also been having this problem. The main canvas working area seems fine but the icons and text in the toolbars, gui menus, etc. are tiny.
Testing on a Macbook Pro Retina Display with Ubuntu 22.10.
I have tried:

Setting the UI scale in wincfg
enabling high DPI mode in the Preview tab of Fusion settings
Changing to different graphics drivers in Fusion settings (OpenGL, DX9, DX11)
None of these things have made a difference to the app scaling.

A month later and still no fix or response from the developer? This snap package runs perfectly for me if it wasn't just for this one problem.

This is a known problem in Fusion, who have recently released a new feature to address it: https://knowledge.autodesk.com/support/fusion-360/troubleshooting/caas/sfdcarticles/sfdcarticles/Performance-Tips-for-running-Fusion-360-on-high-resolution-monitor.html - specifically HighDPIScaling option. It makes fonts bearable on 4K, albeit still rather small.
@Smartich0ke I believe the author of this repository contributed a lot to the community without being remunerated in any way shape or form. Please consider tuning down your expectations on the level of gratuitous support requested from others. I trust the author would also accept contributions to the repository should you be willing, in an open-source spirit, help with the project.


Passing arguments to command

grunt-shell allows passing arguments to commands to be run. Is there a way to do so in grunt-bg-shell as well?

@shashwatblack  I think you can achieve it using cmd function
bgShell: {
  lsTasks: {
    cmd: function(){return 'ls ' + YOUR_PASSED_ARGS}
  }
}

could you provide some code of what are you trying to do?


Only launch pty host when it's actually needed

After https://github.com/microsoft/vscode/pull/182631, we launch the pty host off the main process (or server) as soon as a window asks for a connection. The window connection will always happen after LifecyclePhase.Restored but we could push this even further by only launching the pty host when it's needed.
Currently we pre-emptively fetch terminal profiles but we could change that to happen on demand, meaning if you loaded VS Code without the terminal showing, and then ran the open terminal with profile command, it would be slightly delayed as it needs to wait for the pty host to load up as profiles are evaluated there. The benefits of this are:

Less work on startup (even if it's after restored)
~40mb less constant memory used by the pty host if you don't use the terminal
We'd want to do this anyway if we wanted to lazy load the entire terminal contrib which has been an idea we've thrown around a little in the past

@Tyriar in my local testing, even when I open an empty window I still see the process running:

@Tyriar my output in an empty window just having the output channel opened:
2023-07-04 15:11:31.846 [trace] [RPC Request] PtyService#refreshIgnoreProcessNames(["starship","oh-my-posh","bash","zsh"])
2023-07-04 15:11:31.846 [trace] [RPC Response] PtyService#refreshIgnoreProcessNames undefined
2023-07-04 15:11:31.846 [trace] [RPC Request] PtyService#getDefaultSystemShell(2)
2023-07-04 15:11:31.846 [trace] [RPC Response] PtyService#getDefaultSystemShell /bin/zsh
2023-07-04 15:11:31.847 [trace] [RPC Request] PtyService#getDefaultSystemShell(2)
2023-07-04 15:11:31.847 [trace] [RPC Response] PtyService#getDefaultSystemShell /bin/zsh
2023-07-04 15:11:31.847 [trace] [RPC Request] PtyService#getDefaultSystemShell(2)
2023-07-04 15:11:31.847 [trace] [RPC Response] PtyService#getDefaultSystemShell /bin/zsh
2023-07-04 15:11:31.873 [trace] [RPC Request] PtyService#getEnvironment()

Reopening to get to the bottom of ☝


Fix broken io connector links.

Fixes broken io connector links in master.
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

retest this please

run cpp tests

rerun java8 tests
rerun integration tests
rerun cpp tests

rerun cpp tests
rerun integration tests

rerun cpp tests
rerun integration tests

rerun cpp tests
rerun integration tests


libsystemd-journal is not well-maintained

https://github.com/ocharles/libsystemd-journal/pull/27#issuecomment-1488536165
We should consider taking over maintenance or dropping the dependency.
cc @jutaro since it's a cardano-tracer dep.

cardano-tracer was written by @denisshevchenko , and its currently unmaintained. Maybe @kosyrevSerge knows why this dependency exist.


docs: get docsearch to re-crawl the site

Have some bad urls, want a refresh.
Followup with DocSearch team or look for a way to trigger ourselves.

Added sitemap to site https://github.com/pluralsight/design-system/pull/1477
Sent in a PR to Docsearch for a config update https://github.com/algolia/docsearch-configs/pull/2910


actually use required headers when applying blobs to a view

This PR should be reviewed by: @socrata/pipeline


PyOpenSSL support doesn't work with TLSv1.1 and TLSv1.2

docker-py tries to use latest protocol version in the ssl module, so use ssl.PROTOCOL_TLSv1_2. Unfortunately the version of urllib3 included with requests (and likely upstream?) only has older versions:
https://github.com/kennethreitz/requests/blob/master/requests/packages/urllib3/contrib/pyopenssl.py#L71

So raise this issue against urllib3?


[3811] Add findable to with_vacancies scope for courses

Context
https://trello.com/c/ejcChJPq/3811-change-with-has-vacancies-to-only-include-findable-courses
Changes proposed in this pull request

Add findable scoping to the with_vacancies scope for courses

Guidance to review
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Change with_/has_ vacancies to only include findable courses

Looks good. Can we also change SiteStatus#with_vacancies to keep things consistent, though?

Done, also changed the SiteStatus with_vacancies? as well

Just noticed https://github.com/DFE-Digital/teacher-training-api/blob/3811-change-with-has-vacancies-to-only-include-findable-course/app/models/course.rb#L498-L502
  def ensure_site_statuses_match_study_mode
    site_statuses.select(&:with_vacancies?).each do |site_status|
      update_vac_status(study_mode, site_status)
    end
  end

We've changed how with_vacancies? behaves, have we thought through whether that will be an issue?

Is it worth changing this bit too? It sounds like having findable applied twice will be a null-op, but it could also be confusing to the reader.
https://github.com/DFE-Digital/teacher-training-api/blob/master/app/services/publish_reporting_service.rb#L109-L110
    findable_courses = courses_changed_recently.findable.distinct
    open_courses = findable_courses.with_vacancies

Is it worth changing this bit too? It sounds like having findable applied twice will be a null-op, but it could also be confusing to the reader.
https://github.com/DFE-Digital/teacher-training-api/blob/master/app/services/publish_reporting_service.rb#L109-L110
    findable_courses = courses_changed_recently.findable.distinct
    open_courses = findable_courses.with_vacancies

For this one we did spot it, but decided against changing it, as the findable_courses bit is used elsewhere, as well as having the distinct on the end. It felt like it would be more hassle adding findable to the other uses rather than just having this redundant one here.

Just noticed https://github.com/DFE-Digital/teacher-training-api/blob/master/app/models/course.rb#L498-L502
  def ensure_site_statuses_match_study_mode
    site_statuses.select(&:with_vacancies?).each do |site_status|
      update_vac_status(study_mode, site_status)
    end
  end

We've changed how with_vacancies? behaves, have we thought through whether that will be an issue?

Fixed this, we're now checking it slightly differently, by looking directly at each site statuses vac_status. This bypasses the with_vacancies? findable problem, keeping the original functionality intact!


Configurable location of the gRPC socket (internal.sock)

SUMMARY

For oasis-node the location of the gRPC UNIX socket (internal.sock) needs to be configurable in order to work with other non-root services. I suggest we add an optional configuration option, such as socket: into config.yml.
ISSUE TYPE

Feature Idea

COMPONENT NAME

go/oasis-node/cmd/common
ADDITIONAL INFORMATION

The oasis-node service needs to communicate with other services in order to be useful. We are likely encouraging bad practices, because the gRPC UNIX socket is located at {{datadir}}/internal.sock and we are enforcing strict permissions chmod 700 {{datadir}}. On Debian-like systems services typically open UNIX sockets under /run to expose it to other services (e.g. /run/cups/cups.sock) ("Modern Linux distributions include a /run directory as a temporary filesystem (tmpfs) which stores volatile runtime data.").
Exposing the non-authenticated gRPC socket should not be considered dangerous, especially if you expose it to a trusted other service. And even if you expose it over the network, the most an attacker can do is cause DoS (shutdown), he still can not do perform actually dangerous things, like manipulate the local state, trigger double-signing, steal any funds...
Alternatives considered:

other service needs to run in Docker on Linux and have internal.sock mounted in: good practice, but requires Docker daemon to run as root
other service needs to run as root or as the same user as oasis-node: dangerous practice, because the other service has access to all oasis-node files which defeats the purpose of chmod 700 (other service can has full control, can manipulate local state, cause double signing...)
have a post-start script that waits for oasis-node to perform the startup permission-check and then changes them chgrp sharedgroup {{datadir}} {{datadir}}/internal.sock && chmod 710 {{datadir}} && chmod 770 {{datadir}}/internal.sock: bad practice, ugly workaround
if we transition towards local abstract namespace sockets independent from filesystem \0oasis-node.socket: bad practice, because we throw away filesystem permissions that can limit access to the gRPC socket (every process on the system has full control)
if we transition towards loopback TCP sockets: bad practice, because everything depends on correctly set firewall rules that only give access to the other service (if firewall is incorrectly set, every process or even everyone on the local network has full control)
we could also implement https://github.com/oasisprotocol/oasis-core/issues/3994 and make the other gRPC UNIX socket configurable: good practice, but requires the decision what is "less safe" and what not

Options that do not work:

it does not work if you set special ACL permissions (setfacl) to the other service, because these also get reflected in permissions the startup permission-check still fails
other service can not run in Docker on Mac/Windows, because you can not mount internal.sock inside from outside Docker

https://github.com/oasisprotocol/oasis-core/blob/master/go/oasis-node/cmd/common/grpc/grpc.go#L34


Max validated docker version

I've been able to start up the cluster with docker 17.12.0~ce-0~ubuntu but I do see this message in the output when running terraform apply
[WARNING SystemVerification]: docker version is greater than the most recently validated version. Docker version: 17.12.0-ce. Max validated version: 17.03

I'll close this, not sure it's an issue.


Document use of Astro’s JSX types

What kind of changes does this PR include?

New or updated content

Description

Closes #984
Adds a “Built-in attribute types” subheading to the Typescript page’s “Component Props” section that explains how to use and also extend Astro’s built-in JSX definitions.

@Princesseuh I tried these examples out in the docs repo and it seems to work without the need for the /// directive. Would you mind reading through this and letting me know if you think this covers what you imagined? 💜

@Princesseuh I tried these examples out in the docs repo and it seems to work without the need for the /// directive.

This is true in Astro files only because the language server inject those types into Astro files in the editor, however it's technically improper code 😄 Additionally, it is needed in other files, so definitely needs to be documented. Our own image integration use it in a .ts file, for instance

@Princesseuh Added a note to that effect:


ModuleNotFoundError: No module named 'colossalai._C.cpu_adam'

I meet this error “ModuleNotFoundError: No module named 'colossalai._C.cpu_adam”
I have tried many methods but failed to solve it. Have you ever encountered it? Could you give me some advice， thanks！

I meet this error “ModuleNotFoundError: No module named 'colossalai._C.cpu_adam”
I have tried many methods but failed to solve it. Have you ever encountered it? Could you give me some advice， thanks！


Support Error catch events without error code (catch-all Error events)

Problem you would like to solve
With https://github.com/camunda/zeebe/issues/11126, Zeebe now supports Error catch events without an error code.
This enables users to model a specific response for a known escalation code, and a general response for unknown escalation codes.
Currently, Error catch events without error codes are not supported in the Modeler for Camunda 8 BPMN diagrams.

Proposed solution

[ ] The Modeler allows the creation of Error catch all events without explicitly referencing a specific error, in two ways:

[ ] ErrorEventDefinition#errorRef is optional
[ ] Error#errorCode is optional (https://github.com/camunda/camunda-modeler/issues/3362#issuecomment-1471641550)

[ ] There can only be one Error catch all event per scope
[ ] Support for global errors is validated for Camunda 8.2

Alternatives considered
/
Additional context
In Zeebe, the feature will be available with version 8.2.0-alpha3.

@koevskinikola Thanks for opening the issue.
For clarity could you attach a process that defines a global error?

Hey @nikku, of course.
Here's a process where the two possible cases of a global error catch event are defined:

The Error Start event of the Event sub process references an Error where no errorCode is defined.
The Error Boundary event of the SubProcess doesn't define the errorRef attribute of the errorEventDefinition element.

Let me know if you need more details, or if there's a better way to attach a BPMN process on GitHub.

@koevskinikola You can attach by renaming to diagram.bpmn.txt.
Thanks for sharing the diagram. It is now clear to me that errorRef is optional, but also errorCode can be optional.

Updated the issue accordingly.

Looks good @nikku, thanks!

Moving to ready to potentially pick up this alignment topic in the next iteration(s).

@koevskinikola Can you help me understand what is not allowed. I'm not entirely sure about how scopes work. A couple of examples:
Example 1

Example 2

Example 3

Example 4

Hey @philippfromme,
From the examples you provided above:

Example 1 - not allowed. Event sub-processes are on the same scope as the root (process instance) scope. So two event sub-processes with a catch-all Error start event shouldn't be allowed since we need only one of them to catch all of the "un-assigned" errors.
Example 2 - allowed. The boundary catch-all Error event will catch all "un-assigned" errors in the scope of the sub-process Task (and any child scopes that it has). The catch-all Error start event will catch any "un-assigned" error events from the main process.
Example 3 - not allowed. Similar to example 1, the catch-all Error boundary events are assigned to the same Task scope, so any "un-assigned" errors that bubble up from the Task scope, or any child scopes only need to be caught by one of these events, the second is unnecessary.
Example 4 - allowed. The catch-all Error boundary events are assigned to different Task scopes. So each will catch "un-assigned" errors of the Task scope (and any child scopes) of the Task it borders on. So they aren't in conflict.

I can't think of any other use cases, so I think you managed to cover all of them. :)
Let me know if I can be of more help.

CC @philippfromme.
Did you account for https://github.com/camunda/camunda-modeler/issues/3362#issuecomment-1471641550?

If we want to mirror the Zeebe engine behavior, we have to change the lint rules to always require an error code if an error reference exists but to not require an error reference. So we'd get rid of this error:

I'd say let's change it since the whole point of the linter is to ensure engine compatibility including quirks that might exist in the engine.

@nikku https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/89 changes the rules to mirror Zeebe engine behavior. Can you have a look?


Include child activities when filtering for a parent

Checked for duplicates
Yes - I've already checked
Alternatives considered
Yes - and alternatives don't suffice
Related problems
I'm frustrated when [...] happens as documented in issue-XYZ
Describe the feature request
User-identified usability concern: Would like the ability to filter parent activities (in timeline editor) and know that the related children will be included.
Use case: When filtering for PIMS_Roll, would expect to see the child GNC turns. Could add GNC turns to the active filters, but then this would show all GNC turns in the plan, not just the children related to the parent PIMS activity.

May require a checkbox, maybe should just be default functionality.

[ ] Show child + ancestor activities


Samba: fix missing dependencies for building sambaFull

@edolstra, maybe you could have a look as I'm partly reverting some changes you applied recently

What are those dependencies needed for? AFAIK, stuff like subunit and nss_wrapper is only used by the test suite that we don't run.

Phew! I don't really know what they are needed for. But apparently sambaFull, especially the kerberos related stuff, wouldn't build without it.

sambaFull now builds because I've dropped the heimdal compat patch. Closing.


Make to inherited x11 environment available

It can be used in the X11 environment of the existing desktop.
$ xhost +local:docker # Allows access to X11 Display through unix socket in Docker.
$ docker run --rm -it --name aind --privileged \
	-v /lib/modules:/lib/modules:ro \
	-e DISPLAY=unix$DISPLAY \
        -e INHERIT_DISPLAY=1 \
	-v /tmp/.X11-unix:/tmp/.X11-unix:ro \
        -e SESSION_MANAGER_ARGS="--single-window" \ # It is also available
	aind/aind

Screenshot:

Added environment variables
SESSION_MANAGER_ARGS
Can give options to session-manager.
For a similar situation like this: https://github.com/anbox/anbox/issues/800#issuecomment-457783805
POST_SESSION_SCRIPT
This can be useful after anbox is running.
INHERIT_DISPLAY
If not set and DISPLAY is set, it is automatically set to 1.
DISPLAY
X11 display name

A more detailed X11 implementation is possible.
Below is a simple VNC example.
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: aind
  name: aind
spec:
  replicas: 1
  selector:
    matchLabels:
      app: aind
  template:
    metadata:
      labels:
        app: aind
    spec:
        securityContext:
          privileged: true
...
      containers:
      - name: aind
        image: aind/aind:latest
        tty: true
        securityContext:
          privileged: true
        volumeMounts:
        - name: host-lib-modules
          readOnly: true
          mountPath: /lib/modules
        - name: x11-unix
          readOnly: true
          mountPath: /tmp/.X11-unix
      - name: vnc
        image: vnc-server
        ports:
        - containerPort: 5900
        volumeMounts:
        - name: x11-unix
          mountPath: /tmp/.X11-unix
      volumes:
      - name: x11-unix
        emptyDir: {}
      - name: host-lib-modules
        hostPath:
          path: /lib/modules

Thanks!


Fix Issues caused if DBA Dash service user doesn't have permissions to collect object name

Discussed in https://github.com/trimble-oss/dba-dash/discussions/146
If the DBA Dash service account is unable to collect the object name this can result in a 'same key has already been added' error in the GUI.

Fixing the sort order to match exactly what the app expects will prevent the same key error.
In the collection script, if the object name returns NULL, it gets converted to a blank string.  It would be better to show to object id for the object name.  This would also prevent the error by not having two objects with duplicate names of a blank string.

The problem should be fixed by granting sufficient permissions to the DBA Dash service account but the app can provide a better user experience by implementing these fixes. Additionally, the security document has already been updated with a script that can be used to assign the minimum required permissions for users that don't want to run as sysadmin.

Fix included in latest release.


Ability to subscribe more than one channel

We should add the ability to add more channels. Let's say subscribing
"channels": ["ticker.BTC-PERPETUAL.raw", "ticker.ETH-PERPETUAL.raw"]}
The code creates an array of channel passed for the event may be we should join the string inside to make that as key for multiple channels.

Right now you can subscribe to multiple channels by simply calling subscribe once per subscription. I'm using that in production right now.
It would indeed be more optimal to pass a list or subscriptions, but in that case we need a a list of handlers as well? Else how do we inform your code of BTC ticker updates VS ETH ticker updates?

I was thinking to create an object and keys with a common handler.
Something like this
https://github.com/abhishekgahlot/deribit-v2-ws/commit/d6f68980d94aab3a51927653bfc0485c4433e3e9
it's incomplete and has a bug but the overall idea is visible in the commit.

At that point, I think it would be cleaner to simply use an event emitter
to emit events:
WS.subscribe('perpetual.eth')
WS.subscribe('perpetual.btc') // or:
WS.subscribe(['perpetual.eth', 'perpetual.btc'])
WS.on('perpetual.eth', yourHandler)
WS.on('perpetual.btc', yourBTCHandler)

What do you think?
On Sat, Apr 27, 2019 at 5:18 PM Abhishek Gahlot<EMAIL_ADDRESS>wrote:

I was thinking to create an object and keys with a common handler.
Something like this
abhishekgahlot@d6f6898
https://github.com/abhishekgahlot/deribit-v2-ws/commit/d6f68980d94aab3a51927653bfc0485c4433e3e9
it's incomplete and has a bug but the overall idea is visible in the
commit.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/askmike/deribit-v2-ws/issues/3#issuecomment-487294450,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHMYD7KTSTQ25DOMR4AZJDPSRVERANCNFSM4HHD3ACA
.

--
PGP key at keybase.io/mikevanrossum
https://keybase.io/mikevanrossum/key.asc

@askmike This looks cleaner than implementing an object and filtering out keys. We can do that.

I have a refactor into a class object on my list, similar to what I did
here: https://github.com/askmike/poloniex.js/blob/master/poloniex.js#L21
That will make multiple things easier, including this and less messy
reconnection logic. Not sure when I get around to that though.
On Thu, May 2, 2019 at 1:31 PM Abhishek Gahlot<EMAIL_ADDRESS>wrote:

@askmike https://github.com/askmike This looks cleaner than
implementing an object and filtering out keys. We can do that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/deribit-v2-ws/issues/3#issuecomment-488638666,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHMYD57DYL6UFAPPLSVAVDPTLGIHANCNFSM4HHD3ACA
.

--
PGP key at keybase.io/mikevanrossum
https://keybase.io/mikevanrossum/key.asc

Sure, For now I use your library the same way having a class object for deribit and wrapper on top of deribit-v2-ws. I will try to make a PR soon if possible.

This lib is a class now and it emits events when receiving subscription updates.
Now you can do:
db.subscribe('public','deribit_price_index.btc_usd');
db.subscribe('public','deribit_price_index.eth_usd');

db.on(deribit_price_index.btc_usd', console.log);
db.on(deribit_price_index.eth_usd', console.log);

Does that solve your problem?

@askmike This looks much better and it's possible to create multiple instances now, I was until now using this hack https://github.com/abhishekgahlot/deribit-v2-ws/commit/cc2c28694a4c7d157f405d7da505b697fb152fb5
and I can now use this lib again for multiple instances, Thanks!


Contact Form

I am having issues with the contact form. I found the code that pulls the contact form from getform.io, but I don't know how to link the submit button to send a message to my account (or somewhere I can access it). Would you be able to point me in the right direction?
Here is the image I see when I run a test with the contact form.

I continued to work on this issue and found a solution. My commits were behind and not updating the path for the form. I appreciate all your work putting together a great theme.


Jumpy colour wheels: Workaround by not reading colour data for certain bulbs

Issue
Color handling (as @ebaauw describes in #814 ) remains "jumpy", in the sense that gliding through or setting different RGB or CCT colours will make the colour wheel in the Home app jump between different colours, possibly overriding the colour stored in the colour picker in the Home app.
The issue is probably to colour settings being "fed back" from deCONZ or from the bulb, interpreted by the various layers/APIs and then translated back into some other colour in Home, perhaps with a delay (before which the user has chosen another colour).
Suggestion
Some users use the Home app (possibly with Siri) exclusively to control their lights, as I do. I am not reliant on any colour data going "upstream" from the bulbs and back into Home. I suggest that it would be possible to work around this issue by disabling the ability for deCONZ to feed colour back into Home for chosen bulbs, so that this issue is – while not fixed – at least rendered harmless.
This could for example be implemented in config.json, where the user could specify an array of light IDs that will not have the colour setting read back from the REST API.

That's what I already mentioned:

I should probably ignore xy for non-Hue lights in ct colormode.

Thank you, @ebaauw . There is still "jumpyness" when setting RGB colours, too. Would

ignore xy for non-Hue lights in ct colormode.

address this as well?

The jumping is a combination of poor HomeKit design on Apple's part and poor API design on Hue's and dresden's part; and of using sub-par lights, that don't handle colour too well.
In Zigbee, you send a command to a light to request it to change the colour.  The light reports it's actual colour through attributes.  A proper modelling of this would be to distinguish between target colour vs current colour.  Instead, both target and current colour are represented by the same API attribute and HomeKit characteristic(s).  The same holds for brightness and colour temperature, by the way.
Usually this is not too big a deal, but there are a couple of cases where this hurts:

When you change the brightness or colour (temperature) using a longer transition time.  E.g. when you set the brightness from 100% to 10% over a period of, say, 10 minutes.  You set the slider from 100% to 10%.  But when the light state is refreshed by the gateway/bridge, it "jumps" back to 95%.  The slider will continue to move down to 10% over the next 10 minutes, "jumping" each time the light state is refreshed by the gateway/bridge;
When the target state and actual state differ noticeable.  This typically happens because Zigbee lights use a different colour space than your PC or Phone.  The target colour needs to be translated from RGB Hue / Saturation to CIE 1931 xy; the actual colour from xy back to Hue / Saturation.  The math behind this is no fun, and imperfect (as in: doesn't always return the exact original values).  Usually the differences are too small to notice, but when the requested colour cannot be displayed by the light you'll see the colour "jump".  This will happen more often on sub-par lights which support fewer colours, and don't give a clue what colours they support.

Now, if you would control your lights exclusively by HomeKit, we could decide to ignore the actual light state, and simply only show the target light state in HomeKit.  There might be a mismatch between the displayed colour (temperature) in HomeKit vs the actual colour (temperature) by the light, but you probably wouldn't notice nor mind.  However, when the light would be controlled from elsewhere (using Zigbee controllers, using gateway/bridge rules, using other API clients, like the Hue or Phoscon app), HomeKit wouldn't be updated.

Now, if you would control your lights exclusively by HomeKit, we could decide to ignore the actual light state, and simply only show the target light state in HomeKit. There might be a mismatch between the displayed colour (temperature) in HomeKit vs the actual colour (temperature) by the light, but you probably wouldn't notice nor mind. However, when the light would be controlled from elsewhere (using Zigbee controllers, using gateway/bridge rules, using other API clients, like the Hue or Phoscon app), HomeKit wouldn't be updated.

Thanks for the thoughtful and detailed explanation. For my RGB lamps in particular, I have no other control than through HomeKit, so I don’t mind at all ignoring the actual light state (save for on/off). I can try to hack something together to get this to work the way I imagine and report back my experience.


Revert environment setup via local_env.yml

This was accidentally deleted in https://github.com/alphagov/content-performance-manager/pull/154

Merging as this is a revert of some code that should've not been deleted.
cc/ @tuzz @cbaines @chao-xian
Let me know if you need to clarify anything, please


chore: update to 4337 v0.7 (1/n)

Changes:

updated eth-infinitism/account-abstraction to v0.8
Changed all instances of UserOperation to PackedUserOperation

converting to draft till stack is ready


Add "unsupported browser" page

Description
Implement filter that redirects users with old browsers (IE9-11) to a page which tells them to use one of the modern browsers. Here is how it looks:

Release notes:
Users with legacy browsers such as IE11 and below are now shown a  page asking them to use one of the modern browsers (Edge, Firefox, Safari, Chrome).
Checklist

[x] Created tests which fail without the change (if possible)

Issue(s)
Fixes #512

I didn't use translations for the text on the page. Wasn't sure if I should given that this page is shown before user has a chance to choose language themselves. Do we rely on the detected language coming from Accept-Language header?

More of a tangent, but when I can find who can give me access to the Seattle tag manager data I'll provide some stats on current usage.

@gwendolyngoetz updated safari logo
What's Seattle tag manager? Is it google analytics or something like that?
I think we'll also be able to see useragents from server logs. Though those will require some parsing.


ISSUE-2428 refactor: improve test for EnvsClustersTenantsControllerService

Resolves: ISSUE-2428
https://github.com/Aiven-Open/klaw/issues/2428
What kind of change does this PR introduce?

[ ] Bug fix
[ ] New feature
[x] Refactor
[ ] Docs update
[ ] CI update

What is the current behavior?
Test coverage of EnvsClustersTenantsControllerService class is only 30%
What is the new behavior?
Line coverage of EnvsClustersTenantsControllerService class is increased to 87%

Other information
Refactor some of the code in EnvsClustersTenantsControllerService class
Requirements (all must be checked before review)

[x] The pull request title follows our guidelines
[x] Tests for the changes have been added (if relevant)
[x] The latest changes from the main branch have been pulled
[x] pnpm lint has been run successfully

@luke-zhou Thanks a million for your contribution 🎉 , I am a little busy at the moment but I will try to start the review this evening or tomorrow morning.

Hey @luke-zhou I'm almost finished testing this PR I am sorry for the delay, I've set aside tomorrow to finish it off. Sorry again for the delay!

Thanks, committed suggestion @muralibasani

Thanks, committed suggestion @muralibasani

Hey @luke-zhou I just noticed the last commit I think needs to be signed, would you be able to sign off your last commit?
git commit --amend --signoff

Thanks, committed suggestion @muralibasani

Hey @luke-zhou I just noticed the last commit I think needs to be signed, would you be able to sign off your last commit? git commit --amend --signoff

Hi @aindriu-aiven I have signed the last commit, and pushed. However I am having issues to sign the following two commits, which i made directly on the PR page:

Thanks, committed suggestion @muralibasani

Hey @luke-zhou I just noticed the last commit I think needs to be signed, would you be able to sign off your last commit? git commit --amend --signoff

Hi @aindriu-aiven I have signed the last commit, and pushed. However I am having issues to sign the following two commits, which i made directly on the PR page:

Ah I've seen this before let me check if there is a way to do this and I'll let you know asap. Thanks for checking for me!

Thanks, committed suggestion @muralibasani

Hey @luke-zhou I just noticed the last commit I think needs to be signed, would you be able to sign off your last commit? git commit --amend --signoff

Hi @aindriu-aiven I have signed the last commit, and pushed. However I am having issues to sign the following two commits, which i made directly on the PR page:

Hey I had a chat with some colleagues in this space and apparently we can use
git rebase --signoff <your commit>
This will sign off all the commits you made until HEAD. If you don't mind giving that a go it should let us pass the DCO and merge the PR.
Thanks a million!

Thanks, committed suggestion @muralibasani

Hey @luke-zhou I just noticed the last commit I think needs to be signed, would you be able to sign off your last commit? git commit --amend --signoff

Hi @aindriu-aiven I have signed the last commit, and pushed. However I am having issues to sign the following two commits, which i made directly on the PR page:

Hey I had a chat with some colleagues in this space and apparently we can use git rebase --signoff <your commit>
This will sign off all the commits you made until HEAD. If you don't mind giving that a go it should let us pass the DCO and merge the PR.
Thanks a million!

Hey @luke-zhou just wanted to see if you think you would be able to give the above command a try to sign off those commits, if it doesn't work just let us know and we can see if we can work around it.
Thanks!


Error with windows build

I'm trying to build a windows version (.exe) for my project on linux using CGO_ENABLED=1 CC=x86_64-w64-mingw32-gcc CXX=x86_64-w64-mingw32-g++ GOOS=windows GOARCH=amd64 go build -ldflags -H=windowsgui -o ./builds/jacp-windows-raw.exe but it shows this error:
../../go/pkg/mod/github.com/jpillora/overseer@v1.1.6/proc_slave_windows.go:12:2: missing go.sum entry for module providing package github.com/StackExchange/wmi (imported by github.com/jpillora/overseer); to add:

@jpillora

It looks like it is because this -> https://github.com/StackExchange/wmi doesn't have a go.sum file

and it is imported in https://github.com/jpillora/overseer/blob/master/proc_slave_windows.go

Ah interesting, can you run go mod tidy and send a PR?


Feature: navigation based on history for sub tabs

Fixes #607
Updated the HomeFave view:

When changing tabs, uses router.push instead of router.replace to ensure that the tab navigation history is saved internally by vue-router
Add listener when user presses back button (onpopstate) which ensures the tabs are loaded
a. And also cleanup listener when leaving page

Thank you for the PR as always
🙏


Decoration of TextSpan after WidgetSpan with no height is shifted to the left

Steps to Reproduce
Execute flutter run on the code sample below.
Expected results:
All TextStyle attributes are applied exactly to "Hello".
Actual results:

The text colour and size are correctly applied, but the underline and the yellow background colour are shifted to the left, apparently by the width of the WidgetSpan.
It appears that this doesn't happen if the widget in WidgetSpan has some height.
Flutter has no support for first line indentation yet, as far as I know, so some users may try using WidgetSpan for that purpose and encounter this issue.

Code sample
import 'package:flutter/material.dart';

void main() => runApp(const App());

class App extends StatelessWidget {
  const App({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: Scaffold(
        body: Text.rich(
          TextSpan(
            children: [
              WidgetSpan(
                child: SizedBox(width: 80.0),
              ),
              TextSpan(
                text: 'Hello',
                style: TextStyle(
                  color: Colors.green,
                  fontSize: 60.0,
                  backgroundColor: Colors.yellow,
                  decoration: TextDecoration.underline,
                ),
              ),
              TextSpan(
                text: ' world',
                style: TextStyle(fontSize: 30.0),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Logs
flutter analyze
Analyzing bug...
No issues found! (ran in 1.4s)

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.0, on Microsoft Windows [Version 10.0.22000.675], locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✗] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.1.3)
[!] Android Studio (not installed)
[✓] IntelliJ IDEA Ultimate Edition (version 2022.1)
[✓] VS Code (version 1.67.1)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

Hi @kaboc. Thanks for filing the issue. This issue is reproducible on latest stable and master channels with provided sample code.
The issue will not reproduce if the WidgetSpan has height.

Web (HTML): ✅
Web (CanvasKit): ❌
iOS, Android: ❌
MacOS: ❌

✅: No Issue
❌: Issue reproduced

Demo

HTML
Canvas
iOS
MacOS

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.0, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.0.0 at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (2 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7042882/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Android Studio
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/203.7185775/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.67.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (3 available)
    • iPhone 13 (mobile) • 5F5D609C-3B0B-4C48-872C-961928157361 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)    • macos                                • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 101.0.4951.54

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.794, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.1.0-0.0.pre.794 at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80a51e4f97 (13 minutes ago), 2022-05-18 20:07:12 -0700
    • Engine revision 54b096da96
    • Dart version 2.18.0 (build 2.18.0-130.0.dev)
    • DevTools version 2.13.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7042882/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[!] Android Studio
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/203.7185775/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.67.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (3 available)
    • iPhone (mobile) • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 14.8.1 18H107
    • macOS (desktop) • macos                                    • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome                                   • web-javascript • Google Chrome 101.0.4951.64

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Submitted a fix to Skia: https://skia-review.googlesource.com/c/skia/+/541790

I think is is related
WidgetSpan inside TextSpan is not getting parent styles. But if widgetspan has a sibling textSpan it does. weird.
https://dartpad.dev/?id=2df15c334b139ff3054f010be4a88260

https://github.com/flutter/flutter/issues/104077#issuecomment-1133771312 has a different root cause - SkParagraph was not painting backgrounds and other style decorations for lines that consist entirely of WidgetSpans.
PR to fix this in Skia: https://skia-review.googlesource.com/c/skia/+/543758

The fix is available on the Flutter master branch.
Or you can work around it by switching to the old text layout library (see https://github.com/flutter/flutter/issues/103846#issuecomment-1136334771)

It looks like the fix has arrived in the stable channel, so I think this issue can be closed now. Is there anything still missing or pending?
Thanks for the fix, @jason-simmons! It was very quick.

Closing based on the comments


missing class in generated code?

That schema contains two definitions of the class SearchFeaturesResponse.  How are these handled in the generated code in _protocol_definitions.py?   (Currently I only see one)
The schema project the current server is based on is  version = '0.6.be171b00'
Snippets from this commit follow

One in the file genotypephenotypemethods.avdl,  protocol GenotypePhenotypeMethods

/** This is the response from `POST /genotypephenotype/search` expressed as JSON. */
record SearchFeaturesResponse {
  /**
  The list of matching FeaturePhenotypeAssociation.
  */
  array<org.ga4gh.models.FeaturePhenotypeAssociation> associations = [];

  ...

The second one is found in sequenceAnnotationmethods.avdl

  /** This is the response from `POST /features/search` expressed as JSON. */
  record SearchFeaturesResponse {
    /**
    The list of matching annotations, sorted by start position. Annotations which
    share a start position are returned in a deterministic order.
    */
    array<org.ga4gh.models.Feature> features = [];

    ...

The generated code only has the class associated with sequenceAnnotationmethods.avdl

    def __init__(self):
        self.features = []
        self.nextPageToken = None

This is surely a bug in the schemas, isn't it? We can't have two classes with the same name in the same namespace. I think this is also gone from the latest version of the schemas in master, as these features have been moved into the alpha topic branches.

Both sequenceAnnotationmethods.avdl and genotypephenotypemethods.avdl share the same namespace @namespace("org.ga4gh.methods") each file defines an enclosing protocol.
In the names section of the spec
A name only is specified, i.e., a name that contains no dots. In this case the namespace is taken from the most tightly enclosing schema or protocol. For example, if "name": "X" is specified, and this occurs within a field of the record definition of org.foo.Y, then the fullname is org.foo.X. If there is no enclosing namespace then the null namespace is used.

I'm assuming that the schemas pass validation...
A schema or protocol may not contain multiple definitions of a fullname. Further, a name must be defined before it is used ("before" in the depth-first, left-to-right traversal of the JSON parse tree, where the types attribute of a protocol is always deemed to come "before" the messages attribute.)

Full disclosure, I haven't had the chance to walk through process_schemas.py.  Does it consider the protocol {} element?

I'm not sure that process_schemas can do anything here @bwalsh --- we'll just have to change the names of one or both of these classes in the upstream schemas. This is a schema bug. I wouldn't read too much in to the schemas passing validation: the avro tools are quite weak for this sort of thing.

Agreed.  It having repeated class names is confusing.
-Brian Walsh
On Thu, Aug 6, 2015 at 12:11 PM, Jerome Kelleher<EMAIL_ADDRESS>wrote:

I'm not sure that process_schemas can do anything here @bwalsh
https://github.com/bwalsh --- we'll just have to change the names of
one or both of these classes in the upstream schemas. This is a schema bug.
I wouldn't read too much in to the schemas passing validation: the avro
tools are quite weak for this sort of thing.
—
Reply to this email directly or view it on GitHub
https://github.com/ga4gh/server/issues/548#issuecomment-128477196.

OK, good. I'm going to close this issue in that case.


Update the nodejs slave image to version 8

Hey all,
Raising this pull request if it is feasible to update the nodejs slave image to version 8
If there is something blocker or breaking in this, feel free to advise as I am not aware of that.
It will be good if something like -> for current version ( node 4) -> image `openshift/jenkins-slave-nodejs-centos7
and after update, node 8 -> image openshift/jenkins-slave-nodejs8-centos7 or may be vice versa
cc @gabemontero @sthaha @hrishin

Thanks, but we already have one, see:
https://hub.docker.com/r/openshift/jenkins-agent-nodejs-8-centos7/
https://github.com/openshift/jenkins/tree/master/agent-nodejs-8

Thanks a lot @bparees


Examples in tutorial do not complete successfully

e.g. first example
random Real strength(Person p) ~ Gaussian(10, 2);
random Boolean lazy(Person p, Match m)
 ~ BooleanDistrib(0.1);
random Real pulling_power(Person p, Match m) ~
 if lazy(p, m) then strength(p) / 2.0
 else strength(p);

gives output
Using fixed random seed for repeatability.
tst.blog::1.22: Type Person undefined!
tst.blog::2.21: Type Person undefined!
tst.blog::2.31: Type Match undefined!
tst.blog::4.27: Type Person undefined!
tst.blog::4.37: Type Match undefined!
tst.blog::1.22: Type Person undefined!
tst.blog::2.21: Type Person undefined!
tst.blog::2.31: Type Match undefined!
tst.blog::4.27: Type Person undefined!
tst.blog::4.37: Type Match undefined!

Types must be explicitly defined, e.g.
type Match;
type Person;

in this example.
Obviously, this needs adding to the tutorial.

There are further issues with the first example: adding the lines from the next few pages
random Boolean team1win(Match m) ~
 sum([pulling_power(p, m) for Person p in team1(m)])
 > sum([pulling_power(p, m) for Person p in team2(m)]) ;

obs team1(M0) = {James, David};
obs team2(M0) = {Brian, John};
obs team1win(M0) = true;
obs team1(M1) = {James, David};
obs team2(M1) = {Bob, Andrew};

query strength(James) > strength(Brian);

returns
Using fixed random seed for repeatability.
tst.blog::12.40: syntax error at Symbol: in (tst.blog:12/40(320) - tst.blog:12/42(322))
tst.blog::12.6: error in body of function call
tst.blog::13.42: syntax error at Symbol: in (tst.blog:13/42(375) - tst.blog:13/44(377))
tst.blog::13.8: error in body of function call

Hi clbarnes, the slides use a simplified syntax for demonstration purpose during the course. It just shows how generally a BLOG program looks like.
For the full version, you can check out
https://github.com/BayesianLogic/blog/blob/master/example/tugwar.blog
I will update the slides to emphasize this.

Right. What is the recommended trajectory for someone learning the language from scratch? The user manual is clearly intended as a manual for usage of the blog binary, rather than the language (and even then, isn't as concise or rich as a man page), and the 'comprehensive tutorial about BLOG motivation, syntax, semantics, and examples' contains non-functional examples, only explaining them 50 pages later. The language reference is a single PDF. I appreciate the variety of examples in the examples directory but it's not like they have rich comments to make them approachable to a beginner, or a clear reading order.


error

Description of the modifications
AttributeError: 'FigureCanvasQTAgg' object has no attribute 'set_window_title'. Did you mean: 'setWindowTitle'?  when I run the example code, I get the problem. I don't know if it's because of the version， so I hope to get the answer, thanks!
Useful links and references
No response

I have tried to install  0.4 、0.5  to solve the question, but it is unuseful.

I have experienced the same issue.
Python 3.10.5 [MSC v.1929 64 bit (AMD64)] on win32
ansys-fluent-visualization 0.6.dev0
matplotlib 3.6.1
I think this can be solved by rolling back matplotlib 3.5.3 or earlier. (Not matplotlib >=3.6.0)
OR
edit the source to reflect changes in matplotlib
[https://matplotlib.org/stable/api/prev_api_changes/api_changes_3.6.0.html#:~:text=get_figure_manager%3B use-,frame.canvas.manager,-instead]


Feedback Milestone 2

Hello Group 24,
My name is Pradeep and I will be giving you some feedback on your project.
Overall, the project is a fantastic start and there are some suggestions I have that will greatly improve the quality of your project
I have divided up the feedback into several sections so it is easier for you to implement.
Task 1

Well Done

Task 2

Well Done
Add few comments briefly describing each step

Task 3

Well Done

Task 4

Good Work
Expand on your explanation. Try describing how your findings are addressing the research question. Also, add which plots were primarily responsible in making you reach such a conclusion.

Overall, well done.

Thank you very much for the feedback, Pradeep.


⚠ ClamAV has degraded performance

In e4292c9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 167 ms

Resolved: ClamAV performance has improved in 28deed0 after 18 minutes.


OpenNMT CTranslate2 Support?

Hi All!
Really interested in MLServer, it looks like a really neat way to consistently serve ML models of varying frameworks.
I've gone through the documentation and I have 2 questions:

Any plans to support OpenNMT CTranslate2 inference engine? It's fairly popular for transformer models.
In all the documentation I see a standard set of API endpoints. Is there a way to add a custom endpoint?

Cheers!

Hey @cccs-km ,
Thanks for your interest in MLServer!
To answer your questions,

No plans at the moment - we are always open to external contributions though!
Not at the moment, we follow closely the spec outlined by the Open Inference Protocol and we see that the best value comes from standardising on that set of endpoints.

@andwilson mlops-world 2023


Trigger happy autocomplete

It seems that autocomplete is a bit too trigger happy. The issue with this, besides being annoying, is that the undo history stack is not reachable anymore. The reason for this is that after editing a \cite command, undo will move the cursor back into the brackets, triggering the autocomplete pane.
Additionally, on my system (OS X El Capitan, Atom 1.7.0-beta0, LaTeXTools 0.8.0), autocomplete does not return keyboard focus to the editor window, which is mildly annoying.
Screen recording:

+1 for this

I had a similar issue and it was because I had also installed latex-friend which has a similar feature. It seems that there is a conflict between the two. I uninstalled latex-friend and everything works perfectly now.

The trigger-happy behavior is in part due to some internal atom limitations, though it can be ameliorated.
As for focus not returning to the editor, I am seeing this too. @ig0774 any ideas? It's not consistent, which makes this harder to debug.

On the focus issue, I think this should've been resolved by #78, but if you're seeing it with those changes, I can try to dig a bit more.

Actually, no. I've pulled #78 into master and have been using it for the past few days (yay dogfooding!). It is very weird. What I can see is this:
(1) if you disable auto-triggering, then there is never a problem with the keybinding (C-l,x).
(2) if you enable auto-triggering, the very first time you type e.g. \ref{, you lose focus. Subsequent invocations seem to be a bit random---with \ref{ things seem to work if you actually select a label, though you may lose focus if you "esc out"; with \cite, I seem to lose focus more often
(3) there is also some weird interaction with the undo stack, as @tomrijndorp reported. If I enter a reference and CMD-z my way back to the original state of the file, I still see a blue dot in the corresponding tab. This btw happens both with and without auto-triggering.

One possibility I can think of is that we may not actually be storing a reference to the TextEditor, since its actually storing document.activeElement. Could you try changing line 233 of ltselect-list-view.coffee to this:
    @previouslyFocusedElement = atom.views.getView(atom.workspace.getActiveTextEditor())

That should ensure we try to reset focus on the text editor. If that works, we could pass the triggering text editor via start(), which may result in better behaviour.

@ig0774 Why not just to use the autocomplete+ for this?

@ig0774 yes, that seems to work!
It's a bit weird, because the problem happens in 0.8.0 as well (per @tomrijndorp), where we still use space-pen-views... anyway, that seems to fix the problem, so that's great. Would you be able to whip up a PR for this?
@ivlis autocomplete+ is not smart enough for our purposes. More precisely: it may be OK for references, but with bibliographies, I think showing more info is better---but that requires a panel, rather than a pop-up.

Well, #78 was an attempt to be a really careful port of the space-pen stuff. I didn't want to introduce new behaviour (after the pain of the last ST release). I think the space-pen stuff had that behaviour because they couldn't always rely on there being an active text editor (e.g. with the command palette).

I just pulled #88 into master, and it does solve the focus issue. Thanks @ig0774 !
I've also pushed a small change that should fix the trigger-happy autocomplete issue for citation commands. Hopefully I'll be able to tag a new release this afternoon.


How to close the SFTP session?

When using russh-sftp with a Windows remote, I observe that sftp-server.exe processes are seemingly "leaking" until my program exits.
This is probably due to me dynamically creating new sftp clients and then simply dropping them, which apparently is not sufficient to close the connection.
As client::SftpSession::new() consumes the channel stream however, I'm not sure how to close it.
I see a close() method in RawSftpSession, but would prefer being able to use the high-level abstraction.

Task Manager

Method close() in RawSftpSession closes the handle, not the session. I opened a PR #13  which should fix this. Please reproduce this again and give feedback

I can confirm that the linked PR fixes the issue with both my reproducer as well as my actual application.
Thank you for the very quick fix!

All fixed published in 2.0.0-beta.4. Thank you, I didn’t think I’d miss such a basic thing.


Add files via upload

My program takes any image and converts it into something blot can draw. This is done by representing each pixel as a number of dots, where darker pixels get more dots. The dots are randomly distributed within the pixel. I also wrote some python code to process PNGs into an input for this program. This can take a while to draw though. I have set the current code to draw the HC logo which will take about 30 minutes, but below is shown an image that i made over the course of a day.

Would it be possible to use the photo in the description for the vote in the competition? I wouldn't expect you to draw it because it took like 8 hours but it shows what blot is capable of with a bit of time! if not the hack club logo should draw in a reasonable time, please be patient though!!
-thanks, euan

Hey @EuanRipper
Amazing project! Could try try making it a bit faster? You could try decreasing the resolution its doing (instead of 10 dots per, 5 dots etc)

Hi! I have made it faster, now it draws less dots but connects some of them to maintain the same density of ink!

Hey! Super cool idea! Currently we are looking for pieces with 15-20 hours of effort, and it looks like this could use some more! I could recommend drawing out the photo using flow fields, or some other sort of randomly generated pattern. Please create a new PR once you have updated your art! We are doing this due to new management of the PR system.


ExoPlayer library version should be updated

RtspExtractor embeds a very old version of ExoPlayer. For example, that version still has the com.google.android.exoplayer.source package, which was deleted in April, 2015 (see Issue 1).
The ExoPlay library should be updated to a more recent version.

I've pushed an update to use the latest ExoPlayer version.
It's under the branch exoplayer-update
I want to test it in my demo module but for some reason, I can't reference classes from the library module even with the right compile options in the demo build.gradle.
gotta check that out.

Hello. Will you support new ExoPlayer?

Hey, no I won't in the near future. I am no currently working on video
streaming and am devoting my free time to other projects.
Pull requests welcome :)
--
Arnaud Courbiere
On Tue, Jun 28, 2016 at 9:00 AM, istatkevich<EMAIL_ADDRESS>wrote:

Hello. Will you support new ExoPlayer?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ArnaudCourbiere/RtspExtractor/issues/2#issuecomment-229095737,
or mute the thread
https://github.com/notifications/unsubscribe/ABYqpsTN_7UWeJl1M-x53TpVDWPMEIMvks5qQUUBgaJpZM4GfCpK
.


use tls for nats connections

Add config option to skip server certificate verification. This is needed when running a nats server with self signed certificates.

There is only a changelog missing and a CI run.


Left rotate by d

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
A left rotation operation on an array shifts each of the array's elements unit to the left. For example, if 2 left rotations are performed on array 1,2,3,4,5, then the array would become 3,4,5,1,2.
Given an array arr of integers n and a number d, perform d left rotations on the array. Return the updated array to be printed as a single line of space-separated integers.
Checklist

[✔] I've read the contribution guidelines.
[✔] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
I have added 3 approaches.

Brute force
Extra space
No extra space

The branch is showing conflicts. Try to resolve it from your end.
Conflicting files
Arrays/move_zeroes_end_1traversal.cpp
Arrays/move_zeroes_end_2traversal.cpp


No market price for osmo on cosmos

Overview
When looking up Osmo on cosmus hub, the market data is missing so it has a zero value
References and additional details

Log in app.shapeshift.com and connect to wallet of choice
naviagate to 'Wallet' and select "Cosmo"
Note that an osmo on cosmos has a $0.00 value.

Acceptance Criteria
Market value should be available for osmo on cosmos
Need By Date
No response
Screenshots/Mockups
No response
Ownership

[X] If my bounty needs engineering or needs product I have added the respective labels on the right
[X] As the sponsor of this bounty I will review the changes in a preview environment (ops/product) or review the PR (engineering)

Estimated effort
No response
Sponsor / Stakeholder
No response
Bounty Hunters

Join our discord
Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
Please refer to this link for some basic info
Please do not start work on this issue until you are approved in Gitcoin.

wontfix


Revamped Hackpad - The Cost Effective

Fixed the cad files (use the .stl respectively), the previous pr was accidentally merged. 😭
Here's the non forked repo btw: https://github.com/snaeker58/Karls-Hackpad 👀

Looks good! I would put a switch frame on it, but not mandatory. @Hyatt B.


Module not found: Error: Can't resolve 'axios'

Describe the bug
Using sn-map in Angular v13 app I got a compile error
To Reproduce
Install and reference sn-map in your Angular project, then compile with ng serve.
Expected behavior
This is the errore returned by the compiler:
./node_modules/@nebula.js/sn-map/dist/sn-map.js:23765:14-30 - Error: Module not found: Error: Can't resolve 'axios' in<EMAIL_ADDRESS>Additional context
<EMAIL_ADDRESS>"^3.1.3"
<EMAIL_ADDRESS>"^0.9.0"
TS code
import map from<EMAIL_ADDRESS>...
{
name: 'map',
load: () => Promise.resolve(map)
},

Any solution for this one ?


Secondary HEC output with SELECT mode does not work

Hi,
As talked on slack, here is the issue:
I have set up a secondary HEC destination output in the env file:
SC4S_DEST_SPLUNK_HEC_SPLUNKCLOUDPREPROD_URL=https://http-inputs.mystack-pp.splunkcloud.com
SC4S_DEST_SPLUNK_HEC_SPLUNKCLOUDPREPROD_TOKEN=xxxx
SC4S_DEST_SPLUNK_HEC_SPLUNKCLOUDPREPROD_TLS_VERIFY=yes
SC4S_DEST_SPLUNK_HEC_SPLUNKCLOUDPREPROD_MODE=SELECT

And added a filter to describe the "SELECT" mode in a parser:
application sc4s-lp-spectrum_alert_dest_fmt_SPLUNKCLOUDPREPROD[sc4s-lp-dest-select-d_fmt_hec_SPLUNKCLOUDPREPROD] {
    filter {
        tags(".source.s_SPECTRUM_ALERT_UTC_TZ");
    };
};

The filter here uses the tag referencing the listener which I set up also in the env file:
SC4S_LISTEN_SPECTRUM_ALERT_UTC_TZ_TCP_PORT=10000
This configuration does not work, I receive no data on this secondary HEC output.
When switching to "global" instead of "select", it works.

Hi, I tested the scenario and it works , we found a gap in doc and we will fix it
SC4S_DEST_SPLUNK_HEC_OTHER_URL=xxxx
SC4S_DEST_SPLUNK_HEC_OTHER_TOKEN=xxxx
SC4S_DEST_SPLUNK_HEC_OTHER_TLS_VERIFY=no
SC4S_DEST_SPLUNK_HEC_OTHER_MODE=SELECT

Created file in /opt/sc4s/local/config/app_parsers/selectors/<filename>.conf
application sc4s-lp-cisco_ios_dest_fmt_other[sc4s-lp-dest-select-d_hec_fmt_other] {
    filter {
        'cisco' eq "${fields.sc4s_vendor}"
    };
};

And it worked , the name of the topic should be in lower case "other" not "OTHER"

:tada: This issue has been resolved in version 2.45.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update modmenu version

No idea if this will fix it being incompatible, but I tried.

What should this fix?

I hope it fixes this.

Then it's the wrong place to change that.

What do you mean? 3.1.0 is the latest version of Mod Menu, and ironically it's updatable through Mod Manager. If this isn't the right file to change, feel free to close this PR and fix it yourself.

It's the right file but if someone uses 3.0.0 and hasn't updated yet it will throw the same error

Fixed with 1.2.3+1.18


inlineSuggestionVisible context key is stuck on

I don't have perfect repro steps but I've seen this several times today.

I edit some code and accept an inline suggestion with tab
Maybe I type something else at the same cursor location
Then I press cmd+right arrow to move the cursor to the end of the line but nothing happens. There is no inline suggestion visible at this point
The log shows that acceptNextWord is trying to run here

2023-05-11 20:03:54.732 [info] [KeybindingService]: | Resolving meta+[ArrowRight]
2023-05-11 20:03:54.732 [info] [KeybindingService]: \ From 5 keybinding entries, matched editor.action.inlineSuggest.acceptNextWord, when: inlineSuggestionVisible && !editorReadonly, source: built-in.
2023-05-11 20:03:54.732 [trace] KeybindingService#dispatch ⌘→ [ Will dispatch command editor.action.inlineSuggest.acceptNextWord ]
2023-05-11 20:03:54.732 [info] [KeybindingService]: + Invoking command editor.action.inlineSuggest.acceptNextWord.
2023-05-11 20:03:54.732 [trace] CommandService#executeCommand editor.action.inlineSuggest.acceptNextWord

I cannot reproduce. Do you have reproducible steps?

Not perfect steps but I actually see this pretty often

/duplicate #182249

That is this issue @hediet

Sorry, duplicate of #183909


[homepage-redesign] create new staging for redesign branch

[x] create new staging for redesign branch

https://participedia-homepage-2020.herokuapp.com/


[UNDERTOW-1281] cherry-pick of changes from master branch

If merged, we should update 'Fix versions' field in https://issues.jboss.org/browse/UNDERTOW-1281.

Has there been a specific request for this? I am only merging into 1.3.x if it is needed in EAP 7.0


TWEAK: do not set global log level in NewBinlogSyncer

This reverts https://github.com/siddontang/go-mysql/commit/0baad4d700af91f9031f5b4e3135f4eb53ed35ce.
Setting a global log level in an initializer causes unwanted side effects in many cases. User who needs this should do log.SetLevelByString in main program only.

@siddontang @b13f

Another option will be having an user-provided logger. Since go community does not currently have a consensus on logger interface, this might not be wanted.

Hi @taylorchu
I remember that some users meet the log info problem, so they want to control through configuration.
Maybe we can try:
if cfg.LogLevel != "" {
 	log.SetLevelByString(cfg.LogLevel)
 }

So if the user set a log level config, he/she can use it.

If that is the problem, then setting log level there is not a good solution. Imagine that you create NewBinlogSyncer with "info" and "error" log level. You will expect that one syncer has log level of info, and the other has log level of error. However, the code we have now will only honor the BinlogSyncer of the latter call.
This gets worse if you use  NewBinlogSyncer with goroutines. The logger not only a global state, but that state is controlled by a nondeterministic function call order.

Yes @taylorchu
IMO, I don't like go-mysql contains any log. But the missing log is hard for us to locate problems sometimes.  Maybe a log interface is better, but Go doesn't supply a standard way sadly.
What is your opinion?

type StdLogger interface {
    Print(...interface{})
    Printf(string, ...interface{})
}

Many go logger libraries support this interface, and that includes ngaut/log and logrus.
By default the package's logger uses stdlib logger, and it should be settable.

Good idea, but I want to use different log level, maybe I can define a log interface like https://github.com/grpc/grpc-go/blob/master/grpclog/loggerv2.go#L30.
If the user doesn't pass a logger to go-mysql, we will use a NoopLogger which does nothing by default.

Is there a use case to have different log level within this package github.com/siddontang/go-mysql?
For our usage, we really don't care. Every log line here is equivalent to  "DEBUG" level.

maybe needed. I will merge this PR now, thanks, @taylorchu.
Maybe it is better to support passing a Logger, I will do it later.


Split storage.sw into modules

Motivation
The file has multiple types and it's starting to become large.
We should split the file into modules and reexport to maintain functionality but compartmentalise the code.

It would be nice to do this after https://github.com/FuelLabs/sway/issues/3487 is closed so that importing items from the storage sub-libraries is not too verbose (assuming we're splitting storage.sw into submodules inside a storage folder).


Only stats are being logged

Hey, when I do what documentation says :

run an ubuntu container docker run -t -i ubuntu:14.04 /bin/bash
docker run -v /var/run/docker.sock:/var/run/docker.sock logentries/docker-logentries -t <token> -j -a host=uname -n
echo "some log entry from ubuntu container"

then docker logs ubuntu_container yeilds the echos but logentries Live Tail only yields the docker stats, but no entries at all, no matter what I do I cannot make it log any docker logs ... If I use --no-stats flag then nothing is logged at all
Docker version 1.7.1, build 786b29d

Imho the reason is this :
https://github.com/mcollina/docker-loghose/issues/2

I made some fixes and also added some new feature(s) one week ago: -l  -k  -e
With each of the params you can log a different type of info, -t logs all of them.
Tested with the sample app, cli and also docker. I think it should work also for your use case.

Thank you, I'll test it by the end of the next week and let you know

@mihaidma I finally see them nasty logs, thanks !


Ingestion for S3 bucket containers

Describe your changes :

Initial support for S3 bucket containers. This includes:
Support for ingesting containers where a metadata template file exists in the root path of the bucket, containing information about the path and structure of files which should be considered. Support includes csv, tsv, json, parquet and avro (the same as the datalake connector). Sample metadata file:
{     "entries":[         {             "dataPath": "transactions",             "isStructured": true,             "structureFormat": "csv",             "isPartitioned": false         }     ] }
Support for ingesting a bucket without any structure defined in a metadata file - this is restricted to creating a container entity without any data model and fileTypes, but with details about total nr of files and total size in bytes of all the files(for files in the STANDARD storage type class only).
Also, changed numberOfObjects and size schema types for a container to Number (and consequently to float in python) to support larger values.

Type of change :

[x] New feature

Reviewers
@pmbrull

@cristiancalugaru If I am understanding this correctly, a user will upload a metadata template file to any s3 that they want to ingest into the openmetadata platform?  Is there a reason this design implementation was chosen?  As as user I find this more cumbersome to upload an additional file to s3 to tag which buckets/paths need to be ingested.  Why not configure the s3 connector to accept the bucket and path to ingest from?


Video Element

Hi team,
I have added Video element in my setup.
It is working as expected. But when I clear canvas or delete it by selection or using eraser,
I have set isDeleted : true
and remove iframe tag using
var video_el = document.getElementById(el.id);
video_el.parentNode.removeChild(video_el);
This is working locally. But when we are collaborating, It is not being removed from other clients.
Can you please help me.

Hi. From the code it seems you're managing the video elements separately, so once you delete the element, only the element update itself is propagated to the other clients. They'll need to take care of removing the video from the DOM. Is my assessment correct?
(unrelated: you're deleting all elements on the canvas, not just target element, in your .map()).
Btw, would love to see how it works in practice if your app is public anywhere. I assume the video elements are on top of the canvas, so you won't be able to interleave it with other elements on canvas.

Hi David,
I am adding iframe tags for every video element. But I am managing video
elements like you have managed other elements like text and image.
When I add video to one client it is rendering over canvas inside a div
with position absolute and I am considering x and y as left offset and top
offset.
I also added code to remove iframe in *actionClearCanvas *and

deleteSelectedElements.* If you can help me to find a function which is
being called on another client when we clear canvas that will be great.

But now I am facing typescript *Object is possibly 'null' *issue suddenly
and now the project is not being compiled.
Below is a link to a demo.
https://sleepy-bassi-dcb209.netlify.app/
It is not updated.
I will update you once I am able to deploy the latest changes here.
I have added Eraser, Stickynote, Grab and multiple images upload and pdf
upload.
On Wed, Apr 6, 2022 at 5:26 PM David Luzar @.***> wrote:

Hi. From the code it seems you're managing the video elements separately,
so once you delete the element, only the element update itself is
propagated to the other clients. They'll need to take care of removing the
video from the DOM. Is my assessment correct?
(unrelated: you're deleting all elements on the canvas, not just target
element, in your .map()).
Btw, would love to see how it works in practice if your app is public
anywhere. I assume the video elements are on top of the canvas, so you
won't be able to interleave it with other elements on canvas.
—
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/excalidraw/issues/5019#issuecomment-1090182992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYUWOOIUN4O2NBOUQPYNCLVDV3XHANCNFSM5SVRDGGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

can you share code.
i want test video local


Add Calculate method to CommitmentProof

Added a calculate check on TestVectors that checks each proof calculates up to the expected root.
I couldn't add an equivalent TestBatchVectors since the proofs that are being batched together don't commit to the same root.

@AdityaSripal merged this and tagged it as v0.6.1 if you want to update the imports.


Add weave-kube DaemonSet that can be used with k8s version v1.6

@bboreham @errordeveloper @lukemarsden
ref: #2777

I ran a test to check what happens if you load this file on k8s 1.5:
unable to decode "STDIN": no kind "ClusterRole" is registered for version "rbac.authorization.k8s.io/v1beta1"
unable to decode "STDIN": no kind "ClusterRoleBinding" is registered for version "rbac.authorization.k8s.io/v1beta1"

So we need to figure out how best to help our users.  #2754, perhaps.

@bboreham Totally known issue, this is what's hard with providing manifests; they often have to track the release branches of k8s
In this case; Taints and Tolerations was alpha in v1.5; now beta. The same goes for RBAC, hence the above error. RBAC beta did obviously not exist in v1.5.
We have to provide different manifests for v1.5 and v1.6 users somehow

Comment copied from #2777:
Note https://github.com/weaveworks/flux/issues/400#issuecomment-282859843:

both alpha and beta APIs exist in 1.6, and a role created with v1alpha1 will also be accessible from the v1beta1 API

so I tried this, and it seemed to work at https://circleci.com/gh/weaveworks/weave/8146 (albeit the 'publish' phase failed on branch naming)

I'm seeing a couple of problems when I apply this to the latest beta of kubeadm/k8s.

The taint needs to be specified as a real member and not an annotation.
The kube-dns pod needs to be killed/restarted.  I'm seeing the following in the describe there:

  13m	38s	57	kubelet, kubeadm-master		Warning	FailedSync	Error syncing pod, skipping: failed to "KillPodSandbox" for "2b03d2c9-0909-11e7-b38b-42010a8a0004" with KillPodSandboxError: "rpc error: code = 2 desc = NetworkPlugin cni failed to teardown pod \"kube-dns-3913472980-x7vp7_kube-system\" network: cni config uninitialized"

  13m	4s	60	kubelet, kubeadm-master		Normal	SandboxChanged	Pod sandbox changed, it will be killed and re-created.
  27s	4s	3	kubelet, kubeadm-master		Warning	FailedSync	Error syncing pod, skipping: failed to "KillPodSandbox" for "2b03d2c9-0909-11e7-b38b-42010a8a0004" with KillPodSandboxError: "rpc error: code = 2 desc = NetworkPlugin cni failed to teardown pod \"kube-dns-3913472980-x7vp7_kube-system\" network: required env variables missing"

It looks like there is an error about a missing env variable.  Is this the env variable for the kubernetes service?

"cni config uninitialized" suggests the Weave Net pods never hit started, because they write out the config.  Do you see anything in their logs?

The weave pods were definitely up.  I suggest you try the latest kubeadm to see if you can repo.

The kubelet has some errors like this:
Mar 15 00:46:19 kubeadm-master kubelet[2702]: W0315 00:46:19.180301    2702 docker_sandbox.go:234] NetworkPlugin cni failed on the status hook for pod "kube-dns-3913472980-x7vp7_kube-system": Unexpected command output nsenter: cannot open : No
Mar 15 00:46:19 kubeadm-master kubelet[2702]:  with error: exit status 1
Mar 15 00:46:19 kubeadm-master kubelet[2702]: 2017/03/15 00:46:19 CNI_NETNS env variable missing
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.186691    2702 cni.go:273] Error deleting network: required env variables missing
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.187645    2702 remote_runtime.go:109] StopPodSandbox "f2171645cf9f735b38590d5c19f69f4983f056a07180026cc93c2234738d5126" from runtime service failed: rpc error: code = 2 desc = Network
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.187935    2702 kuberuntime_gc.go:138] Failed to stop sandbox "f2171645cf9f735b38590d5c19f69f4983f056a07180026cc93c2234738d5126" before removing: rpc error: code = 2 desc = NetworkPlug
Mar 15 00:46:19 kubeadm-master kubelet[2702]: W0315 00:46:19.195435    2702 docker_sandbox.go:234] NetworkPlugin cni failed on the status hook for pod "kube-dns-3913472980-5j347_kube-system": Unexpected command output nsenter: cannot open : No
Mar 15 00:46:19 kubeadm-master kubelet[2702]:  with error: exit status 1
Mar 15 00:46:19 kubeadm-master kubelet[2702]: 2017/03/15 00:46:19 CNI_NETNS env variable missing
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.201472    2702 cni.go:273] Error deleting network: required env variables missing
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.202615    2702 remote_runtime.go:109] StopPodSandbox "b11a00b7adb3e17f6e1551c70f01d9066b525589feb1cd0822f7aa5271fc5124" from runtime service failed: rpc error: code = 2 desc = Network
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.202781    2702 kuberuntime_gc.go:138] Failed to stop sandbox "b11a00b7adb3e17f6e1551c70f01d9066b525589feb1cd0822f7aa5271fc5124" before removing: rpc error: code = 2 desc = NetworkPlug
Mar 15 00:46:19 kubeadm-master kubelet[2702]: W0315 00:46:19.210471    2702 docker_sandbox.go:234] NetworkPlugin cni failed on the status hook for pod "kube-dns-3913472980-520s7_kube-system": Unexpected command output nsenter: cannot open : No
Mar 15 00:46:19 kubeadm-master kubelet[2702]:  with error: exit status 1
Mar 15 00:46:19 kubeadm-master kubelet[2702]: 2017/03/15 00:46:19 CNI_NETNS env variable missing
Mar 15 00:46:19 kubeadm-master kubelet[2702]: E0315 00:46:19.216346    2702 cni.go:273] Error deleting network: required env variables missing

I managed to achieve error-message parity with you, @jbeda:
Mar 15 12:06:39 brya-0 kubelet[29821]: W0315 12:06:39.393959   29821 docker_sandbox.go:234] NetworkPlugin cni failed on the status hook for pod "kube-dns-3913472980-f0zpk_kube-system": Unexpected command output nsenter: cannot open : No such file or directory
Mar 15 12:06:39 brya-0 kubelet[29821]:  with error: exit status 1
Mar 15 12:06:39 brya-0 kubelet[29821]: 2017/03/15 12:06:39 CNI_NETNS env variable missing
Mar 15 12:06:39 brya-0 kubelet[29821]: E0315 12:06:39.399982   29821 cni.go:273] Error deleting network: required env variables missing
Mar 15 12:06:39 brya-0 kubelet[29821]: E0315 12:06:39.400960   29821 remote_runtime.go:109] StopPodSandbox "d8bb85a1bd54056f5698561f951204163423ee376235fd7777c193e7c4b7a77c" from runtime service failed: rpc error: code = 2 desc = NetworkPlugin cni failed to teardown pod "kube-dns-3913472980-f0zpk_kube-system" network: required env variables missing"

All of this derives from the kube-dns infrastructure container being dead:
# docker ps -a | grep kube-dns-3913472980-f0zpk_kube-system
d8bb85a1bd54        gcr.io/google_containers/pause-amd64:3.0   "/pause"                 About an hour ago   Exited (0) About an hour ago                       k8s_POD_kube-dns-3913472980-f0zpk_kube-system_7c2bbc2a-096e-11e7-a6df-42010a800004_0

It looks like kubelet is getting hung up on the repeated failure CNI_NETNS env variable missing, when what it should really do is blow the whole thing away and create a new pod.
That environment variable used to be mandated by the CNI spec, but the spec was changed a couple of months back to allow it to be unset, and when I updated the Weave Net plugin to honour that change it all works.
That update to the plugin is in #2850 but is not merged yet.

@luxas I think I'm in parallel with you - see branch luxas-add-head-manifest
I think the file should be named '1.6' rather than 'HEAD'

Merged via the 1.9 branch at ca22ecdf091fec289d31a3e27117934fa4062fe7
I then renamed the file at 9da5cee8d9b5fb89a14e271652e810ab5b0455c2 to emphasise this is the new file for Kubernetes 1.6


Migration: Nginx and Ingress conversion document

In order to achieve a smooth migration from Nginx to Ingress, we need to summarize the list of features/directives in nginx and how to convert them to Ingress model.
It would be better to have following items in the document:

Nginx directive name (linked to its document)
Priority (High, Medium, Low): The priority of implement the support to this directive
Corresponding Ingress/Envoy configuration (linked to its document)
How to convert

https://github.com/alibaba/higress/issues/548


BugFix/IIA-1165: Toggle Buttons on the left when starting Komet are misaligned, have inconsistent sizes, etc

Fixes https://ikmdev.atlassian.net/browse/IIA-1165
Actual Behavior
Toggle Buttons on the left when starting Komet are misaligned, have inconsistent sizes, text color and icon color don't match, etc

Expected Behavior
Icons should all have the texts aligned horizontally, icon sizes should be consistent, icons color and text color should match and other minor tweaks for a more pleasing UI
Steps to Reproduce
1 - Start Komet
Look at the left pane that contains the Toggle Buttons
How it looks after this PR

Thanks for the kind words @carldea glad you like it!


: /usr/local/bin/fsociety: line 2: python: command not found

Checkboxes

[ ] Updated fsociety
[ ] Issue does not already exist
[ ] fsociety issue, not a tool issue

Expected Result

Actual Result

#108 Fixed with installing python2.
sudo apt install python