the arp-scan spams the /var/log/auth.log

Because our arp-scan require sudo, we get excessive logs, every time it is ran.
you can check this with journalctl -b or cat  /var/log/auth.log:
May 10 11:28:03 sudo[26437]:       pi : TTY=unknown ; PWD=/home/pi/MagicMirror ; USER=root ; COMMAND=/usr/bin/arp-scan -q -l
May 10 11:28:03 sudo[26437]: pam_unix(sudo:session): session opened for user root by (uid=0)
May 10 11:28:07 sudo[26437]: pam_unix(sudo:session): session closed for user root

Would be great to find a way to remove this noise, since it is also using up SD space and keeping more important messages from showing in journalctl logs.

@E3V3A - interesting spot. I don’t think there is a way to stop the logging via the code.
A quick search turned up this but I’d want to be sure that there was a way to isolate the configuration so that only sudo requests made by the module bypass the auth.log before recommending.

I managed to get rid of all lines. For the first line:
# sudo nano sudo /etc/sudoers.d/020_pi
Defaults:pi     !logfile, !syslog
pi xxx = (root) NOPASSWD: /usr/bin/arp-scan

For the session lines:
# sudo cat /etc/pam.d/sudo

#%PAM-1.0

@include common-auth
@include common-account
session [success=1 default=ignore] pam_succeed_if.so quiet_success uid = 0 ruser = pi
@include common-session-noninteractive

