rules_xcodeproj fails to process the imported package: DropDownCell.xib' is duplicated in the 'resources' attribute of rule <..>

I trying to import this SPM package and unable to:
        .package(url: "https://github.com/AssistoLab/DropDown.git", revision: "2ab6f6c") // no tags with spm support

the error:
ERROR: /private/var/tmp/_bazel_krypt/de9844abaa047ea4e0868fa95089f211/rules_xcodeproj.noindex/build_output_base/external/rules_swift_package_manager~0.14.0~swift_deps~swiftpkg_dropdown/BUILD.bazel:6:22: Label '@rules_swift_package_manager~0.14.0~swift_deps~swiftpkg_dropdown//:DropDown/resources/DropDownCell.xib' is duplicated in the 'resources' attribute of rule 'DropDown_DropDown_resource_bundle'
ERROR: /Users/krypt/repo/smart-kit/Modules/SupportUI/BUILD.bazel:3:14: Target '@rules_swift_package_manager~0.14.0~swift_deps~swiftpkg_dropdown//:DropDown_DropDown' contains an error and its package is in error and referenced by '//Modules/SupportUI:SupportUI'

the generated BUILD.bazel (line is actually added twice):

load("@bazel_skylib//rules:build_test.bzl", "build_test")
load("@build_bazel_rules_apple//apple:resources.bzl", "apple_resource_bundle")
load("@build_bazel_rules_swift//swift:swift.bzl", "swift_library")
load("@rules_swift_package_manager//swiftpkg:build_defs.bzl", "generate_modulemap", "resource_bundle_accessor", "resource_bundle_infoplist")

apple_resource_bundle(
    name = "DropDown_DropDown_resource_bundle",
    bundle_name = "DropDown_DropDown_resource_bundle",
    infoplists = [":DropDown_DropDown_resource_bundle_infoplist"],
    resources = [
        "DropDown/resources/DropDownCell.xib",
        "DropDown/resources/DropDownCell.xib",
    ],
)

build_test(
    name = "DropDownBuildTest",
    targets = ["@swiftpkg_dropdown//:DropDown_DropDown"],
    visibility = ["//visibility:public"],
)

generate_modulemap(
    name = "DropDown_DropDown_modulemap",
    deps = [],
    hdrs = [":DropDown_DropDown"],
    module_name = "DropDown",
    visibility = ["//visibility:public"],
)

resource_bundle_accessor(
    name = "DropDown_DropDown_resource_bundle_accessor",
    bundle_name = "DropDown_DropDown_resource_bundle",
)

resource_bundle_infoplist(
    name = "DropDown_DropDown_resource_bundle_infoplist",
    region = "en",
)

swift_library(
    name = "DropDown_DropDown",
    data = [":DropDown_DropDown_resource_bundle"],
    defines = ["SWIFT_PACKAGE"],
    deps = [],
    generates_header = True,
    module_name = "DropDown",
    srcs = [
        "DropDown/helpers/DPDConstants.swift",
        "DropDown/helpers/DPDKeyboardListener.swift",
        "DropDown/helpers/DPDUIView+Extension.swift",
        "DropDown/src/DropDown+Appearance.swift",
        "DropDown/src/DropDown.swift",
        "DropDown/src/DropDownCell.swift",
        ":DropDown_DropDown_resource_bundle_accessor",
    ],
    tags = ["manual"],
    visibility = ["//visibility:public"],
)

I attempted to reproduce your error in #738. The build worked. The BUILD.bazel for the DropDown package was:
load("@bazel_skylib//rules:build_test.bzl", "build_test")
load("@build_bazel_rules_apple//apple:resources.bzl", "apple_resource_bundle")
load("@build_bazel_rules_swift//swift:swift.bzl", "swift_library")
load("@rules_swift_package_manager//swiftpkg:build_defs.bzl", "generate_modulemap", "resource_bundle_accessor", "resource_bundle_infoplist")

apple_resource_bundle(
    name = "DropDown_DropDown_resource_bundle",
    bundle_name = "DropDown_DropDown",
    infoplists = [":DropDown_DropDown_resource_bundle_infoplist"],
    resources = ["DropDown/resources/DropDownCell.xib"],
    visibility = ["//visibility:public"],
)

build_test(
    name = "DropDownBuildTest",
    targets = ["@swiftpkg_dropdown//:DropDown_DropDown"],
    visibility = ["//visibility:public"],
)

generate_modulemap(
    name = "DropDown_DropDown_modulemap",
    deps = [],
    hdrs = [":DropDown_DropDown"],
    module_name = "DropDown",
    visibility = ["//visibility:public"],
)

resource_bundle_accessor(
    name = "DropDown_DropDown_resource_bundle_accessor",
    bundle_name = "DropDown_DropDown",
)

resource_bundle_infoplist(
    name = "DropDown_DropDown_resource_bundle_infoplist",
    region = "en",
)

swift_library(
    name = "DropDown_DropDown",
    data = [":DropDown_DropDown_resource_bundle"],
    defines = ["SWIFT_PACKAGE"],
    deps = [],
    generates_header = True,
    module_name = "DropDown",
    srcs = [
        "DropDown/helpers/DPDConstants.swift",
        "DropDown/helpers/DPDKeyboardListener.swift",
        "DropDown/helpers/DPDUIView+Extension.swift",
        "DropDown/src/DropDown+Appearance.swift",
        "DropDown/src/DropDown.swift",
        "DropDown/src/DropDownCell.swift",
        ":DropDown_DropDown_resource_bundle_accessor",
    ],
    tags = ["manual"],
    visibility = ["//visibility:public"],
)

What version of rules_swift_package_manager are you using?

It is imported with bzlmod, I think related lines are:
bazel_dep(name = "rules_swift_package_manager", version = "0.14.0")
bazel_dep(name = "cgrindel_bazel_starlib", version = "0.18.0")

bazel_dep(
    name = "bazel_skylib_gazelle_plugin",
    version = "1.4.2",
    dev_dependency = True,
)
bazel_dep(
    name = "gazelle",
    version = "0.32.0",
    dev_dependency = True,
    repo_name = "bazel_gazelle",
)

bazel_dep(name = "rules_swift_package_manager", version = "0.21.0")
Yeah, I see it is hopelessly outdated

The issue is gone after updating to rules_swift_package_manager version 0.20.0 (for some reason 0.21.0 is not available in bzlmod registry)

There is a bug in the BCR presubmit which is preventing the registry update. Hopefully, it will be available later today.


Frontend token validation Alert Addons

This PR builds upon #8. It includes:

Alert reset button indicating whether the reset has been successful or not. If it is unsuccessful, then it will display the error message.
Error alert for the validate and get info button that displays the error message if the actions has been unsuccessful.

@googlebot I consent.


Bazel can't find ar

Issue description
Bazel builds are failing for me with the latest version (0.11.1) installed from nixpkgs HEAD. I also had the same issue with 0.10.1
Steps to reproduce
git clone git@github.com:tweag/rules_haskell_examples.git
cd rules_haskell_examples
bazel build //vector
...
ERROR: /home/ben/dev/haskell/rules_haskell_examples/vector/BUILD:12:1: SkylarkAction vector/libHSvector-1.0.0.a failed (Exit 1)
src/main/tools/process-wrapper-legacy.cc:58: "execvp(/usr/bin/ar, ...)": No such file or directory
Target //vector:vector failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 11.966s, Critical Path: 9.34s
FAILED: Build did NOT complete successfully

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.
⇒  nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/nhfgv0xxi9shl5lvl9jaf11vmm9zkv42-nix-info
fetching path ‘/nix/store/nhfgv0xxi9shl5lvl9jaf11vmm9zkv42-nix-info’...

*** Downloading ‘http://cache.nixos.org/nar/0m0hkzniw91kgh4vciarl0332phi8hzljw8n91548hkmghm1b305.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/nhfgv0xxi9shl5lvl9jaf11vmm9zkv42-nix-info’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1640  100  1640    0     0  11970      0 --:--:-- --:--:-- --:--:-- 11970

 - system: `"x86_64-linux"`
 - host os: `Linux 4.9.86, NixOS, 17.09.3204.d16ffa15090 (Hummingbird)`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 1.11.8`
 - channels(ben): `""`
 - channels(root): `"nixos-17.09.3204.d16ffa15090"`
 - nixpkgs: `/home/ben/dev/nixpkgs`

nixpkgs is at 9049f8a418b1d29f8066baec681684677fce28c7
See also https://github.com/tweag/rules_haskell/issues/203

This one needs a patch:
src/main/tools/process-wrapper-legacy.cc

You can replace it with $AR that is available in stdenv by using substituteInPlace:
$ nix-shell -p hello --command 'echo $AR'
bash: /run/user/1000/env-vars: cannot overwrite existing file
ar

Ok. the file does not contain this.
Maybe it is decided here somewhere:
https://github.com/bazelbuild/bazel/blob/832a0b193a9a098897781bc6298d0455b3913482/tools/cpp/unix_cc_configure.bzl#L103

This starts working again on a system that has have nix-env -i binutils run...

Probably needs to be in path of bazel then.

@boothead Have you tried adding it to makeBinPath in installPhase?

@abbradar I tried this with this commit: https://github.com/boothead/nixpkgs/commit/a791221178c0b1f7c076da234c193d2de93fda9d It doesn't seem to have made a difference.

@boothead Hm, strange -- exactly this should work in principle. I'll give a better look later; not sure what happens.

Looks like patching scripts that generate scripts helps https://github.com/amckague/nixpkgs/commit/0326835055a5b4e2d2a1074e51e32b0d58136783#diff-4746b81564a01e241a84d455575437e7R89 I'm new to making this kind of change so any suggestions are welcome.

Related https://github.com/NixOS/nixpkgs/issues/24063

Pretty sure I found the root cause of this problem! Turns out bazel provides default locations to standard tools that some bazel rules use. The file I linked is actually auto-generated, which made debugging very confusing. I initially patched that file, without noticing that the build process would just override it with the old one. No idea why they check in the generated file.
Anyway, the tool location is ultimately determined by this function which just uses which to discover the tool locations. So all that is needed is to add bintools to the buildInputs of bazel.
I'll solve this as part of https://github.com/NixOS/nixpkgs/pull/63616.

I was a bit too quick, I'm still not able to properly fix the issue.

Since we are now at bazel 1.1, is this issue fixed?

I am still encountering this issue with Bazel v5.4.0. Adding bintools or binutils to buildInputs or defaultShellUtils did not help. I think this issue needs to be re-opened.


Slow loading in category scenarios

Very slow loading with no progress report when scenarios are built from categories.
The model should open without building the scenarios (that can be a separate step done prior to running the model; should be able to do it manually). However when the model is opened, it should tell us how many scenarios there will be (work out the combinations without doing too much of the dirty work).

Also appears to be very slow when running the model (pre-startup): is it regenerating all the scenarios again?

How much do we need to build for the category scenarios before running them?

Still under investigation 2/21/2023


No support for Compact OS

Compact OS was intentionally removed due to the limited use. It is possible to add it back in however, depending on feedback.

Not required - not going to fix


styles: fix RadioButtonGroup styles in mobile

Changes proposed in this PR:

Fix RadioButtonGroup styles in mobile

[ ] I have followed (at least) the PR section of the contributing guide.
@90milesbridge/tigger

Pull Request Test Coverage Report for Build 3017

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.979%

Totals

Change from base Build 3007:
0.0%

Covered Lines:
2666

Relevant Lines:
3181

💛  - Coveralls


Refactor and add test coverage

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (5e9c7ff) 5.23% compared to head (12dc575) 0.00%.

Files
Patch %
Lines

unbound_exporter.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##            main     #68      +/-   ##
========================================
- Coverage   5.23%   0.00%   -5.24%
========================================
  Files          1       1
  Lines        191      31     -160
========================================
- Hits          10       0      -10
+ Misses       181      31     -150

Flag
Coverage Δ

unbound_exporter_tests
0.00% <0.00%> (-5.24%)
:arrow_down:

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

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


Refactor Cookbook to be Resource Driven

This exposes two new resources, gemrc and bundle_config which allow users to idempotently modify the bundle config files as well as the ~/.gemrc file or the global scoped gemrc file. Additionally, we have added a new test harness around the cookbook and a Rakefile to make interacting with the harness a more delightful experience.
Implements internal feature COOL-551.
/cc @chef-cookbooks/engineering-services @mwrock

👍 modulo Matt's comments and passing travis.

@tas50 If you get a chance, I would love to get a final sign-off. Thanks!


Many thanks 🙏

@hugohomesquita I would like to express my gratitude for this plugin. Kindly continue the outstanding work with it. 😃

Thank you very much! It is very gratifying to know that the plugin is contributing to the community.


Missing IDs

Page is not updated to the current event ids that we see on clients, missing from event id 49 and forward

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

ID: ab6dfd3e-71ac-7987-80d7-5bd380bed06a
Version Independent ID: 9eb3c26b-e61c-2c8b-35a0-415c2a26bfed
Content: Review events and errors using Event Viewer - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/event-error-codes.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Would you be able to help us identify the rest of the missing event IDs? We’d like to come up with revisions accordingly. Thank you.

Any progress here? Still missing ID49 in the list.

@adirdidi There is an independent contributor who is assigned to take care of this issue who can come up with a PR once the essential information has been obtained. That's why help had been wanted. Please let me know if you intend to pick the issue up from here. Thanks.

I didn't removed the help tag.
Regardless, I don't see how someone not from the product group can handle this ask.
And yes - I'm working on it.

Noted, Thank you!

Updated: https://docs.microsoft.com/microsoft-365/security/defender-endpoint/event-error-codes.


Is there any possibility to disable ICE?

There is delay of 3 seconds (fixed) while searching and Connecting to ice candidates as mentioned in rtc_session.dart
_connection.onIceCandidate = (RTCIceCandidate candidate) {
if (candidate != null) {
emit(EventIceCandidate(candidate, ready));
if (!finished) {
finished = true;
/
*  Just wait for 3 seconds. In the case of multiple network connections,
*  the RTCIceGatheringStateComplete event needs to wait for 10 ~ 30 seconds.
*  Because trickle ICE is not defined in the sip protocol, the delay of
* initiating a call to answer the call waiting will be unacceptable.
/
setTimeout(() => ready(), 3000);
}
}
};*
Question 1 . :- is there any possibility to disable ICE from the client side?

Yep, you can disable this by doing (make sure you always search closed GH issues too :-) ):
    // https://github.com/flutter-webrtc/dart-sip-ua/issues/136#issuecomment-732970413
    // Not needed.
    settings.iceServers = [];


Revert host file

Hello.
I overwrote the file host with
echo " 'your home assistant ip' cecotec.das.3irobotix.net cecotec.download.3irobotix.net cecotec.log.3irobotix.net cecotec.ota.3irobotix.net eu.das.3irobotics.net eu.log.3irobotics.net eu.ota.3irobotics.net cecotec-das.3irobotix.net cecotec-log.3irobotix.net cecotec-upgrade.3irobotix.net cecotec-download.3irobotix.net" >> /etc/hosts
Since i did, i have issues to use the conga robot with the Cecotec original app. What should I do to revert the file to the original configuration? maybe:
echo "cecotec.das.3irobotix.net cecotec.download.3irobotix.net cecotec.log.3irobotix.net cecotec.ota.3irobotix.net eu.das.3irobotics.net eu.log.3irobotics.net eu.ota.3irobotics.net cecotec-das.3irobotix.net cecotec-log.3irobotix.net cecotec-upgrade.3irobotix.net cecotec-download.3irobotix.net" >> /etc/hosts
Thanks in advance.

Please, refer to his doc https://github.com/congatudo/stuff/blob/master/docs/stepbystep-valetudo-installation.md
Uninstall valetudo step, in you case, as you don't have a standalone installation, just the third command, the sed one


WIP - colab mods

This PR modifies the india districts study so it runs in the colab notebook.
The main changes:

Added a function to the etl.py to write the files from the API to Google Drive.
I removed the code in the lockdown_evaluation.py top-level script to the body of the python colab notebook and import the etl and lockdown_evaluation scripts into the colab environment
I simply replicated the code 1 to 1 from what was in this repo and there are probably some modifications we'll want to make to write figures to Google Drive.


Do not show "Click to enter title" when timer has expired or switch to input mode on title focus

The window looks strange with the buttons but without the title text box, so we will continue to show the title when the timer has expired but focusing it will switch to input mode.


 Looks like you're missing the keyboard library.

you type ENTER not return

Sounds like a syntax error, did you downloaded or copied the script ?
(Are toi on Linux ? And if yes did you installed Arduino IDE with apt-get ?)

Will close this issue in a bit if we don't get a response from @JonasZaverka

Closing the issue due to inactivity


docs: add docs on how to add sample apps to code base

Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

[ ] Assign members of your team to review the pull request.
[ ] Wait for pull request status checks to complete. If there are problems, fix them until you see that all status checks are passing.
[ ] Wait until the pull request has been reviewed and approved by a teammate
[ ] After code reviews are approved and you determine this PR is ready to merge, select Squash and Merge button on this screen, leave the title and description to the default values, then merge the PR.

PR needs to fix fastlane configuration.
From CI logs, it looks like the SharedFastfile I created didn't actually get imported.


Bug v1.10.0: App asks for Authentication at every Start

Desirable behaviour:
Only ask for Authentication when it is actually needed it. E.g.

when transactions are made
when registering
when submitting attestation

Implemented in previous version.

I even had to enter the pin twice: once in a dialog, once on the new pin entry screen

Consider acceptance points, which never do transfers because observation accounts.

After consideration, The best way to deal with authentication is how it was before. That means: Only ask for authentication when it is actually needed.
The current implementation is actually even worse. Once the app is opened and unlocked one can make a transfers without having to reenter the pin / unlocking. Even if the app sits in the background for a while.
We can think about optionally add the unlocking on app open later.
E.g. Prompting the user if a certain balance is hit, biometric has been activated or just hinting that they can make the account more secure / private.

Our UX should'nt mimic metamask! We have much less value in our wallets so there should be much leaner UX as we don't need to be ultra-secure even with showing balances.
I want to authenticate at most once per session. And if I don't send something to the chain or backup my seed I don't see why we should ask for PIN/biometric

Thanks for the question and the answer.

I like the logic to show the PIN / biometrics only when an interaction with the chain is happening
When backing up the seed also makes a lot of sense.
Delete accounts should also ask for PIN / biometrics

There is no need to enter the pin everytime the app is opened.

at most once per session.

In general good from my pov. When is a session ended? Is it after a certain time?
If there is no reasonable time limit for session (e.g. 10'), I would end a session, once the app is not in the foreground anymore.


Can't join any space using ledger

Describe the bug
Some users report they are unable to join any space using ledger ( i can't reproduce )
To Reproduce
Steps to reproduce the behavior:

click join on any space in home page

Expected behavior
Should be able to join a  space
Screenshots
Desktop (please complete the following information):

OS: unknown
Browser Firefox
Version

Error:
"RPC Error: MetaMask Message Signature: Error:"

https://discord.com/channels/707079246388133940/747049240022482974/888166508080025610

Here is a message of a user, he is on Metamask:
Hello there guys, I am having trouble joining the space that I have created. I want to make sure that everything is working before I invite the community to participate.
https://snapshot.org/#/seachaintoken.xyz
When I click join, the site appears to think a few moments, but I do not join the space
I have disconnected and reconnected my address to the site, as well as cleared my cache, the error persists
firefox, metamask
in the console:
TypeError: o is undefined
sign https://snapshot.org/assets/vendor.a19cc799.js:99
alias https://snapshot.org/assets/vendor.a19cc799.js:99
setAlias https://snapshot.org/assets/index.4eaf7f82.js:120
h https://snapshot.org/assets/index.4eaf7f82.js:120
Is this because I am the space creator?

Hi @metrox-eth, thanks for reporting this issue. We are aware of this and are currently working on a fix.

@metrox-eth it should work now, please let me know if there is any other issue!

@ChaituVR ledger will not work maybe because we are using EIP712 to sign the alias?

@ChaituVR ledger will not work maybe because we are using EIP712 to sign the alias?

@samuveth User said error is
"RPC Error: MetaMask Message Signature: Error:"

so I thought he is using ledger with metamask, 🤔 which was working fine to me, i was able to join a space with ledger through metamask

@ChaituVR Oh, okay that's great!

Hi there, I got the exact same issue. I can add join certain space (e.g. Gitcoin, dx/dy, OlympusDao and many others), but not others (e.g. Sushiswap, Aave, Uniswap, Balancer). I was in the four later and got un-joined without me doing anything. I am still able to vote there if I open them, I just cannot join.
My specs:

OS: Windows 10 Pro H121
Browser: Tried both Brave and Firefox (also tried to clear the cache and open the Ledger Live bridge)
Wallet: Ledger Nano X (Firmware v 2.0.0) connected via Metamask

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Just wanted to flag that the issue is still existing.

@Pi-Rr can try Chrome?

@Pi-Rr I'm still unable to reproduce with my Ledger Nano X.

@Pi-Rr can you try without the ledger bridge? Try turning it off and using ledger with metamask while the ledger app is not running. Also make sure that firmware and metamask are on the latest versions.

I am not using the Ledger bridge, my Ledger is connected to Metamask
directly (using WebHID). Ledger app is closed. Firmware and metamask are on
the latest versions.
On Tue, 14 Jun 2022 at 16:48, Sam @.***> wrote:

@Pi-Rr https://github.com/Pi-Rr can you try without the ledger bridge?
Try turning it off and using ledger with metamask while the ledger app is
not running. Also make sure that firmware and metamask are on the latest
versions.
—
Reply to this email directly, view it on GitHub
https://github.com/snapshot-labs/snapshot/issues/640#issuecomment-1155383231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIQFRZTYGCDT2RZKAA7RGVTVPCSTXANCNFSM5EGFXBWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

For the update, I just tried again and it seems like everything is working fine now; must have been solved by one of the recent update, no clue which one. This issue can be close, thanks :)

@Pi-Rr Thanks for the update!


Hasan Dissertation

@jaywonchung, @yiwenzhang92 or @AmberLJC can you please have a review
#184

@jaywonchung Pushed the requested changes. Please let me know if it works


fix: clean ci

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


[Feature request] Bind reactive data and build loops with html attributes?

Hi @cferdinandi,
I coincidentally found your nice reef project and it's similiar to some other small solutions I know.
Would it possible to add the feature to

bind html value / content by attribute like r-bind="${myvalue}" or optional (hidden) elements with r-if="${condition}"
create loops based on html attribute r-foreach="${myDataArray}".  The main element is used as container (for example a  tag) and html inside as row template filled by array values? array value could be an object.

I would prefer to keep html templates separated and not mixed up / inline with javascript code. So load templates with ajax, bind values / loops to html elements and use reef to fill and update reactive data and routing urls / pages?

I absolutely appreciate the desire for these, but they add a layer of complexity and "done for you" that goes beyond the intended scope and simplicity of Reef.
I'd personally much rather see these hard-coded into the templates themselves, or, in the case of binding, supported via an event listener.

Hi @cferdinandi,
I took a closer look at the examples and documentation. reef is really great! I think you're right to keep the complexity low! So I need to build some helper around reef for my needs :)
I don't like template strings or small template files with just a <li> part for loops.
Maybe I could try to read the "sub-template" part (children of ) and replace it by reef template / data... But I would need to cache that sub-template for re-render...


Corrupt Word File - Xml Parsing error Location: Part: /word/styles.xml

The resulting Word document opens with a warning about an XML parsing error.
During recovery the repair simply lists Styles 1

I have been trying to submit a pull request to fix the issue but am unable to for some reason.

Looking at the repository I see a commit that addresses this issue, but it is not reflected on my system when I install the module with yarn or npm:
yarn add docxtemplater-link-module
yarn add v1.2.1
warning package.json: No license field
warning No license field
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
warning Your current version of Yarn is out of date. The latest version is "1.3.2" while you're on "1.2.1".
success Saved 1 new dependency.
└─<EMAIL_ADDRESS>warning No license field
Done in 1.22s.

released v0.2.4


Add is_deleted field to TimeEntry

Coverage increased (+0.009%) to 93.054% when pulling ca3d5b10f03c37f3c75ab4bd18457925493e4a7a on add-deleted into be61c8b6f1091cd8941aa9b1b4666d5fa0df7460 on master.

LGTM after the one quick docstring update

Coverage increased (+0.009%) to 93.054% when pulling 2e726505aa3b827b691e333fa5962d49565dba14 on add-deleted into be61c8b6f1091cd8941aa9b1b4666d5fa0df7460 on master.


Add public_network_access_enabled variable

Fixes #000
Changes proposed in the pull request:

Add public_network_access_enabled variable

Last year this option was added to the azurerm_postgresql_server resource in the azurerm provider, it would be nice if this option could be used in this module. More information: https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/postgresql_server#public_network_access_enabled
I've formatted and tested the change.

I've added the feature to the test/fixture. I looks like the test failed due to a problem with pulling the docker image, not due to the change.


Add input action handler interfaces for non-digital actions

Describe the problem
Currently we can only handle digital input actions (on/off types) via IMixedRealityInputActionHandler. We should add a generic version of the interface for other input action types, similar to what we have for IMixedRealityInputHandler. #4477 must be fixed first for this.

@Zee2 can you confirm that this is solved in v3?

This is solved by variable selectedness and IVariableSelectInteractor and StatefulInteractable in MRTK3. Three years later. 😛


[Bug] package-lock.json is not consistent between different operating systems.

Describe the bug
In many times, when we install agenta-web in different operating systems, we need to delete package.lock and reinstall dependencies again. It looks like the depencies are different between one os and onother one which leads to a bad experience while working with agenta the especially for first-timers
To Reproduce
Not sure how but from previous experiences we had in agenta:

Someone with MacOs removes package.lock
Install dependencies
Commit it
Someone else with ubuntu pull the package.lock

should encounter  errors when installs npm install

Expected behavior
npm i / npm ci should work without the need of deleting package.lock
Screenshots

No screenshots but this PR could a proof why we have a new package.lock commited whereas no new dependencies are introduced.
https://github.com/Agenta-AI/agenta/pull/645/files
We also should be able to find people complains about this in github issues or in slack

@aakrem can you please confirm that in both OS the node version was the same?

@MohammedMaaz No I don't have information about that. If you think node version might be the issue, does it make sense to add the engine ?
https://docs.npmjs.com/cli/v10/configuring-npm/package-json#engines

@aakrem yes perhaps, it's good to specify the node version in package.json anyway. I'll check if with same node version it still produces this issue on different OS

@aakrem @MohammedMaaz Was this resolved? What's the status on it?

closed due to inactivity


TypeError: must be type, not None

On debian since newer versions of pyscaffold are available on pypi I get this error if pyscaffold is not installed in the virtualenv (pyscaffold gets installed via easy install as egg (:rolling_eyes:) ) before installing the project in dev mode pip install -e .:
pip install -e .
Obtaining file:///vagrant_data/
    Complete output from command python setup.py egg_info:

    Installed /vagrant_data/.eggs/PyScaffold-2.5.8-py2.7.egg
    [pbr] Generating ChangeLog
    ERROR:root:Error parsing
    Traceback (most recent call last):
      File "/vagrant_data/.eggs/PyScaffold-2.5.8-py2.7.egg/pyscaffold/contrib/pbr/pbr/core.py", line 111, in pbr
        attrs = util.cfg_to_args(path, dist.script_args)
      File "/vagrant_data/.eggs/PyScaffold-2.5.8-py2.7.egg/pyscaffold/contrib/pbr/pbr/util.py", line 267, in cfg_to_args
        wrap_commands(kwargs)
      File "/vagrant_data/.eggs/PyScaffold-2.5.8-py2.7.egg/pyscaffold/contrib/pbr/pbr/util.py", line 569, in wrap_commands
        cmdclass = ep.resolve()
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2297, in resolve
        module = __import__(self.module_name, fromlist=['__name__'], level=0)
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/sphinx/setup_command.py", line 23, in <module>
        from sphinx.application import Sphinx
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/sphinx/application.py", line 33, in <module>
        from sphinx.environment import BuildEnvironment
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/sphinx/environment/__init__.py", line 46, in <module>
        from sphinx.util.websupport import is_commentable
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/sphinx/util/websupport.py", line 11, in <module>
        from sphinxcontrib.websupport.utils import is_commentable  # NOQA
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/sphinxcontrib/websupport/__init__.py", line 15, in <module>
        from sphinxcontrib.websupport.core import WebSupport  # NOQA
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/sphinxcontrib/websupport/core.py", line 17, in <module>
        from jinja2 import Environment, FileSystemLoader
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/jinja2/__init__.py", line 33, in <module>
        from jinja2.environment import Environment, Template
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/jinja2/environment.py", line 15, in <module>
        from jinja2 import nodes
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/jinja2/nodes.py", line 19, in <module>
        from jinja2.utils import Markup
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/jinja2/utils.py", line 486, in <module>
        MutableMapping.register(LRUCache)
      File "/home/vagrant/venv/lib/python2.7/abc.py", line 109, in register
        if issubclass(subclass, cls):
      File "/home/vagrant/venv/lib/python2.7/abc.py", line 180, in __subclasscheck__
        if issubclass(subclass, scls):
      File "/home/vagrant/venv/local/lib/python2.7/site-packages/typing.py", line 1265, in __subclasscheck__
        return super(GenericMeta, self).__subclasscheck__(cls)
    TypeError: must be type, not None
    error in setup command: Error parsing /vagrant_data/setup.cfg: TypeError: must be type, not None

Installing pyscaffold in the virtualenv before pip install -e.  solves the problem but is somehow ugly and doesn't really fit in my workflow...
Anything I can do to circumvent this or is this a bug?

Hi, we had this problem once on a Travis build and solved it by pinning Sphinx to version 1.5.6. We could only find out that it has something to do with Sphinx and most likely PBR which is used by PyScaffold 2.5.* in the way it monkey patches into setuptools.
In the current beta of PyScaffold 3 we got rid of PBR and also of the problem you describe. PBR does a lot of black magic and really is a nightmare, that's why we kicked it.
That being said, I could never reproduce the problem locally on my machine. There are two things you could do to help us out extremely:

Describe how we can reproduce the problem by using PyScaffold 2.5.*
Upgrade your package to the current beta of PyScaffold 3 and tell us if you still go the problem.

Your feedback is greatly appreciated!

It can be worked around by installing pyscaffold using pip before pip install -e.:
pip install pyscaffold==2.5.7
pip install -e.

but this is really just a workaround...we should fix this somehow...but I have no clue...

@sebastianneubauer Thanks, this is a workaround but what about the other points from my comment above? Does this also happen if you fix Sphinx to version 1.5.6? Can you reproduce it also directly on your laptop with a minimal example so that I can reproduce it too? I would really like to understand more but so far I could not reproduce it. I agree that a workaround like this should not be necessary.

This error could not be reproduced on the current master which will be released as version 3.0 soon. Closing this issue for now.


homebridge failed to start on 0.2.4 upgrade - TypeError: Service.Television is not a constructor

just upgraded to 0.2.4 release via npm, seeing below on restart of homebridge.
Feb  3 14:28:51 vpie systemd[1]: Started Node.js HomeKit Server.
Feb  3 14:28:53 vpie homebridge[3896]: [2019-2-3 14:28:53] Loaded config.json with 1 accessories and 1 platforms.
Feb  3 14:28:53 vpie homebridge[3896]: [2019-2-3 14:28:53] ---
Feb  3 14:28:54 vpie homebridge[3896]: [2019-2-3 14:28:54] Loaded plugin: homebridge-harmony
Feb  3 14:28:54 vpie homebridge[3896]: [2019-2-3 14:28:54] Registering platform 'homebridge-harmonyHub.HarmonyHubWebSocket'
Feb  3 14:28:54 vpie homebridge[3896]: [2019-2-3 14:28:54] ---
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] Loaded plugin: homebridge-thermometer-am2320
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] Registering accessory 'homebridge-am2320.AM2320'
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] ---
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] Loading 1 platforms...
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] [vHarmonyHub] Initializing HarmonyHubWebSocket platform...
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] [vHarmonyHub] PlatformMode : true
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] [vHarmonyHub] Loading activities...
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] Loading 1 accessories...
Feb  3 14:28:55 vpie homebridge[3896]: [2019-2-3 14:28:55] [Sensor] Initializing AM2320 accessory...
Feb  3 14:28:55 vpie homebridge[3896]: AM23280 sensor options: {"i2cBusNo":1,"i2cAddress":92,"temperature":{"rate":1},"humidity":{"rate":1}}
Feb  3 14:28:56 vpie homebridge[3896]: data = {
Feb  3 14:28:56 vpie homebridge[3896]:   "temperature": 20.9,
Feb  3 14:28:56 vpie homebridge[3896]:   "humidity": 27.8
Feb  3 14:28:56 vpie homebridge[3896]: }
Feb  3 14:28:56 vpie homebridge[3896]: [2019-2-3 14:28:56] [vHarmonyHub] Creating Main TV Service
Feb  3 14:28:56 vpie homebridge[3896]: /usr/lib/node_modules/homebridge-harmony/harmonyAsTVPlatform.js:117
Feb  3 14:28:56 vpie homebridge[3896]:       controlService: new Service.Television(that.name, 'tvService'),
Feb  3 14:28:56 vpie homebridge[3896]:                       ^
Feb  3 14:28:56 vpie homebridge[3896]: TypeError: Service.Television is not a constructor
Feb  3 14:28:56 vpie homebridge[3896]:   at HarmonyPlatformAsTVPlatform.readAccessories (/usr/lib/node_modules/homebridge-harmony/harmonyAsTVPlatform.js:117:23)
Feb  3 14:28:56 vpie homebridge[3896]:   at harmonyPlatform.wsp.onUnpackedMessage.addListener.data (/usr/lib/node_modules/homebridge-harmony/harmonyBase.js:283:33)
Feb  3 14:28:56 vpie homebridge[3896]:   at Timeout._onTimeout (/usr/lib/node_modules/homebridge-harmony/node_modules/websocket-as-promised/dist/index.js:2:4404)
Feb  3 14:28:56 vpie homebridge[3896]:   at ontimeout (timers.js:498:11)
Feb  3 14:28:56 vpie homebridge[3896]:   at tryOnTimeout (timers.js:323:5)
Feb  3 14:28:56 vpie homebridge[3896]:   at Timer.listOnTimeout (timers.js:290:5)
Feb  3 14:28:56 vpie systemd[1]: homebridge.service: Main process exited, code=exited, status=1/FAILURE
Feb  3 14:28:56 vpie systemd[1]: homebridge.service: Unit entered failed state.
Feb  3 14:28:56 vpie systemd[1]: homebridge.service: Failed with result 'exit-code'.

homebrige config:
"platforms": [

	   {
		"platform": "HarmonyHubWebSocket",
		"name": "HarmonyHub",
		"hubIP": "<IP_ADDRESS>",
		"TVPlatformMode" : true,
		"mainActivity" : "Watch Apple TV"
	   }

	]

Hi,
using TVPlatformMode needs homebridge 0.0.46 at least + IOS 12.2 beta !


Replaces Jade with Pug

Closes #46

Coverage remained the same at 100.0% when pulling 35210e094e229e4f7ee040bb841bd419b4a4529d on feature/use-pug into 74f2335c921e9ca0d3969e8fd377a73ffbbc94d6 on master.


Using with Kafka 0.10.x

I am trying to use this with kafka 0.10.x . Has anyone tried that ? Any known issues if its even compatible.

Found a  - fork here that has used kafka 0.10


Open folder as a workspace in Explorer View

Type: Feature Request
While navigating some folders in Ctrl+Shift+E Explorer View via arrow keys, I sometimes want to open a highlighted folder as a separate workspace. I right-clicked (shift+F10 on Windows) to see if I can find a menu like "open in a new workspace," but unfortunately, there was not any similar command.
Could you please consider adding this feature?
VS Code version: Code - Insiders 1.82.0-insider (85daa35afb86202badbee79532537bf226961f25, 2023-08-08T07:41:53.755Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Duplicate https://github.com/microsoft/vscode/issues/41860


quiet option

I am looping over several queries into a catalog to process datasets. Each call to .to_xarray() generates tqdm output...which accumulates with each iteration of my loop. It would be nice to (a) turn this off or (b) replace the status information in situ, rather than append new status bars.

@matt-long, this has been addressed in #119
from intake_esm import config
config.set({'progress-bar': False})


Typescript improvements

This PR makes three small improvements to how types are published and contributes towards #576. (I think only server types still need to be exposed.)

Moved ts-toolbelt from devDependencies to dependencies.
While working with 0.39.2, I got errors because ts-toolbelt wasn’t installed and the exposed types depended on it.

Added a types field to the main boardgame.io package.json. Now the basic types in src/types can be consumed more simply:
import { State } from 'boardgame.io';

Imported and exported the storage API types (FetchResult etc.) in src/types so they can be used in third-party database connectors:
import { StorageAPI } from 'boardgame.io';
const opts: StorageAPI.FetchOpts = { state: true };

Released in 0.39.3.


Support for Genomics Pipeline v2beta

Hi dsub team! My team and I are avid users of your product.  We recently had an issue with Google, and the support agent was quite insistent that we should update the the new v2beta version of the Genomics Pipeline API.  We have been checking master to see if there are any related changes happening, but we haven't seen anything yet.  Since there doesn't seem to be a related open issue, I thought I would open one.
The changes to the API seem (to me) to be quite minimal, but the big advantage for users is enhanced client support from Google's support engineers if any difficulties arise in the use of the software.

Hi @sconstable!
Thanks for the request. We have begun work on a provider for v2beta. Can't give a timeline yet, but it is in the works.

Hi @sconstable ,
dsub 0.3.7 was released which includes v2beta as an experimental provider. Please give it a try by using --provider google-cls-v2


Fix #12292 : error when read one empty column from excel file

Fix #12292

Just made my changes only inside excel.py itself to avoid mess up the html parser.

@jreback please take a look

@BranYang does this also fix #9002

@BranYang this looks good.

@jreback I can confirm that I captured the StopIteration and return an empty DataFrame. So I believe this also fix #9002

thanks @BranYang


Convert all gene IDs to names

Example: https://testdb.dictybase.org/gene/gskA/goannotations
Make sure Uniprot ID -> gene ID goes to name if applicable.
Also update the links to go to gene page, not uniprot.

All names are properly being converted on the frontend now, and I wrote unit tests with 100% coverage to verify this. This will be live in the next release.
The backend has also been patched, and once that goes live this issue and #68 will be fixed.

Fixed in 3.2.0


Submitting a comment title longer than 50 char throws an error

When you try to create a comment on an issue, if the title of the comment is longer than 50 characters, a database error is thrown.
In my case, the offending title was auto-generated off of the issue title.

runtime error
Warning caught during route execution: DBD::Pg::db do failed: ERROR:  value too long for type character varying(50) at /opt/ptrack/pTracker/lib/pTracker/DB.pm line 213.
/opt/ptrack/pTracker/lib/pTracker/DB.pm around line 213
210
211     $self->log_query( $query, @ary );
212
213     eval { $dbh->do( $query, undef, @ary ); };
214     if ($@) {
215         $rv{error}  = 'Failed to create record.';
216         $rv{errstr} = $dbh->errstr;

Added code to limit the length of the default comment title to the length of the database column.
Increased the size of the database column.
Verified that the maxlength of the web page form field was getting set.


Improve documentation on "Monitor Procrastinate in a real environment"

Hi,
We are running a small scale procrastinate setup in our production environment.
The Monitor Procrastinate in a real environment (rst permalink) documentation is something I believe should be expanded upon.

Our strategy is currently to check that there's no stalled/non-completed jobs, using this query:
select coalesce(sum(age(current_timestamp, pe.at)), interval '0 seconds') as total_jobs_age,
       count(pj.id)                                                       as job_count
from procrastinate_jobs as pj
  left join procrastinate_events as pe on pj.id = pe.job_id
where pj.status not in ('succeeded', 'failed')
  and pe.type = 'deferred'
  and pj.id not in (
    -- get a list of all jobs scheduled sometime in the future
    -- e.g. jobs that have been scheduled to run at a later date
    -- or been scheduled for retry
    select pj.id
    from procrastinate_jobs as pj
      left join procrastinate_events as pe on pj.id = pe.job_id
    where pj.status not in ('succeeded', 'failed')
      and pe.type = 'scheduled'
      and pe.at > current_timestamp
    order by pe.at

We have written a Prometheus exporter to expose this, and are then presenting the two values (total_jobs_age, job_count) in a Grafana dashboard with alerts on thresholds.
In addition are we running Sentry, which catches exceptions thrown (nothing special, just sentry_sdk.init(dsn=sentry_dsn, environment=env)).
Curious to hear if there are issues with this approach. Also if there are "blind spots" we should consider.

@kbakk is sentry catching exceptions without additional setup? I wrote a decorator to get it working, but I didn't think to test if errors are captured with just a call to init()

Hey ! I somehow completely missed the notification for the original issue, sorry :/
You're right, there are a lot of things we can do to improve both the processes and the documentation. Would you be interested in contributing in this direction ?

this is the decorator I use for sentry capture:
def pro_wrap(*args, **kwargs):
    "decorator to register queue task + send errors to sentry. takes same arguments as procrastinate app task() decorator. wrapped function must be async"
    queue_name = kwargs.get('queue', '?')
    def inner_decorator(func):
        @functools.wraps(func)
        async def wrapper(*args, **kwargs):
            try:
                return await func(*args, **kwargs)
            except:
                sentry_sdk.capture_exception(tags={'queue': queue_name, 'task': func.__name__})
                raise
        return pro.task(*args, **kwargs)(wrapper)
    return inner_decorator

@pro_wrap(queue='test')
async def crash_task():
    raise IntentionalError

re submitting a PR -- from a design perspective, seems cleaner to offer a generic hook for error capture, rather than supporting sentry_sdk specifically?
it looks like the worker call stack is (single_worker -> process_job -> run_job), with error capture happening in run_job here:
https://github.com/procrastinate-org/procrastinate/blob/781cbdf0ce272f9bc449acbccc8caa65108c6649/procrastinate/worker.py#L239
would you be interested in a PR that adds an error_handler argument to the App class, and then calls that in the except block in L239 above?
(also, it sounds like kbakk is getting this done without fancy extra wrapping? my solution may be overkill)

I'm going to ask a silly question, but given that we already call logger.error(exc_info=True), doesn't the Sentry logging integration already do the same ?

ohhh you're right
I wasn't seeing this before because in my testing setup, the worker doesn't stay up long enough for sentry to report errors. When I leave it up, the sentry logging integration does the trick
yes

That said... It's probably worth it mentioning this in the docs.

I'm happy to submit a docs-only PR that adds a 'sentry' section to the 'monitoring' page. Interested?

Absolutely :)


Filter 13C isotopes

Added a more comprehensive 13C isotope filter to the rows filter. Previously, we only filter for minimum signals in an isotope pattern. For the Isotope finder, which is more inclusive, we need a better 13C filter.

for each row's m/z we search for the +1 13C isotope peak in the isotope pattern and score the height.
there might be multiple signals in range so all are considered - only one needs to match the height requirements which derived from estimated min / max number of carbons.
estimates the minimum and maximum number of 13 carbons based on mass defect and m/z
optionally looks for a -1 13C peak in the pattern (with matching height) to flag the row as an isotope and filter it
optionally uses other elements to make sure we are not looking at a 18O or any other isotope signal (searching for the main signal at -X)
added plenty of tests to check the performance with multiple charge states considered etc

Cummulative histogram of library matches that match isotope filters (13C = Isotope finder; 2iso = Isotope grouper)

Cummulative spectral library matches matching the 13C filter (Isotope finder) or the 2 isotopes by isotope grouper

What exactly are the graphs meant to mean? I dont see the colour of no match and match 2iso in the charts although theyre present in the legend

Oh sorry forgot to say that.
So the no match was filtered. So I used 28 samples, each a mix of extracted compounds. I know what compounds to expect in which sample and filter for library matches to compounds that we expect in these samples. I removed unmatched from because I only wanted to look at those high quality matches.
match 2iso has no unique matches therefore its empty

I updated the figures there was something wrong with my plotting script. Now it makes more sense

Maybe add it as a parameter to the spectral library matching?

What do you mean by adding to as a parameter to spectral library matching?
This is a filter to not really a scoring method

Tested and ready from my side @SteffenHeu


Make the Layer5 a clickable link

Description
Currently, Layer5 logo in this section is not a clickable link.
Expected Behavior
Add a link to the Layer5 logo directing to the Layer5.io website
Screenshots

Environment:

OS: [e.g. Ubuntu]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]
Device: [e.g. laptop, iPhone 8]

hey @Anita-ihuman , i already fixed this in my recent PR

Can I work on this issue @warunicorn19 if you have not merged your changes yet?

@kumarshobhit that'll cause a conflict, since its already a part of an existing PR waiting to get merged, why dont you take a look at some other existing issues?

This issue was beginner-friendly so I thought it will be a good one to start contributing to the project. But yeah, I will surely look at other issues to solve.

@kumarshobhit, apologies for the confusion.
It’s fine, there will be other friendly issues coming up.

yes noted.


Passing Parse object

I am upgrading from v1.3.2 and I used to pass the full Parse Object from one CommonJS module to another via args. Now that I'm running the latest Parse JS SDK, I'm only getting this on the other end:
//after query before send to new commonjs controller
[DEBUG] :  Found CLUB: {"ACL":{"*":{"read":true},"15QSQgv93Z":{"read":true,"write":true}},"Address":"Atlanta, GA","ApprovalRequired":true,"MemberCount":1,"Name":"The Club Name","Public":true,"createdAt":"2016-02-11T22:01:29.361Z","updatedAt":"2016-02-11T22:01:31.796Z","objectId":"fqMZicTJbK"}

//upon receiving input in new controller
ARGS:{"club":{"_objCount":14,"className":"Club","id":"uPB7v2sb7G"}}

It's no longer the full object. What's happening here?

You're printing out the "surface view" of the object, but the fields are still there. Since 1.6.0, printing the object does not show any attributes, since they are provided under a non-enumerable property. If you want to print the attributes, they're available at object.attributes

.get(), .set(), etc all work the same. The only thing that changed there is printing behavior.

Actually .get() not working is the issue:
var args = arguments[0] || {};
Ti.API.debug("ARGS:"+JSON.stringify(args));
Ti.API.debug("CLUB NAME: " + args.club.get("Name"));

output
[DEBUG] :  ARGS:{"club":{"_objCount":14,"className":"Club","id":"uPB7v2sb7G"}}
undefined is not a function (evaluating 'args.club.get(\"Name\")')

I have the exact same code working if I revert back to v1.3.2. thanks!

Okay, well it looks like the real issue is that get is not a function. I don't know what club is, but it's not a Parse.Object. Can you please provide a larger chunk of code that reliably reproduces your issue? Otherwise, there's no way we can debug this.

Sure it's a little more complicated so I'll put it in a private gist so I can delete it once we solve it.
So I'm developing in Appclerator Titanium. Specifically Alloy. Each Window in the App is a controller. The first controller is a list of "Clubs" (club_list.js) that gets its data from (parseClubs.js See line 45 where I am passing the full club object and notice how I am able to use club.get() fine there). When the user clicks on their club it passes the -whole- Parse Object (Club) to the second window which is the ARGS and output I showed you above.
Here's the gist.
https://gist.github.com/nitrag/183c2cc15921ab7447f2#file-parseclubs-js-L45
Here's the expected output (working in 1.3.2)
[DEBUG] :  ARGS:{"club":{"Address":"Atlanta, GA","MemberCount":1,"ApprovalRequired":true,"Name":"Atlanta 4Runner's","Public":true,"ACL":{"setRoleWriteAccess":{},"getPublicWriteAccess":{},"setWriteAccess":{},"toJSON":{},"setPublicWriteAccess":{},"setPublicReadAccess":{},"getReadAccess":{},"getRoleReadAccess":{},"getPublicReadAccess":{},"setReadAccess":{},"permissionsById":{"*":{"read":true},"15QSQgv93Z":{"write":true,"read":true}},"_setAccess":{},"setRoleReadAccess":{},"getRoleWriteAccess":{},"getWriteAccess":{},"_getAccess":{}},"objectId":"fqMZicTJbK","createdAt":"2016-02-11T22:01:29.361Z","updatedAt":"2016-02-11T22:01:31.796Z"}}
[DEBUG] :  CLUB NAME: Atlanta 4Runner's

Actually it's working in 1.5.0. I forgot I was able to upgrade to that. I'm trying to get to 1.7.1 now so that I can stay current with Parse-Server.

This issue still exists but it's probably related to Appcelerator Titanium. I've implemented a workaround on that end.


net: socket: dtls: handle multiple fragments in sendmsg()

DTLS socket should be able to combine more than one message into a single datagram when calling sendmsg().
Resolves #45956

@rlubos please rebase

Done


Fails to set Tracing when a custom function name is used

Nice plugin. There is a small issue that occurs when a custom name element is used with the serverless functions. In this scenario, Tracing is not set.
It should be a pretty easy fix to check for the 'name' element before calling toggleTracing. Something like:
this.toggleTracing((
service.functions[functionName].name ||
${service.service}-${stage}-${functionName}),
(service.functions[functionName].tracing === true)
|| (providerLevelTracingEnabled && service.functions[functionName].tracing !== false)
);
Thanks

Thanks! Added in<EMAIL_ADDRESS>


could not connect to Keycloak

Hello,
I installed terraformer today and it´s not possible for me to get the terraform resources from Keycloak.
I exported the following Parameters:

KEYCLOAK_URL
KEYCLOAK_CLIENT_ID
KEYCLOAK_USER
KEYCLOAK_PASSWORD
I am sure that these are correct and Keycloak is working for me. I can access it via the GUI and Applications redirect to Keycloak for the authentication already.

I created also the terraform provider for keycleak code and run terraform init
terraformer import keycloak --resources=realms

2023/12/05 15:41:17 keycloak importing all realms
2023/12/05 15:41:17 keycloak importing... realms
2023/12/05 15:41:17 keycloak error initializing resources in service realms, err: keycloak: could not connect to Keycloak
2023/12/05 15:41:17 keycloak Connecting....

System: Mac with M1
Terraformer version: version v0.8.24
Terraform version: v1.5.7
Keycloak version: 22.0.5

still not working...

Yeah wanted to follow up on this, am getting the same issue.
2024/03/29 10:34:22 keycloak importing all realms
2024/03/29 10:34:23 keycloak importing... realms
2024/03/29 10:34:23 keycloak error initializing resources in service realms, err: keycloak: could not connect to Keycloak
2024/03/29 10:34:23 keycloak Connecting....

I've tried multiple provider versions including the 4.0.1 that is listed in the docs as supported. No logs on the keycloak side. It's able to get the bearer token though as expected but cant connect.  I see the log line in the verbose output

[DEBUG] plugin.stdio: received EOF, stopping recv loop: err="rpc error: code = Unavailable desc = error reading from server: EOF"

still not fixed...


wmctrl: fix url

Was going to add a test, but it turns out even invoking the bin/“wmctrl”, “-h” will fire up X11. Yup.
Fixes the URL issue anyway.
Closes #28

Heh, Thanks for the quick fix Misty.


Release 8.1.0

https://github.com/silinternational/ui-components/pull/179

https://github.com/silinternational/ui-components/pull/178

https://github.com/silinternational/ui-components/pull/177

deprecates Form's success prop

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG] Proxy settings ignored by app->repos in a proxy using envorinment

Rancher Server Setup
Downstream imported Cluster

Rancher version: 2.7.3

Installation option (Docker install/Helm Chart):
Helm: with the following: v1.25.11+rke2r1

Proxy/Cert Details:

cat /etc/default/rke2-server
HTTP_PROXY=lte-vt50.lvrintern.lvr.de:3128
HTTPS_PROXY=lte-vt50.lvrintern.lvr.de:3128
NO_PROXY=localhost,.lvrintern.lvr.de,.svc,.cluster.local,<IP_ADDRESS>,<IP_ADDRESS>/16,<IP_ADDRESS>/8,<IP_ADDRESS>/16

Information about the Cluster

Kubernetes version: v1.25.11
Cluster Type (Local/Downstream):

Imported
RHEL 9 Nodes
RKE2
VMWare as CSI / CPI onprem

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)
Admin

Describe the bug
Cant download any other Extern Repo defined in rancher within app -> Repos
All repos throw error which suggest that the Proxy is not beeing used. Also my Firewall team see my try not use the proxy.

Result
Error in Download
Expected Result
Repo beeing created

@lokidaibel I had the same config and error message.
I've managed to resolve this issue on my setup by setting Agent Environment Vars in Cluster config.
Found this solution here https://github.com/rancher/rancher/issues/29961#issuecomment-784467292


PSDEVOPS-3839: Speed up performance

Outdated, incorrect: @RedHatProductSecurity/corgi-devs This change should prevent duplicate rows from being inserted into our DB, which will prevent "ModelName.MultipleObjectsReturnedError" when using "ModelName.get". Note that we may have to clean up any existing duplicate rows before the error goes away, but this fix will keep things from getting worse / more duplicates from being added.
Still accurate: It also improves performance slightly, since using get_or_create with only indexed keyword arguments, and moving unindexed keyword arguments to the defaults= dict, speeds up the database operations. The exact reason is that get_or_create first tries to SELECT a row with matching field values / keyword arguments. The defaults= dict is ignored for this step. If no matching row is found, get_or_create then tries to INSERT a row using the provided keyword arguments and the values in the defaults dict. AKA the defaults= dict is not used for the get_ logic, only the _or_create logic.
Since we only provide indexed fields as keyword arguments, the SELECT query for the get_ logic can now use an index scan, instead of being forced to do a whole table scan to match against the unindexed fields. A whole table scan can be very slow for text fields / large tables with lots of data, although Jason's recent MR was a much better improvement :)

generally echo @jasinner comments and LGTM, though lets hold off on this for today ... would like to get this in (along with some other improvements) this week.

I think we should remove the commit which changes corgi/tasks/brew.py::get_or_create functions. I'm getting ComponentNode.MultipleObjectsReturned errors trying to call save_component_taxonomy or SRPMs persisted with these changes. I'm not sure exactly why, but I think slower performance is better than getting errors ingesting which prevents a materalization of component relationships.

So there are a few things here - I had a small oops in the description, where I talked about fixing this issue. I implied that this PR would fix an error we're having on main (so not related to my changes), Component.MultipleObjectsReturned.
But actually the error on main is ComponentNode.MultipleObjectsReturned, which isn't related to and isn't affected by this change. The get_or_create function changes you mention are only for Component and not for ComponentNode, so they aren't the cause of the error you're seeing.

LGTM


firefoxpwa : Failed to install system integration

Describe the bug
When I try to install a webapp I get the following error : Failed to install system integration: Failed to store web app icons: Failed to create icon directory: Not a directory (os error 20)
Steps To Reproduce
Steps to reproduce the behavior:

Install firefoxpwa this way in /etc/nixos/configuration.nix :

{ config, pkgs, lib, ... }:
let
  unstable = import <nixos-unstable> {};
in
{
  environment.systemPackages = with pkgs; [
    unstable.firefoxpwa
  ];
  programs.firefox = {
    enable = true;
    package = pkgs.firefox;
    nativeMessagingHosts.packages = with pkgs; [
      browserpass
      unstable.firefoxpwa
    ];
  };

Install firefox extension https://addons.mozilla.org/fr/firefox/addon/pwas-for-firefox/
Try to install any website as a PWA
Get the following error : Failed to install system integration: Failed to store web app icons: Failed to create icon directory: Not a directory (os error 20)
PWA not installed.

Expected behavior
Have the PWA installed.
Notify maintainers
@camillemndn
@pasqui23
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.82, NixOS, 23.11 (Tapir), 23.11.5742.219951b495fc`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos-23.11, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

