Simple re-encapsulate to http mpegts?

Hello,
I have a rtsp stream that I'm trying to encapsulate into a http stream using go2rtc.
My "video output" end point only outputs rtsp.
My "capture input" only works with http/mpeg_ts
I can do this manually in VLC by opening the rtsp stream and using the "Stream output" option, putting in a IP and picking the MPEG TS encapulation method. It works great with my capture input. If i open the URL in my browser I don't get a video player, just a never ending file.
Can I achieve something like this with go2rtc? I don't think I actually need the m3u or any re-encoding, just take the rtsp and publish it through http.

Oh I see. Thanks for clearing that up. I was working with GPT and it was suggesting something very simiar, but I couldn't find it in the documentation.
Unfortunately that's still not working, to feed into ADPTuner and eventually over to Channels DVR 😞.
However I was able to open go2rtc proxied stream in VLC, so it must have something to do with what ADPTuner and Channels DVR expect and maybe VLC is doing more than I realize when picking such minimal options.
I can check with the developers of ADPTuner and Channels DVR.
I think the way I have this chain setup is going to end up being a stop-gap anyway -- but here's the "Probe" info if there are any clues in it.
I really appreciate the response and will close this issue, the re-encapsulating of rtsp to http does seem to work 🤷♂. Thanks again!
{
  "producers": [
    {
      "id": 101,
      "format_name": "rtsp",
      "protocol": "rtsp+tcp",
      "remote_addr": "10.x.x.x:554",
      "url": "rtsp://10.x.x.x/live/main/av_stream",
      "sdp": "v=0\no=-<PHONE_NUMBER>333766989<PHONE_NUMBER>333831414 IN IP4 <IP_ADDRESS>\ns=rtsp://10.x.x.x/live/main/av_stream\nc=IN IP4 <IP_ADDRESS>\nt=0 0\na=rtcp-unicast: reflection\nm=video 0 RTP/AVP 96\r\nc=IN IP4 <IP_ADDRESS>\r\nb=AS:8\r\na=recvonly\r\na=control:track1\r\na=framerate:30\r\na=rtpmap:96 H264/90000\r\na=fmtp:96 profile-level-id=42002A; packetization-mode=1; sprop-parameter-set=Z0IAKpY1QPAET8s3AQEBAg==,Z0IAKpY1QPA=\r\nm=audio 0 RTP/AVP 97\r\nb=AS:128\r\nc=IN IP4 <IP_ADDRESS>\r\na=recvonly\r\na=control:track2\r\na=rtpmap:97 MPEG4-GENERIC/48000/2\r\na=fmtp:97 streamtype=5;profile-level-id=15;mode=AAC-hbr;sizelength=13;indexlength=3;indexdeltalength=3;config=1190\r\n",
      "user_agent": "go2rtc/1.9.6",
      "medias": [
        "video, recvonly, H264",
        "audio, recvonly, MPEG4-GENERIC/48000/2"
      ],
      "receivers": [
        {
          "id": 102,
          "codec": {
            "codec_name": "h264",
            "codec_type": "video"
          },
          "childs": [103, 110],
          "bytes": 62960316,
          "packets": 44572
        },
        {
          "id": 104,
          "codec": {
            "channels": 2,
            "codec_name": "aac",
            "codec_type": "audio",
            "sample_rate": 48000
          },
          "childs": [105, 111],
          "bytes": 865529,
          "packets": 2558
        }
      ],
      "bytes_recv": 64391405
    }
  ],
  "consumers": [
    {
      "id": 100,
      "format_name": "mpegts",
      "protocol": "http",
      "remote_addr": "10.0.y.y:55555",
      "user_agent": "VLC/<IP_ADDRESS> LibVLC/<IP_ADDRESS>",
      "medias": [
        "video, sendonly, H264, H265",
        "audio, sendonly, MPEG4-GENERIC"
      ],
      "senders": [
        {
          "id": 103,
          "codec": {
            "codec_name": "h264",
            "codec_type": "video"
          },
          "parent": 102,
          "bytes": 11662474,
          "packets": 8252,
          "drops": 36320
        },
        {
          "id": 105,
          "codec": {
            "channels": 2,
            "codec_name": "aac",
            "codec_type": "audio",
            "sample_rate": 48000
          },
          "parent": 104,
          "bytes": 123012,
          "packets": 364,
          "drops": 2194
        }
      ],
      "bytes_send": 6075220
    },
    {
      "id": 109,
      "format_name": "probe",
      "protocol": "http",
      "remote_addr": "10.z.z.z:66666",
      "user_agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
      "medias": [
        "video, sendonly, ALL",
        "audio, sendonly, ALL",
        "audio, recvonly, ANY"
      ],
      "senders": [
        {
          "id": 110,
          "codec": {
            "codec_name": "h264",
            "codec_type": "video"
          },
          "parent": 102
        },
        {
          "id": 111,
          "codec": {
            "channels": 2,
            "codec_name": "aac",
            "codec_type": "audio",
            "sample_rate": 48000
          },
          "parent": 104
        }
      ]
    }
  ]
}


adding test suite for components and hooks

Description
Hi,
This is one of the extremely helpful projects I've ever seen. In this issue, I want to discuss "Do you've any plan to add test suite?". Sometimes I feel like we're using snippets on production so we need to have a test suite which will give us solid confidence.
Anyway, Thanks for maintaining this project.

Thanks for the feedback. We are not currently planning to add a test suite as we do not have the resources to maintain it, but it might be something we look into the future. I am marking as an enhancement and adding it to the workboard.

Thanks @Chalarangelo for recording feedback.

Due to limited resources and seeing how 30code's test suite has fallen into disuse lately, I will close this as it will be postponed indefinitely and reopen if we find the time to put in the effort to implement this later down the line.


Handling parallel request errors with Faraday adapter?

Hello and thank you for this amazing library!
I've been working on integrating it with our existing Faraday adapter and have run into some trouble trying to intelligently handle errors. In particular, there seems to be many scenarios where httpx will raise an error, including whenever a 4xx or 5xx response is received. When this happens it's not possible (as far as I can tell) to understand which request caused the error, and what the results of the other requests were.
conn.in_parallel do
  response1 = conn.get('/one')
  response2 = conn.get('/two')
end # Error is raised here, with no connection to the request/response that generated it

response1.body # we never get here
response2.body

In our use case, depending on which requests result in errors, we would have different behavior upstream.
Would it make sense to offer an option where concurrent requests wouldn't raise errors but, instead, always return a Response Object with the ability to inspect the response to determine the result? Typhoeus' API seems to support this for example, where you can e.g. call response.timed_out? to determine if there was a timeout, rather than having an error thrown.
In any case, hopefully this made at least a slight bit of sense. Thanks for reading!
Faraday Parallel docs for reference

Hi @f1sherman thank you for the report.
I believe this has been reported here and being fixed here (was it you?) .

@HoneyryderChuck that wasn't me, but that's awesome, thank you!
Are there any other cases where httpx might raise an exception, like for a timeout?

httpx does not raise exceptions by itself; instead it provides .raise_for_status API for you to control whether to raise (see [the docs](for more info)).
In the faraday adapter, that's a different matter. The adapter for httpx is following the protocol for single requests, but apparently not for parallel; most because there is no protocol, just a single "reference implementation" (the typhoeus adapter), and the error handling seems to be tightly coupled to it.

Ok sounds good, thanks for all the info! I'll go ahead and close this now.

released v0.22.3

Excellent, thank you!


KeyNotFoundException on old iPhones and iPads

