Add a filler 'mode' option

Option to specify whether to fill above or below the dataset

@charlesmass can you please update the description with the link to the associated ticket discussing that feature, details about how it works (options), screenshots and a jsfiddle to see it in action?

@charlesmass sounds good, but can you explain a bit more what this feature is about with some screenshots?

PR has been rebased and should now work fine on supported browsers.
Usage:
options = {   plugins: {     filler: {       mode: 'above'     }   } }

@charlesmass it looks like several tests are failing with this PR

@benmccann Thanks for pointing that out, I just committed new changes

@kurkle Thanks, 100% agree with your comments.
Will wait for another review and apply the changes you suggested

Thanks for the help. I'm closing this one since I've upgraded it a bit.
#6318

Hello All,
I have a line chart
And I want to change the fill  to be always directed to bottom and not to the top for the negative values.
Maybe the issue here is that The Origin is the 0 ...
I am lost with the current documentation and some closed issues are not helping,
when i set mode : 'above' or mode: 'all' nothing happens


feat(es/minifier): Inline more with sequential inliner

Description:
I wrote descriptions in the code with comments.
Related issue:

This was a wrong approach


:debug on root of a LiveComponent shows warning

Adding a :debug directive to the root element of a LiveComponent shows the following warning:
warning: stateful live components must have a HTML root element

Example:
  defmodule Counter do
    use Surface.LiveComponent

    def render(assigns) do
      ~H"""

        ...

      """
    end
  end

The way that debug currently works on html tags when :code is specified option (i.e. the default) is essentially by wrapping the node in an AST.If with the condition set to true. This was a work around for the limitation created by un-nesting and combining all of the static portions of the template when feeding it into the LiveView EExEngine. While it makes :debug work for html nodes, it also introduces this warning (and slightly changes the generated code). Ideally we also would not modify the code to make :debug work properly.
A workaround for this warning could be to detect when the root element has :debug specified, and special case that; however, I'd really like to find a better way to handle :debug for html nodes that doesn't require wrapping the contents with other nodes not specified by the user.

@lnr0626 would it make sense to have a new AST.Debug (or something similar) and use it instead of an AST.If? This way the EExEngine could print the code as it is without the need to modify it.

i tried using the AST.Container to accomplish something similar - the issue I ran into is that the nodes go through https://github.com/msaraiva/surface/blob/master/lib/surface/compiler/eex_engine.ex#L36-L41 before being printed, which combines the static portions of the html tags.
One possibility would be to check if :debug had the appropriate option set in https://github.com/msaraiva/surface/blob/master/lib/surface/compiler/eex_engine.ex#L565 and https://github.com/msaraiva/surface/blob/master/lib/surface/compiler/eex_engine.ex#L549; and if so perform handle that inside those methods. The downside is we'd have to do the transformation on html tags multiple times, but the upside is that it would be accurately printing out the code for that portion of the tree.
As an example, that might look like:
  defp to_dynamic_nested_html([
         %AST.Tag{
           element: element,
           attributes: attributes,
           children: children,
           debug: debug_opts
         } = node
         | nodes
       ]) do
    content = [
      "<",
      element,
      to_html_attributes(attributes),
      ">",
      to_dynamic_nested_html(children),
      "</",
      element,
      ">"
    ]

    if Enum.member?(:code, debug_opts) do
      contents
      |> List.flatten()
      |> combine_static_portions()
      |> generate_buffer(...)
      |> maybe_print_expression(node)
    end

    [content, to_dynamic_nested_html(nodes)]
  end

The downside is we'd have to do the transformation on html tags multiple times

Uhm. That could hurt performance, doesn't it? If it does, unless you can see another option, I think we should just leave it as it is for now. I mean, there's are more important issues to address and since this is a non-blocking-dev-debug-only issue, we can address it some other time. Up to you :)

yeah - it'd potentially hurt performance pretty badly if a user had a large tree underneath something they marked with :debug, or even worse used :debug on a large series of nested tags (i.e.

...

)

Well, the performance problem will happen only while debugging (which is kind of common in debugging in general), so I'd say we should do it. Then we can try to improve it in the future in case it becomes a real issue.


[Themes] Improve storefront password protection redirect handling

WHY are these changes introduced?
https://github.com/Shopify/cli/issues/4737
Fixes a bug in the storefront password protection detection logic where redirects on the provided URL (e.g. .com -> .se) cause us to detect the storefront as password protected.
WHAT is this pull request doing?

Specifically check for redirects to the /password path, rather than assuming any 302 redirect indicates password protection
Improving debug output for password protection status

How to test your changes?
Replicate

Disable password protection on your store
Add a domain to your store, and set that as your default domain
Provide the original (now secondary) domain in the --store flag value
Run pnpm shopify theme dev with this branch and verify the appropriate error message is shown

Measuring impact

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes

/snapit

@karreiro Checked with Theme Access and this is seemingly unrelated as that is related to the other steps here in fetchDevServerSession


Blog feed: add a first-class API to limit size of the RSS / Atom feeds

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
Add an option: feed.limit: number | false | null = 20
Don't put all the blog posts in the feed by default

20 = default value
false | null = no limit

See also my comments here:
https://github.com/facebook/docusaurus/pull/8378#issuecomment-1651277331
Has this been requested on Canny?
No response
Motivation
The RSS/Atom feeds files are too big for sites with large blogs, notably because the feeds contain the blog content.
It's not super useful to have everything inside the feed, and having the last 20 blog posts by default is good enough.
See https://github.com/facebook/docusaurus/pull/8378#issuecomment-1651277331
API design
No response
Have you tried building it?
No response
Self-service

[ ] I'd be willing to contribute this feature to Docusaurus myself.

