Refactor sendMail transporter calls

In the current implementation, we have a blob for error checking and retry every time that the app has code for sending emails.  This issue code is seen at:
https://github.com/sahat/hackathon-starter/blob/master/controllers/user.js#L354
https://github.com/sahat/hackathon-starter/blob/master/controllers/user.js#L439
https://github.com/sahat/hackathon-starter/blob/master/controllers/user.js#L536
Note that the flash messages and console log messages vary between the above three examples.
Ideally, we want to refactor this into a reusable function and make each controller more compact and readable.  In the case of the above examples, this function can stay in controllers/users.js to make it easier to find and avoid confusing newer developers.

Hi, I would love to give this a try, could you please assign this to me? Thanks!

Can you submit a PR in the next few days?  Thanks!

@YasharF
If I get this correctly,
We need a common function for handling the error ? (i.e. the catch block )
If not, can you please provide some more details on this?

@YasharF  I went through the contact site. The recaptcha has an error stating wrong RECAPTCHA_SITE_KEY whic belongs to the owner....

@bkyogesh28 this github issue is for refactoring sendMail transporter calls.  If you have found a bug or need support, please create a new github issue with the details.  Thank you!

@YasharF, just submitted a pull request here: https://github.com/sahat/hackathon-starter/pull/1105
Please let me know if any other changes are required. Thanks!


Fedora packages failures

mulpyplexer & registry return no urls, and now appear in the Fedora dataset, probably needing to be listed as known failures in tests/test_fedora.py , or need matching to alternative PyPI names.

need to check  the .spec

registry is Windows registry - bit odd that it is on Fedora, except as a mingw package
python-registry is http://www.williballenthin.com/registry/ which is 404

https://pypi.org/project/mulpyplexer/ is at https://github.com/zardus/mulpyplexer .  That can be manually mapped.


added nic.sh to replace nic.py for now

Not as functional as it could be but it's more functional than the current

Oops. This has unnecessary files. Will clean soon


Alignment problem on ARM

co19/LibTest/typed_data/Float32List/Float32List.view_A01_t02: Crash
co19/LibTest/typed_data/Float32x4/toUint32x4_A01_t02: Fail
co19/LibTest/typed_data/Float64List/Float64List.view_A01_t02: Crash

@fsc8000 Does https://github.com/dart-lang/sdk/commit/fb75e1e5e481334260b2fd467393a367fc74389a help with this?

https://github.com/dart-lang/sdk/issues/22151


Datatables inside AdminLTE problem

Dear AdminLTE & friends,
I am creating a Bible with datatables. Searching feature works for me here:
http://torah.byethost14.com/crud/____en_define_table_layout_min.html
But it does not work for me inside the AdminLTE here:
http://torah.byethost14.com/crud/_index2_start_again.html
I have tried loading the scripts before the < / head >
I also tried before the < / body >
What am I missing?

Hey @alexbible,
replace the line:
var table = $('table.table').DataTable( {

with:
var table = $('#talmidimEdition').DataTable( {

thank you Dimitri :)
Here's my progress: http://torah.byethost14.com/crud/pages/tables/_data.html
I really appreciate it,
Alex

@alexible I like ur bible project. What problem are u facing now after the progress. I would like to help u.

Good morning @kressly . Our Abba in heaven is always with us and puts us together :) I'm glad to meet you! Here's my contact info<EMAIL_ADDRESS>// skype id: "smuglovsky" // and here's my work page: www.facebook.com/granjasolidaria (it's in Spanish but you'll enjoy the pictures). I'm normally back from the villages at 9 pm (Ghana time)

Due to some neglect in my part, many issues weren't addressed for some time. Some of these issues are no longer relevant since a solution might have been found. Therefore, we decided to close all issues that are over a month old. However, if the issue still persists and you'd like help, then by all means repost your issue and we will consider it again. In the mean time, we will close this post.
Thanks for your understanding and we apologize for any inconvenience <3.


Rebrand the client

Fix #1233

Revert compiler.xml, modules.xml, faf client prod.xml

Revert compiler.xml, modules.xml, faf client prod.xml

No they are broken/wrong since a long time

This needs further investigation on what certain things do

Do you really want to call the client (and everything, like log files) "Official FAF Client" rather than just "FAF Client"?

Well not sure

FAF client is better than Official Faf Client I think, but thats just my opinion


Move the 'flutter build' commands from CI into MSBuild

The CI workflow is meant to provide the resources and execute the build system. Having it to know which Flutter project to builder (it could be either the current slide show or the new OOBE coming soon) and build it before invoking the actual build system that will output the appx bundle is a bad smell.
Instead we can let the build system (MSBuild) to handle building whatever dependencies the appx might have, only requiring the CI workflow to know that it must provide the Flutter command and that’s it.
This PR applies some renaming to turn it more subproject-agnostic in terms of whether we build the slide show or the OOBE application, as well as removing the build commands from CI workflow YAML files, leaving that to the actual build system (MSBuild) to handle.

This run shows that we won't stop bundling the Flutter artifacts into the x64 part of the bundle.


Hello World example?

This looks like a very interesting project and I'd love to play with it, but I'm having issues getting a Hello World working just from the readme. Is there a simple self-contained project out there that contains some basic RiotTS code, as well as step-by-step instructions on how to run it?

I've just written one here:
https://github.com/nippur72/RiotTS-helloworld
(or see it: http://nippur72.github.io/RiotTS-helloworld/)
it's made in plain command-line style (tsconfig.json) because wasn't sure if you had Visual Studio or not.
Write here if you have questions or need assistance :-)

Wow, thanks for the speedy response - that's exactly what I needed. I'm not using Visual Studio so tconfig,json is perfect for me. I've got a few other questions, but I'll create new issues for them to keep things clean.
Thanks!


Exclusions of disallowed names, fixes #18

Thanks! Sorry, that it took more time to address this.

No worries - thanks!

Which release / when will this be included in Hackage / Stackage, btw?

@danwdart I'm planning to get https://github.com/psibi/shell-conduit/pull/17 merged and release a new version. If this is something you want quickly - I can release without it.

Fair enough, well I'm not in a huge rush. Just curious. I have just pointed my stack dep in my project(s) to my fork for now.
Ta

@danwdart A new version has been uploaded to Hackage. Thanks for the patience!

Great, thanks!


dashboard.js

Session time-out omitted.

@xthxr don't remove the session timeout completely. Make it 30 minutes for security purposes and if possible please Notify users when their session is about to expire, giving them an option to extend it.


UI: Update search components and text field placeholders

TL;DR
Updated UI elements in the trip planner feature for improved user experience.
What changed?

Changed the background color of SearchStopRow from secondary to onSecondaryContainer
Updated the search placeholder text from "Search" to "Search for stop"
Modified text field placeholders:

"From" changed to "Starting from"
"To" changed to "To Destination"

Why make this change?
These changes aim to enhance the user interface and improve clarity:

The new background color may provide better contrast or visual appeal
The more specific search placeholder guides users to search for stops
The updated text field placeholders offer clearer instructions for input

These modifications should result in a more intuitive and user-friendly trip planning experience.

#255  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite

Merge activity

Oct 26, 12:22 AM EDT: A user merged this pull request with Graphite.


ISSUE-1112 Improve leaf node assignment algorithm

Implementation of balancing algorithm suggested by Francois:

Compute the overall cost.
Compute a target cost per node. It could be something like ceil(overall_cost / number of node) * (1 + margin).
For each split, sorted in decreasing order, assign the split with the highest affinity.
Remove node from list as its load exceed the target cost

Currently, job is ordered by cost desc, then split_id.
How about: just ordered by cost desc, no need to involve split_id. hdyt @fulmicoton

I don't think it is harmless.
Can you think of a metric to measure if an algorithm is good or not?

Excuse me, you mean the whole algorithm or the part best of 2 @fulmicoton
And yeah, to measure an heuristic algorithm is a hard question I think. There is edge case which makes the load pretty good or pretty bad.

The best of 2 part is useless. It is hurting the algorithm performance.
Now for how to measure the performance of the algorithm. Well we want 2 properties:

(P1) We want to assign splits to the server with the highest affinity
(P2) We want to make sure that the work is balanced between the servers

It is possible to build a cost function combining the two, or it is possible to express P2 as a constraint, and try to optimize a cost function expressing P1.
The best of 2 part is hurting P1 greatly, and is not necessary anymore.
P2 is already well covered by removing nodes that exceed their budget.

Oh, I don't know: The best of 2 part is hurting P1 greatly, and is not necessary anymore. It will be great help if we could provide some metric like: cpu usage, trace, response latency.
Why I think best of 2 is fine for both case: normally, there might be 1-3 nodes will receive same split_id job due to old/new algorithm.
Anw, I think you have full metrics on a lot of datasets. Let me update the code :+1:

The best of 2 thing will hit the node with the best affinity with a probability of 50% .
Francois algorithm without the best of 2, will hit the node with the best affinity with something much higher.
It depends on the query, but probably consistently above 90%.

Yeah, I know the distribution probability when use best of 2. Just don't know how the performance is impacted when having 2 diff node execute same split_id job.
Thank you for figuring it out. I addressed your comment. PTAL

@linxGnu The idea is to that raising this probability increase our cache hit rate.
For instance we cache a so-called hotcache. If we assume we have just the capacity to store num split/ num servers, we will end up with a cache hit rate about 90% without the best of 2, and around 50% with the best of 2.

@fmassot I'd like your opinion, see comment inline.

@fmassot I'd like your opinion, see comment inline.
@linxGnu I think we need a bunch of unit test for important edge case (I think your implem cover them correctly already).
In particular, I would like to have a unit test for 1000 nodes, 1 split. (the point is that one node need to get the split, even though its cost is much large than total_load / num_nodes)


create new user in Dashboard

How to create new user in dashboard ? cant find ?

To do this in the dashboard, you can do so with the "Sign up with Email" flow. We hope to expose a way to do this as a superuser in the near future, as well.

And then from the Admin view:


Plus symbol at top overlaps tabs and does not allow the last tab to close

Environment
Windows build number: [run "ver" at a command prompt]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

Expected behavior

Actual behavior

Plus symbol at top overlaps tabs and does not allow the last tab to close.

Thanks for reporting this!
Please make sure you fill out the bug template when you make the bug; otherwise, we all get an e-mail saying "there is a bug report" with no info in it.

@Srijan1214 Could you be more specific? Possibly post a screenshot of the issue you're describing?
As a workaround, you could always just close the window to close the last tab...

This is almost certainly a duplicate of #651

I meant #444


Add documentation for how to handle exceptions in skills

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
We need to tell the customer how to handle exceptions in skills properly

tracked by #914


refactor -- start putting together the Winx frontend properly

The entire winx binary relies on the monolithic winx.cpp. While there are advantages for development & testing, it's probably time to start pulling this apart and creating a proper wrapper on top of V8's startup.
Probably want to support the following:

Create Function/Object templates for registration
Create way to load polyfill(s) scripts (embedded or passed in)
Separate CLI parsing logic and setting up globals

Investigate if we need to actually use globals or is there something better

Separate script execution

These few items should clean things up a bit, especially for better JNI integration (right now it's just a hook around the monolithic start up function).

Frontend is a bit better now following the creation of the WinxEngine. I feel there is still a lot of room for improvements.


[ENH] Figure out QC metrics for reconstruction outputs

It would be enormously useful to have QC metrics for reconstruction outputs for sanity checking and pipeline benchmarking.
One great option are some of the fib qc metrics from DSI Studio. @frankyeh are there any of DSI Studio's fib QC measures that can be accessed via the commandline interface?

There is only one for QSDR reconstructed FIB files: the R squared value
between the subject image and template image. It can be accessed by
unzipping the fib.gz file, renaming it as .mat, and loading it as a MATLAB
file.
Another is the neighboring DWI correlation for SRC files.
Frank
On Tue, Sep 15, 2020 at 1:32 PM Matt Cieslak<EMAIL_ADDRESS>wrote:

It would be enormously useful to have QC metrics for reconstruction
outputs for sanity checking and pipeline benchmarking.
One great option are some of the fib qc metrics from DSI Studio. @frankyeh
https://github.com/frankyeh are there any of DSI Studio's fib QC
measures that can be accessed via the commandline interface?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4NAJPES7J4Q5JZBB7TSF6QLFANCNFSM4RNPEMWA
.

What about the algorithm that calculates:
Fiber coherence index: 946058
Fiber discoherent index: 115522

Would this be suitable as a qc metric?

Yes, but it may not be as good as NDC. and thus I haven't put it on the
command line yet.
Frank
On Tue, Sep 15, 2020 at 2:49 PM Matt Cieslak<EMAIL_ADDRESS>wrote:

What about the algorithm that calculates:
Fiber coherence index: 946058
Fiber discoherent index: 115522
Would this be suitable as a qc metric?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178#issuecomment-692905751,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4NSLREORWCC6D4KKS3SF6ZMLANCNFSM4RNPEMWA
.

If you're interested, I'd be happy to benchmark this on a lot of data! I really like that it works on any reconstruction

Sure. I will list it on my higher priority list and should be ready in the
command line in days.
Frank
On Tue, Sep 15, 2020 at 3:00 PM Matt Cieslak<EMAIL_ADDRESS>wrote:

If you're interested, I'd be happy to benchmark this on a lot of data! I
really like that it works on any reconstruction
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178#issuecomment-692912991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4NOHEWNDQRS77QAD6TSF62V7ANCNFSM4RNPEMWA
.

updated.
how DSI Studio supports qc on fib
dsi_studio --action=qc --source=my.fib.gz
On Tue, Sep 15, 2020 at 3:09 PM Fang-Cheng Yeh<EMAIL_ADDRESS>wrote:

Sure. I will list it on my higher priority list and should be ready in the
command line in days.
Frank
On Tue, Sep 15, 2020 at 3:00 PM Matt Cieslak<EMAIL_ADDRESS>wrote:

If you're interested, I'd be happy to benchmark this on a lot of data! I
really like that it works on any reconstruction
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178#issuecomment-692912991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4NOHEWNDQRS77QAD6TSF62V7ANCNFSM4RNPEMWA
.

@frankyeh you are amazing! I will get in touch off github about possible benchmarking strategies.

@frankyeh did the command line call for calculating QC change with this update?

Should still be the same. If not, I will fix it.
Frank
On Tue, Sep 22, 2020 at 12:31 AM Matt Cieslak<EMAIL_ADDRESS>wrote:

@frankyeh https://github.com/frankyeh did the command line call for
calculating QC change with this update?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178#issuecomment-696502618,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4I76QJZ32MNPKZYLBDSHASCDANCNFSM4RNPEMWA
.

@frankyeh the directory where the results file is written changed, no need to fix anything. I am noticing some different behavior in how bvecs are read. Is there any reason that the newest DSI Studio might be flipping the y direction even with --check-btable=0?
dsi_studio from January:

dsi_studio from last week:

Yes, I received a request to flip y if DSI Studio reads b-table from FSL's
bvec because of the orientation convention differences.
I am sorry that this causes a problem here.
If there is a suggestion for a better solution, I will fix it right away.
Frank
On Mon, Oct 5, 2020 at 1:34 PM Matt Cieslak<EMAIL_ADDRESS>wrote:

@frankyeh https://github.com/frankyeh the directory where the results
file is written changed, no need to fix anything. I am noticing some
different behavior in how bvecs are read. Is there any reason that the
newest DSI Studio might be flipping the y direction even with
--check-btable=0?
dsi_studio from January:
[image: image]
https://user-images.githubusercontent.com/170026/95112730-5496b980-070f-11eb-8a4c-04c28a445462.png
dsi_studio from last week:
[image: image]
https://user-images.githubusercontent.com/170026/95112840-7a23c300-070f-11eb-97cd-641655d6c3cd.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178#issuecomment-703779449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4OGPTW55AB4RANMTEDSJH7TTANCNFSM4RNPEMWA
.

would it be possible to revert to the previous behavior? Or add a flag to the commandline to use FSL Mode? The prior behavior was consistent with Dipy and is heavily depended on in qsiprep

also, the fiber coherence index picked up a problem with the flipped data right away. Very excited to calculate this by default from now on!

Just committed a change to revert it.
Sorry for the hassle.
Frank
On Mon, Oct 5, 2020 at 1:45 PM Matt Cieslak<EMAIL_ADDRESS>wrote:

would it be possible to revert to the previous behavior? Or add a flag to
the commandline to use FSL Mode? The prior behavior was consistent with
Dipy and is heavily depended on in qsiprep
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PennBBL/qsiprep/issues/178#issuecomment-703785280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDI4KMDRO6FKUV26ZDQ4TSJIA3XANCNFSM4RNPEMWA
.

Thanks Frank!!


Lombok Dependency is possibly outdated

I'm currently running IntelliJ IDEA 2019.3, and I see the warning:
Lombok Dependency is possibly outdated
	Project "EdgeServer" and Module "EdgeServer.test" seem to have outdated lombok dependency added.
	Configured version "1.18.10", but there is at least version "1.18.12" already released

I'm using the lastest vesion of the freefair plugin:
id "io.freefair.lombok" version "4.1.6"

which is supposed to pickup lombok 1.18.12.
I've tried to force the issue by adding:
annotationProcessor 'org.projectlombok:lombok:1.18.12'

to no avail.
The warning seems spurious.

i use version 2019.2.4 and i have the same problem

Looked on this issue now deeper:
gradle plugin io.freefair.lombok version "4.1.6" simple use lombok in version 1.18.10 internally.
The warning is absolutly correct.
You need to update your gradle plugin to 5.2.1 to get the newest lombok version.


"Invalid librdkafka version 901ff, expected at least 903ff" on Linux

Description
When running version 0.11.0-RC1 on Red Hat linux I get an exception: "Invalid librdkafka version 901ff, expected at least 903ff". Looks like there was a mistake with the native library for that platform?
How to reproduce
Build and run an application that uses the library on RHEL7 in .net core.
Checklist
Please provide the following information:

[ ] Confluent.Kafka nuget version: 0.11.0-RC1
[ ] Apache Kafka version: 0.10.1
[ ] Client configuration:
[ ] Operating system: REHL 7
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

you'll need to install librdkafka manually for RHEL currently (the nuget package is broken and we won't fix this in the current release).
thanks for letting us know you need this.

Hm, It seems very easy to add the binaries to the package? I can build them for you if that is the problem.
The error message above was caused by me still having a dependency on the old rdkafka library which ships with an old version of the library for RHEL... That's a nasty incompatibility.

Bundling pre-built librdkafka binaries is not a problem in itself, but librdkafka's dependencies are.
It depends on openssl, cyrus-sasl, zlib, liblz4, etc, and there is no good way to handle those dependencies automatically from high-level bindings, afaik.
And the errors those dependencies may cause (mismatch, missing, etc) are much harder to identify and troubleshoot for both the end-user and us as maintainers.
So at this time we've decided not to bundle librdkafka but depend on the system's standard packaging methods to provide librdkafka (through Confluent repositories to guarantee the latest version).

@roederja2 yep, you should uninstall all rdkafka-dotnet dependancies, as its shipped librdkafka lib would override confluent-kafka-dotnet or even system ones

I think we could provide useful binaries for common distributions that make it easy to use with not too much effort. Statically link zlib/liblz4, .net core already depends on openssl so that will be available in some form, which leaves only cyrus-sasl. Maybe we could even make that optional?
That would make it massively easier to run confluent-kafka-dotnet on .net core, having to bundle a matching librdkafka out of band isn't trivial.

The librdkafka.redist package (which is based on your original package! :) ) already provides this, and NuGet should sort out the dotnet's dependency on it, so I think we're basically fine here, except for the missing RedHat-built .so files which we'll look into automatic soon.

For info, it should be possible to use existing rdkafka and confluent-kafka side by side (to compare or for upgrading), just prevent copy from the runtimes of rd.kafka.internals (see here for references : https://docs.microsoft.com/en-us/nuget/consume-packages/package-references-in-project-files)
This mean only the dll (or so or other native lib) from librdkafka.redist will be copied to runtime. As librdkafka is API compatible (\o/), this won't be an issue (but it will change some params - read release notes of librdkafka since 0.9.1)

@edenhill Which confluent RPM repository are you referring to?

http://docs.confluent.io/current/installation.html#installation-yum

Sorry to piggyback on this issue (let me know if I should open a new one), but I am trying to use the nuget Confluent.Kafka (0.11.6) in a .Net Core 2.2 project on a windows machine and I got the exact same error. Is there some kind of workaround that I can use to fix this issue?

This error indicates the .net runtime is finding and loading an earlier version of librdkafka than expected. Are you referencing any librdkafka.redist package explicitly? (if so, remove that). Do you have librdkafka.dll installed anywhere else on your system? (if so, try removing this). Finally, try version 1.0-RC1 which fixes many problems with 0.11.6.

@mhowlett I was using the NServiceBus.Transport.Kafka package which seemed to cause the issue. After removing it, it was complaining that it couldn't find the librdkafka library. I then installed the latest librdkafka.redist (0.11.6) package and it worked.
Thanks for the quick reply!


Preserve transaction reference

While internal references are useful, "end-to-end" references are even more so to enable pairing of invoices and payments:
            <Refs>
              <MsgId>37962162598</MsgId>
              <EndToEndId>NOTPROVIDED</EndToEndId>
            </Refs>

Please extend the model to store those too.

Thanks for the suggestion, I've implemented this in version 0.2.0 :)
statement.transactions[0].ref.message_id
statement.transactions[0].ref.end_to_end_id


scrollEnd event is not fired when using snap and mousewheel on bottom or top

Case of  using snap and mousewheel options on bottom or top
scrollStart is fired and scrollEnd is not fired.
also, when using mousewheel, scrollStart is fired twice sometimes.

I found the same problem. ScrollEnd is not fired when scroll to the top or the bottom, when I use snap option.

Same issue here.
Any suggestions?

In order to receive a scrollEnd, I used this trick:
iscrollObject.on('flick', function() { /*Do what you need to do on scrollEnd */});
It worked fine for me without side effect but this is clearly a workaround...

@lasiboy I don't understand your method. Can you give me an example ?

I have an idea.

I have an idea.
The demo below shows my problem. Drag and hold one column to the top, then release when it can't move any further. In this case scrollEnd won't be fired.
http://sandbox.runjs.cn/show/iupihxtw
Then it occurred to me, why not make it move further, and let it bounce back? this way it can snap to the exact position automatically, thus can prevent the problem.
My solution: set iScroll option "bounce" to true.


[ML] Update Delete Inference Endpoint for dry_run and force options

Following on  https://github.com/elastic/elasticsearch/pull/109123 and https://github.com/elastic/elasticsearch/pull/109384, this change updates the elasticsearch specification to include the dry_run and force options for the Delete Inference Endpoint API.

Clearly I don't fully understand what the correspondence between the java code and the specification is. I'm going to do some reading and come back to this.


[bug]: Splash Screen does not complete if the user is not signed in

Currently, the Splash Screen just closes to launch the Sign In activity if the user is not signed in. Hence the splash screen does not get time to complete its animation.

@Sbiswas001 Assign this to me


Updated KeyVault module

Change
Added comments for better separation;
Added publicNetworkAccess parameter;
Added additional log setting for diagnostic settings;
Corrected usage of diagnosticSettingsName;
Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update (Wiki)

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (readme)
[x] I did format my code

Confirmed local that it works


Adding lane detection

Detailed Description
Use a sensor or a computer vision model to enable automatic lane detection. The information thus obtained can be used to improve the data published on the current_heading topic. (might result in interface changes!)
Definition of Done

see description above

Effort Estimate
No response
Testability
No response
Dependencies
No response

This is duplicate of #441. As this issue is already in Backlog should we transfer findings into this issue? Or  should we close this one and move to the already existing issue?


Please could you release this to maven central?

It would be nice to be able to grab this as a dependency from mc.

Okay, I'll cut a new version on Monday.

Version 1.13.0 has been released.


refactor: Remove vendor logging

Asana task: Remove vendor logging code from Screens
Vendor logging will now live in screen_checker. https://github.com/mbta/screen_checker/pull/14

[ ] Needs version bump?

Good thinking! See https://github.com/mbta/devops/pull/220.


feature: Set Meltano system database_uri for different Meltano Environments in meltano.yml env block

Feature scope
Configuration (settings parsing, validation, etc.)
Description
I would like to be able to set a different Meltano system database_uri (this setting) under an env block in meltano.yml. Currently, the only way to change this setting across environments is to change the MELTANO_DATABASE_URI environment variable (&/or .env file variable).
This is not currently possible.
It would look something like this:
environments:
- name: dev
  config:
    database_uri: postgresql://test:test@test.com:5432/postgres
    plugins:
      loaders: ...

Why is this a useful feature?

Keeping the system db separate in different environments is helpful. I like to limit shared resources between environments bc sharing them makes accidentally overwriting from one environment to the other more likely)
Allowing this setting to be included in meltano.yml makes it more visible and transparent to all users, and reduces the amount of manual differentiating that needs to be done in other places.

There is a discussion of this in Slack here: https://meltano.slack.com/archives/C01TCRBBJD7/p1664314402975189

@qbatten we recently made an explicit decision to not allow meltano level settings in environments. https://github.com/meltano/meltano/issues/6247 is the issue.
Environments are for execution configuration management and not for Meltano itself.

I like to limit shared resources between environments bc sharing them makes accidentally overwriting from one environment to the other more likely)