Version is 1.4.8
ThrowHelper.ThrowKeyNotFoundException (System.Object key)
System.Collections.Generic.KeyNotFoundException: The given key '23b7bacd-396e-4011-a1eb-69236f789c08' was not present in the dictionary.
ThrowHelper.ThrowKeyNotFoundException (System.Object key)
Dictionary`2[TKey,TValue].get_Item (TKey key)
CardsView.RemoveRangeViewsInUse (System.Guid gestureId)
CardsView+<OnTouchEnded>d__233.MoveNext ()
AsyncMethodBuilderCore+<>c.<ThrowAsync>b__6_0 (System.Object state)
UIKitSynchronizationContext+<>c__DisplayClass1_0.<Post>b__0 ()
NSAsyncActionDispatcher.Apply ()
(wrapper managed-to-native) UIKit.UIApplication.UIApplicationMain(int,string[],intptr,intptr)
UIApplication.Main (System.String[] args, System.String principalClassName, System.String delegateClassName)
Application.Main (System.String[] args)

Got it, I will have to redesign gestureId handling and make it through list of gestureIds

hmm, it means, that new interaction stats before ending of previous..
yes, i think it would be much more better to have a list of all gestures =0

In my app, I have a slideshow
Maybe user interacts and slideshow...

yes, exactly.. because now gesture id is created in two places.
Ok, cool, i will figure out with it

1.5.1 hope will be working


Update rgeo dependency to 1.0

Fix #26

This PR is included in version 1.0.0: https://rubygems.org/gems/rgeo-shapefile/versions/1.0.0


[Keyboard Manager] Confirmation Dialog for orphaned keys.

Summary of the Pull Request

A confirmation dialog appears for single key remappings if there are orphaned keys.
Apply button is changed to OK, and closes the window after applying the changes.

PR Checklist

[x] Applies to #2461, #2779
[x] CLA signed. If not, go over here and sign the CLA

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

When remapping single keys, a confirmation dialog shows up if there is at least one orphaned key.
Validation happens before the confirmation dialog, so if there are invalid mappings the flyout still appears.

good work, few nit. can the OK button be white, and the same size as the cancel? and can you capitalize anyway to "Continue Anyway", otherwise lgtm

@saahmedm we want the OK button instead of Apply change for the Edit Shortcuts window as well right?

@saahmedm we want the OK button instead of Apply change for the Edit Shortcuts window as well right?

Yes

@arjunbalgovind @saahmedm I added the confirmation dialog for partial remappings (#2815) since those changes depended of the previous changes on this PR. I also think that there will be some conflicts with #2816, so maybe that has to be merged in first.


doc: Fix Install Guide URL

[skip ci]

@etcd-bot ok to test

@directionless thank you for the PR can you please update your commit message to doc: Fix Install Guide URL and I can get this merged, thanks!

You want me to amend that commit, and not adjust it on merge? Okay...

You want me to amend that commit, and not adjust it on merge? Okay...

I know it seems silly but it is outlined in our style guide and having these commits ready to go will help things get commited faster and eliminate human error. I appreciate the PR.
https://github.com/coreos/etcd-operator/blob/master/CONTRIBUTING.md#format-of-the-commit-message


Initialize tarball download dir into proper git repo

Quite a quality update :-)
Note: This has been extensively tested stand-alone but not yet with a fresh SD-Card install


Add terminimal

Adds Terminimal - a lightweight toolkit for displaying output from CLI apps.

Thanks @hugoabonizio - I hadn't noticed that 30 day requirement. I'll get some specs added while that ticks over.


Fix/56-database-read

Made Android version of Firebase.Database.read compatible with nulls, lists and primitive types
Made iOS version of Firebase.Database.read compatible with booleans and numbers, and cleaned up the iOS code for Read in Database.uno

Fixes #56

Wow, sorry for the delay there. Thankyou


Roadmap: Review datatypes of datasources in submitted files & in pipeline

All evidence strings for a datasource are generated as a specific datatype so that the JSON validation can be datatype-specific. The datatypes for different datasources in the evidence strings and in the evidence-data ES index have been assessed here. There are currently two issues that should be investigated:

Some evidence strings are generated as genetic_literature and are then converted to genetic_association in the pipeline. Affected datasources are gene2phenotype, Genomics England and a subset of the UniProt evidence (literature). Tickets have been opened:

#584 Generate Genomics England evidence as genetic_assocation type
#585 Generate Gene2Phenotype evidence as genetic_assocation type
#586 Remove genetic_literature evidence type from schema
#587 Remove genetic_literature -> genetic_association change in pipeline
#630 Change uniprot_literature to genetic_association datatype

Low priority: sysbio and crispr evidence are currently generated as affected_pathways because they are in the Pathways & Systems Biology datatype. It might be good to have a sysbio datatype.

For 1. above, genetic_literature refers to evidence strings that do not provide variant information. Therefore they need to be validated against a different JSON schema than genetic_association. All 5 tickets above related to the removal of genetic_literature from the JSON schema will be closed with this explanation.

Closing this ticket since point 2. will be looked at when Pathways & Systems Biology date_type will be reviewed.


Document for each project the expected number of tool failures.

Tools fail on some of the files. So far, we only were concerned about
crashes in tool in the smoke tests, but recently improvements have
been sen in also reducing the number of files we can't process.
This change documents how many of these failures are to be expected
to make sure we don't add regressions (if failures increase, the
smoke test will fail), but also detect when we see less than expected
failures.
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Codecov Report

Merging #1299 (9767c21) into master (677475e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1299   +/-   ##
=======================================
  Coverage   92.87%   92.87%
=======================================
  Files         339      339
  Lines       23641    23641
=======================================
  Hits        21957    21957
  Misses       1684     1684

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 677475e...9767c21. Read the comment docs.


Check functionality without dummy robots

There was seen missbehaviour without dummy robots. Doublecheck the correct display at the frontend!

It might be, that the map scale is not shown correctly at the frontend


Move collective's location to a separate table

Account's location is currently defined by 4 columns in the Collectives table:

locationName
countryISO
address
geoLocationLatLong

I think all these should be migrated to a separate table for the following reasons:

Location is a generic concept that is meant to be re-used for different purposes. For example, in the Expenses table, we have a payeeLocation field that could similarly benefit from sharing the same validations/columns without having to redefine everything.
It's pretty common for online services to have multiple addresses linked to your profile (e.g. to have different shipment addresses)
The Collectives table is huge, which leads to slow migrations and more complexity to apprehend the model
For the previous reasons, refactoring locations today may be a complex task. We talked about address validation in the past, if we want to do that we may want to add more columns to properly store the address (ie. city, state, etc). Having locations as a separate table would make this easier to achieve and would make migrations faster.

Specs

[ ] Create a new Locations table/model

[ ] name (string, optional)
[ ] address (string, optional)
[ ] country (string[2], required)
[ ] latLong (geometry => https://github.com/opencollective/opencollective/issues/3171, use a string, optional)
[ ] structured (JSON)
[ ] CollectiveId
[ ] isMainLegalAddress

[ ] On the Collectives table (and corresponding History table), remove the columns listed above and replace them by a LocationId column (except for countryISO, as it's interesting to keep track of the collective country somewhere)
[ ] On expenses, replace payeeLocation by a LocationId column
[ ] Have a migration to move all the data to this new model

We need to review these specs in accordance with the introduction of the structured (JSONB) field along with the dynamic form. Currently, structured addresses can only be stored in expenses (payeeLocation). It's a problem as we're now introducing the dynamic form for guest orders > 5K and have to store it in collective.data.

Related: https://github.com/opencollective/opencollective/issues/3171

I'm wondering if perhaps we should ditch the structured JSON field, and add those fields into the table, making the whole thing structured. Easier/faster to query perhaps, and also not having to save the things in two places:

Locations table/model

[ ] name (string, optional)
[ ] address (string, optional)
[ ] addressLineTwo (string, optional)
[ ] postalCode (string, optional)
[ ] city (string, optional)
[ ] state (string, optional)
[ ] country (string[2], required)
[ ] latLong (string, optional)
[ ] CollectiveId
[ ] isMainLegalAddress

And to support what we have discussed with having a public display location in addition to your legal address, on the Collective table:

[ ] displayLocationId
[ ] legalLocationId

Some thoughts on a possible migration strategy and different location requirements

Events:

take locationName, address, countryISO, and use Google Places Autocomplete API to get structured location to put into new table
store displayLocationId on the Event

All other Collectives:

take address, countryISO and use service to get structured address
store legalLocationId on the Collective

Payout methods

structured location is currently stored as part of data JSON field
store locationId on payout method
location should ideally be editable?

Expenses

structured location currently stored as payeeLocation
I'm assuming this address comes from the payout method
Should this location be editable after an expense has been completed?

I'm guessing that could mess up some legal accounting obligations, so perhaps this field should not be linked to another editable location, but in fact stored as is on the expense, or maybe a new location in the Locations table that won't be edited?

Is name a free field? Usable for something like "Europe", "East Germany" or "South Berlin"?
For each location, I don't see a third party identifier. That could help us better track duplicates and identify precisely the entities.
Maybe we don't have to care about Payout methods and Expenses for now, it could make the migration more complicated than it should. Focus on Collectives (the table, so Organizations, Projects, Events and Individuals too).
Can we skip displayLocationId and legalLocationId if we already have the CollectiveId column in the Locations table. It's a better design if we want to have as many locations as we want for a given Collective anyway. We could add some flags in the Locations table if we need to know if it's "display" or "legal".
I think it's important to have clear definitions of what is "display" or "legal". For example, what are the location tags currently set for OCF collectives representing?

On the schema, if location can be attributed to Payout methods and Expenses, I think it should not have a CollectiveId, we could instead add an UserId to it.
isMainLegalAddress is probably information that another table would have like Collectives.MainLegalAddressId
For Global  and Online, maybe we can reserve two special 2-letter codes to use as "country".
I was looking at an example confirguration from open street map for how to load their address data into a relational db, it fits with what you proposed https://github.com/openstreetmap/osm2pgsql/blob/b87b4c49e7e76f95c80fac128cb5ce0063964a64/flex-config/addresses.lua#L6
I think state should be flexible to allow text same as city

@znarf:

Is name a free field? Usable for something like "Europe", "East Germany" or "South Berlin"?

Yes, that would make sense to allow free form when the autocomplete service does not have the specific venue or neighoburhood you are looking for.
But the main case would probably be that name is similar to what we currently have as locationName for Events. I.e. something set by the autocomplete field that could either be a venue name, or the most specific part of the address, like street, city or country etc, to be used as the label for the location.
Europe as a continent is findable through the Nominatim API, and also various neighborhoods, but not South Berlin.
That raises the question, do we want to have continent in the Location schema? We currently have defined region in Discover that include Americas, Europe, Asia, Africa etc.

For each location, I don't see a third party identifier. That could help us better track duplicates and identify precisely the entities.

Interesting. Something like OSM's nodes? I.e. https://www.openstreetmap.org/node/25871341

Maybe we don't have to care about Payout methods and Expenses for now, it could make the migration more complicated than it should. Focus on Collectives (the table, so Organizations, Projects, Events and Individuals too).
Can we skip displayLocationId and legalLocationId if we already have the CollectiveId column in the Locations table. It's a better design if we want to have as many locations as we want for a given Collective anyway. We could add some flags in the Locations table if we need to know if it's "display" or "legal".

Makes sense.

I think it's important to have clear definitions of what is "display" or "legal". For example, what are the location tags currently set for OCF collectives representing?

Yes, good point. I think the display location should solve the purpose of those tags.
The way I see it, some definitions could be:
Display location: Optional public field to communicate your location or area of operation on your profile, and to increase your discoverability in your area.
Legal address: Private field for legal/tax/compliance purposes (optional/required depending on your fiscal host).
@hdiniz:

On the schema, if location can be attributed to Payout methods and Expenses, I think it should not have a CollectiveId, we could instead add an UserId to it.
isMainLegalAddress is probably information that another table would have like Collectives.MainLegalAddressId

Good point, if our intention is to use it there too.
But maybe there is value in keeping location for expenses and payout methods as is, not only to keep things simpler for on, but also to easier ensure that this data is not changed after an expense has been paid. And maybe less value in being able to query expense/payout method locations easily?
The connection between an expense or payout method's location and your legal address could be that your legal address is used to prefill the fields on a new expense or payout method.

For Global and Online, maybe we can reserve two special 2-letter codes to use as "country".

If we allow fully free-form name in the location, without having to specify anything more, maybe that field can be used to specify things like Global or Online.
The question becomes how to handle that input in a nice way, to allow either selecting an autocompleted node from a Geolocation API, or completely free-form just using the name label, or the combination where I want to use a free-form name label in addition to a structured location from the Geolocation API.

I was looking at an example confirguration from open street map for how to load their address data into a relational db, it fits with what you proposed https://github.com/openstreetmap/osm2pgsql/blob/b87b4c49e7e76f95c80fac128cb5ce0063964a64/flex-config/addresses.lua#L6

Interesting! I see that they have something called place to define the type of location that the name represents. Perhaps interesting as well. Generally it would be good to use as much of their structure as possible if we intend to use their Autocomplete API.

I think state should be flexible to allow text same as city

Makes sense, at least if we intend to use the field beyond the US, currently I think that field is hidden in the Expense form when selecting a country other than the US.
But for US states we could perhaps add some validation in the API to make sure it is a valid 2-char state code when country is US, to make querying for those easier. US states are at least used in the Discover dashboard.

If we allow fully free-form name in the location, without having to specify anything more, maybe that field can be used to specify things like Global or Online.

Note that for "Online" locations, you usually want to provide a link. How we do it with events today:

For the latLong column I suggest using a Postgres Point rather than a string.

For the latLong column I suggest using a Postgres Point rather than a string. Also column name is maybe a bit cryptic?

Seems like Sequelize does not support the native postgres Point, only through using their Geometry and Geography data types that require postgis (reference from 2019 but seems to still be the case when testing it)
Right now I've kept the existing JSONB in the draft migration in https://github.com/opencollective/opencollective-api/pull/8626, but another option could perhaps be a separate columns for lat and long if not a string.

Will also need to make sure we keep track of the coordinate system we're using, and keep it consistent.

I think using a Point is also a good practice to allow us to eventually query things based on geo location. We should be able to extend a simple type to support Point without buying into postgis now (though that may be a good idea anyway).

About Postgis: https://github.com/opencollective/opencollective/issues/3171
I would love us to consider other solutions before bringing it back. It makes the setup more complex, and I don't think we're doing enough geographic work to justify its use. Especially now that Postgres has evolved to include some native geometric types.
Sequelize have this related issue: https://github.com/sequelize/sequelize/issues/2839. If someone feels like doing it, we could make a contribution to add support for this feature - even if it's just for Postgres (worth opening an issue in that case). Their team is really great, and they're easy to communicate with.
Otherwise, as @hdiniz mentioned, we should be able to fix this on our side by using a custom type.


Module not correctly generated with custom namespace

Hi!
After changing the namespace and creating a new module via:
php artisan module:make Reservation
The namespaces of the files are correctly set but the folders with the files are created in Modules instead of under the Namespace set.
Thanks!

That's intended, the namespace doesn't set the folder. There is a folder config option for this.

Thanks!
My bad.
And congrats! It is a great package!


Discussion: Automatic padding-top when AppBar fixed

I think it would be useful, that the Panel, the Drawer and the SideBar should have a padding-top equal to the AppBar height by default when it is defined as fixed.
What do you think?

@PolGuixe I agree,  but I'm no sure what is the best option...  Margin or padding

I agree too!

What I'm not sure is if we should try to use Drawer inside Layout. Now using theme property it should be very easy to target from the Panel component. I'm quite busy with other things but we should consider it because some people confuse the NavDrawer and Drawer and it's perfectly understandable.

@javivelasco, you're right... I think @PolGuixe did he mean NavDrawer instead Drawer... However Panel should have padding or margin top to fit correctly with Fixed AppBar

Yes, sorry. I meant NavDrawer. I got confused as well. :)

One thing to keep in mind is that on phones and tablet the NavDrawer and Sidebar are supposed to be over the AppBar.
On desktop, the spec is less definitive.  Generally, the drawer is shown next to the main content area when in "Permanent" mode, although the "clipped under app bar" design is another option. For "temporary" NavDrawers, the drawer opens "above all other content until a section is selected". The material design site itself places temporary drawers above all other content, including the app bar.

@patrick-jones I'd really appreciate if you can go over #666 and check the implementation of the Layout component and subcomponents. I had a couple of issues that made me reimplement it:

It's really important to keep natural scrolling by default in the Panel. If you scroll down and refresh the page should keep the same position.
Reusing the Drawer component because it was being implemented twice.

It's still remaining to make it work in a nested component, although I think the most general use case is to keep it on top of the tree. Maybe you go over it?

I see that's in the css-next branch, right? I'm actually using that already, so I'll update and have a look.  I agree that good scrolling behavior is probably more important than nestability.

Yes, there should be a way of making it nestable too. Its in the css next branch but not released as beta yet

This is done!

can we create drag and drop widget using existing widgets?


W: An illegal reflective access operation has occurred when creating a module.

What you were trying to do
I was trying to create a module like the wiki indicates: ./groovyw module create <moduleName>.
$ ./groovyw module create test
What actually happened
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.apache.ivy.util.url.IvyAuthenticator (file:/home/jvucq/.gradle/wrapper/dists/gradle-7.6-all/9f832ih6bniajn45pbmqhk2cw/gradle-7.6/lib/plugins/ivy-2.3.0.jar) to field java.net.Authenticator.theAuthenticator
WARNING: Please consider reporting this to the maintainers of org.apache.ivy.util.url.IvyAuthenticator
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
How to reproduce
No response
Game version
2.1.0 (develop)
Log details
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.apache.ivy.util.url.IvyAuthenticator (file:/home/jvucq/.gradle/wrapper/dists/gradle-7.6-all/9f832ih6bniajn45pbmqhk2cw/gradle-7.6/lib/plugins/ivy-2.3.0.jar) to field java.net.Authenticator.theAuthenticator
WARNING: Please consider reporting this to the maintainers of org.apache.ivy.util.url.IvyAuthenticator
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
We're doing a create
User wants to create a module named: test
Creating target directory
Creating .gitignore
In copyInTemplateFiles for module test - copying in a build.gradle then next checking for module.json
WARNING: the module test did not have a module.json! One was created, please review and submit to GitHub
Added the remote 'origin' for module 'test' - but the URL https://github.com/DestinationSol/test.git failed a test lookup. Typo? Not created yet?
Created module named test
Operating System
Linux
Additional Info
Distro: NixOS

WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.apache.ivy.util.url.IvyAuthenticator (file:/home/jvucq/.gradle/wrapper/dists/gradle-7.6-all/9f832ih6bniajn45pbmqhk2cw/gradle-7.6/lib/plugins/ivy-2.3.0.jar) to field java.net.Authenticator.theAuthenticator
WARNING: Please consider reporting this to the maintainers of org.apache.ivy.util.url.IvyAuthenticator
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

This is a legitimate warning but it is harmless. It's complaining about one of the libraries provided by Gradle (the groovyw wrapper uses the Gradle runtime underneath). Java 9 changed things to do with reflection and even with Java 11 some libraries are still not compilant with the more strict requirements,
To cover the other warnings here:

WARNING: the module test did not have a module.json! One was created, please review and submit to GitHub

This is an expected warning for a completely new module. It should create a new module.json for you from a template automatically.

Added the remote 'origin' for module 'test' - but the URL https://github.com/DestinationSol/test.git failed a test lookup. Typo? Not created yet?

Most of our game modules live under the DestinationSol organisation on GitHub (https://github.com/DestinationSol/). If you want to use your own user repository then you'll need to create that repository on GitHub first and then change the origin with git remote set-url origin https://github.com/<User>/<Module>.git (run this in the module directory). I can arrange to have modules to be moved into the DestinationSol organisation quite easily, so just ask if you've got a module you'd like to see there.
Since this is not a bug, I'm going to close the issue. The module you tried to create was created successfully, as far as I can tell.

Understood. Thank you!


Code completion with generic type and extends

Sample code:
class Foo {
  String name;
}

class Bar<T extends Foo> {
    getName(T foo) {
      return foo.n<caret>;
    }
}

Problem: name is not suggested by the code completion at <caret> position. T is known to extend Foo, so properties of Foo are expected to be suggested.
(originally filed to WebStorm issue tracker: WEB-13973)

https://dart-review.googlesource.com/c/sdk/+/190723


Update gone wrong?

Yesterday I spent a few hours trying to get boonek's system working again after she updated to the latest version from the master branch. She did that because she had problems with the AO on the very old code basis from ~3 years ago. After the update she couldn't even get a TCP/IP connection to the G4 Host anymore, even though it was running on the same machine. That was the time when they called me.
Potentially there are several issues and I only worked around some of them without being able to estimate all the potential side effects.

I double checked that the path + file was accessible by the user of the machine, but still got the following errors. Also, is it necessary to write to the config file when you just want to run the designer?

Similar errors popped up for the conductor:

There were errors around the commands sent to the G4 Host, so I upgraded the host from v231 to v242. The type of errors changed, but the experiments still didn't work.
At the end I installed the version c24ef4a385d14713062951599bb968a9bbd5f8e1 from 3/28/22 since that one works OK in the behavior room with the G4 Host v242.
I also had to remove one line from the code (not sure what the code does exactly, but now boonek can at least run the example experiment and the ones that worked in the past):

Probably fixed


Does not seem to ignore tokens at 0 hp

Tokens at 0 hp still provide cover and there is an error in the console log:
tokenCover.js:109

       Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'setFlag')
[Detected 1 package: dnd5e-helpers-cover-expansion]
    at modules/dnd5e-helpers-cover-expansion/scripts/modules/tokenCover.js:109:30
    at Function._call (foundry.js:294:14)
    at Function.callAll (foundry.js:253:12)
    at TokenDocument5e._preUpdateTokenActor (foundry.js:18946:13)
    at async TokenDocument5e._preUpdate (foundry.js:18874:7)
    at async ClientDatabaseBackend._preUpdateDocumentArray (foundry.js:10146:7)
    at async ClientDatabaseBackend._updateEmbeddedDocuments (foundry.js:10095:22)
    at async Function.updateDocuments (common/abstract/document.mjs:373:21)
    at async TokenDocument5e.update (common/abstract/document.mjs:456:21)
    at async TokenDocument5e.modifyActorDocument (foundry.js:18756:5)
    at async Function.updateDocuments (common/abstract/document.mjs:373:21)
    at async Actor5e.update (common/abstract/document.mjs:456:21)

I reported the bug a while ago, but yes... it doesn't seem to work well with creatures at 0 hp, it doesn't ignore them.

I think it only happened with linked actors.


fix: change progress-bar to circular

fixes: #12
Changed progess bar to circular and variant prop to represent the color of it

Pull Request Test Coverage Report for Build 98

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.519%

Totals

Change from base Build 86:
0.0%

Covered Lines:
434

Relevant Lines:
521

💛  - Coveralls


concept: no reason to ignore the build here

and this fixes the go vet ./... on golang tip
Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

@cyphar this fixes the go vet too

oh this is the test you moved to fix as well.


meaning of raw_scores and probability prediction

I want to combine several boosting in a ensemble vote and would like to know if LightGBM prob score is calibrated when raw_scores=False

for the binary classification task, the prediction with raw_scores=False is the probability( transform by sigmoid )


Updated SDK to changes in OpenAPI specification

Changes

Updating SDK to changes in OpenAPI. To be merged after: https://github.com/databricks/databricks-sdk-py/pull/354.
Tests

Integration tests running...

[ ] make test run locally
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
Patch coverage is 51.86% of modified lines.

Files Changed
Coverage

databricks/sdk/service/billing.py
ø

databricks/sdk/service/compute.py
ø

databricks/sdk/service/ml.py
ø

databricks/sdk/service/provisioning.py
ø

databricks/sdk/service/oauth2.py
42.30%

databricks/sdk/service/serving.py
46.51%

databricks/sdk/__init__.py
50.00%

databricks/sdk/service/catalog.py
50.00%

databricks/sdk/service/jobs.py
50.00%

databricks/sdk/service/settings.py
55.38%

... and 2 more

:loudspeaker: Thoughts on this report? Let us know!.


Formatting and Testing out ADD function with Date v0.1

Commented out other variables (i.e. endtime/categories)
Testing basic functionalities for ADD functions

Also converted Tags to Categories for Logic packages under this PR


Fixes #2673 NTR-type IIx muscle cell

Added a New term with textual and logical defs, extended description, references and synonyms

It has been addressed in #2731


Bidirectional(LSTM(..., stateful=True)) crashes

nb_samples = 1
nb_timesteps = 1
nb_features = 1
nb_hidden = 1

i = Input(batch_shape=(nb_samples, nb_timesteps, nb_features))
o = Bidirectional(LSTM(nb_hidden, stateful=True))(i)
o = Dense(nb_classes, activation='softmax')(o)
model = Model(i, o)

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-14-f240ae219281> in <module>()
      5
      6 i = Input(batch_shape=(nb_samples, nb_timesteps, nb_features))
----> 7 o = Bidirectional(LSTM(nb_hidden, stateful=True))(i)
      8 o = Dense(nb_classes, activation='softmax')(o)
      9 model = Model(i, o)

/home/carl/anaconda3/lib/python3.5/site-packages/keras/layers/wrappers.py in __init__(self, layer, merge_mode, weights, **kwargs)
    164         config = layer.get_config()
    165         config['go_backwards'] = not config['go_backwards']
--> 166         self.backward_layer = layer.__class__.from_config(config)
    167         self.forward_layer.name = 'forward_' + self.forward_layer.name
    168         self.backward_layer.name = 'backward_' + self.backward_layer.name

/home/carl/anaconda3/lib/python3.5/site-packages/keras/engine/topology.py in from_config(cls, config)
    869                 output of get_config.
    870         '''
--> 871         return cls(**config)
    872
    873     def count_params(self):

/home/carl/anaconda3/lib/python3.5/site-packages/keras/layers/recurrent.py in __init__(self, output_dim, init, inner_init, forget_bias_init, activation, inner_activation, W_regularizer, U_regularizer, b_regularizer, dropout_W, dropout_U, **kwargs)
    675         if self.dropout_W or self.dropout_U:
    676             self.uses_learning_phase = True
--> 677         super(LSTM, self).__init__(**kwargs)
    678
    679     def build(self, input_shape):

/home/carl/anaconda3/lib/python3.5/site-packages/keras/layers/recurrent.py in __init__(self, weights, return_sequences, go_backwards, stateful, unroll, consume_less, input_dim, input_length, **kwargs)
    163         if self.input_dim:
    164             kwargs['input_shape'] = (self.input_length, self.input_dim)
--> 165         super(Recurrent, self).__init__(**kwargs)
    166
    167     def get_output_shape_for(self, input_shape):

/home/carl/anaconda3/lib/python3.5/site-packages/keras/engine/topology.py in __init__(self, **kwargs)
    323             # to insert before the current layer
    324             if 'batch_input_shape' in kwargs:
--> 325                 batch_input_shape = tuple(kwargs['batch_input_shape'])
    326             elif 'input_shape' in kwargs:
    327                 batch_input_shape = (None,) + tuple(kwargs['input_shape'])

TypeError: 'NoneType' object is not iterable

This seems to solve the problem. https://github.com/yukoba/keras/commit/1c8ea3171ba7841b09a6e89a0be9028b1b2a8d70

Nice!

it seems to work

I am getting a similar error for the newly implemented ConvLSTM2D class, when applying it statefully and wrapped with the Bidirectional wrapper. The bug fix before was applied to the get_config() method from the Recurrent class, but ConvLSTM2D inherits ConvRecurrent2D instead, which re-implements the get_config() method.
Could this same fix be applied to the ConvRecurrent2D get_config() method?

Does a stateful model makes sense if we use bidirectional lstm?
If I understand well, in stateful models we carry the states across different chunks of data. But I don't quite get it for bidirectional models, could anyone elaborate?
Thanks!

I don't think a stateful model makes sense in the case of a bidirectional lstm. Stateful models work because we can 'step' forward through the data in a markovian sense, a birectional model would require us to also step backwards, so the same 'chunk' of data could not be appled to the 'backward' work as it would have seen the history and not the future. You could probably run both of the networks statefully and independently and then concatenate the results though.

@Russ09  I agree, so I was wondering, what is Keras doing here, is it just taking the output of the backward network and using as input for the next step?

I haven't revisited this recently, but this issue suggests that Keras doesn't handle it and crashes, as expected, however a warning that bidirectional stateful doesn't work would maybe be more appropriate?

Does anyone understand what is happening for a stateful bidirectional layer? It doesn't crash now, but I'm not sure I understand how the output would make any sense.


Close/reopen an engagement from API

Please submit your pull requests to the 'dev' branch.
When submitting a pull request, please make sure you have completed the following checklist:

[X] Your code is flake8 compliant (DefectDojo's code isn't currently flake8 compliant, but we're trying to correct that.).
[X] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.

@valentijnscholten I think you're more familiar with this area since I haven't used api_v2 as much , at first glance it looks good .

I like the addition, but would like to see the logic moved out of the api file and maybe reuse the logic in engagement/views.py or have shared logic somewhere else. The existing logic also closed the epic in JIRA, so I think that should be performed as well when closing/opening via API.

@valentijnscholten that's a good idea.
There are a lot of possible ways to do it, what does suit better for you guys?
To move out the common logic:

as methods at the Engagement model. e.g. def close(self) / def reopen(self).
in a new module in engagements or in engagements.init.py. Any proposal for naming the module? The methods would be something like: def close(eng) / def reopen(eng).
in new methods in engagements/views.py. Any proposal for naming these methods?
somewhere else. Proposals?

@twsagarcia i would suggest to create this logic on a common module on views. Model is not the correct place to handle logic. You should not overthink where to place the logic, it just a matter of the concept of domain model, probably a helper module under engagements will be okay for this purpose.

Updated PR with @jpescalona and @valentijnscholten proposals.
Please review.

Great, indeed no need to overthink but very good that "logic" like this in one place.

Great job @twsagarcia!

@aaronweaver would you please review & merge if it looks good to you.


Add timeouts on receive adapter

Proposed Changes

Add timeouts on HTTP Server so stuck clients don't block the server.

@davidhadas can we get this in as well?

/lgtm
/approve

/cherry-pick release-1.11

/cherry-pick release-1.12


how to use it with go modules?

Thank you for the project. How to use code-generator with Go projects after v1.11 depending on go.mod? I can't do the usual go get and there are no examples at all on how to use it.

See the sample-controller for usage: https://github.com/kubernetes/sample-controller
They set up a dummy package called tools to do an import that will get it downloaded and vendorable: https://github.com/kubernetes/sample-controller/blob/master/hack/tools.go

@timdorr I'm doing as suggested in sample-controller however, I keep getting error such as
...
go: finding k8s.io/apimachinery v0.17.3
build k8s.io/code-generator: cannot load k8s.io/code-generator: no Go source files
##[error]Process completed with exit code 1.

or
k8s/tools.go:20:8: build constraints exclude all Go files in<EMAIL_ADDRESS>
Any idea how I can get it to play nice with other go modules?

@dvaldivia you're not supposed to build that file or package. Notice the // +build tools constraint which cause that file to be ignored during builds. This idea is simply to have an explicit reference to your tools somewhere in your code, so that Go considers them while resolving dependencies.


Added a link to an MFE demo running on Astro

Hello there 🙋♂
My colleague, Zino Adidi has created an MFE set up using Astro,
He set up a project where he ran Angular, React, Vue, LitElement, and more on a single page. He also detailed a workaround to run Angular on Astro.

Thank you @aleksanderjessitm!

Not a problem. :)


Support being passed a directory

It would be good to be able to pass a directory to the detector, which it will then check for any file it can parse.
I think that for these cases, --parse-as would be ignored.

I realised when doing #73 that we could alternatively support this by having a --ignore-unknown-files flag which'd let you use shell wildcards (+ then you could do recursion e.g. */**).
I suspect that it might still be nice to have native support for passing a directory, as I think shell wildcards are not entirely consistent across environments (especially Windows), but it might be worth going with the flag for now and then see if anyone complains 🤔


Mark as rgb-consumer only transition inputs

Description
This PR fixes the addition of the "rgb input consumer" property only for inputs that are part of RGB transitions.
I got this bug after executing consecutive transfers, and to avoid "dust transaction", I started mixing different terminal derivations (i.e., tapret with bitcoin, opret with change, etc..)

Hi @dr-orlovsky
Could you review this, please ?

I believe this should be fixed (but with a different approach) in https://github.com/RGB-WG/rgb/pull/190


net: context: Allow TCP to use sendmsg()

If sendmsg() is used for TCP sockets, the msghdr->msg_name is not
really used as the socket must already have been connected.
In that case just get the destination address directly from
net_context remote address field.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

If sendmsg() is used for TCP sockets, the msghdr->msg_name is not
really used as the socket must already have been connected.
In that case just get the destination address directly from
net_context remote address field.

Fairly speaking, it's possible (in POSIX) to connect() UDP socket too, which sets the "default" dest address to use for it. I don't know if we implement this feature or not yet. Anyway, looks good.

Fairly speaking, it's possible (in POSIX) to connect() UDP socket too, which sets the "default" dest address to use for it. I don't know if we implement this feature or not yet. Anyway, looks good.

We do support connected UDP sockets, this enables applications to use the normal send() function for UDP packets.

this enables applications to use the normal send() function for UDP packets.

Likewise, it should be possible to use sendmsg() with NULL msghdr->msg_name on such a UDP socket.

Likewise, it should be possible to use sendmsg() with NULL msghdr->msg_name on such a UDP socket.

Yep, good point. I will send a PR that will add the support for that too.


Convert uint32_t array to openCV Mat c++

Hello.
How can i convert your uint32_t *image to openCV Mat in c++?
I had converted but not successed
Here is my code in c++
cv::Mat intMat_RGBA = cv::Mat(cv::Size(oriImgWidth, oriImgHeight), CV_8UC4, image);
std::vector<cv::Mat> RGBA_channels;
cv::split(intMat_RGBA, RGBA_channels);

// remove the alpha channel:
RGBA_channels.pop_back();
std::reverse(RGBA_channels.begin(), RGBA_channels.end());

// and merge back to image:
cv::Mat intMat_BGR;
cv::merge(RGBA_channels, intMat_BGR);

Hi. I'm sorry, but I actually have no idea. I have never worked with openCV in c++ before.

Hi. I'm sorry, but I actually have no idea. I have never worked with openCV in c++ before.

Did you solve it?

OpenCV requires your image array to be in BGRA (instead of the RGBA format used here).  Change the following line in the convertImage function:
image[getRotatedImageByteIndex(y, x, height)] = (hexFF << 24) | (b << 16) | (g << 8) | r;
to
image[getRotatedImageByteIndex(y, x, height)] = (hexFF << 24) | (r << 16) | (g << 8) | b;


feat: add support for diagnostics severity configuration

Signed-off-by: rudeigerc<EMAIL_ADDRESS>Resolves #317
This pull request adds support for diagnostics severity configuration with grammarly.diagnostics.severity. When the parameter is not set, severity falls back to the original configuration.

Are there any plans to include this in the latest release?


Fix string view index in UpdateFileContents()

This fixes #2078.
Since some changes are somewhat independent, they are in three different commits.
This also fixes the misfeature that the string view index had to be disabled in cases the VerilogProject is used where it needs to remove files. So instead of a fractured class use, this is now merely a feature that can be enabled.

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

Comparison is base (a4d61b1) 92.95% compared to head (9f0e6e2) 92.99%.

Files
Patch %
Lines

verilog/analysis/verilog_project.cc
96.77%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2079      +/-   ##
==========================================
+ Coverage   92.95%   92.99%   +0.03%
==========================================
  Files         358      358
  Lines       26446    26466      +20
==========================================
+ Hits        24583    24612      +29
+ Misses       1863     1854       -9

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


Fix developer guide URL

Motivation and Context

Guide link in crate.io, for example, https://crates.io/crates/aws-sdk-s3, is pointing at an old page that has been deleted.
Description

Fix it by using the new link https://docs.aws.amazon.com/sdk-for-rust/latest/dg/welcome.html
Testing

Checklist

[ ] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[ ] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

Though I have updated the generated README.md, it's only a typo fix, should I update CHANGELOG too?
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

it's not critical, but if you update CHANGELOG.NEXT.toml you'll get a credit in the release notes for the next release :-)

@rcoh Thank you, I rebased the main branch and updated the changelog, helping the format is correct.

released in v0.10.1


Support Geo Lookups on the Edge

Currently these are done by Analytics, Target and Audience Manager separately so the data isn't part of the XDM. This would do the lookups before the data goes to AA, AT, and AAM to enable a view that is similar between solutions and make the data available in the XDM.

EXEG-2146


Make the core of Terminal Emulator a library project

I know this project is closed to feature requests, but I am wondering if it would be worthwhile for many to make the core of the terminal emulator a library project.  This would allow other developers to more easily integrate this functionality into their apps.

I'm confused -- are you asking for something other than https://github.com/jackpal/Android-Terminal-Emulator/tree/master/emulatorview ?

No.  In my use, I am using code that is not part of the emulatorview as well.  I am effectively using the entirety of android terminal emulator and then marking that a library project which is in turn dependent on the emulatorview library project.  I thought that would be useful for others.


Would you please add relase on the github?

I am sorry to bother you.Since I am a chinese,I can't easily get the app through the usually way just like yours.
PLEASE ADD the app on the github.
Thany you.

https://github.com/z3r0r4/Kontroller/releases/tag/v0.0.2


Timestamping uuid/hunter results

Want to be able to query data points/uuids that are within a certain time range without getting all matching data points
This would help in a CI to be able to look at the current run compared to any past runs and only fail if the current run has a regression and not pick up any previous regressions

The Date filter should cover this. @paigerube14 let us know if we can close this out!

Thanks @shashank-boyapally for the work! Will close


WIP: Defining tides-data-package.json

Responsive to #93, this attempts to define a tides-data-package profile that largely recreates the profile used by the example Tabular Data Package in the Frictionless Standards, with additional fields mentioned here. I am marking this as a WIP because it's not clear to me yet whether we intend to replicate most of the details I brought over from the original example profile, or if we're going for a significantly more minimal approach. It will likely need tweaks before merging, responsive to a first review from somebody who's in the know.

Do you think we should have the Frictionless Data community help review and provide feedback for the merging between the default Tabular Data Package and the proposed tides-data-package profile?

Question: somewhere it seems like we should be specifying that the schema for each resource should be oneOf each of the TIDES data schemas

Question: somewhere it seems like we should be specifying that the schema for each resource should be oneOf each of the TIDES data schemas

oneOf is primarily suited to uses where the underlying structure differs (e.g. when there's a conditional case where a certain set of fields is required, and a conditional case where another set of fields is required), rather than values of those fields. Since we're not recreating the full table specifications within the data package JSON, I think the approach as of writing is the right balance - keep the schema field here minimal by giving it properties of name and path, and mentioning in the description that those are both meant to reference one of the official TIDES table specs.


POST,PATCH,DELETE relationships

Create pivot migrations on --migrations flag.

Add support for belongsToMany($related, $table = null, $foreignKey = null, $otherKey = null, $relation = null)


Add SSL support to mesos configuration

This PR adds SSL-related configuration settings in /etc/default/mesos when mesos_ssl_enabled is true.

CentOS 7 task Downloading and enable the EPEL repository definitions. fails

Rebase on master, fixed in https://github.com/AnsibleShipyard/ansible-mesos/pull/78

Thank you for contribution :+1:


Unable to Clear getListOfVerifyContexts()

I've noticed that when a request can't be processed, it remains stored in the getListOfVerifyContexts() list, and there's no apparent way to clear it. This results in the list retaining records that should ideally be removed.
For example, in my case, the following contexts remain in the list and are never cleared:
[VerifyContext(id=1724284981409728, origin=https://malicious-app-verify-simulation.vercel.app, validation=VALID, verifyUrl=https://verify.walletconnect.org/, isScam=true), VerifyContext(id=1724286080110007, origin=https://malicious-app-verify-simulation.vercel.app, validation=VALID, verifyUrl=https://verify.walletconnect.org/, isScam=true)]

Questions:

How can I clear this list?
Why are these entries retained?
Is this behavior intentional or a bug? Keeping these entries indefinitely seems unnecessary and could potentially lead to memory issues.

Reproduction Steps:

Send a request of any type of transaction.
Close the app suddenly while the modal is opened.
Reopen the app and check the getListOfVerifyContexts() method.

Hey! The VerifyContext for a given request is removed when the request is responded (approved/rejected). To get the list of pending session requests you can use getPendingListOfSessionRequests and allow users to respond to them when they e.g. reopen their app. Once they do the VerifyContext should be removed.

Hey! I understand that the information is usually removed during a normal flow, but as I mentioned, there are a few scenarios where the flow isn’t followed, causing this info to accumulate.
If I reopen the app after closing app with an opened modal without processing (reject/accept), that context will get stucked forever there.
The function you mentioned won’t work because it requires a topic, and getListOfVerifyContexts only provides IDs, not topics, look.
@Throws /* = Throws */(exceptionClasses = {java. lang. IllegalStateException::class})
public final fun getPendingListOfSessionRequests(
    topic: String
): List<Wallet. Model. SessionRequest>


Cant Run Frontend.

Hello I setup the project locally as per given instructions.
And now running the site.
This is my URL: http://localhost/PROJECTS/laravel_adminpanel/public/frontend
But it shows me Page Not Found error.
Symfony\Component\HttpKernel\Exception\NotFoundHttpException
How can I fix this?

You need to run the server, php artisan serve in the project folder and check at localhost:8000

Delete fronted.
http://localhost/PROJECTS/laravel_adminpanel/public/

That is not the preferred way, run the artisan server or make vhost. The simplest way will be to run the server.

I guess, this issue can be closed. Feel free to reopen.


Interface improvements

I'd like to help improve the Julia interface, perhaps similar to the MATLAB interface. @andreasnoack, could you fill me in on your plans and some good resources on setting the solver parameters?

Hi Robert,
It would be great if you could contribute here. I just threw the current version together to try out the SVD solver on some problems. I'm open for interface suggestions but I think we might be able to do better than MATLAB with our keyword arguments. Please share your ideas.

Nothing concrete yet, also didn't mean to copy the Matlab interface syntactically, just its exposed feature set. Any resources on setting the parameters? I'll try to submit a pull request in the next to weeks.

Any resources on setting the parameters?

Not really but the Primme documentation is okay and they are pretty responsive so I think we can just ask if we'd like some advice.


Phone authentication now do “not a robot” check in seperate browser which make entire procedure slow - Flutter

I solved this issue (App crashes upon phone authentication after changing package name) of app crash by adding implementation 'androidx.browser:browser:1.2.0' into app/build.gradle dependencies.
But NOW whole phone authentication procedure got changed. When I call verifyPhoneNumber it opens a browser window that takes me to a Recapatcha (Not a robot test) before I am able to receive the SMS message and verification code.
But I don't want app to open a browser just to verify it's not a robot it make entire process slow and ugly. Below is the video example.
How to get rid of this issue? It also shows app Firebase app address in the browser link too.

Video example of issue is below
https://drive.google.com/file/d/1G7noQWyyAHvyTo_Te0v6d2O3IvaiClAw/view?usp=sharing

Below is the code snippet of verifyPhone function.
  Future<dynamic> verifyPhone(phoneNo, BuildContext context) async {
    var completer = Completer<dynamic>();
    dynamic newUserResult;

    Future<String> getOTPresult() async {
      await showModalBottomSheet(
        context: context,
        backgroundColor: Colors.transparent,
        builder: (context) => OTPBottomSheet(controller: _otpController);
      return _otpController.text;
    }

    //  >>>>>>>>>>>>> On Complete

    final PhoneVerificationCompleted verificationComplete =
        (AuthCredential authCred) async {
      newUserResult = await signInWithPhoneNumber(authCred);
      completer.complete(newUserResult);
    };

    //  >>>>>>>>>>>>> On Timeout

    final PhoneCodeAutoRetrievalTimeout autoRetrieve = (String verID) {
      print("\n2. Auto retrieval time out");
      completer.complete(newUserResult);
    };

    // >>>>>>>>>>>>>  On manual code verification

    final PhoneCodeSent smsCodeSent =
        (String verID, [int forceCodeResend]) async {
      var OTPDialogResult = await getOTPresult();
       if (OTPDialogResult != null) {
        AuthCredential authCred = PhoneAuthProvider.credential(
            verificationId: verID, smsCode: OTPDialogResult);
         newUserResult = AuthService().signInWithPhoneNumber(authCred);
        if (!completer.isCompleted) {
          completer.complete(newUserResult);
        }
      }
    };

      //  >>>>>>>>>>>>> On Ver failed

      final PhoneVerificationFailed verificationFailed =
        (Exception authException) {
       completer.complete(newUserResult);
    };

    await FirebaseAuth.instance
        .verifyPhoneNumber(
          phoneNumber: phoneNo,
          timeout: Duration(seconds: 50),
          verificationCompleted: verificationComplete,
          verificationFailed: verificationFailed,
          codeSent: smsCodeSent,
          codeAutoRetrievalTimeout: autoRetrieve,
        ).catchError((error) {
      print(error.toString());
    });

    newUserResult = await completer.future;
    return newUserResult;
  }

signInWithPhoneNumber function

  Future signInWithPhoneNumber(AuthCredential authCreds) async {
    try {
      UserCredential result = await FirebaseAuth.instance.signInWithCredential(authCreds);
      User customUser = result.user;
    return _userFormFirebaseUser(customUser).getuid;
  }

  CustData _userFormFirebaseUser(User user) {
    print("----> Inside _userFormFirebaseUser and user ID: " + user.uid);
    return user != null
        ? CustData(
            custId: user.uid,
          )
        : null;
  }

// --- CustData  model class

class CustData {
  String custId;
  String custName;
  String custPhNo;
  String custContactNO;
  DateTime custDateOfBirth;
  Map<String, dynamic> address;
  String cartID;
  CustData({
    this.custId,
    this.custName,
    this.custPhNo,
    this.custDateOfBirth,
    this.address,
    this.cartID,
    this.custContactNO,
  });

  CustData.initial() : custId = '';
  String get getuid => this.custId;
}

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel unknown, 1.22.0-9.0.pre, on Microsoft Windows [Version 10.0.18363.1198], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
[X] Chrome - develop for the web (Cannot find Chrome executable at C:\Program Files (x86)\Google\Chrome\Application)
    ! C:\Program Files (x86)\Google\Chrome\Application is not executable.
[√] Android Studio (version 4.0)
[√] VS Code (version 1.51.1)
[√] Connected device (2 available)

! Doctor found issues in 1 category.

Yes! This SUPER frustrating and makes the whole process useless. The first time to see an authentication provider to do such a thing. I think it's a bug. It's every security concern because someone can use that URL for some manipulations.

Hi @FaizanKamal7
I see there's an open issue addressing the case you described https://github.com/FirebaseExtended/flutterfire/issues/4189.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


[FEAT]: Authentication - Social Login

Description
Integrate social login functionality to provide users with the option to sign in using OAuth feature from their social media accounts. This feature should allow users to log in effortlessly and securely through platforms such as Google, Facebook, GitHub etc.
Acceptance Criteria

OAuth for authentication Implemented
Authenticated user magic link requests created
Backend logic for OAuth authentication implemented

Expected Outcome

Users should be able to log in using their social accounts.

Requirements

Design and implement an endpoint for social authentication
Integrate OAuth for authentication.

User Social Login [GET] /api/auth/social_login?social_auth=AuthType

Social Login Endpoint

Given a request to login with socials, then the social authentication process should be implemented, the system should return the user data, access and refresh token with a 200 success status code.

Authentication Error

Given the user cannot validate social login details, then the system should return a 400 bad request error status with an appropriate error message
Successful Response

{
  "message": String,
  "data":
      "user": {
          "message": String,
          "user": {
            "id": String,
            "email": String,
            "role": String,
            "signup_type": String,
            "is_active": Boolean,
            "is_verified": Boolean,
            "created_at": Date,
            "updated_at": Date,
            "last_login-at": Date
          },
      access_token: String,
      refresh_token: String
}
}

Error Response
{
  "message": String,
  "error": String,
  "statusCode": Int,
}

Split this into each type, e.g Google Auth, Facebook Auth

@markessien  I wanted to ask if Github OAuth is part of the concept of the application. I'm currently without a team, and I'm considering working on the Github OAuth.

@buka4rill I wanted to ask if Github OAuth is part of the concept of the application. I'm currently without a team, and I'm considering working on the Github OAuth.


Cannot read property 'AcceptConfirmation' of undefined

I resolve problem.

Maybe show how you did it. That would be interesting for other peoples with the same problem.


Reset feature flags to their default values

LaboratoryActivity should expose an option to reset local values of feature flags and source values.

#3 should be done first to not be bothered with manual UI updates after reset.

Done in a304130


WIBEth3 decoder and TPC Monitor fcls for DUNE-DAQ 4.4.0

Two new fcl files for ICEBERG, to be run with DUNE-DAQ 4.4.0's data format.  Data taken from May 2024 onwards.  From the iceberg-daq Slack channel:
Shekhar: if any of the above scripts need modifications and/or need new fcl.
even though the hdf5 file using these scripts says we have 30 events, data is there in only 1st 2 events. ????
Tom: That's almost certainly because you are using a DAQ version with a different format from what the offline code is expecting.  Running experiments like to keep data formats constant for this reason.
Shekhar: ICEBERG will keep the DAQ version to fddaq-v4.4.1-a9 (unless forced to change due to DUNE-DAQ request). (edited)

These are the relevant fcl changes. As with the previous WIBEth configurations, ICEBERG is re-using the PD-HD raw decoder, but we include the ICEBERG channel map and number of readout samples.  Enclosed below are event displays of one event in run 11484, TPC 0 and TPC 1, as well as a FFT plot from the TPC monitor fcl.

Assigning Jake as a reviewer as Shekhar doesn't seem to have a github account.


Empty remote configuration value has to return nil for number property

Issue type
Incorrect behavior
Description of the problem
FireBase configuration can return an empty value for a key.
ApplicationConfiguration API can return a nullable value for numberForKey:.
A nil value is expected when remote Firebase value is empty.
Environment information

Application version: Any (included latest iOS 3.1.7 and tvOS 1.0.1)
iOS / tvOS version: Any
Device: Any

Reproducibility
Always reproducible

A fix has been made on feature/fix-empty-remote-configuration.

A fix has been made on feature/fix-empty-remote-configuration.

Merged back into develop, thanks.

Merged back into develop, thanks.


Set Heroku stack to heroku-18 in app.json

As https://github.com/weibeld/heroku-buildpack-graphviz/pull/7 is merged we can upgrade the herokiu stack to heroku-18.
My huginn on heroku build successfully with heroku 18.

Great, thank you! Did you also test the Agent graph generation on the new stack?

Tested, it works.

Great, thanks again!


RESET-LND doesn't create a new wallet

Selecting RESET-LND asked me to setup a new alias, but that was it, it didn't walk me through setting up a new wallet or anything. It seems like all it did was wipe the old lnd and set a new alias.
It looks like the RESET-LND option is supposed to run /home/admin/70initLND.sh but that script doesn't even seem to exist.

Will try to reproduce. Just to confirm:
Are you running v1.7.1 and updated by reflashing the SDcard?
Also please try running:
patch
and see if the issue persists.

Sorry, yes, v1.7.1 on a new SDcard. I actually had already ran patch to fix a different issue.

Yes, instead of
home/admin/70initLND.sh
it should be
/home/admin/config.scripts/lnd.install.sh on mainnet initwallet

Will test the fix, but you could try now.

This did work to create a new wallet, but what I actually wanted to do was restore a wallet from seed. It seems like it should give you that option?

Will need to ping @rootzoll here because that seem to need user flow not present currently in the new setup process.
What you can do now is to delete the existing wallet:
DANGER sudo rm -rf /mnt/hdd/lnd/data DANGER
and use:
lncli create

As always, thanks for the help @openoms!


[WIP] BridgeDb 3.0.0-SNAPSHOT

Hi @mkutmon, I created this branch to test the current BridgeDb 3.0.0-SNAPSHOT jars. Some long-deprecated APIs got removed, so it contains several updates. Note that on of the patches actually fixes a bug in PathVisio (Compat.java line 51, which I'll file as a bug report). I can compile and run BridgeDb this way, paving the way for Java 11 support. What this patch does not do, is take advantage of new features. So, for example, it still provides secondary identifiers.
cc @DeniseSl22

I will rebase it to resolve these issues.

@mkutmon, done. Plz try now.

Crap, this was the wrong PR to merge :( My bad. I should have closed it.
This was the correct branch (but no longer): https://github.com/PathVisio/pathvisio/pull/189


[HOLD for payment 2024-03-26] Tag name field is missing a right caret

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

Version Number: 1.4.52-6
Reproducible in staging?: y
Reproducible in production?: new feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @JmillsExpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1710462014541529
Action Performed:

Go to any "collect" workspace
Click tags >settings
Observe the tag name field

Expected Result:
Tag name field should have a right caret
Actual Result:
Tag name field is missing a right caret
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

I'll work on this one. It's a really simple fix.

We can close this one, there were no contributors involved!


add design document for "named-array"

This PR contains the design document for "named-array" a proposed new package aimed at providing a lightweight, efficient array structure with named dimensions. the main objective of named-array  is to provide a more accessible version of Xarray's Variable class without the heavy dependencies like Pandas.
## proposed name
The "Xarray-lite" name is a placeholder for the time being. we have considered two main candidates for the package's name: "narray" and "yarray." feedback on the proposed names or other suggestions is welcome.
Related issues:

https://github.com/pydata/xarray/issues/5081
https://github.com/pydata/xarray/issues/3981

I'm merging so we can advertise in whats-new.rst.
Thanks @andersy005 !


Update structure of src folder in template

Update the structure of the folder structure in the src folder of this template repo. To see what the updated structure looks like, see issue #29.

@hrshdhgd: great, thank you for the review. Do let me know what you think about the latest changes.


[Bug] [example] IDEA cannot run the examples directly without Lombok plugin

Add a step of Install JetBrains IDEA Lombok Plugin, otherwise the IDEA cannot run the example.

https://github.com/apache/incubator-seatunnel/issues/1840
Purpose of this pull request
Add a step of Install JetBrains IDEA Lombok Plugin in in the document Set Up Develop Environment to remind people to install the Lombok plugin, just as it reminds people to install Scala plugin. Otherwise, IDEA cannot run the examples successfully.

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Good catch, I would like to remove the lombok dependency, we just used it in two class. @CalvinKirs @BenJFan

Good catch, I would like to remove the lombok dependency, we just used it in two class. @CalvinKirs @BenJFan

Agree with remove it

The different idea is that we can use it widely in our projects, it makes our code cleaner, I don't think it's a burden for developers.

Agreed with @CalvinKirs . With lombok, we can save much time to do the trival but necessary work.

The different idea is that we can use it widely in our projects, it makes our code cleaner, I don't think it's a burden for developers.

I'm +1.
Java has many historical reasons that it has to be verbose, but Lombok is a life saver for both code authors and reviewers, authors don't need to write annoying getters/setters/toString/hashcode/equals/etc. methods, reviewers can save more time and mind not to review these boilerplate codes.

OK,  I actually like lombok too. If no other concern, I will merge this PR.

Thanks, looking forward to your next contribution


CLSMetricWithAttribution appears to have the wrong types

Description
Hello,
I am trying to integrate this npm package into my react (ts)/django website to track slow webpages. So far the documentation has been really clear and easy to follow, so thanks for everyone who contributed to that!
For my use case, it is helpful to:

Convert the metrics I want to collect to json schemas,
Use those json schemas in my backend as serialization,
Also use those json schemas to define events for my analytics pipeline.

I have worked out how to use another npm package, ts-json-schema-generator, to generate json schemas from e.g. CLSMetricWithAttribution. So, I decided to start with this as a test case and see what would happen if, upon generating this schema, I tried to validate it with a metric produced by this npm package. I expected, of course, that the metric would be validated just fine, and so I could incorporate this into my backend serialization. However, I came across a hiccup. My metric looked something like this (click to expand):

    CLS metric

        {
          "name": "CLS",
          "value": 1.0004260050639588,
          "rating": "poor",
          "delta": 1.0004260050639588,
          "entries": [
            {
              "name": "",
              "entryType": "layout-shift",
              "startTime": 1824,
              "duration": 0,
              "value": 1,
              "hadRecentInput": false,
              "lastInputTime": 0,
              "sources": [
                {
                  "previousRect": {
                    "x": 8,
                    "y": 8,
                    "width": 1551,
                    "height": 1320,
                    "top": 8,
                    "right": 1559,
                    "bottom": 1328,
                    "left": 8
                  },
                  "currentRect": {
                    "x": 0,
                    "y": 0,
                    "width": 1559,
                    "height": 1328,
                    "top": 0,
                    "right": 1559,
                    "bottom": 1328,
                    "left": 0
                  }
                }
              ]
            }
          ],
          "id": "...",
          "navigationType": "reload",
          "attribution": {
            "largestShiftTarget": "...",
            "largestShiftTime": 1824,
            "largestShiftValue": 1,
            "largestShiftSource": {
              "previousRect": {
                "x": 8,
                "y": 8,
                "width": 1551,
                "height": 1320,
                "top": 8,
                "right": 1559,
                "bottom": 1328,
                "left": 8
              },
              "currentRect": {
                "x": 0,
                "y": 0,
                "width": 1559,
                "height": 1328,
                "top": 0,
                "right": 1559,
                "bottom": 1328,
                "left": 0
              }
            },
            "largestShiftEntry": {
              "name": "",
              "entryType": "layout-shift",
              "startTime": 1824,
              "duration": 0,
              "value": 1,
              "hadRecentInput": false,
              "lastInputTime": 0,
              "sources": [
                {
                  "previousRect": {
                    "x": 8,
                    "y": 8,
                    "width": 1551,
                    "height": 1320,
                    "top": 8,
                    "right": 1559,
                    "bottom": 1328,
                    "left": 8
                  },
                  "currentRect": {
                    "x": 0,
                    "y": 0,
                    "width": 1559,
                    "height": 1328,
                    "top": 0,
                    "right": 1559,
                    "bottom": 1328,
                    "left": 0
                  }
                }
              ]
            },
            "loadState": "dom-content-loaded"
          }
        }

Importantly, note the presence of lastInputTime in the "entries" field. Looking through the type definitions for CLSMetricWithAttribution we can see that it is defined like so:

export interface CLSMetricWithAttribution extends CLSMetric {
    attribution: CLSAttribution;
}

export interface CLSMetric extends Metric {
    name: 'CLS';
    entries: LayoutShift[];
}

interface LayoutShift extends PerformanceEntry {
	value: number;
	sources: LayoutShiftAttribution[];
	hadRecentInput: boolean;
}

Ctrl+clicking on LayoutShift in my editor (I'm using VScode) leads me to this bit of code, that defines LayoutShift: https://github.com/GoogleChrome/web-vitals/blob/9b932519b16f72328c6d8e9814b811f1bc1a0bb5/src/types.ts#L76-L80 This is apparently a modification to the built-in layout-shift to be more accurate (as per the comment), however it does not include the lastInputTime field that I am seeing in my data when using onCLS(console.log) locally in my website browser.
So my question is, why is there this difference between LayoutShift definitions? My best guess is that, looking at the docs for lastInputTime it seems this is an experimental property which was recently added. In this case, does this library need to constantly update the typings whenever the MDN interfaces change as a result? Or am I missing something here? Thanks in advance
Details on how to reproduce
There's an easy way to reproduce the issue I'm having. First, in any existing node workspace, just include some types.ts file or similar that looks like:
export { CLSMetricWithAttribution } from 'web-vitals';

... with web-vitals installed. Then install ts-json-schema-generator with
npm install ts-json-schema-generator

You will need to be in a node workspace with a tsconfig.json for this to work. Ensure "esModuleInterop" is set to "true" in the "compilerOptions". Then, simply:
npx ts-json-schema-generator -f tsconfig.json --path 'types.ts' --type 'CLSMetricWithAttribution' > out.json

this will generate the schema in the out.json (it is quite big! There are ways to reduce this.. but I've been experimenting with the full version to try and get validation to work, see below)
Then, you can copy directly my example from above and in a python shell, do the following:
import jsonschema
import json

with open('out.json', 'r') as f:
    schema = json.load(f)

data = """
# paste the data that I have above here, i.e. the actual CLS metric in the json
"""

jsonschema.validate(json.loads(data), schema)

You should see the following error output:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/.../.venv/lib/python3.11/site-packages/jsonschema/validators.py", line 1332, in validate
    raise error
jsonschema.exceptions.ValidationError: Additional properties are not allowed ('lastInputTime' was unexpected)

Failed validating 'additionalProperties' in schema['properties']['entries']['items']:
    ...

@philipwalton would know more, but I think we only added the attributes we needed by the library rather than looking for this to be comprehensive definition of the type.
lastInputTime isn't used by the library hence it was not needed to add this.
Ideally all these types would be added to the standard lib.dom.ts so we didn't even need to define the fields we used, but that tends to go slower, or wait for them to leave experimental status for that to happen.
Ao all in all I don't think you can depend on this libraries DOM types being complete.

@philipwalton would know more, but I think we only added the attributes we needed by the library rather than looking for this to be comprehensive definition of the type.
lastInputTime isn't used by the library hence it was not needed to add this.

This is exactly right. @rosesyrett if you want to open a PR to add lastInputTime to the LayoutShift type, we'd be happy to accept!

Ideally all these types would be added to the standard lib.dom.ts so we didn't even need to define the fields we used, but that tends to go slower, or wait for them to leave experimental status for that to happen.

Exactly, the tsc policy is to add to the DOM types when implemented in two different browser engines (usually happens automatically when that criteria is met according to https://github.com/mdn/browser-compat-data), so it won't happen for this until Firefox or Safari add CLS.

Thanks for the prompt responses - that makes sense. I guess for now I could just update the "additionalProperties" field to be "true" where I notice there are validation issues like this one, and have some tests that run which will fail if this ever changes in future. Because I'm interested in collecting all the attribution metrics your repo offers, I'm going to write some tests on my end and check for which cases these validation checks fail. I might then open a PR to address these e.g. to add fields in, that would be helpful for us in future!
I recognise this is still potentially a bit fragile; e.g. if the DOM types ever actually changed then the ones here would be inaccurate. But I suppose that's a risk you face even now (e.g. with existing fields). Is there a PR/issue somewhere I could track for something like adding these to the standard lib.dom.ts?


chapter 3.2 line 9. taking the re-projected z value?

"taking the re-projected z value as the keypoints' depth D_{ij}" is actually not what this paper does.
I means "re-project" might not be the right word for this operation.
It actually convets 3D keypoints from world coordinates to camera coordinates. And then taking the z value as the depth D_{ij}.

Hi, thanks for your suggestion.
Your understanding is correct -- By "re-project", we mean converting 3D keypoints from world coordinates to camera coordinates and taking the z values as depths.


Middelman preview and reload very slow

Guys
I know this has been stated a few times, but as my project grows I am running into a constant battle of the reload being extremely poor
I am still using sprockets but recently added the sassc not much happened I think
I do use a few sprockets appends
sprockets.append_path File.join( root, "bower_components" )
sprockets.append_path File.join( root, "source/atom" )
sprockets.append_path File.join( root, "source/molecule" )
sprockets.append_path File.join( root, "source/organism" )
sprockets.append_path File.join( root, "source/template" )
sprockets.append_path File.join( root, "source/page" )
Which could be an issue?
Anyway if I can ask if you can benchmark my project :
https://github.com/iwarner/CodeBlender-Middleman
It could help with your development also?
This is the Gist of my middlemen preview server verbose
https://gist.github.com/iwarner/2783130afe18d2a876f6
Main pause is at line 1170
Just before it starts the server - [2016-03-24 12:14:26] INFO  WEBrick 1.3.1
Gems installed
middleman (4.1.4)
middleman-cli (4.1.4)
middleman-core (4.1.4)
middleman-favicon-maker (4.0.3)
middleman-livereload (3.4.6)
middleman-minify-html (3.4.1)
middleman-syntax (2.1.0)
I always use : "bundle exec middleman server" to initiate
Many thanks

Last time I benchmarked this, it was mostly coming from sprockets. Ping @stevenosloan for visibility.

Hmmm, yeah it could be -- it doesn't look like there's a terribly large number of files that would be handled by SprocketsResource's so I wouldn't think that timing would be too long (2-3 seconds as a guess). I'm out on some terrible internet atm, but will hopefully have a better chance to dig in tomorrow.

Guys
Many thanks for checking this out - Just so you know its
40 secs for server load
== The Middleman is loading

== Preferring use of LibSass
== Sprockets will render css with SassC
== Locales: en (Default en)
== Blog Sources: /docs/:title.html (:prefix + :sources)
== View your site at "http://Ians-Air.home:4567", "http://<IP_ADDRESS>:4567"
== Inspect your site configuration at "http://Ians-Air.home:4567/__middleman", "http://<IP_ADDRESS>:4567/__middleman"
After one file change
DomContetLoaded = 12.91s
Load = 12.91s
My JS is 1.7mb as I am testing a lot of libraries.
Refresh with no file change
Load = 3s

I'll investigate. Blog is really slow right now, will let you know.

Changed a nested all_files.none? to !Set.new(all_files).include? and saved 30 seconds. Hurray?
Still showing 5s spent in Sprockets, via middleman server --instrument

Hiya is that on the server start - the refresh is really what is killing me?
Takes 12 seconds to refresh the page on a change...
Let me try your fix anyway and report back

Yay 15 second server launch.. nice one
Unfortunately refreshing the page is still upwards of 11 seconds, so I think this needs re-opening to see whats going on when a file is changed.

The refresh issue I think may be sprockets related. Looking at being a bit smarter about invalidating it's cache. Are you seeing that 15s on every refresh or just after a reload/file change?

Hiya,
After one file change
Load = 12.91s
Refresh with no file change
Load = 3s

What do you mean "refresh the page"? Like, literally hitting the browser refresh button?
If the sprockets manipulator is ~5s, a file change might end up being that long.
@iwarner Can you run your server with the --instrument flag and report back the results.

Sure - yeah literally cmd+r = 12 seconds to load.
 Let me try again.
It would make sense that the assets are the issue here, but when editing a HAML should it rebuild the assets all the time?

I see updating middleman:sprockets now - thanks steven for adding the timings. I hope this is helping guys, so important to get the speed up on middleman.

== Instrument (sprockets.asset_lookup): 2526.198ms
{:asset=>#<Middleman::Sprockets::Resource path=assets/stylesheets/all.css>}
== Instrument (sprockets.render): 2526.366ms
{:asset=>#<Middleman::Sprockets::Resource path=assets/stylesheets/all.css>}
== Instrument (sprockets.asset_lookup): 7132.2660000000005ms
{:asset=>#<Middleman::Sprockets::Resource path=assets/javascripts/body.js>}
== Instrument (sprockets.render): 7132.451ms
{:asset=>#<Middleman::Sprockets::Resource path=assets/javascripts/body.js>}

Yeah, I don't use Sprockets, so this stuff flies pretty under my radar. It does seem like the haml file change is somehow clearing a cache on the sprockets side.
Sprockets has always been very slow, I wonder if they're doing something horrible like searching all of node_modules/bower_components for something when doing a lookup.

Yeah sprockets getting slow after an html file is touched is my fault. As a quick fix for getting sass partials working I have the cache getting cleared on all file reloads. Working on making that a bit smarter, but so far am only able to improve on file changes that don't cause a server reload.
Speaking of, @tdreyno is there any way to persist something between application reloads? It'd be great to be able to keep the in-memory cache around. I've looked at using the file cache again -- but it was [https://github.com/middleman/middleman-sprockets/issues/53], and appears to still have some tough to tackle issues.
p.p.s
Those specific instrumentation numbers are a little misleading in this case since the asset lookup (which includes a render in Sprockets) is done inside the render method.

@tdreyno I do also have sprockets.append_path File.join( root, "bower_components" ) so I guess I could amplify that problem also.
I will need to check out the new asset pipeline some more, but I pull .scss and .js files from all over the place so just didnt have the time to set it all up.
Thanks @stevenosloan let me know if there is anything else you want me to test @ me here.

@iwarner if you don't mind could you open an issue on the middleman-sprockets project? I think it's fairly clear the issue is coming from there.

@stevenosloan done

@stevenosloan Does a class accessor on your extension module persist between reloads? I never quite know what's happening in which thread, but I would think anywhere global would persist.

Yeah, that totally works -- just need to give it some thought on properly invalidating that / keeping references to linked assets around. Wasn't sure if there were some built in lifecycle hooks to use that I wasn't aware of.

Just in case anyone finds this useful - Middleman shutdown (on projects with sprockets) is quite slow too, sometimes even non-responsive, so I just use killall -9 middleman 🎯


migrate to androidx

without enabling Jetifier this library cannot be used in newer projects

Using the Lib with androidx project produce gradle conflicts


splitcidr

fixes #110

Need to add to:

scripts/fmt.sh
scripts/lint.sh
.gitignore

Also I guess this doesn't yet solve the v6 split?

Yes it's only for v4 for now.

IPv6 doesn't seem to be working. I did ::/127 128 and it panics.

should work now, there was a bug in copying big int to []byte

@rm-Umar great work!


The calendar does not hide when clicking (focus-in) a textarea powered with tinymce

REPRO LINK
https://jsfiddle.net/wr2xqL09/
Your Environment

flatpickr version used: latest (but this information is not very relevant)
Browser name and version: Any (this information is not very relevant here)
OS and version: Windows (but this information is not very relevant)

Description
The flatpickr calendar does not hide when clicking (providing focus) on a textarea powered with tinymce
Additional note
I did not investigate much, but my best guess is that flatpickr and tinymce both compete for a mousepress / mousedown event, which handlers probably conflict.
This is probably more related to a problem within tinymce, than within flatpickr.

Potential hack : mount a "focus" event on tinymce, which closes all the flatpickr
tinymce.init({
  selector: '#textarea_tinymce'

  // here we go.
  // Assuming that all the input powered with flatpickr have a  "flatpickr-input" class
  init_instance_callback: function (editor) {
      editor.on('focus', function (e) {
        var elementsThatMayHaveFlatpickr = document.getElementsByClassName("flatpickr-input");
        console.log("Number of potential flatpickr object : " + elementsThatMayHaveFlatpickr.length);
        for(i = 0 ; i < elementsThatMayHaveFlatpickr.length ; i ++) {
          fp = elementsThatMayHaveFlatpickr[i]._flatpickr;
          if(fp) {
            console.log("flatpickr detected. Closing.");
            console.log(fp);
            fp.close();
          }
        }
      });
    }
});

This is probably more related to a problem within tinymce, than within flatpickr.

I kind of agree here. I wanted to check it out anyway, but it seems I need a key to access TinyMCE now. Anyhow, closing for now as you seemed to have solved it. Please reopen if we can still provide help


Module clash exception needs checkBuildPlan to be aware allowed clashes in yaml file.

@snoyberg
I've made two commits to stackage curator

To add Tests and possibility for module clash exception
https://github.com/drwebb/stackage-curator/commit/00844d59203d78b308a28d2ff51802fbd550d3be

To fix all the name issues I introduced:
https://github.com/drwebb/stackage-curator/commit/795ccfda0c4b9de460c3f37a7ac0c7fd9e4f58db

checkBuildPlan and updateBuildPlan in stackage-curator need to source the values that are present in the build constraints yaml file, used to generate PackageContraints and BuildContraints.
Does it look okay to add as an Map ExportedModuleName (Set PackageName) field the BuildPlan type as a field? You can see as reference the BuildConstraints type:
data BuildConstraints = BuildConstraints
    { bcPackages           :: Set PackageName
    -- ^ This does not include core packages.
    , bcPackageConstraints :: PackageName -> PackageConstraints

    , bcSystemInfo         :: SystemInfo

    , bcGithubUsers        :: Map Text (Set Text)
    -- ^ map an account to set of pingees
    , bcAllowedModuleClashes :: Map Text (Set Text)
    -- ^ map from module names to packages that contain the same exported name
    }

Yes, with the following caveat: I think it should be a Maybe field. The semantics should be: if the value is not present in the YAML file, then checking is disabled. This way, when working on a preexisting snapshot (like lts-2.8), we don't suddenly get failures mid-major release. Does that make sense?

Right makes perfect sense, it's such a small one line change I'll leave making the pull request for this repo when I push the rest of the code to stackage-curator.


fix(ListItemSlider): values sync with SB controls and textbox

Description
The ListItemSlider's component value was not syncing between the component's actual textbox value (when you adjust the value with L/R arrows) and the input storybook control value. By using the storybook client API and signals the storybook value is now responsive (staying in sync) with the value of the slider itself. Similar logic is already used within the Slider component to keep its values synced.
This fix solves the issue of the min and max values not being updated until the arrows were used.
References
LUI-1491
Testing

Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Test Execution Failed.

Test Execution Failed.

@erautenberg would this be considered a separate issue/ticket from the original issue? This is how the Slider is currently operating apart from the ticket.

@ctoddy that's fine with me, please work with Nassar to make a new ticket to track! thank you

Test Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.21.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: