Make the  mapOntologies method accessible

@bgyori  was asking for access to this method to be called chez lui.  I think the re-factoring allows this, but Ben, please check before you approve the PR...?

@bsharpataz, rather than
      val lines = source.getLines().toArray

can you try
      val lines = source.getLines()
      val result: Iterator[ConceptEmbedding] = for {
      ...
      }
      result.toSeq()

The debugger thinks the problem is indeed with delayed evaluation caused by the iterator (which even survives the for loop), but I think that toSeq() (or vector or array) afterwards will prevent the entire file from needing to be read at once.  (I haven't managed to get the environment set up to call this myself.)

I can try later this afternoon, yes :)

See also #520 that you can take credit for.

@bsharpataz any luck on using result.toSeq rather than getLines.toArray?

In the comment above I attempted to fix the issue of getting a string with \n between every character. But I think I must have done something wrong because now I'm getting at "head of empty list" exception and know too little about Scala to solve it. Sorry...

Sorry, i totally forgot to do this! Will do now
On Sun, Nov 25, 2018 at 7:54 PM Benjamin M. Gyori<EMAIL_ADDRESS>wrote:

In the comment above I attempted to fix the issue of getting a string with
\n between every character. But I think I must have done something wrong
because now I'm getting at "head of empty list" exception and know too
little about Scala to solve it. Sorry...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/517#issuecomment-441504354, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFIniclrOBfxoOwXwWIJs9jzwj9Il_Wyks5uy1fMgaJpZM4YryPK
.

@bsharpataz any luck on using result.toSeq rather than getLines.toArray?

I spoke with @marcovzla and he told me that Seq is a trait, so it will try to map it to the closest type of Seq to what it is, giving me a Stream.  So, I can choose List, Array, etc, Array seems fine.  But I will try to wait to the end to not load the whole line at once

@bsharpataz any luck on using result.toSeq rather than getLines.toArray?

I spoke with @marcovzla and he told me that Seq is a trait, so it will try to map it to the closest type of Seq to what it is, giving me a Stream. So, I can choose List, Array, etc, Array seems fine. But I will try to wait to the end to not load the whole line at once

so -- nope.  Seems the autoclose doesn't like me converting after the scope... you're welcome to play

Becky is right, of course.  "this\nis\na\ntest".lines.map { _.toUpperCase }.toSeq doesn't work, but "this\nis\na\ntest".lines.map { _.toUpperCase }.toArray does.  It's probably better than "this\nis\na\ntest".lines.toArray.map { _.toUpperCase } because this one will need room for two arrays, the lines and the uppercase lines.  So, I'm hoping that the toArray can be moved to as late as possible within the block.  (Not that it's a super big deal.)

Looks like I misplaced my conf file but once I reinstated it, it works - thanks for the tip @bsharpataz ! I pushed another commit to rename the IDs used for each ontology in the mapping file. This is good to go as far as I'm concerned.

Sorry, @bsharpataz, but there were merge conflicts with the refactored version in master.  I probably resolved them wrong, but you will be able to fix it in an instant if so.  Search for // Version from master and that might be code that should replace what's there now from this branch.

sorry @kwalcock I don't know what you mean, sorry to be dense!

Can you check OntologyMapper.scala in this branch?  The parts that are likely merged wrong are commented.

ok, yeah, now I see them, ok I'll fix and push... forthcoming

please check me @kwalcock !

You're checked.  It was just unclear to me whether Ben's version was the latest and greatest or not completely.  This compiles and runs fine.  Thanks.


Spike: Algolia search

Although we have a decent search capability in Jekyll it has its limitation. It will not scale with the content of the site and will dramatically slow down once we have a hefty volume of content.
Is it possible for us to use Algolia search? it is used by bootstrap 4 site and they offer the service for free if you display "search by Algolia" sticker - https://www.algolia.com/pricing

Can we use it?
Is SAP legal ok with this?
If not, can we afford to pay for it?

This is how bootstrap implemented search.

@derberg - have you explored this option for Wookie projects?
@joseegm, @xak pls chime in.

 I think it should be easy to get an approval for it as our commerce is already integrated with it https://devportal.yaas.io/services/beta/algoliasearch/latest/
Check with wombats PO for more details.
Anyway, google custom search forever:
https://kubernetes.io/docs/home/
http://loopback.io/doc/
https://devportal.yaas.io/?ft=gcs

maybe later on


YouTube fails with ERROR: unable to download video data: <urlopen error [Errno 111] Connection refused>

Checklist

[x ] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

YouTube fails with ERROR: unable to download video data: <urlopen error [Errno 111] Connection refused>
sudo youtube-dl -v  https://www.youtube.com/watch?v=OkidaZmnADw
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=OkidaZmnADw']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.12.31
[debug] Python version 2.7.16 (CPython) - Linux-5.4.79-v7+-armv7l-with-debian-10.7
[debug] exe versions: ffmpeg 4.1.6-1, ffprobe 4.1.6-1, phantomjs 2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] OkidaZmnADw: Downloading webpage
[youtube] {18} signature length 106, html5 player 5dd3f3b2
[youtube] {22} signature length 110, html5 player 5dd3f3b2
[youtube] {137} signature length 106, html5 player 5dd3f3b2
[youtube] {248} signature length 110, html5 player 5dd3f3b2
[youtube] {136} signature length 110, html5 player 5dd3f3b2
[youtube] {247} signature length 110, html5 player 5dd3f3b2
[youtube] {135} signature length 106, html5 player 5dd3f3b2
[youtube] {244} signature length 110, html5 player 5dd3f3b2
[youtube] {134} signature length 110, html5 player 5dd3f3b2
[youtube] {243} signature length 110, html5 player 5dd3f3b2
[youtube] {133} signature length 106, html5 player 5dd3f3b2
[youtube] {242} signature length 110, html5 player 5dd3f3b2
[youtube] {160} signature length 110, html5 player 5dd3f3b2
[youtube] {278} signature length 106, html5 player 5dd3f3b2
[youtube] {140} signature length 110, html5 player 5dd3f3b2
[youtube] {249} signature length 110, html5 player 5dd3f3b2
[youtube] {250} signature length 106, html5 player 5dd3f3b2
[youtube] {251} signature length 110, html5 player 5dd3f3b2
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://r2---sn-uxaxjvhxbt2u-5at6.googlevideo.com/videoplayback?expire=1609454839&ei=lwDuX4G1N-m-xN8P-pC66Ac&ip=<IP_ADDRESS>&id=o-AMn9rvLtQmuc-AFXijZX1Q9lN-feklEGWDQE18Tg6Ip9&itag=248&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=youtube&requiressl=yes&mh=zg&mm=31%2C29&mn=sn-uxaxjvhxbt2u-5at6%2Csn-hgn7yn7z&ms=au%2Crdu&mv=m&mvi=2&pl=18&initcwndbps=276250&vprv=1&mime=video%2Fwebm&ns=2abIEHX47l7qCpTEPLhlPagF&gir=yes&clen=68877612&dur=1759.033&lmt=1597438225519120&mt=1609433083&fvip=2&keepalive=yes&beids=23886219&c=WEB&txp=5432432&n=I07Zy1kthwsACR_6&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgcgnd8pBk6xMMA1TFT_pmmbf9XYZrXoqXeQ_gSV9h-AkCIBVxEruEuVZ7f0JrTxKs7PbESHaNh-GkLog6Aqn6zoWB&sig=AOq0QJ8wRQIhAIjMSgyZVYstY1R9c7XHlresuEXQmF5pqsR0N_Uf0U9NAiBbn19x-LvryTsmtrzyvok1gS7aRJNQvFN9O7WanphFYw==&ratebypass=yes'
ERROR: unable to download video data: <urlopen error [Errno 111] Connection refused>
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1930, in process_info
partial_success = dl(fname, new_info)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1875, in dl
return fd.download(name, info)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/common.py", line 366, in download
return self.real_download(filename, info_dict)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/http.py", line 351, in real_download
establish_connection()
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/http.py", line 116, in establish_connection
raise err
URLError: <urlopen error [Errno 111] Connection refused>

this link  - https://www.youtube.com/watch?v=OkidaZmnADw
works ok for me.

when I download by youtube-dl  show this error why and what is problem
have a solution

I have Raspberry pi 3 use it as pihole and download  playlist courses from youtube by youtube-dl

Network issue on your side.

Network issue on your side.

Some custom (or default) blocklists on pihole seem to trash *.googlevideo.com queries, so adding the address as a wildcard to the whitelist should fix it completely. Sorry for kinda necro'ing the issue thread, but I haven't seen this written anywhere. Kind of a gesture for the next person.

Some custom (or default) blocklists on pihole seem to trash *.googlevideo.com queries, so adding the address as a wildcard to the whitelist should fix it completely. Sorry for kinda necro'ing the issue thread, but I haven't seen this written anywhere. Kind of a gesture for the next person.

That was it, thanks!


Integrate external manifest file

How do the example load manifest file?
https://github.com/jaketrent/html-webpack-template/blob/master/index.ejs#L2

@tusharmath That's a good question.  I believe that's just an artifact from the template I based this off of.  I can't remember which one that is.  I usually just glance over those lines and haven't even documented it.  If you figure it out, let's add it to the docs.

You could try pushing this issue upstream or using git blame against the original template at html-webpack-plugin. That alone might explain it.

I don't have a ton of context on this, and it's not moving.  Closing for now.  Reopen if you have suggestions/code for how to proceed.  Thanks!


JavaScript Lingo: MDN and Documentation Question issue.

Going though the new Video Challenge and question 3 on JavaScript Lingo MDN and Documentation doesn't seem to be there. Though clicking true answers it correctly, while false gives me the wrong answers animation.

@SaintPeter Not sure if you're the right person to ask, but any idea what this question was supposed to be?

It looks like it's a leftover from the templates I created.
https://github.com/FreeCodeCamp/FreeCodeCamp/blob/staging/seed/challenges/04-video-challenges/jslingo.json#L31
@brianamarie Do you have a 3rd question here, or can you fix this one?


main element with two sections and class 'two-columns' has a too high gap.

example:

A

B

using utility classes fixes this.

A

B


How to adapt path in Config.py?

Hello! @griegler and @dvdhfnr
In README.md you mentioned below:

"Make sure you adapted the paths in config.py to point to the downloaded data!"

Much appreciated if you would specify what action to take, is it changing the value of lpips_root, tat_root, fvs_root OR complement the paths like tat_train_sets = [  ".../training/Barn",.......]?
Thank you!

issue solved


