[Feature request] Implement page transitions

I've noticed is that when building an app with material-ui there is no obvious way to do page transitions which are an important part of the overall UX.
Can we add page transitions in to backlog for future releases, or is there a recommended complementary library?

@owencm if you've got any good references, that'd be wicked 👍

@owencm
PS: I've almost got an implementation ready using react transition group for this material design enter/exit animation of a grid:
https://material-design.storage.googleapis.com/publish/material_v_4/material_ext_publish/0B08MbvYZK1iNTGRLb2Zud2RUNFE/animation-meaningfultransitions-hierarchicaltiming-4do_large_xhdpi.webm

I'm actually more concerned with whole page transitions -
If you open any app and tap buttons you'll notice that the title bar and content all fade in or move in from some direction in some smooth way. I'd love to make it easy to do that with Material UI.
See this example for one very specific set of navigation transitions done in React Native. Something like this that supports a general set would be awesome: https://youtu.be/3W58KampkXc?t=19m3s
I note that some Routers have transition support, for example see https://github.com/maisano/react-router-transition. For that reason I'm not 100% sure yet where these transitions should be implemented. I presume if we want more intelligent transitions that have e.g. different effect on title bar to the contents it will need to be in Material UI directly?

I'm actually more concerned with whole page transitions

I'm also quite interested in this topic. That's definitely something missing to compete with a native app.

for example see https://github.com/maisano/react-router-transition

We have tried using this lib at @doctolib without any luck. The approach used is too simple.
Applying the right transition depending on the using actions is far from being a simple problem.
The first thing that we learned is that we need to split the concern. We have build

one module to handle the transition state problem. It's pretty much answering the following question: which transition do I need to apply, based on the history / touch / network events.
one module to actually apply that transition. The material specification adds an extra complexity layer if we compare to the iOS model, as the transition is context aware. E.g. What makes a good transition.

For that reason I'm not 100% sure yet where these transitions should be implemented.

From my point of view, we should be focusing on addressing the point 2.
Actually, you might be interested in a talk I have given earlier this year: screen-transitions-with-ease.
We are using https://github.com/doctolib/react-router-transitions to solve point 1.

+1

Hello, i'd like to know if the v1-alpha branch would have need for transitions such as the ones located here:
Choreography - Material Spec
I realized that they are grouped into three categories, namely: radial, continuity and creation.
Note: Continuity closely resembles the transitions achieved with the Popover component however with Modals, the origin of the transition is from the top instead of the bottom as shown in the demo on the page.

Hi! I develop a library called yubabajs (see: yubabajs.com) - which is primarily inspired by material animations. I think it would fit lovely with material ui (the landing example is built with it, actually!).
These days you don't want to re-written everything yourself - yubaba has been made to be small, performant, and powerfully customisable - let's talk?

@madou Thanks for sharing your work. I like that: https://twitter.com/olivtassinari/status/1126261655124496389.
I only wish the demo transition duration was faster. How did you pick the value?
We have no plans to work on the transitions for v5 (6-12 months from now).
How do you think that we can help each other? :)

Thanks for sharing your work. I like that:
thanks!

I only wish the demo transition duration was faster. How did you pick the value?
so the it's transition based - but it's all dynamic based on how big the elements are, how far they're moving, and if they're expanding or collapsing.

i take the base values from https://material.io/design/motion/speed.html#duration, implementation here https://github.com/madou/yubaba/blob/master/packages/yubaba/src/lib/duration/index.tsx#L36 it's quite simple atm.
currently the dynamic stuff can't be tweaked via a public api but anything can be changed if it's more correct.

We have no plans to work on the transitions for v5 (6-12 months from now).
How do you think that we can help each other? :)

tbh yubaba's origins have always been about re-creating material motion on the web - so i think it kind of goes hand in hand with a material library hehe.
right now i have a roadmap going through the material motion catalog to implement new animations (and changing the core of the library if it needs) here https://github.com/madou/yubaba/issues/49 which i'll be slowly working through.
i've already shown one example of a transition working with the library - considering material-ui isn't planning on working on transitions for the next major version perhaps i can do more examples using public apis? a cookbook showing how a consumer of material-ui could get motion today which would be a page on the material ui website or something.
i think the holy grail though would be out of the box motion using yubaba.
cheers 😄

Looks really nice from skimming the demos.
I have to agree on the speed issue though. Perhaps something to look at. I believe that following the spec, a particular type of transition takes a certain amount of time, regardless of the size; so it is time, not speed that counts. I might be wrong though.
Does the name 'yubaba' hold special meaning? You might find more traction with a more descriptive name like 'material-motion'.

I have to agree on the speed issue though. Perhaps something to look at. I believe that following the spec, a particular type of transition takes a certain amount of time, regardless of the size; so it is time, not speed that counts. I might be wrong though.

all fair points! it can be very easily tweaked - the main thing i'd want personally is something out of the box that works great. no configuration needed ideally. as a rough feel how faster do you think it should be?

material-motion

ohh that's a good suggestion! "yubaba" doesn't hold any special meaning that i wouldn't want to let go of tbh. always struggled with names.

it's a google lib though

Ah, sorry, I didn't check. I was just throwing out a random suggestion. There will be something along those lines available I'm sure!

as a rough feel how faster do you think it should be?

As I understand it, you are scaling the transition time according to the size of the element / distance to be moved. If I am interpreting the spec correctly, you should instead be using fixed durations regardless of the size or distance.
If that's too rigid, the scaling factor should still be minimal.

scaling the transition time according to the size of the element / distance to be moved

yeah that's right - ill make a issue to investigate it over here https://github.com/madou/yubaba/issues/121 to reduce to noice here ✌

so it turns out that example had a fixed duration of 600 - removing it makes it a little faster and more to spec 👍
anyway my question to you both - do you think element-motion (renamed the library 😅) would provide any value to material-ui - and if yes would you be interested in me writing up interop examples to be referenced in docs?

Sounds good to me!

I'm actually more concerned with whole page transitions -
(...)
I note that some Routers have transition support, for example see https://github.com/maisano/react-router-transition. For that reason I'm not 100% sure yet where these transitions should be implemented. I presume if we want more intelligent transitions that have e.g. different effect on title bar to the contents it will need to be in Material UI directly?

@owencm I've made a library react-tiger-transition that implements whole page transitions, but it also allow to define specific transitions for each component.
As @oliviertassinari defined it on #18392:

It seems to be a generic navigation library, it's not directly related to Material-UI.

It's easy to implement, here is an example preview of animated tabs, available on the documentation.
This example leverages the transitions of the module without using other components, that actually make the navigation easier, because they provide the necessary css layers to build these transitions and integrate with react-router-dom.

I'm closing the issue, it has been open for +3 years but didn't get a lot of traction. I think that we can defer the concern to a third-party library.


app list not showing

thanks for the app
i have a problem in dashboard
when i open the categories change list page
the sidebar app_list not showing
and its generally empty or null

@hypy13,
More detail is needed before anyone can look into this. Some questions that would needed answered are:

What version of Python?
What version of Django?
Did you add  categories and  categories.editor to INSTALLED_APPS in your settings?

I'm closing for now until you can provide more detail.


Using "component" prop in Paper component causes positioning issues with Select component.

The Select popover will get placed in the upper-left corner of the screen when it's closing because the selected option was changed if a component is passed to the "component" prop via MenuProps:{ PaperProps:{ component: ... }}.
Possibly related, the dropdown also will be positioned in the upper-left corner of the screen if the dropdown is open at the time a separate Select component initializes.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
List of options gets placed in the upper-left corner of screen.
Expected Behavior 🤔
List of options should remain positioned relative to the Select.
Steps to Reproduce 🕹
I can't get codesandbox to work with any React apps, but here's the code anyway.
Steps:

Copy the source code from the Customized Selects demo here.
Add as a prop to the Select component the following code:

MenuProps={{
    PaperProps: {
        component: React.forwardRef((props, ref) => <div {...props} ref={ref} />)
        }
    }}

Click on the select to open the dropdown then click on an option that isn't selected.
The dropdown should suddenly jump to the top left corner of the screen while it closes.

Context 🔦
I'm trying to pass a component prop in order to wrap the options list with a custom scroller component, but this issue results.
Your Environment 🌎
I don't have some of the info required here at hand but I'll update with the info later.

Tech
Version

Material-UI
v4.4.0

React

Browser
Firefox / Chrome

TypeScript

Thank you for opening this issue and providing as much detail as possible.
Unfortunately I get a blank space when opening https://codesandbox.io/s/materialui-select-issue-9t4wc. When creating a fresh copy of the demo and applying the diff I have no issues:  https://codesandbox.io/s/material-demo-366vr

Updated initial answer with solution.


Port code to VS2015

