Cannot add an auto-export

Report ID: 5XWNRT9N
After exporting a collection with keep updated checked, the export process runs well and the exported collection appears properly in Preferences - Better BibTex - Automatic export. However, after restarting Zotero, the exported collection just disappeared from Automatic export.
What could go wrong here? Thanks in advance.

That means the DB isn't saved. Could you try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.51-circle-2382.xpi ?

Thank you, but the problem still exists.

Can you

go into the BBT prefs, section Advanced, enable debugging
add a new auto-export
from the gear menu, select "send better bibtex error"
post the error ID (red) here

Yeah sure, please check Report ID: S2TVTXDF .

Could you re-install the latest release and do that again?

Yes, the new Report ID is XWMV2RVF .

I'm sorry to have to subject to this, but I need a full log of what's happening: can you get a "Real-time Debug Output" as per https://www.zotero.org/support/debug_output (which is a pain on Windows, I'm so sorry about that) and drop the resulting log into https://dbinbox.com/retorquere ? Your DB isn't saved on shutdown for some reason, and that reason won't be in the usual log.

Yeah I guess there may be something wrong with Zotero itself, but reinstalling Zotero and BBT does not fix this problem either. I've already dropped the debug log into your dbinbox and I hope that will be helpful.

Oops sorry the log I sent was not a real-time output, I will send you the correct one immediately.

Super -- please make sure you capture the log after you've closed Zotero, as I'm looking for the shutdown event.

I'm afraid that it is very difficult to capture the real-time log which you needed - the Windows console only allows me to copy part of its output (the older output just disappears and cannot be copied) and it rapidly shuts down after closing Zotero. I will let you know if I could figure out anyway to capture the log.

Does it work if you add  > c:\zotero.log to the end of that command?

No, it generates a zotero.log but does not write anything into it.

Dows adding  2>&1 after that change that?

No it doesn't :(

Windows really makes this more of a PITA than it needs to be.
There is a way, but it's even more unpleasant than what you already tried. You could try installing cygwin and script according to these steps (http://superuser.com/questions/325540/is-there-a-windows-equivalent-of-the-script-command-in-linux) and then fire up "bash" , within that "script", and within that the former command without the > redirects. After Firefox is done, type "exit", and you'll be left with a full log in a file named "typescript".
I've searched for a less invasive solution, but I have not been able to find one. Also still looking at reasons why the db doesn't save, but my current hypothesis is that something errors out before it gets to the code that does the save, and it's really hard to guess what.

Ah wait - if you run "cmd", and then run Firefox from there, the command she'll will probably stay open. If we get lucky, the error will be in the part that's still on-screen.

Thank you very much for your suggestions. I am using Zotero Standalone and I have already tried running Zotero using cmd, but that would start a new window for ZoteroDebug and that window would close quickly after shutting down Zotero. Anyway, I'll try cygwin and see if it works.

Crap, if that's the case it's unlikely that cygwin will help. I'll look for
another solution.

I have the same problem. I made an WME, it's rather large (250MB). Hope it helps. https://goo.gl/lBmyRJ

I don't have a Windows PC.

Did you use -jsconsole or -console? If you previously used -jsconsole, could you try
"C:\Program Files (x86)\Mozilla Firefox\firefox.exe" -console -ZoteroDebug > c:\zotero.log

(remove the (x86) as necessary).

Could you give https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.52-circle-2396.xpi a try, with debugging enabled, and after adding an auto-export, letting it sit for 10 seconds, then submit a new error report?

Yes, please see Report ID: UNZ5SDFF .

This is so strange, I can see the autoexport being saved after it's added. Could you try again with https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.52-circle-2397.xpi and submit a new log?

Yes. Please check SVUMKW86 .

I think I have it! FFS, if this really is it (and it must be), you've caught me at my stupidest. Thanks so much for reporting this!
Try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.52-circle-2400.xpi

It works! Thank you so much for your help! :+1:

Errrr.... I should be thanking you. This could have been really, really bad. What was confusing is that BBT really did save these settings. It just didn't load them on startup. I'm cutting a new release right now.

.53 is out.

Thank you!!

Hello retorquere,
It appears that this issue happens again in the latest version of BetterBibTex (.68). The auto export disappeared after zotero restarted. I am using Zotero standalone v <IP_ADDRESS> on a windows machine. I tried to install previous versions of Better BibTex (.53, .57, .66, .67) to see if the issue disappeared. But to make it even weirder, in all the previous versions I have tested, when clicking "export collections", the BetterBibTex format was not there (so as the checkbox of "Keep Updated").
Thank you!


Social cards not working

Twitter
On Twitter the image doesn't show any logo, as you can check on https://cards-dev.twitter.com/validator

And for deeper pages such as sourced.tech/engine the description doesn't match the page.

