Experimental Navigator API

@ericvicenti has implemented a new experimental API for Navigation that offers a much more redux-like implementation. I think it's worth looking into building on top of his API, it matches the philosophy of what this library has done in comparison to ExNavigator/Navigator/NavigatorIOS
I'm going to be spending today looking into integrating the two.
https://github.com/facebook/react-native/commit/a3085464f6ea36fc6b53cd0c711c048ffb1516f9
https://github.com/ericvicenti/navigation-rfc/blob/master/Docs/Navigation.md
https://github.com/ericvicenti/navigation-rfc/blob/master/Docs/Reducer_Actions.md

Excellent! Let me know how I can help, or if you have any feedback.
Those docs are slightly out of date. I'll update them this afternoon :smile:

haha no way wasn't expecting an answer so quickly from you. I'm just going through your examples first and making sure I fully understand them.
Was going to email you and ask if you thought this was a good idea but I'll take the comment as implicit approval

I'm happy to see that you're experimenting with it. Bu my hope is that we won't need to rely on many libraries, so people can easily do navigation out of the box with react-native. If you're missing anything important from the built-in Navigation library, please consider sending a PR to react-native so everybody can benefit!
If you want to create custom reducers or navigation views, those would be great to publish independently!

@ericvicenti Where can we find the documentation for the Experimental Navigator API? I'm starting to work on a project and am looking into my options for navigation, started to use this repository but then saw this issue and your comments so was just curious. Thanks!

You will probably learn the most by looking through the examples: https://github.com/facebook/react-native/tree/master/Examples/UIExplorer/NavigationExperimental
Right now the docs are quite sparse, but I'm working on them here: https://github.com/ericvicenti/navigation-rfc/tree/master/Docs
If you want to contribute to the docs, that would be very welcome!

Okay thanks, I will take a look and try them out.
I won't promise anything, but I will make an effort to if I grasp the understanding of it!

@ericvicenti You new experimental API looks very good, but let me explain why i still want to use my component and probably many other people does.
#1 I want to have all my routes in one place. Like Storyboard in iOS, but more convenient (because of JSX)! I believe navigation logic should be separated from other code.
#2 I want to have very simple transitions. Like 'performSegueWithIdentifier' in iOS.
Sorry, maybe it is already done in your API but I've missed it....

I have to agree w/ @aksonov.
I've actually been spending some time this last week trying to integrate the best of both APIs. Simple transitions, routes defined in a single place, with a more direct connection to redux similar to react-router-redux.
My goal is to publish it by the end of this weekend and I'd love to get the community's feedback on it. I basically started from @aksonov's library and worked towards an interface similar to the react-router-redux ideology.

@MITDelian it would be great if you could make it as 'plugin' for this component, like ExRouter.

The only real issue with that @aksonov is that I think the stack of the Router class needs to be stored in redux rather than in the state. I'll send you an early version this weekend so you can take a look over it and let me know what you think

So there are some slight tweaks I have to make in Actions.js and Router.js in order for it to be more properly "redux" and have all the data stored in the store rather than on a particular component or class state

We could really make Router (now it is BaseRouter) immutable and its Route, so it could be stored in state...

+1!

Yes! That is what I'm working on @aksonov

I have all of the adjustments made to the rest of the codebase as if BaseRouter was immutable, and now I need to actually move the stack to the store and make it so the logic for pop, jump etc are in the reducers rather than in the class

Hm, why? Just modify pop/jump within BaseRouter so it will return new BaseRouter instance. This way all logic still will be within BaseRouter.

This way also it the component still could be backward compatible with ExRouter (sure, some modification of ExRouter will be necessary)

Ahhh smart idea. Hehe this is what I wanted to discuss on gitter the other day

Storing router's transitions stack as redux state is the only feature I'm missing from this great library
:+1:

@grin Any problems to make BaseRouter/Route immutable and store it within your state?

@aksonov No problem at all. That was just meant as encouragement to @MITDelian and his idea that he mentioned he is working on. From a library user perspective this sort of functionality would be really nice to have out-of-the-box and I believe it would make this lib even more attractive.

The problem I don't want this component to depend from redux or any other flux library - many users don't want it at all. Many react native components has own inner state (for example TextInput) and you don't add all their state to redux (for example, in most cases you need to add completed text, but not 'in-progress' text). So you may consider navigation component state as TextInput state or any other...

I mean I understand that TextInput has its own internal state however the navigation state of an application is far more important and I think exposing that to the user and letting them store it in redux is quite important.
It also allows you to start from scratch, load in the store, and then be popped deep into a navigation hierarchy rather than starting from the main page each time you reload

Why not make BaseRouter immutable/stateless and implement state management in an optional decorator?

@MITDelian  any ETA for your changes?
@danielweinmann It looks very good idea, looks like @MITDelian  works on it. I think it even could be not connected with Experimental Navigator API at all. Anyway, PR is welcome.

@aksonov I got a preliminary version working for my fork that I use internally, but it's not really quite ready to be integrated into this library.
Apologies but this (https://github.com/facebook/react-native/issues/5404#issuecomment-188571340) is my priority right now, but I'll dedicate some time to this once it is resolvde

Ok, looks like now this integration is priority No.1 for this component because
"Facebook is dropping support for Navigator and will focus on NavigationExperimental from now on" according to  https://github.com/ericvicenti/navigation-rfc/blob/master/Docs/NavigationOverview.md

Also seems we can't make it fully backward compatible (because of reducers & state), i'm going to work on this for next 3.0 release

Ok, if you need some help - please mention me in issue :)
Also it would be great if you create new issue for 3.0 release with planned subtasks

@philipshurpik Looks like 0.21 or even 0.22-rc doesn't contain latest experimental Navigator API changes... Maybe it is too early to work on it because of unstable API - for example latest Navigator example  cannot be run with 0.21.0

@philipshurpik , all Okey, i've submitted very early alpha, branch 3.0.0alpha, check Example project there. API & syntax is changed. Nested containers (Login, TabBar within Example) don't work yet (you may fix it if you can), but go to Register/Register2/Home works with good animation, thanks @ericvicenti for great library, it is really amazing!

3.0 version is released, too pity that nobody was able to help, i thought open source community could be more active...


[it] Broken links due to using "githubbranch" param

This is a Bug Report
Problem:
Regarding https://github.com/kubernetes/website/issues/31248, I found the similar broken links on other pages.
English pages have already been fixed. ref: https://github.com/kubernetes/website/pull/29311
it pages also need to be fixed.
Proposed Solution:
Fixes broken links including {{< param "githubbranch" >}} to master.
In the case of link to https://releases.k8s.io/, fixes to {{< param "fullversion" >}}.
ref: https://github.com/kubernetes/website/pull/29311#discussion_r685845250

/assign


Update Contributors.md

If you haven't please add your data to data/contrbutors.js as well then only it will be counted as a valid PR. And if you have already done that please give a link to that PR. @DinoSaulo

Congrats 🎉, your PR got merged. To be eligible for the swags from iNeuron X ClueLess. Follow this steps.

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


Resources: DSA resources added

Pull Request
issue : #1416
Description
Please include a summary of the changes and the related issue.
The Dsa resources like Youtube channel, playlists and coding platforms were added
Closes #
Type of change

[X] Adding a new Resource
[ ] Adding a new Category
[ ] Documentation enhancement or fixes
[ ] Website enhancement or fixes

Checklist:

[X] I read carefully CONTRIBUTING.md
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation

Screenshots (if appropriate):
Please add screenshots to help explain your changes.

Additional
previous PR was closed due to conflicts. I have created a new PR to incorporate the required changes. @jfmartinz, please review the changes and merge them to contribute to the project.
Thank you

@jfmartinz , please check my changes and merge the changes to the project to make contribution.
Thank you


Metrics for read-only replicas

It does not appear that queries to read-only replicas show up in a SQL database's metrics (CPU, I/O, query analysis, etc). For a read-heavy workload this means that we cannot see if we are close to hitting resource constraints. Is there a way to view these metrics in the portal for read-only replicas?

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