Note @johnnyreilly would like to work on this soon (https://github.com/facebook/docusaurus/pull/8378#issuecomment-1653493526). If in 2 or 3 weeks there's no PR open feel free to reclaim this issue and submit a PR :)

I've had a crack at it :smile: https://github.com/facebook/docusaurus/pull/9189


Add E2E tests with Plutip+CIP30Mock to CI

Find out how to bundle and serve the examples in CI (normally done with make run-dev from nix shell on developers' machines). We should run the server in background and kill when the tests end
create a separate test suite file (like test/e2e.env) that only includes Plutip+Cip30Mock tests
Run the test suite in a separate nix check.

We probably want to make chromium required in default.nix at this point to actually use it in Hydra

Is this a duplicate of https://github.com/Plutonomicon/cardano-transaction-lib/issues/1205?


ike-scan: init at 1.9.4

Motivation for this change
ike-scan is a command-line tool that uses the IKE protocol to discover,
fingerprint and test IPsec VPN servers.
Related to #81418
Things done

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

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

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

@ofborg eval

Result of nixpkgs-review pr 107314 run on x86_64-linux 1

1 package built:

ike-scan

Result of nixpkgs-review pr 107314 run on x86_64-linux 1

1 package built:

ike-scan

Result of nixpkgs-review pr 107314 run on x86_64-linux 1

1 package built:

ike-scan

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/449

This pull request has been mentioned on NixOS Discourse.


Upgrade spring boot to 2.3.1 & Spring cloud to Hoxton.SR6 & Gradle

upgrade gradle version
update spring cloud version

I had already changed the Gradle and Boot versions, so I put your other changes (the Hoxton version and the code comment updates) in another PR and merged that. I gave you credit in the commit message. I'm closing this, but I want to make sure to give you credit. Thanks much.


Export unblinding methods

This exports all the methods for unblinding an output of the explorer module.
This closes #19.
Please @tiero @louisinger review this.

ACK (even if I don't like the way I did two functions to unblind Utxos.... we could create an issue to merge them @altafan ?)

After a quick look I didn't understand the purpose/usage of tryToUnblindUtxo

tryToUnblindUtxo was needed to have non blocking coinselection AFAIR. Would I go forward merge&publish?


Fix error messages in dotnet-sql-cache tool

@moozzyk
Sample error messages:

An error occurred. Invalid Sql server connection string 'a'. Format of the initialization string does not conform to specification starting at index 0.

An error occurred while trying to create the table and index. The specified schema name "foo" either does not exist or you do not have permission to use it.

:up: :date:

:up: :date:

:ship: :it:


Factorial and sin-approximation with taylor series added

Closing this PR as most code except for sine approximation are duplicates and the PR is quite outdated.
@LamdaLamdaLamda feel free to reopen this PR with updated codes per the repo contributing guidelines


Add index.d.ts file and module resolve setting

First of all, thanks for a nice package.
The reason for this PR is that the following error has occurred:
Could not find a declaration file for module 'zoov'. '$PROJECT_PATH/node_modules/.store/zoov-virtual-300f1217b1/node_modules/zoov/index.js' implicitly has an 'any' type.
  There are types at '$PROJECT_PATH/node_modules/zoov/types/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'zoov' library may need to update its package.json or typings.ts(7016)

There is no file index.d.ts.
https://www.npmjs.com/package/zoov?activeTab=code

This patch is as follows:

Add index.d.ts
Support typescript5's moduleResolutionto bundler https://github.com/microsoft/TypeScript/issues/52363
If not, the following error may occur: https://github.com/nhn/tui.calendar/issues/1398

Thanks for your contribution! I will publish a new version.

0.5.5 published, thank you again

I think this API is comfortable.
const [{ state }, { action }, { selector }] = useStore();
const [state, { action }, { selector }] = useStore();

Thank you for a good library.


⚠ Bofrost.lu has degraded performance

In eb71702, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1953 ms

Resolved: Bofrost.lu performance has improved in 6182b74 after 36 minutes.


Default download group

需求 / Requirement
Can you please add a feature that lets you choose which download group the downloaded galleries will go to instead of having to select the download group for every gallery download
原因 / Reason
Having to choose download group for every gallery wastes too much time
建议实现 / Suggested implemention
Add the option in the setting > download section it can be changed easily
Can work similar to "enable default favourite" option
备注 / Addition details
No response
自查步骤 / Verify steps

[X] 您已仔细查看并知情 常见问题 中的内容 / You have checked Q&A carefully
[X] 您已搜索过 Issue Tracker，没有找到类似内容 / I have searched on Issue Tracker, No duplicate or related open issue has been found
[X] 您确保这个 Issue 只提及一个问题。如果您有多个功能请求，烦请发起多个 Issue / Ensure there is only one feature request in this issue. Please make mutiply issue for mutiply bugs
[X] 您确保已使用最新版本测试，并且该问题在最新版本中仍未实现 / This feature have not supported in latest version

Will be supported in next version.

Supported in https://github.com/jiangtian616/JHenTai/commit/04275bcd6f5268814f3748aa3a02ceaf2f50e6d6, will be released in next version.


Nmap suggester

Hi as a user I wanted threader3000 to print out a formated nmap scan. This is great as now once threader3000 has chewed through the entire port range I get a suggested nmap scan tailored to the box.

Sample output:
------------------------------------------------------------
        Threader 3000 - Multi-threaded Port Scanner
                       Version 1.0.2
                   A project by The Mayor
------------------------------------------------------------
------------------------------------------------------------
Scanning target <IP_ADDRESS>
Time started: 2020-07-22 10:54:43.493182
------------------------------------------------------------
Port 80 is open
Port 443 is open
Port scan completed in 0:01:40.061697
------------------------------------------------------------
suggested nmap:
nmap -p80,443 -sV -sC -o scan.nmap yahoo.com
Press Enter to exit...

Thank you for the suggestions.  I have a few more to add before the next push to Pypi that will include some automation based on your inclusion of the nmap coverage at the end.  I truly appreciate your effort in improving my tool.


Should withArgs work?

Hi. I'm trying to set expectations on my stubbed promises:
  describe.only('test withargs', function() {
    var stub;

    before(function() {
      stub = sinon.stub().withArgs(123).resolves('xyz');
    });

    it('should resolve when passed 123', function() {
      stub(123).should.eventually.equal('xyz');
    });

    it('should not resolve when not passed 123', function() {
      stub(456).should.be.rejected;
    });
  });

However, the second assertion is not rejected.
Unhandled rejection AssertionError: expected promise to be rejected but it was fulfilled with 'xyz'

Should withArgs work?

It has nothing to do w/ withArgs. If you don't return the promise in the second test it's rejected but you never handle it.

Thanks for the quick reply. I'm struggling a bit to work out how to set expectations.
Another way of looking at it:

  describe.only('test withargs', function() {
    var stub;

    before(function() {
      stub = sinon.stub().withArgs(123).resolves('xyz');
    });

    it('should resolve when passed 123', function() {
      stub(123)
      .then(function(result) {
        result.should.equal('xyz');
      });
    });

    it('should not resolve when not passed 123', function() {
      stub(456)
      .then(function(result) {
        result.should.equal('xyz');
      });
    });
  });

I'd expect the second test to fail, but it succeeds.
Sorry for continuing to comment on a closed issue. I'd just like to know if I should expect withArgs to work?
Thanks again.

Return in the first line of each it callback. You're more than welcome to comment on closed issues. It's closed because it's not a not a bug or feature request and that's what I leave issues open for.

Sorry, my idiocy. Thanks.
  describe.only('test withargs', function() {
    var stub;

    before(function() {
      stub = sinon.stub(); // Don't chain, it seems
      stub.withArgs(123).resolves('xyz');
      stub.withArgs(456).rejects();
    });

    it('should resolve when passed 123', function() {
      return stub(123).should.eventually.equal('xyz');
    });

    it('should not resolve when not passed 123', function() {
      return stub(456).should.be.rejected;
    });
  });

:+1: !


Fix table alignments in responsive mode

Also converting to a more sass format

Thanks for this @van-nguyen

I have just merged this into the update-typography branch, thanks for these changes @van-nguyen I will be merging the update-typography branch into master in a few days


wordpress bump dependency mariadb 0.5.9

Closing this failed attempt to script out a version bump. Closing and will re-open for real.


very-verbose fix

I tried to tell you that the PR was not in a good state to merge in the comments, and now master is broken. This PR should fix the break that was just introduced.

ooops i totally misunderstood!

fyi i reverted the other pr for the time being

ok, going with the alternative plan :upside_down_face:
i merged the -w flag!


Samples queens8 does not terminate when it should

tpl samples/queens8 --goal=test
fails to terminate upon producing all solutions.

If I load queens8.pro into REPL and query for rangeList/3, no result is shown:
% tpl queens8.pro
Trealla v0.1alpha, 64-bits (int=128-bits), Dec 31 2016
INFO: consulted 'queens8.pro': 238 terms
?- rangeList(1, 8, Ns).

(0.000) false.
?-

Extract rangeList/3 and create another test case:
rangeList(M,N,[M]) :- M >= N, !.
rangeList(M,N,[M|Tail]) :- M1 is M+1, rangeList(M1,N,Tail).

test :-
  rangeList(1, 8, Ns), write(Ns), nl.

test will genearte infinite results, e.g.:
Trealla v0.1alpha, 64-bits (int=128-bits), Jan 27 2017
?- [foo].
INFO: consulted 'foo': 60 terms

(0.000) true.
?- test.
[1,2,3,4,5,6,7,8]
(abort=a,trace=t,next=;):
[1,2,3,4,5,6,7,8,9]
(abort=a,trace=t,next=;):
[1,2,3,4,5,6,7,8,9,10]
...

In GProlog, SWI and Yap, there is only one result.

It seems that ! did not cut the choice point:
...
###[ follow ] ==> size=2, choice_point=19, curr_choice=18, env_point=32
Call: 8>=8
Exit: 8>=8
###[ follow ] ==> size=2, choice_point=19, curr_choice=18, env_point=32
Call: !
###[ trust_me ] ==> size=2, choice_point=19, curr_choice=18, env_point=32
Exit: !
###[ follow ] ==> size=2, choice_point=19, curr_choice=18, env_point=32
###[ proceed ] ==> size=2, choice_point=19, curr_choice=18, env_point=32
...

If optimize(inline) is enabled, choice point is cutted:
...
###[ follow ] ==> size=2, choice_point=9, curr_choice=8, env_point=32
Call: 8>=8
Exit: 8>=8
###[ follow ] ==> size=2, choice_point=9, curr_choice=8, env_point=32
Call: !
###[ trust_me ] ==> size=2, choice_point=9, curr_choice=8, env_point=32
Exit: !
###[ follow ] ==> size=2, choice_point=8, curr_choice=8, env_point=32
###[ proceed ] ==> size=2, choice_point=8, curr_choice=8, env_point=32
...

Thanks. I hope to look at it this week.

In bifs_iso.c/bif_iso_and() if you comment out the allocate & try_me then queens8 works fine with/without optimize(inline). Now that is how it originally was, and how I believe it should be, optimally. Conjunction shouldn't need to create a choice-point.
Problem is with issue#3 then. So maybe I need to go back and look at that.

After applying your suggested change, a test case in #54 is also affected: halt will not be executed then.
:- initialization(main).

main :- write(a), nl, (write(b), nl), halt.

Listing seems to be correct:
':-'(main,','(write(a),','(nl,','(','(write(b),nl),halt)))).

In Issue #3 if I change
+ (upto(3, I), upto(I, J), + (write([I, J]), nl)), halt.
to
+ (upto(3, I), upto(I, J), + call((write([I, J]), nl))), halt.
it works. The same solution applied to #54-2 above works.
So I feel removing the allocate & try_me is the way to go and the optimal solution is close...

One problem mentioned above still exists, "load queens8.pro into REPL and query for rangeList/3" has no result.


Client-side encryption for Azure job store (resolves #308)

Currently it uses only one key. Having a separate key for each file is certainly possible, but it involves some key distribution which would be slightly trickier.
Only the path to the key file is ever stored in the jobStore, because I was a bit paranoid about stuff getting pickled and passed around so often. If the job store never, ever gets pickled, maybe we could save a bit of I/O and read it in at job store creation.
This client side encryption should work fine for the file job store too.

Can one of the admins verify this patch?

Jenkins, test this please.

Hmm, looks like libffi headers are missing. I'm pretty sure the library itself is installed, since the ctypes package uses it, but Ubuntu/Debian often doesn't include headers. Maybe the libffi-dev package is required.

Also, this PR should have been set to failed status. Looks like the Jenkins user needs write perms to the repo for that to work. I'll fix that.

I'll add libffi-dev to toil-jenkins-slave.
Also: #371

https://github.com/BD2KGenomics/cgcloud/commit/a1ed642fbe34fa12c59a232fc435649525814be4

Jenkins, test this please.

Build was killed after 30min. We were already at 25 min so maybe the additional Azure test put it over the limit. I increased the timeout to 45.

Jenkins, test this please.

Paranoia is good. For the AWS job store, DEBUG log level will leak the SSE-C key since at that level boto logs all HTTP request headers and the key is passed as a header.
We could wrap the key in a Secret class whose str and repr methods return "******" or something and that refuses to be pickled via copy_reg. To get at the key string from a Secret you'd have to call a specifc method. Just brainstorming here.

Yeah, I think something unpickleable would be a great solution. Maybe something similar could be used for the AWS/Azure access credentials.  There are bots that scrape for AWS keys and run up your bill with bitcoin miners.

Hmm, looks like testToilSortOnAWS hangs?

That's weird. Surely only the few option parsing lines I added could affect AWS, and they seem OK to me.

Jenkins, test this please.

OK, it failed yet again, but in a different place. Let me investigate and try to reproduce. Maybe there is a subtle merge conflict.

Maybe the file handle leaks are causing the test failures?

I checked with 100000 tmp files a few hours ago, and python seemed to clean up the fds OK, but can't hurt to do it in the pythonic way. Updated it with a force push--not sure what the protocol is for when to/not to force push.
I think the gridengine test hanging is a fluke--it has only 1 second to find the job--but still can't figure out the AWS test issue, which is obviously not a fluke.

Jenkins, test this please.

I can replicate the hang on the AWS sort tests on my laptop, even on master branch, but only when the test is run along with all the others with make test, not when it's run with make test tests=src/toil/test/sort/sortTest.py::SortTest::testToilSortOnAWS.
Looks like it might be a deadlock in the upload or download stream methods,
since one end is blocking on a futex and the other is blocking on a read from a pipe:
Process 2556 attached with 2 threads
[pid  2579] futex(0xf156f0, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
[pid  2556] read(45,

I might try to get the python gdb extensions to work so I can get a backtrace. But they've never worked correctly for me.

I was surprised to find out that Jenkins builds not commit at the tip of the issues/308-azure-encryption branch, but a merge commit from the tip of that branch into master. I manually ran make test against both commits on the jenkins slave and both succeeded.

Jenkins, test this please.

In the past, Pyrasite worked for me:
http://stackoverflow.com/questions/132058/showing-the-stack-trace-from-a-running-python-application#answer-29881630
You may want to run the tests without capturing of stderr and out:
make test tests="-s src"

I got gdb to work finally (just need to run with the python-dbg executable rather than python2.7).
Interestingly my backtraces for my AWS sort hang (not necessarily the same as the one on jenkins!) show a deadlock when reading the environment.pickle file in worker.py.
I have a hunch that reading the environment.pickle straight into the cPickle.load method may be a race. I'm not totally sure, but I think the GIL cannot ever be released when inside a C extension, and cPickle.load is definitely written in C. So if there is not already data written into the other end of the pipe, the worker will completely deadlock.

OK, added my fix. Seems to work (n=3). I have no clue why the encryption code would trigger this race, but it is definitely a race.

It would be a bug for cPickle not to release the GIL before doing IO and it looks like the latest version does just that:
https://github.com/python-git/python/blob/master/Modules/cPickle.c#L555

This is where mine was stalled:
https://github.com/python-git/python/blob/master/Modules/cPickle.c#L594
That getc can definitely block. It seems like a bug. Maybe it's relying on line-buffering?

The link is outdated. Trying to find the right one.

I had a hard time finding the official repo too. But here it is (2.7 branch):
https://github.com/python/cpython/blob/2.7/Modules/cPickle.c#L591
Seems like the getc is still used without releasing the GIL (but I can't be 100% sure). Empirically, the patch seems to stop the AWS test from hanging for me, but maybe it's a coincidence.

Jenkins, test this please.

Don't know why I have to keep nudging Jenkins to build. I added you to the whitelist explicitly and it should build every PR from you automatically at least from now on.

OK, thanks. Wasn't sure if I had to say the magic words or what. I just logged into your Jenkins instance via Github OAuth. Maybe that was why it wouldn't automatically build.

There may be other places where we call cPickle.load() passing a pipe handle whose other end is written to by the AWS job store.

I found 2 more:
https://github.com/BD2KGenomics/toil/blob/d6e89c9bdf376561366c02b56a7de472f4094cfb/src/toil/job.py#L1120
https://github.com/BD2KGenomics/toil/blob/d6e89c9bdf376561366c02b56a7de472f4094cfb/src/toil/jobStores/abstractJobStore.py#L57

serialiseEnvironment() in common.py does not specify a protocol when calling cPickle.dump():
def serialiseEnvironment(jobStore):
    """
    Puts the environment in a globally accessible pickle file.
    """
    #Dump out the environment of this process in the environment pickle file.
    with jobStore.writeSharedFileStream("environment.pickle") as fileHandle:
        cPickle.dump(os.environ, fileHandle)
    logger.info("Written the environment for the jobs to the environment file")

According to https://docs.python.org/2/library/pickle.html#data-stream-format it uses the text-based protocol 0 in that case. Maybe that's why it's using the readline function that ultimately uses getc?

Yep, looks like that's the reason.  The load_binint, load_binstring, etc. functions all use the fread code (which correctly releases the GIL), but the load_int, load_string, etc. functions all use the getc code.

It's already past the tests were it got stuck in previous builds.

Different, probably unrelated failure.

Doh, looks like I will have to retry on that particular exception. I've never seen that before. Want me to fix it in this PR, or a different one?
But--phew! At least the mystery is solved. That was a tough one. Thanks for the help.

Excellent debugging! What do you think about retracting the workaround from b641a5a and instead making sure dump(...,HIGHEST_PROTOCOL) is always used?

Sure, I think that makes sense. Force pushed that change instead of the other one.
We should also be a little paranoid about this if we eventually migrate pickling to dill.

Blah, the gridengine test failed again, but I really think that's unrelated.

Jenkins, test this please.

Aha, the curse has been lifted.


experiment_id: html table rendering

Indicate the Controlled Vocabulary (CV) of interest: experiment_id  (html rendition)
Is it o.k. for me to upload Charles' script (to produce html table from CMIP6_experiment_id.json) into "src" and the CMIP6_experiment_id.html file into CMIP6_CVs?

@taylor13 already there: src/json_to_html.py if he has tweaked it then it'll need an update..


The spinner is not centered in iPhone 6 or 6+.

The spinner is not centered in iPhone 6 or 6+. Works fine on iPhone 5s or below.

fixed

Thank you!

@ziryanov I upgraded to 0.5 (by running pod update to the latest version), and it actually started to right-align the spinner on my iPhone 6. I wasn't seeing this before, and I downgraded back to 0.4.1, and that resolved my issue.
Thoughts?

@mergesort Try version 0.5.1 please, I guess I finally fixed it.

@ziryanov Yep, all good, thanks!


Anchor Tags

Need to add anchor tags for index project page and link to portfolio page.

Fixed with <a href="https://ait530.github.io/projectsaction/#mlbt">


add darkMode facade

@danharrin personally i think feature flags toggled by methods is easier to configure as no config files require to be published.
regarding your concerns mentioned on discord:

other features are not available in methods.
i think more can be added if there are requests

not something usually controlled programatically
yes and no. taking dark mode for example, can store in DB to persist the preference across devices.
but still its just more of convenience

anyway feel free to close it if you don't see a need

Storing this setting in the DB will not persist it. Enabling dark mode like this just lets the user decide between light & dark :)
As with other static config options, lets keep this in the config file.


python38Packages.fluent-logger: 0.9.6 -> 0.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.fluent-logger is: "A structured logger for Fluentd (Python)"
meta.homepage for python38Packages.fluent-logger is: "https://github.com/fluent/fluent-logger-python"
meta.changelog for python38Packages.fluent-logger is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.10.0 with grep in /nix/store/427g5y1ybm7i2jfps8ca8rqbhpnz2s8l-python3.8-fluent-logger-0.10.0

directory tree listing: https://gist.github.com/e8e6f90582b3e24d91db0f49979f4804

du listing: https://gist.github.com/ad6f7ad5a7dd96a56fd5381363aee3e2

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

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.fluent-logger
python39Packages.fluent-logger

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/427g5y1ybm7i2jfps8ca8rqbhpnz2s8l-python3.8-fluent-logger-0.10.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/427g5y1ybm7i2jfps8ca8rqbhpnz2s8l-python3.8-fluent-logger-0.10.0
ls -la /nix/store/427g5y1ybm7i2jfps8ca8rqbhpnz2s8l-python3.8-fluent-logger-0.10.0/bin

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

2 packages built:

python38Packages.fluent-logger
python39Packages.fluent-logger

Maintainer pings

Result of nixpkgs-review pr 116120 at 4839412e run on aarch64-linux 1

2 packages built successfully:

python38Packages.fluent-logger
python39Packages.fluent-logger

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/fluent-logger/default.nix:28:3:
   |
28 |   meta = with lib; {
   |   ^


ZEVA - Delete rejected/returned ZEV Model

*Title:
Enable BCeID \ ZEVA User to delete rejected or returned ZEV models so they can keep their list of ZEV manageable. Deleted ZEV are still visible to IDIR users.
Description:
If a ZEV model is rejected by government I can't do anything further with this record. I have to submit a new ZEV model if I need to change anything. Once I do that there is no purpose to keep the rejected record so I'd like to delete it if I so choose.
If a ZEV model is returned by government as Changes Requested status I'd like the option to delete it if I so choose.
Add a Delete button to the ZEV model screen when in Rejected or Changes Requested statuses as per wireframe.
Wireframe:
https://preview.uxpin.com/4946124c44da522837a3b54ccb7b5243fa384039#/pages/131174652?mode=fm
Purpose and benefit to user:
Rejected ZEV model records serve no long term purpose and I'd like the choice to delete them.
Acceptance Criteria:
[] Given I am a BCeID Zeva user, When I have a zev model rejected by government, then I want to have the option to delete that model.
[] Given I am an IDIR analyst, When a supplier deletes a rejected zev model, then I still want to be able to see the rejected zev model and its details.
Development Checklist:
[] Add a delete button that's visible to suppliers when they are viewing an individual ZEV model.
[] Make it so that when the button is clicked, the associated ZEV model is given a "Deleted" status in the database.
[] Implement logic that makes it so that IDIR users can still see "Deleted" ZEV models, but BCeID users can't.
Notes:
Deleted ZEV are not deleted from the database and are still visible to IDIR users in Deleted status.

Hey team! Please add your planning poker estimate with Zenhub @emi-hi @jig-patel @JulianForeman @tim738745 @vibhiquartech

Please add your planning poker estimate with Zenhub @JulianForeman

It doesn't appear the last dev checklist item is working, IDIR users cannot see deleted ZEV


Question: Per-service HAProxy templates

hi
thank you for making this project available, we are currently using this in our mesos/marathon setup with HAProxy and recently had an ask about configuring bamboo for "Per-service HAProxy templates" similar to marathon-lb.
Can anyone confirm/deny if this is possible/supported with bamboo?
TIA

Depends what the requirement is from your team.
In most cases, if the services has some naming convention, the template with existing helper function can produce a very complex scenario.
In other cases, you can also deploy a second group Bamboo with completely template if the requirements are difficult to implement in a single template.
I couldn’t find any documentation from marathon-lb about the definition of pre-template. If you know more details, it would be helpful to clarify.
Cheers

On 26 Sep 2017, at 23:43, Mohamed Naseer<EMAIL_ADDRESS>wrote:
hi
thank you for making this project available, we are currently using this in our mesos/marathon setup with HAProxy and recently had an ask about configuring bamboo for "Per-service HAProxy templates" similar to marathon-lb.
Can anyone confirm/deny if this is possible/supported with bamboo?
TIA
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Remove d2to1 requirement

d2to1 was required for support of setup.cfg, but setup.cfg has been supported in
setuptools since 2016.
Removing this dependency makes it easier to build the project in contexts where the official
pypi.org mirror is not available.
Here are more details about the issue:

When installing pysynthetic, setuptools runs the setup.py file ;
setuptools detects that the d2to1 dependency is required, and proceeds to request it;
the issue is that the installation routines in setuptools only work if the pypi.org public repository is available - if one is behind a corporate firewall that blocks access to pypi.org, then the installation of pysynthetic fails;
the only easy workaround that I have found for that issue is to ensure that d2to1 is installed before attempting to install pysynthetic;

Shipping a wheel package on pypi would also alleviate the underlying issue of "ease of installation behind a corporate firewall", but I think that it would probably be easier to remove d2to1 and call it a day, seeing that according to the setuptools documentation, setup.cfg is supported in core setuptools since 2016.

Good job Jean!!
Thank you very much 🙏
In a fact, it would be nicer to provide a wheel but what about just deprecating the library and recommending the usage of dataclasses (https://docs.python.org/3/library/dataclasses.html) 😅

what about just deprecating the library and recommending the usage of dataclasses (https://docs.python.org/3/library/dataclasses.html) 😅

I want to do that! But we have a legacy project that is supposed to be retired soon with dozens of pysynthetic classes and no one on Earth has the will to migrate that application away from pysynthetic and having to re-test that nothing breaks, because the associated code has literally no future.
Anyway, thank you for the update!


Windows toPlainText() inserts extra newlines

When producing plain text output on Windows, a simple list gets extra newlines inserted. I think this is because internally everything is handled as Unix style line endings (\n) rather than Windows style. I can recreate this in a build, but not in a test.
You can try this in the Unity plugin (net202 branch). Firstly, run .\build.ps1 to make sure all the right things are downloaded, then cd into rider and run .\gradlew packReSharperPlugin. In the <root>\resharper\build\distributions\Debug\ folder is a .nupkg file. Unzipping this will give a .nuspec file, with recent changes listed with an extra separating line. Building the same target on Unix generates the plain text changelog without extra newlines.
You can see the difference in this release vs this hand edited release

Hey, Matt – is it still the case? :)


Exception NoSuchFieldException on Android 9

After creating an object of the Socket class, each time I get NoSuchFieldException with stacktrace on an Android 9 device:
W/System.err: java.lang.NoSuchFieldException: No field streamHandlers in class Ljava/net/URL; (declaration of 'java.net.URL' appears in /apex/com.android.runtime/javalib/core-oj.jar)
        at java.lang.Class.getDeclaredField(Native Method)
        at dev.icerock.moko.socket.Socket.addURLProtocols(Socket.kt:95)
        at dev.icerock.moko.socket.Socket.<init>(Socket.kt:29)
        at com.snow4unet.snowtinder.chatdomain.di.SocketFactory.createSocket(SocketFactory.kt:17)
        at com.snow4unet.snowtinder.chatdomain.repository.SocketRepository.getSocket(SocketRepository.kt:24)

I have the same problem. Any update on this?

Unfortunately, there are no updates yet

That's cause by addURLProtocols() method, not sure why is needed? also running from jvm integration test fail because of that

we remove this method at all


All Buttons inside ButtonGroup have same status

Issue type
I'm submitting a ...  (check one with "x")

[ ] bug report
[x] feature request

Issue description
Current behavior:
Button elements in ButtonGroup have same status
Expected behavior:
Each Button should have its own status
Steps to reproduce:
<ButtonGroup>
     <Button status='success'>
         A
     </Button>
     <Button>
        B
     </Button>
     <Button>
        C
     </Button>
</ButtonGroup>

Related code:

insert short code snippets here

Other information:
OS, device, package version
package: 4.3.0

Hi @28harishkumar 👋
You can achieve this by passing appearance or status property to ButtonGroup
import React from 'react';
import { ButtonGroup, Button } from 'react-native-ui-kitten';

const SameStatusButtonGroup = () => (
  <ButtonGroup status='success'>
    <Button>A</Button>
    <Button>B</Button>
    <Button>C</Button>
  </ButtonGroup>
);

Hi @artyorsh
I typed appearance instead of status.
My point is different status inside ButtonGroup
<ButtonGroup>
     <Button status='success'>
         A
     </Button>
     <Button>
        B
     </Button>
     <Button status='danger'>
        C
     </Button>
</ButtonGroup>

@28harishkumar can you share a bit more information about the use-case? Such changes need discussion with our design team

Currently I am not able to give inline Action Buttons with different colors.
Simple use case is providing Save, Save & Add Another and Delete options for some entry. Save with primary status, Save & Add Another with info status and Delete with danger status.
Another use case is providing OK and Cancel buttons.
Althouth all use cases can be covered by replacing ButtonGroup with View and writing extra style. But it will be better if ButtonGroup does not restrict individual Button property.
Additionally I found one more issue. ButtonGroup is accepting white status and it is passing it to Button. Button does not accept white status. It accepts basic status that is not accepted by
ButtonGroup.
It raises following warning:
Button: unsupported configuration.
Check one of the following prop values {
  "appearance": "filled",
  "variants": [
    "white",
    "medium"
  ],
  "states": []
}
📖 Documentation: https://akveo.github.io/react-native-ui-kitten/docs/components/button/api

@28harishkumar we don't think ButtonGroup should be used for this case. Instead, just use multiple Buttons without wrapping into ButtonGroup. See our vision of it.

I found one more issue

Thanks for reporting 👍 We've already got fix for this. Will be available in next release

@artyorsh hey, I came up with a similar case; what I'm trying to do is like the old pagination from bootstrap

Snack has that feature for selecting the platform too:

My first attemp was to set up a ButtonGroup and changing one to the children Button's state as a "Selected one". Is there a way of accomplish that on the current version of the library?
Thanks!

Actually, ButtonGroup is not about selection mode. For this case, you may see this gist I use in my project

@artyorsh this would be a great element to include in a future release.

You can also achieve this by changing the button style instead of the status, for example:
<ButtonGroup status="basic" appearance="outline" >
	<Button
		textStyle={ selected ? { color: "white" } : null }
		style={ selected ? styles.buttonSelected : null }>
			L
	</Button>
	<Button>M</Button>
	<Button>R</Button>
</ButtonGroup>

Also, I realised that using status="basic" on the ButtonGroup on Dark Mode is showing a highlighten white instead on a gray. Is that correct?
Example, using Dark Mode, the border of the basic status is has strong as the control status:
<ButtonGroup status="basic" appearance="outline" >
	<Button>L</Button>
	<Button>M</Button>
	<Button>R</Button>
</ButtonGroup>


sendtoaddress question

invoking "sendtoaddress" can return some returned values

It should return the transaction ID. See the rpc help text in line 433 in rpcwallet.cpp


test: fix flaky test TestNonblockSelectRace

close https://github.com/golang-design/chann/issues/1
This way we ensure that c1 always has a value and that c2 also serves the purpose of the test.

I don't have a better way to fix it, so I'm closing it for now. Thanks!


FileNotFoundError: [Errno 2] No such file or directory

Code version (Git Hash) and PyTorch version
6542289, 0.3.1, running Python 3.6.4
Dataset used
NTU-RGB-D
Expected behavior
run the code
Actual behavior
Traceback (most recent call last):
File "/home/rss/eclipse-workspace/CNN_Skeleton_NTURGB/main.py", line 415, in
with open(p.config, 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: './config/NTU-RGB-D/xview/ST_GCN.yaml'
Steps to reproduce the behavior
I did all steps such as data preperation, testing and evalution and saw the result on cmd. At the moment I want run the code (main.py) on eclipse but the above error appeaered.
Other comments

I have the same problem, can you tell me how you solved it?


Option to display lyrics while moving to other menus

In the origin Georgia theme, you could keep the lyrics visible whilst going through the other menus.
https://user-images.githubusercontent.com/69535896/199346620-82d4b1ed-bc4d-43d2-830b-5103331a8212.mp4
So it'd be nice to make the lyrics a toggle instead of a separate menu

Hi @RafeeDaBoy,
I could make an option for that, but lyrics top menu button will always stay there.
Since Beta 10, you can hide the top menu buttons via Options > Player controls > Top menu.
-TT

That'd be awesome! I don't mind the button, just making the lyrics persistent across the other menus would be nice. Thanks a lot for your time!

Now available in Beta 10c: https://github.com/TT-ReBORN/Georgia-ReBORN/discussions/74#discussioncomment-4140202

Awesome, thanks a lot!


[Bug]: Longform / writing css class serif font not working

Bug Description
The SF theme is supposed to provide a serif font and independent styling designed for writing manuscripts, for Longform notes (i.e., “scenes”) as well as notes with the “writing” css class in the YAML. However, the serif font does not appear for my Longform notes.
Also, I tried adding the “writing” css class to the YAML of a note, and that also had no effect.
Things I have tried
I tried both of these (Longform note and regular note with writing css class) in a fresh vault (Obsidian help vault) with SF and Longform as the only installed theme/plugin (and no additional CSS snippets).
I also tried uninstalling the SF theme, quitting obsidian, deleting the style settings data file, and then restarting obsidian and installing SF again.
I have noticed that, in the SF Style Settings specifically for Longform/writing, changing the font size does affect the font size for Longform notes, but changing the font itself does nothing.
Relevant Screenshot
https://user-images.githubusercontent.com/107888006/174674768-8a23a225-4cf1-4253-bce3-7fd3263d9331.mov
To Reproduce

Create a Longform scene with SF enabled
Write the scene

Obsidian Version
0.14.15
Installer Version
0.11.3
Font Size
16
Which editor are you using?

[X] New Editor
[ ] Legacy Editor

In which mode(s) does the problem occur? (if applicable)

[X] Source (Edit)
[X] Reading (Preview)
[X] Live Preview

Which operating system?

[X] macOS
[ ] Linux
[ ] Windows
[ ] iOS
[ ] Android

Plugin-Specific Problems

[X] I took note of the specific instructions regarding those plugins.

Checklist

[X] I updated to the latest version of the theme (Settings → Appearance → Themes - Manage → Update).
[X] I made sure that the issue is related to this theme by checking that the problem does not occur with the default theme.
[X] In case I am missing an UI element, I have checked that it cannot be re-enabled with the Style Settings plugin.
[X] If the bug occured after updating the theme, I read the last couple of entries in the changelog to make sure the bug is not an intentional change. (In that case, there will most likely be a style setting to turn it off again.)

Your installer version seems quite outdated. Could you try updating the installer and have a look again?
(You can update the installer by downloading the .dmg from the Obsidian website again and installing it over your current Obsidian version. The most up to date installer version is 0.14.6 I think)

Oh wow--this not only solved this issue, but every other issue I has as well. The whole app just looks better! Thanks so much, I had no idea that app installers need to be updated in addition to just updating the app. Sorry to bother you with this! I will go update my topic on the forum now.


Include Private Resolver on the limitations.

Include Private Resolver on the limitations for ExR with fastpath
As per public documents:
What is Azure DNS Private Resolver? | Microsoft Learn
Other restrictions
IPv6 enabled subnets aren't supported.
DNS private resolver does not support Azure ExpressRoute FastPath.

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

ID: 3ef08cf3-dc22-6ef6-e6fe-c17b1f7cfe69
Version Independent ID: dbe4d186-129a-d76f-a5eb-7257226b5b78
Content: About Azure ExpressRoute FastPath
Content Source: articles/expressroute/about-fastpath.md
Service: expressroute
GitHub Login: @duongau
Microsoft Alias: duau

Already addressed


Recieve "while post processing event: entity not found" error when updating tag set

Kong version: Enterprise 2.4
Description:
When I try to do deck sync against my workspace, after adding or removing a tag to a upstream target, I receive the error:
Error: 1 errors occurred:
        while post processing event: entity not found

Running deck sync a second time with the same yaml resolves the issue, no errors and everything is updated.
Just adding and removing to the tags list of the very basic example below was enough to replicate the issue.
_format_version: "1.1"
_workspace: my_workspace
upstreams:
- algorithm: round-robin
  hash_fallback: none
  hash_on: none
  hash_on_cookie_path: /
  name: my_upstream
  slots: 10000
  targets:
  - tags:
      - tag1
    target: localhost:1234
    weight: 100

Hi @rasmusoh , what version of deck are you running. I couldn't reproduce this with latest.

Closing this due to lack of activity. Please re-open if needed.

@rasmusoh did you ever figure out the root of this issue? I'm currently hitting this with Kong 2.8.3 and decK v1.19.1


set "spatial coverage" as required meta for raster

The geo-spatial raster resource type needs to have the "spatial coverage" as the required meta before the resource is set as public.

I am not sure what behavior you are expecting, I loaded Honduras.tif. All coverage fields were filled as NA.  It is possible to make the resource public without filling in spatial coverage.

@dtarb This function won't check the extended metadata of "spatial reference". It checks the Dublin core metadata of spatial coverage as box. I just created one resource on dev site and it works. If you test the function in the same way but it doesn't work as expected, please let me know with the resource link. Thanks.


Test effects of concurrent mutations on directory structure

Right now we don't have any test coverage of concurrent mutation—what happens if objects are created/mutated/deleted on the S3 side while Mountpoint is interacting with them. We need to get test coverage of these behaviors and then define our intended semantics for them.

[x] Concurrent object mutation while reading -- cause future reads to fail when detected
[ ] Concurrent creation/removal of implicit directories -- create/remove foo/bar when foo already existed
[ ] Concurrent creation of explicit directories -- create/remove foo/ when foo already existed

I think fuse test for prefetching covers the case of causing failure to future reads when concurrent mutation are done while reading? Or we talking about something else here?


Azure Redis and Cosmo/MongoDB private endpoint unusable

Describe the bug
When creating Redis and Cosmos/MongoDB services using the private_endpoint_subnet_id and private_dns_zone_ids parameters, a private endpoint and private dns entry are created, but the binding information URI (cosmos/mongodb) and host (redis) are still pointing to the public fqdn and not the private endpoint.
So the communication then resolves to the public IP of the service and we are unable to use the private endpoint.
To Reproduce
Steps to reproduce the behavior:

Create a Redis or Cosmos/MongoDB service with the private_endpoint_subnet_id and private_dns_zone_ids parameters.
Bind your service
View binding information to get credhub  info.
Interogate credhub to get binding information.

Expected behavior
In the binding information for redis, the host should be the .privatelink.redis.cache.windows.net fqdn and not the .redis.cache.windows.net
In the binding information for the cosmos/mongodb, the URI should be to the .mongo.cosmos.azure.com and not the .mongo.cosmos.azure.com
If we are not using the .privatelink.redis.cache.windows.net or .privatelink.mongo.cosmo.azure.com, we are not communicating with the service using the private endpoint.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
We are using the cloud service broker for Azure version 1.2.3
Opsman: 2.10.26
TAS: 2.11.10

Thanks for reporting this issue. The private endpoint properties were added in this PR: https://github.com/cloudfoundry/csb-brokerpak-azure/pull/33. It seems like the implementation might have been incomplete since I would have expected logic around the host output: https://github.com/cloudfoundry/csb-brokerpak-azure/blob/main/terraform/azure-redis/redis-provision.tf#L116 to make it private endpoint aware.
Since this feature came in through a community contribution we, unfortunately, won't be able to prioritize this anytime soon. However, we are more than happy to review a PR addressing this issue.

I will close this issue. After looking into the TLS errors we got when I replace the credhub credentials to use the .privatelink fqdn, I noticed we had TLS errors which made me dig more into the certificate errors. The .privatelink fqdn were not in the certificates SAN's. We then noticed that we had to forward DNS resolution of the  .mongo.cosmos.azure.com and .mongo.cosmos.azure.com to the Microsoft DNS.
With the privatelinks in place, the Microsoft DNS now resolve to the privatelink IP's and the certificates are working.


Add noVNC support and update test script

Feature: noVNC
Update README.md
Added screenshot to test script

Thanks, but please remove Aurora

Closing in favor of https://github.com/aind-containers/aind/pull/73


The text label is a link

OOC, what does it link to?

This link should be a text field (further explaned by “Your Fixie API
key) at the moment it is a Link that points nowhere.
Sorry for not providing that explanation already
------ Original Message ------
From "Justin Uberti" @.>
To "fixie-ai/ai-jsx" @.>
Cc "Nikolaj Ivancic" @.>; "Author"
@.>
Date 1/10/24, 9:10:35 PM
Subject Re: [fixie-ai/ai-jsx] The text label is a link (Issue #531)

OOC, what does it link to?
—
Reply to this email directly, view it on GitHub
https://github.com/fixie-ai/ai-jsx/issues/531#issuecomment-1886084581,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUWGVMXNU33RF34B7F6CYDYN5CZXAVCNFSM6AAAAABBP3KA6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBWGA4DINJYGE.
You are receiving this because you authored the thread.Message ID:
@.***>

My "problem report" is a consequence of not understanding the structure of the API, so what I perceived as text fields are indeed buttons. As some of these buttons do not create any UI changes, I got confused.


ENH: Add check for updates and check whether version has been flagged.

Adds two new functions to assess the current version of fMRIPrep before
operating.
Close #1645.

Changes proposed in this pull request

Documentation that should be reviewed

Any outstanding issue that you can foresee?
I'm thinking we should extend the ArgumentParser object and hook all the extra tests we have enough info to check into the parse_args() - WDYT?

Oh, sorry. I saw this but forgot to comment and then it fell off my radar. Let me have another look through.

 I see that we're wrapping things in try/except blocks, but if the result is hanging instead of a quick 500 error, then we're degrading UX significantly.
I would give an out. I know we've got --no-track, but perhaps we could make a more general FMRIPREP_NETWORK environment variable that can disable sentry and any calls out when set to 0. Maybe there's a better approach, but I think if we don't build in a way to turn it off, we're going to annoy people.

Good catch, what about just a very short timeout?
https://2.python-requests.org/en/master/user/quickstart/
On Fri, Jul 26, 2019, 14:09 Chris Markiewicz<EMAIL_ADDRESS>wrote:

So I remember AFNI's version check being a royal pain when their server
went down. I see that we're wrapping things in try/except blocks, but if
the result is hanging instead of a quick 500 error, then we're degrading UX
significantly.
I would give an out. I know we've got --no-track, but perhaps we could
make a more general FMRIPREP_NETWORK environment variable that can
disable sentry and any calls out when set to 0. Maybe there's a better
approach, but I think if we don't build in a way to turn it off, we're
going to annoy people.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/pull/1715?email_source=notifications&email_token=AAESDRTSGCZOWHMB57EQ2YDQBNRXNA5CNFSM4IGWC642YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD25W77A#issuecomment-515600380,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESDRTM7QDBYFQD5JVBFRTQBNRXNANCNFSM4IGWC64Q
.

Sounds sensible
On Mon, Jul 29, 2019, 10:26 Chris Markiewicz<EMAIL_ADDRESS>wrote:

@effigies commented on this pull request.
In docs/faq_tips_tricks.rst
https://github.com/poldracklab/fmriprep/pull/1715#discussion_r308346549:

@@ -126,3 +126,23 @@ A symptomatic output looks like: ::

If you would like to run fMRIPrep in parallel on multiple subjects please use
this method <https://neurostars.org/t/fmriprep-workaround-for-running-subjects-in-parallel/4449>__.
+
+
+.. _upgrading:
I think we need to rename this file to faq and update any links to
faq_tips_tricks. This is to make the faq.html link work, but I think it's
good to move to a guessable file name anyway.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/pull/1715?email_source=notifications&email_token=AAESDRSNJ77BLD5ZRS3YW3DQB4R4RA5CNFSM4IGWC642YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB74D65A#pullrequestreview-267927412,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESDRWILGNW6F5I5Z5MAQLQB4R4RANCNFSM4IGWC64Q
.


Deprecated warnings during initial install of Sailsjs on Windows 7

Sails version: 0.12.4
Node version: 4.4.7
NPM version: 2.15.8
Operating system: Windows 7 Pro 64bit

Installing on windows 7 64 bit with the command:
`npm -g install sails`
produces the following:
```
> node ./lib/preinstall_npmcheck.js
Sails.js Installation: Checking npm-version successful
npm WARN deprecated<EMAIL_ADDRESS>'native-or-bluebird' is deprecated. Please use 'any-promise' instead.
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>cross-spawn no longer requires a build toolchain, use it instead!

Ok, fixed!

Did you test if the installation was successful? For me the log message just looks like a bunch of warnings that can be ignored.

Yes everything works. The minmatch seems like a security issue however. Seeing a bunch of warnings during install somehow does not inspire confidence. :)

@SilverPaladin re security issue:  We're using minimatch as a dev dependency, via Grunt.  So luckily, the vulnerable code path never touches production Sails apps.  For a detailed breakdown, see: http://blog.sailsjs.org/post/146087338282/node-security-advisory-for-socketio-v146-and  Since updating grunt-sync has caused usability issues and race conditions with other plugins in the past, we're trying to be really careful about bumping any Grunt dependency until we've seen it work perfectly on our own projects for a few weeks.  We haven't gotten there yet.
Re perception: totally understand.  Since it's a purely aesthetic thing, it's super low on the priority list, but I'm chipping away at it when I have time.  Like @abisz said, you can ignore the deprecation warnings.  We keep a close eye on security advisories (we're using Sails.js for all of our own critical production systems, fwiw), and if anything pops up, we deal with it ASAP.  More background on that here and here.
Thanks!

(btw I think I nabbed the lodash one this morning w/ https://github.com/balderdashy/sails-build-dictionary/commit/472347485d10554ee354082c88db8d88570f6b76 -- haven't gone through and given it another pass yet to see what shows up)

I just installed sails and got these warnings on my cmd.
npm WARN deprecated<EMAIL_ADDRESS>'native-or-bluebird' is deprecated . Please use 'any-promise' instead.
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher  to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade  to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>cross-spawn no longer requires a bu ild toolchain, use it instead!
From the posts above looks like we can continue to create a project. But just not sure why are these older versions still used and what are the implications. Or should they be updated.

Hey @Sid6555 I understand your concern.

But just not sure why are these older versions still used and what are the implications.

See https://github.com/balderdashy/sails-docs/commit/32305e471911ddef5b2965dff401c87a8e3941cc (and the two linked threads) for some more background info.

@Sid6555 @SilverPaladin btw re:

Or should they be updated.

Definitely-- they're confusing and scary to folks who are getting started with Sails, plus they make it hard to actually see any real problems.  Expect to see all of them cleared out when we release Sails v1.0 later this year.  And if I ever forget about all of this pain, and use npm deprecate on any of the Sails core packages in the future, please feel free to remind me about this moment and chide me profusely. 🐙


chore(deps): update dependency mini-css-extract-plugin to v0.8.0

This PR contains the following updates:

Package
Type
Update
Change

mini-css-extract-plugin
devDependencies
minor
0.5.0 -> 0.8.0

Release Notes

webpack-contrib/mini-css-extract-plugin
v0.8.0
Compare Source
Features

Add ignoreOrder option (#422) (4ad3373)

v0.7.0
Compare Source
Features

Add ignoreOrder option (#422) (4ad3373)

v0.6.0
Compare Source
Features

added error code to chunk load Error (#347) (b653641)
adding hot module reloading (#334) (4ed9c5a)
publicPath can be a function (#373) (7b1425a)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Enabled.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

:tada: This PR is included in version 2.139.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


deserialization module optimization

What type of PR is this?
/kind cleanup
/kind enhancement
/kind documentation
What this PR does / why we need it:

[ ] deserialization module dfx optimization
[ ] add document
[ ] cleanup redundant process code

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/wip


The website needs a community calendar - Hactoberfest

A community calendar needs to be added to the events page on Ortelius.io

Created a Google Groups<EMAIL_ADDRESS> Added the following code to the Ortelius events page.
Meeting Calendar


Merge to live 2022-06-29

No detected changes to .json files in the root

OPS passed. This can merge.


Ditch "-> ()" in favor of "-> Void"

This change should be done now in early stage, to prevent new code keep following the old convention.

Since Void is just a typealias for (), why do you prefer one over the other? (Just curious, not arguing with your decision.)

I have a mild preference for Void as well, partly from C-family familiarity, partly because () really seems like it should be Unit rather than Void…

I definitely feel like () makes more sense as "this is a set of empty arguments" whereas, like @Catfish-Man says, Void feels like a "nothing return type".

It is considered standard now.
In devforums:

FWIW, we've recently decided to standardize on () -> Void (generally, () for parameters and Void for return types) across all of our documentation.

Similar commit in swift repo: https://github.com/apple/swift/commit/6536edd68c3cf44f0c61eb69c117a69f8b4185c2
I also personal prefer Void over () since it's more readable when you have nest parentheses.

I agree with @thii, () might suggest that a method returns an empty tuple. Void makes it clear what's the expected return type. Obviously that doesn't make a difference for the compiler, but it's simpler to read in my opinion.

Same. I'm completely okay with (), just that it gets quite noisy when you have nested parentheses.
Extreme example:
func blah(f: (() -> ())?)

A bit easier to read with Void:
func blah(f: (() -> Void)?)

I agree with @radex - it gets hard to read with lots of nested parentheses. I'd also vote for -> Void. This change looks good to me.

I personally prefer Void as well, however in earlier development I did run across some hiccups with it not working correctly on linux targets for some reason; however that problem is probably resolved by now


Remove dead links from pt_PT

Hi. This pull request will remove some dead links from pt_PT books

Thanks!


fix: add window postMsg to get base64 on new android

Any reason not to merge both into master?

Also, I think this needs to be merged into the init-workaround branch, or a new branch with them combined; init-workaround is used to set the initial canvas size.

I honestly don't have the details of this other than the onChange event would not fire on new android devices. Not much help from the issue raised on the repo about this but i will ask

Could you perhaps explain in a comment/readme what has changed in newer Android versions and why this is needed?

master is generally kept in sync with upstream for easier pulls

ok. develop it is then


Abstract the flex layout to reusable mixins

Need: bake flex layouts into 'organism' components based on standardized flex layout
Solution: abstract code into mixins/placeholders
Relates to: https://github.com/patternfly/patternfly-next/issues/1528

I can't seem to find a good way to do this right now while remaining consistent with our contribution guidelines. I'll have to revisit at a later date..


Entropy for GPG generation

Figure out a better approach to refill the entropy pool for GPG key generation.
For now simply relying on user doing something with his computer while waiting for the key: poor design!

After working with this package and thinking at the targeted audience, I don't think this is very relevant anymore (few users would generate multiple GPG key in a row). Closing for now.


My-Items Page shows the same item multiple times

As an item owner, I want to have a clear overview of my items. Therefore, each item should be listed only  once.
At the moment:
I 'can manage' item 'Magie' with my personal group and the group 'Magische Muggel' (of which I am a member).
The item is listed twice.

Three times if everyone can manage:

Acceptance Criteria

[ ] Make sure that each item appears only once in a grid view.

This has actually been fixed as a side effect of ea54687b0e68ff07b62cfe99628313442276fbe7. Applying sorts and filters to the items only returns distinct items. However, this only fixes the symptoms not the origin.


rqt_reconfigure does not display any parameters

When I execute "rosrun rqt_reconfigure rqt_reconfigure", the gui window pops up, but I see no parameters to change. Am I missing some installation or configuration to do this? This is what appears when I execute the command:

@csperbeck6, are you still having this issue? The troubleshooting guide talks about using the command line dynamic reconfigure client to confirm what rqt_reconfigure is showing. Can you give that a shot?

After getting no response from @csperbeck6 for quite some time, I'm going to close this issue. If you're still having problems and have time to work with me to reproduce them, please feel free to re-open this issue.


Lucene 9.12.0 support

Summary
Lucene 9.12.0 has recently been released. Publish a new version of Lucas built on that version.

Completed in v0.5.0.


Find valid moves

We need to implement moves generation for
non sliding pieces [useful link]

[x] king
[x] knights
[x] pawns

sliding pieces

[x] rooks
[x] bishops
[x] queen

Done here


tour: [REPLACE WITH SHORT DESCRIPTION]

Context: https://tour.golang.org/basics/1
Change the title above to describe your issue and add your feedback here, including code if necessary

Empty issue, closing.


[test] Replaced some if true {} statements with do {}

What's in this pull request?
Replaced if true {} with do {} in some test files

@swift-ci Please test and merge

@swift-ci test


[stable/cerebro] fixes #12910

Signed-off-by: Balint Csergo<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @davidkarlsen

/ok-to-test

/lgtm

/lgtm cancel

/lgtm


Entire page gets pushed up when clicking input while bottom nav is there

Steps to Reproduce

Tap the TextField
You'll notice the entire page is pushed up when the soft keyboard shows up

Gist: https://gist.github.com/silverAndroid/eb40f5faaa2d8a0296baf2f9c337ec2b

Logs
No exception.
Flutter Doctor
[√] Flutter (on Microsoft Windows [Version 10.0.15063], locale en-US, channel master)
    • Flutter at D:\dev\SDKs\flutter
    • Framework revision 51ab5f36c6 (14 hours ago), 2017-12-08 20:39:56 -0800
    • Engine revision edb0201fa2
    • Tools Dart version 1.25.0-dev.11.0
    • Engine Dart version 2.0.0-edge.a38ac7cf127f4611c873c2f2d523c06ce06b1342

[√] Android toolchain - develop for Android devices (Android SDK 26.0.2)
    • Android SDK at D:\dev\SDKs\android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-26, build-tools 26.0.2
    • ANDROID_HOME = D:\dev\SDKs\android-sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[√] Android Studio (version 3.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[√] IntelliJ IDEA Ultimate Edition (version 2017.3)
    • Flutter plugin version 20.0.3
    • Dart plugin version 173.3727.108

[√] Connected devices
    • LG H915 • LGH9157e1e3243 • android-arm • Android 7.0 (API 24)

cc @cbracken might be fallout from your recent work

@silverAndroid Thanks for reporting!
@Hixie Yep - that looks to be the case. This should be a small patch on the Android side to ensure the keyboard is no longer triggering padding, just the inset. Will take a look.

For the Material side. Fixed by #14406. If you're still seeing issues, please let me know.
We'll need to land a similar patch to fix the Cupertino-specific issue reported by @pulyaevskiy. I've opened #14414 to track that bug.


Request for TypeScript typings

It would be great to have TypeScript typings for this library, published through https://github.com/typings/registry or Definitely Typed.

@webprofusion-chrisc that would be great indeed. I'm using TypeScript in some projects and it is definitely great. I'll try to rewrite this lib during the weekend. Cheers!

Hey @webprofusion-chrisc, this should be fine now. Just rewrote the entire lib in TypeScript and added typings key on package.json.
Cheers!


Corrected regression from issue 21838

Details

A regression from PR https://github.com/Expensify/App/pull/23306
The problem is that only errors for touched inputs are shown, so we should not set lastValidatedValues until at least one input has been touched. Otherwise validation will be skipped right after the input has been touched.
https://github.com/joh42/expensify-app/blob/fix/21838-regression/src/components/Form.js#L144-L153
Fixed Issues

$ https://github.com/Expensify/App/issues/21838
PROPOSAL: https://github.com/Expensify/App/issues/21838#issuecomment-1620203806
Tests

Test 1

Make sure you are using the staging server (Settings > Preferences > Use staging server) so you can use sandbox credentials for Plaid
Go to Settings > Workspace > Bank account > Connect with plaid method > Chase Bank
Use the following credentials for Chase Bank
Username: user_good
Password: pass_good
If a verification is prompted - select mobile and enter "credentials_good" as the verification mode
Checkmark the "Plaid Checking" and "Plaid Saving" and the last 2 options.
Click on continue
Check the T&C and click "Connect account information"
Click on the last confirmation on the plaid modal
Select the bank account ending in 1111
Enter Routing Number: 011401533 an enter an incorrect bank account number (e.g<PHONE_NUMBER>334444)
Click continue
Verify that the server error you got remains if you blur the routing number input field (click outside it, change tabs etc)
Verify that the server error is cleared if you edit the value of either input field

Test 2

Go to Settings > Profile > Display Name
Enter a semicolon in the First name field (;)
Blur the input field (click elsewhere) and make sure an error is shown

[X] Verify that no errors appear in the JS console

Offline tests

Same as tests
QA Steps

Same as tests

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Test 1
https://github.com/Expensify/App/assets/138504087/1269505f-0cd1-499a-a248-21b8d9a490b4
Test 2
https://github.com/Expensify/App/assets/138504087/b0a75aff-29d1-4fb7-960b-446dcb1f9ecc

Mobile Web - Chrome

Test 1
https://github.com/Expensify/App/assets/138504087/93d5a1f2-e356-459f-96ec-a2872b3d5508
Test 2
https://github.com/Expensify/App/assets/138504087/79760916-526b-47bf-b47e-c8f23035db29

Mobile Web - Safari
Test 1
https://github.com/Expensify/App/assets/138504087/3c10b55e-f768-4fc3-bc45-2e8b65402ed2
Test 2
https://github.com/Expensify/App/assets/138504087/fbeefa2c-4bf6-45f0-81ae-d303ae80fdd7

Desktop

Test 1
https://github.com/Expensify/App/assets/138504087/47cced0d-74ce-46bc-a5c5-a1589ac36ff2
Test 2
https://github.com/Expensify/App/assets/138504087/bbed31c5-0664-4ae8-9cea-dde3347b6b30

iOS

Test 1
https://github.com/Expensify/App/assets/138504087/e719a7c4-6978-4980-97f0-4f8b0ddc6d7b
Test 2
https://github.com/Expensify/App/assets/138504087/e5d4c587-6321-4280-a859-766b3231d4fd

Android

Test 1
https://github.com/Expensify/App/assets/138504087/467e498b-f1ce-4228-8435-b10e88e4943d
Test 2
https://github.com/Expensify/App/assets/138504087/25faa34a-ec73-4032-a7c7-65de3a74c303

Ping @eVoloshchak, would be great if you could take a look a this

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.49-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.50-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


update example project

add traditional Chinese

Thanks for this @feikaifeng and I appreciate the work that went into it but I don't see a need to add a new language to the sample app.
Will close.


Roadmap update

I believe that after MS Ignite 2018 roadmap should be updated, as lot of things already changed.

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

ID: f9812a06-9d08-b754-5c2e-d63d038f4dbd
Version Independent ID: e3fc3e85-2821-cfb6-2cd8-80c68420ac99
Content: SharePoint Framework roadmap
Content Source: docs/spfx/roadmap.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Thanks for your post. You can contribute for it by submitting any pull request based on the MS Ignite 2018. Otherwise, the content team still need more time on this issue.

Outdated... the had been updated.


Different colored blocky background for dark themes

Description
This problem appears on Atom 1.32.1 under Ubuntu 18.04 in atom --safe mode. Enable a dark theme and start typing text, and the background alters between dark black, and dark grey, in large blocky chunks. Looks like an Atari 2600 game.
Expected behavior: One consistent background.
Actual behavior: See this picture.

Reproduces how often: All the time.

Thanks for the report!  If you update Settings > Color Profile to Use sRGB color profile, does that make any difference?

I will try this and get back to you in the next several days.

Great, glad that helped :+1:


Use our deep equality handling for element metadata.

Problem:
There's a lot of deep equality checks that happen every frame, which are currently handled by introspective code.
Fix:
Mostly attempt to reduce the introspection by building more manual deep equality checking.
Results:
This appears to be upper bounded to be as good as the existing introspective calls or in a best case scenario was twice as fast.
Commit Details:

Created KeepDeepEqualityCall instances for everything up to JSXMetadata.
Added some more utility functions for deep equality.
Fixed a bug in arrayDeepEquality.
Used the new deep equality functions when reconstructing the metadata
and when the DOM report is pushed.
Fixed some typing issues around SceneProps that were uncovered when
running the deep equality checker.

in a best case scenario

does it mean the performance varied, or you mean for a certain shape of jsxMetadata

does it mean the performance varied, or you mean for a certain shape of jsxMetadata

Yes and impossible to know without creating lots of test cases.


Create google suite for biosimulations domain

Might be worth looking into getting a google suite
We will need emails for information, data privacy etc
If using google cloud, will simplify identity and access roles

https://improvmx.com/ will provide email without needing a google suite

If we create accounts for team members, either through a google suite or through the above link, Auth0 can designate team members as 'admins', 'moderators' or other roles.
This might be helpful for creating accounts for users that are allowed to validate models for example

There is now a google cloud organization for access to cloud resources. Email forwarding is set up using improvmx. Authentication roles have been created for admin and superuser


Provide a way to adapt a Configuration so that it can be used in tasks that use RegularFileProperty

Having a dependency that includes a single file is difficult to wire into a task that was a RegularFileProperty. For example, in a multi-project setup where an executable or single output file is produced by one project and needs to be consumed by another, it can be annoying and difficult to wire it into tasks that make use of RegularFileProperty.
Expected Behavior
Ability to adapt a Configuration instance into a RegularFile  or as a type that can be provided to one of the setters to RegularFileProperty so task inference can be deduced.
Current Behavior
Need to explicitly add a dependsOn/input.files to the configuration in the task, use Configuration.getSingleFile(), and/or perform some other workaround to make sure task inference can be deduced without having to explicitly add it to the task configuration.
Context
In a multi-project build you may download or create an executable or a single consumable file that you want to use in another project. This works alright with JVM JARs since JavaExec makes use of classpath and mainClassName, but doesn't work as well when you want to invoke that executable or make use of that file directly.
A contrived example:
`settings.gradle.kts
rootProject.name = "kotlin-dsl-lab"
include("a")
include("b")

a/build.gradle.kts
plugins {
  base
}

val outputFile = file("$buildDir/file.txt")

val myTask by tasks.creating {
  outputs.file(outputFile)
  doFirst("write file") {
    outputFile.writeText("hello world")
  }
}

val myConfig by configurations.creating {
  outgoing.artifact(outputFile) {
    builtBy(myTask)
  }
}

b/build.gradle.kts
val consumerConfig by configurations.creating

dependencies {
  consumerConfig(project(path = ":a", configuration = "myConfig"))
}

open class RFPConsumer : DefaultTask() {
  @get:InputFile
  val inputFile: RegularFileProperty = project.objects.fileProperty()

  @TaskAction
  fun readFile() {
    println("Content: ${inputFile.asFile.get().readText()}")
  }
}

// fails
val singleFile by tasks.registering(RFPConsumer::class) {
  inputFile.set(consumerConfig.singleFile)
}

// succeeds
val inputWithFiles  by tasks.registering(RFPConsumer::class) {
  inputs.files(consumerConfig)
  inputFile.set(consumerConfig.singleFile)
}

// configuration failure
val inputWithFile by tasks.registering(RFPConsumer::class) {
  inputs.file(consumerConfig)
  inputFile.set(consumerConfig.singleFile)
}

// succeeds
val usingDependsOn by tasks.registering(RFPConsumer::class) {
  dependsOn(consumerConfig)
  inputFile.set(consumerConfig.singleFile)
}

// fails
val adaptWithProvider by tasks.registering(RFPConsumer::class) {
  inputFile.set(layout.file(provider { consumerConfig.singleFile }))
}

Steps to Reproduce (for bugs)
N/A
Your Environment

Gradle 5.1.1

You should be able to use Configuration.elements and Project.layout to do this now, albeit not in a very convenient way.


Realtime support?

Hello, I'm here with small idea/suggestion.
As documentation is not too precise in some cases, maybe realtime-support would be something
which could help a lot of devs and in lot of other thingo'-things?
For example: server on Discord (totally free) or Slack app?

https://gitter.im/bkaradzic/bgfx this works :)

Cool, thanks :)
I didn't know about this.


fix: quote hello_winpath in uninstall.sh

Just noticed that the path to the Windows install directory isn't quoted in the uninstall.sh file.
Because of this, the directory isn't removed on uninstall if the path contains spaces.
Worse, since the path is treated as two separate ones (before and after the space), the uninstall script might rm -rf other, unrelated directories¹ – so I'd say this is quite an urgent fix!

¹ if the users Tim and Tim Scott exist, uninstall.sh runs rm -rf /c/Users/Tim Scott/pam_wsl_hello and boom, say goodbye to Tim's home directory!

Thanks for the catch!


Enable native support for Windows

To enable support for MS Windows we have to make sure that all binary dependencies are compiled and pushed for it.

[x] docling-parse Done in https://github.com/DS4SD/docling-parse/pull/21, released in docling-parse==1.3.0
[x] deepsearch-glm Done in https://github.com/DS4SD/deepsearch-glm/pull/79

ETA: September


chore: merge the same logic

The logic here seems to be mergeable.
Hope vue-next gets better and better.

A normal for loop is faster than forEach.
So I don't think this change is beneficial, it would degrade performance in a performance-sensitive spot.

As @LinusBorg said, for loop here is intentional. Maybe we can explore faster approaches for looping through sets can maps but not the reverse (needs a performance benchmark). Thank you.


Implement handling of local packages in the package-set

Implements support for local packages, as described in https://github.com/spacchetti/spago/issues/88#issue-403519960
Basically we now try to distinguish if "URL"s in the package-set are referring to remote repos, or local filesystem path. In the latter case instead of trying to clone them, we just include their sources in the build (if they are listed as a dependency ofc). This basically mirrors the behaviour of stack.
Practical example: let's say I find a bug in simple-json. I manage to fix it locally, but I also want to see if the fix works for my project.
Since simple-json is already in the package-set, I can add the local package to the overrides in packages.dhall:
...

let overrides = { simple-json = upstream.simple-json // { repo = "../purescript-simple-json" } }

...

Note that if I list-packages, it is correctly listed as a local package:
$ spago list-packages
...
signal                v10.1.0   Remote "https://github.com/bodil/purescript-signal.git"
sijidou               v0.1.0    Remote "https://github.com/justinwoo/purescript-sijidou.git"
simple-json           v4.4.0    Local "../purescript-simple-json"
simple-json-generics  v0.1.0    Remote "https://github.com/justinwoo/purescript-simple-json-generics.git"
smolder               v11.0.1   Remote "https://github.com/bodil/purescript-smolder.git"
...

After adding it to the dependencies in spago.dhall, if I do spago install, it will not download it, but only its dependencies:
$ spago install
Installing 42 dependencies.
...
Installing "distributive"
Installing "refs"
Installing "identity"
Skipping package "simple-json", using local path: "../purescript-simple-json"
Installing "control"
Installing "enums"
Installing "st"
...

@justinwoo note that in his example is an addition, not an override, so you need to specify the dependencies somehow: either listing them or pulling them from a spago config if the package you're importing has one

So in this case you'd want to have a new mkPackage function that can make local library package definitions? Where one of the args passed in would be either the path or the imported record?

@justinwoo could you make an example of how it would look like?
I though about having a mkLocalPackage that takes only two args (name and deps), but in the end it's not worth it, because a usecase could be that once you upload your package, you just swap the local package with the URL of the new git repo. In the end I'm not even sure we need mkPackage, since it's just producing a record.

mkLocalPackage would be useful enough. mkPackage is also fine overall since you know that's what you're making rather than just some random record that you hope is fine, and also since you want to ensure that records are of type Package.


tour: explain how to read a stacktrace

Hello,
I'd like to propose adding to the Tour of Go a page where the user is presented with a stack trace that occurs from a panic and guided into how to read / debug it.
I think this would be valuable for people just learning the language and would help them when they'll run for the first time into a such an issue.
If approved, I'd be happy to send a CL to do this (guidance welcomed).
What do you think?

Great idea. This would be especially useful for people not used to stack traces in other languages like Java or Python.

Great idea.

Thanks for volunteering to do this. I agree this is clearly needed.

@dlsniper thank you for the issue and for volunteering to work on this issue. There is some feedback on your CL from almost 2 years, please rebase and take a look if you can. Thank you.

@dlsniper kindly pinging you to help send the CL instead to the x/website repository, thank you very much!


Fix several issues

Fixes #35, #36 (Just added lsp-workspace-folders-add), and #21 (Accepting (and ((not (package-install 'lsp-mode)) (featurep 'lsp-mode)))).

Thanks for the PR! I will take a look at it.


终端显示'DiffIRS1' object has no attribute 'module'

进行train_DiffIRS2_GANv2.sh时报错
Traceback (most recent call last):
File "DiffIR/train.py", line 15, in
train_pipeline(root_path)
File "/root/autodl-tmp/DiffIR/train_pipeline.py", line 125, in train_pipeline
model = build_model(opt)
File "/root/miniconda3/lib/python3.8/site-packages/basicsr/models/init.py", line 26, in build_model
model = MODEL_REGISTRY.get(opt['model_type'])(opt)
File "/root/autodl-tmp/DiffIR/models/DiffIR_GAN_S2_model.py", line 44, in init
self.model_Es1 = self.net_g_S1.module.E
File "/root/miniconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1185, in getattr
raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'DiffIRS1' object has no attribute 'module'

Have you solved this problem? I encountered the same problem.


fails due to deleted Github user

Example commit which triggers it:
https://github.com/bergie/passport-saml/commit/3ba015843757bde81127b30bf2ec27250c0f773d
Example of what output looks like when the the failure is triggered:
===> GET: https://api.github.com/users/Herbert%20Voj%C4%8D%C3%ADk
<=== 404 Not Found: GET: https://api.github.com/users/Herbert%20Voj%C4%8D%C3%ADk
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x698452]

goroutine 1 [running]:
github.com/itchyny/github-migrator/migrator.(*builder).buildImageTag(0xc0000a9980, 0x0, 0x23, 0xc0001dbf08, 0x0)
        /home/mark/RAgit/github-migrator/migrator/builder.go:240 +0x42
github.com/itchyny/github-migrator/migrator.(*builder).buildUserActionBody(0xc0000a9980, 0x0, 0x74fd1a, 0x27, 0x0, 0x0, 0x0, 0x0)
        /home/mark/RAgit/github-migrator/migrator/builder.go:234 +0x75

This is fixed as well.

Thank you so much. This tool is very helpful. Testing now.

It seems to be running smoothly now. I'm going to get something to eat and will check to see if it's finished later.