To solve this issue, I needed to be sure this folder exists : ~/.local/share/icons/hicolor/

However, when I launch it, I face a new issue : No such file or directory (os error 2)

I'll fix the PWA installation so that it creates required directories if they don't already exists in the next release.

Actually, I'm not sure how this error happens. The code already tries to create the icons directory and its parent directories (with create_dir_all) here. Maybe the problem is the lack of permissions to create a directory?


add test for #116 and resolve a deprication warning for dateutil.parser

@Pradhvan

@barrust looks like the work is almost done. Is there anything you want me to update in the PR ?

Nope! This should be everything; wanted to let you see how tests can be implemented. If you see any other issues, please let the group know!

@barrust thanks. Any idea when would the next release be ?

@Pradhvan it will probably be in the next day or so. I have another PR I want to get in for a new feature to close another issue and once that is in, I will likely push out a new version.

Great 👍🏾 looking forward to it. 😄


[SPARK-35273][SQL] CombineFilters support non-deterministic expressions

What changes were proposed in this pull request?
This pr makes CombineFilters support non-deterministic expressions. For example:
spark.sql("CREATE TABLE t1(id INT, dt STRING) using parquet PARTITIONED BY (dt)")
spark.sql("CREATE VIEW v1 AS SELECT * FROM t1 WHERE dt NOT IN ('2020-01-01', '2021-01-01')")
spark.sql("SELECT * FROM v1 WHERE dt = '2021-05-01' AND rand() <= 0.01").explain()