ID: 7a2961d1-4448-11b7-2dc2-3b738bd70939
Version Independent ID: 206eb022-25c4-66d7-7285-0d53a84956e3
Content: Azure SQL Database - read queries on replicas
Content Source: articles/sql-database/sql-database-read-scale-out.md
Service: sql-database
GitHub Login: @anosov1960
Microsoft Alias: sashan

@stephenjust
Thanks for your feedback. We are investigating this issue actively and will get back to you soon.

@stephenjust Sorry for the delay. I have created a new read-only replica for my db and when I am running any query in read only replica. I was able to see the metrics for the same in read-only db.
Below is the screenshot for the same.

Please confirm if this is what your concern is. or please elaborate if I am not able to understand your issue.

@angoyal-msft I believe @stephenjust is talking about the "Read scale-out" feature of premium/business tier Azure SQL databases, which is found under "Settings" > "Configure" in the Azure portal when a SQL database is selected. I'm not aware of any way to view metrics for the read slaves when using that feature.

@ColeShepherdAG is correct, I was talking about getting metrics from the read replicas when read scale-out is enabled.
What we observe is that in the portal, statistics are only visible from the write master. If we set ApplicationIntent=ReadOnly, we do not observe any resource consumption from those queries in the Azure portal.

@stephenjust One (limited) workaround in the meantime is periodically SELECTing from sys.dm_db_resource_stats while connected to your database with ApplicationIntent=ReadOnly.

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.

@angoyal-msft I think we should get a response to our latest comments before the thread is closed...
Have you confirmed that there are no metrics for only the database created using the "Read scale-out" feature of premium/business tier Azure SQL databases (not including the primary read/write database)? If so, are there any plans to add these metrics to the Azure portal? Can you at least add information about the sys.dm_db_resource_stats table and ApplicationIntent=ReadOnly to this article for other readers?

@ColeShepherdAG Thanks for your useful suggestion.
@anosov1960 could you please add the required information in the document.

Having round-robin to the read replicas as well as metrics would be a great boost for performance. Please add this.

I want to throw my 2 cents in here as well regarding metrics for read only replicas. I would love to take advantage of this as a cost saving feature but currently cannot do so. We rely on metrics and Azure SQL Analytics for alerts and day over day statistics. Right now I don’t see how we can implement this feature with no visibility into the performance of it.

You can get metrics, they're just not provided to you via the portal and come with all of the typical bells and whistles (like alerting, etc.). I've been looking at setting up some monitoring tools of my own since what Azure provides isn't sufficient anyways (like seeing the relative DTU utilization at the application/program level). If your monitoring tools connect using the ApplicationIntent=ReadOnly parameter, you can execute sp_who2 or other SPs that you have to see information specific to the replica. Then, automate calling that SP, gathering results and displaying. I'm looking at this for some added monitoring: https://sqlwatch.io/

I'm looking at this for some added monitoring: https://sqlwatch.io/
How do you get around the lack of a sql agent?

I'm still investigating, but it's my understanding that you just need to invoke a few stored procs periodically. If that's the case, that can be automated in many number of different ways. From a script on some machine to Azure Automation and many things in between.

@angoyal-msft Can you please re-open this issue since it's not resolved?  There's still no way to get metrics from scale out read-replicas via the Azure Portal.

Absolutely needed. I don't want to rely on workarounds for such an important feature like monitoring.
Metrics should be available also for read scale-out DB replicas.

@ejjpi We have reopened this issue and assigned to our author.
@anosov1960 Please check this issue and provide guidance.

@NavtejSaini-MSFT Any updates on this issue? Even we are blocked on querying the metrics of SQL read replica using the Monitoring API.

@angoyal-msft, Any updates on this?  This is a major miss that we need ASAP.

Any updates on this?

#reassign:julieMSFT

@Nikos-K @stephenjust Thanks for contacting Microsoft!  From my understanding, the issue you're describing seems to be an issue with the product, is that correct? If so, unfortunately, this form of contact is intended explicitly for reporting issues with the documentation, and not for the product.  To get further assistance with your issue, we'd recommend engaging the community, or opening a ticket with Microsoft Support. More information can be found here: https://docs.microsoft.com/en-us/sql/sql-server/sql-server-get-help?view=sql-server-2017.
If you're looking to provide suggestions or ideas for improvement for the product,  you can do so at https://aka.ms/sqlfeedback.
I am now Closing this Issue 24239. The @ notification mechanism continues to work even after an item is Closed, in case further communication is needed.

#please-close

Does everyone agree that this is the correct product issue that relates to what is being discussed here?
https://feedback.azure.com/forums/908035-sql-server/suggestions/32899126-enable-query-store-for-collection-on-a-read-only-r

@JeremyWeir While the issue you linked is related to this one, they do not seem to be the same problem.  As I understood it, this issue here was focused on exposing metrics from the read scale-out replicas that could then be used like any other Azure metric (ie: building charts, creating automated alerts, etc), which is related to but separate from having the Query Store enabled for read-only replicas.  That said, I've gone and upvoted your issue as it is equally as important to me as this one is.
I'm also disappointed that this issue was once again closed by simply saying "this isn't a documentation issue, it's a product issue" but without taking the initiative to forward the issue to the appropriate product team within Microsoft and then inform this thread how we could track that new issue.  I worked for Microsoft for 14 years before leaving 4 years ago to start my own company, and while I am extremely happy to recent shift toward open development processes that give customers a direct channel to debug problems, provide feedback, and track progress on the issues they care about, I am still quite disappointed with the way that issues routinely get dismissed as "not my area of concern".  This particular issue is extremely important for anyone who wants to leverage the read-slave replicas which are a key selling feature of the top-tier, most-expensive option for Azure SQL.  The lack of empathy for the customer shown here is a bit depressing.

@kroymann Good point, I think I came to assume that Query Store was what powered the metrics we see for Azure SQL databases, but whether or not that is the case, you're right, any metrics we can get outside of Query Store would be helpful.

@JeremyWeir @kroymann We are learning how to best manage this newer channel. The original intent   and the one it still serves today is to address documentation related topics, both enhancements and corrections that need to be made. This should also include issues where the documentation and the product do not align. As a specific product evolves over time or as changes are made (to the portal as an example) and where the documentation needs a correction, this channel has been very useful in making us aware these issues exist. As for feature requests, this channel is not intended to support these types of requests but given the ease of making suggestions to a specific component or feature through this channel, as the documentation is topic based and a conversation can take place with regard to a specific feature, it is helpful to bring together that UserVoice entry with a specific documentation topic. By detailing the UserVoice link to a feature request, it is the desire of the product group that your feedback is detailed in that forum. I included some relevant UserVoice entries here incase the desired functionality is of any interest.
Round-robin read replicas and provide performance metrics of replicas in portal
Provide insight into the replication lag between Azure SQL Database primary and Read Scale-Out replica
We have implemented some processes to escalate issues that fall in that gray zone that are pure product issue, where the document is correct but the issue is more product related. In these cases, the doc issue will still be closed but the issue will be tracked internally.
We greatly appreciate the feedback we receive and your assistance with raising our awareness to tutorials and samples that simply do not work as designed.

This one is also related and relevant
https://feedback.azure.com/forums/217321-sql-database/suggestions/34337935-monitor-queries-on-secondary-database-when-read-sc

The above feedback link is gone. I still don't see metrics when using read scale-out. I guess the feature is still not there.

@specialforest microsoft likes to break links all the time nowadays but i think this link now lives here https://feedback.azure.com/d365community/idea/58a26c28-3225-ec11-b6e6-000d3a4f0f84


[BUG] Spreadsheet parser enforces Data columns to have data cell

CompositeColumn validates FreeText or Data CompositeCells for Data header, i cannot find any implications in the specification if Data columns MAY be freetext or MUST be data.
https://github.com/nfdi4plants/ARCtrl/blob/403081ee9648a87ffd0198baaebef025df51b006/src/Spreadsheet/AnnotationTable/CompositeColumn.fs#L73
For now i will change all logic to be output column MUST have data cells, This would require an update in CompositeColumn.validate function