We use VS2015, and it would be useful if dotNetInstaller could be built by VS2015. I noticed the sln format was for VS2005, and I verified it can compile in VS2008 without changes (but not VS2015).
I ported the code to compile in VS2015 except the managed c++ CabLib library (and release mode only at the moment). The CabLib library uses the old clr syntax which is no longer supported, and it looked like it would take awhile to port to the new syntax (and error prone when you don't know how to test it). So I used a precompiled binary for CabLib (built with VS2008) and got everything else to compile in VS2015 (release mode only at the moment). So now I can make changes with just VS2015.
I can help you port to VS2015 if it is a feature you would like. One tricky step was fixing ___std_terminate and 'eh vector' linker errors, but luckily I found this german webpage solution:
http://www.speedproject.de/2015/05/12/visual-studio-2015-und-die-universal-crt/
(google translate does a good job at converting it to english).

The reason why dotNetInstaller requires Visual Studio 2005 to build is because Visual Studio 2005 is "the last version to be able to target Windows 98, Windows Me and Windows NT 4.0 for C++ applications."[1] If we upgrade and require to build using Visual Studio 2015, then we lose support for running on those operating systems.
[1] https://en.wikipedia.org/wiki/Microsoft_Visual_Studio#2005

OK, thanks for the reply. It makes sense to keep it in VS2005 format while we want to support those other operating systems.
Its great to see dotNetInstaller is still in active development (the old VS2005 format shocked me a little, and I thought maybe it was no longer under development).
If other people need to compile it in VS2015, I suppose we could create a fork and put the changes in there (but the official one will get the official updates). Supporting both VS2005 and VS2015 in a single repo could be tricky and annoying to maintain.
Thanks for having this open source, it allows us to make minor changes fairly easy.

I thought about 5 years ago that we could drop Windows 98 support, but found out that there were actual users (!). I think it's time though.

After installing Wix Toolset Visual Studio 2015 Extension and Microsoft Visual Studio 2015 Installer Projects Extension, Visual Studio 2015 was able to perform a one-way upgrade of dni.sln.


More digits on cost

Increased resolution on cost. As models have gotten cheaper, it's actually hard to see changes sometimes.

Hey @jamesbraza - we couldn't find any regex relying on cost format. If you can recall where that is, yea I can update. Thanks for flagging!

Hey @jamesbraza - we couldn't find any regex relying on cost format. If you can recall where that is, yea I can update. Thanks for flagging!

Yeah sorry I sent that comment from my mobile last night while unable to sleep.
I realized today that actually cost is not part of the regex: https://github.com/Future-House/paper-qa/blob/e1a8dadb9cfd2c7a5193734c9f3a9fda7c09e7b7/paperqa/agents/tools.py#L30-L32
So we are good 👌


wble-central: If a GATT Timeout occurs, command never exits

Regarding trying to create a profile from https://whad.readthedocs.io/en/stable/cli/ble/ble-spawn.html, I get the following error:
wble-central -i hci0 -b <my device bdaddr> profile mydevice.json
[!] GATT Timeout while discovering services and characteristics. Aborted.
Writing profile JSON data to mydevice.json ...

I know the GATT Timeout isn't an error, and I do see adv_data and scan_rsp values in the mydevice.json, but the error is that the command never exits back to the shell and just sits there on the "Writing profile" line forever.
Tested w/ commit 909b4af17959cac04e21aca5eeb21e6b319da246

It looks like the HCI adapter struggles to disconnect from device, blocking the nice exit procedure of wble-central. I managed to reproduce with a nRF52 USB dongle running ButteRFly, but not with my HCI USB adapter. I fixed some parts of the code to avoid this to happen when using an UART-based interface (nRF52 and others), but I guess this is the same type of behavior for HCI.
Could you paste/attach a full debug log (you can generate such log with --log debug --logfile debug-log.txt options) ?

Thanks to your detailed instructions to build the peripheral_gatt_write example from Zephyr, I've managed to reproduce the issue with my nRF52840 USB dongle and to figure out the root cause of this issue. I've merged a patch into our main branch, you can give it a try to check if this fixes the hang.

Any update on this issue ? Did you have time to test the latest main branch ?


simplexml_import_dom(): Invalid Nodetype to import

Hi! I have installed
"maatwebsite/excel": "~1.3.0",
I have laravel 4.2, I call Laravel-Excel like this
    Excel::create('New file', function($excel)  use($board,$circuits) {

        $excel->sheet('New sheet', function($sheet)  use($board,$circuits) {

            $sheet->loadView('board.show', compact('board', 'circuits'));
        });
    });

and always getting error:
 simplexml_import_dom(): Invalid Nodetype to import
    protected function importDom(DOMDocument $dom)
    {
   ->>     return $this->xml = simplexml_import_dom($dom);
    }

I have installed all requirements... Dont know how to fix this.

This means you most likely have a wrong HTML syntax somewhere in your view file.

Hey guys,
I know this is pretty late, but I wanted to add my experience since I had this issue recently in case anyone else does, too. I had the Invalid NodeType error for only 1 report, and eventually found what in my template was causing the error.
I was using a set of small Blade templates to build reports in an application. On one report, two things in a template broke one of my reports making an Excel file properly, throwing the Invalid NodeType for Import error:
(1) @if() / @endif blade conditionals had to be removed that were wrapping the template tags (although @foreach() seems ok for some reason), and
(2) I had comments at the top of the template page, and they had to be removed as well.
After those two changes, $sheet->loadView(...) worked great for me and generated an Excel file of my report.
Overall, this library is awesome!


how set fontname

could  summernote teams build documents for developers?

Sorry for lack of documentation.
You should put Summernote related options within summernote key as described in https://github.com/summernote/django-summernote#options
SUMMERNOTE_CONFIG = {
    'summernote': {
        'fontNames': [ ... ],
        'fontNamesIgnoreCheck': [ ... ],
    },
}


stream.filter(track=list_of_keywords) not returning tweets with standard operators

Hi,
I am not receiving the tweets when I have build keywords using the standard operators
Below keywords return the proper tweets:
watching now
Boeing flight
Whereas, the below keywords are not returning the matching tweets:
"""toms of maine"""  - (which should return tweet containing toms of maine together)
love or hate - (which should return tweets containing either love or hate)
(hills%20OR%20hill%27s)%20advantage  - (which should return hills advantage or hill's advantage)
When I checked the documentation of standard streaming tweets, it supports all these operators. Whereas, when I pass into the list of keywords, it is unable to fetch the tweets.
Can you please guide if any solution available or suggest how should I get this?
Thanks

"""toms of maine""" - (which should return tweet containing toms of maine together)

I'm unable to reproduce this; using stream.filter(track = ["""toms of maine"""]) works as intended.
There aren't many Tweets that match this though, so it could take a while to receive one.

love or hate - (which should return tweets containing either love or hate)

The OR standard operator is case-sensitive, so this needs to be love OR hate.

(hills%20OR%20hill%27s)%20advantage - (which should return hills advantage or hill's advantage)

You don't need to URL encode this. Tweepy should handle that for you.
Also:

statuses/filter does not allow combining keywords using parentheses

https://twittercommunity.com/t/filter-realtime-problem/113061/2
and despite the documentation:

You cannot use the search operators with statuses/filter

https://twittercommunity.com/t/statuses-filter-and-search-operators/115840/2

Thank you Harmon for detailed answer.


Apply cleanups to solana-core for unified scheduler

Problem
There are various small changes to support block production by unified scheduler.
Summary of Changes
Do them at once in a single pr, rather than 6 different prs.
Commits are well logically separated.
No functional change, so no test is added.
extracted from #3946, see the pr for the general overview.

@apfitzge thanks for quick review. Other than https://github.com/anza-xyz/agave/pull/4123#discussion_r1886895912, is there other concerns? If other changes are mergeable, I'll back-off the commit and create another pr for it, while landing other changes in this pr, if possible..

@apfitzge thanks for quick review. Other than #4123 (comment), is there other concerns? If other changes are mergeable, I'll back-off the commit and create another pr for it, while landing other changes in this pr, if possible..

Still looking, have some other stuff need to review this morning, and didn't get a chance to finish the review of this one yet


Vector.update: Require args to be named

Motivation
foo.update(bar) is a lot more confusing than foo.update(x=bar)
Caveats
This technically is a breaking change, though Vector.update is a niche API and I doubt anyone is writing
foo.update(x, y) instead of Vector(x, y) or such.

I will say, this matches the pattern we've been moving to in ppb as a whole, so big thumbs up from me.

Rebased to deal with merge conflict.
bors r=astraluma,pathunstrom


Updates maven wrapper

Ticket Link
GROUPINGS-1169
List of squashed commits

Updates maven wrapper

Test Checklist

[X] Unit Tests Passed:
[X] Jasmine Tests Passed:
[X] General Visual Inspection:

Hi Kailee, it would be super cool if at the next meeting you could explain the wrapper and why we provide it for this project. I've explained it a number of times -- sometimes not so clearly, it seems -- so it would be great to have another expert go over it with the team!


Register organisation: dropdown icon present on fields without dropdown

Description - What happened? *

Expected Behavior *
If there are multiple options for the job list (or other field) should display the icon otherwise dropdown icon is not required.
Observed Behavior *
Dropdown icon is displayed in many of the fields in the Organization register page without having an actual dropdown.
Steps to Reproduce

Launch the application as Authority admin
Click on My Organization > Users and Roles > Click on Invite User > Enter First Name, Last Name, Email and select Role > Click on send Invitation
Click on the link received in an email and set the password with QR scan and enter OTP > Login with this user > Place cursor on Job Title

Context Information
No response
Relevant log output
No response
Screenshots

The icon shown in the screenshots is the native macOS autofill feature, not a drop-down.
Closing as the form works as intended.


helpq 2022 launch tracker

[ ] check and fix css everywhere
[ ] find out how to create users
[x] enable github login

please comment below if anything else is missing

When logging in and putting an invalid email, it returns a wrong error message: "wrong password" (should be something like "invalid email").
Overall, it seems that one should be able to loging with the username as well as the email? Right now it only says "email" in the login window. So maybe something like "username or email"

I would put the home button (currently top left corner) to the top right corner next to the other menu bar items. I couldn't find it for the longest time.


Connect to FTP

Hey there
I want to connect a real FTP address with user:pass in xcode 9, can be use your framework?
Like this url.

we don't support FTP at the moment, but I'd be happy to accept PR that adds it

I have investigate support for FTP in iOS and it's complicated:

URLSession (and the older NSURLConnection) support only simple FTP downloads.
CFFTPStream was deprecated in the latest OS releases.

Good source is a post on Apple Developers Forums. @krzysztofzablocki do you think that supporting FTP is a good feature for this library?

I don't feel its needed, with etag / last modified we support both AWS and Dropbox like hosting, if someone wants to add I'd be fine accepting but don't feel like its common to use FTP for this kind of problem


Extend test cases

Working on add test cases. Hoping I can increase code coverage to 80%.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0d3ddb8). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master    #109   +/-   ##
========================================
  Coverage          ?   96.8%
========================================
  Files             ?       3
  Lines             ?     313
  Branches          ?      68
========================================
  Hits              ?     303
  Misses            ?       6
  Partials          ?       4

Impacted Files
Coverage Δ

sanic_openapi/__init__.py
100% <100%> (ø)

sanic_openapi/doc.py
98% <100%> (ø)

sanic_openapi/swagger.py
94.54% <100%> (ø)

Continue to review full report at Codecov.

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

@ahopkins All tests are ready now and the coverage looks pretty nice. :sunglasses:

awesome. well done.


Update operator.go

Might be fix for https://github.com/argoproj/argo/issues/1759

@sarabala1979 Please review

Pls add some test

Seems like this PR needs some rework and more discussion in #1759. Closing for now, feel free to reopen/resubmit if needed


feat(spell): Implement Chill Touch

Implement Chill Touch using the same mechanism that midi does, with some cosmetic changes.
Chill Touch is two AEs, one to handle the healing immunity and another to grant attack disadvantage for undead mobs attacking the source token. Both AEs are always applied, the attack disadvantage is gated behind an activation condition.
Because there are two AEs, we have a choice to make with regards to displaying a status icon on the token. Midi sets a false duration on both, displaying the same icon twice. In this PR we set a statusId on only the healing immunity, which will cause it to display a status icon. The attack disadvantage is a "passive" AE and while it will persist to the end of the source turn, it will not display status icon.
There could be an argument made for attaching the icon to the disadvantage AE, but I think it's a bikeshed.

Updated data->system.
People should feel free to modify this if we come up with a better answer for the duration question, but for now this is functional and ready to merge.

Fixed null'ing the spell duration by providing an empty duration object

Many force pushes for a dumb failed experiment to try to fix duration issue

Rebased


Hide comments unless user is logged in. Add login link.

Code looks great, could you add screenshots to the description showing the change on the web page?

Sure thing :) Here's the current state. I'll Upload comparisons from now on.


death of Frank Deem

West Virginia House of Delegates member Frank Deem-R died and has been replaced by former House of Delegates member Tom Azinger-R. He was appointed to the unexpired term and also won the election Tuesday.
reported to me via email

https://travis-ci.org/openstates/people/jobs/455627999


webview_cookie_manager won't work on android

your git reference of webview_cookie_manager won't work on android because it is still on android v1 embedding. So you need to migrate the package to v2

Thanks for the information. But I'm new to flutter and not sure how to fix it.

closing for webview_cookie_manager is no longer a dependency of this application.


External data sync example

Putting up an initial sample as draft PR showing how an app could bridge the gap between an external data source and a Fluid Container, for early feedback.

Closing this PR, not intended for checkin at this time.


Include Transmitted Channel in Composite Generation

In addition to the fluorescence channels, bright field or phase images should be incorporated into the Composite generation.  Medium priority.

Need Composite generation to be able to include one (and only one for now) transmitted channel combined with one or more fluorescence channels.


Lumen support?

Hey thanks for the package. Is it possible to use (compatible) with Lumen?

Im also very interested in this pkg working with lumen.

No - it's currently not compatible, because Lumen does not support FormRequest validation.
https://lumen.laravel.com/docs/5.2/validation

In that case, can you please remove lumen from composer.json

Whoops, sorry about that. Removed it

I added lumen support in #267.
Until it gets merged you can use it from the repo: https://github.com/Brixify/laravel-apidoc-generator
If you have any suggestions/problems please write to the merge request.

@juanvillegas

@mpociot https://github.com/mpociot/laravel-apidoc-generator/issues/5#issuecomment-369709070

@gazben How would you install your fork via composer?

@lmj0011
Add this to your composer.json:
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/Brixify/laravel-apidoc-generator.git"
        }
    ],

