Fixed sql_create_index for mysql

This needs to use "alter table" instead of "create index" due to this 6 year old bug in mysql:
https://bugs.mysql.com/bug.php?id=48875

This will require a ticket and test that makes sure the case of the table name is preserved on index creation. MySQL will never stop to amaze me.

Oops, sorry about the whitespace. This is actually something that has been bugging me for quite a while regarding github. Is there a way to easily fix that mistake I made, or do I need to edit the file again and make a separate pull request?

From the branch name I suppose you used the Web GUI to submit this change. If you want to issue another commit on this branch I think you'll have to check it out locally first.

Duplicated by https://github.com/django/django/pull/6119 (still missing a test though).


ActivityWatch

Project description
ActivityWatch is an app that automatically tracks how you spend time on your devices.
It is open source, privacy-first, cross-platform, and a great alternative to services like RescueTime, ManicTime, and WakaTime.
It can be used to keep track of your productivity, time spent on different projects, bad screen habits, or just to understand how you spend your time.
Metadata

homepage URL: https://activitywatch.net
source URL: https://github.com/ActivityWatch/activitywatch
license: MPL-2.0
platforms: linux, darwin, windows, android

Screenshots

Docs on installing from source: https://docs.activitywatch.net/en/latest/installing-from-source.html

I have an expression for the server in my configs repo. https://github.com/jtojnar/nixfiles/pull/2 Unfortunately, it currently depends on unstable rust so it cannot be packaged in nixpkgs easily.

Made some progress tonight on this with the python server. I can install all modules and run aw-qt.  The flask server responds to requests from the chrome extension, but I get 404s :thinking: Probably some paths that aren't included in the install somehow
Current state: https://github.com/skogsbrus/os/pull/6/files
As for the web ui, I don't think we need to include it? AFAIK it's just the code for the web extensions. The official releases don't even include it.
Copied most of the modules from yours @jtojnar, so really thanks for your work! :pray:

Status update:
Tried adding aw-webui with mkYarnPackage, but have gotten stock on this obscure error (PR above has been updated):
johanan@voidm ~/os/home/pkgs/activitywatch nix-build
these 3 derivations will be built:
  /nix/store/ryj0li9bzcz6ap2k1cl4b2zskv80fbam-aw-webui.drv
  /nix/store/f066h4rcl9jc3xlliw3iqaqk8insgg0x-aw-server-johanan-beta1.drv
  /nix/store/4w12q82p7gfcy4pa06ga3mb27bllmlxf-activitywatch-johanan-beta1.drv
building '/nix/store/ryj0li9bzcz6ap2k1cl4b2zskv80fbam-aw-webui.drv'...
unpacking sources
unpacking source archive /nix/store/a6fjf7fl63qqk7bwssfk8agr50sw09j5-source/aw-server/aw-webui/.
source root is ./
patching sources
configuring
mv: cannot copy a directory, '/build', into itself, '/build/temp'
error: builder for '/nix/store/ryj0li9bzcz6ap2k1cl4b2zskv80fbam-aw-webui.drv' failed with exit code 1;
       last 6 log lines:
       > unpacking sources
       > unpacking source archive /nix/store/a6fjf7fl63qqk7bwssfk8agr50sw09j5-source/aw-server/aw-webui/.
       > source root is ./
       > patching sources
       > configuring
       > mv: cannot copy a directory, '/build', into itself, '/build/temp'
       For full logs, run 'nix log /nix/store/ryj0li9bzcz6ap2k1cl4b2zskv80fbam-aw-webui.drv'.
error: 1 dependencies of derivation '/nix/store/4w12q82p7gfcy4pa06ga3mb27bllmlxf-activitywatch-johanan-beta1.drv' failed to build
johanan@voidm ~/os/home/pkgs/activitywatch

I barely know what I'm doing here so this will take quite a few iterations... :)

@meain helped me solve the above issue. Now I'm just trying to get the static folder that is built by aw-webui to be installed in the same folder as the aw_server site package, since it's looking for a static folder in the same path as server.py.

So, lots of time has past but I've got a local derivation working now. Haven't had the time to look into how to create a service that can be uploaded to nixpkgs, but this works locally with (pkgs.callPackage ./activitywatch {}) (assuming the zip contents exist in ./activitywatch.
Hopefully this is enough for anyone Nix-knowledgeable to package this into something sensible? Otherwise I might be able to do it in a few more months... :)
aw.zip:

default.nix
yarn.lock
yarn.nix

@skogsbrus Thank you for the derivation, perhaps you could use it like this:
{
  home-manager.sharedModules = [
    ({ config, lib, pkgs, ... }:
      with lib;
      let
        cfg = config.programs.activitywatch;
      in
      {
        options.programs.activitywatch = with types; {
          enable = mkEnableOption "Enable activitywatch";
          package = mkOption { type = package; default = pkgs.activitywatch; };
        };

        config = mkMerge [
          (mkIf cfg.enable {
            systemd.user.services.activitywatch = {
              Unit.Description = "activitywatch tray-icon";
              Service.ExecStart = (pkgs.writeShellScript "activitywatch-start" ''
                export PATH=$PATH:${makeBinPath [cfg.package]}
                aw-qt
              '').outPath;
              Service.Restart = "on-failure";
              Install.WantedBy = [ "graphical-session.target" ];
            };
          })
        ];
      }
    )
  ];
}

@voobscout nice, I'll try it out. Would that then be a separate MR to the home-manager repo?

Large release yesterday: https://github.com/ActivityWatch/activitywatch/releases/tag/v0.12.0


Feature/154 request class

Upgrades

new VerifiableCredential and VerifiableCredentialWithType class in ./src/util/ssi/vc.ts
implemented these ./src/util/request.ts to have full Typescript report
implemented these in ./src/services/communication (changed Communication and Messages classes) to have full Typescript support
any data sent by URL is doing using ?payload={base64Urlpayload} structure and base64url type
user.getUsername() is used everywhere and always returns a TonomyUsername type
updated all tests and SDK to use the above

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Tfsec config file doesn't support to define soft-fail

Describe the bug
When we define soft-fail in config file and pass that config file to tfsec cli , it doesn't work.
To Reproduce
Define config like tfsec_config.yml
soft-fail: true  #Runs checks but suppresses error code include-passed: true #Resources that pass checks are included in the result output allow-checks-to-panic: true #Allow panics to propagate up from rule checking verbose: true #Enable verbose logging
tfsec --config-file tfsec_config.yml
Got an error like ##[error]Failed: tfsec detected misconfigurations.
Means soft-fail: true is not working when it's set in config.yml file.
Expected behavior
It should override cli options from config file.
It works for cli option --exclude when we define in config file
exclude:
  - azure-network-no-public-ingress

Ideally it should work for other cli options too.
Output of your tfsec command with --debug flag

System Info

tfsec version: v1.26.0
terraform version: 1.4.6
OS: Linux

Example Code

Additional context

Our efforts have moved to Trivy. Please file a discussion in the Trivy repo if you see it fit.


RG353P Atari ST White Screen & Battery Warning Verbal.Voice HotKeys

After A Few Days Ago I Complaint About R36S And Still Nowhere To Fix Now Today I Brought RG353P And Result Was The Same, And Arkos Was Update And Still Stuck, And There Is No Core In MetaData How Can I Fix It? And What A ShortCut Hot Keys For Verbal Sound Battery Warning? I Try FN+R3 Like RG353V But No Success Already Try FN+Cross, Square, Triangle, L1/2, R1/2, Select Start And Still No Sound, Btw I Used RG351M For Rom In RG353P, Because I Don't See RG353P In Option And I See Someone Use RG351M For This Device

The RG353P units are not supported.  For a list of supported devices, see the main page of the wiki.  For list of hotkeys, check the FAQ on the wiki for a supported device.  RG351M is no longer supported as of 2021.  Did you mean RG353M?  Also, have you tried to update the retroarch cores in Retroarch?  Go to the Retroarch system, launch retroarch, then do core downloader, then update cores.

The RG353P units are not supported. For a list of supported devices, see the main page of the wiki. For list of hotkeys, check the FAQ on the wiki for a supported device. RG351M is no longer supported as of 2021. Did you mean RG353M? Also, have you tried to update the retroarch cores in Retroarch? Go to the Retroarch system, launch retroarch, then do core downloader, then update cores.

Yea Sorry Its RG353M Image, Btw I Found Bout The Battery Warning Its Select+R3 But Atari St Remaining White Screen, Already Up All Cores, Dun Know If Its Because All File Is .IPF Or Something Bug That Never Fix, Kinda Tired Of Searching This Atari St Problem Because My 2 Console Remain The Same, Maybe Will Brought RG552 Hope Better With Android System


Updating gotests binaries

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind bug
/kind cleanup
/kind design
/kind documentation

/kind failing-test

/kind feature
/kind flake

What this PR does / why we need it:
golangci-lint update to work with go 1.22 updated other components as well
This resolves failing tests in https://github.com/nephio-project/nephio/pull/597 and https://github.com/nephio-project/nephio/pull/722
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/approve

/approve


[cudnn] Build error

Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Restored 0 packages from C:\Users\nasreen\AppData\Local\vcpkg\archives in 29.68 ms. Use --debug to see more details.
Starting package 1/32: cudnn:x64-windows
Building package cudnn[core]:x64-windows...
-- Found CUDA V11.6.124
-- CUDNN_INCLUDE_DIR: CUDNN_INCLUDE_DIR-NOTFOUND
-- CUDNN_LIBRARY: CUDNN_LIBRARY-NOTFOUND
CMake Error at ports/cudnn/portfile.cmake:69 (message):
Please download CUDNN from official sources (such as
https://developer.nvidia.com/rdp/cudnn-download ) and extract the zip into
your CUDA_TOOLKIT_ROOT (C:/Program Files/NVIDIA GPU Computing
Toolkit/CUDA/v11.6).  (For example: tar.exe -xvf
cudnn-11.2-windows-x64-v<IP_ADDRESS>.zip --strip 1 --directory "C:/Program
Files/NVIDIA GPU Computing Toolkit/CUDA/v11.6"
Call Stack (most recent call first):
scripts/ports.cmake:145 (include)
error: building cudnn:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with git pull and .\vcpkg update.
package: cudnn[core]:x64-windows -> 7.6.5#4
vcpkg-tool version: 2022-03-30-692785ac944e81417840c6de244fb3e18a4b35eb
vcpkg-scripts version: 27fd32e91 2022-03-31 (13 hours ago)

Thanks for posting this issue. Please download CUDNN as the log:
CMake Error at ports/cudnn/portfile.cmake:69 (message):
Please download CUDNN from official sources (such as
https://developer.nvidia.com/rdp/cudnn-download ) and extract the zip into
your CUDA_TOOLKIT_ROOT (C:/Program Files/NVIDIA GPU Computing
Toolkit/CUDA/v11.6). (For example: tar.exe -xvf
cudnn-11.2-windows-x64-v<IP_ADDRESS>.zip --strip 1 --directory "C:/Program
Files/NVIDIA GPU Computing Toolkit/CUDA/v11.6"

According to the official document, please copy the CUDNN folder to CUDA_PATH first.

@JackBoosY Could you discuss it in more clear detail, I found that my laptop did not support GPU and I need to download vcpkg .. do I need to download vcpkg in the Visual studio code folder or in any folder .. please guide me I need to download darknet to test my model.

@NisreenAlaas Look at this code:
https://github.com/microsoft/vcpkg/blob/e809a42f87565e803b2178a0c11263f462d1800a/ports/cudnn/portfile.cmake#L22
You should unzip the CUDNN package and put it into CUDA_HOME.

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


unicode problem in sqlalchemy interface

I have a problem when I try to store some unicode data with the sqlalchemy interface.
This the error I had.
DataError: (psycopg2.DataError) invalid byte sequence for encoding "UTF8"
I think the type of the column data should be changed to LargeBinary

i had same problem. i changed data field column type to BYTEA and fixed.


Rename classes

Urban should be called built-up
and Rural = Non built up

Commit https://github.com/DynaSlum/SateliteImaging/commit/c806fc606c2a4dd07b02d1e5ff2145a45ec41401


In the Script Manager window the "In T..." and "Status" column headers cannot be resized

Describe the bug
Other column headers can be resized by click-dragging with the mouse, but not the two leftmost columns. I don't even know what the full name of the "In T..." column is.
To Reproduce
Steps to reproduce the behavior:

Go to Script Manager
Hover over separators between the left two pairs of column headers

Expected behavior
The mouse pointer icon should change to the resizing icon and let you click and drag on the separators to move them left and right. As the other separators allow.
Screen shots
https://github.com/user-attachments/assets/40b1dc7c-ca59-41da-86c5-c6723b0274c0
Environment (please complete the following information):

OS: macOS 14.4
Java Version: 23
Ghidra Version: 11.2
Ghidra Origin: official GitHub distro

Additional context
I tried every theme and it makes no difference.

Duplicate of #6918


funky scrolling

the letter containers have nested components
i used overflow: hide on some to prevent ugly outflow, but this makes scrolling / mousewheel get trapped in some of them.
so we'll need to fix that.
hopefully using a canvas render component will make that work #58

#51

this is what i was trying to prevent, tho there are more nested <pre> divs that still freak things out.


[DOCS] ParamValue is undocumented

I wager the reason to be an empty newline between doc comment and class definition in paramlist.h.

On that note: it would really help to have some info about lifetime expectations of ParamValue* types referencing data (vs. owning it; i.e. m_nonlocal==true).
How long does the caller need to ensure the data pointer is valid for the function at hand?


AppCenterTest@1 -  get App center test run id

Hi Team,
is it possible to fetch the Test run id which we get while running this task.

@pramodKarande -- Pramod, thank you for your question, and apologies for the delay in responding.  Please consider these resources, and let me know if they help answer your question:

Azure DevOps Services
Azure DevOps on Stack Overflow

Hi @WilliamAntonRohm
Thank you for responding.
By calling below API call I am able to fetch test runs from App Center
curl -X GET \   -H "X-API-Token: YOUR_API_TOKEN" \   "https://api.appcenter.ms/v0.1/apps/{owner_name}/{app_name}/test_runs"

@pramodKarande -- Pramod, thank you for your followup!
@steved0x -- Steve, please look into the documentation aspects of this issue.

Thank you for your feedback. We are migrating this issue to our new documentation feedback system, and closing this issue.
As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


Image import, line selection and eraser

Feature/enhancement description:

It would be cool and practical to be able to select drawings, import images and erase without erasing a whole line

Closing in favor of #100


403 Error on all request on production

Describe the bug

All requests result in a 403 error since 24.01.2022 13:00h

Expected 204 - ok
Additional context
See  https://github.com/corona-warn-app/cwa-quicktest-onboarding/issues/83
https://github.com/corona-warn-app/cwa-quicktest-onboarding/issues/84

I don't know. But the testresult-server is for quicktests, also, isn't it. I thought the issue should normally placed here.

@jetwes
I think PCR/RAT is not using the same server but I'm not sure.
It's definitely ok to leave this issue open to until the incident is resolved.

Internal Tracking ID: EXPOSUREAPP-11006


Apply doesn't work to user arguments array ?

Hello,
I tried to load multiple css files one by one with a table but the apply doesn't seams to work with a case like thie:
cssUrls = [ "http://../first.css", "http://../second.css" ];
ljs.load.apply(null, cssUrls);

I got this error with Firefox:
TypeError: this.loadcss is not a function
...pply(this,t)}return t.match(/\.css\b/)?this.loadcss(t,e):this.loadjs(t,e)},y={},...
-------------------------------------------^

I have the same kind of error with JavaScript files as urls. Is there an another way to load files one by one from an array ? (v0.1.1)

load takes an array argument. Just use
ljs.load(cssUrls);

Yes but in this case it will load all element in parallel but I want to load one by one in reason of css dependency, so I have to do something like this:
ljs.load(cssUrls[0], cssUrls[1], ..., cssUrls[n], callback);

have you tried a simple ljs.load.apply(ljs, cssUrls);

I copied the whole l.js minified at first line of my JavaScript file, so he will be loaded before its call


Document / demo how to set up drag-and-drop between components outside the tree

It is undoubtedly POSSIBLE to have an element outside the tree component, and be able to drag-and-drop a node to that other component and have it receive and act on it.
Similarly, I assume it is possible to go the other way, as well, if everything is set up correctly.
Request that an example of each of these be added to the documentation.
Thanks so much!  :)

@JohnArrowwood It is not that straightforward :) Cause two dragndrop target and a source should know about each other, for example here is implementation of tree's handleDrop method that handles the event when a node is dropped on the other node.


update match-case to if/else for backward compatibility for Python ve…

…rsions < 3.10

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #
Checklist:

[ ] Docs included if any changes are user facing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 35.431%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4380

Relevant Lines:
12362

💛  - Coveralls

@2020ayao Thank you for creating this PR.
Could you sign the DCO and cla in the following?

https://github.com/kubeflow/training-operator/pull/2099/checks?check_run_id=24535517365
https://github.com/kubeflow/training-operator/pull/2099/checks?check_run_id=24535517401

ping @2020ayao

Thank you for your PR @2020ayao!
Let's discuss it in this issue: https://github.com/kubeflow/training-operator/issues/2096#issuecomment-2129825086
/hold


Enable defining actions bu providing actions programmatically

Is your feature request related to a problem? Please describe.
This is a bit of an edge case but I would like to use custom actions (prerender, postrender, etc) without having to split them out into separate installable package. The reason is that I'm using nexrender-worker programmatically and it would be nice to clearly define an action in the code starting the worker instead of having to rely on installed packages.
Describe the solution you'd like
I have a patch that addresses this that basically adds an actions field to the settings passed to the start function of nexrender-worker which can then be used to associate a particular name with an action function. The way its implemented is also backwards compatible, it will check in the actions object if provided first then if not exist proceed to check the installed packages as usual (including global packages).
Happy to file a PR for this if you think it's good to include, although it will only be useful to whoever is using nexrender-worker programmatically (which is undocumented).

Hey @limzykenneth
Could you show me how does it look like? I don't seem to understand exactly right now, but I think the example will help! :)
Cheers

Sure. In the code starting a worker:
const {start} = require("@nexrender/worker");

const myCustomAction = (job, settings, {input, params}, type) => {
  // Whatever the action needs to do
};

start(process.env.NEXRENDER_HOST, process.env.NEXRENDER_SECRET, {
  workpath: projectsPath,
  actions: {
    "custom-action": myCustomAction
  }
});

This way I don't have to have myCustomAction as a separate package but just include it as a simple function alongside the code I start the worker with.

Oh, that looks awesome! Supporting the idea 150%!

No big hurry on this but I see 1.34.0 is released on GitHub but I'm not seeing it on NPM yet. Both for core and worker

Pushed out an npm update as well (had to mark it 1.34.1 instead).


Pin packages as .dev, unless they specify a version

When a multi-repository project adds a new package with a constraint {= "version"} on an existing one, this will
fail because opam will pin the new package as dev but the old package as a version of it's last release. Making the
bench-dir folder as a dev folder ensures that it's also picked up as a dev version.

Perfect! And the comment is a nice touch! I have no idea why opam made the design decision of not having a version inside the .opam :/


[Feature Request] warning for long assay / study titles

Is your feature request related to a problem? Please describe.
Excel bugs with long (>31 characters) sheet names.
Describe the solution you'd like
Just a quick warning to reduce titles to max. 31 chars and preferentially without specials and spaces.
Additional context
Try
arc s add -s 'This Is a very long Title that Excel does not like'

see Excel error (at least on Mac):
"We found a problem with some content in 'isa.study.xlsx'. Do you want us to try to recover as much as we can? If you trust the source of this workbook, click Yes."

Hey @Brilator, good point! Besides this length constraint, also some other constraints like characters that clash with the different OS path names need to be blocked for some arguments. E.g. for windows:
@muehlhaus, @chgarth, @omaus, maybe we could collect similar constraints here in this issue?

  (Space) is difficult on Linux and should be avoided (though it is not forbidden). Maybe replacing with _ when OS is Linux?
Only / and NUL are forbidden in Linux. Don't know for Mac. @Brilator ?


[oracle_user] state="empty" isn't working with materialized view

On module oracle_user, when using state="empty" on a schema with materialized views, the module is executing the query drop table to drop these views.
The query has to be drop materialized view.

there is no deletion of the triggers when  using state="empty" on a schema.

Fix in version 1.2.1.
https://github.com/ari-stark/ansible-oracle-modules/commit/e65321e671cf5155c0b86a17ee658c030c32b944


Windows 7 Support

I know its probably a long shot, but about 1/3 of steam is still on Windows 7. Any chance for support?

Unfortunately, the Universal Windows Platform (UWP) apps are not officially supported by Microsoft. This projects depends on key APIs that are only supported by UWP, and hence the requirement for Win8/10.
I (or someone) else may be able to create something for Win7, but it seems like there isn't an easy of way doing it in C# (or any other language for that matter).


Add helpers for easier backup restoring

I like this role, but when playing with it I noticed that if you actually want to restore a backup, manage snapshots or anything else, you'll have to hunt down the environment variables in the cronjob files. If there's anything I learned, it's that when you need backups there's generally a sense of panic, so making them easier to reach seemed like a helpful addition.
This pull request adds some helper scripts to restic_user's home directory in their $PATH, with the same naming scheme as the cronjobs. This means that if you have a restic repo that you named "testrepo", you could easily restore backups of it by su'ing into the restic_user and running restic-testrepo commands (provided you set a restic_user and changed the restic_shell to something usable).

I like it!
Since we are already restricting access to those scripts, maybe it will be better to put them in {{ restic_install_path }} not in {{ restic_home }}/bin. This way we don't need to create additional directory and manipulate $PATH.
It would be also nice to document usage of this helper script.

Makes sense to me. I also hope I hereby added sufficient documentation, or is there anything you feel is still missing?

Okay, after working on this a bit more after seeing #34 I hereby consider this feature done, it works quite reliably from my testing too.

Great job! Thank you! :+1:


Adding alphabetic sorting to generatejson.py output

Just adding a sort after os.walk.

