⚠ Bofrost.es has degraded performance

In b21edb9, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1498 ms

Resolved: Bofrost.es performance has improved in 8d250cf after 9 minutes.


[Feature Request]: Possibility to allow SAML and LDAP-Login as AUTH_METHODE at the same time

Describe the feature you'd like
It would be useful to use both SAML and LDAP at the same time as "AUTH_METHODE".
This is especially useful for people who have an Active Directory and Active Directory Federation Services (AD FS) on windows in use.
It could look like this: AUTH_METHODE="ldap|saml2".
If a user already has an account created via LDAP, the account should be mapped with the existing one when using SAML. This is only possible if the provided attribute (for example the email) is the same with the existing mail in bookstack. For now a login with SAML is blocked if the user was previously created via LDAP.
Describe the benefits this feature would bring to BookStack users
Furthermore, you would have a fallback without having to switch back to the other one, should something go wrong, for example because one of the services has failed. This would give redundancy to the whole login. In addition, there are rarely use cases that only allow one of the two methods. When using an AD FS system, there is always an Active Directory behind it. It is therefore easy to fall back if something goes wrong.
Additional context
No response

Hi @Holt31, Thanks for your request.
Can you describe your scenario a little further? Are you intending to use SAML2 and LDAP against the same AFDS system? Is there a particular challenge you're facing that this would solve? You mention redundancy but that seems a bit overkill to be worth our implementation, especially if using the same ADFS system since we're only talking redundancy at a very specific level if so.
Just trying to understand the problem rather than this specific idea of implementation since adding such a feature would raise a bunch of additional scenarios and problems to consider  & address.

For now a login with SAML is blocked if the user was previously created via LDAP.

This is not true if usage of the external_authentication_id is aligned. If they're both using email address a switchover should be fairly painless.

Thanks for the fast reply.
I have actually 2 scenarious.
Scenario 1:
I have my Domain Controller and my AD FS-Server on seperate servers. (technically the AD FS-Server is a replicating domain controller) Since both Domain Controllers are replicating with each other, they both have the same data. One of them doesn't have an open LDAP-Port, so I can't use LDAP on this one. I'm using AD FS for that. Now I want to have LDAP as Fallback, when AD FS isn't working.
Scenario 2 (which I like to implement):
I have an MS 365 Account, where you can use SAML for authentication (Azure Active Directory).
I'm using an onpremise Active Directory to sync Users into the Azure AD.
I want to use the SAML from Azure AD and the LDAP of the Onpremise at the same time..
So basically both scenarios are actually the same thing.

This is not true if usage of the external_authentication_id is aligned. If they're both using email address a switchover should be fairly painless.

Thanks for mentioning it.
I tested it again and noticed that I used the "sAMAccountName" attribute for the external ID when using LDAP and I'm using the "userPrincipalName" attribute for the authentication against AD FS. Since these two differ it makes sense, that a login wasn't possible before. When I changed the ldap attribute to userPrincipalName the login worked.

Bumping this too, as I'm using jumpcloud for LDAP, and also office 365 through jumpcloud for SSO.

I'm closing this to collapse this into #2715, which I've made into the auth-method abstract issue for combining auth options.


Accept empty account proofs to for MptUpdates where new and old values are 0

An example where this happens is when a contract is deployed, and is then reverted.

seems correct


chore: Sort -Z flags match statement

#12182 started work on sorting/organizing things within src/cargo/core/features.rs. While working on #12182, I noticed that this match statement was not sorted very well and made things hard to find.
I decided to sort the match statement in the say way as I suggested for features!:

Stability (always unstable>stable>unstable)
Version stabilized (1.31>1.32)
Feature name

Note: I couldn't find a way to keep this sorted as I want to do with unstable_cli_options! and features!

@bors r+

:pushpin: Commit bb3a72624a94dea7e6fa89b43afefde16ee85187 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit bb3a72624a94dea7e6fa89b43afefde16ee85187 with merge 2579f259771239b96b162e2a38c7101dc3cddedb...


erlang:get_stacktrace was removed from OTP24

https://www.erlang.org/erlang-enhancement-proposals/eep-0047.html
https://github.com/erlang/otp/pull/1634
% pacman -Qs erlang
local/elixir 1.12.0-1
    a functional meta-programming aware language built on top of the Erlang VM
local/erlang 24.0-1
    General-purpose concurrent functional programming language developed by Ericsson

cowboytest % mix deps.compile
===> Fetching rebar3_hex v6.11.3
===> Version cached at /home/devuser/.cache/rebar3/hex/hexpm/packages/rebar3_hex-6.11.3.tar is up to date, reusing it
escript: exception error: undefined function erlang:get_stacktrace/0
  in function  rebar3:main/1 (/tmp/cirrus-ci-build/src/rebar3.erl, line 72)
  in call from escript:run/2 (escript.erl, line 750)
  in call from escript:start/1 (escript.erl, line 277)
  in call from init:start_em/1
  in call from init:do_boot/3
** (Mix) Could not compile dependency :p1_utils, "/home/devuser/.mix/rebar3 bare compile --paths /work/cowboytest/_build/dev/lib/*/ebin" command failed. You can recompile this dependency with "mix deps.compile p1_utils", update it with "mix deps.update p1_utils" or clean it with "mix deps.clean p1_utils"

Sorry, that in rebar3.


Add support for json data visualization

This commit adds the script to support json data visualization,
this works on the fact that the data downloded is in JSON format.
This gives the trends like the most messages in which year, or which month of the year you text the most etc.
Signed-off-by: Farhaan Bukhsh<EMAIL_ADDRESS>

Few suggestion from my side:

Instead of asking for the exact location of message folder we should only ask for for the location of the extracted folder, because the same schema is being followed in the plot_friends.py script
If I don't give the location of the folder while running the script, then it gives an error, like if I run python messages_json_plot.py then it gives an error. Instead it should input location of the folder.
Please add some sample data in the examples folder so that the user can run the scripts without his/her facebook data.
Please also add pandas to requirements.txt file.

I was thinking we should either have all scripts or all classes, not both. Need to standardise this.
Also, no need to make new file, just call this plot_messages.py.
Could you please add a sample json file like @techytushar did in his previous PR?

@kaustubhhiware I don't know I feel that if we write it in terms of classes there can be a lot of reusable code that can be seen, and also IMHO I feel it gives the code a structure but again it's your call. 😄
@techytushar Thanks for the feed back will try to include sample data but I feel it will be great to let them download and use their data.

Ahh my bad I should have put the message for message location

@farhaanbukhsh Yes, now its working, but we should take the users facebook data folder address insted of message folder, as this is the convention going on in all other scripts. :slightly_smiling_face:

@techytushar fixed it :smile:  thanks a lot for suggestion, can @xprilion and @kaustubhhiware and @roopalJazz check it too?

should I merge this in guys? :rofl: been real long?

Merging this in 😄


Build dask / distributed pre-releases on commits

One issue with the distributed and dask pre-release packages is that since they pin their dask-core dependency and are only built on commits to Distributed, they can quickly fall out of sync with new dask-core pre-releases.
This PR resolves this this by also building dask / distributed pre-release packages on commits to Dask, so that new dask-core pre-releases will always have a corresponding distributed and dask.

[ ] Closes #xxxx
[ ] Tests added / passed
[ ] Passes pre-commit run --all-files

Closing this in favor of #8664


new(pkg): support RC releases for vanilla target.

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area pkg
What this PR does / why we need it:
This PR adds support for release candidates for vanilla target.
Is going to be used to improve libs latest-kernel CI to test the build against latest mainline kernel available (RCs too!), so that we get notified asap when a kernel breaks our drivers build! :)
See: https://github.com/falcosecurity/libs/pull/1090
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

new(pkg): support RC releases for vanilla target

Closing and reopening to trigger tide check.
/close

@FedeDP: Closed this PR.

In response to this:

Closing and reopening to trigger tide check.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/reopen

@FedeDP: Reopened this PR.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Improving package setup to pyproject.toml

Description
Improve the approach for packaging and use a pyproject.toml approach
Context
The common approach for a python project to be used is either a direct way, mostly like cloning the repository, and a compacted way, mostly in the form of packaging and/or containerization. In either case, for the same version of the software, the same functionality should be available for any given system independently on how it was installed.
Problem
When the user decides to go for the package/container route of installing the software, there will be dependency issues not straight forward solvable for some cases (more yet to be mapped). A non exhaustive list of these problems is:

Python version and sub-version compatibility.
PYTHONPATH values maybe are not updated programatically.
Order of dependency installs might break out inter-dependencies.
Dependencies to download data might conflict with those for running workloads.

Elements for the solution (Draft)
In general, it might me a good opportunity to update the packaging from a setup.py oriented approach to pyproject.toml approach.
PEP 518 – Specifying Minimum Build System Requirements for Python Projects

This PEP specifies how Python software packages should specify what build dependencies they have in order to execute their chosen build system. As part of this specification, a new configuration file is introduced for software packages to use to specify their build dependencies (with the expectation that the same configuration file will be used for future configuration details).

Externally Managed Environments

To allows a Python installation to indicate to Python-specific tools such as pip that they neither install nor remove packages into the interpreter’s default installation environment

Some other details might be useful:

Every package now should be strictly necessary (at least really really hard to not depend on it)
Use of exact versions for every single package.
Include explicit installation order and without using cache installed versions.
Make sure the latest version of pip is installed.
Use of installation flags for python packages.
Compile/store the package's Wheel (offline file) according to particular architectures.

Ok, @priyakasimbeg, here is my proposition of more actionable items to start the first phase of refactoring: And is in the Datasets installation/downloading process actually, previous to the overall project.
Problems:

Dependencies across datasets might differ in time.
Dependencies across datasets and the submission script, or any other script for that matter, might differ in time.
Pre-processing for one dataset might be different to another one.
Changes and updates into the understanding for each dataset might be at different pace.

Improvement oportunities:

Move from a monolithic dataset config to a per-data set config.
Move from a dataset_setup.py config to a pyproject.toml logic.
Extend/update with the following:

Keep the ~/data/ and ~/temp/data local folder creation.
Define a pyproject.toml file for all datasets.
Within pyproject.toml specify a dependency list for each dataset.
Create a sub-folder per dataset

Create/relocate/expand downloading, pre-processing, eda scripts.
A README.md for each dataset with some of the following:

