Error when terminating a launch config when debugging multiple configs

Launch "Search and Renderer processes" config in vscode repo
Open quick open to kick off the search process
Terminate the search process attach config
See this error:

@roblourens I can not reproduce when running out of latest source. I simply follow your steps with doing nothing special on top
Can you maybe stop on exception thrown and find a stack trace that is causing that message? It does not look like it is coming from debug land

On latest insiders, I still see it 100% of the time but only after the first run after reloading the window.
The error itself comes from the extension host. This stack:
t.getChildren (extHostTreeViews.ts:103)
e.$getChildren (extHostTreeViews.ts:62)
e.invoke (abstractThreadService.ts:42)
e._invokeHandler (rpcProtocol.ts:109)
e._receiveOneMessage (rpcProtocol.ts:96)
(anonymous) (rpcProtocol.ts:32)
(anonymous) (rpcProtocol.ts:157)
(anonymous) (event.ts:382)
e.fire (event.ts:142)
(anonymous) (ipc.net.ts:81)
emitOne (events.js:96)
emit (events.js:191)
readableAddChunk (_stream_readable.js:178)
Readable.push (_stream_readable.js:136)
onread (net.js:560)

It's not coming from one of my installed extensions

My best guess is that it is coming from the loaded scripts view. Thus assigning to @weinand
@roblourens Can you also reproduce if you hide that view?


HTML leaks out in article preview (no escaping)

When there are special characters (like >) in the text of an article, those characters are not properly escaped when NetNewsWire renders the preview for the article in list view. See the attached screenshot. Note: The screenshot is using the Mac app, but the same happens on iOS (both iPad and iPhone).

Platform: Mac OS Sonoma 14.1.1
NNW Version: 6.1.4

What’s the URL of the feed?

What’s the URL of the feed?

Oh! That's a good question because it's a Feedbin newsletter subscription feed. So maybe the bug is in the Feedbin wiring.
Anyway, since it's a subscription feed, I probably shouldn't share the exact URL, but it has the form:
http://axios.com?41character0alpha0numeric0code0123456789


Fix links for Hyperledger Internship Program

Please post your PR's on https://github.com/deepanshu1422/List-Of-Open-Source-Internships-Programs as this current repository is not merging any pull request since November 2019

Please post your PR's on https://github.com/deepanshu1422/List-Of-Open-Source-Internships-Programs as this current repository is not merging any pull request since November 2019

Instead of creating a new clone github project I suggest you to merge these PRs in this very repo only since this is well shared project and numerous people have bookmarked the same.
I'm sure @tapaswenipathak will grant the required permissions to you if not done already.


Bug in authentication tag computation

There's a bug in the way the authentication tag is computed, running all tokens issued by the library undecipherable by other libraries, since they will always fail to check the token authentication if there's authenticated data (note that for compact serialization, the header should always be authenticated).
When computing the authentication tag, first we need to concatenate the data authenticated, the IV, the ciphertext and the length of the authenticated data in 64 bits big-endian. This is the input to the HMAC algorithm that yields the authentication tag. However, the values are concatenated in the _jwe_hash_str() method using a period between them, which is against RFC7518.
This PR addresses the problem by removing the period, using just an empty string as glue for the concatenation. In order to keep backwards compatibility and still be able to decrypt old tokens, the legacy mode of _jwe_hash_str() is kept as is. The temporary version number has been increased to 2, so that all new tokens are encrypted with the correct authentication tag, while all the tokens issued before this change can still be identified and decrypted in legacy mode.

You are absolutely right Demian! I'll provide a fix for this tomorrow.

Cheers for the fix @jaimeperez, LGTM.
That said, I'm no longer actively maintaining this package. Perhaps the good folks at Demonware will still look at and integrate this patch. @Newky?

Hi @coldeasy!
Thanks for taking a look into this and going through my PR!
I don't want to hurry you to accept this PR, but please consider this is a critical issue that makes all tokens ciphered with this library non-compliant with the standard. The library is therefore useless to cipher tokens today, and the more time we take to fix it, the more people might be using it, creating tokens that are potential sources of trouble.
Thanks again!

@jaimeperez yip, my comments were mainly minor - non blocking. We will look at merging this as soon as possible.

Thanks a lot! I'll try to fix at least the defaults-to-none :smiley:

@jaimeperez Thanks very much for your contribution.. and sorry again for the delay in getting this put through. We have been a little oversubscribed recently.
Thanks again!

No problem at all, thanks for merging and the 3.0 release!

Just a heads up in case it causes confusion. My PR was labeled 3.0 release, but its actually 1.0.0, sorry about that :)


Portfolio Finish Mobile version

This is the Personal Portfolio Finish Mobile version feature which includes

works section built utilizing CSS Grid to place elements on the works section.
about section Flexbox was used to place elements in the About section.

Hello @Isaac-ej
This is an additional review as a result of the second opinion request.

In all reviews' end of the summaries, this text includes Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise. This makes the review process easy. Please don't open new pull requests or new repos for your future project.

[ ] Please submit your pull request next review will review it.

[ ] Next code reviewer please re-review this pull request according to the this pull request.

Attention to reviewers
@shahadat3669 and @stanleySimeon: If you can't see the previous review, please don't make it invalid. It is not an invalidation reason. You have two options in such cases:
1- Search for the first review in the closed pull request, which can be found here. If you locate the previous review, you can do a re-review.
2- If you are unable to find the first review, please you are free to review it as a first review.
Thank you for your attention and dedication to conducting thorough reviews.
Cheers and Happy coding! 💻 👏

GitHub | LinkedIn | Twitter
Reviewed by Ahmet Bozacı


Hacky prototype for integrating into View

There's some bugs and my code is very prototype-ish, but this points to how I think ScrollBars should be built into View.
I had to disable ShowScrollIndicator etc as it was confusing things and I don't understand what it's supposed to be doing.
Hope you find this useful!

There's some bugs and my code is very prototype-ish, but this points to how I think ScrollBars should be built into View.

Thanks.

I had to disable ShowScrollIndicator etc as it was confusing things and I don't understand what it's supposed to be doing.

You may not agree with it. But I already explained that this property if a flag to know if the scroll bar visibility is actually invisible caused by the auto-hide or if the user actually hide it by set ShowScrollIndicator to false. If the scroll has the Visible as false and the AutoHide is false, how you restore the original scroll visibility? In this case if the user has set at initialization ShowScrollIndicator as true, the Visible will be also set to true by the AdjustAll method. I hope you understand the what this property is doing. Now, if you don't agree with the feature then it's another subject and if you really want to removed it I don't bother.

Hope you find this useful!

I'll see now. Thanks.

@tig the scroll bar is instable. Doesn't scroll through to the end. I really don't like it. I like the View.ScrollBars.cs implementation.


Maintaining the project

Hiya @johnsensible,
I found myself using your project in multiple projects and always enjoyed it. As far as I can see (issues, pull requests, stars, etc.) it's still quite popular but from the looks of it, also a bit abandoned. I'd hate to see the project die and would like to offer my help maintaining it, mainly sorting out current issues and pull requests, updating the compatibility/tests to work with new Python/Django versions, etc.
Feel free to have a look at my GitHub profile, where I have a couple of projects with a small userbases that I support. Nothing with the popularity of django-sendfile (in terms of stargazers that is ; ) but I am experienced with the 'worklfow' of an open source project, PyPi releases, etc.
Let me know what you think or if you have any questions.

@johnsensible please pass commit privs to that guy if you don't mind, I'm also interested in this package.

By the way, @pawelad, check out my branch and my Tox/TravisCI setup. What I am planning to do is to include all the py3k-related patches from current pull requests and make the package support both Python versions and have a decent build system. How about we work on my branch together, I'll give you privs. After some time, if author of original django-sendfile is still silent, we'll fork. What do you say?

I'd like to find an up-to-date version of this repository as well. I checked yours @mpasternak but I had to debug a fair bit considering that the function now has to be imported from sendfile.core.
Would someone dare setting-up a fork (documented), until this repo is updated?