Before this pr:
== Physical Plan ==
*(1) Filter (isnotnull(dt#1) AND ((dt#1 = 2021-05-01) AND (rand(-6723800298719475098) <= 0.01)))
+- *(1) ColumnarToRow
   +- FileScan parquet default.t1[id#0,dt#1] Batched: true, DataFilters: [], Format: Parquet, Location: InMemoryFileIndex(0 paths)[], PartitionFilters: [NOT dt#1 IN (2020-01-01,2021-01-01)], PushedFilters: [], ReadSchema: struct<id:int>

After this pr:
== Physical Plan ==
*(1) Filter (rand(-2400509328955813273) <= 0.01)
+- *(1) ColumnarToRow
   +- FileScan parquet default.t1[id#0,dt#1] Batched: true, DataFilters: [], Format: Parquet, Location: InMemoryFileIndex(0 paths)[], PartitionFilters: [isnotnull(dt#1), NOT dt#1 IN (2020-01-01,2021-01-01), (dt#1 = 2021-05-01)], PushedFilters: [], ReadSchema: struct<id:int>

Why are the changes needed?
Improve query performance.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.

Test build #138102 has started for PR 32405 at commit 9eefb08.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42622/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42622/

Test build #138109 has started for PR 32405 at commit 26a0c07.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42630/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42630/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42630/

Test build #138102 has finished for PR 32405 at commit 9eefb08.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138102/

Test build #138109 has finished for PR 32405 at commit 26a0c07.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138109/

thanks, merging to master!

late lgtm. Thanks you, @wangyum ~


Remove open bounties from being listed in the front end

Remove open bounties from being listed in the front end to avoid clutter. @Icedcool for more details

Complete bounties :)


DBDAART-2532 Update the PR Template

What is this and why are we doing it?
Updating the PR template for simplicity, consistency, thoroughness, and usability.
Moving the readme to the .github folder to be consistent with our other repos.

Link to the Jira ticket for this change: https://jiraent.cms.gov/browse/DBDAART-2532

What are the security implications from this change?
I don't think there are any security implications. I'm just moving these files to a different folder and updating this template.
How did I test this?
I previewed the markdown of the PR template in my IDE.
I ran the PR template update idea past the team: https://dataconnect-workspace.slack.com/archives/C02QJCMNHR8/p1689952645433389
And we have successfully moved the readme to the .github folder in other repos.
Should there be new or updated documentation for this change? (Be specific.)
I don't think so. I checked https://cms-dataconnect.atlassian.net/wiki/spaces/DW/pages/21959908097/DataConnect+Getting+Started+with+Github but nothing needs to be updated to account for this change.
PR Checklist

[x] The JIRA ticket number and a short description is in the subject line
[x] My code follows any applicable style guides
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

I'm cool with this. It's nice, slim, and straight forward!
One question I have do have, now that we're able to run Databricks python code from VS Code using the plugin, I was thinking it's a good time to get a set of unit tests that actually are valuable. Should we keep that unit test line in there?

There is still a whole section in the PR template about testing.
I felt that having the section and a checkbox was redundant and one of my goals was to make the template simpler and cleaner.


Feature okta auth

Description
Adds an array of data for testing authenticated requests from the front end client against the OKTA jwt decoder.
Type of change

Dummy data implementation
RBAC testing (based on claims array from OKTA)

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Incomplete/work-in-progress, PR is for discussion/feedback

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] There are no merge conflicts

I believe using the claims array provided by OKTA is the best way for us to integrate authorization, and lends itself nicely to allowing claims to fall through on client requests until authorized data can be returned to the client.

I agree with Nick's comment that using the claims array provided by OKTA is the best way to integrate authorization.
A sidenote: Should the "oktaJwtVerifier" information in auth-middleware be moved to a .env file? It seems to be your personal OKTA setup that you created over break, but for the future when it's Lambda's OKTA information, it should maybe be kept secret.


Error during Azcopy sync command

Azopy version 10.1.2
Windows
azcopy sync "https://blobname.blob.core.windows.net/pio-train/?token" "C:\test" --recursive=true
C:\Users\prosbcomrel>azcopy sync "https://blobname.blob.core.windows.net/pio-train/?token" "C:\test" --recursive=true
Job 4cbe4169-4c5f-8e4e-5d75-1a439c951250 has started
Log file is located at: C:\Users\prosbcomrel/.azcopy/4cbe4169-4c5f-8e4e-5d75-1a439c951250.log
0 Files Scanned at Source, 0 Files Scanned at Destination
Cannot perform sync due to error: cannot list blobs. Failed with error -> github.com/Azure/azure-pipeline-go/pipeline.NewError, /home/vsts/go/pkg/mod/github.com/!azure/azure-pipeline-go@v0.2.1/pipeline/error.go:154
HTTP request failed
Get"https://blobname.blob.core.windows.net/pio-train/?token"&timeout=901: dial tcp: lookup blobname.blob.core.windows.net: no such host
How can we reproduce the problem in the simplest way?
It was working a week back in my Azure VM. Encountering the error from yesterday. The same command works from another Azure VM.
Have you found a mitigation/solution?
No

Given the nature of the error messages, it looks like either a typo in the account name, or something blocking the outbound traffic from the VM (e.g. an NSG rule).

Just found out that there is some network connectivity issue. Outbound traffic is all fine. But happened all of a sudden and took me off guard.


How do I format the code inside <?php opening tag

Why does the code below not get formatted to line up properly when I try to format it? I am using VS Code, so I right click and choose Format Document.   It will format correctly if I place the code inside { ... }
Here are the versions of the installation:

VS Code Version 1.35.1 (1.35.1) on Mac OS X
PHP 7.3
PHP CS Fixer 2.15.1 Europe Round by Fabien Potencier and Dariusz Ruminski (2006451)
Name: php cs fixer
Id: junstyle.php-cs-fixer
Description: PHP CS Fixer extension for VS Code, php formatter, php code beautify tool, format html
Version: 0.1.119
Publisher: junstyle
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=junstyle.php-cs-fixer

<?php

phpinfo();
$x = 20;
$x = $x + 1;
    echo $x;

        if ($x == 1) {
            echo $x;
        }

      if ($x == 1) {
          echo $x;
      } else {
          echo('nope');
      }

      function foo() {
          echo 1;
      }

Here is my .php_cs
<?php

$finder = PhpCsFixer\Finder::create()
    //->exclude('somedir')
    //->notPath('src/Symfony/Component/Translation/Tests/fixtures/resources.php'
    ->in(__DIR__)
;

return PhpCsFixer\Config::create()
    ->setRules([
        '@PSR2' => true,
        'strict_param' => false,
        'array_syntax' => ['syntax' => 'long'],
        'braces' => [
            'allow_single_line_closure' => true,
            'position_after_functions_and_oop_constructs' => 'same'],
    ])
    ->setFinder($finder)
;

I am adding this as a comment of another user with same issue from Sep 2017.
https://stackoverflow.com/questions/46305564/fix-indentation-of-php-code-files-with-php-cs-fixer/46305930#46305930

Hi and thanks for your question.
Currently we don't have a rule that will remove the extra indentation, sadly because it would be great to have!

This extra indentation now gets properly fixed by the braces rule, I suggest closing this issue.


Supernovas file

This PR creates a supernovas output file with all the SN rates for Types Ia and Ib integrated for all time and mass ranges.

Deprecated PR. Code for nucleosynthesis is different now and SN rates are saved to a different file.


Node.js Client SDK

Must be written in Typescript and live in /sdk/node/
Handles:

/api/ping
/api/list-runtimes
/api/execute

See: https://github.com/teknologi-umum/pesto/blob/master/rce/src/index.ts

Hi @aldy505
So, I need to write a sample code for consuming/handling pesto API?

SDK is not a sample code. SDK is a client library to consume the API.

Hey @aldy505 I can be of help here. I use Fern (www.buildwithfern.com) to auto-generate SDKs, including Node.js.
Here's an SDK I recently built for another organization: https://github.com/fern-primer/primer-node
Let me know if this is of interest for https://github.com/teknologi-umum .

Hi @dannysheridan unless you can show a sample of code gen'd SDK that you're impling, I'm sorry, I'm not really interested on code gen'd SDK. I want to focus to the DX, not just a functional code that calls the API correctly.

@aldy505 here's a TypeScript example app consuming a Node SDK generated by Fern: https://github.com/fern-primer/primer-node/blob/main/.sample-app/app.ts


Setting This Up On HostGator

Does anyone know how to set this up on a HostGator shared server?

Hey!
To setup AB-Web all you need is to locate your html folder (this has many names) and paste in the AB-Web files. After that config your database.php file and your all set!

Is it in your public_html or main web folder?
What is your site for your panel?
And can you show me more of your folder tree?

It is the public_html fold

er. My panel is enderplex.xyz.

This is what happens when you go to enderplex.xyz. Catchable fatal error: Argument 1 passed to AdvancedBan::initialize() must be an instance of string, string given, called in /home2/mechary/public_html/index.php on line 23 and defined in /home2/mechary/public_html/AdvancedBan/AdvancedBan.class.php on line 30

@MechaRy, you would have to ask your host about updating.

So what version do I need to update to?

@MechaRy, the latest stable version of PHP 7 would be recommended.

The latest they have available is 7.1; will that work?

Yes, php7.1 should work fine


harlequin-mysql: init at 0.3.0

Things done
Added harlequin-mysql python package, its basically an Mysql adapter for the Harlequin terminal IDE for sql.
HomePage: https://github.com/tconbeer/harlequin-mysql

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

[x] 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

[x] Tested basic functionality of all binary files (usually in ./result/bin/)

24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

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

@ofborg build python312Packages.harlequin-mysql

Hi @natsukium , Please review again once.

please rebase your commits.

maintainers: add knvpk
harlequin-mysql: init at 0.3.0

https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions


Adding postRunProcessing infrastructure for hanlding Windows Update.

@jhowardmsft @jstarks
This change introduces the infrastructure that we will use to trigger the necessary update steps on an offline container when hcsshim reports there are pending updates.  This will enable running customized Windows Update code inside the container, with the host finishing the update processing (offline DISM calls and reboots if needed) after the container has shut down.
Signed-off-by: Stefan J. Wernli<EMAIL_ADDRESS>

@thaJeztah @jhowardmsft
Everything but the old TP4 is passing on this change.  Does it look good from a review standpoint?

I'll ask if there's someone to review; it's a bit busy due to preparing the next release-candidate for 1.11

LGTM pending CI passing. @tonistiigi - PTAL.

LGTM


Cannot use with Vuetify

I use Vuetify for my project. And when I use tVue-Miltilanguage, it's replace some class and wrap tag of Vuetify. Could you add more feature look like
 {{ tran('keytranslate') }}

Hi @canhkieu,
I no understand your request, this library have the translate method for programmatic use.


IEnumerable for reader?

At least for class 'OsuDb', many osuers have a large 'osu!.db' file which contains a huge amount of beatmaps. I think there is a better way to handle this since current solution will leads to high memory usage and extremely GC call. Comparing to do things after reading the whole file, I prefer to handle things while reading, and not store the previous object after each enumeration (Users can store it manually).

Seems fair. I'll try to get to this soon.


Add invaders laser and spaceship lives

There is a lot of room for improvement.

well, well, well.. look who decided to show up, pretending they haven’t forgotten everything. yea, you guessed it — it’s-a me
i made some changes to your implementation, but i still want to give it some love before merging
it feels good to be back

i think this is it. could you please take a look?

We should use a condition that prevents an invader from firing if there is a living invader below it in the same column.


Refactor .find()

https://github.com/SpoonX/aurelia-orm/issues/119#issuecomment-227165963
Refactor find to accept path and criteria (relative to resource).

@nexbit Hello again :) Looking at the code, I don't think this makes sense. .find() is a public method used to fetch (find) data for a specific resource. If (for instance) you want to get the delegates you have a couple options:
Association:
entity.find('user').then(user => {
  // you can use `user.delegates` if you set up the association in the entity
});

Repository:
class MyRepository extends Repository {
  findDelegates() {
    return this.findPath(`${this.resource}/delegates`);
  }
}

The repository way is a custom implementation. This has been kept open to allow flexibility. Therefor, I'd say no to refactoring find() (which has a very specific purpose).
Something like this could work, but is a breaking change:
findPath(path, criteria, raw) {
  if (path[0] !== '/') {
    path = `${this.resource}/${path}`;
  }
}

This follows the convention of absolute and relative paths. I'd need a vote on this though.

@RWOverdijk I agree with your reasoning, and maybe the findPath refactoring you suggest could be added in the next major version.

just to make sure: you do know that criteria is used as part of the path if it is a number or string only?
so, find(1) translated to eg api/users/1
or findPath('users',1) to api/users/1

@doktordirk Yeah I used that to retrieve single resources directly, but my original issue was with findPath when retrieving a related resource like orders/1/items, as described in the comment I pointed out in my first post.
I know that libraries like sails or loopback are more plain, given that they are auto-generated for the most part, but other hand-crafted apis can make good use of sub-resources (without abusing it, or it will result in a nesting nightmare).

so you.d need a entity.find then?

I think it would be nice to have a repository method similar to .findPath(), but .call(). Same parameters, but it would call an action on top of the root path.
In other words a User repository would call "user/list/1" for .call("list",1).
I really need this for ASPNetCore where the Controller is "user" but it typically takes an action name as well.
call(path:string, criteria:any) {

        let fullPath = this.getResource() + "/" + path;

		return this.findPath(fullPath, criteria);
    }

If it does exactly the same, you can just extend it in your application.

@RWOverdijk Ok.  I actually tried to extend it, but I am missing something in how I set it up.
I Subclassed Repository to AppRespository.
Then then set it as the Repository using the decorator on the Contact Entity Type.
import {AppRepository} from 'repository/app-repository';
But then when I attempted to get an instance:
entityManager.getRepository('contact');
It returned a regular repository, not an AppRepository so I could call my custom methods.
Am I doing something wrong in the wireup?

@gregoryagu Did you specify the @resource on the entity as 'contact'?
Also, it should be possible to overwrite the default repository. Simply register your custom repository as Repository on DI.

Closing this due to inactivity. Feel free to reopen if this question still exists :)