Official Name, Creator, License.
Data and File Structure.
Exact Full Size (On decompressed + worst case scenario).
Other extra details.

Environment and execution complements:
4. (Good to have) Instructions to avoid terminal locking (a new tab, locked max-resources, tmux).
5. (Good to have) With progress indication in terminal.


Add kernel modules, tuning and support packages for hosts

This patch adds several packages requested by RPC support to all hosts and
containers, adds kernel module loading and sysctl tuning to all hosts.
The reference for all additions can be found in the description and comments
for https://bugs.launchpad.net/openstack-ansible/+bug/1400424

LGTM :+1:
One question.. why debootstrap?

@hughsaunders I have no idea, it was requested in the launchpad bug... ultimately I think the main purpose here is to provide the facility to do these things. This allows support to override if need be.

Yep, lets get this merged :+1:

There is a lot of duplication here w/ what's already defaulted in https://github.com/stackforge/os-ansible-deployment/blob/master/playbooks/roles/openstack_hosts/defaults/main.yml.

debootstrap shouldn't be needed since we're presumably not building images. But it may have been pulled in as a dependency for one of the other packages. I suspect the reason for installing these outside of ansible plays may be chicken-and-the-egg situation where some level of configuration is required before ansible is run, but ansible is clearly the best tool to perform the install.
Either way. :+1: from me.

@mattt416 I guess that duplication is so support can tune and add packages without adding them to OSAD

@nrb The PR is https://github.com/rcbops/rpc-openstack/pull/81

@odyssey4me Thanks. Does anyone know of a reason that this shouldn't be backported to Kilo? @hughsaunders @mattt416 @rackertom


Dev - A little buggy

Someting with the CSS makes a grey box.
Small buttons to the right have jumped up.
The line is too tall between the little boxes.
Logout comes too times.

<?php
$items = array(
	'panel',
	'add',
	'edit',
	'update' => panelbar::link(array(
		'id'     => 'panel',
		'icon'   => 'rotate-right',
		'url'    => site()->url() . '/update-kreditgivare/?update=true',
		'label'  => 'Sync',
		'mobile' => 'icon',
	)),
	'logout'
);

echo panelbar::show( array( 'elements' => $items ) );
?>

I think I fixed parts of it (multiple logout buttons) and will work on the other bugs as soon as I'm tackling responsiveness anyways.

It seems to work now.


DRY up integration tests; remove 'Cleanup' testcase.json rewrite

Fixes #661

Removed testcase.json rewrite
Refactored UpdateGolden usages
DRYed up related test code

seems good, ty!


autoReconnect not catching ECONNRESET

Hello,
I can't make autoReconnect feature work, I'm getting a ECONNRESET error that does not trigger the reconnection logic.
{
  "type": "connection error",
  "error": {
    "code": "ECONNRESET"
  }
}

My code :
this.tweetsStream = await this.twitterClient.v2.searchStream({
  ...
})
this.tweetsStream.autoReconnect = true

Hello Micode, thanks for the report
I guess it's related to this issue https://github.com/PLhery/node-twitter-api-v2/issues/33 which will be fixed by this PR https://github.com/PLhery/node-twitter-api-v2/pull/34


EventsGet

2nd of 3 PRs to implement sync foundations. preceded by #290
Introduces an EventsGet message to retrieve events. Still working on tests for EventsGetHandler but i wanted to kick off the review process because it can be lengthy. I imagine there will be some discussion around MessageReply.
The PR that follows this one will introduce the MessagesGet interface method

added handler tests


”Chibed” is listed in the drop-down menu

Description:
"Chibed" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Chibed" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Chibed, MUREȘ (Chibed)".

