ARIA speaking double up

When I use toastr with VoiceOver on iOS it is saying the message 2 or 3 times. Tested on demo page.
It is using aria-live=polite role=alert
Not sure what is wrong with that, seems sensible to me. It's not very polite though, saying it several times!
Maybe toastr sets the inner html twice or something?

Found the bug. It causes double speaking using both aria-live and role=alert
Recommend removing the role=alert.
Further, it would be good for "error" to show aria-live=assertive and "info" messages to show aria-live=polite

Hey @mike-nelson - would you be interested in creating a PR with your changes?

All done.

Good one. Thanks for implementing!

Just pushed tests up.


Add react example and conversion from CFS package

React plus a document that shows how you can move files from the now depreciated CollectionFS/CFS packages to this one (amazon S3 specifically).

Missed toc

Could you please send same PR to dev branch?

Sure

Looks like there are other commits, can you do a PR for those or want me to include them?

 I don't see a way to exclude those and only allow mine in it.

I don't have access to your project, so can't update files directly right?

It's totally fine, send it


How to get all git repository names of a perticular project

Can you please let me know how to get the list of repos
https://docs.microsoft.com/en-us/rest/api/azure/devops/git/repositories/list?view=azure-devops-rest-5.1#gitrepository

@sunilsankar, you can get the list of repository object using,
git_client = connection.clients.get_git_client()
_repos = git_client.get_repositories(project=project)


Windows+MSVC+scipy>=1.8.0 → segfault

Let's consider sample program and nuitka call for compilation on Windows with MSVC:
https://github.com/belonesox/bug-nuitka-scipy-180-crash

pipenv-based setup-demo.bat for setup pipenv (pls, change your python location on set-python-path.bat).
demo_build.bat for build
trivial test.py

For linux — OK.
For scipy <= 1.7.3  → OK
For scipy >=1.8.0RC1 → test.exe crashed on
import scipy.stats or from scipy.stats import hmean.

Unhandled exception at 0x00D11CBA in test.exe: 0xC0000005: Access violation reading location 0x00000010.

[Inline Frame] test.exe!ATTACH_TRACEBACK_TO_EXCEPTION_VALUE(_object * exception_tb, _traceback *) Line 704
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\include\nuitka\exceptions.h(704)
test.exe!impl_scipy$_lib$_docscrape$$$function__14___init__(const Nuitka_FunctionObject * self, _object * * python_pars) Line 4935
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.scipy._lib._docscrape.cpp(4935)
test.exe!CALL_FUNCTION_WITH_ARGS3(_object * called, _object * const * args) Line 1755
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersCalling2.c(1755)
test.exe!CALL_METHOD_WITH_ARGS3(_object * source, _object * attr_name, _object * const * args) Line 10935
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersCalling2.c(10935)
test.exe!impl_scipy$_lib$_docscrape$$$function__42___init__(const Nuitka_FunctionObject * self, _object * * python_pars) Line 24421
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.scipy._lib._docscrape.cpp(24421)
test.exe!Nuitka_CallMethodFunctionPosArgs(const Nuitka_FunctionObject * function, _object * object, _object * const * args, int args_size) Line 2376
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\CompiledFunctionType.cpp(2376)
test.exe!CALL_FUNCTION_WITH_SINGLE_ARG(_object * called, _object * arg) Line 594
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersCalling2.c(594)
test.exe!impl_scipy$stats$_axis_nan_policy$$$function__6__axis_nan_policy_factory$$$function__3_axis_nan_policy_decorator(const Nuitka_FunctionObject * self, _object * * python_pars) Line 6567
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.scipy.stats._axis_nan_policy.cpp(6567)
test.exe!CALL_FUNCTION_WITH_SINGLE_ARG(_object * called, _object * arg) Line 359
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersCalling2.c(359)
test.exe!modulecode_scipy$stats$_stats_py(_object * module, const Nuitka_MetaPathBasedLoaderEntry * loader_entry) Line 160886
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.scipy.stats._stats_py.cpp(160886)
test.exe!loadModule(_object * module, _object * module_name, const Nuitka_MetaPathBasedLoaderEntry * entry) Line 975
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(975)
test.exe!_EXECUTE_EMBEDDED_MODULE(_object * module, _object * module_name, const char * name) Line 1014
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1014)
[Inline Frame] test.exe!EXECUTE_EMBEDDED_MODULE(_object * module) Line 1087
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1087)
test.exe!_path_unfreezer_exec_module(_object * self, _object * args, _object * kwds) Line 1423
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1423)
[External Code]
python38.dll![Frames below may be incorrect and/or missing, no symbols loaded for python38.dll]
test.exe!CALL_FUNCTION_WITH_ARGS5(_object * called, _object * const * args) Line 3240
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersCalling2.c(3240)
test.exe!IMPORT_MODULE5(_object * module_name, _object * globals, _object * locals, _object * import_items, _object * level) Line 134
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersImport.c(134)
test.exe!modulecode_scipy$stats(_object * module, const Nuitka_MetaPathBasedLoaderEntry * loader_entry) Line 852
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.scipy.stats.cpp(852)
test.exe!loadModule(_object * module, _object * module_name, const Nuitka_MetaPathBasedLoaderEntry * entry) Line 975
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(975)
test.exe!_EXECUTE_EMBEDDED_MODULE(_object * module, _object * module_name, const char * name) Line 1014
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1014)
[Inline Frame] test.exe!EXECUTE_EMBEDDED_MODULE(_object * module) Line 1087
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1087)
test.exe!_path_unfreezer_exec_module(_object * self, _object * args, _object * kwds) Line 1423
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1423)
[External Code]
test.exe!CALL_FUNCTION_WITH_ARGS5(_object * called, _object * const * args) Line 3240
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersCalling2.c(3240)
test.exe!IMPORT_MODULE5(_object * module_name, _object * globals, _object * locals, _object * import_items, _object * level) Line 134
	at C:\Users\stas\.virtualenvs\nuitka-scipy-stats-crash-vzkNlNj9\Lib\site-packages\nuitka\build\static_src\HelpersImport.c(134)
test.exe!modulecode___main__(_object * module, const Nuitka_MetaPathBasedLoaderEntry * loader_entry) Line 8768
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.__main__.cpp(8768)
test.exe!loadModule(_object * module, _object * module_name, const Nuitka_MetaPathBasedLoaderEntry * entry) Line 975
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(975)
test.exe!_EXECUTE_EMBEDDED_MODULE(_object * module, _object * module_name, const char * name) Line 1014
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1014)
[Inline Frame] test.exe!IMPORT_EMBEDDED_MODULE(const char *) Line 1068
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MetaPathBasedLoader.cpp(1068)
test.exe!EXECUTE_MAIN_MODULE(const char * module_name) Line 416
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MainProgram.cpp(416)
test.exe!wmain(int argc, wchar_t * * argv) Line 1101
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\static_src\MainProgram.cpp(1101)
[External Code]
test.exe!modulecode_scipy$stats$_continuous_distns(_object * module, const Nuitka_MetaPathBasedLoaderEntry * loader_entry) Line 448047
	at C:\projects\bugs\nuitka\nuitka-scipy-stats-crash\test.build\module.scipy.stats._continuous_distns.cpp(448047)

Line 4935 of module.scipy._lib._docscrape.cpp
    ATTACH_TRACEBACK_TO_EXCEPTION_VALUE(exception_keeper_value_1, exception_keeper_tb_1);

corresponds to  148 line in scipy\_lib\_docscrape.py
        try:
            self._parse()     # ←←←← here
        except ParseError as e:
            e.docstring = orig_docstring
            raise

All generated sources attached → test-build.zip
(another version … test-build.zip )

Crash not depens:

nuitka version  (reproduced on all since 0.7.0, including factory, but factory got another fail → build.log, not tested before 0.7.0)
Python version (reproduced on w32 python 3.8.10, w64 python 3.10.0)

Not tested — mingw GCC.
I can provide access for VM with Windows, if it helps...

Fantastic, I managed to reproduce this one just now. I am running with a --debug compile now, your stack trace is already somehow helpful, but didn't tell much enough yet. Maybe it's a corruption bug. Since scipy has been working previously, and 1.8.0 will only need hidden imports, this should be git bi-sectable. I assume you say that 0.6.19 worked for you, correct?

As a pro-tip, when adding nuitka to pipenv, enter a shell and go to git repository of Nuitka, then do python setup.py develop which will modify the virtualenv such that it  uses the checkout, no matter now you change it. Then you can switch versions there with ease.
Also, the pipenv file is best provided with the lock file, just in case, because it makes it impossible to not have what you have.
The C compiler barely ever makes a difference at all to compatibility bugs.
I reproduced with 3.10, but I will now switch to Linux, where debugging is usually easier for me. If it's not a corruption bug, it should be reproducible there.

In a unrelated note, the crashing code, I mean the scipy.lib._docscape looks totally like bloat only to me. Once this is resolved, we should strive to remove it from runtime.

I assume you say that 0.6.19 worked for you, correct?
Hmm. Not tested "scipy=1.8.0 + nuitka=0.6.9"

python setup.py develop which