Yes, this was resolved.  Sorry, forgot to update this.  Thanks very much.


feat(test-utils): add types for auto destroy methods

Add types for auto destroy methods, the hook type is typed loosely to be agnostic to testing frameworks.

Related: https://github.com/vuejs/vue-test-utils/pull/1723

Thanks


Project removal suggestion

examples.pyviz.org will move under the HoloViz umbrella. Looking at all the projects, it appears that voila_gpx_viewer does not depend on any HoloViz libraries. I suggest we remove it.
I also suggest the current authors/maintainers of many examples (@jbednar, @jlstevens and @philippjfr) to have a look at your examples and see whether they should be kept as is, updated or removed (using too old APIs, overlapping too much with other projects, ashamed of it :D, etc.). I think it's important as our plan is to have the gallery as a link in the nav bar of holoviz.org, and have holoviz.org linked from every HoloViz website (cc @droumis). As such, the now pretty much unknown to most PyViz gallery will get more exposure, and even if it's an archive of examples users will still draw inspiration from it.

I don't see any examples that should be removed besides voila-gpx that should move to AE5. I assume that nearly all of them need updating for API and library versions.


Leverage exw3 or similar for convenient interface to contracts

There have been some efforts to leverage https://github.com/hswick/exw3, instead of maintaining the Eth.RootChain. Since now the wealth of the contract's APIs is significantly increasing, this maintenance becomes more and more of a hassle.
Let's reconsider this issue. First let's re-assess exw3 and alternatives (if there are any).
Old notes and intermediate results from @pik694 (pasted from pivotal tracker, note this is ~year old):

