Error: Type 'FontFeature' not found

Package
google_fonts
Existing issue?

[X] I checked the existing issues

What happened?
Getting a Major Bug in 6.2.0:
Error (Xcode): ../../.pub-cache/hosted/pub.dev/google_fonts-6.2.0/lib/src/google_fonts_base.dart:69:8: Error: Type 'FontFeature' not found
Relevant log output
No response

I am facing the same issue . No solution yet.

Its probably because of this commit. https://github.com/material-foundation/flutter-packages/commit/e32cc8e6546b5cf12cc2e6a456f0428f68f59d8a
FontFeature is defined in dart:ui. And this commit removed it.
For now downgrading to google_fonts version 6.1.0 would work.

I solved the problem temporarily with version 5.1.0

I am facing the same issue. No solution yet.

Sorry about the inconvenience, I believe this is caused by using an older flutter version, because the following checks pass

stable
beta

You can upgrade flutter with flutter upgrade. I've published a patch 6.2.1 that restricts this package to the current flutter stable and newer versions.


why  nvidia   garphics  control panel  modified settings not in registry

actually it should be under  this key na
[HKEY_CURRENT_USER\Software\Microsoft\Avalon.Graphics\DISPLAY2]
Why   the keys  are not created  in registry ? but key is  not there  on my  windows 10

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

ID: d89a3478-1985-d631-b57e-2d2f540b9734
Version Independent ID: 912bbf84-a3b4-ff78-4464-6851417c0ac2
Content: Graphics Rendering Registry Settings
Content Source: docs/framework/wpf/graphics-multimedia/graphics-rendering-registry-settings.md
Product: dotnet-framework
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

which is your manufacturer of laptop/desktop computer?
and model number

Asus vivobook 15 R542UQ-DM153 with bios X542UQ.305 Laptop .
Gpu is Nvidia GeForce  940MX GDDR5  2GB VRAM
On 24-4-2019, I installed nvidia GeForce DCH based driver 430.39

@RAJU2529
Have you seen this problem only with this driver or with several others?

yes sir I checked all drivers.  problems persists.  This is laptop.
If you have  laptop, please Check the corresponding registry key, that I have mentioned previously.
Thanks sir for reply

@RAJU2529 Apologies for the late response.  Did you find an answer somewhere else or are you still looking for an answer?

@tdykstra. No sir I never got answer

@grubioe @SamBent Can you help with this question?

The keys are not in the registry, by design.  They are used to troubleshoot and work around problems related to WPF's interaction with your video driver;  if you need them, you're expected to create the keys yourself either directly (regedit) or with a tool.
Nvidia's control panel is probably not setting WPF-specific diagnostic registry keys.  You'd have to ask Nvidia about the expected effects of their settings.
WPF does not recognize [HKEY_CURRENT_USER\Software\Microsoft\Avalon.Graphics\DISPLAY2].  The doc page lists the four keys that WPF recognizes;  any others come from somewhere else (not WPF or .NET).

Thanks @SamBent for your help. Closing the issue then.


Stackification

Built for the first time successfully out of the box with Stack, first time I've had that happen with mu-eval. Don't know if it was recent work on mu-eval or Stack. Anyhoo, here it is.

The commit message couldn't be any less informative, could it?

@nikotine I do my best.


Update transform-react-inline-elements.md

Simplify the examples to be more explicit.

Thanks!

And we could mention the deopts here as well?

@hzoo Looks like you have already added the deopt, awesome :+1:.


More reformat

Reformatted all the code using IntelliJ default settings.

i wonder how did you get this format?
I  would suggest to have kind of spotless to allow format as embedded and have it the same for everyone

Codecov Report

Merging #416 (96d3baf) into main (79ec13a) will decrease coverage by 0.22%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##               main     #416      +/-   ##
============================================
- Coverage     93.86%   93.64%   -0.23%
+ Complexity     1983     1979       -4
============================================
  Files           218      218
  Lines          3996     3996
  Branches        384      384
============================================
- Hits           3751     3742       -9
- Misses          147      161      +14
+ Partials         98       93       -5

Impacted Files
Coverage Δ

src/main/java/net/datafaker/FakeCollection.java
86.53% <0.00%> (ø)

src/main/java/net/datafaker/base/BaseFaker.java
89.87% <ø> (ø)

src/main/java/net/datafaker/base/CNPJ.java
100.00% <ø> (ø)

src/main/java/net/datafaker/base/Code.java
97.70% <ø> (ø)

src/main/java/net/datafaker/base/DateAndTime.java
86.76% <ø> (ø)

...in/java/net/datafaker/base/DungeonsAndDragons.java
100.00% <ø> (ø)

src/main/java/net/datafaker/base/EldenRing.java
100.00% <ø> (ø)

src/main/java/net/datafaker/base/Locality.java
96.87% <ø> (ø)

src/main/java/net/datafaker/base/Sip.java
100.00% <ø> (ø)

src/main/java/net/datafaker/base/Unique.java
100.00% <ø> (ø)

... and 20 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@snuyanzin I used IntelliJ default format.

And I don't mind any tools, but I'd rather not install extra plugins in my IDE, and I'd prefer never to see a build failure because someone added a space too many or something. But if mvn clean install or something does a reformat too (without breaking things) then I'm fine.
Reformatting the code actually broke things in Json.java, seems (my) IntelliJ doesn't like the commented out code:

the problem that if it is open source we can not force to have everyone use only a specific IDE with always default settings...

How is that a problem? The code was mis-formatted like this for a few years.
If someone is using any different IDE such as Netbeans or Eclipse, then I hope they use .editorconfig, and if they don't, then I'll fix it.


How to get queries with async/await

Hello,
Is it possible to get query results through async/await, rather than a builder function, so that we can update the widget state more easily? I'm not seeing this on the package description.
Thanks,
Alvin

If you don't want a stream of results, you can access the client directly with the client provider and then use client.query

Future query() async {
SharedPreferences prefs = await SharedPreferences.getInstance();
print("variables query : " + variables.toString());
GraphQLClient _client = clientToQuery(MutationHelper.link(
  prefs.getString(IConstanct.token),
));
return await _client
    .query(
  QueryOptions(
    document: gql(documentNode),
    variables: variables,
    fetchPolicy: FetchPolicy.noCache,
  ),
)
    .then((value) {
  if (value.hasException) {
    print("GQL hasException :" + value.exception.graphqlErrors.toString());
  }
  return value;
});

}


[SPARK-44048][DOCS] Remove sql-migration-old.md

What changes were proposed in this pull request?
This PR removes sql-migration-old.md.
Why are the changes needed?
We can use sql-migration-guide.md directly without redirection:

Does this PR introduce any user-facing change?
No.
How was this patch tested?
manual test.

According to the affected version info of JIRA, I landed it to master branch only.


Return response in the error

Coverage remained the same at 100.0% when pulling 4d2d364e9d73e4f8d51e719d93c79789ab484c6b on maghis:error-resp into 40a5ba433b26008a7d0d0649fd290eacdfebba24 on blakeembrey:master.

This makes sense, but it makes me wonder - is this better suited to generating errors on the response directly instead? Perhaps we want to abstract out the method for generating errors better in the future to accept both an optional request and response?

@blakeembrey probably it's better. I was also thinking that we could type the type the error and provide an isPopsicleError() type guard.


Add device MAC address to device enumeration and display in device config panel

We currently do not report the mac address in WaveForms Live.  We should add the device mac address(es) to the enumeration and display the mac address for each NIC on the WFL config page.
Work around:
OpenScope MZ currently prints it's mac address to the serial terminal (at 1.25 Mbaud) at boot.

Added in https://github.com/Digilent/waveforms-live/commit/a912180d643201899d73f3b790784aba0e5145c0.


Adding badges to track mentorship participation

Based on #1229 @pbanaszkiewicz suggested creating a migration for making new mentorship badges.
Please review closely. First time making a migration, don't want to break anything 😬.

Thanks, @jduckles!


Add distances calculus against vector of strings

Add levenshtein_against_vec, damerau_levenshtein_against_vec,
jaro_against_vec, jaro_winkler_against_vec
to calculate the distances agains a vector of strings

Thanks for the contribution! I added some more tests and published it to Crates.

My pleasure.
Thanks a lot
Best regards
Olivier
Sur 10 juin 2015 06:43, à 06:43, Danny Guo<EMAIL_ADDRESS>a écrit:

Thanks for the contribution! I added some more tests and published it
to Crates.

Reply to this email directly or view it on GitHub:
https://github.com/dguo/strsim-rs/pull/2#issuecomment-110587311

Hi @ovarene, did you have a particular use case for these functions? It's a one-liner now, so I'm thinking about removing them as functions and just providing an example in the readme.

Thanks dany,
yes, I have use for those functions and they can be considered as helpers to avoir rewriting those constantly.
So, if you please could keep them, that would be useful for me.
Best regards
Olivier Varene

Le 31 déc. 2016 à 16:12, Danny Guo<EMAIL_ADDRESS>a écrit :
Hi @ovarene https://github.com/ovarene, did you have a particular use case for these functions? It's a one-liner https://github.com/dguo/strsim-rs/blob/master/src/lib.rs#L125 now, so I'm thinking about removing them as functions and just providing an example in the readme.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dguo/strsim-rs/pull/2#issuecomment-269869114, or mute the thread https://github.com/notifications/unsubscribe-auth/AB26cXBUipXpuhGNiw9AaUHHy9VkRKAdks5rNnDpgaJpZM4E9Qf_.

Did you get a chance to check out the one-liner though?
Instead of using one of these helper functions, you could just do something like:
let distances = strings.iter().map(|b| jaro(target_string, b)).collect();


Double Replacement of & in Feeds

see the &amp; in <author> at http://mikrowelle.me/opus.xml for example

Do you want to fix this? :)


storagenode: add a limit on the number of log stream replicas

The number of log stream replicas in a storage node is not regulated currently. Thus, there is a risk that some misbehaved admin clients can add too many replicas in a storage node, which causes too many garbage replicas and wastes resources.
I am proposing a limit on the number of log stream replicas in a storage node, which can play a role in guarding against creating too many replicas in a single node.

duplicate of #293


Audit log

Add the ability to log INSERT and SELECT queries in an audit log for compliance purposes and performance analysis. Log partial sql statement, origin and the user-id of each query.

Hi @lquerel !
Why don't you use ClickHouse system.query_log table for that purpose? CHProxy populates every request user-agent with additional info: Prepends User-Agent request header with remote/local address and in/out usernames before proxying it to ClickHouse, so this info may be queried from system.query_log.http_user_agent.
So it should be easy to plot performance grpahs directly from ClickHouse. Consider following dashboards as template - https://grafana.com/dashboards/2515

Thanks. I didn't notice the usernames were visible by clichhouse behind chproxy.


Update easy full control

This PR updates the feature/easy_full_control branch with the following:

Configuration::make to easily create a Configuration object by parsing config and navigation graph file paths
Modify GoalCompletedCallback to return the goal remaining time and replan request instead of updating the references
Add Transformation struct and transform helper function
Import task capabilities from config.yaml

WIP: adding minor python bindings for rmf_demos implementation

