[web]: Not Able to Receive SSE Events

Have a Flutter App that is able to receive SSE text/event-stream events when running on mobile but it is not receiving the same SSE text/event-stream events when running on the chrome web.
import 'package:flutter/material.dart';
import 'package:http/http.dart' as http;

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {

  http.Client _client;

  MyApp() : super() {
	print("Ctor called");
	subscribe();
  }

  @override
  Widget build(BuildContext context) {
	print("building..");
	return MaterialApp(
	  title: 'Flutter SSE',
	  home: Scaffold(
		appBar: AppBar(
		  title: Text('Receive SSE Events'),
		),
		body: Center(
		  child: Text('Ready for events..'),
		),
	  ),
	);
  }

  subscribe() async {
	print("Subscribing..");
	try {
	  _client = http.Client();

	  var request = new http.Request("GET", Uri.parse("http://<IP_ADDRESS>:8080/myserver/events"));
	  request.headers["Cache-Control"] = "no-cache";
	  request.headers["Accept"] = "text/event-stream";

	  Future<http.StreamedResponse> response = _client.send(request);
	  print("Subscribed!");

	  response.asStream().listen((streamedResponse) {
		print("Received streamedResponse.statusCode:${streamedResponse.statusCode}");
		streamedResponse.stream.listen((data) {
		  print("Received data:$data");
		});
	  });
	} catch(e) {
	  print("Caught $e");
	}
  }

  unsubscribe() {
	_client.close();
  }
}

On the server-side, I can see that the flutter app always subscribes okay as I can see the Sink being added. And the server-side always dispatches the events to the Sink but the flutter app only receives the events when running as a mobile app and NOT when running as a Chrome web app.
C:\Users\pondeMuse\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel master, v1.10.15-pre.188, on Microsoft Windows [Version 10.0.16299.1087], locale en-GB)
• Flutter version 1.10.15-pre.188 at C:\Users\pondeMuse\flutter
• Framework revision 584ee10c68 (2 days ago), 2019-10-21 07:49:28 -0700
• Engine revision 8aefcd8575
• Dart version 2.6.0 (build 2.6.0-dev.8.0 a61c775db8)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at C:\Users\pondeMuse\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 3.5)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 40.2.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
[!] IntelliJ IDEA Community Edition (version 2019.2)
• IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2018.2.5
X Flutter plugin not installed; this adds Flutter specific functionality.
• Dart plugin version 192.7402
• For information about installing plugins, see
https://flutter.dev/intellij-setup/#installing-the-plugins
[√] Connected device (2 available)
• Chrome     • chrome     • web-javascript • Google Chrome 77.0.3865.120
• Web Server • web-server • web-javascript • Flutter Tools
! Doctor found issues in 1 category.

I keep reading that there could be issues associated with CORS when running on a web browser but as far as I can see, there are no issues with CORS. CORS on the server-side has been configured to accept all origins (i.e. ""), configured to accept "GET" and "OPTIONS" methods, and configured to accept all headers (i.e. "").
Debugging in Chrome browser shows that the response back is a 200 and can see the following response headers:
Access-Control-Allow-Origin: *
Cache-Control: no-cache, no-store, max-age=0, must-revalidate
Content-Type: text/event-stream;charset=UTF-8
Expires: 0
Pragma: no-cache
transfer-encoding: chunked
Vary: Origin
Vary: Access-Control-Request-Method
Vary: Access-Control-Request-Headers
X-Content-Type-Options: nosniff
X-Frame-Options: DENY
X-XSS-Protection: 1 ; mode=block

Unfortunately, I don't know enough about the http protocol so I don't know if any of the response headers above should be reason for concern as to why an event stream cannot be received on a web browser?
As a side note, the flutter/dart code was originally written as angulardart/dart code and that code was able to receive streamed events fine when running on Chrome web browser (Therefore assuing that CORS was configured correctly).
Any help or pointers on this issue would be greatly appreciated. Even a simple "Yes" receiving streamed events is possible in current version of flutter web on Chrome so you must be doing something wrong on the server side. Or "No" this does not work in current version of flutter web on Chrome but eventually it will/never will, etc.

/cc @vsmenon - seems like a package:http issue?

@vsmenon does this look like a Dart issue? There's not much we can do on the Flutter side. This is a network issue, not a UI issue.

@natebosch @grouma - any ideas / tips?

Just for info, the backend server which generates the event stream is a Spring Boot 2 application using spring-webflux for the reactive streams and the app uses spring-security too. I'd like to think that the problem is somewhere in the backend in terms of the whole Spring configuration but I can't see any errors or anything like that anywhere (backend or frontend in Chrome browser). The response from the call to the event stream API is a 200 and there are no CORS related errors showing up (see my previous comment) or anything like that so I don't know where to look next.
One thing I should add here also is that, originally, the client web app was written using Angulardart and at that time, the web app on a Chrome browser was able to receive the event streams without issue. But now I have rerun the Angulardart client on Chrome and now the Angulardart app is also NOT receiving the event stream just like the flutter web app running on Chrome.
I should add here that, both the Angulardart client and Flutter web client share most of the same code as it is mostly all in a pure dart library with only the Widgets for flutter and Components for Angulardart being on their own projects. So it is important to note here that the dependencies' versions that the Angulardart used from the time it worked to the time now when it no longer works have indeed been updated. But unfortunately the dependency version changes have not been recorded in git commits or anything like that just yet so I can't say specifically what libraries have been updated between then and now (But they would definitely include updates to http library)

I have written a simple nodejs server to test with minimal code:
var http = require("http");
http.createServer(function (req, res) {
  var fileName = "." + req.url;

  if (fileName === "./myserver/events") {
	console.log("Responding to request with events")
	res.writeHead(200, {
	  "Content-Type":"text/event-stream",
	  "Connection":"keep-alive",
	  "Access-Control-Allow-Origin": '*',
	  "Access-Control-Allow-Headers": '*'
	});
	res.write("retry: 10000\n");
	res.write("event: connecttime\n");
	res.write("data: " + (new Date()) + "\n\n");
	res.write("data: " + (new Date()) + "\n\n");

	interval = setInterval(function () {
		console.log("Sending event..")
	  res.write("data: " + (new Date()) + "\n\n");
	}, 1000);

	req.connection.addListener("close", function () {
	  clearInterval(interval);
	}, false);
  }
}).listen(8080, "<IP_ADDRESS>");

And it shows the same behaviour. Running the flutter app an an emulated mobile app the events are received okay. But running the flutter app on Chrome web the events are sent by server but not received by Chrome web app. Chrome debugging shows the network time lapse updating every time the server sends an event but the events themselves are never logged in the console or on the App's Widget like it does when run as a mobile app.

Can you reproduce this issue without Flutter Web and strictly [package:sse](https://github.com/dart-lang/sse)? If so let's open a tracking issue on that package to help get to the bottom of this.

@grouma, how should I get the sse client running on a chrome browser if not using Flutter Web? Should I write it using Angluardart?
Using strictly package:sse and nothing else then the client would not be running on a web browser and then the problem would not occur.

There is an example within the repository. No need for AngularDart. Also there are tests which provide alternative examples.

I will have a look at the sse package and see what it does by itself and report back. I should mention however that the current code where I see the issue doesn't use this package at all. The subject name I gave for this issue includes the word SSE but at no point have I ever used package:sse itself. The code has so far seen the problem on Chrome web when using package:http for the client side and also when using package:eventsource for the client side.

Wow, big confusion on my part.
I thought you were having issues with package:sse as it is used heavily in the Flutter Web development infrastructure. FYI, Flutter Tool depends on package:dwds which depends on package:sse.

Using strictly package:sse and nothing else then the client would not be running on a web browser and then the problem would not occur.

package:sse has a backend handler and a corresponding client that runs in an SSE supporting browser such as Chrome.

The code has so far seen the problem on Chrome web when using package:http for the client side and also when using package:eventsource for the client side.

I'm less familiar with these packages. I'd open up an issue on those packages. Alternatively you can try migrating to package:sse as we know this works without issue on Chrome.

I have tried the Flutter client using package:sse and you are correct in that this package does indeed work in Chrome web. However, the reason it works on Chrome web is because the package uses dart:html. But it is my understanding that dart:html only works for web apps and therefore it would NOT work if the client were to be run as a mobile app (if my understanding is correct).
The other packages I've tried: package:eventsource and pure dart:http I believe have been updated to remove the need for dart:html so that the code can be run in both mobile and web but, it is my guess that, although the code itself compiles and launches in both mobile and web, receiving SSE events does not quite work when running in Chrome web (receiving SSE events only works when running as mobile app).
So, to sum up, from the tests I've run so far:

package:eventsource compiles and runs as both mobile and web but SSE events are only received when running as mobile app.
package:sse runs only on web and can therefore only receive SSE events when running as web app.

I have yet to see a working Flutter web client that is able to receive SSE events when running as a mobile app and as a web app. i.e. a Flutter web client that is able to receive SSE events for both mobile and web apps.

Thanks for the investigation.
As stated above, I'm not familiar with package:eventsource. It's worth opening an issue / reaching out to the author to see if that package was ever intended to work in the browser.

I have yet to see a working Flutter web client that is able to receive SSE events when running as a mobile app and as a web app.

You can create an abstraction on top of both packages and use conditional imports to use the correct package given the platform. I'm not sure how ergonomic that solution may be.
Alternatively, I'm open to a pull request on package:sse that introduces a new client for use on Flutter mobile. We can again use conditional imports so that it's ergonomic for users of the package regardless of the platform.

Sounds like there's not much we can do on the Flutter side. I'll close the issue, but feel free to reopen if you believe something in Flutter isn't working.

Yes, no problem. This Ticket can be closed here at least.
As @grouma suggested I will for the moment use conditional imports so that when running on a browser it uses package:sse and when it is running on mobile it uses package:eventsource via an abstracted interface:
export 'sse_event_clt.dart'
  if (dart.library.io) 'sse_event_mob.dart'
  if (dart.library.js) 'sse_event_web.dart';

It would be good however if package:sse (and/or any other http client package for that matter) could be updated to work in both browsers and mobiles/tablets out of the box. This would be useful in Flutter Web 1.9+ world for sure. @grouma, is there anything you would like me to do in order to get a new pull request going to get package:sse updated to do just this?

It would be good however if package:sse (and/or any other http client package for that matter) could be updated to work in both browsers and mobiles/tablets out of the box. This would be useful in Flutter Web 1.9+ world for sure. @grouma, is there anything you would like me to do in order to get a new pull request going to get package:sse updated to do just this?

I don't know what is required to get package:sse working on mobile. I unfortunately won't have time to investigate it either. That being said I'm more than happy to review any PR you send my way.

Hello @grouma,
I will pull the sse project down from github when I get a chance and see if I can figure out what it would take to make it work for mobiles/tablets too. Hopefully not too much. But I don't know when I'll get round to it. I am still learning Dart and everything Flutter myself so I don't know if I know enough yet to get the package updated but will sure give it a go. Next time I post I will post in that project as there is no point posting on this thread any more.
Regards,
PM

@ponderMuse I am stuck with the same issue..initially using Dio but that doesn't work on Flutter Web. Do you have any updates to share on this issue?
Thanks.

my guess is it is because this https://pub.dev/documentation/http/latest/browser_client/BrowserClient-class.html "It is also unable to stream requests or responses; a request will only be sent and a response will only be returned once all the data is available."


Tags on Organization vs. Users

users.find(:id=>{ID}).tags.class
=> Array

organization.find(:id=>{ID}).tags.class
=>  ZendeskAPI::Collection

Out of curiosity is there any thinking / reasoning behind why calling tags for users and organizations return different objects?

Nope, just an oversight. Released 1.11.5 as a fix.


refactor: export the snackbar with an allert to the toast component

Small refactor, pls review

I've merged the changes from  #21, tell does it look right?

@tripleadev/amnesty

👀


LLM-based support bot for Discord support channel

Compilot for Flow
Grant category

Educational material

Description
Problem statement

Target audience: Developers and users within the Flow blockchain ecosystem.
Evidence for the need: As the Flow ecosystem grows, there's an increasing need for an interactive, educational tool that simplifies the learning curve for both new and experienced users. There's a gap in real-time, AI-driven support for developers and users who are navigating the unique aspects of Flow's blockchain.

Proposed solution
Compilot, an AI-driven support bot, will be integrated into Flow's Discord support channel to provide real-time assistance and educational content. It will leverage AI to answer queries, guide smart contract development, and explain Flow’s features and best practices.
Impact

Enhanced Developer Onboarding: By providing instant, AI-driven answers to technical queries, it will reduce the initial learning curve associated with Flow’s unique blockchain architecture and smart contract development.

Community Engagement and Support: Compilot will act as an always-available resource for both developers and users, fostering a more engaged and active community. This continuous engagement can lead to higher retention rates and a more vibrant ecosystem.

Knowledge Dissemination: By providing accurate, easy-to-understand information and tutorials about Flow’s features, Compilot will be instrumental in spreading knowledge within the community.

Cross-Language Accessibility: The introduction of multilingual support in Compilot will open doors for non-English speaking developers and enthusiasts, significantly expanding Flow’s global reach and inclusivity.

Feedback Loop for Continuous Improvement: User interactions with Compilot will generate valuable feedback and insights into common queries and challenges faced by users.

Milestones and funding

Milestone
Deliverables
Timeline
Risks
FLOW proposal

1 - MVP
AI Bot Development & Integration
2 weeks
Technical challenges in AI integration
2600

2 - Educational Content Creation
Database of Flow-specific tutorials and FAQs
1 weeks
Ensuring accuracy and relevance of content
2600

3- Multilingual Support Implementation
Adding support for multiple languages
1 weeks
Translation accuracy and cultural appropriateness
1500

4 - Beta Testing and Feedback Integration
Methods to give feedback to bot conversations and adapt
1 week
Adapting to user needs and feedback
2600

Total funding proposed: 9300 FLOW
Team

Name
Role
Bio
Contact

Ganesh R
CEO
Seasoned founder with extensive experience in AI and blockchain technologies.
<EMAIL_ADDRESS>

Tri Nguyen
CTO
Expert in AI/ML, with a background in software engineering and blockchain development.
<EMAIL_ADDRESS>

Sodiq
Developer
Experienced in building scalable Discord bot
<EMAIL_ADDRESS>

Firebond has been experimenting with LLM for quite some time to enable efficient marketing and analytics for web3 communities. Before Compilot, we have developed a simple LLM model to generate quizzes from developer docs of blockchain projects - https://thirdwiz.vercel.app/ . This project has never been put into production, but the core tech is solid enough to act as the foundation for Compilot. We believe the idea of Compilot is simple yet can bring direct results to communities, and we envision the bot to become a powerful chatbot that can also support other aspects of web3 community management.

We appreciate your proposal. After careful review we've decided not to proceed at this time. Please stay connected to our Twitter and working groups to stay on top of the latest developments in the ecosystem and new grant opportunities.


Lossless image encoding efficiency

This is not a bug per se, more a question.
I expected VVC to beat JPEG-XL at image encoding and it doesn't seem to be the case.
See the attached files. The source file is RGB (24bpp), the result is yuv420p10le (15bpp?).
The VVC file was produced this way:
vvencapp --input screenshot.yuv --size 2560x1408 --format yuv420_10 --frames 1 \
    --preset slower --qp 0 --output screenshot.vvc

File: screenshot.png Dimensions: 2560x1408
vvencapp: Fraunhofer VVC Encoder ver. 1.8.0 [Linux][GCC 13.1.1][64 bit][SIMD=AVX2]
Configuration warning: disabling MCTF (and BIM), because QP < 17

Input     File : screenshot.yuv
Bitstream File : screenshot.vvc
Real Format    : 2560x1408  yuv420p10  60 Hz  SDR  1 frame
               : encode 1 frame
Rate Control   : QP 0
CODING TOOL CFG: CTU128 QTMin44BTT3333332 IBD:0 SAO:1 ALF:1 (NonLinLuma:1 NonLinChr:1) CCALF:1 Tiles:1x1 Slices:1 WPP:0 WPP-Substreams:1 TMVP:1 DQ:1 SDH:0 CST:1 BDOF:1 DMVR:1 MTSImplicit:0 SBT:1 JCbCr:1 CabacInitPresent:0 AMVR:1 SMVD:1 LMCS:2 (Signal:SDR Opt:0) CIIP:1 MIP:1 AFFINE:1 (PROF:1, Type:1) MMVD:1 DisFracMMVD:1 SbTMVP:1 GPM:1 LFNST:1 MTS:1 (IntraCand:3) ISP:1 TS:2 TSLog2MaxSize:5 useChromaTS:0 BDPCM:2 IBC:2 BCW:2
ENC. ALG. CFG: QPA:1 HAD:1 RDQ:1 RDQTS:1 ASR:1 MinSearchWindow:96 EDO:2 MCTF:0 BIM:0
PRE-ANALYSIS CFG: STA:1 LeadFrames:0 TrailFrames:0
FAST TOOL CFG: ECU:0 FEN:3 FDM:1 FastSearch:1 (SCC:2) LCTUFast:1 FastMrg:1 PBIntraFast:1 AMaxBT:0 FastQtBtEnc:1 ContentBasedFastQtbt:0 FastMIP:0 FastIntraTools:0 IntraEstDecBit:1 FastLocalDualTree:1 IntegerET:0 FastSubPel:0 ReduceFilterME:2 QtbttExtraFast:1 FastTTSplit:1 IBCFastMethod:1 FIM:0 ALFSpeed:0 QuantThr:4 SelectiveRDQO:0
RATE CONTROL CFG: RateControl:0 QP:0 LookAhead:0
PARALLEL PROCESSING CFG: NumThreads:8 MaxParallelFrames:4 WppBitEqual:1 WF:0

POC     0 TId: 0 (  IDR_N_LP, I-SLICE, QP 0)          1263376 bits [Y 75.4249 dB    U 72.1552 dB    V 72.5252 dB] [ET    83 ] [L0 ] [L1 ]

	Total Frames |   Bitrate     Y-PSNR    U-PSNR    V-PSNR    YUV-PSNR
	        1    a   75802.5600   75.4249   72.1552   72.5252   74.1361

screenshot.jxl -  31,749 bytes
screenshot.vvc - 157,922 bytes

I.e. 5 times bigger.
screenshot.zip

Please mind, that there is more to encoding lossless than just setting QP to 0.
At least the following adaptations have to be made:

MCTF=0
SAO=0
ALF=0, CCALF=0
Disable Deblocking (DeblockingFilterDisable=1)
LFNST=0
MTS=0 and ImplicitMTS=0
PerceptQPA=0
set maximal block size to 32 (best just CTUSize=32) to avoid coefficient zero-out

There might be more, but I cannot think of any right now (CostMode?).
Also, VVenC does not really support loss-less, so not sure if the decisions are going to be driven properly. Very experimental.

@adamjw24
Looks like this all requires using vvencFFapp.
The options you're talking about, will the net a higher compression ratio?

As a rule new video codecs always improve I frames encoding efficiency, so I thought that VVC/H.266 would beat "classic" image codecs at that, considering all the new encoding methods incorporated into the standard.

Try VTM, it also has the required configuration changes documented in cfg/lossless/ directory.

Thanks a ton, I will give it a try.

I guess thats the issue for VTM. With the questions regarding VVenC being resolved, I will close this issue now, since we already have a lossless reminder issue (#59).


Devices not longer working: "Light does not support that"

Hi all,
I had a Wemos Mini with 8 devices set up for a long time and working perfectly. All the devices do the same thing send a 433 Mhz signal to turn on/off radio-enabled sockets.
Recently I changed the name of my Wi-Fi at home and since then, some of the devices are not responsive. Usually I only said: "Alexa, light On" and she said "OK", and the light switched on. Now for some of the devices Alexa just tells me "Light does not support that". I hope that's the right translation as my Echo devices are set in German.
I did not change anything other than the WiFi name and Password on my Sketch.
Any help or constructive discussion is welcomed.

Hey there,
are you using the newest version of Espalexa? And do you also have these problems running the example sketch? Also are you using any other libraries in your sketch? And if so which ones?
The OTA library for example doesn't seem to be compatible with Espalexa and leads to similar problems.

Yes, I'm using thr latest update. About the example Sketch I don't know because I never tried as my sketch always used to work...
You can have a look at the Sketch I'm using here, as I did a whole template sketch using your library with the only difference that I used libraries to tidy things up and not have everything together on one .ino file
https://github.com/FelixHauser/Easy_Alexa
Something curious occured yesterday in the middle of the night though. I woke up to several Alexa notifications on my phone. Apparently it detected new devices (the same ones) but this time, they work again. Very weird.
I will keep investigating when I get back home in a couple of days.

Hey,
ok, well I guess it's always possible for something to change and your code and/or the library not to work as expected. Amazon seems to have pushed a lot of different updates for Alexa lately, so maybe it has something to do with that...
Anyway, good you got it to work again I guess, though I know it's not as satisfying when you don't fully understand why it works again.
Btw. I'm not the author of Espalexa, in case it seemed that way (credit goes to AirCookie). I just saw the issue you had opened and thought I'd try and help you out...


Fix label checking logic in #3

Description
Label conditions should be compared with Logical AND(&&) operators... 😂

I think it's time to add some test cases haha

I think it's time to add some test cases haha

yeah we definitely do haha


⭐ Polish animation when card enter inventory

Is your feature request related to a problem? Please describe.
Little swipe, comme un drap, it's beautiful, the card rentre dans la sacoche

Higher priority, it was not clear to live tester that the card was unlocked in the inventory.


"repositories" support only https repositories

This is all valid composer repository annotations according to official composer documentation and it works in heroku/buildpacks:20. We have been migrating recently to heroku/builder:20/22 and we cant overcome this issue.
Only first one with https is not producing crash.
    "repositories": [
       {
            "type": "vcs",
            "url": "https://gitlab.com/devops59460/packages/independent/core.git"
        },
        {
            "type": "path",
            "url": "./../code/stripe"
        },
        {
            "type": "vcs",
            "url": "git@gitlab.com:devops59460/packages/independent/api-client.git"
        }
    ]

Build output log:
    ===> ANALYZING
Image with name "tmp-cnb-image-6784113612" not found
===> DETECTING
heroku/php      0.1.2
heroku/procfile 3.1.1
===> RESTORING
===> BUILDING
[Error: Failed to parse 'composer.lock']
There was an error parsing the lock file; it must be a valid
file generated by Composer and be in an up-to-date state.
Check below for any parse errors and address them if necessary.
The parsed contents were invalid.
Details: invalid value: string "git@gitlab.com:devops59460/packages/independent/api-client.git", expected relative URL without a base at line [114](https://gitlab.com/devops59460/services/stripe-recurring-engine/backend-api/-/jobs/6784113612#L114)3 column 9
You most likely created or edited the file by hand, or a merge
conflict was not resolved properly, resulting in a syntax error
in the file. Refer to the docs for information on re-generating
the lock file: https://getcomposer.org/doc/01-basic-usage.md
Please perform the following steps locally on your computer to
resolve this issue before attempting another deploy:
1) Run 'composer update' to re-generate the lock file
2) stage the lock file changes using 'git add'
3) commit the change using 'git commit'
You can run 'composer validate' locally on your computer for
further diagnosis. Remember to also always keep your lock file
up to date with any changes according to the instructions at
https://getcomposer.org/doc/01-basic-usage.md
If you believe this message to be in error, please report it.
ERROR: failed to build: exit status 1

