Fix bug where ReplicationRoles was exposing unsupported roles

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify
Update ReplicationRole entity to remove unsupported replication roles which were exposed as part of Server properties.

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.
Raised breaking change review https://msazure.visualstudio.com/One/_workitems/edit/17690672

Hi @ambrahma, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow breaking change policy to request breaking change review and approval before proceeding swagger PR review. Action: To initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki. If you want to know the production traffic statistic, please see ARM Traffic statistic. If you think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ms/swaggerfeedback. Note: To avoid breaking change, you can refer to Shift Left Solution for detecting breaking change in early phase at your service code repository.

Breaking change review raised here https://msazure.visualstudio.com/One/_workitems/edit/17690672


Hugo Website - Kubernetes Object Management problems

This issue is for tracking
Concepts->Overview->Object Management->Kubernetes Object Management

[ ] Feature Request
[X] Bug Report

Problem:
API reference has a bad link
https://k8sio-hugo-migration.netlify.com/docs/reference/generated/kubernetes-api/v1.10/

Missing link

Kubectl command reference
Kubernetes API Reference

Link to file example is incorrect.
https://k8sio-hugo-migration.netlify.com/docs/concepts/overview/object-management-kubectl/declarative-config/
simple_deployment.yaml docs/concepts/overview/object-management-kubectl instead of simple_deployment.yaml


Add identification of LAS in experiment config for conditional tasks

In the refactor, you can specify GameType (where GameType is an enum and includes the LAS value). GameType.LAS is only added if you use "create LAS game" when building your experiment, i.e., you can't convert a random info stage to GameType.LAS.


Include missing offer details in the offer details overlay

Motivation

All relevant offer details should be shown (that's why we are asking for them after all! 😄)
User Story

As a user, I want to get as much information as possible on each offer.
Proposed Solution

Add the following bits of information which we are already collecting, but not showing to users:

offer language
is the offer free?
group type
mode type
organization website
(optional) a map with a pin at the address; this would be nice-to-have (since seeing a location on a map is much more intuitive than reading an unknown address, then having to search for the address on maps yourself), however we could also move this to the backlog

Alternatives

Additional Context

Not sure if we need UI/UX for this - it's mostly just additional details lines in the left column.
Screenshots

Also resolved by #108


WIP corestore-networker for @next swarm and corestore

Managing connections with Hyperswarm v5 is easier, but I wanted to see if I can do it with the same API as the networker.
Managed to get the basics going, and will get to the extensions feature soon.
Note: I still can't get bootstrapping to work ('too few nodes' answers), so I will get to that in the unit tests soon too.

Evidently, this module won't be needed as much with Hypercore 10.


[1.29] repo: Dev v1.29.3

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

/retest

@adisuissa seems like this branch is a bit broken - but i think we should push this through anyway - we need to reopen the branch to do anything else


bitcoin connect should fire webln:enabled event when connector has successfully connected

This is not yet part of webln.guide?

resolved!

@pavanjoshi914 thanks for updating the webln.guide. We still need to make the change in Bitcoin Connect to actually fire this event


add the possibility to require or hint at kernel features

Issue description
I develop on a custom kernel so I like to keep its configuration tight to speed up compilation time.
I've recently been dealing with packages that wouldn't work without specific kernel features.
I would like to add to packages a field kernelFeatures or requiredLinuxFeatures.
mininet = stdenv.mkDerivation {
name="mininet";
...
requiredLinuxFeatures = [ "NETNS"  ]
}

When this package is added to configuration.nix it should check that the kernel config has "NETNS" enabled. I don't believe it's possible to enable it automatically yet since we don't know the dependencies between the different kernel features (setting CONFIG_NETNS should enable CONFIG_NET but we can't do that yet).
Checking is easier though since one can grep the config and look for NETNS value.
Hopefully someone can build the kernel dependency into the structured linux config  https://github.com/NixOS/nixpkgs/pull/12158

Does system.requiredKernelConfig fit your requirements ? https://github.com/NixOS/nixpkgs/blob/9f7eabcc21e1a2aa1fe834193bbd20b0c1d1208f/nixos/modules/system/boot/kernel.nix#L152-L170

@symphorien Your system wouldn't work outside of NixOS for example.
He wants to have it as part of the responsibility of the packager, not as part of the person who creates a module. I agree that the packager is the best person to specify required kernel modules.
There should also be flags to ignore this feature.

It is as @coretemp says. Basically I want the kernel config to depend on programs requirement rather than having to specify
For instance, if I add the openvswitch program to my system, I would expect my kernel to be rebuilt with CONFIG_OPENVSWITCH without any manual intervention. Or maybe easier as a first step  have nixos-rebuild error out so that I can add it to my config.
As nixpkgs is crossplatform, I am not sure what the best API would be but the idea fits with nixos mindset imo.

Interestingly enough, at least for services/programs one can use boot.kernelPatches to achieve this it seems:
https://github.com/NixOS/nixpkgs/blob/release-18.03/nixos/modules/misc/crashdump.nix

I've started playing with the idea and it makes things easier but the kernel configuration system in nix is too brittle to use this to build the kernel config automatically,
  kernelExtraConfig = ''
          # ideally we should just have to switch OPENVSWITCH on and let nix enable the dependencies
          OPENVSWITCH y

         # but I do it manually & imperfectly.
          NF_INET y
          NF_CONNTRACK y

          NF_NAT y
          NF_NAT_IPV4 y
  '';

I wanted to do nixos/modules/virtualisation/openvswitch.nix
    boot.kernelPatches = [ {
        name = "openvswitch";
        patch = null;
        extraConfig = pkgs.openvswitch.kernelExtraConfig;
        } ];

but that caused some problems so I still manually specify the kernel config, just a bit better via
mykernel = super.linux_latest.override { extraConfig=pkgs.openvswitch.kernelExtraConfig; }.
With the current system we might have several contradicting values for CONFIG_OPENVSWITCH, it seems like the current generate-config.pl will just use the last value. It may be good to display a warning if $requiredAnswers{$1} is already defined in $requiredAnswers{$1} = !(defined $2); :
open ANSWERS, "<$ENV{KERNEL_CONFIG}" or die "Could not open answer file";
while (<ANSWERS>) {
    chomp;
    s/#.*//;
    if (/^\s*([A-Za-z0-9_]+)(\?)?\s+(\S+)\s*$/) {
        $answers{$1} = $3;
        $requiredAnswers{$1} = !(defined $2);
    } elsif (!/^\s*$/) {
        die "invalid config line: $_";
    }
}
close ANSWERS;