So spreadsheet writer fails, if column contains Freetext Cell?
By specification, neither the Data Format nor the Data Selector Format columns are obligator (https://github.com/nfdi4plants/ARC-specification/blob/main/ISA-XLSX.md#inputs-and-outputs). Therefore FreeText (=Single string) should be fine.

Per specification this might be true i am talking specifically about CompositeCell.Freetext added to Input/Output [Data] column.
In addition it seems that the autofill cells on table.AddColumn uses FreeText cells to autofill Data output columns.

https://github.com/nfdi4plants/ARCtrl/blob/403081ee9648a87ffd0198baaebef025df51b006/src/Core/Table/ArcTableAux.fs#L210
This is a really annoying issue for datamap implementation in Swate. Not sure how to handle this


Train data

What was used as the training dataset to train the model?

WIDERFACE as mentioned here: https://github.com/StanislasBertrand/RetinaFace-tf2


nextProps.data.forEach is not a function

Im getting this issue nextProps.data.forEach is not a function only in PieChart and DoughnutChart others working just fine
var PieChart = require("react-chartjs").Pie;
var DoughnutChart = require("react-chartjs").Doughnut;

var chartData = {
    labels: ["Red", "Blue", "Yellow"],
    datasets: [{ data: [300, 50, 100] }]
};

<PieChart data={chartData} />

I can confirm this for Polar Area Chart also.

@FardinKarimi I solved the issue using react-chartjs-2 which is actively maintain BUT if you want to use this version you have to use chartData as an array with PieChart DoughnutChart etc so something like this should work
var chartData = [
    labels: ["Red", "Blue", "Yellow"],
    datasets: [{ data: [300, 50, 100] }]
]

@touqeerkhan11 thanks so far. Using chartData didn't help, so I switched to the maintained react-chartjs-2 and it works well.


Indices/heat index

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #807

[x] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

[x] bumpversion patch has been called on this branch
[x] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Calculation of heat index

Does this PR introduce a breaking change?
No
Other information:
For now I've just added the method as a threshold calculation, but we still need to change a couple things:

[x] The issue discussion points we shouldn't add the index calculation in _threshold.py but in _conversion.py. We can look into humindex which is doing something similar, so the code I have added so far might need some tweaking.
[x] There are no tests yet, so we can look into those for humindex and do something similar as well.
[x] Add @JamieJQuinn to list of AUTHORS.rst

Because I was confused myself, I suggest adding a "Notes" section saying that this is the American index that tries to measure the same thing as the Humidex, which is Canadian. And a similar note in Humidex.

I think I've tackled all comments now. It should be ready for final review :)

Looks like there's an issue with the regex for the release notes. I'll push a fix today!

@raquel-ucl the fix is in master branch now. You just need to rebase/merge master in for the fix !

Added tests for the indicator with the atmosds data. Not sure if that's what you were looking for! It took a while to find data that was not too cold in order to compute the index (needs values > 20C and it's cold over there!)


AsciiDoc preview doesn't hold the right order of Headings

Hey,
Preview of my .adoc file doesn't hold the right order of the headings. There is only one level 2 headings and every other level 3 heading is appended to that one.
This is my  doc file:

This is how it looks in Structurizrz lite:

Is it a bug or am I doing something wrong?

Each section (designated by a level 2 heading) needs to be in a separate file, so you need to split the file at == Components. See https://github.com/structurizr/dsl/blob/master/docs/language-reference.md#documentation for more.

I've structured each of the level 2 headings in independent files starting with corresponding == Component name like this:

and it works like a charm:

Thanks for all your help @simonbrowndotje !


Found too many ServerJSDefine

Hi,
today I tried to start my script, but it doesn't use session.json. Instead It tries to log in(fbchat.Session.login), but with Exception: Found too many ServerJSDefine.
I didnt edit anything.. Suppose Facebook updated their API?
Thanks!

I managed to get it running by removing lines in _session.py:
    if len(define_splits) > 2:         raise _exception.ParseError("Found too many ServerJSDefine", data=define_splits)
I know, that maybe this is not right way, but it is working .. temporary fix :)

thx domcom3 : this temporary fix is ok for me ;)


ci: Adding github action to generate changelog based on dispatch

Closes https://github.com/chakra-ui/chakra-ui-docs/issues/481
Sibling PR on the main repo:

https://github.com/chakra-ui/chakra-ui/pull/5831

📝 Description
I've added a Github action that generates the changelog and opens a PR whenever a dispatch event is sent from the main repo.
⛳ Current behavior (updates)
It seems there is a daily cron job to check for this, this will make the process a lot easier.
🚀 New behavior
The work will be automated and once the dispatch has been sent the changelog will be generated.
💣 Is this a breaking change (Yes/No):
No.
📝 Additional Information
@segunadebayo there is a workflow called changelog, I don't think we will need it now, should I remove it?

This is excellent! Love how you executed it @aacevski.
Feel free to delete the changelog action. I think it was a carry-over from the main repo.


Expand key for template cache in IPFix and v9

Currently V9 and IPFIX templates are cached using template id and exporter address as a key.
Rfc3954 implies that v9 templates should be cached using the following fields:
msg header Source ID
exporter source IP
Template id
As stated below:
Source ID:
A 32-bit value that identifies the Exporter Observation Domain.
NetFlow Collectors SHOULD use the combination of the source IP
address and the Source ID field to separate different export
streams originating from the same Exporter.

@akshah,
A pcap that demonstrates the problem is attached. In packet #1 and #2, template id 257 is being sent on domain id 3.
In packet #3, the same template id from the same host is being send, but now on domain 4. The elements are slightly different on this packet. The last 2 elements are of type mibObjectValueOID (element id 436) and mibObjectValueBits (element id 437), while in packet #1 those fields were element octetDeltaCount (element id 1) and packetDeltaCount (element id 2).
Now comes packet #4, which demonstrates the problem. It has template id 257, and domain 3, so it should be parsed using the template from packet #1, and not the template seen later, in packet #3.
In Wireshark this is done correctly. You can see when looking at packet #4 that the it knows the the template packet is #1, and parses the data in packet #4 based on the template in packet #1.
In vflow without the fix, packet #4 is being parsed based on packet #3, because it came later, and vflow ignores the domain id. In vflow with the fix, packet #4 is being parsed on the packet #1, because that's the packet which matches the combination of sender IP + template id + domain id.
ipfix_different_domain.zip


bytes per sec and bits per sec calcs

[This can enhancement]
Was trying to figure how you are doing calculations in your SpeedTestReport class.
So, if you put eg Report interval to 200ms, and you are getting eg bytes per second on which time period is calculated that throughput? Per 200ms and than getting average for last one second or?
Maybe link to lib code will help me more (was trying to find this in SpeedTestTask.class).
Also, in a case that we are interested in current bytes received/transmit on the nw interface rather than calculated speed, would it be possible to expose that info somehow from Socket or whatever? Like, we set report interval to 100ms and receiving number of bytes transferred every 100ms in your onProgress method?
Sorry if I'm being not so clear...
Best,
G

Was trying to figure how you are doing calculations in your SpeedTestReport class.
So, if you put eg Report interval to 200ms, and you are getting eg bytes per second on which time period is calculated that throughput? Per 200ms and than getting average for last one second or?
Maybe link to lib code will help me more (was trying to find this in SpeedTestTask.class).

The throughput is calculated from the beginning of the test.
For HTTP download, the start of download is set after the http header are received from the server : here
For FTP download, the start is set after the FTP connect here
For HTTP upload, the start is set after the header have been successfully written here
For FTP upload, the start is set after FTP connect here
The report interval does not indicate any modification of the way the throughput is calculated. Note that you shouldn't perform blocking operation in the listener because it executes on the same thread as the socket reading/writing thread.
HTTP Download
The total packet size is set after the header have been sent : here
After that, all the reading operation are performed in downloadReadingLoop() which read chunk of data with maximum size of 65535 octet
The loop end when there is nothing to read anymore, each iteration calls the onProgress callback if a custom report Interval value wasn't defined by user.
In any case the onProgress get a SpeedTestReport with mSocketInterface.getLiveDownloadReport(); or mSocketInterface.getLiveUploadReport(); which return the current throughput calculation depending on the total packet size
 and the temporary packet size (total number of bytes received)
If you look at getLiveDownloadReport(), you will see that it calls getReport with the specified mode (download or upload)
The calculation is only performed in this getReport(SpeedTestMode mode)  method. The specified part that may interest you is here where the calculation is done using BigDecimal.

