Operationalizing with LXD

Required information

Distribution: Ubuntu
Distribution version: 16.04

config:
  core.https_address: '[::]:8443'
  core.trust_password: true
api_extensions:
- id_map
api_status: stable
api_version: "1.0"
auth: trusted
public: false
environment:
  addresses:
  - omitted
  architectures:
  - x86_64
  - i686
  certificate: omitted
  certificate_fingerprint: omitted
  driver: lxc
  driver_version: 2.0.8
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.4.0-93-generic
  server: lxd
  server_pid: 10656
  server_version: 2.0.10
  storage: dir
  storage_version: ""

Issue description
I am attempting to operationalize an environment and experiencing difficulties. I would appreciate pointers wherever available. I will contribute back to the community with this information/setup once accomplished.
Use Case
There are multiple hosts. Each user will receive a container. The containers are spread evenly across the hosts. Each container will have multiple macvlan interfaces with statically assigned IP addresses. The containers will run unprivileged as root (currently). This will become a fully automated workflow in deployment of host configurations and containers alike.
Current Profile Template (Filled)
# Starting easy..network and rootfs.
config:
  environment.http_proxy: omitted
  raw.lxc: lxc.aa_profile = lxc-container-default-with-nesting
  raw.lxc: lxc.id_map = u 0<PHONE_NUMBER>6
  raw.lxc: lxc.mount.auto = cgroup
description: Template developer profile
devices:
  mgmt:
    name: mgmt
    host_name: remingtonc-mgmt
    nictype: macvlan
    parent: enp1s0f0
    type: nic
    hwaddr: be:7b:95:ae:f2:d1
  rootfs:
    type: disk
    path: /
    source: /opt/remingtonc
name: remingtonc

Questions

Are "profiles" for LXD the equivalent of LXC config files? Is it reasonable to have a profile per container given the different network needs? Should we use /etc/lxc/default.conf any more, or profiles? Which has priority?
For macvlan interfaces, is parent an existing device? Is host_name an existing device? Will the interface be created automatically? Are the macvlan interfaces configured with IPs from the host, and not within the container? Should I be using lxc network at all? The above errors out, error: Failed to set LXC config: lxc.network.veth.pair=remingtonc-mgmt
How do we specify the root mountpoint? The above errors out, Config parsing error: Root disk entry may not have a "source" property set. Is there a better way to do this?
For many environment variables - should lxc init ... then lxc file push /etc/environment container/etc/environment be used?
Does security.nesting = true have the same effect as lxc.aa_profile ... etc. as specified above in the raw.lxc sections?
How much of this should be done directly in profile versus via the command line? Is there a better practice either way?

/etc/lxc/default.conf isn't used by LXD
Profiles are a way to group common configuration for LXD containers, note that unlike default.conf, changes to any of the container's profiles after it was created will affect the container (and for most changes, this will be applied immediately to running containers).
For macvlan devices, "parent" is the name of the existing host interface you want to attach to. "host_name" is only relevant when using a veth interface ("bridged" or "p2p" nictype). With LXD, it's recommended to do any IP configuration from within the container, rather than attempt to pre-configure the network device before starting it.
The root filesystem of a LXD container must come from a LXD storage pool. We don't allow arbitrary host paths for that, which is why the "source" property isn't allowed in this case.
It's fine to use "lxc file push" to overwrite /etc/environment before container startup. An alternative would be the "environment.NAME=value" LXD config keys but those will only be set in PID1's environment and in any "lxc exec" sessions, which may not always be sufficient.
security.nesting=true is equivalent (in fact much better) to what you did above
You can load profiles from yaml and apply multiple profiles to each container.

This answered my questions. For further clarification, the parent macvlan device should exist beforehand - it is not automatically created.

The "parent" for a nictype=macvlan device refers to the physical device, not to a macvlan device.
So if you want to connect your container to the physical eth1 interface on the host, then parent is eth1.


WKDownloadHelper is compatible with iPhone but not working with iPad.

I have checked the code base provided given as a reference in your docs, and I have found that WKDownloadHelper is not working with IPAD devices.
can you check me with the given issue.

I'm sorry I don't understand, maybe you are referring to one of the example where I use UIActivityViewController?
Because the rest of the code shouldn't deal with UI so I don't see why it shouldn't' work on the iPad

what I'm trying to tell is, your code runs perfectly on iPhone but when I tried on iPad its not working ,if you have the code for iPad please inform me....

but I don't understand exactly what the problem is, you don't see the popover maybe?
If you download this project and you run it, does it work? https://github.com/gualtierofrigerio/WKWebViewDownloadHelper
You should see the popup from the navigation bar button item in this example

yes , I didn't see the popover on iPad
https://github.com/gualtierofrigerio/WKDownloadHelper/assets/119996617/68cfc9e6-3f98-4e82-97e5-41ef7d33b9fd
how to get the popover on iPad ?

I'll try to help you but posting a video is not going to make a difference here.
First, try this solution
https://stackoverflow.com/questions/62996746/ios-13-5-1-uiactivityviewcontroller-not-showing-on-ipad
and use the rect instead of the source view and see if it helps.

thanks bro , now it works fine.....
that one line of code is missing

ok glad is fixed, if you find other issues on the library itself feel free to report :)

yeah sure....


OSL BSSRDF fixes & test suite update

Looking at the images, some new results look weird on the contact plane.

I kind of agree, actually. Esteban, can you remind me again why you modified the non-OSL test scenes?

I added UV coordinates to the box and the sphere, for the bump tests.

Ah, right. For some reason it seems to lead to noticeably more noise on the floor planes...


feat: ✨ Confirm support for python 3.10

Codecov Report

Merging #11 (aa3e4a6) into main (2bb23c6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          499       495    -4
  Branches        42        42
=========================================
- Hits           499       495    -4

Impacted Files
Coverage Δ

hitcount/mixins.py
100.00% <0.00%> (ø)

hitcount/templatetags/hitcount_tags.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Inference API

Hi, this is a nice work!
Could you please provide an inference api so that, for example, the user only needs to provide the path to the image and the corresponding description?

Hi, this could be implemented by only creating a model, load pretrained weights, and input an image into the model.


provide a demo for visiting web app

How to run a web app in docker environment based velad?

Provided in document


Remove unchecked use of local storage again

Accessing window.localStorage outside of a try catch block causes an exception in private browser sessions with 3rd party cookies disabled. We need to revert the change made for version 0.0.4.

Coverage increased (+1.6%) to 91.071% when pulling 8f46e4469a882ef12394403948d9005ac053cfce on cephalopodMD:master into f663c23ac41beff094378738843584c5754cd6cf on bigpipe:master.

@3rd-Eden could we consider taking in this PR? I saw there was a similar one from earlier this year that hasn't been taken. I thought that a revert might be more acceptable than adding new code.
This package gets included in many projects, including the winston logger package, which I'm using right now. Any bundle that includes this package will throw an uncaught exception and crash in an incognito chrome tab with 3rd party cookies disabled. I can fix this for my case by specifying 0.0.3 of this package in package.json, but I think it would be preferable to make sure that the default/latest version did not have this bug.


When zoom is big, text editor waste a lot of space and some elements aren't scaled

Godot version:
3.1 beta 2
OS/device including version:
Windows 10
Issue description:
When zoom in text editor is big, then:
1.Editor waste a lot of space for showing number of row
2,3. Doesn't scale tabulator and code block icons

We should probably restrict the code font size to the double of the default value (which would be 28) to prevent this. If higher sizes are needed on hiDPI displays, then the Editor Scale value can be increased above 100%.

Still reproducible with indentation carets in 4.0.3:


feat: add clusteraffinity plugin skeleton and some data structures

Description of your changes
Add the cluster affinity plugin.
Fixes #
I have:

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

How has this code been tested
Added unit tests
Special notes for your reviewer
The clusteraffinity plugin Prefilter/Filter/Prescore/Score and some of the unit tests will be in a separate PR.

Hi Zhiying! The node affinity plugin technically speaking does not need to keep states. The plugin state is introduced for the topology spread constraints plugin as each run of the Filter/Score extension point must know a global state, i.e., the current spread across the fleet; this is not true for this plugin IMO.

discussed offline, prefer to convert the affinity terms in one place.
Moved the types.go to the clusteraffinity package.


Error in Select2Editor typescript

Hi Jaiq,
I have custom editor with typescript below
@Serenity.Decorators.registerEditor()
export class ClientEditor extends Serenity.Select2Editor<Object, Object> {
    constructor(hidden: JQuery) {
        super(hidden, null);
        super.addItem("Clieant A", "Clieant A");
        super.addItem("Clieant B", "Clieant B");
        super.addItem("Clieant C", "Clieant C");
    }
}

No data loaded and error as captured

No idea, find out why your text is null

have you solved the problem?
@fazanan

Instead of
super.addItem("Clieant A", "Clieant A");
try
super.addItem({
  id: "Clieant A",
  text: "Clieant A",
  source: null,
  disabled: false
});


Script for dependecies

Saw that a lot of guys got stuck at 94% and having the info dispersed it's not so great, so I've created an npm script that will take care of the dependencies.

Hey @GraphicDesk, thank you for this PR.
Maybe we could install all global dependencies as local dependencies, and run them directly from node_modules, instead of creating an npm script to install global dependencies.
Like this: ./node_modules/.bin/harp compile maybe.
@caiovaccaro, @GraphicDesk  what do you think?
Seems like that "stuck 94%" problem on #72 and #75 is caused by harp's absence.

@Nipher Sounds good from my point of view. However I've got some issues with harp compile.
I'm fond of having tasks for installing, and tasks for compiling, etc... We could remove the "-g" from the script as well ...

Yep. Running everything locally is ideal. Can you do that @Ionut-Milas ?

Merged onto develop.


Add a shim for Databricks 11.3 (Spark 3.3.0)

This PR is evolving the initial work from #7100 and replacing it to fix all the other shims the original PR broke.
Maintaining sources at the directory level for vendor versions cutting across multiple upstream versions is almost impossible without a significant duplication.
This PR proposes a technique to create a patchwork of files. Instead of dealing with intertwined directories and risk breaking the existing shim with every change, we extract reusable pieces and ink individual files with relative paths.
So if we need create a seemingly irreconcilable mix  from file A out of a 340+ source set and file B from a until340 (exclusive) source set we create relative path symlinks for just these two files in 330db
ln -s -r src340+/path/A  src330db/path/A
ln -s -r src/311until340/path/B src330db/path/B