That's worth diving into more as we should make it harder / impossible to overwrite them. Are you seeing overwrites related to schedules (https://github.com/meltano/meltano/issues/6848) or in other scenarios?
FYI @aaronsteers in case you have additional thoughts.

@tayloramurphy and @qbatten - I do think this is a good idea, and we should try to support it. As with most meltano-level settings, we have to make sure we load that setting at the exact right point in the call stack so that it is able to initialize in the proper way. Since there are low-level repurcussions, we just need to make sure the setting is loaded in time to honor it everywhere it is needed.
That said, we were just chatting internally on a related topic of allowing prod environment, for instance, to have a different settings or state backend than the one used for dev.
Right now database_uri serves as the backend for: state, logs, and settings/secrets. In the future, we may carve these out as separately configurable - although this is still a decision we are thinking through.
So, feature-wise, I do think it would be nice to support a different database_uri per environment. We need to make sure it is robustly tested but it seems valuable to have that option.
I'm curious, @qbatten, as a workaround, is it sufficient to specify this using an environment variable that you pre-configure before running Meltano CLI commands? The extra manual step of configuring the env variable is not ideal, but I'm curious if this makes a valid workaround for you for the time being. What do you think?

@aaronsteers yep, that's what I'm doing right now and it works fine! This doesn't feel critical to me, it just would be nice to have.
@tayloramurphy re: #6848, I havent run into any overwrites bc I was concerned about that happening and therefore decided to use different system DBs for dev and prod. I do think these two issues are sort of linked, in that they're both about making environments work smoothly for users.

Somewhat related, but it seems that the database_uri configuration option currently is also not expanding any $ variables from the .env or terminal env. For example having
database_uri: ${DATA_VAR_MELTANO_DATABASE_URI}

And running
DATA_VAR_MELTANO_DATABASE_URI="postgresql://postgres:postgres@some-non-existant-host/meltano" meltano config meltano

outputs
{
   ...
  "database_uri": "sqlite:///./.meltano/meltano.db",
  ...
}

Somewhat related, but it seems that the database_uri configuration option currently is also not expanding any $ variables from the .env or terminal env. For example having

Good point.
Logged here:

https://github.com/meltano/meltano/issues/6920


feat: implement duplication of RootObject children

This commit adds a method duplicate to the RootObject class. The
method creates a copy of self and tries to copy some of the relations
too as well as the reverse ManyToOneRels.
 Last but not
least it adds a duplicate link to the detail_views/detail_generic.html
template.

this looks great thanks! simple and generic.
would it be difficult to write a simple testcase? one object with one attribute and one relation that gets copied, where we check that the attribute is moved over while the relation gets dropped.

I've now added a testcase that creates a copy of a RootObject instance and checks if there is an new id and if the name is the same (note: this will fail once the name gets removed from the RootObject). The duplicate method does in fact also copy the relations - but I've not included a test for that at the moment. I tried to manually create a Property and a Triple, but when I tried to add contenttypes to the property, something in subj_or_obj_class_changed raised an exception. Maybe this is some functionality that needs a full blown ontology to be tested, who know.

And given that this test will fail once the TempEntity is removed it will remind us that the function is no longer working.

Ehm, no, it shouldn't fail

In addition it would be great if you could also add a ticket stating the fact that this relies on the object_inheritance manager and needs fixing as soon as TempEntity are removed. This ticket should also be linked to #6

I don't understand what the connection between TempEntity and objects_inheritance are - the latter is an attribute of RootObject


Mail sent problems

At the moment it is not possible to send an e-mail automatically via the contact form on https://slynite.com/en/contact/. We are already in clarification with our hoster about this problem and hope that this problem will be solved soon.
In the meantime, please contact us via the e-mail address given on the site.

Our hoster has fixed the problems. Mails should be sent again without any problems.


java-class-tools-dev

Not sure, there are some questions, debug look at it.

I totally forgot that there was a dev branch :laughing:
Thanks.


Variables in Kotlin: Android Basics with Compose

URL https://developer.android.com/courses/pathways/android-basics-compose-unit-1-pathway-1of codelab
In which task and step of the codelab can this issue be found?
The progression pathways.
Describe the problem
The progression pathways arnt being completed with a green check after being finished. The first one did but after watching the videos and progressing they arnt getting a green check mark. Which is making it hander to pick up where left off.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @Mcclatchey
Thank you for writing in with your issue.
We were unable to duplicate your issue.
If you are still having a problem and have the time, please create a new issue with all the requested information so we can pass it along to the appropriate team.
Hope you are enjoying the course!
Regards.


[TC-FAN-2.3 & 2.4] RockSupport and WindSupport attribute value doesn't satisfy the spec

Reproduction steps
./chip-tool fancontrol read rock-support 1 1