We want to use this in Zulip (https://github.com/zulip/zulip/) and have been blocking on it having a Python 3 support merged for like a year.  I think it's high time to set up a semi-official fork.  I just forked it to https://github.com/zulip/django-sendfile in the Zulip organization and invited you folks to join as write-access collaborators to help things started, but I have no time at all in the next 2 weeks.
@johnsensible we'd be happy to shut down our fork if you want to start maintaining this again :).
I don't have a ton of time personally, but some other folks from the Zulip project might soon.  I encourage you folks to review and merge PRs there without blocking on me; @adnrs96 is the person in the Zulip project with the most expertise on this.  Let's try to move somewhat cautiously, starting with adding testing infrastructure and then Python 3 support, and having everything be reviewed by someone else from this group.
We'd still need @johnsensible's permission to get the pypi package transferred so that one can do releases, or eventually rename the fork, but we can deal with that once we have something we want to release.  Until then, requirements.txt supports installing from Git tags.

@timabbott I think a lot of the work you need has been done by @mpasternak. I have been using it with Python 3. There was a bit of trial and error due to some package structure difference, and perhaps missing dependencies, but all in all it seemed to work. It may be perhaps easier for your team to review the latter instead.

I was actually imagining we'd just grab those changes until you mentioned having to debug a decent amount, so if you're happy with those changes, we can start from there.
I'm also happy to just follow his fork if other folks want to go with that.  I just want to get things unstuck here :)

I would really enjoy if all this work was based on top of PR#63 where I tried to keep changes minimal to get to Travis CI integration. PR#66 looks good but it looks like changes are more extensive and it should really strip on the tested Django versions to those actually supported upstream.

Is the zulip fork published on PyPI (with an explicit name such as django-sendfile-zulip for the time being)?  That seems like the one missing step here.  Friendly forks can remove pressure on the original author, and other solutions can be done later (merge back, change maintainers, make fork independent).

@merwok not yet, but you can at least now use the GitHub URL in pypi I suppose; I unfortunately got personally a bit swamped just after finding the time to do the fork.  @EvaSDK we'll evaluate the options and consider doing it the other way.
Pinging @umairwaheed and @adnrs96  from the Zulip project to get their opinions on this, since they're more likely to have time to help.  Can you take a look at #63 and let me know whether you think we should base off that instead?

(I should also note that I'm pretty happy to let other people step up for any and all parts of managing this if someone has the time to commit right now; I just want something to be happening here!)

I've created a fork of this project and uploaded it to pypi: https://pypi.python.org/pypi/django-sendfile2
I've fixed a number of issues with Python 3 and added support for newer versions of Django (from 1.8 to 2.0). Feel free to open bugs and submit PRs for anything I've missed: https://github.com/moggers87/django-sendfile2

@moggers87 congrats, thanks for stepping up, I'm using your project now.
@johnsensible thank you very much for the original project and time taken to maintain it.


Remove internal::RawClient::RestoreResumableSession

This function is not used in the client library. Because some applications may have mocked the function (even though it is not called), the function will remain in the library until 2022-10-01, but may be removed after that point.

Waiting until 2022-10-01.


migrate from glusterfs

Since Heketi is in maintenance mode I need to move to another solution.

moved to ceph on proxmox


wsl missing option "--list"

Environment
Windows build number:
Microsoft Windows [Version 10.0.17763.1039]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

at command prompt, type "wsl --list --all"
Expected behavior

should list all available distributions
Actual behavior

Error message "Invalid command line option: --list"

You forgot the first -.

Okay, in your original screenshot you left off the first -. In your new screenshot, you’re running on 17763 which doesn’t have --list...

This is in no way a terminal or console issue

Ok, then it's a documentation issue. I'm on a fresh Win 10 Azure VM fully updated. In what version was --list deprecated?

wsl --list wasn't deprecated. You're using a version of Windows that predates it even existing. You probably want wslconfig /l on your version of Windows.

I´m using Windows Server 2019 an wsl --list is not working.
I think Microsoft have to update their documentation. your products aren´t working this way. So please invest in quality management.

@ITCTOM Thanks for the feedback, but the place to report something like that would be over at https://github.com/microsoft/wsl 😉


Summary reports feature

Ticket No
https://trello.com/c/NO2iAsS1

Closing this merge request since it doesn't address the requirement


Running the migrondi up command second time output error message and nothing else happens

Describe the bug
Trying to run migrondi up or  migrondi down after the first migrondi up gives always an error message "42P07: relation "migration" already exists" as an output and nothing else happens.
To Reproduce
Steps to reproduce the behavior:

Run command migrondi up
Run command migrondi up or migrondi down

Expected behavior
Both commands (migrondi up and migrondi down) should work without any errors related to 'migrations' table existence still after the first run.
Screenshots

Desktop (please complete the following information):

OS: Windows, Linux (Manjaro)
Database: PostgreSQL
Version: 0.7.0

Additional context
I assume that problem exists inside of the Queries.createMigrationsTable function which is used as a implementation for Queries.ensureMigrationsTable functionality. There seems to be handling for situations when table already exists for SQLite databases, but how about other database types?

This is an interesting situation, I'll try to reproduce later on I did test often in postgresql so it hits me in a weird way.
I'd have a few more questions in regards to this

does this happen with any particular schema
did it happen with an schema that contains "migration" as a table as well?

I ask this because the error looks as if there's something trying to add to the migration table
relation "migration" already exists

There seems to be handling for situations when table already exists for SQLite databases, but how about other database types?

We cover each database individually
in Queries.fs:145 we have the createMigrationsTable function that calls createTableQuery
let private createTableQuery driver =
    match driver with
    | Driver.Sqlite ->
        """
        CREATE TABLE migration(
            id INTEGER PRIMARY KEY AUTOINCREMENT,
            name VARCHAR(255) NOT NULL,
            timestamp BIGINT NOT NULL
        );
        """
    | Driver.Postgresql ->
        """
        CREATE TABLE migration(
            id SERIAL PRIMARY KEY,
            name VARCHAR(255) NOT NULL,
            timestamp BIGINT NOT NULL
        );
        """
    | Driver.Mysql ->
        """
        CREATE TABLE migration(
            id INT AUTO_INCREMENT PRIMARY KEY,
            name VARCHAR(255) NOT NULL,
            timestamp BIGINT NOT NULL
        );
        """
    | Driver.Mssql ->
        """
        CREATE TABLE dbo.migration(
            id INT PRIMARY KEY,
            name VARCHAR(255) NOT NULL,
            timestamp BIGINT NOT NULL
        );
        """

I guess I could modify those to include a if not exists clause and even maybe use a not so common name __migondi__migrations or something like that
In any case I'll report back thanks for raising the issue

This has been fixed in v0.7.1 and is being indexed by nuget, I'll publish a GH release soon as well, feel free to re-open if for some reason it doesn't work


Does LEGEND allow users to build models which refer to (model) elements outside of the Legend tooling, for example, could a payment model be enriched with a set of payment purpose codes from a service endpoint?

In this case the payment purpose code would be modelled as an Enumeration, the full set of Enumeration values must be defined within the user’s Legend project, however these values may be mapped to different store values via the user defined model to store mapping.


Selecting Unbound Keybind Triggers All Unbound Keybinds

Minecraft 1.16.4
Forge 35.1.37
MineMenu 1.8.7
Selecting an unbound keybind currently means that every time that menu entry is selected, ALL unbound keybindings are activated at once.
There's nothing in the logs to show, and no crash occurs, however, if you still wish for me to post the logs let me know!

P.S. Unrelated but does MineMenu support some of the more 'out there' MC-supported keybindings such as F13-F25?

P.S. Unrelated but does MineMenu support some of the more 'out there' MC-supported keybindings such as F13-F25?

Nothing that can be done about that, that's just how vanilla treats it.

Nothing that can be done about that, that's just how vanilla treats it.

That's a shame, but I understand!
Just thinking outside the box real quick - is there no way to direct the keypress event to only the specific mod being targeted? I'm not a Java / FML expert, I'm not sure what the limitations are...

That's a shame, but I understand!
Just thinking outside the box real quick - is there no way to direct the keypress event to only the specific mod being targeted? I'm not a Java / FML expert, I'm not sure what the limitations are...


VSImporter Bundle projects don't build with VS2017 RC.

1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppBuild.targets(358,5): error MSB8016: This project does not support the current Configuration Type (Utility).
as an example failure. This project needs re-thought.

Merged into packaging branch.
Bundle projects still need more thought and a redesign.


Improve dashboard design

Pattern we are reusing: https://design-patterns.service.justice.gov.uk/patterns/case-list/
Design on Miro: https://miro.com/app/board/uXjVObfC1o8=/?moveToWidget=3458764525215411685&cot=10

Closed as incorporated in https://github.com/ministryofjustice/laa-crime-apply-prototype/pull/115


Local dependencies- ~ and $HOME not recognized

This isn't a feature request as much as a question (didn't want to file as a bug)- how do I mention a dependency that depends on a local path relative to $HOME?
I've tried both:
[dependencies]
debug = { path = "$HOME/workspace/rust/debug/" }

and
[dependencies]
debug = { path = "~/workspace/rust/debug/" }

but cargo complains in both cases. Is there anything else I could do?

It's currently not possible, cargo doesn't do any sort of interpretation of the path.  These paths are typically used as relative to the current project.  For absolute paths, it is intended to use the full path since it will be specific to the machine it is running on.


Checkout Commit 1

New Feature "Checkout Commit"; Part 1
Related to #3040 - Revert to any single commit
Change Log
zcl_abapgit_git_porcelain

Methods pull and push consider retrieving git objects by any commit hash, not only the branches one

zcl_abapgit_git_transport

Executed Pretty Printer
Added some attributes and related logic to upload_pack to be able to retrieve commits not only by branch commit and be able to specify the deepen level for the commit hierarchie to be retrieved

zif_abapgit_persistence

Added field commit_sha1 as representation of attribute commit_sha1 of online repos to be able to use them in method set of class repo

zcl_abapgit_merge

Adjusted call of method upload_pack to new method signature

zcl_abapgit_repo

Made get_local_checksums public
Now it's possible to set commit_sha1 for repo

zcl_abapgit_repo_online

set_branch_name: renamed iv_branch_name to iv_name
Added getter and setter for commit hash

zif_abapgit_definitions

Added type definition for git commit and a list of the same structure type

Just fyi: Will wait with the second PR til this was successfully reviewed.

hi,
please remove changes in
zif_abapgit_persistence.intf.abap
zcl_abapgit_gui_chunk_lib.clas.abap
zcl_abapgit_merge.clas.abap
zcl_abapgit_repo.clas.abap
zcl_abapgit_repo_online.clas.abap
zif_abapgit_definitions.intf.abap

unless its strictly needed due to refactorings in git_porcelain and git_transport
I cannot keep track of it all at once

Oops..yes, didn't wanted that - sry.

Most of them are already related to each other:

In zif_abapgit_persistence the data types for the repo attribute setter mask is being defined
In zcl_abapgit_repo and zcl_abapgit_repo_online is the definition/implementation for the setter and getter of the commit hash
In zcl_abapgit_merge I had to adjust the call of zcl_abapgit_git_transport=>upload_pack
In zif_abapgit_definitions I added some definitions for commits for zcl_abapgit_git_transport

I was able to remove the changes in zcl_abapgit_gui_chunk_lib though and removed the changes related to beautification in zif_abapgit_definitions.

@larshp Is there anything else I could do or are you still reviewing? Just asking, not trying to put you under pressure :)

So, first method changed is pull, instead of adding a parameter to the method, suggest making a new method pull_by_commit, perhaps rename pull to pull_by_branch but this can be done in the future in a follow up PR.
Method push, is this needed for "Checkout Commit" ?

Method push, is this needed for "Checkout Commit" ?

Yes, to get the full list of objects to be pushed back to the Git Platform I have to determine the objects by the commit and not branch hash. So I separated these different fields and due to that full_tree is being called with the commit hash in push.

okay, so pushing is in scope for this PR? Please help updating the description

Updated description accordingly to your question.

okay, so I suggest closing this PR and opening a new with the sole purpose to
1: rename pull to pull_by_branch
2: create new method pull_by_commit

How should we proceed with push and the other changes?

in another incremental change after the first change

But first we would have to change zcl_abapgit_git_transport=>upload_pack. Should I open a new PR for this too?

yes please

closing


Leaves blocking gray overlay

Leaves blocking gray overlay on https://www.real.de

submitted via google forms


Dynamics-1.1.1 merge regression-1.1

Coverage decreased (-0.1%) to 38.406% when pulling 1f60cc2b6cd3c220c86045dc994da2b3cae93c45 on alvita:dynamics-1.1.1 into af805c4cbf8e8674745149e54ec4fc19a4f73b03 on ringcentral:dynamics-1.1.1.


Update RHEL7 Documentation

The command sudo service postgresql initdb doesn't work within CentOS 7.1, you receive the following error
The service command supports only basic LSB actions (start, stop, restart, try-restart, reload, force-reload, status). For other actions, please try to use systemctl.
I have updated this to /usr/pgsql-9.4/bin/postgresql94-setup initdb

Thanks @kjkeane for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

I have completed the license agreement as procedure, but there are few more changes I have made.  This came up because I currently attempting to migrate Mattermost from Ubuntu 16.04 to CentOS 7.
Changes

Update .conf locations
Update service name when using systemctl
Update sudo command to from sudo --login --user postgres to sudo su postgres

