update to scrcpy v2.4 (debugging)

1、更新到scrcpy v2.4 目前最新版(2024-03-03发布)
2、home back click 已测试

@aentwist Do you have any chance to test this PR? I don't have any android device currently.

I don't know if it is normal but Im actually using an Android emulator on my PC, ldplayer in my case 😅 That is how my project is. Although I do have an android I am not up to speed on using it. Anyways testing using an emulator is probably fine, but I don't really have an interest in this so I won't promise anything right now.

I don't know if it is normal but Im actually using an Android emulator on my PC, ldplayer in my case 😅 That is how my project is. Although I do have an android I am not up to speed on using it. Anyways testing using an emulator is probably fine, ~but I don't really have an interest in this~ edit: I couldnt read it, this is very interesting but I dont want to commit the time right now to figure out how to do thorogh testing. so I won't promise anything right now.

Hey, Im open up for testing only problem is I cant build the av version of your repo. Could u sync your fork for it to work?
Thanks in advance

Seems to work smoothly, thanks!

Cool

This commit has the test suite failing, looks like only one test. The test probably just needs updated for the new version of scrcpy, but I don't know whether anyone is up to speed enough at the moment to fix it.
FAILED tests/test_control.py::test_control_touch - AssertionError: assert b'\x02\x00\x1...0\x00\x00\x01' == b'\x02\x00\xf...0\x00\x00\x01'

I think I can handle this
https://github.com/Genymobile/scrcpy/issues/673#issuecomment-516360374
https://github.com/Genymobile/scrcpy/blob/af573090741e73c66c4a543dcd94fd771c51b7be/app/tests/test_control_msg_serialize.c#L77-L114

Ok got it, the test did just need updated. Next time maybe try using it... spooky otherwise :)


Fix a bug where rotating causes previous state to be lost. Also fixes…

… a crash when allowed invitee roles are fetched, if a user chooses a given collaborator the owned by status is unavailable.

Verified that @doncung has signed the CLA. Thanks for the pull request!


Can't use on Codepen - Uncaught ReferenceError: process is not defined

Hi, i try to create example on Codepen, but when I import
https://cdnjs.cloudflare.com/ajax/libs/react-chartjs-2/2.7.2/react-chartjs-2.js
I have errors in console -
Uncaught ReferenceError: process is not defined
at react-chartjs-2.js:80
at react-chartjs-2.js:4
at react-chartjs-2.js:5
May be you know how to avoid this errors?
Codegen example - https://codepen.io/pelenthium/pen/pKJGge

Yup, I fail to understand how to use the lib at jsfiddle as well https://jsfiddle.net/05n7hogj/3/ :/ It was very easy with previous versions

Should be fixed in https://github.com/jerairrest/react-chartjs-2/releases/tag/2.7.4


Fix images to work with koji 1.18

In koji-builder, the koji package now needs to be installed explicitly.
In koji-db, we need an up-to-date schema. Koji no longer does branches
for individual versions, so let's just put the entire schema in git.
^ NOTE: ideally, the schema would be taken from an image which installs koji to make sure it matches the installed version, that way, we would not have to keep it up to date manually. Rest assured, in the new openshift-y box, that will be the case.

@athos-ribeiro We are installing koji in most images but not the database. Additionally, since the database is based on the postgres image, which is based on Debian, I have no idea how to install koji there (much less guarantee that the version matches what is installed in the other images).
I guess we could put the schema in one of the shared volumes and change the koji-db entrypoint so that it would first copy the schema to the /docker-entrypoint-initdb.d/ directory and then run the default postgres entrypoint script.
Or, if we had a version of docker which supports multistage builds, we could use that, but we don't.
I could try the shared volume approach if you think it is worth looking into.

Not really, since things will change on the openshift box. Thanks for the explanation :)


Fix LDFLAGS in npm-installed zq build

NPM-installed zq started life in #712. Recent overhaul to reporting zq versions in #1249 and further refined in #1256 still left versioning broken in the NPM-built artifacts. Shout out to @mikesbrown who left us the helpful reminder back in #712 [😄 ]:
# If VERSION or LDFLAGS change, please also change
# npm/build.

I've tested the fix in this branch via:
$ git clone https://github.com/brimsec/brim.git
$ cd brim
$ npm install https://github.com/brimsec/zq#5c1d5adc0482b26e321500455477984eb4593a4e
$ npm install
$ zdeps/zq -version
Version: 5c1d5ad
$ npm start
...

Then in another shell:
$ curl http://localhost:9867/version
{"version":"5c1d5ad"}

Fixes #1295.

I appreciate the bonus 👍  from @mikesbrown. Once a Brimmer, always a Brimmer. 😉


Plotting upgrades

[x] Tests added (for all bug fixes or enhancements)
[x] Tests passed (for all non-documentation changes)
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

plot.line() now accepts xincrease, yincrease kwargs (see bottom panel of image).
Additionally, instead of calling autofmt_xdate() which deletes all subplot x-axes except the last one, I rotate and set alignment for the x-axes labels manually. This allows for creating figures where not all subplots have time on the x-axis.

This last commit prevents this warning:
"ResourceWarning: unclosed file <_io.BufferedReader name='/home/deepak/work/python/xarray/xarray/plot/default_colormap.csv'>
  cmap = _load_default_cmap()"

thanks!


ERROR: [FranceTVSite] 1289293-la-terre-vue-de-l-espace: Unable to extract video ID

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.08.19 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
fFrance
Provide a description that is worded well enough to be understood
$ yt-dlp https://www.france.tv/france-2/la-terre-vue-de-l-espace/la-terre-vue-de-l-espace-saison-1/1289293-la-terre-vue-de-l-espace.html
[FranceTVSite] 1289293-la-terre-vue-de-l-espace: Downloading webpage
ERROR: [FranceTVSite] 1289293-la-terre-vue-de-l-espace: Unable to extract video ID; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
$ yt-dlp -U
Latest version: 2022.08.19, Current version: 2022.08.19
yt-dlp is up to date (2022.08.19)
Same URL play well in Firefox 103.0
https://www.france.tv/france-2/la-terre-vue-de-l-espace/la-terre-vue-de-l-espace-saison-1/1289293-la-terre-vue-de-l-espace.html
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ yt-dlp -vU https://www.france.tv/france-2/la-terre-vue-de-l-espace/la-terre-vue-de-l-espace-saison-1/1289293-la-terre-vue-de-l-espace.html
[debug] Command-line config: ['-vU', 'https://www.france.tv/france-2/la-terre-vue-de-l-espace/la-terre-vue-de-l-espace-saison-1/1289293-la-terre-vue-de-l-espace.html']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.08.19 [48c88e088]
[debug] Lazy loading extractors is disabled
[debug] Python 3.8.10 (CPython 64bit) - Linux-5.4.0-124-generic-x86_64-with-glibc2.29 (glibc 2.31)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, pyxattr-0.6.1, secretstorage-2.3.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.08.19, Current version: 2022.08.19
yt-dlp is up to date (2022.08.19)
[debug] [FranceTVSite] Extracting URL: https://www.france.tv/france-2/la-terre-vue-de-l-espace/la-terre-vue-de-l-espace-saison-1/1289293-la-terre-vue-de-l-espace.html
[FranceTVSite] 1289293-la-terre-vue-de-l-espace: Downloading webpage
ERROR: [FranceTVSite] 1289293-la-terre-vue-de-l-espace: Unable to extract video ID; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/home/christophe/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 666, in extract
    ie_result = self._real_extract(url)
  File "/home/christophe/.local/lib/python3.8/site-packages/yt_dlp/extractor/francetv.py", line 292, in _real_extract
    video_id, catalogue = self._html_search_regex(
  File "/home/christophe/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 1254, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "/home/christophe/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 1218, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)

Can you post a picture that it works in the browser?

Hi,
Today, after your message, I try to continue to show the video online (I begin to show yesterday), and it works.

$ uptime
09:30:16 up 2 days, 19:44,  1 user,  load average: 0,75, 1,39, 0,93
$ date
lun. 22 août 2022 09:31:04 CEST

But, after refreshing the page, I see the same message as you (and I cannot see the end now... sad)

As the video is not available, can you close this issue?


Chrome's CSP Blocks loading of bundle.js

On Chrome OSX v49.0.2623.87 (64-bit), the application suffers a fatal crash, because the js/bundle.js refuses to be loaded.
In my console I get the following message:
Refused to load the script 'http://localhost:3000/js/bundle.js' because it violates the following Content Security Policy directive: "script-src 'none'".

This happens right after cloning / npm install / npm start, so I haven't modified any files yet.

I did some digging around, and this was actually caused by me disabling JS in the browser. :disappointed_relieved:


formatBlock crashes with EXC_BAD_ACCESS on Swift

After importing UICountingLabel into a Swift project, setting format works as expected:
myLabel.format = "$%d asdf" // This works

I then attempt to set formatBlock:
totalDebtLabel.formatBlock = {(debtValue) -> String in
    let returnString = "$\(debtValue) asdf"
    return returnString
}

I expect this to work similarly, but instead I get an EXC_BAD_ACCESS crash:
* thread #1: tid = 0x123f44, 0x01f500be libobjc.A.dylib`objc_msgSend + 26, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x22281)
    frame #0: 0x01f500be libobjc.A.dylib`objc_msgSend + 26
    frame #1: 0x01f50eb1 libobjc.A.dylib`objc_release + 65
  * frame #2: 0x000edc05 ExampleApp`reabstraction thunk helper from @callee_owned (@unowned Swift.Float) -> (@owned Swift.ImplicitlyUnwrappedOptional<Swift.String>) to @callee_unowned @objc_block (@unowned Swift.Float) -> (@autoreleased Swift.ImplicitlyUnwrappedOptional<ObjectiveC.NSString>) + 229 at ViewController.swift:31
    frame #3: 0x00122dfc ExampleApp`-[UICountingLabel setTextValue:](self=0x7a099390, _cmd=0x00129f4e, value=20.8544655) + 348 at UICountingLabel.m:204
    frame #4: 0x00122bfd ExampleApp`-[UICountingLabel updateValue:](self=0x7a099390, _cmd=0x00129fe9, timer=0x79ece300) + 621 at UICountingLabel.m:190
    frame #5: 0x00957819 Foundation`__NSFireTimer + 97
    frame #6: 0x002990b6 CoreFoundation`__CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ + 22
    frame #7: 0x00298a3d CoreFoundation`__CFRunLoopDoTimer + 1309
    frame #8: 0x00257e6a CoreFoundation`__CFRunLoopRun + 2090
    frame #9: 0x0025737b CoreFoundation`CFRunLoopRunSpecific + 443
    frame #10: 0x002571ab CoreFoundation`CFRunLoopRunInMode + 123
    frame #11: 0x04f382c1 GraphicsServices`GSEventRunModal + 192
    frame #12: 0x04f380fe GraphicsServices`GSEventRun + 104
    frame #13: 0x00d6c0c6 UIKit`UIApplicationMain + 1526
    frame #14: 0x000f02ee ExampleApp`top_level_code + 78 at AppDelegate.swift:12
    frame #15: 0x000f032b ExampleApp`main + 43 at AppDelegate.swift:0
    frame #16: 0x026aaac9 libdyld.dylib`start + 1

I attempted to debug the problem using image lookup --address and the Zombie profiler, but didn't find anything interesting.

I was able to resolve the problem by having the format block return String! instead of String.

good to know, that's an interesting caveat. Probably the auto-generated Swift bridge expects implicitly unwrapped optionals for NSString * instances. Strange that you can't pass String into String! though, and that this kind of type mismatch isn't caught by the supposedly strongly-typed compiler.
I was very excited for Swift but it seems to have a while to go before it's fully baked...


conda.el no longer seems to work with conda 4.6.1

I diagnosed this by adding the following line to ~/.condarc:
allow_conda_downgrades: true

This let me alternate between these two conda versions in my base environment (I am using miniconda):
conda install conda=4.5.12 # Works

and
conda install conda==4.6.1 # Doesn't work

Restarting emacs between these two versions, I can run conda-env-activate and verify that running which python using a shell command (i.e via M-!) results in the appropriate path when using conda 4.5.12 (e.g ~/miniconda3/envs/{ENV}/bin/python) but this path is inappropriate when using 4.6.1 (~/miniconda3/bin/python). My best guess is there has been some change in how environment activation works.
This is a shame as 4.6.1 has some nice improvements in solver speed!

I found that at least using emacs 26.2 on Windows and the default shell in emacs is cmd, I just overrode  conda--get-path-prefix  like this:
(defun conda--get-path-prefix (env-dir)
  "Get a platform-specific path string to utilize the conda env in ENV-DIR.
It's platform specific in that it uses the platform's native path separator."
  (string-trim (shell-command-to-string (format "conda activate \"%s\"" env-dir))))

I personally did not require the cmd.exe in the original definition of the function although I was experimenting. It may be worthwhile to check which version of conda is being used and fork into separate paths.
It may still require the bash or cmd settings or pulling out what the default shell is for the current emacs environment by pulling out in order until not nil:
explicit-shell-file-name -> (getenv "ESHELL") -> shell-file-name.

@jlstevens closing this as #28 should have fixed it -- please reopen if that doesn't fix it for you!

Great!
I guess another fix will be needed once that deprecation comes round but meanwhile I'll go ahead and test this version (and comment again here if I encounter any problems).

Thank you! I'm planning to look into the fixes needed in conda (and might
ping you, if you have interest) to queue that up, but for now this will
solve our issues.
On Sun, May 19, 2019 at 10:05 PM Jean-Luc Stevens<EMAIL_ADDRESS>wrote:

Great!
I guess another fix will be needed once that deprecation comes round but
meanwhile I'll go ahead and test this version (and comment again here if I
encounter any problems).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/necaris/conda.el/issues/22?email_source=notifications&email_token=AADQPYJYFH3U7TVOKC7JJODPWIBOHA5CNFSM4GSVROUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVXQTMQ#issuecomment-493816242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADQPYKKPIES5ZV2EUX2ECDPWIBOHANCNFSM4GSVROUA
.

--
Rami Chowdhury
"A mind all logic is like a knife all blade - it makes the hand bleed that
uses it." -- Rabindranath Tagore
+1-408-372-6496 / +44-7581-430-517 / +88-0189-245544


Check for ANSI colors in all plain text outputs

I removed the name "fancy outputs" because now all outputs use the same mechanism.
Fixes #330.
Preview of the added documentation:

HTML: https://47-210404706-gh.circle-artifacts.com/0/html/pre-executed.html#Client-specific-Outputs
PDF: https://47-210404706-gh.circle-artifacts.com/0/nbsphinx.pdf#subsubsection.7.1.4

Sadly, there is a problem with LaTeX output:
Page breaks in long code cell outputs with plain text no longer seem to work.
Now there are two nested boxes. Only the outer (framed) box is broken between pages.
If the inner box (created by sphinxVerbatim) is longer than a page, I get an error:
! TeX capacity exceeded, sorry [input stack size=5000].

@jfbu Do you happen to have an idea how to fix this?

@mgeier one can not nest one within another two usages of "framed" environment, but it is possible to put the inner one in a (non-breakable) minipage. If you add
\sphinxverbatimwithminipagetrue

to the "begin" part of your nbsphinxoutput environment, then the inner sphinxVerbatim will use such a minipage. This could avoid the TeX error you reported (untested). But the inner box becomes unbreakable.
The nbsphinxoutput role seems only to use some colours for background and the addition of the prompt? Perhaps you only need to modify the inner sphinxVerbatim way of framing? Perhaps nbsphinxoutput rather than issueing its own "framed" could locally redefine what sphinxVerbatim does? this way we avoid nesting two "framed" things. Is there anything inside the nbsphinxoutput apart from sphinxVerbatim?
For info, here is from LaTeX package tcolorbox documentation, which is a much more modern (but adding big dependencies such as TikZ/pgf) package for breakable boxes:

You can nest an unbreakable tcolorbox inside another tcolorbox, even inside a break- able one. But you cannot not nest a breakable box inside a breakable box. The /tcb/breakable key for a nested box is ignored automatically, i.e. inner boxes are always unbreakable.

in section 18.2 Limitations and Known Bugs.

Looking at the example of nbsphinx own doc I observe that a sphinxVerbatim from inside nbsphinxoutput is always used with \sphinxsetup{verbatimwithframe=false}, and, except for the err output with VerbatimColor={named}{white}. For the standard error stream there is however VerbatimColor={named}{nbsphinx-stderr}.
Thus what remains from sphinxVerbatim usage of framed.sty commands is almost limited to the "continues on next page", "continued from previous page" thing. A strategy could thus be to transfer this mechanisme to outer nbsphinxoutput  environment "output continues on next page", "output continued from previous page", and to castrate the sphinxVerbatim to not use "framed". (the verbatim output by itself allows pagebreaks; the usage of "framed" is only to add background color, and a frame which may incorporate a caption and also implements the "continues/continued"  things)
This appears to be feasible but a bit involved. The simplest could possibly be if you could repalce on the fly "sphinxVerbatim" by "nbsphinxVerbatim", and define the "nbsphinxVerbatim" environment as a copied pasted "sphinxVerbatim" but with the pieces related to Framed removed (of course I can help there). It could also be possible to rather hack the environment to locally alter (via some tricks of LaTeX macros) the meaning of \MakeFramed etc to become no ops, this is more hacky, although some of it looks easy for example exploiting that there is this in original:
  \let\FrameCommand     \sphinxVerbatim@FrameCommand
  \let\FirstFrameCommand\sphinxVerbatim@FirstFrameCommand
  \let\MidFrameCommand  \sphinxVerbatim@MidFrameCommand
  \let\LastFrameCommand \sphinxVerbatim@LastFrameCommand

whose effect can esaily be reverted by locally redefining the right hand sides of the assignments.
In the case of standard error stream, will the box contain anything else? if not then background color nbsphinx-stderr usage can simply get moved to nbsphinxoutput way of using "framed" commands. Then the above strategy looks feasible.

Sorry about the \sphinxverbatimwithminipagetrue advice. Although it does work around some difficulties with the nesting of framed environment, it can not help (in fact it aggravates it) avoid a problem which breaks PDF build under certain circumstances https://github.com/sphinx-doc/sphinx/issues/6804
Best is really to try to avoid nesting "framed" environments... but as you see from Sphinx ticket I referred to, this is not always easy.

@jfbu Thanks a lot for your help!
I tried your suggestion and used Verbatim instead of sphinxVerbatim:  #350.
This basically works, but the verbatim lines are too far apart.
Do you know how to fix this?
If this doesn't work out, I'll have to use sphinxVerbatim again, but without the outer "framed" environment.
I haven't tried that yet, I don't know which of the two options is more promising.

I have a proof of concept of an adapted sphinxVerbatim which can be nested a there
What this does in case of nesting (which here is always the case) is that it measures the length of the code-block and if it is not too close to the textheight it uses the normal sphinxVerbatim (but in a minipage, hence unbreakable), else it uses castrated sphinxVerbatim i.e. without framing. One then keeps things such as line numbers or highlighted lines but one loses the caption if there was one (this is provisory).
For better helping I would need to get a feeling for what can be there in nbsphinxoutput:  multiple sphinxVerbatim's or only one? Is it a problem if individual sphinxVerbatim lose background color`? can they potentially come with some caption?
My inclination is that "continues/continued" hints should be moved over from sphinxVerbatim to nbsphinxoutput and indeed sphinxVerbatim therein should be a version not using \MakeFramed. The latter can be achieved in various ways.

For using the WIP on sphinx-doc/sphinx#6804 in the context here, the nbsphinxoutput environment must issue \makeatletter\spx@inframedtrue\makeatother
but I think it is better altogether indeed if nested sphinxVerbatim simply is one without usage of \MakeFramed. I think ultimately at sphinx-doc/sphinx I will have to define two environments one using "framed" the other not using it (one needs two, because the mechanism for the caption will have to be different; it may be possible to keep using a single env I have to think and consider backwards compatibility)

Although I think you are moving towards not nesting things (which if you can, is indeed the best), I point out that my WIP branch on fixing sphinx-doc/sphinx#6804 is probably done. With this version of sphinxVerbatim you would not have encountered the PDF crash error here.

I think #351 is a better alternative, but I still want to answer your questions:

For better helping I would need to get a feeling for what can be there in nbsphinxoutput:  multiple sphinxVerbatim's or only one?

Only one.

