x/tools/cmd/godoc: Proposal: let godoc recognize mT styled source

Proposal: let godoc recognize marktop styled source
Author: Ohir Ripe [Wojciech S. Czarnecki]
Last updated: 2019/11/28
Discussion at https://golang.org/issue/35896
Related to: #7873, #16666 and other "rich format please" issues.
Abstract
I propose using a ´marktop´ annotations within the ¨Go¨ source documentation. Marktop enrichment is unobtrusive even if read in the ˉraw sourceˉ. Did you notice marktop annotations? These would render:
I propose using a marktop annotations within the Go source documentation. Marktop enrichment is unobtrusive even if read in the raw source.
Background
Current state of ¨Go¨'s source documentation processing is good enough for documenting single ´implemented ˘things˘, ie. functions, variables, constants. It falls short if one must convey a new idea, an unobvious implementation of an algorithm, or even just describe a sequence of events (no lists, sadly).
Proposal
I propose extending go doc processing by a marktop annotations parser implementing both console and html output of described below format. (note that example of keyboard mapping is irrelevant for this proposal).
::: Marktop 2019 :::

 Styling:

 ˟ U+02DF cross      ˟dismiss / back to normal.  AltGr‿ /⃣  (map example)
 ´ U+00B4 acute      ´italics´       ´italics˟         ,⃣   𝑖𝑡𝑎𝑙𝑖𝑐𝑠
 ¨ U+00A8 diaeresis  ¨bold¨             ¨bold˟         .⃣   𝐛𝐨𝐥𝐝
 ˘ U+02D8 breve      ˘ibold˘    ˘bold+italics˟         m⃣   𝒃𝒐𝒍𝒅-𝒊𝒕𝒂𝒍𝒊𝒄𝒔
 ˉ U+00AF macron     ˉfixedˉ     ˉfixed width˟         -⃣  fixed width

An emphasis (styled text) begins after either acute, diaeresis, or breve character - none followed by a cross - and ends at a breve, acute, or diaeresis of the other emphasis' start, or this emphasis
stop. It ends also at a macron or at a cross accent. As the 'fixed font' span begins and ends only with a macron, other three emphases can be used within. An empty line ends all running emphases.

 Structure:

 ¤ U+00A4 currency     ¤ list item             // bulleted list    0⃣
 ¹ U+00A4 supers.one   ¹ list item             // numbered list    1⃣
 ª U+00A4 feminine.o.  ª list item             // lettered list    a⃣
 ¶ U+00B6 pilcrow      section head  ¶(refid)  // Section anchor   P⃣
 § U+00A7 paragraph    subsection    §(refid)  // SubSect anchor   p⃣
 « U+00AB lguillemet   « important »           // styled note      <⃣
 » U+00BB rguillemet                           // closes a note    >⃣

   Bullet surrogates in the source should be indented for readability.
   Any special character can be made ordinary by an immediate dismiss:
   §˟  «˟  ´˟  ¨˟  ˘˟  ˉ˟  »˟  ˟˟  ¤˟  ¹˟  ª˟  ʷ˟  ¶˟

 Hypertext:

 » U+00BB rguillemet   « to be cited »(refid)  // referable note   >⃣
 « U+00AB lguillemet   details there «(refid)  // quote a refid    <⃣
 ʷ»                    « this is on the webʷ»  // extern.link text w⃣
 ʷ U+02B7 modsmall.w     ʷ somesite.tld/path/  // url listed below w⃣

External links are introduced via the « note ending in ʷ» digraph. The url path — without protocol — must be given in the following line prepended by an indented ʷ. If more than one ʷ» is present in a line, their respective url paths are given in separate lines below:

  in our «IEEE-ITSS Open Journalʷ» and also on «our facultyʷ» site.
     ʷ www.ieee-itss.org/oj-its
     ʷ www.ivt.ethz.ch

The «(refid) internal link token always outputs its target's text. On the console it is put before "(referral)" that stays, and the html version makes this text into a hyperlink. Eg. the source of:

    Annolex Editor  ¶(Sect.2)
    ... Please read «(Sect.2) for the primer.

 should output on the console:

    Annolex Editor (Sect.2)
    ... Please read "Annolex Editor" (Sect.2) for the primer.

 but in html it is expected to output a link:

    ✻ Annolex Editor
    ... Please read ͟͟A͟n͟n͟o͟l͟e͟x͟ ͟E͟d͟i͟t͟o͟r for the primer.

The final form of the output, incl. hypertext protocol used, is defined by the marktop processor. This specification only mandates that the plain text renderer — if used at all — remove marktop special characters, spaces following the «, and excessive spaces prepending any of »§¶. Still, all ´refids´ in parentheses must be preserved for humans to see. Accordingly, the source author is expected to keep ´refids´ both short ¨and¨ meaning.

Editing software may apply styles while keeping the syntax visible.

::: Godoc use :::

The § and ¶ not followed by a left parenthesis, and ¤, ¹, or ª
used in the middle of line are ordinary in godoc's marktop.

 Lists:

 ¤ List items should be recognized as such even if user-indented.
   It can not be code, as neither ¤ nor any bullet character can
   possibly open a line of a valid Go source.
 ⬩ Other bullet characters could be recognized as first nonwhites.
 ¤ terminal output should impose uniform indentation on lists
   ¤ ie. godoc list do not nest and the gofmt might know this.

 ¤ List items need to be given without blank lines inbetween to
   keep numerator going. The ¤^ list continuation digraph was
   removed from the specs and left for processor to define
   list numbering and continuation rules.

Linking, quoting and TOC:

 Paragraph text of sub-section may follow heading immediately in
 the next line, as the §(refid) annotation tells author's intent.
 Both ¶() and §() make TOC entries. The »(refid) referrals do not.

 Section head ¶(refid)  ←— TOC: "Section head", Section head styling
 Subsect head §(refid)  ←— TOC: "Subsect head" under "Section head"
   ¤ an item  §(refid)  ←— TOC: "an item", under "Section head"
    ¤ another »(refid)  ←— no toc, "another" highlits as an anchor
 note: « this »(refid)  ←— no toc, "this" highlits as an anchor

Rationale
Documentation that can be styled even with only bold and italics, and one that can be structured to fit the domain, may help package authors to be more precise and unambigous, and help documentation consumers to avoid misunderstandings.
Marktop enabled godoc may encourage a well structured documentation that is written into the program sources even for, or the more for most sophisticated ideas, solutions and code. Now packages of even middle complexity often resort to external descriptions of their api.
Marktop parsing is fast, and there are no ambiguities introduced.
Unlike Markdown that makes raw annotated text almost unreadable, the marktop annotations are barely noticeable unless reader is wilfully scanning for the formatting hints.
Compatibility
This proposal extends documentation source syntax, and this syntax parsing methods, in a way that may not influence any program source but — in theory — might alter the visible html output of some existing documentation.
Even if this would happen, such a change would likely effect in the font decoration or size, and would not affect the meaning.
Implementation
None ideas yet.

Syntax was proven useable yet in the VT180 era (Z80 CPU, 64kB RAM, CP/M OS).
Only change is that the · ° … of then are ˟ ˘ ʷ now. And that ʷ now opens web,
while … then could have asked for diskette change and open a next file to read
or edit.

Open issues

As @wgrr noted in https://github.com/golang/go/issues/35896#issuecomment-559623192 ,
two proposed glyphs are not in the WGL4 set. WGL4 fortunately has, unfortunately
only two, runes that keeps to the top and fits purpose. Not as good as previous ˟ and ʷ,
but still unobtrusive:

  ⁿ U+207F supers.n
  ˙ U+02D9 dotabove

Every step toward Literate Programming is worthwhile. It will be interesting to see how others react to the use of characters beyond 1970s ASCII as formatting requests. In a proposal of my own about using such a character in and addition to ` in Go, the "those strange characters scare me" sentiment seemed pretty strong.

fwiw: glyphs U+02DF and U+02B7 are not present in Go fonts

fwiw: glyphs U+02DF and U+02B7 are not present in Go fonts
Yes, WGL4 does not have them. This qualifies for "Open issues", thank you.

I'm totally in favour for rendering numbered and bulletpoint lists better
(read as ul and li in HTML) but Marktop is just dead ugly to read and
painful to write. It'll make my workday miserable when I have to write
it and when I have to read it.
This proposal aims at too much. I think the current godoc formating lacks
two things: a) Inline verbatim/fixed width/code styling and b) automatic
list detection.
Headings, paragraphs are solved and work (not perfectly but they work).
URLs can be detected reliably and I actually do like to see the URL of
a link before clicking on it, so I do not see the need for a link text that
differs from the URL. (This is different for very technical URLs with e.g.
lots of query parameters, but such URLs are uncommon in documentation).
For b) it is hard to find some heuristic which works well for list existing
in the wild (I tried it on the corpus). This might be fixable as most list
which are not detected by a simple heuristic are rendered badly in the
current version anyway. This might be doable and would make complicated
documentation (e.g. package doc better). Even the spec uses uls and lis.
Common wisdom for a) is to use backticks as in markdown. This is ugly,
simply because it is markdown and input and output render different
characters.  A less invasive option would be to use two spaces which
makes a word/identifier to stand out in plain text:
Foo foos the  target  while aiming at bar.

Being rendered as "Foo foos target while aiming at bar."
I think something like this would help as it makes writing documentation
easier as you have to reword less and have to rename arguments less
to produce understandable documentation.
(Just my contribution to this bikeshedding.)

@vdobler

b) automatic list detection.

This would impose that the author has gotten intimate knowledge about how the
used automat "thinks". This now affects millions of markdDown and yaml users. Process of getting to this knowledge is painful and time consuming.

For b) it is hard to find some heuristic which works well for list existing
in the wild (I tried it on the corpus). This might be fixable as most list
which are not detected by a simple heuristic are rendered badly in the
current version anyway.

Re-read mine's above ;)
There are NO heuristics allowed in the marktop processor. Period.
The author is in control of the output, not an AQI.
Every construct representable in marktop (the list item is one of) is introduced by a single rune and it usually ends where other of its kind begins. The specified hard rule imposes that any marktop introduced change surely ends at an empty line. No exceptions.
Therefore a list is rendered only where intented:
 ¹ List item, ends where another begins
 ¹ next item, begins where other ended

Empty line ended the list. Indent above is not a part of the syntax (hence refferal to the gofmt in the go-focused part of the spec).

Headings, paragraphs are solved and work (not perfectly but they work).

Marktop does not intent to interfere here. It just adds an identifier to the already recognized section, it allows any sentence to have a referral identifier added then used.
The intra-links that once upon a time worked well for the legalese are really useful in technical texts too. (Have you never changed the section title then hunted for references to the old one in prose?)

URLs can be detected reliably and I actually do like to see the URL of
a link before clicking on it, so I do not see the need for a link text that
differs from the URL. (This is different for very technical URLs with e.g.
lots of query parameters, but such URLs are uncommon in documentation).

URLs are not to be detected with marktop. The paths are to be listed.
The intent is to have (web) refferals readable in the source.

Common wisdom for a) is to use backticks as in markdown. This is ugly,
simply because it is markdown and input and output render different
characters.

Backticks are part of the Go language specification and they may carry their meaning even in the prose. Eg. you should use raw string, ie. one in `backticks`.  None of marktop proposed runes is a part of any programming language syntax (one I know of, at least). Marktop was designed with possible ambiguities in mind and aims to not introduce one.

A less invasive option would be to use two spaces which
makes a word/identifier to stand out in plain text:

Mhm... then let your editor software strip an ending one:
...you should use  while  loo…
                                OOOPS!
...you should use  while
loops

Closing this issue for now, until I address weaknesses pointed to me (on irc) mostly
by Windows users (I must admit that I did not know that Microsoft no longer supports creation of custom keyboard mappings, something that was available since XP SP2).

Windows stock US-Intl layout lacks breve and macron
MacOS stock US layout lacks superscript numbers
Various national layouts lack one or the other
possible IDE customization can be thought only "per language".

And 5: I desperately need a name without "mark" in it.  Two out of five participants were not ashamed to acknowledge that they did not read this proposal past the title because they “abhor the very word markDOWN”.
I will, hopefully soon, open an improved version of this proposal.
Thank you all who read this dense text below the title :)


Task/google required changes

Codecov Report
Base: 88.23% // Head: 88.23% // No change to project coverage :thumbsup:

Coverage data is based on head (b28827f) compared to base (0f3eea3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##           development     #167   +/-   ##
============================================
  Coverage        88.23%   88.23%
============================================
  Files               52       52
  Lines             4029     4029
============================================
  Hits              3555     3555
  Misses             474      474

Flag
Coverage Δ

unittests
88.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


'classname' kwarg not being rendered in CharBlock in StreamField

Issue Summary
Following the docs on using StreamField, the class passed as the classname kwarg to blocks.CharBlock() is not being rendered.
class BlogPage(Page):
    ...
    body = StreamField([
        ('heading', blocks.CharBlock(classname="full title")),
        ('paragraph', blocks.RichTextBlock()),
        ('image', ImageChooserBlock()),
    ])
    ...

Steps to Reproduce

Follow the docs on using StreamField.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.7.3
Django version: 3.0.7
Wagtail version: 2.9
Browser version: Firefox 68

This works for me. The bakerydemo project uses classname="title" for heading blocks, and this shows up in the editor form as expected:
https://github.com/wagtail/bakerydemo/blob/2ec4475f75b364c630c5d0191c5a477ca6e1bad6/bakerydemo/base/blocks.py#L26

Perhaps you were expecting the classname to show up in the front-end rendering instead? We should clarify this in the docs (and consider changing the name to form_classname for consistency with structblock and others: #5100)


Different data preprocessing in cyclegan vs. cycada

https://github.com/jhoffman/cycada_release/blob/0a50507e4cb3e1bd6ad0b29dba5d75065658ff48/cycada/data/data_loader.py#L86
https://github.com/jhoffman/pytorch-CycleGAN-and-pix2pix/blob/e484612d83449d05024a7d5fd2e012be53faad85/data/mnist_svhn_dataset.py#L37
Hi, I was comparing the image preprocessing routines in the cycada code and the cyclegan code in the submodule for the SVHN to MNIST task. It seems that in the cyclegan code, the MNIST image is normalized with transforms.Normalize((0.5, 0.5, 0.5), (0.5, 0.5, 0.5)).
In contrast, in the cycada directory the input image is normalized with transforms.Normalize((0.5,), (0.5,)), which I believe only normalizes the first channel of the image? I understand that they don't have to match since the ADDA part is separate from the cyclegan part, but is this difference intentional?

The two are equivalent when the mean/std is the same across all channels. This code also works when we only have a single channel model (ex LeNet for MNIST).


Implemented Feature Request #34 - Pie Chart

Hey @pawank0411,
Implemented Feature Request #34 - Pie Chart
Please let me know whether this PR meets your expectations. Please let me know if I had to do something better...
Thank You

@VishnuSanal Amazing work looks very good 🎉


Failing registration with missing metadata in multilanguage journal

Dear @bozana and @asmecher ,
we just had the situation that the plugin attempted to register an article in a multi-language journal (German & English). The editor entered the metadata for the article's author only in the German form, while leaving the English metadata blank. In the DOI registration process however, the plugin stopped working (and did not process the subsequent articles) when hitting the described article.
Can you reproduce the bug?
Thanks in advance.
Best Regards,
Adrian

Hi @GrazingScientist, thanks for reporting. We would need a few more information in order to figure out what is happening:
Which OJS version are you using?
What is the primary locale of the article? -- Is it German? -- All metadata must be entered in the article primary locale...
When you try to export only that article do you get any errors and what errors exactly?
If export works fine, when you try to register only that article what exactly happens?
It would be good to first use support forum, so that we are able to first check and test and to then create an issue once we figured out that it is a bug or something needs to be done...
Best wishes,
Bozana


bump sdk & inspector (7.3.3)

The issue above was related to the .vsix file that is broken, all good with this PR 👍


Runtime error FFX

Runtime error R6030 - crt not initialized

I have never encountered this before, I will look into it in more detail. Let me know if the latest version still has it, by the way.

Latest version work! Thanks for mod )


[theme] Styled calendar #219

Changes

Changed calendar style.

Screenshots

Checklist

[x] I have read the contributing guidelines.
[x] I have manually tested the proposed changes.
[x] I have written new test cases to avoid regressions. (if necessary)
[x] I have updated the documentation. (e.g. README.rst)
[x] I have added [change!] to commit title to indicate a backward incompatible change. (if required)
[x] I have checked the links added / modified in the documentation.

closes #219

Coverage remained the same at 98.455% when pulling 3447316e2f261376dbe2e2510148e06ff6602180 on issues/219-style-calender into f5cde29f6ba8a3a3742f213094e924a9ecf93a9b on master.

@nemesisdesign Please have a look. We need to reduce padding.


Always generate links for all searches

Related Issue(s):

https://github.com/Healy-Hyperspatial/stac-fastapi-mongo/issues/20

Description:
Always generate links for all searches, since now base links are generated on the request and no longer stored.
PR Checklist:

[x] Code is formatted and linted (run pre-commit run --all-files)
[x] Tests pass (run make test)
[x] Documentation has been updated to reflect changes, if applicable
[x] Changes are added to the changelog

Should we have a test for this?

Should we have a test for this?

added


HAWQ-992. PXF Hive data type check in Fragmenter too restrictive.

Changes in this PR:

Updated ProtocolData to parse new attributes X-GP-ATTR-TYPEMODX-COUNT, X-GP-ATTR-TYPEMODX-Y;
Mapped EnumHawqType to DataType;
Removed HiveInputFormatFragmenter.toHiveType and replaced it with metadata-driven method HiveUtilities.toCompatibleHiveType;

Overall LGTM, but I'd like to spend more time reviewing. This is a lot of code

+1 great job with refactoring


Installation on debian 7 failed

Hello, i try to install the Let's Chat software for my company on debain 7 - 64 bit.
I installed the requirements, cloned the git repository and run npm install.
 ReferenceError: resp is not defined
npm ERR!     at /usr/local/lib/node_modules/npm/lib/cache.js:714:32
npm ERR!     at RegClient.get_ (/usr/local/lib/node_modules/npm/node_modules/npm-registry-client/lib/get.js:111:14)
npm ERR!     at RegClient. (/usr/local/lib/node_modules/npm/node_modules/npm-registry-client/lib/get.js:47:12)
npm ERR!     at fs.js:266:14
npm ERR!     at /usr/local/lib/node_modules/npm/node_modules/graceful-fs/graceful-fs.js:105:5
npm ERR!     at Object.oncomplete (fs.js:107:15)
npm ERR! If you need help, you may report this entire log,
npm ERR! including the npm and node versions, at:
npm ERR!     http://github.com/npm/npm/issues
npm ERR!  Additional logging details can be found in:
npm ERR!     /home/jabber/lets-chat/npm-debug.log
npm ERR! not ok code 0
How can i fix that? Greetings

Try installing libexpat: https://packages.debian.org/sid/libexpat-dev

It works now, thank you!
But i have a new problem, i enabled the XMPP Application. It works to connect, but the web interface is not more reachable and this errors show up: http://img-host.de/bild.php/53964,bildschirmfoto20150216um154331XLFD4.png

Interesting, I installed it on a debian 64 bit. For me the web interface works but the XMPP does not. The port is open but the Login always fails.

@dbr1337 can you paste your settings for http and xmpp?
@Twilek-de are you using the username to login? What happens when you telnet to that port and type something?

http:
enable: true
host: 'localhost'
port: 5000
xmpp:
enable: true
roomCreation: false
debug:
handled: false
unhandled: true
port: 5222
host: <IP_ADDRESS>
tls:
enable: true
key: ./mycert.key
cert: ./mycert.crt

@dbr1337 what address are you using to access the web interface? The host setting under http is currently localhost, which will only be accessible there. Try changing it to <IP_ADDRESS>

I had change it before to <IP_ADDRESS>, now it was localhost. But i changed it and i worked. My next question: i need to connect the xmpp server with active directory and implement the users from there. how can i do this?

We have modules for LDAP and Kerberos auth:
https://github.com/sdelements/lets-chat-ldap
https://github.com/sdelements/lets-chat-kerberos
Please be sure to disable local registration too! https://github.com/sdelements/lets-chat/wiki/Authentication#local-authentication

Closing this. @Twilek-de do you mind opening another issue if you're still having problems getting XMPP to run? CC @sibartlett


Strings buffer size wrong?

In my php.ini:
; The amount of memory for interned strings in Mbytes.
opcache.interned_strings_buffer=32

output
cachetool.phar opcache:status
...
| Strings buffer size  | 24 MiB
...

is this normal? CacheTool 5.0.0

It just means that the currently allocated buffer is 24 MiB, which is lower than the maximum of 32 MiB


search pills: Fix some UI bugs with long pills.

in the search bar
before:

after:

in the typeahead
before:

after:

@amanagr can you review this one?

Pushing to add this change https://chat.zulip.org/#narrow/stream/101-design/topic/search.20pills.20.2326803/near/1883175

This seems right to me, merged, thanks @evykassirer! @amanagr it's still probably worth your doing some testing/review, since extra sets of eyes are valuable this late in the release cycle.


Nuget packages with strong names

Can you update nuget packages with signed versions?

I have not used strong named assemblies in the last years. I'm not against them, but I know they are known to cause trouble. I'll take a look on how popular projects handle this and see if it is possible. The core library has (and will never have) any 3rd party external dependencies - so it should be easy to have it signed. Adapters on the other hand have dependencies which are probably not signed (like Nancy).
Can you provide more details on how strong named packages would be helpful?

I was trying to use it in healthcare project, but app must be signed.

Security reasons.
I was trying to use it in healthcare project, but app must be signed.

Here is a link to the strong naming policy and reasons for the .net corefx libraries. The general idea is to sign core framework libraries for compatibility and not to sign open source, general purpose libraries to avoid issues with dependencies and to have the "drop-in" replacement possibility.
Strong naming does not give you any security. It gives you guarantees of identity of the signed assembly.
That being said I see two options:

publish a .nuget package under a different name (Metrics.NET-Signed) with a signed version of the library with the key included in the repository for posterity.
sign the assembly yourself using something like Nivot.StrongNaming - an example with signing NancyFX here.

Since I personally don't use strong naming and don't foresee ever using it, I don't want to take responsibility of maintaining a strong named package - but it should be easy for anyone to clone the repo and publish the signed package.
I would recommend using option 2 because of the benefit of being able to sign it with your own key (thus getting some "security" as the assembly can't be replaced).


notebook does not allow first code cell to run because import statement does not include parent module.

change: from agents import * to from aima3.agents import *
Add note to add to PYTHONPATH: <aima3 root directory> so the notebook can find the aima3.agents module.

Thanks! Fixed first point via #6908ab5bc4ffa19b40fea90a819c64ac2fdc706a
You should be able to "pip install aima3" or "cd aima3; pip install ." or set the PYTHONPATH, etc. Too many possibilities to list all of them.


TEMP - OLD version working for Myanmar exposure analysis; COUNTRY=myanmar

I believe https://github.com/WFP-VAM/prism-app/commit/48218fd4403e22bf9bf9634b8bf345936f09dc28 might have introduced this regression since we didn't have any way to test it for storms.
This is a temp hack

Things are now working for tropical storm exposure analysis. Thanks a lot @ericboucher!
I tested flood exposure analysis next and see the table results are incorrect (all have a value of 0) and the tooltip has a precision issue. The tooltip precision issue exists for tropical storms too.

Sorry @ericboucher - we found another issue. It seems we're dropping a number of townships during the analysis. For example, the township of Sittwe (TS_PCODE = MMR012001) in Rahkine State, Sittwe district is missing in our results. But it's a densely populated area and fully within the 120km/h wind buffer.
When I run a local analysis I get 138,001 people exposed based on WorldPop constrained, UN-adjusted population.
Here's the result when I run zonal stats locally.

And here's what we see in PRISM

It looks like there's a general problem with boundaries. The list of townships is basically cutoff alphabetically after Rakhine > Sittwe. You can see this at play in the Go to feature too.

Sorry @ericboucher - we found another issue. It seems we're dropping a number of townships during the analysis. For example, the township of Sittwe (TS_PCODE = MMR012001) in Rahkine State, Sittwe district is missing in our results. But it's a densely populated area and fully within the 120km/h wind buffer.

You were right @ericboucher - this was a boundary file issue. I fixed the boundaries and have this working locally and also see it working on the PR deployment https://prism-834.surge.sh/
For some reason, I am not seeing it updated on the surge deployment I've shared with others (https://prism-mmr-demo.surge.sh/). I tried clearing the cache locally on my browser and also cleared the stats API cache. I also tore down the surge project and redeployed but with no luck. If I deploy the same build to a new surge, it works as expected. Not worth spending further time on this, but something to note.

Things are now working for tropical storm exposure analysis. Thanks a lot @ericboucher!
I tested flood exposure analysis next and see the table results are incorrect (all have a value of 0) and the tooltip has a precision issue. The tooltip precision issue exists for tropical storms too.

@ericboucher - this is the issue we need to resolve next.

@ericboucher @wadhwamatic . Fixed the problem with the exposure analysis results and thee differs between tropical storms and the rest exposure analysis  results. I think that right now, works as expected. Tested it to a bunch of countries and seemed to work solidly. Moreover added the escapeCell to the values of the results because some of them are not treated correctly this is now fixed. @ericboucher also I took the initiative to split the functionality a little bit to make smaller units to be more readable.

Thank you @ericboucher and @HarryMytilinaios for the big push on this. I tested across as many scenarios as I could and found no issues. We lost access to earthquake data layers on GeoServer so I couldn't test that but I hope it won't differ from the tropical storm layer analysis when that access is restored.
This is GTG from my side. Thanks again


having problem in mail loop via header

Can some one please help me with this error.
Error:
Command died with status 255: "/usr/bin/perl
/usr/local/bin/pdfconvertme.pl --force-markdown". Command output: Detected
a mail loop via header 'In-Reply-To:
<EMAIL_ADDRESS>aborting! at /usr/local/bin/pdfconvertme.pl line 272, <> line 500.
Final-Recipient: rfc822<EMAIL_ADDRESS>Original-Recipient<EMAIL_ADDRESS>Action: failed
Status: 5.3.0
Diagnostic-Code: x-unix; Detected a mail loop via header 'In-Reply-To:
<EMAIL_ADDRESS>aborting! at /usr/local/bin/pdfconvertme.pl line 272, <> line 500.

Hi,
Typically I don't expect to see an In-Reply-To: header on a conversion request as it usually indicates an autoresponder of some sort.  If you can forward a sample raw message that exhibits this behavior to<EMAIL_ADDRESS>I can take a closer look.
Thanks,
Brian

Hello Brian
Thanks for your support. As you suggested I commented the lines 272/273 (pdfconvert.pl) and its works fine now. I have one last questions and some other things I have done to get it work.
Wow ! It worked. Just for a better understanding this is what I used.

There was problem in the pdfconvert.pl file. The mail_attachment.sh file path is not given as full path. So, I was about to change it to get it work.

Then I had the problem of this mail loop issue in form of in_reply_to. After commenting the lines you mentioned, it worked.

Now, the email are sending with "Unknown sender" and the "from:" tag is empty. Can you please tell me what should I do to make it work?

Thanks for your prompt support.

Hello,
Sorry for not closing the loop on this.  I've pushed out a fix today for issue #1 (commit 6ae3c1a) and there is also a --force-from option you can specify to manually set the from that may fix #3 (located in commit 285fe87).
For #2 - I've made a variable at the top of the script where you can disable this (located in commit 58b0fc9).
Let me know if you have any further questions or issues.
Brian


0.47 broke using shift+capslock as the PowerToys Run key

Microsoft PowerToys version
0.47.0
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce

Assign Shift+Capslock as the shortcut to bring up the PowerToys Run launcher window.
Press Shift+Capslock

✔ Expected Behavior
The launcher window comes up, but the capslock state is not toggled. This was the behavior in 0.45 and earlier.
❌ Actual Behavior
The launcher window came up, but the capslock state toggled on.
Other Software
No response

Uninstalling 0.47 and re-installing 0.45 fixed this issue for me. I guess I'll stay on 0.45 for now.
Note that my current use case is as follows:

I use the Keyboard Manager to re-map Capslock to Shift+Capslock. This causes capslock to no longer toggle unless I press ctrl+capslock, which allows me to toggle it only in the very rare case that I want to toggle it.
I assign Shift+Capslock to PowerToys Run, which due to the keyboard mapping, allows me to bring up the launcher with a single tap of capslock.

Note that using the capslock key like this mimics the keyboard layout on a Chromebook, which replaces the capslock key with a search key.

Will add a option to keep using the old method.
PR: https://github.com/microsoft/PowerToys/pull/13557


ServingRuntime autoscaling monitoring GPU utilization

Hi all,
I noticed in the scaling doc page (https://github.com/kserve/modelmesh-serving/blob/main/docs/production-use/scaling.md) that now is possible to set the ServingRuntime autoscaling with HPA, but using metrics based on cpu utilization.
Is it possible to scale the ServingRuntime using metrics regarding GPU?
Thanks in advance

Not currently.
https://github.com/kserve/modelmesh-serving/issues/329#issuecomment-1442742323
// Autoscaler Metrics Allowed List
var AutoscalerAllowedMetricsList = []AutoscalerMetricsType{
	AutoScalerMetricsCPU,
	AutoScalerMetricsMemory,
}

FYI @Jooho @njhill

@andreapairon
As @ckadner mentioned, it can not use GPU-related metrics.
HPA is a default object provided by kubernetes, and ModelMesh relies on this HPA object to autoscale the ServingRuntime Pods.

@Jooho @ckadner, according to the HPA docs, since k8s v1.23 the autoscaling/v2 API supports custom metrics. I'm not sure what would be involved in exposing this through modelmesh,  however.

@dagrayvid I will look into this soon

@Jooho Is there any new information regarding that question?
We would also like to use custom metrics that are not related to cpu or memory

@GolanLevy at the moment, modelmesh is only supporting cpu/memory only. But the custom metrics are supported by HPA so it would be enhanced.  Contribution is always welcome.


OpenGL ES 2 renderer

You should pick one rendering back-end from the start of a project.

[x] Create a slideshow to explain the advantages and limitations of the GLES 2.0 renderer.
[x] Compare the same scene with GLES 3.0 and 2.0.
[x] Showcase the CPU particles.

Final version of the video is done and uploaded


Lens UI fails to list traces

Describe the Bug
Lens UI shows no traces while the standard ui does
Steps to Reproduce
Steps to reproduce the behavior:

open a project that generates traces
execute latest zipkin: docker run -d -p 9411:9411 openzipkin/zipkin:latest
go to default UI and list them. You will see your traces:

The XHR request is:
Request URL:http://localhost:9411/zipkin/api/v2/traces?limit=10&lookback=900000

click "Lens UI" and click the "magnifying glass" icon to list all your traces (without modifying any of the default filters), you will see that no traces are listed:

The XHR request is:
Request URL:http://localhost:9411/zipkin/api/v2/traces?endTs=1582712351352&limit=10&lookback=300000&remoteServiceName=all&serviceName=all&spanName=all
Note this last XHR request does not return any trace
Expected Behaviour
Traces are listed the same way in both UIs

What happens if you just do normal reload of lens without retaining parameters from the old ui.. then execute search by constructing it?
a guess is that lens isn't clearing the "=all" stuff added by the old UI, so this problem will go away in the next release anyway as the old UI is no longer mounted anyway.

@javigomez Lens is now the default UI, the 'Try Lens UI' button has been removed so this should no longer be an issue. OK to close this issue ?

I am having the same issue
java \
    -Dzipkin.collector.kafka.topic=xxxxxxxx_zipkin \
    -Dzipkin.collector.kafka.bootstrap-servers=XXXXXXXXXXXXX:9092 \
    -Dzipkin.collector.kafka.overrides.auto.offset.reset="earliest" \
    -Dzipkin.collector.kafka.overrides.sasl.mechanism="PLAIN" \
    -Dzipkin.collector.kafka.overrides.security.protocol="SASL_SSL" \
    -Dzipkin.collector.kafka.overrides.ssl.endpoint.identification.algorithm="https" \
    -Djava.security.auth.login.config="jaas.config" \
    -jar zipkin-2.19.2.jar

In the Lens UI:

Latest from today shows JS error:
TypeError: Cannot read property 'map' of undefined
    at TracesTableRow.jsx:141
    at Xo (react-dom.production.min.js:153)
    at ms (react-dom.production.min.js:261)
    at cu (react-dom.production.min.js:246)
    at su (react-dom.production.min.js:246)
    at $s (react-dom.production.min.js:239)
    at react-dom.production.min.js:123
    at t.unstable_runWithPriority (scheduler.production.min.js:19)
    at Ui (react-dom.production.min.js:122)
    at Yi (react-dom.production.min.js:123)

but on the screenshot, you can see that it did find traces.
using a direct url to one of the traces is fine.

Closing as it's an artifact of the transition period where we had both Lens and Zipkin classic running, that time is long behind us. Also no further feedback since February.


fix checkout header generation not uniform

Description
Please describe your pull request:

Why is it necessary?
Headers are not generated uniformly
What does it improve?
Uses the header generation from confirm in shipping_payment to have a single base
Does it have side effects?
none that i can think of

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
--

How to test?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-17655.
Please use this issue to track the state of your pull request.

By the way I think your tests are broken. Travis build for PHP 5.6 failed in the first run: https://travis-ci.org/shopware/shopware/builds/197600294
I had to amend and force push without changing code to re-trigger travis. Build passed the second time around...

We decided to close this PR because of time constraints and limited overall benefit to the product. If this change is still important to you, please create a new issue on https://issues.shopware.com so people can up-vote this feature.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.

Ticket opened!
https://issues.shopware.com/issues/SW-18547
Something like this shouldn't be closed, cause the code quality should be maintained.


add analyze page back in with superset docs

This has a lot of copied content from https://hub.meltano.com/utilities/superset so we can optionally pair it down and just basically link out to that hub page but I think this is a good starting point so we have content on release day.

Pushed a quick commit to change the ordering in the sidebar


[terraform-ibm-modules/terraform-ibm-account-infrastructure-base] Release Pipeline failed 2024-08-10-13:22

For details see: https://github.com/terraform-ibm-modules/terraform-ibm-account-infrastructure-base/actions/runs/10331929231

[1:21:23 PM] [semantic-release] › ✘  An error occurred while running semantic-release: Error: Command failed with exit code 22: ./ci/trigger-catalog-onboarding-pipeline.sh --version=v1.11.5
curl: (22) The requested URL returned error: 424

Retrying: https://github.com/terraform-ibm-modules/terraform-ibm-account-infrastructure-base/actions/runs/10331929231


Fixed duplicate HTML_DOCUMENT variable during test run

Fixed duplicate HTML_DOCUMENT variable during test run by renaming one of them to HTML_DOCUMENT2.

Thanks !
Actually this is the same constant on the 2 files, with the same value.
It would be better to place it in a shared place like spec_helper, could you do that instead ?


Compilation fails for master branch, also using cargo install

   Compiling tera v0.11.9
error[E0308]: mismatched types
  --> cargo-cov/src/error.rs:11:1
   |
11 | / error_chain! {
12 | |     links {
13 | |         Cov(::cov::error::Error, ::cov::error::ErrorKind);
14 | |         Tera(::tera::Error, ::tera::ErrorKind);
...  |
47 | |     }
48 | | }
   | |_^ expected struct `error_chain::State`, found a different struct `error_chain::State`
   |
   = note: expected type `error_chain::State` (struct `error_chain::State`)
              found type `error_chain::State` (struct `error_chain::State`)
note: Perhaps two different versions of crate `error_chain` are being used?
  --> cargo-cov/src/error.rs:11:1
   |
11 | / error_chain! {
12 | |     links {
13 | |         Cov(::cov::error::Error, ::cov::error::ErrorKind);
14 | |         Tera(::tera::Error, ::tera::ErrorKind);
...  |
47 | |     }
48 | | }
   | |_^
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

error[E0308]: mismatched types
  --> cargo-cov/src/error.rs:11:1
   |
11 | / error_chain! {
12 | |     links {
13 | |         Cov(::cov::error::Error, ::cov::error::ErrorKind);
14 | |         Tera(::tera::Error, ::tera::ErrorKind);
...  |
47 | |     }
48 | | }
   | |_^ expected enum `std::option::Option`, found struct `error_chain::backtrace::imp::InternalBacktrace`
   |
   = note: expected type `std::option::Option<std::sync::Arc<error_chain::Backtrace>>`
              found type `error_chain::backtrace::imp::InternalBacktrace`
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0308`.
error: Could not compile `cargo-cov`.