The process is roughly the same for upload and FTP protocol, let me know if you want an expanded explanation for those

Also, in a case that we are interested in current bytes received/transmit on the nw interface rather than calculated speed, would it be possible to expose that info somehow from Socket or whatever? Like, we set report interval to 100ms and receiving number of bytes transferred every 100ms in your onProgress method?

You have the total number of bytes received with getTemporaryPacketSize() and the total expected number of bytes with getTotalPacketSize(). You could get what you want if you made a diff between the temporary packet size received between 2 onProgress(). I don't see the usecase to have that info per onProgress as the speed test value is calculated from the beginning of the speedtest, maybe you have a specific case ?

First thanks for very detail replay. It helped me a lot.
Indeed my use cases are different. I'm more interested into a performance over a time, rather than having final absolute number. My goal is to have time based graphs for throughput in 200ms sampling as default or any other configurable value. This way I'll be able to see performance over time. In any case, from my current knowledge, you are having most complete android lib for HTTP and FTP testing. So, adding more flexibility like this, lib will be more generic and for more use cases. (This one is perfect for quick speedtest report).

Do you mean you want to have the throughput value in the report calculated from the last onProgress report and not from the beginning of the test (and thus also the number of bytes received during this interval) ?

Hi! So throughput is bytes per seconds. And you are calculating it from beginning of the test, having accumulated performance from the beginning. What I would like to have is number of bytes in specific period of time. So, if I'm interested to monitor how much data went in/out buffer every eg. 200ms. And plot that on the time graph. eg. first 200ms was 1MB, next 200ms 2.2MB downloaded/uploaded.
In a case that I use your onProgress with 200ms reporting interval, I'm getting eg in 10th second and 1st 200ms current throughput which was calulated based on other 50 samples (5 samples per second if samppling is 200ms).
Hope it's more clear.

This would be the addition of a new "over time"  mode with a speed test result on each chunk written/received on the Period defined as reportInterval

In next release, setting the following parameter will enable this mode :
speedTestSocket.setComputationMethod(ComputationMethod.MEDIAN_INTERVAL);

The new computation mode : https://github.com/bertrandmartel/speed-test-lib/blob/master/jspeedtest/src/main/java/fr/bmartel/speedtest/SpeedTestTask.java#L919-L931


Polish DefaultPrecompiledScriptPluginsSupport

Omit unnecessary type parameter
Replace !isDirectory with simpler isFile

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Chocolatey fails to install on Windows 2008r2

Powershell plug in is 1.06
Chocolatey plug in is 1.2.4
Chocolatey package is 0.9.10-beta-2016032
The main choco install is failing as below. we are using the beta 9.10 package installed from a private repo.
        my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ï»¿
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Mode                LastWriteTime     Length Name
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ----                -------------     ------ ----
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: d----        07/06/2016     13:13            chocInstall
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Downloading http://myserver/NuGet/api/v2/package/chocolatey/0.9.10-beta-2016032
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: 3 to C:\Users\puppet\AppData\Local\Temp\chocolatey\chocInstall\chocolatey.zip
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Installing chocolatey on this machine
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: The term 'else' is not recognized as the name of a cmdlet, function, script fil
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: e, or operable program. Check the spelling of the name, or if a path was includ
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ed, verify that the path is correct and try again.
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: At line:1 char:5
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: + else <<<<  {
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:     + CategoryInfo          : ObjectNotFound: (else:String) [], ParentContains
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:    ErrorRecordException
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:     + FullyQualifiedErrorId : CommandNotFoundException
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Ensuring chocolatey commands are on the path
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Ensuring chocolatey.nupkg is in the lib folder
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Mode                LastWriteTime     Length Name
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ----                -------------     ------ ----
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: d----        07/06/2016     13:14            chocolatey
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: executed successfully
    my_machine: Debug: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]: The container Class[Chocolatey::Install] will propagate my refresh event
    my_machine: Debug: Exec[install_chocolatey_official](provider=powershell): Executing 'cmd.exe /c ""C:\Windows\system32\WindowsPowershell\v1.0\powershell.exe" -NoProfile -NonInteractive -NoLogo -ExecutionPolicy Bypass -Command - < "C:\Users\puppet\AppData\Local\Temp\puppet-powershell20160607-668-12s53an.ps1""'
    my_machine: Debug: Executing 'cmd.exe /c ""C:\Windows\system32\WindowsPowershell\v1.0\powershell.exe" -NoProfile -NonInteractive -NoLogo -ExecutionPolicy Bypass -Command - < "C:\Users\puppet\AppData\Local\Temp\puppet-powershell20160607-668-12s53an.ps1""'
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ï»¿Downloading http://myserver/NuGet/api/v2/package/chocolatey/0.9.10-beta-2016032
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: 3 to C:\Users\puppet\AppData\Local\Temp\chocolatey\chocInstall\chocolatey.zip
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Installing chocolatey on this machine
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: The term 'else' is not recognized as the name of a cmdlet, function, script fil
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: e, or operable program. Check the spelling of the name, or if a path was includ
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ed, verify that the path is correct and try again.
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: At line:1 char:5
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: + else <<<<  {
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:     + CategoryInfo          : ObjectNotFound: (else:String) [], ParentContains
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:    ErrorRecordException
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:     + FullyQualifiedErrorId : CommandNotFoundException
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Ensuring chocolatey commands are on the path
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Ensuring chocolatey.nupkg is in the lib folder
    my_machine: Notice: /Stage[setup]/Chocolatey::Install/Exec[install_chocolatey_official]: Triggered 'refresh' from 2 events

Should note this only started since the chocolatey module went from 1.2.3 to 1.2.4

And Puppet version is?

The only place in the code that looks like this is : https://github.com/chocolatey/puppet-chocolatey/blob/461a0aa0ffca2fb4431c559ad1468d23a0687061/templates/InstallChocolatey.ps1.erb#L105

I also recommend updating your beta.

It seems the try/catch is likely causing this error later. Perhaps if you can help me adjust that - we can try to determine what is causing it - I tested this scenario with the older PowerShell provider on Windows 2008 R2 and didn't see this issue, so I missed getting this to happen.

Let's try changing that line to } else { (the } is on the line above it, so just remove the line break.

Puppet version is 3.8.4
I've made the change, running a build now to see what happens.

That seems to have done the trick :)

ugh, fun times - let's fix that.

Do you want to follow up with a pull request or do you just want me to fix it?


Moving files to /usr/lib and symlinking to /usr/bin

Discussed with @chxrlt and it seems a better option than including all current files in /usr/bin. Would also allow launching from terminal.

Fixed in branch improved-installer under commit 57ffdf7


Escaping issue for the password

If the Salesforce password contains a $, the force login with -u -p results in "ERROR: INVALID_LOGIN: Invalid username, password, security token; or user locked out.".
For example:
force login -u<EMAIL_ADDRESS>-p test12348$yvz4VQFVVvGJvVWFaMtzc7QV7
would fail with this INVALID_LOGIN error.
However, if you remove the $ from the password and use the new security token, then the login is successful.

Your shell is expanding the variable, yvz4VQFVVvGJvVWFaMtzc7QV7.  Quote your password to prevent shell expansion, force login -u<EMAIL_ADDRESS>-p 'test12348$yvz4VQFVVvGJvVWFaMtzc7QV7'


fix(3069): fetch total numer of lots of tabs

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3069
[x] while Profile Component created, fetch totalCount of each tabs to set.

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at /rmrk/u/DQcegDuBQG6V99hgRd87UJ8anZxTcumJEVBAnAGomXCJ3dc?tab=sold&page=1
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

@petersopko maybe we should just hide tabs if they dont have anything to show?

@petersopko maybe we should just hide tabs if they dont have anything to show?

@zhengow i think it's a good idea, please, can you add that?

already done, plz check

@petersopko maybe we should just hide tabs if they dont have anything to show?

@zhengow i think it's a good idea, please, can you add that?

already done, plz check

@petersopko maybe we should just hide tabs if they dont have anything to show?

@zhengow i think it's a good idea, please, can you add that?

@petersopko add this feature just now, plz check

pay 100

