Improve the data flow

At the moment there are major issues with the way api requests are made and timeouts are being set and requests not being cleaned up. As a first step to counter these issues we need to use some data flow architecture. I'm at the moment researching Redux and will start on an implementation in the next days.
This superseeds #199

I would also recommend looking into Rx as an option. I just finished this excellent series by André Staltz, who wrote the Introduction to Reactive Programming You've been Missing, and authored CycleJS as well as this mind-opening "Unidirectional Interface Architectures" article. This segment of the aforementioned series shows an entirely flow-based representation of the application state based on observables (just a counter example, but if you watch to the end (8 minutes), it shows an interesting reactive way to manage state.
However, I haven't seen any large application use this pattern, though that is more likely due to my inexperience in this area than anything else. Alternatively I have used Redux on some projects recently, and it drastically improves the testability and clarity of the data flow in the application.

@aeschylus thanks for the mention. I've been looking into rxjs and the different ways how to use it in combination with React for some time now. The reason why I did not go the CycleJS route is that it would require a full rewrite of the webui + there are many more people at the moment using and understanding React and I'd love to have some help around here and keep it as accessible as reasonable.
BUT I just discovered this: https://gist.github.com/tomkis1/8b3573ca9483d60c8b34 redux-saga with rxjs observables, so I'm going to try and use this in my PR.
Also you can see the stack I'm using currently in #229 which is redux + redux-saga right now.

Also referencing https://github.com/yelouafi/redux-saga/issues/105 which I need to read, and everyone else interested in sagas and observables should probably as well.

Not sure what the status of this issue is (obviously you went with Sagas...). I use redux-observable in my current project and I like it a lot (and much more than Sagas) so far. Just wanted to advertise it here...
❤ IPFS and would like to contribute!

Thank you @mcmunder I have been using redux-logic in another project very happily since then which rx under the hood but provides a very nice way of interaction without actually having to use rx directly.

Great, thanks! Didn't know about redux-logic so I will definitely give it a closer look!

The coming release (2.0.0) uses redux!


ci: automate releases with release-please

Is your feature request related to a problem? Please describe.
Continous delivery would be nice. Release please promises to take care of that.
Mockups
Release please action creates Release PRs from develop to master, bumping the version and collecting changes to changelog.
After PRs are merged, creates release tag on master and releases to npm and github packages,
Checklist

[x] I've checked and no other issue talks about this.
[x] I have a solution for this. (I am willing to submit a PR solving this.) 😎

Flow updated to this:

Create PR from develop to master
Merge PR
Bot creates version PR
Accept bot PR (automatically?)
Bot creates tags and releases

Flow updated to this:
Create PR from develop to master
Merge PR
Bot creates version PR
Accept bot PR (automatically?)
Bot creates tags and releases

It worked! Bot PR must be accepted tho.


Problems running Node under Beaker on Windows.

I'm having trouble getting Node running under Beaker on Windows 8.1.
I'm running it from the command line which opens the Beaker window automatically in the browser. I create a new empty notebook then open the Language Manager and attempt to switch to Node. I get the following error:

I already have Node installed (v5.6.0). So I updated my preferences to point to it and I still can't start Node. I'm not sure if I have this correct:
{
  "pref-format" : "1",
  "allow-anonymous-usage-tracking" : false,
  "languages" : {
    "Node" : {
      "path" : "C:\\Program Files\\nodejs"
    }
  },
  "edit-mode" : "default"
}

I've checked the console where I'm running Beaker and don't see anything that looks like an error message.
Any idea where I've gone wrong?

I haven't tried the recent versions, for development we use v0.12, so I think that's your best bet.
Node isn't one of the major languages used for data science so we haven't had much demand for it and it's not well supported eg no autotranslation.
I would like to correct that!

looks like it's worse than that, the XSRF fix i put in a while ago appears to have broken node: https://github.com/twosigma/beaker-notebook/issues/3918

note that bug does not explain node not starting

fyi we have updated to node v5 for development and i confirmed it runs node plugin on windows.

https://github.com/twosigma/beaker-notebook/issues/3971

Thanks for looking into this. Sorry I haven't gotten back Beaker-Notebook yet. It's very important to me to get an integration working between this and Data-Forge.


az extension update - Allow single command to update all installed extensions

Related command

extension
Is your feature request related to a problem? Please describe.

az upgrade checks extensions too, but also updates Azure CLI.
az extension update requires --name, --name does not support * / wildcard for all.

Describe the solution you'd like
Let az extension update update all extensions in one command. Some ideas:

Simply az extension update
Or az extension update --all
Or wildcard az extension update --name *

Describe alternatives you've considered

Currently I first list all installed extensions, then run az extension update --name <name_of_extension> one by one.
Additional context

Doc: https://learn.microsoft.com/en-us/cli/azure/extension?view=azure-cli-latest#az-extension-update

Thank you for opening this issue, we will look into it.


Authorization issue with ML pipeline having a Sweep job

We have an ML training pipeline with SDK V2 that uses a Sweep job. The pipelines use the environment that exists in a Shared ML Registry.
We are utilizing the command components for the training and Sweep jobs. All is well if there is no sweep job as we have tested with scoring and monitoring pipelines and with training pipelines if we remove the Sweep job. However, with a Sweep job in pipeline components, no child runs ever passes/completes. Each of the trials inside the child job has an error:
UserError: This request is not authorized to perform this operation using this permission.

The overall job error says:
UserError: This request is not authorized to perform this operation using this permission. Marking the experiment as failed because initial child jobs have failed due to user error

Here is some psuedocode that can help:
try:
    logging.info('Getting latest environemnt from shared registry')
    matching_environments = [env for env in ml_client_registry.environments.list(e.aml_env_name_score) if any(e.target_env_tag in tag_value for tag_value in env.tags.values())]
    if matching_environments:
        pipeline_job_env = max(matching_environments, key=lambda env: env.version)
    else:
       raise Exception(f"No environment with {e.target_env_tag} tag found.")
except Exception as excp:
    logging.error('Unable to get latest environemnt from ML registry.')
    raise excp

@pipeline()
def sweden_backbook_train_with_hyperparameters(
    reg_targets,
    clf_targets,
    ml_datafile,
    preprocessing_config_file,
    port_agg_col,
    tgt_period_col,
    gpu_flag,
    dataset_deploy_flag,
    snapshot,
    deploy_flag,
    project_name,
    model_name,
    output_dir
):
    train_component = command(
	name="backbook-train",
	display_name="Training",
	description="Performs training and produces model",
	inputs={
		"reg_targets": Input(type='string'),
		...
		...
		...
		'reg_colsample_bylevel': Input(type='integer'),
		'reg_reg_alpha': Input(type='integer'),
		'reg_reg_lambda': Input(type='integer')
	},
	code=e.sources_directory,
	command="python -m train.main_portfolio",
	environment=pipeline_job_env,
    ).component(
	reg_targets=reg_targets,
	...,
	...
    )

    search_space = {
	...,
	...,
	'reg_colsample_bylevel': Uniform(min_value=0.3, max_value=1),
	'reg_reg_alpha': Choice([0, 0.01, 0.1, 0.5, 1, 10]),
	'reg_reg_lambda': Choice([0, 0.01, 0.1, 0.5, 1, 10]),
	'reg_tweedie_variance_power': Uniform(min_value=1.2, max_value=1.99),
	'zip_regressor': Choice(['Y','N'])
    }
    sweep_step = train_component.sweep(
	primary_metric="custom_mae_mean_cv",
	goal="minimize",
	sampling_algorithm='bayesian',
	search_space=search_space,
	compute=e.compute_name_train
    )
    sweep_step.set_limits(max_total_trials=int(e.max_total_trials), max_concurrent_trials=int(e.max_concurrent_trials), timeout=int(e.timeout))

We have reviewed all permissions on our end. We use a compute cluster for our pipelines. the cluster has a user-managed identity assigned and that identity has Manager rights on the shared registry. The pipeline works if there is no sweep job.
We believe it is a bug, can we please get some assistance with this?

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

ID: f97f70b2-cf01-935c-3518-8476907c04be
Version Independent ID: 455fd1a3-5529-efb9-8a36-63d43e45d6b8
Content: azure.ai.ml package
Content Source: docs-ref-autogen/azure-ai-ml/azure.ai.ml.yml
Service: machine-learning
GitHub Login: @lmazuel
Microsoft Alias: lmazuel

@obiii
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

Hi,
I have attached the pseudocode in my post.
I followed a combination of documents (both git examples and azure sdkv2 docs):
For command component:

https://github.com/Azure/azureml-examples/blob/main/sdk/python/jobs/pipelines/1d_pipeline_with_non_python_components/pipeline_with_non_python_components.ipynb
https://learn.microsoft.com/en-us/python/api/azure-ai-ml/azure.ai.ml?view=azure-python#azure-ai-ml-command

For sweep:
https://github.com/Azure/azureml-examples/blob/main/sdk/python/jobs/pipelines/1c_pipeline_with_hyperparameter_sweep/pipeline_with_hyperparameter_sweep.ipynb
upgrade pipelines to v2:
https://learn.microsoft.com/en-us/azure/machine-learning/migrate-to-v2-execution-pipeline?view=azureml-api-2

@obiii It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

Hi,
I have attached the pseudocode in my post.
I followed a combination of documents (both git examples and azure sdkv2 docs):
For command component:
https://github.com/Azure/azureml-examples/blob/main/sdk/python/jobs/pipelines/1d_pipeline_with_non_python_components/pipeline_with_non_python_components.ipynb
https://learn.microsoft.com/en-us/python/api/azure-ai-ml/azure.ai.ml?view=azure-python#azure-ai-ml-command
For sweep:
https://github.com/Azure/azureml-examples/blob/main/sdk/python/jobs/pipelines/1c_pipeline_with_hyperparameter_sweep/pipeline_with_hyperparameter_sweep.ipynb
upgrade pipelines to v2:
https://learn.microsoft.com/en-us/azure/machine-learning/migrate-to-v2-execution-pipeline?view=azureml-api-2

@Naveenommi-MSFT
please review it.

Hi @obiii
Thank you for your feedback! Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an Azure support request. Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds. Teams Q&A forum for technical questions about the configuration and administration of Microsoft Teams on Windows. Microsoft Teams Community forum Thank you for your time and patience throughout this issue.

@obiii
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


str_rev.cpp contains program to reverse words of the string in c++.

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

I'm closing the PR because of no corresponding Issue and the code plagiarized from here

I'm closing the PR because of no corresponding Issue and the code plagiarized from here

In that case you should also change your question because question is plagiarized from there only 😀


using mechanize and savon: ruby-ntlm clashes

