Code field doesn't escape HTML.

Bug Report
The code field doesn't escape HTML, instead treating it as a HTML to be rendered.
Expected Behavior
HTML is escaped.
Current Behavior
HTML is not escaped.

Possible Solution
I believe this is due to a 'custom' React state addition, normally React would escape a state value. https://github.com/payloadcms/payload/blob/28e39280943154fd98b3ca9c40a0c26a9488d7cc/src/admin/components/forms/field-types/Code/Code.tsx#L53
Detailed Description
Latest version. No addons / plugins. Standard.

Hey @virtualSpoof — ok, I checked into this. I believe that what is necessary here is to extend the code field with HTML support (right now, we only support JS, but we could easily support HTML as well with a quick PR).
https://payloadcms.com/docs/fields/code#admin-config
Because this is more of a feature request than a bug, as we know that we only support JS at this time, I'm gonna move this to a discussion.
PS - I know I personally asked you to create this issue 👼 .  Upon looking into the underlying causes, I realized this is less of a bug and more of a feature. Do you think you'd be up for taking a crack at a PR here? To add HTML support to the Code field?


Configuration Driven Authorization for FOM

Building on work that the command line tool used for admin of fom users / roles: https://github.com/bcgov/nr_fsa2/issues/26
This ticket will read an Excel SS in onedrive and sync configurations defined in the SS with keycloak.  The job will run in openshfit as a cronjob.  This will enable Julius to make changes to the SS and have them reflected in Keycloak next time the cronjob runs.

Closed in favor of skipping to a GUI for Julius


Cannot complete installing

Hi guys,
I followed all the steps to install Merlin presented here:
http://jrmeyer.github.io/merlin/2017/02/14/Installing-Merlin.html
but instead of using the demo I used the full_voice. Everything looks to be fine after running but I don't have wav files how I think it was supposed to have after training the net.
When I look closely before step 5 I have this error:
IOError: [Errno 2] No such file or directory: '/merlin/egs/stl_arctic/s1/experiments/stl_arctic_full/acoustic_model/nnets_model/DNN_TANH_TANH_TANH_TANH_TANH_TANH_LINEAR_mgc_lf0_vuv_bap_0_6_1024_1024_1024_1024_1024_1024_425.187.train.1000.0.002000.rnn.model'
When I look at that directory there is no such file and I suppose it should have been created...
I cannot solve this problem.
Could you assist me?
Thanks,
Catarina

Check if your training went well -- at least for more than 5 epochs.

Thanks for your answer. Actually, I think it is that. Both demo and full_voice run cannot have more than 2 epoch followed by MemoryError. Can it be lack of capacity of my computer? Or can it be something else?
I'll put here some prints.

Reduce the buffer size in "providers.py"


Jwilaby/qna api integration

This PR adds the ability to list knowledgebases from QnA via Azure

Pull Request Test Coverage Report for Build 172

109 of 121 (90.08%)  changed or added relevant lines in 8 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.8%) to 47.197%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/app/main/src/services/luisApiService.ts
3
4
75.0%

packages/app/client/src/ui/dialogs/getStartedWithCSDialog/getStartedWithCSDialog.tsx
15
16
93.75%

packages/app/client/src/ui/dialogs/getStartedWithCSDialog/getStartedWithCSDialogContainer.ts
5
6
83.33%

packages/app/client/src/ui/shell/explorer/servicesExplorer/connectedServiceEditor/connectedServiceEditor.tsx
9
11
81.82%

packages/app/client/src/data/sagas/servicesExplorerSagas.ts
19
22
86.36%

packages/app/main/src/services/qnaApiService.ts
52
56
92.86%

Files with Coverage Reduction
New Missed Lines
%

packages/app/client/src/data/sagas/servicesExplorerSagas.ts
2
65.25%

Totals

Change from base Build 168:
0.8%

Covered Lines:
2984

Relevant Lines:
5305

💛  - Coveralls


can't get response in fetch api

i have a function using fetch api to call my remoting server through this module,but i can't get response in my function,can anyone help me?

The Fetch API is not part of the request module :(


Video Encryption

I am trying to use this for my project and i have gone through the code and i still cant find where the video calling or audio call is encrypted in the code. This is the main part of my project (the encryption) and i was wondering if you could please point it out or explain how it works and if any keys are being exchanged

There are some links about the security in the WebRTC:
https://bloggeek.me/is-webrtc-safe/
https://webrtc-security.github.io/
Additionally, our SDK has the method replaceMediaStream(MediaStream) which provides you the possibility to set your own media stream after your transformations.
Other helpful links:
https://webrtc.github.io/samples/
https://connectycube.com/2020/04/23/the-current-state-of-e2ee-in-webrtc/


self.base_strategy has no attribute 'rgb_inputs'

Hi, thanks for your valuable work.
When I try to do the simple test as instructionhttps://github.com/MeteorsHub/RelayLearning?tab=readme-ov-file#simple-test, something went wrong in self.base_strategy, I found it is NoneType. I did not change anything in config/relay_test.yaml except for the root_path. The error is as follows:
Traceback (most recent call last):
  File "/home/lzy/mancy/RelayLearning/predict_test.py", line 223, in <module>
    predict_test()
  File "/home/lzy/mancy/RelayLearning/predict_test.py", line 82, in predict_test
    model = get_model(task_conf['model']['name'], **task_conf['model']['kwargs'])
  File "/home/lzy/mancy/RelayLearning/models/__init__.py", line 34, in get_model
    return all_models[name](**kwargs)
  File "/home/lzy/mancy/RelayLearning/models/doublegan.py", line 88, in __init__
    self.replay_post_aug_pipe = get_data_aug_pipe(rgb=self.base_strategy.rgb_inputs) if replay_post_aug else None
AttributeError: 'NoneType' object has no attribute 'rgb_inputs'

It could be a bug about using before defining. I change the rgb=self.base_strategy.rgb_inputs to rgb=True directly. It should solve the problem.


Error reporting / alert

better handle error in the whole app (specially in DropDownPanel)

Error reporting has been improved. Closing it.


Plot: Legend improvements

Closes #409.
https://user-images.githubusercontent.com/1352472/118518207-2e6ed880-b738-11eb-8335-8c4526c5fc20.mp4
This is based on the work in #363, so we should wait for that to be merged first. Sorry about the large PRs.

Thanks for the review! I made the changes.
I also tried to give the legend a background, but couldn't find a way to do it without doing all the size calculation myself. Do you happen to know a way to do this?

Thanks for the review! I made the changes.
I also tried to give the legend a background, but couldn't find a way to do it without doing all the size calculation myself. Do you happen to know a way to do this?

Sure, just add it to a Frame:
let frame = egui::Frame { … };
frame.show(ui, |ui| { /* legend ui */ });

Sure, just add it to a Frame

I tried that and it looks great when the legend is on the top left:

But it doesn't work for all the other cases:

Is there a way to work around this issue? Maybe something like choosing an anchor point for the frame?

Sure, just add it to a Frame

I tried that and it looks great when the legend is on the top left:
…
Is there a way to work around this issue? Maybe something like choosing an anchor point for the frame?

I think this is a bug in Frame. Does this fix it:
diff --git a/egui/src/containers/frame.rs b/egui/src/containers/frame.rs
index 36b272c..6b41e31 100644
--- a/egui/src/containers/frame.rs
+++ b/egui/src/containers/frame.rs
@@ -175,10 +175,7 @@ impl Frame {

 impl Prepared {
     pub fn outer_rect(&self) -> Rect {
-        Rect::from_min_max(
-            self.outer_rect_bounds.min,
-            self.content_ui.min_rect().max + self.frame.margin,
-        )
+        self.content_ui.min_rect().expand2(self.frame.margin)
     }

     pub fn end(self, ui: &mut Ui) -> Response {

?
(that fix may introduce other problems, so please don't commit it)

I think we should get this in even without the legend background

That was it! Works in all corners now. I'll add that in the next PR then. Thanks!


rust-analyzer-unwrapped: 2022-03-07 -> 2022-03-28

oh no, the build fails, after all :cry: lemme close this merge request to avoid confusion and re-open it after fixing the issue

Okie, newest rust-analyzer requires rustc 1.59.0, so lemme start with that


Add a method to strike lightning in the world

Lightning has traditionally been an entity, but it would probably be better if there was a simple method to strike lightning in a world. Something like (*world.World).StrikeLightning(pos) might work.

Pushing this back.

Postponing to v0.3.0.

Random thoughts: Making this an entity might be easier after all. It needs to have access to specific blocks (fire) and needs to damage and set on fire entities around it. This would come with quite some hacks if added to the world package.


Question

Does senders need to fully unblind to allow a collaborative transaction?

I'm sorry can you provide more context? What do you mean unblind?

Just seen this - I meant this in the context of liquid transactions. I think I may have misunderstood the project

@real-or-random I think you mean Liquid rather than Lightning.

Ohh, of course. Brain fart. ;)

Thanks! Do you know if anyone is building anything like this?

There's https://github.com/decred/cspp but it's not in the context of Confidential Transactions.


Sometimes wallet does not detect events (on-chain transaction)

Closing in favor of https://github.com/statechannels/monorepo/issues/1995.


Deduce taxonomy version from individual filings

XBRL filings include the taxonomy used in the filing. This should be dediced automatically, rather than passing a taxonomy directly.

Fixed.


Add howto rename master to main branches

@thomasleese apart from updating Pact tests and other Jenkins references, I got this to work for short-url-manager by following these steps myself. Probably not worth asking others if they can follow along then :)
Thanks for your review!

Looks good, I just had a final thought, could we link to https://trello.com/c/xVhGGzOE/225-change-default-branch-to-main somewhere to remind people to check this to see which apps need switching, and to update it once they have switched an app over.

@thomasleese good idea - done.


Added accessibility support

Added accessibility support so that it's visible while using iOS 9 new feature: UI Recording

I went to WWDC 2015 and I was doing UI-testing. Due to accessible identifier is not set up yet, the hud not able to be UI-recorded automatically by Xcode. After adding those lines of code, it is ui-testable which is great. So, please consider to merge it. Thanks. @samvermette @honkmaster

Done, thanks very much.

You're welcomed. This is an amazing tool and ui-testable is really important for Xcode 7


Fix the minor doc test due to parameter initialization

Hi @stevenbird
This is the fix for https://nltk.ci.cloudbees.com/job/nltk/lastCompletedBuild/TOXENV=py32-jenkins,jdk=jdk8latestOnlineInstall/testReport/nltk.parse/transitionparser/demo/

I took out the unnecessary print in https://github.com/nltk/nltk/commit/0d44cec8565bbd2ae4d59302334e667bdfa4c667


Added Searching Functionality to schedule page

@mariobehling @aayusharora please review.

Current coverage is 42.58% (diff: 100%)

Merging #824 into development will not change coverage

@@           development       #824   diff @@
=============================================
  Files                6          6
  Lines              735        735
  Methods             74         74
  Messages             0          0
  Branches           118        118
=============================================
  Hits               313        313
  Misses             422        422
  Partials             0          0

Powered by Codecov. Last update 9df68cd...ae1d34d

Thanks. Please follow up about implementing SASS and hound-ci issues. Good job!


displ_be: Introduce config parameters to control zcopy functions

This change introdues the following configuration parameters to the
code:
WITH_DRM_ZCOPY - enable drm zero-copy (ON by default);
WTIH_KMS_ZCOPY - enable kms zero-copy (ON by default);
WTIH_DMABUF_ZCOPY - enable dmabuf zero-copy (ON by default).
The purpose of these parameters is to give ability to disable certain
zero-copy functions because there is a posibility to receive wrong
device_name from wayland during zero-copy initialization.
This problem was originally met on imx8qm board, where libEGL.so,
provided as binary by imx-gpu-viv recipe has constant device_name, set
as /dev/dri/card0. But linux kernel creates display subsystem as
/dev/dri/card1 because it has minor 1.
Same implementation was found in Renesas proprietary graphics module -
device name was set to /dev/dri/card0. But error doesn't appear because
display-subsystem has minor 0, so it has name card0.
The solution is to ignore message from Wayland with incorrect name and
use alternative mechanism, which is dmabuf in case of imx8qm.
Signed-off-by: Oleksii Moisieiev<EMAIL_ADDRESS>

This MR is not relevant anymore. Because provided patch was merged to displ_be repo so there is no need to apply it here.


Change "fields" to "columns" in mapping examples

cf https://github.com/omniscale/imposm3/commit/79754e1f8a226e2909a8d9ba6405bd80eeb35f38
:)

👍


V11 compatibility

effect.label has been renamed to effect.name
(So keep it V10 compatible)
   name : effect.label ?? effect.name

Warnings fixed in 1.3.2


Cannot publish package dependent on new version of another package in same changeset

Affected Packages
https://github.com/BitskiCo/bitski-js

packages/browser
packages/provider

Problem
When going to publish a changeset, my browser package fails on npm install because it depends on the latest version of my provider package which is part of the same changeset. Can I somehow specify an order in which the package gets published, or are there any other native ways to get this playing nicely?
Proposed solution
???

@Andarist - you can see what I'm referring to here: https://github.com/BitskiCo/bitski-js/runs/6863876438?check_suite_focus=true
I don't particularly think this is an issue with changesets - more so wondering if there are capabilities and/or additional knowledge I'm missing of getting interdependent packages to play nicely with each other.
Essentially, if we have to update both packages, it currently means we can't include them both in the same changeset and would have to do it one at a time - provider first, and then browser since browser depends on the updated version of provider. I was trying to see if I can somehow publish them together despite this restriction.
The furthest I could ween from the docs is wrt linked packages -- but it was not immediately clear to me whether this addressed my problem: https://github.com/changesets/changesets/blob/main/docs/linked-packages.md

Essentially, if we have to update both packages, it currently means we can't include them both in the same changeset and would have to do it one at a time - provider first, and then browser since browser depends on the updated version of provider. I was trying to see if I can somehow publish them together despite this restriction.

Yes, publishing them together is the easiest - publishing them in a "waterfall" manner would be quite complex to manage.

The furthest I could ween from the docs is wrt linked packages -- but it was not immediately clear to me whether this addressed my problem: https://github.com/changesets/changesets/blob/main/docs/linked-packages.md

It doesn't - it's just a way to change what the computed versions end up being but with this option, you still version the whole monorepo "at once".

If you could prepare a very barebones repo based on npm workspaces, with some dummy public packages in it - then it would allow me to investigate this sooner than later.

Seeing this issue in blockprotocol/blockprotocol.

Version Pacakges PR
Release action – it succeeded only after Attempt 4 and failed three times initially

Our package.jsons have the prepare script, which recreate the contents of corresponding dist folders. Since all changed packages are published in parallel, multiple prepare scripts run simultaneously and some packages fail to build because dist folders in upstream packages are being recreated.
This should be fixable if packages are published sequentially (ideally, following the dependency tree). AFAIU, Lerna does it this way, at least I don’t remember this issue with it in projects I’ve worked on.
In the meantime, we have to re-run the Release script till it settles. @Andarist what else would you recommend?

We could accept a PR that would publish packages in a topological order. If you don't intend to prepare a PR for this then I would recommend you to remove your prepare scripts from packages and just run a single monorepo-wide build before you call changeset publish.

Going to close this as it ended up being an issue with our prepack scripts as well. Removing that enabled the topological order in turborepo with changeset.

@imthinhvu what is your recipe for publishing packages in topological order with Turborepo? I'd love to borrow that.

@imthinhvu what is your recipe for publishing packages in topological order with Turborepo? I'd love to borrow that.

There's nothing magical we're doing - turborepo should support topological build: https://turborepo.org/docs/core-concepts/pipelines#topological-dependency
What we've added is a release npm script that does turbo run build && changeset publish


Powertoys Run ValueGenerator docs

This PR is in reference to https://github.com/microsoft/PowerToys/pull/26097.

Thank you! :)

@alvinashcraft Hey, it seems I made a mistake in the formatting in this section:
https://learn.microsoft.com/en-us/windows/powertoys/run#value-generator-plugin

@IHorvalds  Good catch. I can go try to fix it up quickly.

It looks fine in my VS Code preview pane too. I'm going to try adding a line break after "Examples:" and push to my branch. I can preview on the internal docs site with the windows-dev-docs-pr repo.

It looks good on the review site in my branch after adding some line breaks. I'll get a PR merged ASAP.

That's awesome! Thank you!

You're welcome. The table should be rendering on the live site now. 😊


blur

I'm submitting a bug report
Please tell us about your environment:

Operating System:
Windows 10

Node Version:
68.11.3

NPM Version:
6.6.0

Browser:
Firefox 64.02

Language:
ESNext

Current behavior:
Create a value converter 👍
export class AddStringValueConverter {
  toView(value, string) {
  	let s = ""+value+string;
  	console.log("add-string :",value+"=>"+""+s);
	  return s;
  }
}

Create a template 👍
<template>
<input  value.bind="test  | addString : '.00' & updateTrigger:'blur'" />
<input >
</template>

now input "2" un first field, then puche "tab"  (it display "2.00")
then push "Shift+Tab", input "2"
then it display "2" and not "2.00" !!

What is the expected behavior?
showing "2.00" and not "2"

Sorry to answer so late. ;-)
In your  https://codesandbox.io/s/x761qq5p3q under chromium the bug is present.
Chromium : Version 70.0.3538.102 (Build officiel) (64 bits) Windows 10 Pro.
To get the bog ,input keys like this :
input "45"
input key TAB   => here it become "45.00", and cursor is on second field
input keys SHIFT+TAB  => cursor go to first fiels and "45" is selected
input "45"
input key TAB    => cursor go to second field, and "45" was not converted in "45.00"

Closing per information on #739


Problem with ALVideoPlayer

Hello.
I'm having a problem when trying to stream videos in ALVideoPlayer. These white letters appear on the screen instead of the video. I already imported all the .jar files, I already made the correction with Update.bat but I still have the same error. What to do now ?
I'm using Delphi 10.2 Sydney.

if you apply patchs, this issue will dispear.

Hello, did the bug still present with the actual version of alcinoe that use the very last version of exoplayer jar ?

Without any answer I consider that this bug is now corrected in the very last version of Alcinoe. If not please feel free to reopen it

I already applied all the patches and still this error persists. Some other solution.

does the same error appear in the demo https://github.com/MagicFoundation/Alcinoe/tree/master/Demos/ALFmxControls/_Source ?


Fix quat slerp so it takes shortest path.

The last codeline in our slerp implementation was checking if d < 0, but it was always false.
Verified with a unit test and with the gltf file that's in the bug description.
Fixes #1716.
This regressed in c65e561c59c70fdf2f960823ea15ecef5b5b1bfe.

(The Linux failure is unrelated and Ben already has a fix for it.)


embed::bit_bang_spi

Using 2x embed::output_pin, one of them representing a clock and another representing the output channel and 1x embed::input_pin representing an input channel construct a gpio driven bit banged embed::spi driver.

I can take this one as a way to get used to the code

The tests for this driver will depend on the following spys/mocks:

https://github.com/SJSU-Dev2/libembeddedhal/issues/2
https://github.com/SJSU-Dev2/libembeddedhal/issues/4

.take


MB 6520 Fix Crash For Trying To Validate Undefined TACs

Description
Verify that the tac value exists before trying to get the length.
Reviewer Notes
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make client_run
make server run

Submit a move locally using the customer UI.
I did this by going to local sign in, using the<EMAIL_ADDRESS>(milmove) user, and submitting a move.
Go to the office UI, sign in as someone who can edit the orders.
Select the order that was just submitted.
Edit the order.
Change the department indicator, or click in the TAC field.
Wait for a second or so.
See the crash!
With these changes, it should no longer crash.
Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Request review from a member of a different team.

References

Jira story for this change

Screenshots
If this PR makes visible UI changes, an image of the finished UI can help reviewers and casual
observers understand the context of the changes. A before image is optional and
can be included at the submitter's discretion.
Consider using an animated image to show an entire workflow instead of using multiple images. You may want to use GIPHY CAPTURE for this! 📸
Please frame screenshots to show enough useful context but also highlight the affected regions.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 8e0034e263bc989c598cb97a82a5d895f08cc88d


add images and bintray warning

Add images to install links
Add warning about bintray 404 if you aren't logged in

Tests are failing on the URLs of the icons I added but this is a false positive, pls ignore

When I copy the URLs from the link checker it looks like a genuine 404:

And images don't render on the preview site:

So is it a false positive because the images are added in a separate PR? Right now it's a true positive...

Turns out they got gitignored because the folder is named install, force added them


Implement multi-threaded decompression

Feature request ^

Hi! This should actually be supported already:
https://github.com/addaleax/lzma-native#multi-threaded-encoding

I tried it with threads: 0, but noticed no performance increase. Also the doc says it’s only supported for encoding, which sounds like it’s not supported for decoding?

I tried it with threads: 0, but noticed no performance increase.

Can you share the example with which you tried? If it’s a small-ish file you’re compressing, that might be the case because multi-threading comes with a certain overhead and liblzma only actually makes use of multi-threading for larger chunk sizes.

Also the doc says it’s only supported for encoding, which sounds like it’s not supported for decoding?

Yes, that’s correct. Decoding is much faster than encoding anyway, though.

I am mainly interested in decompression/decoding at the moment, of a 2.6GB file, which takes about 6 minutes on my laptop. Looking at my cpu usage, it is only utilizing one core, whereas I have 8 available. Therefore I expect decompression to take roughly 45 seconds if it is perfectly spread across cores/threads.
Here is some prototype code, taken from https://github.com/nano-wallet-company/nano-wallet-desktop
const lzma = require('lzma-native');
const tar = require('tar-fs');
const tarStream = require('tar-stream');
const progressStream = require('progress-stream');
const Promise = require('bluebird');
const pump = Promise.promisify(require('pump'));
const fs = Promise.promisifyAll(require('graceful-fs'), {
  filter(name) {
    return ['stat', 'readFile'].includes(name);
  },
});

const createProgressStream = (length, onProgress) => {
  const progress = progressStream({ length, time: 250 });
  progress.on('progress', ({ percentage = 0 }) => onProgress(percentage / 100));
  return progress;
};

const extractAsset = async (savePath, extractDir, onProgress) => {
  const extract = tarStream.extract();
  const { size } = await fs.statAsync(savePath);
  const dec = lzma.createDecompressor({ threads: 0 })
  return pump(
    fs.createReadStream(savePath),
    createProgressStream(size, onProgress),
    dec,
    tar.extract(extractDir, {
      fs,
      extract,
      fmode: 0o600,
      dmode: 0o700,
    }),
  );
};

console.log('Extracting')

extractAsset('./data.tar.xz', '.', function() {
  console.log(arguments)
})

Where data.tar.xz comes from https://dkl5m4kebds7n.cloudfront.net/data.tar.xz, a database file, essentially.

I agree that this would be great to have, but the underlying library doesn’t seem to support this yet. :confused:

Got it, thanks.


Unable to run any pytorch model - Tensor type error

Hello,
I have been trying to correctly install TargetCall, but I am unable to run any model regardless of how I install. I even went as far as matching the conda version (4.7.12) on a Linux machine with a GPU. I still get the same error on other machines/other conda versions:

bonito basecaller ../bonito/models/TINYX011/ ../sample_data/fast5// --batchsize 3200 --modeltype tinynoskipx011 > ../sample_data//output.fastq
> loading model ../bonito/models/TINYX011/
> outputting unaligned fastq
> calling: 0 reads [00:00, ? reads/s]Exception in thread Thread-1:
Traceback (most recent call last):
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/threading.py", line 932, in _bootstrap_inner
    self.run()
  File "/home/vshiv/software/TargetCall/bonito/multiprocessing.py", line 183, in run
    for k, v in self.iterator:
  File "/home/vshiv/software/TargetCall/bonito/ctc/basecall.py", line 24, in <genexpr>
    scores = (
  File "/home/vshiv/software/TargetCall/bonito/util.py", line 222, in <genexpr>
    return (
  File "/home/vshiv/software/TargetCall/bonito/util.py", line 217, in <genexpr>
    batches = (
  File "/home/vshiv/software/TargetCall/bonito/ctc/basecall.py", line 22, in <genexpr>
    (k, compute_scores(model, v)) for k, v in batchify(chunks, batchsize)
  File "/home/vshiv/software/TargetCall/bonito/ctc/basecall.py", line 39, in compute_scores
    probs = permute(model(chunks), 'TNC', 'NTC')
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/vshiv/software/TargetCall/bonito/ctc/modeltinynoskipx011.py", line 33, in forward
    encoded = self.encoder(x)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/vshiv/software/TargetCall/bonito/ctc/modeltinynoskipx011.py", line 205, in forward
    return self.encoder(x)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/container.py", line 141, in forward
    input = module(input)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/vshiv/software/TargetCall/bonito/ctc/modeltinynoskipx011.py", line 385, in forward
    _x = layer(_x)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/vshiv/software/TargetCall/bonito/ctc/modeltinynoskipx011.py", line 241, in forward
    x = self.conv(x)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 301, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/vshiv/miniconda3/envs/targetcall/envs/target/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 297, in _conv_forward
    return F.conv1d(input, weight, bias, self.stride,
RuntimeError: Input type (torch.cuda.HalfTensor) and weight type (torch.cuda.FloatTensor) should be the same

I get the same error for any model I try. For reference, I am able to run Guppy and default Bonito models on this machine with this GPU. It seems like an issue with the Tensor type for one of the layers. Please let me know if there is any other info you need to resolve this issue!
I am using pytorch 1.10.2 (installed using the provided requirements.txt environment file)

Additionally, when the RuntimeError occurs, likely due to threading, the command hangs and the only way to cancel is ctrl-z and manually killing the process.

I added in a line for each model to autocast inputs, which seems to resolve the tensor type error. See PR #2.

Hello,
Thank you for looking into this! The latest commit with the higher-level autocast fix works for me now. I assume there is some system specific type issue in pytorch that was causing an issue, which casting fixes. I appreciate your following up on this and implementing the bug fix!
Best,
Vikram

Thanks for your response.
Best,
Banu


Can't use args[...] in place of the Anilist ID

Description
Use of arguments is currently not possible with default code.
Steps to Reproduce

Load the module into your project
Attempt to use the example code like this

const anilist = require('anilist-node');
const Anilist = new anilist();

Anilist.media.anime(**args[0]**).then(data => {
    console.log(data);
});

Receive the "Term provided is not a number!" error

Environment
Node.js Version: 14.15.4
AniList-Node Version: 1.7.0
Screenshots
Additional context
Commented out line 25 in this file
https://github.com/Butterstroke/AniList-Node/blob/master/lib/media.js
Arguments were able to pass through, and the correct data was grabbed.
While the addition of this line helps make sure the correct string is passed, it checks the code before the actual argument and kills the process. Might want to add some exception for args[...]

Do you know what you were passing within the args? While I think adding another exception is a good idea, I don't want to blindly add in an array check and hope the arguments within it are fine.

Do you know what you were passing within the args? While I think adding another exception is a good idea, I don't want to blindly add in an array check and hope the arguments within it are fine.

Thats true also.
The args I passed were always the anime ID. It would work if you directly put the ID into the code, and then wouldn't when you put args[...] in the code, and tried to pass an ID.
Just wanted to bring attention to it. I'm really knowledgeable enough to give code advice so up to your discretion whether you leave it as is or not.

The args I passed were always the anime ID. It would work if you directly put the ID into the code, and then wouldn't when you put args[...] in the code, and tried to pass an ID.

...Wait a moment. Just to make sure I read this right. Are you passing an item within an array, not the array itself, and it is failing? I might have misread this initially thinking it was the whole array. If it is just an item within an array, check to make sure the item is a number type and not a string. You might have to convert it into a number beforehand then.

Just wanted to bring attention to it. I'm not really knowledgeable enough to give code advice so up to your discretion whether you leave it as is or not.

That's fine. I don't mind the advice. I would not have made this public if I didn't want to hear from others about code. 😄

...Wait a moment. Just to make sure I read this right. Are you passing an item within an array, not the array itself, and it is failing? I might have misread this initially thinking it was the whole array. If it is just an item within an array, check to make sure the item is a number type and not a string. You might have to convert it into a number beforehand then.

Not from an array. Just a random value that the user defines, which, elsewhere in my version of the code, is required to be a string of numbers. For example, args[0], which is what I have in my code right now, just passes the first string of numbers that the user specifies to the function.
Anilist.media.anime(args[0])

Not from an array. Just a random value that the user defines, which, elsewhere in my version of the code, is required to be a string of numbers. For example, args[0], which is what I have in my code right now, just passes the first string of numbers that the user specifies to the function.
Anilist.media.anime(args[0])

Ah, I get it now. I was misreading this. You just have to convert the variable type for args[0] into a number and it should work. So something like Anilist.media.anime(Number(args[0])) is fine and shouldn't affect the value outside of the anime method.


Name of Dynamic Block Inserts

Hey there,
I was wondering if there is a way to get the name of a dynamic block from an inserted instance?
Currently I am getting objects with values such as these:
{
    type: 'INSERT',
    handle: '16D8',
    ownerHandle: '2',
    layer: 'Tracing',
    lineweight: 25,
    name: '*U146',  // <------------- Would like an actual name which I can see in AutoCAD but not in the parsed value
    position: { x:<PHONE_NUMBER>310168, y:<PHONE_NUMBER>083853, z: 0 },
    xScale: -1,
    rotation: 89.99999564094279
  }

Thanks in advance!

I tried launching the three-dxf sample with the test file and I can see the correct name shown.
Maybe the name '*U146' you see there is Unicode?

If AutoCAD shows different block names then we may need a sample file. I don't know if it's stored in extended data or a standard dxf code/value pair.

I investigated this a bit more.
This is my understanding. When inserting a dynamic block, AutoCad anonymizes the name (*U... like name). However, if you use the BLOCK_RECORD table in the TABLES section, you are able to following ownerHandles to link a particular insert --> block (anonymized name) --> base block (which is named).
I extended the version of dxfParser I had to also parse the block record table. More than happy to open a PR for this, but perhaps later versions already have this?
Thanks again!

Ah, I see. A PR would be great. There hasn't been any other development on the BLOCK_RECROD table. I'm a little behind on maintaining this repo, but I've been making bits of progress lately and hope to bring in some PRs soon.

I investigated this a bit more.
This is my understanding. When inserting a dynamic block, AutoCad anonymizes the name (*U... like name). However, if you use the BLOCK_RECORD table in the TABLES section, you are able to following ownerHandles to link a particular insert --> block (anonymized name) --> base block (which is named).
I extended the version of dxfParser I had to also parse the block record table which helped with this investigation. More than happy to open a PR for this, but perhaps later versions already have this?
Thanks again!

can you share your code, thank you! @pranavramkrishnan


btr kc client

*Issue:*https://github.com/bcgov/entity/issues/18390
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-130-3to6hcru.web.app


Translate missing keys to spanish

Reports
Sales report
Sent

Hi @gnovaro!
I can help with the translation if you want 😊


Update components-props.md

Is the function 'includes' the better way to check the containing smth in array? No?

While you're technically correct, the documentation of Vue 2 is using ES5 standard and includes is ES7.


fix for v9 SON files

In the CED documentation for the SON library, it says:

In version 9 and above files, if pos is of type TDOF, the disk offset is pos*DISKBLOCK. In version 8 and previous files, the disk offset is pos.

this PR fixes the offsets for v9 files, and fixes #818

Hi @MarinManuel. This is super cool and should help many users.
Thanks a lot.

I have push the file "Two-mice-bigfile-test000.smr"
See https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/src/master/spike2
Could add it here in neo/test/rawiotest and here in neo/test/iotest and check that test are passing ?

The tests are failing as channel_names are expected to be of type 'S' and not unicode. @samuelgarcia was there a reason for this not to be unicode?
There are two options now:

change the definition of the ChannelIndex class and make channel_name unicode. This affects  all ios based on the raw mechanism.
Hackily reading the channel_names as unicode and converting it to type S.

I would go for approach 2, since the ChannelIndex object will anyway be replaced in the future (#817), so we don't need to introduce unnecessary changes for ChannelIndex objects now.

@JuliaSprenger @samuelgarcia  I've added a line to convert back the unicode strings to ascii for the time being. Hopefully that does not introduce any more problems.

Hi all.
@JuliaSprenger this is strange. I thought we fix this old (and very anoying) behavior where everything was ascii string because of python 2. Now python27 is no more here and evrything should be unicode everwhere.
@MarinManuel : I propose to fix this str vs unicode problem outside this PR very very soon. So you could rebase this PR with the offcial fix. is it OK ?

@MarinManuel the format change is in master now, could you rebase and undo your changes in basefromrawio.py?

@JuliaSprenger :Could we first merge this PR before  #775 which is also related to spike2io ?

Sure, then we keep #775 in the queue.

In fact I ask exactly the contary I wanted.
I wanted to say:
"Could we merge first  #775 (mine older) and then #824 (this one)"
Sorry. One day I will read what I am writting before push the green button.

@samuelgarcia Hi, I have reverted the commit on files channelindex.py and basefromrawio.py, is that good enough? Bit of a git newbie here...

Hi @MarinManuel,
there are still conflicts between the current master branch on Neuralensemble and the master branch on your fork. What you can do is rebase your branch on top of the current Neuralensemble master. First make sure you also have an up-to-date local version of that one on your machine by running
git fetch upstream (assuming upstream points to Neuralensemble)
Then you can do the rebasing via
git rebase upstream/master - this will lead to the conflicts pointed out above to appear during the rebase  in your local files. The conflicts are indicated in the files as
<<<<
first set of changes
=======
second set of changes
>>>>>

Please edit the files to include all the changes that are suggested by Neuralensemble master and additionally keep the ones you want to include in this PR.
Finally you add the files using git add <filename> and run git rebase --continue to finalize the rebase process.
To update this PR you then push the changes you made to your fork: git -f push origin master (assuming origin points to your fork). Here you need the -f flag as you want to overwrite the history of the commits in your fork, which were changed during the rebase process.
I hope this helps. Tell us if there's issues during the process.

Hi @JuliaSprenger
Thank you for the detailed explanations. Hope this works now

Hi @MarinManuel
could you modify theses two files like this:
neo/test/rawiotest/test_spike2rawio.py
class TestSpike2RawIO(BaseTestRawIO, unittest.TestCase, ):
    rawioclass = Spike2RawIO
    files_to_download = [
        'File_spike2_1.smr',
        'File_spike2_2.smr',
        ...
        'Two-mice-bigfile-test000.smr', # <<<<<< new line
    ]
    entities_to_test = files_to_download

neo/test/iotest/test_spike2io.py
class TestSpike2IO(BaseTestIO, unittest.TestCase, ):
    ioclass = Spike2IO
    files_to_test = [
        'File_spike2_1.smr',
        'File_spike2_2.smr',
        ...
        'Two-mice-bigfile-test000.smr', # <<<<<< new line

And afte rthis I will be able to merge this PR.
thanks again.

could you modify theses two files like this:
@samuelgarcia yes, done

thanks.
merging now.


add metrics advisor to reference index

Docs Build status updates of commit 8a8d76b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

api/overview/azure/index.yml
:warning:Warning
View
Details

api/toc.yml
:warning:Warning
View
Details

docs-ref-toc/top_level_toc.yml
:white_check_mark:Succeeded

api/overview/azure/index.yml

[Warning-UidNotFound] Unable to find uid "azure.net.sdk.landingPage.services.MetricsAdvisor".
[Warning-UidNotFound] Unable to find uid "azure.net.sdk.landingPage.services.springcloud".

api/toc.yml

[Warning-UidNotFound] Unable to find file with uid "azure.net.sdk.landingPage.services.KeyVault.Administration" referenced by TOC file "api/toc.yml"
[Warning-UidNotFound] Unable to find file with uid "azure.net.sdk.landingPage.services.MetricsAdvisor" referenced by TOC file "api/toc.yml"
[Warning-UidNotFound] Unable to find file with uid "azure.net.sdk.landingPage.services.springcloud" referenced by TOC file "api/toc.yml"
[Warning-UidNotFound] Unable to find file with uid "azure.net.sdk.landingPage.services.Storage.Client.11" referenced by TOC file "api/toc.yml"

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bc1e1e7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-toc/top_level_toc.yml
:warning:Warning

Details

docs-ref-toc/top_level_toc.yml

Line 614, Column 12: [Warning-xref-not-found] Cross reference not found: 'azure.net.sdk.landingPage.services.KeyVault.Administration'.
Line 619, Column 12: [Warning-xref-not-found] Cross reference not found: 'azure.net.sdk.landingPage.services.KeyVault.Common'.
Line 1040, Column 12: [Warning-xref-not-found] Cross reference not found: 'azure.net.sdk.landingPage.services.Storage.Client.11'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Update to propel for date fields

For date field we want just the date and not the time influenced by the timezone.
So we need to handle date colums different than datetime colums.

Closed by #15


Add input file for Vs code example

Add missing input file for vs code example.

Closed.  Fixed in #224 instead.
@Parsistence Can you double check whether we should delete the output.txt file like this PR has?


I run the same code on different devices, but the resolution is different.

Device 1：Allwinner T3 - Android 4.4.3
Device 2：Allwinner H6 - Android 7.0
Run the same program code, use onPlaybackQualityChange get quality, Device1 can reach HD720, and Device2 can only reach LARGE.
What caused this?
WebView？
Screen resolution？（adb shell wm size is 1280x800）
Thanks for the reply!

The web player decides the resolution, it's probably decided based on a combination of network speed and screen resolution.

The network is the same.
Does resolution determine hardware or system?

I am just guessing, but screen resolution is probably part of it.


Production By Scan class not working

This class was missed in refactor of version 6.   This has been corrected and uploaded into jar only zip download as a temporary fix.   It will be included in the next patch 6.1.9.

In addition to this bug fix, reversed the order in which the operations loaded into the dropdown...so that the final op appears first as a default to post production of the entire item to Finished Good status.


Expose TRV offset values in HA - Feature request

bosch_scan.txt
Each TRV has a temperature offset that can adjust the main temperature reading. By exposing this values to HA we can create automations in HA to adjust the TRV temperature based on external sensor.
"id": "/devices/device2/etrv/offset",
"type": "floatValue",
"writeable": 1,
"recordable": 0,
"value": 0.0,
"unitOfMeasure": "C",
"minValue": -2.0,
"maxValue": 2.0,
"stepSize": 0.5
},

Do you know if /devices/ can contain different devices than thermostatic ones?

As far as i know, no. CT200 dosent support external sensors or any other device types. Only thermostatic devices (TRV's)


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_e5qoqyjy9v/clone/recipe/meta.yaml:18: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_e5qoqyjy9v/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_e5qoqyjy9v/clone/recipe/meta.yaml:44: missing_description: The recipe is missing a description
ERROR: /tmp/abs_e5qoqyjy9v/clone/recipe/meta.yaml:44: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_e5qoqyjy9v/clone/recipe/meta.yaml:44: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_e5qoqyjy9v/clone/recipe/meta.yaml:44: missing_dev_url: The recipe is missing a dev_url
Errors were found


go-import meta tag isn't defined for the examples submodule

I have contributed to this issue in the playground repo and as discussed, the core issue appears to be that /gen/examples package has been made a submodule. This breaks the existing playground import structure.
I'm not an expert on the Go module system but I suspect that this is because go mod is searching https://gorm.io/gen/examples for a meta tag that should look something like
<meta name="go-import" content="gorm.io/gen/examples git https://github.com/go-gorm/gen/examples">

This url currently 404s and as a result the import:
gorm.io/gen/examples v0.3.26

fails to resolve and prevents running the playground. The playground can be run by checking out the gen project and using a replace directive but right now I think this is right solution.

The example is just a usage demonstration of the project and is not intended to be used as a Go SDK for import.

Indeed, but it is imported here in playground.


test(craft): reopen test suites

tests results below:

When I tested decrypt_blob separately, pass both relative and absolute paths as parameters are ok. When using cargo test for parallel testing, I suspect that other tests generated/deleted the key used for decrypt_blob testing, resulting in the need to use absolute paths to pass test_decrypt_blob. Trying to solve the question...


Mwpw 139779: Nested fragments with merch cards fix. #1589

Fixes nested fragments with grids of merch cards inside.
Context: Nested fragments use nested value in section metadata style on the parent section.
Merch card initialization should only apply display: contents to parent sections of fragments that do not have any display attribute (set by a class), this PR checks if the parent section has a class of nested before applying display: contents directly to the element to avoid breaking the grid with cards inside.
Resolves: MWPW-139779
Test URLs:
Before: https://main--milo--adobecom.hlx.page/drafts/delta/quiz-results-block/quiz-results?primary=design_ax_indiv,illustr_ax_indiv,pdf_indiv&quizKey=cc-quiz-en-US
After: https://mwpw-139779--milo--axelcureno.hlx.page/drafts/delta/quiz-results-block/quiz-results?primary=design_ax_indiv,illustr_ax_indiv,pdf_indiv&quizKey=cc-quiz-en-US

@Axelcureno I think we need a equal width for the cards

@Axelcureno this fixes have been already done in https://github.com/adobecom/milo/pull/1604
I proposes to proceed with that one.


Cannot find module './lib/colormin'

It actually breaks the code since I just do a fresh npm install. Please modify package.json to include lib folder as well.

Apologies for the inconvenience. :smile:

To remove/add depenencies for any changes like v2.1.2...v2.1.3 does, I'd suggest to follow semver and release a minor/major version to prevent unexpected failure.

Thanks for the fast reply! @ben-eb

Human error. I always try to follow semver. :smile:


Getting - Produced by

Hi, in a cache files there is Produced by, but how to get the with ID and name?

Produced by is only working for series...

End there isn'a way to work with the movies?

ofc there is a way to implement produced by - i could add it

Can you, really? Where can be view that?

I'll check if I can add it while fixing https://github.com/FabianBeiner/PHP-IMDB-Grabber/issues/87.


Can't create a new tab in my browser

hello, I am starting to use this framework and I want to start a browser, go to a url and then create a new tab.
when i try to do that i get the following error:
Traceback (most recent call last):
File "d:\Usuario\Desktop\BotCreadosPorMi\testeodecosas.py", line 21, in
browserReddit.new_tab("https://smailpro.com")
File "D:\python3910\lib\site-packages\splinter\driver_init_.py", line 68, in new_tab
raise NotImplementedError("%s doesn't open a new tab." % self.driver_name)
NotImplementedError: Firefox doesn't open a new tab.

@Berseck20 A well formed issue should include the following:

A reproducible example, formatted such that github displays it as a code block and preserves indentation.
A full traceback

In this particular case, the answer is: You used Firefox as the name of the browser. The correct name is 'firefox'.


Grey text on lightmode is hard to read

On your new Salary Transparency page, on light mode it is hard to read the grey text against the background.

can you please assign me this tax?

sure

sure

ok.. thanks


1.0.x/generators

This PR removes hard-coded tasks for generating specific types of projects, replacing them with a collection of resources in the project collection. Resources'  action attributes supports the same :rm, :ignore, :create, and :sync verbs, but resources can now be defined and modified directly through the Config interface, rather than requiring a code change to the gem.
Using the new generator.project.current reference allows a number of other attributes to be defined per-project-type as well.

LGTM

LGTM


[Docs]Addition of Usage steps in README

Topic: DOCS IMPROVEMENT
Closes

#29

Updated documentation to include instructions on how to use ksctl

@dipankardas011 sounds good. In the new usage.md file, should I also include installation steps? That way users won't have to go back and forth between the Readme.md and usage.md

In Readme I think installation and general usage should be fine and for particular providers we can add those to diff fine

@dipankardas011 sounds good. In the new usage.md file, should I also include installation steps? That way users won't have to go back and forth between the Readme.md and usage.md

Yes
Just make a link for usage in readme✌

@dipankardas011 done

@AvineshTripathi you can review and merge it

Should we add HA cluster is still work in progress @dipankardas011? Rest looks good. Thanks @siddhant-khisty

Yes it will be better to make it Progress sign '🚧 '

Added the HA cluster as in progress

@siddhant-khisty is there any blocking issue that you are facing?

@dipankardas011 nope. Just been a bit busy. Will push soon

@dipankardas011 nope. Just been a bit busy. Will push soon

no problem


Images are not loading for certain queries or page ranges even though it works manually.

The latest version of Grabber, 7.1.1. Default settings for everything. Source used is yande.re.
Query Used: id:552746..573896
Loading the first page works perfectly fine and Grabber states there are 951 total pages and currently on page 1, 20 images are shown out of 19018. Proceed to go to the last page and it will fail with:
yande.re - No result
No valid source of the site returned result.
However, if you were to click on the blue "yande.re" link and opened up the page manually, you will see there are actual results. In addition, if you manually increment the page numbers through the URL, you will still get results past 951. So the total page should be more than 951.
Another example is using the query: id:550746..573896
Loading pg 1043 and 1046 will work fine but pages in between them, pg 1044 and 1045, will result in the same error as before:
"yande.re - No result"
No valid source of the site returned result.

I've had this issue for months, can confirm it is a real issue and it is annoying.
It happens too frequently. It will also refuse to download images.

I could indeed reproduce the issue, and the logs give the reason:
[12:07:25.330][Warning] [yande.re][Xml] Uncaught exception at line 2: TypeError: Cannot read property 'length' of undefined
[12:07:25.331][Warning] [yande.re] Loading using Xml failed. Retry using Json.
[12:07:25.401][Warning] [yande.re][Json] Uncaught exception at line 2: TypeError: Cannot read property 'length' of undefined
[12:07:25.402][Warning] [yande.re] Loading using Json failed. Retry using Html.
[12:07:25.524][Warning] [yande.re][Html] Uncaught exception at line 2: TypeError: Cannot read property 'length' of undefined
[12:07:25.526][Warning] [yande.re] No valid source of the site returned result.

It is caused by a small bug in the code when encountering deleted images, which happen on the specific pages you mention. I just pushed a fix which should solve the issue.


clean up for ubuntu 14.04

Hi there,
Tried to make it work on Ubuntu 14.04. Here is some clean-up working on my machine. Thanks again for implementing SEND.
Best,
Dave

Added bug fixes for
Hash2 in CGA verification
Fixed modifier with non-zero sec value in CGA generation.
Thanks,
Dave


Any plans for fn + num to keypad num?

One feature I loved from Karabiner on macOS 10.11 was using fn + number keys to emulate the keypad keys on extended keyboards. Any plans to bring this back?

https://github.com/tekezo/Karabiner-Elements
Karabiner-Element is a better place to ask :)

Whoops! Didn't realize it was separate. Thank you!


(feature) Datetime format personalization

Asana task:
...
Description:

Related Pull Requests:
...
Backend Dependencies:
https://github.com/bitfinexcom/bfx-hf-server/pull/233
Other Dependencies:
...

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1202373640620648

@Blaumaus could you please resolve conflicts above

@dmytroshch could you please fix above conflicts


3D refinement: iteration counter doesn't apply

Need to monitor for run_class001.mrc or the like to check when 3D refinement is finished. Currently the script it written for 3D classification

Fixed.
https://github.com/leschzinerlab/AWS/commit/fa1970b1cc6d4278d2662e9e74aa3086267a8bce


crypto/pkcs12/p12_mutl.c: Add check and EVP_MD_free() for EVP_MD_fetch()

Add check and EVP_MD_free() for EVP_MD_fetch() to avoid NULL pointer dereference and memory leak, like "md_fetch".
Fixes: fe79159be0 ("Implementation of the RFC 9579, PBMAC1 in PKCS#12")

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to the master and 3.4 branches. Thank you for your contribution.


Add "Login with Google" and |"Login with Facebook" Functionality

I noticed that when a user clicks on the Google or Facebook icon to login with Google or Facebook, nothing happens.
I will like to add this functionality to enable users login with their google and facebook account.

@AtriSukul1508,please assign this issue to me under gssoc'23

Hello @AtriSukul1508 ,please assign this to me

@Maryann-01 go ahead, It would be great if you could add facebook as well

@Maryann-01 Make sure to fetch the repo before making a pr

Okay,I would do that.Thank you.

@AtriSukul1508 ,

This is the image I am seeing anytime I run the app,it is entirely different(the UI) from what is on the vercel live link and even the facebook and google icons are not there.Please check and which one am I to work with?
This is what is displaying on the homepage for the livelink ,

@Maryann-01 It should look exactly like the one in the vercel link. If the problem persists, fork the repo once again and try checking again.

@AtriSukul1508 ,I created a pull request.Please review.

@AtriSukul1508 ,Please review My PR.


Using dart:ui in plain dart application no flutter

Is it possible to use the dart:ui package  in a non flutter (dart) application?
When I try to run my dart file with dart main.dart I get the following error:
Error: Not found: 'dart:ui'

You cannot! That library is only loaded when accessed via a flutter process.

I just wanted to chime in since I was asking myself the same question recently, and I think the problems stems from what we (regular consumers) generally think of as Flutter.
I think most people, myself included, when talking about flutter think about the framework not necessarily the engine.
So, I think this question might actually be asking how can I use the dart:ui package without the Flutter framework not necessarily the engine.  This is of course possible, following the bellow steps. Also, it helps to think of the flutter command line tool not as the framework tool but the engine tool, with just the templates having a preference for the framework.
Steps to an application without the framework:
# pubspec.yaml

name: app
publish_to: none

environment:
  sdk: '>=2.17.0 <3.0.0'

dependencies:
  sky_engine:
    sdk: flutter

I just wanted to chime in since I was asking myself the same question recently, and I think the problems stems from what we (regular consumers) generally think of as Flutter.
I think most people, myself included, when talking about flutter think about the framework not necessarily the engine.
So, I think this question might actually be asking how can I use the dart:ui package without the Flutter framework not necessarily the engine.  This is of course possible, following the bellow steps. Also, it helps to think of the flutter command line tool not as the framework tool but the engine tool, with just the templates having a preference for the framework.
Steps to an application without the framework:
# pubspec.yaml

name: app
publish_to: none

environment:
  sdk: '>=2.17.0 <3.0.0'

dependencies:
  sky_engine:
    sdk: flutter

// main.lib
import 'dart:async';
import 'dart:ui';

void main() {
  final pictures = createPictures().map((e) {
    final b = SceneBuilder()..addPicture(Offset.zero, e);
    return b.build();
  });

  var iterator = pictures.iterator;
  window.onBeginFrame = (_) {
    if (!iterator.moveNext()) {
      iterator = pictures.iterator;
      iterator.moveNext();
    }
    window.render(iterator.current);
  };
  Timer.periodic(Duration(milliseconds: 1000), (_) => window.scheduleFrame());
}

List<Picture> createPictures() {
  return [
    createPicture(Paint()..color = Color.fromARGB(255, 100, 100, 100)),
    createPicture(Paint()..color = Color.fromARGB(255, 30, 120, 120)),
    createPicture(Paint()..color = Color.fromARGB(255, 120, 30, 120)),
    createPicture(Paint()..color = Color.fromARGB(255, 120, 120, 30)),
  ];
}

Picture createPicture(Paint paint) {
  final recorder = PictureRecorder();
  draw(Canvas(recorder), paint);
  return recorder.endRecording();
}

void draw(Canvas canvas, Paint paint) {
  canvas.drawRect(Rect.largest, paint);
}

Then run flutter create --platforms=web .. This adds the necessary files to support the web.
And then simply run: flutter run lib/main.dart or use the IDE of choice to start process.
Just a note, the sky_engine really is the low level engine now called flutter_engine. And as @kevmoo mentioned before running this through flutter is required because the native ui libraries are loaded through it. But the before described steps work for building an application without the Flutter framework.
Also probably the source of confusion is the naming dart:ui but  it helps to think of it as flutter:ui or flutter:low_level_ui_tools


[enhancement] opentelemetry-java-instrumentation sets the project environment-related information from the JVM to environment variables

opentelemetry-java-instrumentation sets the project environment-related information from the JVM to environment variables.

What's the purpose of doing this?

Some external users (some small companies) don't want to modify their existing Dockerfile to set the environment variables required by the probe. Instead, they hope to use JVM startup parameters as a substitute for environment variables.

Some external users (some small companies) don't want to modify their existing Dockerfile to set the environment variables required by the probe. Instead, they hope to use JVM startup parameters as a substitute for environment variables.

all right, I will do it.


Problem with postinstall.sh

After a new install trial failing due too long npm install request, I finished the install by hand :
sudo npm install
then
sudo ./node_modules/gulp/bin/gulp.js b && ./postinstall.sh
Here is what I got  (gulp was run without error) :
chown: changing ownership of ./weboob/requirements.txt': Operation not permitted chown: changing ownership of ./weboob/scripts/accounts.sh': Operation not permitted
chown: changing ownership of ./weboob/scripts/operations.sh': Operation not permitted chown: changing ownership of ./weboob/scripts/install.sh': Operation not permitted
chown: changing ownership of ./weboob/scripts/test.sh': Operation not permitted chown: changing ownership of ./weboob/scripts': Operation not permitted
chown: changing ownership of ./weboob/banks-all.json': Operation not permitted chown: changing ownership of ./weboob/py/accounts.py': Operation not permitted
chown: changing ownership of ./weboob/py/operations.py': Operation not permitted chown: changing ownership of ./weboob/py/lib.py': Operation not permitted
chown: changing ownership of ./weboob/py/connector.py': Operation not permitted chown: changing ownership of ./weboob/py': Operation not permitted
chown: changing ownership of ./weboob/.gitignore': Operation not permitted chown: changing ownership of ./weboob': Operation not permitted
I didn't use cozy-kresus user (sudo -u cozy-kresus) to run this last command as I got write errors on compiling the js client with gulp command.
I think a sudo is missing before launching the scripts  (they are run by the controller which has root rights)
command should be in the package.json :
sudo ./node_modules/gulp/bin/gulp.js b && sudo ./postinstall.sh
The distro used in raspbian wheezy (cozy image for RPI2)

So, the postinstall script tests whether there is a cozy-kresus user before chmoding the directory. The intent hidden behind this is that the postinstall is run by the root user, and thus has the rights to apply the chmod.
Can it be that sudo ./node_modules/gulp/bin/gulp.js b && ./postinstall.sh is actually equivalent to  (sudo ./node_modules/gulp/bin/gulp.js b) && ./postinstall.sh in your shell?

I forgot to add: can you try with sudo ./postinstall, please?

I did, it worked fine. (I updated the first comment) to suggest a double 'sudo' use

Cool, I think we're all set here. Can we close? If you feel like it, feel free to add a comment in the readme or the postinstall script stating that this one should be run with the root rights?

shouldn't be the package.json updated ?
If you consider not, then close

Good call! Rather than running the postinstall.sh script with sudo (we're not even sure sudo is installed on the machine, and I'd rather not add requirements...), I've added an explicit error message proposing a few possible solutions to the problem.

Perfect!.


Vivint camera feed fine on web but not mobile (doorbell cam)

My Vivint doorbell cam integrates well and its stream is visible and live in the HA web interface.
When viewing with mobile, only a black screen is shown--the feed does not come through. All other non-doorbell cameras work fine in mobile.

Have you selected "preload stream"? That could help with the issue on your mobile.

Yup. Out of nowhere it started working fine and now seems reliable. Thanks.


Reviews [NIP-??] Positive Interoperability Tracker

working spec: https://github.com/nostr-protocol/nips/pull/879

app
see reviews
create reviews
comment

coracle
✅
✅
@staab advised coracle implemented

@staab what are the use cases in Coracle for reviews? For instance relay reviews? Marketplace vendor reviews?

Relay reviews only right now. This is a pretty stale feature, and there's not that much data associated with it, so don't consider my implementation canonical.

Has anyone else implemented?

@calvadev @gzuuus @vitorpamplona do yall plan adding reviews NIP in your marketplace apps/segments?

@calvadev @gzuuus @vitorpamplona do yall plan adding reviews NIP in your marketplace apps/segments?

@alltheseas Yes; actually about to wrap up product/merchant reviews based on the above NIP-85 PR. Will add a comment to the PR when done.

do yall plan adding reviews NIP in your marketplace apps/segments?

Yes, I am just waiting for the NIP

@calvadev @gzuuus @vitorpamplona do yall plan adding reviews NIP in your marketplace apps/segments?

@alltheseas Yes; actually about to wrap up and deploy product/merchant reviews based on the above NIP-85 PR within the next week or so. Will add a comment to the PR with the "spec".

Here's the comment:
https://github.com/nostr-protocol/nips/pull/879#issuecomment-2502210146

Yes, will implement them also, not sure when, but would be nice to have a solid spec for them

@alltheseas Yes; actually about to wrap up and deploy product/merchant reviews based on the above NIP-85 PR within the next week or so. Will add a comment to the PR with the "spec".

Here's the comment:
nostr-protocol/nips#879 (comment)

@alltheseas added support for it on shopstr today


Post Processing error:  Failed to invoke function 'onOpen'.

Hi everyone,
I've tried to follow the intro videos very carefully, went trough them twice but every time I'm bumping into the following error while trying to post-process. Can you help out me, the newbie of laser post?
Background: I'm running Fusion 360, the up-to-date version, on a Mac M1 with Sonoma.
Fusion says this: Fusion 360 2.0.17457 arm64 [Native]
macOS 14.0 (23A344) on MacBookPro17,1
The Error log looks (always) like this:
Information: Configuration: LaserPost for LightBurn (1.1.0) Information: Vendor: nuCarve Information: Posting intermediate data to '/Users/XXX/Documents/Freizeit/Laser/Project Files/Weihnachten/house.lbrn' Information: Total number of warnings: 3 Error: Failed to post process. See below for details. ... Start time: Mon Oct 30 17:34:44 2023 Warning: (/Users/XXX/Library/Application Support/Autodesk/Autodesk Fusion 360/C5SQQ35TNEDH/W.login/M/D20190926218999781/CAMPosts/laserpost-lightburn.cps:1981) vector is read-only Warning: (/Users/XXX/Library/Application Support/Autodesk/Autodesk Fusion 360/C5SQQ35TNEDH/W.login/M/D20190926218999781/CAMPosts/laserpost-lightburn.cps:3082) text is read-only Warning: (/Users/XXX/Library/Application Support/Autodesk/Autodesk Fusion 360/C5SQQ35TNEDH/W.login/M/D20190926218999781/CAMPosts/laserpost-lightburn.cps:4416) text is read-only Post processor engine: 4.5991.0 Configuration path: /Users/XXX/Library/Application Support/Autodesk/Autodesk Fusion 360/C5SQQ35TNEDH/W.login/M/D20190926218999781/CAMPosts/laserpost-lightburn.cps Include paths: /Users/XXX/Library/Application Support/Autodesk/Autodesk Fusion 360/C5SQQ35TNEDH/W.login/M/D20190926218999781/CAMPosts Configuration modification date: Mon Oct 30 13:28:31 2023 Output path: /Users/XXX/Documents/Freizeit/Laser/Project Files/Weihnachten/house.lbrn Checksum of intermediate NC data: 58d6b719164e36d70dfbcefd8e1e091c Checksum of configuration: 7eec1d2dd0c14441ae9d8e8cc2b2ccb2 Vendor url: https://nucarve.com/laserpost Legal: Copyright (C) 2023 by nuCarve Generated by: Fusion 360 CAM 2.0.17457 ... Error: Failed to invoke function 'onOpen'. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Error: Failed to invoke 'onOpen' in the post configuration. Error: Failed to execute configuration. Stop time: Mon Oct 30 17:34:44 2023 Post processing failed.
My setup looks roughly this way:

If you could help me out, that would be greatly appreciated!
Cheers,
Daniel

I "solved" the problem by downgrading to LaserPost 1.0 (stable version), instead of using the 1.1 (stable).
Of course not really a fix for the problem, but for me it works now...

I had the exact same problem. But downgrading didn't work either.
Information: Vendor: nuCarve
Information: Posting intermediate data to 'C:\Users\tim\Downloads\tray.lbrn'
Information: Total number of warnings: 1
Error: Failed to post process. See below for details.
...
Code page changed to '1252  (ANSI - Latin I)'
Start time: Saturday, April 27, 2024 3:27:56 PM
Code page changed to '20127 (US-ASCII)'
Post processor engine: 5.131.1
Configuration path: C:/Users/tim/AppData/Roaming/Autodesk/Fusion 360 CAM/Posts/laserpost-lightburn stable.cps
Security level: 1000
Include paths: C:/Users/tim/AppData/Roaming/Autodesk/Fusion 360 CAM/Posts
Configuration modification date: Saturday, April 27, 2024 3:23:41 PM
Output path: C:\Users\tim\Downloads\tray.lbrn
Checksum of intermediate NC data: 3b1ee0aa7b2de294bcba46110fc574b9
Checksum of configuration: 6f25b63a3d0bd424ccd00b445ac40fba
Vendor url: https://nucarve.com/laserpost
Legal: Copyright (C) 2022 by nuCarve
Generated by: Fusion CAM 2.0.18719
...
Warning: LaserPost requires your permission to check for updates.

ALLOW: Select "Yes" on the following dialog (and optionally check the "Remember" box).

DECLINE: You must disable update checking.  Select "No" on the following dialog then change the "LaserPost: Automatic update" option on the post properties page to "Never".

Visit https://nucarve.com/laserpost-permissions for more information.

###############################################################################
Error: Failed to invoke method of class 'PostProcessor'.
Error at line: 3071 (laserpost-lightburn stable.cps)
Failed while processing onOpen().
###############################################################################

Error: Failed to invoke function 'onOpen'.
Error: Failed to invoke 'onOpen' in the post configuration.
Error: Failed to execute configuration.
Stop time: Saturday, April 27, 2024 3:27:56 PM
Post processing failed.

Sorry about very long silence!  The notifications from GitHub were not coming to my inbox, and I missed this.
There was a change in Fusion's implementation of the JavaScript engine (a few times since this issue was open) that caused this.  They improved their compliance to the JavaScript standards and that exposed some bugs in the process.  Version 1.2 fixes most of those, but somebody just sent in a PR for another one (which caused me to visit the github repo to see these unanswered / outstanding issues) so there will likely be a 1.2.1 soon.


Add support for the Haxe Language Server

This adds support for the language server used by the Visual Studio Code Haxe Extension: https://github.com/vshaxe/haxe-language-server
I've found this language server to be tricky to properly attach to, but this config should do the trick.

I don't use haxe so I've been waiting for someone to comment/test. I'll try to give some cursory feedback in the next several days.

No worries, @mjlbach. Let me know if I can provide any more information.

left some comments, mostly want to edit for brevity

Appreciate all your feedback, @mjlbach. If my changes look good, let me know and I'll rebase the fixup away. Otherwise, happy to collaborate some more.

I think it’s probably fine, if you squash and test that it works I’ll merge

From: N. G. Scheurich<EMAIL_ADDRESS>Sent: Sunday, March 7, 2021 1:22:52 PM
To: neovim/nvim-lspconfig<EMAIL_ADDRESS>Cc: Michael Lingelbach<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [neovim/nvim-lspconfig] Add support for the Haxe Language Server (#707)
Appreciate all your feedback, @mjlbachhttps://github.com/mjlbach. If my changes look good, let me know and I'll rebase the fixup away. Otherwise, happy to collaborate some more.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/neovim/nvim-lspconfig/pull/707#issuecomment-792355648, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADFTBJVIY5F2GHAK2W7B5VDTCPVCZANCNFSM4W22XRVA.

Should be good to go now, @mjlbach.


Consider adding a higher level API on top of DIE parsing/traversal

Something like this:
struct Subprogram {
    ...
}

impl Subprogram {
    pub fn new(die: DebuggingInformationEntry) -> Subprogram {
        assert!(die.tag() == DW_TAG_subprogram);
        Subprogram { ... }
    }

    pub fn name(&self) -> Option<&ffi::CStr> { ... }

    pub fn linkage_name(&self) -> Option<&ffi::CStr> { ... }

    pub fn calling_convention(&self) -> Option<DwCc> { ... }

    pub fn is_pure(&self) -> Option<bool> { ... }

    pub fn is_recursive(&self) -> Option<bool> { ... }

    // Etc...
}

Would ideally be able to become owned, if the user wants to copy the data.

When I writing a program that parse global variable addresses( https://github.com/grissiom/elfvars/ ), I found I have to pass the gimli::Abbreviations and gimli::DebugStr all the way down if I want to do "lazy parsing"(travel only portion of the DIE and than parse only the DIEs that relevant).
IMHO, the abbreviations and debug_str associated with a CU does not change in the middle. So a higher level API may group those info into a struct and make the methods easier to use.

It's not just those two that you need. DIEs can contain references to most other sections too, and so you currently need to pass those sections down too if you want to use those values (the dwarfdump example passes down abbrev/line/loc/ranges/str). Definitely open to ideas for making this easier to use though.

If we had an Image for the whole object file, it might help solve the issues here as well as potentially making #182 more ergonomic. The down side would be that gimli has to get a deeper understanding of object files, rather than having it be completely another crate's concern.

Maybe we could handover the "parsing" of the ELF to other crates such as object or xmas-elf. But only store the section data references in the Image.
For #182 , I think we could let Reader to do the ELF parsing for us and we just need to &[uint8] read(".debug_info", offset). If the backend of Reader is memmap, than we could get Zero-copy for free. If the backend is internet, it could do some COW.


Add a restart command to tmuxinator environment

Currently if you are running the tmuxinator dev environment, edit some code, and want to re-run it, you either need to kill and restart tmuxinator (slow) or manually to stop, rebuild, and restart what you changed (annoying). We should instead have one command that's easy to remember and run that just handles this for the developer.

[x] Kill fedimintd, rebuild it, and re-run it in the same pane it currently runs in. See this message. I'm not sure exactly how this would work. This shell process would need to subscribe to some kind of signal to know restart has been requested. (#1589)
[x] Stop the CLN plugin, rebuild it, and re-run it. See this script.

The user should be able to run this command from any tmux pane.

Regarding the second item, I included gw-reload.sh at the bottom of the new restart-tmux.sh script, since that's script @justinmoon refers to. Is there something else that should be done?

I don't think so, good point. Can re-open if there is anything left.


Tracking: add all of the protocol messages

Under src/network/wire/lib.zig are the methods to send messages to another node, by writing those on a "wire".
Under  src/network/protocol/messages/version.zig is the first message we have, along with the methods allowing to serialize it into the wire.
Here is the list of all of the protocol messages: https://developer.bitcoin.org/reference/p2p_networking.html
Please convert the following checkbox to issues and work on it (1 per person at the same time):

[x] version
[ ] verack
[ ] addr
[ ] inv
[ ] getdata
[ ] notfound
[ ] getblocks
[ ] getheaders
[ ] tx
[ ] block
[ ] headers
[ ] getaddr
[ ] mempool
[ ] checkorder
[ ] submitorder
[ ] reply
[ ] ping
[ ] pong
[ ] reject
[ ] filterload
[ ] filteradd
[ ] filterclear
[ ] merkleblock
[ ] alert
[ ] sendheaders
[ ] feefilter
[ ] sendcmpct
[ ] cmpctblock
[ ] getblocktxn
[ ] blocktxn

Hey, I'd like to work on verack message :)


Specific return types for values and values list

Resolves https://github.com/mkurnikov/django-stubs/issues/45
Whew... this was tougher to implement than I first imagined :)

I'm going to run it against internal codebases tomorrow, and merge+release if everything is ok.

Looking forward to hear if you catch any new errors when running on your project.

I ran on an internal codebase, and there were only 2 differences in the reported errors.
In the first case, compared to master, this error disappeared (was a false positive):
def func(arg: Iterable[Tuple[int, datetime]]) -> int:
  pass

func(MyModel.objects.annotate(some_annotation=Max('somefield')).values_list('id', 'some_annotation') # Error occurs here

f.py:63: error: Argument "arg" to "func" has incompatible type "QuerySet[MyModel, Tuple[Any, ...]]"; expected "Iterable[Tuple[int, datetime]]"
The code was erroring because for all mypy knew, the Tuple could have had any number of elements.
With this PR, mypy knows that the Tuple row type in the QuerySet has exactly 2 elements, even though it considered the type of the second Tuple element as Any (because annotations are not supported).
In the second case, the error message was more specific because the type was inferred.
master:
f2.py:92: error: Incompatible types in assignment (expression has type "Optional[Any]", variable has type "str")
PR:
f2.py:92: error: Incompatible types in assignment (expression has type "Optional[*str*]", variable has type "str")
Code looks like this:
def f(default_token: Optional[str]):
    if default_token is not None:
        token = default_token # Inferred to be str
    else:
        token = OtherModel.objects.filter( # Error occurs here
            token__isnull=False,
            # ... other conditions here
        ).values_list("token", flat=True).first() # Inferred to be Optional[str]
    if token:
       ...

Although the code checks that the token var is not None later on (if token), I guess mypy doesn't like a variable that can be both Optional and non-Optional depending on the code-path.

Internal codebases gave me this
[CASE values_with_annotate_inside_the_expressions]
from django.db import models
from django.db.models.functions import Lower

class Publisher(models.Model):
    pass

class Book(models.Model):
    name = models.CharField(max_length=100)
    publisher = models.ForeignKey(to=Publisher, on_delete=models.CASCADE, related_name='books')

Publisher().books.values()
reveal_type(Publisher().books.values('name', lower_name=Lower('name')))

Revealed TypedDict should have lower_name as a key, at least as a
lower_name: Any

This test also shows a lot of errors with stubs, fixed them in https://github.com/mkurnikov/django-stubs/pull/54

Thanks for the feedback. I'll look at this issue with values and kwargs.
Looks like there also may be some problems with the way the TypedDict is constructed.

Tried on internal codebase, no more errors, thanks!


Canvas Performance: severe problems with dom-walker, MutationObserver and ResizeObserver

When dragging the border radius slider of the PromiseCard component, we can see egregious performance:

The entire render would be bad enough at 55ms, but we accidentally run it twice because an activating MutationObserver and ResizeObserver.
These observers do not create accidental re-renders during on-canvas interactions, which is where we tested them, so that's good.
BUT the entire system is not fine grained and were designed in a more naive time:
1: we invalidate the entire scene, not just the element
2: it's not controlled when should this dom-walker run happen then
3: we should either rely on the observers OR on the canvas interaction, but not both
4: the dom-walker should never walk entire subtrees during interactions
Possible ideas for building a better solution:

Rheese proposed that we should store in the jsxMetadata a flag for the metadata staleness.
we could try a model where we more bravely let a lot of metadata remain stale during interactions: 99% of canvas interactions only need to know metadata of the same few elements: the selected element, its parent, siblings, children, and what is under the mouse. We could cut a ton of CPU cycles spent on creating and maintaining the metadata for hundreds of other elements
the ReconstructJsxMetadata is pretty expensive, we should refactor the metadata so that we only need to cheaply join the SpyMetadata and the DomMetadata, but without the need to create a bunch of new objects. Today the spyMetadata creates an entire fake object for specialSizeMeasurements etc, all of this is just putting load on the garbage collector.
Move jsxMetadata out of EditorState, possibly into the editorStore root, so we can more clearly treat it as its own thing.
With React flushSync in the mix, it seems like we -could- create a new approach that relies purely on the mutation observer, but this needs investigation
the actual dom-walker run time and performance should be investigated too.

** TLDR end goal is that during an interaction, we really only call getComputedStyle for a handful of elements, and spend almost zero time on merging large jsxMetadata trees etc. ideally ALL OF the metadata update should take 0.1-0.2ms per frame instead of 7-10ms of today**

@balazsbajorics Is this ticket still relevant?

Note: for labels scrubbing, the performance (at least in vanilla project) is now more than adequate (prob result of #6196 and other performance improvements)

Turns out this actually done not blocked; closing!


Updated MAINTAINERS.md format.

Coming from https://github.com/opensearch-project/.github/issues/121, updated MAINTAINERS.md to match opensearch-project recommended format.

Can we please get this merged? Another code reviewer? @dai-chen ?

@dblock I added Ani and Peng for review.

Bump @anirudha @penghuo ?


Feature Request : Add new app icon

Is your feature request related to a problem? Please describe.
NA
Describe the solution you'd like
Suggest a new icon for app
Additional context
Icon should describe the core functionality of this app : security

This is the best I can design

Hi! Please checkout my design for the app icon

@Bhawna-Ad  Assigning it to you, Please make a PR for this.


Update .deepsource.toml

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

:x: Build Sankhya-SDK-dotnet 3.0.961 failed (commit https://github.com/guibranco/Sankhya-SDK-dotnet/commit/479137ea02 by @guibranco)


Include CGE-Predicted Phenotypes for PointFinder results

As a complement to https://github.com/phac-nml/staramr/pull/167 (which included CGE-Predicted Phenotypes for ResFinder results) it would be nice to do the same thing for the PointFinder results. For example, as shown in the Phenotype column here https://bitbucket.org/genomicepidemiology/pointfinder_db/src/master/salmonella/phenotypes.txt

As discussed, as far as I can tell, the phenotypes.txt files only exist in newer versions of the database (June 2022 onwards).
Since we don't use this version of the database, I will close this, as instructed. If we updated our version of the database used and want to implement this later, the WIP branch is here:

dev/cge-pointfinder-prediction

https://bitbucket.org/genomicepidemiology/pointfinder_db/src/master/salmonella/resistens-overview.txt
The resistens-overview.txt files have a Resistance column that can be used for Pointfinder.

Fixed in #190


05/09/2018 03:09 : Create Pull Request for branch prTesting_2018050903 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Admin star isn't displayed

/cardinal -v?

Confirmed, Cardinal version 6e156dc

@SungMatt Thank you.
What is the join message, which should be visible from the console?


Component Library Version 1.5.0 missing in package.json

It seems that on version 1.5.0 was not set in the package.json, could you please update the file?
https://github.com/Lambda-IT/parlamentsdienste-components/tree/v1.5.0
We have a CI Pipeline for this on our side which will break in such cases.
As an alternative would it be possible that the Library is build and provided packaged on your side e.g. https://github.com/features/packages ?

Oh yes. I forgot that. May we could publish a tar.gz in the Release?
Then we do not have to configure the repository for that.

fixed with v1.5.1 release https://github.com/Lambda-IT/parlamentsdienste-components/releases/tag/v1.5.1

@CuriaPLUS fyi we will prepare a build process over Github Actions to deploy a npm package for every release.


Add ScaNN support in vectorstore.

Description: Add ScaNN vectorstore to langchain.
ScaNN is a Open Source, high performance vector similarity library optimized for AVX2-enabled CPUs.
https://github.com/google-research/google-research/tree/master/scann

Dependencies: scann

Python notebook to illustrate the usage: docs/extras/integrations/vectorstores/scann.ipynb
Integration test: libs/langchain/tests/integration_tests/vectorstores/test_scann.py
@rlancemartin, @eyurtsev for review.
Thanks!

Thanks for the suggestions!
I have address the comments and ran lint / format.
Also feel free to take over this PR if needed.

Let me know if anything else is needed. Thanks! @baskaryan

Another gentle ping. Thanks! @baskaryan


"Subclassed" Categories webservice prototype

Hi there y'all!  I've finished another proof-of-concept of a piggybacked Categories webservice implementation, in which Category is a subclass of Collection and the two share the collection db table.
Please take a quick look.  The code is intentionally rough, there's dangling cruft, spelling errors (!), etc., so, as with the last prototype, I'm looking for high-level feedback as to the approach and its desirability versus the previous prototype and "pure Categories".   If we decide to graft this into develop, I'll make a few passes to ensure that the code is up to snuff, add elasticsearch support, write some tests, convert the db setup script to our migration scheme, etc.  If you see any functional problems that you don't think I'll find during that process, it would be great to know about those too.
Run scripts/install-categories.sh <username> to set up your local db.  It will create a 'SPECIAL' organization named "dockstoreCategories", add the specified existing user as it's admin, and add a column to the collection db table.
After that, fire up the webservice+ui, log in as the dockstoreCategories admin user, navigate to "/organizations/dockstoreCategories", and add some categories!
Some new endpoints are exposed, slightly different than the last prototype:
/categories
/categories/names/{name}
/entries/{id}/categories
There is some philosophical debate about the above naming, it's easy to change if we decide to do so.
Each CollectionEntry now includes a categoryNames property.
In this implementation, Category/Collection identity is decided at creation time and stored in db column collection.dtype, using JPA single table inheritance. Both Categories and Collections are created/managed via the existing Collection endpoints.
The 'SPECIAL' application status has been added and does double duty: both as a simple mechanism to hide the category-managing organization, and as a trigger for Category creation.  The current logic is to create a Category when the creating organization is 'SPECIAL', and a regular Collection otherwise.
Multiple special organizations may exist, and may each own categories, allowing us to partition category management.  The architecture allows categories to be owned by any organization, although it's not possible to configure the system that way via the endpoints at present.  A category owned by a public organization would appear both in that organization's list of collections, and in the global list of categories.
Via the category-specific endpoints, all categories are always visible to everyone.  That's an ok policy, right?
I'll need to add a db constraint to enforce unique lower(names) amongst Categories.
There may be a better adjective to describe a category-managing organization than 'special'.
This code currently bombs the ITs, because I don't have the db migration coded the appropriate way.
Whee!

Spoke to Denis about it. I prefer the other PR. It's more intuitive to me. But if everyone likes this more then 🤷


Bump ruby version

Summary
Fixes https://github.com/Betterment/test_track/issues/52
/domain @Betterment/test_track_core

Hmmm, seems to be failing with a fun:
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
current directory:
/home/travis/build/Betterment/test_track/vendor/bundle/ruby/2.4.0/gems/json-1.8.3/ext/json/ext/generator
/home/travis/.rvm/rubies/ruby-2.4.2/bin/ruby -r
./siteconf20171024-4676-30btk2.rb extconf.rb
creating Makefile
current directory:
/home/travis/build/Betterment/test_track/vendor/bundle/ruby/2.4.0/gems/json-1.8.3/ext/json/ext/generator
make "DESTDIR=" clean
current directory:
/home/travis/build/Betterment/test_track/vendor/bundle/ruby/2.4.0/gems/json-1.8.3/ext/json/ext/generator
make "DESTDIR="
compiling generator.c
generator.c: In function ‘generate_json’:
generator.c:861:25: error: ‘rb_cFixnum’ undeclared (first use in this function)
 } else if (klass == rb_cFixnum) {
 ^
generator.c:861:25: note: each undeclared identifier is reported only once for
each function it appears in
generator.c:863:25: error: ‘rb_cBignum’ undeclared (first use in this function)
 } else if (klass == rb_cBignum) {
 ^
generator.c: At top level:
cc1: warning: unrecognized command line option "-Wno-self-assign" [enabled by
default]
cc1: warning: unrecognized command line option "-Wno-constant-logical-operand"
[enabled by default]
cc1: warning: unrecognized command line option "-Wno-parentheses-equality"
[enabled by default]
cc1: warning: unrecognized command line option "-Wno-tautological-compare"
[enabled by default]
make: *** [generator.o] Error 1
make failed, exit code 2
Gem files will remain installed in
/home/travis/build/Betterment/test_track/vendor/bundle/ruby/2.4.0/gems/json-1.8.3
for inspection.
Results logged to
/home/travis/build/Betterment/test_track/vendor/bundle/ruby/2.4.0/extensions/x86_64-linux/2.4.0/json-1.8.3/gem_make.out
An error occurred while installing json (1.8.3), and Bundler cannot continue.
Make sure that `gem install json -v '1.8.3'` succeeds before bundling.
In Gemfile:
 rails was resolved to <IP_ADDRESS>, which depends on
 actionmailer was resolved to <IP_ADDRESS>, which depends on
 actionpack was resolved to <IP_ADDRESS>, which depends on
 actionview was resolved to <IP_ADDRESS>, which depends on
 rails-dom-testing was resolved to 1.0.7, which depends on
 rails-deprecated_sanitizer was resolved to 1.0.3, which depends on
 activesupport was resolved to <IP_ADDRESS>, which depends on
 json
The command "eval bundle install --jobs=3 --retry=3 --deployment " failed. Retrying, 2 of 3.

Hey @tophernuts, this turned out to be kind of gnarly because it involved upgrading quite a few dependencies in order to work. #56 got it working, so I merged that branch instead. Thanks for opening this PR!


Nuke Gizmo publishing

 Freshdesk Ticket ID: 267
 Freshdesk Ticket Agent: Jakub Jezek
 Freshdesk Ticket Agent Email<EMAIL_ADDRESS> Ticket Priority: Medium
 Freshdesk Ticket Description:
- publishing of a Gizmo subset is having some issues

Agent Jakub Jezek linked Freshdesk ticket 267 for this issue.


Config volume at 55% shows 54% in UI

Describe the bug
When I set my config's volume to 55 which is comfortable for me, the app launches at 54% volume. I thought "oh well maybe it's offset for some reason" so I set it to 56 in the config, and the app launches with 56% volume... very odd indeed.
To Reproduce

Set config volume to 55
launch app
volume is 54%

Expected behaviour

Volume should be 55% in the app

Log and backtrace
spotify-player-24-07-31-17-58.zip
Screenshots

Environment

OS: Windows 10
Application version: git latest (built from source at this commit)
Application features: image,lyric-finder,media-control built with --release on

Additional context
Add any other context about the problem here.

Likely because how volume is set in https://github.com/aome510/spotify-player/blob/fdb846e86e040b7d6ea45b0f578dd1ce7abd0008/spotify_player/src/streaming.rs#L171. It might be easy to tweak the calculation so that the percentage is right

I don't think it can be fixed due to librespot's volume needing to be a u16 number like that. It's a rounding error as 65535 * 0.55 is not a whole number

One way that could "fix" it is to display the volume percentage in the UI as whatever its set in the config lol

I don't think it can be fixed due to librespot's volume needing to be a u16 number like that. It's a rounding error as 65535 * 0.55 is not a whole number

right.

One way that could "fix" it is to display the volume percentage in the UI as whatever its set in the config lol

The volume is what received from Spotify API, so it cannot be changed based on the configs

The displayed volume is what received from Spotify API, so it cannot be changed based on the configs

Ah yes, I assume this would be a problem if you used the app to adjust the volume for other applications and of course as the user changes the volume using the hotkey it wouldn't update the config's value accordingly and this would still show 55%, bummer! Not sure what to do about this then other than suck it up :P

Actually, on this line in the client the volume is an i8, I think whats happening here is the UI is displaying this value AFAIK and instead of rounding (36044/65535 = 0.5499961852445259 rounded up would be 55%), it's chopping the decimals off making it "floor" the value and display 54% (again as far as I can tell)
https://github.com/aome510/spotify-player/blob/314631f9ce74e6449c929ca94d8e0ca553dd97c1/spotify_player/src/cli/client.rs#L404

But I don't fully understand the is_offset check so idk what that code block does if its not offset or what constitutes that flag being set

Further digging revealed what you said is true (not that I doubted it, just wanted to know more). Rspotify returns the volume_percentage which is a u32 meaning our decimals are lost somewhere outside our control :(
Gonna close this as afaik there is nothing that can be done to fix this


Fix CMake config if used as a subproject

With this fix we don't assume that the tensorflow project is the build
root. Instead CMAKE_CURRENT_SOURCE_DIR/CMAKE_CURRENT_BINARY_DIR is used
instead of just CMAKE_SOURCE_DIR/CMAKE_BINARY_DIR, which now builds
successfully, if it is built inside another, larger project.

Can one of the admins verify this patch?

@tensorflow-jenkins test this please.

Seems that the one failing test has nothing to do with my PR?

Yes, the test failure looks unrelated. I'll merge the PR now.

Can one of the admins verify this patch?