And make sure you have this required:         "mpociot/laravel-apidoc-generator": "dev-null-fix",

@lmj0011 no. Look at the example: https://getcomposer.org/doc/05-repositories.md#loading-a-package-from-a-vcs-repository

@gazben
thanks for the example, but I decided to go with https://github.com/FriendsOfPHP/Sami

Check this fork out, it's compatible with Lumen using Dingo Router.
https://github.com/davmixcool/lumen-apidoc-generator
Full support coming soon.


Select into pyarrow.Table with shared CPU memory

similar to https://github.com/gpuopenanalytics/libgdf/blob/master/src/ipc.cu, but on the CPU.

Closed by #19


first draft of about agrc...more to come

Do you want me to keep this pr open while you work on it or are you going to open another one?

ok ready to go, please merge

closed with d91d417


Maria4j Managed process does not start correctly  if "log-error" is defined in the configuration

As a part of getting a error logs, i added "--log-error=error.log" location as part of my configuration for DB Startup. This is meant to redirect stderr to error.log file. Due to this Parent Process fails with the error "Caused by: ch.vorburger.exec.ManagedProcessException: Database does not seem to have started up correctly? Magic string not seen in 30000ms: mysqld: ready for connections". My guess is that parent process which is looking for string "mysqld: ready for connections" does not find as it looking for it in console, but now this is getting written to error.log.
I am wondering if anybody else has faced this issue and if there is a known workaround.

@venkat-raman the idea with this "embedded" library is more that the log from the DB goes into your "normal" Java application log, through the slf4j logging API, which using e.g. logback you can configure to go into a file. You could even configure logback to write the logs from mariaDB4j into a separate file.
So your "--log-error=error.log"  approach is probably typically not used (with MariaDB4j). If you have strong reasons to have to use that instead of "normal" Java logging, then you may be able to achieve what you're after by subclassing the DB class and tweaking how it sets up the logging interception.... (but it's probably easier to use it as intended & as most people do.)
Hope this helps? If I don't hear back from you on this issue, I'll be closing it in 1 month.

I have recently compiled my war file into an executable jar with embeding tomcat and maria.
Eveerything worked fine on simple tests. But when i tried to run on server with MYSQL data,
I get similiar Exception.
I wish is the same that, we could come up with a fix:
ProgramDir: (D:\autosqlweb\) starting MariaDB on loc [D:\autosqlweb\mariaDBData]... ch.vorburger.exec.ManagedProcessException: An error occurred while installing the database 	at ch.vorburger.mariadb4j.DB.install(DB.java:129) 	at ch.vorburger.mariadb4j.DB.newEmbeddedDB(DB.java:76) 	at com.tugalsan.autosqlweb.TK_GWTSQLMariaDB.<init>(TK_GWTSQLMariaDB.java:30) 	at com.tugalsan.autosqlweb.Main.main(Main.java:37) Caused by: ch.vorburger.exec.ManagedProcessException: Program [C:\Users\tugalsan\AppData\Local\Temp\3\MariaDB4j\base\bin\mysql_install_db.exe, --datadir=D:\autosqlweb\mariaDBData] (in working directory C:\Users\tugalsan\AppData\Local\Temp\3\MariaDB4j\base) failed, exitValue=1, last 100 lines of console: Running bootstrap 2017-09-13  9:16:04 7604 [Note] C:\Users\tugalsan\AppData\Local\Temp\3\MariaDB4j\base\bin\mysqld.exe (mysqld 10.1.20-MariaDB) starting as process 7584 ... ERROR: Cannot write to mysqld's stdin FATAL ERROR: database creation failed https://mariadb.com/kb/en/installation-issues-on-windows contains some help for solving the most common problems.  If this doesn't help you, please leave a comment in the Knowledge Base or file a bug report at https://jira.mariadb.org 	at ch.vorburger.exec.ManagedProcess.checkResult(ManagedProcess.java:297) 	at ch.vorburger.exec.ManagedProcess.waitForExitMaxMsWithoutLog(ManagedProcess.java:415) 	at ch.vorburger.exec.ManagedProcess.waitForExit(ManagedProcess.java:385) 	at ch.vorburger.mariadb4j.DB.install(DB.java:127) 	... 3 more Caused by: org.apache.commons.exec.ExecuteException: Process exited with an error: 1 (Exit value: 1) 	at org.apache.commons.exec.DefaultExecutor.executeInternal(DefaultExecutor.java:404) 	at org.apache.commons.exec.DefaultExecutor.access$200(DefaultExecutor.java:48) 	at org.apache.commons.exec.DefaultExecutor$1.run(DefaultExecutor.java:200) 	at java.lang.Thread.run(Unknown Source)


TypeError during ng update: Cannot convert undefined or null to object

Command
update
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Running ng update to upgrade various Angular packages results in a TypeError, halting the update process.
Minimal Reproduction
Command
Executed command: ng update @angular/cdk @angular/cli @angular/core @angular/fire @angular/material (tried one at a time as well)
Error
✖ Migration failed: Cannot convert undefined or null to object
See "/private/var/folders/fr/nddjk7yj29v5f4058jgqbn400000gn/T/ng-JoOh2c/angular-errors.log" for further details.
Error details from the log file
[error] TypeError: Cannot convert undefined or null to object
at Function.entries ()
at _buildPackageInfo (/private/var/folders/fr/nddjk7yj29v5f4058jgqbn400000gn/T/angular-cli-packages-VL3Ti2/node_modules/@angular/cli/src/commands/update/schematic/index.js:434:52)
at /private/var/folders/fr/nddjk7yj29v5f4058jgqbn400000gn/T/angular-cli-packages-VL3Ti2/node_modules/@angular/cli/src/commands/update/schematic/index.js:708:53
at Map.forEach ()
at /private/var/folders/fr/nddjk7yj29v5f4058jgqbn400000gn/T/angular-cli-packages-VL3Ti2/node_modules/@angular/cli/src/commands/update/schematic/index.js:707:27
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async callRuleAsync (/private/var/folders/fr/nddjk7yj29v5f4058jgqbn400000gn/T/angular-cli-packages-VL3Ti2/node_modules/@angular-devkit/schematics/src/rules/call.js:77:18)
Exception or Error
No response
Your Environment
Angular CLI: 16.0.1
Node: 18.16.0
Package Manager: npm 9.5.1
OS: darwin arm64

Angular: 16.0.1
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, localize, material, platform-browser
... platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1600.1
@angular-devkit/build-angular   16.0.1
@angular-devkit/core            16.0.1
@angular-devkit/schematics      16.0.1
@angular/fire                   7.5.0
@schematics/angular             16.0.1
ng-packagr                      16.0.1
rxjs                            7.5.7
typescript                      5.2.2

Anything else relevant?
No response

Can you please share your package.json before the update?


fix: Issue where schemas could be assigned grant permissions after ob…

…jects referencing them.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 96.132%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
1250

Relevant Lines:
1295

💛  - Coveralls


Label not clickable on form?

Is there any way to add the attribute 'for' to the label in ?

If you give the input an id it will add the for attribute matching the id provided.