Using mechanize and savon can lead to clashes of which version of ruby-ntml should be used. The issues is also reported here:
https://github.com/WinRb/rubyntlm/issues/13
The problem is that savon and mechanize each use (by direct or indirect require) ruby-ntlm.
Depending whether we used bundle exec or an installed version of our gem (that uses savon) the order of includes seems to differ and used different version of rubyntlm.
We solved it like this:
https://github.com/zdavatz/oddb2xml/commit/79257135b89b30617970460d4ac3f5771fc26e31

My modified gemspec drops ntlm-http dependency (thus mechanize will crash if savon is not installed). lol...
https://github.com/sparklemotion/mechanize/issues/407

Hi,
I think that, instead of using ntlm-http, mechanize should just use rubyntlm. ntlm-http actually bundles an old version of rubyntlm source code.
I could not find if mechanize used what is added by ntlm-http on top of standard ntlm, but at least it seems to work without any issue.

Thanks for posting!

Before closing, please update the gemspec... I have very bad difficulty using gem command. Actually I didn't want to close #407, but I was just sure my proposal would not be accepted (bah)


Explaination and Example for P value with code.

Issue no.: #3
PROJECT TITLE: Explaination and Example for P value with code.
I divided that issue into two parts one part is to explain the p-vale and the other part is how we implement P-value in data science.

1st part - Detail explaination of the P-value with a suitable example and graphs.
2nd part - Using Data set for explaining the code of P-value.
Dataset used - Boston Housing Data.
Using OLS method for finding P-value.

files:

P-value explaination
P-value by using python code

@Sumit11081996 great work , if u want u can add more issues for statistics


Fix locale dependent tests

The Travis build now passes, I relaunched the failing job on Travis since it failed for an unrelated error.
Feel free to merge!

Just merged it, looks good!


Latest releases not published to Atom

The current release available from the Atom module registry is still 0.2.0. How do we go about getting the latest version published there?

Hmm... I'm not sure what to do!
~/projects/language-graphql $ apm publish 0.5.0
Preparing and tagging a new version ✓
Pushing v0.5.0 tag ✓
Publishing<EMAIL_ADDRESS>✓
~/projects/language-graphql $ apm view language-graphql
language-graphql
├── 0.2.0
├── https://github.com/rmosolgo/language-graphql
├── Syntax highlighting for GraphQL queries
├── 1128 downloads
└── 7 stars

Run `apm install language-graphql` to install this package.

Oh hmmm, I'm publishing to the wrong name, let me check that out.
~/projects/language-graphql $ apm view atom-language-graphql
atom-language-graphql
├── 0.6.0
├── https://github.com/rmosolgo/language-graphql
├── Syntax highlighting for GraphQL queries
├── 36 downloads
└── 0 stars

:tada:
Thanks for bringing this to my attention!


Pull latest Aspen devices into structured dirs

The latest e.g. Aspen-4 chip will be available under ./tests/qpu-test-files/Aspen-4/latest (symlinked appropriately).

Is the failing test here related to your fix over in https://github.com/rigetti/quilc/pull/460 ?

Obviously it's not related to these changes, just wondering.

yes. rebasing onto the fix branch


Master

This does not seem like an intentional pull request.
Feel free to re-open and add more context.


fixed issue for when classification has . in name the termCount was n…

UI changes :

Fix test suite redirect issue after creation
Fixed tags redirect when having . in it
Fix entity popover card details
Fix Container entity loading issue for non-admin user

Backend:

fixed issue for when classification has  '.'  in name the termCount was getting 0.

Issue :
https://github.com/open-metadata/OpenMetadata/assets/66266464/b1d89897-1838-41ee-a11c-5c5fb62e473c
https://github.com/open-metadata/OpenMetadata/assets/66266464/75e51191-93c7-4c85-90ec-b59821a00510

Type of change:

[ x] Bug fix
[ x] Improvement

@ShaileshParmar11  changes made as per comments!


Eslint warnings

Changes

turned of the camelcase eslint rule

API uses snakecase and we cannot do anything about these warnings when we use variables from API responses.

fixed some minor eslint warnings

Codecov Report

Merging #270 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #270   +/-   ##
=======================================
  Coverage   83.23%   83.23%
=======================================
  Files         103      103
  Lines        1193     1193
  Branches      102      102
=======================================
  Hits          993      993
  Misses        181      181
  Partials       19       19

Impacted Files
Coverage Δ

...mart-components/portfolio/share-portfolio-modal.js
92.3% <ø> (-0.2%)
:arrow_down:

src/helpers/shared/helpers.js
90.47% <ø> (ø)
:arrow_up:

src/utilities/constants.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Assumes home directory is /Users/$(whoami)

Due to something related to Migration Assistant, my username is choris but my home directory is /Users/admin_1. This causes Yabai to fail when attempting to create the launch agent.
Symlinking /Users/choris to /Users/admin_1 fixes this issue, but is a workaround I'd rather avoid.

Text of error
yabai: service file '/Users/choris/Library/LaunchAgents/com.koekeishiya.yabai.plist' is not installed! attempting installation..
yabai: service file '/Users/choris/Library/LaunchAgents/com.koekeishiya.yabai.plist' could not be installed! abort..

Steps to reproduce

Create a user with a home directory that is not the output of echo /users/$(whoami).
Install and run Yabai.

https://github.com/koekeishiya/yabai/blob/master/CHANGELOG.md#505---2023-05-21

Oops, I'm on version 5.0.4 because my package manager (Nix) hasn't updated the Yabai package.
Sorry!


Fix for hipsparse in ROCm.

This PR fixes sparse ops for ROCm.
@jakevdp

This is a good idea. Let me work on it . I will update the PR with changes.

@jakevdp could you please take a look at the recent changes. I will squash the commits after you agreed with the changes.

We should fix the mypy errors here (assert sparse_apis is not None should do the trick I think)
Test failures are pre-existing on the main branch; I'll look into getting those fixed (tracked in https://github.com/google/jax/issues/9991)

@hawkinsp do you agree to make similar changes to other hip/cuda modules (hipsolver/cusolver, cublas/hipblas, etc. )?

So much deleted code 😍

@jakevdp I can use my git skills to remove all jax repo if you like.:satisfied:
https://youtu.be/2uYeb_c8_bk

Hi @jakevdp, does this PR need other changes from my side?
I think CI jobs were run before #9991 fix. If you restart the jobs, they all should be green.

@rsanthanam-amd

Hi, sorry I lost track of this. Can you rebase your commit on the main branch to see if it fixes the CI tests? Thanks

Thanks Jake. Rebased the PR and all are green now.

Thanks for working on this! If you're up for it, I think a similar treatment for hipsolver/cusolver and cublas/hipblas would be useful.


Added Linters for CSS, HTML, JS

I have added a GitHub action for lining JS, CSS, and HTML.

PR has been marked as draft as this is a WIP.

You can check the Action dashboard here
For now, the action ran on the whole codebase. I will change the action so that it runs on the PR.

Which Program do you belong to

@anushbhatia I doubt SWOC will accept this PR. So CrossWoc is the only suitable one


Remote share needs to provide Domain Admins with Full Control to run script

Hi,
Per https://github.com/Azure/ArcEnabledServersGroupPolicy/issues/31, if you run DeployGPO.ps1 having once provided Domain Admins with just 'Change' permissions to the remote share per current documentation, you get the following error:
Remote path  \\contoso.com\Deployment\Azure Arc Servers Onboarding found!
Creating remote folder's structure...
Assigning appropriate permissions...
Set-Acl : Attempted to perform an unauthorized operation.
At C:\Users\<username>\Downloads\ArcEnabledServersGroupPolicy_v1.0.8\ArcEnabledServersGroupPolicy_v1.0.8\DeployGPO.ps1:
108 char:1
+ Set-Acl -Path $AzureArcDeployPath -AclObject $Acl
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : PermissionDenied: [\\contoso.com...\AzureArcDeploy:String) [Set-Acl], UnauthorizedAcce
   ssException
    + FullyQualifiedErrorId : System.UnauthorizedAccessException,Microsoft.PowerShell.Commands.SetAclCommand

This issue does not occur if Domain Admins has Full Control of the share.
Suggest an update along these lines?
'The network share should provide Domain Controllers and Domain Computers with Change permissions, and Domain Admins with Full Control permissions.'

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

ID: 6ae288ed-d6fe-6337-482c-8e11cc5ec18b
Version Independent ID: 5b1ec441-86a2-1040-28da-d6f249d1e01e
Content: Connect machines at scale using Group Policy with a PowerShell script - Azure Arc
Content Source: articles/azure-arc/servers/onboard-group-policy-powershell.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

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


{Doc} Mark quoting-issues-with-powershell.md as obsolete

Description
Since we have now added ps1 entry scripts for PowerShell and PowerShell 7.3 fixed the dropping-double-quote issue, this document is no longer valid for the latest versions of these tools.
We notify the reader the document is obsolete.

We defer the merging of this PR until 2.40.0 is officially released.


Epic: Cluster Provisioning - RKE1 Node Provisioning

closing this as we have separate issues to track provisioning and any issues will be logged as separate tickets.


iMessage search extension