This allows to keep the existing shims virtually unchanged. We can review and consolidate common dirs after analyzing symlinks in smaller follow up PRs.
Other changes to make 11.3 work

GpuPredicateHelper now extends and shims PredicateHelper
Allow passing TEST_PARALLEL to test.sh to be able to run integration tests on a small instance
No need to override getSparkShimVersion using the same implementation in every shim

Fixes #6879
Co-authored-by: @amahussein
Co-authored-by: @nartal1
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

I am not a fan of symlinks in general and especially not for source code.
At least windows no longer requires admin privileges to use them, so that is no longer a big concern. But there was a reason why windows wanted them to be admin only. There are a number of potential security issues with trusting symlinks. This is why tar by default will not write a file or a symlink outside of the original base directory it is untarring into. We own this repo, and we trust it enough to build it so it is less of a concern.
There are also a ton of issues with refactoring. Like what happens if I rename a file, or move it. What happens if I refactor a single class and move it out of the file.
Also it is not clear what shim a given file is a part of. I could add a bunch of code to a file that impacts version X.Y.Z but accidentally add it to DB not because DB needs it, but just because I named it that way.
These are not big enough concerns to block this going in. To me it feels more like we are hitting limitations with us trying to break up shims by version ranges. Perhaps we should be looking at breaking things up by features also. Just something to think about.

It's all valid concerns. My comfort level regarding symlinks is limited as well.
I think the construction of links guarantees that we don't read outside the repo root.

To me it feels more like we are hitting limitations with us trying to break
...
Perhaps we should be looking at breaking things up by features also.

This is definitely true especially once we deal  with downstream vendors db and cdh. I've been thinking along the same lines

still issues with db9.1 and db10.4, making it a draft for now

I think the construction of links guarantees that we don't read outside the repo root.

Nope I can totally create links outside of the repo.
$ git init
Initialized empty Git repository in .../test/.git/
$ ln -s /bin/ls ./ls
$ git add ls
$ git commit -a
[master (root-commit) 33ee1bf] Check if symlinks restrict
 1 file changed, 1 insertion(+)
 create mode 120000 ls
$ ./ls
ls

Perhaps we should be looking at breaking things up by features also.
This is definitely true especially once we deal with downstream vendors db and cdh. I've been thinking along the same lines

Then lets spend some time to figure out how we want to do that. I am fine with this as a short term fix, but long term I think we can do a lot better with shims.

Nope I can totally create links outside of the repo.

Of course you can choose to do this. But the method I am suggesting creates the links that don't go outside the repo.

Then lets spend some time to figure out how we want to do that. I am fine with this as a short term fix, but long term I think we can do a lot better with shims.

+1

build

build

@revans2  #7222 is a take on simplification of the shim source code layout. Every shim file explicitly lists all Spark builds it supports.

build

build

build

build


virtctl vnc is barely usable over a slow link

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:
i tried to use virtcl vnc on a relatively slow link. i never saw the console
What you expected to happen:
see a console
How to reproduce it (as minimally and precisely as possible):
when accessing through a vpn for instance, virtcl vnc is not able to display anything
Anything else we need to know?:
it seems to be related to the use of a socket for the vnc connection, since the following ugly hack, which basically consists of switching live the xml definition of my vm to use a tcp port for vnc, allows  me to get the vnc console
NAMESPACE=$(oc project -q)
VM=$1
POD=`oc get pod -o name | grep $VM | cut -d'/' -f2`
oc exec -c compute ${POD} -i -t -- virsh qemu-monitor-command ${NAMESPACE}_${VM} --hmp "change vnc :2"
oc port-forward ${POD} 5902:5902

Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Hey @karmab, just to be sure: what do you mean with "I never saw the console"? Did remote-viewer show "waiting for display 1"? If so, and if you're using Openshift CRI-O, it's fixed by https://github.com/kubevirt/kubevirt/pull/1516

@slintes this is indeed exactly my issue


Transaction not confirmed until today and no money back in my wallet

My transaction is going unconfirmed and I can't resolve it in any way, please help me.

Size
4026 (bytes)

Fee Rate
0.00100298 VTC per kB

Received Time
Feb 11, 2021 10:14:18 AM

Mined Time
N/A

Included in Block
Unconfirmed

3.640519 VTC
adress: vtc1qve62xnudvpfksku8vz3pmexu6eju0533s743x7

that address is not from a bitpay wallet.


feat(metadata): implement PATCH device profile basic info API

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[x] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

run metadata from this branch
verify  PATCH /api/v2/deviceprofile/basicinfo is functioning