@pdobacz
https://github.com/hswick/exw3/ might solve this well enough, most likely we shouldn't do more than use this (in case it works well)
@pik694
The decision was made to split this task into two steps.

Rewrite OMG.Eth.RootChain in order to use exw3. This step is a test, whether exw3 offers all the required functionality (ex. indexed fields in events).
Prepare macros, that would use exw3 and would produce all the code during compilation time, basing on contract's abi.

For now, we managed to refactor the OMG.Eth and OMG.Eth.RootChain to a point where we are able to deploy (needed for testing) the contract via exw3 and the contract registers itself to exw3 - having given the address and abi.
Now we are refactoring functions which call the contract - exw3 seems to be doing the work. Refactor of a one call is just a simple exchange of our previous call into a call of exw3 method - pretty straightforward. We managed to refactor 3-4 functions.
Next step is to rewrite all the remaining calls, then adapt exw3 event's handling.
Some clean up is also needed if it is decided to merge it the the main development.
@pdobacz
WIP in branch https://github.com/omisego/elixir-omg/tree/feature/omg-66_macroify_creation_of_contract_proxy_modules

I believe this is taken care of by https://github.com/omgnetwork/ex_plasma


add --show-in-gui and --recommended-vm-size to compute image update

This adds the ability to set show_in_gui and recommended_vm_size for compute images.
This won't work 100% yet because of these bugs:

https://github.com/Azure/azure-sdk-for-python/issues/793 - recommended_vm_size is not populated
https://github.com/Azure/azure-sdk-for-python/issues/794 - show_in_gui updates do not take effect

But, I can use --show-in-gui with a locally patched azure-storage-python package, and the recommended_vm_Size update works... but it will get clobbered on any subsequent updates because of how you've implemented the update logic.
So there's still value in merging this before the python-sdk fixes are implemented/merged.

I'm ok with adding the additional image metadata fields even though the SDK has issues with it, it will self heal once the SDK is fixed
However, your patch is missing the required unit test coverage. Could you add the tests too ? Thanks

FYI @schaefi azure-servicemanagement-legacy 0.20.5 released with the bug fixes related to this one.
https://pypi.python.org/pypi/azure-servicemanagement-legacy/0.20.5

@schaefi Tests added. Not sure if I handled the boolean flag correctly.
Also, not sure if I need to make a backward-compat change for the show_in_gui flag. It's unclear to me if it's going to default to False now, whereas it defaulted to True when being omitted, apparently.

There seems to be still a flake8 issue
From the travis log above
flake8
./azurectl/instance/image.py:270:20: E111 indentation is not a multiple of four

You can prevent travis from complaining if you run the flake8 and unit test targets in your development environment pior to creating the pull request. Also see:

https://github.com/SUSE/azurectl#contributing
https://github.com/SUSE/azurectl#testing

Hope that helps

There seems to be still a flake8 issue
From the travis log above
flake8
./azurectl/instance/image.py:270:20: E111 indentation is not a multiple of four

You can prevent travis from complaining if you run the flake8 and unit test targets in your development environment pior to creating the pull request. Also see:

https://github.com/SUSE/azurectl#contributing
https://github.com/SUSE/azurectl#testing

Hope that helps

Hi @schaefi. Thanks for the pointers and patience.
I've updated the PR and added a new commit that bumps the version. The tests and flake8 are passing, please take a look.

great work, thanks :+1:

Thanks for reviewing and merging! I'm assuming pushing the new version tag is something you do manually? Can you possibly do that as well? Thanks!

Thanks for reviewing and merging! I'm assuming pushing the new version
tag is something you do manually? Can you possibly do that as well?
Thanks!

sure, you will find roling releases here:
http://download.opensuse.org/repositories/Cloud:/Tools

I will make a submission today, after I've finished my current Azure
upload and testing.


Missing "vagrant-setup-vbox-worker.png"

Expected Behaviour
Image showing
Current Behaviour
Image missing

Possible Solution
URL changed?
or, just reupload?
Steps to Reproduce (for bugs)

Visit https://docs.tinkerbell.org/services/osie/

Thanks @felixwidjaja!


please help me with pre fill input

Hello. Can you please explain me ho to prefill username
For example on the https://passport.yandex.ru/auth
Thx!

I guess that so many people got red domains after setting them up. Can someone or dev add block bots into the script?

How can i update evilginx2?
git pull or what ?

How to make work with hidden service onion websites? They don't have any subdomains because of the way onion addresses are made so seems to be no way to make it work for them?


gps-trigger Functions app with geo-fence trigger not working when deployed in the cloud

I am going through the Geofences lesson using a Raspberry Pi. I am using VS Code to edit remotely from a mac. The python version being used in VS Code on the mac is 3.8.2.
I already verified that the Azure function worked fine with just the iot-hub-trigger.  I also verified that both function triggers (iot-hub-trigger, geofence-trigger) were working when I ran the gps-sensor Azure function locally.
In Step 10 of the Task - test the geofence from the trigger section, I have uploaded the new application settings (MAPS_KEY, GEOFENCE_UDID) and re-published the gps-sensor functions app to Azure.  I am trying to verify that both function-triggers are working in Azure, but it looks like they are not  :-(
The iot-hub-trigger looks like it's working (I don't see any errors when I inspect the Monitor blade in the iot-hub-trigger function).  I am able to see new blobs written to my Azure storage account (although it seems to take a bit longer for them to show up in the Azure Storage Explorer).
The geofence-trigger, however, looks like it's not working.  I see error messages when I inspect the Monitor blade in the geofence-trigger function.  See below images for details:

Please advise....

@jimbobbennett , are you still monitoring and responding to these issues?  I hope I did not exceed the quota for number of issues/questions allowed :-)
I have been waiting to hear back from you in case you need me to test/try something else before I destroy these Azure resources.  I'll wait a few more days...

Apologies, I started a new job recently so haven been buried with getting up to speed on that.
The trigger is being run, so it is unlikely to be an issue with the consumer group or trigger settings. If it runs locally, then the settings would be correct.
Can you post your code?

Which code, specifically, do you want me to post?  And in what format (zipped and attached?) ?

That's weird - that library is part of tk, a graphics library.
Can you post the contents of your requirements.txt file and your init.py files please?

@jimbobbennett , Attached are the requirements.txt file, as well as the __init__.py file for the geofence-trigger.   Note: I had to add a .txt extension to the python file in order to convince this github interface to let me attach it.
requirements.txt
init .py.txt
One difference I noticed between my code and the sample code (besides all the comments I added) was that I still had the following import statement.  I don't think this matters though ?
from turtle import distance
I assume you don't need to see the  __init__.py from the iot-hub-trigger, but I'll include it too since the postage is the same :-)
init .py.txt

@jimbobbennett , I think I figured it out.  My code had the following extra import statement:
from turtle import distance
I can't recall how it got in there (scratch_head), but I compared it against the sample code in the code/functions folder and it was not in that file.
Removing that import command from my code fixed the problem.  It works both locally and in Azure now.

Cool! Weird how that got there - maybe an auto added import because you typed distance and the tools got over helpful.
I'll close this now.


JBPM-5283 - Remove actual owner from roles allowed to activate tasks

I have removed unnecessary permission from operations-dsl.mvel.

Please hold off on merging this -- while what Tomas is saying makes sense with regards to the spec, I think that the code-level interpretation of "Actual owner" may not match that (and may in some cases be the initiator?). I'd like to double check that today.

@mrietveld, thanks for looking at this. I ran into it when I was verifying the documentation and after a short discussion with @mswiderski decided to remove it. But I have not checked any internal interpretation of Actual owner so there is a chance it is done this way on purpose and might cause some problems. If this is a wrong step, we should also see it on failing tests.

It seems that there are no related test failures. Failing ConcurrentHumanTaskTest is passing on my machine. It would be nice if Jenkins can retest this.

@livthomas Thanks for the additional info.
+1

thanks guys @livthomas @mrietveld


Clicking on a sentence moves tracker to it

Oh damn this CL turned out to be much cleaner than I expected.
Very nicely done :D
Just that comment about !important, but eh we will be changing the fading animation later on anyway, so im fine if u squash and merge after leaving a one-line explanation so i understand later.


Feature/reservation destroy service

Hi there 👋🏻
Thanks for taking the time to review the project. Here are some things important to note about it:
Now into the project...
Changes implemented in this PR

Created the reservations destroy service, with the controller, interactor and test.
Updated the equipment migration to add the url field.
Update the equipment model to add an after_commit method to save the url for the imgs.
Add the images for the seeds.

Final notes -Partner
Is expected that this project meets the requirements. Please let me know if anything is left to be done or can be improved. Again, thanks for the review.

Hi @jose-Abel
Good job so far! There are some issues that you still need to work on to go to the next project but you are almost there!
There seems to be one test case failing on my end:

Highlights

white_check_mark  Nice implementations. I left some comments

Required Changes
Check the comments under the review.
Optional suggestions
_Every comment with the [OPTIONAL] prefix is just a personal suggestion that I think can make your code better. I'm open to discussing your opinions around them but are not required so we can continue the development of the project. You can also consider:

x
y
z
Cheers and Happy coding!clapclapclap
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

This is a feature teammate review. Remember that everything said here is intended for both of us to learn and grow as Software Developers computer and we may have different approaches or perspectives. That being said, don't hesitate to comment if you disagree with one required change. We can talk everything through cat

Let's setup the Github action to run RSpec here on Github, we're going to become crazy if test run on my end but not on yours. Thanks!