I don't think a std::unordered_map<std::string, bool> is expressive enough for real use cases. Users of this API should be able to discern between which task requests the fleet adapter should accept based on the descriptions. This might not matter in our demos, but it will certainly matter for real-world use cases when things like inventory need to be accounted for when deciding which tasks to accept.
I've updated this PR to use std::unordered_map<std::string, ConsiderRequest> for task and action consideration settings. For convenience there's also a consider_all() function that allows users to easily get the original boolean behavior.


Author In-Group Analysis Phrase Classification 2 batch.csv file headers say .on which messes up our code

Not erroring, but also not passing with the right result atm when added to the list of tests

There must an error somewhere; right? If you create a PR we can go through the error log.

Yeah! I just resolved this I believe throwing it back in now with the PR soon


handle upstream error

How to capture and handle upstream error in wasm , for example, upstream connect failed.

could you elaborate on what exactly you mean by "handle"? All you can do is operating on these events as specified in Proxy-Wasm specification. You won't be able to do beyond that.
Closing until a further question specific to this SDK arises.

If upstream connection failed, which event will be triggered?


Chef-client is failing because longer cookbook name

We are seeing the following error when executing test_int_server_services_cookbook cookbook on AIX.
================================================================================
Chef encountered an error attempting to load the node data for "node_name"
Unexpected Error:
Chef::PolicyBuilder::Policyfile::ConfigurationError: Error loading policyfile from `policy_groups/local/policies/test_int_server_services_cookbook': Net::HTTPServerException - 404 "Not Found"
System Info:
chef_version=16.6.14
ruby=ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [powerpc-aix<IP_ADDRESS>]
program_name=/usr/bin/chef-client
executable=/opt/chef/bin/chef-client
The test_int_server_services_cookbook cookbook works fine on Linux.
Note If customer rename the cookbook to shorter name  test_services_cb then the chef-client runlist cookbook.
To Reproduce
OS Version: AIX 7.2
Chef Client: 16.6.14
Create the cookbook name similar :  test_int_server_services_cookbook
Run the chef-client --local-mode --chef-license accept
Expected behavior
It should apply the cookbook changes in the chef-client run .
Actual behavior
Chef-client is failing because cookbook longer name (test_int_server_services_cookbook)

One question - can someone extract out the full path to the filename in the local-mode cache? There is a small chance this is hitting an absolute filename limit on AIX


bibliography display 5: short titles for journal volumes

Some modifications are necessary for display of bibliographic data in the case of short titles for journals.
Code example:
<bibl>Brandes in                	<ptr target="bib:NBG34_1896"/>               	<citedRange unit="page">53</citedRange>               	<citedRange unit="item">3</citedRange>   </bibl>
Current display:

Desired display would be something like this:

Brandes in NBG 34 (1896), 53 no. 3

See also #62 and my tentative proposal in #80.

I have implemented it and tested for most of the journal shortTitle. To encode it, <bibl rend="journal"> . Please note, it will work only for the section if the bibliography.

looks nice to me.
so @danbalogh : I think we need to add this <bibl rend="journal"> in the EGD.

@ajaniak : can you display the journal abbreviations in italics?
once that is done, and @danbalogh has confiomed that <bibl rend="journal"> has been added to EGD, I can close this issue.

Done

Thanks. But something is going wrong.

code:

<listBibl type="primary"> 		<bibl n="H" rend="journal"> 			<ptr target="bib:Holle1880_02"/> 			<citedRange unit="page">1-2</citedRange> 			<citedRange unit="part">I A</citedRange> 		</bibl> 		<bibl n="B" rend="journal"><ptr target="bib:Brandes1889_01"/></bibl>	 		<bibl n="S"><ptr target="bib:Sarkar1971-1972_01"/><citedRange unit="volume">2</citedRange><citedRange unit="page">99-101</citedRange><citedRange unit="item">73</citedRange></bibl>	 		<bibl n="TSN"><ptr target="bib:TitiSurtiNastiti2016_01"/><citedRange unit="page">412-413</citedRange></bibl> </listBibl>

display:

Well the Holle entry is handled in Zotero as a singular one. The @rend="journal" is made to deal with the one you have handled as journal at collection levels.
I will see what I can do for your Holle and the rest af the data type journa. But since it will required changing the interaction with the API, so not sure when you will have it.

I am sorry, but I haven't understood your reponse.
You mention only Holle, but what about the fact that <bibl n="B" rend="journal"><ptr target="bib:Brandes1889_01"/></bibl> disappears, in the above example, except for the @n=B?
And what does "the Holle entry is handled in Zotero as a singular one" mean? I don't see what is exceptional about Holle1880_02, except that it has no pagination.

I'll be happy to update the EGD on this, but to be honest, I don't really grasp the details. @arlogriffiths , could you add stubs or comments to the EGD where you want this, containing a draft of how you want the murky details handled. Here's what I do understand:

bibliographic citations will be displayed by default with a full title; if you want a journal abbreviation to appear in place of a full title, then use <bibl rend="journal">

What I do not understand is:

in what situations is this desirable and when is it not? Arlo's comment (the second above) implies that it is desirable in the structured bibliography. Axelle's remark "it will work only for the section if the bibliography" seems to say that it will not even work anywhere outside the bibliography. If I understand that correctly, then why does this have to be so? Do we want full titles appearing in similar citations within a commentary?

If that is what you want, OK by me, I'll live with it.
But if it is not, then are there any situations in which we want the full title to be displayed? If yes, what are they? If not, could not the display of abbreviation instead of full title be somehow handled on a global level instead of having to add @rend="journal" to each and every citation of such items?

the sample code in Arlo's opening comment has text within a bibl element, <bibl>Brandes in <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl>  - This is new to me. Is this even permitted/sanctioned by TEI? If yes, do we really want to use it? If yes, the EGD should tell encoders when to do so, otherwise only people directly instructed by Arlo to do it will do it.

I gather from the above that Arlo may have used this in the structured bibliography, where there was no TEI-sanctioned way to add the text "Brandes in" outside the bibl element. If I get this correctly, then couldn't this be avoided by having only <bibl><ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl>  in the structured bibliography? The fact that the item cited is by Brandes would, I believe, be communicated anyway in the epigraphic lemma, where you would use Brandes in <bibl><ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl> , i.e. put the non-structured text outside the bibl element.

how does the practice recommended for @rend="journal" tie in with the practice recommended for @rend="siglum"? What, anyway, is the practice recommended  for @rend="siglum", another thing I first heard of today, in issue #82 .

part answering @arlogriffiths about the difference of data structuration:
Since not all the data are structured on the same model in Zotero, it means handling each with a different parsing.
So in the case for NBG and equivalent – journal entered as book – I can't format them using the CSL neither the API, so it has to be done in the XSLT itself using some intel from the shortTitle.
The data like Holle1880_02 are entered with a type JournalArticle provide another set of information and structured differently, but from which I can retrieve author name, journal abbreviation and date. In this case, I could handle the display through the CSL processor, but it is not recommended since it will impact all the bibliographical data of the project and not only the html display. (Unlike we decided to sustain several CSL of the same base style for the project... which unlike it is really necessary I would rather prefer to avoid since it leads to confusion). So Holle1880_02 is currently handled both through the parsing of the API and XSLT transformation to provide the same result.
This distinction of dealing with the data means that I need an anchor in the XML to identify the data you consider like journal. The collection anchor from the Zotero API isn't stable enough to compensate the item type book provided.

Some encoders have been using the reference in commentary, lemma or even in the apparatus is full length and not just with @rend='omitname', one of the reason for provided the @rend="siglum" was to be able to shorten the reference. The value of the attribute will probably evolved since at the beginning we reused the siglum content. @manufrancis will provide more details of his needs.
<bibl>Brandes in <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl> doing so isn't prohibited by TEI. It is not good practice though, you don't half structured - half prosed content of an element. If it can work on some element, it is prone to error especially in cases where it is not just a question of copying the data but work the data to render it as you expect it. I added an  as high as possible in the transformation processing to copy the content but I can' t promise it won't mess up other display at some point. I would be more confortable with
<bibl><author>Brandes</author> <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl>. But I understood from previous discussion that you wanted to reduce the structuration of the <bibl>element for a prose content.

Thanks Dan and Axelle for your most recent messages. Clearly, we are runnning into confusions because we each have different expectations, we don't always explain with sufficient clarity what we expect, and Axelle is getting (or in any case sometimes thinks she is getting) conflicting instructions from her various interlocutors. I apologize for being one of the most guilty parties!
I am not sure what is the best road ahead now that we are in this chaotic situatiin. Take a break and return to these matters later, or try to push towards closure of this issue?
Let me say just the following now:

I think it is helpful if we can specify specify the identity of a scholar inside the structure bibliography, in cases where (a) the zotero short title we are using does not itself contain data in its "author" field, and where (b) there is no other reason to mention the reference in the <p> of epigraphic lemma — the Brandes example corresponds to both criteria.

I have no trouble with Axelle's proposal to encode <bibl><author>Brandes</author> <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl>.

I do not see why we would ever want to show full journal titles in our display, because they will always distract from other data that we cannot abbreviate while we are already defining abbreviated journal titles in zotero (see also the appendix in the ZG), and it will presumably be easy to make the full titles visible by mouse-over over any abbreviation, and/or by offering an old-fashioned list of abbreviations.

This whole discussion started because I wasn't getting the display I was hoping to get in structured bibliography, where citing a full journal title seems to me to distract too much from other bibliographic data. Depending on whether or not I use @rend="journal" in the relevant <bibl>s, I can now produce the following two displays:

with @rend="journal"

without @rend="journal"

My objections to these two presentations are that the first shows too little bibliographic information while the second comes with needless clutter and we have an easy way to declutter (see point 3 above).

The display I was actually hoping to get would look like this:

@danbalogh @AnneSchmiedchen @manufrancis : do you like this last display suggestion?
@ajaniak : do you see any problems with it? I believe implementing it may remove the need to ever use @rend="journal" (but there may be other context where that attribute is needed that I am unaware of).

I do like the last display suggestion and would prefer to avoid introducing the complication of @rend="journal" if possible.
My private preference is still for full journal titles in the structured bibliography and, reversing Arlo's number 3 above, I do not see why anyone would ever want to show an abbreviation instead of a full title when there is no need to economise on paper or ink and the presence of a full title does not disrupt the surrounding text. But this is not important enough to argue over, so if everyone else can agree on showing abbreviations there, then no problem on my part.
I am also OK with <bibl><author>Brandes</author> <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl> in cases where it is essential to add such information in the structured bibliography, and if this gets endorsed, I would like Arlo to add a stub to the EGD saying when encoders should resort to this. (Does Axelle still need to work out a way for this to display with an added "in", or is it acceptable without?)

If you agree on the display 5, I might have a technical solution to handle it. But please be aware it will required an intervention at the CSL level, so it means the journal name will always be displayed abbreviated. I won't be able in special case to revert back.
Please be also aware that this solution fits only for the case like Holle = > journal data with a journal type officially declared as such in Zotero. The cases of journal entries declared as book, if you don't want to use the @rend="journal"as shortcut, we will need to revert to a more complex code and we will need to find a way to discriminate the data to avoid noise in the display. I will figure something out with @manufrancis to make sure we take all  bibliographical data into account.
<bibl><author>Brandes</author> <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl>. I do not need the in inside the citedRange. I can display it though when an element <author> is used. I will need to know before adding it into the code where you plan to need it: in all the bibliographical data or only in the <listBibl> element.

I concur with Daniel in prefering full journal titles in the structured bibliography. Like Daniel "I do not see why anyone would ever want to show an abbreviation instead of a full title when there is no need to economise on paper or ink and the presence of a full title does not disrupt the surrounding text."
The abreviation could be displayed and handled with siglum as per my proposal in issue #90.

I do not insist any longer on being able to have abbreviated titles in the structured bibliography.
In apparatus, Manu's suggestion in issue #90 will indeed make it possible, when necessary, to use journal abbreviations. This also answers an unspoken question I had, namely for what good we are defining abbreviated journal titles at all in zotero and in the ZG, if we never plan to use them. @ajaniak: Are then any other contexts where we'll be usingh them?
Could @danbalogh perhaps summarize where we have ended up with this discussion?

I may be missing out on some things, but it seems to me that the gist is the following:

We forget about @rend="journal" and keep the display of bibliographic items as they are now, as Arlo has gracefully conceded to full journal titles being shown.
In certain specific situations (which are still not entirely clear to me, but seem to be when a/ the reference is to an authorless publication, and b/ you do know who the author is and you want to show this in your bibliography, and c/ you cannot do this in the epigraphic lemma because the item belongs to the secondary bibliography), Arlo wants to permit the syntax <bibl><author>Brandes</author> <ptr target="bib:NBG34_1896"/> <citedRange unit="page">53</citedRange> <citedRange unit="item">3</citedRange> </bibl>, i.e. the inclusion of an <author> element in the reference, as the first item before <ptr>.

If my understanding of 2 above is correct, and it really is desirable to create special rules for such a complex special case, then Axelle and Arlo should work out how such items would be displayed (e.g. using "Brandes in" etc.), and Arlo will need to add a stub to the EGD at the point where he wants to discuss this (perhaps the end of §10.4.5 right before the examples? I may then need to split up 10.4.5 into several subsections because it is already a bit unwieldy, and if @rend="siglum" becomes standard, it will get even more complex), and in this stub he should list the exact conditions in which this kind of encoding is recommended (or mandatory). I will then finish up the stub, discuss any outstanding details with Arlo, and possibly do some restructuring in the EGD, pending the resolution on @rend="siglum".

I am fine with Dan's approach on this.

Following the DHARMA brainstorming (2020/12/17), I have re-considered the issue.
Pleasing, I hope, our Dutch PI, I do not care if Journal titles are displayed as abbreviations.
So please, Axelle, do implement, if necessary, as required, and implement a tooltip showing full title of journal at mouse over.

Following the DHARMA brainstorming (2020/12/17), I have re-considered the issue.
Pleasing, I hope, our Dutch PI, I do not care if Journal titles are displayed as abbreviations.
So please, Axelle, do implement, if necessary, as required, and implement a tooltip showing full title of journal at mouse over.

Dear all,
You seem to have settle in favor of a publication title displayed with an abbreviated form and its full expression only in tooltip.
For now, I have created a new CSL to add the abbreviation feature inside the Chicago author-date model. It allows me to export two versions for a bibliographic citation. Let’s take this journal article
->  Venkayya, V. 1896–1897. “Jain Rock-Inscriptions at Panchapandavamalai.” EI 4.14: 136–40. Is displayed in the html a the main text while Venkayya, V. 1896–1897. “Jain Rock-Inscriptions at Panchapandavamalai.” Epigraphia Indica 4.14: 136–40. is provided as a tooltip.
I can’t structure more precisely the data with the current system. Indeed, the citation is constructed through Zotero API so I don’t have enough anchor inside the result to apply the tooltip only on the journal title.
Another downside of this system is that only items recorded with the type Article Journal in Zotero can be exported as such. All the items you have filled in as book requires to set another system. Initially, that’s was the purposes for the attribute @rend="journal". So instead of using the attributes to discriminate, I set the new system on search for the content of @target starting with the string "bib:" and followed by 2 upper-case letters.
<bibl><ptr target="bib:NBG15_1877"/><citedRange unit="page">136-137</citedRange></bibl> will have NBG 15 (1877) as a main text and Notulen van de Algemeene en Bestuurs-vergaderingen van het Bataviaasch Genootschap van Kunsten en Wetenschappen: Deel XV — 1877. 1878. Batavia: Bruining. as the content of its tooltip.
If you wishes to provide an author
<bibl><author>Brandes</author><ptr target="bib:NBG24_1886"/></bibl> to obtain Brandes in NBG 24 (1886) as main text, the tooltip will contain Notulen van de Algemeene en Bestuurs-vergaderingen van het Bataviaasch Genootschap van Kunsten en Wetenschappen: Deel XXIV — 1886. 1887. Batavia: Albrecht & Co.
Regarding the epigraphical lemma and notes for journal entries as book. See #62
It will be given as BEFEO 36 (1936) with the link toward the Zotero entry online.
Please check the outputs and let me know if you are satisfied with it.
Best,

Dear all,
You seem to have settle in favor of a publication title displayed with an abbreviated form and its full expression only in tooltip.
For now, I have created a new CSL to add the abbreviation feature inside the Chicago author-date model. It allows me to export two versions for a bibliographic citation. Let’s take this journal article
->  Venkayya, V. 1896–1897. “Jain Rock-Inscriptions at Panchapandavamalai.” EI 4.14: 136–40. Is displayed in the html a the main text while Venkayya, V. 1896–1897. “Jain Rock-Inscriptions at Panchapandavamalai.” Epigraphia Indica 4.14: 136–40. is provided as a tooltip.
I can’t structure more precisely the data with the current system. Indeed, the citation is constructed through Zotero API so I don’t have enough anchor inside the result to apply the tooltip only on the journal title.
Another downside of this system is that only items recorded with the type Article Journal in Zotero can be exported as such. All the items you have filled in as book requires to set another system. Initially, that’s was the purposes for the attribute @rend="journal". So instead of using the attributes to discriminate, I set the new system on search for the content of @target starting with the string "bib:" and followed by 2 upper-case letters.
<bibl><ptr target="bib:NBG15_1877"/><citedRange unit="page">136-137</citedRange></bibl> will have NBG 15 (1877) as a main text and Notulen van de Algemeene en Bestuurs-vergaderingen van het Bataviaasch Genootschap van Kunsten en Wetenschappen: Deel XV — 1877. 1878. Batavia: Bruining. as the content of its tooltip.
If you wishes to provide an author
<bibl><author>Brandes</author><ptr target="bib:NBG24_1886"/></bibl> to obtain Brandes in NBG 24 (1886) as main text, the tooltip will contain Notulen van de Algemeene en Bestuurs-vergaderingen van het Bataviaasch Genootschap van Kunsten en Wetenschappen: Deel XXIV — 1886. 1887. Batavia: Albrecht & Co.
Regarding the epigraphical lemma and notes for journal entries as book. See #62
It will be given as BEFEO 36 (1936) with the link toward the Zotero entry online.
Please check the outputs and let me know if you are satisfied with it.
Best,

Dear Axelle,
Thanks! I will test on my Pallava files.
Let us discuss this together this Thursday at the office.

Dear Axelle,
Thanks! I will test on my Pallava files.
Let us discuss this together this Thursday at the office.


.Net: Bug: ToolCall issue

Describe the bug
Tool Calling parameters
To Reproduce
I use Ollama
 var kernel = Kernel.CreateBuilder()
            .AddOpenAIChatCompletion(
                modelId: modelId,
                apiKey: null,
                endpoint: new Uri(url))
            .Build();

I used this model
https://ollama.com/library/llama3-groq-tool-use
and Im doung function calling (for gpt4o this code works perfectly)
But I see this as text asnwer from model.
<tool_call> {“id”: 0, “name”: “createReservation-Answer”, “arguments”: {“amount”: 1, “roomName”: “Single Room”, “startDate”: “2024-07-21”, “endDate”: “2024-07-23”}} </tool_call>

and the function doesn't work.
Expected behavior
I expected my function will called.
Platform

OS: [Mac]
IDE: [Rider]
Language: [C#]
Source: [1.16.0]

this is Ollama issue


Extra games may have artificially high Balance scores

I submitted in the Extra category and chose to opt-out of ratings. As a result, my balance rating keeps going up: as I can rate and review others' entries, but nobody can give a rating to my own. While it has been great for me personally, this seems like unintended behavior.

Yes unintended, but also wrong repository: https://github.com/JammerCore/JammerCore


[$250] Search - System message thread opens when opening from Search when system message has reply

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: 9.0-3.0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/43742
Action Performed:

Go to staging.new.expensify.com/
Go to FAB > Pay someone
Enter amount, select a user and pay them
Go to transaction thrad
Right click on the paid system message > Reply in thread
Send a reply in thread
Go to Search
Open the expense sent previously

Expected Result:
Transaction thread will open
Actual Result:
System message thread opens when opening from Search when there is reply in system message
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/582b143e-977d-4537-8168-3e2bf1c47b26
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019c1a520581844a96
Upwork Job ID:<PHONE_NUMBER>827245079
Last Price Increase: 2024-07-22

Issue OwnerCurrent Issue Owner: @getusha

@hayata-suenaga  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

This is not a blocker, but it's kind of an expected result of the one-transaction view design. In essence, an IOU action thread is a transaction thread, so replying to the report action in the thread is leaving a comment on the transaction thread.
I think this is not a bug @lanitochka17
@NikkiWines @trjExpensify to confirm, I am not sure what else we could do about this, but it's an edge case for Send Payment flow

Thread for discussion breadcrumbs: https://expensify.slack.com/archives/C01GTK53T8Q/p1719535523859119

forgot to add the external label 😓

still waiting for proposals

still waiting for proposals

still looking for porposals

Not overdue

@hayata-suenaga i am not able to see the paid system message after paying, could you check this?

I think we made a change about this. @mountiny, could you confirm? 😄

Hmmmm I am not sure about this, I think yous should still be able to see the paid report actions

https://github.com/Expensify/App/issues/45203 ?

Seems like we decided not to show it in this PR https://github.com/Expensify/App/pull/44529/files @puneetlath was there some discussion about this in Slack that we would hide the "paid" report action in the single transaction threads?

Hmm, I don't think that was intended. That PR was just meant to be about removing the flickering effect, not hiding something. My understanding was the flickering effect was from something being rendered, but then removed. But I might have misunderstood. In any case, I think we definitely don't want what is happening in this issue, where we open the thread instead of the report itself.

Got it, sounds good! We will go ahead with re-adding the system message when paying a single action send money transaction in https://github.com/Expensify/App/issues/45203

Not overdue, i think this should be held for https://github.com/Expensify/App/issues/45203 @joekaufmanexpensify

Sounds good! Will do that.

Still held on this issue

@getusha https://github.com/Expensify/App/issues/45203 is closed now. Should we take this off hold now?

Issue not reproducible during KI retests. (First week)

Closing this as it's no longer reproducible.


Add data directory excluding sessions to dump (#586)

This includes the data directory inside a gitea dump. Sessions are excluded if the session provider is set to file and the session directory is a child of the data directory.
Closes #586

Works now. The issue was an unset PROVIDER_CONFIG in custom app.ini. Then the configuration value is just empty and macaron uses a standard data/sessions. Thats another bug that should be resolved, since the expected behavior should session path to be relative to APP_DATA_PATH and not to the executable path. If APP_DATA_PATH is set to another location but PROVIDER_CONFIG is not set, sessions will be still saved to data/sessions.
I've hardcoded data/sessions to match macarons fallback config var as a workaround and it works for now.

LGTM

@bkcsoft Fixed, much cleaner now ;)

LGTM

We have to merge this at first. @bkcsoft if you have any further information, please open another issue.


Kaldi Android: MaceComputer's nnet outpus isn't the same as Kaldi NnetComputer

Model deploy file (*.yml)
......

Describe the problem
Hi, i am using the mace to build some kaldi project in android platform, but it turns out that the result is quite different from that in linux platform without mace.
The debug steps i had done is:

Check the input wav pcm data and feature pipleline in both mace based kaldi project in android and kaldi project in linux, and it's right.
The converted mace model weight, and it's the same as the kaldi model weight.
Use the offical built mace model and the offical build kaldi asr project code, and it's right. The decode result is the same as the text.

The information i want to know:

Why the MaceComputer's nnet outputs isn't the same as Kaldi NnetComputer?

given the same model and config, the same input feature and config

I also try using the offical build kaldi asr project code except using my feature config and model, but the MaceComputer's nnet outputs is also the same as the mace based kaldi project i write. That may mean the way i combining the mace and kaldi is right, but the nnet output is still quite different from the Kaldi NnetComputer.

Thanks.
The detailed log:

the input feature and nnet output

- here i only print some frames and dimensions to better clarify this issue.
...
Matrix<BaseFloat> output;
computer_->GetOutput(&output);
// debug
KALDI_LOG << "nnet out";
for(size_t i = 0; i < 20; ++i){
  KALDI_LOG << output(15, i);
}
...
- The mace based kaldi project's input feature (from input_features_->GetFrame())
AdvanceChunk():mace-decodable-online-looped.cc:132) input feature
AdvanceChunk():mace-decodable-online-looped.cc:137) 1.66497
AdvanceChunk():mace-decodable-online-looped.cc:137) 0.298115
AdvanceChunk():mace-decodable-online-looped.cc:137) 3.35621
AdvanceChunk():mace-decodable-online-looped.cc:177) nnet out
AdvanceChunk():mace-decodable-online-looped.cc:179) -3.32336
AdvanceChunk():mace-decodable-online-looped.cc:179) 1.12159
AdvanceChunk():mace-decodable-online-looped.cc:179) 1.67782
- The linux kaldi project 's input feature (from input_features_->GetFrame())
AdvanceChunk():decodable-online-looped.cc:173) input feature
AdvanceChunk():decodable-online-looped.cc:178) 1.66497
AdvanceChunk():decodable-online-looped.cc:178) 0.298116
AdvanceChunk():decodable-online-looped.cc:178) 3.35621
AdvanceChunk():decodable-online-looped.cc:232) nnet out
AdvanceChunk():decodable-online-looped.cc:234) -2.92612
AdvanceChunk():decodable-online-looped.cc:234) 0.332591
AdvanceChunk():decodable-online-looped.cc:234) 0.31575

the model weight

- the mace version  model weight
  ...
  data_type: DT_HALF
  float_data: 2.63805103302
  float_data: 1.23130202293
  float_data: -0.0862819701433
  float_data: 0.010600999929
  float_data: -0.541815519333
  float_data: -0.247513398528
  float_data: -0.371077686548
  ...
- the kaldi version model weight
...
2.638051 1.231302 -0.08628197 0.010601 -0.5418155 -0.2475134 -0.3710777 ...
...

To Reproduce
Steps to reproduce the problem:
1. cd /path/to/mace
2. python tools/converter.py convert --config_file=/path/to/your/model_deployment_file

Error information / logs
Please include the full log and/or traceback here.
LOGs

Additional context
Add any other context about the problem here, e.g., what you have modified about the code.

@groovemaxRong Sorry for the late reply.
This problem need debug, perhaps you can first debug the correctness between mace and onnx, you can refer to this document.

@lu229 Okay, i get it  and thanks for your reply.

@lu229 Hi, i follow your instrution to validate the mace model. But the python tools/converter.py run --config=${CONF_FILE} --validate --layers 0:-1 did't output the validate file in build/your_model/model/runtime_in_yaml/log.csv.
And i want to know how to get the log.csv file so i can check the acc.
The debug steps i had done is:

validate the fake test input data
the test data generate by /path/to/kaldi-onnx/tools/generate_inputs.py has a right nnet output by nnet3-compute
and the data does exist in the storage dir
check the yml file

subgraphs:
...
validation_inputs_data:
- $datadir/test_input.npy
validation_outputs_data:
- $datadir/kaldi_output.npy
...

try --validate | --validate --layers 0:-1, and the command parameter behind python tools/converter.py is 'run', not 'convert'
the command is run successfully and i can get the weight and .pb file

mace/tools/converter.py:339: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  configs = yaml.load(f)
* Package libs for first_nnet
Start packaging 'test' libs into build/test/libmace_first_nnet.tar.gz
build/test/model/
build/test/model/test.data
build/test/model/test.pb
Packaging Done!

----------------------------------------------------------------------
                               Library
----------------------------------------------------------------------
|          key           |                   value                   |
======================================================================
| MACE Model package Path| build/test/libmace_test.tar.gz|

@groovemaxRong Please plugin phone with your computer, and don't specify the socs in yml file,

@lu229 Thanks for your reply, and i'm using the mace way to debug the model, but it fails with some problems.
The debug steps i had done is:

use the whole model to run the debug code, and it run successfully but the mace output is not right.
use the kaldi nnet-am-copy edit to modify the output from the input side, and the kaldi nnet-compute can process well, but the mace output is empty with some check failed information, which is comfusing.

To Reproduce
I have packed the model and test script in the zip file.  I converted and run your model and found no problems.
the commands are:
python tools/converter.py convert --config=${CONF_FILE}
python tools/converter.py run --config=${CONF_FILE} --validate
The result is:
Pull /data/local/tmp/mace_run/model_out_output_affine to build/first_nnet/_tmp/first_nnet/628dd01a507a63fc2e6b02511b9adae2/Andromeda_msmnile/armeabi-v7a
output.affine MACE VS ONNX similarity: 0.999999952023911 , sqnr: 10115947.350050507 , pixel_accuracy: 1.0

      Similarity Test Passed

@lu229 Okay, thanks for your reply. I find the bug is my kaldi-onnx code may be changed during my debug process and now the test result is right. And thanks again for your help.


Unable to reproduce Llama 3 8b perplexity

When I run the fp16 Llama-3 8b model using the evaluation framework provided I am unable to reproduce the perplexities reported in the PV-tuning paper. I get a perplexity of 5.7559 for Wikitext2 (reported 5.54) and a perplexity of 8.3228 (reported 7.10). The command I used was:
export CUDA_VISIBLE_DEVICES=0
python main.py meta-llama/Meta-Llama-3-70B c4 --no_quant

And I added one 2 lines in main.py after lines 892, to put the model onto the device
else:
    model.to(args.devices[0])

I created a new conda enviroment and installed the packages per requirements.txt. Is there something I am missing to be able to reproduce the fp16 perplexities?

Default model_seqlen argument is 4096. For Llama3-8b we used 8192. Try running eval command like this:
python main.py meta-llama/Meta-Llama-3-70B c4 --no_quant --model_seqlen 8192


Conan 2: CMake output is not streamed

You only get to see the output once it's finished.

This is due to this in conans/util/runners.py because pipes are being used
        try:
            proc = subprocess.Popen(command, shell=shell, stdout=out, stderr=err, cwd=cwd)
        except Exception as e:
            raise ConanException("Error while running cmd\nError: %s" % (str(e)))

        proc_stdout, proc_stderr = proc.communicate()
        # If the output is piped, like user provided a StringIO or testing, the communicate
        # will capture and return something when thing finished


Terminal size in modal/dialog possible regression starting from 2.2.2

Hi there, I think we've found a regression when trying to render terminals in modals / dialogs starting from xterm 2.2.2. I know I'm referring to a super old version, but during the weekend we updated our old xterm 2.2.2 to the new 2.8.1 and we noticed that this suddenly stopped working.
Here are two codepen snippets to see the error:
https://codepen.io/anon/pen/weQqbE >> Terminal renders correctly per styles in dialog using xterm 2.2.2
https://codepen.io/anon/pen/PjxENm >> Last xterm.js 2.8.1 version where you can see the error.
The only difference between the two codepens is just the corresponding xterm.js file

What's the error? Works fine in Chrome v59 for me.

@Tyriar I accidentally updated the code with a workaround for the master version. Try again now with the second link.
This is what I get:

The rows are not being sized correctly:

Yup, realized that. Wondering why it works in 2.2.2 but not in 2.8.1 :cry:

I think it's related to how the popup dialog is being shown/hidden. Can you run xterm.open when you're 100% sure it's visible?

yup, that works. If this works as expected you can go ahead and close the issue as I already found a workaround. I was just wondering how it worked in 2.2 :)

Yeah I think this is expected now as it's a result of the more robust character measurement changes that happened some time ago. I think the root cause is that before the character would be measures by appending an element to body, now it's enclosed within the terminal container so .open needs to be on an element that's visible .

@marcosnils FYI https://github.com/sourcelair/xterm.js/issues/785

@Tyriar IIUC you're planning to add a fix for this?

@marcosnils it may fix this, you may need to call resize again though.


Subsequent setText calls don't display images.

Hello - I have found that after calling setText on the same DraweeTextView twice, images no longer get displayed.
This is observed in places where re-use happens like a RecyclerView.
An example that show this behavior:
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        final TextView textView = (TextView) findViewById(R.id.text_view);

        textView.setText(getSampleText());

        new Handler().postDelayed(new Runnable() {
            public void run() {
                textView.setText(getSampleText());
            }
        }, 5000);
    }

    private SpannableStringBuilder getSampleText() {
        int start;
        SpannableStringBuilder builder = new SpannableStringBuilder();
        builder.append("2333333\n");
        start = builder.length();
        builder.append("[emotion:tv_cheers]");
        DraweeSpan span = new DraweeSpan.Builder("http://static.yo9.com/web/emotions/tv_cheers.png").build();
        builder.setSpan(span, start, builder.length(), Spanned.SPAN_EXCLUSIVE_EXCLUSIVE);
        builder.append("bilibili- ( ゜- ゜)つロ 乾杯~\n");

        return builder;
    }

The second setText will no longer show the image.
PS: This is a great library, thanks for making it available :)

I have not run into this problem in using RecyclerView, calling setText() in ViewHolder's onBind()

The above code will reproduce the bug when run, I can setup a demo project to make it easier to test.  In this case I just added it to an empty android project in the default activity.

Thanks for your report. This bug has been fixed.


Update config-cheat-sheet.en-us.md

Add more information on configuring URI hyperlink rendering for Markdown.
Partially addressing  #8478

Codecov Report

Merging #8497 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8497      +/-   ##
==========================================
- Coverage   41.89%   41.75%   -0.15%
==========================================
  Files         507      511       +4
  Lines       66506    66866     +360
==========================================
+ Hits        27865    27921      +56
- Misses      35063    35365     +302
- Partials     3578     3580       +2

Impacted Files
Coverage Δ

models/release.go
50.22% <0%> (-2.39%)
:arrow_down:

modules/migrations/gitea.go
11.75% <0%> (-1.79%)
:arrow_down:

models/issue_comment.go
49.53% <0%> (-1.34%)
:arrow_down:

modules/migrations/github.go
83.41% <0%> (-0.44%)
:arrow_down:

models/issue.go
50.94% <0%> (-0.38%)
:arrow_down:

routers/user/auth.go
11.94% <0%> (-0.07%)
:arrow_down:

modules/setting/cron.go
25% <0%> (ø)
:arrow_up:

models/repo.go
47.89% <0%> (ø)
:arrow_up:

models/migrations/migrations.go
1.3% <0%> (ø)
:arrow_up:

models/external_login_user.go
23.8% <0%> (ø)
:arrow_up:

... and 8 more

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 ba201aa...f14fded. Read the comment docs.

Build failing due to #8474 unrelated to this PR.

Make lg-tm work


fix: Unable to create a project

Description
Trying to create a new project and it throws an error of failure as shown below in the screenshot section:
Steps To Reproduce
Running the script on the terminal to create a project.
Current Behavior
Error while running the command to create a project: very_good create weather_app --org-name "com.example.www"
Screenshots

Expected Behavior
It should create a project.
Additional Context
Flutter version:
Flutter 2.10.0-0.3.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision fdd0af78bb (13 days ago) • 2022-01-25 22:01:33 -0600
Engine • revision 5ac30ef0c7
Tools • Dart 2.16.0 (build 2.16.0-134.5.beta) • DevTools 2.9.2

Is it not possible to use the beta channel to create a project from CLI?

Is it not possible to use the beta channel to create a project from CLI?

Yes it is, but it looks like you are on an older version of the beta channel. Can you try upgrading and then try again?
flutter upgrade

I'm already on the latest version:
mmateen@XIN717584 testapps % flutter --version
Flutter 2.10.0-0.3.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision fdd0af78bb (2 weeks ago) • 2022-01-25 22:01:33 -0600
Engine • revision 5ac30ef0c7
Tools • Dart 2.16.0 (build 2.16.0-134.5.beta) • DevTools 2.9.2
mmateen@XIN717584 testapps %

@mateenatupday is this still an issue?

Hi @felangel, it's fixed now. Thank you.


Fix deprecation notice

/.../jchart/templatetags/jchart.py:5: RemovedInDjango20Warning: Importing from django.core.urlresolvers is deprecated in favor of django.urls.
  from django.core.urlresolvers import reverse

Coverage remained the same at 95.323% when pulling ac69ad8cf9754de088282084481321f9d031ed75 on intiocean:patch-1 into d147061bbd158a62f8b240c60e76e6d9bee0e40b on matthisk:master.

Hi, thank you for this contribution. It looks like some tests are failing for this PR. Do not see directly why this is, but it probably has to do with the fact that django.urls is not available in older version of Django. We would have to check the docs to see which version of Django introduced django.urls and fall back to django.core.urlresolvers for older versions.

Yay for tests! 😄
Updated - hopefully, they'll all pass now!

Coverage increased (+0.03%) to 95.354% when pulling 8267597b350c2c241a0f491e031c4042a83014c0 on intiocean:patch-1 into d147061bbd158a62f8b240c60e76e6d9bee0e40b on matthisk:master.

Thanks! Will try to perform a patch release in the near future.


Numeric overflow, tgamma

Summary:
Description:
I get the following
('INFO: Initial parameters', {'mu': 0.5, 'phi': 1, 'w_0': 0.69091931142090446, 'w': array([-0.27052855,  0.44121676, -0.23313663,  0.88146095,  0.1719369 ,
       -0.13115881, -0.0343557 , -0.65111515,  0.6491713 ,  0.53574831,
        0.34756648, -0.5797055 , -0.1997476 ,  0.1625951 , -0.1215    ,
        0.13889288, -0.2295226 , -0.70625209,  0.15884236,  0.39814814,
       -0.20826035,  0.29179666,  0.07323041, -0.28598118,  0.35788681,
       -0.45399123, -0.20960651,  0.06401133, -0.2308089 ,  0.24958945,
        0.90866274,  0.56040227,  0.52527381, -0.31835018,  0.54224039,
       -0.05974072, -0.09734139,  0.28207122,  0.7408952 , -0.02405463,
       -0.13910324,  0.10505472,  0.57475112, -0.40856197, -0.65439175,
        0.38553316, -0.35704467, -0.3536128 , -0.21973083, -0.20005977,
       -0.98622562,  0.0452479 ,  0.56066616,  0.55866922, -0.54499827,
        0.29682395,  0.00178766, -0.25977359,  0.19859709,  0.11461594,
        0.43245803,  0.09540582, -0.39263792, -0.10752373, -0.18090921,
       -0.66931966,  0.53382177, -0.25027437,  0.05350934, -0.72707792,
       -0.25122851,  0.20410126, -0.49383767,  0.04720082,  0.14122235,
        1.02608929, -0.15801633, -0.36897585,  0.30015339,  0.32536003,
        0.36998763, -0.25971182,  0.65974291, -0.46843143,  0.49561352,
       -0.04076968, -0.47561484, -0.45428351, -0.28851053, -0.09470851,
       -0.05752959, -0.71429534, -0.05248937, -0.53718625, -0.08136444,
        0.29121389, -0.52904384,  1.17270693,  0.5830163 ,  0.10500311,
       -0.33543509,  0.33968624,  0.39203134,  0.34695858,  0.18946337,
        0.40447077,  0.06844752,  0.75407845,  0.67607977, -0.4774629 ,
       -1.49187844, -1.09431834,  0.19460568,  0.10358412,  0.56036278,
       -0.35100866,  0.43019496, -0.30483862, -0.1053986 , -0.10937985,
       -0.1757629 , -0.52089926,  0.07407917, -0.0123624 ,  0.65450229,
       -0.21042264, -0.1554392 , -0.08431448,  0.17324623, -0.30444665,
        0.49694841, -0.68790722, -0.04641018,  0.55836847, -0.02020365,
        0.37294724,  0.02230654, -0.54061294,  0.13946673,  0.05383285,
        0.14503098,  0.28910242, -0.02209033,  0.15482229,  0.17341048,
       -0.12097925, -0.10156593, -0.88715638,  0.56543924, -0.32493027,
       -0.50438666, -0.45319082, -0.19225265,  0.12725021, -0.22586099,
        0.50104368,  0.27847687, -0.6449265 ,  0.43306655, -0.15190904,
       -0.79205534,  0.54980264,  0.35648734, -0.26078631,  0.70630172,
       -0.13703878, -0.78776348, -0.51106589, -1.07989062, -0.01767688,
        0.33610433,  0.30478381, -0.56005384,  0.72874208,  0.51151234,
       -0.22587442,  0.78917607, -0.61349265,  0.61782415,  0.11982305,
        0.6886022 , -0.1160628 , -0.13232534, -0.35537053,  0.63218095,
       -0.75187122,  0.37333861, -0.16442908, -0.23032689,  0.30354159,
        0.64103289, -0.71608173,  0.40087834,  0.28212035, -0.17129152,
        0.30147572, -0.43807175,  0.15008304,  1.02975868,  0.05971786,
        0.15891724,  0.72823555,  0.04174271, -0.09252343, -0.24688664,
       -0.24991356,  0.03926981,  0.40278358,  0.13980294,  0.15514416,
       -0.27996778,  0.05526526,  0.31514271, -0.01949379, -0.35298036,
       -0.25511445, -0.01539271, -0.04945689,  0.41982199, -0.68251022,
       -0.94827009, -0.03146817,  0.8139154 ,  0.07216416, -0.07586817,
        0.03648963,  1.03343417, -0.30914476,  0.10646841,  0.31749203,
       -0.55588963,  0.03881907,  0.26067885, -0.22743521, -0.11721607,
        0.07830396, -0.78188448,  0.37067349, -0.08065934, -0.05332349,
        0.485692  ,  0.44556641,  0.07613795, -0.45101174,  0.19181284,
       -0.5259726 , -0.11571175,  0.19393979,  0.34773642, -0.545569  ,
        0.46013049, -0.28266539, -0.26642973, -0.06214111,  0.26331269,
        0.64233009,  0.5948235 ,  0.58778781,  0.59356037, -0.50910321,
       -0.69088256, -0.32494297, -0.09768687,  0.50802146, -0.51165875,
        0.00901285,  0.9563969 ,  0.4675932 ])})
('INFO: Sampling with the following args:', {'n_jobs': 1, 'verbose': True, 'data': {'a': array([ 0.01062083,  0.00920438,  0.03228518,  0.04036716,  0.04710877,
        0.04327951,  0.04420888,  0.04589653,  0.02244839,  0.02770248,
        0.04279143,  0.05514997,  0.0395808 ,  0.00491052,  0.01032116,
        0.01297705,  0.01274098,  0.01041489,  0.00912119,  0.00825915,
        0.00911522,  0.01123634,  0.01547942,  0.01905974,  0.02077057,
        0.01655726,  0.01310074,  0.00595009,  0.00595009,  0.00638325,
        0.00614286,  0.00856103,  0.00981466,  0.00981466,  0.00613726,
        0.00746883,  0.00748989,  0.008687  ,  0.00830566,  0.00830566,
        0.00856036,  0.00369409,  0.00369409,  0.00370845,  0.00963354,
        0.01168769,  0.0105434 ,  0.00369409,  0.00414645,  0.00364338,
        0.00364338,  0.01384243,  0.01057491,  0.00680886,  0.00484254,
        0.00322747,  0.00360405,  0.00322747,  0.00387516,  0.00387516,
        0.00387516,  0.00387516,  0.00387516,  0.00387516,  0.00387516,
        0.00360405,  0.00364871,  0.00364871,  0.00364871,  0.00322747,
        0.00322747,  0.00438916,  0.00507643,  0.00412569,  0.00384472,
        0.00707275,  0.00707275,  0.00867561,  0.00845956,  0.00479252,
        0.0040456 ,  0.0040456 ,  0.00512863,  0.00510809,  0.00685027,
        0.00685027,  0.01115808,  0.01278088,  0.01656005,  0.01803778,
        0.02076805,  0.01825713,  0.01133701,  0.0103723 ,  0.01185065,
        0.00718363,  0.00713854,  0.00591808,  0.00388626,  0.00388626,
        0.00322747,  0.00709227,  0.00657202,  0.00715113,  0.00347538,
        0.00347538,  0.00397478,  0.00479568,  0.00479568,  0.00472923,
        0.00416253,  0.00384472,  0.00471685,  0.00501122,  0.00360405,
        0.00322747,  0.00529908,  0.00470202,  0.00534262,  0.00534262,
        0.00987709,  0.00748526,  0.00695678,  0.00583097,  0.00583097,
        0.00638325,  0.00581668,  0.00506592,  0.00514003,  0.01581695,
        0.01165081,  0.01165081,  0.00785325,  0.00785325,  0.00662826,
        0.00582123,  0.00725653,  0.00760497,  0.00760497,  0.01220468,
        0.01220468,  0.01488045,  0.01436035,  0.01211891,  0.00472923,
        0.00799616,  0.00659534,  0.00697323,  0.00697323,  0.00759591,
        0.00756935,  0.00659534,  0.00415095,  0.00456318,  0.00456318,
        0.00415095,  0.00503671,  0.00415095,  0.00467479,  0.00494517,
        0.00494517,  0.00494517,  0.00494517,  0.00494517,  0.00494517,
        0.0050822 ,  0.00556835,  0.00668076,  0.0137375 ,  0.01389175,
        0.01389175,  0.01418248,  0.01433393,  0.01296223,  0.03146598,
        0.03936312,  0.03727928,  0.03727928,  0.00982069,  0.01195713,
        0.01746341,  0.01501806,  0.01420386,  0.01393421,  0.01393421,
        0.00922547,  0.00618316,  0.00831198,  0.00831198,  0.00947366,
        0.00764093,  0.00692903,  0.0043928 ,  0.00364338,  0.00364338,
        0.00369409,  0.00369409,  0.00412569,  0.00607089,  0.00811352,
        0.00719985,  0.00716047,  0.0059275 ,  0.00716147,  0.00628075,
        0.00749198,  0.00662545,  0.00634642,  0.01172086,  0.01747172,
        0.01049894,  0.00584794,  0.00584794,  0.00494517,  0.00635531,
        0.00635531,  0.00635531,  0.00922528,  0.01267093,  0.02580889,
        0.03458147,  0.02848663,  0.00587979,  0.00506699,  0.00506699,
        0.00347538,  0.00706953,  0.00732246,  0.00639505,  0.006249  ,
        0.00613267,  0.00501122,  0.00461696,  0.00501122,  0.00559165,
        0.00723431,  0.00772963,  0.00867561,  0.01096037,  0.01675656,
        0.01307625,  0.01354239,  0.0147082 ,  0.00916072,  0.00384472,
        0.00384472,  0.00322747,  0.00322747,  0.00347538,  0.00347538,
        0.00322747,  0.00322747,  0.00364871,  0.00533147,  0.00347538,
        0.00347538,  0.00322747,  0.00322747,  0.00322747,  0.00322747,
        0.00322747,  0.00322747,  0.00322747,  0.00360405,  0.00412569,
        0.00471685,  0.00470202,  0.00552461,  0.00940612,  0.00940612,
        0.01199075,  0.00987365,  0.00695446,  0.00695446,  0.00366906,
        0.00937997,  0.01202299,  0.01202299,  0.01202299,  0.01915683,
        0.01876063,  0.01469831,  0.016367  ,  0.016367  ,  0.00384472,
        0.00384472,  0.00369409,  0.00513082,  0.0176742 ,  0.01137985,
        0.0092722 ,  0.00963773,  0.01587638,  0.01903979,  0.01632011,
        0.00779347,  0.01156141,  0.0214587 ,  0.02730376,  0.03285507,
        0.02437772,  0.02740781,  0.02051429,  0.01782488,  0.01848238,
        0.01848238,  0.02875188,  0.02875188,  0.02991737,  0.02695712,
        0.02428545,  0.0206146 ,  0.0206146 ,  0.01524124,  0.00826091,
        0.00692806,  0.00692806,  0.00537412,  0.00556835,  0.00556835,
        0.0050822 ,  0.00419148,  0.00928379,  0.01015865,  0.00928379,
        0.00791169,  0.00822767,  0.01029957,  0.01723299,  0.04708352,
        0.02143951,  0.00472923,  0.00490579,  0.00478565,  0.00468712,
        0.00472692,  0.00416253,  0.00384472,  0.0051242 ,  0.0051242 ,
        0.00600382,  0.00600382,  0.01942654,  0.02515779,  0.03410968,
        0.02795266,  0.02881476]), 'phi_max': 1000, 'E': array([[0, 0, 0, ..., 0, 0, 0],
       [0, 0, 0, ..., 0, 0, 0],
       [0, 0, 0, ..., 0, 0, 0],
       ...,
       [0, 0, 0, ..., 0, 0, 0],
       [0, 0, 0, ..., 0, 0, 0],
       [0, 0, 0, ..., 0, 0, 0]]), 'phi_0': 2, 'pi_0': 0.01, 'k': 268, 'mu_0': 0.1, 'v': 347}, 'iter': 8, 'init': <function <lambda> at 0x7f29a69e5758>, 'chains': 2}, array([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]))
WARNING: No variance estimation is
         performed for num_warmup < 20

Iteration: 1 / 8 [ 12%]  (Warmup) (Chain 0)
Iteration: 2 / 8 [ 25%]  (Warmup) (Chain 0)
Iteration: 3 / 8 [ 37%]  (Warmup) (Chain 0)
Iteration: 4 / 8 [ 50%]  (Warmup) (Chain 0)
Iteration: 5 / 8 [ 62%]  (Sampling) (Chain 0)
Iteration: 6 / 8 [ 75%]  (Sampling) (Chain 0)
Iteration: 7 / 8 [ 87%]  (Sampling) (Chain 0)
Iteration: 8 / 8 [100%]  (Sampling) (Chain 0)
#
#  Elapsed Time: 0.012279 seconds (Warm-up)
#                0.005695 seconds (Sampling)
#                0.017974 seconds (Total)
#
WARNING: No variance estimation is
         performed for num_warmup < 20

Iteration: 1 / 8 [ 12%]  (Warmup) (Chain 1)
('DEBUG: The server encountered an unexpected exception: Exception thrown at line 60: Error in function boost::math::tgamma<double>(double): numeric overflow', OverflowError('Exception thrown at line 60: Error in function boost::math::tgamma<double>(double): numeric overflow',))

The model is
data {
  int<lower=1> v;
  int<lower=1> k;

  vector[v] a;
  matrix[v, k] E;

  real<lower=0> phi_max;  // Hyperparameter for "phi"
  real phi_0;             // Hyperparameter for "mu"
  real mu_0;              // Hyperparameter for "mu"
  real pi_0;              // Hyperparameter for "w_0"
}

parameters {
  real<lower=0, upper=1> mu;
  real<lower=0, upper=phi_max> phi;

  vector[k] w;
  real w_0;
}

transformed parameters {
  real alpha;
  real beta_;
  vector[v] pi;

  // Get the rate and shape from "mu" and "phi" for the beta distribution later
  alpha = phi * mu;
  beta_ = phi - alpha;

  pi = 1 ./ (1 + exp(-(E * w + w_0)));
}

model {
  phi ~ uniform(0, phi_max);
  mu ~ beta(mu_0, phi_0);

  a ~ beta(alpha, beta_);

  w ~ normal(0, 1);
  w_0 ~ normal(logit(pi_0), 1);
}

generated quantities {
  vector[v] c;
  vector[v] likelihoods;
  real normalization_constant;
  real constant;

  normalization_constant = 0;
  constant = (tgamma(phi) / (tgamma(mu * phi) * tgamma((1 - mu) * phi)));
  for (i in 1:v) {
    likelihoods[i] = constant * pow(a[i], (mu * phi - 1)) * pow((1 - a[i]), ((1 - mu) * phi - 1));
    c[i] = likelihoods[i] * pi[i];
    normalization_constant = normalization_constant + c[i];
  }
}

Python 2.7.6
>>> pystan.__version__
'<IP_ADDRESS>'

Reproducible Steps:
See above
Current Output:
See above
Expected Output:
I do not really have an expected output, except that the inference should not exit with Error in function boost::math::tgamma<double>(double): numeric overflow
Additional Information:
Possibly related: (https://github.com/stan-dev/stan/issues/476)
Current Version:
v2.14.0

@ariddell Is this enough to run PyStan from? I haven't tried running PyStan (mostly been doing CmdStan)..

Note that this does not deterministically exit with numeric overflow, perhaps due to the randomness of sampling. I cannot get the actual parameter values that cause the overflow because the process just terminates, but the input data is certainly the same across runs

Thanks for adding the details.
On Wed, Mar 22, 2017, 2:38 PM Peter HT Nguyen<EMAIL_ADDRESS>wrote:

Note that this does not deterministically exit with numeric overflow,
perhaps due to the randomness of sampling. I cannot get the actual
parameter values that cause the overflow because the process just
terminates, but the input data is certainly the same across runs
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/stan/issues/2255#issuecomment-288498494, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfA6SaQNvTvUGbtllTNJUsJLNG3_A9Zks5roWq_gaJpZM4MlkPD
.

Thank you for investigating! Please let me know what you find, or if I am doing something wrong on my end.

Can you fix a random seed when you call sampling? I imagine we would need that seed and the data to reproduce. You can also recover the random seed used from the depths of the fit object, but this isn't documented well.
But perhaps this is the expected behavior? If you call tgamma with a value which causes a numeric overflow, shouldn't it raise an exception?

If it happens during sampling it should reject but shouldn't we just return
something sensible on overflow in generated quantities? I think the design
around that was discussed for the binomial RNG recently.
On Wed, Mar 22, 2017, 4:09 PM Allen Riddell<EMAIL_ADDRESS>wrote:

Can you fix a random seed when you call sampling? I imagine we would need
that seed and the data to reproduce. You can also recover the random seed
used from the depths of the fit object, but this isn't documented well.
But perhaps this is the expected behavior? If you call tgamma with a value
which causes a numeric overflow, shouldn't it raise an exception?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/stan/issues/2255#issuecomment-288523368, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfA6ZY91w5yZR8d05OB3-FILC8LSeXRks5roX_6gaJpZM4MlkPD
.

I followed the exception behavior tgamma in C99 (and now in the std:: namespace in C++11).  My reasoning was that programs that don't throw exceptions in basic C++ shouldn't throw exceptions when plugging in autodiff types or you might get excepting programs with autodiff types that would complete with double (because you threw away the NaN or infinite result).  See:
http://www.cplusplus.com/reference/cmath/tgamma/
We've talked about changing that behavior to have everything except early and often when arguments are out of bound.  It'd need a pass through all the functions to make sure we do this.  Then the question arises as to whether we want to do the same thing with the double versions, which means dropping them into our own namespace and being careful with qualifications.

@ariddell: How do I go about "fixing a random seed"?
What will cause "tgamma" to overflow? For example, do the specified inputs above necessarily elicit an overflow? I also notice from (https://github.com/stan-dev/stan/issues/476) that the sample should just be rejected or something?

@ptn24 Does your model run if you remove the generated quantities block?  If the tgamma call overflows in the density calculation (model and transformed parameters blocks) it will reject but the generated quantities block is not part of that so it doesn't reject.  You can always do the generated quantities calculation as a second step.
Negative integers or large values of x will cause overflow and since this is the Gamma function x need not be very large to cause it to overflow double values.  I think this issue is not going to get resolved in the near future because, as @bob-carpenter, mentions there is a slight design issue and making a change will require going through a lot of code.

@sakrejda: I lifted the gamma function evaluation into the application layer, and I notice that the overflow is because the variable "constant" (above) is "nan". Here is the stdout log
./utils.py:248: RuntimeWarning: invalid value encountered in double_scalars
  constant = (gamma(phi) / (gamma(mu * phi) * gamma((1 - mu) * phi)))
('DEBUG: The server encountered an unexpected exception: The constant term should be a number, but is nan (mu=0.03464174185, phi=502.890817548)', AssertionError('The constant term should be a number, but is nan (mu=0.03464174185, phi=502.890817548)',))

What is the best way to proceed? Is it mathematically valid to reject this sample and re-sample?

Nothing that happens in generated quantities affects sampling, but you
can't just throw those values out for inference.  It's probably Nan because
it's too big, so you need an alternative expression.
On Sun, Mar 26, 2017, 1:16 AM Peter HT Nguyen<EMAIL_ADDRESS>wrote:

@sakrejda https://github.com/sakrejda: I lifted the gamma function
evaluation into the application layer, and I notice that the overflow is
because the variable "constant" (above) is "nan". Here is the stdout log
./utils.py:248: RuntimeWarning: invalid value encountered in double_scalars
constant = (gamma(phi) / (gamma(mu * phi) * gamma((1 - mu) * phi)))
('DEBUG: The server encountered an unexpected exception: The constant term should be a number, but is nan (mu=0.03464174185, phi=502.890817548)', AssertionError('The constant term should be a number, but is nan (mu=0.03464174185, phi=502.890817548)',))
What is the best way to proceed? Is it mathematically valid to reject this
sample and re-sample?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/stan/issues/2255#issuecomment-289259261, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfA6ddklUny-srkYD0649XGfB_swhWbks5rpfTIgaJpZM4MlkPD
.

Also, tgamma is prone to overflow---you want to use lgamma
(on the log scale) wherever possible.

Bob

On Mar 26, 2017, at 6:36 AM, Krzysztof Sakrejda<EMAIL_ADDRESS>wrote:
Nothing that happens in generated quantities affects sampling, but you
can't just throw those values out for inference. It's probably Nan because
it's too big, so you need an alternative expression.
On Sun, Mar 26, 2017, 1:16 AM Peter HT Nguyen<EMAIL_ADDRESS>wrote:

@sakrejda https://github.com/sakrejda: I lifted the gamma function
evaluation into the application layer, and I notice that the overflow is
because the variable "constant" (above) is "nan". Here is the stdout log
./utils.py:248: RuntimeWarning: invalid value encountered in double_scalars
constant = (gamma(phi) / (gamma(mu * phi) * gamma((1 - mu) * phi)))
('DEBUG: The server encountered an unexpected exception: The constant term should be a number, but is nan (mu=0.03464174185, phi=502.890817548)', AssertionError('The constant term should be a number, but is nan (mu=0.03464174185, phi=502.890817548)',))
What is the best way to proceed? Is it mathematically valid to reject this
sample and re-sample?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/stan/issues/2255#issuecomment-289259261, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfA6ddklUny-srkYD0649XGfB_swhWbks5rpfTIgaJpZM4MlkPD
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I see. I switched to lgamma, and things seem to be working. Thank you!

I think this is a non-issue or a bigger design issue with how throws in generated quantities are handled.  Either way no straightforward path to a fix.  If you want to re-open solution should be discussed on dev list.

See #1943 for the reject in generated quantities issue.  Solving that issue will solve this one.


On hotfix: add pre-commit config to enforce conventional commits style

This pre-commit config file contains a git hook to enforce the conventional commits style. After you pull these changes locally and install pre-commit (via pip, conda, or brew), install this hook with pre-commit install. Then it will check your commit message every time you attempt to commit and throw an error if it doesn't follow conventional commits guidelines.
(I accidentally opened #35 against main instead of hotfix)

Looks very helpful! Tried locally using a conda env and worked for me


Update faq.yml

Update faq to reflect information from https://supportability.visualstudio.com/AzureIaaSIncubation/_wiki/wikis/BareMetal and Incubation/585012/Connection-from-On-prem-to-AVS-drops-intermittently-with-ASN-prepend
This is important for customers who want to connect on-prem environment to AVS and use private ASN in on-prem

#label:"aq-pr-triaged"


NPM out of sync with Github

'npm install snmp-native' installs a different version of the module compared to what is on Github.  I don't know how many differences there are, but there is one that causes problems for me:
/lib/snmp.js, line 801/807:
if (options.combinedTimeout) {
var combinedTimeoutEvent = function() {
combinedTimeoutExpired = true;
return callback(new Error('Timeout'), results);
};
combinedTimeoutTimer = setTimeout(combinedTimeoutEvent, options.combinedTimeout);
}
The results parameter in the callback should be vbs.  This was fixed back in January, so it makes me wonder how old the NPM code is.

I've pushed a 1.1.1

Perfect, thanks for the quick response!


Add unit test workflow for CI

What this PR does / why we need it:
We want to run unit tests.
Which issue(s) this PR fixes
Issue #https://github.com/rancher/gke-operator/issues/93
Special notes for your reviewer:
Checklist:

[ ] squashed commits into logical changes
[ ] includes documentation
[x] adds unit tests
[ ] adds or updates e2e tests
[x] backport needed

Checklist:

[ ]  squashed commits into logical changes
[ ]  includes documentation
[x]  adds unit tests
[ ]  adds or updates e2e tests
[x]  backport needed

Approving even if the checklist is not completed yet


winsshterm: Update to version 1.25.1 and fix manifest

Fix #10727

[x] I have read the Contributing Guide.

@niheaven Thank you for correcting my commit

/verify


Shorten the tab labels to be one line only in English and French

Shorten the tab labels to be one line only in English and French (at least) and add comment asking translators to be concise.

@teolemon It's already fixed. Issue can be closed

@teolemon It's already fixed. Issue can be closed

Fixed for French. Closing.


Fix bug in StringMatch where separator and slash collided

Fixes #10871.
cc @dangoor

Looks good to me, and I appreciate that you added a test too. I can't test it at the moment because I, uh, don't have a computer (I'm writing this on an iPad). Should I just merge it or does someone want to test it first?

I'd like to have a second pair of eyes look at those lines so I don't break anything at the end of the Release.
If it takes longer, feel free to merge this in at the beginning of Release 1.4. Thanks!

Since StringMatch is sort of delicate, I also think it'd be best if someone could take a little time to test before we merge.

Taking up the task of testing it thoroughly and reviewing.

Could we maybe merge this now that Release 1.4 has just begun so this has a long testing phase?

@MarcelGerber Changes look good and it fixes the issue it addresses. Scenario testings also went fine.
BUT on running the Jasmine Unit Tests, I see that couple of FindInFiles Integration Tests fail and makes Brackets non-responsive in the process. This is reproducible consistently. Can you please verify if it is introduced due to the changes OR the test case needs to be updated.
NB: Ignore QuickOpen Test failures(issue known and fixed in latest master branch).
We can have this PR merged after the test failures gets addressed.

@rroshan1 That was just an issue of this branch not being merged with latest master.
Now that I merged with master (and rebased), all unit tests pass.

Cool !!
I am going ahead with the merge then.


Bengali translation

I would like to add arabic, bengali and hindi languages

@RakibFiha Hi! Rakib, thank you very much for your willingness to help.
Can you please take a look at:
ar.wttr.in
hi.wttr.in
Does it look well to you?
What must be improved in this translations?

Moved Arabic and Hindi translations to separate issues: #429, #430


Exclude tabs by name or hide race reward tabs.

Race reward tabs can't be renamed and always end up as the first tab shown in Procurement.
If they can't be determined programmatically to be race reward tabs, I wouldn't mind adding them manually to my Settings.xml. (Similar to MyTabs section.)
At the moment one could list the tabs that you want specifically, but I'd like to have an option for download EVERYTHING except this tab instead.

I'll try get this improvement in for the next release, do you have a set of .bin files from your Procurement folder that you can send me which include race rewards? I haven't taken part in a race in quite a while, so don't have any race tabs myself to test with.


Review all list APIs and make then honour listall parameter

There are many list APIs that should ideally honour the listall parameter and when set to true should return all the items a caller/account is authorised to see. Due to a historic use of projectid=-1, most APIs do return all project only resources.
With PR https://github.com/apache/cloudstack/pull/3894 the behaviour has been addressed for some APIs such as listRouters etc where all resources are returned for root admin when both listall=true and projectid=-1 is passed. A review of all list APIs and fix to make the behaviour consistent is asked.
ISSUE TYPE

Bug Report
Improvement Request

COMPONENT NAME

API

CLOUDSTACK VERSION

4.13, 4.14+

@rhtyd Should this be fixed before <IP_ADDRESS> release ?

@weizhouapache I'll need to check if others APIs need to be supported, the changes were done solely to support infrastructure (VR etc) list views in Primate.

@rhtyd will you plan to work on this issue ?

@weizhouapache I'm not sure - it would depend on requirement coming from Primate issues/tasks

Ping @GabrielBrascher
This is what we ran into as well

in view of the changes in #3894 it would seem that no further work needs to be done, @rhtyd can you comment?
@wido @weizhouapache did you have anymore issues as a result of this?

in view of the changes in #3894 it would seem that no further work needs to be done, @rhtyd can you comment?
@wido @weizhouapache did you have anymore issues as a result of this?

@rhtyd @DaanHoogland do you think that PR #4276 should extend its goals to more APIs?
Or is it ok to do it "progressively" with different PRs addressing this issue?

@rhtyd @DaanHoogland do you think that PR #4276 should extend its goals to more APIs?
Or is it ok to do it "progressively" with different PRs addressing this issue?

@GabrielBrascher I don't mind progressively solving this but I have two worries:

this issue drags on for the longest while
consistant implementation is going to suffer from scattered work, and we and up with a new discussion on how listAPIs and the listall parameter should work

@GabrielBrascher I don't mind progressively solving this but I have two worries:

this issue drags on for the longest while
consistant implementation is going to suffer from scattered work, and we and up with a new discussion on how listAPIs and the listall parameter should work

cc @davidjumani does your related PR address this?

@rohityadavcloud  Nope but I can look into this if its priority for 4.17

cc @shwstppr

Closing this as several list APIs were improved to support listall or project=-1 param. It could risks the issue of API backward compatibility. I'm not going to work on this, abandoning this one. Others are welcome to raise PRs around this as/when needed.


Improve child delete confirmation flow

I have accidentally deleted child entries on more than one occasion. Need to come up with a way to make the confirmation of this more... dramatic. :rage4:

Closed by 492bcdd


fix small issues in typescript types

This PR simply adds the required generic to the Matcher definition and also declares that this repo is a module.
(the module type definition might not be completely accurate. If not, let me know and I'll fix).
Without these changes, typescript will complain and likely error (depending on your configuration).

Looks good to me (but not a heavy TS user)

Great! Thanks for the speedy help.
Tested locally and it works fine on my machine. 👍

Thank you for improving this lib! :)


Question: Is there a javascript API documentation?

Mocha seems to output its results only over CLI. Is there a way to use mocha programmatically without having to parse stdout for the results?

Hey @moglars! 🙂
The docs say:

For the JavaScript API, view The Source®.

…which, even with that nice and readable source code, I agree may not be the most newcomer-welcoming message. 😑
Other than that though, there is a quick-start wiki page: Using mocha programmatically. Maybe that would be helpful to get you up and running. 🤓

I looked into the code. The event system suits my needs:
var runner = mocha.run()
runner.on('pass', function(test) {console.log(test.title)})
runner.on('fail', function(test, err) {console.log(test.title,err.toString())})

The page https://github.com/mochajs/mocha/wiki/Using-mocha-programmatically doesn't mention that mocha.run() returns a Runner  instance that emits events of interest. I recommend adding a note about this in the wiki page.

If you are interested in writing your own reporter I suggest you take a look at the different already existing ones: https://github.com/mochajs/mocha/tree/master/lib/reporters

I recommend adding a note about this in the wiki page.

Documentation improvements are always welcome! 🤓 I have added it myself this time, but I bet you could do it yourself next time, and get your dent in The Open Source Universe®. 😎

yeah, the wiki needs love.  I'm not sure the "API" is really worth documenting though.  😦


Suggested pattern for a Consume,Transform,Produce use case

Description
Question:  Is there a suggested pattern or ideal way to consume, transform then republish the transformed objects to a new topic?  I don't want to drop a message either.
Would something like this be recommended?
// Configuration
_consumerConfig = new ConsumerConfig()
{
    BootstrapServers = "localhost:9092",
    GroupId = "mygroup-id,
    EnableAutoCommit = false,
    StatisticsIntervalMs = 10000,
    SessionTimeoutMs = 6000,
    AutoOffsetReset = AutoOffsetResetType.Latest,
    ClientId = "clientid",
};

// Create Producer
_producerConfig = new ProducerConfig
{
    BootstrapServers = "localhost:9092",
    Acks = -1,
};
_producer = new Producer<Null, string>(_producerConfig);
_consumer = new Consumer<Ignore, string>(_consumerConfig);

// Main Loop
void ConsumeTransformProduce()
{
    _consumer.Subscribe(new[]{"topic_A", "topic_B", "topic_C"});

    while(!_cancellationToken.IsCancellationRequested)
    {
        var msg = _consumer.Consume(_cancellationToken.Token);
        var topic = msg.Topic;
        var payload = msg.Value;

        if (topic == "topic_A")
        {
            var A1 = Transform_To_A1(payload);
            var A2 = Transform_To_A2(payload);
            _producer.BeginProduce("topic_xformed_A1", ToJson(A1));
            _producer.BeginProduce("topic_xformed_A2", ToJson(A2));
        } else if (topic == "topic_B")
        {
            var B1 = Transform_To_B1(payload);
            _producer.BeginProduce("topic_xformed_B1", ToJson(B1));
        }
        // else if (topic == "topic_C") ... etc

        _consumer.Commit(msg);

        // do we need to flush every time?
        _producer.Flush(TimeSpan.FromSeconds(10));
    }
    _consumer.Close();
}

How to reproduce
Checklist
Please provide the following information:

[x] Confluent.Kafka nuget version: <IP_ADDRESS>-beta2
[x] Apache Kafka version: kafka_2.11-2.1.0
[x] Client configuration:
[x] Operating system: Win10 Pro
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue: no

we plan to add an example + discussion of this. you can't get exactly once semantics yet because transactions are not implemented, but you can get at least once semantics by committing after you produce the new message.

Great! Looking forward to learning from the example and discussion.

https://github.com/confluentinc/confluent-kafka-dotnet/tree/master/examples/ExactlyOnce


update: Improve README examples of number/uint32/base namespace

Resolves # .
Description

What is the purpose of this pull request?

Improves README.md examples of @stdlib/number/uint32/base namespace directories.
Related Issues

Does this pull request have any related issues?
Issue Tracker: #1600

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[ ] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@Planeshifter, @kgryte  Issue Tracker : #1600. If there are any issues or files missing, kindly update regarding the same!

Thanks for your efforts, but this PR cannot land since it changes example code of individual packages (contrary to what the issue proposed) and does not align with project conventions.


feat: Introduce 'sanitizeKeys' config option

Resolves https://github.com/getsentry/raven-js/issues/296
It was quite requested feature by some of the clients recently, so we may get it in, in one form or another. I'm open for the feedback.

very cool! haven't reviewed the implementation in depth yet, but I think it's worth discussing the API/behavior/guarantees/terminology of this feature, because it could become the gold standard for our other clients going forward.
afaik (and I could be very wrong, @bretthoerner @HazAT ) the only clients which support this right now are python and ruby.
Python
docs:
https://docs.sentry.io/clients/python/advanced/#sanitizing-data
implementation:
https://github.com/getsentry/raven-python/blob/master/raven/processors.py

sanitizes a fixed list of keys (AFAIK, not configurable by the user without making a new instance of the processor and passing that in)
filters extra data and http data including query params

Ruby:
docs:
https://docs.sentry.io/clients/ruby/config/#configuration
implementation:
https://github.com/getsentry/raven-ruby/blob/master/lib/raven/processor/sanitizedata.rb

has some default patterns including password and credit card-like strings.
also accepts a few configs including regexes and predicates I think?
filters query strings & json blob

I'd prefer something simple like sanitizeKeys over the general processors pattern we implemented elsewhere. I think also supporting regexp is good.

The main issue I have with RegExp is that it can help people easily shoot themselves in the foot, as this sanitizer goes through all keys of the payload sent to Sentry. Including exception, message, frames, etc. etc.

The main issue I have with RegExp is that it can help people easily shoot themselves in the foot,
as this sanitizer goes through all keys of the payload sent to Sentry. Including exception, message, frames, etc. etc.

I can understand why this can be an example of "shooting yourself in the foot ", but many companies/organization want to scrub PII everywhere (frames, tags, extra information, error title/msg, etc.). Maybe a config that specifies to look through everything / etc.??

@dcramer added support for RegExp.
@MaxBittker handled cyclic references, updated docs and changed behavior to bail quickly with the input if incorrect keys type is used.


Add queryResponse to available plugin props

Fixes #1227
...
Checks

[x] Ran yarn test-build

Changes proposed in this pull request:

Add queryResponse to available plugin props

@imolorhe Can you give any insight on why the CI build failed?

Thank you @isaachvazquez for your PR! I'll release a new version of Altair soon. Hopefully you should be able to get your plugin to work as expected.


prefer-stateless-function triggers despite meeting all criteria with PureComponent

criteria:
When true the rule will ignore Components extending from React.PureComponent that use this.props or this.context.

[x] extends from PureComponent
[x] uses this.props

eslint-plugin-react version:
"version": "6.7.1"

eslint file:
module.exports = {
  parser: "babel-eslint",
  "parserOptions": {
    "ecmaFeatures": {
      "jsx": true
    }
  },
  plugins: [
    "babel",
    "react"
  ],
  env: {
    es6: true
  },
  rules: {
    "react/prefer-stateless-function": ["warn", {ignorePureComponents: true}]
}

Code:
import React, {PureComponent, PropTypes} from "react"
import {connect} from "react-redux"

import Layout from "../Layout"

export default connect()(class SignUp extends PureComponent {
  static propTypes = {
    dispatch: PropTypes.func.isRequired,
    form: PropTypes.shape({
      email: PropTypes.string.isRequired,
      password: PropTypes.string.isRequired
    }).isRequired
  }
  render () {
    const {form} = this.props

    return <Layout subtitle="Join our website!">

        <input type="email" value={form.email} />
        <input type="password" value={form.password} />

    </Layout>
  }
})

Error:

If you use React.PureComponent instead of the named import, does the same thing happen?

Yes.

If I remove the currying, it no longer warns me. It seems like it gets confused by the connect() currying.

Same thing happening here even with a simple component:

// package.json
"eslint": "^4.6.1",
"eslint-plugin-react": "^7.3.0",

// .eslintrc
{
  "extends": "airbnb",
  "parser": "babel-eslint",
  "rules": {
    "no-console": 0,
    "max-len": [
      1,
      120,
      2
    ],
    "no-param-reassign": [
      2,
      {
        "props": false
      }
    ],
    "no-continue": 0,
    "no-underscore-dangle": 0,
    "generator-star-spacing": 0,
    "class-methods-use-this": 0,
    "import/no-extraneous-dependencies": 0,
    "no-use-before-define": 0,
    "react/jsx-filename-extension": [1, { "extensions": [".js", ".jsx"] }],
    "react/prefer-stateless-function": [2, { "ignorePureComponents": true }]
  },
  "env": {
    "jest": true
  }
}

Strange. This case is supported and has tests for it: https://github.com/yannickcr/eslint-plugin-react/blob/master/tests/lib/rules/prefer-stateless-function.js#L42

@lucasbento are you setting your React version correctly in eslintrc? PureComponent doesn't exist before v15.2 or so

Seeing the same issue having just created a new project with create-react-native-app

I found some strange behavior with eslint 4.9.0 and react 16.0.0,:
import React, { PureComponent } from 'react';

// this component would cause error 'react/prefer-stateless-function'
class ArticleCard extends PureComponent {
    render() {
        return (

        );
    }
}

// if I add props reference in component render function, the error would disappear
class ArticleCard extends PureComponent {
    render() {
        const { id } = this.props;

        return (

        );
    }
}

// .eslintrc.json
{
  "env": {
    "browser": true,
    "commonjs": true,
    "es6": true
  },
  "extends": "airbnb",
  "rules": {
    "indent": ["error", 4],
    "linebreak-style": "off",
    "comma-dangle": "off",
    "arrow-body-style": "off",
    "no-trailing-spaces": ["error", {
      "skipBlankLines": true,
      "ignoreComments": true
    }],
    "function-paren-newline": "off",
    "object-curly-newline": ["error", {
      "multiline": true
    }],
    "global-require": "off",

    "react/jsx-indent": ["error", 4],
    "react/jsx-closing-bracket-location": ["error", "after-props"],
    "react/prefer-stateless-function": ["error", { "ignorePureComponents": true }],
    "jsx-a11y/anchor-is-valid": ["error", {
      "components": []
    }]
  }
}

// package.json
"react": "^16.0.0",
"react-dom": "^16.0.0",

"eslint": "4.9.0",
"eslint-config-airbnb": "^16.1.0",
"eslint-loader": "^1.9.0",
"eslint-plugin-import": "2.7.0",
"eslint-plugin-jsx-a11y": "6.0.2",
"eslint-plugin-react": "7.4.0",

Given that we have passing tests, I'm going to close this. If you're still seeing issues in the latest version, please file a new issue.


Add open source license for PicoLA

@OscarShiang , please add an open source license for your project. If you are not sure how to do it, see https://github.com/solvcon/modmesh.

@OscarShiang any update?
The repository is still empty.  According to your schedule, there should be Github Actions set up by now.  Please update your progress.

Sorry for the delayed response.
I have added a license file at #2.

Out of curiosity, should we add the license statement to each source file?

Sorry for the delayed response.
I have added a license file at #2.

This is your project.  Merge as you want.
If you would like to have my review, leave a comment in the PR tagging me.
But for a simple request like this, you just go ahead.  I don't think any review is necessary.

Out of curiosity, should we add the license statement to each source file?

Good question.  It depends.  Either way is OK and has different implications.

This issue has be solved by #2.