Purview dotnet client: Wrong Uri: Doc bug with bad params: Doc bug with undocumented param

Library name and version
"Azure.Analytics.Purview.Catalog" Version="1.0.0-beta.2"
Describe the bug
Tried the example here : https://docs.microsoft.com/en-us/dotnet/api/overview/azure/analytics.purview.catalog-readme-pre

The Url parameter should be Uri
The GetAllTypeDefinitionsAsync call takes a new param called Azure.RequestOptions that seems to be an undocumented type
The call fails with a request to the http://management-babylon-eastus.catalog.clouddatahub.net/catalog/api/atlas/v2/types/typedefs URI, tried the non async version as well with the same result

Expected behavior
retrieve types
Actual behavior
Exception has occurred: CLR/Azure.RequestFailedException
An unhandled exception of type 'Azure.RequestFailedException' occurred in Azure.Analytics.Purview.Catalog.dll: 'Service request failed.
Status: 404 (Not Found)
Content:
{"requestId":"e09eae06-39b0-4a09-98da-db898555e547","errorCode":"RequestUriNotFound","errorMessage":"null for uri: http://management-babylon-eastus.catalog.clouddatahub.net/catalog/api/atlas/v2/types/typedefs"}
Reproduction Steps
Just follow the steps in the getting started doc: https://docs.microsoft.com/en-us/dotnet/api/overview/azure/analytics.purview.catalog-readme-pre
Environment
Windows 10 64-bit working through VSCode with all updates

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Identity:0.27800396,Data Lake Storage Gen2:0.062368814,Cognitive Services:0.04022919'

@marktayl1 I have checked this, and I see the same behavior.  I am checking internally and get back to you.

@marktayl1 You need to use https://{Account Name}.purview.azure.com endpoint to get the results back.  I have tried this and it worked fine.

I will work internally on the documentation update, meanwhile, you can try and let me know in case you see any issues.
Thanks
Saurabh

