Jit bootstrap fixes

Here is a pr to fix the #1319 bug. reset_context can if you are unlucky gc. It allocates 48 bytes and the nursery is 2mb so the odds of it triggering a gc cycle is really low. But if it happens, the two top stack items must be preserved so they can be traced by the gc and the function previously didn't do that.

It seems to show the ui and then crash on Windows 64. Have you tested there?
On Fri, May 22, 2015 at 11:48 AM, Björn Lindqvist<EMAIL_ADDRESS>wrote:

Here is a pr to fix the #1319
https://github.com/slavapestov/factor/issues/1319 bug. reset_context
can if you are unlucky gc. It allocates 48 bytes and the nursery is 2mb so
the odds of it triggering a gc cycle is really low. But if it happens, the
two top stack items must be preserved so they can be traced by the gc and
the function previously didn't do that.
You can view, comment on, or merge this pull request online at:
https://github.com/slavapestov/factor/pull/1323
Commit Summary

Reorganising bootstrap files
bootstrap.x86: new words jit-call-Xarg for calling vm functions with
VM: the delete_context() and reset_context() api functions can be
threads.test: a test that is mostly able to reproduce #1319
VM: possible fix for #1319.
VM: after reset_datastack and retainstack clear the stack segment.
makes
bootstrap.ppc: untested fix for #1319 for ppc
cpu.x86.*: %mov-vm-ptr isn't so useful, shorter and simpler to write
bootstrap.primitives: shorter code for mapping architecture to
assembly file

File Changes

R basis/bootstrap/assembler/ppc.32.linux.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-0 (2)
R basis/bootstrap/assembler/ppc.64.linux.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-1 (2)
R basis/bootstrap/assembler/ppc.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-2 (12)
R basis/bootstrap/assembler/x86.32.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-3 (63)
A basis/bootstrap/assembler/x86.32.unix.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-4 (8)
R basis/bootstrap/assembler/x86.32.windows.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-5 (6)
R basis/bootstrap/assembler/x86.64.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-6 (54)
R basis/bootstrap/assembler/x86.64.unix.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-7 (9)
R basis/bootstrap/assembler/x86.64.windows.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-8 (6)
R basis/bootstrap/assembler/x86.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-9 (0)
R basis/bootstrap/assembler/x86.unix.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-10 (0)
R basis/bootstrap/assembler/x86.windows.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-11 (0)
M basis/cpu/x86/32/32.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-12 (3)
D basis/cpu/x86/32/unix/bootstrap.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-13 (8)
M basis/cpu/x86/64/64.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-14 (11)
M basis/cpu/x86/x86-docs.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-15 (12)
M basis/cpu/x86/x86.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-16 (2)
M basis/threads/threads-tests.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-17 (28)
M core/bootstrap/primitives.factor
https://github.com/slavapestov/factor/pull/1323/files#diff-18 (29)
M vm/contexts.cpp
https://github.com/slavapestov/factor/pull/1323/files#diff-19 (36)
M vm/contexts.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-20 (5)
M vm/cpu-x86.32.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-21 (2)
M vm/os-linux-x86.64.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-22 (2)
M vm/os-macosx-x86.64.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-23 (2)
M vm/os-windows-x86.32.cpp
https://github.com/slavapestov/factor/pull/1323/files#diff-24 (2)
M vm/os-windows.64.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-25 (3)
M vm/quotations.cpp
https://github.com/slavapestov/factor/pull/1323/files#diff-26 (7)
M vm/slot_visitor.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-27 (20)
M vm/vm.hpp
https://github.com/slavapestov/factor/pull/1323/files#diff-28 (2)

Patch Links:

https://github.com/slavapestov/factor/pull/1323.patch
https://github.com/slavapestov/factor/pull/1323.diff

—
Reply to this email directly or view it on GitHub
https://github.com/slavapestov/factor/pull/1323.

I done goofed. I forgot to bootstrap with a new boot image.
Pulled. Closing.


Document the choices of OTA operations

Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[x ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[ ] Run spelling and grammar check, preferably with linter.
[ ] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[ ] Review for wordiness
[ ] Match tone and style of page/section.
[ ] Run make linkcheck.
[ ] View HTML in a browser to check rendering.
[ ] Use semantic newlines.
[ ] follow best practices for commits.

[ ] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[ ] End message with sign off/DCO line (-s, --signoff).
[ ] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for 9e0a9a8c1761cb9a6ecb15c75303df24e0ec11bb are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2576/docs/artifacts/html/index.html

Thank you @Tim-Anderson for the help with this. I would say to remove CLI stuff until we have a chance to discuss with Mike/Mike Sul. Callbacks docs is what we are lacking at the moment.

Docs for ae95aac86d0a44ffdf34d9f8353f95fc4aa9e658 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2580/docs/artifacts/html/index.html

@Tim-Anderson What are we missing on this?
@kprosise @foundriesio/customer-success Do you think it's a good idea to land this on v93?

@vanmaegima I have no problem including this in v93 if it is ready then.

Docs for 91074ce140ef493a1e0643f02b62ccf511aeba47 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2624/docs/artifacts/html/index.html

LGTM


service: deploy: Create webhook receiver

Depends on: #532
We need to make an HTTP server (similar to how the HTTP service works) and create a webhook receiver for GitHub at the /github path.
When the webhook is hit it should call a method named reload() which will be responsible for pulling down the latest version of the Git repo.
First it need to run
https://github.com/intel/dffml/blob/36c12e263f7fcba4da1e4472e35ebbcf4f70df59/dffml/service/dev.py#L403-L414
to ensure that there's nothing that will be overwritten
Then it needs to run git pull if the branch the webhook mentioned is the currently active branch.

For the first pass on this let's just add to the existing HTTP service.


[Feature]: Shrink rocm-llvm distribution packages by not statically linking libLLVM, libclang-cpp etc.

Suggestion Description
For the various supported systems, the rocm-llvm rpm/deb packages are about 1GB of total download size and about 4GB in installation size. The main reason for this seems to be that all llvm binaries were statically linked with libLLVM, libclang-cpp and so on. That means multiple binaries all have their own copy of libLLVM (~130MB), libclang-cpp (~50MB). It also seems they contain superfluous symbols, since they shrink a bit when applying strip on them. The installation also includes static libraries used for linking to llvm/clang libraries.
This makes downloading and installing the ROCm stack unnecessarily cumbersome, specifically when building base containers for development.
Is there a technical reason for the static linking of LLVM? If not, the suggestion would be to make the installation use shared libraries and appropriate RPATHs instead. Debug symbols and static libraries, if needed, should be moved into something like rocm-llvm-dev and not be part of the rocm use case in the installer.
Operating System
No response
GPU
No response
ROCm Component
rocm-llvm

Hi @rbberger, this feature is somewhat complete, but disabled for now. There is a regression in one of the OpenGL conformance tests (issue with registering llvm options) when we enable dynamic linking.
I will try and find out the status of enabling it.

@kzhuravl as an intermediate step you could at least separate the static libraries/headers that get currently shipped as part of rocm-llvm from the rest of the installation and put them in a development package such as rocm-llvm-dev.

@rbberger, package split is implemented internally and going to be included in 6.1. We have split llvm into 2 packages: package 1 (rocm-llvm) includes everything needed for compilation. package 2 (rocm-llvm-dev) includes everything needed for compiler and application developers. If you need, I can share package contents for each package.

Great! That's a start. Don't need the details, I trust you do something reasonable. :wink: Thx for the insight.

Fixed in ROCm 6.1.0


[BEAM-5122] Increase timeout in PubsubJsonIT:testUsesDlq to try workaround pubsub latency.

There was couple more failures due to this test today.
Try increase timeout significantly to check if flake will go out. This is a temporary measure.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


add #html_attr_name method

just a handy helper to add the ActiveRecordBase (on the client)  (it would jus be added to InstanceMethods module)
  def html_attr_name(attr)
    "#{model_name}#{to_key}-#{attr}"
  end

See https://github.com/ruby-hyperloop/hyper-react/issues/244 for details on to_key

This seems less useful than to_key unless for and name params do some magic, which I think they probably shouldn't. As there's no obvious "right" way of naming for/name in HTML (there's probably multiple competing conventions like there are with BEM/OO CSS etc), so I guess many would override this anyway, so we could just leave them to add it in ApplicationRecord themselves.
If Hyerloop had its own form_builder replacement smart input components then this would be a part of it.
But a one size fits all form builder is hard. The Rails one isn't flexible enough to work with frameworks like Bootstrap, so then there was SimpleForm but that's nowhere near simple, I hate it. I'd suggest holding off on this issue until we have a more concrete use-case for it.

are the conventions used by any code?  As in would I ever "have" to use the convention, or is it just simply a convention to keep things tidy.  In which case it seems like it might be helpful. but I dunno.  We can drop it from 1.0 if all agree...

See #245 for how I use this in a smart form input component. Simply some human readable and useful yet unique identifier to link name and from


CI/Testing issues

test_chmod::test_chmod_ugoa seems to fail sometimes on Rust 1.26 for some reason.  Additionally, the sleep tests can easily fail if the system takes too long to wake from sleep (or if threads take too long to spawn).

This issue is mitigated by relaxing the sleep diff.


Missing dependency to playcanvas.Vec2

I've tried to get the library to run for quite some time now.
I was successful at building and adding it to my html page via a script tag.
This is the error i get, when trying to create a new Graph:
Uncaught TypeError: Cannot read properties of undefined (reading 'Vec2')
    at new JointGraph (pcui-graph.js:40913)
    at new GraphView (pcui-graph.js:44975)
    at Graph._buildGraphFromData (pcui-graph.js:45567)
    at new Graph (pcui-graph.js:45543)
    at (index):63

On further inspection it looks like the library is referering to the class playcanvas.Vec2 which is not included in the library itself.
I guess this is the class Vec2 from the playcanvas/engine project?

Yeah, IMHO, PCUI-Graph shouldn't have a dependency on the PlayCanvas Engine. @ellthompson and I have already had a chat about this. We'll resolve it when we have a moment.


Not going to upgrade to IPB4, but have ssl Problem.

Before you ask lol, I will not be upgrading to IPB4 (probably would fix all my problems), as I paid a designer for a custom theme some time ago, and don't see it as a wise decision atm.
Anyways, My problem is the result of changing the entire site to HTTPS via a cloudflare proxy. The site itself works flawlessly, however my login methods are troublesome. Steam Authentication works normally, until after a user signs into steam, upon the 'sign in' (after entering credentials) the page loads, and doesn't move forward. It's almost as if it isn't connecting to the board. (started after changing to ssl)
Any suggestions? / easy fixes?
Currently running 1.1.2, with IPB 3.4.6 & Cloudflare: flexible ssl (always use https page rule)

Is it keeping the user on the steam login page?
Also would still suggest 4.x as that is what IPS is now fully supporting
and they only do security patches for 3.x.

Yes, It loads briefly, and keeps the user on the steam login page, instead of redirecting to the board. I would rather live without steam login than upgrade at this point lol.

I have a feeling on what it is. But I'm not home atm.

@Napsterlove sup bud, hit me up on steam ; would need FTP and ACP.  http://steamcommunity.com/id/madman32395

Sent an invite, awaiting acceptance

Found client setup issue, forgot to turn on SSL with login, as rest of site is https. this was causing openID to freak out as it came from a https site but wanted to return to a http site. openid wasnt having none of that. Ticked on Login with SSL. but then further server issues appeared. @Napsterlove would have to correct redirect loop issue on their server first. Might be a Cloudflare issue. Suggested using Lets Encrypt as a replacement for their SSL certs.


Disallow token change via SetParent after a command has been deserialized

This PR fixes an issue where a command, after deserialization, can have its parent set again, overwriting the original token and breaking scheduler correlation.

Should we have changes in typescript too?

#3396 will align js to this


Feature/translation

This isn't most efficient resolution, but should work.
I have my eye on it in case of future performance problems


talon_summarize but on filtered results.

Is there a similar tabulation like talon_summarize but on the filtered results in the talon_abundance outputs? I would like to get an idea of the classes of transcripts discovered for each of the datasets but not counting things that are filtered out.

Hi,
Currently talon_summarize is not compatible with filtering pass lists. I typically calculate the same sort of statistics by generating a filtered abundance file and using table operations in Python or R to count up the different classes of transcripts or genes.

thank you for your reply and help


[Feat]Added background animation in contributors page.

Closes
Closes #1564

Description
Added background animation in contributors page.

Screenshots

Note to reviewers

Checklist:

[X] I have linked the PR to the correct issue.
[X] I have read the Contribution Guidelines
[X] I have built and tested the changes, and they do not break or show any errors.

@Vaishnavi-Patil2211 Pleas check and merge it !


pull request for e2e test at 12/09/2016 09:26:02

:white_check_mark: Validation status: passed
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.


CI part

Add the CI build framework which is Pip to the project .

https://github.com/BME-MIT-IET/ivt-hw-2023-mega-bite/pull/8


Preserve comments on elided import declarations

Fixes #26079

Ping @rbuckton do you think this approach is reasonable?

This PR hasn't seen any activity for quite a while, so I'm going to close it to keep the number of open PRs manageable. Feel free to open a fresh PR or restart the discussion here.


the first time importing a dat.json file doesn't populate the list with metadata

If you have a dat.json file in a folder and then create a new dat with it, the dat.json file never gets read -- it says 'anonymous' author with the truncated key in the title spot.
But if you restart the app, you'll see that it's able to read the dat.json file from the manifest and displays the expected title and author.

depends on https://github.com/datproject/multidat/pull/8

pull request here: https://github.com/datproject/dat-desktop/pull/316


Refactor dataframe where tests, reduce IF conditional compilation blocks

Split up dataframe read with where test to try and probe at #384
Also tries to remove IF blocks marked as deprecated by Cython, except for those still needed for GDAL version specific functions.

After splicing in changes from #349 to correctly release the Arrow stream, I'm no longer able to easily reproduce the segfaults on repeated reruns of CI.  Given that these were intermittent, it's hard to say for certain, but per the GDAL docs, we should have been releasing the stream:

On successful return, and when the stream interfaces is no longer needed, it must must be freed...

Clarification, it seems that the stream.release is not being called (is NULL) - presumably because the stream was used in the pyarrow reader?  But free(stream) is getting called.  Not sure why either of these would avoid an unpredictable segfault, it seems more likely that initializing the .release method to NULL on the ArrowArrayStream prevents calling a random pointer (per the comment), even though the .release method was not defined prior to these changes (maybe something expecting that interface was expecting a .release anyway).

I merged https://github.com/geopandas/pyogrio/pull/349, so let's see if the failures now disappear on main and other PRs.
Although to be honest I also don't fully understand how the changes would fix the crashes ..

Clarification, it seems that the stream.release is not being called (is NULL) - presumably because the stream was used in the pyarrow reader?

Indeed, that's the idea. When we pass the struct to pyarrow, they will "move" it and take ownership of it (i.e. being responsible of releasing it when done reading the data) by setting the release callback to NULL, to ensure we don't call that anymore.
(https://arrow.apache.org/docs/dev/format/CDataInterface.html#moving-an-array)

Hmm, still a crash on the merge commit on main .. https://github.com/geopandas/pyogrio/actions/runs/8718790747/job/23916758253


[vcr-2.0] Impl findMissingkeys

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

:exclamation: No coverage uploaded for pull request base (fix-vcr-ttl@97b709e). Click here to learn what that means.

Additional details and impacted files
@@              Coverage Diff               @@
##             fix-vcr-ttl    #2596   +/-   ##
==============================================
  Coverage               ?   32.61%
  Complexity             ?     4463
==============================================
  Files                  ?      805
  Lines                  ?    64977
  Branches               ?     7927
==============================================
  Hits                   ?    21194
  Misses                 ?    41559
  Partials               ?     2224

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


link hearings page to hearings on the calendar

Please do work for this task in a branch called issue-XXX.

work done on branch 222


Throw InvalidServiceIdentifierException when extending a protected closure

Fixes #190 by adding a test that checks if the entry you are trying to extend is a protected closure.

Thank you @skalpa.


Is this a bug...? Is this a dream...?

Versions:

ide-helper Version: 3.1.0
Laravel Version: 11.19.0
PHP Version: 8.3.7

Question:

I don't know if this is a bug or my mistake, thanks to the laravel-ide-helper which is getting weirder and weirder as I go.
Over time, the IDE will recognize this as an error, even if it was previously fine.
In the screenshot above, we see that cookie don't have make and forever methods.

This has nothing to do with laravel-ide-helper, which does not annotate this kind of laravel utils.
It doesn't need to because they have the correct annotations:
     * @return ($name is null ? \Illuminate\Cookie\CookieJar : \Symfony\Component\HttpFoundation\Cookie)

That might be an issue of your ide not understanding the $name is null ? phpdoc

This has nothing to do with laravel-ide-helper, which does not annotate this kind of laravel utils.
It doesn't need to because they have the correct annotations:
     * @return ($name is null ? \Illuminate\Cookie\CookieJar : \Symfony\Component\HttpFoundation\Cookie)

That might be an issue of your ide not understanding the $name is null ? phpdoc

Thanks for the answer.
However, my IDE at the time confirmed that there was no problem reading the phpdoc.
I don't know why, but this works again.
I'm not sure, but I expect it to be a temporary error in the IDE.


Add client upload use case to Resumable Upload with tempauthtoken

The following is mentioned in the docs:

Including the Authorization header when issuing the PUT call may result in a HTTP 401 Unauthorized response. The Authorization header and bearer token should only be sent when issuing the POST during the first step. It should be not be included when issueing the PUT.

The uploadUrl that is returned in the POST request looks like this:
https://MYDOMAIN.sharepoint.com/sites/MYSITE/_api/v2.0/drive/items/MYITEM/uploadSession?guid='GUID'&path='~tmpDE_testFile.pdf'&overwrite=True&rename=False&dc=0&tempauth=eyJ0e....
As you can see in the URL, there is a tempauth token (JSON Web Token), so anyone with this link can upload file chunks to this URL (until the URL expires). This is convenient in a scenario where the server requests an upload session/URL from Microsoft Graph, then sends this URL including its tempauth token to the client (e.g. a JavaScript front-end application) which can then upload the actual file to OneDrive. So, instead of
Client --> Server --> OneDrive
the upload will be
Client --> OneDrive
... which reduces overhead and often increases speed of the file transfer, since the developer's server doesn't need to be in between anymore. Please consider adding information about this scenario to the docs. Thanks in advance!

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

ID: c42d74d1-8c6f-5282-9fba-30fdd6449362
Version Independent ID: 88bd8f18-730d-6970-8a02-641346661404
Content: Resumable file upload - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-createuploadsession.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: JeremyKelley

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


regarding issue #54

I have added a scroll to the top button code.
It closes issue #54

Please add hacktoberfest label

star the repo

done⭐


Meta: Allow WPT.sh to import more tests

Specifically, this PR was written to add the ability to import the tests imported in a98d277f9f25cbb74ee6d2f76e1f0cb9ef44ba93.
I also tested my changes on a few others tests, and decided to commit them in 77d01940b667a6e682eb4c2b4e69dfb93d4298c8.

Does this fix https://github.com/LadybirdBrowser/ladybird/issues/2690?

Also, could you please try https://github.com/LadybirdBrowser/ladybird/pull/2216#issuecomment-2504706000 ?

No it doesn't. These are unrelated issues.
Also, instead of commenting on PR to ask people to do stuff for you. Can't you test the thing yourself and potentially avoid useless noise? These two PRs are now linked forever to this one for no good reasons.


Build and Deploy Image

This issue will be used to implement the following solution to https://github.com/bailey-lab/MIPTools/issues/9:

Solution: Manually build and upload the container to Github packages the Sylabs Cloud. For this, we can use Seekdeep, which is able to build the image.
Limitations: We need to manually build the container rather than automating the process.

In particular, every two weeks, a bot will comment on this issue to remind the maintainers to build the container and upload it to the Sylabs Cloud.
Pulling the container will then be as easy as running the following:
singularity pull library://apascha1/miptools/miptools:{tag}

The latest stable release (v0.4.0) and the dev version of MIPTools are now on the Sylabs Cloud.
You may pull the container with:
singularity pull library://apascha1/miptools/miptools:{tag}

There have been no changes in the past two weeks so closing this issue for now. For the time being, we will change the frequency of these reminders to be once every month.

Due to storage limitations on Sylabs Cloud, we have now decided to store our containers using an HTTP directory found here. In doing so, we must make some changes to the documentation and our Github workflow, which notifies maintainers on how to build and deploy the container.

[ ]  Update documentation to point to new storage location.
[ ] Update Github workflow with new instructions on building and deploying the container.

There have been no changes in the past month so closing this issue for now.


Set default GC card size to 251 blocks

Fixes #1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.966%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
120

Relevant Lines:
174

💛  - Coveralls


Loading model with low accuracy

Hi!
I found that when I load the model back using autoencoder.load_state_dict(torch.load(ae_path)) etc., it leads to low accuracy (around 0.3) even though the model achieves around 0.8 accuracy in evaluate_autoencoder during training.
I have to use torch.load(ae_path) and torch.save(autoencoder, f) to get away this problem.
Probably it is a pytorch bug, as discussed here: https://discuss.pytorch.org/t/saving-and-loading-a-model-in-pytorch/2610/6

@posenhuang Did you do model.eval() after loading the model to disable dropout?

Yes, I did, but it it still the same. I'll double check again.

@posenhuang  I met the similar problem. Did you solve it?


cmsisdap: Stackoverflow

Same issue as months ago from here https://github.com/probe-rs/probe-rs/issues/531 I was checking back into, but that issue wants to be for just jlink
Mainly im attempting to use vscode and probe-rs-debugger on embassy. A simple example like this works without embassy, but presumably async stuff is deep enough to trigger?. Uses an nrf52840-mdk which has a cmsis dap on board
https://github.com/jacobrosenthal/blinky-button-demo/blob/nrf52840-mdk/src/main.rs#L24
Trace logs here
https://gist.github.com/jacobrosenthal/921f5306e01e3ef83db36c46e3c157d1
probe-rs-debugger 0.12.0
vscode debugger 0.3.3

I will test this against #893 to make sure we catch this as well. Let's keep this Issue open for now until we know for sure.

@jacobrosenthal Are you able to test this with the VSCode extension and probe-rs-debugger branches I just uploaded? (details in https://github.com/probe-rs/vscode/issues/26#issuecomment-1016870363)

Yeah it looks good!

So I guess this is a different stackoverflow bug. But I confirmed that this fixes embassy programs for both cmsis and jlink

Closing as fixed.


[Udilgram] Line Feed fix URGENT

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Please merge this pull request, someone uploaded the files directly to the repo with bad line feeds and it's breaking my repo because of the recommended GIT settings of using \n for the line feed.

Character Sheet Info Roll20 Internal Use only.

Thank you very much!


Allow external process to signal file addition

Describe the enhancement you'd like
A clear and concise description of what you want to happen.
The ability to call the backend with a file path and any new parameters that are required.
Describe why this will benefit the LibrePhotos
A clear and concise explanation on why this will make LibrePhotos better.
A full scan takes quite some time, but if I'm uploading a file then there are much faster ways of detecting a file had been recently uploaded.
If this  was possible then once a new file is found, calling a librephotos API to register the new file would make it appear in the photos section much faster. Also works not cause the same level of resource use disk io, cpu etc etc
This is similar to how Google photos appear instantly.
Additional context
Add any other context or screenshots about the enhancement request here.
There are many things to watch for directory changes e.g
https://github.com/samuelcolvin/watchfiles
https://github.com/devilbox/watcherd

alternatively, if there is API in librephotos to scan specified file(s), we can use inotify to monitor the file changes, and call the API once changes (addtions, deletions, etc) detected.

seems the api already supports to scan specified file(s):
python3 manage.py scan --scan-files


How high is the power consumption of a trailcam?

Spec how much power is required for a trailcam.

On the trail cam website it says that the trail cam is 12v so about 24w. So we would need around a 50w solar panel.

Any updates?


Capture all cookies and 2fa token

Is there a way to capture all issued cookies using evilgnx?
Similarly, is it possible to capture multiple password fields for 2fa?
Thanks

I see the cookie regex part has been implemented already: #44

What do you mean by multiple password fields?
Can you provide an example of such login page?

Well, 2FA pages often have a password field and a token field.
I managed to capture both by using regex for the password field and patching the code like this:
s.Password += " --- " + password

So that I get both the password and token in the password field.
It's a bit hacky but does the trick.

Added with 2.2.0 update.


core: avoid referencing application classes in bean build items and loading them in core build processors

Processors should avoid loading application classes during augmentation [1] so the reference to
beans class has been removed from Camel*BeanBuildItem and replaced by a simple string. I will
follow up this PR with an additional one after reviewing how Camel*BeanBuildItem are used by
extensions.
[1] https://quarkus.io/guides/writing-extensions#bootstrap-three-phases

ok to test

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/397/Build result: FAILURE[...truncated 1.45 MB...]    at sun.reflect.GeneratedMethodAccessor255.invoke (Unknown Source)    at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)    at java.lang.reflect.Method.invoke (Method.java:498)    at io.quarkus.deployment.ExtensionLoader$1.execute (ExtensionLoader.java:941)    at io.quarkus.builder.BuildContext.run (BuildContext.java:415)    at org.jboss.threads.ContextClassLoaderSavingRunnable.run (ContextClassLoaderSavingRunnable.java:35)    at org.jboss.threads.EnhancedQueueExecutor.safeRun (EnhancedQueueExecutor.java:2011)    at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.doRunTask (EnhancedQueueExecutor.java:1535)    at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run (EnhancedQueueExecutor.java:1426)    at java.lang.Thread.run (Thread.java:748)    at org.jboss.threads.JBossThread.run (JBossThread.java:479)[ERROR] [ERROR] Re-run Maven using the -X switch to enable full debug logging.[ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles:[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException[ERROR] [ERROR] After correcting the problems, you can resume the build with the command[ERROR]   mvn  -rf :camel-quarkus-integration-test-validatorchannel stoppedAdding one-line test results to commit status...Setting status of b29607c154f4451565cccd9339fa5604eb5fb58e to FAILURE with url https://builds.apache.org/job/camel-quarkus-pr/397/ and message: 'FAILURE 200 tests run, 4 skipped, 0 failed.'Using context: default

+1 for the goal, I also thought of doing it. Passing a class proxy instead of the class did not work for me on this place: https://github.com/apache/camel-quarkus/pull/402/files#diff-ec6205752018e84a80fcd9eaa92af189R320 so it probably won't work for you here: https://github.com/apache/camel-quarkus/pull/405/files#diff-ec6205752018e84a80fcd9eaa92af189R191 It might well be a bug in Quarkus.

It should be fine now as the way it is used matches the pattern for which the class proxy is needed. In your code I suspect the problem was the fact that the proxy was owned by a build item and maybe triggering a bug.
The failure on CI is caused by the validator extension, don't know why.

ok to test

the proxy be owned by a build item

That sounds like a valid hypothesis. Many thanks for mentioning it!

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/398/Failed Tests: 3camel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-scheduler: 2org.apache.camel.quarkus.component.scheduler.it.SchedulerTest.testorg.apache.camel.quarkus.component.scheduler.it.SchedulerIT.testcamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-xslt: 1org.apache.camel.quarkus.component.xslt.it.XsltIT.test

@davsclaus ass the pr is failing for unrelated failures I'd go for merging it.
About XSLT, I've raised an issue https://github.com/apache/camel-quarkus/issues/419 and I'm working on a substitution

Increate the CI timeout for that test, as it seems it took a bit more than 2 sec for replying


clean up Quick/Nimble tests, update gitignore

Also:

Updatecodecov.yml
Update .gitignore

Codecov Report

Merging #48 into master will not change coverage.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master      #48   +/-   ##
=======================================
  Coverage   100.0%   100.0%
=======================================
  Files           3        3
  Lines         229      229
=======================================
  Hits          229      229

Impacted Files
Coverage Δ

Sources/MultiProgressView/MultiProgressView.swift
100.0% <100.0%> (ø)

...ources/MultiProgressView/ProgressViewSection.swift
100.0% <100.0%> (ø)


[PE-D][Tester D] Notes for students with the same name act weird

So like turns out if you have 2 students with the same name trying to add a note or delete a note for one of them makes/deletes notes from the other as well, and same problem with participation and attendance

Example for the add note one:
Your browser does not support the video tag.video:https://raw.githubusercontent.com/reneeyeow02/ped/main/files/00b8860e-e6e4-4f4d-8686-a36c97179bd7.mov

Labels: severity.Medium type.FunctionalityBug
original: reneeyeow02/ped#5

Duplicate of #110


Unable to run osxphotos successfully

**Before submitting a bug report, please ensure you are running the most recent version of osxphotos and that the bug is reproducible on the latest version **

If you installed with pipx: pipx upgrade osxphotos
If you installed with pip: pip install --upgrade osxphotos
If you installed the pre-built binary, download and install the latest release

OSXPhotos is upgraded to the latest version.
Describe the bug
A clear and concise description of what the bug is.
When running a command, whether or not I specify the library path using --library or --db, or letting it default, the command fails and errors out, but have tried various differences in the commands, but can never get it to work.
To Reproduce
Steps to reproduce the behavior:

What' the full command line you used with osxphotos?
 The PHOTOS_LIBRARY argument is deprecated and will be removed in a future version of osxphotos. Use --library instead to specify the Photos Library path.
WARNING: photos_library is deprecated. The PHOTOS_LIBRARY argument is deprecated and will be removed in a future version of osxphotos. Use --library instead to specify the Photos Library path.
Usage: osxphotos export [OPTIONS] [PHOTOS_LIBRARY]... DEST
Try 'osxphotos export -h' for help.

Error: Invalid value for '[PHOTOS_LIBRARY]...': Path '3' does not exist.

Expected behavior
A clear and concise description of what you expected to happen.
A successful run of osxphotos
Desktop (please complete the following information):

OS: 11.7.4
osxphotos version 0.58.2

Thanks, appreciated!

You're missing the -- before filename so osxphotos thinks filename is the photos library

You're missing the -- before filename so osxphotos thinks filename is the photos library

Ah, nice catch. However, tried again and is still errors out

Nevermind - worked after adding a path to --exportdb
Thank you!


unexpected result when pasting (template seems to be partially ignored)

given the name of this extension, i suspect this request may be outside its scope, but i'm really hoping it isn't ...
one of the things i need to do is copy a linked page title and article excerpt - for this i'm a template similar to the following:

{{title}}
\n{{text}}
if the {{text}} part has a link in it however, it seems all the selected text is ignored except for the link text and the {{url}} used for the page title is not the page URL

I'm afraid it is the expected behavior for the {{url}} variable.
Maybe we can add another {{page_url}} variable which always expands to the page URL.

that would be super!
will that also allow copying/pasting all of the selected text (rather than just the link text)?

will that also allow copying/pasting all of the selected text (rather than just the link text)?

No, but I fixed it at https://github.com/hnakamur/FormatLink-Firefox/pull/37/commits/0b042433fc567d288622aefb9b1c1152b838d455 in https://github.com/hnakamur/FormatLink-Firefox/pull/37
I just submitted version 5.1.0, you can try it once published.

As for the {{url}} variable, it expands to the page URL if you open the context menu outside of links after selecting text containings links.

actually 'outside of links' doesn't seem accurate - you have to open the context menu while hovering completely outside of any selected text that contains a link for {{url}}} to expand to the page URL
it is a bit awkward to have to click off of the selected text, but it does what i need it to do now so thanks!  i think that adding a {page_url} variable might still be worth while though - i'll let you close this since i don't know what you want to do

I already added {{page_url}} variable in version 5.1.0 which is published by now.
Could you try that?

sorry, i didn't realize that ... and yes, it works fine - thanks


Official ansible-test binary name collision

Hello! Ansible Core Team member here.
https://github.com/ansible/ansible has a tool called ansible-test. It used to be development-only but a few major versions back it started being exposed as ansible-test executable and now if somebody will do pip install ansible-test ansible, they'll conflict trying to overwrite each other.
Our official ansible-test is a test runner that we use in our own CI for running various sanity, unit, and integration tests. Besides that, it's an official solution to running tests in end-users' Ansible Collections. In particular, it supports running tests in docker containers as well as in the host environment.
There's a different type of testing that's not covered by the official ansible-test but is covered by another active community project called Molecule. It allows running roles against automatically orchestrated envs, including container-based, just like this project.
Looking at the activity of this repository, it seems to be unmaintained for almost 3 years. It only seems fair to recommend users to use the official tooling from Ansible Core and Molecule projects.
There's one problem, though: when folks hear about the need to use ansible-test, they often end up in this dead project while they are expected to use the official one. This creates some amount of friction as it is misleading.
@jerm @grinich @rcoh @SJLC @spang @woohgit would you be open to migrating this project to a different name on GitHub and PyPI to help reduce the confusion this collision creates?

cc @mattclay I think we should file a PEP541 request to take it down from PyPI

FTR since nobody here seems to notice the attempt to contact here, I've submitted a PyPI support ticket asking to give us access to https://pypi.org/p/ansible-test: https://github.com/nylas/ansible-test/issues/20.

FTR I've got the ownership over the PyPI name. Ref https://github.com/pypa/pypi-support/issues/892#issuecomment-933319912. Unsubscribing now.


Card tracker to use custom folders

Allow card tracker to use custom folders. This would cover the case for users that just wish to git clone the complete custom-lovelace in their www
This probably needs an implementation in https://github.com/custom-components/custom_cards

Added a pr to fix this https://github.com/custom-components/custom_cards/pull/11

This is corrected in https://github.com/custom-components/custom_cards/pull/11 which is now merged and is a part of v.1.1.9 of the component.


Add Github Action check for PR title to make sure it is semantic

PR titles should be semantic.

@sarabala1979 @whynowy @juliev0 @dpadhiar @terrytangyuan @jessesuen as you know, all commits to master (a) must be semantic (i.e. prefix feat: fix: etc (b) should have the issue suffix (i.e. Fixes #8622).
Commit messages to master are the PR title.
OSS contributors usually fail to get it right. Even when they get it right, I often want to change what they wrote to clarify what changed and why.
Therefore, I don't think we should automated this.


Space isn't cold

Description

Venting an area to space only reduces its pressure, not temperature
Going into space doesn't make you take temperature damage, only pressure damage

Misleading title, space is cold

weird, when you go into space the cold alert doesn't show up, and space never cools down areas exposed to it

To be fair. Space is excellent insulation. Unless you emit radiation.

game design-wise it should definitely be cold though


feat(callhome): add volume replica and state, pool, spdk managed disks and storage media metrics

Added volume replica and status metrics to callhome report
{
  "volumes":
      {
           "volume_replica_counts": {
              "one_replica": 1,
              "two_replicas": 4,
              "three_replicas": 0,
              "four_replicas": 0,
              "five_or_more_replicas": 0
           },
           "volume_state_counts": {
                "unknown": 0,
                "online": 5,
                "degraded": 0,
                "faulted": 0,
                "shutdown": 0
            }
     }
}

Added pool metrics per node to callhome report
"storage_nodes": {
    "nodes": {
      "node-1-153835": {
        "pools": [
          {
            "replicas": 0,
            "replicas_capacity": 0
          }
        ],
        "mayastor_managed_disks_count": 1
      },
      "node-0-153835": {
        "pools": [
          {
            "replicas": 0,
            "replicas_capacity": 0
          }
        ],
        "mayastor_managed_disks_count": 1
      }
    }
  },

Added mayastor managed disks to callhome report
"mayastor_managed_disks": {
    "disk_types": {
      "SSD": {
        "count": 2,
        "capacity":<PHONE_NUMBER>0
      }
    }

Added storage media metrics to callhome report
"storage_media": {
    "count": 9,
    "total_capacity_in_bytes":<PHONE_NUMBER>12,
    "disk_types": {
      "SSD": {
        "count": 6,
        "capacity":<PHONE_NUMBER>76
      },
      "Optical Drive": {
        "count": 3,
        "capacity":<PHONE_NUMBER>
      }
    }
  },

Description

Motivation and Context

Regression

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

bors merge


Suggestion: Batch c11y endpoint

The contextionary is independently scalable, so if c11y related queries are too slow, the user can just scale them up.
However, when using the /v1/c11y/words endpoint there is no benefit if requests are sent in sequence.
It would be nice if the client can send a batch request where weaviate makes sure that the vector calculation happens in parallel, not in sequence for the batch size.
Inspired by a use case from @fefi42 where the vectorizatoin is currenlty a bottleneck
CC @bobvanluijt

Closing for now as the performance at import time is not a major concern at the moment. If it does become one, we can come back to this.


Substitution allowed

Are there any regulations that allow an NCA or MAH to say that this product MAY NEVER BE substituted at dispense?
I know we have rules for cases a doctor MAY prescribe a product with no-substitution-remark, but I think it's always prescription-specific and theoretically optional. I don't know of any possibility to claim strict non-subsitution in a product catalogue.
How is it in other countries? Does anyone have products for which they declare that this must never be substituted?

Out of scope. NCAs don't have such flags.


Fixing magnetometer on iOS

This must be the tiniest PR I've done so far :)
This adds one extra parameter to starting device motion on iOS that enables reading out the magnetometer.
I'm not out of the woods yet, the results returned seem weird, maybe someone else can have a look and see if they can make sense of it. My suspicion is that the magnetometer is the only output iOS does not adjust for device orientation and the counter adjustments currently done are therefor wrong.
I'll be doing more work on this in the coming few days to nail this :)

Is this fix also needed in master? (and wow, that API :D)

@akien-mga, yup, same change, but let me spend an hour more on this over the weekend to figure out whether it's giving out the right values. It may need one or two tweaks.

@akien-mga I'm happy with this, it is working like a charm.
The magnetometer is a noise SOB so it was hard to figure out what the problem was. Turns out the world is round, not flat like those flatearthers claim.
As a result the code aligning with the magnetometer was causing my camera to turn upwards, while the alignment to the accelerometer was pushing it back down again, and they were in a nice tug of war....
Doing a simple:
east = gravity.cross(magnetometer)
magnetometer = gravity.cross(east)
Will horizontally align the vector which is something I'll be adding to the 9dof logic in due time.

Note to self: cherry-pick for master.


PSA: Has "troubleshooting" left you frustrated? Consider "the nuclear option"!

So I just downloaded the latest version of HSTracker, and decided I'd test it from the perspective of a "first-time user"... and to accurately do so requires removing every trace of my previous usage.
It also occurred to me that my "process" for doing so could potentially be helpful to someone, so I may as well document it!  ¯\_(ツ)_/¯
Disclaimers:

It's arguably "more useful" to just update however you normally would, and then work with bmichotte to legitimately diagnose whatever problems arise... but doing the following could still be a "last resort".
Following these instructions will delete all of your saved decks... so make sure they're all synced on your HearthStats account if you want to easily re-import them.
Doing all of this will also require HSTracker to re-download all of the card-art, which (if my experience is any indication) inexplicably takes way longer than it should.

Now... if you've been troubleshooting an issue, I'll assume you've already done basic system-related stuff (e.g., rebooted, disk-repaired, etc.) so let's start the destruction!
First up: open the Applications folder and trash HSTracker itself...

Next: open your hidden Library folder (not to be confused with the Root Library folder or the System Library folder) by clicking on the Go menu...

...then holding-down the option key and clicking Library...

Now the "fun" part: checking every sub-folder for HSTracker items.  I currently have five, but you may find more if you've been using HSTracker for long enough...

Finally: trash all evidence that you ever used HSTracker...

Congratulations!  You may now install everything "from scratch", just like a first-time user!  😜
Obviously, all of this is rather remedial to the average GitHub user... but if some random visitor is browsing this Issues page in desperation, then I hope this helps.
-TM

Hi @TickMan, thanks for writing this.
I have add it as the first wiki page ! https://github.com/bmichotte/HSTracker/wiki/remove-hstracker

Doing all of this will also require HSTracker to re-download all of the card-art, which (if my experience is any indication) inexplicably takes way longer than it should.

Is this still slow with the latest version ? I uploaded all the images on a personal server instead of Hearthstats ones

I have add it as the first wiki page !

Cool!  I vow to use my newfound celebrity to promote goodwill, help the downtrodden, and shame Face-Hunters. 😜

Is this still slow with the latest version ? I uploaded all the images on a personal server instead of Hearthstats ones

I remembered you mentioning that, so I was optimistic when the download started... considerably less so after monitoring that damn progress-bar for 40min.
Admittedly, that was during local "peak hours" (early evening) on a substandard connection... so I just did it again:
Card Download: 82.5MB ÷ 1449sec = 58.3KB/s
Speedtest.net (checked before and after download): 179KB/s
I'm only a "sample size" of one, and there could be a half-planet's worth of "hops" between me and your sever... so it's probably not worth stressing-out over unless others complain. ¯\_(ツ)_/¯
-TM


Hide the terminal profiles dropdown. It wastes space in column mode

Since the latest Insider Release VS Code automatically displays a dropdown with the terminal profiles. If the Terminal is in column mode, that wastes almost half of the screen space. Please let us remove it.
I don't need terminal profiles. Never asked for it, will never use it.
Please don't ruin the UI for the sake of the few users who may need the feature. Thanks

+1

@gotofritz can you provide a screenshot of the suboptimal organization of the UI (eg. in column mode, like you mention)?

Sure, here it is

I found an option to turn it off.

This is tabs not profiles, which is the second most voted feature request in the whole project so there were many who did ask for it.
It is still a work in progress though, right now you can drag the sash to the right to get a collapsed view, but you can also turn off tabs all together to go back to the dropdown via that setting.
I'll keep this open until we improve the panel on the side mode though, the thinking is to show the narrow view by default to minimize the amount of space used.

When you click on the "v"  it says "Select profile ..." and there is no mention of 'tabs' anywhere.... 🤷
OK, fine, tabs then.

the second most voted feature request in the whole project

My understanding was that 'tabs' would replace the awkward pulldown menu next to '... / Debug Console / Terminal' and basically work the same as it does for files - in which case I'm also for it. But your execution does nothing of the sort, it adds a space-wasting menu... so now I have a menu that says 'bash' above the terminal and another one that also says 'bash' on the left of it .... aaaargghh ....

the thinking is to show the narrow view by default

But why? Why waste screen estate like that for a column that has useful information for 10% of its height and the rest is wasted? Have you considered instead a second toolbar, below the one with 'problems / output / ... / terminal console' with your tabs? That would make the best use of the screen space (many of us use laptops!) and would fit in with the existing way tabs are shown for files

Oh well done @Taswen.. I didn't realise this feature was called "Tabs", I was looking for "profiles" in the preferences! That solves my issue thanks!

Maybe we can have settings to move those "Tabs" set to sub-top or bottom when in column mode?

so now I have a menu that says 'bash' above the terminal and another one that also says 'bash' on the left of it .... aaaargghh

The dropdown will go away when you have tabs showing as of today's insiders.
FYI this is the narrow view, you can resize the list to get it:

Maybe we can have settings to move those "Tabs" set to sub-top or bottom when in column mode?

It's a choice between the narrow view or the dropdown when the terminal is on the side, unless we invest a bunch of effort doing a horizontal design. We may end up doing that but probably not this month.

Now by default when you move the panel to the side, the tabs view is minimized. If you still don't want to see it, right click and select Hide Tabs

Great, just got the update and it works very well. Thanks for you efforts @meganrogge @Tyriar and everyone else involved!


Fix travis

OMG thank you so much. This has been driving me crazy, but I haven't had a chance to look into it!


Heap Cascade Sort

Program of Heap Cascade sort
ISSUE #564

Please Merge my PR under gssoc ext and hacktoberfest

I added readme file, please check it and let me know if any changes have to been done.


Feedback from community members

Situation
@scottyeager and I presented a demo on Mycelium. The feedback is amazing. People seem very interested.
A user asked if we were planning to add the following (see below) for Mycelium. I create this issue so we can track interest and get the POV of the dev team on this:

extension browser for mycelium
websocket support for mycelium
filesystem implementation for mycelium
database implementation for mycelium

Scott, feel free to add more insights.

It feels like I'm missing some context here. What is the ultimate goal that this user wants to achieve?
For the individual points:

extension browser in mycelium

Does this mean a browser extension to run mycelium?

websocket support for mycelium

On a technical level, mycelium nodes could communicate over websockets if that is implemented. The question is why that is specifically needed, since a websocket is essentially a tcp connection with s specific frame codec on top (i.e. slightly more overhead than tcp).

filesystem implementation for mycelium

Mycelium as in the application in this repo is only a network tool. Various filesystems exist which can be accessed remotely over the network, and as long as these support IPv6, they should work over  mycelium.

Database implementation over mycelium

Similar to the previous, databases (generally databases which run in cluster setups such as mongo) should be able to run using a mycelium connection out of the box, by configuring peers with the proper mycelium IP.

To give a bit more context, the question from the community member is about the possibility of implementing Mycelium as a web browser extension. The given reasons for this were around the additional security of web browser sandboxing, some users not wanting to run the binary release directly on their own system, and the ease of install.
The question regarding websockets then is related to this, as websockets are the allowed way for a web browser to carry on this kind of connection.

Implementing websocket connections between peers should be fairly straightforward. The main issue with browser extensions is (to my knowledge) the fact that they can't create network interfaces. After all, creating a network interface requires elevated privileges, which the browser by default doesn't have, and if it could do so, allowing extension to do this would also break the sandbox. So while it would essentially be possible to run some form of (lightweight) node in a browser (extension), it would just serve as an additional node in the network which may be used for routing.
As for not installing a node on the local system, if you connect more than 1 device, you could install mycelium on the router, and have the router hand out ip addresses from the /64 subnet you get to devices on the network. This obviously makes all your devices reachable on an essentially public network, and thus needs some firewalling. Alternatively, you can also run mycelium on the router, and set up an ip6 NAT rather easily. This way, other devices have no knowledge of mycelium (they won't even get an IP in the mycelium range), but can still reach services exposed on mycelium. The devices themselves will obviously not be exposed on the network.

Thanks for the details @LeeSmet. It certainly makes sense that a browser extension wouldn't be able to achieve a system wide enabling of Mycelium.
The remaining questions for me are:

Could such a set up in theory enable the browser and extension to work together to render web pages served over mycelium, without the creation of a network interface?
Any chance of an in browser SSH client working over a mycelium connection established by a browser extension?

Those would be the two main use cases I could imagine.

Could such a set up in theory enable the browser and extension to work together to render web pages served over mycelium, without the creation of a network interface?

You'll need to run a mycelium node somewhere to join the network. One possibility would be to run mycelium on a device you can reach, and running a socks5 proxy next to it. If you configure the browser to use the provided socks5 server, it will tunnel all data through it, and the proxy can use the local mycelium node to reach the network. This of course routes all traffic through the proxy. Perhaps using an extension might allow for more granular control over what gets routed where, but I don't have any relevant experience in that matter to provide more insights.

Any chance of an in browser SSH client working over a mycelium connection established by a browser extension?

If you mean to use an extension to run the SSH client while using a (local or otherwise routable from the machine) mycelium node, that should "just work"


Upgrade react native quick sqlite to 1.4.0

Work done

Update @journeyapps/react-native-quick-sqlite to ^1.4.0
Upgrade older dependencies

Will be updating the versions in another PR with some improvements to the demos.


Code for section "List python packages" is version dependant

EXECUTE sp_execute_external_script @language = N'Python'
, @script = N'
import pip
for i in pip.get_installed_distributions():
print(i)
'
GO
Does'nt work on Pyton 3.7. Need to use...
EXECUTE sp_execute_external_script @language = N'Python'
, @script = N'
import pkg_resources
dists = [str(d) for d in pkg_resources.working_set]
print(dists)
'

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

ID: 37033ba2-66fb-1cf5-86e0-03b31c7ba7ac
Version Independent ID: 2dc585ee-ac57-ac0d-cf03-12d623a18ff7
Content: Quickstart: Create Python scripts - SQL Server Machine Learning Services
Content Source: docs/advanced-analytics/tutorials/quickstart-python-create-script.md
Product: sql
Technology: machine-learning
GitHub Login: @garyericson
Microsoft Alias: garye

#assign:garyericson

@MattiasWoldu - Thanks for catching this and for providing sample code! I thought I'd make it look a little nicer (see below), but if you have any other suggestions, let me know.
By the way, you can submit changes like these yourself by editing the article directly. Edits get sent to the author for review and sign-off. If you're interested, here are the instructions: https://docs.microsoft.com/en-us/contribute/
EXECUTE sp_execute_external_script @language = N'Python'
, @script = N'
import pkg_resources
import pandas
dists = [str(d) for d in pkg_resources.working_set]
OutputDataSet = pandas.DataFrame(dists)
'
with result sets(([Package] nvarchar(max)))

I've made the change, should be merged and published tomorrow.
#please-close

-- This example works fine.
EXECUTE sp_execute_external_script @language = N'Python', @script =
N'import pkg_resources
import pandas as pd
dists = [str(d) for d in pkg_resources.working_set]
OutputDataSet = pd.DataFrame(dists)
'
WITH RESULT SETS(([Package] NVARCHAR(max)))
GO


Switch to use typings package and remove checked in typings

I updated the see to use the typings package and check in a typings config so that you don't need to version those files.

I pushed what I had already committed locally and added some of your changes. Instead of grunt-typings I install typings as a postinstall npm step. Any particular reason why you used grunt instead?

Does the postinstall step require you have typings installed globally?

Running from within package.json should work without global installation afaik
tiny keyboard, excuse brevity
On Fri, May 27, 2016 at 10:30 PM -0700, "Matthew Manela"<EMAIL_ADDRESS>wrote:
Does the postinstall step require you have typings installed globally?

You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/cschleiden/vsts-extension-ts-seed-simple/pull/3#issuecomment-222291288


Numpy Dot Large Tensor Fix

This PR fixes numpy dot on large tensors. This fix also requires ILP 64 Blas which CI has not migrated to yet, so for now we are skipping the test.
With this fix numpy dot will no longer error out on > 2**31 dimensions. However there seems to be a precision issue with ILP64 openblas: https://github.com/xianyi/OpenBLAS/issues/2779
So on my remote computer
A = np.ones((1, 2**31))
B = np.ones((2**31, 1))
C = np.dot(A, B)

Will give 1.93274e+09
A = np.ones((2**31, 1), dtype='float64')
B = np.ones((1, 2**31), dtype='float64')

Will give the arithmetically correct result 2.14748365e+09
Issue at https://github.com/apache/incubator-mxnet/issues/18926
Update: regardless of the precision issue mentioned above, this pr should be a general fix for numpy.dot and should be the same regardless of which blas we use

@sandeep-krishnamurthy

@mxnet-bot run ci [windows-gpu]

Can you run the tests using pytest and paste console output here ? (both command and output)

Can you run the tests using pytest and paste console output here ? (both command and output)

This test is skipped for now because we are not yet using ilp64 blas. After we migrate to ILP64 blas/lapck I will unskip this test and check again if it passes.

This test is skipped for now because we are not yet using ilp64 blas. After we migrate to ILP64 blas/lapck I will unskip this test and check again if it passes.

Can you locally run it and paste the output ? I saw the description but your change should atleast be locally  tested.

Rest LGTM ! Can you run the tests manually and paste results here

Rest LGTM ! Can you run the tests manually and paste results here

=================================== test session starts ===================================
platform linux -- Python 3.7.7, pytest-5.4.1, py-1.8.1, pluggy-0.13.1
rootdir: /home/ubuntu/dot_fix, inifile: pytest.ini
plugins: remotedata-0.3.2, openfiles-0.4.0, astropy-header-0.1.2, hypothesis-5.8.3, arraydiff-0.3, doctestplus-0.5.0
collected 1 item
tests/nightly/test_np_large_array.py .                                              [100%]
==================================== 1 passed in 2.75s ===================================
Passes offline.

@mxnet-bot  run ci [unix-cpu]

@sandeep-krishnamurthy can you help merge


Move getStaticPaths instructional content out of the reference page

The getStaticPaths content on our reference page is not reference content - it's better suited for a guides page. We link to it on our Routing guide, but that Routing guide should be the home for this content.
We've also gotten feedback that getStaticPaths is particularly difficult to understand, so it's worth revisiting how we explain it when we integrate it into that guides page.

The tutorial presents getStaticPaths in a more friendly beginner way; our docs could build off of that for our explanatory content.
The examples in the dynamic route section need another pass; we've seen support threads of people being confused by them.

Agreed that this could use another look at how the content is divided between pages.
In particular, and perhaps more urgently, now that each error message has their own page in the reference section, we should decide whether error messages should be their own category. Even the API reference docs for getStaticPaths are getting drowned out by error message returns.
And currently, getStaticPaths() is not even in a heading on the routing page, making it not visible in the right sidebar, but also, it doesn't show up in Algolia results AT ALL.  So urgently, Routing page should get a quick-fix heading for this. But, I also think these Algolia results are not as helpful as they could be.

Note that our search doesn't bring you to the routing page at all - headers need rewriting.


Add cue files to controller image

This allows this image to be used in production. While used locally, the
files can/will be override by any possible volume mounts.
@myitcv I had to do this to deploy the controller image in prod since it was missing this files. Not sure If this is the proper way of doing it, but it works until we find a better way to deploy this.

As we spoke I'm merging this @myitcv until we work on the new controller deployment.


move icons to icons folder

Thanks @murribu I'll grab this as a base for my refactor as I also want to get everything asset related into an assets folder since the goal of #16 is in fact to remove clutter from root directory 👍


cmake: Bump max LLVM version to 19

Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Do I get it right that this is finally an LLVM release where all we have to do is bump the version? Incredible 😮

yes :)

Anyways, I'd prefer if we waited for Nix to add LLVM 19 so that we can run all the tests. Since it should be released tomorrow, it should hopefully not take long before it gets to Nixpkgs.

yocto has clang-19 staged already and waiting for final release
https://github.com/kraj/meta-clang/pull/982
and has been testing this patch

Also a changelog entry would be nice.

OK

LGTM,  but like Viktor said, would be good to wait until we have it in CI until we merge this. Cuz if we merge and we cut a release, might ship a buggy release.

LGTM, but like Viktor said, would be good to wait until we have it in CI until we merge this. Cuz if we merge and we cut a release, might ship a buggy release.

No issues, sounds reasonable.

Looks like ubuntu:devel is already on llvm 19. So our distro build fails in CI.
I'm thinking about merging this but gating llvm 19 behind CMAKE_BUILD_TYPE=Debug. Idea is that we don't want llvm 19 built by packagers yet but also want CI working until nix releases llvm 19.
Or we could move CI to track nix unstable. Kinda hesitant to do this b/c nix is kinda a black box for many folks and I'm not sure we want to be debugging weird nix things.

I'm thinking about merging this but gating llvm 19 behind CMAKE_BUILD_TYPE=Debug. Idea is that we don't want llvm 19 built by packagers yet but also want CI working until nix releases llvm 19.

Sounds like a good idea to me. Also lets people experiment with LLVM 19 before we declare it as officially supported.

Needs this patch applied to flake.nix: https://pastes.dxuuu.xyz/z5unr7

Needs this patch applied to flake.nix: https://pastes.dxuuu.xyz/z5unr7

Still needs this for the LLVM 19 jobs to pass.


Create multiple packages with one NuGet Packager task

I think this should be possible when I use the right "Path/Pattern to nuspec files" because that field supports minimatch patterns. I want to build 2 nuget packages, one from **\Client.csproj and one from **\CanonicalSchema.csproj. I tried this minimatch pattern, but that doesn't work:

**@(CanonicalSchema|Client).csproj

The error I got:

2016-02-10T13:44:17.2552576Z Find-Files -SearchPattern **@(CanonicalSchema|Client).csproj
2016-02-10T13:44:17.2708842Z ##[error]Illegal characters in path.

'@' is part of extended globbing, which Find-Files doesn't support. Patterns are split at semicolons and then (more or less) treated as .Net regexes from there. Try your pattern without the @. If that doesn't work, **\CanonicalSchema.csproj;**\Client.csproj will.

Your last suggestion was my first guesse, but that doesn't work either. When I try **\CanonicalSchema.csproj;**\Client.csproj, this is the result:
Find-Files -SearchPattern C:\tfsBuildAgent\_work\17\s\**\CanonicalSchema.csproj;**\Client.csproj
The path is not of a legal form.
Found files: 0

D'oh, well that's no good. I'll take a look at this when I get into the office.

Ah, found it. There's a bug in the way the task is written - if it doesn't see a * or ? in the pattern, it assumes it's not a pattern but a direct file reference. One workaround is to force it to evaluate your input as a pattern by including a ? somewhere: **\CanonicalSchema.cspr?j;**\Client.csproj
I think we can fix this in a backwards-compatible way by assuming that a semicolon also indicates a pattern.

Nice, thanks for the fix and the work around. I'll give it a try when I'm at work next week.

@tomkuijsten everything working for you?

I just created a test and it didn't work as expected:
Checking pattern is specified
Pattern found in solution parameter.
Find-Files -SearchPattern C:\tfsBuildAgent\_work\48\s\**\CanonicalSchema.cspr?j;**\Client.csproj
The path is not of a legal form.
Found files: 0

That's odd. It got past the bug I knew about because it says "Pattern found in solution parameter".
There's something funky happening with the Find-Files cmdlet that I don't understand. I reproed the bug you're seeing locally.
The best workaround for now is going to be specifying a single pattern with a wildcard that matches both of your project files: **\C*.csproj perhaps? I'm following up with the owners of the Find-Files code to see what I'm missing.

Thanks to @ericsciple I found the right solution. It has to do with relative vs absolute paths. This should fix your issue the right way: $(build.sourcesdirectory)\**\CanonicalSchema.csproj;$(build.sourcesdirectory)\**\Client.csproj

@ericsciple  - Seems like there's a bug maybe.  All input paths should be relative to the root of the repo, right?  Unless it's not a filepath input type ...  If not, I would expect the default to be $(build.sourcesdirectory)***.csproj to encourage the right input ??

@bryanmacfarlane FWIW it also fails with unrooted paths in PowerShell. I had to explicitly add "." to get that working.

@vtbassmatt there's an optional RootFolder parameter that allows you to specify the root folder for unrooted search patterns. I'm not sure offhand what the default is otherwise... I would assume it's the current working directory. I'm not sure, I need to look into this in our tasks.

OK, here's what's going on.

The search pattern input's type is defined as "FilePath" in the task.json. So the input gets rooted with $(build.sourcesdirectory). This answers part of Bryan's question. The problem is that only the first search pattern is rooted, the FilePath input doesn't understand that it's a delimited list of paths.
When the Find-Files cmdlet is processing each individual pattern, it tries to determine what folder to start searching. It tries to grab the path preceding the first wildcard, and passes that to system.io.path.getdirectoryname(...). The problem here is that it ends up passing empty string, which results in the error The path is not of a legal form..

Thoughts on a short-term fix: For the NuGet Packager task, probably the best thing to do is to supply the RootFolder parameter when calling Find-Files. That will make sure each pattern gets rooted. You would probably want to pass whatever value the FilePath input uses to root relative paths. For Build, this is $(build.sourcedirectory). I'm not sure if the NuGet Packager task is commonly used in Release as well. If so you might want to fallback to whatever Release uses (that is, if build.sourcesdirectory is not defined).
...Now I need to go look at the vsbuild/msbuild tasks because they have the same issue :)

Thanks for the investigation and root cause! I'll update.
No idea if this task gets used with RM but for sure the Publisher does. I'll make it work both ways.

I tested your suggestion:  $(build.sourcesdirectory)\**\CanonicalSchema.csproj;$(build.sourcesdirectory)\**\Client.csproj
But this ends in:
Checking pattern is specified
Pattern found in solution parameter.
Find-Files -SearchPattern c:\tfsBuildAgent02\_work\24\s\ c:\tfsBuildAgent02\_work\24\s\**\CanonicalSchema.csproj;c:\tfsBuildAgent02\_work\24\s\**\Client.csproj

The given path's format is not supported.
Found files: 0

Somehow, the  $(build.sourcesdirectory) is prepended anyway. I thought to be clever and changed the pattern to: **\CanonicalSchema.csproj;$(build.sourcesdirectory)\**\Client.csproj
But that ended in:
Executing the powershell script: c:\tfsBuildAgent02\tasks\PublishBuildArtifacts\1.0.4\PublishBuildArtifacts.ps1
Preparing artifact content in staging folder c:\tfsBuildAgent02\_work\24\a...
Copying all files from c:\tfsBuildAgent02\_work\24\s\bin\Release\Bronkhorst.InstrumentCommunication.CanonicalSchema.1.3.1-dev.nupkg to c:\tfsBuildAgent02\_work\24\a\NuGet\Bronkhorst.InstrumentCommunication.CanonicalSchema.1.3.1-dev.nupkg
Copying artifact content to \\server\builds...
Start: AssociateArtifact
Associated artifact 641 with build 563
End: AssociateArtifact

The client package is missing this time...

@tomkuijsten are you on-prem or VSTS? I wonder if you have an older edition of the packager task. I did $(build.sourcesdirectory)\**\ConsoleApplication1.csproj;$(build.sourcesdirectory)\**\ConsoleApplication2.csproj and got:
2016-02-18T17:58:53.3667583Z Checking pattern is specified
2016-02-18T17:58:53.3837648Z Pattern found in solution parameter.
2016-02-18T17:58:53.3867594Z Find-Files -SearchPattern C:\a\1\s\**\ConsoleApplication1.csproj;C:\a\1\s\**\ConsoleApplication2.csproj
2016-02-18T17:58:53.4167608Z Found files: 2
2016-02-18T17:58:53.4237587Z --File: "C:\a\1\s\ConsoleApplication1\ConsoleApplication1.csproj"
2016-02-18T17:58:53.4237587Z --File: "C:\a\1\s\ConsoleApplication2\ConsoleApplication2.csproj"

This is running version 0.1.58 (really 0.1.60, which includes my fix to split on semicolons but is otherwise identical) of the task.

On-premise TFS 2015 update 1, version 1.4.0 of the PublishBuildArtifacts.

What version of NugetPackager?

Oh dah! Sorry, the version of NugetPackager: 0.1.56

The output of the nugetpackager:
2016-02-19T14:16:35.7966247Z Set workingFolder to default: c:\tfsBuildAgent02\tasks\NuGetPackager\0.1.56
2016-02-19T14:16:35.7966247Z Executing the powershell script: c:\tfsBuildAgent02\tasks\NuGetPackager\0.1.56\NuGetPackager.ps1
2016-02-19T14:16:36.0622413Z Checking pattern is specified
2016-02-19T14:16:36.0622413Z Pattern found in solution parameter.
2016-02-19T14:16:36.0622413Z Find-Files -SearchPattern c:\tfsBuildAgent02\_work\24\s\**\CanonicalSchema.csproj;c:\tfsBuildAgent02\_work\24\s\**\Client.csproj
2016-02-19T14:16:36.2497577Z Found files: 1

Your latest snippet looks like it's passing the right string to Find-Files, but now only finding one .csproj. That's different from the snippet you pasted further up -- maybe that was a typo with a space at the beginning of the pattern?

PR #1258 will fix the issues I know about.

That latest snippet is the result when using **\CanonicalSchema.csproj;$(build.sourcesdirectory)\**\Client.csproj.
Maybe I did have a space in front of my first try with $(build.sourcesdirectory)\**\CanonicalSchema.csproj;$(build.sourcesdirectory)\**\Client.csproj, but even if that one worked, I guess that the output would be that same as my last result. I'll give it another shot on Monday at work.

OK, thanks @tomkuijsten. I think we may be dealing with multiple nested issues here. Sorry it's been such a process.

Not a problem, my build is running fine with two tasks now. So everything I can do to help fix some bugs, I'd be glad doing.

You were right about that space. I tried it again with $(build.sourcesdirectory)\**\CanonicalSchema.csproj;$(build.sourcesdirectory)\**\Client.csproj and checked for the space, this time the path was correctly read, but the output is still only CanonicalSchema:
Checking pattern is specified
Pattern found in solution parameter.
Find-Files -SearchPattern C:\tfsBuildAgent\_work\55\s\**\CanonicalSchema.csproj;C:\tfsBuildAgent\_work\55\s\**\Client.csproj
Found files: 1
--File: C:\tfsBuildAgent\_work\55\s\CanonicalSchema\CanonicalSchema.csproj

I checked the second path on the server and it really should have found the Client.csproj. I changed the path again to C*.csproj to check if it could find Client.csproj this time, the output:
Find-Files -SearchPattern C:\tfsBuildAgent\_work\55\s\**\C*.csproj
Found files: 3
--File: C:\tfsBuildAgent\_work\55\s\CanonicalSchema\CanonicalSchema.csproj
--File: C:\tfsBuildAgent\_work\55\s\Client\Client.csproj
--File: C:\tfsBuildAgent\_work\55\s\ClientTestApp\ClientTestApp.csproj

I can't repro this behavior. Once I put the $(build.sourcesdirectory) in both spots, the task picks up both .csproj files. It's interesting that it no longer throws the "path not of legal form" message even though it doesn't find your file.
Let's take the build system out of the equation temporarily. There's a standalone version of Find-Files (that Eric helped me find offline) which might give us some insight. Instructions for that are to run PowerShell and then:
git clone https://github.com/Microsoft/vsts-task-lib.git
cd vsts-task-lib
Import-Module ./powershell/VstsTaskSdk
cd [directory containing your source]
Find-VstsFiles -LegacyPattern ".\**\CanonicalSchema.csproj;.\**\Client.csproj" -Verbose

I'm interested in the output of the above commands, but also feel free to poke around with Find-VstsFiles. It might be interesting to replace both of the "."s with the full source directory.


Operator should integrate with "cloudproviders" to update launch config for new workers

The motivating example is an AWS Autoscaling Group.
When an ASG is created, it is created with a single configured version of Container Linux. The ASG will continue to launch that version even if all the other nodes have updated.
When the group scales up, having to do a update/reboot cycle is just unnecessary churn.
I propose that the update operator can be configured to upgrade such a launch group once a majority of the nodes have successfully updated to a newer version.

/cc @crawford @aaronlevy
If I recall correctly, according to latest discussion we in fact don't want to touch the ASG userdata. An update/reboot cycle is employed in order to provision matching OS/docker/kubelet before letting a node join the cluster.

An update/reboot cycle is employed in order to provision matching OS/docker/kubelet before letting a node join the cluster.

That is the current plan. We would ideally not have to rely on cloud-provider specific mechanics for our upgrade process. There may be a day where it becomes unavoidable - but it's a risk both from permissions perspective, and complexity for every new provider we want to support.

For reference, we now install nodes with a bootstrapper which takes care of upgrading the node OS beforea new node joins the cluster. This still requires a reboot and the installer can opt out of this for testing purposes.


_client.LowLevel.SearchAsync<>() api ignoring type and index name defined in SearchDescriptor<>() object

NEST/Elasticsearch.Net version:5.6.5
Elasticsearch version:5.4.3
Description of the problem including expected versus actual behavior:
We are trying to fetch result from our index using the LowLevelClient. We are using the below SearchAsync API
          var searchDescriptor = new SearchDescriptor<MyType>()
                                                       .Type("mytype")
                                                       .Index("myindex")
                                                       .Query(....)
                                                       .Aggregation(ag => ag.Terms(... Aggregation(ag1 => ag1.Min(...TopHits(...)))));
            var memoryStream = new MemoryStream();
            _client.Serializer.Serialize(searchDescriptor, memoryStream);
            var response = await _client.LowLevel.SearchAsync<byte[]>(memoryStream.ToArray()).ConfigureAwait(false);
           //_client - instance of Nest.ElasticClient
           //Next Step - Deserialize the response

Actual Behaviour - The client is ignoring type and index name and calling POST /_search api
Expected Behaviour - The client should call POST /myindex/mytype/_search api
This is giving me results from another index(a combination of result from various index) and my deserialization is breaking.
Steps to reproduce:

Call _client.LowLevel.SearchAsync<>() with index and type defined in SearchDescriptor object
observe the ElasticsearchResponse.DebugInformstion property to check the API that is called.

Expected behavior
The client should call POST /myindex/mytype/_search api
Provide ConnectionSettings (if relevant):
Provide DebugInformation (if relevant):

Found a workaround
The _client.LowLevel.SearchAsync<>() method have overloaded method _client.LowLevel.SearchAsync<T>(string indexName, string typeName, T t)
Passing the index name and type name will narrow to search to that particular index.
But the question still remains, why it is not taking the index and type name from SearchDescriptor object?

When you target the low level client the SearchDescriptor only describes the the querystring parameters and body of the request.
The low level client has no knowledge of SearchDescriptor<>() and how it exposes the path parameters.
Thanks for reporting but I dont think this is actionable. The work around is the intended usage.


Bot Framework Composer is slow since 1.3.0 update

Describe the bug
I'm creating a bot structure.  When I go through the process of adding a dialog, this takes about a minute beginning to end.  i.e. Click add dialog, wait for window to show, enter name, click ok and wait for dialog to be added.  This did not happen prior to update.  I'm on a high spec Surface Pro 7.
Version
This was an auto-update.  Details in Help>About are
Version 1.3.0
Electron 8.2.4
Chrome 80.03987.165
NodeJS 12.13.0
V8 8.0.426.27-electron-0
Browser
Latest Edge.  However using actual standalone Bot Framework Composer desktop application.
OS
Windows 10
To Reproduce

Open my bot
Click on bot root and choose "Add a dialog"
Enter dialog title name and click "OK"
Wait for dialog to be created and shown in menu
Total time ~60 sec

Expected behavior
I expect this process to execute much quick as before the auto update

This might be the same issue #5396 @lei9444 for awareness

Thank you for following up on this.
Yes, it does look like a similar issue.
I’m also building a bot for my company that we wish to release on the Microsoft Teams app store in January and this update issue has stopped any further development as the Composer is basically unusable.
Thanks,
[cid:image003.png@01D6DDE8.76BE9140]https://moderncollab.co.uk
Dan Burling
Modern Collaboration and Digital Adoption Specialist
Get in touch
+44 7950 90 80 85
linkedin.com/in/danburling
From: Brenda<EMAIL_ADDRESS>Sent: 29 December 2020 00:45
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [microsoft/BotFramework-Composer] Bot Framework Composer is slow since 1.3.0 update (#5411)
This might be the same issue #5396https://github.com/microsoft/BotFramework-Composer/issues/5396 @lei9444https://github.com/lei9444 for awareness
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/BotFramework-Composer/issues/5411#issuecomment-751907807, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASIGJIA5QDLT4U23MOR4UMDSXERCHANCNFSM4VISM5KQ.

@djsbur while the team works through this issue, can you please use a previous version to unblock your development?

Hi @cwhitten
I have reinstalled v1.2.0.  However I have also had to revert to an older backup of the bot that was not amended using v1.3.0 as the bot file makes v1.2.0 hang.  I've lost a lot of work, but at least there's a speedy way forward now.
Looking forward to see what the analysis turns out :)
FYI I do love the Bot Framework Composer (as a non-dev).  Great work to all of your team!

We’ve been looking into this for a few days now. I expect us to have an update on 1.3 that will allow you to re-update soon.
I’m glad you’re finding value in the product!

this one is duplicated, closing it so we can track it in #5396

this one is duplicated, closing it so we can track it in #5396


Order entity by nested entity fields

Hi,
Scenario: I have a simple model of  Products and Categories, just like that:
export class Product extends Model {
    static entity = "products";

    static fields() {
        return {
            id: this.number(null),
            name: this.attr(''),
            category_id: this.number(null),
            category: this.belongsTo(Category, 'category_id')
        };
    }
}

export class Category extends Model {
    static entity = "categories";

    static fields() {
        return {
            id: this.number(null),
            name: this.attr('')
        };
    }
}

Question: Is it possible to get a list of products ordered by its category name and then by product name, something like that:
...
var products = Product.query()
    .with('category')
    .orderBy('category.name')
    .orderBy('name)
    .get();
...

Unfortunately, it is not possible to order by relation fields at this time. You'll have to execute your own sort comparator on the query result.
That being said, I mocked up a comparator that should fit the bill:
const sortable = (fields) => (a, b) => {
  let i = 0, result = 0

  while (i < fields.length && result === 0) {
    const { field } = fields[i]
    const direction = fields[i].direction === 'desc' ? -1 : 1

    const valA = (typeof field === 'function' ? field(a) : a[field]).toString()
    const valB = (typeof field === 'function' ? field(b) : b[field]).toString()

    result = direction * (valA < valB ? -1 : valA > valB ? 1 : 0)
    i++
  }

  return result
}

Essentially, you pass the comparator into Array.prototype.sort with a list argument consisting of the field name and optionally a sort direction (defaults to ascending when omitted) like so:
array.sort(
  sortable([
    { field: 'name' },
    { field: 'age', direction: 'desc' }
  ])
)

Additionally, you can substitute a field name with a closure to return a field, or a nested field in your case, like so:
const products = Product.query().with('category').get()

products.sort(
  sortable([
    { field: 'name' },
    { field: (product) => product.category.name }
  ])
)

Do with it what you may! 👍


Matrix structure limits

Why is it limited to 2x2 and 3x3?

I worked for long time with matrices and there was always 2x2 and 3x3 power.
So, this library is not designed to work with trees. So you probably won't be able to render big data.
However, I don't remember that I make restrictions other than the render class.

Where would I be able to find a forced matrix?


Column with the Guid type is created as a string

I tried to create a database table using a migration with the following method.
SchemaBuilder.CreateTable("Vendors", (table) => table
    .Column<Guid>("Id", column => column.PrimaryKey().WithLength(36))
    .Column<string>("Name", (column) => column.WithLength(100).NotNull())
);

The above command creates a column of type Guid as nvarchar(36) instead of UniqueIdentifier
I am able to workaround this problem like this, but think there
SchemaBuilder.CreateTable("Vendors", (table) => table
    .Column("Id", DbType.Guid, column => column.PrimaryKey())
    .Column<string>("Name", (column) => column.WithLength(100).NotNull())
);

If this is agreed to be a problem, I think the problem is in this line which should be changed to { typeof(Guid), DbType.Guid}

This should be fixed in the latest version. Can you check? There are unit tests too. It's using a string only for the databases that don't support it.

This should be fixed in the latest version. Can you check? There are unit tests too. It's using a string only for the databases that don't support it.

@sebastienros Yes that indeed fixed the issue.
I noticed you changed DbType to Type and you removed the ISqlDialect Dialect from the IStore. To get the name of the server being used ("MySQL","SqlServer"....) is it valid to do this now?
IStore store = serviceProvider.GetRequiredService<IStore>();
string sqlServerName = store.Configuration.SqlDialect.Name;

@sebastienros Yes that indeed fixed the issue.
I noticed you changed DbType to Type and you removed the ISqlDialect Dialect from the IStore. To get the name of the server being used ("MySQL","SqlServer"....) is it valid to do this now?
IStore store = serviceProvider.GetRequiredService<IStore>();
string sqlServerName = store.Configuration.SqlDialect.Name;


fix Constraint Type description typo in docs

Cool project! Was reading the docs and believe this is a typo

Thanks for spotting that @nataliepopescu !


CI test for Instagram and Google+ scraping against live sites

our Instagram and Google+ scraping have both broken recently when their embedded JSON changed, and we didn't notice until days or weeks after. we should add tests against their live sites, similar to facebook_live_test.py, and run them nightly on CircleCI: https://circleci.com/docs/1.0/nightly-builds/

done for instagram!

G+ test is written and checked in, but not yet running due to to #105. i may or may not prioritize that, so I'm closing this issue.


Extract engine api to a separate package

Motivation
Both the beacon node and light client would have to communicate with an execution layer. Hence a need to have the code that does it extracted out of the beacon-node to enable re-use.

can you move the JsonHttpClient to utils?

@dadepo There's a 6,000 line diff in the yarn.lock, this should not happen please review. You should only clean the dependency added here
https://github.com/ChainSafe/lodestar/pull/4798/commits/f6f401d6e09d5148bfb9cfe9f499690dddaee235#diff-51e4f558fae534656963876761c95b83b6ef5da5103c4adef6768219ed76c2deR2325

can you move the JsonHttpClient to utils?

@wemeetagain Lion spotted here that jwt-token can't be used in the browser...and JsonHttpClient depends on it. Since utils package is supposed to be agnostic to both browser and node, I will have to move JsonRpcHttpClient away from the utils package. Unless you have other suggestions to go about this?

@dadepo There's a 6,000 line diff in the yarn.lock, this should not happen please review. You should only clean the dependency added here f6f401d#diff-51e4f558fae534656963876761c95b83b6ef5da5103c4adef6768219ed76c2deR2325

when making changes that involves changing package structures, I often delete node_modules folder and yarn.lock to install everything afresh to be sure I am not getting false positives, where things continue to work because of already resolved, and cached dependencies. This is why the yarn.lock end up with the diffs.
How will you go about this instead in other to be assured no cached files are leading to false positives?

@wemeetagain Lion spotted https://github.com/ChainSafe/lodestar/pull/4798#discussion_r1030303789 that jwt-token can't be used in the browser...and JsonHttpClient depends on it. Since utils package is supposed to be agnostic to both browser and node, I will have to move JsonRpcHttpClient away from the utils package. Unless you have other suggestions to go about this?

We could think about changing the library to sign JWTs to jose which support all sorts of runtimes and is a really high quality library. The downside to the current library is that it is has a much bigger code base and supports a lot of features we don't need at the moment.

@wemeetagain Lion spotted #4798 (comment) that jwt-token can't be used in the browser...and JsonHttpClient depends on it. Since utils package is supposed to be agnostic to both browser and node, I will have to move JsonRpcHttpClient away from the utils package. Unless you have other suggestions to go about this?

We could think about changing the library to sign JWTs to jose which supports all sorts of runtimes and is a really high quality library. The downside is that it has a much bigger code base and supports a lot of features we don't need at the moment.

Also thought about just writing our own lib that does the minimal functionality. I coded this up somewhere, tho it's maybe lost now. But it ends up being like 40 LoC max.

Closing for now, best to re-open with a fresh take if needed


What of anyOf and allOf JSON Schema features?

I see support for oneOf, but what of anyOf and allOf draft 4 constructs?

Hi.
I'm not sure anyOf would work in context of deserializing it into Java/Scala classes.
Same goes with allOf.
Please enlighten me


Update AutoGenerated Password

Change how moderating participant password is created to include a capitalized letter
Remove space inside string interpolation

[#98987112]

:eyes: :+1:


Why the new version of the CLI generated template does not support LAN IP access!

Why the new version of the CLI generated template does not support LAN IP access!
For example: <IP_ADDRESS>:8080

./config/index.js
module.exports.dev.host = '<IP_ADDRESS>'

Using module.exports.dev.host = '<IP_ADDRESS>' works perfectly fin (in webpack template)
In any case, this is something related to webpack not vue-cli nor its templates


fix-path-traversal-vulnerability

This pull request addresses a path traversal vulnerability in the codebase, as reported by Snyk. The vulnerability arises from unvalidated user input being used as a path in the fs.readFileSync function, potentially allowing an attacker to read arbitrary files.
Changes Made

Implemented input validation and sanitization to prevent path traversal attacks.
Enforced strict file path handling to avoid unintended access to sensitive files.
Enhanced security checks to safeguard against unauthorized access.

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

@tan-nhu  Thanks for your feedback and for considering the change. I've updated the configuration as 'code' as you suggested to simplify the script. This change aligns with the current usage pattern and reduces potential complexity.
Regarding the use of restful-react, we'll keep an eye on its deprecation and plan to transition to the new library when needed.and thanks for the input !!
I've also ensured that Prettier has been run against the changed files as requested.
If you have any further suggestions or concerns, please feel free to let me know. Your input is greatly appreciated!

Hi @hitesharinga, the PR has been approved by @tan-nhu. Please merge it so that we can close this. Thank you.

Hey @DharunKumar04, Thank you for the Fix. I have merged it.


Prim algo

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
Article for Prim algorithm
Checklist

[ ] I've read the contribution guidelines.
[ ] I've checked the issue list before deciding what to submit.
[ ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes: #103

@HarshCasper please review

@HarshCasper please review

@HarshCasper done

@HarshCasper please review

@HarshCasper please review

@HarshCasper may I know why my PR is not being reviewed

@HarshCasper it's the 4th time I am getting merge conflicts


Some corrections in Non-Resilient cephblockpool & storageclass

Some corrections/changes to the Non-Resilient/Replica-1 cephblockpool  & the storageclass.

/lgtm


get example app working

This PR

updates the example app to match recent changes to xDripG5
fixes some warnings when pod install is called (see here for reference)
adds a setter for ID in Transmitter
fixes a deprecation warning
updates iOS version in xDripG5.podspec to 10.3 to keep dependencies happy

Note that the compilation order for xDripG5 needs to be changed as a workaround for https://bugs.swift.org/browse/SR-631. Specifically, PeripheralManager+G5.swift and BluetoothServices.swift need to be compiled after PeripheralManager.swift to avoid errors like "'Configuration' is not a member type of 'PeripheralManager'". Not sure how to specify compilation order in CocoaPods.

Resolved with https://github.com/LoopKit/xDripG5/pull/71.  Thanks @ps2.


"Brevent is only available for Owner."

I have xprivacy automatically add brevent to their list in the first time I install Brevent. after I played it little bit, I would like to whitelist brevent from xprivacy, but things happened after I did so, brevent shows me
"Brevent is only available for Owner." and never goes through.
tried clean data / reinstall, which none of them works.
I am in rooted marshmallow custom rom with newest version brevent 3.2.
please help.

solved by myself, I should reboot my phone before I asked here.