[Feature Request] please add an option to change the email intent ui! it is the only reason for me not to use the library.

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a GitHub issue.
For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: all
Android OS version: all
Google Play Services version: all
Firebase/Play Services SDK version: all
FirebaseUI version: 6.2.1

Step 3: Describe the problem:
i want to have a certain design in my app and i cant without the email intent not allowing me to change the design.
Steps to reproduce:

allow me to change the design

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
// TODO(you): code here to reproduce the problem

@netef what do you mean by "change the email intent ui"?  Please provide more information when filing an issue, it's really hard to help from just a title.

hi, thank you for the comment :) i want to make a custom layout for the email activity and i cannot. i am bound to the default layout which is only edit texts and a button at the end and my app design does not look like that. i want to make it more appealing :)

@netef sorry but FirebaseUI is not meant to match the custom style of every app.  We allow you to override the first screen and also change the theme of all following screens:
You can read about those here:
https://firebaseopensource.com/projects/firebase/firebaseui-android/auth/readme#ui_customization
Other than that you'll have to implement your own UI!

i have my own ui but i cannot use your email implementation with it because it is unchangable,
i just want to change the location of the elements and the design not add or remove elements

You can use the base Firebase SDK and build your own UI around it:
https://firebase.google.com/docs/auth/android/password-auth

ok will try this :)

but if i put 3 buttons for login one for google one for facebook and one for email how can i make the email icon go to another activity that will manage the auth as you said?


Display different icons for servers depending on product.

@miq-bot add_label providers/hawkular, topology, ui
cc @abonas

@miq-bot add_label enhancement

All UI PRs typically are required to attach a screenshot for an easier review (manageiq UI team request)

@epwinchell 9c6787c just added the cobbled-up icons that I was talking about.

@miq-bot add_label wip

@miq-bot remove_label wip


Match ActiveSupport Instrumentation event name format spec

Active Support Instrumentation says:

You should follow Rails conventions when defining your own events. The format is: event.library. If your application is sending Tweets, you should create an event named tweet.twitter.

We are not following that format right now.
We are triggering event rack.attack, which means:
event name: "rack"
event namespace: "attack"
We would want:
event namespace: "rack_attack"

This is now available in v6.0.0.


PS1 - Show only m3u files option disappeared after updating software

Because there is no Show only m3u files option, the psx image file of multi cd starts to be displayed again. Please correct it, thank you!

That’s no longer needed.  Per the changelog, While in a system, press the start button then go to UI Settings then go to Theme Configuration then File Extensions

Just found it, thank you!


Improve accuracy and coverage of grammar

This PR is a mishmash of various enhancements that are missing from the current grammar:

Lines beginning with file extensions weren't highlighted
Keywords are matched case-sensitively (the man page says that "case is insignificant")
Escape sequences like \a or ^@ weren't being highlighted
Arguments of TERM lines weren't receiving highlighting (along with lesser-used statements like EIGHTBIT, OPTIONS, etc)

In addition, I've also added various spelling variations to the supported fileTypes array.
/cc @ObserverOfTime

@jolkdarr Note that when installing this grammar through Atom, an older release was downloaded. This indicates that you tagged a release, but didn't publish it to atom.io. You might have to run the following command after publishing a release:
$ apm publish --tag=v1.3.0  # Publish an existing tag
$ apm publish               # Create (and publish) an entirely new tag

The 1.3.0 version wasn't tagged nor published.
Thanks for the notification.
@Alhadis  Your pull request will be merged soon and published in a new release.
Thanks for your contributions.


Undocumented compareWith options in GitHub release task

Options for compareWith parameter are not documented.
##[error]Error: Invalid compareWith attribute: lastRelease.

Only 'lastFullRelease, 'lastNonDraftRelease', or 'lastNonDraftReleaseByTag' options are allowed. For yaml syntax see: https://aka.ms/AA3m1bq

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

ID: 3b12d2c4-f871-1d14-0c16-91e58fb1130b
Version Independent ID: 6b9e1d67-f303-7445-e573-830309ba49a4
Content: GitHub Release task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/github-release.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @vijayma
Microsoft Alias: vijayma

Thanks for letting us know. This issue is fixed in our new task reference, which is generated using the task source code and includes all task inputs, including any aliases here:
https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/github-release-v1?view=azure-pipelines
Please check it out and let us know what you think. Thank you.


update ncurses dependency for Ubuntu/Debian

As of (at least) Ubuntu 22.04 LTS and Debian Buster, there is no longer a w6-dev package. The build works as expected with libncurses-dev instead.
This is a quick fix only based on the latest releases to keep the readme simple. Please reject this patch if you intend to maintain docs for older Ubuntu/Debian versions.
I tested in a ubuntu:22.04 container. I installed these packages prior to building:

autoconf
automake
autopoint
gcc
gettext
libncurses-dev
libgeoip-dev
libmaxminddb-dev
libssl-dev
build-essential

Merged. Thank you!


changing the display name of the amd-gpu operator

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Are you familiar with our operator pipeline?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Your submission should not

[ ] Add more than one operator bundle per PR
[ ] Modify any operator
[ ] Rename an operator
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/assign @mporrato

@mporrato this PR changes the display name for AMD GPU operator, but i see it failed on detect changes. And , according to logs , it says that i cannot change an existing bundle. Does it mean that we need to create a new version of the operator?

@yevgeny-shnaidman yes, that's correct.

@yevgeny-shnaidman since you are using updateGraph: replaces-mode, when publishing a new bundle you need to properly set the spec.replaces field. See https://olm.operatorframework.io/docs/concepts/olm-architecture/operator-catalog/creating-an-update-graph/#replaces .
Also, using :latest in the containerImage spec is a probably a bad idea because latest changes: in general you should point to versioned tags in your registry.


[CDSADAPTERS-2132] Added documentation for --openapi:config-file option

Updated openapi.md to document the --opneapi:config-file option added in the cds-dk package.
Github issue link: https://github.tools.sap/cap/issues/issues/16882
JIRA link: https://jira.tools.sap/browse/CDSADAPTERS-2132

@renejeglinsky could you please review the PR?


Clarify on route filters for Azure public services

added clarity on route filters for azure public services to focus on rationality vs. service specific

@ganesr - happy to review with you. I will update the other articles once we are good on this one :)

@jaredr80 : Thanks for your contribution to the Azure documentation! The author, @ganesr, has been notified to review your proposed change.


Xamarin Studio CocosSharp templates should link against the latest NuGet

The latest cross-platform project includes NuGet packages 1.3.1 instead of 1.4.0. This may confuse new users who are starting up a project (especially those not used to NuGet) who attempt to use the new CCLabel code introduced in 1.4.0.

@vchelaru
I think that might also be what is happening here as well.
XS should automatically prompt for an update so it did not make sense to publish them again.  It seems that may not be the case though.
Thanks for reporting it and we will get new ones out with the next release.
I will mark this as closed for now but it is not forgotten.

@kjpou1 @vchelaru After the next upcoming release, I'll also make sure to provide an updated XS proj. template addin that bundles the latest NuGet packages for offline usage.

Also hoping that we can get the build server to build and update these templates automatically.


Add missing conversion from_str to webp enum

Missed in the PR #98 I proposed with this feature, sorry.
Was implemented as part of #104, but since there was a request to separate stuff from it here's this PR

Thank you so much @Gigas002. I have had a big backlog of reviews so my apologies for the same. If you're no longer interested in the config pr then I'd be willing to look into it ( I understand it's been a while )

Hey, @Shinyzenith! Thanks for reviewing this :)
Separation of #104 indeed seems a bit complicated to me, since I'm not sure my decisions will be met good by the maintainer's team, so I will be glad if you'd help me with it


Where is the Neighborhood Attention 2D?

I wonder where is the  Neighborhood Attention 2D.  I can't find the code

Our standard NA is by default 2D, so it should be in the natten folder. Unless otherwise specified, NA is 2D.

Fixed in PR #55.


QA preparation instructions

We should QA the preparation instructions and make sure that they make sense end-to-end.
Ideally we synchronously run through preparing a system like TYK2.

@IAlibay formatting jumbled here


Should parallel iterators support copy.copy?