Description
Adds a very basic (but useful!) extension to search iMessages. It is implemented using the approach taken by the safari extension in that it uses the WebAssembly version of sql.js to load the SQLite databases of both the iMessage and Contacts apps into memory.
We could alternatively use native bindings to SQLite (such as sqlite3 or prisma) to support standard paging from the fs however this is likely overkill for this simple use-case (tested against my iMessage db that has ~200k messages in it and clocks in at around 160mb and my Contact's db with 339 contacts and is ~6mb). The sql.js approach has the benefit of not requiring unique Node native module binaries per ARM/x86 as well.
Some improvements for future contributors:

Improve contact lookup by searching against normalized phone numbers (Contact's ZABCDPHONENUMBER.ZFULLNUMBER is not normalized it seems)
Improve contact lookup by searching against email addresses (iMessage's handle.id can be a phone number, email address, or null it seems)
Render Contact image avatars (it seems like the data exists in Contact's ZABCDRECORD.ZTHUMBNAILIMAGEDATA however I was not immediately able to determine the binary format)
Avoid loading all data into memory (perhaps via. a WebAssembly-targeted version of Prisma client)
Use a first-class iMessage and Contacts API instead of reading the apps internal data :)

Screenshot

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Could it be an idea to build #1954 into this and make it a single extension instead?
Is there a way to show conversations by default instead of no results?

Hey @pernielsentikaer thanks for reviewing!

Is there no other way to achieve this so we could avoid asking for full disk access?

Non that I could find unfortunately. If Apple ever offered a public Swift or REST API for reading personal iMessages we could potentially use that.

Could it be an idea to build https://github.com/raycast/extensions/pull/1954 into this and make it a single extension instead?

I believe this implementation would support https://github.com/raycast/extensions/pull/1954 's use-case already.

Is there a way to show conversations by default instead of no results?

Sure I can make this change.

Hi @BalliAsghar 👋
Sorry for the late response, your PR #1954 was marked as resolved on my personal list.
Could it make sense to make one big extension with both this and yours?

I'm closing this due to inactivity; feel free to open it again or comment in the thread when you're ready to continue working on it 🙂
You can also catch me in Slack if you want to discuss 😄


runs as background process - no way to wait for it to be done

I think this runs asynchronously in the background.  In my script, I zip the newly created files before I email them, and I'd like there to be a way to wait for the process to complete.

same issue. @iateadonut any solution ?

I usually put in a while, waiting for the file to exist. Anybody has a better solution?

Thank you @pokono !! I've been struggling with this issue for over an hour and this seems to have solved it.

@pokono Thanks you sir!


[SCH-130] Removes left over symbolic link and updates travis yaml to latest standards

Also removed references to ansible-city

Looks good to me :tm:


[Identity] DeviceCodeCredential: Do we need to add success logging?

Other languages seem to have logging on success. Investigate if we already do this. If we don't already do this, add this.

Here's more info:

Upon successful authentication, the user is displayed with message ‘OK’ as an indicator of success on the browser. This message is short, can be improved and not consistent with other languages. In Java, for example, the message displayed to the user is “Authentication Complete. You can close the browser and return to the application.”

Successful browser auth message from Interactive Browser Credential in the Identity SDK.

So, the idea is to change this browser message.
I couldn't find any OK match on msal-node, other than https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/78334c21280ed0d2afd815cb1a7ff16b82f5a311/lib/msal-node/test/network/HttpClient.spec.ts . I believe they're echoing this in the browser somehow.

Ok, so this issue is about two different things.

InteractiveBrowserCredential shows OK on the browser once it succeeds.
DeviceCodeCredential should use the logger on success (as other languages are doing).

Both changes are one-liners. I'll be doing a PR soon.

Fixed with https://github.com/Azure/azure-sdk-for-js/pull/11871

Fixed with https://github.com/Azure/azure-sdk-for-js/pull/11871


observable.convertObject should return the object

So I can write
return observable.convertObject( new MyModule() );

instead of
var myModule = new MyModule();
observable.convertObject( myModule );
return myModule;

I can submit a pull request...

@navels Please submit a PR if you have some working code here.  Also be sure to include test cases to cover the change in functionality.

There are currently no tests for the observable plugin (unless I am looking in the wrong place...).  I suppose I could add one.

Would be great, I don't see any around the observable plugin yet so it would be a good start.

@navels any luck with this pr? i see you have another outstanding pr for a different issue?

Just haven't gotten around to it.  Maybe the juice isn't worth the squeeze :)


Changed font to OpenSans

It looks a lot better on windows. What do you think?

Yep!, looks great in my Lumia 830. :+1:

I cannot get the font to work right with Open Sans, way to thick; Looking for a good font that works with IE and has a weight of around 200;


source.scrapers.crawl may need to be dynamically populated

**Description.
Some scrapers contain large lists of counties that they use to create URLs to call, e.g
const counties = [ 'a', 'b', 'c', ... lots of entries ...];
counties.forEach(c => fetch.page('someurl?' + c));

Sample: US/VA scraper.
Having the user enter all of these things would be a hassle.  Perhaps the scraper can do something like counties.forEach(c => crawl.push(makeCrawlerFor(c)).

I don't think this will be an issue, but it's an interesting test case to consider, and perhaps should be migrated early.

Not necessary to test, this will work fine.


update readme

I am going through the readme and found some necessary punctuation additions, typo fixes and refactors. will raise the PR referencing this issue.
This is part of hacktoberfest!

Closed via #106, open if more problems are found for this issue.


vector class is undefined and an error occurs

M5Display.h
append
#include <vector>

This fixes below error I saw while executing trivial hello world sketch.  This is too easy fix, so am I missing something???
#include <M5Core2.h>

void setup() {
  M5.begin();
  M5.Lcd.setTextSize(3);
  M5.Lcd.print("Hello World");
}

void loop() {
}

error log:
~/Documents/Arduino/libraries/M5Core2/src/M5Display.h:117:12: error: 'vector' in namespace 'std' does not name a template type
       std::vector<DisplayState> _displayStateStack;

I had the same problem, but in my case the issue was installation of wrong library. I was using M5 core2 but had installed library for M5 stack on Arduino IDE

Thanks, fixed


Comments should be larger than one line on main.php

https://github.com/joevegan/ticketdesk/pull/13
This might need some further work. Notably the recent tickets section. But this issue should be good.

Code looks good will test tomorrow and close it out once tested

So one thing to consider that may help. IMO tickets should also have a subject line separate from comments. Recent tickets would then be the subject line and not the comments.

Hmm, I feel like the category/subcategory fields should define "subject". That way reporting would be on standardized categories vs free form subject.
Perhaps we add those to the recent tickets?
Maybe ticket type too. (Request,issue, outage)
Thoughts?

On May 16, 2016, at 10:24 AM, Kamori<EMAIL_ADDRESS>wrote:
So one thing to consider that may help. IMO tickets should also have a subject line separate from comments. Recent tickets would then be the subject line and not the comments.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

So to me a subject gives a little bit different definition to the issue sort of a short description to the issue.
For example depending on the customer of your application say they were creating tickets for customers of their minecraft server hosting company. You could get 2 very different tickets
1.) He can't connect to his servers ip and port
2.) Refund request
In this scenario and with the idea of data reports in the future. You could layout the application  to say
Subject: Refund Request planetcraftservers
Category: Account Management
Sub Category: Refund Request
With this people can quickly tell this is for the account planetcraftservers and get the details about it being a Refund Request in the subject + in the sub-category for later reports.
However, this is just a thought.
Moving on, yes I agree that the categories should be shown instead of the comments simply due to the type of data that may be present could be variable in length/style. If you wanted to keep the comments, I would recommend moving it to its own page or something so its not a small side panel.
Down for any more ideas you have.

Lets add subject, and update the recent tickets to show subject and category/subcategory.
The comments is more the original "request details". If you click the status button next to a recent ticket it will take you to a view ticket screen where you can see and edit the ticket adding more notes as needed so they can always go there to view the full request details and/or work on the ticket.
I planned to add another panel that would auto populate once the client id is typed in with the tickets for that client--what do you think about that? Maybe below those two panels?

On May 16, 2016, at 1:20 PM, Kamori<EMAIL_ADDRESS>wrote:
So to me a subject gives a little bit different definition to the issue sort of a short description to the issue.
For example depending on the customer of your application say they were creating tickets for customers of their minecraft server hosting company. You could get 2 very different tickets
1.) He can't connect to his servers ip and port
2.) Refund request
In this scenario and with the idea of data reports in the future. You could layout the application to say
Subject: Refund Request planetcraftservers
Category: Account Management
Sub Category: Refund Request
With this people can quickly tell this is for the account planetcraftservers and get the details about it being a Refund Request in the subject + in the sub-category for later reports.
However, this is just a thought.
Moving on, yes I agree that the categories should be shown instead of the comments simply due to the type of data that may be present could be variable in length/style. If you wanted to keep the comments, I would recommend moving it to its own page or something so its not a small side panel.
Down for any more ideas you have.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

hey i put the latest build here: http://devmonkeyz.com/ticketdev/ it has
your pr's in it. As well the subject line addition, i updated the db schema
for it too.
Good working with you! :D
On Mon, May 16, 2016 at 1:53 PM<EMAIL_ADDRESS>wrote:

Lets add subject, and update the recent tickets to show subject and
category/subcategory.
The comments is more the original "request details". If you click the
status button next to a recent ticket it will take you to a view ticket
screen where you can see and edit the ticket adding more notes as needed so
they can always go there to view the full request details and/or work on
the ticket.
I planned to add another panel that would auto populate once the client id
is typed in with the tickets for that client--what do you think about that?
Maybe below those two panels?
On May 16, 2016, at 1:20 PM, Kamori<EMAIL_ADDRESS>wrote:
So to me a subject gives a little bit different definition to the issue
sort of a short description to the issue.
For example depending on the customer of your application say they were
creating tickets for customers of their minecraft server hosting company.
You could get 2 very different tickets
1.) He can't connect to his servers ip and port
2.) Refund request
In this scenario and with the idea of data reports in the future. You
could layout the application to say
Subject: Refund Request planetcraftservers
Category: Account Management
Sub Category: Refund Request
With this people can quickly tell this is for the account
planetcraftservers and get the details about it being a Refund Request in
the subject + in the sub-category for later reports.
However, this is just a thought.
Moving on, yes I agree that the categories should be shown instead of the
comments simply due to the type of data that may be present could be
variable in length/style. If you wanted to keep the comments, I would
recommend moving it to its own page or something so its not a small side
panel.
Down for any more ideas you have.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/joevegan/ticketdesk/issues/3#issuecomment-219519986

This issue corrected with Kamori's pr #13


WIP: do not MASQ host2

Signed-off-by: Martynas Pumputis<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin.
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed -->

This change is

test-me-please

Coverage increased (+0.04%) to 44.085% when pulling d4546bad27e29e211fc33beb4dd2c191a678585e on pr/brb/test-no-masq into 82f6aba78b6f4d00e0083cac7ace4fea6f1849f2 on master.

test-me-please

test-me-please

test-me-please

Preceded by #9079.


[3.x] Improve RayCast2D debug shape

Based on #46675, with later improvements.

Highlights the debug shape when the ray is colliding
Correct the length of the ray shape and adjust arrow head size

Thanks!


Maintain string litteral types when used as index signature type

TypeScript Version:  2.4.0
Code
function makeObj(foo: 'FOO') {
    return { [foo]: 1 };
}

makeObj('FOO') // return type: `{ [x: string]: number}`

Expected behavior:
Index signatures should have narrower types. Return type should be { [x: 'FOO']: number}
Actual behavior:
return type is: { [x: string]: number}
Use case
When making forms in React components I often use a method that returns an onChange method to be used on input onChange properties:

class Component<S> {
    state: S;
    setState(partialState: Partial<S>) { }
    onFiledChanged = (filedName: keyof S) => (value) => {
        this.setState({ [filedName]: value });
    }
    render() {
        return <form>
                <input onChange={ this.onFiledChanged('foo') } />
                <input onChange={ this.onFiledChanged('bar') } />
            </form>
    }
}

TypeScript fails to understand filedName is type 'foo' | 'bar'

looks like a duplicate of https://github.com/Microsoft/TypeScript/issues/16687

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Updating schema to sslkeys

Closes #2252

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1545/
Test PASSed.

@smalenfant The issue this closes - #2252 - is already closed. Is this PR still relevant?