😍 Perfect, I’ve sent the payout
💵 $100 @ 72.08 USD/KSM ~ 1.387 $KSM
🧗 EgCzZLpVe7zg7RHh79qPyC8UrykVkA22YY2oj7sWMZf15Ws
🔗 0xf27aeb1126cf77c5e216bfe9251192b0c23d272536c0a6d7864541cf0ee1d997
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Add support for IIinputFieldDescriptor extension methods?

Allowing model validations to be done with code-first (not just annotation-based) inputs.
Say you have an underlying model and you're specifying an IInputObject for it
descriptor.Field(p => p.Id).RegularExpression("\w+")

Hi,
Thanks for getting in touch.
I think for your intends and purposes this package https://github.com/appany/AppAny.HotChocolate.FluentValidation might be more inline with your request.
BR,
Adam


ConfigOptions: add new option rename.sanitizeRenames

incomplete, just prototyping for my own usage right now.
would love an initial pass from someone to let me know if my approach is overall in the right direction

Can you give examples of expected output given a sample schema?

in source? or just here?
an example is an entity rename of Measurement to Measurement<UnitDuration>. that's the test i added'

I would recommend adding a simple example to your description to know what the problem is.
This will most likely require a test case in GenerateOptionsTest with the sanitisation enabled with a rename.


Changed dashboard to show correct disk usage percentage

Fixes small mistake to show the percentage in the disks box of the dashboard.

Good find, thanks!


df.printSchema()

data_df.printSchema() should be df.printSchema()

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

ID: 65331fde-51d0-365e-e182-2c3b01de8f99
Version Independent ID: 79b4f02b-27a5-019a-1b36-4e80a04f539f
Content: Quickstart: Get started analyzing with Spark - Azure Synapse Analytics
Content Source: articles/synapse-analytics/get-started-analyze-spark.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @saveenr
Microsoft Alias: saveenr

@lorenzorizziGH  Thank you for reaching out, not sure if you had any specific feedback or query on any of our Azure documentation. Please feel free to revert back if you have  any specific queries/feedback. Thank you


thanks ! it worked!

Hi bdutro ! you save me a multi thousand usd box i needed to admin pass reset today for a project.
here my contribution info links:
1- tested in a x3650 M4 HD, worked !
2.- I didnt know any on UEFI and PXE, but this next links helpme a lot. At the end I didnt use PXE but a USB pendrive with your file at /efi/boot/BOOTX64.efi
good on PXE ->
https://www.youtube.com/watch?v=5qyTLfJrQM4
a multi function tftp DHCP server
https://www.vercot.com/~serva/download.html

Glad it worked for you! I'll add a note that it works for the M4 models.


Multi-fasta functionality

Is it in the plans to add multi-fasta functionality, instead of having to have separate fasta files for each sequence?

You should already be good! All you have to do is put multiple filenames on
the same line for the -F and -Q options. You can also do it for paired-end
reads, for instance.
Would you let me know how that goes?
Thanks,
Daniel
On Thursday, July 13, 2023, Matthew Nguyen @.***> wrote:

Is it in the plans to add multi-fasta functionality, instead of having to
have separate fasta files for each sequence?
—
Reply to this email directly, view it on GitHub
https://github.com/dnbaker/dashing2/issues/79, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQ5UVNPZDYAQZ5RT3YXFTTXQBX7LANCNFSM6AAAAAA2JR3BMU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Daniel, thanks for your reply.
What I mean by a multi-fasta is a single fasta file that has multiple sequences, a format like this:
>sequenceID-001 description
AAGTAGGAATAATATCTTATCATTATAGATAAAAACCTTCTGAATTTGCTTAGTGTGTAT
ACGACTAGACATATATCAGCTCGCCGATTATTTGGATTATTCCCTG

>sequenceID-002 description
CAGTAAAGAGTGGATGTAAGAACCGTCCGATCTACCAGATGTGATAGAGGTTGCCAGTAC
AAAAATTGCATAATAATTGATTAATCCTTTAATATTGTTTAGAATATATCCGTCAGATAA
TCCTAAAAATAACGATATGATGGCGGAAATCGTC

>sequenceID-003 description
CTTCAATTACCCTGCTGACGCGAGATACCTTATGCATCGAAGGTAAAGCGATGAATTTAT
CCAAGGTTTTAATTTG

Oh great! Just use --parse-by-seq to treat each entry as its own entity.
On Thursday, July 13, 2023, Matthew Nguyen @.***> wrote:

Hi Daniel, thanks for your reply.
What I mean by a multi-fasta is a single fasta file that has multiple
sequences, a format like this:

sequenceID-001 description
AAGTAGGAATAATATCTTATCATTATAGATAAAAACCTTCTGAATTTGCTTAGTGTGTAT
ACGACTAGACATATATCAGCTCGCCGATTATTTGGATTATTCCCTG

sequenceID-002 description
CAGTAAAGAGTGGATGTAAGAACCGTCCGATCTACCAGATGTGATAGAGGTTGCCAGTAC
AAAAATTGCATAATAATTGATTAATCCTTTAATATTGTTTAGAATATATCCGTCAGATAA
TCCTAAAAATAACGATATGATGGCGGAAATCGTC

sequenceID-003 description
CTTCAATTACCCTGCTGACGCGAGATACCTTATGCATCGAAGGTAAAGCGATGAATTTAT
CCAAGGTTTTAATTTG

—
Reply to this email directly, view it on GitHub
https://github.com/dnbaker/dashing2/issues/79#issuecomment-1635097785,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQ5UVKDADAGFA43EAK6MJ3XQCJDXANCNFSM6AAAAAA2JR3BMU
.
You are receiving this because you commented.Message ID:
@.***>


Swift 5.0 | Maintenance update