To learn more, run the command again with --verbose.

I would assume a version mismatch/API change (currently both error-chain 0.11 and 0.12 are used in the build). Maybe checking in Cargo.lock could solve this issue?

@kennytm, the fix in #23 worked for me. Can you push a release to crates.io?

Sorry. Published 0.0.5 now.


Update gatsby.md

Update the descriptive content for Gatsby to match the project homepage at https://www.gatsbyjs.org/.

👍

Can you change the headings to h2's instead h3


articles.mercola.com

From https://mediabiasfactcheck.com/mercola/:

Source: https://www.mercola.com/

At https://articles.mercola.com/sites/articles/archive/2020/05/03/is-the-new-coronavirus-created-in-a-lab.aspx:

Official Media Bias Fact Check Icon (deprecated) presents a CP icon
Fact Layer presents a question mark

This should be fixed, assuming your plugin has updated to the latest definitions it'll map properly for articles.mercola.com and all their various other sub-domains.

Thanks, I don't see a fix yet, how frequent are checks for updates to definitions?

It should update once a day, but you can force an update sooner by going to the plugin options. Not sure about vaxxter, that's been in the definitions a while and it's showing correctly for me. Was there a specific article you were looking at? It's possible the plugin encountered a javascript error of some sort, causing it to default to the questionmark.

Thanks. I didn't think to look for preferences! My bad.
I couldn't reproduce the question mark at https://vaxxter.com/

Re: https://articles.mercola.com/sites/articles/archive/2020/05/03/is-the-new-coronavirus-created-in-a-lab.aspx

It should update once a day,

Pre-update:

Post-update (manual):

– and after the manual update I got the no entry icon, so I guess that updates are not occurring for me. I'm almost certain that I found things similarly outdated with a test profile (just two extensions), if you like I can aim to make the issue reproducible with a clean profile.

The update logic has been rewritten a bit, is this still happening?


linking error when used with wry/tao on macOS aarch64

Bevy version
0.12.1
Relevant system information
Chip: Apple M1 Max
OS: macOS Sonoma 14.3
Rustc: 1.75.0 (stable)
Xcode 15.2
What you did
Consider the following minimal Rust program:
// main.rs
use bevy::prelude::*;

use tao::event_loop::EventLoop;

fn main() {
    let event_loop: EventLoop<()> = EventLoop::new();
}

// Cargo.toml
[package]
name = "mygame"
version = "0.1.0"
edition = "2021"

[dependencies]
bevy = { version = "0.12.1", default-features = false, features = [
    "bevy_render",
    "dynamic_linking",
] }
tao = { version = "0.25.0", default-features = false }
wry = { version = "0.36.0" }

Run it on an ARM Mac and it will cause a linking error. I have tried this on 3 separate M1 Macs and it causes the same issue. I updated the OS and Xcode and that didn't solve the problem either.
What went wrong
Project is expected to build with no issue, or an error at the compile level. Instead I get the following linking error:
error: linking with `cc` failed: exit status: 1
  |
  = note: env -u <redacted env print>

= note: ld: warning: ignoring duplicate libraries: '-lSystem', '-lobjc'
          ld: Undefined symbols:
            _RustObjCExceptionTryCatch, referenced from:
                objc_exception::try_no_ret::hb6c5f46d0765cd58 in libtao-b2a7c9fe6e7760b2.rlib[7](tao-b2a7c9fe6e7760b2.tao.44e5656fbb1da1ec-cgu.04.rcgu.o)
                objc_exception::try_no_ret::heb888d6999c3a1ab in libtao-b2a7c9fe6e7760b2.rlib[7](tao-b2a7c9fe6e7760b2.tao.44e5656fbb1da1ec-cgu.04.rcgu.o)
                objc_exception::try_no_ret::h467675c166da1eb2 in libcocoa-bdc083df61ae20c1.rlib[5](cocoa-bdc083df61ae20c1.cocoa.9d90b2958afad0f3-cgu.2.rcgu.o)
                objc_exception::try_no_ret::ha07b8ef05ba8efa8 in libcocoa-bdc083df61ae20c1.rlib[5](cocoa-bdc083df61ae20c1.cocoa.9d90b2958afad0f3-cgu.2.rcgu.o)
                objc_exception::try_no_ret::hd9a8b230a39d73a6 in libcocoa-bdc083df61ae20c1.rlib[5](cocoa-bdc083df61ae20c1.cocoa.9d90b2958afad0f3-cgu.2.rcgu.o)
                objc_exception::try_no_ret::he0d4cb60d8d7f302 in libcocoa-bdc083df61ae20c1.rlib[5](cocoa-bdc083df61ae20c1.cocoa.9d90b2958afad0f3-cgu.2.rcgu.o)
                objc_exception::try_no_ret::he239be2eac779649 in libcocoa-bdc083df61ae20c1.rlib[5](cocoa-bdc083df61ae20c1.cocoa.9d90b2958afad0f3-cgu.2.rcgu.o)
                ...
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

Switching to Mach-O LLVM linker (aka ld64.lld) doesn't help, and returns a similar error:
note: ld64.lld: error: undefined symbol: RustObjCExceptionTryCatch
          >>> referenced by lib.rs:44 (/Users/redacted/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc_exception-0.1.2/src/lib.rs:44)
          >>>               /Users/redacted/wrkspc/bevy_sandbox/target/debug/deps/libtao-b2a7c9fe6e7760b2.rlib(tao-b2a7c9fe6e7760b2.tao.44e5656fbb1da1ec-cgu.04.rcgu.o):(symbol objc_exception::try_no_ret::hb6c5f46d0765cd58+0xb8)
          >>> referenced by lib.rs:44 (/Users/redacted/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc_exception-0.1.2/src/lib.rs:44)
          >>>               /Users/redacted/wrkspc/bevy_sandbox/target/debug/deps/libtao-b2a7c9fe6e7760b2.rlib(tao-b2a7c9fe6e7760b2.tao.44e5656fbb1da1ec-cgu.04.rcgu.o):(symbol objc_exception::try_no_ret::heb888d6999c3a1ab+0xa4)
          >>> referenced by lib.rs:44 (/Users/redacted/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc_exception-0.1.2/src/lib.rs:44)
          >>>               /Users/redacted/wrkspc/bevy_sandbox/target/debug/deps/libcocoa-bdc083df61ae20c1.rlib(cocoa-bdc083df61ae20c1.cocoa.9d90b2958afad0f3-cgu.2.rcgu.o):(symbol objc_exception::try_no_ret::h467675c166da1eb2+0xb8)
          >>> referenced 17 more times
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

This occurs on both release and debug builds. I tried downgrading wry and tao from their latest versions (0.36 and 0.25 respectively) but still ended up with the same variation of the error
Additional information
I opened a parallel issue in the wry repo (here), as I can't tell which project is actually causing this.

I don't think this is wry, because it's not referenced in the error nor imported by the program. I think this is an issue with tao, which raises an exception when linking to MacOS's Cocoa library. objc_exception also appeared a few times, so it may be related to that.
I'm going to try testing this locally.