Description

Replace the usage description... by description::Symbol... and no need anymore of makeDescription, since description is already aTuple. I keep only the concept of partial vs. full description.

Done!


Address Autocomplete missing city when using Azure

Hi,
Your toolkit is fantastic - thank you for putting this out there!
One glitch I noticed while playing around with it is that the city is left blank when using Azure Maps. I switched to Bing it filled it in without any issues.  Only problem is that Bing doesn't always seem to be able to find every address (at least around where I live - Azure and Google seem to find these addresses without any issues).
One potential thought for future development: it would be nice to be able to type a business name into the search and it would fill out the address.  Not sure if this is possible or not.
Thanks again!
Tommy

I am also seeing this issue. However one thing of note is that it does work specifically for the address shown in the gifs in the documentation: 25 East Court Street, Greenville, SC 29601 I can't find any other address it works for though


Discord Server

https://www.youtube.com/watch?v=8Mgde34m3zI
In the description you link to a subpage called riverside.rocks/discord
Can you add a subpage called "discord" that will redirect to your discord server's invite?

Done.

seems like the discord invite is also invalid -- that may only be for me tho


Lowercase command changes property names, incorrectly

When running the ABL: Convert to lowercase command on the code below,
USING Progress.Json.ObjectModel.*.

DEFINE VARIABLE oJson AS JsonObject.

JsonDataType:STRING.
JsonDataType:NULL.
JsonDataType:NUMBER.

oJson:Add('k', 'v').

The STRING and NULL properties are lowercased, but not the NUMBER.
The Add method is also lower-cased.
using Progress.Json.ObjectModel.*.

define variable oJson as JsonObject.

JsonDataType:string.
JsonDataType:null.
JsonDataType:NUMBER.

oJson:add('k', 'v').

Similar behaviour can be seen with OOABL properties that have ABL keyword names - something like a "Name" or "Value" property.

Will have a look, I think this is the same issue as in the KeywordCase rule.

Fixed in develop


1.20.1 verson

Hi, im running 1.20.1 minecraft server,. is there a possibility you could downgrade it? i have no experience in coding

The mod supports 1.20-1.21 with all minor versions now.


Fix Logo Alignment and Visibility in Navneet Community Partner Section

I noticed that the logos under the Navneet Community Partner section are not properly centered, which affects the overall layout. Additionally, the Next.js logo is not visible. I would like to help by addressing these issues to enhance the visual consistency of the page.
https://github.com/user-attachments/assets/f4c57126-959a-4ef1-bb07-8ed2d9301797

bhai ye navneet kya hai bhai


Different font being used for next event

Hello,
Would it be possible to use the standard font for the next event text?
It looks very tight and different from the font used in the menu bar.
Here's a side by side using the same text:

Nevertheless, amazing app, thanks for the work.

The idea was to make it as compact as possible, because of the limited space in the menu bar.
I will add a font size configuration to the app settings.

Amazing, thank you for how quickly you implemented and released this!


Support Laravel 9

If i try to install this package in a l9 project composer fail to install
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires beyondcode/laravel-favicon ^1.4 -> satisfiable by beyondcode/laravel-favicon[1.4.0].
    - beyondcode/laravel-favicon 1.4.0 requires illuminate/http 5.6.*|5.7.*|5.8.*|6.*|7.*|8.* -> found illuminate/http[v5.6.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev, v7.0.0, ..., 7.x-dev, v8.0.0, ..., 8.x-dev] but these were not loaded, likely because it conflicts with another require.

You can also try re-running composer require with an explicit version constraint, e.g. "composer require beyondcode/laravel-favicon:*" to figure out if any version is installable, or "composer require beyondcode/laravel-favicon:^2.1" if you know which you need.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

1.4.0 doesn't work with the latest Laravel as well.
Problem 1
    - Root composer.json requires beyondcode/laravel-favicon ^1.4 -> satisfiable by beyondcode/laravel-favicon[1.4.0].
    - beyondcode/laravel-favicon 1.4.0 requires illuminate/http 5.6.*|5.7.*|5.8.*|6.*|7.*|8.* -> found illuminate/http[v5.6.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev, v7.0.0, ..., 7.x-dev, v8.0.0, ..., 8.x-dev] but these were not loaded, likely because it conflicts with another require.

You can also try re-running composer require with an explicit version constraint, e.g. "composer require beyondcode/laravel-favicon:*" to figure out if any version is installable, or "composer require beyondcode/laravel-favicon:^2.1" if you know which you need.


change memory order of operations on disconnect_error_ in client_channel

Change the memory order of operations on disconnect_error_ in client_channel according to @dklempner 's comment in #19634 .

#18126 #19614 #19422 #19726


Waiting for BQ Query and Export jobs for more than 5 minutes.

Without this change, the pipeline fails after exports/queries do not complete in 5 minutes.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Python 3.6 PostCommit

Run Website_Stage_GCS PreCommit

r: @chamikaramj @kamilwu PTAL
currently, ReadFromBigQuery is only waiting up to 5 minutes, and failing the job if the query/export don't finish in time

Added a comment above :)
LGTM other than that

retest this please

retest this please

retest this please

Run Python PreCommit

Run Python 3.7 PostCommit

Run Python PreCommit


Azure SQL resource naming considerations

Notably missing from this document are the requirements/limitations for Azure SQL naming. Background on Azure SQL names is especially important for at least three reasons:

the currently unprotected issues uppercase letters cause (it is possible to create SQL Server with upper case, but then other Azure features will fail, including Pipeline tasks and AD Administrator assignment via portal)
the fact that SQL Server is one of the few globally-unique names
the parent/child naming relationship of SQL Server and SQL Database

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

ID: 0e540bb6-9f72-0529-28ea-ae82bc869835
Version Independent ID: 24e71392-985b-59fb-1552-46395b169612
Content: Naming conventions for Azure resources - Best practices for cloud applications
Content Source: docs/best-practices/naming-conventions.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @telmosampaio
Microsoft Alias: pnp

Thank you for your feedback! We have assigned this issue to the author to review further and take the right course of action.


Request to make Tallying Rule Score per Validator queryable easily using method sui_getLatestSuiSystemState

New
As a validator or someone delegating to a validator, tracking a validator's individual Tallying Rule Score can be an important and regular task, especially since it is factored in the rewards calculation:
v : The Tallying Rule score for each validator, with validators with scores of 1 obtaining relatively more stake rewards than validators with scores of 0.
I was not able to find any documentation on how the tallying rule score is calculated. I am aware that the LatestSuiSystemState includes ValidatorReportRecords, but it is not clear how to calculate the individual tallying rule score from the information which validators reported a specific validator.
I would like to request the addition of a key/value TallyingRuleScore to the ActiveValidators array in the sui_getLatestSuiSystemState RPC method and an addition to the Sui documentation to explain how the individual tallying score gets calculated from ValidatorReportRecords.
If I'm missing something obvious here, please let me know :)

Looks useful to us, also having the voting_power::quorum_threshold() value for the epoch would be fine to have to be know when a validator is going to be slashed due to their tallyingRuleScore


Victron integration stops updating values

After updating my the victron integration, I noticed it stops updating/giving current values after a while and only starts giving the current values after reloading the integration.

I have exactly same behaviour.
Latest update 0.1.3
Home Assistant 2023.8.4
Supervisor 2023.08.1
Operating System 10.5
Frontend 20230802.1 - latest
No changes in meantime on the Victron settings or equipment.

@shaman171 and @iJAY86, are you seeing errors like this?

If so, for which devices? My Battery and my Multiplus-ii seem fine, but my Lynx Smart BMS has these errors.

No I do not see any of such errors in HA. There are just no longer updated data and it stop on some "random" last value which was readed from Victron.
Then HA simply "think" that thi value is right and it get no longer any more value and show this one.

If so, for which devices? My Battery and my Multiplus-ii seem fine, but my Lynx Smart BMS has these errors.

No errors

I wonder if we're all seeing this issue: #83

Same issue here.., data no longer updating. And I also had the solar data entities removed, simular to what 'askpatrickw' is experiencing.

Same issue here. The integration worked nicely on version 0.12. Since I upgraded a few days ago it hangs after a short while messing up my PV-based EV-charging.
Log entries (both occured at the time the integration stopped updating data):
Warning:

Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht (Error caused by a custom integration)
Logger: custom_components.victron.coordinator
Source: custom_components/victron/coordinator.py:76
Integration: victron (documentation, issues)
First occurred: 26. August 2023 um 18:34:29 (1 occurrences)
Last logged: 26. August 2023 um 18:34:29
no valid data returned for entities of slave: 20 (if the device continues to no longer update) check if the device was physically removed. Before opening an issue please force a rescan to attempt to resolve this issue

Error:

Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht (Error caused by a custom integration)
Logger: custom_components.victron.coordinator
Source: helpers/update_coordinator.py:229
Integration: victron (documentation, issues)
First occurred: 26. August 2023 um 18:34:29 (1 occurrences)
Last logged: 26. August 2023 um 18:34:29
Error fetching victron data: Fetching registers failed

Not sure, if it is related. The latest Home Assistant bugfix mentions something about Modbus Slave issues:
https://www.home-assistant.io/blog/2023/08/02/release-20238/#release-202384---august-24
https://github.com/home-assistant/core/pull/98644. I'll upgrade to 2023.8.4 and monitor, if that helps.

same problem quits updating until I do a reload of the integration then it works for about a day

The only change from version 0.1.2 to 0.1.3 was an update to the PyModbus library from version 3.1.1 to 3.4.1.
https://github.com/sfstar/hass-victron/commit/36727e8764538f77263d3e0d6fa1b9d46cfaaf80
Version 3.5.0 is already released, and it loos like it has several fixes with the word "retry" in the name.
https://pymodbus.readthedocs.io/en/stable/changelog.html

The only change from version 0.1.2 to 0.1.3 was an update to the PyModbus library from version 3.1.1 to 3.4.1. 36727e8
Version 3.5.0 of the PyModbus library is already released, and it looks like it has several fixes with the word "retry" in the name. https://pymodbus.readthedocs.io/en/stable/changelog.html

okay still have the issue of the integration freezing and not responding so not sure what you are getting at?