v2.10.x: Raises TypeError: mro() returned a non-class ('PydanticRecursiveRef')

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
After accidentally updating a package from 2.7.x to 2.10.x,
got an error message that I had never seen before.
I don't know if it's known issue or not,
but I'm just writing this in the hopes that it will help improve 2.10.x.
(Tested it and worked fine until 2.9.2)
Have a nice day! 👍
Traceback (most recent call last):
  File "/home/user-ubuntu/projects/~~~/.py", line 26, in <module>
    T12[int, int](values=[CombineModel[int, int](field_1=1, field_2=2)])
    ~~~^^^^^^^^^^
  File "/home/user-ubuntu/projects/~~~/.venv/lib/python3.11/site-packages/pydantic/main.py", line 811, in __class_getitem__
    submodel = _generics.create_generic_submodel(model_name, origin, args, params)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user-ubuntu/projects/~~~/.venv/lib/python3.11/site-packages/pydantic/_internal/_generics.py", line 137, in create_generic_submodel
    created_model = meta(
                    ^^^^^
  File "/home/user-ubuntu/projects/~~~/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 137, in __new__
    cls = cast('type[BaseModel]', super().__new__(mcs, cls_name, bases, namespace, **kwargs))
                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen abc>", line 106, in __new__
TypeError: mro() returned a non-class ('PydanticRecursiveRef')

Example Code
from pydantic import BaseModel

from typing import List
from typing import TypeVar
from typing import Generic

_T = TypeVar("_T")

class GenericBaseModel(BaseModel, Generic[_T]):
    ...

# 😡 BAD CASE
# pydantic==2.10.2
# pydantic_core==2.27.1
# 😀 GOOD CASE
# pydantic==2.9.2
# pydantic_core==2.23.4
class EnumerableModel(GenericBaseModel[_T]):
    values: List[_T]

# 😀 GOOD CASE
# pydantic==2.10.2
# pydantic_core==2.27.1
# 😀 GOOD CASE
# pydantic==2.9.2
# pydantic_core==2.23.4
class EnumerableModel(BaseModel, Generic[_T]):
    values: List[_T]

_T1 = TypeVar("_T1")
_T2 = TypeVar("_T2")

class CombineModel(BaseModel, Generic[_T1, _T2]):
    field_1: _T1
    field_2: _T2

class EnumerableCombineModel(EnumerableModel[CombineModel[_T1, _T2]]):
    ...

EnumerableCombineModel[int, int]

# Traceback (most recent call last):
#   File "/home/user-ubuntu/projects/~~~/.py", line 29, in <module>
#     EnumerableCombineModel[int, int]
#     ~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^
#   File "/home/user-ubuntu/projects/~~~/.venv/lib/python3.11/site-packages/pydantic/main.py", line 811, in __class_getitem__
#     submodel = _generics.create_generic_submodel(model_name, origin, args, params)
#                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
#   File "/home/user-ubuntu/projects/~~~/.venv/lib/python3.11/site-packages/pydantic/_internal/_generics.py", line 137, in create_generic_submodel
#     created_model = meta(
#                     ^^^^^
#   File "/home/user-ubuntu/projects/~~~/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 137, in __new__
#     cls = cast('type[BaseModel]', super().__new__(mcs, cls_name, bases, namespace, **kwargs))
#                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
#   File "<frozen abc>", line 106, in __new__
# TypeError: mro() returned a non-class ('PydanticRecursiveRef')

Python, Pydantic & OS Version
pydantic version: 2.10.2
        pydantic-core version: 2.27.1
          pydantic-core build: profile=release pgo=false
                 install path: /home/user-ubuntu/projects/metalicsoft-v1/.venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.7 (main, Dec  8 2023, 18:56:58) [GCC 11.4.0]
                     platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35
             related packages: fastapi-0.115.5 typing_extensions-4.12.2 pydantic-settings-2.6.1
                       commit: unknown

@kim-youngseop-developer,
Thanks for the report.
This is definitely something we want to fix in a v2.10.X patch.

cc @kc0506, I feel this might be related to the mro stuff...

We are also having the same issue with 2.10.x versions. But we are seeing a different regression with the mro() where we are accessing the mro:
derived_from = some_generic_model.mro()
AttributeError: 'PydanticRecursiveRef' object has no attribute 'mro'

@MarkusSintonen,
Thanks for the report. Could you provide a full MRO for your use case? Thank you!

@sydney-runkle :
from typing import Generic, TypeVar
from pydantic import BaseModel
import pytest

@pytest.mark.parametrize("inner_mro_access", [True, False])  # Both fail but it used to work
def test_bug(inner_mro_access: bool):
    T = TypeVar("T")
    C = TypeVar("C")

    class BaseInner(BaseModel, Generic[T, C]):
        def __class_getitem__(cls, params):
            klass = super().__class_getitem__(params)
            if inner_mro_access:
                klass.mro()
            return klass

    class BaseOuter(BaseModel, Generic[T, C]):
        field1: BaseInner[T, C]

    class Inner(BaseInner[int, C], BaseModel, Generic[C]):
        pass

    class Outer(BaseOuter[int, C], BaseModel, Generic[C]):
        field2: Inner[C]

    # inner_mro_access=False. Fails: TypeError: mro() returned a non-class ('PydanticRecursiveRef')
    # inner_mro_access=True. Fails: AttributeError: 'PydanticRecursiveRef' object has no attribute 'mro'
    Outer[int]

Seems like this only shows up with pretty exotic use cases of generics. We certainly want to fix, but this will fall into the scope of v2.10.4


WIP Log and show task history in the UI

Work in progress
Depends on #21

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0e5430f). Click here to learn what that means.
The diff coverage is 7.76%.

@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage          ?   30.04%
=========================================
  Files             ?       34
  Lines             ?      486
  Branches          ?        0
=========================================
  Hits              ?      146
  Misses            ?      340
  Partials          ?        0

Impacted Files
Coverage Δ

src/Composer/ComposerCommand.php
100% <ø> (ø)

src/Jobs/InstallPackage.php
0% <0%> (ø)

src/Extensions/ExtensionManager.php
0% <0%> (ø)

src/Providers/ComposerEnvironmentProvider.php
0% <0%> (ø)

src/Search/FlagrowApi.php
0% <0%> (ø)

src/Jobs/RemovePackage.php
0% <0%> (ø)

src/Listeners/AddSatisConfiguration.php
0% <0%> (ø)

src/Jobs/BasePackageJob.php
0% <0%> (ø)

src/Listeners/AddApiControllers.php
0% <0%> (ø)

src/Listeners/BazaarEnabled.php
0% <0%> (ø)

... and 7 more

Continue to review full report at Codecov.

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

Sure
Le lun. 10 avr. 2017 à 12:20, Daniël Klabbers<EMAIL_ADDRESS>a
écrit :

@Luceos commented on this pull request.
In src/Listeners/AddSatisConfiguration.php
https://github.com/flagrow/bazaar/pull/31#discussion_r110620306:

*/

public function subscribe(Dispatcher $events)
{

   $events->listen(TokenSet::class, [$this, 'setSatis']);

}

/**

* @param TokenSet $event

*/

