nixos/dovecot: add support for sieve extensions, in particular imapsieve and pipe

Description of changes

Finally upstreaming some stuff from my config that I've been running for years. This is used on my mail server, where I use it to drive automatic learning for the spam filter. In action here: https://git.dblsaiko.net/systems/tree/configurations/polaris/dovecot.nix?id=8ea672890c78b059e002920628636234c7541c1f#n54
Adds general support for enabling Sieve plugins/extensions, support for the IMAPSieve plugin and support for specifying which binaries are available from the vnd.dovecot.pipe extension (part of the Extprograms plugin). IMAPSieve is used to trigger Sieve scripts on IMAP user actions, like moving mails around. vnd.dovecot.pipe can be used to run external programs from a Sieve script, for example passing the mail to rspamd to register it as spam.
Documentation for the IMAPSieve plugin: https://doc.dovecot.org/configuration_manual/sieve/plugins/imapsieve/
Documentation for the Extprograms plugin: https://doc.dovecot.org/configuration_manual/sieve/plugins/extprograms/
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@2xsaiko This broke my production deployment of dovecot2 with the following error:
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: after: line 1: error: require command: unknown Sieve capability `fileinto'.
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: after: line 4: error: unknown command 'fileinto' (only reported once at first occurrence).
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: after: error: validation failed.
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: sievec: Fatal: failed to compile sieve script '/var/lib/dovecot/sieve/after'

This probably misinteracts with https://gitlab.com/simple-nixos-mailserver/nixos-mailserver/-/blob/master/mail-server/dovecot.nix?ref_type=heads#L310-330.

After manual fixups, that is adding:
  services.dovecot2.sieve = {
    plugins = [ "sieve_imapsieve" "sieve_extprograms" ];
    extensions = [ "fileinto" ];
    globalExtensions = [ "+vnd.dovecot.pipe" "+vnd.dovecot.environment" ];
  };

It still complains with:
Jan 14 20:43:39 amadeus dovecot2-pre-start[3701122]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 20:43:39 amadeus dovecot2-pre-start[3701174]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 20:43:40 amadeus dovecot2-pre-start[3701175]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory

but is now not broken anymore.

Jan 14 20:43:39 amadeus dovecot2-pre-start[3701122]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 20:43:39 amadeus dovecot2-pre-start[3701174]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 20:43:40 amadeus dovecot2-pre-start[3701175]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory

Hmm, that most likely comes from line 31 in dovecot-2.3.21/src/lib-old-stats/stats-connection.c which is called from line 394 in dovecot-2.3.21/src/plugins/old-stats/stats-plugin.c. Are you sure that error wasn't there before? This change isn't loading the old-stats plugin and I'm not getting that error either (it's also deprecated). Either way, it shouldn't be fatal from what I can see.

You are probably right on the fact this existed before, I am just finishing
mitigating the rest and then I will look further.
Le dim. 14 janv. 2024, 21:22, Marco Rebhan @.***> a
écrit :

Jan 14 20:43:39 amadeus dovecot2-pre-start[3701122]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 20:43:39 amadeus dovecot2-pre-start[3701174]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 20:43:40 amadeus dovecot2-pre-start[3701175]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Hmm, that most likely comes from line 31 in
dovecot-2.3.21/src/lib-old-stats/stats-connection.c which is called from
line 394 in dovecot-2.3.21/src/plugins/old-stats/stats-plugin.c. Are you
sure that error wasn't there before? This change isn't loading the
old-stats plugin and I'm not getting that error either (it's also
deprecated). Either way, it shouldn't be fatal from what I can see.
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/275031#issuecomment-1891062527,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACMZREWULI3PVG47O5LYKTYOQ5APAVCNFSM6AAAAABAYQEB76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJRGA3DENJSG4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

@2xsaiko This broke my production deployment of dovecot2 with the following error:
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: sievec: Error: stats: open(/run/dovecot2/old-stats-mail) failed: No such file or directory
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: after: line 1: error: require command: unknown Sieve capability `fileinto'.
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: after: line 4: error: unknown command 'fileinto' (only reported once at first occurrence).
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: after: error: validation failed.
Jan 14 18:34:30 amadeus dovecot2-pre-start[3655287]: sievec: Fatal: failed to compile sieve script '/var/lib/dovecot/sieve/after'

I can confirm this is an issue. We ran into it after a flake bump yesterday. Can test the PR if desired.


The /preview URL is broken

It leads to a status code 404 error.

Not sure whether we should fix this.  In v18 the /preview URL redirected to /course. For a registered user, the redirect produced the course with the syllabus.  For an unregistered user, the course produced a slightly different paging.  In v18 there was a handler in controllers/utils.py that, for non-registered (transient) users, rendered the views/preview.html.  There was also a redirect controlleres/lessons.py in the CourseHandler that redirected to preview.html.
In v1.10 there is no preview.html in /views and there is no PreviewHandler in controllers/utils (that redirects to view/preview.html).  The CourseHandler controller has been moved into module/courses/lessons.py and the controllers/lessons.py is gone.  The CourseHandler controller no longer redirects transient students to /preview.
So, not sure how to handle this.  It looks like the /preview URL has been done away with in v1.10.


fix: ⚡ Fix dependency vulnerabilities

Disclaimer
This is updating a lot of dependencies. To properly test it locally if you have already pulled the repo before, you must first delete node_modules and re-run npm i

Current main:
50 vulnerabilities (27 moderate, 20 high, 3 critical)

This PR:
10 vulnerabilities (7 moderate, 3 high)

I'm following dependabot alerts for now.
Most of them actually came from the templates.
Fixes:
dependabot/2
dependabot/9
dependabot/50
dependabot/26

(it's not ready for review, I'm trying to trigger the action)

Ready to review.
IMO we should merge it and not Squash it because there is a lot going on, I tried to separate the commits properly

@xat Do you know why there is no action running anymore?

hmm, the action did work in the pr that i just opened. maybe a temp problem?

@melMass i would suggest that we wait a week or so, before merging this one after we ironed out the more pressing bugs that got revealed due to the bigger release. because updating dependencies can bring in further instability, which i would want to avoid until things are bit more settled.
would it make sense to move the "get random objkts" commit into a separate pr, then we could merge that one upfront?
what do you think?

@melMass i would suggest that we wait a week or so, before merging this one after we ironed out the more pressing bugs that got revealed due to the bigger release.

Yes it makes sense, I spent hours on it so I won't have the same time to dedicate fixing it in a week, but we could use it as a reference or something like that 👌

@melMass ah ok, so delaying the merge would create extra effort.
hmm, do you think it is safe enough to merge it sooner? my feeling is that it does require some extended testing.

@melMass ah ok, so delaying the merge would create extra effort.
hmm, do you think it is safe enough to merge it sooner? my feeling is that it does require some extended testing.

Yes it definitely need more extensive testing, but I meant more if we fix other stuffs under it it might be better to start this one over, following it as a ref. I spent most of my time checking breaking changes in libraries, I know them now

Rebased 😏  🤣

looks very nice!

Ok now it's ready to be reviewed for real @xat 😇

from the first review, there are few (small) open questions, but otherwise it looks good!
maybe some non-devs can do a bit of manual testing before we merge it.