This is an issue with Bevy's dynamic_linking feature. If you turn it off, it will no longer fail to compile. This error requires toa, wry, and bevy. (Though only bevy and toa need to be imported.) I'm not sure what's going on, but turning off dynamic linking should fix the issue.

That's very weird. I don't know enough about the libraries (+ Objective C) in question to be able to debug it myself, but let's leave the issue open.

I'm also running into this on a M3 MBP after updating from Tauri 1.6.7 -> 2.0.
Bevy 0.14.2 with dynamic linking working fine with Tauri v1.6.7, after upgrading now a very similar error is showing up:
  = note: ld: warning: ignoring duplicate libraries: '-lSystem', '-lc', '-lobjc'
          Undefined symbols for architecture arm64:
            "_rust_objc_sys_0_3_try_catch_exception", referenced from:
                objc2::exception::try_no_ret::h098343831220d03c in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                objc2::exception::try_no_ret::h0bc559dde5ecde4c in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                objc2::exception::try_no_ret::h11805ba2fc040271 in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                objc2::exception::try_no_ret::h191f290ef3fdd9a2 in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                objc2::exception::try_no_ret::h1b50c4034aadaddb in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                objc2::exception::try_no_ret::h22ce5d32274220da in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                objc2::exception::try_no_ret::h27efcd6461c4a597 in libwry-c9bb8e20141e11ee.rlib[17](wry-c9bb8e20141e11ee.wry.cb4c6861d758e4d1-cgu.14.rcgu.o)
                ...
          ld: symbol(s) not found for architecture arm64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

ld: symbol(s) not found for architecture arm64

This makes me think one of the libraries doesn't support the ARM chips (M1, M3, etc.). Maybe something's Intel-only?


Enhance reliability of zero-downtime deployments

Remove custom graceful shutdown code
Start using spring native graceful shutdown - https://www.baeldung.com/spring-boot-web-server-shutdown

server.shutdown=graceful
spring.lifecycle.timeout-per-shutdown-phase=1m

enhance start script to optionally sleep before executing SIGTERM handler's kill command

PR: https://github.com/NationalSecurityAgency/skills-service/pull/474


Add Aerial theme

I ported a very small theme. I hope that it isn't too small to add. Here is the repo https://github.com/sethmacleod/aerial
Cheers,
Seth

Hello
thank you very much for sharing this theme with the Hugo community. Let's take the last steps before merging your theme:

please make sure the screenshots in the images folder have the required dimensions
the build script is a bit pendantic and expects at least (an empty) static and content folder inside the exampleSite directory. To make them trackable an empty file, e.g. .gitkeep.

Regards,
Digitalcraftsman

OK, I redid the screenshots and created the folders. fingers crossed this time.

Everything looks fine. :+1:


dynamic generation form

Subject of the issue
use ajax to request some data,
then Rendering it to a Form template with hogan.js(http://twitter.github.io/hogan.js/)
it means that the Form is dynamic generation.
$('#form-name').validate()
now it's invalid .
Your environment

jquery                  :    1.12.0
jquery-validate    :    1.16.0
chorme                :    59.0.3071.115

Are you calling $('#form-name').validate() before or after your form was added to the page?
Normally, you should call it after so that the plugin can find your form and initialize it for validation.


freeze on path highlighting when directory have many files

I've found, that zsh-syntax-highlight freezes zsh when I trying to enter some path, when last entered (before the freeze) directory contains many files (i.e. even ls thinks for some time before show the files).
Also, it seems, there is no way to tell the highlighter to don't try to not perform any operations on that dir (I guess, it is somehow related to the check if last path's part is existing file or not) and just highligt it in some predefined way...

I'm guessing it's just being slow, not actually freezes, and this is a duplicate of #379.

Yeah, thanks, it is. I was just unable to find it by any of keywords I tried :)

Cheers.  Any chance you'd like to help with the implementation?  (We're also on #zsh-syntax-highlighting on freenode)

@msva If you get a chance, feedback on #496 would be appreciated.

Well, it looks good for me :) But I guess, it's only @nicoulaj who can merge it. Isn't he?

by the way, I guess, if you sure that new features (like ${:|} and :P) have better performance, you can add $ZSH_VERSION check (but it's just my humble opinion and can be unacceptable here. I'd anyway wait for @nicoulaj comments on your PR...

@msva: @phy1729 and I have push access as well :-)


[SPARK-22638][SS]Use a separate queue for StreamingQueryListenerBus

What changes were proposed in this pull request?
Use a separate Spark event queue for StreamingQueryListenerBus so that if there are many non-streaming events, streaming query listeners don't need to wait for other Spark listeners and can catch up.
How was this patch tested?
Jenkins

Test build #84274 has started for PR 19838 at commit 60035fa.

Test build #84274 has finished for PR 19838 at commit 60035fa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84274/
Test PASSed.

LGTM but I have very limited context on this codepath. Maybe @tdas can also take a very quick look?

Don't you mean "thread" instead of "query" in your summary?

@vanzin yeah. Good catch. Just a typo.

Test build #84319 has started for PR 19838 at commit 9b4ce99.

Test build #84319 has finished for PR 19838 at commit 9b4ce99.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84319/
Test PASSed.

LGTM.

Thanks! Merging to master.


Proposal to use pre-rendered content in tessellate-fragment

Currently when the tessellate-fragment gets a request, it

fetches sources (HTTP)
fetches bundle (HTTP)
fetches content (HTTP)
renders the fragment by loading the bundle (js) and content (JSON) in server memory and uses node virtual machine to run the bundle script.

This requires resources from the server and the same steps are taken for every single request. When there are multiple requests per second, this does not scale.
A proposal to resolve this is to have a pre-rendered fragment storage. tessellate-fragment could (by configuration) look up for a pre-rendered fragment by the URL and language key. If there is a hit, the HTML can be sent immediately to the client skipping all the above mentioned steps. For popular URLs the performance gains are significant and clients can enjoy faster response times.

@mfellner Would really like to hear your ideas regarding this.

This is certainly a useful optimisation we should implement. Just to be clear: we're simply talking about caching here. Where "simply" means it's actually really complicated, since it's caching.
First let's look at all the steps where we might add a cache:

fetch resources for rendering

we can use ETags for caching
we can use a faster resource store

render HTML from resources and context

we can cache the rendered html

Intuitively 1 should be slower since the network is usually slower than CPU time.
In that case we can optimise by adding ETag based caching to tessellate-request. The cache itself should probably be in memory or on disk - ideally we'd use a tiered cache with two layers (a networked cache doesn't make sense). Maybe node-cache-manager could help for example. Another idea would be to make the storage at least for some resources (e.g. bundles) faster. E.g. by using Redis or by moving it onto the same physical node as the fragment-service.
Now, server-side rendering with React is slow too. Instead of caching rendered results we could also look into alternative, faster renderers. But caching is probably the bigger win. There are some things to consider here:

rendering uses multiple inputs:

bundle.js
content.json
HTTP request context
potentially data fetched by React components

cache-invalidation can happen in two ways:

client-based (fragment client needs to send invalidation information)
server-based (fragment checks for new resources and re-renders or not)

cache can be multi-tiered:

memory
disk
redis

cache must be thread-safe and scalable

now that the fragment runs multiple processes, we might write to the cache on two different processes in parallel
memory/disk would be shared by processes on 1 node, redis would be shared by multiple nodes

Those are just my initial thoughts, maybe I forgot something important. What is everyone else's opinion?

render HTML from resources and context
* we can cache the rendered html

The initial proposal is about caching just the rendered content. Caching bundles, content and sources adds more complexity while providing small(er) performance benefits. A big bottleneck is the rendering phase.
I don't think we need to decide which cache will be implemented, caching capability could be activated by a configuration. For us https://aws.amazon.com/elasticache/ seems like a good choice, but this should not be hardcoded into the application code.
Maybe some code clears my thoughts:
return router.get('fragment', '/fragment', async ctx => {
    if(cacheOn) {
      html = getCacheProvider().getOrCreate(headers, query)
    } else {
      html = renderTheSlowWay(headers, query);
    }
  ...
  });

CacheProvider would be an interface for the cache. Deployed services can the use whatever cache is fit for them.


Does not display transactions in braintree but API CALL succeeed in paypal developer

Please try the latest version and reopen if the issue persists.


Cropping Box Units

Hi @metavibor,
Do you think the units of the box in the "Crop..." function should rather be pixels or calibrated?

Done. One can now choose between voxel and calibrated units.


docs: add Vercel AI SDK

This PR adds the Vercel AI SDK to:

the clients page (as a JavaScript / TypeScript library)
to the Basic RAG Guide page

Thank you! Merged!


quick pass at UT for naive cache strategy

also correct a copy-n-paste bug in the docstring of max

ran across the lack of naive testing during the course of reading/learning the code and wrote it up as part of that.

@SEJeff what do you think of this?
@ross I really appreciate your contribution, but it would be better to target the master branch. Would you mind resubmitting this patch against master?

Looks totally reasonable to me. Not the biggest fan that the result ordering is non-deterministic and the way the test has to be written due to that, otherwise looks great.
@ross: thank you for doing this. We all hack on graphite in our very precious free time and contributors who add tests "just because" pretty much make my day. However, would you mind opening a PR against master please? We are very very soon going to stop accepting contributions when the next 0.9.x release is cut.

@SEJeff I'm not fond of merging anything "new" into 0.9.x at this point, but if you want to push the button I'll :eyes: the other way. :wink:

Sorry, the notification email arrived when I was traveling and was dropped until now. Took a quick look at master and I'm not sure the changes here make sense any more. The strategy stuff has been reworked and it looks like the 3 of them now have tests. Lmk if I'm missing something. Otherwise I guess this can be closed/abandoned.

No I entirely agree. @ross would you mind please redoing this PR against the master branch?
We are always down for more tests, please do send us more :)
It is just that we are trying to severely limit merging stuff to master so that we can release 0.9.13 and then put the 0.9.x branch on ice Han Solo style.

No I entirely agree. @ross would you mind please redoing this PR against the master branch?

I looked at it, but the tests I'm adding here are already there so there's no reason to.

No worries, gonna go ahead and close this out. Thanks for the contribution anyways. :heart: :zap: :beers:


removed some irrelevant steps from odo create and cleanup

What type of PR is this?
/kind cleanup
What does does this PR do / why we need it:
Removed the Checking devfile compatibility stage from odo create as it does nothing for the time being.
Which issue(s) this PR fixes:
No issue
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[X] I have read the test guidelines

How to test changes / Special notes to the reviewer:
Now
$$ odo create java tttt
Validation
 ✗  Checking devfile existence [25365ns]
 ⚠  Devfile component type java is not supported, please run `odo catalog list components` for a list of supported devfile component types

Before
$$ odo create java tttt
Validation
 ✗  Checking devfile existence [22940ns]
 ✗  Checking devfile compatibility [9417ns]
 ⚠  Devfile component type ls is not supported, please run `odo catalog list components` for a list of supported devfile component types

Its funny that we are trying to check compatibility of a devfile which doesn't exist

/lgtm

/approve

/retest


