django.core.exceptions.ImproperlyConfigured: Set the DATABASE_URL environment variable

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/environ/environ.py", line 273, in get_value
    value = self.ENVIRON[var]
  File "/usr/local/lib/python3.8/os.py", line 675, in __getitem__
    raise KeyError(key) from None
KeyError: 'DATABASE_URL'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/manage.py", line 31, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 328, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/commands/runserver.py", line 60, in execute
    super().execute(*args, **options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 369, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/commands/runserver.py", line 67, in handle
    if not settings.DEBUG and not settings.ALLOWED_HOSTS:
  File "/usr/local/lib/python3.8/site-packages/django/conf/__init__.py", line 76, in __getattr__
    self._setup(name)
  File "/usr/local/lib/python3.8/site-packages/django/conf/__init__.py", line 63, in _setup
    self._wrapped = Settings(settings_module)
  File "/usr/local/lib/python3.8/site-packages/django/conf/__init__.py", line 142, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/usr/local/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/app/config/settings/local.py", line 1, in <module>
    from .base import *  # noqa
  File "/app/config/settings/base.py", line 44, in <module>
    DATABASES = {"default": env.db("DATABASE_URL")}
  File "/usr/local/lib/python3.8/site-packages/environ/environ.py", line 204, in db_url
    return self.db_url_config(self.get_value(var, default=default), engine=engine)
  File "/usr/local/lib/python3.8/site-packages/environ/environ.py", line 277, in get_value
    raise ImproperlyConfigured(error_msg)
django.core.exceptions.ImproperlyConfigured: Set the DATABASE_URL environment variable

the errors happend when pycharm runserver,  how to solve this?

Hi,
Did you export your DATABASE_URL from your terminal before running your command?

@EVENFATE This may be due to the docs service. Check #2674

@Andrew-Chen-Wang
Hi, for me he does not seems to run docker, or we can't know yet from the actual logs,
as it is mentioned, the error is from the fact that environ package (at the beginning of. base.py  ) raise a key value error.
File "/usr/local/lib/python3.8/site-packages/environ/environ.py", line 273, in get_value value = self.ENVIRON[var] File "/usr/local/lib/python3.8/os.py", line 675, in getitem raise KeyError(key) from None KeyError: 'DATABASE_URL'
which also can be see at the end of the logs:
File "/app/config/settings/base.py", line 44, in
DATABASES = {"default": env.db("DATABASE_URL")}  # <==. key: value that are actually missing.
If an error regarding DOCKER should happen. It would be later, so "use export" can be a way around to run the full package, either from docker-compose or from within a python virtualenv.
Perhaps, if I remember well, base.py is calling from local.py in dev environment.
So, the first error likely to occur is DATABASE_URL, then CELERY_BROKER_URL (eventually if it was requested to be configure like so while initiating the project. and finally if there is an issue with docker you will be notified for nor running your app in Docker environment.

I was inferencing that OP was using Docker based on the traceback from this: File "/app/config/settings/local.py", line 1, in  since we set the Dockerfile dir to be /app.
The bug fix that I filed is basically what you described: DATABASE_URL couldn't find the environment variable, and then you couldn't find CELERY_BROKER_URL. If you try to create a new cookiecutter-django project using Docker, you'll see that's exactly what errors from the docs service.

Yep you are right, i agree

Going to close this as the docs service has been fixed in #2920
Feel free to reopen or create a new issue if the problem persists in a freshly generated project.

@Yugalakisora did you find any solution?

No, but I think that most likely it will be caused by restrictions to connect to the URL, because it is an external server and maybe it is not working

Holy cow - whole weekend gone on this one.... Here we go -
It turns out in your .local/.postgres file, the DATABASE_URL is not set. So add this line
DATABASE_URL=postgres://:@postgres:5432/
Some folks suggest DATABASE_URL=postgres://:@postgres:<IP_ADDRESS>/ but this did not work for me....
Once that was set, I was not able to run the standard (python manage.py createsuperuser) command. Since I'm running docker I needed (docker exec -it  python manage.py createsuperuser). You can find the container ID by running  command (docker ps). Hope this helps. Shout out to Akash Vibhute here
https://gitter.im/pydanny/cookiecutter-django?at=5eb16ed6d898fe7a3768977f

@TElphee01 Correct, it is set in entrypoint under the compose/django directory. If you want to run a command like shell, you'll need to do docker-compose -f local.yml run --rm django python manage.py shell

@Andrew-Chen-Wang  Yes sir that did work as well! Thank you for your help!

@Andrew-Chen-Wang Im getting this for one of my containers deployed on AKS. It works fine on vm but not when i deployed to AKS. I set env variable in configmap.
What would you advise i do next please

I don't know what AKS is but I assume the reason you're bumping into this error is because you're trying to run something via the docker file rather than the compose file

Apologies AKS is azure kubernetes service. Can i not run the dockerfile instead of docker compose file?

the compose file includes an entry point which the dockerfile does not include. What you could do is add entrypoint stuff to your start file. The entrypoint file just ensures all necessary services like Postgres is running

hey gusy, am facing this error please help
web_1    | Traceback (most recent call last):
web_1    |   File "/usr/lib/python3.8/site-packages/environ/environ.py", line 273, in get_value
web_1    |     value = self.ENVIRON[var]
web_1    |   File "/usr/lib/python3.8/os.py", line 675, in getitem
web_1    |     raise KeyError(key) from None
web_1    | KeyError: 'STRIPE_TEST_PUBLIC_KEY'
web_1    |
web_1    | During handling of the above exception, another exception occurred:
web_1    |
web_1    | Traceback (most recent call last):
web_1    |   File "/usr/bin/waitress-serve", line 8, in
web_1    |     sys.exit(run())
web_1    |   File "/usr/lib/python3.8/site-packages/waitress/runner.py", line 270, in run
web_1    |     app = resolve(module, obj_name)
web_1    |   File "/usr/lib/python3.8/site-packages/waitress/runner.py", line 211, in resolve
web_1    |     obj = import(module_name, fromlist=segments[:1])
web_1    |   File "/opt/webapp/sway_backend_15594/wsgi.py", line 16, in
web_1    |     application = get_wsgi_application()
web_1    |   File "/usr/lib/python3.8/site-packages/django/core/wsgi.py", line 12, in get_wsgi_application
web_1    |     django.setup(set_prefix=False)
web_1    |   File "/usr/lib/python3.8/site-packages/django/init.py", line 19, in setup
web_1    |     configure_logging(settings.LOGGING_CONFIG, settings.LOGGING)
web_1    |   File "/usr/lib/python3.8/site-packages/django/conf/init.py", line 79, in getattr
web_1    |     self._setup(name)
web_1    |   File "/usr/lib/python3.8/site-packages/django/conf/init.py", line 66, in _setup
web_1    |     self._wrapped = Settings(settings_module)
web_1    |   File "/usr/lib/python3.8/site-packages/django/conf/init.py", line 157, in init
web_1    |     mod = importlib.import_module(self.SETTINGS_MODULE)
web_1    |   File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
web_1    |     return _bootstrap._gcd_import(name[level:], package, level)
web_1    |   File "/opt/webapp/sway_backend_15594/settings.py", line 293, in
web_1    |     STRIPE_TEST_PUBLIC_KEY = env('STRIPE_TEST_PUBLIC_KEY', 'pk_test_51GtDNLH3O0A61qqEhYxXEKeOf9R7nGgn6BerrgCqoQWNp1m5iyoeMGpn32uSYcV7lcrwdAoW2n3kNNn56QVt2PYm00ZwiVL119')
web_1    |   File "/usr/lib/python3.8/site-packages/environ/environ.py", line 123, in call
web_1    |     return self.get_value(var, cast=cast, default=default, parse_default=parse_default)
web_1    |   File "/usr/lib/python3.8/site-packages/environ/environ.py", line 277, in get_value
web_1    |     raise ImproperlyConfigured(error_msg)
web_1    | django.core.exceptions.ImproperlyConfigured: Set the STRIPE_TEST_PUBLIC_KEY environment variable
backend_web_1 exited with code 1

You are setting STRIPE_TEST_PUBLIC_KEY without the default keyword for env.

here is my env
DEBUG=1
HOST=localhost
PORT=8000
DATABASE_URL=postgres://postgres:<postgres_pwd>@postgres:5432/postgres
REDIS_URL=redis://redis:6379
SECRET_KEY=<random_string_goes_here>

@habenhkt are you aware that you are asking a question in a closed Issue? Besides that, the issue section of github is for issues concerning the project. Your problem is more suitable for in QA site like stackoverflow or the discussion section.
Regarding your question, check the env() method, it is missing the keyword default (settings.py, line 293).

@TElphee01 Correct, it is set in entrypoint under the compose/django directory. If you want to run a command like shell, you'll need to do docker-compose -f local.yml run --rm django python manage.py shell

this works for me. It feels unintuitive, I think many ppl expect to be able to open a shell on the webserver container and run django management commands.


VerifyDirectory doesn't play well with file names starting with dot

Problem
VerifyDirectory has 2 issues with file names starting with dot (e.g. .editorconfig):

When file starting with dot is in the root of directory to be verified, verifier gets an exception (since file name is resolved to empty string which is disallowed):

System.ArgumentNullException
Value cannot be null. (Parameter 'name')
   at Guard.AgainstEmpty(String value, String argumentName) in C:\src\Verify\src\Verify\Guard.cs:line 108
   at VerifyTests.Target..ctor(String extension, StringBuilder data, String name) in C:\src\Verify\src\Verify\Target.cs:line 57
   at InnerVerifier.ToTargets(String path, Func`2 include, IEnumerable`1 enumerateFiles, Object info, FileScrubber fileScrubber)+MoveNext() in C:\src\Verify\src\Verify\Verifier\InnerVerifier_Directory.cs:line 104
   at InnerVerifier.ToTargets(String path, Func`2 include, IEnumerable`1 enumerateFiles, Object info, FileScrubber fileScrubber)+System.Threading.Tasks.Sources.IValueTaskSource<System.Boolean>.GetResult()
   at Extensions.ToList[T](IAsyncEnumerable`1 target) in C:\src\Verify\src\Verify\Extensions.cs:line 6
   at Extensions.ToList[T](IAsyncEnumerable`1 target) in C:\src\Verify\src\Verify\Extensions.cs:line 6
   at InnerVerifier.VerifyDirectory(String path, Func`2 include, String pattern, EnumerationOptions option, Object info, FileScrubber fileScrubber) in C:\src\Verify\src\Verify\Verifier\InnerVerifier_Directory.cs:line 20
   at VerifyXunit.Verifier.<>c__DisplayClass6_0.<<Verify>b__0>d.MoveNext() in C:\src\Verify\src\Verify.Xunit\Verifier.cs:line 64

When the file starting with dot is in subfolder, verifier sees the verification file as an extra file and reports it as file to be deleted - this happens only on unix based OS. Repro: https://github.com/VerifyTests/Verify/pull/698

Root cause
For the first case it's inability to work with empty string as a name. Can be solved by using null instead of empty string - code will then fallback to NameOrTarget.
For the second case - I'm not sure. I'm blocked by the WSL debugging issue https://github.com/VerifyTests/Verify/issues/654 (that's caused by the fact that 'CallerFilePathAttribute' returns paths in windows FS (as that's where binaries were built) - so Path utilities and locating of snapshot files fails to work properly) - so I haven't got a chance to debug this properly (I'd need to setup full build env and remote debugger in my WSL - for which I'll need some extra time).
Possible solution
Both issues can be likely fixed by using a dummy name (e.g. 'trarget') as a filename in case the name resolves to empty string or directory name within the InnerVerifier.ToTargets (with name for nested file would still need to contain the relative dir as today - e.g. nested_dir\target).

This was an interesting one. Rootcause is in subtle different ways of enumerating files.
Snapshot files are enumerated via EnumerateFiles using SearchOptions (https://github.com/VerifyTests/Verify/blob/main/src/Verify/IoHelpers.cs#L33)
The files to be verified are enumerated via EnumerateFiles using EnumerationOptions (https://github.com/VerifyTests/Verify/blob/main/src/Verify/Verifier/InnerVerifier_Directory.cs#L18)
The Directory.EnumerateFiles with SearchOptions are enriched (in runtime) with few extra flags (https://github.com/dotnet/runtime/blob/main/src/libraries/System.Private.CoreLib/src/System/IO/EnumerationOptions.cs#L24) - specifically the AttributesToSkip are cleared.
This means that the caller of VerifyDirectory can workaround this issue via passing the EnumerationOptions explicitly. Though the discrepancy in enumerating still feels as a bug - I'll stitch together a fix proposal.
In order to debug more comfortably under WSL I put together simple mapping of paths - I'll post a draft of that as well (it just needs to be called from all usages of CallerFilePathAttribute - which there are many - so In a first version I'll post just a sample if the idea itself looks OK).

so as you pointed out there were a few moving pieces here.
i went with prefixing with nofilename. thoughts?
i deployed a 18.3.1-beta.1 if you want to play with it

Using nofilename serves the purpose and resolves the issue. Analogous to noextension.
It would be nice if original filename could be preserved (regardles of filename or extension) - but it's definitely not a blocker and current solution is clean and simple from code perspective.
I'll bump our reference. Thanks!

It would be nice if original filename could be preserved

hmmm. i will see how difficult that is


Samsung Smart TV navbar not showing

I don't have a Samsung Smart TV but I got some reports from users that the navbar doesn't show on their TV. (I assume it is the Tizen browser that is used on Samsung TV's but no idea about version numbers or anything)
When sending them to https://getbootstrap.com/docs/4.1/examples/ they don't see the navbar there either.
Unfortunately, this is pretty much all I know. Perhaps there is someone with a Samsung TV that can confirm that this issue exists?

Samsung since 2012 uses webkit engine, but imo these solutions are often outdated. And Bootstrap supports only up-to-date browsers. So, I guess this will be just wontfix.

we'd need some more specific information i'm afraid...at the very least, what model of TV they're using, if they have turned on any special settings (some have those "pretend to be a mobile device" kind of switches, where the browser masquerades as iPad/Safari), etc...

I tried on a 2014 Samsung Smart TV and the navbar did show up albeit broken.
https://developer.samsung.com/tv/develop/specifications/web-engine-specifications#web-engine-version
/CC @mdo

Don't think Samsung TVs are really well supported here. Maybe newer TVs might be?

Unfortunately, the users that reported this are not very tech savvy so I cannot get much information out of them.
From what I understand, this page https://getbootstrap.com/docs/4.1/examples/carousel/ shows the navbar itself but only the search field and button are showing. The rest isn't.
Perhaps @XhmikosR could confirm?
As far as I can tell, everything else is normally working/supported. There just seems to be the issue with the navbar where items aren't showing.

Well, I guess it's a hit and miss without at least the model year or number... Because chances are the TVs are old, but without any real info I'm not sure how can one debug it.
I know from the TV I tried it's from 2014 so flexbox isn't there thus everything looks broken. But the navbar does show up for me. I can try the carousel page tomorrow.

BTW by old I don't mean from 2000 :P But in the world we live in, the web moves very fast and the TVs are definitely not moving fast :)

I'd love to help, but I don't see us supporting rendering in TVs unless they're specifically mapped to our browser versions in .browserslistrc. Other than supporting larger displays, I think this should likely get punted to Slack/StackOverflow. Thoughts?

Well, if there is clear documentation like https://developer.samsung.com/tv/develop/specifications/web-engine-specifications#web-engine-version, then it's just Chrome behind it. For the other cases it's definitely a hit and miss :P

I dont think the Samsung TVs are well supported for this.


How to deploy a model that is already registered in the workspace?

Hello,
The tutorial teaches how to deploy the model that is in the directory ../../model-1/model. Do you have an example of how to deploy a model that is already registered in the workspace?

Regards,
Cristina

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

ID: a852f927-cbbd-634a-d81e-78359d539ee3
Version Independent ID: 0af3f631-2669-38a9-9b17-fe5e9893b711
Content: Deploy an ML model with a managed online endpoint - Azure Machine Learning
Content Source: articles/machine-learning/how-to-deploy-managed-online-endpoints.md
Service: machine-learning
Sub-service: core
GitHub Login: @rsethur
Microsoft Alias: seramasu

Thanks for the feedback! We are currently investigating and will update you shortly.

@crisansou Is this helpful? https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-and-where?tabs=azcli#connect-to-your-workspace

Hi @YutongTie-MSFT ,
Thanks for the answer, but I don't know if that is it. I will try to explain it better. I have a registered model and I would like to deploy to ACI or AKS using the managed endpoint to create multiple versions of models under a single endpoint and perform safe rollout to newer versions.
Is it possible to use the https://azuremlschemas.azureedge.net/latest/managedOnlineEndpoint.schema.json schema for this scenario or should the deployment be done only using the studio? As example shown in the link: https://docs.microsoft.com/en-us/azure/machine-learning/how-to-use-managed-online-endpoint-studio

@crisansou Is this helpful? https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-and-where?tabs=azcli#connect-to-your-workspace

@crisansou Thanks for the details, I have assigned this issue to the author @rsethur for more information.
@rsethur Sethu, could you please take a look on this issue? Thank you.

Hi @crisansou
In order to deploy a registered model with online endpoints, you have two options

Managed
AKS

In machine learning studio UI, in the Models UI page select the model you wish to deploy and the Deploy button in command bar turns gray to blue, from the drop down select 'Deploy to real-time endpointspreview  in the wizard screen you'll get an option to select the Compute TypeManagedorAzure Kubernetes Service`, here's the difference between Managed and AKS : https://docs.microsoft.com/en-us/azure/machine-learning/concept-endpoints#managed-online-endpoints-vs-aks-online-endpoints-preview
If you want to use YAML for deployment and refer to registered model, sample YAML : https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-managed-online-endpoints#define-the-endpoint-configuration
in the model section
`model: azureml:model_name:version

Hi @shivanissambare ,
Got it, thank you!

Hi @crisansou
In order to deploy a registered model with online endpoints, you have two options

Managed
AKS

In machine learning studio UI, in the Models UI page select the model you wish to deploy and the Deploy button in command bar turns gray to blue, from the drop down select Deploy to real-time endpoints (preview)
in the wizard screen you'll get an option to select the Compute Type Managed or Azure Kubernetes Service, here's the difference between Managed and AKS : https://docs.microsoft.com/en-us/azure/machine-learning/concept-endpoints#managed-online-endpoints-vs-aks-online-endpoints-preview
If you want to use YAML for deployment and refer to registered model, sample YAML : https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-managed-online-endpoints#define-the-endpoint-configuration

in the model section
model: azureml:model_name:version

in machine learning studio UI, you can also go to Endpoints Page from the left navigation bar and select +Create button, you can create an endpoint in same way as in models UI, it gives you an option to select registered model


Use generic Github API to download

So,just looking at these, aren't all the app specific version check files able to be made into a single global version handler?

Yes I thought that as well. But doesn't Ombi need to do more work then the others? I can look at consolidating them though for sure. I think this is a first good step though:-)

I don't think Ombi needs to do more. Sonarr/Radarr/Jackett don't actually pull the current version from the app do they? We're just writing the installed version to ServiceVersions.cfg which is determined by the the version downloaded using inc/app-git-latest-release-version.sh I thought? In Ombi's case that would work the same way.


Asa.exe export-collect --analyze does not update the "findings" table

Describe the bug
If you perform "Run analysis" in the GUI, the "findings" table in the sqlite database gets updated (I assume this is the table holding all severe findings and that the higher value in the "level" column, the more severe issue)
To Reproduce
Steps to reproduce the behavior:

Create a baseline: Asa.exe collect --runid BASELINE -c -p -s -r -u -F -C -l
Install some software
Run a new scan: Asa.exe collect --runid EW_INSTALLED -c -p -s -r -u -F -C -l
Analyze the result between the two scans: Asa.exe export-collect --firstrunid BASELINE --secondrunid EW_INSTALLED --analyze

Expected behavior
I would expect that the "findings" table gets updated but it hold zero rows. If the GUI command "Run analysis" does something that you can't do on the command line, this is a bug. Otherwise, please document the command line options it corresponds to.
Screenshots
System Configuration (please complete the following information):

OS Version: [e.g. Windows 10 1803]
Application Version: [e.g. <IP_ADDRESS>]
CLI or GUI: CLI

Additional Context
The CLI tool as well as the database structure is poorly described.

This is not a bug, this is by design.  The export-collect command exports a json report and does not update the database.
The findings section of the database needs to be populated in order for the Gui result paging to function.
I can take this as a feature request for 2.2.

Ok, thank you for this. So there is no way to update this table the CLI? Our company wants to fully script this.
Is there a database decsription available? Is my understranding correct the "findings" table holds all severe findings and that the higher value in the "level" column, the more severe issue?

I actually already have this mostly implemented. I'll have an update of 2.1 available today with a "SaveToDatabase" option.

Is there a database decsription available? Is my understranding correct the "findings" table holds all severe findings and that the higher value in the "level" column, the more severe issue?

Correct.


Only provide baseUrl fix if baseUrl provided

In my scenario, the page is being served by PHP and the URL to the bundle is pointed directly to the webpack dev server.
So, the website is being served at http://my-project.test and the script is pointing to http://localhost:8080/app.js.
By specifying ?/sockjs-node when including the webpack dev server client, it is forcing the sockjs server to be on the same domain as the page. Because of https://github.com/webpack/webpack-dev-server/blob/ab4eeb007283fdf3e8950ff1f3ff8150a4812061/client-src/default/index.js#L30
I need the __resourceQuery to be empty so that it gets the URL for the sockjs server from the getCurrentScriptSource
This PR shouldn't break for anyone already using baseUrl.
Or if there is another way to achieve what I need, please let me know!
Also required by #1390 which I am working on!

Dang it, this broke my solution over here https://github.com/vuejs/vue-cli/issues/869#issuecomment-378758824

Can you explain how to use the valet driver?  I'm getting errors when I try:
08:29:41 ltrain at in ~/www/vue
$ valet which
PHP Warning:  preg_match(): Compilation failed: nothing to repeat at offset 0 in ~/.valet/Drivers/WebpackDevServerBaseDriver.php on line 107

Warning: preg_match(): Compilation failed: nothing to repeat at offset 0 in ~/.valet/Drivers/WebpackDevServerBaseDriver.php on line 107
This site is served by [BasicValetDriver].


Render transparent vector layers to an intermediate canvas

Until now, the features of transparent vector layers were rendered using the layer opacity. This caused colors to mix together and the opacities to stack up to an higher value than the expected layer opacity.
With this commit, the features are rendered at 100% opacity to an intermediate canvas which ensures colors do not mix up even in the case of features using an array of styles. The intermediate canvas is then composed to the map canvas using the layer opacity.
Transparent layers are automatically detected, non-transparent layers are not affected by the change.
Before (the orange mixes together with the white):

Now (no more mixing, exact layer opacity):

I need this fix in some project. Could you please review?

@fredj, I added a rendering test.

Will Vector layers have setOpacity() working as Tile layers ?

Like tile layers, yes.


Post request fails to write

I get this whenever I try to sign in, I think it's a permission issue but I haven't been able to resolve it yet, any help would be appreciated.
Error executing luv callback: ....local/share/nvim/lazy/plenary.nvim/lua/plenary/curl.lua:274: post leetcode.com/graphql - curl error exit_code=23 stderr={ "curl: (23) Failed writing received data to disk/application" } stack traceback:         [C]: in function 'error'         ....local/share/nvim/lazy/plenary.nvim/lua/plenary/curl.lua:274: in function '_user_on_exit'         .../.local/share/nvim/lazy/plenary.nvim/lua/plenary/job.lua:239: in function '_shutdown'         .../.local/share/nvim/lazy/plenary.nvim/lua/plenary/job.lua:46: in function <.../.local/share/nvim/lazy/plenary.nvim/lua/plenary/job.lua:37>         [C]: in function 'wait'         .../.local/share/nvim/lazy/plenary.nvim/lua/plenary/job.lua:485: in function 'wait'         .../.local/share/nvim/lazy/plenary.nvim/lua/plenary/job.lua:452: in function 'sync'         ....local/share/nvim/lazy/plenary.nvim/lua/plenary/curl.lua:292: in function 'post'         ...share/nvim/lazy/LeetBuddy.nvim/lua/leetbuddy/cookies.lua:75: in function <...share/nvim/lazy/LeetBuddy.nvim/lua/leetbuddy/cookies.lua:49>         [C]: in function 'nvim_exec2'         vim/_editor.lua: in function 'cmd'         ...are/nvim/lazy/LeetBuddy.nvim/lua/leetbuddy/questions.lua:160: in function <...are/nvim/lazy/LeetBuddy.nvim/lua/leetbuddy/questions.lua:159>

I believe the issue is due to cURL library. Try checking if it is available in your system (If not sync your distro with latest packages) and then try some basic cURL POST request using the terminal.

Sorry I got it to work, I was using Ubuntu 20.04 but after upgrading to 22.04 it started working.


Consider supporting method get as well for HttpBodyHandler

 Issue by sgjesse
Originally opened as dart-lang/sdk#10650

If a form is posted using GET instead of POST, like this

  <FORM action='/get'
   enctype='application/x-www-form-urlencoded'
   method='get'>

    What is your name? <INPUT type='text' name='submit-name'>

    <INPUT type='submit' value='Send'> <INPUT type='reset'>
  </FORM>

then the form content will be in the query part of the request URL. We should consider supporting that in HttpBodyHandler as well. Strictly speaking it will not be body handler any more then...
Btw. if the form action already contains a query part, e.g. action='/get?xxx=yyy' this will not be part of the request URL.

 Comment by skabet

If it's not part of the request URL, where is it then?

Added Accepted label.

 Comment by sgjesse

Added this to the M6 milestone.

 Comment by sgjesse

Removed this from the M7 milestone.
Added this to the Later milestone.
Removed Area-IO label.
Added Area-Pkg label.

 Comment by larsbak

Removed this from the M6 milestone.
Added this to the M7 milestone.

 Comment by skabet

Removed the owner.
Added Triaged label.

 Comment by sethladd

Is this still valid?

cc @skabet.
Removed Oldschool-Milestone-Later label.
Added Pkg-HttpServer, AssumedStale labels.

 Comment by kasperl

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.


How do I override the default query class?

I thought this should work, but it doesn't.

+1 basically I need to be able to write custom pagintion that does not encounter exponential complexity when using LIMIT and OFFSET.  See:
http://stackoverflow.com/questions/16916633/if-postgresql-count-is-always-slow-how-to-paginate-complex-queries
http://stackoverflow.com/questions/6618366/improving-offset-performance-in-postgresql
With hard-coded Query class this is badly broken for all but the most trivial applications.

Hm. Simply allowing setting a query_class parameter in __init__ and passing it down to _include_sqlalchemy (and ultimately _set_default_query_class) goes most of the way towards fixing an issue like this. However, Model would still hard code the query class as BaseQuery and doing something like...
class SQLAlchemy:
    def __init__(self, query_class=BaseQuery...):
        self.Model = self.make_declarative_base(...)
        # works but you know :/
        self.Model.query_class = query_class

example of this sort of change

Thanks @justanr, helpful example.
This issue has now been eaten by #319.


Animate WizardModal on render

Going to animate the WizardModal on render. It'll be optional. I may add some feedback too animations. Just claiming this, so there isn't duplicate work.

This is closed by https://github.com/masonitedoors/MDL/pull/93


adding docs

 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.:x: millirock sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Upgrade to latest version of Container

I don't believe withMethodCall is part of Container v1.0 (which is currently required for this component). A solution to this would be upgrading to the latest version of Container.

That method has always been part of the definitions within container. Are you having a specific problem?
Sent from my iPhone

On 6 Jun 2015, at 04:37, Michael Meyer<EMAIL_ADDRESS>wrote:
I don't believe withMethodCall is part of Container v1.0 (which is currently required for this component). A solution to this would be upgrading to the latest version of Container.
—
Reply to this email directly or view it on GitHub.

Huh, it seems to just be a PhpStorm issue (withMethodCall not in the autocomplete, plus gives warning). Lesson learned: don't trust PhpStorm all of the time.
Thank you for the awesome work on this component!

Not a problem, the issue here I think comes from that fact that the withMethodCall method is not within the Container class but within a different class that is returned by the add method so an IDE may not pick up on this.


How I can extract Titles, Headers , Photos and respective article information from Newspaper?

Hi,
I have been trying to implement the Newspaper navigator model for my application. However, it is able to detect the regions like title or whole article. But I want to extract title and its below paragraphs for my usecase. How I can do that? Please help me to resolve this issue. Is their any tutorial available to guide on it?
Thanks

U can combine object detection (bigger bboxes) with pdf_parser output (bboxes for every word or line). Filter the lines/words output by the bigger boxes predicted by Vision Models. You can leverage spatial correlation (sort by width, then height) to identify words in the same line or a heading above a paragraph (heading will be one-liner, identified a bbox with bigger area than others plus height of heading < height of paragraph). Hope that helps 👯


Fix for null value write crash

Made some improvements to the code @gstuder-ona already suggested to enable the distinction between null entries and empty entries

LGTM

Thank you for the PR!
This looks good to me, too. Before merging I would like to add some documentation for that. Kafka Backup Architecture should be the best place.

Thank you @LBmaster001 for your work. I have taken your work and extended it a bit. Please have a look in #21. I will close this PR and lets discuss anything else in the new one if there is anything to discuss ;)
(Did not found a way to extend this PR – sorry)


Block peer feature

Is your feature request related to a problem? Please describe.
Nope not a problem.
I like this dashboard solution! It's very neat and well presented, thank you!
I think it would be a good idea to introduce a block feature to peers, so the user does not need to delete a peer but simply temporarily block it. Would be quite handy as the peer doesn't need to apply the configuration again on their device, but instead it's all controlled from the server/dashboard.

Hi, sorry for a super late response. I will add this to the next release suggestion #129  :) I think this is a great idea


Feature Request: Aggregates args field to models

This would be amazing if it can be done, ithe generator to generate aggregate field inside the actual query and return count,avg,sum,min,max for the underlying model
query {   findManyUser {     aggregate(       avg {        age: true       }     )   } }

you need to use this with another query aggregateUser

Off topic but since u obviously have more experience in this, if i wanted to create a aggregate field inside an that will do some aggregate function but not cause the N+1 issue. My approach extends the nexus object autogenerated by prisma-tools but in each iteration it fires a new query
In this example im trying each time i invoke findManyUser to retrieve there avg age from another model(db table) linked with a UserDetail
query findManyUser {    aggregate  }
export const UsersAvgAge = extendType({   type: 'User',   definition(t) {     t.field('aggregate', {       type: 'AggregateUser',       resolve(_parent, args, { prisma }, info) {         const select = new PrismaSelect(info).value         return prisma?.useDetail?.aggregate({           where: {             id: {               equals: _parent.id,             },           },           ...select,         })       },     })   }, })

@btodorce You need to know that our generated CRUD is a copy of Prisma Client API
If prisma findMany API has aggregate inside it so you will find in our GraphQL query
what you need here is using groupBy


Since commit 6ec399, terminal color are lighter then before

Hi there.
This commit add a terminal_color = true as a default config value, before I upgrade to this commit, my terminal looks like:

after:

I think we should consider about this, Is it need to be true as default? and may be add some doc about this new config value, because it took me some time to figure out that is was this config caused these effects.

@wenjinnn - Keeping it true as default is what I would prefer. We need to add it to the README.

Hey, I can also fix it as I mentioned in the PR by using the colors defined in the terminal extensions, which will make the colors brighter again.

Hi, thanks for reply, I just read the PR comment. Personally, I prefer neovim internal terminals to keep the same color as the normal terminal, cuz I customized terminal color like vscode by myself. but other people might like this feature, so I think nothing wrong about this, just add some documentation will be nice.

Hey, I can also fix it as I mentioned in the PR by using the colors defined in the terminal extensions, which will make the colors brighter again.
Since I think (correct me if I'm wrong) the main goal of this plugin is to provide a color scheme which replicates vscode colors as close as possible, using the colors defined in the terminal extensions would better accomplish that task. I'm happy to make that fix, add some documentation to the README and pr that @Mofiqul.

Let's go with as it is by mentioning in Readme


Migrate to PyGame 2

https://github.com/pygame/pygame/releases/tag/2.0.0

Was already usng pygame v2 DUH.
Used the SCALED display mode.


Highlight of Current Item

Particularly when I use the "Find Window" option it normally takes me ages to find the item that it highlighted due to all the different colors on the screen.  While all other colors are configurable, I don't believe this is.  It would be nice if font and bold etc could also be selected.
Possibly could the selected item be bolded or flashed to show it's location?  Maybe selected items could be highlighted with a bar in the thing area just before the first column.

Maybe selected items could be highlighted

You can change the focus and selection colors by enabling the TreeListCustomColorsEnable setting. The default will give a very dark selection color but you can customise those by changing the TreeListCustomColorText, TreeListCustomColorFocus and TreeListCustomColorSelection settings.
The format is RRGGBB and if you're unsure about color codes you can copy them from "html color tables" online such as: https://www.rapidtables.com/web/color/html-color-codes.html

I have a portable install, if I add the lines to "ProcessHacker.exe.settings.xml":
  <setting name="TreeListCustomColorsEnable">1</setting>
  <setting name="TreeListCustomColorSelection">ff00ff</setting>
  <setting name="TreeListCustomColorText">0000ff</setting>

Nothing changes.  The post at "https://wj32.org/processhacker/forums/viewtopic.php?p=9582#p9582" shows an "Options" dialog that I don't have (obsolete?).

shows an "Options" dialog that I don't have (obsolete?)

Both the settings and options window are only available in the Nightly builds:
https://processhacker.sourceforge.io/nightly.php

Ok, got to wonder why but have I set the options correctly and if so why don't they work?
I started with "1" and then added the others when I didn't notice any change.  If I need nightly builds to overcome my issue then this issue shouldn't have been closed?

Ok, got to wonder why but have I set the options correctly and if so why don't they work?

Do you mean they're not working in the nightly build or at all?

If I need nightly builds to overcome my issue then this issue shouldn't have been closed?

If the feature works in the nightly builds then the issue is closed as completed. If the issue is why haven't the nightly builds been made available then that's a different issue.

I am on the latest build of process hacker 2.39.124 (when I click "check for updates" its says it's the latest, when I go to the website that is the latest available).
The post mentioning the settings that I referred to was dated 01 Mar 2018, so the version I have must be OK.  I notice nothing if I add the settings.  I don't understand why you would add code to nightly builds only to remove it when released.  I don't want to use alpha or beta software just to have visible selections.

when I click "check for updates" its says it's the latest, when I go to the website that is the latest available

The nightly builds are a separate branch that is not available via the Check for updates unless you're using a nightly build.

I am on the latest build of process hacker 2.39.124

2.39 is LTS (Long-term-servicing) which does not get updates. Nightly builds are version 3.0 and above which get updated every other day.

I don't want to use alpha or beta software just to have visible selections.

If there is an issue with the nightly build then it's corrected within a day. The question you have to ask is if that feature doesn't work they way you expected because nobody used the nightly build and it's released into LTS then it won't be changed and you still won't be able to use it.
So you have to use the nightly builds to test your feature request and if there is an issue and you report it then it'll be fixed otherwise you'll be stuck with that issue.

Ok thanks, I see now that the "current release" is over 4 years and can't understand why your LTS isn't a good 3.0+ build but whatever...  I will use a nightly build.  Thanks for that info.

In the latest nightly, if indeed the colors are meant to be RGB, then at least "TreeListCustomColorText" doesn't work properly:

FFFFFF = white as expected
0000FF = FF = RED (incorrect, should be Blue)

Some other tests seem to confirm that the colors are not "RGB" but "BGR".
These don't seem to change anything no matter what values I choose:
ff00ff
This value turns the blue background into black (and seems to have nothing to do with focus):
1


model.destroy don't send id in request header

I use default behavior (REST) with PHP server
I defined url for my model and everything is right with create and update.
But with delete method, ID is not send in the header like for others methods
In the source code, I find :
if (!params.data && model && (method == 'create' || method == 'update')) {
  params.contentType = 'application/json';
  params.data = JSON.stringify(model.toJSON());
}

Why dont include delete method ?
if (!params.data && model && (method == 'create' || method == 'update' || method == 'delete' )) {

Yes, delete requests have no body -- the URL itself should contain the ID of the model being deleted.

The ID should be in the URL -- unless you've overridden the url() function to prevent it from being there.

Thanks a lot, I undestrand know and it's so logic !
sorry for the mistake


Add ability to use native OS frame

Enables the functionality to use native OS frame controls. If there is a reason why these were disabled, feel free to close this.
Fixes #1030.

So I went ahead and built all this out, and it works great!
The only thing that its missing is that this doesn't push the native frame to the pop out window of streams/video chats. That window still uses the original discord frame.

Good catch @OneEyeTurner! Didn't think to check that. I just pushed some new changes which should make it so that any pop-out windows have the native frame if the option is turned on. It is done in a somewhat hacky fashion (never used Electron before) so if anyone knows a better solution, feel free to suggest changes.
@rauenzi Anything you'd like me to do before this can be merged?

Closing this as there seems to be no further interest and I myself have moved over to Vencord.

There is definitely interest as there is an outstanding feature request. I just have personally been very busy IRL as well as burnt out on client modding after 7-8 years.

Alright, no worries! Take care and ping me whenever you have some more time, then I'll have a look at making sure it still works.


Scroll Dialog Paper with Keyboard

When a Dialog has scroll='paper' and the content scrolls, shouldn't it allow scrolling with the keyboard (key down/up page down/up)? Using the keyboard with scroll="body" does work. Using the keyboard with scroll="paper" does not work. This may be by design

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
Create Dialog with long content. Set scroll='paper'. On opening Dialog, set focus to Paper.
Pressing page down/arrow down should scroll the content down and vice-versa. Scrolling the paper is preferable in circumstances where we want the actions to always remain visible.
Current Behavior 😯
Keyboard does not scroll the content.
Examples 🌈
https://codesandbox.io/s/04jr7rxzvl
Context 🔦
I'm not sure if this is an accessibility issue or not. I've also confirmed the behavior is the same with Bootstrap second scrolling modal example https://getbootstrap.com/docs/4.3/components/modal/#scrolling-long-content

@JeffEmery Use the body scroll variant if you need this browser feature: https://material-ui.com/components/dialogs/#scrolling-long-content.

Thank you, yes. It does seem that if Material and Bootstrap both have this behavior it is by design. I'm curious to why it's this way. I'll post back if I get a good answer. Thanks!

@oliviertassinari Using body scroll makes the dialog look awful. Is there really no way to enable up/down/pageUp/etc, in the scrolling body of the dialog?

When using body, scrolling with the keys works, but clicking on the scrollbar closes the dialog... So this is not really an option?


pub run should not build the current package if the command belongs to another package

If you are in an application package, and invoke a command such as:
 pub run polymer:new_entry
pub will run barback and build all assets of the current package, but in reality it only needs to build the 'polymer' package in order to run that command. This results in a lot of overhead for large applications.

This issue has been moved to dart-lang/pub#1085.


Astro:self example does not work

The current example in https://docs.astro.build/en/reference/api-reference/#astroself does not work.
Reproduction: https://codesandbox.io/s/astro-self-example-3qv8f2?file=/src/components/NestedList.astro
This causes an error "unexpected return".
---
const { items } = Astro.props;
---

{items.map((item) => {
    if (Array.isArray(item)) {
      // If there is a nested data-structure we render `<Astro.self>`
      // and can pass props through with the recursive call
      return <Astro.self items={item} />;
    } else {
      return item;
    }
  })}

The only way I got it to work was by returning a fragment/element directly from the mapping and do the recursive breaking conditional check inside the fragment/element.
---
const { items } = Astro.props;
---

  {items.map((item) => (

      {Array.isArray(item) ? (
        <Astro.self items={item} />
      ) : (
        item
      )}

  ))}

Thanks @ekafyi ! This is a great reproduction, and perfect timing for me to go over some PRs/Issues with @tony-sull tomorrow! 💜
We'll follow up here, and once I confirm what should work, and what should go in the docs, then I'm happy if you want to make a PR for the docs based on your findings!

on a side note, if anyone has a proper explanation on why the original example does not work (is it a JSX thing? Vite? Astro?) i'll be keen to hear! it's a pattern that works in React-JSX.

Might be related to https://github.com/withastro/compiler/issues/394

Makes sense @delucis , once the compiler bug is fixed, if the original example works then no change to the docs necessary.

I need to make a new label for PRs like this... "Waiting until the problem magically fixes itself?"  😅

Let’s use your example for now @ekafyi! Would you be up for creating a PR?

Sure.
Question: should I only modify the en file or other languages that has the api-reference page as well?

Yay! @ekafyi ! Thanks!
Please modify only the /en/ page, and our automated system will kick in, the translators will be notified if there is an existing page already translated that needs updating.
I know it's just a code sample in this case and you could make the change, even without knowing Portuguese, but the process works, so we'll stick with it!  :)


[HOLD for payment 2024-03-11] [$500] Android - Scan - Second tap on capture button causes error to show up

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.41-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition: Camera access is allowed

Launch New Expensify app
Tap + > Request money
Go to Scan
Tap on the capture button twice

Expected Result:
The second tap on the capture button should be disabled and no error will show up
Actual Result:
The second tap on the capture button is enabled, causing error to show up
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/dc4563ff-7d0e-4157-b8e9-cca754bf634c
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017ba8b79a30323bee
Upwork Job ID:<PHONE_NUMBER>721375744
Last Price Increase: 2024-02-14
Automatic offers:

hoangzinh | Reviewer | 0

We think that this bug might be related to #wave5
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
Android - Scan - Second tap on capture button causes error to show up
What is the root cause of that problem?
In IOURequestStepScan.capturePhoto there is an async operation to capture the photo camera.current.takePhoto().then(), only after the async operation we do the navigation/error handling.
Between the time of requesting the first photo and getting the results if the user calls the capture button again, the camera is already busy and we get an error.
What changes do you think we should make in order to solve the problem?
Add a new flag capturePhotoInProgress and return early from capturePhoto if it is true. Set the variable to true when we start the function and reset it when it is done.
What alternative solutions did you explore? (Optional)

So I see these on upwork. Are they taken or finished since they have a proposal? If I apply there am I wasting connects? I can update this one for sure. You can also disable the button or remove the click handler and add it back later.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/sterlingkooshesh

Proposal
Update the button handler to disable the button to prevent further inputs
Please re-state the problem that we are trying to solve in this issue.
The button is able to bel clicked multiple times before being prevented
What is the root cause of that problem?
Not disabling the button via a state on first click
What changes do you think we should make in order to solve the problem?
Update the button handler to disable the button to prevent further inputs
On success or error, reenable it.
USe the disabled tag. This is very common practice and is well tested.

Still waiting on new proposals given @hoangzinh's comment above

@agent3bood @Krishna2323 @EnderKilledYou Thanks for proposals, everyone. I think we found the reason why it shows error but I don't think we found the root cause yet. The capture button is using Button component, which is already using useSingleExecution hook underneath => Expecting it already supported preventing double click issue
https://github.com/Expensify/App/blob/9843f6641f65d11e125c4325df1f95feae9d1259/src/components/Pressable/GenericPressable/BaseGenericPressable.tsx#L44
We need to find out why it doesn't work in this case.

The issue is because of the async function .then(), so useSingleExecution will not work here.
Let me explain
// 1. `useSingleExecution` will make sure this is called just once
capturePhoto() {
  // ... code

  // 2. this async callback function is added to the event loop to be executed in future tick/time.
  camera.current.takePhoto().then() => {
    // 4. some time later, the event loop will execute this code, it has nothing to do with `useSingleExecution`, and if the camera is busy it will throw.
  }

  //
  // ... code
 // 3. function code is completed, `useSingleExecution` job is done here.
}

Buy why are we still to click twice?
Because useSingleExecution will prevent second execution in the same "tick", but it does not work for async functions, and for that we need a lock/mutex.

@hoangzinh yes it is possible, I have checked the code, for the native singleExecution to work the action must return a Promise.
But because the onPress handler is passed on multiple levels it is not returning the value of the original.
To make it work first we do return camera.current.takePhoto(.....
Then on every child component that wraps the handler it has to return the result as well.
For example this must do
const res = onPress(event);
Accessibility.moveAccessibilityFocus(nextFocusRef);
return res;

Should I update my proposal with the new fix?

@hoangzinh proposal updates, web is not affected by this issue.

@hoangzinh updated.

Fix all components that wrap capturePhoto to return a Promise
I added two places where it need to be updated, GenericPressable.

Applied for the job on Upwork
PR will be ready on Friday.

Update
@hoangzinh
There is still work to be done, just doing return camera.current.takePhoto is not enough because there are two other async operations to be awaited.

the navigation that is done be navigateToConfirmationStep() need to be awaited.
after the navigation we also need to await end of animation transition.
Until the two are done the button needs to be disabled.

My first solution was using useWaitForNavigation() and then await CONST.ANIMATED_TRANSITION, but I want to see if there are similar patterns in the app being used.
Will need another day until I can open the PR.

@hoangzinh what I am saying is there are more async operation going on; originally I thought it was only takePhoto; am working on finding the best solution technically and for the UX as well.
What I suggest is in addition to the original proposal. On the navigateToConfirmationStep I lock any interaction with the screen once the photo is taken. The navigation will be called and transition to next step.

btw, I have sent a proposal request in Upwork and waiting for it to be accepted.

This was deployed to staging, but it seems the behavio we intended to fix is still happening per https://github.com/Expensify/App/pull/37182#issuecomment-1973388484
Does this need to be adjusted/reverted before it goes to prod?

@techievivek were you able to weigh in on this?

Will process payments this eve

@hoangzinh

the current fix still works for double-tripple taps. So in my opinion, we can deliver the current fix. But still discuss with @techievivek if it's worth handling multiple taps in the second PR here https://github.com/Expensify/App/pull/37182#issuecomment-1974018020

I think it will be great if we can fix this completely, even when you keep tapping till the page transitions.

@agent3bood are you able to make another PR for this https://github.com/Expensify/App/issues/36520#issuecomment-1990246780? Please

Yes I can do that, will open PR in a day or two.

PR is merged. Awaiting deploy to staging -> prod

Payment summary:
@agent3bood - $500 - C
@hoangzinh - $500 - C+

The upwork issue closed for inactivity, creating a new one via https://github.com/Expensify/App/issues/40222
Offers being sent from that one

Regression test looks good to me. @greg-schroeder Can we get this added? Thanks.


domain snow.rs expired

You might want to renew it. :)

Oops! Thanks, good catch. A squatter sniped it so I'll just remove it as an official domain for now and setup an alert in case anyone tries any funny business.

Did you mean to remove the domain from the Cargo.toml? The snow.rs domain still shows on the crates.io page, which doesn't look great. (Also means it's harder to get to the docs, since the domain's used for docs as well)


FileEncryption plugin is not installed.

Native: tried calling FileEncryption.encrypt, but the FileEncryption plugin is not installed.
vendor.js:56747 Install the FileEncryption plugin: 'ionic cordova plugin add cordova-safe'
plugin is install and used according to https://ionicframework.com/docs/native/file-encryption/
and nobody fixed this issue.

any change there update me as soon as possible please

after long time issue is fixed ...

I am keen to update this plugin as stated in another open issue, but I've been short of time due to to personal/work reasons. I will push a fix in the coming days and will update the thread.

Thank you i will wait for your feedback or try to solve my own.

By using file plugin now i am able to secure my files by move all file to secure storage  this.file.moveDir(this.myvediostorageDirectory,'Android/data/vedio/',this.file.dataDirectory,'vedio').then(() => {
//   console.info('file exit')
//     })
//     .catch((err) => {
//       console.info(err)
//   })

Any update on this?

nothing updated for now using Cordova file with read permission and store data into data dic

still waiting......................................

React Native is good for me in this case


an FYI - the transformation i've been building up to running across the codebase :)

https://gist.github.com/novaugust/3f7b75f922ba3b137093913ad29b9d88

It's worked terrifically. Only things I'd consider building on top of it:

rewrite foo(bar) |> baz to bar |> foo |> baz (currently does baz(foo(bar))
condense all rewrites into a single line function call if possible (i assume this would involve playing w/ meta line numbers)

That's cool!
Yup, to condense everything into a single line you'd need to traverse the expression and replace the line numbers with the line number of the leftmost node(in this case foo). If any of the line numbers is higher, the formatter will most probably format it as a vertical pipeline. Something like this:
# Postwalk
Macro.postwalk(pipeline_ast, fn quoted -> Macro.update_meta(quoted, &Keyword.put(&1, :line, line_number) end)

# Zipper
Zipper.traverse(zipper,  fn {quoted, ctx} -> {Macro.update_meta(quoted, &Keyword.put(&1, :line, line_number), ctx} end)

# i feel convinced i need fun_meta for some case but haven't found it yet!

Yup, in this case it would be a rare edge case, you'd need some code like this for it to "break":
# First comment
foo
|>
# Second comment
bar()

The second comment will be lost, so you'd probably have to move it to foo, or move foo's comments to bar and discard foo's meta. This kind of situation is always kind of tricky, but for this particular case I'd say its rare enough to not be concerned about.

collected my two transformations in this gist:
https://gist.github.com/novaugust/badfb2a1fe1c7de2d7c76b31b046dafa

Wow that's awesome! I'm glad it worked out nicely :D


HBASE-26108 add option to disable scanMetrics in TableSnapshotInputFo…

…rmat

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 37s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 43s
branch-2 passed

+1 :green_heart:
compile
0m 25s
branch-2 passed

+1 :green_heart:
shadedjars
6m 13s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 20s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 27s
the patch passed

+1 :green_heart:
compile
0m 24s
the patch passed

+1 :green_heart:
javac
0m 24s
the patch passed

+1 :green_heart:
shadedjars
6m  6s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 18s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
12m 19s
hbase-mapreduce in the patch passed.

35m 14s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3519

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 21a8c1580419 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 5ce9339972

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/testReport/

Max. process+thread count
3458 (vs. ulimit of 12500)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 28s
branch-2 passed

+1 :green_heart:
compile
0m 30s
branch-2 passed

+1 :green_heart:
shadedjars
6m 57s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 13s
the patch passed

+1 :green_heart:
compile
0m 30s
the patch passed

+1 :green_heart:
javac
0m 30s
the patch passed

+1 :green_heart:
shadedjars
6m 57s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 23s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
11m 27s
hbase-mapreduce in the patch passed.

37m 52s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3519

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 7742a8eaca00 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 5ce9339972

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/testReport/

Max. process+thread count
3748 (vs. ulimit of 12500)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 38s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 28s
branch-2 passed

+1 :green_heart:
compile
0m 58s
branch-2 passed

+1 :green_heart:
checkstyle
0m 22s
branch-2 passed

+1 :green_heart:
spotbugs
0m 53s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  2s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

+1 :green_heart:
checkstyle
0m 19s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
15m 21s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m  7s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 17s
The patch does not generate ASF License warnings.

37m 37s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3519

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux c9c9448034a1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 5ce9339972

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
96 (vs. ulimit of 12500)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3519/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Forbiden Request Issue When Create an Apps

Hi , when i am request to create an apps using  , i am geeting this error
ETHON: Libcurl initialized
ETHON: performed EASY effective_url=https://api.smooch.io/v1/apps response_code=403 return_code=ok total_time=1.8884370000000001
SmoochApi::ApiError: Forbidden
from /Users/fourtyonestudio/.rvm/gems/ruby-2.4.1/gems/smooch-api-2.0.0/lib/smooch-api/api_client.rb:65:in `call_api'
Here is my request script

api_instance = SmoochApi::AppApi.new
app_create_body = SmoochApi::AppCreate.new
begin
result = api_instance.create_app(app_create_body)
@alert = result
rescue SmoochApi::ApiError => e
@alert = "Exception when calling AppApi->create_app: #{e}"
end

and my config file

require 'jwt'
require 'smooch-api'
payload = {:scope => 'app'}
jwtHeader = {:kid => ENV['SMOOCH_KEY']}
token = JWT.encode payload, ENV['SMOOCH_SECRET'], 'HS256', jwtHeader
SmoochApi.configure do |config|
config.api_key['Authorization'] = token
config.api_key_prefix['Authorization'] = 'Bearer'
end

my environtment variable
SMOOCH_KEY=act_598934a04131babf00e8faf4
SMOOCH_SECRET=OuLhcL-2HFQIxKxXy3hp9AY_
my credential key

actually i found the solution , change the :scope to account

Hi @rahmatullah5,
Your initial comment has been edited to remove your private information.
This is a public repository and issue. Please do not share any personal KEY or SECRET.
Thanks!


[SQL] Support MySQL window function.

Table 12.26 Window Functions

[x] CUME_DIST()
[x] DENSE_RANK()
[ ] FIRST_VALUE()
[ ] LAG()
[ ] LAST_VALUE()
[ ] LEAD()
[ ] NTH_VALUE()
[ ] NTILE()
[x] PERCENT_RANK()
[x] RANK()
[ ] ROW_NUMBER()

Codecov Report
Patch coverage has no change and project coverage change: -0.11 :warning:

Comparison is base (1d72a2c) 36.12% compared to head (92100a7) 36.01%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #688      +/-   ##
==========================================
- Coverage   36.12%   36.01%   -0.11%
==========================================
  Files         214      214
  Lines       23037    23037
==========================================
- Hits         8321     8296      -25
- Misses      13790    13839      +49
+ Partials      926      902      -24

see 3 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Maybe we should add an issue to describe the progress of this work and note some subtasks, it looks like a big feature.

Maybe we should add an issue to describe the progress of this work and note some subtasks, it looks like a big feature.
@csynineyang


Rerun relationship discovery if an ambiguity has been resolved on the inverse

modelBuilder
    .Entity<A>()
    .HasOne(a => a.B)
    .WithOne();

Should be enough to configure
public class A
{
    public int Id { get; set; }
    public B B { get; set; }
}

public class B
{
    public int Id { get; set; }
    public A A1 { get; set; }
    public A A2 { get; set; }
}

Consider scenario in #18335
where it is collection navigation on both sides. And the other side uses NotMapped to ignore it and resolves ambiguity but we don't discover relationship.


API down or unknown error

I tried to run the project remotely on AWS but when tried to open the link: http://localhost:4200/training got the error: API down or unknown error. Do you know what may be the reason?

This means that the gui is not able to reach the api.
please make sure that you have the correct URL for the api in the environment.ts and environment.prod.ts files in the front-end.

I don't have your contact information

我尝试在AWS上远程运行该项目，但是当尝试打开链接时：http：// localhost：4200 / training出现错误：API关闭或未知错误。你知道可能是什么原因吗？

hello

我尝试在AWS上远程运行该项目，但是当尝试打开链接时：http：// localhost：4200 / training出现错误：API关闭或未知错误。你知道可能是什么原因吗？

i am a graduate from USTB ,i have some questions about your https://github.com/scofield1991/Lidar-Aided-ORBSLAM

I tried to run the project remotely on AWS but when tried to open the link: http://localhost:4200/training got the error: API down or unknown error. Do you know what may be the reason?

could you tell me about your contact information,eg ,your email?

Despite been sure of the ips in environment.ts and environment.prod.ts files, I'm still getting the  API down or unknown error. How can I test it better to find the problem?


es:instance-to-envelope() throws XDMP-CHILDNODEKIND error with JSON attachment

Query from functional test testInstanceGetAttachmentsJSON()
import module namespace es = 'http://marklogic.com/entity-services' at '/MarkLogic/entity-services/entity-services.xqy';
import module namespace ext = 'http://refSameDocument#Northwind-Ref-Same-Document-0.0.1' at '/conv/valid-ref-same-doc-gen.xqy';

let $sourceDocument := doc('VINET.json')
return xdmp:from-json-string(es:instance-get-attachments(ext:instance-to-envelope(ext:extract-instance-Customer( ($sourceDocument)))))

Query returns following error:
XDMP-CHILDNODEKIND: $attachment -- element nodes cannot have object node children

Our agreement was that, which this error is acceptable for instance-to-xml-envelope, the legacy function should be backward compatible and quote attachments when needed.

Thanks for adding that point.

I'll change the description --- pretty sure it's instance-to-envelope throwing the error

Fixing this is a backward incompatibility, but not with any release :)

I've got a fix for this bug.

Now that I've looked back, I need to take my fix back out.  I've put code into both envelope functions that quote attachments -- I think I may just have a little bug int he codegen for canonical xml.  Is your patched version of this module anywhere I can get it -- i want the one with instance-to-canonical xml etc etc

valid-ref-same-doc-gen.xqy.zip
attached the zip version

I took a look in your module -- I think I need a minor change, but not the change I originally made.  It looks like my intention is to always store attachments without error.  (Which is different from what we discussed yesterday.)  I'm going to shoot an email to the group about this.

come to think of it, I think that it's best if we just don't error out.
XML envelopes -- quote JSON as string, leave XML alone
JSON envelopes -- quote XML as string, leave JSON alone.

That sounds eminently sensible to me.

OK I have a fix.  Very straightforward it turns out.

Pushing to grechaw/bugfixes, but I'm not going to make a PR until I've looked at a few more of these simple bugs.

Verified the fix to be working as expected.

Shipping the issue.


Clinic Management System

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
This is a very beginner-friendly project in the CPP  that is clinic management. We will have the following functionality in this project:-
Add patient record(first name, last name, age, blood group, and many more)
Addin patient prescription
show data
Update data
Delete data

Tech Stack
C++

Can assign only 3 issues at a time.


Getter function for View::$title so that Pjax requests don't change the page title.

As already mentioned yesterday in this old issue I would prefer to have a View::getTitle() function which returns the Views $title attribute.
And that this function is used where the title of the View is accessed e.g. in class Pjax.
Easy example:
in my layout/main.php I have this code to add the application name to the page title:
<title><?= $this->title ? Html::encode(Yii::$app->name." | $this->title") : (Yii::$app->name.' | Home') ?></title>

And in the views something like:
$this->title = Yii::t('app', MyModel::label(2));
// or something else
$this->title = 'Some title';

With the code in the layout view file the application name is added to the title => Fine.
BUT on every Pjax request only the normal view title is used (so without the application name) and the page title is changed incorrectly. This is because only the $view->title is used.
(code from class Pjax)
if ($view->title !== null) {
     echo Html::tag('title', Html::encode($view->title));
}

Do you have another suggestion how this issue can be fixed?
I mean other then extend the yii\web\View and yii\widget\Pjax classes as overwrite/extend them with functions?
Which would be cumbersome because in the pjax class the title is directly echoed in pjax::init so an setting of the title before parent::init() call would work. I would have to copy and paste the complete init function and change one line $view->title to $view->getTitle() and have to check for changes on every yii update to also re-do the changes to my extended pjax class.
Of course I could change every view where i set the title and add the application name directly.
But in my opinion a getter (maybe also a setter) function for the title would be the easiest.
If the View Class has a getTitle() function (and this function would be used instead of $view->title) we could just overwrite the Class via components and extend it with e.g.
public function getTitle()
{
    if ($this->title === null) {
        return Yii::$app->name;
    }
   return  Yii::$app->name .' | '.$this->title;
}

Then the Application Name (or something else) could be easily added to page and wouldn't change after pjax requests.
This would also help if the title comes from the db (e.g. multi language) or would be different for e.g. mobile devices.

i can't understand your example or how is it changing the title.
can you make an example with copies of your layout and pjax widget?

@Faryshta pjax is designed to find <title> tag inside a response and update document's title Source
You have title in View and once you set it - you use it everywhere. As for me, current behavior is correct and clear therefore here is nothing to change in core. You can create your own Pjax class and override default getTitle() method. Thank you! :)

@SilverFire so you mean I should set the title in "every view" to get the application Name attached to it?
You also don't understand it correctly.
There isn't a getTitle() function in the Pjax class.
There is also none in the View Class!
My request was that a getTitle function is added to the View Class and that this function is used in the Pjax class init() function so this lines are changed
I only have in my /layout/main.php
<title><?= $this->title ? Html::encode(Yii::$app->name." | $this->title") : (Yii::$app->name.' | Home') ?></title>

and in my rendered view e.g. the following title and a grid with pjax enabled.
$this->title = 'test title';

So on the normal render the page title is "my application | test title" and after I sort the grid via pjax the title is only "test title".
And as i have written above the title is echoed directly in the Pjax::init so the only solution is to completely rewrite the Pjax::init function and this could cause trouble if the source is changed in future versions.

so you mean I should set the title in "every view" to get the application Name attached to it?

No, I meant that we have a $title property in View and that's the only place designed to store the page title.

There isn't a getTitle() function in the Pjax class.
There is also none in the View Class!

Wohoops, I thought we have one in the Pjax class.

My request was that a getTitle function is added to the View Class and that this function is used in the Pjax class init() function so this lines are changed

Agree on this. Feel free to create a PR

Do you have another suggestion how this issue can be fixed?

You can use View::beforeRender() method or beforeRender event to setup default page title, and then override it inside particular view.
class MyView extends View
    public function beforeRender($viewFile, $params)
    {
        $this->title = Yii::$app->name . ' | Home'; // default title
        return parent::beforeRender($viewFile, $params);
    }

Relates #662

I can see no reason for the introduction of the getter method for the view title.
Public property has enough flexibility for the manipulation and there are enough ways to setup default title value.
Around the core code we try o minimize usage of the magic getters for the better performance. View::title is just a one of such examples.

Do you have another suggestion how this issue can be fixed?

You can also setup default View::title value at the application configuration level:
return [
    'components' => [
        'view' => [
            'title' => 'My default page title'
        ],
    ],
];

@klimov-paul if I use your beforeRender method it sets the application name correctly after an pjax request. But somehow on my normal render my page title is now
application name | application name | application name | application name | application name | test page
with this code
public function beforeRender($viewFile, $params)
{
    if (!$this->title)
        $this->title = Yii::$app->name . ' | Home'; // default title
    else
        $this->title = Yii::$app->name . ' | '.$this->title; // default title
    return parent::beforeRender($viewFile, $params);
}

I guess this is because beforeRender is called several times.
I also don't think that this is the correct position.
I also doesn't want to set a default page title. I wanted to add the application name per default to the page title. And this could be depended on the current language, output (e.g. export to pdf) different.
Therefore I wanted a simple getter / setter for the now private View::$_title which i can overwrite when i extend my view.

Wait, I meant to create getTitle() in Pjax class, not in View.

@SilverFire this would not really work.
If I add the "application name" to the title in the layout/main.php  (because there is no function in the View to overwrite) I also have to extend the Pjax class to do the same change here as well.
If there is a private $_title in the View class and a getTitle, setTitle function i can just extend the View Class and have
public function getTitle()
{

    $title = parent::getTitle();
    if ($title === null) {
        return Yii::$app->name;
    }
    return Yii::$app->name .' | '.$title;
}

And it works everywhere and no changes to the Pjax init is needed.

In your case it is better to introduce your own method composeTitle(), which will add application name to the title.
I see no reason for suport such flow in the core, since regular page title have no regular pattern (like trainling site name), which is considered a better for the SEO.

I just can't see any performance, BC issues by changing the View title to a private variable and use a function. This would only enhance the flexibility and won't destroy the BC.
Your suggestion to add my own function composeTitle to my extended view would also work.
But then I (and others) would have to change every view template and change the code to use the composeTitle function.
I could also add a $_mytitle variable to my view and my functions to add the application in there (this is not the problem).
But this would also not work when I use several modules from other developers which are all using their own view templates and just use "$this->title" in their views.
If the View::$title would be private and there is a function which we can overwrite this would work just out of the box so also after Pjax requests, other modules etc.

No News about this issue?

Nope. @SilverFire is busy right now.

As @klimov-paul said, separate composeTitle() method solves the problem, however I don't mind to create additional accessors.
My personal opinion: could be done
I'd like to hear @yiisoft/core-developers opinion.

I think it is not necessary to set the title (render title tag) when enablePushState is set to false.
Pjax.php:116 if ($view->title !== null && $this->enablePushState) {

@brdsoft yes but that's totally unrelated. Please create separate issue.

As for this issue, I think current state of things is OK. If you need to add postfix for all titles you may use an event such as EVENT_END_PAGE.

Considering this issue did not get a community support and there are ways to do the same without changes in core, I'm closing this issue.

I use quick and simple solution to solve this problem.
$this->title = 'My title';
if (Yii::$app->request->isAjax) {
    $this->title .= Yii::$app->params['titleEnding'];
}

As Pjax is Ajax request, you can add your site name to the title when request isAjax.
I know this is not the best solution, but it might help for some people.


Write unpacked bags as Standard-IA, not Standard

I had a look at this, but it seems the CloudServer container we use might have a bug when you try to set a StorageClass with PutObject, so we can't test it effectively.

Not worth it, besides these objects should get deleted within 7 days anyway.


Default iOS project from flutter create lacks "+" icon in action button

Steps to Reproduce
flutter create myapp
cd myapp
flutter run

Flutter Doctor
[✓] Flutter (on Mac OS, channel master)
• Flutter at /Users/jackson/git/flutter
• Framework revision facd3e238f (33 minutes ago), engine revision fe509b0d96
[✓] Android toolchain - develop for Android devices (Android SDK 23.0.3)
• Android SDK at /Users/jackson/Library/Android/sdk/
• Platform android-N, build-tools 23.0.3
• Java(TM) SE Runtime Environment (build 1.8.0_91-b14)
[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
• XCode at /Applications/Xcode.app/Contents/Developer
• Xcode 7.3.1, Build version 7D1014
[✓] Atom - a lightweight development environment for Flutter
• flutter plugin version 0.2.3
• dartlang plugin version 0.6.31
It works fine on Android

Good catch!
When we fix this, is it something testable?

Reference tests could catch this kind of thing: https://github.com/flutter/flutter/issues/3227

This is probably a dupe of https://github.com/flutter/flutter/issues/5395

Dupe of https://github.com/flutter/flutter/issues/5410.

If you run flutter build ios && flutter install, it works fine (for me, at least). If I use flutter run, ti can't find its resources (wrong fonts are used).
If I remove the app's preferences plist after using flutter run, and then restart the app, it works fine. So everything's installed correctly, the app just can't find its resources when invoked with the arguments that flutter run uses.

I wonder if https://github.com/flutter/flutter/issues/5417 is at all related.  That was running flutter_gallery on the iOS simulator.

I think it's the same. Looks like we're unable to load assets if we're invoked with the --flx, --dart-main, and --packages arguments that flutter run uses.

I can't test this because of https://github.com/flutter/flutter/issues/5398 (flutter run hangs on project made with flutter create)

As a workaround to #5398, try opening ios/Runner.xcworkspace in Xcode. After doing that, flutter run should work for that project.

https://github.com/flutter/engine/commit/5515d3e27288db7ed1eef5b97a7c2e185881a309 fixes this issue. Needs an engine roll.

Even without that fix, I can't reproduce this.

Going to close. Please reopen if you still see the issue.


dashboard: bisect boot errors

Recently there were a couple of small changes (#3079 and #3080) that enabled the testing of patches when no explicit repro exists (but de facto we can easily reproduce the issue).
Do the same for the automatic cause bisection of boot-time bugs.
Also, this all should probably be expanded to image test-time bugs also.

Closing as a dup of https://github.com/google/syzkaller/issues/2383


Add an option to open files with exported games

Feature request
OS I use : Windows 10
Hello,
When you export a game, you cannot open files with it

When you use a file path as an argument, godot tries to load the file,  and if it is not a *.tscn or a "project.godot", it throws this error
ERROR: No loader found for resource: C:/Users/matth/Documents/Anglais/file.txt
   At: core/io/resource_loader.cpp:186
Then it quits without even loading the code
The problem is that Godot has an API to be used by command line
Usage: C:\Users\matth\Documents\Anglais\Cloze 0.2.exe [options] [path to scene or 'project.godot' file]
That is convenient to be used in the editor, but not so much for an exported project...
As the program closes, you don't have the opportunity to use OS.get_cmdline_args()
I think it might be a good thing to be able to override a little bit the command line API for exported projects.

Well, you can indeed send arguments to the game with the command line or by making a link (on windows at least).
You can get them by code with OS.get_cmdline_args()

Sadly this is not user-friendly...

This issue seems to be a duplicate of #8721 (the underlying cause is the same), so I'll close it. Thanks for reporting it :slightly_smiling_face:


Mobile > Initial Config: Propose the user to register a new tenant when no tenant is available

At start up, check the list of orgs
If none, propose to the user to scan the QR-Code to create one.
Once created, propose the user to register.

After testing, fix the following issues:

[ ] After successful tenant scan, message to register is displayed: propose instead of 'Close' and 'Yes' buttons, 'Sign In' and 'Sign Up'

When Sign In clicked: navigate to the login screen and prefill the tenant name
When Sign Up clicked: navigate to the Sign Up page (current behavior)

[ ] In Sign Up page, if I click on 'Sign In' to go back to the login screen without registering, the tenant in the login screen should have the newly created tenant set
[ ] When latest tenant is deleted in the list and you go back to login screen, both input fields user and password are still filled
(they should be empty)
[ ] Test, test, test... ;-)

Re-tested !


rush runner initial script

@bsiegel  - can you check this out? Need suggestions on cleaner way to implement in cases where

we don't pass a specific directory and want all to execute
should i pass a 3rd argument like - the command it runs (build/lint/audit) and on the basis of that run the rush commands with the respective to or from parameters?

You can handle the difference in behavior based on whether a serviceDirectory is provided or not like thus:

// if serviceDir is undefined, packageNames will be an empty array and packageDirs will be an array containing all package directories
const [packageNames, packageDirs] = getServicePackages(serviceDir);

if (!rushParams) {
  rushParams = [];
}

switch (action.toLowerCase()) {
  case 'build':
    let params = packageNames.map(p => [`--to ${p}`, `--from ${p}`]);
    run('rush', 'build', [].concat(params, rushParams)); // this both flattens the array of arrays produced above, and appends rushParams
    break;

  case 'test':
    let params = packageNames.map(p => `--from ${p}`);
    run('rush', 'test', params.concat(rushParams));
    break;

  case 'lint':
    if (serviceDir) {
      for (const dir of packageDirs) {
        // rushx doesn't accept rushParams
        run('rushx', 'lint', { cwd: dir });
      }
    } else {
      run('rush', 'lint', rushParams)
    }
    break;

  ...
}

Here's a patch that adds an install-run-rushx.js script:
diff --git a/common/scripts/install-run-rush.js b/common/scripts/install-run-rush.js
index<PHONE_NUMBER>d..81ff53a0abe 100644
--- a/common/scripts/install-run-rush.js
+++ b/common/scripts/install-run-rush.js
@@ -34,18 +34,20 @@ function getRushVersion() {
 }
 function run() {
     const [nodePath, /* Ex: /bin/node */ scriptPath, /* /repo/common/scripts/install-run-rush.js */ ...packageBinArgs /* [build, --to, myproject] */] = process.argv;
+    const scriptName = path.basename(scriptPath);
+    const bin = scriptName.toLowerCase() === 'install-run-rushx.js' ? 'rushx' : 'rush';
     if (!nodePath || !scriptPath) {
         throw new Error('Unexpected exception: could not detect node path or script path');
     }
     if (process.argv.length < 3) {
-        console.log('Usage: install-run-rush.js <command> [args...]');
-        console.log('Example: install-run-rush.js build --to myproject');
+        console.log(`Usage: ${scriptName} <command> [args...]`);
+        console.log(`Example: ${scriptName} build`);
         process.exit(1);
     }
     install_run_1.runWithErrorAndStatusCode(() => {
         const version = getRushVersion();
         console.log(`The rush.json configuration requests Rush version ${version}`);
-        return install_run_1.installAndRun(PACKAGE_NAME, version, 'rush', packageBinArgs);
+        return install_run_1.installAndRun(PACKAGE_NAME, version, bin, packageBinArgs);
     });
 }
 run();
diff --git a/common/scripts/install-run-rushx.js b/common/scripts/install-run-rushx.js
new file mode 100644
index<PHONE_NUMBER>0..ba6a8bb39b0
--- /dev/null
+++ b/common/scripts/install-run-rushx.js
@@ -0,0 +1 @@
+require("./install-run-rush");

@KarishmaGhiya I'm pretty sure we still want this script, we just need to wait for your "rushx" PR to get merged to the Rush repo before it will work. What was your reasoning behind closing this PR?

Sorry missed this comment. I accidentally deleted this branch - was cleaning up my repo

Replaced this PR (got closed by mistake) with the PR Rush runner script #4787


going to duplicate the admin module to be used as a registered user module

planning to do so , so i hope most of the things wont go wrong. will default the init process to make the user is assigned with "admin" group in the last field of the t_users . any tips for me in such a plan? thanks!

there is no such plan for the time being.