I fastly switching versions with pipenv (it more reliable, because sometimes «setup develop» ignored and overrided by pip versions, always need to check, that there is  no pip version here). I use «setup develop» only for factory install (it cannot be installed with
  nuitka = {editable = true, ref = "factory", git = "https://github.com/Nuitka/Nuitka.git"}

in Pipfile. But it should! I dont now why it not worked for me :(  (should we consider it as a bug?)

pipenv file is best provided with the lock file,

My bad, forgot to commit it.

I reproduced with 3.10, but I will now switch to Linux, where debugging is usually easier

Hmm. The bug is not reproduced for me on gcc/linux…

I mean the scipy.lib._docscape looks totally like bloat only to me

Of course, everything that can be anti-bloat, should be anti-bloat.

I think this may well be the same root cause as #1532, and I may just have found it. It seems I missed an optimization that copying of dictionaries does, and that for empty directories, the Python3 split table appears to be there, but it's not allowed to use it like that. With many copy calls of kwargs which since some point is statically known to be of dictionary type, a reference to the dictionary keys is lost, and for larger programs that copy star arguments, it is crashing.
I have of course to verify it, but this seems great news potentially.

It is incredibly slow to compile with MinGW64, what is relatively acceptable with MSVC, takes hours to complete with it. I am doing it, because I have hopes for better tracebacks. But I guess, my plan on this one is to make sure all CPython test suites run with 0.8 and debug python versions, in vague hopes of catching it under defined conditions with reference counting tests.

no idea what  {editable = true, ref = "factory", git = "https://github.com/Nuitka/Nuitka.git"} means to pipenv. Does it address the issue with it changing that I was talking about?

It is just result in Pipfile of «pipenv branch install»
pipenv install -e git+https://github.com/Nuitka/Nuitka.git@factory#egg=Nuitka

Sometimes it works, sometimes got strange errors.
I use it because I like to consider Pipfile as «Infostructure as Code», Domain Specific Language to rule all of it (I prefer it over shell commands and manual connections by «setup develop» of some git-branches,  etc).
But it is not critical and not connected to the issue, lets forget about it now here.

When I compiled this with MinGW64 it took an absurd amount of time, and it didn't crash.
Maybe this hints at this being a "use after free" race, that's where C compilers can come in. Not sure how much we can do with Valgrind, in the past it was really bad with Python core already, maybe it improved.

I fixed a reference loss that I found, but sadly, it's still crashing.

I can reproduce it with 1.8.1 as well. I didn't manage to reproduce it on Linux, although I thought I did, but this may not be the same issue that plagues pyqtgraph after all, there it is the case.
I had a vague suspect that dictionary operations might be at fault. I therefore changed all dictionary copying implementations to avoid working with low level details, and us PyDict_Copy, but that didn't help. I also disabled vectorcall usage and offering the slot, both. I disabled optimization of calls for uncompiled functions, for C functions, for constructor quick calls, all didn't change it.
Then I also removed all dictionary operations and all string operations, and still it crashes, pretty much in a reproducible way, which is of couse showing that this code is all harmless.
Next up are free lists, since I recall seeing an object inside the freelist being corrupted, but that may only be a symptom, and not the cause.

I managed to extract a test case from scipy, that indeed crashes too, but doesn't use scipy nor numpy, and as such removes a lot of things out of the equation. Interestingly, with 64 bits Python, MSVC it is crashing, with MinGW64 it is not. Will try to reduce the crasher further now, it's only 3 files that are not extremely large.

With the extract, I was now able to run it through the reference checking method, and indeed on Linux (gcc and clang) and Windows with MinGW64, no issue, but MSVC crashing, so indeed this must be specific to the MSVC compilation, which is very strange.

So, I reduced this very far, to actually just the parsing of string sections. The code that I last caught misbehaving is having this kind of code:
    def _is_at_section(self):
        self._doc.seek_next_non_empty_line()

        if self._doc.eof():
            return False

        l1 = self._doc.peek().strip()  # e.g. Parameters

        if l1.startswith(".. index::"):
            return True

        l2 = self._doc.peek(1).strip()  # ---------- or ==========
        s1 = "-" * len(l1)
        s2 = "=" * len(l1)

        assert type(l2) is str, type(l2)
        return l2.startswith(s1) or l2.startswith(s2)

What happens is that if I look at generated code, the first time l2.startswith returns True, it has a refcount of 794, and then, although from there it passes only through goto statements, becomes negative. Adding assertions at a certain point, makes it go away. The or is a bit delicate with reference counting, but it doesn't make a mistake there, and even if it did, True is a very safe value to mishandle, it doesn't just release.
Maybe the corruption happened before, but it looks all pretty bare bone code. I will make a few more checks, e.g. if the type of the value changes, or something like that.

I noticed, that not releasing l1 makes a difference. I recall that strings owned by the parsed list, were causing crashes before, and here maybe too, not sure. I tried to reduce the code that produces it further. But then I still get this:

Assertion failed: (tmp_return_value) != NULL, file module._docscrape.cpp, line 6173

However, even if I manually modify the code to assign from -1 and have traces it is being assigned from non-NULL only, and remains non-NULL for a while during function cleanup, even tracing its value with %lx to 0. This is just crazy.

For fun, I also disabled LTO mode, just in case, but that didn't make a difference.

This will be in 0.9.6 with a workaround that changes the code in a way, that the generated code being slightly different, does not cause MSVC to miscompile.  I think this is the first time, an actual C compiler bug was encountered. I wonder if somebody wants to dig deeper there, but I am pretty much fed up now.

This was release in the meantime.


A0-1-4: Consider additional uses for parameters

Affected rules

A0-1-4

Description
Add or validate that the following are considered as uses of parameters:

Consider [[maybe_unused]]
Add test for lambda parameters
Consider void casts and assignments to std::ignore.

Example
void f(
  int i,  // compliant
  int j,  // compliant
  int k // compliant
) {
   static_cast<void>(i);
   (void)j;
   std::ignore = k;
}

The query handles lambda parameters because the underlying member function is considered by the general case. Is there more information regarding FPs and lambda parameters?
Same as A0-1-2, is there more information around FPs when they are assigned to std::ignore?

Also, all of the three cases were already being acknowledged by the query, because they all counted as variable reads, hence were UsedParameters.


Allow special characters in tag names

Instead of requiring the name and slug to be identical, we always keep
the inputted name of the tag, and use the slug only for urls and lookup.
Tried creating a tag called 'Økonomi' the other day, and it was saved as 'konomi' :disappointed:

When trying to click a tag with a special character, like 'ø' in it, the server crashes.

When trying to click a tag with a special character, like 'ø' in it, the server crashes.

Where did you find a link that contained a special character?
The idea is that URLs are not supposed to contain special characters anymore, so the links should use the slug as URL, not the name.

When trying to click a tag with a special character, like 'ø' in it, the server crashes.

Where did you find a link that contained a special character?
The idea is that URLs are not supposed to contain special characters anymore, so the links should use the slug as URL, not the name.

I added a tag økonomi to a compendium and clicked it.

I added a tag økonomi to a compendium and clicked it.

Okay, probably a bug in the js, then

ping
Fixed the js bug

btw


Autogrow feature #7644 which resolves issue #7631 not working when text is initialized with @bind-value

Bug type
Component
Component name
MudTextField
What happened?
-Affected implementation feature #7644 which resolves issue #7631 (Feature: AutoGrow and MaxLines)
-Released in version 6.11.1
FYI:
@AntMaster7
@fondraco
Problem
When the component is initialized and the text passed not by a user input (this autogrow works) but some multiline text is passed through the variable @bind-value, the compnent shows just the first line. In this case the AutoGrow feature seems not working.
The AutoGrow is in this case working only when the user clicks into the MudTextField and makes a "enter", at this point the autogrow will resize the component.
Expected behavior
The component MudTextField with AutoGrow="true" must resize the lines also at initialization and when a multiline text value is passed through a property variable binding like:
<MudTextField  AutoGrow="true" MaxLines="5" @bind-Value="@Text" .... />

@code {
    public string Text { get; set; } = ReadText.SomeLargeMultilineContent();
}

Reproduction link
https://try.mudblazor.com/snippet/GkQxlQYhKEzWSuTy
Reproduction steps
Unfortunately the playground is running on the old mudblazor v6.11.0, but the referred bug has been implemented in the newer releases v6.11.1. So this reproduction is not possible on later v6.11.0.
Relevant log output
No response
Version (bug)
6.11.1
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Solution available


Update brain-workshop to 4.8.4

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as<PHONE_NUMBER>4af33e163e1bbe820145568725692c. Your contribution is still credited to you.


O/E data not available

Hi
I had a problem when I was running juicer box dump from HiC data which contain 180M validpairs.
However, it showed "O/E data not available at chr5 BP_25000 KR", and could not generate results file.
But when I extracted only 10M validpairs, it worked well.
May I ask, what should I do to get whole data dump file ?
The code ran as
" ~/HiC-Pro_2.11.4/bin/utils/hicpro2juicebox.sh -i 2221-1.hicup.180M.validPairs -g mm10 -j ~/juicer_tools_1.22.01.jar -r ~/HiC-Pro_2.11.4/annotation/MboI_mm10.bed -o ./"
And it reported as
"WARN [2021-03-08T14:11:31,075]  [Globals.java:138] [main]  Development mode is e                                                                                nabled
O/E data not available at chr5 BP_25000 KR"
Have a nice day.
Best
Yuhao

Hello
It is possible there is no data at 25Kb normalized at KR. Try using SCALE,
our updated balancing normalization (and use Juicer Tools during pre to
ensure that SCALE is included).
On Mon, Mar 8, 2021 at 2:44 AM Stormsky99<EMAIL_ADDRESS>wrote:

Hi
I had a problem when I was running juicer box dump from HiC data which
contain 180M validpairs.
However, it showed "O/E data not available at chr5 BP_25000 KR", and could
not generate results file.
But when I extracted only 10M validpairs, it worked well.
May I ask, what should I do to get whole data dump file ?
The code ran as
" ~/HiC-Pro_2.11.4/bin/utils/hicpro2juicebox.sh -i
2221-1.hicup.180M.validPairs -g mm10 -j ~/juicer_tools_1.22.01.jar -r
~/HiC-Pro_2.11.4/annotation/MboI_mm10.bed -o ./"
And it reported as
"WARN [2021-03-08T14:11:31,075] [Globals.java:138] [main] Development mode
is e nabled
O/E data not available at chr5 BP_25000 KR"
Have a nice day.
Best
Yuhao
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/209, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EWYKNCO2UP64D6OTFYDTCR55DANCNFSM4YY4QLRA
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Hi, Thank you for reply.
But I still can not extract data from any normalization. And in JuicerBox visualization, I can't choose any normalization either.
My hic file was generated from HiC-Pro validpairs, and with less validpairs, Juicer_tools worked well.
May Juicer_tools be not compatible with high validpairs HiC-Pro data?

You cannot choose any normalization at any resolution?
On Tue, Mar 23, 2021 at 2:52 AM Stormsky99 @.***> wrote:

Hi, Thank you for reply.
But I still can not extract data from any normalization. And in JuicerBox
visualization, I can't choose any normalization either.
My hic file was generated from HiC-Pro validpairs, and with less
validpairs, Juicer_tools worked well.
May Juicer_tools be not compatible with high validpairs HiC-Pro data?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/209#issuecomment-804664050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW2Z47P2TJZRM2ND5DDTFA3AHANCNFSM4YY4QLRA
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Yes, all normalization at any resolution in "JuicerBox" and "juicer_tools dump" were failed with hic file which was generated from 180,000,000 validpairs. Only "NONE" could be chosen.
But when I generate hic file with half validpairs, 90,000,000, the normalization worked well.
It's so confused.
Perhaps, there were some problems in the codes I ran? Or juicer_tools is not compatible with too much validpairs from HiC-Pro?
hicpro2juicebox.sh -i hicup.180M.validPairs -g mm10 -j juicer_tools_1.22.01.jar -r MboI_mm10.bed -o ./
hicpro2juicebox.sh -i hicup.90M.validPairs -g mm10 -j juicer_tools_1.22.01.jar -r MboI_mm10.bed -o ./
java -jar juicer_tools_1.22.01.jar dump oe KR hicup.180M.validPairs.hic chr1 chr2 BP 25000 test.txt
java -jar juicer_tools_1.22.01.jar dump oe KR hicup.90M.validPairs.hic chr1 chr2 BP 25000 test.txt
Best

If your hic file is loading, you can just run addNorm on it.
https://github.com/aidenlab/juicer/wiki/AddNorm
On Tue, Mar 23, 2021 at 7:11 AM Stormsky99 @.***> wrote:

Yes, all normalization at any resolution in "JuicerBox" and "juicer_tools
dump" were failed with hic file which was generated from 180,000,000
validpairs. Only "NONE" could be chosen.
But when I generate hic file with half validpairs, 90,000,000, the
normalization worked well.
It's so confused.
Perhaps, there were some problems in the codes I ran? Or juicer_tools is
not compatible with too much validpairs from HiC-Pro?
hicpro2juicebox.sh -i hicup.180M.validPairs -g mm10 -j
juicer_tools_1.22.01.jar -r MboI_mm10.bed -o ./
hicpro2juicebox.sh -i hicup.90M.validPairs -g mm10 -j
juicer_tools_1.22.01.jar -r MboI_mm10.bed -o ./
java -jar juicer_tools_1.22.01.jar dump oe KR hicup.180M.validPairs.hic
chr1 chr2 BP 25000 test.txt
java -jar juicer_tools_1.22.01.jar dump oe KR hicup.90M.validPairs.hic
chr1 chr2 BP 25000 test.txt
Best
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/209#issuecomment-804816968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW3RNZIVDKSQBO2C2XDTFBZNNANCNFSM4YY4QLRA
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org


Return parsed values of AuthnRequest to get attributes like ForceAuthn

Issue: #7
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[ ] Does your submission pass tests?
[x] Have you lint your code locally prior to submission?

Changes to Core Features:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?

@avinashdhinwa
Great Work! LGTM


MQ Build failed with make[1]: Leaving directory

Hi team,
My MQ build is failed with below error, which doesn't explain what exactly is caused this issue?
I am using branch 9.1.2 for creating MQ container -
sudo MQ_ARCHIVE=IBM_MQ_9.1.2_LINUX_X86-64.tar.gz MQ_VERSION=9.1.2 make build-advancedserver
 244.47 MiB / 247.97 MiB [=====================================================]
 247.91 MiB / 247.97 MiB [=====================================================]
 247.97 MiB / 247.97 MiB [==================================================] 8s
Copying config sha256:b1a456dcd8534c4c9d975974f7bb6e1db0b3bf5b93369e6663eeccd8654b8a89

 0 B / 5.61 KiB [--------------------------------------------------------------]
 5.61 KiB / 5.61 KiB [======================================================] 0s
Writing manifest to image destination
Storing signatures
b1a456dcd8534c4c9d975974f7bb6e1db0b3bf5b93369e6663eeccd8654b8a89
+ buildah rm rhel-working-container
e2454c23a08feddfde30e065c46065399124127c47b52745cc4d9847f2952707
make[1]: Leaving directory `/app/appcd/mq-container_b9.1.2'

After putting debug point, we realized that the script is working fine.


build error due to legacy shuffle API

it threw:
nvcc -O3 -std=c++11 --compiler-options=-fPIC -use_fast_math --ptxas-options="-v" -gencode arch=compute_70,code=sm_70 -gencode arch=compute_61,code=sm_61 -gencode arch=compute_52,code=sm_52 -gencode arch=compute_30,code=sm_30 -DOMPI_SKIP_MPICXX --keep-dir /test/workspace/ctr/amazon-dsstne/build/tmp/engine/cuda -I/usr/local/include -isystem /usr/local/cuda/include -isystem /usr/lib/openmpi/include -isystem /usr/include/jsoncpp -IB40C -IB40C/KernelCommon -I/test/workspace/ctr/amazon-dsstne/build/include -I../utils  -c kernels.cu -o /test/workspace/ctr/amazon-dsstne/build/tmp/engine/cuda/kernels.o
ptxas /tmp/tmpxft_00000a17_00000000-8_kernels.compute_70.ptx, line 61962; error   : Instruction 'shfl' without '.sync' is not supported on .target sm_70 and higher from PTX ISA version 6.4

the old CUDA shuffle API have been deprecated in CUDA 9.0, and not available after CUDA 10 (link)

CUDA: 10.1
GPU: NVIDIA Tesla V100

looks like the macro CUDA_VERSION didn't work at https://github.com/amzn/amazon-dsstne/blob/master/src/amazon/dsstne/engine/kernels.h#L264
replacing #if (CUDA_VERSION >= 9000) with #if (__CUDACC_VER_MAJOR__ > 9) will work

after investigation, this problem was caused by old version of CUB, which referred by setup instructions (https://github.com/amzn/amazon-dsstne/blob/master/docs/getting_started/setup.md#cub-setup)
version 1.5.2 is too old for Tesla V100. could anyone update setup instructions?

So first problem is that I do not believe anyone is left at Amazon to accept pull requests. But since you seem to work at NVIDIA and I do too, we could at least maintain my fork of it. I am surprised it's not working under CUDA 10 because I have had no issues building for Turing. But I thought we already handled this correctly because it was an issue as of CUDA 9? Also where is the CUB version in the source? I am betting my local version is newer than 1.5.2 and that might have obscured this for some time.

this line also uses old shuffle API, which needs to be fixed
https://github.com/amzn/amazon-dsstne/blob/master/src/amazon/dsstne/knn/topk.cu#L141


Missing preUpdate function for bitmaptext children causes crash

At the bottom of Phaser.Component.Core.preUpdate() is a loop to call preUpdate() on the objects' children.
A bitmapText object has PIXI.Sprite children without preUpdate members.
So bitmapText seems broken on the current 'dev' branch.  Verify this by selecting the 'Load Bitmap Font' from the 'Loader' examples and then selecting the 'dev' phaser version.

I fixed this in dev a few days ago, have pushed it up now. Still not in the build files though.

That fixes the problem, but this pull also introduces another break.  In PIXI.DisplayObjectContainer.prototype._renderCanvas() you've removed the 'var' from the loop so it'll use a non-local 'i'.  For me it goes into an infinite loop.  Do you want a PR for this?

Nah it's already fixed. It doesn't infinite loop though, just raises it globally. But eitherway, 'tis fixed.

Possibly conflicted with an 'i' in a higher scope, Chrome through a right wobbler and wouldn't die.


different names on github and npm

on npm is es6-template-regex, there's no es6-delimiter-regex on npm.
/cc @jonschlinkert

will fix

done

sorry about that. looks like everything is named using templates on the readme etc. But locally, looks like I had the project in a folder named es6-delimiters-regex. I guess I changed my mind lol

:laughing: :+1:


fix readme POST SetForm

Hi,
Fix small typo

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #50   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines         143    143
=====================================
  Hits          143    143

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 62d56c4...fe86fe5. Read the comment docs.

@agirot Thanks.


Lint'ed cmd/influx package

[ ] CHANGELOG.md updated
[x] Rebased/mergable
[x] Tests pass
[x] Sign CLA (if not already signed)

Refs #4098

+1

I leave to you the call to close this PR since #4703 was merged. Btw, great job @pablolmiranda!


Fix internal Rollbar error caused when ActiveRecord was present, but not used

If the ActiveRecord gem was required in a Rails app, but not actually used or configured (in favor of some other ORM or no ORM), then Rollbar would internally error whenever it tried to send an exception message. This was being caused by the person tracking functionality seeing that ActiveRecord existed, but then it would raise a ActiveRecord::ConnectionNotEstablished error when it actually tried to use the connection. This in turn led to a rather non-obvious error in the logs:
[Rollbar] Reporting internal error encountered while sending data to Rollbar.
[Rollbar] Sending payload
[Rollbar] Got unexpected status code from Rollbar api: 422
[Rollbar] Response: {
  "err": 1,
  "message": "Invalid format. data.context object value found, but a string is required."
}

This should fix this person-tracking situation by first checking whether ActiveRecord is actually connected before trying to do ActiveRecord-specific functions.

@GUI thanks for this! ;-)
@brianr we can merge this, it's a clear bug in our side.

Thanks @GUI! Will get a new release out shortly.

Up on rubygems in version 1.4.1. Thanks again!


NullReferenceException when disposing of ConnectionSettings

NEST/Elasticsearch.Net version:

NEST v7.8.0
Elasticsearch.Net v7.8.0

Elasticsearch version:
N/A
Description of the problem including expected versus actual behavior:
Disposing of a Nest.ConnectionSettings object that has not been used throws a NullReferenceException.
Steps to reproduce:
Simply run this code in a .NET Core project (didn't seem to happen with a .NET Framework project):
using (var settings = new ConnectionSettings())
{
}

System.NullReferenceException: 'Object reference not set to an instance of an object.'
   at Elasticsearch.Net.RequestDataHttpClientFactory.StopCleanupTimer()
   at Elasticsearch.Net.RequestDataHttpClientFactory.CleanupTimer_Tick()
   at Elasticsearch.Net.RequestDataHttpClientFactory.Dispose()
   at Elasticsearch.Net.HttpConnection.DisposeManagedResources()
   at Elasticsearch.Net.HttpConnection.System.IDisposable.Dispose()
   at Elasticsearch.Net.ConnectionConfiguration`1.DisposeManagedResources()
   at Elasticsearch.Net.ConnectionConfiguration`1.System.IDisposable.Dispose()

Expected behavior
An exception is not thrown.
Provide ConnectionSettings (if relevant):
N/A
Provide DebugInformation (if relevant):
N/A

Downgrading to Elasticsearch.Net and NEST version 7.7.1 resolve this issue for me on .Net Core 3.1.

Thanks for opening @reduckted. This is related to the internal HttpClientFactory introduced to refresh clients in 7.8.0. We'll look to fix this.


GuildMessageReactionRemoveEvent bug

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the [latest JDA version][download].
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
GuildMessageReactionRemoveEvent is returning "getMember()" as a null object when I tried to get the User name.
Expected Behavior
bring to my code the object of "getMember().getUser()"
Code Example or Reproduction Steps
Code and the thrown exception
Code for JDABuilder or DefaultShardManagerBuilder Used
Code behind the error
Exception or Error
Full Exception Stacktrace

This is not a bug.
JDA provides the information it's given by Discord. Because Discord doesn't provide a member object this information isn't available. Equally, getMember is marked nullable so your code should account for this.

So, how can I get the information about who "unreacted" to the emote?

So, how can I get the information about who "unreacted" to the emote?

Make a request to the guild to retrieve a member by the id of the user who removed the emote.

Ok, I'll try it, thanks by the way!

Duplicate of #1261 #1275


Anonymous repositories are hidden to authenticated users unless they are a contributor

I understand that at the moment anonymous (unauthenticated) users cannot browse Bonobo.  This is a different issue.
Authenticated users cannot see anonymous repositories in the Index unless they are set as a contributor/admin/team.  Attempting to browse the Repository Details URL directly returns "Unauthorized".
My expectation is that anonymous repositories are shown in the Index to all authenticated users and the repository contents are browsable with non-admin controls.
Git access works correctly - anonymous (unauthenticated) users are allowed to pull from repositories if the Anonymous setting is enabled, and not allowed if set otherwise.  However, since the repository cannot be browsed, the Git URL must be shared outside of Bonobo.

Bonobo v<IP_ADDRESS>8
IIS 10
AD authentication
AD membership

Thanks for this - it's a good point.  At the moment the repo's AnonymousAccess flag is only tested when checking anonymous users for permission to access a repo.  Once a user is named then we no longer test that flag.
I agree that doesn't make much sense, particularly as they can just pull the data anyway via Git, so it's not like it's a security issue.
I don't think it will be very difficult to fix.

This is somehow still an issue for us on 6.3.0. A repo configured with anonymous access, without any contributors or teams, is visible but cannot be browsed by authenticated users. Unauthenticated users can browse the repository just fine. And it can be cloned by anyone.

Was this ever fixed?  I'm still getting this behavior.


BUG-000107342  There is a typographical error in Viewshed.py script from the Military Tools for ArcGIS 10.5.1

Salesforce ID: BUG-000107342
Salesforce Submitter: Emad Mohammed
Salesforce Submit Date: 8/14/2017 9:06 AM
Salesforce Bug Type: Failure/Error
Salesforce Severity: Medium
Repro Data: (n/a)
Work Around: (n/a)
Product: ArcGIS for Defense
Functional Category: ArcGIS for Defense
Client Platform: (n/a)
Version Found: 2.0 Beta
Planned Version Fixed: (n/a)
Comment: (n/a)

Steps to Repro:
Using Military Tools for ArcGIS 2.0 with Portal/Server 10.5.1 when setting up the visibility geoprocessing service on a closed system, the user noticed a small typo in the Viewshed.py script.
The typo is on line 183, and reads: arcpy.env.mask = "in_memory\OutBuffer"
It should read: arcpy.env.mask = "in_memory\OuterBuffer". The "er" is missing from the end of the line between "Out" and "Buffer".
Kindly refer to the following link:
https://solutions.arcgis.com/defense/help/military-tools/web-appbuilder/set-up-elevation-geoprocessing-service.htm

Has anyone talked about this? What is the typographical error?

I have verified that this was fixed in Military Tools for ArcGIS 1.1, which was released in June 2017

@topowright This issue was reported on Military Tools for ArcGIS 2.0 - was it an issue at 1.0, fixed at 1.1 but reintroduced at 2.0?

I wasn't aware of us fixing this until #246 - https://github.com/Esri/visibility-addin-dotnet/pull/246/files - I wasn't aware there was an open issue associated with this, so sorry for not linking to this in the PR

@joebayles the issue was a typo where we were missing the er in the word outer.
@lfunkhouser the issue is present in 1.1 and 2.0. My mistake on the comment above. The issue has been fixed based on the pull request referenced by @csmoore above.

Comment to support: This is verified as fixed and will be fixed in the next release of Visibility and Military Tools for ArcGIS.


feat: #120: change landingpage name

fix #120
changed landingpage => project page
changed homepage => dashboard

still needs to be changed for menubar landingpage but that feature is not yet merged in aisensus-develop


Hide Conference preference row

Hide conference option from preferences when there is no conference data.
This is to avoid confusing users with an option that will do nothing if there isn't conference data.

https://github.com/UCSD/campus-mobile/commit/64ce850ea1632be39817d7d3b8edcbd397b98cfe


40/Refactor masking logic

Closes #40.
This PR replaces the old stash code and item number checks with the pre-existing helper functions. I've also cleaned reduced some duplication of boolean checks. The unit tests pass & manual file generation matches the NC output from the original script from the 1st commit in this repo.
This PR is mainly to have others verify if the logic is correct :-)

Related to this work is refactoring the masking logic across process(), such as this initial experiment to clean up cube filtering and the logic for selecting masking:
https://github.com/ACCESS-NRI/um2nc-standalone/blob/40/refactor-masking-filtering/umpost/um2netcdf.py
I've commented on the logic part in source issue #40 :
https://github.com/ACCESS-NRI/um2nc-standalone/issues/40#issuecomment-2235163183
Thinking how to address this, there's two main options:

Expand issue #40's tasks & include logic refactoring here
Raise a new issue

@blimlim & @jo-basevi do you have any thoughts on these options?

I think a separate issue would work well, but don't have a very strong preference!

I think either works. The same issue is good as it has the discussion already there, but sounds like the implementation has a bit more a wider refactoring with filtering and logic for selecting masking so a separate issue also makes sense?
I've tried to follow the logic in the issue and don't fully understand it yet, but generally it sounds a good plan to filter non-maskable cubes early on and having additional specific warnings.

For a few thoughts:

I'm aiming to keep PRs to 50 changed lines or less & focused on one "topic"
The logic in process() should be tested, but the function is still unwieldy at ~90 lines of code
Testing process() now probably requires fiddly mock setup, which incurs repeat changes every time code blocks are extracted...

I'll migrate the masking logic & discussion blurb to another issue. Fixing/testing the masking logic will be easier, e.g. once the cube renaming code blocks are extracted.

I'll migrate the masking logic & discussion blurb to another issue.

This is now in issue #50.


Makes context for terminating connections and server

Codecov Report
Attention: Patch coverage is 87.50000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 88.17%. Comparing base (60d41d1) to head (40f18af).

Files
Patch %
Lines

server/server.go
71.42%
1 Missing and 1 partial :warning:

channel/connection.go
92.30%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
+ Coverage   87.47%   88.17%   +0.69%
==========================================
  Files          17       17
  Lines         599      592       -7
==========================================
- Hits          524      522       -2
+ Misses         58       54       -4
+ Partials       17       16       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


DropIn does not show in version 1.1.0

Describe the bug
So far Adyen worked for us without problems. But with version 1.1.0 the following error appears whenever start('dropIn') is called:
Possible Unhandled Promise Rejection (id: 0):
Error: Unknown payment method or native module.

Make sure your paymentMethods response contains: dropIn
Error: Unknown payment method or native module.

Make sure your paymentMethods response contains: dropIn

Any idea? I can't find any change logs in 1.1.0 which tells to use the drop in in a different way ...

Hey @RyanLinXiang
What was your previous version?

1.0.0

I see..
Could you try dropin instead of dropIn ?
On 1.1.0 we noticed that some payment method types are case-sensitive


[bgfx] Add new port

Rewrite of #9163
I've only written a port for bgfx.  There's no way to get GENie to use the artifacts from another port (i.e. bx or bimg), so I saw little point in exposing them
There was a bug in vcpkg_build_make as well - the docs specified that there was a TARGET parameter where no such parameter existed.  I added this parameter, as well as two others to override the target for debug/release builds

Seems have problem:
make: *** No rule to make target 'vs2017-debug32'.  Stop.

A couple more changes in vcpkg_build_make.  I added a new WORKING_DIRECTORY option.  I removed the section that closes gpg-agent.exe on Windows since it doesn't work (the code was copy/pasted from the icu port - it still had a ref to a variable that was only in the original port).  It seemed unnecessary since the process is a part of Msys2 and in no way prevents the build from completing
As it turns out, the windows make target for the project is sort of broken.  The makefile just calls devenv PATH_TO_SLN /Build which ends up just launching Visual Studio.  I used the makefile to generate the SLN file, then ran msbuild over that

/azp run

/azp run

/azp run

/azp run

@Outurnate Can you address the review suggestions? Thanks!

Would be really eager to see this library added to vcpkg.

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.


[Question] Mocap not still at the end

From Fetch environments I observe that, even if the target position is reached, the mocap (and the robot) continues to move around the target. Is it only related to the number of learning timesteps ? Or something about how mocap works ?
I also have this problem in my simulation, in which I want to reach a target point minimizing oscillations of a DLO.
If you have any hints about this particular behaviour, please let me know.

Which reward function are you using, for

sparse: This is the expected behavior since no reward is given for stabilizing, just a boolean reward for being "achieving the objective"
dense: One would expect this to not happen, but in practice it still happens (to a lesser extent) due to the small gradients near the objective

Ok, I get the point. I ask you about this because in my simulation I have a strange behaviour.I'm using the following reward:
lambda1 = -2
reward = lambda1d_EE
if d_EE < self.distance_threshold:
reward = 5 - 100d_EE
where self.distance_threshold is equal to 0.03. In this way I obtain a positive reward when I "achieve the goal" (going into the safe zone) but I provide also a gradient to continue moving toward the target point. I don't know why but my body, after it reaches the target, starts to move back and forth at maximum action around the target (going in and out the target zone).

Send a video of your resulting behavior
And what is d_EE

oscillations.webm
d_EE is the distance between the white sphere and the target (red sphere). I considered the action: self.action_space = spaces.Box(-0.04, 0.04, shape=(self.n_actions,), dtype="float32"). So, It can perform every action between -0.04 and +0.04.
After the goal is reached, it starts oscillating : pratically, it continues to move applying before action  +0.04 and at the next step the action = -0.04, then again +0.04 and so on.

This looks to me like a simulation error, you will have to ask the MuJoCo developers


import ppgan.datasets.transforms.functional as custom_F AttributeError: module 'ppgan' has no attribute 'datasets'

python tools/video-enhance.py --input /home/hust/PaddleGAN/applications/video/1.mp4 --process_order DAIN DeOldify EDVR --output /home/hust/output/
Traceback (most recent call last):
File "tools/video-enhance.py", line 18, in
from ppgan.apps import DAINPredictor
File "/home/hust/PaddleGAN/ppgan/apps/init.py", line 16, in
from .deepremaster_predictor import DeepRemasterPredictor
File "/home/hust/PaddleGAN/ppgan/apps/deepremaster_predictor.py", line 24, in
from ppgan.models.generators.remaster import NetworkR, NetworkC
File "/home/hust/PaddleGAN/ppgan/models/init.py", line 20, in
from .makeup_model import MakeupModel
File "/home/hust/PaddleGAN/ppgan/models/makeup_model.py", line 32, in
from ..datasets.makeup_dataset import MakeupDataset
File "/home/hust/PaddleGAN/ppgan/datasets/init.py", line 15, in
from .unpaired_dataset import UnpairedDataset
File "/home/hust/PaddleGAN/ppgan/datasets/unpaired_dataset.py", line 18, in
from .base_dataset import BaseDataset, get_transform
File "/home/hust/PaddleGAN/ppgan/datasets/base_dataset.py", line 24, in
from .transforms import transforms as T
File "/home/hust/PaddleGAN/ppgan/datasets/transforms/init.py", line 15, in
from .transforms import PairedRandomCrop, PairedRandomHorizontalFlip, Add, ResizeToScale
File "/home/hust/PaddleGAN/ppgan/datasets/transforms/transforms.py", line 22, in
import ppgan.datasets.transforms.functional as custom_F
AttributeError: module 'ppgan' has no attribute 'datasets'
can you help me? what is wrong?thinks

update your code and try again? I can not reproduce your error

@majintao1994 If you use code in master branch, not release/0.1.0 branch. Maybe you need to install PaddleGAN from source. Or export PYTHONPATH to the PaddleGAN root dir. Since master branch add new models.

I re installed the environment, but there are new problems. The error is as follows：

need python  version >= 3.6

However, there may be something wrong with your documen

tation

sorry, we will update this doc immediately.

import ppgan.datasets.transforms.functional as custom_F
AttributeError: module 'ppgan' has no attribute 'datasets'

update your code

import ppgan.datasets.transforms.functional as custom_F
AttributeError: module 'ppgan' has no attribute 'datasets'
can you help me? what is wrong?thinks

update your code

I tried the codes below in google colab:
!pip install -U paddlepaddle-gpu==2.0.0rc0
!git clone https://github.com/paddlepaddle/PaddleGAN.git
%cd PaddleGAN/
pip install -v -e .
from ppgan.apps import RealSRPredictor
sr = RealSRPredictor()
sr.run("docs/imgs/monarch.png")

and get:
FileNotFoundError: [Errno 2] No such file or directory: 'README_en.md'
After I edited  'README_en.md' to  'README.md' I got the same message as below when running from ppgan.apps import RealSRPredictor
AttributeError: module 'ppgan' has no attribute 'datasets'

Thanks for the feedback, the difference between aistudio and colab is due to the different python version. This error occurs in python3.6. We will fix it as soon as possible.

Thanks for the feedback, the difference between aistudio and colab is due to the different python version. This error occurs in python3.6. We will fix it as soon as possible.

Now the error becomes:
     19 import paddle
     20 from .base_predictor import BasePredictor
---> 21 from ppgan.datasets.transforms import ResizeToScale
     22 import paddle.vision.transforms as T
     23 from ppgan.models.generators import AnimeGenerator

ImportError: cannot import name 'ResizeToScale'

Also, I have set the colab notebook using GPU but I still receive the massage:
/usr/local/lib/python3.6/dist-packages/paddle/fluid/framework.py:295: UserWarning: You are using GPU version Paddle, but your CUDA device is not set properly. CPU device will be used by default.
  "You are using GPU version Paddle, but your CUDA device is not set properly. CPU device will be used by default."

How can I use GPU in google colab?

Thanks for the feedback, the difference between aistudio and colab is due to the different python version. This error occurs in python3.6. We will fix it as soon as possible.

Now the error becomes:
     19 import paddle
     20 from .base_predictor import BasePredictor
---> 21 from ppgan.datasets.transforms import ResizeToScale
     22 import paddle.vision.transforms as T
     23 from ppgan.models.generators import AnimeGenerator

ImportError: cannot import name 'ResizeToScale'

Also, I have set the colab notebook using GPU but I still receive the massage:
/usr/local/lib/python3.6/dist-packages/paddle/fluid/framework.py:295: UserWarning: You are using GPU version Paddle, but your CUDA device is not set properly. CPU device will be used by default.
  "You are using GPU version Paddle, but your CUDA device is not set properly. CPU device will be used by default."

How can I use GPU in google colab?

colab cuda version is 10.1
you can try following command to intall correspond paddlepaddle:
!python -m pip install paddlepaddle-gpu==2.0.0rc0.post101 -f https://paddlepaddle.org.cn/whl/stable.html

问题过于久远，如果有图像和视频生成的需求，可以使用新的跨模态工具: https://github.com/PaddlePaddle/PaddleMIX/tree/develop


Update Unadjusted Laplace Approx to UGLA in training and paper repo

Update of the paper repo done in: https://github.com/CUQI-DTU/Paper-CUQIpy-1-Core/pull/14
Update of the training material done in: https://github.com/CUQI-DTU/CUQIpy-demos/pull/62
Update of CUQIpy done in: https://github.com/CUQI-DTU/CUQIpy/pull/217

@jakobsj and @nabriis
I think this issue can be closed without any code update. I added above the three PRs that implemented this change (I changed status to in review).

LGTM

Sounds good.  Great that you have added the PRs that dealt with these, for future reference. Please close.


Add integration test decorator

What does this PR do?

add decorator option for integration tests to be ommited when testing with process

closing as these changes have been made in a different pr


add the value of 6 to AirSwingLR

A CS-MTZ16WKE doesn't have LR swing, but returns 6. So I have added 6 so the device works. Obviously I can keep adding this value, but as I can see others on the internet who have a similar problem, it would make sense.

Nor working anymore.

You should not add this here, but to the upstream repository instead.

I've tried adding this mode before but it requires more than just adding it to the list, Panasonic is reusing the values and last time I added it broke the integration for a lot of people. So it needs to be conditionally added depending on the supported featured returned by the device information.

What does it look like when you do not have this vs when you have this?

This is handled in a different way


Oranisation Teams?

I just realized that we do not have teams in this repository. This however is sometimes diffucult if the decision process should be applied...
I think it would be good to have some like: website etc....

The teams are organised at the top level, therefore we do have teams here. You could just add a team for the website or for public relations.
https://github.com/orgs/oemof/teams

I like the idea, especially for the website we should have a team.
@simnh which teams do you suggest. Will you add them?

Yea, the website should have a team.

ok thanks!


TST: sparse.linalg: test all dtypes for lobpcg B-orthonormalization

Reference issue
Closes gh-19446
What does this implement/fix?
Make the test pass for all possible dtypes

The PR is finished and needs reviews and or merge please.


TechDocs: use the content of the readme as description for the techdocs-core package on pypi

Hey, I just made a Pull Request!
Currently, we have no description to our techdocs-core package on pypi, which we want. And we want to use the README in the techdocs-core package for that to avoid having to update a description in many places. According to these docs this PR should solve that.
:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Tricky to test without releasing a new version so let's see if it works. 👍

Tricky to test without releasing a new version so let's see if it works. 👍

haha yeah I was thinking about how to test but lets see...


bug: Text Overlap - Show Content Bar & Cohort 3.0 | Web Dev

Show Content Bar & Cohort 3.0 | Web Dev  Text Overlaps
Screenshot

Video
https://github.com/user-attachments/assets/81d772cb-6474-419c-8725-7da68df22114

hey can i pick this issue

Yes yes


B2: authorization token has expired

Hello,
Rclone 1.34, running on a raspberry pi 3 (with raspbian)
I'm currently in the progress of uploading a ton of data to B2, and unfortunately, issue #310  is still present.
I started rclone at [2016-11-21 19:48] (ah, don't we love logfiles)
After a bit over 24 hours, it started spouting these errors:
2016/11/22 19:59:52 data/5/50045: Failed to copy: Authorization token has expired (401 expired_auth_token)
2016/11/22 19:59:53

I think this is a duplicate of #825 which I've attempted to fix already - try the latest beta and see if that helps.
Please comment in #825 whether it works for you or not!
I'm going to close this one now.


Postgresql reader.js is outdated.

Note: for support questions, please join our Discord server

I'm submitting a ...
[ ] bug report
[ ] feature request
[x] question about the decisions made in the repository

Action taken (what you did)
Clone the repo, spin up my own server, use postgresql as my db.

Import data ok, issue happens when i try to scan data.

Expected result (what you hoped would happen)
Reader scans db.

Actual result (unexpected outcome)
Reader returns no result from db, even thou I have successfully imported data.

Because query.on is deprecated in pg v7, the reader will not work.
Read deprecation guide.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Please advise people to use pg v6 or update reader so people don't run into the same issue.
I want to make a PR but I have never worked with postgresql nor pg, thus help needed.

Thanks for bringing this issue up! Rewriting Postgresql support is actually pretty high on my list.
Gekko should have prompted you to install<EMAIL_ADDRESS>which should work relatively stable. That is the only thing I can recommand for now:
npm remove pg
npm install<EMAIL_ADDRESS>


pymycobot::get_angles may fail depending on current angle condition

Describe the bug
get_angles() may fail.
To Reproduce
changes angles using pymycobot API.
and then call get_angles().
It will empty array as result.
Desktop (please complete the following information):
Python 3.8.7
Additional context
I looked into this issue and found strange behavior as following.
When host acquires angles, host receives response from MyCobot contains servo's response and M5atom response.
Servo's response terms with check sum and it is 0xfe.
Response from M5stom starts with 0xfe, 0xfe two bytes.
At this time. DataProcessor::_process_received() makes mistake on finding header of response of M5atom.
I quick fixed by taking a look data in after header if it is same as 'genre'. If it is not same, it continues from next byte sequence.

Would you mind uploading your codes ? We tried send_angles() and get_angles() multiple times , didn't have this issue , so if you don't mind, we'd like to test it with your code , thanks ~

I am used to using GitHub so I attach changed code here.
Code I have changed is as below. It just DataProcessor::_process_received() only.
Changed lines is marked with command, "# CHANGED"
I also attach binary dump of transmitting data and comment following code. I hope it helps you to understand issue.
---- CODE:start ----
def _process_received(self, data, genre):
if not data:
return []
    data = bytearray(data)
    data_len = len(data)
    # Get valid header: 0xfe0xfe
    for idx in range(data_len - 1):
        if self._is_frame_header(data, idx):
            data_len = data[idx + 2] - 2
            if data_len > 0:
                cmd_id = data[idx + 3]   # CHANGED
                if cmd_id == genre:       # CHANGED
                    break                         # CHANGED
    else:
        return []

    data_len = data[idx + 2] - 2

    # compare send header and received header
    cmd_id = data[idx + 3]
    if cmd_id != genre:
        return []
    data_pos = idx + 4
    valid_data = data[data_pos : data_pos + data_len]

    # process valid data
    res = []
    if data_len == 12:
        for idx in range(0, len(valid_data), 2):
            one = valid_data[idx : idx + 2]
            res.append(self._decode_int16(one))
    elif data_len == 2:
        res.append(self._decode_int16(valid_data))
    else:
        res.append(self._decode_int8(valid_data))
    return res

---- CODE:end ----
Each line is divided by chunk of command or response.
Data start with servo related communication data and last chunk is response from M5atom, I thought.
Last servo response terms with check sum and it value is 0xfe.
Succeeding M5atom response start with 0xfe, 0xfe, header part.
Current code will take termination check sum code and first one byte of data as starting response of M5atom.
And pymycobot library will check command code at wrong data and fail parsing.

Well, I think you may repro more easier when you change angle of J6. I think last response is for J6 servo. Others may not cause this issue. If you have any question, please let me know.

In pymycobot , it will filler out communication messages between Basic and Atom , only read Get_angles()

When you get empty array, have you check Atom's status ? Is it 'yes' or 'no' ? and have you chosen  ' minirobot -> transponder ' ?

Maybe you can upload your code which you used to control mycobot , we'd like to test it with your code , thanks ~

I have not yet checked Atom's status. I am not sure which API is available to see Atom's status.  But even if this issue happened mycobot can be controlled from pymycobot API; changing angle of joints, focus on or off servos. I believe this issue is not problem of Atom or Base. It will be handling problem of received data in pymycobot or something.
I am using transponder and set various angles of all joints. Most cases it works fine; get_angles() returns list of angles of joints.
BTW, it is difficult to provide code controlling mycobot as it is, because it includes many functions for another project. I will try to arrange something to provide for this issue.

Are you using mycobot_280_M5 ? In transbonder, press button A , it will return Atom's status in Basic.

Hi, I face the same with MyCobot Pro 320 M5.
My code is based on mycobot_ros.
The debug branch is https://github.com/tongtybj/mycobot_ros/tree/interface_bug_mycobot320pro_m5.

myCobot 280pi returns empty array for get_angles().

I have also had this issue of receiving an empty array when calling get_angles(). I previous did not have this issue, but now I do and I'm not sure why.


AAC Support

I wanted to rip to AAC rather than MP3 or FLAC. I fiddled around with the CDrip.sh file, couldn't get it to work and finally realized you don't have the faac dependency installed. ripit already supports it. I think you just need the dependency in the Dockerfile. Any chance you could add that?

Sure thing

So unfortunately that didn't work, but I did a few hours of research and figured out why.  I tested a solution, that involves downloading the deb file here:
https://launchpad.net/ubuntu/bionic/amd64/faac/1.29.3+git20170724-1
and then just running
dpkg -i faac_1.29.3+git20170724-1_amd64.deb

After that, it just worked. I am going to attempt to open a PR. I'll get back to you on that.

So. A few hours later I got that to work but....turns out faac doesn’t actually output files Apple (Apple Music, QuickTime, et cetera)....so time for a new approach.
Turns out you can build compatible AAC files with ffmpeg which you are already compiling. For some reason the final bin was still in the /tmp/ffmpeg/bin directory so I moved it to /usr/bin in the install script.
Secondly, ripit incorrectly used the author metadata tag for the artist and Music wasn't recognizing that so I added a sed command to the install script to replace author with artist. It only occurs in the ffmpeg section so it shouldn't affect anywhere else in its code.
Third, for ffmpeg, I found out there's a better quality aac library for ffmpeg you can use rather than the built in one, but it needs to be installed as part of the compile. I wouldn't have tried it out except you were already compiling ffmpeg from source anyway so might as well give it a go. That's what I'm working on now. If its too much trouble and I can't get that part working I'll skip it but either way I should have PR up today.

Okay @rix1337 Got it all working. Take a look if you would. No rush. In the meantime I can use my fork.
https://github.com/rix1337/docker-ripper/pull/38


Share the checkpoint of the reward model and rl-tuned model?

Hi, I wonder if you would open-source the checkpoint of the reward model and rl-tuned model?:)

Hey, sorry, we currently don't have plans to open-source the checkpoints due to legal.


Closing LibGDX game deactivates Windows Night Mode

Issue details
I noticed that my Windows' Night Mode deactives after closing my LibGDX game the first time I open it up. After I reactivate Windows Night Mode again, it doesn't happen anymore.
The slider doesn't change to deactivated, but the orange hue change is just not showing up anymore.
Reproduction steps/code
It's a general issue, not related to any specific code.
Version of LibGDX and/or relevant dependencies
1.9.6
Stacktrace
None.
Please select the affected platforms

[ ] Android
[ ] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[X] Windows
[ ] Linux
[ ] MacOS

I have same problem, this is annoying

Yeah, exactly like you stated.
I just installed f.lux. It's way better and isn't having the same problem as Windows Night Mode.

Ok i rebooted and tested again first with lwjgl and once closed Night Mode stays, then i tested again with libGDX and once closed Night Mode stop to work and i get full brightness again..
So there is something wrong on libGDX part
I tested with https://github.com/LWJGL/lwjgl3-demos  wich uses latest version: 3.1.6-SNAPSHOT

I also Confirm it works with f.lux.
Shouldn't this be a windows bug though? (and if not, then something that affects the parent LWJGL project)

This also happens with Clippy and libgdx apps using the LWJGL 2 backend. I have a feeling it is a video driver issue and unrelated to libgdx, which I doubt messes around with the Windows gamma ramp.

I've also noticed this happening with Minecraft, which would support the idea that this is related to LWJGL.

Lwjgl issue, cannot fix at the libgdx side.


Boot diagnostics to set the storage in different region.

There is PowerShell cmdlet "Set-AzVMBootDiagnostic" to modifies boot diagnostics properties of a virtual machine. It allowed users to change the storage in different region, but the logs are not stored in the storage set.
According to Satyaki Chowdhury, it is a know issue. It is not recommended to have it that way because there will be issues when doing start, redeploy, delete etc. operations on the VM. Could you please add the note it is not recommended to use the storage in different region?

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

ID: 40c0296c-e069-f238-9c39-5d5faf47190d
Version Independent ID: 727c743d-2a48-c355-09dc-82ff1ba78358
Content: Boot diagnostics for VMs in Azure
Content Source: articles/virtual-machines/troubleshooting/boot-diagnostics.md
Service: virtual-machines
GitHub Login: @Deland-Han
Microsoft Alias: delhan

@v-hakob Thanks for providing the details through Feedback.
I have assigned the issue to content author to update the article with a note.

@Deland-Han
Can you please check and add your comments on this doc update request as applicable.

We are currently doing a cleanup of old issues so we can better prioritize and identify documentation bugs moving forward.
If this is still an issue, please open a new feedback item on the document and reference this one. This can be done by adding a # and the issue number. For example #55143.
This will inform us that the problem still exists and we will make sure to add it to the update backlog.
#please-close


This module just OCRs the whole screen

It just reads the whole screen and prints it as whole. How do I make it ocr only specific parts?
And where is the documentation?

I'm guessing you're referring to the behavior you get when running this module, i.e. "python -m screen_ocr". That's just a demo -- this package is intended as a library to be called by other Python code. It does support a bounding_box arg to restrict OCR to a portion of the screen:
https://github.com/wolfmanstout/screen-ocr/blob/ef2dcd10ff3df2d8742c7e1c3f839211de2f5fbb/screen_ocr/_screen_ocr.py#L220
The documentation is simply the code itself, which has docstrings. The file linked to above is the best starting point.

Ok


Genericizing ODConnection to allow for modification of parser, log level, interceptor, and endpoint

[ant:checkstyle] C:\Users\pnied\Documents\git\onedrive-explorer-android\onedriveaccess\src\main\java\com\microsoft\onedriveaccess\ODConnection.java:7:8: warning: Unused import - retrofit.converter.GsonConverter.
You can use gradlew check to see locally if anything doesn't follow the style of the project.

@peternied pull requested wrong SHA - look now

FYI - I just reverted this pull request, it looks like it broke the mainline app functionality

@peternied In what respect? I've not encountered any issues.

I was getting exceptions resolving the url from retrofit, looking into this as it might be that I should be checking for network connectivity...

Nevermind, I'm guessing it was wifi cutting out and completely unrelated to your change... I've reset back to your pull request


Revise use of proxy paths to use full DNS with subdomains

The application is currently managed between development and production using an option in repo vytools and a script called setup.sh to sed (edit replace) localhost out of several places to https://app.virtualyou.info . Replacing localhost with the app DNS, means that paths beyond http://localhost:3000 or https://app.virtualyou.info would be proxy. If the production basepaths are changed to the API DNS things should still work because the cookie domain is set to .virtualyou.info and this notation should allow any subdomain to be the origin because of the virtualyou.info part.
I'll not create an issue for vytools but I'll change the script and keep the current production sed part commented in the script.
The Dockerfile will need modification to be an NGINX server that contains an npm run build or /dist of the application. This image will be v0.2.0 after change of DNS for production, committal, and tagging  v0.2.0. Then we'll test and go from there.

AWS javascript SDK with React is causing all kinds of errors in the built index.js. I commented the use of mail.service.ts and the invite buttons now do nothing. There are other issues too that need to be addressed one by one. Closing this issue.


Add docs

Part II

Cool!  Notes added.  With those changes, I think this is ready to merge.

Done


[5.8] Bumped Laravel version to 5.8

This PR only bumps the versions on master to 5.8, since we now have a 5.7 branch that targets 5.7.x.
What this PR does not do is:

Bump the minimum symfony version (can be done later - probably ^4.1 -> ^4.2);
Alter the minimum required version of PHP (can be discussed later - probably ^7.1.3 -> ^7.2 or ^7.2.7 (7.2.7 is the version Ubuntu 18.04 LTS seemed to give me when I installed php with apt)).

you can use 3.8.* for testbench-core

Yes, good catch @crynobone!

@crynobone Does that dependency exist yet? If not, I'll hold off on updating that version until it does exist, so that travis can continue to pass.

https://packagist.org/packages/orchestra/testbench-core#dev-master
It should, just branch it few days ago.

Does this imply that the 5.7 branch is stable and will not get intentional bc-breaks?

Does this imply that the 5.7 branch is stable and will not get intentional bc-breaks?

Not really, since @taylorotwell created 5.7 branch and master is still aliasing 5.7-dev packages development might be getting dev-master instead of the proper 5.7 changes, which will definitely contain outdated changes (since all changes will first go to 5.7 before merging to master).
This help packages development plan and track BC on the 5.7 branch before the official release if they wanted to.


Move out community, learn, and design into their own repos

Based on the decision to focus on a poly-repo approach, I'll move out (during the focus phase) all things related to the community, learn, and design into their own repos.

Mostly done, so will close this now.


Set up Code climate

Code climate could be used to improve the maintainability and overall code quality.

735a4b0ec423945a079d18fd51800fe057e047bb


Dependecy issues of NetCDF and numpy with mixture of poetry and nox

As can be seen by the failing test from #2 here, there is a problem which probably stems from incompatible binaries of NetCDF and numpy.
If I go into to poetry env via poetry run python, I can successfully  do the following
import xarray as xr
xr.open_dataset('tests/test_data/openMRG_CML_180minutes.nc')

which is the same thing that is done where the test on CI fails. This also fails when test are run via nox -s tests locally.

E   RuntimeWarning: numpy.ndarray size changed, may indicate binary incompatibility. Expected 16 from C header, got 96 from PyObject

src/netCDF4/_netCDF4.pyx:1: RuntimeWarning

The problem seems to stem from an interplay of numpy and netCDF4. See https://github.com/pydata/xarray/issues/7259

Should be fixed with the hack in 41e238ab27e7dee577328d1c138ad8a81df410a6

I dropped Python3.12 support in #2
I close this issue, but at some point the hack with the import of netCDF4 should be reverted.


Use Razee to propagate production builds downstream

Not targeting this for present milestone.  @ralanlittle to have further discussions with other folks to see if this a path we want to go down.

@marikaj123 this is not in 19.17

No doing Razee for the time being.  Will reopen when we decide to do this again.


Simple lure attraction in move_to_fort worker

Add optional lure attraction in move_to_fort

@z4ppy please break down your lure attraction logic into many smaller methods.

@z4ppy need to solve merge conflicts with dev too, so I can merge.

I did some smaller method and is better like that, ty :)

I did some smaller method and is better like that, ty :)

How i can ?
The problem is contributor file

Merge done ! ^^

@z4ppy more merge conflicts, sorry :/

@z4ppy your merge is a bit messy, showing modifications that doesn't belong to your PR... but I really want your feature. Do you mind opening another PR?

I will redo :)


AUI-1811 Change text field's multiline option to radio buttons instead of a button switch

Just started reviewing :) :octocat: Sent from GH.

Just started reviewing :) :octocat: Sent from GH.

Just started reviewing :) :octocat: Sent from GH.

Pull request merged.


Replacing File Explorer with Files (Experimental)

I am unable to see this setting under experimental tab.
Files Version: <IP_ADDRESS>
OS Version: 10.0.22000.556

It's only available in the sideload version https://files.community/download/stable. Then in Files settings > experimental > set as default.


server side validation

Sorry, this isn't really an issue rather a question.  I haven't found any documentation on whether this library has hooks for server side validation.  e.g. pass in an array of errors that match the current form fields names.
Great library BTW!  Works really well and from what I have gone over so far, the code looks really clean.  Nice job.

For Server Side Validation, this is what I have Wiki - Remote Validation

Does that answer your question? If so can we close the issue?
Thanks for the comments by the way ;)

Sorry.  Just looked over your response.  I guess i need to read the documentation better.  Yes, i think that will work well.  Thanks again!

No worries, all good. Nice to know lot of people are using my library ;)


[BUG] ...Toolkit Blank on Open

Describe the bug
Toolkit page blank on open
To Reproduce
Steps to reproduce the behavior:

Go to 'toolkit reports menu button'
Click on 'toolkit reports menu botton'
page is blank

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. OSX, Windows]
Browser [e.g. chrome, firefox]
Toolkit Version [e.g. 2.22.0]

Setting Export
Please replace the text 'REPLACE_ME_WITH_SETTINGS' below with your exported Toolkit settings. You can export these by going to the Toolkit Options page, click the Import/Export Settings button at the top right and copying the text in the modal which appears.
Note! DO NOT replace the leading and trailing ``` characters as they are required formatting characters.
REPLACE_ME_WITH_SETTINGS

The toolkit is currently broken and this duplicates other issues. YNAB released updates to their code on or around the 15th which changed code for transactions which breaks the reports.

Another user reported this today in the YNAB fb group.   So, I went to check the Toolkit Reports in my YNAB.  Mine was fine, at first.  I checked several other "reports" which were all fine until I clicked on "Spending by Payee" report, then everything on the page went blank with no options to change to another report.

If you kept the page open for a long time, your browser does store a "copy" of the page so it dosen't have to reload it each time. When you started clicking reports and changed the view, the page reloaded and the code changed on the YNAB side so the toolkit has nothing to access.
Just have to wait for the fix to be published in the toolkit. I think it is there, just has to be packaged.
Sent from my T-Mobile 5G Device
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: conniew5347 @.>
Sent: Monday, August 21, 2023 11:06:41 AM
To: toolkit-for-ynab/toolkit-for-ynab @.>
Cc: Joshua Saxton @.>; Comment @.>
Subject: Re: [toolkit-for-ynab/toolkit-for-ynab] [BUG] ...Toolkit Blank on Open (Issue #3215)
Another user reported this today in the YNAB fb group. So, I went to check the Toolkit Reports in my YNAB. Mine was fine, at first. I checked several other "reports" which were all fine until I clicked on "Spending by Payee" report, then everything on the page went blank with no options to change to another report.
—
Reply to this email directly, view it on GitHubhttps://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/3215#issuecomment-1686508409, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABCFY6LH4BMMGE3CI4KJFO3XWN2QDANCNFSM6AAAAAA3VTEDHI.
You are receiving this because you commented.Message ID: @.***>

Okay thanks for the info.

From: Joshua Saxton @.>
Sent: Monday, August 21, 2023 8:11 AM
To: toolkit-for-ynab/toolkit-for-ynab @.>
Cc: conniew5347 @.>; Comment @.>
Subject: Re: [toolkit-for-ynab/toolkit-for-ynab] [BUG] ...Toolkit Blank on Open (Issue #3215)
If you kept the page open for a long time, your browser does store a "copy" of the page so it dosen't have to reload it each time. When you started clicking reports and changed the view, the page reloaded and the code changed on the YNAB side so the toolkit has nothing to access.
Just have to wait for the fix to be published in the toolkit. I think it is there, just has to be packaged.
Sent from my T-Mobile 5G Device
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: conniew5347 @.>
Sent: Monday, August 21, 2023 11:06:41 AM
To: toolkit-for-ynab/toolkit-for-ynab @.>
Cc: Joshua Saxton @.>; Comment @.>
Subject: Re: [toolkit-for-ynab/toolkit-for-ynab] [BUG] ...Toolkit Blank on Open (Issue #3215)
Another user reported this today in the YNAB fb group. So, I went to check the Toolkit Reports in my YNAB. Mine was fine, at first. I checked several other "reports" which were all fine until I clicked on "Spending by Payee" report, then everything on the page went blank with no options to change to another report.
—
Reply to this email directly, view it on GitHubhttps://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/3215#issuecomment-1686508409, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABCFY6LH4BMMGE3CI4KJFO3XWN2QDANCNFSM6AAAAAA3VTEDHI.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/3215#issuecomment-1686517973, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABH7NL7YDZHNWGDUNSY3XTTXWN3CDANCNFSM6AAAAAA3VTEDHI.
You are receiving this because you commented.Message ID: @.***>

Watching this thread as I am having the same issue after running through the reports and resetting my time windows.

Watching this thread as well. I wish YNAB would just have a more robust reporting tool.... especially on the app....

Also can confirm this bug
OS: Windows
Browser: Firefox
Toolkit Version v3.7.3

Still having this issue today.
OS: Mac
Browser: Chrome
Toolkit Version: v3.7.3

Hello all --- Josh built and packaged 3.8.0 today so expect it in the browser extensions in a couple days. It should fix the main issue with transactions, although who knows what YNAB will break on the 1st with their next release. Please be patient!


Enabled the share buttons on desktop version

added display block
:)

Haha, I wish it was that easy… The main question is how to integrate it in the layout.

How about enabling the share button in the post's page? Hide the comment
icon, and display the share button?
On Sat, Aug 1, 2015 at 10:41 AM, Sacha Greif<EMAIL_ADDRESS>wrote:

Haha, I wish it was that easy… The main question is how to integrate it in
the layout.
—
Reply to this email directly or view it on GitHub
https://github.com/TelescopeJS/Telescope/pull/1090#issuecomment-126853382
.

--
Regards,
Val Galin
Let's be friends! :)
t: @valgalin http://twitter.com/valgalin
g+: +ValGalin https://plus.google.com/+ValGalin

It's just like how it's implemented on Screenings.io, sharing buttons are
in the post's page. Thoughts?
On Sat, Aug 1, 2015 at 10:58 AM, Val Galin<EMAIL_ADDRESS>wrote:

How about enabling the share button in the post's page? Hide the comment
icon, and display the share button?
On Sat, Aug 1, 2015 at 10:41 AM, Sacha Greif<EMAIL_ADDRESS>wrote:

Haha, I wish it was that easy… The main question is how to integrate it
in the layout.
—
Reply to this email directly or view it on GitHub
https://github.com/TelescopeJS/Telescope/pull/1090#issuecomment-126853382
.

--
Regards,
Val Galin
Let's be friends! :)
t: @valgalin http://twitter.com/valgalin
g+: +ValGalin https://plus.google.com/+ValGalin

--
Regards,
Val Galin
Let's be friends! :)
t: @valgalin http://twitter.com/valgalin
g+: +ValGalin https://plus.google.com/+ValGalin

Yeah that could work in theory, but in practice both pages just include the same template. I wonder if it's worth making the logic more complex just for this…

Hi Sacha, That's what my first thinking: it's a hassle logically. But it
occur to me that I have successfully hidden the discuss icon on the Iris
theme's single post via CSS.
So I made a PR where I showed the share button on the post's page and hide
the discuss icon. Thoughts? :)
On Sat, Aug 1, 2015 at 11:34 AM, Sacha Greif<EMAIL_ADDRESS>wrote:

Yeah that could work in theory, but in practice both pages just include
the same template. I wonder if it's worth making the logic more complex
just for this…
—
Reply to this email directly or view it on GitHub
https://github.com/TelescopeJS/Telescope/pull/1090#issuecomment-126856909
.

--
Regards,
Val Galin
Let's be friends! :)
t: @valgalin http://twitter.com/valgalin
g+: +ValGalin https://plus.google.com/+ValGalin


feat: Matroid restrictions

We define the restriction of a matroid to a set, the resulting partial order on the set of matroids on a type, and give associated API.

bors r+

Bors r+


Add Habitat support

This makes it so you can scan habitat packages and their dependencies.
It should work, however, it's only going to have license_files if a
package contains a file called LICENSE (see habitat-sh/habitat#1738),
which most currently don't.
This is only a first step, and does not take into account "internal"
dependencies, like if a package is built for a Node or Ruby project.
This kind of stuff can be built into the build process for Habitat
packages.
Signed-off-by: Nathan L Smith<EMAIL_ADDRESS>

I need to fix some tests (stub out some files that I had locally but aren't there in Travis), but should be good once that's done.

LGTM when tests are passing

Code wise 👍 .
One thing to be cautious of is that when this code detects a component and sets the license_files for the component to [ ] some of the builds will fail (including automate) because they are set to fail on missing licensing information.
This can be mitigated by providing overrides, relative paths or public urls but we need to include some code like this to make use of this functionality:
https://github.com/chef/license_scout/blob/master/lib/license_scout/dependency_manager/berkshelf.rb#L70-L79
I would try an automate build with this before merging to make sure we don't break the build.

One thing to be cautious of is that when this code detects a component and sets the license_files for the component to [ ] some of the builds will fail (including automate) because they are set to fail on missing licensing information.

@sersut We aren't currently running license scout against habitat stuff in automate, so I don't think it will affect the existing build process.
We will want to get this taken care of before we do, but this addition is just to add initial support that can be built out more fully in the future.

@danielsdeleo @sersut updated with your suggested changes and working specs. Will merge once green if it's ok with you.


Skip config path tests when upgrading to the same revision

It will skip the test when the default test channel setting is to latest/stable;
It will run the test when the default test channel setting is to latest/edge;
It will skip the test when the default test channel setting is to latest/beta. Is this the expected behavior?
$ sudo ./run-all-tests-locally.sh -t test-refresh-config-paths.sh -v
testing snap from default test channel
running single test: test-refresh-config-paths.sh ...	PASSED
DEFAULT_TEST_CHANNEL not set. Setting to latest/beta
snap "edgexfoundry" is not installed
Installing snap from locally cached version
Installing /home/mengyi/Desktop/codereview/edgex-checkbox-provider/data/latest/edgexfoundry_3722.snap with channel
edgexfoundry 2.2.0+2 from Canonical** installed
Installed edgexfoundry  2.2.0+2  3722  -         canonical**  -
Getting the revision number for this channel: 3722
waiting for all services to come online. Current retry count: 1/300
waiting for all services to come online. Current retry count: 2/300
waiting for all services to come online. Current retry count: 3/300
waiting for all services to come online. Current retry count: 4/300
all services up
Refreshing edgexfoundry with channel latest/beta
edgexfoundry (beta) 2.2.0+2 from Canonical** refreshed
waiting for all services to come online. Current retry count: 1/300
all services up
Successfully upgraded:
	from: edgexfoundry  2.2.0+2  3722  -         canonical**  -
	to:   edgexfoundry  2.2.0+2  3722  latest/beta  canonical**  -
Upgraded to the same revision. Skipping test.
edgexfoundry removed

It will skip the test whenever the revision number is same. The relationship to channels depends on what is the current revision on different risk levels.


fix: force light content status bar

Description
for some reason kept reverting to dark text which looked not so good.
Screenshot(s)

Oops. It appears this is my master branch.


Allow customization of censor string for each censor element

I have a need to customize the censor string used for different JSON fields depending on the context. This is needed so that code that parses JSON data replayed from cassettes works properly. Consider, for example, a masked out UUID field. Deserialization of that field may require the value to be a valid UUID, but if it is replaced by a standard string like "*****", the deserialization process will fail ... but only when replaying data from the cassette. Generation of the cassette will work fine, which makes this problem somewhat hard to notice right away.
Further, the exact form of masking may need to be different depending on the context. So for example, you may want to mask a UUID field with a random UUID, but you may need an integer or floating point value to be masked by some fake data that suits that particular data type.

Something that might be even more versatile would be if you allowed the logic for the masking / censoring operation to be defined by a callback function / closure. If the callback was provided some useful contextual information like the key-value being parsed and a reference to where the context is in the JSON parse tree, you could do all sorts of fancy stuff. This would be overkill for my current needs, but might be worth thinking about.


Use case for AllowExplicitVersion

Partial Fixes for https://github.com/dotnet/sdk/issues/2879

I'm trying to rework this pull request based on @dsplaisted feedback, but am struggling with how to concisely phrase a recommendation.  Suffice to say, I agree with @dsplaisted : AllowExplicitVersion should never be encouraged in the documentation on implicit references, even as a workaround.

If it shouldn't be encouraged, why would we document it? If we're going to document it, then we should point out why it's discouraged.

@Thraka  I think AllowExplicitVersion is basically a "measure of last resort".  There's a separate discussion between @dsplaisted and I in https://github.com/dotnet/sdk/issues/3046#issuecomment-477797052 - he is still trying to understand my issue, so I think this PR is in a holding pattern until we can come up with a permanent solution.
So, while I agree with you documentation is the end goal, I'd rather get it right once than rush something out the door.

OK I've labeled this issue. :) Just follow up when you have the right wording.

@Thraka  I have sent @dsplaisted a full repro sln that I've uploaded to my GitHub, as part of https://github.com/dotnet/sdk/issues/3046#issuecomment-478624315 - this repro puts an interesting twist as it seems to indicate in some cases AllowExplicitVersion is absolutely required.  What I don't understand is why, and I am hoping Daniel can shed light on it, as in a more complicated solution, his advice to include the metapackage on Tests project works.

@jzabroski any update?

@dsplaisted  and I closed the related issue - I think that we don't have a good use case for AllowExplicitVersion, other than perhaps the same scenarios solved by TargetFrameworkVersion here: https://docs.microsoft.com/en-us/dotnet/core/versions/selection#self-contained-deployments-include-the-selected-runtime - except, instead of framework version, there might be a case where you need to load a specific version of a third-party library.  Either because a bugfix breaks your program (e.g., surfaces a race condition), or because you're trying to reduce the scope of regression testing (human resource constraints) in a mechanical way.

@jzabroski I closed this, you could rollback your delete and keep the content, but as it stands right now, the PR doesn't contain any changes.

@Thraka Sorry Andy, I was trying to think of where AllowExplicitVersion should go.  I think csproj.md is the right place, but was fumbling with where to put it.

I think it's a good place for it right now. We're going to rework this article eventually. I think that after this section would be good: https://docs.microsoft.com/en-us/dotnet/core/tools/csproj#implicit-version-for-some-package-references


[Fix #7338] Fix finding node as the base for indentation

…gnment
Replace this text with a summary of the changes in your PR.
The more detailed you are, the better.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I've updated the PR. Could you please check it?

@bbatsov @buehmann Guys, what do you think about the PR?

@AlexWayfer @bbatsov @buehmann
Hi there! The PR is very old. So I have updated it. If the issue is still actual would we continue work with it?

Hi there! The PR is very old. So I have updated it.

Thank you.

If the issue is still actual would we continue work with it?

The issue is still open, so I guess yes, it's actual. The best way to be sure is adding auto-tests for your changes.

@AlexWayfer Thank you for your response!
I missed it due to rebasing. I will fix it a bit later

@Neodelf @AlexWayfer any motion on this? I'd love to see this fixed!

I'm closing this PR due to no recent activity. Feel free to re-open it if you ever come back to it.


test csaes to dynamically decide union type

im not super confident about this one, but this would be great feature to not provide _union_type at just go through and match any it can. Otherwise it feels like will be cumbersome to manually preselect type (one that first one matches).
Question also is what error should be returned if no union type subtype does  match, maybe all union type errors joined with OR. :thinking:

Yeah, it's hard to say on this one... 🤔 because if your UI does different things depending on what is selected, and a value can apply to multiple...I guess in those cases you'd just ensure that it was always explicit 🤔
I'll think about this for a bit more.


Windows installer download link broken

The link on this page to the Windows installer is broken: http://phantomjs.org/download.html:
Thanks for your work!

What do you mean?

Works for me as well.


[Map] Map doesn`t correspond to mock-up

Environment: Android 10, Google Pixel 4.
Reproducible: always.
Build found: v1.0.437.
Preconditions
The user is logged to the 'Car' application and map is opened.
Steps to reproduce
Go to the 'Find a Ride' and press 'From'.
Actual result
Map doesn't correspond to the mock-up
Expected result
Map should be a lighter and correspond to the mock-up.
Labels to be added
"Bug" ("bug"), Priority ("pri:low ")

Verified


Some emojis in text config and in top bar result are different

I set up my clock text as follow : 🕓 %H:%M %t 🗓 %A %e %B %Y %t 🦕 Semaine %V. This has produced the expected result until today, when it started to look as follow :

(see how the two last emojis do not match what I have set up)

As of today, the issue seems to have gone :raised_hands:


Enable namespacing in bsconfig

Would it make sense to add "namespace": true into the template?
https://bucklescript.github.io/docs/en/build-configuration.html#name-namespace

I don't think reason-scripts apps should use namespace, it's mostly just for libraries.

IMHO adding namespace wouldn't hurt any apps, and if someone is using it for a library it will only helps

It's currently really difficult to make a library out of an app created from reason-scripts, so idk how relevant  it is. If anything adding that option might be misleading.


Cygwin: slow or reordered letters as I type

Darn, sorry about this, but there is some regression with this commit. It is that typing got a lot slower in a better case. In worse cases - as I type - I see some typed characters being reordered or duplicated. They stay as such until I switch buffers.
I think it's got to do with the fact that calling an external command makes the function slower. If that's the case than the problem was probably there even with the previous cygwin-path-change implementation in this code path (as it also calls an external command).
I currently have a task in my work where I need to use Windows, so I can try to look into it, but would need some guidance or suggestions on what to try.

That's OK, these things happens when evolving software. I suspect that the same few files are send though util.LspFileToUri many many times, it might be worth adding a naive caching around the external commands, something like:
var resolvedUris = {}
export def LspFileToUri(fname: string): string
  if resolvedUris->has_key(fname)
    return resolvedUris[fname]
  endif

  ...
  ...

  resolvedUris[fname] = uri
  return uri

This helped, thanks. Here is the PR, let me know if you'd like to include something in it.

The PR #328 is merged.


added basic GNPyWrapper.jl functionalities

added src files, test files and readme

added new features to GNPyWrapper.jl


Allow kube-proxy flag for commands

The kubernetes client config for .net does not yet support the gcloud authenticator.
Therefore, the support for "kube-proxy" would be nice to use the install / uninstall / run commands

Well it seems that the gcloud authenticator is supported. Closing this for now until it is a real feature request.
/close