test: browser.display.use_document_fonts

see https://github.com/pyllyukko/user.js/pull/300
@nodiscc namely browser.display.use_document_fonts = 0 breaks glyphs/icon fonts. Examples: some uBo icons, icons in imgur.
Test 1 (vanilla Firefox)

set fonts:browser.display.use_document_fonts = 0
set glyphs: gfx.downloadable_fonts.enabled = true
clear cache
visit http://terrillthompson.com/tests/icomoon/

Windows results:  I can see the play, forward and stop buttons. I need to test this with glyphs set to false, to make sure that they are blocked.
I also need to find out what nodiscc means with imgur (I have never been to the site, still haven't as I type). I do know that with =0 and =true, I have zero issues on windows with font awesome, as per the link on the first line.

On imgur I have no top left logo.

browser.display.use_document_fonts=0 no longer breaks icon fonts (including uBO icons) since Firefox 41, even though it still blocks the webpage from selecting standard fonts: see ghacks
Prior to FF 41, this in fact broke uBo icons, but that has changed: see this uBO issue.  I've had this pref set to 0 for a couple of years and uBO icons display just fine here (Windows)
On the other hand, gfx.downloadable_fonts.enabled=false will break icon fonts, including uBO icons.

Yup, that's clear. I started this issue because nodiscc has raised irregularities (which is good, it can uncover stuff like that maxRichResults). In FF53 (debian) nodiscc is missing uBo icons despite having the correct pref values. I cannot replicate it. Part of this issue is we probably need to find out how FF decides something is a "glyph" and also how sites initiate the font (aren't there at least two ways to do this? css and @font-face?).
I will set up a test proc and results and screenshots (nilla FF) for others to replicate - just like we did for maxRich. We'll find the cause. Tests will include with and without uBo. I'll get to it the next day hopefully, unless someone else does it before me

In FF53 (debian) nodiscc is missing uBo icons

FF 45.0.9 ESR

how FF decides something is a "glyph"

There is nothing special about it, it's just a character in the Common unicode table, or even a simple ASCII character. For example on http://imgur.com/ the "cloud icon" (New post button) is a simple \5d character (view-source:http://s.imgur.com/min/global.css?1495492255) which maps to a cloud icon in the character set of the Open Sans font they distribute (https://fonts.googleapis.com/css?family=Open+Sans:300,300i,400,400i,600,600i,700,700i,800,800i).

@nodiscc Yup. But Firefox has a special pref for glyphs, so my question is how does FF determine what it and isn't a glyph - this may explain some font icon behavior on various sites where what we see as an icon font, Firefox may not. But overall, "glyphs" as Firefox sees them are not blocked by document_fonts = 0.
And then we have this issue where your uBo is missing icons but I (and everyone else?) are not. I suspect this is a symptom of some linux or your profile quirk.

I think the difference is installed vs downloadable (or bundled for extensions) fonts. That's why there are 2 prefs afaik

In order to solve what's happening - eg a site does not display glyphs, we first need to understand how Firefox decides if it is a glyph. - THE bugzilla ticket: https://bugzilla.mozilla.org/show_bug.cgi?id=789788
Note: Reading thru that ticket and links, leads me to some ideas about nodiscc's broken icons in uBo
Problem:

Icons are broken due to font suppression (i.e browser.display.use_document_fonts = 0)

I will refer to these "icon-fonts" as glyphs.
I will refer to "web-fonts" as downloadable fonts.
If a font is on your system, it can still be a glyph
It's impossible to determine if a font is a glyph with 100% certainty (at least in FF with rendering and downloading and all that goes on - technically maybe possible, but not feasible)
"the web has a large amount of cesspool dwelling hacks and bullshit developers" - Thorin

Solution: gfx.downloadable_fonts.enabled = true
Implementation: browser.display.use_document_fonts = 0 blocks all fonts except those font families specified, but allows an override if it is determined that the font is a "glyph"
So HTF does FF work out what a glyph is? read on
Interference: @font-face blocks, SVG/EOT/TTF/WOFF blocks, XSS blocks etc
Note: It took them 3 and a 1/2 years to sort this out with a workable but not 100% solution

Proposed Solutions
5 years ago

glyphs should use the private-use area (PUA)

"We use the Private Use (Unicode) range to avoid conflicts with actual letters." - https://github.com/styleguide/css/7.0
"Font Awesome uses the Unicode Private Use Area (PUA) to ensure screen readers do not read off random characters that represent icons" - https://github.com/FortAwesome/Font-Awesome/blob/master/less/font-awesome.less

NOTE: lots (TONS) of glyphs do NOT use PUA codepoints for the icons - Thorin (cesspool and hacks)
Allow any downloadable font specified by the page when it's used for INDIVIDUAL characters, and use the user-specified fonts otherwise
A whitelist (and maybe blacklist)

This is both difficult to maintain and a bad experience for people writing content for the Web platform

Accept all downloadable fonts would fix it (while degrading/altering the use-own-fonts option a bit)

more than a bit; it would make the preference no longer do what some users need it to do - not an option

Add missing character fallback to the font load

I dont think it's desirable to individuate invalid characters (not feasible)

Generalise/ establish a naming convention/pattern for glyphs, eg. accept fonts named ".icon."
NOTE: someone mentioned that in their studies so far they found about 90% of glyphs had icon in the font name - thorin

but we cant foretell what will happen in the future as glyphs become more common

"allow_page_to_choose_TTF" and "allow_page_to_choose_SVG" (something to do with EOT, WOFF, TTF, and SVG formats used in some glyphs) - this would reduce the nummber of glyphs broken

eg github offers the octicon in all four formats, so it should be fixed by this

2 years later..

The more complex the solutions we conceive here the less chance anyone who can code it will be able or inclined to do so
Foreach(webfont){ If ( any of the characters on the page in each webfont are invalid in the 'local-preference-set-font')

Anything along these lines is pretty much doomed to failure. It would require analysis of the entire page... and/or might still be progressively downloading from the network, and/or might be constantly changing...

1 year later

give priority to the user's preferred fonts when the use_document_fonts option is off (0), but for characters not supported in the user's choice of font, use the page's specified fonts (whether local or downloaded or both) as usual, rather than going directly to system fallback fonts. This allows things like Github icons to continue working, as the PUA codepoints used should not be found in the user's pref fonts (normally), and so they'll fall back to the appropriate icon font from the page.
sites and fonts exist that misusing character codes as defined by Unicode, and the result is a lack of interoperability/compatibility
I don't see how the browser can "fix" this, except via some kind of explicit whitelist (or blacklist) of fonts/sites/etc, which would be a never-ending maintenance nightmare.
https://bugzilla.mozilla.org/show_bug.cgi?id=789788#c118 - explains it

So: no whitelists, no blacklists, no checking of font names for strings like *icon* etc - basically a fallback for fonts that use PUA
Notes

You can (always?) tell a PUA when its blocked by the 7-segment LCD-style debug codes in Bitstream Vera and DejaVu Sans (the rectangular icon that looks like a calulator!) - see imgur's missing glyphs are not these, the other test they are
"The default Sans font on most Linux systems (DejaVu Sans) has glyphs for many of the PUA code points."
NoScript can block @font-face which can be a vector of attack which can lead to font breakage including glyphs
AdBlock (ewwww, I hope no-one is using it) can block/allow fonts

*.ttf|
*.woff|
||fonts.googleapis.com^

Conclusion

will do some tests soon and post back
I started to look at imgur's glyphs in the top left nav - I just need to check the unicode code point, but I am 99% sure that these are not PUA
The issue with uBo missing glyphs for nodiscc requires some tests in a nilla FF, but is not the same issue as imgur. Could be DejaVu Sans which has a ton of its own glyphs in PUA code points, which would explain the non calculator images shown - i.e nodiscc's default specified font does not fail those characters and displays the local specified characters - can be tested by changing the font set, clearing cache, and restarting (must restart to load font stuff). This is the exact behavior expected by the code.
FF pref allows PUA glyphs only, not all glyphs - so maybe we should update the description

@nodiscc - what are you default fonts

see Options>Fonts & Colors>Advanced>Serif|Sans-serif|Monospace

PS: sorry for the long post and rambling, but if I wasted my time, you girls can spare 5 minutes, plus you might learn something (I did).

tl;dr:

5 years ago they suggested checking character unicode endpoints to see if they were PUA, and if they were to allow that font (local or downloaded) for that character
2.5 years later, they implemented a variation of that solution. They changed the behavior and wording of browser.display.use_document_fonts (flipped both) so that any fallback went to the user specified fonts. If the user-specified font is not using the endpoints then it will go get the font (assuming no other blocks like uBo, uM, NS, XSS, @font-face etc).
MOST default fonts do not use PUA but some do. And lots of glyphs do use PUA, but not ALL, so is not 100% perfect
"If the user-specified font is not using the endpoints" - DejaVu Sans on Linux may be an issue?

I think we need a better test, but here goes
Test1: https://www.w3schools.com/icons/tryit.asp?filename=tryicons_fa-list

FF nilla or new profile
browser.display.use_document_fonts = 0
gfx.downloadable_fonts.enabled = false
clear cache, quit & start (I did because I read something somewhere abut fonts loading on start)
visit test page
expected behavior: the w3schools "home" and "orientation" icons are displayed as unicode hex symbols (but the menu icon does display?)
gfx.downloadable_fonts.enabled = true
reload test page
expected behavior - we get glyphs (home and orientation)

If you do not see the glyphs, what are your default specified fonts?
Info: http://fontawesome.io/cheatsheet/
fa-home =  []
fa-menu = IDK
fa-rotate = IDK
Why does fa-menu show, but not the others? Need some help here to get the unicode and check if which ones are PUA. Note: my specified fonts are Times New Roman, Arial, Courier New (Windows defaults)
@nodiscc : what do you get? This is not definitive, but we could easily code up a sample which outputs the uBo fonts that you can't see

IDK ... Using the test above, I simply changed the "try it" code, and see for yourself - glyphs missing from w3schools, but the code sample loads them (via cloudflare) as per the right hand side

nice try with the explanation but totally confusing IMO. I think @nodiscc said the same thing but much easier to understand.
the characters used are (from left to right): \e800, \f0c9, \e813
the one you can see under Unicode is, as can be seen in the source code on the left: f03a
to make the hex codes into html characters you prepend the &#x and append a semicolon
if the numbers in the missing icons are not easy enough to read you can see them if you click "Inspect element" and switch to the Style Editor and look at the inline style sheet:
.fa-home:before {content: '\e800'; }
.fa-save:before {content: '\e804'; }
.fa-rotate:before {content: '\e813'; }
.fa-menu:before { content: '\f0c9'; }

uBO has 1 bundled font fontawesome-webfont.ttf, but IDK what the supported font formats are in Linux, maybe @nodiscc's Linux doesn't support TTF fonts and that's why he doesn't see the uBO icons.

OT: frecency (frequency + recency) - from Martin's latest FF article. Thx Martin

A good point here.


Result Table

You can run the command below to see it in action. The results will be displayed using table.print()
npm run example -- ./examples/execAsync.ts -d DB_NAME -e ENGINE_NAME -c 'def output = 123' --poll

Looks pretty awesome! I will say that the function naming is very JS oriented. For example, I don't think slice is a common name in other languages. B2T2 seems to use select and you pass in an array of indexes. Java's ArrayList uses subList. Python uses slice but on the array itself, not a function. I guess we are shooting for being idiomatic in each language so I suppose these names can diverge. 🤷

Looks pretty awesome! I will say that the function naming is very JS oriented. For example, I don't think slice is a common name in other languages. B2T2 seems to use select and you pass in an array of indexes. Java's ArrayList uses subList. Python uses slice but on the array itself, not a function. I guess we are shooting for being idiomatic in each language so I suppose these names can diverge. 🤷

select is used for column selection in the Arrow SDK. I was thinking about doing the same here instead of sliceColumns maybe.
Arrow API seems to be pretty different from language to language https://github.com/apache/arrow. I think it's fine to diverge and be idiomatic as long as all SDKs implement the same set of functionality.

Excited for this. One naming bikeshed comment — since this will eventually be the primary result type, do we want to call it something high-level and Rel-y like Relation?

@vilterp I thought that too at first but according to wikipedia a relation implies no ordering. But our result set has ordering, so I think table is the proper term.

Also we still export type named as Relation which is v1 json relation object.


MakerDAO Ethereum: dailyLiquidateUSD and dailyLiquidateCount potential issues

It looks like the MakerDAO revenue for 12/06/2022 is extremely high (~$30m) because the subgraph is recording roughly ~$220m in liquidations for USDC-A vault on that day. It looks like MakerDAO was liquidating the USDC-A vault for any collateral that was under 101% . In comparison to https://maker.blockanalitica.com/liquidations/?tab=per-date it looks like we are overestimating the amountUSD and liquidation count. IIRC we have slight differences based on the timestamp we track for liquidations, but even if you account for 12/05 and 12/07 our values looks too high. Query for liquidations for USDC-A.

I believe the difference between the number in our subgraph and blockanalitica number is they do not include a subset of the liquidations (looks like they are liquidated at a loss to the protocol). For example, liquidation # 11:

https://maker.blockanalitica.com/liquidations/USDC-A/11/
https://etherscan.io/tx/0xe08f9e75afe6dd023ef1c08b5058c67c532ad96a0c531305734e6c27a8f3a855#eventlog
Those liquidations amount to ~$21m: https://docs.google.com/spreadsheets/d/1_6Ra27FIypoEyouiPVXtgVt8gJcWx69P8i_QLWt2JDA/edit?usp=sharing
I think the total liquidation numbers match after including the excluded liquidations.
Your question does bring up another (old) issue about protocol side revenue - whether we should set negative protocol revenue when collateral is sold at a loss (less than the debt). Makerdao had a similar issue in March 2020 (https://insights.glassnode.com/what-really-happened-to-makerdao/). I believe our decision was to keep calculating protocol side revenue using the penalty fee %, instead of accounting a loss to the protocol, but we may want to revisit this at some point.

Fixed bug that was ignoring liquidation penalty when it was set to 0

Going to leave this open till change is deployed on decentralized network

Going to leave this open till change is deployed on decentralized network

@bye43 Shall we create a separate issue or use #1521 for this? It makes anxious to have a p0 task assigned to me remain open :laughing:

Ahh, gotcha - I'll close the issue and leave it here then and remove the tag if thats ok?


Graph operations & tensors unknown in hub model

I am trying to inspect the graph proto of a downloaded tf-2 bert model.
It seems all named nodes are unknown placeholders. Is there a way to view the internal ops and tensors of the bert graph?
tensorflow_hub == '0.8.0'
tensorflow == '2.3.0'
Reproducing:
import tensorflow_hub as hub
bert = hub.load('~/Desktop/bert/bert_en_uncased_L-12_H-768_A-12_2')
graph = bert.signatures['serving_default'].graph

for node in graph.as_graph_def().node:
  print(f'{node.name} {node.op} {node.input}')

Output:
input_mask [] Placeholder
input_type_ids [] Placeholder
input_word_ids [] Placeholder
unknown [] Placeholder
unknown_0 [] Placeholder
unknown_1 [] Placeholder
unknown_2 [] Placeholder
unknown_3 [] Placeholder
unknown_4 [] Placeholder
unknown_5 [] Placeholder
unknown_6 [] Placeholder
unknown_7 [] Placeholder
unknown_8 [] Placeholder
unknown_9 [] Placeholder
unknown_10 [] Placeholder
unknown_11 [] Placeholder
unknown_12 [] Placeholder
unknown_13 [] Placeholder
unknown_14 [] Placeholder
unknown_15 [] Placeholder
unknown_16 [] Placeholder
unknown_17 [] Placeholder
unknown_18 [] Placeholder
unknown_19 [] Placeholder
unknown_20 [] Placeholder
unknown_21 [] Placeholder
unknown_22 [] Placeholder
unknown_23 [] Placeholder
unknown_24 [] Placeholder
unknown_25 [] Placeholder
unknown_26 [] Placeholder
unknown_27 [] Placeholder
unknown_28 [] Placeholder
unknown_29 [] Placeholder
unknown_30 [] Placeholder
unknown_31 [] Placeholder
unknown_32 [] Placeholder
unknown_33 [] Placeholder
unknown_34 [] Placeholder
unknown_35 [] Placeholder
unknown_36 [] Placeholder
unknown_37 [] Placeholder
unknown_38 [] Placeholder
unknown_39 [] Placeholder
unknown_40 [] Placeholder
unknown_41 [] Placeholder
unknown_42 [] Placeholder
unknown_43 [] Placeholder
unknown_44 [] Placeholder
unknown_45 [] Placeholder
unknown_46 [] Placeholder
unknown_47 [] Placeholder
unknown_48 [] Placeholder
unknown_49 [] Placeholder
unknown_50 [] Placeholder
unknown_51 [] Placeholder
unknown_52 [] Placeholder
unknown_53 [] Placeholder
unknown_54 [] Placeholder
unknown_55 [] Placeholder
unknown_56 [] Placeholder
unknown_57 [] Placeholder
unknown_58 [] Placeholder
unknown_59 [] Placeholder
unknown_60 [] Placeholder
unknown_61 [] Placeholder
unknown_62 [] Placeholder
unknown_63 [] Placeholder
unknown_64 [] Placeholder
unknown_65 [] Placeholder
unknown_66 [] Placeholder
unknown_67 [] Placeholder
unknown_68 [] Placeholder
unknown_69 [] Placeholder
unknown_70 [] Placeholder
unknown_71 [] Placeholder
unknown_72 [] Placeholder
unknown_73 [] Placeholder
unknown_74 [] Placeholder
unknown_75 [] Placeholder
unknown_76 [] Placeholder
unknown_77 [] Placeholder
unknown_78 [] Placeholder
unknown_79 [] Placeholder
unknown_80 [] Placeholder
unknown_81 [] Placeholder
unknown_82 [] Placeholder
unknown_83 [] Placeholder
unknown_84 [] Placeholder
unknown_85 [] Placeholder
unknown_86 [] Placeholder
unknown_87 [] Placeholder
unknown_88 [] Placeholder
unknown_89 [] Placeholder
unknown_90 [] Placeholder
unknown_91 [] Placeholder
unknown_92 [] Placeholder
unknown_93 [] Placeholder
unknown_94 [] Placeholder
unknown_95 [] Placeholder
unknown_96 [] Placeholder
unknown_97 [] Placeholder
unknown_98 [] Placeholder
unknown_99 [] Placeholder
unknown_100 [] Placeholder
unknown_101 [] Placeholder
unknown_102 [] Placeholder
unknown_103 [] Placeholder
unknown_104 [] Placeholder
unknown_105 [] Placeholder
unknown_106 [] Placeholder
unknown_107 [] Placeholder
unknown_108 [] Placeholder
unknown_109 [] Placeholder
unknown_110 [] Placeholder
unknown_111 [] Placeholder
unknown_112 [] Placeholder
unknown_113 [] Placeholder
unknown_114 [] Placeholder
unknown_115 [] Placeholder
unknown_116 [] Placeholder
unknown_117 [] Placeholder
unknown_118 [] Placeholder
unknown_119 [] Placeholder
unknown_120 [] Placeholder
unknown_121 [] Placeholder
unknown_122 [] Placeholder
unknown_123 [] Placeholder
unknown_124 [] Placeholder
unknown_125 [] Placeholder
unknown_126 [] Placeholder
unknown_127 [] Placeholder
unknown_128 [] Placeholder
unknown_129 [] Placeholder
unknown_130 [] Placeholder
unknown_131 [] Placeholder
unknown_132 [] Placeholder
unknown_133 [] Placeholder
unknown_134 [] Placeholder
unknown_135 [] Placeholder
unknown_136 [] Placeholder
unknown_137 [] Placeholder
unknown_138 [] Placeholder
unknown_139 [] Placeholder
unknown_140 [] Placeholder
unknown_141 [] Placeholder
unknown_142 [] Placeholder
unknown_143 [] Placeholder
unknown_144 [] Placeholder
unknown_145 [] Placeholder
unknown_146 [] Placeholder
unknown_147 [] Placeholder
unknown_148 [] Placeholder
unknown_149 [] Placeholder
unknown_150 [] Placeholder
unknown_151 [] Placeholder
unknown_152 [] Placeholder
unknown_153 [] Placeholder
unknown_154 [] Placeholder
unknown_155 [] Placeholder
unknown_156 [] Placeholder
unknown_157 [] Placeholder
unknown_158 [] Placeholder
unknown_159 [] Placeholder
unknown_160 [] Placeholder
unknown_161 [] Placeholder
unknown_162 [] Placeholder
unknown_163 [] Placeholder
unknown_164 [] Placeholder
unknown_165 [] Placeholder
unknown_166 [] Placeholder
unknown_167 [] Placeholder
unknown_168 [] Placeholder
unknown_169 [] Placeholder
unknown_170 [] Placeholder
unknown_171 [] Placeholder
unknown_172 [] Placeholder
unknown_173 [] Placeholder
unknown_174 [] Placeholder
unknown_175 [] Placeholder
unknown_176 [] Placeholder
unknown_177 [] Placeholder
unknown_178 [] Placeholder
unknown_179 [] Placeholder
unknown_180 [] Placeholder
unknown_181 [] Placeholder
unknown_182 [] Placeholder
unknown_183 [] Placeholder
unknown_184 [] Placeholder
unknown_185 [] Placeholder
unknown_186 [] Placeholder
unknown_187 [] Placeholder
unknown_188 [] Placeholder
unknown_189 [] Placeholder
unknown_190 [] Placeholder
unknown_191 [] Placeholder
unknown_192 [] Placeholder
unknown_193 [] Placeholder
unknown_194 [] Placeholder
unknown_195 [] Placeholder
unknown_196 [] Placeholder
unknown_197 [] Placeholder
StatefulPartitionedCall ['input_word_ids', 'input_mask', 'input_type_ids', 'unknown', 'unknown_0', 'unknown_1', 'unknown_2', 'unknown_3', 'unknown_4', 'unknown_5', 'unknown_6', 'unknown_7', 'unknown_8', 'unknown_9', 'unknown_10', 'unknown_11', 'unknown_12', 'unknown_13', 'unknown_14', 'unknown_15', 'unknown_16', 'unknown_17', 'unknown_18', 'unknown_19', 'unknown_20', 'unknown_21', 'unknown_22', 'unknown_23', 'unknown_24', 'unknown_25', 'unknown_26', 'unknown_27', 'unknown_28', 'unknown_29', 'unknown_30', 'unknown_31', 'unknown_32', 'unknown_33', 'unknown_34', 'unknown_35', 'unknown_36', 'unknown_37', 'unknown_38', 'unknown_39', 'unknown_40', 'unknown_41', 'unknown_42', 'unknown_43', 'unknown_44', 'unknown_45', 'unknown_46', 'unknown_47', 'unknown_48', 'unknown_49', 'unknown_50', 'unknown_51', 'unknown_52', 'unknown_53', 'unknown_54', 'unknown_55', 'unknown_56', 'unknown_57', 'unknown_58', 'unknown_59', 'unknown_60', 'unknown_61', 'unknown_62', 'unknown_63', 'unknown_64', 'unknown_65', 'unknown_66', 'unknown_67', 'unknown_68', 'unknown_69', 'unknown_70', 'unknown_71', 'unknown_72', 'unknown_73', 'unknown_74', 'unknown_75', 'unknown_76', 'unknown_77', 'unknown_78', 'unknown_79', 'unknown_80', 'unknown_81', 'unknown_82', 'unknown_83', 'unknown_84', 'unknown_85', 'unknown_86', 'unknown_87', 'unknown_88', 'unknown_89', 'unknown_90', 'unknown_91', 'unknown_92', 'unknown_93', 'unknown_94', 'unknown_95', 'unknown_96', 'unknown_97', 'unknown_98', 'unknown_99', 'unknown_100', 'unknown_101', 'unknown_102', 'unknown_103', 'unknown_104', 'unknown_105', 'unknown_106', 'unknown_107', 'unknown_108', 'unknown_109', 'unknown_110', 'unknown_111', 'unknown_112', 'unknown_113', 'unknown_114', 'unknown_115', 'unknown_116', 'unknown_117', 'unknown_118', 'unknown_119', 'unknown_120', 'unknown_121', 'unknown_122', 'unknown_123', 'unknown_124', 'unknown_125', 'unknown_126', 'unknown_127', 'unknown_128', 'unknown_129', 'unknown_130', 'unknown_131', 'unknown_132', 'unknown_133', 'unknown_134', 'unknown_135', 'unknown_136', 'unknown_137', 'unknown_138', 'unknown_139', 'unknown_140', 'unknown_141', 'unknown_142', 'unknown_143', 'unknown_144', 'unknown_145', 'unknown_146', 'unknown_147', 'unknown_148', 'unknown_149', 'unknown_150', 'unknown_151', 'unknown_152', 'unknown_153', 'unknown_154', 'unknown_155', 'unknown_156', 'unknown_157', 'unknown_158', 'unknown_159', 'unknown_160', 'unknown_161', 'unknown_162', 'unknown_163', 'unknown_164', 'unknown_165', 'unknown_166', 'unknown_167', 'unknown_168', 'unknown_169', 'unknown_170', 'unknown_171', 'unknown_172', 'unknown_173', 'unknown_174', 'unknown_175', 'unknown_176', 'unknown_177', 'unknown_178', 'unknown_179', 'unknown_180', 'unknown_181', 'unknown_182', 'unknown_183', 'unknown_184', 'unknown_185', 'unknown_186', 'unknown_187', 'unknown_188', 'unknown_189', 'unknown_190', 'unknown_191', 'unknown_192', 'unknown_193', 'unknown_194', 'unknown_195', 'unknown_196', 'unknown_197'] StatefulPartitionedCall
Identity ['StatefulPartitionedCall', '^StatefulPartitionedCall'] Identity
Identity_1 ['StatefulPartitionedCall:1', '^StatefulPartitionedCall'] Identity

That's expected for a SavedModel in TensorFlow 2.
As you probably know, the switch to TF2 has changed TensorFlow's Python interface from building a big tf.Graph to eager execution of individual ops or, where necessary, execution of tf.functions that contain composable graph pieces.
The SavedModel format has evolved accordingly: all the actual computation is in the GraphDef.library of functions. The main graph is retained only for compatibility with C++ serving clients, containing function call ops (seen at the bottom of your output) and placeholders for their inputs. The SavedModel signatures simply point to those function calls.
Models like https://tfhub.dev/tensorflow/bert_en_uncased_L-12_H-768_A-12/2 are published from open-source code, so there should usually be no need to look at them as the graph level. But if you insist, try
obj = hub.load("https://tfhub.dev/tensorflow/bert_en_uncased_L-12_H-768_A-12/2")
cf = obj.__call__.get_concrete_function([tf.TensorSpec([None, None], tf.int32) for _ in range(3)])
print(cf.function_def)

or
print(obj.signatures["serving_default"].function_def)

NOTE: I'm explaining this to help understand and debug the system,  not recommending for programmatic use.


Fix grouper sync dest selection not staying checked

Ticket Link
GROUPINGS-1146
Test Checklist

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

The Codacy check is failing because it is not set up to review this branch.  Since the code passes on the master branch, no issue here.


loudbug_bug was actually from fitter

much code that has "loudbug_bug" and the loud dependency probably doesn't work because it was from fitter, which was removed.
So we need to replace "loudbug_bug" for something else or bring back fitter
they all compile and instantiate on mac though

see https://github.com/porres/pd-cyclone/search?p=2&q=loudbug_bug&type=&utf8=✓

no moron, it's from loud indeed...


Indium asks to grab Sodium 5.8 even though I already have it

Hello! I'm having a very odd issue where When I have the latest version of indium it says to get sodium 5.8 (which I already have) and says I have 5.3 which is really weird. I then deleted sodium 5.8 and launched the game and it gave me the same issue. I don't even have sodium at all in my minecraft folder anymore. I just tested if it gives the same message from the Indium 1.0.25 for Minecraft 1.20.1/Sodium 0.5.2 version, I still get the same issue but this time with it saying 5.2. I have no clue why indium is doing this with any version for me.

Screenshot of message and of mods folder?

Closing: insufficient information to diagnose issue


Support for basic UDFs in Peloton

This PR adds PL/pgSQL UDF support to Peloton and some other features/ changes which are needed by the same.
PL/pgSQL UDFs
There are 2 parts to adding support for UDFs:

UDF Registration : This involves parsing the UDF, compiling it into native code and saving it in an in-memory map for quick access in case of future invocations.
UDF Invocation : This identifies that the SELECT query is invoking a UDF and extracts the previously compiled function and executes it.

This PR can be divided into 2 Parts:

UDF Registration support
a. Adding support for handling CREATE OR REPLACE FUNCTION statement which dissects the CREATE FUNCTION statement into UDF function_name, function parameter names, function parameter types, function return type and function body.
b. Handling the UDF function body which includes, parsing into Abstract Syntax Tree form and Just-in-Time compiling it during UDF registration.

UDF Invocation support
a. Adding support to help identify a function_expression to be a UDF.
b. Registering the previously compiled function into the current code_context to allow for invocation.

This PR currently supports UDFs with:

Expression statements
If-Else constructs
Recursive Function Calls

Coverage decreased (-1.1%) to 74.795% when pulling 4d46b52df445d8c1610f528b2d9fa0bdde28a137 on prashasthip:new_master into 4e39dcfdddbc085269a8d3c63b5869e47c35615f on cmu-db:master.

Coverage decreased (-1.1%) to 74.772% when pulling e7129a7c028c196cf5da82779d73e7667dd962b5 on prashasthip:new_master into 4e39dcfdddbc085269a8d3c63b5869e47c35615f on cmu-db:master.

Coverage increased (+0.06%) to 75.978% when pulling 1520ebe400301516145d3b1dee5a4a970c40f31c on prashasthip:new_master into 4e39dcfdddbc085269a8d3c63b5869e47c35615f on cmu-db:master.

@prashasthip Can you rebase and fix the tests?

Coverage increased (+0.02%) to 75.787% when pulling fac21ff6ec32686c63899402a57218cbfb09bff1 on prashasthip:new_master into 5409ea2f202b9c1fc3a199798dbf308e14083cf3 on cmu-db:master.

Coverage increased (+0.4%) to 76.273% when pulling 64dc73f4bbdd562fa456b24e6f2799f57334c9a9 on prashasthip:new_master into 12133fada1e430fc2507afadac52ab3d7cdcbad2 on cmu-db:master.

Coverage increased (+0.4%) to 76.28% when pulling 64dc73f4bbdd562fa456b24e6f2799f57334c9a9 on prashasthip:new_master into 12133fada1e430fc2507afadac52ab3d7cdcbad2 on cmu-db:master.


fix: respect secondary entry file customizations

I'm submitting a...
[x] Bug Fix
[ ] Feature
[ ] Other (Refactoring, Added tests, Documentation, ...)

[ ] Yes
[x] No

looks good!


[macOS] Improve ANGLE support detection.

Tries opening EGL display when checking if ANGLE is supported.

Follow up to https://github.com/godotengine/godot/pull/83753

Thanks!


Size of video is Zero bytes after saving for iPhone device

The video is saved on home path but size of video is 0 bytes.  Then paste the whole output to this ticket? Thanks.

@sainiketan47 please see the issue below and follow the steps to reinstall ideviceinstaller.
https://github.com/isonic1/flick/issues/10

Setting to wontfix. If the steps in issue #10 dont resolve this then I will reopen. Thanks for reporting.


Fix the unlink endpoint, mainly for timelines

Previously, the twitter API was being hit for every timeline linked to a room, and would also pass the twitter ID instead of the screenName.
Also, remove_timeline was being given a screenName when it should be given the id of the profile to remove. This is now fixed.
(See #29 for previous attempt at making a PR)

(Failing test appears to be racing, :disappointed:)
Thanks LGTM


Declare graphql-language-service-utils dependency

Fixes: #1944
I currently get this error when trying to build our react app with uses graphiql:
Module not found: Error: Can't resolve 'graphql-language-service-utils' in '/home/wstott/repos/tritium/user_interfaces/tritium_graphql/node_modules/graphiql/node_modules/codemirror-graphql'

Awesome thank you! I can do a follow up PR after this to import all the LS packages from graphql-language-service, and fix the related issue in graphiql, I will take a look later in the week hopefully

thank you for this, #2037 will take care of this!


Compiler reports warning (too many warnings - 4591) even when all have been ignored

I'm working on a very big repo with very big files, and our test fixtures and contracts are very big.
As such, solc creates warnings for all of those (contract size too big), and we have more than 256.
In the code below, the library ignores warnings 1878 and 5574 for test files, which is fine.
https://github.com/foundry-rs/compilers/blob/67a46cf8e60fd5d8da9b26d14f3d72288c5bbc7c/crates/compilers/src/compile/output/mod.rs#L925-L952
However, we still see the following in the output:
Compiler run successful with warnings:
Warning (4591): There are more than 256 warnings. Ignoring the rest.

It's really confusing to see this message when all of the warnings have been ignored by foundry_compilers.
The clean solution would probably be the following:

Count the number of ignored warnings
Count the number of non-ignored warnings
If non-ignored warnings <= 256 && ignored warnings > 256, then silence error 4591

The ErrorFilter should probably be updated to match.

could you please share your foundry.toml settings for this?

[profile.default]
solc_version = "0.8.26"
evm_version = "cancun"
optimizer = true
optimizer_runs = 20_000
libs = ["node_modules", "lib"]
invariant = { fail_on_revert = true }

remappings = [
    "@chainlink/=node_modules/@chainlink/",
    "@openzeppelin/=node_modules/@openzeppelin/",
    "solady/src/=node_modules/solady/src/",
]

[profile.default.fmt]
line_length = 120
bracket_spacing = true
wrap_comments = true
number_underscore = "thousands"
sort_imports = true

hmm, you're not ignoring this warning, could you try ignoring "too-many-warnings" via
https://github.com/foundry-rs/foundry/blob/6818c846ad54455721deae0adc10efbf87a0f11f/crates/config/README.md#L105-L108

Ah yes that would help, but then we also wouldn't get it when there are more than 256 non-ignored warnings. It's a good workaround I guess, thanks!

Actually I retract what I said, when I put this option in the foundry.toml, I get ALL the warnings even those in the test folder.

this is happening because when you set ignored_error_codes you override default setting which is ignored_error_codes = ["license", "code-size", "init-code-size", "transient-storage"]
Adding those back should help

Yep that worked! Thanks. Do you think this should be handled by the lib though?

this should be handled by the lib though?

what exactly?

Well ideally users shouldn't have to mess with the default settings to achieve this result. Ideally the crate would directly ignore this warning in the case I described in my issue above. What do you think?

okay, I think now I got it.
I'm not sure tbh because I believe solc excludes warnings entirely so we don't necessarily get all warnings, but I think ignoring this warning in case all other 256 warnings are also ignored would be reasonable.

The issue with this is that solc will only show up to 256 warnings. So e.g. if there are 256 ignored warnings + this warning + 256 non-ignored warnings ignored by solc, then silencing "Too many warnings" would leave user with impression that there're zero non-ignored warnings which might not be true. I wouldn't want this to be default behavior, and we already have a way to silence this specific warning if user wants to do so, thus closing as not planned


iosevka-bin: 19.0.1 -> 20.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for iosevka-bin is: Versatile typeface for code, from code.
meta.homepage for iosevka-bin is: https://typeof.net/Iosevka/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 20.0.0 with grep in /nix/store/8hm45ykyai8mlfj426v117kyxbjyg2rm-iosevka-bin-20.0.0
found 20.0.0 in filename of file in /nix/store/8hm45ykyai8mlfj426v117kyxbjyg2rm-iosevka-bin-20.0.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
iosevka-bin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8hm45ykyai8mlfj426v117kyxbjyg2rm-iosevka-bin-20.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A iosevka-bin https://github.com/r-ryantm/nixpkgs/archive/a6462d104a8d7004df933c892c9cf2d42c48ee97.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8hm45ykyai8mlfj426v117kyxbjyg2rm-iosevka-bin-20.0.0
ls -la /nix/store/8hm45ykyai8mlfj426v117kyxbjyg2rm-iosevka-bin-20.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

iosevka-bin

Maintainer pings
cc @cstrahan @montchr for testing.

Result of nixpkgs-review pr 219868 run on aarch64-darwin 1

1 package built:

iosevka-bin


Replace flat_file.cpp with NuDB implementation

Description of the Change

[x] replaces flat_file.cpp with NuDB implementation.
[x] removes white-box tests for flatfile
[x] updates boost (adds additional dependencies) and adds NuDB to docker develop

Closing, because I faced some issues with tests on Linux. Will be opened after I fix them.


Add Spacelift to tech-vocab

FYI we usually do not add commands to the vocabulary as they should be fenced.

Good catch, added a PR to remove the CLI again: https://github.com/coopnorge/vale-coop/pull/27


Lazily enumerate DSI response pages

DFESignIn was fetching all response pages from DSI into one enormous
array, which was causing the workers running DSI fetching jobs to run
out of memory.
This turns DFESignIn#get_response_pages into a lazy enumerator that
just takes life one page at a time.

Review app deployed to https://teaching-vacancies-review-pr-2810.london.cloudapps.digital


ERROR: cannot launch node of type [grid_map_demos/simple_demo]: can't locate node [simple_demo] in package [grid_map_demos]

I build from the source code successly,but when I run :
$roslaunch grid_map_demos simple_demo.launch
It shows:
ERROR: cannot launch node of type [grid_map_demos/simple_demo]: can't locate node [simple_demo] in package [grid_map_demos]
ERROR: cannot launch node of type [grid_map_visualization/grid_map_visualization]: can't locate node [grid_map_visualization] in package [grid_map_visualization]
process[rviz-4]: started with pid [27004]
[ERROR] [1510757591.418195526]: PluginlibFactory: The plugin for class 'grid_map_rviz_plugin/GridMap' failed to load.  Error: Could not find library corresponding to plugin grid_map_rviz_plugin/GridMap. Make sure the plugin description XML file has the correct name of the library and that the library actually exists.

Did you solve this?
Just form the errors, I would guess you need to resource the setup.bash file in our devel folder after building for the first time.

catkin_make --only-pkg-with-deps grid_map_demos
catkin_make --only-pkg-with-deps grid_map_visualization
catkin_make --only-pkg-with-deps grid_map_rviz_plugin

You just need to change the path of the dataset


Figure out DST and timezone changes

Currently oref0 requires that the Pi, pump, and CGM be set to the same timezone, and errors out if (minAgo > 10 || minAgo < -5).  This causes problems when traveling and changing timezones, and every spring and fall when the Pi's clock automatically changes for DST.  I pushed a workaround (https://github.com/openaps/oref0/commit/89b609c3ce966519b94e9bae6a472dbddb55d767) that will help temporarily for tonight's DST change, but at the expense of continuing to operate on stale data for up to an hour (or up to 2 hours if timezones are off to begin with).  We need to figure out a better solution for dealing with timezones.

With the latest dexcom_reader and openaps, we can now do:
openaps use cgm UpdateTime --to now
openaps use pump set_clock --to now
So it might make sense to set up a cron job to do so regularly as long as NTP is working.  Then, any time the rig timezone is updated for DST or travel, the devices can be synchronized and we can guarantee that old timestamps really mean stale data.
Does anyone have any thoughts on whether that is the best approach to solve this, or have any alternate suggestions?

https://gitter.im/nightscout/intend-to-bolus?at=57007597e4a8384a1bbe50dc
@itsn0tm3:
I agree with your idea, as long as NTP doesnt report as not synchronized, then it should in theory be a good timesource to go with. It would also make it simpler since you would only need to set the time in one place so less user error can happen. My only concern is making sure that NTP is actually synchronized before using it as a valid source
@scottleibrand:
I actually have a script that already checks that, so that part is working and tested
in fact, it does the reverse when you're offline and your rig has no idea what time it is: it sets the rig time to the most recent device time
so we could do that when not synchronized, and set the devices per NTP when we are
https://github.com/openaps/oref0/blob/master/bin/clockset.sh
with a few updates, I think that script could pretty easily do both
@itsn0tm3
I think it makes sense
I'd +1 it, I can't think of another way that would be better

The meta-question is: do we want the rig to be the source of truth for time (when it has Internet and can synchronize with NTP)?

Just updated bin/clockset.sh (will push when JetBlue wifi cooperates) to set the device clocks to the current rig time if NTP is synchronized, and tested it by changing the rig from EDT to CDT and running it.  It worked correctly, with the caveat that it looks like it may be miscalculating IOB somewhat as a result.  I suspect the only way to fix that would be to make oref0 aware of the meaning of clock set events in the pumphistory, which seems complicated.  For now I'd rather just document a recommendation that users only change rig timezones by one hour at a time, and do so when COB=0 and IOB =~ 0.

Need to update clockset to only set devices clocks if they're off, so we can add it to a loop if determine-basal fails (due to clock mismatch).


[WIP] Restructure project for tools support core repository

This renames Basic, Utility and their dependencies to TSCBasic and TSCUtility.
This makes it possible to copy their directories to the tools-support-core repository to be able to use them in other projects too.
TestSupport is now divided in TSCTestSupport and SPMTestSupport where the latter contains all SwiftPM specific parts and TSC the generic parts which use the tools support core interfaces.
The idea is to copy the sources and tests to the tools-support-core repository and eventually remove it from here and depend on the new repo. That requires some support in cmake which we don't rely on yet. More information about the motivation can be read here: https://forums.swift.org/t/factoring-out-a-new-repo-for-shared-code-between-llbuild-swiftpm/11429/22
This PR is markes as WIP to work on the documentation and the bootstrapping of the tools-support-core repository.

@swift-ci please smoke test

Once you figure out the bootstrap issues you'll need to test this with an updated sourcekit-lsp as well.  I created https://github.com/apple/sourcekit-lsp/pull/153 with the module name changes.
Incidentally, I noticed while updating the code that there are some APIs named spm_* such as spm_chomp and spm_only that should probably be renamed.

@swift-ci please smoke test

@swift-ci please smoke test

I think I might need to factor more stuff out that is interesting for other projects. For example the ArgumentParser is part of TSCUtility but the whole Commands package defines the tool infrastructure which we might want to refactor out to TSCUtility a bit.
What do you think @aciidb0mb3r? Nevertheless I think we can iterate on the changes in this PR later on after having the first version landed.

@swift-ci please smoke test

@benlangmuir I tested your changes in SourceKit-lsp with the changes here and they worked fine (thanks for working on them!). How can I configure the CI to use your branch instead of master here?

Here are the details: https://github.com/apple/swift/blob/master/docs/ContinuousIntegration.md#cross-repository-testing

https://github.com/apple/sourcekit-lsp/pull/153
@swift-ci please smoke test

Thanks @benlangmuir! It didn't work though. It's complaining about Sources/SKCore/BuildServerBuildSystem.swift:12:8: error: no such module 'Basic' but that file doesn't exist in your branch. Does master needs to be rebased into your branch?

apple/sourcekit-lsp#153
@swift-ci please smoke test

I think I might need to factor more stuff out that is interesting for other projects. For example the ArgumentParser is part of TSCUtility but the whole Commands package defines the tool infrastructure which we might want to refactor out to TSCUtility a bit.

That one is a bit SwiftPM specific that doesn't necessarily make sense as a generic utility but we can discuss that.
What do you think @aciidb0mb3r? Nevertheless I think we can iterate on the changes in this PR later on after having the first version landed.
Yep, let's land with minimal changes first and then we refactor incrementally.

Yeah, I don't mean to move SwiftTool to TSC but more provide infrastructure that SwiftTool can use as well as others. Basically functionality for providing usage information and handling the argument parser calling.

apple/sourcekit-lsp#153
@swift-ci please smoke test

apple/sourcekit-lsp#153
@swift-ci please smoke test

apple/sourcekit-lsp#153
@swift-ci please smoke test


Add support for new quantum-resistant key exchange

According to the release notes for OpenSSH 8.0 (https://www.openssh.com/releasenotes.html), a new quantum-resistant key exchange algorithm was implemented:
 * ssh(1), sshd(8): Add experimental quantum-computing resistant
   key exchange method, based on a combination of Streamlined NTRU
   Prime 4591^761 and X25519.

I second this request 👍  Is there a possibility to get this feature?


"Zănești" and "Traian" are listed in the drop-down menu

Description:
"Zănești" and "Traian" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Zănești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Zănești, NEAMȚ (Zănești)" and "Comuna Zănești, NEAMȚ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Traian" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Traian, NEAMȚ (Zănești)".


Disable interactive transactions when using the Data Proxy

Problem

Interactive transactions doesn’t work with the proxy and triggers errors. We are preventing these errors right now by detecting that both preview feature flags dataproxy and iTX are enabled and preventing generation. If this is promoted to GA and the flag is removed we might lose the ability to detect this is enabled?
Next Steps

[x] Discuss solution
[ ] Decide on a solution in this internal document

Additional context

Internal discussion: https://prisma-company.slack.com/archives/C02FZENHD4N/p1645200596753649

I think this is a good solution. We are available for questions or to chat about this.

Do you all agree this warrants mention in the Docs? Maybe just an admonition in the pages about ITX, to let people know that if they enable data proxy as a preview feature, then itx is automatically disabled? @albertoperdomo @matthewmueller

We're still working on the design of this in this internal document.
It seems like we're settling on either 3.2 or 4.

I'm closing this since we have decided to not to implement it this way.


HOSTEDCP-1818:fix(build): update to the fixed clamav image

What this PR does / why we need it:
The upcoming OpenShift Pipelines which will be deployed shortly has stricter validations on Pipelines and Task manifests. Clamav would fail the new validations. This commit should make us ready.
Fixes #HOSTEDCP-1818
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/approve
/lgtm

/retitle HOSTEDCP-1818:fix(build): update to the fixed clamav image

/jira refresh

/area ci-tooling

/cherry-pick release-4.16


Address two bugs with the handling of prefixes

Motivation
While implementing the delete=clear+save we saw a number of problems. Here are the ones for prefixes
Proposal
Two bugs were addressed:

One when we insert a prefix bug it is already matched by existing prefixes.
One in MapView where a prefix has been deleted and the for_each_key_* is used for the same prefix. Then we need to remove the prefix from consideration.

Test Plan
The CI and a better check.
Release Plan
None.
Links

reviewer checklist

From the summary I cannot tell if the bug is serious (corrupted data) or not (aspirational storage invariant)

From the summary I cannot tell if the bugs are serious (corrupted data?) or not (aspirational storage invariant?)

Serious.


Issue-447 - Interactions of OptMeowt and browser GPC settings

Note about interactions between OptMeowt and browser GPC settings added to the settings page of the OptMeowt extension.

Looks good to me, go ahead and merge and close the issue.


staticman support

Hi,
I tried to modified the theme to support staticman.
Users can follow the Step1, 2, 4 in the instructions to use staticman comment-system.
The workflow:

enable staticman support in params by enableStaticman = true
configure [staticman] (examples were provided in exampleStie config.toml)
add staticman.yml in the root directory for staticmanapp

Then, when someone comments on posts, staticmanapp will send the comment file to the data/comments in the repo and create a pull request. The site-owner can decide whether to merge/show the comments.
I'm not quite familiar with creating websites though, the commited files may need further modifications...
An example site: https://yh.netlify.com/posts/happy-chinese-new-year/
Cheers!
Yu-Han

fix some bugs...

the parameters staticman_send_to, github_pull, staticman_post_to should be modified more, but I couldn't make it work so I just left it as it was...
I think that works!


Re-enable codecov.io

Changes to the incremental build scripts had broken it by mistake.

Current coverage is 87.00% (diff: 100%)

No coverage report found for master at 4c9da91.

Powered by Codecov. Last update 4c9da91...2712cc5


fix(tx-data-provider): do not seed testdata again on upgrade

Description

fix: do not seed testdata again on upgrade
introduced with https://github.com/eclipse-tractusx/tractus-x-umbrella/pull/81
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

Hi @almadigabor, could you please support with the following error? https://github.com/eclipse-tractusx/tractus-x-umbrella/actions/runs/9059263005/job/24886502424?pr=84#step:11:19

Created a PR to this branch where I'm trying to fix the helm dependency hack issue. https://github.com/eclipse-tractusx/tractus-x-umbrella/pull/85


Expose Region default rx2 datarate index

Fixes #327
Ref: DLSettings from LLS 1.0.4.pdf
https://lora-alliance.org/resource_hub/lorawan-104-specification-package/

Getting rolled into a PR that includes AS923 updates.


Implement some frontend test infrastructure

We currently have standard Django tests, which are run on each push thanks to GitHub actions.
It would be very useful to have something in place to help testing the frontend components (whole pages? Vue components? Use Django or Vue tooling?)

#14 should be implemented first to avoid duplicated work.

Update: as a first step, I have setup selenium in Django tests so we can do "in browser" tests. To be continued. State of affairs:

[x] selenium configured and integrated with Django tests (first basic test in place)
[x] selenium-based tests are also run on GitHub actions
[ ] make sure WebPack is run before the selenium-base tests, so all frontend elements are ready for the browser
[ ] write a bunch of "high level", in-browser tests to check the application works as expected from a user perspective
[ ] (maybe) add more fine-grained unit testing for the frontend (i.e, use Jest to test Vue components in isolation)

We have "good enough" frontend testing since a long time using Selenium, closing.


Trigger libvirt_manager molecule on config_drive changes

Since config_drive is used in the libvirt_manager, we want to ensure
it's still properly working.
Also, moving the config_drove workdir to the standard "artifacts" tree,
ensuring we gather the content from the jobs.

recheck

/lgtm


Issue with Markers resolution

I'm completely stucked with a strange issue related to Markers resolution on high-density screens.
I define my Marker icons with only 1 resolution, in res/drawable-mdpi directory.
On one application (OSMNavigator), Marker icons are automatically resized according to the screen density - as needed and expected.
On another application (NooTous), this doesn't work at all: Marker icons are not resized.
Same version of osmdroid (6.1.11). Same target SDK (30). Same test device (API 30, 420 dpi).
The only difference I can see is that the first app is based on old-style Activity model, while the second app is based on Fragment.
(both applications here: https://github.com/MKergall/osmbonuspack)
Issue Type
[ X ] Question

It wouldn't be the first time that Android messes up with drawable expected behaviors; there are already a couple of related issues.
My suggestion: just populate the 5 dpi drawable folders. If that does not work, that's another story.

@monsieurtanuki I also used this workaround. My real problem is that I have the same issue with icons downloaded from Internet (typically: icons referenced in KML files). And for those icons, no way to handle multiple resolutions.
(Could you point me to related issues?)

@MKergall I don't know if this problem is caused by the Android version (30+) or the code.
It could be interesting to have a look at an optimization I coded in Marker.drawAt, that I would sum up as: canvas.drawBitmap( mIcon...) or mIcon.draw(pCanvas).
Could you test if one of them works in your KML case?
Then, it would make sense to be able to force the choice of the good solution - for the moment it's coded as if (mIcon instanceof BitmapDrawable).
And probably being able to say: I want this Marker's icon to be displayed as big as possible in a specified [width, height] rectangle (cf. final int markerWidth = mIcon.getIntrinsicWidth()) is a path to the solution.

All my icons are BitmapDrawables.
I will try to test the other case: mIcon.draw(pCanvas) - I will have to subclass the Marker.
Something even stranger: I tried with another test device: API 27, 420 dpi =>

Marker based on a resource drawable-mdpi is correctly resized (!!!).
KML Marker (icon downloaded at runtime) is still not resized.

Using bitmap.setDensity(DisplayMetrics.DENSITY_MEDIUM); has absolutely no effect.

Probably you'll have more control over the display size with the second syntax, with mIcon.setBounds(mRect) then mIcon.draw(pCanvas).

Yes!!!!
@monsieurtanuki, you pointed directly to the right place :
I removed the "mIcon instanceof BitmapDrawable/pCanvas.drawBitmap" option, and forced "mIcon.draw(pCanvas);" way.
=> Marker icons now have the correct size, in all configurations:

mdpi resources,
downloaded at runtime - as long as I set their density: bitmap.setDensity(DisplayMetrics.DENSITY_MEDIUM);

In addition, the colouring of KML icons (a standard KML feature implemented in OBP), which had also disapeared, is back to normal.

@MKergall Wonderful!
If you can confirm with additional tests, I suggest that you create a PR that removes the "optimized for BitmapDrawable" option that is 15% faster but not 100% supported...


Reduce Rescue Prime's number of rounds from 8 to 7

A hash function switch from Rescue Prime to Tip5 is foreseeable, rendering the change summarized in this issue's title pointless.


Unhandled rejection Error, for Promise.all even though I try to catch them.

This is a Question, I've been banging my head trying to figure out why the Unhandled promise rejection is not being caught.
Node v6.9.1
"bluebird": "^3.4.6"
I'm testing this function:
UserController.prototype.getDashboard = function getDashboard(req, res, next) {
  let pages, user;

  return FB.getAsync('me/accounts')
    .then(fbPages => {
      pages = fbPages.data;
      //The root of all evil
      return Promise.all(pages.map(page => FB.getAsync(`${page.id}/events`)))
      // I even tried adding another catch after each individual mapped value of after the array before
      // without success
    })
    .then(events => {
      let entity = {
        user,
        pages: pages.map((page, i) => _.assign({}, page, { events: events[i].data }))
      };

      return res.send(entity);
    })
    .catch(err => next(Boom.wrap(err)));
};

But when I test it, even though it passes, I get Unhandled rejection Error
This is my test:
 it('handles errors', (done) => {
  let mockError = new Error('test-error');
  let mockRequest = { user: { id: mockUser._id }};
  let mockNext = td.function();
  let capture = td.matchers.captor();

  td.replace(FB, 'getAsync');
  td.when(FB.getAsync('me/accounts'))
    .thenReturn(Promise.resolve(usersTestData.getDashboard.pages));

  td.when(FB.getAsync(`1769754093273789/events`))
    .thenReturn(Promise.resolve(usersTestData.getDashboard.events[0]))

  // Promise rejection was handled asynchronously
  td.when(FB.getAsync(`731033223716085/events`))
    .thenReturn(Promise.reject(mockError))

  td.when(mockNext(Boom.wrap(mockError)))
    .thenDo(() => { done() });

  controller.getDashboard(mockRequest, _.noop, mockNext);
});

The weird part, is that it passes. So the mockNext is being called with what I expected but I get a log in the console with the unhandled rejection. I've tried to handle .catch for each value of the mapping, but it still shows the same error (warning?).
I'm using mongoose(promisified) and testdouble. This is the log from the console after running the test:
dashBoard
Unhandled rejection Error: test-error
✓ handles errors

I also tried return Promise.map(pages, page => FB.getAsync(${page.id}/events)) with the same effect, tests pass, but the same error/warning appears in the conosole.

I found a way to fix this: It has to do with the way the Promises were being rejected.

so what was the fix?


mail/postfix: make delay_warning_time configurable

At least my users prefer to receive a notification if a mail stays in the queue for an extended amount of time. Make that parameter configurable instead of forcing the default (disabled).

If you push to the same branch the PR is updated automatically. You can even rewrite it or just add commits on top.

On 20. Jul 2021, at 22:02, Patrick M. Hausen @.***> wrote:

@pmhausen commented on this pull request.
In mail/postfix/src/opnsense/mvc/app/models/OPNsense/Postfix/General.xml:

@@ -185,5 +185,12 @@
0
Y
</reject_unverified_recipient>

   <delay_warning_time type="IntegerField">

       <default>0</default>

       <Required>N</Required>

I need some help now with github. So if I update and push to my forked repo, how do I update the pull request? Generate a new one instead?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@fichtner maybe this one too when you find time :)

Merged, thanks!


Noise Update Progress

The Noise Update will be brought to Switch. Please be patient as I enjoy the update and do other things people do in their lives, because I am an individual that exists out there. Can you find me? This update will find you.
Tasks left to do:

[ ] Port the thing
[ ] Uhh, maybe include #12

Take as much time as you nee, and good luck with porting it!

how DARE you be a human being!!!n/j

holy crap, we'll be able to play as the yellow noid in pete's tower for the nintendo switch at some point

WAH

How to unlock the game file without a computer💀

How to unlock the game file without a computer💀

use a tinfoil shop i guess

How to unlock the game file without a computer💀

literally none of what you just said makes sense, and the skull emoji doesn't help either

How to unlock the game file without a computer 💀

To legally install it, you need some device running Python and a legal copy of Pizza Tower update. Also, if you’re referring to the Noise Update, it hasnt been ported as of now.

I think he was referring to the save data, how to modify it to 101% without a computer (i guess??)

Take as much time as you need, your work is greatly appreciated!!! :)

What's the progress looking like?

What's the progress looking like?

I haven't gotten as much done as I wanted to due to several reasons. But among other things, I've completely rewritten the FMOD script included in this repository.

Oh boy, I can't wait to fight the Doise.

At this stage nearly all the code and assets from the Noise update are inside the GameMaker project. I can play as him. But the "merging" has caused several regression bugs around every area the port has ever changed (to optimize, or to add new content like mods from the port menu).
Roughly, the tasks I have to do now are:

Fix decompilation/regression bugs by playing the game a lot (I have people helping me with this, don't worry)
Implement new FMOD events (all simple "play a sound" events have already been implemented, in the same manner as the base game, see the top of fmod.gml if you're curious)
Implement swap mode music changing (I have a pretty good plan on how)

I don't know how long it'll take me and I don't think I'll ever do something like this again. Like I mentioned in the other thread I have less free time and my interest in this game has lowered by a lot. But I do want to at least finish this.

Thanks for all your hard work, no matter how long this takes you man!

Quick update. The port is playable start to finish. The part that needs most work is still the SFX. (there are still some FMOD events that are not implemented/not changed to account for the Noise). That, and also I haven't gotten to swap mode at all, but I imagine it won't be (that much) of an issue

The progress is taking a long time, but I will patiently await the release.

I've been on a roll for the past few days. I do think I can say this: Do expect the update to release within a few days from now.

 Thank you for your hard work:]

Have fun!


Bug with selection algorithm

Meshes sometimes don't match up correctly

Problem solved by giving each mesh a name. Problem began because of asynchronous loading of the models.


Dev

Hey @helebest I think @chemik-bit tried to reach you, but wanted to reach out as well - there is some heavy refactor proposals in this PR so would love to hop on a quick call to discuss. Thanks!! Filip


Crashing when opening new buffer

When I open neovim with snack.nvim's dashboard, the background animation correctly starts playing but once I open a file in a new buffer, it crashes with the following message:
Error executing vim.schedule lua callback: ...im/lazy/animatedbg.nvim/lua/animatedbg-nvim/renderer.lua:11: Invalid buffer id: 1
stack traceback:
	[C]: in function 'nvim_buf_get_lines'
	...im/lazy/animatedbg.nvim/lua/animatedbg-nvim/renderer.lua:11: in function 'get_row_content'
	...im/lazy/animatedbg.nvim/lua/animatedbg-nvim/renderer.lua:42: in function 'render'
	...im/lazy/animatedbg.nvim/lua/animatedbg-nvim/executor.lua:84: in function ''
	vim/_editor.lua: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

@ruiiiijiiiiang I couldn't really reproduce it, but I've had that issue before when the buf is deleted.  I added a check and now it should only call nvim_buf_get_lines when the buffer is still valid. Let me know if this fixes the issue :)

@alanfortlink I can confirm this has been fixed with the latest patch. Thank you so much!


Document the upgrade process

Solved in helm chart epi >=0.4.x
https://github.com/PharmaLedger-IMI/helm-charts/tree/master/charts/epi


Eye tracking inaccurate compared to the calibration software on headset, or vrcfacetracking.

This is after a full calibration, and originally I thought this was just a quest pro issue, but it might actually be a problem with the mod.
For instance, Vrcfacetracking for VRC is basically fine, but there's something with the impressive mod that makes the eye tracking a bit inaccurate. Sadly I can't really demonstrate this properly, but something is off.
To reiterate, this is after a calibration, and it's pretty much almost perfect in the calibration software, and in VRC. Just in Resonite somethings off.
Update: on further testing it seems like it's an offset, like the further i am from the center of the screen, the more offset it is.

What is your testing process for this? Make sure you use the AvatarRawEyeData component and not just values from the eye manager.

What is your testing process for this? Make sure you use the AvatarRawEyeData component and not just values from the eye manager when you're measuring the accuracy in Resonite.

I'm using the new eye tracking tester in Resonite Essentials - Examples which uses the eye tracking data from avatarraweyedata.
https://github.com/RileyGuy/Impressive/assets/82356528/46d34d0a-2aad-4313-8861-f98245ca51b2
In this video I'm trying to look at the specular dot on the sphere, but it ends up going past the sphere.

Closing as no longer needed since Resonite supports Quest Pro facetracking natively now.


Google sign go to google <IP_ADDRESS>

I have setup all correctly but i continued to have this page

after sign

the application file here

i read it but if you dont explain error we cannot forward.
Where i fix it?


feat: prototype-pollution vulnerable chat app

Thanks @Kirill89! ✨
Some update I'll run through later is to create aliases for the exploits but will merge for now.


vi bid adapter docs

Documentation for Video intelligence Bid Adapter

@rmloveland looks like layout is broken :( any idea what's wrong there?

@rmloveland the only difference i see from other adapters is empty line between ###bids and actual table. May that be a problem?

@SnoopInf yes that was it, fixed in aad0023

@rmloveland Thank you!


ctr pull should unpack for default platform when transfer service is used

Currently if neither --all-platforms nor --platform is specified in ctr i pull, it will pull using the default platform, but skip the unpack step (if using transfer service). However, we should apply the default platform to unpack as well.
# in main branch, no any snapshot is generated
containerd [main]  ➜ sudo ctr i pull --snapshotter=overlayfs docker.io/library/alpine:latest
...
containerd [main]  ➜ sudo ctr snapshot ls
KEY PARENT KIND
# with this fix, default platform is used for pull and unpack.
containerd [ctr-pull-transfer-service]  ➜ sudo ctr i pull --snapshotter=overlayfs docker.io/library/alpine:latest
...
containerd [ctr-pull-transfer-service]  ➜ sudo ctr snapshot ls
KEY                                                                     PARENT KIND
sha256:75654b8eeebd3beae97271a102f57cdeb794cc91e442648544963a7e951e9558        Committed

hi @dmcgowan could you PTAL this PR when you get some time? thanks.
This makes --snapshotter take effect when no platform is passed (use default platform), when using transfer service.

/cherrypick release/2.0

nice, will remove that from my PR


Set file permissions for keystone to keystone federation

The saml files for keystone should be owned by keystone and have mode 0640.
The shibboleth key/cert file should be owned by _shibd but the rest of the config files will require 644 since they need to be read by the apache process.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

I have removed the Work In Process. It should be good to go.

ok to test

retest

retest

retest


Is there a strategy for synchronizing with upstream?

This repo is based on https://github.com/sigstore/helm-charts/tree/main/charts/scaffold but is not a fork. As the upstream sigstore repo evolves, is there a strategy for updating this repository?
I have some thoughts on this if there is not, based on my experience with OpenShift Serverless midstream repositories such as https://github.com/openshift-knative/kn-plugin-func. Ideally, the OpenShift CI system should be used to regularly run an update script that applies patches and overlays where needed. Here is an example script that does that for Knative Functions: https://github.com/openshift-knative/kn-plugin-func/blob/main/openshift/release/update-to-head.sh.

Closing as this is done now


Unable to install

I am unable to install in replit using npm
node-gyp 9.3.0
Npm 9.1.1
Node 19.0.1
All the packages are latest version
I even did using -g same result
I did even npm I —python=python2.7
Then install same result
sorry there is no way me to copy the logs since I am using mobile

Tried verbose flag

https://github.com/WiseLibs/better-sqlite3/issues/891 https://github.com/WiseLibs/better-sqlite3/issues/892 https://github.com/WiseLibs/better-sqlite3/issues/899

#891 #892 #899
Even if you fix the Python issue (which is probably not a better-sqlite3 issue) Node 19 is not supported yet

Which version of node is supported

Is ok if I use node 18.12.1

Able to install
Steps
npm I —python=python2.7
npm I -g better-sqlite3


Import .bacpac file gives error Unable to connect to master or target server 'AXDB_Copy'. You must have a user with the same password in master or target server 'AXDB_Copy'.

Hi
While importing .bacpac file to local VM machine's SQL server. It gives this error message :
Unable to connect to master or target server 'AXDB_Copy'. You must have a user with the same password in master or target server 'AXDB_Copy'.

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

ID: 3436fbc8-5bdd-dcdc-f5d3-51caf7b6cd76
Version Independent ID: af78d3ef-b329-11e9-896a-aa3394f6c77c
Content: Copy Finance and Operations databases from Azure SQL Database to SQL Server environments - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/database/copy-database-from-azure-sql-to-sql-server.md
Service: dynamics-ax-platform
GitHub Login: @laneswenka
Microsoft Alias: laswenka

Thanks for contacting us. @laneswenka, can you look into enhancing the documentation to provide the information that @burakuygur is looking for.
In the meantime, to get product help you can contact Support using one of the options described here: Support contact options. In particular, you might want to go to the Dynamics 365 for Finance and Operations community page to get help with your questions from the Dynamics community.

Hi @burakuygur can you provide the import command you used with SQLPackage.exe?  Based on your error message it sounds like you set the /tsn:AXDB_Copy but that should instead be localhost.  Want to confirm the /tdn: is set also to a database name that doesn't exist already on the local VM.

Bug 270369 has been logged to track this issue.

Not able to reproduce, please provide more details or we will mark this as resolved


Recreate enabledPackages slice so it is empty

When reading the enabled-packages from config the if UnmarshalKey fails it creates an empty EnabledPackage in the slice. The slice is then populated using the legacy config style with GetStringSlice.
However later the empty EnabledPackage causes Preflight to exit with the error:
2020/02/03 16:24:43 Package with ID "" was specified in configuration but it wasn't found.

This fix recreates the slice when falling back to the legacy method to make sure it's empty before populating.
Signed-off-by: wwwil<EMAIL_ADDRESS>

/lgtm


Freeze all requirements on each release

To avoid packages difference it is better to use particular package versions instead of latest one.

Latest release do this


Ccp 753 change decimal and 1000

added de-CH locale file so the language could be forced to de-CH so that the price will be formatted correctly by the intl-messageformat third party module.

We checked it again and introducing de-CH is not needed. We can simply force the language for one shop and this would use fallback language for translations + correct formatting for everything else.


Native Editor - new telemetry

Stuff to track
•	Every Jupyter hotkey should tracked for their usage respectively [ESTIMATE: 1 day]
•	Telemetry should be added to the mouse actions in each cell to track their usage [ESTIMATE: 1 day]
o	Add cell
o	Delete cell
o	Move cell up
o	Move cell down
o	Run cell
o	Change cell to markdown
o	Collapse code input
o	Collapse code output
•	Telemetry should be added to track how many users click the “Save as Python file” from the native Jupyter window [ESTIMATE: 1 day for the rest]
•	Telemetry should be added to track how many users click the “Save as Notebook file” from the Python interactive window
•	Telemetry should be added to track how many users opened the native Jupyter experience
•	Telemetry should be added to track how many users have .ipynb files in the workspace
o	Of the users that have .ipynb files, what percentage of them open up the native Jupyter window
•	Of those users, how many of them opened it a second day in the same month
•	Percentage of people that only view (i.e only run code but don’t edit it) notebooks vs. edit them
•	What do users do after they are done with their notebook?
•	Telemetry should be added to track the average time it takes to load the ipynb files

For first release we need:

[ ] Jupyter hotkey telemetry
[ ] Mouse use to move cell up / down (on the cell itself)
[ ] Mouse use to run a cell
[ ] Change cell to markdown and back
[ ] Telemetry for each action in the global toolbar.

The following should be tracked in a separate issue:
• Telemetry should be added to track how many users opened the native Jupyter experience
• Telemetry should be added to track how many users have .ipynb files in the workspace


Getting error "Exception calling "SignData" with "3" argument(s): "Invalid algorithm specified."

Any ideas? I believe I have followed the steps correctly.
Exception calling "SignData" with "3" argument(s): "Invalid algorithm specified.
"
At C:\Program Files\WindowsPowerShell\Modules\AdobeUMInterface\AdobeUMInterface.psm1:401 char:5

$JWTSig = ConvertTo-Base64URL -Item ($SignatureCert.PrivateKey.Si ...

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : CryptographicException

One note, my certificate is given to me by my security team, I did not use a self-signed like in your instructions. I am guessing this is the issue.

I am not sure off the top of my head. The error says it does not like the algorithm, so for some reason, your cert and the default SHA256 and/or PKCS1 padding are not compatible. Since it specified algorithm, I am guessing it is the SHA256 part. At least that is my guess. I will look into it some more.
$JWTSig = ConvertTo-Base64URL -Item ($SignatureCert.PrivateKey.SignData([System.Text.ASCIIEncoding]::UTF8.GetBytes($JWT), [System.Security.Cryptography.HashAlgorithmName]::SHA256, [System.Security.Cryptography.RSASignaturePadding]::Pkcs1))

I think this is related to the certificate's signature algorithm and/or .net/powershell version. Could you send me that info? For the .net/powershell version just sending the output of $PSVersionTable would work. For the certificate algorithm, if you look at the certificate properties, there should be details tab. Near the top should be a "Signature Algorithm" field. I can then create one on my side using same settings and see if I can reproduce the issue.

Thanks! I am on Server 2012 R2 PowerShell v5.1 and the signature algorithm
is sha256rsa.
On Fri, Aug 30, 2019 at 4:50 PM Ziviz<EMAIL_ADDRESS>wrote:

I think this is related to the certificate's signature algorithm and/or
.net/powershell version. Could you send me that info? For the
.net/powershell version just sending the output of $PSVersionTable would
work. For the certificate algorithm, if you look at the certificate
properties, there should be details tab. Near the top should be a
"Signature Algorithm" field. I can then create one on my side using same
settings and see if I can reproduce the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zincarla/AdobeUMInterface/issues/10?email_source=notifications&email_token=ACW2IWCIKACEZ6L2K4OPZYDQHGBZZA5CNFSM4ISOPRY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5SX7TQ#issuecomment-526745550,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACW2IWB4L25VCBE4GZUHEHDQHGBZZANCNFSM4ISOPRYQ
.

--
Dan

I can confirm sha256rsa works on Server 2012 R2, however, only when the cert is generated using the "Microsoft Enhanced RSA and AES Cryptographic Provider" CSP. I have so far not found a way to get it to work with the other CSPs. The default CSP when one is not specified on Windows appears to be "Microsoft RSA SChannel Cryptographic Provider". I really do not know what the technical limitation is. I will keep looking into this, however, I don't expect this to be quick.

Thanks! That makes sense since it was generated outside of Windows. I guess
my hesitancy is that I dont want to use a self-signed cert for something
like this. I appreciate your help though, I will try to look into it myself
as well.
On Tue, Sep 3, 2019 at 11:22 AM Ziviz<EMAIL_ADDRESS>wrote:

I can confirm sha256rsa works on Server 2012 R2, however, only when the
cert is generated using the "Microsoft Enhanced RSA and AES Cryptographic
Provider" CSP. I have so far not found a way to get it to work with the
other CSPs. The default CSP when one is not specified on Windows appears to
be "Microsoft RSA SChannel Cryptographic Provider". I really do not know
what the technical limitation is. I will keep looking into this, however, I
don't expect this to be quick.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zincarla/AdobeUMInterface/issues/10?email_source=notifications&email_token=ACW2IWCEE5N5MY3WTENE55DQHZ6MHA5CNFSM4ISOPRY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5YSDIY#issuecomment-527507875,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACW2IWEVOIVANFGXBNKJHR3QHZ6MHANCNFSM4ISOPRYQ
.

--
Dan

No worries. I think I have a quick solution. I think your issue is similiar to here. None of the solutions in there are great in my oppinion, however, the root cause is the provider for the cert does not support SHA2 signing. If that is the case, you could change the signing algorithm to SHA1 instead. This is less secure, but I beleive would work without further issues.
Literally changing
$JWTSig = ConvertTo-Base64URL -Item ($SignatureCert.PrivateKey.SignData([System.Text.ASCIIEncoding]::UTF8.GetBytes($JWT), [System.Security.Cryptography.HashAlgorithmName]::SHA256, [System.Security.Cryptography.RSASignaturePadding]::Pkcs1))

to
$JWTSig = ConvertTo-Base64URL -Item ($SignatureCert.PrivateKey.SignData([System.Text.ASCIIEncoding]::UTF8.GetBytes($JWT), [System.Security.Cryptography.HashAlgorithmName]::SHA1, [System.Security.Cryptography.RSASignaturePadding]::Pkcs1))

(SHA1 or MD5 vs the SHA256)

Hmm now I am getting a different error:
Invoke-RestMethod : The remote server returned an error: (400) Bad Request.
At C:\Program Files\WindowsPowerShell\Modules\AdobeUMInterface\AdobeUMInterface.psm1:408 char:30

... ation.Token=Invoke-RestMethod -Method Post -Uri $AuthTokenURI -Body $ ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-RestMethod], WebException
FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

Bad news. That is happining because on line 393 we specify the algorithm as sha 256. So the dignsture doesnt match what we ste telling it to expect. I looked up the JWS documentation and i dont see sha1. I don't think this work around is possible.

Does your cert, when imported, have a GetRSAPrivateKey() function? There is a C# example which gets the private key from that function. My certs dont have that though when imported into powershell.

No it does not.

I made some progress! I generated a cert with OpenSSL using the same parameters as Adobe's Documentation. Then I joined the private key and cert together into a pfx and imported into Windows. This gave me the same issue you have. Invalid Algorithm. Bunch of digging and testing, I found this guys blog. When you join the private key to the cert as I did, you can specify a CSP to use. If you set it to The "Microsoft Enhanced RSA and AES Cryptographic Provider", it justs works!
:: Create a new cert and private key file per adobe
openssl req -x509 -sha256 -nodes -days 365 -newkey rsa:2048 -keyout private.key -out certificate_pub.crt
:: Now join the key and crt into a pfx file, so we can import into windows. Specify the CSP to enable SHA2 support
openssl pkcs12 -inkey private.key -in certificate_pub.crt -export -CSP "Microsoft Enhanced RSA and AES Cryptographic Provider" -out text2.pfx

Your steps will be slightly different since you are given your cert by your security team, but I beleive you can still use what they gave you to do a similiar work around. If you already have the key and crt files, just join them specifying the CSP. If you have a PFX file, split it into a key and crt file with openssl, then rejoin specifying the CSP. I beleive that would work, anyway.

Hmm see I saw that too but I couldnt get the -CSP to work, at least on
Server 2012 R2. Have OS are you using?
On Thu, Sep 5, 2019 at 2:22 PM Ziviz<EMAIL_ADDRESS>wrote:

I made some progress! I generated a cert with OpenSSL using the same
parameters as Adobe's Documentation
https://www.adobe.io/authentication/auth-methods.html#!AdobeDocs/adobeio-auth/master/AuthenticationOverview/ServiceAccountIntegration.md%23step-2-create-a-public-key-certificate.
Then I joined the private key and cert together into a pfx and imported
into Windows. This gave me the same issue you have. Invalid Algorithm.
Bunch of digging and testing, I found this guys blog
https://hintdesk.com/2011/07/29/c-how-to-fix-invalid-algorithm-specified-when-signing-with-sha256/.
When you join the private key to the cert as I did, you can specify a CSP
to use. If you set it to The "Microsoft Enhanced RSA and AES Cryptographic
Provider", it justs works!
:: Create a new cert and private key file per adobe
openssl req -x509 -sha256 -nodes -days 365 -newkey rsa:2048 -keyout private.key -out certificate_pub.crt
:: Now join the key and crt into a pfx file, so we can import into windows. Specify the CSP to enable SHA2 support
openssl pkcs12 -inkey private.key -in certificate_pub.crt -export -CSP "Microsoft Enhanced RSA and AES Cryptographic Provider" -out text2.pfx
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zincarla/AdobeUMInterface/issues/10?email_source=notifications&email_token=ACW2IWBBBHLYQ5VTUHTJB6LQIFE6PA5CNFSM4ISOPRY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6AHJCQ#issuecomment-528512138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACW2IWD4BJLCAEVXWJFKEJ3QIFE6PANCNFSM4ISOPRYQ
.

--
Dan

Windows 10, I am using the openssl that came with git bash.

New avenue to look down, I found this link Which suggest that setting the CSP on server 2012 is possible with Microsoft's certutil command.

Wooooooooooooooooooooooooooooooooooooooooo
certutil worked:
certutil.exe -csp 'Microsoft Enhanced RSA and AES Cryptographic Provider' -importPFX -p 'Password' .\pfx.pfx
THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU THANK YOU

Woo! I think I will update the code to catch this error and offer a suggestion then

Current commit should notify users of the issue, and recommend re-importing the certificate with a compatible CSP.


Add Nong Song: the cave, the treasure

Song Name
the cave, the treasure
Artist Name
bossfight
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=rZPkHI4oE5c
Direct File Link
No response
Song ID
1358855
Start Offset [ms]
No response
Extra
No response

This submission is stale because it has been open for 14 days with no activity. If it is still relevant, please leave a comment and we will reopen it.

it is not. the responce is still waited upon


Styles for Widgets

Some reusable styles for widgets, coming soon to a style guide near you. And an example for how to layout a widget.

:+1:

I :heart: the idea of getting to re-usable styles and style guidance.  I anticipate we're going to continue to need to add and update widgets, and driving down the marginal cost of doing that would be lovely.  #embracechange.


Multipe Crafters

Issue description:
RS network only using one machine for auto crafting when multiple machines have the same pattern.
I have two solderers working both with the same patterns and when I told the system to produce 3 64k storage discs it only utilized one of the solderers and never used the second.
What happens:
It is only using one machine
What you expected to happen:
It to use multiple machines
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge:
Refined Storage:
Sky Factory 3 Latest Update

Does this issue occur on a server? [yes/no] Yes
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

Implemented in 1.11 (aedaa0740765ef09eab10aab9a6b1a390fd77596), duplicate of #704. Might get backported at some point.

@way2muchnoise please back port it, i cannot find a fix to craft loads of processors in 1.10, it's currently the only thing limiting me from moving on (crafting x64 64K drives) but using only 1 solderer it took 3-4 hours to make just 16 of the 64k drives please fix this


Can't download Twitter GIFs.

When I try to download a normal Twitter video works fine. But if I try to download a Twitter video of a GIF (which YouTube converts automatically) I get this error: "ERROR: Unable to extract twitter card url;". Example:
This should work: http://twitter.com/ErnietheFT/status/657265991135698944/video/1
This should not: http://twitter.com/giphz/status/657991469417025536/photo/1

OS X 10.11, latest youtube-dl, this works for me
This should work: http://twitter.com/ErnietheFT/status/657265991135698944/video/1
Ernie_-_Serious_ear_flappage_going_on.mp4
[download] 100% of 1.42MiB in 00:05

Twitter GIFs will be supported in the next version (cf5881fc4defe3902367fc02b8eb39d912572d5a, sorry I mix two irrelevant changes in a single commit). Note that the downloaded file is MP4 even if twitter says it's GIF.


chatqna HUGGINGFACEHUB_API_TOKEN doesn't seem to be doing all of its job

I see the following in chatqna-tgi's logs (in K8s), installed via https://github.com/opea-project/GenAIInfra/tree/main/helm-charts/chatqna even after passing my correct hugging face token and accepting the conditions to accessing the gated repo for llama3:
export HFTOKEN="..."
export MODELDIR="/mnt"
export MODELNAME="meta-llama/Meta-Llama-3-8B"

helm install chatqna chatqna --set llm-uservice.HUGGINGFACEHUB_API_TOKEN=${HFTOKEN} --set llm-uservice.tgi.volume=${MODELDIR} --set llm-uservice.tgi.LLM_MODEL_ID=${MODELNAME}

kubectl logs chatqna-tgi-f455cdb9b-bqwfv:
{
    "message": "Download encountered an error: \nTraceback (most recent call last):\n\n  File \"/opt/conda/lib/python3.10/site-packages/huggingface_hub/utils/_errors.py\", line 270, in hf_raise_for_status\n    response.raise_for_status()\n\n  File \"/opt/conda/lib/python3.10/site-packages/requests/models.py\", line 1021, in raise_for_status\n    raise HTTPError(http_error_msg, response=self)\n\nrequests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://huggingface.co/api/models/meta-llama/Meta-Llama-3-8B\n\n\nThe above exception was the direct cause of the following exception:\n\n\nTraceback (most recent call last):\n\n  File \"/opt/conda/bin/text-generation-server\", line 8, in <module>\n    sys.exit(app())\n\n  File \"/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py\", line 128, in download_weights\n    utils.weight_files(model_id, revision, extension)\n\n  File \"/opt/conda/lib/python3.10/site-packages/text_generation_server/utils/hub.py\", line 151, in weight_files\n    filenames = weight_hub_files(model_id, revision, extension)\n\n  File \"/opt/conda/lib/python3.10/site-packages/text_generation_server/utils/hub.py\", line 110, in weight_hub_files\n    info = api.model_info(model_id, revision=revision)\n\n  File \"/opt/conda/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py\", line 118, in _inner_fn\n    return fn(*args, **kwargs)\n\n  File \"/opt/conda/lib/python3.10/site-packages/huggingface_hub/hf_api.py\", line 1922, in model_info\n    hf_raise_for_status(r)\n\n  File \"/opt/conda/lib/python3.10/site-packages/huggingface_hub/utils/_errors.py\", line 286, in hf_raise_for_status\n    raise GatedRepoError(message, response) from e\n\nhuggingface_hub.utils._errors.GatedRepoError: 401 Client Error. (Request ID: Root=1-666c9efe-652c658f64be7e2644868213;c60b443a-ef5b-4dde-8165-60966c452697)\n\nCannot access gated repo for url https://huggingface.co/api/models/meta-llama/Meta-Llama-3-8B.\nAccess to model meta-llama/Meta-Llama-3-8B is restricted. You must be authenticated to access it.\n"
  }

MODELNAME="meta-llama/Meta-Llama-3-8B"
meta-llama need special permission with HUGGINGFACE TOKEN, previously we didn't pass the TOKEN to TGI inference server and you can use models like intel/xxx.
The Reorg of helm charts added support for passing HF_TOKEN to tgi server, you can try after that merged (Pay attention to the readme file change with helm dependency update)
Before that, you can try with other models like Intel/xxx instead of the meta-llama

Thanks Dolpher, your fix is working for me.


Stopped ion-spinner Android animation script from running indefinitely.

:+1: We've been having this issue as well

Do you have an example to test against?


Cpplint unit tests

Minor change to get more code coverage on cpplint tool plugin unit test.

Codecov Report

Merging #119 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #119      +/-   ##
==========================================
+ Coverage   65.27%   65.32%   +0.04%
==========================================
  Files          44       44
  Lines        2154     2154
==========================================
+ Hits         1406     1407       +1
+ Misses        748      747       -1

Impacted Files
Coverage Δ

statick_tool/plugins/tool/cpplint_tool_plugin.py
100% <0%> (+1.69%)
:arrow_up:

Continue to review full report at Codecov.

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


Suggestions

Although TableAdapters have their place it would seem prudent to promote working the native data providers detached from any Windows form(s) working with classes and/or class projects then along the same path, promotion of Entity Framework. A proposal would be to rethink the bulleted list above so there is no so much focus on TableAdapter path along with having code samples where various topics show what is needed to explain a action/operation which can point to a working example which can be obtained from GitHub repo. Thinking larger, a mirror of the VB repos for C# to keep both language code samples equal which might be going to far but can't hurt to mention.

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

ID: 53debfcb-6e5d-3779-405b-553fd92683f6
Version Independent ID: 57edc605-ba28-e9e0-937e-b31ea17a519b
Content: Work with datasets in n-tier applications - Visual Studio
Content Source: docs/data-tools/work-with-datasets-in-n-tier-applications.md
Product: visual-studio-windows
GitHub Login: @gewarren
Microsoft Alias: gewarren

@karenpayneoregon Thanks for the feedback!


I found some docs link not in code

Like this:
https://github.com/open-mmlab/mmediting/blob/master/docs/sr_datasets.md

Thanks for the report! It is now at https://mmediting.readthedocs.io/en/latest/sr_datasets.html

Well, I know. I mean i want to add some usage in Vimeo90K Dataset part
https://github.com/open-mmlab/mmediting/blob/master/tools/data/super-resolution/vimeo90k/README.md

In preprocess_vimeo90k_dataset.py
    if mode == 'gt':  # only read the 4th frame for the gt mode
        print('Only keep the 4th frame for gt mode.')
        all_img_list = [v for v in all_img_list if v.endswith('im4.png')]
        keys = [v for v in keys if v.endswith('_4')]

if mode is gt, it's ok. If mode is lq, the variable 'all_img_list' is empty and vimeo90k_train_LR7frames.lmdb is empty.
Is that right? why?

Hello @hahnjiang, sorry for the late reply.
Based on your description, it seems that your lq_path does not contain valid images. Could you provide us with more details?