Is it a problem if individual sphinxVerbatim lose background color`?

Outputs containing stderr should have a special background color.
But it doesn't matter if the outer framed environment or the verbatim environment has that background color.

can they potentially come with some caption?

No. Unless you count the prompt as a caption.

My inclination is that "continues/continued" hints should be moved over from sphinxVerbatim to nbsphinxoutput and indeed sphinxVerbatim therein should be a version not using \MakeFramed.

Yes, that sounds reasonable.
But probably completely avoiding the nested environments is even better, like I've done in #351.

Sorry, I've just missed your latest comment when I posted my comment above ...
I will try your WIP branch.

OK, I tried your branch and when I add \spx@inframedtrue to the nbsphinxoutput environment, it works!
There is no crash and page breaks happen correctly.
Of course this would only work in future Sphinx versions.
Is there a reason to pursue this PR, or is it better to use #351 instead?
I guess based on your comment https://github.com/spatialaudio/nbsphinx/pull/351#issuecomment-552187755 I should use #351, right?

Yes, I agree #351 which avoids nesting things is definitly to be preferred. Although future Sphinx avoids fatal errors if long code-blocks get nested in another "framed" thing, it is still a bit bad that they lose background color generally configured for Verbatim and have no way of inserting the continuation hints across pages.
Thanks by the way for confirmation my WIP appears to work in a systematic nested context which is thus a good test, and for taking the time answering my questions above! (this helps me also for future!)

My inclination is that "continues/continued" hints should be moved over from sphinxVerbatim to nbsphinxoutput and indeed sphinxVerbatim therein should be a version not using \MakeFramed. The latter can be achieved in various ways.

Hi @mgeier I feel bad about adding much entropy at times. I had not looked for a long time at your code and I wrote the above lines without much of a check. Actually the nbsphinxoutput here (or nbsphinxfancyoutput at #351) already does have the needed set-up for the continuation hints reusing the Sphinx macros. Thus all my (repeated) talk about moving these things was non-sense. For some reason I was under impression a plain \MakeFramed was used. Sorry for confused talk.

@jfbu Don't worry, I'm very grateful for all your help!
I'll close this PR now, because it has been superseded by #351.


[Google Dynamic Links for Firebase] Keep deeplink state when return to forceground

I implement [Google Dynamic Links for Firebase] for deeplink and catch scene:

My flutter app (android) has home page and detail page.

Click web link to open app > open detail page (deep link contain page id)
Back to home page in app and press home to make app as background.
Reopen app.
-> Result: app opened and auto direct to detail page (same click deep link case)
Expected: app opened only, not auto direct to detail page. (deep link state should clear)

Please check it. Tks!

Is this like #20944?

@vu111293
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/720. Any further collaboration will be done there.


datacapture : Bug in binding UI element to Questionnaire Item

https://user-images.githubusercontent.com/48797364/103201704-8d14f080-4916-11eb-8338-1e1cf7cc68db.mp4
Sample questionnaire modified by duplicating items. linkid of each item is unique

Thanks Kashyap for creating this issue. Can you please push a branch with your code?
I think there's a real problem with the use of activityViewModels in QuestionnaireFragment.kt. I tested that passing different questionnaires to different questionnaire fragments will reuse the same view model which is definitely incorrect. We need to somehow change the scope of view models to fragment...

@jingtang10 Please check this branch https://github.com/google/android-fhir/tree/kj/sdc-binding-bug

@kashyapj2793, I created PR #134 and tested with your change which I think is working. To verify that on your end, please rebase your branch onto the jt/sdc-set-values branch.
The problem you encountered is very interesting. It's because recycler view reuses (and it's great to see that in action because it improves performance) the questionnaire item views. So if you have ticked a checkbox and scroll it outside of the viewport, recycler view might reuse that checkbox to render another question if that question is to be scrolled into the viewport. Because we had not implemented loading the answers properly from the view model, so when that happened, the adapter didn't set the state of the checkbox correctly. So what you ended up with was a checkbox that had been ticked for a question being reused for another question without being cleared.

I have verified that PR #134 fixes this issue

Thanks Kashyap -- let's keep this open until that PR is merged into master.

I have reset the tip of this branch to the first commit without the fix from the jt/sdc-set-values branch. This is so that until the PR is merged to master, the issue can be reproduced using this branch.

Sorry I didn't mean to close this.
Kashyap -- can you please verify this is indeed fixed for you (rebasing your branch onto master).

Sorry I didn't mean to close this.
Kashyap -- can you please verify this is indeed fixed for you (rebasing your branch onto master).

No problem jing! Yes I have verified that this is fixed!

No problem jing! Yes I have verified that this is fixed!


[Idea for testing scenario] Using fixtures with multiple browser windows/tabs in single test

Hello, can you provide an example of using fixtures in situations when i want to initialize page object for multiple pages inside single test? For example i have Livechat class and i want to init livechat page object for each browser. How to achieve this with using fixtures? I understand how to do this while using a single page instance in your test like this example: https://github.com/MarcusFelling/demo.playwright/blob/main/basic/pom/pom-with-fixtures.spec.ts but not sure how to do this with multiple pages in single test.
  test('Test with multiple livechat windows', async ({ browser }) => {
    const browserWindow1 = await browser.newContext()
    const browserWindow2 = await browser.newContext()
    const browserWindow3 = await browser.newContext()

    const page1 = await browserWindow1.newPage()
    const page2 = await browserWindow2.newPage()
    const page3 = await browserWindow3.newPage()

    const livechat1 = new Livechat(page1)
    const livechat2 = new Livechat(page2)
    const livechat3 = new Livechat(page3)

    await livechat1.startSession()
    await livechat2.startSession()
    await livechat3.startSession()
    ...
   })
})

The best way to approach this would be to create an openLiveChat fixture that is a function - it will create a page and construct livechat. Make sense?


Cant install Pdf.JS on React Native

Configuration:

MacOS and Node 8.x
React Native

Steps to reproduce the problem:

Start a react native project
Yarn add pdfjs-dist
Try to import pdfjs-dist

You will get a error like this

Unable to resolve module ./zlib_bindings ./node_modules/zlib/lib/zlib_bindings doesn't exist

zlib should be available in Node.js. IIRC zlib is only used for the experimental SVG back-end for optimizations, so I'm not sure why you're getting this. Does npm install zlib work? If it does, perhaps we need to add it to the devDependencies in package.json?

Please provide complete example.

Actually, I'll close it won't fix. Even if we will solve zlib_binding, there is no guarantee rest of PDF.js will work -- PDF.js need "native" browser's DOM to work.

React Native code does not run on a full node.js environment, so zlib is not available by default.
If I just add pdfjs-dist to package.json it goes to a zlib com not be found error.
Then I added https://www.npmjs.com/package/zlib
And got that error,
I think it should threat React Native as a browser js not as a node

is it possible to use webGL to achieve this ?

is it possible to use webGL to achieve this ?

Anything is possible -- you just need to implement logic similar to what src/display/canvas.js or src/display/svg.js implements.

Haha thanks you but those files are 2000 lines of code and my webGL knowledge is nihil :)
Also pdfjs doesn't work well with expo yet (zlib). Neither webview
https://github.com/expo/expo-sdk/issues/82

Latter next month I may have to dig in a way to use pdf.js with expo (and react-native).
I think using a abstracted dom or a web view is not the way, maybe gl, svg or canvas, still have to take some time to understand better what pdf.js is capable.
About zlib and other node libraries, I’ve found out you can go around that by implementing some aliases on babel and installing some npm packages. But there is a lot of going around to do and a lot of bugs to catch.

On Aug 23, 2017, at 6:27 AM, De Lille Felix<EMAIL_ADDRESS>wrote:
Haha thanks you but those files are 2000 lines of code and my webGL knowledge is nihil :)
Also pdfjs doesn't work well with expo yet (zlib). Neither webview
expo/expo-sdk#82 https://github.com/expo/expo-sdk/issues/82
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mozilla/pdf.js/issues/8774#issuecomment-324274067, or mute the thread https://github.com/notifications/unsubscribe-auth/AFGkfpPegKgAZd6vkGessquMTtq8VFPwks5sa_CHgaJpZM4O1uKR.

yes, Expo doesn't have way to show PDF right now and neither has PDF.js.
I will use https://github.com/wonday/react-native-pdf and will have to detach from Expo i fear. Don't really want to detach but there is no other way

Was looking into this myself, so figured I might leave some notes for future readers.
Initially I found https://github.com/iddan/react-native-canvas which could move a step forward in at least getting rendering to work. It uses a WebView under the hood.
When looking into WebView support, I did find this issue: https://github.com/mozilla/pdf.js/issues/7157#issuecomment-394066217 , and the comment which points to https://github.com/xcarpentier/rn-pdf-reader-js , which was actually built for Expo. It similarly uses a WebView under the hood (and uses react-pdf instead of pdf.js directly)


Right click enables visual mode by default

Referencing  issue # https://github.com/vim/vim/issues/1188  #1188
I am wondering if this is a bug or by design in VIM 8?  The same is happening via Fedora 25.  When right clicking it puts you into visual mode.  The only fix is to add set mouse-=a or set mouse= to .vimrc
vim-minimal-8.0.124-2.fc25.x86_64
vim-common-8.0.124-2.fc25.x86_64
vim-gtk-syntax-20130716-6.fc24.noarch
vim-filesystem-8.0.124-2.fc25.x86_64
vim-enhanced-8.0.124-2.fc25.x86_64

This is by design. See :help defaults.vim for detail.
This happens if you don't have .vimrc, and also if your .vimrc or the system vimrc source $VIMRUNTIME/vimrc_example.vim.
To work around this, you may want to do one of the following:

Stop sourcing $VIMRUNTIME/vimrc_example.vim.
Add set mouse-=a to .vimrc as you did.
Use another shortcuts. E.g. Shift + Insert, Shift + right click, or etc. (It depends on your terminal software.)

On Wed, Dec 14, 2016 at 7:04 AM, Mas<EMAIL_ADDRESS>wrote:

Referencing issue # 1188
I am wondering if this is a bug or by design in VIM 8? The same is happening
via Fedora 25. When right clicking it puts you into visual mode. The only
fix is to add set mouse-=a or set mouse= to .vimrc
vim-minimal-8.0.124-2.fc25.x86_64
vim-common-8.0.124-2.fc25.x86_64
vim-gtk-syntax-20130716-6.fc24.noarch
vim-filesystem-8.0.124-2.fc25.x86_64
vim-enhanced-8.0.124-2.fc25.x86_64

See :help 'mousemodel'
When 'mousemodel' is set to "extend", a right-click or -drag extends a
selection, a shift-right-click selects the pointed word. When it is
set to "popup" or "popup_setpos" it pops up a context menu.
This option is one of those set by the :behave command. ":behave
mswin" sets 'mousemodel' to "popup", ":behave xterm" sets it to
"extend". Myself, I use neither of these "behaviours", but the
following, which I regard as a kind of "golden middle" between them:
" :behave tonymec
if exists('+selectmode')
  set slm=mouse,key
endif
if exists('+mousemodel')
  set mousemodel=popup
endif
if exists('+keymodel')
  set keymodel=startsel
endif
if exists('+selection')
  set selection=inclusive
endif

Best regards,
Tony.

Thanks guys ,

This was a very bad design decision. Now we got a new entry in this list:

select with left click to copy
select with left click, then right click to copy
select with left click, then right click, then click 'copy' to copy
select with left click, then y to copy
shift select with left click to copy 👎

What to do if you installed VIM 8 by source?

What to do if you installed VIM 8 by source?

How about using a custom .vimrc file?

Adding set mouse-=a to my /etc/vimrc file does not override this setting. ~/.vimrc does not exist nor are there any additional directives below set mouse-=a which may be overriding that setting.

On Linux, 'mousemodel' defaults to extend, meaning that a right clilck extends a selection. If there is no selection yet, it will start Select or Visual mode (depending on the 'selectmode' setting, empty by default, so defaulting to Visual) from the cursor to the mouse pointer location.
To avoid that, you might start a ~/.vimrc with your favourite settings; I recommend starting it with the line
runtime defaults.vim

to get the "enhanced" defaults that Vim 7.4.2111 and later sets if no vimrc is found. Add after that any settings or setting changes according to your own preferences, e.g.
set mousemodel=popup

to get a popup menu instead of a visual selection on right-click.
Best regards,
Tony.

@zimmertr
please read the documentation:
:h skip_defaults_vim

how come no other distros have this problem?

how come no other distros have this problem?

I can confirm that happens on Debian 9, Ubuntu 18.04 and Mint 19.2 as well

I came into the same issue when i used vim on my raspberry pi
Both Ubuntu 19.10 and Raspbian Buster use the same version of vim: 8.1.1401
I've compared the /usr/share/vim/vim81/defaults.vim file on both installs, and noticed that the Raspbian version contains the following lines:
" In many terminal emulators the mouse works just fine.  By enabling it you
" can position the cursor, Visually select and scroll with the mouse.
if has('mouse')
  set mouse=a
endif

I've tried to create a ~/.vimrc with the content as mentioned
runtime defaults.vim
set mousemodel=popup

or with:
set mouse-=a

But it's not working. I'd rather not edit files in /usr/share/vim/ because they are placed there by the package manager. What is the correct fix so i can copy paste into my terminal again?

that should work in theory. Try :verbose mousemodel?  to find out where and when it has been changed.

set mouse =
Works for me, that is set mouse equals to nothing
El mar., 12 de nov. de 2019 19:43, Jimbolino<EMAIL_ADDRESS>escribió:

I came into the same issue when i used vim on my raspberry pi
Both Ubuntu 19.10 and Raspbian Buster use the same version of vim: 8.1.1401
I've compared the /usr/share/vim/vim81/defaults.vim file on both
installs, and noticed that the Raspbian version contains the following
lines:
" In many terminal emulators the mouse works just fine.  By enabling it you
" can position the cursor, Visually select and scroll with the mouse.
if has('mouse')
set mouse=a
endif
I've tried to create a ~/.vimrc with the content as mentioned
runtime defaults.vim
set mousemodel=popup
or with:
set mouse-=a
But it's not working. I'd rather not edit files in /usr/share/vim/ because
they are placed there by the package manager. What is the correct fix so i
can copy paste into my terminal again?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vim/vim/issues/1326?email_source=notifications&email_token=AFP7B2I7LCT3RB4KEVTGF3LQTMWSNA5CNFSM4CZSIEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED4GXWA#issuecomment-553151448,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFP7B2KF7DH3NSKW2E3REEDQTMWSNANCNFSM4CZSIELQ
.

This was a very bad design decision. Now we got a new entry in this list:
* select with left click to copy

* select with left click, then right click to copy

* select with left click, then right click, then click 'copy' to copy

* select with left click, then `y` to copy

* shift select with left click to copy -1

I concur, this new default breaks OS wide UX conventions for right click to context, breaks established user expectations, and modifies the expected behavior of a higher order application (terminal window). This is a UX failure for the text books.
Yes I know I can just add the .vimrc file and restore sanity, but I don't have to do that either to get gedit, my terminal, or LibreOffice to follow this UX pattern. I think a go fund me for some interaction design training would be in order.

Hey what version do I have to downgrade to in order to fix this UX regression? I've got piles of devs whinging like stuffed pigs about this.
Similarly set mouse-=a does nothing, it still goes to 'insert (visual)' mode. Related, there appears to have been another UX failing with auto-indent on paste now.
Solid work angering already short-tempered devs 😂

have you validated your 'mouse' setting? Try running :verbose set mouse? to check where this has been set last.

Yeah the issue was on a fresh install of 8.2. Wiped and tried 9 - same
issue.
Wiped and installed 7.4. All happy again and back to being functional!
------ Original Message ------
From: "Christian Brabandt" @.>
To: "vim/vim" @.>
Cc: "platima" @.>; "Comment"
@.>
Sent: 9/08/2022 13:59:13
Subject: Re: [vim/vim] Right click enables visual mode by default
(#1326)
have you validated your 'mouse' setting? Try running :verbose set mouse?
to check where this has been set last.
—
Reply to this email directly, view it on GitHub
https://github.com/vim/vim/issues/1326#issuecomment-1208948846, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIYAQFGS74HFTSH47A2HKTVYHXTDANCNFSM4CZSIELQ.
You are receiving this because you commented.Message ID:
@.***>

instead on relying on an ancient version (which BTW may have security implications!), I would rather follow the advise from :h skip_defaults_vim to stop sourcing it.

The problem is when you have a dozen folks that have used Vim for 20
years each, and it changes, not to mention they are jumping into SSH
sessions, across computers, etc, it's not exactly feasible to update
vimrc for each as that is user-level. Easier to roll back versions to
one before the UX was broken.
Note I had a gander and found hundreds of complaints on Reddit and
StackOverflow etc about this. Seems like a 'feature change' damn-near
has resulted in a bug. https://i.redd.it/it39s2ehph751.png
------ Original Message ------
From: "Christian Brabandt" @.>
To: "vim/vim" @.>
Cc: "platima" @.>; "Comment"
@.>
Sent: 9/08/2022 15:30:45
Subject: Re: [vim/vim] Right click enables visual mode by default
(#1326)
instead on relying on an ancient version (which BTW may have security
implications!), I would rather follow the advise from :h
skip_defaults_vim to stop sourcing it.
—
Reply to this email directly, view it on GitHub
https://github.com/vim/vim/issues/1326#issuecomment-1209015902, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIYAQFISHR7TNUZCQ6H4ADVYICKLANCNFSM4CZSIELQ.
You are receiving this because you commented.Message ID:
@.***>

I don't buy that it is easier to roll back vim on remote ssh boxes than to either touch ~/.vim/vimrc or either to disable defaults.vim as mentioned here and on several other related issues.

You made your own point there; '~' - with our security we do not have
access to our users home directories. That is up to them.
Nonetheless, this is a UX regression and the majority agree, so we're
all working on ways around it.
------ Original Message ------
From: "Christian Brabandt" @.>
To: "vim/vim" @.>
Cc: "platima" @.>; "Comment"
@.>
Sent: 9/08/2022 16:04:45
Subject: Re: [vim/vim] Right click enables visual mode by default
(#1326)
I don't buy that it is easier to roll back vim on remote ssh boxes than
to either touch ~/.vim/vimrc or either to disable defaults.vim as
mentioned here and on several other related issues.
—
Reply to this email directly, view it on GitHub
https://github.com/vim/vim/issues/1326#issuecomment-1209049354, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIYAQEMXBH77J4FAIAVMZDVYIGJ3ANCNFSM4CZSIELQ.
You are receiving this because you commented.Message ID:
@.***>

For your local system it's easy to configure the mouse as you like.
If you use ssh a lot, then use a terminal or something that doesn't forward mouse events.
Unfortunately there is no tool (screen, tmux, whatever) that has a simple switch to drop mouse events.
The xterm sequence to use is "\033[?1000l" or "\033[?1002l" .
Note that if the mouse was disabled by default then the other half of the world would complain that the mouse isn't working...

Actually, it should work if you use a terminal window in Vim:
tmap <LeftMouse> <NOP>
tmap <MiddleMouse> <NOP>
tmap <RightMouse> <NOP>

Use :terminal to start a shell in a terminal window, then you should be able to use "ssh" without any working mouse clicks.
Disclaimer: Haven't tried it much.

Ah, drag events should also be disabled:
tmap <LeftMouse> <NOP>
tmap <LeftDrag> <NOP>
tmap <MiddleMouse> <NOP>
tmap <RightMouse> <NOP>
tmap <RightDrag> <NOP>

You made your own point there; '~' - with our security we do not have access to our users home directories. That is up to them. Nonetheless, this is a UX regression and the majority agree, so we're all working on ways around it.

So far, the best approach in your case would be putting 'set mouse =' in the global vimrc of the machines they use (iirc, /etc/vim/vimrc.local in debian and derivatives), that disables all mouse things vim likes to do

Valid idea - will look into that, thank you!
Was/is there one set of defaults/changes that was made in vim, besides
the mouse thing? Eg auto indenting when pasting etc? Would prefer to
maintain one versions config across.
Thnx! -K
------ Original Message ------
From "emmandyar" @.>
To "vim/vim" @.>
Cc "platima" @.>; "Comment"
@.>
Date 10/08/2022 1:03:06 AM
Subject Re: [vim/vim] Right click enables visual mode by default (#1326)

You made your own point there; '~' - with our security we do not have
access to our users home directories. That is up to them. Nonetheless,
this is a UX regression and the majority agree, so we're all working on
ways around it.

So far, the best approach in your case would be putting 'set mouse =' in
the global vimrc of the machines they use (iirc, /etc/vim/vimrc.local in
debian and derivatives), that disables all mouse things vim likes to do
—
Reply to this email directly, view it on GitHub
https://github.com/vim/vim/issues/1326#issuecomment-1209639509, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIYAQGCQTAYUITGAOMGX5TVYKFMVANCNFSM4CZSIELQ.
You are receiving this because you commented.Message ID:
@.***>

Nice one!
------ Original Message ------
From "Bram Moolenaar" @.>
To "vim/vim" @.>
Cc "platima" @.>; "Comment"
@.>
Date 9/08/2022 10:48:28 PM
Subject Re: [vim/vim] Right click enables visual mode by default (#1326)
Ah, drag events should also be disabled:
tmap
tmap
tmap
tmap
tmap
—
Reply to this email directly, view it on GitHub
https://github.com/vim/vim/issues/1326#issuecomment-1209481845, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIYAQHAIO5L4T3CNTPVX2DVYJVTZANCNFSM4CZSIELQ.
You are receiving this because you commented.Message ID:
@.***>

I was struggling because in the /etc/vim/vimrc (on Debian 12) file there's this nice explanation:
" If you don't want that to happen, uncomment the below line to prevent
" defaults.vim from being loaded.
" let g:skip_defaults_vim = 1
"
" If you would rather _use_ default.vim's settings, but have the system or
" user vimrc override its settings, then uncomment the line below.
" source $VIMRUNTIME/defaults.vim

The first part is "fine", it disables the defaults.vim, but then nothing is loaded so you end up with no syntax highlight or anything.
The problem is the second part. Uncommenting that does not prevent it from being run after the /etc/vim/vimrc has finished running, so the change I did at the end of /etc/vim/vimrc with set mouse=c was being ignored.
Adding the let g:skip_defaults_vim = 1 after the source $VIMRUNTIME/defaults.vim fixes that.
I expect that the $VIMRUNTIME/defaults.vim was supposed to include that set, but it doesn't so it doesn't work as described.

Adding the let g:skip_defaults_vim = 1 after the source $VIMRUNTIME/defaults.vim fixes that.
I expect that the $VIMRUNTIME/defaults.vim was supposed to include that set, but it doesn't so it doesn't work as described.

No, that was a missing step in my instructions. I'll update that information.


eq-rsi-decimals - Add decimals for RSI/MCI

Add decimal place capability for RSI/MCI surveys and remove description relating rounding pounds and ignore decimal places
What is the context of this PR?
Added decimal: 2 to RSI/MCI json files and removed description text. I tested each schema, adding data as either whole pounds, one decimal place, two decimal places and greater than two decimals (plus strings). All actions produced expected outcomes and figures were rendered correctly to two decimal places on the summary screen.
How to review
Test via the application

Its not updating with comma formatting for employee piping

Comma formatting not applied to employees on summary

Tested all form types and all look fine except two issues raised

The comma issues have been addressed, all displays are now correct

@berianjones needs squashing then this can be merged 👍

Squashed and pushed :-)


Linux x86 build is broken at boringssl's HEAD

Build started failing  with this build (Dec 12)
https://source.cloud.google.com/results/invocations/fd79da6f-fe57-4048-bf90-cd321088f4e0/targets/github%2Fgrpc%2Faggregate_tests/tests
[C]       Compiling third_party/boringssl/crypto/obj/obj.c
mkdir -p `dirname /var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/obj/obj.o`
cc -Ithird_party/protobuf/src -Ithird_party/googletest/googletest/include -Ithird_party/googletest/googlemock/include -Ithird_party/boringssl/include -Ithird_party/cares -Ithird_party/cares/cares -Ithird_party/address_sorting/include -g -Wall -Wextra -Werror -Wno-long-long -Wno-unused-parameter -DOSATOMIC_USE_INLINED=1 -Wno-deprecated-declarations -Ithird_party/nanopb -DPB_FIELD_32BIT -O0 -fPIC -I. -Iinclude -I/var/local/git/grpc/gens -D_DEBUG -DDEBUG -DINSTALL_PREFIX=\"/usr/local\" -DGRPC_TEST_SLOWDOWN_MACHINE_FACTOR=1.000000   -Ithird_party/zlib -Ithird_party/boringssl/include -fvisibility=hidden -DOPENSSL_NO_ASM -D_GNU_SOURCE -DWIN32_LEAN_AND_MEAN -D_HAS_EXCEPTIONS=0 -DNOMINMAX -std=c99 -Wsign-conversion -Wconversion -Wshadow    -Wno-sign-conversion -Wno-conversion -Wno-unused-value -Wno-unknown-pragmas -Wno-implicit-function-declaration -Wno-unused-variable -Wno-sign-compare -Wno-implicit-fallthrough  -MMD -MF /var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/obj/obj.dep -c -o /var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/obj/obj.o third_party/boringssl/crypto/obj/obj.c
third_party/boringssl/crypto/hrss/hrss.c: In function 'vec_add':
third_party/boringssl/crypto/hrss/hrss.c:82:1: error: SSE vector return without SSE enabled changes the ABI [-Werror=psabi]
 static inline vec_t vec_add(vec_t a, vec_t b) { return _mm_add_epi16(a, b); }
 ^
[C]       Compiling third_party/boringssl/crypto/obj/obj_xref.c
mkdir -p `dirname /var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/obj/obj_xref.o`
cc -Ithird_party/protobuf/src -Ithird_party/googletest/googletest/include -Ithird_party/googletest/googlemock/include -Ithird_party/boringssl/include -Ithird_party/cares -Ithird_party/cares/cares -Ithird_party/address_sorting/include -g -Wall -Wextra -Werror -Wno-long-long -Wno-unused-parameter -DOSATOMIC_USE_INLINED=1 -Wno-deprecated-declarations -Ithird_party/nanopb -DPB_FIELD_32BIT -O0 -fPIC -I. -Iinclude -I/var/local/git/grpc/gens -D_DEBUG -DDEBUG -DINSTALL_PREFIX=\"/usr/local\" -DGRPC_TEST_SLOWDOWN_MACHINE_FACTOR=1.000000   -Ithird_party/zlib -Ithird_party/boringssl/include -fvisibility=hidden -DOPENSSL_NO_ASM -D_GNU_SOURCE -DWIN32_LEAN_AND_MEAN -D_HAS_EXCEPTIONS=0 -DNOMINMAX -std=c99 -Wsign-conversion -Wconversion -Wshadow    -Wno-sign-conversion -Wno-conversion -Wno-unused-value -Wno-unknown-pragmas -Wno-implicit-function-declaration -Wno-unused-variable -Wno-sign-compare -Wno-implicit-fallthrough  -MMD -MF /var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/obj/obj_xref.dep -c -o /var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/obj/obj_xref.o third_party/boringssl/crypto/obj/obj_xref.c
In file included from third_party/boringssl/crypto/hrss/hrss.c:28:0:
/usr/lib/gcc/i586-linux-gnu/4.9/include/emmintrin.h:1013:1: error: inlining failed in call to always_inline '_mm_add_epi16': target specific option mismatch
 _mm_add_epi16 (__m128i __A, __m128i __B)
 ^
third_party/boringssl/crypto/hrss/hrss.c:82:56: error: called from here
 static inline vec_t vec_add(vec_t a, vec_t b) { return _mm_add_epi16(a, b); }
                                                        ^
third_party/boringssl/crypto/hrss/hrss.c: At top level:
cc1: error: unrecognized command line option "-Wno-implicit-fallthrough" [-Werror]
cc1: all warnings being treated as errors
Makefile:2943: recipe for target '/var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/hrss/hrss.o' failed
make: *** [/var/local/git/grpc/objs/dbg/third_party/boringssl/crypto/hrss/hrss.o] Error 1
make: *** Waiting for unfinished jobs....
2018-12-12 20:03:13,427 FAILED: make [ret=2, pid=709, time=8.9sec]

@agl looks like this is due to recent hrss changes in boringssl's head:
https://github.com/google/boringssl/commits/master

CPUs without SSE should be considered an abomination IMHO. We should enable SSE instruction set in our 32 bits builds. This will prevent grpc to run on... check notes Pentium II CPUs and earlier, that got released 21 years ago.

-march=pentium3 - msse should do the trick.

Thanks for the notice! Fast feedback is really valuable.
Using gcc -m32 -Q --help=target shows that SSE is enabled by default in GCC 8.2.1 on my development machine, but on an older machine with GCC 7.3.0, it isn't. So while gRPC is probably well advised to enable SSE (and maybe SSE2†) for 32-bit builds, BoringSSL should still build without it. 33714 will fix that.
However, please note that we use the Intel SDE simulator to test BoringSSL on a variety of Intel CPUs, but that only goes back to a Pentium 4—so we have no testing coverage for CPUs older than that.
(† Chromium requires SSE2 and has done for some time.)

I'm not positive, but I think some of our 32-bit assembly might even assume SSE2 at this point.

Yeah, I'm fine requiring pentium 4 as our minimum arch too in order to enable sse2.

The build is green again. Thanks for quick fix!
https://source.cloud.google.com/results/invocations/0b7ad326-7615-4078-9ae3-e229f0781183/targets


SDK Documentation ?

Is there any documentation for the new Scripts feature in SCCM in the SDK?  I have been unable to find any information on the server wmi class or how one might run an approved script against a device via C# - thanks.

@MarkWWest are you asking about a specific API? To what are you referring, "for this new feature"?

Yes, I was just wondering if the SCCM SDK document was updated with CB 1802 to include references to managing the new scripts feature via Microsoft.ConfigurationManagement.ManagementProvider
Feature described here:
https://docs.microsoft.com/en-us/sccm/apps/deploy-use/create-deploy-scripts
I was attempting to find a C# ... the class associated with these Script objects , methods, properties, etc. for interfacing this new feature.  I was unable to locate any reference here:
https://docs.microsoft.com/en-us/sccm/develop/osd/operating-system-deployment-computer-management
Any assistance would be appreciated.

Hi, we are also looking for SDK documentation to use script feature in C#. Can you update if its added ?

@MarkWWest are you able to share if you found any reference ?

@kalyangit , I never found any reference.  The integration points were not documented so i went a different direction.  Though i would revisit this again if the documentation became available.

At this time, I don't think there are any APIs available. So I suggest filing product feedback.
I'll keep this issue open for now as I still want to verify.

There are PowerShell cmdlets that you can use, for example: Get-CMScript
For more cmdlets, see the related links section of that article


Use consistent variable names for df and ddf in groupby docs

[x ] Closes #9263
[ ] Tests added / passed
[ ] Passes pre-commit run --all-files

Changes in documentation to be consistent when documenting pandas and Dask dataframes.

Can one of the admins verify this patch?
Admins can comment ok to test to allow this one PR to run or add to allowlist to allow all future PRs from the same author to run.

add to allowlist


SLS Gallery and Virtual Microscope playbooks don't restart web

learning.yml and sls-gallery.yml have omero_web_systemd_start: False because there was originally an issue that the playbook would not work correctly for a fresh install, perhaps because the web server was started before all configuration and certificates were yet in place.

Looks like I applied the same change in learning in c423a647d6205c6562e56f2dd1fe371319c76c40 during a former upgrade. So let's call this done

(Sorry, was off on vacation in the tropics during this! :palm_tree: Just now back last night.)
@jburel: Given my comment in the description, this was tested for both a fresh server install so that comes up happily first time, as well as for updating an existing installation?


Online Serving (Triton / TFServing) with a GPU on Autopilot

DO NOT MERGE until How-to has been approved.
TO DO: Change README to point to final URL

@ahg-g Would appreciate a review on this one as well, thanks!

Any chance of adding that final URL to the README?

Final URL still pending.


Heterogenous serialized graph testing across backends

Based on work started by @csullivan. Compares serialized graph for a given backend with CPU. Currently only enabled to test GPU backend, although that can be tweaked in future.
Currently compares all intermediate data and so disables any fusions (will likely be extended in future PR).

Please Update branch to rerun ONNX tests on current configuration.


[object_store] Failure to send bytes to azure

Describe the bug
Trying to send bytes to azure blob storage results in hyper::Error(http2, Error { kind: GoAway {"'", FRAME_SIZE_ERROR, Library}}}
To Reproduce
Use put_multipart or put as usual on an azure object store
Expected behavior
Bytes should be sent correctly
Additional context
Might be due to azure only supporting http1.1 while object_store tries to send with http2
https://users.rust-lang.org/t/http2-0-frame-size-error-what-doest-that-mean/81106

Accidentally set use http2 only in config


Add force disconnect, signaling, resolution support, archive layouts, broadcasting, and stream info APIs

@jeffswartz @JayTokBox  Thanks for the comments, let's wait for both of your PRs to go in before we merge this


Cannot find attribute "module" when running finetune.py

When I run the finetune.py, it looks like vla model has no attribute "module".

I have figured out what happened. It is because that I didn't use DDP when I run the code, and after using DDP, the bug disappeared.


Bug: If an error occurs when trying to set a like/dislike, the button remains disabled

Fixed by a10cd87 commit


Correct url to autoscaling faq

what

Correct url to autoscaling faq

why

Fix broken link

references

Previous PR https://github.com/open-guides/og-aws/pull/755
https://aws.amazon.com/ec2/faqs/

Hi @darigovresearch. What do you mean by a more specific pull request?

This makes sense; the Autoscaling service as a standalone didn't exist back when this was originally written...


ptp/ptp4u: add default DomainNumber support #219

Summary
Added a default DomainNumber for subscriptions.
Currently it is It's hardcoded to 0 in
https://github.com/facebook/time/blob/main/ptp/ptp4u/server/subscription.go#L205

Test Plan
Unit test and internal manual test on time servers

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

Coverage data is based on head (9d86362) compared to base (969d480).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #228   +/-   ##
=======================================
  Coverage   58.59%   58.59%
=======================================
  Files         102      102
  Lines        9473     9473
=======================================
  Hits         5551     5551
  Misses       3443     3443
  Partials      479      479

Impacted Files
Coverage Δ

ptp/ptp4u/server/config.go
85.71% <ø> (ø)

ptp/ptp4u/server/subscription.go
98.47% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Waiting for squash 👍


hubble/parser/threefour: handle IPv6 CIDR labels

The IP address in IPv6 CIDR labels is represented with - instead of : as labels cannot contain a :. This commit fixes IPv6 CIDR labels parsing by converting - characters back to : before attempting to parse the CIDR string.
Fixes: #11713

test-me-please

Coverage decreased (-0.04%) to 36.911% when pulling 2d9d8c9d9b1ebd2edb2dbb82a669a49e8357ecf4 on pr/rolinh/hubble-parser-ipv6-cidr-labels into b5c5ca9b63cc14db6bf2d8bac5669f20e0b798cb on master.


Support constraints argument in ForceField.createSystem

Currently, calling createSystem with constraints different than None raises an exception. Since the constraints have been implement, we could now in principle support standard OpenMM constraints specification (e.g. HBonds, HAngles) by "appending" the SMIRNOFF definition of constraint to the stored force field specification.

Yes, that would be great. No one has done anything with them yet since John implemented them, so you're paving the way here. THanks for working on it.

I believe the consensus here, for now, was to add constraints specification exclusively through the offxml files and the <Constraints> tag. For some applications such as relative free energy calculations, selected constraints that are hardcoded in the force field can still be removed with openmm.System.removeConstraint(constraint_idx).
I'm going to close this issue to keep the tracker clean, but feel free to reopen if I misunderstood something.


Create Matlab.gitattributes

Ensure *.m and *.mu files containing MATLAB or MuPad code are
recognised as text files, and all other MathWorks proprietary
compiled file types are recognised as binaries. These includes
MATLAB, Simulink, and MuPAD file extensions.
Sources:
https://uk.mathworks.com/help/matlab/matlab_prog/set-up-git-source-control.html
https://uk.mathworks.com/help/matlab/matlab_env/mathworks-file-extensions.html

Thanks!


Apache 2.4 compatibility

Dear developers,
my question is about the upgrade from Apache 2.2 to Apache 2.4.
The code:
<IfModule mod_setenvif.c>
    SetEnvIfNoCase Referer yougetsignal.com spambot=yes
    ....

    Order allow,deny
    Allow from all
    Deny from env=spambo
</IfModule>

cause an error in apache2ctl configtest:

AH00526: Syntax error on line XXX of /etc/apache2/sites-enabled/XXXX.conf:
Invalid command 'Order', perhaps misspelled or defined by a module not included in the server configuration
Action 'configtest' failed.
The Apache error log may have more information.

According to the official page, the directives must be updated.
http://httpd.apache.org/docs/2.4/upgrading.html
How can I adapt the directive to Apache 2.4 sintax?

In 2.4, such access control is done in the same way as other authorization checks, using the new module mod_authz_host. The old access control idioms should be replaced by the new authentication mechanisms, although for compatibility with old configurations, the new module mod_access_compat is provided. It means you have to add additional module to your httpd.conf (or equivalent file). Module can be loaded by adding such line (sometimes you just have to uncomment this line in configuration file):
LoadModule authz_core_module modules/mod_authz_core.so
LoadModule authz_host_module modules/mod_authz_host.so
Or, try something like:

Require env=spambot denied


Max levelled artifact scanned as lvl 2 instead of lvl 20

Current Behavior
I've tried scanning my artifacts several times and it always brought the same result. Tried scanning and different speeds and reinstalled the software but nothing changed.
Expected Behavior
The scanner to correctly exports the data
Reproducing a Behavior
.
Device OS
Windows 10
Genshin Impact Version
3.4
Inventory Kamera Version
1.3.7
Screen resolution, screen mode, and UI scale
1920x1080 Fullscreen
Additional notes and remarks
No response
Evidence
No response
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[ ] I am using the latest version of the scanner.
[ ] I tried setting the scanning delay to maximum, and it still doesn't work.
[ ] I have attached the video recording to the bug report.
[ ] I have attached the error log to the bug report.
[ ] I am available to later explains the issue.
[X] I have disabled all screen filtering software or plugins (i.e. HDR, Image Sharpening) while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

Please reopen this issue if the problem exists on recent versions


Assignment11 - 6ITA - Vrushangi Mistry

#630
📋 Description
JavaScript Programs
📸 Screenshots
[

]

Please sync the forked branch!


Can't use HydratedBloc on non-Flutter platforms?

Describe the bug
Because of it's dependency on path_provider, the app fails on web and desktop (macOS in my case)
[ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: MissingPluginException(No implementation found for method getTemporaryDirectory on channel plugins.flutter.io/path_provider)
#0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:319:7)
<asynchronous suspension>
#1      getTemporaryDirectory (package:path_provider/path_provider.dart:35:22)
<asynchronous suspension>
#2      HydratedBlocStorage.getInstance (package:hydrated_bloc/src/hydrated_bloc_storage.dart:39:49)
<asynchronous suspension>
#3      HydratedBlocDelegate.build (package:hydrated_bloc/src/hydrated_bloc_delegate.dart:26:33)
<asynchronous suspension>
#4      main (package:tavern/main.dart:21:56)

To Reproduce
Steps to reproduce the behavior:

Attempt to use HydratedBloc on web or desktop
See error

Expected behavior
Either have a check for the platform and use a platform io solution, or provide an alternative storage option for the dev to implement.
**Logs **
Run flutter analyze and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.
Paste the output of running flutter doctor -v here.

➜  ~ flutter doctor -v
[✓] Flutter (Channel master, v1.12.1-pre.59, on Mac OS X 10.15.2 19C39d, locale en-US)
• Flutter version 1.12.1-pre.59 at /Users/thinkdigital/development/flutter
• Framework revision 6cb6857f92 (4 days ago), 2019-11-14 16:50:17 -0500
• Engine revision 77c3512ec8
• Dart version 2.7.0
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/thinkdigital/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• ANDROID_SDK_ROOT = /Users/thinkdigital/Library/Android/sdk
• Java binary at: /Users/thinkdigital/Library/Application
Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5977832/Android
Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.2, Build version 11B44
• CocoaPods version 1.8.4
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 41.1.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] Android Studio (version 3.5)
• Android Studio at /Users/thinkdigital/Library/Application
Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5977832/Android Studio.app/Contents
• Flutter plugin version 41.1.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] IntelliJ IDEA Community Edition (version 2019.2.4)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 41.1.4
• Dart plugin version 192.7402
[✓] VS Code (version 1.40.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.6.0
[✓] Connected device (5 available)
• SM G975U          • R58M23BRC7J                          • android-arm64  • Android 9 (API
28)
• iPhone 11 Pro Max • F3A5BA32-2FC7-491B-9862-A42B54C487A5 • ios            •
com.apple.CoreSimulator.SimRuntime.iOS-13-2 (simulator)
• macOS             • macOS                                • darwin-x64     • Mac OS X 10.15.2
19C39d
• Chrome            • chrome                               • web-javascript • Google Chrome
78.0.3904.97
• Web Server        • web-server                           • web-javascript • Flutter Tools

Additional context
Add any other context about the problem here.

This may be related to #28. Checking that out now

Already a feature with the  HydratedStorage option

@ThinkDigitalSoftware as you mentioned you can provide your own HydratedStorage but you can also just use it for desktop out of the box by including path_provider_fde in your pubspec.yaml. Check out the example.


periodic update to Azure Data Collaboration pre preview

This PR contains breaking changes within the same API version
Microsft.DataCollaboration is not public yet and not used by any customer.
Main Changes:

Added 'SystemData' to many resrouces.
Remove 'participant' as property from proposal and make it a standalone resource
Add the 'sign' action to proposal.
Added the 'currentVersion', 'currentVersionTimestsamp' and 'contractStatus' peoperties to proposal to reflect participant signature
Added 'Resolve' action to 'ScriptReference' and 'DataAssetReference' which resolves the original resource being referenced
Add ResorceType to entitlement
Addded more properties to script reference that reflects the referenced script
Fix some text typos and c\p errors

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

While adding new version the recommendation is to copy the existing version into new directory structure for first commit and then push new changes for easy review. Please create new branch with this recommendation for faster review.

New API's are looking ok, I see many changes in the existing objects which will fall in breaking change category. How are you planning to support existing customers/SDK? Please take a look @ Breaking Change Policy

Re breaking changes got confirmation offline from Hila Yehuda "We still haven’t released our private preview to customers, so these changes don’t break existing customers/SDK. Once released, we’ll handle such changes accordingly."
Marking ARMsignoff based on that, please resolve other comments and work with SDK team as per the process given @ https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/202/Overall-Process-of-AME-Onboarding


Bug in computation of covariance matrix (MultivariateNormalDistribution)

One way of creating a MultivariateNormalDistribution is by specifying the mean and three orthonormal directions (with associated variance) (see https://github.com/unibas-gravis/scalismo/blob/master/src/main/scala/scalismo/statisticalmodel/MultivariateNormalDistribution.scala#L159)
The computation does not compute the correct covariance matrix.

Fixed in #205


use kotlinter to get newer ktlint,  forbid trailing commas

Codecov Report
Base: 83.34% // Head: 83.45% // Increases project coverage by +0.10% :tada:

Coverage data is based on head (5ef9588) compared to base (3c1b1f2).
Patch coverage: 81.96% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
+ Coverage   83.34%   83.45%   +0.10%
==========================================
  Files          45       45
  Lines        1201     1215      +14
  Branches      181      181
==========================================
+ Hits         1001     1014      +13
- Misses        140      141       +1
  Partials       60       60

Impacted Files
Coverage Δ

failgood/jvm/src/failgood/ContextDSL.kt
89.47% <ø> (ø)

failgood/jvm/src/failgood/FailGood.kt
65.04% <ø> (ø)

failgood/jvm/src/failgood/Suite.kt
85.00% <ø> (ø)

failgood/jvm/src/failgood/SuiteResult.kt
15.00% <0.00%> (-0.39%)
:arrow_down:

...ilgood/jvm/src/failgood/internal/Junit4Reporter.kt
65.62% <ø> (ø)

failgood/jvm/src/failgood/junit/ContextFinder.kt
82.35% <0.00%> (ø)

...jvm/src/failgood/junit/FailGoodEngineDescriptor.kt
83.33% <0.00%> (ø)

...failgood/junit/FailGoodJunitTestEngineConstants.kt
0.00% <ø> (ø)

failgood/jvm/src/failgood/mock/Mock.kt
90.78% <50.00%> (ø)

.../jvm/src/failgood/junit/FailGoodJunitTestEngine.kt
85.61% <84.61%> (+0.53%)
:arrow_up:

... and 11 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


How did you evaluate AUC of OpenAI series models in CTR task?

Thanks very much!

The OpenAI model provides the probabilities of the top five tokens. If both "Yes" and "No" are among these five tokens, we proceed with the computation. However, if one of them is not included, we assume that the probability of the other one is the smallest among these five. Note that, this is not a comprehensive solution, but an estimated solution.

Got it! Thank you!


Testing with Mocha

Hi,
I need some guidance on creating mocha tests for a Typeframework application. How do I access the controllers and other objects from the app.js file in a test? Can you please provide an example?
Thanks!

Does this work? Trying with Jasmine and dupertest, but i get an error stating that the "json" and "view" functions are not in the controller. Seems like you have to spin up a server then?


Add task-rpms-signature-scan that is required by EC since Nov 01

PRs that use Enterprise Contract (EC) checks are failing because the task rpms-signature-scan is mandatory.
i.e:
https://github.com/app-sre/go-qontract-reconcile/pull/98
This adds the task to the default pipeline.

@chassing beat me, haha.


vm encryption: support single pass

The deal is to incorporate the new extension which supports to encrypt disk w/o users provided service principals; at the same time, cli should still support old flows once detects the older extension still exists in the vm

#6186 fixed it


Clear redundant debug statements in pr-11756

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @boyuanzz
Sorry for the inconvenience :(.

Retest all please


Crash on deallocating

Hey, awesome library!
I am using it in a project and unfortunately experiencing a crash on deallocating. The pullToRefresh is added in viewDidAppear and is not removed manually. But as far as I see, that should not be a problem, because it should be automatically deallocated and remove the KVO observer by itself. Could you please take a look? The crash dump is:
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'An instance 0x791f2400 of class UITableView was deallocated while key value observers were still registered with it. Current observation info: <NSKeyValueObservationInfo 0x79988290> ( <NSKeyValueObservance 0x79988070: Observer: 0x799875e0, Key path: contentOffset, Options: <New: NO, Old: NO, Prior: NO> Context: 0x79987634, Property: 0x79987ed0> )' *** First throw call stack: (     0   CoreFoundation                      0x02a36494 __exceptionPreprocess + 180     1   libobjc.A.dylib                     0x024f0e02 objc_exception_throw + 50     2   CoreFoundation                      0x02a363bd +[NSException raise:format:] + 141     3   Foundation                          0x02129097 NSKVODeallocate + 353     4   UIKit                               0x012b6348 -[UIView(UIKitManual) release] + 142     5   libobjc.A.dylib                     0x0250334f objc_release + 47     6   libobjc.A.dylib                     0x0250452d _ZN12_GLOBAL__N_119AutoreleasePoolPage3popEPv + 371     7   CoreFoundation                      0x029068a8 _CFAutoreleasePoolPop + 24     8   UIKit                               0x009ea9c5 _prepareForCAFlush + 370     9   UIKit                               0x009f8f3d _afterCACommitHandler + 173     10  CoreFoundation                      0x0294875e __CFRUNLOOP_IS_CALLING_OUT_TO_AN_OBSERVER_CALLBACK_FUNCTION__ + 30     11  CoreFoundation                      0x029486be __CFRunLoopDoObservers + 398     12  CoreFoundation                      0x0293e03c __CFRunLoopRun + 1340     13  CoreFoundation                      0x0293d846 CFRunLoopRunSpecific + 470     14  CoreFoundation                      0x0293d65b CFRunLoopRunInMode + 123     15  GraphicsServices                    0x06568664 GSEventRunModal + 192     16  GraphicsServices                    0x065684a1 GSEventRun + 104     17  UIKit                               0x009c6eb9 UIApplicationMain + 160     18  oozima                              0x00163b71 main + 145     19  libdyld.dylib                       0x03c7ea25 start + 1 ) libc++abi.dylib: terminating with uncaught exception of type NSException
Thanks!
EDIT: I think I found a solution. I was not holding a reference to the pullToRefresh itself, because I am only interested in the action. But it seems that the tableView is being deallocated before, and that produces the crash. I am removing the pull to refresh action now by myself and it's working. If you don't see an issue here, you are free to close this issue right away. Thank you!

@BenchR267 I am seeing this problem also.
Are you able to post an example of your solution. I would be eternally grateful. Thanks

@heliumdice I am now just adding the pullToRefresh in viewWillAppear and removing it in viewWillDisappear. Do you need more code / explanation?

@BenchR267
Thanks for your speedy response. I am doing what you are suggesting but to no avail. So I must be missing something. He's my example. I am subclassing a UITableView and have two view controllers which both contain one of these table subclasses. When I push from view controller A to B everything is fine. But when I pop back from B to A that is when I get the crash. Here is some rough code.
TableViewSubclass : UITableView

(void)addPullToRefreshObserver {
[self addPullToRefresh:self.refresher
action:^{
// Do something
}];
}

(void)removePullToRefreshObserve {
[self removePullToRefresh:self.refresher];
}

ClassA : UIViewController

(void) viewWillAppear:(BOOL)animated {
[super viewWillAppear:animated];
[self.tableView addPullToRefreshObserver];
}

(void)viewWillDisappear:(BOOL)animated {
[super viewWillDisappear:animated];
[self.tableView removePullToRefreshObserver];
}

ClassB : UIViewController

(void) viewWillAppear:(BOOL)animated {
[super viewWillAppear:animated];
[self.tableView addPullToRefreshObserver];
}

(void)viewWillDisappear:(BOOL)animated {
[super viewWillDisappear:animated];
[self.tableView removePullToRefreshObserver];
}

Hm, that's almost my exact solution. The only difference is, I am adding the pull to refresh action in the controller.. Sorry, I don't know a solution for now. I can take a look this evening.

I refined my code a little by removing all the pullToRefresh code out of the tableView subclass. But I am still getting the same crash.
An instance 0x1570cda00 of class SHMTableView was deallocated while key value observers were still registered with it. Current observation info: <NSKeyValueObservationInfo 0x157b874a0> (
<NSKeyValueObservance 0x1578080b0: Observer: 0x0, Key path: contentOffset, Options: <New: YES, Old: NO, Prior: NO> Context: 0x0, Property: 0x157975720>
<NSKeyValueObservance 0x15787ee90: Observer: 0x0, Key path: contentSize, Options: <New: YES, Old: NO, Prior: NO> Context: 0x0, Property: 0x1579778c0>
<NSKeyValueObservance 0x157895480: Observer: 0x0, Key path: frame, Options: <New: YES, Old: NO, Prior: NO> Context: 0x0, Property: 0x157895450>
)'

ClassA : UIViewController
- (void) viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];
       PullToRefresh *refresher = [[PullToRefresh alloc] init];
    UIViewController __weak *s = self;
    [self.tableView addPullToRefresh:refresher
                                 action:^{
                                     [s.tableView refreshVideos];
                                 }];
}

- (void)viewWillDisappear:(BOOL)animated {
    [super viewWillDisappear:animated];
    [self.tableView removePullToRefresh:self.tableView.pullToRefresh];
}

ClassB : UIViewController
- (void) viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];
       PullToRefresh *refresher = [[PullToRefresh alloc] init];
    UIViewController __weak *s = self;
    [self.tableView addPullToRefresh:refresher
                                 action:^{
                                     [s.tableView refreshVideos];
                                 }];
}

- (void)viewWillDisappear:(BOOL)animated {
    [super viewWillDisappear:animated];
    [self.tableView removePullToRefresh:self.tableView.pullToRefresh];
}

Hi @BenchR267, @heliumdice! Thank you for reporting the issue and the workaround :)
Definitely, removing and adding again isn't a good solution. I'll investigate the issue. Stay tuned 😉

I spent some times playing around with the bug and wasn't able to fix it 😞

I am seeing the crash as well, here is how I fix it:
I setup the refresher in viewDidLoad
and in the deinit of my view controller
  deinit {
    if let refresher = tableView?.topPullToRefresh {
      tableView?.removePullToRefresh(refresher)
    }

    if let refresher = tableView?.bottomPullToRefresh {
      tableView?.removePullToRefresh(refresher)
    }
  }

Notice that the above code still crashes with the latest version, there is a new guard variable called observes which is never changed, I've sent a PR which may solve this issue.
And the current solution is not ideal, since we still need to manually remove the refresher. I think due to the way the PullToRefresh is implemented, there is no way for refresher to know when the srollView will be deallocated ;-(

Hi @zipme. Thanks for you descriptive comment!
You're absolutely right. I saw how other pull to refresh components are implemented. It seems DefaultRefreshView should contain all logic

@serejahh that could work, but if we let DefaultRefreshView do all the heavy lifting, wouldn't creating a custom refreshView be more difficult?

@zipme This is true too. I've updated readme and demo. I hope usage will be handy enough

@serejahh 👍

i am getting same issue but i dont know the key value name even i didn't add the observer ,i used UIRefreshControl in my app

@ariaboushehri Make sure that you create PullToRefresh object when passing it as parameter:
tableView.addPullToRefresh(PullToRefresh()) { [unowned self] _ in
    //do staff
}

And then remove all pull to refresh elements (removeAllPullToRefresh method is in develop branch, not released in the newest CocoaPods version yet):
deinit {
     tableView.removeAllPullToRefresh()
}


DBSCAN returns core samples

The DBSCAN in python sklearn returns the indices of core samples. It can be beneficial to find representative samples. Will the DBSCAN in linfa have a similar feature?

We're open to contributions implementing this feature, but as far as I'm aware, not one is currently working on implementing this.


index RDA title properties

currently we consider "Titles" (for purposes of what resources are indexed, and what label we show in search results) to be only a couple BF title properties
add RDA title properties too
@alexiskeely @CECSpecialistI:

please comment below adding the names of the RDA properties you want to be considered as titles
please add the URI of an already created resource on stage that has a value for one of these properties, so we can use for testing

has preferred title of work: http://rdaregistry.info/Elements/w/P10223 - WAU:RT:RDA:Work:monograph
has preferred title of expression: http://rdaregistry.info/Elements/e/P20315 - WAU:RT:RDA:Expression:monograph
has preferred title of item: http://rdaregistry.info/Elements/i/P40085 - WAU:RT:RDA:Item:monograph
has title proper: http://rdaregistry.info/Elements/m/P30156 - WAU:RT:RDA:Manifestation:monograph

TO CONSIDER: approach to configuration. users able to configure?

@alexiskeely Can you point us at an example record in Sinopia dev, stage, or prod that uses an RDA title? Thanks!

Sure, sorry for the delay! The resources below are all in dev.
Work- https://trellis.development.sinopia.io/repository/washington/06ee1b9e-f8ab-444b-974a-2fa193e3f6e9
Expression - https://trellis.development.sinopia.io/repository/washington/a6934df9-7158-46d3-a50f-993135ace180
Manifestation - https://trellis.development.sinopia.io/repository/washington/4c10775c-4659-4cd3-ba95-16dff4277571
Item - https://trellis.development.sinopia.io/repository/washington/b05edb8e-6513-403d-bf37-e5e5d2ddf71d


Is there a way to set language for tagging.py other then English and how? Or is English the only supported language?

When you create your ChatBot instance you can specify the language there. Not all languages are fully supported yet, but there are fallbacks in place to prevent errors for languages that can't be indexed as effectively yet.


Have you heard of Denoify ?

Hello Shimataro,
I am the author of Denoify.
Seems like we are both working on the same solution.
Is deno-module-extension-resolver done or are you planning to invest more time in it?
Because if you do maybe we could consider join forces?
Regards

@garronej
Very sorry for late response. I overlooked this issue :bowing_woman:

Seems like we are both working on the same solution.

Yes, our purpose seems to be same!
(but my tool is very poor; just resolving extensions)

Is deno-module-extension-resolver done or are you planning to invest more time in it?

Maybe done since dynamic imports has been supported a while ago #20.

Because if you do maybe we could consider join forces?

Sure thing!
Is there anything that I can help with?


Pin yarl

Description:
As discussed, pin yarl to a fixed version, together with aiohttp.
While I was at it, I updated aiohttp from 2.3.1 to 2.3.2, which is a bugfix release according to their description.
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

We should pin yarl to 13 and not update aiohttp for a hotfix. Going to close this PR and make a new one that we can merge into a hotfix.


Create a proper test for flink-sql-examples

/packit test --labels sql-example

/packit test --labels sql-example

LGTM. I guess, in the future when we add examples with simpler SQL queries, we can check for more specific messages in the output topic.

yeah sure, this just starting point with example, in future we will use proper clients, flink deployments created by builders etc... :)


User cords

The function that users can enter their own coordinates for the qr-code is not yet implemented.

main.py:
There is an idea to implement the function that allows the user to insert coordinates by himself, but it does not work in the current state.
The coordinates are constructed according to the principle coordinates = x1,x2,x3,x4.
Unfortunately the attempt to enter the coordinates as follows fails:
def usr_cords():
   x1= input('X1 Wert')
   x2= input('X2 Wert')
   x3= input('X3 Wert')
   x4= input('X4 Wert')
cords = (int(x1),int(x2),int(x3),int(x4))
return cords
pdf.py:
Error:
raise ValueError("rect must be finite and not empty") ValueError: rect must be finite and not empty

cords can't be (1,1,1,1). It works.


Allow optimizer to run all enabled commands for a filetype

Currently this package will only run the first enabled command for a filetype.
This update should allow it to run every optimizer that is enabled for that filetype.

Thanks, @hailwood - if you fix those small issues I've noted, this should be good to merge.

Hey @stecman - Those changes should now be up.

Awesome!

Brilliant, thanks. Merged and tagged as 2.2.0.


Update custom-wallets.mdx

Updated getUri() function.
Since WalletConnect does not support V1, needed to make getUri() compatible with V2.
In V1, the way to get the URI was -
const { uri } = (await connector.getProvider()).connector;

But this does not work anymore, so need to update the docs
const provider = await connector.getProvider();
const uri = await new Promise<string>(resolve => provider.once('display_uri', resolve))
return uri

@viral-sangani Thanks for spotting this.

Just needs a `pnpm run lint:fix' to fix the CI error.

@DanielSinclair Updated the PR with lint fixes


Ha Doan: Seperation of Concerns, 3 Weeks

Learning Objectives

Priorities: 🥚, 🐣, 🐥, 🐔 (click to learn more)

There is a lot to learn in this repository. If you can't master all the material
at once, that's expected! Anything you don't master now will always be waiting
for you to review when you need it. These 4 emoji's will help you prioritize
your study time and to measure your progress:

🥚: Understanding this material is required, it covers the base skills you'll
need to move on. You do not need to finish all of them but should feel
comfortable that you could with enough time.
🐣: You have started all of these exercises and feel you could complete them
all if you just had more time. It may not be easy for you but with effort you
can make it through.
🐥: You have studied the examples and started some exercises if you had time.
You should have a big-picture understanding of these concepts/skills, but may
not be confident completing the exercises.
🐔: These concepts or skills are not necessary but are related to this module.
If you are finished with 🥚, 🐣 and 🐥 you can use the 🐔 exercises to push
yourself without getting distracted from the module's main objectives.

[x] 🥚 event-driven programming: identify the concept in a JS program via listeners & handlers
[ ]🥚 entry points: describe what an entry point is, there are 2 kinds in the programs for this module - initialization & interaction. identify them in a program
🥚 function roles: describe what function roles are and why they're important. they can identify a function's role given checklists for each role covered in this module:

[x] listeners: functions that attach event listeners to the DOM
[x] 🥚 handlers: entry point for user interactions
[x] 🥚 utils (utilities): pure functions to help do things with data
[ ] 🐣 components: render data into DOM elements to display for the user
[ ] 🐥 custom events: create custom events events in your components, passing useful data between components and handlers

[ ] 🥚 DOM access: You read and write values from the DOM in an event handler
[ ] 🥚 es5 vs. es6: You can demonstrate the change in developer-experience pre and post es6 by stepping through in the debugger and explaining differences in scoping (global vs. script, block vs. local, modules) between two programs with identical user experience but different implementations.
[ ] 🥚 Scope Hierarchy: You is comfortable navigating different scopes in the browser's debugger to understand an existing application (script, module, closure, local, block)
[ ] 🥚 Code Splitting: You can use ES Modules to split your code into multiple files & folders according to function role, data, listeners and initialization. They can use generated dependency diagrams and documentation to understand and navigate this folder structure.
[ ] 🥚 Dependency Graphs: You can use a project's dependency graph to understand how it is organized and to navigate the source code.
[ ] 🥚 Development Strategies: You can write development strategies that have all of the program's state defined at the beginning, and separate each user story into interface and interaction tasks.
[ ] 🐣 Naming Functions: You can come up with clear and helpful names for the functions in your program. A good function name will take into account the function's role and the program's domain, like in the /naming-variables exercises from Debugging.
[ ] 🐣 DOM manipulation: You can manipulate the DOM when implementing level-appropriate user interactions
[ ] 🐣 Isolating Components: You can use a test.html file to render your components with different inputs
[ ] 🐣 Forms: You can do basic handling of form data via event.target.form
[ ] 🐣 Handling events: You can use the event argument to process user interactions, including bubbled events using event.target
[ ] 🐣 Passing Component Unit Tests: You can write vanilla DOM component functions to pass provided unit tests
[ ] 🐣 refactoring: refactor a single-script tutorial-style web page into multiple files using imports and exports
[ ] 🐥 reverse-engineering: You can incrementally reverse-engineer a level-appropriate user interaction following these steps:

init
Listeners
Handlers
(possibly): Utils, Components, Custom Events

[ ] 🐔 From Spec: given user stories, You can develop a site from scratch using a template repository.
[ ] 🐔 Writing Component Unit Tests: You can write unit tests to validate your component functions using BDD syntax

I Need Help With:
Refactoring exercises - I tried to redo them again at home and got stuck at the parts where I thought I understood. It took time but so far, I've finally figured it out.
What went well?
General understanding of DOM manipulation
What went less well?
Refactoring...I understood better when everything was written in one doc.
Lessons Learned
Sunday Prep Work
The group deliverable

.I understood better when everything was written in one doc.

yeah, but kind of the point is to separate into different folders/files as well

@hadoan149 week3 and retro?


Is it possible for UtahFS to cut a release or add a version tag?

Would perhaps like to include UtahFS in MacPorts, but wondering if it's possible to cut a version, whether as a release or git tag?

Ok, I created release v1.0. Lmk if that's good!

Thank you @Bren2010


Fix wrong height used when using scissor mode with render texture of different height from window

Scissor mode implementation uses global window height instead of the current framebuffer height when calculating scissor mode coordinates, leading to inconsistent scissor mode when drawing to smaller framebuffers.

@4yn thanks!