Miniconda3 persist folder gets cleaned after updating

No sure if this is a bug or it's working as intended. Currently all files under persist/miniconda3/envs will be lost after an update. I presume they should not be deleted as they are under persist folder?

Files under persist should never be deleted, unless you run uninstall with purge flag like scoop uninstall xxx -p.
I tried to reproduce the problem by installing version 4.7.10, create a new env, and update to version 4.7.12. However the problem did not happen.
I can only guess that in old versions of Miniconda, the uninstaller removes the entire env folder by default, but newer versions of Miniconda doesn't do that.
We can solve these kind of problems by handling the env folder in uninstaller.script, but I think there is no need to do that, since the problem does not happen in the latest version of Miniconda.

Files under persist should never be deleted, unless you run uninstall with purge flag like scoop uninstall xxx -p.

great idea.

Files under persist should never be deleted, unless you run uninstall with purge flag like scoop uninstall xxx -p.

great idea.

This is not "an idea". It is a feature of Scoop. You can try scoop uninstall (something) -p to see the effect.


Скласти таймлайн

Скласти список всіх (важливих) життєвих подій, визначних дат, досягнень, періодів, що дали приємний і неприємний досвід, тощо. В цьому конкретному випадку довгих списків не буває, натомість бувають погано згорнуті, тому пакуємо сюди все підряд, а про "лишні" події можна буде промовчати пізніше. Список повинен бути систематизованим і посортованим в такий спосіб, щоб читач швидко зрозумів "паттерн" і міг легко орієнтуватись в просторі і часі цього резюме.

1961
On February 2, Barack Obama, Sr., a Kenyan, and Stanley Ann Dunham, an American, married in Maui. On August 4, Barack Jr. was born in Honolulu, Hawaii.
1962
Barack Obama Sr. received a graduate scholarship to Harvard, so he moved to the east coast to study economics.
1964
Ann Dunham divorced Barack Obama Sr. During this time, Barack Obama Jr. was generally cared for by his maternal grandparents while his mother attended college.
1965
Barack’s mother married an Indonesian man, Lolo Soetoro.
1966
Lolo Soetoro returned to Jakarta, Indonesia.
1967
Ann and Barack Jr. moved to Lolo Soetoro's hometown of Jakarta, Indonesia.
1971
At the age of 10, Barack Obama, Jr. was sent back to Hawaii to live with his maternal grandparents.
1979
Barack Jr. started attending college at Occidental College in Los Angeles.
1981
Obama made his first public speech, stating that Occidental College should support the abolishment of apartheid in South Africa. Thereafter, he transferred to Columbia University in order to major in political science.
1982
Barack Obama Sr. died in a car accident in Kenya.
1983
Barack graduated from Columbia University with a Bachelor of Arts degree, magna cum laude.
1988
Barack Obama began to study at Harvard Law School. After his first year, he worked as an intern for a top law firm located in Chicago. It was here that he met the future Mrs. Michelle Obama.
Advertisement
1990
On February 5, Obama Jr. became the first black president of the Harvard Law Review.
1991
Obama graduated from Harvard with his law degree and began work on his book Dreams of My Father.
1992
In this year, he became the director of Illinois Project Vote. Obama also began to teach constitutional law at the University of Chicago Law School. In October, Obama married Michelle Robinson.
1993
Obama became very interested in race and poverty issues. While still teaching, he started working with the Developing Communities Project as a community organizer. The same year, he joined a law firm that specialized in civil rights legislation.
1995
Obama’s mother returned to Hawaii and was diagnosed with cancer. On November 7, at the age of 52, Ann Dunham died from ovarian and uterine cancer. Dreams of My Father was published that same year.
1996
Obama was elected to the Illinois Senate.
1998
On July 4, the Obamas’ first daughter, Malia Ann, was born.
1999
Obama Jr. ran for Congress. However, he lost to the Republican Bobby Rush.
2001
On June 10, Obama's second daughter Natasha (Sasha) was born.
2002
Obama started to campaign for the United States Senate.
2003
In this year, Obama became the chairman of the Illinois Senate Health and Human Services Committee.
2004
On July 7, Obama was asked to give a keynote speech at the Democratic National Convention. On November 2, he was elected to the United States Senate.
2005
On January 5, Obama was sworn in as a U.S. Senator.
Advertisement
2007
Obama announced his candidacy to become President of the United States.
2008
On June 3, Obama beat Hillary Clinton in the primary race for the presidency, receiving the presidential nomination of the Democratic Party primary election. On November 5, Obama defeated Republican John McCain in the general election and would be the next president.
2009
On January 20, Obama was sworn in as the 44th President of the United States. On February 17, he signed the economic stimulus bill. On September 9, he gave his health care speech. On October 9, he was named the winner of the Nobel Peace Prize.
2010
On January 27, Obama gave his first State of the Union address.
2011
On April 4, he announced his re-election campaign.
2012
On April 2, Obama learned he had the 2,778 convention delegates needed to win the Democratic nomination. On November 6, he won 332 electoral votes against Mitt Romney, with only 270 needed to be re-elected for a second term as president. In December, the devastating mass shooting took place in Newtown, Connecticut and Obama vowed to fight for tougher gun control laws.
2013
During his re-election campaign, Obama promised to reduce the federal deficit by raising the taxes on the wealthy. In 2013, he received bipartisan agreement on tax increases. In October, the government faced its first shutdown in 20 years after Congress didn’t support his legislation to lift the glass ceiling.
2014
In this year, Republicans gained control over both houses of Congress, creating a less than ideal situation for Obama’s final two years in office.
Advertisement
2015
In 2015, Obama enjoyed two major victories in the Supreme Court. First, the Affordable Care Act’s tax subsidies were upheld. Second, the Supreme Court ruled five to four that same-sex marriage is a right guaranteed by the U.S. Constitution.
2016
In March of this year, Obama became the first president to visit Cuba since 1928. In July, he delivered a speech at the Democratic National Convention, announcing his support of Hillary Clinton as presidential hopeful. In July, Obama met with President-elect Trump in the Oval Office to handle the transition of power.
2017
January 20th marked Obama’s last day in office. His first order of business was to take a respite in Palm Springs, California. The Obama family planned to remain in Washington, D.C., at least until their youngest daughter graduates high school.


PRAGMA version reports a wrong version in NPM version

What happens?
PRAGMA version; reports version v0.0.1-dev0 for duckdb-wasm NPM version 1.24.0. Actually it does it since version 1.20 at least.
To Reproduce
Go to shell.duckdb.org look what database says and then you can try command: PRAGMA version; and you will see the same thing.
Browser/Environment:
It doesn't matter
Device:
It doesn't matter
DuckDB-Wasm Version:
1.20 ->
DuckDB-Wasm Deployment:
shell.duckdb.org
Full Name:
Kimmo Linna
Affiliation:
F

Thank you for raising this, I had mis-categorized this bug, but it's actually there already in the duckdb-wasm/packages/duckdb-wasm component. Simplest repro, basically from any kind of setup (node / browser / whatever) is:
select version()

that should mimic the native duckdb CLI in returning the underlying DuckDB version.


Modin on Ray Cluster only running on head node for I/O Operation on TB Scale Parquet data from S3

Hi,
I have an S3 bucket that contains 500 files parquet files (each parquet file is around 120-140mb in size), however when I attempt to load this data into a Modin DataFrame, utilizing ray, only three ray deploy_functions are created. This is resulting in the operation not being properly distributed across a Ray Cluster and only 3 CPUs being used at any one time.
Additionally, because this is only triggering three ray functions, and not utilizing the cluster properly - I am having to scale up the head node to ensure the RAM is > 1TB to prevent out of memory errors as the commands are only being executed on said node. This isn't scaleable as the cost associated with EC2 instances containing > 1TB of ran is significantly higher than having a distributed cluster of multiple smaller EC2 instances. The EC2 instance I am having to use r6i.32xlarge has 128 cpus available, yet I am only able to use three of them - resulting in the data load taking a significant amount of time (upwards of 9 hours for a TB) on a 50000 Megabit connection.
I have attempted to run the following code and both only ever spawn 3 Ray functions:
pd.concat([pd.read_parquet(key) for key in listed_files]) where listed files is the file locations in s3
pd.read_parquet('file_location')
I've attached screenshots of the dashboard as well as the query status
<img width="457" alt="Screen Shot 2023-02-14 at 11 37 58 am" src="https://user-images.githubusercontent.com/55753338/218608795-3dbd0435-6aa8-4be3-8c18-00

0662e1751e.png">

It seems to be a specific issue with these files vs others. Modin works as expected when retrieving datasets in Parquet from other S3 Folders, the only key differences between the data being retrieved in the above session where only 3 CPUs are triggered vs other sessions is as described previously:
Data Size - Each file is 120mb compressed snappy Parquet
Paritioning within S3 - due to the nature of the files, there is no "partitioning" within the S3 bucket for the parquet files as there is no reasonable partition key for the dataset.
Not sure if one of these could be whats driving this problem.


Split regressor estimation and denoising steps

Closes #10.
Changes proposed in this pull request

Collect and run rapidtide on boldref-space BOLD data instead of MNI152NLin6Asym-space data.

This should reduce memory use almost all of the time. Plus it makes it easier to warp the lag map to different output spaces.

Add option to average lag maps across runs before denoising, which (I think) should reduce noise in the lag maps.
Run rapidtide and denoising separately. I think I need to wait until RetroGLM accepts individual files instead of a full rapidtide directory before I can make this work.
Add --spcalculation and --noglm flags to Rapidtide interface.
Add individual output files as outputs to the Rapidtide interface.

Run rapidtide and denoising separately. I think I need to wait until RetroGLM accepts individual files instead of a full rapidtide directory before I can make this work.

I'm ok with making this change, but why is this needed?  If you run rapidtide, even at --outputlevel min, all the files you need to run retroglm will be present in whatever output directory you specified with the proper names, so you can just run retroglm and point it at that work directory.  The only reason I can think of to require this is if you run rapidtide, move and rename the output files, and then run retroglm.

That's a good point. I can reconstruct a rapidtide-esque directory to feed into RetroGLM.
In that case I think I just need the ability to generate the voxel-wise LFO regressor file in each target space. Can you remind me how to do that without doing any denoising?

I've added a new script - retrolagtcs to rapidtide v2.9.8 (currently building containers, but passes all its tests).
You give it an fmrifile (for dimensions), the corrmask, processed mask, lag map, and lagtcgenerator, and it will generate voxel specific shifted LFO regressor file (and derivative files, if requested).  It currently just reads in the fmrifile, but I just realized I can get all the necessary information just from the header, so I'll do that when I have a moment.
fmrifile, cormask, processed mask, and lag map have to have matching spatial dimensions.  The lagtcgenerator can be used unchanged from the initial rapidtide run.