Environment: Run Tests - 4104
Browser: Chrome 123
OS: Windows 10
Steps to reproduce:
1 Open the @Adelina Guliman Test case ui.orase
2. Check TC "Chibed” is listed in the drop-down menu](https://github.com/peviitor-ro/ui.orase/issues/5098)
#5098"
Expected results:
Precondition (request):
The website link (https://peviitor-ro.github.io/ui.orase/) is up and running
The link to the law is displayed : "Sat Chibed, MUREȘ (Chibed)"

Actual result (displayed):
In STEP1 is displayed :
The location is listed in the drop-down menu as "Comuna Chibed, MUREȘ (Chibed)".

Modificat, poti retesta.


Refactor DSS editor webapp

Purpose
Refactoring DSS editor webapp.

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

 All committers have signed the CLA.


A start on more docs

Not finished yet...

Looks good to me so far. Could mention extract is the dual to return, <<= is the dual to >>=, and duplicate is the dual to join? Again maybe that's redundant though, due to being able to see from the types that the arrows are reversed from the Monad equivalents.

Ok, I think this is good to go. I also removed foreverA, since as far as I can tell, it can never work. Alternatively, we could add a Lazy1 constraint.
@garyb Thoughts?

No, come to think of it, I can't think of any useful instances which wouldn't use tailrec's version of forever instead.

If I was going to be extremely petty - we use both flipped and reversed in describing argument order again, and there's a mixture of descriptions ending with a full stop or not. But it looks great to me, nice work. :cool:

No, good points. I'll fix those up then merge, thanks.


Allow to use new version of ESLint

React forces me to use ESLint 5.6.0 on my project and I want to use 5.9.0.
Solution approaches:

Make ESLint an optional dependency so I can use it without breaking React

I think this is the best solution because I don't need the ESLint related functions in my project, I check my files through the IDE, not React.

Upgrade ESLint

This is a less efficient solution because I have to wait for you on every upgrade.

You can eject and then you'll get every tool update "for free". But the point of this project is to give you curated updates which work together well (e.g. sometimes we have to wait for ESLint or plugin bugfixes before deploying a change). So that necessarily means that updates lag behind.
Can you tell me more about why specifically you want to use 5.9.0? We'd happily take a PR with a version bump and cut a new patch if it's important to you.

You can eject and then you'll get every tool update "for free".
Can I "eject" the ESLint checking functionality of react-scripts?
Can you tell me more about why specifically you want to use 5.9.0? We'd happily take a PR with a version bump and cut a new patch if it's important to you.
There is nothing in 5.9.0 which is important, however I want ESLint to be up to date.

Can I "eject" the ESLint checking functionality of react-scripts?

Not currently. I think we'll want to allow something like this in the future though, or at least make it easier for ESLint to coexist with our linter setup. Unfortunately there are some limitations in ESLint that prevent that but we're hoping they'll get resolved in 5.x.

There is nothing in 5.9.0 which is important, however I want ESLint to be up to date.

Fair. If you send a PR with version bump I'll get it into the next patch.


Clean up now-orphaned locations after unmanaging an entity

Invoking "unmanage" operation on an entity leads to orphaned locations. More seriously, orphaned JcloudsSshMachineLocation instances still contain defunct references to the unmanaged entity, which subsequently causes dangling-reference errors on rebind.

Feedback welcome, @ahgittin and @aledsage

Build failure due to missing 0.9.x artifacts in Nexus...

Investigating test failure now...

So the unit tests assert that unmanage leaves orphaned locations. Obviously, I think this is broken behaviour, but let's see what the consensus is.

Linked issue: https://issues.apache.org/jira/browse/BROOKLYN-432

Closing this PR, as my "fix" has some undesired consequences: unmanaging a jclouds machine lcoation causes any underlying VM to be released!


Remove python2 requirement for boardctl

Description of changes

Added patch file to make boardctl compatible with Python 3
Modified board-automation derivation to apply the patch and remove the python2 dependency

Testing
nix shell github:evenfowler/jetpack-nixos/7ea7eb5a633dfe64f7f2947f91caf2a972d11590#board-automation
boardctl -t topo recovery

Looks good to me. Thanks!


Get rid off Null

Is your feature request related to a problem? Please describe.
It would be nice to get ride of null. Null is an ugly data type some developers like to avoid to minimize bugs as stated in the following book Programming Scala: Scalability = Functional Programming + Objects
by Dean Wampler and Alex Payne
Describe the solution you'd like
-1,0 out, or bool to false everything including settings like the following:
//string value of SameSite attribute of the Set-Cookie HTTP respone header
//valid value is either 'Null' (default), 'Lax', 'Strict' or 'None'
"session_same_site" : "Null",
Describe alternatives you've considered
Change the values so that -1,0, or false can be used.

Thanks for your feedback.
Personally I see no big reasons to go away from nullptr in modern C++ (in the general case), although I don’t know what the other active maintainers think.
The reasoning may be different in other languages like Scala (which I’m no expert in, but I know a similar sentiment from Swift development where you usually use optionals, and result wrappers).


Test Cases for API - Containers, DomainName and NetworkMode

Signed-off-by: Rajdeep Dua<EMAIL_ADDRESS>

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "dry-run-test" git@github.com:rajdeepd/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so DO NOT open a new one.

It doesn't have the elements i need
https://github.com/docker/docker/blob/master/api/types/types.go
type ContainerCreateResponse struct {
// ID is the ID of the created container.
ID string json:"Id"
// Warnings are any warnings encountered during the creation of the container.
Warnings []string `json:"Warnings"`

}

Hi @runcom @duglin please review the changes.
Thanks

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "dry-run-test" git@github.com:rajdeepd/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This will update the existing PR, so DO NOT open a new one.

Fixed as suggested by @runcom

@runcom gentle reminder on reviewing the changes

@rajdeepd thanks for your time, you still have to change type getResponse to types.ContainerJSON here

@rajdeepd there are other simple nits I've highlighed but overall LGTM

@rajdeepd you also need to squash your commits

@runcom since these are non sequential commits facing issues in rebase.
Shall i send a new pull request?

@rajdeepd you don't need to do another pr, in this branch just do:
git rebase -i HEAD~2
# then put s instead of pick in the second commit hash
Ctrl^x #if you use nano
# in the next editor session just choose one commit message
Ctrl^x #again
git push -f

Please let me know if you have troubles

i did a merge inbetween as i had conflicts with newer changes from others ..
git log shows
commit 0d9d2a68911b79b2eb939aca5b25c7666417a466
Merge: 94a30a3 3ea59f8
Author: Rajdeep Dua<EMAIL_ADDRESS>Date:   Sat May 2 04:20:37 2015 -0700
Added test cases for Domainname and Network Mode
Signed-off-by: Rajdeep Dua<EMAIL_ADDRESS>
commit 3ea59f8991a4d70254582759044316e522a09ee9
Merge: 3936784 576985a
Author: Jessie Frazelle<EMAIL_ADDRESS>Date:   Fri May 1 15:43:13 2015 -0700
Merge pull request #12932 from tianon/finally-remove-archive-tar-copy

Finally remove our copy of "archive/tar" now that Go 1.4 is the minimum!

commit 393678474f7364b44e7b26b5e9a99aabf1762724
Merge: 1f2f3db d963940
Author: Alexander Morozov<EMAIL_ADDRESS>Date:   Fri May 1 15:10:58 2015 -0700

@rajdeepd ok you should have rebased then, thinking about carrying this

would also need guidance for the future as we wait for reviews conflicts can arise forcing a manual merge

@rajdeepd ok kinda pretty messy here now with a merge commit carrying some modifications and another carrying others.  If you need to rebase upon upstream just run git fetch upstream master; git rebase upstream/master and if any errors occur resolve them. Then add files you edited with git add filename and then git rebase --continue until everything is fine. As a last step git push -f.
The official Docker documentation explains this very well if you're still missing something https://docs.docker.com/project/create-pr/

Thanks @runcom , will keep this in mind for my next pr

@rajdeepd Thanks to you for this contribution!


truffle compile fails on solidity-stringutils

Compiling strings.sol (https://github.com/Arachnid/solidity-stringutils) results in:
[x] $ truffle compile
Compiling Test2.sol...
Compiling strings.sol...
/usr/local/lib/node_modules/truffle/node_modules/solc/soljson.js:1
(function (exports, require, module, __filename, __dirname) { var Module;if(!Module)Module=(typeof Module!=="undefined"?Module:null)||{};var moduleOverrides={};for(var key in Module){if(Module.hasOwnProperty(key)){moduleOverrides[key]=Module[key]}}var ENVIRONMENT_IS_WEB=typeof window==="object";var ENVIRONMENT_IS_WORKER=typeof importScripts==="function";var ENVIRONMENT_IS_NODE=typeof process==="object"&&typeof require==="function"&&!ENVIRONMENT_IS_WEB&&!ENVIRONMENT_IS_WORKER;var ENVIRONMENT_IS_SHELL=!ENVIRONMENT_IS_WEB&&!ENVIRONMENT_IS_NODE&&!ENVIRONMENT_IS_WORKER;if(ENVIRONMENT_IS_NODE){if(!Module["print"])Module["print"]=function print(x){process["stdout"].write(x+"\n")};if(!Module["printErr"])Module["printErr"]=function printErr(x){process["stderr"].write(x+"\n")};var nodeFS=require("fs");var nodePath=require("path");Module["read"]=function read(filename,binary){filename=nodePath"normalize";var ret=nodeFS["readFileSync"](filen
6236544
The "truffle migration" does not complete.

Well this is an odd error (and error message). Reproduced, thanks!

I've confirmed that this is a bug with the solc compiler and not with Truffle. I've written up an example project that exposes the error as well as submitted a bug report to solc.js. Will leave this ticket open until the upstream issue is resolved, after which we'll upgrade our solc version.

Thank you, Tim.
On Aug 29, 2016 9:49 AM, "Tim Coulter"<EMAIL_ADDRESS>wrote:

I've confirmed that this is a bug with the solc compiler and not with
Truffle. I've written up an example project that exposes the error
https://github.com/tcoulter/solc-result-bug as well as submitted a bug
report https://github.com/ethereum/solc-js/issues/44 to solc.js. Will
leave this ticket open until the upstream issue is resolved, after which
we'll upgrade our solc version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ConsenSys/truffle/issues/251#issuecomment-243164929,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWMgoUlNjRhK9hllarQVpIFOnWcUXE7ks5qkv97gaJpZM4Jtaxn
.

This appears to be fixed upstream. Please reopen if you have any issues. Cheers!


Add more tests to security/pkg/server/grpc/authenticator.go (#4007)

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
ba8e4cbeb8d617b64efe287d983b390e6c91d174
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
ba8e4cbeb8d617b64efe287d983b390e6c91d174
link
/test istio-unit-tests

prow/istio-presubmit.sh
ba8e4cbeb8d617b64efe287d983b390e6c91d174
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
ba8e4cbeb8d617b64efe287d983b390e6c91d174
link
/test istio-presubmit

prow/istio-unit-tests.sh
42bfc61a79c7170643a5fa28c0df9591839d1841
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
f34b07bdd9768fcaf5d46884389ff0703193326d
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
f34b07bdd9768fcaf5d46884389ff0703193326d
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
f34b07bdd9768fcaf5d46884389ff0703193326d
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
f34b07bdd9768fcaf5d46884389ff0703193326d
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
f34b07bdd9768fcaf5d46884389ff0703193326d
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
f34b07bdd9768fcaf5d46884389ff0703193326d
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


feat: add price impact

GH Issue #164
What does this PR do?
Add the price impact for exchange
Steps to test

yarn
yarn start
navigate to http://localhost:3000/exchange
pick a pair and enter a valid amount

CheckList

[x] Follow proper Markdown format
[x] The content is adequate
[x] The content is available in both english and spanish
[x] I Ran a spell check

@adriexnet let me know when this PR is done!

@adriexnet let me know when this PR is done!

I just finished it, if it looks good for you go ahead and merge it

@adriexnet  @murillojorge
Can we adjust the line height so that the help text is not hidden behind the front layer ?

@adriexnet @murillojorge
Can we adjust the line height so that the help text is not hidden behind the front layer ?


PASSWORD_RESET_TIMEOUT_DAYS is deprecated

Hi!
PASSWORD_RESET_TIMEOUT_DAYS has been deprecated a while ago and has been removed in Django 4.
However, it is still listed in the instructions of this app.
The appropriate setting is now PASSWORD_RESET_TIMEOUT which is in seconds
Also, since I have set PASSWORD_RESET_TIMEOUT to 604800 in my app (which is 7 days), the password reset page that shows the reset link for a user in the admin says: This link can be sent [...] and it expires in 604800 days.
Can we make that either '604800 seconds' or better yet '7 days' ?
By the way, if I still try to use PASSWORD_RESET_TIMEOUT_DAYS, that settings seems to be ignored because then it says " and it expires in 259200 days." (which is the default 3 days instead of 604800 which would be 7 days)

Thanks for the report. This is now fixed in version 0.7.0.

Thank you!
On Sun, 6 Aug 2023, 22:36 tiborhari, @.***> wrote:

Closed #10 https://github.com/tiborhari/django-admin-reset/issues/10 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/tiborhari/django-admin-reset/issues/10#event-10017898643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADV345EGGREVECA5DSHKFLXT753XANCNFSM6AAAAAA2PVU7JI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Kitchen Sink: Build A Pomodoro Timer App

Build a Pomodoro timer for productivity that you'll add to the kitchen sink. Feel free to use any existing projects of yours or start one from scratch.
For more information about the Kitchen sink; check out:

https://sink.million.dev/
https://github.com/aidenybai/million/blob/main/packages/kitchen-sink/README.md

N.B: Upon PR review, your usage of Million.js is a plus

@tobySolutions could you please assign this issueto me. I would like to work on it.

Can pls assign it to me ?

Hey there @arunavabasu-03, @Devanshukoli requested first; I think it's only fair that we let them have this.

Issue fixed by #695


[BUG] config requires property "detectors"

Describe the bug
When loading the config, I get the error message 'config requires property "detectors"'
Version of Double Take
v1.13.4
Expected behavior
Load DT
Hardware
Docker container
Additional context
I can jump onto the UI for Frigate, Double Take and CodeProject
Config:
# Double Take
# Learn more at https://github.com/skrashevich/double-take/#configuration

mqtt:
  host: <IP_ADDRESS>
  username: mqtt-user
  password: XXXXX

frigate:
  url: http://<IP_ADDRESS>:5000
  update_sub_labels: True

# detector settings (default: shown below)
detectors:
  aiserver:
    url: http://<IP_ADDRESS>:32168
    # number of seconds before the request times out and is aborted
    timeout: 15
    # require opencv to find a face before processing with detector
    opencv_face_required: false
    # only process images from specific cameras, if omitted then all cameras will be processed
    # cameras:
    #   - front-door
    #   - garage

I can't reproduce the error. Please give more information about your environment:

host architecture and OS
docker run command or docker-compose file contents
Any other data that may affect this problem

Strange.
So my install is Proxmox 7.4-3, Docker is running in an LXC container. LXC is Debian 11.
Docker inside that container is running:

CompreFace
Frigate
Double Take - official version
Double Take - your version
CodeProject

I dont have both Double Take versions running at the same time.
I tried installing the official Double Take and that one works with the following config:
# Double Take
# Learn more at https://github.com/skrashevich/double-take/#configuration

frigate:
  url: http://<IP_ADDRESS>:5000
  update_sub_labels: True

mqtt:
  host: <IP_ADDRESS>
  username: mqtt-user
  password: XXXXXX

detectors:
  compreface:
    url: http://<IP_ADDRESS>:8000/
    # recognition api key
    key: XXXXXX
    # number of seconds before the request times out and is aborted
    timeout: 15
    # minimum required confidence that a recognized face is actually a face
    # value is between 0.0 and 1.0
    det_prob_threshold: 0.8
    # require opencv to find a face before processing with detector
    opencv_face_required: false
    # comma-separated slugs of face plugins
    # https://github.com/exadel-inc/CompreFace/blob/master/docs/Face-services-and-plugins.md)
    # face_plugins: mask,gender,age
    # only process images from specific cameras, if omitted then all cameras will be processed
    # cameras:
    #   - front-door
    #   - garage
  # aiserver:
  #   url: http://<IP_ADDRESS>:32168
  #   # number of seconds before the request times out and is aborted
  #   timeout: 15
  #   # require opencv to find a face before processing with detector
  #   opencv_face_required: false
telemetry: false

detect:
  match:
    # save match images
    save: true
    # include base64 encoded string in api results and mqtt messages
    # options: true, false, box
    base64: false
    # minimum confidence needed to consider a result a match
    confidence: 80
    # hours to keep match images until they are deleted
    purge: 48
    # minimum area in pixels to consider a result a match
    min_area: 2500
    #when using substream image, 3600

  unknown:
    # save unknown images
    save: true
    # include base64 encoded string in api results and mqtt messages
    # options: true, false, box
    base64: false
    # minimum confidence needed before classifying a name as unknown
    confidence: 40
    # hours to keep unknown images until they are deleted
    purge: 8
    # minimum area in pixels to keep an unknown result
    min_area: 0

If I run your version of Double Take with the exact same config I get that error.
The compose file for running your version of Double Take is:
version: '3.7'

volumes:
  double-take:

services:
  double-take:
    container_name: double-take
    image: skrashevich/double-take
    restart: unless-stopped
    volumes:
      - /opt/double-take:/.storage
    ports:
      - 3000:3000

The compose file for running your version of Double Take is:

compose file for running original Double Take is exactly same, instead of "image" tag ?

Yes, same:
version: '3.7'

volumes:
  double-take:

services:
  double-take:
    container_name: double-take
    image: jakowenko/double-take
    restart: unless-stopped
    volumes:
      - /opt/double-take:/.storage
    ports:
      - 3000:3000

Hmmmm, I have just played around with the containers, stopping official Double Take, pulling up to date CodeProject etc - I can now run your version of Double Take with the following config (effectively same as before but removing aiserver comments):
# Double Take
# Learn more at https://github.com/skrashevich/double-take/#configuration

frigate:
  url: http://<IP_ADDRESS>:5000
  update_sub_labels: True

mqtt:
  host: <IP_ADDRESS>
  username: mqtt-user
  password: Cheryl198!

detectors:
  compreface:
    url: http://<IP_ADDRESS>:8000/
    # recognition api key
    key: c11aa2c0-ebf8-4894-8124-93953539ff58
    # number of seconds before the request times out and is aborted
    timeout: 15
    # minimum required confidence that a recognized face is actually a face
    # value is between 0.0 and 1.0
    det_prob_threshold: 0.8
    # require opencv to find a face before processing with detector
    opencv_face_required: false
    # comma-separated slugs of face plugins
    # https://github.com/exadel-inc/CompreFace/blob/master/docs/Face-services-and-plugins.md)
    # face_plugins: mask,gender,age
    # only process images from specific cameras, if omitted then all cameras will be processed
    # cameras:
    #   - front-door
    #   - garage
  aiserver:
    url: http://<IP_ADDRESS>:32168
    # number of seconds before the request times out and is aborted
    timeout: 15
    # require opencv to find a face before processing with detector
    opencv_face_required: false
telemetry: false

detect:
  match:
    # save match images
    save: true
    # include base64 encoded string in api results and mqtt messages
    # options: true, false, box
    base64: false
    # minimum confidence needed to consider a result a match
    confidence: 80
    # hours to keep match images until they are deleted
    purge: 48
    # minimum area in pixels to consider a result a match
    min_area: 2500
    #when using substream image, 3600

  unknown:
    # save unknown images
    save: true
    # include base64 encoded string in api results and mqtt messages
    # options: true, false, box
    base64: false
    # minimum confidence needed before classifying a name as unknown
    confidence: 40
    # hours to keep unknown images until they are deleted
    purge: 8
    # minimum area in pixels to keep an unknown result
    min_area: 0

This starts up without any error messages, and CodeProject is running fine however I dont seem to be able to get Double Take to talk to CodeProject - the status dots at top of config page isnt showing CodeProject:

To rule out browser cache, I opened all the sessions in new Incognito windows, same issue (missing aiserver/CodeProject as a status

Are you first installed the original double-take, run it at least once, then run my version in the same volume?

and ensure that you have latest version of container: docker pull skrashevich/double-take:latest. A few days ago, I updated the container in the process of diagnosing another problem, but it could also solve your's.

Perfect! Added latest to the image name seems to have worked - I now have a status for both detectors 😊
I had been using the original Double Take volume and config and just starting/stopping appropriate container - definitely the image is what has fixed my issue.
Thank you for your help, it is much appreciated.
Was you thinking of trying to get your version merged back into the original Double Take? I’m sure a lot of people could benefit from it, especially once CodeProject have coral support on other devices rather than just RPI (which I believe is actively being worked on at the moment) - coral usage is what prompted me to look for a detector that will utilise the coral alongside Frigate

Was you thinking of trying to get your version merged back into the original Double Take? I’m sure a lot of people could benefit from it, especially once CodeProject have coral support on other devices rather than just RPI (which I believe is actively being worked on at the moment) - coral usage is what prompted me to look for a detector that will utilise the coral alongside Frigate

I did important pull-requests to the original project, but there has been no activity for more than six months.  I didn’t realise it is no longer maintained.
Where I have both CompreFace and CodeServer now working, any idea why CodeServer is constantly returning 0% match for all images sent via DT? If I download the original image from Frigate and DT and upload that to face recognition in CodeProject it get either a match or at least a higher than 0% for images from both sources (DT and Frigate)


shawdowoxygen not working

when i run it it never checks any of the accounts no matter what i do

Make sure the config is using the right type of proxies,  and also make sure your proxies are working

ok thanks ill see

YEY it works ty


Visual Basic code

What is the VB code for Creating the class library project?

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

ID: 826876cc-d286-3e10-515e-bba12d85f1f4
Version Independent ID: 5dfbe5a8-f7be-617d-c9ff-db2fa2d6a396
Content: Build a C# .NET Standard library with  Visual Studio 2017 - .NET Core
Content Source: docs/core/tutorials/library-with-visual-studio.md
Product: dotnet-core
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@RichEme Hi! It is the next article listed in the TOC on the left side. Here is a link: https://docs.microsoft.com/en-us/dotnet/core/tutorials/vb-library-with-visual-studio
I'll close this issue as I think I've answered your question.

Thanks. That fixed it.


where is position loss and opacity loss?

In paper, about Loss_refine = L_sds+L_mean+Lopacity。 so where is mean loss and opacity loss in trainer code?

hello，have you solved it?

no
---Original---
From: @.>
Date: Mon, Sep 23, 2024 22:53 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [gsgen3d/gsgen] where is position loss and opacity loss? (Issue#38)
hello，have you solved it?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


preventMarkerRemoval bug? - Prevent removal of first and last vertex on polyline and polygon?

Using preventMarkerRemoval right click to remove vertex, you can accidentally delete a Polyline or polygon.
For example Polyline requires 2 vertexes. If you right click it is "deleted" (actually still an object but cant be visible on map anymore because of missing position).
In my opinion it should not be allowed to remove first and last vertex of polyline?
The issue is the same for polygon if you dont have a total of 3 vertexes left. Not sure how to fix polygon tho..
Do you guys agree that this should be prevented?

This is the expected behaviour.
The layer has no more coords and is also removed from the map. So there is no more reference on the map to the layer -> layer is deleted.
If this is wanted to prevent we can add a option for this.

I think it would be benificial  for geoman to have an option for it. And maybe an option to set an amount of time you need to hold the vertex until it deletes on touch devices?

Good idea, I will take a look on it

Just want to give kudos @Falke-Design for all your help! Latest release 2.8.0 works great as well!

Would it be a good idea to also listen for if marker.on('move', this._onMarkerDrag, this); has been fired? I think so to further prevent deletion of vertex.
For example, lets say long-tap requires to hold for 2 seconds. Then you drag around the vertex, releases after 3 seconds, now the vertex is deleted.
Listen to onMarkerDrag: drag the vertex for 3 seconds, (markerDrag is firing while dragging, preventing the long tap). Vertex is not deleted.
Or maybe there is a better way to handle deletion of vertex on touch overall?


Drop dependency on synom

Synom will need to depend on proc-macro2 as it transitions to parsing tokens instead of strings. And all of this code is being removed from synom anyway in https://github.com/dtolnay/syn/pull/147.

I just copied things over from synom 0.11.3 until it compiled. We can pare this down later if necessary.

👍

My infinite bootstrap plans, noooooooooooo


Update is04 json schemas

A few minor fixes were missing.

As per the naming of the directories in nmos-cpp, they were supposed to reflect released spec versions and the changes you've included haven't made it into a spec patch release yet. But it is awkward as I agree it's better to be on the bleeding edge of the ".x" branches usually, just makes it hard to track. Hmm

Master is now sync'd with the spec ".x" branches not the release tags. Thanks.

Thank you very much, I was not aware of your convention.


Rewrite progress

Commands

[x] ban
[x] delban
[x] desc
[x] fetch

[x] normalize and validate fields where necessary

[x] git
[x] info
[x] latency
[x] modping
[x] move

[x] only repeat pings

[x] mute

[x] mutes should be logged in modlog
[x] response should probably be an embed

[x] note
[x] notes

[x]  mutes should show the duration

[x] pfp
[x] repo
[x] restart
[x] say
[x] setfetch
[x] warn

[x] show warn count in warn log embed

[x] blocklist
[x] top
[x] poll
[x] purge
[x] role
[x] help

[x] support !help info etc

[x] unban

Event-handlers

[x] join events
[x] leave events
[x] message edits
[x] message deletion

Other functionality

[x] poll single-vote enforcement
[x] include context links everywhere it makes sense
[x] keep-at-bottom CONTRIBUTING.md message in server-feedback
[x] Media message logging

Other TODO

[x] Consistent embed format with timestamp etc

[x] improve sql

[x] handle "upsert"s in profile via SQL
[x] de-shittify SQL query for notes

[x] make embed formatting use things like bold etc

[x] use embed for !setfetch explanation

[x] show description at the top of fetch

[x] fix setfetch

[x] !git and !dotfiles should enforce URL format

[ ] !git and !dotfiles should print your own git and dotfiles when ran without arguments

[x] fix memory unit in fetch to use KB rather than KiB

[x] !help should only show non-mod commands for non-mods

[x] if no mods are online/active, modping should ping the mods role

[x] modping should only ping online mods

[x] detect who deleted a message and add that to the bot-messages entry

[x] Numbers should not always be treated as userids - users could be named 502, for example.

[x] error message for unknown user sucks

[x] silent purge? (should this be always, or as an option?)

[x] #feedback posts have the thumbs in the wrong order

[x] bulk-deletions need to be logged in #bot-messages

[x] Helper-specific permissions

[x] spam-detection

[x] role-mentions should be handled by !role

[x] success embed should be green

[x] disambiguation and polls should be able to use the full 20 reactions that are possible

[x] replace "banned" in ban-logs with "yote" because funny

[x] replace "yeeted" with "yote"

[x] ban should obviously not delban

[x] update fetch image on showcase post

[x] fix formatting of Memory in !top

[x] insufficient permissions error message is shit (requires helper_or_mod)

[x] notes does now ago

[x] blocklist is missing usage

[x] Memorys in top (use s/ys$/ies/)

[x] fetch <person> image don't work :(

[x] showcase submissions should be allowed to contain anything that embeds, no?

[x] !roles command as an alias to !role

[x] send :poggersphisch:, :police: and :starecry: emotes for the respective success,  and error messages

[x] notes are fucked (migration script issue)

[ ] ghostping thing should use :reee: and not work in showcase (do we want ghostping detection? currently just disabled it, idk)

[x] top when doing like !top Distro [aA]rch without the backticks needs a good error message

[x] !move should not ping the moving user

[x] blocklist violations don't add notes currently

[ ] spam-protection for all accounts, not just young ones (let's not, for now)

[x] setfetch etc don't check blocklist before running (does this really need fix? :starecry:)

[x] fetch migration - spaces etc in field names need to be adjusted, currently still fucked. i.e. DisplayProtocol vs "Display Protocol"

[x] tag with spaces in name cannot be looked at :(

[x] Failed unmute does not set mute as inactive, meaning that it'll continuously try to unmute them again and again

[x] usage of blocklist should mention the backticks

[x] error while handling reaction_addd event

[x] !desc, !git and !dotfiles should gib error when not set  -> migration issue: empty strings in profiles table should be NULL

[x] !poll multi literally just ded (no reactions)

[x] be a bit less correct in the message.txt message

[ ] maybe use the bucket shit to avoid excessive rate limiting? idk prolly don't matter

[x] command specific help should list subcommands properly

[x] !poll  single vote enforcement broken?????

[x] !poll without args should print usage (currently just deletes the msg)

[x] help for mods is hitting the field limit - use pagination? two mesages?

Rewrite is generally done! This can be closed, as everything that comes up now will be a normal feature/bugticket!


Show remaining character count in polls

It should be clear how many characters are left for poll options and the optional prompt

Even just a maxlength of 64 on the options and 256 on the prompt would be fine.


Fix Exercise 2.7

The literal 1024f generate a compile error because the suffix f is only valid with floating point literals i.e  1024.f, 1024E0f, etc.

Thx.


Collect kubectl information in AKS Windows node log

What type of PR is this?
/kind feature

What this PR does / why we need it:
Collect kubectl information in AKS Windows node log
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.334%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2288

Relevant Lines:
2884

💛  - Coveralls


Show status of all API servers?

Show status of all API servers on init-error page and/or config page and/or a dedicated page? Status includes: up/down, latency, load(if apply), location(if apply), and maybe an description. So user can select fastest server, or maybe eventually auto-select.
Related to #770.

GUI should ask automatically (timeout) to switch API connection after lost connection/blockchain sync problems.

This issue was moved to bitshares/bitshares-ui#49


Add default implementation for other UITableViewDelegate/UICollectionViewDelegate methods

Checklist

[x] All tests are passed.
[x] Added tests.
[x] Documented the code using Xcode markup.
[x] Searched existing pull requests for ensure not duplicated.

Description
As per #16 , when implementing a table view delegate there are currently some methods that aren't implemented. It is currently possible to add these methods by using @objc(…) modifier, but I feel like a default, open for overriding implementation achieves a better experience after all.
Related Issue
Introduction of default implementations for other UITableViewDelegate and UICollectionViewDelegate methods -  #16
Impact on Existing Code
No impact on existing code.

Thanks @bobek-balinek !
Nicely done 👍


Fixes #3420 (partial) - Adding PowerShell modules and initial Analyzer projects

Initial PR for analyzer work.
This adds PowerShell modules for use when developing Terminal.Gui. Future enhancements are planned and suggestions are more than welcome.
Analyzer projects are in this PR but the projects are not included in the solution yet, nor referenced by any other projects in the solution.
This PR is ready to go as-is, or I can add the solution and project file changes to wire it all up before merge if desired.
Proposed Changes/Todos

[x] Add powershell modules/scripts to perform routine operations when developing Terminal.Gui.

[x] Terminal.Gui.PowerShell module: Environment and solution functionality.

[x] Open-Solution: Launches the Terminal.sln file
[x] Close-Solution: Closes running devenv.exe processes with Terminal.sln loaded
[x] Set-Environment: Sets up a standard environment for script execution, in a temporary session.
[x] Reset-Environment: Undoes changes made by Set-Environment or just exits the inner session, if requested.

[x] Terminal.Gui.PowerShell.Analyzers module: Functionality related to analyzer projects.

[x] Build-Analyzers: Builds analyzer projects in Debug and Release mode, with additional functionality exposed via switch parameters.

[x] Add analyzer project with basic enum extension analyzer and generator
[x] Add test project for the analyzer project
[x] Add debug project for the analyzer project

Pull Request checklist:

[x] I've named my PR in the form of "Fixes #issue. Terse description."
[x] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.

Well... It's PowerShell, so it's different, but it's pretty similar.

[x] My code follows the Terminal.Gui library design guidelines
[x] I ran dotnet test before commit
[x] I have made corresponding changes to the API documentation (using <# #> style comments)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any poor grammar or misspellings
[x] I conducted basic QA to assure all features are working

There we go. Now it's ready. :)

Some future additions I've considered so far and may add at some point:

Build, clean, and test functionality for the solution and maybe also for specific projects, to ensure consistent builds.
Helpers for standard git operations, like branching for an Issue and stuff like that.
Running the JetBrains formatter tool on changed files (as reported by git, in the current branch), maybe? And maybe fetching it if it isn't already available locally?

Actually... This does include part of the analyzer projects, too. I'll rename accordingly.

Updated to match what's actually contained in the PR.
I haven't pushed the solution and other csproj file changes to use the analyzers, in this PR, for a couple of reasons:

So everyone else can at least build them and make sure that works in their environments.
I want to make some more changes to the analyzers before we include them in the pipeline.

If those are unimportant to y'all, and you'd rather I just wire them up now, I'll be happy to do so. Otherwise, that'll come in a separate PR when it's ready.

I've tried to read all your posts on these things, but I suck at learning by just reading. I need to party on the code.
How do I use all this stuff "in anger"?

Please add README.md's to ./Scripts and ./Analyzers explaining their purpose and how they are used.

No problemo. Will do this evening most likely.

I've tried to read all your posts on these things, but I suck at learning by just reading. I need to party on the code.
How do I use all this stuff "in anger"?

No worries. That's part of why I didn't go ahead and include it in the solution yet.
Visual Studio problems with Roslyn projects
Visual Studio's Roslyn host is still a .net framework host (why it's netstandard2.0 required) and can't unload one once it's loaded. That has two main impacts:

The first load of the solution from a fresh or dotnet cleaned state will be oddly and unfixably broken. Have to build the roslyn projects and reload VS.
If you make any changes to the Roslyn project code that you want to then use outside of the associated "Debug" project, you also need to rebuild them and reload VS.

That's not reload the solution - any VS process with the solution or the Terminal.Gui.csproj opened have to be closed to unload the old assemblies.
Scripts to the rescue!
That's where the scripts came in - to simplify that.
I wanted to turn that multi-step process into as few as possible. And we're dotnet and it appears a powershell script was at least prototyped at some point in the repo, plus PS is a full and cross-platform dotnet environment, so PS is natural all around.
So I got it down to just a single command that kicks it all off.
And then, through some combination of wanting to be flexible, wanting to write good scripts, wanting to make it easy, wanting to make it re-usable, and some good old-fashioned going a bit overboard, I make them modules, with the intent of adding more over time, including a new build function, which there's a non-working prototype definition of in the current code I have, for people to play around with.
I might even throw up a private PSRepository to mirror the current modules, but signed so PowerShell doesn't complain without you having to Set-ExecutionPolicy -ExecutionPolicy Unrestricted.
More to come
I've actually got even more sprucing up of those, as well, that I'll push into this PR, as well as the requested markdown files.
I went back and re-did some things that organically outgrew their original quick-and-dirty versions, and which no longer fit how I ended up having them all formally defined and organized. So now they are consistent and follow recommendations and literal requirements for powershell (some of which actually are hard requirements and others of which are just cardinal sins), as well as gained some more features and flexibility while I was at it.
Now just import the module and then run whatever commands you want that it exports.
How to use for the source generators
If you already have VS open, that's fine. The script can close it if it was launched with Terminal.sln or currently has Terminal.sln loaded.
Open powershell. Must be PS Core, version 7.4.0 or higher (current is 7.4.2 but that doesn't matter). If you don't know how to check, don't worry. The modules will validate the environment on attempt to import.
Navigate to your repo directory in the powershell session, and into the Scripts directory.
Run this, to import all currently-defined and exported modules and their dependencies.
Import-Module ./Terminal.Gui.PowerShell.psd1

Notice that your prompt will have been slightly modified to indicate you're in our modified environment. No environment changes are permanent. When you Remove the module, call a certain function, or just close the session, the prompt and anything else changed when importing the modules will be back to their original states.
Now play!
You can get the list of Terminal.Gui modules and available commands loaded in the current session with:
Get-Command -Module Terminal.Gui.PowerShell

To get full help for any command, run Get-Help CommandName with or without optional switches for mor info like -Full, -Detailed, and -Examples. -Full is everything including all others.
Example:
Get-Help New-GitBranch -Full

TGPS V:\repos\Terminal.Gui\Scripts> Get-Help New-GitBranch -Full

NAME
    New-GitBranch

SYNOPSIS
    Creates a new branch with the specified name.

SYNTAX
    New-GitBranch -Name <String> [-Push] [-WhatIf] [-Confirm] [<CommonParameters>]

    New-GitBranch -Name <String> -Push [-Remote <String>] [-WhatIf] [-Confirm] [<CommonParameters>]

    New-GitBranch -Name <String> -NoSwitch [-WhatIf] [-Confirm] [<CommonParameters>]

DESCRIPTION
    Creates a new branch with the specified name.

PARAMETERS
    -Name <String>
        The name of the new branch.
        Always required.
        Must match the .net regex pattern "v2_\d{4}_[a-zA-Z0-9()_-]+".
        Must also otherwise be a valid identifier for a git branch and follow any other project guidelines.

        Required?                    true
        Position?                    named
        Default value
        Accept pipeline input?       false
        Accept wildcard characters?  false

    -NoSwitch [<SwitchParameter>]
        If specified, does not automatically switch to your new branch after creating it.
        Default is to switch to the new branch after creating it.

        Required?                    true
        Position?                    named
        Default value                False
        Accept pipeline input?       false
        Accept wildcard characters?  false

    -Push [<SwitchParameter>]
        If specified, automatically pushes the new branch to your remote after creating it.

        Required?                    false
        Position?                    named
        Default value                False
        Accept pipeline input?       false
        Accept wildcard characters?  false

    -Remote <String>
        The name of the git remote, as configured.
        If you never explicitly set this yourself, it is typically "origin".
        If you only have one remote defined or have not explicitly set a remote yourself, do not provide this parameter; It will be detected automatically.

        Required?                    false
        Position?                    named
        Default value
        Accept pipeline input?       false
        Accept wildcard characters?  false

    -WhatIf [<SwitchParameter>]

        Required?                    false
        Position?                    named
        Default value
        Accept pipeline input?       false
        Accept wildcard characters?  false

    -Confirm [<SwitchParameter>]

        Required?                    false
        Position?                    named
        Default value
        Accept pipeline input?       false
        Accept wildcard characters?  false

    <CommonParameters>
        This cmdlet supports the common parameters: Verbose, Debug,
        ErrorAction, ErrorVariable, WarningAction, WarningVariable,
        OutBuffer, PipelineVariable, and OutVariable. For more information, see
        about_CommonParameters (https://go.microsoft.com/fwlink/?LinkID=113216).

INPUTS
    None

OUTPUTS
    The name of the current branch after the operation, as a String.
    If NoSwitch was specified and the operation succeeded, this should be the source branch.
    If NoSwith was not specified or was explicitly set to $false and the operation succeeded, this should be the new branch.
    If an exception occurs, does not return. Exceptions are unhandled and are the responsibility of the caller.

NOTES

        Errors thrown by git commands are not explicitly handled.

All commands have kinda excessive help defined. Side note: I like how they do it in pseudo-mandoc style.
And the above example is in the code I have now (not pushed yet), and will do exactly what it says. Makes a new branch with the specified name, enforcing the v2_####_etc naming restriction, and pushes it to your remote. Or modifications to that behavior as described in the help, and from what I hope are pretty obviously-named parameters in the first place.
And remember PS does tab completion, and any of these that have parameters will help you as much as they can with that.
When I push again, all of the currently-defined commands will be in a working state.
Right now, that list is:
TGPS V:\repos\Terminal.Gui\Scripts> get-command -Module Terminal.Gui.PowerShell

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Function        Build-Analyzers                                    1.0.0      Terminal.Gui.PowerShell
Function        Build-TerminalGui                                  1.0.0      Terminal.Gui.PowerShell
Function        Close-Solution                                     1.0.0      Terminal.Gui.PowerShell
Function        New-GitBranch                                      1.0.0      Terminal.Gui.PowerShell
Function        Open-Solution                                      1.0.0      Terminal.Gui.PowerShell

With the code currently pushed, you can use the Build-Analyzers command, already. It's just a more messy environment, so hang tight for a little bit while I write up some basic README.md files and at least prototype the Build-TerminalGui command beyond just an empty method.
Alrighty. Back to it.

Oh another thing to mention up front:
Since these are intended mostly for our benefit at dev time, I don't have particular interest in supporting a wide range of environments.
Thus, the module manifests require that it is PS Core, Version 7.4.0 or higher, and the host process must be ConsoleHost. That means it is running in a terminal and not hosted within some other application, because that just opens up a ton of stuff to deal with that isn't important for us and can be worked around by simply running pwsh from within whatever that is.
For example, attempting to import the module from a powershell panel inside VS Code results in this:
Import-Module: The name of the current PowerShell host is: 'Visual Studio Code Host'. The module 'V:\repos\Terminal.Gui\Scripts\Terminal.Gui.PowerShell.psd1' requires the following PowerShell host: 'ConsoleHost'.

ConsoleHost is what pwsh identifies itself as no matter what it is running in.
So, simply calling pwsh first, before import-module, is acceptable to powershell.

A little light reading... 😅

Makes sense. I think.
For normal dev I won't have to use the scripts, right? Only if I need to modify the analyzers?

Right. They're just for convenience and for the analyzers.
First time using them, once added to solution, you'll just run Build-Analyzers
Only have to call that again if you change them or accidentally clean the solution or basically anything that would alter the assemblies produced by the analyzer projects.

I just pushed a partial update that includes the reorg, a README.md in the Scripts directory, and everything but Build-TerminalGui currently functional.
Build-TerminalGui just writes output saying it doesn't work yet. But it's still testable in terms of invocation/parameters/whatev.

I've tested these on Ubuntu 23.10 and Windows 11 Enterprise, so far.
They do what I intend them to do, but I wrote them, so what made sense to me may or may not make as much sense to someone else.
So feedback more than welcome for sure.

Side note not for this PR:
I usually also provide a Makefile for convenience and comfort of Linux users, for these sorts of things, that essentially just wrap the same stuff so that those used to just running make can do what they're used to and those from dotnet land can use what they are used to.
I'll probably write those like... way down the line, closer to release time.


Comfort Eagle

We are building a codebase
We are building it bigger
We are widening the coverage
And adding more tests

Codecov Report

Merging #201 into master will increase coverage by 2.55%.
The diff coverage is 94.32%.

@@            Coverage Diff             @@
##           master     #201      +/-   ##
==========================================
+ Coverage   87.63%   90.19%   +2.55%
==========================================
  Files          87       88       +1
  Lines        3721     3733      +12
  Branches      723      730       +7
==========================================
+ Hits         3261     3367     +106
+ Misses        455      364      -91
+ Partials        5        2       -3

Impacted Files
Coverage Δ

src/data/CodonTable.ts
100% <ø> (ø)
:arrow_up:

src/index.ts
0% <ø> (ø)
:arrow_up:

src/data/IUPACData.ts
100% <ø> (ø)
:arrow_up:

src/component/widget/BioblocksSlider.tsx
100% <100%> (ø)
:arrow_up:

src/component/UMAPVisualization.tsx
92.82% <100%> (ø)

src/container/index.ts
100% <100%> (ø)
:arrow_up:

src/component/index.ts
100% <100%> (ø)
:arrow_up:

src/helper/DataHelper.ts
85.62% <45.45%> (-2.84%)
:arrow_down:

src/container/UMAPTranscriptionalContainer.tsx
94.52% <94.52%> (ø)

src/container/UMAPSequenceContainer.tsx
98.98% <98.98%> (ø)

... and 9 more

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 75c6d30...fbe90ca. Read the comment docs.


Intrinsics support for Zihintntl extension

The Zihintntl extension has recently passed AR.  The spec is here: https://github.com/riscv/riscv-isa-manual/blob/10eea63205f371ed649355f4cf7a80716335958f/src/zihintntl.tex
During the AR, we wanted to raise the issue of whether and how the extension would be exposed in the RISC-V C API.  Can y'all ponder the following and opine?
In x86, for example, _mm_stream_pi (https://github.com/gcc-mirror/gcc/blob/e75da2ace6b6f634237259ef62cfb2d3d34adb10/gcc/config/i386/xmmintrin.h#L1279-L1291) is roughly equivalent to c.ntl.all; sd in RISC-V.
(ARMv8 has LDNP/STNP instructions, but I couldn't find an intrinsics mapping for them.)
Zihintntl is more general than x86's solution in a few dimensions:

NTL hints can be used on both loads and stores (in x86, there are only non-temporal stores, AFAIK)
NTL hints can be used on any kind of load and store (in x86, it's only MMX/SSE/AVX, AFAIK)
NTL hints can express a memory hierarchy level (x86 can only express the equivalent of ntl.all, AFAIK)

With this in mind, the questions for the RISC-V C API folks are: how do we expose this facility in the RISC-V C API?  How much of its generality do we expose?  Do you foresee any impediments?
cc @kito-cheng @ptomsich

The specification requires the memory access to be the "immediately subsequent instruction".
This is hard/impossible to guarantee by an intrinsic that does not include the memory access operation.
Therefore I would include the memory access in the API.
I see two possible solutions.
Proposal one: provide NTL loads and stores (similar to the atomics builtins):
 type __riscv_ntl_load (type *ptr, enum ntl_domain domain);
void __riscv_ntl_store (type *ptr, enum ntl_domain domain);

However, this will probably only work reasonably well for single-GPR/FPR-memory transfers. E.g. vector memory accesses probably need a _ntl variant of the vector load/store intrinsics.
Proposal two: no intrinsics, but a NTL function attribute that does not block inlining
// all memory accesses in this function emit an NTL hint before the memory access instruction
__attribute__((target("ntl_domain=DOMAIN"))
static inline void
my_read_buf (uint8_t* buf, size_t n_bytes, uint8_t *src)
{
    __builtin_memcpy(buf, src, n_bytes);
}

I was also envisioning the intrinsic would also contain the load or store. This matches how the non-temporal store intrinsics work on x86: the intrinsic actually performs the store, rather than annotating a separate assignment.

Proposal one: provide NTL loads and stores (similar to the atomics builtins):

That sounds good to me.

Proposal two: no intrinsics, but a NTL function attribute that [does not block inlining] (https://gcc.gnu.org/bugzilla//show_bug.cgi?id=101279)

I don't like idea of function attribute approach but that inspire me another possible solution for that: variable attribute:
uint8_t* buf __attribute__ ((ntl_doman=DOMAIN));

and any load store with pointer with this attribute will add a hint instruction.

I like the pointer attribute approach, if it's feasible to implement.
And of course the x86-style intrinsic can be implemented using the pointer attribute approach with a simple wrapper function.

X86 has MOVNTI instruction for non-temporal store of GPR as part of SSE2.
X86 has MOVNTDQA for non-temporal vector load.

I don't like idea of function attribute approach but that inspire me another possible solution for that: variable attribute:
uint8_t* buf __attribute__ ((ntl_doman=DOMAIN));

and any load store with pointer with this attribute will add a hint instruction.

Yes, that's a better idea than a function attribute!

I like the pointer attribute approach, if it's feasible to implement.

We need to make sure the implement effort on both compiler for variable attribute, I saw load/store in LLVM IR has encode non-temporal, but we might need to extend that to able to express different domain, so I think we need introduce new intrinsic for NTL load/store at first stage.
https://llvm.org/docs/LangRef.html#load-instruction
<result> = load [volatile] <ty>, ptr <pointer>[, align <alignment>][, !nontemporal !<nontemp_node>][, !invariant.load !<empty_node>][, !invariant.group !<empty_node>][, !nonnull !<empty_node>][, !dereferenceable !<deref_bytes_node>][, !dereferenceable_or_null !<deref_bytes_node>][, !align !<align_node>][, !noundef !<empty_node>]
<result> = load atomic [volatile] <ty>, ptr <pointer> [syncscope("<target-scope>")] <ordering>, align <alignment> [, !invariant.group !<empty_node>]
!<nontemp_node> = !{ i32 1 }
!<empty_node> = !{}
!<deref_bytes_node> = !{ i64 <dereferenceable_bytes> }
!<align_node> = !{ i64 <value_alignment> }

In any case, it seems we have a path to some solution.

SiFive folks is implementing builtin now.

Looks like we should also wire this up to the storent-optab.
Here's the equivalent patterns for x86:
; Expand patterns for non-temporal stores.  At the moment, only those
; that directly map to insns are defined; it would be possible to
; define patterns for other modes that would expand to several insns.

;; Modes handled by storent patterns.
(define_mode_iterator STORENT_MODE
  [(DI "TARGET_SSE2 && TARGET_64BIT") (SI "TARGET_SSE2")
   (SF "TARGET_SSE4A") (DF "TARGET_SSE4A")
   (V8DI "TARGET_AVX512F") (V4DI "TARGET_AVX") (V2DI "TARGET_SSE2")
   (V16SF "TARGET_AVX512F") (V8SF "TARGET_AVX") V4SF
   (V8DF "TARGET_AVX512F") (V4DF "TARGET_AVX") (V2DF "TARGET_SSE2")])

(define_expand "storent<mode>"
  [(set (match_operand:STORENT_MODE 0 "memory_operand")
        (unspec:STORENT_MODE
          [(match_operand:STORENT_MODE 1 "register_operand")]
          UNSPEC_MOVNT))]
  "TARGET_SSE")

Proposal for the intrinsic: https://github.com/riscv-non-isa/riscv-c-api-doc/pull/47


Support sphinx-hoverxref

https://sphinx-hoverxref.readthedocs.io/en/latest/configuration.html#confval-hoverxref_domains

This extension requires a backend server to retrieve the tooltip content. Currently, only Read the Docs is supported as backend server. Take into account that your documentation has to be hosted on Read the Docs for this extension to work. [1]

So we can not support hoverxref :'(
[1] https://sphinx-hoverxref.readthedocs.io/en/latest/installation.html


Deploying problem to Hololens

Hi guys,
I got the following error when I try to deploy the app from the Visual Studio to Hololens:
Schweregrad	Code	Beschreibung	Projekt	Datei	Zeile	Unterdrückungszustand Fehler		Der Befehl ""C:\Users\user\Downloads\MixedRealityToolkit-Unity-master\MixedRealityToolkit-Unity-master\Gaze_HolotoolKit\Unity\Tools\AssemblyConverter.exe" -platform=uap -lock="C:\Users\user\Downloads\MixedRealityToolkit-Unity-master\MixedRealityToolkit-Unity-master\Gaze_HolotoolKit\MixedRealityToolkit-Unity\project.lock.json" -bits=32 -configuration=Debug -removeDebuggableAttribute=False -uwpsdk=10.0.16299.0 -path="." 	MixedRealityToolkit-Unity	C:\Users\user\Downloads\MixedRealityToolkit-Unity-master\MixedRealityToolkit-Unity-master\Gaze_HolotoolKit\MixedRealityToolkit-Unity\MixedRealityToolkit-Unity.csproj	436
I use the following versions of the tools:

Unity (Version 2017.2.0f3)
Visual Studio Community (Version 15.5.2)
Latest version of the MixedRealityToolkit-Unity

Does anybody how to fix that issue?

That's easy to fix. Build into an empty folder or delete everything from the one you build into. I think this happens when you build with copy references (which is not necessary). Either the second time you do that this happens, OR, what I think is, when you then disable that flag, the project has duplicates of everything in the build directory and the Unity directory it now references.
I am not sure, but deleting the build stuff fixes it.

Please reopen if you still have specific issues.

I had this same problem and Alexees' fix didn't work for me. The way I fixed it was to rename the folder containing the Unity project to something shorter than MixedRealityToolkit-Unity-Dev_Working_Branch.
I guess the problem was that the path was too long, and at some point some paths or file names were being trimmed. In my case, when trying to build and deploy the Bounding box scene to my Hololens, on Visual Studio Output window I saw a message saying that "the file UnityEnine.UI.dll wasn't found" (notice the missing G on the word Engine - it got trimmed at some point).

Thanks @brunasilvalopes - I had a similar issue where the compiler complained it couldn't find "UnityEgine.Networking.dll" (note "Egine" instead of "Engine"). Your comment prompted me to shorten a long folder name and the app was able to be deployed at least.

See this comment for the cause of the error: https://github.com/Microsoft/MixedRealityToolkit-Unity/issues/1472#issuecomment-386876127


Multiple instances of Spark

Is it possible to run multiple instances of Spark inside an application?
Would it be as simple as instantiating 2 instances of SparkInstance if this were publically visible?

While it may not be quite as quick-start friendly, I think it'd be a better approach to get rid of the singleton instance and have the user manage instance(s).

Well how about having best of both worlds? Internally, Spark uses a singleton SparkInstance. Should SparkInstance be made public, users would have the choice of using static methods and a singleton instance provided by Spark, or instantiate a SparkInstance and use that. Naturally, we'd have to ensure that the instances don't share stuff - for example, singletons such as the route matcher provided by RouteMatcherFactory.

I'm hitting this issue as I'm trying to write an integration test that brings up a number of services (each using spark).  They work fine when launched in isolation, but cause errors when they come up in the same process.  Wondering if some classloader magic might work?

@imrichardcole I have done just the thing with classloaders. Two different spark projects, each in a fatjar that i loaded in its own classloader and started with reflection.

Duplicate of #281 I believe

Was implemented in #499


插件可以commit但无法push到远程仓库

显示commit后会一直报上图的错误

I am in a similar situation. In my case the remote repository is not configured, but I get "fatal: bad revision 'null'" after commit every time.
I have recently investigated but have not yet found the cause.

我刚刚改了一下，用了git push -u xxx master后好像不报错了


We should suspend the pg transaction timeout while waiting for the compiler

If there is a long running compile job (probably computing a migration?), we can spuriously trigger an idle-in-transaction timeout, since even though the transaction is not idle from edgedb's perspective, it is from postgres's.

Another option would be to have a heartbeat task that sends keep alive queries. That might actually be better, since in the common case it would never have to do anything.


Cannot save test filter anymore

Since the new update, I get an error message when I try to save a custom filter in the reporting tool:

already fixed. to test just setup zebrunner ce from branch develop


Add basic schema_view and a default url that maps to it

@marcgibbons, what would you think about adding a schema_view that does basically what the example shows directly to this package?  I liked the old version, I see no reason to require the user to create a whole view just for Swagger (Or, OpenAPI).  So I basically copied the example schema_view and added a route to it in urls.py, so that you can I can include Swagger the same way we used to:
urlpatterns += [
    url(r'^swagger/', include('rest_framework_swagger.urls')),
]

The one difficulty was the fact that in your example, the title for the SchemaGenerator was hard-coded.  So I played around with it, and found that there are two places to include Swagger:

Django App
Django Project

Specified in App
In the first case, it's possible to programmatically get the name of the app, and then you could just insert that.  Just remember to set your app_name in your urls.py by following one of the two examples shown here.
Extending the example above, this then would look like (in increasing order of my personal preference):
urlpatterns += [
    url(r'^swagger/', include((
        url(r'^$', include('rest_framework_swagger.urls'),
        'my-application'
    ))),
]

or
app_name = 'my-application'
urlpatterns += [
    url(r'^swagger/', include('rest_framework_swagger.urls')),
]

(I also considered replacing - and _ characters in the app_name with spaces to generate the api title, just in case.  What do you think?)
Specified in Project
In the second case, I can't find anyplace that specifies a "name" of your Django project, so I just defaulted to grabbing the name of the Django project folder.
Title Set Explicitly
Alternatively, this schema_view implementation also accepts the title as an extra argument, as shown here.
urlpatterns += [
    url(r'^swagger/', include('rest_framework_swagger.urls'), {'title': 'My Application API'),
]

This method of finding a name for the schema seems OK for a first-go to me, but I'm not married to it.  If you like this PR, I'd be happy to modify the documentation accordingly, but I'll wait for a thumbs-up before I go to that effort.

[Current coverage][cc-pull] is 60.60% (diff: 100%)

Merging [#493][cc-pull] into [master][cc-base-branch] will decrease coverage by 24.50%

@@             master       #493   diff @@
==========================================
  Files             3          5     +2
  Lines            47         66    +19
  Methods           0          0
  Messages          0          0
  Branches          2          4     +2
==========================================
  Hits             40         40
- Misses            7         26    +19
  Partials          0          0

Powered by Codecov. Last update [dc1eaec...7c22eb9][cc-compare]
[cc-base-branch]: https://codecov.io/gh/marcgibbons/django-rest-swagger/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/marcgibbons/django-rest-swagger/compare/dc1eaec5b397a7fd2b1e276f7d850afb213a86ac...7c22eb9382a248482f8655120d217dff33215fb0?src=pr
[cc-pull]: https://codecov.io/gh/marcgibbons/django-rest-swagger/pull/493?src=pr

Thanks for this! This is cool, but I am somewhat hesitant to introduce this just yet. Ideally, we'd want to provide all the flexibility of configuration the schema view; the kwargs for the schema generator, view permissions, whether or not to include the request object, etc. Since the schema generation is still very new and still being developed, my preference is to keep this loose for now.
Also, the schema view can originate from a Router (although needs a fix to make it a default) (https://github.com/tomchristie/django-rest-framework/issues/4268)

Ah, well I guess this submission was a bit naive, then.  :)  Looking at the schema documentation in DRF, I only see three kwargs, which wouldn't so bad to support along with the optional request argument.  I don't know much about view permissions, though.
It seems that it would be nice to have something like this in place even if schema generation isn't locked down, as a basis for future modification, but it's not that big of a deal to just include a copy of this view in my project anyways for now.
Is there anything I could do to make this PR useful to this project in any way?  If not, feel free to just close the PR.

@malikoth It is a good idea, and I think a nice convenience to get people quickly up and running. Things are still in flux, so I will keep it in the backlog and revisit. Thank you so much!


[NEW] Add day marker in chatroom

Added Day Marker for messages in chatroom
Closes #825

@pcforgeek Thank you very much for the PR! Feature is working really nice! I had one crash while scrolling thought the messages, not sure if it's related to this change or not. 👍

@rafaelks Happy to help. I didn't face any such issues on my system, but I do hope that you resolve this glitch. Let me know if you need any further help regarding this.

@pcforgeek Could you please solve the conflicts?

@leonardoaramaki Solved the conflicts.

There are a lot of indentation problems on this PR, like if(data.showDayMarker){
You should take a look on the Kotlin Coding Convetions to reformat the code.

@luciofm Changes made.

@pcforgeek I'm sorry, but there are some new conflicts on your PR... can you take a look and @luciofm can you review as soon as he fixes it, please?

@rafaelks @luciofm Fixed the conflicts.

@rafaelks Changes made. Thanks for pointing about Index 0 of 0 error.

I've tried implementing this branch into a fork of RocketChat and found a bug.
When i send a message on a different day, it doesn't show the day marker. Only after I re-enter the chatroom it shows the day marker.

@leonardoaramaki is going to fix the conflicts of this PR, make some code adjustments and merge it for the 2.5.0 release! 🎉

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: rafaelks:white_check_mark: filipedelimabrito:white_check_mark: Pitstopper:white_check_mark: shubhsherl:white_check_mark: leonardoaramaki:white_check_mark: luciofm:x: ericrosenthalYou have signed the CLA already but the status is still pending? Let us recheck it.


comment index.js of TopicMenus dir for better readability

I saw index.js codes of TopicMenus dir which are not commented; for following the best and recommended practices, hoping codes will commented out

I saw index.js codes of TopicMenus dir which are not commented; for following the best and recommended practices, hoping codes will commented out


'tspin -p empty.log' waits until file has content, which may never happen

Thanks for this project, it's great!
One unexpected behavior I've noticed (in 2.0.0) is:
$ rm -f empty.log
$ touch empty.log
$ tspin -p empty.log  # hangs indefinitely

If I leave that hanging, and in a separate Zsh session run:
$ print -rl -- one two three >empty.log

then the first session with the hanging tspin command prints
one
two
three

and exits.
I wouldn't expect tspin -p to wait for any/more input -- is that intentional? I would expect instead for it to print nothing and exit immediately, in the case of an empty file.

Hi @AndydeCleyre and thanks for your detailed description.
This is indeed a bug. tspin -p should exit immediately when it reaches the end of the file, like you expect.
Additionally, I think the behavior you're seeing is something you'd expect if you ran tspin --follow --print (only it would run forever), which is currently unsupported, but I now see that it might be nice to have anyway.

Fixed in 45d3b1e41f9f9c35775954d6d8e3379afbd48cb4. Will be available in the upcoming 2.1.0 release.

Just released 2.1.0. Will close this ticket, thanks for reporting it.


reject partial wildcard hosts in service entry for HTTPS protocol

Fixes https://github.com/istio/istio/issues/44195

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
2eb219c974472c78280daf98f2879405f0544ae9
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
2eb219c974472c78280daf98f2879405f0544ae9
link
true
/test unit-tests-arm64

unit-tests_istio
2eb219c974472c78280daf98f2879405f0544ae9
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


2023 python packaging

[ ] all our projects DO NOT use setup.py and have pyproject.toml instead

[x] packit - #1913
[ ] ogr & remove -W ignore introduced with this commit https://github.com/packit/ogr/pull/769/commits/f4b4a7a8130c23e375607334e22b9804e4dd9d47
[x] specfile - /packit/specfile#177

[ ] all our specfiles use the new pyproject-rpm-macros (%pyproject_{wheel,install,save_*})

[x] packit
[ ] ogr
[x] specfile

[ ] all builds in packit are passing
[ ] testing farm tests are green
[ ] specfiles are up to date in src.fp.o

I'm moving this to ogr since that's the last missing component.
As stated in the last paragraph of https://github.com/packit/packit/pull/1913#issuecomment-1499511831 you have to decide (because of old setuptools version in RHEL9) whether to do it

the specfile way, i.e. have both, pyproject.yaml and setup.cfg in order to use setuptools as the build system or
the packit way, i.e. have only pyproject.yaml and use for example hatchling as the build system

I'd prefer the second one, I tried it in another project and it works like a charm for me.


feat(performance): check if unpatched before disabling

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


test-case

close this PR. Please open another one against qiaozha/azure-samples-js-management

close this PR. Please open another one against qiaozha/azure-samples-js-management


[BUG] Tests fail in CI - investigation needed

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Please enter full details of how to reproduce this bug.
Expected behaviour
A clear and concise description of what you expected to happen.
Log file
Please attach the log file if relevant (e.g. output/cellfinder_2019-12-17_13-29-46.log)
Screenshots
If applicable, add screenshots to help explain your problem.
Computer used (please complete the following information):

OS: [e.g. Linux, Windows, macOS] For macOS, please specify if you have an Intel or M1 CPU (M1 is likely if your mac is new).
Version [e.g. 18.04]
Hardware specs [e.g. 128 GB RAM, Xeon 4114 CPU].

Additional context
Add any other context about the problem here.

Test failures seem to be due to slow download of atlases (related to https://github.com/brainglobe/brainglobe-atlasapi/issues/407).


Keep getting build error

Hello,
I keep getting build error that is stopping me from progressing even though I wrote identical code to the answer.

Hello,
Please open Settings | Advanced Settings, find the option Use ReSharper C++ language engine, disable it if it is enabled, and check if it helps to resolve the problem.


[BUG] Identity ignores options tenant when environment var set

Library name and version
Azure.Identity 1.9.0
Describe the bug
TenantId for DefaultAzureCredentialOptions does not seem to affect environment credential.
Expected behavior
Explicit TenantId should override any TenantId by any other credential used behind the scenes.
Now it does not affect the outcome, and if acquiring token happens with an explicit tenant (same as TenantId), it would still not work.
If this is meant to be like this, the documentation should be fixed as now it says this setting overrides any other.
Actual behavior
TenantId is ignored if environment variables are used
Reproduction Steps

Set up environment variables to connect to Tenant A

_tokenCredential = new DefaultAzureCredential(new DefaultAzureCredentialOptions
        {
            TenantId = "Tenant B"
        });

Get token without explicitly specifying a tenant
It would be Tenant A

Environment
dotnet 7

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @ilya-scale
Could you provide some additional details about which credential in the DefaultAzureCredential chain you are targeting to reproduce this? If you are not sure, logging will reveal which one is selected.
In my testing, the TenantId option is respected for everything but EnvironmentCredential and ManagedIdentityCredential

It is exactly EnvironmentCredential that I use both locally and in production


Terminal cannot escape from yes command using Ctrl+C.

VSCode Version: 1.24.1
OS Version: Manjaro Linux

Terminal cannot escape from yes command using Ctrl+C. It is also applies to the others like Ctrl+Z.
In other terminals, yes command immediately exists when pressing Ctrl+C.
Steps to Reproduce:

Run yes command in the Integrated Terminal.
Try Ctrl+C
Terminal ignores Ctrl+C and keeps printing.

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

Duplicate https://github.com/Microsoft/vscode/issues/42234


Non-enumerable property is enumerable under certain conditions

If the object has 128 keys that include dashes, then the non-enumerable property will be enumerable (also writable, configurable).
Tested on Edge 42.17134.1.0
const test = (length = 128) => {
    let data = {};
    for (let i = 0; i < length; i++) {
        data[i + '-'] = true;
    }

    Object.defineProperty(data, 'nonEnumerable', {enumerable: false});

    return Object.getOwnPropertyDescriptor(data, 'nonEnumerable');
};

https://jsfiddle.net/q690m7w8/

What version of chakracore are you using here? I don't see this reproduce in latest master.

I tested it in Edge itself. The Edge issue tracker points to this repo to JS engine related issues.

Ah ok. Looks like it should be fixed in a future release then :)

I see, thanks!

@MSLaguana I am reproing this on 17704...

Oops, that's because the fix is new enough that it hasn't hit windows at all yet. This works properly on the tip of release/1.10.


dmenu does not receive key presses when it's the first x window to open

If you have no x windows open, and run dmenu, it will not receive key presses.

Thanks!


Point users of deprecated inflector methods to ember-inflector

The @ember/string library doesn't provide these methods, they are to be
found in ember-inflector.

@jrjohnson just after merge realized this should point v2 branch so recreated PR in #2324


pydevd debugger plugin

Closes #214
This is more or less ready for initial usage.
Left to do:

[ ] Document (?)
[ ] Bug fixes (Just looking at the screen shots I am already seeing a few issues).

This is implementing a pydevd resolver for OmegaConf.
pydev is used in PyCharm and vscode as the Python debugger.
The resolver is aware of the special semantics of OmegaConf containers and handles missing, interpolations etc in a better way.
A future enhancement can be to expose the underlying Structured Config type in the debugger.

I need to revisit this, it doesn't quiet do what I want. Closing for now but will keep trying.


⚠ Bofrost.be has degraded performance

In 90c637d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1016 ms

Resolved: Bofrost.be performance has improved in 1af623b after 7 minutes.


feat: eclipse che roles

This is my first try, basically just replicating the che docs. I hope I am getting this about right. I am unsure about how to integrate in site.yml since I guess not every k8s cluster we deploy should also have ché. Hence, there should perhaps be dedicated group of hosts che_cluster or so?
Also, this is completely untested for now.
Any feedback or help welcome.

I have a test cluster that we can use for this. I'll take a look at the documentation, try executing the role and report back.
After the role is in a working state, we can decide whether to integrate it into the main cluster or create a new one.

The pod che-gateway doesn't start without an OIDC provider. We need to look into https://dexidp.io/docs/kubernetes/.