public function setSatis(TokenSet $event)
{

   $env = app()->make(ComposerEnvironment::class);

   $editor = new ComposerFileEditor($env->getComposerJsonPath());

   $host = FlagrowApi::getFlagrowHost() . '/api/satis';

   // TODO: if a "repositories" key is already present in composer.json but uses an array instead of an object, the command quietly fails

So after merging, sounds fine with me. We do need to add it before tagging.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/flagrow/bazaar/pull/31#discussion_r110620306, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFBTrDgcPzT-cgMfvIc7BhmSAHycw6B3ks5rugJkgaJpZM4M4IaG
.

How much of this is still open?

I still want to add a way for the user to clear history item and make that listing a bit prettier. We can't ship such a UI...
Also maybe someone could check my table structure, just so I'm sure it's all right 😇

Task model needs a property dates with executed_at in it so that it will be auto-cast to Carbon.
I've noticed a new way of firing command, but a not-so DRY way of adding tasks. Wouldn't a property on the command (or the class name) be sufficient to log into the task? Perhaps stick most of that logic into the BaseCommand instead..

@Luceos there's already a started_at and finished_at attributes. What would executed_at represent ?

Okay, that's fine. But both need to be in $casts in the model for auto casting to Carbon.

Note to self: there are some changes I did not commit in BazaarPage.js that still ended up in the dist extension.js :see_no_evil: I'll push another commit with a clean dist file

I removed the WIP, I'm done with dev for this feature.
Some testing before merging would be a good idea I guess.


Improve Link My Data section in TrainingFrame

Clear added files only shows for titanic, but not for CIFAR for example.
Rename the titles (label label is not very good)

completed and pushed but found bug on the way, will create new issue.


Repeated outlines and also repeated text from above the outline.

There is a systematic repeating of every outline once, and also of some part of the text string once from above the outline. It is the case for all outlines thorough the entire text.
Three examples from the same PDF are given here. One example for the "Abstract" outline, the other example for the "Introduction" outline, and the third for outline "2.1. Materials":
K. Rahmani aMechanical Engineering Department, Bu-Ali Sina University, Hamedan, Iran; bDepartment of Mechanical Engineering, Najafabad Branch
a ABSTRACT
ABSTRACT In this paper, fabrication and characterisation of Mg\u2013SiC nanocomposite are investigated.
In this paper, fabrication and characterisation of Mg 1. Introduction

Introduction Pure magnesium (Mg) and its alloys are widely used in different applications in aerospace, automotive, sports, biomedical, and electronics industries.

Recently, Atrian et al. [ Dynamic powder compaction is employed in this work to fabricate the specimens. Nanocomposites with Mg powder as the matrix and SiC nano particles as the reinforcement are manufactured using a modified SHPB. Then, the role of reinforcing phase (SiC vol.-%) on density, compressive flow stress, micro-hardness, and microstructural evolution of the specimens are investigated.
Dynamic powder compaction is employed in this 2. Experimental procedure
2. Experimental procedure 2.1. Materials
2.1. Materials Commercial Mg powder (size range of 60\u2013150 \u03bcm,
Commercial Mg powder (size range of 60 99.5% purity, irregular morphology, Merck, Germany)

Apologies, I've not been able to duplicate this, what command line options are you using?

Maybe it is me, that don't quite understand what the generated text (JSON format) should contain.
I converted the PDF file into a JSON formatted file from the command line ("burdoc Nov.pdf"). Then I look into this JSON file for specific text strings. I got the same text string multiple times, even it is only listed one time in the PDF file.
What I don't understand is (as examples):

"Abstract"

It is seen 4 times in the JSON file, but only one time in the PDF file.
In the JSON file, it listed both under a "block_text" one time and under a "text" two times (beside under "toc" off course):

{"spans": [{"text": "Najafabad, Iran", "font": {"name": "font", "font": "AdvOT5fcf1b24", "family": "AdvOT5fcf1b24", "size": 9.0, "colour": 0, "bd": false, "it": false, "sp": false, "sc": false}, "name": "span"}], "name": "line"}], "name": "textblock"}, {"type": "h6", "block_text": "ABSTRACT", "items": [{"spans": [{"text": "ABSTRACT", "font": {"name": "font", "font": "AdvOT2bda31c3.B", "family": "AdvOT2bda31c3.B", "size": 7.5, "colour": 11732792, "bd": false, "it": false, "sp": false, "sc": false},

http://www.tandfonline.com/loi/ypom20", "size": 8.0, "assigned_heading": "h6"}, {"page": 0, "index": [4, null], "text": "K. Rahmani, G. H. Majzoobi & A. Atrian", "size": 12.0, "assigned_heading": "h5"}], "1": [{"page": 1, "index": [5, null], "text": "ABSTRACT", "size": 7.5, "assigned_heading": "h6"}, {"page": 1, "index": [10, null], "text": "CONTACT G. H. Majzoobi", "size": 8.0, "assigned_heading": "h6"}, {"page": 1, "index": [15, null], "text": "KEYWORDS", "size": 7.5, "assigned_heading":

Outline "2.1. Materials"

In the JSON file, it listed both under a "block_text" and under a "text" (beside under "toc" off course):
"block_text": "2.1. Materials", "items": [{"spans": [{"text": "2.1. Materials", "font":
I also converted the JSON file to a text formatted file online: https://onlinejsontools.com/convert-json-to-text. In the converted text the word "Abstract" are here also seen 4 times.
If I convert the PDF file to html format, I see no duplicates at all. Maybe all text from the PDF file are and should all be duplicated in the JSON file in "block_text and in "text"?
output-onlinejsontools.txt
Nov.zip
Nov.pdf

Ah yes. Essentially the "block_text" field is simply a concatenation of the text of each line within that block. Personally I found it useful in simplifying processing the data when distinguishing between lines in a block wasn't necessary but maybe's it something I should rethink.

If the text should be doubled in the JSON-formatted file (in "block_text" and in "text"), maybe let that be an option to have it duplicated or not?
For me, I look at a large text like this: It consists of 1) some single lines (one line with an empty line above and below the line) and 2) some sections (multiple consecutive lines forming a single block of text with an empty line above and below the block). I think this is useful in many cases.
Still, I think having the word "Abstract" mentioned four times should not be necessary. At least it is very confusing if I have to look for an abstract programmatically in the JSON-formatted file.
Best regardsFrank
Den fredag den 26. maj 2023 kl. 20.53.33 CEST skrev jennis0 @.***>:
Ah yes. Essentially the "block_text" field is simply a concatenation of the text of each line within that block. Personally I found it useful in simplifying processing the data when distinguishing between lines in a block wasn't necessary but maybe's it something I should rethink.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>


code owners for sql-bindings

Add code owners for sql-bindings extension

If anyone else would like to be added to the list or removed do let me know :) @microsoft/azure-data-studio-notebook-devs


Incorrect FCAP version on 1.20.2

When trying to install the latest version on 1.20.2 (which is marked compatible with it) Fabric will give an error that Forge Config Api Port version 20.4.3 but that version does not exist on 1.20.2 meaning it is impossible to play.

Thanks for the report.


Feature: trigger function when days, hours, minutes turn to 0

Or add the option to remove 0 days, 0 hours, 0 minutes automatically.
so:
0 days 0 hours 22 minutes 30 seconds
would be:
22 minutes 30 seconds

It should be a great option, if you want to contribute you're welcome :smiley:
That may be easly made by adding a new option like autoRemove: true. With default to false.
I suggest to add a parametric class to the main countdown element like -autoremove if set to true to easly update the countdown CSS in that case.
Then, most of the work is to implement conditional checks to hide unwanted elements.

Fixed by 0bd24f66572209391b9fd6358c404ce148c59a35


[FEATURE] Add metrics to mark active master under HA

Is your feature request related to a problem? Please describe.
Add metrics to mark active master under HA, currently there isn't a convenient way to check cluster's active master.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
/cc @who-need-to-know
/assign @who-can-help-you

@nafiyAix  Can close this


=docc Move to multiple Package.swift files

As we agreed in offline discussions we'll adopt this style for the time being.
Also added the preview script

This is addressing review comments form https://github.com/apple/swift-log/pulls?q=is%3Apr+is%3Aclosed that I missed -- moved to multiple package files


metals: 0.10.1 -> 0.10.2

Motivation for this change
Update metals to 0.10.2
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120093 at fb580e73 run on x86_64-linux 1

1 package built successfully:

metals

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/tools/metals/default.nix:29:3:
   |
29 |   installPhase = ''
   |   ^


db reset should rerun migrations

Part of resetting the database should be redoing migrations. At the moment the database ends up empty.

covered in #41


[SIL] Fix [differentiable] where clause parsing.

Parse where clause before closing ].
This fixes parsing canonical SIL files:
# foo.swift: program containing `[differentiable]` attribute.
$ swiftc -emit-sil foo.swift
$ swift foo.sil # now works

@swift-ci Please test tensorflow


[BUG] applied AWB preset stays after reconfiguration

Describe the bug
Disabling the auto white balance after having it initially enabled, with a preset, does not reset the preset, and possibly also leaves the AWB on.
To Reproduce
#!/usr/bin/python3

import time
import libcamera
from picamera2 import Picamera2, Preview

picam2 = Picamera2()
picam2.start_preview(Preview.QT)

preview_config = picam2.create_preview_configuration(
    controls={
        "AwbEnabled": 1,
        "AwbMode": libcamera.controls.AwbModeEnum.Cloudy
    }
)
picam2.configure(preview_config)

picam2.start()
time.sleep(1)

picam2.set_controls({"AwbEnable": 0})
time.sleep(5)

Expected behaviour
I expect for the returned images to be without any presets and AWB.
With the Cloudy preset, it is most obvious, at least in my environment, that the preset stays.
Hardware :

Raspberry Pi 4, 4G
Camera Module v2.1

Additional context
This effect also persists, if the camera is stopped and reconfigured.

Hello, thanks for answering.
Let me reformulate what I am intending.
I would like to be able to remove any preset that has been set before, e.g. enabling AWB and choosing the Cloudy preset, and then disable the AWB and remove any effect/preset had, so basically a clean slate.
I also tried starting and stopping the camera, to no avail. (I am guessing the AWB values get reset after disacquiring or before acquiring the camera).
One use case I am trying to achieve is to apply a manual frame by frame colour temperature, which works by itself, only the image stays 'tainted' by one of the presets.

Is the typo "AwbEnabled": 1 relevant here?  The control is "AwbEnable" unless I'm not finding some alias hidden somewhere.  But it sounds like AwbMode gets set and sticks independent of AwbEnable?

Hi, I guess I'm still not quite sure I understand, but let me try and explain how things work:
AWB controls the red and blue colour gains. You can find them in the image metadata (picam2.capture_metadata()["ColourGains"]). While the AWB is running ("enabled") you can change its mode, for example to cloudy (picam2.set_controls({"AwbMode": libcamera.controls.AwbModeEnum.Cloudy})) and you can also set it back to normal (picam2.set_controls({"AwbMode": libcamera.controls.AwbModeEnum.Auto})).
You can also disable the AWB completely (picam2.set_controls({"AwbEnable": 0})) at which point the colour gains will never change by themselves. You can set the colour gains (picam2.set_controls({"ColourGains": (1, 1)})), or in fact, setting the colour gains will disable the AWB automatically anyway.
When you re-enable the AWB (picam2.set_controls({"AwbEnable": 1})) it will return to whatever the AWB mode was previously (either Cloudy, Auto or whatever). One thing that might seem confusing is that setting an AWB mode does not disable AWB, it merely restricts the range of possibilities that it searches, but it's still running.
All this state is persistent across everything except stopping and restarting the entire camera system. So camera start/stop/configure will have no effect on any AWB settings (unless you explicitly put AWB controls into your camera modes, but that would be your choice).
I hope that's a bit clearer. If not, perhaps try to include an exact script and say at the end exactly how the AWB setting of the captured image is not as you expect, then we can look at that. Thanks!

Hi again,
Thanks for the explanation and sorry for any confusion. Did not know of the gain values and their relevance for the AWB, that was exactly what I have been looking for.
Getting the metadata when AWB is disabled, returns a tuple, which I am guessing are the default values of the camera.
The tuple value, in my case (1.4193880558013916, 1.7060798406600952), I am getting seems to be persistent across different calls.
So in a sense, setting/applying those numbers, after having AWB enabled with an AWB preset, should get me a 'raw' image, i.e. one with the default values.
As you basically said, my issue was, that I was not aware, where those 'gains' were getting applied. So my intention is/was to set the colours back to a state, where no AWB/AWB preset is getting applied.
Unless I am wrong, please correct me if I am, I think this issue is resolved.
Thanks!
I am achieving that currently with the following script:
#!/usr/bin/python3

import time
import libcamera

from picamera2 import Picamera2, Preview

picam2 = Picamera2()
picam2.start_preview(Preview.QT)

preview_config = picam2.create_preview_configuration(
    controls={
        "AwbEnable": 1,
        "AwbMode": libcamera.controls.AwbModeEnum.Cloudy,
    }
)
picam2.configure(preview_config)

picam2.start()
# print(picam2.capture_metadata())
time.sleep(5)
picam2.set_controls({"ColourGains": (1.4193880558013916, 1.7060798406600952)})
# print(picam2.capture_metadata())
time.sleep(5)
# print(picam2.capture_metadata())

Thanks, closing this one. Please open a new issue if you have other questions/problems.


repo(upd): repo init

Reason for This PR

Move repository to the new organization

Description of Changes

Content update

License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

[x] All commits in this PR are signed (git commit -s).
[x] The reason for this PR is clearly provided (issue no. or explanation).
[x] The description of changes is clear and encompassing.
[x] Any required documentation changes (code and docs) are included in this PR.
[x] Any user-facing changes are mentioned in CHANGELOG.md.
[x] All added/changed functionality is tested.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@294b303). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 2cae501 differs from pull request most recent head 0bcc975. Consider uploading reports for the commit 0bcc975 to get more accurate results

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   80.38%
=========================================
  Files             ?       19
  Lines             ?     1718
  Branches          ?        0
=========================================
  Hits              ?     1381
  Misses            ?      282
  Partials          ?       55

Continue to review full report at Codecov.

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


workflows/ipsec: Cover Ingress

/test

/test

/test

/test

The v1.16 backport was reverted, adjusting the labels so it doesn't get lost.


Add Falco enforcement policy

Enforce the installation of Falco Operator and a Falco instance in all clusters tagged "falco: true"

/assign @jrnp7

Hi @ycao56 please review latest commit, updating to latest apiVersion

Not required but I recommend removing
          namespaceSelector:
            exclude: ["kube-*"]
            include: ["*"]

in every configuration policy as you already hardcoded namespace in the objectTemplate. It has higher priority thus namespaceSelector would be ignored.


task: avoid double https perfix

Hi @FahimNsr I added the missing build results. You need to commit these with your pr. everything else looks good to me.

After https://github.com/sitegeist/Sitegeist.Archaeopteryx/pull/33 i could not encounter this issue anymore ;)
Thanks for your fix though too ^^
https://github.com/sitegeist/Sitegeist.Archaeopteryx/assets/85400359/90051459-a901-4864-8ca4-23ba60edeac5
If i missed oversaw something here please fell free to reopen :D


Task - OTU radial - Biological associations

Biological association sector does not work at all. I cannot select a relationship
The error message. This blocks our abilities to edit data.

Also, once I have the modal open on Biological association, it would not close with Esc or X button

Probably a extend issue. Looking


Make container class' event_stream_filters match event_where_clause for ems_events

The Event Mixin implementation that these classes are overriding generates a condition on a single column based on the ems_events association foreign key.  host_id for Host, ems_id for ExtMangementSystem, etc.
This PR tries to add parity between the new event_stream_filters and event_where_clause where container classes have more complex AND conditions.
Note, the UI doesn't currently display timelines for containers, contain_groups, container_nodes, container_projects, and container_replicators but gets us closer to removing the event_where_clause.
Note 2, the tests describe how some of the current event_where_clause	values use OR conditions.  Currently, this isn't implemented as we'll have to figure out how to combine AND and ORs in the API and expose a generic interface for the UI to build API requests to match these ANDs and ORs.
Note 3, this is a followup to the bullet list of items in https://github.com/ManageIQ/manageiq/pull/22361

@jrafanie Can you look at the rubocops?  They all (or almost all) seem legit.

The UI doesn't currently display timelines for containers, contain_groups, container_nodes, container_projects, and container_replicators

It probably should, so perhaps this enables adding that with the new timelines.

I think this is ready to go.  Adding p/yes? so we can backport this later and not lose it.  I'd like to get this so the UI can freely add the missing timelines for containers and friends if desired and at the same time get closer to removing the event_where_clause.  We can choose to backport this much later if deesired.

This looks like a good change. Holding off on backporting sounds good, too.
Anything outstanding here?

@Fryguy I think this is ready to go for master.  Do you have any further concerns?

Updated to drop petrosian in light of breakage in the UI tests: https://github.com/ManageIQ/manageiq-ui-classic/pull/8810
We can enforce these rules around proper setup for each model's associations on master branch.


ESC signal interference

with the frequency that arduino Mega is operating by default there will be no dead-zones on joysticks so the motor will be operating at all times and does not follow the transmitted signals!

The Servo library is dealing with frequency and timing pretty well, and all RC components are operating the same as Servos. Thus, I have created a servo object "ESC" to over come this problem.


[treehopper] Fix dependence libusb

Fix https://github.com/microsoft/vcpkg/pull/35435#discussion_r1413497744
Note: Fix correct libusb usage.
Error:
/treehopper/src/1.11.3-f321b6be5f.clean/C++/API/inc/LibUsbConnection.h:9:10: fatal error: 'libusb-1.0/libusb.h' file not found
#include <libusb-1.0/libusb.h>
         ^~~~~~~~~~~~~~~~~~~~~
1 warning and 1 error generated.

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Use libusb pkgconfig, not vcpkg's cmake package hack.

The PR https://github.com/microsoft/vcpkg/pull/35498 dose not fix the usage.

Use libusb pkgconfig, not vcpkg's cmake package hack.

I'm pretty sure in general that we currently prefer the unofficial cmake package because we don't have a great answer for what happens for pkgconfig on Windows, as downstream consumers of us don't usually have a copy of pkgconfig that their CMake is going to find.
That also said, that doesn't apply in this case because the libusb backend is never selected on Windows.

because we don't have a great answer for what happens for pkgconfig on Windows, as downstream consumers of us don't usually have a copy of pkgconfig that their CMake is going to find.

Just depend on port pkgconf on windows and cmake will find that without problems.

Just depend on port pkgconf on windows and cmake will find that without problems.

Not in downstream's CMake

Not in downstream's CMake

It does. Did you forgot about:
https://github.com/microsoft/vcpkg/blob/c6f9a7b205b7c6847513a60a9759e1e600e0646d/scripts/buildsystems/vcpkg.cmake#L557-L590
Since CMAKE_PROGRAM_PATH is modified find_program will actually find pkgconf if it is installed by vcpkg

/azp run

LLVM failure was a random ICE we aren't normally seeing in CI, so let's hope simply rerunning fixes it

Just depend on port pkgconf on windows and cmake will find that without problems.

"Without problems" is not true.
Our cmake maintainer functions do not set VCPKG_HOST_TRIPLET. So the host pkgconf is not found without extra options.
Related discussion is (at least) in https://github.com/microsoft/vcpkg/pull/25529.


Simple example no longer runs on scala 3.5.1

import fastparse._
object TestFastParse {
def parseA[$: P] = P("a")
val Parsed.Success(value, successIndex) = parse("a", parseA(_))
assert(value == (), successIndex == 1)
}

.
/home/tim/repos/untitled/src/main/scala/TestFastParse.scala:5:56
Context bounds will map to context parameters.
A using clause is needed to pass explicit arguments to them.
This code can be rewritten automatically under -rewrite -source 3.4-migration.
val Parsed.Success(value, successIndex) = parse("a", parseA(_))

This can be manually worked around by replacing parse("a", parseA(_)) with parse("a", c => parseA(using c)) since a using keyword now appears to be required.

also applies to 3.6. Tho the error message is now No given instance of type fastparse.ParsingRun[$]
The change I applied is:
parse("a", parseA(using _))

which is easier than creating the lambda explicitly. Also: TIL about using _.


[Bugfix] query hive external text table , column case will  lead to reading failed

Steps to reproduce the behavior (Required)
Step1:     in hive
create table test(a int,  b string) stored as csv;
insert into test values(1, 'aa');
Step2:    query in sr
create external table test_sr (A int, B string) properties ()
select * from test_sr;
Expected behavior (Required)
record can be found out.
Real behavior (Required)
throw fail message "Can not get index of column name"
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

So, is there any problem, that you closed this issue?


Add print to file with filename

Codecov Report

Merging #18 (05dee2e) into master (8fa5e18) will decrease coverage by 1.77%.
The diff coverage is 75.00%.

@@             Coverage Diff             @@
##            master      #18      +/-   ##
===========================================
- Coverage   100.00%   98.22%   -1.78%
===========================================
  Files            6        6
  Lines          333      338       +5
===========================================
- Hits           333      332       -1
- Misses           0        6       +6

Impacted Files
Coverage Δ

src/watchpoints/watch_print.py
96.92% <75.00%> (-3.08%)
:arrow_down:

src/watchpoints/util.py
91.11% <0.00%> (-8.89%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix ppf.h

Without the parentheses lfortran compiler exits with error

Hi @jschueller , I have never tried LFortran, although I have always been interested. Is it capable of compiling PRIMA yet? Isn't the error you observed a bug of LFortran? Thank you.

lfortran seems its far from being ready, a workaround like this wont be enough


Fix const correctness for Vector2, Vector3, and Vector4

… Vector4 classes.
This solves the issues of constness discussed in Issue #246, as well as adding const to a few other const member functions in the three Vector classes.

Thanks so much for fixing these up :+1:


Add support for Local Traffic  ››  Profiles : Authentication : OCSP Responders

Is your feature request related to a problem? Please describe.
We want to be able to  use OCSP Authentication Profiles in the LTM module. Today AS3 not only lacks the ability to create a OCSP Responder profile but also the ability to use an AS3 Pointer to a profile under the /Common/ Partition to set the profile.
Describe the solution you'd like
Possibility to add an OCSP Responder Profile.
Describe alternatives you've considered
Only alternative is to use a Virtual Server manually configured under /Common/.

Thanks for reaching out, have created an internal backlog item AUTOTOOL-4265. In order to prioritize this issue, please reach out to us at<EMAIL_ADDRESS>


feat: _ image update_

This PR  contains image creation of image in docker hub

Hi @adkumar1,
I think dataex is even worse than dataexchange. If there is no better description than a general "data-exchange", then i would suggest the long version data-exchange-test-service to indicate, that it is for testing.
Also, please update the workflows like trivy to use the images from DockerHub instead of pointing to GitHub packages is a fork


spawn php-cs-fixer ENOENT

Error: spawn php-cs-fixer ENOENT
at exports._errnoException (util.js:734:11)
at Process.ChildProcess._handle.onexit (child_process.js:1035:32)
at child_process.js:1127:20
at process._tickCallback (node.js:357:13)
When trying to run the atom beautifier

Are you sure this is a problem with this plugin? Atom beautify is also using the php-cs-fixer https://atom.io/packages/atom-beautify


Have CompletionEntryDetails source use a relative path

Fixes #19895

We will need to port this to release-2.6


Session.Send refactor

To be merged after #694.

Codecov Report

Merging #695 into master will decrease coverage by 0.13%.
The diff coverage is 79.64%.

@@            Coverage Diff             @@
##           master     #695      +/-   ##
==========================================
- Coverage    91.4%   91.27%   -0.13%
==========================================
  Files          81       81
  Lines        5581     5693     +112
==========================================
+ Hits         5101     5196      +95
- Misses        302      314      +12
- Partials      178      183       +5

Impacted Files
Coverage Δ

session.go
87.33% <67.5%> (-1.23%)
:arrow_down:

packet_packer.go
85.34% <83.46%> (-3.55%)
:arrow_down:

crypto/cert_manager.go
96.61% <0%> (+0.31%)
:arrow_up:

client.go
92.89% <0%> (+3.15%)
:arrow_up:

Continue to review full report at Codecov.

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


Add OAuth2

I have tried to implement the minimal requirements of Home Assistant without breaking too much of the existing API. It is also possible to use OAuth in any application other than Home Assistant, even from the command line. See sample.py for an example.
https://developers.home-assistant.io/docs/api_lib_auth/#oauth2
Closes: #6

@stianaske this branch is now up to date with the current master. You can go for a first review if you want.
There are still some minor todos, I have to verify the Vorwerk defaults and I have to test this with Home Assistant. But I don't expect any major changes.


include matcher is not diffable enough

Subject of the issue
Consider:
RSpec.describe 'foo' do
  RSpec::Matchers.define :my_include do |expected|
    match do |actual|
      @expected_as_array = [[expected]]
      expect(actual).to include(expected)
    end

    diffable
  end

  let(:expected) { {a: 'a' * 100, b: 2} }
  let(:actual) { [{a: 'a' * 100  , b: 3}] }

  it { expect(actual).to include(expected) }
  it { expect(actual).to my_include(expected) }
end

diff for my_includes shows exact lines where the issue occurred, because include matcher tries to compare single object with the whole array.
It's especially useful when working with real/more complicated data structures, for me it's unfortunately spend 3 minutes reading output or try to diff by hand usually on similar failures.
Your environment

Ruby version: 3.0.2
rspec-expectations version: current mastermain

A PR is welcome. You can start with a pending spec.
If you're up to improve this, check https://github.com/rspec/rspec-support/issues/365 (and listed issues) and https://github.com/rspec/rspec-support/pull/516 for details.

I don't quite understand the expected solution here, while the single expected hash is quite easy, for multiple hashes we probably should find the closest objects in the actual enumerable and show diffs for them, but I guess it will lead to https://github.com/rspec/rspec-expectations/issues/1161.
And even if we knew what should be done, not sure if I'd have the capacity to do it.
So only see something say something issue for now and workaround in a private matcher :sweat_smile:

Our current differ is limited in that it has to diff everything as strings, so it "prints" objects to strings then diffs those strings. As the include matcher is diffing an array vs a single element, it's always going to be "wrong" when it comes to the diff, and we don't currently have an api to provide the differ with a more intelligent solution.
Mutating the expected is a work around, but it might cause problems for re-use (which we support) and other downstream complications, for now I'm going to close this as a "can't fix" as its essentially a duplicate of out "we'd like to improve the differ" issues.

Theres a couple over on rspec/rspec-support, contributions for improving the differ or making it configurable with say super_diff would be welcome


stubborn type after case narrowing from within class with covariant TP

This compiles on 1.2.2 but not master:
interface I<out T> of A<T> | B<T> {}

class B<out T>() satisfies I<T> {}

class A<out T>() satisfies I<T> {
    void foo() {
        I<T> x0 = this;

        switch (x1 = x0 of A<T> | B<T>)
        case (is A<Anything>) {
            A<T> i = x1;         // ok (no longer needs the 'of' above in 1.2.3 ???)
            I<T> j = x1 of A<T>; // ok
            I<T> k = x1;         // error: A<Anything> is not assignable to I<T>
        }
        else {}
    }
}

This may answer my of question:
class C<out T>(C<Anything> ca) {
    C<T> ct = ca of C<T>; // no error on 1.2.3 !!!
}

Oops, must be a side effect of my very recent changes in #6294.

Ok, thanks, reverting those commits works.

Fixed by @jvasileff. Thanks!!


All Tests pass on Linux

Closes #15
Test still hangs at the end (after all tests) before showing results, 4min later

Switched target branch to 0.3.0 where the CI jobs are defined, then closed & reopened to trigger CI.

Awesome stuff! 👏  Looks like it's only failing on a fmt error. Might be fixable by running deno fmt locally.
Also the CI job is still configured mac only:
https://github.com/pskfyi/handy/blob/f530ea80af1e4f6fe52500632a249d59e6c9e3b6/.github/workflows/pr-checks.yml#L40
I think you want to change this to if: (matrix.os == 'macos-latest') || (matrix.os == 'ubuntu-latest').
You may need to rebase on top of 0.3.0 to get the most up-to-date state of the rest of the repo, including the GitHub Actions workflow file referenced above. Here are the rebase steps for reference:
git checkout 0.3.0
git pull # updates 0.3.0 to latest from GitHub
git checkout glob-tests-pass-linux
git rebase 0.3.0 # replay your commits on top of latest 0.3.0

Weird. When I run deno fmt locally everything passes...

I don't see the ./.github folder after pulling the latest 0.3.0. We need to merge this first right?

Added a step to install xclip.  It now fails on copy() not sure why. Locally everything passes (even though it takes 1-7 minutes....)


Show semi-opaque white background for dark mode

This is in response to issue #10 with Twitter and YouTube dark modes causing non-legible text on hover and on text show.
As a suggestion, I used a semi-opaque white for background. In this way, white mode and dark mode show a highlight. I'm open to other UI style fixes.
before:

legibility issue

after:

@googlebot I signed it!

Hi Jack,
Thanks for your contribution. I started looking at this a bit more and I think that we may not actually need this background-color set on hover at all for comments that have not been hidden by tune, to white or a semi-opaque white at all, since the default behavior on YouTube doesn't have any change in background color on hover and we use an outline for Tuned comments on hover. I can't recall if the default behavior for this was different at the time we initially created Tune, or if this is a remnant of an earlier design where we changed the background color of Tuned comments on YouTube to a light gray like we do for Twitter and Reddit.
In a quick test, I tried removing the background color setting without replacing it with anything, and this appeared to work with both light and dark mode on YouTube, and sort of for reddit (though there are some aspects to dark mode that still don't work nicely). Would you mind checking if this works for dark mode on the other sites? For the non-YouTube sites we have the "hover" UI for comments filtered by Tune to be a change in background color to a light gray, so that could be where the default white background setting came from.

I started looking at this a bit more and I think that we may not actually need this background-color set on hover at all for comments that have not been hidden by tune, to white or a semi-opaque white at all, since the default behavior on YouTube doesn't have any change in background color on hover and we use an outline for Tuned comments on hover.

Totally agree. I like your solution and thinking on this. Haven't seen any issues with Disqus.

Updated the branch deleting the hover and background style. For clarity, I know the hover state seemed suspect -- and good to delete, based on your comment.
But did you also want me to delete the background color one for placeholder of tune comment. Sounded like there was some continuing use for that?

Changes look good! Thanks.

But did you also want me to delete the background color one for placeholder of tune comment. Sounded like there was some continuing use for that?

Could you clarify which selector you're referring to? Is it the :host[tune-state="show"] > .--tune-placeholder.--tune-commentVisible one? If so, then, yes, I believe that one is fine to delete as well, since it means "a comment that has been revealed by the user when it was in a hidden state, and also not filtered by tune anymore," which I think is a rare state that can only be reached with that exact sequence of actions (if it can be reached at all...I haven't tested this recently!) Either way, I think it should be a no-op on the non-tuned state of the UI. If you want to be extra cautious, you can test it out with reddit and twitter, which are the only other places we change the background color for this element.

I fixed the ci error in #14

Is it the :host[tune-state="show"] > .--tune-placeholder.--tune-commentVisible one? If so, then, yes,

Yep -- great.

Merged the latest from my other #14. Thanks for the quick approval @rachelrosengoogle!


Update team's page

The Team's page needs to be updated: https://opensciencelabs.org/team/
Board
The current board is Ever, Eunice and Ivan
Collaborators
Add Petrizzo, Francisco, Ingrid, Sandro, Yurely, Anavelyz and Jasmeet

the file to be changed is: https://github.com/OpenScienceLabs/opensciencelabs.github.io/blob/main/pages/team.html

@YurelyCamacho could take a look into that pls?
cc @EverVino

Hola, ya lo hago.
El lun, 25 jul 2022 a las 17:10, Ivan Ogasawara @.***>)
escribió:

@YurelyCamacho https://github.com/YurelyCamacho could take a look into
that pls?
cc @EverVino https://github.com/EverVino
—
Reply to this email directly, view it on GitHub
https://github.com/OpenScienceLabs/opensciencelabs.github.io/issues/40#issuecomment-1194633645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALWDJU6TXSSKDUEWORV7B7DVV37DTANCNFSM54LNJOSQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

thanks for working on that @YurelyCamacho !
PS: next time in the first comment inside your PR please mention which issue you are resolving and this issue will be closed automatically: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue
here in this table (from the link above), you can see an example about how to do that:

Linked issue
Syntax
Example

Issue in the same repository
KEYWORD #ISSUE-NUMBER
Closes #10

Issue in a different repository
KEYWORD OWNER/REPOSITORY#ISSUE-NUMBER
Fixes octo-org/octo-repo#100

Multiple issues
Use full syntax for each issue
Resolves #10, resolves #123, resolves octo-org/octo-repo#100


Can no longer set custom SDK timeout after v20.1.1

Our application has a 15s timeout for web requests, so we previously lowered the read timeout for the Avalara SDK to 8s. Most requests for tax estimation took well under 8s already, and it's preferable for us to handle the timeout, trigger offline tax calculation, and do other request cleanup ourselves versus hitting the global application timeout hander where we have much less control.
Setting custom timeouts for an SDK is a common need and was possible using the connection_options config setting like so:
AvaTax.configure do |config|
  # ...
  config.connection_options = {
    request: {
      open_timeout: 3.seconds,
      timeout: 8.seconds
    }
  }
end

https://github.com/avadev/AvaTax-REST-V2-Ruby-SDK/blob/eb7c20b8e925a3d682f6414207e298e519e0a549/lib/avatax/configuration.rb#L16
These options ultimately get used to instantiate the Faraday request object in the AvaTax::Connection and control the request timeout. (Documentation: https://github.com/ankane/the-ultimate-guide-to-ruby-timeouts#faraday)
However, in https://github.com/avadev/AvaTax-REST-V2-Ruby-SDK/pull/72, any custom timeout options are overridden in the request method, to what looks like 1200s. This means that clients can no longer set custom timeouts on that request, though it would seem like the connection_options setting was added to enable that type of use case.
Would it instead be possible to set the timeout in the default connection options and have the same effect?
https://github.com/avadev/AvaTax-REST-V2-Ruby-SDK/blob/eb7c20b8e925a3d682f6414207e298e519e0a549/lib/avatax/configuration.rb#L32

Comment : closing issue as the fix is merged


Update README.md

Please either add the required packages to the README.md or create a requirements.txt file for this purpose.
Check that the README.md description is up to date.

Note that I removed the quantcomps.py file since it did not seem to be referenced in the notebooks and was not fully documeted, which means that the README is not correct anymore I believe.


work around webkit bug that coalesces multiple spaces

https://bugs.webkit.org/show_bug.cgi?id=112032
https://github.com/mozilla/pdf.js/issues/8935
I ran lint and the tests locally, but 1/465 unit tests is failing whether I use "master" or my feature branch.

Is this solution complete? Are we sure that safari only removes double spaces and not touching leading spaces?

Thanks yury, I think your concern was spot on. I updated the logic.
Snuffleupagus, I completely understand that reasoning. I'm content to close the PR and linked issue, and people will be able to follow the issue to see this specific workaround, since this should be fixed in webkit. However, you mentioned "(at least not in its current form)"--is there a reasonable way to rework this PR to make it optional for people who need to work around the webkit bug?

I'm going to close this pull request, but we can keep the issue open for tracking purposes. Indeed we do not wish to have browser-specific hacks in the core code. Custom deployments may add this, but I don't think we should add it in PDF.js repository.


Migrate to HF datasets for hidden state storage

Storing hidden states in a HuggingFace dataset is a cleaner and more scalable solution which makes it easier to store metadata along with the hidden states, and load subsets of the hidden states into memory

Added branch multiprocessing. It's pretty rough right now -- three biggest things missing are 1) specify Features in the dataset (right now it just pushes hiddens and labels), 2) fixing an annoying multiprocessing bug, and 3) making sure the returned Dataset object by extract_hiddens still works with everything else.
To elaborate a bit on (2), datasets uses os.fork to create new processes, but most people recommend you use os.spawn instead as otherwise you can get the error when you try to move your model onto GPU: RuntimeError: Cannot re-initialize CUDA in forked subprocess. To use CUDA with multiprocessing, you must use the ‘spawn’ start method.
One option would be to force Datasets to use os.spawn when creating new processes, but that might involve messing the internals of their implementation (I think) which would not be ideal. Especially because you can actually use CUDA with os.fork -- my barebones implementation of saving hidden states works fine with multiple processes + os.fork + CUDA. It's when I integrate it into the elk that it breaks. My guess right now is that somewhere before extract_hiddens is called, some part of the model is secretly being moved onto GPU, and CUDA ends up freaking out if you try to reinitialize it on a model that was already on CUDA in the main process on a forked process.


Can I customize datatable layout

Can I customize datatable layout?
I want to move pagination controller to top instead of bottom.
If I can use DOM positioning, it will be the best.
https://datatables.net/examples/basic_init/dom.html
Your Environment

Tech
Version

Material-UI
3.5.1

Material-datatable
0.2.1

React
16.6.3

browser
Safari/Chrome/Firefox/IE/Edge

etc

At current moment no.
Any Ideas how it should be implemented?
To be honest, I do not see to much ways how to do it.


Statically dispatched methods for trait objects with associated data

This RFC describes a way to efficiently implement statically dispatched methods for trait objects.
It allows statically dispatched methods to access implementer's fields directly.
Unlike others approaches it doesn't require implementers to have a specific memory layout.
It provides an alternative solution to one part of "efficient inheritance problem": https://internals.rust-lang.org/t/summary-of-efficient-inheritance-rfcs/494
Rendered view

This seems related to https://github.com/rust-lang/rfcs/pull/1546, right?

Yes, it is related.
This particular rfc describes a way to implement zero-cost field access from trait methods.
On second thought this method might be somewhat over-complicated. I no longer think it's a good idea to implement it as. It still might be useful as an implementation approach, but semantic of associated fields/partial borrowing/inheritance should be finalized first.

 Let's focus on the fields-in-traits story first.


When is Beta 3 expected to be released?

It has been quite sometime since beta 2 is released. When is Beta 3 expected to be released? Any road-map or milestone for Material 2?

https://github.com/angular/material2/issues/3800


Saving things referencing themself is broken.

Saving things referencing themself:
Minimum not-working examples:
(a) setting an attribute
@transform
class T:
    def __init__(self):
        self.a = 1
        self.b = self.a  # problem

    def __call__(self):
        ...

(b) setting at an index
@transform
def f():
    x = [1, 2, 3]
    x[0] = x[1]

Both is due to a missing check in dumptools.var2mod._VarFinder.visit_Attribute (it treats this as an assignment to self (a) or x (b)).

#249


Add post: The relationship between recruitment cycles and academic years

The post can be found here:
https://deploy-preview-1259--bat-design-history.netlify.app/register-trainee-teachers/the-relationship-between-recruitment-cycles-and-academic-years/

 Design history for all things to do with recruitment cycle and academic year


Add MaterialSearch control

Documentation:
https://m3.material.io/components/search/overview
We can use the MaterialEntry as a sample. The MaterialSerach should be really similar to the MaterialEntry.
Use the search button on the keyboard.
We have to send through the parameter the current Text value when we have to invoke from the MaterialSearch the following commands: LeadingIconCommand, TrailingIconCommand and SearchCommand.
Desirable properties (without mention the inherited from BaseMaterialFieldControl):

Text
SearchCommand
SearchOnEveryTextChange (bool, if it is true we have to invoke the SearchCommand on every text change, if not we have to invoke it when we use the search keyboard button).

We are excited to announce that we are working on a new generation of controls based on Material Design V3, which we will be releasing soon.
This ticket has been included/fixed in the upcoming version of this NuGet package, which will include both current and new generations of controls.


Fix usingTLS in idol-library

Bump idolVersion to 24.4 for pending release
Patch release all charts
Add basic python tests using helm template

I can/will add the other charts to the test (at least in basic form) but wanted an opinion on whether this looks plausible/useful or not before spending more time on it.

Think the test setup looks good. Could add some more checks like testing that setting ingress.tls.secretName adds the tls section to the ingress template, but this definitely looks useful.
Wonder if we could add some more error handling to things like render_chart. If a chart fails to render would be good if the tests could give as much info as possible so they can be more useful for debugging.

Think the test setup looks good. Could add some more checks like testing that setting ingress.tls.secretName adds the tls section to the ingress template, but this definitely looks useful.

Didn't want to go back and try and cover everything we've previously added right now.

Wonder if we could add some more error handling to things like render_chart. If a chart fails to render would be good if the tests could give as much info as possible so they can be more useful for debugging.

38b00a3 adds options to enabled --validate and --debug which gives a little more output depending on failure mode, though ultimately once something fails I expect devs to start running helm template manually to figure out what's gone wrong.


clipboard ring

I like Emacs's style (rotate ring is beautiful)
But I also like vim's registers
Also - I like spacemacs/doom's hints
And of course I like lite's command things (hey, lite is fast! )
so.. hmmmmm

well. Sort of works? through registers, recording_register, and target_register
TODO:

list all registers for selection
(including the ring!)
(probably through a pupup buffer? )
(if not - through generalized ResultsView)
(with action on click = set_clipboard and then close the view)

Yep. done aand done


tree.branch is not a function error on ng new with custom schematics

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Angular CLI: 1.5.0
Node: 8.9.1
NPM: 5.5.1
OS: linux x64

Repro steps.
I created a package whiches copy exactly orignal angular schematics and try to initiate a new application using collection of my custom package using ng new -c my-custom-schematics myapp
here is a docker command to reproduce the problem in a clean environment : docker run --rm node:8 sh -c 'npm i --unsafe-perm -g @angular/cli && git clone https://github.com/Neonox31/custom-schematics-example.git && cd custom-schematics-example/dist && npm pack && npm i -g my-custom-schematics && ng new -c my-custom-schematics myapp'
The code of my custom schematics package is located here : https://github.com/Neonox31/custom-schematics-example
The log given by the failure.
Error: tree.branch is not a function
tree.branch is not a function

Desired functionality.
Application should be scaffolded using custom schematics and not print error messages.
/cc @davinkevin

This should have been fixed with the latest Schematics, but I'll file an error on the DevKit repo to make sure we look at it.

Actually already reported, so dupe of https://github.com/angular/devkit/issues/271


Remove optional dependency library specification

I cannot compile properly without these libraries.
If I didn't install libgtest:
CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
  Could NOT find GTest (missing: GTEST_LIBRARY GTEST_INCLUDE_DIR
  GTEST_MAIN_LIBRARY)

If I didn't install libglog:
CMake Error at cmake/FindGlog.cmake:22 (message):
  Could not find glog library.

these dependencies are declared optional because functions relying on these libraries can be turned off by specifying additional CMake options:
cmake ... -DBUILD_TEST=OFF -DENABLE_LOGGING=OFF
https://github.com/rime/librime/blob/99e269c8eb251deddbad9b0d2c4d965b228f8006/CMakeLists.txt#L19
https://github.com/rime/librime/blob/99e269c8eb251deddbad9b0d2c4d965b228f8006/CMakeLists.txt#L21


confirmation message is not provided

in inventory adjustment ,while deleting comment confirmation message is not provided

issue fixed


Javascript maps

[yes ] I have checked that this is not a duplicate problem
[yes ] My Code is in running state
[yes ] I have read the README.md file
[ yes] I have starred the repo (This is optional though :))

Description of Change

[ ] My change is a new addtion.

@vyasriday can u please review above PR's


[6.0🍒] NCGenerics: add availability checking

Explanation: Enforce availability checking for obviously incorrect dynamic casts of types with noncopyable generic arguments. Such casts to a type any P can incorrectly succeed when deployed to older Swift runtimes when the conformance to P was actually conditional on the generic argument being Copyable. This PR doesn't cover all possible situations.
Scope: Source breaking when the deployment target is set to older than the 6.0-runtime Apple OS's.
Issue: rdar://126239335
Original PR: https://github.com/swiftlang/swift/pull/74918
Risk: Low, source break for newly provided feature in 6.0.
Testing: Swift CI
Reviewer: @slavapestov

@swift-ci test

Adding the related NFC that adds a missing return to these changes from this PR: https://github.com/swiftlang/swift/pull/75085
That one was reviewed by @tshortli .

@swift-ci test


Add unit tests

I'll add a pytest based unit testing framework and write tests for the main components.

This is closed as we have CI now with basic tests


Treat deleted and expand parameters as boolean when listing project packages

This means treating those parameters as boolean when calling GET /source//
Closes #106

Thank you for the submission :slightly_smiling_face:


Fix AQE test failures on Databricks 14.3

Build the plugin against Databricks 14.3 cluster using #11467. Once built successfully run the aqe tests by TESTS=aqe_test.py jenkins/databricks/test.sh
The following tests fail
[gw0] [ 68%] FAILED ../../src/main/python/aqe_test.py::test_aqe_join_with_dpp
[gw0] [ 68%] FAILED ../../src/main/python/aqe_test.py::test_aqe_join_with_dpp_multi_columns

Taking a look at this one now.

It appears that this is the result of a new, unsupported exec being used to handle BHJs.
It will be beyond the scope of this issue to add support.  I'm investigating for details on the exec, before I file the issue and disable these tests.


fix(client): Randomly slow youtube download speed

Description
This PR fixes the randomly slow youtube download speed that caused from web client.
Issues to fix
Please link issues this PR will fix:
#[230]
Reminding
Something you can do before PR to reduce time to merge

run "make build" to build the code
run "make format" to reformat the code
run "make lint" if you are using unix system
run "make test-integration" to pass all tests

Thanks a lot for you contribution! Unfortunately one test fails because the changed user agent leads to a missing creation timestamp. I could not find a solution yet.

Thanks a lot for you contribution! Unfortunately one test fails because the changed user agent leads to a missing creation timestamp. I could not find a solution yet.

Thanks for your help! I'll have a closer look this evening

I ran into the issue with slow downloads and tried this patch out and noticed some issues.
This PR introduces a new type for adaptive formats: AdaptiveFormat. The newly separated adaptive formats are thus not included in the Video type so we completely lose access to them. Additionally, even if they were exposed, Client in its current state only works with the Format type. Would it be possible switch back to a single Format type?

I ran into the issue with slow downloads and tried this patch out and noticed some issues. This PR introduces a new type for adaptive formats: AdaptiveFormat. The newly separated adaptive formats are thus not included in the Video type so we completely lose access to them. Additionally, even if they were exposed, Client in its current state only works with the Format type. Would it be possible switch back to a single Format type or create a format interface?

https://github.com/cybre/youtube/commit/4a0c7d672485d96144fa26aebbfc602b3ad2bb2f
I think your solution in this commit is the best for now!

@corny @kkdai, should I fix the conflict and we merge this change?

Please rebase on top of master and then let's check the integration tests.

I tried to rebase your branch on top of master. Unfortuntely test TestYoutube_GetItagInfo shows that the number of itags drops from 26 to 6 for one video and there are more issues.

FYI I am the author of the original "clientVersion": "16.20" code:
https://github.com/89z/mech/commit/f1b0efc090fc2e72e5950380a4ff54f6d92a22ec
I would recommend changing that, as at the time I didnt know how the version was derived, so I created it from trial and error. Ive since learned that its just the version number from the Android app:
PS C:\> googleplay -a com.google.android.youtube
POST https://android.googleapis.com/auth
GET https://android.clients.google.com/fdfe/details?doc=com.google.android.youtube
Title: YouTube
Creator: Google LLC
UploadDate: Mar 16, 2022
VersionString: 17.11.34

should be solved by #244


Only Card on first turn

Is possible on 1 wave use only attack cards?

In the command editor UI, just select attack, next turn/wave then set your 2nd turn/wave skills

Isnt working

When i start nothing happens

Without a video recording using the FGA recording function, we can't tell what's wrong.


Support for Scoped Storage in Android

Version of react-native-audio-recorder-player
Platforms you faced the error (IOS or Android or both?)
Android version >= 10
Expected behavior
To use the new scoped storage solution
Actual behavior
Recorder fails to start since no access to /sdcard is available.
Steps to reprodue the behabior
Play the recorder in a Android >= 10 version device.
Hints and other resources
In order to have compatibility with Android 10, a fix can be applied modifying the application tag in the manifest:
`<manifest ... >

<application android:requestLegacyExternalStorage="true" ... >
...

`
But this is only a workaround, since it will be deprecated in future versions.
Also, related implementation:
https://stackoverflow.com/questions/59327033/android-q-how-to-save-audio-recording-with-mediastore-with-the-correct-file-nam

Nice issue! Could you kindly add this to our readme? It would be nice if you can give us a PR.

@marcosPerezRubio you saved my life!
But can you record an audio? With Android >= 10 and you fix I can record an audio file but it's mute, there is no audio.
The file is heavy, is not 0 KB but it's mute.
I have also the permission to record audio so I can't get it...
@hyochan did you investigate a little in this issue?
Thanks guys!

Ok found the problem...
In the simulator you have to activate the option "Microphone > Virtual microphone uses host audio input".
Never used it on other simulators, but maybe with Android >= 10 is required.

Actually, after testing on a few devices, I found the best solution that works for me and hopefully, this will help other fellow developers too.
import RNFetchBlob from 'rn-fetch-blob';

let dirs = RNFetchBlob.fs.dirs;
let path = Platform.select({
    ios: 'hello.m4a',
    android: `${dirs.CacheDir}/practice.mp3`,
  });

This path will never fails 🙂  because every app will have its own cache directory. The reason I came up with this instead of sdcard is that in one of the Samsung phones, it doesn't have sdcard and I think the directory is not available to set. Please try this method and hope this helps!
If this is working for @mrigo @marcosPerezRubio @hyochan , I will help to write this down in readme.

Thanks for the update, I will try it for sure.
I've in mind to document better the legacy function.
Hope we can sort it out!
El dom., 1 nov. 2020 15:38, Aung Myint Thein<EMAIL_ADDRESS>escribió:

Actually, after testing on a few devices, I found the best solution that
works for me and hopefully, this will help other fellow developers too.
import RNFetchBlob from 'rn-fetch-blob';
let dirs = RNFetchBlob.fs.dirs;
let path = Platform.select({
ios: 'hello.m4a',

android: `${dirs.CacheDir}/practice.mp3`,

});
This path will never fails 🙂 because every app will have its own cache
directory. The reason I came up with this instead of sdcard is that in
one of the Samsung phones, it doesn't have sdcard and I think the directory
is not available to set. Please try this method and hope this helps!
If this is working for @mrigo https://github.com/mrigo @marcosPerezRubio
https://github.com/marcosPerezRubio @hyochan
https://github.com/hyochan , I will help to write this down in readme.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dooboolab/react-native-audio-recorder-player/issues/225#issuecomment-720098527,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSPXYN6SP4IZ5HCJOEDVGTSNVXFRANCNFSM4QSBRXXQ
.

Actually, after testing on a few devices, I found the best solution that works for me and hopefully, this will help other fellow developers too.
import RNFetchBlob from 'rn-fetch-blob';

let dirs = RNFetchBlob.fs.dirs;
let path = Platform.select({
    ios: 'hello.m4a',
    android: `${dirs.CacheDir}/hello.mp3`,
  });

This path will never fails 🙂 because every app will have its own cache directory. The reason I came up with this instead of sdcard is that in one of the Samsung phones, it doesn't have sdcard and I think the directory is not available to set. Please try this method and hope this helps!
If this is working for @mrigo @marcosPerezRubio @hyochan , I will help to write this down in readme.

I've recommend this approach from 2.7.0 and updated the readme. Thanks 🙌

import RNFetchBlob from 'rn-fetch-blob';
let dirs = RNFetchBlob.fs.dirs;
let path = Platform.select({
ios: 'hello.m4a',
android: ${dirs.CacheDir}/hello.mp3,
});
I don't see the file in my device @Aung-Myint-Thein, and for such I guess I'm facing an error on playing the audio saying : open failed: ENOENT (No such file or directory)


Exporters-intro rework and mention benefits of OTLP exporters

Proposes a fix #3295
Closes #3902 - sorry @pegasas it was easier to start from a clean slate than comment on your PR.
If @open-telemetry/docs-approvers agree with this approach, I'll double check if I need to clean up other language Exporter pages (I've only tackled two so far in this PR).

Preview:

https://deploy-preview-3905--opentelemetry.netlify.app/docs/languages/java/exporters/
https://deploy-preview-3905--opentelemetry.netlify.app/docs/languages/go/exporters/

Screenshot (sample)

/cc @cijothomas @pegasas @theletterf @svrnm

@chalin , apologies merging the PR to update the short codes on the exporter pages created that conflict.

@svrnm - no worries. I've rebased, resolved conflicts, and slightly reworked the initial part of the exporters intro. PTAL


reduce network exponential backoff maximum cap to 10s

Issue Number

N/A
Overview

[x] I have reduced the maximum delay in the exponential backoff retry logic in the network follower and logic for waiting for the nodes.

Comments

Right now, when Daedalus is starting, it'll wait for our API to be ready and show a splash screen until it gets ready. We start our API once we know Jörmungandr is up and we check that by making http requests to Jörmugandr. So far, we've been using an exponential backoff, uncapped. Jörmungandr typically takes 30 to 40 seconds to start, which means that, initially we would be trying after 500ms, 1s, 2s, 4s, 8s, 16s, 32s, 64s etc ...
As a consequence, if Jörmungandr starts after let's say 40s, we only get to retry after more than 20s. If it takes a minute, we'll not retry before another minute! Etc ..
So instead, I've switched to using a constant delay, to make the whole thing more reactive. We poll every second and it's a local node, so there's no particular issue with that.
Also, in the chain follower, I've reduced the maximum retry delay to be 10s for roughly the same reason. It was 60s before which gives us a quite poor reactivity.

bors try

Whoops... https://buildkite.com/input-output-hk/cardano-wallet/builds/4023#b69e8995-1fd5-448a-8347-e0226d19d234
I forgot about this newly introduced unit test.

running integration tests locally


Unable to use induction

While on Addition: zero_add, I am unable to use the following rule to proceed: induction n with d hd.
Error:
The tactic 'induction' is not available in this game!

Has been fixed today :+1:


chore: update engines

As long as Mr. Renovate can't handle this

Rerunning all jobs, GH actions seemed to not be able to start a db


wifi simple_link driver build error

Describe the bug
build error
To Reproduce
west build -b cc3235sf_launchxl samples/net/sockets/http_get
Expected behavior
no error
Logs and console output
See for yourself while building the sample.

Duplicate of #27587


resolve a bug at camera module

@lohithbhargav there was a bug at your part of the code in the camera module.

Thanks for letting me know, i had resolved the bug and now ready for the testing.


[PR][CLOSED] [Fix] fix typo

 Issue by sewon-oh
Wednesday Jul 04, 2018 at 06:38 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/94

This commit is to fix typo.

checker-pr-audit-async.sh: Edit some typo in log.

Signed-off-by: sewon.oh<EMAIL_ADDRESS>
sewon-oh included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/94/commits

:octocat: cibot: Thank you for posting issue #118. The person in charge will reply soon.

 Comment by git-bot-sec
Wednesday Jul 04, 2018 at 06:38 GMT

Thank you for submitting PR #94. Your PR has to pass all verificiation processes of cibot before getting a review process from reviewers. We recommend that you read instruction manual in . If you want to monitor the progress status of your PR in detail, visit http://aaci.mooo.com/.


"Unable to write settings" message bar should have button to open User Settings

VSCode Version:1.8.0 Insiders
OS Version:Windows 10
Steps to Reproduce:

Add an extra trailing comma to user settings.json.
Close the settings file.
Try to toggle File > AutoSave.

You see the "Unable to write settings. Please open User Settings to correct..." message.
We should add a button to open User Settings next to Close.

https://github.com/Microsoft/vscode/issues/16580


Syncfusion and export laws

I propose any books with
Syncfusion is a United States company and is subject to US export laws. Consequently, due to your location, we may not allow access to any material on our site.
If you believe that this is inaccurate, please contact<EMAIL_ADDRESS>Thank you.

like
https://www.syncfusion.com/ebooks/azure-functions-succinctly
to be tagged appropriately as risk of them potentially be non free.
Not sure about proper tag name so.

Feedback from affected users on this issue would be appreciated.

Please correct me if I'm wrong, but I think this issue might no longer be relevent today. Syncfusion requires the user to sign up before they can access the book.
They ask for full name, email address, phone number, and that the user verifies their email address.
There is no way to download the book without a working email address.

We do not accept links to pages that require working email addresses to obtain books, but we welcome listings that request them.

Can this be closed?

Ah! I'm so sorry, disregard my previous comment.
I immediately went to try and download the resource as I only read books offline.
Syncfusion resources are available to be read online without an account it seems! Just not to download!


Addition for your link list

Nice write-up! I've done something similar a while ago: https://github.com/l1k/osxparanoia
Maybe you want to add some of my findings to your text or include that in your link list.

Whoa, that's a nice one! I'll definitely add it to the list of resources and you are welcome to PR any of your recommendations into this guide.


Modules create invalid header guard

As requested by issue eProsima/Fast-DDS#195, the generated C++ code honors the module name in the header guard.
However, the technique used to convert lowercase to uppercase must be locale-free.
For instance, if the module name is invalid, the generated header guard becomes İNVALİD in Turkish locale.
toUpperCase method in commit f058a03 should be replaced with std::toupper.
Another issue is that if there are nested modules, the header guard uses the innermost module name of the last struct.
It would be better if it uses the outermost module name.

I'm having the same issue. Any planned updates for this problem?

Hi, Fast DDS Gen in master branch uses java string toUpperCase() method in the guard generation.
We are not planning to change it in shot-middle terms.
Maybe running the Fast DDS Gen from a docker with the desired Locale suits your needs.


Prepare for next development (0.9.0-SNAPSHOT)

Bumping up the version to 0.9.0-SNAPSHOT to prepare for next
development iteration.

Hmm, seems the code style is a bit off, can you try to import with new checkstyle xml and reformat xml file?

Codecov Report

Merging #7356 (93ccda4) into master (7a44f4a) will decrease coverage by 2.12%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #7356      +/-   ##
============================================
- Coverage     71.60%   69.47%   -2.13%
+ Complexity     3298     3210      -88
============================================
  Files          1508     1114     -394
  Lines         74752    52698   -22054
  Branches      10846     7901    -2945
============================================
- Hits          53527    36613   -16914
+ Misses        17593    13477    -4116
+ Partials       3632     2608    -1024

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
69.47% <ø> (+0.06%)
:arrow_up:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

I ran the following command:
mvn --batch-mode release:update-versions -DdevelopmentVersion=0.4.0-SNAPSHOT

It seems that the release plugin changes the XML files in its own formatting style. I don't think that it's a good approach to individually apply the checkstyle for 60+ files. I will try to find the way if there's a way to address this.

@xiangfu0 changed and also updated the document at the following:
https://cwiki.apache.org/confluence/display/PINOT/Creating+an+Apache+Release


MySQL conversion issues on Windows with version 8.0.28

Bug description
The tests fail:
https://github.com/prisma/prisma-engines/runs/4867013985?check_suite_focus=true
Were green with 8.0.27.

Changes in MySQL 8.0.28 (2022-01-18, General Availability)
https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-28.html


Can ssh keys be updated on the dev rewards smartcontract ?

i deleted  the olf ssh and created a new one as the previous one was coming up to expiration date. now i realized I should have created a new file instead of replacing the previous private key stored.

Dublicate of #63


Project editor includes absolute path in config's appbase

I've been working with nunit projects manually and today I noticed that by switching configurations, from the GUI runner, mods are done to the project file.
One of the mods introduces the absolute path just before relative one.
That is, it goes from '......' to 'D:\svnwc......'

This change to the project file was done by the old V2 gui when the project config was changed. So it's really not a project-editor bug, even though the gui made the change using the project editor.
The new project editor under development doesn't do this, so I'm closing the issue.


Not working on Surface RT

https://drive.google.com/file/d/1ivy4B3FoW0Zw7Aqu5OudIFgYqBji3vXY/view?usp=drivesdk

https://drive.google.com/file/d/1-bQcQ-EbT5NTDolgr9sERV7Qc7u2VwsX/view?usp=drivesdk

https://drive.google.com/file/d/1wupa_aAR-NIJN-gyyJROCE6wsWu1-Gs3/view?usp=drivesdk

Do you have Secure Boot disabled?

ARM is indeed not supported.


unknown feature stdsimd

When compiling the project with version 1.78.0-nightly, it fail complaining, that the feature stdsmid is unknown. This occurs when compiling ahash-0.8.6.

Could I have a full error dump please? Including system you are running on, and nu version?


Copy changes to Apexcharts and Infragistics grid promo

Copy changes

Thanks for creating the PR.
I didn't notice this before doing the changes on my side. Closing this and pushing my changes.


Upload Geo-Referenced Video From Garmin Virb

I was wondering if there is going to be support for uploading geo-referenced viddeon from devices like Garmin Virb....which I can currently do with Mapillary. If this can be implemented it would be nice if there is no upload size restriction that is currently in place for Mapillary.

Although it's simpler to take video, the resolution on the virb video(2mp) can make it hard to make details out. Why not just use the time lapse feature every 1 second as it produces higher quality imagery(13mp). This is what I do on mine

Although it's simpler to take video, the resolution on the virb video(2mp) can make it hard to make details out. Why not just use the time lapse feature every 1 second as it produces higher quality imagery(13mp). This is what I do on mine

Can you describe how you do this with a Garmin Virb?  I haven't been able to record more than about a half hour of 1sec timelapse photos this way before the Virb shuts off.  I really think it's some flaw with the Virb.

Weird, I can go 2hours before it starts lagging and needs to be restarted. I went through the phone app and set 1sec intervals at full res. Then later I turn camera on via the power button and press the OK button to start the photo sequence. When I emailed garmin about the lagging they sent me this:
The VIRB XE is set up to record videos for a certain amount of time, it breaks a video into 25 minute segments and handles the Time Lapse in the same fashion. The information, that I received, it is designed  do Time Lapse up to about 4GB of photos, then turn off. I am sorry, but this is normal functionality for the camera.
Maybe mine is broken in a good way?

"The VIRB XE is set up to record videos for a certain amount of time, it breaks a video into 25 minute segments and handles the Time Lapse in the same fashion. The information, that I received, it is designed do Time Lapse up to about 4GB of photos, then turn off. I am sorry, but this is normal functionality for the camera."

ahhh that makes a lot of sense for what I'm seeing - thank you for passing that along!

I've also observed a lot of laggy issues with leaving the Virb XE on timelapse for a long period of time. I've thought about doing video but have also worried about the quality and usefulness. Do you have any examples of the video quality?


Feature/jira prompt

Description
Will prompt the user to upgrade
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Chore: cleanup/renaming, etc
[ ] RFC

Notes
It's only a link without styling as to not be intrusive
Acceptance

[x] I have read the Contributing guidelines
[x] I have read the Code of Conduct

Probably not related to the PR, but it's not working.
Does it work for you?
Screen.Recording.2022-10-26.at.4.33.21.PM.mov

Unrelated, but true, broken somewhere.

Probably not related to the PR, but it's not working.
Does it work for you?
Screen.Recording.2022-10-26.at.4.33.21.PM.mov

Unrelated, but true, broken somewhere.

I'll approve because the code itself looks good.
Once I merge the remaining billing-related PR on the backend, I'll help debug this.


Handle float args to sumproduct with log backend

Addresses #915
Pair coded with @eb8680
This adds kludgey support for *_log backends when using sumproduct(), as needed in HMC.
Tested

added regression tests of sumproduct with float inputs

Codecov Report

Merging #1319 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #1319      +/-   ##
==========================================
+ Coverage   95.79%   95.79%   +<.01%
==========================================
  Files         121      121
  Lines        6313     6318       +5
==========================================
+ Hits         6047     6052       +5
  Misses        266      266

Impacted Files
Coverage Δ

pyro/ops/sumproduct.py
97.3% <100%> (+0.2%)
:arrow_up:

Continue to review full report at Codecov.

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


VirtualService fails to route using non-default cluster domain

Describe the bug
I use a not-default cluster domain k8s.local instead of cluster.local
root@test-547cf5f9-6vc6p:/# ping basic-trtis-deployment.trtlab
PING basic-trtis-deployment.trtlab.svc.k8s.lab (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from basic-trtis-deployment.trtlab.svc.k8s.lab (<IP_ADDRESS>): icmp_seq=1 ttl=64 time=0.058 ms

If I try to use cluser.local, ping fails to resolve
root@s3test2-547cf5f9-6vc6p:/# ping basic-trtis-deployment.trtlab.svc.cluster.local
ping: basic-trtis-deployment.trtlab.svc.cluster.local: Name or service not known

However, the only way to get my VirtualService to route was to use cluster.local instead of k8s.local
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: basic-trtis-deployment-virtual-service
spec:
  hosts:
  - "trt.lab"
  gateways:
  - basic-trtis-deployment-gateway
  http:
  - match:
    - uri:
        prefix: /api/health/live
    route:
    - destination:
        host: basic-trtis-deployment.trtlab.svc.cluster.local
        port:
          number: 8000
  - match:
    - uri:
        prefix: /
    route:
    - destination:
        host: basic-trtis-deployment.trtlab.svc.cluster.local
        port:
          number: 50051

Expected behavior
I expected the FQDN  basic-trtis-deployment.trtlab.svc.k8s.lab to be the value for host
Steps to reproduce the bug
Install k8s using kubespray and change the default cluster domain
Install metallb
Install istio-1.0.6 with helm using the default settings
Version
K8s 1.12.5
Istio 1.0.6
Installation
Kubespray + DeepOps
Environment
BareMetal: CoreOS for management nodes, Ubuntu for GPU compute nodes
Cluster state
Skipping for now...
{{ If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue. }}

We ran into the same issue. And our workaround is putting VirtualService and k8s Service in the same namespace. In your case,  put basic-trtis-deployment-virtual-service and basic-trtis-deployment in the same namespace and then specify host in VirtualService as basic-trtis-deployment.

I think you need to set global.proxy.clusterDomain, see https://istio.io/docs/reference/config/installation-options/

Is there a way for you to infer the proper default for clusterDomain from k8s at install time?

Probably not at install time, maybe at runtime. I think I remember seeing some discussion about parsing it from /etc/resolv.conf and it was decided to not be a good idea. If there is a better way to get the clusterDomain it could be possible