Recipient Variables to batch emails?

I'm trying to send batch emails that would make use of variables, such as %recipient.firstname% etc, as described here.
However, I can't get it to work, since apparently this isn't supported anywhere yet. Is there any chance this could be implemented?

It should work fine like so...
var params = {
  to: ['Person1<EMAIL_ADDRESS>  bcc: ['Person2<EMAIL_ADDRESS>  from: 'Me<EMAIL_ADDRESS>  subject: 'Hey, %recipient.first%',
  text: 'testing message for mailgun',
  html: 'HTML version of body'
};

params['recipient-variables'] = {
<EMAIL_ADDRESS>{
    first: 'Person1Name'
  },
<EMAIL_ADDRESS>{
    first:"Person2Name"
  }
};

mailgun.messages().send(params, function(err, body) {
  if (err) console.dir(err.message);
  console.dir(body);
});

Hi,
I can't figure out the way to send thousands of mails at once with batch sending. I've tried with the API from mailgun documentation, but I don't see how to prepare my batch sending, how to split the values, how to consctruct my JSON the right way...
If someone had an accurate example, it would be great, because I can't find anything on the net.
Thank you guys,

Hello there is complete documentation here.
Sample code:
recipientVars = {
<EMAIL_ADDRESS>{
    first: 'Bob',
    id: 1
  },
<EMAIL_ADDRESS>{
    first: 'Alice',
    id: 2
  }
};

var data = {
  from: 'Excited User<EMAIL_ADDRESS>  to<EMAIL_ADDRESS><EMAIL_ADDRESS>  subject: 'Hey, %recipient.first%',
  'recipient-variables': recipientVars,
  text: 'If you wish to unsubscribe, click http://mailgun/unsubscribe/%recipient.id%',
};

mailgun.messages().send(data, function (error, body) {
  console.log(body);
});

Nice that recipient variables use percentages and regular variables require mustache/handlebar variables.


Failed to sync binlog

Running some methods of TdSharp results in below error which is actually related to tdlib
Here is the info in the debug console after calling TdLib.TdClient.CheckDatabaseEncryptionKeyAsync:
[Client.cpp:278][&td_requests]   Begin to wait for updates with timeout 0.100000
[Td.cpp:3072][#1][!Td][&td_requests]     Receive request 3: checkDatabaseEncryptionKey {
  encryption_key = bytes [0] { }
}
[Binlog.cpp:419][#1][!Td][&status.is_error()]    Failed to sync binlog: [WindowsError : Insufficient system resources exist to complete the requested service. : 1450 : Sync failed]
(process 5840) application exited with code -1073740791.

The method calls a method of tdjson with the same name. Then error occurs.
I'm using Tdlib v1.7.9 and TdLib.Native 1.7.9 which are .net bindings for tdlib. I'm using these on x64 windows 11 Pro. The project is built over .Net 5 for web (Actual target OS is windows server 2019)
I double checked system resources. There was only 15% CPU usage and almost 3 Gb of RAM.
I also disabled anti-virus (i found out it might be the cause) with no luck!
It there something im missing?

This is an error, returned by FlushFileBuffers. It should mean that file system resources are exhausted. Could you check nonpaged kernel memory as described in https://docs.microsoft.com/en-us/troubleshoot/windows-server/backup-and-storage/backup-program-unsuccessful-large-system-volume#symptoms?

@levlam
Kernel paged memory 749 Mb
nonpages 760 Mb
values change since PagedPoolSize registry is equal to zero. I've tried solutions described in the page you mentioned. No luck!
is there any way to cancel the whole backup thing. i dont need to store anything but session.

This is unrelated to backups. It fails exactly when session data is flushed to disk.

I expect that some other program extensively writes to disk.


upgrade the base OS to Debian Stretch latest minor release for Erlang…

… 20 and 19
fixes #114

looks like had a travis building failure; will fix together with #117


2 commit

A Pull Request should be associated with an Issue.

We wish to have discussions in Issues. A single issue may be targeted by multiple PRs.
If you're offering a new feature or fixing anything, we'd like to know beforehand in Issues,
and potentially we'll be able to point development in a particular direction.

Related issue: https://github.com/github/gh-ost/issues/0123456789

Further notes in https://github.com/github/gh-ost/blob/master/.github/CONTRIBUTING.md
Thank you! We are open to PRs, but please understand if for technical reasons we are unable to accept each and any PR

Description
This PR [briefly explain what it does]

In case this PR introduced Go code changes:

[ ] contributed code is using same conventions as original code
[ ] script/cibuild returns with no formatting errors, build errors or unit test errors.

commit2


Completed ID4 Challenge

confirmed:ID4
AccountAddress:2xjgkY6v7sBv72tfGvi3Z97mnze6hSLnU3mKRgaJMgSQB2hwkv
DeviceDetails:verified

confirmed:ID4
AccountAddress:2xjgkY6v7sBv72tfGvi3Z97mnze6hSLnU3mKRgaJMgSQB2hwkv
DeviceDetails:verified


ci: upload and publish JUnit test results for conformance-multi-pool

This commit uploads and publishes JUnit test results of the connectivity tests in the conformance multi pool IPAM workflow.

/test


How to debug?

Following your information to run it in docker-compose works. However I would like to know your setup to develop in the template. Do you run standalone SQL, Mongo and Redis (docker) servers and connect to it or can the existing docker-compose configuration be leveraged?

Hey! In Visual Studio 2022:


Resource classes review

A list of miscellaneous issues seen when going through Resource classes. Should probably be broken up if there is agreement.

Is there any reason this method eagerly loads the users? There is a comment in there that it is doing it on purpose, but I see no use of it in the UI.
Is the lack of a valid Git url an error condition? Just wondering if this should be failing silently vs. returning an error.
Comment refers to workflow instead of container (ridiculously trivial). :)
In DockerRepoResource, several of the Swagger annotations say, for example, "Get the corresponding Dockerfile on Github.". Messages should not be GitHub specific, given other version control systems Dockstore supports.
The findToolByIdAndCheckToolAndUser method will throw a SC_FORBIDDEN error, but some methods that call it should be returning a 404 instead, e.g., here
The word users needs an apostrophe.
In UserResource, there are several references in the Swagger annotations to "logged-in" user. Should probably reword, as logging in is more of a UI concept, and the API works with tokens. Not sure what to say though.

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-657

Is the lack of a valid Git url an error condition? Just wondering if this should be failing silently vs. returning an error.

This might be a tricky one, IIRC, it would be nice to return an error if this was a single tool.
But this code might be re-used for batch refreshes and we don't want to abort the refreshing the whole (say, 20 tools in an org) because one tool couldn't refresh.
A superior solution now that we have validation messages might be to report refresh issues that way

Is there any reason this method eagerly loads the users? There is a comment in there that it is doing it on purpose, but I see no use of it in the UI.

@garyluu  might recall, it was introduced with https://github.com/ga4gh/dockstore/commit/5fb31336b608713655ce725fcf08a4db23153ad3

Originally all users were eagerly loaded.  I then made all of them lazy loaded.  Then I decided if any tools were going to return the users, it should be when we grabbed a single tool...otherwise we might as well jsonignore the entire thing.

The question is if anything uses users though, I can see pulling down starred users to populate the number of stars a tool has and/or the stargazers page.
I see less value in the users that own a workflow though, it might be worth taking it out and seeing what breaks


[BUG] CodeGlancePro is Visible as a Small Vertical Bar in PyCharm Python Console

Which Glance version and IDEA version?

CodeGlancePro 1.6.0-2023.1

Could you show Glance config picture and minimap picture?

What steps will reproduce the issue?

Open Python Console in PyCharm and move the cursor around the text input area. A minimap will appear as a thin vertical bar to the right of the input marker.

Wait until PyCharm 2023.1 is released to see this issue

It should be an official issue


Remove version: Nikkho.FileOptimizer version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/nikkhokkho/files/FileOptimizer/15.90.1730/FileOptimizerSetup.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


Add the Scorecard Action to monitor procfs' security posture

Hey, it's Pedro (see https://github.com/prometheus/common/pull/490 and https://github.com/prometheus/prometheus/pull/12841). I'm back with another security suggestion.
I found the issues fixed by those PRs by scanning procfs with Scorecard. It looks at a repository's settings and configurations to identify potential points of improvement in a project's security posture.
As it happens, the first issues I worked on involved workflows, which procfs pulls from /common and/or /prometheus, so I sent the PRs over there.
Scorecard is also available as a GitHub Action. It then monitors a project's security posture and populates the Security Panel with any tips it may find relevant for the project. In doing so, it can also flag whenever a code or setting change accidentally weakens the project's security.
procfs' current score is 6.8/10, which places it at the top 10% of projects important to the open-source ecosystem.
I'll write a PR implementing the Action and send it along with this issue.

If we're going to include this action, we should add it to https://github.com/prometheus/prometheus/blob/main/scripts/sync_repo_files.sh. This way it's automatically managed in all Prometheus project repos.

Sorry for the delay here, but sure thing, I'd be happy to send this to prometheus/prometheus and add it to the sync script.
I do have one question though. After one of my previous PRs, I realized there's automation keeping things in sync with prometheus/prometheus but there's also prometheus/common, which I'd understood was the container for all "common" things in prometheus projects.
Has /common been deprecated, with /prometheus now being the source-of-truth for common files?


Not able to add custom icons using ClarityIcons.add

ClarityIcons.add({"myLogo": '<svg version="1.1" viewBox="0 0 36 36" preserveAspectRatio="xMidYMid meet" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">		< title > zoom - out < /title>< path d="M16,4A12,12,0,1,0,28,16,12,12,0,0,0,16,4Zm0,21.91A10,10,0,1,1,26,16,10,10,0,0,1,16,25.91Z" class="clr-i-outline clr-i-outline-path-1" /><path d="M31.71,29.69l-5.17-5.17A13.68,13.68,0,0,1,25.15,26l5.15,5.15a1,1,0,0,0,1.41-1.41Z" class="clr-i-outline clr-i-outline-path-2" /><path d="M20,15H12a1,1,0,0,0,0,2h8a1,1,0,0,0,0-2Z" class="clr-i-outline clr-i-outline-path-3" /></svg>'});
Ideally it will be good to link an external file here.
Select one ...  (check one with "x")
[x ] bug
[ ] feature request
[ ] enhancement

Expected behavior

this should display the Logo svg
Actual behavior

Error in the browser console 'myLogo' is not found in the Clarity Icons set.
Reproduction of behavior

https://plnkr.co/edit/2NzkpsExVOOHBg6QYYXy?p=preview

Environment details

Angular version: 2.0.X

Clarity version:

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

@tarundhillon I looked at the Plunkr and it looks like you placed your application logic in the AppModule, which is solely responsible for bootstrapping your application. So you would want to place it in the root component class. Also, It needs to be in a constructor or method to run.

Unfortunately, I was not able to get this solution to work with Angular 4. Here is a fork of the Clarity seed repo with the svg used in the documentation. I am able to console.log the data from the svg immediately after adding it with the api, but in trying to access it in the template using the clr-icon directive it throws the error 'my-first-custom-icon' is not found in the Clarity Icons set. Any help would be much appreciated!

I made a plunkr out of the code in the fork and it works.

Thank you @mathisscott for your quick response! The issue I am experiencing is when I run the app with the angular cli using ng serve. I can perhaps try to modify it to run locally with AOT.

@Shijir may be able to help you figure out the issue. We have clarity-icons running with both ng serve and AOT, iirc.

Okay, thank you. clarity-icons works wonderfully, it's just adding custom icons that is the problem.

The problem you are facing could happen when you are running two instances of ClarityIcons. If you are importing and loading ClarityIcons in your typescript, make sure you are not loading it through script tag too.

It works for me with on Angular4, I have approached it as follows

Define a service
`
import { Injectable } from '@angular/core';
@Injectable()
export class IconService {
constructor){}
icons : any = {
"appLogo": '',
"appLogoWithText": ''
};
public load(){
window['ClarityIcons'].add(this.icons);
}

}
`

Call the load at an appropriate point, for it was when a component is being loaded

  constructor(private iconService: IconService) {         iconService.load();

in the html
<clr-icon [attr.shape]="appLogo" size='48'>

@Shijir, thank you, I removed the line from the angular-cli.json scripts section that includes clarity-icons.js. It works! Very cool.


Footer does not appear on bottom of pages that do not require scrolling

The footer sticks to the bottom of pages that you have to scroll to, but does not stick to the bottom of pages that take up less room than the window on your browser. Compare the parties page to the data page, for example.
I tried to fix this in base.tpl some time this summer but I couldn't make it work.

The reason the footer triggers on scroll was to prevent a flash of the footer before the main content loads on pages that use ajax loading for the content--so I definitely think we shouldn't fix this by disabling the footer trigger.
But yes, I see that it does not correctly load on non-scrolling pages that don't have an ajax load. Thanks for the report on this. I'll take a stab at it.

Explicitly include /static/js/footerFixed.js in pages that have no ajax / page scroll and this will be fixed.

Doesn't force it to bottom on short pages, though?


2.2.2 performance issues

In upgrading from 1.8.4 to 2.2.2 our build went from 1 min 49 seconds to 2 min 53 seconds. It also required me to increase the memory allocated to ant (it was 1024 and is now 2048.)
That's quite an increase in building exactly the same content.

A lot has happened between 1.8 and 2.2, not just DITA 1.3 but also refactoring the preprocessing code to make it easier to test and reason about.
If you're interested in helping us out, it would help to get some statistics which areas have slowed down. If you could run your source with both 1.8 and 2.2 and use
ant <normal-arguments> -logger org.apache.tools.ant.listener.ProfileLogger -l test.log

to collect processing time information to the test.log. Then, send the logs to us. If the processing log contains information you can't share, you can do
grep -F 'ms)' test.log

To just extract the lines that actually contain timing information.

The difference might also vary quite a bit depending on which DITA features the document uses and possibly which transtype you're using.
I'm guessing that 2.2 simply having to parse the DITA 1.3 DTDs explains some of the difference, though.
Here's some benchmark results for running just the preprocessing stage on the DITA-OT user guide on DITA-OT 1.8.4 vs DITA-OT 2.2:
https://s3.amazonaws.com/f.cl.ly/items/15150N0s321j2h2a1g1A/preprocess.html
The mean difference is around 4 seconds.
Incidentally, reusing JVM instances helps quite a bit here, too. This is for publishing XHTML output for the same document with 1.8.4 vs 2.2 with the DITA-OT Gradle plugin:
https://s3.amazonaws.com/f.cl.ly/items/2z151t2a1w3O1r3Z1p0v/dita-ot-gradle.html
That takes the difference down to 0.7 seconds for the entire processing.

1.3 doesn't just mean we use different DTDs. For 1.3 we have to have an additional step to resolve same topic fragments, we have to read key definitions separately from initial parse step, we have to check for branch filtering even if we don't actually have any branch filtering in source, etc. So there's a lot of new stuff that makes things slower. But, without exact number for what actually takes more time, it's difficult to try to optimize.

Marking as obsolete - there have been many changes since this issue came in so there is no reason to compare to 1.8 or 2.2.


MsgPack 1.0.0 throws ExtendedTypeObject is prohibited in this packer.

I have an object that doesn't extend from anything yet when I updated from 0.9.2 to 1.0.0 I get "ExtendedTypeObject is prohibited in this packer." when trying to pack an object. My object isn't anything fancy just strings, ints, and datetimes.
I am using .net 4.7.2 and below is how I am trying to pack an object.
var type = value.GetType();
var typeName = WriteCache.GetOrAdd(type, SubtractFullNameRegex.Replace(type.AssemblyQualifiedName, string.Empty)); // Get type or Register type
using (MemoryStream memoryStream = new MemoryStream())
{
    Packer packer = Packer.Create(memoryStream);
    packer.PackArrayHeader(2);
    packer.PackString(typeName);
    try
    {
        MessagePackSerializer.Get(type, DefaultContext).PackTo(packer, value);
        return new ArraySegment<byte>(memoryStream.ToArray(), 0, (int)memoryStream.Length);
    }
    catch (Exception ex)
    {
        Logger.LogError("MessagePackMapTranscoder", ex, new Dictionary<string, object>
        {
            ["TypeName"] = typeName
        });
        throw;
    }
}

Here is the object I am trying to pack.
[Table]
public class Company
{
    [Column(IsPrimaryKey = true, IsDbGenerated = true)]
    public int Id { get; set; }

    [Column]
    public string Name { get; set; }

    [Column]
    public string Address { get; set; }

    [Column]
    public string City { get; set; }

    [Column]
    public string State { get; set; }

    [Column]
    public string Zip { get; set; }

    [Column]
    public string PhoneNumber { get; set; }

    [Column]
    public DateTime Created { get; set; }

    [Column(IsVersion = true)]
    public DateTime Updated { get; set; }

    [Column(IsVersion = true)]
    public int TimeStamp { get; set; }

    [Column]
    public string TemplatedLogoUrl { get; set; }

    [Column]
    public string UrlName { get; set; }
}

Stacktrace of exception
   at MsgPack.Packer.ThrowExtTypeIsProhibitedException()
   at MsgPack.Packer.PackExtendedTypeValue(MessagePackExtendedTypeObject mpeto)
   at MsgPack.Serialization.DefaultSerializers.MsgPack_MessagePackExtendedTypeObjectMessagePackSerializer.PackToCore(Packer packer, MessagePackExtendedTypeObject value)
   at MsgPack.Serialization.MessagePackSerializer`1.PackTo(Packer packer, T objectTree)
   at MsgPack.PackerUnpackerExtensions.PackCore[T](Packer source, T value, SerializationContext context)
   at MsgPack.PackerUnpackerExtensions.Pack[T](Packer source, T value)
   at MsgPack.Serialization.DefaultSerializers.TimestampDateTimeMessagePackSerializer.PackToCore(Packer packer, DateTime objectTree)
   at MsgPack.Serialization.MessagePackSerializer`1.PackTo(Packer packer, T objectTree)
   at MsgPack.Serialization.PackHelpers.PackToMap[TObject](PackToMapParameters`1& parameter)
   at MsgPack.Serialization.EmittingSerializers.Generated.MyNameSpace_CompanySerializer1.PackToCore(Packer , Company )
   at MsgPack.Serialization.MessagePackSerializer`1.InternalPackTo(Packer packer, Object objectTree)
   at MyNameSpace.MessagePackMapTranscoder.SerializeObject(Object value) in MessagePackMapTranscoder.cs:line 191 (THIS IS MY CODE)
   at MyNameSpace.MessagePackMapTranscoder.Serialize(Object value) in MessagePackMapTranscoder.cs:line 63 (THIS IS MY CODE)
   at Enyim.Caching.MemcachedClient.PerformStore(StoreMode mode, String key, Object value, UInt32 expires, UInt64& cas, Int32& statusCode) (THIS IS MY CODE)

It looks caused by changing default date time serializer between 0.9 to 1.0. This change was intended to improve interoperability for other languages which uses ext-typed timestamp for datetime like values.
You can avoid this error by one of following:

Allow ext type in the context. Namely, SerializationContext.Default.CompatibilityOptions.PackerCompatibilityOptions &= ~PackerCompatibilityOptions.ProhibitExtendedTypeObjects
Change default timestamp serializer to previous one by SerializationContext.Default.DefaultDateTimeConversionMethod = DateTimeConversionMethod.Native

I will also add context options to change behavior to 0.9 for compatibility in 1.1.


Provide exit codes with the runnable jar file version

The runnable jar-file version should provide system exit codes. At present it never terminates. However it should provide codes:

On normal completion: 0
On error: 1

You can provide more granular error exit codes if needed. But this would do for now.

Thanks for pointing this out!

@scandox
When it's terminated, do you need the "On normal completion: 0" message always get printed or only print it when --verbose is specified?

Printed output, apart from cameras, only with verbose. Thanks
On 24 Jul 2015 14:30, "Liuting Du"<EMAIL_ADDRESS>wrote:

@scandox https://github.com/scandox
When it's terminated, do you need the "On normal completion: 0" message
always get printed or only print it when --verbose is specified?
—
Reply to this email directly or view it on GitHub
https://github.com/evercam/evercam-discovery-java/issues/8#issuecomment-124520941
.

Reading over your email I suspect you may have misunderstood my original.
I don't need a printed message at all. I need an exit code passed to the
OS.
As
http://steve-parker.org/sh/exitcodes.shtml
Printed output, apart from cameras, only with verbose. Thanks
On 24 Jul 2015 14:30, "Liuting Du"<EMAIL_ADDRESS>wrote:

@scandox https://github.com/scandox
When it's terminated, do you need the "On normal completion: 0" message
always get printed or only print it when --verbose is specified?
—
Reply to this email directly or view it on GitHub
https://github.com/evercam/evercam-discovery-java/issues/8#issuecomment-124520941
.

I've made it exit with exit code 1/0 for failure/success by using
System.exit(1/0)

And when I run echo $?, it shows the exit code result. Is it what you are talking about?
I was just wondering do you need it to be printed as a logging message or not :) So the answer is no?

Oh sorry ok great. No logging needed as the exit code tells the story
On 24 Jul 2015 14:56, "Liuting Du"<EMAIL_ADDRESS>wrote:

I've made it exit with exit code 1/0 for failure/success by using
System.exit(1/0)
And when I run echo $?, it shows the exit code result. Is it what you are
talking about?
I was just wondering do you need it to be printed as a logging message or
not :) So the answer is no?
—
Reply to this email directly or view it on GitHub
https://github.com/evercam/evercam-discovery-java/issues/8#issuecomment-124530111
.


with karate-gatling dependency version 1.0.1 empty attributes are added to the soap request

It seems that using the following karate-gatling dependency (version 1.0.1)
<dependency>
    <groupId>com.intuit.karate</groupId>
    <artifactId>karate-gatling</artifactId>
    <version>${karate.version}</version>
    <scope>test</scope>
</dependency>

the request are not passed as it is but there's an empty attribute (xmlns) added the the soap request, making my tests failing. That is instead of:
<service>123456789</service>
the request sent is having an empty xmlns attribute:
<service xmlns="">123456789</service>
The problem is quite similar to the one reported in https://stackoverflow.com/questions/57042497/karate-xml-element-changed-to-have-empty-namespace/57042773.
In this case it must be something regarding the xml libraries handled by karate-gatling. With the version 0.96 of Karate (and the equivalent version for karate-gatling) it worked fine.
As attachment the code for replicating the problem, with a README. You have to run it without the dependency and check that it passes. Then decomment the dependency in the pom to see that it fails.
myproject.zip

I definitely don't consider this a priority, tagging as help wanted

@forlixdev here's a way you can help. I think there is a way to force the XML parser used by XmlUtils but it needs some research, e.g. https://stackoverflow.com/a/3316678/143475 - so it would be great if you can investigate, fix and submit a PR. does that make sense ?

Of course, I can do some research about it, even if being only a tester I don't know if I'm able to finalize it.
As you said is not a critical bug, only a bit annoying since it worked with the previous version.

@forlixdev no worries.  do try, any bit of research helps - it may be as simple as copying karate xml to string conversion code and trying it in the offending pom / project etc. we can try ask around for volunteers later

Hi, only to say that I tried but still without luck for now. I'll go on trying.

@forlixdev / @ptrthomas I can take a stab at this issue.

@dinesh19aug by all means 👍

Thanks @dinesh19aug. The last failed attempt I did was to try to use the internal implementation (in karate-core / XmlUtils class / toXmlDoc method):
String provider = "com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderFactoryImpl";
DocumentBuilderFactory factory =  DocumentBuilderFactory.newInstance(provider,XmlUtils.class.getClassLoader());

But the error was always the same mentioned in the issue.

moved to project board: https://github.com/intuit/karate/projects/3#card-61812894

please note that a possible fix has been attempted here: #1882 which may need some community help to complete

Taging senarios with new tag :
@namespaceAware=true
Doing this in ScenarioEngine.evalKarateExpression :
            Tags  tags = runtime.scenario.getTagsEffective();
            boolean namespaceAware =  tags.valuesFor("namespaceAware").isAnyOf("true");
            Document doc = XmlUtils.toXmlDoc(text,namespaceAware);

and this change in XmlUtils
 public static Document toXmlDoc(String xml,boolean namespaceAware) {
        DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
        factory.setNamespaceAware(namespaceAware);

could that be  be an ok solution ?

@PerJV I really appreciate you following up on this. I would add a config property called xmlNamespaceAware in this class and pass it from the ScenarioEngine: https://github.com/karatelabs/karate/blob/v1.2.0.RC1/karate-core/src/main/java/com/intuit/karate/core/Config.java
but before all that, I have these concerns:

this is a problem that happens only when some combination of JARs is on the classpath
I'm not willing to accept any PR without a test-case so that we understand the before-after
unless we understand what is the root cause, and possibly if there is some other XML config that can be force-set, I am not comfortable to proceed
there are many users depending on how the existing XML parsing works, I'm not convinced there is a need to change anything yet

My two cents from a pure user perspective is that if it's only me that I noticed this error, we might find a workaround for this edge case (writing it in the documentation).
Otherwise if the problem is a nagging one, it would be better to handle it in another way for not changing the default xml handling behaviour (that might have a potential big impact), for example changing the behaviour accordingly to a parameter tp be set in karate.config.

@forlixdev my bad, I completely missed that you had submitted a simple, clear way to replicate and could verify that the original solution proposed by @PerJV works
I have just implemented the solution. so adding this one line will solve the problem:
* configure xmlNamespaceAware = true

for completeness, this is also a workaround and way to submit XML without parsing it within Karate, note the use of text instead of def:
* text requestBody =
"""
<?xml version="1.0" encoding="UTF-8"?>
<S:Envelope xmlns:S="http://schemas.xmlsoap.org/soap/envelope/" xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/">
    <SOAP-ENV:Header/>
</S:Envelope>

Thanks for the work and support @PerJV and @ptrthomas

Thanks  @ptrthomas  for adopting this change this  really helped me a lot

1.2.0 released


Add gpu blacklist

Hadoop change
https://github.com/mzmssg/hadoop/pull/1/files
Known issue

when blacklist the higher bit GPU( i.e. black the third and fourth GPU on a 4-GPUs machine), the resouce view will change from 1111 to 11. Idealy, it should be 0011 .
Resource subtraction has bug, which lead YARN get wrong available resource.

TODO:
Docs about how to update GPU blacklist

Coverage remained the same at 51.687% when pulling 9ed323874749e41dbd46a88b71c8fb0fed43e7d7 on zimiao/gpu_blacklist into 621466250a66e6ded49368b886cd58b4838a63e7 on master.

Coverage remained the same at 51.687% when pulling 9ed323874749e41dbd46a88b71c8fb0fed43e7d7 on zimiao/gpu_blacklist into 621466250a66e6ded49368b886cd58b4838a63e7 on master.


Decoupling Wallet UI from Dapp UI - perhaps by returning the API via connect()

From the standard goals:

Decouple the state of the wallet UI from the state of the dapp UI
Wallets should be presented as stateless, isolated, multi-tenant applications to dapps.
When a dapp acts upon a wallet, it should have no effect on any other dapp that may interact with the wallet.
Similarly, when a user interacts with the wallet UI, it should have no effect on any dapp.
When a user changes accounts or networks in the wallet UI, their intent is to see their assets or act with them in the context of the wallet UI.
After a dapp has connected to a wallet and discovered an account, the dapp should be able to request to sign using that account specifically.
When this occurs, the wallet should display account and network changes to sign and send transactions for the dapp, and then return to its previous context.

On these thoughts, is it possible for us to perhaps call a function such as enable() or maybe even connect() that returns an object with an API that is strictly read-only? e.g. pseudocode for a dapp
const api = await wallet.enable(); or wallet.enable(); // this method can throw or return false to handle cases of rejected connections.
// read only methods are returned
api.getChain();
api.getIcon();
api.getAccounts();

// only way to interact is via submission of tx
api.sendAndConfirm();

// (if needed) object can become invalidated if there's a need to reauthorize connection.

Note this is a preliminary thought but I took some of the ideas from cardano's wallet's connections as well as some of eth's

This is largely implemented. Check out https://github.com/wallet-standard/wallet-standard/blob/master/packages/core/standard/src/wallet.ts --
The accounts field, and the connect method which returns accounts, contain WalletAccount objects. These are basically the API of the wallet when acting with an account. These accounts have features, which may be specific to the wallet's implementation and the chain.
And check out https://github.com/wallet-standard/wallet-standard/blob/master/packages/wallets/backpack/src/wallet.ts for how a wallet can wrap its existing API with the standard.

The API being per account means that an account with an empty features API is effectively a read-only account. The wallet can either expose these via wallet.accounts or an app could specifically request read-only access to accounts by calling wallet.connect({ features: [] }).

Closing as this is implemented.


[SBOM] Fix scanRequest type

What does this PR do?
This change fixes the following error:
2024-03-13 21:00:32 UTC | CORE | DEBUG | (pkg/collector/corechecks/sbom/processor.go:178 in processHostScanResult) | processing host scanresult: {invalid request type '*host.ScanRequest' for collector 'host' <nil> 2024-03-13 21:00:32.439957913 +0000 UTC m=+5.131362123 8.802µs <nil>}

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Can we make this log an error log ? I totally missed it

Can we make this log an error log ? Perhaps e2e tests could be useful as well.

Done.

/merge -c rebase

/merge -c

/merge -c rebase

/merge


Error with South migration on Postgres 8.4

Got the following when deploying it for Django 1.2.3, Postgres 8.4 (postgresql_psycopg2) and South 0.7.2. Works fine on my SQLite3 based development env.

Running migrations for mailchimp:
 - Migrating forwards to 0007_extra_info.
 > mailchimp:0001_initial
 > mailchimp:0002_added_queue
 > mailchimp:0003_fixed_template_id
Traceback (most recent call last):
  File "manage.py", line 11, in
    execute_manager(settings)
  File "/srv/project/venv/lib/python2.6/site-packages/django/core/management/__init__.py", line 438, in execute_manager
    utility.execute()
  File "/srv/project/venv/lib/python2.6/site-packages/django/core/management/__init__.py", line 379, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/srv/project/venv/lib/python2.6/site-packages/django/core/management/base.py", line 191, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/srv/project/venv/lib/python2.6/site-packages/django/core/management/base.py", line 220, in execute
    output = self.handle(*args, **options)
  File "/srv/project/venv/lib/python2.6/site-packages/south/management/commands/migrate.py", line 105, in handle
    ignore_ghosts = ignore_ghosts,
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/__init__.py", line 191, in migrate_app
    success = migrator.migrate_many(target, workplan, database)
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/migrators.py", line 221, in migrate_many
    result = migrator.__class__.migrate_many(migrator, target, migrations, database)
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/migrators.py", line 292, in migrate_many
    result = self.migrate(migration, database)
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/migrators.py", line 125, in migrate
    result = self.run(migration)
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/migrators.py", line 99, in run
    return self.run_migration(migration)
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/migrators.py", line 81, in run_migration
    migration_function()
  File "/srv/project/venv/lib/python2.6/site-packages/south/migration/migrators.py", line 57, in
    return (lambda: direction(orm))
  File "/srv/project/venv/lib/python2.6/site-packages/mailchimp/migrations/0003_fixed_template_id.py", line 12, in forwards
    db.alter_column('mailchimp_queue', 'template_id', self.gf('django.db.models.fields.PositiveSmallIntegerField')())
  File "/srv/project/venv/lib/python2.6/site-packages/south/db/generic.py", line 368, in alter_column
    flatten(values),
  File "/srv/project/venv/lib/python2.6/site-packages/south/db/generic.py", line 137, in execute
    cursor.execute(sql, params)
  File "/srv/project/venv/lib/python2.6/site-packages/django/db/backends/postgresql_psycopg2/base.py", line 44, in execute
    return self.cursor.execute(query, args)
django.db.utils.DatabaseError: column "template_id" cannot be cast to type smallint

This project will be maintained at https://github.com/piquadrat/django-mailchimp from now on, if this issue still exists, please report it again at the new location. Thank you


Enabling Halo2 via cargo feature.

I didn't find wise to disable the warnings when building with no backends: it is a reminder that outputing Pilcom JSON is kind of a backend, but it currently doesn't share the same API as Halo2.

What warnings do you mean?

I think halo2 should be enabled by default. It should be an explicit choice to only disable it during development.

I think halo2 should be enabled by default. It should be an explicit choice to only disable it during development.

Done.

Will GitHub allow merging with outdated base branch if there is no conflict?

What warnings do you mean?

Compilation warnings when halo2 is disabled.

Will GitHub allow merging with outdated base branch if there is no conflict?

just rebase


Show on token hover broken

https://ptb.discord.com/channels/915186263609454632/1167221795103985764/1225907509672546405

Fixed in 0.7.5.