JDBC Template Mapping for Reading Database Views (Stopped working from 3.18)

We’ve encountered a problem in sfm-jdbc 3.18+ where we are getting inconsistent sql statements when mapping from jdbc template to Java class using database views for the purpose of search/read. This use to work prior to 3.18.
The error seems to refer to  a mismatch of columns when mapping:
org.springframework.dao.TransientDataAccessResourceException: ConnectionCallback; Column Index out of range, 12 > 11. ; nested exception is java.sql.SQLException: Column Index out of range, 12 > 11.
We are using the following for invocation:
JdbcTemplateMapperFactory.newInstance()
.crud(Car.class, Long.class)
.lazilyTo(jdbcTemplate(), "car_view")

car_view is database view off of a table car with an additional column called car_expenses. Note we are using MYSQL 5.7. Car.class contains property for car_expense.
When we invoke JdbcTemplateCrud.read(ids, entities::add), the last three SQL statements look as follows prior to 3.18:
SELECT * FROM car_view WHERE 1 = 2
SHOW KEYS FROM `car` FROM `db`
SELECT * FROM car_view WHERE id in (5)

On and after 3.18:
SELECT * FROM car_view WHERE 1 = 2
SHOW KEYS FROM `car` FROM `db`
SELECT * FROM car WHERE id in (5)

What seems to happen is that the columns from car_view are fetched and when mapping occurs to Car.class, the ResultSet of car is taken and for each column of car_view, the value from the ResultSet is fetched to map to Car.java setters, however once we reach a column from car_view not in car, then we get the Column Index exception.
In this case the column car_expense not being in car table. (Referring to the logic in map.fieldmapper/FieldMapperimpl.java)
To me, the behaviour should be that the three statements should refer to the provided table from the initial invocation, however this is a different case where we are using a database view.
When tracing the code, the problem seems to stem from org.simpleflatmapper.jdbc.impl.CrudMeta.java:104 where the table name is switched to the one provided by the ResultSetMetadata of the first SQL statement execution.
ResultSetMetadata.getTableName() is giving back the original table name of the database view which returns car, instead of car_view, thus causing this problem to arise where the third sql statement executes on car.
This use to work correctly prior to 3.18, as this logic was introduced through:
https://github.com/arnaudroger/SimpleFlatMapper/commit/32c065b28787dbc7028aeb2b61668a68370b906c#diff-eea491b882fb1bbbc16ce19475df024f
Through the debugger when changing table to point to car_view instead of car, the mapping works as expected.
According to the logic of java.sql.ResultSetMetadata.getTableName(), there is a flag called useOldAliasMetadataBehavior where if true it will return the alias table name otherwise original table name. Default useOldAliasMetadataBehavior=false.
For MYSQL, this flag can be controlled through a parameter passed through database url however this will cause other problems in the mapping logic where keys are no longer found through the database view.
My question is, are views supported through the mapper or are we suppose to map to database views differently?
Your help in clarifying this is much appreciated.
Thanks :)

will have a look thanks for the detailed report!

quick question, I'm planning to fix it in the next release 6.0.2 are you happy to upgrade to that. the changes between 3 to 6 are mainly on the mapper producer change so should not need any change.

Yup, that's fine. Thanks for the fast response :)

which version of mysql connector are you using?

also it looks like from version 3.18.0 it generate the select statement with the column name not the * but I might be mistaken. that would not solve the view problem if the view add a filter on it though. would you mind trying with the last version ?

also checkout that test to see if I missed something
https://github.com/arnaudroger/SimpleFlatMapper/commit/64d24a2ceef868a529de59a6b95da8945a614e82

I'm using mysql connector 6.0.6.
view is declared as:
CRETE VIEW car_view AS
SELECT `c`.*, SUM(.....) AS `car_expenses`
FROM `car` AS `c` LEFT JOIN ...(some other table)
                                   LEFT JOIN ...(some other table)
GROUP BY `c`.`id`

All tables are on the same schema.

thanks got it, it's the group by that changes mysql behavior ...

just pushed 6.0.2 that should have that fixed. could you try it out?

I got this error for the db view after upgrading which makes sense as views don't have keys:
Caused by: java.lang.IllegalArgumentException: No key defined to map to class java.lang.Long, specify key using DSL or add a primary key to the table
After adding keys via: JdbcTemplateMapperFactory.newInstance().addKeys("id").crud(...)...
It is working! :)
Thanks a lot for your help!


Feature/translations & flowbite lib

Use Flowbite lib for quick development (responsive design and component already made)
install translation for eng and fr

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8b237f0) 100.00% compared to head (768af19) 100.00%.
Report is 4 commits behind head on development.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           development        #7   +/-   ##
=============================================
  Coverage       100.00%   100.00%
=============================================
  Files                1         1
  Lines                2         2
=============================================
  Hits                 2         2

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


npm build warnings in CI

I think it would be good if this warning would stop the CI:
  WARNING in ./rdmo/projects/assets/js/projects/components/helper/PendingInvitations.js 25:41-48
  export 'default' (imported as 'baseUrl') was not found in 'rdmo/core/assets/js/utils/meta' (possible exports: baseUrl, language, siteId, staticUrl

Any ideas @MyPyDavid @afuetterer ?

Ive added some settings to the webpack.config in here https://github.com/rdmorganiser/rdmo/pull/1203, it should fail at the build:prop because of that warning.

we have now build:dist that fails on non-performance warnings.


Replace q with $q

q was not working properly. It was difficult to actually flush the promise resolution. Instead, we'll use $q.

:shipit:


Meshes for the Humanoid experiments

Hi,
I don't see any meshes folder for the URDF files placed in iGibson/igibson/data/assets/models.
For instance, fetch.urdf line 11 has meshes/base_link.obj. But there is no such folder.
Where can I get them? Or did I miss something in the instructions?
Kindly help.

Hi, sorry for the late update. I just uploaded all the model assets to this folder. Feel free to download and try again with the code. Thanks!


Spyder fails to launch with PySide on Windows

From<EMAIL_ADDRESS>on 2012-03-22T11:29:36Z
What steps will reproduce the problem?

Update to latest 2.2.0dev ( revision 46defadf01ae )
Launch "bootstrap.py --gui pyside -- debug"

I tried this on more than one machine, and the behavior was the same on both. What is the expected output? What do you see instead? Spyder fails to launch. Traceback is:
c:\src\spyderlib>bootstrap.py --gui pyside -- debug
Executing Spyder from source checkout Revision 1945 :46defadf01ae, Branch: default

Patched sys.path with C:\src\spyderlib
Skipping GUI toolkit detection
Traceback (most recent call last):
File "C:\src\spyderlib\bootstrap.py", line 76, in <module>
from spyderlib import spyder
File "C:\src\spyderlib\spyderlib\spyder.py", line 75, in <module>
from spyderlib.qt.QtGui import (QApplication, QMainWindow, QSplashScreen,
File "C:\src\spyderlib\spyderlib\qt\QtGui.py", line 13, in <module>
from PySide.QtGui import *
ImportError: DLL load failed: The specified procedure could not be found. Please use labels and text to provide additional information. I bisected the repository and found that revision ffbfa56f4872 is the source of the issue. I'll create a code review highlighting the offending line.

The problem shows up in spyder.py. First, line 31 requests an import of Spyder's qt module:
from spyderlib import qt
This is supposed to trigger Spyder's internal logic to import the right flavor of Qt. In previous revisions, this was accomplished by importing all of PySide right from the Spyder's qt module. Now the qt module doesn't really import much from PySide. It just sees if it can get a version number. Then at line 36:
requirements.check_qt()
The requirements checking process actually imports PyQt4.QtCore. The real root of the problem then comes at line 75 where QtGui is imported. What happens is that during the import of PySide.QtGui python finds that it already has a copy of QtCore imported, and it tries to use PyQt4.QtCore with PySide.QtGui, and everything falls apart from there. I verified this using Procmon.exe to watch the system search for modules and DLLs.
As a quick test, if I comment out the requirements check at line 36 of spyder.py, Spyder with PySide launches as expected. The import of PySide.QtGui triggers an import of PySide.QtCore as it should.
Windows 7
Python 2.7.2
Qt 4.7.4
PySide 1.1.0
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=975

From<EMAIL_ADDRESS>on 2012-03-23T00:54:42Z
The https://code.google.com/p/spyderlib/source/browse/spyderlib/requirements.py#49 check_qt() should be modified to check spyderlib.qt.version - this will introduce dependency from spyderlib.qt, but we can fix that by version and binding flavor type explicitly from spyder.py
We also need to expose qt._modname with as a public API - something like qt.binding

From<EMAIL_ADDRESS>on 2012-03-23T02:32:03Z
WIP patch that should fix the issue, but leaves many questionable TODOs
Attachment: fix.qt.requirements.check.diff

From<EMAIL_ADDRESS>on 2012-03-23T00:54:56Z
Status: Accepted

From pierre.raybaut on 2012-03-25T14:29:01Z
This issue was updated by revision 9c4649090f0a .
Status: Fixed

From<EMAIL_ADDRESS>on 2012-03-23T08:14:01Z
Of course, the simple fix was to simply revert revision ffbfa56f4872 :). But I agree that the better approach is to improve Spyder's Qt proxy. Factoring in IPython's need for API #2 complicates matters a bit. It appears there are two design goals:

Allow the user to select which PySide or PyQt. (Which should be default? Not sure PySide is ready yet to become the default.)
Figure out which API version needs to be loaded based upon the needs of other packages and perhaps the user's desire to enforce a specific API version.

Ideally, all of this would be accomplished through importing only version information from the various packages. No actually imports of QtCore, QtGui, etc., should happen until after all the above options have been finalized at run time.


Eliminate console window display when running in separate process

This fix makes use of a pre-release build of NUnit Engine 3.6 to avoid creating a window when launching the agent process. It also fixes a problem that prevented use of RunAsync in the gui when running tests.

You don't want them to be absolute. The projects are downloaded by each developer to a potentially different place on their disk drive
Since @rprouse just merged my fix to the engine into master, I'm going to change the reference form a -ci- to a -dev- build before I merge this.

What I meant was more of a relative absolute. instead of ..\..\packages\NUnit.Engine.3.6.0-ci-03382-pr-146 ..\..\packages\Nunit.Engine_HEAD or something like that. But that would make it hard on NuGet because it does a lot of cross checking of assemblies and versions and dependencies and so forth.

I see... In general, we deliberately avoid any automatic updating of dependencies to the latest versions, preferring to create an issue and PR whenever we need to do it. That keeps things more independent than they would otherwise be. Remember too that making these different components independent of one another was one of our goals for NUnit 3.


Correction of encoding + error with bsdf viewer

@StefanThoene Would you mind checking my pull request?
There are 2 corrections
•	For the encoding I added some lines to check the encoding. I just checked and it should work.
•	For the Speos issue, I have removed the “222”. The code used to be:
bsdf_viewer = CreateObject("SimpleBSDFSurfaceViewer.Application222")
And is now
bsdf_viewer = CreateObject("SimpleBSDFSurfaceViewer.Application")
Thank you.

@SandAuriol  i hope thes changes are fixing the precommit-> i will now start reviewing the changes

Thank you @StefanThoene !


Bug: if you run a single COMMAND that is not COUNT, you cannot view the results.

Repro: set COMMANDS ?= TOP_10
Result:

If you duplicate and rename TOP_10 in the result.json, you can view a report by changing selection.
Therefore I am guessing that since state.mode of Benchmark defaults to COUNT and we only have 1 key (which is not COUNT) therefore we cannot change the mode onChange?
I can't confirm this since I am unable to rebuild the web UI.
As part of this, we should update the development guide with instructions on how to do so.

+1 for instructions on how to rebuild the UI!  Does anyone know how?

I think yarn build is enough to rebuild the UI. Although the website that it loads, is from some out of date json. You still have to make serve_local to see your results.json and the UI changes.


[Feature Request] Container Colors

An option @settings to enable a small colorized line in color of each container would be great.
The small vertical line is best placed at left side of favicon.

I would add that it would be better to have the container tab's custom icon to the left of the favicon on each sidebar tab, along with a border colorizing the tab with the container's specified color, for quick visibility and grouping.


Set LOG_FILE and rename logback.xml

Fixes #1931

or @Sheikah45 have you tried setting logging.file in an environment variable in the Intelij run config maybe that helps.


Add Cordova.gitignore to use with Adobe Cordova projects

Thanks, but se the existing issue at #1061

Oh, sorry I haven't seen that issue.


[#44] Add API to disable tap-to-dismiss

Hi @rnystrom, this PR implement the #44.

Add a delegate method to check allow tap to dismiss. (If no delegate, default can be tap dismiss)
Add a delegate property for some use case, for example, when wrapped the delegate pattern to Delegate Proxy with RxSwift.

If there have any suggestions or concerns, please let me know. 😃 Thanks.

Hi @BasThomas, could you help to review this PR and merge it? 🙏

LGTM :) @marcuswu0814  I don't have the write access to this repo though Im in the GItHawk org group


Implement a normalizer

Dude, an extension called Audio EQ for Chrome does just this. It works seamlessly with Streamus.

Thanks for the tip! I will take a look when I can. The reviews weren't
great for it, but maybe it has some useful tips.
On Fri, Feb 13, 2015 at 11:07 AM, WooHooBar<EMAIL_ADDRESS>wrote:

Dude, an extension called Audio EQ for Chrome does just this. It works
seamlessly with Streamus.
—
Reply to this email directly or view it on GitHub
https://github.com/MeoMix/StreamusChromeExtension/issues/123#issuecomment-74308048
.

Hey guys,
@WooHooBar I have Audio EQ, but it doesn't seem to affect Streamus :disappointed:
(It does affect videos played directly on Youtube) How did you get it working?
@MeoMix Does Audio EQ work with Streamus for you? (Insanely awesome extension by the way, a million thanks!!)
(Chrome 43.0 on Win8.1 here, if that matters)


the location drag 'n drop is bugged when opening modals

I think the router events fight each other, because the back-event on the catch of the modal will be correctly executed. still, the modal remains in the current route. probably because the drag 'n drop component pushes the router state again when syncing?

resolved in #887.


Distributed training error

I want to do distributed training on PASCAL VOC2007. When running the code “ PYTHON=python3 ./tools/dist_train.sh configs/mask_rcnn_r50_fpn_1x.py 2 ”,  the training could not be started. How should I do. The error is shown below:
power@ubuntu:~/mmdetection$ PYTHON=python3 ./tools/dist_train.sh configs/mask_rcnn_r50_fpn_1x.py 2
2018-12-10 21:31:02,984 - INFO - Distributed training: True
2018-12-10 21:31:03,493 - INFO - load model from: modelzoo://resnet50
2018-12-10 21:31:03,747 - WARNING - unexpected key in source state_dict: fc.weight, fc.bias
missing keys in source state_dict: layer3.4.bn1.num_batches_tracked, layer4.2.bn2.num_batches_tracked, layer3.1.bn3.num_batches_tracked, layer4.0.downsample.1.num_batches_tracked, layer4.1.bn1.num_batches_tracked, layer2.1.bn2.num_batches_tracked, layer1.0.bn3.num_batches_tracked, layer3.2.bn3.num_batches_tracked, layer1.0.downsample.1.num_batches_tracked, layer3.0.downsample.1.num_batches_tracked, layer3.3.bn3.num_batches_tracked, layer4.0.bn2.num_batches_tracked, layer1.0.bn2.num_batches_tracked, layer4.1.bn3.num_batches_tracked, layer2.2.bn2.num_batches_tracked, layer3.5.bn3.num_batches_tracked, layer1.2.bn3.num_batches_tracked, layer1.2.bn1.num_batches_tracked, layer2.0.downsample.1.num_batches_tracked, bn1.num_batches_tracked, layer4.0.bn1.num_batches_tracked, layer2.3.bn1.num_batches_tracked, layer1.1.bn3.num_batches_tracked, layer4.2.bn1.num_batches_tracked, layer1.1.bn1.num_batches_tracked, layer4.1.bn2.num_batches_tracked, layer3.2.bn2.num_batches_tracked, layer3.0.bn2.num_batches_tracked, layer3.4.bn2.num_batches_tracked, layer2.1.bn1.num_batches_tracked, layer4.0.bn3.num_batches_tracked, layer2.0.bn3.num_batches_tracked, layer1.2.bn2.num_batches_tracked, layer3.1.bn1.num_batches_tracked, layer3.5.bn2.num_batches_tracked, layer1.1.bn2.num_batches_tracked, layer3.0.bn3.num_batches_tracked, layer1.0.bn1.num_batches_tracked, layer2.2.bn3.num_batches_tracked, layer3.0.bn1.num_batches_tracked, layer3.5.bn1.num_batches_tracked, layer2.3.bn3.num_batches_tracked, layer3.3.bn1.num_batches_tracked, layer2.0.bn2.num_batches_tracked, layer4.2.bn3.num_batches_tracked, layer3.3.bn2.num_batches_tracked, layer3.4.bn3.num_batches_tracked, layer3.2.bn1.num_batches_tracked, layer2.3.bn2.num_batches_tracked, layer3.1.bn2.num_batches_tracked, layer2.1.bn3.num_batches_tracked, layer2.0.bn1.num_batches_tracked, layer2.2.bn1.num_batches_tracked
2018-12-10 21:31:06,964 - INFO - Start running, host: power@ubuntu, work_dir: /home/power/mmdetection/work_dirs/mask_rcnn_r50_fpn_1x
2018-12-10 21:31:06,964 - INFO - workflow: [('train', 1)], max: 12 epochs
Traceback (most recent call last):
File "./tools/train.py", line 81, in
main()
File "./tools/train.py", line 77, in main
logger=logger)
File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/apis/train.py", line 57, in train_detector
_dist_train(model, dataset, cfg, validate=validate)
File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/apis/train.py", line 92, in _dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/usr/local/lib/python3.6/dist-packages/mmcv-0.2.0-py3.6.egg/mmcv/runner/runner.py", line 349, in run
epoch_runner(data_loaders[i], **kwargs)
File "/usr/local/lib/python3.6/dist-packages/mmcv-0.2.0-py3.6.egg/mmcv/runner/runner.py", line 251, in train
for i, data_batch in enumerate(data_loader):
File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 336, in next
return self._process_next_batch(batch)
File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 357, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
KeyError: 'Traceback (most recent call last):\n  File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 106, in _worker_loop\n    samples = collate_fn([dataset[i] for i in batch_indices])\n  File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 106, in \n    samples = collate_fn([dataset[i] for i in batch_indices])\n  File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/datasets/custom.py", line 139, in getitem\n    data = self.prepare_train_img(idx)\n  File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/datasets/custom.py", line 193, in prepare_train_img\n    gt_masks = self.mask_transform(ann['masks'], pad_shape,\nKeyError: 'masks'\n'
Traceback (most recent call last):
File "./tools/train.py", line 81, in
main()
File "./tools/train.py", line 77, in main
logger=logger)
File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/apis/train.py", line 57, in train_detector
_dist_train(model, dataset, cfg, validate=validate)
File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/apis/train.py", line 92, in _dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/usr/local/lib/python3.6/dist-packages/mmcv-0.2.0-py3.6.egg/mmcv/runner/runner.py", line 349, in run
epoch_runner(data_loaders[i], **kwargs)
File "/usr/local/lib/python3.6/dist-packages/mmcv-0.2.0-py3.6.egg/mmcv/runner/runner.py", line 251, in train
for i, data_batch in enumerate(data_loader):
File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 336, in next
return self._process_next_batch(batch)
File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 357, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
KeyError: 'Traceback (most recent call last):\n  File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 106, in _worker_loop\n    samples = collate_fn([dataset[i] for i in batch_indices])\n  File "/home/power/.local/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 106, in \n    samples = collate_fn([dataset[i] for i in batch_indices])\n  File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/datasets/custom.py", line 139, in getitem\n    data = self.prepare_train_img(idx)\n  File "/usr/local/lib/python3.6/dist-packages/mmdet-0.5.4+65a2e5e-py3.6.egg/mmdet/datasets/custom.py", line 193, in prepare_train_img\n    gt_masks = self.mask_transform(ann['masks'], pad_shape,\nKeyError: 'masks'\n'

The converted VOC dataset does not support masks currently. If you'd like to train mask rcnn on VOC, you may use the officially json annotations provided by coco website. In that case, you will use CocoDataset.


Improve local run of system tests

Testing

Trade-offs

Dependencies

Screenshots

How do these changes improve running system tests locally?

It's still work in progress, but currently I can not run the system tests that easily against my local npm start environment. I would like to enable this

i looks like system tests are now working with the instructions given in CONTRIBUTING.md. please reopen if you run into trouble again!


problem of "ValueError: Expected input batch_size (2) to match target batch_size (1)."

train_cylinder_asym.py
Namespace(config_path='config/semantickitti.yaml')
[480 360  32]
  0%|          | 0/1136 [00:00<?, ?it/s]epoch 0 iter     0, loss: 4.095

 88%|████████▊ | 1000/1136 [22:35<02:54,  1.29s/it]epoch 0 iter  1000, loss: 1.071

100%|█████████▉| 1135/1136 [25:31<00:01,  1.11s/it]Traceback (most recent call last):
  File "train_cylinder_asym.py", line 167, in <module>
    main(args)
  File "train_cylinder_asym.py", line 134, in main
    outputs, point_label_tensor)
  File "/home/zdj/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/zdj/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/loss.py", line 1165, in forward
    label_smoothing=self.label_smoothing)
  File "/home/zdj/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/functional.py", line 2998, in cross_entropy
    return torch._C._nn.cross_entropy_loss(input, target, weight, _Reduction.get_enum(reduction), ignore_index, label_smoothing)
ValueError: Expected input batch_size (4) to match target batch_size (1).
100%|█████████▉| 1135/1136 [25:32<00:01,  1.35s/it]

when i set the batch_size=2 of train_data_loader in semantickitti.yaml , it got the above error

当我用了这个问题的方法后 https://github.com/xinge008/Cylinder3D/pull/118#issue-1162504609
新的错误又出现了，如下：
After I use this problem method, https://github.com/xinge008/cylinder3d/pull/118 #Issue-1162504609
New mistakes appear again, as follows：
train_cylinder_asym.py
Namespace(config_path='config/semantickitti.yaml')
[480 360  32]
  0%|          | 0/2271 [00:00<?, ?it/s]epoch 0 iter     0, loss: 4.133

 44%|████▍     | 1000/2271 [11:58<15:10,  1.40it/s]epoch 0 iter  1000, loss: 1.198

 88%|████████▊ | 2000/2271 [23:35<03:06,  1.45it/s]epoch 0 iter  2000, loss: 1.028

100%|██████████| 2271/2271 [26:42<00:00,  1.42it/s]
 32%|███▏      | 729/2271 [08:53<20:49,  1.23it/s]epoch 1 iter   729, loss: 0.737

 76%|███████▌  | 1729/2271 [20:30<06:46,  1.33it/s]epoch 1 iter  1729, loss: 0.704

100%|██████████| 2271/2271 [26:44<00:00,  1.42it/s]
  3%|▎         | 57/2271 [00:50<28:05,  1.31it/s]epoch 2 iter    56, loss: 0.668

Traceback (most recent call last):
  File "train_cylinder_asym.py", line 169, in <module>
    main(args)
  File "train_cylinder_asym.py", line 110, in main
    iou = per_class_iu(sum(hist_list))
  File "/home/zdj/Cylinder3D/utils/metric_util.py", line 16, in per_class_iu
    return np.diag(hist) / (hist.sum(1) + hist.sum(0) - np.diag(hist))
  File "<__array_function__ internals>", line 6, in diag
  File "/home/zdj/anaconda3/envs/pytorch/lib/python3.7/site-packages/numpy/lib/twodim_base.py", line 303, in diag
    raise ValueError("Input must be 1- or 2-d.")
ValueError: Input must be 1- or 2-d.
  3%|▎         | 57/2271 [00:51<33:31,  1.10it/s]

Does anyone know what the problem is?
请问有人知道这是什么问题吗？

@zhaodejian
Hello, i've encountered the same error as well. How did you get around this?
Best,
Rohith.

Same problem, do you have any progress?


{AKS} Update error handling when creating cluster

This checklist is used to make sure that common guidelines for a pull request are followed.

Update error handling when creating cluster.

Catch both CloudError and HttpResponseError, which would be raised since we have adopted track 2 SDK.

Add unit test.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

AKS


Vagrant was deleted from macos-13 image

Description
Vagrant was deleted from macos-13 image. Is it possible to add to it?
It is present in macos-12 image.
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] macOS 13
[ ] macOS 13 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Image macos-13
Is it regression?
yes
Expected behavior
To have vagrant in macos13 image
Actual behavior
There is no vagrant installed
Repro steps
Run a CI with macos-13 and run "vagrant --version"

The PR that deleted vagrant: https://github.com/actions/runner-images/pull/8375

Hello! Vagrant does not run properly on our modern macOS virtualisation so we are unable to restore it and do not plan doing it in the foreseeable future, unfortunately.


[RFC]: add support for multi-line editing to the REPL

Description
This RFC proposes adding support for multi-line editing to the REPL. Currently, we a user does
In [29]: function foo() {
    // ...
}<|>

where <|> represents the cursor, hitting the up arrow auto-completes previous commands, rather than allows a user to move the cursor up to edit existing content.
It would be nice to allow users to support multi-line editing, as otherwise they need to hit ENTER and start over.
Related Issues
No.
Questions

What are the pitfalls and considerations for implementing this? For one, could be tricky to implement due to needing to listen for certain keypress sequences and having to manipulate cursor position via ANSI escape sequences.
Should the default be to move the cursor up? Or is there a use case for hitting the up arrow in multi-line mode and auto-completing previous commands?

Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Another option for this, which may be better, is to enter an "editor" mode. This would be similar to the built-in Node.js REPL which supports the .editor command.

@kgryte I tried the .editor mode in node, but I can't seem to move up to the above line using backspaces or the up arrow.
But by editor mode are you proposing a vi editor like experience?

@Snehil-Shah Oh! Interesting. You're right about .editor. You can do multi-line input without triggering execution, but you cannot return to previous lines.
Yes, I suppose I had in mind something more like nano. vi may be a bit much to replicate. 😅

@kgryte I think having both would be great, a nano type editing mode can allow for writing long scripts and programs with maybe even options to export it to a .js file or execute it independently to the rest of the REPL context (like a playground of sorts), while our current multiline (with ability to edit previous lines) can allow for writing normal multiline commands.

Agreed.

Following up on this issue, with the addition of auto-pairing closing symbols, the current approach of multi-line editing (relying on incomplete expressions) no longer works very well and makes for a frustrating UX. Recently, I ended up turning off autoClosePairs, entering function contents to allow "incomplete expression" multi-line editing to work, and re-enabling.

@kgryte We need a keybinding to make multiline editing easier. Yes ALT+ENTER didn't work the last time we tried, but I assume different terminals might be interpreting it differently because of ENTER. We can try a combination like CTRL+O (like IPython) to enter multiline editing and it should work.

@kgryte I spent some time studying python REPL's new multiline mode. Apparently, it was a big overhaul and I also think they stopped using their standard readline module, and re-wrote it for the REPL (not entirely sure).

Their way of entering multi-line mode is similar to ours where they try to compile the statements, if found incomplete, they add a newline. Manually entering a newline involves adding a backslash at the end of the statement and hitting enter from any cursor position. So,

... prin<|>t(\

press ENTER
... prin
... <|>t(\

In my opinion, it's not at all intuitive and also requires us additional parsing.

Like us, they handle all keypress as events. The up and down key events trigger certain commands as defined here. We can see for the up event, if at the beginning of the line, they cycle through previous commands, and if not, they call the setpos_from_xy method of their reader class with the new co-ordinates in the console window. setpos_from_xy takes these console window coordinates to update their pos integer which is the line position in the buffer which I presume is a string containing the current command, analogous to our _cmd variable.

For comparison, our approach is similar. We handle keypress events up and down. Once we encounter the up event, we update the current line in the _cmd array, update the line index denoting the line number in the input we are on, and update the readline interface properties like line and cursor position to be able to edit that line. We also have to render the lines below the current line with every keypress because apparently, a lot of the common methods of the node's readline interface including things like deleting a character or moving the cursor etc, uses this method called kRefreshLine which uses clearScreenDown to clear everything below the current prompt.

@Snehil-Shah Thanks for taking a look! Your overview seems reasonable to me. And yes, they ended up reimplementing Python's readline. I actually work with the lead dev who spearheaded this effort. Seems like we don't have much, if anything, to take from their efforts.


Add support for PHP 7.4, but drop support for below PHP 7.1

Seems like in order to support PHP 7.4 (#22), there's a bit of an avalanche effect that in order to be able to run the tests in PHP 7.4, Mockery needs to be upgraded, and in turn PHPUnit needs to be updated to such a version that requires you to use : void return typehints in setUp(), thus requiring a PHP version of at least 7.1.
In this PR I've done these upgrades and also changed the README to suggest people to require a not-yet-existing ^4.0 version of this library, assuming this is a big breaking change that would require a major version bump.
This PR is meant to replace #23 as a more comprehensive one that doesn't leave the repository in an untestable state for the newest PHP version it supports.

By the way, I don't know why the travis-ci tick is not appearing here, but the tests pass in CI:
https://travis-ci.org/github/mlanin/laravel-api-debugger/builds/686057337

Sorry for the late response. Awesome pr! But can you please fix conflicts?

@mlanin Done.


Trait and spectral library database construction - SHIFT starting

03/01/2024: We presented a review from ORNL-DAAC datasets and proposed a general structure for the spectral and traits database.

03/08/2024: Evan, Dhruva and Yoseline propose a pilot SQL database design (see attached slides), along with some points to consider and things to explore in the future. Comments on these slides are appreciated.

07/24/2024:
SBG_PLANTS Database ERD Diagram: This file contains the diagram with some tips for those unfamiliar with database jargon. Please leave your comments on the “Comments page.”
SBG_PLANTS Data Dictionary: In this file, you can find the data dictionary describing each table in the database structure. Please also leave your comments in the “Comments sheet.”


Prevent empty string search

There should be no results for an empty search

I like this and am closing in favor of #26


Build error on .git file in collection folder

expected behaviour
If .git is present in a collection folder, the build process ignores it.
actual behaviour
If .git is present in a collection folder, the build process does not finish.
detail and notes
I've abstracted my articles to a separate git repo, and pulled them into the jigsaw project as a git submodule. I just noticed that the local build seems to error and I am assuming this is because it doesn't like the .git in the source/_articles directory. See the printout from npm run watch:
 DONE  Compiled successfully in 14269ms                                                                                21:48:00

       Asset      Size    Chunks             Chunk Names
 /js/main.js  1.31 MiB  /js/main  [emitted]  /js/main
css/main.css  1.45 MiB  /js/main  [emitted]  /js/main

In CollectionDataLoader.php line 104:

  No matching collection item handler for file: .git.

build [--pretty PRETTY] [-c|--cache [CACHE]] [--] [<env>]

[Browsersync] Access URLs:
 -------------------------------------
       Local: http://localhost:3000
    External: http://<IP_ADDRESS>:3000
 -------------------------------------
          UI: http://localhost:3001
 UI External: http://localhost:3001
 -------------------------------------
[Browsersync] Serving files from: build_local/

Assuming this isn't something I've broken, let me know if there's anything I can do to help troubleshoot.
Warm regards,
Dorian

Interesting, I don't know if we've ever come across this use case before. It should be a straightforward fix, we're already filtering out files that start with _ so just checking for . too will probably do it. I'll take a closer look this week.

Oh fabulous. I looked myself to see if I could fix it, but I'm not familiar with PHP, so I couldn't make much out of the CollectionDataLoader.php file. Apologies.

Actually, can you try filtering out .git directly? Something like this:
'filter' => function ($item) {
    return $item->filename != '.git';
}

Same error.
Although, just noticing how I'm dealing with the filename parameter for path:
'path' => '{collection}/{filename}',
Do I need to wrap it?

I'm trying to reproduce this locally and when there's a .git directory everything works fine, but if I make a file called .git I get your error. Can you make sure the specific path causing that error is in fact a valid Git directory?
If it is can you post your whole config for that collection?
Thanks.

It's not a directory. I believe newer versions of git collate submodule git directories at the root under /.git/modules. The .git file in the submodule directory just contains a reference to that location e.g.:
gitdir: ../../.git/modules/source/_articles
Here's my config:
<?php

return [
    'baseUrl' => '',
    'production' => false,
    'siteName' => 'The Armchair Collective',
    'siteDescription' => 'Turning scholarship into wisdom we can use',
    'siteAuthor' => 'Dorian Minors',

    // collections
    'collections' => [
        'articles' => [ // this is a submodule
            'author' => 'Dorian Minors', // Default author, if not provided in an article
            'sort' => '-date',
            'path' => '{collection}/{filename}',
            'description' => 'excerpt',
        ],
		'missives' => [ // this is also a submodule
            'sort' => '-date',
            'path' => '{collection}/{filename}',
		],
        //'authors' => [
        //    'path' => '{collection}/{filename}', // author article collation on author template page
        //],
        'series' => [
            'path' => '{collection}/{filename}',
            'articles' => function ($page, $allArticles) {
                return $allArticles->filter(function ($article) use ($page) {
                    return $article->series ? in_array($page->getFilename(), $article->series, true) : false;
                });
            },
        ],
        'themes' => [
            'path' => '{collection}/{filename}',
            'articles' => function ($page, $allArticles) {
                return $allArticles->filter(function ($article) use ($page) {
                    return $article->themes ? in_array($page->getFilename(), $article->themes, true) : false;
                });
            },
        ],
        'projects' => [
            'path' => '{collection}/{filename}',
            'articles' => function ($page, $allArticles) {
                return $allArticles->filter(function ($article) use ($page) {
                    return $article->projects ? in_array($page->getFilename(), $article->projects, true) : false;
                });
            },
        ],
    ],

    // helpers
    'getDate' => function ($page) {
        return Datetime::createFromFormat('U', $page->date);
    },
    'getExcerpt' => function ($page, $length = 255) {
        if ($page->excerpt) {
            return $page->excerpt;
        }

        $content = preg_split('//m', $page->getContent(), 2);
        $cleaned = trim(
            strip_tags(
                preg_replace(['/
[\w\W]*?<\/pre>/', '/<h\d>[\w\W]*?<\/h\d>/'], '', $content[0]),
                ''
            )
        );

        if (count($content) > 1) {
            return $content[0];
        }

        $truncated = substr($cleaned, 0, $length);

        if (substr_count($truncated, '') > substr_count($truncated, '')) {
            $truncated .= '';
        }

        return strlen($cleaned) > $length
            ? preg_replace('/\s+?(\S+)?$/', '', $truncated) . '...'
            : $cleaned;
    },
    'isActive' => function ($page, $path) {
        return ends_with(trimPath($page->getPath()), trimPath($path));
    },
];

Thanks a lot! I didn't realize submodules worked that way and I assumed it was supposed to be a directory. The .git file is definitely the issue.
I'll look at adding a check to ignore files starting with . (or maybe just named .git), I just want to make sure there's no way it'll break existing sites. @damiani thoughts?


hardsuit needs charging/suit charger

not having it charged should slow you/disable features or something, suit rechargers should be accessible and maybe we do the battery thing again and see how it goes.
rn its no different from softsuits/voidsuits besides toggleable helmet.
could also port modsuits as a sidegrade/engineering thing not a REPLACEMENT though.

i feel like the addition of another slowly depleting thing to juggle would make it painful for jobs that spend a lot of time in hardsuits like engies, especially if they have to respond immediately to something and they're far away from a recharger.
in other words, the thirst mechanic is already punishing enough for engies :(

Hardsuit charge mechanic would only make sense with abilities. Currently they only have spacing protection, damage resistances and weak lights.
And if low charge slows you as proposed, then regular EVA suits will be straight-up better; -25% speed for EVA room suits without needing charge, -15% to -25% speed on most hardsuits before accounting for low charge speed penalties.
No idea how this interact with bloodred and other antag hardsuits, maybe they won't kill station power so they can charge as well

I do not agree with this. What is the scenerio where hardsuits would actually need recharging? They're not even that much of an issue in SS13 where they're even less restricted.

I don't follow. You'd want to keep the hardsuit charged to avoid the penalties that come when its batteries run out... or are you asking how frequently / where the hardsuits would be charged?

Unless there's some sort of non-antag hardsuit that gives an exceptional advantage, what does even giving charge for them do in the first place? All it does is make Engineer/Atmos' job that much harder.
Crew Hardsuits are effectively spacesuits that have a flashlight on the helmet. Maybe sec/captain/HOS might have armor on them and the RD one might be bomb resistant, but overall they're not that insanely broken that they need slowdown + held slowdown + charge.


[Bug Report] vttablet ddl keyspace error

Overview of the Issue
Currently, the DDL statement cannot select the correct database even after invoking 'use database'; it continues to default to the mysql database.I've noticed that the QueryExecutor doesn't record the dbName passed from vtgate.Should we consider fixing this in vtgate or vttablet?
Reproduction Steps
(base) ➜  ~ mysql -u root

Welcome to the MySQL monitor.  Commands end with ; or \g.
Your MySQL connection id is 1
Server version: 8.0.30-Vitess Version: 16.0.0 (Git revision  branch '') built on  by @ using go1.20.4 darwin/arm64

Copyright (c) 2000, 2023, Oracle and/or its affiliates.

Oracle is a registered trademark of Oracle Corporation and/or its
affiliates. Other names may be trademarks of their respective
owners.

Type 'help;' or '\h' for help. Type '\c' to clear the current input statement.
mysql> show databases;
+--------------------+
| Database           |
+--------------------+
| information_schema |
| mysql              |
| performance_schema |
| sys                |
| test               |
+--------------------+

mysql> show tables;
+----------------+
| Tables_in_test |
+----------------+
| t1             |
+----------------+
1 row in set (3.11 sec)

mysql> use test;
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

Database changed
mysql> alter table t1 add column v3 int;
ERROR 1146 (42S02): target: test.0.primary: vttablet: rpc error: code = NotFound desc = Table 'mysql.t1' doesn't exist (errno 1146) (sqlstate 42S02) (CallerID: userData1): Sql: "alter table t1 add column v3 int", BindVars: {}

Binary Version
No response
Operating System and Environment details
No response
Log Fragments
No response

I think keyspace, shard, dbName should be removed in onlineddl.Executor


Pulumi AI shareable links

Description
This PR adds support for sharing conversations via a share button next to the copy button. This allows users to share generated results with others, as well as have others build other their original conversation.

Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.

and I don't really have a way of checking the functionality

@scottslowe Ah, but you do! If you follow the latest preview URL and browse to Pulumi AI (either at /ai or by clicking the link in the upper right corner), you can see the changes Zack's added in this PR and see how they feel. Copying a share link and pasting, say, into an in-cognito window should simulate what the experience will be like for users.

Just tried this out - a couple notes:

The options for sharing were available even before the output finished producing, but didn't include a link to the in-progress response.  This wasn't intuitive - it feels like we should either (a) not allow sharing until the response is finished or (b) allow sharing the link to the result even before the full result is done rendering.
We seem to be using cmd+enter as the newline, and shift-enter submits the output.  This is the opposite of what ChatGPT does - and what I independently expected.
Not technically a change here - but it might be nice to not start a prompt if it's an empty string (or only whitespace), so that hitting  or the wrong + doesn't submit anything, and just leaves the input unmodified.  Again similar to ChatGPT.
Quite likely not for this PR, but it does feel like it would be amazing to do a short summarization of the session, and use that in the Twitter post template - so that each tweet is auto-personalized


[FIX] Removed HISTORY from Package description

Change Summary
Added Release history at readme and removed at the project description
Related issue number
#8113
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

This is already fixed in #8097.

@samuelcolvin you guy are too fast! Thank you!


⚠ Bofrost.at has degraded performance

In bf25e0a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1406 ms

Resolved: Bofrost.at performance has improved in bd90eba.


config.txt

Hi,
I am reading lesson 1.
Didn’t quite understand how the config.txt comes into play. When is it used by the Pi and where does it read it from.
Can anyone please explain?
Thanks.

These specially-named files are recognized by the Raspberry Pi’s GPU on boot-up and used to configure and boostrap the system. bootcode.bin is the GPU’s first-stage bootloader. Its primary job is to load start.elf, the GPU’s second-stage bootloader. start.elf initializes the ARM CPU, configuring it as indicated in config.txt, loads kernel8.img into memory, and instructs the CPU to start executing the newly loaded code from kernel8.img.

Thanks a lot !
I don’t have an actual board (I’m running an emulator) so it might be the reason i don’t fully understand. I just drop those files on my sd card and the GPU will do the rest?


Added new text variable to correctly display the summary price

Left the old price variable in tact as I didn't want to break anything else.

LGTM

Noice.


Deserializing &str should not be supported

It seems to me like the support for deserializing &str should probably be removed (always failing with a panic). Otherwise, whether it works or not is dependent on whether the value to be deserialized contains any special characters, which seems like a footgun. I'm not sure if this is a problem with the changes in #54, but it might be related. If someone wants to avoid this issue, they can put a Cow<'a, str> in their types instead.

Nah, I like this, even if some may consideer it's a hack.

I'd like to add a vote for this. I saw that you already said you like it, but can you explain why?
I see no benefit to allowing this. If you know no decoding needs to be performed, just don't use this crate in the first place.
This is especially problematic because serde_urlencoded will often be called by a dependency; you can run into bugs here without even knowing this crate exists.


SonarQube: Control Character in UNICODECHARNOCRLF in is.js

SonarQube remarks that it is most likely that the Control Character \u0009 is a bug.
0x9 is Tab-Control Character.
is.uchar is used in password-grant-type.js for validating the content of username and password.
The question here is: Is it possible to have Tab as a character of a username and/or password? I say: No
I think this is a valid finding.
I recommend to remove the Tab-Control-Character.
Lets Discuss.

I would keep, since we should not define password requirements and neither does the standard but I will check again what the standard says to be sure on that.

@jankapunkt
you are right.
https://datatracker.ietf.org/doc/html/rfc6749#appendix-A
Appendix A.  Augmented Backus-Naur Form (ABNF) Syntax

   This section provides Augmented Backus-Naur Form (ABNF) syntax
   descriptions for the elements defined in this specification using the
   notation of [RFC5234].  The ABNF below is defined in terms of Unicode
   code points [W3C.REC-xml-20081126]; these characters are typically
   encoded in UTF-8.  Elements are presented in the order first defined.

   Some of the definitions that follow use the "URI-reference"
   definition from [RFC3986].

   Some of the definitions that follow use these common definitions:

     VSCHAR     = %x20-7E
     NQCHAR     = %x21 / %x23-5B / %x5D-7E
     NQSCHAR    = %x20-21 / %x23-5B / %x5D-7E
     UNICODECHARNOCRLF = %x09 /%x20-7E / %x80-D7FF /
                         %xE000-FFFD / %x10000-10FFFF

   (The UNICODECHARNOCRLF definition is based upon the Char definition
   in Section 2.2 of [W3C.REC-xml-20081126], but omitting the Carriage
   Return and Linefeed characters.)


Build permission for Free license users

Hi
I give the build permission for free license users, but they can't access the dataset.
If I set "free license" users as "Members" they can access the dataset.
If I give them build permissions, the can't even see the dataset.
 Do I need a Pro license to access to the dataset?
My workspaces is premium.

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

ID: 16dbd9ca-dcd3-a4f3-43a8-01ead54fc9f9
Version Independent ID: 66217fc3-a852-201c-a563-54ecbd9e5726
Content: Build permission for shared datasets (preview) - Power BI
Content Source: powerbi-docs/service-datasets-build-permissions.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @CristianAven -- thanks for asking! A question for you. When you say:
"If I set "free license" users as "Members" they can access the dataset."
can you describe what access they have to the dataset? Can they create a report based on the dataset?
I ask because you need a Pro license to create content. Check out the Roles table in this article for details. https://docs.microsoft.com/en-us/power-bi/service-new-workspaces#roles-in-the-new-workspaces
That may answer your questions.

Hi Maggie
Yes, according to the test I've done, a Free license user, that is a Member
of a Premium Workspace, can create a report, but only in PBI Desktop. In
PBI Service, they can't create a report.
Thanks for the document. It look interesting.
Regards!
On Tue, Mar 3, 2020 at 12:12 AM Maggie Sparkman<EMAIL_ADDRESS>wrote:

Hi, @CristianAven https://github.com/CristianAven -- thanks for asking!
A question for you. When you say:
"If I set "free license" users as "Members" they can access the dataset."
can you describe what access they have to the dataset? Can they create a
report based on the dataset?
I ask because you need a Pro license to create content. Check out the
Roles table in this article for details.
https://docs.microsoft.com/en-us/power-bi/service-new-workspaces#roles-in-the-new-workspaces
That may answer your questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/powerbi-docs/issues/1883?email_source=notifications&email_token=AKRHGJWD6RZJNI4RNWY7RR3RFRYS7A5CNFSM4K43QDD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENR5IGA#issuecomment-593744920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKRHGJXH6KGD2D5CI5EVI7DRFRYS7ANCNFSM4K43QDDQ
.

--
Atte
Cristian Avendaño E.
+56 9 9144 2649
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Hi, @CristianAven -- just so I'm clear. Are you saying:

A free user is a Member of a Premium workspace, but they don't have Build permissions for the dataset. They CAN see the dataset in Power BI Desktop, and create a report with it.
but
A free user is a Member of a Premium workspace, and has Build permissions for the dataset. They CAN'T see the dataset in Power BI Desktop.
thanks!

Hi, why O365 groups arent enabled to grant access to shared datasets? While creating an App (using reports and datesets copied from different workspaces) I need to provide access indivually to all my users on each specifc dataset. You're able to use an O365 group to grant access to the workspace then It should follow the same logic but it doesnt. Any idea?

Hi @CristianAven and @ferdcoz ,
I see that this Git issue is a bit old, so I'm going to close it.  I hope that you were able to get your questions answered. If not, please revisit the article since it has been updated several times since this Git Issue was submitted.  Another option is to submit your question to the Power BI community (community.powerbi.com).
All the best,
the DOCS team


rename pytest.imi to pytest.ini

pytest configuration was named imi instead of ini
closes #43
Signed-off-by: Jenia Sakirko<EMAIL_ADDRESS>

LGTM


fix(ingest) support newer version of debezium #6927

Summary
newer version of debezium kafka connect config is changed
database.server.name -> topic.prefix

Separating the config used by checking the debezium connector version
document link
https://debezium.io/documentation/reference/1.9/connectors/mysql.html#mysql-property-database-server-name
https://debezium.io/documentation/reference/2.1/connectors/mysql.html#mysql-property-topic-prefix
Issue
https://github.com/datahub-project/datahub/issues/6927
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Can we do this using something like this instead, where we check topic.prefix and fallback to database.server.name if not available?
That way we don't even need to check the plugin version number
# comment on why this was needed - similar to the PR description
server_name = connector_manifest.config.get(
    "topic.prefix", connector_manifest.config.get("database.server.name")
)

@hsheth2
like you said
rather than calling the api to check the connector version,
it is better to use it after checking if the config exists.
@mayurinehate
how about this
    def get_server_name(self, connector_manifest: ConnectorManifest) -> str:
        if 'topic.prefix' in connector_manifest.config:
            return connector_manifest.config.get('topic.prefix')
        else:
            return connector_manifest.config.get('database.server.name')

@hsheth2
2 test failed
cypress - frontend
trivy - spring boot jar security
Is this pr and test failure related?

I don't think the failures are related to changes in this PR, so we should be good to merge it.


Modify README file

The README file must be modified to include a better description of the project. To give a broad idea of the project it allows communities to create DAOs (Decentralized Autonomous Organizations) completely customizable, the customization ranges from governance to financial funding. The platform uses WorldID to proof the humanness of a person, so that everyone inside the DAO is a verified human. The DAO creation progress is made to be extremely intuitive and as easy as possible.
Also, please include the Figma file in the README: https://www.figma.com/design/g9EbRTom8PLsb61pfL0FS1/SocialSphere?node-id=2423-308&t=q2jrldCI23maeoVI-1
You can expand more on this description, and ask any questions if needed.

Can I be assigned to this? I have experience in writing clear and concise documentation. I can enhance the README to include a detailed project description, emphasizing its features like DAO customization and WorldID integration, and link the Figma file as requested.

Can I tackle this one?

May I handle this issue?

hey sir i would love to work on this issue
i'm a blockchain dev and i would love to handle this issue
assign me and i'll get to work

Hello Jeremiah I'm a frontend developer in web3 I have contributed to previous OD hacks. i have made over 85+ contribution in past OD Hacks Here's my OD Hack profile: https://app.onlydust.com/Jemiiah I'm an experienced technical writer as well; i'm well skilled and i can provide solution to this issue with 24hours of assigment

About Me
I am a frontend developer with over 3 years of experience in creating and enhancing web applications. I excel in writing clear documentation and ensuring that project overviews are both informative and engaging.
My Approach to Solving This Issue
I will update the README file with a detailed and engaging description of the project, focusing on its core features and the unique value it provides. I will highlight the customizable nature of the DAOs, the integration of WorldID for human verification, and the intuitive design that simplifies the DAO creation process. The description will be structured to give prospective users and developers a clear understanding of the platform's capabilities and purpose. Additionally, I will include a link to the Figma file for easy reference to the design framework.
Updated README Section Example

SocialSphere
SocialSphere is a powerful platform designed to enable communities to create fully customizable DAOs (Decentralized Autonomous Organizations). This platform provides flexibility in governance, financial funding structures, and other core functionalities, making it ideal for building decentralized, self-sustaining organizations.
To ensure transparency and inclusivity, SocialSphere integrates WorldID for verifying the humanness of members. This guarantees that every participant in the DAO is a verified human, fostering a more authentic and equitable governance process.
The DAO creation process on SocialSphere is designed to be intuitive and user-friendly, enabling communities to build their organizations quickly and efficiently. Whether you're a small community or a large-scale project, SocialSphere adapts to your needs seamlessly.
And then provide the link to the figma which is above
ETA
My estimated time to complete this task is 1–2 hours.

I would like to work on this issue

Could I take a shot at this?

I’d like to resolve this. I will do required research, then edit the README accordingly.
ETA - 2days

Can I attempt this issue?

Could I take on this issue?

Can I work on this, please?

May I take this issue on?

May I pick this up?

hey @Agbeleshe, I'm following up on @evgongora's question and the issue itself. Do you have any updates for this? The ODHack already finished so there should be an open PR.

Closed by PR #32


include custom policy in token_url if provided

I ran into a problem trying to use omniauth-azure-activedirectory-v2 with AAD setup using a custom policy.
In the Microsoft documentation for requesting a token, they want the name of the custom policy in the URL rather than in the body of the request:
POST <tenant-name>.b2clogin.com/<tenant-name>.onmicrosoft.com/<policy-name>/oauth2/v2.0/token

When the underlying oath2 gem creates the request for getting a token via POST, it places all params (which would include anything you've provided in the normal configuration to name your custom policy) in the `body`` of the request. Unfortunately, Microsoft ignores custom policies in the body and only looks for them in the URL.
With this change, if you set a custom_policy in your configuration, it will be included in the URL between the tenant_id and the remaining parts of the path (/oauth2/v2.0/token).

@pond @bagp1 @sierra-alpha - Just wanted to check to see if this change was something you thought made sense to incorporate back into master?

Sorry for the radio silence here, we're overloaded with other work. I've not forgotten this PR (or some other pending issues) and will get to it when I can - apologies - please bear with us! Thanks.

Belatedly, have reviewed it - looks good but there is a Markdown syntax error in your docs that was also in the PR description. I edited the PR description and I'll probably just fix up the README.md afterwards (since I can't edit a file on your fork, of course) for a bit of style housekeeping so that the white space etc. matches the surrounding coding style & just some basics like "tests" instead of "specs", which aren't worth churning a new PR over (I very much appreciate that you added docs at all, awesome).
Will post the diff SHA here after FYI.

@stevenchanin Touch-up to README.md is https://github.com/RIPAGlobal/omniauth-azure-activedirectory-v2/commit/0f83f42ae9b656b8167a6c55917ea45ba963333a - hope that's OK.
Very many thanks for this contribution. It's the details around the edges for compatibility that make a big difference overall to the gem's utility.

This is now pushed to RubyGems as v2.1.0.

Thanks very much for reviewing and merging the pull request. I definitely appreciate you fixing the mistake in the README. Sorry about that. Have a great weekend!


add syntax for module reexport

It would be nice if you could export all the names from an import module in one go. For example if you had the module Else:
module Else where

theNumberOne = 1
theNumberTwo = 2

And you imported it in another module called Something, I want a way to export all the names in Else all at once, like this:
module Something
  ( module Else
  , theNumberThree
  ) where

import Else

theNumberThree = 3

Right now, you'd have to write the equivalent module like this:
module Something
  ( theNumberOne
  , theNumberTwo
  , theNumberThree
  ) where

import Else

theNumberThree = 3

That's gonna get unmanageable as the number of exports in Else increases.

Centralizing all issues about reexports into #931


FSavePNG(io.Writer, image.Image) variant

I wanted to ask if adding an FSavePNG function would be something that you'd consider. The reason for this is, I can connect your package to a web server or for usage where I wouldn't like to write to disk and easily connect it to other usages since it will take an io.Writer.
Also this means that we can also reuse FSavePNG in SavePNG in such a fashion.
func SavePNG(path string, img image.Image) error {
    f, err := os.Create(path)
    if err != nil {
      return err
    }
   defer f.Close()
   return FSavePNG(f, img)
}

func FSavePNG(w io.Writer, img image.Image) error {
    return png.Encode(w, img)
}

If this is something you'd consider, please let me know and I can create a PR for it.
Thank you.

You could do this with the current API:
png.Encode(w, dc.Image())

But I still like the idea. What if we call it EncodePNG instead of FSavePNG? Also add it to the Context, so you can do:
dc.EncodePNG(w)

Sounds like a plan. Cool, let me jump on it.


[firebase_crashlytics] Added more specific instruction in readme

Description
There are two "repositories" developers can put maven, this just specifies that it should be under buildscript so there's less confusion
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Closing as no longer required with the Fabric to Firebase migration being completed.


YARN-11463. Node Labels root directory creation doesn't have a retry logic

Description of PR
Node Labels root directory creation doesn't have a retry logic
JIRA - YARN-11463
For code changes:

[X] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

Thanks @ashutoshcipher for the update. Apart from two nits it looks good to me. Can you please check the test failure?

Thanks @brumi1024 for review. I will look at it.

Hi @brumi1024. Sorry for getting a bit late. I have addressed your comments. Please review. Thanks

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 35s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
16m 38s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
26m 46s

trunk passed

+1 :green_heart:
compile
10m 36s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
9m 13s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m 45s

trunk passed

+1 :green_heart:
mvnsite
1m 58s

trunk passed

+1 :green_heart:
javadoc
1m 54s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 52s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
4m 13s

trunk passed

+1 :green_heart:
shadedclient
22m 16s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 30s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 20s

the patch passed

+1 :green_heart:
compile
9m 49s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
9m 49s

the patch passed

+1 :green_heart:
compile
8m 56s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
8m 56s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m 42s

the patch passed

+1 :green_heart:
mvnsite
1m 47s

the patch passed

+1 :green_heart:
javadoc
1m 42s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 35s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
4m 28s

the patch passed

+1 :green_heart:
shadedclient
22m 48s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m  9s

hadoop-yarn-api in the patch passed.

+1 :green_heart:
unit
5m 38s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
asflicense
0m 57s

The patch does not generate ASF License warnings.

162m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5562/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5562

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 6f4971f4a893 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 596a9fc8b95162ce643e46f4c1329537e7e7231d

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5562/3/testReport/

Max. process+thread count
557 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: hadoop-yarn-project/hadoop-yarn

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5562/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Thanks @ashutoshcipher for the patch, merged to trunk.

Thanks @brumi1024 for review and merge.


Number of members mismatch in title

The number of members reported in the title of a given city often mismatches with the actual number of members in the list. For New York, 3 members are in the list but 7 reported in the title.

See also:

https://meet.hn/city/us-Fort-Collins
https://meet.hn/city/us-Los-Angeles

Hey @lgrcia 👋, thanks for reporting!
A cache issue (which I resolved on launch day) was partly responsible for this.
I initially thought it was the sole cause until I read this and realized the Los Angeles hacker count is still incorrect.
I'm in the process of gathering all the feedbacks hackers made since launch day.
Adding this to the list!

Should finally be fixed with the commit: https://github.com/borisghidaglia/meet-hn/commit/c2aa78b960027c26146c86fadc331efe4767bed5
Don't hesitate to re-open if you notice it again somewhere 🙏


merge from arlac77/template-kronos-module,arlac77/template-github

package.json

fix(engines): add >=13.10.1 remove >=13.8.0 (engines.node)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adjust documentation to point to GitHub Discussions

I can point to https://github.com/mido/mido/discussions/257.

Sounds good.


Android Basics: change-app-icon

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-change-app-icon?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-change-app-icon#5
In which task and step of the codelab can this issue be found?
Change the app icon
Describe the problem

"you may notice that the foreground asset is located in the drawable-v24 folder, while the background asset is in the drawable folder. The reason is because the foreground asset contains a gradient, which was available starting in the Android 7.0 release (also known as API version 24, hence the -v24 resource qualifier). The background asset doesn't contain a gradient, so that can go in the base drawable folder."
That's not correct because the foreground asset is located in the drawable folder while the background asset is in the drawable-v24 folder as clearly explained in the following instructions.

Steps to reproduce?

Go to...documentation
See error... in the Move vector drawable files into -v26 directory section

Versions
Android Studio version: n/a
API version of the emulator: n/a
Additional information
Include screenshots if they would be useful in clarifying the problem.

Duplicate of #161, #165, #166


Implement contract deployment workaround to not depend on EOAs

Until we define how account contracts will be deployed, which is tightly coupled with encryption and nullifier key management, we’ll need a workaround to deploy an account contract that does not rely on EOAs (so we can kill them).
Add a new flag to the Tx object txContext.isAccountDeploymentTx. When set, the tx will deploy a contract but using the txContext.contractDeploymentData.contractAddressSalt the deployment address (not as a salt!). Update tests to use this deployment method, so we can deploy an account contract to an address that matches a public key. This allows current nullifier and encryption key derivation to keep working.
Alternatively, we could implement a UniversalDeployer contract, but we still need a method for deploying the deployer.
In case we settle soon on encryption keys, nullifier keys, and contract deployment, we should be able to avoid this workaround.

Current idea is to calculate the deployment address as hash(master_public_key, byte_code, constructor_args, salt).


PackageManagement API 1.5

cherry-pick of c006a97
PackageManagement API v1 (post-API Review) #4081

/azp run


[5.7] Added callable objects to query builder

We use higher order function in query builderr very often, like this:
$builder
    ->when($value, function ($builder) use ($value) {
        $builder->where('field', '>=', $value);
    })
    ->when(! is_null($otherValue), function ($builder) use ($otherValue) {
        $builder->where('field', $otherValue);
    })

    // ... chian

The problem is huge query builder. My idea for this issue is using callable objects for simplifying code, like this:
$builder
    ->when($value, new WhereGreater('field', $value, true))
    ->when(! is_null($otherValue), new WhereEquals('field', $otherValue))

    // ... chian

This pr add a callable objects for basic queries. No difference in performance between closure and callable objects. This objects may use in other conditions, for example:
// With higher order functions
$model->whereHas('relation', function($builder) use ($search) {
    $builder->where('title', 'like', "%{$search}%");
});

// With callable object
$model->whereHas('relation', new WhereLike('status', $search));

This objects can be written in application code, but nice to have his out of box.
Also i added PHPDoc for autocomplete in builder methods chaining

Would like to see something like this too
$user->whereHas('role', new WhereBelongsTo($role));

instead of
$user->whereHas('role', new WhereEquals('id', $role->id));

@salomoni I believe that would be up to the object to implement in anyway it wants.
However, this opens up the method signatures for several methods from Closure to callable. This means they support a lot more things than just closures and __invoke, this means these methods will support strings and arrays that conform to the callable syntax. This is often forgotten when working with the callable typehint. Could we add additional tests for these scenarios to make sure that this new functionality isn't broken in the future by someone forgetting about this "special-case" callables?

No plans on adding.


Add Chile holidays localization: en_US, es, uk

Proposed change

Add Chile holidays localization: en_US, es (default), uk
Some holidays original names correction

Type of change

[ ] New country holidays support (thank you!)
[x] Supported country holidays update (calendar discrepancy fix, localization)
[ ] Existing code/tests/processes improvement (best practice, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (adds functionality to python-holidays in general)

Checklist

[x] I've followed the [contributing guidelines][contributing-guidelines]
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts
[x] The code style looks good (make pre-commit)
[x] I've added tests to verify that the new code works and all tests pass locally (make test)
[x] The related [documentation][docs] has been added/updated (check off the box for free if no updates is required)

Coverage: 100.0%. Remained the same when pulling 862201d364f09513df0695e0c44994d3f752f4ac on KJhellico:upd-chile-loc into fa8b9441bd99cb27be4e3d9ccd698ac7a67d1679 on dr-prodigy:beta.

👍


cargo: release<EMAIL_ADDRESS>

Now that #117 is merged it would be nice to have a new release 0.33 of ibc-proto-rs so that downstream consumers like ibc-types can upgrade to it.

Closed in #126


⚠ ClamAV has degraded performance

In 52fa4c8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 256 ms

Resolved: ClamAV performance has improved in 7201e36.


add ExtensionVersionDocumenter

LGTM, although the generated md file is hard to find in the repo. Maybe this is something we can include in the changelog for each release

Sure.  I also added a link to it from the readme.


Dependency on python being installed

This is so wrong on many levels.
Publishing a package which cannot be installed because another toolchain (python) is not installed is always a very bad idea. Published packages should always be as immutable as possible, so depending on a local build as part of the installation of a package is, has always been, and will always be a bad idea in dependency management.
And because it is python, one first has to figure out whether it's 3+ or 2+. Of course, since I'm definitely not a python dev, and have on interest in having python installed on my machine, I am not up-to-date with the version incompatibility issues in python, and have no interest in wasting my time finding out.
So please publish this package with python dependencies pre-built. It makes no sense to do otherwise.
Note that I'm filing this issue since react-router depends on this package, installing it is taking way too much time because of this python dependency.

Python 2.7 is a dependency and that's not going to change anytime soon, such is life.  Closing.

Can you give the reason why the published npm package is not published with what-ever-you build pre-built, so that those that depend on node-gyp do not get a transitive dependency on python 2.7 ? I'm kind of curious...

node-gyp is a thin wrapper around GYP, which is a python project.  You need node-gyp to build modules that have native components.


Support setting configuration through environment variable

Allow to setup configurations by environment variable.
What would you like to be added/modified:
Currently only docker image can set the configuration through env. It will be better to extend the support to local run.
Why is this needed:
This is inspired by the request of snap for this issue.

Due to deb/rpm problem of writing to config file, we may need to load the env but not writing to file.

@ngjaying hi, I can take this one

As I mentioned in other issue - maybe usage of hocon would be useful however I don't know whether this idea has been dropped or considered

Thanks. About the hocon, the main concern is that the CI may be broken for format change. And the backward compatibility of previous users/tools is another obstacle.

Thanks. About the hocon, the main concern is that the CI may be broken for format change. And the backward compatibility of previous users/tools is another obstacle.

Indeed so I will write something to handle configs from env. By default every config will be read from file however any env variable associated with the given config will overwrite it.

Indeed so I will write something to handle configs from env. By default every config will be read from file however any env variable associated with the given config will overwrite it.

Sounds good.

Another options is too suport two formats, legacy one as it is, and hocon one with defined priority...

I think it is confusing to have two formats but brings little benefits. I do not favor this idea.

I think it is confusing to have two formats but brings little benefits. I do not favor this idea.

Ok, so lets drop hocon topic


Fix title when subtitle is empty

Remove the hyphen when the subtitle is empty.

Thanks for your help😉


fixing remove

Remove function is removing views in wrong order. As result, I was getting error from libraries like jquery-ui, when destroying resizable, draggable, widgets on view remove.

Can you add a test?

hmm, would be tricky, but I will do it later.

Basically, think about. Make it sense to you? For me reordering make perfect sense. When you think about it. First child views are removed, then parent, and view is removed last.

Yeah it makes at first glance at least.

Indentation is skewed in the change btw.


[Feature Request] Self Cleaning - Fan Mode after Cooling / Drying

Hello there,
first i wanted to thank you for your great work!
When i shut off the AC after cooling / drying with the original remote, the Fan keeps blowing 2h and Vanes are nearly shut.
This will dry the indoor unit to prevent bacteria etc.
Is it possible to integrate this into this Project? I think many People could use this.

Good idea! Came across this feature and enabled it via the remote. It only works if you turn off the unit via the remote.

I have read some forum messages of people adding this to Home Assistant (i did this in NodeRed). i am not sure if we want the unit to think for itself when home automation shuts it, but if you have an idea on how to implement it and document it clearly, we can consider it


feat: Add method to trigger ion-refresher manually

Type: feat
Platform: all
Use case:
I have several "index" views that show remotely loaded content, with filtering available. I would like to use a consistent indicator for refreshing, so I would like to be able to do something like
$scope.$broadcast('scroll.refreshStarted');

which, per my thinking, would cause the following:
// js/angular/controller/refresherController.js
$element[0].classList.add('refreshing');
scrollTo(ptrThreshold, scrollTime); //with an animation

If requested, I can attempt to write this myself, and send in a PR.
If there is another way to handle this that I missed already in place, please let me know

Greetings @kageurufu!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

absolutely desirable feature! :+1: +1

@kageurufu, we're going to hold off on adding new features to Ionic until after the 1.0, but I've added this to our new features trello board. Please vote and comment on it to improve it's consideration for post 1.0.


Buffer overrun in reversible mode

zfp_stream_maximum_size() underestimates the compressed size in reversible mode, which can cause buffer overruns when the data is highly incompressible.

Fixed by commit f3e1ca9 on develop.


Added analysis files

Also, use present tense for commits/pull requests. https://github.com/ReachCP317/Reach/blob/master/CONTRIBUTING.md#styleguides

Also put the analysis file into the docs directory so I can add it to the github pages.


Fix db saving in PlayGameDevLevel view

Coverage remained the same at 62.027% when pulling f9f7ffc2db00912d89363c3473051a1dfbfb3bc7 on fix-game-dev-db into 1865cf721a98086ea6873db7b62b1dbaa366c90a on master.


Update lwr and @lwrjs/api to latest

This PR updates both the lwr and @lwrjs/api packages to the newly released version latest.

testing


Support for Code Llama

Can we use code llama

On the roadmap. Can't say for sure when it'll be ready. pRs welcome!


CM-1147: Fixed missing period between "Open" and gate sentence

Jira Ticket:
CM-1147
Description:
Fixed a missing period between "Open" and the gate sentence following the Park Access Status on the park-operating-dates page.

If you don't mind, I'll fix this CM-1147 along with CM-1201 as they are similar bugs.
CM-1201: https://bcparksdigital.atlassian.net/browse/CM-1201

If you don't mind, I'll fix this CM-1147 along with CM-1201 as they are similar bugs. CM-1201: https://bcparksdigital.atlassian.net/browse/CM-1201

Sure, no problem.  You might just need to remove this media query from listPage.scss
@media screen and (max-width: $mdBreakpoint) {


[terra-tabs]Terra-Tabs on Mobile Apps

Feature Request
The dropdown for terra tabs on  clicking the more button doesn't work properly on  mobile apps.
Description

IOS - Swipe doesn't navigate  the user through the dropdown list.
Android - Focus  goes out of the dropdown once the user reaches the end of the dropdown list.

@rohit-v could you provide more information and screenshots of the workflows?

updated.

@rohit-v As discussed with you offline, since we are unable to get access to run your app could you possibly capture a short video or gif demonstrating the issues you are reporting?
If you open this tab example on your mobile device, are you able to reproduce the issues? If so, could you elaborate the steps you take and the issues you're having?

please find below the Giffy recorded on a Samsung note 10 + emulator running Android 10.
The issue seems to bee with selecting items in dropdown.

@rohit-v , @srijith1979  is this issue occurs in mobile browsers also  when you try to open your js component ?  if yes, could you please mail us the js component code git link to us ?

Had an offline conversation with Dave Casper and he was able to reproduce the issue on the 11th of May.


[Bug] Cant Login

Device information

Tachiyomi version:0.9.2
Android phone :Samsung S10
Android version: 10.0

Steps to reproduce

Start Neko

Settings->MangaDex Settings->MangaDex login

Fill in Username and Password

Press "Login"

Expected behavior
That i can login and use the Neko app
Actual behavior
I cant login and it says  "Unknown Error"
Other details
I already tried to switch my Password cause i thought i just forgot it but no it just doesn't work

Are you still having issues?

Yes i tried it again right now and nothing changed

Did you confirm your account and have you tried using 2fa if it's enabled?

2fa is disabled and to be honest i dont know what you mean with confirmed, if you mean mangazone then no because i never got an email from them and if you mean the neko app then yes i did.

Ok so first thigs first
Thank you for youre help up to this point and that you were ready to answer some random dude on the internet
Now to the important part...
I dont know if you can believe me but i accualy thought up to this point that it says mangazone i only realised it right f*cking now that it accualy says mangaDEX. Like i know i wrote mangadex and i dont know accualy why but it seems that my brain thought something like "MangaDex? More like MangaZone" and cause i just searched for a way to extract my data from mangazone it came really handy for my problem. I again am really really sorry that i took youre time for something accualy so dumb and i hope you can forgive me.
I would like to delete this issue "report" i guess cause there is no problem with the app, theres just a problem with my brain.

Haha no problem. Just glad you figured out the issue 😀

How do I sign up??????

@kentlagorski https://mangadex.org

I'm have having the same problem. I can't sign in. I went to check if I put the wrong password and username but everything was the same and it worked in my internet Brower. I did have 2FA enable but I disable it to see if that would fix it but I still wasn't able to sign in.

Neverminded even thought it said  "Unknown Error" it was able to sing me in.


vimPlugins.sqlite-lua: fix postPatch substituting nonexistent file

Motivation for this change
The vim sqlite plugin was renamed in d28c708bc75425eccff806ca975c2d309c00dd18 but the postPatch override was not fixed.
This PR also fixes the logic to use the appropriate dynamic library according to the platform (.dylib for Darwin otherwise .so).
Things done

[x] Tested to be working in my own configuration.

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 via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"

[ ] Tested execution of all binary files (usually in ./result/bin/)

21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Related to #136683
cc: @teto

Please rebase on master and integrate the review suggestions.

Please rebase on master and integrate the review suggestions.

@SuperSandro2000 done :)