What exactly makes content in right column shift one in the left

I faced the really weird problem and completely stuck with finding a solution.
What I need to know is what css (or maybe something else?) commands make content in the middle column shift to the end of content in the right one. Sometimes it is really handy, because code in the right column maybe longer than explanation so we need to shift content in the middle column.
But sometimes we don't need it. This is why:
I'm using kramdown and MathJax for displaying math. When math is inlined everything is ok. So this content doesn't cause any problems:
3. Пусть характеристический многочлен матрицы $$A_(\epsilon)$$ имеет вид
$$\phi(\lambda)=(-1)^3(\lambda-\lambda_1)^3.$$

Here what it looks like:
![ok]
(https://i.gyazo.com/c5e4e8ed9b36b0ae75ac69d7c755e050.png)
But this content:
3. Пусть характеристический многочлен матрицы $$A_(\epsilon)$$ имеет вид

$$\phi(\lambda)=(-1)^3(\lambda-\lambda_1)^3.$$

Causes this:
![not ok]
(https://i.gyazo.com/95704b52d1a1a58c2d5a676d2f289ad1.png)
I also selected this lines so you understand why I need it to be a seperate paragraph:
![why]
(https://i.gyazo.com/91ae4cf217136d242d6cc6f5226ddeed.png)
Ok so, how do I make this content fit middle column and to be ignored by shifting? I really struggling with this.

I've also find a hot-fix: wrap all such math in  block, but can you tell me what causes this?

Slate doesn't know the mathjax element, so it doesn't know to put it in the left column. If you add a selector for mathjax to this selector it will probably fix the problem.


[Bug]: CANNOT SHARE EMAIL PROPERLY (REGEX ERROR)

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I Want to share my deployed app in the by the appsmith share button but when i enter this email<EMAIL_ADDRESS>which is a valid email address , it didn't provide me the invite button

Steps To Reproduce
1-create and deploy your app
2-Share the app with email address
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud

https://github.com/appsmithorg/appsmith/assets/130959931/bb788317-3b15-4da8-bc74-89d4b985f10d
@mrayushmehrotra - I used the same credentials you provided and could share the app. I have attached the video for reference. Should I be doing something differently?

I don't know, now it's working fine, but in real I have got that error (that's not a fake image I have posted, the cursor style was block at that invite button) so I will close the issue.

@mrayushmehrotra - I had no intention of suggesting the bug was fabricated. I was curious if there was a different sequence of actions that triggered it. I appreciate you closing the bug, and if you encounter a similar situation in the future, please don't hesitate to reach out. Thanks


Example of initialsing base class in a derived class

If I have a class hierarchy using attrs, how do I initialise the parent class ?
Normally the constructor would call super().__init__() right?  How is this done with attrs classes?
Could you provide and example in the examples documentation please :)

Hmmm, I found the "How Does It Work?" documentation, which states that attrs never calls super() and that it walks the class hierarchy to collect the attributes of the super classes.
Still a good example would be good :)

Yeah, if all your bases are attrs classes without __attrs_post_init__, you don't have to do anything. For everything else, there's __attrs_post_init__.

How do you initialise a base class attribute with a different value?
e.g. base class has member x with default of 0 (via attr.ib), and sub-class wants to intialise x to a different value (e.g. 123).
I tried using attr.ib with default, but it seem to add an additional member (i.e. there seemed to be 2 members named x).
I did try def __attrs_post_init__(): in sub-class and set the attribute directly, and that didn't seem to create an extra member.  Is that the right thing to do?  Does that just create a new name?  Maybe I should use super().x = 123?  I presume that works - I've not used super() much.

Generally speaking, we believe that subclassing for object composition is a bad thing, therefore attrs doesn’t have explicit support for many things that it entails.
That said, getting two members x in your case looks like a bug to me and should be fixed.  Would you mind posting a bug with an example?  Thank you!

A while ago I was asking this same question. I was not looking for a way to overwrite an attribute, but just call super().__init__() - the attrs class I was creating was a subclass of Thread and that had to be called.
As I continued learning what I was doing, I noticed a thread was not needed anymore (and therefore the inheritance), but I'm still curious: how to initialize the parent class with attrs? It seems there are two cases:

When subclassing a non-attrs class, call super().__init__() in the subclass' __attrs_post_init__()
When subclassing an attrs class, call super().__attrs_post_init__() in the subclass' __attrs_post_init__()

Is that correct?
Thanks!

Looks correct, yes.  Calling super().__init__() in the second case probably would work too but it would unnecessarily re-initialize the attributes it knows about.

The reason I called __attrs_post_init__() in the second case was not to prevent redundancy (although that is also a good reason), but because __init__() results in a recursion until a RuntimeError is raised.
I did not mention that at first because I thought that was the wrong way to do it. I thought that super().__init__() would call the subclass' __attrs_post_init__() in the end, making it call itself.
Now that you mentioned and after thinking a second time, I think that behavior is actually wrong. super().__init__() shouldn't call the subclass' __attrs_post_init__() and in fact never calls the superclass' __attrs_post_init__(). Reproducing:
import attr

a_post_init_called = False

@attr.s
class A(object):
    a = attr.ib()

    def __attrs_post_init__(self):
        global a_post_init_called
        a_post_init_called = True

b_post_init_called = False

@attr.s
class B(A):
    b = attr.ib()

    def __attrs_post_init__(self):
        global b_post_init_called
        b_post_init_called = True

        super().__init__('')

try:
    b = B('a', 'b')
except Exception as e:
    print(e)

print('post inits called: {}, {}'.format(a_post_init_called,
                                         b_post_init_called))

output:
maximum recursion depth exceeded while calling a Python object
post inits called: False, True

(I passed '' just to show that passing self.a would not cause the problem.)
Just to clarify: I do not think that using attrs like this is correct and I am not asking you to provide anything to support code like this. I am just reporting something that seems wrong.

To summarize: If you subclass from another attrs class, the subclass collects all attributes from the parent class hierarchy, and from itself, and generates a brand new __init__ which accepts all merged attributes from all classes (you can verify that with inspect.signature(thesubclass.__init__)).
But if your superclass did some special __attrs_post_init__ work, then be aware that the rules are as follows:

If your subclass DOES NOT HAVE its own __attrs_post_init__, then the execution is as follows: The subclass runs its own __init__ (which does the attrs work of validating and initializing variables), and then it attempts to call __attrs_post_init__ and calls the PARENT class one!
If your subclass DOES have its own __attrs_post_init__ (meaning there was one in both your superclass and subclass), then the execution is as follows: The subclass runs its own __init__, which then runs its own __attrs_post_init__. It will NOT automatically call the superclass post-init function. So it is YOUR JOB to call super().__attrs_post_init__()

Demonstration of all cases:
@attr.s
class A:
    a = attr.ib()
    def __attrs_post_init__(self):
            print('a hello')

@attr.s
class B(A): # Because this class has a post-init, only THIS post-init will run.
    b = attr.ib()
    def __attrs_post_init__(self):
            super().__attrs_post_init__() # Without manually calling, A's post-init wouldn't run.
            print('b hello')

@attr.s
class C(A): # Because this class lacks a post-init, the one from A will run instead.
    c = attr.ib()

Result in python console:
>>> A(1)
a hello
A(a=1)

>>> B(1,2)
a hello
b hello
B(a=1, b=2)

>>> C(1,2)
a hello
C(a=1, c=2)

Of course, in hindsight, perhaps attr should have been coded so that its __init__ function checks for super().__attrs_post_init__ and if so runs it first, and THEN runs the current class's own __attrs_post_init__. So that both of them are automatically run in the correct order.
But it's way too late to change that design now, since most people have code now that manually calls the superclass post-init function as shown in example B above.
And there's a good argument for the way things are designed, because it means that YOU decide if you want the superclass __attrs_post_init__ to run or not. So I don't really dislike this design. It's fine.
I just wanted to document everything. Thanks @hynek @felipedau @BrendanSimon for all your info in this thread.


TWL modifications

The TWL modifications are mentioned in one place, and there is really barely any information on them. After a bit of googling and asking around I managed to learn they're related to DS mode.
While the guide mentions you have very small chance of having such modifications, there could be an instance where you installed something years ago and forgot, or bought used console.
There isn't words of encouragement that if you do have TWL mods and proceed, nothing bad will happen, either.
I suggest elaborating on that a bit, mentioning common TWL mods, and clearing up on what DSmode stuff is popular but isn't TWL mod, and if you do happen to have one, how to get rid of them.
Personally, I've installed flashcarttimewarp.cia before, and maybe bluecardfix.cia too, I can't remember. And I've had quite a pickle as there is barely anything that I could find via googling, nobody talking about similar concerns or problems, and nobody from eager to help people really knew if this can mess things up.

It would be nice to link to a guide that removes the TWL modifications

On Jun 17, 2016, at 12:47 PM, Azebu<EMAIL_ADDRESS>wrote:
The TWL modifications are mentioned in one place, and there is really barely any information on them. After a bit of googling and asking around I managed to learn they're related to DS mode.
While the guide mentions you have very small chance of having such modifications, there could be an instance where you installed something years ago and forgot, or bought used console.
There isn't words of encouragement that if you do have TWL mods and proceed, nothing bad will happen, either.
I suggest elaborating on that a bit, mentioning common TWL mods, and clearing up on what DSmode stuff is popular but isn't TWL mod, and if you do happen to have one, how to get rid of them.
Personally, I've installed flashcarttimewarp.cia before, and maybe bluecardfix.cia too, I can't remember. And I've had quite a pickle as there is barely anything that I could find via googling, nobody talking about similar concerns or problems, and nobody from eager to help people really knew if this can mess things up.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I found this https://gbatemp.net/threads/arm9lh-preperations-remove-any-twl-modifications.417998/ not realy a guide but if you go down abit it shows you how to remove them

Added :D


[WIP] Refactor Image Gallery

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?
[ ] Yes
[x] No

@damienwebdev it's in a decent state to review now


[REQ] Discord Rich Presence and OneSkyVed's Trolleybus Place

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
the game OneSkyVed's Trolleybus Place should have more activity shown, like what route you're driving on, what type of Trolleybus are you driving, the most cool thing would be speed and if your trolleys are derailed or not.

There is no reason to add all of this this for just one game

Ask the game developer for that.
https://github.com/bloxstraplabs/bloxstrap/wiki/Integrating-Bloxstrap-functionality-into-your-game


Add missing step in Rails getting started doc

I just tried Flowbite this morning on a newly created Rails <IP_ADDRESS> project, following the instructions here. In the end the Javascript part wasn't included, hence the example tooltip wasn't working.
This PR fixes the doc so Flowbite Javascript code is included in your Rails application bundle. Note that it's just a cosmetic change and it does NOT solve the compatibility issue with Rails' Turbo, e.g. #88

Thank you. This fixed my issue!

Thanks @jbbarth for contributing! This will be merged soon.

Merged. Thanks @jbbarth for contributing!


[Outlook] Remove implication item id is getCallbackTokenAsync param

Docs Build status updates of commit edc19f7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit 881c11e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit ca6e25f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.1/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.2/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.3/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.4/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit f97f9d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.1/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.2/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.3/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.4/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.md
:white_check_mark:Succeeded

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

@kbrandl Thanks for reviewing and pointing out the table issue! Should all be sorted out now. Thx.


Java API Reference docs for Event Hubs are not linked to the right place

The TOC on the left for the Azure Event Hubs documentation has the Reference node which expands as below

The link for azure-messaging-eventhubs (latest) currently points to https://azuresdkdocs.blob.core.windows.net/$web/java/azure-messaging-eventhubs/5.0.0-beta.6/index.html which is not really the latest. This should instead point to https://azuresdkdocs.blob.core.windows.net/$web/java/azure-messaging-eventhubs/5.3.1/index.html
Found as part of https://github.com/azure/azure-sdk-for-java/issues/17297
cc @conniey

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

ID: 0a6e2fa9-99f3-7a01-5629-42ff4a762ea8
Version Independent ID: d3992cd6-6809-d757-2725-0be592ee6521
Content: What is Azure Event Hubs? - a Big Data ingestion service - Azure Event Hubs
Content Source: articles/event-hubs/event-hubs-about.md
Service: event-hubs
GitHub Login: @spelluru
Microsoft Alias: spelluru

@ramya-rao-a  Thanks for your feedback! We will investigate and update as appropriate.

@ramya-rao-a - thank you. Fixed the links. The latest one also points to DOCS reference content now.
https://docs.microsoft.com/en-us/java/api/overview/azure/messaging-eventhubs-readme?view=azure-java-stable

@ramya-rao-a  We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Fix compiler warnings

Show compiler warnings when building unit tests.
Fix unused variable warning and miscellaneous conversion warnings.
Clarify logic in LBVHTraverser::setup()

Resolves #26
@joaander

This is merged and tagged. You could bump neighbor to v0.3.2 in HOOMD on your branch silencing warnings.

Thanks, will do.


Receiving a message "Invalid body: Version mismatch." when using /token/generate endpoint while encrypting with the provided scripts

When testing out the /token/generate endpoint, I'm receiving the following message:
{"message":"Invalid body: Version mismatch.","status":"client_error"}
I'm following the steps from the /token/generate documentation, using the Python scripts provide in the Encyrption/Decryption docs:
echo '{"email_hash": "tMmiiTI7IaAcPpQPFQ65uMVCWH8av9jw4cwf/F5HVRQ="}' \
  | encrypt_request.py {redacted} \
  | curl -X POST 'https://prod.uidapi.com/v2/token/generate' -H 'Authorization: Bearer YourToken{redacted}' -d @- \
  | decrypt_response.py {redacted}

I am replacing the key and secret with my own credentials.
encrypt_request.py script:
import base64
import os
import sys
import time
from datetime import datetime

from Crypto.Cipher import AES

secret = base64.b64decode(sys.argv[1])
payload = "".join(sys.stdin.readlines())

iv = os.urandom(12)
cipher = AES.new(secret, AES.MODE_GCM, nonce=iv)

millisec = int(time.time() * 1000)
nonce = os.urandom(8)

print(f'Request timestamp: {datetime.fromtimestamp(millisec/1000)}', file=sys.stderr)
print(f'Request nonce: {int.from_bytes(nonce, "big")}', file=sys.stderr)
print(file=sys.stderr)

body = bytearray(millisec.to_bytes(8, 'big'))
body += bytearray(nonce)
body += bytearray(bytes(payload, 'utf-8'))

ciphertext, tag = cipher.encrypt_and_digest(body)

envelope = bytearray(b'\x01')
envelope += bytearray(iv)
envelope += bytearray(ciphertext)
envelope += bytearray(tag)

print(base64.b64encode(bytes(envelope)).decode() + "\n")

I tried updating the encrypt_python script to have a different value for the version byte in the request envelope, but no change in behavior occurred. Additionally, removing that byte from the request envelope returned a different error: "Invalid Body Length", confirming that the endpoint was indeed receiving the updated envelope as I made changes.
I'm running Python v3.11.2.

@atarassov-ttd I'm now picking up this work from @theandypeterson. Do you think this issue is related to https://github.com/IABTechLab/uid2docs/issues/55? Is there any chance we are simply hitting the wrong subdomain here? Or would you say that expired credentials are the more likely culprit?

Hi @jcerone let me try to reproduce the issue. The python snippet in documentation is tested in python 3.8

I cannot reproduce the issue with python v3.11.2 on debian bullseye. The version byte mismatch means your first byte in the envelop is not \x01. Can you give more info on what OS and shell you are using to run the command?

Hi @scong-ttd I'm running the following:
macOS Ventura v13.3
zsh shell
Python 3.11.2

Here's the output I'm seeing given an arbitrary AES secret I generated as an example:
echo '{"email_hash": "tMmiiTI7IaAcPpQPFQ65uMVCWH8av9jw4cwf/F5HVRQ="}' \
  | python3 encrypt_request.py 3c7d1d1a8e9a54f9f3a3b3c1b8ab849c
Request timestamp: 2023-04-05 10:11:42.056000
Request nonce:<PHONE_NUMBER>92742734

bytearray(b'\x01\x94\xbc\xc2\x03\xbb^\xb8\x98\x8c\xe5\x84P|q\x1c\xb2xl\xab\xee\xf3\x96&\x01\\\xdd\xdd5\x84\xe4\xa2U\xf5\x00\xa2\x97\x1d\xa5\x1d\x8a6\x9c\x99\x17g`iO4\x91\x01+\x00\xceS\x8f\xcd\x1aI\xe9\xb3\xc6\x97\xebK\xcar?\xa3\x0bN\xf6\xb9u\x11\x92[\x0byna\x85\xc4X\x82\xb5\xf75)\x16\x03-\x8eBo$\x90\xfb<o\xbf\x13V\x9b\x92\xe8%')
AZS8wgO7XriYjOWEUHxxHLJ4bKvu85YmAVzd3TWE5KJV9QCilx2lHYo2nJkXZ2BpTzSRASsAzlOPzRpJ6bPGl+tLynI/owtO9rl1EZJbC3luYYXEWIK19zUpFgMtjkJvJJD7PG+/E1abkugl

From the above, it appears the first byte is indeed \x01.

Hi @scong-ttd that's just a 32 letter code ChatGPT generated for me 😜. For clarity, I used the secret we were given as-is in my actual request. I just cleared that up by just showing it as redacted.
In the following code, I used the credentials provided to us by Trade Desk and received the invalid body error. Should I request new keys in this case, since the one's I've been given appear to be invalid?
echo '{"email"<EMAIL_ADDRESS>\
  | python3 encrypt_request.py {redacted} \
  | curl -X POST 'https://prod.uidapi.com/v2/token/generate' -H 'Authorization: Bearer {redacted}' -d @- \
  | python3 decrypt_response.py {redacted}

-----------------------------------------------------------------------------------

Request timestamp: 2023-04-05 10:14:38.034000
Request nonce: {redacted}

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   400  100    77  100   323     99    417 --:--:-- --:--:-- --:--:--   518

Error response:
{
    "message": "Invalid body: Check encryption key (ClientSecret)",
    "status": "client_error"
}

Hi @jcerone, we've sent the keys to Gareth. The one that should be used in this case has a name ending in "-Generator"
The error you're receiving indicates the API key is correct (Authorization: Bearer {redacted}), but the secret key or the encryption code is incorrect.

Hi @jon8787 I'm not sure I'm familiar with Gareth, but it's good to know that one half of our setup is correct and that the problem is with our secret key rather than our implementation. I requested updated credentials from TTD yesterday and will let you know how they work!


Initial refactoring to build from Foundation SASS

Initial refactoring to use Foundation SASS files, start modularizing NetBeans SASS and build to single CSS file.  This also gives us access to all the Foundation includes, etc.
Existing css/netbeans.scss has been moved to scss/common/_netbeans.scss.  Footer extracted to scss/components/_footer.scss as example of modularization.  Existing color and font includes added to _scss/base as well as the settings file from the Foundation template - these need merging at some point.  Eventually _netbeans.scss should be removed and content added where appropriate.
Other minor fixes / changes -

moved breakpoint code to use Foundation include
increased min. font size a bit
cropped hero image to remove "old" logo
updated slider template - moved inline styles into scss file - removed text next to download button.
downsized the Apache Incubator logo in the footer.

Temporarily viewable at https://nb-incubator-ncs.netlify.com/

This is a professional doing stuff. Thanks, Neil!


Issue with BitPing

Getting the following logs continuously when trying to use BitPing.
stdout: Starting Service
stdout: Running aarch64
stdout: /bin/bitping_service: line 26: /opt/bitping/release/bitping-node-armv7-linux: cannot execute binary file: Exec format error

I have fixed the issue in my fork
https://github.com/technotiger/PiCash


Support for fixed-size chunks

Closes  #43.
Previously, when using fixed-size chunks (using the with_chunks() constructor), the second slice was always empty. However, the compiler didn't know about this.
This PR introduces new structs that know that they are fixed-size chunks and therefore, the second slice (that's empty anyway) isn't needed anymore.
Non-fixed-size chunks still work the same as before, containing two slices.
This PR does several things:

move the previous chunk functionality to the chunks sub-module, hopefully making the documentation better structured
create a new fixed_chunks sub-module
get rid of the .split() method, producer and consumer are now directly created by RingBuffer::new(...)
get rid of the previous RingBuffer::with_chunks(..., ...) constructor and replace it with a combined builder-like RingBuffer::with_chunks(...).of_size(...) that returns a producer-constructor pair
get rid of the Copy trait bound on the uninitialized API

As always, the HTML documentation can be created locally with cargo doc.
Rendered HTML files as ZIP: https://github.com/mgeier/rtrb/suites/2660989728/artifacts/58566186

I've created an alternative PR #52.

I'm closing this.
The suggested API it too complicated for very little gain.
The alternative #52 is a bit better and simpler, but still, the "fixed chunks" use case doesn't seem to be that important after all. It is still supported with the two-slices API, where the second slice is empty in the fixed-chunk case.
Some of the changes of this PR have made it into #57.


Run SonarCloud only on pull requests to master

Run SonarCloud less often to reduce build times for normal CI builds.

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


staticMaps does not render image

Other services seem to work well, but staticMaps keeps returning a "Invalid Key" image.

Hi @cacobala , are you trying it from a free account?
The Static Maps API is not available for free accounts. To use it you have to have a pay account. https://www.maptiler.com/cloud/pricing/#allPlans


Add subdiagonalCount:superdiagonalCount: argument labels to Tensor.bandPart.

subdiagonalCount:superdiagonalCount: argument labels improve clarity.
Parameter descriptions copied from https://www.tensorflow.org/api_docs/python/tf/linalg/band_part.
cc @awav

swift-models needs to be updated after this patch:
/swift-models/Transformer/Model.swift:95:29: error: missing argument labels 'subdiagonalCount:superdiagonalCount:' in call
    let mask = ones.bandPart(-1, queryTimeSteps - keyTimeSteps)
                            ^
                             subdiagonalCount:  superdiagonalCount:

One quick nit, however: could you add a deprecated no-label version to help make it a smoother migration for users?

I suspect this will also help get CI to pass (https://source.cloud.google.com/results/invocations/5b144b04-f116-41f9-b435-574cd360de47/targets/s4tf%2Fapis%2Fubuntu%2Fpresubmit/log)

One quick nit, however: could you add a deprecated no-label version to help make it a smoother migration for users?

Sure! Done in https://github.com/tensorflow/swift-apis/pull/588/commits/71db9e55d52586db97109e7150662d5957021102.

@dan-zheng, Agree. The verbosity is an important thing and it is good to have the naming to avoid confusion. I don't have a strong opinion about how the arguments should be called in bandPart case. Although, I must ask, what do you think about slightly shorter version subdiagCount and superdiagCount?

@dan-zheng, Agree. The verbosity is an important thing and it is good to have the naming to avoid confusion. I don't have a strong opinion about how the arguments should be called in bandPart case.

I'm glad we all agree!
The Swift API design guidelines clearly (haha) advocate clarity:

Clarity at the point of use is your most important goal. Entities such as methods and properties are declared only once but used repeatedly. Design APIs to make those uses clear and concise. When evaluating a design, reading a declaration is seldom sufficient; always examine a use case to make sure it looks clear in context.

Clarity is more important than brevity. Although Swift code can be compact, it is a non-goal to enable the smallest possible code with the fewest characters. Brevity in Swift code, where it occurs, is a side-effect of the strong type system and features that naturally reduce boilerplate.

Although, I must ask, what do you think about slightly shorter version subdiagCount and superdiagCount?

The Swift API design guidelines recommend avoiding abbrevations:

Avoid abbreviations. Abbreviations, especially non-standard ones, are effectively terms-of-art, because understanding depends on correctly translating them into their non-abbreviated forms.

The intended meaning for any abbreviation you use should be easily found by a web search.

"Easily found by a web search" is a concrete measure that we can verify:

https://lmgtfy.com/?q=subdiag
https://lmgtfy.com/?q=superdiag

The top results for "subdiag" and "superdiag" are not related to diagonals from linear algebra: "diag" refers to something else, as in "subdiagram" or "super diagnostics".
Later results do reference subdiagonals and superdiagonals, showing that "subdiag" and "superdiag" are common in-domain abbreviations. They're very reasonable as local variable names.
To adhere to the API design guidelines and to maintain consistency with the rest of the codebase, I'd recommend that we continue to avoid abbreviations.

Review feedback has been addressed.
Merging to avoid conflicts with pending patch for TF-1076: using @derivative for derivative registration.


change http to https

fix the little flaws found on http protocol that is used,
one of the web using the http protocol which is already supporting more secure protocol,
which is https

do fixing #3335
@brandonblack i do  pulling from the latest master but the build status still the same, red.

Strange!  The Travis build #7475 it is one of the strange "extra GitHub commits" Merge 97544fa into 0e9109c, and it is not a build of 97544f which is the submitted PR.
I have seen these magical GitHub commits cause problems before, and the easy solution was to start with a fresh branch, using a new branch name (never use 'master' for pull requests), and rebuild the branch contents before submitting a new pull request.

Erh, what are you talking about? The URLs need to be alphabetised, that's all. See for yourself:

TestPedantic#test_grammars_are_sorted [/home/travis/build/github/linguist/test/test_pedantic.rb:26]:
https://svn.edgewall.org/repos/genshi/contrib/textmate/Genshi.tmbundle/Syntaxes/Markup Template (XML).tmLanguage should come after https://bitbucket.org/Clams/sublimesystemverilog/get/default.tar.gz

@Lidilidian Please move the modified URL so it's listed underneath https://bitbucket.org/Clams/sublimesystemverilog/get/default.tar.gz.

There's no need to close this pull request, by the way. Simply push the changes to your fork's branch, and they'll take effect here.

Erh, what are you talking about? The URLs need to be alphabetised, that's all.

@Alhadis , my apologies.  Yes, I see now only the order of entries needs to be changed to fix this build breakage.

Merged manually. Thanks @Lidilidian 👍


Mbio: add expandable info section to apps

Our apps run some serious computations that are worth explaining a bit to the user. Also we've had user feedback asking for more details on the functions we use for the computations, etc. So, we'd like to add an expandable section, just like with the mosaic plots, that offer more description about the computation.
Each app would use a specific text that describes the computation, and i think the expandable section could go right below th ecomputation inputs. So for example in the next line below Data and Method, but before step 2 in the screenshot. Totally happy to make mockups if that would help!

This ticket will require work from a few folks. To start:

[ ] @dpbisme and @asizemore need to write app-specific text
[ ] a kind frontend dev could help with the implementation? @dmfalke just tagging you here because i believe you already talked to dan about this. But once we're ready we can get anyone with time to implement!

adding comments from the other ticket here, to consolidate the two
There are a lot of places wed like to do some explaining for the computes. some things that come to mind immediately:
Tell people were using pcoa in beta div, and why and what it might be good or bad at
Tell people what parameters were using w maaslin2 in diff abund, and why and what it might be good or bad at
Tell people about correlation methods used in the correlation apps, particularly once we introduce compositional methods for taxa-taxa
Tell people what prefilters were using for features in correlation apps
Tell people where they can find the R if theyd like
Other things i havent thought of yet


URI vs URL and going forward

Nobody can say that I haven't tried... In fact, I have tried twice but "fixing" our uri implementation isn't possible anymore.
Basically there are two camps - camp A correctly says that our URI has encoding/decoding issues and looses data. After fixing said issues, camp B correctly says that the URI behaviour has changed and breaks their extensions/persisted data. We are dead in the water as it seems and we need to find a way forward.
Generally, the URI doesn't cause trouble when its used "internally", e.g. renderer talking to extension host or other pieces, but its decode/encode problems show when interacting with a 3rd player, usually a browser when clicking a link or when calling vscode.env.openExternal. Instead of changing the whole of our uri we should look at those problems specifically and I believe the way out is the use of the standard URL-object, e.g places that feed urls into the system, namely link detection and openExternal, should accept an URL-object in addition to our URI object. While adding overload/confusion it provides a clear separation and ends our catchup game.

Work packages

[ ] use URL for our internal link detection. This will help with the majority of cases, like this https://github.com/microsoft/vscode/issues/45515
[ ] support URL in our API, namely openExternal and DocumentLinkProvider. This will help with custom links, like here https://github.com/microsoft/vscode/issues/79474

@jrieken I just hit an issue today with dollars in a URLs becoming encoded:
vs.Uri.parse("http://localhost:1234/$debug").toString()
"http://localhost:1234/%24debug"

I'm not sure whether this is a known issue and/or covered by this one (the above mentions openExternal, but I'm not using that here, I'm just passing the URL around through some service, and one seems to fail if I give it %24 where there was a dollar - I don't know if that's reasonable for it to do though).

@mjbvz fyi - I have pushed https://github.com/microsoft/vscode/commit/a71694386db4d321ccb6cafcc9e796e320319183 which helps with links that are rendered in Webviews.

To verify

use this link https://firebasestorage.googleapis.com/v0/b/brewlangerie.appspot.com/o/products%2FzVNZkudXJyq8bPGTXUxx%2FBetterave-Sesame.jpg?alt=media&token=0b2310c4-3ea6-4207-bbde-9c3710ba0437
have it inside an editor and click on it from the editor
have it as rendered in markdown and click on it from the markdown preview
have it as jsdoc-markdown hover and click on it from there
have it in the terminal (echo ...) and click on it from there
in ALL cases it should show a loaf of bread


deps(release/v7.5.x): bump SDK to v0.47.12

Description

closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

Only callbacks and ibc-go need the bump? No wasm?

Correct, there's no wasm in this branch.


AngularFileUpload - Adding events & such

Adding proper event to progres event, and add the error & success callbacks with proper defs.

Arg, it failed the tests.  Can anyone help me out here?  I'm not terribly familiar with the needs for a TypeDef file

https://travis-ci.org/borisyankov/DefinitelyTyped/builds/55535760
----------------- For file:angular-file-upload/angular-file-upload.d.ts
angular-file-upload/angular-file-upload.d.ts(31,3): error TS7020: Call signature, which lacks return-type annotation, implicitly has an 'any' return type.
angular-file-upload/angular-file-upload.d.ts(39,3): error TS7020: Call signature, which lacks return-type annotation, implicitly has an 'any' return type.

please try compile angular-file-upload/angular-file-upload.d.ts with --noImplicitAny option.

should be handled by https://github.com/borisyankov/DefinitelyTyped/pull/4407 now


How can I test sourcebuster?

Hi,
Sourcebuster looks great! Though I'm having an issue testing it..
In my page I call sbjs.init() and then something like console.log(sbjs.get.current.src);. When I visit my page directly I properly get (direct). And if I visit my site via site.com?utm_source=blah, I properly get blah.
However, how can I test that 'google' or 'facebook', or others get returned properly (before I deploy to production)?
I tried having google redirect me to my local/dev site, e.g. by visiting https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&cad=rja&uact=8&url=http%3A%2F%2Fblah.com and clicking 'The previous page is sending you to http://blah.com.' But console.log(sbjs.get.current.src); still outputs (direct).
For Facebook, I messaged myself the url to my local/dev site and then clicked that link, though console.log(sbjs.get.current.src); still outputs (direct).
I also used Referer Control to spoof the referrer header, and can verify that the GET header correctly contains Referer:http://google.com, console.log(sbjs.get.current.src); still outputs (direct).
I also attempted to clear cookies before each of those tests.
Any help, insight you could provide would be excellent! Thank you!

Hi, thanks!
Most likely this is the issue with https -> http navigation. Can you check the value of document.referrer in the browser console?
Answering the title question, I implemented sourcebuster in ruby at the first place and in those times I used this to test it.

Thanks so much! You're absolutely correct! I adjusted my local dev to serve via https, and then used the Google redirect (https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&cad=rja&uact=8&ved=0ahUKEwjCtv3TosfSAhUs6YMKHbS8CfAQFggaMAA&url=https%3A%2F%2Fmy.local.site&usg=AFQjCNH6cQf-rEVsmH54WujQjUyMobzjaw&bvm=bv.148747831,d.amc) I see the referrer as 'google'.
I'll have a read through the ruby testing solution you linked to. Closing this issue out. Thanks!! :)


net.batch_query parameters serialization wasn't matched to docs.

Added

ResponseHandler type description into modules.md.

Fixed

net.batch_query parameters serialization wasn't matched to docs.
Module description in docs generator contains null instead of summary.
Function result section header hadn't the line separator before.

Version 1.6.2


clearLocalStorage()

Is your feature request related to a problem? Please describe.
It's good to reset things before starting, especially when not working with a fresh instance of the product.
Describe the solution you'd like
A new function/method.

Hi, there is already available a command called clearLocalStorageSnapshot. The "Snapshot" suffix was added to clearly indicate that the real localStorage is not being cleared, only the internal snapshot used by the package to save and restore it is.
Please refer to the README.md file for further info.

What if the app updates storage after the snapshot was taken?

The app localStorage is cleaned between every test automatically by Cypress (that is the reason of this package to exist). If you want to clean the localStorage explicitly Cypress already provides a command: cy.clearLocalStorage

Doh, thanks!


New flag to handle inline string parsing

Should fix issue 15 (tested on Windows and linux)

I think this is OK

This needs more refinement.
You can not just do
echo "+--+\n| |\n+--+" | svgbob -s

It should still work just like before if you remove the -s flag. I made the modification so that you can still do :
echo "+--+\n| |\n+--+" | svgbob
and for the cases where you don't want to use echo :
svgbob -s "+--+\n| |\n+--+"

Ah ok, got it.
However there is an inconsistency though.
This works fine.
svgbob -s "+--+\n| |\n+--+"
but not
svgbob -s "hello"
neither
svgbob -s "----------+-----------"
This works though
svgbob -s "||||||"
I'm guessing there is a conflict with subcommands with the clap library.

Ah yes it seems clap is parsing the command in an unexpected way, but as it suggests in some case, you can do this :
svgbob -s -- "hello"
which works fine for me

Ok, I've changed the Readme and I will release a new svgbob_cli into crates.io shortly.


[Question] for nvidia shield users

I'd like to know if its possible to use external drive to download the torrents in the nvidia shield directly
thanks in advance

can't say about nvidia shield in particular, but for arbitrary android tv box the approach is to format external HDD as internal storage and move apps data to it (necessarily).
see https://github.com/elgatito/plugin.video.elementum/issues/283#issuecomment-451776413
and then set download folder to something like Android/data/org.xbmc.kodi/elementum/ (full path would look like /storage/emulated/0/Android/data/org.xbmc.kodi/elementum/) because of security constraints.
if by "directly" you mean without formatting - then no, android security restriction (at least it used to be).

can't say about nvidia shield in particular, but for arbitrary android tv box the approach is to format external HDD as internal storage and move apps data to it (necessarily).
see #283 (comment)
and then set download folder to something like Android/data/org.xbmc.kodi/elementum/ (full path would look like /storage/emulated/0/Android/data/org.xbmc.kodi/elementum/) because of security constraints.
if by "directly" you mean without formatting - then no, android security restriction (at least it used to be).

I use actually a mibox and it has a usb por 2.0 formating like you said it makes the box slow due the 2.0 speed, i never tried using like that, but as nvidia shield has a 3.0 port it works good? It has to be a external ssd?
I got the box so im deciding if i keep or return cuz the box is freaking expensinve here, as one of the reasons I want to be able to download torrents using the box. Thank you very much for replying and im sorry abt my english

So, I use mi box 3, it has usb 2.0 (but my hdd has 3.0). And it works fine for me.
When I tried with usb stick - then yes, it was extremely slow.
But with hdd I don't feel any speed issues.
But nvidia shield might have some security quirks, so I suggest you to test approach I described above first.
Also note that android 10 is currently not supported by elementum (see relevant issue).

also you can do some simple math to estimate your use case:
let's say for DBRemux with size 60GB and length 90min we got about 11MB per second. (of course your internet also should support that).
then theoretical speed of usd 2.0 is 480 Mb/s => 60 MB/s. in reality (on PC) it is about 40 MB/s. Which should be enough.
of course you should have at least decent HDD.
the best way to check is to install some hdd speed test application on android and see real results. then you can find out what kind of releases your hdd will be able to handle.

also you can do some simple math to estimate your use case:
let's say for DBRemux with size 60GB and length 90min we got about 11MB per second. (of course your internet also should support that).
then theoretical speed of usd 2.0 is 480 Mb/s => 60 MB/s. in reality (on PC) it is about 40 MB/s. Which should be enough.
of course you should have at least decent HDD.
the best way to check is to install some hdd speed test application on android and see real results. then you can find out what kind of releases your hdd will be able to handle.

Thank you very much for all the info, it has been very useful. I only tried using as internal storage a 3.0 usb stick that i found recomended in a facebook group, never tired any hdd, i have 2 hdd 3.0 im gonna try them.

When you turn off the mibox the hdd turn off too?

if by "turn off" you mean sleep (since there is no "turn off" in android tv) - then yes, hdd also goes to sleep.
but i also exit kodi before sleep (iirc i had some issues with hdd sleep without exit from kodi).

When you turn off the mibox the hdd turn off too?

if by "turn off" you mean sleep (since there is no "turn off" in android tv) - then yes, hdd also goes to sleep.
but i also exit kodi before sleep (iirc i had some issues with hdd sleep without exit from kodi).

yes, this is what i meant, sorry, putting it in sleep mode. So I need to close kodi every time I stop using? sometimes the exit from kodi doesnt work, it get stucked in a black screen or something =x

i mean, you better to test it by yourself on your system. it may vary.
i do this by force of habit :-)
btw, if it got stuck - you can stop it in android setting.
also iirc i changed settings of hdd to go to sleep earlier than it was by default (WD has utility for their drives, for example).

i mean, you better to test it by yourself on your system. it may vary.
i do this by force of habit :-)
btw, if it got stuck - you can stop it in android setting.
also iirc i changed settings of hdd to go to sleep earlier than it was by default (WD has utility for their drives, for example).

Yes, it can be stopped in android settings, one time I erased data instead by mistake, fortunelly i make backups of my kodi
I'll try with my hdds and probably return the nvidia (havent opened the box yet)
Thanks again, you have been kind

Your HDD has exernal power source?

no.
i have Western Digital My Passport USB3.0 USB 3.1 (WDBEMM0010BSL 1 TB), you can see specs here
https://translate.google.com/translate?sl=ru&tl=en&u=https://www.nix.ru/autocatalog/external_hdd_western_digital/WD-WDBEMM0010BSL-EEUE-My-Passport-USB30-Drive-1TB-Silver-25-EXT-RTL_139262.html

no.
i have Western Digital My Passport USB3.0 USB 3.1 (WDBEMM0010BSL 1 TB), you can see specs here
https://translate.google.com/translate?sl=ru&tl=en&u=https://www.nix.ru/autocatalog/external_hdd_western_digital/WD-WDBEMM0010BSL-EEUE-My-Passport-USB30-Drive-1TB-Silver-25-EXT-RTL_139262.html

Ah ok, thanks

Can you name any good app to run the hdd speed test?

for android tv - not really.
you can try https://play.google.com/store/apps/details?id=mark.fakedevicetest (i used it for phone)
or something from https://play.google.com/store/search?q=hdd speed&c=apps
and i guess you will need to download apk and install it manually, since these tools probably will be unavailable in android TV play marker.

That will be no problem, thanks

for android tv - not really.
you can try https://play.google.com/store/apps/details?id=mark.fakedevicetest (i used it for phone)
or something from https://play.google.com/store/search?q=hdd speed&c=apps
and i guess you will need to download apk and install it manually, since these tools probably will be unavailable in android TV play marker.

I have a thumb drive 3.0 with 64gb I tested it and it was around 6mb/s writing speed, i decided to test it and it worked but sometimes it was a bit laggy. I saw a lot of people were recomeding thje sandisk SDCZ430 so i decided to buy and its around 17mb/s writing speed and its working perfectly. I don't really need much space just to alocate the actual torrent im watching i nthe moment, i don't keep them.
This is just my feedback and again thank you for sharing your experience with me, it was really helpful

@elgatito if you want to close this, maybe it will be helpful for others. Thx

I would not recommend to use USB stick, since they all very slow (in comparison to hdd) unless you use some ssd nvme usb stick (which will be unreasonably expensive for this use case).
And with HDD you can contribute to community by keeping and seeding files.
Anyway, in your case you also can try yo use "memory storage" option if you don't need to download something beforehand and watch later. This probably will be more smooth experience than with usb stick.

@D4anielCB could you please rename this issue, e.g. "How to use external HDD with elementum on android"

@D4anielCB ?


Update to 0.7.4

Update Cloud controller to 0.7.4
Update cloud api to work with changes from 0.7.4

see also this controller pr


Contact Us for International Affiliates

Email addresses for each of the affiliates:
Nigeria -<EMAIL_ADDRESS>Botswana -<EMAIL_ADDRESS>Congo -<EMAIL_ADDRESS>Kenya -<EMAIL_ADDRESS>Ghana -<EMAIL_ADDRESS>UK -<EMAIL_ADDRESS>Canada -<EMAIL_ADDRESS>cc @aaronschachter @DeeZone @mshmsh5000

closing, this was done on those static content pages!


Game crashes on start up after updating to ver.1.39

After updating to 1.39, for some reason the game keeps crashing on the logo when you open the game. I don't know if its because of maid fiddler, but it started working after I uninstalled maid fiddler.

The tool seems to work fine on 1.39 on my end. Make sure you have installed the tool properly.
If you encounter any problems, make sure to include an error log (or a detailed description of the error) before creating an issue. That way the problem can actually be fixed.


Odd error during next build

Using:
"next": "13.5.2",
"next-sanity": "^5.5.4",
"@sanity/client": "^6.4.12",
When I run next build I get the below:
./node_modules/@sanity/client/src/SanityClient.ts:60:3
Type error: Property '#clientConfig' has no initializer and is not definitely assigned in the constructor.
58 |    * Private properties
59 |    */

60 |   #clientConfig: InitializedClientConfig

This appears to have been an issue with an old/bad tsconfig.json. Updated to match latest Next release file and can now build.


[WIP] macOS Getting Started README to ease development

These documentation changes help a macOS user get started with development on this repository from absolute zero. Tries to address #30, your most ancient bug.
This documents my experience getting a decent editing setup on macOS.
A suggestion for a productivity-friendly editor.
How to run tests.
How to build executables
Where to find application entry points.

@theaeolianmachine can you look into merging or closing this when you have a moment? It's got a lot of good stuff, but docs have also changed significantly since it was submitted. Maybe it's good to convert into some new changes like with @robertDurst's docs changes.

This info has now been superceded by our newer docs, which describe the new Go modules approach, and also the alternative docker workflow for devs. Closing.


Marketplace review - Extension not working

Rayner Pupo:
"I can't make it work, I've installed vscode and the ghc-mod addon, restarted the app and none of the features are working so far. I've opened the Developers console and there isn't any error or warning."

Troubleshooting steps:

Have you installed everything needed for the extension to run?

[X] Vscode
[X] Vscode ghc-mod extension
[ ] ghc-mod itself? (either via caball install ghc-mod, stack, or a different method: http://www.mew.org/~kazu/proj/ghc-mod/en/install.html)

Add this to your user or workspace settings to enable the most verbose logging: "haskell.ghcMod.logLevel": "log"

What VS Code version, ghc-mod version, and OS version are you using? For example:
code -v -> 1.6.1
ghc-mod --version -> ghc-mod version <IP_ADDRESS> compiled by GHC 7.10.3

I may have the same problem in vs code 1.7.2 and ghc-mod version <IP_ADDRESS> compiled by GHC 8.0.1
I've set logLevel to log. There is only a stack trace in extension console:
TypeError: Cannot read property 'apply' of null
    at CallbackList.invoke (/home/wiz/.vscode/extensions/hoovercj.vscode-ghc-mod-1.0.1/server/node_modules/vscode-languageserver/node_modules/vscode-jsonrpc/lib/events.js:103:38)
    at Emitter.fire (/home/wiz/.vscode/extensions/hoovercj.vscode-ghc-mod-1.0.1/server/node_modules/vscode-languageserver/node_modules/vscode-jsonrpc/lib/events.js:167:36)
    at /home/wiz/.vscode/extensions/hoovercj.vscode-ghc-mod-1.0.1/server/node_modules/vscode-languageserver/lib/main.js:287:39
    at handleNotification (/home/wiz/.vscode/extensions/hoovercj.vscode-ghc-mod-1.0.1/server/node_modules/vscode-languageserver/node_modules/vscode-jsonrpc/lib/main.js:267:17)
    at process.callback (/home/wiz/.vscode/extensions/hoovercj.vscode-ghc-mod-1.0.1/server/node_modules/vscode-languageserver/node_modules/vscode-jsonrpc/lib/main.js:363:13)
    at emitTwo (events.js:106:13)
    at process.emit (events.js:191:7)
    at process.nextTick (internal/child_process.js:744:12)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Hi @wiz, I'm sorry you're having this problem.
This looks similar to the stack trace reported here: https://github.com/hoovercj/vscode-ghc-mod/issues/39
After looking at the logs in the developer tools, that user found that the solution in that case was to run cabal install.
Please let me know if that helps. I now see that changing over to properly use output channels for logging needs to move up on my priority list.

I invoked ghc-mod manually and it indeed shows the symbol information requested.
I've tried to set executablePath to stack wrapper (I don't use cabal) and nothing changed.

Hm. You can open the developer tools (in the command palette" "toggle developer tools") and you should see more useful logs there from the extension. Does that show anything useful?

Welp.. It actually works now. Damn heisenbugs :smile:
So, my config is
"haskell.ghcMod.executablePath": "/home/wiz/.local/bin/stack-ghcmod",

Which points to an script containing simply
stack exec ghc-mod -- $@

My stack support needs improved, but I'm glad that it's working for you for now. If you have the same problem again and can get more information from the developer tools, please paste it here and reopen the issue.
Thanks for your patience and I hope you find the extension useful!

@wiz I'm glad you were able to workaround my lack of stack support, but you might find that it has improved. The setting haskell.ghcMod.executablePath": "stack" should help you avoid needing a bash script for indirection.

Thanks! I'll sure check it out.


Loading resources over HTTP should produce informative, consistent exceptions

Issue: we don't currently know what passing an invalid URL to XMLHTTPRequest will do across different platforms, and we're not sure what the correct behavior should be. We probably want to throw a JavaScript exception, but should that be thrown in the Send() method or the Open() method?
Things to consider:

How closely should we match behavior on the web? The spec does not explicitly specify where exceptions should be thrown, but typically browsers will throw DOMExceptions on open() if the path is invalid.
Is it even possible to throw exceptions from open() on all platforms? That would require that the platform allows us to parse the URL as a separate action from the network request, which may not be the case for all platforms. (it is at least true for Android)
What are we currently doing on Windows and iOS?

The solution here is to decide on a contract and ensure that it's implemented consistently across all platforms. It would be nice to have unit tests for these exceptions as well.
Feel free to add any further comments @bghgary

As of right now, the behavior on Windows is similar to the (unpatched) Android behavior. Attempting to load "noprotocol.gltf" or similar causes the following error:

Exception thrown at 0x00007FFDCBA54B89 (KernelBase.dll) in Playground.exe: WinRT originate error - 0x80070057 : 'noprotocol.gltf is not a valid absolute URI.'.

This leads to the program aborting. We may want to make this behavior match the new Android behavior, where the exception is passed to the Javascript code.

Waiting on #802, want to make sure that the code has strong unit testing in place before making further changes in order to prevent regressions.

Got an interesting issue while doing a Validation tests run on iOS device:
ValidationTests[9302:12667787] App Transport Security has blocked a cleartext HTTP (http://) resource load since it is insecure. Temporary exceptions can be configured via your app's Info.plist file.
2021-07-02 14:52:23.013913+0200 ValidationTests[9302:12667787] Cannot start load of Task <CBB9B7B6-C19A-494A-88EE-C6511FDD3F6A>.<28> since it does not conform to ATS policy
2021-07-02 14:52:23.018328+0200 ValidationTests[9302:12667787] Task <CBB9B7B6-C19A-494A-88EE-C6511FDD3F6A>.<28> finished with error [-1022] Error Domain=NSURLErrorDomain Code=-1022 "The resource could not be loaded because the App Transport Security policy requires the use of a secure connection." UserInfo={NSLocalizedDescription=The resource could not be loaded because the App Transport Security policy requires the use of a secure connection., NSErrorFailingURLStringKey=http://models.babylonjs.com/Tests/TextureTransform/TextureTransformTest.gltf, NSErrorFailingURLKey=http://models.babylonjs.com/Tests/TextureTransform/TextureTransformTest.gltf, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataTask <CBB9B7B6-C19A-494A-88EE-C6511FDD3F6A>.<28>"
), _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask <CBB9B7B6-C19A-494A-88EE-C6511FDD3F6A>.<28>, NSUnderlyingError=0x28180a250 {Error Domain=kCFErrorDomainCFNetwork Code=-1022 "(null)"}}

I didn't go any further with that. But it might be an interesting test to check https and http work on all platforms.


fix: use palette touchable /w underlayColor & consistency style changes

The type of this PR is: Enhancement

This PR resolves FX-2254
Description
ArtistSeriesListItem was using TouchableOpacity from react-native and not Touchable from palette.
Furthermore the underlayColor wasn't the same as other parts of the app so added "black5" to match the other Touchables.
The screenshots below show the touchable behaviour when the user is pressing the button.
Also made the <ArtistSeriesListItem /> to take 100% width of the list and adjusted margins accordingly for consistensy as suggested from design here

Android
iOS

More screenshots with all affected screens

ArtistSeries
ArtistAbout

ArtistSeriesMoreSeries
Artwork

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

Changelog updates

Cross-platform user-facing changes

Fix add touchable underlay color in ArtistSeriesListItem

Good observation. Sure thing!
I also noticed another inconsistency that was discussed on this slack thread with Johnathan as well and will address everything from the reply in this pr.

#squashongreen


[bug] NSIS updater assets are not created if specify targets to ["msi", "app", "dmg", "updater", "nsis"]

Describe the bug
if I specify ["msi", "app", "dmg", "updater", "nsis"] in targets, then updater assets(.zip/.sig) are created for msi but no updater assets are created for nsis, if don't specify targets, then updater assets are created for both msi and nsis.
Reproduction
No response
Expected behavior
No response
Platform and versions
[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 114.0.1823.67
    ✔ MSVC:
        - Visual Studio Community 2019
        - Visual Studio 生成工具 2022
    ✔ rustc: 1.70.0 (90c541806 2023-05-31)
    ✔ Cargo: 1.70.0 (ec8a8a0ca 2023-04-25)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (environment override by RUSTUP_TOOLCHAIN)
    - node: 16.18.0
    - yarn: 3.2.4
    - npm: 8.19.2

[-] Packages
    - tauri [RUST]: 1.4.1
    - tauri-build [RUST]: 1.4.0
    - wry [RUST]: 0.24.3
    - tao [RUST]: 0.16.2
    - @tauri-apps/api [NPM]: not installed!
    - @tauri-apps/cli [NPM]: 1.4.0

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1420/

Stack trace
No response
Additional context
No response

I couldn't reproduce, please share a minimal repro.
Here is the output when "targets": ["msi", "nsis", "updater"]
Finished release [optimized] target(s) in 1m 35s
        Info Target: x64
     Running candle for "main.wxs"
     Running light to produce D:\.cargo-target\release\bundle\msi\tauri-app_0.0.0_x64_en-US.msi
        Info Target: x64
     Running makensis.exe to produce D:\.cargo-target\release\bundle\nsis\tauri-app_0.0.0_x64-setup.exe
    Bundling D:\.cargo-target\release\bundle\msi\tauri-app_0.0.0_x64_en-US.msi.zip
    Bundling D:\.cargo-target\release\bundle\nsis\tauri-app_0.0.0_x64-setup.nsis.zip
    Finished 2 bundles at:
        D:\.cargo-target\release\bundle\msi\tauri-app_0.0.0_x64_en-US.msi
        D:\.cargo-target\release\bundle\nsis\tauri-app_0.0.0_x64-setup.exe
        D:\.cargo-target\release\bundle\msi\tauri-app_0.0.0_x64_en-US.msi.zip (updater)
        D:\.cargo-target\release\bundle\nsis\tauri-app_0.0.0_x64-setup.nsis.zip (updater)

    Finished 2 updater signatures at:
        D:\.cargo-target\release\bundle\msi\tauri-app_0.0.0_x64_en-US.msi.zip.sig
        D:\.cargo-target\release\bundle\nsis\tauri-app_0.0.0_x64-setup.nsis.zip.sig

no updater assets are generated for NSIS when I set "targets": ["msi", "app", "dmg", "updater", "nsis"],, once I removed this line, they are generated correctly. I will try to make an minimal app later.

Try running with --verbose flag as well and post the output here

I can finally reproduce the bug, it is related to the order you define the targets, if updater target is not the last one in the list, it will behave weirdly.
Until a proper fix is released, just keep updater target at the end of the list and all should be fine.


Add "till click" support for iOS

This allows to disable the timer, so it disappears as soon as you touches it.
Soory but I'm only able to implement this in iOS maybe you can do this for the others.
Cheers Osile

Old but gold, how about updating and merging this? Seems like a nice addition 👍


Prevent linkify-urls from running multiple times on review comments

Both .blob-wrapper and .comment-body matches a code block in review comments, so the linkify functions got run for multiple times.

LINKED ISSUES:
Fix #3226

TEST URLS:
https://github.com/sindresorhus/refined-github/pull/3225#discussion_r439535538

SCREENSHOT:
Before:
After:

I think we should limit the comment-body selector instead. It should only work on code (I think)

Do you mean it should apply to standalone comments (i.e. not review comments)?

I think this feature needs to be slightly rewritten to avoid this duplicate select.all usage
Can you try picking up https://github.com/sindresorhus/refined-github/pull/3091 instead? It should be better because:

it only affects visible code, also used in #2737
it selects the specific elements instead of "observe their parent + select.all inside"
it should also solve https://github.com/sindresorhus/refined-github/issues/2091, which was for linkify-code but also affects linkify-urls afaik


Fix doc issue with unListed map, unREPLable

I was working through some of the syntax page with the REPL and saw that the squares example didn't have a List. (turns out it does in the current source, but the previous usage in the Record section did not). Presuming everything should REPL...

Seems okay to me, thanks!


Slim 976 couple unbound connected

Implemented cucumber scenarios for coupling unbound devices.

This pull request makes the build fail.

This pull request makes the build fail.

retest this please

This pull request makes the build fail.

All is well.

All is well.


Suboptimal SQL generated on Left Join

Left join generates suboptimal query when more then two entities are used in a query.
When I use left join with two entities it works fine but when I include third entity in the query It devides SQL in two portions. The first retrieves the whole table and the second does the left join. It also discard any filter associated with the query.
LINQ Query
void Main()
{
	var rep = new Repository();

	var q = from entity in rep.All<Entity>()
		    join attr in rep.All<EntityAttribute>() on entity.Id equals attr.EntityId into et
		    join mod in rep.All<ModuleDefinition>() on entity.ModuleSubmoduleId equals mod.Id
		    where mod.Id  < 20000
		    select new { entity, et , mod};

	var r = q.ToList();

}

Generated SQL
SELECT [mod].[ID], [mod].[CONNECTION_DEFINITION_ID], [mod].[DESCRIPTION], [mod].[INSERT_TIME], [mod].[INSERT_USER_ID], [mod].[IS_ACTIVE], [mod].[IS_INTERNAL], [mod].[NAME], [mod].[ROW_VERSION], [mod].[TFS_PATH], [mod].[TYPE_ID], [mod].[UPDATE_TIME], [mod].[UPDATE_USER_ID]
FROM [dbo].[MODULE_DEFINITION] AS [mod]
GO

SELECT [entity].[ID], [entity].[DESCRIPTION], [entity].[INSERT_TIME], [entity].[INSERT_USER_ID], [entity].[IS_AUDITABLE], [entity].[IS_SYSTEM_TABLE], [entity].[MODULE_SUBMODULE_ID], [entity].[NAME], [entity].[ROW_VERSION], [entity].[UPDATE_TIME], [entity].[UPDATE_USER_ID], [attr].[ID], [attr].[DATA_TYPE_ID], [attr].[DESCRIPTION], [attr].[ENTITY_ID], [attr].[HAS_TRANSLATION], [attr].[INSERT_TIME], [attr].[INSERT_USER_ID], [attr].[IS_NULLABLE], [attr].[NAME], [attr].[ROW_VERSION], [attr].[UPDATE_TIME], [attr].[UPDATE_USER_ID]
FROM [dbo].[ENTITY] AS [entity]
LEFT JOIN [dbo].[ENTITY_ATTRIBUTE] AS [attr] ON [entity].[ID] = [attr].[ENTITY_ID]
ORDER BY [entity].[ID]
GO

EF Core version: 2.1.4
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows
IDE: e.g. Visual Studio 2017

Note from triage: This query may not have a translation in SQL. @smitpatel to respond.

This query contains a non-flattened GroupJoin. Grouping cannot be represented in SQL. Hence we have to do client computation to generate Groups. If you apply any operator after producing group, it will also do client evaluation. Hence you get additional SQL which will get data to perform the join on client side.
The easy work-around would be to do GroupJoin last so that everything before that translates to server. In this case it is possible because your join is not using anything from Grouping (if it was using then there is no way other than client eval). You can move join before groupjoin and make it work on server side like this
var query = (from b in db.Blogs
                join bd in db.BlogDetails on b.BlogDetailId equals bd.Id
                join p in db.Posts on b.Id equals p.BlogId into ps
                where bd.Id < 2000
                select new { b, ps, bd }).ToList();

SQL
      SELECT [b].[Id], [b].[BlogDetailId], [bd].[Id], [p].[Id], [p].[BlogId]
      FROM [Blogs] AS [b]
      INNER JOIN [BlogDetails] AS [bd] ON [b].[BlogDetailId] = [bd].[Id]
      LEFT JOIN [Posts] AS [p] ON [b].[Id] = [p].[BlogId]
      WHERE [bd].[Id] < 2000
      ORDER BY [b].[Id]

dear smitpapel,
we are using efcore in a very key Project. this problem cannot be a by design problem. we are not talking about groups. I think there is a misunderstanding here.
Our problem is just fecthing related entity datas by using left join. If we use one related entity in our linq, efcore creates left join successfully but when we add one more join relation for another left join related entity in our linq, it starts to generate seperate non-filtered/full-select query for each entity. In our case this query generation approach is unacceptable. Every entity/table will hold millions of records. so fetching all entities without filtering and then joinin these entities in application memory is not acceptable. there should be a way to generate SQL Left join statements when there is more than two entity relations in linq statements.

When you write this join mod in rep.All<ModuleDefinition>() on entity.ModuleSubmoduleId equals mod.Id in your query you have access to et group also. That is how LINQ works but that is not something SQL can process. That is the reason you can actually change order of joins to make this work otherwise, your query cannot be translated to server at all. If you are loading related data only then consider using Include instead. While we may generate similar query for Include but we are certain that we are not using groups anywhere other than top level select. But for the query you are writing it is not possible to figure that out in expression tree without doing complex analysis which may still fail if group is being referenced.
Above query will throw in 3.0 pipeline so you will not end up with loading whole table accidently.


fix TypeError on node[child]; return a str from str

The changes for this are also on https://github.com/ryanfox/bktree/pull/4


limit numpy

The recently released breaking version of Numpy requires some housekeeping before it can be integrated.
Failed to import(cimport) numpy ndarray used in data/_adjustments.pyx.
Upon fixing that there was some other issue related to pandas.
Have not investigated further.
I'd like to suggest limiting Numpy for now

@lhjnilsson if you have a fix for data/_adjustments.pyx please let me know.
I have applied a number of changes on #259 but I'm facing now this error on data/_adjustments.pyx, we can consolidate fixes on #259

@gnzsnz have not looked into it any further actually.
ps. is there any discord or so for all things zipline- related from @stefan-jansen ? i know there is a forum.. but something more ad-hoc suited


Intellisense for dart not working

Issue Type: Bug
I use vs code with dart,
the Intellisense is not working, and also - go to definition (F12)
VS Code version: Code 1.22.2 (3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9, 2018-04-12T16:38:45.278Z)
OS version: Windows_NT x64 10.0.15063

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6500 CPU @ 3.20GHz (4 x 3192)

Memory (System)
15.89GB (5.39GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe --open-url -- vscode:extension/Dart-Code.dart-code

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

dart-code
Dar
2.12.1

beautify
Hoo
1.3.0

csharpfixformat
Leo
0.0.69

PowerShell
ms-
1.7.0

debugger-for-chrome
msj
4.3.0

vscode-icons
rob
7.23.0

format-all-files
xyk
0.2.0

Please file issues against the Dart extension you are using.


output directory does not exist

after compiling and running METISSE, I got the following error
At line 62 of file ../src/evolv_metisse.f90 (unit = 120)
Fortran runtime error: Cannot open file './output/evolve_09628M.dat': No such file or directory
It turns out that this is because I did not have an existing output directory. Simply doing 'mkdir output' resolved this issue.
Possible solutions include adding an empty output directory to the repository, adding a note about this to the documentation or configuring the write statement to check and create the output directory if it does not already exist.

Thanks for pointing this out Simon. This has been fixed in the latest version.


Can't Handle Unicode Correctly

The NRPE charm crashes when receiving unicode data:
2018-02-24 00:36:35 INFO monitors-relation-changed Traceback (most recent call last):
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/var/lib/juju/agents/unit-nrpe-47/charm/hooks/monitors-relation-changed", line 3, in <module>
2018-02-24 00:36:35 INFO monitors-relation-changed     services.manage()
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/var/lib/juju/agents/unit-nrpe-47/charm/hooks/services.py", line 27, in manage
2018-02-24 00:36:35 INFO monitors-relation-changed nrpe_helpers.NagiosInfo(),
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/var/lib/juju/agents/unit-nrpe-47/charm/hooks/nrpe_helpers.py", line 205, in __init__
2018-02-24 00:36:35 INFO monitors-relation-changed self['nagios_hostname'] = self.principal_relation.nagios_hostname()
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/var/lib/juju/agents/unit-nrpe-47/charm/hooks/nrpe_helpers.py", line 167, in nagios_hostname
2018-02-24 00:36:35 INFO monitors-relation-changed     principal_unitname = hookenv.principal_unit()
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/var/lib/juju/agents/unit-nrpe-47/charm/hooks/charmhelpers/core/hookenv.py", line 220, in principal_unit
2018-02-24 00:36:35 INFO monitors-relation-changed     md = _metadata_unit(unit)
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/var/lib/juju/agents/unit-nrpe-47/charm/hooks/charmhelpers/core/hookenv.py", line 520, in _metadata_unit
2018-02-24 00:36:35 INFO monitors-relation-changed     return yaml.safe_load(md)
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3/dist-packages/yaml/__init__.py", line 94, in safe_load
2018-02-24 00:36:35 INFO monitors-relation-changed     return load(stream, SafeLoader)
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3/dist-packages/yaml/__init__.py", line 70, in load
2018-02-24 00:36:35 INFO monitors-relation-changed     loader = Loader(stream)
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3/dist-packages/yaml/loader.py", line 24, in __init__
2018-02-24 00:36:35 INFO monitors-relation-changed Reader.__init__(self, stream)
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3/dist-packages/yaml/reader.py", line 85, in __init__
2018-02-24 00:36:35 INFO monitors-relation-changed self.determine_encoding()
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3/dist-packages/yaml/reader.py", line 124, in determine_encoding
2018-02-24 00:36:35 INFO monitors-relation-changed     self.update_raw()
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3/dist-packages/yaml/reader.py", line 178, in update_raw
2018-02-24 00:36:35 INFO monitors-relation-changed     data = self.stream.read(size)
2018-02-24 00:36:35 INFO monitors-relation-changed   File "/usr/lib/python3.4/encodings/ascii.py", line 26, in decode
2018-02-24 00:36:35 INFO monitors-relation-changed     return codecs.ascii_decode(input, self.errors)[0]
2018-02-24 00:36:35 INFO monitors-relation-changed UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 311: ordinal not in range(128)
2018-02-24 00:36:35 ERROR juju.worker.uniter.operation runhook.go:107 hook "monitors-relation-changed" failed: exit status 1

I believed this to be a unicode python bug such as the one described in:
https://bugs.launchpad.net/charms/+source/ceph-radosgw/+bug/1542074

(by hloeung)
This is probably similar to LP: #1721200

(by stub)
Is metadata.yaml in the built charm valid UTF-8? The yaml library is failing to parse it.
It may be triggered by the version of the yaml library installed. nrpe is not a reactive charm, as it is using the python3-yaml deb package version of the library.

(by stub)
(charmhelpers.core.hookenv.principal_unit() is falling back to compatibility code, attempting to to discover the principal unit name by looking at the metadata.yaml of the other cohosted units. So the metadata.yaml triggering this could be the primary charm's or another subordinate).

(by xavpaice)
This needs some further testing to get a reliable reproducer, I suspect it's only an issue when there's non-ascii chars in the data passed over the relation between the subordinate and principal.


ML Model: experiment with a Transformer model

Depending on the size of the dataset, using a Transformer model from https://github.com/huggingface/transformers could boost the accuracy of the classifier.
I can give a hand if needed=)

Yes, I will be looking into using one of those this iteration! Any help would be appreciated :)
I was planning on working with them via https://simpletransformers.ai/docs/multi-class-classification/, as it seems to wrap up the models in a way that makes quite a bit more sense to someone with zero ML experience (me).
How much data do you think would be needed for one of those models to work well?

Ref https://github.com/microsoft/vscode/issues/97537

Yes, simple-transformers should work.
50 to 100 samples for each label would probably work well already. Is the human-labelled dataset accessible somewhere? (or one should just call GitHub API?)
(BTW: how do you differentiate between labels that were human-annotated vs. labels that were attributed automatically?)
Also this model card – trained on CodeSearchNet by @hamelsmu – might be of interest – i.e. you would probably want to two steps:

pre-train your model on a massive amount of (non-labelled) issues' text (maybe just the vscode repo, but probably a larger slice of GitHub)
Then finetune a multi-class classification layer on your labelled dataset.

Happy to help if needed!

@jlewi is working on building an issue label classifier right now for Kubeflow, specifically this project https://github.com/kubeflow/code-intelligence/tree/master/Label_Microservice
I would suggest using Google BigQuery to get a large batch of issue data at once.  Here is a blog post that will help you How to Automate Tasks on GitHub With Machine Learning for Fun and Profit

pre-train your model on a massive amount of (non-labelled) issues' text (maybe just the vscode repo, but probably a larger slice of GitHub)

Does this mean I would not be using a base like bert-base-uncased but rather training my own? vscode has ~90000 issues, how many do you think this would need?

how do you differentiate between labels that were human-annotated vs. labels that were attributed automatically

Using the github graphQL api I can get the timeline of label events, which includes the creator of each event. I filter out the known-bots:
https://github.com/microsoft/vscode-github-triage-actions/blob/master/classifier/train/fetch-issues/createDataDir.ts#L267-L272

Is the human-labelled dataset accessible somewhere?

I don't think I'm at liberty to post it myself, what with GDPR/etc.
One can get a copy of the data by running
INPUT_TOKEN="{github pat}" GITHUB_REPOSITORY="Microsoft/vscode" node classifier/train/fetch-data/index.js

on the deep-investigations branch of this repo

@ThilinaRajapakse, is it possible to pre-train models in simple-transformers? I wasn't able to find anything referencing it in a skim of the docs.

Yes, it is possible. I'm still working on completing the documentation at simpletransformers.ai, so it's actually not added there yet. But, you can find the relevant info in the readme.
You can start with a pre-trained model like bert-base-cased and pre-train it on the large, unlabelled Github issue dataset.
Pre-training a pre-trained model sounds weird, but the way I think of it is, pre-training is like teaching a language to the model. bert-base-cased already "knows" English, but probably isn't familiar with the programmer-jargon of Github issues. So, the idea is to retain the English knowledge while adapting the model to understand the more technical language found in issues. This is done by using the same technique (masked language modeling) that was used to initially train the original, pre-trained bert-base-cased model, hence the term pre-training.
Then, you can take the model that was pre-trained on the Github issues, add the multi-class classification layer (done automatically when the model is loaded to a classification model), and fine-tune it on your custom, labelled dataset.
I hope that made sense!

@ThilinaRajapakse what's the relation of simple transformers to Hugging Face?  Is this a separate library?
Does anyone have an example of using a hugging face model that is fine tuned on a dataset and adding a custom head for a classification task, just using Hugging face and not another library?  I looked through the docs and wasn't  clear if there was something like this included cc: @julien-c

Simple Transformers is built on top of and relies on the Hugging Face library, but it is a separate library. I am not affiliated with Hugging Face. It was something I wrote for my own convenience in the beginning but some people seem to find it useful, so I've kept expanding it.

@ThilinaRajapakse it does seem useful a bit nervous about introducing more dependencies but I think your abstraction is really useful especially when doing common tasks.  Thanks for sharing!

Thank you!

Just an FYI for bulk downloads we switched to using BigQuery to fetch the data to avoid  hitting GitHub's GraphQL API limits. The code to construct the comment stream and labels based on GitHub events is here
https://github.com/kubeflow/code-intelligence/blob/master/py/code_intelligence/github_bigquery.py
The BigQuery data is from the GitHub Archive which I believe also makes tarballs available.
https://www.gharchive.org/
This means we also have negative examples; i.e. we have issues where a label was added by either a human or bot and then removed by a human because it wasn't correct.
@julien-c Do you have suggestions about how we could incorporate these negative examples into training?

Do you have suggestions about how we could incorporate these negative examples into training?

I know you didn't ask me the question but I can try to answer_ the fact that you initially had the wrong label, but you have the correct label now doesn't seem like something you would handle differently from the main case, (aside from this suggesting that this is a much harder example for your model to classify, but would have to look at those examples to determine this).
If you don't have the right label and only the wrong label, I would suggest going through and hand labeling those if possible.  If this is not feasible you could explore some variation of label smoothing but I'm not sure this will work and is more of a wild idea to give "a slightly stronger negative label"  to the known negatives (example 0.1) vs. the other classes for which the label is unknown (example 0.15).  This is a wild idea but is the best thing I can think of to take advantage of this "negative information".  P.S. I haven't done a literature search for this, it is quite possible someone has tried this

Does anyone have an example of using a hugging face model that is fine tuned on a dataset and adding a custom head for a classification task, just using Hugging face?

Yes, alternatively to simple-transformers you should be able to use huggingface/transformers to do this reasonably easily. We recently released a Trainer that lets you train or fine-tune model on different tasks with a common API (documentation is still WIP) https://github.com/huggingface/transformers/tree/master/examples

I've run fine-tune training on the full vscode issues dataset, and quickly looking through the results it seems like the vocabulary of the model doesnt get expanded to include works from the fine-tuning dataset? For instance, vscode and debug dont show up in the outputted vocab.txt file. Is this expected? It seems to me that that's leaving a lot of performance on the table.

The vocabulary of the model doesn't get expanded through fine-tuning the language model. The BERT vocabulary has unused tokens (~1000 for bert-base-uncased) which you can replace with your own tokens. You can add new tokens to the vocabulary by editing the vocab.txt file and replacing the unused tokens (will look like [unusedXXX]).
That said, my understanding is that this usually doesn't significantly improve the performance of the model. I think it is because the model can "learn" the meaning of words like vscode even if it tokenized into smaller parts. Since BERT's representations are context-dependant, it probably doesn't matter too much whether vscode appears as a single token or as several tokens next to each other.
I think it's better to consider doing this later on if/when you are looking for any possible improvement.
It's probably best to take the model you fine-tuned and try it out on the actual classification. You can load the fine-tuned model as a classifier with the snippet below.
from simpletransformers.classifcation import ClassificationModel
import logging

logging.basicConfig(level=logging.INFO)
transformers_logger = logging.getLogger("transformers")
transformers_logger.setLevel(logging.WARNING)

train_args = {
    "reprocess_input_data": True,
    "overwrite_output_dir": True,
}

# Using the file path from your snippet
model = ClassificationModel("bert", "finetuned", args=train_args)

Added a classifier using bert fine-tuned on the vscode issue-base. Initial results are promising:
54 issues
  43 triaged  (80%)
    4 incorrectly triaged (8%)
    39 correctly triaged (72%)
  11 skipped (20%)

Implementation at: https://github.com/microsoft/vscode-github-triage-actions/tree/master/classifier-deep.


Bumped development RELEASE for 1.22-9

Bumped development RELEASE for 1.22-9
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold


[ENHANCEMENT] Correct keyboard binding

Hi,
I was reading the README and saw you have an issue regarding the keyboard binding on OSX where JavaFX relies on the 'QWERTY' map. I had the exactly same problem I solved in SlideshowFX so I point you out my solution if you want.
I created the KeyEventUtils class where you can find an example of usage here.
Hope it helps.
Cheers

Dear @twasyl, thank you for sharing KeyEventUtils class. In latest versions of AsciidocFX, I had developed two helper class to manager KeyBoard events seamlessly.
https://github.com/asciidocfx/AsciidocFX/tree/master/src/main/java/com/kodcu/keyboard
These helpers are now good.