[1688392778.789627][3332:3334] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0202 Attribute 0x0000_0007 DataVersion:<PHONE_NUMBER>
[1688392778.789796][3332:3334] CHIP:TOO:   RockSupport: 3

./chip-tool fancontrol read wind-support 1 1

[1688392809.598691][3335:3337] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0202 Attribute 0x0000_0009 DataVersion:<PHONE_NUMBER>
[1688392809.598838][3335:3337] CHIP:TOO:   WindSupport: 3

Bug prevalence
Whenever I do this
GitHub hash of the SDK that was being used
7ae5919a046a18c7e60537aff814977ee424b9cf - Master
Platform
raspi
Platform Version(s)
RPI - 4, 8GB RAM
Anything else?
Spec Reference: https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/FanControl.adoc#5-data-types

As per the SPEC the maximum support value for RockSupport can be 2 and WindSupport can be 1 but the value reproduced is 3, This issue is reflected in write function of Rocksetting and Wind setting as we are able to write more than the supported modes.

As per the SPEC the maximum support value for RockSupport can be 2

No, this is a bitmap.  The maximum value is 7.  The value 3 means "RockLeftRight and RockUpDown are supported, RockRound is not supported."

and WindSupport can be 1

It can be 3, which means "both SleepWind and NaturalWind are supported".


Cannot authenticate Spotify

What version of Music Assistant has the issue?
2.1.4
What version of the Home Assistant Integration have you got installed?
2024.8.2
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
I cannot authenticate with Spotify. The Authenticate button does nothing but bring up a loading icon.
How to reproduce
Click Authenticate button.
Music Providers
Spotify
Player Providers
N/A
Full log output
It won't let me attach it for some reason, it's greyed out (MacOS).
Additional information
I've been keeping an eye on the documentation recently for the Spotify setup process and noticed it change several times, most recently I noticed this line appeared:
"You can only configure the Spotify provider from a device which is on the same subnet as the MA server (and not via a VPN)"
I am on the same LAN, but I have a fear that when you say "same subnet" that might not actually be what you mean? I am on the same subnet, of actual real IP4 addresses, not NAT. Let me know if this makes a difference...
What version of Home Assistant Core are your running
2024.8.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Please update to 2.2.2

Ahh I did check to see if there was one, but it's not showing up.

@KE55ARD is this now resolved after updating?

Nope not been able to update, not seeing anything newer than the v2.1.4 that I'm on.

Well that is a HA or docker problem that you need to resolve

Indeed, I have finally received the update after a HA core update and managed to authenticate again. Thanks!


Timeline for releasing TNG?

It looks like this is in the middle of a rewrite.  My team is considering using this package and aren't sure which version to use.
Is there somewhere that this type of information is being tracked?
Thanks

I've been tracking progress using the 2.0 milestone. We're pretty close to having something ready to release in an alpha/beta state, but any feedback or testing would be very much appreciated.
I think @dantswain was hoping to do a documentation pass, too.

Also, a little more detail in case it's helpful:
The 1.x line was a simple wrapper around the Apache Thrift Erlang library and required integration with the thrift compiler tool. It worked well, but if you wanted more of an Elixir-style interface on top of the generated Erlang code, you'd need to use something like Riffed.
The 2.x line reimplements everything in native Elixir code so there's no longer any dependency on the Apache Thrift toolchain. This lets us use Elixir-native concepts and constructs from end to end. Because it has so many new pieces, we've been taking our time with it, but we use it internally, and it works quite well.
I recently merged (3d237d95b04e4bb74febff100e48f44aec7db877) the thrift_tng branch into master in preparation for an official 2.0 release. I'll remove the thrift_tng branch soon. It's now stale, but I didn't want to break too many people who were pointed directly at it while they transition.

We also are using the 2.x master, and it works well. I would highly recommend going that way for a new project.

What's the status of this? There seem to be changes coming in, but no 2.0.

The 2.x master code is actively in-use within Pinterest. We've unfortunately just been distracted from putting together a fully documented, etc. 2.x release.
The next move might be to take what's there now and package it as 2.0-RC1 or similar prerelease, but I don't have a timeline for that.


Card Link

Hi when using the following
card = dtc.Card(
            link='https://linkhere.com',
            image='image_path_here',
            title='Title text',
            description='Description text',
            badges=['Badge 1', 'Badge 2', 'Badge 3'],
            git='https://githublinkhere.com',
            dark=True
        )

Is there a way to force the link not to open in a new tab?

@firmai please upgrade to the next release and you will have this feature,
You just need to pass the argument:
openNewTab=False
default is true
for more details check the docs
https://romanonatacha.github.io/dash_trich_components/
pip install dash-trich-components
or
pip install --upgrade dash-trich-components


Suggestion - add a 64 bit mode

Hello,
These days, 32 bit is basically obsolete, with most computers being 64 bit. Could you please add a 64 bit mode? I'll be happy to help if you could explain the codebase to me.
-Ndrewxie

Many new OS's mainly ship as 64 bit binaries, so obtaining good OS images would be easier.

I'm too lazy to create a new issue for this, but performance is become a big problem. v86 is rendering the text "as you go", instead of grouping text operations together (create an array, and make all render operations modify the array instead. In the end, just dump the array's contents to the canvas). That would increase the rendering speed dramatically and make it feel less laggy.

The architecture that v86 is using has been phased out by MANY OS developers, meaning that it should be replaced as quickly as possible. Arch linux main page:
Due to the decreasing popularity of i686 among the developers and the community, we have decided to phase out the support of this architecture.

32 bit IS obsolete (duh), but sadly it isn't as easy as just adding " a 64 bit mode". v86 has been around for ~3 years now and its emulation level is about what a Pentium 1 can do. The first Pentium came out on March 22, 1993, making the support level on v86 about 24 years old. Even if it could handle 64-bit architectures many operating systems still wouldn't be supported. Before 64-bit support is added it would make more sense to finish and close these for added support:
https://github.com/copy/v86/issues/48
https://github.com/copy/v86/issues/97

I don't think that 32-bit is obsolete, the vast majority of applications still compile on 32 bit, and most operating systems still support 32 bit (including most BSDs, several major Linux distributions and all Windows versions). If you're looking for a small Linux that still supports 32 bit, try Alpine Linux.
Besides, JavaScript doesn't support 64 bit operations (or addressing more than 4 GB of memory), so this would negatively impact performance. I'm working on a Web Assembly port, which will eliminate this problem.
And the last problem is that this will take several man-months, which I don't have time to invest besides my job.
Closed as a duplicate of #133.

#97

Luckily PAE isn't a prerequisite for 64 bit and in fact PAE mode precludes 64 bit mode.

I'm too lazy to create a new issue for this, but performance is become a big problem. v86 is rendering the text "as you go", instead of grouping text operations together (create an array, and make all render operations modify the array instead. In the end, just dump the array's contents to the canvas). That would increase the rendering speed dramatically and make it feel less laggy.

Contributions are welcome, but please use a profiler before making such assumptions.


maximum number of digits per phone number

in some countries more digits are used than the application allows entry, and for importing a spreadsheet for better identification and editing should be created as an example for your own information

Hello, there is no digits limit, I invite you to verify if the number you are inserting is in the format <country_code><number>, for example if the number is 123456789 and the country is Italy (+39), you have to write<PHONE_NUMBER>9.
If this is still not working, please send me an example of what is the problem.
I am here to listen if you mean that a different entry should be better.
For now, I am not able to import spreadsheets, but only a text file with the comma-separated values. Do you mean that the numbers.txt file is not really useful to understand?

Hello, tnk to answer,
i would like to know where i could extended the amount of digits for gerate
numbers.
i would know 2, how i can make a translate for you ( for portuguese).
So you are try to saying i can import any ".txt" with numbers separated for
comma(,) but is able to have numbers like(+55) =<PHONE_NUMBER>9
( i find here to put de 55 so it's fine )
Em qua., 15 de jun. de 2022 às 15:06, Alberto Pasqualetto <
@.***> escreveu:

Hello, there is no digits limit, I invite you to verify if the number you
are inserting is in the format <country_code>, for example if the
number is 123456789 and the country is Italy (+39), you have to write
39123456789.
If this is still not working, please send me an example of what is the
problem.
I am here to listen if you mean that a different entry should be better.
For now, I am not able to import spreadsheets, but only a text file with
the comma-separated values. Do you mean that the numbers.txt file is not
really useful to understand?
—
Reply to this email directly, view it on GitHub
https://github.com/albertopasqualetto/Bulk-WhatsappWeb-Sender/issues/3#issuecomment-1156776972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APWAKW2TEYPHJV2HQZYDQ4TVPILTNANCNFSM5Y4ELQAQ
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

--
Italo Yuri,
Gerente em TI, Tec. Tecnologia da Informação, Analista de Sistemas,
Desenvolvedor de OS Mobile
+55 21 984284441 - TIM
@.***

I do not understand what you mean with this:

i would like to know where i could extended the amount of digits for gerate numbers.
I said that, when you run the program, you have to select a plain text file that contains the numbers (it is not necessary to be a .txt, but it also can be a .csv or whatever extension).
So exactly like this:
So you are try to saying i can import any ".txt" with numbers separated for comma(,)

(+55) =<PHONE_NUMBER>9 number should be transformed to<PHONE_NUMBER>749
A file should appear like this:
5521982584749,<PHONE_NUMBER>910,5512331131,<PHONE_NUMBER>9

For the translation, I have no idea how to implement it right now, but foremost it's not a priority: there are other technic parts to implement first.

From the latest release (v1.5.0) you can add numbers without country prefix.
Since any translation is not currently planned, I am going to close this issue.


Use separate query ID space for different DNS servers

Related: #3972
Motivation:
DnsNameResolver limits the number of concurrent in-progress DNS queries
to 65536 regardless the number of DNS servers it communicates with. When
the number of available DNS servers are more than just one, we end up
using much less (65536 / numDnsServers) query IDs per DNS server, which
is non-optimal.
Modifications:

Replace the query ID and context management with
DnsQueryContextManager

Eash DNS server gets its own query ID space

Result:
Much bigger query ID space, and thus there's less chance of getting the
'query ID space exhaustion' error

@serphacker @dmk23 PTAL

@normanmaurer and you as well once you are ready after the flight.

Just a quick note - organization of query ID space per server should be as efficient as possible. E.g. allocating 65K arrays per server would be problematic if we are using 10K servers or so...
The whole idea of using lots of resolvers is to get around the rate limits, so the actual number of queries per server is not expected to be very large ...

Quick note - we should not allocate too much memory per server. E.g. if we allocate 65K arrays per server and have 10K servers that would be a problem.
Note, the whole reason for using lots of servers is to get around the resolver rate limits. So the actual number of concurrent queries per server is not expected to be very large...

@dmk23 This PR doesn't use an 64k-element array anymore. It uses a hash map.

Anything else?

Ship it

Am 09.11.2015 um 04:56 schrieb Trustin Lee<EMAIL_ADDRESS>Anything else?
—
Reply to this email directly or view it on GitHub.

Cherry-picked into 4.1 (120ffaf880b81c260141c9e46cec239ac235aeb9) and master (e33ee56d5b2a57abe9b8ebe150ed10bfd71820fa).
Thanks @trustin !

Thanks, @normanmaurer :-)


