Error upgrading to 11.5.0

Describe the bug
Using node 18.3.0 when upgrading from 10.5.1 to 11.5.0 I get this error:

   ╭────────────────────────────────────────╮
   │                                        │
   │   Update available 10.5.1 → 11.5.0     │
   │   Run npm i -g netlify-cli to update   │
   │                                        │
   ╰────────────────────────────────────────╯

gatsby-agilitynerd (3.8.6)> npm i -g netlify-cli                                                                                 master [!]
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
npm notice Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
npm notice Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
npm ERR! Cannot read properties of null (reading 'pickAlgorithm')

npm ERR!      /Users/xxxxx/.npm/_logs/2022-08-27T20_05_14_255Z-debug-0.log

Steps to reproduce
Here is the content of the npm logfile:
2022-08-27T20_05_14_255Z-debug-0.log
Configuration
No response
Environment

npx envinfo --system --binaries --npmPackages netlify-cli --npmGlobalPackages netlify-cli                                    master [!]

System:
OS: macOS 12.5.1
CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
Memory: 1.97 GB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.3.0 - ~/.nvm/versions/node/v18.3.0/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.11.0 - ~/.nvm/versions/node/v18.3.0/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
npmGlobalPackages:
netlify-cli: 10.5.1

This might be an issue with npm. (see https://stackoverflow.com/questions/69567381/getting-cannot-read-property-pickalgorithm-of-null-error-in-react-native) Can you try the suggestions there?


MBTR about mbtr.create()  ---displays only features

HI! I have a question. When I use mbtr.create() , it returns array with shape[positon, features] such as below.
(0, 0)	0.00844207
(0, 1)	0.012111196
(0, 2)	0.01720821
(0, 3)	0.024216358
(0, 4)	0.033753663
(0, 5)	0.046600137
......
I want to display only features such as below .
0.00844207
0.012111196
0.01720821
0.024216358
0.033753663
0.046600137
How should I do?
Thanks!

Thank you so much!!  And Sorry I overlooked that.
I came up to one question after this . (I shouldn't write it in this issue)
Original paper expressed that canonical choices of gk for 'k' =1,2,3,4....
But , dscribe module's 'k' is from 1 to 3.
In dscribe perhaps k=1 : atom counts, k=2 : (inverse) distances, k=3 : angles, I think.
Is  dihedral angles(k=4) not included in dscribe?
Best regards

You are right, dihedral would be the logical continuation. So far, k = 4 is not implemented since for machine learning it generally does not improve the overall accuracy. Unless there is a clear need from the users' side, we leave it at k = 3.

OK.Thanks!


Quick view of FTP files

What feature or improvement do you think would benefit Files?
Quick view of FTP files on devices on the network. For example mobile phone content.
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.2715
Comments
No response

Thanks for the feedback, Files can already view FTP content by typing ftp://<ip>:<port> into the path bar or right clicking the network sidebar item and adding it as a drive/folder.

Hello!
The image viewer does not allow moving more than 10 images within a folder.
Is there a solution for it?

What image viewer? If you just mean Files then try doing the same in Windows File Explorer. If Windows File Explorer works then please say where you are moving to and from.

Hello!
I use Microsoft photos by default.
Until now, if I opened a photo in a folder, I could move between the photos
with the mouse or buttons.
Now you have to select the pictures inside the folder, I can only move
between them and it allows a maximum of 10 pictures.
Thank you for your attention!
Josh65-2201 @.***> ezt írta (időpont: 2024. máj. 25.,
Szo, 20:24):

What image viewer? If you just mean Files then try doing the same in
Windows File Explorer. If Windows File Explorer works then please say where
you are moving to and from.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/14088#issuecomment-2131384489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2KDCBDY5G7KIVCMZ4NGU3ZEDJNPAVCNFSM6AAAAAA73D4WFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZRGM4DINBYHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thats a different issue, We're tracking this in #15409. You can check there or click subscribe in the right sidebar for updates.


Why are we not using the WebView's cache?

I noticed that caching is explicitly set to WebSettings.LOAD_NO_CACHE in WebViewYouTubePlayer.initWebView.
Is there a reasoning behind this?
I set it to WebSettings.LOAD_DEFAULT and did not encounter any problems, while saving a lot of network traffic (thumbnails, JS and CSS for the embedded player).
The doc for reference: https://developer.android.com/reference/android/webkit/WebSettings#LOAD_NO_CACHE
WebSettings.LOAD_NO_CACHE: Don't use the cache, load from the network.
WebSettings.LOAD_DEFAULT: Default cache usage mode. If the navigation type doesn't impose any specific behavior, use cached resources when they are available and not expired, otherwise load resources from the network.

Yeah, we should probably use the cache.

Feel free to send a PR for this, if you have time. Otherwise I'll do it before releasing the next version.

Thanks to @flauschtrud , this will be available starting from version 11.1.0.


perf: Reduce the impact of 'QueryRenderChild.isFilenameUnique()' on rendering speed

Description
In a vault with 6,700 markdown files, I found that the call on QueryRenderChild.isFilenameUnique() for every task that is rendered in search results was taking nearly as long as the actual rendering:

Required Steps
I'm thinking of make Task store TasksFile - and then having all tasks from the same file store the same TasksFile object.
That way, TasksFile can efficiently store whether the filename is unique and cache the results.
This is also a necessary step on the way to efficiently providing access to file properties/YAML/frontmatter to searches.
How to Test
No response

Another way to address this would be to call getMarkdownFiles() at the start of the search and pass it in to each QueryRenderChild.isFilenameUnique() call.
So there would be only one expensive call per render of a query - rather than one per task rendered.

Another way to address this would be to call getMarkdownFiles() at the start of the search and pass it in to each QueryRenderChild.isFilenameUnique() call.
So there would be only one expensive call per render of a query - rather than one per task rendered.

Doing this, the result is not as clear-cut as I would have hoped:

Number of Tasks
Original render time (ms)
New render time (ms)

1
6.299999952316284
13.700000047683716

25
42.799999952316284
36.699999928474426

50
80.60000002384186
79.30000007152557

100
146.29999995231628
157.10000002384186

150
199.30000007152557
188.60000002384186

200
270.89999997615814
243.10000002384186

250
388.89999997615814
307.39999997615814

500
648
549.5

750
1172.2999999523163
1035.6999999284744

1000
1436.1000000238419
1197.3999999761581

2000
3021.399999976158
3805.1999999284744

3000
4647.799999952316
9491.399999976158

4000
8601.600000023842
12144.300000071526

5000
9529.099999904633
8763.399999976158

Closing, at the experiment was not successful enough to be worth continuing.


Kata Containers 1.9.3

Version bump no changes

/test

I see an error on travis:
ERROR: check failed for URL https://www.qemu.org/Features/VirtioVsock after 3 tries
ERROR: Invalid URL 'https://www.qemu.org/Features/VirtioVsock' found in the following files:


Environment in dotenv files seems not correctly applied.

I am setting PUBLIC_URL on .env.production but it seems that .env* files are not read while building (and overriding) the webpack configuration. Indeed, the configuration of the InterploateHtmlPlugin does not "see" the variables I set on these files, while the message at the end of the build (The project was built assuming it is hosted at /MY_PUBLIC_URL) correctly reports it.
If I add require(resolveConfigFilePath("env.js")); before line https://github.com/sharegate/craco/blob/c4e8dd40aa875fcef02467dac444122bbf241956/packages/craco/lib/cra.js#L20
everything works.
Indeed, the original script react-scripts/scripts/build.js loads the environment before building the configuration (see https://github.com/facebook/create-react-app/blob/45bc6280d109b7e353ffaa497a0352c9d040c36d/packages/react-scripts/scripts/build.js#L22).

@patricklafrance how do I access the variables inside my .env inside craco.config.js ?

@patricklafrance @abenhamdine how can I access the variables in my .env from craco.config.js ? Currently am getting the value as undefined.
/* .env */
REACT_APP_FOO="bar"

/* craco.config.js */
console.log('env....', process.env.REACT_APP_FOO);  // getting undefined here

https://github.com/djdmbrwsk/dotenv-cra

https://github.com/djdmbrwsk/dotenv-cra


How do I get evaluate() to work?

First off, thanks for the tutorial!  I'm trying to get evaluate to work at the moment.  I have num_keypoints=4. The model trains smoothly when I just comment out evaluate during training, but I need to evaluate the performance now so I'm trying to get that to work. I did what you instructed in your tutorial:

Update. It’s possible not to edit pycocotools/cocoeval.py file in pycocotools library to change kpt_oks_sigmas, but to edit coco_eval.py file, as Diogo Santiago suggested:

# self.coco_eval[iou_type] = COCOeval(coco_gt, iouType=iou_type)
coco_eval = COCOeval(coco_gt, iouType=iou_type)
coco_eval.params.kpt_oks_sigmas = np.array([.5, .5]) / 10.0
self.coco_eval[iou_type] = coco_eval

Since I have num_keypoints=4, I instead wrote np.array([.5, .5, .5, .5]) / 10.0 for kpt_oks_sigmas.
But I'm still getting the same error: ValueError: operands could not be broadcast together with shapes (4,) (17,)

Hoping you can help me with this! 🤞

😃 Fixed!
Apparently, the solution on the article was right. But for some reason, when I edit on Colab and save, it doesn't actually get saved. So I had to already have changed coco_eval.py before I even cloned it on my Colab runtime.
I basically did the same thing, but this time I downloaded the training scripts from PyTorch and created my own repository to clone:

Download the scripts from pytorch/vision/references/detection and created my own repo storing it
Open coco_eval.py
Edit the lines somewhere under __init__ for iou_type in iou_types:- self.coco_eval[iou_type] = COCOeval(coco_gt, iouType=iou_type)
+ coco_eval = COCOeval(coco_gt, iouType=iou_type)
+ coco_eval.params.kpt_oks_sigmas = np.array([.5, .5, .5, .5]) / 10.0 # 4 keypoints
+ self.coco_eval[iou_type] = coco_eval

At the start of the Training Notebook:!git clone https://github.com/<username>/<repo-name>.git
%cd <repo-name>

hi ,thanks! I have num_keypoints=4,i got the same error:ValueError: operands could not be broadcast together with shapes (55,) (17,) and i use you solution . but i got the other error:

can you help me?

Thanks, man. I've been struggling with this error. Apparently, making direct changes to the file in Colab doesn't update it. weird.

Thank you for clarifying a lot of things. I was trying a similar thing for 6 points, but I am getting an error as followed after the explanation-
Please could someone help me about this? That error resides in roi_heads of the pytorch repository. What changes should I do?
I understood that the error is because,  gt_kp_in_image has size torch.Size([3, 2, 3]) (because I am having 6 keypoints)
but, midx is tensor([0, 0, 0, 0, 0, 0, 1, 3, 3, 3, 1, 1, 3, 1, 2, 2, 1, 1, 3, 3, 4, 0, 1, 2, 3, 4, 5]) which goes out of bounds.
I really am having a tough time figuring out what exactly is "midx" and what does matched_keypoints mean. I'd appreciate if someone helps me. Thank you so much. I have give more information if needed.
"IndexError                                Traceback (most recent call last)
Cell In[16], line 21
18 num_epochs = 20
20 for epoch in range(num_epochs):
---> 21     train_one_epoch(model, optimizer, data_loader_train, device, epoch, print_freq=1000)
22     lr_scheduler.step()
23     evaluate(model, data_loader_test, device)
File ~/Downloads/Encoderless/pytorch_repo/engine.py:32, in train_one_epoch(model, optimizer, data_loader, device, epoch, print_freq, scaler)
30 print(images[0].shape, targets[0]['boxes'].shape)
31 with torch.cuda.amp.autocast(enabled=scaler is not None):
---> 32     loss_dict = model(images, targets)
33     losses = sum(loss for loss in loss_dict.values())
35 # reduce losses over all GPUs for logging purposes
File ~/.local/lib/python3.10/site-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
1496 # If we don't have any hooks, we want to skip the rest of the logic in
1497 # this function, and just call forward.
1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1499         or _global_backward_pre_hooks or _global_backward_hooks
1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
1502 # Do not call functions when jit is used
1503 full_backward_hooks, non_full_backward_hooks = [], []
File ~/.local/lib/python3.10/site-packages/torchvision/models/detection/generalized_rcnn.py:105, in GeneralizedRCNN.forward(self, images, targets)
103     features = OrderedDict([("0", features)])
104 proposals, proposal_losses = self.rpn(images, features, targets)
--> 105 detections, detector_losses = self.roi_heads(features, proposals, images.image_sizes, targets)
106 detections = self.transform.postprocess(detections, images.image_sizes, original_image_sizes)  # type: ignore[operator]
108 losses = {}
File ~/.local/lib/python3.10/site-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
1496 # If we don't have any hooks, we want to skip the rest of the logic in
1497 # this function, and just call forward.
1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1499         or _global_backward_pre_hooks or _global_backward_hooks
1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
1502 # Do not call functions when jit is used
1503 full_backward_hooks, non_full_backward_hooks = [], []
File ~/.local/lib/python3.10/site-packages/torchvision/models/detection/roi_heads.py:877, in RoIHeads.forward(self, features, proposals, image_shapes, targets)
874         raise ValueError("both targets and pos_matched_idxs should not be None when in training mode")
876     gt_keypoints = [t["keypoints"] for t in targets]
--> 877     rcnn_loss_keypoint = keypointrcnn_loss(
878         keypoint_logits, keypoint_proposals, gt_keypoints, pos_matched_idxs
879     )
880     loss_keypoint = {"loss_keypoint": rcnn_loss_keypoint}
881 else:
File ~/.local/lib/python3.10/site-packages/torchvision/models/detection/roi_heads.py:322, in keypointrcnn_loss(keypoint_logits, proposals, gt_keypoints, keypoint_matched_idxs)
313 print("\n")
314 """
315 """
--> 322 kp = gt_kp_in_image[midx]
323 heatmaps_per_image, valid_per_image = keypoints_to_heatmap(kp, proposals_per_image, discretization_size)
324 heatmaps.append(heatmaps_per_image.view(-1))
IndexError: index 3 is out of bounds for dimension 0 with size 3"


chore(requirements): Add ones for docs

This explicitly lists dependencies for documents generation in a top-level requirements file, which seems like a good idea in general and is especially useful if one uses virtualenvs like I do. Also, hope it'll save someone from the rather cryptic errors one can get having python-sphinx installed with OS package manager, but not having Sphinx installed with pip.
Signed-off-by: Dmitriy Volkov<EMAIL_ADDRESS>

It would probably be best to put it under extras_require in setup.py, as that follows how the rest of the dependencies are listed.
It would look like this:
extras_require={
    'doc': ['Sphinx', 'sphinx-rtd-theme', 'sphinxcontrib-napoleon']
}

Well, I have thought about that, and were unsure which option I should go with. Sure, using extras aligns with how optional deps are currently managed here, but I generally prefer pbr's approach to the question (no deps at all in setup.py, all in requirements files and pulled in by setup.py).
So, shall I move them to extras_require?

@wldhx I'd prefer it if they went under extras_require, if possible?
(Apologies for this taking forever, moving between places)

@Half-Shot No problem, done. I'm sorry for high latency too - were busy.
Then there's also the question of whether to pin or not to pin. You didn't mention your preferences on that, so I took the liberty: 74fb9b9 pins to version number.

This should have been merged ages ago...Sorry for the lateness.


proj: configure tasks statuses

These allow for the following workflow :
backlog > ToDo > Plan > Dev > Test > Docs > Review > Done
Nicely viewed on project's Kaban board.
Note that tasks should be ordered by priority, top to bottom, for each status/column.
I believe this should allow for a decent tasks and issues workflow.
Not all steps are mandatory of course.
By default, tasks/issues are added to backlog (no status) before they are planned for current/next iteration.
Work in progress should be limited to what can be achieved during iteration.
Theoricaly, only backlog and Done should be populated at the end of each sprint.

Please review and close if that's OK for y'all


chore: merge develop into master

A summary of what changes this PR introduces and why they were made.

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactor
[ ] Performance
[ ] Tests
[ ] Build
[ ] Documentation
[ ] Code style update
[ ] Continuous Integration
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Does this PR release a new version?

[ ] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the develop branch, not the master branch
[ ] All tests are passing
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Codecov Report

Merging #115 into master will increase coverage by 4.24%.
The diff coverage is 89.02%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
+ Coverage   87.28%   91.52%   +4.24%
==========================================
  Files          27       30       +3
  Lines         739      861     +122
==========================================
+ Hits          645      788     +143
+ Misses         94       73      -21

Impacted Files
Coverage Δ

src/include/cpp-crypto/transactions/deserializer.h
100% <ø> (ø)
:arrow_up:

src/include/cpp-crypto/transactions/serializer.h
100% <ø> (ø)
:arrow_up:

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

src/include/cpp-crypto/transactions/transaction.h
66.66% <0%> (+66.66%)
:arrow_up:

src/helpers/crypto.cpp
100% <100%> (+3.22%)
:arrow_up:

src/helpers/crypto_helpers.h
100% <100%> (+50%)
:arrow_up:

src/common/network.cpp
100% <100%> (ø)

src/managers/network_manager.cpp
100% <100%> (ø)

src/common/configuration.cpp
100% <100%> (ø)

src/include/cpp-crypto/identities/publickey.h
100% <100%> (ø)
:arrow_up:

... and 34 more

Continue to review full report at Codecov.

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


Prefer using lhttpc over ibrowse if possible.

I don't particularly like the ibrowse source code, so if possible, consider using lhttpc instead.

Merged to master.


Delete CODEOWNERS

[ ] Have you followed the contributing guidelines?
[ ] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

I' apologize I'm unable to comply please delete any and all info and
accounts associated with this email and I apologize thank you have a good
day good bye
On Mon, Jul 11, 2022, 4:55 PM Surachai1994 @.***> wrote:

Have you followed the contributing guidelines
https://github.com/github/opensource.guide/blob/HEAD/CONTRIBUTING.md?
Have you explained what your changes do, and why they add value to
the Guides?

Please note: we will close your PR without comment if you do not check
the boxes above and provide ALL requested information.

You can view, comment on, or merge this pull request online at:
https://github.com/github/opensource.guide/pull/2892
Commit Summary

9227fe1
https://github.com/github/opensource.guide/pull/2892/commits/9227fe1ed1d48bf23e493a7e6e9544ffcbf64b46
Delete CODEOWNERS

File Changes
(1 file https://github.com/github/opensource.guide/pull/2892/files)

D CODEOWNERS
https://github.com/github/opensource.guide/pull/2892/files#diff-fcf14c4b7b34fe7a11916195871ae66a59be87a395f28db73e345ebdc828085b
(1)

Patch Links:

https://github.com/github/opensource.guide/pull/2892.patch
https://github.com/github/opensource.guide/pull/2892.diff

—
Reply to this email directly, view it on GitHub
https://github.com/github/opensource.guide/pull/2892, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY73US2XDDX4YTTYSUMJATDVTSX7XANCNFSM53JFZESA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


DO NOT MERGE: Fix redirectable with blank uris

NOTE: This PR starts from Doorkeeper 5.0.2 as a base.  This is intended to serve as a temporary source for our gem in the interim while we upgrade to Ruby 2.4+.  This branch is pointed to from our gemfile, do not merge this PR!
Summary
We are fixing redirectable? returning the wrong result as a consequence of a different error name type.  This should function correctly independently of error name type.
In base Doorkeeper, there is a PR and commit that the maintainer is looking to push into the latest 5.2 that fixes the issue in one way https://github.com/doorkeeper-gem/doorkeeper/commit/444fdbed7aaf8df023a4daf0bacdd31ffb8374eb where their intent is that validations (and the error response generated) need to happen in a very specific order and would avoid running into this issue.

Thinking about closing this PR, but we are actively using it in our bigmaven gemfile and I cannot risk someone accidentally coming back here to delete it.

That commit looks like it's been rolled up into this PR and comment https://github.com/doorkeeper-gem/doorkeeper/pull/1277/files#r310095549
As soon as we get to Ruby 2.4/2.7 we should consider upgrading.
@jhmoore Is there anything we can do to limit access to this fork or prevent merging while this gets worked on?


bridges | disable all actions when menus are open

Motivation
closes #272
All actions where still fully functional when the pause or goal menu was open. They should be paused/disabled instead.
What was changed/added
Added global variables to keep track of when menus are open and changed functions to be dependent on these variables.
Also wrote new global function to change visibility of all ui elements which also disables them when invisible.
Testing
https://github.com/mango-gremlin/arch-enemies/assets/116217918/8b8cde02-88a7-4ce1-a6b8-cae678a56fad
Additional Information
Remind me to add similar functionalities to overworld menu.

There is an error when reaching the goal and pressing continue, but this is related to bridge_connections/edges which I am not at all familiar with.

There is an error when reaching the goal and pressing continue, but this is related to bridge_connections/edges which I am not at all familiar with.

Did you test it while directly starting in grid.tscn? As the levels are now connected to the main overworld, you need to start there and enter the level for the bridge_edge to be set correctly. Without doing that, the level doesn't know its own bridge edge.


Implement compiler support for dictionary

(Based on #3775, #3760, #3715)
Implement most dictionary operations.
Still missing:

comparisons (equality)
boxing and unboxing
cpython side interface
refct types are not supported; only works on POD types.
typeinference backtracking magic
(am I missing anything?)

Actual changes starts at https://github.com/numba/numba/pull/3777/commits/c56c5ac5795166dfdc3835dccf4019c127ca10ed.
The major changes are in:

numba/dictobject.py  for the compiler support of the dictionary
numba/types/containers.py for the dictionary typing
numba/tests/test_dictobject.py for the tests

Note: TypeRef is added to allow referencing any numba-type as global-value or arguments as used in the constructor of dictionary.  It is different from NumberClass, which is for i.e. np.float32.

There are 2 unrelated failures.

linux py35_np111_cov is failing due to network connection error.
windows py27 is failing due to DLL hell.

https://github.com/numba/numba/pull/3777/commits/d5befda4842a9f6994cecde595630c99de8213cb added a unsafe casting check in _cast

Included tests from @stuartarchibald.
3 tests are disabled:

2 tests are using unicode_type that needs refct.  They will be handled in a later PR.
1 test is disabled due to lack of support for using dict.keys() as a set-like object.

Thanks for all the work on this @sklam, it seems stable as a ground work PR for the python side of this feature. Testing covers the functionality presented and the implementation seems to match the definition, marking for merge!


DockerComposeContainer has to be started explicitly?!

This is unlike GenericContainer(which is not good because it makes it unfamiliar) and I'm not sure if its intentional; However if it is intentional this should be mentioned in the docs.

Both DockerComposeContainer and  GenericContainer implement Startable and have to be either explicitly started with Startables#start or by using the JUnit rules / extensions.
There is @ClassRule in the docs.

I am using testcontainers-spock and looks like the @org.testcontainers.spock.Testcontainers annotation starts the containers that are non static fields of the class if the fields are explicitly typed(you cannot omit the type in groovy for these fields). Again @org.testcontainers.spock.Testcontainers lacks any javadoc to explain these stuff, but it seems you are no fan of documentation.


New PR to test SignCLA at 2018-11-28 17:41:47.275881704 -0800 PST m=+4.948255547

Body of PR

unsigned-fail-staging: test succeeded


Fix memory leak at statistics

Closes #80

Iam currently in the middle of testing my changes. I think i can release this very soon. 😁


ListHooks is not needed as part of the scm.SCM interface. I'm unsure if we need a test that checks that our manifest-based app creation installs hooks correctly.

ListHooks is not needed as part of the scm.SCM interface


Refactor the integral data types, and some casting, for BsMemoryAllocator.h

Mostly it's removing casts that I don't see the benefit of, and normalizing the integer data types used as function parameters. E.g. using size_t when counting "number objects" and such.

Looks good, merged as is.


prettied and fixed

For some reason some browsers hate the generic color names such as "white" so let's change those to #fff which is hex for white, and #000 being hex for black.
switched fixed attachment into 1 line
centered the background for a better view
better code structure (4 indents instead of 6/8 couldn't tell)

Well thank you I uploaded it to github without looking at the result. Dunno how all these tabs got there.
Appreciate it.

np


feat: add testing

We had no test so we were not catching wrong package update.
Split a bit the code so we can start testing

Merging directly because it's currently broken


Find / create missing linked notes

Is your feature request related to a problem? Please describe.
Sometimes adding a link to a note before it exists is convenient. This is particular true for tags. While writing, I often add links to tags. Only after do I realize that they don't exist yet. This has resulted in many notes linking to tags that don't exist.
To solve this I'd have to go through them one by one and click one each tag, and create missing files.
Describe the solution you'd like
A way to more efficiently create linked notes -- there might be a few ways to do it:

When adding a link, if a note is not found, add a suggestion to create it
When running Doctor, create all notes that are linked but don't exist (or at least show a list of notes that are linked but don't exist)
Add a new command that can find all broken links

Describe alternatives you've considered
grep all [[]] and then looping through them in bash to create files.

Great suggestion. To start off, I think adding this to Doctor (2) makes sense. I also like 1.
We can do either opt in or opt out via config (eg. create a file if link doesn't exist and then notify user that a file has been created - this would happen on file save to avoid creating spurious files).
Would the opposite of this also be useful? (eg. delete empty notes that have been around for longer than one day)

I don't think immediately deleting would be good because it's destructive,
but having a search that lists all empty files would be very helpful.
Another extension accomplished functionality similar to search results by
opening a new file and dumping the search results there. I can't recall
which one at the moment though but it worked well.
On Mon, Nov 16, 2020 at 10:18 PM Kevin Lin<EMAIL_ADDRESS>wrote:

Great suggestion. To start off, I think adding this to Doctor (2) makes
sense. I also like 1.
We can do either opt in or opt out via config (eg. create a file if link
doesn't exist and then notify user that a file has been created - this
would happen on file save to avoid creating spurious files).
Would the opposite of this also be useful? (eg. delete empty notes that
have been around for longer than one day)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/issues/355#issuecomment-728090335,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI44VCKT7R7BNWNX7OC5YLSQEYCFANCNFSM4TWXJZWQ
.

I'm working my way towards adding a doctor command action CREATE_MISSING_LINKED_NOTES to start with. Once I get that to work, I'll look into adding a config to enable this function on save.

I think that this has been implemented, maybe this issue can be closed?

You are correct, @chmac,

Doctor now has the createMissingLinkedNotes command that will create notes that are linked to but don't exist,
and findBrokenLinks will show a list of broken links

In workspaces with two or more vaults the above commands only work on links the vault prefix though, if you have a multi-vault setup and links without the dendron:// prefix, it won't work.
@ognjenio, if you would still like an option to show a suggestion to create a note when adding a link, please open a new issue for that.


Support for interpolation (aten::upsample_nearest)

Description
Support for aten::upsample_nearest3d, aten::upsample_nearest2d, and aten::upsample_nearest1d
Also fixed a bug that forced input tensors to be at least 4 dimensional.
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation and have regenerated the documentation (make html in docsrc)
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes

@abhi-iyer I think you probably should rebase this branch on master to resolve the conflicts


Windows heating

Hello. Congratulations for your plugin.
But I have problem. It's possible to enable only Windows heating?  Because actualy only climatisation work. Before With weconnet, I can activate Clim or Windows heating or both.
Thank's very much for your work.

Hi,
Thanks for the suggestion. Unfortunately with the current API it is not possible. This option is also not available in the Volkswagen app, so probably this functionality was removed from the API at some point.

If this feature is available on the app

Looks like your car is fancier than mine :). Unfortunately in my ID.4 it is not available, so I can't inspect the backend communication with the app to see what happens exactly.

what are you inspecting with? i try "charles proxy" but it don't work. I want to try to see what is the command that is sent from my iphone and try to integrate it

I use Wireshark.

Unfortunately I don't have the time to walk you through the process, but look for mitmproxy tutorial. Mitmproxy captures the iphone traffic, decrypts the ssl and shows on screen.

no problem. I use a version of your plugin that I modified because many things did not work for my vehicle.
for the api I also modified them because the refresh times made the volkswagen application crash. so you do not need to put it in your version I will do it myself 🙂

so I am analyzing HTTPS requests, but I have a problem with mitmproxy
i have installed the certificate and I am confident in my iphone, I have set the proxy of the mac that uses mitmproxy and I get the traffic visualization but is it normal that the server must be unreachable? as soon as I deactivate the proxy settings in my iphone the communication returns in the volkswagen application ?

No, the communication should be normal. Can you take a look at the mitmproxy log (maybe increase verbosity level)?

it is not verbosity level , i have this fault : warn: [21:40:58.888][<IP_ADDRESS>:56107] Client TLS handshake failed. Client and mitmproxy cannot agree on a TLS version to
use. You may need to adjust mitmproxy's tls_version_client_min option.

I have not encountered this error, so I don't know. I remember I had to disable strict certificate checking or something like that, but that is probably unrelated to the error you are seeing.

I stumbled across this while looking into moving to homebridge from ha.
Would this repo help? I also have the newer module and it works https://github.com/robinostlund/homeassistant-volkswagencarnet/tree/master


[CWC Toggle] Add AI enhancements

[!WARNING]
Blocked by

https://github.com/carbon-design-system/carbon/issues/15472

Closing as not planned for now. We can re-open if this gets prioritized in a future phase.


[WIP] get_features type fix

Please do not merge yet!
This PR will be to try to address KBASE-3172.  I will add mocks for how the code could work and ask for comments.

Current coverage is 68.71%

Merging #133 into develop will not affect coverage as of 3529c1f

@@            develop    #133   diff @@
=======================================
  Files            11      11
  Stmts          2362    2362
  Branches          0       0
  Methods           0       0
=======================================
  Hit            1623    1623
  Partial           0       0
  Missed          739     739

Review entire Coverage Diff as of 3529c1f

Powered by Codecov. Updated on successful CI builds.

This is a Python translation of what I think the thrift spec I wrote in the jira ticket would look like.  I am not sure it's what people are asking for though.  Pretend that if there's an mRNA feature associated with these dummy features, it doesn't get returned.
result = api.get_features(types=['CDS','gene'],feature_ids=['kb|x.CDS.y', 'kb|x.gene.z'])
print result
{'kb|g.x.CDS.y':
{
'CDS':
{'feature_id':'kb|g.x.CDS.y', 'feature_function': ...},
'gene':
{'feature_id': 'kb|g.x.gene.m': ...}
},
'kb|g.x.gene.z':
{
'CDS':
{'feature_id':'kb|g.x.CDS.n', 'feature_function': ...},
'gene':
{'feature_id': 'kb|g.x.gene.z': ...}
}
}

After talking with Jason and Marcin, I think we have a better idea for the return structure.
result = api.get_features(types=['CDS','gene'],feature_ids=['cdsid1', 'geneid2'])
print result
{
  'geneid2':
  [
      {'feature_id':'cdsid3', 'feature_type': 'CDS','feature_function': ...},
      {'feature_id': 'geneid2': 'feature_type': 'gene', .... }
  ],
  'cdsid1':
  [
      {'feature_id':'cdsid1', 'feature_type': 'CDS', 'feature_function': ...},
      {'feature_id': 'geneid4': 'feature_type': 'gene', ....}
  ]
}

Having a list means that if you do result = api.get_features(types=['CDS'],feature_ids=['cdsid1', 'cdsid2']) then you can always look only at result[key][0].  And if you don't submit a types list, meaning you want every feature that matches the CDS, then you very likely want every feature anyway and need to look at every feature in the list (e.g., you're doing a track browser).  This comes with the drawback that if you make the first call above where types=['CDS','gene'], you may get unwanted information and you have to inspect every feature in the list to see which one you want, whereas with the tuple substructure you could simply ignore the types you don't want.
By the same reasoning, to optimize making it easy to call the function for the common use cases of 1) I want only one type, or 2) I want all types, I'm leaving the inputs the same, because in both cases, I will probably generate a list from get_feature_ids and not want to have to generate a list of tuples where every type is the same.
The tuple-style would make the return result nonredundant:
result = api.get_features( [ {'cdsid1','CDS'}, {'geneid2','gene'} ] )

But I think the benefits of not having to make these tuples in the common cases outweighs the drawbacks.
Comments?

The new proposed Thrift spec.  No new Feature_data_by_type object should be needed (woo-hoo!), so it's just a change to the get_features() spec.
map<string,list<Feature_data>> get_features(1:required string token,
  2:required ObjectReference ref,
  3:list<string> feature_type_list,
  4:list<string> feature_id_list) throws ( ....

If no feature_type_list is provided, return all types.  If no feature_id_list is provided return all features.

My get_feature_ids() - strand bugfix is also in here.  When pull #132 is merged it should make that redundant.

How does this reconcile with how get_feature_ids() works currently?
Why not duplicate the interface for get_feature_ids()?  You could then just internally call get_feature_ids to do the filtering and then fetch only that data.  You basically have to duplicate this kind of logic anyway, so why not take advantage of the code that already does this?
The only remaining item then would be to decide what format the ids should be given in.  We could accept the output of get_feature_ids() as the input format for get_features(), meaning instead of a list you pass in the dictionary returned by get_feature_ids() and we deal with that.  This would at least be internally consistent, and then you could preserve method chaining.
Example:
(assuming you got back your feature ids from somewhere other than get_feature_ids())
result = api.get_features(feature_ids={"gene": ['geneid2'], "CDS": ['cdsid1']})
print result
{
'geneid2':
{'feature_id': 'geneid2': 'feature_type': 'gene', .... }
'cdsid1':
{'feature_id':'cdsid1', 'feature_type': 'CDS', 'feature_function': ...}
}
(if you used get_feature_ids())
result = api.get_features(feature_ids=api.get_feature_ids(filters={"type_list": ["gene", "CDS"]})["by_type"])

From what I could tell looking at the thrift specs, the code, and example returns from get_feature_ids(), get_features() currently can't accept the output from get_feature_ids() in general, since in some cases get_feature_ids() returns a dict, but get_features() requires a list as input.  And I think that the output for group_by options like by_region is also more complex than just {key:[ids]}, so the structure is a little different from the other group_by structures (I didn't look at that as carefully though).  It might be possible to reconcile all of the group_by formats so that get_features() could operate on any possible get_feature_ids() output, but I was planning on only addressing the issue Jason found, in order to save time.
Alternatively, if get_feature_ids() group_by region is supposed to be returning a dict with the values as a list, and not a more complex structure, then we can make sure it does that in all cases, and then it'd be easier (probably a lot easier but that's just a hunch) to make get_features() be able to take get_feature_ids() output as input.
Tomorrow I can reproduce all the possible outputs that get_feature_ids() emits so we can compare them.

Sorry, but I will not be in favor of quick fixes here, this has to be done correctly.  Let's determine first what the right thing to do is, and then we can worry about time.  I'll make the changes if needed.

Ha, I got get_feature_ids() grouped by region earlier this week.  Here's what that output looked like (from Slack).
{'by_region':
    {'kb|g.166819.c.0':
        {'+':
            {'1002883-1004221': ['kb|g.166819.mRNA.743', 'kb|g.166819.CDS.713', 'kb|g.166819.locus.886'],
            '1002088-1002643': ['kb|g.166819.locus.516', 'kb|g.166819.mRNA.649', 'kb|g.166819.CDS.637'],
            '1004617-1005487': ['kb|g.166819.mRNA.57', 'kb|g.166819.CDS.262', 'kb|g.166819.locus.73'],
            '1006886-1009535': ['kb|g.166819.locus.186', 'kb|g.166819.CDS.183', 'kb|g.166819.mRNA.276']
            },
        '-':
            {'1010461-1010670': ['kb|g.166819.mRNA.61'],
            '1001186-1001957': ['kb|g.166819.mRNA.263', 'kb|g.166819.locus.166', 'kb|g.166819.CDS.41'],
            '1000296-1001148': ['kb|g.166819.locus.122', 'kb|g.166819.CDS.133', 'kb|g.166819.mRNA.301'],
            '1006208-1006898': ['kb|g.166819.locus.987', 'kb|g.166819.mRNA.706', 'kb|g.166819.CDS.748'],
            '1010461-1011308': ['kb|g.166819.locus.142'],
            '1010097-1010580': ['kb|g.166819.CDS.135', 'kb|g.166819.mRNA.61'],
            '1010396-1010540': ['kb|g.166819.CDS.135']
            }
        }
    }
}

One compromise that would be easy would be to make get_features() take the output that get_feature_ids() emits with group_by=type.  This way that can still be chained without having to modify get_feature_ids() now, and later we can work on making get_features() work with get_feature_ids() in more cases.
I just saw your comment but I'm going to post this first.

I can do the changes, it just means I'd spend less time on other tasks (like the Perl docs or more rigorous testing).  If you really do have the time for these (which you're not supposed to, you're supposed to be on leave!) that's great too.  (We can also tag-team; you could work on fixing the get_feature_ids() output and I could work on fixing get_features().)

FWIW my preference is that get_features() doesn't have to try to figure out the structure of its inputs before proceeding; I feel like that introduces too much complexity.

Jason pointed out that the default structure for the get_features_id() default group_by (by type) does have a "by_type" key, so we'd potentially want get_features() to take an input structure like this:
{ "by_type": {"gene": ['geneid2'], "CDS": ['cdsid1']} }

So then accepting a list of tuples would solve that, which would mean
get_feature_ids() output would change.  Let's write out some examples fully
to see what everything would look like before we change code or thrift
specs.
On Thursday, December 10, 2015, kkellerlbl<EMAIL_ADDRESS>wrote:

FWIW my preference is that get_features() doesn't have to try to figure
out the structure of its inputs before proceeding; I feel like that
introduces too much complexity.
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/data_api/pull/133#issuecomment-163734941.

Okay!  We talked after our standup, and we have agreement on inputs and outputs that's expressive enough for what we need while minimizing the changes needed.
get_features() will take as input a list of tuples:
[ ('gene', 'featureid1'), ('CDS','featureid2'), ('typeN','featureidN'), ... ]
It should support 'all' as a wildcard type.
In addition, get_feature_ids() will return as output, instead of a list of feature ids, a list of feature tuples.  So, for example, the by_region output will now be (much truncated):
{'by_region':
    {'kb|g.166819.c.0':
        {'+':
            {'1002883-1004221': [ ('mRNA', 'kb|g.166819.mRNA.743'), ('CDS','kb|g.166819.CDS.713'), ('gene','kb|g.166819.locus.886')],
            },
        '-':
            {'1010461-1010670': [('mRNA','kb|g.166819.mRNA.61')],
            }
        }
    }
}

This way, no matter what the structure of the get_feature_ids() output is, the lists that are the leaf values can always be used as input to get_features(), without needing to munge the values in the list.
Any objections to what I've written?

I will be updating the Thrift specs to reflect our talk and my summary above.

The balance is low complexity return vs being able to make complex queries
for features. This is becoming hard so realistically we will need higher
level methods on top of this anyway.
Still, getting all ids and features for a single contig is a standard
operation and that return does look pretty complex now because the return
will be three deep: by_region, contig_id, and region coordinates. Is that
right? Imagine we are asking for all mRNAs from a contig (in this example
from a small region)
{'by_region':
{'kb|g.166819.c.0':
{'1002883-1004221': [ ('mRNA', 'kb|g.166819.mRNA.743')]
}
}
}
However, since the main use of this method for now is supplying inputs to
other methods, its really more of matter of making the other methods
compatible. I think soon we will need another version of get_feature_ids
which just returns a list of ids.
Also, I'm worried about feature ids here again. Are the 'gene' values going
into get_features() of the type 'kb|g.166819.mRNA.743', that is they are
fully qualified?
A little disconcerting that the CDS and mRNA ids do not match -- they won't
match exactly due to alternative splicing but the euk convention is
generally id.1, id.2 for two different mRNA/cds originating from the same
genomic locus.
Finally, what specific CDS and mRNA ids are we suggesting that developers
use in their code? Since the original ids in different containers
overlapped it should be something like the fully qualified ids.
p.s. We still have '|' in KBase ids??? I thought this was our chance to
improve ...
On Thu, Dec 10, 2015 at 2:59 PM, kkellerlbl<EMAIL_ADDRESS>wrote:

Okay! We talked after our standup, and we have agreement on inputs and
outputs that's expressive enough for what we need while minimizing the
changes needed.
get_features() will take as input a list of tuples:
[ ('gene', 'featureid1'), ('CDS','featureid2'), ('typeN','featureidN'),
... ]
It should support 'all' as a wildcard type.
In addition, get_feature_ids() will return as output, instead of a list of
feature ids, a list of feature tuples. So, for example, the by_region
output will now be (much truncated):
{'by_region':
{'kb|g.166819.c.0':
{'+':
{'1002883-1004221': [ ('mRNA', 'kb|g.166819.mRNA.743'), ('CDS','kb|g.166819.CDS.713'), ('gene','kb|g.166819.locus.886')],
},
'-':
{'1010461-1010670': [('mRNA','kb|g.166819.mRNA.61')],
}
}
}
}
This way, no matter what the structure of the get_feature_ids() output is,
the lists that are the leaf values can always be used as input to
get_features(), without needing to munge the values in the list.
Any objections to what I've written?
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/data_api/pull/133#issuecomment-163779653.

This PR is not meant to address the structure of the return value when using group_by=region.  The only change there is that the list of features is now becoming a list of feature tuples.
The input to get_features() will be a list of feature tuples (instead of a list of feature ids, as it is now).  So you might call
result = api.get_features( [ ('gene','geneid1') , ('CDS','cdsid2'), ('all', 'featureid3') ] )

In the thrift spec I called the new type "Feature_tuple" but I am not tied to that label.

OK, but this is a really near term fix in my mind -- the fact that we need
this reveals a data modeling flaw.
Going forward we can't solve this with fully qualified ids? That sounds
like the right long term solution. Having redundant ids is not just a
problem for the data API -- how will people know which id they are
referring to in their code?
On Thu, Dec 10, 2015 at 4:19 PM, kkellerlbl<EMAIL_ADDRESS>wrote:

This PR is not meant to address the structure of the return value when
using group_by=region. The only change there is that the list of features
is now becoming a list of feature tuples. The input to get_features() will
be a list of feature tuples

feature id tuples to be exact -- as long as the arg data is exactly
provided by other methods (eg get_feature_ids) we are good. otherwise there
are dependent changes worth making.

(instead of a list of feature ids, as it is now). So you might call
result = api.get_features( [ ('gene','geneid1') , ('CDS','cdsid2'), ('all', 'featureid3') ] )
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/data_api/pull/133#issuecomment-163793551.

I think all of the issues you bring up are out of scope for the work on this PR, which is really meant as a fix for bug KBASE-3172.

Fine by me.
On Thu, Dec 10, 2015 at 4:45 PM, kkellerlbl<EMAIL_ADDRESS>wrote:

I think all of the issues you bring up are out of scope for the work on
this PR, which is really meant as a fix for bug KBASE-3172.
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/data_api/pull/133#issuecomment-163797663.

I will be adding a very large commit which is the Thrift libs generated from the new spec by 0.9.3.  Please do not merge this PR!  It is definitely not ready.  (I can close it if people think there is a risk of accidentally merging, but I'd like to leave it open so you can watch my progress.)

I just made some commits for get_feature_ids(), but the work is still in progress, and I'm pretty sure tests will fail.  Please don't merge, but read and comment if you have questions or objections so far.

As we mentioned in Slack and on our standup, there is a possibility that other methods may need to be updated to accept a list of tuples of ('feature_type','feature_id') as input.  I'm hoping to fix get_features() in this PR, then make new PRs for other methods that need to change (even though there would be spec changes for those).

Also, I will probably need to fix the tests, since it's likely they will currently fail.

I just made a new commit for get_features.  I checked the kbase/develop version of get_features, and it also emits a Region Thrift object instead of a generic dict() for the feature locations.  That's not fabulous, but at least my changes didn't break anything that was working before.
There are still no tests (or tests are broken) and I still need to look at the example Jason found to see whether my changes actually fix the problem (which was the whole point, after all).

I just added some code to api.py as what I hope is the final changes needed for get_features().  I still expect tests to fail, but if by some miracle they pass, please do not merge, as I would like to make sure that the tests are actually testing the right things.

Keith, we're basically dumping the feature tuples because it just leads to complications, but you have things in here which are not related.  I will go through your changes compared to the kbase/develop branch to see what we can keep.  In hindsight we should have discussed the feature tuples more before going in that direction, and this was on me for not being able to spend more time when this came up.

Ok, sounds good.  I thought I tried to minimize how much non-get_features() code snuck in but maybe I missed some.  I hope it's not very much.
This will also impact the PR for the perl_clients code; it's worse there because I was less careful about separating out the code.  I'll browse the diffs and see what might be impacted.

Ok, I'm going to close this since we ended up not going down this route.


[UNR-1888] Bugfix: Guns were not being attached to players sometimes

In AGDKCharacter::BeginPlay a callback is registered on the UEquippedComponent::HoldableUpdated. This callback is used to attach the new AHoldable to the player's skeleton.
Blueprint subclasses: BP_Base_Character, BP_FPS_Character, BP_SimulatedPlayerCharacter had overridden BeginPlay but not called the Parent function, so the callback was not attached.
This meant that if a weapon was checked out after the character, it would not be attached, and would stay floating where it was spawned.
Tests
Tested in cloud deployment with human + sim players. Saw no floating guns / empty-handed players (which were quite common)


Possible error in SwerveMath::calculateMaxAngularVelocity

I'm new and trying to read through and understand the YAGSL library code.  If I understand this function correctly, the purpose of this function is supposed to be to calculate the maximum angular velocity of the robot possible from the maximum linear speed of the robot.
The given equation in line 129 is
return maxSpeed / new Rotation2d(furthestModuleX, furthestModuleY).getRadians();
[meter/sec] / [radians] = [meters/(sec*radians)]
which is the speed of the robot divided by the angle of furthest module in radians, something I don't believe was intended.  I suspect the intended value is the maxSpeed divided by the radius of the circle centered on the theoretical center of the swerve drive system and running through the furthest module, which would be
return maxSpeed / Math.sqrt(furthestModuleX*furthestModuleX+furthestModuleY*furthestModuleY);
[meters/sec / radius(meters)] = radians/sec
Also, in line 36 of SwerveControllConfiguration this function is called just using the position of module 0 without checking to see which module is farthest from the center.
My apologies if I misunderstand the code or made a mistake somewhere...

Hmm... Could you reconsider and/or explain your reasoning?
If I set the "left" value of the location of all the swerve modules to 0 (only one has to be 0, but it has to be the first module the library reads because it's always index 0 that's used so it's easier just to set them all to 0 for debugging), e.g.
backleft.json:
...
    "location": {
      "front": -13.5,
      "left": 0
    }
...

this results in
maxSpeed / new Rotation2d(13, 0).getRadians()
which leads to
StrictMath.atan2(0, 13);
which is equal to zero.
In the debugger, this yields a maxAngularVelocity of infinity.  Is an infinite maxAngularVelocity the intended behavior if the left location of a swerve module happens to be 0?
Again, apologies if I don't understand the code properly.


Dependabot reporting on deleted files

I've committed a package.json with a library subdirectory included in my project (not using package manager for this project).
Dependabot reported a security alert.
I've deleted the package.json and thought that it would resolve it.

But I'm still getting the alert listed in my daily e-mail notification (and in the list of alerts on repository Security page). I've tried to dismiss the alert with "A fix has already been started", didn't change anything.
The alert page further shows a warning message "Dependabot cannot update this dependency" detailed as "Dependabot couldn't update this dependency to the required version as it doesn't support your dependency files.".

I converted some repos from yarn.lock to package-lock.json and I only get Github Security Advisories for the long-since deleted yarn.lock file. It is unclear if the existing package-lock.json file is monitored.

I have monorepo and deleted one of the packages. But still getting alerts once a while (until dismissed) for no longer existing package-lock.json

Getting this as well.

I too am having this issue.  The files it's reporting have been deleted for months.  Even the folders that they used to be in are long gone.

The same for me. File and directory have been deleted a while ago but I still get new notifications about vulnerabilities.

Did you have a branch with those deps still available ?

What would fix this?

Did you have a branch with those deps still available ?

In my case, no.  There are only two branches and the files were completely removed from both.  I haven't checked it in a while as all I was getting was false reports so I disabled the feature.

I also recently experienced this - received an alert against a package.json file that had been deleted from the master branch weeks ago. Clicking the link to the package.json in the alert resulted in a 404. We do have stale branches that will still include that package, but my understanding was that dependabot should only pay attention to the default branch.

We had this issue today (GitHub Enterprise 2.22.4).
A workaround -

Replace the deleted package.json file with one that has zero dependencies. Commit it.
Delete the package.json file. Commit it.

The result of this is the zeroing out of the Dependency Graph for that file, but the file remains mentioned in the dependency graph view. You shouldn't get any more security warnings for those outdated dependencies.

I know the Dependency Graph team is aware of this, I'll link them to this thread

+1 - Receiving the same issue with deleted directories reporting vulnerability alerts regarding package-lock.json files for numerous repos. The alert is dismissed and reappears a few weeks/months later.  Similar to @karlhorky with new alerts coming in.

+1
I have a project where a deleted Pipfile.lock keeps getting vulnerability findings, even though the file was deleted on all branches months ago.

I have a similar problem.
I stopped using Yarn, but my security alerts indicate that dependabot is still reading from yarn.lock.
At the moment running npm audit locally shows 0 vulnerabilities.
Please give us some way to manually force dependabot to recognize the new lock file.

The Dependency Graph fixed an issue with deleted manifests a while ago, these issues might pre-date the fix? If you let me know the repo this is happening on I can force a re-detect, or feel free to contact support and they should be able to do it as well.
Unfortunately there is currently no way to run the re-detect yourself, and (possibly confusingly) our team doesn't maintain this feature, but I'm happy to help and so are our wonderful support engineers.

@jurre is the link enough or should I also contact support like you said? Where can I do this?

Hi @raniesantos, I just kicked off the re-detection. I was out for a few days over Easter hence the slow response.
FWIW support can be contacted via https://support.github.com/

I also need a re-detect, on repo https://github.com/ccrraaiigg/caffeine/. Thanks!

We had this issue today (GitHub Enterprise 2.22.4).
A workaround -

Replace the deleted package.json file with one that has zero dependencies. Commit it.
Delete the package.json file. Commit it.

The result of this is the zeroing out of the Dependency Graph for that file, but the file remains mentioned in the dependency graph view. You shouldn't get any more security warnings for those outdated dependencies.

Just had the issue (on a previously existing 'package.json' deleted a while ago).
This workaround seems to have solved it.

Can you please seriously consider giving repo owners more direct control over this?
I thought my issue would have been solved already, but it just came back.

I got the same, I get alerts for Gemfile.lock which does not exist....

Happening to me too... super annoying

I'm back! I just got a report for an axios vulnerability for a repo that doesn't even use axios. Yaaaay!!!!

It makes very little sense that the alert is sent week upon week, while the file is actually missing. And marked as inaccurate every week.

Hey, I know it's somewhat confusing but dependabot-core is not used for the alerts feature on GitHub. This issue is on that teams radar though, so I'm going to close this out from our issue tracker. When running into this the easiest way to reach that team is to go via github support, but we'll try to forward things as best as we can.

Acknowledging that this might not be a dependabot issue - has anyone figured out how to address this when the Dependency Graph is showing previously deleted vendored dependencies, e.g historically source controlled Ruby gems and gemspecs?
According to https://docs.github.com/en/code-security/supply-chain-security/understanding-your-software-supply-chain/troubleshooting-the-dependency-graph it's not supposed to look at vendored deps anyway (for whatever reason).
e.g. it wasn't doing so until recently, but https://github.com/gocd/gocd/network/dependencies has started showing stuff long-since deleted e.g

Deleted in https://github.com/gocd/gocd/commit/f8132a1e8f9e1ebf58c1f188f247fdfa6a032e40.
Since the source is not a single Gemfile or GemFile.lock I'm not sure what workaround might exist here. Tried adding an empty folder with .gitkeep but didn't seem to do the trick. Will try raising a ticket with Github Support, but thought someone might have noticed this and figured this one out.

You could also try creating a Code Security discussion... that's the officially recommended "public" place for raising issues about dependency graph or security alerts. But based on your description, this may be specific to your repo in which case a support ticket is probably your best bet.

Thanks @jeffwidman - will see how I get on with support and try the community otherwise. Reason I asked is that this seems to have started in the last few days around the time of the launch of the new Dependency Graph view, and the behaviour contradicts documentation so is at least a documentation bug, if not a "real" bug - usually such things affect multiple folks.
No idea if the original bug discussed here on inexplicably stale dependency graph entries has been addressed - maybe it hasn't been and some recent changes to include vendored dependencies (which seems reasonable to me, personally) has just exposed some repos with similar issues.

Just had this issue with a old yan.lock file pointing to since than long remove package. (this commit is three year old). Since then I moved from yarn.lock to package-lock.json.
I fixed my issue by adding a new yarn.lock file (by running 'yarn'). commited this. Then deleted to yan.lock file again, and commit this again. Looking at the alert page, it immediately 'removed' the issue.
Hope this helps some of us. Cheers!

I'm still seeing this in some of my repos.  Very, very annoying to get these emails for fixed issues, so I'm disabling the dependabot scan.

I'm still seeing this in some of my repos. Very, very annoying to get these emails for fixed issues, so I'm disabling the dependabot scan.

You can use the virtual agent to ask GitHub to redetect dependencies for a repo. Try this search?

While that virtual agent is a nice high-tech work-around (can't say yet if it actually does something), but how about solving the underlying problem?
I am (so far) receiving Rust dependency errors for a Python project that doesn't contain a single line of Rust in any branch any more. That indicates that Dependabot uses some hallucinated project state as basis for it's reports which is a twofold security issue in itself:

It might miss actual problems because reporting on some outdated state
It will make users annoyed because of reporting hallucinated problems, making them switch off an otherwise very useful tool.

So, Github, please fix the underlying issue.

@domschl   Based on  https://github.com/dependabot/dependabot-core/issues/2041#issuecomment-979953683  I think the issue here is that it relates to some problematic caching or something else outside dependabot itself so we'd need to find the right place to get this tracked on GitHub side of things if it's still happening, either via GH support or some other repo?

Some problematic caching? Shouldn't dependabot kind of know what data it is operating on?
No worries, I simply remove the bot on all of my projects.


Constraint counts for the accumulation verifier

What is the easiest way to produce data used in Figure 7 here: https://eprint.iacr.org/2020/1618.pdf? Also, how about the number of constraints for the accumulation verifier for the NARK for R1CS in this paper?
It looks like the examples (e.g., scaling-as) report the prover time/verifier time/etc. but not the number of constraints of the accumulation verifier.

Hey Srinath!
I pushed a commit which provides a breakdown of constraints, matching the numbers reported in the figure. To obtain these numbers, you can run the following commands:

for R1CS NARK accumulation: cargo test constraints::tests::simple_accumulation_test --features r1cs-nark-as  -- --nocapture --test-threads=1
for IPA PC accumulation: cargo test constraints::tests::simple_accumulation_test --features ipa-pc-as  -- --nocapture --test-threads=1 (You might need to manually change the degree being accumulated)
for trivial PC accumulation: cargo test constraints::tests::simple_accumulation_test --features trivial-pc-as  -- --nocapture --test-threads=1

Thanks a lot @Pratyush! This worked.
On a related note, when I do cargo build --release --example scaling-nark --features="hp-as r1cs-nark-as", it successfully builds the binaries. But when I run, say:
./target/release/examples/scaling-nark 10 12 1
I get the following error:
================ Benchmarking NARK without zk ================
thread 'main' panicked at 'assertion failed: R1CSNark::<Affine,\n           PoseidonSponge>::verify(&ivk, &[v, a, a, a, a], &proof,\n                                       Some(PoseidonSponge::new()))', examples/scaling-nark.rs:95:9
Another minor issue is that it seems like the following line in scaling-nark.rs:
if args.len() < 4 || args[1] == "-h" || args[1] == "--help"
should be:
if args.len() < 3 || args[1] == "-h" || args[1] == "--help", right?

Ah thanks for pointing that out, I've fixed it now =)

It now works, thanks! :)


fix: shorter names for some instances

This is two changes.

Firstly I manually name some instances (because I am debugging some tensor product stuff and it's getting annoying). For example

Algebra.TensorProduct.instCommRingTensorProductToCommSemiringToAddCommMonoidToNonUnitalNonAssocSemiringToNonUnitalNonAssocRingToNonAssocRingToRingToAddCommMonoidToNonUnitalNonAssocSemiringToNonUnitalNonAssocRingToNonAssocRingToRingToModuleToSemiringToCommSemiringToModuleToSemiringToCommSemiring : CommRing (A ⊗[R] B)
now becomes Algebra.TensorProduct.InstCommRing : CommRing (A ⊗[R] B)

toRing := blah generates a slightly less convoluted term than blah: ring with.

thanks!
maintainer merge

LGTM, thanks!
bors r+


contextData in Office.SmartAlertsEventCompletedOptions

I've got a question regarding the contextData member of the Office.SmartAlertsEventCompletedOptions interface:

The contextData option specifies any JSON data you want to pass to the add-in when the Don't Send button is selected. If you include this option, you must also set the commandId option. Otherwise, the JSON data is ignored.

How does the add-in receive the context data passed through this option?

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

ID: 08db1c81-31cc-68a0-3912-fb625814aa0f
Version Independent ID: 08db1c81-31cc-68a0-3912-fb625814aa0f
Content: Automatically check for an attachment before a message is sent - Office Add-ins
Content Source: docs/outlook/smart-alerts-onmessagesend-walkthrough.md
Service: outlook
Sub-service: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @krzysztof-plutoflume, thanks for submitting this Outlook question. @samantharamon Could you take a look?

Thanks for your explanation, very clear.
If the task pane with the given commandId is already open when the user clicks Don't Send on the Smart Alerts dialog, is the task pane supposed to receive an updated initialisation context (with the value as set for contextData in Office.SmartAlertsEventCompletedOptions)?
At the moment, the initialisation context doesn't seem to be updated, and an already opened task pane isn't reloaded when the user clicks Don't Send.
Is there a reliable way to signal to the task pane that the user clicked Don't Send, so that we can update the contents of the task pane to explain to the user why the email was blocked from sending?

That's a great question. I'm also able to replicate the behavior you're seeing. Let me follow up on this with the team.


fixed broken links to Isotropix products in Readme.md

Description
Not much to say :) The links to Isotropix products are incorrect (Clarisse) or broken (Angie) I just fixed and grouped them, I hope it's okay?
Checklist:

[x] I have previously submitted a Contributor License Agreement.

Can you take care of the CLA and DCO?
(For the DCO, just git commit --amend -s ought to do the trick, then push --force again)

Hmm, I thought we already had a CLA signed, I'll check it, sorry!

Make sure you're using the same email and github ID that you used to register the CLA.


Update README.md

Updating for promises, arrow functions, let/const.

@johnpapa,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Hi @balajikris  , we're trying to update the readme files of samples since we are linking to them more from the docs. Can you take a look at this please?

LGTM. With this change, readme.md and actual code is out of sync, syntactically at least. Should there be a work item to upgrade the actual code in the sample to ES6 syntax ?

i think that would be nice. but not critical


add .dockerignore file

Solves: https://github.com/danielfsousa/express-rest-es2017-boilerplate/issues/103

Coverage remained the same at 90.09% when pulling 913d9652f0bdc99e413208706a02208ed1178446 on irokhes:add-dockerignore-file-103 into 55b04247d2a97ab5b66385cd057cd6a4c5780b2d on danielfsousa:master.


Upgrade Cargo from 1.4.19 to 1.5.0

1.5.0 brings support for EAP 6.4.7+

Backport of #245

+1
Contains fix for Email support for newer versions of EAP6.4.


Added spanish and multiselector to allow more than two languages

Hi! I have made spanish translation for your app and added a multi language selector. I'm really interested in this project and I will try to help you.
I hope that you accept my pull request!
Greetings!
P.S: If you could commit your code in english I would be grateful!

Thank you very much, this project has a lot to be optimized, I will gradually optimize it, thanks again


Scroll to validation error on all forms

I was creating an event and entered everything but the name, hit publish and nothing happens because I left the name field empty.
Can we have an event on the publish/save click so that if there is a validation error the user is scrolled to the first validation error on the page.

Technical spec
We have this on the edit dojo form, so it can easily be used as an example. See here https://github.com/CoderDojo/cp-zen-platform/blob/cf2e0c2eaecd208b91fc1bba1e2871e8f155059b/web/public/js/controllers/edit-dojo-controller.js#L36 and here https://github.com/CoderDojo/cp-zen-platform/blob/4674958d8d71c899c4fb199d402b3eb367f02efc/web/public/templates/dojos/dojo-form.dust#L287 for where it is already used.
You'll need to edit here https://github.com/CoderDojo/cp-zen-platform/blob/cf2e0c2eaecd208b91fc1bba1e2871e8f155059b/web/public/templates/dojos/events/dojo-event-form.dust#L265 and add a new function to scroll to validation here https://github.com/CoderDojo/cp-zen-platform/blob/cf2e0c2eaecd208b91fc1bba1e2871e8f155059b/web/public/js/controllers/dojo-event-form-controller.js

This is live now, thanks to @iwtcs and congrats on your first deployed change!


Knowledge - Errors relating to  json: unsupported value: NaN\n: exit status 1 seen when asking question about  knowledge file.

Electron build - 9bf8a0660d4ad
Knowlege tool version - v0.4.13
Steps to reproduce the problem:

Create an assistant and add file -
2023q4-alphabet-earnings-release.pdf

Chat with the assistant.

Notice that the chat window is stuck in this state forever- This issue is tracked in https://github.com/gptscript-ai/desktop/issues/459

Quit this thread and relaunch it
We see the welcome message presented .
Ask any question about this knowledge tool
Following errors are seen

Full error message:
"Error: ERROR: 2024/09/06 12:04:55 INFO Retrieving sources for query query=\"What are Alphabet's earnings?\" datasets=\"[cpvo6f 827]\"\n2024/09/06 12:04:55 DEBUG Using default DSN dsn=\"sqlite:///Users/sangeethahariharan/Library/Application Support/gptscript/knowledge/knowledge.db\"\n2024/09/06 12:04:55 DEBUG Using default VectorDBPath vectordbPath=\"/Users/sangeethahariharan/Library/Application Support/gptscript/knowledge/vector.db\"\n2024/09/06 12:04:55 DEBUG Using embedding model provider provider=openai config=\"{BaseURL:https://gateway-api.gptscript.ai/llm APIKey:REDACTED Model:gpt-4 EmbeddingModel:text-embedding-ada-002 EmbeddingEndpoint:/embeddings APIVersion:2024-02-01 APIType:OPEN_AI AzureOpenAIConfig:{Deployment:}}\"\n2024/09/06 12:04:55 DEBUG Loading retrieval flows from config flows_file=blueprint:context dataset=\"[cpvo6f 827]\"\n2024/09/06 12:04:55 DEBUG Query Modifier custom configuration name=enhance config=\"{Model:{OpenAI:{BaseURL:https://gateway-api.gptscript.ai/llm APIKey:REDACTED Model:gpt-4o EmbeddingModel: EmbeddingEndpoint: APIVersion: APIType:OPEN_AI AzureOpenAIConfig:{Deployment:}}}}\"\n2024/09/06 12:04:55 DEBUG Retriever custom configuration name=subquery config=\"{Model:{OpenAI:{BaseURL:https://gateway-api.gptscript.ai/llm APIKey:REDACTED Model:gpt-4o EmbeddingModel: EmbeddingEndpoint: APIVersion: APIType:OPEN_AI AzureOpenAIConfig:{Deployment:}}} Limit:3 TopK:10}\"\n2024/09/06 12:04:55 DEBUG Retriever custom configuration name=bm25 config=\"{TopN:10 K1:1.2 B:0.75 CleanStopWords:[auto]}\"\n2024/09/06 12:04:55 DEBUG Retriever custom configuration name=merge config=\"{TopK:10 Retrievers:[{Name:subquery Weight:0xc0049bec28 Options:map[limit:3 model:map[openai:map[apiKey:936e67682fc20f78:6508016f6d5de9069bc6093e78d3f68453dde78e991fc8e33174629fff56a694 apiType:OPEN_AI baseURL:https://gateway-api.gptscript.ai/llm model:gpt-4o]] topK:10]} {Name:bm25 Weight:0xc0049bec58 Options:map[b:0.75 cleanStopWords:[auto] k1:1.2 topN:10]}] retrievers:[]}\"\n2024/09/06 12:04:55 DEBUG Postprocessor custom configuration name=similarity config=\"{Threshold:0.4 KeepMin:5}\"\n2024/09/06 12:04:55 DEBUG Postprocessor custom configuration name=reduce config={TopK:10}\n2024/09/06 12:04:55 DEBUG Loaded retrieval flow from config flows_file=blueprint:context dataset=\"[cpvo6f 827]\"\n2024/09/06 12:04:55 DEBUG Retrieving content from dataset dataset=\"[cpvo6f 827]\" query=\"What are Alphabet's earnings?\"\n2024/09/06 12:04:55 DEBUG Prompting LLM prompt=\"The following query will be used for a vector similarity search.\\nPlease enhance it to improve the semantic similarity search.\\nQuery: \\\"What are Alphabet's earnings?\\\"\\nReply only with the JSON {\\\"result\\\": \\\"<enhanced-query>\\\"}.\\nDo not include anything else in your response and don't use markdown highlighting or formatting, just raw JSON.\"\n2024/09/06 12:04:56 DEBUG Modified queries before=\"[What are Alphabet's earnings?]\" queryModifier=enhance after=\"[What are the quarterly and annual earnings reports for Alphabet Inc., including revenue, profit, and any financial forecasts or statements?]\"\n2024/09/06 12:04:56 DEBUG Updated query set query=\"What are Alphabet's earnings?\" modified_query_set=\"[What are the quarterly and annual earnings reports for Alphabet Inc., including revenue, profit, and any financial forecasts or statements?]\"\n2024/09/06 12:04:56 DEBUG Retrieving documents from merging retriever query=\"What are the quarterly and annual earnings reports for Alphabet Inc., including revenue, profit, and any financial forecasts or statements?\" datasetIDs=\"[cpvo6f 827]\" where=map[] whereDocument=[]\n2024/09/06 12:04:56 DEBUG Retrieving documents from retriever component=MergingRetriever retriever=subquery\n2024/09/06 12:04:56 DEBUG Prompting LLM prompt=\"The following query will be used for a vector similarity search.\\nIf it is too complex or covering multiple topics or entities, please split it into multiple subqueries.\\nI.e. a comparative query like \\\"What are the differences between cats and dogs?\\\" could be split into subqueries concerning cats and dogs separately.\\nThe resulting subqueries will then be used for separate vector similarity searches.\\nJust changing the phrasing of the input question often won't change the semantic meaning, so those may not be good candidates.\\nLimit the number of subqueries to a maximum of 3 (less is ok).\\nQuery: \\\"What are the quarterly and annual earnings reports for Alphabet Inc., including revenue, profit, and any financial forecasts or statements?\\\"\\nReply with all subqueries in a json list like the following and don't reply with anything else (also don't use any markdown syntax).\\nResponse schema: {\\\"results\\\": [\\\"<subquery-1>\\\", \\\"<subquery-2>\\\"]}\"\n2024/09/06 12:04:57 DEBUG SubqueryQueryRetriever generated subqueries queries=\"What are the quarterly earnings reports for Alphabet Inc., including revenue, profit, and any financial statements? | What are the annual earnings reports for Alphabet Inc., including revenue, profit, and any financial statements? | What are the financial forecasts for Alphabet Inc.?\"\n2024/09/06 12:04:57 DEBUG filtering documents where=map[] whereDocument=[]\n2024/09/06 12:04:58 DEBUG SubqueryQueryRetriever retrieved documents query=\"What are the quarterly earnings reports for Alphabet Inc., including revenue, profit, and any financial statements?\" len(docs)=10\n2024/09/06 12:04:58 DEBUG filtering documents where=map[] whereDocument=[]\n2024/09/06 12:04:58 DEBUG SubqueryQueryRetriever retrieved documents query=\"What are the annual earnings reports for Alphabet Inc., including revenue, profit, and any financial statements?\" len(docs)=10\n2024/09/06 12:04:58 DEBUG filtering documents where=map[] whereDocument=[]\n2024/09/06 12:04:58 DEBUG SubqueryQueryRetriever retrieved documents query=\"What are the financial forecasts for Alphabet Inc.?\" len(docs)=10\n2024/09/06 12:04:58 DEBUG Retrieved documents from retriever retriever=subquery numDocs=10\n2024/09/06 12:04:58 DEBUG Retrieving documents from retriever component=MergingRetriever retriever=bm25\n2024/09/06 12:04:58 DEBUG Retrieving documents from dataset component=BM25Retriever dataset=827\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=c37af36e-b379-4fec-b020-b46f6c3e1dab removed=3 total=10 lenContent=121 lenCleaned=54\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=8617e429-733e-40de-bd64-ce5c6a455e7a removed=109 total=362 lenContent=3046 lenCleaned=2064\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=3b67731a-8bcd-4ced-b3c3-a2f5c7ed3839 removed=174 total=408 lenContent=2745 lenCleaned=1900\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=f67a814e-129e-425b-8b31-a705f2a8c0de removed=3 total=8 lenContent=117 lenCleaned=33\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=19314c12-b553-40ad-8968-353c02c2ca99 removed=209 total=495 lenContent=3546 lenCleaned=2230\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=c41a8d93-c36a-442d-8a85-703b2030919c removed=81 total=241 lenContent=2253 lenCleaned=1344\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=3163a4e7-fb49-409f-9e7d-f4eb135d4ca4 removed=82 total=299 lenContent=3037 lenCleaned=1780\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=157cb2d4-ac6b-44f1-85b7-4d82ae52f6ca removed=10 total=36 lenContent=375 lenCleaned=215\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=38e1a1fa-1629-4747-ac0a-dd763dfec921 removed=15 total=74 lenContent=715 lenCleaned=423\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=5266c75d-9cf4-4c20-a8e5-18ac4e4ff428 removed=47 total=108 lenContent=817 lenCleaned=527\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=7feff116-b381-4c68-8e31-e027e4fb3a00 removed=54 total=144 lenContent=1028 lenCleaned=710\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=59cf99bf-5f42-47c9-8e51-7b1941a55c00 removed=48 total=188 lenContent=2435 lenCleaned=1170\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=a27f9071-b7a9-4feb-a8b0-56fc9a42b978 removed=45 total=134 lenContent=1119 lenCleaned=686\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=558c6107-c596-497c-b2d0-963c01474067 removed=7 total=33 lenContent=402 lenCleaned=205\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=a6bf171b-6438-4625-a43b-2bd90b0ac06a removed=14 total=81 lenContent=1092 lenCleaned=493\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=ce5e3ec7-486b-421b-9750-841eb8d980a4 removed=72 total=217 lenContent=1908 lenCleaned=1123\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=8a17868c-1e97-4b8c-b591-abf28a71633e removed=29 total=103 lenContent=1280 lenCleaned=542\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=38002309-0062-4c99-8780-0de73a2e2850 removed=25 total=118 lenContent=1618 lenCleaned=731\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=b6f0d0cd-c63d-4b8a-8315-4231d079f91e removed=109 total=266 lenContent=1918 lenCleaned=1404\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=8565b150-4b44-4352-aabb-564942b5df50 removed=50 total=119 lenContent=847 lenCleaned=636\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=27a2c3e9-2776-44dc-8e32-464eb43d1160 removed=33 total=111 lenContent=1358 lenCleaned=645\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=458d0fde-f988-4c05-9c24-e64f7b17f616 removed=35 total=182 lenContent=2341 lenCleaned=1089\n2024/09/06 12:04:58 DEBUG Removed stopwords langs=[en] document=7c08907d-fac3-4ef7-a8b9-51f646312529 removed=31 total=180 lenContent=2075 lenCleaned=1191\n2024/09/06 12:04:58 DEBUG Retrieved documents from retriever retriever=bm25 numDocs=10\n2024/09/06 12:04:58 DEBUG Normalized score retriever=bm25 score=NaN minScore=3.4028234663852886e+38 maxScore=-3.4028234663852886e+38 normalizedScore=NaN\n2024/09/06 12:04:58 DEBUG Normalized score retriever=bm25 score=NaN minScore=3.4028234663852886e+38 maxScore=-3.4028234663852886e+38 normalizedScore=NaN\n2024/09/06 12:04:58 DEBUG Normalized score retriever=bm25 score=NaN minScore=3.4028234663852886e+38 maxScore=-3.4028234663852886e+38 normalizedScore=NaN\n2024/09/06 12:04:58 DEBUG MergingRetriever topK=10 numDocs=13\n2024/09/06 12:04:58 DEBUG Retrieved documents num_documents=10 query=\"What are the quarterly and annual earnings reports for Alphabet Inc., including revenue, profit, and any financial forecasts or statements?\" datasets=\"[cpvo6f 827]\" retriever=merge\n2024/09/06 12:04:58 DEBUG Keeping document below threshold docID=c37af36e-b379-4fec-b020-b46f6c3e1dab score=NaN threshold=0.4000000059604645\n2024/09/06 12:04:58 DEBUG Keeping document below threshold docID=8617e429-733e-40de-bd64-ce5c6a455e7a score=NaN threshold=0.4000000059604645\n2024/09/06 12:04:58 DEBUG Keeping document below threshold docID=c41a8d93-c36a-442d-8a85-703b2030919c score=NaN threshold=0.4000000059604645\n2024/09/06 12:04:58 DEBUG Keeping document below threshold docID=7c08907d-fac3-4ef7-a8b9-51f646312529 score=0 threshold=0.4000000059604645\n2024/09/06 12:04:58 DEBUG Postprocessed RetrievalResponse num_responses=1 original_query=\"What are Alphabet's earnings?\"\n2024/09/06 12:04:58 json: unsupported value: NaN\n: exit status 1"

This behavior is also seen when testing with
Holiday List_2024.pdf

Another instance were I see this problem happen many times:

Chat with Tildy.
Using "Add knowledge" add holiday pdf file.
Use "Save Assistant As" option to create a new assistant.
Chat with the assistant and ask information about holiday pdf file.

This results in "Errors relating to json: unsupported value: NaN\n: exit status 1 " error most the times.
But I have gotten responses successfully when taking to this assistant some times.

I am also seeing this issue when testing with
Insurance_Handbook_20103.pdf

This should be fixed by https://github.com/gptscript-ai/knowledge/pull/109 which is released in https://github.com/gptscript-ai/knowledge/releases/tag/v0.4.14-rc.1
You should be able to swap out the used knowledge binary by setting the KNOWLEDGE_BIN environment variable to the path to the new knowledge executable when starting the desktop app.

Tested with build - a66cd29e
This issue is not seen anymore.


Demo repo please

Hi, please add a panel-demo repo so that we can see how your demo site is made.
Also it will be great if you can add a gallery module as well.
Thank you in advance.

https://github.com/laravelpanel/demo for demo and  for gallery https://github.com/serverfire/gallery


LocateDependenciesTask can be up to date.

Build failed https://gradle.com/s/73ta2chcuieyu

Build failed https://gradle.com/s/x4hb3zbkpnsqk


Docusaurus 2.0 NDS-336

This bumps Docusaurus to v2.0, which was recently released. I also updated things to make better use of Docusaurus features:

Swizzled the ColorModeToggle to replace it with the NDS Switch, which means we don't need to swizzle the whole NavBar and should reduce maintenance. (baec227619f7b71ac02316cb2f12b7311a215775)
Swizzled the Admonition component to replace it with the NDS Callout, which allows us to use the ::: syntax in markdown to set our callouts. This should fix NDS-273 (1877d403a3948456b7f1c283b373899269c22ac0, 1877d403a3948456b7f1c283b373899269c22ac0)
Various additional little tweaks to make the Infima tokens use NDS tokens.
The "dos & don'ts" callouts were inconsistent in their plural/singular forms so I made them all singular. (16c067f9c55ee07e545e868a09182615a67c8617)

Amplify didn't have the latest Node 16 version, so I had to manually update it in the Amplify admin. It's building correctly now, so I'm going to merge this.


copy cli config definition window visibility defaults

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
fixes yarn tauri dev not setting window.visible to true by default

It defaults to true as-is for me.

i think it doesn't default to true on tauri dev since it rewrites the config.

this is specifically for the tauri dev command since this affects the cli's config definition only.  only affects configs that don't set visibility or decorations


podmanager: Ensure cgroups mounts are private

On some platforms, such as Ubuntu Xenial, the cgroups location defaults
to being shared. The mapping in of cgroups can then cause an exponential
growth of the mountinfo file which causes large CPU spikes parsing it on
container spin up.
This ensures the cgroups mounts that are mapped in are private and do
not propagate. This is controlled just at the launching of the stager by
the podmanager.
FIY @schancel @mbhinder

LGTM

D'oh.  I wish I would have considered this.  Oleg mentioned this DDoS awhile back.

Yeah... I strace'd it and saw a constant read from a single fd, so quickly looked it up in /proc and it was reading its mountinfo. Sure enough, within a few launches, mountinfo had over a million entries.


feature: add envs for pouch exec

Ⅰ. Describe what this PR did
Provide new Environments for pouch exec
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please also the corresponding cli integration test for this pull request. Thanks a lot. @knightXun

@allencloud I will add tests for this.

@pouchrobot test

LGTM, and would you like to sign the cla again? Since Alibaba org has updated something, then everyone of us needs to sign it again. @knightXun


Transfer rate:  0.00 MB/s

Hi, and thanks for this cool project. I have build the hardware and programmed the MCU und FPGA. Everything worked fine until the test procedure.
I got 0.00 MB/s at the transfer speed, and i have no clue what i can do to fix that!
Could you give me a hint?

Most likely you have some issues with the USB PHY IC and it is not producing 60 MHz clock. Up to the test point it would matter, but the test part needs it.
So, measure that clock and the input 26 MHz clock.

I faced similar issue: https://www.youtube.com/watch?v=3Boyl4IId2g
I checked the 26MHz clock and it is fine, so probably is some issue on USB3343, right?

Do you see 60 MHz as well? If not, then yes, something is wrong with the USB PHY. It should start producing 60 MHz clock for the FPGA. Without it programming will work, but no data will be sent.

And the most obvious thing to check that is it soldered in a correct orientation.

I think the orientation is correct because I looked the datasheet before soldering, see attached image.
The 60MHz should be on pin 2 of USB3343, correct? So, that could be the issue, there is nothing on this pin!

Yes, this looks like a correct orientation. And yes, pin 2 should have 60 MHz.
Another thing to double check is that you have  8.06K (R11) resistor mounted and the value is correct. Apparently, it does not produce any clock when this resistor is not mounted.

Yes, it is mounted, but I checked on multimeter and its value is 8.25K. Is it inside the tolerance range?

It needs to be 8.06 +/- 1%. So, it should be in the range 7.98 - 8.14 kOhm. I don't know what will happen if it is outside of that range. I don't think it is that sensitive for the purpose of the clock generation, but who knows.

I have had this same issue and wanted to share here in case others happen to find this thread when searching on how to solve the problem.
In mi case it was bad soldering on the USB3343. I guess it can be a bit tricky to solder by hand...
I fixed it just by de-soldering and re-soldering the USB3343. First remove all the solder in the IC and the board. Prime the ground pad with very little solder, using the heat gun solder the IC as usual, press down with a pair of tweezers on the IC until everything cools down to make sure it properly seats and its not floating on the solder pad. Next using a good amount of flux solder the external pins, I would recommend drag soldering technique there.

@jpenalbae Thank you Jaime, that was my suspicious too! I didn't have the change to resolder it yet. How are you testing it? Did you connect some device to J2 (USB-C) connector during the speed test?

Speed test only needs host connection, target can be disconnected. The only reason USB3343 is necessary for the test to work is that it provides 60 MHz clock that is needed for everything to work.

@jpenalbae Thank you Jaime, that was my suspicious too! I didn't have the change to resolder it yet. How are you testing it? Did you connect some device to J2 (USB-C) connector during the speed test?

I would try to re-soldering the same USB3343 you have in the board right now, that is what I did. If that didn't work I was planing on trying with a new one, but it wasn't necessary in my case.
For testing, as have been already mentioned around here, make sure 26MHz clock in, 60MHz clock out, download the datasheet for USB3343 and double check everything is in place. For my case it was pretty obvious cause besides the 60MHz clock, the integrated 1.8v LDO (pin 23) wasn't working either, so it was a bad USB3343 or a bad solder job.

I have the same issue, as well; however, I am able to measure 60MHz at the FPGA.  Any pointers on where to look next?

Was FPGA firmware upload correctly? Is the FPGA type (including speed grade) correct?

Was FPGA firmware upload correctly? Is the FPGA type (including speed grade) correct?

Flashing the FPGA appears to succeed:
$ ./usb_sniffer_linux --fpga-flash usb_sniffer_impl.jed
Programming FPGA flash
Erasing flash
Programming configuration data .....
Verifying configuration data
Programming and verifying Feature Row
Programming and verifying FEABITS
Exiting programming mode
...done

I purchased a LCMXO2-2000HC-5TG100C from DigiKey, which I believe matches what is specified in the BOM.

Yes, this all this is correct. The other thing to check is that you have 30 MHz on the IFCLK (FPGA pin 88). This clock is generated by the MCU and FPGA is using it to send the data.
And check CTRL_CLK/CTRL_DATA. Those pins are used to enable the test mode. There is no persistent clock on them, but they should some activity when the test mode is started.
All those signals go from the MCU to the FPGA, so check on the FPGA side, so you verify soldering on both sides at the same time.

Thanks! I found a few loose pins on the MCU. After fixing those, I'm seeing 40-50MB/s as expected.

Hi, and thanks for this cool project. I have build the hardware and programmed the MCU und FPGA. Everything worked fine until the test procedure. I got 0.00 MB/s at the transfer speed, and i have no clue what i can do to fix that! Could you give me a hint?

Okay, after a long time i came back to this project. And i found a heavy bug. I had to rotate the USB3343 about 90 degrees. Uhaa.. hopefully the chip was not damaged!
But after a short test i got the desired 40-50MB/s. That's fine!
Now i got some other errors with the wireshark capture. I would like to test some sticks laying around, but all of them are not really send the right datas. I got "USB PHY Error". Could it be, that the USB3343 is faulty because of my lack of competence? :-/ Maybe i should order a new one, and try it out.

Did you configure the correct capture speed? If yes, then I would assume the PHY is damaged and replace it before spending more time investigating that.


grammar fix "sended" -> "sent"

damn, it hurts
by the way, any plans to update the library?

Sorry, my English is not the best :P
I don't really have plans to update it, right now I'm using Elixir and I don't have many time, sorry :(

what a pity, it used to be a cool  wrapper for old good telegram-bot library; but today dependencies seems to be broken


Update restrictions to restrict trigger only

#267 introduces restrictions that went beyond restricting the calling IP address and also restricted viewing run history
this reverts part of #267 to only restrict calling IP

Deploy Link

There're no version update. Are we good with that?

I think so, I didn't for the last one, as the module itself is not changed


The One Probe Showing Error on Brewing Still

This may be to do with TOP Addons, but looking at the Brewing Still shows an error in TOP:

does this output anything in the console because it doesn't for me and i'm not sure where to begin with this


Any event trigger upon midRoll ad playback?

Currently my analytics are getting screwed as I am unable to track when a midRoll ad is played. The vastLoadedCallback function is being called only during preRoll.

Hi @gouthamkanade, can you share more on how are you tracking the midRoll? Maybe using vastLoadedCallback is not the right approach.
Currently all vast is loaded when the player starts playing a video, that is why you are seeing this function only being called for preRoll. We do have a PR open that will solve this issue https://github.com/fluid-player/fluid-player/pull/721.

@gouthamkanade The PR previously mentioned should have corrected the issue, can you confirm this?

Closing due to inactivity


refactor: add a lean version of the C# code

@arthur404dev  hello man, do you have some news about PR?
Thanks

Thanks @arthur404dev  0/


How to use TrySkipTurn / UndoSkipTurn

I have attempted to use them to count legal turns / threats by the opponent like:
        if (board.TrySkipTurn()){
            // their mobility/threats will only be deducted when we're not in
            // check, but when we are our mobility is mostly decreased anyway
            score -= board.GetLegalMoves().Length + board.GetLegalMoves(true).Length;
            board.UndoSkipTurn();
        }

this mostly seems to work except once in a while my bot now attempts to make an illegal move elsewhere in the code.
Am I using it wrong?
I'll be testing my bot without this fragment now, to find out if it was in fact this code that caused to bot to get confused.

My bot has just made an illegal move with this fragment commented out, I guess this wasn't the culprit.