@trentjw What I am getting at is that you either need to dowgrade to version 0.1.2 or wait until the author releases a new version which contains the 3.5.0 library of PyModbus which hopefully fixes the issue we are having.

Not sure, if it is related. The latest Home Assistant bugfix mentions something about Modbus Slave issues: https://www.home-assistant.io/blog/2023/08/02/release-20238/#release-202384---august-24 home-assistant/core#98644. I'll upgrade to 2023.8.4 and monitor, if that helps.

Installing Homeassistant 2023.8.4 unfortunately did not improve the situation. The Victron integration still hangs after some time

I have the exact same problem since the upgrade.  😟

Me too…

reverting back to version 1.2 as suggested is a fix for now until it is broken later I went back to it and it reporting as it should
https://github.com/sfstar/hass-victron/tree/v0.1.2

Same problem even with the latest update 0.1.5
Stops updating data for all entities after short time

I see this after a short network disconnect. The plugin does not restart after the network Returns.
I have a unreliable connection over LTE, so I see this Situation almost daily.

@pos-ei-don could be the reason. I will check this. Was never a problem for months until around JULY

I can confirm also for me after update of HA and victron to latest versions it works already since yesterday without troubles.
Thank you for fix! 👍

Ok Internet connection was stable but updating stopped for all entities about 8h ago. Any other ideas to solve it?

Just to be clear: I spoke from small interruptions, nothing you can find with a ping per second.

Ok that could be a reason. I also use LTE connection.
I update my VenusOS device hoping this will solve the problem but no luck. It works for some hours and then stops updating.

Since this issue was reported there have been several improvements made regarding connection stability and recovery.
Are the issues you are experiencing still regularly occurring or can the issue be closed as resolved?

It happens occasionally every few weeks. Reloading the integration solves the problem.

I found a solution for my update problem. I use the external control ess mode and set the acpowersetpoint every 10 seconds (that works fine). In order to update the sensor values im home assistant, the configured update interval for the plugin must be < 10 seconds (e.g. 9 seconds).
Reason and possible bug in the plugin:
Whenever i set acpowersetpoint the plugin prints [custom_components.victron.coordinator] Manually updated victron data in the debug log but does not update the sensor values in home assistant.
Now after configuring the update interval for 9 seconds, i see custom_components.victron.coordinator] Fetching victron data and [custom_components.victron.coordinator] Finished fetching victron data in 0.463 seconds (success: True)in the logs.

Sorry was on holiday the last couple of weeks and noticed remotely once that the integration indeed stops updating values. So this issue needs further investigation.


chore: added category file

Related to #11

I'll do minor adjustments later but that is looking good :+1:


Type system support for structs ending in zero-sized arrays

Binary Ninja Version: 2.4.2993-dev, 479483ef
Platform: Ubuntu 20.04.3 LTS
The type system does not allow creating a structure which ends with a zero-sized arrays.  For example, I'd like to create a structure that looks like the following (32-bit integers, pointers, etc.):
struct mystruct {
  int tag;
  int length;
  uint8_t value[0];
};

This structure should be 8 bytes in size.  Instead, the output in the types window after creating this structure is the following:
struct mystruct
{
    int32_t tag;
    int32_t length;
    uint8_t* value;
};

The types window also specifies this as a 12-byte structure (which is correct for the structure that was created, but not for the one I'm trying to create).
Please add support for structures which end with a zero-sized array, as these are common for variable-sized structures in C.

This issue with the parser producing the wrong output was fixed last night. The issue where we infer the type of items past the end of the structure still needs to be done.

Closing this in favor of: #2655


internal err occurred ?? it there any detail ?

the doc is too little for pipeline

Does this error continue to show up, @mouuii ?  If you click on "Inspect" and bring up the browser console does it show any more details?


Update example to avoid showing go vet warnings

Minor fix to help users not get confused by the sneaky go vet warning.
Previous output
{
    "value": "./prog.go:6:4: Println arg list ends with redundant newline\nGo vet exited.\n\nHello WebDriver!\n\n\nProgram exited."
}
./prog.go:6:4: Println arg list ends with redundant newline
Go vet exited.
Hello WebDriver!

New output
{
    "value": "Hello WebDriver!\n\nProgram exited."
}
Hello WebDriver!

Thank you!


[PF-1246] Force update log4j transitive dependencies

Force transitive log4j api and slf4j library dependencies to the latest version due to security concerns. We do not use log4j core, so nothing to update there.

Per initial discussion with infosec:

That's a good question, I'm not sure. If it's possible to upgrade the underlying version of log4j or to implement the mitigation that would be better than not.

We don't use the log4j core implementation anywhere (we use Logback instead) but we do pull in the log4j api and slf4j adaptor libraries as transitive dependencies. I think we don't strictly need this, but I'm not 100% confident in that.

See https://github.com/DataBiosphere/terra-external-credentials-manager/pull/84 and https://docs.spring.io/spring-boot/docs/2.6.1/gradle-plugin/reference/htmlsingle/#managing-dependencies.dependency-management-plugin.customizing


annotation with prokka: tutorial

any feedback appreciated, thanks!

@AnnaSyme anything we can help you here?
We would also appreciate any feedback about the template, the ease of use and contribution ... any feedback would really help us in this project :)

Thanks @bgruening - all seems clear to me so far, but wondering what the next steps are. I think it's ready for merging.

@AnnaSyme all good. I guess I was just confused by the WIP in the header. I have removed the WIP now. We just need to wait for an other review I guess and then we can merge it.
Thanks a lot!

@Slugger70 @tseemann does one of you have time to review this?

@AnnaSyme in this PR are many images that are not used. At least I could not find it. Can you please remove them?

Thanks @bgruening - removed unnecessary images

Thanks @AnnaSyme! Thats a great start for the Genome Annotation community here!


Add tag app; Concept model many-to-many w/ Paper

more context in the Content Grouping doc

@vlazzle the code looks good but I think we should make a change
I think concepts should be associated with unified_document (which is out generic document object) as opposed to papers. That way, we can apply concepts to questions, posts, ... in addition to just papers. I think the change here shouldn't be too time consuming to make. @le-sun / @lightninglu10 lmk if this makes sense
Also, I think it may make sense to open a new PR with just the changes. I'm not sure why I'm seeing what appears to be unrelated diffs in your PR (see my comments). Maybe something happened during rebase?

I agree. Having concepts under unified document makes it more generic.

Closing this, let's continue in #1155


[FR] - Keys import/export for Daedalus

External
Please implement a method for any advanced user/enduser/operator/owner to be able to import address & staking vkeys/skeys into Daedalus via an "Advanced" option in the Menu or via specific cli calls on the commandline to the cardano-wallet.
Also please add the option to export keys like the staking vkey/skey in some way.
Thx!

This is for the Daedalus team

@gitmachtl This is probably stale by now but this belongs in https://github.com/input-output-hk/daedalus


Use checkLocationServicesIsEnabled to check location service without show popup

What I need is show returned value if current location is enabled or not without show popup. Thanks

This plugin only popup show if location service is turned off.

@webyonet  I think it does show even if location is enabled :/

@privateOmega
Simulator may have this problem. Would you test it on a real device?

@webyonet I am using it on a real device. Android 7.1.1. Can you please confirm and modify it?


client: given both retries and errors, prefer retries

Now that some errors (e.g. UnknownTopicOrParition) set both a retry flag and
an error, we should handle retries first. This is sort-of non-idiomatic in that
updateMetadata now returns a non-nil error with a "valid" result, but I think
that's OK.
@Shopify/kafka follow-up to #372

+1 this makes sense

Good tests.
(It would be nice to also have tests that the retry behaviour is sane - somehow I missed writing any of those the first time around - but one thing at a time).


[IOTDB-2468][Compaction]There is 'null' in logs

[Compaction]There is 'null' in logs

Coverage decreased (-0.03%) to 67.72% when pulling b00658c9ffb29f2ff0956b06a9ff6514ec02b986 on choubenson:fixCompactionBug into ed816e440d9f1a99c96b2bb69d88bb813553448e on apache:master.


Engine.update() with a large delta crashes browser tab

I inadvertently passed a big delta to Engine.update and ended up crashing my browser tab with 100% CPU usage.
This code reproduces it for me but I suppose the actual delta value might depend on the hardware.
const crash = () => {
  const { Engine, Bodies, World } = Matter
  const engine = Engine.create()
  const body = Bodies.circle(0, 0, 1)

  World.add(engine.world, body)
  Engine.update(engine, 1000000)
}

This Codepen crashes for me in Safari, Chrome seems OK but with high CPU usage: https://codepen.io/scriptie/pen/MWYXWyW?editors=0011

well you see, its trying to run a lot of time in a little bit of time, its like if your playing a game, and you try to run it at 1000x speed, it doesnt work very well trying to do 1000x more things at the same time.

@LeotomasMC Didn't seem sarcastic! I assumed it was to do with the number of calculations involved (bigger delta == more stuff I guess).
I suppose from a usability perspective I'd enforce some kind of (configurable) delta cap to prevent locking the browser. I appreciate it's the library consumer's responsibility, but it would be a 'nice-to-have' to avoid having to kill browser tabs during development.

the delta is the delta time, or how much time it is trying calculate for. im not sure exactly how its implemented though (is it doing many updates, or 1 update calculating longer time, or a mix of both).
out of curiosity, how do you end up with a large enough delta to crash the browser?

@anthonygood I renamed the const to max_cpu_ms in the last version . Which is configurable to the lowest expected fps, here 10 fps for 100 ms. But can be configured down to 3 or 1 fps if needed. Note that in case the fps is tool low, there is no solution to prevent instabilities as the jitter becomes too big to get compensated by the delta corrections.
@LeotomasMC in case the tab is in the background, and the user reactivate it after 5 min or more, that delta time, if fed to the physics engine result in a huge number that can result in lots of NaN and might lead to crashing the browser I assume. So limiting the max delta time fed to the physics engine is important.

Yeah I'd say this is down to the library user to maintain a reasonable delta but I know what you mean. I think this is already implemented in the built in runner, but intentionally not directly in the engine which is considered lower level.
Not sure what is going on specifically to cause this but essentially the lack of accuracy caused by a high delta causes growing instability, so even if any errors were fixed it would definitely still be unusable!


