Improve safety of environment check

Currently, our service provider checks the environment and disables everything if it is production.
https://github.com/laracasts/cypress/blob/86b523e313c37e6b156131b63a90fc3a1197c5c4/src/CypressServiceProvider.php#L12-L14
This is probably fine for most situations, but one of my projects uses "prod" instead of "production". I think we should reverse this logic to only permit cypress testing in given environments, rather than prohibiting it. This would be more flexible, and would be a safer default that protects against values other than "production".
If approved, I'll put together a PR that changes the following.
- if ($this->app->environment('production')) {
+ if (in_array($this->app->environment(), explode(',', env('CYPRESS_ENVIRONMENTS')))) {
     return;
  }

This will give us more control with our environment files
CYPRESS_ENVIRONMENTS=local,testing

Unfortunately though, this is a breaking change. But given the gotcha I encountered of 'prod' != 'production' and how serious the security implications would have been, I think it is worth it. One option however to avoid the breaking change is to only apply this logic if a CYPRESS_ENVIRONMENTS value exists, and otherwise fall back to assume "production". Either way, the documentation should still be updated to improve awareness of environment names.

Note that the string 'production' is hardcoded in a few places in the Laravel framework for similar protections (e.g. not running the database seeder by mistake in production).
There is also a isProduction() method on the app instance, also checking if the env is the string 'production'.

e.g. not running the database seeder by mistake in production

Oh wow, I didn't realize this. Ok, I think my safest option is to definitely just refactor my project to always use "production". It might still be worth noting in the docs that we're using this exact name, but for now we can close this issue. Thanks @Krisell!


Windows App Sdk 1.3 is not stable and might cause fail fast in Diagnostics::GetRuntimeElement

Describe the bug
KernelBase.dll!RaiseFailFastException()
combase.dll!RoFailFastWithErrorContextInternal2(HRESULT hrError, unsigned long cStowedExceptions, _STOWED_EXCEPTION_INFORMATION_V2 * * aStowedExceptionPointers) Line 1468
	at onecore\com\combase\winrt\error\error.cpp(1468)
Microsoft.ui.xaml.dll!Diagnostics::GetRuntimeElement(struct IInspectable *,class std::shared_ptr<class Diagnostics::RuntimeObject>)
Microsoft.ui.xaml.dll!XamlDiagnostics::GiveRootsToCallback(class Microsoft::WRL::ComPtr<struct IVisualTreeServiceCallback> const &)
Microsoft.ui.xaml.dll!std::_Func_impl_no_alloc<`XamlDiagnostics::AdviseVisualTreeChange'::`2'::<lambda_1>,long>::_Do_call()
Microsoft.ui.xaml.dll!Microsoft::WRL::Details::DelegateArgTraits<long (__cdecl ABI::Microsoft::UI::Dispatching::IDispatcherQueueHandler::*)(void)>::DelegateInvokeHelper<Microsoft::WRL::Implements<Microsoft::WRL::RuntimeClassFlags<2>,ABI::Microsoft::UI::Dispatching::IDispatcherQueueHandler,Microsoft::WRL::FtmBase>,`Advising::RunOnUIThread'::`2'::<lambda_1> &,1>::Invoke()
CoreMessagingXP.dll!Microsoft::UI::Dispatching::DispatcherQueue::DeferInvokeCallback(void *)
CoreMessagingXP.dll!CFlat::SehSafe::Execute<<lambda_a81ff790741c2a62f2197c2561f5fe49>>()
CoreMessagingXP.dll!Microsoft::CoreUI::ActionCallback::ImportAdapter$(class CFlat::Box$1<struct CFlat::FunctionPointerAndUserData$1<long > > *)
CoreMessagingXP.dll!Microsoft::CoreUI::Messaging::MessageSession::Callback_InvokeDeferInvoke(class Microsoft::CoreUI::ActionCallback *,struct System::UIntPtr)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::DeferredCall::Callback_Dispatch(void)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::DeferredCallDispatcher::Callback_OnDispatch(void)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::Dispatcher::Callback_DispatchNextItem(class Microsoft::CoreUI::Dispatch::DispatchItem *)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::Dispatcher::Callback_DispatchLoop(enum Microsoft::CoreUI::Dispatch::RunnablePriorityMask)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::EventLoop::Callback_RunCoreLoop(enum Microsoft::CoreUI::Dispatch::RunMode)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::UserAdapter::DrainCoreMessagingQueue(enum Microsoft::CoreUI::Dispatch::UserAdapter$UserPriority,void * *)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::UserAdapter::OnUserDispatch(bool,enum Microsoft::CoreUI::Dispatch::UserAdapter$UserPriority,void * *)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::UserAdapter::OnUserDispatchRaw(enum Microsoft::CoreUI::Dispatch::UserAdapter$UserPriority,bool,void * *)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::UserAdapter::DoWork(struct HWND__ *,enum Microsoft::CoreUI::Dispatch::UserAdapter$UserPriority,bool)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::UserAdapter::HandleDispatchNotifyMessage(struct HWND__ *,unsigned __int64,__int64)
CoreMessagingXP.dll!Microsoft::CoreUI::Dispatch::UserAdapter::WindowProc(struct HWND__ *,unsigned int,unsigned __int64,__int64)

Error message:
Unhandled exception at 0x00007FFBFBDDE488 (Microsoft.ui.xaml.dll) in MyApp.exe: 0xC000027B: An application-internal exception has occurred (parameters: 0x0000021B6B77ACA0, 0x0000000000000002).

Steps to reproduce the bug
Unknown but Windows App Sdk 1.2 is much stable than 1.3
Expected behavior
No response
Screenshots
No response
NuGet package version
Windows App SDK 1.3.0: 1.3.230331000
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

Can you provide some repro steps on the application you are running and what was being done when this callstack was produced.  Thanks!

@bpulliam Sorry but I do not know what code will trigger that. Basically, it appears randomly.

In App Center I see much more crashes after updating Windows App SDK to 1.3 from the latest 1.2 release.
I am not sure if this is related, but I get System.ArgumentException: "Value does not fall within the expected range" with the inner exception message: "The parameter is incorrect".

In 1.3.1, we fixed the top crash cause (a bug with System Backgrounds) and there is one more top hitter that appears to be an application bug since it only appears in a single application (we've reached out to the 3rd party developer). If you are seeing a different crash from this issue (specific to Diagnostics), please file a new issue and include a repro app (preferred), a crash dump, or (at the very least) a call stack.  That will help us immensely.  Thanks!

@bpulliam Windows App Sdk 1.3 is very easy to crash, especially drag and drop, crashed the app even debugging in Visual Studio

@bpulliam I see the crashes only in App Center. I cannot reproduce it myself. But it crashes very often. The call stack is not very interesting:

@Scottj1s Now I feel being pushed from pillar to post. Our issue has nothing to do with AppCenter. It may look like that, because of the stack trace, but it is in fact an unhandled exception that I get when using WinAppSDK 1.3, but that I do not get with WinAppSDK 1.2.x.  I can pinpoint it down to having to do with some particular XAML/C# code, but I can assure you that there's nothing wrong with this code. It just regularly crashes when using the latest WinAppSDK version.

@bpulliam @Scottj1s If I had a crash dump or stack trace with more information, I would have sent it to you. I know that it is very difficult to act on issue without crash dumps and stack traces, but the least you can do is to take me seriously.

@wbokkers I'm sorry if we made you feel that way. I assure you we are taking this seriously. It is my belief that we aren't finding this because we're looking in the wrong place.  We'll keep the search going.

@wbokkers I assure you we're not ignoring your report or punting it.  Based on the screen shot of your crash, the indication is that this is AppCenter-related, even if it might be exacerbated by Windows App SDK 1.3.  I've informed the team and they are currently investigating, and have identified some recent changes in AppCenter code that might be related.  My suggestion to open an issue with them was to (hopefully) ensure better tracking of the issue and any fix.  Thanks for reporting this issue and your patience as we get it resolved.

@Scottj1s @bpulliam Thank you for your responses and thank you for looking into this. I don't understand the mechanism causing AppCenter to behave worse with a different Windows App SDK version, while the unhandled exception comes before the actual crash, but I have to trust you on that.
Maybe I was a bit too blunt in my responses (I am a Dutch guy after all). I really do appreciate all your effort on Windows App SDK.

@wbokkers Thanks Wim - appreciate your response as well.  We could be completely off the scent here, just going on the screen shot call stack.  If so, we'll course correct and take it from there.

Hi @wbokkers , I'm from the AppCenter SDK team. As you correctly noted, the appcenter file is in call stack because it catched the unhandled error and then sends the report to the AppCenter portal.
Scott is also correct, we do have some code related to WindowsAppSDK  and WinUI which needs adjustment. However, I believe this particular crash is not related to the AppCenter SDK.
Is the crash reproducing for you locally? If yes, you can try to remove AppCenter from your project and check if the app still crashes.

I don't think those issues related to AppCenter, it just catches the unhandled exception and record it, that's why it shows on the stack trace. Did WinUI team build a real app to test the WinUI? If WinUI team did, they would find thousands of bugs during development and everyone who in charge of this project will be on pins and needles.

@wbokkers , Is that a complete call stack on your picture? Can you try to check RAW (Reports -> Report -> RAW (button on the right side)) crash report if there is more information on this crash.

@DmitriyKirakosyan

Is the crash reproducing for you locally? If yes, you can try to remove AppCenter from your project and check if the app still crashes.

Unfortunately I can't reproduce this crash locally.

Is that a complete call stack on your picture? Can you try to check RAW (Reports -> Report -> RAW (button on the right side)) crash report if there is more information on this crash.

This stack is all I have. I would expect some stack trace leading to the unhandled exception, but this stack seems to be unrolled first before sending the crash report to App Center.
I also send some logging (from the unhandled exception handler) of my own to App Center (Attachments), but here the stack trace is empty as well.
The RAW reports are not very interesting:
Package: Xelion
Version: 8.2.23048.0
OS: WINDOWS
Windows: 10.0.22621
Date: 2023-05-08T12:19:55.1544174Z
Most crashes are coming from Windows 10.0.22621, but I see some crashes from other Window versions happening as well.

Windows App Sdk 1.3.2, still the application crashed even debugging in Visual Studio

@Scottj1s Yes, the System.ArgumentException issue was resolved some time ago (can't remember when exactly). However, I did not try the 1.5 release yet. I am currently on the latest stable release.

It feels like the latest stable release (1.4.5) is quite stable. Today I put our app with this SDK in production. No crashes yet (we are on 1500 users now). On the previous release 1.4.4 I still saw some crashes (0.1% of the users). Most crashes are caused by the LayoutCycleException. Let's hope this will change in future release.

On one of my computers, this issue still exists and could be reproduce nearly 100%. I have to restart the application lots of times to make it work once. Using Windows App SDK 1.6.241114003 currently.

@zhuxb711 please create a new issue for this.


helm v3.11.3 exploitable with CVE-2021-28235

As helm v3.11.3 has a dependency on etcd which is present in etcd version < v3.5.8 which have CVE-2021-28235 is helm can be exploitable with this?
Output of helm version: 3.11.3
Output of kubectl version:
Cloud Provider/Platform (AKS, GKE, Minikube etc.):

@sethiyash helm is just only a client tool.

Yeah, that's a false positive.

Helm doesn't use etcd, right now (i.e. in 3.11.3). I noticed that GitHub and Trivy don't provide a result for etcd, either. I am curious what tool presented a message about this.
Note, helm has dependencies on k8s packages and the modules those packages are in (e.g. the Kubernetes API server) will use etcd. This is not in the code path for Helm, as far as I can tell (i.e. using go mod why).

 It walks up the trees of dependencies and flags things that are in their go.mod files. It's really annoying.

https://pkg.go.dev/golang.org/x/vuln/cmd/govulncheck is extra nice. It "uses static analysis of source code or a binary's symbol table to narrow down reports to only those that could affect the application."
I do sympathize with folk who are following procedure, and reporting vulnerabilities as their tools report. But yes, the tools source-source dependency tracking methodology and reporting false positives is at best unfortunate. As it at least distracts from real problems.


Can still add or delete row when no row is selected.

Still allows for plotting of points and deleting based on the previously clicked row.

More details? What did you click

On 27 Jan 2022, at 09:25, Ed Ivimey-Cook @.***> wrote:
—

Selected a row - then deselected by clicking it again. Then press delete or add points.

Hmm weird - didnt we resolve that yesterday?

On 27 Jan 2022, at 09:27, Ed Ivimey-Cook @.***> wrote:
Selected a row - then deselected by clicking it again. Then press delete or add points.
—
Reply to this email directly, view it on GitHub https://github.com/EIvimeyCook/ShinyDigitise/issues/7#issuecomment-1023009962, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEEEI4DES472DDZOWPB3RMDUYEFZVANCNFSM5M5K3GPA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.

Yeh! Thought so! I pulled the latest version...
So it still works after you've clicked two points for a group. This is when you're about to click two points for a group...

Thin this is now solved.


Throw IllegalArgumentException when Duration.create receives invalid arguments.

Fixes #1179.  Throwing IllegalArgumentException for invalid arguments is more
consistent with the rest of the opencensus-java API.

Rebased, since CI was stuck.

Codecov Report

Merging #1198 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1198      +/-   ##
============================================
+ Coverage     81.88%   81.89%   +<.01%
+ Complexity     1230     1229       -1
============================================
  Files           192      192
  Lines          5979     5982       +3
  Branches        553      555       +2
============================================
+ Hits           4896     4899       +3
  Misses          935      935
  Partials        148      148

Impacted Files
Coverage Δ
Complexity Δ

...i/src/main/java/io/opencensus/common/Duration.java
100% <100%> (ø)
14 <9> (-1)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks!


Autograder: not telling what is wrong and the output matches the sample

Which autograder check is failing?
Run Education/autograding@v1

What is happening?
It says that my output does not match the test but I tested it on the terminal and it matches
Failed workflow run URL
https://github.com/comp426-2022-fall/a03-teresa7218/actions/runs/3476619079
What should be happening?
it should pass the test
Screenshots of your local test(s)

The autograder for a03 is not set up yet, so if your code is working as desired, you do not need to worry. I am unsure when it will be set up but in the meantime you can move to a04 if you are confident your a03 code works as intended


Incorporate a new background

To be designed by Damla.

The assets are available here courtesy of Damilico: https://www.figma.com/file/nPW7ZYZY9g7TCIudIkW37y/ANKA-RELIEF?node-id=0%3A1&t=9qGmwX07TP5eZ2Lc-1

Hi, not only bg image, also the button and learn more need to be updated right?

Hi @haticecs, yes. I don't think the PR has worked on them, reviewing from mobile at the moment. If not, we should open a new issue for incorporating the new design.

The linked pr is related to the icon change as far as I checked, not this issue. Sure we can. I am looking into this rn.

@haticecs, there's a good looking PR on this already, please skip this: https://github.com/neu-fi/turkiye-relief-dao/pull/42/files


Create a Demo for v0.8.0 (Jessie Release) v2.1.0 (Stretch Release) + OpenJDK 10

It is necessary to create a new Demo for this new Release.
The idea about about Audio Streaming is nice.

Some processes were disabled:
https://github.com/ev3dev-lang-java/ev3dev-lang-java/issues/270

Video:
https://www.youtube.com/watch?v=VbGNev6Jdl8&feature=youtu.be&t=2m50s

Added in project


Cannot access anything, how to reinstall from scratch?

Codebase
Built-in vanilla Drupal
Host OS
macOS
Docker compose file
version: "2"

services:
  mariadb:
    image: wodby/mariadb:10.1-2.3.3
    environment:
      MYSQL_ROOT_PASSWORD: password
      MYSQL_DATABASE: drupal
      MYSQL_USER: drupal
      MYSQL_PASSWORD: drupal
#    volumes:
#      - ./mariadb-init:/docker-entrypoint-initdb.d # Place init .sql file(s) here.
#      - /path/to/mariadb/data/on/host:/var/lib/mysql # I want to manage volumes manually.

  php:
# 1. Images with vanilla Drupal – wodby/drupal:[DRUPAL_VERSION]-[PHP_VERSION]-[STABILITY_TAG].
    image: wodby/drupal:8-7.1-2.4.4
#    image: wodby/drupal:8-7.0-2.4.4
#    image: wodby/drupal:7-7.1-2.4.4
#    image: wodby/drupal:7-7.0-2.4.4
#    image: wodby/drupal:7-5.6-2.4.4
#    image: wodby/drupal:6-5.6-2.4.4
#    image: wodby/drupal:6-5.3-2.4.4
# 2. Images without Drupal – wodby/drupal-php:[PHP_VERSION]-[STABILITY_TAG].
#    image: wodby/drupal-php:7.1-2.4.3
#    image: wodby/drupal-php:7.0-2.4.3
#    image: wodby/drupal-php:5.6-2.4.3
#    image: wodby/drupal-php:5.3-2.4.3
    environment:
      PHP_SENDMAIL_PATH: /usr/sbin/sendmail -t -i -S mailhog:1025
      DB_HOST: mariadb
      DB_USER: drupal
      DB_PASSWORD: drupal
      DB_NAME: drupal
      DB_DRIVER: mysql
      # PHP_XDEBUG: 1
      # PHP_XDEBUG_DEFAULT_ENABLE: 1
      # PHP_XDEBUG_REMOTE_CONNECT_BACK: 0         # This is needed to respect remote.host setting below
      # PHP_XDEBUG_REMOTE_HOST: "<IP_ADDRESS>"  # You will also need to 'sudo ifconfig lo0 alias <IP_ADDRESS>'
    volumes:
      - codebase:/var/www/html
# Options for macOS users (https://docker4drupal.readthedocs.io/en/latest/macos)
#      - codebase:/var/www/html:cached # User-guided caching
#      - docker-sync:/var/www/html # Docker-sync

  apache:
    image: wodby/drupal-apache:2.4-1.0.2
    depends_on:
      - php
    environment:
      APACHE_LOG_LEVEL: debug
      APACHE_BACKEND_HOST: php
      APACHE_SERVER_ROOT: /var/www/html/web
    volumes:
      - codebase:/var/www/html
# Options for macOS users (https://docker4drupal.readthedocs.io/en/latest/macos)
#      - codebase:/var/www/html:cached # User-guided caching
#      - docker-sync:/var/www/html # Docker-sync
    labels:
      - 'traefik.backend=apache'
      - 'traefik.port=80'
      - 'traefik.frontend.rule=Host:drupal.docker.localhost'

#  varnish:
#    image: wodby/drupal-varnish:4.1-2.1.2
#    depends_on:
#      - nginx
#    environment:
#      VARNISH_SECRET: secret
#      VARNISH_BACKEND_HOST: nginx
#      VARNISH_BACKEND_PORT: 80
#    labels:
#      - 'traefik.backend=varnish'
#      - 'traefik.port=6081'
#      - 'traefik.frontend.rule=Host:varnish.drupal.docker.localhost'

#  redis:
#    image: wodby/redis:3.2-2.1.2

#  adminer:
#    image: wodby/adminer:4.3-1.1.0
#    environment:
#      ADMINER_SALT: adminer-salt
#    labels:
#      - 'traefik.backend=adminer'
#      - 'traefik.port=9000'
#      - 'traefik.frontend.rule=Host:adminer.drupal.docker.localhost'

#  pma:
#    image: phpmyadmin/phpmyadmin
#    environment:
#      PMA_HOST: mariadb
#      PMA_USER: drupal
#      PMA_PASSWORD: drupal
#      PHP_UPLOAD_MAX_FILESIZE: 1G
#      PHP_MAX_INPUT_VARS: 1G
#    labels:
#      - 'traefik.backend=pma'
#      - 'traefik.port=80'
#      - 'traefik.frontend.rule=Host:pma.drupal.docker.localhost'

  solr:
## wodby/drupal-solr:[DRUPAL_VERSION]-[SOLR_VERSION]-[STABILITY_TAG].
    image: wodby/drupal-solr:8-6.6-2.2.0
##    image: wodby/drupal-solr:8-6.5-2.2.0
##    image: wodby/drupal-solr:8-6.4-2.2.0
##    image: wodby/drupal-solr:8-6.3-2.2.0
##    image: wodby/drupal-solr:8-5.5-2.2.0
##    image: wodby/drupal-solr:7-5.4-2.2.0
    environment:
      SOLR_HEAP: 1024m
    labels:
      - 'traefik.backend=solr'
      - 'traefik.port=8983'
      - 'traefik.frontend.rule=Host:solr.drupal.docker.localhost'

  mailhog:
    image: mailhog/mailhog
    labels:
      - 'traefik.backend=mailhog'
      - 'traefik.port=8025'
      - 'traefik.frontend.rule=Host:mailhog.drupal.docker.localhost'

#  nodejs:
#    image: wodby/drupal-node:1.0-1.0.0
#    environment:
#       NODE_SERVICE_KEY: node-service-key
#    labels:
#      - 'traefik.backend=nodejs'
#      - 'traefik.port=8080'
#      - 'traefik.frontend.rule=Host:nodejs.drupal.docker.localhost'
#    volumes:
#      - ./path/to/your/single-page-app:/app
#    command: sh -c 'npm install && npm run start'

#  node:
#    image: node:alpine
#    working_dir: /app
#    labels:
#      - 'traefik.backend=node'
#      - 'traefik.port=3000'
#      - 'traefik.frontend.rule=Host:front.drupal.docker.localhost'
#    expose:
#      - "3000"
#    volumes:
#      - ./path/to/your/single-page-app:/app
#    command: sh -c 'npm install && npm run start'

#  memcached:
#    image: wodby/memcached:1.4-2.0.0

#  rsyslog:
#    image: wodby/rsyslog

  athenapdf:
    image: arachnysdocker/athenapdf-service
    environment:
      WEAVER_AUTH_KEY: weaver-auth-key
      WEAVER_ATHENA_CMD: "athenapdf -S"
      WEAVER_MAX_WORKERS: 10
      WEAVER_MAX_CONVERSION_QUEUE: 50
      WEAVER_WORKER_TIMEOUT: 90
      WEAVER_CONVERSION_FALLBACK: "false"

  portainer:
    image: portainer/portainer
    command: --no-auth
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    labels:
      - 'traefik.backend=portainer'
      - 'traefik.port=9000'
      - 'traefik.frontend.rule=Host:portainer.drupal.docker.localhost'

  traefik:
    image: traefik
    command: -c /dev/null --web --docker --logLevel=INFO
    ports:
      - '8000:80'
#      - '8080:8080' # Dashboard
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

volumes:
  codebase:
## Docker-sync for macOS users
#  docker-sync:
#    external: true

Trying to get this up and going. I have the latest Docker for Mac.
The first time I ran docker-compose up, everything went fine. I could access Portainer, and saw the Drupal install screen.
When I clicked "next" to install Drupal, it said "Drupal is already installed" ?? despite not installing, and then I could not access anything anymore. I got a authorization error about connecting to the database:
php_1        | [17-Aug-2017 16:05:25] WARNING: [pool www] child 39 said into stderr: "NOTICE: PHP message: Drupal\Core\Database\DatabaseAccessDeniedException: SQLSTATE[HY000] [1045] Access denied for user<EMAIL_ADDRESS>(using password: NO) in /var/www/html/web/core/lib/Drupal/Core/Database/Driver/mysql/Connection.php on line 156 #0 /var/www/html/web/core/lib/Drupal/Core/Database/Database.php(376): Drupal\Core\Database\Driver\mysql\Connection::open(Array)"
So I decided to abort and reboot, docker-compose stop, docker-compose up. At this point, portainer no longer connected "Endpoint not found" and Apache/PHP would not respond to requests.
What happened? How can I start over from scratch with the same docker-compose.yml file?

I was able to ssh into the apache service and rm the settings.php file and start over, and get Drupal installed (I did not put the right db host in).
However, Portainer still refuses to cooperate, and I do not know how to reset it.

Well, maybe this is a Chrome/cookie issue... I opened Portainer URL in an incognito window and I got the Portainer welcome screen, and was able to then see all of my D4D containers and images.
What is going on there? How can I avoid that in the future?

There's a known issue that URLs like *.docker.localhost work only in chrome for some reason. Adding *.docker.localhost domains to /etc/hosts should help.

About "Drupal is already installed", the problem seems located at the database step, the password is empty (cleared by the browser ?).
All you have to do is type it (drupal) and voila !


Plans for 1.21.2+ ArmorMaterial?

I was going to switch to Trimmed for Cinderscapes in 1.21.2+, but then I discovered it isn't available.  So, as those of us who like pain will do, I tried to port it.  However, I quickly discovered they have yeeted the entire ArmorMaterial registry and materials are now just an anonymous collection of Item Settings...
Do you plan to move ahead with a 1.21.2+ version?  I guess the likely approach would be to create an ArmorMaterial registry in Trimmed, and fake up entries for the vanilla materials, then inject the material resource location and an accessor into ArmorItem ... or maybe inject it into ArmorMaterial and inject the entire ArmorMaterial into ArmorItem.

Hmm, guess I'll make a separate Cinderscapes release for 1.21.2, then.

Sorry, I just don't find it worth supporting 1.21.2 when 1.21.3 is just a hotfix for that version. Aside from that, 1.21.3 is now released https://modrinth.com/mod/trimmed/versions.


Tracking issue for changes/fixes in latest SDL version

Fixed by https://github.com/libsdl-org/SDL/commit/6c962177272c6a2ce684e4298b40c25ebdd1ac74:

https://github.com/ppy/osu-framework/issues/4159
https://github.com/ppy/osu/issues/12228
https://github.com/ppy/osu-framework/issues/4050 (probably fixed by something else)
[ ] #5136

Fixed by https://github.com/libsdl-org/SDL/commit/5ff42438e36e98c9d72ac70f5f5ce4599b96d3d2:

https://github.com/ppy/osu-framework/issues/4055
relevant: https://github.com/ppy/osu-framework/issues/1679
SDL no longer sends SDL_WINDOWEVENT_ENTER and SDL_WINDOWEVENT_LEAVE events if any button is pressed when the mouse leaves and enters the window. This introduces a nasty bug with MouseHandler where relative mode won't be re-enabled if the mouse leaves and enters the window while holding down any buttons. This is by design.
[ ] #5163

Fixed by https://github.com/libsdl-org/SDL/pull/5398:

https://github.com/ppy/osu/issues/17231
This PR introduces a bug with our IME handling,  as SDL will now send multiple SDL_TextInputEvents when the text is longer than 32 characters. Should be easy to fix, either delay resetting of recentImeResult, or match the SDL text with our composition text using string.Contains.
[ ] https://github.com/ppy/osu-framework/pull/5141
[ ] update ResetIme to use SDL_ClearComposition on non-windows platforms.

General:

[ ] Reach out to users affected by https://github.com/ppy/osu/issues/17381 and check if it's fixed.

Things to note
https://github.com/flibitijibibo/SDL2-CS is not updated to the latest SDL, and there is no open PR on their side. We can set the hints, but we won't be able to use SDL_ClearComposition.

With the three PRs, we should be ready for an SDL bump. SDL2-CS was also recently bumped to 2.0.22. Making use of new APIs, such as SDL_ClearComposition(), can be added later.
We may want for https://github.com/libsdl-org/SDL/issues/5572 to be fixed, as noticed on discord.

I'll look to do a bump today or tomorrow if no one else beats me

@Susko3 1.0.518-alpha is now propagating to nuget. I'll leave it in your hands to consume it in the order of your choosing.


Request credentials for an os-climate bucket

Requesting credentials for github user: borisyy13
Requesting credentials for the bucket: physrisk-hazard-indicators
Email address to send a credential download link<EMAIL_ADDRESS>Request checklist:

[x ] User can establish connection with the bucket
[ x] User can read files from the bucket
[ ] User can write files to the bucket

do not need writing

@borisyy13 Can you answer this question?
If you still need this can you reply on the issue?

Closing due to lack of response.


Demo not working on Chrome or Firefox

Demo located on https://amw.github.io/jpeg_camera/demo/ is not working on Chrome and Firefox. The console throws the next error:
Uncaught DOMException: Failed to execute 'getImageData' on 'CanvasRenderingContext2D': The source width is 0.(…)

Do you know steps to reproduce this issue?

Hello ! Thanks a lot for this library !
Using Chrome, Version 56.0.2924.87 on Win10, just loading the page reproduces.
Also on Firefox 49.0.1 on Win10, we have an issue
"You have denied camera access.
Look for camera icon around your address bar to change your decision."
We are considering using this library in our webiste. Thus we would like to help fixing the issues, would a few man days for our team be helpful ?
<EMAIL_ADDRESS>

I have just tested with Chrome 56.0.2924.87 (64-bit) on macOS and it worked fine.
Did you mean that you want to commit some of your developers to investigate and fix the issue? That is always welcome.
If you meant to hire me to help you fix then I'm sorry to say that I can't commit to this at this point in time.


185995457 view data as graph on table toolbar

Fulfills this ticket:
[DataView] View Data as Graph on Table Toolbar
Deployed branch:
https://collaborative-learning.concord.org/branch/view-data-as-graph-on-table-toolbar/?appMode=demo&demoName=CLUE&fakeClass=1&fakeUser=student:1&problem=1.1&unit=example

@bgoldowsky So I got stuck writing these cypress tests, and after a long bit of head scratching I was wondering why I couldn't get the "View Data Button" to click. It turns out it actually was clicking but the url params for table_tool_spc.js  use
?appMode=qa&fakeClass=5&fakeUser=student:5&demoOffering=5&problem=2.1&qaGroup=5 which defaults to unit=sas - which does have a "Table Tile" but no "XY Plot"  in the toolbar.  Since it's missing the XY Plot, nothing happens on the screen.
We can submit a PR on Monday to Leslie to add an xy plot entry it to the clue-curriculum repo.
Meanwhile xy_plot_tool_spec goes to url : http://localhost:8080/?appMode=qa&fakeClass=5&fakeUser=student:5&qaGroup=5&unit=brain which does have both Table tile and XY Plot.
I was thinking of writing the test in there (but I'd argue it makes more sense to test the "click behavior" inside of the table tile cypress test.
Additionally, the test might also conflict with my other already added test to our other unmerged branch (https://github.com/concord-consortium/collaborative-learning/pull/1966/files) .
So I'll take it care of this Monday morning.

@dennisrcao You should add the run regression label to this PR so cypress tests run for it.


--fast-run thoroughness

Changes

Update: Lower n from 10k to 5k in some cases
Update: Add --fast-run to SNOMED builder

Test passes after merging other branches into this one. Merging this now.


Issue dependency cordova-plugin-compat

I get the problem with the following error message:
D:\barcode_scanner>cordova` plugin add cordova-plugin-compat --save
Installing "cordova-plugin-compat" for android
Plugin doesn't support this project's cordova-android version. cordova-android:
6.3.0, failed version requirement: <6.3.0
Skipping 'cordova-plugin-compat' for android
Adding cordova-plugin-compat to package.json
Saved plugin info for "cordova-plugin-compat" to config.xml

according https://github.com/apache/cordova-plugin-compat, cordova-plugin-compat is now included in cordova-android 6.3.0, no need add plugin again.
What code should I change to support for cordova-android 6.3.0?

yeah, it's included. But as it's not installed for Android you should not have problems using barcode scanner plugin.
Anyway we should remove the dependency. The code is located in plugin.xml and package.json, feel free to send a PR

still showing the same error when cordova build android.
D:\barcode_scanner>cordova plugin rm cordova-plugin-compat --save
Error: The plugin 'cordova-plugin-compat' is required by (cordova-plugin-barcode-scanner), skipping uninstallation. (try --force if trying to update)

D:\barcode_scanner>cordova plugin rm cordova-plugin-compat --save --force
The plugin 'cordova-plugin-compat' is required by (cordova-plugin-barcodescanner
) but forcing removal
Uninstalling cordova-plugin-compat from android
Removing "cordova-plugin-compat"
"cordova-plugin-compat" is required by (cordova-plugin-barcodescanner) but forcing removal.
Removing plugin cordova-plugin-compat from config.xml file...
Removing cordova-plugin-compat from package.json

D:\barcode_scanner>cordova build android
ANDROID_HOME=C:\Users\IHSANA\AppData\Local\Android\sdk\
JAVA_HOME=C:\Program Files\Java\jdk1.8.0_65\
Subproject Path: CordovaLib The Task.leftShift(Closure) method has been deprecated and is scheduled to be removed in Gradle 5.0. Please use Task.doLast(Action) instead. at build_7b85bpvs7evi3dyajqu6gxtnv.run(D:\barcode_scanner\platforms\andr
oid\build.gradle:141)
The JavaCompile.setDependencyCacheDir() method has been deprecated and is schedu
led to be removed in Gradle 4.0.
Incremental java compilation is an incubating feature.
The TaskInputs.source(Object) method has been deprecated and is scheduled to be
removed in Gradle 4.0. Please use TaskInputs.file(Object).skipWhenEmpty() instead.
[Fatal Error] :18:139: The value of the attribute "prefix="xmlns",localpart="android", rawname="xmlns:android"" is invalid. Prefixed namespace bindings may not be empty.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'android'.
> Exception while parsing the supplied manifest file D:\barcode_scanner\platform
s\android\AndroidManifest.xml

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug
option to get more log output.

BUILD FAILED

Total time: 1.504 secs
Error: cmd: Command failed with exit code 1 Error output:
[Fatal Error] :18:139: The value of the attribute "prefix="xmlns",localpart="and
roid",rawname="xmlns:android"" is invalid. Prefixed namespace bindings may not b
e empty.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'android'.
> Exception while parsing the supplied manifest file D:\barcode_scanner\platform
s\android\AndroidManifest.xml

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug
option to get more log output.

but no problem if without cordova-plugin-barcodescanner
D:\barcode_scanner>cordova plugin rm cordova-plugin-barcodescanner --save
Uninstalling cordova-plugin-barcodescanner from android
Removing "cordova-plugin-barcodescanner"
Removing plugin cordova-plugin-barcodescanner from config.xml file...
Removing cordova-plugin-barcodescanner from package.json

D:\barcode_scanner>cordova build android
ANDROID_HOME=C:\Users\IHSANA\AppData\Local\Android\sdk\
JAVA_HOME=C:\Program Files\Java\jdk1.8.0_65\
Subproject Path: CordovaLib
The Task.leftShift(Closure) method has been deprecated and is scheduled to be r
.........
BUILD SUCCESSFUL

Total time: 29.436 secs
Built the following apk(s):
        D:/barcode_scanner/platforms/android/build/outputs/apk/android-debug.ap

I've released a 7.0.0 version to resolve this issue. If you are using cordova android 6.3.0 or later you should pin your barcode scanner version to 7 or higher. For those who are still on cordova-android < 6.3.0 continue using barcode scanner ^6.0.0

solved, thank you

Not solved - see #565.
I got v6.0.8 to work with android platform v6.3.0 as explained at https://stackoverflow.com/questions/46562289/multiple-dex-files-define-lorg-apache-cordova-buildhelper/46957938#46957938:
phonegap plugin rm cordova-plugin-compat --force
phonegap plugin add<EMAIL_ADDRESS>phonegap platform rm android
phonegap platform add<EMAIL_ADDRESS>
So maybe upgrading cordova-plugin-compat to v1.2 instead of removing it might have been a better option.

@jlchereau nope, we want to get rid of compat altogether. If you are using cordova-android >= 6.3.0 use barcodescanner 7+. If you are using cordova-android < 6.3.0 use barcode scanner ^6.

Thank you for your reply. Unfortunately, v7.0.0 does not work on Android (works on iOS). See #565.

Where do we get barcodescanner 7+ ? The releases only show 6.0.5 as the highest version ( https://github.com/phonegap/phonegap-plugin-barcodescanner/releases ).

It's publised on npm https://www.npmjs.com/package/phonegap-plugin-barcodescanner

OK, thanks!

this solved - see #565.
I got v7 to work with android platform v6.3.0 as explained at.

phonegap plugin rm cordova-plugin-compat --force

phonegap plugin add<EMAIL_ADDRESS>

ionic cordova platform rm android

ionic cordova  platform add<EMAIL_ADDRESS>

So maybe upgrading cordova-plugin-compat to v1.2 instead of removing it might have been a better option.


LocationRepository API injection

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2563   👈

This comment was auto-generated by Graphite.


Let "terminal" quick pick on terminals

Refs: https://github.com/microsoft/vscode/issues/93641
Currently we only activate the terminal quick pick when the user types term.
I find this a bit hard to discover, should we also trigger when the user types terminal.
That way it will be more aligned with debug and tasks who use full names and not shortcuts?

Unrelated to my changes, so unassigning myself. Change would be easy to do.

How would we handle this then? Just not show the alias?

I don't think we need to do this considering term was selected as it's short and you type this out, we also have edt and ext as well so we should solve this there as well if we want to do that. But no one has asked outside afaik.

Nobody will ask becuase users do not find this out imho
Yeah I would not render the alias there.

To get both term and terminal to work you need to register both. I am not sure what the suggested solution is here really?

@isidorn I doubt users would just type in terminal(space) assuming it's going to go something without knowing about the feature (via docs or ?).
@bpasero if we wanted to do this for terminal, we should also add an "editor" alias for "edt" and an "extension" alias for "ext". To do this it would probably be better to pass a string | string[] for prefix in PickerQuickAccessProvider to lessen the overhead? I don't think we should do this personally though.

@Tyriar if users are aware of debug(space) and tasks(space) they are more likely to type in terminal(space) than term(space)

The reason I picked edt over editor was to be a little less likely to collide with searching literally for a file called editor. It is not a perfect solution I agree, on the other hand I doubt a lot of people are actually typing edt vs. using a keybinding.
I think every quick picker can decide what prefix to use, there is no real rule currently.

I say we close this as users haven't asked for it, longer names are harder to use and it has more chance for collisions in go to file.


[WIP] Generalize CXCancellation optimization pass to generic gate-inverse pairs

Summary
Closes #6576
A generic gate-inverse cancellation pass can result in shorter circuit operations. This can lessen device noise and error by reducing the number of operations needed to generate the desired output.
Details and comments
To-do

[ ] test
[ ] release notes

 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.

Approximate order to attack these TODOs:

write lots of tests

especially good if we have failing tests for the below tasks

accept a pair of inverse gates (e.g. RXGate(-np.pi/4) and RXGate(np.pi/4))

make sure the self-inverse gates still work (["h", (^, ^)])

error checking: can we make sure that each single item is self inverse, and each item that is a pair is a gate-inverse pair

If you need a break, learn about the differences between lists and tuples (esp. the concept of mutability)


Custom check suport

I'll merge this and enhance the traversal function a bit, its doesn't need to be that complicated ;)


TCK does not support fail test header option anymore.

TCK does not support fail test header option anymore.

Removed the 'fail' option from RequestResponse and RequestStream Tests
Rewrote the tests accordingly

Have not fixed Channel test yet. I will try to fix the channel test in the next commit

Hi Junaid, are you an FB intern/employee continuing the work on the TCK? And have the specifications for the project changed?

I am an intern. I am not aware of any tck specification. But I have neither seen support for "fail" in rsocket-java nor in  rsocket-cpp

Are you interning with the Lithium team? The reason for including the fail headers is to assert negative behavior. For example, if we want to test that a socket is timing out correctly when waiting for a packet that never comes. We want to know that reactive socket fails correctly when it's supposed to.

I'm actually in favour of this, since I think these tests can be better expressed without just saying they fail due to a timeout.
e.g. "awaitNoAdditionalEvents 2000" seems better than "pass = false"
And for other types of failures we should be handling them as signals e.g. https://projectreactor.io/docs/core/release/api/reactor/core/publisher/Signal.html
cc @benjchristensen @lehecka

@xytosis Nice to see you staying involved, want some rsocket tasks to hack on?

@yschimke ah okay that makes sense, it seems like things have changed a bit. I'd also be happy to hack at any non-urgent tasks that you guys have, should be a fun way to pass some time.
I'll accept this PR once the changes to channel have been made.

@xytosis there are tasks in rocket-java if you want to grab one

@junaidkhalid have you tested these against rsocket-java?

Yes, I did. And they work (RR, RS and FnF).
Channel tests do not work. I didnot get time to debug that. Currently the test/resources folder in rsocket-java does not include any channel test. I am not sure whether tests are broken or something is missing with channel suppport


Create SPPCC-v0.12.ckan

Space Plane Colour corrections. Not sure what the extra "P" is for (parts or pieces, I guess), but that's what he calls it.

I think the extra P is with the S, so SPace. Anyway merging with my thanks.


Parse graphql-js types into Nexus structures

Still needs tests to confirm everything is working correctly, but this should allow better interop between existing graphql-js types and nexus by first parsing to nexus compatible structures before processing, so things like type-extension will work correctly.
Fixes: #70, #88
/cc @janicduplessis @otrebu @Weakky

Could you please add some explanation because it looks strange for me?
So, let me explain how I understood the changes:
You have built graphql-js scheme which you convert into nexus SchemeBuilder blocks in order to convert this again into graphql-js scheme. Am I right?

Yeah, thought about this a little more and agree it is confusing. Closing in favor of #143 which will solve the same underlying issue.


Allows to add multiple mapped attributes for a claim in claim-config.xml configuration file

According to the current implementation,  An error occurred while trying to update claim value which has multiple mapped attributes and that have been added when server startup using claim-config.xml file.

This has been fixed a long time ago with the mentioned PRs


Some meshes not visible

Hello,
I managed to get it to work on my fully dynamic project.
All my meshes/models get created in runtime and it somehow works.
Some of the meshes though were not shown at first.
After playing around with the settings a bit I enabled/disabled "Enable Object Moving" and now I also see these models.
Even if it's disabled now. Perhaps some cache that got created? unsure...

You need to have "Enable Objects Moving" on in order for objects to spawn/despawn/move


Run in SAP Business Technology Platform - Cloud Foundry environment

few thoughts:
a: the table maintenance itself will always be part of a larger application, so the CF deploy is just an example, which might become generic for more future open-abap projects
b: is it possible to move the CF stuff to a new folder /cf/ ?
c: how to test the deployment via github actions? is it possible to spin up a CF environment, deploy to that, and check via curl that the application is running?

a) OK
b) I've moved the mta.yaml to cf/.
c) when CF credentials are provided yes. Will give it a try.

c: is it possible to spin up a CF environment in github actions, just without credentials?

I've found cf-mini but that didn't start. So my approach now would be to build the MTAR, extract its content and then run it. That is a bit like the CF buildpack does it. What do you think?

I think CF should help the ecosystem more to test stuff
anyhow, merging, thanks 👍


Support for timezone

Can you please add support for timezones? http://graphite.readthedocs.io/en/latest/render_api.html#tz
Thanks!

Fixed by https://github.com/dgryski/carbonapi/commit/c606fb5e4a05408b031cdc611f6dd6c755e72c92


Unable to download index of package "download"

get https://my kellnr instance/api/v1/cratesio/do/wn/download
HTTP/1.1 404 Not Found
Date: Thu, 20 Jun 2024 10:42:11 GMT
Server: Apache
content-length: 0
Connection: close

Where:
get https://index.crates.io/do/wn/download
will return a OK and json payload.
This is due to the fact that in cratesio_api.rs the download url is registered for /api/v1/cratesio/*/*/download and in cratesio_prefetch_api.rs the index URL is resisted for /api/v1/cratesio/*/*/*.
Best is to register the paths with so many wild cards on a unique base path. Simplest and backwards compatible solution is to register the download URL as:  /api/v1/cratesio-registry/*/*/download.
Found this issue while looking at the code chance that it is a real issue is slim as the crate "download" is yanked. But shows a risk when using wild cards on the same base URL.
For the kellnr private registry there is probably a similar issue. For the crates "download", "yank" and "unyank".

Good catch!
I'll separate the index and the registry api with a unique base path. Crates.io does it with different sub-domains but that solution won't work well for kellnr.
For the "download", "yank" and "unyank" on the private API: That is not an issue as they are "GET", "DELETE" and "PUT" and as such are different enough.

As the crate registry protocol (https://doc.rust-lang.org/cargo/reference/registry-web-api.html) expects the registry API to be under /api/v1/crates/X I'll move the index to it's own path. This behavior is more similar to the behavior of crates.io where the index is on another sub-domain.
Unfortunately, this is a breaking change, as all clients of Kellnr now have to update their cargo.toml files to point to the new index. But I think a more clean solution is worth the break.
TODO: Document the change for a future kellnr 6.0.0 release.

Hmmm, I would not really be fond of a breaking change in the API. At work we just migrated from the git index to the sparse index hosted with Kellnr also needing a URL change. Getting a breaking change like this would not be happily received in the organization.
Looking at the registry documentation I also get the idea that it  usage of "https://crates.io/api/v1/crates" for the downloads is not what you are required to used.  And setting the conig.json dl value to 'my-kellnr.com/api/v1/cratesio-registry/' or something completely different is valid.
Looking at the documentation there are 3 API's:
Index That specifies that at the configured URL in cargo there should be a config.json at the root and how the URL is build to get the index information for a crate. This is the only URL that needs to remain stable for backwards compatibility. But cargo has no requirements about the base URL. So the /api/v1/cratesio/ that kellnr uses now is fine (as long as its not classing with the web API). But also not required, hosting it at /cratesio is also fine.
WEB API The config.json 'api' field contains the base URL and the API specifies the extensions to it like: /api/v1/crates/new. This API is optional, and mainly used for publishing crates.
The download API. The config.json dl field contains the URL that is extended with /{crate}/{version}/download. If the dl field contains variables like {crate} the field is used as is.
So to mine understanding, we only need to keep the index URL's stable. All the other URLS can be moved around provided the config.json is filled with the correct values. My suggestion would be to change the config.json to for the crates.io proxy:
{
  "dl":"https://my-kellnr.nl:443/download/v1/cratesio",
  "auth-required":false
}

Note I removed the api field,  as the crates.io proxy function does not support publishing. And currently it points to the local kellnr registry and that seams wrong. (Field is optional according to the documentation).

You are right. It's enough to move the download URL alone. I just implemented it and it seems to work fine. The change will be in the next Kellnr release. No breaking changes needed :)


UI Accessibility Fixes to Follower UI

[ ] Change background to white and text to dark teal on all Follower UI Screens
[ ] Fix Analytics screen colors so it's still usable

Follower Script Screen (after entering address and hitting submit)

[ ] Increase size of font in Sample Script text.
[ ] Add helper text above the Sample Script text box, "You can use this sample script for your call."

I investigated the styles - the follower UI styles are tightly coupled to the Campaigns styles, as the green b.g. is on .
Easy way:

change rules on , which globally changes Campaign screens also
wrecks Analytics page, but other campaign screens do fine (used white divs for most of UI anyway)

Hard way:
- refactor html to create containers for different app areas

seems like this was addressed in #225


Update template to Core 2.2

Hi, came across your template whilst looking into setting up a new Suave project. I noticed it was still targeting 2.0 which is past end of life so I thought I'd bump it and raise a PR.
I also took the liberty of adding an .editorconfig file. Hope you don't mind. I think it matches your existing code style.

@GraanJonlo thx, but you better create a PR to this official project https://github.com/SuaveIO/Suave.Template


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1665 ms

Resolved: Bofrost.es performance has improved in b17c616 after 7 minutes.


Deprecation of Canvas.getMatrix from API 16 for hardware accelerated canvases

Issue Type

[x] Improvement

Description and/or steps/code to reproduce the problem
According to Android docs:

void getMatrix (Matrix ctm) was deprecated in API level 16.
Hardware accelerated canvases may have any matrix when passed to a View or Drawable, as it is implementation defined where in the hierarchy such canvases are created. It is recommended in such cases to either draw contents irrespective of the current matrix, or to track relevant transform state outside of the canvas.

I don't know if anyone ever noticed a problem while using this deprecated method, but here is where it's used in osmdroid:

ItemizedOverlay.draw
MyLocationNewOverlay.drawMyLocation
NonAcceleratedOverlay.draw

If it's a bug, version(s) of android this affects:
API 16+
Version of osmdroid the issue relates to:
All

Closed as recent PRs limited the use of Canvas.getMatrix to a class (NonAcceleratedOverlay) that is not used.
Reopen if needed.


Minor fix on string #51

This pull request just goes for this readme.md file, so, there is no break on code.

Thank you. Merged.


Jonmv/vespa cli submit non java

@mpolden please review. @hakonhall FYI.

Hang on, fixing some output.

Hmm, there's something wrong. It fails to pick up deployment.xml, for some reason ...

There ...


🛢 Only fetch scratch org changes if you haven't done so recently

This will simply not run the background task, nor update the model and trigger websocket events. Perhaps we want to actually return a different response if the server doesn't need to refetch the ScratchOrg's data?  There's no particularly appropriate HTTP status, but 200 would at least work.

 If my Dev org was recently checked for uncaptured changes, I don't want it to automatically check again.

@jgerigmeyer let me know if I need to make the changes discussed in the main post. Otherwise, free for you to play with.

@davisagli If you have a minute to give this a quick review that'd be swell 👍


Alternative online exporter

https://arc-export.vercel.app/
And here is the GitHub repo lcandy2/arc-bookmarks-export
Fully open source and exporting locally, deployed by vercel which trustable.

That's great! But why would one prefer the option of installing dependencies, raising the server, and running it through a web interface, instead of just running a single script with a single command? And by the way, you need to install pnpm first, which unlike python is not pre-installed.
Vercel is just one of the hosting platforms. It is in no way responsible for what is hosted on it. I don't understand what kind of trust you're talking about.
But in any case, it's great to have alternatives. Good job!

Cause not everyone understands python and cli? that's why i am make an alternative.
Vercel is trustable is because you could see the deployment from the repo, the website you see will only be the one in the repo. also i can changed to the github pages sometime.

It's great that you did this. Having alternatives is always a good thing!


use new cached clusterInfo() in built-in filters

Tracking follow up from https://github.com/envoyproxy/envoy/pull/4600

Done


bazel: only use python3

Motivated by https://github.com/NixOS/nixpkgs/pull/201859.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Successfully built all bazel versions, macos 10.15, nixos x86_64 & aarch64. Built bazel_6.tests & bazel_5.tests on linux x86_64 & aarch64 (already broken for me on darwin)

bazel_4.tests & bazel_3.tests build, aarch64-linux too.


chore: import repository https://github.com/rajatgupta24/quickstart-003.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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

Test name
Commit
Details
Rerun command

verify
28b82b8ad2108db3a8af6b1aa1dda1dff3c70180
link
/test verify

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


Implement a --version parameter

A colleague asked for adding a --version parameter. This is a bit tricky when dealing with GIT, unless I start using proper release tags and using git archive to do so. Even so, it seems things are a wee bit more complicated to have something like the old "$Id$" string in other version control systems.

I really would like this to be automatic, and not use fixed strings anywhere (which I certainly will someday forget to update). Suggestions welcome.

Please have a look at https://github.com/roseswe/gittag
I use this e.g. with https://github.com/cfg2html/cfg2html


Which paper does SkipLastGNN come from?

This is mentioned in https://arxiv.org/abs/1904.03751
https://arxiv.org/abs/2007.03092
It's used as an example of custom message passing class in PyG

This is mentioned in https://arxiv.org/abs/1904.03751
https://arxiv.org/abs/2007.03092
It's used as an example of custom message passing class in PyG

Thanks

Thanks


Update PULL_REQUEST_TEMPLATE.md

What does this PR do?
This PR makes changes to the existing PR template. Better PRs will hopefully lead to moving faster.
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Read through the proposed PR template. I made some minor changes to the verbiage, which will hopefully help provide more information to the individuals who are tasked with reviewing & merging. For example, one change is a specific request for manual or automated tests. If no tests are written, an explanation should be included in the PR.
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Can one of the admins verify this patch?


Release v1.0.0-beta2

What is the purpose of this change? What does it change?

Releasing the v1.0.0-beta2 version of odo
Was the change discussed in an issue?
fixes #???

How to test changes?

Release notes
Enhancements

Resolved odoignore not working in windows (https://github.com/openshift/odo/pull/1711)
Hidden the component get command (https://github.com/openshift/odo/pull/1734)
New workflow for odo storage (https://github.com/openshift/odo/pull/1547)
Fixed multiple deployments for local component (https://github.com/openshift/odo/pull/1684)
Fixed odo login not working properly for insecure https connections (https://github.com/openshift/odo/pull/1652)
Added --all flag for odo component delete to allow removal of both config and component in one command (https://github.com/openshift/odo/pull/1731)
Added show flag for odo watch (https://github.com/openshift/odo/pull/1659)
odo catalog list now doesn't require a .odo/config.yaml to be present in the working directory (https://github.com/openshift/odo/pull/1714)
odo project delete now doesn't break after consecutive runs (https://github.com/openshift/odo/pull/1710)

Testing

Better organisation of the tests by follow ginkgo standards (https://github.com/openshift/odo/pull/1612)
Used Gomega watcher for json tests (https://github.com/openshift/odo/pull/1625)

Internal

Fixed invalid paths using rsync on windows (https://github.com/openshift/odo/pull/1705)
Used a more robust project.Exists to reduce flaky tests (https://github.com/openshift/odo/pull/1719)

Documentation

Modified README for formatting and clear installation instructions based on OS (https://github.com/openshift/odo/pull/1735)
Fixed typos in the documentation (https://github.com/openshift/odo/pull/1666)
Migrated to .adoc from .md for documentations (https://github.com/openshift/odo/pull/1645)

extension of cli reference file should be adoc

@surajnarwade resolved it

draft release notes in the description, please

@mohammedzee1000 added release notes

Looks ok, but Maybe a bit more description
See previous release notes https://github.com/openshift/odo/releases

/lgtm

/refresh

/retest


FlutterError: Null check operator used on a null value in DragGestureRecognizer.acceptGesture

Problem:
We received via crashlytics several dozens reports of FlutterError: Null check operator used on a null value in the acceptGesture method of the DragGestureRecognizer class on our production app.
Stack traces:
Non-fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Null check operator used on a null value. Error thrown Instance of 'yx'.
       at .#0      DragGestureRecognizer.acceptGesture (package:flutter/src/gestures/monodrag.dart:321)
       at .#1      GestureArenaManager.sweep (package:flutter/src/gestures/arena.dart:157)
       at .#2      GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:362)
       at .#3      GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:338)
       at .#4      RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:267)
       at .#5      GestureBinding._handlePointerEvent (package:flutter/src/gestures/binding.dart:295)
       at .#6      GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:240)
       at .#7      GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:213)
       at .#8      _rootRunUnary (dart:async/zone.dart:1206)
       at .#9      _CustomZone.runUnary (dart:async/zone.dart:1100)
       at .#10     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1005)
       at .#11     _invoke1 (dart:ui/hooks.dart:265)
       at .#12     _dispatchPointerDataPacket (dart:ui/hooks.dart:174)

Output of flutter doctor -v:
[√] Flutter (Channel stable, 1.22.5, on Microsoft Windows [Version 10.0.19042.685], locale en-US)
    • Flutter version 1.22.5 at C:\flutter
    • Framework revision<PHONE_NUMBER> (4 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\XXX\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_HOME = C:\Users\XXX\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[!] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.52.1)
    • VS Code at C:\Users\XXX\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.18.0

[√] Connected device (1 available)
    • Pixel 3a (mobile) • 94AAY0LLBG • android-arm64 • Android 11 (API 30)

! Doctor found issues in 1 category.

Output of flutter doctor -v:
[√] Flutter (Channel stable, 1.22.5, on Microsoft Windows [Version 10.0.19042.685], locale en-US)
    • Flutter version 1.22.5 at C:\flutter
    • Framework revision<PHONE_NUMBER> (4 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\XXX\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_HOME = C:\Users\XXX\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[!] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.52.1)
    • VS Code at C:\Users\XXX\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.18.0

[√] Connected device (1 available)
    • Pixel 3a (mobile) • 94AAY0LLBG • android-arm64 • Android 11 (API 30)

! Doctor found issues in 1 category.

kindly provide minimal reproducible code of this issue.

kindly provide minimal reproducible code of this issue.

Hi @talent-apps
Can you please provide your pubspec.yaml, the list of affected devices and if possible your flutter run --release --verbose running on the same OS version as the affected devices and a minimal reproducible code sample?.
Thank you

Hi @talent-apps
Can you please provide your pubspec.yaml, the list of affected devices and if possible your flutter run --release --verbose running on the same OS version as the affected devices and a minimal reproducible code sample?.
Thank you

These crash reports come from crashlytics.
This issue occurs on multiple devices (36% Samsung, 32% Xiaomi, 10% Oppo, 6 % Huawei, 16% Other) and multiple Android versions (61% Android 10, 26% Android 9, 7% Android 7, 6% Android 8).
From the reports we see that this error occurs in multiple screens of the app so we are not sure which piece of code is relevant.
The only "draggable" widget that is common to these screens is a TabBarView.

These crash reports come from crashlytics.
This issue occurs on multiple devices (36% Samsung, 32% Xiaomi, 10% Oppo, 6 % Huawei, 16% Other) and multiple Android versions (61% Android 10, 26% Android 9, 7% Android 7, 6% Android 8).
From the reports we see that this error occurs in multiple screens of the app so we are not sure which piece of code is relevant.
The only "draggable" widget that is common to these screens is a TabBarView.

Hi @talent-apps
It is still not clear how to reproduce this issue. For anyone to help you, we need a code that reproduces the issue so we know what causes the issue and where it is happening. Please try to find a reproducible code sample based on your current implementation.
Thank you

Hi @talent-apps
It is still not clear how to reproduce this issue. For anyone to help you, we need a code that reproduces the issue so we know what causes the issue and where it is happening. Please try to find a reproducible code sample based on your current implementation.
Thank you

Hi!
I also have this same problem in my production app using version 1.22.4 of the flutter.
I still couldn't simulate the error. But from the crashlytics records, it seems that it usually occurs on a screen that has a vertical scroll with children that have a horizontal scroll.

Hi!
I also have this same problem in my production app using version 1.22.4 of the flutter.
I still couldn't simulate the error. But from the crashlytics records, it seems that it usually occurs on a screen that has a vertical scroll with children that have a horizontal scroll.

@RafaelBarbosatec Can you provide a code sample of it?

@RafaelBarbosatec Can you provide a code sample of it?

I'm trying to reproduce. On my device it doesn't happen at all. I don't know if it's intermittent. But as soon as I can reproduce, I'll put the test code here.

I'm trying to reproduce. On my device it doesn't happen at all. I don't know if it's intermittent. But as soon as I can reproduce, I'll put the test code here.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


I added CSS theming

It saves the theme into the browser

Nice work, though I noticed the datatables remained their initial colour - this theming mechanism merely changes the background colour of the page. If possible, try to change the datatable colour, I followed this guide but had no luck.

Okay2 I'll look into that guide you provided and add it by the way I can't seem to have any data.

You likely overflooded the API and your requests will be dropped for sometime, you will probably see an API error in the log. You have two options - You can use an API key, or, you can use existing data by directly inserting the cache.
In the function fetch_new_data() in server.py
Replace the line coins = self.read_coins_file() with:
coins = []

Replace the line self.__cache = ls.copy() with:
self.cache = [{'img': 'icons/algo.png', 'price': '$ 1.26', 'change_colour': 'green', 'change': '+3.99%', 'code': 'ALGO', 'name': 'Algorand'}, {'img': 'icons/ada.png', 'price': '$ 2.87', 'change_colour': 'red', 'change': '-2.49%', 'code': 'ADA', 'name': 'Cardano'}, {'img': 'icons/dash.png', 'price': '$ 254.07', 'change_colour': 'green', 'change': '+0.63%', 'code': 'DASH', 'name': 'Dash'}, {'img': 'icons/keep.png', 'price': '$ 0.47', 'change_colour': 'green', 'change': '+1.33%', 'code': 'KEEP', 'name': 'Keep Network'}, {'img': 'icons/xmr.png', 'price': '$ 305.11', 'change_colour': 'red', 'change': '-1.76%', 'code': 'XMR', 'name': 'Monero'}, {'img': 'icons/ocean.png', 'price': '$ 0.95', 'change_colour': 'green', 'change': '+2.40%', 'code': 'OCEAN', 'name': 'Ocean'}, {'img': 'icons/sc.png', 'price': '$ 0.02', 'change_colour': 'green', 'change': '+8.65%', 'code': 'SC', 'name': 'Siacoin'}, {'img': 'icons/trx.png', 'price': '$ 0.10', 'change_colour': 'red', 'change': '-0.25%', 'code': 'TRX', 'name': 'Tron'}, {'img': 'icons/zec.png', 'price': '$ 163.60', 'change_colour': 'green', 'change': '+1.57%', 'code': 'ZEC', 'name': 'Zcash'}]

This stops the server from using API data and instead uses the cache supplied above. Make sure to revert this change before commits

Ohhh okay2 thank you will do.  I wanna do a sanity check before I do my things is that do you want the data tables to have dark and light mode or change the color(red or green) depending on the data.

Here is a rough template for what I have in mind:

Of course, if you have any ideas/changes, feel free to implement them, I'm sure my design skills are lacking compared to masses, this is just a template.
If I understand CSS correctly, this would also be in the form of
.theme_1 {
  background-color: x
  more-stuff: x
}

If implemented correctly, this would mean new themes could be selected with ease, via a select button, e.g.

Where each selection would trigger a different CSS theme.


Swagger not generated from Controllers in different assemblies with version 12.0.5 and NetCore22

Controllers outside project specified in swaggerGenerator are ignored with v12.0.5 and NetCore22. I tried with NSwag.MSBuild and NSwagStudio. Same issue, there only generate swagger for controllers in main project.
It works well in v12.0.4 and NetCore21.

Maybe thats because this changed in asp.net core?

@RSuter Yep, services.AddMvc().AddApplicationPart(assembly); solve the problem.
Documentation: https://docs.microsoft.com/en-us/aspnet/core/mvc/advanced/app-parts?view=aspnetcore-2.2

I think this extension was already needed in 2.1, no?

No, but it may be a regression. It worked without it in 2.1.
In the documentation it is specified:

By default MVC will search the dependency tree and find controllers (even in other assemblies).

It should work without AddApplicationPart.


[dq] Use separate precompute execution state for each running graph

Changelog entry
...
Changelog category

Bugfix

Additional information
...

Incorrect fix


Events aren't loaded inside a directive

When I wrap the chartist directive inside the link of another directive, events are not called.
Here is a working plunker http://plnkr.co/edit/Z5yYElynZWI0MabZHK5c?p=preview
mydirective.js
'use strict';

angular.module('myApp')
.directive('widgetDirective', [function() {
  return {
    restrict: 'A',
    scope: true,
    templateUrl: 'tpl.html',
    link: function($scope, $element, $attrs) {
      $scope.chartistEvents = {
        draw: function(obj) {
          // Never here
          if(obj.type === 'bar') {
            obj.element.attr({
              style: 'stroke-width: 30px'
            });
          }
        }
      };
     $scope.barData = {
        labels: ['Lundi', 'Mardi', 'Mercredi', 'Jeudi', 'Vendredi'],
        series: [
          [5, 4, 3, 7, 5],
          [2, 1, 1, 0, 0],
          [0, 2, 1, 0, 0],
        ]
      };
      $scope.barDataOpts = {
        stackBars: true,
      };
    }
  };
}]);

tpl.html
<chartist class="ct-chart" chartist-data="barData" chartist-chart-type="Bar" chartist-chart-options="barDataOpts" chartist-events="chartistEvents"></chartist>

Very weird. I will have to look more into the lifecycle of directives, because it works when it is in the controller: http://plnkr.co/edit/30i9NYSZAvzwPsDsHusE?p=preview

Closing this for lack of activity. Feel free to reopen if anything.

The reason for this is because postLink functions are called in reverse order (children first, then their parents in turn).
angular-chartist calls renderChart when ctrl.element is set, which is done in the chartist directive's postLink function. If a parent directive then sets some events in its postLink function, that will happen after renderChart has been called.

Note that just defining the link property on a directive definition object is a shortcut to defining a postLink function.

I was tearing my hair out about this - huge thanks @redders6600 for pointing me in the right direction.
For a more concrete solution, one way of solving the events issue is to stick it in your pre-link rather than the post-link (the default 'linking' action).
link: {
    pre: function preLink(scope, iElement, iAttrs, controller) {
        // scope.eventFunctionsToPass = {} and whatnot
    },
    post: function postLink(scope, iElement, iAttrs, controller) {
        // Other linking behaviour (this is where 'regular' link functions get executed)
    }
}


Debug a flutter app on an iOS device

A developer should be able to debug their running app on an iOS device.

cc @chinmaygarde @iposva-google @devoncarew

Device-based development is not in scope for the hackathon.

This has made a ton of progress.  @chinmaygarde or @tvolkert does this work now?

Not yet.  We're wiring up unit tests for the engine, after which I'd like to get native integration tests working, including device-based testing.  We may need to use a local lab to get it working.

Just tried this again. From Atom, I set a breakpoint and then launched the app. The app started on my device, but the breakpoint didn't trigger. I tried two different breakpoints.

I'm pretty sure this works now.


[SPARK-28695][SS] Use CaseInsensitiveMap in KafkaSourceProvider to make source param handling more robust

What changes were proposed in this pull request?
SPARK-28163 fixed a bug and during the analysis we've concluded it would be more robust to use CaseInsensitiveMap inside Kafka source. This case less lower/upper case problem would rise in the the future.
Please note this PR doesn't intend to solve any kind of actual problem but finish the concept added in SPARK-28163 (in a fix PR I didn't want to add too invasive changes). In this PR I've changed Map[String, String] to CaseInsensitiveMap[String] to enforce the usage. These are the main use-cases:

contains => CaseInsensitiveMap solves it
get... => CaseInsensitiveMap solves it
filter => keys must be converted to lowercase because there is no guarantee that the incoming map has such key set
find => keys must be converted to lowercase because there is no guarantee that the incoming map has such key set
passing parameters to Kafka consumer/producer => keys must be converted to lowercase because there is no guarantee that the incoming map has such key set

How was this patch tested?
Existing unit tests.

Can one of the admins verify this patch?

Test build #108969 has started for PR 25418 at commit 2925640.

Test build #108969 has finished for PR 25418 at commit 2925640.

This patch fails Scala 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/108969/
Test FAILed.

Can one of the admins verify this patch?

Test build #108971 has started for PR 25418 at commit d5ba9f1.

Test build #108971 has finished for PR 25418 at commit d5ba9f1.

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/108971/
Test PASSed.

cc @HeartSaVioR @HyukjinKwon @cloud-fan this is a follow-up on https://github.com/apache/spark/pull/24967

Test build #109039 has started for PR 25418 at commit c0b945c.

Test build #109039 has finished for PR 25418 at commit c0b945c.

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/109039/
Test PASSed.

thanks, merging to master!


Convert xorm literal queries to method calls

The literal queries contained the keyword LIMIT, which doesn't exist in MSSQL. When I tried migrating, the process failed upon their execution. This change fixed it.

LGTM

LGTM


Confusing words on Blazor hosting models

Blazor app hosting models
Blazor apps can be hosted in IIS just like ASP.NET Web Forms apps. Blazor apps can also be hosted in one of the following ways:

Client-side in the browser on WebAssembly.
Server-side in an ASP.NET Core app.

 Two or three? If there are only two hosting models (as the following of this article indicates), then the first two sentences should be revised to remove ambiguity.

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

ID: 4b45d194-8874-5bd1-31eb-767145461904
Version Independent ID: 9817ff2d-afe1-8a5f-1e1a-7c7eadba5cf0
Content: Blazor app hosting models
Content Source: docs/architecture/blazor-for-web-forms-developers/hosting-models.md
Product: dotnet-architecture
Technology: blazor
GitHub Login: @danroth27
Microsoft Alias: daroth

Sending email to @danroth27


Testing withPaginator method

Hello! I try to test my controller but I couldn't mock response's withPaginator method. I saw that method coming from Laravel's specific Response class. So I don't know how can I test that response line. Should I change Interface or is there any clean way to do?
Here's the code I would like to tests:
return $this->response->withPaginator(Test::paginate(20), new BarTransformer());

It's my test method:
public function testIfRequestHasNotSearchQuery()
{
    $response = Mockery::mock('EllipseSynergie\ApiResponse\Contracts\Response');

    $this->app->instance(\EllipseSynergie\ApiResponse\Contracts\Response::class, $response);

    $paginator = new LengthAwarePaginator(['foo'], 1, 1, 1);
    $barTransformer = new \Foo\Transformers\BarTransformer($paginator);

    $response->shouldReceive('withPaginator')->with($paginator, $barTransformer)->andReturn('foo');

    $response = $this->json('GET', '/api/v1/foo', [], [
        'token' => 'foobarbaz'
    ]);

    $this->assertResponseOk($response);
}

Runned test log:
[2016-12-02 08:29:47] testing.INFO: {}
[2016-12-02 08:30:13] testing.ERROR: Mockery\Exception\NoMatchingExpectationException: No matching handler found for Mockery_3_EllipseSynergie_ApiResponse_Contracts_Response::withPaginator(object(Illuminate\Pagination\LengthAwarePaginator), object(Foo\Transformers\BarTransformer)). Either the method was unexpected or its arguments matched no expected argument list for this method

Thanks for helping!

Yes you can mock response. However, you need to pass exactly the same instance to the withPaginator method when you try to test them using $response->shouldReceive('withPaginator')->with().
You test fail because the $barTransformer = new \Foo\Transformers\BarTransformer($paginator); is not the same instance then the one created in your controller $this->response->withPaginator(Test::paginate(20), new BarTransformer());. Same thing for Test::paginate(20).
I think the best way to resolve this is to validate the input like this $response->shouldReceive('withPaginator')->with(Mockery::type(LengthAwarePaginator::class), Mockery::type(BarTransformer::class))->andReturn('foo');
Hope it's make sense to you !


The "createReport" function Error: Can't read the data of 'the loaded zip file'

Dear author,
I used the "createReport" function in my code.
But the console print the error when the code run.
The Error is:
Error: Can't read the data of 'the loaded zip file'. Is it in a supported JavaScript type (String, Blob, ArrayBuffer, etc) ?
code:
const tempDoc=fs.readFileSync('/test/example.docx')
const tty=async()=>{
try{
const reportb=await createReport({
tempDoc,
data:{
name:'tom'
},
});
}catch(error){
console.log(error)
}
}
tty()
project version:
The docx-templates version is the latest by npm install.
I find some Issues about this error in the docx-templates project，but it can't fixed the error.
Can you tell me where is the wrong in my program and how to fix it？
Thank you.

Yes, you are not passing a template. Note that createReport expects an object with a property called template. You passed an object with the property tempDoc instead.
You should study javascript object shorthand syntax, see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Object_initializer#property_definitions
        const reportb = await createReport({
            template: tempDoc,
            data: {
                name: 'tom'
            },
        });

Thanks for your suggestion!
I tried to add this 'template' when I created this issue ago.
But the problem is still unsolved.
code:
async function tty() {
try {
const reportb = await createReport({
template: tempDoc,
data: {
name: 'Tom'
},
});
} catch (error) {
console.log(error)
}
}
tty();
I don't know how can fixed it by another way.

Yes, you are not passing a template. Note that createReport expects an object with a property called template. You passed an object with the property tempDoc instead.
You should study javascript object shorthand syntax, see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Object_initializer#property_definitions
        const reportb = await createReport({
            template: tempDoc,
            data: {
                name: 'tom'
            },
        });

Thanks for your suggestion!
I tried to add this 'template' when I created this issue ago.
But the problem is still unsolved.
code:
async function tty() {
try {
const reportb = await createReport({
template: tempDoc,
data: {
name: 'Tom'
},
});
} catch (error) {
console.log(error)
}
}
tty();
I don't know how can fixed it by another way.

@gjlong
Please use markdown code blocks when you post code snippets in github issues.
The 'Can't read the data of 'the loaded zip file'. Is it in a supported JavaScript type (String, Blob, ArrayBuffer, etc) ?' error means that the template you are providing is not a valid zip file (.docx files are zip files).
You should double-check whether the docx file you are using as a template is valid.
Also, I don't know what environment you are using so it is impossible for me to tell what your implementation of fs.readFileSync actually does. Make sure it produces a Buffer (when using NodeJS).
The below works for me without problems.
import { createReport } from "docx-templates";
import fs from 'fs'

async function tty() {
    try {
        const tempDoc = fs.readFileSync('./example.docx')
        const reportb = await createReport({
            template: tempDoc,
            data: {
                name: 'Tom'
            },
        });
        console.log('result:', reportb)
    } catch (error) {
        console.log(error)
    }
}
tty();

@gjlong Please use markdown code blocks when you post code snippets in github issues.
The 'Can't read the data of 'the loaded zip file'. Is it in a supported JavaScript type (String, Blob, ArrayBuffer, etc) ?' error means that the template you are providing is not a valid zip file (.docx files are zip files).
You should double-check whether the docx file you are using as a template is valid.
Also, I don't know what environment you are using so it is impossible for me to tell what your implementation of fs.readFileSync actually does. Make sure it produces a Buffer (when using NodeJS).
The below works for me without problems.
import { createReport } from "docx-templates";
import fs from 'fs'

async function tty() {
    try {
        const tempDoc = fs.readFileSync('./example.docx')
        const reportb = await createReport({
            template: tempDoc,
            data: {
                name: 'Tom'
            },
        });
        console.log('result:', reportb)
    } catch (error) {
        console.log(error)
    }
}
tty();

The Github add code is not working in the browser.
This example code is running in the weixin mini program.
The weixin mini program environment is javascript core and v8，not NodeJS.
So I think the Buffer is not as same as in the NodeJS.
And the example code can run in the browser environment.
I think your opinion is right.


Add Mythos & Logos

please add assetlist.json files for each of these

Are these chains live?  Is the no code, genesis, or explorers for either of them?

These chains are alive. They are currently being furnished with all smart contracts for testing in production mode the inter-chain dTypeDB:
https://commonwealth.im/evmos/discussion/4980-dtypedb-the-first-interchain-database
https://youtu.be/a9RI6xNzGpI
https://mythos.provable.dev/rpc/status
https://logos.provable.dev/rpc/status
Their source code is based on Ethermint and is open source to volunteers only (for now). We hope to have the necessary recognition to open source to all.


Error in uploading last 5 journals

I got a mission now on INARA that says:
Clothing seeks sightseeing adventure
For Future of Arro Naga
Origin: Shinrarta Dezhra [Jameson Memorial]
Destination: Wolf 497
Influence gain: +
I did source him 3 clothing, but it clearly was not clothing that gave me the mission.
Thanks for making the program. :)

I have to admit I have not tested passenger missions extensively :) Thanks for reporting this, will check

This is how an entry looks, notice there are multiple destinations too:
{ "timestamp":"2018-12-21T03:11:49Z", "event":"MissionCompleted", "Faction":"Los Chupacabras", "Name":"Mission_Sightseeing_name", "MissionID":442949477, "Commodity":"$Clothing_Name;", "Commodity_Localised":"Clothing", "Count":3, "NewDestinationSystem":"Shinrarta Dezhra", "DestinationSystem":"LHS 3675$MISSIONUTIL_MULTIPLE_INNER_SEPARATOR;Hecate$MISSIONUTIL_MULTIPLE_INNER_SEPARATOR;Tujing$MISSIONUTIL_MULTIPLE_FINAL_SEPARATOR;HIP 57822", "Reward":4978600, "FactionEffects":[ { "Faction":"Los Chupacabras", "Effects":[ { "Effect":"$MISSIONUTIL_Interaction_Summary_SP_up;", "Effect_Localised":"The security status of $#MinorFaction; has improved in the $#System; system.", "Trend":"UpGood" } ], "Influence":[ { "SystemAddress":3932277478106, "Trend":"UpGood", "Influence":"+++++" } ], "ReputationTrend":"UpGood", "Reputation":"+" } ] }

Fix released in <IP_ADDRESS>4


Local demo runs correctly on recent version of PCF dev

Currently, given a recent version of PCF dev:
$ cf version
cf version 6.40.1+85d04488a.2018-10-31
$ cf dev version
CLI: 0.0.11
cf: v4.5.0
cf-mysql: 36.15.0

Then the local demo scripts to not successfully deploy the service broker and demo app.

Closing as won't fix for now. Below are the notes for running the demo with this PCF Dev version info:
cf version 6.40.1+85d04488a.2018-10-31
$ cf dev version
CLI: 0.0.11
cf: v4.5.0
cf-mysql: 36.15.0```

1) It looks like the API endpoint has changed from the script value of `https://api.local.pcfdev.io` to `https://api.dev.cfdev.sh`. This could be a simple update to check for both in the dependencies script. I wanted to call it out though.

2) I ran into an issue where `http://host.pcfdev.io` no longer points correctly to the host machine (where Conjur is running). I did find this issue (https://github.com/cloudfoundry-incubator/cfdev/issues/36) where others had encountered something similar, but this problems seems to be re-introduced with the latest PCF dev version (0.0.11). Switching to the IP address works for me, as the last comment on that issue mentions, but that isn't really a sustainable or portable solution.

3) The demo is using an old version of Conjur Open Source (`0.3.0`). Bumping to `1.0` doesn't work out of the box for me, but I don't know if any of you have already tried and resolved this locally. It's also not clear to me if we should consider switching the demo (or adding an option) to use V5 EE instead of Open Source.


1.2.2 Router reloading page hash changes in Chrome

So I've created the most simple Router application here with<EMAIL_ADDRESS>var Backbone = require('backbone');
var Router = Backbone.Router.extend({
	routes: {
		":codename(/*path)": "onRoute"
	},
	onRoute: function () {
		var p = document.createElement('p');
		var args = Array.prototype.slice.call(arguments);
		p.innerText = args.join(", ");
		document.body.appendChild(p);
	}
});

new Router();

Backbone.history.start();

Using browserify and beefy to run this on localhost:1337.
To duplicate in Chrome Version 44.0.2403.157 (64-bit) (OSX Yosemite)

Open the developer tools

In address bar type some random fragment: #foo -- See foo, ,

In address bar type some next random fragment: #bar -- See now foo, , \n bar, ,

Ok that seems normal.

Close the developer tools

Repeat steps 2 and 3 a couple of times:  Be sure to occasionally type fragments that you've used before.  The router is continually reloading the page when the fragment changes.

Expected:
Should not reload the page on fragment changes.
It seems like for some reason Backbone Router behaves correctly when the developer tools are open, and poorly when they are closed.  Safari also seems to suffer from this to some lesser degree.
Maybe related to #3773 ?
Note that this seems apart from the known Webkit issue when loading the same url + fragment twice.
https://bugs.webkit.org/show_bug.cgi?id=24578

This may be an issue in Chrome/Webkit.  Testing with flatiron/director reveals the same behavior.

Ok.  FWIW - this issue appears to be caused by a combination of beefy index.js 1337 --live -- -d + Chrome/Webkit + a catch-all type of Route like "*notFound" or ":codename(/*path)" with Backbone.Router (or in director something like "/(.*)").
Running python -m SimpleHTTPServer results in the expected behavior of the router.
Removing the --live flag from beefy also results in correct behavior.
Interestingly that turning  on the Chrome Developer Tools somehow "fixes" this problem when beefy is in "live" mode.

Referring to the changes in 1.2.0:
"As a neat side effect, Backbone.History no longer uses jQuery's event methods for pushState and hashChange listeners. We're native all the way."
Changing this;
$(window).on('hashchange', hashChangeCallback);

to this, worked for me;
window.addEventListener('hashchange', hashChangeCallback);

I eventually found this to be rooted in a Chrome instability, shutting down Chrome and restarting it fixed the behavior


Solution
Added cookbook demonstrating Pinecone + Portkey integration for RAG applications, showing how to combine vector search with Portkey's API management and monitoring capabilities.
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Infrastructure change (CI configs, etc)
[x] Non-code change (docs, etc)
[ ] None of the above

Test Plan

Verified notebook executes end-to-end

@DosticJelena Can you take a look?


in_http: Fix to check keepalive enabled explicitly

This PR contains imported patch which are from https://github.com/fluent/fluentd/commit/4ebce91bdaa2fbc8ca61fafe38256ef731b6bcbc partially and https://github.com/fluent/fluentd/commit/67f9825853a692d99075e9e67d0a87272f4a19ed.
Note that this imported patch is rewritten to adopt without new v0.14 API and with current master.
Related to this developers TODO:

in_http test fix to check HTTP keepalive is enabled explicitly
https://github.com/fluent/fluentd/commit/67f9825853a692d99075e9e67d0a87272f4a19ed

ref: https://github.com/fluent/fluentd/wiki/Developer-notes-for-fixes-on-v0.14#plugin-miscellaneous

Oh! This PR hits this rake issue: https://github.com/ruby/rake/issues/114

LGTM

Thanks!


Access to the raw message text

Using the REST api or ciscospark SDK I'm able to get the raw message text. However, in bot.hears() flint.hears() it seems like whitespace has been stripped down to a single space? I haven't delved very deep into this yet, but I have a use case where the original whitespace (specifically line breaks) in the message is significant.

You are correct. Here is portion of the code that does that.
https://github.com/nmarus/flint/blob/master/lib/flint.js#L628
I would be interested in the use case you are running across to better understand. Feel free to drop me an email at<EMAIL_ADDRESS>if you are open to share.

I'm simply creating a bot to create polls for rooms. I want to keep the formatting dead simple and minimize the margin of error, so the title/question is the first line with the subsequent non-empty lines are alternatives. I've already implemented this using ciscospark, before I learned about flint and decided to port it over. I don't really see a way of getting the same behavior without fetching the message object again (which seems a bit unnecessary.
Example:
@Bot /create What's the answer?
1
42
1 million

Enhancement applied in v.4.4.0.
See details here.


Suddenly not loading css and img resources

It was all working pretty nice! Until today. Now only images from s3 are loaded but all external css, icon and images are not!
I have a service that receives an base64 encoded html payload (so it can easily be sent as json), converts it back to utf8 to finally set this as the puppeteer page content.
I double checked if utf8 html was wrong and it worked nicely saving the content as test.html and opening in chrome.
Coincidence or not, this same response in Postman preview mode is not loading css and img too

@filenda Can you share any sort of repro or URL I can look at?

@alixaxel tks for getting back to me! I've come to realize that it was something to do with SSL certificate that was expired/not valid from the host we were grabbing the assets. Uploaded the file to an S3 public bucket and voilá. I know there's a know property to ignore ssl/https errors, will try that too.


hex value is converted to a number in getAccessibilityContrastColor function

Conversion should remain as a string value.

Fixed.


Expected behavior in Chrome?

When I test this in Chrome 55+, the native date picker overlaps with this script's version (depending on where I click in the input box), and only the native picker will let me choose a date.

Is this expected behavior? If so, do I need to detect the user's browser and only load this library if the browser is IE, Firefox, or Safari?

same behaviour confirmed on android 5.1 webview/browser.
feature request to use modernizr to detect native datepickers?

It would appear that an update in Chrome broke the native detection code.

Thanks a lot for your report!
Your example case is the library's debug page.  In the HTML for that page, the library <script> is included with the data-nodep-date-input-polyfill-debug attribute, which makes the library ignore its detection code.
If you're just playing with the debug page, just remove that attr to see the true library behavior.
If you're including the library in your app, use one of the methods described in the README.
If there are any details I've missed, please comment and tell me!

Thanks very much @brianblakely