Also I alternate between building builtin modules (to use with qemu -kernel , this way I don't have to build the initramfs) and typical kernels. But the current system seems to
my $answer = "";
# Build everything as a module if possible.
$answer = "m" if $autoModules && $alts =~ /\/m/ && !($preferBuiltin && $alts =~ /Y/);
# `OPENVSWITCH m` in extraConfig will override the $preferBuiltin
$answer = $answers{$name} if defined $answers{$name};

so one should write in extraConfig sthg akin to "OPENVSWITCH ${if preferBuiltin then "y" else "m"}", there is an opportunity for an alias there.

Now that structured kernel config is available upstream, we can decide on an API. Some random thoughts:
Current mechanisms are just checks but I propose to add an option that proactively enforces kernel options.
(A drawback is that custom kernel configurations invalidate the cache, well it's already the case for me).
    system.requiredKernelConfig = with config.lib.kernelConfig; [
      (isYes "SERIAL_8250_CONSOLE")
      (isYes "SERIAL_8250")
    ];

system.requiredKernelConfig is used in

minimal-kernel.nix:
    (map (builtins.getAttr "configLine") config.system.requiredKernelConfig))

and in nixos/modules/system/boot/kernel.nix|

    # nixpkgs kernels are assumed to have all required features
    assertions = if config.boot.kernelPackages.kernel ? features then [] else
      let cfg = config.boot.kernelPackages.kernel.config; in map (attrs:
        { assertion = attrs.assertion cfg; inherit (attrs) message; }
      ) config.system.requiredKernelConfig;

I would like to update these configs with structured kernel configs (see a bit later on how to use it).
I would like packages to be able to communicate their configuration/needs, for instance installing openvswitch should turn on CONFIG_OPENVSWITCH. This should be done independently of the nixos modules as some packages can be installed without modules/overriden etc. Like mininet requires the kernel to support namespaces. meta.kernel.requiredConfig = [ NET_NS ] etc.
Now I wonder how to pass on those dependencies to the kernel configuration. Should the kernel depend on all packages and check for their config requirements ?
At this point, it should be ok to add a boot.kernel.structuredConfig setting.

closed in favor of https://github.com/NixOS/nixpkgs/issues/69014


Issue search is case-sensitive

Description
When you search for issues either for a single project or the whole organization, "javascript" does not find issues containing "JavaScript". This is more than unusual for a current search implementation.
Search should be case-insensitive.
Gitea Version
1.20.3
Can you reproduce the bug on the Gitea demo site?
No, because https://try.gitea.io/ not available
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Ubuntu
How are you running Gitea?
unknown
Database
MySQL/MariaDB

This depends on different database when search engine is DB.

Same problem with Postgres. This makes it really hard find all issues pertaining to a topic.

Nevermind, still had to switch ISSUE_INDEXER_TYPE to db. Now case-insensitive search seems to work fine.

It depends on the ISSUE_INDEXER_TYPE, which is defaulted to bleve when searching with keyword.
If you are using DB as ISSUE_INDEXER_TYPE, then it's up to the DB type.

We close issues that need feedback from the author if there were no new comments for a month. :tea:


bin/setup No Longer Works

-Z was removed from Salt Bootstrap a while back: https://github.com/saltstack/salt-bootstrap/commit/18e1b973f24ffd975184bec03d1c957f339de1a7
The salt essentials setup script currently calls Salt Bootstrap with -Z, which causes the script to fail: https://github.com/craig5/salt-essentials-utils/blob/master/bin/setup#L69

Thank you for filing the issue. I will work on updating the script.
Stay tuned...


commit

FIXED #Issue 658.
REVAMPED THE CONTACT US PAGE, SUITING THE WEBSITE, ENHANCED FONT SIZE AND STYLE.
LOOKS BETTER NOW AND SUITS THE FEEL OF THE WEBSITE.
KINDLY MERGE THIS PULL REQUEST.

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

hey ur contact page is looking like this, which is not good at all :(

What i made was this
There has been some problem in copying the code while creating pr
I'll make new commit


chore(deps): update terraform-module to v1.3.0

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/event-notifications/ibm (source)
module
minor
1.2.2 -> 1.3.0

terraform-ibm-modules/scc-workload-protection/ibm (source)
module
minor
1.2.1 -> 1.3.0

Release Notes

terraform-ibm-modules/terraform-ibm-event-notifications (terraform-ibm-modules/event-notifications/ibm)
v1.3.0
Compare Source
Features

DA solution updates:- removed kms_region variable as its now programmatically determined- existing_kms_root_key_id has been replaced by existing_kms_root_key_crn- existing_resource_group has been renamed to use_existing_resource_group (#178) (e05d41d)

v1.2.3
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#174) (cf5aee0)

terraform-ibm-modules/terraform-ibm-scc-workload-protection (terraform-ibm-modules/scc-workload-protection/ibm)
v1.3.0
Compare Source
Features

add guid output (#85) (29ccb2e)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.4.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Checking all pythons in DEFAULT_PYTHON.

<EMAIL_ADDRESS>is 'python3' and 'python' and find_python_config() tries all python configurations in DEFAULT_PYTHON.
But when 'python3' is not found, investigate_python_config() throws exception and returns from find_python_config() without checking 'python'.
This fix checks all pythons in DEFAULT_PYTHON.

@shkit Thank you very much!


materialize 0.97.8 changed API for Modals

materialize changed the API for Modals slightly which breaks the integration in angular2-materialize.
https://github.com/Dogfalo/materialize/releases/tag/v0.97.8

@pepperbob thanks for the heads up. It's actually been reported this morning (https://github.com/InfomediaLtd/angular2-materialize/issues/149) and a new version (6.0.0) of angular2-materialize has been released.
Are you able to grab the new version and confirm this is working properly for you? Thanks.
The example code in the app has also been updated to reflect the changes in Materialize. It's using MaterializeActions to trigger open/close: https://github.com/InfomediaLtd/angular2-materialize/blob/master/app/components/dialogs.ts
openModal() {
    this.modalActions.emit({action:"modal",params:['open']});
  }
  closeModal() {
    this.modalActions.emit({action:"modal",params:['close']});
  }

Oh, thanks for the quick reply - I couldn't find and issue about it - but I didn't expect it to be closed already! Will check it out. Thanks!

i'm experiencing still problems when using the modals, even with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>when i'm implementing the modal like [https://github.com/InfomediaLtd/angular2-materialize/blob/master/app/components/dialogs.ts] nothing happens when i click the button.
when testing the modal with $('#modalId').modal('open'); it throws an error Uncaught TypeError: $overlay.velocity is not a function.... i checked that the right jQuery version etc is installed but i can't fix this error.
does this belong to this issue, or should i open a new open?
thanks in advance - and by the way - thanks for your awesome work!

@jroehl there are other issues related to the velocity error. It's related to your configs and how you import MaterializeCSS and jQuery. It depends on the type of config you have (Webpack, CLI, JSPM, etc.). Please search through the issues and maybe add your comments on the relevant ones that related to your specific config.
As a starting point I suggest cloning this repo and using the example app in there. The config of the example app is using JSPM (jspm install, live-server). If that works then it's surely related to how you configure your specific repo.
I'll close this issue for now unless anyone find specific issues related to the new version of MaterializeCSS and compatibility with angular2-materialize.


will istio have to notify every Ztunnel and waypoint in ambient mode when even just a new service added into cluster

As described in the title.
In my opinion, cause the istio use iptable technology to redirect traffic. When there is a new service A being added to the cluster, all the ztunnel and waypoint in the cluster have to update the configuration, cause they need to know the new serviceA's traffice rule.
And the sidecar mode also has this issue right? Is this recognition correct?

Yes. But it has nothing to do with iptables specifically - the iptables is static. But the proxy (envoy/ztunnel) do have config about each service.


add library config changes to migration guide

As far as I can tell this is so far only mentioned in this issue comment - it seems to have changed after the blog post listing all the changes in config: https://github.com/webpack/webpack/issues/13584#issuecomment-863272803
(I also needed this question and answer to figure it out: https://stackoverflow.com/questions/77951581/webpack-5-builds-for-aws-lambda-with-typescript-runtime-handlernotfound)

I think I still need someone with vercel access to approve a deployment before this can be merged.


Fix publish again

Fixes the github actions to run npm install before running npm publish (otherwise it fails because of the missing husky dependency)
Also adds support for the release/skip label so that PRs like this one don't trigger a publish (There's no need to republish since the only thing this PR modifies is github action logic)

The PR check keeps failing for some reason with a 500 server error saying it can't check out this repository which is just weird. I think a github service is down and it has nothing to do with the code


pythonPackages.pygments: 2.5.2 -> 2.6.1

Motivation for this change
New release: https://github.com/pygments/pygments/releases/tag/2.6.1
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Running Pygments on Python 2.x is no longer supported.

I just noticed this is a major breaking change. Wondering how to tackle this.

python 2 is EOL: https://github.com/NixOS/nixpkgs/pull/77714#issuecomment-640556454

I just noticed this is a major breaking change. Wondering how to tackle this.

If there's a use case for the python2 (some application uses it), then it will be worthwhile to freeze the package for the application.
Ideally there would be no uses cases for python2, mentioned above.

Pretty sure something major is being touched by this.
[0/30670 built, 64/24244/28058 copied (160509.0/183327.7 MiB), 79361.0/92867.2 MiB DL] fetching source from https://cache.nixos.org

A wild guess would be that it's asciidoc and it's touching the whole documentation.
all-packages.nix
--
2314-  asciidoc-full = appendToName "full" (asciidoc.override {
2315:    inherit (python2Packages) pygments;
2316-    texlive = texlive.combine { inherit (texlive) scheme-minimal dvipng; };
2317-    enableStandardFeatures = true;
2318-  });
--
2320-  asciidoc-full-with-plugins = appendToName "full-with-plugins" (asciidoc.override {
2321:    inherit (python2Packages) pygments;
2322-    texlive = texlive.combine { inherit (texlive) scheme-minimal dvipng; };
2323-    enableStandardFeatures = true;
2324-    enableExtraPlugins = true;

Pretty sure something major is being touched by this.

It's a dependency of sphnix, and a lot of packages use that to generate documentation.

part of https://github.com/NixOS/nixpkgs/pull/89723


feature: Custom User authentication with external datasource

Additional simple controllers to create and view user's posts

Task linked: CU-8696fxav3 Custom User authentication with external datasource


running as root doesn't show up anything

VSCode Version:1.18.1
OS Version:Ubuntu 17.10 on Wayland

When I try to open a directory as root it doesn't show up anything, even any terminal message.
Steps to Reproduce:

sudo code /var/www/html/ --user-data-dir=/home/user/.vs-root
VS Code window won't appear.

Reproduces without extensions: No

We don't have any plans to improve running VS Code as sudo any time soon. In the meantime I suggest symlinking html to your home directory and from working there.


[BUG] Bento components on NEXTJS - ReferenceError: self is not defined

Following guide at
https://bentojs.dev/documentation/guides/bento-next.js-guide/
and gets this error:
ReferenceError: self is not defined
Maybe bento expects window? And next is SSR?

None of the components works.
ReferenceError: self is not defined


How to get board id

I am trying to implement a Kanban Board in my project but I am not able to understand how can I get source and target board ids in dropEl callback to update task in db?

Hello,
Maybe this link can help you. In that case, I'm showing you up the target sortening order. But you can verify the column name and write to database as you wanted. Please let me know if you reached your goal or need more help.

Hello,
Maybe this link can help you. In that case, I'm showing you up the target sortening order. But you can verify the column name and write to database as you wanted. Please let me know if you reached your goal or need more help.


Migrating from 1.0.0 to 2.1.0 ignores @include annotation

Summary
I am upgrading apollo SDK from 1.0.0 to 2.1.0. In the latest version generated model ignores the @include annotation form the fragment.
Version
2.1.0
Description
Query
query ProductDetailsQuery($frontPage : Boolean = false,$id: int) {
  details(id: $id) {
    ... on Product @include(if: $frontPage) {
      size
      name
      link
    }
    id
    description
}

We are using the above query on the Home screen for various scenarios.
1st scenario: we are passing frontPage=true
2nd scenario: we are sending frontPage=false
The previous version(1.0.0) generates the following model
 final @Nullable Integer size;
But after migration, it's generating
 final Integer size;
Because of this in the 2nd scenario we are getting the following exception cause: java.lang.IllegalStateException: corrupted response reader, expected nonnull value for size.
it was working fine with 1.0.0.
Any changes in a model generation logic when we use @include?

Any plan to release this fix?

Releases typically happen every couple of weeks. If you want to try before the next one, you can use the snapshots. See the releases section of the README for more details on how to use SNAPSHOTS.

Hi,
Have we changed a model generation logic?
v2.2.1 creates the following class
  public interface Product {
@NotNull String __typename();

@NotNull String description();

ResponseFieldMarshaller marshaller();

  final AsProduct.Mapper asProductFieldMapper = new AsProduct.Mapper();

  final AsProduct1.Mapper asProduct1FieldMapper = new AsProduct1.Mapper();

}
to access, size have to do
((ProductDetailsQuery.AsProduct)ProductDetailsQuery.Data.product()).size()
and size is
   final int size;
its not @Nullable
why are we creating 2 FieldMapper AsProduct,AsProduct1
Whereas on the older version 1.0.0
ProductDetailsQuery.Data.product().size
and size is nullable

@martinbonnin @sav007 may I know what is the best practice here

@GauravP123 are you still using that same query?
query ProductDetailsQuery($frontPage : Boolean = false,$id: int) {
  details(id: $id) {
    ... on Product @include(if: $frontPage) {
      size
      name
      link
    }
    id
    description
}

I would expect that query to produce a single AsProduct that can be consumed like:
data.details.asProduct()?.size
data.details.id

Getting AsProduct1 from the above queries sounds weird, let me know if that's the case and I'll dig more.

I am using a different query. I have shared the above query as an example.
with v1.0.0 it was working properly and  int size is nullable
v2.1.0 it was generating final int size n its non-null and able to access it ProductDetailsQuery.Data.product()).size()
v2.2.0 able to access it viva ((ProductDetailsQuery.AsProduct)ProductDetailsQuery.Data.product()).size() nfinal int size n its non-null

I think this is the expected behaviour. I believe the size field itself is not optional. What is optional is whether to include the inline fragment or not. In Java you can check whether product() is an instance of AsProduct to access it fields.
Is there something you cannot do with version 2.2.0 ?

yeah, size is not optional but when we use @include previously(v1.0.0) it was creating @Nullable

Yep, this changed. I don't have the exact reason why on top of my head but I'm pretty sure there must have been a good reason. Maybe it's because a given field can be included in multiple inline fragments as well as in the parent field at the same time.

we are using  generateModelBuilder.set(true) not the kotlin one coz of this data.details.asProduct() is not available We have to type case but somehow it's not working. Let me check if get something


datasource added for dataone datasets

I am currently trying to add tests for the data sources!

Should I close this? Do you need any of the work from it?


Raspberry PI2 stucks in Configuring eth0 with DHCP

I tried to install a PI2 two times. The first time went fine, during the second installation (another sd card) the PI hangs at the message:
Configuring eth0 with DHCP

and doesn't continue with the installation.
Any ideas what could be the reason?

I could solve the issue, by resetting the name that my router assigned to the mac of the pi. I don't know if its an issue on the routers side or in the PI, therefore I leave the issue open.

Wasn't reproducible.

Wasn't reproducible.


License

I am looking at using and expanding upon this library in a project, I need to know the licensing terms though before I proceed.
Thank you

https://github.com/NVentimiglia/Websockets.PCL/blob/master/License.md
Thanks, added


Install ST-Link drivers solution is broken again

This issue was opened before and closed 3 weeks ago. Link still appears to be broken and requires a login after which Sharepoint indicates the following:
That didn't work
We're sorry, but<EMAIL_ADDRESS>can't be found in the microsoft-my.sharepoint.com directory. Please try again later, while we try to automatically fix this for you.
Original issue: https://github.com/MicrosoftDocs/azure-docs/issues/31133#issue-443575336

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

ID: 0cf9b53c-0438-f23c-fd2e-d57481d57e85
Version Independent ID: f01afd66-d093-ac44-2b20-df225530b39e
Content: IoT DevKit to cloud -- Connect IoT DevKit AZ3166 to Azure IoT Hub
Content Source: articles/iot-hub/iot-hub-arduino-iot-devkit-az3166-get-started.md
Service: iot-hub
GitHub Login: @wesmc7777
Microsoft Alias: wesmc

@whaakman Thank you very much for bringing this to our attention! We shall evaluate and update as appropriate.

@JimacoMS3 Issue appears to be not resolved. Can you take a look and provide your suggestions?

@whaakman We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


File rights are ignored with unlink

Hello,
I found that even if a file is not writable, unlink will return true and the file will be removed from directory with no error.
Example:
<?php
$root = \org\bovigo\vfs\vfsStream::setup();
$file = new \org\bovigo\vfs\vfsStreamFile('foo', 0);
$root->addChild($file);

// This should output: bool(false)
// Acutal result: bool(true)
var_dump(unlink($file->url()));

I'm doing a pull request to fix this problem :)

My bad, I didn't know it was expected to work this way.

Yes, that's how Linux works. See https://speakerdeck.com/mikey179/testing-filesystem-related-code-with-vfsstream-june-30-2016-phpugmrn?slide=22 for a test you can perform on your system to see for yourself. In short: whether you can delete a file/directory doesn't depend on the rights you have for the file/directory you want to delete, but on the rights of the directory you want to delete in. The reason is that the deletion changes the directory where it is in, and not the file/directory that is about to be deleted.
But I appreciate your effort of providing a PR together with the bug report. :)


Implement customization for object serializers

Classes and properties can be marked up with the IonSerializer to be used for serialization/deserialization.
[IonSerializer(typeof(MyIonCarSerializer))]
public class Car
{
}

The IonSerializer Attribute must specify a IonSerializer with a no-arguments public default constructor.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing in favor of https://github.com/amzn/ion-object-mapper-dotnet/pull/35/.


1.7 and 1.8 versions are not working

Can you give more info? What are you trying to do that is not working? iOS or android? What errors are you getting?

Didn't hear back on this so assume it is solved (I can't see any errors on my side)


"ReferenceError: jsPDF is not defined" with Meteor

Hi - Really nice work on the docs for this repo. :+1:
Do you know if anyone has gotten this to work within a Meteor project?
I'm currently getting "ReferenceError: jsPDF is not defined"
=> Modified -- restarting.
W20150407-11:48:14.168(-5)? (STDERR)
W20150407-11:48:14.169(-5)? (STDERR) /Users/chip/.meteor/packages/meteor-tool/.<IP_ADDRESS>bea6ne++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150407-11:48:14.169(-5)? (STDERR)                                            throw(ex);
W20150407-11:48:14.169(-5)? (STDERR)                                                  ^
W20150407-11:48:14.172(-5)? (STDERR) ReferenceError: jsPDF is not defined
W20150407-11:48:14.172(-5)? (STDERR)     at app/lib/jspdf.plugin.autotable.js:287:4
W20150407-11:48:14.172(-5)? (STDERR)     at app/lib/jspdf.plugin.autotable.js:289:3
W20150407-11:48:14.172(-5)? (STDERR)     at /Users/chip/code/invoicethat.com/.meteor/local/build/programs/server/boot.js:222:10
W20150407-11:48:14.172(-5)? (STDERR)     at Array.forEach (native)
W20150407-11:48:14.172(-5)? (STDERR)     at Function._.each._.forEach (/Users/chip/.meteor/packages/meteor-tool/.<IP_ADDRESS>bea6ne++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150407-11:48:14.172(-5)? (STDERR)     at /Users/chip/code/invoicethat.com/.meteor/local/build/programs/server/boot.js:117:5
=> Exited with code: 8

It appears that it is failing on line 284 of this plugin due to doc being undefined:
283     function getStringWidth(txt) {
284         return doc.getStringUnitWidth(txt) * doc.internal.getFontSize();
285     }

Here's the code I'm using as an example:
client/lib/PDF.coffee
class PDF
  columns: ->
    [
      {
        title: 'ID'
        key: 'id'
      }
      {
        title: 'Name'
        key: 'name'
      }
      {
        title: 'Country'
        key: 'country'
      }
      {
        title: 'IP-address'
        key: 'ip_address'
      }
      {
        title: 'Email'
        key: 'email'
      }
    ]

  data: ->
    [
      {
        'id': 1
        'name': 'Shaw'
        'country': 'Tanzania'
        'ip_address': '<IP_ADDRESS>'
        'email'<EMAIL_ADDRESS>      }
      {
        'id': 2
        'name': 'Nelson'
        'country': 'Kazakhstan'
        'ip_address': '<IP_ADDRESS>'
        'email'<EMAIL_ADDRESS>      }
    ]

  generate: ->
    doc = new jsPDF('p', 'pt')
    doc.autoTable @columns(), @data()
    uri = doc.output 'datauristring'
    console.log "uri ", uri

module?.PDF = PDF
window?.PDF = PDF

When I comment out the doc.autoTable it calls doc.output successfully, so it appears that jsPDF is loaded properly.
Any suggestions would be helpful. Thank you.

Sounds strange. I just tried to generate a basic table with meteor and it worked. Could you try to port your example to javascript and see if that works. I haven't had the opportunity to use coffee script yet...

I tried converting your example to javascript now and including it in the meteor hello world example now which worked. This means that it must be some kind of loading order issue in your case? This is the code that worked:
if (Meteor.isClient) {

  Template.hello.events({
    'click button': function () {
      var columns = [
        {
          title: 'ID',
          key: 'id'
        }, {
          title: 'Name',
          key: 'name'
        }, {
          title: 'Country',
          key: 'country'
        }, {
          title: 'IP-address',
          key: 'ip_address'
        }, {
          title: 'Email',
          key: 'email'
        }
      ];

      var data = [
        {
          'id': 1,
          'name': 'Shaw',
          'country': 'Tanzania',
          'ip_address': '<IP_ADDRESS>',
          'email'<EMAIL_ADDRESS>        }, {
          'id': 2,
          'name': 'Nelson',
          'country': 'Kazakhstan',
          'ip_address': '<IP_ADDRESS>',
          'email'<EMAIL_ADDRESS>        }
      ];

      var doc, uri;
      doc = new jsPDF('p', 'pt');
      doc.autoTable(columns, data);
      uri = doc.output('datauristring');
      console.log("uri ", uri);

    }
  });
}

Thanks for your prompt response and it was a really good idea to start over with a clean slate. It allowed me to step back a bit and think about how I was loading the files. I tried your suggestion and was able to get it to work by putting those files under client/compatibility/, so it was my mistake all along.
At any rate, I have created a jspdf package on atmosphere and was considering doing the same for your plugin if I have the time and you're ok with it.
Thanks again for your help. Really nice work on this plugin!  :smiley:

Sure! Let me know if you have any other issues or suggestions!


fix: undefined error in localhost (#357)

Reference issues
fix #357
What does this issues fix?

handle the undefined varible onchange which is causing errors in localhost

Thank you for your contribution, @darwin808
But I do not understand why we need this. This package is for usage in Chrome Extensions. You won't have access to Chrome Extensions API outside the extension context.


Arc-ify the Ipfs struct

This allows us to pass the central Ipfs object around when needed, improving what we can do within its child objects. In addition:

impl Default for Bitswap (now possible)
replace Arc<Repo> with a reference to Ipfs in IpldDag and Ipns, which are now created on-the-fly when needed
poll for BitswapEvents using NetworkBehaviourEventProcess
unArc the Repo

bors r+


How to render tab scores with rhythm information?

Hi,
I'm trying to implement some kind of MusicXML player with this library. First I want to focus on using tab music scores.
Right now I have an render output like this:

As you can see, for the tab notes, there is no rhythm information shown, or rhythm bars, I don't know the correct term. So the user can never know, if it's a eighteenth or sixteenth note.
I was wondering, if there is a way to get an output like the web-player of songsterr.com:

Here you can see the rhythm information being rendered underneath.
Can you think of a way, on how to achieve something similar? It doesn't have to look exactly like that, I just want the user to know, what the rhythm looks like.
Thanks in advance.

Hi Andreas,
Rendering music is a very complex task and Lomse never gave priority to tablature notation. Currently, for tablature notation, Lomse only supports displaying the strings and the tab numbers. And rendering stem and flags for tablature will require additional development. But if you are planning to develop an application focused on tablature notation, probably you will need support to render additional symbols and notations and, thus, that will require more development.
I am willing to help you and develop everything that is necessary but I also need help. Lomse is not a project developed by a company but a project developed by myself as a hobby. I work alone on this, and people using Lomse occationaly send PRs. But there is no team of people regularly collaborating, sending PRs and helping to discuss needs and specifications.
Therefore, first of all, someone will have to take care of determining the additional symbols and notation required for fully supporting tablature for guitar. I'm not a professional musician and my music studies were centered on piano, chamber music and choral music. Thus, I have no knowledge of tablature notation and, thus, it is very difficult for me to determine what is needed and the rules of tablature notation. For instance, you are suggesting to display flags and stems under the string lines, as in this image:

But in the Wikipedia entry for guitar tablature you can see this image:

In the image, stems and flags are rendered also above the string lines. Thus it is necessary to have someone who knows the notation and can define the rules before proceeding with the programming.
Second, my time is limited and I cannot assume any commitment to develop a feature within a certain period. I'm currently working on MusicXML export and I will not be able to start the development of stems and flags for tablature probably until mid July, assuming that by them we will have clear specifications of the tablature rules for stems an flags. I do not expect this feature to be very complex to develop and so probably it will be ready by beginning of September. Anyway these are just estimates, not commitments.
And this is basically the situation.

I am closing this issue as there is no further feedback.


Ssh Fixes and Improvements

This handles some quality of life issues with the new ssh system. Broadly speaking, it:

properly handles user input with identity files
allows multiple identity files to be used if set up in the ssh config file
allows a different preference of authentication methods in the ssh config files
extends the timeout on input to 60 seconds instead of 15
adds support of several ssh config file keywords to turn off various authentication methods

okay, i've fixed the dummy key to have a length of 2048 and merged in main again


AstroTuxLauncher TMPFS issue

Running the atrotux ptero egg was failing due to not enought disk space when downloading depot downloader to /tmp
I fixed this by changing wings config to increase tmpfs to 10gb instead of 100mb. I think there may be a better solution of having the depotdownloader downloaded to a directory in the container rather than tmp, but I am not sure how to acheive that. I tried binding a host dir to /tmp in the container, but that didnt seem to work.

This is just how AstroTuxLauncher handles game downloads, there's a note stating a tmpfs increase is required in the README.


Failed to import image: Error response from daemon: exec: "xz": executable file not found in $PATH

RancherOS Version: (ros os version)
1.0.1
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
bare metal
I have a bare-metal build farm (cattle cluster) that is using gitlab-ci runners.
The runner is failing to load the initial helper image because xz isn't available to build the image
When I pulled the image explicitly it worked okay
I opened an issue with gitlab because it took me a while to work out what image was needed to be pulled.
https://gitlab.com/gitlab-org/gitlab-ci-multi-runner/issues/2533
Can we get xz command made available ?

Would be nice to have.

it feels a little wrong to need to add things to a base micro distro to allow something that runs in containers to work - When the stack clash work and post 1.0.3 release catchup calms down, I'll look into it - I need to do some serious playing with GitLab anyway :)

https://gitlab.com/gitlab-org/gitlab-runner/issues/2533

To switch to ubuntu console is a good way.
Can we close this issue, or what I need to do?

We have xz command.
[root@ros-v1-1-2 docker]# ros -v
ros version v1.1.2

[root@ros-v1-1-2 docker]# xz -h
Usage: xz [OPTION]... [FILE]...
Compress or decompress FILEs in the .xz format.

  -z, --compress      force compression
  -d, --decompress    force decompression
....
....

Close this issue, if you still have any questions, please let me know.


Path with reserved characters does not return percent encoding (PSR-7 violation)

I found a possible double issue with the management of encoded/unencoded paths.
Given this path<EMAIL_ADDRESS>(an email address as path segment)
the following snippet return wrong results.
$request = new Request('GET', 'http://example.com/email/foo+extension@example.com');

echo $request->getPath(); /*<EMAIL_ADDRESS>*/

First error: getPath MUST return percent-encoded values.

The value returned MUST be percent-encoded, but MUST NOT double-encode
any characters. To determine what characters to encode, please refer to
RFC 3986, Sections 2 and 3.3.

https://github.com/php-fig/fig-standards/blob/17945666b0b5c78691dca65405aae21e3e403539/proposed/http-message.md

So $request->getPath(); MUST return /email/foo%2Bextension%40example.com (characters + and @ percent-ecoded`
Second issue: +, @ and the rest of reserved words if present in the path are interpreted as already encoded
What's the problem? IMO is the same problem fixed on the release 5.0.3 with the query parameters encoding

Guzzle v5 will not change this behavior.


feat(gltf): Read skinning data

#49  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommy-xr and the rest of your teammates on  Graphite


Support conditional branching for task progression

Fixes #1955

Adds support for

[x] Adds Conditions and Expression classes and converters from JSON config (see spec here: google/ground-platform#1478)
[x] Supports all, any, and one match types for lists of expressions and selections in tasks.
[x] Refactors saving task position to current task ID and dynamically calculates the task sequence on Previous/Next.
[x] Handles dynamically moving between tasks based on conditions.
[x] Progress bar shows dynamic position on the task sequence.

[x] Used the following configuration (minus acls) to verify that tasks progress as expected.:
[x] Test when the last task in the progression is conditional.

{
  "title": "A conditional survey",
  "jobs": {
    "qKxHiyBN9KqWhVXWp11W": {
      "dataCollectorsCanAdd": [
        "points",
        "polygons"
      ],
      "index": 1,
      "tasks": {
        "RyWKlphglHQzcmuqEPlk": {
          "required": false,
          "id": "RyWKlphglHQzcmuqEPlk",
          "type": "draw_area",
          "label": "Only if you picked x and y",
          "condition": {
            "matchType": "MATCH_ALL",
            "expressions": [
              {
                "expressionType": "ANY_OF_SELECTED",
                "taskId": "GqEBDoAoEnPbMmRbje5c",
                "optionIds": [
                  "MFOdfshBZSieX5dddPfj"
                ]
              },
              {
                "expressionType": "ANY_OF_SELECTED",
                "taskId": "GqEBDoAoEnPbMmRbje5c",
                "optionIds": [
                  "Po2zyacw6FVFE6gaL8km"
                ]
              }
            ]
          },
          "addLoiTask": false,
          "index": 3
        },
        "myaWTKIBVd38e1TLfiDs": {
          "required": false,
          "addLoiTask": false,
          "label": "Only if you picked x",
          "index": 2,
          "condition": {
            "expressions": [
              {
                "expressionType": "ANY_OF_SELECTED",
                "taskId": "GqEBDoAoEnPbMmRbje5c",
                "optionIds": [
                  "MFOdfshBZSieX5dddPfj"
                ]
              }
            ],
            "matchType": "MATCH_ALL"
          },
          "type": "drop_pin",
          "id": "myaWTKIBVd38e1TLfiDs"
        },
        "trmOruwBVL1uaiUNAJ4D": {
          "index": 5,
          "label": "Did you get to the end?",
          "id": "trmOruwBVL1uaiUNAJ4D",
          "type": "text_field",
          "addLoiTask": false,
          "required": false
        },
        "GqEBDoAoEnPbMmRbje5c": {
          "label": "Pick an choice, any or all choices!",
          "options": {
            "Po2zyacw6FVFE6gaL8km": {
              "code": "",
              "index": 1,
              "label": "y"
            },
            "MFOdfshBZSieX5dddPfj": {
              "label": "x",
              "code": "",
              "index": 0
            },
            "rVbIQmFIajZvnlSIl7n3": {
              "index": 2,
              "code": "",
              "label": "z"
            }
          },
          "index": 1,
          "cardinality": "select_multiple",
          "type": "multiple_choice",
          "id": "GqEBDoAoEnPbMmRbje5c",
          "required": false,
          "hasOtherOption": true,
          "addLoiTask": false
        },
        "wKK9LEHjo9oWeu1i71Ek": {
          "id": "wKK9LEHjo9oWeu1i71Ek",
          "type": "drop_pin",
          "label": "Pick your current location",
          "index": 0,
          "required": true,
          "addLoiTask": true
        },
        "UjSOau5djBlhTKkGgRZQ": {
          "index": 4,
          "label": "Why did you pick x, y, and z?",
          "type": "text_field",
          "required": false,
          "addLoiTask": false,
          "id": "UjSOau5djBlhTKkGgRZQ",
          "condition": {
            "expressions": [
              {
                "expressionType": "ALL_OF_SELECTED",
                "optionIds": [
                  "MFOdfshBZSieX5dddPfj",
                  "Po2zyacw6FVFE6gaL8km",
                  "rVbIQmFIajZvnlSIl7n3"
                ],
                "taskId": "GqEBDoAoEnPbMmRbje5c"
              }
            ],
            "matchType": "MATCH_ALL"
          }
        },
        "n296k4Rz8gkZAMNVULDU": {
          "type": "text_field",
          "addLoiTask": false,
          "index": 6,
          "label": "(Secret end if you picked z) Did you actually get to the end?",
          "condition": {
            "expressions": [
              {
                "expressionType": "ANY_OF_SELECTED",
                "taskId": "GqEBDoAoEnPbMmRbje5c",
                "optionIds": [
                  "rVbIQmFIajZvnlSIl7n3"
                ]
              }
            ],
            "matchType": "MATCH_ALL"
          },
          "required": false
        }
      },
      "strategy": "AD_HOC",
      "name": "Pick various options to test conditions",
      "id": "qKxHiyBN9KqWhVXWp11W",
      "defaultStyle": {
        "color": "#F37C22"
      }
    }
  },
...
}

@gino-m

[ktfmt] Invalid formatting for: /workspace/ground/src/main/java/com/google/android/ground/ui/datacollection/DataCollectionFragment.kt
[ktfmt] Invalid formatting for: /workspace/ground/src/main/java/com/google/android/ground/ui/datacollection/DataCollectionViewModel.kt
[ktfmt] Invalid formatting for: /workspace/ground/src/main/java/com/google/android/ground/persistence/local/room/converter/ConverterExt.kt

Fixed those last minute code formatting, sorry about that! Doing one more test to make sure and then merging.

W000t!!!


Docstring formatting does not always respect line length in dynamic mode

First off, thanks so much for the great work on the docstring formatter! I'm very much looking forward to using it for the Polars code base.
When running the new docstring formatter on our code base, I found a number of examples where dynamic line length mode did not produce the expected result. See the three examples below:
def example1():
    """
    Docstring example containing a class.

    Examples
    --------
    >>> @pl.api.register_dataframe_namespace("split")
    ... class SplitFrame:
    ...     def __init__(self, df: pl.DataFrame):
    ...         self._df = df
    ...
    ...     def by_first_letter_of_column_values(self, col: str) -> list[pl.DataFrame]:
    ...         return [
    ...             self._df.filter(pl.col(col).str.starts_with(c))
    ...             for c in sorted(
    ...                 set(df.select(pl.col(col).str.slice(0, 1)).to_series())
    ...             )
    ...         ]
    """

class Nested:
    def example2():
        """
        Regular docstring of class method.

        Examples
        --------
        >>> df = pl.DataFrame(
        ...     {"foo": [1, 2, 3], "bar": [6, 7, 8], "ham": ["a", "b", "c"]}
        ... )
        """

def example3():
    """
    Pragma comment.

    Examples
    --------
    >>> af1, af2, af3 = pl.align_frames(
    ...     df1, df2, df3, on="dt"
    ... )  # doctest: +IGNORE_RESULT
    """

In ruff 0.1.8, running the ruff docstring formatter on these with line-length = 88 will produce line lengths over 88.
I think examples 1 and 2 are clear violations. Example 3 is debatable whether the result is desired or not.

Thanks for the feedback!
I wonder if this is happening because we only subtract $indent_level * indent_widthbut not necessarily the space required by >>> `?

I wonder if this is happening because we only subtract  but not necessarily the space required by >>> ?

That might be it actually. I figured it might be because of edge cases with classes or something, but indeed a simple example like this also exceeds the line length, but only by 4 characters:
def example3():
    """
    ...

    Examples
    --------
    >>> af1, af2, af3 = pl.align_frames(
    ...     df1, df2, df3, on="dt", foo="looooooooooong_string"
    ... )
    """


SFZ Instrument: show all Parameters, improve layout

Looks good! In general I don't like knobs in a scrollview but this is an improvement over what we have currently. Thanks!

Thanks for improving my code. I don't like knobs in scroll view neither. there is only enough space on iPad for all 38 params. I'll try to add functional grouping: eg. turning off "Filter enable" will hide all filter params.


[Plot] Histograms

Hi all, I have a couple issues with the display of histograms :
Context : displaying histograms :
Plot.addCurve(x, y, histogram='left')
# with len(x) = len(y) + 1

the first one was, AFAIK, already there in 0.4 : the x/y log actions throw an exception (because x and y lengths aren't the same probably).
the second one is a regression and probably appeared with the introduction of objects : addCurve throws an exception if x and y don't have the same length.

I made a few modification, now this should be ok. But we should wait for @t20100 to review it before merging.
There is quit a lot of new classes and I don't know the structure of the code. My patch is maybe not ''adapted" to the OO architecture.'


enhancement: support for slow connection

Codecov Report
Base: 45.84% // Head: 45.66% // Decreases project coverage by -0.17% :warning:

Coverage data is based on head (5512913) compared to base (858d905).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #147      +/-   ##
==========================================
- Coverage   45.84%   45.66%   -0.18%
==========================================
  Files          22       22
  Lines        1719     1719
==========================================
- Hits          788      785       -3
- Misses        843      846       +3
  Partials       88       88

Flag
Coverage Δ

unittests
45.66% <ø> (-0.18%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

executor/executor.go
40.88% <0.00%> (-1.66%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Rename is truth to IsTruthWithoutNull

What problem does this PR solve?
Related pr: https://github.com/pingcap/tidb/pull/19621
What is changed and how it works?
Rename ast.IsTruth to ast.IsTruthWithoutNull.
Check List
Tests

Unit test
Integration test

Code changes

Has exported variable/fields change

Side effects

Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch 3.0,4.0

LGTM.
And should we also do the same thing on isfalse(separating it to IsFalseWithNull and IsFlaseWIthoutNull)?

Now only IsTruth function maybe set KeepNull true.

@AilinKid Please help us merge this PR.


[DOCS] Change "state mutated" to "state changed"

One of the basic principles of Redux - its state should never mutate. Rephrase sentence to prevent confusion about Redux state changes.

Thanks!


Fixing leaderboard feedback

Should address all feedback for the leaderboard in https://github.com/stanford-crfm/helm/issues/2291

When mousing over a model name, should show more information about the model, either as a tooltip (preferred) or via a link to the models page (need a href anchor).

Fully addressed

We still write ‘Model/adapter’ - should just be ‘Model’ for simplicity

Could not find where this still appears in the leaderboard upon a quick visual inspection, let me know if i missed something

When mouseover heading (e.g., F1), should tooltip show the description based on the schema

Fully addressed

We still show ⚠; would just drop all the train-test contamination stuff from the frontend since we’re not consistently updating this

Mostly addressed, this won't appear in the front end leaderboard anymore, but the JSONs contain this still so we'll want to remove whatever outputs this if we don't want it appearing elsewhere.

Mostly addressed, this won't appear in the front end leaderboard anymore, but the JSONs contain this still so we'll want to remove whatever outputs this if we don't want it appearing elsewhere.

Sent you #2297 to remove ⚠ from the JSON as well.


Could not create another certificate, reached the maximum number of available certificates

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues. Related issues #3123 #3472

Issue Description
Running CERT_DEVELOPMENT=true fastlane cert on 2 different devices with the same AppleID trying to create 2 Development certificates with the same name (<AppleID user name>) as the result command execution on one of the machines crashes with error Could not create another certificate, reached the maximum number of available certificates. But creating this certificate with XCode works because it names the certificates as <AppleID user name> (<Device Name>).
Similar Issues exist when creating certificate manually with Apple Developers Portal, it automatically names the certificate with your username and when you are trying to generate the second one it returns the error Maximum number of certificates generated.
Is this related? Is Spaceship uses the above portal feature to generate the certificate or this can be fixed?

After running nuke im not able to recreate a Development cert. I the other hand im able to create Adhoc and Appstore.


Question regarding TensorRT version

hi @leimao,
I noticed that the inference relies on TensorRT 8, however, just following the docs and adjusting the inference calls does not work for TensorRT 10.0.1. Specifically, the functions execute_v2 and execute_async_v3 of https://docs.nvidia.com/deeplearning/tensorrt/api/python_api/infer/Core/ExecutionContext.html#tensorrt.IExecutionContext.execute_async_v3. For the do_inference calls in your common.py. Thank you for uploading this repository, it has been very helpful so far.

For future reference, I got a working version by first creating an execution context and then using execute_async_v3 like this,
def do_inference(engine, inputs, outputs, stream, bindings):
    # Transfer input data to the GPU.
    kind = cudart.cudaMemcpyKind.cudaMemcpyHostToDevice
    [cuda_call(cudart.cudaMemcpyAsync(inp.device, inp.host, inp.nbytes, kind, stream)) for inp in inputs]

    # Run inference.
    context = engine.create_execution_context()
    for i in range(engine.num_io_tensors):
        context.set_tensor_address(engine.get_tensor_name(i), bindings[i])

    context.execute_async_v3(stream_handle=stream)

    # Transfer predictions back from the GPU.
    kind = cudart.cudaMemcpyKind.cudaMemcpyDeviceToHost
    [cuda_call(cudart.cudaMemcpyAsync(out.host, out.device, out.nbytes, kind, stream)) for out in outputs]
    # Synchronize the stream
    cuda_call(cudart.cudaStreamSynchronize(stream))
    # Return only the host outputs.
    return [out.host for out in outputs]

I noticed I put this issue on the wrong repository, my bad.


validate doesn't work with glob provider

glob provider is not supported by validate command
$ ./cmd/otelcol-sumo validate --config "glob:/asd/*"
Error: unsupported scheme on URI "glob:/asd/*"
2024/04/08 13:55:57 collector server run finished with error: unsupported scheme on URI "glob:/asd/*"
$ ./cmd/otelcol-sumo validate --config "file:/asd/*"
Error: failed to get config: cannot resolve the configuration: cannot retrieve the configuration: unable to read the file file:/asd/*: open /asd/*: no such file or directory
2024/04/08 13:56:01 collector server run finished with error: failed to get config: cannot resolve the configuration: cannot retrieve the configuration: unable to read the file file:/asd/*: open /asd/*: no such file or directory

Fixed in https://github.com/open-telemetry/opentelemetry-collector/pull/10203, which will be included in 0.102.0.


skip deutero filter

goes with https://github.com/chanzuckerberg/idseq-web/pull/639

This will only be run for certain specific samples?

Perhaps we should just skip it for all the samples and filter on the web app to be more flexible?

I was thinking about making it a filter in web app too (like blacklist), but didn't we decide the list was too long for that? It is still a great idea.
I think it makes a lot of sense to have it be a property of the host_genome though: conceptually, it's definitely a part of host_filtering

Actually it's not so hard. We can filter it directly from the taxon_lineage table, right? Isn't deuterstroms at like phylum level or something?

Conceptually, yes. Except it's not one of the levels we're currently considering: it's superphylum.


Updates Domains API template

Description

Anonymizes values (e.g. removes employee references)
Updates README with up-to-date links
Fixes mobile styles
Moves restrictedDomains to a consts file so we can use it to conditionally render the remove button
Updates AUTH_BEARER_TOKEN in Vercel project (new build needed to pic up new values)

Demo URL
https://domains-api-git-fix-domains-api-template.labs.vercel.dev
Type of Change

[x] Example updates (Bug fixes, new features, etc.)

New Example Checklist

[x] 📱 Is it responsive? Are mobile and tablets considered?

Not related to these changes, but there seems to be a bug where the domains refresh randomly. Is that intentional?
https://github.com/vercel/examples/assets/32464864/7db0e1d3-e35f-4211-ae3f-23d049524d7d

Removing or adding a domain also triggers a refresh of all others.
https://github.com/vercel/examples/assets/32464864/b8d240fe-ef28-4638-af1c-6b6ebb3b77bf

On mobile view, the buttons go over the card container. Perhaps we want to text overflow here?

@delbaoliveira how small you going? Mobile seems to look ok on actual devices and I have a break-all.

There seems to be a bug where the domains refresh randomly. Is that intentional?
Removing or adding a domain also triggers a refresh of all others.

Both expected behavior. Quoting the README:

When a domain is first added to a project, we use SWR to periodically check if the domain's DNS records are configured correctly.


Class Google\Task\Composer is not autoloadable

Environment details

OS: Windows 10
PHP version: 7.4
Package name and version: 2.7

Explanation
Hi, I'm trying to use the library just to implement "Login with Google" in PHP ( I dont need any other google service).
I tried to configure the post update CMD like documentation says in its basic example:
{
    "require": {
        "google/apiclient": "^2.7"
    },
    "scripts": {
        "post-update-cmd": "Google\\Task\\Composer::cleanup"
    },
    "extra": {
        "google/apiclient-services": []
    }
}

Whe in run composer update, after dependencies are installed, it display:
Class Google\Task\Composer is not autoloadable, can not call post-update-cmd script

Do I need to do something else to make post-update-cmd command to work?
Thank you!

@cacharrin thank you for bringing this to our attention! this is because we need to release a new version with the namespaces that were added in #1937

We've released v2.8.0! Please update to this version and you'll be good to go!

Thank you :)
Worked like a charm.


how to change the display name for Company communicator application

Is it possible to change the display name of the application in the highlighted area referred in the below screenshot? If yes, could you please provide screenshots of the corresponding parameters in Azure?

Hi @PrithuMahisha ,
Thanks for raising the query here. Please follow the below steps.
To update name of CC

Login to Azure Portal with Global Admin Credentials (portal.azure.com)

Go to Resource Groups and Under your subscription select the resource group where the app is deployed.

Click on Azure Bot(user) --> under settings click bot profile-->Change display name as per the organizational need.

Please repeat the above steps for Azure Bot(Author)

And update the icons name in the manifest files for both author and user. Re-upload the manifest files again.

Note:-

There might be chance that desktop app will get cached. However, it can be validated by checking in the browser version of the app.

And if company communicator is of 5.x version, needs the header text needs to changed then follow this
link

To change the header text, Update the Reactapp header text as per your organizational requirement present in the configuration section of App service.
Save it and then restart the app service.

@PrithuMahisha , Could you please let us know the status

Hi @gsv022, thanks for providing the inputs. I tried to change the names as mentioned. I am able to change name in React App header text. However, changing name in Azure Bot(user) --> under settings click bot profile helps to change the name only in the App store (highlighted in the image shown below), not in the intended place.

@PrithuMahisha , Please make sure that author bot profile has been updated in the resource group and if it is not reflecting try opening the web version of CC

@gsv022 We have checked in web version, the header text has been changed but the Tab name is still showing the old name.

Please make sure that Author manifest is updated with the required names. And in the azure portal, Azure Bot(author)  is updated with the required name. These are the two places to be updated. If already done and not reflecting, please share the author manifest file and screenshot showing Bot profile section of Azure Bot(Author)

Please find the below screenshots:

Screenshot of Manifest file:

Hi @PrithuMahisha , Thanks for raising the query here. Please follow the below steps. To update name of CC

Login to Azure Portal with Global Admin Credentials (portal.azure.com)
Go to Resource Groups and Under your subscription select the resource group where the app is deployed.
Click on Azure Bot(user) --> under settings click bot profile-->Change display name as per the organizational need.

Please repeat the above steps for Azure Bot(Author)
And update the icons name in the manifest files for both author and user. Re-upload the manifest files again.

Note:-

There might be chance that desktop app will get cached. However, it can be validated by checking in the browser version of the app.

And if company communicator is of 5.x version, needs the header text needs to changed then follow this link

To change the header text, Update the Reactapp header text as per your organizational requirement present in the configuration section of App service.
Save it and then restart the app service.

Hi @PrithuMahisha , Thanks for raising the query here. Please follow the below steps. To update name of CC

Login to Azure Portal with Global Admin Credentials (portal.azure.com)
Go to Resource Groups and Under your subscription select the resource group where the app is deployed.
Click on Azure Bot(user) --> under settings click bot profile-->Change display name as per the organizational need.

Please repeat the above steps for Azure Bot(Author)
And update the icons name in the manifest files for both author and user. Re-upload the manifest files again.

Note:-

There might be chance that desktop app will get cached. However, it can be validated by checking in the browser version of the app.

And if company communicator is of 5.x version, needs the header text needs to changed then follow this link

To change the header text, Update the Reactapp header text as per your organizational requirement present in the configuration section of App service.
Save it and then restart the app service.

Thank you for this, but also where do I re upload the manifest files after update


[FEATURES] CNCF TAG-Runtime Discussion/Presentation

Hello Fluid team,
I'm one of the co-chairs of the CNCF TAG-Runtime. I'm reaching out and I think it would be great for you to present/discuss the project at one of our meetings. For example, a general overview of the project or/and a demo.
Feel free to add it to our agenda or reach out to me (raravena80 at gmail.com)
Thanks!

@raravena80 Thank you for reaching us out! We are working on v0.6.0 release this month. We'd like to introduce Fluid in next month. And will keep in touch with you.

@raravena80 Sounds good! we certainly would love to. We'll get prepared and update the agenda, thanks a lot!

@RongGu following up with this. Any chance you could present in our meeting soon?  Thanks!

Hey @raravena80 , thank you for inviting us. We would love to. When is your routine meeting usually held? Thanks.


Bump LND from 0.17.5 to 0.18.3

What
Updates LND  guide to 0.18.3.
Why
Keeping up with latest developments and improvements.
Release notes:

0.18.0
0.18.1
0.18.2
0.18.3

Scope

[X] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

Anyone care to test / review this?


Make valid? private

Per conversion in #263
valid? is widely used method in Rails and Lotus apps whereas in Reform
this method is undocumented, is not a part of public interface, and
should be used only in Reform internals.
Making valid? private should raise awareness to developers that they
might be calling something they shouldn't.

how use should matchers with Reform?
How do I test if a contract is invalid under trailblazer? I'm used to operation.contract.valid?
Sorry to ask here on the PR, but I was tracking the issues and get here hahaha
thank you

Here is an example from Trailblazer book which I highly recommend to read
it "invalid email" do
  res, operation = Comment::Create.run(
    comment: { user: { email: "1337@" } }
  )

  res.must_equal false
  operation.errors.messages[:"user.email"].must_equal(["is invalid"])
end

or if you test your contract as stand-alone thing, you could just use its api in your tests the same way you use it in operations
rspec example
describe MyContract do
  subject { described_class.new(MyModel.new) }
  before  { subject.validate({}) }

  it do
    expect(subject.errors[:my_attr]).to include("can't be blank")
  end
end

thank you @spectator , was thinking on how to test using the released API on v2
@apotonick nickitito, why break the API? Reform 2 was released weeks ago, could at least deprecated the #valid? :crying_cat_face:

@fernandes Celsito, #valid? is still there (and will stay there), you can use #send. It is incorrect to use contract internals in an operation test. You should use operation.valid? instead! :caipirinhas:

yeah... but shoulda-matchers relies on #valid? and I suspect that other rails stuff too (I confirm once I got it)
we do fuckin' need this emoji :caipirinhas:  hahahah

But valid? is simply the wrong entry point in Reform.
The validation flow in Reform and "Rails" is different on purpose: A form is not a long-lived "model" that you change and validate and change and validate, it is a "use once" throw-away object where there is no place for a public valid? method.
 You never call #validate twice on a form in production code, so why would we allow that in tests?
I am still unsure if I like the matcher approach at all... in the book, we use full-blown #validate tests that are way closer to production-like data than isolated matcher tests, so.... I don't know... if you want to use matchers, you can make valid? public, but I really discourage this approach and matchers in general. :wefuckinneedcaipis:

yeah, #validate is called just one time, then I call #valid? like any controller action would do... I don't think the test diverges from production
I got your point, I'm gonna change the whole test suite to check if op is valid and is errors where added (for me a great huge step for a minor reform update 2.0.2 -> 2.0.3) :kissing_heart:
:caipisftw!!:

Yes, I do not understand.
Where does "any controller action" call valid?. Do you have an example why you need this valid? call instead of the validate call?
This was a bug that was fixed, the valid? method was planned to be private from 2.0 onwards. Sorry sweetheart! :kiss:

(Perfect excuse, from now on I will always say "this was a bug I had to fix it without a major version bump).
BTW, you can use form.send(:valid?), the method will stay but it's not a public concept, if you need it that badly.

never mind, I'm gonna change here on my tests
perfect excuse, great hahahaha

So you gonna change the tests the way they are done in the book? Details!!!! :beers:

Where does "any controller action" call valid?. Do you have an example why you need this valid? call instead of the validate call?

see page # 15 of your book :stuck_out_tongue_winking_eye:
thats what I said, its "normal" behavior


Renew issued certificate

First of all congratulations, I think it's a great project.
I miss a feature that I hope you can implement. It is about renewing a certificate generated by the application.
I don't know if you could implement this feature.
Thanks in advance.

sure, thanks for asking. I will look into this. it should be quite straight forward to do it.

The renew function had been added.
You can now do this using RESTful API:
POST /ca/<ca-id>/cert/<cert-id>/<days>
Authorization: xxx

---
Requires admin role

Alternatively, admin UI is provided too.


kmail: init at 17.04.0

Motivation for this change
Package kdepim for kde-5.
This PR replaces https://github.com/NixOS/nixpkgs/pull/20531
Most executables work except kmail, kontact. blogilo and akregator.
These problems seem all due to QtWebEngine.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@viric, @ttuegel and @zraexy: please try this patch and see if it works for you.
It does not work well for me and I've exhausted my time on this PR for now.

KMail starts for me, but I still have problems with Akonadi. I'm blocked on #20585 at the moment.
One thing to note: because Akonadi registers a D-Bus service, it will not work correctly unless installed into the user or system environment, i.e. kdepim applications cannot be run from the store.

I've done my testing with nix-env -f. -i kde5.kdepim which if I understand correctly should be fine.
I'm also seeing that mariadb error (#20585) when first starting akonadi (via kmail). Some akonadi functionality like storing calendar items (korganizer), addresses (kaddressbook) and notes (knotes) works fine.
On my normal install kmail starts but shows no messages. On a clean desktop, kmail crashes on startup (but the initial config dialog does show).

If i install kde5.kdepim and kde5.kdepim-runtime, I can create a maildir agent in akonadiconsole. Mail that are copied in that directory are picked up. So that part seems to work.
Starting the gui for kmai, akregator or kontact still does not work.

It seems QtWebEngine is not properly installed. It is not in /nix/store/<HASH>-qt-5.6.1-1/lib/.

It seems QtWebEngine is not properly installed. It is not in /nix/store/-qt-5.6.1-1/lib/.

which is fixed by https://github.com/NixOS/nixpkgs/pull/20569

kmail, akregator and kontact all crash in the same line in grantleethemes.

This PR depends on https://github.com/NixOS/nixpkgs/pull/20569

Cleaned up the PR.

What is the status here?

It builds for me with master. Yet on my machine kmail does not run. I believe the situation might be better for @ttuegel .
It seems I'll have to make debug build to figure out the problem for my situation. Due to the large number of packages, that does not seem an easy endeavor.

Is there some news on that project?

No, I've not looked at it for a while. The packages have been reshuffled since the last work on this.
I'm still on KMail 1 for now.

Now that 17.03 has plasma5 this has higher urgency, I guess.

Now that 17.03 has plasma5 this has higher urgency, I guess. especially since kmail seems to be gone from 17.03.

messagelib in applications 16.12.3 still requires WebKitWidgets
https://cgit.kde.org/messagelib.git/tree/CMakeLists.txt?h=v16.12.3
This requirement is gone in v17.03.80.

@grahamc @lprndn @ttuegel
This patch is now updated to work with NixOS 17.03.

starting kmail in a clean nixos 17.03 gives this:
ERROR: Caught a segmentation fault while loading plugin file:
/nix/store/zz6zv5kq75h2d0ma7xg452wm1wbx7llx-gst-plugins-base-1.10.4//lib/gstreamer-1.0/libgsttypefindfunctions.so

The current version can run kaddressbook, but kmail, kontact and akregator crash on startup.

@ttuegel  do you have an opinion on the patch set as it is now?

The only comments I have are:

Try to remove extraneous imports like kdeWrapper (which is not needed for the libraries).
Please add the libraries to the list in all-packages.nix.

With the patch to avoid a crash in Grantlee, KMail does not crash, but it does terminate with an error.
$ kmail
org.kde.pim.kidentitymanagement: IdentityManager: There was no default identity. Marking first one as default.
sonnet.core: No language dictionaries for the language: "en"
sonnet.core: deleting 0x1ecdef0 for "en"
sonnet.core: Unhandled script 126
sonnet.core: Unhandled script 127
sonnet.core: Unhandled script 128
sonnet.core: Unhandled script 129
sonnet.core: Unhandled script 130
sonnet.core: No language dictionaries for the language: "en"
sonnet.core: No language dictionaries for the language: "en"
sonnet.core: No language dictionaries for the language: "en"
sonnet.core: No language dictionaries for the language: "en"
sonnet.core: deleting 0x2136b30 for "en"
sonnet.core: deleting 0x2136968 for "en"
sonnet.core: No language dictionaries for the language: "en"
sonnet.ui: No dictionary for " "en" " staying with the current language.
sonnet.core: No language dictionaries for the language: "en"
this does not work on a KActionCollection containing actions!
org.kde.akonadi.ETM: GEN true false false
org.kde.akonadi.ETM: collection: QVector()
org.kde.akonadi.ETM:
grantlee.template: "Plugin library 'kde_grantlee_plugin' not found."
org.kde.akonadi.ETM: Subtree:  4 QSet(14, 13, 12, 11, 6, 5, 4)
org.kde.akonadi.ETM: Fetch job took  264 msec
org.kde.akonadi.ETM: was collection fetch job: collections: 7
org.kde.akonadi.ETM: first fetched collection: "Local Folders"
org.kde.pim.akonadicore: "Unable to obtain agent type ''."
org.kde.pim.akonadicore: Failed SpecialCollectionsRequestJob::slotResult "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.mailcommon: "The Email program encountered a fatal error and will terminate now.\nThe error was:\nUnable to obtain agent type ''."
org.kde.pim.akonadicore: "Unable to obtain agent type ''."
org.kde.pim.akonadicore: Failed SpecialCollectionsRequestJob::slotResult "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.mailcommon: "The Email program encountered a fatal error and will terminate now.\nThe error was:\nUnable to obtain agent type ''."
org.kde.pim.akonadicore: Failed to request resource "" : "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.akonadi.ETM: collection: QVector()
org.kde.akonadi.ETM: Fetch job took  499 msec
org.kde.akonadi.ETM: was collection fetch job: collections: 6
org.kde.akonadi.ETM: first fetched collection: "Search"
org.kde.pim.akonadicore: "Unable to obtain agent type ''."
org.kde.pim.akonadicore: Failed SpecialCollectionsRequestJob::slotResult "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.mailcommon: "The Email program encountered a fatal error and will terminate now.\nThe error was:\nUnable to obtain agent type ''."
org.kde.pim.akonadicore: Failed to request resource "" : "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.akonadi.ETM: Fetch job took  44 msec
org.kde.akonadi.ETM: was collection fetch job: collections: 0
org.kde.pim.akonadicore: "Unable to obtain agent type ''."
org.kde.pim.akonadicore: Failed SpecialCollectionsRequestJob::slotResult "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.mailcommon: "The Email program encountered a fatal error and will terminate now.\nThe error was:\nUnable to obtain agent type ''."
org.kde.pim.akonadicore: Failed to request resource "" : "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.akonadicore: "Unable to obtain agent type ''."
org.kde.pim.akonadicore: Failed SpecialCollectionsRequestJob::slotResult "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.mailcommon: "The Email program encountered a fatal error and will terminate now.\nThe error was:\nUnable to obtain agent type ''."
org.kde.pim.akonadicore: Failed to request resource "" : "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.akonadicore: "Unable to obtain agent type ''."
org.kde.pim.akonadicore: Failed SpecialCollectionsRequestJob::slotResult "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
org.kde.pim.mailcommon: "The Email program encountered a fatal error and will terminate now.\nThe error was:\nUnable to obtain agent type ''."
org.kde.pim.akonadicore: Failed to request resource "" : "Cannot connect to the Akonadi service. (Unable to obtain agent type ''.)"
sonnet.core: Removing loader:  Sonnet::Loader(0x7fee1bd65190)

A dialog shows this message:
The Email program encountered a fatal error and will terminate now.
The error was:
Unable to obtain agent type ''.

Updated to 17.04.0 but still crashing kmail on startup when running in NixOS 17.03.

The crash is at least in part due to Akonadi trying to compare timestamps to a file in the Nix store; of course that isn't going to be helpful! I'm trying to patch around it now.

KMail is working on my branch, but I haven't had a chance to look at the rest of the PIM suite yet.

@ttuegel I'm looking forward to trying your branch tomorrow

@ttuegel how should one install kmail at the moment? I'm testing on a system with no system-wide kde, but a different desktop and run nix-env -f. -i kmail.

@vandenoever It should work with nix-env -i. It does depend on some KDE services, but those should be automatically installed by nix-env. You may need to run akonadictl start before kmail; it can start Akonadi itself, but I noticed that the first-run wizard gets confused if KMail has to start Akonadi.

kmail runs and does not crash as easily. (I can make it crash by doing 'Save as draft' on a mail when no account is set up.)
An issue is that no icons are visible except two envelopes in the 'Help' menu. Installing oxygen-icons5 does not help, not even when it is followed by kbuildsycoca5.

I can make it crash by doing 'Save as draft' on a mail when no account is set up.

That may--or may not--be a packaging problem. The KDE PIM team really loves null pointers. 😉

Installing oxygen-icons5 does not help, not even when it is followed by kbuildsycoca5.

The default icon theme is breeze-icons; does it help to install those? It is also possible that Qt is trying to imitate your GTK theme, which seems to be its fallback if KDE is not running.

Adding breeze-icons does not help. I'm not running gtk, but luminence desktop. That is just to see if kmail works fine in a non-kde environment. Normally, I do run kmail under plasma.
I'm now recompiling akonadi with a oxygen-icons5 as a propagated build input.

Including oxygen-icons5 with akondi does not solve the icon problem.
I've now tried to run kmail from @ttuegel's branch with stable nixos 17.03. That fails with this error:
$ kmail
This application failed to start because it could not find or load the Qt platform plugin "xcb"
in "/nix/store/ssk5scyy69zzzwlmbqvp74383vch01if-qtbase-5.8.0/lib/qt5/plugins".

Available platform plugins are: wayland-org.kde.kwin.qpa, minimal, offscreen, xcb, vnc.

Reinstalling the application may fix this problem.
Aborted

That's probably caused by the fact that @ttuegel's branch uses Qt 5.8.0 and NixOS 17.0.3 is on Qt 5.6.2.

Including oxygen-icons5 with akondi does not solve the icon problem.

Try running with the environment variable XDG_CURRENT_DESKTOP=KDE.

That's probably caused by the fact that @ttuegel's branch uses Qt 5.8.0 and NixOS 17.0.3 is on Qt 5.6.2.

That isn't supposed to happen anymore because the QPA plugin path is hardcoded. I will investigate.

Try running with the environment variable XDG_CURRENT_DESKTOP=KDE.
That makes no difference. Still no icons in kmail from your branch.
breeze and oxygen icons are installed, kbuildsycoca5 was run.

On nixos-unstable konsole also does not show icons when not running under plasma. Even with XDG_CURRENT_DESKTOP=KDE.
Excerpt from configuration.nix:
  # Enable the KDE Desktop Environment.
  services.xserver.displayManager.lightdm.enable = true;
  services.xserver.desktopManager.lumina.enable = true;
#  services.xserver.displayManager.sddm.enable = true;
#  services.xserver.desktopManager.plasma5.enable = true;

$ nix-env -q
breeze-icons-5.33.0
firefox-52.0.1esr
git-2.12.2
kmail-17.04.0
konsole-17.04.0
oxygen-icons5-5.33.0
pstree-2.39

When I install plasma from nixos-unstable, konsole and kmail have icons. They also have the same Qt version.
Account assistant shows triple and double entries.

Nice that it works on unstable. I'm not willing to make that my daily work environment though.

On nixos-unstable konsole also does not show icons when not running under plasma. Even with XDG_CURRENT_DESKTOP=KDE.

No icons when Plasma is installed, but not running? Or no icons when Plasma is not installed?

When plasma is installed and running, icons are visible. Otherwise they are not.
When plasma is installed, but the running desktop is lumina, then kde applications (dolphin, konsole) do not show icons.

@vandenoever I opened #25762 to track the problem with the icons.

I've now tried to run kmail from @ttuegel's branch with stable nixos 17.03. That fails with this error:

The problem of missing platform plugins is something that I fixed in master, but I had to rollback slightly to get Akonadi working, so I need to fine-tune this more.

Superseded by https://github.com/NixOS/nixpkgs/pull/28418


Unable to  process braw file

I collect a video with BMD camera, and try to process the video with the extract frame in the example. Get the following error:
  File "examples/extract_frame.py", line 67, in <module>
    main(sys.argv[1:])
  File "examples/extract_frame.py", line 38, in main
    clip = verify(codec.OpenClip(opts.input))
  File "/home/zhangsiyu/miniconda3/envs/pybraw/lib/python3.7/site-packages/pybraw/__init__.py", line 32, in verify
    assert result.is_success(), f'unsuccessful result code: {ResultCode(result).to_hex()} ({ResultCode(result).name})'
AssertionError: unsuccessful result code: 0x80000008 (E_FAIL)

I tried the same code with that sample braw file provided Filmplusgear-skiers-Samnaun-2019-dci-Q5.braw and everything is fine.
pytest tests says
FAILED tests/test_torch.py::test_read_frames[device_cuda0] - ValueError: Pipeline blackmagicRawPipelineCUDA is not supported by this machine
FAILED tests/test_torch.py::test_read_frames[device_cuda1] - ValueError: Pipeline blackmagicRawPipelineCUDA is not supported by this machine

with the provided sample input

I also tried with the latest Blackmagic RAW SDK, the results are even worse: most of tests failed.
Below is some properties of my video, compared with the sample braw file. Hope this would help

Unfortunately I do not own any Blackmagic equipment and do not have the time to investigate compatibility issues. I will now archive the repository to indicate that it is no longer maintained.


Possible to force re-render after SVG file change?

Thanks for this project! It's great to be able to edit files and see the preview side-by-side in VS Code.
Sometimes, when I make a change in the text of the SVG file, the preview updates, even before I save the file. That's great! I'm generating the SVG file with some code, though, and when I re-run the generation program, the preview often doesn't seem to update. Sometimes it does update, but not always. The only pattern I've noticed is that the longer I've been looking away from the preview, the more likely the preview is to not update.
Would it be possible to have a 'Force SVG Preview Refresh' command? A file watch might be nice, but given the way the plugin currently watches the SVG file buffer before it's even saved, this might be harder than just re-rendering from whatever's on disk.
Thanks!

This means that vscode.workspace.onDidChangeTextDocument may not fire for file updates on disk. I think a little.