Changes not persisted to disk immediately

I noticed this during my testing, if i add or remove values and kill the app asap, on my next session my addition or removal didn't get saved to disk. I think this has to do with the synchronisation call that needs to be made to persist changes to disk.
Am i doing something wrong, or is it indeed the case? and if so, can we maybe add an optional method that immediately flushes changes to disk?

The method already exists, and it's Defaults.synchronize(). Note that it's not recommended to use it in normal circumstances. But if you need to ensure synchronization for testing reasons, there you go


Topic types are lowecased

Reference: https://developer.zendesk.com/rest_api/docs/web-portal/forums

@sudpaw broken link? I tried finding it myself but I couldn't find any references to topic apart from https://developer.zendesk.com/rest_api/docs/help_center/topics ..

It looks like they're fully removing the Web Portal / Forums implementation, and moved everyone over to Help Center.
https://support.zendesk.com/hc/en-us/articles/207323377

@matthewtckr thanks! "no longer be able to access your Web portal beginning on November 1, 2016." guess we need to cleanup some endpoints / classes.. @sudpaw have you migrated to Help Center?

ohx all for the info. Sorry for the late response.


Support Android as a host system for running VSCode

Now that there are smartphones with 12-16GB of RAM that can directly connect to an external monitor and present a desktop-like environment to users, coupled with VS Code's remote development features, if we can run Code on Android, we could potentially only use our phones for software development.

VS Code is based on Electron which likely won’t ever support Android, so it’s unlikely that VS Code the desktop client would. However VSO may in the future: https://visualstudio.microsoft.com/services/visual-studio-online/


feat: support device_id in embed

Signed-off-by: bwanglzu<EMAIL_ADDRESS>Goals:

This pull request allow user to specify a device_id along with their cuda device. This give users more flexibility when inferencing with GPU, with no breaking change.
I re-write the embed documentarray page completely, with more practical examples on how to embed using 3rd party providers, such as torchvision, timm, paddlepaddle.vision, transformers, openai and cohere. every code snnippt can be copy-paste and run as it is (except 2 endpoint needs user token)

Usage:
da.embed(
    embed_model='resnet_50_without_fc',
    device='cuda',
+   device_id=2,  # use the cuda device with index 2
    ...,
)

This one should also fix a paddle issue: paddle paddle use gpu:device_id to select cuda devices, not cuda. docs here.
The previous version of ONNX seems to be not working, now i'm adopting the code from ONNX
Need to test:

[x] cuda on torch
[x] cuda on tf
[x] cuda on paddle
[x] cuda on onnx

Need to change:

[x] code
[x] documentation

torch:

tensorflow:

2022-12-19 09:40:34.230259: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1613] Created device /job:localhost/replica:0/task:0/device:GPU:2 with 22288 MB memory: -> device: 2, name: NVIDIA GeForce RTX 3090, pci bus id: 0000:4b:00.0, compute capability: 8.6

onnx:

Codecov Report
Base: 83.00% // Head: 78.08% // Decreases project coverage by -4.91% :warning:

Coverage data is based on head (95eefde) compared to base (1443376).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #954      +/-   ##
==========================================
- Coverage   83.00%   78.08%   -4.92%
==========================================
  Files         155      155
  Lines        8049     8060      +11
==========================================
- Hits         6681     6294     -387
- Misses       1368     1766     +398

Flag
Coverage Δ

docarray
78.08% <0.00%> (-4.92%)
:arrow_down:

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

Impacted Files
Coverage Δ

docarray/array/mixins/embed.py
8.82% <0.00%> (-1.07%)
:arrow_down:

docarray/array/storage/opensearch/find.py
26.66% <0.00%> (-50.00%)
:arrow_down:

docarray/array/mixins/delitem.py
48.88% <0.00%> (-40.00%)
:arrow_down:

docarray/array/mixins/pydantic.py
61.53% <0.00%> (-38.47%)
:arrow_down:

docarray/array/storage/base/getsetdel.py
50.32% <0.00%> (-36.61%)
:arrow_down:

docarray/array/storage/sqlite/getsetdel.py
73.33% <0.00%> (-24.45%)
:arrow_down:

docarray/array/mixins/setitem.py
49.59% <0.00%> (-24.40%)
:arrow_down:

docarray/array/mixins/getitem.py
68.25% <0.00%> (-22.23%)
:arrow_down:

docarray/array/queryset/lookup.py
58.39% <0.00%> (-20.44%)
:arrow_down:

docarray/array/queryset/parser.py
68.33% <0.00%> (-20.00%)
:arrow_down:

... and 36 more

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

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

@alexcg1 thanks for your review. All updated

@bwanglzu I saw a couple of other little things to improve in my review

feature should be added in v2


Uncontrolled Memory Allocation in netty-codec 4.1.45.Final

There is a high severity vulnerability found in netty-codec [4.1.0.Final, 4.1.46.Final). The version<EMAIL_ADDRESS>is currently being used by http4s-async-http-client 0.21.7. I was wondering whether it was possible to release a hotfix with the dependency updated to 4.1.46.Final or higher?
Please refer to https://app.snyk.io/vuln/SNYK-JAVA-IONETTY-564897 for more details on the vulnerability.

I can start looking into this.

It looks like async-http-client >= 2.11.0 is sufficient.  I'm not sure offhand why sbt-updates isn't prompting us for this.

Oh.  Well, nuts:
     // Broke binary compatibility with 2.10.5
    dependencyUpdatesFilter -= moduleFilter(organization = "org.asynchttpclient", revision = "2.11.0"),

We could explicitly bump netty in series/0.21.  It might anger the sbt-explicit-dependencies plugin.

I went ahead and grabbed this one, because I'd like to release it today.

Fixed by #3747.


Trait to update cache

Add a trait to the base crate to define updating a cache from something like an event.
This allows implementing updates for any gateway event, or a guild create struct, etc.

Closed by https://github.com/rarity-rs/cache/commit/4e8580812c9f97b974ea199e95fdf292cd294aa8


[issue 2387] Update history status of the local history in case timeout

Describe the task
Update history status of the local history in case timeout
Actual: The history show status = Failed if timeout (2 minutes)
Expect: Sync status with Subscan to show correct status

🚀 Deployed on https://pr-2456--sw-web-runner.netlify.app


Convert print statements to logging

As explained in #5, it is not always desirable for libraries that messages are printed into standard output.
This PR changes print statements for messages like
converting nn.Conv2d and nn.Linear in out model ...

into log messages.
Logging allows for finer tuning for what to do with the printed output.
Also, by default it goes to standard error, which means that it does not
interfere with other scripts standard output.
Users of pytorch2timeloop-converter can then decide to print all messages with a setup like this:
import logging
logging.basicConfig(level=logging.DEBUG)

or decide to silence specific loggers
logging.getLogger("pytorch2timeloop.converter_pytorch").propagate = False
logging.getLogger("pytorch2timeloop.utils.hooks").propagate = False

logging.getLogger("pytorch2timeloop").propagate = False  # silence both

Fixes #5

Please take a look :)


[FEATURE] Create a backend to store onet user data

Is your feature request related to a problem? Please describe.
We want to be able to store the onet data that is relevant to the user so that we don't have to keep recalculating the bessi results every time the app is loaded up.
Describe the solution you'd like
We should save user data in mongodb that can be updated using a go program every time bessi results are changed.

Being worked on in this repo: https://github.com/ApoorvaAditya/skills-to-jobs-building-block.


Outdated/incorrect version in package.json

Hi, thanks for the library. I'm trying to use it with yarn but it doesn't like the version specified in package.json.
$ yarn add https://github.com/rsms/raster.git#v6
yarn add v1.19.1
[1/4] Resolving packages...
error Can't add "raster": invalid package version "4".

I believe the field is required to be semver formatted, so in this case it would be 6.0.

Actually it seems I'm late to the party: https://github.com/rsms/raster/pull/5.

The package.json file is only used for dependencies — any other metadata is unused. I didn’t realize folks would include it with npm/yarn. I’m not even sure you’d get the right files even if the version was correct. Let’s see if we can get #5 landed.


Mishandling eval TxnId cookie causing errors under stress

We found multiple errors that only manifest in 8.x under stress tests and only when eval is used and a d-node is stopped and restarted.  The errors are:

REST-INVALIDMIMETYPE: (rest:INVALIDMIMETYPE) Content-Type must be one of: 'text/xml', 'text/json', 'application/xml', 'application/json', Received: text/plain
XDMP-UPDATEFUNCTIONFROMQUERY or 'Operation not allowed on the currently executing transaction with identifier declareUpdate'
documents not retrieved even though they're in the db

These all appear to stem from a TxnID cookie which is coming from the /v1/eval or /v1/invoke endpoints and inappropriately applied to subsequent HTTP requests which should not send that cookie with the TxnID cookie from the previous request.

Did not see these errors in the last run on 02/22/2017.


Question: why does sign in omit the session information?

Currently, the sign in method explicitly omits session on the resolved account object. I did not expect this behavior, as I was storing a single account reference and using the presence of session to determine if the user was "logged in" or not. (session is present after account.get(), but not account.signIn())
What is the main reason for this behavior? In my case, I want the session information after signing in, as I've just opened a new session. However, I'm very new to hoodie, so it's totally possible I'm misusing it right now.
In short, can/should sign in return the session information?

hey @dominicbarnes it’s been a while I’ll have to look up the reason, but I agree it’s confusing at first sight. I think it was to make it coherent with .signUp() and .signOut(), but maybe there was something else.
You can do a hoodie.account.get('session') after the sign in

@gr2m yeah, that's exactly what I ended up doing :)


Changed volume declarations in Windows compose to use relative paths

What this PR does

Fixes the Windows compose yaml file to use relative paths for volume declarations. The existing version in main requires an exact path match to a particular dev machine.

How to test this PR

Run docker compose -f .\docker-compose-win.yml up -d on a Windows machine.

The compose should succeed and all containers should start successfully.

What this PR does not do

Set the grafana volume /var/lib/grafana, because I could not figure out what the source should have been. I left a comment in the yaml for this line.

Docker Version Info
Client:
 Cloud integration: v1.0.35-desktop+001
 Version:           24.0.5
 API version:       1.43
 Go version:        go1.20.6
 Git commit:        ced0996
 Built:             Fri Jul 21 20:36:24 2023
 OS/Arch:           windows/amd64
 Context:           default

Server: Docker Desktop 4.22.0 (117440)
 Engine:
  Version:          24.0.5
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.6
  Git commit:       a61e2b4
  Built:            Fri Jul 21 20:35:45 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.21
  GitCommit:        3dce8eb055cbb6872793272b4f20ed16117344f8
 runc:
  Version:          1.1.7
  GitCommit:        v1.1.7-0-g860f061
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Docker Compose Version Info
Docker Compose version v2.20.2-desktop.1

Hey @Bnewman thanks for this!! Seems I did forget to adjust the windows yaml to correctly use volumes, that is what the grafana data is missing.
I work mostly on Mac and there the config is better.
So I will merge this pull request and adjust the volumes :)


nix-fast-build: init at 1.0.0

Description of changes
nix-fast-build combines the power of nix-eval-jobs with nix-output-monitor to speed-up the evaluation and building process.
cc @Mic92
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 313554 run on x86_64-linux 1

2 packages built:

nix-fast-build
nix-fast-build.dist

nix-fast-build has recently received its 1.0.0 release, this probably could be updated to use the release tag instead of HEAD.
Also, minor nit but I think with lib; is actively discouraged in the meta field.

I must have missed this pull request.

Best rename the PR title too.

Result of nixpkgs-review pr 313554 run on x86_64-linux 1

2 packages built:

nix-fast-build
nix-fast-build.dist

Can I encourage you to avoid rec in favor of a top-level let in block? It's bad practice.

Can I encourage you to avoid rec in favor of a top-level let in block? It's bad practice.

i don't think it matters much in this case as it has basically 0 effect on the expression. #315337 would probably be a better place to discuss it

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1803


Adding the 404 page

I can add a 404 page like this:

can you assign this issue to me with labels gssoc-ext and hacktoberfest

On which urls this page will be displayed, share some invalid url's please

To your information, the url will be updated dynamically and there would be a check for valid and invalid urls so we dont need a 404 page in this
Sorry for this time!
You can search for some other issue which I would be happy to assign


Add HEADER_SEARCH_PATHS support

HEADER_SEARCH_PATHS seems not to be supported. I tried
targets:
  TWCrypto:
    platform: iOS
    type: framework
    deploymentTarget: "10.0"
    sources:
    - TWCrypto/TWCrypto
    settings:
    - HEADER_SEARCH_PATHS: $SRCROOT/OpenSSL/include
    dependencies:
    - framework: TWCrypto/OpenSSL/lib/libcrypto.a

But the setting did not get pick up.

settings is a dictionary, not an array. Try this:
targets:
  TWCrypto:
    platform: iOS
    type: framework
    deploymentTarget: "10.0"
    sources:
    - TWCrypto/TWCrypto
    settings:
      HEADER_SEARCH_PATHS: $SRCROOT/OpenSSL/include
    dependencies:
    - framework: TWCrypto/OpenSSL/lib/libcrypto.a

Closing this for now, let me know if you have any other issues

@yonaskolb Does this still work? I have
    settings:
      HEADER_SEARCH_PATHS:
        - "$(inherited)"
        - "$(PROJECT_DIR)/$(TARGET_NAME)/Externals/include"

in my project.yml under the correct target but it is not writing it into the project file.

@nickplee,  yes that should work. Are you merging it with a template that has a base definition, as that will take precedence

@yonaskolb Thank you for the response! That was exactly the issue. Seems like odd default behavior to me that the template would override the project specific settings, but I have not thought through it as thoroughly as you likely have.

Yeah the merging is simple key value merging, that doesn't have knowledge of how settings are structured, so this is a pitfall one can fall into. Theoretically that case could be handled manually


Major and minor refactorings

We should probably do a merge commit for this to preserve history?

Move common forms to Common.purs

Why make the above change in your first commit? I don't understand why this should be done and without knowing that, it just seems like a pointless change to me.

Move common forms to Common.purs

Why make the above change in your first commit? I don't understand why this should be done and without knowing that, it just seems like a pointless change to me.

I was thinking of also moving the scmPrefixed values into the Constants file, where they would also be wrapped in Identifier. The dependency tree would look like:
Syntax -> Constants -> Common
Syntax -> Common

@purefunctor if these refactorings are unrelated it might be worth to have each be in their own pull request, so they can have separate discussions

@purefunctor is it worth to bring this up to date, or should we close it?

I think this should be closed, I'll re-open PRs over time for the changes here.


External Logger port support

We're integrating this library into our embedded application, and it would be nice to be able to use our currently implemented logging (just console) mechanism to output libosdp messages out-of-box. So offering a way to configure the library to use a logger port would be very useful.
Thanks!

Hey, this is already supported :). Have a look at osdp_logger_init which takes osdp_log_puts_fn_t (doc). Let me know if that doesn't fit your use case for some reason.

I think our specific issue is that we use a logging framework that's actually very similar to what is built into libOSDP (we actually run into build issues when compiling - multiple function definitions). So if there is a way to define a port for all the logging macros that would be useful for our use case.

Ah, I see. Which project/environment are you building LibOSDP in? Maybe the namespace "logger_" in utils/logger.c is too generic. Can you provide the build logs so I can see what sort of conflicts these are? If it is a known popular project/environment, I can try to refactor names in LibOSDP so they don't conflict there.
 :)

We're building as a part of a C application using CMake and ARM GCC.
And maybe I will do that haha, I have already forked and made some changes to get them to play nicely, so I can just clean that up a bit!
Also while I have you and unrelated, is there API docs I can reference for the python implementation (specifically for the ControlPanel)?

Unfortunately no. I haven't gotten around to documenting it yet. The python sample cp_app.py should give you an idea of how to use the module. You can also have a look at the list of methods here and each of those methods have a doc string right above it which may provide some insights.
Just an FYI, there two kind/levels of Python support right now (honestly I don't know which is better, at different times either looks better than the other). Inside python/ there is a python-c-bindings module that exposes the raw C API over python with some (un)marshalling of C structures to python dicts. Here, you should find 1-1 python calls for the C methods. After this was developed, for ease of testing with pytest, I wrote some wrappers on top of this module that does some more device lifecycle management; this can be found at pytest/testlib/. Feel free to choose the one that best fits your need/taste.

This is all great info thank you! I will also close this issue and take a stab at implementing the logger framework port as well.


Update twitter status to contain the amount of karma gained that day.

This requires parsing the karma log.

Nominating this for twitter status milestone.


WIP: Expose type annotations in Python API

Work in progress. Help would be appreciated
Goal
Expose type annotations via a Python "stub" so that tools like mypy and PyCharm can infer whether appropriate types are passed to clingo's methods.
References:

https://www.python.org/dev/peps/pep-0561/
https://mypy.readthedocs.io/en/latest/installed_packages.html

I just learned that this exists. What I noted before was that the code completion engine I am using already uses type hints in the docstrings for semantic completion. I am using youcompleteme, which uses jedi under the hood. Ideally, it would be possible to generate the clingo.pyi file from the docstrings. Otherwise, maintaining this will be cumbersome.

Have you stopped working on this? I am not opposed to merging something but it needs to be maintainable. I do not want to maintain signatures in two places. Here is an idea how to generate the clingo.pyi file. Writing the clingo_abc.py file by hand seems okay to me. Otherwise, this also needs to be part of the install target.
import clingo
import types
import sys

def get_sig(entry):
    return entry.__doc__.strip().splitlines()[0]

functions = []
classes = []
modules = []
other = []

for name, entry in sorted(clingo.__dict__.items(), key=lambda x: x[0]):
    if name.startswith("_") and name != "__version__":
        continue
    if isinstance(entry, type):
        classes.append((name, entry))
    elif isinstance(entry, types.BuiltinFunctionType):
        functions.append((name, entry))
    elif isinstance(entry, types.ModuleType):
        modules.append((name, entry))
    else:
        other.append((name, entry))

sys.stdout.write("from typing import *\n")
sys.stdout.write("from clingo_abc import Application, Observer, Propagator\n\n\n")

for name, entry in other:
    if isinstance(entry, clingo.Symbol):
        sys.stdout.write("{}: Symbol\n".format(name))
    elif isinstance(entry, str):
        sys.stdout.write("{}: str\n".format(name))
    else:
        sys.stderr.write("unhandled entry: {}\n".format(name))

sys.stdout.write("\n\n")

for name, entry in functions:
    sys.stdout.write("def {}: ...\n".format(get_sig(entry)))

sys.stdout.write("\n\n")

for name, entry in classes:
    pass  # TODO

I am totally in favor of making this more maintainable and I'm still interested in updating it so that it can be ready for merging. However, I don't think I'll be able to work on it until the summer, since the academic year keeps me pretty busy

Let's see. I might have a look earlier. In principle, I want to make a release in the near future. To fully generate the stub file, I should also add signatures to the docstrings of some classes. For example, for the iterators and sequences.

I'll continue developing this in the feature/mypy branch. There is also a refined generator for the stub file in scratch/mypy now. If everything goes well, this will be part of the next release.

Closing this and continuing in the mentioned PR. If you like, you can give this a try. It should already work. There is just one todo left.


Add room in titlebar to click and drag window around when restored

SUMMARY: In VSCode for Windows, users are no longer able to easily drag a window around by clicking and dragging the title-bar.
The search input was added to the title-bar in a recent update.
CURRENT:

BEFORE:

Chrome addresses this by always leaving some blank space to the left of the minimize/restore/close buttons:

For me the window is draggable with a small area between the ... menubar entry and the Go Back button, and also with a thin strip below the command centre.

Well "recent" update isn't quite true. That input field is the "Command center" and was introduced in the June 22 update. I'm currently on my phone so I don't know what customization options there are or have been added in those 18 months, but probably you can find something with that information.
If nothing is available I'd appreciate an option for the bar to stay compact unless focused.

@LWChris thanks for the input. I updated the issue request to be more accurate.

duplicate https://github.com/microsoft/vscode/issues/169380


[CXF-8668]:Set SniHostCheck to false for SSLNettyClientTest

@jimma should we backport it to master and 3.5.x / 3.4.x with appropriate default? Thanks!

@jimma should we backport it to master and 3.5.x / 3.4.x with appropriate default? Thanks!

@reta , Yes.  I already sent PR to fix these branches.


Splash can't load the site

Splash can't load the site
That page www.zomato.com fails to load in splash

It returns HTTP 403 error for me (Access Denied), it looks more like a protection measure, not a Splash bug.

I've tried to open the mentioned site several times from 3 default Scrapinghub/Splash container instances located at localhost, Digital Ocean and AWD EC2 Container accordingly.
Results:

http://<IP_ADDRESS>:8050  - 200  everything is ok
Digital Ocean  - HTTP Error 504 (Gateway Time-out)
AWS EC2 Container - 403 forbidden

User-Agent headers change has no effect.
Another site https://heureka.sk shows the following results:

http://<IP_ADDRESS>:8050  - 200  everything is ok
Digital Ocean  - 200
AWS EC2 Container - 403 forbidden

Why Splash acts differently for various  hosting providers?

I've found the reason of 403 response for the second site  https://heureka.sk
They don't allow requests from US IP addresses. Splash  eu-central-1 based instance returned 200 status code.


ipfetch: update to fix outdated domain

Description of changes
Updated to latest version for fix of a subdomain change breaking functionality
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 322457 run on x86_64-linux 1

1 package built:

ipfetch


[jfpwor] Newly created class category not selected

From @dalehenrich slack comment:
I noticed that if I create a class in a not-yet-existing class category, the class is displayed in the current Class Category (in this case Rowan3Stub-Monticello PackageBrowser...
note no class category selected
the new YYY class category isn't visibile in list
have to close open browser to see the new class category ...

I'll need a server with the rowan stub code in it to make the fix.
Also, will need a way to know if the server has stub code loaded as anything testing this fix would require stub code.

Should be fully(?) fixed as of this commit https://github.com/GemTalk/JadeiteForPharo/commit/3a9864bd014fb9fdf172de3305f4815e8990f3d8


Multiple = generate excessive parentheses.

Expected
a = b = c = d = 7;

Actual
a = (b = (c = (d = 7)));

Steps To Reproduce
/// <summary>
/// Creates a simple class
/// </summary>
public class App
{
    // Use the [Ready] attribute to automatically
    // run this method when the page is ready.
    [Ready]
    public static void Main()
    {
        int a,b,c,d;
        a=b=c=d = 7;
    }
}

Related to #1353.

Yes, the same issue as #1353.


MAT-7672 solve vulnerability issue

MADiE PR
Jira Ticket: MAT-7672
(Optional) Related Tickets:
Summary
All Submissions

[x] This PR has the JIRA linked.
[ ] Required tests are included.
[x] No extemporaneous files are included (i.e Complied files or testing results).
[x] This PR is merging into the correct branch.
[ ] All Documentation needed for this PR is Complete (or noted in a TODO or other Ticket).
[ ] Any breaking changes or failing automations are noted by placing a comment on this PR.

DevSecOps
If there is a question if this PR has a security or infrastructure impact, please contact the Security or DevOps engineer assigned to this project to discuss it further.

[x] This PR has NO significant security impact (i.e Changing auth methods, Adding a new user type, Adding a required but vulnerable package).
[ ] All CDN/Web dependencies are hosted internally (i.e MADiE-Root Repo).

Reviewers
By Approving this PR you are attesting to the following:

Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose.
The tests appropriately test the new code, including edge cases.
If you have any concerns they are brought up either to the developer assigned, security engineer, or leads.

need to make a release


Impala schema selection "Mark as Active" broken

Impala schema selection "Mark as Active" broken.
DBeaver 4.2.3 installed on Mac OSX Sierra.
Repro steps:

Set up working JDBC connection to Cloudera Impala.
Expand the new data source.
Right click on a schema, click Mark as Active.
Query:
select * from tableInSelectedSchema

Note that this query does not work. The schema is not Active. (Do not select * from selectedSchema.tableInSelectedSchema; as this WILL obviously work)
Also, the text in the dbeaver toolbar does not change from "" to the selected schema.

This is a feature of Cloudera Hue, which allows setting the Active schema in the Impala Query editor.
Having this feature would boost efficiency in developing queries.

Noting that this hasn't worked since I started using DBeaver in July 2017.

DBeaver doesn't have preconfigured Impala driver. Thus there is no configuration for active schema read/write.
I think we need Cloudera driver in default DBeaver config.
Could you post what driver configuration do you use?

Hi! I am using the Cloudera JDBC Driver for Impala, downloadable here:
I am using the Cloudera Impala JDBC 41_2.5.34 driver, specifically.
(I have not tested newer versions)
Here it is:
Cloudera_ImpalaJDBC41_2.5.34 (2).zip
Driver config:

List of JARs:

Libraries:

Create new connection wizard:

Thanks.
I'll add Impala config template in the next DBeaver version.

Thank you!


Bulbs serial number

Is it possible to expose the bulbs serial numbers as well using aiohue? I don't see that in the model, but I'm not sure if it's even possible to do using the bridge api

If you are referring to the 6 digit codes on the lamp, those are not available on the bridge API

Yes, correct.. Thanks.. Guess we can just close this one then :)


YouTube: Unable to log in: HTTP Error 400: Bad Request

Before you mark as duplicate
I've reviewed the previously "closed as duplicate" issues and cannot seem to find the original issue to comment on. Please document the issue # or docs to reference, so that I can go there. I'd be willing to comment on already closed issues to help others with the same issue as well.

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.01.15
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl -v -u "........................." -p "...................................." --download-archive "channels/hay-video.txt" --mark-watched --cookies "cookies.txt" -i -o "channels/%(playlist)s/%(title)s-%(id)s.%(ext)s" ""https://www.youtube.com/channel/UCCpKoJIkTW7eYk6Jxvhjz6g/videos""
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-u', 'PRIVATE', '-p', 'PRIVATE', '--download-archive', 'channels/hay-video.txt', '--mark-watched', '--cookies', 'cookies.txt', '-i', '-o', 'channels/%(playlist)s/%(title)s-%(id)s.%(ext)s', 'https://www.youtube.com/channel/UCCpKoJIkTW7eYk6Jxvhjz6g/videos']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.01.15
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg git-2019-10-21-a174e5f, ffprobe git-2019-10-21-a174e5f
[debug] Proxy map: {}
[youtube:channel] Downloading login page
[youtube:channel] Looking up account info
[youtube:channel] Logging in
WARNING: Unable to log in: HTTP Error 400: Bad Request
[youtube:channel] UCCpKoJIkTW7eYk6Jxvhjz6g: Downloading channel page
[youtube:playlist] Downloading login page
[youtube:playlist] Looking up account info
[youtube:playlist] Logging in
WARNING: Unable to log in: HTTP Error 400: Bad Request
[youtube:playlist] UUCpKoJIkTW7eYk6Jxvhjz6g: Downloading webpage
[download] Downloading playlist: Uploads from Hay Video
[youtube:playlist] playlist Uploads from Hay Video: Downloading 2 videos
[download] Downloading video 1 of 2

Description

Seeing login warnings repeatedly when downloading from YouTube. Used to work without this error a few weeks ago, updated to the latest version and it is causing these warnings. I have logged out in a browser on the same machine and logged back in successfully using the same credentials. I have updated the cookies.txt after successful manual login as well.
It does seem to download the videos properly.

Found related issue #21313 as well


Combining Partial<>, generics and conditional types in spread arguments causes TS2370.

Bug Report
🔎 Search Terms
2370 generic conditional partial
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about partial

⏯ Playground Link

Playground link with relevant code
💻 Code

type AlwaysArrayType<T> = T extends unknown [] ? T : unknown[];

function thisWorks<T>(...args: AlwaysArrayType<T>) {}
function thisToo<T>(...args: Partial<unknown[]>) {}
function andThisAsWell<T extends unknown[]>(...args: Partial<T>) {}

function butThisDoesnt<T>(...args: Partial<AlwaysArrayType<T>>) {}
//                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^----- TS2370: A rest parameter
//                                                                  must be of an array type.

🙁 Actual behavior
TS2370
🙂 Expected behavior
No error, since the passed in type already is an array type.

A Partial of some type assignable to unknown[] is not necessarily an array.

Ah I see 😅 thanks!


Hide immunization card for patients over 5yrs

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #https://github.com/opensrp/fhir-resources/issues/814
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

@DebbieArita I tested this and it works well. I will however create and issue to figure out why the view renderer does not pass the visible attribute as expected.
cc: @ellykits

@DebbieArita I tested this and it works well. I will however create and issue to figure out why the view renderer does not pass the visible attribute as expected.
cc: @ellykits

We managed to resolve this... @dubdabasoduba

@DebbieArita Please add another PR targeting the release branch.


Add an icon for opening each exporter results

For each type of exporter, add an icon that when clicked will open the relevant exporter

Csv and json will open the file
Ynab will open the ynab website in the correct url in the default browser
Sheets will open the google sheet we are exporting to in the default browser

Good idea!

Sure, no problem...
We are talking about this screen right?:

I will take a look later and let you know if I have any question about how should I implement it

I can take it.

Fixed in #445

@ErezBiren Tried it now.
First, it is working even if an exporter is disabled, I don't know if it is the wanted behavior.
Second, nothing happened when clicking on the JSON exporter.

@baruchiro @brafdlog
would you like the open result button to be hidden or disabled when the exporter is disabled?

The JSON exporter results seem to work fine for me.
Did you check that the file path is correct?

1 makes sense to me
Havent tried the json yet, can try later

Ok, but to hide or should we disable the button?

Hide


Segmentation Fault during iOS release build

Steps to Reproduce

flutter build ios --release on CI w/ Xcode 11.4

Logs
Build will fail with a Segmentation Fault:
    ===== CRASH =====
    si_signo=Segmentation fault: 11(11), si_code=1, si_addr=0x400113680040
    version=2.8.0-dev.20.10 (beta) (Mon Apr 20 12:00:58 2020 +0200) on "macos_x64"
    pid=18384, thread=5383, isolate=(null)(0x0)
    isolate_instructions=0, vm_instructions=10ef83500
      pc 0x000000010f46c46f fp 0x00007000052dde70 dart::MarkingVisitorBase<true>::VisitPointers(dart::RawObject**, dart::RawObject**)+0x6f
      pc 0x000000010f46c00d fp 0x00007000052ddea0 dart::MarkingVisitorBase<true>::DrainMarkingStack()+0x12d
      pc 0x000000010f46ca6c fp 0x00007000052ddf10 dart::ConcurrentMarkTask::Run()+0x6c
      pc 0x000000010f29f95d fp 0x00007000052ddf40 dart::ThreadPool::Worker::Loop()+0x3d
      pc 0x000000010f29f790 fp 0x00007000052ddf70 dart::ThreadPool::Worker::Main(unsigned long)+0x70
      pc 0x000000010f209011 fp 0x00007000052ddfb0 dart::ThreadStart(void*)+0x61
      pc 0x00007fff6b301109 fp 0x00007000052ddfd0 _pthread_start+0x94
      pc 0x00007fff6b2fcb8b fp 0x00007000052ddff0 thread_start+0xf
    -- End of DumpStackTrace
    Target kernel_snapshot failed: Exception: Errors during snapshot creation: null
    build failed.
    Command PhaseScriptExecution failed with a nonzero exit code
    note: Using new build system
    note: Building targets in parallel
    note: Planning build
    note: Constructing build description

[✓] Flutter (Channel beta, v1.17.0-3.2.pre, on Mac OS X 10.15.4 19E287, locale en)
    • Flutter version 1.17.0-3.2.pre at /Users/anka/flutter
    • Framework revision 2a7bc389f2 (3 days ago), 2020-04-21 20:34:20 -0700
    • Engine revision 4c8c31f591
    • Dart version 2.8.0 (build 2.8.0-dev.20.10)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /usr/local/share/android-sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_SDK_ROOT = /usr/local/share/android-sdk
    • Java binary at: /Library/Java/JavaVirtualMachines/adoptopenjdk-8.jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_252-b09)
    • All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
    • Xcode at /Applications/Xcode-11.4.1.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    • CocoaPods version 1.9.1
[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
[!] Connected device
    ! No devices available
! Doctor found issues in 2 categories.

Please re-file against dart.


Meteor is fundamentally broken

https://github.com/meteor/meteor/issues/5124
Security updates for the version of Node that this application is running on end this October 1 (2016-10-01) -- https://github.com/nodejs/LTS#lts_schedule
If they get it fixed by that time then fine. But if not, there needs to be some very serious consideration given to what needs to be done to move this to a platform that isn't so bound by non-native code.
I'll try to devote some time to that endeavor in the coming months.

According to this post Meteor 1.4 will use Node 4. I'll try the beta once it's out so we can migrate it when it gets released.
Here's the Meteor 1.4 release milestone: https://github.com/meteor/meteor/milestones/Release 1.4
And here's the pull request for the upgrade to Node 4: https://github.com/meteor/meteor/pull/6921

That's good. But my opinion remains the same -- if it can't track the platform it runs on then it is broken.
Personally, I use Marko for templating, Hapi for a framework, and Objection if I want some sort of ORM-like tool.
But I have to say, Mongoose + MongoDB is a rather speedy combination. I was able to dramatically reduce response times in an application with it -- https://github.com/jscas/cas-server-mongo-repositories

Meteor 1.4 runs on node.js 4 (LTS).


[ZEPPELIN-1376] Add proxy credentials for dependency repo for corporate firewall use-cases

What is this PR for?
When using Zeppelin behind corporate firewall, sometimes the dependencies download just fails silently. This PR has 2 objectives:

add proxy credentials information for dependencies repo
raise clear error message in case of dependencies download failure

There are 3 commits.
The first one add extra inputs in the form for adding new repository

The second commit fixes some issues and display a clear and explicit error message when download of dependencies fail.
Before that, when the download fails, we can see the below behaviour

the error message is displayed twice because the call twice the method checkDownloadingDependencies();. One in the success callback of:

 $scope.updateInterpreterSetting = function(form, settingId) {
              ...
            $http.put(baseUrlSrv.getRestApiBase() + '/interpreter/setting/' + settingId, request)
              .success(function(data, status, headers, config) {
                $scope.interpreterSettings[index] = data.body;
                removeTMPSettings(index);
                thisConfirm.close();
                checkDownloadingDependencies();
                $route.reload();
              })
              .error(function(data, status, headers, config) {
             ...
    };

Another call is inside success callback of getInterpreterSettings()
var getInterpreterSettings = function() {
      $http.get(baseUrlSrv.getRestApiBase() + '/interpreter/setting')
      .success(function(data, status, headers, config) {
        $scope.interpreterSettings = data.body;
        checkDownloadingDependencies();
      }).error(function(data, status, headers, config) {
      ....

The problem is that $route.reload(); in the success callback of updateInterpreterSetting() will trigger init() then getInterpreterSettings() so checkDownloadingDependencies() is called twice.
I remove the call to checkDownloadingDependencies() from success callback of updateInterpreterSetting()
The second modification is on class DependencyResolver. In the screen capture above, we get a cryptic NullPointerException coming from DefaultRepositorySystem. I now catch this NPE to wrap it into a more sensible and clearer exception:

  public List<ArtifactResult> getArtifactsWithDep(String dependency,
    Collection<String> excludes) throws RepositoryException {
    Artifact artifact = new DefaultArtifact(dependency);
    DependencyFilter classpathFilter = DependencyFilterUtils.classpathFilter(JavaScopes.COMPILE);
    PatternExclusionsDependencyFilter exclusionFilter =
            new PatternExclusionsDependencyFilter(excludes);

    CollectRequest collectRequest = new CollectRequest();
    collectRequest.setRoot(new Dependency(artifact, JavaScopes.COMPILE));

    synchronized (repos) {
      for (RemoteRepository repo : repos) {
        collectRequest.addRepository(repo);
      }
    }
    DependencyRequest dependencyRequest = new DependencyRequest(collectRequest,
            DependencyFilterUtils.andFilter(exclusionFilter, classpathFilter));

 //Catch NPE thrown by aether and give a proper error message
    try {
      return system.resolveDependencies(session, dependencyRequest).getArtifactResults();
    } catch (NullPointerException ex) {
      throw new RepositoryException(String.format("Cannot fetch dependencies for %s", dependency));
    }
  }

The result is much more cleaner

The last commit is just doc update

What type of PR is it?
[Improvement]
Todos

[ ] - Code Review
[ ] - Simple test with no Internet connection
[ ] - Test within a corporate firewall env with a third-party dependency, requiring download

What is the Jira issue?
ZEPPELIN-1376
How should this be tested?
Simple test

git fetch origin pull/1369/head:WebProxy
git checkout WebProxy
mvn clean package -DskipTests
bin/zeppelin-daemon.sh restart
disconnect from the Internet (pull out the cable, shutdown wifi ...)
add a random dependency to the Spark interpreter (take info.archinnov:achilles-core:4.2.2 for example)
validate the change, you should see an error popup on the top-right corner saying that Zeppelin cannot download the dependency

Corporate firewall test

follow the steps above for simple test
create a new repository (see how to here) and set the proxy information
retry the steps above to ensure that the download is successful

Screenshots (if appropriate)
See above
Questions:

Does the licenses files need update? --> NO
Is there breaking changes for older versions? --> NO
Does this needs documentation?  --> YES, DONE

LGTM - how could we have automated tests for this?

@felixcheung
That's the main issue I'm facing now, how to automate test with corporate firewall ... I'm going to ask someone to test the this PR at his place (he has a corporate firewall)
But apart from manual testing the firewall proxy, I don't see an easy way to create integration test for this feature, unless we build a Docker image of Zeppelin with a Docker image of a HTTP Proxy server ...

@doanduyhai Could you please specify the title of this PR?

@jongyoul Sorry, it was a bad copy/paste between Github & JIRA, fixed now

@astroshim @minahlee Can you review this PR?

About the second commits, Is it intended behavior that the error message bar is stacked whenever save the other interpreter setting?

@astroshim
It is explained above:

The problem is that $route.reload(); in the success callback of updateInterpreterSetting() will trigger init() then getInterpreterSettings() so checkDownloadingDependencies() is called twice.

Every time you save an interpreter setting, the JS script calls $route.reload() so function checkDownloadingDependencies() is called again, creating a new error popup.
In a normal workflow, the user will just remove the popup by clicking on the x icon.
As an enhancement, I propose:

set the property combineDuplications  from ngToast (http://tamerayd.in/ngToast/) to true
also, we can set a timeout so that the error popup disappear after a while

What do you think ?

Wow very quick response!!:
I think set combineDuplications true is better but I don't have much talent in UI.:)

Ok let me push a new commit with the combineDuplications = true

@astroshim Done, now the popup error is only displayed once even if you update many other interpreters.

@doanduyhai Good. I am going to check first commit(proxy) soon.

@doanduyhai BTW how come only spark interpreter shows error message bar?

Proxy repo tested and it works properly. :+1:

@astroshim

BTW how come only spark interpreter shows error message bar?

The error popup applies to all interpreters, try to add a fictive dependency to angular or shell interpreter and you'll see also the error popup. The error message mentions clearly the <group>.<interpreter> that has the dependency issue

@doanduyhai Could you check the infinitely downloading library problem(like following screen shot)?
Strangely this problem happens all interpreter except spark. Is this problem happen only to me?

I also checked the interpreter setting page like @astroshim  did.
(edit interpreter -> add one:two:1.1.1 to the dependency table -> save )
Two things are different with master branch.

Only Spark interpreter throw the ng-toast error message.
The other interpreters' status circle is looping infinitely. It need to be reload to turn to red and also don't show the ng-toast error message.

Let me investigate the root cause, it should be some Javascript subtle bug

OKKKK I've found the issue, it's tricky.
I have added a non-existing dependency to the angular interpreter and I can also see a spinning circle forever. With Web Dev Console, I can see the return JSON result to the call GET /interpreter on the REST API. Below is the JSON result for Angular:

...
    "status":"DOWNLOADING_DEPENDENCIES",
    "errorReason":"Could not find artifact one:two:jar:1.1.1 in central (http://repo1.maven.org/maven2/)",
...

So the status is wrong, it should be "ERROR" instead of "DOWNLOADING_DEPENDENCIES". Let me debug the server code

@astroshim @AhyoungRyu it tooks me hours but I have found the root cause of the issue and it's pretty ugly. Ready to hear ?

First issue is we do not set the InterpreterSetting status and errorReason together. For example here: https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java#L398

    ...
    setting.setStatus(InterpreterSetting.Status.DOWNLOADING_DEPENDENCIES);
    interpreterSettings.put(setting.getId(), setting);
    ...

So here we set the status to DOWNLOADING_DEPENDENCIES but the errorReason stills contains the previous value, possible something like "Cannot download dependency x.y.z ....". That explains why we see
...
    "status":"DOWNLOADING_DEPENDENCIES",
    "errorReason":"Could not find artifact one:two:jar:1.1.1 in central (http://repo1.maven.org/maven2/)",
...

Second issue is more serious, it is concurrency issue. We set the status of the dependency download in a separate thread: https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java#L400-L433

...
catch (Exception e) {
            logger.error(String.format("Error while downloading repos for interpreter group : %s," +
                    " go to interpreter setting page click on edit and save it again to make " +
                    "this interpreter work properly.",
                setting.getGroup()), e);
            setting.setErrorReason(e.getLocalizedMessage());
            setting.setStatus(InterpreterSetting.Status.ERROR);
...

The problem is that the initial status is set to DOWNLOADING_DEPENDENCIES and then the interpreter status is returned by the caller class to the client:  https://github.com/apache/zeppelin/blob/master/zeppelin-server/src/main/java/org/apache/zeppelin/rest/InterpreterRestApi.java#L113-L135
I added extra log message to debug:
            ...
            logger.info("************** DEBUG_ZEPPELIN setting status ERROR");
            setting.setStatus(InterpreterSetting.Status.ERROR);
            setting.setErrorReason(e.getLocalizedMessage());
            ...

    ...
    logger.info("************** DEBUG_ZEPPELIN setting debug " + setting.toDebugString());
    return new JsonResponse<>(Status.OK, "", setting).build();
    ...

And below is what is displayed in the logs:
INFO [2016-09-07 12:35:26,151] ({qtp240166646-20} InterpreterRestApi.java[updateSetting]:132) - ************** DEBUG_ZEPPELIN setting debug DOWNLOADING_DEPENDENCIES :
ERROR [2016-09-07 12:35:26,539] ({Thread-25} InterpreterFactory.java[run]:433) - Error while downloading repos for interpreter group : angular, go to interpreter setting page click on edit and save it again to make this interpreter work properly. : Could not find artifact one:two:jar:1.1.25 in central (http://repo1.maven.org/maven2/)
org.sonatype.aether.resolution.DependencyResolutionException: Could not find artifact one:two:jar:1.1.25 in central (http://repo1.maven.org/maven2/)
        at org.sonatype.aether.impl.internal.DefaultRepositorySystem.resolveDependencies(DefaultRepositorySystem.java:375)
        at org.apache.zeppelin.dep.DependencyResolver.getArtifactsWithDep(DependencyResolver.java:164)
        at org.apache.zeppelin.dep.DependencyResolver.loadFromMvn(DependencyResolver.java:116)
        at org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:79)
        at org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:96)
        at org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:88)
        at org.apache.zeppelin.interpreter.InterpreterFactory$3.run(InterpreterFactory.java:425)
Caused by: org.sonatype.aether.resolution.ArtifactResolutionException: Could not find artifact one:two:jar:1.1.25 in central (http://repo1.maven.org/maven2/)
        at org.sonatype.aether.impl.internal.DefaultArtifactResolver.resolve(DefaultArtifactResolver.java:537)
        at org.sonatype.aether.impl.internal.DefaultArtifactResolver.resolveArtifacts(DefaultArtifactResolver.java:216)
        at org.sonatype.aether.impl.internal.DefaultRepositorySystem.resolveDependencies(DefaultRepositorySystem.java:358)
        ... 6 more
Caused by: org.sonatype.aether.transfer.ArtifactNotFoundException: Could not find artifact one:two:jar:1.1.25 in central (http://repo1.maven.org/maven2/)
        at org.sonatype.aether.connector.wagon.WagonRepositoryConnector$4.wrap(WagonRepositoryConnector.java:971)
        at org.sonatype.aether.connector.wagon.WagonRepositoryConnector$4.wrap(WagonRepositoryConnector.java:966)
        at org.sonatype.aether.connector.wagon.WagonRepositoryConnector$GetTask.flush(WagonRepositoryConnector.java:707)
        at org.sonatype.aether.connector.wagon.WagonRepositoryConnector$GetTask.flush(WagonRepositoryConnector.java:701)
        at org.sonatype.aether.connector.wagon.WagonRepositoryConnector.get(WagonRepositoryConnector.java:452)
        at org.sonatype.aether.impl.internal.DefaultArtifactResolver.resolve(DefaultArtifactResolver.java:456)
        ... 8 more
 INFO [2016-09-07 12:35:26,541] ({Thread-25} InterpreterFactory.java[run]:437) - ************** DEBUG_ZEPPELIN setting status ERROR

So the DOWNLOADING_DEPENDENCIES status is returned to the client before the download error status is set in a separate thread.
This issue is indeed already there in the master and it not related to this pull request

@astroshim @AhyoungRyu
Ok I've found the issue, it's my bad:
 for (var setting = 0; setting < $scope.interpreterSettings.length; setting++) {
        var setting = $scope.interpreterSettings[setting];
        if (setting.status === 'DOWNLOADING_DEPENDENCIES') {
          isDownloading = true;
        } else if (setting.status === 'ERROR') {
          ngToast.danger({content: 'Error setting properties for interpreter \'' +
            setting.group + '.' + setting.name + '\': ' + setting.errorReason,
            verticalPosition: 'top', dismissOnTimeout: false});
        }
      }

I'm using the variable setting as an index for the iteration of the for loop as well as the variable for the setting object itself, which breaks awfully the for loop.
Now it's fixed, see below

@AhyoungRyu @astroshim Is the fix ok for you now ? The build is green

@doanduyhai Sorry to keep you waiting.
It works like a charm now but the error count number at the beginning of message appears a little strange like following screen shot.

@astroshim
The number you see in front of each error popup is normal, it comes from the fact that now we set combineDuplications  = true for ngToast. The number corresponds to the time the error popup is duplicated

@doanduyhai I know that number means but as you can see the screenshot i attached, the number seems doesn't match with count.

@astroshim

the number seems doesn't match with count

What do you mean by this ? Which number did you expect to see ?
On the animated screencapture, the count started with 1 then 2 for both popups.
Then you close one so the count is reset to 0 but the 2nd popup is still there and continue counting to 4.
Do not forget that the checkDownloadingDependencies() method is called multiple times:

everytime an interpreter setting is updated (2 times indeed)
everytime the status returned by the server is DOWNLOADING_DEPENDENCIES

@doanduyhai Could you take a look at the following screenshot?
After the alluxio and angular dependency error occurs, I just saved alluxio interpreter setting several times but the angular error count is more then alluxio.
I thought the error count would be same.

and Is it possible to close the pop-up when user move to a different page?

@doanduyhai Sorry for my late response and thanks for the kind reminder.
I tested again and checked the below issues I mentioned are gone :D

(edit interpreter -> add one:two:1.1.1 to the dependency table -> save )
Two things are different with master branch.

Only Spark interpreter throw the ng-toast error message.
The other interpreters' status circle is looping infinitely. It need to be reload to turn to red and also don't show the ng-toast error message.

But I found another issues in the latest update.

edit Alluxio interpreter -> add aaa:bbb:1.1.1  to the dependency field ->  go to another page like "Credential" or "Configuration" ->
The ng-toast error message is still shown as @astroshim said in the above comment. How about adding some timeout property to the message?

edit Alluxio -> add aaa:111:1.1.1 & bbb:<IP_ADDRESS> to the dependency field -> save
I think the number of errors and reasons need to be same. Currently, while the number of errors keeps increasing whenever I save the change, the only one reason "Could not find artifact aaa:111:jar:1.1.1 in central (http://repo1.maven.org/maven2/)" is shown. Need to show "Could not find artifact bbb:<IP_ADDRESS> in central (http://repo1.maven.org/maven2/)" as well if we want to show the number of errors.

The number of error should be 2 in the above case I guess. But as you can see the number is 1 since checkDownloadingDependencies() is called only one time.

@AhyoungRyu

The ng-toast error message is still shown as @astroshim said in the above comment. How about adding some timeout property to the message?

I knew that there is a timeout option for ng-toast but I chose not use it. The simple reason is that I consider the dependency issue is a serious issue so we want the user to explicitly/manually close the error popup and acknowledge that he is aware about the download issue.
What is your opinion about that ? Adding a timeout to automatically close the error popup is very easy.

Need to show "Could not find artifact bbb:<IP_ADDRESS> in central (http://repo1.maven.org/maven2/)" as well if we want to show the number of errors.

There is a simple explanation for this. sonatye.aether library tries to download the dependencies one after another and raise an exception as soon as it cannot find/download a dependency. In your example, we'll see the error popup for the 2nd dependency if the download of the 1st is successful
 synchronized (repos) {
      for (RemoteRepository repo : repos) {
        collectRequest.addRepository(repo);
      }
    }
    DependencyRequest dependencyRequest = new DependencyRequest(collectRequest,
            DependencyFilterUtils.andFilter(exclusionFilter, classpathFilter));
    try {
      return system.resolveDependencies(session, dependencyRequest).getArtifactResults();
    } catch (NullPointerException ex) {
      throw new RepositoryException(String.format("Cannot fetch dependencies for %s", dependency));
    }

We cannot change the behavior of sonatype.aether. We can change our source code to download dependency one by one but it is very slow and require changing the interface of a lot of methods

@astroshim @AhyoungRyu
The wrong counting on error popup is now fixed. Every time you

reload the interpreter page
update an interpreter setting

all the error popup are removed and the Zeppelin re-display error popup only when necessary
Also, when you navigate to different pages, the error popups are removed.
The latest commit reset the ngToast every time you "enter" a page. With this change we don't need to set a timeout on the error popup anymore

@astroshim @AhyoungRyu
Please see the final result, it should make you both happy :)

@AhyoungRyu @astroshim
Please tell me if it's ok so that I can merge. We need to merge this PR before I can move on the refactoring task (of changing synchronized blocks by appropriate datastructures)

@doanduyhai So sorry for my late response ㅠ_ㅠ
Tested again and checked

when you navigate to different pages, the error popups are removed.

the above issue is gone! 👍
However, so sadly another issue is emerged. After removing the wrong dependency, the error message should be gone. But I think it's not. Here is the procedure how you can reproduce this.

edit Alluxio interpreter dependency -> type a:b:1.1.1 and save -> surely the ng-toast error message is shown up
edit Angular interpreter dependency -> type a:b:1.1.1 and save -> ng-toast error message is also shown up for this interpreter
remove those wrong dependencies from both Alluxio and Angular -> save -> the status circles for that two interpreters are turning to green
but the ng-toast error messages are still shown up even if I reload the page multiple times

@AhyoungRyu
I followed your procedure but it works perfectly for me, see below screenshot. Did you clean your browser cache when testing the new commit ? Maybe there is some old JS code left-over

@doanduyhai Yeah your screenshot is saying so. Let me test it again. Thanks for your quick response!

@doanduyhai Checked again and it works well! After removing the wrong dependency, the error message doesn't show up anymore.

OK I'll merge if no more remarks

Those changes are only for dependencies given in the interpreter definition, right?
Using a %spark.dep section in a notebook seems to uses a SparkDependencyContext which does not inherit from AbstractDependency.* and so does not use the proxy definition at all. Right?
Wouldn't it be useful to give the %dep section access to the proxy too?


Enhance Visual Style and Fix Bugs on "Badge Explorer" Page - Part 2

We aim to enhance the visual style of the "Badge Explorer" page implemented in another issue (https://github.com/cs-internship/cs-system/issues/50). The desired layout includes a collapsible tree menu on the left for navigating categories and badges, and a section on the right for displaying badge details when a badge is clicked.
This issue focuses on improving certain visual details and ensuring the page is bug-free.
The improvements are divided into two issues.
Improvements We Aim to Resolve in This Issue:
Right Section:

Use the default Bootstrap font size and spacing for the titles and descriptions in "Badge Requirement", "Activity Requirement" and "Approving Steps".
If the badge has no activity or badge requirements, display a message indicating that there are no such requirements.
When showcasing a badge in the badge requirement section, use the default "Bootstrap Badge" style with a number of badge requirements in front of it.
Remove excess white space at the end of the requirement sections.
Reduce the space between the left and right sections of the page.
Use the "Bootstrap Card" for the different steps in the "Approving Steps" section.

Additional Notes:

Ensure consistency with Bootstrap framework guidelines, as the project utilizes Bootstrap as its style framework.
Avoid adjusting default Bootstrap values as much as possible.
Ensure the style is responsive to adapt to various screen sizes.

Steps to Participate:

Comment on this issue with your estimated timeframe for completing the task.
The administrators will review your proposed timeline.
If approved, the issue will be assigned to you.
Upon successful completion within the agreed timeframe, you'll receive the issue reward.

Contribution Guidelines:

Please refer to the Contribution Guidelines for detailed instructions on contributing to the project.

Deliverables:

The PR for this issue should be approved by @mehrandvd. He will announce his approval on this issue.

Reward:

The reward will be $15, which will be paid in $USDC (ERC20).

Update:

We don't want the "Bootstrap Badge" to have the primary style or be clickable. We want the badge to have the default, non-clickable styles.

Hello @mehrandvd
I will deliver this task in 10 days (by 2024 - 8 - 15).

Hello @mehrandvd, This task is ready

Hello @mehrandvd I will deliver this task in 2 days (by 2024 - 8 - 7).

Great, it's yours now :)

Thanks @mehrandvd

Hello @mehrandvd and @Ali-Sdg90
This #56 issue is done and the link to the video: https://www.youtube.com/watch?v=mWk0pcQ7gLc

It looks good to me, @Ali-Sdg90 do you confirm this?

It looks good to me, @Ali-Sdg90 do you confirm this?

Yes. I confirm all items in the issue have been addressed correctly🎉

Great, @ZahraaaRezaeiii please send your wallet address for the reward :)

Thanks  @mehrandvd
This is my USDC receive address: 0xd616044e54e215a1AbFfb290082639BA986ea45f

@ZahraaaRezaeiii Your 15$ reward for this task is paid now. Congrats :)
https://etherscan.io/tx/0x0e990ffd784eac0c297fde26304e8a385a7bbae7de4699865ceed9c1c88c0e40

@ZahraaaRezaeiii Your 15$ reward for this task is paid now. Congrats :)
https://etherscan.io/tx/0x0e990ffd784eac0c297fde26304e8a385a7bbae7de4699865ceed9c1c88c0e40

Thank you so much! I'm really glad to have completed the task successfully. I appreciate the reward! 😊


Summing Merge Tree table does not return correct rows

Hi Team,
I am using your example to understand the Summing Merge Tree. I executed statements into file located at https://github.com/yandex/ClickHouse/blob/master/dbms/tests/queries/0_stateless/00043_summing_empty_part.sql
CREATE DATABASE IF NOT EXISTS test;
DROP TABLE IF EXISTS test.empty_summing;
CREATE TABLE test.empty_summing (d Date, k UInt64, v Int8) ENGINE=SummingMergeTree(d, k, 8192);

INSERT INTO test.empty_summing VALUES ('2015-01-01', 1, 10);
INSERT INTO test.empty_summing VALUES ('2015-01-01', 1, -10);

OPTIMIZE TABLE test.empty_summing;
SELECT * FROM test.empty_summing;

INSERT INTO test.empty_summing VALUES ('2015-01-01', 1, 4),('2015-01-01', 2, -9),('2015-01-01', 3, -14);
INSERT INTO test.empty_summing VALUES ('2015-01-01', 1, -2),('2015-01-01', 1, -2),('2015-01-01', 3, 14);
INSERT INTO test.empty_summing VALUES ('2015-01-01', 1, 0),('2015-01-01', 3, 0);

OPTIMIZE TABLE test.empty_summing;
SELECT * FROM test.empty_summing;

DROP TABLE test.empty_summing;

I am expecting below for final select statement:
d				k		v
2015-01-01		1		0
2015-01-01		2		-9
2015-01-01		3		0

however below result is being shows at console.
SELECT *
FROM test.empty_summing

┌──────────d─┬─k─┬──v─┐
│ 2015-01-01 │ 2 │ -9 │
└────────────┴───┴────┘

Is this a bug OR expected behaviour. Please help me to understand the process.

This behaviour is expected. As documentation says:

If the values were null in all of these columns, the row is deleted. (The exception is cases when the data part would not have any rows left in it.)


browserifyOptions mapping unclear

I've only figured out two working browserifyOptions by reading stackoverflow posts
standalone
debug
The readme links to https://github.com/substack/node-browserify#usage
However what are they, in JSON?
For example I want to use the option:
--no-browser-field

What is it supposed to look like in the options?
    browserifyOptions: {
      nobrowserfield: true
    }

That doesn't seem to work. Should one include the dashes, or no dashes? Are they supposed to be camel-case? Is there a list of options that are ported over?

Same here. This needs to be elaborated better.

There is no mapping; browserifyOptions is directly passed to the browserify API. If you need further documentation, you'll have to look at browserify itself. I agree that the browserify docs are lacking in this respect, but there is nothing grunt-browserify can do about this.
From a superficial glance at the browserify source, suppose that the option @mflux is looking for is browserField: false, but I'd have to try it out to be sure.


Malformed Pester It ... -Skip (test) leads to invalid cast

Bug Report
After this morning's Update-module DbaChecks:
  [-] Error occurred in Context block 0ms
    InvalidCastException: Invalid cast from 'System.Boolean' to 'System.Management.Automation.ScriptBlock'.
    PSInvalidCastException: Cannot convert value "False" to type "System.Management.Automation.ScriptBlock". Error: "Invalid cast from 'System.Boolean' to 'System.Management.Automation.ScriptBlock'."
    ArgumentTransformationMetadataException: Cannot convert value "False" to type "System.Management.Automation.ScriptBlock". Error: "Invalid cast from 'System.Boolean' to 'System.Management.Automation.ScriptBlock'."
    ParameterBindingArgumentTransformationException: Cannot process argument transformation on parameter 'Test'. Cannot convert value "False" to type "System.Management.Automation.ScriptBlock". Error: "Invalid cast from 'System.Boolean' to 'System.Management.Automation.ScriptBlock'."
    at <ScriptBlock>, C:\foo\dbachecks\checks\Instance.Tests.ps1: line 333
    at DescribeImpl, C:\Program Files\WindowsPowerShell\Modules\Pester\4.7.3\Functions\Describe.ps1: line 199

General Troubleshooting steps

[X] Verified running the latest release of dbachecks?

[X] Verified errors are not related to permissions?

[X] Can duplicate in new/clean PowerShell session (clean = powershell -NoProfile)?

Version Information

Operating System : windows 10
PowerShell Version: 5.1
Pester Version: 4.7.3

Steps to Reproduce
If you run the memdump check, you should get the failure regardless of SQL version.
Description of Bug
If you look at Instance.tests.ps1, line 333, the culprit is the -Skip ($InstanceSMO.Version.Major -lt 10), with that space character after the p and before the (.
Pester's parsing of the It keyword gets confused because -Skip is a switch and doesn't usually take a parameter value. The Pester/parser thinks that ($InstanceSMO.Version.Major -lt 10) is supposed to be the {script-block}, tries to cast a boolean to a script block and dies.
Pester needs a colon between the p and the ( :
-Skip:($InstanceSMO.Version.Major -lt 10)
I also see that the line is recently altered, so it's suspicious.
Preliminary tests verify my theory; I'll be submitting a patch shortly.
<--
PLEASE include as much information as possible if this is a bug
Explain what it does that you would like it not to do
or
What it doesnt do that you would like it to do :-)
-->

Thank you :-)
I had caught and resolved this already with https://github.com/sqlcollaborative/dbachecks/commit/310f02ee5d5c5e74563bf32b86737afeff369332


[Feat]Quick links in footer

Description
Present quick links link are not correctly mapped. I would like to enhance the footer.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[ ] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @pavitraag, this issue has already been assigned to someone. Please look at any other issue, and I will be happy to assign it to you.

im interested please assign me, ill share my proposed solution as soon as u ping me

hey @pavitraag I hope you are doing well!

It has been 2 days and you haven't responded to anything. Unfortunately, I have to close this issue because there are many duplicate issues raised. Please don't be discouraged. Thank you.


Edit: Pluto TV Pluto After School Cartoons

Channel ID (required)
ForeverKids.us
Channel Name
Forever Kids
Alternative Names (optional)
No response
Network
No response
Owners
No response
Country
US
Subdivision
No response
City
No response
Broadcast Area
s/US
Languages
eng
Categories
kids
NSFW
FALSE
Launched
No response
Closed
No response
Replaced By
ForeverKids.us
Website
https://pluto.tv/en/live-tv/forever-kids
Logo
https://images.pluto.tv/channels/56171fafada51f8004c4b40f/colorLogoPNG.png
Notes
the channel on the playlist currently listed as "Pluto TV Pluto After School Cartoons" is actually called "Forever Kids" and should be updated to reflect that

Forever Kids already in the database: https://iptv-org.github.io/?q=Forever Kids
To fix the description of the stream you just need to set its tvg-id to ForeverKids.us:
https://github.com/iptv-org/iptv/blob/master/streams/us_pluto.m3u#L1280-L1281


End points are giving 404 NOT found error.

I tried using a curl command for resizeRBD and updatePool and the endpoints are giving 404 error.
curl -XPUT -d '{"Rbd.pool_id":"4","Rbd.name":"mmrbd1","Rbd.size":1048}' http://<IP_ADDRESS>:9292/1.0/e73b770e-5d3c-475e-87e2-c83e7deb2f15/CephResizeRbd
curl -XPUT -d '{"Pool.pool_id":"3","Pool.poolname":"test-test-1","Pool.pg_num":64,"Pool.min_size":1}' http92/1.0/e73b770e-5d3c-475e-87e2-c83e7deb2f15/CephUpdatePool
Both of the above commands are giving 404 error, so the UI team is not able to test the UI for the same.

@nthomas-redhat @anivargi @shtripat @r0h4n @brainfunked We need to fix this issue.

@cloudbehl this is expected to work. One would need to confirm that the instance you are hitting up is configured correctly.

Fixed.


Fix key expire timing

See https://github.com/EnjoyTech/redis-semaphore/pull/2 for overview of how redis-semaphore works and original change to expire the exists key prior to the available key.
Although the fix above fixed one race condition, it introduced another, which is that the available key expires in between the time a lock is initialized (when it checks and sees the exists key is present) and the time it grabs a resource off the available key's queue. It thus has no resource to grab, and hangs forever.
So let's say a semaphore with the name "FOO" is created with an expiration of one minute, and then re-invoked one minute later. Here is what can happen.

FOO semaphore initialized
SEMAPHORE:FOO:EXISTS initialized w/ 1 minute expiration
SEMAPHORE:FOO:AVAILABLE initialized w/ 1 minute expiration
SEMAPHORE:FOO:VERSION initialized w/ 1 minute expiration
SEMAPHORE:FOO:GRABBED initialized but deletes as soon as semaphore released
1 minute passes
FOO semaphore invoked again before any keys expire
determines that SEMAPHORE:FOO:EXISTS key is still valid, so does not re-initialize semaphore
SEMAPHORE:FOO:EXISTS expiration passes, but its expiration was extended as part of the check in the prior step, so it does not expire
SEMAPHORE:FOO:AVAILABLE expires, since nothing has touched it yet
new semaphore goes to pop a resource off SEMAPHORE:FOO:AVAILABLE, but since it was expired, it is empty and blocks forever
bad stuff happens

This PR delays the expiration of the available key (and the version key, which is entirely irrelevant) by 10 seconds from the time the exists key is expired.
If the exists key expires (and the available key hasn't expired), a new invocation will delete and re-initialize the available key anyway. Expiring the available key is thus not urgent, just a matter of cleanup.
Testing

[x] Run gem's tests
[x] Manually test in console with Gemfile pointed to local code and ensure lock still works.
[x] Manual hacking to artificially delay the time between checking the exists key and grabbing a resource with a sleep call.  To reproduce, locked with first semaphore with a relatively short expiration, then locked with 2nd so that available key expiration from the first will expire during the added sleep call. After making the fix, unable to reproduce the problem in the same manner.

Made this PR against wrong version of repo. Meant to make it against our forked version.


docs(version): ✨ Pulling version number from helia.

Fixes the package version on the docs hosted on helia.io.

Pulls the latest version from helia.
Updates the root package.json
Builds the docs.

This should fix the version in the next release.

This is a strange fix because v2.0.3 is the current version of helia which is compliant with v2.0.0 of the helia interface.
In theory we'll see lots of patch releases of helia but very few releases to the interface.
The API docs are monorepo docs rather than specific to a given module and the version number comes from the root package.json.
Perhaps there's a config key for the root typedoc.json to remove display of the monorepo version as it doesn't make much sense to the consumer?  Then we don't need contortions like the one introduced in this PR.

typedoc.json has "includeVersion": false but doesn't seem to work 🤔.

It was being hard-coded in aegir's docs command, this should be fixed now.

Alternative: https://github.com/ipfs/helia/pull/296

Closing in favour of https://github.com/ipfs/helia/pull/296


Could you pls provide a pretrained model ?

As title. Really interest in your work. Many thanks.

I think there is already one: https://github.com/yuanming-hu/exposure#using-the-pretrained-model

Just make sure you have cloned the repo recursively:
git clone https://github.com/yuanming-hu/exposure --recursive


Improvements & Fixes to MyWorlds

Summary:

Fix for the nether not working when being unloaded and loaded again if MyWorld's Portal teleportation management is turned off
Fix for the nether loading during a start despite being unloaded in worlds.yml if AllowNether is set to true in server.properties, in that case just let the game what it wants to do and unload it immediately after starting is finished
Add compatibility for custom nether/overworld scales set by datapacks - fixed
Fix a crash regarding TwistingVines maxGrowthAge being set to more than 25 in purpur.yml
Fix POI warnings - fixed

Longer explanations:

when vanilla handles the portals, when I unload the nether with MyWorlds and load it again, the portal doesn't send me through. I need to restart the server in order for portals to work again, which is a bit inconvenient.
this is just a QoL thing, it would be nice to be able to allow the nether in the server.properties while still keeping it deactivated so one can activate it on demand. I guess a simple solution would be that MyWorlds checks and unloads unloaded worlds after the server has been started
This is an issue that totally prevents any usage of a datapack changing the nether scale. Of course there are many legitimate reasons for wanting this, for example when a server either wants to nerf the nether or even increase its effectiveness, which allows for customizations. If there is an uncomplicated solution to this by reading datapacks, it would be great. It would be inconvenient having to set it in myWorlds as well, but of course having an option to configure it in MyWorlds would not be bad as well (during the creation of the nether then)
This is rather critical and I could isolate the issue to MyWorlds. The Purpur.yml allows to configure the MaxGrowthAge of twisting Vines and Kelp. I don't know if a value above 25 is even allowed, but in either way it should not cause a crash.
The server log looks like this:
https://mclo.gs/fQ6WFzM
https://pastebin.com/1j2Wbr9z (as a backup paste)
The only solution to this is interestingly to deactivate MyWorld's Portal teleportation handling, which is how I encountered the other issues. However, when doing /tpp into the nether despite MyWorld's portal logic being deactivated, the server crashes anyway.
Just added to this issue because it turned out to be a part of what I encountered

For the whole process of finding out those bugs read here https://discord.com/channels/415909893233442826/607574123366973461/830536232811102248
Otherwise, don't hesitate to ping me whenever you need me to test something! :D

double_nether_height.zip
nether_scale_1.0.zip
The two resource packs. I use "double nether height" only and changed the value in its configuration.
One note regarding the nether height resource pack:

Installation of this pack was a little weird for me. It didnt work right away. What I did to get it to work was let the nether world directory build by running the server.  Bring the server back up and check to see if your nether roof is at 256 instead of 128. If not, try again unti it does.

https://github.com/PurpurMC/Purpur/commit/0623652013aad0b48b8f6902a87034b8838adc9a
That explains the vines crashing issue, so at least this part is resolved


Swift header file not created correctly when building with Xcode 10.2

Swift 5 seems to have changed the format of the generated Swift header file by adding the following conditionals dependent on the architecture:
#if 0
#elif defined(__arm64__) && __arm64__
...
#elif defined(__ARM_ARCH_7A__) && __ARM_ARCH_7A__
...

When building the iOS platform with cocoapods-binary the generated Swift header file only has __arm64__ and __ARM_ARCH_7A__ but does not have any conditionals for the simulator architectures and therefore the Swift parts of the framework can't be used by Objective C code in the app.
Similar issue in Carthage - https://github.com/Carthage/Carthage/issues/2718

fixed


add: Polynomial Regression

Describe the algorithm/code
polynomial regression is a form of regression analysis in which the relationship between the independent variable x and the dependent variable y is modelled as an nth degree polynomial in x. Polynomial regression fits a nonlinear relationship between the value of x and the corresponding conditional mean of y. Please Assign this to me under hacktoberfest tag
Do you want to work on this feature?

[X] I am willing to implement this feature.

Hi @adi271001! 👋🏻
You can start working on this issue! Do let me know if you need any help. ✨

created pull request and added codes


Self attested attributes in proof template

On BPA UI we do not have an option to add self attested attributes in a proof template, which is a standard aries implementation.

The answer is a bit late, but now this is possible by creating a proof template without any restrictions, additionally the trusted issuer did can be set to the did of the agent that owns the self attested attribute(s).

Implemented with https://github.com/hyperledger-labs/business-partner-agent/pull/839


[EBPF-594] Refactor uprobe attacher to receive a process-monitor-like interface

What does this PR do?
This PR refactors the UprobeAttacher struct to receive a ProcessMonitor interface instead of using pkg/process/monitor.ProcessMonitor directly.
Motivation
Remove direct dependency on ProcessMonitor so that other modules (GPU/DI) can use the event monitor stream directly.
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

This PR doesn't change behavior, as it only adds the new interface and updates the caller to use the old ProcessMonitor.
Feedback is welcome on the use of the monitor.ProcessCallback type (see comment)

Please deploy to staging for dogfooding and sanity

Deployed to oddish-c, seems to be working correctly.

/merge


⚠ Guya.moe Proxied has degraded performance

In 1b3b040, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 13001 ms

Resolved: Guya.moe Proxied performance has improved in e55979e after 9 minutes.


Gallery, Pesto: Hero UP transition shouldn't be linear

In a design review, it appears that the Hero Up transition of a recipe card in Pesto is linear. It was recommended that we use the "standard curve" from https://material.google.com/motion/duration-easing.html#duration-easing-natural-easing-curves instead. The recommendation is to avoid linear.

"Because this is a curve which needs to be contiguous with the fling gesture velocity, use the Deceleration curve (“Easing out”), with the initial velocity == fling velocity" and "Android Launcher or Google Now Launcher uses this logic, if you want to see it in action"

Also applies to Hero Down.

Our first two reviews of Gallery pointed this out twice.


Fix error with incorrect syntax in props

Codecov Report

Merging #28 into master will decrease coverage by 0.52%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage     100%   99.47%   -0.53%
==========================================
  Files           8        8
  Lines         185      191       +6
  Branches       20       22       +2
==========================================
+ Hits          185      190       +5
- Misses          0        1       +1

Impacted Files
Coverage Δ

src/getPropTypes.ts
100% <100%> (ø)
:arrow_up:

src/utils.ts
100% <100%> (ø)
:arrow_up:

src/PropTypesCompletionItemProvider.ts
98.03% <50%> (-1.97%)
:arrow_down:

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 57a0913...09b4db9. Read the comment docs.


Delete manifests/m/Malwarebytes/Malwarebytes/<IP_ADDRESS>9 directory

The is just a installer file version, not an application version.
Checklist for Pull Requests

[X] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?
[X] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
Status Code: 200
(Automated message - build 876)

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
(Automated message - build 598)

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
(Automated message - build 598)

Hi @jpdoherty,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 876.)

The version is just incorrect. It is only the version of the installer file and not the application itself. There is an error somewhere in the identification of the current application version. <IP_ADDRESS>0 is the latest application build.

Hello @jpdoherty,
This PR removes the last version of the package from the repository.
Please check if the package requires an update or is available from another source.
Template: msftbot/lastVersionRemaining

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
Status Code: 200
(Automated message - build 876)

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
Status Code: 200
(Automated message - build 876)

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
Status Code: 200
(Automated message - build 876)

URL: https://data-cdn.mbamupdates.com/web/mb4-setup-consumer/MBSetup.exe
Status Code: 200
(Automated message - build 876)


update(kernels): update kernel json lists.

This PR updates the list of kernels from the latest crawling. Do not edit this PR.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: poiana
To complete the pull request process, please assign fededp after the PR has been reviewed.
You can assign the PR to them by writing /assign @fededp in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
for now.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: poiana
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval by writing /assign @fededp in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Test-infra miss support for ubuntu and minikube.

Minikube support is now in place.
We decided to avoid using ubuntu and instead go with ubuntu-$flavor.
As soon as:

https://github.com/falcosecurity/driverkit/pull/201
https://github.com/falcosecurity/falco/pull/2178
https://github.com/falcosecurity/test-infra/pull/806
https://github.com/falcosecurity/test-infra/pull/783
are merged, we can go on and merge this one!

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, poiana
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 300ce0bb02673b6981037933661f47cf5565e20e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr, poiana
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I think we can go on and merge this one; then once https://github.com/falcosecurity/test-infra/pull/783 is merged too, we will have proper prow jobs to build minikube drivers.

/unhold


#104: Implemented log_values for logging user defined values.

@robintw @jvdzwaan let me know if this is what you had in mind

Thanks @trbedwards!
I see you are storing a list of dictionaries in the database. What I had in mind was a single dictionary the user can add values to (e.g., for this run I want to store color=blue and favorite=whatever and for the next it's arbitrary_value=q). But maybe there is a good reason to use a list of dicts. What was your reasoning behind that?

Thanks @trbedwards - it's really wonderful to see another PR so soon after the sprint (I assume you got home ok and everything? We arrived back this afternoon, exhausted!).
This looks great - although I do have the same question as above about the list of dictionaries. I also wonder whether there could be a way to change the function signature to make it as easy to use as possible. Ideally I think it'd be great if we could call it as:
log_values({'a':5, 'b':10})
log_values(a=5, b=10)
log_values(a=5)

The latter two could be done using kwargs, which is a dict anyway. Doing that while also allowing the first one should be manageable with an optional argument and doing a bit of checking of the keys in kwargs - what do you think?
In a wonderfully-ideal world I would love to be able to do log_values(variable) and have recipy automatically grab the name of the variable as well as the value. This would make it as 'effortless' as it could possibly be - so if we called log_values(cutoff_threshold) we'd get the same as doing log_values({'cutoff_threshold':cutoff_threshold}) - but I think this might be going a bit far...! (If someone comes up with a magic way of doing that then it could be a future PR - don't hold up this one for my insane ideas...!)

I used a list because I was getting errors like unhashable type 'dict', but I'll see if I can use a dictionary instead somehow

The latest commit should address these issues @jvdzwaan @robintw. Let me know what you think

Thanks @trbedwards - that's great :-) All merged now!

Cool! Thank you @trbedwards !


reduce verbosity with a single option

Steps to reproduce the problem:
mitmdump by default generates a log of output to display flow information, which makes it difficult to see any logging from a script. You can lower mitmdump's logging verbosity, but that does not change the amount of information logged for the flows.
Example: Run mitmdump --set termlog_verbosity=error
This will still print a lot of information about flows, even though verbosity is dialed down to the lowest setting.
The answer is to also set flow_detail=0, but it would be nice if all verbosity settings could be combined into a single setting.
Any other comments? What have you tried so far?
System information

Mitmproxy: 3.0.4
Python:    3.5.2
OpenSSL:   OpenSSL 1.0.2g  1 Mar 2016
Platform:  Linux-4.10.0-42-generic-x86_64-with-Ubuntu-16.04-xenial

Thanks, this is very helpful feedback!


Cannot find module './enUS.json' or its corresponding type declarations

I have a vue package with component library mode, and it export json as:
export { default as enUS } from './enUS.json'
after build, it genenate d.ts file but without the single json file in '/dist/'
when another project use this package, when check with vue-tsc
throw error:
Cannot find module './enUS.json' or its corresponding type declarations

Can you provide repro case?


Reimplement the cloud module with asyncio and httpx

Re-implement the cloud module with asyncio.

Replaced requests with httpx
Remove misc unused methods
Add some unit tests
Improve error handling

Throw proper exceptions for API errors
Throw proper exceptions for HTTP errors
Only retry the API on timeouts

This will close #15 when merged to main


Clear-counters.py update

Require PR #53 to be merged first and code update to reflect inventory API change


deps: update DMN-related dependencies

Closes #2908

👍
@barmac Can you make sure this ends up on master not develop? That's true of everything we want for the release.


Do not escape double or single of quotes by default

This follows #176 and makes output from HTMLRenderer shorter and more human-readable - by not escaping both double and single quotes by default.
I.e. instead of "something" we want to output "something".
If required, one can still force the HTMLRenderer (or any of its descendants) to escape either type of quotes:
with HTMLRenderer(html_escape_double_quotes=True, html_escape_single_quotes=True) as renderer:
    output = renderer.render(Document('"something"'))

Done.


[Feature]: Auto Collapse ExpandMenu on button select

**Is this request for a new component, or additional functionality for an existing component? **
Additional functionality for ExpandMenu
Have you validated that your desired feature or component is not already in the backlog?
Yes
Describe the component and functionality you would like to see in the starter kit
It would be good to have an option to auto collapse the menu on button select. Currently if the menu is expanded and then upon selecting a button, app navigates to the other screen. when it returns to the original screen, the menu stays expanded. Also, the property isExpanded could not be modified at the moment. This property could be useful to decide the required behavior
Describe alternatives you've considered
No alternatives
Additional context

I will take a look at adding this for future releases.


chore: Upgrade to python 3.11

Describe your changes
Related Issues / Projects
Checklist

[ ] I've manually tested my code
[x] The changes pass pre-commit checks (make lint)
[x] The changes follow coding style

I would hope you didn't do this manually but looking at the fact you didn't change pyupgrade from 37-plus to 311-plus does not inspire confidence

I would hope you didn't do this manually but looking at the fact you didn't change pyupgrade from 37-plus to 311-plus does not inspire confidence

I just modified the Pipfile and let the lock file regenerate, is there anything i forgot to modify or should do differently?

should update pre-commit pyupgrade args to do 311

also, this definitely needs to be tested

I would hope you didn't do this manually but looking at the fact you didn't change pyupgrade from 37-plus to 311-plus does not inspire confidence

I just modified the Pipfile and let the lock file regenerate, is there anything i forgot to modify or should do differently?

should update pre-commit pyupgrade args to do 311

done, i need testers :3

https://github.com/osuAkatsuki/bancho.py/pull/493 will change deps

can you fix conflicts? want to get this merged. @7mochi
also, python 3.12 is very close to releasing as well. should be looking into that after this PR (especially typing changes)

:fire: nice work