Hi @kjkeane thanks!
Are these changes strictly for CentOS, or do they apply to RHEL too?

So far only CentOS since I do not have RHEL image to configure, but from my experience they are pretty similar.  I am going to see if I can do this in a RHEL evaluation image.

Hey @JeffSchering I was able to confirm this morning that they are the same.  I used an evaluation ISO (RHEL 7.3) I got from Red Hat, but I had to use curl instead of wget to download the rpm for Postgres, because wget is not installed by default in CentOS or RHEL.

Thanks again @kjkeane! Would you mind also updating the download step to use curl instead of wget?

@JeffSchering No problem, I have updated wget to curl -O in the documentation.

Big thanks @kjkeane and @JeffSchering!


🐛 BUG: kv-binding goes wrong?

What version of Wrangler are you using?
2.0.3
What operating system are you using?
Windows 10
Describe the Bug
kv-binding goes wrong?
i've create a kv and bind it in wrangler.toml:
kv_namespaces = [
  { binding = "COMMENTS", id = <my key> }
]

(i can confirm it on Cloudflare dashboard.)
but when i call COMMENTS, throw a error ReferenceError: COMMENTS is not defined:
index.js:
export default {
  async fetch(request) {

    let value = await COMMENTS.get('aaa');

    // Return the value, as is, for the Response
    return new Response(value)
  },
};

wrangler cli error:
下午1:43:19 GET / 500
✘ [ERROR] Uncaught (in promise) ReferenceError: COMMENTS is not defined

      at Object.fetch (index.js:4:17)

✘ [ERROR] Uncaught (in response) ReferenceError: COMMENTS is not defined

✘ [ERROR] Uncaught (in promise) ReferenceError: COMMENTS is not defined

      at Object.fetch (index.js:4:17)

✘ [ERROR] Uncaught (in response) ReferenceError: COMMENTS is not defined

but if i use the env.COMMENTS everything works fine:
export default {
  async fetch(request, env) {

    let value = await env.COMMENTS.get('aaa');

    // Return the value, as is, for the Response
    return new Response(value)
  },
};