saw the slack convo. I don't use the tool, but maybe one day I will when it has all of the features. Thanks for fixing.


Toggle file doesn't update list, deleted file marked as (empty)

When toggle file, list doesn't update itself.
Example:

toggle file (state: added, list[filename])
toggle another file (state: added, list[filename, filename2])
toggle this file again (state: removed, list[filename, (empty)]

I understand the problem with implementation such feature, my goal is point out this issue

interesting.  What are the apis you are calling?

marks api

Same here, I added this to my harpoon.lua (follow your video) and got the same error:
vim.keymap.set("n", "<leader>r", mark.rm_file)

I had to manually remove it from .local/share/nvim/harpoon.json


Made static editor read-only

Changes Unknown when pulling ab08c5ff2b5d286aac78cd5950e21dda1d3a8379 on lramirez:readonlyeditor into ** on tdd-pingis:master**.

Nice


Stratified splitter

integrated StratifiedSplitter from datastack

Codecov Report

Merging #6 (e56403c) into master (8c5e253) will increase coverage by 0.01%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   57.14%   57.16%   +0.01%
==========================================
  Files          44       44
  Lines        2329     2332       +3
==========================================
+ Hits         1331     1333       +2
- Misses        998      999       +1

Impacted Files
Coverage Δ

src/ml_gym/data_handling/postprocessors/factory.py
89.85% <66.66%> (-1.19%)
:arrow_down:

src/ml_gym/blueprints/constructables.py
77.53% <100.00%> (+0.08%)
:arrow_up:

Continue to review full report at Codecov.

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


Run sky:init before creating a snapshot

If we've never run sky:init before, the mojom package will be empty and the
snapshot will fail.

/cc @eseidelGoogle

Lgtm
On Monday, July 27, 2015, Adam Barth<EMAIL_ADDRESS>wrote:

/cc @eseidelGoogle https://github.com/eseidelGoogle
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/pull/311#issuecomment-125401291.


Test.

Codecov Report

Merging #6 (6c502b6) into master (0f6eb80) will decrease coverage by 5.27%.
The diff coverage is 96.66%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   94.68%   89.40%   -5.28%
==========================================
  Files          55       55
  Lines       11718    11638      -80
==========================================
- Hits        11095    10405     -690
- Misses        623     1233     +610

Impacted Files
Coverage Δ

databricks/koalas/indexes/base.py
97.09% <93.33%> (-0.32%)
:arrow_down:

databricks/koalas/base.py
93.46% <100.00%> (-4.07%)
:arrow_down:

databricks/koalas/indexes/datetimes.py
92.85% <100.00%> (+1.55%)
:arrow_up:

databricks/koalas/indexes/numeric.py
92.59% <100.00%> (+3.70%)
:arrow_up:

databricks/koalas/plot/plotly.py
9.47% <0.00%> (-87.37%)
:arrow_down:

...bricks/koalas/tests/plot/test_frame_plot_plotly.py
23.33% <0.00%> (-76.67%)
:arrow_down:

...ricks/koalas/tests/plot/test_series_plot_plotly.py
26.92% <0.00%> (-71.26%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
27.58% <0.00%> (-64.92%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
47.82% <0.00%> (-52.18%)
:arrow_down:

databricks/koalas/typedef/typehints.py
65.92% <0.00%> (-28.50%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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


Cannot read property 'dropBufferSupport' of undefined

Hi,
unfortunately I come across this error using ioredis in typescript:
TypeError: Cannot read property 'dropBufferSupport' of undefined     at RedisManager.set (/Users/x/Documents/Github/x/backend/node_modules/ioredis/built/commander.js:110:26)
I use a standard setup and a simple test
this.client = new Redis({   host: Config.redis.host,  port: Config.redis.port,                      dropBufferSupport: false                 });
.....
 this.client.on('ready', async () => {             Logger.info("Redis connected and healthy.");               try {                 await this.set('Test', 'X');                  console.log(await this.get('Test'));                 this.health = true;              } catch(err){                  console.error(err);             }         });
Node-Version: 13.10.1
Redis-Version: 5.0.3
ioredis: 4.16.3
Help would be highly appreciated :-)

It seems to be a compatibility issue. Does simply removing node_modules folder and running npm i again solve the issue? Also, can you upgrade ioredis as the version is dated.

Hi,
thank you for your reply!
I had a typo in my version and fixed the description.
Unfortunately, deleting the node_modules folder and reinstalling it did not help.
I also manually tried to enable / disable the dropBufferSupport but the same error occurred.
Do you have other ideas?

I am also facing this issue. @JannikSt Were you able to figure it out?

@JannikSt, From your example I would assume it's a problem with the way you reference ioredis and use functions - when you use an arrow function, this context is retained from where you defined the this.client.on call. Try using a plain function over an arrow function, those can be bound and called with context supplied to them. Otherwise, a more elaborate example would help.
I encountered this problem when trying to build a module interface in a file and just export the set and get functions from the module:
// wrong.js
const cache = new Redis(config);
module.exports = {
  set: cache.set,
  get: cache.get,
};

// working.js
const cache = new Redis(config);
module.exports = {
  set: async (key, value, ttl=30) => cache.set(key, value, 'EX', ttl),
  get: async (key) => cache.get(key),
};

// alternative.js
const cache = new Redis(config);
module.exports = {
  set: cache.set.bind(cache),
  get: cache.get.bind(cache),
};

@alesbolka Thank you! This got me too...

@alesbolka Thank you! This got me too...


Allow updating a shallow-cloned repo

Before this patch, if the repo you were updating was shallow, Copier failed to update it.
Now it should update any repo, no matter its depth. We only need to fetch 1 commit from it, after all.

Codecov Report

Merging #304 (a1b3a6a) into master (ef7ddd2) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #304      +/-   ##
==========================================
- Coverage   97.48%   97.41%   -0.08%
==========================================
  Files          39       39
  Lines        2028     2010      -18
==========================================
- Hits         1977     1958      -19
- Misses         51       52       +1

Flag
Coverage Δ

unittests
97.41% <100.00%> (-0.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

copier/main.py
97.70% <100.00%> (ø)

tests/test_updatediff.py
100.00% <100.00%> (ø)

tests/conftest.py
85.71% <0.00%> (-14.29%)
:arrow_down:

copier/cli.py
89.87% <0.00%> (-0.13%)
:arrow_down:

tests/test_config.py
98.19% <0.00%> (-0.08%)
:arrow_down:

copier/config/user_data.py
95.98% <0.00%> (-0.04%)
:arrow_down:

tests/test_copy.py
99.22% <0.00%> (-0.01%)
:arrow_down:

tests/test_tools.py
100.00% <0.00%> (ø)

tests/test_prompt.py
100.00% <0.00%> (ø)

copier/config/objects.py
100.00% <0.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


Predicting the Optimal Harvest Time for Wine Grapes

I would like to work and develop a machine learning model that can predict the optimal harvest time for wine grapes based on historical climate data, soil type, and grape variety.

hi @ramana2074 create a new api using flask for that..
1.include ipynb files in notebook folder

can you please let me know how much time do you need to complete this issue @ramana2074

having a bit trouble with dataset. 2 days may be.

Are you adding frontend and api...or just simply adding notebook files... @ramana2074


fix: clean up npm publish to only include lib, README.md, LICENSE

Coverage remained the same at 93.407% when pulling f64e6acea4743d268b147ea408d0363abf21d4e9 on fix/npm-publish-clean into c6d64510fa65703bff6f90651c80debfb4c30842 on master.


Cover states wrong for Velbus covers (always open) in Home Assistant

Velbus covers always have the state "open". When closing a cover, the state changes to "closed" while closing, but it reverts to "open" when the covers are fully closed. The position is correct though between 0 and 100.
I have 5 Velbus "cover" modules of the same type "VMB2BLE", with Velbus build "1935"
You can see the behavior in the following screenshots.

I updated the Velbus "VMB2BLE" modules to the latest build "2205", but get the same behavior.
I'll see if I can get some logs, I'm using Velserv at the moment.

@Cereal2nd , I had the chance to get the Velbuslink logs. See attachment:
I've only opened Velbuslink, did a quick connect and a sync and then closed a cover. I waited until it was fully closed and done. Afterwards I opened the same cover again.
Could you check this out and make anything of it?
COVER.vlp.2022-09-22_23h29m34s.log

@Cereal2nd , I checked the blinds on my system and I think I know what is going wrong:
The blind status message indicates when the blinds start moving and when the moving stops. It indicates if it is moving up (0x1) or down (0x2), when it stops the status indicates stopped (0x0). This is incorrectly interpreted by the functions is_up and is_down in the file blind_status.py. The position could help here to interpret a correct status. When motion has stopped, the position being 0 should return open, when position is 100, it is closed.
But how should HA interpret the status of blinds when they are for example at 50% ? Should the status then be open or closed ?

@DannyDeGaspari , @Cereal2nd :
Great find, sounds logical to me indeed what is happening and why it jumps back to “open” when the blinds are fully closed after the full motion.
Checking the position to determine whether the blind is closed or open, seems a viable solution.
In my opinion, the status can always be “open” unless it is fully closed. When the blinds are open for only a little bit, in theory these have been opened and thus are “open”.
Unless someone has a valid point obviously to do it differently?

this can indeed be a solution, but not all cover modules have the position value available

logs from my installation with a VMB2BLE build 1935 on address 04

when fully closing the screen on channel 2:

when fully opening the screen on channel 2:

when closing the screen and halting it after a few seconds, causing it to report being closed at 12%:

when closing the screen further and halting it again, causing it to report being closed at 22%:

velbuslink_covers.log

Suggestion: In https://github.com/Cereal2nd/velbus-aio/blob/fc95dcd09d032f248f0f64838c5ec0e5ec66918a/velbusaio/messages/blind_status.py#L52, I don't think you should look to self.status to determine is_up and is_down, but rather look to self.position:
    def is_up(self):
        """
        :return: bool
        """
        return self.position == 0

    def is_down(self):
        """
        :return: bool
        """
        return self.position == 100

It's not clear to me how/if these functions are used? So I'm refraining from making a PR myself for now.

Feel free to make the changes and open a pullrequest :)


Better support for conda environment

I got an error when trying to install the conda environment:
$ conda env create -f data/env/py3torch3cuda9.yml
Collecting package metadata (repodata.json): done
Solving environment: failed

ResolvePackageNotFound:
  - numpy==1.14.3=py36h14a74c5_0

I see in the README that I'm supposed to remove the hash, which did work for me. It's a little bit annoying to have to do this manual step to install the toolkit though. I would suggest the following step:

Move the conda environment files into the github repository, instead of in the zip.
Have the default one be a somewhat more relaxed one that works in many environments, but keep the strict one around for reproducibility of the numbers in the paper.

If this sounds good, I can send a PR, what do you think?

Should be fixed by #15


Validate VSphere user privileges during upgrade

Issue #, if available:
#5757
Description of changes:
Adding the method to validate Vsphere privs to the upgrade flow.
Also moving the method to validator.go
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


the code question

C in the formula for the aggregated feature G is denoted by 'mask' in the code, why is it not denoted by 'sim' at the beginning.

Yes, I also have the same question. The 'mask' in the code should be 'sim'. Maybe the code makes a mistake here.

@YangTracy @woaixueyingying Thanks very much for the detailed question. I will double-check it and reply by this week.

您发给我的邮件我已经收到了，谢谢！

I also found this problem, so what exactly is happening and what effect would replacing c with the sim in eq cause?

@YangTracy @Volodymyr233 @woaixueyingying
Thanks for this issue!
Double-check with co-authors, this is a small bug, no influence to our conclusions and contributions.
We re-trained coc-tiny on ImageNet1k with same settings and achieved 72.2, an improvement of 0.4 (reported 71.8).
The slight improvement can be contributed to the soft/weighted summation, instead of mask (cn be considered as a hard version of C in Eq. 1).
We will updated all checkpoints once we finished re-training all models. Again, thanks for your kind reminder.


how syslog should be configured to receive data from fluentd

Hi,
I use https://github.com/fluent/fluentd-kubernetes-daemonset/blob/master/fluentd-daemonset-syslog.yaml with v.1.3
In the syslog-ng all I see is
Aug  5 09:23:40 syslog-7cb88f68dd-5svn8 syslog-ng[1]: Invalid frame header; header=''
Aug  5 09:23:40 syslog-7cb88f68dd-5svn8 syslog-ng[1]: Syslog connection closed; fd='21', client='AF_INET(<IP_ADDRESS>:45090)', local='AF_INET(<IP_ADDRESS>:601)'
I was trying to configure JSON parser but without luck.
Can anyone of you share the Syslog configuration?

@okkez Do you have this problem on your test?
From the error message, syslog-ng assumes octet_counting frame by default?

I'm not sure...
@cosmo0920 Could you take a look?


Write React.js documentation for Components section

In the second version of documentation tabs, we will be writing documentation for React components.
This card includes writing content for all pages that currently implement HTML and Design tabs.

[ ] Add React tabs to pages
[ ] Write documentation
[ ] Codeblocks
[ ] Translations

Broken into smaller cards and added to the React components Milestone


Check devDependencies when development flag is active

Before the onlyDirectDependenciesFilter is initialized for the
devDependencies, the packageJsonContents.devDependencies needs to be
checked instead of the packageJsonContents.dependencies, otherwise
if there are not any dependencies in the package json, the output
will be empty.

Good spot! Thank you very much for your contribution!

I still get empty list, am I missing something?


Support Inet database field types

Seems people want this.  See #1146 and #1115.  Also, this ActiveAdmin issue.

I believe this is not the correct approach to solve this. There is no reason for this class, as it has no implementation.
The problem is that some code defaults to column.type, which is not correct in most cases.
https://github.com/justinfrench/formtastic/blob/2f8b987ee4880c4aa06e6ec799eafe48fa46e316/lib/formtastic/helpers/input_helper.rb#L291
We would have to define some mapping, default or something. But defining a class for every column type we want to alias is not the way to go.

Are you suggesting to add an inet switch like this?
when :inet
  return :string

Yes. And appropriate test that verifies what it does.

@mikz - simple enough.  Sorry about that empty commit, let me know if you want me to remove https://github.com/justinfrench/formtastic/pull/1276/commits/7a3649e04728ac6864cebd2b373f01a38d057bd6

@mikz - I am not sure whats going on with Travis but it appears the build is broken.  My change didn't cause it.  I can run RSpec and get 100% passing on my local system.  Ruby MRI 2.6.1 w/ RVM.


[WFCORE-1713]: Migrate ModelControllerMBeanTestCase from wildfly test suite to core test suite.

Migrating the test from full using ServiceActivator.
Jira:https://issues.jboss.org/browse/WFCORE-1713

Windows Build 3594 outcome was FAILURE using a merge of fde276e1829ee325e9e1242cf902532bbf58e54f
Summary: Tests failed: 1 (1 new), passed: 3767, ignored: 56 Build time: 00:59:33
Failed tests
org.jboss.as.test.integration.management.cli.CliAliasTestCase.testAliasPersistence: java.lang.AssertionError: Alias was not saved to .aesh_aliases
	at org.jboss.as.test.integration.management.cli.CliAliasTestCase.testAliasPersistence(CliAliasTestCase.java:195)


cloudinary view images

Add costume photos from cloudinary
changed Costume model to one photo
changed Costume constroller params to photos


Pass req object to .add() callback

Pass Node req object to the callback of .add(). This will come in handy when dealing with multiple-domain app.

Hey, thanks, this is a very useful addition, I'm sure a lot of people will appreciate it.   Thanks also for the example usage in the README.  Released in v0.0.25 :D


Move default branch from "master" -> "main"

@jrbourbeau and I are in the process of moving the default branch for this repo from master to main.

[x] Changed in GitHub
[x] Merged PR to change branch name in code (xref #487)

What you'll see
Once the name on github is changed (the first box above is Xed, or this issue closed), when you try to git pull you'll get
Your configuration specifies to merge with the ref 'refs/heads/master'
from the remote, but no such ref was fetched.

What you need to do
First: head to your fork and rename the default branch there
Then:
git branch -m master main
git fetch origin
git branch -u origin/main main

Thanks a lot for working on this! Let me know if there is something you expect me to do, e.g. follow https://github.com/github/renaming#renaming-existing-branches to do the master -> main renaming via the Github web UI.

I renamed the default branch through the web UI and merged #487.
cc @dask/dask-jobqueue for more visibility about the master -> main renaming.
Thanks @jsignell!

Thanks @lesteve!


Show Default Text Buttons only if show*Button is true

https://github.com/Jacse/react-native-app-intro-slider/issues/104

@Jacse

As mentioned in #104 I believe this is already the case.


Vundle support

Any hope of seeing this plugin support the Vundle installation method?

I use Vundle for Vim and vim-plug for NeoVim, and both work fine.
Do you just mean that I should mention it in the docs? In which case you're probably right.

I had just tried adding it with Vundle under Macvim, and it didn't seem to work. I thought it would have worked. I'll try it again tomorrow and see if I missed anything.
On March 29, 2017 1:08:01 AM CDT, Nate Kane<EMAIL_ADDRESS>wrote:

I use Vundle for Vim and vim-plug for NeoVim, and both work fine.
Do you just mean that I should mention it in the docs? In which case
you're probably right.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/nathanaelkane/vim-indent-guides/issues/123#issuecomment-289992317

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

The documentation has been updated with Vundle installation details.


Crashes due to Date bug in NuGet

Due to this bug in NuGet: https://nuget.codeplex.com/workitem/4412
Paket crashes like this:
Something went wrong with the download of Newtonsoft.Json.FSharp 3.2.2 - automatic retry with --force.
loaded details for<EMAIL_ADDRESS>from url 'https://www.myget.org/F...b/api/v2'
Downloading Newtonsoft.Json.FSharp 3.2.2 to /Users/henrik.feldt/.local/share/NuGet/Cache/Newtonsoft.Json.FSharp.3.2.2.nupkg
Paket failed with:
	Argument is out of range.
StackTrace:
    at System.DateTime.op_Subtraction (DateTime d, TimeSpan t) [0x00000] in <filename unknown>:0
  at System.DateTimeOffset.get_UtcDateTime () [0x00000] in <filename unknown>:0
  at System.DateTimeOffset..ctor (DateTime dateTime) [0x00000] in <filename unknown>:0
  at System.DateTimeOffset.op_Implicit (DateTime dateTime) [0x00000] in <filename unknown>:0
  at System.IO.Compression.ZipArchiveEntry.get_LastWriteTime () [0x00000] in <filename unknown>:0
  at System.IO.Compression.ZipFileExtensions.ExtractToFile (System.IO.Compression.ZipArchiveEntry source, System.String destinationFileName, Boolean overwrite) [0x00000] in <filename unknown>:0
  at System.IO.Compression.ZipFileExtensions.ExtractToDirectory (System.IO.Compression.ZipArchive source, System.String destinationDirectoryName) [0x00000] in <filename unknown>:0
  at System.IO.Compression.ZipFile.ExtractToDirectory (System.String sourceArchiveFileName, System.String destinationDirectoryName, System.Text.Encoding entryNameEncoding) [0x00000] in <filename unknown>:0
  at System.IO.Compression.ZipFile.ExtractToDirectory (System.String sourceArchiveFileName, System.String destinationDirectoryName) [0x00000] in <filename unknown>:0
  at<EMAIL_ADDRESS>(Microsoft.FSharp.Core.Unit unitVar) [0x00000] in <filename unknown>:0
  at Microsoft.FSharp.Control.AsyncBuilderImpl+callA@803[System.String,Microsoft.FSharp.Core.Unit].Invoke (Microsoft.FSharp.Control.AsyncParams`1 args) [0x00000] in <filename unknown>:0

Because this is the output on the files in the nupkg:
✗ l
total 184
-rw-r--r--@ 1 henrik.feldt<PHONE_NUMBER>  78848 Jan 14  2007 Newtonsoft.Json.FSharp.dll
-rw-r--r--@ 1 henrik.feldt<PHONE_NUMBER>   9375 Jan 15  2007 Newtonsoft.Json.FSharp.xml

...which is wrong, obviously. I haven't been able to solicit a response from any NuGet person. They must be busy adding more powershell commands to visual studio instead of fixing the core.
I can work around this now, by using Paket pack. Leaving this bug report here though. If nothing else, please help me vote for that bug! =)

I don't see a way to work around this. It throws deep down in .NET framework. Any ideas?
/cc @yishaigalatzer @csharpfritz @martenba please look at this issue. Thanks.

Looking at https://github.com/SuaveIO/suave/issues/190 (followed the link to a link :-)) it seems Mono related / related to System.IO.Packaging which doesn't seem to properly capture file timestamps.

@haf do you see the same behavior with latest paket pack? We changed our zip library and now use the default .NET 4.5 stuff.

The thing is; this wasn't paket pack, it was nuget pack.

yes I understand that. And you said paket pack worked. I just want to make
sure we don't have a regression after the change to .NET 4.5
2015-04-09 9:46 GMT+02:00 Henrik Feldt<EMAIL_ADDRESS>
The thing is; this wasn't paket pack, it was nuget pack.
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/757#issuecomment-91141517.

Let me check if paket pack works.

It turns out that now that I pushed a new version of the package it's not erroring out anymore, despite continued use of NuGet.exe (I didn't finished the WIP yesterday to use paket fully).
I think that my upload was corrupted actually: https://twitter.com/nuget/status/585894933178945537 -- as I was uploading exactly before they announced their problems.

seems to be fixed in https://github.com/fsprojects/Paket/issues/761

:heart_decoration:


Invalid value 'https://someurl' of type 'NSURL' for string

Goals
What do you want to achieve? -->String
Expected Results
What did you expect to happen? -->
Achieved String
Actual Results
 Throw in RLMUtil.mm on line 180
@throw RLMException(@"Invalid value '%@' of type '%@' for '%@%s'%s property '%<EMAIL_ADDRESS>                        obj, [obj class],                         prop.objectClassName ?: RLMTypeToString(prop.type), prop.optional ? "?" : "",                         prop.array ? " array" : "", objectSchema.className, prop.name);
Nothing printing because it is throwing on this line, but in the debugger I can reproduce the message and see the following:
"Invalid value 'https://assets.myurl/file.png?1510040391' of type 'NSURL' for 'string' 'MyObject' property 'imageUrl'"
Steps to Reproduce
Try to achieve string that has real URL value.
Code Sample
@objcMembers
class MyObject: Object, Decodable{
enum CodingKeys: String, CodingKey{
case imageUrl
}
@objc dynamic private(set) var imageUrl: String!
convenience required init(from decoder: Decoder) throws{
let container = try decoder.container(keyedBy: CodingKeys.self)
self.init()
self.imageUrl = try? container.decode(String.self, forKey: .imageUrl)
var realm = try? Realm()
do {
try realm?.write {
realm?.add(self, update: true)
 }
} catch let error as NSError {
assertionFailure(error.localizedDescription)
}
}`
Version of Realm and Tooling
Realm framework version: 3.11.0
Realm Object Server version:  No Realm server
Xcode version: Version 10.1 (10B61)
iOS/OSX version: 11.0.1

@adamofsky did you solve this?


Add google cloud configuration object for kubeclient

Kubeclient doesn't support GCE authentication and there is currently no plan to merge the PR that would add support for it. To solve our problem I've added a small wrapper class based off the kubeclient PR. It provides a subclass of the Kubeclient::Config which will look for the approriate auth type and if found will fetch a new (1 hour lifetime) token.
Kubeclient GCE PR
@jeremywadsack, since you were the author of the original PR you may be interested in this.

I've addressed the review comments the configuration is now called GoogleFriendlyConfig.


[docs] Mention that proxy works only for pages served over HTTP (not HTTPS and not HTTP/2)

I guess we need to add a note to the Run tests guide as well as to the three reference topics.

This issue persists until we do https://github.com/DevExpress/testcafe-hammerhead/issues/2668


Implement sizeof for some bytes-like objects

As it appears we don't actually support sizeof for bytes-like types on PyPy, but they are easy to implement and commonly encountered, go ahead and implement custom sizeof for these bytes-like types.

[x] Tests added / passed
[x] Passes black dask / flake8 dask

+1

@jakirkham -- can you rebase against / merge from master so we can pick up the parquet test fixes?

Thanks for the ping. Yep, merged with master. 🙂

Looks like CI is now happy 💚


Please restore the /upgrade config option of wslconfig.

Lastest wsl update removed support for the Volfs leaving only DrvFs in wsl1. And before that, the options for upgrading from Volfs was removed because it was considered buggy.
Please at least restore the possibility to upgrade instead of leaving all my work unreadable.

Very sparse on details here - what versions are you referring to here?

@benhillis the latest I installed through Windows update last Friday on my Windows 10 insider preview build.
After that, every files appeared as missing in the legacy install path. Also, as this is a bug, please remove the feature label.
wsl --update was repurposed and wslconfig complains the /update flag is no longer recognized.

@benhillis but please add the bug label.


Unable to build OVA image; failed attempt to change modification time.

Issue description
When trying to build an OVA image on nixos-17.09, the procedure seems to fail, when certain packages are in the closure. The error is repeatable on my system and issues the following error:

copying closure closure to /tmp/nix-build-nixos-ova-17.09.git.d16ffa1-x86_64-linux.drv-0/root...
...
warning: the group 'root' specified in 'build-users-group' does not exist
error: changing modification time of '/nix/store/cp3cxn5rgk56l9wi21d2byd3s9jpidna-python3.6-msgpack-python-0.4.7/lib/python3.6/site-packages/msgpack/__pycache__': Operation not permitted
builder for ‘/nix/store/dn82qprrhvxgad5z12jlfgyn4hw3d9vq-nixos-ova-17.09.git.d16ffa1-x86_64-linux.drv’ failed with exit code 1
error: build of ‘/nix/store/dn82qprrhvxgad5z12jlfgyn4hw3d9vq-nixos-ova-17.09.git.d16ffa1-x86_64-linux.drv’ failed

This might be related to #28768, but I understand the issue not enough to provide a fix.
Steps to reproduce
Copy the following code to reproduce-ova-fail.nix:
{ nixpkgs ? <nixpkgs> }:

let
  pkgs = import nixpkgs {};

  imageModule = {
    imports = [ "${nixpkgs}/nixos/modules/virtualisation/virtualbox-image.nix" ];
    virtualbox.baseImageSize = 30 * 1024;
    environment.systemPackages = with pkgs;[
      python36Packages.msgpack
    ];
  };

in
{
  virtualbox-image = ((import "${nixpkgs}/nixos/lib/eval-config.nix" {
    system = "x86_64-linux";
    modules = [
      imageModule
    ];
  }).config.system.build.virtualBoxOVA);
}

Then build it with
nix-build reproduce-ova-fail.nix -I nixpkgs=/path/to/nixpkgs/on/nixos-17.09

This should result in the error message showed above.
Technical details

system: "x86_64-linux"
host os: Linux 4.9.86, NixOS, 17.09.git.8b54eb7 (Hummingbird)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 1.11.16
nixpkgs: /home/justin/nixpkgs

What I should have mentioned: Not having python36Packages.msgpack in the closure leads to a successfull build.

I am also seeing this, though not using msgpack. I do have something in environment.systemPackages though, so I wonder if that's part of the problem.

I can't reproduce the issue anymore on nixos-18.09 as well as on nixos-unstable.


Added dynamo write, Removed endpoint when on AWS

Hi there,
Feel free to take or leave these changes, but this is what I needed to get it to run in AWS for me.

Give serverless DynamoDB access
babel polyfill line was giving me an error
You can't pass in an empty string for endpoint to the dynamoDBCache when in AWS

Fixes #1


Minimum text size: Ignore invisible elements

e.g. noscript tags
Ref #74 #75

Text nodes that are not visible do not have a font size.  Some of these may be within a  tag, or within a hidden menu, only shown at a particular screen width, or otherwise not currently visible in a page.


Anoxon/st provision fix

Description

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

A very descriptive PR


[wip] Better documentation for all supported libraries

This PR should fix #125. TODO:

[x] scikit-learn
[ ] lightning
[ ] xgboost
[ ] sklearn-crfsuite
[ ] add links to tutorials from docs for all supported libraries
[ ] make sure it is documented how is explain_weights / explain_prediction dispatched
[ ] add links to "Supported Libraries" section from other docs if appropriate

Current coverage is 97.29% (diff: 100%)

Merging #134 into master will not change coverage

@@             master       #134   diff @@
==========================================
  Files            37         37
  Lines          2179       2179
  Methods           0          0
  Messages          0          0
  Branches        420        420
==========================================
  Hits           2120       2120
  Misses           29         29
  Partials         30         30

Powered by Codecov. Last update 43458b9...29a0e95

@lopuhin this is ready for your review :)


Read catalog and WCS coordiantes.  Fix bug in PSF and compare to astrometry.net.

Hey Jeff,
I'd like to merge this with master and get your sign-off before diving into moving the rest of Celeste over to world coordinates.
Ryan

Looks good, though the unit tests don't pass on Travis. The error message is
ERROR: `read_header` has no method matching read_header(::ImageHDU, ::Type{ASCIIString})
 in fetch_image at /home/travis/.julia/v0.3/Celeste/src/SDSS.jl:25

Hmm, I made a change to FITSIO.jl to implement that function.  It's been
merged, but maybe Travis hasn't picked it up yet.
On Wed, Jun 24, 2015 at 6:01 PM, Jeffrey Regier<EMAIL_ADDRESS>wrote:

Looks good, though the unit tests don't pass on Travis. The error message
is
ERROR: read_header has no method matching read_header(::ImageHDU, ::Type{ASCIIString})
in fetch_image at /home/travis/.julia/v0.3/Celeste/src/SDSS.jl:25
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/38#issuecomment-115058824
.

Indeed, the Travis CI is using FITSIO.jl 0.7.0, and my system describes FITSIO.jl as 0.7.0+.  The tests pass on my machine.
Do you know how to correctly specify this in the REQUIRE file?  The julia docs for REQUIRE only describe pre-releases, like 0.8-.  I'm not sure what exactly 0.7.0+ means.

I think the + means you've got a prereleased version of fitsio.jl checked
out. One way probably is to put the line Pkg.clone("http...") in the test
script. Another is to ask Kyle to push his changes by tagging it as a new
version. There's probably a better way still, but these are 2 ideas.
On Fri, Jun 26, 2015, 4:51 PM Ryan<EMAIL_ADDRESS>wrote:

Indeed, the Travis CI is using FITSIO.jl 0.7.0, and my system describes
FITSIO.jl as 0.7.0+. The tests pass on my machine.
Do you know how to correctly specify this in the REQUIRE file? The julia
docs for REQUIRE only describe pre-releases, like 0.8-. I'm not sure what
exactly 0.7.0+ means.
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/38#issuecomment-115921372
.

Kyle pushed a version and the tests now pass.

Looks good. Is there a way to get test coverage of some of this new code?

Yes, but things are going to continue to change for a bit. Let me get
everything working on real images and then fill the test coverage back in
so as not to have to keep updating tests.
On Fri, Jul 3, 2015, 18:28 Jeffrey Regier<EMAIL_ADDRESS>wrote:

Looks good. Is there a way to get test coverage of some of this new code?
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/38#issuecomment-118447911
.


Update README.md

Remove whitespaces
Description
Fix README
Type of change
Fix Readme.
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Thank you!


RuntimeError: There is no current event loop in thread 'Thread-2'

what about my problem? I use mac

I have the same problem!

linux ubuntu  , python version is Python 3.6.3 :: Anaconda custom (64-bit)

@DavidFranck @oldFuryBird
Hi, i got the same problem too.
It seems to be the tornado cause this, maybe you installed the latest tornado==5.0, please downgrade to 4.5.2 use pip install tornado==4.5.2. This works for me.


Error code 10012

Ubuntu 18
Python 3.6.9
error after 3 days or so:
Traceback (most recent call last): -126 (0.04x), Entry: 57224.09, Balance: 0.00xxxxxxx, PNL: -0.00000590.
File "run.py", line 215, in
p = s.my_position(symbol=config.SYMBOL)['result']
File "/usr/local/lib/python3.6/dist-packages/pybit/init.py", line 790, in my_position
auth=True
File "/usr/local/lib/python3.6/dist-packages/pybit/init.py", line 1501, in _submit_request
time=dt.utcnow().strftime("%H:%M:%S")
pybit.exceptions.InvalidRequestError: Can not get symbol position info (ErrCode: 10012) (ErrTime: 07:41:51).
Request → GET https://api.bybit.com/v2/private/position/list: {'api_key': 'xxxxxx', 'recv_window': 5000, 'symbol': 'BTCUSD', 'timestamp':<PHONE_NUMBER>105, 'sign': 'xxxxxxxxx'}.
It's safe to add ErrCode: 10012 to line 51 like:
retry_codes={10002, 10012, 10006},
Or will needed another way ?
thanks

Yeah, if this is a temporary error that you encountered, you can add it to retry_codes to force retry.

@verata-veritatis already added and so far so good no more errors (+6 days running).
It's possible to have a more basic version without auto generated orders ? Like only to have manually orders, eg: I put a manual order sell limit at 60.000 then when opened the script put an order to close by defined value in settings like 2.000 then when hit 58.000 the position will be closed then the script put another sell order at 60.000 ? And keep repeating this.

I got another error after 10 days or so running without issues:
Traceback (most recent call last):
File "run.py", line 215, in
p = s.my_position(symbol=config.SYMBOL)['result']
File "/usr/local/lib/python3.6/dist-packages/pybit/init.py", line 790, in my_position
auth=True
File "/usr/local/lib/python3.6/dist-packages/pybit/init.py", line 1372, in _submit_request
time=dt.utcnow().strftime("%H:%M:%S")
pybit.exceptions.FailedRequestError: Bad request. retries exceeded maximum. (ErrCode: 400) (ErrTime: 21:06:55).
Request → GET https://api.bybit.com/v2/private/position/list: {'api_key': 'xxxxxx', 'recv_window': 7500, 'sign': 'xxxx', 'symbol': 'BTCUSD', 'timestamp':<PHONE_NUMBER>304}.
I don't code but I can see this is likely to be related to the pybit

I got another error after 10 days or so running without issues:
Traceback (most recent call last):
File "run.py", line 215, in
p = s.my_position(symbol=config.SYMBOL)['result']
File "/usr/local/lib/python3.6/dist-packages/pybit/init.py", line 790, in my_position
auth=True
File "/usr/local/lib/python3.6/dist-packages/pybit/init.py", line 1372, in _submit_request
time=dt.utcnow().strftime("%H:%M:%S")
pybit.exceptions.FailedRequestError: Bad request. retries exceeded maximum. (ErrCode: 400) (ErrTime: 21:06:55).
Request → GET https://api.bybit.com/v2/private/position/list: {'api_key': 'xxxxxx', 'recv_window': 7500, 'sign': 'xxxx', 'symbol': 'BTCUSD', 'timestamp':<PHONE_NUMBER>304}.
I don't code but I can see this is likely to be related to the pybit

Yikes, interesting. 400 would mean the request is bad but it doesn't seem like it is consider it's been working for 10 days consecutively without issue. I'll play around with error handling to see it this can be avoided. Thanks!

I'm getting ErrCode 10012 (not even in the official APIs doc)
I think it's a problem by their side and not the code. It occours randomly when there is a lot of frenzy in the market.
I would like to find a way to tell the code to retry instead of crashing the app.

I'm getting ErrCode 10012 quit often, always in the worst situations. (the error is not even documented in the official APIs doc)
I think it's a problem by their side and not the code. It occours randomly when there is a lot of frenzy in the market.
I would like to find a way to tell the code to retry instead of crashing the app.
It is always related to the /private/position/list. If I debugged this right it happens randomly when the bot is adding more to the current position.

Check the original post here. You can use the retry_codes argument.


[BUG]f5-journeys_postgres_1 keep restarting PORT:0 Windows 10

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

On Windows10 WSL, install docker, docker-compose, f5-journeys-app, start docker-compose
Enter command 'docker-compose up -d'
All 3 processes are red. Manually start f5-journeys_redis_1 and f5-journeys_acc-api_1. Works. But f5-journeys_postgre_1 fails.
https://localhost:8443/ also failed to connect.

postgres_1  | Error: Database is uninitialized and superuser password is not specified.
postgres_1  |        You must specify POSTGRES_PASSWORD to a non-empty value for the
postgres_1  |        superuser. For example, "-e POSTGRES_PASSWORD=password" on "docker run".
postgres_1  |
postgres_1  |        You may also use "POSTGRES_HOST_AUTH_METHOD=trust" to allow all
postgres_1  |        connections without a password. This is not recommended.
postgres_1  |
postgres_1  |        See PostgreSQL documentation about "trust":
postgres_1  |        https://www.postgresql.org/docs/current/auth-trust.html
Expected behavior
https://localhost:8443/ should open the Journeys GUI.
Screenshots

Context

Please specify, app version:
NA
And Platform version of the system you want to migrate.
NA

Have you followed the env preparation procedure, especially (2) copying env file?
cp sample.env .env

@zahajkiewicz would the variable WORKING_DIRECTORY=/tmp/journeys be different for Windows?
Like WORKING_DIRECTORY="C:\Users\bob\journeys" assuming user created a folder in "C:\Users\bob".

Yes, it should look different. But it seems .env file, where postgres vars are declared:
DB_NAME=postgres
DB_USER=postgres
DB_PASS=postgres
DB_HOST=postgres

was not copied (it resides in the current dir - repository, not WORKING_DIRECTORY).

I have good news to report.
I was able to start Journeys-app after changing the WORKING_DIRECTORY variable.
On PC, find current working directory.
nge@FLD-L-00060668:/mnt/c/Users/ge/f5-journeys$ pwd
/mnt/c/Users/ge/f5-journeys
Update .env with above working directory
nge@FLD-L-00060668:/mnt/c/Users/ge/journeys$ vi .env
WORKING_DIRECTORY=/mnt/c/Users/ge/journeys
Restart docker-compose from PowerShell. Successful.
PS C:\Users\ge\f5-journeys> pwd
Path
C:\Users\ge\f5-journeys
PS C:\Users\ge\f5-journeys> docker-compose up -d
Recreating f5-journeys_postgres_1 ...
Recreating f5-journeys_postgres_1 ... done
Recreating f5-journeys_acc-api_1  ... done
Creating f5-journeys_journeys_1   ... done
Creating f5-journeys_celery-worker_1 ... done
At this point, the GUI is accessable from https://localhost:8443/
I also tried to start docker-compose from WSL but failed.
The command 'docker-compose' could not be found in this WSL 1 distro.
We recommend to convert this distro to WSL 2 and activate
the WSL integration in Docker Desktop settings.

Issue resolved.


Question: Grid Error

Hello,
This should probably be tagged as a question instead of as a bug, I'm trying to follow your examples from goTop and using the following code snippet (some bits are stripped out) I'm getting the following error when rendering the grid.
var (
	grid  *ui.Grid
	plot0 *w.Plot
	par0  *w.Paragraph
)

func initWidgets() {
	plot0 = w.NewPlot()
	par0 = w.NewParagraph()
}
func setupGrid() {
	grid := ui.NewGrid()
	termWidth, termHeight := ui.TerminalDimensions()
	grid.SetRect(0, 0, termWidth, termHeight)
	plotRow := ui.NewRow(1.0/1,
		ui.NewCol(1.0/6, par0),
		ui.NewCol(5.0/6, plot0),
	)
	grid.Set(plotRow)
}
func main() {
                if err := ui.Init(); err != nil {
			log.Fatalf("failed to initialize termui: %v", err)
		}
		defer ui.Close()
	       initWidgets()
       	       setupGrid()
...
	      ui.Render(grid)
}

[5/5]0xc0000ac3c0[5/5]0xc0000ac370panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1115845]
goroutine 1 [running]:
github.com/gizak/termui.(*Grid).GetRect(0x0, 0x0, 0x6, 0xffffffffffffffff, 0x0)
:1 +0x5
github.com/gizak/termui.Render(0xc0000bbc20, 0x3, 0x3)
/Users/keithknott/go/src/github.com/gizak/termui/render.go:21 +0x189
main.main()
/Users/keithknott/go/src/github.com/keithknott26/linegraph/linegraph.go:266 +0xd99
Any idea why this happens? Perhaps I'm defining the grid incorrectly? The same code works fine when I move it back into main()

So the issue is with this line:
grid := ui.NewGrid()

which should be changed to
grid = ui.NewGrid()

with the colon removed.
Instead of assigning to the global grid variable, you're creating a new local grid, so the global grid stays nil. Definitely one of the gotchas with Go. :)


Fix for failing test: test_connection_error

http://fooobarbangbazbing.httpbin.org does not seem to be an unknown domain anymore.  I've also split up this unit test into two tests since they test slightly different things.

LGTM

Thanks! :cake:


Update LiteCollection.where() to use function chaining

LiteModel.where() has been updated to use function chaining using LiteQuery. LiteCollection.where() should follow the same syntax.

LiteCollection.where() now supports function chaining, as of #36


fix: don't insert gensis block twice

Can you elaborate? What is not being safeguarded? Which constraint on what? The objective of this PR was not to safeguard anything, it was just so Stratus could be restarted without restarting the db, since the database rightfully does not allow for two genesis blocks to be inserted.


feat: Remove lenra_ui_runner dependency

#__
Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[x] I made my own code-review before requesting one

Description of the changes

The lenra_ui_runner dependency was not necessary anymore.

:tada: This PR is included in version 1.0.0-beta.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added life to observations of living entities.

I added a simple check to added life to the observations from living entities similar to the check for item stack specific values.
I'm working on a project with player/entity interaction and couldn't find any other way to get the life of an entity.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

OK. But seems like that a mew 'LICENSE AGREEMENT' established?
来自Cyanogenmod Mail
Microsoft Contribution License Agreements<EMAIL_ADDRESS>
This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.￼
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/Microsoft/malmo","title":"Microsoft/malmo","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/Microsoft/malmo"}},"updates":{"snippets":[{"icon":"PERSON","message":"@msftclas in #536: \r\nThis seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.\n_Thanks,\nMicrosoft Pull Request Bot_"}],"action":{"name":"View Pull Request","url":"https://github.com/Microsoft/malmo/pull/536#issuecomment-302114786"}}}

@StStevens great, thanks for this!
Could you also check/update the python samples? There are a few which use namedtuple for parsing the entities, which I think will break unless the new field is added to them.
Cheers!

@DaveyBiggers  You were correct about that breaking examples. I took a look and updated all the 4 python examples that used namedtuple for parsing and added another field to the EntityInfo namedtuple. I also updated the reward_from_items example to be the same as the other 3 files even though the fields are unused.

Brilliant, thanks so much for this, and for cleaning up reward_from_items.


Feature Request: Cluster Admin creation

Would it be possible to create a resource for rosa create admin?
I believe it is the last manual command execution via the cli I have in my build, and it would be great to fully use the rhcs provider for this instead.

Thank you for the feedback.
This is already in our backlog of features and also requires some additions to our API.

Hi @OurFriendIrony ,
Version 1.2.2 already include the option to create cluster with default admin user
In order to use that, please include the admin credentials in cluster creation

Closing this issue as it was already solved in verion 1.2.0


Frozen UTXO shall be released by the same address.

Currently, there is no rule for the address of the output of the transaction when frozen UTXO is released. As a result,
frozen UTXO can be released and transferred to another address and locked from that address for a period of time.
The frozen funds are not allowed to be used for a period of time after being released.
However, if the address changes after it are frozen, it may be considered that the payment has already occurred.
Therefore, even after freezing, the address must be the same as the input of the transaction.

However, if the address changes after it are frozen, it may be considered that the payment has already occurred.

Why would it "be considered"? The UTXO is frozen and can't be spent regardless of the destination address.

As far as I see you're talking about the case where someone melts a locked utxo to a different address. So the wallet of the second address would need to detect that it's a melting transaction, therefore it cannot be spent yet.
This is the same as if the user melted the utxo to itself. The wallet would still need to know this UTXO is not spendable until the melting period is over.
So even if you add a protocol rule that utxo can only be melted to the same address, the wallet would still need to be able to detect this melting transaction and make sure that the "balance" shows that melting is not spendable yet.

I had the same thought as @AndrejMitrovic  before I thought of the following case.
However, when a person purchases an item using frozen UTXO, the person who receives the payment will unfortunately suffer.
This issue is to prevent such mishaps.

the person who receives the payment will unfortunately suffer.

Their wallet should let them know the UTXO is not spendable.

Shouldn't Agora support verification of transactions created without using a wallet?

But it already does?

As mentioned by @AndrejMitrovic , the wallet should support this. I was partial to the argument that this could constitute a payment though, but then, one could perform a payment by selling their private key just the same.


Larger tap target for ... buttons

Too small, should be 44x44

Bug Report Dump (Auto-generated)

Version 1.12.0 (1166)
Device: iPhone 7 Plus (iOS 11.1)
TestFlight: false

Ah so that's why they weren't always working as well. Makes sense.


[ArPow Tarball] Review Arcade patches required to build prototype tarball

Arcade requires 2 additional patches to build successfully in the tarball prototype.  These patches need to be reviewed and if valid, should be integrated into Arcade.

@dseefeld - can you please provide a link to the additional patches you are referring to?

Here are the two patches from the ArPow prototype being referenced - https://github.com/dseefeld/linker/tree/tarballCreation/src/SourceBuild/tarball/content/patches/arcade

Duplicate of #2345.


Synthes 3D

How can I adjust the code so that the generated super resolution is saved as the input 3D nii.gz format? @wendy-xiaozong


Scheduled zero arg jobs fail

It appears as though scheduled zero-arg jobs end up getting their [] arg list cast into a {} with the LPUSH command.
Verk.Supervisor.start_link

defmodule Job do
  def perform do
    IO.inspect "hello"
  end
end

job = %Verk.Job{class: Job, args: [], queue: "default"}

Currently this results in:
15:02:39.544 [info]  Elixir.Job<PHONE_NUMBER>1859802746 start
15:02:39.548 [debug] Job failed reason: %ArgumentError{message: "argument error"}
15:02:39.548 [info]  Elixir.Job<PHONE_NUMBER>1859802746 fail: 5 ms
15:02:39.552 [error] GenServer #PID<0.196.0> terminating
** (stop) :failed
Last message: {:"$gen_cast", {:perform, %Verk.Job{args: %{}, class: "Elixir.Job", enqueued_at: nil, error_backtrace: nil, error_message: nil, failed_at: nil, finished_at: nil, jid: "13837351251859802746", max_retry_count: 25, original_json: "{\"error_message\":null,\"max_retry_count\":25,\"jid\":\"13837351251859802746\",\"class\":\"Elixir.Job\",\"retried_at\":null,\"queue\":\"default\",\"finished_at\":null,\"enqueued_at\":null,\"error_backtrace\":null,\"args\":{},\"failed_at\":null,\"retry_count\":0}", queue: "default", retried_at: nil, retry_count: 0}, #PID<0.197.0>}}
State: nil

And the redis console output:
1518724956.160306 [0 <IP_ADDRESS>:54702] "ZADD" "schedule" "1518724956" "{\"retry_count\":0,\"retried_at\":null,\"queue\":\"default\",\"max_retry_count\":25,\"jid\":\"13837351251859802746\",\"finished_at\":null,\"failed_at\":null,\"error_message\":null,\"error_backtrace\":null,\"enqueued_at\":null,\"class\":\"Elixir.Job\",\"args\":[]}"
1518724957.987005 [0 lua] "LPUSH" "queue:default" "{\"error_message\":null,\"max_retry_count\":25,\"jid\":\"13837351251859802746\",\"class\":\"Elixir.Job\",\"retried_at\":null,\"queue\":\"default\",\"finished_at\":null,\"enqueued_at\":null,\"error_backtrace\":null,\"args\":{},\"failed_at\":null,\"retry_count\":0}"

Thanks for the bug report! Yeah this looks wrong 🤔

Looks like this comes from the cjson package used in the lua scripts:
<IP_ADDRESS>:6379> EVAL 'return cjson.encode(cjson.decode(\'{"args": []}\'))' 0
"{\"args\":{}}"

See also https://github.com/mpx/lua-cjson/issues/11#issuecomment-287617268.
Not sure what can be done. 😞
@nsweeting The only workaround I can think of at the moment is to have at least one argument for such jobs even if it's not used.
defmodule Job do
  def perform(1) do
    IO.inspect "hello"
  end
end

job = %Verk.Job{class: Job, args: [1], queue: "default"}

Not Ideal but better than getting that error.

Maybe we could try to "patch" the JSON when decoding? We know that args is an array. I wonder what's the best place to do this... Also maybe check if it's possible to enforce

What about something like this - https://github.com/edgurgel/verk/pull/154?


Enable private load balancer usage for the kube api servers on the seeds

How to categorize this issue?
/area networking
/area security
/kind enhancement
/kind discussion
/priority normal
What would you like to be added:
I would like to add the to-private-networks networkpolicy to the dependency watchdog prober.
here:
labels:
  ...
  networking.gardener.cloud/to-private-networks: allowed

Why is this needed:
This would enable the usage of private load balancers as kube api server on the seeds. Usage of private load balancer increases the security and is a requirement for some teams to operate.

Seems like there is no obvious reason for not doing such a change.
Feel free to drop a PR with your improvement.


Integrate JWT information into http:RequestContext

Description:
http:RequestContext contains the meta-information about the http:Request. It is required that the http:RequestContext contains the auth/authz information (of the original HTTP request) which could be extracted from the JWT.

@ayeshLK Please share the initial design of the task.


Translation bug: IPTT filter panel

Affects: Filter panel for both versions of the IPTT form -- Target vs Actuals and Recent Progress
We'd like to translate the options for the following filter menus:

[ ] Level
[ ] Type

These are database values. As such, they can't really be translated unless we remove them as customizable options in the admin.
See Django models:
Level
IndicatorType
Related to: #896

The translation of levels will be addressed as part of Satsuma; our Result Framework templates will not be editable database values, and therefore can be translated.
"Type" is not a high priority field to improve, so closing this ticket.


Some questions

Thank you for your exceptional work and valuable contributions to this field. I have two questions:

What is the difference between BeTopNet-full and BeTopNet-e2e? I might have missed it, but I couldn't find an explanation anywhere.
Do you plan to release the code for nuPlan?

Looking forward to your reply.

Hi Bozhou, thanks for your interests in our work! The "-full" version denotes the model using 64 modes and NMS for post-processing, while the "-e2e" one works directly on 6 modes. We are also working on the scrutinization for nuPlan project, due to some delay for ongoing project, you may expect the release on next month or upon.
Best

I got it, thanks for your reply.


Restrict workflows to subfolders.

Only run code checks when code/ is touched; latex when talk/ is touched.

Very good idea !

Very good idea !

I know, but now nothing is running. 😂


Do not publish global.json as part of the content

I've recently started using this library in my blazor app. One change i decided on was removing my global.json file and pinning to the latest SDK available. When i deleted it i noticed the project still locates and uses the global.json file in .nuget\packages\sotsera.blazor.toaster\0.5.2\contentFiles\any\netstandard2.0\global.json instead.
Please consider not publishing it as part of the NuGet package to allow users to decide their own SDK version.

Ciao Ran,
sorry for that, I didn't realize that it was going to be bundled with the library. I'll try to have a new version out later today.
Thanks for pointing that out!

Seems like that's the default for the component library template, Maybe we should open an issue in Blazor's repo to change that.

I just pushed a new version (0.5.3) without the global.json files.
I tried to created a new bloazorlib project but it didn't contain the file so I looked into the toaster history and found a commit where I manually added the global.json file so I'm the one to be blamed for it :smile:

Hah. I was sure i saw it included in a new blazorlib project, but maybe I missed something. Either way, thanks for the quick reply and release :)


added missing braces

Phew! I just had to do that before continuing with my talk!
Thanks to my audience at DevDay!

I feel like our project is being abused for the purpose of demos :)

:innocent:

excellent contribution!


nixos-rebuild switch --build-host requires sudo

Describe the bug
For some reason, when using nixos-rebuild switch --build-host user@remote --flake <path> it requires a sudo password.
It works with root@remote, but i don't understand why i have to be root on the remote to deploy to localhost.
Output log:
warning: Git tree '/home/cute/dev/dotfiles' is dirty
building the system configuration...
warning: Git tree '/home/cute/dev/dotfiles' is dirty
trace: warning: `overrideScope'` (from `lib.makeScope`) has been renamed to `overrideScope`.
trace: warning: nixfmt was renamed to nixfmt-classic. The nixfmt attribute may be used for the new RFC 166-style formatter in the future, which is currently available as nixfmt-rfc-style
warning: The interpretation of store paths arguments ending in `.drv` recently changed. If this command is now failing try again with '/nix/store/ks8rfjlxd5hrinr37f3l56nnb38pdj3f-nixos-system-laptop-24.05.20240408.4cba8b5.drv^*'
sudo: a terminal is required to read the password; either use the -S option to read from standard input or configure an askpass helper
sudo: a password is required

I have also tried specifying --use-remote-sudo and being root on the localhost just in case, it doesn't help:
NIX_SSHOPTS="-o RequestTTY=force" nixos-rebuild switch --build-host user@remote --use-remote-sudo --flake <path>
Steps To Reproduce
Steps to reproduce the behavior:

Run sudo nixos-rebuild switch --build-host user@remote (it's important to specify user, not root)

Expected behavior
I should be able to use --build-host without sudo on the remote user@remote
Metadata
cute@laptop ~> nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.7.11, NixOS, 24.05 (Uakari), 24.05.20240403.08b9151`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - nixpkgs: `/nix/store/wx38bcq0bi8xhia8yq21syxlkf65fk4j-source`

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

Looking at the implementation of nixos-rebuild:
https://github.com/NixOS/nixpkgs/blob/02fe29395df8e5571291ebcc6114f94dd33f722b/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh#L212-L227
The offending call to sudo is inserted because the variable $useSudo is set to 1. The only place I could find this variable to be set is when dealing with target hosts:
https://github.com/NixOS/nixpkgs/blob/02fe29395df8e5571291ebcc6114f94dd33f722b/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh#L244-L258
So sudo is inserted when $remoteSudo is not the empty string. The only case I found where $remoteSudo is set is when the --use-remote-sudo flag is used:
https://github.com/NixOS/nixpkgs/blob/02fe29395df8e5571291ebcc6114f94dd33f722b/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh#L164-L166
All in all, I don't understand what's happening because the bug you described still happens even when --use-remote-sudo isn't used. But there's a very easy temporary fix (successfully tested on my side): create your own copy of the nixos-rebuild script, remove the part where sudo is inserted, and use this modified script to rebuild your system.

Ok so in fact in NixOS 24.05 the nixos-rebuild is slightly different and there is another place where useSudo is set:
https://github.com/NixOS/nixpkgs/blob/0c5b4ecbed5b155b705336aa96d878e55acd8685/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh#L163-L165
So as soon as there are users in the wheel group (this is what it looks like the SUDO_USER is) this sudo will be inserted, causing the bug. An even simpler fix would thus be to run
SUDO_USER ="" nixos-rebuild --build-host ...

But the lines at fault have been removed in nixpkgs-unstable, so I assume the issue has been fixed and hasn't been closed yet?


Updated doc for PROPERTY_HINT_RANGE (or_greater/or_lesser)

Replaced allow_greater & allow_lesser with the correct options or_greater & or_lesser in the documentation for PROPERTY_HINT_RANGE & PROPERTY_HINT_EXP_RANGE
Code reference:
https://github.com/godotengine/godot/blob/3418f76a9eab9f496e5b26310bd3bc1125b8119b/editor/editor_properties.cpp#L2979-L2984
https://github.com/godotengine/godot/blob/3418f76a9eab9f496e5b26310bd3bc1125b8119b/editor/editor_properties.cpp#L3032-L3037

Thanks!

Cherry-picked for 3.1.2.


refactor(react): update hot reload implementation

Description:
Update hot reload implementation to get reload working again.
Motivation and Context:
Hot reloading isn't triggering instant rerender.
How Has This Been Tested?
Manually
Types of changes:
Bug fix
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.009%) to 19.171% when pulling e26361b305b91a8cbca4645b51014d2803503a25 on mrfelton:fix/react-hot-loader into 174aa8714507b0c72e88f0db772b235add7444b6 on LN-Zap:next.


Does not work on elixir 1.10

[x] elixir-sense version not compatible - bump to incorporate fixes from
https://github.com/elixir-lsp/elixir_sense/commit/59fe0512ce078750878ae4b4939b323f8560cc84
https://github.com/elixir-lsp/elixir_sense/commit/8b54a77cb97fa5d6d1558f60789d16dfcc5e91fd
https://github.com/elixir-lsp/elixir_sense/commit/fc8f3265443abed448fc1b5774f6175192f604e3
https://github.com/elixir-lsp/elixir_sense/commit/1e528d66127da872bdfa0bc8a02763375287a01c
https://github.com/elixir-lsp/elixir_sense/commit/5f259223d070336eeceab321642969aae97e034b
https://github.com/elixir-lsp/elixir_sense/commit/1aa09ab39f705110bb93e9624f85ae4c310c8ef9
https://github.com/elixir-lsp/elixir-ls/pull/87

[x] https://github.com/elixir-lsp/elixir_sense/issues/76 needs fixing

[ ] https://github.com/elixir-lsp/elixir_sense/issues/75 needs fixing

[ ] dialyzer tests failing - asked for help on elixir-forum https://elixirforum.com/t/elixir-v1-10-0-rc-0-released/28086/42

[ ] clear_cache moved to Mix.State https://github.com/elixir-lsp/elixir-ls/pull/91

[x] Code.load_file/1 is deprecated https://github.com/elixir-lsp/elixir-ls/pull/93

Things like Mix.State is also a private API, so please don't continue using private APIs. Open up an issue so we provide a public API for you to call.
About the dialyzer one, please provide a project that reproduces the error. I am assuming mix dialyze or similar reproduces it? It does look like an Elixir bug.

@lukaszsamson thanks for putting this list together and tracking down these issues!
@josevalim Posted on elixir-lang-core https://groups.google.com/forum/#!topic/elixir-lang-core/_LrWhl-kUZY (although re-reading your comment, perhaps I should have just raised an issue directly). Although in either case, this is just one of the private api usages, I'll try and do a sweep to look for the others.

@josevalim I looked deeper into dialyzer issue. It turns out that dialyzer is not required to trigger it at all and it might be a consequence of the fixtures used in this project.
I am able to reproduce from the root of this repo
$ iex -S mix
Interactive Elixir (1.10.0-rc.0) - press Ctrl+C to exit (type h() ENTER for help)
iex(1)> mixfile = "apps/language_server/test/fixtures/dialyzer/mix.exs"
"apps/language_server/test/fixtures/dialyzer/mix.exs"
iex(2)> Kernel.ParallelCompiler.compile([mixfile])
{:ok, [ElixirLS.LanguageServer.Fixtures.Dialyzer.Mixfile], []}
iex(3)> Mix.Task.run("compile", ["--return-errors", "--ignore-module-conflict"])

19:44:23.319 [error] Task #PID<0.524.0> started from #PID<0.541.0> terminating
** (MatchError) no match of right hand side value: false
    (elixir 1.10.0-rc.0) src/elixir_erl_var.erl:14: :elixir_erl_var.translate/4
    (stdlib 3.11) lists.erl:1354: :lists.mapfoldl/3
    (elixir 1.10.0-rc.0) src/elixir_erl_clauses.erl:20: :elixir_erl_clauses.match/3
    (elixir 1.10.0-rc.0) src/elixir_erl_clauses.erl:28: :elixir_erl_clauses.clause/6
    (elixir 1.10.0-rc.0) src/elixir_erl.erl:230: :elixir_erl.translate_clause/3
    (elixir 1.10.0-rc.0) src/elixir_erl.erl:220: :elixir_erl."-translate_definition/4-lc$^0/1-0-"/2
    (elixir 1.10.0-rc.0) src/elixir_erl.erl:220: :elixir_erl.translate_definition/4
    (elixir 1.10.0-rc.0) src/elixir_erl.erl:183: :elixir_erl.split_definition/11
Function: #Function<9.64842280/0 in Mix.Tasks.Compile.Protocols.consolidate/6>
    Args: []

It seems that the protocol defined upper in directory hierarchy ends up in the fixture dir

I am looking into this now. Thank you so much for isolating those, it makes my life much easier!

@lukaszsamson so the command above compiled just fine for me. I am thinking you have some stale files somewhere. Try running git clean -dfx and removing _build and deps. Or try a separate checkout and see if you can reproduce it there.

@josevalim yes, it must have been some stale state. I remember doing mix clean && mix deps.clean --all && rm -r .elixir_ls though, it seems this was not enough. Anyway, thanks for your help.


disposeNestedNavigators = false in TabNavigator not working

I initialized the TabNavigator like below, however the OnDispose still be called when switching tabs and ScreenModel has been reset when coming back to the Tab,  If it's a bug or somehow I used in the wrong way

            TabNavigator(
                tab = Tab,
                disposeNestedNavigators = false
            )

Ok, The problem is using BottomSheetNavigator in the Tab,  it has own navigator and do not provider params to set disposeNestedNavigators=false.


[ALLUXIO-3342] Replay journals in parallel (#8095)

https://alluxio.atlassian.net/browse/ALLUXIO-3342
Journals can be replayed independently, so we
can greatly improve master startup time by replaying
the journals in parallel instead of one after another.
Running locally with a new journal and default
configuration, this change reduces master startup time
from 26 seconds to 11 seconds.

alluxio-bot, check this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/927/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/928/
Test PASSed.


Internal network in LogicalInterconnectGroup

Scenario/Intent
I want to configure InternalNetwork for a logical Interconnect group. Is network name supportred? If yes, waht is the attribute name? InternalNetworkName? internalNetworkName?
Environment Details
Module Version:
Ansible Version: 2.9.10
OneView Appliance Version: 5.30.00-0421400
OneView Client API Version: 1800
Python Version: 3.6.8
python-hpOneView SDK Version: I have cloned SDK files from path https://github.com/HewlettPackard/oneview-ansible
**Platform:**RHEL 7.8

Currently, passing internalNetworks to LIG by names is not supported. We will take this as an enhancement request and will provide the fix in future release.

Currently, passing internalNetworks to LIG by names is not supported. We will take this as an enhancement request and will provide the fix in future release.

@DungKHoang , I have added a fix for this issue. So you can add internal networks by passing "internalNetworkNames" key and values should be passed as list.
internalNetworkNames:
  - "test-network-1"
  - "test-network-2"

Fix is merged to master branch and can be available in the upcoming release.


Filtering Event of Informers After Watched Resource Update

Problem Statement
In case we during the reconciliation update a dependent resource, we receive a new event regarding the change of the dependent resource from the informer. In general this even can be skipped since we know that the state of the target resource is already the desired one. The way to do this would be compare resourceVersion of the resource of the received event with the resource version of a cached resource (not the current informer cache).
Proposed Solution
One relatively transparent way would be to propagate the resourceVersion to the informer to cache it. When we receive the next event compare the resources versions, if are the same just skip the event. But either is the same or not, instantly remove the cached resourceVersion from the cache. This way we don't cache too long the version, and don't have to specially handle the cleanup on resource delete event.
Note that dependent resources could do the propagation automatically, while on low level API this needs to be done explicitly by the implementation.

Regarding concurrency issue would a semaphore mechnism allows to update the resource while blocking the evensource until the resourceVersion is propagated.  ? The drawback is that the informer will not be able to process new events, so maybe it can retain events concerning only the resource being edited.
This should have no impact because we are still in the reconcile loop and imho, we should return the same version of the dependent resource that raise the reconciliation. That is to say, two different calls in the same reconcile loop to context.getSecondaryResource shall return the same resource version.

Regarding concurrency issue would a semaphore mechnism allows to update the resource while blocking the evensource until the resourceVersion is propagated. ? The drawback is that the informer will not be able to process new events, so maybe it can retain events concerning only the resource being edited

This is not really feasible, because there might be other events, meanwhile, and the controller might fail or other errors might happen etc, that would need special care etc. Also the event can be lost in some corner cases.
I think the proposed solution is good enough, btw we already do something similar with the custom resource, that seems to be work fine.

That is to say, two different calls in the same reconcile loop to context.getSecondaryResource shall return the same resource version.

Ideally yes, there should be kinda "snapshot isolation", but its not that big issue if doesn't imho, also it can be handled on code level, plus usually the resource only read once. I pretty sure that controller-runtime is not doing this either.

The linked implementation does the job, but not always, especially locally the event is sometimes arrived before the temporal cached is updated. I guess this is the best we can do ATM.

An additional improvement could be that have this as a two phase update. Like mark that we will update resource so, no events are processed from that point. Maybe record the events coming meanwhile. And after the create/update make a second call to evaluate the situation. That will solve the concurrency issues, but will complicate the implementation.


[WIP]BZ#1486224 - Refactor sui group switching to use api

https://bugzilla.redhat.com/show_bug.cgi?id=1486224
Right now it uses headers, taks a LOOOONNNGG time to take effect, most of the time feels broken, gonna switch this over to using api.
In order to keep excessive api calls down, require this work to be completed first

[x] https://github.com/ManageIQ/manageiq-api/issues/74

@AllenBW this ready to go?

@chriskacerguis Nope, still gotta do work, above mentioned pr was just merged in

Updated da message

@chriskacerguis can you set the Milestone to Sprint 70

@JPrause done.


Conversion to docx

Note that that can be done with pandoc:
pandoc manuscript.tex --biblatex --bibliography=main.bib -o manuscript.docx --citeproc --filter=pandoc-crossref --cls=/path/to/stylefile

You need to install pandoc-crossref with
brew install pandoc-crossref

Find and download citation-style from CSL
I haven't figured out a good way yet to deal with numbered equations, pandas is not doing it itself. There's this package, but it currently doesn't work for me:

good numeric style: elsevier-vancouver