Swift 5.0
Updated swift-tools-version to 5.0 and declared platforms (https://github.com/SvenTiigi/Playground/commit/95a724dc01a0d3d6dd2dc98ddc028e0784d37e3f)
Fix build error
Previously when building playground via swift build the following build errors occurred:
Playground.swift:1:1: error: hashbang line is allowed only in the main file
Playground.swift:328:1: error: statements are not allowed at the top level

They were fixed by simply renaming Playground.swift to main.swift (https://github.com/SvenTiigi/Playground/commit/72c411901c462106fca1e25090457145e5bb6a88, https://github.com/SvenTiigi/Playground/commit/e6dfcd45e23738f5e0d21493736bc2fdd6a8272d)
Mint support
As Marathon is deprecated this PR adds support for Mint to playground by adding a executable product to the Package.swift file (https://github.com/SvenTiigi/Playground/commit/bb93c6241bf0775922614306c2792e46a0d5eea4)
$ mint install JohnSundell/Playground

The installation instructions in the README.md file has been updated according to the changes in this PR.

Thanks for this @SvenTiigi, and sorry that I haven't replied until now. I just went through this project and updated it to Swift 5.1 across the board, cleaned up all Marathon references, and so on. So this is no longer needed, but thanks anyway 👍

Thanks for the replay @JohnSundell, looking forward to the next release of Playground ✌


build: bump fuseki base container version

related to DEV-249

@irinaschubert this is very strange. Locally I got an error in the SearchRouteV2R2RSpec but now it runs through. The failed test on Github-CI was another test. I have now rerun the Github-CI tests. Let's see what happens.


Not able to scroll down on touch devices through image

When I am using smartZoom plugin i can't scroll page on touch devices (ipad, iphone) when i scroll through image. On desktop it works fine with mouse wheel.

Same problem here. We should be able to scroll down the content when the bottom of the image reaches the bottom of its container and scroll up when it reaches its top.

I provided a fix to allow scrolling when the target (image) is not zoomed. I think it make sens in terms of UX. #27


Include AdmittingDiagnosisSequence in sm_index

This sequence is populated for CCDI, CMB and RMS collections, and not populated for radiology collections.
SELECT
  PatientID,
  AdmittingDiagnosesCodeSequence
FROM
  `bigquery-public-data.idc_current.dicom_all`
WHERE
  collection_id = "ccdi_mci"

A related topic was also brought up by a user in this post: https://discourse.canceridc.dev/t/how-to-access-accompanying-clinical-data-for-the-ccdi-mci-collection/636.


This package seems to no longer work; I'm getting these error messages

+1 on this. Any help appreciated please :)
I wonder if Dukascopy started requesting a logon recently which broke this? The behaviour when going incognito supports this theory, i.e. when pressing 'Download' incognito mode for the first time you're asked to log on, but not on subsequent downloads. See this note on incognito.
res = await loop.run_in_executor(None, lambda: requests.get(url, stream=True))  in fetch.py line 24 doesn't send cookies.  Here are the docs, but I've not found a way to send them yet.
However sometimes this code has worked for 1 day at a time, so I'm not sure....

+1 Please update if anyone solves this. Many thanks

Getting the same errors, anyone have potential solutions?

I could download data for 2003 in some pairs, and two weeks at a time from a month back.  But I got asked to log in at apparently random intervals.

@smorrel can you please implement it for firefox cookies? just add an argument for the cookiefile path. I've tried to do it myself but I'm not that advanced, and I could not make it to work. But I guess this is the way to do it: https://stackoverflow.com/questions/49502254/how-to-import-firefox-cookies-to-python-requests

Hi, after some investigation I might have missed diagnose the problem... you could try using JFOREX to download historic data quite easily instead, it’s what worked best for me.

Can you please guide me on how to use JFOREX? I need stock data feed.

Try to limit number of threads default is 10, I am using 2 threads and it seems to work, anything above 2 is throwing the Exception as on the first comment.
Example of working call:
"duka EURUSD -s 2017-01-01 -e 2017-12-31 -t 2"

@milo-hyben thanks! that worked for me.

@nick2012 JFOREX workes fine to download historical data. Just open a demo account on dukascopy, download and install JFOREX and sign in with your demo account.
In the menubar click on "View" --> "Historical Data Manager".
On the popup window you can make your settings and download everything, it's pretty straight forward.
I downloaded the complet Dukascopy dataset in a few hours.

Does it still work for anyone? I get Request failed ... even with 1 thread

not working

Try to limit number of threads default is 10, I am using 2 threads and it seems to work, anything above 2 is throwing the Exception as on the first comment.
Example of working call:
"duka EURUSD -s 2017-01-01 -e 2017-12-31 -t 2"

wow it really works... Thanks alot


[Sema] Fix Issue #63291 by aligning error messages for failure to infer generic parameter and opaque result type

Aligned the errors messages caused by failure to inferring generic parameter and opaque result type, by changing the messages appropriately in the include/swift/AST/DiagnosticsSema.def file, as well as the tests file test/type/opaque.swift and test/type/opaque_return_named.swift.
Resolves https://github.com/apple/swift/issues/63291.

@AnthonyLatsis This is the new pull request.

Hey @Rajveer100,
Thanks for the pull request. @AnthonyLatsis’s comment from your previous pull request are still valid though. I would suggest that

You squash your three commits into one
Edit your commit message to have a [Sema] prefix
Force push the changes to your branch

For each of these bullet points, there should be plenty of tutorial on the internet of how to do them either on the command line or in whichever Git GUI you are using.

Sure! I was just gonna do that...These mistakes are increasing my knowledge a ton! Thanks!

OK, I just thought that you opened a new pull request because you didn’t know that you could force push, so I thought I’d mention some tips.

Made the changes!

There is still a redundant merge commit.

It’s not really required but since we are already doing a Git lesson: You can get rid of the merge commit in your git history by rebasing on top of main instead of merging main into your branch. That way this PR will only contain a single commit.
Just pull recent main, switch to your branch and run git rebase main. If the merge commit is still in your git history, you can delete it using git rebase as well.

Pulling is probably not necessary in this case, but if choose to, make sure to use update-checkout.

Done!

One more suggestion. Consider placing the bug link in the commit message body rather than the title. For example:
[Sema] Align error messages ...

Fixes <link>.

The <link> would be of the form #issue_number or {url}?

Amended!

@swift-ci please smoke test macOS

Test failed, what could be the cause?

The failure isn’t your fault. Can you get rid of the merge commit before we re-trigger the tests?

I have done it, you may re-trigger the smoke test! @AnthonyLatsis

@swift-ci Please smoke test

@ahoppen What would be some next potential projects I can contribute to and what would be the steps for swift-evolution proposal (and maybe you could guide me for GSoC 2023) as I came across certain issues like implementing C++ libraries for interoperability.

@swift-ci Please smoke test

It really depends what you want to work on.
Regarding the compiler, I don’t know of any specific issues in the compiler codebase that would be good to get started, apart from the list of issues marked as “good first issue” that you probably already know about: https://github.com/apple/swift/labels/good first issue.
Keep in mind there are other projects like swift-syntax or sourcekit-lsp in the Swift ecosystem that are mostly implemented in Swift and which also have issues that would be good starting points. Especially for those projects that have fewer issues, it might also be worth scanning through issues that aren’t marked as “good first issue” as we sometimes forget to add the label.
Regarding swift-evolution: You can participate in review threads on forums.swift.org. I would advise against trying to create an evolution proposal unless you’ve got a specific language change in mind that you want to push.
As for GSoC, in the past years we have always proposed ideas on swift.org and announced them in the forums but you can also propose your own project ideas if you have any. If you do, I would suggest you create forum post that describes your project idea.

Thanks! I will look into the other repos as well, and regarding swift-evolution I actually meant to solve the issues having that tag, and not mentioning my own ideas as of now, so I was clueless on how to even make a proposal for the same. The same goes for GSoC, I don't know how to start with such things, felt little clueless, hence asked. Maybe as I contribute with time, I will get some idea.

I would suggest that you start with smaller issues to get familiar with the code base, the process etc. and not start with implementing entire language features – that’s what I did as well when I got started contributing to Swift as well.
Regarding GSoC, I would keep an eye out in the forums because that would be the place we make any announcements regarding GSoC 2023.

Cool! I too actually thought the same, that with time, I can contribute and gain experience and gradually understand everything by usual search, mistakes and grinding, instead of asking questions to do this and that, but had to do a little journalism approach (I had even asked Anthony if I bothered him a lot😂) at least in the beginning to get a push start and setup my arena!

Why are the tests failing by the way?

The Linux failure seems to be some kind of CI hiccup. It looks like the run was aborted by the force push and didn’t re-trigger when I requested it.
@swift-ci Please smoke test Linux

Oops, sorry for that, actually I clicked the sync fork by mistake in my repo, and that caused a redundant merge commit, then had to force push.

macOS test has passed, others have failed.

@swift-ci please smoke test Linux

Nice, so only windows test remains.

We can skip the Windows build, it cannot possibly be related to a change like this if the other jobs passed.

Oh alright, by the way, could you review my next PR!


Munich-Air MD11 Livery

🔗 Upload or paste a link to the CSL file(s) that you want me to add.
https://munich-air.de/uploads/files/MD11_DMA.zip

🛑 You grant me a license to redistribute the files, and you affirm that you have the legal right to grant that license under copyright law.
I grant you a license to redistribute the files, and i affirm that i have the legal right to grant that license under copyright law

✈ Write here the corresponding aircraft and airline ICAO codes.
Its a new Virtual Airline with the Name: Munich-Air. Our ICAO code is: DMA

Ready to go in BB\BB_Virtual.

BB_Munich_Air_VA.zip

https://munich-air.de/uploads/files/MD11_DMA.dds
I updated the dds because I had forgotten the black nose and the red winglets

The .zip above is updated and include the new .dds.
Happy landing.


fix: issues in splitting URL based on ":"

We should perform a "max split" of 2, not undetermined, in order to make it robust.

This will need a patch release ASAP - other projects might be affected (not only PyAnsys Geometry)


Add Highlight In SQL

Description
Support highlighting function in SQL query engine.
Issues Resolved
Github Issue: 636
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented.

[x] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@forestmvey looks like a test is failing now fyi.
    org.opentest4j.AssertionFailedError: expected: org.opensearch.sql.ast.expression.HighlightFunction@9443150<HighlightFunction(highlightField=fieldA)> but was: org.opensearch.sql.ast.expression.HighlightFunction@60bdba16<HighlightFunction(highlightField=fieldA)>
        at app//org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:55)
        at app//org.junit.jupiter.api.AssertionUtils.failNotEqual(AssertionUtils.java:62)
        at app//org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
        at app//org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:177)
        at app//org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:1124)
        at app//org.opensearch.sql.sql.parser.AstExpressionBuilderTest.canBuildHighlighFunction(AstExpressionBuilderTest.java:314)

Once HighlightOperator goes away (or we can explain what it's for), other comments are addressed, and the CI tasks pass, I think it's good to go.

At least, the two failing actions would be fixed by merging from upstream:main (https://github.com/opensearch-project/sql/pull/704) - I believe


Textfield: Support Read only fields

Describe your feature request
Textfields should support read only mode
Context
https://material.io/design/components/text-fields.html#input-types

[ ] Add prop
[ ] Update props table
[ ] Add doc demo
[ ] Add storybook demo

Since the text does not change this is already supported with react native editable https://facebook.github.io/react-native/docs/textinput#editable


Set module type to ESM in https://github.com/serlo/frontend-e2e-tests

@eliflores It would be very cool if you could take this over

Moving this issue back to the backlog, since the configuration required by Codeception needs to be in a CommonJS format - see comment here.


AirBnb zoom out menu effect

Hi folks,
I would like to know if is possible to reproduce the AirBnb menu effect, like this:
http://nerds.airbnb.com/wp-content/uploads/2013/11/AirNav-576x1024.jpg
Thanks!

Yes by setting a custom tween callback. See example.

@justinobney  can you send example link

I would also like to know the tween.
Sorry, but I cannot even seem to get the rotate work on my app :(


Allow setting WHISPER_MODEL to model ID

If WHISPER_MODEL contains a '/', faster-whisper would interpreted it as HF Model ID:
https://github.com/SYSTRAN/faster-whisper/blob/v1.0.2/faster_whisper/transcribe.py#L102
https://github.com/SYSTRAN/faster-whisper/blob/v1.0.2/faster_whisper/utils.py#L56
However faster-whisper-server currently only allows sizes:
pydantic_core._pydantic_core.ValidationError: 1 validation error for Config
whisper.model
  Input should be 'tiny.en', 'tiny', 'base.en', 'base', 'small.en', 'small', 'medium.en', 'medium', 'large', 'large-v1', 'large-v2', 'large-v3', 'distil-small.en', 'distil-medium.en', 'distil-large-v2' or 'distil-large-v3' [type=enum, input_value='bofenghuang/whisper-large-v2-cv11-german-ct2', input_type=str]
    For further information visit https://errors.pydantic.dev/2.7/v/enum

Thanks, for creating the issue! I'll make the change to allow passing model ids.

Passing in model IDs is now supported!
I've also added /v1/models and /v1/models/{model} routes


Set (width,height) of arc/circle to (diameter,diameter)

This PR (delete as applicable)

Fixes a bug

Describe the changes below:
Set (width,height) of arc/cricle to (diameter,diameter), not (radius,radius). #4123

👍


Scoped Type Variables?

Does Flow have/intend to have any support for scoped type variables?

Nope!
/* @flow */

type Merge<T> = (a: T, b: T) => T;

// hypothetical immutable map
declare class Map<K,V> {
  (): Map<K,V>;
  insertWith(fn: Merge<V>, k: K, v: V): Map<K,V>;
}

declare function foldr<A,B>(fn: (a: A, b: B) => B, b: B, as: A[]): B;

function insertMany<K,V>(merge: Merge<V>, vs: [K,V][], m: Map<K,V>): Map<K,V> {
  function f([k,v]: [K,V], m: Map<K,V>): Map<K,V> {
    return m.insertWith(merge, k, v)
  }
  return foldr(f, m, vs)
}

test.js|14 col 22 error|  identifier K
|| Could not resolve name
||
test.js|14 col 24 error|  identifier V
|| Could not resolve name
||

If you add the type parameters to f you get some very readable errors that make it clear the K and V are not the same in the inner and outer functions.
test.js|15 col 12 error|  call of method insertWith
|| Error:
test.js|13 col 23 error|  V
|| This type is incompatible with
test.js|14 col 3 error|  some incompatible instantiation of V
||
test.js|15 col 12 error|  call of method insertWith
|| Error:
test.js|14 col 14 error|  K
|| This type is incompatible with
test.js|14 col 3 error|  some incompatible instantiation of K
||
test.js|15 col 12 error|  call of method insertWith
|| Error:
test.js|14 col 16 error|  V
|| This type is incompatible with
test.js|13 col 1 error|  some incompatible instantiation of V
||
test.js|15 col 12 error|  call of method insertWith
|| Error:
test.js|14 col 16 error|  V
|| This type is incompatible with
tes2.js|14 col 3 error|  some incompatible instantiation of V
||

See #291

See #160, too. I think this can be closed, because the issue is already being tracker.

Thanks!

@DavidHarrison If you were holding off using Flow for this, you'll be happy to know that this feature just landed in master.

That's great, thanks! Flow is coming along really well.


[T3A3][W09-A1]Ng Yong Sheng

Bot error; ask tutor on it.

Some comments added. Please close the PR after reading comments.


[Edison Arduino] Illegal arguments for construction of _exports_Pwm on IO 10/11 (PWM swizzler)

Environment:
root@blurr:~# cat /etc/version && node -e 'console.log(require("mraa").getVersion());' && node -v
edison-rel1-maint-weekly_build_16_2014-10-14_14-56-19
v0.5.4-2-gffdd828
v0.10.28

Error:
root@blurr:~# node
> var m = require("mraa")
undefined
> var p = new m.Pwm(11)
Error: Illegal arguments for construction of _exports_Pwm
    at repl:1:9
    at REPLServer.self.eval (repl.js:110:21)
    at repl.js:249:20
    at REPLServer.self.eval (repl.js:122:7)
    at Interface.<anonymous> (repl.js:239:12)
    at Interface.EventEmitter.emit (events.js:95:17)
    at Interface._onLine (readline.js:202:10)
    at Interface._line (readline.js:531:8)
    at Interface._ttyWrite (readline.js:760:14)
    at ReadStream.onkeypress (readline.js:99:10)

(blurr => edison)

FWIW, I would vote for just closing this one - we state clearly in the docs that we only support PWM on default swizzler pins and anything else (e.g. allowing to use 10 and 11) would indeed require something outside of the library control - a user action on swizzler config - which IMHO is more dangerous than beneficial.
Same as right now someone could be confused by not being able to drive PWM on pins 10 and 11, they would be confused by GPIO pins being activated unexpectedly. And as long as the former could only cause frustration (until the documentation is read), the latter could cause some actual damage, that's why I believe the current approach is safer thus better.

@arfoll, do you think we can close this one, IMHO it's quite straightforward?
@rwaldron, any additonal suggestions on how we could make it clear, now that some time passed (I see just one mention so far, so looks like there weren't too many problems with that)?

There is nothing to be done in software space.


Login methods

An organization can be connected to one or more login methods, the methods that are allowed for a user to login. Examples:
Password
Magic Link
Google OAuth
A specific SAML provider

The specified login methods are what the #5 show, and allow. Users may not login via a login method not allowed by the organization.

An organization admin can choose their login methods


Releasing is somewhat bumpy

It don't exactly know what it is, but releasing feels more exciting and adventurous than it should be, I'll let this issue open to remind of that.

It feels better now


agent: Use NegotiateAPIVersion for docker client

WithAPIVersionNegotiation makes the Docker client negotiate down to a lower
version if Docker's current API version is newer than the server version.

I have tested this as follows, in an attempt to reproduce an issue reported by @Buadhai when using an older apt-installed version of Docker on a Raspberry Pi.
The SSH Gateway (server) is a private Debian 10 host.  The agent is to be installed on a Raspberry Pi 4 with the latest Raspberry Pi OS, based on Debian 10 Buster.  Docker not previously installed on this device.
# apt update
# apt upgrade
# apt install docker.io
# docker --version
Docker version 18.09.1, build 4c52b90
# apt install docker-compose
# docker-compose --version
docker-compose version 1.21.0, build unknown
# usermod -aG docker pi

This is the same version used by @Buadhai.
Check Docker installation works:
# docker run hello-world
Attempted to register the agent by running the curl command specified by the server.  This returned no errors but the registration failed: the device pending and rejected lists on the server were empty.  These are the symptoms reported by @Buadhai but he also saw an error in syslog:

msg="Error response from daemon: client version 1.40 is too new. Maximum supported API version is 1.39"

This error was NOT seen in syslog during this test.
Repeated the registration attempt using the new agent version:
# docker rm shellhub
# docker run -d \
       --name=shellhub \
       --restart=on-failure \
       --privileged \
       --net=host \
       --pid=host \
       -v /:/host \
       -v /var/run/docker.sock:/var/run/docker.sock \
       -v /etc/passwd:/etc/passwd \
       -v /etc/group:/etc/group \
       -e SERVER_ADDRESS=http://<address of server> \
       -e PRIVATE_KEY=/host/etc/shellhub.key \
       -e TENANT_ID=<tenant id> \
       gustavosbarreto/shellhub-agent:arm64v8

This apparently fails to start:
# docker ps
CONTAINER ID        IMAGE                                    COMMAND             CREATED             STATUS                          PORTS               NAMES
46efbce17d59        gustavosbarreto/shellhub-agent:arm64v8   "/bin/agent"        22 minutes ago      Restarting (1) 43 seconds ago                       shellhub

Nothing is seen on the server.
Repeated in the foreground:
# docker stop shellhub
# docker rm shellhub
# docker run \
    --name-shellhub \
   etc

This reported:

standard_init_linux.go:207: exec user process caused "exec format error"

@gustavosbarreto I think Raspian uses ARM32. So it needs to be the other arch.
@sixhills could you check the output of:
file /bin/bash

So we check the file format your system is using?

@gustavosbarreto I think Raspian uses ARM32. So it needs to be the other arch.
@sixhills could you check the output of:
file /bin/bash

So we check the file format your system is using?

@otavio @sixhills
Replace the arm64v8 tag with arm32v6 in the docker run command.

@gustavosbarreto @otavio Yes, Raspberry Pi is 32-bit ARM.  There's a 64-bit beta available but not widely used and not yet recommended.
# file /bin/bash
/bin/bash: ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux-armhf.so.3, for GNU/Linux 3.2.0, BuildID[sha1]=3e5e2847bbc51da2ab313bc53d4bdcff0faf2462, stripped

Replacing the arm64v8 tag with arm32v6 in the docker run command works!  The device is registered on the server succesfully and I can ssh to it via the server.  Great work.  My only concern is that I wasn't able to reproduce the "client version 1.40 is too new" error seen by @Buadhai.

So it seems it worked just fine. I think we can merge this PR then.

My only concern is that I wasn't able to reproduce the "client version 1.40 is too new" error seen by @Buadhai.

@sixhills Where you unable to reproduce using the default install script or with custom docker run command?

@gustavosbarreto  The error message "client version 1.40 is too new" wasn't logged to syslog at all during my testing: not with the default install script and not with the custom docker run command.  I'm wondering whether there's something non-standard on @Buadhai's system that causes this.


Missing BICs for Portuguese banks

Hi,
We have been using your lib to determine the SWIFT (BIC) code from IBANs, and we found that the following are missing:
Code    Name                            SWIFT
0033	BANCO COMERCIAL PORTUGUÊS, SA   BCOMPTPL
0018	BANCO SANTANDER TOTTA, S.A.     TOTAPTPL
We are not sure if more are missing, but you can get a complete list here: https://www.santander.pt/pdfs/pdfs-empresas/netbanco-empresas-ajuda-online/netbanco-empresas-ajuda-online-documentos/sepa/Enderecos_SWIFT_Bancos_portugueses.pdf
Is it possible to add these to your lbirary? We are currently hardcodingg the SWIFT codes for these banks.

Hi @tiagoafseixas,
would it be possible for you to create a pull-request where you add the missing bank entries to the manual_pt.json? I think I will currently not find the time to write a script that parses the PDF file you linked. Your help would be greatly appreciated.

Will do. I was checking previous pull requests and it seems as if you created branches for other issues. Do you want to do the same here?

I'm not sure if I understand what you mean by that. Just create a regular pull-request by forking this repo and applying the changes on a branch on your fork.


add logic to load /etc/postfix files

Hi, I really like your project
I badly needed to add custom versions of transport, sender_bcc_maps, recipient_bcc_maps,
so I simply:

created a new directory under config (i.e. etc-files)
added this small snippet to start-mailserver.sh in order to copy files in /etc/postfix

for f in /tmp/docker-mailserver/etc-files/* ; do         base=$(basename $f)         echo copying $base         cp $f /etc/postfix done
It works for me, but do you suggest a better way?
Thank you
Roberto

Hi @RobertoPirrone, thanks for your message!
This can help? https://github.com/tomav/docker-mailserver/wiki/Override-Default-Postfix-Configuration

Hi
Not exactly. I already used the config/postfix-main.cf file for other info (mydestination and the names of bcc maps files).
But I will try to follow your advice, probably I can embed the files in a "text" type map, I will investigate. Each file is just a couple of rows, it won't be that difficult
I will report my findings, thank you

"text" type map was a pipe dream. The one I was thinking of is "inline", but sadly it will be available only in postfix 3.
So I give up, and keep the small change in start-mailserver.sh
BTW, /etc/postfix/main.cf is not overwritten, but changed whenever you restart the container. So, if you inject some error (as I did with my tests), you must either recreate the container, or manually modify the file inside the container. Not a big deal, but maybe overwrite main.cf at every container restart could be less error prone.

Could you paste the diff of the change you made?

Actually I’ m not very proud of my, ehhm, solution. It’ ugly as sin, but it works for me.
I decided that the easiest path is just copying the files that I need at startup.
So:
under config (mapped as /tmp inside the container), I created a subdirectory etc-files, where I put the hand made files (for me these files are local_recipients  recipient_pcre  sender_pcre  transport)
inside docker-compose.yml I added the startup command:
command: /bin/bash -c "sleep 3; cp /tmp/docker-mailserver/etc-files/* /etc/postfix; rm /var/run/dovecot/master.pid;  /usr/local/bin/start-mailserver.sh"
hope it helps, bye

Il giorno Sep 8, 2017, alle ore 6:55 PM, Johan Smits<EMAIL_ADDRESS>ha scritto:
Could you paste the diff of the change you made?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tomav/docker-mailserver/issues/529#issuecomment-328157917, or mute the thread https://github.com/notifications/unsubscribe-auth/AVSxEqnPTWMl2rOmComyoCjhAUAFWwFgks5sgXGWgaJpZM4MKU-y.

Thats not that elegant indeed ;-)
Could you share the config file(s) that you need so that we can implement this feature?

Il giorno Sep 10, 2017, alle ore 10:37 AM, Johan Smits<EMAIL_ADDRESS>ha scritto:
Thats not that elegant indeed ;-)
Could you share the config file(s) that you need so that we can implement this feature?

Well, the logic behind these files is that I have an external archiver, so I had to add rules both for sender and recipient, and add a transport rule
Actually if only /usr/local/bin/start-mailserver.sh could copy the files found in ad hoc directory, that could be fine (and a little bit more elegant….)
regards

[root@docker-int etc-files]# cat local_recipients
@most.it xx
@ast.to.it xx
[root@docker-int etc-files]# cat recipient_pcre
/^(.)@([^@]+)$/<EMAIL_ADDRESS>[root@docker-int etc-files]# cat transport
archive2 smtp:[<IP_ADDRESS>]:10025
.archive2 smtp:[<IP_ADDRESS>]:10025
archive3 smtp:[<IP_ADDRESS>]:10025
.archive3 smtp:[<IP_ADDRESS>]:10025
examples.private smtp:examples.private
[root@docker-int etc-files]# cat sender_pcre
/^(.)@([^@]+)$/<EMAIL_ADDRESS>—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tomav/docker-mailserver/issues/529#issuecomment-328328202, or mute the thread https://github.com/notifications/unsubscribe-auth/AVSxEuEEAV2KWuGBjC_JdJLIAy4TZAvAks5sg5_PgaJpZM4MKU-y.

Is this still relevant (no action since 2017)?

Well, I use my workaround, it' ugly, but plain simple and works for me.
Since I'm the only one that has this problem,  I think that we can close this thread
Thanks to everybody, tomav mailserver is a beatiful project