Thanks! Just curious, why does it need both the corrmask and the processed mask? Wouldn't just the processed mask be enough?

To be honest, I just copied that out of retroglm.  It probably just needs corrmask (which you could give it twice, I guess).  Processed mask is all the voxels you can attempt to fit ("valid voxels"), corrmask is all the places the fit succeeded.  In practice they are usually almost identical. I guess the question is, in a nominally valid voxel, where for whatever reason rapidtide could not decide on a delay, is it better to do nothing, or regress out the LFO at zero time delay?  🤷🏻♂ - I don't feel strongly one way or the other.

...and now it's <IP_ADDRESS>, because I made a stupid mistake and you can't redeploy to pypi with the same release tag...

Thanks so much!
So my plan is basically going to be:

Run rapidtide on boldref-space derivatives to get the mask, lag map, lagtcgenerator, and a few other things.
Average lag maps across runs if users want.
Warp the lag map and mask to desired output spaces.
Run retrolagtcs to get the voxel-wise regressor in each output space.
If users want denoised outputs, run retroglm to actually denoise the BOLD data in those output spaces as well.

That all looks good, although one thing I don't really know is how much variation in delay maps between runs within a subject is 1) rapidtide fit instability (not real) 2) the effects of other noise processes confounding the fit (not real) and 3) actual physiological variation between runs due to changing circulatory patterns (real).  I think number 3 is pretty small - the fact that delay generally seems pretty stable over 5-15 minutes would argue that the delay pattern is pretty fixed, at least during resting state, but I know that Kevin Whittingstall showed that functional activation could cause acute arterial dilation upstream of a strongly activated region, which I think would probably shift the timing between arterial territories.  So yes, make averaging between runs an option, but maybe not the default behavior.

Oh that's really good to know. I'll make sure it's not the default then. Now I want to dig into that, maybe with HCP-YA.

We have a paper currently under review looking at repeatability in the HCP-YA data.  I can send you a copy if you'd like.

That would be great, thanks! Please send to salot at pennmedicine dot upenn dot edu.

Are you waiting for me to do something on this pull request?

No, I just haven't had time to work on fMRIPost-rapidtide for the past couple of weeks. Hopefully I'll be able to carve out some time for it in the next few.

Ok, no problem.  I just didn't want to be holding things up without knowing it.


fixed typo in std/range/package.d

newValues3 -> newValues4

Those commits are just a result of updating my fork of phobos. Currently, I'm pulling from master to my repo, then pushing back to my repo(using SourceTree/OSX). Not sure how to do this another way.

Ok... So I can't figure out how to fix this. I'll have to re-fork phobos and try rebase next time to eliminate the extra commits.


Python3 package

Making the tool better compatible with python3 and adding a setup file which makes it installable

Hey There,
I tried installing it and i get the following error message when i run pip install . command on the source directory.

Could you please check what the issue might be?

Thanks for checking :) The issue seems related to the setup file and the python version you are using... I forgot to mention something important:

Requires a virtualenv to run since the dependencies are now fixed in the requirements.txt file
I tested in on python 3.8. Cannot guarantee it works with previous versions but I can try to make it compatible if you can tell me which ones you want to target.

Thanks Ic4nt,
I am running python 3.8. i tried running in venv and still ended up with same error.

Thanks

I apologize for the delay. I fixed the setup.py file, the issue was related to the pip version. On my Ubuntu 20.04 python3 doesn't install wheel by default when creating the virtual environment so I had to do it manually. This is a problem with my machine or with ubuntu. Adding wheel on the top of the requirements.txt doesn't solve.
lollo@lapdog:~$ python -m venv /tmp/test
lollo@lapdog:~$ source /tmp/test/bin/activate
(test) lollo@lapdog:~$ pip install wheel
Collecting wheel
  Using cached wheel-0.37.1-py2.py3-none-any.whl (35 kB)
Installing collected packages: wheel
Successfully installed wheel-0.37.1
(test) lollo@lapdog:~$ cd dev/github.com/ioc-parser/
(test) lollo@lapdog:~/dev/github.com/ioc-parser$ pip install .
Processing /home/lollo/dev/github.com/ioc-parser
Collecting PyPDF2==1.26.0
  Using cached PyPDF2-1.26.0.tar.gz (77 kB)
Collecting beautifulsoup4==4.9.3
  Using cached beautifulsoup4-4.9.3-py3-none-any.whl (115 kB)
Collecting pdfminer==20191125
  Using cached pdfminer-20191125.tar.gz (4.2 MB)
Collecting requests==2.25.1
  Using cached requests-2.25.1-py2.py3-none-any.whl (61 kB)
Requirement already satisfied: wheel in /tmp/test/lib/python3.8/site-packages (from iocparser3==1.0.0) (0.37.1)
Collecting soupsieve>1.2; python_version >= "3.0"
  Using cached soupsieve-2.3.2.post1-py3-none-any.whl (37 kB)