Add package dependencies for the install script.

Add package dependencies needed for the install script, or else the script will fail on Linux distributions without these already installed.

we do not intend to support distributions where those are not installed: https://docs.pivpn.io/faq/#what-boardsoses-does-pivpn-support and we are currently in the process of adding support for Alpine (already available on test branch).
https://github.com/pivpn/pivpn/pull/1567

Those packages are available by default on the distributions we support.

Maybe there are some VPS out there without those packages (although not having systemd is unusual). Is that the case @yanxke?

the script will fail on Linux distributions without these already installed.

I think there are some hard to die debian based distros with the old sysvinit. all those packages come default in any modern distro. haven't found any without it even less on VPS.

Those packages are available by default on the distributions we support.

Maybe there are some VPS out there without those packages (although not having systemd is unusual). Is that the case @yanxke?

It's actually for development and development in Docker on the default Ubuntu image.  Since the scripts look for files in hard coded locations, I don't want to change and mess up my regular settings.  Adding these dependencies don't hurt the normal workflow since they are required anyway.

Adding these dependencies don't hurt the normal workflow since they are required anyway.

The first two for sure, but the third is not, and even conflicts with other packages (I tested on ubuntu).
$ sudo apt install systemctl
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 init : PreDepends: systemd-sysv but it is not going to be installed
E: Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.

It's actually for development and debugging in Docker on the default Ubuntu image.

I personally use QEMU with -snapshot so that changes are discarded when turning off the VM.

Those packages are available by default on the distributions we support.

Maybe there are some VPS out there without those packages (although not having systemd is unusual). Is that the case @yanxke?

It's actually for development and debugging in Docker on the default Ubuntu image. Since the scripts look for files in hard coded locations, I don't want to change and mess up my regular settings. Adding these dependencies don't hurt the normal workflow since they are required anyway.

for docker is not that linear as adding the dependencies, its actually much more than that and you might be better of with the Alpine support added recently on the test branch.
you will still have the issue with iptables rules, net.ipv4.ip_forward and a bunch more of others.
PiVPN its not that docker friendly (yet)


[Task]: Add linting checks for JSON configs to avoid parsing errors

SubTasks
Recently we observed that due to an error in the setting JSON, the smart JSON substitution toggle was removed from the UI.
Ref: https://theappsmith.slack.com/archives/C040LHZN03V/p1721124893238579

pls provide access of slack if possible

Moved to in progress now, as this is getting prioritised now.


Selected publication gone

Hi again,
In the current version, the selected publication section is disappeared. See the Demo
http://themes.gohugo.io/theme/academic/
which has selected publication section and the example site
http://gcushen.github.io/hugo-academic-demo/
I do remember that by setting the parameter selected to true, we have  selected publication section. But apparently, it's gone in the latest version ...

Sorry it was the problem of Hugo! I needed to install the latest version :)


The cloned emulator blob container doesn’t auto display when cloning one new cloned blob container

Storage Explorer Version: 1.23.0-dev
Build Number: 20220124.2
Branch: main
Platform/OS: Windows 10
Architecture: ia32
Emulator Version: Azurite 3.14.3
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Install and run Azurite 3.14.3 -> Launch Storage Explorer.
Expand 'Local & Attached -> Storage Accounts -> (Emulator - Default Ports)(Key)' -> Blob Containers.
Create a blob container -> Right click the blob container -> 'Clone...'.
Type 'clo1' in the clone dialog -> Click 'Clone'.
Right click the new cloned blob container -> Click 'Clone...'.
Type 'clo2' in the clone dialog -> Click 'Clone'.
Check whether the blob container 'clo2' auto displays.

Expected Experience
The blob container auto displays.
Actual Experience
The blob container doesn't auto display.
Additional Context

The blob container displays after manually refreshing.
Here is the record:

Verified this issue on the build main/20220325.1. Fixed.


Add possibility to register components in module

I think we can have the module name collisions.
For example, you have ConfigModule in the Users and Groups page. In both cases it compiles to the "ConfigModule" string.
I suggest adding autoincrement counter to the module decorator to avoid this problem.


Dlib not installing

I get this error as seen below.
Please advise
(tf-gpu) C:\Users\Conda\HelloWorld\Face>pip install C:\Users\PC User\Downloads"dlib-19.22.99-cp39-cp39-win_amd64.whl"
WARNING: Ignoring invalid distribution -illow (c:\users\conda\envs\tf-gpu\lib\site-packages)
ERROR: Invalid requirement: 'C:\Users\PC'
Hint: It looks like a path. File 'C:\Users\PC' does not exist.

Please advise
(tf-gpu) C:\Users\Conda\HelloWorld\Face>pip install C:\Users\PC
User\Downloads"dlib-19.22.99-cp39-cp39-win_amd64.whl"
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
ERROR: Invalid requirement: 'C:\Users\PC'
Hint: It looks like a path. File 'C:\Users\PC' does not exist.
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/datamagic2020/Install-dlib/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP3TB6DGEQBVDMTFK75QGLTVXLBVXANCNFSM55P7L4RA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks!  That got it going.
On Thu, Aug 4, 2022 at 2:29 AM Data Magic @.***> wrote:

Comand is not prepared well, run below one.
pip install "C:\Users\PC User\Downloads\dlib-19.22.99-
cp39-cp39-win_amd64.whl"
On Thu, 4 Aug, 2022, 12:56 AM Bill Strum, @.***> wrote:

I get this error as seen below.
Please advise
(tf-gpu) C:\Users\Conda\HelloWorld\Face>pip install C:\Users\PC
User\Downloads"dlib-19.22.99-cp39-cp39-win_amd64.whl"
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
ERROR: Invalid requirement: 'C:\Users\PC'
Hint: It looks like a path. File 'C:\Users\PC' does not exist.
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/datamagic2020/Install-dlib/issues/2, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AP3TB6DGEQBVDMTFK75QGLTVXLBVXANCNFSM55P7L4RA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/datamagic2020/Install-dlib/issues/2#issuecomment-1204818869,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2XRUJ3LYXDRODZ4LAFCSTVXNPNDANCNFSM55P7L4RA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Not to sound greedy but do you have dlib gpu creator?
---------- Forwarded message ---------
From: Bill Strum @.>
Date: Thu, Aug 4, 2022 at 5:47 PM
Subject: Re: [datamagic2020/Install-dlib] Dlib not installing (Issue #2)
To: datamagic2020/Install-dlib <
@.>
Thanks!  That got it going.
On Thu, Aug 4, 2022 at 2:29 AM Data Magic @.***> wrote:

Comand is not prepared well, run below one.
pip install "C:\Users\PC User\Downloads\dlib-19.22.99-
cp39-cp39-win_amd64.whl"
On Thu, 4 Aug, 2022, 12:56 AM Bill Strum, @.***> wrote:

I get this error as seen below.
Please advise
(tf-gpu) C:\Users\Conda\HelloWorld\Face>pip install C:\Users\PC
User\Downloads"dlib-19.22.99-cp39-cp39-win_amd64.whl"
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
ERROR: Invalid requirement: 'C:\Users\PC'
Hint: It looks like a path. File 'C:\Users\PC' does not exist.
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
WARNING: Ignoring invalid distribution -illow
(c:\users\conda\envs\tf-gpu\lib\site-packages)
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/datamagic2020/Install-dlib/issues/2, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AP3TB6DGEQBVDMTFK75QGLTVXLBVXANCNFSM55P7L4RA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/datamagic2020/Install-dlib/issues/2#issuecomment-1204818869,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2XRUJ3LYXDRODZ4LAFCSTVXNPNDANCNFSM55P7L4RA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Bring sync functionality inside a coroutine

Done with 6ad80be66073ba7ba9a6a9bda1ab836da28d1c61. :fire: :fire: :fire:


Empty notification that doesn't go away.

Just opened my GitHub page, and it seems like I have a notification, as you can see here:

When I open it, it says I have one unread notification and that it's coming from your repository. However, when I open it, there's nothing. As you can see below:

I don't know how to reproduce it, I just opened GitHub and it was there. I tried refreshing the page and browsing into other pages, but it's still there. Maybe it's a GitHub related glitch but because it has your repository on the notifications, maybe it's a bug or something. Any clue on what's causing that?

Just experienced it on mobile too, it's not a bug from the extension.
Something from the GitHub's end is acting weird.

Yeah I've seen this bug before, it wasn't ours


McLaren Commit #2: Complement to McLaren Commit #1 --> addition of signal_source_path within MetadataTools

… to match the shape of the return from getSource

@microsoft-github-policy-service agree [company="{McLaren Automotive Ltd}"]

@microsoft-github-policy-service agree company="McLaren Automotive Ltd"


Remove phonenumbers import from top of PhoneNumberField

Currently, in the PhoneNumberField module we are importing phonenumbers at the top of the module. This prevents the field from working if phonenumbers isn't installed, but it also prevents the documentation from generating. If we replace the phonenumbers failed import with an unbound LocalProxy, then we can still get the same error, but generate docs without phonenumbers.
Context here: https://github.com/croscon/fleaker/pull/21#discussion_r91964342

This is closed cause we aren't playing around with extras_requires.


No text in the notification

Insertion
=> [#<RailsPushNotifications::Notification:0x007f1a15c33878   id: 7,   destinations:    ["ciJie4Om2aw:APA91bHckWobcMLNzOXxe9eXl0X4FuoDym9IlcBjvWOiwjtq0LptHEdy0ubgUahqb-79TF6X5J9eUUL-TRo3jACQBGmL1vPhP9cSb2TXnVCAU-iIlvQc1Kps-Yg0aUiVhjXZHBAq04Po"],   app_id: 7,   app_type: "RailsPushNotifications::GCMApp",   data: {:text=>"Awesome bags is in Live In near you"},   results:    #<RubyPushNotifications::GCM::GCMResponse:0x007f1a15824098     @canonical_ids=0,     @failed=0,     @results=[#<RubyPushNotifications::GCM::GCMResultOK:0x007f1a154037c0>],     @success=1>,   success: 1,   failed: 0,   sent: true,   created_at: Thu, 28 Jul 2016 09:50:22 UTC +00:00,   updated_at: Thu, 28 Jul 2016 09:50:22 UTC +00:00>]
Notification is receiving but no text is showing in the notification.
PFA.

Hi @errakeshpd
I think the problem is that you aren't parsing it properly on your device. The sent notification contains the data key and it will contain your { text: "Awesome ..." } following the official documentation.
I'd suggest printing it in the logs to double check what you're getting and then parse it properly.
Hope it helps.

Closing as inactive... Please reopen it if you need further help.


clean up semantics of createNativeQuery()

This work cleans up the API for creating native SQL queries:

introduces ResultSetMapping to reify @SqlResultSetMapping
allows non-entity classes to be passed to createNativeQuery(), just as they are allows as arguments of createQuery()

The JPA spec is quite vague in the area of native SQL queries, and it's possible that this improves on JPA and Hibernate ORM.

Note that I've designed this API in anticipation of:
https://hibernate.atlassian.net/browse/HHH-14051

Any objections to this change? I believe it's the right thing to do.

I've moved ResultSetMapping to its own package, and added javadoc. Going to merge this now.


Does it run in Background?

How can I put the server to run in background?

The answer to this question is highly dependent on what device you are running the server on.
For Windows 10 IOT on the Raspberry PI you can utilise a background task.
For regular Windows 10 I think the window needs to stay active for it to keep on running, if the screen is locked the app is put it into sleep mode (see: https://github.com/tomkuijsten/restup/issues/55).
Hope this helps.

Well, I'm developing for Windows Mobile. So, can I run the server in background?

Last time I looked at doing that (about a year ago) that wasn't possible, since you're only allowed to run very limited tasks in the background that are triggered by something. Also you wouldn't want to run much in the background on a phone anyway since the battery would drain pretty fast with that + other apps running at the same time.
But if you do find a way to do it, please let us know and we can add it to the FAQ.

As far as I know, there is no way to always run a background task on mobile. You can set triggers from which background tasks are run (timer, OS events etc), but you will only have a limited amount of time while running the background task and it can be killed by the OS. There is enough MSDN  documentation on this matter, I suggest to close the issue.

Agreed, closing.


SA1121: UseBuiltInTypeAlias

http://www.stylecop.com/docs/SA1121.html

I unfortunatly started working on this without looking at this issue. So I have a second implementation of this. I will open a pull request for it because it contains a bunch of unit tests which should be usefull with your implementation as well.
I don't know if a code fix is needed for this because visual studio has a refactoring for this.

We can talk about the code fix possibilities in #304.
I propose that we review pull request #306 and merge it, and then you update #350 to just address #303. Does this plan work for you? Also, can you help with the review of #306?


buildDotnetModule: fix sandboxed builds on darwin

Description of changes
ICU tries to unconditionally load files from /usr/share/icu on Darwin, which makes builds fail in the sandbox. Thus, let's disable ICU during the build on Darwin by setting DOTNET_SYSTEM_GLOBALIZATION_INVARIANT 0.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

cc @7c6f434c for potential hints on alternate solutions

I don't use Darwin and haven't used anything .NET for quite some time…
It looks like there is no from-source build of .NET in Nixpkgs, am I right? Patching binaries to put a longer path to the store copy of ICU doesn't sound like a good time, so just disabling ICU sounds reasonable.

I started doing a source-build of the dotnet sdk: https://github.com/NixOS/nixpkgs/pull/190129
I haven't had a chance to work on it in a while, but I did have it patching dependencies like ICU to the nix store.

What I'm certainly confused about is why the behavior differs between macOS and Linux -- if I recall correctly from stracing it, ICU never tries to access the data files on Linux. I'll look into it a bit more before merging this, to see if I can come up with anything better.

Are you sure this affects the output of the compiler, @mdarocha? That's unfortunate...
I'll try to look into this more.

Are you sure this affects the output of the compiler, @mdarocha? That's unfortunate...
I'll try to look into this more.

See https://learn.microsoft.com/en-us/dotnet/core/extensions/globalization-icu
If you consider this fix critical enough that forcing the changes mentioned there, go ahead. I would prefer to be careful about such a global change though.
Maybe you'll find an idea for another approach in the docs above? There's some mention of using a local icu library.

OK, so basically «nothing works on macOS with a reasonable build configuration» is getting upgraded to «significantly degraded internalisation support». And source builds seem to be a lot of work still. Not super-happy about All That, but it's an improvement for now.


RECONSTRUCT OTHER SCENES

HELLO, EXCUSE ME, THANKS FOR THE GREAT WORK, CAN I USE SPSNERF TO RECONSTRUCT OTHER SCENES

Hi, thank you for your interest in our work. Yes you can generate your own dataset following this: https://github.com/LulinZhang/SpS-NeRF?tab=readme-ov-file#1-prepare-dataset

ok，thank you


Too much ports can't be released after ddp-sharded training

🐛 Bug

I just tried the new feature ddp-shared training, everything goes well except that there are too much ports remained opened after training. Is the phenomenon normal?

Please reproduce using the BoringModel and post here

To Reproduce

...
trainer = Trainer(max_epochs=opt.epochs, gpus=2, accelerator='ddp', plugins='ddp_sharded')
trainer.fit(model, train_loader, val_loader)

Expected behavior

ports should be released right after training is finished
Environment
Note: Bugs with code are solved faster ! Colab Notebook should be made public !

IDE: Please, use our python bug_report_model.py template.

Colab Notebook: Please copy and paste the output from our environment collection script (or fill out the checklist below manually).

You can get the script and run it with:
wget https://raw.githubusercontent.com/PyTorchLightning/pytorch-lightning/master/tests/collect_env_details.py
# For security purposes, please check the contents of collect_env_details.py before running it.
python collect_env_details.py

PyTorch Version (e.g., 1.0):
OS (e.g., Linux):
How you installed PyTorch (conda, pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Additional context

do you also have hanging processes in the background, or just the blocked ports? (check with pgrep python or similar)
could this be a problem with ddp alone? or is it really only with fairscale/sharded? cc @SeanNaren

(base) ➜  ~ pgrep python
840
851
864
....
31307
31342

(base) ➜  ~ lsof -i
COMMAND     PID   USER   FD   TYPE    DEVICE SIZE/OFF NODE NAME
....
node       4138 aiyolo   19u  IPv4 870494046      0t0  TCP localhost:35723->localhost:38164 (ESTABLISHED)
python     4579 aiyolo   12u  IPv4 571538969      0t0  TCP localhost:49729 (LISTEN)
python     4579 aiyolo   14u  IPv4 571538971      0t0  TCP localhost:40373 (LISTEN)
python     4579 aiyolo   16u  IPv4 571538973      0t0  TCP localhost:51103 (LISTEN)
python     4579 aiyolo   18u  IPv4 571538975      0t0  TCP localhost:33943 (LISTEN)
python     4579 aiyolo   23u  IPv4 571538979      0t0  TCP localhost:35083 (LISTEN)
....

(base) ➜  ~ nvidia-smi
Wed Dec 16 16:56:29 2020
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 440.100      Driver Version: 440.100      CUDA Version: 10.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 108...  Off  | 00000000:01:00.0 Off |                  N/A |
| 71%   84C    P2   249W / 250W |   5778MiB / 11176MiB |     98%      Default |
+-------------------------------+----------------------+----------------------+
|   1  GeForce GTX 108...  Off  | 00000000:02:00.0 Off |                  N/A |
| 21%   54C    P2    61W / 250W |    689MiB / 11178MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      1363      G   /usr/lib/xorg/Xorg                            40MiB |
|    0      1486      G   /usr/bin/gnome-shell                          47MiB |
|    0     31342      C   ...gy/miniconda3/envs/pytorch36/bin/python  5677MiB | # not my process
+-----------------------------------------------------------------------------+

hope it can help understanding my question.

It seems that the opened ports could be normally released when performing a standard mnist train, so the phenomenon may be caused by something else.


Renaming repository

Hi all, just a note that I will be moving this repository under ergochat eventually: https://github.com/ergochat/
Not sure exactly what the repo name will be when it's under there, but will do my best to try and keep it similar.

Sling mentioned possibly putting /client and /ircmap into a separate branch until they're fully, properly production-ready (since all the other things in this repo are production-ready, but those two aren't. I got a note back from the person that I knew was using /client at some point - and they're using the newer irc lib fork thing that sling's been working on under here anyways.
So yeah, I'm totally happy to push /client and /ircmap into a separate branch, sounds good to me.


The month picker sometimes skips a month

Both on pressing the + sign or the arrow. Is it the date -> month conversion?

Added


Website link: http://www.jscsi.org should be http://jscsi.org

The main website link listed for this project fails to load because www subdomain doesn't seem to be configured.

fixed, thanks and sorry for the delay


Implement dependency injection

To have a single source of truth for data and following SOLID principle we can use Dagger Hilt to implement dependency injection in the app. So kindly assign this issue to me

@Yash-Parsana is the issue still open I'd like to work on it


Add zap/hclog interface

Accidentally targeted a branch other than main and then deleted it, reopened as #21


sails-mongo cant find some ObjectID's starts with 6

Node version:  10
Sails version (sails):  1.2.4
ORM hook version (sails-hook-orm): 3.0.1
Sockets hook version (sails-hook-sockets):  -
Organics hook version (sails-hook-organics):  -
Grunt hook version (sails-hook-grunt):  -
Uploads hook version (sails-hook-uploads):  -
DB adapter & version (e.g. sails-mysql@5.55.5<EMAIL_ADDRESS>Skipper adapter & version (e.g. skipper-s3@5.55.5):  -

Recently, probablya after Mongo Atlas upgrade to 4.0, sails-mongo can't seem to find ID's starting with 6.
For example;
Model.findOne({ id: '5f6dec64b3471800aca76559' )} returns result
Model.findOne({ id: '60e58bc917317c00ae752f04' )} returns undefined
I can also confirm this record exists on MongoDB Atlas dashboard.
it's a weird one.

Hi @sercanov, thanks for reporting this. I wasn't able to reproduce this on a MongoDB Atlas cluster using MongoDB 4.4.6.
Are you able to query the records with IDs starting with 6 by other criteria?
If you can create a minimal sails app in a github repo that reproduce the issue you're seeing, we'd be happy to take another look.


Appget List Packages Feature Request

When managing a large number of packages as an administrator it is often necessary to check and see if an existing package is installed either by package name or package name and version number. On Linux this can be accomplished via apt list --installed or yum list all, etc. On Windows, Chocolatey supports this with choco list --local-only (I think).
Ideally, the output would lend itself well to being searched by grep or Select-String or redirected to file with one line per package.

Done. See https://docs.appget.net/commands/#list


Use powershell commands in script task

Hello, sorry for the perhaps noobish question but is it possible to use the powershell sdk commands inside a regular script task and not write a complete custom task?
What I want to do in the end is loop though a parameters.xml file and look up the parameters in there against the running release definition. I've seen that this can be done with environment variables, but that does not include secret variables, something that i'd need for my scenario.
Perhaps there is a different way to do this than using the sdk?

No. With the current design ad-hoc scripts don't get access to secret variables.
I believe there are a few tasks on the gallery that do token replacement within files.

Allright, thank you for the info :)
I will look and see if I find one and perhaps look at writing a custom one.

I ended up using this Task, I think that's the one you were referring to @ericsciple
https://marketplace.visualstudio.com/items?itemName=qetza.replacetokens


router documentation is unclear about CanActivate vs. CanActivateChild with component-less routes

I initially created this Stackoverflow question, but was asked to file this as an issue here for further discussion: http://stackoverflow.com/questions/40163348/canactivate-vs-canactivatechild-with-component-less-routes?noredirect=1#comment67759108_40163348

Is it possible for anyone from Angular to respond to this?

As one S.O. respondent noted, the difference is explained in the docs (my emphasis added):

As we learned about guarding routes with CanActivate, we can also protect child routes with the CanActivateChild guard. The CanActivateChild guard works similarly to the CanActivate guard, but the difference is its run before each child route is activated. We protected our admin feature module from unauthorized access, but we could also protect child routes within our feature module.

The issue becomes apparent when you consider navigating from child-to-child (see respondent's example).  The CanActivate on the parent component presumably allowed navigation to the first child. The parent's CanActivate is not called again as long as sibling children navigate among themselves. But the parent's CanActivateChild will be called.
I'm not sure if this clarification is needed in the docs. @brandonroberts ?

If any clarification could be added, its that CanActivateChild can be used without a component-less route, but the difference between it and CanActivate is already explained.

Well, at least now that the SO answer is there and properly google-indexed people (like me) that don't get the docs on a first read can find the relevant information and an example. :-)

@brandonroberts
Looks like CanActivateChild documentation is incorrect.

We can have multiple guards at every level of a routing hierarchy. The router checks the CanDeactivate and CanActivateChild guards first, from deepest child route to the top. Then it checks the CanActivate guards from the top down to the deepest child route.

It doesn't make sense to me at all to run CanActivateChild from bottom to top + it doesn't actually work like this http://plnkr.co/edit/H7plpGEh83LjWKWtUtff?p=preview
what do you think?


Add svg format to $fileUploadBlacklist

1. Why is this change necessary?
The .svg file format allows the insertion and execution of arbitrary javascript leading to XSS attacks
2. What does this change do, exactly?
Do not allow uploading .svg image files to the server
3. Describe each step to reproduce the issue or behaviour.
The step to reproduce the issue is described in detail in the report: https://huntr.dev/bounties/778474ad-8ee4-48a6-8562-7c56ee222d44/
4. Please link to the relevant issues (if any).
https://huntr.dev/bounties/778474ad-8ee4-48a6-8562-7c56ee222d44/
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

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


When restarting the pseudo-distributed sandbox the clients do not find the combiner

If you start the example sandbox as such:
docker-compose -f ../../docker-compose.yaml -f docker-compose.override.yaml up
Then stop, and restart, the clients do not manage to attach to the combiner (now set in the db).
Workaround: Flushing the db before restart.
How if should work: The default combiner name is "combiner" and the clients should manage to get assigned to this upon restart.
Severity

[ ] Critical/Blocker (select if the issue makes the application unusable or causes serious data loss)
[ ] High (select if the issue affects a major feature and there is no workaround or the available workaround is very complex)
[x] Medium (select if the issue affects a minor feature or affects a major feature but has an easy enough workaround to not cause any major inconvenience)
[ ] Low (select if the issue doesn't significantly affect the user experience, like minor visual bugs)

This has been solved via a more robust check usign grpc health checking.


Integration tests - don't check priority for tests using short_tag fixer

Closes #1615

ping @gharlan

Thanks!


Identifying the satellite

Thank you for your work on this project.
One of the key features that rhasspy had was being able to identify the satellite that was sending the message that allowed for logic to process the response based on location. Is it possible to have this added?
Another idea but one that may have far less use cases is the option to specify a media player in HA to receive the response rather than the device. I know it is possible to customize the response manually for each intent but it would be useful to be able to specifiy in the the start script of the satellite and have HA use it.
Thanks again for your work.

Ive just come across this HA community message re a change merged a couple of days ago.


Add the entire survey to the navigation drawer

Issue

close #652

Overview (Required)

Add the entire survey to the navigation drawer. At this point, go to https://google.com when you press it.

Links

Screenshot

Before
After

LGTM!!


What handler / mapper to call AppendMapping on?

Type of issue
Missing information
Description
In https://github.com/gzadro/.NET-MAUI-custom-Entry-control, the mapping code is run by calling Microsoft.Maui.Handlers.ElementHandler.ElementMapper.AppendToMapping(IView, Action):
https://github.com/gzadro/.NET-MAUI-custom-Entry-control/blob/f3dea0697762e6c8492a8259f71dc77ee3c78285/CustomInputMaui/MauiProgram.cs#L24-L30
In the Map method, the handler is being cast to an IEntryHandler after all:
https://github.com/gzadro/.NET-MAUI-custom-Entry-control/blob/f3dea0697762e6c8492a8259f71dc77ee3c78285/CustomInputMaui/Platforms/Android/CustomEntryMapper.cs#L21-L22
Looking at the API, I would find it more intuitive to use Microsoft.Maui.Handlers.EntryHandler.Mapper.AppendToMapping(IEntry, Action) (like in the documentation about handlers) but it seems to stop functioning when I use that method. If that downside would not be there, then it would be possible to avoid the casting.
I can't find documentation on where the difference is explained, though.
Can you please explain

what the difference between EntryHandler and ElementHandler is?
what the difference between EntryHandler.Mapper and ElementHandler.ElementMapper is?

Page URL
https://learn.microsoft.com/en-us/dotnet/maui/user-interface/handlers/customize?view=net-maui-8.0
Content source URL
https://github.com/dotnet/docs-maui/blob/main/docs/user-interface/handlers/customize.md
Document Version Independent Id
70227b04-808f-1370-186b-10cfef525959
Article author
@davidbritch
Metadata

ID: 659665da-2ff3-3550-2e0c-20902c15c3e6
Service: dotnet-mobile
Sub-service: dotnet-maui

Update - for me, empirically it seems that:

ElementHandler.ElementMapper should be used for custom-defined properties in an inherited class (e.g. if you add CornerRadius yourself to a subclass of Entry). A custom-defined CornerRadius mapping is not called via  EntryHandler.Mapper for me.
EntryHandler.Mapper should be used for built-in properties from the "built-in properties" like BackgroundColor or IsEnabled. The mappings for these properties do not work for me if I configure them using ElementHandler.ElementMapper.

It would really help me to have this explicitly documented though, as I'm not sure if this is the intended behavior, and if so - how should people know what property to map where?


Added method for computing reference angular acceleration, force, torque and rotor speeds

Added update_ref method in quadrotor_control.py.
Added a second derivative of yaw in circular_traj.py.
Added a line in simulate.py for testing
Used basic_usage.py to test.
I have trie to plot the computed Force and Moments, and they look reasonable.

Nice addition, thanks!


Cannot compile swift example with Xcode 9.2

I cannot run swift example, I have got these errors after pod install command:

I have the same issue

Any update on the same I am too facing the same issue.

Yes, you need to do this: https://github.com/GoogleCloudPlatform/ios-docs-samples/blob/master/speech/Swift/Speech-gRPC-Streaming/BUILDFIXES
Change non modular headers.

Just don't use normal pod install instead use ./INSTALL-COCOAPODS


Fixed doctype.

@ziahamza
Just <!doctype> is not an acceptable doctype. Had to change this because grunt html minifier refuses to minify index.html of this project.
Also three tabs on line 262 got stripped, hope that's not a problem.

Yes, I like up and this more compliant... Lgtm :)


Clipper2 replacing clipper1

THIS PROJECT IS IN DEVELOPMENT MODE. Any pull requests should merge to branch dev, otherwise will be rejected immediately.
Describe your changes
replace  Clipper1 vs Clipper2
Issue ticket number and link
#819
Checklist before requesting a review

[ x] I have performed a self-review of my code
[x ] If it is a core feature, I have added thorough tests.  (l;ua/cpp test from clipper1)

Thanks, but please let the folder name lowercase like clipper2

Thanks, but please let the folder name lowercase like clipper2

no problem


[APIM]Fix issue with creating consumption tier instance

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

APIM

Hi Guys, any update for this PR? should be very quick to review~

@haroldrandom will help with this PR after holiday.


How to use in browser with vanilla JS? CJS - import issue, ESM - 403 forbidden

Dear David,
could you please help on how to use the module in the plain-text javascript page?
My page is:

The error I get in browser console is:

If I use ESM build, then the result in browser console is:

Please help with the module usage.
Thanks in advance and have a good day.

This package should be use in conjunction with a bundler, like esbuild or web pack to create browser compatible assets.

I've created a bundle with esbuild, like:
.\node_modules\.bin\esbuild lib\esm\index.js --bundle --sourcemap --target=chrome58 > stdnum.esm.js
Than included the resulting file in the index.html like:
<script src="stdnum.esm.js"></script>
No errors in console, so file is loaded fine.
But how to use the stdnum APIs? The "stdnum" variable is not defined...