Wrong Square Colors for the Chess Summary/Difference Visualization.

Currently chessboard square's coloring is inverted.
Screenshot:

Solution:
Change the light color square to dark color and the dark color square to light color.
The colors (gray, blue, or brown) are not important just that the darker one is in the left bottom corner.
Screenshot:

fixed in badc592c39c1e6764db44dcd0e8dbae27251bdfd
if this change works on your end, this issue can be closed

Works, too :+1:
Thanks


updated cache doc

Proposed changes
Added description in cache backend section
Types of changes
What types of changes does your code introduce to the project: Put
an x in the boxes that apply

[ x ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after
creating the PR. If you're unsure about any of them, don't hesitate to
ask. We're here to help! This is simply a reminder of what we are going
to look for before merging your code.

[ x ] I have read the CONTRIBUTING doc
[ x ] I have signed the CLA
[ ] I have added tests that prove my fix is effective or that my
feature works
[ x ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in
downstream modules

Further comments
updated changes as per Issue

I have read the CLA Document and I hereby sign the CLA

@daminichopra that's great, thanks alot!! there's just a couple of things we could improve on, see my comments above.

recheckcla

recheckcla

@daminichopra that's great, thanks alot!! there's just a couple of things we could improve on, see my comments above.

I have updated commit after some changes. Please let me know if any modification is required. Thank you

I have updated commit after some changes.

Good stuff!

Please let me know if any modification is required

Yes, if you could please add a sentence to explain what caching has to do w/ geo-location. If you like you can just copy-paste the example sentence I added as a review comment.
Also, the makrdown linter is moaning about lines being too long:
docs/manuals/index.md:308:81 MD013/line-length Line length [Expected: 80; Actual: 181]
docs/manuals/index.md:331:81 MD013/line-length Line length [Expected: 80; Actual: 123]

If you could please fix those two lines above and make sure any other lines you're going to add get truncated at 79 chars. So we can keep the linter happy :-)

I have updated email for commit but CLA is getting failed again and again.

I looked into that and it's really beyond me. The error message is so cryptic it really doesn't help. We're going to need some time to figure out why the CLA action is failing.
Thanks again for your contribs!!

I have updated commit after some changes.

Good stuff!

Please let me know if any modification is required

Yes, if you could please add a sentence to explain what caching has to do w/ geo-location. If you like you can just copy-paste the example sentence I added as a review comment.
Also, the makrdown linter is moaning about lines being too long:
docs/manuals/index.md:308:81 MD013/line-length Line length [Expected: 80; Actual: 181]
docs/manuals/index.md:331:81 MD013/line-length Line length [Expected: 80; Actual: 123]

If you could please fix those two lines above and make sure any other lines you're going to add get truncated at 79 chars. So we can keep the linter happy :-)

I have updated email for commit but CLA is getting failed again and again.

I looked into that and it's really beyond me. The error message is so cryptic it really doesn't help. We're going to need some time to figure out why the CLA action is failing.
Thanks again for your contribs!!

@c0c0n3 , I have Added a sentence in doc and fixed markdown linter issue.
Also, I am looking into cause of failing CLA issue.
Thanks for suggestions for this PR.

@daminichopra the cla issue was due to the fact that some of the commits where signed as "root", i fixed that

@c0c0n3 test_reporter is still failing, which look extremely odd.

master builds without error
test pr for verifying if there is some weird change in circle-ci, builds without error

i am clueless

@daminichopra the cla issue was due to the fact that some of the commits where signed as "root", i fixed th

@c0c0n3 test_reporter is still failing, which look extremely odd.

master builds without error
test pr for verifying if there is some weird change in circle-ci, builds without error

i am clueless

@chicco785 , I will check PR locally once

@daminichopra the cla issue was due to the fact that some of the commits where signed as "root", i fixed th

@c0c0n3 test_reporter is still failing, which look extremely odd.

master builds without error
test pr for verifying if there is some weird change in circle-ci, builds without error

i am clueless

@chicco785 , I will check PR locally once

locally all works for me...

Some tests are failing for master and PR IN my env


Even though the endpoint time out occurs, the socket connection is not closed

Problem
In WSO2 APIM 3.2.0 we can observe that the socket connection is not closed even after the endpoint time-out occurred as the socket connection is closed once the socket timeout. We suspect this can lead to some other issues.
Solution
We believe that it is better to close the socket connection when the endpoint timeout occurs instead of keeping it open until the socket timeout.
Affected Component
APIM
Version
3.2.0
Implementation
No response
Related Issues
https://github.com/wso2/api-manager/issues/1746
Suggested Labels
No response

Closing this issue as the public fixes [1] and [2] have already been merged.
[1]https://github.com/wso2/wso2-synapse/pull/2076
[2] https://github.com/wso2/wso2-axis2/pull/273


Update guidance-compute-single-vm-windows.md

Fixing typos and removing unnecessary commas.

Hi @alexandair, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

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

@alexandair - The comma usage in the original texts is actually correct. If you could revert the comma changes in the submission, the rest of the changes would be mergeable.

@tysonn I think I've reverted all comma changes.


New Release for ttyd

It seems that there are lots of new features and bug fixes added in since last release (1.5.2 on Aug 5, 2019). When do you plan a new release?

https://github.com/tsl0922/ttyd/releases/tag/1.6.0


Explicit dependency is setup when using Bicep to deploy child resources

This statement needs to be revised when using Bicep to deploy.  With Bicep, the dependsOn is explicitly set when using child resources.
An implicit deployment dependency isn't automatically created between a child resource and the parent resource. If you need to deploy the child resource after the parent resource, set the dependsOn property.
Reference in the document:  https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/resource-dependency#child-resources
resource sqlserver 'Microsoft.Sql/servers@2019-06-01-preview' = {
  tags: tags
  location: resourceGroup().location
  name: sqlServerName
  identity: {
    type: 'SystemAssigned'
  }
  properties: {
    administratorLogin: sqldbUsername
    administratorLoginPassword: sqldbPassword
    minimalTlsVersion: '1.2'
    publicNetworkAccess: 'Disabled'
  }

  resource sqlserver_audit 'auditingSettings@2020-11-01-preview' = {
    name: 'default'
    properties: {
      isAzureMonitorTargetEnabled: true
      state: 'Enabled'
    }
  }
}

Generated ARM only for audit settings.  Notice the depends on is explicitly generated but it's not something I needed to explicitly define in Bicep.
        {
            "type": "Microsoft.Sql/servers/auditingSettings",
            "apiVersion": "2020-11-01-preview",
            "name": "[format('{0}/{1}', parameters('sqlServerName'), 'default')]",
            "dependsOn": [
                "[resourceId('Microsoft.Sql/servers', parameters('sqlServerName'))]"
            ],
            "properties": {
                "isAzureMonitorTargetEnabled": true,
                "state": "Enabled"
            }
        }

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

ID: 3d1485ea-dc5f-10f7-fc72-6fbd4c03a57b
Version Independent ID: 78bcf563-7214-762f-b82f-6b75b14edc63
Content: Set deployment order for resources - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/resource-dependency.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@SenthuranSivananthan - Thanks for brining this to our attention. We will investigate it further and update you shortly.

@SenthuranSivananthan - The Bicep documentation about dependencies describes the implicit dependency for nested resources.
#please-close


Authorization code flow  - onSuccess does not return code

I'm having an issue while logging in using Authorization code flow
My login configurations are
const googleLogin = useGoogleLogin({
    flow: 'auth-code',
    scope: 'profile email https://www.googleapis.com/auth/drive',
    onSuccess: onSuccess,
    onError: onFailure,
    select_account: true,
    useOneTap: true,
  });

  async function onSuccess(response) {
      console.log(response.code); // undefined
      if (response) {
        const resp = await signInUser(response));
        console.log(resp);
      }
    }

In some/most cases it works fine and we are getting the response correctly just like this
 {
  "code": XXXXXXXXXX,
  "scope": XXXXXXX,
  "authuser": "0",
  "hd": "XXXXXX",
  "prompt": "consent"
}

But in some cases the onSuccess callback runs with code value being undefined.
This happens for a specific user. The user who is trying to login has authenticated google account. (Not with gmail but some other email domain). He is also granting required access to the application from google account screen.
Expected Behaviour
onSuccess should return code or in case of error it shouldn't run and only onError callback should run.
Any help ?

Maybe your domain email doesn't allow to expose data for other applications
or you can specify hosted_domain prop and try if it works


PWA Service Worker integration improvement

With Angular 5, we can use the angular/service-worker stable library.
Currently, ionic integrate the service worker inside the index.html.
Now, with angular 5 we can use the service-worker library as a module to register the service.
Like this :
imports: [
    ...
    ServiceWorkerModule.register('service-workerr.js')
]

With this line, we can remove the service worker inscription from the index.html.
But, the main improvement i wanna talk is about the way to update the service worker if they are a update.
Indeed, with the default version of the service worker we use the cacheFisrt strategy.
So, if they are a update of the PWA, the client doesn't now it and keep the old PWA.
If you wanna keep update your clients you need to use the networkFirst strategy. But, with this strategy you lose speed because you need to call your server first...
So, we need a hybrid strategy :

use the cacheFirst strategy
when we launch the app, we can see if the service-worker between the client and the server is the same or not
if its the same...do nothing
if not, tell it to the user and reload the app with the new service-worker

=> this strategy is available here (with the old service-worker of angular) : https://github.com/shprink/ionic-angular-twitter-pwa
With the new angular service-worker library, we can do that easily wih SwUpdate :
this.swUpdate.available.subscribe(event => {
      console.log("new update available");
}):

I think it's important default behaviour to improve ionic app's.
Ionic version:
[ ] 2.x
[X ] 3.x
[ ] 4.x
I'm submitting a ...
[ ] bug report
[X] feature request

Hello! Thank you for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#330


Release 0.11.0

Release Notes

Web UI: v2.5.1
IPFS: go-ipfs v0.4.22

🚨 Breaking Changes

[ ] Remove code to disable old auto launch at startup.

✨  Features

[ ] js-ipfs implementation support (#601)
[ ] Move configuration to one place (#1082)
[ ] Improve UX of 'Download Hash' (#1135)

Postponed from #951:

[ ] New Tray Icon (#956)
[ ] Co-host websites (#1034)

🛠 Fixes and Maintenance
If any...
Before Release

[ ] Make sure i18n is updated (tx pull -a)
[ ] Make sure we're using the latest released version of Web UI
[ ] Make sure we're using the latest released version of go-ipfs
[ ] Update Release Notes accordingly

Release

[ ] Release Candidate

[ ] 💻 Bump version on package.json
[ ] 💻 git tag $version
[ ] 💻 git push && git push --tags
[ ] ⏰ Wait...
[ ] 🔗 Link release notes to this issue

[ ] Final

[ ] 📝 Create draft and copy the release notes above
[ ] 💻 Bump version on package.json
[ ] 💻 git tag $version
[ ] 💻 git push && git push --tags
[ ] ⏰ Wait...
[ ] 📃 Copy the release notes above to the GitHub release
[ ] 🚢 Ship!
[ ] 📑 Update README.md versions
[ ] 📦 Update Homebrew
[ ] 📦 Update Chocolatey

Postponed
If any...

This issue seems a bit outdated given that 0.10.4 has newer versions of webui and go-ipfs than this issue.
Also I believe "New Tray Icon" has already been shipped.


Rework the import error message for testing lib

The real exception backtrace is helpful

Why do you changed to use PROJECT_NAME global variable? Using project_name argument (which is set to PROJECT_NAME) seems better.

I did not notice that. It's a copy past from pyfai or fabio.

Here it is fixed. Thanks for the feed back.


Prohibit month/day being 0 in DateFormatter (#31)

As pointed out via issue #31, the month or day should not be allowed
to be zero. This commit only handles that issue, and does not add
monthly-based date formatting.

Hey @heejongahn thanks for the PR and test cases 👍 , sorry we did a hotfix yesterday night, so could you please solve your conflicts? Basically, just exclude any file in dist folder, we will regenerate js.

Sure, no problem. I'll solve it as soon as I get home this evening.
2016. 7. 15. 06:38에 "Jeff J"<EMAIL_ADDRESS>작성:

Hey @heejongahn https://github.com/heejongahn thanks for the PR and
test cases 👍 , sorry we did a hotfix yesterday night, so could you
please solve your conflicts? Basically, just exclude any file in dist
folder, we will regenerate js.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nosir/cleave.js/pull/35#issuecomment-232858124, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGdIyPw54Iil707VWdDoCucnaTljc6E2ks5qVw7FgaJpZM4JMj_u
.

As you said :) 👍

@trjiang This change breaks the backspace delete character behaviour, please follow general test process before merging, thanks.
No worries. I added a hotfix commit, will fix from my side.

Aha, I should've considered that.. Thank you for a hotfix @nosir !

@heejongahn no problem


Update overview.md

/hold
This PR is not for master.

/hold
This PR is not for master.

https://github.com/k8smeetup/website-tasks/issues/3431我是在这个页面接的任务

@Sophy417 That repo has been deprecated for a long time... Sorry if this is confusing.

@Sophy417 The whole zh localization team work is now part of the kubernetes/website repo. Just search issues here with label language/zh.

/hold
This PR is not for master.

那是应该提交到哪个分支吗？是对哪个分支版本的修正吗？

If you are fixing bugs on an old branch, propose your PR to that specific branch, say release-1.16.
If you are working on the master branch, submit your PR to master instead.
We don't actively work on old/archived versions, although critical bug fixes are still accepted.

If you are fixing bugs on an old branch, propose your PR to that specific branch, say release-1.16.
If you are working on the master branch, submit your PR to master instead.
We don't actively work on old/archived versions, although critical bug fixes are still accepted.

好的


[FEATURE] - Avoid displaying the webview on every opened VS Code instance

Today, when the “What’s New” window appears, it is shown on every VS Code instance.
The idea is that the “What’s New” window only be shown in one VS Code window (preferably the “active”). VS Code itself works this way.

It may be worthwhile to take into account different Profiles as well:

I think the switching from one profile to another is still within the same VSCode instance, so it may be worth adding logic for this.
Currently after updating Bookmarks, I may have to close the "What's new" tabs popups quite a few times, even if I read it the first time.

Hi @hyperupcall ,
The reason behind #42 is to deal with it. Profiles may use different versions of the same extension (until you update it) so, when you switch between profiles, the extension may think it has been downgraded and would fire the what's new window. With #42, this would not happen anymore, after every Profile to be updated to the version of the extension that does support that.
At first, I tried to identify the Profile to achieve that instead, but there is not API available, unfortunately.
Hope this helps

I see - it is unfortunate there is no API available but I am glad that you considered the option. Thank you also for making the Bookmarks extension!

Let's hope it works as expected. And once this issue is also released, I expect the window to be displayed only once, when really expected.


0.1.125

Resolves #2255 #2237 #2209 #2134

Nice! Is #2140 planned for this release?
Maybe add PowerPoint RAG support in the Changelog.

@jannikstdl 0.1.126 most likely :)


Missing data while utf-8 transmission

I'm trying to send the jpg file via BTSerial on ESP-32.
It works fine on python scripts and Serial Bluetooth Terminal app for android. But with react-native-bluetooth-classic it lose some data in each chunck. I tried to use both
 this.state.btDevice.onDataReceived((data) => {this.onDataReceived(data.data)})
 this.state.btDevice.read();

But every of them cut the the chunks. For example first chunks is
���� on react-native-bluetooth-classic
\xff\xd8\xff\xe0\x00\x10JFIF\x00\x01\x01\x01\x00\x00\x00\x00\x00\x00\xff\xdb\x00C\x00\ in python
connection options are
     connectionOptions: {         CONNECTOR_TYPE: "rfcomm",         DELIMITER: '\r',         READ_SIZE: 4096,         DEVICE_CHARSET: "utf-8",       },
please help

With regards to the code you posted, you provided both
 this.state.btDevice.onDataReceived((data) => {this.onDataReceived(data.data)});  // Sets an OnRead event
 this.state.btDevice.read(); Manual Read

You should not be using both of these at the same time, it should be an either or.  So if you're doing both, I can see why you would be missing data.
Secondly, if you're sending a JPG file, you don't want to be sending UTF8, you want to be sending Binary data, no?  You'll need to implement your own Binary data connection for parsing the information correctly.  Or you'll have to write your own encoding/decoding to map it.
Anyhow, need a bunch more information about what you're actually doing (more than just 2 lines).
I have a couple issues I need to look at this weekend, if you can provide more details, I will try to look at this one in more detail as well.

Thank you for fast response. I tired to use two methods, each separate, not both in one time.
Yes, I'm sending the JPG file. I think its byte array, but in Serial Bluetooth Terminal app utf-8 was chosen. That's why it make me think that that is utf-8.
How can I implement my Binary data connection for parsing the information correctly with react-native-bluetooth-classic
Thank you a lot for help

I know that Serial Bluetooth has a charset setting, but I'm unsure about how that's actually applied to the connection.  I've seen the repository for that project on Github but haven't actually looked into the source code.
By default Serial Bluetooth receive is set to CR-LF I believe, but you only have the delimiter set to LF is it possible that part of the Image is matching the /r and therefore the first chunk is being received?  Or have you updated Serial Bluetooth to match the delimiter?
There are a bunch of issues and documentation showing the starting point for dealing with Binary data.  Since it was really an after thought of both this (and the original library) I really haven't put much time into worrying about it, but it does work as seen from others.
If your peripheral is sending UTF8 encoded image data, then you should be fine accepting it, you'll just need to match up the delimiters.
I suggest you do some debugging of the Android code and provide your findings.  I can help with suggestions from there.

I tried different delimiters (\r, \n, \r\n, null, empty), the result was the same - some data was not received.
I will try to deal with Java code to understand how to fix it.

Sounds good.
There isn't much to it.

Data is read
Data is added to buffer
Buffer is decoded based on charset
If there is an OnRead listener it will look for a delimiter and send the content of the message.  Then return the remaining data to the Buffer
If there is no OnRead listener, data just builds up in the buffer until either a Read or an OnRead listener is added.

So there are limited spots where the issue could be occurring:

When the data comes in do you get all the bytes?
When it's decoded from the buffer and looked for the Delimiter is it found?
When the message is sent to React Native side, what format is it in?

Feel free to keep me posted.

Hey, any luck with debugging the Java side to see if anything is apparent?

0

Eah, it works well with Java code!

So we've got no idea where things are going wrong?

Unfortunately I can't say what is wrong with react-native-bluetooth-classic, I will be happy to use it ir there will be no losses. I just took the Bluetooth serial android library and it works. https://developer.android.com/reference/android/bluetooth/BluetoothSocket

Right on. Sorry it wasn't working for you, but glad your eup and running.

Thank you! Good luck!


[WiP] Additionnal features and methods for Mesh module : Issue #48

Some bug fixes
New features to add elements to an existing ElementDict object
New features to compute the interface between two meshes (Example with a mesh of Segments at the interface of two meshes of Triangles)

Adding the node_tag attribute for NodeMat classes. Otherwise it is going to be to complicated to manage submeshes.

Modifications from discussion #48 have been taken into account.

Thank you for your contribution :)


Update home page to feature install script for 0.0.3

For release 0.0.3 we'll have the nice install script #7 and planning for a quick video showing how easy it is to install https://github.com/owncast/owncast/issues/257.
I don't know exactly what the page should consist of going forward, but maybe it's time to update the home page to feature the "easy to setup" nature of Owncast since it's getting to the point where it's appealing to more people.  It's still primarily a documentation page of course, but it's also our public presence until some point in the future when we want a "marketing" site that has a different target audience.
I still think it's worthwhile having a blurb on the home page explaining who it's useful for up front without making them dig into more docs, but I don't think the long extended story is what we should keep.  Maybe it's too early to be thinking of such things, but maybe a shorter tag line might be helpful.
Example: Homebrew does a good job of making the install script instructions the most important thing on the page.  They have a tag line of "The Missing Package Manager for macOS (or Linux)" and it's clear upfront what it does.
Let's discuss and decide how to tweak the home page going forward.

As an aside, I just noticed the Caddy v2 site and they do a nice job embedding a setup example.  Scroll down to A whole new server experience.  It seems to be all in javascript, very cool!
https://caddyserver.com/v2

That asciinema is cool.  Just tried it:

Yeah asciinema is fun! And it's just a web component, so it would be easy to embed in Hugo.
The homepage probably has multiple goals from a marketing perspective. A few things we should hit:

Easy of install - show the install script, etc
Easy to switch - use the streaming tools you already know (OBS, etc), it's Twitch in a Box
Easy to try - maybe link to the demo server?
Easy to customize - embed the chat stream, customize the UI, etc

As we get the directory built out and maybe populated, we could link to that too.

I just pushed a 0.0.3 branch that adds a mention of the install script (thanks to @jmhobbs!) on the home page and the quick start, and and asciicast of it in use on the home page.  I'm sure we should re-word things, pretty them up, and organize things a bit, but I just wanted to make a first stab at it.  But anybody is free to iterate on it as they see fit.
Like Matt said, maybe we should add a "Try" page that points to the demo servers.  Though admittedly I find our broadcasting demo server a bit hard to clearly explain.
We can probably rename the "embed" page to "Customize" to have more about the web UI and the options around all those components.  And the Broadcasting software can be made a bit friendlier by talking in terms of switching and not just clinical documentation on how to setup pieces of software.  Both are useful.

Tweaking.


example notebook issues

The example is breaking on reading the datapackage when I try to run it straight in python:
>>> from pathways import Pathways
>>> import numpy as np
>>> p = Pathways(
... datapackage="datapackage_sample/datapackage.json",
... debug=True)
Invalid datapackage: Descriptor validation error: {'path': 'mapping/mapping.yaml', 'profile': 'data-resource', 'name': 'mapping', 'format': 'yaml', 'mediatype': 'text/yaml', 'encoding': 'utf-8'} is not valid under any of the given schemas at "resources/33" in descriptor and at "properties/resources/items/oneOf" in profile
Invalid datapackage: Descriptor validation error: 'data-resource' is not one of ['tabular-data-resource'] at "resources/33/profile" in descriptor and at "properties/resources/items/properties/profile/enum" in profile
Invalid datapackage: Descriptor validation error: {'path': 'classifications/classifications.yaml', 'profile': 'data-resource', 'name': 'classifications', 'format': 'yaml', 'mediatype': 'text/yaml', 'encoding': 'utf-8'} is not valid under any of the given schemas at "resources/34" in descriptor and at "properties/resources/items/oneOf" in profile
Invalid datapackage: Descriptor validation error: 'data-resource' is not one of ['tabular-data-resource'] at "resources/34/profile" in descriptor and at "properties/resources/items/properties/profile/enum" in profile
Log file: /home/marc/.local/state/pathways/log/pathways.log

I have the example folder straight from the repo with git clone.

Yes, these are warnings only. It complains that some resources (the YAML files) in the data package do not comply with the tabular data format. It does not prevent the code execution, but yes, I should fix/silence that.

Right, thanks.
Two minor other points:

The decryption key is in the notebook
Typo in first sentence of 'avoiding double counting' section:

When working with IAM/ESM, the focus s in the total quantity

OK, fixed. I'll fix the warnings later today.

The calculation is also not working, I expect due to a specific version of bw2calc not being specified:
>>> p.calculate(
...     methods=[ # <-- we list LCIA methods here
...         "EF v3.1 EN15804 - climate change - global warming potential (GWP100)",
...         'IPCC 2021 - climate change: fossil - global warming potential (GWP20)'
...     ],
...     regions=["EU",], # <-- the regions to conduct the assessment for
...     scenarios=[
...         "Scenario A",
...         "Scenario B",
...     ],
...     variables=[
...         "technology A",
...     ],
...     years=[
...         2020,
...         2030,
...         2040,
...         2050
...     ],
...     use_distributions=500, # <-- if >0, Monte Carlo analysis
...     multiprocessing=True # parallelize calculations for faster results
... )
Calculating LCA results for some model...
--- Calculating LCA results for Scenario A...
------ Calculating LCA results for 2020...
------ Calculating LCA results for 2030...
------ Calculating LCA results for 2040...
------ Calculating LCA results for 2050...
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/marc/.conda/envs/pw/lib/python3.11/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
                    ^^^^^^^^^^^^^^^^^^^
  File "/home/marc/.conda/envs/pw/lib/python3.11/multiprocessing/pool.py", line 48, in mapstar
    return list(map(*args))
           ^^^^^^^^^^^^^^^^
  File "/home/marc/.conda/envs/pw/lib/python3.11/site-packages/pathways/lca.py", line 632, in _calculate_year
    lca = bc.MultiLCA(
          ^^^^^^^^^^^^
TypeError: MultiLCA.__init__() got an unexpected keyword argument 'demands'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/marc/.conda/envs/pw/lib/python3.11/site-packages/pathways/pathways.py", line 478, in calculate
    years, p.map(_calculate_year, args)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/marc/.conda/envs/pw/lib/python3.11/multiprocessing/pool.py", line 367, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/marc/.conda/envs/pw/lib/python3.11/multiprocessing/pool.py", line 774, in get
    raise self._value
TypeError: MultiLCA.__init__() got an unexpected keyword argument 'demands'
>>>
>>> (pw) marc@framework7640u:~/Downloads/temp/pathways/example/datapackage_sample$ cd mapping/
  File "<stdin>", line 1
    (pw) marc@framework7640u:~/Downloads/temp/pathways/example/datapackage_sample$ cd mapping/
         ^^^^
SyntaxError: invalid syntax
>>> (pw) marc@framework7640u:~/Downloads/temp/pathways/example/datapackage_sample/mapping$ la
  File "<stdin>", line 1
    (pw) marc@framework7640u:~/Downloads/temp/pathways/example/datapackage_sample/mapping$ la
         ^^^^
SyntaxError: invalid syntax
>>> mapping.yaml
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'mapping' is not defined

My bw setup is:
# packages in environment at /home/marc/.conda/envs/pw:
#
# Name                    Version                   Build  Channel
bw2calc                   1.8.2           py311h38be061_0    conda-forge
bw2data                   3.6.6              pyhd8ed1ab_0    conda-forge
bw2io                     0.8.12             pyhd8ed1ab_0    conda-forge
bw2parameters             1.1.0              pyhd8ed1ab_0    conda-forge
bw_migrations             0.2                pyhd8ed1ab_0    conda-forge
bw_processing             1.0                pyhd8ed1ab_0    conda-forge
libwebp-base              1.4.0                hd590300_0    conda-forge

# packages in environment at /home/marc/.conda/envs/pw:
#
# Name                    Version                   Build  Channel
bw2calc                   1.8.2           py311h38be061_0    conda-forge
bw2data                   3.6.6              pyhd8ed1ab_0    conda-forge
bw2io                     0.8.12             pyhd8ed1ab_0    conda-forge
bw2parameters             1.1.0              pyhd8ed1ab_0    conda-forge
bw_migrations             0.2                pyhd8ed1ab_0    conda-forge
bw_processing             1.0                pyhd8ed1ab_0    conda-forge
libwebp-base              1.4.0                hd590300_0    conda-forge

OK, fixed in https://github.com/polca/pathways/commit/02a702f62dbab8d6163fa54da673a625e35b1efe. I forgot to specify versions for bw2calc in the conda recipe file. It should have installed bw2calc from bw25.

Thanks, will try again soon

Datapackage validation issues silenced in https://github.com/polca/pathways/commit/dbc3ecee61b97ce85080ede433ec1f0f1cf86e23
But only those caused by the presence of the YAML (other validation errors will still be printed), which do not have a corresponding profile schema in data packages because they are not tabular data file formats.

That doesn't work in conda, when it pulls 2024.10.30 is still installs bw versions as listed above. The dev versions are considered 'lower' by conda. What does work is bw2calc = 2.0.dev23 etc.
I now installed the correct versions with
conda create -n pw -c conda-forge -c romainsacchi -c cmutel pathways bw2calc=2.0.dev23 bw2data=4.0.dev59

OK, thanks. I'll change that.

OK, fixed in 02a702f. I forgot to specify versions for bw2calc in the conda recipe file. It should have installed bw2calc from bw25.

Also I think you need to write the comparison and version together like suggested here:
https://github.com/polca/pathways/actions/runs/11592170068/job/32273640352

OK, fixed in 02a702f. I forgot to specify versions for bw2calc in the conda recipe file. It should have installed bw2calc from bw25.

Also I think you need to write the comparison and version together like suggested here: https://github.com/polca/pathways/actions/runs/11592170068/job/32273640352

Yes, I changed that to bw2calc==x.x.devx https://github.com/polca/pathways/blob/7bf0c7f76dc2534b95b42bae73ceb09571bf9681/conda/meta.yaml#L29

One more thing @romainsacchi I see the tests keep failing. Perhaps have a look at that. Besides that I think my review is done and approved from my side :)

I fixed it.


Add discoverable URIs from /v1 root

This fixes #359

I'd also add a handler for /, which would include links to /v1, /osbapi, /healthz, etc.


Check if html has been initialised

Check if html has been initialised, then get currentValue.
Current version throws error ERROR TypeError: Cannot read property 'currentValue' of undefined

Thanks, @ssuperczynski .
This will be included in next release.
Follow angular commit convention for the next PRs :smile:

Will do, thanks ;)

is the PATCH an urgent need? If so I will release a minor version right away.

Yes, it breaks whole page

Okay. The patch is released now @ssuperczynski

Can confirm, works great now. Thanks!

@ssuperczynski . I need a general opinion.
Which one do you prefer? html or ngModel for HTML binding


Implement associations in Generic Object.

Purpose or Intent
Add relationships in Generic Object.

@gmcculloug @Fryguy Please review.

LGTM

👍


[Bug]: Open Repo button opens the app in new tab instead of the repository

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://user-images.githubusercontent.com/80334441/174563639-49bef866-d8be-4934-9024-5e56de296cb9.mov
Steps To Reproduce

Go to an app connected to git which has merge conflicts
Error modal opens with Open Repo button
Click on the button and observe

Public Sample App
No response
Version
Release

@Parthvi12 I am not able to reproduce this. Let's have a call tomorrow.

@Parthvi12 I am not able to reproduce it. Please provide more details.

@IAmAnubhavSaini  to reproduce this error :

Go to github repo and change something ex: query name from Query1 to Query2
Change the query name from app as well Query1 to Query3
Commit the changes and it will show  message to pull the remote changes.
Close the modal and click on pull changes icon, it should open the modal mentioned in the issue

Let me know if you are still having trouble reproducing this.

@Parthvi12 Where did it used to come?


WORK:4978629: Added generated files from Hybrid team Hydra-specs changes

1.Default site support
2.Support connectivity status in connection object
3.Add mandatory param: BillingType to new AzureDedicatedCircuit
4.Fix for Reset-preshared API

Hi @Nilambari, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (nilamd). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Will rerecord failing tests and reopen new merge request for this. Thanks!


Flask g.user is None

I ran superset export-dashboards on a fresh install of latest master. I see the same error for superset export-datasources.
Expected results
Expect command to export resources.
Actual results
There was an error when exporting the dashboards, please check the exception traceback in the log
Traceback (most recent call last):
  File "/home/fedora/superset/repo/superset/superset/cli.py", line 269, in export_dashboards
    for file_name, file_content in ExportDashboardsCommand(
  File "/home/fedora/superset/repo/superset/superset/commands/export.py", line 50, in run
    self.validate()
  File "/home/fedora/superset/repo/superset/superset/commands/export.py", line 67, in validate
    self._models = self.dao.find_by_ids(self.model_ids)
  File "/home/fedora/superset/repo/superset/superset/dao/base.py", line 74, in find_by_ids
    query = cls.base_filter(  # pylint: disable=not-callable
  File "/home/fedora/superset/repo/superset/superset/dashboards/filters.py", line 73, in apply
    if is_user_admin():
  File "/home/fedora/superset/repo/superset/superset/views/base.py", line 258, in is_user_admin
    user_roles = [role.name.lower() for role in list(get_user_roles())]
  File "/home/fedora/superset/repo/superset/superset/views/base.py", line 251, in get_user_roles
    if g.user.is_anonymous:
AttributeError: 'NoneType' object has no attribute 'is_anonymous'
2021-05-12 11:34:09,092:ERROR:superset.cli:There was an error when exporting the dashboards, please check the exception traceback in the log
Traceback (most recent call last):
  File "/home/fedora/superset/repo/superset/superset/cli.py", line 269, in export_dashboards
    for file_name, file_content in ExportDashboardsCommand(
  File "/home/fedora/superset/repo/superset/superset/commands/export.py", line 50, in run
    self.validate()
  File "/home/fedora/superset/repo/superset/superset/commands/export.py", line 67, in validate
    self._models = self.dao.find_by_ids(self.model_ids)
  File "/home/fedora/superset/repo/superset/superset/dao/base.py", line 74, in find_by_ids
    query = cls.base_filter(  # pylint: disable=not-callable
  File "/home/fedora/superset/repo/superset/superset/dashboards/filters.py", line 73, in apply
    if is_user_admin():
  File "/home/fedora/superset/repo/superset/superset/views/base.py", line 258, in is_user_admin
    user_roles = [role.name.lower() for role in list(get_user_roles())]
  File "/home/fedora/superset/repo/superset/superset/views/base.py", line 251, in get_user_roles
    if g.user.is_anonymous:
AttributeError: 'NoneType' object has no attribute 'is_anonymous'

How to reproduce the bug

Install Superset latest master in editable mode
Run superset export-dashboards

Environment
(please complete the following information):

superset version: 0.999.0dev
python version: Python 3.8.7
node.js version: v14.16.0

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Hi @krsnik93, do you run superset init before doing this?

@zhaoyongjie yes + superset db upgrade

is this still happening? @krsnik93

@junlincc same error, retested today, latest commit 281d637674ff8c75faea99ae5fc84d3ef6576dce

In the same vein (I believe), I've tried to replicate on a Fedora32 VM running Superset 1.1.0 and get:
$ /opt/superset/venv/bin/superset version
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Superset 1.1.0
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
$ /opt/superset/venv/bin/superset --version
Python 3.8.7
Flask 1.1.2
Werkzeug 1.0.1

[fedora@temperature ~]$ /opt/superset/venv/bin/superset export-dashboards
logging was configured successfully
INFO:superset.utils.logging_configurator:logging was configured successfully
/opt/superset/venv/lib64/python3.8/site-packages/flask_caching/__init__.py:201: UserWarning: Flask-Caching: CACHE_TYPE is set to null, caching is effectively disabled.
  warnings.warn(
Starting export
INFO:superset.utils.dashboard_import_export:Starting export
Traceback (most recent call last):
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
    self.dialect.do_execute(
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/default.py", line 608, in do_execute
    cursor.execute(statement, parameters)
sqlite3.OperationalError: no such table: dashboards

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/superset/venv/bin/superset", line 33, in <module>
    sys.exit(load_entry_point('apache-superset==1.1.0', 'console_scripts', 'superset')())
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/flask/cli.py", line 586, in main
    return super(FlaskGroup, self).main(*args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/flask/cli.py", line 426, in decorator
    return __ctx.invoke(f, *args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/flask/cli.py", line 426, in decorator
    return __ctx.invoke(f, *args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/opt/superset/venv/lib64/python3.8/site-packages/superset/cli.py", line 392, in export_dashboards
    data = dashboard_import_export.export_dashboards(db.session)
  File "/opt/superset/venv/lib64/python3.8/site-packages/superset/utils/dashboard_import_export.py", line 31, in export_dashboards
    for dashboard in dashboards:
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/orm/query.py", line 3535, in __iter__
    return self._execute_and_instances(context)
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/orm/query.py", line 3560, in _execute_and_instances
    result = conn.execute(querycontext.statement, self._params)
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/base.py", line 1011, in execute
    return meth(self, multiparams, params)
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/sql/elements.py", line 298, in _execute_on_connection
    return connection._execute_clauseelement(self, multiparams, params)
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/base.py", line 1124, in _execute_clauseelement
    ret = self._execute_context(
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/base.py", line 1316, in _execute_context
    self._handle_dbapi_exception(
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/base.py", line 1510, in _handle_dbapi_exception
    util.raise_(
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/util/compat.py", line 182, in raise_
    raise exception
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/base.py", line 1276, in _execute_context
    self.dialect.do_execute(
  File "/opt/superset/venv/lib64/python3.8/site-packages/sqlalchemy/engine/default.py", line 608, in do_execute
    cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) no such table: dashboards
[SQL: SELECT dashboards.uuid AS dashboards_uuid, dashboards.created_on AS dashboards_created_on, dashboards.changed_on AS dashboards_changed_on, dashboards.id AS dashboards_id, dashboards.dashboard_title AS dashboards_dashboard_title, dashboards.position_json AS dashboards_position_json, dashboards.description AS dashboards_description, dashboards.css AS dashboards_css, dashboards.json_metadata AS dashboards_json_metadata, dashboards.slug AS dashboards_slug, dashboards.published AS dashboards_published, dashboards.created_by_fk AS dashboards_created_by_fk, dashboards.changed_by_fk AS dashboards_changed_by_fk
FROM dashboards]
(Background on this error at: http://sqlalche.me/e/13/e3q8)

Here, Superset seems to look for dashboards in a SQLite db… yet I've been using PostgreSQL as back-end database (and have defined dashboards in my instance that's running otherwise smoothly):
$ cat /opt/superset/superset_config.py | grep -C 0 SQLALCHEMY_DATABASE_URI
SQLALCHEMY_DATABASE_URI = 'postgresql://superset:<password>@localhost/superset_backend'

Closing as I am now able to do superset export-dashboards followed by superset import-dashboards, as well as superset export-datasources followed by superset import-datasources, on latest master without errors.

This happens if you run superset export-dashboards without having created an admin user with superset fab create-admin

superset export-datasources gives me this error on 2.0.1

I had same problem on 3.1.2 and finally discovered that issue happens when there are no users with login "admin", because of CLI exporting using this user
    g.user = security_manager.find_user(username="admin")

https://github.com/apache/superset/blob/6ba1dfbfd597efa72db2c632b554cbe1cddd4fd4/superset/cli/importexport.py#L79
so to fix this issue you just need to create user with "admin" login and access enough to export


Brew emits error upgrading sile

Apologies as I'm not a brew expert. If I should go bug them, let me know. My previous reinstall command no longer works.
So, instead I ran (totally a guess):
$ brew upgrade sile
[lots of output]
Error: Permission denied @ unlink_internal - /usr/local/share/ghostscript/9.05/Resource/Font/logo10.pfb

I'm not sure if this constitutes a real install problem. Sile appears to be there shows version <IP_ADDRESS>.

I don't think this is a SILE problem at all, as SILE does not depend (directly or indirectly) on ghostscript - see brew deps sile.
What is happening is that after you do a brew upgrade, Homebrew performs a brew cleanup to tidy up all your installed packages, and this is where it's finding an error - so after the SILE part has already happened.