@xat Mmhh so while debugging the feed thing (that's actually search), it seems src/pages/feed isn't used anymore?

nice work :>


Update delegate urls argument for dashboard oauth

Description
There was a misconfiguration in the --openshift-delegate-urls parameter
for the authentication overlay of the dashboard. This caused
authentication/authorization to fail for some use cases, particularly
when attempting to authenticate to the dashboard programmatically using
a bearer token. This change updates the argument to a valid value to fix
this use case.
How Has This Been Tested?
This has been tested on an openshift 4.11 cluster by performing the following:

Deploy the ODH operator to your cluster from the operator catalog
Create a namespace for testing this change:

oc new-project auth-fix

Create a kfdef to deploy the current version:

cat <<EOF | oc apply -f -
apiVersion: kfdef.apps.kubeflow.org/v1
kind: KfDef
metadata:
  name: opendatahub
  namespace: auth-fix
spec:
  applications:
    - kustomizeConfig:
        overlays:
          - authentication
        repoRef:
          name: manifests
          path: manifests
      name: odh-dashboard
  repos:
    - name: manifests
      uri: 'https://github.com/opendatahub-io/odh-dashboard/tarball/main'
EOF

Run the following command to attempt to log into the dashboard using the odh-dashboard service account. The command should output 403, indicating that the service account has been restricted from accessing the dashboard:

curl --header "Authorization: Bearer $(oc get secret -n auth-fix -o jsonpath='{.data.token}' `oc get secret -n auth-fix | grep odh-dashboard-token | awk '{print $1}'` | base64 -d)" -s -o /dev/null -w "%{http_code}\n" -k https://$(oc get route -n auth-fix odh-dashboard -o jsonpath='{.spec.host}')

Update the kfdef to deploy this change by running the following:

cat <<EOF | oc apply -f -
apiVersion: kfdef.apps.kubeflow.org/v1
kind: KfDef
metadata:
  name: opendatahub
  namespace: auth-fix
spec:
  applications:
    - kustomizeConfig:
        overlays:
          - authentication
        repoRef:
          name: manifests
          path: manifests
      name: odh-dashboard
  repos:
    - name: manifests
      uri: 'https://github.com/accorvin/odh-dashboard/tarball/fix-oauth'
EOF

Run the curl command again. This time it should print a status code of 200, indicating that the service account has successfully reached the dashboard:

curl --header "Authorization: Bearer $(oc get secret -n auth-fix -o jsonpath='{.data.token}' `oc get secret -n auth-fix | grep odh-dashboard-token | awk '{print $1}'` | base64 -d)" -s -o /dev/null -w "%{http_code}\n" -k https://$(oc get route -n auth-fix odh-dashboard -o jsonpath='{.spec.host}')

Navigate to the dashboard in a browser. Confirm that you can log in and use the dashboard as normal.

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

@samuelvl @cfchase -- could you take a look at this PR?

/approve

/retest


Support docker-build module with CI pipeline

Is your request related to a problem? Please describe.

I've been trying to use docker-build submodule to build a container image in GitHub Actions, and deploy it with CodeDeploy using deploy module but docker_image  resource which will be automatically created by docker-build module will be  forcibly replaced every time when CI pipeline get triggered withtout changing any code because docker_image refaers to local image, and so the local iamge will be reseted every CI process.
Describe the solution you'd like.
To avoid the avobe behavior, It would be better to have an option to specify whether local docker image is stored on local or not by using ignore_changes of terraform lifecycle. Since ignore_changes doesn't support dynamic paramter on it currently, I think one of the workaround to implement this is like the following:
resource "docker_image" "this" {
  count = var. store_on_local == true ? 1 : 0
  name = local.ecr_image_name
  triggers     = var.triggers
  lifecycle {
    ignore_changes = [id, image_id]
  }
}

resource "docker_image_with_ci" "this" {
  count = var. store_on_local == true ? 0 : 1
  name = local.ecr_image_name
  triggers     = var.triggers
  lifecycle {
    ignore_changes = [id, image_id]
  }
}

If there is another workaround for this problem, or I misunderstand something, please correct me. Thanks!

I think is the same issue : https://github.com/terraform-aws-modules/terraform-aws-lambda/issues/517

I create a pull request https://github.com/terraform-aws-modules/terraform-aws-lambda/pull/518 but no new from the reviewer

Fixed in https://github.com/terraform-aws-modules/terraform-aws-lambda/pull/518 by @IlyesDemineExtVeolia

@IlyesDemineExtVeolia Thanks! I've checked out if your changes doesn't trigger to replace docker_image resource but   docker_image resource seems to still has problem that I mentioned here while docker_registry_image will be updated properly based on the new trigger you changed though 🤔


Improve error handling and messaging in injection_napari_dependencies hints

Description
Arg... I'm so mad, I just wrote a long detailed explanation for this and then somehow lost it before i could submit :angry:  ... so now you get the short version :/
This PR came about because, on main, if you run the following code:
@inject_napari_dependencies
def func(param: 'UnknownType' = 1): ...

you'll get NameError: name 'UnknownType' is not defined.
However, since it has a default, it really isn't necessary to resolve it.
So this PR:

breaks out the type resolution logic into an independent, reusable module, with lots of new tests
adds lots of docstrings and inline comments about how it all works
catches a number of potential "gotchas" when injecting dependencies into a function (for example, a function with a required parameter that had no type hints would have previously made it through without warning, but then fail when called later).
is more forgiving in the cases above, when a default parameter is already provided
adds a lot of informative errors and/or warnings (all of which can be suppressed or raised depending on parameters)
brings test coverage on the _injection module close to 100%

some additional context here:  I see inject_dependencies as the best way forward to use "regular old" well-annotated python functions to provide context specific functionality (such as acting on the selected layer, getting a viewer, etc.).  I see it as something that both plugin functions will go through as well as internal functions.  So it's worth making it very solid and clear

Looks ok.
I have one question (that does not need to be resolved in this PR). If someone will use inject dependency on the method of qt widget form plugin it may keep it alive even after the close of the widget if it is connected to some signal?

If someone will use inject dependency on the method of qt widget form plugin it may keep it alive even after the close of the widget if it is connected to some signal?

i'm actually thinking about pulling this out into a new package.  It's rather generic.  (the only part about this that is specific to napari is a single dictionary that injects names).  That would allow easier performance scrutiny/benchmarks, test iteration, etc...  working on it here: https://github.com/tlambert03/in-n-out
(name can change :joy: partially in honor of our hackathon, but also cause this processes ins and outs :joy:)

Do you want to wait until you have it sorted out in it's own package ?

Do you want to wait until you have it sorted out in it's own package ?

well, I'll leave it up to you and @napari/core-devs.  Since this is strictly an improvement over the existing behavior (aims to have no breakages, supports more use cases, and provides more useful feedback when it fails), this could be merged while I work on the other package ... which is coming along. It would be useful for some of the other plugins/commands PRs
But, if the burden of review feels too big, that's fine too.

since there's been no response, and since this is included in in-n-out (which is included in #4784), I'm going to close this.  but this is a strict improvement over what we currently have, so if for any reason #4784 doesn't get accepted, this should be reopened and merged


Change necessary files check to use iget for permission issues

I had added a feature that killed the pipeline if necessary files were not found, this doesn’t work with the updated view permissions on cyverse because it can’t find the files, need to update the code to use iget.

2023 April 14 - Farabi also is reporting this issue.


Add FirebaseUser.updateProfile to [auth] for [ios/android]

Library
Class
Member
Platforms

e.g auth
e.g FirebaseUser
e.g updateProfile
Android, iOS

Currently, one needs to: .android.updateProfile(UserProfileChangeRequest.Builder().setDisplayName()) which is awkward and can't be done in commonCode.

Any update on this? I'm also not able to access anything related to updating the user's profile in the common module.

Not yet, but PRs welcome!

Feel free to take a stab on this @vladimirpetrovski

Im making an implementation of this. Hope to have something early next week.


Signal hound a few more changes

While trying to resolve a usb comm error for @nataliejpg I cleaned up the driver a bit

Add types to ctypes dll calls matching the api defined in Signal Hounds headers. The only real changes are a few uint -> int that should have no real consequence as we are dealing with small positive numbers
Rewrite _get_sweep_data to capture data directly into a numpy array saving a loop over the data
refactor the averageing logic into _get_sweep_data removing the avg function. This saves a few redundant dll calls
remove the retry functionality from _get_sweep_data. The api docs are quite clear that there is no point in retrying when the communication is lost. The only way forward is to close the device. All other errors relate to invalid input and will not be fixed by retrying

@nataliejpg confirms that this is working


mingwSearchDirs has no effect

Brief Issue Summary
mingwSearchDirs has no effect
Expected:
I have two MinGW installations in c:\mingw and c:\mingw_differentinstall.
I added both dirs to
"cmake.mingwSearchDirs": [
        "C:\\MinGW", "C:\\mingw_differentinstall"
    ]

I expect both MinGW installations to be present as kit after scan for kits.
Apparent Behavior:
CMT finds only the C:\mingw installation, which is also in PATH. This behavior is also described in #309, workaround seems to be to add it to PATH.
As  I have two different installations, I don't want to add both to PATH.
CMake Tools Log
Developer Tools Log
Platform and Versions
[[For issues, fill out the following. Otherwise delete this section.]]

Operating System: Windows 7
CMake Version: 3.11
VSCode Version: 1.23.0
CMake Tools Extension Version: 0.11.1
Compiler/Toolchain: MinGW

Other Notes/Information

I can confirm it works now.


C6 Xingqiu should only proc on a 5 sword wave

See the code:
https://github.com/genshinsim/gcsim/blob/468a704a5fdeb67e84738b454979e32068c75b6a/internal/characters/xingqiu/burst.go#L19
This check is insufficient. If Xingqiu's Burst ends on a 3 sword wave, then c.nextRegen will be True for the first wave of the next Xingqiu Burst, which leads to it proccing C6. I believe it should also check for c.numSwords == 5.
originally found by Dooners#6709:
https://discord.com/channels/845087716541595668/869210750596554772/970055888910045194

You're right. I'm closing this issue.


Media plugin bug on Android when pressing home and reentering application which calls TakePhotoAsync() again

From @igorczapski on January 4, 2016 14:44
Steps to reproduce:

After Activity initialization call:
var photo = await CrossMedia.Current.TakePhotoAsync(new StoreCameraMediaOptions{
Directory = SOME_DIRECTORY,
Name = SOME_NAME
});
When you have photo app opened, press HOME button on device.
Reenter the app, go to your activity which calls method above.
InvalidOperationException -> Only one operation can be active at a time.

It doesn't matter if it's Xamarin.Forms or Xamarin.Android, obviously.
Workaround:
In Xamarin.Android just use the standard way with intents and OnActivityResult, or at least you can make some use of intent creation method, from this library.
var media = new MediaImplementation();
var intent = media.GetTakePhotoUI (new Abstractions.StoreCameraMediaOptions {
Directory = DIRECTORY,
Name = NAME
});
StartActivityForResult (intent, ...);
Copied from original issue: jamesmontemagno/Xamarin.Plugins#174

Not able to reproduce

Hey James, I'm hitting this issue on an Android app, so I tried to reproduce it on the sample available in the repo and it's fairly easy to do it.

Run MediaSample.Droid
Tap on "TAKE PHOTO"
When the camera opens, tap the home button on the device
Tap on the app launcher icon
Tap on "TAKE PHOTO"
💥

Maybe this should be reopened?


TASK [../roles/mapr-configure-security-fetch-keys : Copy keys to local]

from sites/mapr-core.yml

name: Fetch keys, when no CLDB change happened
hosts: mapr-cldb[0]
roles:

{role: ../roles/mapr-configure-security-fetch-keys,
when: mapr_security != 'none' and (cldb_changes|int) == 0}

And we have in role: ../roles/mapr-configure-security-fetch-keys, this stuff :+1:

name: Copy keys to local
fetch: src=/opt/mapr/conf/{{ item }} dest=/tmp/{{ item }} flat=yes
with_items:

maprserverticket
cldb.key
ssl_keystore
ssl_truststore

Then generate an error
TASK [../roles/mapr-configure-security-fetch-keys : Copy keys to local] *****************************************************************************************************
failed: [<IP_ADDRESS>] (item=maprserverticket) => {"changed": false, "item": "maprserverticket", "msg": "file not found: /opt/mapr/conf/maprserverticket"}
failed: [<IP_ADDRESS>] (item=cldb.key) => {"changed": false, "item": "cldb.key", "msg": "file not found: /opt/mapr/conf/cldb.key"}
failed: [<IP_ADDRESS>] (item=ssl_keystore) => {"changed": false, "item": "ssl_keystore", "msg": "file not found: /opt/mapr/conf/ssl_keystore"}
failed: [<IP_ADDRESS>] (item=ssl_truststore) => {"changed": false, "item": "ssl_truststore", "msg": "file not found: /opt/mapr/conf/ssl_truststore"}
Cheers

Are you setting up a cluster with security?
Are you running the script a second time?
Please send me a private message with the group_vars/all configuration or post it here and remove sensitive stuff.

I faced the same error when setting up a cluster without security (security_all: none in group_vars/all) configuration.
I made a temporary fix by moving
- name: Copy keys to local
  fetch: src=/opt/mapr/conf/{{ item }} dest=/tmp/{{ item }} flat=yes
  with_items:
  - ssl_truststore
  - mapruserticket
  when: inventory_hostname == groups["mapr-cldb"][0]

to a new role (for example mapr-client-configure-fetch-key) and modify site-client.yml as well
- name: Fetch files from cluster required for client
  hosts: mapr-cldb[0]
  roles:
    - {role: mapr-client-configure-fetch-key, when: mapr_security != 'none'}
    - mapr-client-configure-nativesecurity-fetch

Hope to see the official fix soon.
Thanks a lot.

Is it during cluster or client setup?

It is during client setup.

ok, cluster setup works properly?
Since you already created a fix, can you create a pull request?

Yes, cluster setup works properly. Only had this problem with client setup.
I'll create a pull request.

Great, thanks.

merged.


Hello, I'd like to know if this work can be used in real-time detection?

Hello, I'd like to know if this work can be used in real-time detection?

This work cannot run in real time

This work cannot run in real time


text update found by kevin

Update some text listed in #1272

Register an a-class seedlot:

[x] I would remove the subheading "Register a new A-class seedlot" (redundant with heading above)

Originally posted by @kevinginley in #1272

A-class seedlot created:

[x] Change "Seedlot's main screen" to:

"Seedlots main screen"
(removing apostrophe)

Originally posted by @kevinginley in #1272

Client search modal:

"To view more information about the client, you can go to client search screen."

[x] change to:

"To view more information about the client, go to the client search screen."

Originally posted by @kevinginley in #1272

Collection methods:

[ ] Would it make sense to list the collection methods in alphabetical order?

Originally posted by @kevinginley in #1272

Cone and pollen count:

"Enter the estimative of Cone and pollen count for the orchard's seedlot (*required)"
I would change it to:

[x] "Enter the cone and pollen count estimate for the orchard's seedlot"

SMP success on parent:

"Enter the same SMP success on parent or Non-orchard pollent contaminant to all parent trees"

[x] I would change "Non-orchard" to "non-orchard"

Calculation of SMP mix:

"Enter the estimative of SMP success for the orchard's seedlot"
I would change to:

[x] "Enter the SMP success estimate for the orchard's seedlot"

Originally posted by @kevinginley in #1272

File upload:

"Note: When uploading a file all previously filled data within the table will be replaced."
Please change to:

[x] "Note: When uploading a file, all previously filled data in the table will be replaced."

Originally posted by @kevinginley in #1272

Breeding value of SMP mix used

The subheading is redundant here in my opinion. I would remove that and update the heading to read:
"Breeding value of SMP mix used on parent"

Originally posted by @kevinginley in #1272

"SMP Mix geospatial summary"
Please change to:

[x] "SMP mix geospatial summary"

Originally posted by @kevinginley in #1272

[ ]

Step 6:

"Enter the extractory agency information and extraction's star and end dates for this seedlot"
change to:

[x] "Enter the extractory agency information and extraction's start and end dates for this seedlot"

(change "star" to "start")

"Enter the seed storage agency information and storage's star and end dates for this seedlot"
change to:

[x] "Enter the seed storage agency information and storage's start and end dates for this seedlot"

(change "star" to "start")

Originally posted by @kevinginley in #1272

[x] Change instances of "Tree Seed Center" to instead read "Tree Seed Centre"

This one's a bit tricky, but in Canada we use the French spelling of center, so it should actually be "Tree Seed Centre" instead of "Tree Seed Center"
When Tree Seed Centre comes up in data queries, it is spelled the French way in the database, so it would be nice to keep consistent.
Hopefully this is more of a find-and-replace kind of function in the front-end

Originally posted by @kevinginley in #1272

Declaration modal:

Declaration:
"Please, be sure to review the content and check if everything is correct with your seedlot registration.
Go to first step and review the form."
Please change to:

[x] "Please review this form to ensure the information is correct for this seedlot registration.

Click here to go back to the first step."
And then have "Click here" link to the first step in the form

Originally posted by @kevinginley in #1272

Do we want to remove the subtitle for the favorite card too?
@kevinginley

Yes, thank you!

Reduced story point from 2 > 1, less complicated task.

@ngunner15
Looking good! A few remaining changes:
Client search
Please change "go to client search screen." to:
"go to the client search screen."
When I click "go to client search screen", it didn't take me anywhere. Also, I'm confused, because aren't we already on the client search screen?
Step 5:
I requested
"Enter the estimative of Cone and pollen count for the orchard's seedlot (*required)"
be changed to:
"Enter the cone and pollen count estimate for the orchard's seedlot"
My apologies - I didn't mean to remove "(*required)". Can you please update it to read:
"Enter the cone and pollen count estimate for the orchard's seedlot (*required)"
Please update the heading from
"Breeding value of SMP mix used" to:
"Breeding value of SMP mix used on parent"
and remove the subheading "Check the breeding value of SMP mix used on parent:
Step 6 -
"Click here" should link to the first step in the current form, if possible.
Thank you

Not sure if this is the right ticket to bring this up, but I noticed some funky text alignment issues on Step 5 when I had SPAR only take up half of my screen.

and also the placement of the search button alignment is looking a little funny in client search:

@ngunner15 Looking good! A few remaining changes: Client search
Please change "go to client search screen." to: "go to the client search screen."
When I click "go to client search screen", it didn't take me anywhere. Also, I'm confused, because aren't we already on the client search screen?
Step 5:
I requested "Enter the estimative of Cone and pollen count for the orchard's seedlot (*required)"
be changed to: "Enter the cone and pollen count estimate for the orchard's seedlot"
My apologies - I didn't mean to remove "(*required)". Can you please update it to read: "Enter the cone and pollen count estimate for the orchard's seedlot (*required)"
Please update the heading from "Breeding value of SMP mix used" to: "Breeding value of SMP mix used on parent"
and remove the subheading "Check the breeding value of SMP mix used on parent:
Step 6 - "Click here" should link to the first step in the current form, if possible.
Thank you

Completed all changes except Step 6, I will need more time to figure out how to add a link.

I'm not able to add a link to this component

@ngunner15 Looking good! A few remaining changes: Client search
Please change "go to client search screen." to: "go to the client search screen."
When I click "go to client search screen", it didn't take me anywhere. Also, I'm confused, because aren't we already on the client search screen?
Step 5:
I requested "Enter the estimative of Cone and pollen count for the orchard's seedlot (*required)"
be changed to: "Enter the cone and pollen count estimate for the orchard's seedlot"
My apologies - I didn't mean to remove "(*required)". Can you please update it to read: "Enter the cone and pollen count estimate for the orchard's seedlot (*required)"
Please update the heading from "Breeding value of SMP mix used" to: "Breeding value of SMP mix used on parent"
and remove the subheading "Check the breeding value of SMP mix used on parent:
Step 6 - "Click here" should link to the first step in the current form, if possible.
Thank you

Completed all changes except Step 6, I will need more time to figure out how to add a link.

To be completed by @mgaseta


Wayland support?

There is no built in wayland support in polybar. But you may be able to make it work, using xwayland.


"ImportError: No module named fields" when install

Hi,
suddenly the cookbook started giving this kind of error when running:
--
     * execute[python manage.py syncdb --noinput] action run

       ================================================================================
       Error executing action `run` on resource 'execute[python manage.py syncdb --noinput]'
       ================================================================================

       Mixlib::ShellOut::ShellCommandFailed
       ------------------------------------
       Expected process to exit with [0], but received '1'
   ---- Begin output of python manage.py syncdb --noinput ----
       STDOUT:
       STDERR: ImportError: No module named fields
       ---- End output of python manage.py syncdb --noinput ----
       Ran python manage.py syncdb --noinput returned 1

--
I tried to run the command by hand and I can get:
/opt/graphite/webapp/graphite# python manage.py syncdb --noinput
ImportError: No module named fields
Cookbook version is 1.0.2 and OS is Ubuntu 14.04.2 LTS.
Anyone else experiencing this?

I have an older setup which was previously installed fine; just did a quick pip- check; the old setup has django-tagging (0.3.6) but the new - broken one - has django-tagging (0.4)...

We just ran into the same issue.

Hi,
Yep same issue using Ubuntu 14.04, cookbook version 1.0.2
================================================================================
    Error executing action `run` on resource 'execute[python manage.py syncdb --noinput]'
    ================================================================================

    Mixlib::ShellOut::ShellCommandFailed
    ------------------------------------
    Expected process to exit with [0], but received '1'
    ---- Begin output of python manage.py syncdb --noinput ----
    STDOUT:
    STDERR: ImportError: No module named fields
    ---- End output of python manage.py syncdb --noinput ----
    Ran python manage.py syncdb --noinput returned 1

    Cookbook Trace:
    ---------------
    /home/vagrant/chef-solo/cookbooks-2/lxc/libraries/monkey.rb:20:in `monkey_shell_out!'

    Resource Declaration:
    ---------------------
    # In /home/vagrant/chef-solo/cookbooks-3/graphite-config/recipes/single_node.rb

     76: execute "python manage.py syncdb --noinput" do
     77:   user node['graphite']['user']
     78:   group node['graphite']['group']
     79:   cwd "#{base_dir}/webapp/graphite"
     80:   creates "#{storage_dir}/graphite.db"
     81:   notifies :run, "python[set admin password]"
     82: end
     83:
   Compiled Resource:
    ------------------
    # Declared in /home/vagrant/chef-solo/cookbooks-3/graphite-config/recipes/single_node.rb:76:in `from_file'

    execute("python manage.py syncdb --noinput") do
      action "run"
      retries 0
      retry_delay 2
      default_guard_interpreter :execute
      command "python manage.py syncdb --noinput"
      backup 5
      creates "/opt/graphite/storage/graphite.db"
      cwd "/opt/graphite/webapp/graphite"
      group "graphite"
      returns 0
      user "graphite"
      declared_type :execute
      cookbook_name :"graphite-config"
      recipe_name "single_node"
    end

I got around this temporarily reopening the resource in the recipe for my wrapper cookbook (simplified below):
r = resources(python_pip: 'django-tagging')
r.action :install
r.version "0.3.6"

and it's able to complete the syncdb operation. Hopefully someone finds it a helpful starting point until the issue is resolved upstream.

Temporary solution in Chef,
python_pip "django-tagging" do
  version "0.3.6"
end

Or manually,
pip install django-tagging==0.3.6

This is an issue here as well, on Ubuntu 14.04.
@bobhlo: The chef solution above didn't work for me but the manual one did. Thanks!

include_recipe 'python::pip'
@rambler just put
include_recipe 'python::pip'
before
```python_pip "django-tagging" do
version "0.3.6"
end

@rambler just put
include_recipe 'python::pip'
before
python_pip "django-tagging" do
    version "0.3.6"
end

@rambler, may need to remove newer version before applying old one
python_pip "django-tagging" do
  action :remove
end
python_pip "django-tagging" do
  version "0.3.6"
end

+1 - any chance of this being fixed?

Yes. I have also encountered and verified the suggested solution. We will try and get this fixed and release soon.

Friendly reminder that over 4 months have passed.

@bobhlo - "pip install django-tagging==0.3.6"
Thank you for the manual fix. It worked for me.


Move default btn methods to plugins

locator, zoomToArea, and fullscreen behave exactly like plugins.
These 'default' plugins have been moved to the plugins module from the default view to clean up the default view.
Methods have been moved out of the  tag for better maintenance and linting.

@simon-leech The plugin must accept a true flag. I did a commit for this. I agree that the default should be off. We should discuss what's the preferred setting for the default view.

@simon-leech can you re-review. The requested changes have been made.

@dbauszus-glx Is there a reason why there is a plugin param being passed to the function but isn't used?

@RobAndrewHurst No particular reason. It's just the default first param for these type of plugin methods. Could be null but doesn't hurt to know what the first param is.


tx-generator: reduce tracing output

Reduce the tracing output in the tx-generator/splitting phase.
Only trace summary message.

bors r+

bors r+

bors r+

bors r+

bors r-

bors r+


Try to improve on_delete settings

Do more to prevent accidental cascade deletion of builds. Allow cascade deletion of perf summaries.

My challenge is knowing when on_delete=models.SET_NULL is safe. How do we know if a view or controller will 500 if a build's org is null. and so forth.
Is there test code that renders the views with some of these values as NULL?

@prescod great point; I'll take another look at that.

@prescod ok, I took a closer look at the fields which are switching to SET_NULL. I'm pretty confident that it's okay to do this for org_instance and current_rebuild because it is already the case that they are not always set.

@prescod ok, I took a closer look at the fields which are switching to SET_NULL. I'm pretty confident that it's okay to do this for org_instance and current_rebuild because it is already the case that they are not always set.

Okay, as long as you've done that review, I'm happy.


[Candidate 1v1 Dammed Scorpion ZH v2] Supply Piles near spawn are not mirrored

Issue Description
Supply Piles near spawn points are not mirrored.
Expected behavior
Supply Piles near spawn points are mirrored.
Screenshots

As designed?

Also best to keep them more in a line formation towards the supply center, so that china trucks don't mess up and drive circles all day.

I expect the piles to be at the top of the map, right to the main supply, not south of it. Then they are correctly mirrored, or am I mistaken?

IMO keep them as on sand scorpion cause the ppl are used to it and it went well all the years back without any problems

Ok looking at map preview it actually looks mirrored. Not sure why I expect them on the other side.

Spacing to map border can be made consistent:

moved the bottom players crates a bit up, fixed.

Bottom right Supply Piles have much tighter space than the ones at top left.

Still happens in latest version.

No blocker. As designed. Cannot be easily tweaked without changing Piles distance to Command Center.


No implementation of plus(x)

Description
Some more spring cleaning to do.
The following Stan signatures currently dont have a Stan Math implementation:
int plus(int)
real plus(real)
vector plus(vector)
row_vector plus(row_vector)
matrix plus(matrix)

This is the named version of the unary + operator. So these two are the same
int i = +x;
int i = plus(x);

Current Version:
v3.2.0

There is also the question if this function is actually needed as its essentially just a no-op. Given that it's not even documented in docs we could also remove it without consequences.
In C it's used to promote unsigned types to convert unsigned int to signed and short to int. The other use is to ensure that an expression's result is treated as an rvalue.
The only counterpoint is to support it for symmetry with the unary minus().


codec parsing failed first

Overview
Create codec by av.CodecContext.create('hevc', 'r'), The result is empty execution time execute
packets = codec.parse(video_bytes)
Expected behavior
The resolution succeeds after the second attempt
Actual behavior
20240115-203922
Versions
OS: ubuntu 20.4
PyAV: runtime:12.0.2
python: 3.10.9

You need to install PyAV with pip install av, not pip install pyav. The version you installed doesn't have hevc compiled in.
Also please use text like this instead of images:
import sys
import av

f_w = open("hevc_test.h265", "br")
video_bytes = f_w.read()
codec = av.CodecContext.create("hevc", "r")
packets = codec.parse(video_bytes)
for packet in packets:
    for frame in codec.decode(packet):
        if isinstance(frame, av.VideoFrame):
            image = frame.to_image()
            image.save(f"demo.jpg")
            sys.exit()

thanks
I uninstall pyav and install with pip install av  , But the problem that couldn't be solved the first time remains


Power Rename can cause file paths to exceed MAX_PATH, making files inaccessible

Creating a new issue based on #8706
that is closed by the original creator,
because his specific situation is corrected.
Still caused by Power Rename.

ℹ Computer information
copy from original issue

PowerToys version: 0.25
PowerToy Utility: Power Rename
Running PowerToys as Admin: yes
Windows build number: 19042.685

📝 Provide detailed reproduction steps (if any)

Select a number of files
Use Power Rename (possibly with unintentional results)
causing the filenames to become too long
Files become inaccessible

✔ Expected result
Files are renamed and accessible
❌ Actual result
In a certain situation, the total file path is too long and cannot be accessed via Explorer or apps.
(afaik) Situation can depend on

registry key HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\FileSystem \ LongPathsEnabled with value 0 or 1
Group Policy: Local Computer Policy > Computer Configuration > Administrative Templates > System > Filesystem > NTFS

We could prevent this by checking the reg value and (if 0) then check the total length of every file that will be renamed. Give the user a warning/error message with the reason. Or start paths with \\?\.

@enricogior, this should be fixed for stability.

I just want to write down some thoughts I have about this.
First the question of how and when to notify the user. We could:

Check in every single update of the file name preview list. Mark the relevant files with a red (background)color and show a InfoBar. A MessageBox is not acceptable here, because that's way too "blocking". Maybe this can be combined with #1853.
Check all files when Rename is clicked, before actually renaming the files.
Check every file when it is his turn. This can spawn a lot of messages and that makes it a bad idea.
Skip bad files and show a message after the process. Seems simple to build, but is obviously very user un-friendly.

In any case, the user needs to know why there was a problem, and how they can fix it. This needs more text then will fit in a simple message box, so a link to some (web)page is going to be needed. We can link to e.g. https://docs.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation and/or copy the text to our own docs?

If it's already in awareness, then feel free to ignore the comment and the linked comment.

you may want to rename the title to reflect FilePaths instead, because no matter what A FileName itself can't exceed MAX_PATH(that is 255 char) because FileName length(255char) is hard coded/defined somewhere in a WDK(windows driver kit) header itself (that I can't recall right now).
Turning On System Wide LongPathsEnabled is not enough , each I really mean each executables **(those exe(s) which wish to use it)** itself also has to opt-in to use long Path Awareness (via per executable embedded app manifest).
This Comment says further : https://github.com/microsoft/PowerToys/issues/14080#issuecomment-1030839630

I posted #19887 which was closed as a duplicate of this, though (IMHO) this appears to be the exact opposite of what I reported:  I was trying to shorten the name of a file whose name was too long for PowerRename to recognize.  File Explorer had no trouble with the file, though.  I was using v0.59.1 when I discovered and reported 19887.
Also, (just my opinion here...) I don't think PowerRename should require users to enable long paths in the registry in order to get the basic long (approximately 32K chars) path support allowed with the \\? prefix on a path.

@nurbles I see your point. I do think it's the same subject, though. But your comment (there) has a few interesting things, so you're welcome to copy-paste it into this issue :-)


AgoraAudioFrameDelegate not firing onRecordAudioFrame: (4.1.1)

We are attempting to access audio frames when using Agora, however this delegate function doesn't seem to be getting called.
We are correctly setting the delegate method, as true is returned from: (BOOL)setAudioFrameDelegate:(id<AgoraAudioFrameDelegate> _Nullable)delegate.
Some delegate methods are called as expected, such as getObservedAudioFrame() and getRecordAudioParams(),  however onRecordAudioFrame is never called.

Hope a ticket was opened at the right place, going to close this issue now.


What does "Performing unique to ensure RDF semantics..." mean? (IndexBuilderMain log)

Can somebody briefly explain this to me?
There is probably a better and clearer way to say this.

Yes, I agree. It's just a unique over the list of all triples, because afaik the RDF (and thus turtle, etc) graph has a set of edges and there is no concept for duplicate triples/edges, yet automatically generate KB input files featured duplicates on some occasions (e.g. after reconciliation). To have identical behavior/output to the triples scores we compared QLever to in our evaluation, the unique was needed


Update Magnum with new scene tools, bindings and plugin additions

Motivation and Context
Continuation of #2100, with another ~10 days and >10 thousand lines of changes on the Magnum side. A large batch of features got finished and merged to master this time, and some breaking changes were needed to keep the API organized -- see below.
What's new:

Fixed some minor build issues with GCC 13
New expanded() and collapsed() APIs on multidimensional strided (bit) array views, similar to what Numpy's reshape() does
New pixelFormat() helper API for conveniently changing component count or sRGB encoding of an arbitrary PixelFormat
(Contributed by a 3rd party) The UfbxImporter plugin now has complete support for FBX skins and animations, allowing @0mdc to replace Assimp's broken FBX loading
New simplifyFailEmpty option in MeshOptimizerSceneConverter that'll make it fail in case simplification produces an empty mesh. By default it produces an empty MeshData, which can't be used to produce a valid glTF file. Related is also a minor fix to a bug that caused the plugin to trigger an assertion in presence of an empty mesh.
New forceBitDepth option for StbImageImporter and PngImporter that allows opening 16-bit images as 8-bit and vice versa. Added to work around a limitation of the Basis library which can only deal with 8-bit-per-pixel images.

New tools, especially of interest to @jturner65:

SceneTools::parentsBreadthFirst() and SceneTools::childrenDepthFirst() utilities for returning a scene hierarchy in various orders depending on desired use case
SceneTools::combineFields() that makes it possible to conveniently compose a SceneData out of loose data arrays.
SceneTools::filterFields(), SceneTools::filterOnlyFields() and SceneTools::filterExceptFields() for removing a particular field (such as mesh assignments, or transformations) from a scene as a whole. These are similar to existing attribute filtering tools for meshes and materials.
SceneTools::filterFieldEntries() for removing individual field entries (for example, all references to a particular flowerpot) from the scene
SceneTools::filterObjects() for removing individual objects (for example, openings objects) from the scene
A unified set of MeshTools::copy(), MaterialTools::copy() and SceneTools::copy() helpers for the rare cases where a mesh, material or scene data instance needs to be copied
Except for combineFields(), all of the above are also exposed to Python in the scenetools and meshtools modules, together with an ability to export scenes through trade.AbstractSceneConverter.

Additions to the magnum-sceneconverter utility, which you however usually use via the prebuilt tools instead of through Habitat:

New --passthrough-on-mesh-converter-failure and --passthrough-on-image-converter-failure options, making it possible to fall back to the original data for a particular failed -M or -P conversion
New --prefer option, adding an ability to choose an alternative plugin implementation, such as SpngImporter instead of the default PngImporter that may be as much as 2x faster
New --set option, allowing to set global plugin options like overriding bit depth of imported images, quality of exported JPEGs or Basis compression options. This used to be only partially possible before, done in a hacky way with [imageConverter] options in GltfSceneConverter, and --set is now the preferred way.
New --object-hierarchy option for --info-objects that'll visualize the object hierarchy by indenting the output. Useful for discovering object groupings or redundant nodes in a scene.

:warning: Breaking changes: :warning:

Trade::LightData::Type is now Trade::LightType so it's no longer needed to include the whole class for an enum declaration
SceneTools::flattenTransformationHierarchy3D() is renamed to SceneTools::absoluteFieldTransformations3D(), same change is done for the scenetools.flatten_transformation_hierarchy3d() Python binding that's now scenetools.absolute_field_transformations3d()
MeshTools::owned() is now MeshTools::copy()

The old C++ APIs are still present but marked as deprecated and thus causing compiler warnings. For the Python bindings however it's a hard change with no fallback. I updated the affected code here, some changes are needed in habitat-lab as well, but there I unfortunately have no easy way to verify the changes. @aclegg3 can you handle the updates on the habitat-lab side? Thank you.
How Has This Been Tested
:green_apple:

This now includes also a GLFW application EGL fix for @eundersander and https://github.com/facebookresearch/habitat-sim/pull/2116.

Assuming the tests pass on https://github.com/facebookresearch/habitat-lab/pull/1356 against this branch I think we can merge both.


Issue 1067

Before submitting (must do checklist)

[ ] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Proposed Changes

Prediction intervals for bottom-up reconciliation

Prediction intervals for top_down reconciliation

Closing issues

closes #1067

Codecov Report

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

@@                   Coverage Diff                    @@
##             hierarchical_pipeline    #1079   +/-   ##
========================================================
  Coverage                         ?   86.22%
========================================================
  Files                            ?      158
  Lines                            ?     8733
  Branches                         ?        0
========================================================
  Hits                             ?     7530
  Misses                           ?     1203
  Partials                         ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


sources api

Anybody know if this api is covered?
https://devnet.familysearch.org/docs/api/source_links_module

It isn't yet. Would you want to help with some code?
On Dec 13, 2011 3:17 PM, "Roger Pack" <
<EMAIL_ADDRESS>wrote:

Anybody know if this api is covered?
https://devnet.familysearch.org/docs/api/source_links_module

Reply to this email directly or view it on GitHub:
https://github.com/jimmyz/ruby-fs-stack/issues/6

It isn't yet. Would you want to help with some code?

Yeah if I get to it before you do.
-r


Support for Spanish language

I used the Negex algorithm to deal with Spanish text. With the help of a Spanish Language expert, a list of negex terms based on the lists provided in the original paper was created. I'd like to contribute those to this repository. To that effect, I've created a fork of this repository. I opened this issue so there could be a discussion as to how this extension can happen.

Hey, awesome idea! Happy to collaborate on it and make it happen.

@Siva227 - I pushed a change to the develop branch that should make it cleaner/easier to handle multi language support.
commit
Perhaps you'd like to update your fork and work from here? I believe you already have located Spanish termsets. You can update negspacy/termsets.py with them and then we can add a few test cases.

Great! I'll add the termsets to negspacy/termsets.py. Just so there's no confusion, these are the tags I used and the corresponding mappings to the tags you're using.
[PREN] - Prenegation rule tag - A term which negates the medical finding(symptom, disease, adverse event)that follows it
[POST] - Postnegation rule tag - A term which negates the medical finding(symptom, disease, adverse event)that precedes it
[PSEU] - Pseudo negation tag - Not a negation even though it contains a negation term
[CONJ] - Conjunction tag

PREN - Preceeding negations
POST - Following negations
PSEU - Pseudo negations
CONJ - Termination
I'll proceed with these and add necessary test cases.

Yup I believe that looks right!

Hello! Is there any work on this lately?

Not that I’m aware of

Taken care of by #51 . Thanks @j6e !


Extend class props with Parse.Object.extend()

This is a feature request/proposal.
It would be cool if class props were passed down to extended objects via Parse.Object.extend().
For example:
var BaseObject = Parse.Object.extend('__BaseObject__', {}, {

  // class props

  query: function () {
    return new Parse.Query(this.className);
  },

  count: function () {
    return this.query().count();
  }

});

var User = BaseObject.extend('User');

User.count().then(function (count) {
  console.log('There are ' + count + ' users');
});

Thoughts?

Unfortunately, there are a number of reasons why we'll probably never do this.
In the theory OOP inheritance, it's potentially unsafe to automatically inherit all statics from the parent, since they may not be relevant to the current class, or they may attempt to use other behaviors that have changed.
In JS's prototypal inheritance, this changes a little, but the concepts are the same. And since we're encouraging developers to move away from the legacy .extend() method, towards true ES6 classes which don't support static inheritance by default, it wouldn't be a great developer experience to add another feature that acts differently from the direction the language itself is moving.
If anything, JS makes it easier to make mistakes here, since now you have to guarantee that your method is being called with the proper context. In my personal opinion, SomeObject.count() should be rewritten as count(SomeObject) (ignoring naming and namespaces for a moment), which removes any ambiguity around context.
If you really wanted to achieve something like this, you can reference the parent's class like so:
query: function() {
  return this.__super__.query();
}

Heck, you could even use this to add your own extends-like method which automatically applies the the parent's static methods. Though, even in that scenario, I'd encourage explicitly-provided methods over automatically inheriting everything.
function customExtend(base, className, instanceMethods, classMethods) {
  // Include the ones I always want to provide
  var allClassMethods = {
    query: ...
    count: ...
  };
  if (classMethods) {
    for (var c in classMethods) {
      allClassMethods[c] = classMethods[c];
    }
  }
  return base.extend(className, instanceMethods, allClassMethods);
}

Hope that all makes sense!

@andrewimm Ah, that makes total sense. I hugely appreciate the explanation! Thank you.


feat: add --platform flag to override haste.defaultPlatform

Summary
Adds a new flag, --platform, to override haste.defaultPlatform option. This is useful for tests that need to be run against multiple platform targets. Previously, you would need separate Jest configs for each platform and set haste.defaultPlatform accordingly. With this flag, you can have one config and instead specify --platform ios instead.
Test plan
Normally, running react-native tests without a haste configuration will fail:
test-app % yarn jest
 FAIL  test/App.test.tsx
  ● Test suite failed to run

    Cannot find module '../Libraries/Image/Image' from '../../node_modules/react-native/jest/setup.js'

      at Resolver.resolveModule (../../node_modules/jest-resolve/build/resolver.js:311:11)
      at Object.<anonymous> (../../node_modules/react-native/jest/setup.js:58:4)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.028 s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

With this change, you can now specify the platform from the command line:
% yarn jest --platform ios
 PASS  test/App.test.tsx
  ✓ renders correctly (239 ms)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        0.747 s
Ran all test suites.
✨  Done in 2.33s.

Codecov Report

Merging #11764 (cdbf2e3) into master (8d33ba1) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #11764      +/-   ##
==========================================
- Coverage   69.00%   68.98%   -0.02%
==========================================
  Files         312      312
  Lines       16339    16345       +6
  Branches     4736     4740       +4
==========================================
+ Hits        11275    11276       +1
- Misses       5036     5041       +5
  Partials       28       28

Impacted Files
Coverage Δ

packages/jest-config/src/setFromArgv.ts
75.60% <0.00%> (-12.97%)
:arrow_down:

packages/expect/src/utils.ts
96.13% <0.00%> (+0.55%)
: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 8d33ba1...cdbf2e3. Read the comment docs.

A changelog entry seems to be missing 🙂

Thanks, I thought I pushed it already 😄

@SimenB: I would really appreciate it if you reviewed this PR and let me know how I can improve it.

@sigveio, @SimenB: Please let me know if there is someone else I can ping to help review this PR. Thanks 😄

Hi! Sorry about the delay.

Is there any reason you cannot use the existent --haste flag? I'd really love to not add any more flags 😅

Is there any reason you cannot use the existent --haste flag? I'd really love to not add any more flags 😅

Ah, it didn't occur to me that it exists (I combed through CLI options looking for it), even though I touched the code 😛
Does that mean that we could do something like --haste '{ defaultPlatform: "ios" }'? Compared to --platform ios, I think the latter is a lot more user-friendly, don't you think? It's not exactly what comes to mind if anyone asked me how to target a specific platform.

@tido64 --haste='{"defaultPlatform": "ios"}' seems to be working. The value must be JSON string.

I agree it's more user friendly, but a metric ton of options isn't very nice either... Also, passing both haste and this new flag would conflict, and which case should win isn't obvious to the end user I think regardless of which way we land it.
(And yeah, there's a bunch of CLI args we haven't documented - jest --help lists them all tho)


Missing documentation for RecognizedText property for Pen Input control

[Enter feedback here]
Pen Input control has a RecognizedText property. However, there is no documentation on this property. I'm particularly interested in knowing which platforms have support for text recognition.

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

ID: 1bfd429b-1c01-e596-5637-91ab84067297
Version Independent ID: 3b9d95f2-5e5c-1f0a-a320-a6b3f57b7549
Content: Pen input control in Power Apps - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/controls/control-pen-input.md
Service: powerapps
Sub-service: canvas-maker
GitHub Login: @chmoncay
Microsoft Alias: chmoncay

Thanks for your comment, @leeprtc. This issue has been assigned to the doc author for review.


Fixing some windows issues - path slashes

Was trying to run the project in windows and it had made some really funky paths.
1: some paths had mixed back and forward slashes
2: some paths were incorrectly concatenated, like C:/Users/frtn/.esmd/npm/react@18.3.1/node_modules/C:/Users/frtn/.esmd/npm/react<EMAIL_ADDRESS>Trying to fix both in here.
Tested only in my windows computer and things seem to work well with a few packages I tried, so you may need to test it yourself before merging for the other OSs to make sure I didn't break them 😅

Hold - found other windows issues, will try to fix in this PR

Oh, esbuild has more issues, so nvm I guess...

sorry, i never try to develop this project on a Windows machine

thanks anyway!


adding home link to flyout menu

Why is this change necessary?

The home link is missing in the flyout-menu according to this (issue)[https://github.com/shopwareBoostDay/platform/issues/379]

What does this change do, exactly?

Adding the home option if it's configured to the flyout menu

Describe each step to reproduce the issue or behaviour.

Open the shop on a mobile device and open the flyout menu. The home link is not visible

Please link to the relevant issues (if any).

This issue

5. Checklist

[ ] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-23279
Please use this issue to track the state of your pull request.

Hello @NinjaArmy,
While testing your changes we found two issues.

By using the block() function it is not possible for extensions to override the navigation link's content
The imported block is used for the main navigation. Hence, the applied CSS classes do not match the rest of the navigation. This results in a wrong display for medium view ports:

Hello @NinjaArmy,
Thank you again for updating your PR. Sry if it takes a little bit longer to merge it. This is because we are discussing some unclear points related to your MR right now.
I will keep you up to date :)
regards,
Sebastian


fls: init at 2022-10-25

removed unnessesary diff
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

resolves #218262

Please update to latest commit and migrate to pkgs/by-name.


Properly display the final segment of the path

This fix addresses issue #3

Great work Ryan. I've added you to a new cdnjs team called Atom Devs, feel free to read/write/close issues etc etc
Mainly because I'm not even using Atom at the moment and haven't had the time to maintain.

Thanks! I'm happy to maintain it for the foreseeable future.


[resolves #252] Add profile to push docker images to public docker regis...

...try

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/237/
Test PASSed.


chore: add quality check and PR checks

Issue #, if available: #5
Description of changes:
This PR introduces CI workflows needed to run validity and quality checks on PRs and commits.
Namely, the PR adds:

Workflows and scripts needed to check that PRs have valid attribution and linked issue
Quality check workflow to run linting and unit + contract tests on PR updates and merge commits on main
A GitHub Action to cache & install npm packages which can be reused across current and future workflows

The PR intentionally leaves out e2e tests for now as they require a GitHub PAT and we have yet to agree on how to move forward on that front.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you so much Andrea for all fixes and improvements  — reviewing and
merging tomorrow first thing in the morning
On Mon, 6 May 2024 at 14:42, Andrea Amorosi @.***>
wrote:

@dreamorosi https://github.com/dreamorosi requested your review on: #14
https://github.com/aws-powertools/actions/pull/14 chore: add quality
check and PR checks.
—
Reply to this email directly, view it on GitHub
https://github.com/aws-powertools/actions/pull/14#event-12714496432, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBALRJGYINPLHVAJAOTZA53ERAVCNFSM6AAAAABHIZMX42VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSG4YTINBZGY2DGMQ
.
You are receiving this because your review was requested.Message ID:
@.***>


destination folder does not exist error

the program does not create the parent folder of destination file if it is not exist

fixed on 2.0 beta


Question: E27?

Hi guys,
Anyone was successful with E27?

after a quick google search i've seen that sonoff e27 refers to either sonoff slampher (fully supported) or sonoff b1 bulb (use sonoff-bulb branch - not merged currently to master - and read this thread too https://github.com/peterbuga/HASS-sonoff-ewelink/issues/34)

Thanks for the quick reply!
I'll test a sonoff B1 at weekend and I just download the sonoff-bulb to put in my HASS, but I have others sonoffs (pow R2 and RF), and my RF not working with sonoff-bulb, only with normal sonoff.
How can I use all my sonoffs and the future B1? I need the 2 libs(sonoff and sonoff-bulb)?

@neomvll sonoff-bulb it's an extension of the master branch and should work just fine for all the other devices. you might experience the problems reported here or here that i haven't got the time to see what's wrong with them and fix them.
if you experience other problems, please open another issue titled properly and try to give me as many error logs and describe what is happening / what is not working, a simple "not working" is not enough :P. otherwise feel free to add debug details to the issues mentioned above


DevOps / Continuous Deployment

Will need a deployment into some test environment for automated integration tests

Implemented as GitHub Actions, with artifacts being produced


[Releases] Make the bump-oss-version script do git push as well

The idea is that when you run node ./scripts/bump-oss-version.js 0.31.0-rc.0 the script should prompt:

Would you like to run "git push origin 0.31-stable v0.31.0-rc.0"? [Y/n]

and do the git push if told to do so.

To be done just before next release (so we can also test it out)


added support for bearer tokens

Ok I'll add a test later.

@frankrousseau , should i publish this or do you want to wait for test ?

Don't publish it please.


cmake.buildDirectory does not correctly expand in tasks.json

Brief Issue Summary
The ${config:cmake.buildDirectory} does not correctly expand in tasks.json. It should be build-debug with the Debug configuration preset selected but it is the default of build:
{
    "tasks": [
        {
            "type": "shell",
            "label": "Get build directory name",
            "windows": {
                "command": "echo ${config:cmake.buildDirectory}"
            },
            "group": "none"
        }
    ],
    "version": "2.0.0"
}

{
  "version": 2,
  "configurePresets": [
    {
      "name": "debug",
      "displayName": "Debug",
      "description": "Sets debug preset",
      "generator": "Ninja",
      "binaryDir": "${workspaceFolder}/build-${presetName}",
      "cacheVariables": {
        "CMAKE_BUILD_TYPE": "Debug",
        "CMAKE_TOOLCHAIN_FILE": "raspberry-pi-gcc.cmake",
        "CMAKE_INSTALL_PREFIX": "${workspaceFolder}/install-${presetName}"
      }
    }
  ]
}

CMake Tools Diagnostics
{
  "os": "win32",
  "vscodeVersion": "1.71.0",
  "cmtVersion": "1.12.26",
  "configurations": [
    {
      "folder": "d:\\pi_game",
      "cmakeVersion": "3.22.22040401",
      "configured": true,
      "generator": "Ninja",
      "usesPresets": true,
      "compilers": {}
    }
  ],
  "cpptoolsIntegration": {
    "isReady": true,
    "hasCodeModel": true,
    "activeBuildType": "Debug",
    "buildTypesSeen": [
      "Debug"
    ],
    "requests": [
      "file:///d%3A/pi_game/pi/src/main.cpp"
    ],
    "responses": [
      {
        "uri": "file:///d%3A/pi_game/pi/src/main.cpp",
        "configuration": {
          "includePath": [
            "d:/pi_game/pi/src"
          ],
          "defines": [],
          "compilerPath": "c:/sysgcc/raspberry64/bin/aarch64-linux-gnu-g++.exe",
          "compilerArgs": [],
          "compilerFragments": [
            "-g",
            "-std=c++17"
          ]
        }
      }
    ],
    "partialMatches": [],
    "targetCount": 1,
    "executablesCount": 1,
    "librariesCount": 0,
    "targets": [
      {
        "name": "pi",
        "type": "EXECUTABLE"
      }
    ]
  },
  "settings": [
    {
      "communicationMode": "automatic",
      "useCMakePresets": "auto",
      "configureOnOpen": true
    }
  ]
}

Debug Log
[main] Building folder: pi_game
[main] Saving open files before configure/build
[main] The folder containing the CMake cache is missing. The cache will be regenerated.
[main] Configuring folder: pi_game
[main] Saving open files before configure/build
[driver] Start configure
[driver] Running pre-configure checks and steps
[cmakefileapi-driver] Configuring using preset
[cmakefileapi-driver] Invoking CMake C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.EXE with arguments ["-DCMAKE_BUILD_TYPE=Debug","-DCMAKE_TOOLCHAIN_FILE=raspberry-pi-gcc.cmake","-DCMAKE_INSTALL_PREFIX=d:/pi_game/install-debug","-Sd:/pi_game/pi","-Bd:/pi_game/build-debug","-G","Ninja"]
[proc] Executing command: "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.EXE" -DCMAKE_BUILD_TYPE=Debug -DCMAKE_TOOLCHAIN_FILE=raspberry-pi-gcc.cmake -DCMAKE_INSTALL_PREFIX=d:/pi_game/install-debug -Sd:/pi_game/pi -Bd:/pi_game/build-debug -G Ninja
[cmake] -- The C compiler identification is GNU 10.2.1
[cmake] -- The CXX compiler identification is GNU 10.2.1
[cmake] -- Detecting C compiler ABI info
[cmake] -- Detecting C compiler ABI info - done
[cmake] -- Check for working C compiler: C:/SysGCC/raspberry64/bin/aarch64-linux-gnu-gcc.exe - skipped
[cmake] -- Detecting C compile features
[cmake] -- Detecting C compile features - done
[cmake] -- Detecting CXX compiler ABI info
[cmake] -- Detecting CXX compiler ABI info - done
[cmake] -- Check for working CXX compiler: C:/SysGCC/raspberry64/bin/aarch64-linux-gnu-g++.exe - skipped
[cmake] -- Detecting CXX compile features
[cmake] -- Detecting CXX compile features - done
[cmake] -- Configuring done
[cmake] -- Generating done
[cmake] -- Build files have been written to: D:/pi_game/build-debug
[cmakefileapi-parser] Read reply folder: d:\pi_game\build-debug\.cmake\api\v1\reply
[cmakefileapi-parser] Found index files: ["cache-v2-7249ce2d208b346e4d57.json","cmakeFiles-v1-8f315e5963e5eb01eb60.json","codemodel-v2-7753ac01f77ce7fac582.json","directory-.-Debug-d0094a50bb2071803777.json","directory-src-Debug-c94b7c2814d5a21df52c.json","index-2022-09-03T19-23-07-0059.json","target-pi-Debug-5c14d6ca945b533dec30.json","toolchains-v1-0ff44c010df1e7999c73.json"]
[cache] Reading CMake cache file d:/pi_game/build-debug/CMakeCache.txt
[cache] Parsing CMake cache string
[build] Starting build
[driver] Start build
[proc] Executing command: "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.EXE" --build d:/pi_game/build-debug
[build] [1/2] Building CXX object src/CMakeFiles/pi.dir/main.cpp.o
[build] [2/2] Linking CXX executable bin\pi
[cmakefileapi-parser] Read reply folder: d:\pi_game\build-debug\.cmake\api\v1\reply
[cmakefileapi-parser] Found index files: ["cache-v2-7249ce2d208b346e4d57.json","cmakeFiles-v1-8f315e5963e5eb01eb60.json","codemodel-v2-7753ac01f77ce7fac582.json","directory-.-Debug-d0094a50bb2071803777.json","directory-src-Debug-c94b7c2814d5a21df52c.json","index-2022-09-03T19-23-07-0059.json","target-pi-Debug-5c14d6ca945b533dec30.json","toolchains-v1-0ff44c010df1e7999c73.json"]
[build] Build finished with exit code 0
[cache] Reading CMake cache file d:/pi_game/build-debug/CMakeCache.txt
[cache] Parsing CMake cache string
[extension] [2493] cmake.build finished (returned 0)

Additional Information
VSCode about:
Version: 1.71.0 (user setup)
Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Date: 2022-09-01T07:36:10.600Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
Sandboxed: No

Because I can't read / found some tripe somewhere on the internet! The correct expansion should be ${command:cmake.buildDirectory}.


Remove Visio files since out of date

Does this also apply to the PDFs and PNGs in the Documentation directory?

@rjyounes I don't know how many users looked at the giant PDF rather than loading into Protege and poking around. Technically we have embedded docs, though logical grouping and smart layout is not inherent.

Need to remove.


Add link to v5 on the main v6 description page

Hi,
It took me 3 google searches, and 20 pages to find the link.. And I'm a little bit familliar to github
What is it if you are llow computer skilled, abandon ?
Suggestion :
Please find old v5 downlood link (not developped anymore)
[[[[
](
)
](https://github.com/dasher-project/dasher/releases)](https://github.com/dasher-project/dasher/releases)
Thx

@sjjhsjjh Can you do that? I unfortunately don't have any write access to this repo


some words not in xlnet vocabulary ,especially name

❓ Questions & Help

I face the problem when xlnet tokenizer encodes name
from pytorch_transformers import *
import torch
tokenizer = XLNetTokenizer.from_pretrained('xlnet-base-cased')
model = XLNetModel.from_pretrained('xlnet-base-cased')
sents = ["here is a dog","michael love mary <pad>"]
input_ids =[tokenizer.encode(sent) for sent in sents]

model(torch.tensor(input_ids))[0]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: expected sequence of length 4 at dim 1 (got 8)

I think the reason is that xlnet can't encode the name "michael " and "mary" , instead it encodes "michael" to "_mi" "cha" "el" and mary  to "m" "ary"  respectively . And this result leads input_ids to has two different length id list . As the result , when I fed the torch.tensor(input_ids) to XLNetModel would cause "ValueError" problem , how could I fix it?
input_ids
[[193, 27, 24, 2288], [12430, 2409, 530, 564, 17, 98, 1449, 5]]
[tokenizer.tokenize(sent) for sent in sents]
[['▁here', '▁is', '▁a', '▁dog'], ['▁mi', 'cha', 'el', '▁love', '▁', 'm', 'ary', '<pad>']]

Hi!
XLNet uses a SentencePiece tokenizer which splits the words into subword units. In your case, it splits the two examples in different sized sequences, which can't be parsed as a Tensor which requires an input matrix (and not a list of lists).
You should pad your sequences after they have been tokenized so that they all are of equal size. Then they can be converted to a tensor and fed to the model.
My purpose is to fed each word's contextual embedding to transformer layer to obtain sentence embedding
However ,  by this way I can't get each word's contextual embedding , is there any solution to fix this problem or just use xlnet function to get sentence embedding?


com.facebook.react.common.javascriptexception{}

how to fix it.. thanks.

Hi @adylimmo , i'm not really sure what's going on in here. Could you send me what are your dev environment versions?

hi @JuneDomingo ,
im use
OS windows 10
by step :

git clone https://github.com/JuneDomingo/movieapp.git
cd movieapp
make file .env in folder movieapp
npm install --save
react-native run-android (installed in android 5.1.1)
shake and im select debug

and then show error com.facebook.react.common.javascriptexception{} ,
how to fix.
thanks.

Sorry i do not know anything
2017-01-15 18:32 GMT+03:00 adylimmo<EMAIL_ADDRESS>
hi @JuneDomingo https://github.com/JuneDomingo ,
im use
OS windows 10
by step :

git clone https://github.com/JuneDomingo/movieapp.git
cd movieapp
make file .env in folder movieapp
npm install --save
react-native run-android (installed in android 5.1.1)
shake and im select debug

and then show error com.facebook.react.common.javascriptexception{} ,
how to fix.
thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JuneDomingo/movieapp/issues/6#issuecomment-272702442,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO5qktC9lKe5Mizm8uRJKVPnLKg0tXeks5rSjwRgaJpZM4Ljr_y
.

@adylimmo , can you try just running npm install or install YARN  in your machine and run ``yarn install.
Make sure you also include the API_VARIABLES etc. into your .env file.

thanks @JuneDomingo is work..
i'm solved "yarn install"


Question: How to add attributes to table columns?

Hi,
looking to try an add a new property to the table column adder and an auto-generated value similar to the 'default' property.
When the user clicks 'new column' another property will be listed along with a pre-filled value under 'Value'.
I've been sifting through the code but just wondering if there's a better method :) I appreciate it.

I think this should be more generic (we need a configuration for all new objects default name).

Hi Serge,
Sorry for the confusion, I'm not requesting a feature, my team and I are trying to use dbeaver for our project and require the addition of a auto-generated value such as the default property. We've been digging into the code but cannot find a reliable way to create a new property. We would like to be able to click "add new column" and then have a security label checkbox appear in order to auto fill in the column for row level security.
Thanks!

If you need this for some custom driver/extension then you can annotate Java class getter with @Property. Thus it will become available in property list.


Support RabbitMQ

Add an IBroker implementation for RabbitMQ.

A first implementation of Silverback.Integration.RabbitMQ is included in the pre-release 2.0.0-rc1.


travis file also needs update.  fixes #49

connects to #49

Coverage remained the same at 75.207% when pulling 6b48282a50b09282b87006bc9cd27a718c72600f on lholmquist:49-need-to-update-travis-to into 338448440e9a5ee8fd40a62c825a9c2ca27ef764 on bucharest-gold:master.

Coverage remained the same at 75.207% when pulling 3c524845c2545d16a7af57d303248fbdb31a4f42 on lholmquist:49-need-to-update-travis-to into 338448440e9a5ee8fd40a62c825a9c2ca27ef764 on bucharest-gold:master.


pythonPackages.{abodepy,lomond}: init

Motivation for this change
Support abode in home assistant
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.

Result of nixpkgs-review pr 99380 1

5 packages built:

python27Packages.lomond
python37Packages.abodepy
python37Packages.lomond
python38Packages.abodepy
python38Packages.lomond


Move People Between Loops

If a host has multiple Loops they should be able to move participants between these loops.

Duplicate of #352


Add an extra_keyword parameter to UVCal

Addressing issue #232

@plaplant Can you take a look at this? It's addressing your issue.


COMP: Mark DCMTKImageIO::CanWriteFile argument as unused

To address:
/ITK/Modules/IO/DCMTK/src/itkDCMTKImageIO.cxx:153:45: warning: unused
parameter 'name' [-Wunused-parameter]
bool DCMTKImageIO::CanWriteFile(const char *name)

#include "itkMacro.h", before using itkNotUsed.

To keep the build times down and the number of include's at the top of the file reasonable, we only #include what is required.
We even have a script to help apply this:
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Utilities/Maintenance/FindRedundantHeaderIncludes.py


attempted an AZ Login --user-device-code against an azure Stack instance. Code generation worked. Copied Device code to webpage then the following error occurs in powershell window

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
'NoneType' object is not iterable
Traceback (most recent call last):
Temp\pip-install-yf22vmg2\knack\knack\cli.py, ln 215, in invoke
azure\cli\core\commands\__init__.py, ln 654, in execute
azure\cli\core\commands\__init__.py, ln 718, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 711, in _run_job
Local\Temp\pip-install-yf22vmg2\six\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 688, in _run_job
azure\cli\core\commands\__init__.py, ln 325, in __call__
site-packages\azure\cli\core\__init__.py, ln 545, in default_command_handler
azure\cli\command_modules\profile\custom.py, ln 150, in login
site-packages\azure\cli\core\_profile.py, ln 240, in find_subscriptions_on_login
site-packages\azure\cli\core\_profile.py, ln 276, in _normalize_properties
TypeError: 'NoneType' object is not iterable

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --use-device-code

Expected Behavior
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6
Installer: MSI

azure-cli 2.6.0

Additional Context

add to S171

add to S172

This is a known issue of Azure Stack (https://github.com/Azure/azure-rest-api-specs/issues/9567) pending a fix from service side.

Error message is improved in #15550.


Dependencies not included in POM

Hello.
I'm trying to use this very interesting library from your personal Maven repository, but it seems there are not dependencies specified in the POM file.
When I include org.clapper.javautil:3.0.1 it downloads only the javautil.jar. It misses the required libraries (javamail, ASF, ASM).
Maybe they should be added to the POM, so my build system (I use Gradle) will be able to download all the required jars?
Cheers,
Fabio

I need to take the time to convert that library to Maven. I haven't done so. Once I do, the deps will be in the POM.

Okay, I take that back. I forgot I'd converted it to buildr. I'll look into this. Thanks.


If using CXX1XSTD should also use CXX1X compiler

readxl sets PKG_CXXFLAGS to include CXX1XSTD (added in commit https://github.com/hadley/readxl/commit/c957ce154a31713f400e348ca30034a66bfcd9be), but does not use the CXX1X compiler.
If the user speficies an alternative compiler for CXX1X (and CXX1XSTD) in Makevars this can result in readxl failing to build (see https://travis-ci.org/leeper/rio/builds/131807863#L1641-L1645 for an example).
In addition the comment confuses me, if the issue is compiling C code with the C99 standard as the submission comments suggest, should we not be setting PKG_CFLAGS=-std=c99 instead?

as an intermediate fix, @jimhester suggested (if I understood correctly) to amend my ~/.R/Makevarsas per https://github.com/wesm/feather/issues/154.
I tried the below .travis.yaml additions:
before_install: |
    mkdir ~/.R
    cat <<EOF > ~/.R/Makevars
    CXX1X=g++-4.9
    CXX1XSTD=-std=c++11
    echo "CC=gcc-4.9 >>" ~/.R/Makevars

But still got a failing build, this time erroring out on feather again.
Installing feather (0.3.1) ...
[1] "Command failed (1)\n\nFailed to run system command:\n\n\t'/home/travis/R-bin/lib/R/bin/R' --vanilla CMD INSTALL '/tmp/RtmpsvJQDC/feather' --library='/home/travis/build/maxheld83/goodlife/packrat/lib/x86_64-pc-linux-gnu/3.3.2' --install-tests --no-docs --no-multiarch --no-demo \n\nThe command failed with output:\n* installing *source* package 'feather' ...\n** package 'feather' successfully unpacked and MD5 sums checked\nPlatform is little endian. Good.\n** libs\ng++ -std=c++0x -I/home/travis/R-bin/lib/R/include -DNDEBUG -I. -I/home/travis/R-bin/include -I\"/home/travis/build/maxheld83/goodlife/packrat/lib/x86_64-pc-linux-gnu/3.3.2/Rcpp/include\"   -fpic  -g -O2 -c RcppExports.cpp -o RcppExports.o\nIn file included from feather/api.h:24:0,\n                 from feather_types.h:2,\n                 from RcppExports.cpp:4:\n./feather/common.h:24:28: error: both 'const' and 'constexpr' cannot be used here\n./feather/common.h:25:28: error: both 'const' and 'constexpr' cannot be used here\nIn file included from feather/api.h:25:0,\n                 from feather_types.h:2,\n                 from RcppExports.cpp:4:\n./feather/io.h:73:29: error: expected ';' at end of member declaration\n./feather/io.h:73:35: error: 'override' does not name a type\n./feather/io.h:74:26: error: expected ';' at end of member declaration\n./feather/io.h:74:28: error: 'override' does not name a type\n./feather/io.h:75:59: error: expected ';' at end of member declaration\n./feather/io.h:75:61: error: 'override' does not name a type\n./feather/io.h:93:29: error: expected ';' at end of member declaration\n./feather/io.h:93:35: error: 'override' does not name a type\n./feather/io.h:94:26: error: expected ';' at end of member declaration\n./feather/io.h:94:28: error: 'override' does not name a type\n./feather/io.h:95:59: error: expected ';' at end of member declaration\n./feather/io.h:95:61: error: 'override' does not name a type\n./feather/io.h:108:29: error: expected ';' at end of member declaration\n./feather/io.h:108:35: error: 'override' does not name a type\n./feather/io.h:109:26: error: expected ';' at end of member declaration\n./feather/io.h:109:28: error: 'override' does not name a type\n./feather/io.h:110:59: error: expected ';' at end of member declaration\n./feather/io.h:110:61: error: 'override' does not name a type\n./feather/io.h:150:16: error: expected ';' at end of member declaration\n./feather/io.h:150:18: error: 'override' does not name a type\n./feather/io.h:151:29: error: expected ';' at end of member declaration\n./feather/io.h:151:35: error: 'override' does not name a type\n./feather/io.h:152:51: error: expected ';' at end of member declaration\n./feather/io.h:152:53: error: 'override' does not name a type\n./feather/io.h:172:16: error: expected ';' at end of member declaration\n./feather/io.h:172:18: error: 'override' does not name a type\n./feather/io.h:173:29: error: expected ';' at end of member declaration\n./feather/io.h:173:35: error: 'override' does not name a type\n./feather/io.h:174:51: error: expected ';' at end of member declaration\n./feather/io.h:174:53: error: 'override' does not name a type\nmake: *** [RcppExports.o] Error 1\nERROR: compilation failed for package 'feather'\n* removing '/home/travis/build/maxheld83/goodlife/packrat/lib/x86_64-pc-linux-gnu/3.3.2/feather'"
Error: Command failed (1)

Failed to run system command:

	'/home/travis/R-bin/lib/R/bin/R' --vanilla CMD INSTALL '/tmp/RtmpsvJQDC/feather' --library='/home/travis/build/maxheld83/goodlife/packrat/lib/x86_64-pc-linux-gnu/3.3.2' --install-tests --no-docs --no-multiarch --no-demo

The command failed with output:
* installing *source* package 'feather' ...
** package 'feather' successfully unpacked and MD5 sums checked
Platform is little endian. Good.
** libs
g++ -std=c++0x -I/home/travis/R-bin/lib/R/include -DNDEBUG -I. -I/home/travis/R-bin/include -I"/home/travis/build/maxheld83/goodlife/packrat/lib/x86_64-pc-linux-gnu/3.3.2/Rcpp/include"   -fpic  -g -O2 -c RcppExports.cpp -o RcppExports.o
In file included from feather/api.h:24:0,
                 from feather_types.h:2,
                 from RcppExports.cpp:4:
./feather/common.h:24:28: error: both 'const' and 'constexpr' cannot be used here
./feather/common.h:25:28: error: both 'const' and 'constexpr' cannot be used here
In file incl
Execution halted

The command "R -e "0" --args --bootstrap-packrat" failed and exited with 1 during .

Your build has been stopped.

interestingly, just setting dist = trusty also solves the issue, though I am not sure whether that is recommended:
language: r

sudo: false

dist: trusty

install:
  - R -e "0" --args --bootstrap-packrat

warnings_are_errors: false

cache:
  packages: true
  directories:
    - $TRAVIS_BUILD_DIR/_bookdown_files
    - $TRAVIS_BUILD_DIR/packrat/

before_script:
  - chmod +x ./_build.sh
  - chmod +x ./_deploy.sh

script:
  - ./_build.sh
  - ./_deploy.sh

notifications:
  email: false

The first example creates a HEREDOC, so you just need to add CXX=g++-4.9.
Trusty has an older version of GCC as the default, so doesn't run into this issue.


How to use "Highcharts.setOptions"?

I cant see "Highcharts.setOptions" in the global scope. Im using webpack, es6 and react-highcharts 12.0.0.
How can I setOptions?

Nvm, I figured out.
import Highcharts from 'react-highcharts';
Highcharts.Highcharts.setOptions({
  lang: {
    shortMonths,
  },
});

Nvm, I figured out.
import Highcharts from 'react-highcharts';
Highcharts.Highcharts.setOptions({
  lang: {
    shortMonths,
  },
});

It's not working for me !


is the dev branch the master branch? kinda confusing

Documentation Feedback
Provide a brief summary of what you would like to see changed in our documentation.
Feel free to provide any suggestions of content or examples you’d like us to include.
Affected documentation page: Insert a link to the affected page

Hi! yeah, dev is the main branch and the one that gets released.


Failed to load EMI with kubejs for 1.21

debug.log

KubeJS has added EMI category removing support


Is there a way to use this without MeteorDocument etc?

Thank you for making this package available. It looks like great work!
I wanted to ask whether it was possible to just subscribe and get notifications when a document changes/additions/deletions happen as opposed to having to create local MeteorDocument classes to represent the remote data and all that.
Does that make sense? Thanks!

I managed to do what I needed to do by extending the "Client" DDPClient that's inside of the Meteor class. It seems a bit hacky, though, and it'd look better if Meteor itself provided a delegate that you can hook into for "raw" updates.
This is what my extension looks like:
extension Meteor {

    public class Client: DDPClient {

        public override func documentWasAdded(collection:String, id:String, fields:NSDictionary?) {

            print( "ADDED: \(collection) \(id) \(fields)" )

        }

        public override func documentWasChanged(collection: String, id: String, fields: NSDictionary?, cleared: [String]?) {

            print( "ADDED: \(collection) \(id) \(fields) \(cleared)" )

        }

        public override func documentWasRemoved(collection: String, id: String) {

            print( "REMOVED: \(collection) \(id)" )

        }

    }

}

I did in one of my project without [MeteorDocument,Visit here for the the documentation and you can see the Boilerplate I made here for demonstration.

You have identified the key methods, but you should not extend Client. What you are trying to do is already available via AbstracCollection. Just subclass it and override those methods.

Awesome. I'll give that a whirl. Thanks!


Mistake in python 3.7(autopep8 v2022.1.12871025)

Traceback (most recent call last):
  File "C:\Users\Oleg Kolesov\AppData\Local\Programs\Python\Python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Users\Oleg Kolesov\AppData\Local\Programs\Python\Python37\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "c:\Users\Oleg Kolesov\.vscode\extensions\ms-python.autopep8-2022.1.12871025\bundled\libs\autopep8.py", line 4590, in <module>
    sys.exit(main())
  File "c:\Users\Oleg Kolesov\.vscode\extensions\ms-python.autopep8-2022.1.12871025\bundled\libs\autopep8.py", line 4536, in main
    wrap_output(sys.stdout, encoding=encoding).write(fixed_stdin)
  File "C:\Users\Oleg Kolesov\AppData\Local\Programs\Python\Python37\lib\codecs.py", line 377, in write
    data, consumed = self.encode(object, self.errors)
  File "C:\Users\Oleg Kolesov\AppData\Local\Programs\Python\Python37\lib\encodings\cp1251.py", line 12, in encode
    return codecs.charmap_encode(input,errors,encoding_table)
UnicodeEncodeError: 'charmap' codec can't encode character '\udc98' in position 255: character maps to \<undefined\>

@inkoff Does running autopep8 from your terminal using the same python work? Can you share your environment variables?

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


documentation usage of 'unpopular'

Riverpod front page readme says it is 'unpopular' but pub.dev says it has 1000+ likes and 98% popularity :)
https://github.com/rrousselGit/river_pod#will-provider-be-deprecatedstop-being-supported
It also says Experimental, perhaps this piece of the readme can go away with the 1.0 release??

Haha yes, you're right. This will be updated with the v1.

I've removed the notes in the Riverpod readme. I'll transfer this issue to package:provider to update its readme to mention riverpod


Reworked into Python3

Looking around the web for some scrapping in Python I've found your scripts, and I'm reworking them into Python3. Check and merge it if you want. Anyway, could I use them for educational purpose?

Forgot to write the output to json file, will add it in the next version


Query on Evaluation Capabilities for Reasoning Chains in Embodied Intelligence Models

Thank you for your contributions to embodied intelligence. However, from the content of the article and the way it is invoked here, it seems to only support the evaluation and comparison of models, and does not support the evaluation and comparison of reasoning chains similar to ReAct. I'm not sure if my understanding is correct?

Thank you for your contributions to embodied intelligence. However, from the content of the article and the way it is invoked here, it seems to only support the evaluation and comparison of models, and does not support the evaluation and comparison of reasoning chains similar to ReAct. I'm not sure if my understanding is correct?

I think it's like this. They require the generation of LLM outputs that conform to their defined specifications to be placed in the "llm-response-path", without including how to generate the llm response.


Refactoring

:tada: This PR is included in version 1.0.0-alpha.27 :tada:
The release is available on:

GitHub release
v1.0.0-alpha.27
v1.0.0-alpha.27

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.27 :tada:
The release is available on:

GitHub release
v1.0.0-alpha.27
v1.0.0-alpha.27

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
v1.0.0
v1.0.0

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
v1.0.0
v1.0.0

Your semantic-release bot :package::rocket: