You have alpha version marked as latest on npm

Describe the bug
This is not a software bug, but a logistics one. Sorry if I submit it to the wrong place. Currently, when someone tries to install @tanstack/react-virtual from npm, the version that gets installed (and is marked as latest) is 3.0.0-alpha.0, but the current latest is 3.0.0-beta.65.
Your minimal, reproducible example
run npm i @tanstack/react-virtual
Steps to reproduce

Run npm i @tanstack/react-virtual
See that the version is 3.0.0-alpha.0

Expected behavior
The version should either be 2.x or 3.0.0-beta.65
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

any operating system, any browser

tanstack-virtual version
3.0.0-beta.65
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Duplicate of #380, which was closed with the following reason:

We're releasing soon on main. Stay tuned.

That was on August 30, so take "soon" with a grain of salt.

Yes, we know, please bear with us and install latest version [v3.0.0-beta.66](https://github.com/TanStack/virtual/releases/tag/v3.0.0-beta.66) till we sort it out.


how to change the confidence score in train net.py when eval-only?

I have tried to modify the SCORE_THRESH_TEST/ INSTANCES_CONFIDENCE_THRESH in defaults.py, but it doesn't work and the APs and other metrics don't change.

Hi~
Yep, changing SCORE_THRESH_TEST/ INSTANCES_CONFIDENCE_THRESH in defaults.py won't change the result.
This is because Sparse R-CNN only output 100/300 boxes in total for each image, so our implementation is putting all output box into the evaluation code.

Hi~
Yep, changing SCORE_THRESH_TEST/ INSTANCES_CONFIDENCE_THRESH in defaults.py won't change the result.
This is because Sparse R-CNN only output 100/300 boxes in total for each image, so our implementation is putting all output box into the evaluation code.


README.md: updating minimun required Rust version (1.65+ -> 1.71.1+)

A simple change in the README.md so that we indicate the minimum required rust version.
This version is to prevent the next compilation error:

Good point! 🧐
There is one thing that we can do to automatically "enforce" the rust toolchain version: To add a rust-toolchain.toml file (like this).
But in that case, I am not sure if the CI's rust-toolchain GH action will respect that file.

Good point! 🧐
There is one thing that we can do to automatically "enforce" the rust toolchain version: To add a rust-toolchain.toml file (like this).
But in that case, I am unsure if the CI's rust-toolchain GH action will respect that file. Could you check it?

I think this is a great idea! I added a simple rust-toolchain.toml file 🥳

Merging 🚀


Add es-VE (Spanish as spoken in Venezuela) profanities to es.json

fixes #20

@MrBenJ May I ask you to review these changes?
@Leodau Don’t worry about the Travis failure. That’s an issue unrelated to your code.

@Leodau @MrBenJ Good stuff, thanks!

@Leodau @MrBenJ @Murderlon Thanks all!! 🎉


update index.js pl

Summary
Since the admin panel in Polish were written in a mix of Polish an English,
folowing the index.js file for en locale I provided the translations missing from the locales > pl > index.js file.

Test plan

Checklist
Please add a x inside each checkbox:

[x] I have read the contribution guidelines.
[x] Code is formatted via running yarn format.
[x] Tests are passing via running yarn test.
[x] The status checks are successful (continuous integration). Those can be seen below.

A picture of a cute animal (not mandatory but encouraged)

@annakocot, looks like CI is failing. Can you make sure to run yarn format and yarn test per https://github.com/netlify/netlify-cms/blob/master/CONTRIBUTING.md#pull-requests

Sorry about that, i think i got it fixed


Provide default role

Hello Ivy Team,
Thanks a lot for providing the CMS Editor. We are very happy about this tool and its functionalities.
After the deployment, there is only a small "thing" that we are missing: a default role.
Is it possible to add a default role to the project (not the demo version), which can be used directly after deployment to grant access to the CMS Editor?
Then it's really "ready-to-use". :-)
Thanks a lot!

Hi @tvtphuc-axonivy,
Thx for your message. The Axon Ivy Portal Admin role is named AXONIVY_PORTAL_ADMIN. Well, I suggest to use CMS_ADMIN to keep your formatting.

Thanks @mbrueckner-kdx. I will take CMS_ADMIN as role 's name

Hi @mbrueckner-kdx, may I ask which version of the CMS Editor you are using? Is it version 10.x.x, 11.x.x, or 12.x.x?

Hi, the request was for 11.x, but we will use 12.x in future.

Hi, the request was for 11.x, but we will use 12.x in future.

Hi @mbrueckner-kdx , can you increase the version and use version 12 ?? because the version 11 is not supported any more.

Yes, we can and we will do it :-)


Adds upgrade failed callback to web sockets

This, or some variant of it, is necessary for a consumer of the API can deal with non-101 http status codes from the server via RFC2616 (also specified in the WebSocket specs RFC6455 Section 4.1)
An alternative implementation might be to augment onFailure() to supply the Response and Reason, but this approach makes the upgrade path explicit.
commit 8ce30f297f81c19d44ff0de2f7d31c3c07cdb0c8
Author: Peter Werry<EMAIL_ADDRESS>Date:   Tue May 12 12:01:49 2015 +0100
Update websocket tests for new onUpgradeFailed() callback

commit bd4db0ced6ad1944bbc111b23a2a7ff9dbb2085d
Author: Peter Werry<EMAIL_ADDRESS>Date:   Sat May 9 14:57:33 2015 +0100
Adds onUpgradeFailed() WebSocketListener callback

Just curious: when would this happen in practice? Ie. do real applications need to differentiate?

We talked about this at some point. A potential solution would also be to deliver the nullable Response to onFailure. I'm really hesitant about yet another callback... we have a lot as is.

+1 to optional response to onFailure().

I'll give an example of why you would need to receive the HTTP response from the server:
You have an application that uses long living tokens to connect to the websocket and identify a particular session. When the application connects the server can return different errors like token expired, token invalid, application needs upgrade, etc. Then the application can decide what to do based on the HTTP response.
It's one of the advantages of the websockets, you can use the upgrade step (by setting headers) as an authorisation/authentication step and only upgrade after this is accomplished.

Jake thanks for getting back. I agree about the added callback being a bit clunky. I'd be happy to bin this PR and issue a new one with a nullable response to onFailure(). Is this the flow you guys would prefer? Or would you rather I amend to this one to maintain the discussion?

Going to look at this tonight because I'd really like this to make it into 2.4 which is coming very soon.


docs: Add an example field to argo archive list command. Fixes #11898

Fixes #11898
Motivation
Changes to support more detailed information when the user runs the argo archive list command
Modifications
Added examples for some common usages of argo archive list command

Verification
RUN argo archive list --help

@agilgur5  Could you help point out where and why the codegen test is failing? I used the same spacing format as before for the other command, but it is still failing!!

Superseded by #13662


test: add tests

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Work on placing work close to execution

I didn't find that part in the paper, are we not posting that part for review?

I finished the review and other then this issue it looks fine. Thanks.

I added a section in the proposal overview which describes how this works in Binding execution and allocation to resources, and there's an example there, of binding allocation and work to an execution context.

Already addressed on the paper.


Full text search with "_" doesn't give expected results

Describe the bug
Full-text search with "_" doesn't give expected results. See also knora-api issue #1434.
To Reproduce Steps to reproduce the behavior:
Given this resource:

The following search in unsuccessful:

This one is even worse:

OPTIONAL: Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem (drag-and-drop the image directly here).
Desktop (please complete the following information):

OS: macOS 10.14.6 (18G103), French
Browser Firefox
Version 70.01.1

Additional context
Add any other context about the problem here.

this issue has been moved to YouTrack: DSP-357


awkward api due to code duplication between EphemeralSecret and StaticSecret

new(), diffie_hellman(), pubkey computation etc. are all the same. This leads to a bit of an awkward use of the API downstream, as e.g. noise-rust-crypto is basically picking one of the two types arbitrarily to perform diffie hellman, or to compute a pubkey.
It would be great to have a `SecretKey`` that implements all those things, and have the two special secret types just wrap it.

Hmm, an EphemeralSecret is just a StaticSecret where the compiler enforces at compile-time that the key is only used once.  If you don't want to have two code paths, this means that you're OK with relaxing this compile-time verification.  So it seems like an alternate solution would be to do
type SecretKey = x25519_dalek::StaticSecret;

and use that SecretKey type for both static and ephemeral keys.  Would that work?

where the compiler enforces at compile-time that the key is only used once.

I see, so maybe it's better for the downstream library to also do that.
Thanks for the clarification.

Maybe, maybe not! I think that for a known protocol like Noise that already handles the static/ephemeral distinction, it may not be helpful to have two code paths with two different types. If it's not, you can use the type alias as above, but if it is, you can have two code paths.


unify license name

Weird I could have sworn I standardized all of these a while back. We're trying to use the standardized short identifiers from the Linux Foundation's SPDX license list - can you update the changed files to use those?
https://spdx.org/licenses/

I'll do that in a few days.

Let's pretend all GPL family license are version only because it would consume to much time to check if they are version only or version and later.

:heavy_check_mark: done @erquhart

bump @erquhart

So sorry for the delay on this - thanks @noraj!


Error when invoking the sample command

Installed the plugin via vim-plug. Then, tried running this in command mode:
:call notification#show('Hello World')
but there's an error and no popup is shown:

Am I missing something? Thanks

What version of vim are you using? Could you tell me the full output of :version?

popup_create() is added in Vim 8.1.1364+.
Your Vim version is too old or your Vim is neovim.
Note:  I think vim-notification should check the version though.

@4513ECHO @Shougo I'm using Neovim 0.6.1 (Jan 01, 2022 release):
NVIM v0.6.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>
Is this expected not to work in Neovim? Thanks

vim-notification doesn't support neovim plugin.
So it is expected.
You should use nvim-notify instead.

@Shougo Ok will close this issue then. And thanks for recommending nvim-notify, realized that there are other notify Vim/Nvim plugins!


Is PunctuationMode implemented?

I tried to use the PunctuationMode setting when doing speech recognition, but no luck. Regardless of what value I give to the setting, the output is the same.
In the input audio file I use the speaker clearly indicates full stops and commas, by punctuating the speech. The output has the words "full stop" and "comma" inserted in addition to the "." and ",".
Is this functionality implemented or this is a parameter that is planned to be used in the future.

@wolfma61 @PanosPeriorellis Could you comment on this ?

I assume you are using batch transcription and are setting PunctuationMode accordingly?
I just tested, and I got result with punctiation commands replaced with the correct punctiations.
I could be an issue with the audio? Can you give us some sample audio?

Hello,
Apologies for getting back to this so late. It's a side project. Here are the details:

I use batch transcription (and translation)
I implemented this both in python and java using the examples you provided. I include here snippets of code in python
I use uksouth service region
Initialisation:

from_language = 'en-GB'`
translation_config = speechsdk.translation.SpeechTranslationConfig(subscription=speech_key, region=service_region, speech_recognition_language=from_language, target_languages=target_languages)

I was not sure how I set the punctuation mode. I did it using translation_config.set_property_by_name("PunctuationMode", "Dictated")
You can download the wav file from:
WAV file
The output is (the first three sentences)

Recording for Constantine full stop.
Ladies and gentlemen comma you have described the pay gap that exists column there 16 % comma and more in some places full stop.
Of course comma. We also have the pace the pension bake up comma, which is very comma very serious full stop.

As you can see the system transcribes "full stop" and "column". I thought setting PunctuationMode to Dictated deals with this. What I am doing wrong?
Thank you

It looks like you're using what we call online transcription (where audio is streamed for transcription, as opposed to audio files being transcribed in bulk. Batch transcription is described here).
The setting to enable dictation mode for punctuation is SpeechConfig.enable_dictation().

Closing the issue as resolved, please re-open if more information needed.


Some of the Cookbooks are missing from the list of Cookbooks

Type of issue: documentation

Impact: no functional change

Development Phase: request
Other information

If the current behavior is a bug, please provide the steps to reproduce the problem:
There has been new content added (new Cookbooks), but not all are listed on the cookbooks "table of contents": https://github.com/chipsalliance/chisel3/blob/master/docs/src/cookbooks/cookbooks.md
This means they do not currently show up on the website (which was probably intentional @azidar , @jackkoenig ?) but they will also not show up once we release 3.5.0 and switch the website to refer to that.
We should look at all the cookbooks and update the Cookbook Table of Contents to make sure they are all captured.
What is the current behavior?
Only some of the cookbooks (in https://github.com/chipsalliance/chisel3/blob/master/docs/src/cookbooks/) are listed on: https://github.com/chipsalliance/chisel3/blob/master/docs/src/cookbooks/cookbooks.md
What is the expected behavior?
All of the cookbooks, especially DataView and the Hierarchy one, should be listed show up as of 3.5.
These should also show up on the sidebar at https://www.chisel-lang.org/chisel3/docs/cookbooks/cookbooks.html, this would need to be updated https://github.com/freechipsproject/www.chisel-lang.org/blob/ef93edd993b9e64a26d076626597b0ca10e3f696/docs/src/main/resources/microsite/data/menu.yml#L18 but I don't think we want to do that until after the 3.5 bump, so maybe that should be its own issue?
Please tell us about your environment:

What is the use case for changing the behavior?
Find the new and exciting content for the 3.5 bump!

Hello @mwachs5  When I click on the present cookbooks it gives a 404 page.

@Burnleydev1 what link exactly is giving you the 404? I just checked them again (I think!)

These ones

General Cookbooks
Naming Cookbook
Troubleshooting Guide

Okay thanks for the clarification @mwachs5

Thanks for the clarification I'll work on it right away

Specifically, in this directory the hierarchy and naming docs don't show up on that front page:
https://github.com/chipsalliance/chisel3/tree/master/docs/src/cookbooks

I can actually see the naming docs on my end but I'm not seeing the dataview docs

ah yeah you are right i mean hierarchy and dataview.

Oh, then its more complicated than I thought it was.


can you update these dependency

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
supabase-py 0.0.2 requires gotrue==0.2.0, but you have gotrue 0.5.0 which is incompatible.
supabase-py 0.0.2 requires postgrest-py==0.4.0, but you have postgrest-py 0.10.2 which is incompatible.

The pypi package is called supabase instead of supabase-py, try installing the first.
De: @.>
Enviado: lunes, 23 de mayo de 2022 1:04
Para: @.>
CC: @.***>
Asunto: [supabase-community/supabase-py] can you update these dependency (Issue #212)
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
supabase-py 0.0.2 requires gotrue==0.2.0, but you have gotrue 0.5.0 which is incompatible.
supabase-py 0.0.2 requires postgrest-py==0.4.0, but you have postgrest-py 0.10.2 which is incompatible.
—
Reply to this email directly, view it on GitHubhttps://github.com/supabase-community/supabase-py/issues/212, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AP7W3QOOZEUXX36JIGKLDOLVLK4ORANCNFSM5WUBY4WA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

The pypi package is called supabase instead of supabase-py, try installing the first.


Notify user if pointing to incorrect FluidSynth architecture

this is a bit of a PEBKAC but i spent actual days trying to figure out why fluidsynth wouldn't load for me until i pointed foo_midi to a 32-bit version on a whim (my jaw subsequently dropped to the floor after clicking help->about and seeing x86). i think it would greatly help everyone if there were some kind of robust, specific error or the like thrown to foobar's console if you load a mismatched dll instead of Literally Nothing

Noted.  I'll add some extra error messages in the next version.


Problem with decompiling function that has more than 2 returns(c/c++).

Describe the bug
It don't decompile correctly. When decompiling a function with multiple returns, the type of the function will be regarded as void ,but it should be int or so. And when compared with IDA Pro, I noticed there are missing returns. It's quite annoying.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Correctly analyses functions with multiple returns.
Environment (please complete the following information):

OS: Linux & Windows
Java Version: 20 OpenJDK
Ghidra Version: 10.3.1]
Ghidra Origin: official GitHub distro

In order for us to respond in any meaningful way, can you please provide a specific example with a sample binary attached?


Exception trying to run coursier on Windows 10

I get an exception trying to run coursier on Windows 10.
I downloaded coursier with `curl -Lo coursier https://git.io/coursier-cli"
Running from git-bash I get
Exception in thread "main" java.lang.ExceptionInInitializerError
        at coursier.bootstrap.launcher.d.c(Unknown Source)
        at coursier.bootstrap.launcher.a.a(Unknown Source)
        at coursier.bootstrap.launcher.Launcher.main(Unknown Source)
Caused by: java.lang.RuntimeException: java.io.IOException: Cannot run program "powershell.exe": CreateProcess error=5, Access is denied
        at coursier.bootstrap.launcher.l.a(Unknown Source)
        at coursier.bootstrap.launcher.l.a(Unknown Source)
        at coursier.bootstrap.launcher.j.a(Unknown Source)
        at coursier.bootstrap.launcher.h.a(Unknown Source)
        at coursier.bootstrap.launcher.g.a(Unknown Source)
        at coursier.bootstrap.launcher.b.<clinit>(Unknown Source)
        ... 3 more
Caused by: java.io.IOException: Cannot run program "powershell.exe": CreateProcess error=5, Access is denied
        at java.lang.ProcessBuilder.start(Unknown Source)
        ... 9 more
Caused by: java.io.IOException: CreateProcess error=5, Access is denied
        at java.lang.ProcessImpl.create(Native Method)
        at java.lang.ProcessImpl.<init>(Unknown Source)
        at java.lang.ProcessImpl.start(Unknown Source)
        ... 10 more

I get the same exception running coursier.bat from Windows Command Prompt.
I can run powershell.exe myself with no problem.
I'm running Java 8
java version "1.8.0_221"
Java(TM) SE Runtime Environment (build 1.8.0_221-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.221-b11, mixed mode)

Do you have any ideas what else I might try? Is there perhaps a workaround where I can tell coursier what shell to use?

This handled by @soc's directories-jvm library, powershell is run around here.
I'm not sure why or how we can get that error.

Could this be a corrupt powershell.exe like described here or there?

Could this be a corrupt powershell.exe like described here or there?

I don't think so, because I can run powershell.exe just fine from an elevated or non-elevated command prompt,  or also from bash.
Thanks for the pointer to the relevant code. Does coursier require powershell?

@utahkay Yes, PowerShell is required. Any further debugging hints are appreciated to identify the root cause.

@soc Maybe directories-jvm could accept a custom path to powershell.exe, via a Java property say? That would be helpful to debug those cases.

@alexarchambault Sure, if it helps! But I think it's weird because we are simply relying on the PowerShell that the system gives us ... even if we could specify the path explicitly, I have a bad feeling about second-guessing the operating system.
@utahkay Is there anything special about your system? Developer settings on? WSL1/WSL2 installed? etc.


As a user, I want to be able to search by tags

(friends, colleagues, etc)

As there are already tags implemented, will use that and make them searchable instead.


Fix protocol detection regexp

"https://github.com/Shougo/dpp-protocol-git".match(/^(?<protocol>[^:]+):(?<host>[^\/]+)\/(?<name>.+)/) fails because // pattern is missing.
This adds it.

Thanks.


Use attribute ops API for C-FIND query construction

findscu can now describe nested data elements in the C-FIND queries!

Nice feature!


Website Styling and SEO

Would like to copy similar styling for the solar-scorecard portion of the website. (i.e Background, Scroll Effects, Navigation Bar, etc.)
And have it linked to the repo so that it automatically updates when main page is updated.

@ijskor
I think I saw you mention something in Slack about some design ideas. If so, does this issue still apply? It's certainly doable, but it could get messy dealing with two separate repos.
Any thoughts on taking the lazier (but potentially prettier) approach of using an existing style like Google's Material Design?

I like the idea of a quick design approach.
My friend designed a general layout idea that we can work from.

Going to roll with Material Design. Nice and cohesive and well-thought-out. Will have a better result than if we Frankenstein it.


Add README for edi integration samples

Description:
$subject in the repo
https://github.com/ballerina-guides/integration-samples/tree/main/edi-package-generation

Even though both edi-package-generation and edi-code-generation are kept under integration samples, they are not mentioned under integration samples listed in the website.
These were initially used for EDI Tool Learn page. However, with the latest revamp the usages are now embedded in the page https://github.com/ballerina-platform/ballerina-dev-website/blob/master/swan-lake/integration-tools/edi-tool.md


Wrong separation before special characters

The CamelCaseToSeparator does not work correctly when having a lower-char-character and then a special-character, for example:
<?php
$filter = new \Zend\Filter\Word\CamelCaseToDash();

echo $filter->filter('Möbel'); // Möbel - correct
echo $filter->filter('Zubehör'); // Zubeh-ör - NOT correct

This happens to be the case at all times when there is a lower-char character in front of a special character. As this level of regex is too complex for me, i can't provide a fix for this.
It's easily noticeable when having this output:
<?php
$filter = new \Zend\Filter\Word\CamelCaseToDash();
foreach(range('a','z') as $i) {
    echo $filter->filter($i.'ä').'
';
    echo $filter->filter(ucfirst($i.'ä')).'
';
    echo $filter->filter($i.'ö').'
';
    echo $filter->filter(ucfirst($i.'ö')).'
';
    echo $filter->filter($i.'ü').'
';
    echo $filter->filter(ucfirst($i.'ü')).'
';
    echo $filter->filter($i.'ß').'
';
    echo $filter->filter(ucfirst($i.'ß')).'
';
}
?>

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Setting array state with empty array results in array with empty string

I have this code
  const [locationIds, setLocationIds] = useQueryState(
    'locationIds',
    queryTypes.array(queryTypes.string).withDefault([]),
  );

When calling this
setLocationIds([]);

It results in an empty query: ?locationIds=, and the state is populated with ['']. I worked around it by wrapping the setState with a check, but it's suboptimal. When setting an empty array, it should not be populated with an empty string

Facing the same issue. Does anybody know why this is happening?

The issue is in the array parser:
https://github.com/47ng/next-usequerystate/blob/aa9609841c8d9b1d09efe76513586f5db8c5c448/src/defs.ts#L197
"".split(',') returns [''], so to properly handle the empty array case, a test should be performed on the query value (once trimmed) to see if it's an empty string, and if so return an empty array explicitly.

The issue is in the array parser:
https://github.com/47ng/next-usequerystate/blob/aa9609841c8d9b1d09efe76513586f5db8c5c448/src/defs.ts#L197
''.split(',') returns [''], so to properly handle the empty array case, a test should be performed on the query value (once trimmed) to see if it's an empty string, and if so return an empty array explicitly.
Would you like to open a PR?

Sure, sending in a PR

Thanks guys!

Fixed in #330 by @prasannamestha, thanks!

Fix is shipped in version 1.7.3.

🙌


minimizable, maximizable, and closeable

How to do that? Should this feature be added to this library?

Yes I was planning to add this feature. I'll try to do it today.

Thanks, my current solution is a bit messy, so yeah I'll update the library as soon as the feature implemented

Options added in v0.8.0. Thanks for your interest.


TravisCI tests are failing

Since https://travis-ci.org/tabatkins/bikeshed/builds/97584799 on Dec 17. https://github.com/tabatkins/bikeshed/compare/2a7526fd5a28...dbca5934a813 broke it.

fixed


Quasar not found vue-interactive-paycard

I installed this into my quasar project, but when I try to run the project I'm getting this error:
`ERROR  Failed to compile with 1 errors                                                   10:16:18 PM
This dependency was not found:

boot/vue-interactive-paycard in ./.quasar/client-entry.js

To install it, you can run: npm install --save boot/vue-interactive-paycard
`
Please any info will be appreciated, thanks
Thanks

I was missing to create a file inside boot folder.


Overhaul rendering to stream results in real time

Closes: #489
Up until now (post-concurrency changes), we've been rendering results in a deterministic way by storing all of the file paths in a slice, sorting the slice, and then iterating through a map using the sorted keys. This prevents any output feedback while a scan is actually occurring and is especially apparent when scanning directories with many files.
This PR implements real-time streaming of results using a handy package that maintains order as best as possible when using concurrency. I was originally trying to roll a custom priority queue implementation and this eliminated the need for that luckily.
As far as I can tell, the output is pretty consistently sorted, but I added explicit sorting to two of our tests to ensure that the result data is what we expected. We probably shouldn't care about a deterministic scan output in general (especially when some files will take longer than others to scan).
I also verified that --err-first-hit and --err-first-miss also work with this implementation (multiple runs to show consistency):
$ for i in (seq 1 3); go run cmd/mal/mal.go --err-first-hit analyze /usr/bin/; end
🔎 Scanning "/usr/bin/"
👋 "/usr/bin/AssetCacheLocatorUtil": matched requested condition
🔎 Scanning "/usr/bin/"
👋 "/usr/bin/AssetCacheLocatorUtil": matched requested condition
🔎 Scanning "/usr/bin/"
👋 "/usr/bin/AssetCacheLocatorUtil": matched requested condition
$ for i in (seq 1 3); go run cmd/mal/mal.go --err-first-miss analyze /Library/Application\ Support/BTServer/; end
🔎 Scanning "/Library/Application Support/BTServer/"
👋 "/Library/Application Support/BTServer/pincode_defaults.db": matched requested condition
🔎 Scanning "/Library/Application Support/BTServer/"
👋 "/Library/Application Support/BTServer/pincode_defaults.db": matched requested condition
🔎 Scanning "/Library/Application Support/BTServer/"
👋 "/Library/Application Support/BTServer/pincode_defaults.db": matched requested condition

This PR also restructures scan.go to group related functions and make the code more readable since recursiveScan was getting to be a massive function.

Converting this to a draft to refactor the refactor to use standard Golang constructs. I also found a data race in diff.go which needs to be fixed.


categorial variables

Hi,
very nice work.
I would like to know how would you interpret contribution of categorical variables.
My data is mixed, containing continuous and categorical features.
I have 2 output neurons.
so if particular categorical variable has high score then how would you interpret it ? For example I have one variable family_history with values 0 and 1. Deeplift gives high importance score to this variable. So which category of this variable is important actually ? Having family history (1) or not having it (0). Hope my question is clear
thank you,

Hi @misbahch6,
It would depend on three things:
(1) What was the reference a.k.a. baseline that you used?
(2) Which output task are you computing the importance with respect to?
(3) What is the sign of the importance that you observe for a particular example?
Say you used a baseline of all-zeros. Now consider an example where the family history is a 1. The difference-from-baseline for family history for this example is a +1 (because 1-0 = +1). Now let's say that when you compute the scores for output neuron 1, you observe positive importance for family history, whereas if you compute the scores for output neuron 2, you observe a negative importance for family history. This indicates that family history being "1" (rather than zero) contributes positively to output neuron 1, whereas it contributes negatively to output neuron 2. In all, this suggests that a family history of 1 is important for receiving a high score in output neuron 1, whereas a family history of 0 is important for receiving a high score in output neuron 2.
Does that make sense?
Best,
Avanti

thank you so much. it is super clear response
:100:


Isolate dev code

i'm a terrible reviewer! treemap.css is missing... @kieferm mind adding it in master?

I think i had that file in another branch that was not yet merged.  In either case I just added it to master.


Adding slides and websites of tutorials

I (as a tutorial co-chair of ACL 2022) am wondering how we can associate tutorial slides from ACL Anthology. There are probably two ways to do this:

Collect tutorial slides from instructors and include them as an attachment. I found one instance in https://aclanthology.org/events/acl-2019/#p19-4

In order to realize this, I may need to specify a file in papers.yml:
  badges:
    - type: Presentation
      url: presentations/1.pdf

Add a link to the website of a tutorial and ask tutorial instructors to upload the slides in the website. I've never seen this instance on ACL Anthology. I guess that the corresponding papers.yml could include something like this:

  badges:
    - type: Web
      url: https://some-url-of-the-tutorial/

I think the option 2 would be convenient because tutorial instructors can update their slides in the last minute or evan after a presentation. So, my questions are,

Is it allowed to put type: Web in badges?
Can ACL Anthology show a URL with this badge?

I think that you just add your file to the attachment folder and use the attachment field.
In the field attribute you should apply the local path to the file (e.g.,   - file: attachments/90_zip) or a URL (e.g.,   - file: https://some-url-of-the-tutorial/).
Finally, the attribute type type can be set to presentation.
Here a possible example
  attachments:
  - file: attachments/1.pdf
    type: presentation

@mjpost do you confirm it?

In any way, please refer to this section of our "guide"
https://github.com/rycolab/aclpub2#papersyml

I read this guide before.
But I also noticed that attachments fields were probably replaced with badges in sigdial example. Which is correct?
I'm not sure what kind of strings are allowed in the type field. The documentation says,
- type: Dataset | Note | Poster | Presentation | Software | Attachment

but I can only see relased_code and released_model_artifact in the sigdial example. These lowercase fields are not mentioned in the documentation.

This looks correct (what Daniel noted). released_code and released_model_artifact should not be used. We could add code as another option, however.

I think that you just add your file to the attachment folder and use the attachment field. In the field attribute you should apply the local path to the file (e.g.,  - file: attachments/90_zip) or a URL (e.g.,  - file: https://some-url-of-the-tutorial/). Finally, the attribute type type can be set to presentation.
Here a possible example
  attachments:
  - file: attachments/1.pdf
    type: presentation

@mjpost do you confirm it?

@mjpost @chokkan you are right, the example was incorrect.
I updated it.
Notice that according to the guide, the attribute file can be used to express a file in the submission, while url is a link to external resources that should be linked in the Anthology.
Are the example and guide clear now for you?

@crux82 Thank you for updating the example and documentation. They are clear to me.


Fix Lint/UselessAccessModifier false positive (#6534)

Resolves https://github.com/rubocop-hq/rubocop/issues/6534

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Looks good to me. Can you squash your commits into one?

Thanks!


How do you use this with jest.requireActual?

const mockConsole = jest.requireActual( "jest-mock-console");

describe('api calls', () => {

  beforeAll(() => {
    expect(jest.isMockFunction(mockConsole)).toBeFalsy();

Except that line is failing the test.
I'm using automock: true, in my jest config, which is clearly part of the issue. But I should be able to use requireActual to get around that. But it doesn't appear to work.
Also tried:
const {mockConsole} = jest.requireActual( "jest-mock-console");

But then the test fails:
TypeError: mockConsole is not a function

> 11 | const restoreConsole = mockConsole();

Also tried
const mockConsole = jest.requireActual( "jest-mock-console");

But then the mock-check line expect(jest.isMockFunction(mockConsole)).toBeFalsy(); fails.

Next up I tried grabbing some code from the readme and turning automock off in the jest config:
import mockConsole from "jest-mock-console";

describe('mock console', () => {
    it('works', () => {
        const restoreConsole = mockConsole(); // mockConsole returns a function to restore it back to normal
        console.error('This will not show in the test report');
        expect(console.error).toHaveBeenCalled();
        restoreConsole();
    });
});

That's the whole file.
which fails:
Matcher error: received value must be a mock or spy function

    Received has type:  function
    Received has value: [Function error]

      23 |     const restoreConsole = mockConsole(); // mockConsole returns a function to restore it back to normal
      24 |     console.error('This will not show in the test report');
    > 25 |     expect(console.error).toHaveBeenCalled();

"jest": "27.1.0",
"jest-mock-console": "1.1.0",

✗ node --version
v14.17.6

I am using jest-mock-console  in another project with no issues -- but that project is using jest 27.0.6. Could that be the issue?

Ignore this for now, I'm getting inconsistent failures.

I'm going to close this issue for now. Please let me know if you are still having issue with it.


Streaming doesn't work correctly with caching

System Info
LangChain version : 0.0.273
Python version : 3.10.8
Platform : macOS 13.5.1
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[X] Callbacks/Tracing
[ ] Async

Reproduction
import langchain
from langchain.chat_models import ChatOpenAI
from langchain.callbacks.streaming_stdout import StreamingStdOutCallbackHandler
from langchain.cache import InMemoryCache

langchain.llm_cache = InMemoryCache()

llm = ChatOpenAI(model_name="gpt-3.5-turbo", streaming=True, callbacks=[StreamingStdOutCallbackHandler()])
resp = llm.predict("Tell me a joke")
resp1 = llm.predict("Tell me a joke")

output:
Sure, here's a classic one for you:

Why don't scientists trust atoms?

Because they make up everything!

Expected behavior
I'd expect both responses to be streamed to stdout but as the second one is coming from the memory cache, the callback handler on_llm_new_token is never called and thus the second response never printed.
I guess on_llm_new_token should be called once with the full response loaded from cache to ensure a consistent behavior between new and cached responses.

I made a bunch of fixes for Lanarky for this to work.  See https://github.com/auxon/lanarky/commit/b5904a0a58e508f227c5f3da5f6ce06758509b54, if it helps.

Does the llm.stream() work for your use case?

Does anybody have a working sample on how to do caching and streaming with OpenAI? I've looked at the code of the streaming implementation and it doesn't look like it contains any caching calls.
There's also an open ticket on this issue within the JS SDK: Caching not implemented for OpenAI LLM stream usage · Issue #3582 · langchain-ai/langchainjs


Allow checking out from tags, not just branch

The git syntax previously used to check out a branch was 'git checkout origin/'. This worked for branches but not tags such as '1.0.8-tentative'. Removing 'origin/' lets us do tags too.

But doesn't removing the origin break branches since we don't create local branch?

It doesn't seem to. I tested this syntax on the command line and it works fine for branches that both have and haven't been checked out previously.

When you do a 'git checkout foo' and 'foo' exists in the origin, it automatically creates a local branch of the same name that tracks the remote.

Ok, I tried it and it didn't worked, but maybe it's because I have multiple remotes, or because there is a notion of "original" remote.
Anyway, let's go with that then. Though in theory you can have a tag and a branch having the same name I don't think we'll ever care about that here.


reel_board: running samples/subsys/shell/fs/ failed

Describe the bug
running samples/subsys/shell/fs/, and the result just keep waiting for some inputs until the system hangs.
And do you all think this test case shouldn't be ran at reel_board in daily report.
To Reproduce
Steps to reproduce the behavior:
west build -b reel_board samples/subsys/shell/fs/
west flash
see error
*** Booting Zephyr OS build v2.4.0-rc1-8-g27e1fd69f987  ***
uart:~$
Environment (please complete the following information):
OS: Fedora28
Toolchain: Zephyr-sdk-0.11.3
Commit ID: g27e1fd69f987

Not sure what exactly fails here, this is a sample that expects input, so it needs a harness which would exclude it from automated testing.

uart:~$ help
Please press the <Tab> button to see all available commands.
You can also use the <Tab> button to prompt or auto-complete all commands or its subcommands.
You can try to call commands with <-h> or <--help> parameter for more information.
Shell supports following meta-keys:
Ctrl+a, Ctrl+b, Ctrl+c, Ctrl+d, Ctrl+e, Ctrl+f, Ctrl+k, Ctrl+l, Ctrl+n, Ctrl+p, Ctrl+u, Ctrl+w
Alt+b, Alt+f.
Please refer to shell documentation for more details.
uart:~

A harness for the above command and some of its output would cover this.

not a reel_board issue

Hi @nashif @pabigot , I totally agree with you all, but you know, daily testing is just testing automatically, and when running this test case, it will be blocked here 'uart:~$ ' to wait input until time out. And it will be reported as a failure in the daily report. Is there OK? And could you all provide a solution to solve this problem?

BTW, When using command 'sanitycheck' to run this test case, human-computer interaction cannot be performed and report a timeout failure directly.


Manually add size hint to multipart Part

I've been working on a project that sends files to an API using multipart/form. However, if I want to load these files as a Stream, the Content-Length header and this causes most servers (such as those created with warp) to reject the body. After looking at the implementation details, it looks like Part::stream has no way to indicate its size, even when I manually implemented http_body::Body with a wrapper type due to the WrapStream type being used underneath the hood in the crate. I am trying to avoid reading large files into memory and then passing them as raw bytes.
I was wondering if I could add a new method to Part called stream_with_length(stream, length) so that I can manually specify the length of the file I am sending. I'm perfectly willing to implement it, but just wanted to see if it would be something that would be accepted

Yep, I'd be for adding such a constructor. Just to check, do you need to set the header for a specific part in the body, or the top level request header?
POST /blah HTTP/1.1
Content-Length: 300 (this one?)
Content-Type: multipart/form

--boundary
Content-Disposition: form-data; name="foo"
Content-Length: 200 (or this one?)

some 200 bytes
--boundary--

It would be for each part so the multipart logic can correctly calculate the top level content length

Sounds good!

Sweet! I'll go ahead and work on this tomorrow. I don't think it should be too difficult!


Inquiry to Training Resources

Hi,
thanks for your great work.
For the training resources, I found you mentioned that, to train open-sora we need maybe 128-256 A100 gpus to train 1 week in #157 .
Could you explain the resource requirements in more detail? For example in VAE training, how much resources are needed? And in t2v training how much?
And could you explain why so many resources are needed in the project? I found another project that needs much fewer. colossalAI
"在我们的重现过程中，我们使用了 64 个 H800 GPU 进行训练。第二阶段的训练量总计为 2,808 GPU 小时，约合 7,000 美元，第三阶段的训练量为 1,920 GPU 小时，约合 4,500 美元，我们成功地将Open-Sora 复现过程保持在约 10,000 美元。"
Thanks.

Thanks for your reply.
Do you mean the resources mentioned in #157 are just used to train the LatteT2V model?
Could you please offer more details about how the resources are used and estimated? Such as num of frames, dataset, batch size, resolution, etc.
It seems that the training script in the text_conditioned folder does not support 1 min videos.
Thanks a lot.


Add Swift linter to CI

Fixes #2157

Fixing a few linting issues

I'll fix the Connlib.xcfilelist in a subsequent PR.

A minor issue: Since CallbackHandler.swift has moved, it should be removed from swift/apple/FirezoneNetworkExtension/Connlib.xcfilelist as well.

It actually moved into that directory, so I think the file list is actually valid?

The Connlib.xcfilelist specifies which files are copied from rust/connlib/clients/apple in the build process. This is to make peace with the Xcode build system, telling it that this file might be missing now, but it's okay because it will be created during the build.

Ah, got it. Thanks for clarifying


suggest to fix markdown

It was OK on GitHub.com preview but not good on Docs.Microsoft.com web.

@yoichinak : Thanks for your contribution! The author, @markingmyname, has been notified to review your proposed change.

Verified editorial correction.


Vagrantfile should read IP address from JSON configuration file

@clubdeuce I would rather have the command update Vagrantfile based on the JSON configuration.
I'd rather minimize the work Vagrantfile does and have it be a configuration output artifact rather than have Vagrantfile do much work.
I'm going to close this ticket as won't fix and add another one here: https://github.com/pressboxx/guest-cli/issues/32


Review morning session

-Fetch API data
-Build with react components
-Followed JS best practices.
-No linter errors.

Status: Approved :tada: :heavy_check_mark:

Hey @EvansSnave ,
Your tests are complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

Good README file. :heavy_check_mark:
Correct Gitflow. :heavy_check_mark:
No Linters error :heavy_check_mark:
The App work as Expected :heavy_check_mark:
DRY :heavy_check_mark:

Optional suggestions
[OPTIONAL] It would be helpful to have a more descriptive summary: To know the changes, why you made them, and their impact. The more descriptive summary would make it easier to review your pull request and give you feedback.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me (@Grandi0z) in your question so I can receive the notification.


Graphs tab on TensorBoard failing with a JavaScript error

Built from TOT today (Feb 10th) with GPU support.  TensorBoard loads and the Events tab works, but when I go to the Graph tab, I see Uncaught TypeError: Polymer.dom(...).unobserveNodes is not a function.
I'm a TensorFlow rookie, so good chance I did something wrong.

@mebersole Can you clarify "built from TOT"? Did you built it using bazel or using gulp? The right way is to run:
bazel build tensorflow/tensorboard
bazel-bin/tensorflow/tensorboard/tensorboard --logdir path/to/log/dir

I did this today and tensorboard is working.  In case you are building your own front-end (typescript, html), in which case you are running bower install, make sure you choose polymer 1.2.4 if bower asks you for which version of polymer. I tried running bower install from scratch, and purposely chose polymer 1.1.5 which resulted in the same error (Uncaught TypeError: Polymer.dom(...).unobserveNodes is not a function) that you have.
Hope this helps!

@dsmilkov Thanks for the help.  I just cloned 967cd485 to avoid issue #1076 (I worked around it by building and installing a pip package) and built with bazel build -c opt --config=cuda tensorflow/tensorboard.  I then ran with bazel-bin/tensorflow/tensorboard/tensorboard --logdir /home/ubuntu and the Graph tab is still giving me the Uncaught TypeError: Polymer.dom(...).unobserveNodes is not a function error and showing a blank page other than the header bar (title and tab links).  Any suggestions?

I should say I worked around #1076 when I first reported this issue.  I can now run TensorBoard directly with bazel.

Also, I have this set up on a test AWS instance.  If it's easier, I can pass you the login information and you can look at it live.

For what it's worth I'm having exactly the same problem with the latest build.    Building on Ubuntu 14.04.2 and running TensorBoard using bazel.

@mikehaley did you build with GPU support -c opt --config=cuda?  Maybe that will help narrow down the issue.

Yes.  Built with CUDA.   Also updated polymer to 1.2.4 @dsmilkov suggested.

Hi guys,
If you open the console while your browser is pointed to <IP_ADDRESS>:6006 and type Polymer.version do you get 1.1.5 or 1.2.4 or something else?
If you get 1.1.5 when you need to update to 1.2.4. To do this, download polymer 1.2.4. Unzip and copy the 3 html files (polymer*.html) into the external polymer directory under site-packages dir: /usr/lib/python2.7/site-packages/external/polymer (overwrite the previous files).
Let me know how it goes. We will also be releasing a patch for 0.7.0 where we ship Polymer 1.2.4 with it.

Going to 1.2.4 of polymer worked for me.

Thanks @dsmilkov.   Forcing polymer to 1.2.4 works.   It was indeed running 1.1.5 before despite the settings in bower.json.

Closed in #1134


[Meson] dependency to build ml-service

Code clean to build ml-service,

fix dependency to gio library.
remove unnecessary declaration in meson script.

Signed-off-by: Jaeyun<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #229. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Send a 406 when package is already registered

The registry-client is already listening for this code, so we should send it.

Client has been already updated to handle 403, but I liked the tests, so I commited them: https://github.com/bower/registry/commit/c36cfdb979f234ca2d6b4974778cd73bd4af661d


Replace module on the motherboard

Hello! Tell me if there is somewhere an instruction(tutorial) on how to replace the BCM94360NG M.2 WiFi / ac and BT4LE module on the motherboard. My motherboard asus z390-e Gaming she looks like yours.

There really isn't much to make a tut on.  There are some spring loaded retainers that hold the IO Shield, take photo of how they go in, and then remove them. IO Shield should come off, leaving the WIFI module EMI shield. Unscrew the screws that hold down the EMI shield, and take out the module. The only issue I had was the head sinks were not tall enough to house the BCM9436XX w/ the m.2 adaptor. I had to Dremel/ mill out an opening for my WIFI module to fit.

Thank you.


Wait for rasa --help output for 20 seconds on Windows

Proposed changes:

increase the timeout for test_cli_start from 10 seconds to 20 seconds

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

Thanks for fixing this @alwx Should we create an issue to debug our CI starting times? I assumed top level imports of the wrong things are slowing us down once again.

The fix was already just pushed to master :( https://github.com/RasaHQ/rasa/commit/23262e1280feb7c6ef43b56186d9e26a6f1ab6bb

Can we also include this in the release branch please?


Canister slider snaps back to minimum when the UI is opened

Can't reproduce. I assume its been fixed at some point.

Can't reproduce. I assume its been fixed at some point.

This definitely still happens.
You open the UI, raise the pressure, close it. And when you reopen it the pressure pops back to 0.

Oh... I think I was accidentally testing on dirty/modified branch


Unity 4 settings 100% icompatible with Unity 5 - setup documentation, please!

Not a super-advanced programmer with 10+ years of unity experience under the belt, still trying to figure out all this injection stuff. Have a project due, seconds counting down and the entire project is a sea of red because of Unity 5.

DotNet 4.6.2
MVC 5

I need information on how to set up Unity 5 under MVC 5. The files generated by Unity 4 do not work, and cannot work with Unity 5. As just one example,
UnityConfig.RegisterComponents();
in Global.asax.cs no longer works because… drum roll …RegisterComponents() no longer exists in UnityConfig.cs with Unity 5
There are 100 other items I could bring up that suddenly went sideways, but I am not even sure if they are related to Unity 5, I am just trying to get one small cog to work right with zero information available. There isn’t even documentation under the wiki!
And attempting to revert back to Unity 4 just broke a dozen other things, causing things like System.Web.Mvc, System.Web.Helpers and other things to be dropped (and not findable/re-referenceable) from the entire system. My controllers can’t even extend Controller because System.Web.Mvc is no longer being referenced and v5 is no longer findable to be referenced.

If you have such a tight schedule, I would suggest to stick with v4.0.1 for now. It is tried and true and it works.
If you insist on v5, I would create issue here and Alessandro will help you.

The Unity MVC package does not require Global.asax.cs to call RegisterComponents because it is called automatically by WebActivatorEx before the web application is started. About the Microsoft.AspNet.Mvc dependencies I did not experience any kind of trouble upgrading the project I am working on from Unity 4.0.1 to 5.0 except namespaces being renamed.
I understand you are in trouble right now but please, do as Eugene said: open a new bug in the Unity.Mvc project and explain what is not working for you. Also, post the package.json file of your project so I can take a look at it.

Thank you all for the quick replies.

All I am looking for is documentation. I really don’t want to bug anyone if I can muscle through anything that’s comprehensive for Unity v5 under MVC 5. Even code examples would be real swell.
I have temporarily reverted back to 4.0.1 until this project is “done” and shipped to the client, I will be using it as a testbed for a v5 upgrade path on my own time.
Since I am using MVC 5, I do not have a package.json to provide. I am still gun-shy about Core v2, and am not willing to commit until I have done a lot more testing with non-critical stuff.

Unity documentation is very old and we are trying to do our best to update it ASAP. We are few and contributors are welcome.
MVC 5 integration should be really simple: if you create a new MVC project from within Visual Studio just use NuGet to pull in the project the Unity.Mvc package and it should take all the dependencies by itself. Also the Unity.Mvc package adds two files to your project (in App_Start directory): UnityMvcActivator.cs and UnityConfig.cs. The first one is the one which integrates Unity in MVC 5; it uses WebActivatorEx to attach Unity as a dependency resolver before your application is started and dispose of it when the webapp shuts down. The other file is the one you need editing: in the RegisterTypes function you should register extensions and types to resolve or you have to load from a configuration file. The two files have some comments. Just try on a brand new project to test it. I will provide better documentation soon enough.
package.json is the file which is created by NuGet to store your dependencies from NuGet packages. You MUST have it. Check on the project directory.


reversible block database does not, replay blockchain?

Hello,
I run
./start.sh --hard-replay --wasm-runtime wabt

and wait for the replay finished. And run ./start.sh again, the logging prints
2018-10-07T12:53:44.692 thread-0   chain_plugin.cpp:666          plugin_startup       ] Blockchain started; head block is #1, genesis timestamp is 2018-06-01T12:00:00.000
...
3020000 fork_database_exception: Fork database exception
fork database exists but reversible block database does not, replay blockchain
    {"blog_head":19612486,"head":17981709}
    thread-0  controller.cpp:311 init

    {}
    thread-0  chain_plugin.cpp:669 plugin_startup

The head block is #1 and the hard-replay failed?

How you fix it?


Supervised finetuning problem

If I understood correctly, during the supervised finetuning phase, you are training the model on the whole input. If this is the case, I believe this is a mistake and making the training procedure harder than it should be. You should be only using the token labels of the assistant tokens and therefore making the model only learn the assistant behavior. If I understood it wrong, please correct me

I think you are right. Using the default data collator trains on the entire input rather than only the response from the assistant. You can use DataCollatorForCompletionOnlyLM where you can calculate loss only on the response tokens

Thank you for your response. Do you think it is hard to create a collator that is a mixture of both? The collator that fills the whole sequence seems a really good ideia

@DebarshiChanda this is correct!
@DogitoErgoSum , there is an attempt to introduce a new data collator in https://github.com/lvwerra/trl/pull/456 we'll let you know as soon as there is any progress

Alright, thank you very much!

I have another question a bit off topic but still relative. During PPO, you use the reward model to classify the input-output pair and that is your reward. Due to this, do you reward the model behavior to generate similar sentences to the input as well? If yes, can I solve this by only changing the reward model to classify only the output? Or do I need to change how the training is done in the PPOTrainer class?

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.


Phase1 doesn't take into account list of signers

At phase1, signer tries to send p2p messages to every party holding a share regardless whether this party appear in list of signers. In other words, at phase1 party sends n messages, but should send t+1 messages.
Here's the code that causes this behaviour:

Phase1::start generates list of messages. Messages are sent to every party listed in the map in phase1.mta_a
The map is constructed in Phase1::new, note that every party from keygen is included, regardless whether this party appears in signers list

Suggested solution: filter out parties that are not listed in signers list from the map in Phase1::start

ING has no intention of resolving this, nor on maintaining this further, and hence, this repo will be archived. Its usage is no longer recommended, see the updated README.


Discussion of parameters in config.yml

Agreed that the config.yml file is probably not needed in future with the extensive usage of the database. Discuss here the meaning of the individual parameters and how to replace them.
Parameter currently set in config.yml:

DB-related parameters (useMongoDB, mongoDBConfigFile): use hardwired values (what about passwords?) and/or environmental variables
modelFilesLocations: not need anymore as we expect always a connection to the database (would that work already now? see below where it is called)
outputLocation: replace by default value or overwritten by command line parameter
testDataLocation: should be replaced by hardwired value to tests/resources, best through a function defined in tests/conftest.py
dataLocation: used to get

schema files in data/parameters (camera-efficiency_parameters.yml, array-simulator_parameters.yml, ray-tracing_parameters.yml, layout-array_parameters.yml, telescope-position_parameters.yml, simtel-runner-array_parameters.yml)
layout files from data/layout/ used for e.g. for trigger sims: --> move to DB?
default CORSIKA parameters: ./data/corsika/corsika_parameters.yml
default simtel_array parameters: data/parameters/simtel-runner-ray-tracing_parameters.yml

simtelPath: use environmental variable
submissionCommand: used for production.py and sim_showers_for_trigger_rates.py : use command line or environmental variable? This is specific to the setup and we need to revisit it (current setup work (as far as I can see) with qsub; we move at DESY to HCcondor; this should be a flexible setup)
extraCommands : related to job submission and currently only used to set environmental variables correctly for running sim_telarray (ZSTD). Might add this similar as simtelPath as required environmental variable?

Issue #59 is related to this discussion.
@orelgueta - let me know your opinion, I would then prepare a list of tasks.
modelFilesLocations
find . -name "*.py" -exec grep -H modelFilesLocations {} \;
./simtools/config.py:        allLocations = get(par="modelFilesLocations")
./simtools/config.py:        "modelFilesLocations": ".",
./simtools/tests/test_config.py:            "modelFilesLocations",
./simtools/tests/test_config.py:        print("modelFilesLocations: {}".format(cfg.get("modelFilesLocations")))
./simtools/model/array_model.py:        modelFilesLocations=None,
./simtools/model/array_model.py:        self._modelFilesLocations = cfg.getConfigArg(
./simtools/model/array_model.py:            "modelFilesLocations", modelFilesLocations
./simtools/model/array_model.py:                    modelFilesLocations=self._modelFilesLocations,
./simtools/model/telescope_model.py:        modelFilesLocations=None,
./simtools/model/telescope_model.py:        self._modelFilesLocations = cfg.getConfigArg(
./simtools/model/telescope_model.py:            "modelFilesLocations", modelFilesLocations
./simtools/model/telescope_model.py:        modelFilesLocations=None,
./simtools/model/telescope_model.py:            modelFilesLocations=modelFilesLocations,
./simtools/model/telescope_model.py:            mirrorListFile = cfg.findFile(mirrorListFileName, self._modelFilesLocations)
./simtools/model/telescope_model.py:                "Using the one found in the modelFilesLocations"
./simtools/model/telescope_model.py:                "Using the one found in the modelFilesLocations"
./simtools/model/telescope_model.py:                cameraConfigFile, self._modelFilesLocations
./simtools/db_handler.py:            file = cfg.findFile(fileName, cfg.get("modelFilesLocations"))
./simtools/db_handler.py:        _yamlFile = cfg.findFile(_fileNameDB, cfg.get("modelFilesLocations"))
./simtools/db_handler.py:        yamlFile = cfg.findFile("parValues-Sites.yml", cfg.get("modelFilesLocations"))

Yes I agree with everything and we actually took care of quite a lot already in #266.
The DB password might still have to be given in a config file, unless I manage to find a smart solution.

One additional thought and to avoid ending up in a inconsistent state of configuration parameters given through the command line, through environmental variables, through default values, etc. Why aren't we more flexible and allow all?

any configuration parameter can be set through the command line parser
complex and long configurations can be given through a configuration file (in e.g., yaml format)
a subset of configuration values can also be given by environmental variables (probably no need to do this for all)
for easy repeatability - we can write the configuration parameters into a file when running an application and reuse it later.

There are packages providing this, e.g. https://github.com/bw2/ConfigArgParse
On our level, it seems almost easier to add a file reader / writer to commandline_parser module and let this one also the environmental parameters.

Yes, this is a good idea. I think the original intention was to at least not enforce the use of a config file. Making it optional is best indeed.
I don't think we should add a package for this. The functionality we need is quite basic and can be done with the commandline_parser.
Saving the configuration into a file that can be used later is a very nice idea and probably necessary for provenance anyway.

A bit confused about the configuration discussion and I really don’t want to get this out of hand and a too big endeavour.
Main goal is to simplify the handling of config.yml and best avoid it. I've scanned again through the code and this is my suggestion:

set simtelPath
—> suggest to require always environmental variable
get outputLocation
--> use default ./, otherwise set via command line
get dataLocation
--> use default ./data, otherwise set via command line
get modelFilesLocations
--> what to do? Still needed?
get submissionCommand and get extraCommands
--> this is very specific to ./simtools/simulator.py and runners: suggest to have it as a command line argument
finding a file: e.g. cfg.findFile(mirrorListFileName, self._configFileDirectory)
--> function could be part of io_handler
get useMongoDB --> not needed anymore, as always True (?)
mongoDBConfigFile: not sure what the idea is here?
--> suggest to default to dbDetails.yml and otherwise add a command line argument?

Open questions are:

Where to read environmental variables?

could be a simple addition to CommandLineParser
or add a new Configuration class, which is super to CommandLineParser

reading and writing of configuration values

argparse allows to read command line argument from file (one argument per line, see https://docs.python.org/3/library/argparse.html#argparse.ArgumentParser.convert_arg_line_to_args)
writing of configuration could be done in a similar way as addition to the CommandLineParser

Access of parameters in the code: change CommandLineParser (or if required, a Configuration class) to be a Singelton and callable.

So main question is if we want to add the functionality of getting variables from the environment and writing the command line arguments to a file to the CommandLineParser or add a new class for that. I don't have a strong opinion.
My feeling is that this solution would be not too big and disruptive.

Additional note on submissionCommand and get extraCommands: this probably can go into a the production configuration files, which we anyway require at this point (e.g., https://github.com/gammasim/gammasim-tools/blob/master/tests/resources/prodConfigTest.yml). There is already a dataDirectory field for the corsika/sim_telarray out, which is specific to the computing site.

set simtelPath
—> suggest to require always environmental variable

Please allow also command-line input

get outputLocation
--> use default ./, otherwise set via command line

Good. BTW, if these are arguments from the user on the command line, I suggest to switch to snake case everywhere (not in the code, just in the argument).

get dataLocation
--> use default ./data, otherwise set via command line

Can we change the name to inputData/input_data? Also the directory name? This is what is used now in io_handler (would require a change only there.

get modelFilesLocations
--> what to do? Still needed?

I want to get rid of this. I suggest that either you do not touch it for now and I deal with it after the PR or that we work together in the same branch and I remove it while you deal with the other config points discussed here.

get submissionCommand and get extraCommands
--> this is very specific to ./simtools/simulator.py and runners: suggest to have it as a command line argument

Sounds good.

finding a file: e.g. cfg.findFile(mirrorListFileName, self._configFileDirectory)
--> function could be part of io_handler

Sort of related to the modelFilesLocations point above. Suggest to check if can be removed completely. If not, move to io_handler.

get useMongoDB --> not needed anymore, as always True (?)

Correct, not needed anymore.

mongoDBConfigFile: not sure what the idea is here?
--> suggest to default to dbDetails.yml and otherwise add a command line argument?

Yes, though I also want to get rid of this file, but I am not yet sure we can, so let's go with your suggestion.

Open questions are:

Where to read environmental variables?

In the config module/class I think, when you define it and transfer the args to it.

could be a simple addition to CommandLineParser

I don't think it should be in the command line parser, the parser should do only that, parsing the command line.

or add a new Configuration class, which is super to CommandLineParser

Yes, that could work. The only problem is accessing the configuration class instance we discussed earlier this week, but it shouldn't be too complicated.

reading and writing of configuration values

argparse allows to read command line argument from file (one argument per line, see https://docs.python.org/3/library/argparse.html#argparse.ArgumentParser.convert_arg_line_to_args)
writing of configuration could be done in a similar way as addition to the CommandLineParser

Again, I am not sure we should expand CommandLineParser to deal also with files. Especially since I think we will use yaml for configuration rather than an argument per line system. I would pass the file name to the configuration module and read it there.

Access of parameters in the code: change CommandLineParser (or if required, a Configuration class) to be a Singelton and callable.

So main question is if we want to add the functionality of getting variables from the environment and writing the command line arguments to a file to the CommandLineParser or add a new class for that. I don't have a strong opinion.

I guess my opinion is to have two separate classes for this. Not a strong opinion, so if you find a reason to keep it all in one class (with a different name, not CommandLineParser), then I don't mind of course.

My feeling is that this solution would be not too big and disruptive.


Feature Proposal: Implement Message Queue publisher

Currently we only support db store publisher (e.g neo4j, mysql,neptune). But it would be pretty easy to support message queue publisher using the interface (e.g SQS, kinesis, Eventhub, kafka) which allows push ETL model support.
There is a pr (https://github.com/amundsen-io/amundsendatabuilder/pull/431) which unfortunately isn't get merged. The pr could be used as an example on how to support this kinda of ETL paradigm. The author has his own repo to illustrate this kinda ETL paradigm.

cc @Wonong


raco new --list fails silently

raco new --list fails silently
% raco new
% ls
% raco new -list
% raco new
% raco new -h
qi %

Thanks for making an issue for this, I've fixed it - I was unaware that I needed to tell the info.rkt to use a specific module for it to run the main.

Thank you. I was madly scrabbling through main.rkt thinking this all looks fine!


Update to 0.4.13

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


layers: Fix race issue in decompression layer

May fix issue 312

CI Vulkan-ExtensionLayer build queued with queue ID 53492.

CI Vulkan-ExtensionLayer build # 750 running.

CI Vulkan-ExtensionLayer build # 750 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 53591.

CI Vulkan-ExtensionLayer build # 751 running.

CI Vulkan-ExtensionLayer build # 751 passed.

The internal CI run that passed over the weekend has the decompression tests disabled on the failing devices. I'll start another run with those tests reenabled.

Device::init_queues() may be unrelated to my change, because this change only modified the shader. Can you check if that problem existed before?

Most likely there's a bug in your test and yeah it probably existed before. Probably the test is requesting an extension or feature that isn't present on this device. I can send you the vulkaninfo output for it in a minute but I don't have time for remote control debugging experiments. If you find something suspicious, update the PR and I'll start another CI run.

vulkaninfo_Samsung_Galaxy_S32_Ultra.txt

vulkaninfo_Google_Pixel_7.txt

Looks like https://github.com/KhronosGroup/Vulkan-ExtensionLayer/pull/328 also fails on Android?

closing this in favor of https://github.com/KhronosGroup/Vulkan-ExtensionLayer/pull/328 .  In the future please use git push -f your-pr-branch to update the code in a PR rather than making a new one.


Check for uncommitted files before running sync

Closes #788
Code Changes

[x] add a util function that throws an exception if there are uncommitted files in the repo
[x] call the util function from the sync CLI command
[ ] Add tests

Steps to Confirm

[ ] sync throws an error with unstaged files in the manifest dir
[ ] sync works with no unstaged files in the manifest dir

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Write some things here about the changes and any potential caveats

final product will look something like this:
>fides: fidesctl sync
Loading resource manifests from: .fides/
Taxonomy successfully created.
There are unstaged changes in your manifest directory: '.fides/'
Aborting sync!

running into a bug no where the constant changing of the analytics id is being troublesome and making this change not work as intended...going to need to fix that too

lgtm


Potential Offer Handling

Opening up to express some ideas around how we'd handle #81 and #84, not intended for merge.
We've already got a lot of the machinery in place to handle the onion messaging side of things in LDK, so I'd like to try to take advantage of that - if we provide an OfferMessageHandler that correctly queues messages, the existing onion messenger will drive sending / notifying of onion messages. That leaves our code responsible for:

Creating an invoice request and adding it to the handler's existing queue
Handling the invoice that comes back (validating it vs the original request / amount)
Tracking state to make sure we don't process anything twice

Hope that this adds some context to the review!

@carlaKC Ahh yup this makes sense to me, I think. I'll refactor with this (or something like it) as the base.

I think I understand the direction here but where I'm generally confused is... how do you imagine us calling this in the CLI? Would the idea be to run a second instance of the OfferHandler struct? Then expose the methods we need on the OfferHandler struct so that we can use those in the CLI?

Closing this out as it was just an example- @orbitalturtle feel free to take this over / squash your changes as needed.


The user ID of the message from Teams is encrypted

I built a java bot and released it to potal. azure.com
The user ID is encrypted when i send a message to bot from teams, but local is nomal.
Just like this.
teams_user_id:29:1Fj7dCmV6KsUklmp4GQyvfz1_Lqv0KKfoY8xvFnqk9TuFnBwviMzUlz93-C0KyrRR75P8ZovCEaGKYgF63w

@Jeff-Zhang-Github - To be clear, this is a feature of Teams, not Bot Framework, and is by design. As you can see, in this doc, the user Id represents the Azure AD object id which can be used maliciously hence it being encrypted.
I don't know if there is a way to decrypt it. For that you will need to reach out to someone on the Teams team. This forum is active and may serve you. Be sure to append the office-teams-windows-itpro tag when asking a question.

@stevkan Thank you for your help and advice on this matter,I got it from aadObjectId.


Cannot find module './lib/React'

When I install the package on my project and try to run it, without importing it into anywhere I get this error:
Error: Cannot find module './lib/React'
If i remove the package then it all works fine but adding it again brings back the error

Sorry for the 2.5 yr delay :weary:   Closing as probably no longer relevant, however if it is, please let me know and I'll give it my full attention.  Apologies.


Proposal: daemon defaults for runtime constraints

Today, the resource constraints values is defined on each docker run command. Thinking about cloud-scale Docker deployment, could be nice to define this defaults on Docker daemon. I think that /etc/default/docker / DOCKER_OPTS could be a nice place to configure it. This way, each container would start with right resources.

seems orchestration layer is more meaning instead of add other options in docker daemon

Seems like we'll have daemon-wide seccomp profiles soon. Implementing daemon wide constraints is indeed a task of orchestration.


11 member federation failed to setup

10 was successful. We should investigate if we enforce total number of peers == 3f + 1.

if we enforce total number of peers == 3f + 1.

At least not consistently if so. There's at least one 5/6 federation out there. I'd guess the error came from something else.


Compatibility with Objective-C

Hello Jerome,
Thanks for making such a great framework and sharing it with the world.
I am currently working on an Objective-C project and I was wondering if you could make your framework compatible with Objective-C.
Thank you.

Thanks for your feedback, I will address this as soon as possible.

Hello,
I've tested LocationPicker in Objective-C, there's only one method addButtons that is not compatible. I will solve it and release version 1.0.3 several hours later when I'm available. I will inform you in this issue when I'm done. You can integrate it to your project now if this doesn't bother you.
To use LocationPicker in Objective-C
@import LocationPickerViewController;    //If you installed via Cocoapods
@import LocationPicker;    //If you installed in any other way

@iosmahdi 1.0.3 has been released. LocationPicker now is compatible with Objective-C. If you encounter any other issues, please don't hesitate to create an issue or a pull request. Thank you. 😊


"Wrong message content type" error when sending videos with SendMessageAlbum

I'm tying to send some videos with SendMessageAlbum (java) but I get the Wrong message content type error in return.
The method works fine with photo messages but I can't send videos with it while it is possible with original android client.
Is this a bug or I am doing something wrong?

You are doing something wrong. Could you collect TDLib log with verbosity level 4 and send it to @tdlib_bot in Telegram? Then it will be clearly seen, where is the problem.

Saw the log and found out my problem, thanks for your answer!


create class for inline positioned lists

replace org create alignment hack
fix #6464
Signed-off-by: Michael Gnehr<EMAIL_ADDRESS>(I hope this is better)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@46d3d10). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #7393   +/-   ##
=========================================
  Coverage          ?   41.19%
=========================================
  Files             ?      469
  Lines             ?    63536
  Branches          ?        0
=========================================
  Hits              ?    26176
  Misses            ?    33939
  Partials          ?     3421

Continue to review full report at Codecov.

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

Can you please provide screenshot

it doesn't really look different, it just wrap the check boxes in inline positioned div.
-> Added screenshot in merge description.


Add a role for RoboCup 2021 with appropriate logging

Looks like DataLogging is failing to compile due to the issue fixed in #473.

I think @ysims suggested earlier that the role be named something like webotsrobocup instead of something specific to this year, particularly if we end up using webots again next year.


Example showing v6 code

Example of formState is showing to import the error from useForm() hook. But according to latest API change it should be imported from formState.
v6 :
const {
    register,
    handleSubmit,
    errors,
    formState: { isDirty, isSubmitting, touched, submitCount },
  } = useForm<FormInputs>();

v7:
const {
    register,
    handleSubmit,
    formState: { isDirty, isSubmitting, touched, submitCount , errors },
  } = useForm<FormInputs>();

Link : https://react-hook-form.com/api/useform/formstate

ty


sof-firmware: 1.5.1 -> 1.6

Motivation for this change
Bump to latest stable, hopefully more stable than version before. Testing it on X1 carbon gen7.
cc @zuigon
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Unfortunately on kernels > 5.9.3 microphone is broken either on 1.5.1 or 1.6.

Bump sof-frimware to official 1.6 release. Microphone now works (kernel 5.9.9) ! I had to play a little bit with pavucontrol.

After one week of extensive testing (jitsi calls) I didn't found any issues.
Pinging random people.. @SuperSandro2000 can you please merge this one.

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

1 package built:

sof-firmware


WIP: Enforce lightning invoice expiry

Fixes https://github.com/fedimint/fedimint/issues/749
In previous PR (https://github.com/fedimint/fedimint/pull/727), we exposed the ability to set a custom invoice expiry but did not enforce it. This PR enforces the expiry by adding a new LightningConsensusItem for clock time
Todo:

Need to fix integration test (https://github.com/fedimint/fedimint/commit/4eb650a8b47abc1b404bb7c97bf8b49ec786293b) for checking that the expiry is enforced
Added epochs to integration tests that were timing out (https://github.com/fedimint/fedimint/commit/614ca5163142252f1d8ee0a9a7da3c5b68515966), need to confirm that they still work as intended

I suggest modifying await_consensus_proposal in the LN module to continue only when the time will generate a change in invoice expiry (you can see how this is done in the wallet module).

Added logic in consensus_proposal to only propose new clock time if an invoice would expire to avoid continually producing epochs. I put the logic in consensus_proposal instead of await_consensus_proposal to based on the logic changes I saw in the wallet .

Rather than have the client fetch the consensus time and validate the invoice client-side, we could have the federation validate and generate a consensus time on submission of the tx.

I moved the validation logic from client-side to federation in https://github.com/fedimint/fedimint/pull/1074/commits/1ea20954bb029c09c36cf2febdcf621f064a5bac

Are you planning to continue working on this @wilfredallyn

Yes, I am going to revise this and finish it

@justinmoon, I'm closing this after discussing with @joschisan. The increased complexity with a new consensus item (clock time) may not be worth it for this use case (enforcing invoice expiry). If this issue gets picked up in the future, these things need to be fixed:

enforce expiry on incoming payment (was implemented on outgoing)
update code to work with aleph bft


[minor] make the "link" logo more prominent

One small stylistic remark, I find the "link" logo that link to the personal websites too easy to ignore. It would be nice if the personal
website links were a bit more prominent.

I still find the original "link" logo clearer than the two "arrow" variants.


MacOS Support

MacOS support would be great. If this could be added I would seriously consider switching to this crate as my daily driver for GUI automation.

Hello @FrankenApps. I dont have a MacOs machine, but will see if I can run a VM instance using one and develop support. It is one of main tasks on TO DO list surely

Got the machine , starting working on it

macOS support included from version 0.2.0. Issue closed


How to build it ?

Describe the bug
No public doc on how to build it or if one exists, it is not linked to main README.
Steps to reproduce the bug

Goto winappsdk rep page on github
scroll down to section on instructions to build it

no section exists or no links embedded
Expected behavior
winappsdk builds !!
Screenshots
No response
NuGet package version
No response
Packaging type
No response
Windows version
No response
IDE
No response
Additional context
No response

By "it" are you asking how to build WinAppSDK?

yes

There is now the BuildAll script in the root.


site broken on iPhone 5 in Los Angeles

A User in Los Angeles claims site was not working, using mobile safari in an iPhone 5.
plan to test using an iPhone 5C I have lying around

I'm going to close this.. we need to focus on worry about the mobile analytics in ionic instead of iPhone 5 issues with the desktop site.
if we get a visit from a mobile site perhaps we could do a redirect to the mobile app link.. but for now it's not an issue I want to tackle.. we'd need more data to proceed further.


VSCode hangs on developer tools

Issue Type: Performance Issue
VSCode hangs whenever I toggle developer tools and execute some javascript on console. I have to manually kill all the running  code processes to get back to normal state and close code.
VS Code version: Code 1.36.0 (0f3794b38477eea13fb47fbe15a42798e6129338, 2019-07-03T13:20:56.241Z)
OS version: Linux x64 4.13.1-041301-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3 CPU       M 380  @ 2.53GHz (4 x 2526)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
0, 1, 0

Memory (System)
5.63GB (3.76GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    1	   121	  3084	code main
    0	    35	  3088	   zygote
    0	   213	  3132	     window (Untitled-1 - Visual Studio Code)
    0	   104	  3383	       extensionHost
    0	    86	  3407	     shared-process
    0	     0	  3633	       /bin/sh -c /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    0	     0	  3634	         /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    1	    92	  3607	     window (Issue Reporter)
    0	   110	  3116	   gpu-process

Workspace Info
;

Extensions (6)

Extension
Author (truncated)
Version

better-comments
aar
2.0.5

gitlens
eam
9.8.2

rest-client
hum
0.21.3

cpptools
ms-
0.24.0

material-icon-theme
PKi
3.8.0

java
red
0.46.0

Does it also happen if you run VSCode with all extensions disabled?
If so, please post a simple example of what you execute on the Developer Tools Console to trigger the hang.

@bpasero Electron update?

VSCode hangs whenever I toggle developer tools

This is odd, but VSCode is not officially supporting the developer tools of chrome. I suggest you try to reproduce this in Electron and then report this to Electron https://github.com/atom/electron


Add config

Can you add configs to present meta and stack.
Like that:
logger.add(winston.transports.Console, toYAML.config({
    metaLevel: 'warning', // or printMeta: true
    stackLevel: 'error' // or printStack: true
}));

I think it will be more usefull.

@DatoBula, thank you, good idea. metaLevel/stackLevel looks really useful. You can rewrite your PR to this type options and i merge it. With tests please.
Or i can do it by myself, but next week.

OK, I'll try to change options and create new PR.


feat(@angular/cli): add a bundle dependencies flag

This flag allows people who know what theyre doing to bundle the
server build together with all dependencies. It should only be
used when the whole rendering process is part of the main.ts
or one of its dependencies.
Fixes #7903

Brilliant - this is much quicker than I expected :) Great work.
@hansl Will you also be tweaking the default config so that it doesn't try to uglify server-side builds? Otherwise it will lead to an error when trying to minify a typical self-contained build. Or will this be a different config option?

@SteveSandersonMS if you use ng build --app 1 --aot --environment=prod --output-hashing=false then it'll do mostly the same build for your sever app but without minification

@Toxicable Thanks for the suggestion. Unfortunately in 1.5.0 that doesn't seem to be sufficient to avoid build warnings. Have filed a new issue to track that: #8035


Refactor search hits into more generic component

We currently have a lot of duplicate code, we should refactor this into a more generic and reusable component.

Might not be worth it, since they are similar but not necessarily the same

Not sure if this is still needed


⚠ Processors has degraded performance

In 017421c, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 98 ms

Resolved: Processors performance has improved in 4cbe38b.


selection:changed event handler returns cells already removed in the graph

Describe the bug
When a node and an edge are deleted together with Graph.removeCells, the selected property passed by Graph's selection:changed event contains cells that don't exist in the graph.
Your Example Website or App
See the minimal reproducible example below.
Steps to Reproduce the Bug or Issue

Paste the following code into an HTML file and run.

Open the developer console. In the graph, use command or Ctrl to first select a node and then an edge. Then, press Backspace or Delete to delete them.
In the console, one array of cells returned by selection:changed should contain a null returned by getCellById, indicating that the cell is no longer in the graph.

Expected behavior
selection:changed should always return cells that exist in the graph.
Screenshots or Videos
The following screenshot shows the developer console after a bug is reproduced:

Platform

OS: macOS
Browser: Chrome 103.0.5060.134

Additional context
The bug is likely caused by a corner case in Model.removeConnectedEdges. Because a node's connected edges are removed after the node itself but before unselecting the node, the selection still contains the already-removed node. When the selection:changed event is fired by the removal of connected edges, the removed node is contained in the selected property.

更新到 1.33.0 版本。

已更新


chore(package): update mocha to version 2.4.4

http://greenkeeper.io/

Current coverage is 83.62%

Merging #17 into master will not affect coverage as of 9a30318

@@            master     #17   diff @@
======================================
  Files            4       4
  Stmts          171     171
  Branches        41      41
  Methods          0       0
======================================
  Hit            143     143
  Partial          0       0
  Missed          28      28

Review entire Coverage Diff as of 9a30318

Powered by Codecov. Updated on successful CI builds.


Lighting Issues

I am having similar issues to Issue #1, unless I change the shader code to
shader_type spatial;  render_mode skip_vertex_transform, unshaded;
Obviously, I'd much prefer lighting. Any ideas as to what is causing this?

https://user-images.githubusercontent.com/30809120/105030409-5e6fce80-5a19-11eb-84d8-1784c4197f64.mp4

https://user-images.githubusercontent.com/30809120/105030409-5e6fce80-5a19-11eb-84d8-1784c4197f64.mp4

I am having similar issues to Issue #1, unless I change the shader code to
shader_type spatial; render_mode skip_vertex_transform, unshaded;
Obviously, I'd much prefer lighting. Any ideas as to what is causing this?


dependency range bumps are ignored

Packages within a monorepo that have a dependency (or devDep in npm) will be bumped when the related package is bumped. Unfortunately, this broken if the dep/devDep is specified as a range. The following snippet out of @covector/apply shows how the version is bumped.
let version = semver.inc(
  pkg.pkg.dependencies[dep],
  bumpType
);
if (version) pkg.pkg.dependencies[dep] = version;

However, semver.inc() can only bump "clean" version numbers and the version variable here returns null.
Note: until this bug is resolved, we recommend pinning all internal / monorepo dependencies if you expect covector to apply the bump for you.
Also note: yarn ignores the version number and happily symlinks these in a workspace. npm does not and will fall back to pulling from the registry. This might mean that, some publishes work fine if the package still falls within the dependency range that you specified. However, since it will, currently, never be incremented, a package may eventually be bumped outside of that range and begin failing. (This is even more nuanced when your versions are < 1 as the commonly used ^ works differently for version numbers < 1.

Part of me wonders: do you ever want anything than a pin for an intra-repo dependency? Since the updating of dependencies is all built and tested together, what is the harm if @same/a always points to a specific version of @same/b?
It almost feels like even if we fixed this bug, it would be even better to have a --strict mode raised an error if you had anything other than a point release for internal dependencies.

I have faced this in rust projects recently too, and I never want anything other than a strict pin within the confines of the monorepo.


Small improvements for token-assert module

/cc @mdevils

Coverage remained the same when pulling 6da56c1c5a4b927929511433993e4a5ad731ba92 on markelog:token-assert into 241accf81729228a97b239ce7726ab44371a7ab9 on jscs-dev:master.

Hi, Oleg.
Checked out this PR. For me it became less clear.
@mikesherov, can you please check this out?

Checked out this PR. For me it became less clear.

Could you elaborate?

Will merge this for now, since i need this for further work on other things, will revert this, if these changes would not be acceptable

What motivated the property name change? I agree with Murat that nextToken was a better name. I'd think what is currently token should be prevToken and nextToken/subjectToken should just be token or currentToken.

This nextToken/subjectToken could be very well previous or the next token, so i would choose a name which would be agnostic to the direction, since "direction" doesn't matter, because it doesn't reflected in checks of those methods.  Same with token property.
Bikeshedding process is as fun as i remember it.

Errr... why did you merge? For me nextToken is way clearer than subjectToken.

@mdevils

Will merge this for now, since i need this for further work on other things, will revert this, if these changes would not be acceptable

I think Oleg just wanted some solid ground to work off of.
I do think though that "next" and "previous" imply position in text. @markelog's desire to rename them is good. I am not sure "subjectToken" is a good name either.
@markelog, you can't start painting a red shed blue and then complain how not fun it is to paint!


Cancel method and event

Allow the user to properly cancel the preload at any time by calling preload.cancel(), aborting the XMLHttpRequest. Add support for a cancel event too, like preload.canceled = () => { ... }.

This has been added already on 1.4.0, preload.cancel() method and preload.oncancel = items => {} event.

Thanks @andreupifarre!


Add an example email Card prior to sending

This Card will display what an example email will look like. Probably just pulls info from the first row.

I made an example of this on jared-example-email but it looks bad. I think if we are going to show an example email, we should do it in a dialog. This won't be done by the milestone.

I think this may be best done by sending a test email to users.

Missed the second milestone.

Closed by #52.


output unprocessed ids to csv

id list of rows which not included at analytics

SELECT
  b1.id,
  b1.time_in,
  b1.payment_in,
  b1.currency_in,
  b1.payment_out,
  b1.currency_out,
  b1.category_code,
  b1.category_name,
  b1.payment_desc
FROM balance b1
  LEFT JOIN (
              SELECT b1.id
              FROM balance b1
                INNER JOIN balance b2 ON b2.time_in = b1.time_in
              WHERE
                b1.currency_out = 'RUB' AND b2.currency_out = 'USD' AND
                b1.payment_out < 0 AND b2.payment_out > 0 AND b1.category_name = 'Переводы/иб'
              UNION
              SELECT id
              FROM balance
              WHERE
                currency_out = 'RUB' AND currency_in = 'USD' AND
                payment_out < 0 AND payment_in < 0 AND category_name != 'Переводы/иб'
            ) b2 ON b2.id = b1.id
WHERE b1.account_id = :account_id AND b2.id IS NULL;


Added Babeldates plugin

This PR adds a plugin which formats dates using Babel (see getnikola/nikola#2606). I know the bug reported there is not on your side, but just in case someone finds it useful, I share the plugin mentioned.

Woops, tests fail.

Don't worry, I will fix them.

Because I have not fixed them :-)

I guessed so ;) Just wanted to ping this in case you forgot ;)

42d6e2b

Thanks. I'm a disaster with wrapping.

NP, besides the real problem is that the thing we do to merge requirements from all plugins for testing is very stupid.