(dependent on https://github.com/edgexfoundry/go-mod-core-contracts/pull/715)
New Dependency Instructions (If applicable)

Codecov Report

Merging #3926 (ad60142) into main (81bad76) will increase coverage by 0.10%.
The diff coverage is 78.12%.

@@            Coverage Diff             @@
##             main    #3926      +/-   ##
==========================================
+ Coverage   46.11%   46.22%   +0.10%
==========================================
  Files         111      111
  Lines        9591     9623      +32
==========================================
+ Hits         4423     4448      +25
- Misses       4781     4788       +7
  Partials      387      387

Impacted Files
Coverage Δ

internal/pkg/infrastructure/redis/client.go
0.00% <0.00%> (ø)

...nal/core/metadata/controller/http/deviceprofile.go
87.71% <100.00%> (+1.45%)
:arrow_up:

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 81bad76...ad60142. Read the comment docs.

recheck

@hahattan please update go.mod and go.sum to resolve the conflict


Fix MemoData treated as MemoFormat due to changed method signature

LGTM

Don't merge yet, should add a test

@wltsmrz do you have a test to add?

@wltsmrz bump

Coverage remained the same at 87.52% when pulling ffd30eb372ec8eef22a07c1b1e2dd4012848285f on fix-memos into<PHONE_NUMBER>310ced91569882d07e005489de1870 on develop.

LGTM


Destroying VM fails

Great plugin, but I am having an issue with destroying VMs. When I destroy, with the VM up, it asks for credentials to remove the computer from the domain as expected, and then it reboots. Vagrant then appears to be sitting and waiting for the machine to shutdown, which never happens.
# vagrant destroy
Are you sure you want to destroy this machine and disconnect from domain.local? (y/n)y
Please enter your domain username: tboyce
Please enter your domain password (output will be hidden):
==> default: "Running Windows Domain Provisioner"
VERBOSE: Performing the operation "Remove-Computer" on target "tboyce-mgc".WARNING: The changes will take effect after you restart the computer
tboyce-mgc.Restarting computer for updates to take effect.
==> default: Attempting graceful shutdown of VM...
==> default: Checking if box 'win81-vs2015' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Fixed port collision for 3389 => 3389. Now on port 2200.
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
==> default: Forwarding ports...
    default: 3389 => 2200 (adapter 1)
    default: 22 => 2222 (adapter 1)
    default: 5985 => 55985 (adapter 1)
    default: 5986 => 55986 (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
    default: WinRM address: <IP_ADDRESS>:55985
    default: WinRM username: vagrant
    default: WinRM transport: plaintext

After waiting for a few minutes vagrant prints this error:
Timed out while waiting for the machine to boot. This means that
Vagrant was unable to communicate with the guest machine within
the configured ("config.vm.boot_timeout" value) time period.

If you look above, you should be able to see the error(s) that
Vagrant had when attempting to connect to the machine. These errors
are usually good hints as to what may be wrong.

If you're using a custom box, make sure that networking is properly
working and you're able to connect to the machine. It is a common
problem that networking isn't setup properly in these boxes.
Verify that authentication configurations are also setup properly,
as well.

If the box appears to be booting properly, you may want to increase
the timeout ("config.vm.boot_timeout") value.

Thanks @tboyce, could you please provide a Gist containing a debug log for this when it breaks again? Instructions for verbose logging can be found here: https://www.vagrantup.com/docs/other/debugging.html
Also, if you could please provide some detail on your OS version and anything else that might be relevant that would be great. I'm going to guess Windows 8.1 based on that box name though!

Seems like there might be some potentially sensitive data in the debug log. If I can find the time I will see if I can reproduce in a clean environment. The host and guest OS are both Windows 8.1 and I am using VirtualBox and winrm to communicate.

Thanks Tim. If it's easier feel free to obfuscate the data.
On Fri, Mar 4, 2016 at 12:59 AM, Timothy Boyce<EMAIL_ADDRESS>wrote:

Seems like there might be some potentially sensitive data in the debug
log. If I can find the time I will see if I can reproduce in a clean
environment. The host and guest OS are both Windows 8.1 and I am using
VirtualBox and winrm to communicate.
—
Reply to this email directly or view it on GitHub
https://github.com/SEEK-Jobs/vagrant-windows-domain/issues/10#issuecomment-191771416
.

--
Matt Fellows

http://www.onegeek.com.au/

Looks to be related to #16


Allow choosing a folder to zip deploy

It would be amazing if we could prompt the user to select the folder to zip deploy. Often in web apps we deploy a dist or public folder. Let's allow the user to select the folder to deploy so we don't deploy everything, including source code.
cc @timheuer as FYI

I was looking for the same thing.

Instead of starting on the Azure App Service instance, if you start from the folder in the Workspace, there is a "zip this folder and deploy"

nice find!
would love to see a way to prompt the user from the extension too.

Yes @johnpapa -- don't close this please...if anything this is now an issue to IMPROVE the discoverability and capability of the existing functionality.

Unfortunately it's not working for me. I can select the subscription, but not the Web App. I only get the option to add a new Web App. All my Web Apps are listet in the Azure App Service Panel.

@Globalrun Are all of your Web Apps Windows based?

Yes they are. The only thing, I can think of, that could be different to a default installation is the fact, that I have two Azure subcriptions. But it works all fine in the Azure panel.

Is this possible to pass options too at the same time? (main concern here is cleaning additional files at destination)

@Globalrun This should have been fixed in 0.5.0. It will be published to the marketplace in a few days.

@johnpapa @timheuer currently the deploy command will ask you to pick a workspace folder when there is multiple workspace folders. Now the question is, do you want the prompt to go as deep as the folder structure goes? Or just first level of folders under the workspace folder root?
I'm also thinking about adding a workspace setting to remember the choice so you don't have to choose the same folder each time you deploy.
Thoughts? (especially about the folder depth)

@whiskeyjay I think you need to go deeper.  The example we were working with was that the workspace represented the full source, but you can do a build within that which will produce a static site output and then put things in the 'dist' folder which is the one you actually want to zip up.  Make sense?


Tooltips flash when Task Manager is open

Operating system or device:
Windows 7 Ultimate 64x
Issue description (what happened, and what was expected):
The tooltip of menu elements are flashing, and it normally wouldn't.
Steps to reproduce:
Open Task Manager, put your mouse over any menu element, and wait a bit until the tooltip show up.
Really unnecessary to fix, but maybe you find another problem related to what is causing this, so I decided to relate anyways.

Duplicate of #3835.


[TASK] Implement reusable Avatar component

Task Description
The purpose of this task is to implement a reusable Avatar component that can be exported for external use.  Avatars can be used throughout the dapp to represent entities and specifically Avatars are used to display validators and authority sets.
Note: used in conjunction with user ID or in table; should not be used itself without supplementary content.
Task Deliverable

[x]  Create a folder Avatar
[x]  Implement the Avatar component with the applicable props
[x]  Create a folder AvatarGroup
[x]  Implement the AvatarGroup component with the applicable props
[x]  Implement both dark and light variations
[x]  Document each component with an example
[x]  Ensure component is exportable

Component properties:

size="small | regular";
type="single | stacked";
dark mode="off | on";

Sizes
You should be able to change the size of the avatar with the heightand widthCSS properties. However the default sizes should be as followed:

small: 24x24
regular: 48x48

Example:
<Avatar
  alt="Webb Logo"
  src="webblogo.png"
  sx={{ width: 24, height: 24 }}
/>

Stacked
AvatarGrouprenders its children as a stack. Use the maxprop to limit the number of avatars. If you need to control the total number of avatars not shown, you can use the total prop. This would add the total number of avatars not shown. See Figma designs for example.
Example:
<AvatarGroup max={4}>
  <Avatar alt="Authority1" src="/static/images/avatar/1.jpg" />
  <Avatar alt="Authority2" src="/static/images/avatar/2.jpg" />
  <Avatar alt="Authority3" src="/static/images/avatar/3.jpg" />
</AvatarGroup>

Relevant links

Figma Avatar Design System

Closed in #541


fix(slurm_ops): ensure that all slurm daemons are installed

This PR ensures that all the Slurm daemons are installed on the host with the necessary client commands. This mimics the behavior of the Slurm snap where all the daemons are installed but are controlled through orchestration with the charms. This also helps that our version of Slurm is consistent across all machines since Slurm doesn't guarantee backwards compatibility between daemon versions.
Fixes #28

Hmm... forgot to catch that slurm-wlm is the package being used to get the version of Slurm installed on the host. Can switch to slurm-client instead as that package is installed on all the nodes.

We should omit the slurm-client package from slurmdbd and slurmrestd. Can we use the specific slurm package that is installed for each respective daemon?

Is there a reason to add all of the slurm daemons to every node?

Hey hey @jamesbeedy - thanks for chiming in here!

We should omit the slurm-client package from slurmdbd and slurmrestd. Can we use the specific slurm package that is installed for each respective daemon?

Omitting the slurm-client package is possible; I mostly added it because (a) we can use it in place of the slurm-wlm package to grab the version of Slurm installed on the units and (b) being able to talk to Slurm from anywhere in your cluster is nice. We can play around with it however since slurm-client is brought in as a dependency on the units that need the client commands to be available.

Is there a reason to add all of the slurm daemons to every node?

As for installing the Slurm daemons on each node with _AptManager, this is to emulate the behavior of the Slurm snap. All the daemons are provided by the Slurm snap, and through the charms we can control which daemon is started up. Makes it easier for ensuring a consistent Slurm version/API across the entire cluster. We take the "kitchen sink" approach here since it makes abstracting the differences between how the formats are handled easier to work with. We can play around with it however. Originally we were just using the slurm-wlm to install the daemons.
Eventually where we want to get is that we can provide a base Slurm implementation that anyone can use, but then also support side-loading your own version of Slurm for those that require custom extensions.

@jedel1043 @jamesbeedy commit https://github.com/charmed-hpc/hpc-libs/pull/32/commits/1fe995bd2d6b00ba51738245f7accdb564aa0206 enables _AptManager to manage a specific Slurm daemon rather than the "kitchen sink". Let me know what you think!

Thanks @jedel1043 @jamesbeedy - will hold off on vbump after I add the necessary bits for jwt and cgroup editing.


feat: Sphinx doc improvements

Adjusted index to look better and separated each section to their respective file
Added External Processing Client example
Adjusted some typos
Added references to 'source' files and internal links to the documentation

Is it possible to check the new version here:
https://pweiber.github.io/service-extensions-samples/
If the content does not update, try refreshing your cache and it should be good, let me know your thoughts


2.3 Release Notes - HTTP Global Handlers

Description

These are no longer defined in the default.ini file, but have been moved to the couch.app context. If you need to customize your handlers, you can modify the app context using a couchdb.config file as usual.

I need to customize a handler and can't find any reference to couchdb.config in the documentation.

This requires a compile from modified source, the couchdb.config file is autogenerated from the source. See the file src/couch/src/couch.app.src for more info.


Custom AWS CF templates

How to generate custom AWS templates from installer.

@joel-hamill let's get someone to user test this.


Add framework to manage JSON data

Problem
The C++ SDK will need to use JSON files for configuring some of the integration tests.
Currently, the project doesn't have any JSON framework to manage such data.
Solution
To add a new JSON framework in the project used for the integration tests.
Alternatives
To make our own implementation for reading configuration data from a JSON file. Using this approach will take some time and it's not a good idea at the moment.

PR #208 was merged into main branch.


s4696681: Implementation of GCN Model for Node Classification on the page-page Facebook dataset

This is my pull request to merge the node classification facebook page-page problem for the Report Assessment for COMP3710
My implementation is a Graph Convolutional Network for the purpose of classifying nodes in the dataset
My implementation does the following:

imports and preprocesses the facebook page-page dataset

Creates an adjacency matrix from the data provided in musae_facebook_edges.csv
Normalises this adjacency matrix as required by the GCN
creates the feature vectors for each node, since there were an inconsistent number of features per node, bag of word feature vectors were created
encodes classifications to be integers instead of strings so they can be used in the model
create the tensors required by pytorch to be fed into the model

The model itself consists of two Graph Convolutional Layers and the model is tuned and trained using nested cross validation
This pull request contains the required files to satisfy the Task Sheet:
modules.py (contains model architecture)
dataset.py (contains preprocessing of data)
train.py (contains the training for the model)
predict.py (is the test driver, where through this all the other files are called)
README.md (contains information about the project and how to run the model)

This is an initial inspection, no action is required at this point
Difficulty: Normal
Readme Overall: Good

Project Overview: Good
Model: Good
Data + Preprocessing: Good
Training / Loss Curve(s): None
Result Demonstration: Good
References: Good

Functionality:

Multi-layer GCN: Good, 2 layers.
Semi supervised: Good
Accuracy: OK. Not mentioned if its Test or Train accuracy.
TSNE or UMAP: Good, though would have expected to see a bit more separation at them mentioned (91%) accuracy.
Discussion: Good

Code: Has early stop in the code, but also uses multiple epochs as a searchable hyper parameter, which is redundant.

Consistent with Results: OK.
File Structure: Good
Commenting: Good
Commit frequency: Good
Commit messages: Good

Other Comments:
Epochs should not need to be in the hyper parameter search. You do have an early stop in the code anyways. You could have used the number of GCN layers as a hyper parameter.

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage, no training plots -2
Module present
Commenting minimal -1
No Data leakage
Difficulty: Nromal -5
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe acceptable, no refs -1, no usage -1
Model/technical explanation
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good

Restore repo README to allow merge, does not affect grade only merge.

No feedback attempt and no feedback marks lost.


lowercase_constants: Enable configuration.

This issue is to have the lowercase_constants directive configurable. (Issue #3026)
The configurable option would let you choose in between 'upper' or 'lower' case syntax.
Here's an example on how it would be:
lowercase_constants:
   syntax: upper

or
lowercase_constants:
   syntax: lower

The default would be lower to be backward compatible.
Why this request ?
CMS's like Drupal 8 are using uppercase constants (TRUE, FALSE, NULL) and I would like to have a proper tool that follows the Drupal 8 "standard" in our custom code at European Commission.
The PHP version you are using ($ php -v):
=> PHP 7.2.14
PHP CS Fixer version you are using ($ php-cs-fixer -V):
=> PHP CS Fixer 2.14.1-DEV Sunrise by Fabien Potencier and Dariusz Ruminski
The command you use to run PHP CS Fixer:
=> Via grumphp
The configuration file you are using, if any:
        array_syntax:
            syntax: short
        blank_line_after_opening_tag: true
        blank_line_before_statement:
            statements:
                - break
                - continue
                - declare
                - return
                - throw
                - try
        include: true
        # Drupalism:
        class_attributes_separation: false
        declare_equal_normalize:
            space: single
        declare_strict_types: true
        is_null: true
        lowercase_static_reference: true
        # Drupalism: lowercase_constants (unable to configure it in php-cs-fixer. Provide a patch?)
        native_function_invocation: false
        # Drupalism:
        no_blank_lines_after_class_opening: false
        no_closing_tag: true
        no_extra_blank_lines:
            tokens:
                - extra
                # Drupalism: - curly_brace_block
                - parenthesis_brace_block
                - square_brace_block
                - return
                - throw
                - use
                - use_trait
        no_trailing_whitespace_in_comment: true
        no_unneeded_final_method: true
        single_blank_line_at_eof: true
        single_line_after_imports: true
        strict_comparison: true
        strict_param: true
        trailing_comma_in_multiline_array: true
        yoda_style:
            equal: false
            identical: false
            less_and_greater: false
            always_move_variable: false

If applicable, please provide minimum samples of PHP code (as plain text, not screenshots):

before running PHP CS Fixer (no changes):

if ($var === null) {
  // Do something
}

with unexpected changes applied when running PHP CS Fixer:

if ($var === null) {
  // Do something
}

with the changes you expected instead:

if ($var === NULL) {
  // Do something
}

Closes due to merge of #4275


Contributed command is not visible in command palette for webview panel

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.64.2
OS Version: Windows 10

Steps to Reproduce:

Create a webview using vscode API,
webviewPanel = vscode.window.createWebviewPanel(
'my.webview'
'My Webview',
vscode.ViewColumn.Beside,
{
retainContextWhenHidden: true
}
);
Try to contribute new command to above webview panel in package.json
{
"command": "my.command",
"title": "My Command",
"enablement": "activeCustomEditorId == 'my.webview'"
}

Issue: Contributed command "my.command" will not be visible in the command palette(we have to select webview and open command palette and search for command)
I have used few more enablement conditions but I am not able to contribute command to webview panel.
e.x.:  "enablement": "activePanel== 'my.webview'", "enablement": "focusedView== 'my.webview'
If we contribute new custom editor then we can contribute command to its webview panel but I am creating simple webview panel.
Any way we can refer a webview panel based on its view type inside when/enablement conditions?

activeCustomEditorId is only set for custom editors (not regular webviews). For those you currently have to set your context keys, see markdownPreviewFocus in the build-in markdown extension: https://github.com/microsoft/vscode/blob/307364a17478bb2045d3fb29399f99ecb06f6cb6/extensions/markdown-language-features/src/features/previewManager.ts#L58


Addfavicon

fix favicon
#880

There have some conflict issue check it once and make a new PR. ok

how to fix the error


It is not an issue, but the documentation needs correction. When you inject the service the name in docs is wrong.  service:notification-messages should be service:notification-messages-service

Strange... doing notifications: service('notification-messages') worked fine for me

That is interesting, because after upgrading to the new ember-cli,
I need replace notification-messages to notification-messages-service, because without it I have error in tests.
Assertion Failed: Attempting to inject an unknown injection: 'service:notification-messages'

this issue is related only in tests. In develop I can normally use this injected service in both names.
Ember CLI: 2.17.1
Ember CLI Qunit 4.2.1

I could confirm this behavior. Only notification-messages-service is working in unit tests. I'm not quite sure why notification-messages works at all cause that's not the name of the service exposed.
Nevertheless notification-messages-service is not a good name. Maybe it should be renamed to notifications? In that case we wouldn't have to write notifications: service('notification-messages-services') but only notifications: service().
ember-source: 2.16.2
ember-cli: 2.16.2
ember-cli-qunit: 4.0.2
ember-qunit: 2.2.0

This is related to #169 and should also be fixed by #171.

Getting problems using both versions of service name. If I inject as:
notifications: service('notification-messages')

I get Attempting to inject an unknown injection error. If I inject as:
notifications: service('notification-messages-service')

then the notifications don't display. So for now, the most reliable way for me is:
import CliNotifications from 'ember-cli-notifications/services/notification-messages-service';

export function initialize(application) {
    application.register('service:cli-notifications', CliNotifications);
    application.inject('component:notification-container', 'notifications', 'service:cli-notifications');
    application.inject('component:notification-message', 'notifications', 'service:cli-notifications');
}

Later you inject it as service('cli-notifications'). And of course instead of cli-notifications you can use any name you want.
Maybe it helps someone.

This whole thing is solved in https://github.com/mansona/ember-cli-notifications/pull/251#issuecomment-552737340 🎉
If anyone here was still having an issue they can try out the new version before it is released 👍 thanks for the report


#parseKORE strange behaviours when having malformed terms

K-master: 5cb2dfe
Usage: kompile test.k ; krun
module TEST
  imports INT
  imports K-REFLECTION

configuration <k> 1 </k>
              <names>
                <name multiplicity="*" type="Set"> wrap(#token("a", "A")) </name>
              </names>

syntax KItem ::= wrap(A) [klabel(wrap), symbol]
syntax A ::= B
syntax B ::= C | "b1" [klabel(b1), symbol] | "b2" [klabel(b2), symbol]
syntax C

rule <k> 1 => 2 </k>
     <names> .Bag =>
       <name> wrap(#token("a", "A")) </name>
       <name> #parseKORE("Lblwrap{}(Lblb1{}())"):KItem </name> // missing injection
       <name> #parseKORE("Lblwrap{}(inj{SortB{},SortA{}}(Lblb2{}()))"):KItem </name>
       <name> #parseKORE("Lblwrap{}(\\dv{SortB{}}(\"b3\"))"):KItem </name>  // missing injection
       <name> #parseKORE("Lblwrap{}(inj{SortB{},SortA{}}(\\dv{SortB{}}(\"b4\")))"):KItem </name>
       <name> wrap(#token("c", "C")) </name>
     </names>

rule <k> 2 </k>
     <names>
       <name> wrap(_:B) </name> => .Bag
       ...
     </names>

endmodule

Output: kore (trimmed)
// before rule 2
Lblwrap{}(\dv{SortA{}}("a"))
Lblwrap{}(inj{SortB{}, SortA{}}(Lblb1{}()))) // injection suddenly there
Lblwrap{}(inj{SortB{}, SortA{}}(Lblb2{}())))
Lblwrap{}(\dv{SortA{}}("b3"))) // sort of variable is changed for some reason
Lblwrap{}(inj{SortB{}, SortA{}}(\dv{SortB{}}("b4"))))
Lblwrap{}(inj{SortC{}, SortA{}}(\dv{SortC{}}("c"))))

// final output
Lblwrap{}(\dv{SortA{}}("a"))
Lblwrap{}(inj{SortB{}, SortA{}}(Lblb1{}()))
Lblwrap{}(\dv{SortA{}}("b3"))

I would expect #parseKORE to be a bit smarter, or at least return some error messages (#966).
b1  is a malformed term where it's missing an injection. Rule 2 doesn't apply, but the pretty-printer still thinks there is an injection there for some reason.
b2 is correct and rule 2 applies, but it looks exactly like b1
b3 gets its sort completely changed. Why? I specifically say SortB, and now it has SortA
b4 looks right, and the rule applies correctly

Requires validation to be added in parseKORE and the internal translation step to runtime terms.

#parseKORE has been removed.


typo fix misconfiguration in example

Proposed changes
Issue Number: close #xxx

I found some configuration in the example that was inconsistent with the context, and fixed.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


fsck --decrypt re-encrypts all the files even those with no need to touch

Summary
fsck --decrypt reencrypts all the files even those with no need to touch.
I mentioned that allready in #2459 and was told to create a separate issue.
Steps To Reproduce
gopass fsck --decrypt
Expected behavior
should only re-encrypt files where the key changed or such but should ignore files without the need for re-encryption.
Environment

OS: fedora 37
OS version: 6.0.14-300.fc37.x86_64
gopass Version: master
Installation method: go install from master used the latest package before
BTRFS

Yes, this is correct. I now understand the issue and I think we have PR #2492 open already they might fix it.


Material Preview doesn't Update in Filesystem

Tested versions
v4.3.dev3.official [36e943b6b]
System information
Windows 10
Issue description

I made some changes to the Material as you can see it in the inspector to the left. It has this orange albedo texture applied to it.
But this change is not reflected in the FileSystem, there the material is still white (default).
I was expecting the material in the filesystem to look like the one in the inspector.
Steps to reproduce
Create a Material change its properties in the inspector. Have a look at it in the filesystem view.
Minimal reproduction project (MRP)
Godot 4.3 Demo Bug Report.zip

Still happening in v4.3.beta3 (linux), but in my case it shows nothing at all (not the white sphere)

I just had something this happen in 4.3rc3 as well:

I just duplicated the ArchFrameMaterial and it appeared green in the FileSystem list (which is actually wrong, it's the blue color seen in the door frame in the 3D view) until I did "Save All Scenes" and then instead the thumbnail went black and white.


Update secrets manager rotation rules to include a schedule expression

Do not merge until test secret has rotated on old schedule on Tuesday 3rd Dec


Client Allocation Request for: 後憲保全股份有限公司

Client Allocation Questions
Core Information

Name: 後憲保全股份有限公司
Website / Social Media: http://www.mp.url.tw/html/index.php
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized: f1swdd3b3bxpacza7upcl3xizvps2ndmgnqihfqmi
Notary Requested: Broz221

@terry62326794 Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Hi. I have sent you E-mail to get more private detailed information. Please check.

Ok，I have sent it back.

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzaceaqommzqu5fgrwfyi4guc7fpdejrgbqoaxfrnf5tnjy746jdhhvpw

Address

f1swdd3b3bxpacza7upcl3xizvps2ndmgnqihfqmi

Datacap Allocated

5TiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzaceaqommzqu5fgrwfyi4guc7fpdejrgbqoaxfrnf5tnjy746jdhhvpw


Option for number of significant digits - Calculator and Unit Conversion plugins for PowerToys Run

Description of the new feature / enhancement
When using the Calculator/Unit Conversion plugins in PowerToys Run, a very small number can end up not showing enough numbers for the result to be accurate, and can sometimes even be misleading.
Here's an example:

This image makes it look like 1 gram is actually equal to 0 megatonnes period, instead of a very small fraction of a megatonne.
.
Ideally, very small results would always display 0.00000.... something. It DOES cutoff the digits sometimes like here:

But it would be nice to have the option in the Calculator/Unit Conversion plugins to set how many digits should be displayed before cutting off the result.
Scenario when this would be used?
I often deal with small numbers, so this can be a problem for my calculations/unit conversions when using PowerToys Run
Supporting information
No response

Duplicate of #2265

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


ENH skglm import time is high

In [1]: import time

In [2]: t0 = time.time()
   ...: import skglm
   ...:
   ...: print(time.time() - t0)
0.7723352909088135

I thought this was due to numba compilation, but it's not fixed on #44. importing Lasso from sklearn.linear_model takes 1e-5 s, that is weird

I think it has to do more with importing our dependencies, namely, numba, sklearn, scipy among others.
Through the example below I managed to reduce the import time to 0.15 on my local machine.
import time

start = time.time()
import scipy
end = time.time()
print(f"time to import scipy: {end - start}")

start = time.time()
import sklearn
end = time.time()
print(f"time to import scikit learn: {end - start}")

start = time.time()
import numba
end = time.time()
print(f"time to import numba: {end - start}")

start = time.time()
import skglm
end = time.time()
print(f"time to import skglm: {end - start}")

Ouput
time to import scipy: 0.16607046127319336
time to import scikit learn: 0.4579463005065918
time to import numba: 0.2870755195617676
time to import skglm: 0.14461469650268555

Difficult to say. Prefer to use -X importtime flags in Python interpreter for a better visualization of import times


An example for Labstack echo

Update documentation to include a straight forward method for Gin/Echo users to use this library.

There are examples in each adapter, e.g. https://github.com/jakecoffman/crud/blob/179960a3da81074f5cb66db7e27f1c8efeb76334/adapters/echo-adapter/example/main.go
Maybe I should mention that in the readme!


nyx: use python3 by default

Motivation for this change
https://pythonclock.org/
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

cc @offline.


Unable to connect to Windows 10 machines over RDP since v2.1.0 upgrade

Describe the bug
Since upgrading to v2.1.0 I've been unable to connect to any of my Windows 10 machines via Remmina.
Log is here: https://gist.github.com/nlyan/753897188a4dab850c8f42682a476681
This seems to be related to https://github.com/FreeRDP/FreeRDP/issues/5491
To Reproduce
Unknown
Environment (please complete the following information):
Arch Linux
[andrew@ziggy ~]$ xfreerdp --version
This is FreeRDP version 2.1.0 (n/a)
[andrew@ziggy ~]$ openssl version
OpenSSL 1.1.1g  21 Apr 2020`

Confirmed rolling back to FreeRDP 2.0.0 solved the issue:
https://gist.github.com/nlyan/69c1a756a3169d5c8d9536ef473686d5
Remmina was compiled againt 2.0.0 it seems, if that makes any difference.

Duplicates #6148
Currently not entirely sure where the issue resides (certificate or the check) as we actually widened the allowed certificate purpose (ANY in contrast to SERVER in previous versions)


Added new Ruby version to .travis.yml and fixed 2 rubocop issues in specs.

Added 2.2 to .travis.yml to build with Ruby 2.2.
Fixed 2 rubocop issues in specs.

new pull request

Appears not interested in PR so closing. Good Luck with the project.


Feature: add limit and skip_rows to IPC/Parquet Reader

The ParquetReader already has a limit argument.
This would allow users to process files larger than memory in multiple chunks.

Something like this: https://github.com/jorgecarleitao/arrow2/pull/609 ?
I am a bit uncertain what you mean with process files larger than memory: isn't this supported via the iterator API by not collecting all the record batches in one go?
It is true that decompressed IPC supports more exotic access patterns, but with compression the best we can do is to seek across record batches.

I am a bit uncertain what you mean with process files larger than memory: isn't this supported via the iterator API by not collecting all the record batches in one go?

It could be that I don't understand the iterator API correctly. With an IPC I often seem to get a single RecordBatch? (which is exactly what I want if I want to read all, because I don't have to rechunk). Can the size of the recordbatches be controlled?

An IPC file is basically:

magic bytes
[Dictionary messages]
RecordBatch[0]
RecordBatch[1]
RecordBatch[N]
footer
4 bytes with footer len (i32 little endian)
magic bytes

The footer contains the schema and a vector with the the position and length (in bytes) to the beginning of every RecordBatch, as well as the number of rows on each of them.
So, we can offset and limit the reading. If the IPC file has a single record batch, we can also do each of them, but we do need to decompress before doing so. For large batches, this does yield a lot of gains on the used memory.
#609 is just performing a post-allocation limit; for a limit within batch, we need to change the deserialization code (it is mostly mechanical changes, but still).


Condition for determining file vs folder is unsafe

Many files don't have extensions (and some folders contain dots).

When migrating to the new API I changed this behavior. It now uses the Python built in functions to detect if its a file or folder.

That's actually not safe either I think. In case it's the API I think it is it guesses based on the name.


Differences using PluginManager as GE plugin and standalone application

Comparing the GE plugin and the standalone application I noticed some differences that I cannot explain:
Tab "Browse"
The GE plugin doesn't find anything:

While the standalone application finds some packages:

BTW: What exactly does it search for? The search field is empty.
Tab "Installed"
The GE plugin lists itself.

The standalone application lists nothing.
Tab "Updates"
The GE plugin crashes:

The standalone application does not crash, but remains empty (I guess that's ok, since I do not have any plugins that might be updated).

There are differences in startup arguments for standalone and GE integrated instances.
Browse
GE - Filters only packages with dependency on GitExtensions.Plugins (which is a kind of meta package to mark packages as GE plugin). There is also a "bug" when using nuget.org (#67) as GE.Plugins is not yet pushed there and nuget.org requires dependencies to exist. So in GE mode with nuget.org nothing is found, but if you change the source to https://www.myget.org/F/neptuo-gitextensions/api/v2 plugins will appear.
Standalone - As search box is empty, it shows a head of the nuget.org feed. No additional filters.
Installed
GE - Startup arguments contains a --selfpackageid to support self update. With this parameter PM is able to override itself when new version is available.
Updates
Bug #70.

This would be a great addition to the readme

@RussKie I'm going to integrate it.

Thank you for the explanations.

GE - Startup arguments contains a --selfpackageid to support self update. With this parameter PM is able to override itself when new version is available.

Is it the idea to keep the PluginManager as a plugin itself?
Wouldn't it be more robust to have the PM tightly bundled to GE, probably even as part of GE's code so that everything gets compiled and installed together?

This may change in the future, but right now PM is designed as a plugin and also will be distributed as pre-installed plugin in the upcoming GE release.
As it is still in preview, it enables to update separately.
We have discussed the option of integrating source code into the main GE repository and the long term vision is to separate plugins from the main repository and make them features to install.
PM may be different and one time it may the only pre-installed plugin. Than it may be the right choice to disable the self update. What's positive is that it will be a single-line change :-)

I'm going to close this one. If you have any further questions, feel free to reopen it.


can't add ems, or eps images

it would be nice to have the ability to insert *.emf images.

Hi all,
was there any followup or update to this?
I'm also facing this use case:  how to include vector graphics (my specific use is plots generated by matplotlib) in a vector form rather than as a bitmap image form?
In my case (matplotlib) I could generate eps, svg or png vector forms of a plot.  I tried looking to see if it's possible to add any of those, but couldn't see any way to (...excuse me if I've missed it!)
As the original poster stated, this has tremendous output quality plus file size benefits, and I imagine it's a not-uncommon use-case for people generating doc's in python.
thanks,
Richard

just wanted to Update this Issue


Gulp is not running tasks in parallel

Hello,
I'm new to this gulp and have searched enough here for the problem I am looking at.
Problem:
I have a gulp task that I want run for different regions (against an array).
I created a function to create stream and do the job. I created a gulp task which calls this function in a loop against the array elements.
Some of the tests in the task (stream) takes lot of time, so I want to run it in parallel.
The idea is, while I want my gulp task to run in parallel, I also want to generate a report later and consolidate all the results in a table.
While i searched for answers on this forum I see people want to run it in sequence, and I have a contract that gulp is not running in parallel for me.
Here is how my gulpfile.js looks like:-
//function to create stream of task
function regionalSuite (awsRegion, done) {
      global.awsRegion = awsRegion;
      console.log("\n\n Tests running for Region :: ", awsRegion);
      var stream = gulp.src(['test/apis/**/*_spec.js'])
               .pipe(plugins.mocha({
              reporter: 'mochawesome',
              reporterOptions: {
                reportDir: './coverageReport/allRegionsSuite/mochawesome/'+ awsRegion
              },
              require: ['./tps-common/index.js'],
              timeout: 100000000
          }));

       stream.on("end", function(){
         done();
       });
};

//gulp task to run all tests in all aws Regions
gulp.task('test:allRegionsSuite', ['pre-test'], function (done) {
    let i = 0;
    let total = allRegions.awsRegionList.length;

    function _done() {
        if (i < total) {
            regionalSuite(allRegions.awsRegionList[i], _done);
            i++;
        } else {
            done();
        }
    };
    _done();
});

//gulp task to call 'test:allRegionsSuite' task and then generate consolidated report later.
gulp.task('testAllRegionWithReport', ['test:allRegionsSuite'], function (done){

    var mytable = new Table({
        head: ['Region','Duration', 'Passed', 'Failed', 'Pending', 'Total' ]
        , colWidths: [20, 10, 10, 10, 10, 10 ]
        });

     generateReport(mytable);

  });

Expected:
I want each of the gulp task for each awsRegion to run in parallel but currently its running in parallel.
is my loop making it slow?
Using
gulp version : 3.9.1
npm version : 3.8.3

This is not a support forum. Please direct support questions to StackOverflow.


Windows Python 2.7 path error

Summary:
Using python 2.7 and Subliminal 2.0rc1 on Windows causes either a path error if passed a non-unicode path or a Unicode error if passed unicode.
Minimum steps to reproduce:
Use the library code from https://github.com/Diaoul/subliminal, enable logging, and change the '/video/folder' to a path containing a unicode character such as ō.
Traceback (most recent call last):
  File "C:/src/indexer/testsubs.py", line 17, in <module>
    videos = [v for v in scan_videos('G:\\Shōws') if v.age < timedelta(weeks=2)]
  File "build\bdist.win-amd64\egg\subliminal\core.py", line 458, in scan_videos
ValueError: Path does not exist

With unicode path:
DEBUG:rebulk.rules:Checking rule condition: <TitleFromPosition>
Traceback (most recent call last):
  File "C:\Python27x64\lib\logging\__init__.py", line 853, in emit
    msg = self.format(record)
  File "C:\Python27x64\lib\logging\__init__.py", line 726, in format
    return fmt.format(record)
  File "C:\Python27x64\lib\logging\__init__.py", line 465, in format
    record.message = record.getMessage()
  File "C:\Python27x64\lib\logging\__init__.py", line 329, in getMessage
    msg = msg % self.args
  File "C:\Python27x64\lib\site-packages\rebulk\match.py", line 781, in __repr__
    return "<%s:%s%s%s%s%s>" % (self.value, self.span, flags, name, tags, defined)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc5 in position 5: ordinal not in range(128)

This works just fine using Python 3.

Thanks for the very detailed report, I reported the second issue upstream to rebulk (https://github.com/Toilal/rebulk/issues/3)
As for the first one, the path is tested using os.path.exists so make sure you pass bytes encoded with the proper encoding (same as your filesystem, CP1252 I guess).
I guessed you saved your script file as utf-8 so the encoding of 'G:\\Shōws' is utf-8. Try saving your file as CP1252.
A cleaner solution would be to work with unicode paths and encode them with your file system encoding when using it.
path = u'G:\\Shōws'
videos = [v for v in scan_videos(path.encode('cp1252')) if v.age < timedelta(weeks=2)]

An even cleaner solution would be to work only with unicode paths and never encode them but it doesn't work due to a bug as you reported.

@Diaoul The problem is on windows the console uses cp1252 (or other relevant Windows encoding depending on user settings), but the actual path on Windows is stored as unicode.  For example in Windows console G:\Shōws is actually displayed as G:\Shows and in explorer it shows correctly as G:\Shōws.  (More specifically Windows uses utf-16-le for NTFS, but the standard libraries work with the path as unicode)
Using cp1252, ascii, utf-16-le (the "actual" Windows encoding), or any other relevant encoding os.path.exists returns false.  Using unicode os.path.exists returns true.
The most reliable way on Windows would be to use unicode.  The only other option I'm aware of would be to use the "short" path name (e.g. os.path.exists('G:\SHWS~1') returns True).  The problem with the short path name is it requires importing the win32api to get it and it mangles the name, which subliminal needs to determine the desired subtitle.
In addition passing the short file path causes other issues.  For example scan_videos uses os.path.walk which doesn't work properly with unicode contents on Windows unless the path passed to it is unicode.  So when you build filepath (core.py L486) it joins the short filename with the console representation of the video file (e.g. cp1252) causing ValueError: Path does not exist for contents with unicode in the names.
I'm not sure if these are the only issues caused by this, but it should at least give you some ammunition to approach this.

Oh I forgot to add that using mbcs for encoding type also doesn't work.

I've configured the whole environment on Windows to fix the guessit part of the issue. It seems OK now, i'll commit and release soon.
But I still have an error related to legendastv provider:
ERROR:subliminal.core:Unexpected error in provider 'legendastv'
Traceback (most recent call last):
  File "C:\idea-projects\subliminal\subliminal\core.py", line 118, in list_subtitles_provider
    return self[provider].list_subtitles(video, provider_languages)
  File "C:\idea-projects\subliminal\subliminal\providers\legendastv.py", line 408, in list_subtitles
    return [s for l in languages for s in self.query(l, title, season=season, episode=episode, year=video.year)]
  File "C:\idea-projects\subliminal\subliminal\providers\legendastv.py", line 312, in query
    titles = self.search_titles(sanitize(title))
  File "C:\Python\27-subliminal\lib\site-packages\dogpile\cache\region.py", line 1045, in decorate
    key = key_generator(*arg, **kw)
  File "C:\Python\27-subliminal\lib\site-packages\dogpile\cache\util.py", line 84, in generate_key
    return namespace + "|" + " ".join(map(to_str, args))
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf6' in position 1: ordinal not in range(128)

I'm running the README script too, in a folder named C:\Shöws containing a dummy movie named töst.mkv.

Seems it's related to python2: https://bitbucket.org/zzzeek/dogpile.cache/issues/94/unicode-support-in-function_key_generator
Here is the relevant documentation: https://dogpilecache.readthedocs.io/en/latest/api.html#dogpile.cache.region.CacheRegion.cache_on_arguments.params.to_str

This is not a bug (yet) as this is due to a hypothetical series name with non-ascii characters searched on Addic7ed provider. I'd rather fix this with an actual real use case.

I've tried to fix this but a bunch of other issues happen when dealing with non-ascii series name and none of them are easy to fix.
I suggest moving to python 3 if encoding is a real issue.

@Diaoul Actually its reproducible with any series having a non-ascii name.  This started with a user unable to search the series Sekkō Boys.  Also while moving Medusa to python 3 is in progress, we still have to support python 2 on devices which may not have python 3 available such as some NAS's.

Well, I'm not gonna fix python2 :)
I'll do my best as long as it stays non-intrusive but I won't invest too much effort into this given that python3 is a future-proof solution.
Please open a separate issue as this one is solved so we can keep things clear.
@Toilal: please tell me when you make a new release

Rebulk has been released, so user only have to perform a pip install --upgrade to upgrade rebulk.


Native Filter can't work when face NULL value or empty string

Native Filter can't work when facing NULL value or an empty string, and this behavior is not consistent with explore page.
https://user-images.githubusercontent.com/2016594/159150669-a4cb27cf-9049-4caa-b983-b18ab5d69d1a.mov
How to reproduce the bug

make a temporary table with a single text column by database client or SQLLab

create table testing_empty_string(
  name varchar(255)
);
insert into testing_empty_string
values
  (null),
  (''),
  ('foo');

create a dataset from above table
create a chart and dashboard from above dataset
create a Native Filter from the dashboard
got error when select <NULL> on dashboard
got an empty placeholder when select <empty string> on dashboard
got an error when select <NULL> on explore

Expected results

Native filter should work with NULL
Native filter should show  when got a empty string
filter should work with NULL on explore page

Actual results
Native filter can't work when facing NULL or Empty string
Environment

browser type and version: latest Firefox
superset version:  current master branch(d645579cd)

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Repro'd, thanks for reporting!

Fixed by https://github.com/apache/superset/pull/19326


Refactoring DataFunctionArgs to LoaderFunctionArgs and ActionFunctionArgs

Hi Kent,
I've noticed that The Epic Stack is utilizing DataFunctionArgs for the loaders and actions arguments, while the Remix documentation now advises using LoaderFunctionArgs and ActionFunctionArgs. If you think updating this would be beneficial, I'd be happy to assist.
The action plan involves identifying all instances where DataFunctionArgs is used, replacing it with the appropriate type, and updating the imports as needed.
P.S.: I've already located these instances. My only uncertainty is regarding the file entry.server.ts, where DataFunctionArgs is employed to type the handleError function. I believe LoaderFunctionArgs would be a suitable replacement here.

This will need to be addressed before Client Data comes to the Epic Stack, and since it's in unstable pre-release now I think it'd be a good time
Regarding handleError, I think we can actually delete it? It doesn't appear to be used https://github.com/search?q=repo%3Aepicweb-dev%2Fepic-stack handleError&type=code
https://github.com/remix-run/remix/discussions/7634#discussioncomment-7256034

I'm really bummed about this decision by Remix, but whatever it's fine.
handleError is used by remix: https://remix.run/docs/en/main/file-conventions/entry.server#handleerror
Anyone can feel free to make this change. PRs welcome.

handleError is used by remix: https://remix.run/docs/en/main/file-conventions/entry.server#handleerror

Ah makes sense, then @diegotc86 as documented there you can use the type LoaderFunctionArgs | ActionFunctionArgs for that function


Add state management to ui example in readme

remove dollar signs from comms and flutter_comms installation step

add Sending messages from State Management to UI section to flutter_comms readme

Closes #69

@bartekpacia .


Restart L2 agent when OVS is restarted

refs:

https://gerrit.mcp.mirantis.com/c/packaging/sources/neutron/+/198875
https://docs.openstack.org/neutron/latest/ovn/ovn_agent.html

Story created: https://storyboard.openstack.org/#!/story/2011258

Q: Can this by ported to OVNAgentExtension?

RFE: https://bugs.launchpad.net/neutron/+bug/2086776

Related Meeting logs:

https://meetings.opendev.org/meetings/networking/2024/networking.2024-11-12-14.00.log.html


fix modify template structure

Signed-off-by: liuying<EMAIL_ADDRESS>
What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #47
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f3f19c0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #48   +/-   ##
=========================================
  Coverage          ?   47.46%
=========================================
  Files             ?        6
  Lines             ?      552
  Branches          ?      152
=========================================
  Hits              ?      262
  Misses            ?      227
  Partials          ?       63


BUG: Get image geometry from GDCM Secondary Capture

Secondary capture images can have pixel spacing and image orientation, but the GDCM code explicitly ignored these values (whereas DCMTK code read them).
Per David Clunie (@dclunie) the editor of the standard, it was clarified in CP-586 that PixelSpacing is the correct place to store the best known value (e.g. one that has been corrected for distortion) while other fields like the NominalScannedPixelSpacing, which GDCM used before this commit, if stored at all should have the uncorrected values.
For these reasons this commit makes the default behavior of GDCM for SC match what it would do if SC images were MR or CT in terms of handling the image geometry, which is a better default than the previos behavior.
https://dicom.nema.org/medical/dicom/Final/cp586_ft.pdf
Fixes #4109

IMHO, the idea to read IPP/IOP/PixelSpacing from the SecondaryCaptureImageStorage is good, there are many such data sets (they can be considered Extended Secondary Capture), but it should be done in GDCM or documented that the changes have to re-applied after every GDCM update.

ImagePosition (Patient) should be updated here, otherwise it will be 0,0,0 as before:
https://github.com/InsightSoftwareConsortium/ITK/blob/cdbbdf9441dab496196ba6843eb4a1c2d441e449/Modules/ThirdParty/GDCM/src/gdcm/Source/MediaStorageAndFileFormat/gdcmImageHelper.cxx#L580

The PR will not enable Pixel Spacing (0x0028,0x0030) for MediaStorage::SecondaryCaptureImageStorage, actually it can  done by setting ImageHelper::ForcePixelSpacing to true, s. here. Change of MediaStorage::MultiframeTrueColorSecondaryCaptureImageStorage spacing is not related to MediaStorage::SecondaryCaptureImageStorage.

it should be done in GDCM or documented that the changes have to be re-applied after every GDCM update.

Very good point!  I wasn't sure how ITK interacts with the upstream repository and mostly wanted this change to go into Slicer's fork, but yes, coming up with the best conforming solution in GDCM makes good sense and then it can work its way into ITK and then Slicer.

Probably it should work if both spacing attributes will be read for SC Image and (0028,0030) will have higher priority, if both are available.

Yes, I interpreted CP 586 as saying that PixelSpacing is the preferred way to express the true value.  It might make sense to have a fallback to look for one of the other spacing tags if PixelSpacing is missing, but it felt like an obscure corner case and I didn't see any easy way to restructure the code since I'm not familiar with it.
I agree with your other points @issakomi  and would be happy to see a better implementation of this that covers more ases.  Let's see if @malaterre or others have suggestions on how best to approach this.

S. https://github.com/malaterre/GDCM/pull/158/files

Superceded by #4521


ISPN-8344 DSL queries filtering only on type are always executed with…

…out index
https://issues.jboss.org/browse/ISPN-8344

Needs rebase to be able to run CI

44 failures, lots of BindException(...) due to port conflicts, they don't seem related, running one more build

Now 39 failures, some of them in query

@anistor, looks like the 8.2.x branch is kind of broken

This, or 8.2.x without these commits?

@anistor It seems the failures in this PR are already in 8.2.x, I can integrate this, but would not release 8.2.9.Final before sorting out those failures

merged, thanks @anistor !

Thanks @gustavonalle ! Will postpone 82x then.


[codec,progressive] revert 4e24b966c86fdf494a782f0dfcfc43a057a2ea60

do not reset progressive surface numUpdatedTiles after progressive_decompress pass. The updates might accumulate until the frameId changes, only then reset.
fixes the long standing graphical glitches when moving windows.

@freerdp-bot would you please test again ?


Add Semaphore ABI to utils package

Description
Since the data package and the subgraph app need the Semaphore contract interface (ABI), that JSON file must currently be copied manually every time the contract changes. The ABI could be automatically copied into the utils package, which can, in turn, be used by data and subgraph.
Contract interfaces are generated when the yarn compile command is executed in the packages/contracts folder. A script could be run immediately after compilation to copy the file artifacts/contracts/Semaphore.sol/Semaphore.json to packages/utils/src/semaphore-abi.json. Then all packages/apps that need the interface can import it from @semaphore-protocol/utils.
Additionally, all devs or external apps that need the Semaphore ABI will be able to import it directly from the @semaphore-protocol/utils package.

@cedoor can I be assigned this?

@Dozie2001 of course!


openDKIM key length

This is a wish for a feature:
Some DNS servers restrict the length of TXT fields.
The opendkim default key length seems to have been increased from 1024 to 2048 bits. In my case, the DKIM entry now does not fit the TXT field anymore.
I now would like to have an option to shorten the key length back to 1024 bits to make it fit. I understand that it of course would be better to keep the longer keys and fix the problem on the DNS side, but my provider seems unable to do so.
What about a corresponding environment variable, like DKIM_KEY_LENGTH ?

I have the same problem, I cannot past all the key (about 16 digit missing)

Can you provide a PR?

This is a duplicate of #786 .
In fact debian uses key size length of 2048 as default since January 2017(!) Those bloody debians living on the edge ;)

opendkim (2.11.0~alpha-9) unstable; urgency=medium

Set umask to 0007 in opendkim.service so opendkim socket is group readable
(Closes: #851141)
Set opendkim-genkey to default to 2048 bit sha-256 only keys based on
current best practices
Add debian/patches/nsupdate_output.patch to fix opendkim-genzone output to
be usable for nsupdate (Closes: #849540)
-- Scott Kitterman<EMAIL_ADDRESS>Wed, 25 Jan 2017 09:58:07 -0500

can anyone (@cruisemaniac, @youaresofunny, @leokun) provide a PR accompanied by a small test for the size generated once as 1024 and once as 2048 Bit key?

#868 is merged. You can now generate with any key size


Support JogCon position

Need to sniff the protocol while playing some Ridge Racer 4 and figure out how to enable JogCon mode. Without it, it just shows up as a standard controller.

Done in version 0.2


CI: Update Codecov Action

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.96%. Comparing base (c5f7849) to head (b71233b).
Report is 5 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   67.44%   67.96%   +0.52%
==========================================
  Files           1        1
  Lines        1118     1130      +12
==========================================
+ Hits          754      768      +14
+ Misses        364      362       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Connector fails to start due to CLASSPATH issues on current master [MACOS]

Hi,
When using the current master with the instructions from the readme, the connector does not start on MacOs. It seems to be specifically due to the way that
mkdir bin/jar/ && tar -C bin/jar/ -xf kcbq-confluent/build/distributions/kcbq-confluent-*.tar
extracts the content into bin/jar/kcbq-confluent-1.1.0-SNAPSHOT/ instead of bin/jar directly? The exception thrown is:
[2018-09-28 12:21:32,123] ERROR Stopping after connector error (org.apache.kafka.connect.cli.ConnectStandalone:100)
java.util.concurrent.ExecutionException: org.apache.kafka.connect.errors.ConnectException: Failed to find any class that implements Connector and which name matches com.wepay.kafka.connect.bigquery.BigQuerySinkConnector available connectors are: io.confluent.connect.hdfs.HdfsSinkConnector, org.apache.kafka.connect.source.SourceConnector, org.apache.kafka.connect.tools.VerifiableSourceConnector, org.apache.kafka.connect.file.FileStreamSinkConnector, org.apache.kafka.connect.file.FileStreamSourceConnector, org.apache.kafka.connect.tools.VerifiableSinkConnector, io.confluent.connect.hdfs.tools.SchemaSourceConnector, org.apache.kafka.connect.sink.SinkConnector, io.confluent.connect.jdbc.JdbcSourceConnector
	at org.apache.kafka.connect.util.ConvertingFutureCallback.result(ConvertingFutureCallback.java:80)
	at org.apache.kafka.connect.util.ConvertingFutureCallback.get(ConvertingFutureCallback.java:67)
	at org.apache.kafka.connect.cli.ConnectStandalone.main(ConnectStandalone.java:97)
Caused by: org.apache.kafka.connect.errors.ConnectException: Failed to find any class that implements Connector and which name matches com.wepay.kafka.connect.bigquery.BigQuerySinkConnector available connectors are: io.confluent.connect.hdfs.HdfsSinkConnector, org.apache.kafka.connect.source.SourceConnector, org.apache.kafka.connect.tools.VerifiableSourceConnector, org.apache.kafka.connect.file.FileStreamSinkConnector, org.apache.kafka.connect.file.FileStreamSourceConnector, org.apache.kafka.connect.tools.VerifiableSinkConnector, io.confluent.connect.hdfs.tools.SchemaSourceConnector, org.apache.kafka.connect.sink.SinkConnector, io.confluent.connect.jdbc.JdbcSourceConnector
	at org.apache.kafka.connect.runtime.Worker.getConnectorClass(Worker.java:226)
	at org.apache.kafka.connect.runtime.Worker.startConnector(Worker.java:166)
	at org.apache.kafka.connect.runtime.standalone.StandaloneHerder.startConnector(StandaloneHerder.java:250)
	at org.apache.kafka.connect.runtime.standalone.StandaloneHerder.putConnectorConfig(StandaloneHerder.java:164)
	at org.apache.kafka.connect.cli.ConnectStandalone.main(ConnectStandalone.java:94)

When I add that specific directory to my classpath manually (export CLASSPATH="$CLASSPATH:$BASE_DIR/../../bin/jar/kcbq-confluent-1.1.0-SNAPSHOT/*") , the connector comntinues, but fails with a different error:
Caused by: org.apache.kafka.common.config.ConfigException: Invalid value com.wepay.kafka.connect.bigquery.schemaregistry.schemaretriever.SchemaRegistrySchemaRetriever for configuration schemaRetriever: Class com.wepay.kafka.connect.bigquery.schemaregistry.schemaretriever.SchemaRegistrySchemaRetriever could not be found.
Is the above a known issue, or am I doing something wrong? Thanks in advance!

Hi,
No I've just took the Confluent distribution locally on my mac and ran it from there. Version 1.1.0 seems to work however, not sure why though.
Cheers,
Patrick

how resolve this?
i have same issue


No warning from missing function

Now running plugin 1.4.1 with Dart 1.10.0.
I have a single test.dart file:
main(List<String> args) async {
	assert(true);
	print('hi');
	await foo();
	try {
		// stuff
	} catch {
		notHere();
	}
}

foo() {}

which lives on my desktop.
Expected: a warning that notHere() doesn't exist.
Actual: no warning.
Screenshot:

(see "unknown include #regex.dart" in console... related?)

Here's what webstorm shows:

It may be that files under "test" are ignored altogether by the DAS.

I mean, in the Dart plugin.

hm, the file 'test.dart' isn't under test. It's in the Desktop directory.

Yes, it must be the plugin's fault, but I was thinking maybe the plugin makes the DAS ignore files under "test" in a pub package. Not sure at all, though. Need to look into it.
Is your file in a pub package folder structure?

My test.dart file is literally inside of the Desktop directory. It's the only file in the Desktop directory.

Thanks. If you get no errors shown at all for broken code, then probably a single-file scenario like this one is unsupported atm by the plugin. I think it always checks for a pubspec.yaml and ignores files if it doesn't find one.

Thanks for checking it out. The sublime plugin should work for a single stand-alone file, if that file is a library (not a part).

Hi,
We just ran into this scenario with an internal user. We wanted to open Sublime, create a new Dart file, and analyze it. We didn't have a pubspec.yaml file.
What do you need from us to make this scenario work? :)
Thanks!!

I seem to remember in the very beginning we decided against it. But can't really say for sure any more.
The plugin should be modified to allow this. I don't think it will be too much work, but I havent' gotten around to it yet.

Perhaps the issue is... how do we know if we're in a Dart project?
If I have an empty directory, with a single Dart file (which is an entry point or a library file (not a part file)), I would want to treat that as a "Dart project".
But, this would require us to assume a bunch more about the environment. So I'll start a thread on analyzer-discuss@ to see what other people think.

Some logic and ideas for how to detect a Dart project. https://groups.google.com/a/dartlang.org/forum/#!topic/analyzer-discuss/HVLgtVXyB18

In the meantime, I updated https://github.com/guillermooo/dart-sublime-bundle/wiki/Installation-and-Basic-Configuration with explicit instructions to add a pubspec.yaml file. Hope that helps!

Just tried with a sindle main.dart file inside an otherwise empty folder and everything seems to be working (using repo's head, not latest release). I don't remember changing anything related to this scenario.

Thanks!
On Sat, Jun 13, 2015 at 17:51 Guillermo López-Anglada <
<EMAIL_ADDRESS>wrote:

Closed #525
https://github.com/guillermooo/dart-sublime-bundle/issues/525 via
9f565bf
https://github.com/guillermooo/dart-sublime-bundle/commit/9f565bf1366e760c8d8a2ccf5738f0f1745136bc
.
—
Reply to this email directly or view it on GitHub
https://github.com/guillermooo/dart-sublime-bundle/issues/525#event-330220833
.


Update CHANGES and NEWS for 1.1.1h release

Citing pull request #11346:

Since the grand markdown revamping in #10545, the CHANGES and NEWS files contain notes for all releases, so they need to be updated, too.
As long as there is only one stable branch, I can take care of it every time a release has happened.
Note: For 3.0 and beyond, the ideal solution would be to update the CHANGES and NEWS files on master only, and backport the relevant sections to the respecitve stable branches using a script when creating the release. But that's a task for a separate pull request in the future...
cc @mattcaswell

Since then, the markdown versions on the master branch haven't been updated, which was reported by Michael Wojcik on openssl-users. This is fixed by the first commit.

Author: Dr. Matthias St. Pierre<EMAIL_ADDRESS>Date:   Thu Sep 24 07:58:52 2020 +0200
Update CHANGES and NEWS for 1.1.1h release

While working on it, I adjusted the CVE link style, see second commit.

Author: Dr. Matthias St. Pierre<EMAIL_ADDRESS>Date:   Thu Sep 24 08:11:00 2020 +0200
Change CVE link style in CHANGES and NEWS

Replace [collapsed reference links][] for the CVEs by
[shortcut reference links], in order to to improve the
readability of the raw markdown text.

Consistently add parentheses around the CVE links at the
end of the CVE descriptions. (The NEWS file already had
the parentheses, in the CHANGES file they where missing.)

[collapsed reference links]:
  https://github.github.com/gfm/#collapsed-reference-link

[shortcut reference links]:
  https://github.github.com/gfm/#shortcut-reference-link

@mattcaswell when and how does the changelog on the OpenSSL website get updated?

@mattcaswell when and how does the changelog on the OpenSSL website get updated?

I think it gets updated automatically whenever we do a release. Or we can make it happen manually.

Unsure why this is but when downloading the 1.1.1-h release from the openssl website and building from the download the version that is built is not 1.1.1-h
The version built from 1.1.1-h download is:
OpenSSL 1.1.1b  26 Feb 2019

@gregbown there are in general two ways how you can end up with the wrong openssl version: you either executed the wrong openssl executable, or the excutable loaded the wrong openssl libraries.
Please check whether you executed the correct binary
which openssl

and whether your binary loaded the correct shared libraries:
ldd $(which openssl)

Note: a version mismatch between application and library would be detected automatically by openssl version and displayed as follows:
OpenSSL 1.1.1h  22 Sep 2020 (Library: OpenSSL 1.1.1d  10 Sep 2019)

BTW: this pull request is still waiting for some review @openssl/committers.

@mspncp Regarding building 1.1.1-h Thank you for taking the time to educate me regarding inspecting the binary that was loaded. Ironically my system was loading the version that came with the latest Perl installation that I installed to build openssl.  Than you again

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master, thank you for the review!

d8dc853825 Change CVE link style in CHANGES and NEWS
6ffc312776 Update CHANGES and NEWS for 1.1.1h release


Re-Introduce support for RSA signatures

If I understand correctly, the TLS 1.3 prototype does currently not support RSA signatures, which isn't because there's any particular issue with it or its relation to TLS 1.3, but simply to accelerate the initial development of the prototype - @hannestschofenig correct me if I'm wrong.
Now that we're getting closer to upstreaming it, support for RSA should be reintroduced.

Related to #92 by @lhuang04.

Related to #92 by @lhuang04.

For example, https://github.com/hannestschofenig/mbedtls/blob/tls13-prototype/library/ssl_tls13_generic.c#L1843 needs changing because it infers the signature algorithm from the key bitlength. This is fragile and should be replaced by a switch on the actual certificate key type.

For example, https://github.com/hannestschofenig/mbedtls/blob/tls13-prototype/library/ssl_tls13_generic.c#L1843 needs changing because it infers the signature algorithm from the key bitlength. This is fragile and should be replaced by a switch on the actual certificate key type.


Remove pry package

This is a debugging tool that should be removed from the Gemfile.

Moved this under development group.


added dkms documents

Added dkms requirements docs in .pdf format.

This pull request adds pdf documents to the repo that were formerly stored in google docs and linked to in the main dkms document. Adding them will enable us to update the links in the main dkms document so they no longer point to google drive.

@brentzundel Are you sure that we can store here documents with "Evernym" name? What is the license?

the PR approval and merging process is at best obscure and difficult to follow, at worst deliberately obtuse and byzantine. Documentation of the way the process is supposed to work would be extremely beneficial.

This PR has been superseded by https://github.com/hyperledger/indy-sdk/pull/649


adding test for CTFE strings

Ok I going to test this. Thank you.

Ok merged the other PR. Can you integrate this into an existing test ?

let me check where it'll fit.

could not find a place to stick it in

You can certainly tweak some of the existing test that use string ctfe to hit the right codepath. Also, there is nothing actionable in "I couldn't find a place to stick it in". If you don't tell the problem you face, you'll end up with PR in limbos.

We have NO tests that use CTFE strings!
it is completely uncovered!

There are various test that use mixin.

all the mixins are preformed on STATIC strings.
there is no operation going on ... as conact is not supported yet.

That is why you need to add this in one of these test so that are not STATIC anymore and go through the faulty codepath.

gotcha

@deadalnix  there you go!

You know what, I merged you diff, I shouldn't have.

This does not test the actual code it is supposed to test.

IT DOES TRIGGER!

revert my patch.
run the test.
see the light.


CLientApp folder is not created for the SPA option

Expected Behavior
An empty ClientApp folder will be created for npm to play with
Actual Behavior
No folder is created
Steps to Reproduce the Problem

Run command line for template as per doco

Specifications

Version: 2.6.0
Platform:
Subsystem:

If you create the folder, even if you put in the two files in the template on github or not you then get the error:

npm ERR! enoent ENOENT: no such file or directory, open 'D:\Xxx.Web\ClientApp\package.json'


Implementation of internal DSC resource WaitForSome

In PsDesiredStateConfiguration v2.0 resource WaitForSome is no longer supported (see https://learn.microsoft.com/en-us/powershell/module/psdesiredstateconfiguration/invoke-dscresource?view=dsc-2.0#notes and https://learn.microsoft.com/en-us/powershell/dsc/overview?view=dsc-2.0) as it is a cross system resource. A workaround is needed to have full compatibility with mof configurations built with Windows Powershell.
A possible solution to the issue is implementing in Powershell an alternative resource (inside the PSDSCAgent module) - with a different name but same behavior - and have the process applying the configuration recognize old WaitForSome resources and call the internal one instead. This way the solution is self-contained.

Documentation


Add loadTracher.sh and fix init and append of rpd file

loadTracer.sh will init rpd file and wait for for profiling to start.
runTracer.sh will init rpd file and start profiling right away

Great fix, thanks.


Qemu emulation: kernel time and hw timers are not synchronized

Describe the bug
I am experiencing strange behaviour when using the Counter device with qemu_cortex_m0.  I have experienced two, possible related, bugs.

Kernel time and Counter device is not advancing at correct rate.
The kernel ticks are supposedly with 1MHz and the Counter device at 16MHz, still I am only seeing a 8x different in advanced time as measured with a program like this:

uint32_t k_now, c_now, k_now1, c_now1;
k_now = k_cycle_get_32();
res = counter_get_value(counter_dev, &c_now);
printk("start: Counter=%u, Kernel=%u\n", c_now, k_now);

k_now1 = k_cycle_get_32();
res = counter_get_value(counter_dev, &c_now1);
printk("Counter=%u Kernel=%u rate=%u\n", c_now1, k_now1, (c_now1-c_now)/(k_now1- k_now));

Counter device is not emulated properly when the system is sleeping.
I am running a very simple program where a callback is registered with overflows of the counter which prints the kernel time of that event:

uint32_t prev_overflow=0;
static void  overflow(const struct device *dev, void *user_data) {
    uint32_t now = k_cycle_get_32();
    printk("Overflow @ %u Delta=%u\n", now/1000, now-prev_overflow);
    prev_overflow=now;
}

int main() {
    // Configure an overflow callback
    top_cfg.callback = overflow;
    top_cfg.ticks = counter_get_max_top_value(counter_dev);
    res = counter_set_top_value(counter_dev, &top_cfg);
    if (res != 0) {
        printk("error");
        return 1;
    }

    // Start timer
    counter_start(counter_dev);
    while(true) {}
    return 0;
}

The overflow callback is called regularily every 8ms (measured using k_cycle_get_32).  This is wrong, but consistent with the what we observed in (1). The simulation speed is also reasonably fast.
However, if I insert a sleep in that while loop I get really strange behaviour.  By adding a small sleep of 1ms I know only get overflows invoked every 20 ms. If I increase the sleep til 100ms I get 110ms between each overflow.   Also, the simulation speed drops by maybe 10x
It seems as if the emulated Timer peripheral is not synchronized to the kernel time when we go to sleep. Or maybe the interrupts from the Timer are, for some reason, dropped when we enter sleep.

This should not be assigned to @anangl. Leaving it unassigned for now.

@carlescufi it was assigned to @anangl  because this qemu uses nrf timer AFAIK.

This part here needs investigating:
https://github.com/zephyrproject-rtos/zephyr/blob/main/boards/arm/qemu_cortex_m0/nrf_timer_timer.c#L238-L239

@erlingrj can you paste the whole reproducer you are using for the first bug?

@erlingrj can you try again with the fix in the https://github.com/zephyrproject-rtos/zephyr/pull/58753 PR?
I cannot precisely reproduce your use case because I'm not entirely clear which counter you are using given that by default the qemu_cortex_m0 board is exposing none.

This part here needs investigating: https://github.com/zephyrproject-rtos/zephyr/blob/main/boards/arm/qemu_cortex_m0/nrf_timer_timer.c#L238-L239

@carlescufi  qemu_cortex_m0 looks to be using the nRF51822 for emulation beneath the hood ... is someone at Nordic actively maintaining that upstream in QEMU to maybe give some context to the 1MHz comment?

This was hopefully fixed by #58753


AudioEngine at recorder side to mix audio

This Pull request adds a new AVAudioEngine to mix audio at recorder side. The mixed audio is given to SDK to send it out to the remote participants.

Added a new AVAudioEngine to record audio
New player and recording nodes.

Echo cancellation
Since the mixed audio is played and the mixed audio recorded by the same VP IO audio unit, the echo cancellation is not an issue.
TODOs

[ ] Readme update
[x] Diagram to demonstrate record mixing pipeline
[x] Fix audio buffer memory leak
[x] Audit existing comments, and write more comments/documentation in code
[x] Fix AudioPlayerNode leak

@ceaglest What do you think about following iOS 11 warnings -

@ceaglest What do you think about following iOS 11 warnings -

These APIs appear to be available on iOS 11 and later. Can we mark this class as supporting only iOS 11 and above, even if the project itself supports earlier versions?

@ceaglest Thanks for the detailed review. Merging the PR.


Add custom error pages

Context
We don't have GOVUK style error pages for these HTTP response codes in TTAPI.
https://trello.com/c/4PUK3wCt
Changes proposed in this pull request

Copy the ones from Publish, along with the routing and controller code.
Tweak the service name in the header so it makes sense across Support, New Publish, and these error pages.

Guidance to review

Should be fairly easy to trigger a 404 and 403 in the review app, a 500 probably easier to do locally
If testing locally, you'll need to set config.consider_all_requests_local to false in development.rb

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

Pretty sure the other PR covered this

Yes, covered here https://github.com/DFE-Digital/teacher-training-api/pull/2340