This is as designed. When you write a worker as a service-worker format (i.e. with addEventListener('fetch', () => ), then bindings are available as globals in the Worker. When using the modules format worker (i.e with export default { fetch(request, env) {...) then bindings are available on env. Sorry for the confusion! We should have easily referrable docs on this, and I'll follow up on this internally.
Closing this issue since it works as expected.

https://developers.cloudflare.com/workers/runtime-apis/kv/#referencing-kv-from-durable-objects-and-workers-using-modules-syntax


feat: port noncomm_ring tactic

This aims to be a faithful implementation of mathlib3's noncomm_ring tactic.

bors r+


Updating selectText dynamically for Added Items

Issue summary
Thank you very much for the excellent library.
After the user selects an option, I want (selectText) to reflect the selected option. I managed to get it done successfully for options, which have been defined before hand. However, it does not work if the user types in a new value, which was not in the original items list.
Your help is highly appreciated.
Library versions
expo: 32.0.0
react-native: "https://github.com/expo/react-native/archive/sdk-32.0.0.tar.gz"
react-native-multiple-select: 0.5.3
Reproducible Code
class ShiftRateScreen extends Component {
state = {
	selectText: 'Role Name',
	itemsList: items,
	selectedItems: [],
};

onSelectedItemsChange = (selectedItems) => {
	console.log('onSelectedItemsChange - SelectedItems: ', selectedItems);
	this.setState({
		selectText: selectedItems[0],
		selectedItems: selectedItems,
	});
};

onAddItem = (newItemsList) => {
	console.log('onAddItem - newItemsList: ', newItemsList);
	this.setState({
		selectText: newItemsList[newItemsList.length - 1],
		itemsList: newItemsList,
		selectedItems: newItemsList[newItemsList.length - 1],
	});
}

render() {
	return (
		<View>
			<View>
				<MultiSelect
					items={this.state.itemsList}
					uniqueKey="id"
					ref={(component) => { this.multiSelect = component; }}
					single={true}
					hideSubmitButton={true}
					selectText={this.state.selectText}
					searchInputPlaceholderText="Type or select a Role"
					filterMethod="full"
					canAddItems={true}
					onAddItem={this.onAddItem}
					onSelectedItemsChange={this.onSelectedItemsChange}
				/>
			</View>
			<View>
				{this.multiSelect && this.multiSelect.getSelectedItemsExt(this.state.selectedItems)}
			</View>
		</View>); } }

I figured it out. I was missing the following prop:
<MultiSelect
....
selectedItems={this.state.roleSelectedItems}
....
/>


[WIP] [enhancement] BE support multiple s3 auth info

What type of PR is this：

[ ] bugfix
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

S3ClientFactory can generate client using variant S3 auth info now.
HdfsScanNode add THdfsProperties so that FE can pass different S3 auth info to BE

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

 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.comboqiu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


Implement Get a specific mentors’ details endpoint (Admin)

Description:
This issue involves implementing an API to get all mentors. The endpoint should allow Admin to make a GET request to {{baseUrl}}/api/admin/mentors/{{mentorId}}.
{
    "mentor_id": 1,
    "created_at": "2023-06-30T10:00:00Z",
    "updated_at": "2023-06-30T14:30:00Z",
    "state": "active",
    "category": "Technology",
    "application": {
        "field1": "value1",
        "field2": "value2"
    },
    "availability": true,
    "profile": {
        "created_at": "2023-06-29T09:15:00Z",
        "updated_at": "2023-06-30T16:45:00Z",
        "primary_email"<EMAIL_ADDRESS>        "contact_email"<EMAIL_ADDRESS>        "first_name": "John",
        "last_name": "Doe",
        "image_url": "https://example.com/mentor_profile_image.jpg",
        "linkedin_url": "https://www.linkedin.com/in/johndoe",
        "type": "DEFAULT",
        "uuid": "12345678-1234-5678-1234-567812345678"
    }
}

Tasks:

Create a controller for the /controllers/mentor endpoint in the backend (create the route mentor if not created).
Parse and validate the request body to ensure it matches the expected format.
Implement appropriate error handling and response status codes for different scenarios (e.g., validation errors, database errors).
Write unit tests to validate the functionality and correctness of the endpoint.

API documentation: https://documenter.getpostman.com/view/27421496/2s93m1a4ac#8744a3ee-970f-489a-853d-8b23fdee8de3
ER diagram: https://drive.google.com/file/d/11KMgdNu2mSAm0Ner8UsSPQpZJS8QNqYc/view
Acceptance Criteria:

The profile API endpoint /controllers/mentor is implemented via a GET request.
The request body is properly parsed and validated for the required format.
Appropriate error handling is implemented, providing meaningful error messages and correct response status codes.
Unit tests are written to validate the functionality and correctness of the endpoint.

Additional Information:
No
Related Dependencies or References:
No

@janithlahirukariyawasam @anjula-sack
Please assign this issue to me!

@janithlahirukariyawasam @anjula-sack Please assign this issue to me!

Go ahead @subhashree9898

@subhashree9898 Are you working on this?

@anjula-sack I'm really sorry for the late reply. Please give me 2 days.

Implemented with https://github.com/sef-global/scholarx-backend/pull/111


A proposal on how styling could (should?) be made.

Style component with css, not html attributes (where not needed), improved mobile view, removed deprecated css, moved data to .ts file to keep view definition not that tightly coupled with data.

Thanks Bartek! I will try to buy some time and review your PR ASAP, as I am on holidays until Wednesday.

No problem. I was hoping to catch you today to agree on some other things, but enjoy your holiday! I will try to pick a way on my own and then let you decide if it was implemented as you would want it :P


there is no documentation on conditionals that I can find

this json works fine in the editor but there are no examples of it in the documentation; or anywhere else that I can find. I know this is not a bug per say, But I would be glad to add documentation if I could figure this one out...
`{
"$schema": "https://vega.github.io/schema/vega-lite/v5.json",
"data": {
"name": "data",
"values": [
{
"agent": "bart",
"avg_score": "3.0000000000000000",
"call_count": 5,
"label": "Answer Time",
"max_score": 5,
"percentage": "0.60000000000000000000",
"score_average": "3.0000000000000000",
"score_calc": "0.00",
"score_desc": "agent % from avg",
"score_percentage": "0.60000000000000000000"
},
{
"agent": "mabel",
"avg_score": "3.0000000000000000",
"call_count": 5,
"label": "Answer Time",
"max_score": 5,
"percentage": "0.60000000000000000000",
"score_average": "3.0000000000000000",
"score_calc": "0.00",
"score_desc": "agent % from avg",
"score_percentage": "0.60000000000000000000"
},
{
"agent": "bart",
"avg_score": "5.8000000000000000",
"call_count": 5,
"label": "Negative Words",
"max_score": 10,
"percentage": "0.58000000000000000000",
"score_average": "5.8000000000000000",
"score_calc": "0.00",
"score_desc": "agent % from avg",
"score_percentage": "0.58000000000000000000"
},
{
"agent": "mabel",
"avg_score": "5.8000000000000000",
"call_count": 5,
"label": "Negative Words",
"max_score": 10,
"percentage": "0.58000000000000000000",
"score_average": "5.8000000000000000",
"score_calc": "0.00",
"score_desc": "agent % from avg",
"score_percentage": "0.58000000000000000000"
}
]
},
"description": "",
"encoding": {
"y": {
"axis": {
"labels": true,
"tickBand": "center",
"tickOpacity": 0,
"title": ""
},
"field": "agent",
"type": "nominal"
}
},
"height": {"step": 16},
"layer": [
{
"encoding": {
"tooltip": [
{"field": "AgentCalls", "title": "calls", "type": "quantitative"},
{"field": "AvgAgtScore", "title": "avg score", "type": "quantitative"}
],
"x": {
"aggregate": "average",
"axis": null,
"field": "percentage",
"scale": {"domain": [0, 1]}
}
},
"mark": {"color": "#ddd", "tooltip": true, "type": "bar"},
"transform": [
{
"groupby": ["agent"],
"joinaggregate": [
{"as": "AvgAgtScore", "field": "avg_score", "op": "average"}
]
},
{
"groupby": ["agent"],
"joinaggregate": [
{"as": "AgentCalls", "field": "call_count", "op": "average"}
]
}
]
},
{
"encoding": {"text": {"field": "AgentPercentage", "format": ".2%"}},
"mark": {
"align": "left",
"tooltip": "the agent percentage",
"type": "text",
"x": 5
},
"transform": [
{
"groupby": ["agent"],
"joinaggregate": [
{"as": "AgentPercentage", "field": "percentage", "op": "average"}
]
}
]
},
{
"encoding": {
"color": {
"condition": {"test": "datum['ScorePercentageCalc'] < 0 ", "value": "Red"},
"value": "Green"
},
    "text": {
      "field": "ScorePercentageCalc",
      "format": ".2%",
      "type": "quantitative"

    },
    "tooltip": [
      {"field": "score_desc", "title": "description"},
      {
        "field": "ScorePercentage",
        "format": ".2%",
        "title": "avg for all scores",
        "type": "quantitative"
      }
    ]
  },
  "mark": {
    "align": "left",
    "baseline": "middle",
    "dx": 24,
    "fontWeight": "bold",
    "tooltip": "test",
    "type": "text"
  },

  "transform": [
    {
      "joinaggregate": [
        {
          "as": "ScorePercentageCalc",
          "field": "score_calc",
          "op": "average"
        }
      ]
    },
    {
      "joinaggregate": [
        {
          "as": "ScorePercentage",
          "field": "score_percentage",
          "op": "average"
        }
      ]
    }
  ]
}

],
"title": "Agent  Scores",
"width": 200
}`

In most cases we transform options from snake case to camel case and everything from Vega-Lite is supported. The idea is that mirroring examples from Vega-Lite docs should be fairly automatic, in this case you could do:
|> Vl.encode(:color,
  value: "green",
  condition: [text: "datum['ScorePercentageCalc'] < 0", value: "red"]
)

It's not practical to cover all examples from Vega-Lite docs, but a PR to the docs is welcome :)

This case is different from most encode I have used so far… an example might have been good but that being said, it seems obvious now…
😊
From: Jonatan Kłosko @.>
Sent: Tuesday, May 2, 2023 4:30 AM
To: livebook-dev/vega_lite @.>
Cc: Joe Waldner @.>; Author @.>
Subject: Re: [External] [livebook-dev/vega_lite] there is no documentation on conditionals that I can find (Issue #45)
In most cases we transform options from snake case to camel case and everything from Vega-Lite is supported. The idea is that mirroring examples from Vega-Lite docs should be fairly automatic, in this case you could do:
|> Vl.encode(:color,
value: "green",
condition: [text: "datum['ScorePercentageCalc'] < 0", value: "red"]
)
It's not practical to cover all examples from Vega-Lite docs, but a PR to the docs is welcome :)
—
Reply to this email directly, view it on GitHubhttps://github.com/livebook-dev/vega_lite/issues/45#issuecomment-1531163470, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3JAA3E264G5LOYVWAJ3GYLXEDHY5ANCNFSM6AAAAAAXSGGLFY.
You are receiving this because you authored the thread.Message ID: @.***>

just one tiny change; its 'test' not 'text' :-)


fix badge, bump support to py3.9

Codecov Report

Merging #38 (dd2688c) into main (6d7aecc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   48.71%   48.71%
=======================================
  Files          11       11
  Lines         273      273
  Branches       44       44
=======================================
  Hits          133      133
  Misses        140      140

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 6d7aecc...dd2688c. Read the comment docs.


Add link to FlatDoc

Add link to FlatDoc documentation generator

Why should I add it? I might be missing the point, but it seems like it only render the readme, which is already pretty readable on GitHub.

It uses README but gives structured way for documentation by providing good navigation bar on the left,  description and details in center and code examples on right panels and everything on single page, which I think makes developer life easier.

I don't really see the point tbh, but I'm going to pass on it for the reason that it's a front-end thing and doesn't use Node in any way which is what this list is about.


Menu breaks workbench if window is too short

This can corrupt the workbench position after the menu is closed:

tracked in #52533


Style views to look like designs

Noga has provided us with designs (to be uploaded). Now we will style the views to look like our new designs.

We have used the T4h estimate, but we believe it should take two hours

This took us 6 hours (including video editing & hosting)


Resizing in VST3 not working

Hi,
I'm experimenting with plugin-things and having a lot of fun with it!
It seems like window resizing is not working, at least for VST3 on Windows. I'm pretty sure because the onSize callback isn't doing anything. See:
https://github.com/ilmai/plugin-things/blob/13fa1e501dcd0e9cb53348c3ba8d42ba9f535835/plinth-plugin/src/formats/vst3/view.rs#L140
Are you aware of this?
Are you open to contributions? Then I can probably fix this (and perhaps other things) in a PR.

Hey, thanks for the compliment!
You're correct that resizing is not yet implemented for VST3, feel free to submit PRs for this or other bugs or missing features. If you're going to submit something that requires extensive changes, especially if it changes interfaces, it would be best to create an issue first so we can discuss the details.

Cool! I don't think resizing is complex to implement. I'll try something over the weekend!

Quick update; I got a dirty proof of concept of resizing the plugin window working on Windows. It didn't require any refactoring or other big things. I will test this on macOS in the coming days. I do not have a good Linux VST3 setup where I can test, but I own Bitwig which I can install in a Linux VM to test.
Probably will have to do resizes on CLAP and maybe AudioUnit too (saw there was some code in the AU format, but haven't seen it yet).

Linux/X11 too :-) Only one issue with Bitwig that is a known bug with Bitwig, see https://github.com/juce-framework/JUCE/commit/94330cbb533442818dce4e84100863f6574da0ce
For now I'm not going to solve it, but it might be trivial in the future to force a re-layout or something like JUCE is doing. All other DAW's I've tested on Linux have working resizing windows now (REAPER, Renoise, Ardour/Mixbus, Waveform).

I have opened a draft PR for this, see #3
It's still WIP, and the above stuff is still open for discussion.

Thanks for the effort, I've been super busy with other stuff but I'll take a look when it's a bit less hectic around here


Game won't Launch/run

Describe the bug
just stuck on the launcher/running page and the game won't even load I INSTALLED AND DID EVERYTHING ALREADY but it just won't launch-
Screenshots

It just stuck like this- I let it load for A WHOLE DAY but the game won't launch/run-
System Information (please complete the following information):

macOS: version 14.1.1
Device: MacBook Pro 13-inch M2 2022
CPU: Apple M2
RAM: 8GB

Additional context
I've already done all the other steps this last step is what's the problem- it just wont launch/run- pls help😭

same lmk if you figure it out

It doesn't work on arm yet, it's written on the release page.


Add option to getLegendData to return colors in a different format than RGBA

Context
When using getLegendData method, for instance, in a Ramp in order to assign colors to values, you get the color in RGBA format. We've discussed in the past that this is the best way to return colors. However, in almost all of the examples where we're using this method, we have the following function to convert the color from RGBA to HEX because it's easier for us when using it in the HTML
function rgbToHex(color) {
    return "#" + ((1 << 24) + (color.r << 16) + (color.g << 8) + color.b).toString(16).slice(1);
}

Since getLegendData can receive options, such as the number the label for the 'others' values, it'd be great to have another one that let you change the color format.
What do you think?

👍 Nice idea!
If we're already going to provide options  to that method, then I would for it. I found that piece of code in the examples quite obscure for the end user. We could add something like:
{
   colorFormat: 'hex' /* keeping 'rgba' as default, to keep compatibility */
}

Right now we have some examples using rgba without that ugly function and I think that it can be enough, should we close this one @elenatorro ?

I'd prefer to keep it since it's not related only with the examples but in other use cases. I wouldn't close it, I'd just give it a low priority or just label it as an enhancement.


Add scales to the vis when laying out by numeric attributes

Show the scale for x and y. Add some padding to the vis to put the scale in and to give it some breathing room.

This is done


refactor: removed Ogmios schema package dependency

Context
We must removed dependency on the Ogmios schema package and have our own types in the core package so we can have more typescript-friendly types.
Proposed Solution
Port over the required types from the Ogmios schema package and do the necessary adjustments.
Important Changes Introduced
Dependency with @ogmios/schema removed.

Some of the properties being typed as bigint caught my attention this time around. I now recall the reason Ogmios WSP schema has them as UInt64, however, it's not specified to be bigint in the generated TypeScript definition.
https://github.com/CardanoSolutions/ogmios/blob/c45d1a1a65d7f2fa35a0d60be25fe2baf0f58172/architectural-decisions/accepted/013-typescript-client_bigint-parsing.md
CardanoSolutions/ogmios@0a91b15
https://github.com/CardanoSolutions/ogmios/blob/d16e4ede0d40d0b4d40f2fb47f265513942d845f/clients/TypeScript/packages/client/src/util.ts#L27-L67
Given Ogmios has a valid understanding of legitimate value needing to be bigint, I think the simplest approach is to revert any changes from number to bigint

The revert on the changes is done :)


AuthenticationError: Update to the newest version of Robinhood to access this feature.

Checklist

[x] I am on the latest pyrh version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.

Description
Crashes when running from pyrh import Robinhood rh = Robinhood(username="email", password="password")
This error shows up when you login with correct credentials, however it doesn't ask 2FA.
Steps/Code to Reproduce
Run the program with successful credentials
A different error will pop up if you provide the wrong credential as usual saying that the password or email is incorrect, meaning the connection between the api and robinhood still works.
Results
  File "/home/runner/HeftyRedundantElement/main.py", line 4, in <module>     rh.login()   File "/home/runner/HeftyRedundantElement/.pythonlibs/lib/python3.10/site-packages/pyrh/models/sessionmanager.py", line 162, in login     self._login_oauth2()   File "/home/runner/HeftyRedundantElement/.pythonlibs/lib/python3.10/site-packages/pyrh/models/sessionmanager.py", line 453, in _login_oauth2     raise AuthenticationError(msg) pyrh.exceptions.AuthenticationError: Update to the newest version of Robinhood to access this feature.
Versions
Linux-6.2.16-x86_64-with-glibc2.39
Python 3.10.14 (main, Mar 19 2024, 21:46:16) [GCC 13.3.0]
pyrh 2.1.2

I took a look at what requests the website front end was sending, and saw they're currently sending this version header:
X-Robinhood-API-Version: 1.431.4
whereas pyrh is sending version 1.0.0 by default.
https://github.com/robinhood-unofficial/pyrh/blob/e301e8018abc1f8afe5d7aeaebdfa188783a4772/pyrh/models/sessionmanager.py#L42
Setting headers = {"X-Robinhood-API-Version": "1.431.4"} and passing that into the constructor fixed it for me, but I fully expect this to crop up again at some point.

Hey @cejohnson, how are you setting the header? Thanks!

You can pass them in as a kwarg to the constructor:
headers = {"X-Robinhood-API-Version": "1.431.4"}
rh = Robinhood(username=args["username"], password=args["password"], mfa=args["mfa"], headers=headers)
rh.login()

I assumed the headers passed in here would be merged with the default headers, and the SessionManager docstring seems to support that, but I'm not seeing this actually happen in the code so it may be a full replacement. However, the above is working for me.

You can pass them in as a kwarg to the constructor:
headers = {"X-Robinhood-API-Version": "1.431.4"}
rh = Robinhood(username=args["username"], password=args["password"], mfa=args["mfa"], headers=headers)
rh.login()

I assumed the headers passed in here would be merged with the default headers, and the SessionManager docstring seems to support that, but I'm not seeing this actually happen in the code so it may be a full replacement. However, the above is working for me.

https://github.com/robinhood-unofficial/pyrh/blob/e301e8018abc1f8afe5d7aeaebdfa188783a4772/pyrh/models/sessionmanager.py#L105
Actually, it replaces--if you put up a PR. I can cut a release. We may actually want a better solution here, where users can change the API version on the fly.

PR created.
I thought about adding an api_version kwarg to the __init__ but figured that could introduce ambiguity and complexity around whether that argument or headers took precedence... so instead I just bumped the version to keep things simple.

@cejohnson
I am still getting the original error after doing the following changes:
from pyrh.models.sessionmanager import HEADERS
HEADERS["X-Robinhood-API-Version"] = "1.143.4" # as of today [2024-12-05] this is the version

sm = SessionManager(
    username=username,
    password=password.get_secret_value(),
    mfa=secret.get_secret_value(),
    headers=HEADERS
)

sm.login()

Error:
AuthenticationError: Update to the newest version of Robinhood to access this feature.

Yeah, it stopped working for me in the last day or two as well. It looks like they're now rejecting the default user-agent with the "update" error, but if I change that I get:
pyrh.exceptions.AuthenticationError: Unknown login error
which really isn't an improvement. I don't see any obvious differences from the flow in the browser vs what the code seems to be doing, but apparently something changed somewhere. If I have time I'll try to look into it later.

The problem is talked about in another RH repo... Looks like they solved it.
https://github.com/jmfernandes/robin_stocks/pull/510


Fixing CleanWebpackPlugin path for webpack.json.

- Summary

AssetsPlugin in webpack.common.js is writing to site/data, not site/content.
Fixing CleanWebpackPlugin path for webpack.json accordingly.
- Test plan

npm run start and npm run build building site successfully.
- Description for the changelog

Fixing CleanWebpackPlugin path for webpack.json.
- A picture of a cute animal (not mandatory but encouraged)

Thanks!


dump1090-fa version on i386

The file adsb-receiver/bash/variables.sh has following
FlightAware PiAware
DUMP1090_FA_VERSION="3.5.1"
PIAWARE_VERSION="3.5.3"
This works OK on armh (RPi/OPi).
On Raspbian Stretch x86 on i386 computer the install fails.
On i386, after building dump1090-fa, when script runs install command (dpkg -i ), it fails, saying it did not find the package dump1090-fa ver 3.5.1.
When I checked, found that the build pkg has actually built dump1090-fa ver 3.5.3, and then trying to install ver 3.5.1.
I then manually gave command dpkg -i to install dump1090-fa 3.5.3, and it was successfully installed.

Fixed in commit e2647fa2ad600996581e3467c77b3f19747d3205.


Malloc error?

Hi,
I'm using the reference code from https://platform.stability.ai/docs/features/text-to-image?tab=python and sometimes it does work as intended, but about half of the runs (in a seemingly random fashion), Python crashes with the following error message:
Python(13180,0x1712a7000) malloc: Corruption of free object 0x142738330: msizes 8/7 disagree
Python(13180,0x1712a7000) malloc: *** set a breakpoint in malloc_error_break to debug
The mysterious part is, this happens AFTER the created image has successfully been saved to disk – I get the image, and then the malloc crash. I don't think it's rest of my program, as it works perfectly well if I replace the Stability api call with DALL-E, imagenAIry etc. – it only happens when using stability-sdk. Versions are stability-sdk 0.4.0, Python 3.10.10 (Flask app), Macos 13.13 (M1).
Any ideas anyone?

Update, not a solution, but if someone should have the same problem: I just used the REST API, that works fine and there is even a Python code example: https://platform.stability.ai/rest-api#tag/v1generation/operation/textToImage


Add :prefix to telemetry metadata

Thanks @hogjosh but unfortunately this is not enough.
The prefix can be given as option, set on the query or on the schema, and this change considers only one of the ways.
Furthermore, Ecto 3 allows you to set the prefix in any part of the query, so which one of them would you actually use? I think the best option here is to have a way to pass extra metadata to the Ecto events (or generally find a better way to attach random metadata to telemetry events, potentially using pdict). We will continue building on this as we collect more feedback. :+1:

Ah okay, that makes sense.  I only pass it as an option currently, but knowing it can now be part of the query gives me some things to think about elsewhere.  Thanks!


SQL-76:Pattern matching functions in MySQL

Removed inserts and cleaned the scripts to just use sql queries

@milos-simic, please review and accept


Asuswrt Updating device list from asuswrt took longer than the scheduled scan interval 0:00:20

Home Assistant release (hass --version): 0.79.2
Python release (python3 --version): 3.6.5
Component/platform: asuswrt
Description of problem:
I have an Asus GT-AC5300 running in router mode. The corresponding asuswrt component is configured using normal password access, not telnet or ssh. I am using this to track devices in my network with an interval of 20 seconds.
Problem-relevant configuration.yaml entries and steps to reproduce:
device_tracker:

platform: asuswrt
host: !secret net_host
username: !secret net_username
password: !secret net_password
interval_seconds: 20

I get this from time to time. It's probably more due to network lag, more than an issue with hass


Fix error in pair potential derivative

I was unable to come up with a minimal unit test that reproduces the error Levi was seeing, but I am pretty sure this if-if-else should be an if-elif-else so that exactly one branch is taken when the parameter being differentiated is rmin.

Try to add a test where LJ = WCA + LJ tail, with rmax & rmin set on both


bot.identity.name

the bot passed into the interactive_message event controller does not appear to contain the bot.identity.name attribute... is this normal and if so, how can I get the name?

@Intellogo I can confirm that this is the same as on my computer. Not sure exactly why but basically on interactive message callback a new bot is spawned, so in general it might be missing some things (particularly if you attach custom stuff to the bot when the team is first connected like i do). I suspect that when the first bot is created it gets the identity name attached but for the interactive message callback it doesn't happen. Basically the bot on callback is not the same bot that normally "hears", so you have to be careful with what you do with it. You can see this in SlackBot.js:
                    var bot = slack_botkit.spawn(team);

                    bot.team_info = team;
                    bot.res = res;

                    slack_botkit.trigger('interactive_message_callback', [bot, message]);

                    if (configuration.interactive_replies) {
                        message.type = 'message';
                        slack_botkit.receiveMessage(bot, message);
                    }

Closed as answered


Fixed bug regarding printing level=1, fix #146 - [merged]

In GitLab by @cmastalli on Mar 15, 2019, 15:02
Merges topic/cb-verbose-bug -> devel

In GitLab by @nmansard on Mar 15, 2019, 21:01
thx

In GitLab by @nmansard on Mar 15, 2019, 21:01
merged

In GitLab by @nmansard on Mar 15, 2019, 21:01
mentioned in commit 202c015c9ffa66474a8d0fa00f3a3f685eb75c00


Remove gradle.ext.version property

The gradle.ext.version property is no longer necessary. Let's remove this.
Originally posted by @seeseemelk in https://github.com/MockBukkit/MockBukkit/pull/403#discussion_r887157535

Closed by #424


Db updates

Change description

Limit the initial worksheets page to 30 worksheets as it was taking a while to load (#31)

Includes a button to view all worksheets if needed (still slow to load)
Needs to sort well otherwise its pointless - sorts by run ID instead of worksheet ID as it's more reliable
Added diagnostic boolean field to worksheet model to easily separate from live worksheets and help with sorting
Updated tests to use new URLs

Add a modal before assigning the 1st IGV check that makes sure the referral paperwork is checked (#32)

Only needed for 1st check, wont pop up for any others
Wont allow access unless the form is completed, will give 404 error if you try to type the direct URL in for a sample
If 1st check is unassigned then the check will need to be done again, not for any further checks

Changed the text in the final submit dropdown
Unassign now available for the 2nd check of a fail (#30)
Make check comment fields longer (#22), 2000 characters instead of 500
Stop unassign form from resubmitting but making it redirect instead (#23)
Make reference_reads_2 an optional field in the database upload (#20)

Justification for minor update
No effect on processing of data, only how it's visualised
Testing description

Issue 31 - updated tests & manual inspection of the test database:

check the two pages work and worksheets numbers are limited (I set the limit to 2 rather than make >30 dummy worksheets, then changed it back)
check the ordering of samples by run ID
check that the diagnostic true/false ordering works

Issue 32 - manual inspection of the test database

check that modal only pops up when assigning a first check
check that the 404 is thrown when trying to type a sample URL in manually
check that the form will need to be resubmitted after a first check has been unassigned

Issues 22, 23, 30 - manual inspection that they behave as expected
Issue 20 - upload a sample with NA for ref reads 2 and check that it uploads okay (you can manually edit the one in this repo)

Test data location
N/A
Verification results
All elements behaved as expected, all unit tests pass

How do we a specify that a run isn't diagnostic on upload e.g training runs? Training runs will be sorted to the top of the list unless we mark as not diagnostic. At the moment we'll need to upload and then change in the backend.

It'll need to be done manually in the backend, I'll update the SOP

For existing runs that were uploaded pre-changes, that have already had a first check but not a second check, when you then try to do the second check it fails as it says it hasn't had a paperwork check. The only way I managed to fix was in the backend. Not sure how many will be in this scenario? Perhaps quite a few if we push the changes mid day on Monday.

It'll only apply to new checks, I'll set all half done checks to True so that they dont cause any issues

What are we going to do in Bfx for the paperwork check when we're going in to fail a sample?

No paperwork check

For the "Sample passed, check complete but extra check required (don't do at first check)" option - can we error out if they do try to do this at first check?? I can forsee accidental clicking.

It'll just complete as normal for a first check, I've added an issue to not show this during a first check (#34)


Lesson3 interface usage

When using interface keyword instead abstract when defining a parent object, in the inheriting object it is not possible to "override" methods. Since you can not define methods with overloads or body in the interface, there also is no need to try it. Any ide gives the hint or error not to find a matching method in the parent when using override.
So for me it seems to be not correct to use it like:
Interface Ixy {
Void print ();
}
Void z (var param) : Ixy {
void override print (){statement;};
}
Please excuse any unaccuracy in writing code as I do it in the metro on my mobile.
I would appreciate a short statement on this.
Kind regards!

The correct usage as I understand and daily use it would be without trying to override it.
The interface just "tells" the inheriting object that there is the need of a method but nothing else.

You are completely right!
I think the following StackOverflow article nicely explains all the finer details:
http://stackoverflow.com/questions/3621410/why-are-c-sharp-interface-methods-not-declared-abstract-or-virtual

eventually it would be good to adopt the Lesson 3 - code as follows(?):
FROM:
`interface A
{
void Print();
}
class B : A
{
public override void Print()
{
Console.WriteLine("I am a B!");
}
}
class C : A
{
public override void Print()
{
Console.WriteLine("I am a C!");
}
}
...
// A x = new A(); // not possible, interfaces can't be instantiated
A y = new B();
A z = new C();
//x.Print();
y.Print();
z.Print();
Output:
I am a B!
I am a C!`
TO:
`
interface A
{
void Print();
}
class B : A
{
public void Print()
{
Console.WriteLine("I am a B!");
}
}
class C : A
{
public void Print()
{
Console.WriteLine("I am a C!");
}
}
...
// A x = new A(); // not possible, interfaces can't be instantiated
A y = new B();
A z = new C();
//x.Print();
y.Print();
z.Print();
Output:
I am a B!
I am a C!
`

done


Contradictory indications about Teams users and ACS SDK in 2 pages

Reading https://learn.microsoft.com/en-us/azure/communication-services/concepts/teams-interop#external-userm, it is said that:

It is currently not possible for a Teams user to join a call that was initiated using the Azure Communication Services Calling SDK.

And reading https://learn.microsoft.com/en-us/azure/communication-services/concepts/interop/teams-user-calling, we can find that:

The Azure Communication Services Calling SDK for JavaScript enables Teams user devices to drive voice and video communication experiences.

Which one is true?

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

ID: e6a1963a-5fe0-9152-bcda-f2aa82bd65e9
Version Independent ID: 2425e174-9fd7-9750-6864-2645bfb83bfa
Content: Teams interoperability - An Azure Communication Services concept document
Content Source: articles/communication-services/concepts/teams-interop.md
Service: azure-communication-services
Sub-service: teams-interop
GitHub Login: @tophpalmer
Microsoft Alias: chpalm

@bankair
Thanks for your feedback! We will investigate and update as appropriate.

@bankair We agree that these appear to be conflicting statements. We have assigned this to the author to provide more clarity.

Hi @bankair,
Thank you for pointing this out. We will update the documentation to make the difference clear.
First statement: It is currently not possible for a Teams user to join a call that was initiated using the Azure Communication Services Calling SDK.
Means that if ACS user calls to another ACS user, Teams user from Microsoft Teams or with ACS calling SDK can't join this call.
Second statement: The Azure Communication Services Calling SDK for JavaScript enables Teams user devices to drive voice and video communication experiences.
Means that Microsoft 365 users can use ACS calling SDK to start a call, accept a call or join Teams meeting. Start a call method is only allowed to another M365 user or phone number, not ACS user.
The statements are not conflicting, but we will improve the language in the documentation to make the difference clear. Thank you

@BryanTrach-MSFT @Naveenommi-MSFT Please close the ticket as Tomas responded

@bankair
We are going to close this thread as answered, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


Improve dynamic rate change in H264 encoder

This change fixes the following issues:

Rate changes requested before the minimum interval has passed are not ignored
anymore. Instead, they are executed in Encode when the minimum interval has
passed. This way there is no risk of ignoring a significant rate change anymore.
Small rate changes are still ignored so that streaming does not stutter too much
at steady state on e.g. HL1.
The minimum interval is now 5s down from 15s so the stream gets to full speed
in a more reasonable time.

Improves microsoft/MixedReality-WebRTC#107

Ping @LoadLibrary for final review.

if (_sampleAttributeQueue.size() <= 2) {

I know it is not related to your change, but I'm wondering:
Is the size of the encoder pipeline always 2?
Is this the right operator to compare with size()?

Refers to: third_party/winuwp_h264/H264Encoder/H264Encoder.cc:377 in be0304f. [](commit_id = be0304f2b0cbeb1758644ba738c16b8a505164af, deletion_comment = False)

Is the size of the encoder pipeline always 2?

🤷♂ I don't know why '2' and I am not sure that this piece of code is useful TBH!

Is this the right operator to compare with size()?

Not sure what you mean.


Linking playbooks in dataplane ceph services

Following services will now use playbooks from edpm-ansible collection, rather than the previous structured 'role' field:

ceph-client
ceph-hci-pre

Depends-On: https://github.com/openstack-k8s-operators/edpm-ansible/pull/279

recheck


URL rewrite performing 301 redirect

1. What version of Caddy are you using (caddy -version)?
0.10.4
2. What are you trying to do?
Rewrite incoming URLs, so assets are served correctly and everything else resovles to index.html.
3. What is your entire Caddyfile?
:5000, https://redacted.domain {
  root /src/dist

  tls {%CADDY_TLS_SETTING%}

  rewrite /revision {
    to REVISION
  }

  rewrite / {
    regexp .*
    to {path} /index.html
  }

  gzip

  log stdout
  errors stdout
}

4.
In Docker:
CMD ["caddy", "--conf", "/src/Caddyfile", "--log", "stdout"]

5. Please paste any relevant HTTP request(s) here.
 What did you expect to see?
Caddy is serving a static SPA, so we expected that the page corresponding to /people in the app would render. This is what happens in 0.10.3 and is confirmed by reverting to that version of Caddy.
7. What did you see instead (give full error messages and/or log)?
Caddy redirects all of the requests to /.
8. How can someone who is starting from scratch reproduce the bug as minimally as possible?
Build a minimal frontend app that rewrites all requests to index.html and uses client-side routing to decide what to render on the client.

Hey @leemachin, thanks for taking the time to fill out the issue template!
This is expected behavior. Your rewrite rule is telling Caddy to rewrite all requests to the same path (if it exists on disk) or, if not, to /index.html.
If the path is rewritten to /index.html, the static file server issues a redirect to canonicalize the path to /, which is correct for serving index files.
If you change your rewrite rule to:
rewrite {
    regexp .*
    to {path} /
}

then it will not issue the redirect anymore, since the rewritten path is correct this way.
Hope this helps!

Hey, I have a potentially related problem happening to me ever since the unofficial joshix/caddy Docker image updated 18 days ago. Sadly I don't have time at the moment to exactly narrow it down, but it seems that Caddy started throwing 301s where it did not before, which was breaking our app.
This is my configuration:
<IP_ADDRESS>:8080 {
        rewrite {
                to {path} {path}/ index.html
        }

        header / Cache-Control "no-cache"
}

In joshix/caddy before 0.10.x, if we were to curl http://localhost:8080/folder/, we would get 200 OK.
In joshix/caddy after 0.10.x, if we were to curl http://localhost:8080/folder/, we would get 301, and it would redirect to http://localhost:8080/folder.
I will absolutely try to dig into this later when I have more time, but it seems awfully like some kind of regression, and it might be more obvious to some people. I realize the joshix/caddy image is not official and was reasonably out of date until recently, but it looks like this may possibly coincide with this bug: #1177. We rely on directory-style URLs in our SPA, so without a solution this causes quite some trouble.
Hopefully this isn't wholly unrelated to this thread, but I apologize if it happens to be. Either way, again, I'll try to take a deeper look at this later on.

@johnwchadwick I think (hope) that's different. Did you try with Caddy 0.10.4 (released 2 days ago)?

@mholt thank you for the explanation. If this is expected behaviour, why is it that 0.10.3 and 0.10.4 behave differently with the same config? I've looked through the changelog and commits and cannot see anything that would break things.
We've since reverted to the previous version (easy fix ;)) but I will use your suggestion and see how that goes, since it would be good to stay updated.

@leemachin

If this is expected behaviour, why is it that 0.10.3 and 0.10.4 behave differently with the same config? I've looked through the changelog and commits and cannot see anything that would break things.

Commit f4b6f15e07b91976d6579c8445a3321dcc63d8ae.
It fixed a regression. :wink:

@mholt to confirm the config in https://github.com/mholt/caddy/issues/1737#issuecomment-312294705 or https://github.com/mholt/caddy/issues/1737#issuecomment-312339423 is the recommended way to handle SPAs since the regression fix?

@OmgImAlexis I dunno, I haven't tried it, but it seems like it would do what you need!

@mholt I am still having this issue as of version 0.10.6. I have also tried the solution that you posted above, but Caddy continues to bring me back to / with a 301.

@dtrinh100 Make sure to clear your browser cache; check it with curl just to be sure!

@psrcek In Caddy 2, you can turn off URI canonicalization: https://caddyserver.com/docs/json/apps/http/servers/routes/handle/file_server/
We can also expose that for the Caddyfile (https://caddyserver.com/docs/caddyfile/directives/file_server) but we just haven't gotten around to that yet.

Thanks guys for your help, but as I use Caddy v2, I can't convert the old syntax:
rewrite {
  to {file} index.html
}

to the new one : rewrite ...
Any ideas ? Thanks

@mkhennoussi please ask your usage questions at https://caddy.community. This is the issues board for bugs and feature requests.

Thanks guys for your help, but as I use Caddy v2, I can't convert the old syntax:
rewrite {
  to {file} index.html
}

to the new one : rewrite ...
Any ideas ? Thanks

You can use try_files directive in v2:
handle {
  root {$WWW_ROOT}
  try_files {path} /index.html
  file_server
}

Thanks guys for your help, but as I use Caddy v2, I can't convert the old syntax:
rewrite {
  to {file} index.html
}

to the new one : rewrite ...
Any ideas ? Thanks

You can use try_files directive in v2:
handle {
  root {$WWW_ROOT}
  try_files {path} /index.html
  file_server
}

I use vue with history mode, it dose not work. Anything help?

Hey @Virgil-N, I created this repo showcasing how to use it with Vue router in history mode: https://github.com/RecuencoJones/caddy-vue-router
Looks like since rc version we also need to add matcher to root directive.
You can test with npm
npm install
npm run serve:caddy

or docker
npm install
npm run build
docker build -t caddy-vue-router .
docker run -dit --name caddy-vue-router -p 8080:8080 caddy-vue-router


Edit pass of collect metrics

Eliminate future tense.
Use bigger ideas, fewer words : Our modern design hinges on crisp minimalism. Shorter is always better.
Short sentences and fragments are easier to scan and read. Prune every excess word.

Internal previews

📄 File
🔗 Preview link

docs/core/diagnostics/metrics-collection.md
Collect metrics

@gewarren  can you tell me where the snippets go and how to reference them? I'm going to change
 Thread.Sleep(1000); to
 Thread.Sleep(rand.Next(200, 1500));
to make it more interesting. Code needs formatting too.


[Unity][Training] Training Utils: Loss functions, Optimizers, Trainer, SetupTrainer

This PR introduces several utils to support training models in Relax IR, including:

Loss functions. A library that generates relax functions representing common loss functions, such as MSELoss and CrossEntropyLoss.
AppendLoss pass. You can specify one function (the backbone function) in the IRModule, and one loss function to AppendLoss. It will attach the loss function to the backbone function, and add the result function to the IRModule.
Optimizers. A library that generates relax functions representing common optimizers, such as SGD and Adam.
SetupTrainer. A pass that transform the backbone module (a IRModule containing only one function named "backbone") into a module that contains all functions used in the training process.
Trainer. A runtime utility that stores parameters, states, etc, and helps to run the training IRModule.

Co-authored-by: Chaofan Lin<EMAIL_ADDRESS>

Separated into many pull requests


chore(colors): fixed regressions from the color updates

Fixed scrollbar styles in all themes
Replaced gray0* colors with the updated values

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


feat: Issue-242 - Translation to PT-BR

Implementation of the Brazilian Portuguese translation.

Added the pt-br.yml file with all the translations.
Tested locally and had no issues.

@nzakas kindly review and feel free to make adjusment as needed.

Hi @nzakas I have a doubt about that EasyCLA validation. Whenever you're available, could you give me a hand here?

Thanks! I'll ask for some feedback from folks online, since I don't really know Portuguese. :)

If you don't mind I've asked some of my brazilian friends to review it too. ;)

Never too many eyes!


Problem with tab-character

When an attribute value string contains a tab character (0x09), during saving it stays as is and won't be changed to "	". On loading however it will be read as a space character (0x20).
I found that the 0 in the first line of the chartypex_table is responsible for that behaviour.
static const unsigned char chartypex_table[256] =
{
3,  3,  3,  3,  3,  3,  3,  3,     3,  0,  2,  3,  3,  2,  3,  3,     // 0-15
Is this intentional? (The comments for enum chartypex_t) hint in that direction.
Changing the '0' to '2' fixes the problem:
3,  3,  3,  3,  3,  3,  3,  3,     3,  2,  2,  3,  3,  2,  3,  3,     // 0-15

Edit: Tab and won't be changed to

During loading, pugixml performs attribute value normalization as described here: https://www.w3.org/TR/xml/#AVNormalize that replaces tab characters with a single space.
However, this doesn't really preclude escaping tab characters during output. At first glance it feels like this is an omission - pugixml does escape both forms of newlines in attribute values. I'll need to take a closer look at this but it might be that this warrants a change in behavior so that you can save tab characters in attributes.

Brief survey of other major XML libraries:

.NET System.Xml XmlDocument doesn't escape tab character
Python ElementTree doesn't escape tab character
Python lxml escapes tab character using
JavaScript DOM API escapes tab character using

It looks like libraries that don't escape tab character in attribute values make it impossible (AFAICT) to save this character into an attribute value, since trying to use 	 will just escape the ampersand. I'm going to fix this, which should also make it so that attributes with 	 roundtrip properly.


Remove Win32 Code

After #1835 we're no longer building Windows 32 bit binaries, we can remove the code that handles that case.

This is referring to the code in main.js, not the build scripts.


Stamina trait

@AndrewPochapsky fixed, did some more refactoring to completely remove msg.rs and moved everything to staking.rs
Refactoring should be done with this commit too? Ik it was out of the scope of the PR but oh well, it was hard not to lol


Update Implementation Section of Developer Guide

Codecov Report

Merging #160 (a7d16f2) into master (1e73038) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #160   +/-   ##
=========================================
  Coverage     48.75%   48.75%
  Complexity      569      569
=========================================
  Files           123      123
  Lines          2640     2640
  Branches        286      286
=========================================
  Hits           1287     1287
  Misses         1271     1271
  Partials         82       82

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/model/ModelManager.java
63.33% <ø> (ø)

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 1e73038...a7d16f2. Read the comment docs.


Add DVCC control/BMS control

Currently, the code only acts as a battery monitor. It can replace a BMV and can display most relevant BMS data. However, it is not able yet to control the current of Victron charging and discharging devices via the Cerbo.
As @CodeChief and some other requested, it would be nice to have this. Currently we have added four relevant paths in a test code:
/Info/BatteryLowVoltage
/Info/MaxChargeVoltage
/Info/MaxChargeCurrent
/Info/MaxDischargeCurrent

However, even when trying to set DVCC to on, the "BMS control" stays off. We need to test if these fields have influence on Victron devices and if yes, if DVCC needs to be set to ON or if this is not relevant.

Added to latest code. Note: only supports this for 1 USB cable at a time. BMS support with multiple 123\SmartBMS to USB cables probably does not work as that needs extra code for cooperation.

DVCC calculation / Docu
want to learn better the calculation behind and how the settings on DVCC values are done... Feel free to point me into any doc files / wikis

Dear Sir, Madam,
The development mailbox will not be read until the 10th of August. For urgent support, please contact your dealer.
For 123ignition, see https://123ignition.com/dealers/
For 123electric, see https://123electric.eu/dealers/
Best regards,
Team 123ignition/123electric


[Refactor] refactor tooltips with antd Popover

Description

Currently the react-tooltip has some issues. Since we are planning to introduce antd into the project, we may consider using Popover to replace the react-tooltip.

It just so happened that I learned a lot about antd in the process of OSPP, so leave this task to me!

#601 is related to this issue, please pay attention to this problem when working on antd


nix-weather: init at 0.0.3

Description of changes
Changelog: https://git.fem.gg/cafkafk/nix-weather/releases/tag/v0.0.3 https://git.fem.gg/cafkafk/nix-weather/releases/tag/v0.0.2
Signed-off-by: Christina Sørensen<EMAIL_ADDRESS>
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@freyacodes I've added you as maintainer and co-author on this, and included some of the changes from #340477 (see https://github.com/NixOS/nixpkgs/compare/639b2f1a5c19cbeb760eaa63b24929c794e816df..5658c35d7fa10816a0d11d283ab197f6b7070a78). Hope that's okay :3

Result of nixpkgs-review pr 340518 run on aarch64-darwin 1

2 packages failed to build:

nix-weather
nix-weather.man

Result of nixpkgs-review pr 340518 run on aarch64-darwin 1

2 packages built:

nix-weather
nix-weather.man

Result of nixpkgs-review pr 340518 run on x86_64-darwin 1

2 packages built:

nix-weather
nix-weather.man


Python r string looks weird.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.64.2
OS Version: Windows_NT x64 10.0.22000

Steps to Reproduce:

Create a python file
Type a r string like:
folder = r"C:\intel\code"
The r string looks weird:

Maybe the letters' color should be the same in this case?

I've already disabled all the extensions.
I also tried other themes like One Dark Pro, still the same.

Apparently this is a feature: https://github.com/MagicStack/MagicPython/issues/114#issuecomment-349115524
You can use uppercase R to highlight as a raw string.


Using Eu and Us flags together breaks the Eu flag

This can been seen in this playground link: https://svelte.dev/repl/1b898f87c1a743f8a4874e4f3a403d50?version=3.59.1

Try Svelte-Flag-Icons.

Yup that worked, thank you. I dunno if I should close this issue or not though since that's a different package

The Svelte-flags icons have some bugs and that's why I create Svelte-Flag-Icons.
Thank you for your contribution.


timezone parameter

the timezone of the container is set to a default
is there anyway of creating a timezone (TZ) parameter so user could set it in the creation of the container (using Synology) or giving instructions on how to set it

I would also like to see this parameter added. Would help with logs if the correct timezone was applied

The simple way is to mount the base systems on into the container.
-v /etc/localtime:/etc/localtime
I unfortunately don't have time at the moment to look into updating the image todo it via a flag.
I'm happy to accept a pull request if anyone would like to tackle this.


Long checkout times in DevOps builds

https://typescript.visualstudio.com/TypeScript/_build/results?buildId=33363
The Checkout task takes 1 minute 40 seconds, most of which is pulling what seems like a rather large amount of Git history.
(abridged history)

Initialized empty Git repository in /home/vsts/work/1/s/.git/
##[command]git remote add origin https://github.com/Microsoft/TypeScript
##[command]git config ...
remote: Enumerating objects: 80684
remote: Enumerating objects: 125103
remote: Enumerating objects: 159133, done.
remote: Counting objects:   0% (1/159133)
...
remote: Counting objects: 100% (159133/159133)
remote: Counting objects: 100% (159133/159133), done.
remote: Compressing objects:   0% (1/82374)
remote: Compressing objects:   1% (824/82374)
...
remote: Compressing objects: 100% (82374/82374)
remote: Compressing objects: 100% (82374/82374), done.
Receiving objects:   0% (1/159133)
Receiving objects:   1% (1592/159133)
...
Receiving objects:  99% (157542/159133), 194.80 MiB | 13.35 MiB/s
remote: Total 159133 (delta 92786), reused 130877 (delta 72676), pack-reused 0
Receiving objects: 100% (159133/159133), 194.80 MiB | 13.35 MiB/s
Receiving objects: 100% (159133/159133), 201.77 MiB | 10.67 MiB/s, done.
Resolving deltas:   0% (0/92786)
Resolving deltas:   1% (944/92786)
...
Resolving deltas: 100% (92786/92786), done.
From https://github.com/Microsoft/TypeScript

Perhaps this could be improved by using git checkout --depth=100 or some similar optimization?

actions/checkout now takes <=15 seconds. https://github.com/microsoft/TypeScript/runs/2694783826?check_suite_focus=true ✅


Perm/upgrade perm

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

An explanation of the use cases your change solves

Links to any other associated PRs

[ ] I have reviewed the contributing guide

[ ] I have viewed, signed, and submitted the Contributor License Agreement

[ ] I have made this pull request to the master branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

It did end up succeeding:
https://travis-ci.org/cloudfoundry/cloud_controller_ng/builds/305570783


Include all email data in 'extra' response

This allows consumers to check whether or not Github has verified the emails, or use non-primary email addresses if they want. Important for consumers that need validated emails.
Additionally, a new 'display_email' field is added to info, which will always be the email displayed on your GitHub user page.
Fixes #36

I'd still accept this if it were changed to put an unverified email in an unverified_email key instead of the email key.

Merged in #48


Node.js bindings: Add bindings for gpio_open_by_label and gpio_open_raw

This adds missing bindings for gpio_open_by_label and gpio_open_raw.
Signed-off-by: Sudarsana Nagineni<EMAIL_ADDRESS>

@edersondisouza @cabelitos @gabrielschulhof Could you please have a look?

No big deal, just a minor suggestion.


Proposal: podcast:sponsored used to identify episodes or segments that are sponsored by a particular advertiser or brand

podcast:sponsored: This tag could be used to identify episodes or segments that are sponsored by a particular advertiser or brand. This could allow advertisers to track the performance and effectiveness of their sponsored content, and could help data scientists to analyze the impact of sponsored content on listener engagement and behavior.

I try to look at everything from the perspective of "How does this benefit the audience?"
I imagine this could be used, like the disclosure feature on YouTube videos, to indicate to the audience that the episode has some kind of "paid" placement.
This tag could potentially even be used to help disclose affiliates, as is required in many places.


0.18.0 broke file tree display

After updating ex-mode today, directories or projects that i opened did not display in the file tree pane.  Pane remained blank after opening project.   I disabled ex-mode and restarted atom - file tree was back.

Can you please provide some steps on how to reproduce this bug, and please post the output of apm --version and atom --version. If you use atom beta, then the command is apm-beta and atom-beta.

On Aug 21, 2017, at 4:32 AM, Edvin Hultberg<EMAIL_ADDRESS>wrote:
Can you please provide some steps on how to reproduce this bug, and please post the output of apm --version and atom --version. If you use atom beta, then the command is apm-beta and atom-beta.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/lloeki/ex-mode/issues/194#issuecomment-323668229, or mute the thread https://github.com/notifications/unsubscribe-auth/ABy9aXklx_XX0pCHvAqekx8cC9WC3qQAks5saTKMgaJpZM4O8Z6J.

∑ apm --version
apm  1.18.2
npm  3.10.10
node 6.9.5 x64
python 2.7.10
git 2.12.2
∑ atom --version
Atom    : 1.19.0
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0
I installed your latest ex-mode, and then after a restart, I could not get any files or directories to show up in the leftmost pane.  (Both atom and atom-beta).
Disabling ex-mode did not bring up the file tree, but after disabling ex-mode and a restart, file display came back.
regards
Stephen Wright

Thanks @switheyw. Just to be sure, when you mention "leftmost pane", do you mean the tree view? I'm not able to reproduce this, and I suspect there are some other reason for your issue.
Also, I notice you are using atom 1.19.0, which is outdated. Can you please try to reproduce using atom 1.19.3? :-)

Hi,
ya, meant the tree view.  Mostly I just use the beta, so I was not aware that regular atom was updated
This is probably something that I am somehow doing if you can’t replicate it.  I’m on the road for the next few days,
but i’ll try to update atom and go through the tests again.
Stephen

On Aug 23, 2017, at 3:40 AM, Edvin Hultberg<EMAIL_ADDRESS>wrote:
Thanks @switheyw https://github.com/switheyw. Just to be sure, when you mention "leftmost pane", do you mean the tree view? I'm not able to reproduce this, and I suspect there are some other reason for your issue.
Also, I notice you are using atom 1.19.0, which is outdated. Can you please try to reproduce using atom 1.19.3? :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/lloeki/ex-mode/issues/194#issuecomment-324247873, or mute the thread https://github.com/notifications/unsubscribe-auth/ABy9aZZ5QLBOHAy33XCQNsWRK840mg-Dks5sa9eIgaJpZM4O8Z6J.

hey,
I’ve been on the road for awhile, but just retried ex-mode and ALL IS WELL.   I had been having trouble with the initial stages of opening a file and or project folder.  I’d end up with that error which mentioned
a problem with panes.  Perhaps the tree-view problem was a side-effect of that aspect of Atom.   So sorry for wasting your time.  The ex-mode package is great.
Stephen

On Aug 23, 2017, at 3:40 AM, Edvin Hultberg<EMAIL_ADDRESS>wrote:
Also, I notice you are using atom 1.19.0, which is outdated. Can you please try to reproduce using atom 1.19.3? :-)


multiplayer crashing when trying to place  workbench or campfire

Invalid type in function 'place_item'. The Object-derived class of argument 1 (Resource (InventoryItem)) is not a subclass of the expected argument class.

Fixed on 2.0.1

Fixed on 2.0.1

Where can i get the 2.0.1 version, isn't this 2.0.0 verson?

Fixed on 2.0.1

Where can i get the 2.0.1 version, isn't this 2.0.0 verson?

In version 2.0.1 itself, however the store has a delay in approving the update.

Fixed on 2.0.1

Where can i get the 2.0.1 version, isn't this 2.0.0 verson?

In version 2.0.1 itself, however the store has a delay in approving the update.

So it is still being approved, the 2.0.1 version?

Fixed on 2.0.1

Where can i get the 2.0.1 version, isn't this 2.0.0 verson?

In version 2.0.1 itself, however the store has a delay in approving the update.

So it is still being approved, the 2.0.1 version?

Updated now on godot asset lib

Fixed on 2.0.1

Where can i get the 2.0.1 version, isn't this 2.0.0 verson?

In version 2.0.1 itself, however the store has a delay in approving the update.

So it is still being approved, the 2.0.1 version?

Updated now on godot asset lib
I cannot find it anywhere, the 2.0.1. If i download it from asset store or from github it still gives the same error when i try to place campfire or workbench perhaps you can link it here ?

Sorry for the delay @Prashanna135 , there was a small issue with inputs not configuring themselves. Fixed in 2.0.2


support url paths for flat-manager-client?

./flat-manager-client push --commit --publish $(./flat-manager-client create "https://example.com/flatpak/api/" main) ~/test-build/local-repo
Api call to https://example.com/api/v1/build failed with status 404, details: {'status': 404, 'error-type': 'no-error', 'message': 'No json error details from server'}

your urljoin call replaces the path instead of appends to it. remove the leading '/' in "/api/v1/build"


chore: bump std

Incorporates the change https://github.com/leanprover/std4/pull/92 into mathlib.

bors r+


Invalid operation for scaling

The operation for scaling is a subtraction instead of a multiplication.
https://github.com/qupath/qupath-extension-stardist/blob/f5842b404ac460bf4e60218c3de177900918f892/src/main/java/qupath/ext/stardist/StarDist2D.java#L540

Oh dear - good catch, thanks @qjuppet


ssh deploy and firecracker

there is a easy way to run firecracker with weaver ssh deploy.
docs: https://docs.nats.io/using-nats/nex
code: https://github.com/synadia-io/nex
This gives you a cloud in a box in many ways.
When you mix weaver with NATS you can orchestrate your "cloud".
I just adding this as an Issue even though its not an issue because the 2 things works so well together IMHO.
The SSH deployer is missing a few things are documented here: https://serviceweaver.dev/docs.html#ssh-experimental-limitations
The NATS Cli can be used to control Weaver, so I might make a "weaver nats deploy" etc ...
Am just thinking out loud here to see if anyone is into this. Would appreciate any feedback about this concept / approach.

nats is pluggable so you could also have nats weaver if you really wanted, ping me if you're curious
Your basic nats context usage is really just natscontext.Connect(os.GetEnv("NATS"), opts) that's all.  Our integrations in cli is more complex because we also want to support passing/overriding some flags from command line, most people wont need that, simple one liner is enough.

hey @ripienaar
yep a NATS weaver is exactly what I am working on. SO you can just connect over the NATS control plane.   I dont know how to reach you :) I am on Telegram though...

Hey @ripienaar
If you have time for a chat about this can you grab me on telegram or suggest what you want to use ?

Slack is where nats support happens.


[Feature] Autoplay support

I recently switched to this bot from SudanPlayz Bot as it's not well maintained, and it's sooo much better. The only thing I've missed though when switching has been their autoplay feature, which auto plays more music based on what was queued. It's a fun way to find new music with friends, and I miss it.

Autoplay is more something from YouTube, as it has to query any possible ideas for new music. Although you can queue playlists that have been recommended by YT to you. If you (or someone you know) is interested in making a AutoPlay PR, go on ahead.


Add message for generic parsing exceptions

Description
When the input XML is in some way invalid but it can still be parsed, for example
<topic id="%50">

the XML parser will throw an exception with message

Attribute value "%50" of type ID must be an NCName when namespaces are enabled.

We currently capture this and log out as an error message.
In order to give as many messages an ID as possible, we add a message ID for generic parsing errors where the <reason> of the message is provided by the XML parser and the <response> is left empty. The logged message will be

Error: file:/Volumes/tmp/src/topic.dita:3:17: [DOTJ088E] Attribute value "%50" of type ID must be an NCName when namespaces are enabled.

or for example with incorrect attribute name that goes against the DTD

Error: file:/Volumes/tmp/src/topic.dita:3:36: [DOTJ088E] Attribute "xml:langx" must be declared for element type "topic".

Motivation and Context
Add error ID for parsing exceptions to improve debugging.
How Has This Been Tested?
Manual testing.
Type of Changes

New feature (non-breaking change which adds functionality)

Documentation and Compatibility
This is a generic error message, so we don't know in advance what the error message will be. Users may override the message configuration to be e.g.
<reason>Error in parsing file: %1</reason>

if they want to add more context to the message.

@jelovirt - this is a nice enhancement, especially now being able to override this class of message as desired.


Update dependencies for Play 2.6 RC2

Since there is a Play 2.6.0 RC-1 and Json library is now a separate project a minor change is required.

It failes as Play 2.6 RC1 is using snapshot of akka-http-core version 10.0.6+7-e2ba6752. To build I would have to add "Akka Snapshots" at "http://repo.akka.io/snapshots/" to the resolvers... which is not very desirable.

Coverage remained the same at 89.31% when pulling ca36df198bf8d789d8c4d1f818ea257211106675 on evojam:play2.6 into 676836393f435ba9532d6d8ed4edb23d5b6b7b56 on iheartradio:play2.6.

Updated to RC2

@kailuowang According to the https://playframework.com/documentation/2.6.x/Migration26 play JSON moved to separate project. Currently the version matches each other although I think it should be explicitly defined.

@abankowski thanks for the explanation. That makes sense.

published. https://bintray.com/iheartradio/maven/play-swagger/0.6.1-PLAY2.6-RC2


[FLINK-14010][coordination] Shutdown cluster if ResourceManager terminated unexpectedly

What is the purpose of the change
Context can be found at FLINK-14010
Brief change log
Add callback logic when ResourceManager terminates unexpectedly, complete DispatcherResourceManagerComponent#shutdownFuture exceptionally.
Verifying this change
The change is technically trivial and can be easily reasoned. However, I'd like to introduce a test of it, but cannot find a good way.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes, now in YARN deployment Flink App can correctly response to shutdownRequest)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Hi @tillrohrmann Thanks for your review.
I add a DispatcherResourceManagerComponentTest and expose some visibility, please double check it doesn't cause other issues.
Besides, I find another code change required by this approach that actually we are unable to deregisterApplication on DispatcherResourceManager closed because RM is already terminated.
It seems we twist codes a bit more so although the logic is correct at the moment I still prefer just call onFatalError in YarnResourceManager#onShutdownRequest which fixes the original issue specifically. We cannot do anything when we lost connection to YARN RM and lost leadership semantically. Anything I miss we can gain by gracefully closing ClusterEntrypoint?

Thanks for your insight @tillrohrmann. I put on the version causing fatal error in YarnResourceManager#onShutdownRequest as well as the test. Please review and share the idea which one you prefer after both implementation are present.