Another one:
[Error: Failed to parse 'composer.lock']
There was an error parsing the lock file; it must be a valid
file generated by Composer and be in an up-to-date state.
Check below for any parse errors and address them if necessary.
The parsed contents were invalid.
Details: invalid value: string "code/stripe", expected relative URL without a base at line 45 column 9

Same issue here, with a "composer" type repository:
   "repositories": {         "my-yoast": {             "type": "composer",             "url": "https://my.yoast.com/packages/"         },         "wpackagist": {             "type": "composer",             "url": "https://wpackagist.org"         }     }
It works correctly using composer CLI, but when running pack build reports the same parse error

Having a similar issue here, looks like it's having a problem with a "type: git" packages as well.
[Error: Failed to parse 'composer.lock']
There was an error parsing the lock file; it must be a valid
file generated by Composer and be in an up-to-date state.

Check below for any parse errors and address them if necessary.

The parsed contents were invalid.

Details: invalid value: string "git@gitlab.com:hpierce1102/ClassFinder.git", expected relative URL without a base at line 1243 column 9

Running composer validate return no issues with the file, and using composer install in local machine does work correctly.
Version used: heroku/builder:22 with heroku/php:0.1.2

The culprit is the url crate, it cannot parse these URLs, see e.g. https://github.com/rust-lang/cargo/issues/12295 or https://github.com/rust-lang/cargo/issues/12404.
Best option is to parse as a string rather than a URL

@ngatti-tmm it should work when you make it an array of objects instead and drop the keys.
The reason is that otherwise, you do not get order guarantees, and order matters greatly for package lookup when Composer solves dependencies across repositories.
I'll see if there is a way to have a more ergonomic error message for exactly this case.

@ngatti-tmm it should work when you make it an array of objects instead and drop the keys.

Hi! thanks for the info, but I tried with
[
  {
    "type": "composer",
    "url": "https://my.yoast.com/packages/"
  },
  {
    "type": "composer",
    "url": "https://wpackagist.org"
  }
]

And it failed, but now with a strange composer.lock error:
===> BUILDING
Timer: Builder started at 2024-06-03T19:36:07Z
target distro name/version labels not found, reading /etc/os-release file
Running build for buildpack<EMAIL_ADDRESS>Looking up buildpack
Finding plan
Creating plan directory
Preparing paths
Running build command
Timer: Builder ran for 14.965122ms and ended at 2024-06-03T19:36:07Z
ERROR: failed to build: exit status 1
[Error: Failed to parse 'composer.lock']
There was an error parsing the lock file; it must be a valid
file generated by Composer and be in an up-to-date state.
Check below for any parse errors and address them if necessary.
The parsed contents were invalid.
Details: invalid value: string "git@github.com:Yoast-dist/wordpress-seo-premium.git", expected relative URL without a base at line 7060 column 9

But composer install, validate and update they all work without any issues on other environments, also it worked with the older image.

Yup, now you're hitting the "real" issue described at the very top, @ngatti-tmm :)
Working on it!


acme.sh: add bind.dnsutils to PATH

Motivation for this change
This is required for nsupdate functionality.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Hi! The reason CI is failing is because you need to add bind to the arguments at the top of the file so Nix can find what bind.dnsutils refers to, like so:
{ stdenv, lib, fetchFromGitHub, makeWrapper, curl, openssl, socat, iproute, unixtools, bind }:
However, there's a slightly simpler way: instead, add dnsutils to the arguments list, and then drop the bind. prefix. E.g.:
{ stdenv, lib, fetchFromGitHub, makeWrapper, curl, openssl, socat, iproute, unixtools, dnsutils }: and lib.makeBinPath [ .... dnsutils ].

lgtm.


markdown table needs  rather than newline

Developmental_Trauma_Disorder_A_Legacy_of_Attachment_Trauma.pdf
Correctly interprets the table, but does not correctly render the table.
|Criterion|Subcriteria|
|---|---|
|Criterion A: Lifetime contemporaneous exposure to both types of developmental trauma|- A1: traumatic interpersonal victimization
- A2: traumatic disruption in attachment bonding with primary caregiver(s)
|
|Criterion B: Current emotion or somatic dysregulation (4 items; 3 required for DTD)|- B1: Emotion dysregulation
- B2: Somatic dysregulation
- B3: Impaired access to emotion or somatic feelings
- B4: Impaired verbal mediation of emotion or somatic feelings
|
|Criterion C: Current attentional or behavioral dysregulation (5 items; 2 required for DTD)|- C1: Attention bias toward or away from threat
- C2: Impaired self-protection
- C3: Maladaptive self-soothing
- C4: Nonsuicidal self-injury
- C5: Impaired ability to initiate or sustain goal-directed behavior
|
|Criterion D: Current relational- or self-dysregulation (6 items; 2 required for DTD)|- D1: Self-loathing or self viewed as irreparably damaged and defective
- D2: Attachment insecurity and disorganization
- D3: Betrayal-based relational schemas
- D4: Reactive verbal or physical aggression
- D5: Impaired psychological boundaries
- D6: Impaired interpersonal empathy
|

thanks again!

Thanks for raising this issue. We are working on it!

Cells containing newline should now contain  tags.


[SPARK-47833][SQL][CORE] Supply caller stackstrace for checkAndGlobPathIfNecessary AnalysisException

What changes were proposed in this pull request?

SPARK-29089 parallelized checkAndGlobPathIfNecessary by leveraging ForkJoinPool, it also introduced a side effect, if something wrong, the reported error message lost caller side stack trace.
For example, I meet the following error on a Spark job, I have no idea what happened without the caller stack trace.
2024-04-12 14:31:21 CST ApplicationMaster INFO - Final app status: FAILED, exitCode: 15, (reason: User class threw exception: org.apache.spark.sql.AnalysisException: Path does not exist: hdfs://xyz-cluster/user/abc/hive_db/tmp.db/tmp_lskkh_1
	at org.apache.spark.sql.errors.QueryCompilationErrors$.dataPathNotExistError(QueryCompilationErrors.scala:1011)
	at org.apache.spark.sql.execution.datasources.DataSource$.$anonfun$checkAndGlobPathIfNecessary$4(DataSource.scala:785)
	at org.apache.spark.sql.execution.datasources.DataSource$.$anonfun$checkAndGlobPathIfNecessary$4$adapted(DataSource.scala:782)
	at org.apache.spark.util.ThreadUtils$.$anonfun$parmap$2(ThreadUtils.scala:372)
	at scala.concurrent.Future$.$anonfun$apply$1(Future.scala:659)
	at scala.util.Success.$anonfun$map$1(Try.scala:255)
	at scala.util.Success.map(Try.scala:213)
	at scala.concurrent.Future.$anonfun$map$1(Future.scala:292)
	at scala.concurrent.impl.Promise.liftedTree1$1(Promise.scala:33)
	at scala.concurrent.impl.Promise.$anonfun$transform$1(Promise.scala:33)
	at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:64)
	at java.util.concurrent.ForkJoinTask$RunnableExecuteAction.exec(ForkJoinTask.java:1402)
	at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
	at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
	at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
	at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:157)
)

Why are the changes needed?

Improve error message.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

New UT is added.
Was this patch authored or co-authored using generative AI tooling?

No

cc @srowen @viirya @dongjoon-hyun @yaooqinn

cc @gengliangwang @LuciferYang @mridulm WDYT of this approach for stacktrace enhancement? Or do you have other suggestions?

@mridulm In this case, ThreadUtils.parmap catches and wraps the internal AnalysisException with a SparkException, if we throw SparkException directly, the callsites are retained, but propagating SparkException instead of the real exception AnalysisException might not desirable behavior, I believe the original author also wants to keep AnalysisException so unwrap in-place. I checked other callers of ThreadUtils.parmap, they throw SparkException directly so no such issues.

@yaooqinn thanks for reviewing, I addressed all comments

Merged to master. Thank you @pan3793


Message: Class 'Mpdf\Mpdf' not found

Saat print data peserta Message: Class 'Mpdf\Mpdf' not found

jalankan perintah composer install
Pada tanggal Sab, 30 Sep 2023 18.41, Asra Media Teknologi <
@.***> menulis:

Saat print data peserta Message: Class 'Mpdf\Mpdf' not found
—
Reply to this email directly, view it on GitHub
https://github.com/nabilchen96/ppdb-online/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJDWTCG3DH2HRYXXNJT5HYLX5AAQNANCNFSM6AAAAAA5NP57ZU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


feat: add prevent no label execution

For https://github.com/autowarefoundation/autoware.universe/pull/173

@KeisukeShima Although I'll merge this now for autoware.universe, please leave your reviews and questions on this PR if there are any comments. :pray:


Fix Python path variable name in README.md

Python path environment variable should not have an underscore

That's right! Thanks for your carefulness!


Feature/mat/ci

Adds CI build using headless invocation of STM32CubeIDE.  Builds all configurations in the project - Debug and Release.
WiP/PoC for review and incremental adjustment.
Some improvements:

presently using a docker image based on STM32CubeIDE 1.4.0.  More recent versions are available, such as this one, https://github.com/xanderhendriks/docker-stm32cubeide

The repo presently includes the output build directory Debug.  We can move these to github releases.  Automated release builds via an action that creates a release when a tagged build is done performed.

 Development Examples: remove hard-coded PRODUCT_UIDs from sample apps and example repos.


feat(engine): add TextFixEngine

Follow #142
introduce textlint-engine-core and adaptor engines.
textfix-engine.js
textlint-engine.js
are adaptor interface of textlint-engine-core.
Additionally, config has not default formatterName.
default formatterName is defined in engine.

TextLintCore is growing.
I think that TextLintCore need to extensible system like Plugin.

To be clear about difference of linter and fixer.

Linter* process in parallel.
Fixer process in series.


Loses connection after 30 seconds

ReactiveMongo 1.1.0-RC6
MongoDB 5.0.13
Linux 5.19.11-arch1-1
OpenJDK 17.0.5
scala 3.2.1 (also happens with 2.13.10)
with and without playframework

When launching a project using reactivemongo 1.1.0-RC6, such as lila (playframework) or lila-ws (non playframework), reactivemongo seems to lose connection after about 30 seconds. It logs errors, and queries fail.
The problem did not exist on version 1.0.10.
I made a little repository with details and a minimal reproduction environment: https://github.com/ornicar/rm-issue
Note that it does not happen every time with that repo, for reasons yet unknown.
But it does happen every time when running lila or lila-ws.
I'll post more info as I keep digging into this.

I found that creating 2 connections with the same driver allows to reproduce it consistently after just 10s.
  val driver = new AsyncDriver

  (1 to 2).toList map { i =>
    Await.result(
      for
        conn <- driver.connect(List("localhost:27017"))
        db <- conn.database(s"test-rm-$i")
      yield db,
      3.seconds
    )
  }

  for i <- 0 to 12 do
    println(i)
    Thread sleep 1000

  driver.close()

root> runMain Minimal
[info] running Minimal
[sbt-bg-threads-9] INFO reactivemongo.api.Driver - [Supervisor-1] Creating connection: Connection-1
[reactivemongo-akka.actor.default-dispatcher-5] INFO reactivemongo.core.actors.MongoDBSystem - [Supervisor-1/Connection-1] Starting the MongoDBSystem
[reactivemongo-akka.actor.default-dispatcher-5] INFO reactivemongo.core.netty.Pack - Instantiated reactivemongo.core.netty.Pack
[sbt-bg-threads-9] INFO reactivemongo.api.Driver - [Supervisor-1] Creating connection: Connection-2
[reactivemongo-akka.actor.default-dispatcher-5] INFO reactivemongo.core.actors.MongoDBSystem - [Supervisor-1/Connection-2] Starting the MongoDBSystem
[reactivemongo-akka.actor.default-dispatcher-5] INFO reactivemongo.core.netty.Pack - Instantiated reactivemongo.core.netty.Pack
0
1
2
3
4
5
6
7
8
9
10
[reactivemongo-akka.actor.default-dispatcher-6] WARN reactivemongo.core.actors.MongoDBSystem - [Supervisor-1/Connection-2] The entire node set is unreachable, is there a network problem?
11
12
[sbt-bg-threads-9] INFO reactivemongo.api.Driver - [Supervisor-1] Closing instance of ReactiveMongo driver

Will have a look

You can try 1.1.0-RC7-SNAPSHOT

And 1.1.0-275c4ca-RC7-SNAPSHOT

Thank you for looking into this! In which repository are these published?

nevermind, found them in https://oss.sonatype.org/content/repositories/snapshots

Yes, I can't reproduce it in the test project either, no matter how many connections I make :) Awesome! Can't wait to see what the fix looks like :+1:

Thx, will publish fix (idle limit on signaling channel needs some update)

I managed to reproduce it consistently, all it needed was to read from a secondary.
I made a new issue for it: https://github.com/ReactiveMongo/ReactiveMongo/issues/1185

Published 1.1.0-d9cc5339-RC7-SNAPSHOT with more appropriate fix

Awesome, thank you! Will try it out in prod very soon.

Is the code published on another public repo?

1.1.0-RC7 published: https://search.maven.org/artifact/org.reactivemongo/reactivemongo_2.13/1.1.0-RC7/jar


Transfer the repository to the SirixDB organization

Hello,
we could transfer the repo to our origanization, such that all clients are together in one place. Of course you would get all rights as well as you can choose the license and so on :+1:
kind regards
Johannes

Sounds good @JohannesLichtenberger

Let me know if there is anything I need to do to facilitate the process

I created the repo and invited you :-)

Hey I'm sorry can you sent the invitation again, it got lost in my inbox, and when I found it, it had already expired

Done :-)

Okay, I accepted the invitation. Quick question, do I just make a new branch of the repo and make a PR to commit to the repository, or is there another way?

I think you can also transfer your repo or you simply commit/push your stuff to the new repo :-)

Okay pushed my most recent changes to the new Github repo.. I have a local copy of that version of the repository anytime I want to push to the sirix repo I will just update the files and then push it


Speed up Matplotlib import

Working on speeding up Matplotlib import. For now it has to generate the font cache at startup, I'd like to get rid of this.

I'm removing the multiple Pyodide back-ends as we don't really need them (yet?)

looks good to me, and if tests pass it can't be that bad =)


DataTable with select row feature cannot get selectRows out of DataTable properly

Detailed description

Describe in detail the issue you're having.
There is no proper way of getting/pulling selected rows from DataTable.

When using DataTables and a custom render function injected, selectedRows becomes available but you are not able to pull the this info out anywhere else. If you try to save the selected rows to local state, you get a warning/error from React because you should NOT be updating state during a render.
Normally components have a "onChange=" that allows developer to pull/save the values from a component to their state store, which can later be processed.. But because selectedRows is only available in a render, if you attempt to save those values to local state, error is thrown and also leads to infinite loop.
I did talk to a carbon-component's expert 1-2months ago about this, and there doesn't seem to be an alternative
Solution: DataTable needs an onSelectChange= which is triggered to pass values from DataTable to be processed (such as save to local react component's state and/or do operations).

Is this issue related to a specific component? DataTable

What did you expect to happen? What happened instead? What would you like to see changed?
DataTable maintains it's own collection of data.. but with the select row features (user can check/select rows), this information needs to be accessible to the code to process the information collected from the user.
There currently is no proper way to collect selected rows to outside of DataTable. If using the render's selectedRows variable and try to save it to say, local state, React throws errors as you shouldn't be updating state during a render. Components should process/compute values before reaching render(). Also, if we just ignore the error/warning from react and directly update local state.. we will trigger infinite loops.
Infinite loops are caused because render is suppose to be the final step in the react life-cycle, but because we're doing a state update, the life-cycle is triggered again and cycles again.. (Don't know if i worded this properly).
Basically, since we update during a render, react will get "something updated in state, cycle through ... -> componentDidUpdate() -> render()" again. and since we hit render again, we will cycle through again.
So to resolve this issue, we need something like a onSelectChange=, which will trigger a function in the event when someone select/deselects a row. From here, we can process/save the selected Row information and it wouldn't be done through the render()

What browser are you working in? Chrome (unrelated to browser)

What version of the Carbon Design System are you using?
"carbon-components": "^9.1.0",
"carbon-components-react": "^6.5.3",

What offering/product do you work on? Any pressing ship or release dates we should be aware of?
IBM product

Steps to reproduce the issue