Facebook
Facebook (https://developers.facebook.com/tools/debug/sharing) warns that the social image is not a valid link, and instead displays a random image it found (the OSS one)

For sourced.tech/engine even more warnings appear and the text in the link again doesn't match the contents.

could you provide our fb:app_id?

I don't care as long as it displays correctly, not sure how serious that link is.
If needed, we do have a facebook page, maybe @vcoisne can help providing the details.

@campoy @vcoisne I think it's not a link but an app id.
After reviewing some info about it, seems to be related to gathering analytics from fb apps that might be not being used by us.
Ignoring that field, I deployed the fix into staging and it works, so the PR is now ready to be validated/merged/deployed.
https://github.com/src-d/landing/pull/323
If you prefer to validate the changes in production, just say so, and I'll merge/deploy on my own ;)

Deployed into production
https://sourced.tech


LP-cert,Def_Contry, AFD_Char_limit

Learning Path certificate
Default country code: App Config
Note on AFD 5 character limit

@v-sambati : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit 756df9f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-community-training/content-management/create-content/create-learning-path/create-a-learning-path.md
:warning:Warning
View
Details

microsoft-community-training/infrastructure-management/install-your-platform-instance/installation-guide-detailed-steps.md
:white_check_mark:Succeeded
View

microsoft-community-training/media/Default country.PNG
:white_check_mark:Succeeded
View

microsoft-community-training/media/LP_cert.PNG
:white_check_mark:Succeeded
View

microsoft-community-training/settings/configurations-on-the-training-platform.md
:white_check_mark:Succeeded
View

microsoft-community-training/content-management/create-content/create-learning-path/create-a-learning-path.md

Line 45, Column 5: [Warning-alt-text-bad-value] Alt text 'LP_cert' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@v-sambati : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit affae50:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-community-training/content-management/create-content/create-learning-path/create-a-learning-path.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/install-your-platform-instance/installation-guide-detailed-steps.md
:white_check_mark:Succeeded
View

microsoft-community-training/media/Default country.PNG
:white_check_mark:Succeeded
View

microsoft-community-training/media/LP_cert.PNG
:white_check_mark:Succeeded
View

microsoft-community-training/settings/configurations-on-the-training-platform.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added or Deleted File(s)
This PR contains added or deleted file(s).

File Change Percent
This PR contains file(s) with more than 20% file change.

Image File
This PR contains an image file(s).

PRMerger Results

Issue
Description

Added or Deleted File(s)
This PR contains added or deleted file(s).

Yaml File(s)
This PR contains a change to a .yml file.

Changed Files
This PR contains more than 10 changed files.

File Change Percent
This PR contains file(s) with more than 20% file change.

Image File
This PR contains an image file(s).

Json File
This PR contains json file(s).


Changes for dpdk in private network

What does this PR do?
Creates the DPDK VM for the private network
Do you have any concerns with this PR?
No
How can the reviewer verify this PR?
By deploying the VM on the private network
Any background context you want to provide?
NA
Screenshots or logs (if appropriate)
NA
Questions:

Have you connected this PR to the issue it resolves?
No
Does the documentation need an update?
No
Does this add new Python dependencies?
No
Have you added unit or functional tests for this PR?
NA

@bo-quan - if you feel this patch operates properly, I don't see any reason why this should not be merged.


Contract path not found for hash 8a441098187753c038189610ef0aa17fa120bf211b5130a2c7f273a8273818ae

When TestB inherits TestA without a constructor, this error occurs when compiling after instantiating TestB:
Error: Command failed: /Users/alan/Library/Caches/hardhat-nodejs/compilers-v2/zksolc/zksolc-v1.3.6 --standard-json    --solc /Users/alan/Library/Caches/hardhat-nodejs/compilers-v2/macosx-amd64/solc-macosx-amd64-v0.4.23+commit.124ca40d
Contract path not found for hash 8a441098187753c038189610ef0aa17fa120bf211b5130a2c7f273a8273818ae

Excuse me, there is an error in this bug or the code I wrote.
The following is the source code
//SPDX-License-Identifier: Unlicense
pragma solidity ^0.4.23;

contract TestA {
    address private adminOwner;

    function setAdminOwner(address newAdminOwner) public {
        adminOwner = newAdminOwner;
    }
}

contract TestB is TestA {

    constructor(address owner) TestA() {
        setAdminOwner(owner);
    }

    function setGreest() public {

    }
}

contract Greeter {
    string private greeting;
    TestB public test;

    constructor(string memory _greeting) public {
        greeting = _greeting;
    }

    function greet() public view returns (string memory) {
        return greeting;
    }

    function setGreeting(string memory _greeting) public {
        greeting = _greeting;

        test = new TestB(msg.sender);
    }
}

@uF4No still this bug looks fixable. Checking now.

@epldlnwynn We've identified this is caused by a bug in older versions of Solidity and it was fixed in solc v0.5.3 so that's the minimal version you'd have to use

@epldlnwynn unfortunately, it's a bug in solc: it produces inconsistent dependency relations in EVM assembly and I cannot track factory dependencies.
It was fixed in solc v0.5.3:
~/src/contracts/dependency-evmla  $SOLIDITY_HOME/target/release/zksolc bug.sol --solc solc-0.5.2
Contract path not found for hash '6f5cbe16ef4ef9f1783eb68aa47b1ee81ddf7e171b578b31db9b91cddbcf1008'
~/src/contracts/dependency-evmla  $SOLIDITY_HOME/target/release/zksolc bug.sol --solc solc-0.5.3
Compiler run successful. No output requested. Use --asm and --bin flags.

Please update the project to at least v0.5.3.


Popup giving a 404

The plugin still works, it pops up in the geofs window asking you to enter a frequency.
But once you enter the frequency, the popup that would start the actual session returns a 404.

The server's probably down. I doubt it works anymore...

The server's probably down. I doubt it works anymore...

what!! how can you say that? i hv been trying soo hard to get it right . now can you  please tell me if there is any way to fix it?

I really afraid the problem is on our side here... Since Google took down our extension ~two years ago, we never had enough interest to bring it back up. Sorry bud 😞

It's ok I guess nothing much can be done about it. But by the way I had to
tell that there are aircraft missing you in geofs with me. A320
(airberlin), 757 (delta) a330(Singapore and Air Asia) and some others are
missing. Can you help?
On Fri, Oct 9, 2020, 5:29 PM Yotam Salmon<EMAIL_ADDRESS>wrote:

I really afraid the problem is on our side here... Since Google took down
our extension ~two years ago, we never had enough interest to bring it back
up. Sorry bud 😞
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/geofs-plugins/voice-atc/issues/4#issuecomment-706139456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARGLBJ3KXGOJIC5VBWGJOQTSJ33LXANCNFSM4MZRN4XQ
.

It's OK all right? 😊 You don't have to get soo serious.
On Fri, Oct 9, 2020, 7:00 PM Yotam Salmon<EMAIL_ADDRESS>wrote:

Maybe Harry Xue can help you. I don't remember exactly but I think he was
one of the people who developed them. Those are not mine. 🤷
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/geofs-plugins/voice-atc/issues/4#issuecomment-706181865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARGLBJYH3ZZNF6HMS43R5BLSJ4GBDANCNFSM4MZRN4XQ
.

Sure thing...


adds compatibility for es5 - TFS2017

Team Foundation Server 2017 runs node v5.x. This change makes the compiled code work with older versions of node.

Yeah, terrifying af. TFS Update 1 RTM uses 5.10.1 specifically. In their defence, it's "only" 2 years old... 😂


Recriate mark2cure platform for crowdcuration

Mark2cure is a very similar project to this one, which has lead to very cool publications.
Their GitHub repository is well organized but has not been maintained for the past 3 years.
It would be nice to clone the repository and get a modifiable instance of Mark2cure running, so we can play around and prototype the idea.

I was not able to do it myself :(
I left a log of my failed attempted at this notebook.

Closing the issue after the failed attempt. Could be reopened if someone is interested.


Configuration should work with default values

Hey, first thanks for this cool plugin. It looks like exactly what I need!
I installed the Gem, added it to the Jekyll plugins configuration and on jekyll build I get:

~/.local/share/asdf/installs/ruby/3.1.0/lib/ruby/gems/3.1.0/gems/jekyll-picture-tag-ng-0.6.0/lib/jekyll-picture-tag-ng/jekyll/site.rb:11:in write': undefined method []' for nil:NilClass (NoMethodError)
  if config["picture_tag_ng"]["parallel"]
                             ^^^^^^^^^^^^
    from ~/.local/share/asdf/installs/ruby/3.1.0/lib/ruby/gems/3.1.0/gems/jekyll-4.3.2/lib/jekyll/site.rb:82:in `process'
    from ~/.local/share/asdf/installs/ruby/3.1.0/lib/ruby/gems/3.1.0/gems/jekyll-4.3.2/lib/jekyll/command.rb:28:in `process_site'
    from ~/.local/share/asdf/installs/ruby/3.1.0/lib/ruby/gems/3.1.0/gems/jekyll-4.3.2/lib/jekyll/commands/build.rb:65:in `build'
    from ~/.local/share/asdf/installs/ruby/3.1.0/lib/ruby/gems/3.1.0/gems/jekyll-4.3.2/lib/jekyll/commands/build.rb:36:in `process'
    from ~/.local/share/asdf/installs/ruby/3.1.0/lib/ruby/gems/3.1.0/gems/jekyll-4.3.2/lib/jekyll/command.rb:91:in `block in process_with_graceful_fail'

If I add the default configuration from the README to the jekyll config, it does work!
It would be much nicer if the plugin worked by using default values. If one needs to change from default, then one could start adding the config :)

You are absolutely right, I just published an updated version that should fix this (https://github.com/pcouy/jekyll-picture-tag-ng/commit/8fe29e73bc266244e2533681e47572787a6a8713)
Thank you for taking the time to try the plugin and report the issue.


⚠ ClamAV has degraded performance

In 6407d1e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in bd08722.


skip items with nil values

Is it possible to have an option for ignoring nil values like this one in JBuilder https://www.rubydoc.info/github/rails/jbuilder/Jbuilder:ignore_nil!
I'm using JBuilder everywhere, and we have a convention to skip items with nil values (client wants it). I'm going to use Blueprinter instead, but the last thing which blocks to do that is this missing option.

Hi @skcc321, thanks for reaching out.
I'll bring this up to the team and see if (and in what way if so) we'd like to support something like this.
In the meantime, this is something that can currently be accomplished using Blueprinter's conditional field functionality.
i.e.:
field :name, unless: ->(user, _options) { user.name.nil? }

Though obviously this is not ideal to have to specify for every single one of your Blueprint fields. I'll re-post when I get the team's feedback on this.

Thank you @mcclayton for taking care of this issue. I appreciate it a lot.
The suggested approach gives the ability to achieve expected behaviour.
But, it would be great to be able to set skip_nil per Blueprint serializer, but not global.
For example, I have external integration with third-party service which wants to skip_nils.
but also I have another integration wich wants nils.
the described case can be covered easily with JBuilder directly inside view https://www.rubydoc.info/github/rails/jbuilder/Jbuilder:ignore_nil!
Is there any problem to put option on some Plueprint serializer (parent serializer) so all nested serializer will behave the same way?
Anyway, I'm excited about the current solution. So thank you for what you have done.

@skcc321 Ahh that makes sense. I've been sick recently and been wrapped up in some other work -- but once I feel better and wrap up my other work, I'll put some thought into an inheritable configuration  solution 👍

I'm going to go ahead and close this, as this should now be possible using global if/unless blocks as of https://github.com/procore/blueprinter/pull/131 being merged and with the 0.18.0 release.


ValueError: Resource handles are not convertible to numpy.

python main-makespan.py
2021-06-27 23:37:25.581570: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
Jobs number:  10
Tasks number: 93
Task instances number mean:  45.376344086021504
Task instances number std 85.48783652126134
Task instances cpu mean:  0.5264810426540284
Task instances cpu std:  0.10018140357202873
Task instances memory mean:  0.009175121384696406
Task instances memory std:  0.002757219144923028
Task instances duration mean:  74.68815165876777
Task instances duration std:  45.40343044250821
680 0.45179080963134766 62.05685685072286 1.4292964198242561
********** Iteration 0 ************
2021-06-27 23:37:36.427508: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
Jobs number:  10
Tasks number: 93
Task instances number mean:  45.376344086021504
Task instances number std 85.48783652126134
Task instances cpu mean:  0.5264810426540284
Task instances cpu std:  0.10018140357202873
Task instances memory mean:  0.009175121384696406
Task instances memory std:  0.002757219144923028
Task instances duration mean:  74.68815165876777
Task instances duration std:  45.40343044250821
680 0.4897310733795166 62.05685685072286 1.4292964198242561
Traceback (most recent call last):
File "D:/2021/Bin-packing/deepjs/CloudSimPy-master/playground/Non_DAG/launch_scripts/main-makespan2.py", line 132, in
train()
File "D:/2021/Bin-packing/deepjs/CloudSimPy-master/playground/Non_DAG/launch_scripts/main-makespan2.py", line 96, in train
p.start()
File "D:\anaconda3\envs\deepjs\lib\multiprocessing\process.py", line 105, in start
self._popen = self._Popen(self)
File "D:\anaconda3\envs\deepjs\lib\multiprocessing\context.py", line 223, in _Popen
return _default_context.get_context().Process._Popen(process_obj)
File "D:\anaconda3\envs\deepjs\lib\multiprocessing\context.py", line 322, in _Popen
return Popen(process_obj)
File "D:\anaconda3\envs\deepjs\lib\multiprocessing\popen_spawn_win32.py", line 65, in init
reduction.dump(process_obj, to_child)
File "D:\anaconda3\envs\deepjs\lib\multiprocessing\reduction.py", line 60, in dump
ForkingPickler(file, protocol).dump(obj)
File "D:\anaconda3\envs\deepjs\lib\site-packages\tensorflow\python\framework\ops.py", line 763, in reduce
return (convert_to_tensor, (self.numpy(),))
File "D:\anaconda3\envs\deepjs\lib\site-packages\tensorflow\python\framework\ops.py", line 742, in numpy
raise ValueError("Resource handles are not convertible to numpy.")
ValueError: Resource handles are not convertible to numpy.
This bug occurs. Can anybody help me with this?
Thanks!!!

请问解决了吗？我也遇到了相同的问题。

兄弟们解决了吗？


tooling: Add macro for generating templated content

This macros provides an easy way to generate templated content from a jinja template and json/yaml configs
Signed-off-by: Ryan Northey<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

This only provides the macro. I intend to follow up with PRs to add a dockerhub readme #25550, improve the release messaging on the gh release page and a few other uses


fix #14: no auto Lightning prefund

Codecov Report

Merging #32 into master will decrease coverage by 0.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   90.83%   89.98%   -0.86%
==========================================
  Files          16       16
  Lines         611      609       -2
  Branches       29       29
==========================================
- Hits          555      548       -7
- Misses         48       53       +5
  Partials        8        8

Impacted Files
Coverage Δ

src/uplink.ts
90.47% <ø> (ø)
:arrow_up:

src/settlement/lnd.ts
95.91% <ø> (-0.17%)
:arrow_down:

src/services/switch.ts
81.33% <0%> (-8%)
:arrow_down:

src/utils/middlewares.ts
88.33% <0%> (+1.66%)
:arrow_up:

Continue to review full report at Codecov.

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

@rhuairahrighairidh Tests are passing... finally


Add support for ruby 1.9

Motivation
Some of the gems in the Calabash toolchain still support ruby 1.9

test-cloud-command-line
run-loop
Calabash 0.x iOS and Android

Calabash 2.0 requires ruby > 2.0
I would very much like to drop ruby 1.9 support, but I don't think I can yet - I would need to make a run-loop 2.0 release which I am not prepared to do yet.
I want to use this gem because it provides an out-of-the-box solution to a recurring problem .
@krukow What do you think about dropping 1.9 support?
I am concerned that by removing the *args I going to break some callers.  The tests pass on ruby 1.9 - 2.2.3, but I worry that there are some callers out there that might be affected.
@TobiasRoikjer Can you take a look?  Specifically at the change from *args to args...

@kamstrup I made you a Calabash owner so you can edit this PR directly.

Awesomesauce! Thanks Moody!


Zqs 1202 upgrade qiskit

Description
Upgraded qiskit to version .38. Some minor changes to qiskits optimize function was required.
Please verify that you have completed the following steps

[x] I have self-reviewed my code.
[ ] I have included test cases validating introduced feature/fix.
[ ] I have updated documentation.

Codecov Report
Base: 97.21% // Head: 97.55% // Increases project coverage by +0.33% :tada:

Coverage data is based on head (1a01f39) compared to base (74c8e8f).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
+ Coverage   97.21%   97.55%   +0.33%
==========================================
  Files          26       25       -1
  Lines         861      776      -85
==========================================
- Hits          837      757      -80
+ Misses         24       19       -5

Impacted Files
Coverage Δ

src/orquestra/opt/optimizers/qiskit_optimizer.py
100.00% <100.00%> (ø)

src/orquestra/opt/api/optimizer_test.py

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

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


[iOS] Replace LicenseList plugin with official one

Idea Description
I implemented LicenseList plugin in #479 , and offered support of SPM in https://github.com/cybozu/LicenseList/issues/4 .
And now, LicenseList officially supported BuildToolPlugin. So we should replace it.
Reference images and links

LicenseListPlugin

After created this issue, I notice this info.

In this project, these packages are local plugin (Local defined plugin) and the why of using local plugin is performance issue of Xcode. When using remote package plugin, Xcode CPU usage becomes so high (over 100%).

If LicenseListPlugin could have same problem, we can use only official release bundle.

Let me try this issue👀
I think this issue is not urgent, so I believe that I take enough time including my investigation into SwiftPM Plugin since Xcode14.😄

Sorry for late response 🙏🏼
@fummicc1
Thanks! Assigned 👍🏼


Bootstrap table css not included with _content/Radzen.Blazor/css/default.css

Bootstrap table css not included with _content/Radzen.Blazor/css/default.css
To Reproduce
Steps to reproduce the behavior:

Include default.css file which should include boostrap according to the Get Started page

Alternatively you can include  which embeds Bootstrap.

<link rel="stylesheet" href="_content/Radzen.Blazor/css/default.css">

Add the following to your page (Sample from bootstrap):

      #
      First
      Last
      Handle

      1
      Mark
      Otto
      @mdo

      2
      Jacob
      Thornton
      @fat

      3
      Larry
      the Bird
      @twitter

You can see the following result is not rendering bootstrap tables correctly:

This is by design - the Radzen themes do not include the complete Bootstrap CSS as they are not using most of it. Feel free to include Bootstrap in your app and use the base version of the theme as mentioned in the documentation.

This is by design - the Radzen themes do not include the complete Bootstrap CSS as they are not using most of it. Feel free to include Bootstrap in your app and use the base version of the theme as mentioned in the documentation.

@akorchev,
Thank you for clarifying and I resolved the issue by including complete Bootstrap.
May I suggest updating the documentation to reflect what Bootstrap capabilities are included with the default.css?
My assumption was that "embeds Bootstrap" meant includes full Bootstrap and it took me a while to figure out why tables weren't working. The clarity could help prevent others from making the same assumptions I did and save them some trouble.


Improves the display of reconciliation results

Fixes #6154
Changes proposed in this pull request:

Now, the no. of matched candidates that are visible after reconciliation is performed is set to a default of 3(which can be changed easily by some minor code changes as I have introduced a visibleCandidate variable in the code)
The user can see all the matched candidates by clicking on see more option, and go back to the previous no. by clicking on the see less option.
The see more and see less option would only be shown to the user if the no. of matched candidates are greater than 3(visibleCandidates).

The cypress tests fail weirdly, @wetneb can you look into this please-
1)After inspecting the tests, I found out that the test expects only 1 recon candidate in the 1st cell and 5 recon candidates in the last cell(but actually the no. of recon candidates is 6 in the first and last cells), while 6 candidates for 2nd to 5th candidates work perfectly fine.


TLS Certificate for camlistore.org expired

The TLS certificate for the HTTPS server at https://camlistore.org has expired on 17 June, could someone please renew it?

Fixed.
/cc @mpl


Updates for Monitor, Operational Insights and condition check for han…

Updates for Monitor, Operational Insights
Bug fix for handling zero non net core projects during build within a scope.

Merging as this is blocking other critical publish jobs.


Missing raycast export

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13523/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13523/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13523/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13523/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13523/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13523/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13523/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13523/merge#BCU1XR#0


Control of TXLED and RXLED

It looks like the production Arduboy has both a TXLED and RXLED wired on the same pins as the Arduino Leonardo, Micro and Pro Micro (and likely others). Therefore, I assume the Arduboy's  pins_arduino.h file will include the definitions for controlling these LEDs as others do. E.g.
#define TX_RX_LED_INIT	DDRD |= (1<<5), DDRB |= (1<<0)
#define TXLED0			PORTD |= (1<<5)
#define TXLED1			PORTD &= ~(1<<5)
#define RXLED0			PORTB |= (1<<0)
#define RXLED1			PORTB &= ~(1<<0)

#define LED_BUILTIN_RX 17
#define LED_BUILTIN_TX 30

For reference: Example 1: Blinkies!
Is this adequate to allow a sketch to control these LEDs or should we include library functions to make it easier?
Note that the RXLED is on the same pin as the DevKit's LED.

For reference: Example 1: Blinkies!

Note that, contrary to that article, you can control TXLED as digital Pin 30 (like RXLED is Pin 17). I think this was added in Arduino version 1.5+

What file are these in?  I didn't see them when I looked.  I think eventually we should have a nice API wrapper so we can hard code the PORTS and bit stuff rather than people using digitialWrite (which is super expensive).
I was planning on getting to it but hadn't decided an API yet... one method, two?  Boolean?  They aren't hardware PWMable.

What file are these in?

The file is pins_arduino.h which is located in a subdirectory of the variants directory for the board that you are using. The variants directory is somewhere under a hardware directory. The location of hardware differs depending on whether it's an official Arduino installed as part of the IDE or a third party board installed as an add-on. The locations are probably also dependent on the host O/S.

I'm very familiar with that file... it's a trusted companion... but I must have missed the TX/RX stuff... looking again now.

    void setRGBled(uint8_t red, uint8_t green, uint8_t blue); // already coded
    void setRXled(bool onoff);
    void setTXled(bool onoff);

And we just wrap those defines nicely with an API.  Thoughts?

The Arduino Style Guide for Writing Libraries states:

Use read() to read inputs, and write() to write to outputs, e.g. digitalRead(), analogWrite(), etc.

Which would mean using write instead of set.
void writeRGBled(uint8_t red, uint8_t green, uint8_t blue); // already coded
void writeRXled(bool onoff);
void writeTXled(bool onoff);

Or the more compliant postfix instead of prefix: RGBledWrite(), etc.?
And should we provide complimentary read functions?
And if we decide on this, then maybe readButtonState() or readButtonStatus()  (or the postfix versions) instead of the pollButtonState() or pollButtonStatus() that you are proposing in place of getInput().
Strangely, the guide says later:

Try to avoid boolean arguments. Instead, consider providing two different functions with names the describe the differences between them.

But maybe that refers to functions with multiple parameters?? Or non-I/O related functions??

Or the more compliant postfix instead of prefix: RGBledWrite(), etc.?

I don't see where it mandates the order. And pins are inputs and outputs, lights are lights.  They can be wired to pins yes, but those low level functions are for working with pins - and the concepts are named accordingly.  We don't write to the arduboy screen, we draw to it... higher level gets to name things higher level. :-)

And should we provide complimentary read functions?

I don't think that's necessary, if someone wants to do crazy animation or a tight loop that should probably be tracking the status themselves in a counter, not polling for it each time.  Plus RGB isn't an input that you can just poll... I'm pretty sure you could get those numbers back by reading the IO counter ports and inverting, but it's not analogous to analogRead() or anything - it's a whole other thing.
I think you could argue the boolean thing both ways. If you're doing that though you're not really "writing" anything anymore (even at the function level)... write(true) is maybe writing something... TXon() and TXoff() is just state toggling... I wouldn't call it writeTXon or TXonWrite, that's just silly.
So what about:
  ledTXon();
  ledRXoff();
  // ...

And lets remember it's a guide, not a Bible. :)

void writeRGBled(

While you might write the red inverse duty value, write the green inverse duty value, and write the blue inverse duty value when you do them all together now you're doing a new thing.  You're setting a color, a much higher level concept. :-)
I also like setRGBcolor(r,g,b).

And pins are inputs and outputs, lights are lights.

So what type of things do you think the guide is referring to when it says:

Use read() to read inputs, and write() to write to outputs

If it's not for creating more specific functions for things attached to pins, what are they referring to?

If it's not for creating more specific functions for things attached to pins, what are they referring to?

It's for writing to pins.  They couldn't name it any differently because they don't KNOW what you are attaching.  If you're attaching a printer you might want to print... if you're attaching a gun you might want to shoot... the higher level the function the higher level the concept.  Your api would be print("Bob") or shoot(5)... look at some of the other Arduino libraries, not just the guide.
if you're selling someone a board with 20 switches they might all say "switch one", "switch two", but someone is going to write "fridge", "washer", "drier" on the switches, and they'd be insane not to. :)

Look at the mouse api for example.  You don't write mouse status to USB, you click and move and press and release things.  The functions match the function.
I think you missed the very top of the guide:
Be kind to the end user. Assume you are writing an API for an intelligent person who has not programmed before. Come up with a clear mental model of the concept you’re working with, and the terms and functions you will use.

There are things you write and read, and then they are things you shoot, click, launch, boot, process, etc... and sometimes if you're at a higher level you are setting things, not writing them... search the entire Arduino source for set.*( and you'll find plenty of set, including setTX and setRX for one example, for setting those pins.

writeRGBled() or writeRGBcolor() read as clearly to me as setRGBled() or setRGBcolor. (Except, being Canadian, I would keep entering colour and have to go back and correct it to color :wink:)
Actually, thinking further, writeRGBcolor() (or set) is a bit vague. RGB alone doesn't really describe a specific object. If you're going to include color in the function name, it should also indicate that it's an LED, such as setRGBledColor(). So I'd just use setRGBled() without color. Besides, we're actually setting intensity as well as color. (We might even want to overload the function, at a later date, to allow HSV or HSL, or something else.)
From a user standpoint, I don't look at it as being three separate LEDs in one package. I see it as a single point of light that can be set to many colors and intensities, like a pixel on a display. So just like I write to a RAM location, I'm writing to a pixel or other light source.
Anyway, I personally don't care whether you use set or write, or prefix or postfix, or separate functions instead of boolean arguments, I was just pointing out what it says in the guide (at least, my interpretation). To me, set and write are synonymous, as are get and read. I've had no problem understanding any of them and all would be fine by me.

It's for writing to pins. They couldn't name it any differently because they don't KNOW what you are attaching.

The could have named them setDigitalPin(), getDigitalPinState(), setAnalogOutput(), getAnalogInput(). :wink:
OK, I'll shut up now :smile:

As I said they use set* in plenty of other places... it's all contextual.

@rogosher Any thoughts on:
 ledTXon();
 ledRXoff();

Any thoughts on:
ledTXon();
ledRXoff();

If the TXLED0, TXLED1, RXLED0, RXLED1 macros are kept in the the Arduboy version of pins_arduino.h, they will do same thing. @yyyc514, I know that you don't like this kind of obfuscation, but they're already documented and accepted by the Arduino community, so I don't have a problem with just using them directly.
Then again, adding the functions doesn't hurt, and they're a bit clearer than the macros. They won't add to code size if they're not used.
Personally I would prefer single functions (regardless of the actual name) with a boolean on/off parameter, regardless of if you interpret the style guide as not recommending them.
void setRXled(boolean on);
void setTXled(boolean on);

They can be used to force the LED to a desired state as easily as two separate functions, but it makes it a bit easier to restore a previously saved state:
void setRXled(savedLEDstate);

instead of
savedLEDstate ? ledRXon() : ledRXoff();

// or more likely for a programmer unfamiliar with using the conditional expression:
if (savedLEDstate == ON) {
  ledRXon();
} else {
  ledRXoff();
}

I'll think about it more. One advantage of functions (other than being easy to find in the Api docs) is that if you use them a few times they start to save you flash space. Those defines just get added to your code over and over again.
Not sure why arduino itself doesn't provide a helper for this. Maybe it assumes using the LEDs for serious stuff someone would know all this and act accordingly?

Not sure why arduino itself doesn't provide a helper for this.

Because the TX and RX LEDs were put there mainly to provide USB communication status. Their use for other purposes isn't common, and possibly not even encouraged.

Except we are really custom hardware - and they have no purpose at all when not connected to USB. But I guess I see why they didn't encourage going crazy with them. I could see going either way on this.

OK, I'm in in the air now - does anyone else have an opinion on whether the TX/RX should be included?

It doesn't matter much to me if they're included or not, but as I've already said, if you do add them, I'd prefer two functions that accept a boolean argument instead of four separate on only or off only functions

I think I'm going to side with Arduino for now.  These are hardware LEDs for TX/RX and if someone wants to use them it's also easy enough to use the provided defines.


Guzzle HTTP instead of plain CURL

It's really good that you are writing PHP API library for Shopify.
I want to suggest to you to use Guzzle HTTP which is advanced HTTP request library instead of reinventing the wheel with custom CURL classes and etc.

Hey @sv3tli0 - thanks for the feedback :) We'd like to support Guzzle alongside cURL so you can choose either one. I'll leave this issue open until we add Guzzle as an option.

Guzzle is initially based on curl, that is why I recommend it :)

Hi @sv3tli0,
I asked myself the same question when I first saw this repository. If we check other famous companies Amazon, Google and Facebook, we'll see that they're all using Guzzle. Unfortunately looks like Shopify doesn't have a dedicated PHP team to help out so I would say that it's up to the community to take some initiative. Using guzzle would require a big chunk of refactoring here. The good point is that the API is still small so if @joshubrown gives the thumbs up for this action, it would be easier to work on this now before the API gets more code.
@joshubrown Sorry to disagree but I don't think that providing both Guzzle and non-Guzzle would work. I know that you're pretty busy with Webinars and I'm not sure if you guys have more hands to work on this API there at Shopify.
I'm pretty busy as well but I'd love to help out on this refactoring to use Guzzle instead.

https://github.com/facebook/php-graph-sdk/blob/5.5/composer.json#L20
https://github.com/aws/aws-sdk-php/blob/master/composer.json#L20
https://github.com/google/google-api-php-client/blob/master/composer.json#L15

@rodolfobandeira if @joshubrown agree's I'll be willing to help as long Shopify supports the maintenance of this.

I gonna help too

ping @joshubrown

OK, you are convincing me :) Thanks for the PR @rodolfobandeira! I will review it this weekend.

up

@joshubrown Hi, Will the pr will be merged?

Hey @joshubrown, Looks like you're not working at Shopify anymore? Are you still maintaining this repository?
Hey @slince, The PR I made is just a kick start on using Guzzle. A good amount of code is necessary to replace the existing classes to use Guzzle instead of raw CURL. Kind of involves some re-design and architecture in order to do that. I've been trying to find some spare time to help out but it's hard. Maybe I can get back to this in December.

@rodolfobandeira  @joshubrown  @yurivecchi  @all
Hi i have created a new repo https://github.com/slince/shopify-api-php,  Hope it is more reliable than this repo;  :yum:

Hi @rodolfobandeira, that's correct I'm no longer at Shopify. I did chat with them about the repo and there should be someone new taking a look here soon. Thank you @slince for your interest and contributions!


TransactionView

Problem

VersionedTransaction deserialization is incredibly costly. One of main reasons if because the struct forces us to have several internal allocations.

Summary of Changes

Add a new TransactionView which can either:

copy data into a Boxed array
take ownership of an already Boxed array

The TransactionView simply calculates offsets into the data, and stores those offsets which can later be used for accessing fields

Fixes #

Going to split this into a few PRs so that these items can be more carefully reviewed in small chunks.


Remove cheatsheet.md

Much of the content in docs/source/cheatsheet.md already exists in Plone 6 Documentation and other resources. This file should be deleted.

please
Kindly assign me that issue.
Best Regards,
Aman gupta

@Amangupta7011 no. See https://6.docs.plone.org/contributing/first-time.html#things-not-to-do.


when JS SDK of this project will update ?

Hi, can you tell me when you will update to new JS sdk of twilio ?

Hi @mtariqsajid, Twilio Developer Educator here.
Unfortunately, I don't have an exact date for you, but this project is at the top of my list for updating.
In the meantime, the Voice JS SDK Quickstart for PHP is updated to use Version 2, so that can serve as a guide for you.  The Migration Guide may also help.

Looking forward for the update
Thank you for the update


Additional curation target types

Would be interesting to support (or explicitly type) curation of

transaction hashes: imagine it would be quite interesting to view a collection of novel transactions (fund transfers) deploys, etc. 🤷♂
Block number... again not exactly sure why, but this seems to be an interesting data point that could be collated in compelling ways.

https://github.com/public-assembly/curation-protocol/blob/main/src/interfaces/ICurator.sol#L8-L21

so I think this would be really cool and totally represent meaningful blockchain data to curate, however there is an issue with adding this functionality for a few reasons:

see screenshot below for what information is stored in the listing struct. as of now, all Curation Types have an address component (line 26 of ICurator.sol). as listings allow u to curate bytes data (transaction hashes) or uints (block numbers), we would need to allow for the curation of other solidity data types, which would require foundational changes into how data is stored.

this could potentially be addressed by extracting curation types + listing structs out of the curation protoocl and into external onchain registries (like we did for access control), but that is a significant rewrite of the current architecture

am very open to this, but initial thinking at solution would require something pretty significant


gamified_tree folder added

I've added section successfully, plz accept the pull request

Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.


[buildkite] Update S3 credential handling

We were alerted that our S3 credentials had leaked in a build that dumps the environment when it errors out [0].  This PR rotates to a new key, and also wipes out the key in the environment during the build itself, so that important values won't get written out in debugging information like that.
[0] https://github.com/SciML/SciMLBenchmarksOutput/blob/8d80b1840228c63e8c6fdc597049211a8b916dea/markdown/DynamicalODE/single_pendulums.md

Looks like it's working!


⚠ Bofrost.nl has degraded performance

In 74b992f, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 3199 ms

Resolved: Bofrost.nl performance has improved in c8e8a9b after 2 hours, 24 minutes.


[feature] support Unsupported() types

schema.prisma
generator kysely {
  provider           = "prisma-kysely"
  output             = "../../build/prisma/primary"
  fileName           = "types.ts"
  unsupportedFilename = "unsupported-types.ts"
}

model Payment {
  id     BigInt                              @id @default(autoincrement())
  amount Unsupported("money_with_currency")?
}

types.ts (generated by prisma-kysely)
import UnsupportedTypes from '~/unsupported-types';

export type Payment = {
    id: Generated<bigint>;
   amount: UnsupportedTypes['money_with_currency'] | null;
}

unsupported-types.ts (defined by us manually)
export type MoneyWithCurrency = {
  amount: bigint;
  currency: CurrencyCode;
}

export const Unsupported = {
  'money_with_currency': MoneyWithCurrency
} as const;

Would this be possible? We're currently stuck with this and realised that Prisma wouldn't support custom type just for Postgres use case. Would be great if we can make this happen. Thanks.

Prisma side feature request to put Unsupported into DMMF so it is available for generators: https://github.com/prisma/prisma/issues/20756

thanks @janpio for linking that, this is my issue I opened last week. I'm looking for an alternative in the meantime to deal with this as it could be sometime before this is added to the DMMF

(Your username did seem familiar 😆)


Connector for MQTT

Big +1 for MQTT support.
Is this on the roadmap?
I am wondering how this can be embedded into NATS? Does it have some kind of pluggable architecture?
@mcqueary can you please provide a few details/description of the idea?

@drasko Thank you for your interest!  This is something we are gathering requirements about.  Could you provide more information about your use case?

@ColinSullivan1 we are currently using NATS in our https://github.com/Mainflux/mainflux project. This is a messaging bridge for IoT.
Devices today connect to cloud via low-overhead protocols like MQTT or CoAP. NATS do not support neither type of connection.
Use-case would be following: http://scn.sap.com/community/pi-and-soa-middleware/blog/2016/02/21/uniting-amqp-and-mqtt-message-brokering-with-rabbitmq
Need that devices connect via MQTT demands the MQTT broker in the system. Since MQTT broker is present, then NATS role is somehow redundant, as pub/sub can be replaced by MQTT broker, and today there are very scalable and performant MQTT brokers. Here is an example of one: http://emqtt.io/ and yet another one: https://vernemq.com/

We would like to use MQTT / NATS bridge for use with Aither Micro Services

@kristianmandrup this bridge is not so hard to implement and we at Mainflux are using Aedes MQTT broker to bridge it with NATS - take a look here: https://github.com/mainflux/mainflux-mqtt/blob/master/mainflux-mqtt.js
Using this appoach we were able to bridge various protocols, like on the picture here: https://github.com/mainflux/mainflux, and also persist data which is published via NATS and stored into InfluxDB - https://github.com/mainflux/mainflux-influxdb-writer
Having MQTT broker independant from NATS let's you choose your favorite broker. We are currently experimenting with NodeJS and Erlang (https://github.com/mainflux/emqttd-docker) implementations.

Sweet :)

Whilst I agree that having a broker outside of nats is possible, it's also another moving part to go wrong, to monitor and to maintain.  +1 for having a native possibility of connecting via mqtt.


Update shell docs, build steps, and tooling scripts.

I had to:
% cd shell/components/strategy-explorer
% bower install

to fix strategy explorer throwing Polymer missing error on app shell load with dev tools open.
Related, we also need to update shell/README.md, tools/install and tools/test to do whatever is needed for the shell.
FYI @sjmiles @smalls

I'm thinking about adding that build step to shell's package.json-based build. There's some work in #703 and #687 that'll obviate the need for that, but it should be easy and it'll make our build more consistent.
@shaper @sjmiles thoughts?

Sounds ok to me. I'm wondering why we didn't have this issue previously but it'll go away soon anyway with the changes you noted.


add test for writing fgb and loading it after with FgbWriter and FgbR…

Adds a failing test that writes a FGB file using FgbWriter and attempts to read the file back in using FgbReader.
Writing is successful but reading is not returning an error IoError(Error { kind: UnexpectedEof, message: "failed to fill whole buffer" })
I have noted that files created using the FgbWriter have resulted in features counts double what they should be, it is possible I am not correctly setting the header features_count and the header is corrupt causing the file to load incorrectly.
The file does load with GDAL and shows the doubling of features count. These extra features are filtered out in the attribute editor of QGIS, so they seem to partially exist?
@pka I hope this PR helps. My Result handling could do with some work.

Updated PR to include suggested fixes. Test passes.

No worries! Glad I can help.


should we make the code ready to copy and paste for reuse?

originally suggested by @adamkucharski
one approach would be to have:

runnable chunks
with no intermediate outputs to be printed and
with "the" table or figure at the end as proof of the capacity of the script to solve the problem

other one is encouraging to clone and reuse from repo. Actually, from the template we expect all "steps in code" to be within one chunk only.

reference:

https://www.reconverse.org/incidence2/articles/incidence2.html#computing-incidence-from-a-linelist
https://github.com/reconverse/incidence2/blob/fa764c65452b4ebfc28245299900717367599254/vignettes/incidence2.Rmd#L10-L19
or simpler, remove the code lines that call the printing

alternatively, @adamkucharski proposes to use tabset panels to show output code in one tab, and ready to copy and paste in a second tab.

Making it copy-pasteable is a great idea! For the latest quarto version, it only requires an update to the _quarto.yml file: https://github.com/quarto-dev/quarto-web/blob/1ef8c3c4ba444687a27513aa6a63076b934927df/_quarto.yml#L648
For documentation with code, I recommend taking a look at the Stripe docs. They are some of the best I've seen: https://docs.stripe.com/api
If you are interested in reproducible/executable code within the browser, it might be a great test case for webR. If that is of interest we could do a trial run and report back what we learned.

If I understand correctly, the current issue is that you need a way to concatenate all the code chunk into a single chunk, isn't it? Then, it's doable with a knitr hook and we can have it in a <details> section at the end of the page:
Full code
# Here is the entire code to copy/paste

thank you for both replies.

Making it copy-pasteable is a great idea! For the latest quarto version, it only requires an update to the _quarto.yml file: https://github.com/quarto-dev/quarto-web/blob/1ef8c3c4ba444687a27513aa6a63076b934927df/_quarto.yml#L648

Nice feature. Hopefully the copy button in quarto is enabled by default

If I understand correctly, the current issue is that you need a way to concatenate all the code chunk into a single chunk, isn't it? Then, it's doable with a knitr hook and we can have it in a <details> section at the end of the page:

this looks doable. so, If we name a chuck, like this whole chunk, we could make it visible at the end?
the issue
To rephrase the issue, intermediate outputs are shown out of the chunk, cutting the content that the copy button covers. While on package websites output is printed within the chunk (looking like a reprex output)
My question is, what style or knit options feature could we replicate from {pkgdown} to replicate this behaviour in a quarto website? I though this could be more easy to implement, but the easiest you suggest is welcome!
what we want

what we have

an expected result of the improvement would be
demography_vector
#> [1]  3453670  9761554 18110368 19288101  9673058

demography_groups
#> [1] "[0,5)"   "[5,18)"  "[18,40)" "[40,65)" "65+"

If you want to replicate the pkgdown behaviour, this is the relevant line:
https://github.com/epiverse-trace/ColOpenData/blob/ab7ac6f27c5ef811c59fddbc2dd40c552b10684b/vignettes/demographic_data.Rmd#L15

You're right! Looks like collapse = TRUE is necessary as well for the desired effect: https://github.com/epiverse-trace/ColOpenData/blob/ab7ac6f27c5ef811c59fddbc2dd40c552b10684b/vignettes/demographic_data.Rmd#L13-L16

You're right! Looks like collapse = TRUE is necessary as well for the desired effect: https://github.com/epiverse-trace/ColOpenData/blob/ab7ac6f27c5ef811c59fddbc2dd40c552b10684b/vignettes/demographic_data.Rmd#L13-L16

ok, done, I thought that collapse action was to fold the code chunk, haha, thanks!


Cannot read property 'hideMessageBarAlert' of undefined

Whenever I trigger an alert nothing seems to happen and I get this:
Possible Unhandled Promise Rejection (id: 0): Cannot read property 'hideMessageBarAlert' of undefined TypeError: Cannot read property 'hideMessageBarAlert' of undefined     at Object.module.exports.hideAlert (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:16093:29)     at Object.module.exports.showAlert (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:16071:6)     at _callee2$ (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:80269:512)     at tryCatch (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:5108:30)     at GeneratorFunctionPrototype.invoke [as _invoke] (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:5384:12)     at GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:5141:13)     at tryCatch (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:5108:30)     at invoke (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:5184:12)     at http://localhost:8081/index.ios.bundle?platform=ios&dev=true:5229:1     at tryCallTwo (http://localhost:8081/index.ios.bundle?platform=ios&dev=true:2626:1)

Hi,
Can you please paste your code and can you explain a bit your context?
Which version do you use?
Can you also please check you implemented the following step:

Add a  to your top parent view/component. (Children will use that one)
Register and unregister the alert in, respectively, componentDidMount and componentWillUnmount
Call MessageBarManager.showCurrentAlert({...}) with the right properties

Thanks,

Version<EMAIL_ADDRESS>var MessageBarAlert = require('react-native-message-bar').MessageBar; var MessageBarManager = require('react-native-message-bar').MessageBarManager;
...
`
componentDidMount: function() {
   MessageBarManager.registerMessageBar(this.refs.alert);
        MessageBarManager.showAlert({
          title: 'Success',
          message: 'Job has been marked as produced',
          alertType: 'success',
          // See Properties section for full customization
          // Or check `index.ios.js` or `index.android.js` for a complete example
        });

},
componentWillUnmount: function() {
  // Remove the alert located on this master page from the manager
  MessageBarManager.unregisterMessageBar();

},
render: function() {
return (

            {toReturn}
            <FooterBar style={gStyles.footerBar} coreRoute={this.state.coreRoute} root={this}></FooterBar>
            <MessageBarAlert ref="alert" />
        </View>

   );

}
`
I can successfully get the contents of MessageBarManager and MessageBarAlert when I put them through a console.log in my render function - so I know they are accessible and in scope.

Ok so I think I may be closer. My this.refs is empty - pretty sure that's the source of the problem. Looks like this can happen - I'm trying to figure out why. I'm going to call this issue closed for now.  See: https://github.com/root-two/react-native-drawer/issues/55

Yeah ok so if you are using a navigator component in any way (even ExNavigator like I'm using) this will wipe your refs object.
I fixed my problem by wrapping a new view around my MessageAlertsBar and the navigator and this brought my refs back. Now it works.

just ran in to a similar situation but had a loader before rendering which messed things up as well
error
    if(!this.state.loaded) return this.renderLoader();
    return <View style={{flex:1}}>
        {this.renderApp()}
        <MessageBarAlert ref="alert" />
    </View>

works
<View style={{flex:1}}>
        {(this.state.loaded) ? this.renderApp():this.renderLoader()}
        <MessageBarAlert ref="alert" />
</View>

@bfarrgaynor I reproduce the bug and the only way I fixed it for now is to do a
MessageBarManager.hideAlert();
Actions.mypage();

@jaywalklabs this is normal, in your first try the MessageBarAlert component is not loaded in the DOM, you can't then access and use it. Only your second version can work if you want to access the MessageBarAlert while the loading is performed

I'm having this problem with react-native Navigator. I'm not sure how to resolve.

I made the object global by declaring it into a global variable in my index time and am able to call it from any route.

From: Eamonn Hynes<EMAIL_ADDRESS>Sent: Sunday, February 5, 2017 5:18:02 PM
To: KBLNY/react-native-message-bar
Cc: Brendan Farr-Gaynor; Mention
Subject: Re: [KBLNY/react-native-message-bar] Cannot read property 'hideMessageBarAlert' of undefined (#7)
I'm having this problem with react-native Navigator. I'm not sure how to resolve.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/KBLNY/react-native-message-bar/issues/7#issuecomment-277554482, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABKFjLZj6iq6tbnVavPRmlQ4fZNtLP4Eks5rZkqagaJpZM4IK_JE.

OK, so I think when using Navigator, there are alerts appearing/disappearing on other views.
I managed to clean up the code, but I can't pin-point the exact source of the error message.

@bfarrgaynor i am using react-navigation. can you share a gist or something showing how you did that global variable and call please?

@bfarrgaynor nvmnd got it working. i was not correctly rendering to keep the refs. works great now.

I'm having the same issue, does anyone have a good example

@AlmogRnD can you share your code to show where you are getting an error?

@temitope I switched to a different message bar package

@AlmogRnD gotcha. thx for update

@temitope
What different message bar do use? This package is real PITA 👎

@wzup fwiw i forked it. its simple enough to not have to be such a PITA but maintenance - https://github.com/temitope/react-native-message-bar

@AlmogRnD What package you switch?


[Teva 1790] add radio/checkbox component variants

https://dfedigital.atlassian.net/browse/TEVA-1790

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

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

Instead of having two different components that are essentially exactly the same, I think we could probably have one component and use .with_variant to render a different template depending on whether it is checkboxes or radios

Instead of having two different components that are essentially exactly the same, I think we could probably have one component and use .with_variant to render a different template depending on whether it is checkboxes or radios

Can you provide screenshots, as the orgs available on the review app don't have >10 schools?


Bug: ADT post-processing fails when cleanTagCounts assumptions are not met

After the v0.5.2 release 😞 , we have observed that a small amount of datasets fail during post_process_sce specifically because of ADT quality control statistics.
In certain (infrequent!) circumstances, the assumptions of cleanTagCounts(), which uses ambientContibSparse() in the absence of isotype controls, are not met (source).
From the source:

The assumption here is that of sparsity, i.e., no more than \code{prop * nrow(y)} features should be actually present in each cell with a non-zero number of molecules.
This is reasonable for most tag-based applications where we would expect only 1-2 tags (for cell hashing) or a minority of tags (for general CITE-seq) to be present per cell.
Thus, counts for all other features must be driven by ambient contamination, allowing us to estimate a scaling factor for each cell based on the ratio to the ambient profile.

So, we'll need to get a patch in for this. We have discussed taking the following strategy -

If discard has NA values, instead use zero.ambient for filtering: cells get "Keep" if  zero.ambient is TRUE and "Remove" if it is FALSE.
If both discard and zero.ambient have NA values, we might just fail on the post-processing altogether? Or, open for discussion here!

Worth noting that zero.ambient is always returned by cleanTagCounts() regardless of whether any isotype controls are present, so this strategy can be used universally.
CC @jashapiro @allyhawkins

So, we'll need to get a patch in for this. We have discussed taking the following strategy -

If discard has NA values, instead use zero.ambient for filtering: cells get "Keep" if  zero.ambient is TRUE and "Remove" if it is FALSE.
If both discard and zero.ambient have NA values, we might just fail on the post-processing altogether? Or, open for discussion here!

Just for clarification, is this on a cell-by-cell basis? I would expect based on your description that if one discard is NA they all are, but I still might implement this so that we can accommodate NA if it occurs in just some cells. Though as I write that I'm not confident I like it.

Just for clarification, is this on a cell-by-cell basis?

Yes, it's cell-by-cell. My description may have been misleading, sorry! If one is NA, it's not a guarantee that others are also; there are NA, FALSE, and TRUE all in one library.

Fixed and closed by #379


make run target not working when running on GKE

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
make run target seem to not work when running GKE
$ make run
<EMAIL_ADDRESS>rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases
<EMAIL_ADDRESS>object:headerFile="hack/boilerplate.go.txt" paths="./..."
go fmt ./...
go vet ./...
<EMAIL_ADDRESS>build config/crd | kubectl apply -f -
customresourcedefinition.apiextensions.k8s.io/controlplanes.gateway-operator.konghq.com created
customresourcedefinition.apiextensions.k8s.io/dataplanes.gateway-operator.konghq.com created
customresourcedefinition.apiextensions.k8s.io/gatewayconfigurations.gateway-operator.konghq.com created
kubectl kustomize https://github.com/kubernetes-sigs/gateway-api.git/config/crd?ref=main | kubectl apply -f -
customresourcedefinition.apiextensions.k8s.io/gatewayclasses.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/gateways.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/httproutes.gateway.networking.k8s.io created
CONTROLLER_DEVELOPMENT_MODE=true go run ./main.go --no-leader-election
INFO: development mode has been enabled
INFO: leader election has been disabled
1.65732348903128e+09	ERROR	Failed to get API Group-Resources	{"error": "no Auth Provider found for name \"gcp\""}
sigs.k8s.io/controller-runtime/pkg/cluster.New
	/Users/jarek.mroz@konghq.com/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.3/pkg/cluster/cluster.go:160
sigs.k8s.io/controller-runtime/pkg/manager.New
	/Users/jarek.mroz@konghq.com/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.3/pkg/manager/manager.go:322
github.com/kong/gateway-operator/internal/manager.Run
	/Users/jarek.mroz@konghq.com/git/gateway-operator/internal/manager/run.go:84
main.main
	/Users/jarek.mroz@konghq.com/git/gateway-operator/main.go:71
runtime.main
	/opt/homebrew/Cellar/go/1.18.3/libexec/src/runtime/proc.go:250
unable to start manager: no Auth Provider found for name "gcp"
exit status 1
make: *** [run] Error 1

Expected Behavior
Should work.
Steps To Reproduce
1. Setup google cloud cluster
2. Run `make run` command

Kong Ingress Controller version
No response
Kubernetes version
No response
Anything else?
No response

Needs checking if this is related to https://github.com/Kong/kubernetes-testing-framework/issues/345

This happens because gcp is deprecating (> v0.25.0) the gcp auth plugin to auth (https://cloud.google.com/blog/products/containers-kubernetes/kubectl-auth-changes-in-gke).
If you use gke-gcloud-auth-plugin it works without changing the codebase. For this you will need to update the credentials using the plugging as follows.

Install the pluging gcloud components install gke-gcloud-auth-plugin
flag export USE_GKE_GCLOUD_AUTH_PLUGIN=True
gcloud container clusters get-credentials [your-cluster]

This will update the ./kube/config to use gke-gcloud-auth-plugin to auth against the cluster removing the error.

I recommend that we document (for now, just in the README.md is fine) the steps needed to run this on GKE and consider that resolution for this issue.

Confirmed that when using the new plugin, make run works properly with a GKE cluster.


Support Angular 1.4

In bower.js, the angular version is hardcoded to 1.3.3. Is there a reason it's not supporting 1.4?

the latest release version (1.2.3) is modified to support angular >=1.3. there is no constraint on the versions it depends. It can work in angular 1.1, 1.2, 1.3, 1.4


Swift 3.0 Preview 1

(copied from open-swift/C7#46) Not sure what to set .swift-version to. After installing Xcode 8.0, swiftenv versions returns 3.0 as the latest version. However, this will probably not work on Ubuntu, where it is swift-3.0-preview-1.
Lets get this merged asap regardless :)

S4 and C7 are compiling for Preview 1 without any issues.

I'll update Travis with the zewo .travis.yml that we're using. It works but it's static at preview 1.


Capture output, reports, and document the view (for small files) extract view

Capture the small file execution output.
Capture the reports.
Document the view and the results and reports.

View 10700 in SAFRWBTR environment 4 (DM12).


fix: update deps that were causing tests to fail

:tada: This PR is included in version 1.22.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Implement Task.timed

@guizmaii has mentioned on Twitter that he's often missing following method:
final def timed: Task[(Duration, A)]

See Scalaz-ZIO
Task has methods like deferAction, async0 and cancelable0 which allow access to the Scheduler (for clock) so it should be very easy to implement.
@guizmaii
Do you still want to make PR or you don't mind if someone else takes this?

Hi @Avasil
I still want to do it but if someone does the job it's ok for me.
I'll be in a train all the day tomorrow, going to Scotland. I'll try to push you a PR. :)

Hi @Avasil and @alexandru,
I've pushed an initial implementation here https://github.com/monix/monix/pull/715.
Let me know your thinking about it.


Please invite me to the community

Name
super0111
Additional Context
I hope to join for learning new programming technology

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is an open source solution for serverless platforms, it is designed for developing solutions and enhancing creators to create their infrastructure 💻 Great to have you in our community ✨ Looking forward to your valuable contributions! 🤓 You can also check our community ✨ Github ✨ LinkedIn 💜 You can also share our community with other folks who want to learn about development, DevOps, tech, and contribute together and with us 👉💜👈


ShowDriverAMapForARequest

Are we talking about giving the driver a map once he's accepted a request to navigate to the user?
Using intents for now?

Yes and yes. Probably a "see route" button

Moving to Part 5 to do work with static map images.

requirement label removed, moving this to a "do if there is time"

Removing people assigned, this has been pushed back to project till project part 6 until further notice


fix fragdepthblock issue

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15204/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15204/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15204/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15204/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15204/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15204/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15204/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15204/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15204/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15204/merge/testResults/webgpuplaywright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15204/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Don't start update activity using startActivityForResult

The home activity does not need to read any result data from the update activity so use startActivity instead of startActivityForResult

@phillipm Can we close this? Sounds like the logout logic needs to get added back in, which means this setup would stay

No, this PR is still valid. In a future PR I will logout directly from the update screen when the user chooses to perform the update installation. That action will also not need to use startActivityForResult

Ok, will merge then. Sounds like we should definitely get that other PR into 2.25 though.


New property to enable/disable the Portal service directory

Based on Esri best practices, I have added a new property to the ArcGIS_Portal resource to enable or disable the services directory. I noticed that some of the existing properties such as EnableAutomaticAccountCreation will only get evaluated during Test-TargetResource if they are set to $true which means that if DSC is used to switch these settings back and forth when needed instead of just at initial configuration time, they would not get changed based on the configuration property values. To get around this issue, I used a slightly different logic.
Also updated an incorrect variable name that was causing output to not show correctly.

@PleaseStopAsking I don't see the relevant logic for 'EnableAutomaticAccountCreation' issue. Will you be creating another PR for it?

@shailesh91 I should have worded my initial comment different now that read it again. I did not touch the existing logic for 'EnableAutomaticAccountCreation'. I just used different logic for the new property.
The existing logic for 'EnableAutomaticAccountCreation' is
if ($result -and ($EnableAutomaticAccountCreation)) {...}
which only gets called when 'EnableAutomaticAccountCreation' is set to $true. If you change this to $false and try to rerun the resource, it will skip this as it only accepts $true which I believe is the default behavior of PowerShell when you evaluate a variable for $true. This can be tested by using the following example which always returns 'yes' as $result is $true
$result = $true
if ($result) {
    write-host 'yes'
} else {
    write-host 'no'
}

To get around this, I chose to do the following, which allows the property to change the setting successfully back and forth between $true and $false.
if ($result -and ($null -ine $DisableServiceDirectory)) {
Hopefully that makes sense.

@PleaseStopAsking Makes sense. Will test this once and merge soon.


Gameification: "Success" Badges and other motivational issues

I know there are more gameification feature requests open, but I haven't found one which adresses this feature. If so, please mark it as duplicate.
Feature description
Provide a "Success" Badge feature which give the user badges for performed tasks, e.g.

shared the app with 5 contacs
shared the app with 10 contacts
Triggered 5 downloads via the share feature
Triggered 10 downloads via the share feature
Used app for X days in a row
Uploaded a positive test (yes, this is a little bit morbit)
Uploaded a negative test
...

Problem and motivation
One of the main issues of the app is the low usage rate, of the app itself but as well of the uploads. We know from a lot of experience, that those little nudges help to motivate people... it's not much work to implement but might help to spread the app....
Is this something you're interested in working on
Happy to help with the Spec, coding has to be done by someone else.

Related Issues:
#248
#291
Internal Tracking ID: EXPOSUREAPP-6161

Related Issues:
#248
#333
#332
#291

Also display:

Total contacts registered
Contacts with negative test encountered
Low risk contacts  with positive tests met
High risk contacts met(implemeted)
People love statistics

@DanAmt
See #5. The app does not know how many non positive contacts the user had. Low risk contacts, like high risk contacts, are shown in the app.

Where are these contacts shown ?
How so ? All contacts must be collected by the OS.

Are the definitions of a low and high risk contacts open somewhere. I think
app should be as little black box as poss
Tim @.***> schrieb am Fr. 2. Apr. 2021 um 11:20:

@DanAmt https://github.com/DanAmt
See #5 https://github.com/corona-warn-app/cwa-wishlist/issues/5. The
app does not know how many non positive contacts the user had. Low risk
contacts, like high risk contacts, are shown in the app.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/corona-warn-app/cwa-wishlist/issues/409#issuecomment-812448450,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHTLPT4YO4RHGQP3W4CELFLTGWD5TANCNFSM4Z4CQLAQ
.

--
D https://keys.mailvelope.com/pks/lookup?op=get&search=0x1AC8D96B3C4A2D8E
aniel

Where are these contacts shown ?

Low risk contacts are shown in the app on the main page and inside of the contact diary. Take a look at the screenshots on https://www.coronawarn.app/en/screenshots/.

How so ? All contacts must be collected by the OS.

Correct, all contacts are collected by the OS, but the OS only hands the positive encounters over to the app.

Are the definitions of a low and high risk contacts open somewhere. I think
app should be as little black box as poss

Take a look at https://www.coronawarn.app/en/faq/#encounter_but_green, https://www.coronawarn.app/en/faq/#encounter_criteria and https://www.coronawarn.app/en/faq/#encounter_19_calc.

@Velligis128
Thanks. Internal Tracking ID: EXPOSUREAPP-6161

@Ein-Tim We got a few issues on gamification, incentives and related. I suggest opening a new overview issue, what do you think?

Thanms @Ein-Tim , checking this.

@dsarkar
I'll create a separate table in the statistics overview by todays evening, hope thats fine for you 👍
Have nice easter days everyone!

@Velligis128
Thanks. Internal Tracking ID: EXPOSUREAPP-6161

Thanks for pushing it. If i can help, please let me know

Also display :
* Total contacts registered by this user

* Contacts with negative test encountered by this user

* Low risk contacts  with positive tests met

* High risk contacts met(implemeted)
  People love statistics

I would support this but I think there si already a issue asking for things like that, in addition with a comparision to the average in the area. (The comparision is a bit tricky due to the data protection restrictions within the app)

Can you reference the issue ?

@DanAmt
From you #248


Feature request: Work on Chrome New Tab Page (NTP)

Recently Chrome changed their new tab page and Stylebot no longer works there.
I relied on Stylebot to remove the most visited sites (speed-dial).
"www.google.com**newtab": {
    "_enabled": true,
    "_rules": {
      "#most-visited": { "display": "none" },
      "body": { "min-height": "316px" },
      "html": { "overflow-y": "scroll" }
    }
}

Can Stylebot ever style those internal pages? Currently it's chrome-search://local-ntp/local-ntp.html. Any workarounds? Removing that Chrome "feature" was a really nice feature.

Works again, sorry to bother


Remove duplicated line

What changed?
Removing duplicated line

Why?

How did you test it?

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 0188d893-e8aa-4e0b-9f7b-ea02db1066bf

0 of 0   changed or added relevant lines in 0 files are covered.
104 unchanged lines in 18 files lost coverage.
Overall coverage decreased (-0.004%) to 57.295%

Files with Coverage Reduction
New Missed Lines
%

common/persistence/executionManager.go
2
78.94%

common/persistence/nosql/nosqlplugin/cassandra/tasks.go
2
75.57%

common/persistence/statsComputer.go
2
95.71%

common/util.go
2
53.73%

service/history/handler.go
2
47.15%

service/history/task/transfer_active_task_executor.go
2
72.24%

common/log/tag/tags.go
3
49.04%

common/persistence/nosql/nosqlTaskStore.go
3
61.07%

common/types/history.go
4
45.35%

service/history/historyEngine.go
4
68.48%

Totals

Change from base Build 0188d7c9-fb87-419e-b893-61f23adef50c:
-0.004%

Covered Lines:
87118

Relevant Lines:
152051

💛  - Coveralls


refactor(ast_tools): remove dead code in AstBuilder generator

ReferenceFlags no longer appears in the AST.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6757  👈
#6756
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

Tiny change, so merging without review.

Merge activity

Oct 21, 8:11 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


only_show_on not working for one_to_many relationships

The pull request  #36 has the only_sort_on, work for both many to many and one to many relationships
The commit 0429ee86ddccbdf16d9c9a649e3b34123a07c3cf changed this para-dime to only allow the only_sort_on and dont_sort_on to work with many to many relationships and and disables the feature for one-to-many relationships.
Normally one-to-many relationship would only sort on the specific model they relate to but my use case is a nested one to many
Comic has-many Chapters
Comic has-many Pages
Chapter has-many Pages
Inverse
Page Belongs-to one Chapter
Page Belongs-to one Comic
Chapter Belongs-to one Comic
Comic <- Chapter <- Page
Chapters and pages can be sorted, Chapters should only be sorted when viewed under a comic, and pages should only be sorted when viewed under a chapter.
Adding the has sortableTrait to both chapters and pages, makes them sortable, however pages which should only be sortable under chapters is also sortable under comics, which breaks the ordering should the sorting be edited there.
The following lines
https://github.com/optimistdigital/nova-sortable/blob/58167609b9a35eba40e7a4f4f42b2eb50eda0354/src/Traits/HasSortableRows.php#L52-L70
Need to be placed outside the if statement on line
https://github.com/optimistdigital/nova-sortable/blob/58167609b9a35eba40e7a4f4f42b2eb50eda0354/src/Traits/HasSortableRows.php#L36
So that they also execute on all relationships not just many to many relationships.

Hey @Jnewbon! Thanks a lot for the very thorough report! As I dug into it, there were many issues I uncovered. I fixed the issue you reported and many more in version 2.1.6. I tested it on both ManyToMany and HasMany relationships and it seemed to work as described now. Please do test out version 2.1.6 and let me know if it anything is still broken. Thank you and good luck! :)


chore: update CHANGELOG for 202204 release

This change is

The PR says, "I did not change all of them", so perhaps we need to be a little nuanced? I'm not sure as that may make it less clear.

enumerated services not affected


Added the items to bring during different weather

I have created this pr to solve issue #4.
Here are some images

Hello Priyaraj! The PR Looks good, I just have a few questions about the file structure and the files changed.
Hello :). Priyaraj, this side.

[ ]  What is the need for ItemCards component? I am unable to see any imports or reference to it in any other file.
-> I have deleted the ItemCard. I was trying something else. I am not using it now as it was failing the checks. Don't know why, things were working fine on my local machine.
[ ]  There are a number of changes made to yarn.lock files changing certain dependencies. I believe they were intentional? My react knowledge limits my understanding of the changes in the lock file
-> Yes, they were intentional.
[ ]  It is just a suggestion, I know that we aren't really using the card which was present in the beginning however, Raunak's work was to make that card responsive, so your changes to that card file overwrite his contribution to the project. It's not a big deal but I would have liked it if that file was left unchanged.
-> Raunak had no issues with it as his contribution still remains and as mentioned above the itemCard was failing the checks.
[ ]  There are also about three changes on package.json, are they intentional?
-> Yes, they were necessary.
[ ]  Also, aren't watches needed always? Like, isn't it always a good idea to have a watch or a backpack? I feel those things are very obvious things, I would love to know your thought process to know why you choose to keep those elements.
-> Yes, but I wanted to add different things.
[ ]  The font of "Dont Forget to bring your" and the font inside the cards too are different from any other heading on the webpage. It looks a little off to me, it kills consistenceny. I would love it if this could be changed.
-> Agreed. It's changed now.

Things which I have inspected and looks good to me:

[x]  The cards look amazing
[x]  The cards are responsive and looks good on smaller screen sizes as well
[x]  They have cool on hover animation and it looks amazing on the website.
[x]  The context feels right for the cards come up, but extensive testing might be required for a complete analysis but on a cursory glance it looks amazing!
[x]  I don't see any merge conflicts, lock files may create them but I leave it to @grace-omotoso 's discretion.
[x]  The padding between the cards looks good, and three cards on the page is a great idea, not too many not too little!


Ads not loading on Windows Phone 8

I was updating an app for Windows phone 8 target and ads stop loading about a day ago on devices after receiving the latest 8.1 update.  I thought I might have messed up something in my app, but I just went ahead and downloaded and installed the demo app following the instructions in the docs.  The demo app is not loading / showing any ads as well.
I have tested it on a Lumina 635 (8.10.12397.895) and a 1520 (8.10.14219.341).  Both have the latest version of 8.1 installed and have internet access.  I have also tested the demo in the Windows phone emulator with the same results.  Is this an admob network issue?

Same here - all my existing apps using this plug-in do not show any ads on the WP8 platform.
The exact same app on Android are showing, so it must be an issue with the server-side platform @Googles...
It happened before (six months ago) and it solved itself automatically, but this time it takes longer maybe (or they just pulled out the support for WP - like they always used to do).

Is there any way to tell that no ad has been loaded? We could then change the css dynamically to at least get rid of the empty space...

https://github.com/floatinghotpot/cordova-admob-pro/wiki/1.1-API-Overview
You can check if the banners fails to load, but I've not tested this; so I can't say if the event is triggered in our situation...
Also, I'm not sure we can "use" the empty space with CSS when an error occurred; it's not an element in the HTML-page (I think) but a native element in the app (coded in C#).
So you must check if an error has occurred first and than "resize" the browser-element (which contains the apps-content) to full-size, using a callback with C# (I guess... I'm not sure, because I'm not the author is this plug-in).
On Android it's more easy; the browser-element uses the overlap of the ad and ignores it when no ad is shown.

Same issue.
Impressions from Admob network on windows phone stopped starting 17 sept. Any helps?

Just checked another Admob plug-in for Cordova and that one isn't working as well.
So it seems a server-side / back-end issue at Google, and not really a plug-in related issue (or it's related to all Admob Cordova plug-ins... can't tell).
I will wait another few days, before I remove the plug-in at all; the black empty hole at the bottom of my app-screen isn't looking that nice..

Is there a plugin for native windows ads?
Brad Lawrence
President & CEO, Trunica Inc.
<EMAIL_ADDRESS>From:  "dMOTION.com"<EMAIL_ADDRESS>Reply-To:  floatinghotpot/cordova-admob-pro
<reply+009eeeb71094f71da8559d723bcc4c67feffad5a79f2448f92cf00000001121782449
<EMAIL_ADDRESS>Date:  Monday, September 21, 2015 at 4:31 AM
To:  floatinghotpot/cordova-admob-pro<EMAIL_ADDRESS>Cc:  nielsen<EMAIL_ADDRESS>Subject:  Re: [cordova-admob-pro] Ads not loading on Windows Phone 8 (#225)
Just checked another Admob plug-in for Cordova and that one isn't working as
well.
So it seems a server-side / back-end issue at Google, and not really a
plug-in related issue (or it's related to all Admob Cordova plug-ins...
can't tell).
I will wait another few days, before I remove the plug-in at all; the black
empty hole at the bottom of my app-screen isn't looking that nice..

Reply to this email directly or view it on GitHub
<https://github.com/floatinghotpot/cordova-admob-pro/issues/225#issuecomment
-141907723> .

Native Window ads must be coded in C# / Silverlight.
But you can try this plug-in for Windows Cordova apps (not Windows Phone, but Windows (tablet/desktop) - 'though it does support WP as well (which isn't working right now - because of the ad-issue).
https://github.com/cranberrygame/cordova-plugin-ad-admob

Ok I will wait for the updates in the next days too.

Yeah - but keep in mind that nothing has to be updated; it's not the plug-in that fails, but the service.
Even without updates on our side, the ads must be able to show up (it happened before, but this time it really takes longer).

ok :)

Almost a week without ads on my Cordova Windows Phone apps - so I decided to dig in the code myself.
It looks like there is nothing wrong with the plug-in itself, because it does reserve space for the ad - and doesn't trigger any (failure) events.
After fiddling around with older version of the plug-in (I tested all branches and versions from 2.7.7 to 2.9.1) I found the solution - at least for my self...
The included .dll file (included in the plug-in) is not supported by Google (or is causing troubles anyway).
So I decided to replace these files;

\plugins\com.google.cordova.admob\src\wp8\GoogleAds.dll
\plugins\com.google.cordova.admob\src\wp8\GoogleAds.XML

Both files are included in the plug-in and at least the .DLL file is outdated - or something.
After deleting the files (you can also rename them - you to be save), I downloaded the original .DLL file / .XML file from Google itself;

https://developers.google.com/admob/wp/download

Simply extract both files from the .ZIP to this directory;

*\plugins\com.google.cordova.admob\src\wp8*

Rebuild the solution and... the ads are showing up again - tested on both WP and Android (although I didn't change anything for the Android version).
For my own app (which was using plug-in version 2.7.1) I had to upgrade to the "local" plug-in version 2.7.8 before the ads showed up again;

https://github.com/floatinghotpot/cordova-admob-pro/archive/regio.zip

But I'm not sure this is the case for everyone (I can not use higher version than 2.7.8 because my CLI of Visual Studio (which I use to build the apps) does not support the higher version of the plug-in (due to using the RPM).
Hope this helps - please be careful when deleting files and updating; always have a good back-up and try all things first with a copy of your project.

Thank you for your solution @dmotion.
Replacing the dll with the newest, seems to work fine also for me!

Glad it worked for you as well!
The .DLL-file (the "communicator" between the plug-in and Google Ad-services) in the original plug-in seems outdated (created way back in 2013).
So updating this .DLL-file with the newest one (see the link in my comments above) can solve the problems...
I hope the dev will integrate / update the new .DLL-file in his plug-in... and remember never download .DLL-files from third-party sites - only from the original sites (just to be sure).

Worked for me as well, thanks for getting us back on track!

Thanks to dmotion for pointing out the root cause, the DLL was outdated.
I've updated the DLL in the plugin and re-published to npm (plugin id: cordova-plugin-admobpro), but as cordova plugin registry is deprecated and now read-only, the plugin with id "com.google.cordova.admob" cannot be updated.
If your cordova CLI in visual studio is newer than 5.0, you can directly add plugin cordova-plugin-admobpro.
Or else, you can try use npm to download it to your local folder, npm install cordova-plugin-admobpro then you can find the plugin in ./node_modules/cordova-plugin-admobpro, then add it from local.


Rails 5 support

Will html-pipeline be updated with ActiveSupport 5 and therefore Rails 5 support any time soon?

I haven't had time to set up tests with a new gemfile. It shouldn't be difficult though.

Awesome! Rails 5.0.0.beta4 just came out, and allegedly rc1 will be coming in the next few weeks. It'd be great to have support for it :)

@connorshea what sort of issues are you having? I am also running Rails 5.0.0.beta4 and am getting the syntax highlighting and tables don't appear to be working. Not sure if it is me though :/

@jaykilleen Unfortunately because the gemspec requires activesupport < 5 we can't use it without forking the gem and changing that ourselves. We haven't been able to test it with Rails 5 yet because of other gems as well, despite my having spent the last few months trying to update gems >.<

hi there! any news?

Thanks!

@connorshea Please try 2.4.2 on Rails 5, please report if any problem, thank you.

Thank you @JuanitoFatas! We'll try upgrading and report back if there are any problems.


Webpack

Coverage increased (+0.9%) to 92.147% when pulling 12ce84ede25583af63b875e23ebfc724f09d5f75 on webpack into caf50848d8f86fcb95f70784267420fc4e388dde on master.


pure Python mode fails to import

This was reported here: https://github.com/python-poetry/poetry/issues/6078
It should be relatively simple to add these missing functions to the pure Python version.
@pekkarr how does the aur ship packages? If it ships binaries those appear to be broken otherwise it would not even attempt to use the pure Python fallback version. You might want to set the environment variable RAPIDFUZZ_BUILD_EXTENSION while packaging to make sure that when a build error occurs it does not simply package the pure Python version.

AUR doesn't ship binaries. It contains build scripts that download the source, compile it and make Arch packages that can be installed via Arch's package manager, pacman. The build script for RapidFuzz can be found here.
The build sets RAPIDFUZZ_BUILD_EXTENSION=1 when calling python -m build, which should ensure that the build fails when scikit-build fails.
I'll continue to investigate the issue.

From the error message it appears it does not find the so files for this user. However the immediate issue should be fixed with: https://github.com/maxbachmann/RapidFuzz/commit/2412e5707f7c7ae6098053a0b4ff2a5e535bc00f
Not sure whether this is relevant, but @AkechiShiro writes that he installed python-rapidfuzz-git. I am unsure how the two aur packages are related to each other: https://aur.archlinux.org/packages/python-rapidfuzz / https://aur.archlinux.org/packages/python-rapidfuzz-git

he installed python-rapidfuzz-git

That's the issue then, the python-rapidfuzz-git package is severely outdated and doesn't ensure that scikit-build is installed during build, which is the cause of the issue. I'll contact the maintainer of the package.
In the meantime, the best solution is to use the python-rapidfuzz package.

Yes but the error indicates that the version he actually uses is a lot newer. The whole fallback behavior was only added in v2.1.0. Maybe something breaks when installing both of them?

The -git suffix means that the build source is always the head of the main git branch. So the source code is the latest but the build script was made before 2.0.0 was released and doesn't have scikit-build.

The -git suffix means that the build source is always the head of the main git branch

This sounds awful. There is no guarantee for the main branch to to never be broken between releases. Thanks for the explanation.

This sounds awful.

Yes, -git packages are very unstable.

Thanks for your help in figuring this out.

@pekkarr @AkechiShiro It appears the package was added to the normal archlinux package repository as well: https://archlinux.org/packages/community/x86_64/python-rapidfuzz/

Yes, it's now official Arch package and the AUR package will be removed soon.