I was doing it with our account name.
var client = new PurviewCatalogClient(new Uri(https:/redacted_my_account_name.catalog.purview.azure.comhttps://purview-to-adb-purview.catalog.purview.azure.com), credential);
That is not the issue.  Someone already repoed this, I am confused by this response.
Thanks,
Mark
From: SaurabhSharma-MSFT @.>
Sent: Tuesday, January 18, 2022 8:56 AM
To: Azure/azure-sdk-for-net @.>
Cc: Mark Taylor (PIHV) @.>; Mention @.>
Subject: Re: [Azure/azure-sdk-for-net] Purview dotnet client: Wrong Uri: Doc bug with bad params: Doc bug with undocumented param (Issue #26325)
@marktayl1https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmarktayl1&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=bqcHwJnD0kpIVihaMGFtRZoTgF09syz0k5n8xPVBezQ%3D&reserved=0 You need to use https://{Accounthttps://{Account Name}.purview.azure.com endpoint to get the results back. I have tried this and it worked fine.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F38112130%2F149982362-d458000c-5ed4-4099-bd50-98e43e09ab0c.png&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=15XHqo%2FS62%2FuJBWGa4XNby4eAmMhgccmAbpyLhoRvz8%3D&reserved=0
I will work internally on the documentation update, meanwhile, you can try and let me know in case you see any issues.
Thanks
Saurabh

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-net%2Fissues%2F26325%23issuecomment-1015615605&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ii81grqmxqCQEyuz9zstiLLIURlWpy3EKybxgFEK7fQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACS3TYLQIO7IIATD3THSOLLUWWLQLANCNFSM5L3KOL5Q&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=4qg%2BIWI%2F6hiVMvfXhQ%2FyUJCQPSNKXLQEDQsU07ipu04%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362612364|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=qEV5AIf80G%2B%2FZyyP9xgGDJX5VKPy3VEwOZxm6PlhTYo%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362612364|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=niGNxpF9jtaZFOaIRstOKLDb1eOWfA9HidpBBhLqgyY%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@marktayl1  When calling Purview account url you need to pass -  https:/redacted_my_account_name.purview.azure.com instead of using https:/redacted_my_account_name.catalog.purview.azure.com.

Okay, thanks.  Sorry, misunderstood.  That worked.
From: Mark Taylor (PIHV)
Sent: Tuesday, January 18, 2022 9:20 AM
To: Azure/azure-sdk-for-net @.>; Azure/azure-sdk-for-net @.>
Cc: Mention @.***>
Subject: RE: [Azure/azure-sdk-for-net] Purview dotnet client: Wrong Uri: Doc bug with bad params: Doc bug with undocumented param (Issue #26325)
I was doing it with our account name.
var client = new PurviewCatalogClient(new Uri(https:/redacted_my_account_name.catalog.purview.azure.comhttps://purview-to-adb-purview.catalog.purview.azure.com), credential);
That is not the issue.  Someone already repoed this, I am confused by this response.
Thanks,
Mark
From: SaurabhSharma-MSFT<EMAIL_ADDRESS>Sent: Tuesday, January 18, 2022 8:56 AM
To: Azure/azure-sdk-for-net<EMAIL_ADDRESS>Cc: Mark Taylor (PIHV<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-net] Purview dotnet client: Wrong Uri: Doc bug with bad params: Doc bug with undocumented param (Issue #26325)
@marktayl1https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmarktayl1&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=bqcHwJnD0kpIVihaMGFtRZoTgF09syz0k5n8xPVBezQ%3D&reserved=0 You need to use https://{Accounthttps://{Account Name}.purview.azure.com endpoint to get the results back. I have tried this and it worked fine.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F38112130%2F149982362-d458000c-5ed4-4099-bd50-98e43e09ab0c.png&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=15XHqo%2FS62%2FuJBWGa4XNby4eAmMhgccmAbpyLhoRvz8%3D&reserved=0
I will work internally on the documentation update, meanwhile, you can try and let me know in case you see any issues.
Thanks
Saurabh

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-net%2Fissues%2F26325%23issuecomment-1015615605&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ii81grqmxqCQEyuz9zstiLLIURlWpy3EKybxgFEK7fQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACS3TYLQIO7IIATD3THSOLLUWWLQLANCNFSM5L3KOL5Q&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362562398|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=4qg%2BIWI%2F6hiVMvfXhQ%2FyUJCQPSNKXLQEDQsU07ipu04%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362612364|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=qEV5AIf80G%2B%2FZyyP9xgGDJX5VKPy3VEwOZxm6PlhTYo%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|marktayl%40microsoft.com|887a87949d1243e32ca808d9daa357d1|72f988bf86f141af91ab2d7cd011db47|1|0|637781217362612364|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=niGNxpF9jtaZFOaIRstOKLDb1eOWfA9HidpBBhLqgyY%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@marktayl1 Great. Thanks for the confirmation.  We will proceed with the closure of this issue.
Thanks
Saurabh


Full Example Request

Hey. This looks really good and I'm really interesting in using but it isn't clear to me how to use the thing.
Can you include a simple worked example the readme? Thanks in advance.
Jacob

Yes, I have the same issue. I am following everything, have my css files but can't make it work with my notebooks, can't apply it on them.

@mcolic Did you remove your old ipython profiles? I'm still trying to figure out how to get jupyter to ignore them if they already exist. What's your platform and browser?
@barnettjacob https://github.com/transcranial/jupyter-themer#usage Once you install it, the command will be available in the command line. For example: jupyter-themer -c base16-light -l wide -t serif

So I should remove all python profiles and then work on this one?

My browser is safari and platform is python 3

@mcolic Yeah, if you upgraded from ipython to jupyter, your old ipython profiles are probably still present (for example, at ~/.ipython). Jupyter does away with separate profiles, and uses a single config. However, I think it still tries to use old ipython CSS files if they exist.

Hey, thanks for making this! You are right, about the ipython profiles as well. For others, if you simply rename your 'default_profile' folder to say 'ignore_default_profile' then this will work. Start the notebook, then in a seperate command line run "jupyter themer -c cobalt" for example. I'm not sure why people are confused about how to use this, as your instructions were quite clear.
Thanks again!

Thanks for following up, I got this working. Your usage section in the readme probably isn't explicit enough for those of us without a bunch of command line experience!


Is this dope?

I think this is dope. Plz confurm

Somethings value is determined by the fiction the collective human consciousness has constructed around it. For something to have meaning is to believe it has meaning.
So yea it's dope. Thx.


Polishing the KeptnConfig implementation

Goal
Clean up the implementation of KeptnConfig
Technical Details
In the manifests, we are configuring a default URL for the collector but it is ignored since we prefer the value coming from a KeptnConfig. We should make use of the env var to initialize a defaultKeptnConfig until a CRD is reconciled.
Given that we reconcile it, a validation webhook with the following behavior:

if there is already an existing KeptnConfig CRD

if they have the same name, succeed
otherwise fail

otherwise succeed

The RBAC should also be cleaned up:

RBAC should grant read-only permissions on both, spec and status
no finalizer is needed

Acceptance Criteria

[x] Use env vars as defaults
[x] Introduce a validating webhook
[x] RBAC is cleaned up

DoD

[x] Tests for the validating webhook
[ ] Documentation update

@thisthat is this ticket done, or is something missing ?

The documentation is not yet done. We can close it with #1239


GH-3891 - Configure quarantineMode

Purpose
Describe the problem you want to address or the feature you want to implement.
Closes #3891 : Allow for Quarantine threshold values to be configured (instead of hardcoded to 3). This should include the number of retries, as well as the minimum number of passes.
Approach
Describe how your changes address the issue or implement the desired functionality in as much detail as possible.
I amended quarantineMode to optionally accept a Dictionary. Parse the input (and validate), then use this to set the threshold values of failedThresholdReached and passedThresholdReached
References
Provide a link to the existing issue(s), if any.
#3891
Pre-Merge TODO

[x] Write tests for your proposed changes
[ ] Make sure that existing tests do not fail

Hi @rob4629
Thank you for contributing to TestCafe. You expose internal failedThreshold and passedThreshold parameters via quarantineMode { "passCount": 1, "retryCount": 5 } API.
Description of parameters:

passCount is the number of passed test runs to consider the test passed
retryCount is the number of failed test runs to consider the test failed

In general, this approach is good.
At the same time, we believe that the retryCount parameter should be a full test run count that TestCafe will execute.
For example:

consider the test passed if at least one run passed - quarantineMode { "passCount": 1 }
increase the number of test run attempts - `quarantineMode { "retryCount": 10 }

This approach doesn't seem to require a lot of changes. You need to calculate the failedThreshold value as a result of the retryCount minus passCount expression and validate if that passCount is less than retryCount.

Thanks for the feedback @miherlosev. I can implement that change today. Does passCount seem a sensible/explicit name to you, or should this be passThreshold?
Also, it looks like my change has broken existing functionality:
testcafe -q tests/myFixture/myTest.js

ERROR The "--quarantine-mode" option value is not a valid key-value pair.

If I explicitly add the testpath argument afterwards, it behaves as expected
testcafe -q -t tests/myFixture/myTest.js

Although I'm validating it's a valid Key-Value pair in getQuarantineOptions, should I add some sort of filtering/conditions to _parseQuarantineOptions?

This approach doesn't seem to require a lot of changes. You need to calculate the failedThreshold value as a result of the retryCount minus passCount expression and validate if that passCount is less than retryCount.

Ok, I don't think I understand your suggestion... here's how it sounds to me (sudo code)
const failedThreshold = passCount < retryCount ? retryCount - passCount : DEFAULT_QUARANTINE_THRESHOLD;

const failedThresholdReached = failedTimes >= failedThreshold;
....

testcafe -q                                // no change
testcafe -q passCount=1                    // failedCount = 2
testcafe -q retryCount=5                   // failedCount = 2
testcafe -q retryCount=10                  // failedCount = 7
testcafe -q retryCount=10,passCount=1      // failedCount = 9

In general, you understand the explanation correctly.
Just one correction: you should validate that the passCount is less than retryCount.
Pseudocode
validateArguments({ retryCount, passCount });

const failedThreshold = retryCount - passCount;

const failedThresholdReached = failedTimes >= failedThreshold;

// All correct
testcafe -q                                // no change
testcafe -q passCount=1                    // failedThreshold = 2
testcafe -q retryCount=5                   // failedThreshold = 2
testcafe -q retryCount=10                  // failedThreshold = 7
testcafe -q retryCount=10,passCount=1      // failedThreshold = 9

// Raise an error about incorrect parameters
testcafe -q retryCount=3,passCount=4

Ok, I've added further validation to the argument-parser, plus 2 new tests to verify passCount < retryCount.
I've also implemented the failedThreshold as per your request, however I believe we should have a minimum failedThreshold of 3, rather than setting failedThreshold to 1 for -q retryCount=4 or -q passCount=1.
Ultimately the decision is yours, but I look forward to hearing your thoughts.

I think that in the previous comment we made a mistake and failedThreshold should be calculated like this: failedThreshold = retryCount - passCount + 1.
And also for compatibility with the current implementation, the values of the variables should be passCount = 3 and retryCount = 5 by default. The remaining parameters are calculated based on these values.

Is there anything I need to do, in order for this PR to be reviewed?
I understand if it's a case of just waiting until you have availability. I just want to make sure I haven’t forgotten to do something.

I've added 3 tests, please let me know if you need more coverage. It also highlighted a bug I had introduced in the _setFailedThreshold() function... so thanks for the prompt!

Do you need me to do more for this PR? It seems like my local master branch has become corrupted... So I'm trying to resolve that before I can investigate these failing tests

@rob4629 Great PR.
We've moved our documentation to a separate repository and it's not available for open-source contributions anymore. Please rebase your PR and remove all documentation changes (changes to .md files) from this PR.
I'll make sure that the documentation for this new feature makes it into the docs.
Thanks.


[Feature] Add URL Base Subfolder Option

My changedetection.io server is behind a reverse proxy and I use individual folders per site.  I see a new URL base path option for just notifications but no way to specify subfolders for the site.
Ex: domain.com/changedetection.io/

note to self or others who want to PR https://stackoverflow.com/questions/18967441/add-a-prefix-to-all-flask-routes

@aaronrunkle I had a bit of a dig around but couldnt figure it out, I think its because I'm using the eventlet wrapper, it does not pickup the script_path properly... other wrappers like gunicorn i think will do it better... or uwsgi unsure.. help welcome

so http://<IP_ADDRESS>/app loads but, the links in the watch-table (and styles) dont see the /app ...

https://www.digitalocean.com/community/tutorials/how-to-serve-flask-applications-with-uswgi-and-nginx-on-ubuntu-18-04

Also.. need to change to
# Good (preferred).
CMD ["gunicorn", "-c", "python:config.gunicorn", "hello.app:create_app()"]

@dgtlmoon  Thanks!
Looks like it's mostly working for me.  Three issues I've found thus far -
The save button doesn't seem to redirect properly.  Looks like /settings is hardcoded here:
https://github.com/dgtlmoon/changedetection.io/blob/master/backend/templates/settings.html#L7
Same with /login when a password is set:  https://github.com/dgtlmoon/changedetection.io/blob/master/backend/templates/login.html#L5
Also cannot add new watched items (/api/add):
https://github.com/dgtlmoon/changedetection.io/blob/master/backend/templates/watch-overview.html#L7

@aaronrunkle thanks for the feedback! just pushed a commit now, give it 30min~ to rebuild

Working great so far; thank you!

@dgtlmoon
One more
https://github.com/dgtlmoon/changedetection.io/blob/master/backend/static/styles/styles.css#L96

@aaronrunkle hmmm how to solve that one?

@dgtlmoon You can use relative URL's in the CSS URL function, like ../images/gradient-border.png

@deakster wont work in the case that the page is /edit/foobar or /confirm/delete/watch or something, must be a smarter way, see my link there, using a regex in the route

https://github.com/dgtlmoon/changedetection.io/pull/141#issuecomment-882038304

It might be considered a hack, but using a parent-path (../) as in my PR #272 for gradient-border.png works fine.  For the docker implementations of this project, that appears valid (ie parent paths are not disallowed in the web server).  For personal implementations in other web engines, they would need to allow parent paths if they are disallowed by default, or had been explicitly disallowed by the admin.  If an admin had gone to the trouble of disabling parent paths, I would like to think that they also understand how to rewrite html and turn the path back into a valid one for their specific situation.
My knowledge of web servers is not strong, so parent paths may not be an issue if the url does not go above the root.

AFAIK you dont specify the sub-folder when your server is configured correctly, it will automatically learn it
for example here https://github.com/dgtlmoon/changedetection.io/wiki/Running-changedetection.io-behind-a-reverse-proxy-sub-directory
if you are running the application at http://myserver.com/app for example.. I think this resolves it.. if not.. please open a new issue with the individual issues

Yes, the local (to the changedetection.io instance) web server sort of auto-learns the subfolder - I suspect this is a feature of the framework you are using.  It makes implementation (for the user) very simple, the only downside is that for any elements that need absolute paths specified for some reason, you have no awareness of the correct full path and run into issues like with gradient-border.png.
Could you detect the X-Forwarded-Prefix header and use it to pre-pend on elements?  That would be a cleaner solution than my ../ hack.

@jeremysherriff can you tell me which elements arent functioning still? there's better ways in Flask to handle it (you can set a special regex handler for the request path for example)

@dgtlmoon as per my PR #272 there's only two that I have found;

gradient-border.png as specified in styles.css, and
Google-Chrome-icon.png in watch-overview.html

The edits in that PR cater for both reverse-proxy+subfolder and standard http://ip:port implementations.
Note that the reverse-proxy configuration needs a minor tweak, as per my comments in issue #273 (relates specifically to the edit for styles.css).  Without this, people still wouldn't see the border but that's the only effect.
Can you use a regex handler in Flask to resolve those static components?

The link in the wiki is broken.
If anyone else came here looking for how to enable it, it's documented below:
https://github.com/dgtlmoon/changedetection.io/blob/23476f0e70662480431a970c987ecaf8208dfd88/changedetectionio/changedetection.py#L131

The link in the wiki is broken, but works smoothly with after  adding the environment variable USE_X_SETTINGS.


I was expecting your file to be named _posts/0000-01-02-alih90.md.

I was expecting your file to be named _posts/0000-01-02-alih90.md.
Let's edit this pull request to fix both of these issues.
:keyboard: Activity: Fixing the file name in your pull request

Click the Files Changed tab in this pull request.
Verify that the only file edited is named _posts/0000-01-02-alih90.md.
If the file is improperly named, or not in a proper location, use the edit function to correct it.
Above the contents of the file, select all text in the field that contains the filename and delete it.
Continue pressing your backspace key to also delete any directory names that exist.
Type the proper filename:_posts/0000-01-02-alih90.md

Scroll to the bottom and enter a commit message and commit in the Commit Changes section.

Note: You can find directions on how to move files while editing in our documentation.
If you would like assistance troubleshooting the issue you are encountering, create a post on the GitHub Community board. You might also want to search for your issue to see if other people have resolved it in the past.

I'll respond below when I detect a commit on this branch.
Originally posted by @github-learning-lab[bot] in https://github.com/alih90/github-slideshow/pull/3#pullrequestreview-922425684

done


More obvious download links please

It looks currently the github pages at http://macvim-dev.github.io/macvim/ is the place to find the download links for MacVim.
The README.md, being the most "obvious" place to find download links, however is a copy from VIM, which is kinda like a trap, as it points the download page to http://www.vim.org/download.php, over there, the mac download page is again pointed back to https://github.com/macvim-dev/macvim, none of them mentioned http://macvim-dev.github.io/macvim/. Which is fortunately, included in the project description.

How about it https://github.com/macvim-dev/macvim/tree/test/readme

@splhack, yes!
That's the simplest solution, though more information could be added.
BTW, it took me a little while to figure out that you've updated the README.md in branch test/readme.
Thanks


SOLR-15067 Child doc transformer should use the perSegFilter cache

Change-Id: I6cc22a9e13aeaef37a1e005b111a59bb7412023d

Description
The ChildDocTransformerFactory creates a QueryBitSetProducer every time.  These are heavy to compute; they should be cached.  BlockJoinParentQParserPlugin does this correctly, using the "perSegFilter" named Solr cache.
Solution
The solution for this was to copy the implementation of the cache usage from BlockJoinParentQParser. We decided not to create a new class for this since the solution was quite simple
Tests
Unit tests.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

I have added a new commit which contains the new entry in the CHANGES.txt


Character encoding bug with CSV export

I was just poking around, and at this URL, I encountered this error:
'latin-1' codec can't encode character '\u2019' in position 27: ordinal not in range(256)

This is fixed in master - should be releasing the fix as Datasette 0.30 in the next few days. #584


Speed up text generation, remove faker from text config builder

Caching of text configs brings about 20% of speed
before
Benchmark                                         Mode  Cnt     Score    Error   Units
DatafakerPassword.password10                     thrpt   10  1293.016 ± 38.115  ops/ms
DatafakerPassword.password10WithUpper            thrpt   10  1059.780 ± 83.480  ops/ms
DatafakerPassword.password10WithUpperLower       thrpt   10   963.004 ± 79.944  ops/ms
DatafakerPassword.password10WithUpperLowerDigit  thrpt   10   992.366 ± 32.792  ops/ms

after

Benchmark                                         Mode  Cnt     Score     Error   Units
DatafakerPassword.password10                     thrpt   10  1577.841 ± 112.689  ops/ms
DatafakerPassword.password10WithUpper            thrpt   10  1262.548 ±  74.436  ops/ms
DatafakerPassword.password10WithUpperLower       thrpt   10  1212.872 ±  27.308  ops/ms
DatafakerPassword.password10WithUpperLowerDigit  thrpt   10  1203.032 ±  45.527  ops/ms

Codecov Report

Merging #534 (75ce64f) into main (fae5694) will decrease coverage by 0.17%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##               main     #534      +/-   ##
============================================
- Coverage     91.91%   91.74%   -0.18%
+ Complexity     2359     2355       -4
============================================
  Files           247      247
  Lines          4949     4953       +4
  Branches        530      530
============================================
- Hits           4549     4544       -5
- Misses          272      279       +7
- Partials        128      130       +2

Impacted Files
Coverage Δ

...c/main/java/net/datafaker/providers/base/Text.java
68.88% <82.14%> (+2.22%)
:arrow_up:

...rc/main/java/net/datafaker/providers/base/Aws.java
100.00% <100.00%> (ø)

...in/java/net/datafaker/providers/base/Internet.java
91.77% <100.00%> (-0.40%)
:arrow_down:

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

.../java/net/datafaker/service/FakeValuesService.java
86.02% <0.00%> (-0.87%)
:arrow_down:

.../datafaker/transformations/sql/SqlTransformer.java
86.29% <0.00%> (-0.75%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add heartbeat to etcd quorum check

Depends on #12427
Adds a heartbeat written to a key in an interval (1min) by the operator. Each etcd client installs a watcher to watch the heartbeat key. When the heartbeat is not updated in 2*interval, the quorum check will start failing:
KVStore:                Ok   etcd: 1/1 connected, lease-ID=29c6732d5d580cb5, lock lease-ID=29c6732d5d580cb7, has-quorum=2m2.778966915s since last heartbeat update has been received, consecutive-errors=1: https://<IP_ADDRESS>:2379 - 3.4.9 (Leader)

When enough consecutive errors have accumulated, the kvstore subsystem will start failing:
KVStore:                Failure   Err: quorum check failed 8 times in a row: 4m28.446600949s since last heartbeat update has been received

Coverage increased (+0.03%) to 36.967% when pulling ff1cae1220937d6a16920081cdafab2b9a18b438 on pr/tgraf/etcd-heartbeat into e8d73073c789d1cdb4f2a2e1a5f83e725379a4fd on master.

test-me-please

test-me-please

test-me-please

test-me-please

@tgraf by default, how long would this take from an etcd outage (or cilium-operator node becoming unavailable) before Cilium agents begin restarting?


⚠ Bofrost.at has degraded performance

In 7b91eb1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 848 ms

Resolved: Bofrost.at performance has improved in 05cea1b after 8 minutes.


(maybe) Misprinted Journey to Oldtown CP Code?

For Chapter Pack: "Journey to Oldtown" the code is JoO, maybe it shoud have been JtO?

Thank you!!

Fixed.

Hello,
there is 2 Journey to Oldtown packs in the packs listing... One JoT, the other one is JoO.

@Alsciende You are the only one that can delete the pack with code 'JoO' from the database :)

Done


Adjust TEPs query used to generate summary widget and detail view

Currently, we generate a link to the SGC if the target has a TEP. The link appears in the cross-references section of the target profile page.

These links are generated by the TepLink.js file and use an object of gene IDs, symbols, and links to construct the link — see lines 19 - 160.
Can we please update the link construction logic to use the GraphQL API?
TEP information is available from the target endpoint:
query tepsInfoForRIPK2 {
  target(ensemblId: "ENSG00000104312") {
    id
    approvedSymbol
    tep{
      name
      uri
      therapeuticArea
      description
    }
  }
}

Query: RIPK2
Once we have adjusted the TepLink.js file to use the GraphQL API, can we also please remove the TEP summary widget and detail view?

@carcruz can you please work on this and adjust how we construct TEP links?

Ticket closed as work completed, merged into main, and included in the platform-app 21.09.5 release


How to simulate prismatic joint in matter-js like in box2d

Hi,
I tried to simulate "prismatic joint" with Matter.Constraint, but it did not work. I'm trying to move the box along x axis.
How can i achieve this type of joint?

Missing joints are: Prismatic Joint, Pulley Joint etc. as listed e.g. here: https://github.com/shakiba/planck.js/wiki/Joint
I think, such joints are essential features for a "Physics Engine for the Web"...?
The feature request is from 2017. Are there any plans of integrating these joints / constraints?

Hi.
The following joints are used in Box2D. These are useful for students studying Mechanical Engineering etc.

Distance Joint (present in matter.js)
Revolute Joint  (present in matter.js)
Prismatic Joint
Pulley Joint
Gear Joint etc.

Why are they not present in matter.js?
https://github.com/shakiba/planck.js/wiki/Joint


Updated to support PHP 8.2

Changes on CRC32 version and other items to remove deprecated issues from PHP 8.x

@robsonala can you sign this please? It looks great and I would like to have this merged 👍


Add an alias_method setter for setting urls

It is a common convention in Ruby to use a method name with a trailing = as the "setter method" for a given instance.
Aliasing the current PaginationPage#set_url method name for backwards compatibility

Oh dear, starting to realise that I hardly know any Ruby :)


[docs] Add LireLactu to the showcase

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@lvauvillier Sweet, thanks!


Refactor common table property validation to reduce duplication

Move duplcated code for table property validation to one place
Minor check-style fixes.
fix possible nulll pointer warning (FateServiceHandler)

I think there are some issues with the comments being reformatted in Property.java. Did the formatter do this?

I’ll see if I can fix it – but I didn’t do it.  Not sure where its coming from at this point.
From: Dave Marion @.>
Sent: Thursday, July 1, 2021 7:43 AM
To: apache/accumulo @.>
Cc: EdColeman @.>; Author @.>
Subject: Re: [apache/accumulo] Refactor common table property validation to reduce duplication (#2186)
I think there are some issues with the comments being reformatted in Property.java. Did the formatter do this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/apache/accumulo/pull/2186#issuecomment-872174301 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVYUFFISZW4UVSBNBQO3V3TVRIDVANCNFSM47TLQI7Q .  https://github.com/notifications/beacon/ABVYUFGOXQ2O4EEJ5EZVGW3TVRIDVA5CNFSM47TLQI72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGP6FFXI.gif


Fix panic

When typing immediately until didChangeConfiguration arrival, completion method make panic.

Thanks for contribution. Merged.


Improve developer experience

Based on feedback from discord, this PR will:

install rustup components on demand
track the Cargo.lock file, as  we are an application

This should be merged before #73, and I'll rebase #73 later.

Nice, looks good!


Browser preview doesn't work in Safari

I know its not the typical use case, but using code-server (vscode on a remote server) with browser preview (in iOS) would be a great for web development on iPad, but it appears to render a checkered grid instead of the page.
Looking at the css, there is a css class being applied to the rendered page: .img-hidden{ display: none }. My guess is that this class is supposed to be removed somehow when rendering is finished?
How the page is shown initially:

Turning off the display: none css attribute:

This extension has never been tested outside of Electron, so that's a new use-case. The rendering pipeline is using the hidden image to convert the base64 PNG to an image, which is then rendered to the
So something must go wrong in the process of decoding the base64 to img.


Get response headers from request

I wanted to be able to get the response headers from a request to handle the x-rate-limit-remaining in our app.
I've made it an optional setting before the request and added the ability to retrieve them after the request via a private property.
Added a phpunit test for this going forwards too.

@eddieharrison Any plans to move this forward? I'd also need this feature to retrieve rate limit headers.


Resizing offscreen canvas

This is for your offscreen canvas example. When you try to upload a second image an error occurs because there is an attempt to resize a canvas that has already transferred control to offscreen.

I am also running into this issue with a website i'm developing. The only difference is that my website requires the canvas to be resized after certain events occur.
I would resize a normal canvas with code like this:
this.$.canvas.width = this.size * window.devicePixelRatio;
this.$.canvas.height = this.size * window.devicePixelRatio;

this.$.canvas.style.width = this.size + 'px';
this.$.canvas.style.height = this.size + 'px';
this._ctx.scale(window.devicePixelRatio, window.devicePixelRatio);

Ever since I started using offscreen canvas that code would throw the same error as above. I tired instead setting the width and height of the offscreen canvas object. I also set the scale in a web worker because i'm running the animation off the main thread. There is no error when I do this however it does not seem to resize properly.
Main
this._offscreen.width = this.size * window.devicePixelRatio;
this._offscreen.height = this.size * window.devicePixelRatio;

this.$.canvas.style.width = this.size + 'px';
this.$.canvas.style.height = this.size + 'px';

Web Worker
self.onmessage = e => {
  if (e.data.devicePixelRatio) {
    ctx.scale(e.data.devicePixelRatio, e.data.devicePixelRatio);
    ctx.commit();
  }
}

I was wondering if anyone knows how to resize an offscreen canvas properly.

I could probably do that as a last resort. I guess I was hoping I missed something and there is an easier way to resize it without destroying the old one and creating a new one.


Add srcset attribute to mj-image*

According to the documentation mj-image should be able to handle srcset attribute. It was in v3 but it isn't anymore the case in v4.
srcset attribute is back in this PR.

Thanks for this @jtassin ! Fix #1004


Support for "adjusted pointers" / CONTAINING_RECORD (i.e. pointers into the middle of structures)

Some compiled code will pass around a pointer to a structure member as if it were a pointer to the beginning of a structure. Currently Binary Ninja doesn't support this type of construct. The windows kernel frequently uses the CONTAINING_RECORD macro to get a pointer to the base of the structure.
Describe the solution you'd like
Provide the ability to tell Binary Ninja that a pointer is pointing at some offset within a structure.
Related to #184

Example testcase is well documented documented here: https://research.nccgroup.com/2020/05/04/cve-2018-8611-exploiting-windows-ktm-part-2-5-patch-analysis-and-basic-triggering/#tidying-hex-rays-output-with-shifted-pointers
Blocked on #2932. Also the workflow of getting the types out of nt and into tm is a bit dicey (#2913).
tm.sys.txt

Would this also address the structure detailed here? May be a good sanity check for feature
https://hex-rays.com/blog/igors-tip-of-the-week-57-shifted-pointers-2/

Implemented in dev build 3751.


Crash in mrb_hash_keys after hash keys are modified

The following input demonstrates a crash:
h = {}
a = []
200.times do |n|
  a[0] = n
  h[a] = 0
  puts h
end

This issue was reported by https://hackerone.com/ssarong.
The crash occurs after kh_resize increases the size of the hash table. It considers all the keys to be equal and so after the resize kh_size(h) is 1. But the one remaining mrb_hash_value still uses the old numbering, and so a crash occurs when mrb_hash_keys tries to write beyond the end of its array: https://github.com/mruby/mruby/blob/905f46a129de65098b3130789bb9c412201f4cb4/src/hash.c#L766

It's also apparent from the output that insertions into the hash don't always work:
{[0]=>0}
{[1]=>0, [1]=>0}
{[2]=>0, [2]=>0, [2]=>0}
{[3]=>0, [3]=>0, [3]=>0, [3]=>0}
{[4]=>0, [4]=>0, [4]=>0, [4]=>0, [4]=>0}
{[5]=>0, [5]=>0, [5]=>0, [5]=>0, [5]=>0, [5]=>0}
{[6]=>0, [6]=>0, [6]=>0, [6]=>0, [6]=>0, [6]=>0, [6]=>0}
{[7]=>0, [7]=>0, [7]=>0, [7]=>0, [7]=>0, [7]=>0, [7]=>0, [7]=>0}
{[8]=>0, [8]=>0, [8]=>0, [8]=>0, [8]=>0, [8]=>0, [8]=>0, [8]=>0}         <--- incorrect
{[9]=>0, [9]=>0, [9]=>0, [9]=>0, [9]=>0, [9]=>0, [9]=>0, [9]=>0, [9]=>0}
...

I have heard from @matz.
The behavior is undefined if key objects are rewritten later (in CRuby).

@ksss OK, so the insertion behaviour is not incorrect then, and we have some flexibility in how to fix the crash.


Strapi bugs

There are currently two strapi bugs that I have temporary fixes for.
The first one is on build.
https://github.com/strapi/strapi/issues/5439
The second one is to save models through the GUI.
https://github.com/strapi/strapi/issues/5095
These will eventually have to be resolved.

I fixed this by changing the default values of my integer types from strings to numbers. I'm not 100% sure this is a migration issue... but I'm suspicious it was.

    "major_version": {
      "type": "integer",
      "default": "0"
    },

    "major_version": {
      "type": "integer",
      "default": 0
    },


[Enhancement] DISCOVERY - Intellisense for Sage components

Description
To better serve developers, look at options for adding Intellisense for Sage components.

Slack discussion around this feature

As a stop-gap, VSCode snippets could provide a ton of value if rolling out Intellisense is a large effort.
Resources
To reference creating an extension in VSCode, see article here: VSCode First Extension
To reference how Tailwind implements their Intellisense, see repo: Tailwind CSS's GitHub Intellisense repo
To reference the snippets Gist @monkeypox8 created, see Gist: Kevin's Snippet Gist

Moved to Jira https://kajabi.atlassian.net/browse/UXD-74


I forgot.


[Question] How to Disable DDL Queries During PostgreSQL Event Store Initialization in Restricted Environments?

Disintegrate uses a PostgreSQL event store and initializes by running DDL queries (via SQLx) to create tables like event and event_sequence. These scripts can be moved to SQLx migrations, including the domain_identifier_columns.
However, in some environments, database admins do not allow running DDL queries. Is there a way to skip these queries during initialization?

Hi @gmkumar2005,
Thanks for the feedback!
At the moment, it is not possible. We'll add this feature in the next release of the library.


How to set collapse key in laravel fcm

How can call CollapseKey function in laravel fcm integration

collapse_key is a field for the Legacy Messaging HTTP API, the Firebase SDK uses the V1 HTTP, where this field is not available.


Ec proofreading odata docs

Closed the previous pull request by mistake. Need to verify that change made by MS is incorporated.

If you close a PR, can always reopen it

Yes, I saw that when I pushed a further edit.
Ela
From: Adam Dupaski<EMAIL_ADDRESS>Reply to: mendix/docs<EMAIL_ADDRESS>Date: Friday, 28 February 2020 at 08:02
To: mendix/docs<EMAIL_ADDRESS>Cc: Ila Gordhan<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Ec proofreading odata docs (#2308)
If you close a PR, can always reopen it
[image]https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F20336433%2F75518194-a1d7dd80-5a00-11ea-802a-3b057ba2ea41.png&data=02|01|ila.gordhan%40mendix.com|67ea98664e3b4c66889708d7bc1c276c|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184701417057963&sdata=gRczQzDAoqh1ekbssSZzTg8nb5JnCVH%2BtDzTtFBtIXE%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F2308%3Femail_source%3Dnotifications%26email_token%3DAOJFHGS6JPP3IUMJRYXXNXLRFCZHXA5CNFSM4K5ANVJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENHNWTA%23issuecomment-592370508&data=02|01|ila.gordhan%40mendix.com|67ea98664e3b4c66889708d7bc1c276c|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184701417057963&sdata=iJy1UKI5m4GQMkVOtv%2FEX78EYrzWrgk1n31GR0H6n3A%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOJFHGU2R4DSX74ODDQ24XDRFCZHXANCNFSM4K5ANVJA&data=02|01|ila.gordhan%40mendix.com|67ea98664e3b4c66889708d7bc1c276c|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184701417067959&sdata=MN8WVnnPBbdBrwcb6TK64LX%2FIWKIKZm2SNdi5m1BqLs%3D&reserved=0.

@ilagordhan please fix the Travis build error for this PR: https://travis-ci.org/mendix/docs/builds/655908681?utm_source=github_status&utm_medium=notification

I would appreciate some guidance on interpreting Travis build errors. Can we discuss next week?
From: Adam Dupaski<EMAIL_ADDRESS>Reply to: mendix/docs<EMAIL_ADDRESS>Date: Friday, 28 February 2020 at 08:41
To: mendix/docs<EMAIL_ADDRESS>Cc: Ila Gordhan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Ec proofreading odata docs (#2308)
@ilagordhanhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Filagordhan&data=02|01|ila.gordhan%40mendix.com|b87c9a3b4bc245347eeb08d7bc21a0ba|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184724927197443&sdata=%2BYAU61PkO7WEZ4tIpq9HcuXIYFStJfZboTSNN3B62gs%3D&reserved=0 please fix the Travis build error for this PR: https://travis-ci.org/mendix/docs/builds/655908681?utm_source=github_status&utm_medium=notificationhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.org%2Fmendix%2Fdocs%2Fbuilds%2F655908681%3Futm_source%3Dgithub_status%26utm_medium%3Dnotification&data=02|01|ila.gordhan%40mendix.com|b87c9a3b4bc245347eeb08d7bc21a0ba|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184724927207439&sdata=SiDMxcOzavxARgXU6U4xatugGjMs0tiUdjQFqxXrMQM%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F2308%3Femail_source%3Dnotifications%26email_token%3DAOJFHGVTNRXE5GPDVGM7POTRFC52TA5CNFSM4K5ANVJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENHSRSQ%23issuecomment-592390346&data=02|01|ila.gordhan%40mendix.com|b87c9a3b4bc245347eeb08d7bc21a0ba|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184724927207439&sdata=UGDmAMB5Ki7PCvdINimOTjNHOihzWyDNZb3WxQjRLgw%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOJFHGWJZUPH6YWKU54LOPLRFC52TANCNFSM4K5ANVJA&data=02|01|ila.gordhan%40mendix.com|b87c9a3b4bc245347eeb08d7bc21a0ba|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184724927207439&sdata=EdErQ9ZTl9Jq%2FN6s46lMFtRTe2fhBxlKnyt3q3aM7EQ%3D&reserved=0.

Normally Travis build errors show up in the Errors section at the bottom of the log.
But the error here is in the doc metadata, so it shows up earlier in the log: [07:39:53] [MENU BUILD] No metadata for file: /refguide/published-odata-resource.md, is the block at the top correct?
You can check the metadata of the doc and find the mistake - it is a small one and easily fixable.

Thanks for the heads up.
From: Adam Dupaski<EMAIL_ADDRESS>Reply to: mendix/docs<EMAIL_ADDRESS>Date: Friday, 28 February 2020 at 10:09
To: mendix/docs<EMAIL_ADDRESS>Cc: Ila Gordhan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Ec proofreading odata docs (#2308)
Normally Travis build errors show up in the Errors section at the bottom of the log.
But the error here is in the doc metadata, so it shows up earlier in the log: [07:39:53] [MENU BUILD] No metadata for file: /refguide/published-odata-resource.md, is the block at the top correct?
You can check the metadata of the doc and find the mistake - it is a small one and easily fixable.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F2308%3Femail_source%3Dnotifications%26email_token%3DAOJFHGWQW2QGEIHFT7L62A3RFDIFLA5CNFSM4K5ANVJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENHZ5KI%23issuecomment-592420521&data=02|01|ila.gordhan%40mendix.com|9d4f859964e44be57d9508d7bc2df2d9|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184777847258743&sdata=xbIaxS0JXBP1AORe29O7sVkI62Jg5TBnEEC5quqm71g%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOJFHGQQMUPAFAKFNNMNGQTRFDIFLANCNFSM4K5ANVJA&data=02|01|ila.gordhan%40mendix.com|9d4f859964e44be57d9508d7bc2df2d9|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637184777847258743&sdata=kzDG7WieY3KWj8b9T1HFD%2B3LmLnzbqrPe62ljLuRCno%3D&reserved=0.

Thanks for the feedback.


We found some opportunities to use the diamond operator in the source code.  We changed the source code and hope you are interested in this kind of change.

(refactor) some opportunities to use the Diamond-operator

There are many syntactical changes that can be applied to our source code since our base was Java 1.5 at the time of our first release.  However, we'd rather not refactor this quite yet against our master branch, but these kind of changes will happen in a future refactoring branch.  We'll close this issue for now but appreciate your effort to help.  In fact, if your a Java developer, please have a look at our issues listing that are tagged as a bug or enhancement and work to resolve those.  That kind of effort would be very appreciated.  Thanks again.

@desmorto in fact, if you want I can begin to create a Java8 branch for you to request a pull against.  That would give us a base to begin most of the refactoring work and you could gladly contribute to that branch.  Again, we don't want to introduce major code changes in master directly just yet...but do that work in another branch that we could call Java8-refactor.  Thanks again.  Let me create that branch now.

@desmorto Done.  Go ahead with a new pull request against 'java8-refactor'.  And you can continue your contributions to that branch.


Executable doesn’t have execute permissions.

From https://github.com/harshadgavali/gnome-x11-gesture-daemon/pull/1#issuecomment-908519282

@drunckj Tracking issue here.
Does issue still occurs with https://github.com/harshadgavali/gnome-x11-gesture-daemon/releases/tag/git_release
btw you don't need to restart system.
Run inside extracted folder
gnome-extensions disable gestureImprovements@gestures
systemctl --user status gesture_improvements_gesture_daemon.service | grep 'Active:'
sh install.sh
systemctl --user daemon-reload
gnome-extensions enable gestureImprovements@gestures
systemctl --user status gesture_improvements_gesture_daemon.service | grep 'Active:'

Just to be on safe side, I commented to restart system on README.

@drunckj Tracking issue here.
Does issue still occurs with https://github.com/harshadgavali/gnome-x11-gesture-daemon/releases/tag/git_release
btw you don't need to restart system.
Run inside extracted folder
gnome-extensions disable gestureImprovements@gestures
systemctl --user status gesture_improvements_gesture_daemon.service | grep 'Active:'
sh install.sh
systemctl --user daemon-reload
gnome-extensions enable gestureImprovements@gestures
systemctl --user status gesture_improvements_gesture_daemon.service | grep 'Active:'

Just to be on safe side, I commented to restart system on README.

The below commands perfectly fine. The service gets started when the extension gets enabled. The user doesn't need to logout and login again.
The releases zip still requires a logout but it works perfectly after logout. Checked it in 3 vms and it works .Mark the issue as solved.


Discord Chat Link Is Invalid

The link to the Discord discussion server is invalid on the "dev" page. My guess is that it's just expired but a non-expiring invite could be used here instead.
Quick reference to the line in question: https://github.com/tightenco/onramp/commit/024f66acaf6754b678bac497259a5dcfdbecf11c

Seems to suddenly be working again. Was maybe just a Discord issue? Oh well, closing this. Thanks!


Bad Oompa Loompas.

Installed docker on an ubuntu machine at digital ocean. Cloned this repo. Ran setup.sh. Ran run-taiga.sh. Connect to ipaddress:80. See login form. Login form redirects to /error after a second and shows "Something happened and our Oompa Loompas are working on it. Try reloading again soon." How do I fix?

I had the same error.
Exists a solution?

Hi there, I just had the same. Frustrating, the rest worked like a charm. I found it was probably about some localhost setting to be replaced with my server IP address, but I was not able where to put that in so far...

Same problem

The problem is because the frontend build. I'm refactoring the whole lot (and updating it to the latest taiga version). Hopefully, it will be fixed soon.

Yves
2015-07-21 0:06 GMT+02:00 Ivan Pedrazas<EMAIL_ADDRESS>
The problem is because the frontend build. I'm refactoring the whole lot
(and updating it to the latest taiga version). Hopefully, it will be fixed
soon.
—
Reply to this email directly or view it on GitHub
https://github.com/ipedrazas/taiga-docker/issues/12#issuecomment-123062447
.

Right now It works but I need to update the doc.

I'm totally not a python guru (php guy) and got this error just a few minutes ago.  I also used a fresh Digital Ocean Docker Droplet (14.04 ubuntu x64 / Docker 1.7.1).  Here is a python traceback during
-> Generate sample data
Trying import local.py settings...
Monkey patching...
Traceback (most recent call last):
File "manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python3.4/site-packages/django/core/management/init.py", line 385, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python3.4/site-packages/django/core/management/init.py", line 377, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/usr/local/lib/python3.4/site-packages/django/core/management/base.py", line 288, in run_from_argv
self.execute(*args, **options.dict)
File "/usr/local/lib/python3.4/site-packages/django/core/management/base.py", line 338, in execute
output = self.handle(*args, **options)
File "/usr/local/lib/python3.4/site-packages/django/db/transaction.py", line 394, in inner
return func(*args, **kwargs)
File "/taiga/taiga/projects/management/commands/sample_data.py", line 117, in handle
self.users.append(self.create_user(counter=x))
File "/taiga/taiga/projects/management/commands/sample_data.py", line 351, in create_user
full_name = full_name or "{} {}".format(self.sd.name('es'), self.sd.surname('es', number=1))
File "/usr/local/lib/python3.4/site-packages/sampledatahelper/mixins/localized_mixin.py", line 33, in name
return Name().generate(self, locale, number, as_list)
File "/usr/local/lib/python3.4/site-packages/sampledatahelper/l10n/names.py", line 43, in generate
names = self.get_names(locale)
File "/usr/local/lib/python3.4/site-packages/sampledatahelper/l10n/names.py", line 24, in get_names
self.__load_locale(locale)
File "/usr/local/lib/python3.4/site-packages/sampledatahelper/l10n/names.py", line 19, in __load_locale
Name.data[locale] = json.load(fd)
File "/usr/local/lib/python3.4/json/init.py", line 265, in load
return loads(fp.read(),
File "/usr/local/lib/python3.4/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 3925: ordinal not in range(128)
taiga-front
taiga-back
postgres
root@crossoff:~/taiga-docker# ls
backend  docker-compose.yml  frontend  install-docker.sh  LICENSE  README.md run-taiga.sh setup.sh
root@crossoff:~/taiga-docker# ./run-taiga.sh
postgres
taiga-back
taiga-front

Patience... The new taiga docker is coming. I just need to find some quiet time to finish it off :smile:

Everything should be fine now. Remember to delete your old containers.

why does this step:
docker run --name postgres  postgres
run in the foreground now for me and not background? it's causing your setup script to hang for me at that step.

Oh, it must be because I forgot to put the -d flag ... :facepalm: :)
I've fixed it already. @ceesco53 are you running boot2docker?

I am not using boot2docker - I'm using a digitalocean droplet which comes with ubuntu 14.04 / docker 1.7.1 preinstalled.  I'll edit my comment here in a few minutes when I get to my work laptop with the right ssh key on it...

Excellent!
Let me know how it goes

@ipedrazas The setup.sh script worked flawlessly.  Well done!  Now I'm interested in best practices for updating the front/back-end/schema.  Maybe another thread or update the readme?  I'd appreciate the insight - being a docker noob for the foreseeable future.  Anyhow, I'd say you can close this ticket again.

@ceesco53 I guess that the answer to that question is "it depends", I would like to think that 3 containers are fully independent but that's not exactly true.
Anyway, happy that it did work for you.


ArrayHelper::isIn, problem with non-strict comparison

What steps will reproduce the problem?
Execute next row
ArrayHelper::isIn('IAB25', [0, 'IAB26']);
What is the expected result?
false
What do you get instead?
true
Additional info

Q
A

Yii version
2.0.10

PHP version
5.5.37-1+deprecated+dontuse+deb.sury.org~precise+1

Operating system
Ubuntu 12.04.5 LTS

You should set $strict paramter to true:
ArrayHelper::isIn('IAB25', [0, 'IAB26'], true);

Ok. But in the class yii\helpers\BaseHtml I have a problem with method renderSelectOptions: in the line 1724 $strict selected as false, so option with value IAB25 will improperly marked as selected.

I have not selected the correct title for issue, sorry.

Open a separated one.


[Fix] Fix coverage when saving 2d and 3d results simultaneously

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

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

Hi @windmakeppcool,
We'd like to express our appreciation for your valuable contributions to the mmdetection3d. Your efforts have significantly aided in enhancing the project's quality.
It is our pleasure to invite you to join our community thorugh Discord_Special Interest Group (SIG) channel. This is a great place to share your experiences, discuss ideas, and connect with other like-minded people. To become a part of the SIG channel, send a message to the moderator, OpenMMLab, briefly introduce yourself and mention your open-source contributions in the #introductions channel. Our team will gladly facilitate your entry. We eagerly await your presence. Please follow this link to join us: https://discord.gg/UjgXkPWNqA.
If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID".
Thanks again for your awesome contribution, and we're excited to have you as part of our community!

Please sign the CLA license, thanks

Ready to be merged @Tai-Wang


Improvment to define group extractor claim

Description:
This is an improvement to the existing DefaultGroupExtractor to be able to configure the user claim from a configuration file rather than the hard-coded organization claim URI.
To enable this add the following configuration under  tag in <APIM_HOME>/repository/conf/api-manager.xml file
<DefaultGroupExtractorClaimUri>http://wso2.org/claims/role</DefaultGroupExtractorClaimUri>

Affected DOCs: Sharing Applications Between Multiple Groups and onwards
Note: The fix was given in WSO2-CARBON-PATCH-4.4.0-2810 as WUM Update

@tmkasun , here is the draft page [1] that I did with this WUM update. Please let me know whether any further changes are needed before I add these changes into the live docs. [1] https://docs.wso2.com/display/AM210/Copy+of+Sharing+Applications+Between+Multiple+Groups#CopyofSharingApplicationsBetweenMultipleGroups-Definingacustomuserclaimasthegroupextractorclaim

Based on the following WUM updates [4], I updated the APIM 2.1.0 [1], 2.2.0 [2] and 2.5.0 [3] docs with details on this new feature.
[4]
WSO2-CARBON-PATCH-4.4.0-2810 | API Manager 2.1.0 |<EMAIL_ADDRESS>| 06/12/2018 |
WSO2-CARBON-PATCH-4.4.0-3665 | API Manager 2.2.0 |<EMAIL_ADDRESS>| 12/09/2018 |
WSO2-CARBON-PATCH-4.4.0-3792 | API Manager 2.5.0 |<EMAIL_ADDRESS>| 01/08/2019 |
[1] https://docs.wso2.com/display/AM210/Sharing+Applications+Between+Multiple+Groups#SharingApplicationsBetweenMultipleGroups-Definingacustomuserclaimasthegroupextractorclaim
[2] https://docs.wso2.com/display/AM220/Sharing+Applications#SharingApplications-Definingacustomuserclaimasthegroupextractorclaim
[3] https://docs.wso2.com/display/AM250/Sharing+Applications#SharingApplications-Definingacustomuserclaimasthegroupextractorclaim

This information was added in APIM 2.6.0 based on the fix done in https://github.com/wso2-support/carbon-apimgt/pull/1420
[1] https://docs.wso2.com/display/AM260/Sharing+Applications#SharingApplications-Definingacustomuserclaimasthegroupextractorclaim


Strange behavior with ENABLE_VIRTUAL_TERMINAL_INPUT | ENABLE_LINE_INPUT | ENABLE_PROCESSED_INPUT

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.153]
PowerShell Core 7.0.0
.NET Core 3.1.200

Steps to reproduce
C# program:
using System;
using System.Diagnostics;
using System.Text;
using static Vanara.PInvoke.Kernel32;

namespace Test
{
    static class Program
    {
        static void Main()
        {
            var stdin = GetStdHandle(StdHandleType.STD_INPUT_HANDLE);

            Debug.Assert(GetConsoleMode(stdin, out CONSOLE_INPUT_MODE inMode));

            inMode |= CONSOLE_INPUT_MODE.ENABLE_VIRTUAL_TERMINAL_INPUT;

            Debug.Assert(SetConsoleMode(stdin, inMode));

            Console.WriteLine(Console.ReadLine());
        }
    }
}

(Uses the Vanara.PInvoke.Kernel32 NuGet package for convenience.)
You will need to test this program in both CMD and PowerShell Core.
Try doing these:

Type aaa bbb ccc and press backspace a couple times.
Type aaa bbb ccc and press control + backspace several times.

Also, try using special keys (such as arrow keys) after the program closes.
Actual behavior
For the first part:

Words are deleted with each backspace press.
Characters are deleted with each control + backspace press.

This behavior happens in both CMD and PowerShell Core.
For the second part:
After closing the program, you'll notice that e.g. using the arrow keys echoes VT sequences (such as ^[[D^[[A^[[C for left, up, right) rather than doing what they're supposed to. This behavior persists even if the ReadLine + WriteLine is removed from the program.
This behavior only happens in CMD. PowerShell Core seems fine.
Expected behavior
I have no idea to what extent, if any, this behavior is actually buggy/unintended. But here is what I would expect as a user:

Backspace and control + backspace behavior should not be swapped like this. This seems very strange and unlike other terminals.
Console modes set from within a program should not persist after that program exits. PowerShell Core appears to get this right, while CMD does not, for whatever reason.

Perhaps worth noting that running CMD and PowerShell Core under Windows Terminal has no impact on the behavior seen here.
Also, I wasn't sure whether to file this as two separate issues. I can do that if necessary.

Thanks for doing all the legwork here! This is, unfortunately, to a fairly great extent "by design." The input and output modes are global state, and while we're thinking about what it would take to change that we're definitely concerned about backwards compatibility.
When an application exits and leaves the console in a state different from the one it has when it launched (which can happen for a number of reasons: it wasn't authored correctly, it crashes, it returns control of the console so the shell before it's ready (#4921)), we have little choice but to get into a weird state.
You see this with most terminals on the market, unfortunately.  It's impossible to solve in that case, but given that we own the console subsystem in its entirety we can put together a better story.
I'm going to close this one as a /dupe of #4954, which we'll use to track the overall scenario.

Thanks for the response @DHowett-MSFT!
I just wanted to clarify a few things:
For the mode issue, my understanding (after reading the linked issues) is that the reason my particular sample here works properly in PowerShell is that PowerShell actually makes an effort to set a sensible console mode before/after running a program, while CMD does not. Is that correct?
If so, then I suppose the best I can do right now is try really hard to set the console mode back to its original state, even in the case of a crash, to make things work under CMD?
Regarding the backspace issue I also brought up here, as far as I can tell, this isn't explained by the linked issues. To be clear, this issue happens even if you run the sample program from a 'clean' console.

while CMD does not

Exactly this.
The backspace issue is an intricacy of how backspace is encoded in ENABLE_VIRTUAL_TERMINAL_INPUT mode, actually. When an application requests VT input, backspace is translated as 0x7f (DEL) instead of 0x8 (BS). Ctrl+Backspace is, then, translated as 0x8 (BS) instead of 0x7f (DEL). It's simply a matter of coincidence that they're flipped, but COOKED_READ then can't tell if the user actually pressed ctrl+backspace or it's only receiving 0x7f because it's in VT input mode.

(because COOKED_READ isn't supposed to be used in VT input mode)

What does COOKED_READ refer to here?

Ah, sorry. The input mode that CMD and legacy versions of PowerShell (without PSReadline) use. It's the mode there the console host handles producing a single line of input for the client application and only returns it when the user mashes Enter.

It's the opposite of "raw" input :wink:

OK, so to be clear, is the implication here that ENABLE_VIRTUAL_TERMINAL_INPUT in and of itself effectively means "raw mode"? In other words, that I'm pretty much expected/required to unset ENABLE_ECHO_INPUT | ENABLE_LINE_INPUT | ENABLE_PROCESSED_INPUT at the same time as I set the former?

It would be ideal if you do so, yeah -- if all you want is actual VT input, unmanipulated by the console host, you're going to want to disable line and processed input.
Most VT input consumers are doing their own line editing and want to get hold of the arrow keys and other keys germane to line editing, or they want to get mouse input in a cross-platform way, and having the processed or line input modes enabled will make it difficult to do that.

It would be ideal if you do so, yeah -- if all you want is actual VT input, unmanipulated by the console host, you're going to want to disable line and processed input.

But the thing is, that's not what I want. 🙂 For this project I'm working on, I'm trying to expose a consistent terminal interaction API that works the same way on Windows and Unix systems, and this issue touches on a pretty fundamental aspect of that.
I'm attempting to get the same behavior on Windows that you get in the Unix world: VT keyboard input at all times, with cooked mode by default, and raw mode optionally available via tcsetattr/termios. That's why I'm enabling ENABLE_VIRTUAL_TERMINAL_INPUT but leaving the other flags enabled, only disabling them when the user of the library explicitly requests raw mode.
Is there nothing that can be done to make the Windows console host handle this situation properly?

Sorry, I misunderstood your use case. I'll try to break down a couple of the scenarios and the flags that'll light them up. Would you let me know if I've captured the ones you're looking for?

If you want completely raw input, with VT sequences, as they are generated by a terminal:

use ENABLE_VIRTUAL_TERMINAL_INPUT, disable ENABLE_LINE_INPUT, ENABLE_ECHO_INPUT, ENABLE_PROCESSED_INPUT
Try to always use ReadConsoleInput, as that's the only way to be notified of window size changes (we have a workitem tracking a real SIGWINCH-like signal)

If you want input that is broken up by line...

... containing VT and control characters (^C, ^H, ^M):

use ENABLE_VIRTUAL_TERMINAL_INPUT | ENABLE_LINE_INPUT, disable ENABLE_PROCESSED_INPUT, your choice for echo
lines will be delimited by ^M

... containing VT, but no control characters (I think this is the Unix TTY vt-cooked mode?):

use ENABLE_VIRTUAL_TERMINAL_INPUT | ENABLE_LINE_INPUT | ENABLE_PROCESSED_INPUT, your choice for echo
lines will be delimited by ^M ^J (the Windows way)
you will never receive ^H in your buffer

... with Console-mediated line editing, but no VT:

use ENABLE_LINE_INPUT | ENABLE_PROCESSED_INPUT
lines will be delimited by ^M ^J (the Windows way)
you will never receive ^H or ^[[A in your buffer

Okay, now I get it. Item 2 sub 3, the one I think is vt-cooked mode, has the wrong behavior for backspace; this seems to be true back a couple major releases of windows. That's not excellent.
I'll check this out versus the version that introduced ENABLE_VIRTUAL_TERMINAL_INPUT.
Sorry, I had to talk myself through your problem. :smile:

Maybe it's easier to explain with this C program:
#include <unistd.h>

int main()
{
    char buf[4096];

    write(STDOUT_FILENO, "reading: ", 9);
    ssize_t len = read(STDIN_FILENO, buf, sizeof(buf));
    write(STDOUT_FILENO, buf, len);

    return 0;
}

Run on a Linux system (or WSL) with gcc test.c && ./a.out.
You will notice that you are able to do fairly basic line editing while the program reads from stdin, i.e. the TTY is in cooked mode. The exact level of line editing available will probably depend on your terminal and termios settings (the latter of which is usually the shell's responsibility). In my case, I'm able to use Backspace, but not arrow keys.
But, you will also notice that if you type some special keys, they are picked up as VT sequences by the program. For example, aaa followed by Ctrl+X and Enter prints aaa for me.
This is the behavior I'm trying to achieve.
To be clear, though, I don't particularly care what amount of editing features are available in cooked mode. It's okay that those are different between terminals and platforms. The important thing here is achieving the cooked + VT behavior, but not breaking Backspace / Ctrl+Backspace behavior on Windows in the process.

@DHowett-MSFT This issue is quite annoying as it is making ENABLE_VIRTUAL_TERMINAL_INPUT quite useless when in cooked mode.
I have tried to investigate this and the end story is that the key is mapped by this table here for some reasons that I don't know:
https://github.com/microsoft/terminal/blob/1e22bfa8d6f86d0546c3f162d171274cf453db65/src/terminal/input/terminalInput.cpp#L80
I could make a PR and remove this line, but I'm not sure about the consequences. Thoughts?

I tested by modifying this two places in src\terminal\input\terminalInput.cpp:
https://github.com/microsoft/terminal/blob/1e22bfa8d6f86d0546c3f162d171274cf453db65/src/terminal/input/terminalInput.cpp#L78-L81
static constexpr std::array<TermKeyMap, 19> s_keypadNumericMapping{
    TermKeyMap{ VK_TAB, L"\x09" },
-    TermKeyMap{ VK_BACK, L"\x7f" },
    TermKeyMap{ VK_PAUSE, L"\x1a" },

https://github.com/microsoft/terminal/blob/1e22bfa8d6f86d0546c3f162d171274cf453db65/src/terminal/input/terminalInput.cpp#L218-L221
static constexpr std::array<TermKeyMap, 14> s_simpleModifiedKeyMapping{
-    TermKeyMap{ VK_BACK, CTRL_PRESSED, L"\x8" },
-    TermKeyMap{ VK_BACK, ALT_PRESSED, L"\x1b\x7f" },
-    TermKeyMap{ VK_BACK, CTRL_PRESSED | ALT_PRESSED, L"\x1b\x8" },
+    TermKeyMap{ VK_BACK, CTRL_PRESSED, L"\x7f"},
+    TermKeyMap{ VK_BACK, ALT_PRESSED, L"\x1b\x8" },
+    TermKeyMap{ VK_BACK, CTRL_PRESSED | ALT_PRESSED, L"\x1b\x7f" },

And the terminal seems to behave correctly in with or without ENABLE_VIRTUAL_TERMINAL_INPUT
There are a few other places in the file where VK_BACK is mapped to x7f, so not sure If I should change these as well.

@xoofx I think this might be relevant context: https://github.com/microsoft/terminal/issues/5957#issuecomment-630493811

wow, ok, thanks for the link, what mess 😅
So basically:

CTRL-H is usually mapped to \x8, e.g deletes a char on the left in a prompt shell
Backspace is usually mapped to \x7f, e.g deletes a char on the left in a prompt shell
CTRL+Backspace does not exist

While we would hope such mapping, as the one we have on a regular powershell/cmd on Windows:

CTRL-H = \x8
Backspace \x8
CTRL+Backspace \x7f (so Backword)

But it's incompatible with vt/unix terminals and CTRL+Backspace does not exist... 🤦
So not sure how to solve this then.

So thinking more about it, in the presence of ENABLE_VIRTUAL_TERMINAL_INPUT, I would propose that wt:

Keeps CTRL-H = \x8
Keeps BackSpace = \x7f but interpret it as real backspace instead of transforming it to a backword
Could translate CTRL+Backspace to \x17 (so CTRL+W, Backword). If not keep CTRL+Backspace = \x8. I don't know how much Unix folks cares about CTRL+Backspace mapping to \x8.

Tested this fix proposal for readDataCooked.cpp by having both \x8 and \x7f deleting previous space when ENABLE_VIRTUAL_TERMINAL_INPUT is setup:
diff --git a/src/host/readDataCooked.cpp b/src/host/readDataCooked.cpp
index 169b0973..c33e6f2f 100644
--- a/src/host/readDataCooked.cpp
+++ b/src/host/readDataCooked.cpp
@@ -503,6 +503,8 @@ bool COOKED_READ_DATA::ProcessInput(const wchar_t wchOrig,
         return true;
     }

+    bool isInVirtualTerminalInputMode = _pInputBuffer->IsInVirtualTerminalInputMode();
+
     if (wch == EXTKEY_ERASE_PREV_WORD)
     {
         wch = UNICODE_BACKSPACE;
@@ -558,7 +560,7 @@ bool COOKED_READ_DATA::ProcessInput(const wchar_t wchOrig,
                     _currentPosition -= 1;

                     // Repeat until it hits the word boundary
-                    if (wchOrig == EXTKEY_ERASE_PREV_WORD &&
+                    if (!isInVirtualTerminalInputMode && wchOrig == EXTKEY_ERASE_PREV_WORD &&
                         _bufPtr != _backupLimit &&
                         fStartFromDelim ^ !IsWordDelim(_bufPtr[-1]))
                     {
@@ -636,7 +638,7 @@ bool COOKED_READ_DATA::ProcessInput(const wchar_t wchOrig,
                     NumSpaces = 0;

                     // Repeat until it hits the word boundary
-                    if (wchOrig == EXTKEY_ERASE_PREV_WORD &&
+                    if (!isInVirtualTerminalInputMode && wchOrig == EXTKEY_ERASE_PREV_WORD &&
                         _bufPtr != _backupLimit &&
                         fStartFromDelim ^ !IsWordDelim(_bufPtr[-1]))
                     {

Hi folks, any thoughts on @xoofx's comments above?

short notes I'll fix up tomorrow morning: the fullest solution is to do translation at read time not insert time, but that's Hard to impossible. These modes together will probably always be a little fucky, but they could be less fucky. Proposed fix seems close, but is just all backspace. We need to flip the meaning of DEL and BS in this mode, not just always do the backspace thing

"tomorrow morning" lmao sorry for the delay

The problem is that we translate INPUT_RECORDs into their VT equivalent at the time they are written into the Input Buffer. The most correct solution would be to do taht translation at the time the events are read, but that's basically impossible. What if the caller tried to only read one event, but the VT translation would convert it to multiple events. What happens then? The buffer isn't quite set up to handle that kind of scenario.
These modes will always be weird - consider, you're in VT Input + cooked read, and you press the arrow keys. The events that get read are all VT encoded, like ^[[A! Yikes!
Even if this mode is fucky, it could sensibly be less fucky. The fix above is really close, but that seem slike it always results in both DEL and BS being treated as backspace. We should make sure that DEL gets treated as "backspace" and BS gets treated as "delete", as usual.


Link to violin plot in R broken

Link to violin plots in R is broken. This should correct it.
I was not sure whether the html page was automatically rebuilt from the Rmd, I modified both. As well as modal in index.html.
You can squash this PR for a cleaner history...

Thanks a lot, much appreciated!


An editing error in the Tutorial Documentation

Thank you for ur contribution, I am a newbie to this project, maybe I found an editing error in the latest Tutorial Documentation. The detailed description is shown in the figure.

it should be python tools/test.py configs/balloon/mask_rcnn_r50_caffe_fpn_mstrain-poly_1x_balloon.py work_dirs/mask_rcnn_r50_caffe_fpn_mstrain-poly_1x_balloon/latest.pth --eval bbox segm

Seems you are right, you can create a Pull Request to fix this error


cuda11.2 and bitsandbytes-0.39 or 0.26 are not compatible??

if i installed bitsandbytes_cuda112-0.26.0.post2-py3-none-any.whl
The error would be "AttributeError: module 'bitsandbytes.nn' has no attribute 'Linear8bitLt'"
Then if i installed bitsandbytes-0.39.0-py3
The error would be "CUDA Setup failed despite GPU being available #396"
Please help!

I'm on Cuda 117 in WSl and getting same error.
AttributeError: module 'bitsandbytes.nn' has no attribute 'Linear8bitLt'


Installation with AUFS backend on Ubuntu14.04

Hi,
This is more an observation rather than reporting an issue. This is my environment:

Docker 1.6
Ubuntu 14.04 LTS
Linux kernel v3.13.0

I am facing similar issues as reported in https://github.com/docker/docker/issues/4036. One of the options suggested is to use AUFS as the backend. In fact, Ubuntu installation script at https://get.docker.com/ installs kernel extras package itself so that AUFS support is available.
I noticed that http://aufs.sourceforge.net/ says AUFS3 doesn't support my kernel version anymore. Given this, I wonder if the installation script should warn the users if they are on an unsupported kernel version?

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Though this is references #4036, the point being discussed is not the same. So, I am re-opening it.

@tusharm It has been detected that this issue has not received any activity in over 1 year. Can you please let us know if it is still relevant:

For a bug: do you still experience the issue with the latest version?
For a feature request: was your request appropriately answered in a later version?
Thank you!


Allow forcing the use of the generic updater in extra-files

Is your feature request related to a problem? Please describe.
Since #2072, any JSON, XML, YAML or TOML file is updated using their corresponding updater, meaning that the value of a top level version key, if existing, is bumped. It should be possible to override this default behavior.
My use case is to bump a docker image tag in a docker-compose.yaml file, that contains a top-level version key, like such:
# docker-compose.yaml
version: "3.8"  # Docker version: https://docs.docker.com/compose/compose-file/04-version-and-name/

services:
  my-service:
    ...
    image: my-org/my-repo:v0.1.0 # x-release-please-version

I tried two things:
"extra-files": ["docker-compose.yaml"]

When using this config, the version: "3.8" is replaced by version: v0.1.1, and the image value is left untouched.
"extra-files": [
  {
    "type": "yaml",
    "path": "docker-compose.yaml",
    "jsonpath": "services.my-service.image"
  }
]

When using this second config, the image value is transformed to v0.1.1, deleting the my-org/my-repo: part.
Describe the solution you'd like
Allow something like this, that won't attempt to modify the top-level version:
"extra-files": [
  {
    "type": "generic",
    "path": "docker-compose.yaml"
  }
]

Describe alternatives you've considered
As specified there, the version key is not strictly necessary for a docker-compose file, so removing this key leads to the desired behavior. However, this is only a temporary workaround.

The jsonpath must begin with a $. You'll get an error because jsonpath library doesn't support - (https://github.com/dchester/jsonpath/issues/90#issuecomment-397335671):
Error: release-please failed: Lexical error on line 1. Unrecognized text.
$.services.my-service.image
-------------^

The property must be enclosed in quotes:
"extra-files": [
  {
    "type": "yaml",
    "path": "docker-compose.yaml",
    "jsonpath": "$.services[\"my-service\"].image"
  }
]

@regseb this won't work with type as it will reformat the file and remove comments and purge quotes as per issues https://github.com/google-github-actions/release-please-action/issues/896 and https://github.com/googleapis/release-please/issues/2195
We would want to keep the generic updater with yaml files if we want or prevent the yaml updater from reformating the whole file when setting the new version


I have an error in importing cudamat.

Hi,
I set up cudamat in Windows environment.
At first, I tried to install cudamat in source directory, but I failed.
And so, I retried it as you guided (pip install http://gitbut....zip), and I succeeded installing it.
I confirmed the cudamat directory was created in Anaconda's site-packages directory.
And then, I tested whether it works well. (python -> import cudamat)
But, it won't work well. I had following error messages.
Please, let me know how to solve this problem. Thank you.
=========================================
Traceback (most recent call last):
File "", line 1, in
File "C:\Anaconda3\lib\site-packages\cudamat_init_.py", line 1, in
from .cudamat import *
File "C:\Anaconda3\lib\site-packages\cudamat\cudamat.py", line 15, in
cudamat = load_library('libcudamat')
File "C:\Anaconda3\lib\site-packages\cudamat\cudamat.py", line 13, in load_library
basename + sysconfig.get_config_var('SO')))
File "C:\Anaconda3\lib\ctypes_init.py", line 425, in LoadLibrary
return self.dlltype(name)
File "C:\Anaconda3\lib\ctypes_init.py", line 347, in init
self._handle = _dlopen(self._name, mode)
OSError: [WinError 126]

Googling for WinError 126 tells that it's The specified module could not be found. This either means the libcudamat DLL is missing, or it depends on other DLLs that are missing. It might be the same problem as in https://github.com/cudamat/cudamat/issues/81#issuecomment-313652149, where the files were given a different name during compilation than expected during import. I'm neither a Windows nor Anaconda user, so it's difficult for me to understand what's happening.
Can you do dir C:\Anaconda3\lib\site-packages\cudamat, and python -c "import sysconfig; print sysconfig.get_config_var('SO')"?

C:\Program Files\Python35\Lib\site-packages\cudamat>python -c "import sysconfig; print (sysconfig.get_config_var('SO'))"
.pyd
C:\Program Files\Python35\Lib\site-packages\cudamat>dir
Volume in drive C is OS
Volume Serial Number is 9D68-3FF4
Directory of C:\Program Files\Python35\Lib\site-packages\cudamat
12/23/2017  02:41 PM              .
12/23/2017  02:41 PM              ..
12/23/2017  02:41 PM            47,232 cudamat.cu
12/23/2017  02:41 PM            47,348 cudamat.py
12/23/2017  02:41 PM            26,861 cudamat_kernels.cu
12/23/2017  02:41 PM               831 learn.cu
12/23/2017  02:41 PM               504 learn.py
12/23/2017  02:41 PM               374 learn_kernels.cu
12/23/2017  02:41 PM               898 libcudalearn.cp35-win_amd64.exp
12/23/2017  02:41 PM             2,304 libcudalearn.cp35-win_amd64.lib
12/23/2017  02:41 PM           256,512 libcudalearn.cp35-win_amd64.pyd
12/23/2017  02:41 PM             9,620 libcudamat.cp35-win_amd64.exp
12/23/2017  02:41 PM            18,070 libcudamat.cp35-win_amd64.lib
12/23/2017  02:41 PM           503,296 libcudamat.cp35-win_amd64.pyd
12/23/2017  02:41 PM           176,308 rnd_multipliers_32bit.txt
12/23/2017  02:41 PM                43 init.py
12/23/2017  02:41 PM     pycache
14 File(s)      1,090,201 bytes
3 Dir(s)  19,413,135,360 bytes free
C:\Program Files\Python35\Lib\site-packages\cudamat>

Hmm, that's the same problem as in https://github.com/cudamat/cudamat/issues/81#issuecomment-313652149. My guess would be that you accidentally used a different Python version for compilation.
What command did you run for installing cudamat?
What happens if you use: pip install http://github.com/f0k/cudamat/archive/simplify-compile.zip? That one uses a slightly different approach for compilation.

Hi. I have the same error in Windoze. I tried pip install http://github.com/f0k/cudamat/archive/simplify-compile.zip
Then cudamat was installed. But it still cannot be imported.
I was wondering which files I should rename? Or how could I fix it otherwise?
Thanks!


Restore native link functionality

Fixes https://github.com/allenai/scholar/issues/26381
This puts the native link functionality back in to the reader so things like links to other pages or internal links to headers work again.
@andrewhead I wanted to understand if there were overlapping issues you ran into before where these link annotations and our own would fight, do we need to bump these down in the z-order or something for that case or do you think this'll be okay?
Before:
Example from the PAWLS paper where this links to the app aren't clickable today:

After:
They are clickable and have a highlight hover:

One of the challenges of the overlapping bounding boxes is that, if the two boxes are slightly different, clicking on two different places in a citation could then have different interactive behavior. A long-term solution to this might be iterate over the ScholarPhi bounding boxes and the pdf.js bounding boxes, and disable those in pdf.js that overlap with ScholarPhi's.
An easier stopgap solution I think could be to do as you mentioned, and make sure that ScholarPhi bounding boxes appear on top.

@andrewhead do you happen to know of an example paper off the top of your head that has this overlap behavior I could use to test that out? I can do some random sampling but otherwise I think we might have to deploy it as is and wait for someone to report an issue on a specific paper?


[Bug] update README.md for single master cluster.

[BUG]/[FEATURE] title
What changes were proposed in this pull request?
Why are the changes needed?

What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @waitinfuture

LGTM. Thanks.


joints example not properly working

TL;DR: fix in rapier: https://github.com/dimforge/rapier/pull/660
description
steps:

cargo run --example joints2

Expected render

Actual render (as of 56f98706751cb5cf74cb82e2b73749282c8ed)

Same behaviour observed for joints3

Analysis
introduced in https://github.com/dimforge/bevy_rapier/pull/525 (5fb8a4e1eba588cd44a779911703d1f84dc3f1ad)
Position matrix for the dynamic entities contain infinity values 🤔

Will be fixed with rapier 0.21, keeping opened until we update.


feat(instrumentation-graphql): add option to ignore resolver spans

Which problem is this PR solving?
#1739
Short description of the changes
Add the option to ignore resolver spans.
Creating a span for each resolve can cause way too many spans to be created (e.g. 100k elements in the response equals at least 100k spans, most of them will be dropped). This causes huge latency increases - my test query with an in memory resolver went from 1.2s to 8-9s. With ignoreResolveSpans: true the response time went back to ~1.2s.
For reference, the Java implementation of GraphQL instrumentation ignores resolvers by default and there isn't even a possibility to enable creating spans from resolvers.
I think ignoring resolvers should be the default behaviour in the Node.js instrumentation as well, but for backwards compatibility I set the default value to false in this PR. Up for discussion whether to ignore resolvers by default or not.

@pichlermarc Are there any additional approves this PR needs?

Just noticed that this PR missed the documentation update on the README.


Regression: System.NotSupportedException: The stream does not support writing.

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[x] I have provided test coverage for my change (where applicable)

This is a duplicate of #1014. It happens when trying to upload a file larger than RequestStream.DEFAULT_SWITCHOVER_THRESHOLD.
Stacktrace:
System.NotSupportedException: The stream does not support writing.
   at System.Net.Http.StreamContent.ReadOnlyStream.Flush()
   at Nancy.IO.RequestStream.MoveStreamContentsToFileStream()
   at Nancy.IO.RequestStream.MoveStreamOutOfMemoryIfExpectedLengthExceedSwitchLength(Int64 expectedLength)
   at Nancy.IO.RequestStream..ctor(Stream stream, Int64 expectedLength, Int64 thresholdLength, Boolean disableStreamSwitching)
   at Nancy.IO.RequestStream..ctor(Stream stream, Int64 expectedLength, Boolean disableStreamSwitching)
   at Nancy.Owin.NancyMiddleware.<>c__DisplayClass2_1.<<UseNancy>b__1>d.MoveNext()

@thefringeninja, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

ping @NancyFx/most-valued-minions are we happy to pull this?

@thefringeninja it's a bit sad that you haven't poked @damianh every single day to review and pull this 😉

In fairness it was pointed out to me....


[PLT-789] - fix github-update job

PLT-789 - fix github-update regex pattern Now the regex will only loop through branches that start with release or hotfix
fix pull request template file name

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: PLT-789
The new version
New release version candidate: v3.0.1?

Reviewers, please check if the release candidate is present in the changes of this PR in the examples section!

What is the new behavior?
The github-approve job now has a updated regex to loop through only branches that start with release or hotfix.
The bug was that, when the project has legacy branch names with release or hotfix in the middle of the name like something-hotfix-wherever the job tryies to update a branch hotfix-wherever that does not exists and crashes like this example:
https://app.circleci.com/pipelines/github/dafiti-group/catalog-front/1039/workflows/82b8e570-0633-4906-9e6e-0319fecab7f2/jobs/3309
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:d758c22cd4a52a9ad882f5da4dcb65453d1f0b32

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:f571eb07bac02b1951135748db90328d4a6c9e20

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:ec483cc200b5c81df6870af13b7a8c1dbba42b85

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:70fbd24e40467bea04cbce50009a0b5421eb6df4

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:2e3dc11fc71567827b15fe4e2d15e484206b2692

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:ca0b04a23dc279708a68368ab67877c54d27d7c8

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:ca0b04a23dc279708a68368ab67877c54d27d7c8

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=3.0.1

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/dafiti-group/orb-standard-pipeline?version=dev:ca0b04a23dc279708a68368ab67877c54d27d7c8


Contact us Page

Fixes #2
Added contact us form in contact us page and maps in it with responsive behavior.
@smaranjitghose

@smaranjitghose


Updating to 1.18.2

Hello!
In the process of updating origins to 1.18.2 I had to update this. As far as I know it works mostly the same as before.
Some Tag objects needed to be changed to TagKeys as minecraft has done. Along with that, some of the mixins broke in 1.18.2 so I fixed those (only a method signature and change from a double to a float afaik).
As with calio the version i bumped the mod to may be too high as I don't know what counts as a breaking change. For what its worth, origins itself did not need to change what it was doing with regards to this library in particular.
One other note is that I couldn't figure out how to get the specific structure that's being located (locating a structure now takes a tag instead of a structure feature) in order to get its start. I just used the block position straight from locating the structure but that's probably not correct.

Before I forget again, the trouble I had with structures was in ModifyPlayerSpawnPower and getStructureLocation


German added

Please do not change anything or integrate it into the website, as this is still in the beta! Please do not cooperate either, as I will finish this in a very short time... I will install the updates here one by one and announce when it is ready...

Please do not change anything or integrate it into the website, as this is still in the beta! Please do not cooperate either, as I will finish this in a very short time... I will install the updates here one by one and announce when it is ready...

Sure np, I’ve marked this as draft until is ready

The flags are from

The flags are from https://twemoji.twitter.com/

And how do I find a specific country flag? The names are all only shown in codes!

For the german flag 🇩🇪 : https://github.com/twitter/twemoji/blob/master/assets/svg/1f1e9-1f1ea.svg

You can use this website

German translation completed! Ready for the release of Alpha Version 1.0

@MrMicky-FR

@MrMicky-FR
German translation completed! Ready for the release of Alpha Version 1.0

Hi, looks great, thanks a lot !
Just looks line the title section at the tops of the .md files aren’t translated yet, as well as the home link in the navbar.
I’ve also updated the config, so the german preview is now available here: https://deploy-preview-8--azuriom.netlify.app/de/

@MrMicky-FR
So?

Great, just didn’t saw it before but the AzLink page isn’t translated: https://deploy-preview-8--azuriom.netlify.app/de/azlink

But somehow it has no effect!

Hi, I fixed the issues, is it ready for you so I can merge ? For me looks good 👍

I deleted this because I couldn't see any differences in my instance! But yes, everything else is ready!

Great thanks !

Great thanks !

I will also keep it updated in case there are any updates...