MutithreadIterators and MultiprocessIterator don't correctly handles copy.copy. I propose to make MultithreadIterator.__copy__ and Multiprocessiterator.__copy__ raise NotImplementedError, if Chainer developers agreed.
Even though MultiprocessIterator once implemented (hopefully) correct __copy__ function (#3076), current code lacks some additional arguments. MultithreadIterator doesn't implement __copy__ at all.
Currently there are very simple working tests about copy.copy as below. But these tests don't ensure correct handling of race conditions, resource managements and etc.

https://github.com/chainer/chainer/blob/v6.0.0b1/tests/chainer_tests/iterators_tests/test_multithread_iterator.py#L234
https://github.com/chainer/chainer/blob/v6.0.0b1/tests/chainer_tests/iterators_tests/test_multiprocess_iterator.py#L272

I'm not sure about MultiNodeIteratorMaster or MultiNodeIteratorSlave, whereas I guess the situation is not much different.
In Chainer core, only this line uses copy.copy(iterator), and the usage is just a fallback of Itrator.reset. Both MultithreadIterator and MutliprocessIterator supports reset method.

https://github.com/chainer/chainer/blob/v6.0.0b1/chainer/training/extensions/evaluator.py#L205

The other usage founds in an example. This example uses a custom iterator, so it is unrelated.

https://github.com/chainer/chainer/blob/v6.0.0b1/examples/ptb/train_ptb_custom_loop.py#L59

Thanks for the issue. We'll take a look at this.

Copy.copy approach has been replaced by the reset method, but for backward compatibility, we do not want to replace copy.copy with a NotImplementedError. We will raise a PR to add clarity to this.

I think we can leave the copy.copy tests as is, for backwards compatibility. With the changes above, I think we've addressed this issue. If no further request, I'll close once #5882 is merged.

@Crissman All right. Thank you very much.


Accounts created with new version cause insta-ban from grindr

Tested this out at least 3 different times, and each time resulted in a insta-ban.

Used fuckr to create a new account.
Once logged in, before doing anything else, I exited fuckr and started it up again. It let me in fine.
Tried to create a profile, added headline, age, weight, etc... (Regular stuff, nothing spam or anything)
Exited fuckr and started it up again.
Got the unexpected error and couldn't log in anymore. Account was banned.

Appears you must create the account you want to use and setup the profile on an actual mobile device. Will be trying that out next and see if it lets me log on/off fine from fuckr.
In the meantime, I'd be wary using any "personal/main" accounts right now.
Windows x64 version

Closing due to APIv2 shutdown.


Linux - Apache2 - Class "Webkul\Product\DataTypes\CartItemValidationResult" not found

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2.2.2
Issue Description
When i try to add an item into the cart, i got this exception:
Class "Webkul\Product\DataTypes\CartItemValidationResult" not found
Preconditions
Clean install, with product.
Steps to reproduce

Clean install
Create a product
Go to the frontend page of the product and click 'Add to Cart'
See error

Expected Result
No exception
Actual Result
I got exception with a notification: Class "Webkul\Product\DataTypes\CartItemValidationResult" not found
Screenshots

Additional context
I found out the problem source is that the Webkul/Product/src/Datatypes folder should spelled DataTypes with capital T.
The linux system doesn't like that

Spelling is correct on yur mentioned version,
https://github.com/bagisto/bagisto/tree/v2.2.2/packages/Webkul/Product/src/DataTypes


Updated release notes

@cs-abhishek-shukla

Release notes look good to me
I made a few changes in [docs/usage.md] as well, please make sure those changes are also reviewed


Fix KeyError when running via initialize_model() for image features.

Training a model using model.initialize_model(); model.train_online(); fails for image inputs.
As far as I understand, that field (model_definition['data_hdf5_fp']) is only created in preprocess_for_training, which only is called from model.train(). This change fixes the resulting KeyError, however it feels symptomatic. It would require a deeper knowledge of the code base than what I have at this time to ensure preprocessing is done the same in model.train() and model.initialize_model()

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

Thanks for this PR. @ydudin3 can you please take a look at the deeper problem here? (the assumption that there's always a hdf5 file available) and also can you confirm if this solves the problem for the moment?

@laserbeam3 we did some changes in the image feature in the https://github.com/uber/ludwig/pull/219 commit. Can you please check if it solves the issue that originated this PR?

Closing this because other PRs should have solved this issue already. Feel free to reopen if the problem emerges again.


Confidence

Is there a way to estimate a confidence associate with a pixel-level depth prediction?

There are some basic approaches. For example, you can feed an original image and a horizontally flipped image to the model, and measure the disagreement of model predictions. This disagreement can serve as confidence. You can also measure the disagreement between different models to probe the difficulty of the sample. You may refer to the research area of uncertainty estimation for regression tasks.


Okta integration blank page

I have configured okta and when I browser disptach url its redirecting to my okta sso after entering details its showing blank page, any thoughts on this ?
I could able to integrate using above steps, when I open url its redirecting to okta , once I have enter credentials it logged in, but its showing blank page always
I have added below env varibales in my .env file
DISPATCH_AUTHENTICATION_PROVIDER_SLUG=dispatch-auth-provider-pkce
DISPATCH_JWT_AUDIENCE=0oaxxxxxxxx
DISPATCH_PKCE_DONT_VERIFY_AT_HASH=true
DISPATCH_AUTHENTICATION_PROVIDER_PKCE_JWKS=https://trial-2372xxx.okta.com/oauth2/v1/keys
VUE_APP_DISPATCH_AUTHENTICATION_PROVIDER_SLUG=dispatch-auth-provider-pkce
VUE_APP_DISPATCH_OPEN_ID_CONNECT_URL=trial-237xxx.okta.com
VUE_APP_DISPATCH_CLIENT_ID=0oaxxxxxxx
VUE_APP_DISPATCH_AUTHENTICATION_PROVIDER_PKCE_CLIENT_ID=0oa2rprxxxxxxxxxxx
VUE_APP_DISPATCH_AUTHENTICATION_PROVIDER_PKCE_OPEN_ID_CONNECT_URL=https://trial-237xxxx.okta.com/
VUE_APP_DISPATCH_AUTHENTICATION_PROVIDER_USE_ID_TOKEN=true
DISPATCH_UI_URL=http://<IP_ADDRESS>:8000/
DISPATCH_JWT_EMAIL_OVERRIDE=0oa2rprxxxxxx

?
& where should we give permission to users since its not local users.
ex<EMAIL_ADDRESS>should get admin access to default project<EMAIL_ADDRESS>should get read access to default organization & project ? How can we achieve this

Regarding your server error. I would double-check your JWKS endpoint here:
DISPATCH_AUTHENTICATION_PROVIDER_PKCE_JWKS=https://trial-2372xxx.okta.com/oauth2/v1/keys
Judging by the documentation (https://developer.okta.com/docs/reference/api/oidc/#response-example) it should return JSON with valid kid (which is what our code assumes) there could be a configuration error or maybe the URL is just wrong.
Once a user logs in, they will have a user created for them with default permissions. If you wish to modify these permissions you can do so via the UI by navigating to Settings > Members and editing their member.
To establish the "first admin" problem we typically recommend logging in and then elevating yourself to be admin via the cli command:
dispatch ❯ dispatch user update --help
Usage: dispatch user update [OPTIONS] EMAIL

  Updates a user's roles.

Options:
  -o, --organization TEXT         Organization to set role for.  [required]
  -r, --role [Owner|Manager|Admin|Member]
                                  Role to be assigned to the user.  [required]
  --help                          Show this message and exit.

@kevgliss I can able to get the key value, kid value was here
https://trial-2372159-admin.okta.com/oauth2/v1/keys

Are you able to inspect the authorization header value being sent? Is should be a base64encoded string that has a 'kid' in there.
https://github.com/Netflix/dispatch/blob/master/src/dispatch/plugins/dispatch_core/plugin.py#L98

Thanks @kevgliss , I could able to integrate okta in dispatch , added steps in https://github.com/Netflix/dispatch/issues/1597


ask for help

I guess it's an accidental error. Maybe you are running two experiments simultaneously and they happen to write the same file at the same time? I think you can resume from the previous checkpoint and continue training.

hi， i also have trained the model but the loss is larger than yours（more than 4.0），and after epoch =300，the err  not change to smaller ，even becoming bigger sometimes。do you have changed some param in opt.lua?


enable/disable option

It would be useful to have a mechanism to start / stop profiler when it is at production (IMO).
I am not sure if it is possible or not, but a page for authenticated users that disables profiling or may be updating SILKY_INTERCEPT_PERCENT to 100 may be suitable for the task.

Just as a note removing silky from installed apps won't work in a production setting. If you try and import the profiler decorator anywhere else after you've removed it from installed apps it throws an error:
Traceback (most recent call last):
  File "/opt/cloudscheduler/web_frontend/cloudscheduler/cloudscheduler_web/wsgi.py", line 14, in <module>
     application = get_wsgi_application()
   File "/usr/lib64/python3.6/site-packages/django/core/wsgi.py", line 13, in get_wsgi_application
     return WSGIHandler()
   File "/usr/lib64/python3.6/site-packages/django/core/handlers/wsgi.py", line 140, in __init__
     self.load_middleware()
   File "/usr/lib64/python3.6/site-packages/django/core/handlers/base.py", line 37, in load_middleware
     middleware = import_string(middleware_path)
   File "/usr/lib64/python3.6/site-packages/django/utils/module_loading.py", line 17, in import_string
     module = import_module(module_path)
   File "/usr/lib64/python3.6/importlib/__init__.py", line 126, in import_module
     return _bootstrap._gcd_import(name[level:], package, level)
   File "<frozen importlib._bootstrap>", line 994, in _gcd_import
   File "<frozen importlib._bootstrap>", line 971, in _find_and_load
   File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
   File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
   File "<frozen importlib._bootstrap_external>", line 678, in exec_module
   File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
   File "/usr/lib/python3.6/site-packages/silk/middleware.py", line 10, in <module>
     from silk.collector import DataCollector
  File "/usr/lib/python3.6/site-packages/silk/collector.py", line 9, in <module>
     from silk import models
   File "/usr/lib/python3.6/site-packages/silk/models.py", line 55, in <module>
     class Request(models.Model):
   File "/usr/lib64/python3.6/site-packages/django/db/models/base.py", line 108, in __new__
     "INSTALLED_APPS." % (module, name)
 RuntimeError: Model class silk.models.Request doesn't declare an explicit app_label and isn't in an application in INSTALLED_APPS.

I've been trying to come up with a way to disable silky in production that doesn't involve scrubbing all the decorators from my function definitions. These are really the only ideas i have left:

Wrap the silky profiler decorator in a decorator of my own that conditionally applies the silky decorator.
Conditionally import a dummy decorator that does nothing instead of the silky profiler decorator.

What I ended up doing is creating a configuration option to remove silky from installed apps and middleware then instead of importing directly from silky I import the "silk_profile" decorator from this file:
from functools import wraps

from myapp.lib.csv2_config import Config
config = Config('web_frontend')

if config.enable_profiling:
    from silk.profiling.profiler import silk_profile
else:
    def silk_profile(name=None):
        def s_profile(f):
            @wraps(f)
            def wrapped(*args, **kwargs):
                r = f(*args, **kwargs)
                return r
            return wrapped
        return s_profile

How about not removing silk from installed apps but removing it from Middlewares?
I think that would work

When changing  SILKY_INTERCEPT_PERCENT or similar, one needs to deploy&restart, which is not nice in production.
Maybe there could be  SILKY_CONFIG_FETCHER  -- (path to) function  which fetches configs as dict. By default it would just collect standart settings, that.startswith('SILKY_'). And we could use DB to store settings.
ps.: quick search: seems https://bitbucket.org/jiaaro/django-tupperware/ can do dynamic settings overriding

How about using jazzband/django-constance as an optional solution?
User can change SILKY_INTERCEPT_PERCENT or enable/disable silk from admin page.


this app screwed the phone screen

Just installed the app on a Xiaomi Mi 8 stock rom, rooted. Created a profile and the screen was shrinked to the center of the display. I can't touch any button. I have uninstalled the app from adb pm uninstall but still the screen is unusable and shrinked. I can only reflash my phone. Disappointed.

Found a solution:
From adb shell, type:
wm size reset
wm density reset

Hi, I also have the same problem as you on the Xiaomi Mi 9T,
thanks for the solution.

Hi people,
Second Screen made my phone go into permanent loop boot... Any tips on how to exit the boot loop? I do not seem to get the brief access to adb shell, but I might be doing something wrong...
https://android.stackexchange.com/questions/234549/bad-screen-resolution-permanently-crashes-poco-f1

Found a solution: From adb shell, type:
wm size reset wm density reset

Hi @farmerbb , Could you please help if you can? Second Screen made my phone go into permanent loop boot... Any tips on how to exit the boot loop? I do not seem to get the brief access to adb shell, but I might be doing something wrong... https://android.stackexchange.com/questions/234549/bad-screen-resolution-permanently-crashes-poco-f1

Thanks, this really helped me. I was unlucky that my Mi8 always fails to boot and then rebooted into TWRP after applying a 720p profile with this app, even removing this app with TWRP adb didn't work, then I saw this and spammed these commands on my PC connected to my phone during the boot
adb shell wm size reset
adb shell wm density reset

and it worked!


x/tools/gopls: duplicates multi-line comment on first function after imports on every save

What version of Go are you using (go version)?

$ go version
go version go1.15.5 windows/amd64

gopls version
golang.org/x/tools/gopls v0.5.4
<EMAIL_ADDRESS>h1:PkQWDfzw8CxoseHDQniF2GhzmndN8KnQL6gcG2gnF3Y=

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\kdeconinck\AppData\Local\go-build
set GOENV=C:\Users\kdeconinck\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\kdeconinck\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\kdeconinck\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\kdeconinck\DEV\github.com\kdeconinck\GOSudoku\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\KDECON~1\AppData\Local\Temp\go-build977075402=/tmp/go-build -gno-record-gcc-switches

What did you do?
Create a new Go source file and add a function to the file.
Add a comment to that function which spawns multiple lines.

// Package main implements the main entry point for the application.
package main

// PRIVATE: The method `main` implements the MAIN entry point for the application.
//          This method is executed when the application is started.
func main() {
	// TODO: Implement the 'MAIN' entry point for the application.
}

Next save the document.

What did you expect to see?
The document should be saved the same way as it was written.
What did you see instead?
The lines below the first line of the comment of the method is duplicated.

// Package main implements the main entry point for the application.
package main

// PRIVATE: The method `main` implements the MAIN entry point for the application.
//          This method is executed when the application is started.
//          This method is executed when the application is started.
func main() {
	// TODO: Implement the 'MAIN' entry point for the application.
}

When saving again, there's a duplication again:

// Package main implements the main entry point for the application.
package main

// PRIVATE: The method `main` implements the MAIN entry point for the application.
//          This method is executed when the application is started.
//          This method is executed when the application is started.
//          This method is executed when the application is started.
//          This method is executed when the application is started.
func main() {
	// TODO: Implement the 'MAIN' entry point for the application.
}

Related issues.
https://github.com/golang/go/issues/39147
Additional information:
When the first method in the Go source file is a method that does NOT contain a comment or that does contain a comment which does NOT span multiple lines, other methods in this file containing comments which spans multiple lines are NOT duplicated.

// Package main implements the main entry point for the application.
package main

func temp() {}

// PRIVATE: The method `main` implements the MAIN entry point for the application.
//          This method is executed when the application is started.
func main() {
	// TODO: Implement the 'MAIN' entry point for the application.
}

Thank you for reporting this issue. The fix for this should be in the pre-release for gopls/v0.5.5 (which will be out later today). Can you try it out and confirm that it fixes the issue? You can download it by running GO111MODULE=on go get<EMAIL_ADDRESS>

Thanks for the feedback.
I have installed it locally in my project and the issue seems to be gone.
Ticket can be closed.


Add optional .riak.node.network_interface_name to config.json

This PR adds some mechanisms necessary for the scheduler to pass configuration to its executors via their execution environment.
The first example is RIAK_MESOS_NODE_IFACE: a value optionally specified in config at .riak.node.network_interface_name. Details of its purpose can be found here: basho-labs/riak-mesos-executor#26
TODO Revert the riak.conf.default file - we don't need to use this variable by default, only if an operator wishes to do so. EDIT: DONE

👍 LGTM


publish a picture as story

Hello,
is it possible to publish a picture as a story via that api?

#165


passing components folder to daprd

Fixes https://github.com/dapr/cli/issues/366

https://github.com/dapr/cli/blob/737a4e6e776466e772db9b3a1c91dd4bbdbad1bd/pkg/standalone/run.go#L47

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Thats what the code is doing, its getting the default path

There's a components path field on the Config struct

Thats only set when user overrides it. Here its using the default path when not overridden by user, see here https://github.com/dapr/cli/pull/367/files#diff-184e85420a4841e42d77c23247296911R273

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Thats what the code is doing, its getting the default path

There's a components path field on the Config struct

Thats only set when user overrides it. Here its using the default path when not overridden by user, see here https://github.com/dapr/cli/pull/367/files#diff-184e85420a4841e42d77c23247296911R273

link may be wrong. https://github.com/dapr/cli/blob/c37118166b1b6c869cd2795863eb99f4960fe100/cmd/run.go#L248
this is not the current directly.
the code always set componentsPath.. But I cannot find where componentsPath is set by default.

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Thats what the code is doing, its getting the default path

There's a components path field on the Config struct

Thats only set when user overrides it. Here its using the default path when not overridden by user, see here https://github.com/dapr/cli/pull/367/files#diff-184e85420a4841e42d77c23247296911R273

link may be wrong.
https://github.com/dapr/cli/blob/c37118166b1b6c869cd2795863eb99f4960fe100/cmd/run.go#L248
this is not the current directory.
the code always set componentsPath.. But I cannot find where componentsPath is set by default.

Indeed. See my comments above.

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Thats what the code is doing, its getting the default path

There's a components path field on the Config struct

Thats only set when user overrides it. Here its using the default path when not overridden by user, see here https://github.com/dapr/cli/pull/367/files#diff-184e85420a4841e42d77c23247296911R273

link may be wrong.
https://github.com/dapr/cli/blob/c37118166b1b6c869cd2795863eb99f4960fe100/cmd/run.go#L248
this is not the current directory.
the code always set componentsPath.. But I cannot find where componentsPath is set by default.

Indeed. See my comments above.

Adding the correct link, componentsPath is getting set here https://github.com/dapr/cli/blob/fix_366/pkg/standalone/run.go#L273
which is calling func at https://github.com/dapr/cli/blob/fix_366/pkg/standalone/run.go#L347

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Thats what the code is doing, its getting the default path

There's a components path field on the Config struct

Thats only set when user overrides it. Here its using the default path when not overridden by user, see here https://github.com/dapr/cli/pull/367/files#diff-184e85420a4841e42d77c23247296911R273

link may be wrong.
https://github.com/dapr/cli/blob/c37118166b1b6c869cd2795863eb99f4960fe100/cmd/run.go#L248
this is not the current directory.
the code always set componentsPath.. But I cannot find where componentsPath is set by default.

Indeed. See my comments above.

Adding the correct link, componentsPath is getting set here https://github.com/dapr/cli/blob/fix_366/pkg/standalone/run.go#L273
which is calling func at https://github.com/dapr/cli/blob/fix_366/pkg/standalone/run.go#L347

Can we use the CLI flags to set default values? I want to keep it consistent.

Need to make sure that field gets populated with the default dir path if not supplied by the user.

Thats what the code is doing, its getting the default path

There's a components path field on the Config struct

Thats only set when user overrides it. Here its using the default path when not overridden by user, see here https://github.com/dapr/cli/pull/367/files#diff-184e85420a4841e42d77c23247296911R273

link may be wrong.
https://github.com/dapr/cli/blob/c37118166b1b6c869cd2795863eb99f4960fe100/cmd/run.go#L248
this is not the current directory.
the code always set componentsPath.. But I cannot find where componentsPath is set by default.

Indeed. See my comments above.

Adding the correct link, componentsPath is getting set here https://github.com/dapr/cli/blob/fix_366/pkg/standalone/run.go#L273
which is calling func at https://github.com/dapr/cli/blob/fix_366/pkg/standalone/run.go#L347

Can we use the CLI flags to set default values? I want to keep it consistent.

Sure, we can

I think you forgot to paste the function to cmd/run.go


Version Tagging

I've added a file version_config.cmake with the versions of all the packages we build.

I have fixed all the issues you pointed out and a couple more.
Travis is building with system HDF5, SWIG, FFTW3, Boost.
Travis actually builds the superbuild, without those sub-projects.
However, build of SIRF actually fails because it does not find STIR and ISMRMRD and therefore it builds only the utils. The main difference with the update scripts is that in that case cmake was called specifying the CMAKE_PREFIX_PATH pointing to STIR and ISMRMRD etc. https://github.com/CCPPETMR/CCPPETMR_VM/blob/master/scripts/UPDATE.sh#L85

See my comments on #30. If necessary, set STIR_DIR but you must have changed something that breaks this.

you might be able to see which commit broke the build by going through Travis logs (although it looks like it only runs for every push, not every commit)

Investigating I found that STIR doesn't actually create the INSTALL/lib/cmake directory and doesn't fill it.
Additionally, it seems that including the version_config.cmake in the initial CMakeList.txt was creating other troubles as Gadgetron not configuring...

i'm guessing it's not checking out the correct version. STIR 3.0 didn't create the config, which is why you need the git hash. possibly the same for gadgetron. You can go into the STIR directory that was created and do a "git status" there to check.

That's exactly the same conclusion I came to: all the packages that are git fetched do not behave as expected.  Probably for the reason that you mention.
Now I'd like to know which are the exact commit hash I have to use for STIR, Gadgetron and ISMRMRD. The ones I assumed to be correct aren't and I can't go trial-and-error.

git hashes are in my previous comments for version_config.cmake.

The culprit was the outdated version in the version_config.cmake file for STIR, Gadgetron and ISMRMRD.
Now it does compile and passes the tests and I've run all the python examples. The only bit I had to run manually was to make (copy) a configuration file for gadgetron and source the file to set environment variables.
I've tried to use the hash you gave me for STIR but it wouldn't work. Currently this build works with all the latest code from the CCPPETMR branches, not the upstream.

thanks. this looks nearly ready.
I think it'd be cleaner to actually just remove the lines that you commented out in the External* files. Otherwise we'll have to remove them later. what do you think?
And sorry for the git hash. Using the top of the CCP PETMR forks is fine.
By the way, I intend to "squash and merge" this to generate a single commit, so don't merge this into another branch. Or let me know if that'd be a problem.

I waited to rebuild and test on my machine before committing.
I've removed all the dangling comments and hopefully made it more consistent in the use of references to project URL/TAG.


Ticked version, regenerated if needed. (Double-check reqs!)

Made using tick_my_feedstocks.py!

[x] I have vetted this recipe

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Added some comments on the diff above. Making these changes allowed the recipe to build through for me locally. Would recommend trying them here to see if they work for you.

Good notes - accepted a different patch earlier (ref) so going to close and reopen from the new version.


Introduce DALROOT and Major Minor versions in env scripts

Description
Adding DALROOT variable
Adding Major and Minor version env variables to identify DLL versions
Updating path for installed lib search

/intelci:run

Shouldn't it be 'ONEDALROOT'

@rlnx  - on system we are instilled in 'dal' dir to align with other components.
From variable name - short variant is also aligned with other libs
So the main thought was to align with other components

/intelci:run


FIX: Redirect responses written to cache despite empty body.

Checking for empty content later in the method, once content for 301/302 redirect responses has been created.

Shouldn't we write a php file that will perform the redirect in the cache folder?

@dhensby it does fall back to that using OldPageRedirector::onBeforeHTTPError404() but having a statically cached redirect is faster.
Also, there are quite a few edge cases that OldPageRedirector does not cover. For instance, one query relies on URL segment alone:
https://github.com/silverstripe/silverstripe-cms/blob/3/code/controllers/OldPageRedirector.php#L49
After discussing with @tractorcow, this filter should probably include ParentID = 0 so that it doesn't find other pages nested in the site tree with the same URL segment (an issue we bumped into recently). But there are other edge cases and OldPageRedirector should probably be rewritten. Perhaps we need to denormalize the data in the versions table to look up a page ID for old URLs.

After discussing with @tractorcow, this filter should probably include ParentID = 0 so that it doesn't find other pages nested in the site tree with the same URL segment (an issue we bumped into recently). But there are other edge cases and OldPageRedirector should probably be rewritten. Perhaps we need to denormalize the data in the versions table to look up a page ID for old URLs.

I've always found this behaviour helpful :/

It wouldn't be an alteration of the behaviour to set ParentID = 0 but rather fixing a bug, unless I'm missing something. When the method tries to resolve the URL path into a page ID it recursively calls itself starting with the root node, it doesn't always get that root node if there is more than one page in the site tree with the same URL segment - it gets the first one which might be nested somewhere in the tree (this is the problem we were bumping into).
So if it starts with an incorrect root node it can't find the leaf node and instead of a 301 redirect it continues with the 404.
Fixing the ParentID for the root node query is a small patch, there are other edge cases that aren't covered. It's actually quite a tricky problem because 1 URL path can potentially resolve to more than 1 page ID over the lifetime of a site.

@frankmullenger at the moment if you have a page on your site which is:
example.com/page-1/child-page
but you navigate to
example.com/child-page
It will 301 to example.com/page-1/child-page
Yes, if there's more than 1 page with the URL segment of child-page the first one found will be used; but this isn't a problem as you could never resolve that issue, really.
It may not have been an intended use when this was written, but it's expected given how it's written and that change in behaviour will impact people
In fact, won't adding ParentID = 0 actually mean that the purpose it was intended for won't be matched (ie: this was to redirect sites that had non-nested URLs to nested URLs)? So this behaviour was indeed intended...


Can this lib read in CSV? or just export in CSV?

I need to be able to support reading XLSX and CSV. Do I need a seperate lib to read in CSV? Or can this library also read in CSV?

this is a duplicate of 489


SIP Voicemail with Node.JS

Thanks!
There are additional files that we usually add. You can take this example as a reference: https://github.com/signalwire/guides/tree/main/Voice/Voicemails to Email IVR with NodeJS
So we'll need:

a Dockerfile
env.example file (the app should load the configuration from environment variables)
package.json
package-lock.json

Finally, I suggest naming the file simply "index.js" for consistency.

project completed in PR#48


Use process.exitCode instead of process.exit()

On certain platforms (e.g. Windows) the console streams are async,
since process.exit() immediately exists the process the console
streams may not have been flushed when the process ends.
Migrating to process.exitCode assignment, will allow the process to
end "naturally" when all async work is completed but still use the
specified exit code.

Change Type

[ ] Feature
[ ] Chore
[x] Bug Fix

Change Level

[ ] major
[ ] minor
[x] patch

Further Information (screenshots, bug report links, etc.)
Checklist

[ ] Added tests / did not decrease code coverage
[ ] Tested in supported environments (common browsers or current Node)

😞 sorry, missed the conventional changelog mention in the contributing guide, will update. Sorry for the noise.

OK, fixed up the commit message to have the right format. Also noticed that the link in the contributing.md was out of date so I submitted https://github.com/entropic-dev/entropic/pull/280 to fix it.


gen_random_hex works across whole one-click-app YAML

This PR contains a couple improvements around the $$cap_gen_random_hex(...) replacement in one-click apps:

gen_random_hex now works across whole YAML, not only under the caproverOneClickApp.variables section.
gen_random_hex(n) generates $n$ ascii characters, not the hex representation of $n$ bytes.

I confirmed both these behaviors now match what the Caprover frontend UI does.
Added a test suite around variable substitution, including bringing the tox invocation up-to-date.

This is ready for review & merge.
I'm happy to handle any merge conflicts between #6 and #7, if needed.


[SPARK-12041] [ML] [PySpark] Add columnSimilarities to IndexedRowMatrix

Add columnSimilarities to IndexedRowMatrix for PySpark spark.mllib.linalg.

cc @yanboliang

Test build #47222 has started for PR 10158 at commit d96099b.

Test build #47222 has finished for PR 10158 at commit d96099b.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47222/
Test FAILed.

Test build #47223 has started for PR 10158 at commit 4a56473.

Test build #47223 has finished for PR 10158 at commit 4a56473.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47223/
Test FAILed.

Merged build finished. Test FAILed.

Test build #47225 has started for PR 10158 at commit f607c37.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47224/
Test FAILed.

Merged build finished. Test FAILed.

Test build #47225 has finished for PR 10158 at commit f607c37.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47225/
Test FAILed.

Test build #47227 has started for PR 10158 at commit 7332bd1.

Test build #47227 has finished for PR 10158 at commit 7332bd1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47227/
Test PASSed.

LGTM @mengxr

cc @mengxr

@vectorijk thanks for the PR, it looks good to me. cc @jkbradley

LGTM  I'll just run the tests once more before merging

Test build #2331 has started for PR 10158 at commit 7332bd1.

Test build #2331 has finished for PR 10158 at commit 7332bd1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merging with master

Thanks for the PR!


docs: update README.md

Update some rules in README.md

Git Branch Rules
Git Commit Rules

@jgw0915 @TsaiJouWu Please help to check the PR and approve it.
If you have any questions please let me know first.
Thanks!


Example of initializing force simulation should wait until data is loaded.

Reading the Force-Directed Graph example, I find it hard to understand, when the simulation starts.
According to the the API, "The simulator starts automatically". So that would be here:
var simulation = d3.forceSimulation()
    .force("link", d3.forceLink().id(function(d) { return d.id; }))
    .force("charge", d3.forceManyBody())
    .force("center", d3.forceCenter());

Hmmm, without nodes, there wont be much to simulate. Continue reading the code, I assume, it might happen here:
simulation
  .nodes(graph.nodes)
  .on("tick", ticked);

Now it has nodes and can work. But it still has no links. Hmmm. So does it run considering the other forces? Well, Javascript is singlethreaded, so the time callback cannot execute anyway. So presumably it the timer fires, but the callback will be run after this function, or more exactly, after the XmlHttpRequest callback is handled. Lots of pondering. Continue reading
simulation.force("link")
  .links(graph.links);

I assume that now the simulation has all arguments and can run considering the link forces. Which in fact is correct. Anyway, I find the API a bit confusing in this point (otherwise the new force API is absolutely ingenious!).
Debugging reveals, that force.initialize() is called 3 times, computing
nodeById = map$1(nodes, id),

twice, unnecessarily. So maybe it makes sense to have an API method alike
simulation.start(nodes, links)

or more abstract (or precise)
simulation.start(all parameters required to run the configured simulation)

? For me that would have been clearer. Most functions would fall into category configure while start, restart, stop would be category run.
thx so much for d3
urbanhop

It starts upon creation. That is what is meant by “automatically”. If you prefer you can explicitly stop the force layout upon creation and then start it again later. (However stopping the force layout does not prevent the forces from being reinitialized when the nodes change, since starting and stopping only affects the internal timer, and the simulation can be run manually by calling simulation.tick.)
You can change the nodes after the force layout starts. Yes, this does cause the constituent forces for be reinitialized, but this cost is typically negligible so it doesn’t seem urgently necessary to optimize this. I don’t want to denounce this work until the next tick because it makes debugging slightly harder, but I suppose it would be an option.
You could not implement the type of simulation.start API you propose without the simulation knowing about all possible forces and their parameters. The design is intended to be generic and extensible in that the simulation can be run with arbitrary forces that have arbitrary behavior and arbitrary parameters; the simulation does not know what links are, for instance. Furthermore forces can be bound and unbound dynamically, so it doesn’t make sense to pass all the parameters through start. I suppose you could use weakly (string) named parameters defined in the simulation as is done with the forces, but that seems worse than having the forces define their own parameters and by worry about potential name conflicts.
There is a slight problem with the example on that the simulation can run for a tick with an empty graph (no nodes). This is mostly harmless especially since the ticked listener is not yet registered, but it does mean that the simulation can cool slightly while the data is loading. I can see if there is an easy way to avoid this, though I don’t think it will require any changes to the API. (Deferring creation of the simulation until the data loads, for example, would do it.)

Think its fine as it is. After some hours working with it and now reading your reply, I got the feeling for the API, which favors convenience over minor inefficiencies.
Thank you for your time.

I am reopening this issue as a reminder to myself to fix the issue you raised with the example. Thank you for your feedback!

I ended up with this sequence:
var canvas = document.querySelector("canvas"),
    context = canvas.getContext("2d"),
    width = canvas.width,
    height = canvas.height;

d3.json("miserables.json", function (error, graph) {
    if (error) throw error;

    var simulation = d3.forceSimulation()
        .nodes(graph.nodes)
        .force("link",
            d3.forceLink()
            .id(d => d.id)
            .links(graph.links))
        .force("charge", d3.forceManyBody())
        .force("center", d3.forceCenter(width / 2, height / 2))
        .on("tick", ticked);

    function ticked() { ...

Some options:

Defer automatic starting until the first tick listener is scheduled. This way, a simulation won’t start until someone is listening. This would be backwards-compatible with the current behavior, but would avoid the need to simulation.stop when computing a static layout, and would reduce the likelihood of the simulation starting before it‘s ready. (I say likelihood because you can still register your tick listener before configuring the simulation… This would fix the example, however.)

Disable automatic starting, period. This would not be backwards-compatible with d3@4, but it’s how d3@3 did it.

Defer automatic starting until nodes is non-empty. This feels a little too magical because a simulation could have forces that do things even if the simulation has no nodes.

@mbostock
I happen to build a force layout. My requirement is on clicking of a node several other nodes are to be added to the graph and are to be linked to the initial clicked node. After I update my node_array and link_array, the tick function runs its implementation to plot the nodes. But I see that all links appear on the graph but not all nodes, the nodes are situated at the left corner and multiple clicks of the initial node the nodes come back and sit in their position.
function update(graph_nodes, graph_links,selectedNode)
{
var gl = [], gn = [];
    if(selectedNode &&  selectedNode.id !== concept)
    {
       level2_relations = level2.get(selectedNode.id)
        // console.log(level2_relations);
        level2_relations.forEach(function(value){
            if (value.CUI1 !== concept && value.CUI2 !== concept)
            {
            gn.push({'id':value.CUI2})
            gl.push({'source':value.CUI1, 'target':value.CUI2})
            }
        });
        console.log("New Nodes: ",gn.length,gl.length);
    }
   let updated_nodes = graph_nodes.concat(gn);
   let updated_links = graph_links.concat(gl);
   console.log("Updation",updated_nodes.length, updated_links.length)
    return [updated_nodes, updated_links];
}

function restart(graph_nodes, graph_links,selectedNode) {

    updated = update(graph_nodes, graph_links,selectedNode)
    let updated_nodes = updated[0];
    let updated_links = updated[1];

    console.log("Restarting", updated_nodes.length, updated_links.length);

    // Apply the general update pattern to the links.
    link = g.selectAll(".link").data(updated_links);
    link.exit().remove();
    link = link.enter().append("line").attr("class", "link")
                .attr("stroke","black").attr('stroke-width', 1).merge(link);

    // Apply the general update pattern to the nodes.
    node =g.selectAll(".node").data(updated_nodes);
    node.exit().remove();
    node.enter().append("circle").attr("class", "node")
    .style("fill","#81D4FA").attr("r", 8).merge(node);

    // Update and restart the simulation.
    simulation.nodes(updated_nodes);
    simulation.force("link").links(updated_links);
    simulation.restart();
  }

function selectNode(selectedNode, graph_nodes, graph_links) {

    // level2_specific_relation = level2.get(selectedNode.id);
    // level2_specific_relation.forEach(function(value_level2){
    //     graph_nodes.push({'id':value_level2.CUI2});
    //     graph_links.push({'source':value_level2.CUI1, 'target':value_level2.CUI2});
    // });

    flag = true;
    restart(graph_nodes, graph_links, selectedNode);
    // const neighbors = getNeighbours(selectedNode, graph_links)
    // node.style('fill', nod => getNodeColor(nod, neighbors));
    // textElements.attr('fill', nod => getTextColor(nod, neighbors))
    // link.attr('stroke', lin => getLinkColor(selectedNode, lin));

}

The execution goes like on clicking on a node selectNode function is called.

I've implemented solution 1 in #165. We start when any listener is added (ie "tick" or "end") so it should cover @IPWright83 's use case too (though I'd be reassured if it could be tested independently).
demo notebook @ https://observablehq.com/d/0e3586acc3d9cd55

@Fil really disappointingly my new job doesn't give me the chance to use D3 anymore, which is such a shame as it's an amazing library.
I will see if I can find some time to test though

We finally decided against this change. Use .stop() then .restart() when the data is ready.


Tofu pipeline

Basic tofu dataset pipeline set up. There are 4 different granularity settings, which are admittedly defined in quite a convoluted way, but I think the comments and readme explain how to use them.

Finally: I'd suggest renaming data_utils.py to load_data.py or even just tofu.py

I'd go tofu.py

On all the comments re indexing etc. I don't think we need to reinvent the wheel if something already works. But my main suggestion is to consider whether it would be helpful to create author_id and question_id columns in the dataset at the beginning, and then use those in the various sampling/filtering functions. One pro of this is it might make some analysis easier later - e.g. we wouldn't need to keep propagating information about which questions/authors have been removed to figure out which rows in the retain/forget sets correspond to which authors (as we could just refer to the author_id).
HuggingFace Datasets also have some in-built functions that could help: https://huggingface.co/docs/datasets/en/process . E.g. with an author_id column the author_level split could just be something like:
forget_set = all_data.filter(lambda row: row["author_id"] in forgotten_author_numbers)
retain_set = all_data.filter(lambda row: row["author_id"] not in forgotten_author_numbers)

On all the comments re indexing etc. I don't think we need to reinvent the wheel if something already works. But my main suggestion is to consider whether it would be helpful to create author_id and question_id columns in the dataset at the beginning, which could also be used in the various sampling/filtering functions. One pro of this is it might make some analysis easier later - e.g. we wouldn't need to keep propagating information about which questions/authors have been removed to figure out which rows in the retain/forget sets correspond to which authors (as we could just refer to the author_id).
HuggingFace Datasets also have some in-built functions that could help: https://huggingface.co/docs/datasets/en/process . E.g. with an author_id column the author_level split could just be something like (probably not the most efficient way):
forget_set = all_data.filter(lambda row: row["author_id"] in forgotten_author_numbers)
retain_set = all_data.filter(lambda row: row["author_id"] not in forgotten_author_numbers)

I was thinking about this too - am open to arguments either way for now, but we'll almost certainly want to have our own indexing for each level of hierarchy when we start generating our own data in #2


'load 'neo4j/tasks/neo4j_server.rake'' in the Rakefile doesn't work

The documentation says to add 'load 'neo4j/tasks/neo4j_server.rake' to the Rakefile to get the neo4j tasks. This does not work, as rake reports the following error:
LoadError: cannot load such file -- neo4j/tasks/neo4j_server.rake

FYI you should only do that if you're not using Rails.
If you're not using Rails, how are you including the gem?  That error makes me thing that the neo4j-core gem isn't in the load path.  Also, what version are you using?

Oops. Adding the appropriate line 'require neo4j-core' to my Rakefile fixed this one. Chalk this up to me still learning about Ruby / Rake.

Glad that it's fixed, though one thing I would also suggest is, if you're using bundler (that is, if you have a Gemfile) you should perhaps run bundle exec rake instead of just rake.  Bundle does all of the requires for you and makes sure that you're running in the right environment


[Enhancement] create_emr_cluster failed  version  > 6.8.0

When you try to instantiate a new cluster with
 wr.emr.create_cluster(subnet_id=subnet_id,
                                       cluster_name=cluster_name,
                                       emr_release='emr-6.9.0')

It will failed with the following error:
botocore.exceptions.ClientError: An error occurred (ValidationException) when calling the RunJobFlow operation: Classification 'spark-log4j' is not valid for parent classification 'null'.
It is because the configurations loaded by default is not valid anymore for version > 6.7.x
https://github.com/aws/aws-sdk-pandas/blob/a279ce9e538905bd22c66031d657c762218e5b38/awswrangler/emr.py#L157-166
A dynamic configurations loading should be implemented in order to avoid the error depicted above.
A potential quick solution could be:
  # Configurations
  args["Configurations"] = (
      [
          {
              "Classification": "spark-log4j",
              "Properties": {"log4j.rootCategory": f"{pars['spark_log_level']}, console"},
          }
      ]
      if not pars["configurations"] and pars["emr_release"] in (supported_versions)
      [
          {
              "Classification": "spark-log4j2",
              "Properties": {"log4j.rootCategory": f"{pars['spark_log_level']}, console"},
          }
      ]
      elif  not pars["configurations"] and pars["emr_release"] in (supported_versions)
      else pars["configurations"]
  )

I will create a pull request when I have time to work on it

Hi @LLejoly, making sure this is not a duplicate of #1938 which should have been fixed in #2065. Could it be that you are not using the most recent version of the library?


JsSymbol

Rendered: https://github.com/chrisbajorin/rfcs/blob/symbol-primitive/text/0000-symbol-primitive.md

@kjvalencik  In regards to Eq, are you just looking for a mention that strict equality should hold for all primitive values, or is this something that fits within the reference-level explanation? I see Eq currently has a blanket implementation on Handle<T>, so I'm not quite sure how this fits in code-wise.

Updated to reflect our discussion in https://github.com/neon-bindings/neon/pull/761

@chrisbajorin This looks really great! I'm going to put it into final comment period! Thanks for your contributions!

BTW, I just wanted to note that the description() method can be technically thought of as a convenience method, since you can implement it with other primitives, i.e. the Neon equivalent of:
const getter = Object.getOwnPropertyDescriptor(Symbol.prototype, 'description').get;
getter.call(sym)

Not a crucial point but it makes me feel confident that the description() method is a nice convenience to have and doesn't fundamentally add any complexity.


Where are the examples?

The godoc pages refer to examples, such as counter-go. The links point to github but basically lead nowhere. Where are they found? Thank you.

Right, the old examples are gone and documentation should be updated. I believe the new examples have to be written, in less quantity but higher educational quality. PRs are welcome ;)

Thanks. I'm unqualified to help, but look forward to it.


Depreciate shop_deployment_names

shop_deployment_names has been more or less replaced by shop_domains.  Depreciate and remove the former as it's just redundant data.

Also, I'm not sure why we need to store ipfsPinner, ipfsGateway and ipfsHash in shop_domains since that is already stored in shop_deployments? Why no just storing a deployment_id in shop_domains?

shop.hostname may also no longer be needed?  May as well be reviewed as part of this.


Improved theme and UI

Changed the whole theme of website to teal color. I have also improved css.
Before:

After:

Can you accept this pull request


Is there a plan to support the sip protocol?

Expected behavior
Actual behavior
Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
Netty version
JVM version (e.g. java -version)
OS version (e.g. uname -a)

@lixuanqun Maybe you will interesetd in https://github.com/aboutsip/sipstack/tree/master/sipstack-netty-codec-sip ?

Nope, but contributions are welcome!

@lixuanqun Maybe you will interesetd in https://github.com/aboutsip/sipstack/tree/master/sipstack-netty-codec-sip ?

@Shoothzj   Yeah, that's it

Like said above we are not planing to add it to Netty directly. That said it might be provided by another project that implements it as Netty codec.

Like said above we are not planing to add it to Netty directly. That said it might be provided by another project that implements it as Netty codec.

Yes, I understand.


HA Add-On Cloudflare fatal error: Failed to create tunnel

The problem
Hi,
I have a below problem to create a tunnel:
Can you please look and try to help me to resolve my issue?

Add-on: AIS Cloudflared
Use Cloudflare's AIS Tunnel to remotely connect to Home Assistant without opening any ports
Add-on version: 0.1.12
You are running the latest version of this add-on.
System: Home Assistant OS 11.1  (amd64 / qemux86-64)
Home Assistant Core: 2023.11.0
Home Assistant Supervisor: 2023.10.1
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
Log level is set to INFO
[21:00:06] WARNING:
[21:00:06] WARNING:     ⣴⣿⣿⣆
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿    ⣴⣿⣿⣆
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿    ⣴⣿⣿⣆
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿    ⠻⣿⣿⠏
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿   ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿    ⠻⣿⣿⠏
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:    ⣿⣿⣿⣿⣿⣿
[21:00:06] WARNING:     ⠻⣿⣿⠏
[21:00:06] WARNING:
[21:00:06] INFO: Checking add-on config...
[21:00:06] INFO: Checking for existing certificate...
[21:00:06] NOTICE: No certificate found
[21:00:06] INFO: Checking the subdomain...
[21:00:06] NOTICE: Please wait for subdomain check in AIS
[21:00:07] INFO: Subdomain OK
[21:00:07] WARNING: Creating tunnel: https://pascalpppp.paczka.pro/
[21:00:07] INFO: Creating new certificate...
[21:00:07] NOTICE: Please wait for the AIS certificate
[21:00:07] INFO: AIS Authentication successfull!
[21:00:07] INFO: Checking for existing certificate...
[21:00:07] INFO: Existing certificate found
[21:00:07] INFO: Deleting old tunnel...
2023-11-02T20:00:22Z WRN Your version 2023.8.2 is outdated. We recommend upgrading it to 2023.10.0
REST request failed: Get "https://api.cloudflare.com/client/v4/accounts/6a35989f5da41aa4dfe1cfbba525e913/cfd_tunnel?is_deleted=false&name=pascalpppp": net/http: request canceled (Client.Timeout exceeded while awaiting headers)
[21:00:22] INFO: Deleted old tunnel
[21:00:22] INFO: Creating new tunnel...
2023-11-02T20:00:37Z WRN Your version 2023.8.2 is outdated. We recommend upgrading it to 2023.10.0
failed to create tunnel: Create Tunnel API call failed: REST request failed: Post "https://api.cloudflare.com/client/v4/accounts/6a35989f5da41aa4dfe1cfbba525e913/cfd_tunnel": net/http: request canceled (Client.Timeout exceeded while awaiting headers)
[21:00:37] FATAL: Failed to create tunnel.
s6-rc: warning: unable to start service init-cloudflared-config: command exited 1
/run/s6/basedir/scripts/rc.init: warning: s6-rc failed to properly bring all the services up! Check your logs (in /run/uncaught-logs/current if you have in-container logging) for more information.
/run/s6/basedir/scripts/rc.init: fatal: stopping the container.

What version of Cloudflared has the issue?
2.x.x
What was the last working version of Cloudflared?
No response
What type of installation are you running?
Home Assistant OS
Add-on YAML Configuration
No response
Anything in the logs that might be useful for us?
No response
Steps to reproduce the issue
All streps are describe here:
https://ai-speaker.discourse.group/t/darmowy-tunel-z-wlasna-nazwa-subdomeny-dla-kazdego/3566/11?fbclid=IwAR1XvNYA_gbwRnGnv_8nTwxJIcy5PolPmpN9NXggS6KIBTyA_YZISlR38Iw
Additional information
No response

Hi @pascalppp
please update the add-on and try again

I checked after update

it works for me

Hi,
After upgreading is workin fine!
Thank You for help! :)


TextureAtlasLayout handles not being inserted on all entities which have a spritesheet

The system which is responsible for inserting TextureAtlasLayout handles onto enities is only doing so when asset events for the spritesheet are emitted.

Fix will be in the next release


ci: set preview pages permissions

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

v0.2.10
GitHub release

Your semantic-release bot :package::rocket:


Back merge JASMIN reader updates

This plugin adds the support for JASMIN soil moisture analysis
in LVT. JASMIN is the high resolution soil moisture analysis
over Australia produced by the Bureau of Meteorology in Australia
Resolves #549
(cherry picked from commit e7d58e01a3faa1768de180111453082fff6f0ad8)

See PR #550.

Code compiles and looks good.  I see no differences in results.  I approve.


Documentation 3.4/example generation improvements

Scope & Purpose

choosable storage engine (or agnostic)
-> required for hot backup documentation examples (RocksDB required)
logCurlRequestPlain() / logPlainResponse() for better formatting (e.g. batch requests) of @EXAMPLE_ARANGOSH_RUN in DocuBlocks

Backport of #9498 (#9478, #9489 and some more fixes) and #9509 to 3.4
Preview: https://deploy-preview-105--zealous-morse-14392b.netlify.com/docs/3.4/http/batch-request.html#executes-a-batch-request

@dothebart Looks alright to me in the generated examples over in the my docs test PR:
https://github.com/arangodb/docs/blob/514a516d36392a89e37add8b8e15162586d93fd3/3.4/generated/Examples/WalPropertiesGet_mmfiles.generated
https://github.com/arangodb/docs/blob/514a516d36392a89e37add8b8e15162586d93fd3/3.4/generated/Examples/RestWalPropertiesGet_mmfiles.generated
It's probably just the file in this PR which was generated with RocksDB. None of the .generated files here should matter to the documentation system.

would be good to have the proper files here anyway.

Regenerated examples:
https://github.com/arangodb/arangodb/pull/9557/commits/4b6ab838987306a75306d7fff13ac0e26f3e48ec#diff-c1aa492cbc71a57d7add7f1fb0de8944


lint: prohibid shadowed variables

Is there a linter for this in golangci-lint or elsewhere?
I'd have to see some examples of issues from shadowing; I tend to prefer shadowing over using contrived names.

govet is for linter to detect such things

One of cases when shadow may cause issues
func Foo() (n int, err error) {
    if true {
        err := fmt.Errorf("Invalid")
        return
    }
    return
}

ah, cool.  we're already running go vet here


Fix Insert sample event data error with module six not found

Issue:
Fail to insert sample data with ModuleNotFoundError for 'six' module
Details:
Encountered a 'six' module not found error within the google/cloud/pubsub_v1 module in event handler
Reproduction Steps:

Deploy Four Keys.
Insert sample data using the following command:

python3 data-generator/generate_data.py --vc_system=github

Stack trace
ModuleNotFoundError: No module named 'six'
at . ( /usr/local/lib/python3.10/site-packages/google/cloud/pubsub_v1/publisher/client.py:22 )
at . ( /usr/local/lib/python3.10/site-packages/google/cloud/pubsub_v1/publisher/init.py:17 )
at . ( /usr/local/lib/python3.10/site-packages/google/cloud/pubsub_v1/init.py:18 )
at . ( /app/event_handler.py:20 )
at ._call_with_frames_removed ( :241 )
at .exec_module ( :883 )
at ._load_unlocked ( :688 )
at ._find_and_load_unlocked ( :1006 )
at ._find_and_load ( :1027 )
at ._gcd_import ( :1050 )
at .import_module ( /usr/local/lib/python3.10/importlib/init.py:126 )
at .import_app ( /usr/local/lib/python3.10/site-packages/gunicorn/util.py:359 )
at .load_wsgiapp ( /usr/local/lib/python3.10/site-packages/gunicorn/app/wsgiapp.py:48 )
at .load ( /usr/local/lib/python3.10/site-packages/gunicorn/app/wsgiapp.py:58 )
at .wsgi ( /usr/local/lib/python3.10/site-packages/gunicorn/app/base.py:67 )
at .load_wsgi ( /usr/local/lib/python3.10/site-packages/gunicorn/workers/base.py:146 )
at .init_process ( /usr/local/lib/python3.10/site-packages/gunicorn/workers/base.py:134 )
at .init_process ( /usr/local/lib/python3.10/site-packages/gunicorn/workers/gthread.py:92 )
at .spawn_worker ( /usr/local/lib/python3.10/site-packages/gunicorn/arbiter.py:589 )
How to fix this error
I updated google-cloud-pubsub to 1.7.2 then the issue has resolved and succeeded to insert sample data

https://github.com/dora-team/fourkeys/issues/470

This didn't fix this issue for me.
Updated requirements.txt with the newer version (which is still ancient btw), made sure to use a commit SHA instead of latest before running cloud build to push the updated Docker image, deployed a new revision with the new image, same error.
Is this project unmaintained now? No commits since May 😭

This didn't fix this issue for me.
Updated requirements.txt with the newer version (which is still ancient btw), made sure to use a commit SHA instead of latest before running cloud build to push the updated Docker image, deployed a new revision with the new image, same error.
Is this project unmaintained now? No commits since May 😭

This may fix the issue for you - https://github.com/dora-team/fourkeys/pull/476


Implement psr/container 1.1

This PR proposes to upgrade Pimple's PSR-11 implementation with parameter type declarations from psr/container 1.1.
Since the Container class is final, I have also added a return type to the has() method as a preparation for version 2 of the interfaces. Unfortunately, the ServiceLocator class is not final, so I could not add the return type here as that would be a BC break. If I'm not mistaken, this would be the only change necessary for psr/container 2.

Thank you @derrabus.

There's a discussion happening under merge commit that this changeset break BC:

https://github.com/silexphp/Pimple/commit/15077931f9adb3f45a135af01837d7386ffd5a89#r47959463


Don't update BuildConfig for scratch builds

When submitting a scratch build, ideally we would create a stand-alone 'Build' object instead of updating any existing BuildConfig and instantiating it.
By doing this, we can avoid scratch builds from interfering with chain rebuilds.

Fixed via #452


Tiny clean-up

https://latkin.org/blog/2015/05/18/null-checking-considerations-in-f-its-harder-than-you-think/

Not quite sure why Pack is failing, I suspect using $GITHUB_RUN_ID is too high a number.


Pipeline updates and code reusability

@philippjfr I just wanted to ask or make sure that we were thinking about code reusability with the changes that you made to the Panel Pipeline this morning. I'm thinking of the grabcut app and I really like how the same code base can be implemented in the notebook or as an app. Will that still be true of the new approach?

We discussed this in a meeting, I don't think there's an issue.


this.renderer.setStyle is not a function

I am importing the module with DragAndDropModule.forRoot()
The HTML is

{{event.title}}

and the event in the component as:
draggableEvents: CalendarEvent[] = [{
   title: 'Available',
   draggable: true,
   start: new Date(),
   color: this.colours.green
 },
 {
   title: 'UnAvailable',
   draggable: true,
   start: new Date(),
   color: this.colours.red
 }
 ];

What version of angular are you using?

"@angular/animations": "^4.0.1",
"@angular/common": "^4.0.1",
"@angular/compiler": "^4.0.1",
"@angular/compiler-cli": "^4.0.1",
"@angular/core": "^4.0.1",
"@angular/forms": "^4.0.1",
"@angular/http": "^4.0.1",
"@angular/platform-browser": "^4.0.1",
"@angular/platform-browser-dynamic": "^4.0.1",
"@angular/platform-server": "^4.0.1",
"@angular/router": "^4.0.1",
"rxjs": "^5.1.0",
"typescript": "^2.2.2",
"zone.js": "^0.8.5"

Sorry - the issue has gone away now. It seem other components on the page may well have been causing issues rather than this one


Option to remove or add special characters

Currently the list of special characters is hardcoded into the function, which might generate a password that a website does not allow for. Give the option to remove or add from the special characters list.

Pushing this over to 0.4 in favor of more urgent issues in 0.3.


Fingerprint dialog

Hi !
this is a very good work from you !!! I have tested it out and the only problem is if i delete my fingerprint and start the app again a dialog will be shown up with "fingerprint aborted". Normally there shouldn´t be a window if the fingerprint is deleted.
maybe you can fix this ?

Seems that this is happening when you access the fingerprint settings, you don't even need to delete the fingerprint. I'll look into it.
Thanks!

Hi
i will try a fix now i will change in EasyFingerprint.java
if (fingerManager.isHardwareDetected()) {
to
if (fingerManager.isHardwareDetected() fingerManager.hasEnrolledFingerprints()) {
maybe this could be the fix

You're right, I wasn't taking into account if the user had enrolled fingerprints!!
The issue is that android cancels the autentication when you access the fingerprint settings menu, sending an error with id 5 (FingerprintDialogActivity - onAuthenticationError()).
This was not doing anything but showing an error in the dialog. I'm going to modify it so the dialog is closed when this error appears.
Regards and thanks for your help!


[Mystical World] Nether Ore

A possible niche for Mystical Ore would be generation of nether-based ores. Heck, even for embers. The Nether is desolate, boring place which you go to when you need glowstone or bricks.

I'm happy to leave that to a dedicated nether ores mod. Perhaps a unique ore added for the nether, but no "nether ores" mod is really needed in Mystical World


Unsupported method GET for dashboard

Since v2.1.2 I'm getting the following error when trying to access https://domain.com/ (or /dashboard):
The GET method is not supported for this route. Supported methods: HEAD. This error appears before being logged in (e.g. private tab) as well as after having logged in (/login is working).

Did you run
php artisan route:cache
```php artisan cache:clear``
Aftter updating?

Yes, all steps of the upgrade instructions in the README.md have been executed. I just executed them again but this didn't fix it. Doing a completely new installation and just copying over the sqlite-db and the .env lead to the same result.

Ok, thanks. Interesting, these arent happening with MySQL ill switch over to Sqlite and have a look

Possibly fixed now: 20058cb9c9147105e5187fdccb9ea1e9bd55bd09 ?
Needed to define the keys as a string for Sqlite

This has fixed many issues but not: The GET method is not supported for this route. Supported methods: HEAD. on route ("/" or dashboard)

Searching that error up just brings commands:
php artisan optimize
and/or
php artisan cache:clear
php artisan route:cache
php artisan config:cache
php artisan view:clear

What environment are you hosting it in?

It's a standard cPanel shared hosting. my-idlers used to work until the recent update. I just executed the optimize command but this didn't help.

Only the index / home page is effected. The pages for server, shared, resellers, settings etc. are working fine.

I noticed this issue is happening on new installations only. Upgrading my existing installation works fine. Maybe a bug at the dependencies?

@Gamerboy59 I'm really unsure, I cant reproduce it. Was thinking it was something to do with the shared hosting handling redirects weirdly.

I just found the solution. The optimization has an additional cache which needs to be wiped too. Now it's working.
php artisan optimize:clear
php artisan route:clear


[APIView] Misc Java-specific improvements

Add new diagnostic rule that checks if URLs have locale language tags in them (e.g. 'en-us').

/check-enforcer evaluate

/check-enforcer override


use  resolve.fallback  to polyfill nodejs  modules  still not  work

Other relevant information:
webpack version:  5.11.1
Node.js version: 10.15.3
Operating System: mac os 10.14.5
Additional tools:
i  upgrade webpack4  to webpack5
but i  got a lot of  nodejs polyfill error

"dependencies": {
"@tencent/AUI-cli": "^0.0.24",
"@tencent/beaconapi": "^1.2.3",
"@tencent/exeditor-basic": "^0.4.0-alpha.3",
"@tencent/exeditor-common-plugin": "^0.4.0",
"@tencent/exeditor-server-plugin": "^0.4.0",
"@tencent/luna-upload": "^0.1.5",
"@tencent/oms_ui": "^1.1.26",
"@tencent/oms_ui_reactEcharts": "^0.0.11",
"@tencent/omscomp": "0.0.1-7.7",
"acorn": "^6.0.0",
"acorn-jsx": "^5.0.1",
"antd": "^3.3.2",
"axios": "^0.16.2",
"babel-polyfill": "^6.23.0",
"browserslist": "^2.11.3",
"classnames": "^2.2.5",
"commander": "^2.15.1",
"dva": "^2.4.1",
"dva-loading": "^3.0.21",
"ejs-loader": "^0.3.0",
"history": "^4.7.2",
"js-beautify": "^1.6.14",
"jsonp": "^0.2.1",
"moment": "^2.22.2",
"nan": "^2.12.1",
"nprogress": "^0.2.0",
"numeral": "^2.0.6",
"path-to-regexp": "^1.7.0",
"prop-types": "^15.5.10",
"qs": "^6.5.0",
"query-string": "^5.0.0",
"react": "16.6.0",
"react-countup": "^2.1.1",
"react-dom": "16.6.0",
"react-helmet": "^5.1.3",
"url-loader": "^4.1.1",
"viewerjs": "^1.0.0",
"xlsx": "^0.14.1"
},
"devDependencies": {
"@babel/core": "^7.9.0",
"@babel/plugin-proposal-class-properties": "^7.8.3",
"@babel/plugin-proposal-decorators": "^7.8.3",
"@babel/plugin-proposal-object-rest-spread": "^7.9.0",
"@babel/plugin-syntax-dynamic-import": "^7.8.3",
"@babel/plugin-transform-modules-commonjs": "^7.9.0",
"@babel/plugin-transform-runtime": "^7.9.0",
"@babel/plugin-transform-spread": "^7.8.3",
"@babel/preset-env": "^7.9.0",
"@babel/preset-react": "^7.9.1",
"@babel/runtime": "^7.12.5",
"@tencent/dwt": "^3.0.3",
"@tencent/dwt-reporter": "^3.0.1",
"awesome-typescript-loader": "^5.2.1",
"babel-eslint": "^10.0.1",
"babel-jest": "^26.3.0",
"babel-loader": "^8.1.0",
"babel-plugin-add-module-exports": "^1.0.2",
"babel-plugin-dev-expression": "^0.2.1",
"babel-plugin-dva-hmr": "^0.4.2",
"babel-plugin-import": "^1.1.1",
"babel-plugin-transform-remove-console": "^6.9.4",
"babel-plugin-transform-runtime": "^6.9.0",
"babel-runtime": "^6.9.2",
"case-sensitive-paths-webpack-plugin": "^2.3.0",
"chalk": "^3.0.0",
"clean-webpack-plugin": "^3.0.0",
"constants-browserify": "^1.0.0",
"copy-webpack-plugin": "^7.0.0",
"core-js": "^3.6.4",
"cross-env": "^5.0.5",
"cross-spawn": "^6.0.5",
"css-loader": "^3.4.2",
"detect-port": "^1.3.0",
"dva-model-extend": "^0.1.2",
"enzyme": "^3.11.0",
"enzyme-adapter-react-16": "^1.15.5",
"eslint": "^5.9.0",
"eslint-config-alloy": "^1.4.2",
"eslint-import-resolver-node": "^0.3.2",
"eslint-loader": "^2.1.1",
"eslint-plugin-import": "^2.14.0",
"eslint-plugin-jsx-a11y": "^6.1.2",
"eslint-plugin-react": "^7.11.1",
"fbjs": "^0.8.16",
"gulp": "^3.9.1",
"gulp-clean": "^0.3.2",
"gulp-rename": "^1.2.2",
"gulp-replace": "^0.6.1",
"gulp-uglify": "^3.0.0",
"hard-source-webpack-plugin": "^0.13.1",
"hoist-non-react-statics": "^2.3.1",
"html-webpack-plugin": "^5.0.0-alpha.17",
"identity-obj-proxy": "^3.0.0",
"inquirer": "^5.1.0",
"jest": "^25.1.0",
"jest-canvas-mock": "^2.2.0",
"jest-html-reporter": "^3.2.0",
"jsdom": "^16.4.0",
"less-loader": "^5.0.0",
"less-vars-to-js": "^1.1.2",
"mini-css-extract-plugin": "^1.3.3",
"mockjs": "^1.0.1-beta3",
"node-fetch": "^2.6.1",
"optimize-css-assets-webpack-plugin": "^5.0.3",
"os-browserify": "^0.3.0",
"path-browserify": "^1.0.1",
"postcss-loader": "^3.0.0",
"postcss-plugin": "^1.0.0",
"react-dev-utils": "^10.2.1",
"redbox-react": "^1.2.10",
"redux-mock-store": "^1.5.4",
"run-sequence": "^2.2.1",
"script-ext-html-webpack-plugin": "^2.1.4",
"stream-browserify": "^3.0.0",
"style-loader": "^1.1.3",
"sugarss": "^1.0.1",
"terser-webpack-plugin": "^2.3.5",
"thread-loader": "^2.1.3",
"webpack": "^5.11.1",
"webpack-dev-server": "^4.0.0-beta.0",
"webpack-manifest-plugin": "^3.0.0",
"webpack-merge": "^4.2.2"
},

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us help you by recreating the issue using one of our templates.
please fill out all fields, otherwise we can't help


Four new examples and 1D silo bug fix

Please see this comment: https://github.com/MFlowCode/MFC/issues/174#issuecomment-1599685561


Fixing formatting in one sentence

One small fix in EN version.

Thank you @Koziolek, I appreciate your help!

Thank you @Koziolek, I appreciate your help!


Does the latest version of WCT work with bare import specifiers?

Just wondering if WCT supports the new bare import specifiers, I'm using<EMAIL_ADDRESS>

The prerelease has support via --module-resolution: https://github.com/Polymer/tools/blob/master/packages/web-component-tester/CHANGELOG.md#660-pre3---2018-03-21

If you install @next and run with --npm --module-resolution=node then bare specifiers should be rewritten to paths.

So: wct --npm --module-resolution=node?

So: wct --npm --module-resolution=node?

Yes. You can also set "npm:true" and "moduleResolution:node" in your wct.conf.json

Sorry, I'm getting this error: Error: wct-browser-legacy not installed. Please change --wct-package-name flag or install the package. Do I just need to do an npm install of wct-browser-legacy? Is there documentation for any of this yet?


DBZ-5113 Use bson document

https://issues.redhat.com/browse/DBZ-5113
Use BsonDocument for type safety

@Naros @jpechane before I merge with main branch to resolve conflicts, could you help me kick off the unit tests for this branch? Just want to make sure the change does not break any existing features. Thanks

@y5w Hi, it is not possible to execute CI for PRs with conflict. You just need to execute the tests locally.

@y5w Hi, it is not possible to execute CI for PRs with conflict. You just need to execute the tests locally.

Got it, I merged with main branch - PTAL @jpechane

@y5w Could you please get rid of the merge commit and idally squash evertyhing into a single one?

@y5w Could you please get rid of the merge commit and idally squash evertyhing into a single one?

@jpechane sure, done

@Naros those are generated by mvn process-sources automatically

@y5w In general LGTM, thanks. I am mostly concerned about the doc id handling.

Updated to use BsonValue to handle ID, ptal @jpechane @Naros thanks!

@y5w Applied, thanks a lot!


Add back/front-channel logout arguments

It adds back/front-channel arguments to the provider to support management of these logout flows.

Thank you!


spell check: adding a word with zg adds it to the wrong dictionary

Steps to reproduce

Set the dictionary to another language, e.g. Catalan with :set spell spelllang=ca
Encounter a word not present in the dictionary, but which is correct. For example "moltíssim".
Push zg to add it to the dictionary. Reports:

Word 'moltíssim' added t...im/spell/en_gb.utf-8.add

... which the British English dictionary, specified system-wide, and not the dictionary I had set for use when starting to edit this file.
Expected behaviour
That the new word added with zg gets added instead to ~/.vim/spell/ca.utf-8.add
Version of Vim
VIM - Vi IMproved 8.1 (2018 May 18, compiled Feb 01 2022 09:16:32) Included patches: 1-2269, 3612, 3625, 3669, 3741 Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version with GTK3 GUI.
Environment
OS: Ubuntu 20.04
Terminal: gnome-terminal
Logs and stack traces
No response

When I try the reproduction steps, I get: "Word 'moltíssim' added to ~/.vim/spell/ca.utf-8.add"
I suspect you have some option set to use a different spell file. See ":help spell-load".

closing as unreproducible


Unix CI builds

I started decoupling Archive from PS/PS in https://github.com/PowerShell/PowerShell/pull/2743
To guarantee that we are running tests on Unix, we should migrate https://github.com/PowerShell/PowerShell/blob/471594c6ab97622d2d622cbb0ef057f58a46be2a/test/powershell/Modules/Microsoft.PowerShell.Archive/Pester.Commands.Cmdlets.Archive.Tests.ps1 and enable Travis CI for this project.

Fixed; module and tests updated; Travis CI enabled.


Update nf-winbase-logonuserw.md

Note: As of January 2023, it is not possible to use the LOGON32_LOGON_NEW_CREDENTIALS logon type with a Group Managed Service Account (gMSA).

Thanks for updating the docs.


Recently used docs: last accessed "undefined"

Category

[ ] Question
[x ] Bug
[ ] Enhancement

Expected or Desired Behavior
"Last accessed" field would show date logged in user opened doc
Observed Behavior
Output renders as Last accessed: undefined
Steps to Reproduce
Add recently used docs web part to a page
``
Thanks for your contribution! Sharing is caring.

Hi, I am unable to reproduce this issue. Could you try with the latest starter kit and let me know if your issue is resolved?
Also, if this issue no longer applies to you, would you mind closing it?

Thanks - no longer an issue - closing.


Add type hints

Resolves #1407
This adds a types dependency extra, requiring  types-oauthlib, types-requests, and typing-extensions.

To provide type hints to library users, you have to add py.typed marker.
https://peps.python.org/pep-0561/#packaging-type-information

Yup, I plan on doing that when type hinting (i.e. this branch / PR) is more complete.

@Harmon758 any news on this PR? I'd like to contribute to this if you don't have time. Do you think we could merge this one? Then we'll be able to make multiple PRs in chuncks to add missing types :)

This is still a draft PR.
Feel free to PR to the branch. If your PR gets merged, then it'll automatically become part of this PR, since this PR is to merge the branch into the master branch.


Things outstanding to get to an A+

Hello,
I was going through your project and you have not met all of the criteria for an A+. One of the things is really easy, and the other one is slightly more involved.
Here are the two things missing:

[x] Plots are missing titles and some labels.
[x] Upon closer look, we haven't seen evidence of the "complex method chaining" criteria and also there are no separate .Py files for any of the analysis book which is a requirement for A+ and A.

Complex method chaining

At least three advanced operations (e.g anonymous or lambda functions in assign or apply functions)
At least two method chains

At the moment, we're leaning towards downgrading your project to a B, but if you can fix these things before April 24th, we can award you your contracted grade of an A+ or A.
Of course, you are more than welcome to just accept the B, but please let us know either way what you decide.
Thanks,
Saira Furqan

Just wanted to ask a quick question,
We're working on this issue right now, we'll be done implementing the changes by tomorrow. You mentioned needing this done by the 24th, do we have the usual 6pm deadline on the 24th or can we submit until midnight? Eitherways should be fine we should have the changes implemented by noon but I just wanted to make sure about how much time we have.

We finished addressing the feedback:

For the plot titles and missing labels, my files (analysisIfaz) already had titles and labels. There were a few missing for Ananya's files we made sure we added those. I remember you mentioned having a legend for the titles, I tried adding the legends to some plots but the plots become very 'wordy' and it takes too much space. I believe having a legend included at the beginning of the analysis like we have (via markdown) is a more efficient way of conveying the information.
Added more complex method chains (5+ lambda functions each for me and Ananya). Created a seperate .py file to import Ananya's functions.

Hopefully this is enough to atleast upgrade from B to A, if not A+.

@IfazYeanathChowdhury You created.py a file but not called that anywhere in your code.

I created the .py file for importing Ananya's functions and called it for her code, for some reason for my analysis files creating a seperate .py file to import functions was crashing my entire analysis so I avoided it. However, I added a-lot more apply functions to my code though to make it more complex.


Model corruption when moving a command action with min/max_items inconsistency past a menu action

When you move a command action that is in error (particularly, has the min/max inconsistency error) "past" a menu item, the command action in error becomes a menu action - with even more errors!

Fixed in 1.4.1


DRILL-8345: Fix Eclipse compatibility in the Maven resources plugin config

DRILL-8345: Fix Eclipse compatibility in the Maven resources plugin config
Description
The Maven resources plugin config in /pom.xml makes use of the undocumented variable maven.multiModuleProjectDirectory to copy /git.properties to target/classes in every sub-module. This variable breaks the project build in Eclipse 2022-03, even when a .mvn directory is present at the project root, and was probably never meant for use by the public. Replacing it with the session.executionRootDirectory Maven variable fixes the build in Eclipse and retains the intended copying of git.properties during builds.
Documentation
N/A
Testing

Import the Drill Maven project from scratch into Eclipse 2022-03 and run a unit test
Import the Drill Maven project from scratch into IDEA 2021.2.4 and run a unit test
Delete all but the root git.properties then build the project using mvn and check that a git.properties is present in target/classes of all modules.
Test TestDynamicUDFSupport based on maven.multiModuleProjectDirectory in Eclipse (see code comment in this PR).
Run mvn license:check to test session.executionRootDirectory in the Maven license plugin.

@jnturton, there are also other places where this variable is used. Should similar changes also be made there?

@jnturton, there are also other places where this variable is used. Should similar changes also be made there?

@vvysotskyi ah, thanks I'll see if I can change the other two.

Firstly, the other occurrences of maven.multiModuleProjectDirectory cannot be changed session.executionRootDirectory without causing issues and secondly the Eclipse problems I'd run into were due to my recent version of Eclipse holding onto a very old version of m2e, probably from a previous installation long ago. After purging Eclipse completely and reinstalling I've found that there is no longer any need for pom changes to make Drill build in Eclipse.


Oddly restrictive (and outdated) requirements in Windows wheel

$ wget https://files.pythonhosted.org/packages/64/20/76af36cad6754a15f39d3bff19e09921dec72b85261e455d4edc50ebffa8/mxnet-1.7.0.post2-py2.py3-none-manylinux2014_x86_64.whl https://files.pythonhosted.org/packages/2a/4f/fc1adda6d1ceef5c9b3921e6c6d208b86b1b3ed61686a84c1b1bcc821798/mxnet-1.7.0.post2-py2.py3-none-win_amd64.whl
[...]

$ bsdtar -xOf mxnet-1.7.0.post2-py2.py3-none-manylinux2014_x86_64.whl mxnet-1.7.0.post2.dist-info/METADATA | grep '^Requires-Dist:'
Requires-Dist: numpy (<2.0.0,>1.16.0)
Requires-Dist: requests (<3,>=2.20.0)
Requires-Dist: graphviz (<0.9.0,>=0.8.1)

$ bsdtar -xOf mxnet-1.7.0.post2-py2.py3-none-win_amd64.whl mxnet-1.7.0.post2.dist-info/METADATA | grep '^Requires-Dist:'
Requires-Dist: numpy (<1.17.0,>=1.8.2)
Requires-Dist: requests (<2.19.0,>=2.18.4)
Requires-Dist: graphviz (<0.9.0,>=0.8.1)

As you can see, the requirements of the Windows wheel are very different from the Linux wheel. They have older versions (in the case of requests there's not even any overlap between supported versions), and they restrict minor versions, leaving only a very narrow range of supported versions.
This isn't a bug per se, but it seems very suspicious. Is this really intentional?

cc @yajiedesign

@leezu  seems like this issue is applicable for 1.6.0 is actual as well for Windows. Could you check it? For now this issue looks like as nightmare.

The same is actual also for 1.6.0: META-DATA from wheel mxnet-1.6.0-py2.py3-none-win_amd64.whl:
Metadata-Version: 2.1
Name: mxnet
Version: 1.6.0
Summary: MXNet is an ultra-scalable deep learning framework. This version uses openblas.
Home-page: https://github.com/apache/incubator-mxnet
Author: UNKNOWN
Author-email: UNKNOWN
License: Apache 2.0
Description-Content-Type: UNKNOWN
Requires-Dist: numpy (<1.17.0,>=1.8.2)
Requires-Dist: requests (<2.19.0,>=2.18.4)
Requires-Dist: graphviz (<0.9.0,>=0.8.1)

MXNet Python Package

The same is actual for 1.7.0: META-DATA from wheel mxnet-1.7.0-py2.py3-none-win_amd64
Metadata-Version: 2.1
Name: mxnet
Version: 1.7.0
Summary: MXNet is an ultra-scalable deep learning framework. This version uses openblas.
Home-page: https://github.com/apache/incubator-mxnet
Author: UNKNOWN
Author-email: UNKNOWN
License: Apache 2.0
Requires-Dist: numpy (<1.17.0,>=1.8.2)
Requires-Dist: requests (<2.19.0,>=2.18.4)
Requires-Dist: graphviz (<0.9.0,>=0.8.1)
MXNet Python Package

Most definitely a nightmare. This makes installing other co-dependencies, including gluoncv , difficult to do automatically. Consistent installation from pip is an essential feature of any popular deep learning framework being used across multiple platforms. We already have a large portion of code base written in mxnet and migrating all of it to something else like pytorch will be a major pain.
Is there any way to contribute an updated wheel?