Write the DataTable render code as demonstrated in the DataTable documentation.
Change DataTable's render={} to a custom function call, which contains the same contents previously provided.
As a custom function call, selectedRows are now available (which is ONLY available through this custom render)
Try to save the values from selectedRows to local state
Error/Warning is thrown by react:
Warning: Cannot update during an existing state transition (such as within renderor another component's constructor). Render methods should be a pure function of props and state; constructor side-effects are an anti-pattern, but can be moved tocomponentWillMount.
Also, infinite loops will trigger as you are updating in render, which any update to state triggers a new cycle through -> ... -> componentDidUpdate -> render, which again, render will update again, and then another cycle begins infinitely.

Additional information
    dataTableRender({ rows, headers, getHeaderProps, getSelectionProps, onInputChange, selectedRows }) {
		this.setState({
			tableSelectedRows: selectedRows
		});

        return (
            <TableContainer>
                <TableToolbar>
                    <TableToolbarSearch onChange={onInputChange} />
                </TableToolbar>
                <Table>
                    <TableHead>
                        <TableRow>
                            <TableSelectAll {...getSelectionProps()} />
                            {
                                headers.map(header => (
                                    <TableHeader {...getHeaderProps({ header })}>
                                        {header.header}
                                    </TableHeader>
                                ))
                            }
                        </TableRow>
                    </TableHead>
                    <TableBody>
                        {
                            rows.map(row => (
                                <TableRow key={row.id}>
                                    <TableSelectRow {...getSelectionProps({ row })} />
                                    {row.cells.map(cell => (
                                        <TableCell key={cell.id}>{cell.value}</TableCell>
                                    ))}
                                </TableRow>
                            ))
                        }
                    </TableBody>
                </Table>
            </TableContainer>
        );
    }

    render() {
        return (
			<DataTable
				headers={this.props.headers}
				rows={this.props.rows}
				render={this.dataTableRender}
			/>
        );
    }

Hi @David-Yuen! 👋 Thanks for taking the time to write this issue 😄 Definitely agreed that this is an issue currently. I think that, ideally, we'd surface an onChange prop to DataTable that you can use to register for state changes instead of reading directly from the render instance props.
Typically, the way folks can use this information is inside of one of their event methods in the returned markup. For example, if I wanted something to happen on click and that method needed access to the selectedRows then I could pass the rows into that handler. Unfortunately, since this is a render prop setting state from within it is not supported unless it's through a handler.

I've started implementing this feature (as I need DataTable to work properly). Though I cannot decide to what onSelectChange should return.
Should it return:

a single row.id (the row that is affected)
object containing row.id + isSelected (boolean) (the row that is affected)
all rows that are selected (using the getSelectedRows())

?
or is there something else?

think it makes sense to return rowId + isSelected.
I'm writing the code one way that i will not disturb the existing code much. Will look forward to comments or how to write it better afterwards

Thinking more about this, would be great for us to follow Downshift's model as described in: https://github.com/paypal/downshift#onstatechange

From the link @joshblack put here, I got an impression that Downshift has so many states that one-callback-per-state approach doesn't make sense there. Seems that if our data table falls in that category, or should, would be a great thing to discuss.

I think DataTable could potentially, at a high-level we have the following states currently:

Row selection change
Sort state change
Expansion change

Looking forward to if we support a paginated view, or windowed, there could be more. There could also be more with editing potentially.

For reference, here are the state change types for downshift: https://github.com/paypal/downshift/blob/master/src/stateChangeTypes.js
So not too many, definitely more than what we have though, for sure 😄


Not detecting input on Mac OSX 10.10.5

Title pretty much says it all. Everything builds just find, I can setup an input manager and map inputs, but when I check for input, nothing happens.

Could you elaborate on which input devices you use? Mouse? Keyboard? Gamepad?
Also, it sounds like it might work for you on other versions of OS X. Does it?

I've tried detecting keyboard and mouse input. And I have not tried it on any other version of Mac OSX. I'm sorry if this is all quite vague, I just don't have much to supply in terms of information about what's going on.

Sorry, I can only poke around as I don't have a Mac here right now to test. Do you have a window? The window has to be active ("key" in the OS X terms) in order to receive input events.
You could also try placing some breakpoints to see where it goes wrong. For keyboard inputs, the function OnDeviceInput() in lib/source/gainput/keyboard/GainputInputDeviceKeyboardMac.cpp should be called. For mouse inputs, MouseTap() in lib/source/gainput/mouse/GainputInputDeviceMouseMac.mm should be called.

I just finished trying to detect input in a Cocoa application written in Objective C++, just incase the window OpenGL / GLUT has created was some how causing a problem and I have the same issue, everything builds fine, but no input detection. I haven't messed around with the breakpoints much, but I'll give them a try in the morning.

Any news on this issue?
If you could send me<EMAIL_ADDRESS>a minimal project that exhibits the problem, I could try reproducing it here.

I've just tested on 10.10.5 again and it works for me. Is anyone still having problems?


Vendor prefixes aren't parsed as the react documentation recommends

Description
Using a vendor prefix within a style tag results in a warning of react:

Warning: Unsupported vendor-prefixed style property mozHyphens. Did you mean MozHyphens? Check the render method of StatelessComponent.

HTML-Element:

Example

According to the docs of react, vendor prefixes should begin with a capital letter. (https://facebook.github.io/react/tips/inline-styles.html)

Dependencies

react-templates: 0.5.2
react: 15.1.0

I don't understand... you are using a lowercase prefix and React is putting a warning as per documentation. Why should this be an issue?

Usually I write CSS in lowercase. It would be great, if React Templates would output react-compatible JavaScript-styles, no matter how the input is formatted.

Ok, now I get what you mean, you want it to parse style attributes and pass them to react. I guess the same could be said for event handlers (onclick vs onClick).
Currently react-templates does not parse anything, but I understand your point of writing CSS (and HTML in general) the way it's supposed to be (with advantage of intellisense etc etc).

Well, actually this turns to be a bug, because you can't specify an uppercase starting letter even if it's legit.
Currently,
Example

is wrongly turned into:
var templateRT = function () {
    return React.createElement('span', { 'style': { mozHyphens: 'auto' } }, 'Example');
};

(notice the lowercase mozHyphens)
So definitely something to be fixed.

Seems that handleStyleProp is the relevant function. (https://github.com/wix/react-templates/blob/ff5bccdd93e81cbabf4d1837d98de16de0615174/src/reactTemplates.js#L221)

I'm just fixing it

Ok, I've submitted a PR for this. Vendor prefixes taken into consideration are: moz, o, webkit and ms no matter what case you use, it's always correct.


MWPW-142724 | Pointing oneplans flow to offer one sheet

Pointing oneplans flow to offer one sheet
Resolves: MWPW-142724
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/drafts/apganapa/draft-one-plans-pricing?martech=off
After: https://MWPW-142724-2--express--adobecom.hlx.page/drafts/apganapa/draft-one-plans-pricing?martech=off

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

Comparison is base (41e86c4) 33.70% compared to head (c7f704b) 33.76%.

Files
Patch %
Lines

express/scripts/utils/pricing.js
59.37%
52 Missing :warning:

express/blocks/pricing-cards/pricing-cards.js
75.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #660      +/-   ##
==========================================
+ Coverage   33.70%   33.76%   +0.05%
==========================================
  Files         129      128       -1
  Lines       24892    24941      +49
==========================================
+ Hits         8391     8422      +31
- Misses      16501    16519      +18

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


Skybox not found, although set.

Describe the bug
I'm using Enviro Sky & Weather -- it sets the skybox material. I can verify in Scene mode it is set. In play-mode it is still set both at the initial menu screen and then in the play scene. Yet, I'm receiving a warning about Crest not finding the skybox. Also, coloring does look to be incorrect. Screenshots of all below.
Screenshots / video
Here are shots of each step: https://imgur.com/a/MIVTcLj
Log
Validation: There is no skybox set in the lighting settings window. Ocean colour will be incorrect. Alternatively, try the Procedural Sky or Override Reflection Cubemap option on the ocean material. Click this message to highlight the problem object.
UnityEngine.Debug:LogWarning(Object, Object)
Crest.ValidatedHelper:DebugLog(String, MessageType, Object) (at Assets/Crest/Scripts/OceanValidation.cs:49)
Crest.OceanRenderer:Validate(OceanRenderer, ShowMessage) (at Assets/Crest/Scripts/OceanRenderer.cs:1101)
Crest.OceanRenderer:OnEnable() (at Assets/Crest/Scripts/OceanRenderer.cs:311)
Versions
Built-in
Platform
PC - Windows

Here's a screenshot of the Ocean settings, in case it helps: https://i.imgur.com/KIS7mbx.png
This is a branch of Master as of Aug 12th.

Thank you for reporting. The ocean looks correct. It appears to be a false positive. My guess is the Enviro sets the skybox in play mode. Looking at the screenshots, the skybox is set for the menu scene. Is it possible that when you change scenes there is no skybox briefly before Enviro sets it up?

Very odd. Okay, well thank you for verifying. The water looks awfully blue in those screenshots I added, when the sky is all reddish pink. Should that be adjusted automatically or is the base blue color of the water a constant and it's just the reflections/lighting which changes?

Those colours are determined by the colours on the ocean material. The sun and sky contribute colours as reflections.

is the base blue color of the water a constant and it's just the reflections/lighting which changes?

That is pretty much correct.
On a second look, the last screenshot in the gallery doesn't look right. Whenever you see black where the skybox colour should be, then that is normally something wrong. Being a darker scene, it is a bit harder to tell though. If you look at the other screenshots, you will notice a lighter colour on the ocean surface which is very similar to the skybox colour closer to the horizon.
Have you tried running the ocean validator in edit mode? Since it seems to only be a problem with play mode, have you tried launching directly into the scene with Crest?

I'll close this for now due to inactivity. Let me know if the issue persists. I'll see the reply.


layers: Use correct sType

Currently the layer uses the wrong sType for its VkDevicePrivateDataCreateInfo which leads to a validation error and probably unintended behavior.

Author refrogerator not on autobuild list. Waiting for curator authorization before starting CI build.

 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.

Author refrogerator not on autobuild list. Waiting for curator authorization before starting CI build.

CI Vulkan-ExtensionLayer build queued with queue ID 37594.

CI Vulkan-ExtensionLayer build # 708 running.

CI Vulkan-ExtensionLayer build # 708 passed.


Launch/Debug multiple configurations at once

In launch.json you are able to define multiple launch configuration but you can only select one to run with F5. Is there a way to launch two configuration with single Debug command? I want to start and debug server side node app and also use chrome debug extension for client side, all from same VSC instance.
thx

@isidorn fyi

Currently this is not supported. In order to do this you need to open two instances of vscode

In order to do this you need to open two instances of vscode

That was my first try, but when trying to open the same project folder in a new instance, VSCode simply focus on the already opened window, instead of opening the project in the new one.
My team really needs this, it's terrible to stop client debugging, and then attach server one, and then stop, and attach client again. In fact, we're using the browser itself for client debugging, and VSCode for Node debugging, only because of this lack of support.
I have opened https://github.com/Microsoft/vscode-chrome-debug/issues/140 before actually finding this thread.
Is this feature becoming available anytime soon?

fyi @weinand

@buzinas I recommend to create a sub folder for the server or client part of your project. Then you can easily create another VS Code window where you can run another debugger. We are using this approach  and it works great.

@weinand Yes, since debugging both in the same instance would be much harder to accomplish - removing the limitation would be good enough (and probably an easy thing to ship).
Since our codebase huge, and we have lots of build processes / continuous integration etc etc, it would be kind of complex to make a folder refactor as you're suggesting. But I'll make a clone of the repo and give a try, maybe it's simpler than it seems.

@buzinas as I said, the sub-folder can be empty. The only thing that VS Code will create in that folder is another launch.json file. So no folder refactor is necessary.

@weinand Could you please show me an example of that?

@buzinas here we go:

Client and server code are in the same project.
_dummyFolder only exists to be able to open another VS Code window on it.
launch.json in _dummyFolder has 'go up' one level to refer to the server code.

@weinand  Oh, that's really cool. I'll give it a try! Thank you :)

@gregvanl I think my workaround from above would be a good FAQ item for the debugger section?

@weinand Although it's launching both here now, the breakpoints don't work.
It seems that the new instance doesn't send debugging info to the master one. How do you debug the server in that example of your screenshot?

@buzinas yes, you are right, the VS Code on the _dummyFolder has no access to source, so it cannot set breakpoints.
Solution: don't make the _dummyFolder a sub-folder but make it a super-folder:

That's kindda problematic for our project.
But thanks for your time and attention! :)
For the next projects I'll use this approach, and for the current ones, I'll be waiting till we can open the same project in two different instances. Could this be done with an extension? If yes, I would consider developing it myself.

@buzinas but every folder has a parent folder, right? So VS Code can always open it (even if you cannot freely choose the name).
No, an extension cannot fix the issue that a folder cannot be opened in two VS Code windows. (Most likely the problem is just a single if-statement that needs to be removed ;-)

Any updates on the possibility to attach multiple debuggers from the same VS Code window @weinand ? We've got a project with around 40 node processes (and counting), and I really like the ability to debug through VS Code. It's quite hard to follow a chain of requests through multiple services via the debugger if I cannot attach multiple debuggers at once. This was possible via multiple chrome tabs when using node-inspector.

This will be addressed as part of https://github.com/Microsoft/vscode/issues/13426

Since this is being addressed as part of #13426, tagging this one as duplicate to help in our iteration tracking queries


Support for the Soulreaper axe.

Hey!
This PR is to add support for the Soulreaper axe's strength bonus.
There's a comment referencing this PR which is just there to show a potential improvement for the future.
Other than that this is feature complete and tested by myself to be working.

Hey @hugocowan! This looks great and I'm going to be testing it for the next few days if I can get my hands on an axe!

So I thought I would do some videos to show it working.
Without using piety (prayer bonus), the numbers are accurate and I tested with a few different strength bonuses:
https://github.com/j-cob44/max-hit-calc/assets/10326958/e89e8928-256c-468d-a18c-363f190dd8e3
When I enable piety though, the numbers are off:
https://github.com/j-cob44/max-hit-calc/assets/10326958/241d7315-87fd-4680-939c-b7f892f7b41d
And honestly it confuses me what's going on here. Especially since using the special attack at the end actually allows for a higher max hit than usual. According to the wiki article, I thought that the special max hit and standard max hit would be the same, but clearly I'm mistaken.

Hey @hugocowan! Thanks for the videos!
It definitely is very bizarre. What it looks like to me is a rounding error of some kind, which in my experience is usually the case when I start adding special bonuses in. The trick is finding the correct ordering of bonuses as they are calculated by the game.
My initial thought was that this line:
https://github.com/j-cob44/max-hit-calc/blob/3041c6dbc427bfc7bfc671087aa240e6e20481fb/src/main/java/com/maxhitcalc/MaxHit.java#L241
might need to be rewritten into two lines:
double effectiveStrength = Math.floor((Math.floor(strengthLevel * prayerBonus) + styleBonus + 8) * voidBonus);
effectiveStrength = Math.floor(effectiveStrength * soulStack);

and this shouldn't be a problem since soulStack is always returned as at least 1.
However, each time I watch the video you sent, I get less sure of my advice. Could you make two videos wearing the exact same equipment where the only difference is the use of piety?
Let me know what happens, and I'll keep investigating on my end too.

Hey @hugocowan! Thanks for the videos.
According to the wiki, the soul stack boost is additive with prayer bonus. So the fix could be by adding the soulStackBonus to prayerBonus instead of multiplying it into the equation. Prayer bonus is always returned as 1, so by making
https://github.com/j-cob44/max-hit-calc/blob/3041c6dbc427bfc7bfc671087aa240e6e20481fb/src/main/java/com/maxhitcalc/MaxHit.java#L72-L78
return as just 0.06 * soulStack is necessary.
Then before
https://github.com/j-cob44/max-hit-calc/blob/3041c6dbc427bfc7bfc671087aa240e6e20481fb/src/main/java/com/maxhitcalc/MaxHit.java#L241
we could write:
prayerBonus += soulStackBonus;

double effectiveStrength = Math.floor((Math.floor(strengthLevel * prayerBonus) + styleBonus + 8) * voidBonus);

Let me know what happens since the wiki is wrong sometimes, and I'll keep investigating on my end too.
In the meantime, I'm still looking into why the max hit is different on the special attack. Could you make two new videos with the exact same equipment on and the only difference between the two is the use of piety?

Alright, so this is with the updated code, and with no armour on.
No piety:
https://github.com/j-cob44/max-hit-calc/assets/10326958/f4dca399-01c9-425a-a919-863f9bce9a01
Aand piety:
https://github.com/j-cob44/max-hit-calc/assets/10326958/31b2909e-2a57-4857-9606-82e9be1824f2
It looks like there is a bit of a rounding issue still with piety enabled, maybe it's a bit better though. Also the special attack definitely adds damage... so who knows how that should be calculated.

https://github.com/j-cob44/max-hit-calc/assets/10326958/74795a4c-bb0c-4ee7-9bfa-161dda402e55
Here's an example with piety and armour. Looks a lot better than before, but still some small rounding errors and a big difference with the spec hit.

I think there will definitely need to be a special attack calculation made. I reached out to Mod Ash on twitter for more info about the axe's special attack. In the past, he's been very helpful for clarification when I've asked him questions. So we'll see what he says.
In regard to the normal max hit, it definitely seems like a rounding error. Whenever I've encountered errors like these in the past, it's usually been a bit of trial and error at finding the correct order for when bonuses are applied. Unfortunately for us, I believe we are doing it correctly according to the wiki. With your videos, I'm able to work with the formulas on my end and see if I can match the result. Please let me know if you make any progress on it!

Hey @hugocowan, I talked with Mod Ash a little bit and still haven't quite narrowed down why the special attack is hitting higher than it is supposed to. In my testing and with my formula it seems to be 1 stack higher than it the normal attack for some reason. I'm not sure if that is the correct reason but it definitely seems to be an extra 6% higher of a hit.
For the normal hits, I'm almost to confident enough to integrate the code into the plugin. I have one request, just for more testing to see if I can narrow down where the rounding error is occurring. Could you make some videos going through each strength prayer spell up to piety, with no other items equipped? Thank you for you work and help so far.

https://github.com/j-cob44/max-hit-calc/assets/10326958/941d7305-bb1b-4535-913f-71a415cfba82
Here you go, all the strength bonuses in order. Sorry it took a while for me to get around to!

Hey @hugocowan, thanks for the video. Using it, I've found the value that the rounding is off by. The solution seems to be subtracting a constant value between [-0.009, -0.014] from the result of the stack formula. Currently I don't know why this is the case, but it reminds me of how the slayer helm is listed in game as a 15% damage bonus but is actually 16.66_% or 7/6.
Changing here:
https://github.com/j-cob44/max-hit-calc/blob/83fe7c6ad9fa7cd79e81169ca89302c3e2127c31/src/main/java/com/maxhitcalc/MaxHit.java#L240-L244
to:

if (prayerBonus > 1)
{
    prayerBonus += soulStackBonus - 0.009;
}
else
{
    prayerBonus += soulStackBonus;
}

Please try it out and let me know if this cleans up this rounding error. I'm still looking for the complete fix, but we can't let perfect be the enemy of good.

Wow okay, I would not have worked that out myself, nice one : D I'll give it a test.

Hopefully it works well, I've just been running it through the formula so hopefully it continues reproducing the same result when actually implemented. I'm confident the problem is when adding the soul stack bonus to the prayer bonus, I'll still be working on it in the meantime!

Yep, it works. Tried with/without armour + various strength prayers, always correct. Aside from the special attack obvs

Updated the PR

Awesome! Glad its working!
It seems the special attack is much easier to calculate as it is max hit * (0.06 * stack) and the bonus is calculated afterwards, like other special attacks. So it's bonus just needs to be added to the "list" in MaxSpec, at line 165, before the range checks:
if(weaponName.contains("Soulreaper axe"))
{
    return 1 + MaxHit.getSoulStackBonus(client);
}

However, the special attack max hit will then duplicate the stack bonus which makes it a little more complicated. I would propose adding a new argument to MaxHit.calculate() which is just a bool that checks if you are calculating for special attacks purposes.
https://github.com/j-cob44/max-hit-calc/blob/ad1d9823324d6a94ef76c956ca33f23108c39123/src/main/java/com/maxhitcalc/MaxSpec.java#L309-L309
With that, you can pass a true value down through to MaxHit.calculateMeleeMaxHit() and add a check that doesn't add the soul stack bonuses when true.
https://github.com/j-cob44/max-hit-calc/blob/ad1d9823324d6a94ef76c956ca33f23108c39123/src/main/java/com/maxhitcalc/MaxHit.java#L240-L249
You could then encapsulate these lines above with something like if(!specialAttackCalc) { ... }
But you can do it however you would like or if you find a more efficient way! Thanks for your work so far!

Alright, so that just works. I gave it a test briefly and yeah. Perfect scaling for max hit and max spec. Right every time. Let me know if you want more vids or anything like that.

Great! Can you make a video showing the special attack values working?
Also, when you're ready to do so, this can be changed back from a draft to a ready request!

https://github.com/j-cob44/max-hit-calc/assets/10326958/c9ab733d-9ba9-4232-a805-daca3ff62c6b
Here you go, a video with all strength bonuses showing spec hits. Also threw in some strength gear at the end.


Make sync tool support testnets and timestamp forking

What was wrong?

Timestamp forking wasn't supported in the sync tool
Testnets weren't supported by the sync tool
There was a bug with chain_id being hardcoded as 1 in some places.

How was it fixed?
Fixed all the above.
Cute Animal Picture

Codecov Report
Patch coverage: 87.09% and project coverage change: +0.01 :tada:

Comparison is base (4478ef9) 82.48% compared to head (4be7aac) 82.49%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #744      +/-   ##
==========================================
+ Coverage   82.48%   82.49%   +0.01%
==========================================
  Files         570      570
  Lines       31571    31583      +12
==========================================
+ Hits        26040    26054      +14
+ Misses       5531     5529       -2

Flag
Coverage Δ

unittests
82.49% <87.09%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/ethereum/genesis.py
90.47% <77.77%> (+10.47%)
:arrow_up:

src/ethereum/arrow_glacier/fork.py
50.37% <100.00%> (ø)

src/ethereum/berlin/fork.py
95.59% <100.00%> (ø)

src/ethereum/byzantium/fork.py
97.59% <100.00%> (ø)

src/ethereum/constantinople/fork.py
98.07% <100.00%> (ø)

src/ethereum/gray_glacier/fork.py
50.37% <100.00%> (ø)

src/ethereum/istanbul/fork.py
95.19% <100.00%> (ø)

src/ethereum/london/fork.py
92.22% <100.00%> (ø)

src/ethereum/muir_glacier/fork.py
51.44% <100.00%> (ø)

src/ethereum/paris/fork.py
94.23% <100.00%> (ø)

... and 3 more

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Looks like the fork spec tool needs to be updated?


PR commit race condition

Report
What did you do?
Run bundle exec danger --dangerfile=PreBuild.dangerfile --danger_id=prebuild
Run build scripts
Run bundle exec danger --dangerfile=PostBuild.dangerfile --danger_id=postbuild
What did you expect to happen?
Run Danger twice, one before builds scripts and one after, without problems.
What happened instead?
The second danger crashed.
fatal: Not a valid branch point: '<hash>'.
fatal: ambiguous argument 'danger_head': unknown revision or path not in the working tree.

The <hash> in the error message points to a commit that was sent to the PR branch after the build started and as such doesn't exist in the current build git repo when danger runs.
The following build (that's executed on the <hash> commit) will succeed without problem.
Your Environment

GitHub
Jenkins 2.7.2
Danger 3.3.2

Weird, Danger should be fetching commits when GitHub is saying use a commit which doesn't exist in the current git repo, could that be failing somehow?

When trying to run the git commands locally I get:
git rev-parse --quiet --verify <hash> returns the hash ✅
git branch test_head <hash> gives me fatal: Not a valid branch point: 💀
So that's probably what happens.

git fetch solves it

o.0 - hrm, interesting, if you ( or anyone else who is) in  this position again, can you please try see if -git -no-pager show <hash> and see if that returns an empty value
We just switched from that to rev-parse

Alright, will see if I can reproduce 👍

git --no-pager show <hash> gives me fatal: bad object on commit that hasn't been fetched yet and the content of the commit if it has.

Ah now I get it. It should actually be git rev-parse --verify --quiet "<hash>^{commit}" otherwise it just verifies that it's a valid sha1 😆

So now I'm wondering; why does danger need the latest commit instead of just using the current commit?

Made a PR to fix this but I'm still curious about the reasoning for doing this at all :)

So now I'm wondering; why does danger need the latest commit instead of just using the current commit?

This is a complex problem with keeping github metadata and user metadata state in order, in the end we opted to just always rely on github metadata here - as we can have some guarantees there across all CI implmentations

Confirmed that the crash doesn't happen anymore with v3.4.2 🎉
However this makes Danger report status on a future commit on github.

that is probably for the better, GitHub hides comments on older commits right?

Don't think so but the main problem is that the status gets set on the next commit.

That's based on the PR diff that GitHub offers via their API, example if that's saying that the change is related to a specific commit that's all we can work with I'm afraid

So user makes commit A -> build A starts
User makes commit B -> build B starts
Build A reports success -> commit B looks successfull
Not sure if this can be fixed easily and it's not really problem if the repo have other branch requirements set (which they should).

@orta Yeah I guessed that it was like that... I'll keep an eye out for any solutions in the future but I think that this original issue is fixed 👍


[AIRFLOW-2554] Enable convenience access to in/outlets in templates

Make sure you have checked all steps below.
JIRA

[X] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2554
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

This change allows easy access to inlets/outlets in templates e.g.
File(name="myfile") can be accessed by {{ myfile.XXXX }}. In case
it the name collides with dag level names (e.g. "dag") in can be
accessed by {{ inlet['dag'] }}.
Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Current tests cover this.
Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[X] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Added.
Code Quality

[X] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3453 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3453      +/-   ##
==========================================
+ Coverage   76.78%   76.79%   +0.01%
==========================================
  Files         203      203
  Lines       15112    15116       +4
==========================================
+ Hits        11603    11608       +5
+ Misses       3509     3508       -1

Impacted Files
Coverage Δ

airflow/models.py
87.26% <100%> (+0.06%)
:arrow_up:

Continue to review full report at Codecov.

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

lgtm! would be useful to add pydoc for inlets/outlets in the BaseOperator constructor

Good point. Will.do

@bolkedebruin  any progress with that?


Remove logdata and msg fields from JSON audit log rule elements

Writing macro-expanded strings to JSON elements during the post-logging
phase can be misleading, because it's possible that variable contents
(such as MATCHED_VAR) could have changed after the rule match, altering
their expected contents. Writing macro-epanded audit data really only
makes sense when the macros are expanded immediately following the
rule match. See issue #1174 for more details.

Merged.


Question: Async data

Hello!
What would be a suitable scenario for an application that needs to load remote configuration from an API or load configuration from a file? One approach could involve creating an Initialization scene and transitioning to another scene after the configuration is loaded. However, a challenge arises when using the editor mode and pressing 'Play' on a different game scene. In such cases, it becomes necessary to load the Initialization scene. Unfortunately, calling ReflexSceneManager.LoadScene from the IInstaller in the Menu scene (whether in Awake, Start, or InstallBindings methods) can lead to freezing issues in the editor. (2022.3.7f1)
How do others handle asynchronous calls with Reflex?

Sorry for late reply, I didnt received any notification about your question.
The usually best approach is to as you described, to load all the remote configuration on a boot scene for instance, construct all the dependencies you need, and open your game scene by:
ReflexSceneManager.LoadScene("Gameplay", LoadSceneMode.Single, descriptor =>
{
    descriptor.AddSingleton...
    descriptor.AddSingleton...
    descriptor.AddSingleton...
});

So you still respect the immutability of the container while providing for example different implementation for your services depending on a remote configutation.
Regarding the editor issue, I dont think you should care much about it, but yeah I agree, is annoying to remember to open the correct scene before entering playmode, however I see this as the industry standard, I never worked on a project that does not force you to do this.
However I found this repo here:
https://github.com/CSaratakij/SceneSelector
that seems to solve this issue, so you can definetely try this out


Path to jQuery fixed in three html example files

Good call—going to handle this manually since the compiled zip has a conflict. Thanks for pointing this out!


[youtube] Don't download all API json pages when using the arguments "--dateafter --break-on-reject"

Checklist

[X] I'm reporting a site feature request
[X] I've verified that I'm running yt-dlp version 2021.12.01. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
https://www.youtube.com/c/BritishMovietone/
https://www.youtube.com/user/PewDiePie
Description
For certain youtube channels it is possible that all of the API json pages are downloaded even if not required.
As an example:
yt-dlp --dateafter now-1day --break-on-reject https://www.youtube.com/c/BritishMovietone/
This command downloads thousands of pages before exiting despite no videos matching the --dateafter argument (as of writing).
The desired behaviour would be that the program exited on realisation that the first page resulted in no matching videos.
This is on the assumption that the pages are in chronological order but with debugging the youtube downloader, the assumption appears to be valid.

After reading the comments on #1018 let me explain a use-case:

A users list of youtube subscriptions exists in a csv file (retrieved via takeout.google.com).
The csv file is parsed and all videos in each channel that is less than a day old are downloaded.
These videos then get served via a personal media-server.

Fixed again by https://github.com/yt-dlp/yt-dlp/commit/a30a6ed3e49a0e037af6d5e26ecef3f3eba67d33
Need to use the extractor arg youtubetab:approximate_date
e.g. yt-dlp --dateafter now-2week --break-on-reject https://www.youtube.com/c/BritishMovietone/ --extractor-args youtubetab:approximate_date


Unified Login in Debian

High,
I am trying to use the nodes in Debian ( Raspberry-Pi Bookworm).
The following error shows when the Unified Login node is activated:
"Error: ENOENT: no such file or directory, open '.node-red\Siemens.Automation.WinCCUnifed Project CA.cert.pem'"
The certificate is in the node-red folder and is properly selected with the Browse button.
Thanks in advanced

Version 1.2.0 released


A number of calendar chart examples are broken

One-line summary [问题简述]
On the website demos, a couple of calendar examples are broken:
https://ecomfe.github.io/echarts-examples/public/editor.html?c=calendar-effectscatter
https://ecomfe.github.io/echarts-examples/public/editor.html?c=calendar-vertical
https://ecomfe.github.io/echarts-examples/public/editor.html?c=calendar-horizontal
Version & Environment [版本及环境]

ECharts version [ECharts 版本]:
Browser version [浏览器类型和版本]:
OS Version [操作系统类型和版本]:

Expected behaviour [期望结果]
ECharts option [ECharts配置项]

option = {

}

Other comments [其他信息]

I am having some wired problems with calendar also. This problem happens in every browser of my machine (Windows 10 x64). But not on my VM (Linux, Windows 10 and Android).
I got this error when I try to render my chart: Uncaught TypeError: Cannot read property '0' of undefined
at Object.buildPath .
Trying to debug the problem, I realized that are missing some points in the shape.points array (mising 4 and 5)

But I cant figure out  why this points are missing in the first place.

Thanks!
It has been fixed just now.
See #6543


Can I migrate existing pages from BrowserCMS to CamaleonCMS?

I have a existing Rails Application and I am current using BrowserCMS. I want to start moving pages over from BrowserCMS to CamaleonCMS. I am having difficulties moving my existing pages from BrowserCMS to CamaleonCMS. Any suggestions would be greatly appreciated.

Hi @owen2345 I don't mean to be pushy but I am working on a project for my company and could really use your expertise on the subject of this issue.

Hi@mkernsNCR,
please contact me on skype: owen-2345
Regards,

Thanks for your reply @owen2345! Would you be free for a Skype call around 3pm? My colleague will be joining us.
Regards!

Yes, that is ok!

Perfect!


Does using rest-auth effect any of all-auth other functionality?

Hello,
Thanks for reviving this project. It is very helpful.
Quick question I see all-auth is a dependency for social auth, which is fine, but I am also using all-auth because of its user invitation feature. Is that something that will still work if I add dj-rest-auth?
If so has that feature been added to the rest auth functionality? Or would that need to still use all auth traditional forms?
Any insights you can provide would be much appreciated.

Hi @Redhaus
Invitation endpoints are out of scope (for now) in this package, but I think it is straightforward to add your own views that do invitations.
My advice is to spend a few minutes studying all auth's invitation code (the form code and view code), then port that same business logic to a new DRF API endpoint.

I'm going to close this issue for now.

Will do. Thanks for the advice and the direction.
Benjamin Read  |  Creative Director  |  Redhaus  |<PHONE_NUMBER>  |
Redhausdesign.com http://redhausdesign.com
On Sat, Apr 18, 2020 at 11:49 AM Michael<EMAIL_ADDRESS>wrote:

I'm going to close this issue for now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jazzband/dj-rest-auth/issues/45#issuecomment-615925248,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCVN3PYDEKAFCJHMHRCJG3RNHY5NANCNFSM4MKOHEAA
.


feat: support insert multiple assets via SDK fuel.addAsset

Closes #584

I found two things;

When adding an asset manually inside the Wallet. Assets page, add a new asset, add an asset_id that already exists, add name and symbol, and click on save. The Wallet just updates a current asset without alerting it that already exists and blocking the action.
On the origin info is showing the wrong domain.

About point 1. I think we can do in another issue / PR as it's orthogonal to this issue/PR objective
https://github.com/FuelLabs/fuels-wallet/issues/598


Add test cases for TypedProperties implementation

Please add test cases for TypedProperties implementation. The example is org.apache.shardingsphere.underlying.common.config.properties.ConfigurationPropertiesTest
The remains test cases need to be finished are:

[ ] Test case of OrchestrationProperties
[ ] Test case of ZookeeperProperties
[ ] Test case of EtcdProperties
[ ] Test case of NacosProperties
[ ] Test case of ApolloProperties

I want to try the ZookeeperProperties test cases @terrymanu

I want to try the ZookeeperProperties test cases @terrymanu

Nice, anything could be discussed about here.

I want to try the ApolloProperties test cases @terrymanu

@meAsky Welcome

Test case of ApolloProperties has been merged into the master in another pull request

I want to try the EtcdProperties  test cases

@terrymanu I want to finish the rest test case, Test case of OrchestrationProperties and  Test case of NacosProperties.


Update components and TF providers

Component
Current Version
Latest Version
Issue
PR

kubernetes
v1.20.4
v1.20.5

calico
v3.17.0
v3.18.1

etcd
v3.4.14
v3.4.15

cert-manager
v1.0.3
v1.2.0

contour
v1.10.0
v1.13.1

dex
v2.27.0
v2.28.1

external-dns
3.4.9
4.9.3

metallb
v0.1.0-789-g85b7a46a
v0.9.6

metrics-server
2.11.2
2.11.4

openebs
2.6.0
2.7.0

prometheus-operator
0.43.2
v0.46.0

rook
v1.4.6
v1.5.9

velero
2.13.6
2.15.0

aws-ebs-csi-driver
0.9.0
0.9.14

cluster-autoscaler
1.1.0
9.9.2

linkerd
2.9.0
2.10.0

istio
1.7.0
1.9.1

node-problem-detector
v0.8.5
v0.8.7

TF Provider
Current Version
Latest Version
Issue
PR

Packet
3.2.1
v3.2.1

AWS
3.12.0
v3.33.0

Azure
2.41.0
v2.52.0

Local
2.0.0
v2.1.0

Null
3.0.0
v3.1.0

CT
0.7.1
v0.8.0

Missing in the list seems to be the Terraform TLS provider

Missing in the list seems to be the Terraform TLS provider

Please feel free to edit the list as you see fit.

Only metallb is left, as this uses our images and seems to be a custom built I didn't yet touch it. #1457 focuses on the Helm components

We've updated most components. Let's update metallb and close this issue.

Working on updating metallb


spanish translation

hi @jgthms, thanks for your tutorial, it has helped me teaching other people the first steps into javascript.
May I translate it into spanish?

thanks!


After subclassing ARSegmentPageController for customising header KVO for "segmentToInset" doesn't work?

I added this line in my viewDidLoad :
[self addObserver:self forKeyPath:@"segmentToInset" options:NSKeyValueObservingOptionNew context:NULL];
But when call comes here on this line
CGFloat topInset = [change[NSKeyValueChangeNewKey] floatValue];
Its crashes with this message.
-[NSConcreteValue floatValue]: unrecognized selector sent to instance 0x7fc7ebed4aa0
Can you explain what I am doing wrong here? Or can you put your customised header code in sample project that would really help.

It's 'segmentTopInset' ,not 'segmentToInset'.
发自我的 iPhone

在 2015年5月17日，21:24，kidsid49<EMAIL_ADDRESS>写道：
I added this line in my viewDidLoad :
[self addObserver:self forKeyPath:@"segmentToInset" options:NSKeyValueObservingOptionNew context:NULL];
But when call comes here on this line
CGFloat topInset = [change[NSKeyValueChangeNewKey] floatValue];
Its crashes with this message.
-[NSConcreteValue floatValue]: unrecognized selector sent to instance 0x7fc7ebed4aa0
Can you explain what I am doing wrong here? Or can you put your customised header code in sample project that would really help.
—
Reply to this email directly or view it on GitHub.

[self addObserver:self forKeyPath:@"segmentTopInset" options:NSKeyValueObservingOptionNew context:NULL];
doesn't help. Its same crash. In-fact  I added "segmentToInset" by looking into your sample project only.

Can you paste your code here?
发自我的 iPhone

在 2015年5月18日，01:04，kidsid49<EMAIL_ADDRESS>写道：
[self addObserver:self forKeyPath:@"segmentTopInset" options:NSKeyValueObservingOptionNew context:NULL];
doesn't help. Its same crash. In-fact  I added "segmentToInset" by looking into your sample project only.
—
Reply to this email directly or view it on GitHub.

in .h file its
@interface GWProfileViewController : ARSegmentPageController
and then in viewDidLoad
[self addObserver:self forKeyPath:@"segmentTopInset" options:NSKeyValueObservingOptionNew context:NULL];

and then KVO:-
-(void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change context:(void *)context {
CGFloat topInset = [change[NSKeyValueChangeNewKey] floatValue];
NSLog(@"top inset is %f",topInset);

if (topInset <= self.pager.segmentMiniTopInset) {
    self.pager.title = @"Siddarth Chaturvedi";
    self.pager.headerView.imageView.image = self.blurImage;
}else{
    self.pager.title = nil;
    self.pager.headerView.imageView.image = self.defaultImage;
    //self.blurImage = [self.pager.headerView.imageView.image applyDarkEffect];
}

}
App crashes on method first line :
CGFloat topInset = [change[NSKeyValueChangeNewKey] floatValue];

@AugustRush able to figure it out whats happening?

Did you call [Super viewDidLoad]? Or,can you pasted more details codes?
发自我的 iPhone

在 2015年5月19日，20:16，kidsid49<EMAIL_ADDRESS>写道：
@AugustRush able to figure it out whats happening?
—
Reply to this email directly or view it on GitHub.

@AugustRush Yeah! I will. But can you add the custom header example in your sample project? That would helpful for anyone who is trying to use this :-)

Okay,I will add a demo in a while.
发自我的 iPhone

在 2015年5月20日，13:01，kidsid49<EMAIL_ADDRESS>写道：
@AugustRush Yeah! I will. But can you add the custom header example in your sample project? That would helpful for anyone who is trying to use this :-)
—
Reply to this email directly or view it on GitHub.

You can check it out
发自我的 iPhone

在 2015年5月20日，13:01，kidsid49<EMAIL_ADDRESS>写道：
@AugustRush Yeah! I will. But can you add the custom header example in your sample project? That would helpful for anyone who is trying to use this :-)
—
Reply to this email directly or view it on GitHub.

@AugustRush Thanks!


Add support for Feature-Policy header

More details:

https://wicg.github.io/feature-policy/
https://www.chromestatus.com/feature/5694225681219584

Now supported in Chrome 60


Drop unused test/cmd package

Changes
I haven't found any usage of this package through knative or knative-sandbox and it gets rid of a weird dep... so yay?
/assign @julz @dprotaso

cc @cardil - I recall you were looking to replace some bash scripts with Go - is there any utility in keeping this?
cc @kvmware

+1 from me for removal. This isn't related to what we should be doing with knative/hack ie. we shouldn't shell out left and right. Also, there are already packages that can do that better.

/lgtm