Unfortunately, AFAIK, PAM does not seem to keep track of the processes. So yes, using that second line would remove all (sudo based) root requests from most non-interactive process. (Which I don't mind.)
In order to work around it, we'd have to create a separate user or service for this... which is out of scope for most MM modules. But it may still be possible using other methods provided by the sudo node module....
We could offer the silence solution as a script to run, for those users who want more silent logs. I think that's a good idea, because my auth.log was going into the megabytes, just after a day.


staking

i have sataked my coin in desktop wallet i also got my first mint reward. but after i got my reward it turned into too young (blue colour).so is there any problem ??? or we have to wait for 24 after every minting reward ?????

see i have just recived new minting reward my my mature coin change to too young coin.

@bang121 every time you get a reward, the transaction resets, so yes, it's the intended behavior.

then how it is possible 10 % per month in last 58 hr i have just received 1.59 coin. how u calculate 10% staking reward ? as per calculation it should be .75%return and till now i have received 1.59 coin.  please explain in details. thank you in advance


[ogre] Do not use strict mode by default in ogre for backward compatibility

Describe the pull request

What does your PR fix? All versions of ogre since it was added in vcpkg had the strict resource mode disabled by default. https://github.com/microsoft/vcpkg/pull/15194 added support to enable it as a feature, that is great for user interested in it, but also added it as a default feature, meaning that downstream users of the ogre library may break (see for example https://github.com/robotology/robotology-superbuild-dependencies-vcpkg/pull/39#issuecomment-764457000) without any change at all in the Ogre version, just a bump in the vcpkg port version. This PR just removes the strict feature from the default enabled features, but any interested user can still enable it if interested.

Which triplets are supported/not supported? Have you updated the CI baseline? The change should not affect the CI baseline.

Does your PR follow the maintainer guide? Yes

fyi @j-rivero the regression introduced by https://github.com/microsoft/vcpkg/pull/15194 will probably affect CI jobs in which you use ogre for ignition-rendering/gazebo if you updated vcpkg, a possible workaround is https://github.com/robotology/robotology-superbuild-dependencies-vcpkg/commit/2666dfd17d1b99d473ffb29e8e55bd343bd39142 .

fyi @j-rivero the regression introduced by https://github.com/microsoft/vcpkg/pull/15194 will probably affect CI jobs in which you use ogre for ignition-rendering/gazebo if you updated vcpkg, a possible workaround is https://github.com/robotology/robotology-superbuild-dependencies-vcpkg/commit/2666dfd17d1b99d473ffb29e8e55bd343bd39142 .

Well, indeed, we had the same problem. Our software (https://github.com/IRCAD/sight) used ogre with the non strict resource manager. Even if we had fixed our code to be able to work with "strict" mode, my opinion is that the non strict should also be the default since 'strict' code will also work, even if it may be a bit slower when loading resource

Well, indeed, we had the same problem. Our software (https://github.com/IRCAD/sight) used ogre with the non strict resource manager. Even if we had fixed our code to be able to work with "strict" mode, my opinion is that the non strict should also be the default since 'strict' code will also work, even if it may be a bit slower when loading resource

Well, indeed, we had the same problem. Our software (https://github.com/IRCAD/sight) used ogre with the non strict resource manager. Even if we had fixed our code to be able to work with "strict" mode, my opinion is that the non strict should also be the default since 'strict' code will also work, even if it may be a bit slower when loading resource

Thanks @dweckmann !

Well, indeed, we had the same problem. Our software (https://github.com/IRCAD/sight) used ogre with the non strict resource manager. Even if we had fixed our code to be able to work with "strict" mode, my opinion is that the non strict should also be the default since 'strict' code will also work, even if it may be a bit slower when loading resource

Thanks @dweckmann !

Reasonable, thanks!

Reasonable, thanks!


Don't wait till final child state is done being configured in order to load parent templates and controllers

In my app, I have a root state with a menu bar, and a root.ticket state as it's child.  I am dynamically resolving the root.ticket template using a templateProvider which makes an ajax call to get the template.  If this resolution is slow, the entire screen is blank until the ajax call fails or succeeds.  It would be nice if the parent states' controllers and templates were loaded before the child finished.  That would let me at least display the menu bar located in one of the views in root state while the child template was loading.
Similar issue:
http://stackoverflow.com/questions/18961332/angular-ui-router-show-loading-animation-during-resolve-process

+1

Did anything ever happen with this?

Not yet, sorry. We moved it to post-1.0. There are some tricky interactions that we'll have to account for when we implement this.

Did this end up getting fixed by any chance? I am using v0.3.2 and still see child states blocking parent states from loading.

Did this end up getting fixed by any chance? I am using v0.3.2 and still see child states blocking parent states from loading.

It's not really "broken" (it works this way by design).  If you prefer, you can avoid using the resolve feature, and move all your state management into your UI components.
The original bug report is about template loading.  They wondered why: when a parent template is loaded, but the child is not, why doesn't the parent load immediately?  You can achieve that by routing to components in 1.0 (or routing to component templates in 0.3.x, such as template: '<foo-directive></foo-directive>')
In 1.0 we've also exposed the $view service which can be used to completely customize how  and when views are rendered.

For anyone interested in solving their similar issues... here is how I ended up tricking the router to do basically what I wanted, and without having to actually modifying my route configs.
⚠  Disclaimer: this is still an hack, it works® tho.
Preconditions: I have a lot of nested states, which have a lot of resolves (for data fetching, lazy loading modules, etc)... however, the first parent state doesn't have any resolve, never: it's just a template that renders like an app layout with an <ui-view>:
$stateProvider
    .state('myState', {
      // e.g. the app layout
      template: '<header/><ui-view>Loading…</ui-view><footer/>',
    })

    .state('myState.child', {
      url: '/my-url',
      template: '<my-component></my-component>',
      resolve: {
        // e.g. async functions which may take a while
        load: importModule('whatever'),
        fetch: fetchData('whatever'),
      },
    })

So far so good. The problem comes when from another part of the app we want to link to the myState.child; the router will not render anything until everything has been resolved. Result? The app seems stuck, no Loading… indicator. :feelsgood:
Solution? Create a function that first enters the parent state (so the spinner will appear, no resolves), then, once succeeded, it redirects to the original state:
yourModule
  // rename it to whatever you prefer
  .factory('hackedState', ($state, $rootScope) => {

    'ngInject';

    return {
      // HACK: part 1
      go(stateName, stateParams) {
        // save the current state
        $rootScope.pendingState = {
          name   : stateName,
          params : stateParams,
        };
        // go to the parent state
        return $state.go(stateName.split('.')[0]);
      },
    };
  })
  .run(($state, $rootScope) => {

    'ngInject';

    $rootScope.$on('$stateChangeSuccess', (event, next, nextParams) => {
      // HACK: part 2
      if ($rootScope.pendingState) {
        const { name, params } = $rootScope.pendingState;
        $rootScope.pendingState = undefined;
        // clean the pending state and transition to it
        return $state.transitionTo(name, params, {
          location: 'replace',
          relative: next,
        });
      }
    });

  });

Now you can use hackedState.go('myState.child', { /*params*/ }) and the loading indicator will appear.
:suspect:


[Proof of concept] Generate spec from the compiler

This is a proof of concept following from dlang/dlang.org#1683 and the idea is to generate the spec based on the ddoc comments.
This PR isn't perfect yet, but I would wait with further tweaking until we decide that this is the direction in which we want to go.
The respective PR at dmd is https://github.com/dlang/dmd/pull/6877

More makefile rules D:

I am opposed to this solely because it further complicates the build process. It needs to have a strong justification for the additional complication.

I am opposed to this solely because it further complicates the build process. It needs to have a strong justification for the additional complication.

Well, the idea is to move to a stage where the spec can't get outdated anymore, because it's generated from the library frontend.
CC @andralex

cc @RazvanN7 this is the kind of stuff we need to enable with the parser as a library.
This looks like a bit of a steep upfront cost. @MartinNowak what do you think?

cc @RazvanN7 this is the kind of stuff we need to enable with the parser as a library.

@RazvanN7 yes the libdparse API is pretty sweet. In case you aren't familiar a couple of examples:

Dscanner analysis plugins
assert_rewrite_magic
dlang/tools

tests_extractor
has_public_example

using Dscanner to auto-upgrade Phobos

@andralex @WalterBright we still depend on the high-level decision whether we want to go into this direction.

@andralex @WalterBright we still depend on the high-level decision whether we want to go into this direction.

Seeing that the updating the spec manually is tedious work that often gets forgotten, it would be great if we can have a general go ahead on automatically generating the spec from the compiler sources.
I'm pretty sure that once we have a green light on this direction that @CyberShadow and I can work out the technical details ;-)

I do think automation in general is great. Automation of documentation is obviously good, too. However, keeping the solution commensurate with the problem and advantageous (in spite of added complexity) compared to simpler alternatives is a difficult balancing act. This PR does not seem like a slam dunk in terms of bang for the buck. All of a sudden we have libdparse and mustache and new scripts to run - just to keep in sync a few names that are seldom updated.
I was thinking of easing into things before getting too sophisticated. For example, some of the compiler documentation pages can be written - with plan old ddo(c|x) - to take part to the specification pages. At least, the spec could point to something like "Click here for the feature documentation in the reference implementation".


Settings Sync: Do we really need one GitHub Account so that we can sync different VS Code settings??

Hello
I wonder that M$ VS Code Settings Sync does not ask which GitHub Gist should be used to sync the settings.
Does M$ really expect that a user has exactly one VS Code configuration??
Or, does M$ expect that if a user has multiple VS Code settings (e.g. for different projects which completely different plugins, keyboard settings, etc.) that the Customers create one GitHub account, so that each VS Code setting can be synced?
VS Code has a lot very useful features, but such design decisions are a shame, because if they are done right at the beginning, then the implementation is almost free to have. It's much more expensive to change the current Settings Sync concept which is sometimes right for spare time programmers.
So, please: Think twice.
Kind egards,
Thomas

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


IPv6 not supported

     Comment: An exception occurred in this state: Traceback (most recent call last):
                File "/usr/lib/python2.7/dist-packages/salt/state.py", line 1843, in call
                  **cdata['kwargs'])
                File "/usr/lib/python2.7/dist-packages/salt/loader.py", line 1795, in wrapper
                  return f(*args, **kwargs)
                File "/var/cache/salt/minion/extmods/states/ufw.py", line 85, in allowed
                  from_addr=from_addr, from_port=from_port, to_addr=to_addr, to_port=to_port)
                File "/var/cache/salt/minion/extmods/states/ufw.py", line 45, in _as_rule
                  cmd.append(_resolve(from_addr))
                File "/var/cache/salt/minion/extmods/states/ufw.py", line 27, in _resolve
                  return socket.gethostbyname(host)
              gaierror: [Errno -9] Address family for hostname not supported

IPv6 addresses are getting resolved via socket.gethostbyname(host) causing a failure.

The regex also returns an empty value (if it would work).
A workaround for this is to disable the gethostbyname for now and the regex, and just return the host as is. Using the gethostbyname like this is also ill-advised as it returns an IPv4 address, not both.
My suggestion would be to completely disable host resolutions and pass everything to UFW as is.


The unoconvert module needs to be separated from the unoserver project.

Currently, the unoconvert module is included in the unoserver project. If I deploy unoserver on Machine A using Docker, in order to access this service on Machine A, I must also install unoserver on Machine B to use unoconvert. Normally, it would be nice if I deploy a server on Machine A, all I need on Machine B is a client to access it.
Moreover, since unoserver depends on LibreOffice, Unoserver needs to be installed by and run with the same Python installation that LibreOffice uses. However, unoconvert should only depend on whether the remote unoserver is running, so there should not be so many restrictions when installing the unoconvert module.

You don't need Libreoffice to install unoserver, or run the unoconvert module. It's only needed when running unoserver.


getSidecarScope cleanup

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
b2d776c21234bf9ad966d1a6bcad879a9eab32a6
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Javascript library should do input validation

Passing values as strings instead of numbers will produce unexpected results, due to Javascript string handling.
The inputs (lat, lng, code length) should be coerced to numbers (using Number()) and if it produces NaN throw an exception.
While you're there, adding an exported constant for the code length e.g. OpenLocationCode.CODE_PRECISION_NORMAL and OpenLocationCode.CODE_PRECISION_EXTRA seems like a good idea.

This got addressed in #57.


Copy to clipboard button

The copy to clipboard button that has been added to the markdown files does not work in the VSCode markdown preview, even with changing the security settings so that "Markdown-preview-enhanced: Enable Script Execution" is true (checkbox is checked) and/or running the command "Markdown: Change Preview Security Settings" from the preview and selecting "Disable: Allow all content and script execution".
It may be better to use Javascript to add the button rather than add the button explicitly in the markdown file. Some links that may be helpful:
https://junges.dev/2-how-to-add-github-copy-to-clipboard-button-on-your-docsblog
https://spdevuk.com/how-to-create-code-copy-button/
Then we can decide whether or not to disable scripts in VSCode - if it is disabled at least user don't see a button that doesn't work.
I'm not sure if this issue still applies as I know you had a different style of the docs as work in progress. Perhaps in the short term (for 0.2 release) we can just remove the copy to clipboard buttons?

Ah that's even better if the new doc design adds the copy button automatically.
So yes, let's do it like this:

Your Doc Design PR - I guess this already removes the buttons and any JS that's no longer needed?
My PR making the container README into a tutorial index (https://github.com/r-devel/r-dev-env/pull/94) - I will probably need to resolve conflicts once your Doc Design PR is merged in.

Fixed in #98.


Implements the column statistics for atlas proxy

Summary of Changes
This PR implements the column statistics available for the atlas proxy, that was not there before.
Tests
This updates the test cases related to get table details from atlas proxy.
Documentation
Documentation explains how to add the column statistics attributes in the atlas itself, including the code to do that using Python.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test
[x] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

@jinhyukchang @feng-tao ready for review

Codecov Report

Merging #52 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage      75%   75.15%   +0.15%
==========================================
  Files          19       19
  Lines         940      946       +6
  Branches       71       72       +1
==========================================
+ Hits          705      711       +6
  Misses        212      212
  Partials       23       23

Impacted Files
Coverage Δ

metadata_service/proxy/atlas_proxy.py
93.2% <100%> (+0.26%)
: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 ab78684...7207b05. Read the comment docs.

I'm sorry @feng-tao I somehow missed your message.

No I don't think we should update the version for this change, I am working on popular table improvements, and will bump the version with that change.
Our sister team is working on all kind of statistics (including data quality). They are exploring different libraries/methods to do that. Apache Griffin, Deequ etc. For now they are sticking with Deequ, and using it with sparkDataFrame.
ref: https://github.com/awslabs/deequ/blob/master/src/main/scala/com/amazon/deequ/examples/data_profiling_example.md

thanks @verdan


initial move to k3s-io

Signed-off-by: Jacob Blain Christen<EMAIL_ADDRESS>

Part of rancher/k3s#2189


PMM-1899: Add size to determine if query got truncated.

Changes:

Double the maximum size of examples.
Store size of original example.
The first idea was to store information if query was truncated or not, but this was providing limited information, e.g. you couldn't determine what was the original size of sample. With size, we can now determine if query was truncated, what was original size, and what was maximum example size to which query was truncated.

Related PRs:

https://github.com/percona/qan-api/pull/52
https://github.com/percona/go-mysql/pull/28
https://github.com/percona/qan-app/pull/79
https://github.com/percona/pmm/pull/135

Please merge at will.


basicTable组件，[nzDataSource]传入一个数组泛型是一个interface，报错找不到length

I'm submitting a...

[ ] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Support request => Please do not submit support request here

Current behavior

Expected behavior

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Environment

Angular version: X.Y.Z

ng-zorro-antd version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

Minimal reproduction of the problem


Disappeared large number CC

I encountered a weird issue.
I'm using large CC numbers for an instrument.
Two CCs are missing : Pan Tom1 (CC 453) and Tune Tom2 (CC 474), in red circle :

But if I change them to small numbers, they appeared :

And if I only load only the Tom1 or Tom2 sfz file, the large CCs also appeared.

This is unexpected. Can you share the files?

Here's the files :
test-master.zip
The files came from our private repo. It's a prototype Drumkit sample library project, so it contains a lot of sfz presets.
But basically you just load the "NewDrums.sfz" in the 3rd folder, the Stereo-Kit version to check.


add autofeedback ICL and cli log10 feedback predict

log10 feedback predict --task_id 04405cbc-3420-4901-97b6-f8b7ed248205 --completion_id 447c3b69-6aea-4f4d-95e6-cbf37adca846

log10 feedback predict --task_id 04405cbc-3420-4901-97b6-f8b7ed248205 --content "{\"article\": \"We have been together over two years, but got in a very large fight today. We have large fights occasionally where my boyfriend just gets blinded by his anger. \\n\\nI have never ever liked jewelry, and haven't worn a single item of jewelry since elementary school, my ears aren't even pierced.  But I do make a point to him that I don't like jewelry, and never wear it, while trying to be as polite as possible. For some reason, he can't understand this about me and continues to get angry that he buys me jewelry I don't wear, etc. etc. He just seems to not understand I don't want this type of typically \\\"nice\\\" gift. I still thank him for the original intent, but at what point should he start treating me as a person with individual likes and dislikes? He just expects me to like typical \\\"girl\\\" things. He's very upset with me and calling me ungrateful. Sigh. Maybe he's right. It's just something I will never be able to like but am expected to be grateful for anyway :/ \\n\\nAlso, I would understand if we just started dating, but it's been two and a half years of this.  He's trying to break up with me right now.\"}"

Highlevel - tried and it worked 🎉
One snag - if environment isn't correctly setup, the error message is a bit hard to grok:
ValueError: Sample larger than population or is negative

Maybe we make a better one?


React Client: game scene -- first steps

react-ui baby steps
starting effort on ActiveClues
add vote interaction in ActiveClues

So far looks like:


Graphics abstraction

Implements a graphics backend for OpenGL and Vulkan. This has not been tested enough, but will be replaced and redesigned soon enough for me to care.

Sounds like a pain. Thanks for the contribution, me.


"featured" dashboard

Hi, I have not found any documentation on the "featured" dashboard - How to create etc. Thanks.

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

ID: 91935bb2-c486-cb9f-be71-ae18ad28cf96
Version Independent ID: 761de430-9bc7-4d7d-4c56-d14e6354194c
Content: What is a dashboard for Power BI service consumers? - Power BI
Content Source: powerbi-docs/service-dashboards.md
Service: powerbi
GitHub Login: @maggiemsft
Microsoft Alias: maggie

@AzizQuazi Thanks for your comment. We’ve forwarded your feedback to the writer, who should respond to this thread. Thanks!

Hi @AzizQuazi
Try this link: featured
Take care,
Michele


[BUG] Can't access to __reactInternalInstance

What happened?
I'm trying to acces __reactInternalInstance$ from a HTMLElement
Version
Latest
What OS are you seeing the problem on?
Windows
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

@terrainwax can you please provide some example code snippet


remove erroneous urlChanged check

Currently ampersand-router does not work because this.fragement is changed and then urlChanged is called, which will subsequently always return false.
Also the test for this appeared to be broken, and passing only because of this bug.

:+1: works for me now

Running this locally, I see that the test does need to be changed but I'm trying to figure out why. Previously the code in this test worked as it was https://github.com/AmpersandJS/ampersand-router/blob/d86e0bcb2180bb0b8fde098d2f68d9018a3fd0b7/test/index.js#L293-L302. Is it worth exploring as to why the original test no longer passes?

It seems to me like that test is trying to handle a case when pushState is false and the current url's path is described by the #route and that calling navigate('route') shouldn't trigger a route change, but it does currently.
I'm +1 on the erroneous urChanged check being removed, but -1 on the test being modified. I'm working on debugging the test right now and will see what I can find.

The original test passed because urlChanged was always going to return false. You could have navigated anywhere and the test would have still passed.

And yes I wasn't 100% on the test, push up whatever you find if you can make it closer to the original.

The urlChanged check was added pretty recently in 701580944c2860906d769d55911bbd225d659dfc. I checked out the commit before that (d86e0bcb2180bb0b8fde098d2f68d9018a3fd0b7) and stepped through the test in question and the reason loadUrl is never called is because the trigger option defaults to false, so the test is not testing for what it says (loadUrl is not called for identical routes.) but instead it ends up testing that loadUrl is never called because trigger is false.
So I'm a +1 on this now, because its obvious that the extra urlChanged() check breaks stuff and the existing test is now actually testing what it should.

published as v3.0.2


Incompatibility with JetBrains 2021.2 tools

Plugin 'Logstash Config Files Support' (version '0.63-EAP') is not compatible with the current version of the IDE, because it requires build 211.* or older but the current build is IU-212.4746.92

I get a similar error in PyCharm.
Thank you for this great plugin. I hope you have time to support the new JetBrains tools.

@airikh, hi, thanks for pointing this out
I did an update as soon as JB approved it the plugin will be available

Thanks a bunch, @ikorennoy! Much appreciated.


Plugin completely unresponsive

Hi,
I have tried using Better Font Picker on Chrome, Safari, and FireFox, so it's not a browser issue.
I select it by highlighting the text whose font I want to change, Right Click > Plug Ins > Better Font Picker.
Then, this page comes up for a 0.5 seconds before switching to the second image here.

At this point, it doesn't let me select the search box to type anything. In a minute, it tells me the page is unresponsive.
I have a Macbook Pro with OS Catalina. How can I fix this issue?
Thanks!

Thanks for reporting. I'll look into this.

I have the same problem ;(

having the same problem on firefox :(

Having the same problem on Yandex Browser


Configuration options around auth

For some use cases auth is being handled completely out of band from Stargate thus making the available auth related modules a burden. We need to add a new config option that will do the following

Disable auth-api module
Not create auth related keyspace/table
NOOP authn/z related calls in all other modules
As possible, remove auth related dependencies in other modules

Aside from disabling the auth-api module, one way to accomplish the other items would be to add a new auth-noop-service that satisfies the authentication interfaces but is a NOOP for every call since the expectation is that the requests have already been authenticated and authorized.

@marksurnin is there anything I missed from our discussion earlier?

@dougwettlaufer that sounds good, thanks! Yes, we went with a NoopAuthnTableBasedService that extends AuthnTableBasedService.

What is the connection between this issue, and the auth field under StarfieldSpex on K8ssandra (docs link)?


feat: key not from file & other improvements

This adds support for constructing a ServiceAccount directly from a ServiceAccountKey, rather than from a file. It also:

Adds interior mutability using a tokio RwLock to enable cloning ServiceAccount while re-using the current access token across threads/instances. Also allows ServiceAccount to be used without needing to be mut.
Makes ServiceAccountKey and its fields public to allow parsing or constructing the struct from any source (e.g. string, bytes, etc.) and also allows the fields inside to be re-used by other modules (e.g. reading project_id necessary for FCM)
Adds ServiceAccountBuilder to enable the use of a variety of constructors, including the ability to re-use reqwest::Client across instantiations which enables sharing connection pools. Existing from_file() and user_email() methods continue to work for backwards-compatibility, but the builder pattern is generally a better alternative
access_token() now returns AccessToken instead of e.g. "Bearer token" which allows the use of reqwest's .bearer_token() method without needing to parse-out the token. Also enables the invoker to see when the token will expire.
Asserts that the returned token is indeed a bearer token
from_file() retained as a helper method and now returns a Result type since file reading and parsing is performed in this function, rather than lazily in access_token(). Also now accepts any path-compatible value, rather than just &str.
Stores RsaKeyPair in JwtToken rather than inputs to improve efficiency
Catch and re-throw any non-200 status codes when requesting an access token. Also generally improve error handling by returning the original error, rather than converting into a string.
Resolves various clippy errors

Resolves #28

Thanks for the review! I'm working on some fixes for the above as well as some other tweaks


[feat] Explainable state modified node comparisons

resolves #7109
Description
Spike to try updating node comparisons to leverage an object with a __bool__ method that additionally carries requisite data to log detailed diffs during state based node selection.
Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

Because of a number of changes to nodes.py and because we're in the middle of removing the old metric code, this PR is in a difficult to resolve state. Also the same_contents method has switched to doing "|" between the different checks, which is different behavior and might be breaking, so I'm thinking we don't want to do that at this time. The comparison improvements do look valuable, however. My suggestion would be to wait a bit until the metric switchover is a bit farther along, and then update this (removing the switch to "|" and try to get this included in 1.6.

Makes sense to me. We can close this out and I can follow up after the release with a new PR with fresh eyes.
cc: @gshank

Thanks @z3z1ma @gshank! For what it's worth - this is still a really cool idea, and I'd love to keep making progress toward it (for v1.7 if not for v1.6)


auto completion didn't work

Summary
The auto completion didn't work.
Extension version
Ansible v0.4.5
VS Code version
Version: 1.60.2 (Universal)
Ansible Version
ansible [core 2.11.5]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['~/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.9/site-packages/ansible
  ansible collection location = ~/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/local/bin/ansible
  python version = 3.9.7 (default, Sep  3 2021, 12:37:55) [Clang 12.0.5 (clang-1<IP_ADDRESS>)]
  jinja version = 2.11.3
  libyaml = True

OS / Environment
MacOS Big Sur
Relevant log output
No response

Judging by the version this is related to the plugin version sourced here: https://github.com/ansible/vscode-ansible


Support for SGW <IP_ADDRESS>/16  for yc_sgw

Hello, will there be support for SGW <IP_ADDRESS>/16  for yc_sgw?

Error: Error while requesting API to create subnet /omitted/ rpc error: code = InvalidArgument desc = Illegal argument <IP_ADDRESS>/30 does not belong to allowed ranges
Error: Error while requesting API to create route table: /omitted/ rpc error: code = InvalidArgument desc = Illegal argument <IP_ADDRESS> value does not belong to allowed range [<IP_ADDRESS>/8, <IP_ADDRESS>/12, <IP_ADDRESS>/16]


encoding python functions (code objects)?

hi, I was wondering if jsonpickle can help me encode python functions (code objects)? it seems like libraries like dill and cloudpickle seem to be able to do this. I would like to embed some code objects within a larger json payload and encode it within jsonpickle

We don't currently support this but... if it's possible that'd be awesome.  Maybe we can leverage those modules, or reuse the same approach, and provide a module like we do for numpy?
Pickling a scope and a function seems like a pretty crazy awesome use case if it's possible.

Hmm...if jsonpickle were given existing pickled data, could it convert it to JSON format somehow?


Update complaints

Closes #161.

This change is

Закривати цей пр і перекидати його на master?

Reviewed 2 of 7 files at r2.
Review status: 0 of 7 files reviewed at latest revision, 3 unresolved discussions.

op_robot_tests/tests_files/aboveThreshold.robot, line 8 [r3] (raw file):
openeu

op_robot_tests/tests_files/aboveThreshold.robot, line 18 [r3] (raw file):
delete

op_robot_tests/tests_files/aboveThreshold.robot, line 24 [r3] (raw file):
Передививсь, що наробив ребейс.

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 3 unresolved discussions.

op_robot_tests/tests_files/aboveThreshold.robot, line 8 [r3] (raw file):
Done.

op_robot_tests/tests_files/aboveThreshold.robot, line 18 [r3] (raw file):
Done.

op_robot_tests/tests_files/aboveThreshold.robot, line 24 [r3] (raw file):
Done.

Comments from Reviewable

Conflicts!

Someone who will merge must resolve them :)

Reviewed 1 of 8 files at r1, 1 of 1 files at r15, 1 of 2 files at r16, 1 of 1 files at r18, 3 of 3 files at r19.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

op_robot_tests/tests_files/aboveThreshold.robot, line 375 [r19] (raw file):
Ребейз видалив.

op_robot_tests/tests_files/service_keywords.py, line 424 [r19] (raw file):
чому не if  complaintID == element['complaintID']?

op_robot_tests/tests_files/service_keywords.py, line 435 [r19] (raw file):
Аналогічно if  document_id == element['title']. Забери print-и.

op_robot_tests/tests_files/brokers/openprocurement_client.robot, line 434 [r19] (raw file):
Тут потрібно логувати тендер?

Comments from Reviewable

Review status: all files reviewed at latest revision, 4 unresolved discussions.

op_robot_tests/tests_files/aboveThreshold.robot, line 375 [r19] (raw file):
Done.

op_robot_tests/tests_files/service_keywords.py, line 424 [r19] (raw file):
Done.

op_robot_tests/tests_files/service_keywords.py, line 435 [r19] (raw file):
Done.

op_robot_tests/tests_files/brokers/openprocurement_client.robot, line 434 [r19] (raw file):
Це просто перенесений Log.  В майбутньому треба пройтися по всіх файлах і видалити всі логи.

Comments from Reviewable

Reviewed 1 of 1 files at r22, 1 of 2 files at r23, 1 of 1 files at r25, 3 of 3 files at r26.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Remove hardcoded parameters, minor improvements for parameters-as-functions

Content

Removes entr_coeff detr_coeff and entr_tau from the config, makes them parameters

Update edmfx_bomex_box_v2.toml to include the detr_coeff change

Adds  parameters for previously hardcoded values in the held suarez forcing_tendency, update the parameter struct accordingly
Removes SpongeParameters - I think we can keep these in the top-level struct
Improves the parameter functions, auto-generate them using @eval
Adds some tests to ensure parameters are properly read in. I introduced a bug in the atmos tomls where different parameters had the same alias, leading to parameters not being properly set. This bug has been fixed by ClimaParameters #136, which also includes an error check for unique aliases.

Looks good, thanks! In general, how do we determine if some parameters (e.g. sponge) should be in the top-level struct?

For now, we can just keep adding parameters to the top level because there aren't too many. If we keep adding parameters, we should probably find better ways to break them up.

bors r+


Replaced Stop with Dispose for all features and node.

Replaced Stop with Dispose for all features and FullNode.
The reason for that is that all Stop() calls actually dispose stuff instead of stopping the feature.
And in case of node there were Stop() and Dispose():  Stop() would do the disposing (leaving IsDisposed flag false), Dispose() would call Stop() which is a very counter-intuitive behavior.
This is discussed in #799

Hmm I am not sure I am to happy with the name change @bokobza @fassadlr can you guys comment?
If we change to Dispose we sort of give the code a different meaning.
Stop to me feel it makes this more part of the IDisposable pattern and by using Start/Stop that give us more control.
If we did change Stope -> Dispose
Then I suggest we also change Start -> Initialize

@dangershony i know what you mean. however, to achieve what you are suggesting, we should distinguish between disposing the object and stopping. currently, what Stop usually do is that it does both and in many cases there is no stopping code, so it only disposes
if we make such distinction, then we will be have Start/Stop as well as Dispose, but as mentioned earlier, the way .NET Core chose is to only use Dispose unless the object is somehow restartable (i.e. it makes sense to do something with the object after it is stopped) and in many cases it does not make sense in our code. maybe in some classes it would - such should be separated, the rest could probably adopt this Dispose only approach

@dangershony Yes I agree, we can't rename Stop and keep Start. We have to be consistent throughout.
@noescape00 A while back there was a discussion between myself and Dan to possible implement a IStartable interface which will have Start and Stop methods. This is to ensure that features get started and stopped. We haven't gotten around to do it yet but the idea was to actually force all features to start and stop. @noescape00 please note that some features' "Stop" methods actually are a teeny bit more complex, i.e. disposing of asyncloops etc and as such I also agree that we should keep Start and Stop from an architectural view point.
In the future Start and Stop logic might even get more complex and then we'll have to revert what you did now. Its just better to allow room for the future.
HOWEVER, I do feel that we need to decide on a pattern, its not good practise to mix the 2 in my opinion. Either have a Start and Stop pattern (which I prefer) Or use the Dispose pattern throught to stop asyncloops etc.
(Edit what @Aprogiena said)

In the future Start and Stop logic might even get more complex and then we'll have to revert what you did now. Its just better to allow room for the future.

Not sure it will be about reverting, more like adding.
Right now most of the code don't implement Start() and Stop() properly. In my opinion this naming convention suggests that we can do
Start();
Stop();
Start();

and everything would actually work. But most of the features are not restartable so having this names is not really logical for me.
Though I agree this PR doesn't really solves the problem because it moves from one badly implemented pattern to another slightly less badly implemented.

i agree with @noescape00 (seems like i agree with everyone here actually). so at this point it seems that very little (if any?) Stop of any class is implemented as it should (start-stop-start possible). in that case we should have these Disposes as @noescape00 implemented but we should also do what @dangershony suggested at the same time - rename Start to Initialize

i.e. it makes sense to do something with the object after it is stopped

Ok yeah this convinced me, we are indeed disposing/shutting-down, not stopping a restartabe object
And seems we are in agreement to rename Start to Initialize

@noescape00 do you use resharper or are you as fast as @Aprogiena in typing? 👍

@dangershony resharper. It has an awesome renaming feature that would rename every override, reference in xmldoc and comments.


CSS changes

Extra button removed.
Added space between input field and button.
Removed extra white space that overflows at right side by adjusting image size.

It resolve #1

Hey Fais I'm not able to reproduce this issue.
Can you share how to reproduce it.

Extra button removed.

Also we don't need to remove the button.
Just the space would be fine I think.
Can you provide the width of your display screen?
Here's the screenshot of the website which i'm able to see right now -

The website is looking good in your screen shot, I dont know exactly how to fix this issue. Let me close this PR now.

Thank you Fais for taking your time out to contribute to the project and thank you for closing the issue 😄 I hope we can collaborate in the future!👍🏻

Ofcourse yes brother, let's learn together 👍


Card links: divider line style

Description
The card link divider line doesn't connect to either edge. This makes design look a little messy.

Android and iOS divider lines
Divider lines for material design (Android) either are 'full width' or 'inset' (connected to right edge).

Divider lines for iOS typically are 'inset' (connected to right edge).

https://github.com/nhsuk/nhsapp-frontend/issues/10#issuecomment-2250632579

further to the above, i'm also not entirely sold on the way we use dividers in a stack of card links – the current approach seems a little messier than it needs to be. if the divider went all the way to the edge of the card on one (right) and both sides, i think we could remove a little visual noise.


Missing flow .npy file

Hi, I came across your paper recently and thought it was really impressive. I am running the test script using the command provided, but I get an error
flow0 = np.load(flow[0])
IndexError: list index out of range

This line in datasets.py indicates that there should be a flows folder with the contents being .npy. Is it possible to provide this folder?
flow = sorted(glob.glob(os.path.join(self.data_root.replace('vimeo_triplet', ''), 'flows', name, '*.npy')))
Thanks

Hi, thanks for raising this issue, we have updated the Prepare Dataset part in the README file, please follow the instruction to compute the GT flows by using lite-flownet.


Impossible to click on View Less

Impossible to click on View Less because the feedback is before it. For this feedbacks and also the others we should add the possibility to close them like so.

Ressources
https://mui.com/material-ui/react-snackbar/
https://mui.com/material-ui/react-alert/

For me, we should simply be able to close the feedback even if it reappears on refresh.
I don't think it's our feedback used in a normal way.
Use this component : https://www.figma.com/file/S1UKYgWewNqNHZFAaBUilG/Starknet-ID?type=design&node-id=1496%3A4488&mode=design&t=vhPGKStwJOUycGTo-1


fn resolve_divisor_{32,64}: Cleanup and make safe

These could be further cleaned up by parameterizing over u32 and u64, but it's not that important yet.  I would like to replace the ulog2 and u64log2 calls with {u32, u64}::ilog2, but ilog2 was stabilized in Rust 1.67, and our pinned nightly is a bit behind that.  Would it be a good idea to upgrade that to something newer?  Unfortunately, we'll have to stay on nightly for a while longer until we finish getting rid of varargs, extern types, and intrinsic calls.

I think bumping to the latest nightly at some point would be good, esp if there's new stabilized stuff that we want to use 👍


IComponent.SetParametersAsync Documentation Improvements

Is your feature request related to a problem? Please describe.
It is currently not clear whether all parameters or only changed parameters are passed to IComponent.SetParametersAsync() as the parameters argument.
Describe the solution you'd like
The XML doc comments could be improved to explain the parameters argument better.
Additional context
https://github.com/aspnet/AspNetCore/blob/master/src/Components/Components/src/IComponent.cs

Want to send a PR?

Want to send a PR?

Part of the reason I made this issue is because I don't know, myself.

OK no sweat. I just was suggesting in case you were eager to.


zip method added

@rbnquintero cool

@rbnquintero could you also update the readme to include the new API usage? That would be awesome!


Test against Mavrin polynomial curves

Adds tests to make sure that the Lz and mean charge curves match the Mavrin polynomial fits.
Fixes a bug which led to an underprediction in Lz at high temperatures.

Comparing results to Mavrin
Solid line is radas, dashed line is Mavrin. The legend gives the value of $n_e \tau$


geowave-deploy-1.0.0-SNAPSHOT-accumulo.jar

Nowadays i'm trying to set the CDH Accumulo for using geowave.
just input a geowave jar file for accumulo in ext directory. However after that all accumulo module is down with that error log above. it can not read and accept the geowave jar file for accumulo avaliable.
In my guess, This logs looks like to relate with commons-vfs2-2.0.jar that CDH has. but not sure.
Something need a new jar file for CDH Accumulo?
as the geowave manual said, i already konw that geowave can be also working with CDH.
So i'd like to seek advice for solving this problem.
I'm looking forward to the fastest possible response.
my System info is;
CDH version : Cloudera Express 5.15.1 (#3 built by jenkins on 20180731-0834 git: 6e046fd999fd99e7e12299386ba69b23afa43c60)
Java version : 1.8.0_181
Accumulo version : 1.7.2-cdh5.5.0
Please let me know advice ASAP
Thank you for your reading

Plus after input the geowave jar file, it does not enter into accumulo shell left this log;

then ingesting the shp file, it can make empty '_GEOWAVE_METADATA' table only.
is it something need the other setting using CDH accumulo?

Plus.

when scanning the  'GEOWAVE_METADATA' table, it says the logs

presumably this is a jar you built from the source code yourself?  you can try using our hosted jars here and make sure the permissions are open enough for the accumulo tablet server process to read (which I'm assuming you already checked but figured I'd mention to make sure).  Using lib/ext locally on each tserver should be fairly straightforward, although we always put the jar on HDFS and use accumulo namespaces as described here

anybody please show me how to set up the geowave in cloudera? Nobody has some examples about it? rpm set up is not work and then unclear to me then making jar file for tools and accumulo with cloudera is no manual something

Thank you for your quick answer rfecher  you r very faster then my work off...lol
yap this jar file built from the source code myself..coz i'm bad at dealing with rpm..anyway i'll try....im very confuse that cloudera is too different.
is it something different between Cloudera and Apache during the installation process? such a location of accumulo jar file(yep i did that it'd be at 'lib/ext' but not sure it's the same with it) or making geowave tools alias in cloudera? or operation System using tools CLI?
then what is the jars mass( this: http://locationtech.github.io/geowave/packages.html) it's avaliable?
Finally did you have some manual or examples about geowave setting in cloudera?


Lay out preferences?

I'm not a fan of the new layout.
Having a big bar at the top of the page seems inconvenient.
Everything seems so extremely zoomed in right now.
Perhaps an option to move the bar back to the side or have it minimized.
Would also be nice to alter the size of text and spaces inbetween.

Probably not going to make this configurable since it's hard enough to get the layout agreeable without trying to make two layouts.
Might make it so you can scroll past the nav bar, rather than it being sticky though.


fix(dateFilter): follow the CLDR on pattern escape sequences

When there are two single quotes "''" (quotes for clarification) that are not part of an escape sequence, then this sequence should be handled as one single quote. See http://www.unicode.org/reports/tr35/tr35-dates.html#Date_Format_Patterns second and forth examples

LGTM - please merge.


[CELEBORN-1105][FLINK] Support Flink 1.18

What changes were proposed in this pull request?
Why are the changes needed?
flink-1.18.0
./bin/start-cluster.sh
./bin/flink run examples/streaming/WordCount.jar --execution-mode BATCH

Caused by: java.lang.NoSuchMethodError: org.apache.flink.runtime.io.network.partition.consumer.SingleInputGate.<init>(Ljava/lang/String;ILorg/apache/flink/runtime/jobgraph/IntermediateDataSetID;Lorg/apache/flink/runtime/io/network/partition/ResultPartitionType;Lorg/apache/flink/runtime/executiongraph/IndexRange;ILorg/apache/flink/runtime/io/network/partition/PartitionProducerStateProvider;Lorg/apache/flink/util/function/SupplierWithException;Lorg/apache/flink/runtime/io/network/buffer/BufferDecompressor;Lorg/apache/flink/core/memory/MemorySegmentProvider;ILorg/apache/flink/runtime/throughput/ThroughputCalculator;Lorg/apache/flink/runtime/throughput/BufferDebloater;)V
	at org.apache.celeborn.plugin.flink.RemoteShuffleInputGate$FakedRemoteInputChannel.<init>(RemoteShuffleInputGate.java:225)
	at org.apache.celeborn.plugin.flink.RemoteShuffleInputGate.getChannel(RemoteShuffleInputGate.java:179)
	at org.apache.flink.runtime.io.network.partition.consumer.InputGate.setChannelStateWriter(InputGate.java:90)
	at org.apache.flink.runtime.taskmanager.InputGateWithMetrics.setChannelStateWriter(InputGateWithMetrics.java:120)
	at org.apache.flink.streaming.runtime.tasks.StreamTask.injectChannelStateWriterIntoChannels(StreamTask.java:524)
	at org.apache.flink.streaming.runtime.tasks.StreamTask.<init>(StreamTask.java:496)

Flink 1.18.0 release
https://flink.apache.org/2023/10/24/announcing-the-release-of-apache-flink-1.18/
Interface org.apache.flink.runtime.io.network.buffer.Buffer adds setRecycler method.
[FLINK-32549][network] Tiered storage memory manager supports ownership transfer for buffers
org.apache.flink.runtime.io.network.partition.consumer.SingleInputGate constructor adds parameters.
[FLINK-31638][network] Introduce the TieredStorageConsumerClient to SingleInputGate
[FLINK-31642][network] Introduce the MemoryTierConsumerAgent to TieredStorageConsumerClient
Does this PR introduce any user-facing change?
How was this patch tested?
flink-1.18.0 ./bin/flink run examples/streaming/WordCount.jar --execution-mode BATCH
Executing example with default input data.
Use --input to specify file input.
Printing result to stdout. Use --output to specify output path.
Job has been submitted with JobID d7fc5f0ca018a54e9453c4d35f7c598a
Program execution finished
Job with JobID d7fc5f0ca018a54e9453c4d35f7c598a has finished.
Job Runtime: 1635 ms

Thanks, merge to main


Create and publish Gradle artifact for Herald

[Who]  As a developer
[What] I would like to consume Herald for iOS as a Gradle dependency
[Value] To make integration and regression testing easier and more reliable
Acceptance criteria:-

[x] Make static final configuration class static settable for apps
[x] Double check existing adopters' code to ensure required hooks are available.
[x] Access the develop / beta version as a gradle dependency
[x] Access the latest full release as a gradle dependency
[x] Instigate a process to build and publish this on each change to develop/master

Waiting on assignment of Deployer right for com.vmware.herald via OSSRH. May take a couple of days due to the thanksgiving break.

Moved to v1.2 due to delay in getting access to maven publishing rights

Final part to do is test the release process for OSSRH and GitHub Packages. WIll also add workflow files to manage this process.

We now have push rights to maven central. I shall try and push out a v2.0.0-beta1 build today.

Tested and verified. Fixed in develop.


Adapting BL parser to latest page changes

Hi @dominikgehl. You were faster than me. :) I was just sending PR.
I was thinking of doing it this way:
    d = d.replace('\n', ' ')
    # Extract last line. Use non-greedy matching.
    data = sc.find(r'
 ?Datum, Geheilte (?:geschätzt|kalkuliert), Aktive Fälle, (?:Verstorbene|Todesfälle).*? ([^ ]+) ?
', d)
    if data:
        c = data.split(',')
        print('Date and time:', c[0].replace('-', '.'))  # 24-03-2020 -> 24.03.2020
        active_cases = int(c[2])
        recovered = int(c[1])  # "Calculated"?
        deaths = int(c[3])
        print('Confirmed cases:', active_cases + recovered + deaths)
        print('Recovered:', recovered)
        print('Deaths:', deaths)
        continue
    data = sc.find(r'
 ?Datum, Bestätigte Fälle, Geheilte (?:geschätzt|kalkuliert), (?:Verstorbene|Todesfälle).*? ([^ ]+) ?
', d)
    if data:
        c = data.split(',')

        print('Date and time:', c[0].replace('-', '.'))  # 24-03-2020 -> 24.03.2020
        print('Confirmed cases:', c[1])
        print('Deaths:', c[3])
        print('Recovered:', c[2])
        continue
    data = sc.find(r'
 ?Datum, Normale Station, Intensivstation.*? ([^ ]+) ?
', d)
    if data:
        c = data.split(',')
        print('Date and time:', c[0].replace('-', '.'))  # 24-03-2020 -> 24.03.2020
        print('Hospitalized:', int(c[1]) + int(c[2]))
        print('ICU:', c[2])
        continue
    assert False, "Did not match any csv format in the iframe"

Your method is also fine I guess.

One way or another both methods looks good to me. It is a bit disappointing they don't directly provide confirmed cases anymore, but my guess is this is due to the bar graph stacking in their visualisation at https://www.baselland.ch/politik-und-behorden/direktionen/volkswirtschafts-und-gesundheitsdirektion/amt-fur-gesundheit/medizinische-dienste/kantonsarztlicher-dienst/aktuelles/covid-19-faelle-kanton-basel-landschaft  (they want to bars stack properly without extra logic).

Note that for BL and ZH the scrapers are no longer running on GitHub Actions since the data is added directly by the cantons on GitHub. But since the scrapers are used elsewhere, it's of course still okay to fix them.


[$500] Thread - Newly created thread report in Room remains in LHN if it is not commented on

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.3.86-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Go to https://staging.new.expensify.com/
Log in with any account.
On LHN, click on any Room and send a message.
Create a thread, don't comment.
Go back to LHN.

Expected Result:
The newly created thread does not appear.
Actual Result:
The newly created thread report remains in LHN if it is not commented on.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/115492554/c4962dc8-4e9d-4b89-abc5-6fc6ae6e4722

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~012abbff085af3a667
Upwork Job ID:<PHONE_NUMBER>457190912
Last Price Increase: 2023-10-18

Proposal
Please re-state the problem that we are trying to solve in this issue.
Empty thread on a message in room appears in LHN
What is the root cause of that problem?
THis here
https://github.com/Expensify/App/blob/e8cb124e83c9ce588c1fb4402827880839ed88e9/src/libs/ReportUtils.js#L3208-L3209
return false for chats which are policy rooms even though they are threads
What changes do you think we should make in order to solve the problem?
We should check if the thread is a policy room and also not a thread before returning false by changing this to
if (excludeEmptyChats && isEmptyChat && isChatReport(report) && !(isChatRoom(report) && !isThread(report)) && !isPolicyExpenseChat(report) && canHideReport) {
        return false;
    }

What alternative solutions did you explore? (Optional)

I've reproduced this too now

I've reproduced this too now

What's your platform? I can't reproduce this now (Chrome Version 117.0.5938.149 (Official Build) (arm64), macOS 14.0).

@dejavuwl Version 117.0.5938.149 (Official Build) (arm64), with v1.3.86-1 of New Expensify!

Hm so the idea of removing this check was because new thread reports should get created under the hidden notification preference for all users. So I thought we could make a backend change to ensure that it's hidden for the user that initiates the thread as well but I went back on that because of this PR which allows users to preemptively join threads which creates the thread. And these empty threads should not appear in the LHN. So we do need to bring the logic back.

Makes sense, thanks for the clarification. Let's bring it back then 👍

@parasharrajat Good question, agreed on discussing it here! I think that pinning the new thread should make it remain visible on LNH as pinned 👍

Payouts due:
Issue Reporter: N/A - Applause

[x] Contributor: $500 + 50% Urgency = $750 @dukenv0307 - Contract paid and ended

Contributor+: $500 + 50% Urgency = $750 @parasharrajat via Manual Request
Eligible for 50% #urgency bonus? Yes
Upwork job is here.

QA test created: https://github.com/Expensify/Expensify/issues/333491
Closing, thanks all

$750 approved for @parasharrajat


Some flutter libraries have unnecessary import directives

Details

Please tell us exactly how to reproduce the problem you are running into, and how you measured the performance.

Build a Dart SDK which can detect unnecessary imports. Use https://dart-review.googlesource.com/c/sdk/+/177221.

Analyze Flutter sources.

Witness many files which contain import directives whose used elements are provided by another import directive.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

No thank you.

Switch flutter to master channel and run this app on a physical device
using profile or release mode. Verify that the performance issue can be
reproduced there.

n/a

Target Platform: any
Target OS version/browser: any
Devices: any
Logs
n/a

Is this still an issue? If so I'd like to solve it

I think there are still a few lingering libraries which have unnecessary imports, but they're hard to detect today; I am using a local branch of Dart where I enable the analyzer to detect such code.
If you're inspired though, there is a similar issue which the Flutter developers may appreciate visibility into. There are some files in this repo with imports like this:
import 'dart:io';
// ...
import 'package:file/file.dart';

File f ...

Each of these two imported libraries provide a class named File, so which one is referenced in the library? It turns out there is a little-known rule in the language specification which says that any non-system (non-dart:) import elements shadow any would-be colliding system (dart:) imports, so the File from package:file wins. However, I'm fairly certain most developers would like this made explicit. That is, I think a change to hide File in the dart:io import would be an improvement. I think this goes for Directory as well.

@srawlins this might be unrelated, but i always wondered - if we have a single file to import package:app/app.dart that export all files that we potentially might need, and we import it in any place of our project, does it affect on something? like speed of compilation, speed of analyzer, whatever?


Question: reading arbitrary JSON objects

Please excuse another Newbie question - this time on the Read operation: how can I read an arbitrary JSON document?
Your TODO sample app and CrudSamples use a "flat" document structure, from which each element can be extracted  like this:
Map<String, Object> map = rev.getBody().asMap();
String type =  map.get("type"));

How do I get an object if it is not flat? My document is of the form:
"v": {
    "address": "00:02:5B:00:66:77",
    "endReminder": false,
    "startTime": 60
  }

map.get("v").toString() gives  this (note no quotes so can't be treated as JSON):
{address=00:02:5B:00:15:20, endReminder=false, startTime=60}
On the other hand, rev.getBody().asBytes() gives this:
{"v":{"address":"00:02:5B:00:15:20","endReminder":false,"startTime":60}}
from which I can use substring() to give me the following JSON string:
{"address":"00:02:5B:00:15:20","endReminder":false,"startTime":60}
which I can finally convert into a Java object (in this case using Gson) like this (where 'v' is the JSON string from the previous line):
 Gson gson = new Gson();
 ReminderRecord reminderRecord = gson.fromJson(v, ReminderRecord.class);

This works for me but feels inelegant (and my CouchDB document is still very simple). Is there a general way of obtaining Java objects from an arbitrarily complex JSON document body?

As you have seen the sync-android DocumentRevision gives you the option of retrieving the document body as the raw byte[] or as a Map<String, Object> (which is about the closest Java type to a JSON object without importing another library).
When using the Map<String, Object> form if the document is not flat you can just extract a value into another Map by casting e.g.
Map<String, Object> map = rev.getBody().asMap();
Map<String,Object> v =  (Map<String, Object>) map.get("v"));
String address = (String) v.get("address");

This requires you to know what types you are expecting or to do some instance checks or other introspection to determine the types. The advantage is that the Map allows for completely arbitrary JSON.
Alternatively use the byte[] with your preferred deserializer (I guess Gson). In that case you can deserialize into a model type as in your example. Note that you can use additional classes to represent nested JSON objects e.g.:
    class Doc {
        public ReminderRecord v;
    }
    class ReminderRecord {
        public String address;
        public boolean endReminder;
        public long startTime;
    }
Doc d = gson.fromJson(new String(rev.getBody().asBytes(), "UTF-8"), Doc.class);
ReminderRecord reminderRecord = d.v;

This approach ties in nicely with POJOs, but lacks flexibility in that it will only deserialize into the model fields, so isn't as flexible for arbitrary content (although you can deserialize into fields of "arbitrary" types like Map.
or you can deserialize into something like Gson's built-in JsonObject for arbitrary JSON e.g.
JsonObject doc = gson.fromJson(new String(rev.getBody().asBytes(), "UTF-8"), JsonObject.class);
JsonObject reminderRecord = doc.getAsJsonObject("v");

This is very similar to the Map<String, Object> but approach, but you can use Gson's utilities for doing the type checking and casting of the fields (e.g. getAsJsonObject).
Which route you choose will depend on the needs of your application, for example, whether you have a complete model that you can use or whether you need to use one-size-fits-all objects like Map or JsonObject.

Hope that helps.


KeyError raised when cyvcf2 loads FindSV vcfs

I'm getting a "KeyError" when cyvcf2 tries to fetch INFO-fields from a FindSV VCF. Known issue? Any additional refactor in progress around there or can I safely debug? ;-)
[2016-03-11 10:43:24,499] puzzle.server.factory     ERROR    Exception on /variants/P2760_102_head_5.vcf [GET]
Traceback (most recent call last):
...
 File "/Users/daniel/sandbox/puzzle/puzzle/server/blueprints/variants/templates/sv_variants.html", line 3, in top-level template code
    {% from "macros/filters.html" import filters_form %}
...
 File "/Users/daniel/sandbox/puzzle/puzzle/server/blueprints/variants/templates/sv_variants.html", line 38, in block "container"
    {% for variant in variants %}
 File "/Users/daniel/sandbox/puzzle/puzzle/plugins/vcf/mixins/variant_mixin.py", line 119, in variants
    case_obj=case_obj,
  File "/Users/daniel/sandbox/puzzle/puzzle/plugins/vcf/mixins/variant_mixin.py", line 232, in _format_variants
    info_dict = dict(variant.INFO)
  File "cyvcf2/cyvcf2.pyx", line 1271, in cyvcf2.cyvcf2.INFO.__next__ (cyvcf2/cyvcf2.c:26404)
    return name, self._getval(info)
File "cyvcf2/cyvcf2.pyx", line 1218, in cyvcf2.cyvcf2.INFO._getval (cyvcf2/cyvcf2.c:25476)
    raise KeyError
KeyError
[2016-03-11 10:43:24,503] werkzeug                  INFO     <IP_ADDRESS> - - [11/Mar/2016 10:43:24] "GET /variants/P2760_102_head_5.vcf HTTP/1.1" 500 -

Guess it has to do with inconsistent header but you should look closer.
We have a lot of wierd entries in our file but no complains yet...

Yep, there were a couple of inconsistencies in the INFO variable types from FT. Works on my minimal test now. Will pass the issue on.. :)

Good. Can we close?


Remove Microsoft Edge on Windows Server 2016

As far as I know, it's not possible to run/install Microsoft Edge on Windows Server 2016 so if that is true, this combination should not be listed under "Supported Browsers"

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

ID: 0073f5a8-daf6-7f14-127f-e0c9d80af78d
Version Independent ID: d6d0afdf-3619-ca03-718b-a40bfd07bab9
Content: Azure Active Directory Conditional Access settings reference
Content Source: articles/active-directory/conditional-access/technical-reference.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@MicrosoftGuyJFlo John, "Microsoft Edge" browser should be removed from the Windows Server 2016 and Windows Server 2019.

Updated in PR in private repo


TEP-0121: Implement Retries in TaskRun

Changes
This PR implements Retries in TaskRun, and removes the logic that PipelineRun controller relies on RetriesStatus to determine the termination of a TaskRun or CustomRun/Run.
Key Changes:

New Retries field in both v1beta1.TaskRun and v1.TaskRun
Archive retry attempt history in RetriesStatus for a failed TaskRun, before sending kubernetes and cloud events before a reconcile loop ends.
Unit Tests to test the TaskRun object changes, especially the changes on status.conditions and status.retriesStatus after being reconciled once (one reconcile loop).

fix #5755
fix #5248
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
API change: new `Retries` field in `v1beta1.TaskRun` and `v1.TaskRun`.

Thanks @XinruZhang and thanks for making the comprehensive changes for the respective conversions.

Closes #5248
@XinruZhang could you test the behavior of your PR against #4071 (comment)?

Yes it works as expected:

TaskRun hi retries once, and fails on TimeOut on the second attempt.
TaskRun bye succeeds.
PipelineRun fails on TimedOut.

apiVersion: tekton.dev/v1beta1
kind: TaskRun
metadata:
  creationTimestamp: "2022-11-29T16:38:48Z"
  generation: 2
  labels:
    app.kubernetes.io/managed-by: tekton-pipelines
    tekton.dev/memberOf: tasks
    tekton.dev/pipeline: please-say-bye-again
    tekton.dev/pipelineRun: please-say-bye-again-gpgb4
    tekton.dev/pipelineTask: hi
  name: please-say-bye-again-gpgb4-hi
  namespace: default
  ownerReferences:
  - apiVersion: tekton.dev/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: PipelineRun
    name: please-say-bye-again-gpgb4
    uid: ad62ddd9-eb41-4123-9b32-87aeee37d57d
  resourceVersion: "118235147"
  uid: 698a3ed1-90da-43d3-8753-e3950be1119f
spec:
  resources: {}
  retries: 2
  serviceAccountName: default
  status: TaskRunCancelled
  statusMessage: TaskRun cancelled as the PipelineRun it belongs to has timed out.
  taskSpec:
    steps:
    - image: alpine:3.12
      name: hi
      resources: {}
      script: |
        echo 'hi'
        sleep 10
        exit 1
  timeout: 1h0m0s
status:
  completionTime: "2022-11-29T16:39:07Z"
  conditions:
  - lastTransitionTime: "2022-11-29T16:39:07Z"
    message: TaskRun "please-say-bye-again-gpgb4-hi" was cancelled. TaskRun cancelled
      as the PipelineRun it belongs to has timed out.
    reason: TaskRunCancelled
    status: "False"
    type: Succeeded
  podName: please-say-bye-again-gpgb4-hi-pod-retry1
  retriesStatus:
  - completionTime: "2022-11-29T16:39:03Z"
    conditions:
    - lastTransitionTime: "2022-11-29T16:39:03Z"
      message: |
        "step-hi" exited with code 1 (image: "docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69"); for logs run: kubectl -n default logs please-say-bye-again-gpgb4-hi-pod -c step-hi
      reason: Failed
      status: "False"
      type: Succeeded
    podName: please-say-bye-again-gpgb4-hi-pod
    startTime: "2022-11-29T16:38:48Z"
    steps:
    - container: step-hi
      imageID: docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69
      name: hi
      terminated:
        containerID: containerd://4a4744d380847cde62a9118bb5f2ec22e88d83bd69831406a50024d6925afc6c
        exitCode: 1
        finishedAt: "2022-11-29T16:39:02Z"
        reason: Error
        startedAt: "2022-11-29T16:38:52Z"
    taskSpec:
      steps:
      - image: alpine:3.12
        name: hi
        resources: {}
        script: |
          echo 'hi'
          sleep 10
          exit 1
  startTime: "2022-11-29T16:39:03Z"
  steps:
  - container: step-hi
    imageID: docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69
    name: hi
    terminated:
      exitCode: 1
      finishedAt: "2022-11-29T16:39:07Z"
      reason: TaskRunCancelled
      startedAt: "2022-11-29T16:39:06Z"
  taskSpec:
    steps:
    - image: alpine:3.12
      name: hi
      resources: {}
      script: |
        echo 'hi'
        sleep 10
        exit 1

apiVersion: tekton.dev/v1beta1
kind: TaskRun
metadata:
  creationTimestamp: "2022-11-29T16:39:07Z"
  generation: 1
  labels:
    app.kubernetes.io/managed-by: tekton-pipelines
    tekton.dev/memberOf: finally
    tekton.dev/pipeline: please-say-bye-again
    tekton.dev/pipelineRun: please-say-bye-again-gpgb4
    tekton.dev/pipelineTask: bye
  name: please-say-bye-again-gpgb4-bye
  namespace: default
  ownerReferences:
  - apiVersion: tekton.dev/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: PipelineRun
    name: please-say-bye-again-gpgb4
    uid: ad62ddd9-eb41-4123-9b32-87aeee37d57d
  resourceVersion: "118235223"
  uid: 7c8564bf-6d8b-4491-a323-8620cb690bde
spec:
  resources: {}
  serviceAccountName: default
  taskSpec:
    steps:
    - image: alpine:3.12
      name: bye
      resources: {}
      script: |
        echo 'bye'
  timeout: 1h0m0s
status:
  completionTime: "2022-11-29T16:39:12Z"
  conditions:
  - lastTransitionTime: "2022-11-29T16:39:12Z"
    message: All Steps have completed executing
    reason: Succeeded
    status: "True"
    type: Succeeded
  podName: please-say-bye-again-gpgb4-bye-pod
  startTime: "2022-11-29T16:39:07Z"
  steps:
  - container: step-bye
    imageID: docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69
    name: bye
    terminated:
      containerID: containerd://f54ec9f73202fe0b7fea7dc58db92fec7ca4bccaf06bdccaaf0fe0cad3acaafe
      exitCode: 0
      finishedAt: "2022-11-29T16:39:11Z"
      reason: Completed
      startedAt: "2022-11-29T16:39:11Z"
  taskSpec:
    steps:
    - image: alpine:3.12
      name: bye
      resources: {}
      script: |
        echo 'bye'

apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:
  creationTimestamp: "2022-11-29T16:38:48Z"
  generateName: please-say-bye-again-
  generation: 1
  labels:
    tekton.dev/pipeline: please-say-bye-again
  name: please-say-bye-again-gpgb4
  namespace: default
  resourceVersion: "118235224"
  uid: ad62ddd9-eb41-4123-9b32-87aeee37d57d
spec:
  pipelineRef:
    name: please-say-bye-again
  serviceAccountName: default
  timeouts:
    pipeline: 1m0s
    tasks: 19s
status:
  completionTime: "2022-11-29T16:39:12Z"
  conditions:
  - lastTransitionTime: "2022-11-29T16:39:12Z"
    message: 'Tasks Completed: 2 (Failed: 1, Cancelled 0), Skipped: 0'
    reason: Failed
    status: "False"
    type: Succeeded
  finallyStartTime: "2022-11-29T16:39:07Z"
  pipelineSpec:
    finally:
    - name: bye
      taskSpec:
        metadata: {}
        spec: null
        steps:
        - image: alpine:3.12
          name: bye
          resources: {}
          script: |
            echo 'bye'
    tasks:
    - name: hi
      retries: 2
      taskSpec:
        metadata: {}
        spec: null
        steps:
        - image: alpine:3.12
          name: hi
          resources: {}
          script: |
            echo 'hi'
            sleep 10
            exit 1
  startTime: "2022-11-29T16:38:48Z"
  taskRuns:
    please-say-bye-again-gpgb4-bye:
      pipelineTaskName: bye
      status:
        completionTime: "2022-11-29T16:39:12Z"
        conditions:
        - lastTransitionTime: "2022-11-29T16:39:12Z"
          message: All Steps have completed executing
          reason: Succeeded
          status: "True"
          type: Succeeded
        podName: please-say-bye-again-gpgb4-bye-pod
        startTime: "2022-11-29T16:39:07Z"
        steps:
        - container: step-bye
          imageID: docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69
          name: bye
          terminated:
            containerID: containerd://f54ec9f73202fe0b7fea7dc58db92fec7ca4bccaf06bdccaaf0fe0cad3acaafe
            exitCode: 0
            finishedAt: "2022-11-29T16:39:11Z"
            reason: Completed
            startedAt: "2022-11-29T16:39:11Z"
        taskSpec:
          steps:
          - image: alpine:3.12
            name: bye
            resources: {}
            script: |
              echo 'bye'
    please-say-bye-again-gpgb4-hi:
      pipelineTaskName: hi
      status:
        completionTime: "2022-11-29T16:39:07Z"
        conditions:
        - lastTransitionTime: "2022-11-29T16:39:07Z"
          message: TaskRun "please-say-bye-again-gpgb4-hi" was cancelled. TaskRun
            cancelled as the PipelineRun it belongs to has timed out.
          reason: TaskRunCancelled
          status: "False"
          type: Succeeded
        podName: please-say-bye-again-gpgb4-hi-pod-retry1
        retriesStatus:
        - completionTime: "2022-11-29T16:39:03Z"
          conditions:
          - lastTransitionTime: "2022-11-29T16:39:03Z"
            message: |
              "step-hi" exited with code 1 (image: "docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69"); for logs run: kubectl -n default logs please-say-bye-again-gpgb4-hi-pod -c step-hi
            reason: Failed
            status: "False"
            type: Succeeded
          podName: please-say-bye-again-gpgb4-hi-pod
          startTime: "2022-11-29T16:38:48Z"
          steps:
          - container: step-hi
            imageID: docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69
            name: hi
            terminated:
              containerID: containerd://4a4744d380847cde62a9118bb5f2ec22e88d83bd69831406a50024d6925afc6c
              exitCode: 1
              finishedAt: "2022-11-29T16:39:02Z"
              reason: Error
              startedAt: "2022-11-29T16:38:52Z"
          taskSpec:
            steps:
            - image: alpine:3.12
              name: hi
              resources: {}
              script: |
                echo 'hi'
                sleep 10
                exit 1
        startTime: "2022-11-29T16:39:03Z"
        steps:
        - container: step-hi
          imageID: docker.io/library/alpine@sha256:c75ac27b49326926b803b9ed43bf088bc220d22556de1bc5f72d742c91398f69
          name: hi
          terminated:
            exitCode: 1
            finishedAt: "2022-11-29T16:39:07Z"
            reason: TaskRunCancelled
            startedAt: "2022-11-29T16:39:06Z"
        taskSpec:
          steps:
          - image: alpine:3.12
            name: hi
            resources: {}
            script: |
              echo 'hi'
              sleep 10
              exit 1

/retest

/retest

/retest
flake: TestReconcile/default_embedded_status

/hold
One thing we haven't fully investigated is Transient Errors. We wouldn't want to retry a TaskRun when transient errors detected. For example: waiting for the remote resource to be ready, waiting for volumes to be mounted. Thanks @pritidesai  for pointing this out!

/unhold
TL;DR: We do NOT and should NOT mark the resource as following for transient errors
Type: Succeeded
Reason: xx
Status: False

The transient errors in taskrun implementation: once tr.IsDone() is true, the controller stops reconciling:
https://github.com/tektoncd/pipeline/blob/8c59d00342e9479e68e9c52f6d18b62ac3015489/pkg/reconciler/taskrun/taskrun.go#L127
And if the TaskRun failed to retrieve the remote Task, the taskRun marks the resource as Ongoing:
https://github.com/tektoncd/pipeline/blob/8c59d00342e9479e68e9c52f6d18b62ac3015489/pkg/reconciler/taskrun/taskrun.go#L336
For reference, in PipelineRun, it marks the PipelineRun status as Running when there's remote resource to be resolved, it's worth mentioning that PipelineRun controller tries to resolve those resources on each reconcile loop:
https://github.com/tektoncd/pipeline/blob/8c59d00342e9479e68e9c52f6d18b62ac3015489/pkg/reconciler/pipelinerun/pipelinerun.go#L526-L535
That's said, we won't mark the resource as Done (Succeeded=False) for transient errors.

/retest
because of TestReconcile

Thanks @jerop! We've discussed offline, will break this PR into smaller pieces, and targeting to get all sub PRs merged before v0.43.0. Thanks!

Thank you for breaking it down @XinruZhang!
/hold

Closing this PR because it is separeted into two sub prs: https://github.com/tektoncd/pipeline/pull/5842 and https://github.com/tektoncd/pipeline/pull/5844

Closing this PR because it is separeted into two sub prs: https://github.com/tektoncd/pipeline/pull/5842 and https://github.com/tektoncd/pipeline/pull/5844


NPM release versions 4.X

Could you please publish the the 4.X releases of zombie to npm?

It's on npm, you can use npm info to find all published versions and install it, or install with the tag next.

Ah yes, I see it with npm view zombie versions.  I was going by the "3.1.1 is the latest of 116 releases" copy seen on https://www.npmjs.com/package/zombie.


ADDRESS STREET data is split amongst two columns

As you can see from the screenshot, it looks like part of the ADDRESS STREET is bleeding over into ADDRESS SUFFIX. Looking at the original source, it appears to exist in there unfortunately and so some cleaning will be required by the script.

I looked at this problem and I'm not sure I'd really be improving the data by applying a transformation to each row. It's true that the ADDRESS STREET field sometimes spills over into the ADDRESS SUFFIX field, but sometimes both fields are filled out correctly. At least currently, when there's a problem with a row, we know it's the result of a police officer putting data in the wrong field. If we transform each row, there will be multiple possible causes for errors:

The police officer made an error that our code didn't cover
The police officer filled out the form correctly, but our code made it wrong
The police officer made one error, and our code changed it to a different error

We don't want these address fields to go in the public dataset anyway. I think the solution is to pass the data we have for geocoding with a space between the contents of the ADDRESS STREET and ADDRESS SUFFIX fields, and if that fails we should pass them without a space. In either case, we might want to delete any hashmarks and numbers at the ends of those fields, because that information is commonly duplicated in the ADDRESS ALI field, and we don't need apartment numbers for geocoding anyway.

Great analysis and sounds like you have a solid path forward; let's go with your recommendations.


Port ME conduits to 1.10

Just port of ME conduits from master and adopt to 1.10.

Looks good, I'm impressed.
(Ignore the travis failures, our build just takes too long in 1.10.2)

@HenryLoenwind please check-in import's orders.


Broken Regressions Fixes

Description

Fix the currently broken regressions tests
Related Issue

N/A
Motivation and Context

Some of our regression tests that we rely on for testing Lean are broken, thankfully there are only a few and seem to be related to recent additions such as #5389 and #5379
Requires Documentation Change

N/A
How Has This Been Tested?

After these updates all regression pass as expected except two.
These two are BasicTemplateIndexAlgorithm and IndexOptionCallITMGreeksExpiryRegressionAlgorithm
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Hey @gsalaz98 , do you mind checking out the two remaining broken regressions?

BasicTemplateIndexAlgorithm
IndexOptionCallITMGreeksExpiryRegressionAlgorithm

I think they were broken on addition and I'm not sure how they should be acting 😄

Hey @gsalaz98 , do you mind checking out the two remaining broken regressions?
* `BasicTemplateIndexAlgorithm`

* `IndexOptionCallITMGreeksExpiryRegressionAlgorithm`

They were broken on addition and I'm not sure how they should be acting 😄

IndexOptionCallITMGreeksExpiryRegressionAlgorithm is expected to be broken for now, but I think I never got around to updating BasicTemplateIndexAlgorithm w/ the push for index options.


Serverinfo not working for some servers

Features in the sinfo command was the cause to this whenever it is committed in servers that has no features

Has been fixed


Adding nexus staging plugin

Allows jobs to add "closeRepository" or "closeAndPromoteRepository" to the task list.
https://github.com/Codearte/gradle-nexus-staging-plugin/

@ljacomet ping

@akomakom before merging this, could we make sure closeAndPromoteRepository is disabled? I do not want a release build just yet to be synced to maven central without some checks before.

@ljacomet I could add some code that will raise an error if closeAndPromoteRepository is on the task list - aside from that there is nothing in the plugin config to disable it.
Either way, the ony time closeAndPromoteRepository would be used is when it is explicitly passed on the command line, and only on hosts that have the credentials configured.

Some mess up with the rebase here - there is a not needed merge commit

Yup fighting with git on this one

... I guess there is always --force.
Anyway, I think that this plugin doesn't work with JDK1.6 anymore, so this may be throwaway.

Can't this be closed then?

Closing, I'll revisit this down the line.

@ljacomet do we still want to do this?  Now that ehcache3 requires JDK8, this plugin would work.  I also disabled auto promote on my branch.  I'll open a new one if this is useful.


ObjectDeletionNotification::_objectIdRef should be IsReadOnly==1

** Proposal to the 5G-xhaul call on 6th of July: **
UML and YANG are already corrected.
Existing implementations are basing on the corrected YANG.
Just closing the issue


HCN_E_NETWORK_ALREADY_EXISTS

[Enter feedback here]

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

ID: 140e34a7-d710-125c-2cc8-4cf0bde73862
Version Independent ID: 92d3242f-3503-e770-575d-909e2e0411a4
Content: HCN Error Code
Content Source: virtualization/api/hcn/Reference/HCNHResult.md
Product: virtualization
GitHub Login: @Keith-Mange
Microsoft Alias: kemange

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Access to built json data in /dist/assets/data on the client-side

Hello everyone,
I noticed that gridsome generates the underlying json data for the collection page / static queries under /dist/assets/data during build time.
Is there any way I can have access to this generated static json on the client-side?
My scenario:
/artist/:id
This is a straightforward one node fetch template
/artist/:id/featured
This is a template that uses the artists id and feeds it to a page-query which then searches which tracks they appeared on.
I want to load the json raw data from the "/artist/:id/featured" template on the client side using this.$fetch and make it available in the /artist:id template page as a list of tracks.
Currently, it only appears that you can access the fully generated html/vue/js builds, but I have no way to obtain the results of the underlying already-run query as json.
I also have no issues exposing this json data to the client side through the browser.
Thanks for this great piece of software!

Apologies my bad, I just discovered that $fetch naturally fetches access to the underlying page-query results in json, so my question does not make sense.
You can close this issue now.


datasets

Will you consider open-sourcing your training data?

Hi there!
Unfortunately open-sourcing our training data is not in our plans. We have however released several evaluation sets at https://github.com/revdotcom/speech-datasets that might be of some help.


Permission model perfection

Solution for #728
In Map fragment, it's showing "Insufficient permissions" snack bar when user denies but rather it should show a snackbar with action using shouldShowRequestPermissionRationale.

I tried to running connectedCheck and it's failing build with an exception.
testEventAPIResponse
Test failed to run to completion. Reason: 'Instrumentation run failed due to 'java.lang.NoSuchMethodError''. Check device logcat for details

ATM, reverting #725 should fix the builds, as the builds before that don't fail.

That's not the solution because we have to update the dependencies someday
in future. I have tried to figure out the issue but couldn't as I haven't
written instrumentation test cases since more than a year and lately
libraries have got huge updates!
On Sat, Oct 15, 2016, 6:24 PM Harshit Dwivedi<EMAIL_ADDRESS>wrote:

ATM, reverting #725
https://github.com/fossasia/open-event-android/pull/725 should fix the
builds, as the builds before that don't fail.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/pull/729#issuecomment-253982563,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2NEGpdoo16Dyu4ziKhg4DPGwvwCozZks5q0M0QgaJpZM4KWOcw
.

--
Paresh Mayani | Manager  | Google http://www.meetup.com/GDG-Ahmedabad/
Developers http://www.meetup.com/GDG-Ahmedabad/  Group Ahmedabad
http://www.meetup.com/GDG-Ahmedabad/
Technical Lead - Android | Lets Nurture http://www.letsnurture.com/
@pareshmayani https://twitter.com/pareshmayani | gplus.to/Paresh.Mayani |
Stackoverflow http://stackoverflow.com/users/379693/paresh-mayani | GitHub
https://github.com/PareshMayani

@PareshMayani I tried downgrading the dependencies to lower version and the same error was there.
It was gone when I reverted to the commit prior to that Pull Request.
So I think that the issue is not related with the dependency version.

@the-dagger  Reverting back is not always the solution. did you tried to look into the issue exactly?

Current coverage is 11.10% (diff: 0.00%)

No coverage report found for development at 80fe9d4.

Powered by Codecov. Last update 80fe9d4...ef4c02a

Thanks @the-dagger for merging it. Finally :)


Push start boxes config to spads and chobby

Needs to be broken down, not sure whatever we should just push directly to https://github.com/beyond-all-reason/spads_config_bar/blob/main/etc/mapBoxes.conf file, or have a new plugin for that as suggested by Yarbiz: https://discord.com/channels/549281623154229250/1093658550330470490/1094244266173276281
This also includes pushing starboxes to Chobby to LuaMenu/configs/gameConfig/byar/savedBoxes.dat

Current status: UI implementation in progress, near finalized. Not started on script for importing startboxes to rowy and then exporting them out.

This is now fully done


Copy/pasting of connected elements not working on Windows/Mac

Copying multiple datanodes or lines is fine, but if the elements are connected then copy/pasting doesn't work (only occurs on Windows).
Reproduce:

Open new pathway
Draw catalytic reaction template
Select whole reaction and try to copy/paste (works on Linux but not on Windows/Mac)

Error: Unable to paste pathway data
java.lang.NullPointerException?: null
java.lang.NullPointerException?
at org.pathvisio.core.model.PathwayElement?$MPoint.getAbsolute(PathwayElement?.java:351)
at org.pathvisio.core.model.PathwayElement?$MPoint.unlink(PathwayElement?.java:461)
at org.pathvisio.core.model.Pathway.forceRemove(Pathway.java:402)
at org.pathvisio.core.model.Pathway.remove(Pathway.java:388)
at org.pathvisio.core.model.Pathway.forceAddObject(Pathway.java:266)
at org.pathvisio.core.model.Pathway.add(Pathway.java:262)
at org.pathvisio.core.gui.PathwayTransferable?.getTransferData

Let's see if we can fix this for PV4...


Extract the global coverage runner and configurable saveCoverage.tmpl path

Extract the global coverage runner incase it needs to be run manually, like:

if (typeof runCoverageGlobal !== "undefined")
  runCoverageGlobal();

This is necessary before calling browser.refresh() or if there is any normal form posts to server.

Make the template file path configurable from the Gruntfile. Currently it looks at specific path: "resources/saveCoverage.tmpl"

Looks good to me. Thanks for the contribution and apologies for the slow response.


Adding screenshots to the README

It would be great if there were screenshots

Thanks for the interest! This project is still in a very early stage and I'm not working on it actively, but I will definitely add screenshots when there is more show.


Refactor encoding process of structHead and structKey

Codecov Report

Merging #224 (34caf07) into master (102554c) will decrease coverage by 0.50%.
The diff coverage is 99.10%.

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
- Coverage   84.09%   83.58%   -0.51%
==========================================
  Files          49       49
  Lines       23590    22811     -779
==========================================
- Hits        19837    19066     -771
+ Misses       2925     2916       -9
- Partials      828      829       +1


Simple/Basic tutorial

Hello,
Perhaps a very simple question despite having dug into the graphormer documentation: Assume given a csv training file containing tens of thousands of SMILES and their corresponding properties, say just 2 properties. For example the training .csv file will look like
SMILES, Property 1, Property 2
SMILE1, some_number11, some_number21
SMILE2, some_number12, some_number22
SMILE3, some_number13, some_number23
...

At this point, I only know that I will have to invoke the command fairseq-train  --user-dir path/to/graphormer with a lot of settings from the fairseq page (https://fairseq.readthedocs.io/en/latest/command_line_tools.html#fairseq-train) and Graphormer's documentation. On the website, I could just roughly make sense of the oc20 (Train a New Model) example that uses lmdb files. So for the very simple case mentioned above (1 huge training csv file, and a separate test csv file), how do I go about using Graphormer?
I would appreciate if someone could give some pointers or direct me to the correct destination in the documentation that I might have overlooked.
Thank you.
Best,
Jacey

We're working on making the custormer dataset more friendly to be used, and also prepare some related tutorials, please stay tune.

Save this variable.

Perhaps I have a question about this too since I have been struggling to make my training converge using my own dataset.
Could you please kindly confirm if the workflow detailed below is correct:

given a dataset similar to that above (SMILES, Property), first convert smiles into DGL using some package (DGL-LifeSci, etc.)
generate data split and identify indices/range of the train/valid/test
reconstruct the dataset as a list of tuples [(DGL1, property1), (DGL2, property2), ... ]

Thank you.


No '' needed after $(SolutionDir) as it already has it.

No '' needed after $(SolutionDir) as it already has it.
so it should be:
copy "$(SolutionDir)packages\Microsoft.Web.WebView<IP_ADDRESS>.149\build\native$(PlatformTarget)\WebView2Loader.*" "$(OutDir)"

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

ID: f1c3ae57-a8c8-4615-7dcf-855e006a4043
Version Independent ID: 39480ab3-700f-9151-cd72-a12aba59c63d
Content: Microsoft Edge WebView 2 for Win32 apps - Microsoft Edge Development
Content Source: microsoft-edge/hosting/webview2/GettingStarted.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Thanks for reporting this! I've lumped a fix for this in another PR. https://github.com/MicrosoftDocs/edge-developer/pull/268

Resolved by https://github.com/MicrosoftDocs/edge-developer/pull/268