Compilation instructions incorrect for Linux

Following instructions in readme:
kjotte@daedalus:~/git/redsea$ gcc -o rtl_redsea -lm rtl_redsea.c
rtl_redsea.c: In function ‘main’:
rtl_redsea.c:91:5: error: ‘for’ loop initial declarations are only allowed in C99 mode
     for (int iq=0;iq<=1;iq++) {
     ^
rtl_redsea.c:91:5: note: use option -std=c99 or -std=gnu99 to compile your code

Code compiles correctly with command:
kjotte@daedalus:~/git/redsea$ gcc -std=gnu99 -o rtl_redsea rtl_redsea.c -lm

Thanks!

You added -std=gnu99, but you didn't move the "-lm" parameter to the end of the command as nivex did. If it's not at the end i get linker errors on my linux box.
Great tool btw.

Right, fixed now.


chore: release 0.15.0

In this PR:

Update package versions to 0.15.0
Update release notes
Update templates branch 2b3ee1b

Using templates 0.14.0 for templates is ok.


Show better error for failing to connect to local server

https://github.com/Microsoft/vscode-cosmosdb/issues/770

Looks like a dupe that has been addressed elsewhere. Closing.


Uncaught TypeError: ajaxify.ajax is not a function

How do I deal with this?

Try with require('@stanimirdim92/ajaxify'). I have registered the package with a scope due to name conflicts.

My files are local so interference shouldn't be a problem. I'm using Browserify and it's claiming ajaxify.ajax is not a function.

Yup, my bad. I'm working on fixing the problem.

Please, run npm update and include this in your file. require('@stanimirdim92/ajaxify');

It is still not seeing ajaxify.ajax as a function. Possibly, it is something on my end since I am unfamiliar with module.exports = factory;. Any ideas where I can start troubleshooting?

Got it!
5. var ajaxify = require('./ajaxify');
6. var ajaxify = ajaxify(window); // made an ajaxify instance


Sigh resign entitlements per single app extension

Checklist

[ x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[ ] I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
[x ] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary.

Motivation and Context

Resolves #20922
Description

resign.sh is patched to allow specifying multiple entitlements files. The patch takes the app-identifier from the files and when it resigns an app extension it uses the correct entitlement file. In this way, one can specify an entitlements file for the main app and one for each app extensions.
Given that resign.sh does many things, I payed attention to use the final bundle identifier that the resign() function uses. Specifically, I use the $BUNDLE_IDENTIFIER after the function prints the following logs:
     log "Current bundle identifier is: '$CURRENT_BUNDLE_IDENTIFIER'"
     log "New bundle identifier will be: '$BUNDLE_IDENTIFIER'"

Please, review if this is correct.
I tested with my app and it works.
Testing Steps

any update about this PR?
i really need the capability as well

Hi @joshdholtz, can you take a look at this? Thank you.

hi @fastlaneTeam , can you review this PR? Thanks!

This is also an issue for us. Can this be merged soon?

Any update about this PR?

Any updates? This blocks us with resigning apps with app extensions.


Make UIPlugin work in a headless mode

Objective
Make UIPlugin usable if rendering is disabled.
Fixes #3815
Solution
Introduced a check in UIPlugin whether rendering is enabled; if not, rendering-dependent systems and resources are not added.

Will be investigating further into this!
This should have been a draft, rather than a PR ready for review, sorry.


copr: fix error when cast const Enum to any type

What is changed and how it works?

Issue Number: Close https://github.com/tikv/tikv/issues/14156
Close https://github.com/pingcap/tidb/issues/40341
What's Changed:
Make is_expr_eval_to_scalar can identifie const Enum.

copr: fix error when cast const Enum to any type

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
Please add a release note.

Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with None.
If this PR will be picked to release branch, then a release note is probably required.

/cc @sticnarf

/merge

/merge

/test

/test

/test


appthreat-depscan: init at 2.1.0

Motivation for this change
Tool to audit dependencies based on known vulnerabilities and advisories
https://github.com/AppThreat/dep-scan
Related to #81418
This requires cdxgen.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

appthreat-depscan
python310Packages.appthreat-vulnerability-db
python39Packages.appthreat-vulnerability-db


Support clearing history state

Added https://discuss.prosemirror.net/t/reset-history-plugin-state/1883
https://github.com/scrumpy/tiptap/issues/491

See my comment in https://github.com/scrumpy/tiptap/issues/491

Left a comment there. Closing this


Defs/region

Solving issue #72

@danielhuppmann sorry seems I have forgotten one step before the pull request.... Anyway, what changes in this file in only definition of subregions for france.....

resolved the merge conflict


AFRAME is not defined

AFRAME is not defined as it seems, in the entire projekt. (In my own test/mycontroller.js)
Even existing pages such as
index.php?setup=web_a-frame/City bank&model=City%20bank%20aframe&aframe=true

that are references by 'web_a-frame/default.js AND city bank.js" deliver the same error or are not found.
I either completely missed sometihing or A_Frame is not properly imported / used yet.
PS: Are there any working A-Frame elements in the Getaviz project on the development branch?

What exactly is the error message and where is it shown? And what is not found, city bank.js?
A-Frame is imported in the index.php (starting at line 26).
In the data directory is City bank aframe which contains a small aframe visualization using the city metaphor. That's the visualization that should be diplayed when going to http://localhost:8082/ui/index.php?setup=web_a-frame/City bank&model=City bank aframe&aframe=true

Maybe your browser does not like the space in the URL (between "City" and "bank"). Try to replace it with %20

Link is correct / page found but it as I mentioned, AFRAME is not defined:
(for your Link: http://localhost:8082/ui/index.php?setup=web_a-frame/City bank&model=City bank aframe&aframe=true)
GET http://localhost:8082/ui/node_modules/aframe/dist/aframe-v0.9.1.min.js net::ERR_ABORTED 404 (Not Found)
camera-beta.js:4 Uncaught ReferenceError: AFRAME is not defined
at camera-beta.js:4
(anonymous) @ camera-beta.js:4
AframeActionController.js:95 Uncaught ReferenceError: AFRAME is not defined
at Object.initialize (AframeActionController.js:95)
at initializeApplication (Application.js:36)
at Application.js:29

Can you please share your docker-compose.yml file?

version: "3"
services:
# db:
#     image: mysql:5.7
#     environment:
#         MYSQL_ROOT_PASSWORD: dbpassword_for_user
#         MYSQL_DATABASE: eval_server_development
#         MYSQL_USER: db_username
#         MYSQL_PASSWORD: dbpassword_for_user
#     restart: always
# eval:
#     build: evaluationserver
#     restart: always
#     env_file: evaluationserver/env
#     command: "./bin/wait-for-it.sh db:3306 -s -t 30 -- ./bin/docker_start"
#     #volumes:
#     #- ./evaluationserver:/usr/src/app
#     ports:
#         - "8081:8081"
#     depends_on:
#         - db
frontend:
build: ui/
restart: always
volumes:
- ./ui:/var/www/html/ui # uncomment only for ui development
- data-gen:/var/www/html/ui/data-gen
ports:
- "8082:80"
networks:
- default
depends_on:
- backend
backend:
build: generator2/org.getaviz.generator/
restart: always
volumes:
- data-gen:/var/lib/jetty/data-gen
#- ./generator2/org.getaviz.generator/target/org.getaviz.generator-1.0.0-SNAPSHOT.war:/var/lib/jetty/webapps/root.war # uncomment only for generator development
#- ./generator2/org.getaviz.generator/settings.properties:/opt/config/settings.properties # uncomment only for generator development
#- ./generator2/output:/var/lib/jetty/output/ # uncomment only for generator development
#- ./generator2/logs:/var/lib/jetty/logs # uncomment only for generator development
ports:
- "8083:8080"
networks:
- default
depends_on:
- neo4j
- volumes-provisioner
neo4j:
image: neo4j:3.5.1
restart: always
ports:
- "7474:7474"
- "7687:7687"
environment:
NEO4J_AUTH: "none"
networks:
- default
volumes-provisioner:
image: hasnat/volumes-provisioner
environment:
PROVISION_DIRECTORIES: "100:101:0755:/var/data"
volumes:
- data-gen:/var/data
volumes:
data-gen:
networks:
default:
driver: bridge

Ok, so does the file ui/node_modules/aframe/dist/aframe-v0.9.1.min.js exist on your device? Did you execute npm install as described in the wiki?

Yes I did, and just now to verify again.

So the file does exist?

ui/node_modules/aframe/dist/aframe-v0.9.2.min.js instead of ui/node_modules/aframe/dist/aframe-v0.9.1.min.js
Give me a sec. might just be the problem since index.php does not find it

in index.php:
$loadFramework = "<script src="node_modules/aframe/dist/aframe-v0.9.1.min.js">";
--> So it does not find it? A-Frame update and the link was hardcoded maybe?

changed it everywhere from v0.9.2. This is working now: http://localhost:8082/ui/index.php?setup=web_a-frame/City bank&model=City bank aframe&aframe=true
Thank you very much!

Well, regarding your issue: It should wrok if you change it to 0.9.2
The dependencies are defined in the package.json. There it says version 0.9.1 explicitly (line 9). Therefore, npm should always download 0.9.1 Although, when running npm install it prints the following:

$ npm install
audited 374 packages in 1.382s
found 13 vulnerabilities (5 low, 2 moderate, 6 high)
run npm audit fix to fix them, or npm audit for details

If you used npm audit fix npm will update your package.json to use Aframe 0.9.2 instead of 0.9.1, which will break index.php. You should see in your git clinet that package.json changed accordingly. Just revert your changes to the original state of the development branch and it should work again.
Anyway, most likely we should change the import to aframe-master.min.js instead of aframe-v0.9.2.min.js


Gurobi error

Describe the bug
When I try to run the code using Gurobi I receive the following error: "Try another solver, or solve with verbose=True for more "
cvxpy.error.SolverError: Solver 'GUROBI' failed. Try another solver, or solve with verbose=True for more information.
I only receive this error for N >= 13, for N<13 it runs without errors. I think it's not related to my license, because Gurobi
works fine for me solving other problems.
To Reproduce
import numpy as np
import cvxpy as cp
import random
N = 13
#uncomment to run without errors
#N = 12
pts= np.array([(random.uniform(0.0, 1.0), random.uniform(0.0, 1.0)) for i in range(N)])
state_matrix = cp.Variable((N,N),boolean=True)
point = state_matrix @ pts
distances = []
for i in range(0,N-1):
distances.append(cp.norm(point[i]-point[i+1]))
distances.append(cp.norm(point[0]-point[i+1]))
objective = cp.Minimize(cp.sum(distances))
constraints = [ state_matrix @ np.ones(N) == 1, state_matrix.T @ np.ones(N) == 1,
state_matrix[0,0]==1]
prob = cp.Problem(objective, constraints)
result = prob.solve(cp.GUROBI,verbose = True)
print(prob.value)
Expected behavior
Run without errors
Output
                            Numerical solver

(CVXPY) Nov 08 05:04:21 PM: Invoking solver GUROBI  to obtain a solution.
Restricted license - for non-production use only - expires 2022-01-13
Parameter OutputFlag unchanged
Value: 1  Min: 0  Max: 1  Default: 1
Changed value of parameter QCPDual to 1
Prev: 0  Min: 0  Max: 1  Default: 0
Gurobi Optimizer version 9.1.2 build v9.1.2rc0 (linux64)
Thread count: 2 physical cores, 4 logical processors, using up to 4 threads
Traceback (most recent call last):
File "erro.py", line 29, in
result = prob.solve(cp.GUROBI,verbose = True)
File "/usr/local/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 472, in solve
return solve_func(self, *args, **kwargs)
File "/usr/local/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 978, in _solve
self.unpack_results(solution, solving_chain, inverse_data)
File "/usr/local/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 1302, in unpack_results
"Try another solver, or solve with verbose=True for more "
cvxpy.error.SolverError: Solver 'GUROBI' failed. Try another solver, or solve with verbose=True for more information.
Version
OS: Debian GNU/Linux 10 (buster)
CVXPY Version: cvxpy==1.1.15

Additional context
This is the Travelling Salesman problem with origin defined at (0,0)

So you can solve other large problems with Gurobi? You're sure you don't have the free, limited license that only works on small problems?

Yes, I already used the free version of Gurobi, it tells you when your problem exceeded the limit of variables for the free license. Do you also have a Gurobi license to test?

Can you include the full verbose output? (Everything that is printed after calling the solve method with verbose=True)

1.1.15
                                 CVXPY
                                v1.1.15

===============================================================================
(CVXPY) Nov 12 09:14:19 PM: Your problem has 324 variables, 3 constraints, and 0 parameters.
(CVXPY) Nov 12 09:14:19 PM: It is compliant with the following grammars: DCP, DQCP
(CVXPY) Nov 12 09:14:19 PM: (If you need to solve this problem multiple times, but with different data, consider using parameters.)
(CVXPY) Nov 12 09:14:19 PM: CVXPY will first compile your problem; then, it will invoke a numerical solver to obtain a solution.
                              Compilation

(CVXPY) Nov 12 09:14:19 PM: Compiling problem (target solver=GUROBI).
(CVXPY) Nov 12 09:14:19 PM: Reduction chain: Dcp2Cone -> CvxAttr2Constr -> ConeMatrixStuffing -> GUROBI
(CVXPY) Nov 12 09:14:19 PM: Applying reduction Dcp2Cone
(CVXPY) Nov 12 09:14:19 PM: Applying reduction CvxAttr2Constr
(CVXPY) Nov 12 09:14:19 PM: Applying reduction ConeMatrixStuffing
(CVXPY) Nov 12 09:14:19 PM: Applying reduction GUROBI
(CVXPY) Nov 12 09:14:19 PM: Finished problem compilation (took 1.756e-01 seconds).
                            Numerical solver

(CVXPY) Nov 12 09:14:19 PM: Invoking solver GUROBI  to obtain a solution.
Restricted license - for non-production use only - expires 2022-01-13
Parameter OutputFlag unchanged
Value: 1  Min: 0  Max: 1  Default: 1
Changed value of parameter QCPDual to 1
Prev: 0  Min: 0  Max: 1  Default: 0
Gurobi Optimizer version 9.1.2 build v9.1.2rc0 (linux64)
Thread count: 2 physical cores, 4 logical processors, using up to 4 threads
Traceback (most recent call last):
File "erro.py", line 30, in
result = prob.solve(cp.GUROBI,verbose = True)
File "/usr/local/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 472, in solve
return solve_func(self, *args, **kwargs)
File "/usr/local/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 978, in _solve
self.unpack_results(solution, solving_chain, inverse_data)
File "/usr/local/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 1302, in unpack_results
"Try another solver, or solve with verbose=True for more "
cvxpy.error.SolverError: Solver 'GUROBI' failed. Try another solver, or solve with verbose=True for more information.

With N= 12
1.1.15
                                 CVXPY
                                v1.1.15

===============================================================================
(CVXPY) Nov 12 09:17:33 PM: Your problem has 144 variables, 3 constraints, and 0 parameters.
(CVXPY) Nov 12 09:17:33 PM: It is compliant with the following grammars: DCP, DQCP
(CVXPY) Nov 12 09:17:33 PM: (If you need to solve this problem multiple times, but with different data, consider using parameters.)
(CVXPY) Nov 12 09:17:33 PM: CVXPY will first compile your problem; then, it will invoke a numerical solver to obtain a solution.
                              Compilation

(CVXPY) Nov 12 09:17:33 PM: Compiling problem (target solver=GUROBI).
(CVXPY) Nov 12 09:17:33 PM: Reduction chain: Dcp2Cone -> CvxAttr2Constr -> ConeMatrixStuffing -> GUROBI
(CVXPY) Nov 12 09:17:33 PM: Applying reduction Dcp2Cone
(CVXPY) Nov 12 09:17:33 PM: Applying reduction CvxAttr2Constr
(CVXPY) Nov 12 09:17:33 PM: Applying reduction ConeMatrixStuffing
(CVXPY) Nov 12 09:17:33 PM: Applying reduction GUROBI
(CVXPY) Nov 12 09:17:33 PM: Finished problem compilation (took 1.297e-01 seconds).
                            Numerical solver

(CVXPY) Nov 12 09:17:33 PM: Invoking solver GUROBI  to obtain a solution.
Restricted license - for non-production use only - expires 2022-01-13
Parameter OutputFlag unchanged
Value: 1  Min: 0  Max: 1  Default: 1
Changed value of parameter QCPDual to 1
Prev: 0  Min: 0  Max: 1  Default: 0
Gurobi Optimizer version 9.1.2 build v9.1.2rc0 (linux64)
Thread count: 2 physical cores, 4 logical processors, using up to 4 threads
Optimize a model with 61 rows, 192 columns and 913 nonzeros
Model fingerprint: 0x5137725a
Model has 12 quadratic constraints
Variable types: 48 continuous, 144 integer (144 binary)
Coefficient statistics:
Matrix range     [1e-02, 1e+00]
QMatrix range    [1e+00, 1e+00]
Objective range  [1e+00, 1e+00]
Bounds range     [1e+00, 1e+00]
RHS range        [1e+00, 1e+00]
Presolve removed 15 rows and 35 columns
Presolve time: 0.01s
Presolved: 46 rows, 157 columns, 706 nonzeros
Presolved model has 12 quadratic constraint(s)
Variable types: 36 continuous, 121 integer (121 binary)
Root relaxation: objective 3.429089e-01, 23 iterations, 0.00 seconds
Nodes    |    Current Node    |     Objective Bounds      |     Work

Expl Unexpl |  Obj  Depth IntInf | Incumbent    BestBd   Gap | It/Node Time
H    0     0                       5.6255157    0.34291  93.9%     -    0s

0     0               0       5.6255157    0.34291  93.9%     -    0s
H    0     0                       5.0074656    0.34291  93.2%     -    0s
0     0    0.34291    0   30    5.00747    0.34291  93.2%     -    0s
0     0    0.45275    0   26    5.00747    0.45275  91.0%     -    0s
0     0    0.67454    0   36    5.00747    0.67454  86.5%     -    0s
H    0     0                       4.8091866    0.67454  86.0%     -    0s
0     0    0.95878    0   40    4.80919    0.95878  80.1%     -    0s
0     0    1.01737    0   42    4.80919    1.01737  78.8%     -    0s
0     0    1.01737    0   44    4.80919    1.01737  78.8%     -    0s
0     0    1.01737    0   43    4.80919    1.01737  78.8%     -    0s
0     0    1.01740    0   45    4.80919    1.01740  78.8%     -    0s
0     0    1.01743    0   45    4.80919    1.01743  78.8%     -    0s
0     0    1.01743    0   45    4.80919    1.01743  78.8%     -    0s
H    0     0                       4.7739065    1.01743  78.7%     -    0s
H    0     0                       4.0287297    1.01743  74.7%     -    0s
0     2    1.01743    0   45    4.02873    1.01743  74.7%     -    0s
H  243   179                       3.8920129    1.04829  73.1%  19.3    0s
H  364   195                       3.8776428    1.40110  63.9%  18.3    1s
H  366   186                       3.7856800    1.42404  62.4%  18.2    1s
H  402   199                       3.7547396    1.61476  57.0%  16.6    3s
418   210    1.83394   10   99    3.75474    1.63806  56.4%  16.0    5s
H  436   209                       3.4947892    1.66206  52.4%  15.3    6s
491   245    3.49479   20   99    3.49479    1.70780  51.1%  13.6   10s
H  516   246                       3.4253015    1.72232  49.7%  12.9   11s
570   284    2.26020    8   88    3.42530    1.74345  49.1%  33.9   15s
H  589   280                       3.3106586    1.75486  47.0%  32.8   16s
641   315    3.31066   10  102    3.31066    1.78495  46.1%  30.1   20s
715   364    3.31066   12   98    3.31066    1.81981  45.0%  27.0   25s
1043   493    2.55802   54   43    3.31066    2.06410  37.7%  42.3   30s
H 1086   457                       3.3063683    2.06410  37.6%  41.6   30s
H 1295   434                       3.1749746    2.19997  30.7%  40.8   30s
H 1349   378                       3.1018483    2.19997  29.1%  40.1   30s
1349   378              46       3.1018483    2.19997  29.1%  40.1   30s
H 1439   350                       3.1018483    2.31984  25.2%  39.7   30s
H 1510   315                       3.1018483    2.35990  23.9%  39.6   30s
H 1799   352                       3.0951631    2.47018  20.2%  40.0   31s
1799   352              51       3.0951631    2.47018  20.2%  40.0   31s
H 1810   350                       3.0951631    2.50442  19.1%  39.9   31s
3491   221              47       3.0951592    2.87777  7.02%  36.0   33s

Cutting planes:
Gomory: 1
MIR: 200
Flow cover: 37
Explored 4014 nodes (135232 simplex iterations) in 33.77 seconds
Thread count was 4 (of 4 available processors)
Solution count 10: 3.09516 3.09516 3.10185 ... 3.78568
Optimal solution found (tolerance 1.00e-04)
Best objective 3.095159172359e+00, best bound 3.095159172359e+00, gap 0.0000%
                                Summary

(CVXPY) Nov 12 09:18:07 PM: Problem status: optimal
(CVXPY) Nov 12 09:18:07 PM: Optimal value: 3.095e+00
(CVXPY) Nov 12 09:18:07 PM: Compilation took 1.297e-01 seconds
(CVXPY) Nov 12 09:18:07 PM: Solver (including time spent in interface) took 3.382e+01 seconds

Okay, so Gurobi encounters some kind of error, which CVXPY translates into a generic "Solver Error":
https://github.com/cvxpy/cvxpy/blob/master/cvxpy/reductions/solvers/qp_solvers/gurobi_qpif.py#L30-L42
https://github.com/cvxpy/cvxpy/blob/master/cvxpy/reductions/solvers/conic_solvers/gurobi_conif.py#L37-L51
It seems like CVXPY shouldn't drop the low level status information, since it makes it hard to debug what's going on in cases like this.
We do save the Gurobi model in the solution, so users can look at it. But the SolverError is thrown before the solution is unpacked into the problem's solver_stats:
https://github.com/cvxpy/cvxpy/blob/master/cvxpy/reductions/solvers/conic_solvers/gurobi_conif.py#L37-L51
We should try to save as much information in the problem as possible, before throwing SolverError. That way users could catch the exception and inspect the solver_stats to see what was going on.

@akshayka Agree re keeping the low-level status information
Meanwhile, @rodfloripa I ran your code, the error that is raised is:
gurobipy.GurobiError: Model too large for size-limited license; visit https://www.gurobi.com/free-trial for a full license
If you have a license / can obtain an (academic) license, it is usually enough to place the gurobi.lic file at the correct location.

Thanks @phschiele !!

@akshayka What changes do I do to https://github.com/cvxpy/cvxpy/blob/master/cvxpy/problems/problem.py to print the detailed error?


Mouse fix for high-DPI non-macOS systems (e.g. Wayland)

This is a follow-up to the Wayland mouse offset issue in #21 — mouse offset issues were still present on High-DPI Wayland setups (notably in non-stretched fullscreen). The change below fixes the issue (by moving the DPI-handling code out from behind an #ifdef __APPLE__ check, and fixes a few minor issues with it (none of which were causing problems for me, but theoretically could).
The "DPI hack" for Apple Retina systems is actually necessary for all high-DPI systems. Enable it unconditionally — it's harmless on non-high-DPI systems — and clean it up a little. In particular:

Use SDL_GetRendererOutputSize() instead of SDL_GL_GetDrawableSize(),
which is not guaranteed to work on non-OpenGL-based renderers.
Move the display-mode related stuff behind a video.useDesktopMouseCoords
check: ideally we should be using SDL_GetWindowSize() instead.
Update the comments to describe the DPI calculation more fully.

Note that, in the soon-to-be-released SDL 2.0.18, the helper function SDL_RenderWindowToLogical() could probably replace a lot of this. (Similarly, the use of SDL_SetWindowMouseRect() will make more sense instead of warping for the software cursor case), but those should wait until after 2.0.18 is fully released and available.
I confesss, I don't really understand the use of the current desktop mode and mouse coordinates everywhere — my feeling is that settings video.useDesktopMouseCoords to false unconditionally should be more stable, even on non-Wayland systems — but I've not changed that here, just in case there's something I'm missing.

Thanks for the request. However, I'll put this on hold for a little while until I get it properly tested on Windows, macOS and Linux (all in normal DPI and high-DPI modes). I have a feeling it could break something, so I don't want to pull it just yet. Also I need to get Linux set up on my second laptop and stuff before I can test this.
The current display/mouse code is a mess, it has been hacked over and over to fix issues, so it might not make a lot of sense (or even work properly, for that matter).

Merged it now, after testing that it works fine.


ZITADEL metrics

Adds ZITADEL CockroachDB metrics and dashboards
Number of metrics to be sent to cloud to be discussed, see internal/operator/boom/application/applications/prometheus/helm/rules.go

The new Health / ZITADEL dashboard:

The new Health / ZITADEL Details dashboard:


Documentation updates, bump to 1.1.5

@dcchivian can you review and merge this PR? should be quick to review - documentation updates.


Scale param for coverflow effect added.

Now you have possibility to use scale parameter in coverflow effect.

Merged, thanks!


add a convenience method CityMap.canRemove

Since a public method "canAdd" already exists and is obviously meant to determine the result of adding a tile to the map without actually adding something, a "canRemove" method should be implemented.

CityMap now contains a method "canRemoveAt"


error when building firebase-admin

ncc version: 0.16.1
Steps to reproduce
git clone https://github.com/MAPESO/ncc-bug-firebase
cd ncc-bug-firebase
yarn install  # dependency installation
ncc build index.js   # build in single file
node dist/index.js  # note the error output

logs of build

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/write-file-atomic" attempted to require "worker _threads" but could not be resolved, assuming external

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/hash-stream-validation" attempted to require "fast-crc32c" but could not be resolved, assuming external.

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/retry-request" attempted to require "request" but could not be resolved, assuming external.

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/write-file-atomic" attempted to require "worker _threads" but could not be resolved, assuming external.

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/debug/src" attempted to require "supports color" but could not be resolved, assuming external.

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/node-fetch/lib" attempted to require "encoding" but could not be resolved, assuming external.

ncc: Module directory "/Users/mark/ncc-bug-firebase/node_modules/bytebuffer/dist" attempted to require "memcpy" but could not be resolved, assuming external.

(node:41366) UnhandledPromiseRejectionWarning: Error [ERR_UNHANDLED_ERROR]: Unhandled error. (/Users/mark/ncc-bu
g-firebase/google/api)
    at EventEmitter.emit (events.js:171:17)
    at EventEmitter.<anonymous> (/Users/mark/ncc-bug-firebase/dist/index.js:187929:15)
    at Object.onceWrapper (events.js:273:13)
    at EventEmitter.emit (events.js:182:13)
    at fn (/Users/mark/ncc-bug-firebase/dist/index.js:187764:17)
    at statter (/Users/mark/ncc-bug-firebase/dist/index.js:187802:7)
    at walkdir (/Users/mark/ncc-bug-firebase/dist/index.js:187933:3)
    at Function.module.exports.walkdir.sync (/Users/mark/ncc-bug-firebase/dist/index.js:187679:10)
    at module.exports.COMMON_PROTO_DIRS.map.dir (/Users/mark/ncc-bug-firebase/dist/index.js:51861:17)
    at Array.map (<anonymous>)
(node:41366) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by thro
wing inside of an async function without a catch block, or by rejecting a promise which was not handled with .ca
tch(). (rejection id: 1)
(node:41366) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise r
ejections that are not handled will terminate the Node.js process with a non-zero exit code.

we can extract as something interesting is this phrase "but could not be resolved, assuming external". ncc does not offer much detail but I feel that these modules are causing the problem in some way
this is curious because that route exists but not in that order
(node:41366) UnhandledPromiseRejectionWarning: Error [ERR_UNHANDLED_ERROR]: Unhandled error. (/Users/mark/ncc-bug-firebase/google/api)

the correct order would be Users/mark/ncc-bug-firebase/dist/protos/google/api
result of index.js 51861
return walk.sync(dir)

result of index.js 187679
  return walkdir(path,options,eventHandler);

result of index.js 187764
emitter.emit('fail',path,err);

result of index.js 187802
statAction(ex,stat);

result of index.js  187929
emitter.emit('error',path,err);

result of index.js 187933
statter(path,1);

The underlying error here is in google-gax, in this file -<EMAIL_ADDRESS>The code that is causing the problem is:
// INCLUDE_DIRS is passed to @grpc/proto-loader
const INCLUDE_DIRS = [];
INCLUDE_DIRS.push(googleProtoFilesDir);
// COMMON_PROTO_FILES logic is here for protobufjs loads (see
// GoogleProtoFilesRoot below)
const COMMON_PROTO_DIRS = [
    // This list of directories is defined here:
    // https://github.com/googleapis/googleapis/blob/master/gapic/packaging/common_protos.yaml
    'api',
    path.join('iam', 'v1'),
    path.join('logging', 'type'),
    path.join('monitoring', 'v3'),
    'longrunning',
    'protobuf',
    'rpc',
    'type',
].map(dir => path.join(googleProtoFilesDir, 'google', dir));
INCLUDE_DIRS.push(...COMMON_PROTO_DIRS);
const COMMON_PROTO_FILES = COMMON_PROTO_DIRS
    .map(dir => {
    return walk.sync(dir)
        .filter(f => path.extname(f) === '.proto')
        .map(f => path.normalize(f).substring(googleProtoFilesDir.length + 1));
})
    .reduce((a, c) => a.concat(c), []);

where walk.sync is trying to do a dynamic directory walk on a dynamically known directory.
There is no general analysis fix unfortunately here, so like the express templates case, we need to work out what is a specific fix we can inline for this exact package to make things work out ok.
If I run this on my machine, I get COMMON_PROTO_FILES containing:
[ 'google/api/annotations.proto',
  'google/api/auth.proto',
  'google/api/backend.proto',
  'google/api/billing.proto',
  'google/api/client.proto',
  'google/api/config_change.proto',
  'google/api/consumer.proto',
  'google/api/context.proto',
  'google/api/control.proto',
  'google/api/distribution.proto',
  'google/api/documentation.proto',
  'google/api/endpoint.proto',
  'google/api/experimental/authorization_config.proto',
  'google/api/experimental/experimental.proto',
  'google/api/expr/v1alpha1/cel_service.proto',
  'google/api/expr/v1alpha1/checked.proto',
  'google/api/expr/v1alpha1/conformance_service.proto',
  'google/api/expr/v1alpha1/eval.proto',
  'google/api/expr/v1alpha1/explain.proto',
  'google/api/expr/v1alpha1/syntax.proto',
  'google/api/expr/v1alpha1/value.proto',
  'google/api/expr/v1beta1/decl.proto',
  'google/api/expr/v1beta1/eval.proto',
  'google/api/expr/v1beta1/expr.proto',
  'google/api/expr/v1beta1/source.proto',
  'google/api/expr/v1beta1/value.proto',
  'google/api/field_behavior.proto',
  'google/api/http.proto',
  'google/api/httpbody.proto',
  'google/api/label.proto',
  'google/api/launch_stage.proto',
  'google/api/log.proto',
  'google/api/logging.proto',
  'google/api/metric.proto',
  'google/api/monitored_resource.proto',
  'google/api/monitoring.proto',
  'google/api/quota.proto',
  'google/api/resource.proto',
  'google/api/service.proto',
  'google/api/servicecontrol/v1/check_error.proto',
  'google/api/servicecontrol/v1/distribution.proto',
  'google/api/servicecontrol/v1/log_entry.proto',
  'google/api/servicecontrol/v1/metric_value.proto',
  'google/api/servicecontrol/v1/operation.proto',
  'google/api/servicecontrol/v1/quota_controller.proto',
  'google/api/servicecontrol/v1/service_controller.proto',
  'google/api/servicemanagement/v1/resources.proto',
  'google/api/servicemanagement/v1/servicemanager.proto',
  'google/api/source_info.proto',
  'google/api/system_parameter.proto',
  'google/api/usage.proto',
  'google/iam/v1/iam_policy.proto',
  'google/iam/v1/logging/audit_data.proto',
  'google/iam/v1/policy.proto',
  'google/logging/type/http_request.proto',
  'google/logging/type/log_severity.proto',
  'google/monitoring/v3/alert.proto',
  'google/monitoring/v3/alert_service.proto',
  'google/monitoring/v3/common.proto',
  'google/monitoring/v3/dropped_labels.proto',
  'google/monitoring/v3/group.proto',
  'google/monitoring/v3/group_service.proto',
  'google/monitoring/v3/metric.proto',
  'google/monitoring/v3/metric_service.proto',
  'google/monitoring/v3/mutation_record.proto',
  'google/monitoring/v3/notification.proto',
  'google/monitoring/v3/notification_service.proto',
  'google/monitoring/v3/span_context.proto',
  'google/monitoring/v3/uptime.proto',
  'google/monitoring/v3/uptime_service.proto',
  'google/longrunning/operations.proto',
  'google/protobuf/any.proto',
  'google/protobuf/api.proto',
  'google/protobuf/descriptor.proto',
  'google/protobuf/duration.proto',
  'google/protobuf/empty.proto',
  'google/protobuf/field_mask.proto',
  'google/protobuf/source_context.proto',
  'google/protobuf/struct.proto',
  'google/protobuf/timestamp.proto',
  'google/protobuf/type.proto',
  'google/protobuf/util/json_format.proto',
  'google/protobuf/util/json_format_proto3.proto',
  'google/protobuf/wrappers.proto',
  'google/rpc/code.proto',
  'google/rpc/error_details.proto',
  'google/rpc/status.proto',
  'google/type/calendar_period.proto',
  'google/type/color.proto',
  'google/type/date.proto',
  'google/type/dayofweek.proto',
  'google/type/expr.proto',
  'google/type/fraction.proto',
  'google/type/latlng.proto',
  'google/type/money.proto',
  'google/type/postal_address.proto',
  'google/type/quaternion.proto',
  'google/type/timeofday.proto' ]

Reading https://github.com/googleapis/googleapis/blob/master/gapic/packaging/common_protos.yaml, it seems the list above is actually fixed by package version of the google-proto-files package, so that one fix could simply be to do the following transformation:

When processing google-gax/build/src/grpc.js, replace the definition of COMMON_PROTO_FILES to simply be the inlined list above, provided the version of google-gax matches our current major.
We then need to make sure we keep the above list in sync with what is defined in google-proto-files. One option might be to actually build this list in the ncc build itself with this package as a dependency to make sure it stays up-to-date.

I just tested out the above code replacement manually but unfortunately that then gives me a new error:
Error: no such type: Document
    at Service.lookupType (/node_modules/protobufjs/src/namespace.js:382:1)
    at Method.resolve (/node_modules/protobufjs/src/method.js:148:1)
    at Service.resolveAll (/node_modules/protobufjs/src/service.js:111:1)
    at getProtobufServiceAttrs (/node_modules/grpc/src/protobuf_js_6_common.js:106:1)
    at loadObject (/node_modules/grpc/src/protobuf_js_6_common.js:134:1)
    at values.forEach.nested (/node_modules/grpc/src/protobuf_js_6_common.js:143:1)
    at Array.forEach (<anonymous>)
    at loadObject (/node_modules/grpc/src/protobuf_js_6_common.js:142:1)
    at values.forEach.nested (/node_modules/grpc/src/protobuf_js_6_common.js:143:1)
    at Array.forEach (<anonymous>)

which unfortunately doesn't give a good stack trace but seems like it would be a dynamic require issue in dynamically requesting a .proto file, since the standard dynamic require analysis issues would apply there.
Unfortunately we still haven't made much progress on dynamic require approaches though, and would need to start with a much simpler case to work up to this sort of a problem.
So for now, unfortunately, solving this firebase-admin case remains elusive.

@guybedford I really appreciate the research you just did, thank you very much. I'll have to resort to "mongoDB Atlas" as serverless db.

@rauchg That now and now-dev are not compatible with firestore will be a problem. Users will not have many options when using DB serverless

I finally managed to find a fix for this in https://github.com/zeit/webpack-asset-relocator-loader/pull/12.

@guybedford firebase-admin works perfectly usingncc but using now-dev I get the following error Error: package.json does not exist at /private
this error is related to this issue #306

If it works with ncc, then this is not the appropriate place for the issue.
Please discuss with TooTallNate.

I am using ncc to build my ts api project which i later upload to AWS serverless lambda. I do use firebase-admin and custom node10 runtime on lambda. After executing my function I get following error:
"errorMessage": "/var/task/lib/src/node/extension_binary/node-v64-darwin-x64-unknown/grpc_node1.node: invalid ELF header",
is it related to ncc? Let me know if I need to create a separate issue

@Yuripetusko This error looks like you compiled on Mac and then ran on Linux. Perhaps you should try deploying to Now so that your build environment matches your run-time environment.

Thanks @styfle

I still get this error when deploying on now, when using now-cli everything goes ok, but on now i get:
ERROR	Unhandled rejection: { Error: ENOENT: no such file or directory, open 'google/protobuf/api.proto'     at Object.openSync (fs.js:443:3)     at Object.readFileSync (fs.js:343:35)     at fetch (/var/task/page.js:32228:34)     at Root.load (/var/task/page.js:32262:13)     at Root.loadSync (/var/task/page.js:32303:17)     at Object.loadSync (/var/task/page.js:46295:17)     at Object.8ZNE (/var/task/page.js:34417:37)     at __webpack_require__ (/var/task/page.js:23:31)     at Object.<anonymous> (/var/task/page.js:128795:25)     at Object.b1/w (/var/task/page.js:129054:30)   errno: -2,   syscall: 'open',   code: 'ENOENT',   path: 'google/protobuf/api.proto' }

Hi @diogoviannaaraujo
The @now/node builder no longer uses ncc.
Can you make sure you using the latest @now/node in your now.json and update the CLI to the latest version by running npm i -g now@latest?

Hi,
I was using @now/next and got to work by using next@canary dependency
Thanks

On Aug 19, 2019, at 9:58 AM, Steven<EMAIL_ADDRESS>wrote:
Hi @diogoviannaaraujo
The @now/node builder no longer uses ncc.
Can you make sure you using the latest @now/node in your now.json and update the CLI to the latest version by running npm i -g now@latest?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Implement model training for different frameworks in solaris.nets

@nrweir commented on Wed Apr 17 2019
This needs to be done for every framework to be used in the module:

[ ] TF/keras
[ ] PyTorch
[ ] Darknet (@avanetten I could probably use some help with this)

This will also depend at least partly on how the various checkpoints, metrics, losses, etc. are implemented. I should try to structure at least one of each of those first.

Resolved in #73


arrayToHtmlList update

Description

So our function arrayToHtmlList is in a pretty bad space and could use an update. We already had some discussion over at #675 but I wanted to move the update discussion over to an issue for more visibility.
Current problem:
It uses an awkward query selector instead of the document.prototype.getElementByID() function, and it uses an IIFE to declare an intermediate variable closure when we could do with just API chaining the return values.
Another issue is we are just forcing it inside a list item. We could easily allow a formatter to be passed as the last argument with the map footprint and a default that won't change functionality, however the change in footprint because a bit... awkward as it becomes data, binding, callback, but it was an old monster to begin with.
With those two issues in mind we should look at either updating or cleaning up the snippet so it is more maintainable than it's current form or decide to leave as it.

const arrayToHtmlList = (arr, selector, elemTag) =>
  (
    container => container.innerHTML += arr.reduce((acc, el) => `${acc}<${elemTag}>${el}</${elemTag}>`, '')
  )(document.querySelector(selector));

@skatcat31 what do you think about implementing this way?

@Siarhei-Zharnasek I feel like if we're going to do that we should do a formatter function so they can choose their own entire string

+1 on document.selectElementById. Either that or we should pass a query selector and use document.querySelector instead. Either way, it's awkward right now.
IIFE has to go, let's do something better instead.
I like the formatter idea. From a usability perspective, it's a lot better. From a learning standpoint, this might land the snippet in advanced (depending on verbosity), but it's a very good teaching opportunity.

TL;DR: Let's fix that, it's an awkward snippet and all the proposed suggestions are pretty good.

function arrayToHtmlList(arr, id, elemTag) {     let parentElem = document.getElementById(id);     parentElem.innerHTML += arr.map((item) => {         return <${elemTag}>${item}</${elemTag}>     }).join('') }
@skatcat31 What do you think about this?

@Ankitb09 again what advantage does that bring over adding a default formatter so they can handle the full string if wanted?

Hello,
I have never contributed to open source before but this issue seemed like a good one to start. I hope I can contribute.
const arrayToHtmlList = (arr, listID, formatterString = '

', marker = '|') => {
  document.querySelector(`#${listId}`).innerHTML += arr.map(item => formatterString.replace(marker, item)).join('');
});

What do you think about this?

Sorry i made a typo in the above code ...  The marker was missing in the formatterString variableShould be:
const arrayToHtmlList = (arr, listID, formatterString = '
|
', marker = '|') => {
  document.querySelector(`#${listId}`).innerHTML += arr.map(item => formatterString.replace(marker, item)).join('');
});

An example usage of this would be
arrayToHtmlList(['item 1', 'item 2'], 'listId'); // use default template string
arrayToHtmlList(['item 1', 'item 2'], 'listId', '
This is the added item: |
'); // only use default replace marker but overwrite template string
arrayToHtmlList(['item 1', 'item 2'], 'listId', '
This is the added item: ###
', '###'); // overwrite template string and replace marker --> full flexibility

You're right I was missing a closing arguments parens.
Reduce might be an option
const arrayToHtmlList = (arr, listID, formatterFunction = (acc, item) => ((acc +=`<li>${item}</li>`), acc)) => {
  document.getElementById(listID).innerHTML = arr.reduce(formatterFunction, '');
};

however we run into a lot of really nasty parens in the footprint and having to keep around an empty string in body, which is something we need to do regardless(the join). It does however reduce the time complexity by a significant margin(O(n) vs (2n))
This however means that any formatter passed will be a reduce function and thus must remember the accumulator, or in other words more developer debt versus operation debt.
So then the question becomes do we allow more time complexity or more developer complexity?
We could avoid this by using a custom for loop accumulator so that a map compatible function gets passed, manually applied, and then added to the base string, solving both problems.

@skatcat31 Although this is not the most aesthetic implementation, it's the best solution in my eyes.
I think it is preferable to overwrite the innerHTML rather than concatenating because of the cleaner representation, as you have mentioned before.

@skatcat31 Wanna rewrite this one as discussed? Or I could give it a shot in a few days.

Hey all!
Just found (and really like) your project! Great work.
I'd like to through an implementation into the ring. Essentially a modification of a few previous suggestions.
const arrayToHtmlList = (arr, listID, formatterFn = (item) => `<li>${item}</li>`) => {
    document.getElementById(listID).innerHTML += arr.reduce(
        (acc, curr, idx) => acc += formatterFn(curr, idx)
        , ''
    );
};

Also, it strikes me that by adding the default formatterFunction, this function effectively becomes appendArrayOfItemsToElement.
Example:
arrayToHtmlList(['foo.js', 'bar.js', 'baz.js'], 'body-id', (src) => `<script type="text/javascript" src="${src}"></script>`);

Man this shows how busy a full time high cognitive load job gets. We should make a final decision on this soon and release the change

2 more suggestions:

The same as @skatcat31 's most recent Array.reduce method but using insertAdjacentHTML instead of innerHTML:

const arrayToHtmlList = (arr, listID, formatterFunction = (acc, item) => ((acc +=`<li>${item}</li>`), acc)) =>
  document
    .getElementById(listID)
    .insertAdjacentHTML('beforeend', arr.reduce(formatterFunction, ''));

Reasoning:

The insertAdjacentHTML() method of the Element interface parses the specified text as HTML or XML and inserts the resulting nodes into the DOM tree at a specified position. It does not reparse the element it is being used on, and thus it does not corrupt the existing elements inside that element. This avoids the extra step of serialization, making it much faster than direct innerHTML manipulation.

Source: MDN Element.insertAdjacentHTML

Similar to @nickforddesign 's suggestion, creates and returns new element instead of manipulating an existing element:

const arrayToHtmlList = (
  arr,
  tagName = 'ul',
  formatterFunction = (acc, item) => ((acc +=`<li>${item}</li>`), acc)
) => {
  const list = document.createElement(tagName);
  list.insertAdjacentHTML('beforeend', arr.reduce(formatterFunction, ''));
  return list;
};

const arrayToHtmlList = (arr, listID, formatterFunction = (item) =>
${item}
) => {
document.getElementById(listID).innerHTML = arr.map(formatterFunction).join('');
};

This is the easiest and cleanest way to take a constant array and convert it to an array list with it's inner.HTML. If it were PowerShell I would just use a HTML fragment for the formatting. Not sure what your gaining here. It should output just fine or you could convert to straight xml? this is just output
${item}

Hi   I  would like to   add something**:**
**.**using a  classic  for loop  will be  more efficient   than  a  reduce function  especially when converting big array .
by the way this is my first  contribution on GitHub👶
also   i'm writing   a new  snippet right now   any suggestion other than what's mentioned in the contribution Guideline

After reading through all the comments and the previous PR, I would also suggest returning just the HTML and letting the developer decide what to do with it. As the name also suggests "array to html list". It does not imply anything about changing the DOM.
peace :v:

@meysam81 I think we will probably stick with that and maybe add a blog for clarification/exploration on the subject.

I'm closing this and will probably add a blog soon to clarify and explore ideas from this thread.


Remove all Console.WriteLine messages by default from the ConsoleLifetime impl

https://github.com/aspnet/Hosting/blob/f9d145887773e0c650e66165e0c61886153bcc0b/src/Microsoft.Extensions.Hosting/Internal/ConsoleLifetime.cs#L37-L39
We should just use the logger instead.

@glennc @Tratcher side effect of this change is the following:

This is release mode in the default template. There's no output by default since we only log warning.

By design?

I'm not sure it's good though, at least not by default. You can't really tell what's going on.


Can I add aliases to the commands? (Complete beginner to github so I have no idea how this works...)

I know how to add it into the code, but I don't know how to work on github to get it done :( (P.S. This is my first issue on github)

For example for _s, I wanted to add another alias (_skip) like the other bots have... Will be easier for users to migrate

Sure @chiraggarg03 , that's a nice one, we can definitely add that. So do you want to take this up or we can add this?

@niloysikdar @amanjha8100 @driftywinds Thanks a ton everyone! I've submitted the pull request.


remove not needed file

What type of PR is this?
/kind cleanup
What this PR does / why we need it:

remove not needed file

/assign @saschagrunert @puerco @Verolop @xmudrii @palnabarun
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/lgtm


Add condition to check content length range in post policy

**
if min and max equal to 0 in the same time , it will throw an error

# #1675

#1675


Interface to access HDF5 files

We should implement an interface to download data through SFTP and concatenate them. This will be used for post-processing.

I believe this is partly fixed by commit 0939866fd20748dc3676658462ee583cb23b4a42, which implemented join_hdf5.py. I'm no longer sure that a functionality to download HDF5 files on the fly is useful, as people are already accustomed with a variety of tools that are probably better suited (scp, FileZilla, etc.).
Can we close this issue?


Add warn or error when using async setup() outside

What problem does this feature solve?
Currently when using async setup() on a component that is not wrapped in  results in not rendering the component. This is expected behaviour but not having a warn or an error creates a poor development experience. So by adding at least a warn it will make sure the people won't look at a blank screen not knowing what they did wrong ( like I did )
What does the proposed API look like?
Whenever you leave
async setup () {}

a warn to be thrown in the console.
Example:

[Vue warn]: Components using async setup() should be wrapped in Suspense in order to render

And maybe a tip/reminder that a separate function can be used.
Example:

[Vue warn]: Components using async setup() should be wrapped in Suspense in order to render. Consider using separate function to use await in.

Duplicate of https://github.com/vuejs/vue-next/issues/2603


[BUILD][TEST][SPARKR] better file list regex to kick off tests

What changes were proposed in this pull request?
more file regex
How was this patch tested?
Jenkins, AppVeyor

Test build #81596 has started for PR 19177 at commit da74d71.

Test build #81596 has finished for PR 19177 at commit da74d71.

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

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

Merged build finished. Test PASSed.

don't see an error in appveyor log

Test build #81610 has started for PR 19177 at commit 4021363.

it looks like as if this appveyor.yml is not enough to trigger appveyor test? it doesn't seem to be kicking off a run

Test build #81610 has finished for PR 19177 at commit 4021363.

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

Merged build finished. Test PASSed.

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

Yes.. will try to take an action for AppVeyor issue soon ..

Merged to master.

ah yes, i missed the little tick above for this build
https://ci.appveyor.com/project/ApacheSoftwareFoundation/spark/build/1746-master


Fix BoneMenu Error

Following the v1.3.3 version, when setting up BoneMenu the mod always throws a System.MissingFieldException: Field not found: 'BoneLib.BoneMenu.Page.Root' error. To fix this, surprisingly the methods responsible for checking BoneLib version in OnLateInitializeMelon had to be moved to OnInitializeMelon which fixed the issue

Found out that after just rebuilding the code mod, it works, I find it likely that when the mod was built, an old version of BoneLib was used


Ont 363

I recommend at least PR 41 be merged first to run CI tests.

@plbt5 , another proposal that I need to transcribe into a branch carries class definitions.  It's ONT-379 / CP-18.  All of the classes in the investigation namespace will have definitions after that (or if not, we need to file a fixer ticket for CASE 0.4.0).  We know we need definitions for the properties.


Call Func Track blocks transform Object when AnimationTreePlayer is playing in editor

I have created an example that shows this behavior.
When added to the animation test2 in example Call Func Track and AnimationTreePlayer is playing in editor transform player object is reset and blocked in game.
Call Func Track in root node blocks movement, body node blocks rotate.
If delete both track from test2 animation or stop playing AnimationTreePlayer in editor control is back in the game. In some cases stop playing AnimationTreePlayer in editor does not solve the problem.
In editor player object is blocked too when AnimationTreePlayer is playing.
I added to the example a cube that moves with keys WSAD, space jump and mouse rotate.
I hope that this example will help because otherwise I can not explain it.

AnimationTreePlayer will be reworked for 3.0, would be good to test this when that happens.

I'm glad to hear it

I'm in the process of revisiting old issues. Still happens in 3.0.2.
In fact I independently came across this issue in my project couple days ago, and I opened new issue. I will close this one as a duplicate since I believe my description is a little better. Feel free to add your comment to https://github.com/godotengine/godot/issues/18084 if I missed some information.


Fix features issues with init param and property setter

Description
This introduces a feature_defaults init parameter and property setter for the three layer types that support that (points, shapes, and vectors). It also fixes some related issues that occur when adding feature-derived colors when modifying the feature defaults in-place.
Note that the feature_defaults is deliberately a little different to the current_properties setter, as the latter also has the side-effect of setting the feature values of the selected elements to the new current property values. Also, using the feature_defaults setter with keys/columns that are extra/missing compared to features will result in an error.
Type of change

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

References
Closes #5632
Closes #5634
How has this been tested?

[x] added new tests to cover bugs and setter
[x] all tests pass with my change

Sorry, meant to open this in my fork.

@brisvag : added you as a reviewer since you poked the features and found these bugs.

Thanks for the quick review.

One thing: there's a few codecov warning about the setters not being tested. Maybe it's good to add some test for those as well?

Thanks codecov - it makes a fair point.
Can you think of an interesting usage of the features_default setter that's very different to the tests I added?
One thing I might do is to modify those tests. That way they handle setting the default value in the three ways we support (initializer param, in-place modification, property setter) - what do you think? Typically, I don't like testing multiple actions in the same test, but they are at least exercising the same usage/behavior. And I'm wary of adding a very similar test for each of the supported layers.

One thing I might do is to modify those tests. That way they handle setting the default value in the three ways we support (initializer param, in-place modification, property setter) - what do you think?

I did that in faf9622. If you don't like it, I'll revert it and write some other separate tests.

I think we can merge, this CI failure is everywhere :)


@craigspaeth => center auction page content for large widths

This fixes: https://github.com/artsy/force/issues/200#event-800980796
Before, this page was inheriting margin 0 200px, and left-aligning at large widths:

Now, the content is centered:

Sry for the delay, taking a look at this today.

Easy peasy. Thanks!


RPC message parser can support arbitrary whitespace such as '\n' within and between JSON RPC messages

Specification
Currently the agnostic RPC message parser separates the messages based on the}{ boundry., which makes it difficult to read messages since they are in a single line.
The objective is to improve readability by formatting a message with newlines and whitespaces, and ensuring that the parser is able to handle these characters.
// OLD
{some: 'message'}{other: 'message'}...

// new
{some: 'message'}
{other: 'message'}
...

This should be possible by adjusting the separator parameter which should be able to discard the newline and other whitespace characters when constructing the JsonStreamParser. (Constructing the stream)
Additional context

Related MatrixAI/Polykey#249
Related MatrixAI/Polykey#498
Tasks

[ ] 1. Format the message stream to be human-readable by incorporating newlines and whitespace.
[ ] 2. Modify parser to discard ' ', \n, \t, \r, \f, \v, \u200B, \u00A0.

Resolved in Factor out rpc

Hey @addievo don't close issues directly. Only when a PR is merged is the issue auto-closed.

Since the PR hasn't been merged. You need to say Fixes #1 in the list. No need for checkboxes. The template was designed to be good enough.

The core goal of this issue is to allow the stream JSON parser to allow for arbitrary white-space for it's input.
What the binaryToJsonMessageStream middleware does is takes a stringified JSON stream and converts it to JSON objects. Right now the input stream is expected to be separated by nothing. That is the messages are formatted with {...message}{...message}.
We want to allow for arbitrary white space in the stream when separating the messages. So any combination of white space should be valid on the stream. For example the following should be valid
// Normal separation
{...message}{...message}{...message}

// Spaces
{...message} {...message}    {...message}

// New lines
{...message}
{...message}
{...message}

// Any combination
{...message}                                  {...message}
                       {...message}
{...message}

Note that this is on the input to the parser and stream transform.
This is to allow flexibility with 3rd parties using the RPC API. I for example could connect to the rpc through netcat and write my own hand crafted JSONRPC messages that are separated by newlines.

Whitespace removal - possible change required in jsonstreamparser. See https://github.com/juanjoDiaz/streamparser-json/issues/35 - I suspect this requires going into the source code and modifying the tokeniser so it tokenises a whitespace token based on an array of candidate characters. Extend the separator option to have an array of potential strings instead of a single string like ['\n', '\t', '\r']. Alternatively, allow separator to instead be a regular expression. Remember that the tokeniser should be capable of greedily eating up all the separators into 1 token.

Will depend on https://github.com/juanjoDiaz/streamparser-json/issues/35. Putting this into the to do to revisit.

https://github.com/juanjoDiaz/streamparser-json/pull/36

juanjoDiaz/streamparser-json#36

@addievo what does that have to do with this problem?

In case you don't follow the streamparser-json repo, it turns our that using random whitespaces as separators was always allowed by simply setting the separator to and empty string ''

@amydevs can you verify if this is is already done as per the comment by @juanjoDiaz?

Upstream says it is supported already. @tegefaulkes @aryanjassal can check if this can be a quick fix.

Make sure to close this as you're working on it @aryanjassal.

I don't think I've gotten around to working on this issue yet, but I will update it when I start.


fixing environment

Description:
Fixing envioroment

This PR is outdated, will be close when revert-fix branch is merge


Question: Changing date to human readable format

I may miss the line, but it is possible to change "the number" before an url, to something more like
20191119-213530? For the folder structure and the index page.

Not currently, the plan is to move away from timestamps entirely and use hashes instead: https://github.com/pirate/ArchiveBox/issues/74
I'm going to close this for now, because while we may add support for other timestamp formats in the url to get snapshots in the future, it's not going to be until after v0.4 has been released and used for a while.


goverlay: 0.7.1 -> 0.9.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for goverlay is: "An opensource project that aims to create a Graphical UI to help manage Linux overlays"
meta.homepage for goverlay is: "https://github.com/benjamimgois/goverlay"
meta.changelog for goverlay is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/r50pm2w9ziq7rnab149nrr4w6pz4ycc5-goverlay-0.8/bin/goverlay had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.8 with grep in /nix/store/r50pm2w9ziq7rnab149nrr4w6pz4ycc5-goverlay-0.8
found 0.8 in filename of file in /nix/store/r50pm2w9ziq7rnab149nrr4w6pz4ycc5-goverlay-0.8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
goverlay

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r50pm2w9ziq7rnab149nrr4w6pz4ycc5-goverlay-0.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goverlay https://github.com/r-ryantm/nixpkgs/archive/441e5e4050b58f7d0b3331c951d776c043b73d1a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r50pm2w9ziq7rnab149nrr4w6pz4ycc5-goverlay-0.8
ls -la /nix/store/r50pm2w9ziq7rnab149nrr4w6pz4ycc5-goverlay-0.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

goverlay

Maintainer pings
cc @kira-bruneau for testing.

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

1 package built:

goverlay

It's builds ok, but it fails to find the breeze qt theme now:

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for goverlay is: "An opensource project that aims to create a Graphical UI to help manage Linux overlays"
meta.homepage for goverlay is: "https://github.com/benjamimgois/goverlay"
meta.changelog for goverlay is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9 with grep in /nix/store/vq53jgxh635s5nzybz2a4ryp8xxyscr8-goverlay-0.9
found 0.9 in filename of file in /nix/store/vq53jgxh635s5nzybz2a4ryp8xxyscr8-goverlay-0.9

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goverlay

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vq53jgxh635s5nzybz2a4ryp8xxyscr8-goverlay-0.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goverlay https://github.com/r-ryantm/nixpkgs/archive/4c8117dcac0b530afb09bcb0fa3a39fdc6b8252d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vq53jgxh635s5nzybz2a4ryp8xxyscr8-goverlay-0.9
ls -la /nix/store/vq53jgxh635s5nzybz2a4ryp8xxyscr8-goverlay-0.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

goverlay

Maintainer pings
cc @kira-bruneau for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for goverlay is: "An opensource project that aims to create a Graphical UI to help manage Linux overlays"
meta.homepage for goverlay is: "https://github.com/benjamimgois/goverlay"
meta.changelog for goverlay is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.1 with grep in /nix/store/jp6ks8icz5zr03blwbcy9k2gxx2ac9kj-goverlay-0.9.1
found 0.9.1 in filename of file in /nix/store/jp6ks8icz5zr03blwbcy9k2gxx2ac9kj-goverlay-0.9.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goverlay

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jp6ks8icz5zr03blwbcy9k2gxx2ac9kj-goverlay-0.9.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goverlay https://github.com/r-ryantm/nixpkgs/archive/e91751bd4936dd103b0b4053efc41d0ade9cff62.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jp6ks8icz5zr03blwbcy9k2gxx2ac9kj-goverlay-0.9.1
ls -la /nix/store/jp6ks8icz5zr03blwbcy9k2gxx2ac9kj-goverlay-0.9.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

goverlay

Maintainer pings
cc @kira-bruneau for testing.


pkg/option: remove dependency on viper

This reduces the number of dependencies imported into the package by 23. No
functional change is intended.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>Further PRs will remove the dependency on viper, as it pulls in a hefty number of dependencies wherever it is invoked.

This change is

test-me-please

Build failure: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/7680/testReport/junit/k8s-1/12/K8sPolicyTest_Basic_Test_Validate_to_entities_policies_Validate_toEntities_All/
Appears unrelated. Trying again.

Build failed - Jenkins wasn't even able to check out the code:
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/7706

test-me-please

test-me-please

Coverage increased (+0.005%) to 30.421% when pulling 8794052d752cce18093e5f46cab22b4c68f0215d on pr/ianvernon/remove-viper-option-dep into ddb79e0a6b526e8c00df38285b162b52ae67e110 on master.


Refactor registerInvoker

Refactor registerInvoker overloads to use generic argument extraction helper arg.

Current coverage is 62.10% (diff: 72.72%)

No coverage report found for master at 9ac8d86.

Powered by Codecov. Last update 9ac8d86...78fbeb7


daemon: refactor Hubble Exporters as a cell

Continuation of https://github.com/cilium/cilium/pull/35206

Introduce a new hubble-exporters cell that provides the static and dynamic exporters using observer Options.
The static exporter now uses the context received from OnDecodedEvent instead of embedding one at construct time.
Add OnExportEvent hook system to the exporter to provide extension capabilities.
Allow providing a custom writer to exporters.
Allow providing a custom encoder to exporters.

Related: #35514

Nice work @devodev 👍
Some comments and questions inline. Also I noticed that in exporter.Stop() we're closing the writer, which can be setup with os.Stdout and I don't think we're supposed to close stdout. Not to be addressed in this PR, but something to look into. cc @chancez whom I added as a reviewer as well since he has context about export.

Looked into it, and it seems we use a noopWriteCloser to avoid calling Close on the underlying writer.

Looked into it, and it seems we use a noopWriteCloser to avoid calling Close on the underlying writer.

Yep, there was previously a bug where we were closing os.Stdout that I fixed in a49276f38490b016d1af5f2721a88a9f2af1be40.

I hopefully addressed all the PR comments.
Thanks for the feedback, we should be ready to launch tests :)

/test

I have pushed 3 new commits that decouples the writer/encoder implementation from the exporter. I also merged the exporteroption package into exporter as I don't see the benefit of having these in a separate package and it avoids circular dependency issues when trying to use exporter types as options.

/test

Overall this looks good to me. It's definitely an improvement and I'm not going to block this from being merged.
However, I think I agree with David's point of observeroption.Option being too small and generic for the hive object graph. Currently no cell provides this to the global object graph, but the Hubble Cell still takes it as an input. This means that any Cell can still provide arbitrary options to the Hubble cell, which feels weird. An observer option is not really a service a cell can Provide.
In my mind it would for example make more sense if we'd basically provide the exporter itself. Maybe something like
// Provided by exporter cell
type Exporter interface {
  Export(ctx context.Context, ev *v1.Event) (bool, error)
}

...

// In hubble cell
for _, e := range h.exporters {
  observerOpts = appned(observerOpts, observeroption.WithOnDecodedEvent(e.Export)
}

It's very nitpicky, but Export is the service the exporter actually provides not the observer option. Also that way we don't open the hubble cell to receiving arbitrary observer options from arbitrary cells, which IMO can lead to fairly weird and unexpected spooky actions at a distance and harder to follow dependencies.

All good points, thank you 🦾 .
This PR changed quite a bit over time and the more I work on it the more I think like you and prefer this more explicit approach. It's also a good example we are setting for future cell refactoring in the hubble cell.
I'll rework this quickly.

/test

@kaworu ping 🏓


Artwork Template Request

Could you provide a print template to make custom artwork for the Gamepad-LED?
A PSD?

Ok I'll put it in the project.


[SPARK-27671][SQL] Fix error when casting from a nested null in a struct

What changes were proposed in this pull request?
When a null in a nested field in struct, casting from the struct throws error, currently.
scala> sql("select cast(struct(1, null) as struct<a:int,b:int>)").show
scala.MatchError: NullType (of class org.apache.spark.sql.types.NullType$)
  at org.apache.spark.sql.catalyst.expressions.Cast.castToInt(Cast.scala:447)
  at org.apache.spark.sql.catalyst.expressions.Cast.cast(Cast.scala:635)
  at org.apache.spark.sql.catalyst.expressions.Cast.$anonfun$castStruct$1(Cast.scala:603)

Similarly, inline table, which casts null in nested field under the hood, also throws an error.
scala> sql("select * FROM VALUES (('a', (10, null))), (('b', (10, 50))), (('c', null)) AS tab(x, y)").show
org.apache.spark.sql.AnalysisException: failed to evaluate expression named_struct('col1', 10, 'col2', NULL): NullType (of class org.apache.spark.sql.t
ypes.NullType$); line 1 pos 14
  at org.apache.spark.sql.catalyst.analysis.package$AnalysisErrorAt.failAnalysis(package.scala:47)
  at org.apache.spark.sql.catalyst.analysis.ResolveInlineTables.$anonfun$convert$6(ResolveInlineTables.scala:106)

This fixes the issue.
How was this patch tested?
Added tests.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10579/
Test PASSed.

Test build #105307 has started for PR 24576 at commit 397409b.

Test build #105307 has finished for PR 24576 at commit 397409b.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10600/
Test PASSed.

Test build #105328 has started for PR 24576 at commit d38b371.

Test build #105328 has finished for PR 24576 at commit d38b371.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10604/
Test PASSed.

Test build #105332 has started for PR 24576 at commit a5c2416.

Test build #105332 has finished for PR 24576 at commit a5c2416.

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

Merged build finished. Test PASSed.

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

cc @cloud-fan

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10625/
Test PASSed.

Test build #105360 has started for PR 24576 at commit a5c2416.

Test build #105360 has finished for PR 24576 at commit a5c2416.

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

Merged build finished. Test PASSed.

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


Auth Method

I am using alexa cake bake in Siemens Unified Comfort Panel which is not letting me open amazon sign in page from local host.😟
I also tried to get cookie.mid from web developer tool from site but that is also not working.
If I open link from doc it shows can not get cookie.mid
I tried with email id password also but it is showing me "Amazon-Login-Error: Important Message!: To better protect your account, please re-enter your password and then enter the characters as they are shown in the image below."is there any way out?please help

same error no fix for that thats not fixable thats because amazon thinks your nodered is a bot and tries to hack the account and gives out a chapta


Bug: C++ base_directory in Project settings is not works well

Description:

When I set base_directory to "generated" and generate code, files are not created.
The project file is located at src/ui

Would I be correct in assuming that you only have a single "generated/" directory under src/ui/? Also, is there also a "generated/" prefix in the publicationframe that you specified as the base_file for the frame you are creating?

This has been fixed via PR #1371 and should appear in Daily Builds starting on 1/8.
The issue was that the Project file specified generated as the base_directory, and the form's base_file also specified the same folder. A check is now made for that situation, and it will now work correctly with or without the folder name in the form's output file starting with the same folder name as the Project's base_directory.
Thanks for reporting this!


RI 15.1 -> master

@rainersigwald if CI passes I would like to do this now to get a new build of master out. Holding off on #1836 for now.


[#149] Fix static minibuffer height for dynamic collections

This lets the minibuffer grow dynamically until selectrum-num-candidates-displayed is reached. This restores the behaviour before #133 which introduced this new/wrong behaviour.

I will merge this right away because the bug can be quite annoying and this fixes it. We can still discuss additional changes if you like that I can implement later.


how to enable include mode without using web server

how to enable include mode without using web server.
i dont want use web panel is there any method include mode and exclude mode without web server ?

MQTT?

MQTT? Websocket?

how to enable include mode with MQTT  ?

I have updated my commets with links

I have updated my comment with links
thank u sir


What is the encouraged way to detect a window focus loss?

Hello,
I'm trying to detect whether my root view has lost focus.
So I thought this would trigger whenever a focus loss occurs:
protected override void OnDeactivate()
{
    Debug.WriteLine("I've lost focus in OnDeactive'");
    base.OnDeactivate();
}

But unfortunately this only gets called upon a window minimize. A focus loss doesn't trigger OnDeactivate.
So what I wrote instead, just to see whether it even should be triggered upon a focus loss is this:
protected override void OnViewLoaded()
{
    ((RootView)View).Deactivated += (sender, args) =>
    {
        Debug.WriteLine("I've lost focus in Deactivated'");
    };
    base.OnViewLoaded();
}

That works, but it's not very MVVM, I can call OnDeactivate() from there so that conductors trigger OnActivate() on their items etc. etc., but then I'm also responsible for calling OnActivate().
Also this might be undesirable since minimize guarantees the user can't see the window whereas a focus loss does not. (For me the focus loss is a requirement), so what's the suggested route? :)
Oh, and thanks for your work, I've been enjoying Stylet

You can use Stylet's Actions with events.
<Window ... Deactivated="{s:Action WindowDeactivated}">

Then define a WindowDeactivated method in your VM.

Hey, thanks for the quick reply
That'll work for me. But to let child VMs know about the focus change I'm best off with the EventAggregator?

Probably. It might make sense to replace the WindowConductor with something of your own, which activates/deactivates the main window VM based on focus state, but it's a little hard to tap into that at the moment: you'll have to subclass the WindowManager.

Thanks, I'll go with the EventAggregator

If take a PR which let's you replace the WindowConductor a bit more easily, but I'm afraid I don't have much time to do it at the moment.


no navigation at all

There is no navigation in the docs - you forgot that?
You have a content list on the left side, but there is no navigation at all, at least I can not see one. Also there does not seem to be any kind of keyboard navigation. Why not?

The left navigation bar isn't just a content list, it has sections that expand and collapse, and the sections contain more pages.
Can you please provide examples of what you have in mind?

I'm going to raise a GitHub issue about adding keyboard navigation to the docs. (it will be in a different repository)

Not applicable.


Add zif_aff_types_v1=>ty_header_80_cloud

#662 requires ty_header_80_cloud in zif_aff_types_v1.

As an alternative to merging schneidermic0:add-header-80-cloud into WDFdaniel:feature/suco you can also add an temporary exemption for abaplint here until SUCO is merged:
https://github.com/SAP/abap-file-formats/blob/06134bd217a27ed1b039bf6b963f99c32af793e8/abaplint.jsonc#L198
Otherwise, when merging WDFdaniel:feature/suco (with schneidermic0:add-header-80-cloud merged into it) this PR gets obsolete as the commits are coming from the SUCO Pull Request.

zif_aff_types_v1=>ty_header_80_cloud will be introduced with #662. Therefore, I will close this PR.


Add a note point  in Prerequisites of openebs0.2 documentation about Enabling Intel virtualization hardware extensions in BIOS

@raghavarora16 #99
make a note point to the Beginners so that they will check once with
Enabling Intel VT and AMD-V virtualization hardware extensions in BIOS

@sandhya  : All the beginners , if u are getting any trouble by setting up a project with Linux suppose vagrant up , immediately check ur bios mode and enable the Intel virtualization acceleration key.


basis lint exit status is 0 when warnings found

I have several presets. basis lint when running on the whole set of presets finds warnings in certain presets, but gives 0 return code anyway.
macbook:web-frontend ivstrelkov$ ./node_modules/.bin/basis lint
Run 7 lint tasks:
  chubaka/app          FAIL 3 warnings found
  chubaka/maintenance  OK
  chubaka/landing      OK
  chubaka/terms        OK
  khansolo/app         FAIL 3 warnings found
  khansolo/landing     OK
  khansolo/terms       OK

All tasks done in 6.409sec
macbook:web-frontend ivstrelkov$ echo $?
0

Moved to https://github.com/basisjs/basisjs-tools-build/issues/9


Problem with Purchase Line Discounts

When posting a Purchase Order, the Line Discount Amount is not being taken into consideration. After checking the code, I have found that there is a problem with the codeunit 90 in the DivideAmount function.
    "Line Amount" := ROUND(PurchLineQty * "Direct Unit Cost",Currency."Amount Rounding Precision");
    IF PurchLineQty <> Quantity THEN
      "Line Discount Amount" :=
        ROUND("Line Amount" * "Line Discount %" / 100,Currency."Amount Rounding Precision");
     "Line Amount" := CalcLineAmount;

And it should be: "Line Amount" := "Line Amount" - "Line Discount Amount";

HI @gasalazar,
This repository is meant for the new development environment, not for bugs in the applications business logic.
We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:

Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using.
Thanks for reporting the issue.


add Cake Wallet & Monero.com to InternalVerificationInfoDatabase.kt

The contribution guide says

Make one pull request for every app instead of multiple apps in one pull request.

😉


Declare a typealias for finder to improve readability

Hope it makes code readability better, following lambdas function to function is a little hard for me..

@JakeWharton is this repository maintained?

Nope
On Fri, Apr 19, 2019 at 6:49 AM Keivan Esbati<EMAIL_ADDRESS>wrote:

@JakeWharton https://github.com/JakeWharton is this repository
maintained?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JakeWharton/kotterknife/pull/85#issuecomment-484849493,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQIENWR7EOQUTX2F2UDHDPRGPR3ANCNFSM4HHDHDYQ
.

Nope
…
On Fri, Apr 19, 2019 at 6:49 AM Keivan Esbati @.***> wrote: @JakeWharton https://github.com/JakeWharton is this repository maintained? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#85 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAQIENWR7EOQUTX2F2UDHDPRGPR3ANCNFSM4HHDHDYQ .

Want me to update README?

No thanks

You made me happy
Thanks


[BUG] Unable to run H2O LLM Studio CLI in T4 Colab

🐛 Bug
Just tried running the provided Colab.
This step gave error !make setup > /dev/null
ERROR: Couldn't install package: {}
Package installation failed...
/usr/lib/python3.10/subprocess.py:1072: ResourceWarning: subprocess 812 is still running
_warn("subprocess %s is still running" % self.pid,
ResourceWarning: Enable tracemalloc to get the object allocation traceback
sys:1: ResourceWarning: unclosed file <_io.TextIOWrapper name=4 encoding='utf-8'>
ResourceWarning: Enable tracemalloc to get the object allocation traceback
sys:1: ResourceWarning: unclosed file <_io.TextIOWrapper name=7 encoding='utf-8'>
ResourceWarning: Enable tracemalloc to get the object allocation traceback
make: *** [Makefile:29: setup] Error 1

I can't reproduce the error on the linked colab notebook. It runs through smoothly. Nevertheless, I'll go ahead and upgrade it to the latest release version (v1.11.1)

Updated both with the latest release version and tested successfully on T4:
https://www.kaggle.com/code/ilu000/h2o-llm-studio-cli/
https://colab.research.google.com/drive/1soqfJjwDJwjjH-VzZYO_pUeLx5xY4N1K?usp=sharing


Having 2 Modals

Hi
I want to get a confirmation when someone wants to close the modal. This confirmation is itself another modal! But when I summon the other modal it goes behind the original one!
is there any work around?
Thnx

Hi,
Tingle is not intended to allow multiple modals opened at the same time. You have basically two solutions :

close the first modal by using a callback and open the second one (you can always reopen the first one later)
replace content and buttons dynamically on the first modal to propose confirmation

It doesnt make sense anyway. A modal is consiered to be a a singular entity, not just another window.
BTW: I'd go with option #2 - replace content :)
cu, w0lf.

Hello,
thanks for your response. I tried your 2nd option. So basically I use modal.getContent() and save it in a variable then use modal.setContent("Bla Bla Bla...") to create my confirmation message.
The problem is when I want to set the modal content to the initial modal using my saved variable. Then I get this error:

HierarchyRequestError (DOM Exception 3): The operation would yield an incorrect node tree.

Probably a bit late but:
Have you tried adjusting your z-indexes? If you assign an extra class to your second modal ("more-important-modal-than-the-first-one") with a higher z-index, it should appear on top of the previous one.
I wouldn't bug my users like this anyway..

Hi @BardiaKh,
Sorry for the delay. If you're still struggling with this one, could you open a new issue and attach some code related to the setContent problem?


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


style: redesign programming card with css grid

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: https://vercel.com/tecogill/prolang-api/6vz1CZrVq3qmcEnWbq6aN7mkQNDZ
✅ Preview: https://prolang-api-git-style-redesign-ui-tecogill.vercel.app


BigCommerce detection not working

Platform
Which platform does the bug apply to?

WebExtension (Google Chrome, Mozilla Firefox)
NPM
Bookmarklet
Website

Describe the bug
For some sites, BigCommerce detection is not working.
To Reproduce
Steps to reproduce the behavior:

Go to https://www.blackdiamondequipment.com/en_US/shop/built-for-beyond/
Check eCommerce section and you will not see 'BigCommerce'

Current detection logic -
"BigCommerce": {
"cats": [
6
],
"description": "BigCommerce is a hosted ecommerce platform that allows business owners to set up an online store and sell their products online.",
"html": "]+cdn\d+\.bigcommerce\.com/",
"icon": "BigCommerce.svg",
"pricing": [
"low",
"recurring"
],
"saas": true,
"scripts": "cdn\d+\.bigcommerce\.com/",
"url": "mybigcommerce\.com",
"website": "http://www.bigcommerce.com"
},
This site has this CDN when you look at the HTML but it's in an  tag
<img data-src="https://cdn11.bigcommerce.com/s-hgn1l9sh63/images/stencil/550w/attribute_rule_images/8367_source_1613594431.png" src="https://cdn11.bigcommerce.com/s-hgn1l9sh63/images/stencil/550w/attribute_rule_images/8367_source_1613594431.png" alt="Recon Access Mitts" class="fade-image lazy-image img-loaded" style="transition:opacity 0.4s ease-out;" data-v-37e7a394="">

@rockeynebhwani Hey mate , thanks , updated.


Fixes/copyright year

Related Issue

Issue No. #370

Changes you made :

Linked the file with function to update year with Contact, About , Favourite , and Our Tours page
Manually updated year for current year

Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

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

Output Screenshots :

@inderharrysingh Nice one
Star the repo ⭐ to show your love.


Add docs on how to create a new Cog

Add details in CONTRIBUTING.md on how to create a new command within a cog, and how to attach it to the bot

Documentation for how to create a response button should also be added.
These docs can be added to a Guides subheading within CONTRIBUTING.md.

Documentation for how to create a new environment variable should also be added.


*: replace pilosalib with new pilosa driver

Will fail until https://github.com/src-d/go-mysql-server/pull/373 is merged and this is updated

As @kuba-- said in go-mysql-server issue it would be nice to add a note in the documentation to migrate from old pilosa driver. This may be done after the code is merged.

now, tests are failing

@kuba-- yep:

Will fail until src-d/go-mysql-server#373 is merged and this is updated

Should be fixed now

Still failing, because of integration tests. For instance we have just one constructor NewDriver, now.


Generated SDK for v2023-04-01 (GA)(Do Not Release before 1 May 2023)

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Workloads

Since it's beta and there is no official release before, the breaking change is fine.

Hi @archerzz
This would be a stable release since it's for a GA api-version. We are still improving tests internally, so please hold the merge before we confirm here. We'll communicate for release separately as we want it close to GA announcement i.e. 15/5/2023.

CC @ArthurMa1978 for his awareness

Hi @archerzz  @ArthurMa1978
Please help with release dates.


in yew version = "0.21.0"  and yewdux = "0.9.3" error will generated

in  use_store
let (store, dispatch):(Rc,Dispatch) = use_store::();
Error: the trait bound impl yew::functional::hooks::Hook<Output = (Rc<store::Store>, yewdux::dispatch::Dispatch<store::Store>)> + '_: Hook is not satisfied
the trait Hook is implemented for BoxedHook<'_, T>

The Yew version needs to be updated. Should have some time soon to release a fix

The Yew version needs to be updated. Should have some time soon to release a fix

From what I found out, updating yew to v0.21.0 is not going to be enough.
There has been some breaking changes, see: yewstack/yew#3289.
I'm receiving other error, let me quickly give a sample:
#[derive(Clone, Debug, Default, PartialEq, yewdux::store::Store)]
pub struct StateSprites(RefCell<HashMap<String, SpriteData>>);

pub enum SpritesManagerMsg {
    SpritesInjected(Rc<StateSprites>),
}

pub struct SpritesManager {
    _state_listener: Dispatch<StateSprites>,
    state: Rc<StateSprites>,
    element: Element,
}

impl SpritesManager {
    const ID: &'static str = "svg-sprites";
}

impl yew::Component for SpritesManager {
    type Message = SpritesManagerMsg;
    type Properties = ();

    fn create(ctx: &yew::Context<Self>) -> Self {
        let cb = ctx.link().callback(Self::Message::SpritesInjected);
        let dispatch = Dispatch::<StateSprites>::subscribe(cb);

        // Redacted to for simplification
    }

    // Rest is redacted to for simplification
}

I'm receiving this error message on the:
let dispatch = Dispatch::<StateSprites>::subscribe(cb);
Diagnostics:
1. expected a `std::ops::Fn<(std::rc::Rc<utils::sprites::store::StateSprites>,)>` closure, found `yew::Callback<std::rc::Rc<utils::sprites::store::StateSprites>>`
   the trait `std::ops::Fn<(std::rc::Rc<utils::sprites::store::StateSprites>,)>` is not implemented for `yew::Callback<std::rc::Rc<utils::sprites::store::StateSprites>>`
   the trait `yewdux::subscriber::Callable<S>` is implemented for `yew::callback::Callback<std::rc::Rc<S>>`
   required for `yew::Callback<std::rc::Rc<utils::sprites::store::StateSprites>>` to implement `yewdux::subscriber::Callable<utils::sprites::store::StateSprites>` [E0277]
2. required by a bound introduced by this call [E0277]

Any updates? I am too getting the same error.


ERROR: Service 'frontend' failed to build: Error parsing reference:

I'm getting this error. Please help to resolve.

@smanikandan-btech what version of docker-compose are you using?

@chrisleekr  I'm using docker-compose version 1.22.0

Hm, strange.
I am currently using
Docker version 18.06.1-ce, build e68fc7a
and
dockerdocker-compose version 1.22.0, build f46880f
I just cloned the repo and tried to build again, it worked.
Are you using latest docker? Could you try with latest docker?

@chrisleekr Thanks for your suggestion. I will try and let you know.

@smanikandan-btech have you fixed the issue? If yes, I will close the issue.

@chrisleekr  No. I didn't fixed. I have no idea. Can't get time.


[Android] Fix the crash issue when use the cordova plugin.

In cordova-plugin-crosswalk-webview, the AttributeSet is null, please
look at
https://github.com/crosswalk-project/cordova-plugin-crosswalk-webview/blob/master/platforms/android/src/org/crosswalk/engine/XWalkCordovaView.java#L55
app will crash with null pointer.
Add the conditional for AttributeSet.
BUG=XWALK-6960
(cherry picked from commit 5bb898ac7311eace05a9a67d92ed5f25107fc34c)

@lincsoon Please help merge to Crosswalk-20, thanks.


ethjs-rpc swallows RPC error details.

ethjs-rpc
Issue Type

[X] Bug (https://github.com/ethjs/ethjs-rpc/blob/master/.github/CONTRIBUTING.md#bug-reports)

Description
JSON-RPC error details are swallowed by the library.  https://github.com/ethjs/ethjs-rpc/blob/master/src/index.js#L49 has
String(err) || (JSON.stringify(responseObject.error, null, self.options.jsonSpace))

When err is null, this code will evaluate to null (the string) because String(null) === "null" which is truthy.
Steps to reproduce

Make a request that results in a JSON-RPC erorr (e.g., call a contract function that throws an exception).

Versions

Node/NPM: v8.5.0/5.3.0

Is this project still maintained?  This issue is killing us, just need the PR lined off this issue to be merged and an NPM release cut.  😕