Collecting pycryptodome
  Using cached pycryptodome-3.14.1-cp35-abi3-manylinux2010_x86_64.whl (2.0 MB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.9-py2.py3-none-any.whl (138 kB)
Collecting chardet<5,>=3.0.2
  Using cached chardet-4.0.0-py2.py3-none-any.whl (178 kB)
Collecting idna<3,>=2.5
  Using cached idna-2.10-py2.py3-none-any.whl (58 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2021.10.8-py2.py3-none-any.whl (149 kB)
Building wheels for collected packages: iocparser3, PyPDF2, pdfminer
  Building wheel for iocparser3 (setup.py) ... done
  Created wheel for iocparser3: filename=iocparser3-1.0.0-py3-none-any.whl size=13881 sha256=18471d0d117b1d8ac362f846002abc5091c638f503b21e981a22f272bc19f40c
  Stored in directory: /home/lollo/.cache/pip/wheels/bf/0f/60/6abfede98d6355625e4133203435461abcea28b0b63b501bcc
  Building wheel for PyPDF2 (setup.py) ... done
  Created wheel for PyPDF2: filename=PyPDF2-1.26.0-py3-none-any.whl size=61085 sha256=b3b41cce65efc9b59c4dc84ff671496e004e63ec873012ded9f4a89c5be8a6d2
  Stored in directory: /home/lollo/.cache/pip/wheels/b1/1a/8f/a4c34be976825a2f7948d0fa40907598d69834f8ab5889de11
  Building wheel for pdfminer (setup.py) ... done
  Created wheel for pdfminer: filename=pdfminer-20191125-py3-none-any.whl size=6140086 sha256=dcf6f6e835a7dd0b3427f6f13796c2000c9a95d10675404feef31899dfb4276e
  Stored in directory: /home/lollo/.cache/pip/wheels/1c/28/7d/f390b82bb0307deb63ff27a1474fd308ec68ee028cb9ab6283
Successfully built iocparser3 PyPDF2 pdfminer
Installing collected packages: PyPDF2, soupsieve, beautifulsoup4, pycryptodome, pdfminer, urllib3, chardet, idna, certifi, requests, iocparser3
Successfully installed PyPDF2-1.26.0 beautifulsoup4-4.9.3 certifi-2021.10.8 chardet-4.0.0 idna-2.10 iocparser3-1.0.0 pdfminer-20191125 pycryptodome-3.14.1 requests-2.25.1 soupsieve-2.3.2.post1 urllib3-1.26.9


After finished workout, not getting clock displayed

after the workout is finished I am not getting the clock, but workout paused screen (when locked). when unlocked it's fine

Did you restart the watch after installing APK? What is the firmware version installed on watch? This may happen if you have the wrong version installed for your firmware too...

This was a bug in GreatFit versions 3.1 and lower. Do you have the latest version installed?

I think it was a glitch. Working fine now.


Added an option to Settings.

Added an option to Settings.
The option is a Check Box called "Click on select"
The choice only applies during BOM creation, so if the setting is changed a new BOM needs to be created.
What it does is change the way you interact with the page. If the setting is selected you have to click items to select instead of moving the mouse over.
The reason for it: "I was sick of knocking my mouse and seeing the wrong items highlighted"
Hopefully I have done things correctly. do check it, I have placed comments where I changed things in the code.
I am OK with html and JS, but don't now how this becomes an addon.

This is impossible to review because of all the formatting changes. Undo irrelevant changes and also remove the .vs folder from your git (you can add it to gitignore).

It was basically a change at "ibom.js" from line 681.
		var handler = createRowHighlightHandler(tr.id, references, netname);
		/*
			Tim (Palingenesis)
			For choice of using select instead of mouse over
		*/
		//tr.onmousemove = handler;   /*  Tim (Palingenesis)	*/
		if (settings.selectOnClick) {
			tr.onmousedown = handler;
		} else {
			tr.onmousemove = handler;
		}
		/*	End	*/

In "ibom.html" I added a new Check Box "checkbox_SelectOnClick" after dragCheckbox from 119
	<label class="menu-label">
		<input id="dragCheckbox" type="checkbox" checked onchange="setRedrawOnDrag(this.checked)">
		Continuous redraw on drag
	</label>
	<label class="menu-label">
		<input id="checkbox_SelectOnClick" type="checkbox" checked onchange="setSelectOnClick(this.checked)">
		Select On Click<br>(Effects On Create Only)
	</label>

I changed/added new setting in the relative files to do with settings to create a new setting called "settings.selectOnClick"
in "util.js" and  "render.js"
Can you see what I was doing?
I was adding a choice to select items instead of just moving the mouse over.
Having to keep the mouse over items while you work on the parts is I little silly.
Should I try push again?


Help needed to prepare dataset.

downloaded from https://www.votchallenge.net/vot2019/dataset.html
VOT2019 dataset preview
but did not have list.txt
FileNotFoundError: [Errno 2] No such file or directory: 'LightTrack/lib/utils/../../dataset/VOT2019/list.txt'

Hi, sorry for the late reply. Please download the VOT2019 dataset using the official VOT Toolkit. Thanks.

list.txt is the name of each folder in VOT2019, similar to the following:
agility
ants1
ball2
ball3
basketball
birds1
bolt1
……

You can write a python script directly to generate it.


When should we switch to Java 8 as min req?

This issue is to track the question of when to base the Azure Management Libraries for Java on Java 8, collecting the pros and cons and eventually resulting in a decision.
Pros:

Lambda support - which we'd take advantage of quite heavily in the SDK, as we already have the awkward Java-7 style equivalent of that in many places, and would also be able to remove a bunch of code
Java 7 "end-of-life'd" (no public updates) as of April 2015. It's been a while...

Cons:

about 28% of Java users are still on Java 7 still, per RebellLabs.

Pros: streams!

Also, CompletableFuture!

We're making the switch as part of our conversion to new code generators and autorest v3. This will bring the management libraries in sync with the design guidelines. https://azure.github.io/azure-sdk/java_introduction.html. I'm going to close this issue as its tracked as part of the larger conversion.


Create/update user oidc/entity/policy during cert creation

Invoke the oidc/entity primitives during cert creation to give user vault oidc auth access, as well as cert crud permissions.

#66 merged; closing


grey panda antenna mount 28 degrees by @mhevery

Looks like this has more than the Grey Panda Antenna mount?

Oh, I forgot I had a PR open for this and was doing the giraffe work in the same repo.
Want me to put the giraffe in a separate PR or just update the PR description?

Separate. I'll upstream the mount right away

Thanks for accepting the mount PR.
I've separated the Chrysler/Jeep/Dodge giraffe into a separate PR, which is ready for review: https://github.com/commaai/neo/pull/42


feat: use flavor for host, scheme and logo

Codecov Report
Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (c6fa84d) 56.58% compared to head (cf1099a) 56.53%.

Files
Patch %
Lines

lib/tools/functions.dart
45.45%
6 Missing :warning:

...cinema/repositories/session_poster_repository.dart
0.00%
1 Missing :warning:

...b/vote/repositories/contender_logo_repository.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #195      +/-   ##
==========================================
- Coverage   56.58%   56.53%   -0.05%
==========================================
  Files         160      160
  Lines        3492     3502      +10
==========================================
+ Hits         1976     1980       +4
- Misses       1516     1522       +6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

On Android, redirect scheme does not work after login. This should be updated to match the flavor:
https://github.com/aeecleclair/Titan/blob/cc115630a26e6a84ed338b773891621ed3cabb21/android/app/build.gradle#L61-L63

On Android, redirect scheme does not work after login. This should be updated to match the flavor:
https://github.com/aeecleclair/Titan/blob/cc115630a26e6a84ed338b773891621ed3cabb21/android/app/build.gradle#L61-L63

It should now be fixed

This would need to be tested on iOS before merging @armanddidierjean


Interpolation

-toTau now works as interpolation if targetType is :\tau. The same for toMatfreq
-add getValue as interpolation

Codecov Report

Merging #12 (545bbff) into master (d267352) will decrease coverage by 8.30%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
- Coverage   61.45%   53.15%   -8.31%
==========================================
  Files           1        1
  Lines          96      111      +15
==========================================
  Hits           59       59
- Misses         37       52      +15

Impacted Files
Coverage Δ

src/green/Green.jl
53.15% <0.00%> (-8.31%)
:arrow_down:

Continue to review full report at Codecov.

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


Getting unsigned ax attributes from an openid provider causes a "NoneType object has no attribute get" error

When dealing with openid ax atrributes, the consumer.py has a SuccessResponse.getSignedNS that returns the signed attributes or None is any of the signed attributes are not signed (link).
IMO this method should raise an exception or just return the signed ones. Returning None means that no attributes were signed at all and also fails when dealing with the result of this method, as instead of an empty array, you get None.
When an OpenID provider redirects to an url with present but unsigned ax attributes, the fromSuccessResponse will fail with a "NoneType object has no attribute get" as this line will try to get the current mode on a None object

This repo is being archived. Closing issue.


ObjectCleanerThread leaking

Expected behavior
After using Netty, all Netty threads should terminate. This is the behavior of Netty 4.1.19. But ObjectCleanerThread leaks since 4.1.20.
Actual behavior
ObjectCleanerThread only terminates when JVM terminates.
Steps to reproduce

Start to use Netty (NettyRuntime.setAvailableProcessors, EventLoopGroup etc.)
Use Netty
Stop using Netty by eventLoopGroup.shutdownGracefully()
ObjectCleanerThread still in thread list

Minimal yet complete reproducer code (or URL to code)
  public void shutdown() {
        eventLoopGroup.shutdownGracefully();
        Set<Thread> allThreads = Thread.getAllStackTraces().keySet();
        for (Thread thread : allThreads) {
            if (thread.getName().equals(ObjectCleaner.class.getSimpleName() + "Thread")) {
                logger.info("found Netty object cleaner thread after shutdown");
            }
        }
    }

Netty version
4.1.20, 4.1.21, 4.1.22
JVM version (e.g. java -version)
openjdk version "<IP_ADDRESS>"
OpenJDK Runtime Environment (Zulu build <IP_ADDRESS>+11)
OpenJDK 64-Bit Server VM (Zulu build <IP_ADDRESS>+11, mixed mode)
OS version (e.g. uname -a)
Darwin Jorg-Prantes-MacBook-Pro.local 16.7.0 Darwin Kernel Version 16.7.0: Thu Jan 11 22:59:40 PST 2018; root:xnu-3789.73.8~1/RELEASE_X86_64 x86_64

@jprante thats most likely because you used the PooledByteBufAllocator.DEFAULT and so a reference is still be hold to it. Because of this the cleaner thread is still alive as there is still allocated memory that was not collected.

@jprante that said in 4.1.21 the thread is daemon, so it shouldn't stop your JVM from exiting https://github.com/netty/netty/pull/7627.

I have several integration tests on a single JVM where I need to restart Netty completey, including PooledByteBufAllocator.DEFAULT usage. A JVM exit is not possible.
How can I kill the thread?

Run your tests with the leak detector level set to paranoid to try and find the leaking bytebuf(s).

Gotcha. I think this issue can be closed.
Probably a small note in the documentation can be helpful to look for pooled byte buffer leaks when the cleaner thread keeps hanging around.

Simplified shutdown
There's no more releaseExternalResources(). You can close all open channels immediately and make  all I/O threads stop themselves by calling EventLoopGroup.shutdownGracefully().

https://github.com/netty/netty/wiki/New-and-noteworthy-in-4.0

Sorry, I have to come back.
I can not find my ByteBufs are leaking, and I find no problems with PooledByteBufAllocator.DEFAULT.
I am running with leak detector on "paranoid" and it does not report anything.
But the "ObjectCleanerThread" is still running and does not terminate.
The test consists of a simple Netty-based HTTP client. Starting the client, and shutting it down completely after issuing a simple HTTP GET request, should be successful - with the ObjectCleanerThread exiting.
The test can be run with
./gradlew test -Dtest.single=SimpleHttp1Test
I have prepared a gist with the test output will log level ALL.
https://gist.github.com/jprante/216348c3a9839ab96bad5129baefe631
Please note the last line, it prints "ObjectCleanerThread," after the test has been run and the Netty client has finished.  In the gist, I have also included the complete test source code (285 lines) with only Netty as dependency.
My project is at
https://github.com/jprante/netty-http-client
Maybe you want to have a look. Thank you.

@jprante the problem is once you used PooledByteBufAllocator.DEFAULT the thread can only exists once the JVM terminates as there is a static instance of PooledByteBufAllocator.

Closing


Test full node catching up when far behind with blocks having lots of Txs

That means if I sync and I am far behind, I would be able to sync fully no matter the height difference. I think we should test how the sync of a chain (with a lot of txs) performs. Can be a follow-up. VA

Originally posted by @nashqueue in https://github.com/rollkit/rollkit/pull/1204#discussion_r1339321177

Hi @Manav-Aggarwal, I see in the document of BlockStoreRetrieveLoop, it says It keeps track of the last retrieved block's height and every time the current block store's height is greater than the last retrieved block's height, it retrieves all blocks from the block store that are between these two heights but in the code we are set lastBlockStoreHeight = 0, so It will always retrieve from 0 to current block store's height, should we add a param in like startHeightRetrive in Manager.BlockStoreRetrieveLoop func ?

test: test P2P catchup
rollkit/rollkit


WebAudioPluginCompositeNode shouldn't create any context

WebAudioPluginCompositeNode constructor has:
this.context = context ? context : new AudioContext;
This line has no sense, it should work like any built-in audio node, throw an error if no context passed. Any built-in node has ability to create contexts.

Fixed.


[1.3] chip-tool buil fail

Reproduction steps
source scripts/activate.sh
./scripts/examples/gn_build_example.sh examples/chip-tool out/host/
This error will occur if you build any project.
ERROR at //.gn:15:1: Unable to load "/home/xh/silabs/matter/examples/chip-tool/build_overrides/build.gni".
import("//build_overrides/build.gni")

Bug prevalence
any time
GitHub hash of the SDK that was being used
no
Platform
efr32
Platform Version(s)
No response
Type
Test Improvement
Anything else?
log.txt

Well, does the file /home/xh/silabs/matter/examples/chip-tool/build_overrides/build.gni exist?
In general, examples/chip-tool/build_overrides/build.gni does exist in the Matter repository (see https://github.com/project-chip/connectedhomeip/blob/22b9c96b691d3bb5a5d3db510db353b3b2bc2f74/examples/chip-tool/build_overrides and https://github.com/project-chip/connectedhomeip/blob/22b9c96b691d3bb5a5d3db510db353b3b2bc2f74/examples/build_overrides/build.gni) , so if it doesn't exist for you the first step is figuring out why not.

Well, does the file /home/xh/silabs/matter/examples/chip-tool/build_overrides/build.gni exist?
In general, examples/chip-tool/build_overrides/build.gni does exist in the Matter repository (see https://github.com/project-chip/connectedhomeip/blob/22b9c96b691d3bb5a5d3db510db353b3b2bc2f74/examples/chip-tool/build_overrides and https://github.com/project-chip/connectedhomeip/blob/22b9c96b691d3bb5a5d3db510db353b3b2bc2f74/examples/build_overrides/build.gni), so if it doesn't exist for you the first step is figuring out why not.

Thank you for your reply. This question is due to the failure of my first construction, I deleted the content in Examples/Chip-Tool/Build_overrides. After I revoke the operation, I can build it normally.


Fix panic when invoking a function from another module that has a decreases_by, fixes #249

@Chris-Hawblitzel I'm going to land this, but please have a look.


[FLINK-18048] Fix --host option for standalone application cluster

What is the purpose of the change
When we use the following command to start a Flink application cluster, the specified hostname could not take effect. The root cause is HOST_OPTION is not added to options in StandaloneApplicationClusterConfigurationParserFactory. It will be a critical issue when we deploy Flink on container environment. Because users usually want to specify a given hostname.
./bin/standalone-job.sh start --host external-hostname --job-classname org.apache.flink.streaming.examples.join.WindowJoin

For the old StandaloneJobClusterConfigurationParserFactory, it has the same issue.
So this PR will add HOST_OPTION in StandaloneApplicationClusterConfigurationParserFactory#getOptions()
Brief change log

Add HOST_OPTION in StandaloneApplicationClusterConfigurationParserFactory#getOptions()

Verifying this change

New added unit test StandaloneJobClusterConfigurationParserFactoryTest#testHostOption()
Manually test a Flink application cluster with given hostname via --host

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

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

cc @tillrohrmann since this issue is introduced via commit 9c4c138a2db7ab84733482d9ba9508b30056843d, would you please have a look?

cc @kl0u Although it is an old bug, it is related to application mode. Could you please have a look? I really hope it could be fixed in release-1.11 and backport to 1.10.

I will review it later today @wangyang0918 and thanks for working on it!


⚠ Fortnite has degraded performance

In 8ce0c3e, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1632 ms

Resolved: Fortnite performance has improved in dcb78b5.


Allow compatibility with DRF 3.7

Hi @pkolios, thank you for contributing. It seems two tasks are failing with:
module 'django.contrib.postgres.fields' has no attribute 'JSONField'

Maybe DRF 3.7 dropped support for older versions of django?

True, it looks like it's django 1.10 and upwards. http://www.django-rest-framework.org/#requirements
How would you like to proceed?

@pkolios let's remove support for django versions older than 1.9, we'll release this update as a new minor version (0.12.x as you already indicated in the README).

Cool, anything else left to do on this PR?

this brings borth drf 3.7 and dj2.x compat


Possible item id class in IntentPickerSheetView

The internal adapter uses the component labels for the id, as pasted below. This could cause a clash as two applications could give an Activity the same label. It would be safer to hash the componentName instead.
    @Override
    public long getItemId(int position) {
        return activityInfos.get(position).label.hashCode();
    }

Fixed, thanks!


Get Image Characters and Get Image Tags

Hello, first off thanks for making this API.
I made a go wrapper for this, but seem to be getting Server Error (500) for the following endpoints so was not able to implement these:

Get an Image's Characters: e.g., https://api.nekosapi.com/v3/images/1449/characters
Get an Image's Tags: e.g., https://api.nekosapi.com/v3/images/1/tags

Are these endpoints no longer meant to be used? Or am i maybe missing something?

Hi! Yeah, those endpoints have been failing for a while. I'm currently working on rewriting the API from scratch since it's been super slow lately so I'm not sure if those endpoints will be fixed for now as I want to focus on writing a better version instead. I think that error is because of internal validation of the response before sending it, so it shouldn't be something too difficult to fix. I'll check it out when I'm back home.


F #5522: Remove manual pages

This commit breaks 5.4.3 build: https://github.com/OpenNebula/packages/issues/2

Yes, the manual pages are currently generated during the build from help returned by each CLI command. Unfortunately, this part is not public yet. For now, you have to take the manual pages from the VCS history or provide stub files.

Took man folder from 5.4.2 release and build succeeded...
Thanks!


Allow keybinding to expand hover without focusing on hover part

Testing #210904
With a hover showing, I'd expect the keybindings to expand/shrink the hover even if I have clicked on a hover element. Maybe in this case we can expand/shrink all the hover elements?

With the PR https://github.com/microsoft/vscode/pull/235848, all the sub hovers will be expanded/contracted if no subhover part is selected.


fix(appeal): additional appeals fix

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 25 (56.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 38.795%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/appeal/service.go
14
25
56.0%

Files with Coverage Reduction
New Missed Lines
%

core/appeal/service.go
1
57.19%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2176

Relevant Lines:
5609

💛  - Coveralls

@rahmatrhd is it duplicated with https://github.com/odpf/guardian/pull/186 ?


Migrated from createClass

Changed from
React.createClass
to
extends Component
in order to remove the deprecation warning when using the lib.

Thank you!


[Documentation] Added the missing links for online documentation

Added the missing links for the online documentation:

S3 Guide
Bloom filter Datamap
Lucene Datamap

Can one of the admins verify this patch?


Implement prompt section for hostname

Feature Description
This feature is implemented once we have reached feature parity with the following zsh section: https://github.com/denysdovhan/spaceship-prompt/blob/master/sections/host.zsh
Expected Behaviour

Reach feature and configuration parity with the implementation in spaceship-prompt
By default, only show the hostname when there is an SSH connection

@Snuggle It sure does! 😄
You're welcome to give it a shot if this feature is valuable to you.

Taken care of in #49.
Thank you for your help @Snuggle! 🎉


怎么才可以挖OL，有教程吗

❓ Questions and Help
Please note that this issue tracker is not a help form and this issue will be closed.
Please contact the development team on Discourse

看不懂，可以用其它方式对话吗，想请教你一下

下载好了，就不知道怎么搞才可以挖矿

@liangping  can you close this with https://github.com/OLSF/libra/commit/58591b93dfc5a9ac67fda09e848de5aeb5186f47

Done


Dedup of code during data prep

Thank you for sharing the details on this work. This is indeed impressive!
It was mentioned that a repo level dedup was performed. Did you guys consider exact and fuzzy dedup at file level also or did some studies to see if repo level dedup performed better?
Also, was your repo level dedup, an exact dedup or a fuzzy dedup?

We employ a fuzzy deduplication approach, utilizing the MinHash algorithm. However, when this method is applied at the file level, it may lead to the removal of certain files in a repository, thereby leaving the repository incomplete.

@guoday Thanks for your response. So if I understand right, you employ fuzzy dedup at repo level. Is that correct?

yes.

same as the llama

Thank you @guoday

Hey, can you share a small portion of data, and the dependency parsing code, deduplication script? that would be of great value! thank you!


Batect doesn't work with Java 10

Returns error:
./batect: line 62: ((: 0 2018-04-17: syntax error in expression (error token is "2018-04-17")
Output of java -version:
java version "10.0.1" 2018-04-17
Java(TM) SE Runtime Environment 18.3 (build 10.0.1+10)

Thanks for the bug report @minnn-minnn, I'll take a look.

This has been fixed with 23e6d63 and will be part of the next release.
Thanks again @minnn-minnn!
(also, GitHub tells me this is your first ever issue on GitHub... congratulations 🎉)


Simplify benchmarks & enable them on all devices.

This PR:

Enables benchmarks on all devices (whitelisted in support.js)
Makes buttons mobile friendly with MDL.
Adds viewport meta to the page so it scales for phones
Moves common benchmark logic to benchmark_util.warmupAndBenchmarkGPU. This function warms up the GPU, times, and returns the result, and it changes functionality based on whether timers are enabled.

This change is

 Great cleanup!

Reviewed 12 of 13 files at r1.
Review status: 12 of 13 files reviewed at latest revision, 1 unresolved discussion.

demos/benchmarks/benchmark.ts, line 19 at r1 (raw file):

5000
i'd lower this to 2500 given that matmul tests are N^3. If we allow the last test to be 4800 for example, the next one will be at least double.

also maybe rename to LAST_RUN_CPU_CUTOFF_MS.

Comments from Reviewable


dapr init 404 Not Found

$ bin > dapr init
⌛  Making the jump to hyperspace...
❌  Downloading binaries and setting up components...
❌  cannot get the latest release version: https://api.github.com/repos/dapr/dashboard/releases - 404 Not Found
$ bin > dapr --version
CLI version: 1.2.0
Runtime version: n/a
$ bin > uname -a
Linux computende4 5.10.27-051027-generic #202103310028 SMP Thu Apr 1 02:16:48 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Also, you should be able to workaround this by manually specifying the version:
dapr init --dashboard-version=0.6.0 --runtime-version=1.2.2

Also, you should be able to workaround this by manually specifying the version:
dapr init --dashboard-version=0.6.0 --runtime-version=1.2.2

thanks, it works.
it might be the firewall issue, could you apply this url to be modification ?
https://api.github.com/repos/dapr/dashboard/releases

I've created a PR to fall back on a secondary source of truth for the release version. It may help this case.


LOCATION and TBLPROPERTIES clauses in CREATE TABLE LIKE statement

Hi! If i create a table using the LIKE statement and add LOCATION and/or TBLPROPERTIES clauses to it, then returns [].
stmt = 'CREATE EXTERNAL TABLE IF NOT EXISTS schema.specific_table LIKE schema.table_template LOCATION "/path/to/table" TBLPROPERTIES ("external.table.purge" = "true")'
DDLParser(stmt).run(output_mode="hql")

@lapter57 , hi! thanks for reporting, will take a look at the evening

@lapter57  hi again! sorry for the delay, fix released with version 0.26.1, test added - https://github.com/xnuinside/simple-ddl-parser/blob/main/tests/test_hql_output_mode.py#L2251 if will be anything new - feel free to open new issue :) and thanks for report one more time!


FIX: Invalid argument warning in backtrace

We log errors in Sentry, and for some reason whenever there’s a database connection error a second error is logged:

E_WARNING: Invalid argument supplied for foreach() {"code":2,"message":"Invalid argument supplied for foreach()","file":"./vendor/silverstripe/framework/src/Dev/Backtrace.php","line":125} []

I’m not sure why the argument list is empty, but it clearly isn’t safe to assume arguments are always present in the backtrace 😄

Released in 4.11.6


On demo.plone.org form block is broken.

It emits an error. See https://demo.plone.org/block/form/edit

The error traces shown by a production build of volto are pretty much useless, since they are positions in the minified bundles.
There's a more useful trace in the browser console, which uses the sourcemap to show source filenames:

In this case, it might be a problem with the error that is passed into the FormFieldWrapper:

Updates/improvements have been made to volto-form-block and the supported backend recently, that caused some issues with enabling volto-form-block in another project, like requiring a spam protection. We briefly discussed this at the beethoven sprint this week. collective.honeypot is a good simple protection.
I could look into it, I see the backend add'on collective.volto.formsupport is in the backend setup.py, but not pinned in constraints.txt.
I would like to check if it is 'wise' to add form support to a demo site where we give people admin accounts so they can start using any hacked/open smtp server/account they find.  Ok, we should reset the instances every day, but still. I'd like to check possible security implications.


Exeption on empty of datetime value (created_at, updated_at...)

Laravel-mongodb Version: 3.6
PHP Version: 7.3.11
Database Driver & Version: 1.7

Description:
I have table called is employees, this is sample fields:

name
birth_date (date time)
created_at (date time)
.....
When I query from this table and one of date time fields value is empty I've got exeption: Data missing

Steps to reproduce

Insert record with empty date time value (not set default)
Query from db this record
Throw exeption

Expected behaviour
It will be return empty same with MySQL normal.
My investigate
The issue will be fixed if I change Jenssegers\Mongodb\Eloquent\Model function asDateTime to like this:
protected function asDateTime($value)
    {
        // Convert UTCDateTime instances.
        if ($value instanceof UTCDateTime) {
            return Date::createFromTimestampMs($value->toDateTime()->format('Uv'));
        }
        try {
            return parent::asDateTime($value);
        } catch (\Exception $exception) {
            return parent::asDateTime(Carbon::now());
        }

    }

But I think this is not solution because I try to use empty by now() is not a correct solution.

👉It would be nice if you implemented it in the latest version of the package!!

Hello,
I wasn't able to reproduce this issue.
Do you use some kind of relations with MySQL?
Thanks.


[Microsoft Teams] return a file to a user when using teams

I am in the process of migrating a slack bot implemented using botkit 0.5.5, to teams using botkit 4.9.0.
Everything is going really except I can't seem to figure out how to access the teams api from the bot instance to upload a file.
For example, in my legacy code I have the following, which utilises the underlying slack API to return a snippet to the end user:
  return new Promise((resolve, reject) => {
    return convo.context.bot.api.files.upload({
      token: convo.context.bot.config.token,
      channels: convo.context.channel,
      mode: "snippet",
      filename: filename,
      content: content
    }, (err) => {
      if (err) {
        reject(err);
      } else {
        resolve();
      }
    });
  });

Is this possible with teams and the latest version of botkit?
Any help would be much appreciated!
Botkit version: 4.9.0
Messaging Platform: Microsoft Teams
Node version: v12.16.2
Os: Mac OS

Many thanks @benbrown - I will check it out 👍


Include docker support for development environments

Hi,
I was missing a quick start setup for vutuv, so I included docker support. Now you can build and run vutuv with just one command (tested on linux). This could lower the barrier for new developers, also you keep your system clean.
Hope you like it.

WOW! Thank you! I just asked our Twitter feed for feedback. We're busy fixing bugs so we have to rely on the community in this one.


spell checker

is codemirror-spell-checker really necessary?
it brings in lots of other dependencies i don't want/need (it seems to only be english wish my website don't use).
I think it should be an optional dependency. Also, isn't possible to use the native spell checker somehow?
(related: https://github.com/cfinke/Typo.js/issues/68, cross-js#dont-use-buffer)

if i want codemirror-spell-checker i probably would want to do something like
const EasyMDE = require('easy-markdown-editor')
const speller = require('easy-markdown-editor/speller')

const easyMDE = new EasyMDE({ speller })

or manually have to install it myself and your lib only use it if it exist

It wasn't my choice to implement it but I think it's a nice feature that provides added value to the editor.
Removing the spell checker actually only saves less than 30kb in the final package.
If you don't want to use the spell checker, just put spellChecker: false in the options.

If you don't want to use the spell checker, just put spellChecker: false in the options.

That won't help reduce the bundle size.

For me 30kb is quite much when you have a budget to follow. if i can get rid of it i take it!
mather of fact when i removed one unnecessary line of code in typo-js (new Buffer()) i saved 20kb by not including Buffer in a minified bundle
sure we both think 30kb don't sound like much this days but that is not the size what is making the page slow. it's what it needs to do with it also.
I'm trying to follow lighthouse audit and get a better score, a better score means better search result
https://github.com/GoogleChrome/lighthouse/issues/1902
https://infrequently.org/2017/10/can-you-afford-it-real-world-web-performance-budgets/

I see your point.
A modular setup would be really useful in certain situations, but I am not sure how to implement that in the package.
It must not compromise any existing functionality or platform support.


Inconsistency with boolean negation as void type guards with --strictNullChecks

TypeScript Version<EMAIL_ADDRESS>
Search Terms: void guard strictNullChecks boolean negation refinement
Code
// Toggle strictNullChecks off and see the type error appear
// on line 7, while 2nd fn never errors and 3rd always errors
const getMaybeStringLen = (maybeString: string | void) => {
  if (maybeString === null || maybeString === undefined) {
    return undefined;
  }
  return maybeString.length;
};

const getMaybeStringLenBoolNegation = (maybeString: string | void) => {
  if (!maybeString) {
    return undefined;
  }
  return maybeString.length;
};

const getMaybeStringLenBoolCoercionNegation = (maybeString: string | void) => {
  if (!Boolean(maybeString)) {
    return undefined;
  }
  return maybeString.length;
};

Expected behavior:

strictNullChecks: false behaves less strictly than strictNullChecks: true. Admittedly a loose criterion, but it seems like the stricter check fails with strictNullChecks: false and passes with strictNullChecks: true, while the type coercive check passes in both cases.

My confusion stems from the fact that ! is a valid type guard for void for both compiler flags, while checking against null and undefined is valid for strictNullChecks: true only.
As a bonus, while ! is always valid, !Boolean(x) is never valid.
Actual behavior:

getMaybeStringLen...

...typechecks with strictNullChecks: true
...does NOT typecheck with strictNullChecks: false

getMaybeStringLenBoolNegation...

...typechecks with strictNullChecks: true
...typechecks with strictNullChecks: false

getMaybeStringLenBoolCoercionNegation...

...does NOT typecheck with strictNullChecks: true
...does NOT typecheck with strictNullChecks: false

Playground Link: http://www.typescriptlang.org/play/#code/PTAEBUHsHNoGwKagM4BcBOBLAxqgcgK5xwDCAFgtgNbKiQBm9oAhgHYAmKCSqFoqATwAOSBOnSR0LISOboAUCDqtQcTKyQB2ADSgA7mUyJQAJg6h6KjQDcxoMRPS02nAMzpOzOHuYDaDyWR5bEhWNFBoBFQAWV8AIwQAZQx1aAAZBBUAXlAACgBbeKSU1mgALhQS6FAAH1BrSEx2AEpQLIA+UABveVBQTCYCouSsUrasnNYiOFq6woEEkdTxnIIOBHp1BBbu3r7QdCiCdBU19g2t9gBuPYBfPcPUY5V5xaqAOkRS3hvbm+DQuFIjFhlUMqwAEKQSBwPAIaDMVCYUJtPKvYqjcqVTG1eqNHYdXZ9AZ5ACE6KWpVaPX2ByOJ1AZwuGmudwe9JeoMxn0y0B+8j+8gBYVQESisQWGNS4KhMJIkDE2GRrDhCKRKJyQ0llKxaBxdQaTVahJp-UGpNliDYWremOa1L2fUez0Z602LJufXuTo5oApHy+fLIvxuQA
Related Issues: #1806, #8322, #10564

Alleged bug aside.
It's very suspicious when you accept void as an input to a function

Sure, that was just a contrived example, though. A more realistic situation would be when handling some promise that has a .catch().
// foo's type is Promise<string | void>
const foo = new Promise<string>(res => res('foo')).catch((err: any) =>
  console.log('I don't know what option types are')
);

You await a function that returns a value like this, and now you have to deal with a Something | void.
I think this is irrelevant to the current question, though.

I would personally expect all 3 examples to error, no matter what.
Because void doesn't actually mean "no value". It isn't an empty type.
It really means "I may or may not have a value but don't try to use me!"
So, you could have a variable of type string|void... But it has a number value!
The only thing I would expect is that too narrow string|void to string, you have to use typeof myVar == "string"

Yes, that would also be much more consistent to me.
The issue body is a bit long, so to summarize:

My confusion stems from the fact that ! is a valid type guard for void for both compiler flags, while checking against null and undefined is valid for strictNullChecks: true only.
As a bonus, while ! is always valid, !Boolean(x) is never valid.

I do not think !maybeString should be a valid guard in any case.

Yeah, to clarify on what @AnyhowStep said from a more solid theoretical basis: void is essentially the dual of any.  Where any is the type system promising you you can do anything you want with it and it won't get in the way (but things may go sour at runtime), void is the opposite: the type system is telling you it will put any value it wants there, so don't get in its way.  So for example, you can't narrow string | void in general because the void could itself be a string:
function foo(): string { return "foo"; }
let bar: () => void = foo;
let result: void | string = bar();

Here, if you're only allowed to see the type of bar, you would think result should be undefined (and therefore distinguishable from string), but it's not!  It's actually a string!  So if you're handed a "value" of type void, you can't do anything with it.  Your only option is to discard it.

Thanks, that does clarify things.
Would you recommend I edit the original issue to focus more on the fact that !foo works as a valid type guard for void, and that comparing against undefined and null do in some cases? Because while I definitely do appreciate being informed about the theoretical underpinnings of void, I'm currently more curious about 1. the current behavior of Typescript, and 2. (tangentially) how to reconcile this with the fact that many failable async operations return Promise<Something | void> given how .catch() is treated.

The fact that !foo works as a typeguard for void I personally would consider a bug, but I'm not sure it's changeable at this stage due to backward compatibility concerns.
It's generally safe to assume that void === undefined so long as you're only dealing with first-order functions: a function declared as returning void will produce a compile-time error if you try to actually return something from it.  However, the minute you start dealing with higher-order functions such as callbacks it's no longer a safe assumption because any other return type is assignable to => void (as illustrated in my example above).

Now the above having been said, Promise<void>, interestingly, doesn't suffer from the above issue:
(async () => {
    async function foo(): Promise<string> { return "foo"; }
    let bar: () => Promise<void> = foo;  // <-- type error here
    let result: void | string = await bar();
})();

I'm not yet convinced that's enough to guarantee Promise<void> actually resolves to something falsy at runtime, though.

Likely a duplicate of #32809; the first two functions are leaking false information about void.

Probably the right answer is that in all cases, void shouldn't narrow at all.

Oh good, @RyanCavanaugh agrees with me that the current typeguard behavior is wrong. :smiley:

Yep, seems like a duplicate of #32809.

I'm re-opening this because my change in #33199 does not fix this issue.
@sandersn Please feel free to re-open/label the issue if you disagree with my changes.


Performance issue linked to current colors palette for big sprites

For big images (> 200x200), processing frames to extract used colors is time consuming.
Either :

deactivate for big sprites
rely on webworkers for frame processing

Fix will be released with v0.5.0


更新不到最新版本

jmessage_flutter 0.2.0 which doesn't match any versions, version solving failed.
pub get failed (1)

可以尝试连接 GitHub master ，不用 pub


Fix #546: infinite loop when $PS4 calls external command

When using set -x with a $PS4 variable which calls a external
command (e.g. PS4='+ $(true)') the color functions end in a infite
loop.
The solution is shift the arguments unconditionally in the color
funtions.
This fixes #546

Thanks!

Great! Thanks!


DefaultHeaders plugin no longer working after ZF3 upgrade

I updated my project, mostly zend-servicemanager and zend-eventmanager from v2 to v3, and now the From header is missing.
Zend\Mail\Transport\Smtp transport expects either a Sender or at least one From address in the Message; none provided

MtMail version is the same (1.5.1).
If i set the header manually it works.
I'm not sure whats going on. Will investigate a bit more.

They are using different instances of the EventManager

The first instance Composer makes for itself in its factory at https://github.com/mtymek/MtMail/blob/master/src/Factory/ComposerServiceFactory.php#L41
Then after this the ServiceManager calls Composer::setEventManager() because of EventManagerAwareInterface
I think the best solution is to remove EventManagerAwareInterface and setEventManager

This is the same issue as https://github.com/mtymek/MtMail/issues/36